base.js: use string.endsWith

@jkphl unsure why you were ignoring the case here, but I think this solution should be as close as the previous code, but clearer.
That being said, if renderConfig.dest can be undefined/null or not a string, this will throw now

Not sure I follow why we need to ignore case in the first place? Regardless, unless I'm mistaken, this patch should be equivalent with the previous solution. What you suggest would be new behavior?

@XhmikosR I might be missing something, so please correct me in that case, but I think there's a subtle difference between the patch and the previous behaviour:

Before, the config value renderConfig.dest (as given by the user, could be lower case or upper case or mixed) was matched case-insensitively against a potential file extension, and the extension was appended in case it wasn't already part of the path.
With the patch, the renderConfig.dest is compared to the lower-cased version of the file extension in a case-sensitive way (according to MDN at least). So some/path.CSS (= configured by the user) wouldn't match .css, resulting in some/path.CSS.css.

if (!renderConfig.dest.toLowerCase().endsWith('.' + extension.toLowerCase())) {
    // …
}

should fix this, right?

I know that endsWith ignores case, but the behavior is the same I think. Previously, the regex compared case insensitively the extension, which is the same as this patch, isn't it? You weren't checking render.dest but only the last part, which is the extension.

@XhmikosR I was relying on MDN saying that endsWith() is case-sensitive (without testing this myself). If it's case-insensitive, however, and we can rely on this, then YES, the patch should be fine!

endsWith is case sensitive, hence why I'm lowercasing the extension. Sorry, that was a typo above.

@XhmikosR But wouldn't you have to make sure the user-defined config value renderConfig.dest is lower-cased as well before comparing the two? I should add that for the result it doesn't matter whether the extension is lower or upper case. Instead it should follow the user's wish. But for the comparison we need to make sure that both sides of the equation are normalized, shouldn't we? Hence the renderConfig.dest.toLowerCase() just for comparison purposes. What am I missing? 🤔

Either I'm too tired or you are too :P
So, previously, renderConfig.dest wasn't lowercased either AFAICT. Only the extension part was, right? Which is what I'm doing in this patch too.

So, previously, renderConfig.dest wasn't lowercased either AFAICT.

Yeah, but that's why I was using a case-insensitive regex. Right? 😉

But I'm lowercasing the extension part which is what the regex also checked?

Either way, let's leave this for now and move to another PR. This is a trivial patch, we can revisit later.

Ok, let me try some examples. Previously, the lines in question were:
if (!renderConfig.dest.match(new RegExp('\\.' + extension + '$', 'i'))) {
	// …
}

Assuming extension was "css" (or "CSS", doesn't really matter), and ignoring the negation in the condition, the test would yield:

renderConfig.dest = "path/to/something.css" → true
renderConfig.dest = "path/to/something.CSS" → true
renderConfig.dest = "path/to/something.cSs" → true
renderConfig.dest = "path/to/something" → false

With the new code
if (!renderConfig.dest.endsWith('.' + extension.toLowerCase())) {
	// …
}

we'd always compare to the lower-cased extension, i.e. "css", in a case-sensitive manner, which would yield:

renderConfig.dest = "path/to/something.css" → true
renderConfig.dest = "path/to/something.CSS" → false
renderConfig.dest = "path/to/something.cSs" → false
renderConfig.dest = "path/to/something" → false

Please tell me if I made a mistake, but I see quite some differences in the results?

No, you are right. My mind was stuck at believing that lowercase === ignoring case. Let's drop this for now and I will revisit later if needed.
Thanks for pointing out the obvious, I need to sleep more :)

@XhmikosR No worries! I very much appreciate your efforts, so I'm just trying to look at least somewhat eager. 😉


Merging dev to main for release 3.1.1

Codecov Report

Merging #896 (2392cfd) into main (9cdfce8) will decrease coverage by 0.13%.
The diff coverage is 72.52%.

@@            Coverage Diff             @@
##             main     #896      +/-   ##
==========================================
- Coverage   86.89%   86.75%   -0.14%
==========================================
  Files         175      175
  Lines        4509     4576      +67
  Branches     1532     1563      +31
==========================================
+ Hits         3918     3970      +52
- Misses        290      301      +11
- Partials      301      305       +4

Impacted Files
Coverage Δ

...t-magma-dom/src/components/Accordion/Accordion.tsx
100.00% <ø> (ø)

...ma-dom/src/components/Accordion/AccordionPanel.tsx
100.00% <ø> (ø)

...mponents/FormFieldContainer/FormFieldContainer.tsx
100.00% <ø> (ø)

...t-magma-dom/src/components/PasswordInput/index.tsx
100.00% <ø> (ø)

...es/react-magma-dom/src/components/Search/index.tsx
88.88% <ø> (ø)

...ges/react-magma-dom/src/components/Table/Table.tsx
88.88% <ø> (ø)

.../react-magma-dom/src/components/Table/TableRow.tsx
60.93% <60.00%> (-0.43%)
:arrow_down:

...react-magma-dom/src/components/InputBase/index.tsx
75.43% <65.30%> (-0.47%)
:arrow_down:

...a-dom/src/components/Accordion/useAccordionItem.ts
100.00% <100.00%> (ø)

...act-magma-dom/src/components/Datagrid/Datagrid.tsx
92.00% <100.00%> (+0.51%)
:arrow_up:

... and 6 more

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


Fixed the main cause of missing baseflow. Variable name primary & secondary fluxes were not consistent

[Short description explaining the high-level reason for the pull request]
Variable names were inconsistent. Fluxes from both soil & groundwater storages were named as primary_flux and secondary_flux in the main execusion vs. primary_flux**_m** and secoundary_flux**_m** in conceptual_reservoir_flux_calc.
As a result, the baseflow was entirely missing in CFE. Fluxes calculated in conceptual_reservoir_flux_calc were not passed the main flux calculation.
I fixed the bug in the Python version on bmi-python, but here I am trying to make a similar pull request in the C version.
Changes

Changed variable name 'primary_flux' to 'primary_flux_m'
Changed variable name 'secondary_flux' to 'secondary_flux'
On both soil and groundwater storage

Testing

Not tested as I am not proficient in C. I appreciate somebody could check it.

I realized that I accidentally closed the pull request and branch.
Reopened because the issue is still there (may not automatically merge, so please see the suggested changes).


Feature: Enable caching in argo listing

🔖 Feature description
Enable argo listing through cached resources.
🎤 Pitch / Usecases
Currently we list all cluster's argo cd apps which gets quite heavy sometimes. To add cache usage support to avoid this bottleneck.
🔄 Alternative
No response
👀 Have you spent some time to check if this issue has been raised before?

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

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

fixed with attached PR


Allow jira to pick up on multiple issue types

At present the jira collector can only collect on one issue type. We want to be able to pick up estimates assigned to other types (such as investigation spikes and defects). This change allows multiple issue types to be specified in the config.
Note the name of the config item was corrected so configs will have to be updated.
@pxk5958

Added as part of #974


Return Markdown files to /docs directory

In PR #2342 I moved some existing Markdown to a Hugo-specific directory without realizing that it needs to stay in the /docs directory. This PR returns those files to their original home.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #2344   +/-   ##
=======================================
  Coverage   44.99%   44.99%
=======================================
  Files          92       92
  Lines        9336     9336
=======================================
  Hits         4201     4201
  Misses       4457     4457
  Partials      678      678

Flag
Coverage Δ

#linux
49.29% <ø> (ø)
:arrow_up:

#windows
41.25% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Looks like the right set of files; we also probably want to remove the small markdown table that is used by Hugo but gets rendered in GH (title, draft status)


Adding and duplicating components

The component-ids for the meta components while adding should be same in both components dispatch action and code dispatch action.
As of now,  componentIds for the meta components are stored as key-value pair. Type is stored as the key and the component-id is stored as the value. After assigning the values, the componentIds is sent to the babel plugin and is stored. There is a problem in this method, There can be two or more similar components in a meta component. At that time, the key-value pair. So we have to find another way to do it.


Update dfp-model-card.md

Updating per Model Card++ 3.0 Release.
Description

Closes
By Submitting this PR I confirm:

I am familiar with the Contributing Guidelines.
When the PR is ready for review, new or existing tests cover these changes.
When the PR is ready for review, the documentation is up to date with these changes.

Hey Devin!  Can you get this merged so I can get this approved?

@HesAnEasyCoder Can you update your PR to target the current development branch: branch-24.06? Then we can run CI and get this merged.

Done!

/ok to test

/merge


Turn the example model run into a testing fixture

What existing problem does the pull request solve and why should we include it?
In order to test most components, we need to execute a model run.  However, we don't want to execute a model run for every test.  This Pull Request runs a model run (on the example network) and then executes a series of tests on that model run so that we can independently test each component.
What is the testing plan?
The change IS the test.

[x] Code for this PR is covered in tests
[ ] Code passes all existing tests @inrokevin or @i-am-sijia please test on an EMME-licensed machine

Code formatting
Code should be PEP8 compliant before merging by running a package like black

[ ] Code linted

Applicable Issues
Closes #66

duplicate!


pt-br language added to supported texts

I made some pt-br translation for the current supported texts.

Hi @recs182 you should update translation. Some new things has been added

I'll update as soon as I can! Thanks for letting me know.


[Improvement  Request]Report sharing

Prediction report will be shared via email, rendered using HTML. So it requires to pick report of a patient from api and then send it to the given email.

#LGM-SOC you can assign it to me whenever you want as it need to be done in flutter it will take me around 5-10 days to complete it as I don't have experience on Flutter

Actually, the app was designed for doctors. So I think that the feature is useless.

ok we can improve this from SOS to Doctor help button which can share notification to doctors for help for example
I am a heart specialist so I need a recommendation for some cases I can click on this btn and add in patient details and that can be shared with other heart specialists?

Report sharing is a part that needs to be implemented. We can integrate your suggestion with that.

ok can you can tell me a bit about that what you have in mind about report sharing feature as in flutter side its just input fields main work has to be done on python side to send it as message/ Email that I can do easily. so if I can get more information about that this will be good for me.

Prediction report will be shared via email, rendered using HTML. So it requires to pick report of a patient from api and then send it to the given email.
You can create a new issue and start working on it.

let me update this issue for this only feature and I will start working on this.

Should I Assign this to you?

yeah sure @Ashuto7h

hi can you can share me the database structure of your database I need to check it out for my report sharing options

you can access the collection using /col/test url.
It has everything under one collection only.

ok cool will do that


Can't able to login through by different PC

Expected behavior:
should sign in and show dashboard
Actual behavior:
Generating Error:
[Report Only] Refused to load the script 'http://ipaddress:7020/livereload.js' because it violates the following Content Security Policy directive: "script-src 'self' 'unsafe-inline' 'unsafe-eval' ipaddress:7020 <IP_ADDRESS>:7020 undefined:7020".
Steps to reproduce:
host on node server and try to access through by different PC on local network
Screenshots (if applicable):

OS and Browser:
Windows 10, Browser google chrome

Facing the same issue here. Anybody please help.


Added test to check multitarget images consistency

We need to verify that when we use multitarget and apply a transform to the equal two images, transformed images are also equal.

it was added in one of previous PRs that you approved https://github.com/albu/albumentations/pull/214/files#diff-c14a2f858bf526cb9ca600fb52c50342


jax RuntimeError: Invalid argument: Cannot copy value to device 'gpu:0' with 'cpu' backend on trying to run W7X example

Hi, I wanted to try running your code on the GPU, but got the following error:

Code execution command, error
dominik@dell ~/Code/DESC % XLA_FLAGS=--xla_gpu_cuda_data_dir=/opt/cuda python desc examples/DESC/W7X -g 0 -p                                                                        [1]

 ____  ____  _____   ___
|  _ \| ___|/  ___|/ ___|
| | \ | |_  | (__ | |
| | | |  _| \___ \| |
| |_/ | |__  ___) | |___
|____/|____||____/ \____|

DESC version 0.2.0, using JAX backend, jax version=0.2.6, jaxlib version=0.1.57, dtype=float64
Using device: gpu:0
Reading input from /home/dominik/Code/DESC/examples/DESC/W7X
Output will be written to /home/dominik/Code/DESC/examples/DESC/W7X.output
================
Step 1/10
================
Spectral resolution (M,N,delta_lm)=(4,2,4)
Node resolution (M,N)=(6,3)
Boundary ratio = 1.0
Pressure ratio = 1.0
Zeta ratio = 1.0
Error ratio = 0.0001
Perturbation Order = 1
Function tolerance = 0.0001
Gradient tolerance = 0.0001
State vector tolerance = 1e-06
Max function evaluations = 5000
================
Precomputing Fourier-Zernike basis
Computing initial guess
Compiling objective function
Starting optimization
Warning: Desired error not necessarily achieved due to precision loss.
         Current function value: 15.013052
         Iterations: 3539
         Function evaluations: 3625
         Gradient evaluations: 3612
Start of Step 1:
Weighted Loss:  2.010e+14  errFrho:  1.984e+07  errFbeta:  2.922e+06  errRb:  1.192e-14  errZb:  8.984e-16  errL0:  0.000e+00
End of Step 1:
Traceback (most recent call last):
  File "desc/__main__.py", line 181, in <module>
    main(sys.argv[1:])
  File "desc/__main__.py", line 155, in main
    iterations, timer = solve_eq_continuation(
  File "/home/dominik/Code/DESC/desc/continuation.py", line 362, in solve_eq_continuation
    callback(x, *args)
  File "/home/dominik/Code/DESC/desc/objective_funs.py", line 96, in callback
    errRb, errZb = bdry_fun(
  File "/home/dominik/Code/DESC/desc/boundary_conditions.py", line 226, in compute_bdry_err_four
    lamda = eval_double_fourier(cL, lambda_idx, NFP, vartheta, zeta)
jax._src.traceback_util.FilteredStackTrace: RuntimeError: Invalid argument: Cannot copy value to device 'gpu:0' with 'cpu' backend

The stack trace above excludes JAX-internal frames.
The following is the original exception that occurred, unmodified.

--------------------

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "desc/__main__.py", line 181, in <module>
    main(sys.argv[1:])
  File "desc/__main__.py", line 155, in main
    iterations, timer = solve_eq_continuation(
  File "/home/dominik/Code/DESC/desc/continuation.py", line 362, in solve_eq_continuation
    callback(x, *args)
  File "/home/dominik/Code/DESC/desc/objective_funs.py", line 96, in callback
    errRb, errZb = bdry_fun(
  File "/home/dominik/Code/DESC/desc/boundary_conditions.py", line 226, in compute_bdry_err_four
    lamda = eval_double_fourier(cL, lambda_idx, NFP, vartheta, zeta)
  File "/home/dominik/.local/lib/python3.8/site-packages/jax/_src/traceback_util.py", line 133, in reraise_with_filtered_traceback
    return fun(*args, **kwargs)
  File "/home/dominik/.local/lib/python3.8/site-packages/jax/api.py", line 371, in f_jitted
    return cpp_jitted_f(*args, **kwargs)
RuntimeError: Invalid argument: Cannot copy value to device 'gpu:0' with 'cpu' backend

I'm not sure whether this is jax-specific; I haven't really used it, so I don't know how to reproduce it in a MVE to send over to them.

I can't seem to reproduce it on my end, but I'm using an older version of jax (0.1.76, with jaxlib v0.1.55). We still haven't tested exhaustively on the GPU so its possible the bug is on our end. I'll try to look into it more tomorrow

Good catch, I'll try the older versions today and report back!

I've got it working at least for a few steps; didn't continue yet, because I'm trying to bisect on the jax versions that cause this to fail. It seems kind of independent of jaxlib: pip install --upgrade jax==0.1.77 jaxlib==0.1.57+cuda111 -f https://storage.googleapis.com/jax-releases/jax_releases.html worked. I'm currently between jax 0.2.0 and 0.2.2.

Yup, 0.2.1 works and 0.2.2 fails, thus seems to be the culprit. I saw jax 0.2.5 was reported as problematic in #44; this might be some left-over bug.

Bisect log

pip install --upgrade jax==0.2.0 jaxlib==0.1.57+cuda111 -f https://storage.googleapis.com/jax-releases/jax_releases.html works
pip install --upgrade jax==0.2.4 jaxlib==0.1.57+cuda111 -f https://storage.googleapis.com/jax-releases/jax_releases.html FAILS
pip install --upgrade jax==0.2.2 jaxlib==0.1.57+cuda111 -f https://storage.googleapis.com/jax-releases/jax_releases.html FAILS
0.2.1 works

For now, pinning jax < 0.2.2 in requirements (down from <=0.2.5) might be sufficient!

Awesome, thanks! I'll update requirements.txt accordingly. The jax API has been changing a lot (they're still in beta as well) so we've been slow to adopt new versions but looks like we should maybe be even slower and more cautious.
Also FYI, the example input file for w7x likely won't converge to a very good solution, I'm still trying to figure out what some good settings are, and we're also in the midst of some major code changes that should improve speed and robustness and make the interface a bit nicer and easier to extend.Of course feel free to play around with it, but just be aware that its still very much in a pre-release state so things will likely change quite a bit.

Sure, like you said - just playing around with it for now :) Thanks for the heads-up, though! The DSHAPE one seemed to converge much more smoothly and quickly. The DESC paper is still on my to-read stack, but is it fair to say tokamak calculations will generally converge faster than stellarator ones, due to axisymmetry making things easier?


Don't change kakoune's directory, use absolute paths when needed

This is solution 2 from #406

Thanks for taking this on!
With this variant we should probably keep grep-jump working even if Kakoune's working directory is different from the project root - by making the paths relative to Kakoune's $PWD.
If we add a "root directory" option to grep-like buffers, then grep-jump could just use that instead (option 4).
Option 6 seems also nice although maybe a bit too magical.

If we add a "root directory" option to grep-like buffers, then grep-jump could just use that instead (option 4).

If we don't want to make paths relative to Kakoune's $PWD, but keep them relative to the project root, then we can probably implement this fairly easily: simply add a buffer-local alias that overrides grep-jump. The behavior is the same, but it prepends the project root before jumping to relative paths.

Switched from option 2 to option 5 (copy needed parts from Kakoune).

grep-jump in lsp-next-match should also be replaced by lsp-goto-jump, right?

Added a uniform way to handle all those grep/goto/diagnostics buffers.
I'm fairly happy with this, because it allows to treat *diagnostics* like a first-class *make* buffer: lsp-next-location *diagnostics* works, and also displays the error message in the prompt. We couldn't use make-next-error because that hardcoded *make* as buffer name.
cc @lenormf maybe you have feedback on the direction

Merging, thanks! I'm glad this is finally fixed.
Moving my additions of lsp-next-location to a follow-up.


Kubeless app

Thank you for wanting to submit a Pull Request to the Civo Kubernetes Marketplace repository!
If your pull request is to submit a new application to the marketplace, please answer the following questions:

[x] Have you followed the guidelines in our Contributing document in setting up the application?
[x] Have you checked that the application your submission is proposing to add does not yet exist on the Marketplace, or as a pull request to be processed?
[x] Have you tested your application on the Civo managed Kubernetes service? Please include a screenshot.
[x] Can you confirm that the software you are submitting licensed for use in a publicly-available Kubernetes marketplace? (Apache License 2.0)

Hi @gilsdav, thank you very much for the submission.
We'd be happy to merge this in and add it to our markplce, however there are a few bits we'd like changed first:

The description is currently too long (see the failing circleci build above). Could you possibly edit/trim it a little

You use the latest RELEASE in the install.sh - we prefer a fixed version, that is then updated later via PR. The reason is that then upstream releases don't completely change or break, causing us not to have a working version in our marketplace.

You use 0.1.0 in the manifest.yml, this should be the version of the software installed rather than the version of the marketplace app. So after doing 2. above please put the same version here.

Thanks very much for the submission, will be live on Civo in a few minutes.


Exploratory Data Analysis of Death Rates

Have you completed your first issue?

[X] I have completed my first issue

Guidelines

[X] I have read the guidelines
[X] I have the link to my latest merged PR

Latest Merged PR Link
https://github.com/UTSAVS26/PyVerse/pull/684#event-14763858737
Project Description
The main goal of the "Death Rate Analysis" project is to assess mortality statistics across various countries based on different causes of death. The purpose is to aid in improving health policies by analyzing gender-wise death rates and understanding their relationship with factors such as the Human Development Index (HDI) and Gross Domestic Product (GDP).
Full Name
Sharayu Anuse
Participant Role
GSSOC-Ext, Hacktoberfest

@UTSAVS26 I need help for the following issue.

@sharayuanuse i will correct the issue by today itself, you can try cloning in evening.


Template fix for Ubuntu

Hi,
This pull request is based on this other fork:
https://github.com/jamessthompson/puppet-unattended_upgrades

Hello, this PR has no explanation of what it changes, why it's changing it, or why it should be merged. It has two separate commits with identical messages.
I welcome contributions, but can't accept this.

Hi,
This pull request is a follow up of this. It add a basic support for Ubuntu since it use Unattended-Upgrade::Origins-Pattern for repos configuration.
I ve tested this on UBUNTU 14.04 LTS.
I hope you can reopen and merge.

Gladly, but it has merge conflicts :smile:


UrlEncode user input in client

This is linked to #29126, this disallow any ../ on a query from the client. This could lead to some really messed up queries.
Currently, a request with ../something like docker network rm ../foo/bar returns a 301 which is not really handled at all.
$ docker network rm ../foo/bar
# […]
resp &{301 Moved Permanently 301 HTTP/1.1 1 1 map[Content-Type:[text/plain; charset=utf-8] Location:[/toto] Date:[Mon, 05 Dec 2016 11:16:35 GMT] Content-Length:[0]] 0xc42000e500 0 [] false false map[] 0xc420424000 <nil>}
../foo/bar

We should discuss what to do about 301 http code too.
/cc @thaJeztah @dnephin @justincormack @cpuguy83
🐸
Signed-off-by: Vincent Demeester<EMAIL_ADDRESS>

Looks like build is failing;
13:47:19 # github.com/docker/docker/client
13:47:19 client/client_test.go:161: multiple-value c.getAPIPath() in single-value context
13:47:19 FAIL	github.com/docker/docker/client [build failed]

@thaJeztah oups :sweat:

I would prefer a different approach:
1 - the client should not try to do URL tweaking - it should take all user names "as is" and just URL escape them so that the HTTP layer doesn't see it as ".." at all.  It should see it as "%2E%2E" instead.
2 - the daemon should be the one to block the use of ".." in an incoming message since people can do just "curl" cmds to by-pass the docker cli.  This assumes that the use of ".." is seen by our code at all - which it may not be - the http layer may send back a 301.  In which case I think we should either do the redirect or show an error - instead of being silent like we are today.

I do agree with 1. :angel:

the daemon should be the one to block the use of ".." in an incoming message

yeah, I don't think the daemon will see those; /v1.25/networks/../containers/foo is the same as /v1.25/containers/foo. That's not a problem directly if people are calling the API directly, but allowing it from the CLI is odd, and something we (I think) should prevent.
Of course, unless an authorization plugin  is installed, having access to the API allows you to do any action, so we're not protecting against anything, but it would be a good thing to disallow it.

Updated the PR to encode the path instead of checking for ../ :angel: (the returned error is a little bit odd though >_<)
$ docker network rm ../foo/bar
Error response from daemon: page not found

That's odd, I would expect the same error as this:
docker network rm asdasd
Error response from daemon: network asdasd not found

if we're not seeing that then I suspect its because the escaping isn't being done right, on one or both sides because ../foo/bar should just be a name/string, not part of the path.

See: https://github.com/gorilla/mux/issues/132
and https://github.com/docker/docker/pull/16577
just remembered

# docker network rm ../volumes/toto
Error response from daemon: network ..%2Fvolumes%2Ftoto not found

@duglin just did the remove endpoint but we should do that for all :sweat:

yup - easy way to bump up your commit count  LOL

I'll do all at once :angel: so just 1 commit :stuck_out_tongue_closed_eyes:

sgtm

Hum CI failures are not the one I though… I'll have to look into it 😅

Removing this from the 1.13 milestone, as this is too risky to include now

Sounds good to me 😇

Hum, I'll close it for now as I don't think this is the right fix… I'll take another look.


New endpoint

Hi,
Please could you add this endpoint for Unical and Schuster stove :
https://unical.agua-iot.com/
Brand code 289982
Thanks :)

For test i manually modified file and it's ok 👍

Done! The vendor app is called APP-O BIOEN right?

Yes it’s correct 👍🏻


Mixed Content w/ HTTPS

Since one of the last updates to 3.x, OpenWebUI no longer loads. The DevTools console throws:
Mixed Content: The page at 'https://example.com/' was loaded over HTTPS, but requested an insecure resource 'http://example.com/api/v1/tools/'. This request has been blocked; the content must be served over HTTPS.

TypeError: Failed to fetch
    at window.fetch ...

The fetch function in question makes a request to an API endpoint that has a constant in it which is hardcoded as http:
https://github.com/open-webui/open-webui/blob/0917fa6f4ac015810211d85eb41da5d335762952/src/lib/constants.ts#L7
This causes issues, as the page is trying to load a http resource over a https page, essentially.
A potential solution would be getting the protocol to correctly prepend http or https, depending on the instance. Maybe location.protocol?

I just posted about the same issue (https://github.com/open-webui/open-webui/issues/3074). So, me too!


no machine can be scheduled, return TMP_NO_AVAILABLE_GPU

Short summary about the issue/question: In short: after update to 1.1 my remote experiment stopped working with subject error. I figured the source of it and propose some modifications, which will make debugging more clear.
Details
When NNI is run in remote mode, master machine generates from a hard-coded pattern a bash script which is meant to be uploaded to all workers and then run there as gpu_metrics_collector.sh. This script just updates current state of GPUs on workers. There are two weak points to me here:

If this script fails by any reason, master just doesn't get anything in response, and for some reason (!!) processes this as TMP_NO_AVAILABLE_GPU. Which of course is not the case. Correct handling of this scenario would be really great. I.e. "No response from workers, check that script output on worker machine".
gpu_metrics_collector.sh has a hardcoded call to /usr/bin/python3 where NNI expects its module to be installed. To me it was a reason of error, since default python3 which had NNI module was not installed/linked to this particular path. I would refrain from using hardcoded path to python and also provide some stderr feedback from gpu_metrics_collector.sh.

Thanks!

How to reproduce it: run remote experiment with two workers. workers machine shouldn't have default python with installed NNI under this path /usr/bin/python3 .
nni Environment:
nni version: 1.0 (Upgrade to 1.1 didn't solve the problem)
nni mode(local|pai|remote): remote
OS: Ubuntu 16.04.6 LTS
python version: 3.6.8
is conda or virtualenv used?: no
is running in docker?: yes
need to update document(yes/no):
Anything else we need to know:

Thanks for your feedback.
We have noticed the GPU resource detector is error-prone and the error message is vague.
We are planning to refactor the detecting and reporting mechanism, and to show error messages in web UI.
There is a hotfix PR #1707 for another issue, which may solve this problem (to some extent) as well.

Thanks for your feedback.
We have noticed the GPU resource detector is error-prone and the error message is vague.
We are planning to refactor the detecting and reporting mechanism, and to show error messages in web UI.
There is a hotfix PR #1707 for another issue, which may solve this problem (to some extent) as well.

the fix will go with the upcoming release, please stay tuned.

According to the code, I think this script uses the first python3 appears in PATH, not hard-coded path.

According to the code, I think this script uses the first python3 appears in PATH, not hard-coded directory.

can we close this issue? is the problem fixed upstream? @liuzhe-lz


Feature Request: Buy all Covenant sleeve upgrades before destroying BN

Current best way to buy Covenant upgrades is to --disable-auto-destroy-bn on BN10 and spend as much money as possible manually.
Automating this would improve Sleeve progression but would also dumb-down the average user.
Is an API available for this?

Afraid not. Can't be automated.
On Thu., May 12, 2022, 5:53 p.m. Evan G, @.***> wrote:

Current best way to buy Covenant upgrades is to --disable-auto-destroy-bn
on BN10 and spend as much money as possible manually.
Automating this would improve Sleeve progression but would also dumb-down
the average user.
Is an API available for this?
—
Reply to this email directly, view it on GitHub
https://github.com/alainbryden/bitburner-scripts/issues/130, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AARN33LUYUMV25NWRUR6QPTVJVVTXANCNFSM5VZONWNA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


domain-filter with multiple domains

Currently it's not possible to specify multiple domains to --domain-filter. I believe it's useful to have this option if you have multiple ingress controller or services exposing LoadBalancer to the public and some DNS records of these services  should managed independently.

May be obvious, but our current work around is to simply run it twice, each with their own distinct domain filter which prevents them from clobbering one another.

@totallyunknown I agree, the --domain-filter should be more flexible. Willing to describe a proposal and/or do a PR?

Yep, could be possible to run multiple pods with different --domain-filter. I can create a PR to allow ´--domain-filtermultiple times like the--sourceflag. Another option could be a comma separated list like--domain-filter="foobar.com, blabla.org, example.net`. I tend to the last option, because this is better to inject via environmental variables.


Need to use Helm in Airgapped environment

I have a K8S cluster setup in an airgapped environment which is making difficult to download the public charts and associated docker images.
We also use harbor and by using helm-mirror I can do download the public charts to harbor. Also I can download the docker images, tag it and push it to our repository.
The problem is needing to add the Harbor credentials to these charts and it is extremely time consuming to complete all these steps which I have mentioned (even with some scripts in place 😢 )
I am looking here for guidance if there are any easier ways to use public helm charts to be used in airgapped environments.

I think this is a use case https://github.com/helm/helm/issues/7754 is attempting to solve. Can you please weigh in on that ticket? Thanks!


linePlusBarChart tooltip shows 12/31 always

using v1.8.4
http://plnkr.co/edit/vOZ9Tx?p=preview
and also chart.lines.interactive(false) does not turn off line tooltips

sorry opened in wrong repo


(maint) this fixes a typo in the setup for installing released PDB

Refer to this link for build results (access rights to CI server needed):
http://jenkins-enterprise.delivery.puppetlabs.net/job/enterprise_puppetdb_init-multijob_githubpr-stable/125/
Test PASSed.


replicate train requires inputs to be zip files

Training Llama 13B fails with the following error:
replicate train a16z-infra/llama-2-13b-chat<EMAIL_ADDRESS>--destination=andreasjansson/abc-llama

Error: failed to parse inputs: failed to upload file: file is not a zip file
Usage:
  replicate train <owner/model[:version]> [input=value] ... [flags]

Flags:
  -d, --destination string   Destination model for training
  -h, --help                 help for train
      --json                 Emit JSON
  -s, --separator string     Separator between input key and value (default "=")
      --web                  View on web

@andreasjansson This is a limitation of our experimental DreamBooth API for file uploads, which supports only Zip files. To work around this, you can pass a URL to a different file type.

@andreasjansson Follow up on this — I was wrong! You can now upload any supported files as of v0.2.2.
...though until this PR is merged, .jsonl isn't a supported file format.


Opening map slows game continually to the point of client freezing

Describe the bug
Opening the map becomes slower and slower every time I open it. It eventually gets so laggy to the point where the game freezes.
To Reproduce
Steps to reproduce the behavior:

Open the map
Close the map
Repeat as process gets slower and eventually entire game freezes

Expected behavior
Opening the map should be as fast and smooth as it is the first time
Environment (please complete the following information):

OS: macOS
RuneLite version: 1.4.25
Launcher version: 1.6.1

wasn't able to reproduce

Can we get someone to reproduce on macOS? on 1.6.1?

Can't reproduce on Mac

Ok thanks - I'll see if I can track down how it's happening. Maybe a memory leak that only manifests in opening the map after extended play?

I have the same problem, not to the point of freezing, but it definitely halves my performance when I have the map open.

I've tried on 2 windows devices, haven't been able to reproduce.

@BigBopi What environment are you in?


Robert/fix timing for mulitpurpose sample endpoint

The suggestSample endpoint (which allows an optional search string as a query parameter) is
multipurpose -- it does both text suggestions and will also return all (or a subset) or completions
for a particular column. This change updates the SpandexServlet to log timings for each of those
use-cases separately.
Additionally, this includes a few outstanding improvements in the log analysis script.

This build looks good.


[cxx-interop] Propagate CONFORMS_TO attribute to derived classes

If struct Base is a public base class of struct Derived, and Base is annotated with __attribute__((swift_attr("conforms_to:MyModule.MyProto"))), Derived will now also get a conformance to MyProto.
rdar://113971944

@swift-ci please smoke test


Unable to find Pytorch for CUDA 11.01

📚 Installation
Error:
**python -c "import torch; print(torch.cuda.is_available())"

False**

Environment

OS: Windows 10
Python version: 3.7.1
PyTorch version: 1.5.0 (CPU toolkit)
CUDA/cuDNN version: 11.01
GCC version: 7.4.0
How did you try to install PyTorch Geometric and its extensions (pip, source):
Any other relevant information:

Checklist

[*] I followed the installation guide.
[*] I cannot find my error message in the FAQ.
[ ] I set up CUDA correctly and can compile CUDA code via nvcc.
[ ] I have cloned the repository and tried a manual installation from source.
[ ] I do have multiple CUDA versions on my machine.
[ ] I checked if the official extension example runs on my machine.
[ ] The offical extension example runs on my machine.

Additional context

You need to install a PyTorch version with CUDA Toolkit 10.1.

@rusty1s I mistakenly installed Pytorch version with CUDA Toolkit 10.2, will this work? Or should I uninstall Pytorch and then install 10.1?
Also, I have a windows machine as mentioned do I still need to follow these commands provided in this link.
I can install on Ubuntu 20.04, virtual machine as well, the only issue is CUDA has issues on running on a Virtual Machine.

@rusty1s can you please update me on this?

Both Windows and Ubuntu installation should be fine if you have a PyTorch version with CUDA installed. We support CUDA 9.2, 10.1 and 10.2. You can follow the installation instructions in the document you linked. Please keep me updated on the progress you make and if there are any errors.

I have Pytorch and CUDA installed.
PS C:\Users\Jayesh> python -c "import torch; print(torch.__version__)"
1.5.0
PS C:\Users\Jayesh>  python -c "import torch; print(torch.version.cuda)"
10.2

In the place mentioned where we need to specify our CUDA version between the following options, how do we choose one?
where ${CUDA} and ${TORCH} should be replaced by your specific CUDA version (**cpu, cu92, cu101, cu102**) and PyTorch version (1.4.0, 1.5.0), respectively.

Since you have CUDA 10.2 installed, you use cu102 with PyTorch 1.5.0.

Okay, I tried with this, I am getting this message:
PS C:\Users\Jayesh> pip install torch-scatter==latest+${cu102} -f https://pytorch-geometric.com/whl/torch-${1.5.0}.html
Looking in links: https://pytorch-geometric.com/whl/torch-.html
ERROR: Could not find a version that satisfies the requirement torch-scatter==latest+ (from versions: 0.3.0, 1.0.2, 1.0.3, 1.0
.4, 1.1.0, 1.1.1, 1.1.2, 1.2.0, 1.3.0, 1.3.1, 1.3.2, 1.4.0, 2.0.2, 2.0.3, 2.0.4, 2.0.5)
ERROR: No matching distribution found for torch-scatter==latest+

Try
pip install torch-scatter==latest+cu102 -f https://pytorch-geometric.com/whl/torch-1.5.0.html

Yes, I am sorry. I just realized my mistake. Thank You very much.


Select optgroup to select all children

It would be nice to have a property on the optgroup list item to allow it to be "selectable". On selection, all of the children of the optgroup would be selected. For example, in CodeExample4 if you were to select the continent "North America", this would select all countries in North America (United States, Canada, etc.). I've attempted to implement this, but it has proven difficult as optgroup menuItems and nested div menuItems look to all be output as a single array. If anybody has an example of this implementation I'd love to see it.

@koltclassic  Did you find a solution?

@the-stranger I have not found a solution to this, yet.

Please check release 1.9.0


Cosmetic modification

*Several cosmetic modifications

dephasing code with retropie-es code. merge would be more complicated.
eventuallly, push it on retropie-es.


touches property does not fire update event in hasMany relationship

Laravel Version: 6.18.31
PHP Version: 7.4.8
Database Driver & Version: Mysql 5.7

Description:
It seems because of batch update of relations updated_at in touchOwners method ($this->$relation()->touch();) , updated event does not fire in parent model when using hasMany relationship.
public function touchOwners()
{
     foreach ($this->touches as $relation) {
         $this->$relation()->touch();

         if ($this->$relation instanceof self) {
             $this->$relation->fireModelEvent('saved', false);

             $this->$relation->touchOwners();
         } elseif ($this->$relation instanceof Collection) {
             $this->$relation->each->touchOwners();
         }
     }
}

Steps To Reproduce:
1- create a product and brand model which a brand has many products
2- add protected $touches = ['products']; in brand model
3- add observer for products.
4- product updated event will not fire when updating brand

Because it doesn't retrieve all the models. It's a batch operation.


Mccalluc/more negative tests

(I want to sort out the conflicts locally... requires some judgement.)


Closes #1177 #1167 pkgdown updates

Close #<insert_issue_number>
Thank you for your Pull Request! We have developed this task checklist from the Development Process Guide to help with the final steps of the process. Completing the below tasks helps to ensure our reviewers can maximize their time on your code as well as making sure the admiral codebase remains robust and consistent.
Please check off each taskbox as an acknowledgment that you completed the task or check off that it is not relevant to your Pull Request. This checklist is part of the Github Action workflows and the Pull Request will not be merged into the devel branch until you have checked off each task.

[x] Code is formatted according to the tidyverse style guide
[x] Updated relevant unit tests or have written new unit tests - See Unit Test Guide
[x] If you removed/replaced any function and/or function parameters, did you fully follow the deprecation guidance?
[x] Update to all relevant roxygen headers and examples
[x] Run devtools::document() so all .Rd files in the man folder and the NAMESPACE file in the project root are updated appropriately
[x] Address any updates needed for vignettes and/or templates
[x] Update NEWS.md if the changes pertain to a user-facing function (i.e. it has an @export tag) or documentation aimed at users (rather than developers)
[x] Build admiral site pkgdown::build_site() and check that all affected examples are displayed correctly and that all new functions occur on the "Reference" page.
[x] Address or fix all lintr warnings and errors - lintr::lint_package()
[x] Run R CMD check locally and address all errors and warnings - devtools::check()
[x] Link the issue so that it closes after successful merging.
[x] Address all merge conflicts and resolve appropriately
[x] Pat yourself on the back for a job well done! Much love to your accomplishment!

Hi Ben, sorry that I was dragged by a study deliverable yesterday so did not get to work on issue #1167 yesterday. Your updates on Get Started Page looks great and much easier for the new users to follow!  I also added some minor comments.

Hi Ben, sorry that I was dragged by a study deliverable yesterday so did not get to work on issue #1167 yesterday. Your updates on Get Started Page looks great and much easier for the new users to follow! I also added some minor comments.

No problem Rena!  I don't see your comments in this PR.  Were they in the issue or the PR

@bms63 You should see my comments now.

@bms63 You should see my comments now.

These are great tyty!

Wooo....kept adding stuff in...had to stop myself.  @yangc71  time to reviwe?

Thanks @bms63, looks good to me


Kiwix.Wikivoyage version <IP_ADDRESS>

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Wikivoyage by Kiwix is part of the Kiwix family of apps. I have already submitted "Kiwix JS" and "WikiMed by Kiwix" to the WinGet repository. The GitHube repo and branch for this app is here: https://github.com/kiwix/kiwix-js-windows/tree/Kiwix-JS-Wikivoyage. I am the lead developer of this UWP app.


[security ] Understanding and defense from CSRF attacks

**Topic Proposed **
Cross Site Request Forgery
Use of  CSRF tokens in sites running on Flask python framework to prevent form  submissions from attackers
**Key Takeaways **
Understanding what is a CSRF attack and how it is executed
Understanding and how to generate strong and genuine CSRF tokens for use in defense from these attacks.
Reference
N/A

Thank you for your response and we thank you for submitting your topic. After some careful consideration it struck us that this topic may be a bit over saturated throughout other blog sites and official documentations as previously mentioned on your topic form by our content moderator.
We typically refrain from publishing content that is covered widely on the net or other blogs. As we're more interested in original, practitioner-focused content that takes a deeper dive into programming-centric concepts.


Handshake failure if client doesn't include TLS Extensions for ECC in client hello

Hello,
starting from version 1.0.2k, handshake fails trying a connection between a server that allows ECC cipher (e.g.  ECDHE-RSA-AES128-GCM-SHA256) and a client that propose ECC but doesn't include TLS Extensions for ECC in client hello. There is the same result with latest 1.1.1
According to RFC 4492(https://tools.ietf.org/html/rfc4492) it seems that exstensions are not mandatory, if client doesn't propose them the server is free to choose any one of  the elliptic curves or point formats listed in Section 5 , instead server returns no shared cipher error message and exits
From RFC:

The client MUST NOT include these extensions in the ClientHello
message if it does not propose any ECC cipher suites.  A client that
proposes ECC cipher suites may choose not to include these
extensions.  In this case, the server is free to choose any one of
the elliptic curves or point formats listed in Section 5.  That
section also describes the structure and processing of these
extensions in greater detail.

The same configuration worked well in previous version (1.0.2j), it seems that this different behavior has been introduced by the commit 292bb56846ad78ac7b68a00c92153c0c9471665b  that changed   the client hello check (file  ssl/s3_srvr.c) in particular lines 1468-1470
how to reproduce:
use openssl version >=1.0.2k
server side configuration: openssl s_server -accept  -CAfile  -cert  -key  -cipher ECDHE-RSA-AES128-GCM-SHA256

I can confirm this behaviour for 1.1.1 and 1.1.0, but not for 1.0.2. In my tests it seems to work just fine in 1.0.2.
The removal from 1.1.1/1.1.0 appears to have been a deliberate decision. See #1597. It seems the RFC allows us free to make our own choice on how to handle this, and from 1.1.0d onwards we have chosen to take a more conservative approach and not negotiate EC based ciphersuites if the EC extensions are missing. Arguably that change shouldn't have happened in a stable branch, and should have waited for 1.1.1 - but it seems academic now.

Dunno if it's related, but I feel it's worth mentioning that RFC4492 is recently obsoleted by RFC8422. So if quoting standards, it's worth pointing at the current one ;)

Hello OpenSSL community,
Wanted to report the new issue but it looks it is related to this conversation.
When ClientHello message proposes ECC cipher suite (TLS_ECDHE_ECDSA_WITH_AES_128_CCM_8 in my case) without two extensions:  Supported Elliptic Curves (10) and Supported Point Formats (11), server (that has TLS_ECDHE_ECDSA_WITH_AES_128_CCM_8 cipher suite in its certificate) does not fall back to choose any one of elliptic curves or point formats and fails to negotiate the connection with the client.
The server results in a fatal handshake failure alert due to the lack of common cipher suites.
According to RFC 8422 standard “A client that proposes ECC cipher suites may choose not to include these extensions. In this case, the server is free to choose any one of the elliptic curves or point formats listed in Section 5.”
The version that I am using is 1.1.0j. It seems like the version 1.1.1 has the same issue.
Please correct me if I am wrong here.
Also was that feature intentionally removed starting from version 1.0.2k and will not be added back again?
Thanks and best regards.

@mattcaswell any update on @sevabo question? I am running into the same issue. In order to use TLS13 we upgraded to openssl 1.1.1 from 1.0.2g and we are running into this issue. Legacy apps that are used to not sending the extension now fail.

It looks like #8115 is an attempt to resolve this issue, but the approach described in #2237 might be better overall.
I do not think there is a definitive answer to @sevabo's question ("will we definitively not add back default-group support?") at this time, though one should generally assume continuation of the status quo.

Closing as duplicate of #2237


BUG: special: fix hyp2f1 behavior in certain circumstances

Hyp2f1 previously failed when c ==b and was a negative integer greater than -100. This resolves the behavior by implementing a solution given on stackexchange to fix the behavior so that it aligns with what people expect.
I broke out the functions for maximum clarity. Maybe the compiler can optimize them away?
Closes #7961.

Also, I'm more than happy to accept feedback to improve performance.

Unmarking as WIP because we get an XPASS from the test.

@pv What should the function name be?

I cannot find a reference to that formula other than the stackexchange post.

I have (hopefully!) implemented the collector pattern correctly.

It is 15.4.2 in Abramowitz & Stegun.

Now that you are editing it, please extend the mpmath comparison test to cover very large, small, and also negative values of a and x.

I see that @pv already had some of the same comments as me, apologies for any repeats.

Before I proceed, I would like to note that this comment by @pv is correct:

This sort of statements should be accompanied by reasoning why it is so, otherwise it's very easy to make mistakes.

Unfortunately, because I am not very strong in this area, it's going to take me some time to work out the behavior of the function to make sure that I'm not missing anything important. Of course I would not mind someone else working this out if they have time.

it's going to take me some time to work out the behavior of the function to make sure that I'm not missing anything important

Probably the safe thing to do for now is to always use the series and follow @pv's advice to track the error being accumulated and then bail out with nan if it is too large. This corrects some of the problematic cases and doesn't return wrong/uncertain results in the other ones.
Further improvements can always be made later. :-)

... and an estimate for the relative error in the sum is
1e-16 * (1 + largest_term_fabs/fabs(sum)). We typically start
returning nan if it gets worse than 1e-7 (ie sqrt eps).

@pv The error is checked at the end to avoid a performance cost (checking again and again inside of the loop).

The point is floating point loss of precision when summing alternating
series. I've explained how to do it in a way that's certainly correct,
and don't really want to start reviewing alternatives --- it is doubtful
the performance differences matter much in practice, as long as you get
the big-O right.

Exercise:
import mpmath, numpy as np
from scipy.special import poch, gamma

a = -12.5
b = -60
x = 0.9

k = np.arange(0, -b + 1)
terms = poch(a, k) * x**k / gamma(1 + k)
s = terms.sum()
maxterm = abs(terms).max()

mpmath.mp.dps = 100
s_mp = float(mpmath.hyp2f1(a, b, b, x))

print("rel. error:", abs(s - s_mp)/abs(s_mp))
print("rel. error estimate:", np.finfo(float).eps * (1 + maxterm / s))
print("last term error est:", np.finfo(float).eps * (1 + abs(terms[-1])
/ s))

a = -12.5

I misread ax  < 1.0 and bracketed the wrong variable (a rather than x). Silly me.

Reopened to get @person142's build ahead of me.

@pv Concerns have been addressed and I think the CI will pass. Would you like me to clean up the commits?

@pv Does this still need more work?

The fmax.h code is needed now in ~three different places in scipy.
Maybe should be combined to a single file and just incldue paths adjusted.

Are you thinking ~three places in just special or in multiple submodules?

$ find scipy -name 'fmax.h'
scipy/spatial/ckdtree/src/fmax.h
scipy/optimize/_trlib/trlib/fmax.h

And now that I look there's code for setting the fpu mode in _lib, which is another C99 feature that was (in a roundabout way) recreated in special for wrightomega. So gathering C99 stuff in _lib sounds like a great idea.

fmax.h duplication addressed in #8003. You can merge that one first and then I can rebase and fix the duplication in this PR or you can merge this one first and I can fix the duplication in that PR.

Unfortunately MSVC 2008 isn't happy. The linker error points to an unresolved external symbol, but fmax should be static, not extern.

Wow, that was just nasty. I picked up another _c99compat.h header.

That was more than I bargained for.
@pv Please merge #8006 first and then I will rebase for a clean patch.

thanks, merged


Break time does not stop, no notification, goes to -XX seconds. Android 7.1.1

When the break is finished the clock goes to -XX seconds until -59 and jumps to 00 and -01 to -59 and so on. No vibration, no "break is over" notification.
On an Axon 7 using Android 7.1.1

Issue seems to be fixed on ZTEs update to B02

Sorry. Issue not fixed in ZTE's update. Still persists. I'm pretty new to github and don't know how to reopen it. I'll try to see whether there are some situation where it doesn't come up.


Enha: additional seeds and crops

What would you like to see changed in one sentence?
all seeds and crops needed for first study 3a (v1, v2, v3)
Elaborate on your suggestion above as needed.
The game currently supports 2 types of seeds and crops:

tomatos,
corn

We also need:

beetroot,
carrot,
eggplant,
pumpkin,
parsnip

Sprites for all the above types of crops are available in images/objects/items. It's just a matter of extending all enums and making sure they are properly displayed in the Inventory menu.
Which aspect(s) of the project is this enhancement affecting?
game-ui, game-gameplay
Code of Conduct

[X] I agree to follow this project's Code of Conduct.

@sloukit what about those existing crops:

WOOD,
APPLE,
BLACKBERRY,
BLUEBERRY,
RASPBERRY,
ORANGE,
PEACH,
PEAR

They are not mentioned in excel sheet - do we want to keep them or hide in study 3a?

@sloukit what about those existing crops:

WOOD,
APPLE,
BLACKBERRY,
BLUEBERRY,
RASPBERRY,
ORANGE,
PEACH,
PEAR

They are not mentioned in excel sheet - do we want to keep them or hide in study 3a?

You can hide the crops that are not in the excel sheet for study 3a!

@sloukit
crops that need to be added need to have their own growth speed:
{
    "corn": 1,
    "tomato": 0.7,

    "beetroot": 0.7,
    "carrot": 0.7,
    "eggplant": 0.7,
    "pumpkin": 0.7,
    "parsnip": 0.7,
}

and value at the market:
20,  # CORN
40,  # TOMATO

20,  # BEETROOT
20,  # CARROT
20,  # EGGPLANT
20,  # PUMPKIN
20,  # PARSNIP

4,  # CORN_SEED
5,  # TOMATO_SEED

6,  # BEETROOT_SEED
6,  # CARROT_SEED
6,  # EGGPLANT_SEED
6,  # PUMPKIN_SEED
6,  # PARSNIP_SEED

please review the values.

@sloukit what about those existing crops:

WOOD,
APPLE,
BLACKBERRY,
BLUEBERRY,
RASPBERRY,
ORANGE,
PEACH,
PEAR

They are not mentioned in excel sheet - do we want to keep them or hide in study 3a?

You can hide the crops that are not in the excel sheet for study 3a!

After playing the game for a bit longer, I realized that those crops actually play some role. They are collectibles that can be sold on the market, and they can be gathered in the forest. Even the tutorial is encouraging to do so. So now, I'd rather suggest keeping them always available. There are no seeds for those, so the player can't plant them and it's not interfering with the idea of making one new crop available with each game level.

There is one last concern related to crops and the market. In the market we can see the number of items we possess, we can buy seeds and sell crops. However player can't see the price and amount of money he possesses although it is calculated. At some point, we can run out of money and won't be able to buy goods, but without any information about why. Is this supposed to be like this or should I create a bug for it?

@sloukit crops that need to be added need to have their own growth speed:
{
    "corn": 1,
    "tomato": 0.7,

    "beetroot": 0.7,
    "carrot": 0.7,
    "eggplant": 0.7,
    "pumpkin": 0.7,
    "parsnip": 0.7,
}

and value at the market:
20,  # CORN
40,  # TOMATO

20,  # BEETROOT
20,  # CARROT
20,  # EGGPLANT
20,  # PUMPKIN
20,  # PARSNIP

4,  # CORN_SEED
5,  # TOMATO_SEED

6,  # BEETROOT_SEED
6,  # CARROT_SEED
6,  # EGGPLANT_SEED
6,  # PUMPKIN_SEED
6,  # PARSNIP_SEED

please review the values.

That seems perfectly fine to me, thank you!

@sloukit what about those existing crops:

WOOD,
APPLE,
BLACKBERRY,
BLUEBERRY,
RASPBERRY,
ORANGE,
PEACH,
PEAR

They are not mentioned in excel sheet - do we want to keep them or hide in study 3a?

You can hide the crops that are not in the excel sheet for study 3a!

After playing the game for a bit longer, I realized that those crops actually play some role. They are collectibles that can be sold on the market, and they can be gathered in the forest. Even the tutorial is encouraging to do so. So now, I'd rather suggest keeping them always available. There are no seeds for those, so the player can't plant them and it's not interfering with the idea of making one new crop available with each game level.

Yes, you're right. I'd keep those fruits in the game too, at all times. You could add them to the excel sheet, just so we can keep track of what is included.

There is one last concern related to crops and the market. In the market we can see the number of items we possess, we can buy seeds and sell crops. However player can't see the price and amount of money he possesses although it is calculated. At some point, we can run out of money and won't be able to buy goods, but without any information about why. Is this supposed to be like this or should I create a bug for it?

Thanks for pointing this out, I hadn't noticed - at one point the amount of money was definitely displayed in the market (see screenshot). So it must have been removed at some point. I'll try to find out where this happened and let you know.

@sloukit what about those existing crops:

WOOD,
APPLE,
BLACKBERRY,
BLUEBERRY,
RASPBERRY,
ORANGE,
PEACH,
PEAR

They are not mentioned in excel sheet - do we want to keep them or hide in study 3a?

You can hide the crops that are not in the excel sheet for study 3a!

After playing the game for a bit longer, I realized that those crops actually play some role. They are collectibles that can be sold on the market, and they can be gathered in the forest. Even the tutorial is encouraging to do so. So now, I'd rather suggest keeping them always available. There are no seeds for those, so the player can't plant them and it's not interfering with the idea of making one new crop available with each game level.

Yes, you're right. I'd keep those fruits in the game too, at all times. You could add them to the excel sheet, just so we can keep track of what is included.

ok, but please don't add them to the Excel, I've already implemented them as default list of crops

There is one last concern related to crops and the market. In the market we can see the number of items we possess, we can buy seeds and sell crops. However player can't see the price and amount of money he possesses although it is calculated. At some point, we can run out of money and won't be able to buy goods, but without any information about why. Is this supposed to be like this or should I create a bug for it?

Thanks for pointing this out, I hadn't noticed - at one point the amount of money was definitely displayed in the market (see screenshot). So it must have been removed at some point. I'll try to find out where this happened and let you know.

I see, so now, the list of crops is so long that the amount of money is below the bottom of the screen. I'll fix it. Unfortunately, the Inventory menu needs more refactoring, there is no scrolling implemented to handle more crops than can fit on the screen.

There is one last concern related to crops and the market. In the market we can see the number of items we possess, we can buy seeds and sell crops. However player can't see the price and amount of money he possesses although it is calculated. At some point, we can run out of money and won't be able to buy goods, but without any information about why. Is this supposed to be like this or should I create a bug for it?

Thanks for pointing this out, I hadn't noticed - at one point the amount of money was definitely displayed in the market (see screenshot). So it must have been removed at some point. I'll try to find out where this happened and let you know.

I see, so now, the list of crops is so long that the amount of money is below the bottom of the screen. I'll fix it. Unfortunately, the Inventory menu needs more refactoring, there is no scrolling implemented to handle more crops than can fit on the screen.

Okay, I also just noticed that the amount of money is still there, but just too far down. We already have a scrolling system implemented in this PR. It might save some time to just copy it.

Okay, I also just noticed that the amount of money is still there, but just too far down. We already have a scrolling system implemented in this PR. It might save some time to just copy it.

Yes, I've seen it in other menus, but it seams that each menu is implemented a bit differently. Still, it's helpful to reuse this code 💪 Thanks for the tip!

this issue has been fixed in PR #249.
Crops that are visible depend on the game level and round (level).
In the shop menu (press y in debug mode), there are some additional features added:

list scrolling
menu title and column headers
additional column with crop value
crop icon on the left of crop name (as an alternative version, it can be moved inside the white bar, but then it needs to be scaled down by 2)
action label changed from 'buy' / 'sell' to 'buy one' / 'sell one'
footer label 'your money' added

after a review of the PR, this issue can be closed

PR #249 has been merged, it's fixed now


⚠ Mobitopup has degraded performance

In ab428b2, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 3460 ms

Resolved: Mobitopup performance has improved in 4dc65a0.


python311Packages.dissect: 3.11 -> 3.12

Description of changes

Things done

Built on platform(s)

[ ] 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

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

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

[ ] Fits CONTRIBUTING.md.

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

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

52 packages built:

python311Packages.dissect-util
python311Packages.dissect-util.dist
python312Packages.dissect-btrfs
python312Packages.dissect-btrfs.dist
python312Packages.dissect-cim
python312Packages.dissect-cim.dist
python312Packages.dissect-clfs
python312Packages.dissect-clfs.dist
python312Packages.dissect-cstruct
python312Packages.dissect-cstruct.dist
python312Packages.dissect-esedb
python312Packages.dissect-esedb.dist
python312Packages.dissect-etl
python312Packages.dissect-etl.dist
python312Packages.dissect-eventlog
python312Packages.dissect-eventlog.dist
python312Packages.dissect-evidence
python312Packages.dissect-evidence.dist
python312Packages.dissect-executable
python312Packages.dissect-executable.dist
python312Packages.dissect-extfs
python312Packages.dissect-extfs.dist
python312Packages.dissect-fat
python312Packages.dissect-fat.dist
python312Packages.dissect-ffs
python312Packages.dissect-ffs.dist
python312Packages.dissect-hypervisor
python312Packages.dissect-hypervisor.dist
python312Packages.dissect-jffs
python312Packages.dissect-jffs.dist
python312Packages.dissect-ntfs
python312Packages.dissect-ntfs.dist
python312Packages.dissect-ole
python312Packages.dissect-ole.dist
python312Packages.dissect-regf
python312Packages.dissect-regf.dist
python312Packages.dissect-shellitem
python312Packages.dissect-shellitem.dist
python312Packages.dissect-sql
python312Packages.dissect-sql.dist
python312Packages.dissect-squashfs
python312Packages.dissect-squashfs.dist
python312Packages.dissect-thumbcache
python312Packages.dissect-thumbcache.dist
python312Packages.dissect-util
python312Packages.dissect-util.dist
python312Packages.dissect-vmfs
python312Packages.dissect-vmfs.dist
python312Packages.dissect-volume
python312Packages.dissect-volume.dist
python312Packages.dissect-xfs
python312Packages.dissect-xfs.dist


Add Configure SLO section to Create Database dialog for Azure servers

Adds a new section for selecting the DB's Edition, Backup Storage Redundancy level, Service Level Objective, and Max Size.
Closes https://github.com/microsoft/sqltoolsservice/pull/2094

I bumped the STS version to <IP_ADDRESS> to pick up the SLO changes. Here are the other commits rolled into that version.

Looks like it's also an issue an SSMS. According to the azure database prototype, scripting isn't supported for data warehouse. I'll fix it separately, since it affects scripting in general


AlertRibbon: Warning for using CMS markup type in content

Bug already reported ?

[X] I confirm that I have checked if the bug already has been reported

For which framework/library you are reporting the bug
React
Component name
AlertRibbon
Description
https://github.com/sebgroup/green/blob/main/libs/react/src/lib/alert-ribbon/alert-ribbon.tsx
Having difficulties when using CMS markup translation fields in the AlertRibbon component, because alert-ribbon__content has <p> for children elements. The markup field from CMS can have either <p> or <div> elements and thus throwing this similar warning for both of them:

This happens because paragraph should not nest <p> or <div> elements.
Steps To Reproduce
No response
Current Behaviour
No response
Expected Behaviour
No response

I would argue that the warning is actually correct in this case, and that the use of divs or paragraphs or other markup, except perhaps simple formatting like <strong> and <em>, is not supported in AlertRibbons.
Basically, there should never be more than a single paragraph in an AlertRibbon, so if the content is editable in a CMS you should use a limited content field. Something that is limited to around 75 characters and perhaps only accepts plain text. It's inadvisable to use WYSIWYG content in an AlertRibbon.
If the CMS does not support this, you should sanitize the content before handing it off to the AlertRibbon.


Add Risk level 0

Add color to risk level. Updated risk level design & wording in
profile. With @marimuraki :D

@AmirHadjihabib Screenshots?


FEATURE - Add ui5 library reference writer module

As a developer using SAP Fiori tools, I want to add a reference to library present in my workspace, so that I am able to use reuse components in my Fiori elements or Freestyle applications
Technical Design
New module :

@sap-ux/ui5-library-reference-writer

Acceptance Criteria
Given I have a Fiori project and a library/component
when I invoke this writer with the correct data
then the library/component will be added to my Fiori project
Notes
Tasks

[ ]
[ ] Unit Tests added
[ ] Test Cases defined

Closing as @sap-ux/ui5-library-reference-writer is published


Question about Unlabeled Data

I am using your example code from the following library on a dataset created for one of our applications:
https://www.tensorflow.org/neural_structured_learning/tutorials/graph_keras_lstm_imdb
Can you please clarify how and where in the code the data has been split between the labeled and unlabeled sets? I see the concept of supervision ratio and see that you took 90% of the original train_dataset as validation_dataset and the remaining 10% as train_dataset. Do you mean the validation_dataset was used as the unlabeled dataset? But that's not what it looks like based on the documentation here:
https://www.tensorflow.org/neural_structured_learning/api_docs/python/nsl/keras/GraphRegularization#fit

Thanks for the question, @bibudhlahiri !
Your understanding is correct that the validation_dataset is used as the unlabeled set.
But before training, the unlabeled set is merged with the labeled set using nsl.tools.pack_nbrs to create "neighbor-augmented" training examples. That is, in addition to each example's own feature (word), they will also contain features from its neighbors (NL_nbr_0_words, NL_nbr_1_words) and the weight of its neighbors (NL_nbr_0_weight, NL_nbr_1_weight). In the tutorial, the pack_nbrs step is happened before the train-validation split, so effectively we allow both labeled and unlabeled examples to be neighbors of (labeled) training examples.
When calling graph_reg_model.fit, the train_dataset already contains neighbor-augmented training examples, so we do not have to specify the unlabeled set again. The argument validation_data=validation_dataset is for monitoring validation accuracy, just like in ordinary model training.

Closing this issue due to inactivity. Please feel free to reopen if you have further questions.


fix typos

Thanks for this! Proofreading text that you've recently written doesn't work that well 😅
Also, congrats for being the first community member to have a pull request approved! You will be remembered by both me & git log.


2D pose estimation network

Hi,
Good work!
May I know the details of 2D pose estimation network in your paper? similar to 'Simple baselines for Human Pose Estimation' or modified Resnet50?

Hi, thanks for your interest in our work.
We use the 2D pose network presented in "Integral Human Pose Regression, ECCV 2018" (Resnet50 with deconv head). I think it is quite similar to the one in "Simple baselines for Human Pose Estimation".
Best,
Long

Thanks for your quick response!


Remove the 'version' from the client

Fixes #208
This PR aims to remove the 'version' parameter from the client since there is only one API version available.

LGTM! By the way, I see you wrote "Related to #208" in the description. If the goal was for the issue to be closed when the PR is merged, you should instead write "Fixes #208". See https://docs.github.com/en/enterprise/2.16/user/github/managing-your-work-on-github/closing-issues-using-keywords

Yes indeed, I'll do that now!


How to set up cors-anywhere with my own domain

i'd like to set up my own domain, like:
cors-anywhere.MYDOMAIN.com.br, how could I do it without heroku?
Im setting up this API on my digital ocean server.

I already solved thanks


Fix GitHub typo Git'h'ub to Git'H'ub.

I signed it.

It's here, also. Throughout the different READMEs there are "github", "Github" etc might be worth making them all "GitHub" in the one pr at some point

OK, I'm try it.

I have completed fix it.
Changes are document only, source code is not done because it may be destructive(e.g. /packages/angular-cli/commands/github-pages-deploy.ts)

Ignoring the CLA bot because the original author signed it (see comments above), and I'm a Google employee.

LGTM, thanks!


There's no printable version of the resulting form

Could we create a single-page version of the form, that we could then style for print - so we can explore how feasible it might be to auto-generate a printable / PDF version of a form?

Closing this issue for now. This is something that should be on the trello backlog (ICEBOX) to consider later in public beta


Image uploader picker component for avatar

The ImagePicker is usable however it still requires a server side endpoint to be developed for social media sources, for now it is mocked so you can interact with it.

Is this ready to be reviewed and merged? The backend has been merged already.

@asood123 Yes it is ready to be reviewed and merged


Gallery App has app icon

[x] iOS
[x] Android
[x] multi-res

This is done on both iOS and Android. It's actually the only app we have which has the same icon on both when built from source. :)


is it 0.13 compatible?

so i tried to compile my project and this happened (i did not forgot to add things to build system).
/home/tymek/.cache/zig/p/12208fab56fa3082d207beb09b1d5134a447ce006e78026f6951d89dd8c1279bded5/src/httpz.zig:233:10: error: no field named 'struct' in enum '@typeInfo(builtin.Type).Union.tag_type.?'
        .@"struct" => H

/home/tymek/.cache/zig/p/12208fab56fa3082d207beb09b1d5134a447ce006e78026f6951d89dd8c1279bded5/src/httpz.zig:233:10: error: no field named 'struct' in enum '@typeInfo(builtin.Type).Union.tag_type.?'
        .@"struct" => H,
        ~^~~~~~~~~
/usr/lib/zig/std/builtin.zig:256:18: note: enum declared here
pub const Type = union(enum) {
                 ^~~~~
src/main.zig:20:30: note: called from here
    var server = httpz.Server(void).init(alloc, .{
                 ~~~~~~~~~~~~^~~~~~
referenced by:
    callMain: /usr/lib/zig/std/start.zig:524:32
    callMainWithArgs: /usr/lib/zig/std/start.zig:482:12
    remaining reference traces hidden; use '-freference-trace' to see all reference traces

There's a zig-0.13 branch

how do i zig fetch from the branch?

zig fetch --save git+https://github.com/karlseguin/http.zig#zig-0.13

There's a zig-0.13 branch

does the main branch target upstream?

Yes, it does.
Although, I think when Zig 0.14 is released, I'm going to change things around. I'll make master track Zig 0.14 and do any new development in a separate branch.

@karlseguin well, thanks. that would allow for better dev exp.
what about versioning your library? I think it would be better to open a new issue for it