#UI - sort comment order

It just a little improvement :
is ti possible to invert the order of comment in order to have the newest on top ?
if it's just a value to modify in one config file witch one it is ?
Thanks in advance
Personnaly we use a lot the comment to precise the advancement of task, so after some comment we lose so time to scroll to the end.
And maybe later on devellopement, one usefull function (when we are 3-4 to work on one task) will be the able to answer to one particular comment. But sorry, I don't know how to code, so I cannot give help for this

With the last development version, it's possible to choose the sorting direction for comments.

Merci :-)


precompiled Rocm binary can't generate image

I've tried using the Rocm precompiled binary because I have a 7900xtx and the result is the image below.
I also tried compiling but received a lot of errors. I also have Rocm 6.1 installed if it matters.
Can you please tell me what I'm doing wrong?
This is the command I've used in windows 10 power shell:
.\sd.exe -m .\OpenDalleV11.safetensors -o test.png --cfg-scale 7 -H 512 -W 512 --steps 40 -b 1 --type q8_0 --vae .\sdxl_vae.safetensors --prompt "A lovely cat" -v
This is the log:
Option:
n_threads:         12
mode:              txt2img
model_path:        .\OpenDalleV11.safetensors
wtype:             q8_0
vae_path:          .\sdxl_vae.safetensors
taesd_path:
esrgan_path:
controlnet_path:
embeddings_path:
stacked_id_embeddings_path:
input_id_images_path:
style ratio:       20.00
normzalize input image :  false
output_path:       test.png
init_img:
control_image:
clip on cpu:       false
controlnet cpu:    false
vae decoder on cpu:false
strength(control): 0.90
prompt:            A lovely cat
negative_prompt:
min_cfg:           1.00
cfg_scale:         7.00
clip_skip:         -1
width:             512
height:            512
sample_method:     euler_a
schedule:          default
sample_steps:      40
strength(img2img): 0.75
rng:               cuda
seed:              42
batch_count:       1
vae_tiling:        false
upscale_repeats:   1
System Info:
BLAS = 1
SSE3 = 1
AVX = 1
AVX2 = 1
AVX512 = 0
AVX512_VBMI = 0
AVX512_VNNI = 0
FMA = 1
NEON = 0
ARM_FMA = 0
F16C = 1
FP16_VA = 0
WASM_SIMD = 0
VSX = 0
[DEBUG] stable-diffusion.cpp:147  - Using CUDA backend
ggml_cuda_init: GGML_CUDA_FORCE_MMQ:    no
ggml_cuda_init: GGML_CUDA_FORCE_CUBLAS: no
ggml_cuda_init: found 1 ROCm devices:
Device 0: AMD Radeon RX 7900 XTX, compute capability 11.0, VMM: no
[INFO ] stable-diffusion.cpp:171  - loading model from '.\OpenDalleV11.safetensors'
[INFO ] model.cpp:737  - load .\OpenDalleV11.safetensors using safetensors format
[DEBUG] model.cpp:803  - init from '.\OpenDalleV11.safetensors'
[INFO ] stable-diffusion.cpp:182  - loading vae from '.\sdxl_vae.safetensors'
[INFO ] model.cpp:737  - load .\sdxl_vae.safetensors using safetensors format
[DEBUG] model.cpp:803  - init from '.\sdxl_vae.safetensors'
[INFO ] stable-diffusion.cpp:194  - Stable Diffusion XL
[INFO ] stable-diffusion.cpp:200  - Stable Diffusion weight type: q8_0
[DEBUG] stable-diffusion.cpp:201  - ggml tensor size = 400 bytes
[DEBUG] clip.hpp:171  - vocab size: 49408
[DEBUG] clip.hpp:182  -  trigger word img already in vocab
[DEBUG] ggml_extend.hpp:990  - clip params backend buffer size =  125.22 MB(VRAM) (196 tensors)
[DEBUG] ggml_extend.hpp:990  - clip params backend buffer size =  710.31 MB(VRAM) (517 tensors)
[DEBUG] ggml_extend.hpp:990  - unet params backend buffer size =  2925.36 MB(VRAM) (1680 tensors)
[DEBUG] ggml_extend.hpp:990  - vae params backend buffer size =  94.47 MB(VRAM) (140 tensors)
[DEBUG] stable-diffusion.cpp:323  - loading weights
[DEBUG] model.cpp:1389 - loading tensors from .\OpenDalleV11.safetensors
[INFO ] model.cpp:1535 - unknown tensor 'cond_stage_model.logit_scale | f16 | 1 [1, 1, 1, 1, 1]' in model file
[INFO ] model.cpp:1535 - unknown tensor 'cond_stage_model.text_projection | f16 | 2 [768, 768, 1, 1, 1]' in model file
[DEBUG] model.cpp:1389 - loading tensors from .\sdxl_vae.safetensors
[INFO ] stable-diffusion.cpp:422  - total params memory size = 3855.36MB (VRAM 3855.36MB, RAM 0.00MB): clip 835.53MB(VRAM), unet 2925.36MB(VRAM), vae 94.47MB(VRAM), controlnet 0.00MB(VRAM), pmid 0.00MB(VRAM)
[INFO ] stable-diffusion.cpp:426  - loading model from '.\OpenDalleV11.safetensors' completed, taking 8.01s
[INFO ] stable-diffusion.cpp:446  - running in eps-prediction mode
[DEBUG] stable-diffusion.cpp:481  - finished loaded file
[DEBUG] stable-diffusion.cpp:1265 - txt2img 512x512
[DEBUG] stable-diffusion.cpp:1018 - prompt after extract and remove lora: "A lovely cat"
[INFO ] stable-diffusion.cpp:564  - Attempting to apply 0 LoRAs
[INFO ] stable-diffusion.cpp:1023 - apply_loras completed, taking 0.00s
[DEBUG] conditioner.hpp:325  - parse 'A lovely cat' to [['A lovely cat', 1], ]
[DEBUG] clip.hpp:311  - token length: 77
[DEBUG] ggml_extend.hpp:941  - clip compute buffer size: 1.40 MB(VRAM)
[DEBUG] ggml_extend.hpp:941  - clip compute buffer size: 2.33 MB(VRAM)
[DEBUG] ggml_extend.hpp:941  - clip compute buffer size: 8.58 MB(VRAM)
[DEBUG] conditioner.hpp:453  - computing condition graph completed, taking 155 ms
[DEBUG] conditioner.hpp:325  - parse '' to [['', 1], ]
[DEBUG] clip.hpp:311  - token length: 77
[DEBUG] ggml_extend.hpp:941  - clip compute buffer size: 1.40 MB(VRAM)
[DEBUG] ggml_extend.hpp:941  - clip compute buffer size: 2.33 MB(VRAM)
[DEBUG] ggml_extend.hpp:941  - clip compute buffer size: 8.58 MB(VRAM)
[DEBUG] conditioner.hpp:453  - computing condition graph completed, taking 26 ms
[INFO ] stable-diffusion.cpp:1147 - get_learned_condition completed, taking 185 ms
[INFO ] stable-diffusion.cpp:1168 - sampling using Euler A method
[INFO ] stable-diffusion.cpp:1172 - generating image: 1/1 - seed 42
[DEBUG] ggml_extend.hpp:941  - unet compute buffer size: 132.05 MB(VRAM)
|==================================================| 40/40 - 9.33it/s
[INFO ] stable-diffusion.cpp:1203 - sampling completed, taking 4.77s
[INFO ] stable-diffusion.cpp:1211 - generating 1 latent images completed, taking 4.79s
[INFO ] stable-diffusion.cpp:1214 - decoding 1 latents
[DEBUG] ggml_extend.hpp:941  - vae compute buffer size: 1664.00 MB(VRAM)
[DEBUG] stable-diffusion.cpp:888  - computing vae [mode: DECODE] graph completed, taking 0.60s
[INFO ] stable-diffusion.cpp:1224 - latent 1 decoded, taking 0.60s
[INFO ] stable-diffusion.cpp:1228 - decode_first_stage completed, taking 0.60s
[INFO ] stable-diffusion.cpp:1328 - txt2img completed in 5.58s
save result image to 'test.png'
This is the generated image:

Try a standard command to test if it really doesn't work.
One from the examples would be ok.

You can also try running
https://github.com/leejet/stable-diffusion.cpp/pull/291
This depends on the vulkan drivers


Add BLIP2 generated captions of selected objects

Pull Request Description
Added a new column named caption to all dataset splits. This column contains captions describing the regions of images enclosed within bounding boxes. These bounding boxes were generated using the BLIP2 model.
Please review and let me know if there are any changes needed.

Thanks! I would put the files to the repository root as they have a similar format to other bits of the dataset, and I feel that the cognitive load of having one more subdirectory for three smallish files is too much.

Also, since in our dataset there is only one bounding box per image, I propose to keeping only the primary keys (URLs) and the generated captions in these new files. That would prevent everyone from messing up with coordinates or questions.


ignore dep upgrade for opensearch and kubernetes

This PR ignoes upgrades to opensearch and kubernetes packages, as we might inadvertedly upgrade it too far and break compatibility.

duplicate #171
closing


Suppress noisy pull output from docker run commands

This is annoying noise in the CI logs.

Apparently this flag was only added to docker run very recently in version 23 https://docs.docker.com/engine/release-notes/23.0/ :-(


[Feature]: Deterministic burner accounts

Feature Request
Problem: New burner accounts are always created with a random keys and address, stored on the browser.
That causes many drawbacks:

Users need to be aware of their accounts' vulnerabilities, export, and store keys securely.
If the browser data is cleaned and the private key is lost, the account and assets are also lost.
Recovering one address requires the technical ability to dig into a block explorer and find the deployment transaction.
Increases support occurrences from users who don't know where their keys are and/or lost their accounts
Terrible user experience

We don't want to expose players to another security risk and we don't want players to lose assets.
From one seed phrase, Ethereum and Starknet wallets always create the same account address sequence wherever they are recovered. Understanding the importance of their seed phrase and storing them securely is something every user needs to understand and be comfortable with before starting to interact with on-chain games.
Proposed Solution
I propose implementing Deterministic Burner Accounts, from a a seed and an index, similar to Ethereum and Starknet wallet accounts.
Obviously, we don't want access to players' seeds and keys! This is the proposed workflow:

Request player to sign a specific message, known by the game developers
The signature is never stored or displayed to the player.
Use @scure/bip32 to create keypairs and addresses
Deploy, fund, and use accounts to play the game
Players do not need to securely store their keys
Browser data can be deleted at any time, for security concerns
Anywhere player connects their wallets, by signing the same message, their accounts and keys can be recovered and used.

This solution is already working on this branch, to be pulled soon:
https://github.com/rsodre/dojo.js/tree/deterministic-burners
My solution includes:

Passing optional seed and index to the burner manager create() method
Simulate account address generation with a new burner manager generateKeysAndAddress () method
Key derivation implementation using @scure/bip32
Storing the account index, master account, and metadata (signed message contents) to the burner storage

Alternatives
None, currently we can only create random burner accounts.
Related Code

BIP-32 Hierarchical Deterministic Wallets details how to create accounts from a seed and index
Someone drafted a proposal for Account keys and addresses derivation standard on Starknet
ArgentX uses this technique for key derivation.
how

Additional context
No response
If the feature is accepted, would you be willing to contribute it?

[X] Yes I would be willing to contribute

This solution is already working on this branch, to be pulled soon:
https://github.com/rsodre/dojo.js/tree/deterministic-burners
If this feature is accepted, please assign it to me.


Fix URL to indexer/#search-assets in Retrieve Assets Information section

Currently this link navigates to https://developer.algorand.org/docs/features/features/indexer/#search-assets which 404s (notice the extra "features" in the url.)
I have not tested this because I am not sure how.

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


Add more support for function parameters

Every parameter has the type 'word'. Make it able to be other types as well.

@tthuma1 Could you please elaborate on the objective a bit? I would like to work on this issue if possible

@Starkl7 Sorry for the late reply. I have been out of the loop with this project for quite a while, so I don't remember anymore what was meant by this. Maybe Abdur can give you some guidelines.


fix: transactions filter in mobile viewport

Closes
Description

Current Tasks

[ ] (task 1)
[ ] (task 2)
[ ] (task 3)

Codecov Report

Merging #134 (b54c6eb) into main (091581f) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head b54c6eb differs from pull request most recent head 240ca89. Consider uploading reports for the commit 240ca89 to get more accurate results

@@          Coverage Diff          @@
##            main    #134   +/-   ##
=====================================
  Coverage   9.32%   9.32%
=====================================
  Files         65      65
  Lines        858     858
  Branches     297     297
=====================================
  Hits          80      80
  Misses       777     777
  Partials       1       1

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 091581f...240ca89. Read the comment docs.


Fix Wire.elements-by-name

Before this little code:
$driver.url("http://google.de");
my ($input) = $driver.elements-by-name('q');
failed like:
Type Array does not support associative indexing.
in block  at /home/froggs/dev/p6-selenium-webdriver/lib/Selenium/WebDriver/Wire.pm6 line 454
in method _elements at /home/froggs/dev/p6-selenium-webdriver/lib/Selenium/WebDriver/Wire.pm6 line 453
in method elements-by-name at /home/froggs/dev/p6-selenium-webdriver/lib/Selenium/WebDriver/Wire.pm6 line 489

:+1:


fixed up the issue of having to insert the inline Paystack script tag…

Added an ngAfterViewInit() function to print the script tag for Paystack Inline Standard. To reduce the stress of adding the script tag after npm install.

Thanks for contributing, I'd take time to look at this and get back to you

No problem. Thanks

There is an examples folder containing an Angular project where you can test your changes. Run node try-example on the main directory. This will copy your changes there so you can test it. Remember to remove the inline paystack script in the index.html

Done! I did all what asked. You should be able to merge now.

Alright. I'd check it and merge

Hello Sirbootoo
I tried running it but it failed. Inserting the script in the module hinders the paystack script from loading on time. Hence the component view loads before paystack's script making the button not to work. It even fails to load when network is slow.
I think its better in the html. Thanks for contributing but I'd have to close this pull request


Adapter doesn't update cards when notifyDataSetChanged is called

I tried using BaseAdapter and ArrayAdapter both, but calling notifyDataSetChanged on adapter object never calls the getView method of the adapter even though the data has changed and the count is >0
The same adapter and view works fine with a normal ListView
The use case is, the card's view has a button, on click of it, some text changes in the cards' source array and after that I am calling notifyDataSetChanged on the adapter associated, but am never seeing any update on the view.
Have checked by Logging and Debugging that control never goes in getView if I am using SwipeFlingAdapterView. It works well when I use a ListView.

This solution to this issue serves my purpose : https://github.com/Diolor/Swipecards/issues/29
But please add this in the library doc itself


Update asset pipeline documentation to explain components

The asset pipeline can attach components to objects. We should document this in https://github.com/AmbientOrg/Ambient/blob/main/docs/src/asset_pipeline.md, and make it clear that these components should be used for static data, not dynamic data: you should add max_health, but not health.
The reason for this is that hot-reloading will replace those components, so we want to prevent accidents from occurring where gameplay components are replaced when they shouldn't be.

Commit mentions static data for components, but doesn't explain how to actually use them. Will document as part of #66 .

I've updated the book to mention this, but it needs a bit more TLC I think


Custom title pr 968

Fixes Issue #964
The creator of PR #968 never came back to fix a merge conflict although the changes were very good. Instead I created a new branch, merged their changes there and fixed the conflicting files (package-lock.json). I have also merged in all current changes since the PR was from January. Prettier has adjusted some of the files, but the main changes were to the user profile.

Well spotted Tom, updated it now.


Fix for targeting all li's on page

Fix for issue where all first li's across the page get an active class when running showTabs

Hi @AwlO !
Could you explain what problem does your solution fix?

When you have more li's with the class active on your page, it doesn't take into account which are from tabs and which aren't. This solutions fix that, so only search for active li inside the tabs.

Late response, but yeah, exactly as @german95n said

I just came up with a simular fix.


Feature/distributed moments and covariance

implementation of https://github.com/IntelPython/daal4py/issues/51 and https://github.com/IntelPython/daal4py/issues/52

Please also add the distributed examples to docs:

doc/examples.rst
doc/algorithms.rst
doc/scaling.rst


Broken - Google changed TTS API

Hi, thank you for this nice project, I hope it hasn't been abandoned.
Recently, Google changed something with their tts service which breaks functionality.
https://github.com/noelportugal/google-home-notifier/issues/46
https://github.com/zlargon/google-tts/issues/17
It should be easily fixed...

Please fix this ... I would love this project to be maintained .. great interface ... would be amazing if this worked :)


[Bug] Block Internet switches don't seem to work

The problem
If I use the Asus Router app to block internet to a client, I'll get a "block internet" entity in HA. These entities don't seem to work or update, though. If I turn off Parental Control, the entities are no longer provided by the integration as well. If I block a client in the Asus app, it doesn't reflect in HA, and if I turn it on in HA using the entity, it doesn't reflect in the Asus app.
Your device model
ET12
Firmware type
Stock
Firmware version
<IP_ADDRESS>.388.24542_g719e902
Integration version with the issue
0.28.0
Method of the integration installation
HACS
What version and type of Home Assistant installation do you use
2024.1.2
Is there anything useful in the logs?
This error originated from a custom integration.

Logger: custom_components.asusrouter.switch
Source: custom_components/asusrouter/switch.py:217
Integration: AsusRouter (documentation, issues)
First occurred: 13:07:27 (6 occurrences)
Last logged: 13:12:23

Unable to set state with an exception: sequence item 0: expected str instance, NoneType found

Diagnostics information
No response
Additional information
No response

Hello. Sorry for the late reply. Let's try to sort it out, so the issue can be fixed
When you set the block from the mobile app, does it change in HA to the state unknown?

Or does it become unavailable? Also, any change of the state in HA can take up to 30 seconds depending on the update interval of the integration

In any case, it is supposed to work from HA and change the state in the mobile app.
Can you please turn on debug logging and share the part of the log when you try to set

I've been tooling around this morning and I think it's working correctly, but I was missing a piece.
The switch.zenwifipro_parental_control needs to be on.
Blocking a device would cause an entity to be created in HA for that device switch.device_block_internet. This entity would only be available when the block was active, and trying to turn the block off through HA would cause it to temporarily turn off, then turn right back on. Turning the block off in the Asus app or GUI would lead to "this entity is no longer provided by the integration" in HA.
What I did is add everything I want to block to parental control-schedule block in the Asus app with no time scheduling. The switch.device_block_internet entities immediately came back, and when I flip them in HA it is shown in the Asus app/GUI, and visa-versa. Adding the devices to parental control is key, I didn't realize Asus groups internet blocking under parental controls.

The new release is already available in HACS and should solve this issue. In case if the issue persists, please, reopen it


Kubernetes Follow Logs command should invoke in new terminal

The shared terminal might be running another task. Additionally, since it's possible for the command to be run a long running task in the shared terminal, which could block future tasks. We should change the follow logs command to use a new terminal.

since I did the same for the Kube events, I can take this up once the other PR is merged.

@kameshsampath thanks for the offer to help - I'm almost finished with a solution to this :)

no worries.


[stable/percona-xtradb-cluster] Added affinity to the StatefulSet

What this PR does / why we need it:
This PR gaves posibility to add affinity and anti-affinity rules for the StatefulSet and  effectivily manages pods based on node labels and topology keys.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @paulczar

/ok-to-test


PIP-31: Consumptional Fee Model

Abstract
This issue proposes an enhancement to the Pactus fee model by incorporating dynamic fee adjustments based on network conditions and transaction demands. By introducing the consumption fee, we can allow transactions with zero or low fees to be processed in the network. However, these transactions may take longer than usual to be confirmed.
Proposal: https://pips.pactus.org/PIPs/pip-31
Discussion: https://github.com/pactus-project/PIPs/discussions/172
Sub tasks

[ ] Add configuration consumptional fee
[ ] Calculating consumption when committed new block
[ ] Update query calculating the fee based on consumption fee
[ ] Detecting new account
[ ] Show consumptional configuration node in node information

@Ja7ad
I think these 2 sub-tasks can merge into one.
 Update query calculating the fee based on consumption fee
And
 Update query calculating the fee based on consumption fee


Fork, Commit, Merge - Easy Issue (Java)

Fork, Commit, Merge - Easy Issue (Java)
Create a Simple Greeting Application Using Java
Note: You don't have ask permission to start solving the issue or get assigned, since these issues are supposed to be always open for new contributors. The actions-user bot will reset the file back to previous state for the next contributor after your commit is merged. So you can just simply start working with the issue right away!
Navigate to the tasks/java/easy directory from the root of your project.
Then open a file called GreetingApp.java.
The goal of this task is to create a basic Java application that prompts the user for their name and then prints a greeting message.
For example:
Enter your name: John
Hello, John! Nice to meet you!

Acceptance Criteria:

The application must prompt the user to enter their name.
The application must print a greeting using the provided name.

To run the program, right-click anywhere in the Java file editor and select Run Java.
Alternatively, if you have the Code Runner extension installed, you can click the Play button in the top-right corner of the editor.
The program will run in your Terminal.
If the program works as requested in the task, you are ready to make a pull request!

To work with this issue, you need to have Java installed to your local machine.
Check out README.md for more instructions of installing Java and how to make a pull request.
Feel free to ask any questions here if you have some problems!
Also, kindly give this project a star to enhance its visibility for new developers!

please assign this issue to me sir, i can solve it easily

please assign this issue to me sir, i can solve it easily

Hi! As it says on the description, you don't need to ask permission to get assigned to these educational issues:
You don't have ask permission to start solving the issue or get assigned, since these issues are supposed to be always open for new contributors. The actions-user bot will reset the file back to previous state for the next contributor after your commit is merged. So you can just simply start working with the issue right away!


[Zeppelin 796] Duplicated notebook names should not be allowed

What is this PR for?
When a notebook is created/cloned/imported the title/name of the new notebook should not be duplicate of the existing notebook's title/name
What type of PR is it?
Improvement
Todos

Task
What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-796
How should this be tested?
Create a notebook with name 'Note', Notebook should allow user to create a note
When a user try to create/clone/import a notebook with name 'Note', Notebook should not allow user to create note.
Screenshots (if appropriate)
Screenshot of error message displayed during create/clone notebook

Screenshot of error message displayed during import notebook

Questions:
Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

I think its okay to have a duplicate name, if the noteId is different nothing bad will happen. What kind of case are you trying to solve here?

@anthonycorbacho : Lets say if i create a note with name 'Note'. If zeppelin allows me to create a new note with name 'Note', then after sometime when i search for one of this note, I am going to get confused here.

I see your point, but let say i want to import a note and i have a note with the same name, it will not allow me to do that and i feel like its a pretty bad user experience here. What do you think?

@anthonycorbacho : For import note function too, we will have same problem as i pointed out first. Anyway for import note too, any user can give a new name for importing notebook

@All : Please review this PR and let me know your comments

@rajarajan-g There are some conflicts in here. Could you handle them? :)

Ping

Now i have resolved the conflicts. Please review

Any updates ?

IMHO, automatically adding " (1)", " (2)" .. etc to a note name in case of duplicate name found
might be a better solution than disallowing import at all. A variation on this idea - adding " (import date/time)" suffix to note name would do too (only in case if dup name found).
It'll be then at user's discretion to rename notes appropriately - so they would still be allowed to import a note.


Disable codecov + configure pytest FutureWarnings

What does this PR do?
In the CI, we currently run the tests with the --cov option to get a code coverage report. However, as pointed out by @osanseviero in this PR (https://github.com/huggingface/huggingface_hub/pull/925), it is broken at the moment and only check coverage of the tests themselves. It has apparently already been discussed here (internal url) and here (internal url). The purpose of the PR is to simply remove coverage for now as it is broken and give unreliable information. I will create a separate issue to fix it and enable it again.
Also, this PR sets the -Werror::FutureWarning flag by default. This should lead to less friction as contributors will run exactly the same tests locally and on CI.

As a side note, I am personally in favor of code coverage in PRs. However since it's broken and it's less a priority to fix it, I find it preferable to disable it for now.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


conditional forwarders replicated in AAD DS

Please clarify if the conditional forwarders are replicated in AAD DS

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

ID: f23ba67e-5544-77f8-b6e0-1c7b653f68d9
Version Independent ID: 2639d2fc-fefd-f3ad-621d-cc27ebe78576
Content: Manage DNS for Azure AD Domain Services
Content Source: articles/active-directory-domain-services/manage-dns.md
Service: active-directory
Sub-service: domain-services
GitHub Login: @Justinha
Microsoft Alias: justinha

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

@Acenl12
I'm going to assign this to the document author so they can take a look at it accordingly
@Justinha
Can you please check and add your comments on this doc update request as applicable.

@Acenl12 sorry for delay. I created an internal work item to get this clarified. We are migrating repos so need to close this. #please-close


A comment on this sample

The sample makes use of Fiddler, so it might be nice to find a few words on how to set this tool up to match the experience shown at

@adriatic The use of Fiddler in the sample was merely for illustrative purposes to demonstrate that the JWT middleware will download the RS256 keys automatically from the Auth0 website.
Adding any more detail around Fiddler is outside of the scope of the sample.
Please refer to the Fiddler website if you require more information on how to configure it:
http://www.telerik.com/fiddler

@jerriep Please allow me to make something clear - with a very few exceptions, my comments are not meant to be a sign that I need additional clarifications, but rather that I am trying to point out what is likely to be perceived as insufficiently explained by an average visitor.
It is disappointing to find myself assessed as someone who needs information on Fiddler (I actually wrote a very similar tool long time ago). Alternatively, I would think that you do not find my attempted services as useful, so I will "cease and desist".

@adriatic Please feel free to continue contributing your thoughts. But understand that we need to assess your ideas and then based on that make a decision whether adding the information you suggested will contribute or distract from the tutorial.
In this case, - as pointed out previously - any discussion around Fiddler is outside of the scope of the quickstart. The screenshot you referenced is there merely to prove that the keys are being downloaded automatically from the Auth0 website. My thoughts are therefore that any further explanation of Fiddler will distract from the actual purpose of the tutorial.

It is disappointing to find myself assessed as someone who needs information on Fiddler

I have no idea as to your knowledge of Fiddler or the lack thereof. I base my response purely on your request "a few words on how to set this tool up to match the experience shown". Based on that my impression was that you - personally - require assistance on how to set this up. If you consider this an insult of your own abilities, then I apologise. It was not meant as such. It was meant as trying to assist someone who, in my view, clearly asked for assistance.

It was meant as trying to assist someone who, in my view, clearly asked for assistance.

That conclusion is incorrect: from the getgo, my participation was driven by my desire to help you with a view of an Auth0 novice, who has a very positive view on Auth0 and does not mind spending the time to type a lot, if such information could be of use. At no point would I even dare to use our "relationship" (one document writer to another) as a means to get technical support for me personally; that would not be ethical, and I have several friends on Auth0 support team as well as very friendly support team that I would use in such case.
Yes, we do have a miscommunication - my attempt to help was interpreted to be asking for help. For that I can only blame myself as the size of this error reflect very poorly on my communication skills - and consequently my ability to help.

But understand that we need to assess your ideas and then based on that make a decision whether adding the information you suggested will contribute or distract from the tutorial.

The possibility that my ideas could distract the flow of the tutorial (note that directly reflects on skills as a technical writer) completely demotivates me from offering a single word.

The reason why the entire Signature Validation section was added was because the feedback we were indeed getting on the quickstart was that people believed the signature validation was not done properly because they could not see any place where the signature of the JWT was being validated.
That section was added to prove that the signature was indeed being validated, and the Fiddler screenshot was added to prove that the RS256 keys were indeed being downloaded from Auth0 and they did not have to do it manually (as the JWT middleware on many other platforms require).
Since that section was added we have not gotten feedback again on possible lack of signature validation. So that side note clearly resolved that question in the mind of readers.
Yours is the first ever feedback I received on further clarification on configuring Fiddler. Should I get more feedback like this I will consider adding links to help developers get going with Fiddler. For now, based on the current feedback, I do not deem this necessary.
I therefore keep to my original assessment that adding any more information on Fiddler will distract from the intended purpose of the tutorial - which is to help developers configure Auth0 in their ASP.NET APIs.

It seems that I acted as a primadona - but that was not my intent and I apologize if that was your perception. Other than trying to help by stating my own interpretation of various texts, I do not have any other agendas. You are perfectly correct when saying

I therefore keep to my original assessment that adding any more information on Fiddler will distract from the intended purpose of the tutorial - which is to help developers configure Auth0 in their ASP.NET APIs.

and I fully respect that.

No worries :)

Thank you :smile:


Non-conforming refinement

Hey,
I'm having some issues in my code that I believe is related to some inconsistencies in the face neighbor data when I perform refinement on a non-conforming mesh.
If I look at the sizes at some of the face neighbor data structures formed during mfem::ParMesh::ExchangeFaceNbrData(), the sizes of the data different is different if I...
1.) Exchange data only after I refine
#include <iostream>
#include "mpi.h"
#include "mfem.hpp"

// 1.) Exchange data only after I refine
int main(int argc, char* argv[]) {
   MPI_Init(&argc, &argv);
   MPI_Comm comm = MPI_COMM_WORLD;
   int myrank;
   MPI_Comm_rank(comm, &myrank);

   // Create non-conforming mesh
   mfem::Mesh mesh(10, 10, mfem::Element::QUADRILATERAL, 1, 1.0, 1.0);
   mesh.EnsureNCMesh();
   mfem::ParMesh pmesh(comm, mesh);

   // Perform refinement through GeneralRefinement
   pmesh.UniformRefinement();

   pmesh.ExchangeFaceNbrData();
   std::cout << myrank << ": After refinement - face nbr elements size: " << pmesh.face_nbr_elements.Size() << std::endl;
}

gives me
0: After refinement - face nbr elements size: 20
1: After refinement - face nbr elements size: 20

2.) Exchange data before and after I refine
#include <iostream>
#include "mpi.h"
#include "mfem.hpp"

int main(int argc, char* argv[]) {
   MPI_Init(&argc, &argv);
   MPI_Comm comm = MPI_COMM_WORLD;
   int myrank;
   MPI_Comm_rank(comm, &myrank);

   // Create non-conforming mesh
   mfem::Mesh mesh(10, 10, mfem::Element::QUADRILATERAL, 1, 1.0, 1.0);
   mesh.EnsureNCMesh();
   mfem::ParMesh pmesh(comm, mesh);

   // Exchange face neighbor data
   pmesh.ExchangeFaceNbrData();
   std::cout << myrank << ": Before refinement - face nbr elements size: " << pmesh.face_nbr_elements.Size() << std::endl;

   // Perform refinement through GeneralRefinement
   pmesh.UniformRefinement();

   pmesh.ExchangeFaceNbrData();
   std::cout << myrank << ": After refinement - face nbr elements size: " << pmesh.face_nbr_elements.Size() << std::endl;
}

gives me
0: Before refinement - face nbr elements size: 10
0: After refinement - face nbr elements size: 10
1: Before refinement - face nbr elements size: 10
1: After refinement - face nbr elements size: 10

I think part of the problem is that the non-conforming refinement procedure as currently implemented does not seem to delete the face neighbor data (so the second ExchangeFaceNbrData() is a no-op).
Thanks for your help!
@kswartz14 @Dan2997925

Hi @junkudo,
Can you try adding a call to DeleteFaceNbrData() (this is a method in ParMesh) in the beginning of ParMesh::NonconformingRefinement()?
Let us know if this resolves the issue, so we can make a branch/PR to fix it.
Thanks!

Hey @v-dobrev,
Putting in DeleteFaceNbrData() seems to resolve the issue.
0: Before refinement - face nbr elements size: 10
1: Before refinement - face nbr elements size: 10
0: After refinement - face nbr elements size: 20
1: After refinement - face nbr elements size: 20

Thanks!

OK, great! I will create a branch with the fix in a moment.

Hey @v-dobrev, when can I expect this fix to be merged into the master branch?

I just merged it.


Animating opacity to a lower level removes the image.

If animating a layer with an opacity to a lower level, it removes the image instantly.
If writing code like this.
{image = "s01/D/evil_toy.png", opacity = 1, animate = {opacity = 0, delay = 450, duration = 100, trigger = Panels.Input.A}},

the image will just dissapear.
I found the root cause to be in the Panels module, where the following code is (https://github.com/cadin/panels/blob/d412f0056b8b1c3d6780f8e53e6b4be7fde87354/modules/Panel.lua#L393C7-L393C7):
if anim.opacity then
   local o = (anim.opacity - layer.opacity) * layerPct
   layer.alpha = o
   if o <= 0 then
      layer.visible = false
   else
      layer.visible = true
   end
end

The o is negative when fading to a lower opacity.

I guess the only thing actually needed for this to work would be:
if anim.opacity then
   local o = (anim.opacity - layer.opacity) * layerPct + layer.opacity
   layer.alpha = o
   if o <= 0 then
      layer.visible = false
   else
      layer.visible = true
   end
end

Fixed (https://github.com/cadin/panels/commit/9a38a5e913c64b3b20072ea1bf19f81c322a1ad5).
You can pull the latest to get the fix.
Thanks for your help.

Thank you for a nice framework! :)


HIP linker script fails due to empty HCC_HOME in FindHIP.cmake

In cmake I use hip_add_executable which is defined by FindHIP.cmake. I only have nvcc platform at the moment. However, a recent change to FindHIP.cmake leads to a wrong call of hipcc_cmake_linker_helper.
The HCC_HOME is empty, so hipcc_cmake_linker_helper mistakenly gets the next argument as the HCC_PATH, which is actually the object files, so nvcc does not get the object files anymore and fails:
# /usr/bin/cmake -E cmake_link_script CMakeFiles/bufferCopy.dir/link.txt --verbose=1 /libraries/hip/git-nvcc9.1/bin/hipcc_cmake_linker_helper     CMakeFiles/bufferCopy.dir src/bufferCopy_generated_bufferCopy.cpp.o -o bufferCopy -lcudart -lpthread -lrt
nvcc fatal   : No input files specified; use option --help for more information

To make it working again I had to define HCC_HOME outside and before the hip_add_executable macro in FindHIP.cmake:
if(NOT DEFINED(HCC_HOME))
  set(HCC_HOME "${HIP_ROOT_DIR}/bin/hcc")
endif()
###
set(CMAKE_HIP_LINK_EXECUTABLE "${HIP_HIPCC_CMAKE_LINKER_HELPER} ${HCC_HOME} <FLAGS> <CMAKE_CXX_LINK_FLAGS> <LINK_FLAGS> <OBJECTS> -o <TARGET> <LINK_LIBRARIES>")

Edit: added last line to code (must set CMAKE_HIP_LINK_EXECUTABLE with the new HCC_HOME).

That is definitely odd. We already have something similar already in place in FindHIP.cmake. See https://github.com/ROCm-Developer-Tools/HIP/blob/master/cmake/FindHIP.cmake#L557-L559. This should have worked fine in your case. Do you have a modified or an older version of FindHIP.cmake?

any news on this? We had to change our cmake code to workaround this issue with HIP, see cross-ref

I ran into the same issue with rocm 3.5.1.
I used HIP_ADD_EXECUTABLE inside a function so HIP_HIPCC_CMAKE_LINKER_HELPER was not properly set. This causes  causes unexpected error messages from hipcc_cmake_linker_helper like wrong permissions when it tried to execute a .o file in my case.
Is there a way to make the error message more from hipcc_cmake_linker_helper more useful in diagnosing this problem?

Same issue with 3.6.0.  As described above, we avoided HIP_CLANG_PATH not being set by changing the cmake function(...) that called HIP_ADD_EXECUTABLE to a macro(...)

@tdd11235813, apologies for the lack of response. Can you please check latest ROCm 6.0.2 (HIP 6.0.32831) to see if your issue has been addressed? Thanks.


Drawing sprites on iOS

Hi, today I had some time to test your engine an I really appreciate the effort, but I ran into a problem.
If I use the old version 6.2 I can easily draw sprites, but with version 8.1 nothing happens.
I tried all documentation hints and tutorials I could find.

I'm sorry, I assigned the sprite in the render method... my bad...

Indeed, the sprite loads asynchronously when created, so if you re-assign every render method to a new instance, it will never finish loading to be rendered. Your code should look something like this:
  // constructor
  Sprite sprite = new Sprite('player.png'); // starts loading the sprite
  this.myObj = new SpriteComponent.fromSprite(64.0, 64.0, sprite);

 // render
 this.myObj.render(canvas); // render will do nothing until the sprite is loaded

Also, consider using the new BaseGame in 0.8.x, as it deals with rendering and updating for you (simply add your components to the array via the add method.
I'm glad your are having progress with flame, if you have any more questions or suggestions, feel free to ask :)


1.0Ha sub-release not supporting multiple mappings

The 1.0Ha sub-release doesn't correctly read the country mappings and, in instances where they would be more than one country mapped, only the last one is read.
E.g.:
link = { eu4 = TUR v2 = TUR } # Ottomans
link = { eu4 = k_ottoman v2 = TUR } # Ottomans
link = { eu4 = k_anatolia v2 = TUR } # Anatolia => Turkey

The converter will completely discard the first two lines and generate a new country for the EU4 "TUR", assigning them a random flag, as same thing happens also, I guess, with CK2 name-to-tile file.

Have I to give a proof, here I upload the savefile, along with the mod and the log generated.
mod.zip
Furtherly going on, upon loading it into Vic2, the game refuses to let me select any country, in both bookmarks (1821 and 1836). It also seems to have re-populated Africa (I cleansed it in the EU4 savefile).

Right.  So the second problem (which I don't get myself but others have) can be fixed just by reversing (or unreversing rather) the order of the bookmarks in the bookmark file in the mod.  I'm not quite sure what to do with 1821, as I don't want to have it as the "default" if things aren't working with it but I do want to have one.  I seem to remember there being an 1821 mod for Vicky 2, I might look into how that did it and/or see if the author is still about to ask if we can just take bits.
The first is more complicated, I think it might be an issue with the new parser, but I'll be looking into it anyway.  Thanks for digging into the issue, there have been a couple of reports of flag oddness and I had no idea where to start looking!

I'm a bit confounded by the country mapping thing, but it should be easy enough to check they're getting read right (just drop a breakpoint at the end of mappers::CountryMappings::readRules() and see what made it into the list. Looking at the code, it should work (but if testing shows otherwise, it clearly doesn't).
Do note that you now need to make sure EU4 is capitalized in a mapping, and you need to use Vic2 (with the v capitalized) instead of v2.


OCIError: ORA-01704: string literal too long

Hi,
I'm receiving the following error when using the activerecord-oracle_enhanced-adapter and squeel:
OCIError: ORA-01704: string literal too long
I'm using ActiveRecord/Rails 3.0.9, activerecord-oracle_enhanced-adapter 1.3.2 and squeel 0.8.5.
To reproduce this error:

$ cat Gemfile
source 'http://rubygems.org'
gem 'rails', '3.0.9'
gem 'ruby-oci8'
gem 'activerecord-oracle_enhanced-adapter'
gem "squeel"
$ rails generate model Post content:text
$ rake db:migrate
$ rails console
$> p = Post.create(:content => "a")
$> p.content = "b" * 4001
$> p.save

The problem is that the value is longer than 4000 characters even though it is a clob that allows for more, and the value is placed directly in the SQL query as a string which is not allowed. The weird thing is that the create call with a value for content that is longer than 4000 characters works perfectly.
If I don't put squeel in the Gemfile, then the save above works perfectly.

This has been open for months and no new info provided. In the interest of cleaning up mah githubs, I'm gonna close this.


typo in error response parameter description

c outside backtick formatting

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

Verified the update.
@TErenberger Thanks!


bogus URL should be removed

Testing #10911
Not sure what the URL that initially appeared was from, but could that be empty instead since it's invalid?

Do you have a screenshot? I assume you're talking about the placeholder text? It's supposed to show you what a URL should look like

I was confused because the URL seemed random. Maybe it could say example or something?

Part of my confusion was I expected it to be filled in for me 🤔 from the terminal but maybe that's too much too ask 😅

I ran into the same today and the problem is

Copied a new server URL
Open VS Code, choose to add a server
The quick pick has a URL filled in, but not the one from my clipboard, and in @meganrogge 's case and some of my cases, the URLs are invalid
Pasting from clipboard updates the input box, meaning the original URL was not from clipboard (or something is wrong with the VS Code clipboard API)

I don't think we're pre-populating that input box anymore with that experiment (I cannot repro at all). @meganrogge or @rebornix - can you confirm if you're still seeing?

Re-assigning as this is a UI issue with regards to remote kernel connection and I believe this falls under kernel selection UI.

This has been fixed


[WIP] Add plugin API to add common page config tabs

Needs to be reopened against codevise/pageflow:master once codevise/pageflow#1164 and #18 are merged.
Let plugins add tabs to the configuration editors of all pages. This
can be used to store settings for cross page type functionality in the
page configuration.
REDMINE-16823

see codevise/pageflow#1168


add workflow to test postman collections against docker compose scripts

Add a GitHub Actions workflow to run Stargate Postman collections using the Postman CLI against backends started using docker compose scripts.
Checklist

[ ] Changes manually tested
[x] Automated Tests added/updated
[ ] Documentation added/updated
[x] CLA Signed: DataStax CLA

This initial version will run against the latest released docker images tagged v2. We can modify this if desired to build the current snapshot and generate local docker images to be used by the docker compose scripts.

Let's merge and then test once it's on the main..

agree. I have been testing it locally with https://github.com/nektos/act but would like to see how it does in GitHub.


ptex-bind: please add @gsakura as a maintainer

Hi @scott-wilson
Please add @gsakura as a maintainer to this repository.
I'd like to add him as a reviewer on https://github.com/vfx-rs/ptex-bind/pull/2 but the github UI didn't let me add him. Feel free to reach out to me on the ASWF slack or email if you have any private questions.
cheers!

I'll do you one better. I'll make sure you're an admin for this repo. Give me an hour or so. Just have a few things to do today.

@davvid you should have permission to add them as a maintainer. I think you should be able to add them to the @vfx-rs/ptex-maintainers group. If not, I'll see what else needs to happen to give you permissions for that.

I was able to add Garret to the project directly but unfortunately I was not able to add him to the ptex-maintainers group (not sure why ~ probably more permissions I imagine).
This is good enough for me, but let me know if there's something else we should do to get him added to the team instead. Adding him to the team does seem preferable.
Thanks @scott-wilson!


Multiple arguments in POSTGRES_INITDB_ARGS

I want to pass multiple arguments to initdb but looks like that's not working.
docker-compose.yml
version: '3.1'

services:
  db:
    build: .
    restart: always
    environment:
      POSTGRES_PASSWORD: mypass
      POSTGRES_INITDB_ARGS="--data-checksums --locale=ru_RU.UTF-8"

Dockerfile, required to add ru_RU.UTF-8 locale to image
FROM postgres:12
RUN localedef -i ru_RU -c -f UTF-8 -A /usr/share/locale/locale.alias ru_RU.UTF-8

Error:
db_1  | initdb: unrecognized option '--data-checksums --locale=ru_RU.UTF-8'

If I set POSTGRES_INITDB_ARGS to --data-checksums or to --locale=ru_RU.UTF-8 it's working fine.
That seems odd because if I test corresponding line from docker-entrypoint.sh it working fine
Code below is simply executed in bash console
POSTGRES_INITDB_ARGS="--data-checksums --locale=ru_RU.UTF-8"
eval 'initdb --username="$POSTGRES_USER" --pwfile=<(echo "$POSTGRES_PASSWORD") '"$POSTGRES_INITDB_ARGS"

Output
The files belonging to this database system will be owned by user "hatifnatt".
This user must also own the server process.

The database cluster will be initialized with locale "ru_RU.UTF-8".
The default database encoding has accordingly been set to "UTF8".
The default text search configuration will be set to "russian".

Data page checksums are enabled.
...

Ok I got it, it's all about quotes and docker-compose. As stated in compose docs.

There is no special handling of quotation marks. This means that they are part of the VAL.

So if I define my env variable in docker-compose.yml like
POSTGRES_INITDB_ARGS: --data-checksums --lc-numeric=en_US.UTF-8

or in .env file like
POSTGRES_INITDB_ARGS=--data-checksums --lc-numeric=en_US.UTF-8

everything work as expected.


Other efforts in this space

There is a really fast library for HPC dynamic/streaming/evolving graphs maintained by my research group at GT. You can definitely look to it for inspiration about performant code in this space.
http://www.stingergraph.com/index.php?id=introduction

Thanks a lot for the reference Jame :+1:
I will definitely look into it.


Fix few formulations

few changes, replicated in all versions where I found that mistake.

@microsoft-github-policy-service agree [company="default"]

@microsoft-github-policy-service agree company="default"


Support configuring KMS API endpoint for auto-unseal

Removed --auto-unseal-region which was not used
Added --auto-unseal-kms-key-region which was what was actually being used

Thanks for the PR and thanks for catching this!


Successful/Failed build notifications

It would be a really nice feature to be able to add a web hook to notify something like a slack channel that a build succeeded or failed. Right now I'm just checking s3 for the new log files manually. If there is already an easier way I'd love to hear it.
I tried to start working on this locally but I can't get everything to play nicely together. Are the docs on getting this running locally up to date? If I can get this running I will start working on it.
Thanks for the great work on this and empire btw.

I just merged https://github.com/remind101/conveyor/pull/30 which has some better instructions for bootstrap in dev and running the tests. Let me know if that helps!
I started on a branch that would add an events system here and we could provide various implementations for event handling (e.g. Slack). I didn't go very far with it, but might be good for reference.


 OK!
Testing regions that respond faster than 0.05 seconds:
The closest region is .

The script found the best servers from the region closest to you.
When connecting to an IP (no matter which protocol), please verify
the SSL/TLS certificate actually contains the hostname so that you
are sure you are connecting to a secure server, validated by the
PIA authority. Please find bellow the list of best IPs and matching
hostnames for each protocol:
Meta Services:  //
WireGuard:  //
OpenVPN TCP:  //
OpenVPN UDP:  //

Turns out, curl doesn't like timeouts below 1 second unless it's recompiled to not use the system resolver - see https://github.com/curl/curl/blob/master/lib/hostip.c#L700
This pull detects this case, and updates the maximum latency to 1 second so curl (and thus get_region_and_token) doesn't simply silently fail.
I tried doing some trickiness with ( bash -c 'sleep $maximum_allowed_latency && kill $$ & curl ...' ) but unfortunately it didn't work out too well.
When passed --connect-timeout 0.999 <IP_ADDRESS>, curl returns 28 if it doesn't accept the timeout, or 7 if it accepts the timeout but can't connect to <IP_ADDRESS>. I added -m 1 as a sanity check in case something even weirder happens on someone's system.

Would it be more elegant to count the number of results we get from scanning all regions, and if the count is 0 let the user know he should increase MAX_LATENCY? This would also cover the case where the user actually doesn't have any region in range.

Can you please also share the following information?

distribution
curl version
compilation flags (USE flags in case of Gentoo)

Would it be more elegant to count the number of results we get from scanning all regions, and if the count is 0 let the user know he should increase MAX_LATENCY? This would also cover the case where the user actually doesn't have any region in range.

Perhaps - however curl immediately choking on the sub-second latency setting and all regions having high latency are two separate outcomes that probably should have different descriptive output from the script.
I can add a check for nresults=0 but I think also leaving in the curl check makes sense.

Can you please also share the following information?

Gentoo (good guess!), net-misc/curl-7.72.0::gentoo  USE="-adns -alt-svc -brotli ftp -gnutls -gopher http2 -idn imap ipv6 -kerberos -ldap -libressl -mbedtls -metalink (-nghttp3) -nss openssl pop3 progress-meter (-quiche) -rtmp samba smtp ssh ssl -static-libs -telnet -test tftp -threads (-winssl)" ABI_X86="32 (64) (-x32)" CURL_SSL="-gnutls -libressl -mbedtls -nss openssl (-winssl)"
I think (but haven't tested) that adns is the relevant flag - but it's disabled by default in Gentoo so it's fair to assume that most/all Gentoo users will encounter this situation.

Yes, I had a feeling it was Gentoo. :laughing:
I am really trying to keep the scripts in this repo as simple as possible, so that anybody and read and modify. Gentoo users have advanced understanding of Linux and coding, and they will be able to find any bug, especially if they are already pointed in the right direction. If we just spit out a message if $count==0, this will cover all cases and it will also keep the code simple, and this change is actually required for all distributions (even the binary ones that have a compile all packages with all flags). I actually wanted to do this before the release but I had too much work to do.

I added a check for num(results) = 0, is there anything further you'd like me to change to resolve this pull?

I added a check for num(results) = 0, is there anything further you'd like me to change to resolve this pull?

This part is awesome:
# test if we got any valid results
if ! grep -q '[[:alnum:]]' <<< "$bestRegion"; then
	echo "No region responded within ${MAX_LATENCY}s, consider using a higher timeout"
	echo "Example: $ MAX_LATENCY=1 $0"
	exit 1
fi

I am going to test this out and merge it.
I am still not sure on this part:
# test if curl accepts timeouts below 1s - see https://github.com/curl/curl/blob/master/lib/hostip.c#L700
curl -s --connect-timeout 0.999 -m 1 <IP_ADDRESS> &>/dev/null
if [ $? -eq 28 ]
then
  echo "Your system's curl doesn't accept timeouts below 1 second"
  echo "Altering MAX_LATENCY to 1s"
  export MAX_LATENCY=1
fi

Gentoo has this bug opened since 2019: https://bugs.gentoo.org/699632
Maybe a good step would be to fix the Gentoo bug, instead of changing scripts used by all distributions.

Glad you like my nresults=0 test :)
I've noted that there's numerous places in the scripts where a Here String (see man bash) could significantly simplify things, I can send simpler pulls for those if you like.
I wouldn't hold my breath on Gentoo bugs - most are solved very quickly, but if the cases for and against are subtle or trivial to fix by the end user, it can take a good long time.
The curl test I've proposed is near-instantaneous and anyone unaffected won't even notice it happening - that's why I deliberately chose an IP address that would instantly fail and has zero privacy implications - but crucially will fail with a different exit code from the fault condition of interest.
More importantly, it provides direct feedback for afflicted users and automagically alters the test conditions to Just Work - which I believe plays directly to the simplicity requirement of these officially published scripts.
Consider what your My First OpenWRT user might see if their curl has also been built with this (arguably legacy) setting - almost instantly says all regions have too much latency while obviously not even checking - compared to the results with this simple and harmless test being present which simply says "your curl can't test <1 second, so we're now testing 1 second - here are your (valid) results".
Meanwhile, unafflicted users won't even notice the millisecond or few it takes to confirm that curl accepts sub-second timeouts, and will also receive their valid results in a (rather more) timely fashion.

PS: it's but a moment's trivial work for me to turn on the adns flag, but that's not my concern - I like PIA, I like connecting to PIA with shell scripts, and I want other PIA users to also have a nice time doing so.

I tested the code (among a lot of other stuff) and added this to master:
if [ -z "$bestRegion" ]; then
  echo ...
  echo No region responded within ${MAX_LATENCY}s, consider using a higher timeout.
  echo For example, to wait 1 second for each region, inject MAX_LATENCY=1 like this:
  echo $ MAX_LATENCY=1 ./get_region_and_token.sh
  exit 1
fi

Let's give this project time to mature and see if we have other Gentoo users with issues before adding too much code to handle exceptions that might not really cause bugs. Here is an example where a lot of exceptions have been handled (29398, 43885, 66162, 37354, 57859, 18245, 24447, 86433), however the bugs are over 15 years old and nobody spent time to clean the script ever since.
I will keep an eye opened to see if this pops up ever again. I feel the fix added now is simple enough and should satisfy all scenarios.

Consider what your My First OpenWRT user might see if their curl has also been built with this (arguably legacy) setting

Sadly we have to rewrite everything for OpenWRT.


Unpublish button text is overlapping on small screens

Hi!
Just a UI feedback that the buttons texts are overlapping, which make them unreadable, on small screens or when you reduce the window size, as shown on the following screenshot:

I think the text should be able to wrap when it doesn't have enough width to be in one line.

Thanks, I'll look into this. Unfortunately the strapi design system is not responsive yet which leads to issues like this.
I will see what I can do to fix this manually.

I think what would be ideal here is to change the text to just publish and unpublish with some sort of description text under it.
If you have any suggestions let me know.

This should now be fixed in the latest release


feat: Added Baby Ripple, Binance Coin & Monero

Please can you add this missing coins into library and release a new version, thanks 😄
@marksuurland

Please can you add this missing coins into library and release a new version, thanks 😄
@marksuurland

Done, some minor comments.
Thanks for your effort!

Please can you add this missing coins into library and release a new version, thanks 😄
@marksuurland

Done, some minor comments. Thanks for your effort!

Hello, thanks for approval :D . I made the rest of the changes to the comments.

@mlipovaca currently on a small holiday (back on Sunday) will publish the new version then!

nobody uses Monero, or Binance coin. why was this added?

In this fork we are trying to support more coins, and I think bnb/monero are validate reasons (when it comes to market cap) to be added

Hi @mlipovaca saw there is an issue in the cnBase58.js you added:


Local source

Hi @patricoferris ! I have made the changes as per your guidance. It works well now. Kindly review and advice on further changes.

Duplicate of #23


Inconsistent naming convention in JSON

I was reviewing the JSON at https://github.com/TimothyStiles/poly/blob/prime/data/puc19static.json and I found that there is an inconsistency in "Meta" and "Locus". In Meta, there is GenbankDivision field, and in Locus, there is GenBankDivision field. Poly should be consistent.

Not sure when fixed, but there is now not GenbankDivision in Meta, so closing.


fixed factors

Related Issuse

Issue #223

Closes: #223
Describe the changes you've made
The changes are as follows :

Modified the function Factors to show all the factors of a given number(s) which was initially showing prime factors which was basically the same code written for HCF finding also.
Modified the README.md with  the functionality of the code and  for LCM and HCF also.

Checklist:

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

Screenshots

Original
Updated

original screenshot
updated screenshot

So, Initially "Factors" does the same thing as the HCF "prime factors" does there was no new functionality and not all factors were shown in that. Just like the below Screenshot (here factors and hcf, shows the same result of prime factors only)

Here is the modifications :
For single input all factors will be shown like the screenshot below :

For multiple inputs all factors of individual input will be shown like this screenshot below :

It seems you changed the functionality of Factors button
Don't remove the previous logic for Factors as it is Calculating Prime Factors
Just change Text of factors button to Prime Factors
and
add a new button named factors which will calculate factors with your logic

Factor previously does the same thing which was done by it in hcf so the same code was written twice initially so I think there should not be another factor button but If its not a good idea i will change it.

Can you help, how to create new tag and add functionality to it?

No no, just create another button, which onclick calls another function

Unable to find the code written for these tags :/

Hey @sairish2001 Should I work on this one?

If I could get help 🥺

@keshavgbpecdelhi Ya thats what i stated as to help you
@sairish2001 Can this be a collaborative issue?

@keshavgbpecdelhi first of all update your repo it isn't updated

@keshavgbpecdelhi Ya thats what i stated as to help you
@sairish2001 Can this be a collaborative issue?

No this can't be collaborative issue

@keshavgbpecdelhi
refer to appjs/logicjavascript.js
[start](https://github.com/sairish2001/makesmatheasy.github.io/blob/b1e666d69bd7b58ae345751e7103297410689fef/appjs/logicjavascript.js#L389
But actually there's no need to change anything in these functions.
Just create new function___(){  } and a new Button in index.html then apply onclick to this button

@keshavgbpecdelhi Update ur forked repo
use
git checkout main
git fetch upstream(first add git remote add upstream <then this repo's link>
git merge upstream/main
git push origin main

This will make merge conflicts go away

Didn't understood what to do, first and then what! => git fetch upstream(first add git remote add upstream <then this repo's link>
I have to write this as it is? => git merge upstream/main

First of all add upstream using
git remote add upstream https://github.com/sairish2001/MakesMathEasy.git
Then do this
git checkout main
git fetch upstream
git merge upstream/main
git push origin main

This error I got after following the above steps :
$ git push origin main Total 0 (delta 0), reused 0 (delta 0) To https://github.com/keshavgbpecdelhi/MakesMathEasy.git  ! [remote rejected] main -> main (refusing to allow a Personal Access Token to create or update workflow .github/workflows/issue-labeler.ymlwithoutworkflow scope) error: failed to push some refs to 'https://github.com/keshavgbpecdelhi/MakesMathEasy.git'
@khareyash05 pls look into this 🙏🏻

@keshavgbpecdelhi I am sorry mate I dont know why this error is coming up

@sairish2001 I have done the required changes @here
If there is a need of anything or if I have to resolve the conflict, please look into this.
Thankyou 🙏🏻

@keshavgbpecdelhi, go through this https://jessehouwing.net/cannot-push-to-github-without-workflow-scope/, this may help you.

@shonali2600 I read the article, I think this is not related with the current issue we are facing here. Here we are having merge conflicts. If you have something related with that, it would be highly appreciated.
Thanks

And also this issue is already resolved by someone


perf: NetworkBehaviour.netIdentity cache: use bool instead of null check for significant performance improvement in 4k benchmark

Use is operator or cast netIdentityCache as (object) to avoid unity's == operator instead of adding new bool & some complexity to the code

Use is operator or cast netIdentityCache as (object) to avoid unity's == operator instead of adding new bool & some complexity to the code

yep, we can check if it is true null. we dont need to call GetComponent if it has been destroyed.
This stuff will also not be needed if we just call GetComponent  in awake instead of using lazy initialization

fixed in #2339


Update README.md to add full instructions on integration via Bird UI …

…Theme Admin by STUX.

Thanks!


Test pledge calculator ES6 code

I rule and have completed some work on Case Manager that's ready for review!
wanted to run #1696 in CI.

Did some linting and worked thru a couple of unforseen js issues. Smashing merge!


[0.12.x] Fix missing filters for get settings validator (#5480)

Backport
This will backport the following commits from master to 0.12.x:

Fix missing filters for get settings validator (#5480)

Questions ?
Please refer to the Backport tool documentation

Don't wonder, I tried out this back port tool locally by adding a PAT because I thought it will be easier, but no, actually it was some kind of weird first. Not sure if it wouldn't be easier to do it with git directly.

Should work now, so it can be merged. I think the rest of the failures are not related to my changes, because they are also present in the previous runs on the 0.12.x branch.


[Feature][build] Update maven build, the alert plugins need put in plugins dir when install

This is a sub task of https://github.com/apache/incubator-dolphinscheduler/issues/3049
Update maven build, put the alert plugins in the ${install_dir}/lib/plugin/alert/${plugin_module_name} directory (assembly build and rpm build).

Let me implement this

Sorry, I'm busy recently. I don't have time to do this task. I hope other partners can continue

close by #4259


⚠ Prueba_Error_500 has degraded performance

In 591f0cb, Prueba_Error_500 ($URL_ERROR_500) experienced degraded performance:

HTTP code: 200
Response time: 18939 ms

Resolved: Prueba_Error_500 performance has improved in 3fe3359 after 567 days, 6 hours, 4 minutes.


Fix make ship

set_user_id.py needs to run from $CARAVEL_ROOT, otherwise it gives a
warning which ends up in the mag2gds_caravel.tcl script, and causing
make ship to fail:
Error:  Cannot find top-level RTL caravel_user_project/verilog/rtl/caravel.v.  Is this script being run in the project directory?

@antonblanchard @RTimothyEdwards
I don't think this is the solution to the problem.
What the script is trying to do is find the predefined user_id with set_user_id.py -report
Normally, the commands are run in this order.
export USER_ID=01234ABD
make set_user_id
make gpio_defaults
make ship

I believe the problem is that you are trying to run make ship before you set the user_id.
I think the changes you implement will cause the default ID to be pulled from the default caravel.v definition which is definitely not what you want.

Alternate fix went upstream, so closing.


remove/change size limit in upload

Trying to upload Himme (5.2MG) gave this error. Our largest dataset is over 5 mb, but it can be divided to parts. Himme, however, is just over 2 and it would be good to upload it as a whole, and we've seen that there is no problem to run the matching on it.

I tried reviewing the logs sudo journalctl --unit=gunicorn >> ~/log.txt . The error seems unrelated (see  #97 ).

Some further digging with the assistance of ChatGPT narrowed it down to the NGINX configuration. The default there is 1MB. I added client_max_body_size 25M; in the HTTP block of the /etc/nginx/nginx.conf file.
@sinairusinek please test this fix.


Add support for AlternateNamesV2 (and other new files?)

Apparently a few new files have been introduced. We'll need to add support for those.

Support for AlternateNamesV2 was added in v1.5.1 (see #7 )
Thanks @rodoch! Good work!

Whoops, didn't mean to close. We still need to asses what other files and if/how to add support.


Link to Wikipedia entry on Ptah

Try out https://docs.readthedocs.io/en/stable/tutorial/index.html#triggering-builds-from-pull-requests

📚 Documentation preview 📚: https://ptah--3.org.readthedocs.build/en/3/

Pre-build doesn't work.


Anonymous function expression fails to compile

Description
The Serulian Guide indicates syntax for anonymous function expressions, but the compiler throws an error.
Reproduce

Visit https://serulian.io
Write the following in the embedded editor:

myFunc := () => 1

Click "Run"

Expected:
Compiles without error
Actual:
ERROR: /b/playground.seru:15:1:
  Unexpected token at root level: tokenTypeIdentifer

  myFunc := () => 1
  ^^^^^^^^^^^^^^^^^

This seems to extend to all resolve statements.

Resolve expressions are not allowed at the top-level; if you want to assign an anonymous function at the top-level, it must be created as a variable:
var<function<int>()> myFunc = () => 1

Only variables, functions and types are allowed at the top-level, to ensure random state is not executed on module load.

So you cannot export an immutable value from a module?

Currently, no, but this is a very good idea. Will add.

Solved by allowing constant variables to be declared at the top level in v0.2.0:
const myFunc function<void>() = function () { ... }


[ImgBot] Optimize images

This was done by ImgBot on my own fork, which I then immediately merged and turned into this manual pull request.
Below is reproduced ImgBot's own message on the content of this PR.

Beep boop.
Your image file size has been reduced by 30% 🎉

Details

File
Before
After
Percent reduction

/media/faq/disable-detect-maps.png
125.56kb
84.72kb
32.53%

/media/faq/options-panel.png
104.59kb
72.31kb
30.87%

/quicktype-logo.svg
15.86kb
15.64kb
1.36%

Total :
246.01kb
172.67kb
29.81%

📝docs | :octocat: repo | 🙋issues | 🏅swag | 🏪marketplace

Interesting! No, thank you for now.


Custom validation for .gif type

Hi,
My control allows upload gif, png, jpg file types. I want validate min width and min height if file type are png and jpg. With gif type, I dont want to implement that validation. How can I do it. Thanks

Use 'ngf-resize-if`


HTTP 203 podcast episode 8

@petele I was about to approve this, but it has some metadata I'm not familiar with. Should probably be you.

Cheers!


How to set color of bottom bar and its items?

Hello, just started checking this out and it seems really cool. However I can't seem to find out how to set the background color or item selected color? Thanks and continue the great work!

As of now, the colors are automatically determined according to the Material Design specification.
When you have only three items, the active item color will be the same as your primary color. Otherwise the items are white and the background will be your primary color.
I'll add methods for customizing these in the next version.

Thanks for the reply! However it seems that this is not happening. They are always with a white background and a black icon.

Now you can customize the background color for each tab when there's more than five of them.
Like this.


Explore native configuration inference for programmatic API

Almost every time I use Jackson to convert something to a POJO, I end up adding that POJO class to the @SerializationHint annotation. Can anyone tell me why this process is necessary (from an end-user point of view) and if it could be improved in the future, is it a really annoying process?
Why do Spring Native and Quarkus need to support third-party libraries instead of creating a standard support agreement?
Sorry for the newbie questions, but I think many others will ask the same, thanks for the amazing work.

Probably worth to explore later.

Hi, is there any updates? Thanks!

On Spring Native side no, but we are going likely to explore this in Spring Framework 6 and Spring Boot 3.

Spring Native is now superseded by Spring Boot 3 official native support, see the related reference documentation for more details.
As a consequence, I am closing this issue, and recommend trying your use case with latest Spring Boot 3 version. If you still experience the issue reported here, please open an issue directly on the related Spring project (Spring Framework, Data, Security, Boot, Cloud, etc.) with a reproducer.
Thanks for your contribution on the experimental Spring Native project, we hope you will enjoy the official native support introduced by Spring Boot 3.


Deletion of service/rc/pods within a namespace is unsuccessful through UI

**Rancher Version:**v.1.2.0-pre4-rc5
**Docker Version:**1.12.2
**OS and where are the hosts located? (cloud, bare metal, etc):**Ubuntu, Cloud
**Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)**Single node
**Environment Type: (Cattle/Kubernetes/Swarm/Mesos)**Kubernetes
Steps to Reproduce:

Create a Kuberentes(1.4.4) set up with infra catalog qa branch
Create service/rc/pods within a namespace "test"
Delete the service/rc/pod through UI

The service/rc/pods created within a namespace do not get deleted.
UI Error:
Error deleting
Cannot read property 'exitCode' of undefined

API error:
{"exitCode":1,"stdOut":"","stdErr":"Error from server: replicationcontrollers \"k8testrc1\" not found\n","error":{"Stderr":null}}

I have exactly the same issue with rancher 1.2 final.

Same 1.2.0
Is there a manual workaround I can employ to remove the unwanted pod outside of the UI?

You can still delete the pod via the Kubectl user interface or kubectl on your local machine with kubectl delete pod <pod-name>

I've hit the same problem. Would be nice if it could be solved in a dot release. ( 1.2.1 )

@rezonant 1.3 is not months away.
@alena1108 It looks like handling the defaultNamespace doesn't work anymore.  Deleting something in the default NS works, in another one says not found.

@soumyalj please test with kubectld image rancher/kubectld:v0.5.1 (last build in vnext branch, the one with skydns and helm)

Tested with rancher 1.3.0-rc1 with Kubernetes template v1.5.1-rancher1-1. Deletion of services/rc/pods within a namespace is successful through UI.


Showing more padding

Hi,
Showing more padding in between password textfield & confirm password textfield even i didnt give one.

Hello @AdelSadiq
I apologize for the error
Use New Version (2.2.5)
Thank you.😊


SCRIPT5009: "Globalize" is undefined

I'm using:
bundles.Add(new ScriptBundle("~/bundles/jqueryval").Include(
"~/Scripts/jquery.validate*"
, "~/Scripts/jquery.globalize/globalize.js"
, "~/Scripts/jquery.globalize/cultures/globalize.culture.de-DE.js"
));
and the error is:
SCRIPT5009: "Globalize" is undefined
Datei: jquery.validate.globalize.js, Zeile: 6, Spalte: 2

Hi Otto,
Remember to serve up Globalize before any of the other scripts.


[Refactor] Move change db and change catalog from GlobalStateMgr to ConnectContext

Why I'm doing:
What I'm doing:
Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[ ] Enhancement
[x] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

Background description（Why I'm doing，What I'm doing） is strongly suggested.


Backport "chore: Bump mtags to latest stable" to LTS

Backports #20442 to the LTS branch.
PR submitted by the release tooling.
[skip ci]

No regressions detected in the community build up to lts-20411.
Reference


make sidepanel collapsible

I'm going to keep old arrow icon for now as it makes more sense in this case compared to burger icon
resolves #51

Don't merge, issue with duplicate sidebar on mobile when switched to map


Add tab support

Adding support for tabbed content. Styling will probably need some tweaking (not my strong point haha) but it does work.
Added an example by converting the git install instructions in the /content/agile/gitflow guide.

Just noting down a couple observations here that someone can pick up and tweak:

[ ] (Mobile and Desktop) Clicking on a tab takes you to the top of the page
[ ] (Mobile) Links within tabs overflow off the page

(Mobile and Desktop) Clicking on a tab takes you to the top of the page

I've noticed the jump to top on web sometimes too, I think it's a side-effect of how they've implemented the tab link. I'm going to see if I can hack the JS with a different implementation.

(Mobile) Links within tabs overflow off the page

Will make some more sacrifices to Ye Olde CSS gods and try to fix this.

@johnharris85 I'm also happy to take a look at some point, or perhaps someone on our team can. Just knew if I didn't note down those observations somewhere I'd immediately forget them :)

OK https://github.com/vmware-tanzu/tanzu-dev-portal/pull/1235/commits/4e441e5151654762a50dadb831137ab0bed0e5c5 fixes the link wrapping.

Tab jumping should be fixed too now (in https://github.com/vmware-tanzu/tanzu-dev-portal/pull/1235/commits/4696efec139a665a850cc1192316cd0049d1c5cf). Basically the way tabs are implemented in Docsy is using anchor links. The existing behavior for anchor links makes this a bit messy in terms of where it should actually jump to. Also if you're clicking a tab you're looking at it anyway, so you likely don't want it to jump anywhere. The (somewhat hacky?) fix I've put in is just to prevent scrolling to the target if the ID starts with #tabs- (as it always will when using the tab shortcodes).


Evaluation errors submitting my test results to the PASCAL server

I  test the  pascal voc 2012 data, and submit my test results to the evaluation serber.
I got the following Evaluation errors:
http://host.robots.ox.ac.uk:8080/eval/view_submission/17144/
Anybody knows how to solve it?
Thanks!

It seems because I wrote the image with wrong mode,
After adding :   im = im.convert('P', palette=Image.ADAPTIVE, colors=256) before saving the image, The server can do the evaluation. But still not right, The evaluation result is very bad.


Add rust preinstalled

Would it be possible to add rust preinstalled on this image? It would reduce our CI/CD time by quite a while.
Thanks

Rust will be included in Windows 2022 images moving forward (either the June or July release will include it)


Issue with k8s.io/docs/tutorials/kubernetes-basics/create-cluster/cluster-interactive/

This is a...

[ ] Feature Request
[x ] Bug Report

Problem:
Run tutorial on my iPad. Was not Able to fill in the questionaire at the end: could not scroll past først question.
Proposed Solution:
Page to Update:
https://kubernetes.io/...

Unfortunately this is a limitation of katacoda and not the documentation
/triage support
/close


Feat: tests, nix / Fix: on_event sucess

Issues Mentioned

https://github.com/mhartington/formatter.nvim/issues/327

Feature Add tests and Nix

We need tests to ensure that the plugin is running and if we change something, we can be sure that wont brake anything.
Nix is for give tools to the developer contribute to the project.

Impact

Criticality

[x] High impact
[ ] Medium impact
[ ] Low impact

Description
It just adds 2 tests for the plugin. - Low impact
Edit on_event success condition. - High impact
It adds nix for development tool - Low impact

Bugfix

Issue description
Rubocop wasn't applying changes when flag --auto-correct was called.
(maybe for C# too)

Root cause of the issue
"on_event" condition for success wasn`t being triggerd.

Solution
Change "on_event" data to run success when it has some value not null

Changelog

It just adds 2 tests for the function "start_task" .

Edit on_event success condition for run success if receive data.

It adds nix for development tool.

README

Tests

Sumary

[x] start_task when file is given by stdin and buffer does not format should return message.
[x] start_task when file is given by stdin when buffer formats should return message.

Notes
N/A

Visual evidences

Are there some breaking change being introduced?

[x] Yes (on_event success data conditional)
[ ] No

Additional information

Contact:

Discord: Outhback#6519

Merged, but for future reference try to keep a PR isolated to a single purpose

@OuthBack Can you explain the bug fix here? As written it causes formatter.nvim to always apply the changes even when the formatter tool returns a nonzero exitcode. What were you seeing the necessitated altering that check?

Right, but for most formatters you don't want to use the output when data ~= 0 because that means that the formatter exited with a nonzero exit code, indicating failure (usually due to a syntax error or similar in the file).
Removing that check breaks most formatters and causes the buffer to be replaced with an error message if the formatter fails. Perhaps you just need the ignore_exitcode option for your case?


Removing Tags property in table definition

Tags property is only supported in cloud formation templates, and not in DynamoDB API.
Fixes issue: serverless offline start fail when specifying Tags
Serverless: Watching with Webpack...
Serverless: DynamoDB - Error -

  Unexpected Parameter -----------------------------------

  Unexpected key 'Tags' found in params

     For debugging logs, run again after setting the "SLS_DEBUG=*" environment variable.

  Get Support --------------------------------------------
     Docs:          docs.serverless.com
     Bugs:          github.com/serverless/serverless/issues
     Forums:        forum.serverless.com
     Chat:          gitter.im/serverless/serverless

  Your Environment Information -----------------------------
     OS:                     darwin
     Node Version:           6.11.4
     Serverless Version:     1.24.1

Changes: Like for TimeToLive, we are removing the property before creating the table.

@ChristopheBougere can you please update the branch?

@rehrumesh yes done :)

Similar PR is already merged.


Cannot connect to Cassandra in test-containers with datastax-java-core-driver on Apple Macbook M1

I am having an issue in my integration tests on Apple M1. The tests can not connect to Cassandra running in test-containers.
Query timed out after PT2S
com.datastax.oss.driver.api.core.DriverTimeoutException: Query timed out after PT2S
	at com.datastax.oss.driver.api.core.DriverTimeoutException.copy(DriverTimeoutException.java:34)
	at com.datastax.oss.driver.internal.core.util.concurrent.CompletableFutures.getUninterruptibly(CompletableFutures.java:149)
	at com.datastax.oss.driver.internal.core.cql.CqlRequestSyncProcessor.process(CqlRequestSyncProcessor.java:53)
	at com.datastax.oss.driver.internal.core.cql.CqlRequestSyncProcessor.process(CqlRequestSyncProcessor.java:30)
	at com.datastax.oss.driver.internal.core.session.DefaultSession.execute(DefaultSession.java:230)
	at com.datastax.oss.driver.api.core.cql.SyncCqlSession.execute(SyncCqlSession.java:54)
	at com.datastax.oss.driver.api.core.cql.SyncCqlSession.execute(SyncCqlSession.java:78)

The stack is:

Azul Java 11.0.13+8
kotlin("jvm") version "1.5.21"
Docker version 20.10.12, build e91ed57
implementation("com.datastax.oss:java-driver-core:4.13.0")
testImplementation("org.testcontainers:cassandra:1.16.2")

There are no errors in container, but the tests fail with timeout. Also, I am able to connect to cassandra via Intellij, when I pause the tests in debug session.
I see, that there was an issue with JNA not supporting the arm64 and this was fixed  in 5.7.0 version.
If I check the cassandra, ran by test containers, it uses the jna-5.6.0.jar. Could it possible be the reason?
The same versions and code work well on Intel-based Macbook

I also see that some .dylib are generated while running the tests

I also see that some .dylib are generated while running the tests

The reason was in datastax java driver timeout being 2 seconds by default. I've raised it to 5 and it helped. See https://datastax-oss.atlassian.net/browse/JAVA-2979


GitHub Actions are failing

The current behavior or issue
The pylint checks for some repositories are failing their GitHub Actions
The steps taken to reproduce the behavior or issue, or specify a location where the steps were recorded
See the following repos:

agpypeline
drone-makeflow
transformer-plotclip
template-transformer-simple
transformer-soilmask-by-ratio

Expected behavior
Pylint shouldn't report any issues
Screenshots
N/A
Add other supporting information that may be useful
N/A
Completion criteria
This is done when pylint runs on the repost

transformer-plotclip has an outstanding change that needs to be resolved first


Update 055-aggregation-grouping-summarizing.mdx

replace avg to _avg
Describe this PR

Changes

What issue does this fix?

Any other relevant information

Thank you @krychaxp we will run a quick tech review on this, and them get it merged. Those leading underscores are important, thanks for helping us out.


Improvement in Write a React Component from Scratch module

Describe the Issue
As per the instructions in Write a React Component from Scratch module, users are supposed to write a React component from Scratch for which they will need to make use of React and ReactDOM. Considering that, the very first instinct of the user will be to import React and ReactDOM by doing the following:
import  React  from  'react';
import  ReactDOM  from  'react-dom';

With the above two import statements in place, the user ends up with Uncaught ReferenceError: require is not defined error .
In order to make the user aware that they don't need to import anything since React and ReactDOM have already been imported for them behind the scenes, I think a note should be added within the Write a React Component from Scratch module, that says the following:
NOTE: React and ReactDOM have already been imported for you behind the scenes.
Affected Page
https://www.freecodecamp.org/learn/front-end-development-libraries/react/write-a-react-component-from-scratch
Expected behavior
A note should be added within the Write a React Component from Scratch module, that says the following:
NOTE: React and ReactDOM have already been imported for you behind the scenes.
System

Device: Laptop
OS: macOS 13.0.1
Browser: Chrome
Version: 107.0.5304.110

Does this need to be changed for other languages too? If yes pls provide note text.

"Write a React Component from Scratch" module is available in multiple languages. This requires localised note. Shall it be done for english only for now?

I'm not sure if that would be needed. Was importing react and ReactDOM introduced in earlier challenges?

Hey @AnonSar thanks for responding. I don't think it is possible to add comment only for one language. bcz any new changes in comment should be added to comment dictionary, and comment dictionary has a key that is mapped to its localisation in all other languages.
![Screenshot 2023-06-13 at 2 42 56 PM](https://github.com/freeCodeCamp/freeCodeCamp/assets/28575935/3ae7e3e7-87b6-4e40-82

ea-6e1ab013b81f)


[STORM-2644] Show message when result not found on deep search page

Deep search shows a blank area if no results are found. see https://issues.apache.org/jira/browse/STORM-2644

Show some message to avoid confusion about whether the searching is finished or not.

I would prefer one message if nothing is found. When there are hundreds of workers, this would make users scroll through many "Result not found" messages before getting to results that are returned.

Thanks. Will fix it

Addressed comment from @knusbaum .
Tested some simple cases on firefox, chrome and safari. The following screenshots are from testing on firefox.

+1


Windows permissions error

Sometimes when I'm running Gulp, there will spontaneously be an issue with a directory I've cleaned and Gulp suddenly can't access it. If I look into it, I find that the folder has lost its owner and Windows says "Unable to display current owner" in the folder's properties -> Security -> Advanced. From here it's nearly impossible to get rid of. I have to execute a takeown using the administrator account through CMD ran as administrator, and even that didn't really work the last time, so I restarted and the folder was gone.
I realize that's an incredibly unscientific rundown and doesn't help to narrow down the issue, but you can find my gulpfile here, as perhaps something sticks out as unusual. The build folder is the one that's getting borked. Most recently, the issue happened during the imagemin task run as part of the build task, but this isn't consistent.
This rarely happens (and I'm sure I couldn't recreate it if I tried), so I know I'm kind of shooting in the dark. Maybe something in my tasks is unusual?

This would most likely happen when you try to write to your build directory while cleaning it but almost all your tasks signal completion correctly (except css-to-sass) and you are using run-sequence so I'm not sure. Maybe just try fixing that one task?

Can you elaborate some? What do you mean by signal completion correctly?

To follow up, I removed the css-to-scss task entirely from both the default and build tasks, and the error still occurs. I can replicate it fairly consistently on at least one project by running the default task, saving a file that triggers the watch task, quitting the gulp task and then trying to run gulp build, which results in Error: EPERM, mkdir 'path\to\build\img'.

Further thoughts on this?

Managed to distill it some:
Here is the gulp build task I'm running:
// Build Task
gulp.task('build', function(cb) {
	runSequence('clean', 'clear-cache', 'build-images', cb);
});

And its subtasks:
gulp.task('clean', function(cb) {
	del(['build'], cb);
});

gulp.task('clear-cache', function() {
	cache.clearAll();
});

gulp.task('build-images', function() {
	return gulp.src(paths.images)
		.pipe(plumber())
		.pipe(imagemin({
			progressive: true,
			interlaced: true
		}))
		.pipe(gulp.dest(destPaths.images))
		.pipe(notify('Image optimized!'));
});

All the plugins being used are the latest versions, and here are their definitions:
var gulp = require('gulp');
var cache = require('gulp-cache');
var del = require('del');
var imagemin = require('gulp-imagemin');
var notify = require('gulp-notify');
var plumber = require('gulp-plumber');
var runSequence = require('run-sequence');

Here are the paths and destPaths variables being referenced:

// Define our paths
var paths = {
	...
	images: 'img/**/*.{png,jpg,jpeg,gif,svg}'
};

var destPaths = {
	...
	images: 'build/img'
};

If I run gulp build twice in succession, I pretty consistently get:
[09:27:04] Using gulpfile q:\wamp\...\gulpfile.js
[09:27:04] Starting 'build'...
[09:27:04] Starting 'clean'...
[09:27:04] Finished 'clean' after 4.75 ms
[09:27:04] Starting 'clear-cache'...
[09:27:04] Finished 'clear-cache' after 4.7 ms
[09:27:04] Starting 'build-images'...
[09:27:04] Plumber found unhandled error:
 Error: EPERM, mkdir 'q:\wamp\...\build\img'
[09:27:04] gulp-imagemin: Minified 2 images (saved 532 B - 41.7%)
[09:27:04] Plumber found unhandled error:
 Error: EPERM, mkdir 'q:\wamp\...\build\img'

With ... being my file path. If this looks like it may rather be a gulp-imagemin bug that gulp as a whole, I can move the issue there, as well.
Thanks ahead for any advice.

For what it's worth, I had a very similar issue which I eventually tracked down to Sublime Text 3 (running on Windows 10 and NTFS). I excluded the build folder from from the project, so that it isn't displayed in the folder sidebar. This was enough to avoid EPERM errors. The strange thing was that del/rimraf clearly thought it had deleted the files, but listing the content of the directory afterwards showed the stray file still there, and writing to it failed since it's in some strange state.

Yeah, guess I could follow up here: may be a thing with Atom as well. If it's open in there, it's claiming ownership it seems, preventing me from doing anything with it.

I'll probably add that our CI system occasionally throws this error. Our build agents all share an NPM cache which is likely the root however it occurs even when only one agent is building and hence none of the files should be touched by any other process.

I'm getting a similar error when trying to copy files/directories to a network drive, though the error is caused by a chmod in this case:
[17:25:27] Error: EPERM: operation not permitted, chmod 's:\lwebwww\lweb802debug\css'
  at Error (native)

I'm not sure there's anything we can do about other processes claiming ownership of files; however, in going over some code for gulp 4, I found that vinyl-fs wasn't using graceful-fs when creating directories and we had some chmod/utimes problems that we've since fixed.  If you are using gulp 3, I suggest upgrading to gulp 4 and see if that helps.


Auto-scratch clears on result screen are shown as failed

Auto-scratch clears on result screen are shown as failed, yet they are saved as assist easy clears.

I think LV skin references will be printing new options later


E2E - add coverage for multiple taxonomy leaf nodes

Current e2e tests one new taxonomy node at a time. Need to test running them through together to have caught this bug: https://github.com/instructlab/sdg/pull/85
the current e2e behavior was necessary with the old code path to make sure we exercised all taxonomy types. With the new version in the library, we don’t need to worry about that and can run all 3 into a single generate run

@russellb do you still intend on doing this? if not is there someone else from SDG that can take this on?

@danmcp @alimaredia i think the large job is currently testing 3 new leaf nodes no? is this one done?

Yes, I'd say this is done. The large job is currently testing knowledge, free form skills, and grounded skills.