Prometheus-operator pods are failing

@brancz
Prometheus-operator was working perfectly after the upgrade of k8s cluster to v1.8.2. Now again I see many of my pods are failing. I deleted the monitoring namespace and redeployed from the latest manifests but still, the deployments are failing. recently I have upgraded the k8s cluster to v1.8.2. What is the issue here.?

Could you please be more specific? What pods are failing? Which exact versions are you using?

@brancz
namespace "monitoring" created
customresourcedefinition "alertmanagers.monitoring.coreos.com" created
customresourcedefinition "prometheuses.monitoring.coreos.com" created
customresourcedefinition "prometheusrules.monitoring.coreos.com" created
customresourcedefinition "servicemonitors.monitoring.coreos.com" created
clusterrole "prometheus-operator" created
clusterrolebinding "prometheus-operator" created
deployment "prometheus-operator" created
service "prometheus-operator" created
serviceaccount "prometheus-operator" created
servicemonitor "prometheus-operator" created
alertmanager "main" created
secret "alertmanager-main" created
service "alertmanager-main" created
serviceaccount "alertmanager-main" created
servicemonitor "alertmanager" created
configmap "grafana-datasources" created
configmap "grafana-dashboard-k8s-cluster-rsrc-use" created
configmap "grafana-dashboard-k8s-node-rsrc-use" created
configmap "grafana-dashboard-k8s-resources-cluster" created
configmap "grafana-dashboard-k8s-resources-namespace" created
configmap "grafana-dashboard-k8s-resources-pod" created
configmap "grafana-dashboard-nodes" created
configmap "grafana-dashboard-pods" created
configmap "grafana-dashboard-statefulset" created
configmap "grafana-dashboards" created
deployment "grafana" created
service "grafana" created
serviceaccount "grafana" created
clusterrole "kube-state-metrics" created
clusterrolebinding "kube-state-metrics" created
deployment "kube-state-metrics" created
role "kube-state-metrics" created
rolebinding "kube-state-metrics" created
service "kube-state-metrics" created
serviceaccount "kube-state-metrics" created
servicemonitor "kube-state-metrics" created
clusterrole "node-exporter" created
clusterrolebinding "node-exporter" created
daemonset "node-exporter" created
service "node-exporter" created
serviceaccount "node-exporter" created
servicemonitor "node-exporter" created
clusterrole "prometheus-k8s" created
clusterrolebinding "prometheus-k8s" created
prometheus "k8s" created
rolebinding "prometheus-k8s-config" created
rolebinding "prometheus-k8s" created
rolebinding "prometheus-k8s" created
rolebinding "prometheus-k8s" created
role "prometheus-k8s-config" created
role "prometheus-k8s" created
role "prometheus-k8s" created
role "prometheus-k8s" created
prometheusrule "prometheus-k8s-rules" created
service "prometheus-k8s" created
serviceaccount "prometheus-k8s" created
servicemonitor "prometheus" created
servicemonitor "kube-apiserver" created
servicemonitor "coredns" created
servicemonitor "kube-controller-manager" created
servicemonitor "kube-scheduler" created
servicemonitor "kubelet" created

Kubernetes-Version: v1.8.2
Prometheus-operator version: v2.2.1
Failed deployments:
kube-state-metrics:
error: Failed create pod sandbox. Error syncing pod

grafana:
error: Failed create pod sandbox. Error syncing pod

Failed Pods:
NAME                                   READY     STATUS              RESTARTS   AGE
alertmanager-main-0                    0/2       ContainerCreating   0          12m
grafana-5564f8647-gctz9                0/1       ContainerCreating   0          12m
kube-state-metrics-77b668df49-hnsrs    0/4       ContainerCreating   0          12m
kube-state-metrics-86d4f9d8fc-pklmn    4/4       Running             0          12m
node-exporter-77pgn                    2/2       Running             0          12m
node-exporter-78gd6                    2/2       Running             0          12m
node-exporter-7wbh8                    2/2       Running             0          12m
node-exporter-9bkgr                    2/2       Running             0          12m
node-exporter-bj6n7                    2/2       Running             0          12m
node-exporter-btwp4                    2/2       Running             0          12m
node-exporter-lnpg7                    2/2       Running             0          12m
node-exporter-lvpff                    2/2       Running             0          12m
node-exporter-mzhbd                    0/2       ContainerCreating   0          12m
node-exporter-p28j4                    2/2       Running             0          12m
node-exporter-r4xxf                    2/2       Running             0          12m
node-exporter-shrdh                    2/2       Running             0          12m
node-exporter-skg9c                    2/2       Running             0          12m
node-exporter-tffnx                    2/2       Running             0          12m
node-exporter-vf497                    2/2       Running             0          12m
prometheus-operator-6fc659894f-msn5r   1/1       Running             0          12m

Logs:
"kube-rbac-proxy-main" in pod "kube-state-metrics-77b668df49-hnsrs" is waiting to start: ContainerCreating

"grafana" in pod "grafana-5564f8647-gctz9" is waiting to start: ContainerCreating

Jithins-MacBook-Pro:prometheus jithinraj$ kubectl get prometheus -n monitoring
NAME      AGE
k8s       14m
Jithins-MacBook-Pro:prometheus jithinraj$ kubectl get Alertmanager -n monitoring
NAME      AGE
main      14m
Jithins-MacBook-Pro:prometheus jithinraj$ kubectl get DaemonSet -n monitoring
NAME            DESIRED   CURRENT   READY     UP-TO-DATE   AVAILABLE   NODE SELECTOR                 AGE
node-exporter   15        15        14        15           14          beta.kubernetes.io/os=linux   15m

Note: Prometheus and Alert manager pods are missing:  I faced this issue before when I was using old k8s version. It got fixed after the cluster update.

Can you check whether the statefulset was created for Prometheus?
Aside from that, yes you should update your cluster, 1.8 is soon end of life and I think (without specifically checking) we require at least version 1.9, ideally latest: 1.11.

It got fixed after the cluster update.

Great :slightly_smiling_face: . In that case I'm closing this here.

@brancz Is there any way to use Prometheus-operator without upgrading the k8s cluster to any newer versions. It is risky to upgrade the production cluster since all our production applications work well with older k8s version. My organization doesn't recommend upgrading the cluster only for Prometheus-operator.

@jithinraj89 I am sorry, but we do not plan on supporting older versions of Kubernetes. Your best bet is either to use an older version of the Prometheus Operator, or eventually update your production cluster as Kubernetes 1.8 is soon going end-of-life.


fix: Bar chart order

SUMMARY
The goal of this PR is to fix the order of the Bar chart
BEFORE
https://user-images.githubusercontent.com/60598000/154296688-96327e55-73d1-42f6-a41e-7137fce73b8b.mp4
AFTER
https://user-images.githubusercontent.com/60598000/154297218-6815a418-793c-49a2-b66b-08b6abaf981c.mp4
TESTING INSTRUCTIONS

Open the sample Total Sales per Market (Grouped by Genre) Bar chart
Add a sort by metric
Make sure the order is correct

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

/testenv up

I found an issue that the when i put different sort by values, the order still stay the same. For example, i use sort by sum(na_sales )and sum(eu_sales), the series are in the same order, and it is not in the order of what i selected.

I think the current behavior on master is making more sense, when i select sort by sum(na_sales), the bar series show in order sum.
For example sort by sum(na_sales)

@jinghua-qa I think there is a bit of confusion around the original issue. I agree with you that my method here isn't following the order in the data panel and that looks incorrect. Is there an actual problem that you are identifying with the sorting on master right now, or am I trying to fix a problem by creating one? 🤦


Specify Python version as strings

Why:
Closes #11019

What's being changed:

Changing Python versions to be strings on both the interactive and non-interactive pages
Using single quotes to be consistent with existing usage

Single quote in existing code
https://github.com/github/docs/blame/b9dc0417694f59b81f2f597bf72bdf116d05f88a/content/actions/automating-builds-and-tests/building-and-testing-python.md#L157
Check off the following:

[ ] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).

Nothing loading here so can't check 😕

[ ] For content changes, I have completed the self-review checklist.

Semi completed  as staging doesn't seem to load anything
Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@thewheat I am sorry, this was already fixed internally. So, I will be closing this PR now 💛 Thank you for your patience and your time in contributing to the GitHub docs 💖

What was the pulled request about ?
Sent from Yahoo Mail for iPhone
On Thursday, November 4, 2021, 12:53 PM, Ramya Parimi @.***> wrote:
@thewheat I am sorry, this was already fixed internally. So, I will be closing this PR now 💛 Thank you for your patience and your time in contributing to the GitHub docs 💖
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Read From multiple relays

Because the webclient is only reading from one relay at a time, it's not displaying full information on profiles and bounties.
Change default to read from multiple relays.

I don't think I'll have this done for the hackathon, I'll need to do a deeper rewrite for much of the relay logic. The good news is I have this working on another app so we will eventually be able to get it working here as well.

This is now implemented


added servo on/off interface to realtime robot controller client

added servo on/off interface to realtime robot controller client

yay! thanks!


[Bug]:  Adding animation on blog page

Contact Details
<EMAIL_ADDRESS>What happened?
I see that the blog page of website needs to have a better ui and also for better ux i suggest of adding some of the animations like in other sections of the website. @pranshugupta54
What browsers are you seeing the problem on?
Chrome, Microsoft Edge
Relevant log output
No response
Are you interested in contributing?

[X] I'd like to work on this issue

Code of Conduct

[X] I have used the search function to check if an issue already exists
[X] I agree to follow this project's Code of Conduct

Which program are you contributing in?
gssoc

Blogs are integrated with Hashnode


Zero division easter egg

This is just a little idea i had: We could add a small easter egg: the application will not say error after a few times of trying, but things like: "Stop dividing by zero!", "Seriously, stop", "STOP IT!!!!!!". And things like that. I thought this will be a cool little secret.

I can implement this tomorrow if you approve it @HuyHung1408

That's very cool, you can implement it

Also - I made one now, we could try this - if a user(probably a 6-year old) does division by zero(we know math, it leads us to those two spiralling loops mathematicians call "infinity", ain't it?), we could show warnings and end with "Fluent Python Calculator will shut down if there is one more division by zero. It leads you nowhere!".
Creativity at its finest.
The name is too long, can we try finding a shorter one?

Also(another also ,oops another one) - welcome to the team @HuyHung1408!

Yeah, wee definitely need to change the name. And btw @HuyHung1408 i am having issues with compiling. win32mica doesnt seem to work when the program is compiled.

Did you include Ntkutils?

Yes, and btw ntkutils had a bug with mica that i fixed yesterday. Will push to pypi and github tomorrow.

Also(another also, oops another one) - welcome to the team @HuyHung1408!

Hi y'all XDD

Yeah, wee definitely need to change the name. And btw @HuyHung1408 i am having issues with compiling. win32mica doesnt seem to work when the program is compiled.

I faced that issue too

Added it in! What do you guys think?
https://user-images.githubusercontent.com/86362423/178089981-c8e17324-818e-4c6f-84ba-a0f4a7e95299.mp4

Added this in, what do you guys think?
https://user-images.githubusercontent.com/86362423/178092447-711c81e7-019e-4b9d-a336-9b010c2ae9e1.mp4

cool!

Nice!

Thanks y'all

Believe it or not @not-nef's txt2 inspired me to continue coding 🤯

Yooo that makes me veeery happy! 😊

Add the rickroll!!! It's gonna be so sick! My new rickroll weapon to use on bullies is ready. Imagine the frustration when someone does that, not knowing the fact that division happens when both values have some real relation. Real, not rational, meaning irrational too. But Zero, it is not really common, so it leads to infinity. Since it is Not Defined, we declare it as that.
I might as well teach math at school lol

added rickroll when they choose no when fluxcalc ask them to stop or you want to open like directly

Add the rickroll!!! It's gonna be so sick! My new rickroll weapon to use on bullies is ready. Imagine the frustration when someone does that, not knowing the fact that division happens when both values have some real relation. Real, not rational, meaning irrational too. But Zero, it is not really common, so it leads to infinity. Since it is Not Defined, we declare it as that.
I might as well teach math at school lol


Issues with 0.2.0-alpha-20221230.0

@orchetect I am getting some errors when I try using the new build. Am I doing anything wrong?
My .sh script.
#!/bin/sh

TOOL_PATH="/Users/xxx/Desktop/Marker_Data/App/0.2.0/markers-extractor-cli"
FCPXML_PATH="/Users/xxx/Desktop/Marker_Data/App/0.2.0/Render/Marker Data Demo_V1.fcpxmld"
OUTPUT_DIR="/Users/xxx/Desktop/Marker_Data/App/0.2.0/Output"
ERROR_LOG="/Users/xxx/Desktop/Marker_Data/App/0.2.0/log.txt"

$TOOL_PATH "$FCPXML_PATH" "$OUTPUT_DIR" --export-format csv2notion --log-level debug --log $ERROR_LOG

Error
WARNING: Marker at 00:03:45:03 is out of bounds of its parent clip.
MarkersExtractor/resource_bundle_accessor.swift:40: Fatal error: unable to find bundle named MarkersExtractor_MarkersExtractor
/Users/xxx/Desktop/Marker_Data/App/0.2.0/Tester_still.sh: line 8:  2460 Illegal instruction: 4  $TOOL_PATH "$FCPXML_PATH" "$OUTPUT_DIR" --export-format csv2notion --log-level debug --log $ERROR_LOG

I am able to process the same FCPXML with the previous 0.1.1 build without any errors.

Thanks for the heads-up, I found the source of the bug.

Thanks for the heads-up, I found the source of the bug.

Side note - For --log-level <trace,debug,info,notice,warning,error,critical>. What is the difference between trace and debug? Which should I use for error reporting?

Side note - For --log-level <trace,debug,info,notice,warning,error,critical>. What is the difference between trace and debug? Which should I use for error reporting?

Fixed. New alpha build is up:
https://github.com/TheAcharya/MarkersExtractor/releases/tag/0.2.0-alpha-20221231.0

Fixed. New alpha build is up:
https://github.com/TheAcharya/MarkersExtractor/releases/tag/0.2.0-alpha-20221231.0

What is the difference between trace and debug? Which should I use for error reporting?

debug is adequate for error reporting.

What is the difference between trace and debug? Which should I use for error reporting?

debug is adequate for error reporting.

Thanks works for now. Will close this for now.

Thanks works for now. Will close this for now.


where is the 'admin-config' directory?

I'm learning ng-admin and research it's source code. It's very awesome. but I have a problem.

I can't find out where this file is. Is it not open source?

Oh , you made it a seperated npm module.
sorry i missed it.

We indeed splitted it into a dedicated module, as it is common to both ng-admin and an on construction [react-admin].


singlediode error with very low effective_irradiance

Describe the bug
Since pvlib 0.9.4 release (https://github.com/pvlib/pvlib-python/pull/1606) I get an error while running the single-diode model with some very low effective irradiance values.
To Reproduce
from pvlib import pvsystem

effective_irradiance=1.341083e-17
temp_cell=13.7

cec_modules = pvsystem.retrieve_sam('CECMod')
cec_module = cec_modules['Trina_Solar_TSM_300DEG5C_07_II_']

mount = pvsystem.FixedMount()
array = pvsystem.Array(mount=mount,
                       module_parameters=cec_module)

system = pvsystem.PVSystem(arrays=[array])

params = system.calcparams_cec(effective_irradiance,
                               temp_cell)

system.singlediode(*params)

    303 """
    304 Vectorized golden section search for finding maximum of a function of a
    305 single variable.
   (...)
    342 pvlib.singlediode._pwr_optfcn
    343 """
    344 if np.any(upper - lower < 0.):
--> 345     raise ValueError('upper >= lower is required')
    347 phim1 = (np.sqrt(5) - 1) / 2
    349 df = params

ValueError: upper >= lower is required

Expected behavior
This complicates the bifacial modeling procedure as run_model_from_effective_irradiance can be called with very low irradiance values estimated by pvfactors (at sunrise or sunset for instance).
Versions:

pvlib.__version__:  0.9.4
pandas.__version__: 1.5.3
python: 3.10

Additional context
v_oc is negative in this case which causes the error.
from pvlib.singlediode import _lambertw_v_from_i
photocurrent = params[0]
saturation_current = params[1]
resistance_series = params[2]
resistance_shunt = params[3]
nNsVth = params[4]
v_oc = _lambertw_v_from_i(resistance_shunt, resistance_series, nNsVth, 0.,
                              saturation_current, photocurrent)

That's unexpected, thanks for reporting.
I'll note that the negative Voc results from taking the difference of two very large but nearly equal numbers. It's likely limited to the CEC model, where the shunt resistance is inversely proportional to irradiance, which would be about 1e19 at photocurrent of 1e-17 for this case.

Now this gets strange: the Voc value is positive with pvlib v0.9.3. The function involved pvlib.singlediode._lambertw_v_from_i hasn't changed for many releases. In both v0.9.3 and v0.9.4, the lambertw term overflows so the Voc value is computed using only python arithmetic operators and numpy.log.
I'm starting to think the error depends on python and numpy versions.

The difference between 0.9.3 and 0.9.4 here may be due to slightly different values returned by calcparams_cec.  Compare the output of print(list(map(str, params))); I get slightly different saturation current values for the given example.  Maybe the changed Boltzmann constant in #1617 is the cause?

+1 to #1617 as the likely culprint. I get the positive/negative Voc values with the same python and numpy versions but different pvlib versions.

To illustrate the challenge, this line computes the Voc.
Stripping out the indexing the computation is
    V = (IL + I0 - I) / Gsh - \
        I * Rs - a * lambertwterm

With pvlib v0.9.4, Io is 7.145289906185543e-12. a is not affected, since a value of the Boltzmann contant is inherent in the a_ref value from the database. (IL + I0 - I) / Gsh is<PHONE_NUMBER>36.40567, I * Rs is 0, and a * lambertwterm is<PHONE_NUMBER>36.40569
With pvlib v0.9.3, Io is 7.145288699667595e-12.  (IL + I0 - I) / Gsh is<PHONE_NUMBER>29.58397, I * Rs is 0, and a * lambertwterm is<PHONE_NUMBER>29.58395
The difference defining Voc is in the least significant digit.

Increasing the iterations that solve for lambertwterm doesn't fix this issue.

This smells to me like the inevitable error from accumulated round-off.
FWIW, negative Voc can be achieved in 0.9.3 as well -- try the given example but with effective_irradiance=1.e-18.  The difference is that before #1606, it led to nans and warnings instead of raising an error.

@pasquierjb I recommend intercepting the effective irradiance and setting values to 0 which are below a minimum on the order of 1e-9 W/m2. That will propagate to shunt resistance = np.inf, which changes the calculation path in pvlib.singlediode and gives Voc=0.
I'm not sure we'll be able to extend the numerical solution of the single diode equation to be accurate at very low but non-zero values of photocurrent (and/or enormous but finite values of shunt resistance.)
I note that pvlib.pvsystem.calcparams_desoto doesn't like effective_irradiance=0. but is OK with effective_irradiance=np.array([0.]).  Has to do with trapping and ignoring division by zero warnings and errors.

Have you tried setting method=‘newton’ instead of ’lambertw’? https://pvlib-python.readthedocs.io/en/stable/reference/generated/pvlib.pvsystem.singlediode.html#pvlib-pvsystem-singlediode

Setting method='newton' gets a solution to this case. method isn't available as a parameter of the PVSystem.singlediode method so @pasquierjb would need to change his workflow to use it. Something for us to consider adding.

My workaround for this issue was to first filter very low effective_irradiance values (<1e-8), and then filter photocurrent and saturation_current parameters when effective_irradiance=0 and made them =0. This assures that you won't get negative v_oc values.


Make preset-create-react-app compatible with Yarn PnP/Yarn 2

Related to https://github.com/storybookjs/storybook/issues/9527.
What I did

Add some missing peerDependencies 🔍
Use Plug'n'Play API to find react-scripts package's location directly with the dependency tree. See https://yarnpkg.com/advanced/pnpapi 🚅
Reformat README because I had to 🤷♂

How to test
Unfortunately, there is no great way to test it for now 😏. Locally:
Checkout this branch and build @storybook/preset-create-react-app, then in a tmp folder:
mkdir test-preset-cra && cd test-preset-cra

# Use Yarn 2 inside this folder
yarn set version berry
touch yarn.lock

# Generate app and init SB
yarn dlx create-react-app my-app && cd my-app
yarn dlx -p @storybook/cli@next sb init

# Hack to make `@storybook/addon-links` works with Yarn 2 for now
yarn add @storybook/client-api@next

Link preset-create-react-app package to the local version, to do so add in package.json:
"resolutions": {
    "@storybook/preset-create-react-app": "portal:/PATH_TO_SB_PRESETS_REPO/packages/preset-create-react-app"
}

Finally, run yarn storybook 🎉

@mrmckeb Thanks for your feedback! I fixed them in a few more commits, however, I can't push (even with -f 😄) them to my own branch on GH 🤔 I have the following error:
remote: Resolving deltas: 100% (14/14), completed with 4 local objects.
remote: error: GH006: Protected branch update failed for refs/heads/add-yarn-2-support.
remote: error: 8 of 8 required status checks are expected.
To https://github.com/storybookjs/presets.git
 ! [remote rejected] add-yarn-2-support -> add-yarn-2-support (protected branch hook declined)
error: impossible de pousser des références vers 'https://github.com/storybookjs/presets.git'

I'm not admin for this repo, so can you check in the settings if this branch is listed as protected? Or anything else that can explain this error 🙃

Sorry, there were some changes that @shilman and I were making yesterday.
When this PR is merged, we'll have a new CI - I think the check requirements were accidentally applied to all branches. I'll try to look this morning with @shilman.
https://github.com/storybookjs/presets/pull/103
Maybe subscribe to that PR, then when it's merged you can rebase and we can merge this in. Again, I'll work on branch protection today)

@mrmckeb new commits pushed 🚀!

Thanks @gaetanmaisse. We're still waiting for this to merge unfortunately, I'll chase @shilman again :)
https://github.com/storybookjs/presets/pull/103

Hi @gaetanmaisse, this has finally been merged - if you can rebase/solve conflicts, I think we can get this out!

@mrmckeb, great! I will do it as soon I have time too and tag you back 😉

@mrmckeb thanks for merging this PR 🙇 🎉 ! Are you the release guy too or is it @shilman? Do you know when the next release is planned?


New version (1.9.22) asks for .NET 8.0.0, even tho it's already installed

Previous version (1.9.21) didn't, and everything worked fine

Same.

how to install the old version then? or is it not possible?

Go to releases and download the second one
El sáb, 13 de ene de 2024, 8:22 p. m., carburador77 <
@.***> escribió:

how to install the old version then? or is it not possible?
—
Reply to this email directly, view it on GitHub
https://github.com/shaked6540/YoutubePlaylistDownloader/issues/219#issuecomment-1890792381,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APW7ADAFOFPEJDL6XRJJU2TYOMJJ7AVCNFSM6AAAAABBW2VMAOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJQG44TEMZYGE
.
You are receiving this because you commented.Message ID:
@.***>

I love this program but it's always the same issue with .NET, every time... it asks us to download the wrong one

so what worked for me was APS.NET Core Runtime 8.0
Unfortunately, the installer links to the wrong version, again

Same issue

so what worked for me was APS.NET Core Runtime 8.0
Unfortunately, the installer links to the wrong version, again

This worked! Thanks! Windows thought it was the same file but running it worked!

Just a heads up, playlist downloader links you to the thing it needs, but that thing also needs the base SDK, which it warns you about on the site you're sent to


.depends() for key not being used in a LuCI form

Hey,
I would like to use the .depends() method to conditionally render the LuCI form option based on the value of a key that is not being used or displayed in the form.
From the documentation:

Dependency constraints allow making the presence of option elements dependant on the current values of certain other options within the same form.

My scenario, in which pukrequired is just set by the netifd protocol handler on the fly, and never really needs to be seen or changed by the user.
o = s.taboption('sim1', form.Value, 'puk', _('PUK (Personal Unlocking Key)'),
		_("Your SIM requires a PUK to unlock it."));
o.depends("pukrequired", "yes" );

Is it possible for this to be implemented?  If so, could someone please point me to the relevant code?  If not, any comments on potential workarounds for such a scenario?

I think that you might use DummyValue for pukrequired and use it as a dependency for puk (like now).
That might be be easiest approach.

Thank you, that was straightforward to get the value, although I did need to set a uci value to some HTML in proto handler because I wanted to make the dummy value completely hidden.  Bad smell.  So I added a new member called hidden to DummyValue to allow us to do this quite nicely from anywhere.  I hope this can be accepted in a MR after some review and discussion.
Before
o = s.taboption('general', form.DummyValue, 'pukrequired');
o.rawhtml = true;
o = s.taboption('sim1', form.Value, 'puk1', _('PUK (Personal Unlocking Key)'),
	_("Your SIM requires a PUK to unlock it."));
o.depends({ pukrequired: "<p hidden>yes</p>", simslot: "1" })

After:
o = s.taboption('general', form.DummyValue, 'pukrequired');
// the new member
o.hidden = true;

o = s.taboption('sim1', form.Value, 'puk1', _('PUK (Personal Unlocking Key)'),
	_("Your SIM requires a PUK to unlock it."));
// no more bad smells
o.depends({ pukrequired: "yes", simslot: "1" })

I wanted to make the dummy value completely hidden

Why not just show that on the screen as an informative item?

I wanted to make the dummy value completely hidden

Why not just show that on the screen as an informative item?

In this case, the mere presence of the puk field was considered to be enough information, and the other one was cluttering the UI with non-essential information.


Different system compile error

Hi
18 days in a line Git stats 69 commits  Exist in version.  https://github.com/stressapptest/stressapptest . #96   #97
Making all in src make[1]: Entering directory '/tmp/stressapptest-master/src' make  all-am make[2]: Entering directory '/tmp/stressapptest-master/src' g++ -DHAVE_CONFIG_H -I.     -DCHECKOPTS -Wreturn-type -Wunused -Wuninitialized -Wall -O3 -funroll-all-loops  -funroll-loops -DNDEBUG -MT main.o -MD -MP -MF .deps/main.Tpo -c -o main.o main.cc In file included from finelock_queue.h:29,                  from sat.h:28,                  from main.cc:18: os.h: In static member function ‘static uint64 OsLayer::GetTimestamp()’: os.h:284:11: error: ‘__rdtsc’ was not declared in this scope      tsc = __rdtsc();            ^~~~~~~ os.h:284:11: note: suggested alternative: ‘__dt ’      tsc = __rdtsc();            ^~~~~~~            __dt make[2]: *** [Makefile:326: main.o] Error 1 make[2]: Leaving directory '/tmp/stressapptest-master/src' make[1]: *** [Makefile:192: all] Error 2 make[1]: Leaving directory '/tmp/stressapptest-master/src' make: *** [Makefile:312: all-recursive] Error 1
This error occurs when you are compiling on centos8.3 debian10.9, but not when you are compiling on ubuntu20.04

Hi,
I had a quick look at this using a debian 10 docker container, and could repro the issue. It looks like debian 10 uses gcc 8.3, which needs x86intrin.h include to find __rdtsc() intrinsic. Below patch fixes the problem. No time to check on CentOS8.3, though. I do not claim this is the right solution - stackoverflow seems to think that x86intrin.h is the correct include, but it is obviously not needed for gcc 9.4 used in Ubuntu.
diff --git a/src/os.h b/src/os.h
index 06987f9..f170991 100644
--- a/src/os.h
+++ b/src/os.h
@@ -34,6 +34,7 @@

 #if defined(STRESSAPPTEST_CPU_X86_64) || defined(STRESSAPPTEST_CPU_I686)
 #include <immintrin.h>
+#include <x86intrin.h>
 #if defined(_MSC_VER)
 #include <intrin.h>
 #endif

Oddly I get this exact same error attempting to compile on Ubuntu 20.04 on a USB stick.

The proposed fix worked for me on RHEL 8.6/8.7.

adding helped to compile
include <x86intrin.h>

adding include <x86intrin.h> helped me too on Debian 10.13, without it the packaged version stalled exactly as described here.

The proposed fix worked for me on ubuntu 20.04.

Do you have any timelines on getting this fixed via PR?

@sebhz, do you plan to fix it? This problem is blocking our test on RHEL8.x. Thanks

PR opened: https://github.com/stressapptest/stressapptest/pull/105

https://github.com/stressapptest/stressapptest/pull/105 was merged. This issue can probably be closed.


Parsing of string and \n

Hi,
If a string has a line feed within it's content, it will be parsed as such and not ignored.
For example the string in the array below :
[100.1 (\110\145\n\154\154\157)]TJ
Will be parsed to "He\nllo".
Now I am not sure though what should be the right behaviour in case of valid Carriage Return / Line feed in a text, but I thought it would be escaped ("\n")
Existing code although expect the \n to be kept in string ( and not in Hex)
parse_string("(x\nx)").parse_token.should eql("x\nx")
parse_string("(x\rx)").parse_token.should eql("x\nx")
parse_string("(x\r\nx)").parse_token.should eql("x\nx")

I'm not 100% sure on what the spec requires us to do here. In the absence of additional info I'm going to close this issue, but feel free to open it if you want to continue the discussion.


trivial readme edit

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: cfryanr
To complete the pull request process, please assign brenodemedeiros
You can assign the PR to them by writing /assign @brenodemedeiros in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@googlebot I signed it!

@googlebot I signed it!

@googlebot I signed it!

@googlebot I signed it!


Merge branch main into 3.0

Process:
$ git checkout 3.0
$ git pull
$ git checkout -b <this-branch>
$ git fetch
$ git merge main

Then a seemingly lot of merge conflicts. I ended up regenerating package-lock and thankfully there are not many changes. Just the releases.

Is this a "squash and merge" situation, or a "rebase and merge" situation?

This was not merged with GitHub UI – which would have squashed the history.
Instead I checkout out the 3.0 branch.
git pull
git git merge aug-21-merge-3.0 --ff-only # Ensure this is successful
git push


add program to convert Ensembl HPRC GFF3 to bigGenePred style bigBed …

Ensembl HPRC import

Moved to software/


Compression return code

I figured out that if the compression can not be set, the comp-test script will continue testing with the last set compression type. Inside the test_results files, the user can not figure that out again. Therefore this patch ensures that set compression will return 0 before doing the tests.
You can reproduce this issue with checkout out a plain zfs without zstd support and do a full algo test. The results inside test_results will keep at gzip-9 as this is the last working set compression command in the script.
With this patch, the results file will just skip the invalid compression types instead of should incorrect values from a previous test.

@dan-and You are completely right...
I also came across more issues:

As Brainslayer discovered we should try and read multiple files, because decompression is not threaded.

We should look into using /dev/shm instead of ramdisk, that might be a little more efficient I guess?

Sure there are more things to work on, but I would like to do one step at a time ;-)

@dan-and agreed.
Also:

The advice from brainslayer to use bonnie++ instead might be interesting
Setting prefetch to (at least) 128K also seems reasonable, but i'll await the testing by brainslayer before going byond that.


CSS is not minified on the download page

On page:
http://prismjs.com/download.html
Choosing the Minified version options does not minify the CSS. The Javascript is minified.

Closing as duplicate of #956.


[Tracking][DIF] PLIC

This issue is intended to track the development of the PLIC Device Interface Functions (DIF) library.

[x] DIF library landed (#1282)
[x] In-tree "driver" uses DIF library (no in-tree driver)
[x] Unit tests (#1641)
[x] DV extensions (no longer relevant)

The PLIC DIF library is being implemented by @silvestrst

@lenary I can't seem to edit the issue description, but there are some PLIC RTL changes coming, see #1491 , so the corresponding DIF will have to be refactored accordingly.

In-tree "driver" uses DIF library - not relevant, we won't have PLIC driver in OT proper, it will be Tock driver based on DIF PLIC.
Unittests are merged, DIF is merged, DV uses DIF PLIC in uart test. Please update this issue.

Issue updated, but I think as soon as we start doing signoff meetings, we'll move to the checklist files instead. I'll keep this open until we have a DIF checklist for PLIC.


Wierd Bugs

On Linux and Windows (Chrome Browser) the Jumping is very buggy.
On Mobile (Chrome & Firefox) the Controls are like this
The „Chat“ and „Options“ Buttons aren’t working on Mobile.

Can you make separate issues for each? Thanks

Closing issue for now as separate bug reports should be filed.


fix(ngAnimate): properly cancel-out previously running class-based an…

…imations
Prior to this fix the addition and removal of a CSS class via
ngAnimate would cause flicker effects because $animate was unable
to keep track of the CSS classes once they were applied to the
element. This fix ensures that ngAnimate always keeps a reference
to the classes in the currently running animation so that cancelling
works accordingly.
Closes #10156
Closes #13814
This updated PR adds a new test that ensures the buggy scenario is correctly reproduced. We need to actively start the animation by flushing a requestAnimationFrame, otherwise the addClass / removeClass values are not correct.

@gkalpak I've updated the merge code and added a test for this path. PTAL.

LGTM


Fix miradb fixture test warning

Purpose
Fixes the test warning:
DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated since Python 3.3, and in 3.10 it will stop working
    from collections import Generator

Changes

Import Generator from typing rather than collections

Checklist

[ ] Applicable documentation updated (guides, quickstart, postman collections, tutorial, fidesdemo, database diagram.
If docs updated (select one):

[ ] documentation complete, or draft/outline provided (tag docs-team to complete/review on this branch)
[ ] documentation issue created (tag docs-team to complete issue separately)

[ ] Good unit test/integration test coverage
[ ] This PR contains a DB migration. If checked, the reviewer should confirm with the author that the down_revision correctly references the previous migration before merging
[ ] The Run Unsafe PR Checks label has been applied, and checks have passed, if this PR touches any external services

Ticket
Fixes #434

Good spot, thanks @sanders41


Better playlist documentation

I'm trying to figure out how to download all the songs of a playlist, and the documentation is nonexistent, and the example, leaves a lot to be desired.
Can i get a working example of downloading all the songs of a video, into a directory, and after they are all downloaded, doing something with the song info?

Actually, after running the playlist example you have, it only downloads the first video

I can confirm that the playlist example only downloads the first video

@aequasi playlist is not fully functional there is no cue processing implemented yet.

Until something is added to the library to parse playlists, this (rough) example with calls directly to the executable might help you out @aequasi
var getPlaylistUrls = function() {
    var playListJson;
    // the windows exe is just a version from the website with python prepackaged
    if (process.platform === "win32") {
        playListJson = exec.execFileSync("youtube-dl.exe", [url, "--flat-playlist", "-j"]);
    } else {
        playListJson = exec.execSync("python node_modules/youtube-dl/bin/youtube-dl " + url + " --flat-playlist -j");
    }

    // Remove the trailing \n
    var outString = playListJson.toString().replace(/\n$/, "").split("\n");
    var finalJsonString = "[";
    for (var entry in outString) {
        if (entry == outString.length - 1) {
            finalJsonString = finalJsonString + outString[entry] + "]";
        } else {
            finalJsonString = finalJsonString + outString[entry] + ",";
        }
    }
    var finalJson = JSON.parse(finalJsonString);
    //console.log(finalJson);

    var urls = [];
    // We don't want the deleted videos, since they can't be downloaded
    for (var video in finalJson) {
        if (finalJson[video].title !== "[Deleted Video]") {
            urls.push({
                title: finalJson[video].title,
                url: "http://www.youtube.com/watch?v=" + finalJson[video].url
            });
        }
    }
    console.log(urls);
};

Yeah, I've got something similar to that right now.

@aequasi @pendo324 have a look at latest master this will handle playlist also example has been updated playlist.js
ToDo

[ ] playlist cleanup if video does not exist


Check local containers

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
How do I configure the container so it can access published ports of other containers?
I know that I can use network mode: host or put all containers in one network, but is there another way like granting a cap or something else?
Current compose file:
# Simple docker-composer.yml
# You can change your port or volume location

version: '3.3'

services:
  uptime-kuma:
    image: louislam/uptime-kuma
    container_name: uptime-kuma
    volumes:
      - ./app-data:/app/data
    #ports:
    #  - 3001:3001
    network_mode: host # for communicating with local containers
    dns:
      - <IP_ADDRESS>
    restart: unless-stopped

Thanks in advance!
🐻 Uptime-Kuma Version
1.10.2
💻 Operating System and Arch
Ubuntu 20.04
🌐 Browser
Chrome 95.0.4638.69
🐋 Docker Version
20.10.10
🟩 NodeJS Version

+1 for this. I'd like to monitor internal services within the same docker network, but alas, even with th "Ignore TLS/SSL error for HTTPS websites" option turned on, uptime kuma still counts it as being down.


Function to take in options and produce puml files

Require a function that will take in config and options to be able to:

Only ingest otel data into data holder using config.
Only stream data from data holder to create puml files.
Only extract JSON PV job files from folder and convert to puml files.
String 1+2 together.

def otel_to_puml(
  components: Literal["all", "otel_to_puml", "pv_to_puml"],
  otel_to_puml_options: Optional[OtelPumlOptions] = {
    config: IngestData,
    ingest_data: bool
  },
  pv_to_puml_options: Optional[PVPumlOptions] = {
    file_directory: Optional[str],
    file_list: Optional[list[str]],
    job_name: str,
    group_by_job_id: bool,
  },
  output_file_directory: str = "puml_output"
  ) -> None:

Outcomes:

Tests created
Functionality implemented
Tests passed
Reviewed and approved

PR fixed this


ARROW-9382: [Rust][DataFusion] Simplified hash aggregations and added Boolean type

See individual commits for details. The last one is the commit that adds a new feature - boolean type to group by.
The rational to move some of the code to hash.rs is that hashing is something that goes beyond group by; repartitions, window functions, all have an hashing mechanism. By decoupling the aggregates from the hashing we are preparing the code base to support the other operations.

(Of course I can also take ARROW-9550, if you want!)

Closing in preference of #7936 .


Add support for multiple commands?

It looks like multiple commands aren't supported:
steps:
  - label: ':hammer: Test'
    command:
      - 'echo COMMAND_1'
      - 'echo COMMAND_2'
    plugins:
      docker-compose:
        run: tests
        config: docker-compose.buildkite.yml

I see that it tries to run: docker-compose -f docker-compose.buildkite.yml -p buildkite82973f2986fe406c9a67a03338bca53e run tests echo COMMAND_1 echo COMMAND_2
And my guess is that https://github.com/buildkite-plugins/docker-compose-buildkite-plugin/blob/master/hooks/commands/run.sh#L43 passes in $BUILDKITE_COMMAND as an array.
Would it be possible to have that line in run.sh iterate over each item of $BUILDKITE_COMMAND and run then sequentially?
As a workaround, I'm including the repeated step inside the first command (a shell script), but it'd be nice to specify multiple command to re-use e.g. a script that processes xunit.xml output files.

Probably the more docker-compose like alternative would be to translate it to a single command using /bin/sh or /bin/bash:
steps:
  - label: ':hammer: Test'
    command: >
      bash -c "echo COMMAND_1 &&
                     echo COMMAND_2"
    plugins:
      docker-compose:
        run: tests
        config: docker-compose.buildkite.yml

Do you specifically want to run it multiple times in different containers? I'd be interested hearing more about the specific usecase if you can share.

Hrm that's fair, I'll go the bash route, thanks!
In my case, I have a command that runs tests and generates XML files for unit test results and coverage results. I'm re-using that test in a pipeline for phabricator, and in that pipeline I want to add a follow-up command to upload this data to phabricator. So it's only a slight difference for me to use bash -c "cmd_1 && cmd_2".
I was mostly hoping for multiple command to work since it's consistent with running it outside of the docker-compose plugin, and it might be confusing to others setting things up that this isn't the way to do it. Feel free to close if you don't think it's worth fixing, just my 2c.

Yup, I hear ya. Torn between the expectations that docker-compose brings and buildkite-agent does :)
Let's leave this open and see what other people think. Thanks for raising the issue!

@lox It would be nice if buildkite automatically translated multiple commands into the bash format.  I am running into the issue of having 2 commands.  The first being a wait_for_it.sh script that allows me to wait for a mysql container to start.

This limitation (which is completely understandable) should probably be documented in the README - even though nothing in the README gives the impression that multiple commands would be supported, it'd have saved me a bit of debugging.

I think we've settled on making /bin/bash -c the default "shell" and letting folks customize this with a shell directive.

@lox On further testing, I can't actually get the bash -c workaround to work, because of escaping applied to the run command. For example, using a command like:
  - label: "single quotes, folded"
    command: >
      sh -c 'echo foo && echo bar'
    plugins:
      docker-compose#v1.8.2:
        run: some-container

Results in a run line like
docker-compose -f docker-compose.yml [...] run some-container sh -c \'echo foo \&\& echo bar\'

Which is a foo: line 1: syntax error: unterminated quoted string. Same goes for all combinations of double and single quotes I've tried (including using a simple YAML string as opposed to a folded one). I just can't get the plugin to leave my command alone and pass it as-is into the run line.
(The obvious workaround in the meantime is to shove the two commands into a script, of course)

I think we only fixed that recently in https://github.com/buildkite-plugins/docker-compose-buildkite-plugin/pull/95.

I agree with @dominics, would be nice to have that explicitly documented. It was my expectation that commands and command would work out of the box as this is a feature of buildkite however I can see this is not how it was designed to work. Maybe explicitly stating unsupported features in the readme would be helpful?

Unfortunately && would need a shell to interpret, which lots of docker containers don't have.
The plan currently is to detect commands that need a shell and show a warning. I'd suggest being explicit and using /bin/sh -e -c "Hello world; Hello Universe" as your command to get the semantics you are expecting (provided /bin/sh) is in your image.

@lox is it true that we can’t guarantee a shell will be present? What happens if a shell script is used as the command, but there’s no shell? Will this simply fail?
I’d like to make sure we document this caveat, because I don’t believe it’s obvious!

Unfortunately && would need a shell to interpret, which lots of docker containers don't have.

Do you have an example of this? I’d be surprised that sh wouldn’t even be common to them

My suggestion would be to copy the buildkite docker plugin shell option into the docker compose plugin

I think the problem remains that multi-line commands still won’t work, because our scripts don’t properly append them! Subsequent lines appear to end up being run on the host after the docker-compose call.

Subsequent lines appear to end up being run on the host after the docker-compose call.

Hmmm, I didn't realize that, that is next level terrible. I will look at this today.

Do you have an example of this? I’d be surprised that sh wouldn’t even be common to them

@ticky yeah, it's super common. Most golang containers don't have a shell, they just have binary. Also windows images.

Just in case someone else face the same issue, I explained a new approach to run multiple commands on the docker-compose  here


Disable loading of widgets of deactivated plugins

Widgets of deactivated plugins should neither be loaded nor be able to be added to the widget list, since they cause a JS error and crash the widget panel.
This fix ensures that only Shopware widgets or widgets of active plugins are loaded. This also ensures they are not shown in the widget list (therefore cannot be added to the widget panel)
This PR does not introduce any breaking changes.

Thanks for your Pull Request :+1:
One thing, can you please revert the chmod changes of
engine/Shopware/Controllers/Backend/Widgets.php.  The permissions in your PR are changed from to 644 to 755.

I changed the permissions and updated the branch.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/#/issues/SW-16135.
Please use this issue to track the state of your pull request.


macOS: fix iTunes automatically opening

Fixes iTunes opening if it's not running on macOS. I believe this is because of a new cache extension in Sierra, because I'm sure I fixed this bug before...

:+1:


Make reCAPTCHA optional

Current state:

We use reCAPTCHA for anonymous operations that can be abused

For example, "register a new account"
Or, "reset my password"

We believe it is important for any production system to use reCAPTCHA to protect their server and their AWS SES reputation
However, it is an extra burden for people who are just getting started, or running Medplum in a protected environment
Therefore, we'd like to make reCAPTCHA optional

Required fixes:

Allow blank/missing recaptchaSecretKey server config setting to make reCAPTCHA optional on server

See server config settings: https://github.com/medplum/medplum/blob/main/packages/server/src/config.ts
See server config docs: https://www.medplum.com/docs/self-hosting/config-settings#server-configdisabled"

Allow blank/missing RECAPTCHA_SITE_KEY app setting to make reCAPTCHA optional on the client
Handle disabled reCAPTCHA in the two places we use it

New user registration:

Server side: https://github.com/medplum/medplum/blob/main/packages/server/src/auth/newuser.ts
Client side: https://github.com/medplum/medplum/blob/main/packages/react/src/auth/NewUserForm.tsx

Password reset:

Server side: https://github.com/medplum/medplum/blob/main/packages/server/src/auth/resetpassword.ts
Client side: https://github.com/medplum/medplum/blob/main/packages/app/src/ResetPasswordPage.tsx

@tonylee80 - want to take a look at this one?


fix set default value for variable

Previous code set a specific variable value on :host, which does not make sense for variables since we probably want to allow overrides from ancestors.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

replaced with https://github.com/open-wc/open-wc/pull/1491


Cannot find any example how to use errorLink

There is no example how to use Error link. The only lines of mention is: reportClientException
But there are different kind of exception like onGraphQLError, onException with complex inut and output types / streams. Befre update it was simple:
final ErrorLink errorLink =
      ErrorLink(onGraphQLError: (ErrorResponse response) async {
    var isAutheError = response.exception.graphqlErrors;
....
}

ErrorLink is reexported from gql_error_link – here's an example.


fix initialized_pg3 approach following outsourcing of pg3

closes #1383
to be merged after #1384

(this is still about 40 seconds slower than before for some reason)


2.2.3 For 1.14.4 Server Not working

When i go to join my server with your mod on it come up with
Internal Exception:io.netty.Handler.Codec.EncoderException:java.lang.nosuchFiedError: INGREDIENT_VANILLA
I did updated my Forge. (28.0.75)
I have other mods it was working with before. (it's not them because I try going back to an older update where it did work)
i add 2 more thinking it was them, but i remove them and still get the same Error.
Also i tried going back to 2.2.1 Still doesn't work.

edit:
I know it's this mod because i remove it and i could join my server.
Also sorry if spelling is bad not my strong suit.

Apotheosis has not been upeated to forge 75 yet.  Use 45 for now.


Go generate.

Updates the Copywrite header in our generated files to 2023.

The only thing would be to add an automated pipeline that runs at the end of the year that creates this PR, but I think that may be more trouble than it's worth.


Install issue

This is probably not the place for this but in case someone else also encounters this error I will try to solve it and share the results here. When I run
cargo install cargo-generate --features vendored-openssl
I get:
Caused by:
process didn't exit successfully: C:\Users\PC\AppData\Local\Temp\cargo-installD3wUvn\release\build\openssl-sys-a7c49c2d5741077c\build-script-main (exit code: 101)
--- stdout
cargo:rustc-cfg=const_fn
cargo:rustc-cfg=openssl
cargo:rerun-if-env-changed=X86_64_PC_WINDOWS_MSVC_OPENSSL_NO_VENDOR
X86_64_PC_WINDOWS_MSVC_OPENSSL_NO_VENDOR unset
cargo:rerun-if-env-changed=OPENSSL_NO_VENDOR
OPENSSL_NO_VENDOR unset
running "perl" "./Configure" "--prefix=C:\Users\PC\AppData\Local\Temp\cargo-installD3wUvn\release\build\openssl-sys-02ecb9bfd246b8a5\out\openssl-build\install" "--openssldir=SYS$MANAGER:[OPENSSL]" "no-dso" "no-shared" "no-ssl3" "no-unit-test" "no-comp" "no-zlib" "no-zlib-dynamic" "no-md2" "no-rc5" "no-weak-ssl-ciphers" "no-camellia" "no-idea" "no-seed" "no-engine" "no-asm" "VC-WIN64A"
--- stderr
thread 'main' panicked at 'called Result::unwrap() on an Err value: Error { kind: NotFound, message: "program not found" }', C:\Users\PC.cargo\registry\src\github.com-1ecc6299db9ec823\openssl-src-111.25.0+1.1.1t\src\lib.rs:497:39
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
warning: build failed, waiting for other jobs to finish...
error: failed to compile cargo-generate v0.17.6, intermediate artifacts can be found at C:\Users\PC\AppData\Local\Temp\cargo-installD3wUvn
What I tried so far is:
1- Installing strawberry perl. Didn't work.
2- This
3- This
Still trying to solve it.

Since I was battling with this error for approximately 2 hours I forgot to add somethings:
1- I ran win10 machine.
2- Before

Caused by:

there is one more line:
error: failed to run custom build command for openssl-sys v0.9.80

Works like a charm in ubuntu. Also if I just bypass the problematic line it works in windows too. So I guess it is solved.


Include a fixed GeneralStacker

Tracks obs by player rather than concatenating every player's obs together
Uses a faster method for obs stacking

I think this fix is valid for the code in question, but this code doesn't really belong in this repo, so it's not worth reviewing it here.


Config.Yaml Code doesn´t work!

So I am going through the config.yaml documentation and I am pasting this:
lovelace:   mode: "storage"   resources:     - url: "/hacsfiles/button-card/button-card.js"       type: "module"     - url: "/hacsfiles/my-cards/my-cards.js"       type: "module"     - url: "/hacsfiles/kiosk-mode/kiosk-mode.js"       type: module     - url: "https://fonts.googleapis.com/css2?family=Montserrat:wght@100;200;300;400;500;600;700;800;900"       type: css   dashboards:     HaCasa:       mode: "yaml"       title: HaCasa       icon: mdi:script       show_in_sidebar: true       filename: "dashboard/HaCasa/main.yaml"
But I get this error:
Configuration errors
Invalid config for 'lovelace' at configuration.yaml, line 26: Url path needs to contain a hyphen (-) for dictionary value 'lovelace->dashboards', got {'HaCasa': {'mode': 'yaml', 'title': 'HaCasa', 'icon': 'mdi:script', 'show_in_sidebar': True, 'filename': 'dashboard/HaCasa/main.yaml'}}
I am pasting as is with no changes.
Help?

Make sure it looks like this:

I can confirm the same issue but this fixed it:
Changing the original dashboard name frome HaCasa: to mini-hass:
Now configuration is checked successfully

@mariosemes that is weird! I was almost certain I changed it all to HaCasa. Can you tell me where you changed this exactly?

I had the same issue, but changed the name in configuration.yaml:
dashboards:     mini-hass:       mode: "yaml"       title: HaCasa       icon: fapro:house       show_in_sidebar: true       filename: "dashboard/HaCasa/main.yaml"

Can confirm, issue is still available. Changing to "mini-hass" fixed the problem.

lovelace:
  mode: "storage"
  resources:
    - url: "/hacsfiles/button-card/button-card.js"
      type: "module"
    - url: "/hacsfiles/my-cards/my-cards.js"
      type: "module"
    - url: "/hacsfiles/kiosk-mode/kiosk-mode.js"
      type: module
  dashboards:
    mini-hass: # Change this line from `HaCasa` -> `mini-hass`
      mode: "yaml"
      title: HaCasa
      icon: mdi:script
      show_in_sidebar: true
      filename: "dashboard/HaCasa/main.yaml"

So I made a small mistake, because mini-hass is wrong. The name works because it has a hyphen which I guess is mandatory since some HA version. So you could change it to hacasa-dashboard but mini-hass will still work. I've updated the docs.


Extracted samples from Sun Banks

The following pull request has extracted samples from the sun banks

I'm going to reject this PR for a few reasons.

Its deleting files that should not be deleted. Namely changes in the Nokia8b folder and others.
Not squashed and rebased.
Too many changes not very well labeled.

But because I want to preserve the SunBank changes, I have created a new PR https://github.com/heyigor/miniBAE/pull/7 with just those changes.
For future PRs, please read https://github.com/heyigor/miniBAE/blob/main/README-pull-request-etiquette.md


Split start page into 2 pages + bump version

Summary
This PR

Splits the start page into 2 pages
Removes the "welcome" page

New users are redirected to the "start" page, not the welcome page

Adds a new "start" button (it's green) and a "go back" link with a little chevron
Updates CSS for lists a tiny bit
Bumps the version so that we can do a release

Screenshots

before
after

1 landing page with all the requirements checks
2 pages: 1 for if they qualify and the 2nd for if they are ready.

N/A

Merging!


Reply to this email directly, view it on GitHub https://github.com/datasciencemasters/go/issues/173?email_source=notifications&email_token=ALLYOWYWSOSX6S2ZK6O63BLRFCVWNA5CNFSM4K5J7QOKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IRAND3A, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALLYOW32KTCLKQEBQ25QVO3RFCVWNANCNFSM4K5J7QOA.


Need to add a space after some field headers in display of unique page of a record

When you expand the record in a normal results display, the fields properly display the head (for example, "Variants:") followed by a blank space after the colon. But when you go directly to a page -- https://deep.pennds.org/1114/ -- there is no space after the colon.
Add a space in this view. The following field headers are affected:
Date of First Production (Annals):
Total Editions:
Variants:

This should be fixed automatically when you copy the code from the results display version of the record to the individual page version of the record.

Closing because I added #188


Edited the eda.ipynb notebook

Please accept PR :)

which codes are different? could you give me an example? As Ruiz made the changes, I don't know much about it.

I didn't mean it had code that is different from main, I meant the files suppose to only have the changes Ruiz made, not all the history change (the changes I made this afternoon). But after you Merge branch 'main' into pipeline, it resolved the issue.


beta7 IMigrator.Apply creates SQL Server Database but no tables

I have a project that needs to support Sqlite for development and SQLServer for production. I managed to get this to work and correctly migrate the database using beta 6, but in beta 7  dbContext.Database.GetService().Apply() creates the database but no tables. I have two separate assemblies that create the migrations: 1 for Sqlite and 1 for SQLServer. Each has a DbContext that inherits from my runtime DbContext and each has a Startup class with the appropriate sql connection to generate the migration classes. I use dnx ef migrations add to create my migration and it looks fine. Then, I change the DbContext attribute in the generated migration classes to point to my runtime DbContext. However, at runtime IMigrator.Apply() runs, creates the database, but no tables, and does not throw an error. I can see that _migrationAssembly.Migrations has the correct migration, but, again, only the database is created and none of the tables. Am I missing something?

This turns out to have been an error on my part. I was not doing anything with my caught Exception variable so it was never populated. However, the error was being logged which I hadn't noticed.


refactor: encapsulation of ChangeLock

What is changed and how it works?

Issue Number: ref #13694
What's Changed:

Encapsulate last_change_ts and versions_to_last_change into one enum.

Related changes

PR to update pingcap/docs/pingcap/docs-cn:
Need to cherry-pick to the release branch

Check List
Tests

Unit test

Side effects
Release note
None

@MyonKeminta PTAL

/test

/merge


character-count displays undefined when maxlength and maxwords are both undefined

https://github.com/alphagov/govuk-frontend/blob/1d2fcb655361b52efcd851b7b9da3b0e77f05c83/src/govuk/components/character-count/template.njk#L29
When maxlength and maxwords are both undefined, the following message is observed:
You can enter up to undefined characters

Hello @omair-ali,
The maxlength or maxwords options are required for this component to work. So you will need to define these if you want the component to work as expected.
Is there a usecase you've come across where you would want to use this component without either of these required options?

Hi @nickcolley,
Thanks for reviewing this request. The reason we have raised this is because there is no requirement / validation criteria. The users of this component can pass in undefined and nothing restricts them or invalidates the input suggesting either has to be defined.
We are looking at this from our Scala Twirl based lib which mirrors this. In order to have a parity with govuk-frontend we think if this requirement is incorporated here then our mirrored lib can restrict our client too from not providing one of the values. If we are going to enforce this then our lib will start diverging from govuk-frontend. HEnce, we think this could be a useful addition here too. Please advise, thanks

With our Nunjucks templates we don't have a way to validate required fields and display an error or warning to our users.
The required options are documented in the 'Nunjucks tab > Nunjucks macro options' section the website.
We recommend that people validate their data before they pass it to inputs rather than relying on the templates themselves which contain minimal logic.
If we were to change this we'd have to do this for all templates not just the character count component.
We're open to any suggestions that could make it easier for you to port these templates but I'm not aware of a way to do validation of required fields within Nunjucks templates.
Interested to hear your thoughts.

@natcarey any thoughts?

Thanks @nickcolley.
So in HMRC we're building a port of govuk-frontend, I know that OPSS are doing the same.  Our testing strategy is based on an a black-box comparison of the html generated by the components in govuk-frontend  with the html generated by our port when using the same component and the same input.
If we output the same HTML from the same input Parameters then the CSS/JS/Assets will all be compatible.
With this approach I advocate for "feature parity" and "bug parity" - if we come across something we feel is a bug (in this case telling the user "You can enter up to undefined words") then we should match the behaviour.  If we act differently in cases we perceive as bugs then we no longer have an unopinionated  port.
When we encounter things we perceive as bugs we intend to match the behaviour but raise the issue.   So without a change from GDS we'll implement the "feature" that leaving both parameters empty produces "You can enter up to undefined words".
One of the reasons we like to raise these things is that we know we're quite intensive users and we might spot things that were undesirable.
Outcomes of this are:

GDS change and HMRC change in-step
GDS don't change and HMRC match the behaviour
GDS don't change and HMRC decide a different behaviour for this case

In future we can raise these kind of issues or not raise them, we're always happy to raise solutions too like we did with https://github.com/alphagov/govuk-frontend/issues/1618

Heya @natcarey :)
As far as I know this behaviour is shared by all components, we assume that users will input require options since they would not work otherwise.
As it stands I'm not sure of a way we can make examples like this throw errors or let users know they failed to use required options.
One thing I have considered is that the component does not output a message if the user fails to include these options but that could arguably be less helpful than an 'undefined' message as at least an 'undefined' message provides feedback that something is not right.
If you have a suggestion of what approach we should consider across all components when a user fails to input require options we'd be open to it.
So if you could help me consider other options that'd be helpful, look forward to your thoughts.

I believe this was fixed in v4.4.0 – creating a character count with no maxlength or maxwords now results in a character count with no count message (effectively just a textarea).


Model 7: Add unincorporate_reference method.

I disabled pclean_lib_test because there's a bug where make_pclean_samples isn't sampling unique entity vectors (and incorporating the same set of entity vectors twice should raise an assertion -- it wasn't doing that earlier, and this PR fixes that). I confirmed that it isn't sampling unique entity vectors at head.


The argument 'num_round' should be 'num_epoch'

python example/mnist/mlp.py
Traceback (most recent call last):
File "mlp.py", line 28, in
model.fit(X=train, eval_data=val)
File "/home/liwei/Documents/mxnet-master/python/mxnet/model.py", line 743, in fit
**(self.kwargs))
File "/home/liwei/Documents/mxnet-master/python/mxnet/optimizer.py", line 47, in create_optimizer
**kwargs)
TypeError: init() got an unexpected keyword argument 'num_round'

Sorry I will fix the example today. Recently we decide to change the
convention, we rename epoch to batch, and round/iteration to epoch. So
num_epoch will work.
On Thu, Oct 29, 2015 at 04:28 Wei Lee<EMAIL_ADDRESS>wrote:

python example/mnist/mlp.py
Traceback (most recent call last):
File "mlp.py", line 28, in
model.fit(X=train, eval_data=val)
File "/home/liwei/Documents/mxnet-master/python/mxnet/model.py", line 743,
in fit
*
*(self.kwargs)) File
"/home/liwei/Documents/mxnet-master/python/mxnet/optimizer.py", line 47, in
create_optimizer **kwargs)
TypeError: init() got an unexpected keyword argument 'num_round'
—
Reply to this email directly or view it on GitHub
https://github.com/dmlc/mxnet/issues/420.


chore: split the common module

One system test failed for me. I had to apply this patch:
diff --git a/system-test/spanner.js b/system-test/spanner.js
index f3a92af..4803de0 100644
--- a/system-test/spanner.js
+++ b/system-test/spanner.js
@@ -3603,9 +3603,20 @@ describe('Spanner', function() {
             },
           ];

-          assert.throws(function() {
+          const expectedErrorMessage = [
+            `Row at index 0 does not contain the correct number of columns.`,
+            `Missing columns: ${JSON.stringify(['NumberValue'])}`,
+          ].join('\n\n');
+
+          let caughtErrorMessage;
+
+          try {
             transaction.insert(table.name, rows);
-          }, /Error: Row at index 0 does not contain the correct number of columns\.\\n\\nMissing columns: \["NumberValue"\]/);
+          } catch (e) {
+            caughtErrorMessage = e.message;
+          }
+
+          assert.strictEqual(caughtErrorMessage, expectedErrorMessage);

           transaction.end(done);
         });

Nice catch!  And thank you for the patch.  I'm really looking forward to being able to run system-tests automatically via kokoro.  Fix applied.


Fixes case insensitive comparison in non-english locale problem

TabMon cannot run on computers with locale settings other than English. I set the locale of the datatable used to en-US to fix this problem.

Thanks for looking into this. I changed the base to dev so I can push to master once vetted.


Implement WeatherKit

https://developer.apple.com/weatherkit/get-started/

This requires a $100/yr subscription


"Delete" does nothing in Functions List view

I wanted to delete some refactoring functions to test the import/export functionality, but 'Delete' in the 'Functions List' view does nothing. There are no information/error boxes and the function is still visible.

Fixed! An info message was added that explains that it is now allowed to delete records.


Support loading individual FileDescriptorProtos

Currently the entire set of file descriptors must be built into a FileDescriptorSet at once. It may be useful to add files to an existing set.
We can use Arc::make_mut to clone the FileDescriptor if anyone has references to it.
Currently the descriptor API also doesn't support inspecting the file structure at all, the closest we can get is looking at package_name.

It may also be desirable to share a FileDescriptor between multiple other dependent FileDescriptor instances. For example if we store a static instance of the google.protobuf file descriptors to implement #10 then we could share that instance to avoid duplication.

Copying DescriptorPool from the C++ API probably makes sense here. For example
struct DescriptorPool { ... }

impl DescriptorPool {
    ... existing FileDescriptorSet apis moved here ...

    fn new() -> DescriptorPool;

    fn add_file(&mut self, f: FileDescriptorProto);
    fn add_file_set(&mut self, f: FileDescriptorSet);
}


algolia search api hidden

Algolia search api and other tokens hidden with js Obfuscator for safe use of the api key
This way the api key cant be easily copied up by just viewing the source codes.

Thanks for the PR @midhruvjaink, but as @xtuc mentioned, the keys only have read-only access to the babel website search index provided by Algolia.
For mroe info check out:
https://community.algolia.com/docsearch/
https://www.algolia.com/doc/api-client/javascript/api-keys/#generate-key
/cc: @maxiloc


Feature properties are inconsistent (OCC-91)

Describe the bug
The features "Orchard Core Commerce", "Orchard Core Commerce Session Cart Storage", "Orchard Core Commerce Settings Currency Selector" are all under the Commerce category and use the same name pattern. The features "Orchard Core Commerce - Tax" and "OrchardCore.Commerce.ContentFields" both use different naming patterns, and are under the Content Management feature.
To Reproduce
Search for "commerce" under the Features admin page.
Expected behavior

All OCC features under the Commerce category.
Consistent naming pattern. I recommend "Orchard Core Commerce - ".

Screenshots

Jira issue

Good point, Same for the recipes too.


ComboBox: Move filtered options out of state

FilteredOptions in ComboBox are derived from other state variables and should not be part of state. Also its creating this issue reported by @bvleur :
"If you arrive at a input value matching an option by pressing backspace, the isMatchingOption won't be set to true."

Fix:
I do not wanted to move logic for calculation of filtered option in render method as that calculation may not be needed each time combobox is  rendered. I have rather moved it to a variable (instance variable) of ComboBox, so that we calculate it only when:

Component is initialized or receives props.
Value of component changes.


refactor: rename rmrk2 to ksm

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[ ] Feature
[ ] Refactoring

Context

[x] Closes #5418

Before submitting pull request, please make sure:

[ ] My contribution builds clean without any errors or warnings
[ ] I've merged recent default branch -- main and I've no conflicts
[ ] I've tried to respect high code quality standards
[ ] I've didn't break any original functionality
[ ] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </bsx/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Had issue bounty label?

[ ] Fill up your KSM address: Payout

Community participation

[ ] Are you at KodaDot Discord?

Screenshot 📸

[ ] My fix has changed something on UI; a screenshot is best to understand changes for others.

The thing is that once marck will be 100% ready we plan to:
merge rubick@008 & marck@001 into rubick@9.
This will use only one prefix.
I am aware that renaming RMRK to KSM should be an issue. Because it is hardly baked into code.
Therefore proposal to rename RMRK2 to KSM. Should not be an issue.
Rmrk2 is currently in alpha/internal testing so breaking urls should not be an issue 🤗

and what would be shown in the gallery detail? Also kusama?

If I understand that correctly rmrk1 is kusama as well - so we won't distinguish between RMRK1 & RMRK2

and what would be shown in the gallery detail? Also kusama?

If I understand that correctly rmrk1 is kusama as well - so we won't distinguish between RMRK1 & RMRK2

Yup both are kusama based NFTs, they use same protocol, but different version (obtained from indexer) and as I mentioned they will be merged together soon.

@vikiival oki makes sense, would be nice if its merged together, since both version dont have offers anyway :D 🚀
thanks

Yup both are kusama based NFTs, they use same protocol, but different version (obtained from indexer) and as I mentioned they will be merged together soon.

Statemine be like

merge rubick@008 & marck@001 into rubick@9.
This will use only one prefix.

how we will distinguish nfts on frontend if it's rmrk1 and rmrk2? @vikiival

how we will distinguish nfts on frontend if it's rmrk1 and rmrk2?

We have version stored on the Subsquid
{
  "data": {
    "nftEntities": [
      {
        "name": "Mohawk",
        "version": "2.0.0"
      },
      {
        "name": "Earring",
        "version": "2.0.0"
      },
      {
        "name": "MOONSAMA SPECIAL",
        "version": "2.0.0"
      },
      {
        "name": "MOONSAMA SPECIAL",
        "version": "2.0.0"
      },
      {
        "name": "Referendum 201",
        "version": "2.0.0"
      }
    ]
  }
}

version 1 would be 1.0.0

anything we are waiting for?

anything we are waiting for?
Yup I would like to deprecate /rmrk2 urls so anything which will run on KSM will have /ksm prefix in url

yup, until we are showing in description something like ksm/rmrk1 and ksm/rmrk2 it's good to go

I'll make follow-up issue


[Proposal] Introduce unified way how to track agent quality

Proposal
I would very much appreciate there to be a unified way to find out how well did agent do, in a predefined range for all environments. For example, a method that returns 1 on "perfect agent" and 0 on "noop-agent" or the worst agent.
Motivation
I am a student at CTU and I'm trying to learn reinforcement learning. This would be my 3rd attempt, both attempts before were unsuccessful. Besides personal reasons, something that is regularly counter-intuitive to me and took me some time to get over is that there is no "solved :fireworks: :fireworks: :partying_face: " fanfare for the agent.
The success looks like agent getting evaluation score at -20 instead of a -100 score, which is extremely underwhelming and not unified across environments. (What is this environments best score? Is it -20? Is it -10? Or perhaps +100? There is no way to programmatically find out and I gotta check the docs)
For example, this is output from a stable baselines training progress:
----------------------------------
| rollout/            |          |
|    ep_len_mean      | 70.4     |
|    ep_rew_mean      | -70.4    |
|    exploration rate | 0.05     |
| time/               |          |
|    episodes         | 4884     |
|    fps              | 704      |
|    time_elapsed     | 283      |
|    total timesteps  | 199689   |
| train/              |          |
|    learning_rate    | 0.01     |
|    loss             | 0.0816   |
|    n_updates        | 49672    |
----------------------------------

Is -70 good? Is -70 bad? I have to actively think about it, which is putting more mental burden on coders. A single number showing "0.99" would be just amazing.
Pitch
Introduce an API to gymnasium.core.Env to facilitate evaluation of agents. With the following signature:
class Env(...):

   ...

    def rate(self) -> float:
        """
        Rate method would rate the last run on a scale of 0 to 1, where 0 is the worst run possible,
        and 1 is a perfect agent that took all the right steps.

        :return: A float representing the rating of the last run.
        :raises NotImplementedError: If the method is not implemented by the environment author.
        :raises RuntimeError: If the env hadn't finished yet
        """
        raise NotImplementedError

When rate() returns > 0.95, the agent can be considered "good enough" and the environment "reasonably solved".
Both of which definitions are up to the environment author.
Alternatively, the default implementation of rate could just be last_total_return / best_possible_return with some math magic to account for negative rewards.
Alternatives
Any alternative has the issue that newcomers don't expect this to even be an issue. As a newcomer, I expect to be able to see, trivially, if the last agent "did well". Not to code an entire patchwork of approaches to approximate this. With that being said
Ad-hoc computation:
:heavy_plus_sign: Would be flexible
:heavy_minus_sign: Would be a huge function, with definitions of rate() for each environment
:heavy_minus_sign: Wouldn't be integrated into other frameworks ( Such as SB3 )
:heavy_minus_sign: Would break if the environment gets updated with a different reward system
RewardWrapper:
:heavy_plus_sign: Universal for all environments
:heavy_minus_sign: Cannot be accurate, at best it can tell how well did the agent perform compared to other agents
Gymnasium-Robotics/GoalEnv:
:heavy_minus_sign:  Enforces different observations
:heavy_minus_sign: Is not installed by default
:heavy_minus_sign: Doesn't actually help, when the environment I am learning on doesn't extend GoalEnv
Additional context
I seem to remember that environments had some kind of reward_range property, which would alleviate some of these issues, but I can't find it, perhaps I hallucinated it or it was removed?
Additionally, it is my ultimate goal to rate the agents in a human-friendly way. I do not care about this specific implementation.
Checklist

[X] I have checked that there is no similar issue in the repo

There already exists a reward_thre shold for some environments
not all environments have a best_possible_return (or in many cases it is not known)

Oh you're right, I didn't think to check the registry for this information, thanks! You may close the issue then

@richard-hajek you can close you own issues
https://docs.github.com/en/issues/tracking-your-work-with-issues/closing-an-issue

I am perfectly aware, but sometimes maintainers want issues to stay open for sometime to allow for more discussion.


Add static code analalysis

based on branch from @RobertSluka #85 but modified commit history to drop empty commits and remove changes to readme

closes #85


Arbitrary-less law checks

addresses https://github.com/garyb/purescript-quickcheck-laws/issues/35

I like this :+1:

Supercededby #57


is it possible to use same keys in listBox array

Perhaps I am missing something.
I want to create a:
$field->listBox([
'val1'=>'label1',
'val1'=>'label1',
'val2'=>'label2',
])

Note that I want to have "val1" more than once, since there are cases where I need the same key to be in the list many times, especially on multiple select. Using pure HTML and PHP I can create a select with the same val option more than once and successfuly POST those on PHP as distinct array entries... So it is possible.
The problem is that arrays cannot have the same Key more than once, so how can I implement this with an active form field?
Same issue applies to all field types that have the $items parameter I believe...

This is not currently possible. The fix is not trivial.

Yes, I understand that due to the current functionality, its difficult, if not impossible to make the same function backwards compatible. Plus this affects all field types that use the $items array parameter...
Not sure how this could be resolved elegantly without B/C.

like you both are stating: this is hard in the current architecture. A lot of parts have to be rewritten for this rare usecase. It would be easier to fix this with a trick. You assign random numbers to the keys beforehand (perhabs after an underscore) and after rendering you remove the these numbers with a regex.


[SMALLFIX] fix hive checker jar path

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/20302/
Test PASSed.

Merged build finished. Test PASSed.


Contact form: From- and To-addresses

Currently the contact form is only able to send mails to @wh2 accounts, but has to support @agdsn.de addresses.
The mail-field in the user form should not be editable, but the mail forward may be linked there.

Although it is possible to send the mails directly to<EMAIL_ADDRESS>this is not an active mailing list. Wouldn't it be more feasible to use the already established<EMAIL_ADDRESS>addresses? This is what I will do in order to leave the contact form in a working state.
If in the long run, a global<EMAIL_ADDRESS>mailing list will have been established, you may reopen the issue and tell me so.


Chore/update venus

关联的Issues (Related Issues)

close
改动 (Proposed Changes)

附注 (Additional Info)

自查清单 (Checklist)
在你认为本 PR 满足被审阅的标准之前，需要确保 / Before you mark the PR ready for review, please make sure that:

[ ] 符合Venus项目管理规范中关于PR的相关标准 / The PR follows the PR standards set out in the Venus project management guidelines
[ ] 具有清晰明确的commit message / All commits have a clear commit message.
[ ] 包含相关的的测试用例或者不需要新增测试用例 / This PR has tests for new functionality or change in behaviour or not need to add new tests.
[ ] 包含相关的的指南以及文档或者不需要新增文档 / This PR has updated usage guidelines and documentation or not need
[ ] 通过必要的检查项 / All checks are green

Codecov Report

Merging #154 (814a139) into release/v1.13 (0421112) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@              Coverage Diff               @@
##           release/v1.13     #154   +/-   ##
==============================================
  Coverage          34.46%   34.46%
==============================================
  Files                 41       41
  Lines               2591     2591
==============================================
  Hits                 893      893
  Misses              1542     1542
  Partials             156      156

Flag
Coverage Δ

unittests
34.46% <ø> (ø)

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


Making after confirmation redirections configurable

For now, in https://github.com/augnustin/pow/blob/master/lib/extensions/email_confirmation/phoenix/controllers/confirmation_controller.ex redirection routes are hard coded:
  defp redirect_to(conn) do
    case Pow.Plug.current_user(conn) do
      nil   -> routes(conn).session_path(conn, :new)
      _user -> routes(conn).registration_path(conn, :edit)
    end
  end

Shouldn't those be configurable just like after_sign_in_path etc.?

I haven't decided how extensions callback routes should work yet #141, and you can still override the routes by customizing path_for/5.


[Snyk] Security upgrade python from 3.12.3-slim-bookworm to 3.12.4-slim-bookworm

Snyk has created this PR to fix 4 vulnerabilities in the dockerfile dependencies of this project.
Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Snyk changed the following file(s):

local.dockerfile

We recommend upgrading to python:3.12.4-slim-bookworm, as this image has only 47 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Integer Overflow or Wraparound SNYK-DEBIAN12-ZLIB-6008963
  132

Out-of-Bounds SNYK-DEBIAN12-GLIBC-1547196
  91

Out-of-Bounds SNYK-DEBIAN12-GLIBC-1547196
  91

CVE-2005-2541 SNYK-DEBIAN12-TAR-1560620
  91

Allocation of Resources Without Limits or Throttling SNYK-DEBIAN12-SYSTEMD-6277507
  86

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Allocation of Resources Without Limits or Throttling

Superseded by newer PRs.


Split storage of default value SQL and computed value SQL

Because not all provider support both concepts we were storing these things as a single string and then letting other metadata determine how to use the value. However, for cases like that in issue #4501 Migrations assumes a computed column because it doesn't know that the value specified is actually a default and not computed expression. The fix is to stop combining the metadata.

:shipit:

cc @lajones do we need to update reveng?

@natemcmaster - am I missing something? Looks like @ajcvickers already updated the reveng code as part of this.

@lajones I didn't investigate thoroughly, but does this change mean, in some cases, we should be scaffolding HasComputedValueSql() instead of HasDefaultValueSql()?

@natemcmaster. I looked briefly and it looks like we should at least revisit it. At the moment we just get back an is_computed column and then use that to decide how to set ValueGenerated. I've raised issue #4944 to track this.


Improve event invocations

We should maintain our own subscriber list, rather than generate one every time we raise an event.

Added


In-flight deposit errors when a payment triggers another deposit

Expected Behavior

Current Behavior

Error is triggered on transfer, after propose install
backend.js:6 Error: {}
    at index.js:10931
    at f (runtime.js:45)
    at Generator._invoke (runtime.js:264)
    at Generator.e.<computed> [as throw] (runtime.js:98)
    at r (asyncToGenerator.js:3)
    at u (asyncToGenerator.js:29)

Steps to Reproduce (for bugs)

@LayneHaber @rhlsthrm can you describe the collateralization scenario that would cause this?

Queues up deposit -> deposits more than it should and runs through collateral

Duplicate, closing


Lack of 'parentComponent' functionality

Hello, I have found that at the moment it's not possible to use the parentComponent functionality to set a parent component or root, I have not found any entries in the codebase. {mocks: {$root}} and none of the other options I found in the documentation or issues for vue-test-utils v1.x doesn't work. Can you tell me if there's another way to use the add parent element functionality or is it just planned?

@Akulko Theoretically, if you are in heavy need, you can always create wrapper around mount which will provide your functionality by creating own component, and doing find. Something like (pseudo-code):
function mountWithParent(ChildComponent, { parentComponent: ParentComponent, ...mountOptions }) {
  const wrapperComponent = {
     inheritAttrs: false,
     components: { ParentComponent, ChildComponent },
     template: '<ParentComponent><ChildComponent v-bind="$props" /></ParentComponent>'
  }

  const wrapper = mount(wrapperComponent, mountOptions);
  return wrapper.findComponent(ChildComponent);
}
// ...
mountWithParent(Child, { ..., parentComponent: Map });

(please take a note that proper implementation might also require passing slots to ChildComponent to work for all cases)

Ok, thank you so much!

@matthew-dean did you ever figure this out? I am trying to emulate a parentComponent with my test wrapper. Since I'm testing lots of components that use vuetify, I wanted to wrap everything with a v-app component. I made a wrapper function like so:
import {
  mount as vtuMount,
  shallowMount as vtuShallowMount,
} from '@vue/test-utils';

export function mount(Component, opts = {}) {
  const wrapper = {
    name: 'Wrapper',
    components: { Component },
    inheritAttrs: false,
    template: '<v-app><Component v-bind="$props" /></v-app>',
  };
  return vtuMount(wrapper, makeOptions(opts)).findComponent(Component);
}

but when I try to render a component, the props arent rendered:
mount({
  template: '{{ msg }}',
  props: ['msg'],
  data() {
    return {
      foo: true,
    };
  },
})

simply returns


support polymorphic type and locally abstract type

Close #163

My brain was bent a little with all these “special” type constructs, and I doubted what should be a child/parent of what. Now, looking again, it seems clear and valid. Thanks!


Created resistance experiment

Created resistance experiment according to new template

@CloudyPadmal Changes done :+1:

Breadboard design is not included, components table is not followed.
instead of multiple resistors, please specify exactly which resistors.
Don't add you own section. Please follow the template. Is expected results and not observation

https://github.com/fossasia/pslab-experiments/blob/master/contribute/README.md
Read the readme if unsure


qcal: update 0.9.1 -> 0.9.2

Description of changes

update 0.9.1 -> 0.9.2 (release notes)
use substituteInPlace --replace-fail instead of deprecated substituteInPlace --replace (see https://github.com/NixOS/nixpkgs/pull/260776)

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

friendly maintainer ping: @antonmosich

@antonmosich I think you have to approve this again because @ofborg requested another review after you had already submitted yours.


feat(upgrade): updated all dependencies

What are you adding/fixing?
Support for angular 6 & rxjs 6
Have you added tests for your changes?
No
Will this need documentation changes?
No
Does this introduce a breaking change?
No
Other information
There are some warnings when you start the example app for the bundle size and system.import() into @angular/core

Would love to see this supported for Angular 6

Come on guys, just one review and we are ok. News about this?

Would love v6 support soon :)

Well, until the PR get approved I published my forked version
https://www.npmjs.com/package/@hackages/ngxerrors

What blocks this? @mgol @paullinney Is there any reason or we just have to wait.

@tamasfoldi I'm not a maintainer of this module so I can't do anything.

guys, review?

@toddmotto could you have a minute to look into it?

Pleeease 🙏🏻

so is this dead?

Silence here, no any movement?

Silence, why not to merge?

Paging @toddmotto! Is this package dead, or...?

Yes, I did it

How can I help? I am getting a "TypeError: rxjs_Observable__WEBPACK_IMPORTED_MODULE_1__.Observable.combineLatest is not a function"

How can I help? I am getting a "TypeError: rxjs_Observable__WEBPACK_IMPORTED_MODULE_1__.Observable.combineLatest is not a function"
I Just installed and used Hackages package, he updated with latest rxjs@6 and angular@6

This problem is because api of rxjs on version 6 is broken with 5.

Yeah :) I just wanted to fix this issue - but I noticed that ngx-errors is more broken than I thought :( I can't compile on windows. ... no cp / no mkdir -p ... somebody should really fix this. Back here in a few days - i might go with a @hack-hackages/ngx-errors :)

I tried twitter without success.

@egandro maybe this can help you https://stackoverflow.com/questions/35980322/cant-find-combinelatest-in-rxjs-5-0. The problem I think is that you have two combineLatest (operator and function) and you don't use the good import for that.

@anymaniax i'll migrate to your fork, when updating Angular to 6.1.x and beyond this thing breaks like a stick. Besides it's the only package requiring rxjs-compat so i'm leaving to your repo man. All issues, requests, and stuff will be posted over there... be prepared hahaha. I'll help in anything that i can. Greetings.

@egandro maybe this can help you https://stackoverflow.com/questions/35980322/cant-find-combinelatest-in-rxjs-5-0. The problem I think is that you have two combineLatest (operator and function) and you don't use the good import for that.
I think this must be fixed in the ngx source. I will do that tomorrow - forking from your branch and you create a @hackages :)

bump

I give up here. I just copied the 3 files in my project and remove ngx-erros from the dependency list.
Thx for your help! Maybe somebody will fix this project some day...


Rename packages

This PR renames the javy npm module to javy-cli, and introduces a new npm module called javy.
The javy module contains our typings for the Javy-specific code (like the Javy global) as well as the convenience functions layered on top.
I will add tests for the javy package in a separate PR.

CI reports
   |                                ^^^^^^^^^^^^^^^^^^^ help: change this to: `copied_engine_path

but I'm not sure how this change has to do with any of that 🤔

I think that's from the latest version of clippy, if you try and build main with the most up-to-date stable version, you get that error

FINALLY CI IS HAPPY


Images not working

I enjoyed the information given by the page but i would love it if the images would work properly :)

Hello and thank you for opening this issue! Can you provide more specifics? What is not working properly?

sorry i just recognized that i am in the wrong version.
https://release-v0-11.docs.openservicemesh.io/docs/guides/ha_scale/scale/
The images wont work here but for v1 they work. Google sent me to the old guides for any reason.