[0.3.0] Toctree caption styling in version 0.3.0. Serif vs SanSer

Problem
With the new version of the theme, the toctree captions are not properly styled. A serifed font is used instead of the usual sans-serif font.
See the screenshot here:

Another issue (which is not new in 0.3) is that the toctree captions are almost as dark as the background color in the sidebar:

A project where you can see these issues is xtensor: http://xtensor.readthedocs.io/en/latest/
Reproducible Project
.. toctree::
   :caption: INSTALLATION
   :maxdepth: 2

Expected Results
I expect the toctree caption to be in a sans-serif font like the rest of the theme, and like in earlier versions of the theme.
I would prefer the toctree caption color to be a bit brighter in the sidebar for readability, as per screenshot above (cc @ellisonbg).
Environment Info

Python Version: Any
Sphinx Version: Any
RTD Theme Version: 0.3

Ok thanks. FYI The issue of the color in the sidebar has come up in discussions about the documentation for Project Jupyter in a few occasions.


Idea for overhaul

I've kept saying in several places that we may need to re-design this library from scratch.

https://github.com/jotaijs/jotai-tanstack-query/issues/32#issuecomment-1523619697
https://github.com/jotaijs/jotai-tanstack-query/pull/38#discussion_r1317912221
https://github.com/jotaijs/jotai-tanstack-query/pull/40#issuecomment-1710943917

A bit of context: https://blog.axlight.com/posts/you-might-not-need-react-query-for-jotai/

I had been struggling with the mismatch, and ended up with creating a new library jotai-tanstack-query.

I thought, const [dataAtom, statusAtom] = atomsWithQuery(...) was a nice hack, but it's not nice in DX. The library impl is also complicated.

The mismatch is still there. We have several issues reported. In fact, jotai-urql is overhauled due to the same reason.
So, the general idea is to re-design the binding api fully from the TanStack Query perspective.
I can help from the Jotai perspective, but we need someone with TanStack Query background to maintain this project.

To add some feedback on DX for a newcoming :

indeed the 2-atoms-returned logic seems strange, and not really aligned with any jotai usual logic
even more misleading : the fact both atoms are actually writeable atoms, which basically blew my mind (what are the setters actually supposed to do ?)
The typing is broken, and you inevitably encounter this one and have to take 15 minutes to find this issue to get a workaround
I personally hard tried to go with Suspense and ErrorBoundary (a hacky react-native port of it) without any actual success. Rolled back to the statusAtom
statusAtom typing is not correctly discriminated (might be a react-query trouble though); such as isSuccess and isError are not discriminating data


Add metric from pathfinder to pre-heat NUTS

We are by default using pathfinder.jl to pre-heat our NUTS sampling by providing an (approximate) sample from the posterior distribution as the initial point for NUTS warm-up, as per here.
The reason we didn't also include an estimate of the underlying kinetic energy matrix/inverse curvature metric was because of the API to Turing not being straightforward.
As of https://turinglang.org/docs/tutorials/docs-16-using-turing-external-samplers/ this appears to be fixed/made easy to do, and so we should have metric pre-heating.

Unfortunately, AdvancedHMC.NUTS doesn't have an adtype option because its designed to interface directly with a LogDensityProblemAD.ADgradient object... which means that we can't direct port across.
I've flagged this with Turing https://github.com/TuringLang/Turing.jl/issues/2248

The direction of travel here:
https://github.com/TuringLang/Turing.jl/issues/2248#issuecomment-2148237328

This can go forwards after #262

Update: f2f chat with @dylanhmorris and @athowes (don't have his git handle) as part of STF meeting included discussion around when sometimes Pathfinder doesn't improve NUTS pre-heating.
Some experimentation on the associated branch to this issue is having mixed results with including the metric in pre-heat; I'm de-prioritizing investigating this for now but would like to keep this issue open.


In MySQL 5.7.x JSON Type record insert failure not seen in MariaDB 10.3

3140 (22032): Invalid JSON text: "Missing a name for object member." at position 1 in value for column 'Galaxies_2.json_record'.
https://travis-ci.org/planckbit/DataFleaker/builds/558641320

#For Mysql 5.7 and greater needed double quotes for it to be accepted as a json_type.
This issue is not the case for MariaDB. This fix works for both.
strQuery = "INSERT INTO "+collectionTable+"(json_record) VALUES(""+json.dumps(records).replace(""","""")+"")"


Multiple interfaces and servers?

Hi!
So it looks like we've got our configuration now running very stably, very swimmingly - but it LOOKS like, from the interface, I can create AND manage multiple WireGuard servers/interfaces. Trouble is, it doesn't seem like I can do that - I've created wg1 and wg2 interfaces with the intent of segregating users and their access to our internal resources, but I don't get any kind of a management UI below the wg0 one. I just get an error: "The server interface or ip wg1 already exists in the database".
EDIT 2019 on 12 February 2021:
When I try, I get nothing in the Gunicorn error log, and the following in the access log:
<IP_ADDRESS>:0 - "POST /api/v1/server/add HTTP/1.1" 400

I figured this out:
I don't exactly know what was going on, but I figured that since @perara never mentioned the ELRepo's pre-built Wireguard binary (kmod-wireguard in CentOS 7-speak) in the README, that I'd give either wireguard-dkms or the kernel module a try.
I opted for the kernel module, blew up my database (rm -rf /opt/wg-manager/wg_dashboard_backend/database.db), restarted my wg-manager service, and gave it a whirl. It now seems to work great, at the very least, multiple server interfaces do appear to be showing up. I will be doing some further testing on this, and I plan to add some comments later on as to some good iptables rules.
Shout-out to @perara - you've definitely got the lead contender for web-based WireGuard management interfaces thus far. It's a little finicky to set up, and if I could ask for one thing, it'd be some more verbose logging of what's going on on the inside of the backend there, but otherwise this is excellent. I would be delighted to help the community in any way that I can - I'm not much of a Python coder, but I can learn, but my real talent is in documentation! I'd be happy to draw up another bare-metal install guide! By golly we'll get this thing up and working for our obtuse network yet!

Hmm. I seem to be having trouble creating a third interface.

Same issue.
I imported server wg0.conf configuration of other servers, and also add keys, the "add server" button is active, but there is no response after click.

I will update this to add that, yes, updating the database manually via the command line or through a database manager such as DBeaver (I was actually doing it via the command line which was needlessly arduous but I guess I'm a sucker for punishment) works just fine, and we've cleaned up our iptables and hardware firewall issues enough to allow connectivity to most all of our production environment through WireGuard.
This DOES work, but it is pretty unclean.

Hi,
Sorry for being on the inactive side of things these days. Im up to my ears in finishing up my PhD thesis so hobby projects like this gets, lets say down priorized..
However  i keep an eye out for any contributions.
I agree on the point of logging, currently theres is very little of that going around.
Ill see if i manage to reproduce the bug with multiple interfaces. Generally i have not seen this behavior, but surely this is probably something ive missed.
In general, i hope to get up unit tests to kerp stuff like this working. Some of the stuff internally is, to put it nicely, garbage, so eventually I hope to better integrate with the wg api

Very cool find, @mvargasevans, I should've caught that in my DBeavering around the database, but I absolutely never did. Been running this for a little over a year in production, though, and it's been very stable and has handled updates exceptionally well, and it does a really nice job for us. I'm working on a million other projects but, hopefully, as things cool down at work, maybe I could get my hands dirty and look at some of the code - this seems like a fun and educational project!

Hi, im actually working on improving some of these issues that had been reported recently.
The largest change is towards better Wg integration with creating devices manually and setting up iptwbles manually.
Also fixing quirks like this..
Ill push the update to dev when its ready, however, i need to ensure that i dont break everything:)

Is this multi-interface handling fixed?
I would need to create 100+ of them, and +1 master-group for myself to reach all clients of all the sub-groups.
(to reach small organisations = pizzerias)

Is this manager suitable for that ?

Thanks!


Identation Issues When copy/pasting multiline statements into ipython or python shell

Environment
Windows build number: Microsoft Windows [Version 10.0.19041.388]
Windows Terminal version (if applicable): Windows Terminal Preview Version: 1.2.2022.0
WSL 2 Ubuntu 20.04
Python 3.7.8
ipython 7.17.0

Steps to reproduce
Copy some python code like:
v = 1
a = 2
c = 3

def abc():
    a = 1
    b = 2

And paste into ipython (python shell shows a similar behavior but it's harder to show the issue)
Expected behavior
This is what I would get if doing the same in a ubuntu installation
In [1]: v = 1
   ...: a = 2
   ...: c = 3
   ...:
   ...: def abc():
   ...:    a = 1
   ...:    b = 2

Actual behavior
In [3]: v = 1

In [4]: a = 2

In [5]: c = 3

In [6]:

In [6]: def abc():
   ...:         a = 1
   ...:             b = 2

Each statement goes to a separate line (which gets ran), and the sttm within the function are wrongly indented.
I guess both share the same cause.
Now I don't know if this is a Windows Terminal bug per se or something on the python side of things, since if I do the same and paste into vim, the indentation is fine.

Related: https://github.com/microsoft/Terminal/issues/395#issuecomment-602254180

This is definitely a /dup of #395. There's a special input mode that lets an application know when text is being pasted so it can do special things like showing you all the commands before they're committed.


Add link to web translation tool

... and a new gitignore entry
@helloworld1 I'll create pulls from time to time with updates from Crowdin if there are some
If you like admin access to the crowdin project just write to me :)

Thanks for setting up Crodwin. It looks pretty nice!

@helloworld1 updated

@helloworld1 updated


[ko] Translate docs/reference/labels-annotations-taints to Korean

This is a Feature Request

What would you like to be added

Translate docs/reference/labels-annotations-taints to Korean

https://kubernetes.io/docs/reference/labels-annotations-taints/
https://github.com/kubernetes/website/blob/master/content/en/docs/reference/labels-annotations-taints.md

Why is this needed
Because there is no Korean-translated version of that page.
Comments

/language ko
/assign

/triage accepted

This issue was resolved by the PR #28479.


Test for STM live-lock

#486 reported about a STM live-lock
a test should surface what are the circumstances that this can occur.
Some suggestions were made:

setupTrace but not shutdown
exception in the dispatcher or enqueuing function
may use 'Async.linkOnly' instead of 'Async.link'

Fix merged.


Create table pay per request

Notes
Defining a table where you do not specify read or write capacity, should create a table with 'pay per request' pricing.
Public API changes
Fixed

Tables can now be created with "pay per request" billing mode if read_capacity_units or write_capacity_units are not set.

Thank you for your contribution.
So, if provisioned throughput is not set, we always should use PAY_PER_REQUEST

🙏🏼


Value representation overhaul

This is part of https://github.com/stellar/rs-soroban-env/issues/679,  it is this repo's parallel change to https://github.com/stellar/stellar-xdr/pull/70 -- see that PR for details.

I think this PR needs to be updated after https://github.com/stellar/stellar-xdr/pull/70 is merged.


Wrong max age when set?

I build my cookies like below:
let xsrf_cookie =
                    Cookie::build(("xsrf_token", access_token_details.token_uuid.to_string()))
                        .domain(&state.config.host)
                        .path("/")
                        .secure(true)
                        .max_age(Duration::seconds(state.config.access_token_max_age * 60))
                        .http_only(false);

It sets max-age at the end in the cookie followed by a trailing = after that making  the time invalid.
The cookie value looks like set-cookie 	xsrf_token=a9fe33ce-4ba0-4401-b4ce-f44174d9d7a2; Secure; Path=/; Domain=localhost; Max-Age=300=

Never mind, it was coming from elsewhere.


Fix flash of unstyled content

We have the option to render the styles on the server. We should.

Ok, I'm going to make a bunch of comments in a bunch of comments:

I'm not sure I love the mix of concerns that @asyncConnect has in app/about/About.js. It should just make sure things resolve, there should probably still be an @connect there (just like app/todo/Todos.js). (That way it is easy to implement some form of caching and canceling of the request later).

I'm unconvinced that putting everything into app  is the right nove. Maybe having it as they originally did with app/shared, app/client, and app/server is right. I just don't like putting things into helpers that are not intended to ever be used on the client.

 It is seriously dumb. If we are using karma + jsdom anyways, can we just pull it all back in similar to how I had it before with jsdom-global and without karma?

Also, it looks like the SSR of the styles never happened because you moved src/server/Html/index.js to src/server/html.js and I added it to the original file. Merge conflicts are magic.

Also, it looks like this doesn't use actions for fetching the data, instead it is defining how to fetch the data inside the component file. I am going to pull that back out as well.

Close #11


Is swagger.json regenerated on every server restart?

Thinking about performance. Say hundreds of large entities in production, or even locally for "nodemon" integration test runs where every change restarts the server.
I know it's likely very fast anyway, but here is an idea:
From Discord
[4:27 AM] Marsup: Last time I read the code, it was cached in memory, so it's lost on every restart, but it doesn't seem to cost that much
[4:35 AM] Marsup: Oh, it may be able to hook into hapi's caching mechanism, so maybe there's a way to persist that, it's just not documented

@icetbr please see https://github.com/hapi-swagger/hapi-swagger/blob/7a0b7d5f2c7d4d7ac52a096e6339ed638f3b457a/usageguide.md#caching and https://github.com/hapi-swagger/hapi-swagger/blob/7a0b7d5f2c7d4d7ac52a096e6339ed638f3b457a/test/Integration/cache-test.js#L34 to see if that helps. Server restarts would clear cache.

Just now learning about all the caching options from Hapi. Catbox has a lot of plugins! This one would save to disk if I wanted to
https://www.npmjs.com/package/catbox-fs
If swagger uses hapi cache mechanism, it might be very simple to prevent regeneration if I desire.
Thanks for pointing out the docs. I had read it before, but I didn't know how Hapi cache worked.


SPARK-10883: use relative location of scalastyle-config.xml file in o…

…rder to be able to build individual module

Can one of the admins verify this patch?

Test build #1832 has started for   PR 8949 at commit ad6a30f.

Test build #1832 has finished for   PR 8949 at commit ad6a30f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Per comments in the JIRA, I think this is unnecessary as you can use the standard Maven syntax to build modules correctly. Do you mind closing this PR?

@jbonofre, could you please either address @srowen's comments or close this PR? I would close it myself but I do not have the required GitHub repo permissions. Thanks!


OLS-688: install ols using operator for e2e

Description
attempting to refactor the e2e ols installer logic to install using the latest OLS
operator instead of manually creating resources
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library
[ ] Bump-up library or tool used for development (does not change the final image)

Related Tickets & Documents

Related Issue #
Closes #

Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] PR has passed all pre-merge test jobs.
[ ] If it is a core feature, I have added thorough tests.

Testing

Please provide detailed steps to perform tests related to this code change.
How were the fix/results from this change verified? Please provide relevant screenshots or results.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (main@de7285a). Learn more about missing BASE report.
Report is 25 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1124   +/-   ##
=======================================
  Coverage        ?   95.44%
=======================================
  Files           ?       56
  Lines           ?     2371
  Branches        ?        0
=======================================
  Hits            ?     2263
  Misses          ?      108
  Partials        ?        0

/retest

@asamal4 ptal at the watsonx.test_api: test_model_evaluation test failure in this job run:
https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_lightspeed-service/1124/pull-ci-openshift-lightspeed-service-main-e2e-ols-cluster/1808989305238982656
based on the overall logs:
https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/test-platform-results/pr-logs/pull/openshift_lightspeed-service/1124/pull-ci-openshift-lightspeed-service-main-e2e-ols-cluster/1808989305238982656/build-log.txt
It looks like the test is evaluating all the model/provider pairs, even though the config is specifically watsonx/granite:
evidence:

in the test failure output "bam" appears even though the provider is watsonx:

question = 'what is kubernetes?', provider = 'bam'

the eval args seen in the overall log show the provider+model set to watsonx/granite as expected:

Response evaluation arguments: Namespace(keyword='', markexpr='', maxfail=0, continue_on_collection_errors=False, confcutdir=None, noconftest=False, keepduplicates=False, collect_in_virtualenv=False, importmode='prepend', basetemp=None, durations=0, durations_min=0.005, showfixtures=False, show_fixtures_per_test=False, version=0, plugins=[], traceconfig=False, verbose=0, no_header=False, no_summary=False, reportchars='fE', disable_warnings=False, showlocals=False, tbstyle='auto', showcapture='all', fulltrace=False, color='auto', code_highlight='yes', capture='no', runxfail=False, pastebin=None, assertmode='rewrite', xmlpath='/logs/artifacts/junit_e2e_watsonx.xml', junitprefix='watsonx', doctestmodules=False, doctestreport='udiff', doctestglob=[], doctest_ignore_import_errors=False, doctest_continue_on_failure=False, last_failed_no_failures='all', stepwise=False, stepwise_skip=False, logger_disable=[], asyncio_mode=None, cov_source=[], cov_report={}, cov_config='.coveragerc', no_cov_on_fail=False, no_cov=False, cov_append=False, cov_branch=None, eval_provider='watsonx', eval_model='ibm/granite-13b-chat-v2', eval_out_dir='/logs/artifacts', eval_query_ids=None, eval_scenario='with_rag'

that same log shows this output right after the eval args:  Model evaluation for bam+ibm/granite-13b-chat-v2

in fact that is the only output regarding model evaluation that i see in the entire build log (maybe because it on'y failed when running against watsonx, so we only captured the output for that run?  i'm not sure, pytest does some weird things to stdout unfortuantely).
it looks like maybe the messages themselves are misleading about what is actually being done (it is just gathering the test questions/expected responses for bam and the other models, vs actually attempting evaluation for bam), so maybe all that's needed is some cleanup/clearer messages, but it's definitely confusing right now.

Only scenario, where we will attempt to call api with different provider/model for model_evaluation is when one of the earlier test suite failed for response_evaluation and didn't create the CSV file.. And we will attempt to call API again, but it will fail as OLS is not installed with all provider/model.

@asamal4 thanks for the explanation.  debating what, if anything, we could do to improve the debuggability of this behavior.  It seems like if we know the provider/model is not configured/enabled, we could throw a cleaner error that basically explains what you just explained to me.  ("earlier invocations of the provider/model failed, so we don't have response data to compare to, current suite invocation does not include the provider/model configuration so we cannot retry the invocation")
but in any case i understand the observed behavior now, thank you

@xrajesh @onmete in the latest run the only failure was for transcript storage checking, specifically the test expected to find rag chunks in the transcript and did not:
https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_lightspeed-service/1124/pull-ci-openshift-lightspeed-service-main-e2e-ols-cluster/1809262114733297664
        assert transcript["metadata"]  # just check if it is not empty
        assert transcript["redacted_query"] == "what is kubernetes?"
        # we don't want llm response influence this test
        assert "query_is_valid" in transcript
        assert "llm_response" in transcript
        assert "rag_chunks" in transcript
        assert isinstance(transcript["rag_chunks"], list)
>       assert len(transcript["rag_chunks"])
E       assert 0
E        +  where 0 = len([])

tests/e2e/test_api.py:1179: AssertionError}

i don't think that has to do w/ my changes here, but might indicate some flakiness around whether or not we consistently determine that there are sufficiently high enough scoring chunks to include them, with the query that's being used in this test case?
Either that or something went wrong actually storing the RAG metadata w/ the transcript.
/retest

/retest

/retest

/retest

@xrajesh @tisnik ok i think this is ready.  Assuming you agree w/ the general concept, you can merge this as is or you're welcome to cherry-pick it into another PR if you want to make significant changes prior to merging.
Note that this currently uses the catalog/operator image from quay.io which means it's only as current as the last mirror time.  An optimization would be to use the CI catalog+operator image (built from the last merge to the operator repo), which would be a more current image than waiting for the mirror.  But as mirroring happens hourly, it should not be a huge issue unless you are trying to coordinate changes to the operator+operand.
The primary benefit to this change is that the e2e tests OLS as it would be installed/configured by the operator (modulo the modifications that the install/setup logic does such as changing the data collector sidecar container for stage operation), so we know that we're using all the same RBAC rules, operand deployment definition, etc.
it also ensures that the config yaml OLS is consuming is one that was generated by the operator, based on the config CRD.

/assign @xrajesh

@tisnik @xrajesh bump (if you guys are busy w/ release stuff/other priorities no worries, please just give me an ack that you've seen this and will get to it later)

/lgtm

/retest

Thank you @bparees . Sorry - was really focused on the release out . We did it ! Thank you @bparees
This PR -  other than quay.io mirror may be using previous image - The catalog image itself needs to be built after the bundle is built in a PROW run. Or you may be using the catalog which points to the previous version of bundle.
Keep this concern away - I think we will merge this PR and update it - by installing the bundle rather than the catalog.
Ben FYI - Even in Konflux release , we have 2 pipelines (1) Releases , operand and bundle container image (2) adds the bundle to public RH operator index (its a container ) . If you really look at it ,as long as we are consistently testing using the bundle - that is good , thats what is shipped to customers.

/retest

/retest

/retest

/approve

/override "Red Hat Konflux / ols-e2e-tests-rosa / lightspeed-service"

/override "Red Hat Konflux / ols-enterprise-contract / lightspeed-service"

/test e2e-ols-cluster-4-15


AOT can't work out with lazy loading component

I'm using angular2-router-loader to lazy loading component
here is some code:
`const fromListRoutes: Routes = [
{
path: 'form-list',
loadChildren: '../form-list/form-list.module#FormListModule'
}
];
const appRoutes: Routes = [
{ path: '', component: HomeComponent },
{ path: 'home', component: HomeComponent },
{ path: 'about', component: AboutComponent },
...fromListRoutes,
{ path: '**', redirectTo: '/home', pathMatch: 'full' },
];
export const routing = RouterModule.forRoot(appRoutes);`
then AOT operation:
npm run ngc
got error message:

Error: Cannot determine the module for class FormListComponent

The  angular2-router-loader component might not be AOT conform and cannot be used in production. Need to check though. A lot of Angular 2 third party components are not suitable for production builds...

I see. do you have options, for lazy loading component, one of components have dynamic-reactive from.
Something like this:

            Address {{i + 1}}
             1" (click)="removeAddress(i)">

            <address [group]="address" [addressError] = "addressErrors[i]"></address>

When I run

npm run start

got this:

I don't know it's a issue of angular2-router-loader or webpack itself?

Could be a angular problem, Versions 2.3.0, 2.3.1 and 4.beta don't work!
Use Angular 2.4.0. If the  angular2-router-loader is using one of these broken versions, it won't work
Greetings Damien

I am using angluar2  2.4
but angluar router loader still won't work.

How was  angular2-router-loader built?

through npm i angular2-router-loader -D
and

{
test: /.ts$/,
loaders: [
'awesome-typescript-loader',
'angular2-template-loader',
'source-map-loader',
'angular2-router-loader'
]
}

here is my source code.
except the dynamic reactive from, the others in sub route works fine.
so odd.

Hi damienbod
I think it's a issue of angular2. I just put dynamic reactive form of my code into angular2byexample
and gulp play, still won't work.

hi Daminebod
I know what's wrong in my code. I forget add addresscomponet in module.
But the error messages is not very friendly.

Woah, The error message gives you no indication of this.

hi damienbod
something like this:

no error, just no result in console

https://github.com/damienbod/Angular2WebpackVisualStudio/blob/master/LAZY_LOADING.md


RectangleHandleCap doesn't exist.

UnityEditor.Handles' does not contain a definition for RectangleHandleCap' Error in Unity 5.4

Use 5.6 instead, I am not supporting earlier versions.  You can check an earlier revision to see how it was handled before.


Replace Ordering/Comparator chaining with a specific implementation

Summary of the changes / Why this improves CrateDB
This replaces the ordering chaining using nullsFirst / reversed with
an implementation that handles both reverse and nullsFirst flag
directly.
In addition, it removes the implicit reversed that we had in place for
priority queues. This sometimes resulted in double reversing which is
confusing and overhead.
Main motivation is to reduce the call stack for ordering operations and
to have a better basis to integrate expression evaluation into ordering
later on.
Checklist

[x] User relevant changes are recorded in CHANGES.txt
[x] Touched code is covered by tests
[x] Documentation has been updated if necessary
[x] CLA is signed
[x] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

Looks like it doesn't have a real performance impact:
Benchmark                      Mode  Cnt  Score   Error  Units
compareNullAwareComparator     avgt   25  6.329 ± 0.046  ns/op
compareStackedGuavaOrdering    avgt   25  6.641 ± 0.052  ns/op
compareUsingComparing          avgt   25  6.584 ± 0.097  ns/op
compareUsingLambdaCompareImpl  avgt   25  6.109 ± 0.333  ns/op

    private Comparator<Row> legacyCompare;
    private NullAwareComparator<Row> newCompare;
    private Comparator<Row> comparing;
    private Comparator<Row> directcmp;
    private Row val1;
    private Row val2;

    @Setup
    public void prepareData() {
        Random rnd = new Random();
        legacyCompare = OrderingByPosition.rowOrdering(0, false, false);
        newCompare = new NullAwareComparator<>(row -> (Comparable) row.get(0), false, false);
        comparing = Comparator.comparing(row -> (Comparable) row.get(0));
        directcmp = (o1, o2) -> ((Comparable) o1.get(0)).compareTo(o2.get(0));

        val1 = new Row1(rnd.nextInt());
        val2 = new Row1(rnd.nextInt());
    }

    @Benchmark
    public int compareStackedGuavaOrdering() {
        return legacyCompare.compare(val1, val2);
    }

    @Benchmark
    public int compareNullAwareComparator() {
        return newCompare.compare(val1, val2);
    }

    @Benchmark
    public int compareUsingComparing() {
        return comparing.compare(val1, val2);
    }

    @Benchmark
    public int compareUsingLambdaCompareImpl() {
        return directcmp.compare(val1, val2);
    }

But I'd still like to get it in for the cleanup of the reverse handling + to have this Function<T, Comparable<Object>> keyExtractor in place for later re-use in my window function PR.

ok to test


Update Triggers

Added knowledge champion/knowledge champions

that's a good one! thanks for the PR @thomasward1212
published as v1.5.13


Flex container-fluid inside navbar

Operating system and version: Windows 10
Browser and version: Chrome 63.0.3239.84
Bootstrap version: 4.0.0-beta.2
Reduced test cases and potential fixes using CodePen

Create .container-fluid block inside .navbar block
Add .row and two or more columns inside navbar
The columns get placed one under one.

Adding .d-block class for .container-fluid block resolves the problem. But I think initially .container-fluid inside .navbar shouldn't be display: flex.

The navbar wasn't intended to have grid classes within, which is why the containers are set to a display: flex, but that's beside the point. When you have a flex container (anything with display: flex), the elements within that will be flex children. Unless a width is specified, those flex children will collapse. Add .w-100 to the .row if you need that particular markup.


Create CMake Macro for Targets

Looks like the template to collect sources and build a firmware image is gonna be extremely cookie cutter. This can probably be made into a macro. Then the top level CMakeLists.txt file will be fairly clean and we won't have to get crazy with sub dir files.

addressed by #10


[Skype For Business]Error: Failed to start conversation: no conversation ID returned issue.

SDK Platform: Node.js

Active Channels: Skype For Business

Deployment Environment: Auzure Bot Service

Issue Description
It shows below exception:
Error: Failed to start conversation: no conversation ID returned. and
ChatConnector: startConversation - error starting conversation.
Error: Request to 'https://api.skypeforbusiness.com/platformservice/botframework/v3/conversations' failed: [500] Internal Server Error at Request._callback
Code Example
module.exports = function (context, myTimer) {
const builder = require('botbuilder');
const GraphService = require('graph-service');
const appInsights = require("applicationinsights");
const tenant = process.env['tenant'];
const clientId = process.env['clientId'];
const clientSecret = process.env['clientSecret']
const api = new GraphService(tenant, clientId, clientSecret);
appInsights.setup()
    .setAutoDependencyCorrelation(false)
    .setAutoCollectRequests(true)
    .setAutoCollectPerformance(true)
    .setAutoCollectExceptions(true)
    .setAutoCollectDependencies(true)
    .start();

let client = appInsights.defaultClient;

const sendProactiveMessage = (addressToInitiate) => {

    var addr = {
        id:"",
        channelId:"skypeforbusiness",
        user:{
            id:"sip:emaiId"
        },
        channelData: {
                tenant: {
                    id: process.env['tenant']
                }
            },
        bot:
        {
            id: process.env['botId'],
            name: process.env['botName']
        },
        serviceUrl:process.env['serviceUrl']
    };

    let msg = new builder.Message().address(addr).text("Hello");
    bot.send(msg, function(err, address){
        context.log(err);
        context.log(address); //Address is showing null here.
    });
    context.log("Message sent to: " + addr.user.id);
}

var connector = new builder.ChatConnector({
    appId: process.env['appId'],
    appPassword: process.env['appPassword']
});

var bot = new builder.UniversalBot(connector);
var addressesToInitiate = ["sip:emailId"];

addressesToInitiate.forEach(function (addressToInitiate) {
    sendProactiveMessage(addressToInitiate);
});
context.done();

};
Expected Behavior
We are expected that   bot.send() give address as response and in address we required conversation Id.

Hi,
I am still waiting workaround regarding same
could you please help me for my query as i mentioned above
Regards,
Amit Gobare.

@amitgobare23 You have constructed an address object manually, but that won't work... you need to get the address information from the context.conversationReference object from an actual conversation with a real user.
If your bot has not contacted the end user at least once, and save the address info somehow, then the bot will not have the correct address information to be able to send the proactive message.
Please refer to this documentation on sending proactive messages:
https://docs.microsoft.com/en-us/bot-framework/nodejs/bot-builder-nodejs-proactive-messages
You can also find a working code example for reference here:
https://github.com/Microsoft/BotBuilder-Samples/tree/master/Node/core-proactiveMessages

For skype the conversation id remains same for one particular user but in case of skype for business the conversation id changes the number of times you initiate chat with user. So how can a bot be a ble to send proactive message.
Ideally bot should be able to send the proactive message based on sip address.


Fix typos in VisualElement Behaviors property

"Behavior" -> "Behaviors"
"propery" -> "property"
Fixes #175

Docs Build status updates of commit 8faef92:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/Xamarin.Forms/VisualElement.xml
:white_check_mark:Succeeded

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


fix: prevent marker flickering

This PR closes #17 by only removing markers when the component is destroy, and changing options when options change (rather than re-rendering)

Thanks for your contribution @dblythy :pray:

Thanks for the awesome work on this project!!


Why doesn't best.pth File Automatically Generate?

I'm encountering an issue where the best.pth file does not automatically generate as expected in my project.
Despite following the usual procedures and ensuring my configurations are correctly set, this file, which should represent the best model state based on validation performance, is not being created.
Could there be a specific reason for this, or is there a step I might be missing?
Any insights or suggestions on how to troubleshoot this problem would be greatly appreciated.

Additionally, I've noticed that my logs do not display class-wise accuracy, which is crucial for a detailed analysis of my model's performance across different categories.
I am unsure if this feature requires additional configuration or if it's a limitation of the tools I'm using.
Could someone advise on how to enable or set up logging to include class-wise accuracy?
Detailed steps or references to documentation would be extremely helpful.

The CheckpointHook of MMDetection V2.25.3 can not save the best checkpoint by default. I recommend you use the Co-DETR implemented by MMDetection V3, which supports such checkpoint saving features.

Okay, Thanks.

The CheckpointHook of MMDetection V2.25.3 can not save the best checkpoint by default. I recommend you use the Co-DETR implemented by MMDetection V3, which supports such checkpoint saving features.

my mmdet was an older version.
Thanks for letting me know.

@TempleX98
Are you saying that the 'best.pth' file is generated starting from mmdetection version 3.x.x?  I think that's incorrect.
I added the checkpoint_config option to config file to automatically generate the 'best.pth' file, and it was automatically created.
checkpoint_config = dict(     interval=1,     max_keep_ckpts=3,     save_last=True, )


Fail fast when java runtime not found in java native contract

using executable bin pathrather than sh to start docker container and host process to fail fast
another benefit is that you can get more detail of native process starting failed  for troubleshooting

Codecov Report

Merging #205 (c7b210e) into master (f4a06ec) will increase coverage by 0.62%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##           master     #205      +/-   ##
==========================================
+ Coverage   48.87%   49.50%   +0.62%
==========================================
  Files         124      124
  Lines       10739    10797      +58
==========================================
+ Hits         5249     5345      +96
+ Misses       4390     4349      -41
- Partials     1100     1103       +3

Impacted Files
Coverage Δ

bcs/contract/native/contract_process.go
74.78% <80.00%> (+15.96%)
:arrow_up:

bcs/contract/native/runtime.go
68.13% <100.00%> (+55.22%)
:arrow_up:

bcs/consensus/xpoa/schedule.go
48.21% <0.00%> (-11.34%)
:arrow_down:

bcs/consensus/tdpos/common.go
64.06% <0.00%> (-3.13%)
:arrow_down:

bcs/contract/xvm/code_manager.go
63.63% <0.00%> (-2.73%)
:arrow_down:

bcs/consensus/tdpos/tdpos.go
38.81% <0.00%> (-1.69%)
:arrow_down:

bcs/consensus/xpoa/xpoa.go
53.36% <0.00%> (+2.14%)
:arrow_up:

bcs/contract/native/process_manager.go
71.92% <0.00%> (+3.50%)
:arrow_up:

bcs/consensus/pow/pow.go
42.96% <0.00%> (+5.41%)
:arrow_up:

... and 1 more

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 f4a06ec...c7b210e. Read the comment docs.

Codecov Report

Merging #205 (c7b210e) into master (f4a06ec) will increase coverage by 0.70%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##           master     #205      +/-   ##
==========================================
+ Coverage   48.87%   49.57%   +0.70%
==========================================
  Files         124      124
  Lines       10739    10797      +58
==========================================
+ Hits         5249     5353     +104
+ Misses       4390     4341      -49
- Partials     1100     1103       +3

Impacted Files
Coverage Δ

bcs/contract/native/contract_process.go
74.78% <80.00%> (+15.96%)
:arrow_up:

bcs/contract/native/runtime.go
68.13% <100.00%> (+55.22%)
:arrow_up:

bcs/consensus/xpoa/schedule.go
48.21% <0.00%> (-11.34%)
:arrow_down:

bcs/consensus/tdpos/common.go
64.06% <0.00%> (-3.13%)
:arrow_down:

bcs/contract/xvm/code_manager.go
63.63% <0.00%> (-2.73%)
:arrow_down:

bcs/consensus/tdpos/tdpos.go
38.81% <0.00%> (-1.69%)
:arrow_down:

bcs/network/p2pv2/server.go
55.02% <0.00%> (+1.18%)
:arrow_up:

bcs/consensus/xpoa/xpoa.go
53.36% <0.00%> (+2.14%)
:arrow_up:

bcs/contract/native/process_manager.go
71.92% <0.00%> (+3.50%)
:arrow_up:

bcs/consensus/pow/pow.go
42.96% <0.00%> (+5.41%)
:arrow_up:

... and 3 more

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 f4a06ec...c7b210e. Read the comment docs.

Codecov Report

Merging #205 (6db9333) into master (f4a06ec) will increase coverage by 0.58%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##           master     #205      +/-   ##
==========================================
+ Coverage   48.87%   49.46%   +0.58%
==========================================
  Files         124      124
  Lines       10739    10797      +58
==========================================
+ Hits         5249     5341      +92
+ Misses       4390     4349      -41
- Partials     1100     1107       +7

Impacted Files
Coverage Δ

bcs/contract/native/contract_process.go
68.90% <80.00%> (+10.08%)
:arrow_up:

bcs/contract/native/runtime.go
69.23% <100.00%> (+56.32%)
:arrow_up:

bcs/consensus/xpoa/schedule.go
48.21% <0.00%> (-11.34%)
:arrow_down:

bcs/contract/xvm/code_manager.go
63.63% <0.00%> (-2.73%)
:arrow_down:

bcs/consensus/tdpos/tdpos.go
39.66% <0.00%> (-0.85%)
:arrow_down:

bcs/consensus/xpoa/xpoa.go
52.40% <0.00%> (+1.18%)
:arrow_up:

bcs/contract/native/process_manager.go
71.92% <0.00%> (+3.50%)
:arrow_up:

bcs/consensus/pow/pow.go
42.96% <0.00%> (+5.41%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #205 (42b4e11) into master (f4a06ec) will decrease coverage by 0.03%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master     #205      +/-   ##
==========================================
- Coverage   48.87%   48.84%   -0.04%
==========================================
  Files         124      124
  Lines       10739    10797      +58
==========================================
+ Hits         5249     5274      +25
- Misses       4390     4423      +33
  Partials     1100     1100

Impacted Files
Coverage Δ

bcs/contract/native/contract_process.go
58.82% <50.00%> (ø)

bcs/contract/native/runtime.go
13.18% <50.00%> (+0.28%)
:arrow_up:

bcs/consensus/xpoa/schedule.go
48.21% <0.00%> (-11.34%)
:arrow_down:

bcs/consensus/tdpos/common.go
64.06% <0.00%> (-3.13%)
:arrow_down:

bcs/contract/xvm/code_manager.go
63.63% <0.00%> (-2.73%)
:arrow_down:

bcs/consensus/tdpos/tdpos.go
38.81% <0.00%> (-1.69%)
:arrow_down:

bcs/consensus/xpoa/xpoa.go
53.36% <0.00%> (+2.14%)
:arrow_up:

bcs/consensus/pow/pow.go
42.96% <0.00%> (+5.41%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #205 (a239508) into master (f4a06ec) will increase coverage by 0.50%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##           master     #205      +/-   ##
==========================================
+ Coverage   48.87%   49.38%   +0.50%
==========================================
  Files         124      124
  Lines       10739    10797      +58
==========================================
+ Hits         5249     5332      +83
+ Misses       4390     4359      -31
- Partials     1100     1106       +6

Impacted Files
Coverage Δ

bcs/contract/native/contract_process.go
68.90% <75.00%> (+10.08%)
:arrow_up:

bcs/contract/native/runtime.go
69.23% <100.00%> (+56.32%)
:arrow_up:

bcs/consensus/xpoa/schedule.go
48.21% <0.00%> (-11.34%)
:arrow_down:

bcs/consensus/tdpos/common.go
64.06% <0.00%> (-3.13%)
:arrow_down:

bcs/contract/xvm/code_manager.go
63.63% <0.00%> (-2.73%)
:arrow_down:

bcs/consensus/tdpos/tdpos.go
38.81% <0.00%> (-1.69%)
:arrow_down:

bcs/consensus/xpoa/xpoa.go
52.40% <0.00%> (+1.18%)
:arrow_up:

bcs/contract/native/process_manager.go
71.92% <0.00%> (+3.50%)
:arrow_up:

bcs/consensus/pow/pow.go
41.06% <0.00%> (+3.51%)
:arrow_up:

... and 1 more

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 f4a06ec...a239508. Read the comment docs.

Codecov Report

Merging #205 (c0e93bd) into master (f4a06ec) will increase coverage by 0.59%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##           master     #205      +/-   ##
==========================================
+ Coverage   48.87%   49.47%   +0.59%
==========================================
  Files         124      124
  Lines       10739    10797      +58
==========================================
+ Hits         5249     5342      +93
+ Misses       4390     4350      -40
- Partials     1100     1105       +5

Impacted Files
Coverage Δ

bcs/contract/native/contract_process.go
68.90% <75.00%> (+10.08%)
:arrow_up:

bcs/contract/native/runtime.go
69.23% <100.00%> (+56.32%)
:arrow_up:

bcs/consensus/xpoa/schedule.go
48.21% <0.00%> (-11.34%)
:arrow_down:

bcs/consensus/tdpos/common.go
64.06% <0.00%> (-3.13%)
:arrow_down:

bcs/contract/xvm/code_manager.go
63.63% <0.00%> (-2.73%)
:arrow_down:

bcs/consensus/tdpos/tdpos.go
38.81% <0.00%> (-1.69%)
:arrow_down:

kernel/consensus/base/driver/chained-bft/smr.go
52.41% <0.00%> (+0.96%)
:arrow_up:

bcs/network/p2pv2/server.go
55.02% <0.00%> (+1.18%)
:arrow_up:

bcs/consensus/xpoa/xpoa.go
52.40% <0.00%> (+1.18%)
:arrow_up:

bcs/contract/native/process_manager.go
71.92% <0.00%> (+3.50%)
:arrow_up:

... and 3 more

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 f4a06ec...c0e93bd. Read the comment docs.

Codecov Report

Merging #205 (c0e93bd) into master (f4a06ec) will increase coverage by 0.77%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##           master     #205      +/-   ##
==========================================
+ Coverage   48.87%   49.65%   +0.77%
==========================================
  Files         124      124
  Lines       10739    10797      +58
==========================================
+ Hits         5249     5361     +112
+ Misses       4390     4332      -58
- Partials     1100     1104       +4

Impacted Files
Coverage Δ

bcs/contract/native/contract_process.go
68.90% <75.00%> (+10.08%)
:arrow_up:

bcs/contract/native/runtime.go
69.23% <100.00%> (+56.32%)
:arrow_up:

bcs/consensus/xpoa/schedule.go
48.21% <0.00%> (-11.34%)
:arrow_down:

kernel/consensus/base/driver/chained-bft/smr.go
52.41% <0.00%> (+0.96%)
:arrow_up:

bcs/network/p2pv2/server.go
55.02% <0.00%> (+1.18%)
:arrow_up:

bcs/consensus/xpoa/xpoa.go
54.32% <0.00%> (+3.10%)
:arrow_up:

bcs/contract/native/process_manager.go
71.92% <0.00%> (+3.50%)
:arrow_up:

bcs/consensus/pow/pow.go
42.96% <0.00%> (+5.41%)
:arrow_up:

bcs/network/p2pv1/conn.go
61.29% <0.00%> (+6.45%)
:arrow_up:

... and 1 more

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 f4a06ec...c0e93bd. Read the comment docs.

Codecov Report

Merging #205 (6f92908) into master (b21e379) will increase coverage by 0.43%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##           master     #205      +/-   ##
==========================================
+ Coverage   47.96%   48.39%   +0.43%
==========================================
  Files         133      133
  Lines       11341    11339       -2
==========================================
+ Hits         5440     5488      +48
+ Misses       4782     4722      -60
- Partials     1119     1129      +10

Impacted Files
Coverage Δ

bcs/contract/native/contract_process.go
68.90% <75.00%> (+10.08%)
:arrow_up:

bcs/contract/native/runtime.go
69.23% <100.00%> (+56.32%)
:arrow_up:

bcs/consensus/tdpos/common.go
64.06% <0.00%> (-3.13%)
:arrow_down:

bcs/contract/xvm/code_manager.go
63.63% <0.00%> (-2.73%)
:arrow_down:

bcs/consensus/tdpos/tdpos.go
43.04% <0.00%> (-2.18%)
:arrow_down:

bcs/network/p2pv1/server.go
56.46% <0.00%> (-1.37%)
:arrow_down:

kernel/consensus/base/driver/chained-bft/smr.go
52.41% <0.00%> (-0.97%)
:arrow_down:

bcs/consensus/tdpos/schedule.go
48.40% <0.00%> (-0.92%)
:arrow_down:

bcs/contract/native/process_manager.go
71.92% <0.00%> (+3.50%)
:arrow_up:

... and 1 more

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 b21e379...6f92908. Read the comment docs.


reactive_pinput backspace bug on mobile

I've been using the reactive_pinput package for a while now, and recently discovered an issue on iOS and Android that the backspace key doesn't actually delete anything. I have a field that requires 6 numbers. Once I input all 6, then try to delete characters, it doesnt delete anything. The causes issues if wrong numbers were input. Are there any known work arounds for it, or is this a known issue already? I'll work on getting a sample project together to demonstrate this bug a little later today.

@vasilich6107 So sorry for the delay. I completely spaced on this. I will dig back into this today and tomorrow to see if it is part of this package or the pinput package.

After digging into this more, an AutofillGroup is the culprit for things not working. Let me test it directly with the Pinput package and see if it's still an issue.


ERR_PNPM_NO_OFFLINE_TARBALL while bulding off main

Running phase: unpackPhase
@nix { "action": "setPhase", "phase": "unpackPhase" }
unpacking source archive /nix/store/w7gm2zyz0k6rdz281809jkpzlf0h3d91-source
source root is source
Running phase: patchPhase
@nix { "action": "setPhase", "phase": "patchPhase" }
Running phase: updateAutotoolsGnuConfigScriptsPhase
@nix { "action": "setPhase", "phase": "updateAutotoolsGnuConfigScriptsPhase" }
Running phase: configurePhase
@nix { "action": "setPhase", "phase": "configurePhase" }
no configure script, doing nothing
Executing pnpmConfigHook
Configuring pnpm store
Installing dependencies
Scope: all 2 workspace projects
Lockfile is up to date, resolution step is skipped
Progress: resolved 1, reused 0, downloaded 0, added 0
Packages: +544

Progress: resolved 1, reused 0, downloaded 0, added 0
Progress: resolved 544, reused 10, downloaded 0, added 0
 ERR_PNPM_NO_OFFLINE_TARBALL  A package is missing from the store but cannot download it in offline mode. The missing package may be downloaded from https://registry.npmjs.org/js-yaml/-/js-yaml-4.1.0.tgz.
Progress: resolved 544, reused 10, downloaded 0, added 0

Same error

I'll switch vencord over to nixpkgs for now and it should work fine I hope

works now with merge

This issue is still around for me

This issue is still around for me
It is too for me, the referenced PR did not seem to change anything...?


The url of ffmpeg need to update in ubuntu 16.04 Dockerfile

Describe the bug
The ffmpeg url(https://github.com/fcwu/docker-ubuntu-vnc-desktop/blob/xenial/Dockerfile#L36) is not found
Versions:

OS: Ubuntu 16.04
image tag xenial

it works now


Shift-PageUp

The doc says that Shift-PageUp should scroll back but it does not work.
The shortcut Mod-PageUp works thought.
Could you please add the missing shortcut.
Thanks.

The "shortcut" is already there. For example it works with st which doesn't provide an internal scroll back buffer and as a result doesn't intercept Shift-PageUp keys.
The problem is likely that the environment from which you run dvtm (e.g. a X terminal emulator) consumes the Shift-PageUp keys.
Maybe MOD+e (copy mode using your editor) or MOD+E (scroll back history using your pager) is a usable alternative.

That was indeed an issue with the terminal, alacritty.
I added these lines to the config file:
  - { key: PageUp,   mods: Shift,   chars: "\x1b[5;2~"                   }
  - { key: PageDown, mods: Shift,   chars: "\x1b[6;2~"                   }

and now it works.


Remove restart_command for puma

This way it work more stable for me

What do you mean by more stable? USR2 should restart workers (https://github.com/puma/puma/blob/6479e6b26b2b4ac7d09a78cd7b1e04470d8a213e/docs/signals.md)

USR2 for soft restart workers. If you need hard restart puma, use eye stop puma; eye start puma


Training and Testing Datasets

Discussed in https://github.com/weecology/DeepForest/discussions/448

Originally posted by antoniosilva9116 May 16, 2023
Hi!
First of all, I would like to thank you for sharing your work with us. My question is about the dataset for training and testing the pre-trained model in ImageNet, the produced released NEON.pt. Is it all available at https://zenodo.org/record/5914554#.ZGOPrdLMKTe, or did you not provide it?

Sorry, can you clarify which datasets you are looking for that are not there? The training and test dataset is in the linked zenodo archive.

Oh, of course. Those that you used for producing the NEON.pt the release train model. So, to clarify my question the Zenodo archive that I provide is the dataset that you use to produce the NEON.pt?

yes
On Wed, May 17, 2023 at 4:17 AM antoniosilva @.***>
wrote:

Oh, of course. Those that you used for producing the NEON.pt the release
train model. So, to clarify my question the Zenodo archive that I provide
is the dataset that you use to produce the NEON.pt?
—
Reply to this email directly, view it on GitHub
https://github.com/weecology/DeepForest/issues/449#issuecomment-1551204553,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJHBLFBYXSFU375A5KGA63XGSXUDANCNFSM6AAAAAAYDXGCWY
.
You are receiving this because you were assigned.Message ID:
@.***>

--
Ben Weinstein, Ph.D.
Research Scientist
University of Florida
http://benweinstein.weebly.com/

I'm trying to implement other state-of-the-art pre-trained object detection methods, YOLOv8, or use different backbones like ViT to fine-tune using the same dataset. The main goal is to improve the provided model.


Ensure artifacts have the correct file permissions

After staging a flexible style project, we need to ensure the artifact is readable by everyone or docker wont be able to run it. Deploys fail to verify the vm is running and an 502 is presented when trying to hit the url.

More info :
see https://github.com/GoogleCloudPlatform/gcloud-maven-plugin/blob/1a623532890aedb444e9ad1ad508c17005262d66/src/main/java/com/google/appengine/gcloudapp/GCloudAppStage.java#L91


"Ktlint is not yet configured for this project" is shown on each and every file

With 0.20.0-beta-4 when you open any file, the header is shown.
It would maybe make sense to show this header only on files that are actually affected by the ktlint plugin.

Tnx for the nice suggestion.


fix kafka_client_api_version

What does this PR do?
fixes an issue when pinning the api_version.
2 issues:

api_version expects a tuple but kafka_client_api_version is a string
there is a bug with KafkaAdminClient where pinning any api_version will raise an IncompatibleBrokerVersion. Setting api_version=None would be a temporary fix.

Motivation
found bug while investigating something.
Additional Notes

Created a PR over at kafka-python to fix pinning api_version for KafkaAdminClient.

https://github.com/dpkp/kafka-python/pull/1953

Related: https://github.com/dpkp/kafka-python/issues/1675

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

Alternatively, can we path kafka-python the same as how we did it with jaydebeapi (https://github.com/DataDog/datadog-agent/pull/4402)?
we just need to insert 1 line at kafka/admin/client.py https://github.com/dpkp/kafka-python/pull/1953/files
then we can keep pinning the api_version in https://github.com/DataDog/integrations-core/pull/5007/files#diff-deed983f73a04522ac621d7f6d0c8403R132-R133


timed out waiting for pods with filter -l io.cilium/app=etcd-operator to be ready

https://jenkins.cilium.io/job/Ginkgo-CI-Tests-Pipeline/6582/testReport/Suite-k8s-1/10/K8sFQDNTest_Restart_Cilium_validate_that_FQDN_is_still_working/
/home/jenkins/workspace/Ginkgo-CI-Tests-Pipeline/src/github.com/cilium/cilium/test/ginkgo-ext/scopes.go:367
etcd-operator is not ready after timeout, pods status:
 NAME                                    READY     STATUS     RESTARTS   AGE
cilium-etcd-7tlswxqtd6                  0/1       Init:0/1   0          4m
cilium-etcd-operator-77d4ddf8c6-6djcv   1/1       Running    0          10m
cilium-etcd-wvdjkg7ftn                  1/1       Running    0          4m
etcd-operator-65476dd78f-45mv5          1/1       Running    0          4m

Expected
    <*errors.errorString | 0xc42bec0980>: {
        s: "Timeout reached: timed out waiting for pods with filter -l io.cilium/app=etcd-operator to be ready",
    }
to be nil
/home/jenkins/workspace/Ginkgo-CI-Tests-Pipeline/src/github.com/cilium/cilium/test/k8sT/assertionHelpers.go:88

Also affects v1.4: https://github.com/cilium/cilium/issues/7546

Hit here as well: https://jenkins.cilium.io/job/cilium-ginkgo/job/cilium/job/master/2725/

Not observed in a while, closing for now


⚠ Processors has degraded performance

In 98ae8c2, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 70 ms

Resolved: Processors performance has improved in d13b93b.


Expose collision contact info

World:collide currently returns a boolean indicating whether the shapes collided.  More specifically, it indicates whether there were any contact points between the two shapes.  However, those contact points have a lot of extra useful info!  It would be really neat to know where on the shapes the collisions took place, the normal vectors of those contact points, etc.  ODE is already giving us all this info, so it would be as simple as returning it in a table or exposing it from a callback/iterator interface.
There are lots of places to go from here, like allowing people to filter the contacts, create contact joints manually, set per-contact friction/restitution, and maybe more.  But starting with the raw info would power lots of cool things that aren't possible right now.

I sketched out the following change

Remove World:collide, World:computeOverlaps, World:overlaps, and the World:update callback.
Add World:get/setContactFilter (name very TBD)

This is a function you can optionally set that will get called when World:update wants to create a contact.
It gets passed shapeA, shapeB, x, y, z, nx, ny, nz, depth
It should return collide, friction, bounce to control whether/how the collision happens (replaces :collide).

It's a breaking change, but I think it has some advantages over the current API:

Easier to use (just set a callback when creating the world, no need to call overlaps/computeOverlaps/collide).
Less confusion about how :collide interacts with :update.
Exposes all of ODE's contact info.

Disadvantages are that it's a breaking change, and it calls a callback more often.

After thinking about it more, the above idea doesn't make it easy enough to ask "what are the contacts between 2 colliders/shapes".

This is implemented as World:getContacts now.  It takes 2 shapes and returns a table of contacts, where each contact is a table with the position, normal, and overlap depth.  This does mean that the collision test gets duplicated if you also need to collide the 2 shapes, but I think that's okay for now.


Create a 0.7.1 release in master branch

I think we should continue creating "releases" on the master branch. This shows also what the last version was, and also gives a convenient way to get get to a certain release from the master branch.

Agreed, thanks for pointing it out


Compile to browser?

One question, this library works in browser?
I tried with browserify but dont work...
sorry for my bad english

Confirmed that it doesn't work in the browser. Its looking for a global require function that doesn't exist. Also the bower repo points here (I assume it should point to msgpack-js-browser?)


Please remove any other slugify library and install unicode-slugify

[x] Using latest version as provided on the master branch
[x] Searched for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug
[ ] Feature Request
[x] Question
[ ] Other

Description

Log

Passing the eg. line for downloading a song got this two output:

ERROR: Oops! unicode-slugify was not found.
INFO: Please remove any other slugify library and install unicode-slugify

Those are the packages installed with pip:

beautifulsoup4  4.6.0
certifi         2018.4.16
chardet         3.0.4
idna            2.6
logzero         1.5.0
lyricwikia      0.1.9
mutagen         1.40.0
pafy            0.5.4
pathlib         1.0.1
pip             10.0.1
PyYAML          3.12
requests        2.18.4
setuptools      38.5.2
six             1.11.0
spotipy         2.4.4
titlecase       0.12.0
unicode-slugify 0.1.3
Unidecode       1.0.22
urllib3         1.22
wheel           0.30.0
youtube-dl      2018.4.16

Don' t know what is wrong and where to look for other slugify libraries.
Thanks for your time.

Take a look at #171.

I was using pip instead of pip3, after reinstalling quite everything it works.


add apple silicon building

this should work, untested though

Instead of replacing the existing macOS builder, why not add it as an addition?

I think that it is just an extra step in the builder.

I mean instead of replacing macos-latest, just append it to the matrix.

macos-11 is just macOS 11 but macos-latest is 10.15 I don't see why we need to have two duplicate builds for 10.15 and 11 (intel)?

Oh, okay.

i was wanting to make sure it worked, because 10.15 doesnt suppose arm as far as i know

darwin arm64 isn't in the glob, i dont think it'll get uploaded without that

Oh right

also what about DMG? You had it in Catalyst

I don't know if you want it or not. You can also add PKG support too if you want that. There are a lot of forge makers.


ChannelOptions.MaxReceiveMessageLength Has no effect.

The value set using ChannelOptions.MaxReceiveMessageLength is not respected when Server is created, the channel size is defaulted to 4MB. Please let me know if there is issue with my code of the gRpc framework
Server Creation Code
public static readonly int DataSizeLimit = 64 * 1024;
DataChannelSettings = new List<ChannelOption>()
        {
            new ChannelOption(ChannelOptions.MaxSendMessageLength, DataSizeLimit),
            new ChannelOption(ChannelOptions.MaxReceiveMessageLength, DataSizeLimit),
            new ChannelOption(ChannelOptions.MaxMessageLength, DataSizeLimit),
        };

myServer = new Server(DataChannelSettings)
            {
                Services = { RPC.BindService(this) },
                Ports = { new ServerPort(myAddress.Item1, myAddress.Item2, ServerCredentials.Insecure) },
            };

GRPC Version 2.37.1 with .Net Framework 4.8

Mistake on my side, Some static's and tuples screwed me up, Closing the issue,
The above code works fine now


Fix predicate pushdown in JDBC for CHAR columns

cc @electrum

sorry for the spam. This needs to be solved on per-db basis. This wasn't fixing anything in practice.


FormField validation BC-break (3.2)

https://github.com/silverstripe/silverstripe-framework/pull/3636 added Validator type hinting to a number of form field validate() methods which causes errors to crop up if any of the fields have been subclassed (i.e. validate() should be compatible with FormField::validate(Validator $validator)). Looks like this was done before 3 was branched off master.

As this was originally my PR I'm happy to go through and make a PR against 3.2 to remove the type hinting unless you have a fancy merge planned? :smile:

Removing the typehint won't work, since SubClass::validate() ParentClass::validate($args) still throws strict errors.
You can use func_get_args and instead use phpdoc to document the parameter instead (hack we used for SiteTree::canView extra params :) ) Just make sure to mark it to be actually added to parameter list in 4.0

@tractorcow #3636 didn’t actually add the parameter, it just added type-hinting to it. The PHP error is a bit misleading, it should really say Declaration of ChildClass::foo($arg) should be compatible with ParentClass::foo(Bar $arg), but it doesn’t :/
@stevie-mayhew I haven’t looked at fixing this, so if you fancy it then that’d be most welcome :)

Oh, in which case, awesome. Sometimes it's nice to be wrong.

Its better to be wrong than quiet :smiley:

Fixed with #4175


Handle encryptor retrieval failure better

Waits longer for the encryptor image to show
up. fails more gracefully
if the image isn't created for some reason

LGTM


RatbagdResolution: emit properties changed from signal callbacks

This allows clients to respond to changes in the active resolution. It's not the cleanest approach, but it works. To clean it up, we could cache the changed resolutions and make sure to only emit the signal on those, just as the FIXME already notes.
References https://github.com/libratbag/piper/pull/132.

I'm attempting to do the same for ratbagd_profile_set_active, but I'm starting to get some No active profile. Please report this bug to the libratbag developers from Piper with that change... haven't figured out why yet. It's getting late so I'll continue tomorrow; feel free to merge this already, I can just open a new PR once I get that one working too.


crane plugin-manager list not working as expeceted

What version of crane are you running, and what are your clutsters+platform
[alpatel@alpatel kafka]$ crane version
crane:
Version: v0.0.3
crane-lib:
Version: v0.0.6
OS: linux, source cluster k3d, destination OpenShift
What did you expect to happen?
$ crane plugin-manager list command should list the Plugins available
What actually happened?
[alpatel@alpatel kafka]$ crane plugin-manager list
ERRO[0000] Error reading ImageStreamPlugin plugin manifest located at https://github.com/konveyor/crane-plugins/raw/main/plugins/ImageStream/index.yaml - Error: error
converting YAML to JSON: yaml: line 163: mapping values are not allowed in this context

Please include any relevant logs or errors
I think the urls here https://github.com/konveyor/crane-plugins/blob/main/index.yaml#L5 might be wrong

Considering this closed by #19 , @JaydipGabani could you confirm?


App crashes when setting a backlink manually

Steps to Reproduce
I do not currently have a minimal example that reproduces the crash. It crashes repeatedly in large app, and the cause seems to be me setting a backlink's value (unnecessarily). I have tried to reproduce the issue with a minimal example but so far have been unable to.
However this is the stacktrace I get with GDB:
    thread '<unnamed>' panicked at 'called `Result::unwrap()` on an `Err` value: SendError { .. }', src/txn.rs:118:56
    note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
    fatal runtime error: failed to initiate panic, error 5

    Thread 8 "io.flutter.ui" received signal SIGABRT, Aborted.
    [Switching to Thread 0x7fffe8c71700 (LWP 684659)]
    __GI_raise (sig=sig@entry=6) at ../sysdeps/unix/sysv/linux/raise.c:50
    50	../sysdeps/unix/sysv/linux/raise.c: No such file or directory.
    (gdb) bt
    #0  __GI_raise (sig=sig@entry=6) at ../sysdeps/unix/sysv/linux/raise.c:50
    #1  0x00007ffff4442859 in __GI_abort () at abort.c:79
    #2  0x00007fff9dc701b7 in  () at /tmp/project/app/build/linux/x64/debug/bundle/lib/libisar.so
    #3  0x00007fff9dc7012f in  () at /tmp/project/app/build/linux/x64/debug/bundle/lib/libisar.so
    #4  0x00007fff9dc6fc82 in  () at /tmp/project/app/build/linux/x64/debug/bundle/lib/libisar.so
    #5  0x00007fff9dc912a8 in  () at /tmp/project/app/build/linux/x64/debug/bundle/lib/libisar.so
    #6  0x00007fff9dc91226 in  () at /tmp/project/app/build/linux/x64/debug/bundle/lib/libisar.so
    #7  0x00007fff9dc911e2 in  () at /tmp/project/app/build/linux/x64/debug/bundle/lib/libisar.so
    #8  0x00007fff9dbaa000 in  () at /tmp/project/app/build/linux/x64/debug/bundle/lib/libisar.so
    #9  0x00007fff9dbaa2d2 in  () at /tmp/project/app/build/linux/x64/debug/bundle/lib/libisar.so
    #10 0x00007fff9dc12d85 in  () at /tmp/project/app/build/linux/x64/debug/bundle/lib/libisar.so
    #11 0x00007fff9dc0ac76 in isar_put_all () at /tmp/project/app/build/linux/x64/debug/bundle/lib/libisar.so

And this is the stack trace I get when running with RUST_BACKTRACE=full:
    thread '<unnamed>' panicked at 'called `Result::unwrap()` on an `Err` value: SendError { .. }', src/txn.rs:118:56
    stack backtrace:
       0:     0x7f5e7b4f14fd - <unknown>
       1:     0x7f5e7b496c0c - <unknown>
       2:     0x7f5e7b4eff44 - <unknown>
       3:     0x7f5e7b4f0640 - <unknown>
       4:     0x7f5e7b4efa31 - <unknown>
       5:     0x7f5e7b5112a8 - <unknown>
       6:     0x7f5e7b511226 - <unknown>
       7:     0x7f5e7b5111e2 - <unknown>
       8:     0x7f5e7b42a000 - <unknown>
       9:     0x7f5e7b42a2d2 - <unknown>
      10:     0x7f5e7b492d85 - <unknown>
      11:     0x7f5e7b48ac76 - isar_put_all
      12:     0x7f5ea420598b - <unknown>
    fatal runtime error: failed to initiate panic, error 5
    Aborted (core dumped)

Code sample
N/A
Version

Platform: Linux
Flutter version: 2.10.2
Isar version: 2.2.0
Isar Generator version: 2.2.0

Is there a way for me to (further) symbolicate the crash report?

This should be fixed :+1:


Invoking the dbg! macro snippet on a negated expression leaves the negation operator outside the dbg!()

Expected
Calling the postfix dbg completion on a negated expression will wrap the whole expression including the negation operator.
Observed
When the postfix dbg completion is requested, the original negation operator remains outside the freshly inserted dbg!() macro.
Repro
enum NestedStuff {
    Variants,
    Dont,
    Matter,
}

impl NestedStuff {
    fn random_method(self) -> bool {
        return true;
    }
}

fn some_func() {
    let my_enum = NestedStuff::Matter;
    if !my_enum.random_method().dbg$0 {  // choose the dbg snippet and have this line convert to
 //`if !dbg!(my_enum.random_method()) {`
        println!("Hello");
    }
}

Version info
rustc -V
rustc 1.58.0 (02072b482 2022-01-11)

rust-analyzer --version
rust-analyzer ba3305480 2022-02-07 stable

using lsp-mode in doom emacs, if that's relevant.

This seems like intended behavior. At the very least, I like that it functions the way it does currently!

I often use dbg!() for debug printing and found it confusing when dbg!() printed a method that returns true, but the branch wasn't actually taken, because the negation operator outside dbg!() converted the value to false. I would find it more obvious, if the dbg postfix completion included negation by default to prevent such surprises.
https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=a51870009964dee5fa1c615d4b65014a

This also happens with .if completion.

Ye if is a definite contender here as well, we should look at all the postfixes and check where checking the precedence might make sense


chore: added deployments page to nextgen nav sidebar

In navigation bar:

Current FTUX page:

Nextgen page:

Looks awesome!


Control LEDs over UART

As suggestions were encouraged, I would like to see how to control the LED color and speed of the blinker. It would nicely tie together section 2 and 3.

Thanks, that's a great idea! I will add it ASAP.

Added Section 4: Happylife, includes demo of controlling LED color and blink speed via UART


Update messaging and header

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


Webpage settings doesn't respect WebPageSrc

When setting webpage generation, WebPageSrc is not taken into acount. For example, if I change on WEBPAGE 1 line WebPageSrc=scripts/web/spherestatusbase.htm to anything else (like scripts/web/status.json), sphere sends warning with WARNING:'spherestatusbase1.htm' not found....
This is likely because filename seems hardcoded in CWepPageDef.cpp

This seems to be little more complicated. If we allow using filename from config, using m_sSrcFilePath = rid.GetResPage(); it can still fail to load. It will happen here, if we use file extension, that is not in allowed extension list.


setup script: send debug / trace info to a log file

don't throw so much log info during setup
just tell what is happening (steps)
write rest to a log file

New deploy script using - edge-launchpad, displays limited log. Edge-Launchpad doesn't write logs to a file. It makes sense to have logging ability for edge-launchpad.
Creating an issue for edge-launchpad for the same - issue13


Misc. updates for Compara

All tested on my sandbox

Table of multiple alignments
Static version on staging vs dynamic version on my sandbox

Consensus cigar-lines
I actually feel like it's made my sandbox faster than the live site !

BRCA2: www vs sandbox
U6: www vs sandbox

For the table of multiple alignments, there is a related change in public-plugins. I've replaced the HTML code of the table in ensembl/htdocs/info/genome/compara/multiple_genome_alignments.html with this
[[SCRIPT::EnsEMBL::Web::Document::HTML::Compara::format_wga_table()]]

Super, thanks a lot ! I'll now push the changes to public-plugins
Matthieu


sourcegraph: add login field to AuthInfo

Is this just to save one extra API call? If so, LGTM.

Yes, this will save a roundtrip to a remote server (eg. federation root) in doing Users.Get to fetch the login.


Added support for BigDecimal field

A simple PR adding support for BigDecimal fields.

Tests seem a bit flaky:
- Error responses are handled correctly *** FAILED *** (32 milliseconds)
[info]   org.apache.http.conn.HttpHostConnectException: Connect to localhost:64011 [localhost/<IP_ADDRESS>] failed: Connection refused (Connection refused)

AFAIAC this is good to be merged. Let me know if you want me to make further changes.

Nice! This is quite useful to me.  It looks like this may pass its tests if they're just run again.  Any chance of getting this merged?

@truppert I do not have permissions to restart the build but I don't see anything wrong with this and tests are passing locally. It looks to me that this repo might be abandoned.

Hey everyone, sorry for the long wait. Changes look good but there's indeed
something wrong with the tests. I'll give it a look this week end and
release a new version
El mié., 21 mar. 2018 21:29, Anargyros Akrivos<EMAIL_ADDRESS>escribió:

@truppert https://github.com/truppert I do not have permissions to
restart the build but I don't see anything wrong with this and tests are
passing locally. It looks to me that this repo might be abandoned.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/paulgoldbaum/scala-influxdb-client/pull/30#issuecomment-375084829,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdXdKALVoGvkWpj_jlBA5RBWI2cRfkFks5tgrgOgaJpZM4P9_0i
.

Codecov Report

Merging #30 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #30   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          14     14
  Lines         199    212   +13
  Branches       19     23    +4
=====================================
+ Hits          199    212   +13

Impacted Files
Coverage Δ

.../scala/com/paulgoldbaum/influxdbclient/Point.scala
100% <100%> (ø)
:arrow_up:

.../scala/com/paulgoldbaum/influxdbclient/Field.scala
100% <100%> (ø)
:arrow_up:

...a/com/paulgoldbaum/influxdbclient/HttpClient.scala
100% <0%> (ø)
:arrow_up:

...ala/com/paulgoldbaum/influxdbclient/InfluxDB.scala
100% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@argyakrivos thanks for taking the time for the PR and sorry it took so long to merge. I will publish a release including your changes in a minute


Detect resource file overrides

Depends on zsh >= 5.3, where "-v" was introduced. As this was relatively
recent (2016-12-12) there is a fallback mechanism which reverts to the
old functionality.
Closes zsh-users/antigen#598

Isn't it enough to use:
ANTIGEN_RSRC="${ANTIGEN_RSRC:-${ADOTDIR:-$HOME/.antigen}/.resources}"

Which defaults ANTIGEN_RSRC to whatever value it set, otherwise fallbacks to ADOTDIR/.resources

This was implemented in #588 and is currently available in develop. Ready for next release 2.2.2.


Throw when Stripe.js is not available

Also makes sense to make 'Stripe' a constant in the index and then just inject it elsewhere.
Per https://github.com/bendrucker/angular-stripe/issues/7#issuecomment-77184418, this seems to be a common mistake

Also it would be nice to not have this break everything if Stripe.js is not available (happens in offline mode).

What does "offline mode" mean? I don't understand the use case you're referencing.

Sitting in the plane coding :-)

Ah, okay. So the reason I hadn't encountered this is because I'm still bundling stripe-debug.js directly. That results in a console.warn so I'm going to have to do the CDN route.
Not sure what I can do here though. Obviously everything's going to break down if you hit code that's making network calls to Stripe. The only thing that'll break up front is the stripeProvider.setPublishableKey method. Letting that fail silently would create opportunities for bugs to slip into live code.
There's two options here: one is to include a super light weight shim only in your dev builds that ensures window.Stripe is at least an empty object.
I've also considered writing a full mock Stripe.js implementation with an Angular-like API for mocking and flushing requests. If you think that's something you'd use I'll consider starting that soon.

Okay, got it. Actually it's not a problem, if the actual call to Stripe fails (since it is not available). But the application should at least be runnable. Maybe check if setPublishableKey is available?

That's the problem though. What's supposed to happen if it's not? In production, the application should fail to bootstrap. In development, you want it to fail silently. There's no having it both ways (and the first one is correct), at least from angular-stripe's perspective. You'd have to supply a mock, whether that's a simple window.Stripe = {} or something more sophisticated.

Okay I tried that:
<script>
  window.Stripe = {}
</script>
<script src="https://js.stripe.com/v2/"></script>

It gives another error now:
TypeError: Cannot read property 'validateCardNumber' of undefined

So it won't be quite that easy, you'll need this instead:
window.Stripe = {
  card: {},
  bankAccount: {}
}

Please be super careful with this. Also, stay tuned for https://github.com/bendrucker/mockstripe.

Yeaa, it feels like kind of a hack. It would be super cool, if angular stripe could be configured to use a Stripe mock (if that's what you have in mind). I could imagine sth. like this:
angular
  .module "myApp"
  .run (
    stripe
    myEnv
  ) ->
    stripe.setUseMock myEnv == "development"
    stripe.setPublishableKey configuration.read "stripe_public_key"

It's virtually never appropriate for libraries to provide an API for using a mock, though plenty of libs do it anyway. You either use the module system or provide a hook for overriding specific internals (like async scheduling).
The point of making a Stripe constant is that you could use dependency injection to provide it more easily:
$provide.constant('Stripe', mock)

When you choose to overwrite that value is up to you. It's a build concern and you should never be putting code that tries to do environment detection like myEnv == "development" in production code.


Font can't be loaded.

OS: Arch Linux

As you can see I'm using Hack Nerd Font Bold 20 and it's even "activated" but the bars on the top and on the bottom don't show any glyphs. They are shown in gvim.
I'm using the neoclide-git from the AUR.
My ~/.vim/vimrc (the necessary part):
" -----------------
"  GUI - Settings
" -----------------
set guifont=Hack\ Nerd\ Font\ Bold\ 20
set linespace=4

let g:neovide_refresh_rate=60
let g:neovide_transparency=0.85
let g:neovide_no_idle=v:false
let g:neovide_fullscreen=v:false
let g:neovide_cursor_animation_length=0.13
let g:neovide_cursor_trail_length=0.5
let g:neovide_cursor_antialiasing=v:true
let g:neovide_cursor_vfx_mode = "ripple"
let g:neovide_cursor_vfx_opacity=200.0
let g:neovide_cursor_vfx_particle_lifetime=1.2
let g:neovide_cursor_vfx_particle_density=7.0
let g:neovide_cursor_vfx_particle_speed=10.0
let g:neovide_cursor_vfx_particle_phase=1.5
let g:neovide_cursor_vfx_particle_curl=1.0

The guifont format is incorrect.
set guifont=Hack\ Nerd\ Font:h20

might work better

Cool thank! Now it's big enough but how can I set the style?

There is not currently a way to set the weight globally, but individual syntax highlightings can set bold. For instance I see in your view that the NORMAL in the bottom is bold due to the highlighting there.

Ok, thank you!


Gun melee, CM scatter, bipod, magharn

About the PR
This adds the magharn and the bipod. Also adds firemode-dependent gun stats and changes scatter calculation to match how it's done in CM. All guns have been changed to use this system.
For full details, see the changelog.
Closes #3121, closes #1049 and closes #3162.
Should also deal with #3150, but I can only reproduce the bug on the live server, so I will close this one manually after the PR is merged and I have confirmation that it's fixed.
Why / Balance
Because porting.
Technical details
For upstream file changes, see the next section.
RMCSelectiveFireComponent and RMCSelectiveFireSystem now handle the majority of gun stats.
Use RMCSelectiveFireComponent to set values defined in it instead of GunComponent.
Don't use GunWieldBonus. It's scuffed, and we need wielding to change the base values of the gun, so this is now contained in RMCSelectiveFireComponent.
Burst scatter calculation doesn't work quite like CM for a few reasons: in CM, burst scatter instantly jumps to the maximum for the burst amount, and the direction your character is spraying in tracks the target you clicked on to shoot.
The latter can't really be implemented here, so if we also used the former, burst fire would both have more scatter and, usually, shoot slower than every other firemode. And would, therefore, be even more dogshit than it already is.
So instead I made the scatter ramp up. The hard cap for burst scatter in CM is reached at a 6-shot burst, so this is the number of shots it takes to reach maximum scatter. Bursts will have the same scatter peak as in CM, but will ramp up to it instead of instantly spraying in random directions.
AngleDecay is now always set to 0, as in CM, scatter is instantly reset to 0 after you're done shooting a burst or full-auto spray.
AngleIncrease is now automatically set for any firemode that contains a ShotsToMaxScatter value in its modifier set. Otherwise, this is set to 0, since in CM scatter only ramps up in full-auto and has a concrete number of shots before it reaches its peak.
I'll see what I can do to condense the modifier refresh methods in RMCSelectiveFireSystem in the next PR.
The attachment examine window currently won't properly display modifiers if their conditions are identical in everything but the contents of their whitelist or blacklist. I'll fix this in the next PR. We don't have any attachments that need it anyway, so it's not an issue yet.
Upstream file changes
This changes two sets of upstream files:
The first set is the client MeleeWeaponSystem and a new component: AltFireMeleeComponent. This mirrors the changes from here: https://github.com/space-wizards/space-station-14/pull/29484 because I'm tired of waiting for it to get merged. This adds melee to guns.
The second set includes GunComponent.cs and SharedWeaponSystem.Interactions.cs. The first now gives access to RMCSelectiveFireSystem, while the latter sends out RMCFireModeChangedEvent when a weapon's firemode is changed.
The former allows RMCSelectiveFireSystem to make changes to a weapon's GunComponent, which is necessary, as this system needs to be able to change the weapon's base stats. This could have been accomplished by adding a bunch of setter methods to SharedWeaponSystem, but I decided against it as this lets us have more control over the component without me having to make an upstream PR to add a method for every little change we might need.
The latter is needed to allow RMCSelectiveFireSystem to detect when a gun's firemode is changed and alter its values accordingly.
Media
Different weapon stats for different firemodes:

Attachment modifiers:

Melee with guns:
https://github.com/user-attachments/assets/8df74695-e3af-48a5-8cc0-f4962ed2e685

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog
:cl: Sigil, Cepha

add: Ranged weapons can now be used to attack in melee. Right-click to do so. The base damage for most guns is 5, but attachments may modify it.
add: A new attachment has been added: the magnetic harness. If your weapon is knocked out of your hands or you drop it, this attachment will make the gun snap to your armour. It fits on the M54C, the M4SPR, the M77, the M83, the M42A2 and the M63.
add: A new attachment has been added: the bipod. You can examine it ingame to see the modifiers. It fits on the M54C, the M4SPR and the M96S.
add: A new attachment has been added: the M44 magnum sharpshooter stock. It fits on the M44 revolver. While inactive, the stock prevents the weapon from being fired. When active, it strongly reduces wielded scatter.
add: Attachments can now be toggled through the verb menu. Toggling attachments on items you don't have equipped or held takes longer.
add: Cycling firemodes now has a keybinding. The default is control+spacebar. This can be changed in the options menu.
add: The BFA now adds burst fire to weapons that don't have it, like it's supposed to.
add: Examining attachments how lets you check their modifiers. Click the question mark icon in the corner of the examine window to do so.
add: Guns can now have different stats depending on the selected firemode. Most of the time, semi-auto will have lower scatter than the other modes, while burst fire will fire slower overall, but have lower scatter than full-auto.
add: Implemented CM-style scatter calculation. It only increases when firing on burst or full-auto. It's set to zero the moment you stop shooting. Full-auto and burst fire generally have higher scatter and are more adversely affected by the gun not being wielded.
tweak: The UGL can now only be fired if the gun it's attached to is wielded.
tweak: Added use delays for attachmetns. You can no longer spam toggle rail and grip lights.
tweak: All implemented weapon stats adjusted to match their CM counterparts.
fix: The smartpistol can now hold the barrel charger, the extended barrel, the recoil compensator, the suppressor, the rail light, the reflex sight, the red-dot sight, the burst fire assembly and the laser sight. Like it was supposed to from the start.
fix: The UGL and the masterkey no longer uncock if you shoot the gun they're attached to.
fix: The arm brace no longer causes an error when its user is handcuffed.
fix: The bayonet no longer deals 20 extra damage when used as a knife.
fix: Guns with size-changing attachments should no longer spontaneously shrink down and fit in your pockets.
fix: Xenos are no longer capable of removing attachments from items.
fix: RDS removed from the SG equipment rack, as its only modifiers are for an unimplemented mechanic and it's not supposed to be publicly available yet.
fix: The rail light now fits on the smartgun.
fix: The MK2 UGL can no longer shoot further than 7 tiles away. The UGL no longer overshoots your cursor.

The code is ready to be reviewed. Made this as a draft to make sure it's not merged before I get the HPR sprites. Once I have them, I'll add them in, fix the attachment offsets on it, and make a video that showcases the thing, and then this will be ready for merging.

I have no idea why it's saying the conflicts are still there. Pretty sure I fixed them.

We should just drop the HPR addition for now so cepha can make the sprites on their own time, we can add the HPR later

We should just drop the HPR addition for now so cepha can make the sprites on their own time, we can add the HPR later

Removed it from public access.


Polar files

Hi, I saw on the docs that this takes .hrm polar files.
I have an H10 but haven't come across any way of getting the .hrm files from it yet, do you know how I can access them so I can provide your hrv tool the RRI?
Thanks

Hi @sambilbow, how are you doing? Sorry for the late reply, I am returning from vacations.
I've googled some specs about the H10 chest strap and found this answer from Polar support:
https://support.polar.com/en/support/how_long_a_training_session_can_i_record_with_h10
"H10 records HR data once per second and ..."
It says that the monitor record the Heart Rate every 1 second.
As far as I know, for an accurate Heart Rate Variability analysis, it is recommended to have the Heart-Rate recorded Beat-by-Beat, otherwise, Time and Frequency Domain methods won't be able to extract information about Heart-Rate fluctuations.
Despite that, probably there is a way to export the recorded data to a  .txt and/or .csv file, which the hrvmodule supports.
Let me know if I can help to analyse .txt or .csv files.
Cheers

Hi @sambilbow, how are you doing? Sorry for the late reply, I am returning from vacations.
I've googled some specs about the H10 chest strap and found this answer from Polar support:
https://support.polar.com/en/support/how_long_a_training_session_can_i_record_with_h10
"H10 records HR data once per second and ..."
It says that the monitor record the Heart Rate every 1 second.
As far as I know, for an accurate Heart Rate Variability analysis, it is recommended to have the Heart-Rate recorded Beat-by-Beat, otherwise, Time and Frequency Domain methods won't be able to extract information about Heart-Rate fluctuations.
Despite that, probably there is a way to export the recorded data to a  .txt and/or .csv file, which the hrvmodule supports.
Let me know if I can help to analyse .txt or .csv files.
Cheers


Need to setup CI so that it runs the tutorials on simulators for validation

Issue is transferred to https://github.com/Qiskit/qiskit-ibm-runtime/issues/176 since this repo is being archived


[BUG]pod_name is null

[BUG]pod_name is null
{"starttime":1686193822,"cgroupid":1,"pns":0,"pid":64825,"tid":64825,"uid":0,"gid":0,"ppid":0,"pgid":0,"sessionid":4294937234,"comm":"modprobe","pcomm":"","nodename":"","retval":0,"exe_hash":"-1","username":"root","exe":"-1","syscall":"execve","ppid_argv":"-1","pgid_argv":"-1","pod_name":"-1","cwd":"-1","tty_name":"-1","stdin":"-1","stdout":"-1","dport":0,"dip":"<IP_ADDRESS>","sport":0,"sip":"<IP_ADDRESS>","family":0,"socket_pid":0,"socket_argv":"-1","pid_tree":"","argv":"/sbin/modprobe ip_tables","priv_esca":0,"ssh_connection":"-1","ld_preload":"-1"}

issue received, I'll look into this case this weekend.

Can not reproduce the issue, since we get the from the /proc/<pid>/environ, would you please start a process inside the pod, like "nc -lvvp xxxx", and execute cat  /proc/<pid>/environ to see whether it contains POD_NAME or MY_POD_NAME


Fullscreen fails the first time when emulating iOS devices

Tested with Chrome Mobile emulation tool. Using Chrome 39.0.2171.95 m.
All iOS devices fail to lunch fullscreen mode the first time, returning "Uncaught TypeError: undefined is not a function". They do work after a second click/emulated tap.
Try it yourself here: http://examples.phaser.io/_site/view_full.html?d=display&f=fullscreen.js&t=fullscreen

@belohlavek Thanks for reporting this.
Unfortunately I do have a access to iOS; any further details (ie. the expression / line that causes the exception) would be appreciated.

iOS doesn't support the full screen API anyway, so this is probably just Chrome emulation messing up: http://caniuse.com/#feat=fullscreen

As @photonstorm said it's just Chrome emulation messing up, some other devices don't even react to the fullscreen API so this is clearly not a Phaser issue. Closing.

But the error message is a Phaser issue, per the contracts.
If the API doesn't work the ScaleManager should simply call the the onFullScreenError signal (and not display an exception).

The behaviour doesn't happen on device though, just in Chrome emulation. Feel free to fix it specifically for that if you want, but it feels like wasted time to me.

Oh, I misread that as "Chrome on iOS".


chore(blockifier): add a check on deploy syscall gas cost in the happ…

…y flows of deploy tests

This change is

crates/blockifier/src/execution/syscalls/syscall_tests/deploy.rs line 63 at r1 (raw file):
Previously, meship-starkware (Meshi Peled) wrote…
It's a good question. I am not sure; I think Arni wrote this test, maybe it's worth asking him.
@ArniStarkware  WDYT?
I think the objective is to confirm that if we are not calling the constructor the gas_cost is zero

Talked F2f.
We want to check that the inner call consumes no gas, but we do want to check that the Tx at large consumes some gas.