add hexadec to excess3 converter

Related Issue

Info about Issue or bug

fix #3287
Describe the changes you've made
A clear and concise description of what you have done to successfully close your assigned issue. Any new files? or anything you feel to let us know!
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.

Screenshots

@sairish2001 new pr #3626


GCS decompressive transcoding not supported when reading

Problem description
Using smart_open (unreleased version with GCS support) to download files from GCS with transparent decompressive transcoding enabled may lead to incomplete files being downloaded depending on the compressed file size.
With Google Cloud Storage there is the option to store gzip-compressed files & use decompressive transcoding to transparently decompress these when downloading. Decompression is thenhandled by Google servers. In this case, the filename wouldn't have any compression extension (eg. file.csv), however when inspecting it's metadata, it would contain something like that:
{
    "Content-Type": "text/csv; charset=utf-8",
    "Content-Encoding": "gzip"
}

This would be fine if it weren't for the fact that in such cases, Blob()._size will return the compressed size. Since smart_open uses this to understand when to stop reading, it results in incomplete files.
Steps/code to reproduce the problem
write ~400KB file (larger than smart_opens's default buffer size)
$ cat /dev/urandom | gtr -dc A-Za-z0-9 | head -c 400000 > rand.txt

$ ls -l
total 1024
-rw-r--r--  1 gustavomachado  staff  524288 Feb 17 12:34 rand.txt

upload file to GCS
$ gsutil cp -Z ./rand.txt gs://my-bucket/
Copying file://./rand.txt [Content-Type=text/plain]...
- [1 files][293.8 KiB/293.8 KiB]
Operation completed over 1 objects/293.8 KiB.

resulting (compressed) file is 293.8 KiB.
check file metadata
$ gsutil stat gs://my-bucket/rand.txt
gs://my-bucket/rand.txt:
    Creation time:          Mon, 17 Feb 2020 13:45:36 GMT
    Update time:            Mon, 17 Feb 2020 13:45:36 GMT
    Storage class:          MULTI_REGIONAL
    Cache-Control:          no-transform
    Content-Encoding:       gzip
    Content-Language:       en
    Content-Length:         300842
    Content-Type:           text/plain
    Hash (crc32c):          Ko+ooA==
    Hash (md5):             8C6OlwZIR+fgRMy2xmQqLw==
    ETag:                   CNWW+Kjc2OcCEAE=
    Generation:<PHONE_NUMBER>379733
    Metageneration:         1

download file using smart_open (gcloud credentials already set)
>>> from smart_open import open
>>> with open('gs://my-bucket/rand.txt', 'r') as fin:
...     with open('downloaded.txt', 'w') as fout:
...         for line in fin:
...             fout.write(line)
...
348550

check resulting file size
$ ls -l
total 1472
-rw-r--r--  1 gustavomachado  staff  348550 Feb 17 14:48 downloaded.txt
-rw-r--r--  1 gustavomachado  staff  400000 Feb 17 14:45 rand.txt

original file is 400KB however downloaded file is 348KB. not sure why it's still bigger than the 300842 reported by Google, though.
Versions
Please provide the output of:
>>> import platform, sys, smart_open
>>> print(platform.platform())
Darwin-18.7.0-x86_64-i386-64bit
>>> print("Python", sys.version)
Python 3.7.2 (default, Dec  9 2019, 14:10:57)
[Clang 10.0.1 (clang-1<IP_ADDRESS>)]
>>> print("smart_open", smart_open.__version__)
smart_open 1.9.0

smart_open has been pinned to 72818ca, installed with
$ pip install git+git://github.com/RaRe-Technologies/smart_open.git@72818ca6d3a0a99e1717ab31db72bf109ac5ce65

Checklist
Before you create the issue, please make sure you have:

[x] Described the problem clearly
[x] Provided a minimal reproducible example, including any required data
[x] Provided the version numbers of the relevant software

Possible solutions
Setting buffer_size to a value larger than the compressed file size will of course download it in it's entirety, but for large files that would mean loading the entire file into memory.
A reasonable option would be to check Blob().content_type, and if it is equal to 'gzip', call Blob().download_as_string with raw_download=True, and then somehow handle decompression internally with the already-existing decompression mechanisms
If the maintainers agree this would be a viable solution, I'll be happy to provide a PR implementing it.

Thanks for reporting this and providing clear descriptions and solutions! Your possible solution seems like a good start, but how will we be able handle compression formats other than gzip? Could we just switch raw_download=True for everything or create a transport_param to toggle raw_download? Is there a case when someone would want to not read a raw download of a compressed file?
I think we will need to come up with a way to tag compressed files with their appropriate content_type metadata on uploads as well.

I misunderstood, other compression formats would not have this problem as they are not transparently decompressed by Google. It looks like the raw_download method on Blob.download_as_string has been removed in the latest version of google-cloud-python so I'm not sure we should use something that is being phased out.

It'd be good if we can avoid adding more options. Can you think of a logical way to handle this?
Why don't we just perform all the compression/decompression on our side?

Ok @gdmachado's suggestion is probably what we want to do then. If Blob.content_encoding == 'gzip' and file_extension != '.gz' then we save state so smart_open knows the file was transcoded to gzip and download the raw compressed data which will have a size true to Blob.size. I think smart_open.gcs.SeekableBufferedInputBase.read will have to use this state to know to decompress the data before it is returned to the user.

I can confirm this problem, though I noticed it in a different manner. I tested with a file similar to above, but used "transport_params=dict(buffer_size=1024)" to force it to stream in parts. I have 4 cases to share with you. Testing with gcs files named file.txt and file.txt.gz, and ignore_ext=True and False. Data was uploaded to gcs

file.txt with ignore_ext=True, file.txt with ignore_ext=False, file.txt.gz with ignore_ext=True
Reads first buffered chunk ok, subsqeuent chunks throw "urllib3.exceptions.DecodeError: ('Received response with content-encoding: gzip, but failed to decode it.', error('Error -3 while decompressing data: incorrect header check'))"

file.txt.gz with ignore_ext=False
raises "OSError: Not a gzipped file (b'FI')" on immediately

I modified the smart open code to add the "raw_download=True" to the download_as_string() call, and here are the results.

file.txt with ignore_ext=True, file.txt with ignore_ext=False, file.txt.gz with ignore_ext=True
File streams from open() object as a gzipped binary. Size of the data matches the compressed size on gcs. Able to decompress this data successfully using gzip.decompress(data), and the data matched exactly.

file.txt.gz with ignore_ext=False
File streams from open() object as uncompressed data. File size matches the RAW size before upload. Data matches the raw uncompressed data exacly.

Hope this helps y'all to find a good solution. Included below is the quick and dirty script I used to demo this.
set -e
GCP_BUCKET=???
echo "START" > rand.txt
cat /dev/urandom | gtr -dc A-Za-z0-9 | head -c 40000 >> rand.txt
echo "END" >> rand.txt
ls -al rand.txt
gsutil cp -Z rand.txt ${GCP_BUCKET}/rand.txt
gsutil cp -Z rand.txt ${GCP_BUCKET}/rand.txt.gz
gsutil ls -l ${GCP_BUCKET}/rand.txt ${GCP_BUCKET}/rand.txt.gz

for f in ${GCP_BUCKET}/rand.txt ${GCP_BUCKET}/rand.txt.gz ; do
  for ignore_ext in True False; do
    python -c "
from smart_open import open
import gzip
print('\n\n--------${f} ignore_ext=${ignore_ext}----------')
with open('${f}', 'rb', ignore_ext=${ignore_ext}, transport_params=dict(buffer_size=1024)) as f:
  data = f.read()
  print(len(data))
  print(data[:5])
  print(data[-4:])
  raw = gzip.decompress(data)
  print(len(raw))
  print(raw[:5])
  print(raw[-4:])
print('Done')
    " || echo "Function failed"
  done
done


Always promote int8 to int32 in commonShflSync

Fix for https://github.com/ROCmSoftwarePlatform/triton/issues/367

If the type here is Fp8 this won't work at the moment, but fp8 support isn't complete. We'll revisit.


[v2.7] Adjust object conflict messages seen in provisioning log

Issue: https://github.com/rancher/rancher/issues/39108
Problem
There are intermittent error messages reported in the provisioning log relating to object version conflicts. In all observed scenarios these error messages are benign and resolve themselves automatically. However, the presence of these error messages and subsequent UI display gives the end user an impression that something has failed, when in fact the issue is only temporary.
Solution
Do not display these error types in the Rancher UI, but do continue to print them into the Rancher logs.
I've also tried to use generic.ErrSkip in some places I suspect these errors are coming from, however this did not result in the removal of the error message.
Testing

Create a rancher server
provision an RKE2 cluster on any infrastructure provider. Create 3-5 nodes in a single all role node pool
observe that no object conflict errors are reported in the provisioning log within the UI

Engineering Testing
Manual Testing
I've done the above
Automated Testing
n/a
QA Testing Considerations
reproducing the original error is difficult, i've had some success by limiting the resources available to the Rancher server.
Using 3-5 nodes seems to have this error appear more frequently.
I've also seen this error occur when scaling up and down, but even when doing this the error is still not consistent.
Regressions Considerations
This change omits certain log messages, it has no impact on existing functionality outside of what is shown in the UI.

I've updated this PR to append a clarifying message to the end of these error messages, allowing them to be still be shown in the UI but indicating to the user that they are not significant errors or cause for concern


Expose Nettest{Component,Image}

Expose as generic Nettest image (instead of specific to metrics proxy) so that it can be consumed by other projects (such as subctl) and properly overriden in downstream forks.
Signed-off-by: Mike Kolesnik<EMAIL_ADDRESS>

The use of nettest is a convenience for now, the metrics proxy might not continue using it. Conceptually it’s a different component which just happens to use the nettest image.

The question is do we treat nettest as a component or not?
I think it's already is a sort of component, and in that case while I agree conceptually the metrics proxy is a distinct component, I dont see a benefit of making that separation now (we could always separate them if necessary).
Regardless, I don't mind to add a separate nettest component and leave the proxy be, even though they both use the same image.
Thoughts?

OK, I hadn’t understood that the important part of this is exposing nettest to make it overridable ;-).
I agree we need to make nettest overridable, and it is sort of a component (for subctl at least in diagnose etc.); let’s keep the metrics proxy component separate and add a new nettest component.


Introduce "keywords_*" fields

closes #185
closes #174
closes #169

#174 wants to determine status for a repertoire, while these keywords are at the study level. @bcorrie is this sufficient for your requirements? Likewise for #169 where I think we also need to add pair_id at the rearrangement level.

@schristley I agree that we need additional keyword fields on lower levels and guideline how the terms "trickle down" (or "float up") between these levels. Is it already clear that we need one the repertoire level (if that is still the correct term)?

@schristley is it ok to proceed and merge? Or is there anything that need to be clarified?

@bussec looks good!


New bootstrap theme and demo application

This is a newly created edx bootstrap theme. It looks very different from v1, and will now be delivered with a documentation site.
SCSS & Theme Structure

The edx-bootstrap "core" consists of bootstrap itself + open-edx extensions
(currently these extensions are a new button variant, btn-inverse-{color}, and expanded color utilities.
The edx and open-edx themes. Themes contain imports for 1. fonts, 2. variables, 3. the core, and 4. overrides.
The only required scss file for any user of open-edx will be scss/core/core.scss

Doc site

To run locally npm install && npm start
Deployments happen manually for now using npm run deploy

The updated README: https://github.com/edx/edx-bootstrap/blob/b3ab7e722ce831bb3c3f7464e4dafbca1ed575d4/README.rst


New ergo-test package ; More Engine consolidation

Coverage decreased (-0.5%) to 98.405% when pulling 1908e2f7ce74bd224845902f279fc53c6a57fe99 on jeromesimeon:js-fix-engine-consolidation into 19518b742fe7cf0aa3adadf27051c536157a4c4c on accordproject:master.


Fix NavigatorCard mobile toggle spacing

Bug/issue #, if applicable: 90968150
Summary
Fixes the navigator Medium size toggle spacing, when open.

Dependencies
NA
Testing
Steps:

On Medium size, open the navigator
Assert the toggle is not flush to the edge of the screen, and is inline with the rest.

Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[ ] Added tests - no, css only
[ ] Ran npm test, and it succeeded
[ ] Updated documentation if necessary

@swift-ci test

@swift-ci test


Updated alert to always show to user whenever triggered

Fixes: #586
If the topbar is on the screen the notification appears below it, else the notification appears on the top.
Large Screens

Medium Screens

Small Screens

Signed-off-by: Matheus Fernandes<EMAIL_ADDRESS>

This looks really classy, thanks :+1: That being said, there are some places in which we use the alert and alert-info (or other) classes without them being actually "alerts" :D Take for example the /admin/registries page:

In this case I'd prefer the old behavior. Also, I'd also prefer the old behavior in the pages in which users are not logged in and they mess up in a form. For example:

Therefore, this new implementation is handy in most cases, but there are these corner cases that I'd rather keep the old behavior. What do you think ?

@mssola I agree with you!
Correct me if I'm wrong, but only "danger" and "info" alert are used. So, we could use more alert types to group the alerts in float and fixes  alerts... What do you think?

@msfernandes agreed. Feel free to add more alert classes.

Now, only error messages are displayed as "float-alert". So, when the messages are only for information the alert is shown as in the old behaviour.
I also updated the icon in "danger" alerts to one "!" instead of the "i".

@msfernandes looks good to me. Note however that some errors are treated as information though! For example, in the pictures you uploaded on your first comment show some of the alerts that should be considered float-alert :) (there are others though, like creating an invalid team).

From the UX and UI point of view, it looks good to me. That being said, you should rebase your commits onto current master, because right now it's impossible for me to review your changes :sweat_smile:

I can see that you've added more flashy classes. That's fine, by notice that the vast majority of alerts are floating. Therefore, I'd say that the new classes should be "fixed_alert" or something like that, and those will only be applied in those cases I pointed out.
This way, you would avoid all the majority of class changes, and you would also avoid to add all the time:
$('#alert .alert').addClass('float-alert');
refreshFloatAlertPosition();
setTimeOutAlertDelay();

Agreed :)
But, about the code repetition, I think that's necessary because the notification is triggered by ajax and the refreshFloatAlertPosition was already executed. So, when the notification is displayed, the position must be adapted to the page scroll, to not override the navbar and to do this I need to execute refreshFloatAlertPosition again, every time that a notification is displayed...

@msfernandes isn't there a way to do this always ? I fear that all this code repetition will be troublesome in the future.

@mssola I don't think so...
What we can do is change the notification position (put in right inferior corner, for example). In this form we don't need to use the refreshFloatAlertPosition because the position will be always on the same location.
The setTimeoutAlertDelay is important to dismiss the alert after 4 seconds. But need to be executed every time that the notification is triggered, because the time should start counting after that.
So, it's possible reduce these three lines repetition to one...

What we can do is change the notification position (put in right inferior corner, for example). In this form we don't need to use the refreshFloatAlertPosition because the position will be always on the same location.

I see that the alternative is worse :) Let's keep what you have here.

The setTimeoutAlertDelay is important to dismiss the alert after 4 seconds. But need to be executed every time that the notification is triggered, because the time should start counting after that.

I did not notice that :)

So, it's possible reduce these three lines repetition to one...

Yeah, why not ? :) You can have just one function that calls all these functions. That would be better in this case I think.
So, long story short:

You should treat alerts just like you did in your very first commit, and add special classes for the ones I'm mentioning in this comment.
You should write a function that includes refreshFloatAlertPosition and setTimeoutAlertDelay and use them in AJAX files.

@msfernandes that looks better :) Now you only have to fix this failing test and rubocop, and I think it will be ready to be merged :)

Perfect :clap: Squash your commits and I'll merge it.

@mssola squashed!

@msfernandes cool, merged ! :clap: Next time, instead of including all your commits' descriptions into your squashed one (and also including sign-offs), just write a single description and only sign it once. Otherwise it's a bit confusing :wink:

sorry.. hahaha


Fix archives page, Remove archives from header, Add archives to footer

Fixes

Fixes #498 by @zackkrida
Description
Moves header archive items to the /archives page (using a new page-with-title.html template), and adds an "Archives" link to the footer.
Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main or master).
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no
visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Updated screenshot @kgodey @TimidRobot


Can you add introductory price?

Your awesome package is well used.
I need introductory price infomation.
Can you add introductory price?
getIntroductoryPrice()
getIntroductoryPriceCycles()
getIntroductoryPricePeriod()

OK. We will implement these methods. Thank you for your concern.

@gitbyzin
Just wondering. Is this feature supported in Android too?

@gitbyzin Sorry for late reply. But, I found there would be nothing to implement in source code. It's some more setting in iTunesConnect product page.
It looks like matter of parsing the product information.


Update OriginStake.toml

Description
Previous PR
https://github.com/anoma/namada-testnets/pull/1546
https://github.com/anoma/namada-testnets/pull/1620
https://github.com/anoma/namada-testnets/pull/1656
All previous genesis validators should name their PRs "Update {validator_alias}.toml for tesntet-15" and provide links to previous PRs merged.
If this is an UPDATE for a previous genesis validator
Checks are made against your net_address. If this has changed since the previous testnet, make sure you provide links of previous prs merged from your previous git username.
Thanks in advance!
Checklist before merging

[ ] Only one toml is added in this PR
[ ] The file being added is indeed a .toml file
[ ] The toml being added is to the correct folder, and only to the correct folder
[ ] The eth_hot_key and eth_cold_key are present
[ ] The email, discord, elements telegram, and twitter fields are present and valid

Successfully validated transactions.toml file for commit hash 318b539


Checkbox inner state doesn’t update when outer state changed

When playing around examples I found strange behavior in Todo example.

Add 2 todos
Mark one done
Switch to only done todos
Switch to undone todos

Result: undone todo marked as done, internal checkbox state doesn’t reflect external change

Could you specify the file to be seen?

https://github.com/revery-ui/revery/blob/master/examples/TodoExample.re#L142
Here we set checked prop from outside
After filtration as I mentioned above Checkbox component doesn’t respect that value and uses internal state instead.
Maybe it doesn’t initialized keeping its previous state

Also if I add 3 todos and mark as done one of them, this issue doesn't replicates, but once there is equal amount of done and undone todos, issue replicates


I could not find the extension method "EnableCallContextPropagation" .

Description

[Enter feedback here]
Page URL
https://learn.microsoft.com/en-us/aspnet/core/grpc/clientfactory?view=aspnetcore-8.0
Content source URL
https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/grpc/clientfactory.md
Document ID
caebee8f-f4aa-17a4-3681-d10c4695e9c0
Article author
@JamesNK

@yerald231ger, thanks for relating your experience with the topic.
You do have the right NuGet package and the latest version of it.  EnableCallContextPropagation() would be available.  However there must be a context here under which it would not be, and I am not spotting it.
This would seem fine, adding a gRPC client for SationConsole.SationConsoleClient with the server address set to http://localhost:5050. Then, it’s attempting to enable call context propagation for that client:
services.AddGrpcClient<SationConsole.SationConsoleClient>("GaxVol.Client", options =>
{
    options.Address = new Uri("http://localhost:5050");
}).EnableCallContextPropagation();

Assuming the namespace is made available:
using Grpc.AspNetCore.Server.ClientFactory;
Given it is likely very specific to the context of your specific app/code, I recommend posting to a community support group like Stack Overflow with the asp.net-core tag.  There is also a grpc-dotnet tag.
grpchttps://stackoverflow.com/questions/tagged/grpc-dotnet.
• Stack Overflow
• ASP.NET Core Slack
• ASP.NET Gitter
Potential bugs in the ASP.NET Core framework and product feedback can be reported at dotnet/aspnetcore issues (include full repro steps and cross-link to a repro project in GH when appropriate).
@JamesNK, in case there is an opportunity for content improvement that would be related to this example.

The docs already say which package is required here: https://learn.microsoft.com/en-us/aspnet/core/grpc/clientfactory?view=aspnetcore-8.0
I don't think there is anything to improve.

You're right, I miss the nuget package. I was confusing it with
Grpc.Net.ClientFactory.
Thanks.!
On Thu, Jul 11, 2024 at 9:20 PM James Newton-King @.***>
wrote:

The docs already say which package is required here:
https://learn.microsoft.com/en-us/aspnet/core/grpc/clientfactory?view=aspnetcore-8.0
I don't think there is anything to improve.
—
Reply to this email directly, view it on GitHub
https://github.com/dotnet/AspNetCore.Docs/issues/33049#issuecomment-2224350104,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACRVEQXOQWEYRU364PKW5MTZL5DRHAVCNFSM6AAAAABKWFG5Q2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRUGM2TAMJQGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Gerardo Sánchez Hernández - Ingeniero en Sistemas Computacionales
Tel:<PHONE_NUMBER>
Correo: @.*** | @.***


Limit carb entry to Max Carbs limit

Currently, if a user enters more carbs than their Max Carbs setting, the value of Max Carbs is entered. This PR prevents any carbs from being entered if the value is greater than the Max Carbs setting.
I also updated the Bolus module's button to match the formatting of the AddCarbs module.
Marked as draft because I'd like feedback before updating all the localization files.
I also thought it might be nice to make the "enact bolus" and "save and continue" text orange if IOB+bolus>MaxIOB or if COB+carbs>MaxCOB, but have not added that yet.

This looks good to me! I have not tested yet, but I like what I see from the screenshots.

Good catch, @bjornoleh. I’ll try to figure that out and add it to this PR as well.
Any thoughts on also adding a warning here if MaxIOB or MaxCOB is exceeded? Maybe just keep the text the same but make it orange and add a warning/confirm popup/alert?

Max IOB and Max COB are originally intended to harness automated dosing, so I am unsure if it's worth adding notifications during manual entries? Just a thought :-)

This looks great!
Similarly to what @bjornoleh found, if you follow the same steps when entering carbs, the same thing will happen :)

It was recommended in Discord that we use this issue for discussion on max FPU thresholds.
In case we do want to bundle that discussion here (or maybe eventually breakout to a different issue?):
The idea of reusing the max carb threshold for max fat and max protein entries was floated. A few people shared concerns with this (myself included) due to people with different diets that may lean toward higher fat and protein meals and lower carb meals.
I thought I'd upload a sampling of 3 meals someone may eat who targets ~20g carbs in a meal, with significantly higher fat and protein.
In these examples (breakfast, lunch, dinner), someone may want a max carb threshold of something like 50g (roughly double the highest meal in these examples), whereas a max fat and protein might need to be more like 150 or 200 (roughly 150% of the highest entry shown).
Breakfast:

Lunch:

Dinner:

For Max FPU, I was thinking of just using Max Carbs as the limit for carb equivalents, not for grams of fat/protein.
So if have an “override with a factor of” is set to 1, a 100g max carb limit would be either 111g of fat or 250g of protein or 10.9F0.4P=100 if a combo of fat and protein is entered.
If set to 0.5, a 50g Max Carb limit would be 111g fat or 250g protein, or 0.50.9F0.4P=50
My reasoning on this is that when you enter “fat” and “protein”, you’re really just entering carbs since that’s all oref understands anyway.
So that option is more algorithm based as you’re limiting the inputs of carbs to oref, whether those carbs are entered as carbs or just converted to carbs from fat and protein.
Let’s call the above Option A.
Option B could be having three settings: “Max Carbs”, “Max Fat”, “Max Protein”. I think it’d be weird to have protein and fat share a “Max Fat/Protein” setting since Fat is worth twice as many carbs as protein is.
Option C could be a single “Max Carbs from Fat and Protein” setting and have work basically the same as Option A except checked against a nee setting instead of just “Max Carbs”
Feel free to suggest an option D, E, etc.

Just to be sure I understand this, this only affects the bolus calculator based on the Max Carbs limit (max you can add in one sitting); but will not have any effect on the MaxCOB safety limiter?

Just to be sure I understand this, this only affects the bolus calculator based on the Max Carbs limit (max you can add in one sitting); but will not have any effect on the MaxCOB safety limiter (nor are any limits in the bolus calculator affected by MaxCOB)?

Correct.

Or, well, I suppose not necessarily the bolus calculator, just the “add meal” screen, but that screen will take you to the bolus calculator screen.

@MikePlante1 No it won't 🥳 Disabled that annoying feature a long time ago haha. Anyway, ok so it only affects the Add Meal-thingy. Splendid. :)

I’d go with either a single guardrail for “max fat/protein” or two, one for “max fat” and one for “max protein”.
I think 1 is probably fine, since the biggest thing this guards against is order of magnitude jumps where fat fingering turns 60 into 600 for instance.
More details shared in Discord discussion for reasoning.

I think this needs to be added as a new issue, @bjornoleh:

But, I also noticed a peculiarity that was not introduced here. If this is something that could be fixed here, please do, otherwise we can enter a separate issue:
Observation:
Enter a value in the Enact Bolus: Bolus - Amount field Tap Clear and see the entry being cleared out Tap Doneto escape the keypad (there is no other way, except swiping down) Watch how the previous bolus entry reappears.
Some recent PR introduced a button to close the keypat, not sure if any of that could be used here.
If this is outside of the scope of this PR (it is!), we'll copy this to a separate issue!

Added Max Fat and Max Protein settings.
I think this one is now ready for review.

Merging with two approvals


Add author

Hi, Im looking through your code wanting to add the username as author of the PDF. Maybe you have an idea how?
Thanks!

This would require modifying the source code to

fetch the username from Grafana (that is if there is an API call for that). Or inject the username as a command line parameter.
pass the username to the LaTeX templating code as part of the template data struct
update the tex template to use this new data.

I don't plan on adding this functionality.


Animating with opacity works only to increase opacity

I am working with the animation api and I want some thing to fade in and out but the out is an issue.
For example:
'use strict';

var React = require('react-native');
var {
  AppRegistry,
  StyleSheet,
  View,
  Animation,
  TouchableHighlight
} = React;

var AnimationExample = React.createClass({
  componentDidMount: function() {
    Animation.startAnimation(this.refs.box, 600, 0, 'easeInOutQuad', {'opacity': 1});
  },

  closeBox: function() {
    Animation.startAnimation(this.refs.box, 600, 0, 'easeInOutQuad', {'opacity': 0});
  },

  render: function() {
    return (
      <View style={styles.container}>
        <TouchableHighlight underlayColor="rgba(0,0,0,0)" onPress={this.closeBox}>
          <View>
            <View style={styles.box} ref="box" />
          </View>
        </TouchableHighlight>
      </View>
    );
  }
});

var styles = StyleSheet.create({
  container: {
    flex: 1,
    justifyContent: 'center',
    alignItems: 'center',
    backgroundColor: '#FFFFFF',
  },
  box: {
    margin: 10,
    width: 100,
    height: 100,
    backgroundColor: '#CCCCCC'
  }
});

AppRegistry.registerComponent('AnimationExample', () => AnimationExample);

On click the box will just go to opcaity: 0 with  no animation.

Sorry about this, we removed the Animation API because it was written in a day and not tested anywhere. In the meantime you can use https://github.com/chenglou/react-tween-state or requestAnimationFrame with setState where you manually update the opacity.


Print a warning for Viewports that have 2D usage and HDR enabled

2D usage + HDR isn't supported, so the usage must be changed to 3D to be able to use HDR.
I wanted to print the node path using KoBeWi's suggested solution, but that didn't work either.
See https://github.com/godotengine/godot/issues/41151 and https://github.com/Zylann/godot_heightmap_plugin/issues/269.
Note: Not applicable to the master branch as the notion of viewport usage is gone there.

Thanks!


Disable GenAI disclaimer

Since we don't currently have any GenAI features, the disclaimer on the landing page can be removed

Thanks to Philip this was closed.


pull request for e2e test at 02/02/2017 12:56:15

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


When I play iHeartRadio for some hours,SampleApp will exit.

When I play iHeartRadio for some hours,SampleApp will exit by the following error:
2017-09-19 15:24:30.627 [  d] 0 ContextManager:updateStateLocked:action=updatedState,state={"token"\:"amzn1.as-ct.v1.Dee-Domain-Music#ACRI#url#ACRI#083e94da-99bf-4f6a-9a4b-e32078018051\:1"\,"offsetInMilliseconds"\:6238378\,"playerActivity"\:"PLAYING"},namespace=AudioPlayer,name=PlaybackState
2017-09-19 15:24:31.009 [ 25] 0 HTTP2Stream:receivedRequestId:value=x-amzn-requestid\: 0672fefffe093f48-00005b9d-00006acf-4c20b41341414351-50fe2fb8-293
2017-09-19 15:24:31.020 [ 25] 0 CurlEasyHandleWrapper:reset:responseCode=HTTP_RESPONSE_SUCCESS_NO_CONTENT
2017-09-19 15:24:31.504 [ 25] 0 HTTP2Stream:receivedRequestId:value=x-amzn-requestid\: 0672fefffe093f48-00005b9d-00006acf-4c20b41341414351-50fe2fb8-295
2017-09-19 15:24:31.517 [ 25] 0 CurlEasyHandleWrapper:reset:responseCode=HTTP_RESPONSE_SUCCESS_NO_CONTENT
bluetoothd[253]: Endpoint unregistered: sender=:1.1 path=/MediaEndpoint/A2DPSource
2017-09-19 15:24:35.528 [  2] E MediaPlayer:handleBusMessageError:source=audio_sink-actual-sink-pulse,error=pa_stream_writable_size() failed: Connection terminated,debug=pulsesink.c(1752)\: gst_pulseringbuffer_commit ()\: /GstPipeline\:audio-pipeline/GstAutoAudioSink\:audio_sink/GstPulseSink\:audio_sink-actual-sink-pulse
bluetoothd[253]: Endpoint unregistered: sender=:1.1 path=/MediaEndpoint/A2DPSink2017-09-19 15:24:35.559 [  2] 0 MediaPlayer:callingOnPlaybackError:error=pa_stream_writable_size() failed\: Connection terminated
2017-09-19 15:24:35.577 [  2] 9 AudioPlayer:onPlaybackError:error=pa_stream_writable_size() failed\: Connection terminated
2017-09-19 15:24:35.588 [  d] E AudioPlayer:executeOnPlaybackError:error=pa_stream_writable_size() failed\: Connection terminated

Looks like your Bluetooth device got disconnected which triggered the media player .
Where there other audio output devices configured where you wanted it to switch to  ?

Hi @NyxRockchip2017 , it looks like pulse audio of gstreamer is crashing. We will investigate it further and tell you. Thanks!

Can you tell us for how many hours did you play iHeartRadio?

What platform are you using? Linux or Raspberry Pi? Can you check if you have pulseaudio installed by running a command "pulseaudio --version".

My platform is Linux and the pulseaudio's verison is pulseaudio 9.0

Hi @NyxRockchip2017,
Our next release will includes improvements on the MediaPlayer. My suggestion is to wait for it and let us know if it still occurs.
Thanks,
Benny

How soon does the new version release?

Hi @NyxRockchip2017,
Thanks for asking about this, but we can't comment on schedule.
Thanks,
Benny

Closing due to inactivity. Please try again with the latest version and reopen if you're still seeing the issue.
Thanks,
Sanjay


ZCash sighash upgrade

The ZCash blossom hardfork changes the sighash algorithm for replay protection. Unfortunately, unlike BCH which accomplishes this by changing the algorithm for the old version and by keeping the algorithm the same post-fork, ZCash requires all wallets to upgrade to the new algorithm.
For us this means we need to make a small one-line change to the sighash algorithm so transactions can be made. Prior to this change blockbook will reject transactions signed with the old algorithm.
This does potentially create a more serious surface area for bugs if funds are in a 2 of 3 escrow address and one party signed to release the funds pre-fork and the other party post-fork as the transaction will be invalid.
We are considering a change to the protocol to request a party to resign the transaction, though this does not guarantee that party will come back online to do so.

For acceptance criteria, we should run through all order scenarios with ZEC to make sure they all continue to function as expected.
We'll need to step through the purchase and order tests in https://github.com/OpenBazaar/openbazaar-desktop/issues/1766 for ZEC, and at minimum a subset of the same tests for other coins to make sure they're not affected.

Resolved with #1926


Add CSS games subcategory under Frontend category.

Fixes Issue
Closes #2258: Add CSS Games subcategory under Frontend
Changes proposed

Create a new subcategory called CSS Games under the frontend category.
Modify the CategoryDescriptions.ts file to include CSS Games.
Create a new file called css-games.json in the frontend database folder to include all the URLs related to CSS Games.
Modify the sitemap.xml file to include CSS Games.
Modify the index.ts file in the database folder to include CSS Games.
Modify the data.ts file in the database folder to include CSS Games.

Learn CSS Grid, Flex and more by Playing Games

css grid garden
flex box froggy
flukeout
coding fantasy

Screenshots

Note to reviewers

@CBID2 why have you renamed to games? won't this be confusing for other people?

@rupali-codes Please deploy.
I second this, it can be games too but considering it is in frontend -> css games would be better because most of the game platforms are related to CSS.

@CBID2 why have you renamed to games? won't this be confusing for other people?

@rupali-codes Please deploy.
I second this, it can be games too but considering it is in frontend -> css games would be better because most of the game platforms are related to CSS.

Oh I see.

@rupali-codes & @Anmol-Baranwal, I approved the PR.

NP, thank you!


Error fetching dependancy while running vagrant up

Description:
There was an error pulling in some dependencies while running vagrant up, tried to repro this issue 3 times with different setups but could not get a solid repro.
Steps:

Clone latest versions of obc-peer, obc-dev
Run vagrant up

Results:
Logs show vagrant was unable to pull in a dependency and failed
Expected Results:
Failure to pull in a dependency might be caused by network issues or network issues on dependency host.  Logged here for informational purposes that this can and did happen.
Logs to prove it:
==> default: Updating certificates in /etc/ssl/certs...
==> default: 19 added, 19 removed; done.
==> default: Running hooks in /etc/ca-certificates/update.d....
==> default: done.
==> default: Executing: gpg --ignore-time-conflict --no-options --no-default-keyring --homedir /tmp/tmp.GLWb6GFfxt --no-auto-check-trustdb --trust-model always --keyring /etc/apt/trusted.gpg --primary-keyring /etc/apt/trusted.gpg --keyserver hkp://p80.pool.sks-keyservers.net:80 --recv-keys 58118E89F3A912897C070ADBF76221572C52609D
==> default: gpg:
==> default: requesting key 2C52609D from hkp server p80.pool.sks-keyservers.net
==> default: ?:
==> default: p80.pool.sks-keyservers.net: Host not found
==> default: gpgkeys: HTTP fetch error 7: couldn't connect: Success
==> default: gpg:
==> default: no valid OpenPGP data found.
==> default: gpg:
==> default: Total number processed: 0

Monday traffic makes external dependencies less than 100% reliable, nothing we can do short of mirroring all dependencies.


Footer links should accept a flexible number of items

5 items breaks the layout.

@theadamparker @Jura -- https://github.com/undp/design-system/pull/1114
I've attached some screen shots of how my setup can adjust from medium to x-large, in this case with 6 menu sections
It can handle up to 3 across on medium, 4 across on large, and 6 across on x-large. It moves additional menu sections to a new row, while always keep the search, globe, term links and social media links section off to the right side (in-line if there is room)
let me know what you think
x-large

medium

large

Example of 8 items

Example of only 3 items

@paboden the wrapping approach is exactly what we need. However, the legal links (report fraud, terms of use, etc.) should remain aligned with the top row.
Example:

This will mean fewer items can be placed in the first row, but it is more closely aligned with the original visual appearance.

@theadamparker pushed up a fix. Take a look and let me know what you think. Cheers

Verified https://develop.design-system.pages.dev/?path=/story/templates-country-homepage--country-homepage

@mrelevo , please verify

verified in Dev

verified in storybook


docs: README.md updated to be in sync with docs.minio.io

Added API Reference Section.
Added individual links to each API.
Brought it as close as possible to docs site.
Add constructor so that this document is complete in itself.
Removed How to Run this example section.
Changed desc to description in the table heading.
Call it Full Docs Site to Complete Documentation.

Can you fix merge conflicts on this patch @koolhead17 ?

Done 👍

Atul, there are more edits to be done in this doc:

Formatting issue:  bullet point for Google Cloud storage is not consistent
Quickstart example guide: The half of the example is copied wrongly. You need to delete that and continue with the Params and Description right after the text "You need four items in order to connect to an object storage server."
Move the object operations and file object operations below the API Reference : Presigned Operations.


Allow to hook into any writable stream

This would add the option streams that would accept an Array of Writable streams. By default if would be:

[process.stdout, process.stderr]  when calling hookStd([options], transform)
[process.stdout]  when calling hookStd.stdout([options], transform)
[process.stderr]  when calling hookStd.stderr([options], transform)

The library would then do the same usual, but would get the stream to hook into fro m the new option rather than from process.

Seems like a good option to have. What is your use-case?
It is a little bit weird that hookStd.stdout({streams: [process.stderr]}) would be allowed. Should we maybe contain the option to the main hookStd()? Or do you have a use-case I'm not seeing?

I guess when using hookStd.stdout and hookStd.stderr the streams option will be ignored.
The use case is to use a transform function to replace sensitive text from the console output. I also want the user to be able to customize the stdout and stderr used by the library and ultimately pass them to signale.

I guess when using hookStd.stdout and hookStd.stderr the streams option will be ignored.

👍

Alright. PR welcome :)

Here is the PR: #19


Re-enable Kotlin plugin smoke tests

Fixes https://github.com/gradle/gradle-private/issues/3282 and improves the DSL around accepting deprecation warnings for plugins.

@bot-gradle test and merge

OK, I've already triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.


Cancellation changes and drop extreneous sources

Rename CancellationGroup to CompletionGroup

Also, add a way to install completion handlers that are called when
a member of a completion group completes. This allows us to define
supervisors for coroutines or actors. Note that, unlike for futures
we sometimes do not want to wait for termination of a coroutine (or
actor) in a specific val. Instead, the coroutine could be terminated
when the enclosing completion group completes (i.e. gets cancelled).
But we still want to handle abnormal early termination of the coroutine
somehow. Completion handlers are the solution to this. A handler
can be installed to be valid in a scope. Then, if any cancellable objects
terminate, the handler is called.

Move sources that have to do with general suspension handling to
a dotty test directory.

Closing as it's already in tree


Updated libuv reference url with correct line number

Updated the libuv reference url  to point 1.0.0 source and corrected the line number

@imranbaloch Thanks for your contribution.  We really appreciate it.
:shipit:


Documentation surrounding Neo4j 2.2.x requirements

With the release of Neo4j 2.2.2 there have been a few modifications to the initial install that require some additional tweaks from a developer to get up and going. To ensure ease of entry for new developers using neomodel we need to document these requirements and at least point the developers in the right direction. One known piece is authentication which is documented well by @nigelsmall in the py2neo authentication docs.

@robinedwards Where would you like this documentation covered? I'm thinking about adding it into the README under a heading of something like Transitioning from 1.x to 2.x but didn't know if there would be a major version change for the release when the py2neo update branch is merged into master. Or I can start a new document under the docs area but I wasn't sure if it warranted a completely new doc.

I think all this brilliant work is definitely worth a version bump, perhaps call it 2.0. I think it would be nice if it lived in docs/ so its available on readthedocs

Sweet :) and cool good point!


fix(sec): upgrade com.ning:async-http-client to 1.9.0

What happened？
There are 1 security vulnerabilities found in com.ning:async-http-client 1.8.3

CVE-2013-7398

What did I do？
Upgrade com.ning:async-http-client from 1.8.3 to 1.9.0 for vulnerability fix
What did you expect to happen？
Ideally, no insecure libs should be used.
The specification of the pull request
PR Specification from OSCS

Plugin dependencies only affect tests. There is no security impact.


Improve performance of text inputs

This makes inputs only commit changes when they loose focus, instead of on every key press. This eliminates the janky feeling when typing in huge records while on a relatively slow device.

@wojtek-krysiak should I target this PR at master or some other branch?


when i run demo.py the error occurs

when i was running my  own dataset,there is a issue exist as follow:>Restoring from checkpoint failed. This is most likely due to a mismatch between the current graph and the graph from the checkpoint. Please ensure that you have not altered the graph expected based on the checkpoint.

How did you solve this error?


Comment replies not showing up

Here is a potential bug scenario:
I have a treasury proposal and it is created via Polkadot JS portal referendum page and then the respective post is updated on both Subsquare and Polkassembly.

User posts a comment on Subsquare. The comment also shows up on Polkassembly.
Then the comment is replied via Polkassembly. This reply is showing up on Subsquare but not showing up on Polkassembly.
Currently this scenario is reproducible in referendum 700, and comment - https://polkadot.polkassembly.io/referenda/700#662724f62a748388a6f6d6d1

The problem has been successfully resolved! Thank you for your cooperation!


[DOCS-55] Layout for Cobalt Core Contributors to our Best Practices for Security guide

This PR sets up a template for Cobalt Core Contributors to our "Best Practices for Security" guide. You can review the build here: https://deploy-preview-112--cobalt-docs.netlify.app/bestpractices/
I've also included a "Readme" that describes what our contributors are doing, here: https://github.com/cobalthq/cobalt-product-public-docs/blob/DOCS-55_layout_for_Cobalt_Core_Contributors/BestPracticesSecurityGuide.md
DO NOT MERGE this PR before our license PR https://github.com/cobalthq/cobalt-product-public-docs/pull/92 is approved and merged.

Thanks for approving, @sullikat

Merging.
Note: this won't show up in our ToC until I remove toc_hide = true from the BestPractices/_index.md file.


ProcessPoolExecutor in py38

Hello,
I don't have simple reproducible example but the same tox suite running pytest --cov --cov-append reports slightly differently in py37 and py38.
I set up my .coveragerc as follows
[path]
source =
    src
    */site-packages
    *\site-packages

[run]
branch = true
parallel = true
source =
    my_package
    tests

Running in py37 correctly report the code under src/my_package and tests
In py38, i see additional lines .tox/py38/.../site-packages/my_package/*.py. The only hits for those files are actually callback code called in a ProcessPoolExecutor
I tried setting concurrency = multiprocessing but it didn't help.
As a workaround, I'm omitting these files when reporting but it's still puzzling as it only happens with 3.8 :)

It's hard to know what's going on without being able to reproduce it myself, but here are some thoughts:

Are you sure your package is installed in site-packages in both 3.7 and 3.8?
Can you run coverage with --debug=trace?  That will show what files are being considered for measurement.

Hi,
Thanks for getting back to me.
I setup a repo at https://github.com/0x0L/testcov
I'm able to reproduce it locally on my machine
tox 3.20.1
py37
---------- coverage: platform darwin, python 3.7.9-final-0 -----------
Name                    Stmts   Miss Branch BrPart     Cover   Missing
src/gruik/init.py       6      0      0      0   100.00%
tests/test_a.py             3      0      0      0   100.00%
TOTAL                       9      0      0      0   100.00%
py38
---------- coverage: platform darwin, python 3.8.3-final-0 -----------
Name                                                      Stmts   Miss Branch BrPart     Cover   Missing
.tox/py38/lib/python3.8/site-packages/gruik/init.py       6      2      0      0    66.67%   9-10
src/gruik/init.py                                         6      0      0      0   100.00%
tests/test_a.py                                               3      0      0      0   100.00%
TOTAL                                                        15      2      0      0    86.67%

It definitely looks like it but I see no mention of any change to the ProcessPool in the release notes for 3.8

@nedbat
Actually the issue is not python 3.8 specific. In the example repo https://github.com/0x0L/testcov I added py39 and changed the order of the envlist. It looks like only the first listed env gets a correct reporting (ie with the callback correctly identified).

@nedbat
Actually the issue is not python 3.8 specific. In the example repo https://github.com/0x0L/testcov I added py39 and changed the order of the envlist. It looks like only the first listed env gets a correct reporting (ie with the callback correctly identified).


Added Hex-color

Proposed changes:
Added Hex-color
Type of change:
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Other (please specify)
[ ] This change requires a documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

Additional info (if any):

@shubhanggupta2000
I'm sorry but you cannot use any libraries.


remove typings, use @types instead

Still do make a yarn to update yarn.lock

Can you push it then ? 😀


Rollback with service constraint will create two services and never finish rollback

Rancher Version: master 7/26
Steps to Reproduce:

Upgrade & rollback kubernetes

Results: kubelet_2 creates twice and one keeps stopping and starting, causing rollback to never finish.
Expected: Rollback to finish

Version - v1.2.0-pre1-rc7
Verified fixed


LT_* sorted before LS_*

Hey,
I've just discovered that in the DATA-declarations the table elements (LT_) are sorted before structures (LS_) by abap-cleaner.
Before:

After:

Why?
Is there a way to change this?

Thanks!
Regards,
Vlad

I guess it could makes sense for LIKE LINE OF?

Hi Vlad,
this is because "Rearrange local declarations" (with the option "Rearrange DATA: … order by first usage") determines the first usage of the respective variable in the code and then rearranges the DATA declarations in that order. So, from your example I would expect that the variable lt_addrnr_objnr is used at an earlier point in the following code than the variable ls_addrnr_objnr.
This is indeed different with LIKE, because then, the first usage is already within the DATA section. For example, in this code,
    DATA ls_any_struc TYPE ty_s_struc.
    DATA lt_any_table LIKE STANDARD TABLE OF ls_any_struc.

    lt_any_table = VALUE #( ).
    ls_any_struc = VALUE #( ).

the order will (and must) be kept, because ls_any_struc is already used in the DATA section (and therefore it is used before lt_any_table).
Kind regards,
Jörg-Michael

Hey @jmgrassau & @fabianlupa
you're both right, that makes sense now.


Whitelisting IPs using traefik or ufw

I am running my execution client RPC through traefik as I want it public but to only allow a whitelist of IP's to access it.
Is there a recommended way of adding a whitelist to traefik or putting UFW on the server with traefik?
Thank you.

Yes, there is. You can place ufw "in front of" docker, instructions here: https://eth-docker.net/Support/Cloud

The tutorial only states to use it if not encrypting the traffic with traefik. What is the recommended setup while using Traefik?
If I want to whitlist IPs then is it better to unencrypt the traffic and expose the ports while having ufw?

You can use it for traefik just as well. 443 in that case. It’s a more specific use case and one most users don’t have; you can absolutely do things that way.

I followed the tutorial and debugged a but with no success.
I installed and ran ufw denying all incoming and allowing all outgoing traffic.
This is my rules.
30303                      ALLOW       Anywhere
OpenSSH                    ALLOW       Anywhere
443/tcp                    DENY        Anywhere
443                        DENY        Anywhere
30303 (v6)                 ALLOW       Anywhere (v6)
OpenSSH (v6)               ALLOW       Anywhere (v6)
443/tcp (v6)               DENY        Anywhere (v6)
443 (v6)                   DENY        Anywhere (v6)
Yet, I able to run curl commands to my RPC layer.
It seems how ethd is setup restricts ufw from touching it.

As long as you followed the instructions on placing ufw "in front of" docker, you should not be able to curl to this from the Internet / a public IP.

I updated the after.rules and before.init, and I am able to block ports only when specified.  One step in the right direction :)
Keeping the 443 DENY Anywhere blocks the RPC communication, but when I do not have any rules on 443 and use sudo ufw default deny incoming I am still able to curl.
This time I made sure to read through the documentation and cannot find what I am missing.
Thanks again for your help.

Yes this is deliberate. The default deny doesn’t impact docker in this setup. The idea is that even with ufw “in front of”, stuff like P2P ports still works by default. All mapped docker ports remain reachable by default. What’s in front of is the user table, not all of ufw.
the user can then decide to close off ports, for example
allow proto tcp from  to any port 443 comment “RPC for grandma”
deny proto tcp from any to any port 443

That worked! Today I learned that the order of ufw rules matters a lot.
I thought a deny rule had priority over all others.
Thank you for your help.

Glad that worked!
Yes ufw works like every other stateful firewall in the market: Rules are evaluated in order, top down.


Question about the bounding box 3d visulization.

Hi, thanks for the great work! I want to visualize the 3d bounding box. I use the bounding box information in the 3d_bbs json files. Are bounding boxes in the lidar coord? I use lidar2img to project the boxes and got results like this.
What should I do to get the correct boxes?

I got it. The origin should be set to (0.5, 0.5, 0.0)


wigth of toolbar

From 007NOT on February 05, 2009 13:29:31
see image:
long word can't be displayed
Attachment: noname.png
Original issue: http://code.google.com/p/wwwsqldesigner/issues/detail?id=9

From ondrej.zara on February 05, 2009 04:46:38
Input buttons cannot wrap text. If you want them wider, edit the css of your
installation to suit your needs.
Status: WontFix

From 007NOT on February 05, 2009 04:59:56
ok, can you make this as setting in xml file?
can you fix next bug
see image 2
Attachment: noname.png

From ondrej.zara on February 05, 2009 06:37:05
Regarding the original issue: setting in which XML file? This is relevant to page
look/design and must be therefore set in CSS. The bar's default width is 150px, feel
free to edit this to suit your needs.
Regarding the second issue: this is not a bug, it is a feature. Field comments are
displayed next to "Edit comment" button.


feat: add form style to pagination params

Copied from #228 description
The Open Payments API doc examples prepend the pagination parameter when showing an example call to list outgoing or incoming payments:

Specifically,
curl --request GET \
     --url 'https://openpayments.guide/alice/incoming-payments?pagination=first=10&cursor=uuid' \
     --header 'accept: application/json'

// inclusion of "pagination" is the issue here

https://docs.openpayments.guide/reference/list-outgoing-payments
However, we do not use root pagination parameter when we parse the query parameters when we are handling GET /(incoming | outgoing)-payments calls in backend, we only look at first last cursor params:
https://github.com/interledger/rafiki/blob/a0302b390192f001fc4baa6bc55ae8dcfb13143a/packages/backend/src/open_payments/payment_pointer/routes.ts#L50
https://github.com/interledger/rafiki/blob/a0302b390192f001fc4baa6bc55ae8dcfb13143a/packages/backend/src/shared/pagination.ts#L9-L20
We should update Open Payments API docs to ignore pagination as the root key parameter for the example requests.
ie, it should be just
curl --request GET \
     --url 'https://openpayments.guide/alice/incoming-payments?first=10&cursor=uuid' \
     --header 'accept: application/json'

Tested on integration branch:


Progress on dropwizard applications with jrestless on aws lambda?

I saw comments on issues #27 and #34 with @injeniero and @vikrampatankar regarding running dropwizard.  I think @injeniero may have a working example, and I see a repo was created but never filled.
Has jrestless ever been used for dropwizard? If so, any hints on how to do that?
Many thanks

Hi @kennovak-zy ,
No news regarding this unfortunately. I really hoped @injeniero would share how he managed to make JRestless work with dropwizard.
So I can just repeat what has been said here: https://github.com/awslabs/aws-serverless-java-container/issues/128#issuecomment-369317745
It should be possible somehow... but I don't know for sure since I don't use dropwizard.


Introduce swiper into the encapsulated Dialog, but warn  Cannot read property '$swiper' of undefined"

  <template>
    <el-dialog
      :visible.sync="visible"
      :title="speedInfo.name"
      :before-close="closeDialog"
      width="40%"
      class="dataScreenDialog cff speedInfoDialog"
    >

{{speedInfo.name}}

{{speedInfo.time}}

        <swiper class="swiper gallery-top" :options="swiperOptionTop" ref="swiperTop" >
           <swiper-slide v-for="item in speedInfo.image" :key='item.index'>      </swiper-slide>

        </swiper>

        <swiper class="swiper gallery-thumbs" :options="swiperOptionThumbs" ref="swiperThumbs">
           <swiper-slide v-for="item in speedInfo.image" :key='item.index'></swiper-slide>
        </swiper>

    </el-dialog>
  </template>


Method for URL-safe project serialization

It's not as easy as you'd think to serialize the state of a project for e.g. sticking in a URL, because btoa throws on non-Latin-1 characters, so special handling there is required. Also, base64url is a better scheme (- and _ instead of + and /), because it's less likely to get mis-encoded somehow (especially the + which will become a space).
Let's add something like serialize() and deserialize() to project and ide, to make this use case simpler.

how come btoa throws on non-latin characters? isn't it supposed to handle general bytes? therefore encoding anything, whatever the encoding?

how come btoa throws on non-latin characters? isn't it supposed to handle general bytes? therefore encoding anything, whatever the encoding?

btoa does handle general bytes, but it's not smart enough to know how to encode a multi-byte character if it encounters one while iterating over a JS string (since JS strings are variable length UTF-16). When it gets to a multi-byte UTF-16 character, it sees it as a number that is too large to fit in a byte, and errors. So, the easiest solution is to convert the string to UTF-8 first.

how come btoa throws on non-latin characters? isn't it supposed to handle general bytes? therefore encoding anything, whatever the encoding?

btoa does handle general bytes, but it's not smart enough to know how to encode a multi-byte character if it encounters one while iterating over a JS string (since JS strings are variable length UTF-16). When it gets to a multi-byte UTF-16 character, it sees it as a number that is too large to fit in a byte, and errors. So, the easiest solution is to convert the string to UTF-8 first.

https://developer.mozilla.org/en-US/docs/Glossary/Base64#the_unicode_problem

Thanks for the details! I feel there should be a warning somewhere in the API since this is so common in browser code! I'd have expected any string given to btoa to be converted to a Uint8Array before being converted to base64 to avoid any encoding issue...

BTW I see many "solutions" on the web mentioning the use of a TextEncoder to convert everything to UTF-8 before doing the base64 conversion, why isn't the version chosen for https://github.com/lit/lit.dev/blob/fd4c34e71b47267f3672a2debe52807042f22cc2/packages/lit-dev-content/src/pages/playground.ts#L31 ?

BTW I see many "solutions" on the web mentioning the use of a TextEncoder to convert everything to UTF-8 before doing the base64 conversion, why isn't the version chosen for https://github.com/lit/lit.dev/blob/fd4c34e71b47267f3672a2debe52807042f22cc2/packages/lit-dev-content/src/pages/playground.ts#L31 ?

I don't know, we should probably use it! I thought maybe TextEncoder wasn't available in all browsers at the time it was written, but that doesn't seem to be true. 🤷


[Updated PR#1854] Added aggregate-sample function to generate random result from query

This PR was recreated from #1854
The sample() function is used to generate random result based on the given size. It can be used together with eloquent functions like ```where()`` and such.
Co-Authored-By: @adlanarifzr

Do we really need to add this feature?

While it does look like a neat feature, I also think it's really important to add it to the library. My idea of this library is to make it super easy for Laravel developers to switch between SQL/Mongo database implementations without having to change too much of their code. This sample method feels very Mongo oriented. But then again, there are already quite some Mongo specific features in the library right now.

There are some of issues asking for randomness.
https://github.com/jenssegers/laravel-mongodb/issues/107
https://github.com/jenssegers/laravel-mongodb/issues/1493
For now the only way to get random data is to use collection. In Laravel there is a method to called inRandomOrder  see here to achieve that without using the collection. In my opinion If this package has to be "super easy for Laravel developers", it is not possible to run a raw query to get random data from the database.

There are some of issues asking for randomness.
#107
#1493
For now the only way to get random data is to use collection. In Laravel there is a method to called inRandomOrder see here to achieve that without using the collection. In my opinion If this package has to be "super easy for Laravel developers", it is not possible to run a raw query to get random data from the database.

My suggestion would be to add this function in the next version when we refactor a little bit of pipeline, not this maintenance release which is almost fixing everything.
Regarding #107, #1493 we have important issues such not working relations, enchantments could be worked on when everythings works as expected at least.
What do you think about this @Smolevich ?

After internal discussion we have decided this is not the right time to add this new feature in the current release.

I thank this feature we are need.
better than sample() to inRandomOrder()  can support both SQL/Mongo it better.


[ovn] Chassis as external port host edpm config

This patch allows for configuring a Chassis as a host for external ports. This feature was introduced in ML2/OVN by https://review.opendev.org/c/openstack/neutron/+/894767.
This patch creates a new configuration for edpm_ovn called "edpm_enable_chassis_extport" which when enabled sets the "enable-chassis-as-extport-host" to the OVN CMS Options.

/ok-to-test

@umago going forward, please join this org and openshift org on github so that CI is triggered for your PRs automatically. Thanks!

recheck

recheck


Separate tvOS and iOS Realm Frameworks in folders in the project

:tophat: What? Why?
Separating tvOS and iOS frameworks at a project level
:family: Dependencies?
Head of the 🚂  is https://github.com/redbooth/RealmResultsController/pull/75
:clipboard: Developer checklist

[x] Documentation
[x] Unit Tests
[x] Update README (if necessary)

:ghost: GIF

👍

👍


Add Scaling workload

DNM yet, Thanks

Ok This is ready.  It includes a var for pass/fail criteria (EXPECTED_SCALE_DURATION) . I would like to use this model in conjunction with the scraper and comparison tooling for all future pass/fail test criteria.

@jmencak @chaitanyaenr PTAL


Added drawer to all screens

https://user-images.githubusercontent.com/31162981/149653211-65bb764f-8293-4421-9868-e669de604466.mp4

Imo, I think we should separate the header and drawerslide so that not all the pages are bind to the drawer menu. What do you think?

The drawer may need to be shown or hide depends on the page.
The Header component might need more logics in it.
Example, we can have like a way to update the header title depending on a page.


how to trouble shoot 'AI: Process was called on the TelemetrySink after it was disposed, the telemetry data was dropped.'?

List of NuGet packages and version that you are using:
Runtime version (e.g. net461, net48, netcoreapp2.1, netcoreapp3.1, etc. You can find this information from the *.csproj file):
Hosting environment (e.g. Azure Web App, App Service on Linux, Windows, Ubuntu, etc.):

Microsoft_ApplicationInsights_AspNetCore_Version: 2.18.0
netcoreapp3.1 & Windows
What are you trying to achieve?
We are seeing 'AI: Process was called on the TelemetrySink after it was disposed, the telemetry data was dropped.' what does it mean and how could we trouble shoot it? Thanks~
We need some hint about why this would happen
What have you tried so far?
Search the internet but nothing helpful like https://github.com/search?q=Process+was+called+on+the+TelemetrySink+after+it+was+disposed%2C+the+telemetry+data+was+dropped.&type=code

For context, this error is thrown from here:
https://github.com/microsoft/ApplicationInsights-dotnet/blob/288d34438bf5f4f2a893363c94a6b62e09f58505/BASE/src/Microsoft.ApplicationInsights/Extensibility/TelemetrySink.cs#L205-L211
https://github.com/microsoft/ApplicationInsights-dotnet/blob/288d34438bf5f4f2a893363c94a6b62e09f58505/BASE/src/Microsoft.ApplicationInsights/Extensibility/Implementation/Tracing/CoreEventSource.cs#L533-L541
The TelemetrySink is a member of the TelemetryConfiguration and would only be disposed if the TelemetryConfiguration was disposed.
You should review your code and confirm that you're only creating one instance of TelemetryClient and TelemetryConfiguration for the entire lifetime of your application.

It worked, thanks! We replace the singleton in AddApplicationInsightsTelemetry

@xieofxie do you have an example of replacing the singleton in that call? I have exactly the same issue.

Same problem here, I'm curious what your fix was exactly @xieofxie

@isaacabraham @ThunderDev1 Here is what I did:
        // https://github.com/microsoft/ApplicationInsights-dotnet/blob/main/NETCORE/src/Microsoft.ApplicationInsights.AspNetCore/Implementation/TelemetryConfigurationOptions.cs
        // the above one creates a configuration so we should not use it
        private class TelemetryConfigurationOption : IOptions<TelemetryConfiguration>
        {
            public TelemetryConfiguration Value => TelemetryConfiguration.Active;

            public TelemetryConfigurationOption(IEnumerable<IConfigureOptions<TelemetryConfiguration>> configureOptions)
            {
                var configureOptionsArray = configureOptions.ToArray();
                foreach (var c in configureOptionsArray)
                {
                    c.Configure(this.Value);
                }
            }
        }

        public static void Configure(IServiceCollection services, bool isWebJob)
        {
            services.AddApplicationInsightsTelemetry();
           // after the above, replace the TelemetryClient & IOptions<TelemetryConfiguration>
            var descriptor = services.Single(descriptor => descriptor.Lifetime == ServiceLifetime.Singleton && descriptor.ServiceType == TelemetryClient.GetType());
            services.Remove(descriptor);
            services.AddSingleton(YourTelemetryClient);

            descriptor = services.Single(descriptor => descriptor.Lifetime == ServiceLifetime.Singleton && descriptor.ServiceType == typeof(IOptions<TelemetryConfiguration>));
            services.Remove(descriptor);
            services.AddSingleton<IOptions<TelemetryConfiguration>, TelemetryConfigurationOption>();
            ...

^ Looks like some hack. Won't recommend doing this.

Our setup looks like this:
`
var enableAppInsights = Configuration.GetValue("ApplicationInsights:Enable", false);
//Add Application Insights logging & telemetry, and the Kubernetes Enricher
if (enableAppInsights)
{
//Application Insights Logging
services.AddSingleton(typeof (ITelemetryChannel), new ServerTelemetryChannel());
services.AddApplicationInsightsTelemetry(Configuration);
services.AddCloudRoleNameInitializer($"{APIServiceName} API");
//Add Profiler to initialize trace data to Application Insights
services.AddServiceProfiler(Configuration);

services.AddApplicationInsightsKubernetesEnricher();

}
`
We never instantiate the telemetry client or the options or any other part of AI in our code and we still have this problem.

If the "hack" posted by @xieofxie works, then that would be indicative of a bug somewhere that the hack is working around.

For context, this error is thrown from here:
https://github.com/microsoft/ApplicationInsights-dotnet/blob/288d34438bf5f4f2a893363c94a6b62e09f58505/BASE/src/Microsoft.ApplicationInsights/Extensibility/TelemetrySink.cs#L205-L211
https://github.com/microsoft/ApplicationInsights-dotnet/blob/288d34438bf5f4f2a893363c94a6b62e09f58505/BASE/src/Microsoft.ApplicationInsights/Extensibility/Implementation/Tracing/CoreEventSource.cs#L533-L541
The TelemetrySink is a member of the TelemetryConfiguration and would only be disposed if the TelemetryConfiguration was disposed.
You should review your code and confirm that you're only creating one instance of TelemetryClient and TelemetryConfiguration for the entire lifetime of your application.

This doesn't help us because we never create or request or inject or otherwise use TelemetryClient and TelemetryConfiguration in our code at all.

Our configuration looks like this:
"ApplicationInsights": {
  "Enable": true,
  "LogMessageFormat": "${message} ${exception:format=ToString,Data}",
  "InstrumentationKey": "<guid>",
  "EnableRequestTrackingTelemetryModule": true,
  "EnableAdaptiveSampling": true,
  "EnableActiveTelemetryConfigurationSetup": true
}

Dears, any update of this issue ?
We're working with .NET 8, and we're facing this issue at out WebAPI projects and Azure functions projects as well.
For WebAPI Projects, we're using this :
services.AddApplicationInsightsTelemetry(options => options.ConnectionString = applicationInsightsConnectionString);
for Azure functions, we're using this :
services.AddApplicationInsightsTelemetryWorkerService(options => options.ConnectionString = applicationInsightsConnectionString);
services.ConfigureFunctionsApplicationInsights();


Corrector optimization

Corrector is currently using multiprocessing only for raw data processing, and many tasks are preformed in a single main thread that can use only one CPU core and therefore negatively impact corrector throughput.
This pull request optimizes corrector to increase throughput. Additionally a check if processed document already exists in clean_data is added (this check existed before last corrector rewrite, but was broken since that time).

Performing sanitise_document and correct_structure in worker threads instead of main thread.
Deleting duplicates in threads instead of returning to_remove_queue and slowly processing that in main thread.
Adding check if processed document already exists in clean_data (was broken after "matching of documents by xRequestId" commit).
Using multiprocessing for updating status of orphans that reached timeout.
Using multiprocessing for processing of documents without xRequestId.
Optimizing document sanitizer (using translate method instead of slow and deprecated bleach.clean).
Fixing calculation of total documents processed by corrector.

Hello @VitaliStupin!
Thank you for the pull request, we will review it as soon as possible.


Styled Components v6 types breaks react-docgen-typescript prop inference

Styled Components v6 now have their own type definitions, unfortunately it breaks props inference through react-docgen-typescript.
I've posted this also to the react-docgen-typescript repository however I'm not sure where is the right place, I'll be happy to look in to it more if someone can point me to the right direction.
v6
System:

OS: macOS 12.5
CPU: (10) arm64 Apple M1 Pro
Memory: 108.22 MB / 32.00 GB
Shell: 5.8.1 - /bin/zsh

Binaries:

Node: 16.16.0 - ~/.nvm/versions/node/v16.16.0/bin/node
Yarn: 1.22.19 - ~/.nvm/versions/node/v16.16.0/bin/yarn
npm: 8.11.0 - ~/.nvm/versions/node/v16.16.0/bin/npm

npmPackages:

styled-components: ^6.0.0-beta.2 => 6.0.0-beta.2

v5
System:

OS: macOS 12.5
CPU: (10) arm64 Apple M1 Pro
Memory: 99.06 MB / 32.00 GB
Shell: 5.8.1 - /bin/zsh

Binaries:

Node: 16.16.0 - ~/.nvm/versions/node/v16.16.0/bin/node
Yarn: 1.22.19 - ~/.nvm/versions/node/v16.16.0/bin/yarn
npm: 8.11.0 - ~/.nvm/versions/node/v16.16.0/bin/npm

npmPackages:

styled-components: ^5.3.5 => 5.3.5

Reproduction
Here is a repository with v5 - https://github.com/DavidReinberger/storybook-typescript-react-doc-gen-styled-components-5, where it works flawless (even with default exports).
Here is a repository with v6 - https://github.com/DavidReinberger/storybook-typescript-react-doc-gen-styled-components-6
In both repositories, run npm install && npm run storybook
Steps to reproduce

Open ButtonStyled story
Only as and forwardedAs props are visible3.

Expected Behavior
All props are inferred as with version 5.
Actual Behavior
No props are inferred.

Fixed by #3815

It looks like there was a regression and this doesn't work anymore in 6.1.1
Here's the updated repo, steps to reproduce are the same as in initial post.
https://github.com/macko911/storybook-typescript-react-doc-gen-styled-components-6


Command shortcuts are dead on MacOS

In MacOS some of the key shortcuts/symbolds are not working. For me Commands shortcuts are dead. Only the settings cmd shortcut works.
MacOS Monterey v12.4
Macbook Air 2018

Do you know what key layout you're using? There's some existing bugs for German/Spanish key layouts on MacOS.

Hungarian layout, but I use karabiner layout changer because of im new macos user. I tried without it but same problem.

Duplicate of #1094


Question: support ZodDate type

Hi,
I'm using a preprocessor to cast ISO 8601 date string query params to Date objects.
Example code: /api/v1/something?fromDate=2022-10-12T06%3A44%3A10Z
My query definition is something like this:
z.object({
  fromDate: z
    .preprocess((arg) => {
      if (typeof arg === 'string') {
        const date = new Date(arg);
        if (isValid(date)) {
          return date;
        }
      }
      return arg;
    }, z.date())
    .optional()
    .openapi({
      type: 'string',
      example: new Date(),
      param: { description: 'From date filter, expects an ISO 8601 date string' },
    }),
})

It would be nice if the library would automatically handle this type properly so I wouldn't have to set the type field manually. The same also applies to request bodies that could contain serialized date strings.
If this is something you want to support I could contribute a PR, just let me know.

Swagger UI renders the above example as

Hey @heivo , that's definitely something we want to support. It's essentially supporting the ZodDate type. Would be great if you can open a PR for that. Some pointers:

You can add a test in spec/types/date.spec.ts similar to the other tests, you can run them with npm run test
The main function that infers types is OpenAPIGenerator#toOpenAPISchema, you can add a check for ZodDate there. You will need to add the type to the isZodType function definition as well
Make sure to run npm run prettier before submitting the PR - that will format the code according to the repo rules

Let me know if I can assist with anything!

Fixed in #51


Fix photo name to albertsutz.png

Fix photo name to enable tracking in tP progress dashboard

LGTM!


[Brainstorm] What kind of data type conversions and other utilities do we want in the cli tool?

Currently:

Address to ScriptHash

What else? :)

Here are a few utilities that might be useful based on questions I've seen come up in Discord:

Script hash to address can also be useful (reverse of above)

base58.b58encode_check(b'\x17' + val)

Byte array string to int conversion:

int.from_bytes(val, "little")

Hex string to int conversion:

int.from_bytes(bytearray.fromhex(val),"little")

Hex encode byte array string

binascii.hexlify(val)

Great suggestions!
Been playing with it, but how do you pass b'#\xba\'\x03\xc52c\xe8\xd6\xe5"\xdc2 39\xdc\xd8\xee\xe9' as cli argument? With the quotes and double-quotes it's hard.
All this is almost better suited for a web-frontend perhaps?

Agreed that a web front-end would probably be better. It would likely solve the CLI parameter issue you are describing.
To pass the parameter, using something like the Prompt parser, I think you can do bytearray(b'#\xba\'\x03\xc52c\xe8\xd6\xe5"\xdc2 39\xdc\xd8\xee\xe9').


Move documentation build scripts and fix link to github tests

Found the link on the github testing badge was broken so fixed that in b1ff04a

Decided to knock out https://github.com/agentos-project/agentos/issues/48 while I was in there (although, I'll hold off on fixing "good first issues" in the future)

comments addressed and tests passing.  merging!

comments addressed and tests passing.  merging!


New release?

Having https://github.com/JunoLab/atom-latex-completions/pull/22 on the release version would be fantastic - it would be nice to put \Theta vs. \theta to rest once and for all.

Done.


Missing CONTRIBUTING File

Hello,
i cant find a Contributing file, like it is provided for some other Repositories. e.g. scancode-toolkit Contributing File
Could you please add one?
Thanks!

Hi @blaemmle
Thank you for the report. The contributing file with some additional docs are on the way. They will be reflected here soon after we set up RTD (https://github.com/nexB/vulnerablecode/pull/527)

@blaemmle Good catch! I reckon you may be willing to contribute then?

Hi @pombredanne
Probably not by myself. But i like to enable collegues to contribute and therefore i need the documentation to enable the process.

Like, can we simply reuse https://github.com/nexB/scancode-toolkit/blob/develop/CONTRIBUTING.rst?


Update formatter.rb

Changes lines.each to each_line because the former is deprecated as of Ruby 2.0

+1

I will review this in the evening to ensure there are no issues. Good catch.

Since there was delay on merging I pulled down locally and double checked on a couple versions. Everything looks good. Thanks!


CASMCMS-8742 - IMS remote build node configuration

This is a manual back-port for:
https://github.com/Cray-HPE/docs-csm/pull/4910

[X] If I added any command snippets, the steps they belong to follow the prompt conventions (see [example][1]).
[X] If I added a new directory, I also updated .github/CODEOWNERS with the corresponding team in [Cray-HPE][2].
[X] My commits or Pull-Request Title contain my JIRA information, or I do not have a JIRA.

The shell check errors are unrelated to the content of this PR. They should not hold up the merge.


Add quickstart script for macos

I thought that a script with all the commands needed to get started on macos would be useful. We should probably link to it in the docs.

jup, looks good


docs(iroh-net): Update discovery and dialing docs, signatures

Description
This updates the documentationfor the dialing and discovery modules.
It also fixes up some signatures to be more consistent with the rest
of iroh-net:

NodeId is used to identify an iroh-net node that is communicated
with.  PublicKey is only used in contexts where
encryption/decryption happens

NodeId is Copy and we pass it by value normally.  Not by reference.

Some more changes:

DialFuture is entirely unused, probably left over from a refactor.
Removed.

Use pub(super) explicitly, helps clarity of intention as well as
dead-code detection.

Breaking Changes
The following APIs now take NodeId by value rather than by reference:

iroh_blobs::downloader::Dialer::is_pending
iroh_net::dialer::Dialer::abort_dial
iroh_net::dialer::Dialer::is_pending
iroh_net::Endpoint::connect_by_node_id

Other changes:

iroh_blobs::downloader::ProviderMap::get_candidates yields NodeId rather than &NodeId

Notes & open questions

Change checklist

[x] Self-review.
[x] Documentation updates following the style guide, if relevant.
[ ] Tests if relevant.
[x] All breaking changes documented.

I really need to work on my commas... thanks!


.

it said windows  protected ur pc  when i tried  to launch

Windows verification costs hundreds of dollars per year. You would find this issue with many free and open source projects. If you want to check if this project is a virus, review the code yourself and compile a windows build yourself, or upload the binaries to virustotal.


Initializing distributed halts for a long time and then timeout

Running the training script with sh run _128_b.sh
Below is the log:

loaded pretrained LPIPS loss from taming/modules/autoencoder/lpips/vgg.pth
VQLPIPSWithDiscriminator running with hinge loss.
loaded pretrained LPIPS loss from taming/modules/autoencoder/lpips/vgg.pth
loaded pretrained LPIPS loss from taming/modules/autoencoder/lpips/vgg.pth
VQLPIPSWithDiscriminator running with hinge loss.
loaded pretrained LPIPS loss from taming/modules/autoencoder/lpips/vgg.pth
VQLPIPSWithDiscriminator running with hinge loss.
loaded pretrained LPIPS loss from taming/modules/autoencoder/lpips/vgg.pth
VQLPIPSWithDiscriminator running with hinge loss.
VQLPIPSWithDiscriminator running with hinge loss.
loaded pretrained LPIPS loss from taming/modules/autoencoder/lpips/vgg.pth
VQLPIPSWithDiscriminator running with hinge loss.
loaded pretrained LPIPS loss from taming/modules/autoencoder/lpips/vgg.pth
VQLPIPSWithDiscriminator running with hinge loss.
[rank: 6] Seed set to 0
Initializing distributed: GLOBAL_RANK: 6, MEMBER: 7/32
[rank: 4] Seed set to 0
Initializing distributed: GLOBAL_RANK: 4, MEMBER: 5/32
[rank: 5] Seed set to 0
Initializing distributed: GLOBAL_RANK: 5, MEMBER: 6/32
[rank: 3] Seed set to 0
[rank: 1] Seed set to 0
Initializing distributed: GLOBAL_RANK: 3, MEMBER: 4/32
Initializing distributed: GLOBAL_RANK: 1, MEMBER: 2/32
[rank: 7] Seed set to 0
Initializing distributed: GLOBAL_RANK: 7, MEMBER: 8/32
[rank: 2] Seed set to 0
Initializing distributed: GLOBAL_RANK: 2, MEMBER: 3/32

Seems the code has no problem finding my GPUs
GPU available: True (cuda), used: True
TPU available: False, using: 0 TPU cores
IPU available: False, using: 0 IPUs
HPU available: False, using: 0 HPUs
[rank: 0] Seed set to 0
Initializing distributed: GLOBAL_RANK: 0, MEMBER: 1/32
[rank: 3] Seed set to 0
[rank: 7] Seed set to 0
[rank: 6] Seed set to 0
[rank: 4] Seed set to 0
[rank: 1] Seed set to 0
[rank: 5] Seed set to 0
[rank: 2] Seed set to 0

Anyone encountered the error before?

In case someone has the problem, change the lines here to match your machine environment: https://github.com/TencentARC/Open-MAGVIT2/blob/main/configs/imagenet_lfqgan_128_B.yaml#L5-L6


Searching is not working in the Completed tab of Shipments, Returns, or Purchase Orders

Current behavior
No results are displayed, and the search appears non-functional.
Expected behavior
The search functionality should return relevant results based on the search query entered.
Steps to reproduce the issue
Navigate to the relevant module (Shipments, Returns, or Purchase Orders).
Click on the Completed tab.
Use the search bar to enter a valid keyword or identifier.
Observe the results.
Environment details
App version : v2.28.0
App environment : UAT
Additional information
https://jam.dev/c/4eb15705-b64c-43f5-af6d-eb9b4b87b6b5

Issue verified and working fine in release v2.28.1. Hence, closing this ticket.


Providing a redis client instead of a url

Currently, the redis adapter should be given a url of redis or a backend
I couldn't figure it out from the code - is it possible to supply a redis client to be used by moneta?
We currently have a redis setup which includes namespacing, sentinels, instrumentation, connection pools, etc and it would be nice to have moneta use that instead of a bare Redis.new client
Is that possible, to supply an object that would bahve as a redis client?

Hi there, yes this is possible.  To do it, you use the :backend option.  For example:
my_store = Moneta.build do
  adapter :Redis, :backend => Redis.new
end

Does that answer this for you?

.... or even just:
my_store = Moneta.new :Redis, :backend => Redis.new

Thanks!
Will open a new issue if issues arise

Am I using this correctly?
Doing
my_store = Moneta.new :Redis, :backend => Redis.new
my_store.get 'test'

gives me
NoMethodError: undefined method `get' for #<Moneta::Transformer::MarshalPrefixKeyMarshalValue:0x007ff7ce6d86c0>


java.lang.ClassException occurred when creating edges on Graph Editor

OrientDB Version: v3.0.23
Java Version: official Docker image
OS: official Docker image on Ubuntu 18.04
Expected behavior
An edge can successfully be created on the Graph Editor without any error when with the command:
CREATE edge FriendOf FROM #21:1 TO #22:0

Actual behavior
The following error occurred in a red box:
java.lang.ClassCastException: com.orientechnologies.orient.core.record.impl.ODocument cannot be cast to com.orientechnologies.orient.core.record.OEdge

The created edge cannot be shown on the graph after the execution of the above command.
However, when the nodes are selected with:
SELECT * FROM V LIMIT 10;

The edge has actually be created and shown on the graph.
Steps to reproduce
The error can easily be repeated as above.

Hi @tomlee
I managed to reproduce and fix the problem. I'm pushing the fix now
Thanks
Luigi

Thanks.


Switch Keras backend to Numpy

Hello!
Can i switch Numpy for Keras backend?
because i need learn numpy backend without tf, theano, or cntk.

From keras docs:

At this time, Keras has three backend implementations available: the TensorFlow backend, the Theano backend, and the CNTK backend.
TensorFlow is an open-source symbolic tensor manipulation framework developed by Google.
Theano is an open-source symbolic tensor manipulation framework developed by LISA Lab at Université de Montréal.
CNTK is an open-source toolkit for deep learning developed by Microsoft.
In the future, we are likely to add more backend options.

Oke thank's


Question: Unnecessary equality checks on UnsafeBuffer.wrap(...) methods?

This is not actually an issue. I was studying the code (masterful work, thanks for keeping it open) and following lines from UnsafeBuffer.wrap(...) methods piqued my curiosity:
        if (buffer != byteBuffer)
        {
            byteBuffer = buffer;
        }

        if (buffer != byteArray)
        {
            byteArray = buffer;
        }

What is the rationale for equality checks before assigning? This always happens in wrap(...) methods but I couldn't figure out why one would deliberately do that. Is that some visibility trick for Unsafe publication semantics, perhaps?

Turns out there is a good reason, I should have checked git blame first. Sorry.
https://github.com/real-logic/agrona/commit/532f97e31e47045983b528e700258b9d17b591e1