Missing Procedure, Function

Hi Team,
I am facing problem in DBeaver which not showing all procedure & Function from schema.
In Oracle SQL Developer, I am able to see many procedure but the same schema in DBeaver not showing.
Regards,
Noushad

I have faced with the same issue for Postgres 9.4
Ctrl+Shift+D shown the function by partial name. After that I pressed Enter to move to the definition and nothing has happened (even no errors in Errors Log view).
DBeaver: 7.0.4
JVM: Oracle 1.8.0_141
OS: Fedora 26

@nouchutm is it still actual against 7.2.2?

For me on Postgres this problem still exist. I see my procedure in the list, but there are nothing happened after press on OK button. Just popup dialog gone.

cannot reproduce it against 7.2.4.
What OS do you use?

I don't see the any changes on version 7.2.4.
The same behavior as I described.
Bug still exist!

OS in the same as in the initial message.

Can't reproduce. Maybe it is OS-specific or database specific.
@mstup could you check error logs for any related messages/warnings? (https://github.com/dbeaver/dbeaver/wiki/Log-files)

Hi @p91paul ,
Still SQL Developer we can see those procedure and function. The issue happen while access in production db, I have developer permission, i cannot able execute those procedure or Function in SQL Developer, only i can view the procedure or function.
Regards,

@nouchutm for your issue you can very likely solve it by enabling this option in connection properties:

DBA_* views (es DBA_TABLES) allow you to see object which you have no GRANTS over. They are also faster to query; however, Oracle requires some specific system grants to access them which not all users have, so dbeaver defaults to ALL_* views for reading objects.

Great! Thank you so much :).
its working now

Can't reproduce. Maybe it is OS-specific or database specific.
@mstup could you check error logs for any related messages/warnings? (https://github.com/dbeaver/dbeaver/wiki/Log-files)

There are nothing in log files that could be related to this issue.


[$500] Room - App crashes when tapping on room header after leaving the

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Go to a room.
Tap room header > Share code.
Copy the share code.
Leave the room.
Paste the share code in another chat.
Tap on the link.
Tap on the room header.

Expected Result:
App does not crash.
Actual Result:
App crashes when tapping on room header after leaving the
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[ ] Android: mWeb Chrome
[x] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/8af0fcae-edd4-4e95-a42f-98d56330c901
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~010ed7138f88427989
Upwork Job ID:<PHONE_NUMBER>125584384
Last Price Increase: 2024-01-30

We think that this bug might be related to #vip-vsb
CC @quinthar

Proposal
Please re-state the problem that we are trying to solve in this issue.
App crashes when opening a header of a room with no members
What is the root cause of that problem?
The issue is in this check:
https://github.com/Expensify/App/blob/5d0931dc47bb5b01d0ff7a3d7662d7cea78f9868/src/components/MenuItem.tsx#L548-L552
We make a check subtitle ?? subtitle === 0, which never reaches the subtitle === 0 part as 0?? is not nullish. As a result, we have a "0" displayed with no text wrapper, which causes the crash.
What changes do you think we should make in order to solve the problem?
We should change that check to:
!!subtitle || subtitle === 0

What alternative solutions did you explore? (Optional)


Question about performance gap between valid set & test set of VB-DMD dataset

First of all, thank you very much for providing the code with such good quality!
I am currently trying to reproduce the result of the model on the VB-DMD dataset, which I download from the link here. The training set I used is the clean & noisy_trainset_28spk_wav, where I split all 468 files from the speaker p286 as my valid set. The command I used for training is as follows:
python train.py --base_dir VB-DMD_dataset/ --accelerator gpu --gpus 2 --batch_size 12 --no_wandb --max_epochs 160
To my surprise, the result I got on my valid set is very poor according to the tensorboard's log: The PESQ score is about 2.2, and the ESTOI value converges 0.82. However, after I test the model on the testing set, the result is much closer to the paper's result: The PESQ score is 2.73 (plus-minus 0.55), and the STOI score is 0.86 (plus-minus 0.10). Now here are my questions:

Do you have any clues about why the model's performance on my valid-set is so bad?
Right now the PESQ score I got on the testing set is not ideal compared with the paper's result (2.73 v.s. 2.93). I know that the batch-size in my current setting is 24 instead of 32. However, do I need to change other hyper-parameters during training as well if I would like to reproduce your result? If so, could you give me a simple command showing how to set them?

Thank you in advance for your time and help!

Hey, thanks for your interest!

This probably depends on the choice of speaker(s) in the validation set. We refer to our baseline Diffuse and have chosen speakers p226 and p287 (see here).
No, you do not need to change other hyperparameters for the training. The command is train.py --base_dir /data/VoiceBank/ --batch_size 8 --gpus 4. Hyperparameters such as spec_factor, spec_abs_exponent, sigma_max etc. do not need to be explicitly specified, since the values used in the paper are given as default values.

Hope that helps!

Thanks for the reply! I retrained the model with your instruction but still get a similar result on testing set (PESQ ~ 2.7). The pre-trained checkpoint you provide indeed achieves ~ 2.9 on PESQ score, so I think somehow the default training setting on my side is not optimal. The GPUs I used for training is A40, but it shouldn't make such a huge difference. Do you have any suggestions for me to check something else? And, if it is possible, would you like to re-train the model as well with default setting to confirm it will generate the correct result?

I compared the released code with the code we used for the pre-trained model checkpoint, and there was indeed a mismatch on one hyper-parameter. The pre-trained model checkpoint uses centered=True, which should also be the default setting when training SGMSE+. We have updated the code accordingly. Thanks you for bringing this issue to our attention and helping us finding the bug in the code.
We retrained the model with the updated code on VoiceBank-Demand, and the model achieved PESQ: 2.93, ESTOI: 0.86, SI-SDR: 17.4, which is very similar to the values reported in the paper. The small deviation could be due to the stochastic nature of the method and the training procedure.
We encourage you to pull the updated code and start another training. Please let us know if it works properly now.

Hello Julius, thank you for the code update! I've re-run the experiment and this time the evaluation result is good now : ).


Add support for map_boolean_as_boolean

what
Implemented postgres_settings for map_boolean_as_boolean as well as map_jsonb_as_clob and map_long_varchar_as.
why
map_boolean_as_boolean defaults to false and this is not what we want for our DMS.
references
closes #35

💥 This pull request now has conflicts. Could you fix it @finchr? 🙏

done.

@joe-niland I just pushed code with an object implementation and complete set of attributes. I also updated the readme.

/terratest

Thanks @finchr looks good.
It seems the tests are failing due to engine_version = 3.4 - I'm not familiar with this service, but shall we try with 3.5?
I'm aware this has nothing to do with your change, so I can try to look at fixing this separately when time.

/terratest

Thanks @finchr looks good. It seems the tests are failing due to engine_version = 3.4 - I'm not familiar with this service, but shall we try with 3.5? I'm aware this has nothing to do with your change, so I can try to look at fixing this separately when time.
cc: @johncblandii

@joe-niland I updated the engine_version to 3.5 and did another push


Remove unused callback parameter.

The callback parameter of both _parseNodeAnnotations and _parseChildNodesAnnotations was not being used at all.
I did grep for it on Polymer/polymer and all official elements packages and couldn't find any reference to it.

I signed it!

LGTM, I'll handle the merge conflict.

Actually, this was removed in https://github.com/Polymer/polymer/commit/35a1b942ad2da000aaa6a7675d864aefe1e6a5a1

Sorry that took so long :(


overlapblock Creator - Web design #15

https://images01.nicepagecdn.com/page/42/41/html-template-42415.jpg Design a overlapblock like the above image
Create a file "overlapblock_NUM_by_(YourName).html" in the modules/overlapblock folder.
Use simple HTML and CSS and JAVASCRIPT to create the overlapblock animation. You can use the interface development https://kwickerhub.com/ (This project) or you can write the lines of code yourself.
Step by Step Guide.
Fork this project(Use the 'fork' button in the top right corner) and Clone your Fork.
git clone https://github.com/YOUR_USERNAME/frontend
Open your code Editor and Create a file "overlapblock_NUM_YOUR_NAME_.html" in the "modules/overlapblock" folder of this project you just cloned. No need to create head, title and body tags, Just Add a div tag with some embedded style(i.e use the style tag) and the script tag where necessary.
If you want to add an image resource, please add it in the folder ••
"modules/overlapblock/images_and_icons"
We recommend you use an svg for your image/icon.
Push your Code: You need to push your recent changes back to the cloud. Use the command below in the main directory of this Repository
git push origin dev
or use a GUI tool to avoid mistakes or complexity. LOL.
Then make you Pull Request...
Below is a example of the code structure we need:
'''

'''
For more examples:
modules->buttons
modules->cards
for examples of contributions or Pull request accepted in to the project.
And please do not wait for this issue to be assigned to you as we have limited hands but have a lot to do/cover. Please send in your PR.
Good-luck.

Please assign me with this issue, I would like to work on this task.

Hi @NtemKenyor, I have raised a PR for the above issue. Can you please review it?


kernel: random: ifdef z_early_boot_rand_get

This function had a to sys_rand_get() even without random source. As Zephyr is
built with linkage garbage collection and this function is called only if
either STACK_CANARIES or STACK_POINTER_RANDOM is enabled and these options
automatically enable a random source.
Signed-off-by: Flavio Ceolin<EMAIL_ADDRESS>

Fixes #20136

@stephanosio Can you take a look again ?


CSI driver fixes (path, locks, etc)

The path cleanup might need to be modified to take care of the old invalid socket locations on existing systems.
But future systems should be working correctly now.

Cleanup CSI Plugin paths
Ensure only a single in flight CSI operation for a resource
Validate csi requests on entry and improve logging

The 1st & 2nd commits are merged in https://github.com/longhorn/longhorn-manager/pull/927.
For the 3rd one, can you provide more contexts & explanations besides statement kubernetes cannot assumed that they are operating in the same process.? I still don't know why we need a lock for it.

@joshimoo It seems that we forget this PR. I think we can merge it after removing the first 2 commits.

Thanks @innobead I will put the csi volume locks on the back burner for the moment. They are a good addition but our driver might run into issues with nfs mounts once we enable hard mode. I plan on dealing with it at that point.


chore: Update dependencies

This pull request was auto-generated by GitHub Actions.

Codecov Report

Merging #94 (2573f73) into master (e220f00) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #94   +/-   ##
=======================================
  Coverage   86.49%   86.49%
=======================================
  Files           3        3
  Lines         237      237
  Branches       45       45
=======================================
  Hits          205      205
  Misses         32       32

Impacted Files
Coverage Δ

deps.ts
100.00% <100.00%> (ø)

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 e220f00...2573f73. Read the comment docs.


feat: ext lemmas for MultilinearMap

I tried to write a Prod one too, but the dependent types become unreasonable so it would probably be useless.

Thanks! :tada:
bors merge


Audiences feature

What do these changes do?

Will pick the TOC restructuring changes after this release when we move to the new Hugo website

Type of documentation

[x] New documentation
[ ] Documentation update
[ ] Hotfix

Migrated to hugo repo


Added wireframes file

This PR adds a wireframe file to the project.

Thanks, Kira great work with the Initial wireframe we can add to it and discuss any accessibility/useability stuff for the website next session


[cocoapods-bazel] Make targets use external repository names

Instead of using //<pod dir> for dependencies, use @<pod name>//:<label>. This makes deps work properly in our repository.
Test Plan: Regenerated PINCache (deps on PINOperation) with clyde ios sync-pods and clyde ios build passed.

This is going to need a fix I think for some of these nested packages (there was logic before, but I removed it to see what broke. Turns out, plenty of things broke :D )


After close docker-machine and container, the tables in database disappear.

After close docker-machine and container, the tables in database disappear.

You remove docker machine? or run docker-compose down?

No, I  just run docker-machine stop.

I have the same problem too. I didn't down with docker-compose down. I just stop with docker-compose stop and ctrl + d to exit from bash. But after I restart my laptop everything lost.
Can anyone help ?

After my previous comment. I try to use docker-compose stop and then docker-machine stop.
And then try to start it again, my data is still alive!!
I think I get it wrong I don't have to ctrl + d. just stop it.

I installed laradock on 2 environments
- digital ocean (docker 1.11.2 on ubuntu 14.04 droplet)
- and my labtop (mac osx el capitan, docker version 1.11.2, docker-composer version 1.7.1)
On digital ocean after I stop with 'docker-compose stop' command then quit ssh terminal and login back to turn on droplet. everything is awesome. My data still alive. I found /var/lib/mysql file.
But when I back to check on my laptop. after I stop with 'docker-compose stop' and try to start with 'docker-compose start' again, everything was gone. no data and no /var/lib/mysql file.
Can anyone suggest me for this situation ?

You could try changing the binding of the volumes in the docker-composer.yml. For now, the script let docker manage the volumes. You can change this to ask docker to put in your host the data, it reduces the risk of losses.
    data:
        build: ./data
        volumes:
            - /data/mysql:/var/lib/mysql
            - /data/pgsql:/var/lib/postgresql
            - /data/maria:/var/lib/mariadb
            - /data/redis:/var/lib/redis
            - /data/memcached:/var/lib/memcached
            - /data/neo4j:/var/lib/neo4j/data

This should make the data folders available on your host at the path /data/SERVICE

@Belphemur Wow!! It's work!!
thank you very much. You help me save my time :)

@Belphemur Sorry. It does not work after I stop docker machine with 'docker-machine stop default'. After I start docker-machine again, all things disappeared.
Could you review my step,

I edit docker-compose.yml replace with your comment.
Add phpmyadmin container and link to mysql
Install laradock (follow video demo).
after laradock installed, execute with 'docker exec -it laradock_workspace_1 bash' command.
Install laravel 'composer create-project --prefer-dist laravel/laravel blog'. After finished move all files to var/www/laravel' and delete blog folder.
create directory at /data (still in bash and create /data/mysql)
still in bash and go to var/www/laravel and php artisan migrate. In this step, everything is ok. I have all data in my db.
exit from bash
'docker-compose stop'
'docker-machine stop default' and ctrl + d and exit docker terminal
open docker terminal and 'docker-machine start default'
'docker-compose start' in my project_folder/laradock
check my data in phpmyadmin but it gone.

I'm not sure where I have to create folder for service data

in the container bash ? /data/mysql ?
in my project_folder/laradock/data ?

Sorry for long question. I'm a new for docker and try to understand.
Thank you for your answer.

ah that's a different issue. This one is directly link to docker-machine, not docker or docker-composer:
docker/machine#179
You have some information about this problem on stack overflow: http://stackoverflow.com/questions/30040708/how-to-mount-local-volumes-in-docker-machine
This thread advice to try https://github.com/synack/docker-rsync

@Belphemur thank you. I will try and report :)
I think it's different too.
I'm sure I did all things with the same process.
All things are very awesome on digitalocean but bad in my local docker.

I have the same problem, I found
data:         build: ./data         volumes:             - /data/mysql:/var/lib/mysql             - /data/pgsql:/var/lib/postgresql             - /data/maria:/var/lib/mariadb             - /data/redis:/var/lib/redis             - /data/memcached:/var/lib/memcached             - /data/neo4j:/var/lib/neo4j/data
this is already in my docker-compose.yml but somehow /data/mysql is not created.

Create the folders yourself first, then docker will mount and write to them.

I've already used volume mapping and same problem happens to me, but in my case I keep DB app always open so when I restarted docker compose while db app is open it says tables doesnt exist, then I exit app, stop docker, restart docker then restart app and now I can access db tables


Error when installing ingress for nethermind helm chart

When configuring ingress for the nethermind helm chart version 0.3.4 I get this error
error calling include: template: no template \"nethermind.httpPort\" associated with template \"gotpl\"\n", "stderr_lines": ["Error: UPGRADE FAILED: template: nethermind/templates/ingress.yaml:3:16: executing \"nethermind/templates/ingress.yaml\" at <i
I was looking at it and it seems that it's because this line is still there in the ingress template https://github.com/ethpandaops/ethereum-helm-charts/blob/nethermind-0.2.0/charts/nethermind/templates/ingress.yaml#L3  but this line has been removed from the helpers template .https://github.com/ethpandaops/ethereum-helm-charts/blob/nethermind-0.2.0/charts/nethermind/templates/_helpers.tpl#L72
I could add a quick PR to fix  :)

Fixed by https://github.com/ethpandaops/ethereum-helm-charts/pull/140


New entry in scpi_special_number_t suggested

CONFigure command accept following values for the 1st arguments: DEF, MIN, MAX, AUTO, . Examples:
CONF:VOLT:DC DEF
CONF:VOLT:DC 12.34
CONF:VOLT:DC AUTO
I'd like to use SCPI_ParamNumber to parse command, but scpi_special_number_t enumeration has no option for AUTO value. Can you add it? Or can I add an exception for this command only to support AUTO option?
Thanks

I have implemented better handling of this in experimental branch. You are not forced to use default list of special values but you can define it per parameter.
In the first step, there is nothing against adding AUTO as additional special value.
In the second step, I will try to backport this feature from experimental branch to old version.

Thanks for reply. I'll look forward to the next release.

Thank you!


Dashboard only showing data from Nov

Describe the bug
Dashboard on the main site only displays data in November, currently only showing 2 days worth of data.
Screenshots

Desktop (please complete the following information):

OS: Windows
Browser Chrome
Version 94.0.4606.81

that was a temporary result of resyncing the graph. Resolved for now.


Add more Authwit examples and add to docs

The current references to the authwit patterns rely heavily on token examples. Create additional references and link to them in the docs.
Some additional use cases to consider:

multisig accounts
token transfers from other contracts. example here

Multisig contract notes here: https://docs.google.com/document/d/154RW7PZ2KPlo7RWh-hojh9qZ9ubNs8XCq33rSzSX9HI/edit#heading=h.887n9glyov61
branch: https://github.com/AztecProtocol/aztec-packages/tree/spetal/multisig

Document we return IS_VALID_SELECTOR instead of a boolean from is_valid?  It’s due to risk of accidentally calling another function with the same selector or a fallback function that returns something that gets understood by the caller as a boolean. But it’s not documented anywhere.

I was considering picking up this issue but the ikigai example is too outdated and unfinished to use it as reference and regarding the multisig it would make sense to wait for it to be merged so that we can use the #include_code macro without which it would most likely get outdated and broken.
@critesjosh Do you know what is blocking the multisig from being merged?

@spalladino did some low level hacking around the kernel circuits, iirc, and I'm not sure that the changes we made ever made it into master.

They didn't, as they require calling a private function as unconstrained from a contract, which I don't know whether it's tracked.

closed by https://github.com/AztecProtocol/aztec-packages/pull/6220


Orchestrator crashes when using the godot-minimal-theme

Describe the bug
When installing the godot-minimal-theme as a custom theme in Godot, the editor crashes. This is because Orchestrator makes a few assumptions about the types of StyleBox implementations that are part of the base Godot editor.  This theme specifically changes the style of ItemList from a StyleBoxFlat to a StyleBox, and our implementation attempted to adjust some corner radiuses which cannot be applied to basic StyleBox types.
Expected behavior
Editor shouldn't crash.
Actual behavior
No response
How to Reproduce?
No response
Godot full version
4.2.2-stable
Orchestrator version
2.1.dev2
Additional information
No response

Did you test with the 1.5.0 https://github.com/passivestar/godot-minimal-theme
with the current 2.1 dev3 Orchestrator? Because, whenever I use the theme it just keeps crashing when creating or open .os
godot 4.3.beta2

Hi @neilfranci, thanks for the feedback.
No, we did not primarily because we do not yet officially support Godot 4.3 as there were some API changes in the code interfaces between Godot 4.2 and 4.3 at the GDExtension level and in some of the Godot widgets we use.  While a stock Godot 4.3 environment may work with Orchestrator, it's not yet tested and may result in crashes due to API changes.
We'll take a look and add a fix if godot-minimal-theme fails on Godot 4.3 and Orchestrator 2.1 for the dev4 build where we intend to officially support Godot 4.3.


Invulnerable while ultimate happening on both characters

Unified UltimateOn on Player.h
Set methods to check wether or not a Character is vulnerable (in case we want to establish other parameters for invulnerability)

Task linked: CU-nqfdhn Update Fang and Oni ultimates to make them invulnerable

I don't see the utility of implementing the IsVulnerable method but then call directly !ultimateOn on most checks when it's the same as calling IsVulnerable. Also, I find confusing that Fang & Onimaru have a IsVulnerable function that behaves differently from the one in playerController when they are both named the same

It's true that the function IsVulnerable is useless because we are using always the attribute ultimateOn and the purpose is the same. I would delete this function in the Player, Onimaru and Fang classes.


[1.21] Tools and weapons don't show default modifiers

Normally weapons and tools show modifiers

Yours do not

Huh! with this many bugs... I should have made it ALPHA I'll try to find what on earth causes this!

They changed the way tools and weapons are registered.
I tweaked yours to look like this

I added baseAttackDamage of 3 since your sword was only giving attack damage of 4 on ruby sword.
example code:
    public static final Item RUBY_SWORD = new SwordItem(INSTANCE,             (new Item.Settings()).attributeModifiers(SwordItem.createAttributeModifiers(INSTANCE, 3, -2.4F)));

Attack damage of 7 comes from:
1 (builtin my Minecraft) + 3 (from the material) + 3 (baseAttackDamage at register) = 7

Thanks for this solution and all your help. Also, if you would like to speed things up, you could fork the rep and make a PR.

Some other thoughts.
it is not recommended to instantiate utility class
 public static final ArFoMiToVi AR_FO_MI_TO_VI = new ArFoMiToVi();
or to register items from an instance object
AR_FO_MI_TO_VI.RegisterAll();
you can call you register methods directly from onInitialize()
Minerals.RegisterAll();         Foods.RegisterAll();         Misc.RegisterAll();         ArmorMaterials.RegisterAll();         ToolMaterials.RegisterAll();         Villagers.RegisterAll();         Entities.RegisterAll();
Lastly, it is not a good idea to name classes the same as vanilla, such as ArmorMaterials and ToolMaterials
Keep up the hard work.

oh, one more thing.
Don't do this:
public static final String NAMESPACE = "mc-extended";
do this:
public static final String MC_EXTENDED = "mc-extended";
your mod id gets used throughout the game and messages get put into logs with the id.  It should be unique to your mod, not generic.

Yeah, if you did a thing or two (A PR or so) you'd be in the contributors...

Per your suggestions I'll change those... I thought I had a long time ago.


Install Crystal via homebrew in OS X

I couldn't test it for real because I don't have a working Travis environment but the generated script is correct and according to this calling brew is possible.
Note that from some versions ago installing Crystal via brew includes shards so it's not needed (I think that might also be the case with apt-get but I preferred not to change that because I'm not sure).
/cc @asterite @jhass @waj @will

I think we should install the latest shards not the shipped one in either case for now, that's why they're separate projects still.

@jhass mmmm then this PR is not complete still, I'll update it with that. Do you think it could be installed with brew, or should it use the same github release download?
and thanks for the very fast answer!

I don't know

maybe @ysbaddaden has an advice on this?
Current situation:

brew: 0.5.3
shipped with crystal 0.10.0 via brew (latest crystal release): 0.5.4
shipped with crystal HEAD via brew: 0.5.4
github release 0.5.4

With the current situation and the PR as is, we would be getting the lastest shards, but of course that doesn't mean that's guaranteed to be always like that.

We don't have to install Shards anymore, it's already bundled. In my projects that run on Travis I'm just using "crystal deps install". If a project needs a more recent version, it can install it manually via its travis.yml
I vote for skipping the (re)installation of Shards.

I don't bundle it with the nightly packages yet iirc

Thanks Julien,
It's definitely present in the brew packages (both stable and head), so at
least for osx it can be safely skipped (i.e. as is in the PR).

My biggest point of always fetching the latest version is a more consistent build environment across all variants.

Shards is also bundled in the Debian packages. I dont know about the Linux nightly.

bump

Hello. Is there a consensus on this PR?

I didn't see a convincing counter argument here yet, so my vote stays at keeping the build environments more consistent and always installing the latest shards.

OSX binaries of Shards are lagging behind. Shards is bundled with all Crystal releases. I usually release a new version shortly before a Crystal release (so the latest version is bundled). I can't see any reason to keep installing Shards anymore —unless we're using nightly releases.


feat(tests): STOP- assignment- Add Tests for STOP Opcode and Blob Data Handling in Shard Blob Transactions for EIP-4844

🗒 Description
This test covers:

Transactions containing blob data with KZG commitments and proofs
Validation of STOP opcode (0x00) in transaction data
Execution simulation to verify that transactions with the STOP opcode halt as expected

🔗 Related Issues

https://github.com/ethereum/execution-spec-tests/issues/972

Thanks for the feedback, I'll  try to modify and commit changes today by my midday (EST)

Thanks for the feedback, I'll try to modify and commit changes today by my midday (EST)

As I mentioned in DMs, feel free to take as much time as you like/can afford :+1:


fix(code-editor): fixed @babel/preset-env version (7.22.9)

in the @babel/preset-env version @7.22.10 version, changed the introduction method of plugin-syntax-xxxxxx. Causing the new version to be unusable.(https://github.com/babel/babel/commit/6b91b9b2af22251fe04f0b6f1d8f74c18a414a10#diff-2281f56422073bffd4b039230999ad8fe300fa0ce0fe9af45580695488c82f68)
fix: alibaba/lowcode-engine#

 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.


Continuous integration and versioned NuGet

I should be able to do full end-to-end build/test of a package locally as well as see the CI (travis?) status and be use FsStorm via NuGet reference.
(I can set this up)

agree - we can start with ProjectScaffold


Question: API Available?

Hi there!
First of all thanks for this fantastic work! My question is: Do you know any open source REST API for the World Factbook that I could use?
Thanks,
-Thomas

Not sure I understand.   This is a REST HTTP API for the World Factbook.   What is missing?
For example, for Germany (gm) try https://github.com/factbook/factbook.json/raw/master/europe/gm.json and so on.
Note: The data gets updated once a week.   For more questions, please see  https://github.com/factbook/factbook.json#questions-comments


Support changing colour when device is off

I have developed a NodeRED Home Lighting Zone Management system, which outputs separate messages for brightness and colour.
In the case where the light is off (because I set brightness to 0), then I set the scene in my zone management system which turns the light on and sets the colour, this is usually no problem. My system outputs the brightness first, which turns the bulb on, then the colour. After all, if the system set the colour first, and the bulb was turned off, then the bulb would come on without having received the correct colour. This is because it appears you cannot set colour of a bulb that is OFF.
 This may be because someone has turned the light off in the app (not through my system) or MQTT didn't deliver messages in order. Either way it doesn't work, as you cannot set the colour of an OFF bulb.
I presume this is a Hue limitation (I see no reason why a bulb shouldn't have a colour state that persists whether the bulb is on or off, but hey...)
But is there a way to make this work smoothly?
For example, HueMagic might be able to support setting the colour of a bulb that is off?
Another example, it may be possible to query the bulb to see if it's actually off before attempting to set it (maybe this could be a workaround that I could implement without a resolution of this issue here on GitHub?)

Do you have the latest version installed? You can change light and group properties like colours etc. on turned off devices with version 2.8.0+ (or did I misunderstood something?).


NaN volume causes ValueError in slippage.py

I have plugged in a custom datasource that is missing some data. What is causing me problems right now is that the volume is missing on a few places and this causes ValueError: cannot convert float NaN to integer in zipline/finance/slippage.py#L217 due to event.volume being NaN a few rows above.
I'm just wondering if this is expected or if the NaN should have been handled earlier?

I am having this same error. Tried filtering out the NaNs (Fillna, dropna, etc). Nothing seems to fix it. Further more I cant actually see any NaNs in the data I sampled! Any Ideas?


Setting defaultCommandTimeout in cypress.json is not applied

cypress-testing-library version: 2.3.1
node version: 9.0.0
npm (or yarn) version: 1.10.1

Relevant code or config
        cy.visit("/")
            .getByTestId("signUpButton", {timeout: 6000})

What you did:
I wanted to select a few elements, with a longer timeout. After a few selects, I figured it would be just easier to set the default command timeout in cypress.json, so I don't have to set this every time I call getBy... methods. So I went to cypress.json and put  "defaultCommandTimeout": 6000.
What happened:
The default 6000ms command timeout wasn't applied.
Suggested solution:
I guess we should modify the defaults object (https://github.com/kentcdodds/cypress-testing-library/blob/master/src/index.js#L5):
var defaults = {
  timeout: Cypress.config().defaultCommandTimeout
};

Thanks @adrian1358! I'd love a PR for this :)

:tada: This issue has been resolved in version 2.3.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


New port allocation rule

Here is the new Magellan port rule
Standard Magellan port convention:
// let n = settings.BASE_PORT_SPACING - 1;
portOffset       : selenium server
portOffset + 1 : pre-assigned for mocking (available for application to use)
...
portOffset + n : available for application to use

We move the port reserved for selenium server from portOffset + 1 to portOffset.  This will allow us allocate less port for scenarios which only one port is required (for selenium server).
This PR also allows executor to have its own port allocation rule. If executor defines its own port rule magellan will use it instead.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@2d6570f). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master     #219   +/-   ##
=========================================
  Coverage          ?   99.13%
=========================================
  Files             ?       28
  Lines             ?     1150
  Branches          ?      193
=========================================
  Hits              ?     1140
  Misses            ?       10
  Partials          ?        0

Impacted Files
Coverage Δ

src/cli.js
98.98% <ø> (ø)

src/worker_allocator.js
100% <100%> (ø)

src/test_runner.js
98.62% <100%> (ø)

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 2d6570f...6fc5542. Read the comment docs.


Upgrade terraform-provider-azurerm to v3.57.0

This PR was generated via $ upgrade-provider pulumi/pulumi-azure.

Upgrading terraform-provider-azurerm from 3.56.0  to 3.57.0.
Fixes #1274
Upgrading pulumi-terraform-bridge from v3.47.2 to v3.47.4.

@t0yv0 I had to regenerate the patches (via make upstream.rebase) due to conflicts, which is why the owner was edited from Ian to me


[StimulusBundle] add togglePassword component to official packages list

Q
A

Bug fix?
no

New feature?
no

Tickets
N/A

License
MIT

TogglePassword component has been released with v2.11.0 so we can now add it to the official UX packages list 😃

Thank you Félix!


Print failed and pending specs at the end of the test run

Love this gem, but knowing what specs failed or are pending is really helpful when running specs. I just modified the example_xxx methods to call super just like the progress formatter does.
Let me know if you have any questions.

I didn't know anyone was actually using this gem! Thanks for the patch, man.
:shipit:

No problem. Glad to pitch in.
-AH


Technical audio specification and requirements missing for linear video playback

Hello,
within the Windows 10 Mixed Reality Portal it is possible to watch linear VR 360° films (LatLong).
I cannot find a specification about the technical audio format that is supported, similar to this YouTube guide, scroll down to "Technical Requirements: https://support.google.com/youtube/answer/6395969?hl=en
Can I create an H.265 MP4 or MOV video using AAC and 6 audio channels with "Ambisonics B-Format First Order with Head-Locked Stereo format: W, Y, Z, X, L, R as a 6-channel audio track 48 kHz" - for example?

@jessemcculloch Who could we pull in to answer this?

@backhausmartin Did you have a specific article that you're referencing for this issue?


Comparing by tuples is not always correct

Another Hynek-didn’t-quite-fully-understand-Python bug: while I maintain that establishing order using tuples makes totally sense, Hypothesis just taught me about an invariant in container equality I wasn’t fully aware of:
>>> from math import nan

>>> nan == nan
False

>>> (nan,) == (nan,)
True

The problem here is that tuples compare by identity first and by value second.  And nan is identical but unequal.
I’ve noticed this when I tried to implement our __eq__ using a string of and s instead of building two tuples which I’m sure is faster.
My first gut instinct was to use tuples again.  My second one is that we’re probably masking some bugs somewhere by this behaviour.  And it totally is a bug in attrs.
What to do?

Obligatory ping to @ericvsmith since they seem to do it the same way in data classes.

I think the conflict is unresolvable. As long as you define the behavior, the user has to live with the results of depending on it. Using nans as keys in a dict or in sets has the same issue. And it's all made worse by float('nan') returning a new nan:
>>> s = {nan}
>>> nan in s
True
>>> float('nan') in s
False
>>> float('nan') == nan
False
>>> float('nan') is nan
False

I do agree it's unfortunate that changing to and chaining might be prevented by the wacky nan behavior.
Do you have any numbers on and chaining speed vs. tuple creation? I might change dataclasses to define comparison by and chaining if it's demonstrably faster.

My initial reaction: f*** floats (not helpful, I know). Is the chain of ands approach faster?

I sort of agree: I don't think you should do anything special for anything where is is True, but == is False. nan is the only thing that falls in this category that I know of.
There are many instances of this being raised on bpo. Here's an interesting one https://bugs.python.org/issue4296 with a quote by Mark Dickinson:

But in general I don't think the fact that NaNs are weird
should get in the way of optimizations.  In other words,
I'm not sure that Python should be asked to guarantee
anything more than "b == b" returning False when b is
a NaN.  It wouldn't seem unreasonable to consider
behaviour of nans in containers (sets, lists, dicts)
as undefined when it comes to equality and identity
checks.

Although he seems to walk back part of his comment, so I don't know where it stands in the end, unfortunately. Also, it's old.

So I did a quick benchmark:
ands:
pyperf timeit --rigorous -s "from t import c1, c2" "c1 == c2"
.........................................
Mean +- std dev: 549 ns +- 20 ns

tuples:
pyperf timeit --rigorous -s "from t import c1, c2" "c1 == c2"
.........................................
Mean +- std dev: 533 ns +- 21 ns

So looks like ands are even slower for some reason!

OK I think it’s fine to not change anything so far.  At some point we maybe should document this a bit.


runtime: way to check if pointer satisfies (*Pinner).Pin

What version of Go are you using (go version)?

$ go version

go version go1.21.0 linux/amd64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=''
GOARCH='amd64'
GOBIN=''
GOCACHE='/home/username/.cache/go-build'
GOENV='/home/username/.config/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='amd64'
GOHOSTOS='linux'
GOINSECURE=''
GOMODCACHE='/home/username/go/pkg/mod'
GONOPROXY=''
GONOSUMDB=''
GOOS='linux'
GOPATH='/home/username/go'
GOPRIVATE=''
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/usr/local/go'
GOSUMDB='sum.golang.org'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/usr/local/go/pkg/tool/linux_amd64'
GOVCS=''
GOVERSION='go1.21.0'
GCCGO='gccgo'
GOAMD64='v1'
AR='ar'
CC='gcc'
CXX='g++'
CGO_ENABLED='1'
GOMOD='/dev/null'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -m64 -pthread -Wl,--no-gc-sections -fmessage-length=0 -ffile-prefix-map=/tmp/go-build3016021898=/tmp/go-build -gno-record-gcc-switches'

What did you do?

I'm writing a C wrapper library. I'm trying to handle void* pointer. I'm experiencing crashes about cgo pointer has Go pointer to Go pointer, so I've updated to the latest go release (1.21) and now panic adives to use Pinner.
When I use Pinner it works sometimes, but sometimes it crashes because of panic: runtime error: runtime.Pinner.Pin: argument is not a Go pointer.
I've used dlv to chack this pointer and it seems to be unsafe.Pointer as usual, so I have no idea why it panics this time.
Sadly I have no reproducable code snippet. The only thing I can post is link to my dev branch in project when this happens: https://github.com/gucio321/cimgui-go/tree/input-text-with-hint
What did you expect to see?
I think that even if this isn't a bug in Pinner, there should be some way to check if a pointer (or unsafe.Pointer) is a "Go Pointer"
What did you see instead?
I see no way to fix my bug
I'm sory if I've used a wrong template

A "Go pointer" is a pointer to memory that was allocated by Go code in the ordinary way, rather than by using something like syscall.Mmap.  To be clear, it has nothing whatsoever to do with the type of the pointer.
In general it should be possible to keep track of where you allocate your pointers.  Passing a C pointer (a pointer to memory allocated by C code) to runtime.Pinner.Pin is a logic error.

In general it should be possible to keep track of where you allocate your pointers. Passing a C pointer (a pointer to memory allocated by C code) to runtime.Pinner.Pin is a logic error.

Sure, but it is quite difficult to determine if specific unsafe.Pointer was allocated by go or c. It should be possible but I'd duplicate logic from runtime package and it'll ofc hit performance of my code. I saw a function runtime.isGoPointerXXXX. is it possible to have it exported?

It does seem like there is a problem here. If you are writing a general-purpose library that wraps C code, then you may want to pin the pointers being passed into your library before passing them onward to C. Pure Go general-purpose libraries can be invoked with memory that was allocated by mmap or C (provided they don't retain those pointers after returning), but this kind of C wrapper using pinner cannot.
Perhaps Pin should silently allow pinning of non-Go pointers (there is nothing for it to do) instead of panicking.

I, at least, am not worried about that level of compatibility.

cc @golang/runtime

I agree it's super annoying that Pin panics. The original intent was to mirror SetFinalizer, but I agree it's completely reasonable to do something different here. Thanks @doujiang24 for sending a CL!

Looking back at the reviews for the original Pinner implementation, it looks like the possibility came up but we preferred to stick close to SetFinalizer's semantics, I guess just as a conservative starting point.
One thing I'm slightly concerned about is that there's no explicit escape of pointers passed to Pin. One danger with ignoring non-heap pointers is that we could pretend to pin a stack pointer and then the stack could move. (We currently have no way to pin a stack pointer.) I'll comment on the CL.

One thing I'm slightly concerned about is that there's no explicit escape of pointers passed to Pin. One danger with ignoring non-heap pointers is that we could pretend to pin a stack pointer and then the stack could move. (We currently have no way to pin a stack pointer.) I'll comment on the CL.

Just kidding. I forgot that we put the pointer into (*pinner).refs and the *pinner has a finalizer. So it's always on the heap, and in turn anything passed to Pin has to escape.


[AutoDiff] Remove References to Deprecated @autodiff Attribute

There are other instances of the use of autodiff such as autodiffApply_ and autodiff_function, however, as far as I can tell, these aren't related and are still required. Wanted to confirm.

@swift-ci please test tensorflow


Autoplay interval (between each slide)

Hi there,
Slick quickly became my favorite slider / carousel because it's flexible, light and well coded/maintained. But... I went through the documentation and Github issues and I can't find the option to set the time interval between each slide.
The autoplaySpeed seems to be the time before the slider starts. The speed option is the transition time (slide / fade). But how to set the time interval between each slide, for instance 3 seconds?
Please let me know :+1:

similar to this question, if it's possible to set time interval in between slides, is it possible to set unique values from slide to slide?

Here, try this :)
http://jsfiddle.net/dkj3hrqk/1/
(the time is not 3 seconds here, its set super fast to show you you can change the speed.)
Simon.


Exhaustive switch could also check for invalid values

Minor debugging helper feature.
The documentation says that switch requires either exhaustive handling of all valid values or mandatory else branch.
If the switch is exhaustive it would be still useful (in debug mode) if compiler inserts hidden else branch which then shows the error and exits. Enumerated value may not be always valid (e.g. when set by C code).

If one wants to guard against invalid values always and explicitly he could use "invalid branch", like:
enum { Foo, Bar };

switch (val) {
case Foo => ...
case Bar => ...
else! => assert(false);
}

else! (or invalid etc) is different from else that it doesn't "swallow" anything valid.

const Suit = enum {
    Clubs,
    Spades,
    Diamonds,
    Hearts,
    Quantoms,
};

test "switch safety" {
    var a: Suit = undefined;
    *@ptrCast(&u8, &a) = 5;
    switch (a) {
        Suit.Clubs, Suit.Spades, Suit.Diamonds, Suit.Hearts, Suit.Quantoms => {}, // ok
    }
}

$ ./zig test  test.zig
Test 1/1 switch safety...reached unreachable code
/home/andy/dev/zig/build/lib/zig/std/special/zigrt.zig:16:35: 0x0000000000203555 in ??? (test)
        @import("std").debug.panic("{}", message_ptr[0..message_len]);
                                  ^
/home/andy/dev/zig/build/test.zig:12:5: 0x0000000000203475 in ??? (test)
    switch (a) {
    ^
/home/andy/dev/zig/build/lib/zig/std/special/test_runner.zig:9:21: 0x00000000002161e8 in ??? (test)
        test_fn.func();
                    ^
/home/andy/dev/zig/build/lib/zig/std/special/bootstrap.zig:60:21: 0x00000000002160a7 in ??? (test)
    return root.main();
                    ^
/home/andy/dev/zig/build/lib/zig/std/special/bootstrap.zig:47:13: 0x0000000000215f60 in ??? (test)
    callMain(argc, argv, envp) %% std.os.posix.exit(1);
            ^
/home/andy/dev/zig/build/lib/zig/std/special/bootstrap.zig:34:25: 0x0000000000215eb0 in ??? (test)
    posixCallMainAndExit()
                        ^

Tests failed. Use the following command to reproduce the failure:
./test

works!

Oh, that's nice. The error could be more informative, showing the actual invalid value.

I would welcome such a contribution. There is a precedent set for unwrapping an error value.


// fixed css files not found and not cached

This is a little patch, regarding the stylesheets for PrestaShop 1.7 are not available for now.
This make the gamification module looking for the url for every request in back office.
Blackfire profile of comparison: https://blackfire.io/profiles/compare/7e10d121-8715-4dd1-bfe0-1f39053bc5ea/graph
Regards,

Hi @mickaelandrieu,
Okay to merge this PR, but it must be created for the dev branch, not master. :disappointed:

Hi @mickaelandrieu,
Thanks to highlight this issue, instead of doing it directly in the module I made few changes directly on the gamification API. All CSS should now be displayed correctly in BO.
Therefore I close this PR,
Best regards,


New-Arrivals Page update

📝 Description
Created the New Arrivals page which was empty earlier. Now it has a nav bar, corousel, product cards and a footer.  Made it Responsive
🔧 Fixes #(issue)

[ yes ] I have performed a self-review of my own code.
[ yes ] I'm a contributor in Hacktoberfest'21.

This is fix for issue #116


add byText to EspButton, EspEditText, EspTextView  with resourceId as…

add byText() to EspButton.java, EspEditText.java, EspTextView.java  with resourceId as param
add scrollTo() to EspView.java
add overloading assertTextIs() with resourceId as param to use them directly to EspButton.java & EspTextView.java

Changes will be added with the new library concept.


should audio recording work?

Thanks for putting this together.
I'm trying to record video w/ audio. Out of the box this doesn't seem to work (on Ubuntu)--is it something you've looked into?

yeah I gave that a shot. it doesn't fail outright, but the program doesn't stop after the timeout when audio: true.
thanks for the quick response--will keep at it & update for posterity if I find something

@brandonmp did you find a workaround for the issue that the timeout doesn't work with audio: true?

Audio recording works well, insted audio: true you should use:
audio: {
                  mandatory: {
                      chromeMediaSource: 'system'
                  }
}

@brandonmp,
I have tested @bitk0der 's solution and it works well. Feel free to open the issue if needed.

Any ideas on getting audio to work in docker? This fork (https://github.com/pzagor2/puppetcam) works awesome for docker but having issues with audio.  @pzagor2

@FisherRobison I haven't spend any time trying to get the audio to work.

@FisherRobison  Have you tried this?
Looks like you need to map some devices from host machine to the container

Audio recording works well, insted audio: true you should use:
audio: {
                  mandatory: {
                      chromeMediaSource: 'system'
                  }
}

Unfortunately, audio is not recording in Windows 10.
I could only able to full screen (not chrome tab) capture with audio included in Windows 10.
Reference:
https://www.electronjs.org/docs/api/desktop-capturer


Postgres Support?

Is there any way to implement postgres into this or can you do that with the in-built sequelize provider?

Sequelize is a promise-based ORM for Node.js and io.js. It supports the dialects PostgreSQL, MySQL, MariaDB, SQLite and MSSQL and features solid transaction support, relations, read replication and more.

http://sequelize.readthedocs.io/en/v3/

Oh right so in the code example from the docs I'd just do this
this.settings = new SequelizeProvider('link to PG database', {
            idColumn: 'guild_id',
            dataColumn: 'settings'
        });

would pg need to be installed as a dependency as well?

Yes you would have to install and set up a PostgreSQL server, it isn't like SQLite3.

Alright thanks


Decide on API for github.com/gopherjs/gopherjs/build

For module support, GopherJS needs to use the go tool under the hood (strictly speaking we could sit GopherJS atop go/packages to make it build tool agnostic, but we haven't done that for now).
But the Playground (https://gopherjs.github.io/playground/) runs github.com/gopherjs/gopherjs/build in-browser. Hence it does not have the go tool available. The Playground is, however, limited to standard library imports.
jsgo is more advanced in that it can handle non-standard library imports.
Hence in its most basic form we need to make github.com/gopherjs/gopherjs/build be driven by some sort of "driver" that knows how to resolve import paths (in some context). The simplest way to do that will be to provide an implementation of the following interface:
type Resolver interface {
    Load(conf *Config, patterns ...string) ([]*Package, error)
}

The Package type here is a cut-down version of https://godoc.org/golang.org/x/tools/go/packages#Package that will also have a field:
Open(path string) (ReadSeekCloser, error)

for subsequently opening files listed in the Package.
where:
type ReadSeekCloser interface {
    io.Reader
    io.Seeker
    io.Closer
}

This is, in effect, the last remaining piece of work to bring full module support to GopherJS. Given the command line version of GopherJS works just fine as of https://github.com/myitcv/gopherjs/tree/f5b96be2a04cf73ef5198ad6520e80375fd9764b, we could decide to leave github.com/gopherjs/gopherjs/build as broken for now, merge the command line module support upstream and revisit at a later date. This would be at the cost of the playground which would be stuck on Go 1.11.5.
Any thoughts/contributions on the above welcome.
cc @hajimehoshi  @paralin @dave @dmitshur

Since this issue is about the github.com/gopherjs/gopherjs/build API, it should be in the https://github.com/gopherjs/gopherjs/issues tracker. Do you mind moving it there?

we could decide to leave github.com/gopherjs/gopherjs/build as broken for now, merge the command line module support upstream and revisit at a later date. This would be at the cost of the playground which would be stuck on Go 1.11.5.

Given our limited resources (per https://github.com/gopherjs/gopherjs/issues/894), I think this is a viable cost-cutting measure to take.
The way I see it, it's a limitation of the upstream Go project. Before modules, go/build was the canonical package to gather information about Go packages. After modules, golang.org/x/tools/go/packages became that. go/build did not require the use of exec.Command which allowed the GopherJS compiler to be compiled to JS (via itself) and the GopherJS Playground as we know could exist. When migrating to module mode, it requires the use of golang.org/x/tools/go/packages which in turn requires support for exec.Command that the GOARCH=js environment does not support. Therefore the GopherJS Playground cannot exist as is.
We can consider alternatives to make it work, such as:

coming up with our own API, as you mentioned here
changing the GopherJS Playground so that it performs Go compilation on the backend (on a server) rather than in the frontend (in the user's browser)
consider pointing to jsgo as an alternative

However, I don't think the GopherJS Playground working as it does today should block GopherJS gaining support for modules. I think we should add module support, and work on the GopherJS Playground orthogonally (if anyone has time left to do so, otherwise it will stay at 1.11.5 until someone does).

consider pointing to jsgo as an alternative

The jsgo compiler and playground will have exactly the same limitations. The jsgo compiler does compilation server-side, but it does it all in-memory. It would need significant changes to work with x/tools/go/packages. The jsgo playground does compilation server-side and also client side, so would similarly need a lot of work.

Please try not to change the call signature of gopherjs/build.


Is Camera Module 3 compatible?

Is Camera Module 3 compatible?

No. I don't believe so. Module 3 requires using the libCamera interface and not the original MMAL interface used by this software.

Thats a pity, module 3 camera is so much better. Are there any plans to support module 3 and older cameras as well in the future, perhaps by detecting the camera fitted during nstallation and then loading the appropriate interface?

Hi @canuckdev, you might be interested in this. Testing was conducted on the camera module 2 and Arducam OwnSight 64 MP camera, but given that this approach uses Picamera2 (developed by Raspberry Pi themselves), it should be compatible with the module 3. Be aware that not all functionality has been implemented but it should be a good starting point ^_^


New tests fail around year parsing

I was having some issues on my CentOS 7 server, so I added 2 new tests to ccronexp_test.c.
check_next("0 */2 * * * *",     "2012-07-01_09:00:00", "2012-07-01_09:02:00");
check_next("0 */2 * * * *",     "2018-09-14_14:24:00", "2018-09-14_14:26:00");

The first one passes, the second one does not.
Program output:
Initial: 2018-09-14_14:24:00
Expected: 2018-09-14_14:26:00
Actual: 2008-09-14_14:26:00
ccrontest: ccronexpr_test.c:220: check_next: Assertion `0' failed.

Another that does not pass is if I alter the first one to have a year of 2013 instead of 2012.
Initial: 2013-07-01_09:00:00
Expected: 2013-07-01_09:02:00
Actual: 1900-07-01_09:02:00
ccrontest: ccronexpr_test.c:220: check_next: Assertion `0' failed.

Thanks for the report! Going to look into this, but cannot do so this week. Alternatively - can happily merge a PR for this.

fix merged.


Using custom created traffic groups in float ip( "class": "SelfIP"), and mac address masquerade("class": "MAC_Masquerade") configuration

Environment
Declarative Onboarding Version: 1.29.0
BIG-IP VE Hyper-v: BIGIP-<IP_ADDRESS>-0.0.14
Is your feature request related to a problem? Please describe.
Usage of custom traffic groups(other than traffic-group-local-only and traffic-group-1) in the "class": "SelfIP" configuration.
Usage of custom traffic groups(other than traffic-group-local-only and traffic-group-1) in the "class": "MAC_Masquerade"
At this point we can create other traffic groups but cannot assign MAC_Masquerade or float IP's to the newly created traffic groups.
Describe the solution you'd like
I would like to be able to chose a custom created traffic group for the Selft IP and MAC_Masquerade configuration.
Use case in active-active HA configuration
Describe alternatives you've considered
Currently modifying float ip's and traffic-group configuration manually:
modify net self { float IP } traffic-group { custom traffic group }
modify cm traffic-group { custom traffic group } mac { MAC }
Additional context
None

Bumping this issue. I assume DO supports traffic groups to support their use in AS3, however it is critical that SELFIPs also have the same support. This is a standard common use case with BIG-IP.

We also have a need for this functionality. It's a big limitation in the use of traffic groups in IaC deployments. The workarounds using API / tmsh are problematic because changing the traffic group for floating IPs causes a temporary interruption, so each time the DO config is deployed and then the float IPs are moved back to the right groups, there's a blip.


chore: support auto-handle in more branches

support auto-handle tidb-test merge in more branches.

/hold

/approve cancel
we can make it simple and do not repeat ourself.

/unhold


[FLINK-2764] [WebClient] [0.10.0-milestone-1] WebClient cannot display multiple Jobs

Can I merge this?

If no objections I will merge this.

Is this just a fix for the milestone-1 or the master?

Both. I labeled it as milestone-1 because I opened this PR before it was decided that the next RC will be forked from master. So I would just merge in into master right now (and it will be included in milestone-1 automatically).

Merging this now to master.

Thanks for fixing this!


[QA] Ask Astro bot after deployment with PROD database

We need to perform the final QA after deployment to PROD

Testing completed, Looks good for release
Results


Feature Request: More traditional sidebar option

Hi,
It'd be neat if we could get a traditional sidebar option - I've been looking at moving from Dokuwiki and this the key thing that I can't find nor make. Would help people find the pages they want a lot quicker without relying on search or having a good 100+ links on the main page...
Something like:

Having standard text titles (or even books) with the chapters listed underneath would be more useful for getting users to where they need to be versus the current sidebar.
Is anything like this planned?
Thanks.

Personally, I prefer the feed that is provided.

Personally, I prefer the feed that is provided.

Yeah, it might be best to have some sort of options (but I'd guess that'd be a lot more difficult).
Currently I have the sidebar style I'd like as the start page, but once you leave that, there's no quick jumping, which is a shame.

I want this too. This is a nice feature to speed up the navigation and the suggestion from @moqmari seems good enough.


OOP school library: refactor your code

Refactoring to adopt SOLID principles

In this branch the following tasks were done:

Added menu_option.rb file
Added lib_methods.rb to include the App methods
Refactored App class

The codebase has been refactored to suite the SOLID principles


ARROW-5875: [FlightRPC] integration tests for Flight features

This is only a minimal test (currently the basic auth protobuf). I have more tests, but they require other fixes, so I'd rather get the basic framework down and then add to it, over creating an enormous review.

Looks good to me. The only thing I didn't quite understand was the change to ServerAuthInterceptor was this required from the integration testing or is this a cosmetic change bundled in?

Hmm, that is mostly a cosmetic change bundled in to allow you to override the response code/message, but I can remove the RuntimeException catch as that's unnecessary/too broad.

Agreed, that also keeps the behaviour roughly as before (minus the extra info catches)

Alright, I've removed the extra catch block from the Java side.

:+1:  looks good to me!

I've rebased this to fix merge conflicts. Looks like the Ruby build fails in an unrelated issue, trying to fetch something from sourceforge to build docs.

@wesm is this good to merge from your side?

Rebased and fixed deprecations.

Rebased again to fix merge conflicts.

+1, sorry for the delay in merging this


Namespace registration does not consider namespaces registered via Topology

When namespace ads in the director are populated via topology, we would expect that an origin trying to register an already-registered-in-topology namespace should fail to register with the registry. However, we don't do anything to check this because the registry is completely unaware of topology (all of that code lives in the director).
I haven't checked the consequences of this, but I suspect the pelican origin will show appear in the Director as a valid origin for serving the same namespace.

Another note, I remember Brian talked about this when I tried to figure out the WebUrl VS Url for serverAd. If we want to handle the conflict between topology and registry, we might want to consider merging the ADs from topology and registry/director.


wanted to change my login's tried reinstalling it, but it has never worked anymore

That's my error, maybe you can help me?

require_certificate is missing in the list of options. See https://github.com/home-assistant/hassio-addons/blob/master/mosquitto/README.md for the latest documentation and example.


Added common Builder trait for EntityBuilder and LazyBuilder

PR for #422.
Hopefully I did it right.

This change is

There should also probably be a serde-gated MarkedBuilder with marked.

This is breaking change indeed (as witnessed tests failing because they don't import the trait). I am not against doing one for this as it seems that it would be useful.
It probably needs the aforementioned MarkedBuilder which should be extension trait to the Builder trait.

Also we need to add EntityResBuilder to this, which was merged yesterday.

EntityResBuilder has an extra parameter in it's with (a storage), not sure what to there.

@GraemeWilde ah yeah.  Ok, just ignore it then.

I also updated the changelog and the specs version to 0.12.0 (I think thats what will need to happen?).

Is it okay if I make all the doc mentions of stuff into links?

I used to do that but I don't anymore because the links don't work with
reexports.

Are you talking about when you pub use re-export something from a different crate?

@GraemeWilde Yes, but it doesn't even have to be in a different crate. If we reexport it in e.g. the prelude of Specs, the link will break.


Webview displays blank after going to background

When app loads for the first time, the webview works fine. If it goes in the background after some time, bringing the app back to the foreground shows nothing. What needs to be done to allow the webview to load again after it goes back to the foreground? Using react native

I experienced the same issue, will try to resolve this by wrapping Webview in a React Error Boundary to log the exact error and trigger an update in a key, so Webview component instance would be new.
My worry is Webview could be silently failing outside of ReactNative and the Error Boundary wouldn’t catch the error in this case, but will report in a few days with findings

Using onContentProcessDidTerminate resolved an issue for me.
What about you @melihozlem ?

It still happens

This is the code
    const webViewRef = useRef()
    return (
    <WebView
      source={{ uri: 'http://' }}
      onContentProcessDidTerminate={() => webViewRef.current.reload()}
      ref={webViewRef}
      style={{ marginTop: 20 }}
    />
    )

@melihozlem what version of react-native-webview are you using? I can’t reproduce your issue using the latest one. Your issue might be related to this PR that was recently merged

Using 13.3.0 in packge.json. Is there a cli command to check?

@melihozlem you could check in your lock file, depending on the package manager:

yarn.lock
package-lock.json
etc.

13.3.0 is the latest and it's unlikely you didn't reinstall packages the last time you updated the version in package.json.
So I think the simplest way to understand your issue would be for you to create a minimal reproducible example of how you use it.

What I pasted is the entire app. There is nothing else. If the app goes to background, if other apps are used, after some time, coming back shows white screen

Have similar issue.
"react-native-webview": "11.23.0",

same case happen to me in android @TheAlmightyBob

What platform is this on?

I'm on Android. After go background for 5-10min, this will happen.

I think for Android, we should use onRenderProcessGone
nativeEvent.didCrash will be false if this is just a normal technical adjustment that Android OS recycle the webview.
You can reproduce this even on development, just move expo to the background, then, Genshin, start! After a while you go back, will found the warning  WARN  WebView Crashed: false
Anyway, we can try reload the webview.
webViewReference.current?.reload?.(); will not work.
``ts
const [autoIncrementingNumber, setAutoIncrementingNumber] = useState(0);
return (
<WebView
key={autoIncrementingNumber}
onRenderProcessGone={syntheticEvent => {
const { nativeEvent } = syntheticEvent;
console.warn(
'WebView Crashed:',
nativeEvent.didCrash,
);
// webViewReference.current?.reload?.();
setAutoIncrementingNumber(autoIncrementingNumber + 1);
}}

seem to work.

We're seeing this issue when app goes into background for iOS and Android.

@melnikovkolya Do you know that your users are hitting onContentProcessDidTerminate? (do they see the alert from alertGeneralError() or are you collecting telemetry on this?)

We also struggle with this issue. After the app has been active OR inactive for a while, the webview suddenly goes blank. The issue can be reproduced by opening a bunch of other apps (preferably 5-10 large games). When the app is then opened again, onContentProcessDidTerminate fires and the webview goes blank.
This is a critical issue for us, as we would like to avoid refreshing the app to preserve the user state in the webview.

whenever I open gmeet and go inside the app again, it turn white screen ,         "react-native-webview": "^13.6.0",

Hi,
I managed to establish that this was happening due to the WebView process crashing on iOS.
I found the following script that allows to imitate the termination the WebView process.
#/bin/bash

# Source: https://gist.github.com/jstoone/c7b5a37dedc7cd2bab3696ca387e85f0

# Find process with name containing 'launchd_sim'
SIMULATOR_PID=$(pgrep launchd_sim)
echo "Simulator PID: $SIMULATOR_PID"

# Find sub-processes for the app's WebViews
WEBKIT_CONTENT_PID=$(pgrep -P $SIMULATOR_PID 'com.apple.WebKit.WebContent')
echo "WebKit WebContent PID(s):"
echo $WEBKIT_CONTENT_PID

# Send kill signal to the selected processes.
kill $WEBKIT_CONTENT_PID
echo "WebContent processes terminated."

In any case, my original issue was that I had some regular expression running inside the WebView (Safari) that used the lookbehind operator, and that was not supported on iOS up to version 16.4.

I think for Android, we should use onRenderProcessGone
nativeEvent.didCrash will be false if this is just a normal technical adjustment that Android OS recycle the webview. You can reproduce this even on development, just move expo to the background, then, Genshin, start! After a while you go back, will found the warning  WARN WebView Crashed: false
Anyway, we can try reload the webview.
webViewReference.current?.reload?.(); will not work.
  const [autoIncrementingNumber, setAutoIncrementingNumber] = useState(0);
  return (
    <WebView
      key={autoIncrementingNumber}
      onRenderProcessGone={syntheticEvent => {
        const { nativeEvent } = syntheticEvent;
        console.warn(
          'WebView Crashed:',
          nativeEvent.didCrash,
        );
        // webViewReference.current?.reload?.();
        setAutoIncrementingNumber(autoIncrementingNumber + 1);
      }}

seem to work. I don't know if this works for nativeEvent.didCrash === true.

Works fine also when didCrash = true.
Tested using source={{ uri: autoIncrementingNumber < 5 ? "chrome://crash" : EXPO_PUBLIC_APP_URL }} taken from Android docs.

fix: reload on both onRenderProcessGone onContentProcessDidTerminate

what's the aim of using key and autoIncrementingNumber instead of Webview reload?

Most probably, we just need to use onContentProcessDidTerminate.
Related issues:

Webview showing blank screen #2849
White screen on iOS + injectJavaScript does not work in that state #2298 (comment)
IOS WebView White Screen Issue #2302 (comment)

I experience the same issue with<EMAIL_ADDRESS>and<EMAIL_ADDRESS>Will try to:

Reload the WebView on onContentProcessDidTerminate
If it doesn’t work, then would wrap WebView in a React Error Boundary to log the exact error and trigger an update in a key, so WebView component instance would be new to see if it works w/o the necessity to clear WebView’s cache (Webview showing blank screen #2849).

<ErrorBoundary> // onError, it increments autoincrementingNumber
  <WebView key={autoincrementingNumber} />
</ErrorBoundary>

In my case, we don’t load any 3rd party’s code, and there’s no state within the app that could go corrupted, so I’m certain the issue is on ReactNative’s side. My worry is: Webview could be silently failing outside of ReactNative and the Error Boundary wouldn’t catch the error in this case, but will report in a few days with findings.

what's the aim of using key={autoIncrementingNumber} instead of reloading the Webview?

@mrzslr

instead of reloading the Webview?

Due to this bug https://github.com/react-native-webview/react-native-webview/issues/3126 , I'm using
source={{
            html: `<!doctype html><meta charset="UTF-8" />
`,
            /**
             * Add baseUrl to fix `SecurityError: Failed to read the 'localStorage' property from 'Window': Access is denied for this document.`
             * @url https://github.com/react-native-webview/react-native-webview/issues/1635#issuecomment-1021425071
             */
            baseUrl: 'http://localhost',
          }}

So I can't directly refresh it. I need to trigger full react component reload, so its preload script take effect and fetch content in preload script and setInnerHTML.
But I guess directly refresh it will also work, if you are using a proper uri or source.

Thank you for the answer!

We also struggle with this issue. After the app has been active OR inactive for a while, the webview suddenly goes blank.
"react-native-webview": "^13.6.4",

I should have keep this issue opened.我应该一直打开这个问题。
While there is a workaround, but page reload will always cost a lot of time, make user experience not good.虽然有解决方法，但页面重新加载总是会花费大量时间，使用户体验不佳。
Is the solution for these two function do a  key refresh?：
onRenderProcessGone
onContentProcessDidTerminate

So is this problem fixed in latest version or not? Why is the issue closed without any clear statement in this regard?

It is not fixed. We can open a new issue, and be aware not letting @github-actions close it.

Who came up with the silly idea of automating closing of issues in this way? An issue should be closed when it's resolved or the maintainers decide they won't fix it. Either way at least one knows what he is expected to do next.

Either way at least one knows what he is expected to do next.

If the webview is crashed, calling the reload function won't do anything. In that case we create a new instance of the webview by changing the key

Also struggling with this on iOS. Why is that ErrorBoundarys do not work for this prob?

I've been really struggling with this over a week and almost ready to say I can't use Webview because of this. Having a white screen of death just sucks
The core of the problem lies in understanding when you need to reload the Webview vs just reloading it every time it comes to the foreground
I found this on another issue I'm going to try at least for crashing and since we are using shareIntent I'm going to check this as wwlll
const WebViewExample = ({uri}) => {
const webViewRef = useRef();
// You can use uuid instead.
// We have to change the key because sometimes it is not
// re-rendered in android platform after reload.
const generateRandomKey = () => parseInt(Math.random() * 100000, 10);

const [key, setKey] = useState(generateRandomKey());

return (
    <WebView
        key={key}
        ref={webViewRef}
        syle={{flex: 1}}
        source={{uri: uri}}
        onLoadEnd={data => {
            const {nativeEvent} = data;
            const {title} = nativeEvent;

            if (!title.trim()) {
                webViewRef.current?.stopLoading();
                webViewRef.current?.reload();
                setKey(generateRandomKey());
            }
        }}
    />
);

};


How can I change the color of the NODE in the ‘weathermap’ plugin?

’ve installed weathermap, and I’m looking to create an image where it shows whether the NODE is UP or DOWN. I’d like it to be green when the device is functioning normally, and turn it to red when it’s malfunctioning. However, it seems to be just white.
Currently, my LINK can change its color based on bandwidth, but the NODE is always white. I’m not sure where I went wrong with the settings. Below is my configuration file.
HTMLSTYLE overlib
HTMLOUTPUTFILE A_Rack.html
IMAGEOUTPUTFILE A_Rack.png
KEYPOS DEFAULT -1 -1 Traffic Load
KEYTEXTCOLOR 0 0 0
KEYOUTLINECOLOR 0 0 0
KEYBGCOLOR 255 255 255
BGCOLOR 255 255 255
TITLECOLOR 0 0 0
TIMECOLOR 0 0 0
SCALE green 0 0   0 255 0     # Green color range for normal status
SCALE red 1 1   255 0 0       # Red color range for abnormal status
SET key_hidezero_DEFAULT 1
NODE DEFAULT
MAXVALUE 100
LINK DEFAULT
BANDWIDTH 100M
NODE 22
LABEL Switch6
INFOURL http://<IP_ADDRESS>/device/device=22/
OVERLIBGRAPH http://<IP_ADDRESS>/graph.php?height=100&width=512&device=22&type=device_bits&legend=no
POSITION 142 155
USESCALE green absolute
USESCALE red absolute
SET category bandwidth

Where do you have data available for whether the device is up or down?
Weathermap just turns numbers into colours - you need to have the numbers
from somewhere...
On Thu, 18 Jan 2024, 05:17 Julian, @.***> wrote:

’ve installed weathermap, and I’m looking to create an image where it
shows whether the NODE is UP or DOWN. I’d like it to be green when the
device is functioning normally, and turn it to red when it’s
malfunctioning. However, it seems to be just white.
Currently, my LINK can change its color based on bandwidth, but the NODE
is always white. I’m not sure where I went wrong with the settings. Below
is my configuration file.
`HTMLSTYLE overlib
HTMLOUTPUTFILE A_Rack.html
IMAGEOUTPUTFILE A_Rack.png
KEYPOS DEFAULT -1 -1 Traffic Load
KEYTEXTCOLOR 0 0 0
KEYOUTLINECOLOR 0 0 0
KEYBGCOLOR 255 255 255
BGCOLOR 255 255 255
TITLECOLOR 0 0 0
TIMECOLOR 0 0 0
Define color scales
SCALE green 0 0 0 255 0 # Green color range for normal status
SCALE red 1 1 255 0 0 # Red color range for abnormal status
Default settings
SET key_hidezero_DEFAULT 1
TEMPLATE-only NODEs:
NODE DEFAULT
MAXVALUE 100
TEMPLATE-only LINKs:
LINK DEFAULT
BANDWIDTH 100M
Regular NODEs:
NODE 22
LABEL Switch6
INFOURL http://<IP_ADDRESS>/device/device=22/
OVERLIBGRAPH
http://<IP_ADDRESS>/graph.php?height=100&width=512&device=22&type=device_bits&legend=no
POSITION 142 155
Use USESCALE to specify colors
USESCALE green absolute
USESCALE red absolute
SET category bandwidth
`
—
Reply to this email directly, view it on GitHub
https://github.com/howardjones/network-weathermap/issues/316, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAG6YJJJAIL7A4UN3FQEIZ3YPCV5JAVCNFSM6AAAAABB7VBO3OVHI2DSMVQWIX3LMV43ASLTON2WKOZSGA4DONJSGMYDCMI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Sorry, I'm a beginner. Could you please help me by providing some examples to show me how to configure it?


Cursor donut should be hidden?

Not sure if this is expected?   Or if you should be able to edit the cursor as part of the controls?

I think it's expected. Cursor isn't anything special, it's just a mesh + raycaster in front of the camera. I don't think it's important enough to warrant special treatment, and lets the editor remain a bit agnostic.

Yes, it's expected, actually you could edit it and change the material or geometry as you want just like any other entity added to your scene.


feat(cone/caratheodory): add caratheodory theorem for convex cones

Carathéodory's theorem states that if x is in the convex cone generated by a set s, then x is a convex combination of linearly independent elements of s. This is the replacement of the usual statement of vector spaces having bases for the setting of convex cones.
This is needed for proving duality theorems for linear programs.

[x] depents on: #8878

It looks like the repeats all the work used to establish convexHull_eq_union except now in this more restricted setting. Wouldn't it be possible to use convexHull_eq_union to obtain this result. E.g., is this true?
lemma toPointedCone_eq_smul_convexHull :
    s.toPointedCone 𝕜 = {y | ∃ x ∈ convexHull 𝕜 (s ∪ {0}), ∃ t : 𝕜, 0 ≤ t ∧ t • x = y} :=
  sorry

It looks like this repeats all the work used to establish convexHull_eq_union except now in this more restricted setting. Wouldn't it be possible to use convexHull_eq_union to obtain this result. E.g., is this true?
lemma toPointedCone_eq_smul_convexHull :
    s.toPointedCone 𝕜 = {y | ∃ x ∈ convexHull 𝕜 (s ∪ {0}), ∃ t : 𝕜, 0 ≤ t ∧ t • x = y} :=
  sorry

This statement is true but you still need a way to go from AffineIndependent to LinearlyIndependent. The fact that the difference vectors from a single point in an AffineIndependent set are LinearlyIndependent is not sufficient here. I could not figure out to cross this bridge.

Are you saying that https://leanprover-community.github.io/mathlib4_docs/Mathlib/LinearAlgebra/AffineSpace/Independent.html#affineIndependent_iff_linearIndependent_vsub is not sufficient?
If so, can you provide a formal statement of what you need?

Ok, I found a reference online that uses one to prove the other - although it is in the other direction.
I'll try to see if I can flip the proof: https://www.math.cuhk.edu.hk/course_builder/1819/math4230/caratheodory.pdf

Shameless self-advertising: Corollary 1.2.2 in
https://webusers.imj-prg.fr/~antoine.chambert-loir/enseignement/2020-21/gt/toptrop.pdf

Shameless self-advertising: Corollary 1.2.2 in https://webusers.imj-prg.fr/~antoine.chambert-loir/enseignement/2020-21/gt/toptrop.pdf

Thank you! I'll open up a thread on Zulip to get help with formalizing this.

@AntoineChambert-Loir
I've looked at the proof in your notes and while it does connect the two Caratheodory theorems it is much much longer. The easier direction in your proof is to prove the Cone version version and then use it to prove the Conic Hull version. Here we are trying to do the other direction which goes through several intermediate steps, each of which will be a big Mathlib PR. I won't have time to do this right now.
@ocfnash Let me know what you suggest. There is no shortcut to go from Convex Hull to Cones (there is one to go the other way around).

AntoineChambert-Loir
I've looked at the proof in your notes and while it does connect the two Caratheodory theorems it is much much longer. The easier direction in your proof is to prove the Cone version first and then use it to prove the Conic Hull version. Here we are trying to do the other direction which goes through several intermediate steps, each of which will be a big Mathlib PR. I won't have time to do this right now.
ocfnash Let me know what you suggest. There is no shortcut to go from Convex Hull to Cones (there is one to go the other way around).

I suggest you start by proving the lemma whose statement I provided here --- it would be a useful contribution in its own right and it looks to me like it should be easy to obtain toPointedCone_eq_union from convexHull_eq_union using it.
I find it very hard to believe there is no shortcut between these two results, but even if there is something I have overlooked we prefer to put in the extra work and avoid the sort of code duplication currently proposed. Mathlib works because we all put in the extra effort.

Sounds good. I'll work on these smaller lemmas first.

(since Apurva has contributed extensively to mathlib3)

@apurvanakade Are you planning to continue working on this PR? If not, please feel free to close it.

I haven't had time to work on Mathlib of late, sorry. I'll close the PR.

Feel free to reopen it when you have time again!


Grab edit does not work

The tilde grab mode has been broken for a while.
It is useful for developing scenes, so we need to fix that bug.

Possibly related: https://github.com/webaverse/app/pull/2800
Though that PR doesn't come close to what we need from this feature yet (smooth movement, pulling into inventory, helpers UI, drop placement).

Closing in favor of the old issue https://github.com/webaverse/app/issues/2527


Group logs from requests and responses

Right now, I am using HttpLoggingInterceptor.Logger (com.squareup.okhttp3:logging-interceptor:4.4.1) to intercept logs from Retrofit and log them into the console or file (using https://github.com/elvishew/xLog)
The thing is I want to group logs from a request or response rather just logging whatever strings comes from
HttpLoggingInterceptor.Logger.log(message: String)
So I subclassed HttpLoggingInterceptor.Logger and worked on a simple string manipulation logic here to group these logs.
abstract class MyHttpLogger : HttpLoggingInterceptor.Logger {
    private var sb = StringBuilder()

    /**
     * This combines logs from Retrofit into one block of related logs (request, response)
     * Sample request:
     * --> GET https://domain.com/api/v1/endPoint
     * --> END GET
     * Sample Response:
     * <-- 200 https://domain.com/api/v1/endPoint (167ms)
     * ...
     * <-- END HTTP (2871-byte body)
     *
     * @param message the line-by-line logs from Retrofit
     */
    override fun log(message: String) {
        if (message.isBlank()) return

        try {
            if (message.startsWith("-->")) {
                if (message.contains("--> END")) {
                    // its the end part of the request network call logs
                    sb.append(message)
                    groupedLog(sb.toString()) // logs that should be grouped
                    sb.clear()
                } else {
                    // its the beginning part of the request network call logs
                    sb.clear()
                    sb.append(message + "\n")
                }
            } else if (message.startsWith("<--")) {
                if (message.startsWith("<-- END")) {
                    // its the end part of the response network call logs
                    sb.append(message)
                    groupedLog(sb.toString()) // logs that should be grouped
                    sb.clear()
                } else {
                    // its the beginning part of the response network call logs
                    sb.clear()
                    sb.append(message + "\n")
                }
            } else if (sb.isNotEmpty()) {
                // middle logs between the beginning and ending tags
                sb.append(message + "\n")
            }
        } catch (e: Exception) {
            e.printStackTrace()
        }
    }

Most of the time this works. But when multiple APIs are called at the same time, I expect that logs from those calls are messed up here and it breaks the logic.
So there should be a way like an ID or source param in override fun log(message: String) to know where this log is coming from.
Like is it coming from request of API#1, or response from API#2, etc.
What do you think?

HttpLoggingInterceptor isn't under particularly active development.  And we wouldn't break the public APIs of it.
So probably best to copy the implementation into your project and make the changes you need.
You could also look at EventListeners, which are another way of getting call events.
Or look at alternatives such as https://github.com/ihsanbal/LoggingInterceptor


Rename "citadel" cert to "istiod" (#21613)

Rename "citadel" cert to "istiod"

Additionally, use standard naming and directory structure

fix build

fix multiple register

Fix one more path

regen kustomize

fix gen

(cherry picked from commit 07e8a0ca275d3cf956d28f5244f54ae939a84e3a)

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio_release-1.5
e7f4e07b75e4966c90b225add8da748b30c80f28
link
/test gencheck_istio_release-1.5

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


UnicodeConversionError in Surface compiler

Issue #90, which is already fixed, reports a problem with special characters used in expressions.
I ran into a similar problem and am not sure how this case is different. The problem can be reproduced by using the same input as in #90 in the compiler_test.exs file:
Test to reproduce the problem
  test "component with expression using special characters in the parameter" do
    code = """

{{ "héllo" }}

    """

    [node | _] = Surface.Compiler.compile(code, 1, __ENV__)
  end

Error
** (UnicodeConversionError) invalid encoding starting at <<233, 108, 108, 111, 34, 32>>
     code: [node | _] = Surface.Compiler.compile(code, 1, __ENV__)
     stacktrace:
       (elixir 1.11.2) lib/string.ex:2299: String.to_charlist/1
       (elixir 1.11.2) lib/code.ex:838: Code.string_to_quoted/2
       (surface 0.1.1) lib/surface/compiler/helpers.ex:30: Surface.Compiler.Helpers.interpolation_to_quoted!/2
       (surface 0.1.1) lib/surface/compiler.ex:210: Surface.Compiler.convert_node_to_ast/3
       (surface 0.1.1) lib/surface/compiler.ex:169: anonymous fn/3 in Surface.Compiler.to_ast/2
       (elixir 1.11.2) lib/enum.ex:2181: Enum."-reduce/3-lists^foldl/2-0-"/3
       (surface 0.1.1) lib/surface/compiler.ex:168: Surface.Compiler.to_ast/2
       (surface 0.1.1) lib/surface/compiler.ex:275: Surface.Compiler.convert_node_to_ast/3
       (surface 0.1.1) lib/surface/compiler.ex:169: anonymous fn/3 in Surface.Compiler.to_ast/2
       (elixir 1.11.2) lib/enum.ex:2181: Enum."-reduce/3-lists^foldl/2-0-"/3
       (surface 0.1.1) lib/surface/compiler.ex:168: Surface.Compiler.to_ast/2
       (surface 0.1.1) lib/surface/compiler.ex:111: Surface.Compiler.compile/5
       test/compiler_test.exs:94: (test)

Fixed in https://github.com/msaraiva/surface/commit/24da80b5dbd2225987fe95a44ebd1cb7fc19abe2.
@hok thanks for the report and the provided test! ❤


[WIP]2.x

Support get result from stream
Support get uri of result in s3
Support Typescript definition

Coverage decreased (-5.03%) to 92.308% when pulling 29e5f024f7d8615f71e49ebfe4104df928ca297d on 2.x into 346713ce584b3fb6cc77d94d336766bad028acd2 on master.

Coverage decreased (-5.03%) to 92.308% when pulling 29e5f024f7d8615f71e49ebfe4104df928ca297d on 2.x into 346713ce584b3fb6cc77d94d336766bad028acd2 on master.

Coverage decreased (-5.03%) to 92.308% when pulling 6350cfac3c06c49aa3954c45569d991bdcef9024 on 2.x into 346713ce584b3fb6cc77d94d336766bad028acd2 on master.

Coverage decreased (-5.03%) to 92.308% when pulling 6350cfac3c06c49aa3954c45569d991bdcef9024 on 2.x into 346713ce584b3fb6cc77d94d336766bad028acd2 on master.

Coverage decreased (-3.8%) to 92.308% when pulling fb66485dc41526101b83e829ae685172313ac267 on 2.x into 28e58b8aa4550c5a4d31891100d55d0f1aaca5db on master.

Coverage decreased (-3.8%) to 92.308% when pulling 5e6e6b7c901e566d65e620fa21ae37f66f644cc4 on 2.x into 28e58b8aa4550c5a4d31891100d55d0f1aaca5db on master.


katex: init at 0.10.2

Motivation for this change
Add katex, a program that accepts a subset of TeX math and converts it to HTML.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

nodePackages_10_x version works fine though there is no attribute for the nodePackages_8_x package (I'm assuming there should be one as a load of node v8 files have changed). I don't really know enough about js to review properly and I don't know how hard this would be to merge at this point

@mt-caret Do you still need katex to be included in nixpkgs?

Please rework your PR. It now has a merge conflict after PR https://github.com/NixOS/nixpkgs/pull/89184 has been merged

@veprbl @prusnak Apologies for the late response. I'm no longer super interested in including this in nixpkgs, so I'm closing this PR.


Obstacle in state-based case

Hi, I've got question about obstacle definition in state-based case. README states about "metric distance to obstacles". On the other hand Obstacle msg has position and scale. Is position relative? What exactly is scale? Regards,

The relative position between the center of the quadrotor and the center of the sphere obstacles.
The scale means the size of the obstacles, in radius.
it is defined in
here

Great, thanks


build: Update Vulkan-Headers and VUL to 1.3.278

CI Vulkan-ExtensionLayer build queued with queue ID 135800.

CI Vulkan-ExtensionLayer build # 842 running.

CI Vulkan-ExtensionLayer build # 842 passed.


gopkg import with the mocks subpackage

The mocks subpackage imports github.com/Shopify/sarama. The README suggests using gopkg.in  (which github.com/wvanbergen/kafka does), but it leads to conflicts when using mocks like so:
cannot use "gopkg.in/Shopify/sarama.v1".NewConfig() (type *"gopkg.in/Shopify/sarama.v1".Config) as type *"github.com/Shopify/sarama".Config in argument to mocks.NewSyncProducer

Hmm, cc @wvanbergen this is actually kind of annoying; if we switch the import paths for the mocks to use gopkg, then it will break for anybody using master directly (of which there are at least a few)...

It might be possible to work around this via https://golang.org/doc/go1.4#canonicalimports? I'm not 100% sure, I think it depends on exactly how gopkg.in proxies the repo.

/me suppresses rant about the shitty package management tooling in go.
I don't know how that would work. Based on those docs, adding a canonical import path will just make one of the two locations fail. gopkg.in would have to do some rewriting as part of the proxying for it to work, but that is kind of against the idea of a canonical import path.

maybe we can just use a relative import? import './mocks'

Relative imports break go get and a bunch of other things, I think... maybe worth trying though to see?

Unfortunately it seems like there is still no real solution to this issue yet. Here are my thoughts:
Did you consider moving the mocks into the github.com/Shopify/sarama package directly? There are no extra dependencies this would introduce there and just simplify the code and imports. This change could be introduced under a new major version (i.e. gopkg.in/Shopify/sarama.v2) since moving the mocks package obviously breaks BC.
Of course this would break all users that import the github repository directly but if this is actually an issue for the library then I wonder why you are using gopkg.in in the first place. Maybe the actual solution should be to stop advertising the usage of gopkg.in alltogether?

I wonder if this is still an issue with go modules?

@bai since Sarama requires go modules for some time now, is there any benefit to still using gopkg.in anymore? Perhaps we should enable the canonical import path of github.com/Shopify/sarama everywhere?

@dnwe I'm 👍for this change.

Closing this as outdated, please re-open if you still think there is something we can do here


Correct the bug report for cargo clippy --fix

What does this PR try to resolve?
close https://github.com/rust-lang/cargo/issues/11877
Correct the bug report for cargo clippy --fix.
How should we test and review this PR?
See the unit tests.

Some tests like check_fixable_warning_for_clippy do similar stuff I believe.

@weihanglo Friendly ping~
Could you please take a look when you have time? Thanks!

Only some minor suggestions.

Thanks for your review!  💚 💙 💜 💛 ❤ All addressed!

@bors r+

:pushpin: Commit 08169fd015c4b4aeff2118753c7adb5d873f908b has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit 08169fd015c4b4aeff2118753c7adb5d873f908b with merge b0742b2145f02d3557f596d1ee4b36c0426f39ab...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing b0742b2145f02d3557f596d1ee4b36c0426f39ab to master...


Tooltip message failing to display and config settings not affecting build

I have managed to get this somewhat working - but I'm having to use a few workarounds due to the config settings not working.
const markdownLibrary = markdownIt({
  html: true
}).use(codeClipboard.markdownItCopyButton, {
  // Style attributes of clipboard icon. default to the following.
  iconStyle: 'font-size: 15px; opacity: 0.8;',
  // Class attributes of clipboard icon. default to "mdi mdi-content-copy".
  iconClass: 'mdi mdi-content-copy',
  // Name of HTML tag of clipboard icon. default to span.
  iconTag: 'span',
  // Name of clipboard button css class. default to code-copy.
  // This class should be the same as the plugin options
  buttonClass: 'code-copy',
  // Style attributes of button. default to the following.
  buttonStyle: 'position: absolute; top: 7.5px; right: 6px; cursor: pointer; outline: none; opacity: 0.8;',
  // Additional class attributes in addition to the plugin option(buttonClass). default to empty.
  additionalButtonClass: '',
  // Name of title attribute in Button. default to "Copy".
  title: 'Copy',
});

None of the attributes above are affecting the output when I test it.

I checked custom attributes and it works fine(tested with eleventy 1.0.0 and 2.0.0-canary.23).
Did you rebuild(or start) eleventy after the change?

Not sure what it was. Seems to be fixed now. Sorry about that!