triton server CANNOT work with nemo under python3.10, make it impossible to train a ppo using reward/critic server

nemo and transfomer_engine both require python>=3.10，I trained reward model and all things went well. But when I tried to server reward model to train ppo, It is reported only python3.8 is allowed to use triton server. I was kind of stucked here and it makes me wonder how do you train the whole pipeline safe and sound? please help.
Traceback (most recent call last):
  File "/checkpoint/binary/train_package/serve_reward_model.py", line 86, in main
    triton.serve()
  File "/root/.local/lib/python3.10/site-packages/pytriton/triton.py", line 443, in serve
    self.run()
  File "/root/.local/lib/python3.10/site-packages/pytriton/triton.py", line 414, in run
    self._triton_server.start()
  File "/root/.local/lib/python3.10/site-packages/pytriton/server/triton_server.py", line 151, in start
    env = self._get_env()
  File "/root/.local/lib/python3.10/site-packages/pytriton/server/triton_server.py", line 301, in _get_env
    env_path = get_triton_python_backend_python_env()
  File "/root/.local/lib/python3.10/site-packages/pytriton/server/triton_server.py", line 76, in get_triton_python_backend_python_env
    raise RuntimeError(
RuntimeError: venv for python backend not found at /root/.cache/pytriton/python_backend_interpreter. Please run pytriton in python 3.8 environment to create the venv. Refer to https://github.com/triton-inference-server/pytriton/blob/main/docs/installation.md for more details.

hi! i've tested building this container and then running RLHF on it: https://github.com/NVIDIA/NeMo-Aligner/blob/main/Dockerfile is it crashing with this container?

Can you confirm the output of
python -c "import pytriton; print(pytriton.__version__)"

in your container?


Moves bar and arc opacity styling to CSS; supersedes #1069.

See original PR at #1069.
Original PR was tested and RTBC, and it doesn't seem like anything's really changed since then, but might be worth testing briefly to ensure it still works.

I didn't believe I would have permission to approve this, but I was wrong.

Released as v0.4.15.


i18n(pt-BR): Add integrations-reference.md PT-BR translation

What kind of changes does this PR include?

[ ] Minor content fixes (broken links, typos, etc.)
[ ] New or updated content
[x] Translated content
[ ] Changes to the docs site code
[ ] Something else!

Description
PT-BR translation for integrations-reference.md.

A few pages left 🚀 ! Well done!

Thanks for the review, @victor0x16!

Thanks folks!


Move cli to separate package to remove bin dependencies?

Just wanted to see what the demand is for removing the bin dependencies. This hasn't come up yet, so I don't see it as a big issue. However, I do want to do some cleanup here, so curious if I should do this sooner than later.
If sooner, I'll have to break the cli out into a separate crate. I'd prefer not to do this, just because it's extra maintenance overhead.
If later, I can work on https://github.com/rust-lang/cargo/issues/1982, which would allow binary-specific dependencies in cargo.
Feedback welcome.

I figured that nobody really cares about the cli bin. So, I'm moving it to examples, if anybody wants it back I could publish it as a separate crate.

closed by 2bc3d8ef261aedddaf75d95ba7307458bc5cc4ab removing cli from bin


Multiple concurrent invocations of the attest command can result in lost attestations

Running cosign attest <image> (almost) concurrently can have the side effect that attestations written to the container registry previously are overridden by later invocations:

cosign attest no 1 reads existing attestations
cosign attest no 2 reads existing attestations
cosign attest no 1 writes attestations with new attestation1
cosign attest no 2 writes attestations with new attestation2 and the effect that attestation1 is lost

This proposed change to the OCI distribution spec would help: https://github.com/opencontainers/distribution-spec/issues/250
We could hack around this with some terrible locking implemented over OCI. @imjasonh @jdolitsky any precedent for that in OCI?

That OCI spec change was proposed more or less to solve exactly this issue, for cosign specifically. The same can happen if two process are cosign signing the same image.
I think we might be close to having OCI specify the recommendation of using ETags for this, but it won't necessarily be enforced by all registries. The concern in OCI has been that some legacy registries use backing storage that's fundamentally eventually consistent (e.g., old AWS S3 buckets), so some registries may not be able to enforce an ETag. So folks will want a way to know whether their ETag was honored or not, so clients like cosign can fail if a race condition can't be ruled out.
In any case I'd like OCI and registries to be responsible for solving this in an HTTP-standard way like ETags, rather than bolt on locking into cosign and any other tools.

I just want to double-check my understanding as a consumer here:
Is the current state?:

This race-condition is real, but ETag support can't be universally supported by registries so solving this concurrency issue generically isn't feasible.
Instead, I should look into the new Referrers API in OCI 1.1, and just avoid this concurrency issue entirely by not trying to write to the same tag while uploading attestations? (noting that cosign doesn't currently support that for attestations, only deprecated SBOMs, but it likely will fix that soon)


Learning how to branch

Created a new branch to complete the school assignment

@ShelbyDiamond
Excellent
• Student forked repository, added her PM as a Collaborator, created a Pull Request and added PM as a Reviewer on her Pull Request.
• Student researched how to setup SSH keys with Github, so that she does not need to input her username/password each time you push.
• Student researched how GPG key is a better option for her.
• Student shared her research resources with PM group.
Requested Improvements
** No room for improvements on the MVP. Stretch goals completed.
Questions
If you have any questions over this assignment, feel free to ask them during your 1:1 meeting along with questions for next week objectives.
Rating: {1-3}
3


DOC: update docs to use new sort_values

Follow-up for #10726.
There, the specific section on sorting in basics.rst is adapted, but the rest of the docs should be updated as well.

I'll try to get this done today!

Just one question left:

api.rst

l. 441: should the ref to sortlevel be removed from the autodoc?


add reconstruct-historic-states cl cmd and MIN_VALIDATOR_WITHDRAWABILITY_DELAY config

Add the --reconstruct-historic-states command to start cl to facilitate querying old epoch info;
Adding configuration information such as MIN_VALIDATOR_WITHDRAWABILITY_DELAY and MAX_SEED_LOOKAHEAD can extend the validator exit time > the synchronization committee time and avoid continuing to generate income in the withdrawal_done state.

If the second point is adopted, please tell me the latest modified branch

you can add extra cl flags to your start up argument by adding it to participant.beacon_extra_params array.

E.g
participants:
  - el_client_type: geth
    el_client_image: ethpandaops/geth:verkle-costs-activation-switch-0f04aba
    cl_client_type: lighthouse
    cl_client_image: ethpandaops/lighthouse:verkle-trees-capella-2ffb8a9
    beacon_extra_params array:
       - "--reconstruct-historic-states"
  - el_client_type: geth
    el_client_image: ethpandaops/geth:verkle-costs-activation-switch-0f04aba
    cl_client_type: lodestar
    cl_client_image: ethpandaops/lodestar:g11tech-verge-815364b
network_params:
  electra_fork_epoch: 1
  network: holesky-shadowfork-verkle
additional_services:
  - dora

Once https://github.com/ethpandaops/ethereum-genesis-generator/pull/94 merged, I'll open a new PR where MIN_VALIDATOR_WITHDRAWABILITY_DELAY can be configured. MAX_SEED_LOOKAHEAD is a preset, and can't be adjusted by the config parameters.


Retain probe values

Just a proposal
When I have to restart HA I loose the values of the probes. Would it be possible to change the Arduino code to have these values retained. Where and how, please
And thank you for a great project

I made some updates to make this configurable via changing line 42 in the config.h file to true.
#define MQTT_RETAIN_TEMP         false //If you want the temperature sensor value to be retained in MQTT. (default: false)
Can you go ahead and switch to the retainTemp branch, set MQTT_RETAIN_TEMP to true and let me know if everything works as expected.
Thanks

Dis as instructed (I think). But no luck. After restart of HA the probe entities are listed as unavailable. Battery percent "survives" the restart.

As you can see from debug information from HA MQTT, the battery_level (the first one) has the Retained flag set whereas the prove 1 sensor (the second one) hasn't got that flag set
Received 17.10.49
QoS: 0, Retained
Payload: 100
Received 17.17.54
QoS: 0
Payload: 100
Received 17.39.08
QoS: 0
Payload: 23
Received 17.39.08
QoS: 0
Payload: 22

When I made the update i forgot i also needed to make a small change in the mqttAnnounce portion of the code as well. I just made the changes and validated on my HA instance that the values are now retained.
Can you go ahead and do a git pull to grab the latest code then one again make the change on line 42 in the config.h and let me know if it's now working as expected for you.

Yes, now it works
Thank you very much for your effort and speed (not the least)
I think it's an improvement of a very useful integration. Especially in the summer here in Denmark when grill-evenings are a great pleasure.
Restarting HA may be less often - I hope, but right now it's necessary when upgrading etc.

I just pushed the changes out to the main branch and updated the README documentation. Thanks and Happy Grilling!


Progress bar is not responsive (or not respecting the limits) [Migrated]

Since the original repository has been archived, I can't post a comment to the original issue (https://github.com/daxartio/termynal/issues/15) so i continue here.

I fixed it in 0.10.1

It's not fixed for me with 0.10.2:
termynal_override.webm

Hi!
I added horizontal scroll only. It seems enough, isn't it?

But i agree that for this case will be better to have 100% without scroll
The problem is that it needs to be done dynamically using JS code because the progress contains letters that need to be removed or added

@daxartio == @axtarov?

The problem is that it needs to be done dynamically using JS code because the progress contains letters that need to be removed or added

Hmmm ok. Seems tricky indeed.

@daxartio == @axtarov?

Yeah. Sorry about that. My account was banned

I'm here


clustermesh: fix remote service deletion on endpointslicesync

This fixes remote service deletion in endpointslicesync (see second commit).
And it also fixes tests by using EventuallyWithT instead of waiting for the controller queue to be empty. The queue being empty does not signify that the reconciliation is done as the controller pops elements when the reconciliation starts and not when it ends.
cc @giorio94
Fixes #32931

Thanks for the feedback and finding the issue + the root cause of the test @giorio94 :pray:.
I unfortunately had to remove one of the test involving a remote service and no local service because I cannot really create an endpointslice locally and check that it's removed because this deletion is made through the normal ownerreferences.

/test

I unfortunately had to remove one of the test involving a remote service and no local service because I cannot really create an endpointslice locally and check that it's removed because this deletion is made through the normal ownerreferences.

How about asserting in one of the tests that the resulting endpointslice has the correct owner reference? That would give us confidence that k8s will then take care of removing the stale endpointslices.

/test

/test

/test

@MrFreezeex One of the unit test failures are legitimate:
 --- FAIL: Test_meshEndpointSlice_Reconcile/Create_service_then_global_service (0.01s)

I0611 09:13:26.678082   49847 endpointslice_controller.go:296] "Starting endpoint slice controller"
I0611 09:13:26.678103   49847 shared_informer.go:313] Waiting for caches to sync for endpoint_slice
I0611 09:13:26.678110   49847 shared_informer.go:320] Caches are synced for endpoint_slice
    endpointslice_test.go:161:
        	Error Trace:	/home/runner/work/cilium/cilium/pkg/clustermesh/endpointslicesync/endpointslice_test.go:161
        	Error:      	Not equal:
        	            	expected: "service"
        	            	actual  : "Service"

        	            	Diff:
        	            	--- Expected
        	            	+++ Actual
        	            	@@ -1 +1 @@
        	            	-service
        	            	+Service
        	Test:       	Test_meshEndpointSlice_Reconcile/Create_service_then_global_service

@MrFreezeex One of the unit test failures are legitimate:
 --- FAIL: Test_meshEndpointSlice_Reconcile/Create_service_then_global_service (0.01s)

I0611 09:13:26.678082   49847 endpointslice_controller.go:296] "Starting endpoint slice controller"
I0611 09:13:26.678103   49847 shared_informer.go:313] Waiting for caches to sync for endpoint_slice
I0611 09:13:26.678110   49847 shared_informer.go:320] Caches are synced for endpoint_slice
    endpointslice_test.go:161:
        	Error Trace:	/home/runner/work/cilium/cilium/pkg/clustermesh/endpointslicesync/endpointslice_test.go:161
        	Error:      	Not equal:
        	            	expected: "service"
        	            	actual  : "Service"

        	            	Diff:
        	            	--- Expected
        	            	+++ Actual
        	            	@@ -1 +1 @@
        	            	-service
        	            	+Service
        	Test:       	Test_meshEndpointSlice_Reconcile/Create_service_then_global_service

Woops indeed my bad!

@MrFreezeex, could you please rebase this PR? That should fix the failing tests

/test

/ci-ingress

/ci-ginkgo

/ci-eks

/ci-ipsec-upgrade

/ci-ingress

@MrFreezeex can you rebase? it looks there are a couple of conflicting files.

/test


Python package setup and basic tests

@tabatkins This PR makes the following changes:

Adds a setup.py file and appropriate python packaging metadata. This should allow you to upload the file to pypi*, which would make it pip installable.
Makes some minor Python 3 compatibility fixes.
Adds some simple tests that run through the included examples and verify they're valid XML. This doesn't test the program logic much, but is better than nothing.
Adds support for the tox test runner to run tests against python 2.7, 3.4, 3.5 and 3.6, showing statement coverage reports after each test run:

---------- coverage: platform darwin, python 3.5.1-final-0 -----------
Name                   Stmts   Miss  Cover   Missing
----------------------------------------------------
railroad_diagrams.py     535    195    64%   25, 27, 50, 76-77, 87, 112-113, 136, 138, 177, 217-218, 239-258, 261-287, 292-295, 298-318, 321-395, 421-422, 486-512, 515-601, 622-623, 662-663, 668-669, 681-682, 687-688, 702-703, 716-717, 733-734, 747-748, 764-765, 776-777, 790-791, 799-806

I'd like to make two (unrelated) pull requests after this one:

Add __eq__ and a descriptive __repr__ method. I've found this makes working with the library much more user-friendly, and makes tests easier to write.
Add an option to inline styles (defaulted to True) so a separate stylesheet is not required to render them. Currently the SVGs are pretty broken without the default stylesheet, which means they cannot be used by image processing libraries like imagemagick, so I think inlining the CSS is the right way to go:

styled
unstyled

After that, I think it'll be ready to cut a release on pypi.
Thanks again for writing this library!

* Note: If you're unfamiliar with the upload process (which is a little confusing), I'd be happy to upload it for you, and transfer ownership of the pypi package to your pypi account. Otherwise, this is a good guide.

I've updated the python code to add a <style> element. Annoyingly, imagemagick can't even handle that, but it seems a much cleaner solution than inlining the CSS on each element. Users of this library can inline the CSS using a tool like premailer before passing them to those tools.  I don't know Javascript very well, but I'd be happy to make a similar update to the Javascript code if you'd like.
The example HTML generated by the python script is here: https://jsfiddle.net/r4cs3wkd/

Thanks for this work!  I'll handle moving the style over to the JS port.


couldn't build sdk from sources

following the guide - clone (v6.0.424) - build.sh -> cd to folder with sources, ./prep.sh, ./build.sh
getting always one error:
/opt/dotnet/build_sdk/uplevel_repo/.dotnet/sdk/6.0.200/Sdks/NuGet.Build.Tasks.Pack/buildCrossTargeting/NuGet.Build.Tasks.Pack.targets(221,5): error : Could not find a part of the path '/opt/dotnet/build_sdk/uplevel_repo/src/source-build/artifacts/source-build/self/src/src/MSBuildLocator/src/MSBuildLocator/build'. [/opt/dotnet/build_sdk/uplevel_repo/src/source-build/artifacts/source-build/self/src/src/MSBuildLocator/src/MSBuildLocator/Microsoft.Build.Locator.csproj]

@bykvaadm - This makes sense.  Have you looked at the changes to source-build post 6.0 (e.g. 8.0)?  The source is now gathered into a VMR (Virtual Mono Repo) located at https://github.com/dotnet/dotnet.  This is the repo you would use to build .NET source-build from.  I suspect this may provide the same thing you are doing.

[Triage] We noticed you are trying to build v6.0.424.  Just want to note that source-build is only supported on the 1xx feature band branches at this point in time.  Of course you are free to build any feature branch but just be aware it will likely require work.
Given .NET 6 is nearing EOL (Nov 12, 2024), this documentation request doesn't meet the criteria for changes we would make.

Have you looked at the changes to source-build post 6.0
Nope, as our product is build by 6.x

Ok, anyway thanks for reply


Hardware Requirements

Please provide hardware and OS recommendations.

On which hardware did the author test this project?
What is the range of setups that this has been tested on? e.g. Colab
What are some known issues and workarounds with hardware? For example on my Sonoma M1 Max I had to install tensorflow-text with
pip install tensorflow_text-2.15.0-cp310-cp310-macosx_11_0_arm64.whl

@Respho Could you send me the file tensorflow_text-2.15.0-cp310-cp310-macosx_11_0_arm64.whl? It fails to build on my machine.

https://github.com/sun1638650145/Libraries-and-Extensions-for-TensorFlow-for-Apple-Silicon/releases


Fatal: unable to connect to github.com

All versions return a Git error, unable to connect to github.com git://github.com/jogboms/flutter_cache_manager
using dart 2.1.0-dev.1.0

check version 0.5.8


Misleading statement

ON this page you state
"Vulnerability assessment scanning for VMs - The vulnerability scanner included with Azure Security Center is powered by Qualys."
This implies it is part of the free service but it isn't included with ASC, it is only included if you purchase Azure Defender for servers. I believe this is just a hangover from before the rebrand of the ASC Standard tier to Defender

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

ID: 8432d9d8-e414-acf4-9287-ebbafab23c7e
Version Independent ID: 9eb184bd-e66d-6b1b-6b86-509ebe70bd3e
Content: Azure Defender for servers - the benefits and features
Content Source: articles/security-center/defender-for-servers-introduction.md
Service: security-center
GitHub Login: @memildin
Microsoft Alias: memildin

@thommck
Thanks for your feedback! We will investigate and update as appropriate.

Thank you! I agree with the change and made a pull request to update the language. https://github.com/MicrosoftDocs/azure-docs-pr/pull/168280
CC @memildin


improve  navigation bar

can u assign me this

its fine or any changes u need ?

I want to do it can you please assign it to me :)

if the issue is open I think I can complete this issue so assign me if this issue is still open


BlockI Refactor

I've been meaning and trying really hard to simplify and reduce each Block's turn method for the past few weeks. But each of their implementations is their own can of worms with different patterns which require different attention and a lot more work therefore.
However, I finally got to a very satisfying solution with BlockI since not only it's much more readable, but it should improve its turning rendering performance for really-really low-end hardware by reducing its uses of the dot operator in for-loops and by using generator expressions.

That's great! The code looks much more readable.
Making each Block's turn method better was too difficult for me.
This time, I found out about the itertools library.
Thank you for your time! 👍

I'm glad you're still learning new stuff. Have a look at the functools library as well. These two libraries use a different paradigm of thinking, called declarative programming, which is different than the imperative programming that you're used to with procedural and OOP. Think of "Haskell" in this case, though not exactly that either.
Also, if you haven't already, have a look at the Zen of Python. These are recommended coding guidelines that one can live by.


ArtifactPage needs a time series charts

What is it?
It should show a summary metrics block and a set of time series metrics that are relevant to the type of artifact that it is.
Architecturally, instead of having 1 ArtifactPage, we probably want different ArtifactPageForWallet, ArtifactPageForRepository, ArtifactPageForPackage, ArtifactPageForContract etc etc.
Then on the server-side code, render the right one with the right data depending on the artifactType

I updated the ArtifactPage to be much more generic, so that you can pass in as props, exactly which keyMetrics to show and which metricIds to pull time series metrics for.
Then in code, you can switch depending on the artifact source and type, as shown in this PR
https://github.com/opensource-observer/oso/pull/2132
I'm gonna close this out for now, because it should be hopefully trivial to plumb different metric IDs (once they're computed) to different artifact types.


Date comparing

Thanks for perfect abstraction like now + 2.days. It will be fantastic if you add dates comparing!

@Koshub I also want to compare dates. I try to add it!

@naoty nice, looking forward to it :+1:

@Koshub I have added it at 9ed880f0350f15a567d04f64b8c318a106127787 and released as 0.3.0! Please try to compare dates.

@naoty  Thanks. Although there are not all operators (like <,>,>=,<=,==,!=) this is better than nothing. Thanks again

@Koshub All operators are supported because Equatable protocol, which requires <,  and Comparable protocol, which requires ==, is implemented. Look at tests!

@naoty Sorry. I didn't know about that.

@Koshub Thank you for the issue!


New project structure

With v2 and ESM, so I also want to make a cleaner project structure.
(Things that I don't explain will be as they are now.)
.github/
benchmark/
gulpfile.js/
src/
  core/  # this is where everything that makes up Core will live
  languages/  # this will be what is now "components/"
  plugins/  # this contains all plugins (JS+CSS) minus their website
  shared/  # this will contains shared logic between Core, languages, and plugins. "util" might not be the best name.
  components.json
tests/
website/  # will be where everything about the website lives
  assets/
  examples/
  plugins/  # the plugin pages (only the HTML)
  index.html

How about shared instead of util?

Would it make sense to move the site to a separate repository?

If I'm contributing to the website, I'd rather just clone the website.
This would cut down on the issue/PR volume.
You could give people commit-access to the website without allowing them access to the library. Or vice-versa.

The website includes the generated documentation site of the library as well as library code (demos). Of course, this dependency only goes one way (the library doesn't depend on the website), so it's possible, it just doesn't seem worth the hassle.

This has been implemented in v2.


Cross-Validation & Regularization Technique Implemented

Description
File: main.py

Added K-Fold Cross-Validation:

Implemented K-fold cross-validation to train the model on different data splits.
Split the dataset into K folds and trained the model on K-1 folds while validating on the remaining fold.
Added L2 Weight Decay to Optimizer: Updated the optimizer to include L2 weight decay.
optimizer = torch.optim.AdamW(unet.parameters(), lr=1e-4, weight_decay=1e-5)

File: lora.py

Added Dropout to LoRA Layers:

Modified the LoRALayer class to include a dropout layer.
Updated the forward method in LoRALayer to apply dropout to the input tensor x.
Modified the LoRALinear class to pass the dropout rate to the LoRALayer.

File: train.py

Implemented Early Stopping:

Added early stopping based on validation loss to prevent overfitting and optimize training time.
Tracked the validation loss and stopped training if the loss did not improve after a set number of epochs.

Fixes # (#9 #8 #7 )
Type of Change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update
[ ] Other (please specify):

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
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] New and existing unit tests pass locally with my changes

Additional Information

Name: Sanjeev Kumar
<EMAIL_ADDRESS>Roll No:2405605
Branch: CSE

@sohambuilds

@sohambuilds please review this

Running on colab should be possible, as we have tried it. If there is a specific issue that you need to troubleshoot, you may join the WhatsApp group for the ML contributors: https://chat.whatsapp.com/Kx8okfEdirALcC8UeFtl5j
Also, do note that implementing early stopping for few shot learning(very few samples) is not desirable.

Please run the following in your colab runtime:
!pip uninstall -y tensorflow !pip install tensorflow-cpu Runtime > Restart runtime

Please commit the updated code, and your dataset, if you're using a new one. You can add the samples later, I will merge it for now.

Merging now. Sample generation is not proper. need to check.


password

hello.
the library requires a password
https://gitlab.server-kampert.de/esp32/libraries/rak3172
https://gitlab.server-kampert.de/esp32/libraries/rak3172.git

Hi,
please clone it from GitHub, because my GitLab server isn´t running currently.


Print QR

https://client.poscariloom.com/public/offline?request=eyJpZFVzdWFyaW8iOiJkZW1vcGRzQGNvbnRhYmlsaXVtLmNvbSIsInJlZkV4dGVybmEiOiI4ODFjNmE3Mi1hYmMwLTQ5MDctYmE5Mi0wYzA1YjJhMzMzMWEifQ==
When I pass this data to the PrintQRCode() this only print the number of the selected QRModel:
if I select QRCODE_MODEL2, print "2", if I select QRCODE_MODEL1, print "1". Maybe it's for the length of the URL? I try in Android app, and pass this URL and it prints excellent. Any idea why this happens?
I see in other issue, solution that if you modify the length of the data string in the project code, after clone this, need to function ok. but I did this and anything happen.
thanks!

A few questions for you:

Have you tried to run the Console Tests ? It has an option to test QRCodes
If you use another string as QRCode data, does it print correctly?
What printer model are you using? Does it support QRCode ESC/POS commands?
Can you provide the full code you are using ?

Hi @igorocampos.

Have you tried to run the Console Tests ? It has an option to test QRCodes

Yes I Tried, and I don't have any case approved

If you use another string as QRCode data, does it print correctly?

Yes, when I pass short string, this print perfectly. I have Thermal Printer that not print anything, only the number of the QRModel, and in other Thermal Printer is not Printing the QR completely. Another to this, I downloaded an App, to my tablet, and I add the  long QrData and Print Excellent

What printer model are you using? Does it support QRCode ESC/POS commands?

Basically this printer doesn't model. But I attach the info of this Printer. It supports QRCode ESC/POS commands

Can you provide the full code you are using

**
Here I get the URL to create the QR.
public static string GetUrl(string userId, string cartId)
{
var baseUrl = Consts.BaseOfflineReceiptUrl;
var args = GetParams(userId, cartId);
string content = JsonConvert.SerializeObject(args);
var bytes = Encoding.UTF8.GetBytes(content);
var encodeContent = Convert.ToBase64String(bytes);
var url = baseUrl + encodeContent;
        return url;
    }

    var url = GetUrl( userId, cartId);
    e.PrintQRCode(url, TwoDimensionCodeType.QRCODE_MODEL2, Size2DCode.LARGE, CorrectionLevel2DCode.PERCENT_30),

**

Thanks!

Reading what you said, sounds like we truly have a bug, but I can't identify where. Are you able to post here the said "Technical Manual of Portable Receipt Printer" for details? More specifically what that manual says about print QRCode command?

I look for "Technical Manual of Portable Receipt Printer" in google, but i don't find anything. I find only this (the attached files), but I think that you know the

m.

Does it work if you use PrintQRCode(url) without the additional parameters?  Maybe one of the parameters you are setting does not work.

No. It doesn't work either.

Model 2 should be able to receive up 4296.
I'm pretty sure we had someone testing pretty big strings 2k+ big. I can look for the issue number about it, but perhaps that 58 cap is on the printer you are using?

Here it is
https://github.com/lukevp/ESC-POS-.NET/issues/71#issuecomment-701730942

Yes, I see this post, but I don't know why when I send 59 characters to the printer, doesn't print.

I think it is a restriction imposed by the hardware rather then ESC/POS. Are you able to contact the manufactor and ask them? You should be able to provide all bytes you sent the printer for their analysis

The model of the Printer is 58HB6 - Thermal Printer. It's made in China...I don't have any contact.
I install this App in my tablet, and when I add the string to generate the QRCode, and then print, it prints perfect.
https://play.google.com/store/apps/details?id=com.qrmango.print&hl=en_US&gl=US.

Now that is interesting! What do you think about it @lukevp ? Maybe contact that app's dev to ask for the ESC/POS command they are using? And see what's the difference? No idea of what else could be causing the cap, if not the hardware...

I saw the code, with the two libraries and it's the same bytes (with a many differences) and the result is the same with my thermal printer.
I have 3 thermal printers. One of these has USB and Bluetooth connection (it's a mini portable Thermal Printer), the other two only supports USB connection (one of these is Epson Printer).
The other library was function with the other two printers, but with the USB and Bluetooth Printer is not print the QR.
I see in some Apps to print on Android that you have select Printer Type, and these are the options: EpsonPrinter, StarPrinter and Generic Text Printer. Maybe this is the problem.


Set HTTP_PROXY gets ignored for goss binary download

We are running our packer CI in an environment where you need an http_proxy to connect to the internet (or most other services for that matter)
The provisioner doesnt seem to care about the proxy being set.
Thus the download from github just fails.
==> vsphere: Provisioning with Goss
    vsphere: Installing Goss from, https://github.com/aelsabbahy/goss/releases/download/v0.3.13/goss-linux-amd64
    vsphere: Downloading Goss to /tmp/goss-0.3.13-linux-amd64
==> vsphere:   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
==> vsphere:                                  Dload  Upload   Total   Spent    Left  Speed
==> vsphere:   0     0    0     0    0     0      0      0 --:--:--  0:02:10 --:--:--     0curl: (7) Failed to connect to github.com port 443: Connection timed out
==> vsphere: --2020-12-02 14:52:58--  https://github.com/aelsabbahy/goss/releases/download/v0.3.13/goss-linux-amd64
==> vsphere: Resolving github.com (github.com)... <IP_ADDRESS>
==> vsphere: Connecting to github.com (github.com)|<IP_ADDRESS>|:443... failed: Connection timed out.
==> vsphere: Retrying.
==> vsphere:
==> vsphere: --2020-12-02 14:55:10--  (try: 2)  https://github.com/aelsabbahy/goss/releases/download/v0.3.13/goss-linux-amd64

Btw, http proxy support for goss just landed
https://github.com/aelsabbahy/goss/pull/589

marking as closed


Controller keeps scheduling Pods from a cancelled Build

Steps to reproduce:

Set up buildkite/agent-stack-k8s with max-in-flight: 2
Push a new commit to an open PR which, it triggers 10 new jobs, controller schedules 2 Pods at a time respecting max-in-flight: 2
Cancel the build in Buildkite UI / API, all 10 new jobs display as Cancelled in Buildkite UI
Observe controller still scheduling Pods for that build flooding the K8S cluster (kubectl describe pod shows BUILDKITE_COMMIT which is cancelled) and preventing other Jobs from being handled

Environment:

AWS EKS v1.25
buildkite/agent-stack-k8s v0.15.0

Thanks for the report @artem-zinnatullin, I can see how this happens.


Spec updates

Easiest to follow this PR if you read from bottom up (types -> service -> test -> spec)

Codecov Report

Merging #107 (02b5f37) into main (6579a33) will decrease coverage by 0.37%.
The diff coverage is 53.57%.

@@            Coverage Diff             @@
##             main     #107      +/-   ##
==========================================
- Coverage   59.41%   59.03%   -0.38%
==========================================
  Files           3        3
  Lines         239      249      +10
==========================================
+ Hits          142      147       +5
- Misses         81       84       +3
- Partials       16       18       +2

Flag
Coverage Δ

unittests
59.03% <53.57%> (-0.38%)
:arrow_down:

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

Impacted Files
Coverage Δ

server/service.go
64.97% <53.57%> (-0.81%)
:arrow_down:

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 6579a33...02b5f37. Read the comment docs.


Use Client in an async fashion

I simply want to create a client and pass it to a number of threads to do a publish_event (just looking at throughput). No matter how I try to move, clone, copy Client into another thread, it fails in various ways. Any thoughts on how to accomplish? Here is some example code you can beat up:
    // Create the client
    let mut client = dapr::Client::<dapr::client::TonicClient>::connect(addr).await.unwrap();
    let foo = std::sync::Arc::new(client);
    for x in 0..10 {
        let cli = foo.clone();
        tokio::spawn(async move {
            println!("In spawn");
            let mut metadata = HashMap::<String, String>::new();
            metadata.insert("count".to_string(), x.to_string());
            cli
                .publish_event(
                    "pubsub".to_string(),
                    "A".to_string(),
                    "text/plain".to_string(),
                    format!("{} => hello from rust!", &x).into_bytes(),
                    Some(metadata),
                )
                .await;
        });
    }

FYI here is the full code sample that works on an R-Pi4. The problem is this is SLOOOOOWWWWWW. This gets through messages at about 191 messages/sec. On an example C# multi-thread program I can get 754 messages/sec. I imagine a bulk of the time is spent in creating and tearing down the client.
use std::{collections::HashMap, thread, time::{Duration, Instant}};

#[tokio::main]
async fn main() {
    // TODO: Handle this issue in the sdk
    // Introduce delay so that dapr grpc port is assigned before app tries to connect
    thread::sleep(Duration::from_secs(2));
    let start = Instant::now();
    let mut handles = vec![];
    for x in 0..1000 {
        handles.push(sender(x));
    }
    futures::future::join_all(handles).await;
    println!("All done. Elapsed: {} ms", start.elapsed().as_secs_f32());
}
async fn sender(count: i32) -> Result<(), Box<dyn std::error::Error>> {
    //println!("--> function");
    // Get the Dapr port and create a connection
    let port: u16 = 45045; //std::env::var("DAPR_GRPC_PORT")?.parse()?;
    let addr = format!("https://<IP_ADDRESS>:{}", port);

    // Create the client
    let mut client = dapr::Client::<dapr::client::TonicClient>::connect(addr).await?;

    // name of the pubsub component
    let pubsub_name = "pubsub".to_string();

    // content type of the pubsub data
    let data_content_type = "text/plain".to_string();

    // topic to publish message to
    let topic = "A".to_string();

    // message metadata
    let mut metadata = HashMap::<String, String>::new();
    metadata.insert("count".to_string(), count.to_string());

    // message
    let message = format!("{} => hello from rust!", &count).into_bytes();

    client
        .publish_event(
            &pubsub_name,
            &topic,
            &data_content_type,
            message,
            Some(metadata),
        )
        .await?;
    //println!("<-- function");
    Ok(())
}

So the publish_event function is fully asyncronous and returns a future that can be await'd on. In general the pattern goes:
Create a batch of messages, and save their futures to a stream. Then you can await the stream of futures. I'll take a pass at the example you gave a little later today, but posting this now with the hopes that my reply might be enough to unblock you.

Closing in favour of #91 as this will partially resolve this.


Search link in dashboard side nav points to wrong route

Prerequisites

[x] I am running the latest version
[x] I checked to make sure that this issue has not already been filed

Expected Behavior
Search link in dashboard side nav should point to /search route
Current Behavior
Search link in dashboard side nav currently points to /visualizations route

#429 closes this issue


Refactoring helpers module to be somewhat more pythonic

cmp function was removed, as it essentially just wraps normal condition check.
Explicit inheritance from object was removed.
Retrieval of default value from dictionary was simplified.
I've also slightly expanded the docstrings so they would be more accurate,
and reorganized conditional on line 108 to short circuit the evaluation.
Comparing two dictionaries is relatively expansive compared to plain boolean value.

Eh, license appears to have an EOF issue?

This needs rebase.

done

recheck

LGTM

/lgtm


[build-script] Accept --ninja-bin and sync with sourcekit's build-script-helper

For now, just thread ninja_bin through as an environment variable.
Future commits will make use of this during testing. Also pass the
toolchain path and remove unnecessary differences with the sourcekit-lsp
build-script-helper script.

Please test with following pull requests:
https://github.com/apple/swift/pull/26269
https://github.com/apple/sourcekit-lsp/pull/124
@swift-ci please test


Add Location Strategy Keyword

This was discussed both in #333 and #364 and here's a preliminary thought that I had.
Instead of tying the Add Location Strategy to javascript as I originally intended I've designed it so that a location strategy is actually implemented as a keyword. This gives a lot of flexibility in that location strategies can (if desired) be written directly in RF as a user keyword.
Keep in mind that this is just a rough draft to get started with.
Idealistically I'd have this
*** Settings ***
Library    Selenium2Library
Resource    Locators
Resource    Web Navigation Stuff

*** Keyword ***
Setup Locators
    Add Location Strategy    custom    My Custom Locator

*** Test Cases ***
My Test
    [Setup]    Setup Locators
    Open Browser To My Site
    Click Element    custom=mySelector[example=true]

@emanlove, could you take a look at this and give me your thoughts?
Thanks.

This is now feature complete and ready for review. I've added tests and documentation.
It includes two new keywords:

Add Locator Strategy
Remove Locator Strategy

The locator strategy itself is a user keyword as shown in the documentation. This allows easy integration with JavaScript (which was my original goal), but also covers any other applicable situation.
An added locator strategy will stay present throughout the duration of testing unless removed by Remove Locator Strategy.

Add Locator Strategy is a global action. When this keyword is called, the locator strategy that is specified doesn't go away unless Remove Locator Strategy is subsequently called.
Global alterations of a testing environment by a test are generally unwise. @pekkaklarck, @emanlove what do you think of this behavior? I suppose it would be possible through library listener methods to automatically unregister the locator strategy after the scope it was registered in has subsided.
Which would be better? The current behavior (after it's set anywhere it stays until it is removed) or the scoped behavior (where if it's set in a test it's removed after that test is complete).

@zephraph, automatically removing locator strategies sounds like a good idea. Library listeners ought to make implementing it pretty simple. Should probably be configurable with an argument like auto_remove=False, though.

I've added a specific test because I was in talks with @IlfirinPL on #376. This test case will likely be removed before merge along with the changes mentioned by @pekkaklarck above.
Don't merge this as is.

@pekkaklarck, when you get a chance could you review this?
Adding the auto unregister made it a bit more complicated.

@emanlove, could you please review this? Actually, look over everything in the 1.7 Milestone. I really want to go ahead and push a new release this weekend.

A large part of the bulk of this is the scoping as suggested by @pekkaklarck. The original implementation made it persistent by default. This implementation makes it scoped by default (which required hooking into listeners). I try to make a little event system so if we needed to do stuff like this in the future we could just add new events.

Sorry, I was on a business trip with limited time for other work. What commits/code should be reviewed?

Just look here and see if anything jumps out at you. I really would like you to look over the event system I wrote for scoping the  locator registration and tell me if it's too much.

The event system looks pretty neat but I'm not entirely sure is it necessary for this usage. Are you planning to use it for something else too? If not, then there's a risk that YAGNI.
I also added some nitpicking comments about style. I'm not sure is there any agreed styles for this project so feel free to ignore them.

Well, it's quite possible I could use it again in the future, but I can't guarantee that. That's just how I thought to implement it. Also, I trimmed the scope events down a bit. It's more DRY now.

Ok, looks good.

So, are we good to merge?

I have  an idea on similar enhancement:
https://github.com/divfor/robotframework-selenium2library/pull/6
This enhancement is to simplify locator usage. Generally it is difficult for junior engineers and customer to understand the locators written in user keywords or test steps. It would be better if they can use Web UI texts/words prompting for target element instead of locators. The real locators could be mapped by prompting UI text in variable files created and maintained by senior engineers. If so, the test cases will be more easier to read.

I'm not really following you. The custom locators written here should be specified in a resource file or library, not included in a test. I wrote this feature with javascript frameworks like angular and ExtJs in mind, though it's not limited to those sorts of things. I would say it's more up to the locator developer to document their locator usage. By and large this should be a rarely used feature though.


Update windows python installation script to freeze numpy version

@WilliamDAssafMSFT - can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


[FEATURE] Mapbox/CodeIT GeoJson support

Describe the feature you want

Mapbox GeoJson
I was checking CodeIT documentation and I could not find a way to give mapbox a geojson file to load. I would like to have a few points on my map.

Hi @trmendes. From what I understood, the theme doesn't support this feature yet. Improvements are needed.

Hi @trmendes,
I added the GeoJson support. I added the sample config file that will be used for loading the sources and configuring the layers. These two properties are taken from the official docs of Mapbox.
{
  "sources": [
    {
      "id": "logo",
      "properties": {
        "type": "geojson",
        "data": "/features.geojson"
      }
    }
  ],
  "layers": [
    {
      "id": "logo-layer",
      "type": "fill",
      "source": "logo",
      "paint": {
        "fill-color": ["coalesce", ["get", "fill"], "#005c7c"],
        "fill-opacity": 0.5
      }
    }
  ]
}

The sample shortcode with provided properties file:
{{< mapbox lng=28.836 lat=47.021 zoom=8 marked=false properties="/static/features-layers.json" >}}

The result:

Super. Thanks @victor-pogor
I look forward to trying it out with my collection :)


Package LMQL as a Nix flake

Allows nix run github.com:eth-sri/lmql#playground to invoke the playground, even on a system that previously didn't have Python, node.js, or any other dependencies (beyond Nix itself) installed.
Allows nix develop github.com:eth-sri/lmql# to start a shell in which Python has all the dependencies to import lmql and use relevant libraries, even if (again) there wasn't even a Python 3.10 interpreter at all previously.

I consider this PR ready to be merged, is that also the status from your side? I just want to test myself once before pressing the button.

There's some room to enhance the process documentation -- I realized (during a bike ride yesterday) that I haven't documented updating the yarn.nix and yarn.lock files when JavaScript dependencies are updated -- but that can presumably go in as a separate PR. AFAIK, we're good to merge.


#2987: Calculated columns dialog: filter out irrelevant functions from suggestions dropdown

1.20.1, power pack 1.1.10
Function suggestions still contain too much irrelevant entries, that might be confusing for a regular user.
For example, packages:

Expected results:
Define which functions are actually useful in calculated columns, filter out all irrelevant ones.
For packages: are there any real use cases? If not, it might be more useful to filter it out completely.

This issue has been mirrored in Jira: https://reddata.atlassian.net/browse/GROK-16492

v. 1.21.1
PowerPack v.1.2.0
We are waiting for feedback about filtering

v. 1.21.1
PowerPack v.1.2.0
There are still a lot of irrelevant functions:

JS, DBScalarQuery, Call, DeployPackageVersion etc.
functions from packages, e.g,.

Some of these appear in suggestions dropdown, some are present only in the right side panel.

Hi @katerina-stepanova , I agree regarding the follwoing functions: Call, DeployPackageVersion, JS. They should be marked as internal and not be included into the list.
But as for DBScalarQuery - as far as I know, it can be used in the 'Add new column' dialog. As well as package functions. I mean, potentially any function which returns some scalar value (or a vector function which returns column and is marked by special tag) can be used in 'Add new column'.
Moreover, ability to use package functions straight from 'Add new column' dialog is an improvement itself.
Basically, we introduced different kinds of filtrations, sortings and hints to help user to navigate through the list of functions easily. Like autocompletion filtering as you type, functions filtering by categories (using checkboxes), functions search. Also if you click on a column in a column list, functions in the right widget will be sorted with the most relevant once on top.
Maybe I do not fully understand why would wider list of functions confuse users. Considering all available hints...


Punishments#checkLockExpiration: Permalock check

Updated

From now on, we should probably get used to using const instead of let for variables that aren't modified.


Update setup.py

Specified correct pymanopt version, because the latest pymanopt updates are not yet reflected in the morphomatics source code.
Included ipygany, in case someone doesn't have it installed.

Dear Dimitrios,
thanks for your contribution. We really appreciate your effort.
I will happily add the version requirements of pymanopt, however would like to omit the ipygany requirement as it is only used in the tutorials and not needed for morphomatics itself - can you update the PR accordingly?
As for the tutorials, I will update the Jupyter Notebooks to install ipygany instead.
Best,
Christoph

Dear Christoph
Thank you and your colleagues for your wonderful research!
I hope to find some time and help with the pymanopt update in the coming weeks.
I deleted ipygany from the PR, per your instruction.
Best,
Dimitrios

That's great - thank you very much!
As for pymanopt we are currently working on the next major release based on jax for numerical computations.
Here we will also migrate to the current manopt version and it should be ready for release any time soon.
We are very happy about your interest in morphomatics and hope that it will help you with your research.
Take care,
Christoph


Add cypress tests that will verify social login

Add Cypress test on FE repo to verify that it's possible to log in with social account AND user is then redirected to page with booking - automatically.
Related bug: #592

Since visiting external sites is not a good practice, check Cypress.io docs.
I followed the recommendation of one the maintainers see -> https://github.com/cypress-io/cypress/issues/1380#issuecomment-368663427
I was trying to follow these examples from official cypress repo, but in kiwi we call our own backend and is this backend the one who makes the final call to get the redirect link. Maybe this is why there are no cypress social login test in kiwi frontend either, only kiwi login test.
Opinions, recomendations?


Show inherited functions in Debug (when deploying with foundry)

Description
This implements the same inheritedFrom functionality previously done for hardhat. Also, I did a small renaming in hardhat-scripts.
First try was here: #602
:warning: contains changes from #603
Additional Information

[x] I have read the contributing docs (if this is your first contribution)
[x] This is not a duplicate of any existing pull request

Related Issues
Related to discussion #558 and PR #564
Your ENS/address: harmont.eth

Also, I did a small renaming in hardhat-scripts

What did you update? I think it would be really great if we update it in main first so that we don't diverge cli from main


Do not show changelog after update

Is your feature request related to a problem? Please describe.
I really like Altair, but it really annoys me every time it updates and opens a new tab with the Changelog.
Describe the solution you'd like
I would prefer to not see the changelog automatically but instead have an option to view it if I want to see it.
Describe alternatives you've considered
Other options

Add a setting to enable/disable the automatic opening of the changelog
Show the changelog in a Modal dialog when opening Altair

Additional context
using Chrome/Firefox extension

@azapps I've added options in the extension. In the next update, you would be able to go into the extension management page and disable the changelog option. 😇

Thank you ;)

Even after some updates I cannot see the option in the settings dialog:

@azapps It's in the extensions options page


SystemExit

Could not open URL. Please check your internet connection and/or ssl settings
If you are using proxy, make sure your proxy settings is configured correctly
An exception has occurred, use %tb to see the full traceback.

Hi @onlinehuazai
The user agent was changed on the Joeclinton1 fork, it can maybe solve your issue like it is just before the exit.
headers = {}
headers[
    'User-Agent'] = "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/88.0.4324.104 Safari/537.36"
if cur_version >= version:  # If the Current Version of Python is 3.0 or above
    try:
        req = urllib.request.Request(url, headers=headers)
        resp = urllib.request.urlopen(req)
        respData = str(resp.read())
    except:
        print("Could not open URL. Please check your internet connection and/or ssl settings \n"
              "If you are using proxy, make sure your proxy settings is configured correctly")
        sys.exit()

See #298 to install it.

Hi @onlinehuazai
The user agent was changed on the Joeclinton1 fork, it can maybe solve your issue like it is just before the exit.
headers = {}
headers[
    'User-Agent'] = "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/88.0.4324.104 Safari/537.36"
if cur_version >= version:  # If the Current Version of Python is 3.0 or above
    try:
        req = urllib.request.Request(url, headers=headers)
        resp = urllib.request.urlopen(req)
        respData = str(resp.read())
    except:
        print("Could not open URL. Please check your internet connection and/or ssl settings \n"
              "If you are using proxy, make sure your proxy settings is configured correctly")
        sys.exit()

See #298 to install it.

it do not work

Like the error message says, you can display the full traceback to have more details.
If you use a proxy, do not forget to indicate it by adding --proxy ip:port to the command.
If it is an SSL issue, you can try to install again Python.

Like the error message says, you can display the full traceback to have more details.
If you use a proxy, do not forget to indicate it by adding --proxy ip:port to the command.
If it is an SSL issue, you can try to install again Python.

Tahnk you. I do not have a proxy,  so I can not use this project?

You can use this project without proxy.
In fact the part of this project which simply query an url to get its content fails.
A possible cause of this issue is if you use a proxy to have not indicated in the command arguments.
So if you don't use proxy, the issue is probably in ssl settings.
If installing again Python and the project don't solve the issue, you need to display the traceback to have an error message to search on the internet the solution.
Do not hesitate to post the traceback here.


Weekly Downloads Python Podcast

ssia 😁

Hey @ephes,
the README gets automatically generated based on the yaml format. I created https://github.com/EngineeringKiosk/GermanTechPodcasts/pull/238 to add the data into the right format + place.
Thanks for the contribution!

Ups, sorry - it probably too early in the morning, thanks 😄


[help] Don't cache /api in cacheMaps

So what I have is an app with two parts, the front-end app with this plugin installed and an API. Both are under the same domain with the app being in the / route while the api in the /api.
Problem now is that it seems the SW is also caching the requests from the app to the /api. Trying to add a match regex to exclude /api URLs in cacheMaps but doesn't work.
      cacheMaps: [
        {
          match: /^(?!\/api\/).*/,
          to: '/'
        }
      ],

What's the correct way of doing it?

@mcchrish if you aren't able to specify correct RegExp (which I expect to be possible though), you can use a function instead, like this:
{
  match: function(url) {
    if (url.origin !== location.origin) return;

    if (url.pathname.indexOf('/api/') === 0) {
      return;
    }

    return new URL('/', location);
  }
}

or also set requestTypes: ['navigate'] to match only navigate requests instead of all requests.

There is doc about this actually: https://github.com/NekR/offline-plugin/blob/master/docs/cache-maps.md

requestTypes: ['navigate'] did it, man. Thanks so much for this! 👍


Update modelator to 0.2.0

Description
This updates to the new Modelator 0.2.0. Some caveats:

I am manually setting the model checker to TLC (the default in Modelator 0.2.0 is Apalache), because the TLA+ spec seems not to run in Apalache.
I had to get rid of an assert I don't understand on https://github.com/informalsystems/ibc-rs/commit/60e3185e9249e1fb499758f37cc72a728e3c187b#diff-ec1a9a4dbc3d8d3aa74543450bb465b3c01d62036a5449fbba077e185f4ce323L66 because it was failing.

Let me know if this looks OK otherwise, and I can add the unclog file and other housekeeping stuff to this PR.

For contributor use:

[x] Added a changelog entry, using unclog.
[ ] (not applicable) If applicable: Unit tests written, added test to CI.
[ ] (not applicable) Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[x] Updated relevant documentation (docs/) and code comments.
[x] Re-reviewed Files changed in the Github PR explorer.

Security audit is failing but that appears to be a Tendermint dep?

I'm not sure I understand the relation between this PR and the previously-opened one #812. Could you clarify if the current PR complete supersedes (so we can close and ignore) #812?

Security audit is failing but that appears to be a Tendermint dep?

Indeed, not exactly Tendermint dep, but fixing the security audit is blocked by prost & tonic (ref https://github.com/informalsystems/ibc-rs/pull/1166#issuecomment-879173781)

I'm not sure I understand the relation between this PR and the previously-opened one #812. Could you clarify if the current PR complete supersedes (so we can close and ignore) #812?

We should probably wait for @andrey-kuprianov to weigh in, but #812 brings in a whole new API (the EventRunner API). It is also significantly behind master in other respects, which makes completing and merging it hard.
This PR uses the same API as the Modelator tests in master (the StepRunner API). This is a very straightforward PR, which just involved changing some function names. It unblocks my work on the ICS07 upgrade client mbt tests, since there were some bugs in the old modelator currently used on master.
We may want to finish #812 switch over to the EventRunner API at some point, but IMO that is a separate task and this PR should be merged first.

I'm not sure I understand the relation between this PR and the previously-opened one #812. Could you clarify if the current PR complete supersedes (so we can close and ignore) #812?

We should probably wait for @andrey-kuprianov to weigh in, but #812 brings in a whole new API (the EventRunner API). It is also significantly behind master in other respects, which makes completing and merging it hard.
This PR uses the same API as the Modelator tests in master (the StepRunner API). This is a very straightforward PR, which just involved updating the modelator version and changing some function names etc. It unblocks my work on the ICS07 upgrade client mbt tests, since there were some bugs in the old modelator currently used on master.
We may want to finish #812 and switch over to the EventRunner API in the future, but IMO that is a separate task and this PR should be merged first.

Got it, thanks for the explanation Jehan, that's very helpful!
Copying @cezarad here if she wants to weight in by reviewing this PR.


Preserve case in suggested values

See HXLStandard/hxl-proxy#210 reported by @cjhendrix

This was fixed in passing by #138


Add Dark Mode

https://github.com/Sweeistaken/MintHTML/commit/a271bf05d58253684074150a676352fa6ea2d667
Made first tweaks by making the darkmode toolstrip inject the following CSS:
html{
color-scheme: dark;
}

TODO: Make the Application darkmode, and allow checking system darkmode.

https://github.com/Sweeistaken/MintHTML/commit/ff66b044360504e0b4a68222e6704d080e567dcc

TODO: Make a custom toolstrip renderer


[Suggestion] Mechinism to import/export the contents of a punched tape between saves/worlds

Being able to import and export punched tape data, will allow for easily moving control panels and sequencer programs between games and players. It will also allow for an easier view of long tapes (things like panel tapes) in external tools like notepad or vscode
I see two simple-ish ways to serialize the tape each with pros and cons

use ASCII
pros

easy to read

cons

not all of the 256 ASCII chars are printing so some creative escaping will be needed

use hex values
pros

captures all of the possible tape values

cons

much harder to read and edit by humans

Thinking about this base 64 of the raw bytes of the punches is most likely the best option for import/export


Review all the unwrap from the Windows implementation

We should go through all the unwrap in the Windows implementation make sure the all make sense and see if we can handle them more gracefully.
Not all of them make sense, see #1191.

I've just checked and we're only unwrapping calls to locking which should be fine, so closing.


🎉 1.2.0

ℹ About this release

Version: 1.2.0
Type: minor
Last Release: 1.1.1 (5/31/2022, 8:30:44 AM) [?]
Commits to merge: 24 [?]

🐛 Bug Fixes

Fix Kinderschwimmen on thursdays (53f2fc8)

🆕 Features

Add Kinderschwimmen on Thursday (052e7d5)

📦 Development Dependencies

Update @amanda-mitchell/semantic-release-npm-multiple from ^3.2.1 to ^3.5.0
Update @types/node from ^17.0.31 to ^17.0.42
Update @typescript-eslint/eslint-plugin from ^5.22.0 to ^5.27.1
Update @typescript-eslint/parser from ^5.22.0 to ^5.27.1
Update dayjs from ^1.11.2 to ^1.11.3
Update eslint from ^8.15.0 to ^8.16.0
Update eslint-plugin-jsonc from ^2.2.1 to ^2.3.0
Update rrule from ^2.6.9 to ^2.7.0
Update ts-node from ^10.8.0 to ^10.8.1
Update typedoc from ^0.22.15 to ^0.22.17
Update typescript from ^4.6.4 to ^4.7.2

:tada: This PR is included in version 1.2.0 :tada:
The release is available on:

v1.2.0
GitHub release
Docker container (sebbo2002/fitness-first-ical-proxy:1.2.0)
v1.2.0

Your semantic-release bot :package::rocket:


[FSDE-10070] Fixed Drawer close button issue

Creating this PR for testing
https://ctl.atlassian.net/browse/FSDE-10070

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-234/1/. :white_check_mark:

You can check this PRs instance in https://5455a6c6cda2f15e3ad48b8ef4eb4922fc59344f-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-234-ux-chi.kubeodc-test.corp.intranet/ (internal)

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-234/2/. :white_check_mark:

You can check this PRs instance in https://42c220224851b433570daf03297109ad10fb69fc-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-234-ux-chi.kubeodc-test.corp.intranet/ (internal)

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-234/3/. :white_check_mark:

You can check this PRs instance in https://880c0ab8950264457b4df38d9931076f973429c8-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-234-ux-chi.kubeodc-test.corp.intranet/ (internal)


Not functioning with GS110EMX

System Health details
Unsure where this is.
Checklist

[X] I have enabled debug logging for my installation.
[X] I have filled out the issue template to the best of my ability.
[X] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
[X] This issue is not a duplicate issue of any previous issues.

Describe the issue
Attempted to connect to two separate GS110EMX switches and got "Unknown error occurred"

I have tried this with the stock firmwares on the device - <IP_ADDRESS> - and then with upgraded current firmware <IP_ADDRESS>.  Neither firmware version was able to be connected and scraped by this addin.

Reproduction steps

Load Repo
Install Integration
Configure connection

Debug logs
Will collect and report back

Diagnostics dump
N/A

Hi @Tiger770, your switch is not supported (yet). Can you share the source of the login page and the exact URL of the screenshot above and of the page with 'Port Status' (see side menu in the screenshot)?

Login page source:
`

  |
-- | --
  |
  |
  |
  |
  |
  |
  | NETGEAR GS110EMX
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  | GS110EMX - 8-Port Gigabit Ethernet Smart Managed Plus Switch with 2-Port 10G/Multi-Gig Uplinks
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |  Login
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  | Password
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  | Copyright © NETGEAR, Inc. All rights reserved
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |

NETGEAR GS110EMX

GS110EMX - 8-Port Gigabit Ethernet Smart Managed Plus Switch with 2-Port 10G/Multi-Gig Uplinks

 Login

Password

Copyright © NETGEAR, Inc. All rights reserved

`
The Exact URL is http://<IP_ADDRESS>/
The URL for the Monitoring Port statistics is http://<IP_ADDRESS>/homepage.html
`

  |
-- | --
  |
  |
  |
  |
  |
  | NETGEAR GS110EMX
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  | GS110EMX - 8-Port Gigabit Ethernet Smart Managed Plus Switch with 2-Port 10G/Multi-Gig Uplinks
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  | We are delighted to have you as a customer.Registration confirms
  | your email alerts will work,lowers technical support resolution
  | time and ensures your shipping address accuracy.We'd also like
  | to incorporate your feedback into future product development.
  |
  | NETGEAR will never sell or rent your email address and you may opt out of communications at any time.
  |
  | Please register now.You can also register at a later time by
  | choosing Help>Registration from the menu toolbar.
  |
  |
  |
  | Contacting Registration server. Please wait...
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  | Change Admin Password
  |
  |
  |
  |
  | As a security requirement, you must change the default admin password.
  |
  |
  | Password must be 6-20 characters, and contain at least one uppercase letter, one lowercase letter, and one numeral. Allowed symbols are !@#$%^&*()
  |
  |
  |
  |
  |
  |
  | New Password
  |
  |
  |
  |
  |
  |
  | Confirm Password
  |
  |
  |
  |
  |
  |
  |
  | Passwords must match.
  |
  |
  |
  |
  |
  | Change Password
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  | Copyright © NETGEAR, Inc. All rights reserved
  |
  |
  |
  |
  |
  |
  | Alert
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |
  |

NETGEAR GS110EMX

GS110EMX - 8-Port Gigabit Ethernet Smart Managed Plus Switch with 2-Port 10G/Multi-Gig Uplinks

Copyright © NETGEAR, Inc. All rights reserved

Alert

`

It dawned on me you may want the port status page, not the port statistics one.  So here's the port status page and source:

Port Status Source.pdf

@Tiger770 I can't copy the source from your PDF files (looks like the print converted them into graphics). Could you just upload the HTML sources? (Github does not allow HTML attachments, but if you change the filetype to .txt it works).

Try these.  Also, I realized if I overed over the links in the frame on the left I could pull the full URL to the Status and Statistics page.  This is them:
Login Source.txt
Port Statistics Frame Source.txt
Port Status Frame Source.txt
Port Status Page
http://<IP_ADDRESS>/iss/specific/port_settings.html?Gambit=qkkfnehlsmgmehiiilbhkfjahloicifimmjm
Port Statistics Page
http://<IP_ADDRESS>/iss/specific/interface_stats.html?Gambit=qkkfnehlsmgmehiiilbhkfjahloicifimmjm


Properly log path in web proxy

This PR:

Fixes the logging on the path (to close #8)
Moves the proxyRes event handler inline so that intellisense of the callback arguments can be properly inferred

No worries - it's really nicely put together. Looks like Peril is stuck for some reason - can't merge until it's resolved. Any ideas @renanborgez ?

I have no idea why, but I can merge it as admin, may I? @francisrstokes

Absolutely 👍


Minor correction in RegNet docs

Corrections:

Removed "Fast and Accurate Model Scaling" paper from references in BASE_DOCSTRING, since we have not implemented the Z variant of RegNets.
Minor nit in name of blocks: (Y,Z) -> (X,Y).
Changed default weights initialization to "imagenet" following the convention of rest of the applications.

@fchollet
Thanks for the approval!

Thanks for merging @fchollet!


Add a hide_url option to suppress URLs in login item subtext

Allow for a bit of a terser view of login items by giving the option to exclude URLs from results

Closing this in favor of #19


fix typo

Coverage decreased (-0.1%) to 50.904% when pulling 21565a884db2a2e933c5cd295c8ce850789da570 on lcestou:patch-2 into 1a140d21e2b9403b54ed0d793608c817f9f6c15d on TimothyLuke:master.


Unified Items Compatibility

Not sure what to do with this as it could be the other mod but is there a way to get Unified Items materials to work with this?  Aluminum ore is the specific issue (although there could be more) as it has two entries for the OreDictionary one under oreAluminum and another under oreAluminium.
Can I add it somehow or is that something you have to do?

Currently only 'ores' with one entry can be unified, but I can fix it

Alright. Thanks


Running on external Video

How do i run the same code on some video i have on my system? Any help guys?

I could not even run the code , can you help me please,I get the error below

@yazidabdulsalam Looks like you do not have opencv library installed in your environment.

pip install opencv-python? or cv2

@Mrinal18 You need to open the video stream with videopath instead of 0 (webcam)
cv2.VideoCapture(videoPath)


Tooltips

Add tooltips for reek.
Related to https://github.com/SublimeLinter/SublimeLinter3/issues/178

See also https://github.com/Microsoft/TypeScript-Sublime-Plugin/search?utf8=✓&q=tooltip

@darrenleeweber Thanks for the suggestion however I think this feature belongs to SublimeLinter3 repo and from what you linked they are already working on it. Once SublimeLinter gets this feature this plugin  will probably support it out of the box.

OK, no problem.  FYI, I'm finding this snippet useful at lines 409-421 of sublimelinter.py:
                    # show_popup(self, content, flags = HTML,
                    #            location = -1, max_width = 320, max_height = 240,
                    #            on_navigate = None, on_hide = None)
                    import string, re
                    pat = r'\b(([\w-]+://?|www[.])[^\s()<>]+(?:\([\w\d]+\)|([^%s\s]|/)))'
                    pat = pat % re.escape(string.punctuation)
                    http_regex = re.compile(pat)
                    style = ''
                    content = '
' + '
'.join(line_errors) + '
'
                    content = http_regex.sub(r'\1', content)
                    view.show_popup(style + content, location=-2, max_width=1000, max_height = 300,
                                    on_navigate=self.on_navigate, on_hide=None)
                    sublime.set_timeout_async(view.hide_popup, 5000)

See also https://github.com/SublimeLinter/SublimeLinter3/pull/450

Looks cool!


fix Travis build

https://travis-ci.org/ageweke/fortitude/builds

started failing with https://github.com/ageweke/fortitude/commit/3de1286652874506802a75befde0f11d79a0ec67

Done!


Fix definition of bits in Interrupt enum

The problem
When using Can::enable_interrupt or Can::disable_interruptwith the Interrupt enum (not the Interrupts flags), the incorrect bits in the interrupt enable register are modified.
Example
The code
fn read_ier() -> u32 {
    unsafe { core::ptr::read_volatile((0x4000_6400_usize + 0x14_usize) as *const u32) }
}

can.enable_interrupt(bxcan::Interrupt::Fifo0MessagePending);
rprintln!(
    "Interupt enable register set using single interrupt: {:#010x}",
    read_ier()
);
can.disable_interrupts(bxcan::Interrupts::all());
rprintln!("Interupt enable register cleared: {:#010x}", read_ier());
can.enable_interrupts(bxcan::Interrupts::FIFO0_MESSAGE_PENDING);
rprintln!(
    "Interupt enable register set using interrupt flags: {:#010x}",
    read_ier()
);

prints
Interupt enable register set using single interrupt: 0x00000001
Interupt enable register cleared: 0x00000000
Interupt enable register set using interrupt flags: 0x00000002

Why this happens
The Can::enable_interrupt and Can::disable_interruptfunctions convert the Interrupt into a bit flag. However, the Interrupt discriminant values are not flags, they are bit positions. In the above example, Interrupt::Fifo0MessagePending (defined as 1) gets converted into Interrupts::TRANSMIT_MAILBOX_EMPTY (also defined as 1), and the transmit mailbox empty bit gets set in the interrupt enable register.
The fix
This change simply changes the discriminant values of Interrupt so that they have the same values as the corresponding Interrupts flags. The modified code correctly enables interrupts in my tests so far.

Thanks!
bors r+


support for visual/global/lookat

Is your feature request related to a problem? Please describe.
Better specification of default camera to control the opening view of a model. Currently, we only support visual/global/azimuth and visual/global/elevation
Describe the solution you'd like
support visual/global/lookat
Additional context
Say you have a large house scene and you want to control which room should we start in. Or maybe it should be the entrace. This feature will enable this capability.

This is vale controlled by <statistic center/>

Documentation

Exactly what I needed. Thanks @yuvaltassa
Should we add a line in the doc next to visual/global/elevation/azimuth

The lookat point can be specified using statistic/ center


Darkside treestate support

Closes #389
How to test this:
compile the code with make
then start darksidewalletd
./lightwalletd --log-file /dev/stdout --darkside-very-insecure  --darkside-timeout 1000 --gen-cert-very-insecure --data-dir . --no-tls-very-insecure
reset the chain
grpcurl -plaintext -d '{ "saplingActivation" : 663150, "branchID" : "2bb40e60", "chainName" : "main"}' localhost:9067 cash.z.wallet.sdk.rpc.DarksideStreamer/Reset
stage the tree state
grpcurl -plaintext -d '{
  "network": "main",
  "height": "663150",
  "hash": "0000000002fd3be4c24c437bd22620901617125ec2a3a6c902ec9a6c06f734fc",
  "time":<PHONE_NUMBER>,
  "saplingTree": "01ec6278a1bed9e1b080fd60ef50eb17411645e3746ff129283712bc4757ecc833001001b4e1d4a26ac4a2810b57a14f4ffb69395f55dde5674ecd2462af96f9126e054701a36afb68534f640938bdffd80dfcb3f4d5e232488abbf67d049b33a761e7ed6901a16e35205fb7fe626a9b13fc43e1d2b98a9c241f99f93d5e93a735454073025401f5b9bcbf3d0e3c83f95ee79299e8aeadf30af07717bda15ffb7a3d00243b58570001fa6d4c2390e205f81d86b85ace0b48f3ce0afb78eeef3e14c70bcfd7c5f0191c0000011bc9521263584de20822f9483e7edb5af54150c4823c775b2efc6a1eded9625501a6030f8d4b588681eddb66cad63f09c5c7519db49500fc56ebd481ce5e903c22000163f4eec5a2fe00a5f45e71e1542ff01e937d2210c99f03addcce5314a5278b2d0163ab01f46a3bb6ea46f5a19d5bdd59eb3f81e19cfa6d10ab0fd5566c7a16992601fa6980c053d84f809b6abcf35690f03a11f87b28e3240828e32e3f57af41e54e01319312241b0031e3a255b0d708750b4cb3f3fe79e3503fe488cc8db1dd00753801754bb593ea42d231a7ddf367640f09bbf59dc00f2c1d2003cc340e0c016b5b13"
}' localhost:9067 cash.z.wallet.sdk.rpc.DarksideStreamer/AddTreeState

obtaining the TreeState
grpcurl -plaintext -d '{ "height" : 663150 }' localhost:9067 cash.z.wallet.sdk.rpc.CompactTxStreamer/GetTreeState
removing a tree state
grpcurl -d '{ "height" : 663150 }' localhost:9067 cash.z.wallet.sdk.rpc.DarksideStreamer/RemoveTreeState
clearing all tree states from cache
grpcurl -d '{}' localhost:9067 cash.z.wallet.sdk.rpc.DarksideStreamer/ClearAllTreeStates
known issues
somehow the get tree state always attempts to call Zcashd
grpcurl -plaintext -d '{ "height" : 663150 }' localhost:9067 cash.z.wallet.sdk.rpc.CompactTxStreamer/GetTreeState
pacu@pacu lightwalletd % grpcurl -plaintext -d '{ "height" : 663150 }' localhost:9067 cash.z.wallet.sdk.rpc.CompactTxStreamer/GetTreeState
ERROR:
  Code: Unknown
  Message: zcashd did not return treestate

Yes, this definitely should be fixed, and it's not hard to do.
One suggestion I have, would it be okay if cash.z.wallet.sdk.rpc.DarksideStreamer/Reset clears the treestate, instead of having a separate ClearAllTreeStates gRPC? That gRPC currently clears all existing darkside state, so it would be simpler if that's how we clear all of the treestates.

I was thinking of TreeStates being reused across tests ...

Ah, I see; also I just noticed that you're already implemented this in this PR, so I'm totally fine with that. All this stuff is to make you're (wallet devs) lives' easier, so however you think it should work is what we should do!
I'll try to get to this (review what's here, rebase, make any additional tweaks) soon.

@LarryRuane I'm working on this again and I see that I'm not overriding the getTreeState method at the right level.
I'm returning the Tree state here
case "z_gettreestate":
		var heightStr string
		err := json.Unmarshal(params[0], &heightStr)
		if err != nil {
			return nil, errors.New("failed to parse gettreestate request")
		}

		height, err := strconv.Atoi(heightStr)
		if err != nil {
			return nil, errors.New("error parsing height as integer")
		}

		treeState := state.stagedTreeStates[uint64(height)]

		if treeState == nil {
			return nil, errors.New(fmt.Sprint(
				"there is no TreeState Staged for height \"",
				height,
				"\". Stage it using AddTreeState() first"))
		}

		return json.Marshal(treeState)

but this actually makes the real lwd be executed instead and the expected tree state format is the one returned by zcashd
ZcashdRpcReplyGettreestate struct {
		Height  int
		Hash    string
		Time    uint32
		Sapling struct {
			Commitments struct {
				FinalState string
			}
			SkipHash string
		}
		Orchard struct {
			Commitments struct {
				FinalState string
			}
			SkipHash string
		}
	}

shall I be attempting to override the compactStreamer gettreestate call instead by adding a
func darksideRawRequest(method string, params []json.RawMessage) (json.RawMessage, error) {
	switch method {
	/// other cases
        /// ...
	case "gettreestate":
		return nil, errors.New("Hello this is working")

instead?

@LarryRuane I had a super old lightwalletd repo! it dated when this repo was over on zcash-hackworks. I fixed that.
I consider it ready for review.

This looks good, Pacu, but 2 things, can you squash the latest commit, and also I just noticed that the commit message for the first commit seems to be wrong, can you check?

This looks good, Pacu, but 2 things, can you squash the latest commit, and also I just noticed that the commit message for the first commit seems to be wrong, can you check?

Done! thanks for catching that! My repo was quite broken and a rebase had mixed a bunch of stuff.


Build multiple secilc versions in our github actions

This doesn't do anything yet about supporting variable cascade behavior based on secilc version, but it does test against multiple versions, so we can 1. ensure we remain regression free across multiple versions and 2. Have testing infrastructure ready to go once we support variable behavior based on target secilc version.
The current testing only goes back to 3.0.  Ideally I'd like it to go back to 2.7.  See the third commit message for details.
The plan is to first merge this and get the improved test coverage.  Then we can add support for multiple secilc versions into cascade and pass the version information in an environmental variable from the github job to cargo test.  Then cargo test can build policies using the appropriate secilc version flags.  For if no environmental variable is supplied, cargo test will use a sensible default output target (3.0 seems like a good default).  Since cil features are generally backwards compatible, targeting 3.0 should work across all versions > 3.0 as well.


Is there a way to get an "Uint8ClampedArray" and create a sharp from it?

What are you trying to achieve?
I'm trying to do some transformations to get a Uint8ClampedArray  since I need it to run blurhash on it, a blur library you can find here, but to get the Uint8ClampedArray I need to run some transformation from the UInt8Array that I get from running .raw() function.
Is there a way where I can get a Uint8ClampedArray from the sharp library, and set a sharp from a Uint8ClampedArray? Since I need to run this backward to get a base64 image at the very end?
Have you searched for similar questions?
Yes, no luck so far, I was wondering maybe the creator or mantainers of the library can have a clue on this.
Are you able to provide a standalone code sample that demonstrates this question?
const { data, info } = await sharp('input.jpg')
  .raw()
  .toBuffer({ resolveWithObject: true });
data.toClampedArray() // This is the method or hack I'm looking for

Are you able to provide a sample image that helps explain the question?
No.

The Node.js Buffer type is already 8-bit unsigned so using something like Uint8ClampedArray.from(data) should "just work".

Hope this helped, please reopen with more details if not.

I can confirm this conversion is correct, thank you! However, not quite close enough to my needs. (Sorry to hijack this thread, but my problem also involves Uint8ClampedArray from sharp, so thought it good to continue here).
I need Uint8ClampedArray as an input to jsQR, and I'm running into the same issue as: https://stackoverflow.com/questions/52194980/in-node-js-why-isnt-my-qr-code-being-read-correctly
It seems one potential answer is given as:

Busy day... The "Malformed data passed to binarizer" error happens when height/width don't match up with the data you are passing. I might be wrong, but toRGBA8 returns 4 channels (rgba) where only 3 channels (rgb) are expected. – Thomas R. Koll Nov 6 '18 at 16:57

@lovell, do you happen to know if the Buffer output from sharp('input.jpg').toBuffer() is 4 channels (rgba) or 3 channels (rgb)? If rgba, is there a way to output rgb? Thank you!

Solved, it seems .raw() is simply needed in sharp('input.jpg').raw().toBuffer() to get everything to be compatible with jsQR. Thank you anyway, and hope this helps someone in the future.

For future reference, this is how to calculate a 4x4 blurhash under Node using sharp and blurhash:
const buf = await sharp('image.jpg').resize(64,64).ensureAlpha().raw().toBuffer()
const hash = await blurhash.encode(Uint8ClampedArray.from(buf), 64, 64, 4, 4)

I'm ignoring aspect ratio since it's all super blurry of course.

But also note you can use a data URI as a backdrop and get a JS-less blurry preview. You need an image, which is much bigger than a blurhash, but still smallish (200 bytes or so), and with some cut'n'pasting you only need to store a part of that (https://github.com/joe-bell/plaiceholder/discussions/226)


Write a C++ program to check lower case

Description
Write a C++ program to check lower case
Check if given character is lower case (a-z).
Input  : a
Output : Lower Case

How to contribute

Save the solution in program/CheckLowerCase.cpp file
Add CheckLowerCase.cpp file in check-lower-case folder

!assign


[ci] release

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
Releases
<EMAIL_ADDRESS>Patch Changes

#383 0ebc47e Thanks @delucis! - Fix edge case where index files in an index directory would end up with the wrong slug

size-limit report 📦

Path
Size

/index.html
13.15 KB (+0.01% 🔺)

/_astro/*.js
16.21 KB (0%)

/_astro/*.css
8.54 KB (0%)


The compatible platforms should include anthropic's Claude as it's an option in the config file itself

https://github.com/sigoden/aichat/blob/8b3bd63c3c976ee1081a751b1eb5f0dab6763c72/Argcfile.sh#L81

It is a list for OpenAI-compatible platforms, but Claude is not.
Cluade has its own separate command.
https://github.com/sigoden/aichat/blob/8b3bd63c3c976ee1081a751b1eb5f0dab6763c72/Argcfile.sh#L225


Enable configurable date format

Right now, it's hardcoded ISO style.
https://github.com/cntrump/hugo-notepadium/blob/3b406de2f4809fea3541a2a3d4ef873728cab959/layouts/_default/single.html#L7
It would be nice if there were a way to configure this from params without having to override the partial.

OK, a global var will be added:
[params]
dateFormat = "2006-01-02"

working on it...

Done.


TF Inverse operations fail on 2080ti GPU with a cublas error

I'm trying to use a loss function which needs an inverse op, but fails on tf 2.0 with the following error:

Other models/loss functions that don't use inverse ops train as expected using the GPU. Any pointers would be most appreciated.

@krsna6 ,
Thank you for reporting the issue, Can you share a simple and standalone code to reproduce the issue?
Provide the exact sequence of commands / steps that you executed before running into the problem.

@krsna6 ,
Hi, any update on the issue ?Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


ElasticSearch 7 Compatibility

Hi! Are there any plans to add ElasticSearch 7 compatibility?

@indiv0 We haven't planned to add an ES7 module yet. However, since the indexing module is pluggable in Conductor, this should be a straightforward implementation of the IndexDAO interface.
Please feel free to contribute your implementation as this would help out the community greatly.

@sfeitler-oracle Thanks for looking into ES7 integration. We can deprecate ES5 and replace with ES6/7 modules.
However, the cleaner option would be to move version specific code from this integration test to each module version. I.e waitForGreenCluster can be moved to EmbeddedElasticSearch implementations, say EmbeddedElasticSearchV5, which already has start, stop etc. methods, thereby keeping the tests independent of versions.

Ok, I'll investigate that.

@sfeitler-oracle where you able to progress on ES 7


Update kudu to aware of Go application

sample deployment log:
Counting objects: 3, done.
Delta compression using up to 12 threads.
Compressing objects: 100% (2/2), done.
Writing objects: 100% (3/3), 1.40 KiB | 0 bytes/s, done.
Total 3 (delta 0), reused 0 (delta 0)
remote: Updating branch 'master'.
remote: Updating submodules.
remote: Preparing deployment for commit id 'd0dd07d372'.
remote: Generating deployment script.
remote: Generating deployment script for Go Web Site
remote: Running deployment command...
remote: Handling Go deployment.
remote: Prepare workspace
remote: GOROOT D:\Program Files\go\1.4.2
remote: Creating GOPATH\bin D:\local\Temp\21322fab-f3c2-4cc3-a4ff-6fd58aef35d1\gopath\bin
remote: Creating GOPATH\pkg D:\local\Temp\21322fab-f3c2-4cc3-a4ff-6fd58aef35d1\gopath\pkg
remote: Creating GOPATH\src D:\local\Temp\21322fab-f3c2-4cc3-a4ff-6fd58aef35d1\gopath\src
remote: Creating D:\local\Temp\21322fab-f3c2-4cc3-a4ff-6fd58aef35d1\gopath\src\azureapp
remote: Copy source code to Go workspace
remote:
remote: -------------------------------------------------------------------------------
remote:    ROBOCOPY     ::     Robust File Copy for Windows
remote: -------------------------------------------------------------------------------
remote:
remote:   Started : Thursday, April 23, 2015 10:35:48 PM
remote:    Source : D:\home\site\repository\
remote:      Dest : D:\local\Temp\21322fab-f3c2-4cc3-a4ff-6fd58aef35d1\gopath\src\azureapp\
remote:
remote:     Files : *.*
remote:
remote: Exc Files : .deployment
remote:             deploy.cmd
remote:
remote:  Exc Dirs : .git
remote:             .hg
remote:
remote:   Options : *.* /NDL /NFL /S /E /DCOPY:DA /COPY:DAT /NP /R:1000000 /W:30
remote:
remote: ------------------------------------------------------------------------------
remote:
remote:
remote: ------------------------------------------------------------------------------
remote:
remote:                Total    Copied   Skipped  Mismatch    FAILED    Extras
remote:     Dirs :         2         0         1         0         0         0
remote:    Files :         1         1         0         0         0         0
remote:    Bytes :     3.4 k     3.4 k         0         0         0         0
remote:    Times :   0:00:00   0:00:00                       0:00:00   0:00:00
remote:    Ended : Thursday, April 23, 2015 10:35:48 PM
remote:
remote: Resolving dependencies
remote: .............
remote: Building Go app to produce exe file
remote: Copy static files for deployment
remote: KuduSync.NET from: 'D:\home\site\repository' to: 'D:\home\site\wwwroot'
remote: Deleting file: 'hostingstart.html'
remote: Copy web.config
remote:         1 file(s) copied.
remote: Finished successfully.
remote: Deployment successful.
To https://kudu@gotry3.scm.xiaowu1.antares-test.windows-int.net:443/gotry3.git
 * [new branch]      master -> master
Branch master set up to track remote branch master from https://kudu@gotry3.scm.xiaowu1.antares-test.windows-int.net:443/gotry3.git.

looks good


APIController cannot register on router

APIController cannot register on router.
is this behavior intentional?
user can only register their apps api controllers on the root api instance.
but we often have a common app route on every app imo.
now:
PAYMENT_PREFIX = "/payment/"

@api_controller(PAYMENT_PREFIX)
Class SomePaymentController1:
    pass

@api_controller(PAYMENT_PREFIX)
Class SomePaymentController2:
    pass

prefer:
@api_controller()
Class SomePaymentController1:
    pass

@api_controller()
Class SomePaymentController2:
    pass

payment_router.add_controller("", SomePaymentController1)
payment_router.add_controller("", SomePaymentController2)

api.add_router("payment", payment_router)

@zhoujingryan this is not supported at the moment

Also checkout this out https://github.com/eadwinCode/django-ninja-extra/discussions/162 if that helps

It'd make a lot of sense to be able to register them on routers as well as the base api.
There is a Router class in ninja_extra after all, and you can use it like a regular ninja router, but you can only attach normal function based views to it and not the ninja_extra controllers.
The workaround for me is to use the desired full path in the controller definition. But of course if I change the routers I have to manually edit those paths as well.

I will look into this


Disable implicit integer conversion warnings

The "llbuildSwift" target should match the setting for GCC_WARN_64_TO_32_BIT_CONVERSION of our other targets.

@swift-ci please smoke test

@neonichu No point of CI as it doesn't build the Xcode project.


Update test_error_message_includes_stage test

On behalf of @katsiapis

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

LGTM; errors look unrelated and same tests passed in non-actions suites.

I really don't think we should be breaking the fusion here. The point of this test was that we implicated the right stage despite fusion.

I really don't think we should be breaking the fusion here. The point of this test was that we implicated the right stage despite fusion.

Should I revert it?


Chipmunk not closing properly

Prerequisites
affected version: 2.3.1
make sure to check this before filing an issue:

[x] I am running the latest version
[x] I checked to make sure that this issue has not already been filed

Describe the issue
What did not work as expected?
Chipmunk is not closed properly after deleting the .chipmunk and running it for the first time.
Steps to Reproduce
Please provide detailed steps for reproducing the issue.
Remove the .chipmunk folder. (e.g rm -rf .chipmunk)
Start Chipmunk. rake start
Close Chipmunk.

It's relevant only for developing mode (starting with rake dev or npm run electron), because in developing mode chipmunk starts without launcher.


Update Makefile, add multi-arch build.

This change is part of the issue https://github.bus.zalan.do/teapot/issues/issues/3283

:+1:

👍🏽


about the better prediction error

Hi, Hugo Chan, nice to meet you, I am recently focused on this technology which might help us to get to know more about our end-user's thoughts, I read about the MIT paper, they claim their prediction error is 1.71cm on mobile and 2.53cm on a tablet.
And in this repo's README, it says" better validation error (2.19 cm vs. 2.514 cm)", is this the comparison to the MIT <eye_tracking_for_everyone> result?
Thanks!

Hello @amdgigabyte , the validation errors mentioned in README are on the smaller dataset instead of the origin dataset used in the paper.

@hugochan so, the result 2.19cm is on mobile or on a tablet?

@amdgigabyte Since I am not the person who created this smaller dataset, I am not 100% sure. BTW, just found the official repo was made public, you might want to check out this.

I had already checked out the original repo. But at present, the original dataset cannot be downloaded, so I have to find other datasets.
o(╥﹏╥)o


reject request if --require-auth but missing id-claims

Fixes CHRON-395.
PR Checklist
Please complete this checklist after opening your PR

[x] I have updated documentation as necessary
[x] I have updated helm chart(s) if needed
[x] I have added tests if needed
[x] All new and existing tests are passing
[x] Any breaking changes are clearly flagged and documented
[x] I’ve included a link to any relevant ticket(s)

What's the relationship to whether or not the user provides --id-claims with this?

If they don't provide it, the default is iss sub. It'd interact in that, if a JWT is now failing due to --require-auth, they could choose different ID claims fields that are present in the JWT. However, choosing different fields has the effect of changing the user's Chronicle identity.

If they don't provide it, the default is iss sub. It'd interact in that, if a JWT is now failing due to --require-auth, they could choose different ID claims fields that are present in the JWT. However, choosing different fields has the effect of changing the user's Chronicle identity.

Thanks for confirming.


Streaming TextureView

Is it possible to stream a specific TextureView/SurfaceView rather than streaming my entire display?
I am not sure where to start. Can someone point me in some direction? Thanks!

Hello,
Without modify the library it is not posible.
I'm not sure if it is working but you can stream an android view using AndroidViewFilterRender (You can stream display and set this filter with full size using setSize(100,100)). I recommend you test it but maybe it is not working because SurfaceView and TextureView are specials views and get frames using canvas not always work for it.
If you want stream SurfaceView or TextureView you need a proper way to extract frames from it and render another Surface (VideoEncoder inputSurface) with that frames but depend of the way you are doing it it is not always possible.


Correct copy-pasted mispelling of title

(this was causing a fair few pages to just have blank titles in browser tabs)

Also cheers for the shoutout at tonights meetup :P


Apply fixes to M1/MacOS related stack allocation to all relevant spots

#5928 applied fixes for supporting M1/MacOS but missed places. This applies the necessary changes to all relevant spots.

bors merge


Shader behaves incorrectly

Godot version
4.0.stable
System information
Windows 11, GLES3, AMD RX 580 (4 GB), AMD Adrenalin 23.2.2
Issue description
I have two palette swap shaders (this one and another simpler one I found by googling "godot palette swap shader"). They behave correctly in 3.5.1.stable and 4.0.stable when using the Vulkan renderers, but do not work correctly at all in the GLES renderer on 4.0. The first shader doesn't change, and the second shader sets the entire sprite to be a color which is blended between the two colors I have in my palette texture.

Shaders rendering correctly when using Mobile renderer

Shaders rendering incorrectly when using Compatibility renderer
Steps to reproduce

Download reproduction project
Toggle between compatibility and either Mobile or Forward+ renderer
Witness shaders behaving incorrectly

Minimal reproduction project
broken shaders.zip

Something that I noticed - when I open up the reproduction project, it lets me know that glow is not supported when using Compatiblity mode. This shows up only in the reproduction project and not in my own project for some reason.

Both shaders are fixed if you change the palette uniform to
uniform sampler2D palette : filter_nearest, repeat_disable;

Yes there's still an inconsistency between the filtering modes in compatibility mode I think, forget if it's been changed

Both shaders are fixed if you change the palette uniform to
uniform sampler2D palette : filter_nearest, repeat_disable;

Doing this fixes the issue that I had, thank you!

Yes there's still an inconsistency between the filtering modes in compatibility mode I think, forget if it's been changed

That would be neat to know. I think that having some form of consistency or place where this is documented would be helpful for writing shaders.

Oh no this is backwards, the default filtering is now the same as vulkan, so this is because of difference with how 3.x does it, previously the compatibility renderer used the old defaults

My bad the issue is not there any longer, the default filtering is linear with mipmaps and repeat enableld.


mdc-data-table Pagination Demo

Pagination demo shows only 3 items despite apparent 35 items in array.  Also does not update items when controls for page size or forward/back.

The demo does not fetch any data. The real app should listen to navigation events and load data accordingly.