Connection refused issue when trying to check the exchange api

curl -u ${HZN_EXCHANGE_ROOT_USER_AUTH} ${HZN_EXCHANGE_URL}admin/version
curl: (7) Failed to connect to <IP_ADDRESS> port 3090: Connection refused
Can anybody help why is this issue??

double check if your HZN_EXCHANGE_URL still points to the correct ip address. I'm using DHCP so it might change after each reboot ....
J.

Not sure if this helps - depending on what steps you had been used before - this could be the start of research:
a) if you are on a private network this will return the ip address of your router:
export MY_IP=curl -s httpbin.org/ip | jq -r .origin
b) this will return your local hosts ip address:
export MY_IP=ifconfig | egrep 'inet ' | awk '{ print $2 }' | egrep -v '^172.|^10.|^127.' | head -1

EXCHANGE_AGBOT_NAME=agbot1
EXCHANGE_AGBOT_TOKEN=agbot1pw
EXCHANGE_URL=http://<IP_ADDRESS>:3090/v1
EXCHANGE_ROOT=root/root
EXCHANGE_PASSWORD=Horizon-Rul3s
EXCHANGE_ORGANIZATION=testorg
EXCHANGE_ADMIN_USERNAME=admin
EXCHANGE_ADMIN_PASSWORD=adminpw
&& ./prime-exchange.sh
openhorizon/amd64_exchange-api:2.12.3 is the version of the container running for exchange api
is there an issue with the values set can you please let me know

Closing the issue with export MY_IP=ifconfig | egrep 'inet ' | awk '{ print $2 }' | egrep -v '^172.|^10.|^127.' | head -1
Issue resolved.


ENGESC-9803. Add entitlement for enable/disable diagnostics (API - 50…

…3 if not enabled)
See detailed description in the commit message.

merging fix, as hotfix will be needed on this on Monday, so we have the build ready earlier


Fixed kelly helper setup

Pbrun -> Horus
atk rate go brrrrrrrr

earth-shattering PR, almost as much as #89


Style prop value must be an object

When using the library I get this annoying message in my console due to the Map’s style prop.
Style prop value must be an object from react/style-prop-object
Any plans on renaming this prop to avoid this type of warning? Perhaps theme would be a nice alternative. Thoughts?

I had the same problem, in object I used template string and after that all works well :thumbsup:
Something like that :arrow_down:

Oooh, I didn't consider that @Maciekbe89, that's clever. I'll give it a try.


kraken sell command executed three times instead of once

After running this sell command once I found in the kraken order history three times a selling with each 20% of the original bitcoin value, bottom line I sold 60%. First time I had an issue like this, so not sure if repeatable.
zenbot sell kraken.XXBT-ZEUR --pct=20

I am having same issue. It does not happen always.

Closing due to inactivity.


Slow Performance and crashes on large types

Currently, the extension slows down all hovers and can even crash when computing a very large type.

Refactored code to use home-brewed formatting, and removed as many heavy computation TS morph calls as possible. Closing for now.


CodeMonkeys.DependencyInjection.* is limited to Ninject and DryIoc

Which package of CodeMonkeys is your request related to?
CodeMonkeys.DependencyInjection.*
Is your feature request related to a problem? Please describe.
Currently a framework user can't implement new dependency injection providers.
Describe the solution you'd like
A solution to extend the framework dependency injection capabilities. Incl. support for accessing the underlying implementation.

Why?
He can use the interface IDependencyContainer and use any dependency injection framework he likes.

Yeah I looked up what I meant with this issue. Created it a little bit ago. The user can do this but can't use the base DependencyInjection package. Its currently only consumeable by the framework internal DI packages because of the InternalsVisibleTo stuff.
This ticket was about if it makes sense to open that/change that in some way.


channelInterpretation cannot be changed from 'discrete'

Newbie here. After bumping from v13.8.27 to v14.7.77 I started getting the following error when creating new components (Tone.Panner, Tone.Reverb, etc):
Uncaught DOMException: Failed to set the 'channelInterpretation' property on 'AudioNode': ChannelSplitter: channelInterpretation cannot be changed from 'discrete'
A simple example
const panner = new Tone.Panner(1).toDestination();
const synth = new Tone.AMSynth().connect(panner).start();
synth.triggerAttackRelease("C4", "4n");

The error appears right after calling Tone.Panner. No issue when just calling Tone.AMSynth
Many thanks,
Diogo

Unable to recreate this error. Is there more context that you can give? platform or other parts of your setup?
Here's an example of your code running. I had to remove synth.start() in the second line because that's not part of the API.

Ok my apologies, found the issue. Apparently some conflict was occurring when also calling a web audio API script:
https://jsfiddle.net/w0a1of8t/4/
Here it throws a "Script error", but running it locally I get the error mentioned before (both on Chrome and Firefox).
Both scripts appear to be identical, so I'm not sure where the problem comes from, or why it gives problems with Tone v14 but not v13. !

great! glad you found the issue


Update to rails 7.1

Thank you for the PR! However, I chose to do the changes myself to keep this in line with my other gem.
Still, some notes on your PR is that you change many things which are not related to the subject. Many changes are likely just from running a linter/formatter which makes the reviewing harder when there are actual changes at the same time.
I just release 0.1.1 with the compatibility. Regards


[monitoring v2] when upgrading from v9.4.20x -> 14.5.100, GKE sometimes fails with error no endpoints available for service "rancher-monitoring-operator"

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

deploy GKEv2 downstream cluster
deploy monitoring v9.4.20x on that cluster with default settings
let install finish and upgrade to 14.5.100

Result:

sometimes, this error is seen

Error: UPGRADE FAILED: failed to create resource: Internal error occurred: failed calling webhook "prometheusrulemutate.monitoring.coreos.com": Post https://rancher-monitoring-operator.cattle-monitoring-system.svc:443/admission-prometheusrules/validate?timeout=10s: no endpoints available for service "rancher-monitoring-operator"

Other details that may be helpful:

2/6 installs had this occur on fresh GKE clusters

this also happens when updating prometheusOperator.tls.internalPort
download the logs here

upgrading the installation without any changes after seeing this error will successfully deploy monitoring with the desired changes

Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): v2.5.8-rc11
Installation option (single install/HA): single

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): hosted, GKE

Machine type (cloud/VM/metal) and specifications (CPU/memory): 2vCPU, 3 nodes

Kubernetes version (use kubectl version):  v1.17.17-gke.3700

Docker version (use docker version):19.3.14

This seems to be a well-known and never-resolved issue in upstream e.g. https://github.com/prometheus-community/helm-charts/issues/727, https://github.com/prometheus-community/helm-charts/issues/108.
For anyone observing this issue, the core issue here is that kube-prometheus-stack / rancher-monitoring enables Admission Webhooks by default, which utilizes a feature for Dynamic Admission Control of PrometheusRules based on an endpoint exposed by the Prometheus Operator deployment.
As a result, every time a PrometheusRule is created or updated, the Prometheus Operator deployment must be up and ready to accept requests, but if a helm upgrade causes the Prometheus Operator deployment to be updated and introduces / updates PrometheusRules, it's possible that the Admission Webhook fails because no endpoints available for service "rancher-monitoring-operator", which results in the helm upgrade failing.
Workaround
Just trigger an update once more. Since the PrometheusOperator deployment would have already been upgraded on the failed upgrade, the endpoint that the admission webhook seeks to hit will still be up when triggering the second upgrade.

@gaktive @Jono-SUSE-Rancher @deniseschannon since this is a known issue with upstream with an easy workaround, I think we should move this out of 2.5.8.

For those finding this issue, I also was directed here via
Error: failed to create resource: Internal error occurred: failed calling webhook "prometheusrulemutate.monitoring.coreos.com": Post "https://rancher-monitoring-operator.cattle-monitoring-system.svc:443/admission-prometheusrules/validate?timeout=30s": dial tcp xx.xx.xx.xx:443: connect: connection refused

Deleting the half-baked rancher-monitoring and rancher-monitoring-crd apps, then finding and deleting related rancher-monitoring secrets, then installing Monitoring again pushed me past this error.
I was battling resources leftover from Rancher 2.4 monitoring which hadn't been completely removed despite disabling monitoring in Cluster Manager.
I also needed to update the certificate directory as referenced here: https://github.com/rancher/rancher/issues/29815

I was battling resources leftover from Rancher 2.4 monitoring which hadn't been completely removed despite disabling monitoring in Cluster Manager.

@krumware there’s a good chance the reason why these would not be completely removed is because you’re using Alerting V1. Try running https://github.com/bashofmann/rancher-monitoring-v1-to-v2#check-why-monitoring-v1-is-not-disabled to audit why this is the case

Hi @aiyengar2, I had already followed the steps outlined in the guides for deleting all alerts and alert groups, etc, and was unable to get the legacy monitoring running again, so there was no turning back. This was also run on a cluster without any custom alerts or dashboards, only those that came out of the box in 2.4 or older.  I will create a fresh cluster on 2.5 to check for resource parity with the cluster I just migrated.
I have another cluster which had the same problem of not being able to disable the V1 monitoring system completely that I will try the monitoring script on. (two test clusters before moving on to production clusters)
Should we need to run the migration helper on clusters with only the out-of-the-box monitoring and dashboards?
Final other question, we upgraded from a 2.4 version with monitoring 0.1.x. Should monitoring in 2.4 have been updated to 0.2.1 in Cluster Manager before following the instructions to migrate monitoring to the new operator? I'm curious if that upgrade needs to be clarified as part of the migration instructions.
Thanks!

Should we need to run the migration helper on clusters with only the out-of-the-box monitoring and dashboards?

If it has out-of-the-box monitoring and dashboards, you shouldn't need to do anything except click on the Disable Monitoring button. However, it's probably a good idea to run the script on any cluster since it will notify you / confirm if it is safe to update to V2 and is non-invasive (e.g. it only makes GET requests, to my knowledge).
If this script passes and you still have Monitoring V1 components pop back up, please file an issue under Rancher (the script will also tell you to do exactly that).

Final other question, we upgraded from a 2.4 version with monitoring 0.1.x. Should monitoring in 2.4 have been updated to 0.2.1 in Cluster Manager before following the instructions to migrate monitoring to the new operator? I'm curious if that upgrade needs to be clarified as part of the migration instructions.

Maybe I'm misunderstanding the question here, but Monitoring V1 and Monitoring V2 are entirely independent of each other. The core reason for why they cannot be installed together is because you cannot have multiple Prometheus Operators in the same cluster (which both charts, based on their defaults, attempt to deploy).
Therefore, if you are migrating from Monitoring V1 (at v0.1.x or v0.2.x) to Monitoring V2 (at 9.4.x or 11.5.x), it doesn't matter if you upgrade Monitoring V1 before upgrading if you are anyways going to fully uninstall Monitoring V1 before going to V2.

@aiyengar2 I understand on both, thanks!
I updated my existing documentation issue/PR, as seen in the above mentions

Hello,
I installed latest https://github.com/prometheus-community/helm-charts/tree/main/charts/kube-prometheus-stack
I am on MetalLB, KubeSpray, Ingress-nginx.
with hostNetwork: true in the value.yaml

Any way to overcome this.


'loss_dict' error from 'train_one_epoch'

Navigating through the code in 'train_one_epoch', running this line:
loss_dict = model(image,targets)
gives the error:

397         # RPN uses all feature maps that are available
--> 398         features = list(features.values())
399         objectness, pred_bbox_deltas = self.head(features)
400         anchors = self.anchor_generator(images, features)
AttributeError: 'tuple' object has no attribute 'values'

Can anyone help?

Related to https://github.com/pytorch/vision/issues/1699 and https://github.com/pytorch/vision/issues/1701
Sorry for the delay in replying, I was on holidays.

Hi, to solved this issue is requiered to list the images too:
images,targets = next(iter(train_loader))
images = list(image for image in images)
targets = [{k: v for k, v in t.items()} for t in targets]
output = model(images,targets)   # Returns losses and detections


Commit text on cmd/ctrl+enter

Just like the message editor on GitHub does

发送到发送到

History:
@vjeux had an idea https://github.com/excalidraw/excalidraw/issues/1158 to commit texts on enter click. And it was implemented within 2 days.
In those two days, i was frustrated because there was no way to commit a text item without clicking on esc and selecting it again.
While i was thinking of opening an issue to complain, i thought it would be faster to make a PR 😂
#1185
I think shift/cmd is better. But it's my personal preference ...

Oups I was too fast to merge it.. felt OK :) So let's have a follow up if needed.

no @lipis it's ok. Let's discuss this further but keep it merged 😆

i was frustrated because there was no way to commit a text

I hope you are less frustrated now, during these difficult times :)

Yes, i am. Thanks @lipis

Why is this closed @lipis?

@j-f1 fixed via https://github.com/excalidraw/excalidraw/pull/1185 with a followup https://github.com/excalidraw/excalidraw/pull/1188


chem dispensers should protect their jugs from explosions

Description
bombing chem is basically round ending since it destroys all chemicals
if chem dispensers protect jugs from explosions, which either remain in the frame for reconstructing or get ejected after prying the board out (either would be better than how it currently is) it would both be life saving and give a physical reason to make a chem dispenser instead of mixing things by hand

ngl containers passing through explosions is generally pretty garbage. It might be worth either removing it or changing it to be opt-in rather than opt-out. It seems that a majority of scenarios seem to prefer not having it, since containers are a very general engine-level concept.

ngl containers passing through explosions is generally pretty garbage. It might be worth either removing it or changing it to be opt-in rather than opt-out. It seems that a majority of scenarios seem to prefer not having it, since containers are a very general engine-level concept.

See: secoffs going off like fireworks, nukie medics pissing omnizine, nukies chain-exploding c4 when something explodes near them

i think it was changed to be storage/inventory instead of recursive container check idk why it happens

it should be opt in regardless, its caused a lot of issues since implemented


Wiki? Examples?

After reading whole README I still have no idea how this work, even "quick follow example" that is first example doesn't work. There is no wiki on how to properly start using this module. Could someone please tell me what I am doing wrong?

Thank you very much, will try it out as soon as I have some spare time :) It is actually windows server 2012 R2 running on VM.


About usage of  \tag_if_active:T and variants

In current latex-lab-toc.dtx https://github.com/latex3/latex2e/blob/develop/required/latex-lab/latex-lab-toc.dtx I see
\cs_new_protected:Npn \@@_toc_contentsline_begin:nnn #1 #2 #3 %#1 level, #2 content, #3 destination
  {
   \tag_if_active:T
     {
%    \end{macrocode}
% We detect the intended level by checking the value of<EMAIL_ADDRESS>...

but there is no usage of \tag_if_active:T in the  end part
\cs_new_protected:Npn \@@_toc_contentsline_end:n #1 %#1 level name
  {
    \int_compare:nNnF { \use:c{toclevel@#1} } > {\use:c{c@tocdepth}}
...

I guess this is normal?
Where should I expect \tag_if_active:T to occur?

I guess this is normal?

Well yes and no ;-). It doesn't really harm that the \tag_if_active:T test is missing at the end (at the begin it is mostly there to avoid that all this code runs unneeded), but it would be more consistent to test in both places. But this part of the code will be replaced by tagging-sockets once they are available and then this inconsistency will vanish.

I am planning to do this kind of code
\ExplSyntaxOn
\ifdefined\IfDocumentMetadataTF
\AtBeginDocument{%
  \IfDocumentMetadataTF{\tag_if_active:T{\Etoc@tagactivetrue\Etoc@etoctaggingtrue}}{}%
  }%
\fi
\ExplSyntaxOff

to set some Boolean at begin document which will influence whether I should worry about tagging. Very first use case is for the config points in \@starttoc as I don't use the latter, so I have t invoke these config points directly (also some etoc usages of \tableofcontents will not want any tagging).
Does it look ok?
I am worred that the \@starttoc config points may be used for other things than tagging.

Does it look ok?

The test looks fine (not sure if the \ifdefined\IfDocumentMetadataTF is really needed but it shouldn't harm).

I am worried that the @starttoc config points may be used for other things than tagging.

I'm not sure that I understand what the problem is. But do not rely on \@starttoc@cfgpoint@before to stay. At the time the code was written sockets didn't exist yet. This code will be updated in the next months. Generally we will take care that tagging code is in its own socket, so that tagging can be activated/deactivated easily.

I'm not sure that I understand what the problem is.

etoc does not use \@starttoc.  It will execute the \@starttoc@cfgpoint@{before,after} (or their successors) manually ;  at some internal places (emptines checks for the TOC) it of course does not do that ; user will have have an interface to disable tagging ; in brief these config points will be done conditionally on tagging.  But I was not sure their would be there in kernel only  for tagging matters.  So in future, I was worried inhibiting execution if not tagging would have consequences.

But do not rely on \@starttoc@cfgpoint@before to stay. At the time the code was written sockets didn't exist yet. This code will be updated in the next months. Generally we will take care that tagging code is in its own socket, so that tagging can be activated/deactivated easily.

Ok. Point taken. But I need to prototype.  For simple usage of etoc only for its \localtableofcontents things are already seemingly working fine.  I will update my tagging branch when upstream LaTeX changes. I will probably do a release at least in time for TL2024 insisting tagging support is unstable on my side, but this way people can try out.

Ok. Point taken. But I need to prototype.

Oh that's fine. After all we are prototyping too ;-).  I'm grateful for everyone who tries and gives feedback. And it would be interesting to see tagging examples of more complicated tocs made with etoc.


tab: include HOMEBREW_VERSION

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[x] Have you successfully run brew tests with your changes locally?

Proposed in #1592.

It may be worth having a default here rather than nil. Thoughts?

I thought about this, but I couldn't think of anything obvious. Any given version string would be incorrect.

Yep, seems reasonable. Thanks! @ilovezfs any thoughts?

Perhaps fall back to git describe --tags origin/master with some designation that it's bogus?

@ilovezfs Sorry, I meant of the general PR idea. We will always have HOMEBREW_VERSION set, I was just thinking about how to handle tabs that predate it but I don't think there's a good way (and for what we're using it for: nil would be fine)

I don't see any actual immediate harm in including it in the receipt, but I'm skeptical we'll ever use it in a trustworthy fashion. I think I'd still prefer a new field, rather than trying to make meaning out of incorrect data and a version string.
Since the problems concern preexisting tabs that will not have had this field at all, I think this would only be usable for some future scenario that has happened yet, unless the mere existence of the homebrew_version field is serving as a proxy for meaning the run time dependencies field is "v2" given that the timeline will roughly correspond to the actual fixes assuming they're imminent, at which point we might as well reduce it to "runtime_dependencies_version=2," since presumably we'll need to bump it again at some point for some currently unknown reason.


[ID] Rename "Job" Concept page.

Rename Job page name and update respective page contains Job links as mentioned in #28772
/language id
@kubernetes/sig-docs-id-reviews

I think somebody need to approve it @kubernetes/sig-docs-id-owners @kubernetes/sig-docs-id-reviews

/lgtm
/approve

I think somebody need to approve it @kubernetes/sig-docs-id-owners @kubernetes/sig-docs-id-reviews

In @kubernetes/sig-docs-id-owners only @irvifa can do this, she has wider approval scope, while we are only limited to sig docs id. Or maybe someone from sig docs, @sftim could you help us for approving this one since you seemed already done a review for this PR?


[FE-TASK-3808]  - Agency help question move

EASI-3705 - FEATURE
EASI-3808 - TASK
Changes and Description

Moved agencyOrStateHelp from page one of opsEvalAndLearning to page 2 of generalCharacteristics
Moved agencyOrStateHelp likewise in readonly
Updated queries to new task list location
Updated unit tests
Updated cypress test
Updated translation location

How to test this change

PR Author Review Checklist

[ ] Met the ticket's acceptance criteria, or will meet them in a subsequent PR.
[ ] Added or updated tests for backend resolvers or other functions as needed.
[ ] Added or updated client tests for new components, parent components, functions, or e2e tests as necessary.
[ ] Updated the Postman Collection if necessary.

PR Reviewer Guidelines

It's best to pull the branch locally and test it, rather than just looking at the code online!
Check that all code is adequately covered by tests - if it isn't feel free to suggest the addition of tests.
Always make comments, even if it's minor, or if you don't understand why something was done.

@patrickseguraoddball , I think you might need to merge main into the feature, and then merge the feature into this branch to correct some of the changed files showing (like the migrations that split the CR and TDL table)


Test cases looked up by boost macros

The test cases got found based on the boost test macros presence in code, i.e. one need exactly "BOOST_AUTO_TEST_CASE(foo_test)" (or any other test case macro) to be present in the test cpp for the foo_test to be found by the adapter. In case another macro is defined like
#define MY_TEST(name) BOOST_AUTO_TEST_CASE(SPECIAL_NAMING_MACRO((foo_test))
and later used as
MY_TEST(foo_test) {...}
it is not recognized as a test case.
In case the above mentioned macro SPECIAL_NAMING_MACRO is used directly in the BOOST_AUTO_TEST_CASE the adapter recognizes the macro name as the test name.

Hi @LaoTszy and thank you for your interest in the project.
First impressions seems that this issue is caused by source code discovery.
Could you kindly note if the following: Discovering (SourceCodeDiscoverer) is listed in 'Tests' section of the Output Window please? Note that using list content discovery, this issue should be resolved automatically. For more details, please refer to issues #120, #130 and #133. You could also force the use of list content discovery as explained in the Wiki.

Thank you for such a fast reply. And sorry for mistaking my local problems for the adapter issue.
Yes, the Discovering (SourceCodeDiscoverer) is there, right after it complains about missing debug symbols, which is probably the root of the problem. Although that is still unclear to me, since the pdb file is right there with the test executable.


[Task] Describe the Lenra app REST API

What should be done
Define an OpenAPI file to describe the Lenra app REST API

This is done


Opinionated evaluation framework for tap and target capabilites (Gold/Silver/Bronze)

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2829
Originally created by @aaronsteers on 2021-07-02 19:48:07

The Singer ecosystem has a LOT of great taps - and over time, clear best practices and must-have features have slowly emerged. This issue will attempt to enumerate what would make a "Bronze", "Silver", or "Gold" level of quality for a given tap or target.

🥉 "Bronze"-level capabilities - Entry-level stability and interoperability

:small_blue_diamond: Catalog (newer) or Properties (legacy)
:small_blue_diamond: Discovery to print catalog json

🥈 "Silver"-level capabilities - Good user experience across the board, fits most use cases

:small_blue_diamond: Catalog (not Properties, which is legacy)
:small_blue_diamond: State and incremental replication
:small_blue_diamond: Selection of streams and properties via catalog metadata
▪ACTIVATE message type tolerance (targets only, receiving ACTIVATE doesn't cause failure)

🏆 "Gold"-level capabilities - "just works" for every foreseeable use case

Log-Based Replication (if applicable)
Target Schema Mapping - (targets only)
:small_blue_diamond: Record flattening - (targets only, columnar only)
▪ACTIVATE message type support (taps and targets)
Complex data types - (for targets only, fallback to string rather than failing if data type is unknown)

:gift: "Bonus" capabilities - additional to the above. Not a level, just a list of additional features.

:small_blue_diamond:Stream Maps
:small_blue_diamond: BATCH message type
:small_blue_diamond: --about to print capabilities and config expectations
▪ --demo to send dummy data and config-less discovery (Upcoming)

:small_blue_diamond: = Features that come out-of-box with the SDK (or will soon)
▪ = Features assisted by the SDK but still requiring some developer effort
Please post in the comments if you'd like to propose additional items in this taxonomy - or to move something around.

View 32 previous comments from the original issue on GitLab


test

up

@jimmy128bit you should use https://reviews.llvm.org/
This is the review system for llvm :-)


Update 03-Library-Management.md

Resolver.combineDefaultResolvers's first argument is Vector[Resolver] with sbt 1.6.2. So update the 03-Library-Management.md.

Hi @liang3zy22,
Thank you for your contribution! We really value the time you've taken to put this together.
Before we proceed with reviewing this pull request, please sign the Lightbend Contributors License Agreement:
https://www.lightbend.com/contribute/cla


separate out linear versus non-linear power spectrum computations

It would be nice to enable more options for the linear versus non-linear power spectra. For example, right now to use the comic emu, you have to also run class even if this may not be needed.

This gripe of mine is related: https://github.com/LSSTDESC/CCL/issues/337

So @alexander-mead this PR would do the following.
Restrict the transfer function to one of

bbks
eisenstein_hu
boltzmann_class

It would restrict the non-linear matter power spectrum to one of

linear (e.g., no corrections to linear)
halofit
halo_model
emu

Any restrictions on combining them have been removed. Users be warned!
I think this addresses #337 at the cost of still having two lists. Given everything else, this seems least worst. Thoughts?

Also using the halo_model option with anything but class won't work until I do #599 later this week hopefully.

This sounds very sensible to me.

I suppose in the (far?) future it could be worth allowing users to initialise a cosmology object and then to compute lots of different non-linear P(k), which may end up going into different parts of the same cosmological analysis (e.g., Fast-PT for intrinsic alignments, halofit for non-linear P(k)).

That sounds nice, but I am going to punt hard on that for as long as possible.


PD Bugs: Pre-wet tip behavior doesn't match tooltip

current behavior
For "Pre-wet tip", the tooltip always says it will aspirate and dispense 2/3 of the pipette's max volume.
However, the implemented behavior is different:

Transfer: uses the given volume of the step, or the volume of the chunk when the given volume gets split into chunks. The unit test matches this behavior.
Consolidate: Uses the given volume. Unit test matches this behavior, but the name of the test is misleading (it references chunking, but consolidate doesn't chunk)
Distribute: Not implemented, test exists but is unfinished & skipped.

steps to reproduce
Check pre-wet tip
expected behavior
I think we agreed the 2/3 of pipette-with-tip max capacity is the way to go?
danger zone
Would cause an implicit change to loaded-&-resaved protocols if we change the behavior :/

We think this was accidentally removed from the bug backlog. Adding it back in.


Can't train on custom dataset: DataLoader worker process 0. AND object of type 'int' has no len()

Hi,
I'm trying to get YOLACT(++) to work with a custom dataset using Google Colab because I don't have acces to a powerfull GPU myself.  When I run train.py it loads the .json files from the config.py and says "Begin training". But then I get the following error and I have no clue where to start looking or what I'm doing wrong. I googled for both errors but I can't find any specific information on how to solve the issue.

Could someone please help me out here? If not I will go back to Mask-RCNN but I would like to get this working it annoys me

I presume that you are trying to do transfer learning with your own config and dataset. Try to search how to do transfer learning for Yolact++. https://github.com/dbolya/yolact/issues/36 will tell you how to do fine tuning. If you want to only modify the last soft-max layer, https://github.com/dbolya/yolact/issues/334 may be helpful.
On top of this, you need to call pretrained weights when doing the transfer learning (e.g., yolact_resnet50_54_800000.pth)


fix(devcontainer): override config values

https://github.com/SoftwareAG/cumulocity-devicemanagement-agent/issues/25#issuecomment-915094898
Instead of updating agent.ini, I modified devcontainer settings to override config values. This is because a serial and filenames for certs are determined after creation of the container.

closing in favor of https://github.com/SoftwareAG/cumulocity-devicemanagement-agent/pull/35


Possibly unintentional recursion in Dry::Schema::Result#message_set

Describe the bug
Dry::Schema::Result#message_set can call Dry::Schema::Result#errors, which recursively calls message_set, leading to a stack overflow on TruffleRuby.
To Reproduce
On TruffleRuby.
If we create a simple schema such as
s = Dry::Schema.define do
  required(:foo) { filled? }
end

and execute it with an object on which it will fail like this
r = s.call( { } )

Then you will get a stack overflow when trying to print the result object.
Analysis
Dry::Schema::Result#message_set calls the message compiler with the result ast. If the ast is not empty then the message compiler will do
        current_messages = EMPTY_ARRAY.dup
        compiled_messages = ast.map { |node| visit(node, EMPTY_OPTS.dup(current_messages)) }

The visit call resolves to a Dry::Schema::MessageCompiler.visit_predicate call, which builds an options hash containing various bits of information including :input which points to the original result object. It then does message[predicate, options] which results to Dry::Schema::Messages::Abstract#call which does cache.fetch_or_store(args.hash) do ... end. That attempt to calculate a hash of the splatted arguments will recursively hash on the options passed in ultimately hitting the original result object, whose hash method is defined vai Dry::Equalizer to be based on output and errors. Calling errors calls message_set, and we recurse round the circle again.
The stack overflow does not occur on MRI because Array#hash and Hash#hash use rb_hash internally and that checks exhaustively for recursion and substitutes 0 for recursive hashes.
We will look at fixing the recursion guards in TruffleRuby, but that is probably simply hiding an underlying issue as I doubt message_set and errors were intended to recursively call each other.
Expected behavior
message_set and errors should not recursively call each other.
Your environment

Affects my production application: NO
Ruby version: TruffleRuby RC17
OS: Oracle Linux 7.5

Thanks for this detailed investigation. I'll fix this in 1.0.1 soon.

@aardvark179 I've fixed it but it seems like travis is unable to install truffleruby https://travis-ci.org/dry-rb/dry-schema/jobs/529697296#L483


[Feature] Option to teleport tokens instead of moving them with keybind

Switch between moving/teleporting tokens on keybind so that the token won't trigger undesired template/MATT/terrain effects.
Aditional suggestion: make this configuration be unique to players and npcs tokens, so that you can choose whether the default warping is teleport or move to different types of tokens. (default move npc, default tp player, for instance, and switch them on keybind)

My preference would be to be able to set whether it should teleport or move automatically, not require a keybind to get a teleport.
I want my moves to be a teleport always.  I am typically using it when I am repositioning a PC from one part of the map to another (maybe they were afk for a bit, maybe they missed last session, etc.).  I never want them triggering anything while moving, I just want them to appear in the place I've dragged them.
Requiring a keybind to get this behavior is adding back the issue that was the whole reason I requested this code/module in the first place - I don't want to have to remember to use a modifier key to get a teleport and avoid having the token animate across the map or trigger things along the way.

Alright, will go over these during the weekend! I will definitely try to include both options!

It could be a keybind to toggle between options, instead of being a hold/release keybind.

Long time no see.
Made a PR in MATT to allow better integration, cause any other way I was going about, caused issues with Drag Ruler's integration.
Should be out as soon as MATT updates (and if) 🤞
My initial thinking was a boolean setting to bypass MATT triggers if the Token Warp cancels movement animation.
If needed, I could support also a keybind to override, but at that point, I would say that Drag Ruler's teleport keybind works too (at least for most users which in my mind use Drag Ruler).
WDYT? 🤔

Next MATT release will make sure that when a preUpdateToken hooks passes animate: false, no Tiles will be triggered by that "movement".

MATT 11.18 handles interaction with modules which for any reason pass a animate:false parameter in the preUpdateToken dnd5e hook, better.
When that parameter is encountered, MATT will disallow triggering of tiles in the path of the dragged tokens.
Slight change to the logic of Disable token animation movement by default is therefore needed, as with that setting now on, no MATT whatsoever can be triggered.
I will probably rework that setting to only affect dragging tokens around the map, whereas using the keyboard for moving will trigger tiles normally.


Fix bsongen_test skipImports function.

@sougou
It was also sometimes skipping the comment, which caused test failures
in Travis.

Changes Unknown when pulling b0c6b0630fc23f0e41681f137f35e49b413fd346 on bsongen_test into ** on master**.

lgtm


Fix get test loss from python frontend

Right now we read the tail of the log file from the ErrorSparseTask.
We should probably create an UDP/TCP interface in the error task,

#50


Flickering

Need a low latency filter function that can be implemented using glsl.

https://github.com/natural-harmonia-gropius/hdr-toys/commit/cd2d78f6b61a1f6c2ea898a3ab1c968a10670f5b
8 frames performed well in my test sample and hopefully did not bring any other regressions.


Test Accounts PHONE_CODE_INVALID

I used the provided Example and only made the following modifications:

apiId
apiHash
request.useTestDc = true;
Then use the test account

I would like to ask what is the test account code?

I also experienced this, I thought I was stupid

@levlam
when will this be fixed?
because if you use a new account it will be banned automatically

Perfect kub , faster
On Thu, 17 Oct 2024, 22:09 AZKA, @.***> wrote:

@levlam https://github.com/levlam
when will this be fixed?
because if you use a new account it will be banned automatically
—
Reply to this email directly, view it on GitHub
https://github.com/tdlib/td/issues/3083#issuecomment-2419816869, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BJ4PV43IGDRLQ63HPYMIJMDZ37HLPAVCNFSM6AAAAABPY66WP2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMJZHAYTMOBWHE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I see the issue too.
`
% ./tg_cli --test --api-id=94575 --api-hash=a3406de8d171bb422bb6ddf3bbd800e2 -v 1
[ 1][t 1][1729729075.190335988][cli.cpp:1465][!Td]	Receive result [1][id=1] Ok {
}
[ 1][t 1][1729729075.190428018][cli.cpp:1465][!Td]	Receive result [1][id=2] Ok {
}
td_cli> sap +9996621234
[ 1][t 1][1729729080.874551057][cli.cpp:1465][!Td]	Receive result [1][id=3] Ok {
}
td_cli> cac 22222
[ 1][t 1][1729729086.659242153][cli.cpp:1611][!Td]	Receive error [1][id=4] Error {
code = 400
message = "PHONE_CODE_INVALID"
}
`

Same issue here. On test dc 2.

It's been a long time, has anyone fixed it?
I tried it earlier, it's still the same

Also have the same issue. While the test api isn't working spent 3 accounts being banned testing on production(I know it is not recommended to test on production, but had no other option). It would be nice to notice developers through docs about
non-working test environment. Although, why am I surprised? Feedback is not held in high esteem by both support and developers of Telegram.

still experiencing this issue as well.

@levlam PTAL, gentle ping.
Can't perform automated QA without test accounts, pretty critical issue.
I fully rely on it to make 100% sure my libraries work correctly.
Please fix it, really need it!

The same problem, please fix...

Is the test DC environment still supported?

can we bump this issue, i cant get any test phone register as per the documentation without getting a phone code invalid error

I also encountered this problem, what options do we have? If I use a real account, the system logs me out of all devices after successful authorization using TdLib. I recently started digging into this, maybe I don’t understand how to test authorization correctly?


Pie chart - add an option to allow labels being oriented perpendicular/horizontal to the tangent lines

Demo: http://jsbin.com/jufuniniku/2/edit?html,js,output
Pie chart labels often overlap on top of each other, making things difficult to read. It would be helpful if an option can be implemented allowing the label texts to be placed horizontally (and/or perpendicularly) against the local tangents, so that the labels will looks like being radiated from the center point of the chart and won't overlap on top of each other.

This is kind of a duplicate of #224 . We need to find a simple and nice way to solve this issue. Let's keep track of this in the earlier ticket.


Add an About Pokemon franchise page

There are now empty About pages regarding the Pokemon franchise, Pokemon Showdown, and the ToBeTheBest extension

#75


Prefer File.exist? to avoid deprecation

This very small change avoids a deprecation warning in later Rubies.

Released in 1.2.0


Scrolls far slower in Chrome than any other browser

I'm sure its something to do with my implementation but if you view the following site in Chrome it seems to have a long delay before scrolling between sections... whereas Firefox and Edge scroll it instantly. Any Ideas?
https://gearworks.000webhostapp.com/

Found it. For anyone else experiencing this issue, in my case the cause was applying scroll-behavior: smooth to the body tag. Removing this css fixed the issue.


Page content not current

This page is listed as being a part of .Net 6, but several references seem to be out of sync.  The page does mention that, due to changes in .Net 6, the WebApplicationFactory must be positioned to reference the Program class since Startup is no longer used, but the following paragraphs that involve the WebApplicationFactory still reference the Startup class.   I am in the process of converting an application from .Net Core 3.1 to 6 and updating the integrations tests is not going well.  I'm getting errors in every test that make no sense, and this page is not helpful at all.  I have created two custom instance of WebApplicationFactory, and now after upgrading every project to .Net 6, none of my integration tests pass.  Not only do they all fail, but they fail because of exceptions that occur in the custom WebApplicationFactory classes, so the tests are not actually being executed due to problems in the initialization of the tests.

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

ID: 77889d43-fedb-f8b4-8fea-379ec946555c
Version Independent ID: f499b131-8b14-7e10-1728-543fdd5a5656
Content: Integration tests in ASP.NET Core
Content Source: aspnetcore/test/integration-tests.md
Product: aspnet-core
Technology: aspnetcore-test
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Duplicate of these.


Support 'group' entries in sidebar?

I also have for stories that include the meta with documentation (+ stories) but also include some manual .mdx files with code examples. I noticed that for these folders, the tags also don't show up.
(but maybe that's an odd setup I use..)
(for these 'groups' the icon is also a folder, instead of the component square icon)

Originally posted by @Fedduh in https://github.com/Sidnioulz/storybook-addon-tag-badges/issues/15#issuecomment-2482303149

@Sidnioulz it's probably this PR from a long time ago that I forgot about. Will try resurrecting it now https://github.com/storybookjs/storybook/pull/26633

@Sidnioulz Resurrected here https://github.com/storybookjs/storybook/pull/29655 and it should already be supported in the index

@shilman thanks!
When I get round to addressing this, I'll open a PR to extend the tags intersection logic to group entries then.

Storybook PR merged! I'll work on this once Storybook 8.5 is officially released :)


Update documentation on linting and formatting, and add gh action workflow to run linter on new pull requests

completed in #31


feat: show in-progress upgrades w/ some kind of TUI library

Probably bubbletea would be pretty nice for that.

THIS https://github.com/elulcao/progress-bar but somewhat modified

Neeeever mind! We will just wait until this is done https://github.com/containers/bootc/issues/924


New version: TeaSeis v0.4.0

Registering package: TeaSeis
Repository: https://github.com/ChevronETC/TeaSeis.jl
Created by: @samtkaplan
Version: v0.4.0
Commit: b2d71cb252f55c5998a1e77cdae75ea381ea983b
Reviewed by: @samtkaplan
Reference: https://github.com/ChevronETC/TeaSeis.jl/commit/b2d71cb252f55c5998a1e77cdae75ea381ea983b#commitcomment-41226044

I will add a compat entry for LightXML, and try again.


Update Pages “about”

Automatically generated by Netlify CMS

👷 Deploy Preview for vakumposeti processing.
🔨 Explore the source changes: 71fbc0932b0b0bcfe600d97cec7e6c66f31a6682
🔍 Inspect the deploy log: https://app.netlify.com/sites/vakumposeti/deploys/61d373976cf9610007b219f8


plugin.setOptions should be able to keep user provided settings

in the goal of moving presets to external plugins (#249) we need to allow users to override the plugin settings, right now this isn't possible since plugin.setOptions is called as the last step and will override all the settings.
I think we should call plugin.setOptions() passing an empty object { esformatter: {}, indent: {}, lineBreak: {}, whiteSpace: {} } and later we deepFillIn the default options and merge the user settings (overriding the default and plugin settings).

ahh, we also need to pass any other property that is not esformatter, indent, lineBreak, whiteSpace to the plugin.setOptions call otherwise plugin might not work as expected (eg. it expects some option to toggle the behavior..)
we should make this clear somehow..

ok, I'm just dumb.. we can pass the FULL user settings to plugins making sure they contain all the required properties (esformatter, indent, lineBreak, whiteSpace) and later we merge the user settings again overriding any changes that the plugin might have done.. that should give the expected result.


TiKV duration is high when TiFlash is down

Bug Report

What version of TiKV are you using?

v6.5.0
What operating system and CPU are you using?

Not related
Steps to reproduce

17 TiKV, 10 TiFlash with 2 millions regions, run sysbench workload, down all TiFlash nodes and check the duration.
What did you expect?
No difference.
What did happened?
Duration increases.

related issue https://github.com/tikv/tikv/issues/13396


headwind and Solidjs class support?

I cant get headwind to work in Solidjs tsx component that is using class instead of React using className
i tried but not working :(
  "headwind.classRegex": {
    "html": "\\bclass\\s*=\\s*[\\\"\\']([_a-zA-Z0-9\\s\\-\\:\\/]+)[\\\"\\']",
    "javascriptsolid": "(?:\\class\\s*=\\s*[\\\"\\']([_a-zA-Z0-9\\s\\-\\:\\/]+)[\\\"\\'])|(?:\\btw\\s*`([_a-zA-Z0-9\\s\\-\\:\\/]*)`)"
  },

I know this is technically a necropost, but I feel like others will come across this in the future and want a solution.
"javascriptsolid" isn't a file type recognised by VSCode. All JSX and TSX files match to "javascriptreact" and "typescriptreact" respectively.
With that in mind, the following regex can be used that maintains support for both className and class:
// settings.json

{
    /* Other settings... */

    "headwind.classRegex": {
        "javascriptreact": "(?:\\b(?:className|class)\\s*=\\s*[\\\"\\']([_a-zA-Z0-9\\s\\-\\:\\/]+)[\\\"\\'])|(?:\\btw\\s*`([_a-zA-Z0-9\\s\\-\\:\\/]*)`)",
        "typescriptreact": "(?:\\b(?:className|class)\\s*=\\s*[\\\"\\']([_a-zA-Z0-9\\s\\-\\:\\/]+)[\\\"\\'])|(?:\\btw\\s*`([_a-zA-Z0-9\\s\\-\\:\\/]*)`)"
    }
}

The regexes replace className with the non-capturing group (?:className|class).
P.S. You may need to restart VSCode after updating settings.json.
Hope this helps 👍


How about using insert template on the next line after the cursor instead of at the beginning?

How about using insert template on the next line after the cursor instead of at the beginning?
That way it could be used like a snippet like vim-sonictemplate!

supported.


unauthenticated chart in simple web page gets "Refused to display..."

The documentation states,

For the "Unauthenticated" option, you can copy the provided snippet and include
it in any web site; no extra steps are required.

I have Apache 2.4 running locally. My Chrome is Version 73.0.3683.103 (Official Build) (64-bit)
For the Chart, I have set Enable unauthenticated access: ON. I copy the embed URL text and place it in a web page,
<html>
<body >
<iframe
  style="border: none;border-radius: 2px;box-shadow: 0 2px 10px 0 rgba(70, 76, 79, .2);"
  width="640"
  height="480"
  src="https://charts.mongodb.com/charts-viz-nqlhg/embed/charts?id=SNIP&tenant=SNIP">
</iframe>
</body>
</html>

The chart flashes and then goes away. The console shows,
Refused to display 'https://www.mongodb.com/cloud' in a frame because it set 'X-Frame-Options' to 'deny'.
The URL works fine by itself (outside the IFrame)
https://charts.mongodb.com/charts-viz-nqlhg/embed/charts?id=SNIP&tenant=SNIP
For reference, see
https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/X-Frame-Options

Hi @johnnymaxwell - apologies for the very slow response. It looks like the frame was redirecting to https://cloud.mongodb.com which is a completely different URL that doesn't like being rendered in a frame. A valid Charts embed URL should not be redirecting, but if the URL thinks it needs an authenticated session it may do that. Since you asked the question we have deployed a couple of updates - do you still see this behaviour?

Hey @tomhollander. I wanted to try without authentication and copied the link is given by MongoDB, still gives HTTP header issue. What action should we take to embed on a web application in an iframe?

OK, sorry to hear it's still not working. The problem is still probably what I described though - the page that's refusing to load in the page is cloud.mongodb.com, not charts.mongodb.com, so there's a redirect happening for some reason. I can't test your chart because of the redactions, but please try my sample chart at https://charts.mongodb.com/charts-tomh-yfpqi/embed/charts?id=ed1cc666-850e-4f42-8b58-b6c85424ac36&tenant=a73f55eb-58b5-4318-967b-50ac703df9db and let me know if it can be embedded, and also if it helps you understand why the redirect is happening. You could also try loading your embed URL as a full page (not in an iframe) to see if that gives you any clues.

After a bit more investigation it seems that charts-viz-nqlhg does not resolve to a Charts on Atlas tenant. Are you sure the URL is correct?

Hey @tomhollander! Sorry for the lateness. Yes, It's correct URL that given by you.  I guess parameters are causes the issue.

@bsonmez Can you send the exact URL you are trying to load in an iframe? Feel free to email me at tom.hollander at mongodb.com if you don't want to share publicly.

I have the same issue.  Here is my url.

Hi @gofreddo
I'm able to load that chart in an iframe without any issues. What browser/OS are you using?

Brave
Version 0.68.138 Chromium: 77.0.3865.75 (Official Build) (64-bit)

Hi @gofreddo, @johnnymaxwell  and @bsonmez, apologies for the delay in responding. We have observed this behaviour when running browser plug-ins that selectively block certain scripts from running. Do you have any browser extensions running?

@tomhollander Hey Tom! Thank you for respond. What I guessed is the same. Most probably AdBlock or similar scripts block the connection. I have some extensions so I believe that was the cause.

Brave blocks ads by default.

Ok. There are no ads being served, but it seems we may have pinpointed the cause. I'll see if there's anything we can do about this.

So for Brave, the issue seems to be caused by the default cookie blocking behaviour. If you change from the default setting "only block cross-site cookies" to "allow allow cookies" then the charts load correctly. We'll investigate if there's anything we can change on our side to work with the more aggressive cookie blocking but changing the setting may get you unstuck in the meantime.

We've found a solution to this, and we should have it fixed in our December release of Charts.


Add iOS refresh frontend cache reset instructions to README

Closes #417

@junior466 I've tried to add a troubleshooting entry in case someone else runs into the problem you reported on #417. I don't have an iOS device, would you mind verifying these instructions and let me know if I could make it any more accurate/useful? (i.e. are they the exact menu option names, etc).
Thanks!

@dermotduffy Sure thing. Exact instruction:
In the app, select Configuration->Companion App->Debugging. You will now see the option to “Reset frontend cache”. It’s pretty straightforward process.
I actually only thought of it due to how when opening an issue here requires you to select that you have “cleared the browser cache”. That made me quickly realize that the app must also have some sort of cache.


Read in architecture from a json definition

Instead of having hard coded netflixoss, lamp, migration architectures, have them defined as json files and use a single architecture.go package to read them in and launch them.
Started coding architecture.go - it's not part of the build yet.

Implemented first version, works with
spigo -a test
which reads in json/test_arch.json or whatever the file is named


Getting the market api without the token and cookies

https://developers.veliainn.com/ Instead of "acting" as the web browser. This act as the client. I feel like forwarding this to you since I used your code.

what is this, is a bdo api? unnoficial or official?

what is this, is a bdo api? unnoficial or official?

It is the api that used by the game client. The endpoint is diferrent from web browser. The data response are diferrent too.

It is the api that used by the game client. The endpoint is diferrent from web browser. The data response are diferrent too.

It is the api that used by the game client. The endpoint is diferrent from web browser. The data response are diferrent too.

yeah i joined discord, this is definetly the way to go if u plan on make something with market, u dont need account + u can make each request with different proxy and will be total different "person" for api. Thanks for posting.

That returns a 404 tho. Is there an update on the transfer to PA and an API associated with it?


Possible to specify name attribute of file?

Works great, but is it possible to set the name attribute on the file input? I'd like to use several uploaders on a page. After the upload, I'm having trouble figuring out which file came from which uploader.

I would either have separate endpoints on the server for different upload types, or have separate methods in your component using the uploader - each could send an extra identifying post var.

Thank you for the reply. Much appreciated.


FDA FSMA Food Traceability List reference

It would be good for FoodOn to review and ensure coverage for the FDA Food Safety Modernization Act (FSMA) Section 204 - Food Traceability List which is due to provide farm-to-fork traceability of higher-risk foods by 2026.
https://www.fda.gov/food/food-safety-modernization-act-fsma/food-traceability-list

+1 There are standard objects in there that may be interesting.


Invalid Hook Call

Describe the bug
I'm getting an Invalid Hook Call error when using <ClientOnly>.
If I comment out the ClientOnly tags, it works fine. When I put ClientOnly in, I get the error below

Error: Invalid hook call. Hooks can only be called inside of the body of a function component. This could happen for one of the following reasons:

You might have mismatching versions of React and the renderer (such as React DOM)
You might be breaking the Rules of Hooks
You might have more than one copy of React in the same app
See https://reactjs.org/link/invalid-hook-call for tips about how to debug and fix this problem.
at resolveDispatcher (/Users/connorvanooyen/Desktop/node_modules/react/cjs/react.development.js:1476:13)
at Object.useState (/Users/connorvanooyen/Desktop/node_modules/react/cjs/react.development.js:1507:20)
at Object.useHydrated (/Users/connorvanooyen/Desktop/node_modules/remix-utils/build/react/use-hydrated.js:24:43)
at ClientOnly (/Users/connorvanooyen/Desktop/node_modules/remix-utils/build/react/client-only.js:22:27)
at processChild (/Users/connorvanooyen/Desktop/Yellowbrick/yb-stats-dash/node_modules/react-dom/cjs/react-dom-server.node.development.js:3353:14)
at resolve (/Users/connorvanooyen/Desktop/Yellowbrick/yb-stats-dash/node_modules/react-dom/cjs/react-dom-server.node.development.js:3270:5)
at ReactDOMServerRenderer.render (/Users/connorvanooyen/Desktop/Yellowbrick/yb-stats-dash/node_modules/react-dom/cjs/react-dom-server.node.development.js:3753:22)
at ReactDOMServerRenderer.read (/Users/connorvanooyen/Desktop/Yellowbrick/yb-stats-dash/node_modules/react-dom/cjs/react-dom-server.node.development.js:3690:29)
at renderToString (/Users/connorvanooyen/Desktop/Yellowbrick/yb-stats-dash/node_modules/react-dom/cjs/react-dom-server.node.development.js:4298:27)
at handleRequest (/Users/connorvanooyen/Desktop/Yellowbrick/yb-stats-dash/api/index.js:48:49)

Your Example Website or App
localhost
Steps to Reproduce the Bug or Issue

Create Remix Run app
In Index, import and add ClientOnly with any test code in it
Run npm run dev
Error

Expected behavior
Should not have error
Screenshots or Videos
No response
Platform

OS: macOS
Browser: Chrome

Additional context
No response

Are you using React v18?

No.

Please try to create a reproduction repository so I could debug it


[CSL-2432] Generate arbitrary 12 word backup phrase

This issue is fixing:

wrong implementation of Arbitrary BackupPhrase which picked arbitrary 12 words from bip39 english mnemonic dictionary

this is not sufficient to be bip39 compatible as it doesn't take checksum into account

Arbitrary (V1 BackupPhrase) is using correct 12 word BackupPhrase implemented above instead of correct hardcoded BackupPhrase
documentation should contain arbitrary BackupPhrase now
problems with integration tests which are relying on generating arbitrary BackupPhrase for generating new wallets - https://iohk.myjetbrains.com/youtrack/issue/CSL-2398

This issue is not fixing:

the distinction between paper vend backup phrase (9 words) and normal backup phrase (12 words)

this should be handled with https://iohk.myjetbrains.com/youtrack/issue/CSL-2048
we gain only stronger type safety

documentation would be more correct with the distinction above as some endpoints are using peper vend backup phrase (and documentation is showing 12 words mnemonic instead of 9 words)

note that most of the certificates are already redeemed (http://cardanoexplorer.com/genesis) so with time this is less and less important - but still incorrect (should be handled with https://iohk.myjetbrains.com/youtrack/issue/CSL-2048 )

QA
Tested with ghci (with temporarely adding default Show instance to BackupPhrase):
> import Pos.Util.BackupPhrase
> import Test.QuickCheck
> (generate $ arbitrary) :: IO (V1 BackupPhrase)
BackupPhrase {bpToList = ["humble","neither","mixed","jump","exile","heavy","pill","minute","area","inspire","clinic","manual"]}
> (generate $ arbitrary) :: IO (V1 BackupPhrase)
BackupPhrase {bpToList = ["plate","phrase","evolve","maximum","pluck","toward","faint","film","bargain","casual","certain","give"]}
> (generate $ arbitrary) :: IO (V1 BackupPhrase)
BackupPhrase {bpToList = ["pave","actress","picture","coil","smoke","few","crowd","dutch","inflict","draw","lens","effort"]}

Each call generates different arbitrary bip39 12 word backup phrase.
Passing multiple wallet generations in integration test:
[TEST-LOG] Running: GetAccount
[TEST-LOG] Running: PostWallet
[TEST-LOG] Request: NewWallet {newwalBackupPhrase = <backup phrase>, newwalSpendingPassword = Just <passphrase>, newwalAssuranceLevel = NormalAssurance, newwalName = "Wallet", newwalOperation = CreateWallet}
[TEST-LOG] Success!
[TEST-LOG] Running: UpdateAccount
[TEST-LOG] Running: PostWallet
[TEST-LOG] Request: NewWallet {newwalBackupPhrase = <backup phrase>, newwalSpendingPassword = Just <passphrase>, newwalAssuranceLevel = StrictAssurance, newwalName = "Wallet", newwalOperation = CreateWallet}
[TEST-LOG] Success!
[TEST-LOG] Running: GetAddress
[TEST-LOG] Running: UpdateAccount
[TEST-LOG] Running: PostAddress
[TEST-LOG] Running: PostAddress

I'm curious about how the "guaranteed not to fail" bit works out, and if we can implement a test for it.

only by code review. The bits that generate bip39 are from here https://github.com/input-output-hk/cardano-sl/blob/master/wallet/src/Pos/Util/Mnemonics.hs#L51 and it is a pure function that generates failure only if it doesn't satisfies some of the conditions:

"entropy must be a multiple of 4 bytes"

which it is as we are using 16 bytes for 12 word mnemonics https://github.com/input-output-hk/cardano-sl/pull/2759/files#diff-f78b72ef586b87d206550724ec229577R39

"maximum entropy is 64 bytes (512 bits)"

the same answer as the above.

"non-ASCII characters not supported"

Hm - this is a bit tricky to review as I have to dig into ByteString and toMnemonic internals. And I have just tried to do so for 15 min - but concluded that it is easier in this case to just make a safe loop as you have suggested :) Thanks for a careful review!


Fix pulse drawer bug for truncating Acquire command

Summary
Truncation for Acquire command is not properly implemented. Pulse drawer crashes when we try to truncate Acquire command with plot_range option. In this PR, pulse length of Acquire is adjusted to fit in with specified plot range.
Fix #3076
Details and comments

You mean adding a test just to check if it has no error? or adding a test with new reference figure?

Adding a test to make sure it has no error, thank you.


Infix operator support

Currently, we have to do this awkward mess
a := (a) + (b).

If we forget the parentheses, + gets called with no arguments and then b gets called on +. Ideally, we could designate all symbols which are made up of certain symbols [which ones?] as "infix operators" which always take an argument.

The above definition of "operator" has been changed in accordance with #21, as of 4caf83b.


Added js5_6_1.html

Added js5_6_1.html (cloning and inserting)

Added js5_6_1.html (cloning and inserting)


Lecture 06 - definitions

I used ?X etc. for unification variables (as in the lecture), which doesn't look good (IMO) in rendered pdf. Let me know if you want me to change it to something else.

Woudn't it be better to add a new macro with a parameter? Such as
\newcommand\UVar[1]{?\!{#1}}

Writing math in LaTeX is different than usual coding. Usually, user defined macros for syntax plays two roles: they make easier to typeset math and make easier to change the notation in the entire document. If we change our mind, and decide to use e.g., Greek letters, this general macro with one parameter would not help, in opposite to these more specialized ones. We can consider the combined approach:
\newcommand\UVar[1]{?\!{#1}}
\newcommand\UVarX{\UVar{X}}
\newcommand\UVarY{\UVar{Y}}
\newcommand\UVarZ{\UVar{Z}}

The gain is marginal: macro \UVar is (and should be) used very locally, to define the next three macros, but creates temptation to use it in other places. This in turn will ruin our hope for ease of change of the notation.


Add test CI of github actions

🚀 Feature Request

CI/CD is very important for promptulate. Add test of github actions is necessary!

has done


Refactor nested generic display in TypeNameHelper

This implementation for getting display name of nested generics types uses Type.DeclaringType to display it instead splitting full name string and parsing int values in it.

@AndreyTretyak,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@smitpatel @pranavkm  Could you please have a look at this or other PR for TypeNameHelper? It will help me avoid conflict or resolve them faster.

There are no tests. What does this solve? What is the issue? Avoid moving around code/changing parameter names, order. It makes harder to diff.

It removes the need of parsing string with full type name and takes info directly from type object.
This'll reduce amount of allocations during generics name creation.
It will not change external behavior, that's why I haven't change tests.

I've maid simple benchmarkin app, here results:
                Method |      Mean |     Error |    StdDev |  Gen 0 | Allocated |
---------------------- |----------:|----------:|----------:|-------:|----------:|
     TestOldSingleType |  2.380 us | 0.0446 us | 0.0438 us | 0.3128 |    1316 B |
     TestNewSingleType |  2.354 us | 0.0136 us | 0.0127 us | 0.1678 |     704 B |
 TestOldDifferentTypes | 43.553 us | 0.3613 us | 0.3380 us | 5.7373 |   24281 B |
 TestNewDifferentTypes | 37.777 us | 0.1897 us | 0.1584 us | 3.3569 |   14105 B |

TestOldSingleType - calls current implementation for type Outer<int>.F<int, Outer<int>.E<string>>.
TestNewSingleType - calls PR implementation for type Outer<int>.F<int, Outer<int>.E<string>>.
TestOldDifferentTypes  - calls current implementation for all types that were used in unit tests.
TestOldDifferentTypes  - calls PR implementation for all types that were used in unit tests.

@AndreyTretyak - Thanks for info. Approach looks good.

@ajcvickers - We use similar code in EF Core also. Do we want to update EF Core repo too?

@smitpatel Looks like we should--can you open an issue?

I've made small update to remove second call to GetGenericArguments() method.
In benchmark results methods with updated version named  TestUpdatedSingleType and TestUpdatedDifferentTypes .
                    Method |      Mean |     Error |    StdDev |  Gen 0 | Allocated |
-------------------------- |----------:|----------:|----------:|-------:|----------:|
         TestOldSingleType |  2.446 us | 0.0488 us | 0.0501 us | 0.3128 |    1316 B |
         TestNewSingleType |  2.403 us | 0.0288 us | 0.0269 us | 0.1678 |     704 B |
     TestUpdatedSingleType |  1.994 us | 0.0267 us | 0.0250 us | 0.1564 |     660 B |
     TestOldDifferentTypes | 44.175 us | 0.7055 us | 0.6600 us | 5.7373 |   24281 B |
     TestNewDifferentTypes | 38.424 us | 0.6892 us | 0.6446 us | 3.3569 |   14105 B |
 TestUpdatedDifferentTypes | 30.140 us | 0.6014 us | 0.5022 us | 3.1738 |   13400 B |

@AndreyTretyak I was playing around with how GetGenericArguments() works with nested types to see why you had this method written this way. I think we can further simplify this method. Could you have a look at https://github.com/aspnet/Common/pull/281/files and see if it makes sense or if I'm just missing some scenario?

GetGenericArguments() returns number of generic args for current type including the types needed for declaring types too. For a closed generic it returns actually array of types. But a declaring type of nested closed generic is open generic (why???? alas breaking change so that is not gonna change ever). But as @pranavkm pointed out it still has correct counts.
Inside a nested type, how many generic args do I have? GenericArgs of myself - GenericArgs of declaring type. If this is 0 we can return without printing generics.
Unless there is any specific benefit of using ArraySegment, lets just work with simple math here. We have a full array. We know offset for each type (from declaring type), We know how many generics each type has. So go over only that part of array and it will print out things correctly. Also let's just use recursion to return how many arguments have been processed so we don't need to go to declaring type again.

I've tried option with recursive method that returns offset and it results in a duplicated call of GetGenericArguments() for the initial type. Judging from profiling results it's quite expensive so I would like to avoid it.
Here simple pseudo-callstack for Level1<int>.Level2<bool>.Level3<int> to ilustrate my point:

call GetGenericArguments() for Level1<int>.Level2<bool>.Level3<int>
call ProcessGenericTypeWithReturn for Level1<int>.Level2<bool>.Level3<int>
call ProcessGenericTypeWithReturn for Level1<int>.Level2<bool>
call ProcessGenericTypeWithReturn for Level1<int>
call GetGenericArguments() for Level1<int>
call GetGenericArguments() for Level1<int>.Level2<bool>
duplicated call of GetGenericArguments() for Level1<int>.Level2<bool>.Level3<int>

Here a small demo of my point on sharp.io (I had to fake some reflection call, because it's prohibited there).
I've put here a link to gist of the method that I've tried.

Benchmark for:

version in dev branch (TestOldSingle, TestOldDifferent);
version in this PR (TestLastVersionSingle, TestLastVersionDifferent);
version with return of offset, duplicated call (TestReturnOptionSingle, TestReturnOptionDifferent).

                    Method |      Mean |     Error |    StdDev |  Gen 0 | Allocated |
-------------------------- |----------:|----------:|----------:|-------:|----------:|
             TestOldSingle |  2.243 us | 0.0187 us | 0.0175 us | 0.3128 |    1316 B |
     TestLastVersionSingle |  1.856 us | 0.0081 us | 0.0076 us | 0.1564 |     660 B |
    TestReturnOptionSingle |  2.009 us | 0.0170 us | 0.0159 us | 0.1678 |     704 B |
          TestOldDifferent | 41.059 us | 0.3918 us | 0.3473 us | 5.7373 |   24281 B |
  TestLastVersionDifferent | 27.550 us | 0.1457 us | 0.1292 us | 3.1738 |   13400 B |
 TestReturnOptionDifferent | 32.207 us | 0.3072 us | 0.2566 us | 3.2959 |   14073 B |

Thanks for the PR!

@AndreyTretyak - Can you rebase & squash into 1 commit on the top of dev?

@smitpatel I'll do that in couple hours sorry for the delay.

I got this. Thanks again!

@pranavkm Thank you for help with the merge, and for spending time on the investigation of GetGenericArguments() it improved code drastically.
@smitpatel Thank you for reviewing my PR's, it helps a lot.

I've only just noticed that this PR changed output for open genericis
Old:
typeof(List<>) -> List
typeof(Dictionary<,>) -> Dictionary
New:
typeof(List<>) -> List<>
typeof(Dictionary<,>) -> Dictionary<, >
I'll make PR that add the unit tests for open generics, but should I return behavior to old version? (new version looks more precise form my point of view).

Create a PR to put the "available" name (instead of full name) for open generics. I believe it is good to print List<> as List<T>.
@Eilon - Any comments?

I like this option even more, will do that, thanks

Hmm I guess it depends what we want this to be. Do we want things to look like what .NET says the type name is, or do we want it to look like how C# represents the type in source code?


.gemwarrior file not created.

Possibly due to selecting the Log menu option prior to actually playing the game:
/-+-+-+ +-+-+-+-+-+-+-\
|G|E|M| |W|A|R|R|I|O|R|
\-+-+-+ +-+-+-+-+-+-+-/
=======================
 (N)ew Game
 (A)bout
 (H)elp
 (L)og
 (E)xit
=======================

> l
An unexpected error occurred while running :
  No such file or directory - /Users/eric/.gemwarrior

I guess I should check for that file before reading from it, eh? Fixed.


Another take at fixing symlinks

Another take at fixing symlinks, a combination (merge) of #8144 and #10454 into one commit. That should fix the symlink issues from my origional commit.
For more information see my bottom response in task #8144

Something weird happened in your git branch.

Yea, I'll close this and try a new branch

My next attempt is #10883


Syscall number mapping issue

The rt_sigreturn syscall (syscall number: 15) is not added to the block list in "4.15.0-1045-aws" kernel version. Because, the rt_sigreturn syscall number is not mapped when using the AWK technique in syscall.py. Please refer the below snippets.

rt_sigreturn not added to nginx.seccomp.json

$ grep rt_sigreturn results/nginx.seccomp.json
<EMPTY>

Syscall number mapping code in syscall.py
https://github.com/shamedgh/confine/blob/8946203fea43236581559fc95208019f6645ff97/python-utils/syscall.py#L14

Debug log snippet on nginx container

<SKIP>
Traversing libc call graph to identify required system calls
allSyscallsOriginal: {0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 16, 17, 18, 19, 20, 21, 22, 23, 24, 25, 28, 29, 30, 32, 33, 35, 38, 39, 40, 41, 42, 43, 44, 45, 46, 47, 48, 49, 50, 51, 53, 54, 55, 56, 59, 60, 61, 62, 63, 67, 72, 74, 77, 78, 79, 80, 81, 82, 83, 84, 85, 86, 87, 88, 89, 90, 91, 92, 93, 94, 95, 96, 97, 99, 100, 102, 104, 105, 106, 107, 108, 109, 110, 111, 112, 115, 116, 128, 130, 131, 133, 137, 138, 140, 141, 142, 143, 144, 145, 146, 147, 149, 157, 158, 160, 161, 165, 186, 188, 189, 190, 191, 192, 193, 194, 195, 196, 197, 198, 199, 201, 202, 203, 204, 213, 218, 221, 228, 229, 231, 232, 233, 234, 235, 257, 262, 263, 267, 269, 273, 275, 277, 280, 288, 290, 293, 296}
<SKIP>

On the above debug log, rt_sigreturn syscall number (15) is not added in the list of syscall number mapping.
Current workaround: Using libseccomp-rs, we created a custom syscall number mapping code.

Hello @shamedgh ,
I have created a PR (#9 ) which proposes a solution to this issue. Please review the PR and let me know your comments or suggestions.
Regards,
Gowri Sankar

Was resolved by PR (#9)


[1.12.2] [suggestion] Liquid Nitrogen

My suggestion is to add Liquid Nitrogen as a Coolant for the Condenser and maybe the Other Coolant Blocks like a weaker version of Liquid Helium.

Added for the next build ;)

Thank you :)


minor updates

@vladai78 : Thanks for your contribution! The author, @radicmilos, has been notified to review your proposed change.

#sign-off


fix: get rid of deprecated API use: vim.lsp.buf_get_clients()

buf_get_clients is deprecated.
vim.lsp.get_clients: 0.10+
vim.lsp.get_active_clients: 0.5+

There are two other instances of use of a deprecated method in init.lua: line 80 (get_active_clients) and 91 (buf_get_clients).

thank you


Superfluid staking PR

Closes: #XXX
Description

For contributor use:

[ ] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer

Codecov Report

Merging #1283 (30f659d) into main (7d45d8e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1283   +/-   ##
=======================================
  Coverage   20.08%   20.08%
=======================================
  Files         203      203
  Lines       26786    26786
=======================================
  Hits         5379     5379
  Misses      20409    20409
  Partials      998      998

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 7d45d8e...30f659d. Read the comment docs.

This PR does:

Merges all MD files into one ginormous README for easy reading
A brief added introduction docs to the superfluid state machine itself.

lgtm. This closes #1073. Update PR with link to it.

lgtm. This closes #1073. Update PR with link to it.


Integrate a link checker

Description:
Need to integrate a link checker.
Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

Closing as this is handled by running external link checkers at the moment.
Will reopen in the future if it needs to be prioritized.


chore: use codeigniter4/.github/.github/ for phpcpd

Description
It seems that phpcpd  was not setuped correctly. I couldn't figure out why.
https://github.com/codeigniter4/CodeIgniter4/actions/runs/10152748314/job/28077826735?pr=9085
Run shivammathur/setup-php@v2
/usr/bin/bash /home/runner/work/_actions/shivammathur/setup-php/v2/src/scripts/run.sh

==> Setup PHP
✓ PHP Updated to PHP 8.1.29

==> Setup Extensions
✓ dom Enabled
✓ mbstring Enabled

==> Setup Tools
✓ composer Added composer 2.[7](https://github.com/codeigniter4/CodeIgniter4/actions/runs/10166194607/job/28115790693?pr=9090#step:3:8).7
✗ phpcpd Could not setup phpcpd

==> Sponsor setup-php
✓ setup-php https://setup-php.com/sponsor
0s
Run phpcpd --exclude system/Test --exclude system/ThirdParty --exclude system/Database/SQLSRV/Builder.php --exclude system/Database/SQLSRV/Forge.php --exclude system/Database/MySQLi/Builder.php --exclude system/Database/OCI[8](https://github.com/codeigniter4/CodeIgniter4/actions/runs/10166194607/job/28115790693?pr=9090#step:3:9)/Builder.php --exclude system/Database/Postgre/Builder.php --exclude system/Debug/Exceptions.php --exclude system/HTTP/SiteURI.php --exclude system/Validation/Rules.php --exclude system/Autoloader/Autoloader.php --exclude system/Config/Filters.php -- app/ public/ system/
/usr/local/bin/phpcpd: line 1: error: command not found
Error: Process completed with exit code [12](https://github.com/codeigniter4/CodeIgniter4/actions/runs/10166194607/job/28115790693?pr=9090#step:3:13)7.

Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[ ] Conforms to style guide

Unfortunately, that did not work. Please drop the last two commits.
Run phpcpd app/ public/ system/ --exclude system/Test
  phpcpd app/ public/ system/ --exclude system/Test
  --exclude system/ThirdParty
  --exclude system/Database/SQLSRV/Builder.php
  --exclude system/Database/SQLSRV/Forge.php
  --exclude system/Database/MySQLi/Builder.php
  --exclude system/Database/OCI8/Builder.php
  --exclude system/Database/Postgre/Builder.php
  --exclude system/Debug/Exceptions.php
  --exclude system/HTTP/SiteURI.php
  --exclude system/Validation/Rules.php
  --exclude system/Autoloader/Autoloader.php
  --exclude system/Config/Filters.php
  shell: /usr/bin/bash -e {0}
  env:
    COMPOSER_PROCESS_TIMEOUT: 0
    COMPOSER_NO_INTERACTION: 1
    COMPOSER_NO_AUDIT: 1
phpcpd 6.0.3 by Sebastian Bergmann.

Found 11 clones with 439 duplicated lines in 15 files:

  - /home/runner/work/CodeIgniter4/CodeIgniter4/app/Config/Filters.php:27-105 (78 lines)
    /home/runner/work/CodeIgniter4/CodeIgniter4/system/Config/Filters.php:40-118

  - /home/runner/work/CodeIgniter4/CodeIgniter4/system/Autoloader/Autoloader.php:496-567 (71 lines)
    /home/runner/work/CodeIgniter4/CodeIgniter4/system/CodeIgniter.php:238-309

  - /home/runner/work/CodeIgniter4/CodeIgniter4/system/Debug/BaseExceptionHandler.php:101-150 (49 lines)
    /home/runner/work/CodeIgniter4/CodeIgniter4/system/Debug/Exceptions.php:379-428

  - /home/runner/work/CodeIgniter4/CodeIgniter4/system/Database/BaseBuilder.php:659-697 (38 lines)
    /home/runner/work/CodeIgniter4/CodeIgniter4/system/Database/SQLSRV/Builder.php:119-157

  - /home/runner/work/CodeIgniter4/CodeIgniter4/system/Debug/BaseExceptionHandler.php:70-106 (36 lines)
    /home/runner/work/CodeIgniter4/CodeIgniter4/system/Debug/Exceptions.php:342-378

  - /home/runner/work/CodeIgniter4/CodeIgniter4/system/Validation/Rules.php:125-159 (34 lines)
    /home/runner/work/CodeIgniter4/CodeIgniter4/system/Validation/StrictRules/Rules.php:145-179

  - /home/runner/work/CodeIgniter4/CodeIgniter4/system/Validation/Rules.php:184-217 (33 lines)
    /home/runner/work/CodeIgniter4/CodeIgniter4/system/Validation/StrictRules/Rules.php:206-239

  - /home/runner/work/CodeIgniter4/CodeIgniter4/system/HTTP/SiteURI.php:337-368 (31 lines)
    /home/runner/work/CodeIgniter4/CodeIgniter4/system/HTTP/URI.php:1038-1069

  - /home/runner/work/CodeIgniter4/CodeIgniter4/system/Database/BaseBuilder.php:2623-2648 (25 lines)
    /home/runner/work/CodeIgniter4/CodeIgniter4/system/Database/Postgre/Builder.php:326-351

  - /home/runner/work/CodeIgniter4/CodeIgniter4/system/Database/BaseBuilder.php:3083-3107 (24 lines)
    /home/runner/work/CodeIgniter4/CodeIgniter4/system/Database/SQLSRV/Builder.php:613-637

  - /home/runner/work/CodeIgniter4/CodeIgniter4/system/Database/Forge.php:1150-1170 (20 lines)
    /home/runner/work/CodeIgniter4/CodeIgniter4/system/Database/SQLSRV/Forge.php:305-325

0.39% duplicated lines out of 111222 total lines of code.
Average size of duplication is 39 lines, largest clone has 78 of lines

https://github.com/codeigniter4/CodeIgniter4/actions/runs/10171059110/job/28131363812?pr=9090

Thanks @kenjis, I have created the code readability, please review.

Try this: options: >- instead of double quotes

Thanks @paulbalandan, applied.

@datamweb @paulbalandan Thanks! LGTM!


Add AssignmentPattern

Current coverage is 99.74%

Merging #13 into master will not affect coverage as of 0fc1387

@@            master     #13   diff @@
======================================
  Files           76      77     +1
  Stmts          771     777     +6
  Branches        95      95
  Methods          0       0
======================================
+ Hit            769     775     +6
  Partial          0       0
  Missed           2       2

Review entire Coverage Diff as of 0fc1387

Powered by Codecov. Updated on successful CI builds.


require pep257 for dev users

pep257 has been renamed to pydocstyle

Thanks @r3d4rmy, updated.

Thanks for the pull request, sorry for the delay, I've been OOTO for the last couple weeks.

Merged in 9f6c9b6a97f099d09b99710763ba38e651002fb1


Enhance income and expense commands so that they don't necessarily require a date field

Solved by ce18e5046136694c359ec0480226807375dc972d


'eosiolib/core_symbol.hpp' file not found

When I compile my code for abi , I got the following error. However, if I compile the code with v1.1.0, the compilation is fine. Any suggestion?
$ eosiocpp -o testapp.wast testapp.cpp ---No error
$eosiocpp -g testapp.abi testapp.cpp
In file included from ~/myapp/eos/contracts/testapp/testapp.cpp:1:
In file included from ~/myapp/eos/contracts/testapp/testapp.hpp:14:
In file included from ~/myapp/eos/contracts/eosiolib/asset.hpp:5:
~/myapp/eos/contracts/eosiolib/symbol.hpp:2:10: fatal error: 'eosiolib/core_symbol.hpp' file not found
#include <eosiolib/core_symbol.hpp>
^~~~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.
Error while processing ~/myapp/eos/contracts/testapp/testapp.cpp.

same problem to me,help.

@srmcatee1110 That workaround worked. Thanks! Btw, I think community should fix this asap. It is broken for a while.

Btw, just curious if I would like to contribute some patch, which branch should I submit to? Is there a process on doing that?

@chuang39 , against release/1.2.x
But as to my knowledge, the *.in files are used during the build/install process, so you possibly should await a comment of some core-dev.
To ensure its a bug: this sequence should clone/build/install all relevant files:
git clone https://github.com/EOSIO/eos.git --recursive
cd eos
./eosio_build.sh
sudo ./eosio_install.sh

Lazaridis, Thanks for response. Yep, I think I followed that process. Should be easy to repro.

I can confirm your bug, by adding an #include <eosiolib/symbol.hpp> to the hello.cpp demo.
related: #5285 #5297
Problem
Summary: core_symbol.hpp is generated based on core_symbol.hpp.in. It does not find its way to an include-path
Workaround
copy the generated file into a location where its found by eosiocpp
cd eos
cp build/libraries/chain/include/eosio/chain/core_symbol.hpp contracts/eosiolib/

see a comment:
https://github.com/EOSIO/eos/pull/5401/files/5b114ad976d20e6827bb2a5ad17ef945c5db5cc9

@lazaridiscom See my reply ;)
https://github.com/EOSIO/eos/pull/5401/files/5b114ad976d20e6827bb2a5ad17ef945c5db5cc9#r212826380

Please test with cdt. Please ask in eosio.stackexchange.com if you have further questions.


no reaction

Create public/private gist and Add file to gist no reaction,but I can insert and open.
System:win7 64 ultimate
Software:Sublime Text3

1 gist: add file to gist (this item is not in the list of commands, if the file is already in the current gist)
2 appears a list of gists
3 choose the right gist
4 in the status bar you receive the file name
5 enter the name of the file

Can you try to update gist plugin and try to reproduce the issue.


links are repair

Closes
Closes #1036

Description
links are now taking to respective page
Screenshots

https://github.com/OSCode-Community/OSCodeCommunitySite/assets/96969496/a825e2f2-afd7-47e6-b94e-11d99bbc6677
Note to reviewers

Checklist:

[ x] I have linked the PR to the correct issue.
[ x] I have read the Contribution Guidelines
[ x] I have built and tested the changes, and they do not break or show any errors.

@Vaishnavi-Patil2211 please review the pr


fix: minor changes in Groups/Defs

Remove instance and add to_additive
Point some comments to the right issue
Resolves the issue in #707
Might conflict with #717

bors merge


bug: dgsQueryExecutor.executeAndExtractJsonPathAsObject cannot deserialize DateTime scalars

Expected behavior
The framework should serialize the json response to an object, including DateTime scalar values.
Actual behavior
Jackson's ObjectMapper does not know about the scalar and a DgsQueryExecutionDataExtractionException is thrown.
Steps to reproduce
Please see the issue I opened in dgs-examples-java.

I'm not sure if this is as simple as adding
.registerModule(JavaTimeModule())

to the ObjectMapper initialization.

Thanks for reporting, we will look into it.

@srinivasankavitha I managed to fix it and will have a pull request in soon.


Play/Pause/Download Video in FlatList

I understand that this Issues of library is only to specific library problem and not what I am about to ask. I have already posted a question on Stackoverflow. But noone answered, since its related to video, I am sure someone must have come across this feature. So giving it a shot here. Apologizes again. :)
The idea is simple play a video which is being shown currently in the flatlist, pause the rest.
This is my FeedCard -> MyFlatListItem
class FeedCard extends PureComponent {

    constructor(props){
        super(props);
        this.state = {
            pauseVideo: true
        };
    }

    playThisVieo = (playVideo) => {
      this.setState({
          pauseVideo: !playVideo
      });
    };

    render() {
        return (
            <Card>
                <CardItem cardBody>
                    <View style={styles.preview}>
                        <Video
                            source={{ uri: this.props.videoURL}}
                            ref={(ref) => {
                                this.player = ref;
                            }}
                            rate={1.0}
                            volume={1.0}
                            muted={false}
                            paused={this.state.pauseVideo}
                            resizeMode="stretch"
                            repeat={true}
                            playInBackground={false}
                            playWhenInactive={false}
                            ignoreSilentSwitch={"ignore"}
                            progressUpdateInterval={250.0}
                            onLoadStart={() => this.player.seek(0)}
                            onLoad={this.setDuration}
                            onProgress={this.setTime}
                            onEnd={this.onEnd}
                            onError={this.videoError}
                            onBuffer={this.onBuffer}
                            onTimedMetadata={this.onTimedMetadata}
                            style={{
                                position: 'absolute',
                                top: 0,
                                left: 0,
                                bottom: 0,
                                right: 0,
                            }}
                        />
                    </View>
                </CardItem>
            </Card>
        );
    }
}

Now all I have to do is renderThisItem In FlatList and then in onViewableItems, only play that video which is being shown.
constructor(props){
    super(props);
    this.viewabilityConfig = {viewAreaCoveragePercentThreshold: 70};
}

 <FlatList
        horizontal={false}
        data={this.props.feeds}
        renderItem={this.renderFeeds}
        viewabilityConfig={this.viewabilityConfig}
        onViewableItemsChanged={this._onViewableItemsChanged}
 />

 renderFeeds = (feed) => {
    return (
        <FeedCard
            ref={(ref) => this.feedCardRef = {...this.feedCardRef, [`REF-FLATLIST${feed.item.feedID}`]: ref}} //Saving Refs Here
            videoURL={feed.item.videoURL}
            //Other Props
        />
    );
};

    _onViewableItemsChanged = (props) => {
        const changed = props.changed;
        const viewableItems = props.viewableItems;
        console.log(changed);
        console.log(viewableItems);
        changed.forEach(item => {
            if (!item.isViewable) {
                this.feedCardRef[`REF-FLATLIST${item.key}`].playThisVideo(item.isViewable);
            }
        });
        viewableItems.forEach(item => {
            if (item.isViewable) {
                this.feedCardRef[`REF-FLATLIST${item.key}`].playThisVideo(item.isViewable);
            }
        });
    };

Now the above works well for playing video only if its the one being shown in the screen, but I also want to download the video only if that video is being currently shown in the screen. How do I control the download of a video? I couldnt find anything in the readme.
StackOverFlow Link

@kk1429 I have the exact same implementation and struggling to not download the remaining videos and only the one which is playing. did you manage to get a solution. ??

Did either of you find a way to achieve this or a good workaround?

Hello,
I have same  problem if i'm read 3 videos on ligne.
Any solution to read very fast video .mp4 on ligne , if on click next video or back video .
Thanks :)

You could use a thumbnail and first start rendering the video behind the thumbnail when it is in the center of the  screen (calculated based on offsetY of the ListView and the index + height of the list items). After the video is loaded  (onLoad) event you could fade out the thumbnail and start playing the video. furthermore you could debounce on the  onScroll event handler of the ListView, to ensure that a video component is only loaded after there was no scroll for the last few milliseconds or so. This is what I am basically implementing. If you guys are interested I can share some code after I finished it.

@abdelhakimkotti The first step of the process is https://github.com/n1ru4l/react-in-center-of-screen which I now use to determine whether a list item is in the center of the screen.
Code for the video component with a preview image will follow soon!

@n1ru4l I'm subscribing and awaiting your suggestion.

@pribeh I am currently working on our android part of the application. On android the video component (with preview image) does not work as performant as it should be. I will try to provide updates (and hopefully I can also open source that component).

@elzii sorry currently I have not time to work on open sourcing that component. I will try to do it asap, I cannot give a guaranteed date 😕

``

I'm also struggling with this scenario, anyone here who found the solution? I only want to download the visible video in flatlist.

Hi, friends I have solved the issue for my react native video project.
the issue was that all videos are playing in Flatlist but we need to play only singal video on the current viewport and pause the rest.
just do the following steps to solve all videos playing issue
1: npm install @svanboxel/visibility-sensor-react-native
2: import VisibilitySensor from '@svanboxel/visibility-sensor-react-native'
3: do this
import VisibilitySensor from '@svanboxel/visibility-sensor-react-native'
const video = ()=>{
const [paused, setpaused] = useState(true)
return(
     <VisibilitySensor onChange={(isVisible)=>{
        return(
          console.log(isVisible),
          isVisible?setpaused(false):setpaused(true)
        )
      }
    }
    >
    <View>
     <Video
              source={{uri: 'https://d8vywknz0hvjw.cloudfront.net/fitenium-media-prod/videos/45fee890-a74f-11ea-8725-311975ea9616/proccessed_720.mp4'}}
              style={styles.video}
              onError={(e) => console.log(e)}
              resizeMode={'cover'}
              repeat={true}
              paused={paused}
            />
      </View>
    </VisibilitySensor>
)
}

4: I have just given you the basic structure you can add styling stuff as your requirements.
5: remember that always add your view/video elements between the VisibilitySensor tags, otherwise it will not work.
6: this code will give you true when your video component will render in flatlist viewport and remainig will give you false. with this you can manage play/pause state of video element.
thanks...

Moved to #2668