FIX Handle exceptions in the verify method

Fix for https://github.com/silverstripe/silverstripe-mfa/issues/378
Travis failure is PHPCS not liking <?php declare(strict_types=1); which was added previously to some various files
I'm thinking we just remove the strict_types?

I'm thinking we just remove the strict_types?

Drop the declaration to a new line and PHPCS will be happy. You can also run vendor/bin/phpcbf and provide the module name to fix.

 I think that the interface should deal with a 500 response, rather than just removing the error in this one place. Other errors thrown (resulting in 500s) are going to have the same problem as described in #378.

@ScopeyNZ
a) What are legitimate errors?
This is catching unhandled expections that are not turned into json (via Result::create()).  On a dev environment I guess this isn't great because you get the exception output in the response, I guess getting 'The was an unknown error' back isn't so helpful there.  On live though you just get a blank response and a 500 status code, hence the endless spinner.
b) What is "the interface"?  (Sorry I'm not too familiar with this module)

So the interface would be the thing that dispatches a request, and shows a spinner while it waits for the response. That should be updated to handle 500s. I don't know what the 500 error is here, but you're catching Exception, which is just one step above catching Throwable. The server throws a 500 because something's wrong. That's perfectly fine behaviour. Returning 200 OK - Everything is wrong is not really a good result.

I was being a little terse there, it's actually returning a 401, but still "an unknown error occurred" with a 4xx response (client problem), is not really semantically correct. It should be a 5xx response.

@ScopeyNZ OK I have a corresponding one that I think does this for 429 on the mfa module - https://github.com/silverstripe/silverstripe-mfa/pull/401 - so would it work if we close this PR and update the other PR to handle 500?

Sound good to me.
In fact, you could probably just fill in the "default" case in that with an "unknown error" message.

@ScopeyNZ OK have removed the 500 logic from this PR.  I've kept this PR open to just fix up the PHPCS issues


Add new custom robot failed

Hi,
I am trying to add a new robot into Maniskill environment. The robot code is here.
Following the tutorial, demo_robot  is invoked as here.
At first, it works properly. However, after pulling the latest commit changes of Maniskill, I encounter the following issue. It seems related to this update, but I am not sure the reason for this.

2024-11-11 17:04:12,744 - mani_skill  - WARNING - Agent drv70l_robotiq is already registered. Skip registration.
Traceback (most recent call last):
File "/home/eric/anaconda3/envs/maniskill/lib/python3.9/site-packages/gymnasium/envs/registration.py", line 802, in make
env = env_creator(**env_spec_kwargs)
File "/home/eric/ManiSkill/mani_skill/utils/registration.py", line 182, in make
env = env_spec.make(kwargs)
File "/home/eric/ManiSkill/mani_skill/utils/registration.py", line 79, in make
return self.cls(_kwargs)
File "/home/eric/ManiSkill/mani_skill/envs/tasks/empty_env.py", line 25, in init
super().init(*args, robot_uids=robot_uids, **kwargs)
File "/home/eric/ManiSkill/mani_skill/envs/sapien_env.py", line 335, in init
obs, _ = self.reset(seed=[2022 + i for i in range(self.num_envs)], options=dict(reconfigure=True))
File "/home/eric/ManiSkill/mani_skill/envs/sapien_env.py", line 832, in reset
self._reconfigure(options)
File "/home/eric/ManiSkill/mani_skill/envs/sapien_env.py", line 670, in _reconfigure
self._load_agent(options)
File "/home/eric/ManiSkill/mani_skill/envs/tasks/empty_env.py", line 38, in _load_agent
super()._load_agent(options, sapien.Pose())
File "/home/eric/ManiSkill/mani_skill/envs/sapien_env.py", line 419, in _load_agent
agent: BaseAgent = agent_cls(
TypeError: init() got an unexpected keyword argument 'initial_pose'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
File "/home/eric/ManiSkill/mani_skill/agents/robots/drv70l/test.py", line 3, in
demo_robot_script.main()
File "/home/eric/ManiSkill/mani_skill/examples/demo_robot.py", line 35, in main
env = gym.make(
File "/home/eric/anaconda3/envs/maniskill/lib/python3.9/site-packages/gymnasium/envs/registration.py", line 814, in make
raise type(e)(
TypeError: init() got an unexpected keyword argument 'initial_pose' was raised from the environment creator for Empty-v1 with kwargs ({'obs_mode': 'none', 'reward_mode': 'none', 'enable_shadow': True, 'control_mode': 'pd_joint_pos', 'robot_uids': 'drv70l_robotiq', 'sensor_configs': {'shader_pack': 'default'}, 'human_render_camera_configs': {'shader_pack': 'default'}, 'viewer_camera_configs': {'shader_pack': 'default'}, 'render_mode': 'human', 'sim_config': {'sim_freq': 100, 'control_freq': 20}, 'sim_backend': 'auto'})

Here is the environment:

gymnasium                 0.29.1
mani-skill                3.0.0b12
pip                       24.2
python                    3.9.20
pytorch                   2.3.0
pytorch-kinematics        0.7.4
pytorch-seed              0.2.0
sapien                    3.0.0b1
scipy                     1.13.1
setuptools                75.1.0
torch                     2.5.1
torchaudio                2.5.1
torchtriton               2.3.0
torchvision               0.20.1

Thanks.

The bug is here
https://github.com/ruoxianglee/ManiSkill/blob/305da944f12be79ff0a9034508873ac23e8de831/mani_skill/agents/robots/drv70l/drv70l.py#L45-L73
I recommend following the current panda.py arm implementation, you don't need to override the original __init__ function. You can, but make sure to pass any *args, **kwargs from init to the super init.

Thanks. It works.


Last resilience4j release fully support Reactor , RxJava2 and completable future in their spring boot starters AOP

Hey All ,
Small question .
Last resilience4j release fully support Reactor , RxJava2 and completable future in their spring boot starters AOP , can we reuse it here or the spring cloud starter here is serving different needs  like have unified way to define circuit breaker whatever the implementation behind it ?
i can help if you would like.
Thanks a lot,

@Romeh can you describe more about how you would think it would change the implementation?  We already support reactive types

@ryanjbaxter
Right now the resilience4j spring support the following return types :

Completablefutre
RxJava2
Reactor
synchronous java types

Plus the spring aspect annotation support (maybe it is not planned here for the spring cloud circuit breaker usage ).
can we add the missing 2 types as well (RxJava2 , completable futures)?
Thanks a lot,
Romeh

@ryanjbaxter  gentle reminder for your valuable feedback ;)

@Romeh thanks for the reminder.  The team was having a face to face meeting this week so I was waiting to talk about it in person.  What we were leaning towards possibly having a CircuitBreaker interface that returns a Publisher rather than Mono and Flux.  I think that would cover most of the cases above.
Monitoring is something we will want to look into as well, at least come up with examples showing that because the underlying library is going to expose that information for us.

@ryanjbaxter ,
For the type of metrics and health endpoints exported under spring boot starter for resilience4j are documented here :
https://resilience4j.readme.io/docs/getting-started-3#section-metrics-endpoint

we have also micrometer exporter which can be used with dashboards: https://resilience4j.readme.io/docs/micrometer
Grafana support : https://resilience4j.readme.io/docs/grafana-1

Please ping me whenever u have questions !

Regarding supporting other Publisher types.  It did not seem to make sense to try and support such a generic interface, as the Publisher interface itself does not offer much functionality and we found that the user would have to cast it back to the type they passed in anyways.  We could try and figure out the type but that might end badly if we guess wrong.  So in the end we decided to leave the interface as it and keep it to just Mono and Flux.  That said if users do want to use something like RxJava2 or Futures etc they can use the RxJava2Adapter from Project Reactor or the methods themselves on Mono and Flux to adapt to the APIs.  We can always add direct support for these libraries if we find a strong need to.


feat(autoware_launch): add rviz ad api adaptors

Signed-off-by: Takagi, Isamu<EMAIL_ADDRESS>Description

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[ ] I've confirmed the contribution guidelines.
[ ] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

Duplicates https://github.com/autowarefoundation/autoware_launch/pull/70.


enhancing wishlist and help page in profile

Related Issue
[Cite any related issue(s) this pull request addresses. If none, simply state “None”]
Fixes:  #4685
Description
enhancing the wishlist page in my profile and help page in dropdown menu of profile

Type of PR

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes. Make sure to attach before & after screenshots in your PR.]
Before
https://github.com/user-attachments/assets/2362d16d-6394-410c-b7ec-484ddb4cdd47
vs After:
Checklist:

[x] I have made this change from my own.
[x] I have taken help from some online resources.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers and screenshots after making the changes.

the book wishlist page isnt responsive , fix it also .

the book wishlist page isnt responsive , fix it also .

https://github.com/user-attachments/assets/656e02ea-114e-4588-8277-bd00d771929f

there are 2 options Settings and inbox .
u can either create some page for them ... or delete those from dropdown .
there are other pages as well like badges , blog and contributor page . which need enhancement .
u may work on these issues and raise pr .


No way to re-enable the arrow keys

I want to be able to use the arrow keys to move around, but they are disabled by default, and it no longer seems possible to change this
Previously, I set disabled_keys to an empty table and that worked, but it no longer does, I've also tried
disabled_keys = {
	["<UP>"] = nil,
	["<DOWN>"] = nil,
	["<LEFT>"] = nil,
	["<RIGHT>"] = nil,
}

but that did not work

You should change it to:
disabled_keys = {
	["<UP>"] = {},
	["<DOWN>"] = {},
	["<LEFT>"] = {},
	["<RIGHT>"] = {},
}

See the readme for more information.


[vcpkg] shorten license instruction in  portfile.in.cmake

@atkawa7,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot


Wrong phred score base with low quality Nanopore data

I'm playing with confidr and Nanopore data right now and get an error from bbduk_bait.
Traceback (most recent call last):
  File "/home/bioinfo/miniconda3/envs/confindr/lib/python3.7/site-packages/confindr_src/confindr.py", line 1067, in confindr
    fasta=args.fasta)
  File "/home/bioinfo/miniconda3/envs/confindr/lib/python3.7/site-packages/confindr_src/confindr.py", line 647, in find_contamination
    returncmd=True, threads=threads)
  File "/home/bioinfo/miniconda3/envs/confindr/lib/python3.7/site-packages/confindr_src/wrappers/bbtools.py", line 258, in bbduk_bait
    out, err = run_subprocess(cmd)
  File "/home/bioinfo/miniconda3/envs/confindr/lib/python3.7/site-packages/confindr_src/wrappers/bbtools.py", line 16, in run_subprocess
    raise subprocess.CalledProcessError(x.returncode, cmd=command)
subprocess.CalledProcessError: Command 'bbduk.sh in=/home/bioinfo/analyses/bacillus_gta_cfia_run1/filtered/BA-12-InfantFormula_filtered.fastq.gz outm=/home/bioinfo/analyses/bacillus_gta_cfia_run1/qc/confindr/BA-12-InfantFormula_filtered/trimmed.fastq.gz ref=/media/36tb/db/rMLST_2022-08-10/Bacillus_db.fasta threads=48' returned non-zero exit status 1.

Running the bbduk command to have a more explicit error message gave:
Input is being processed as unpaired
Started output streams:	0.039 seconds.
Changed from ASCII-33 to ASCII-64 on input Z: 90 -> 59

The ASCII quality encoding offset (64) is not set correctly, or the reads are corrupt; quality value below -5.
Please re-run with the flag 'qin=33', 'ignorebadquality', or '-da'.
Problematic read number 0:

The error is caused because the quality scores are low and bbduk sets automatically the phred score base to 64 instead of 33. Maybe it adding an option to set the phred score scheme would help resolve that issue. Or you could simply hard code qin=33 in all the bbduk commands since most usable sequencing data are using a 33-based phred score.

 Respond to Marc Duceppe's issue

Hi @duceppemo, thanks for pointing this out—we'll implement one of those two options into the upcoming version of ConFindr (probably the hard-coding).

Resolved by ac3b976.


socket:  code:1001 reason :Stream end encountered

Integration Method: CocoaPods
Xcode Version: 11.3.1
Library Version:
Platform: iOS
Language: Objective-C
Description:
Expected Behavior:
"webSocketDidOpen" then "didCloseWithCode"
code:1001 reason :Stream end encountered
socket url is "wss://switchboard.mixpanel.com/connect?key=***&type=device"

Hi @AracyGit , do you still have the issue? It might be a connectivity issue at that point. I will close this one now but please contact  Mixpanel support if the problem exists or reopen it if you think this is SDK related.


A second set of eyes on Tractability guide needed

https://github.com/uccser/cs-field-guide/blob/guide-content/text/curriculum-guides/ncea/en-assessment-guide-level-3-complexity-tractability-TSP.md

Checked, minor wording changes made
commit 3cabead8eb6358c41e9da88f77ec2f2b7816ab5c


Sum Of Highest & Lowest Element Of An Array

DS In C To Find Sum Of Highest & Lowest Element Of An Array

Thank you for your contribution 👍

Thank You Brother For Accepting My Pull Request


Which asset extractor did you use?

Some images failed to extract.
I used Asset Ripper and Asset Studio.
Asset Ripper is missing files more than Asset Studio.
OS: windows11.
App Version: 4.1.0
App region: jp
Unity Settings: 2022.3.21f1
I don't remember exactly but byte -1 is in the error message.

I've mostly used Razmoth's fork of AssetStudio besides UnityPy
With the latter I've also used it in sssekai_blender_io that does involve image/Texture2D conversion
Technically (for Unity textures) the latest versions UnityPy should offer the best compatibility since it's using the de facto ASTC texture codec implementation whilst using AssetStudio's Texture2D converter to handle other formats
The game (just like most, if not all mobile Unity games) uses ASTC for the majority of its texture encoding - I'd assume it's one of these textures that didn't work on AssetStudio/AssetRipper since their implementations of ASTC codec probably doesn't offer the same robustness of the offical one which could cause them to fail on some edge cases:

https://github.com/Perfare/AssetStudio/blob/master/Texture2DDecoderNative/astc.cpp
https://github.com/AssetRipper/TextureDecoder/blob/master/AssetRipper.TextureDecoder/Astc/AstcDecoder.cs

I've dug around a bit and found some related issues as well,which again suggests it's the issues with the said format.

https://github.com/AssetRipper/AssetRipper/issues/1119
https://github.com/AssetRipper/TextureDecoder/issues/59 (deleted for some reason)

TL;DR try Raz's fork or dump the textures with UnityPy. BTW I'd be interested in the files that don't work if you can post them here too^^

Thanks for your answer and cool program!
I will test it in the next version.


Clean up CONTRIBUTING

refs https://github.com/keep-network/pm/issues/3
This is part of a broader cleanup effort to add/update all of our main repos with a CONTRIBUTING file. In the process, I

Standardize line length to 80
Add, update, and verify installation instructions for each repo

tagging @pdyraga for review

It looks like this repo is for tBTCv1 which we are sunsetting. I don't think we need to update anything here now.

works for me!


IME - Cursor position ScrollViewer issues

Describe the bug
ITextInputMethodClient for TextBox isn't reporting the cursor position relative to the current scroll offset
To Reproduce
Start IME input and scroll. The IME position isn't changing.
Expected behavior
IME completion window should move to scrolled offset.

https://github.com/AvaloniaUI/AvaloniaEdit/issues/352  Is this the same bug?


support dates (INT64 only) in plain encoding

Fixes
Issue #81
Description
This supports dates in plain encoding, IN64 resolution only (up to seconds)
Todos

[x] unit/integration tests
[x] documentation

I need this built, so confirming myself, feel free to reopen if needed )


Add DNS for artifacts

Once we have #243 completed, we want to have a DNS setup for our dependencies so we developers don't have to use Jitpack's  github url

https://jitpack.io/docs/#custom-domain-name

com.walletconnect.v2:sign:release_version

we'll probably want to move away from Jitpack as we add more modules. Jitpack is not very good for providing customization of dependencies (having multiple artifacts in a single repo, fully custom DNS names). Here's a guide on how to use MavenCentral https://getstream.io/blog/publishing-libraries-to-mavencentral-2021/


Temporal Convolution Network (TCN)

I checked your examples but I still cannot figure out how to build TCN using Pytorch C++.
Can you give me guidance?
or It would be great if the TCN example is added.
Thank you

TCN is not implemented in this repository.
I plan to add it in the future, and I don't know when it will be.
Thank you!

Thank you for your quick reply and your consideration.
It would be a great help to see the example soon.
Thank you,


new package: emacs-nox 26.3

Description of package
Checklist

[x] Latest upstream version.
[x] I agree to maintain this package (required).
[x] I agree to notify KISS if I can no longer maintain this package (required).

decided against the mods i had, going basically vanilla for both Emacses

good to go here


Good Code

[Enter feedback here]

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

ID: 93e96a57-d317-52f8-eec6-1f6d4648c0bb
Version Independent ID: 93e96a57-d317-52f8-eec6-1f6d4648c0bb
Content: About the list of tools - Microsoft Edge Development
Content Source: microsoft-edge/devtools-guide-chromium/about-tools.md
Product: microsoft-edge
Technology: devtools
GitHub Login: @MSEdgeTeam
Microsoft Alias: msedgedevrel

Closing as incomplete.


Reverting a change is not reflected on non-digest version

We've been having a problem with an app where sometimes it would seem like assets weren't recompiling when they should. Almost like sprockets didn't 'see' that they had changed. The weird part was that the digest version of the asset would have the change, but the non-digest version would not.
I think I've finally tracked this down to being a bug with non-stupid-digest-assets. Here's a series of steps that I just used to verify the behavior in a new rails app.
$ rails new asset-test
$ cd asset-test
$ git init .
$ git add .
$ git commit -m "new rails"
$ echo 'gem "non-stupid-digest-assets"' >> Gemfile
$ bundle install
$ git add .
$ git commit -m "add non-stupid-digest-assets"
$ rake assets:precompile
I, [2016-11-14T11:37:06.085170 #44963]  INFO -- : Writing /Users/jgreen/projects/clickfunnels/asset-precompile-test/public/assets/application-5c8884cc03b1598ffec78532914008dbc272b7bea6b8fc8d1e5301c82b001d0c.js
I, [2016-11-14T11:37:06.091784 #44963]  INFO -- : Writing /Users/jgreen/projects/clickfunnels/asset-precompile-test/public/assets/application-5c8884cc03b1598ffec78532914008dbc272b7bea6b8fc8d1e5301c82b001d0c.js.gz
I, [2016-11-14T11:37:06.096176 #44963]  INFO -- : Writing /Users/jgreen/projects/clickfunnels/asset-precompile-test/public/assets/application-e80e8f2318043e8af94dddc2adad5a4f09739a8ebb323b3ab31cd71d45fd9113.css
I, [2016-11-14T11:37:06.096365 #44963]  INFO -- : Writing /Users/jgreen/projects/clickfunnels/asset-precompile-test/public/assets/application-e80e8f2318043e8af94dddc2adad5a4f09739a8ebb323b3ab31cd71d45fd9113.css.gz
$ ls public/assets/
application-5c8884cc03b1598ffec78532914008dbc272b7bea6b8fc8d1e5301c82b001d0c.js
application-5c8884cc03b1598ffec78532914008dbc272b7bea6b8fc8d1e5301c82b001d0c.js.gz
application-e80e8f2318043e8af94dddc2adad5a4f09739a8ebb323b3ab31cd71d45fd9113.css
application-e80e8f2318043e8af94dddc2adad5a4f09739a8ebb323b3ab31cd71d45fd9113.css.gz
application.css
application.css.gz
application.js
application.js.gz
$ echo 'alert("hello");' >> app/assets/javascripts/application.js
$ rake assets:precompile
I, [2016-11-14T11:40:10.828850 #45388]  INFO -- : Writing /Users/jgreen/projects/clickfunnels/asset-precompile-test/public/assets/application-1581d4bcb794e954f986c800d8fe207217efbdd704e70f9cedb8adfc2cbe0e1b.js
I, [2016-11-14T11:40:10.835785 #45388]  INFO -- : Writing /Users/jgreen/projects/clickfunnels/asset-precompile-test/public/assets/application-1581d4bcb794e954f986c800d8fe207217efbdd704e70f9cedb8adfc2cbe0e1b.js.gz
$ ls public/assets/
application-1581d4bcb794e954f986c800d8fe207217efbdd704e70f9cedb8adfc2cbe0e1b.js
application-1581d4bcb794e954f986c800d8fe207217efbdd704e70f9cedb8adfc2cbe0e1b.js.gz
application-5c8884cc03b1598ffec78532914008dbc272b7bea6b8fc8d1e5301c82b001d0c.js
application-5c8884cc03b1598ffec78532914008dbc272b7bea6b8fc8d1e5301c82b001d0c.js.gz
application-e80e8f2318043e8af94dddc2adad5a4f09739a8ebb323b3ab31cd71d45fd9113.css
application-e80e8f2318043e8af94dddc2adad5a4f09739a8ebb323b3ab31cd71d45fd9113.css.gz
application.css
application.css.gz
application.js
application.js.gz
$ grep hello public/assets/application.js
alert("hello");
$ git checkout app/assets/javascripts/application.js
$ rake assets:precompile
$ grep hello public/assets/application.js
alert("hello");
$ grep hello app/assets/javascripts/application.js
<no output, indicating no match>
$ grep hello public/assets/application-5c8884cc03b1598ffec78532914008dbc272b7bea6b8fc8d1e5301c82b001d0c.js
<no output, indicating no match>
$ grep hello public/assets/application-1581d4bcb794e954f986c800d8fe207217efbdd704e70f9cedb8adfc2cbe0e1b.js
alert("hello");

I think what's happening is this:

On the first pass (before adding alert('hello');) sprockets is generating the file with a digest of 5c8884.... And then non-stupid-digest-assets is copying that fingerprinted file to the non-digest location.

Then on the second compilation (with the alert added), sprockets is generating a digest of e80e8f..., and non-stupid-digest-assets is then copying that file to the non-digest location.

On the 3rd run (after removing the previously added line), sprockets is again generating a digest of 5c8884..., which is the same as from the first run. Using this digest it looks in public/assets and sees that a file with that fingerprint already exists, so it does not recompile the file, and does not generate any output. I'm guessing that the lack of output or action from sprockets then causes non-stupid-digest-assets to think that nothing has happened, so then non-stupid-digest-assets doesn't take any action, which then leaves the non-digest version of application.js in an inconsistent state. At the end of this application.js reflects the state of the previous compilation, instead of the most recent one.

Hmm that theory makes sense to me - I'll have a look and see if there's a way to work around that


fixing install switches

The passthrough args to nbextension install were completely broken. This adds some more knowledge about the passthrough kwargs based on the current --help.
Definitely needs to go into the tests!

The build fail is the missing osx and windows build members.


Date delimeters/input mask when typing manually

Current solution:
I have datepicker with possibility to manually type date in the input. To handle this and prevent errors, currently I'm getting input value, onChange handler is validating if format is MM/DD/YYYY or YYYY/MM/DD, and only then setting new date value.
Problem
I'm validating ONLY for slash sign / as a delimeter between day, month and year, user has to type /  it manually.
Would like to have:
To ensure better UX and prevent from errors I'd like to have kind of "placeholder" or autocomplete for delimiter like - or /. So when typing eg. 2021 input would automatically concatenate delimeter sign  2021- and I can type further.
Is somehow possible to do such trick with current options/props or new feature is required? Thanks in advance for any tips!

I found solution myself, maybe will help someone. Keyword to search for is input mask. There're libraries for that, for my case I added react-maskedinput and DatePicker got customInput prop for personalization.
import MaskedInput from 'react-maskedinput'; <DatePicker     customInput={<MaskedInput mask="11/11/1111" placeholder="MM/DD/YYYY" />} //otherProps

My variant:

import InputMask from 'react-input-mask';

    <DatePicker
      ...
      placeholderText="MM/DD/YY"
      dateFormat="MM/dd/yy"
      customInputRef="inputRef"
      customInput={<InputMask mask="99/99/99" />}
    />

HI i need masked input only for time in 12 hour format with AM and PM.
i tried mask with [' ',
/\d/,
/\d/,
${timeDelim},
/\d/,
/\d/,
' ',
/[AaPp]/i,
/[Mm]/i,]
but i could get the proper mask for time.
any suggestion please .

Using PatternFormat from react-number-format package
import {useState} from 'react';
import DatePicker from "react-datepicker";
import "react-datepicker/dist/react-datepicker.css";
import {registerLocale} from "react-datepicker";
import {ru} from 'date-fns/locale/ru';
import {PatternFormat} from 'react-number-format';

registerLocale('ru', ru);

const Example = () => {
  const [startDate, setStartDate] = useState(new Date());
  const [endDate, setEndDate] = useState(new Date());

  return (
    <>
      <DatePicker
        locale="ru"
        dateFormat="dd.MM.yyyy"
        customInputRef="getInputRef"
        customInput={<PatternFormat
          format="##.##.####"
          allowEmptyFormatting={false}
          mask="_"
        />}
        placeholderText='ДД.ММ.ГГГГ'
        selected={startDate}
        selectsStart
        startDate={startDate}
        endDate={endDate}
        onChange={(date) => setStartDate(date)}
      />
      <DatePicker
        locale="ru"
        dateFormat="dd.MM.yyyy"
        customInputRef="getInputRef"
        customInput={<PatternFormat
          format="##.##.####"
          allowEmptyFormatting={false}
          mask="_"
        />}
        placeholderText='ДД.ММ.ГГГГ'
        selected={endDate}
        selectsEnd
        startDate={startDate}
        endDate={endDate}
        onChange={(date) => setEndDate(date)}
      />
    </>
  );
};

package.json
{
  "name": "test-react-datepicker",
  "private": true,
  "version": "0.0.0",
  "type": "module",
  "scripts": {
    "dev": "vite",
    "build": "vite build",
    "lint": "eslint .",
    "preview": "vite preview"
  },
  "dependencies": {
    "react": "^18.3.1",
    "react-datepicker": "^7.5.0",
    "react-dom": "^18.3.1",
    "react-number-format": "^5.4.2"
  },
  "devDependencies": {
    "@types/react": "^18.3.3",
    "@types/react-dom": "^18.3.0",
    "@vitejs/plugin-react": "^4.3.1",
    "eslint": "^8.57.0",
    "eslint-plugin-react": "^7.34.3",
    "eslint-plugin-react-hooks": "^4.6.2",
    "eslint-plugin-react-refresh": "^0.4.7",
    "vite": "^5.3.4"
  }
}


Add a test for Thread::Pool

This is a test for #34
This test pass with v0.1.7. and does not pass with v0.1.6.

Thanks, but closing this, I already wrote specs that didn't pass on 0.1.6, and much more.

Thanks for your review.
Sorry, I missed your specs.
Can you tell me which specs are for this?

@nownabe just pushed them, the #done? and #idle? specs would fail on 0.1.6.

@meh I got it! Thanks!


Log file location incorrect

When the installer is used:

On a Mac the log file is in the app, called output.log (next to the executable jar)
On Windows there seems to be no sign of it.

This was fixed by an update to the Install4j installer (pushed) which was redirecting the log.
Also updated the main class to be MultiBitInExecutableJar (which performs the redirection).
To retest either rebuild an installer or wait until I build some and copy them to multibit.org releases dir.
Awaiting review and closing

After using the latest installers here are the results (after a smooth installation experience):

OS X - no sign of the log (expected to find it in Application Support/MultiBit) or in a wallet directory but nothing
Windows - all good (located in Roaming/MultiBit/log as expected)

Bouncing back so you can you provide the OS X location.

for OSX it is stored as Application Support/MultiBit/log/multibit.log
Please check if you have a log directory that contains it.
This is the same as previous versions

OK. False alarm. Turned out that I need to exercise the application more than I was doing to trigger the log write. All good now.
Closing.


Trusted Checkpoint failing to Initialize.

Describe the bug
Trusted Checkpoint Failing to Initialize.

Expected behavior
Within the Trusted Checkpoint input, I'm supposed to input the Block Root hash and its supposed to produce the below image...

Instead it shows:

Steps to Reproduce
According to the video: https://www.youtube.com/watch?v=3O7nNHhdqlc&t=68s

Open https://beaconcha.in/
Click on Epoc
Click on Slot
Copy Block Root hash
Open https://light-client-demo.lodestar.casa/
Paste Block Root hash into Trusted Checkpoint [input]
Click Initialize From Trusted Checkpoint
Error: Error initializing from trusted checkpoint ---hashnumber--- : Epoch is not a number: NaN

Desktop:

OS: Windows 10
Version: Online Issue, (Version not Applicable)
Branch: Online Issue, (Version not Applicable)
Commit hash: Online Issue, (Version not Applicable)

@jeluard This seems to be a valid issue we need to address.  https://github.com/ethereum/consensus-specs/blob/dev/specs/phase0/beacon-chain.md#checkpoint


Geographic coordinate - latitude and longitude

Description of the issue
The definition of the entity "geographic coordinate" says: "A geographic coordinate is an information content entity explicitly stating the geographic position of a zero-dimensional spatial region on Earth, by using a set of numbers with respect to a geographic coordinate system." But how can a "set of numbers" be added to the entity "geographic coordinate", where the numbers express the latitude and longitude?
Ideas of solution
My suggestion is to add the entities "latitude" and "longitude" for example, which can be assigned a value via the data property "has number". Furthermore, it has to be thought about how the new entities "latitude" and "longitude" can be related to the entity "geographic coordinate".
Workflow checklist

[ ] I discussed the issue with someone else than me before working on a solution
[ ] I already read the latest version of the workflow for this repository
[ ] The goal of this ontology is clear to me

I am aware that

[ ] every entry in the ontology should have a definition
[ ] classes should arise from concepts rather than from words

I think, we should add:

cartesian spatial coordinate datum (IAO_0000400): A cartesian spatial coordinate datum is a representation of a point in a spatial region, in which equal changes in the magnitude of a coordinate value denote length qualities with the same magnitude

one dimensional cartesian spatial coordinate datum (IAO_0000401) A cartesion spatial coordinate datum that uses one value to specify a position along a one dimensional spatial region
two dimensional cartesian spatial coordinate datum (IAO_0000402) A cartesion spatial coordinate datum that uses two values to specify a position within a two dimensional spatial region
three dimensional cartesian spatial coordinate datum (IAO_0000403) A cartesion spatial coordinate datum that uses three values to specify a position within a three dimensional spatial region

And then redefine geographic coordinate to: A geographic coordinate is ~an information content entity~ a two dimensional cartesian spatial coordinate datum explicitly stating the geographic position of a zero-dimensional spatial region on Earth, by using a set of numbers with respect to a geographic coordinate system.
Proposals for latitude and longitude:

latitude: A latitude is a quantity value that is part of a geographic coordinate and quantifies the angle from a zero-dimentional spatial region on the Earth's surface to the equatorial plane.
longitude: A latitude is a quantity value that is part of a geographic coordinate and quantifies the angle east or west of a from a zero-dimentional spatial region on the Earth's surface to the reference meridian between the two geographical poles.

Inspired by the definitions of http://semanticscience.org/resource/SIO_000318 and https://ontobee.org/ontology/SIO?iri=http://semanticscience.org/resource/SIO_000319

Adding coordinate data properties

Data property “has measurement value” (http://purl.obolibrary.org/obo/IAO_0000004)
Data property “has x coordinate value” (http://purl.obolibrary.org/obo/IAO_0000404)
Data property “has y coordinate value” (http://purl.obolibrary.org/obo/IAO_0000406)
Data property “has z coordinate value” (http://purl.obolibrary.org/obo/IAO_0000405)

I think, we should import:

cartesian spatial coordinate datum (IAO_0000400): A cartesian spatial coordinate datum is a representation of a point in a spatial region, in which equal changes in the magnitude of a coordinate value denote length qualities with the same magnitude

Since apparently a lot of Ontologies are using that system, it seems to work. I think the definition is extremely convoluted though. Also, two minor complaints:

We don't have the object property denotes which is part of the definition and axioms of the IAO class. So maybe we should import that as well.

http://purl.obolibrary.org/obo/IAO_0000219
Definition: A primitive, instance-level, relation obtaining between an information content entity and some portion of reality. Denotation is what happens when someone creates an information content entity E in order to specifically refer to something. The only relation between E and the thing is that E can be used to 'pick out' the thing. This relation connects those two together. sense 3: To signify directly; refer to specifically

I think in which equal changes in the magnitude of a coordinate value denote length qualities with the same magnitude is a horrible way to describe it. I am still not sure if I understood correctly. If it is a representation of a point in a spatial region, how can changes to the coordinate value denote length qualities? A point has no length. They are probably referring to the lengths of the axis, but those are not even mentioned, which is weird considering that is the most important part of the system, right?

I think a definition in this direction would be better (I know this is for the system and not the datum, we would then add the corresponding information content entity as well, which would also solve my next concern):
A Cartesian coordinate system is a coordinate system that specifies each point uniquely in a plane by a pair of numerical coordinates, which are the signed distances from the point to two fixed perpendicular directed lines, measured in the same unit of length. http://purl.obolibrary.org/obo/STATO_0000009

And then redefine geographic coordinate to: A geographic coordinate is ~an information content entity~ a two dimensional cartesian spatial coordinate datum explicitly stating the geographic position of a zero-dimensional spatial region on Earth, by using a set of numbers with respect to a geographic coordinate system.

I am fine with that proposal, but maybe we should define geographic coordinate system as well, since it is the basis of the definition. That definition could maybe be something like A geographic coordinate system is a cartesian coordinate system that represents the Earth.

Proposals for latitude and longitude:

latitude: A latitude is a quantity value that is part of a geographic coordinate and quantifies the angle from a zero-dimentional spatial region on the Earth's surface to the equatorial plane.
longitude: A latitude is a quantity value that is part of a geographic coordinate and quantifies the angle east or west of a from a zero-dimentional spatial region on the Earth's surface to the reference meridian between the two geographical poles.

I am fine with these proposals.

2. I think "in which equal changes in the magnitude of a coordinate value denote length qualities with the same magnitude" is a horrible way to describe it. I am still not sure if I understood correctly. If it is a representation of a point in a spatial region, how can changes to the coordinate value denote length qualities? A point has no length. They are probably referring to the lengths of the axis, but those are not even mentioned, which is weird considering that is the most important part of the system, right?

I understand the definition in way that the _"in which equal changes ...-" part refers to the spatial region and not to the point. It is a characteristic of such a spatial region "equal changes in the magnitude of a coordinate value denote length qualities with the same magnitude" -- and not a characteristic of the point!

I understand the definition in way that the "in which equal changes ..." part refers to the spatial region and not to the point. It is a characteristic of such a spatial region that "equal changes in the magnitude of a coordinate value denote length qualities with the same magnitude" -- and not a characteristic of the point!

You are probably right, I still think that the definition is just not that well articulated. I am okay either way, but I would prefer we add cartesian coordinate system as well as cartesian coordinate system datum.

And then redefine geographic coordinate to: A geographic coordinate is ~an information content entity~ a two dimensional cartesian spatial coordinate datum explicitly stating the geographic position of a zero-dimensional spatial region on Earth, by using a set of numbers with respect to a geographic coordinate system.

I am fine with that proposal, but maybe we should define geographic coordinate system as well, since it is the basis of the definition. That definition could maybe be something like "A geographic coordinate system is a cartesian coordinate system that represents the surface of the Earth".

Cartesian coordinate systems are rectangular coordinate systems. Geographic coordinate systems are usually spherical coordinate systems. (Sorry for referencing wikipedia)
Importing the cartesian stuff from IAO is nice, but does't serve for this use case...
Therefore, I propose for a quick solution:
Leave geographic coordinate as is and implement latitude and longitude as proposed, until we are done with the quantity values.

Cartesian coordinate systems are rectangular coordinate systems. Geographic coordinate systems are usually spherical coordinate systems. (Sorry for referencing wikipedia)

Geographic coordinates on a sphere are rectangular coordinate systems, too. That is why I thought that matches.
But thinking again, the in which equal changes in the magnitude of a coordinate value denote length qualities with the same magnitude part of the definition does not hold on a sphere as the distance between two longitudes depends on the latitude.
I just had a look into GEO, there is the following structure:

data item (IAO_0000027), already in the OEO

measurement datum (IAO_0000109): An information content entity that is intended to be a truthful statement about something (modulo, e.g., measurement precision or other systematic errors) and is constructed/acquired by a method which reliably tends to produce (approximately) truthful statements.

geodetic coordinate measurement datum (GEO_000000787): A measurement of location on the Earth that uses a single number to specify a coordinate in a coordinate system known as a geodetic datum

latitude coordinate measurement datum (OBI_0001620): A measurement datum that is the measure of the latitude coordinate of a site. Alternative term: latitude
longitude coordinate measurement datum (OBI_0001621): A measurement datum that is the measure of the longitude coordinate of a site. Alternative term: longitude

The GEO structure looks suitable for our purposes.

So we need to import the following three classes: geodetic coordinate measurement datum (GEO_000000787), latitude coordinate measurement datum (OBI_0001620) and longitude coordinate measurement datum (OBI_0001621). Who can do that?

From OEO dev meeting 76:
This issue was not yet implemented as it needs to be rediscussed after the restructuring of the quantity values.

I can do the import.
@nelekoehler any idea of how to relate to quantity values?


KIP-50: Velodrome/Uniswap Liquidity Mining Test #2

When opening a pull request to submit a new KIP, please use the suggested template: https://github.com/Kwenta/KIPs/blob/master/kip-X.md
We have a GitHub bot that automatically merges some PRs. It will merge yours immediately if certain criteria are met:

The PR edits only existing Draft PRs.
The build passes.
Your Github username or email address is listed in the 'author' header of all affected PRs, inside .
If matching on email address, the email address is the one publicly listed on your GitHub profile.

There is an error in my data in the report. I need to recalculate it with different data.  Don't merge yet please.


Compatibility with newer hhvm versions

This PR makes this library compatible with newer runtime versions.
It allows modern versions of the autoloader to be installed without conflict.
It updates broken require_once statements.
It also lints with new linters.

Thnks a lot!


[BugFix] Fix CTAS with TopN + cast bug

What type of PR is this：

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

Which issues of this PR fixes ：

Fixes #9104
normal SQL execute order:
select -> order by -> limit
CTAS SQL execute order:
select -> order by -> limit -> cast to target schame type(projection)
so...TopN can't reslove projection column
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

run starrocks_admit_test


WIP: changes to source code dataset, issues and diffs, stack exchange

Changes made:

added Ed's code for downloading and formatting stack exchange
added issues and diffs dataset; it is filtered at the repo level for repos that are in the source code dataset.
source code data saving scheme is different; makes tokenization slightly easier
Switched the tokenizer to tiktoken. This has a different vocabulary than the neox tokenizer, but is good enough for ballpark estimates.

Changes that need to be made

Julia, R, and jupyter-notebook subsets of the source code dataset have been manually inspected quite comprehensively. Need to do this with the rest of the subsets.
Need to manually inspect Github issues and diffs dataset.
Python tokenization hangs at the same particular iteration each time I try it. Figure out what's going on. Temporary solution is a short circuit that counts all python files as containing 0 tokens.

Merging this code so it doesn't diverge from main and converting TODOs to issues.


Integration with device_preview to correctly update MediaQueryData

Issue Description
When using flutter_screenutil with device_preview, the MediaQueryData may not be updated correctly, causing widgets to use the physical device's dimensions instead of the preview dimensions. This can be resolved by configuring ScreenUtil correctly.
Solution
Ensure ScreenUtil.configure is used to update the MediaQueryData within the ScreenUtilInit builder. Here's an example:
void main() async {
  WidgetsFlutterBinding.ensureInitialized();
  FlutterNativeSplash.preserve(widgetsBinding: WidgetsFlutterBinding.ensureInitialized());

  await App.onInit();
  await App.session.close();
  App.session.fetch().then((value) => FlutterNativeSplash.remove());

  runApp(
    DevicePreview(
      enabled: App.device.isWeb || App.device.isIPad,
      builder: (context) {
        return ScreenUtilInit(
          designSize: const Size(360, 690),
          minTextAdapt: true,
          splitScreenMode: true,
          useInheritedMediaQuery: true,
          builder: (context, child) {
            ScreenUtil.configure(
              data: MediaQuery.of(context),
            );
            return const EzStore();
          },
        );
      },
    ),
  );
}

class EzStore extends StatelessWidget {
  const EzStore({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp.router(
      builder: (context, child) {
        return DevicePreview.appBuilder(context, child!);
      },
      title: 'EZ Store',
      routerConfig: appRouter,
      debugShowCheckedModeBanner: false,
      theme: AppMaterialTheme(createTextTheme(context, "Montserrat", "Montserrat Alternates")).light(),
      darkTheme: AppMaterialTheme(createTextTheme(context, "Montserrat", "Montserrat Alternates")).dark(),
    );
  }
}

Additional Information
This ensures the MediaQueryData from device_preview is used correctly by ScreenUtil.
References

Issue #333
Pull Request #491

flutter doctor

[✓] Flutter (Channel stable, 3.22.2, on macOS 14.5 23F79 darwin-arm64, locale pt-BR)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2023.2)
[✓] IntelliJ IDEA Ultimate Edition (version 2024.1)
[✓] VS Code (version 1.91.0)
[✓] Connected device (5 available)
[✓] Network resources

flutter_screenutil: ^5.9.3
device_preview: ^1.2.0

I hope it helps

Thanks @paulocagol, although your code snippet helped me, I still have to do a hot-restart so the changes update. But it's still better than before.

what for the font size (sp) its not working with device preview ?


Can't run ABSA solution with docker

Hi @danielkorat ,
When I clone and try to run the merged solution I get an error shown in previous open bug (#178).
I have successfully built a docker but I can't get the to localhost:5006 and open the ABSA solution UI.
Here is the Dockerfile you can run and reproduce:
FROM ubuntu:18.04
LABEL maintainer="Intel AI Lab NLP"

RUN apt update && \
    apt install -y bash build-essential ca-certificates python3 python3-pip git vim && \
    rm -rf /var/lib/apt/lists

RUN python3 -m pip install --no-cache-dir --upgrade pip setuptools

WORKDIR /workspace
RUN git clone https://github.com/NervanaSystems/nlp-architect.git
RUN cd nlp-architect/ && \
    git checkout 66863f850822b4686c0df1dd88a3bf614f234818
RUN cp -R nlp-architect/solutions nlp-architect/nlp_architect/
RUN python3 -m pip install -e nlp-architect && \
    pip install -r nlp-architect/solutions/absa_solution/requirements.txt
RUN python3 -m spacy download en

WORKDIR /workspace/nlp-architect
CMD ["/bin/bash"]

EXPOSE 8080
EXPOSE 8888

This one was solved with reinstalling Bokeh to be 1.2.0.


Bump Thor to 0.20.0

To pull in this fix: https://github.com/erikhuda/thor/pull/575

cc @UserNotFound - this fixes the issue you ran into here: https://aptible.slack.com/archives/C0GN45N59/p1508457537000162
cc @fancyremarker

In the way of verifying this change, I am:

Running the integration tests that invoke the CLI
Going through the Thor changelog.
And... testing out a few things manually

Codecov Report

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

@@           Coverage Diff           @@
##           master     #239   +/-   ##
=======================================
  Coverage   95.67%   95.67%
=======================================
  Files          58       58
  Lines        2772     2772
=======================================
  Hits         2652     2652
  Misses        120      120

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 bf681a8...b6569cb. Read the comment docs.


Any sample for shared element between two activities

I cant seem to figure out how to use this lib for transiting from one to another with a shared element. Any sample?

Hello. Unfortunately, this library cant be used for transitions between Activities or Fragments. Its only animates view changes in layout


Update the version in the badge

I think the badge should show the number of commits from the latest release.
cf. https://shields.io/badges/git-hub-commits-since-latest-release-branch

Thanks!


HADOOP-18949. upgrade maven dependency plugin due to security issue.

Description of PR
https://issues.apache.org/jira/browse/HADOOP-18949
How was this patch tested?
For code changes:

[x] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
16m 52s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+0 :ok:
xmllint
0m  0s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
44m  5s

trunk passed

+1 :green_heart:
compile
0m 28s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
compile
0m 30s

trunk passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
mvnsite
0m 35s

trunk passed

+1 :green_heart:
javadoc
0m 34s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javadoc
0m 31s

trunk passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
shadedclient
80m 10s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

-1 :x:
mvninstall
0m 56s
/patch-mvninstall-hadoop-maven-plugins.txt
hadoop-maven-plugins in the patch failed.

+1 :green_heart:
compile
0m 20s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javac
0m 20s

the patch passed

+1 :green_heart:
compile
0m 19s

the patch passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
javac
0m 19s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
0m 21s

the patch passed

+1 :green_heart:
javadoc
0m 20s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javadoc
0m 19s

the patch passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

-1 :x:
shadedclient
3m  8s

patch has errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
0m 20s

hadoop-maven-plugins in the patch passed.

+1 :green_heart:
asflicense
0m 33s

The patch does not generate ASF License warnings.

107m 28s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6219/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/6219

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient codespell detsecrets xmllint

uname
Linux 02f3285db716 4.15.0-213-generic #224-Ubuntu SMP Mon Jun 19 13:30:12 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 1876ce132340ba8f0c4535c3439763afb8be8320

Default Java
Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6219/1/testReport/

Max. process+thread count
727 (vs. ulimit of 5500)

modules
C: hadoop-maven-plugins U: hadoop-maven-plugins

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6219/1/console

versions
git=2.25.1 maven=3.6.3

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 55s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+0 :ok:
detsecrets
0m  1s

detect-secrets was not available.

+0 :ok:
xmllint
0m  1s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
44m 28s

trunk passed

+1 :green_heart:
compile
0m 30s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
compile
0m 29s

trunk passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
mvnsite
0m 35s

trunk passed

+1 :green_heart:
javadoc
0m 35s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javadoc
0m 31s

trunk passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
shadedclient
80m  3s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m  0s

the patch passed

+1 :green_heart:
compile
0m 20s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javac
0m 20s

the patch passed

+1 :green_heart:
compile
0m 20s

the patch passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
javac
0m 20s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
0m 22s

the patch passed

+1 :green_heart:
javadoc
0m 20s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javadoc
0m 19s

the patch passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
shadedclient
34m 58s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
0m 26s

hadoop-maven-plugins in the patch passed.

+1 :green_heart:
asflicense
0m 40s

The patch does not generate ASF License warnings.

123m 19s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6219/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/6219

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient codespell detsecrets xmllint

uname
Linux 829b02e7e960 4.15.0-213-generic #224-Ubuntu SMP Mon Jun 19 13:30:12 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / e5f31c9c8c74b47f776cef18819b64f7f7dcd7ca

Default Java
Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6219/2/testReport/

Max. process+thread count
555 (vs. ulimit of 5500)

modules
C: hadoop-maven-plugins U: hadoop-maven-plugins

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6219/2/console

versions
git=2.25.1 maven=3.6.3

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

merged. In need of backporting, presumably?

@steveloughran I'm neutral about backporting this. It doesn't seem like the biggest issue and there will eventually be a hadoop 3.4.0 release.


⚠ DesiTorrents has degraded performance

In 964d562, DesiTorrents ($DST) experienced degraded performance:

HTTP code: 200
Response time: 3817 ms

Resolved: DesiTorrents performance has improved in 2b5af15.


DR:S Trinity Avowed Sword and Brand Raidboss calls not working.

Description
7/23/21
During the Trinity Avowed boss in Delubrum Reginae (Savage) the Raid Boss plugin has stopped accounting for sword cleaves AND brands. So it will call the sword safe spot, but not where to stand cardinal from the safe spot. And during the second half of the fight, the callouts don't account for brand debuff when telling you where to go.
Confirmed running latest version.

Were there any errors in the overlayplugin log when this happened?

Does it automatically save the log anywhere? I can post it if needed. If not, I'm going on another run tonight. If there's an option to turn on for logging, I can, then report it when I'm done.

No, it does not.  While ACT is open, you can see the log in Plugins ->OverlayPlugin.dll -> the log window at the bottom.
If this is still having issues, please zip up your Network log file and upload it somewhere that I can take a look at.  I gave this a run through a previous log I had, and things seemed ok.
Sorry for the hassle!!

Here is the network log for yesterday during the run where the calls didn't work properly. I asked another person using Cactbot on the same run, they also were having issues on Trinity Avowed with the same calls.
Network_22006_20210723.zip

Here is the network log from my friend who was on the same run and having the same issue, if it'll help.
DRS 7-23 run.zip

Ok, I think I found the issue.  You can change your overlay's url to https://quisquous.github.io/cactbot/ui/raidboss/raidboss.html and that will use the version that includes that change.  Let me know if that fixes things!

Will do! Run is in an hour. Will report back afterward. Is the proper place for this Raidboss > General > URL?

Yeah, that's it exactly.  You can verify it's working if you go to Summerford Farms and do a /countdown 5.

Completed the run. It worked perfectly, thank you for the quick fix!

There's another bug on Avowed now, I think. It's not calling out the first set of meteors in Savage anymore. The ones WITHOUT the brand. Hasn't for the last few runs I've been on. Overlay isn't reporting any errors in text.


App should prompt for a file/folder location when generating a playground

When clicking "Create Playground" the app should prompt for a file/folder name to create rather than assuming one.
The default could be a new folder with the same name as the package, in the last directory that the user selected (default to "Desktop" on first run?) but the final destination/name should be user defined.

Fixed in https://github.com/SwiftPackageIndex/SPI-Playgrounds/pull/3


[FEATURE REQUEST] - Put Trades on Hold

Is your feature request related to a problem? Please describe.
When mapping or fighting a boss i like to put my trades on hold with a message like "Currently in a map. Do you want me to whisper you back when i am finished?". At this point i would like to have the option to turn down the opacity of the notification panel so it doesn't block any view on the screen.
Describe the solution you'd like
A solution i'd like is having the option to hide the window when using response buttons. And only show window again when hovering over the invisible window or when another trade whisper comes in.
Describe alternatives you've considered
Instead of only hiding the window to have the option to change between opacity while on 'hold' mode. Or to minimize it into a smaller window.
Additional context

since 1.3.9 there was added a buton to task bar to hide messages, it will stay hidden until new message will come or you will alt tab to other program and alt tab to poe


Potential issue with ways and relations processing in new OSM File Loader (Load only) tool

Hi @ThomasEmge ,
I hope I am not bothering you to much with yet another potential issue, but I think there may be an important issue with the ways and relations processing in the new OSM File Loader (Load only) tool.
As I am currently unable to use my ArcGIS Renderer for OpenStreetMap with the new OSM File Loader (Load only) output due to a field naming conflict between the old Load OSM File tool and the new tool, I haven't yet properly evaluated the output of the new tool.
However, as a first test, I decided to download a section of Berlin with the Bode Museum (http://www.openstreetmap.org/relation/4211594), that we have been using to tackle the OpenStreetMap Simple 3D building relations in the solved issue #72.
Once I loaded the data with the new OSM File Loader, and selected buildings only using:
 osm_building IS NOT NULL AND osm_building NOT IN ('no','')
I noticed an issue though, there seemed to be an overlapping polygon feature at the same location as the Bode Museum's multipolygon relation. Subsequently, I discovered that this feature, which uses the Bode Museum's way outline, falsely carries the attributes of another way object. It shows up as a building feature with the tag:
building=commercial
However, there is no commercial building at this location, just the Bode Museum. Checking the individual ways, including the outline way of the Bode Museum on the OpenStreetMap website (http://www.openstreetmap.org/way/313659705), that does not carry any tags of its own, it became clear these tags must be coming from somewhere else, another feature. This is also confirmed by the wrong OSMID on the feature, which lists as w14582095. Checking this feature out on the OpenStreetMap website, it turned out to be another building quite some distance away from the Bode Museum. See this image, the lower left downward pointing arrow denoting the true  w14582095 commercial building, the upper right one pointing to the Bode Museum:

It appears there may be some issues with the indexes or dictionaries keeping track of the objects, maybe a similar issue as the one in #117?
Anyway, the following images show more detail and explain the issue in more detail. I have also included a ZIP from the Berlin OSM file I used, so you can test and see if you come up the same result.
As a start, the following image shows the Bode Museum multipolygon relation as it is visible on the OpenStreetMap website. Notice the relation ID of 4211594:

The second image shows the multipolygon relation as loaded by the "old" Load OSM File tool. Notice in the identify window that it has the proper OSMID as on the OpenStreetMap website. This looks good:

Next up is the same multipolygon relation as loaded by the "new" OSM File Loader (Load only) tool. Notice this is also OK, and displaying the proper OSMID and properties / tags:

Now the issue though: by using the following selection statement:
osm_building IS NOT NULL AND osm_building NOT IN ('no','') AND OSMID = 'w14582095'
a building polygon representing the outline of the Bode Museum is selected. This has wrong polygon / way information associated with it. Remember that I checked the way outline of the Bode Museum on the OpenStreetMap website, and it did not have this information associated with it. Notice the OSMID displayed is 14582095. By using this OSMID on the OSM website, I was able to identify the feature where these wrong tags came from, see the first image I posted of the secondary building some distance away from the Bode Museum.
You can also directly select it using this link:
http://www.openstreetmap.org/way/14582095#map=19/52.51053/13.37911

*** EDIT: *** forgot the ZIP with the osm XML file, here it is:
OSM_Berlin.zip

Very nice catch. Thank you for the excellent detective work. The fix will go in very soon.
The reason here was an 'incomplete' way (id: 14582095) meaning that not all referenced nodes by the way where included in the osm file itself. The feature should have been ignored by the loading process.
Just out of curiosity how was this osm produced?

Just out of curiosity how was this osm produced?

This is output of QGIS's Download Data option for OpenStreetMap (under the Vector \ OpenStreetMap menu option), which is essentially equivalent to the Download OSM Data (XAPI) option of the ArcGIS Editor (that I still can not run due to the blocking toolvalidator issue).
However, I wouldn't be surprised at all if there are a couple of other tools in the OpenStreetMap software ecosystem, that produce similar "incomplete" osm XML files. E.g. many editors also have options for downloading "incomplete members".
So it is definitely good to have this fix in, as there will undoubtedly be people with similar datasets trying to import with the ArcGIS Editor's options.

@ThomasEmge ,
Was this fix supposed to be included in the last files you made available? Because I still see the false commercial building after doing a re-import of the data...
Or is the fix only going in with the final build?

This fix should be in the latest build.

I re-installed fully??
Will look at this tomorrow. Its bed time here...

Wrong button...

This fix should be in the latest build.

@ThomasEmge
I now double checked, but it is still not good.
I first uninstalled the ArcGIS Editor for OpenStreetMap. Since I noticed the uninstall failed to delete the DLLs from the 64bit install folder (C:\Program Files\ESRI\OSMEditor), I deleted the "ESRI" folder manually. I guess the uninstall failed due to the fact that, in order to test previous DLLs you made available, I manually overwrote these files.
Anyway, after this clean up, I downloaded the files again from the link you made available, and re-installed. After a full reboot, I used the OSM File Loader (Load only) tool again to import my Berlin OSM XML file.
Same result, the false commercial building is still being created.
Are you absolutely sure the fix was included in the last build, and that you made the right setups available to me via the download link?

Yes, I double-checked yesterday as well.
Looking for the office building shows that it doesn't exist.

And here is the outline of the Bode Museum

@ThomasEmge ,
I am getting this with the following query:
osm_building IS NOT NULL AND osm_building NOT IN ('no','') AND OSMID = 'w14582095'
Are you absolutely sure you supplied me with the right version of the tool??

I just updated the download location. The commit and the setup build had indeed a different date.

I just updated the download location. The commit and the setup build had indeed a different date.

Thanks, that looks much better! Just the Bode Museum multipolygon relation left as building:

And with that, I think I can close this for now. I will keep an eye on other issues though in further testing.


Re-evaluate catch-all exception handling throughout the codebase

Summary
In the same category as #9679
During the work of converting from using FindBugs to SpotBugs (#10407) I found that there was a particular class of bugs that we had strewn throughout our codebase - the REC_CATCH_EXCEPTION. The bug code, by definition states

REC: Exception is caught when Exception is not thrown (REC_CATCH_EXCEPTION)

This method uses a try-catch block that catches Exception objects, but Exception is not thrown within the try block, and RuntimeException is not explicitly caught. It is a common bug pattern to say try { ... } catch (Exception e) { something } as a shorthand for catching a number of types of exception each of whose catch blocks is identical, but this construct also accidentally catches RuntimeException as well, masking potential bugs.

A better approach is to either explicitly catch the specific exceptions that are thrown, or to explicitly catch RuntimeException exception, rethrow it, and then catch all non-Runtime Exceptions, as shown below:

try {
    ...
} catch (RuntimeException e) {
    throw e;
} catch (Exception e) {
    ... deal with all non-runtime exceptions ...
}

To summarize, essentially what I saw is that there are many places throughout our codebase where we catch(Exception e) where Exception is never thrown. It's essentially a catch-all. However, most situations we either don't want this, or might want to handle the class of RuntimeException differently.
To rememdy this we should:

Remove the REC_CATCH_EXCEPTION from the spotbugs exclusion list
For each instance, carefully evaluate how we want to handle the exceptions and either make a class-wide exception for spotbugs, OR improve the error handling to split between RuntimeException and other Exception

Urgency
I would estimate this to be middle-to-low priority.

Will close it for now since we are trying to eliminate this case in new code.


Add --dry-run option

The design doc specifies a --dry-run option, which allows simulation of a download/build without actually doing it.
https://github.com/ThePHPF/pie-design?tab=readme-ov-file#--dry-run

More info on how to approach this issue: https://github.com/php/pie/pull/118#discussion_r1868870427


Languages support

Can it be used to recognise different languages, Like i speak in one language and by using the text i get from it i translate it to other language using google translate api , basically want to build something similar to google translate app, speech to speech translation and i am using following algorithm
record sound from microphone, encode it to flac format, send it to google for recognition via speech API, when you receive the answer, translate it by google translate API to desired language, after translation, convert the translated text to speech, you could use Google speech API for that

You can start the speech recognition in the language you want. Just invoke setLocale before starting the speech recognition. Then, you can do all the rest you described in the callback where you get the text.

But set Locale has very limited number of languages in the list , like for example it do not have spanish in it.
Using your code i speak in spanish it returns me the text , can i use that text directly to translate to desired language
Please guide me man m in a real trouble

Locale has all the locales and languages supported by Android. Take a look here. After you get the text from the speech recognizer, is up to you to send it the translation service (e.g. Google Translate) and receive the response. That is out of scope regarding this library.

Library ignores setLocale for recognition.. It applies to text to speech engine only.. in code you apply it to speechRecognitonEngine too but it doesnt take effect.. What is problem ? I setLocale before invoking startListening method.. it only recognizes speech in english..

Maybe the language you are setting is not supported on the combination of android version/vendor/Google support. Library only wraps system services.

Doesnt it support Russian, Turkish, Spanish, Italian, French ? None of them works (


Add Pluralsight to the list

I left the source empty since we don't have any sort of official announcement but I work there.

Yup! I know for sure we've got at least 2 teams using F# and other teams are F#-curious haha


Confidential Transfer Support in @solana/spl-token

I have been messing with confidential transfers on the client side recently and ran into some issues with the @solana/spl-token API that I'd like to highlight here.

getMintLen() does not return the correct byte size for a mint with the confidential transfer extension.

This method seems to be the suggested way for determining how much space to allocate when creating an account with extensions from the client side. Using this function with the ExtensionType.ConfidentialTransferMint type as input returns a size of 267 bytes.
267 bytes does not seem to be the correct size for a mint with the ConfidentialTransfer extension. Attempting to initialize a mint account with 267 bytes allocated will result in an error when you eventually call the InitializeMint  instruction on the Token22 program.

An InvalidAccountData error is thrown because the program checks that the size of the given account matches exactly what is expected for the given extension.

The correct amount of bytes is 235. Allocating a buffer of exactly 235 bytes when creating the account will pass this check.

You can even test this by running the commandline example in the Confidential Transfer quick start guide and inspecting the transaction in an explorer.

spl-token --program-id TokenzQdBNbLqP5VEhdkAS6EPFLC1PHnBqCXEpPxuEb create-token --enable-confidential-transfers auto

Here is an example transaction on Devnet. You can see the account is 235 bytes long.

There is virtually no support for any of the Confidential Transfer instructions or any of the relevant types in the @solana/spl-token API. I assume this is intentional, but I have begun implementing the API following the same format as the other Token Extension methods for my own use cases. Which is a bit of a pain so far and I'm not sure if this is completely viable. I am curious why they are not supported yet and if there are plans to implement them?

It seems like the "Configure a token account for confidential transfers" commandline example does not even work in the Confidential Transfers quick start guide.. I get a "Transaction simulation failed: Attempt to load a program that does not exist" RPC error. I'm at least able to invoke the ConfidentialTransfer::ConfigureAccount instruction via typescript on devnet, so I thought this error response was odd. But that might be unrelated.

Commandline RPC Error:
Error: Client(Error { request: Some(SendTransaction), kind: RpcError(RpcResponseError { code: -32002, message: "Transaction simulation failed: Attempt to load a program that does not exist", data: SendTransactionPreflightFailure(RpcSimulateTransactionResult { err: Some(ProgramAccountNotFound), logs: Some([]), accounts: None, units_consumed: Some(0), return_data: None }) }) })

Repo with my code.

I apologize for the late response on this. The wrong size for the confidential transfer mint and accounts in token-js will be fixed once https://github.com/solana-labs/solana-program-library/pull/6487 is merged.
There has been some delay with supporting confidential transfers in token-js as we plan to compile the zk logic with wasm. We hope to have them all supported within a couple of weeks.
Also, unfortunately, it is currently not possible to run the confidential transfers outside of tests since the zk proof verification program is behind a feature-gate (https://github.com/solana-labs/solana/issues/32613). We are working hard to get the program activated, so please stay tuned.

I am building the token-2022 program from the master branch, locally and running the local validator like this:
solana-test-validator -r
--bpf-program TokenzQdBNbLqP5VEhdkAS6EPFLC1PHnBqCXEpPxuEb
/target/deploy/spl_token_2022.so
I can create the mint with the confidential transfer extension enabled and can deposit into confidential account, apply pending balance and can verify the encrypted available/pending balances on the explorer fine.
transfer and withdraw fail, however, with zkproof errors. Am I missing something in the build process or is something broken?
transfer error:
+ spl-token transfer --confidential 5WTfhiKQpEoisZmVca29tSKyPFqkDzRxuhoXujm34qQA 10 34DYPAVxjs4trEyHnbBhtKiNttTXzkyeqYusKsCLWTrY Confidential Transfer 10 tokens   Sender: H4Ngfi9uMWzdKXd3WS6TbxCGQxhvnrK3pmFo84HrcMPN   Recipient: 34DYPAVxjs4trEyHnbBhtKiNttTXzkyeqYusKsCLWTrY Error: Client(Error { request: Some(SendTransaction), kind: RpcError(RpcResponseError { code: -32002, message: "Transaction simulation failed: Error processing Instruction 0: invalid instruction data", data: SendTransactionPreflightFailure(RpcSimulateTransactionResult { err: Some(InstructionError(0, InvalidInstructionData)), logs: Some(["Program ZkTokenProof1111111111111111111111111111111 invoke [1]", "VerifyBatchedRangeProof128", "proof_verification failed: RangeProof(AlgebraicRelation)", "Program ZkTokenProof1111111111111111111111111111111 failed: invalid instruction data"]), accounts: None, units_consumed: Some(200000), return_data: None }) }) })
withdraw error:
+ spl-token withdraw-confidential-tokens 5WTfhiKQpEoisZmVca29tSKyPFqkDzRxuhoXujm34qQA 20 Withdrawing 20 confidential tokens Error: Client(Error { request: Some(SendTransaction), kind: RpcError(RpcResponseError { code: -32002, message: "Transaction simulation failed: Error processing Instruction 0: invalid instruction data", data: SendTransactionPreflightFailure(RpcSimulateTransactionResult { err: Some(InstructionError(0, InvalidInstructionData)), logs: Some(["Program ZkTokenProof1111111111111111111111111111111 invoke [1]", "VerifyWithdraw", "proof_verification failed: RangeProof(AlgebraicRelation)", "Program ZkTokenProof1111111111111111111111111111111 failed: invalid instruction data"]), accounts: None, units_consumed: Some(110000), return_data: None }) }) })

Can you verify if the same error happens with the latest spl-token-cli build, which is 3.4.0? These type of errors were reported with 3.3.0, but should be fixed with the latest cli release.

thanks for your super quick response. I am indeed running with 3.3.0.
In general, should I be using solana-labs/solana or the agave fork for the validator/cli tools etc..?

yes, that was indeed the error with 3.3.0. Just built and tried with 3.4.0 and both transfer/withdraw work fine too. Thanks for your quick pointer.
Are audit keys supported yet?

Audit keys are supported in the program, but we have not added the functionality to decrypt yet. That will be coming very soon.

spl-token-cli 3.4.1 still cannot work
I run the script here: https://github.com/solana-labs/solana-program-library/blob/master/token/cli/examples/confidential-transfer.sh
+ spl-token configure-confidential-transfer-account --address 29HvjNB2v1iHYQm3na9i3KPDpFsLcFYnhARVp71qwZJC
Error: Client(Error { request: Some(SendTransaction), kind: RpcError(RpcResponseError { code: -32002, message: "Transaction simulation failed: Attempt to load a program that does not exist", data: SendTransactionPreflightFailure(RpcSimulateTransactionResult { err: Some(ProgramAccountNotFound), logs: Some([]), accounts: None, units_consumed: Some(0), return_data: None, inner_instructions: None }) }) })

@huckym Could you take a look at it? Thanks a lot!

@doutv The error says: "Transaction simulation failed: Attempt to load a program that does not exist" -- the zk proof program has not been enabled on any network, so this error is expected. Once the zk proof program is enabled, then it'll work

@doutv The error says: "Transaction simulation failed: Attempt to load a program that does not exist" -- the zk proof program has not been enabled on any network, so this error is expected. Once the zk proof program is enabled, then it'll work

Thanks for your reply.
I thought it was enabled since Solana version is 1.18 now.
The status here should be updated: https://spl.solana.com/token-2022/status

My apologies, the info is being updated with https://github.com/solana-labs/solana-program-library/pull/7222


Can βTorrent download anonymously with a browser based VPN add-on/ extension?

Hi Diego. I've read your answer at Does WebTorrent expose my IP address via BitTorrent?

It exposes the facing IP, if you use a full VPN it shouldn't expose your real IP.

and others' comments across the boards. In that context, let me ask two questions:

Do you know if currently there is a specific browser-based VPN add-on/ extension for either Waterfox, Firefox or Chrome which is capable of providing full anonymity for downloading torrents? Or is a system wide VPN mandatory for that level of anonymity?

Because even with any of the 7 premium VPN proxies enabled in Best Proxy Switcher (Waterfox/ Firefox), https://whoer.net always also displays my real IP underneath my LAN IP.
Rather that expanding testing to other plugins and other browsers I thought I asked you as the expert first, rather than going down another potential rabbit hole.

Do you know if my real IP output at whoer.net with VPN (currently via Best Proxy Switcher) for sure indicates that my real IP is also visible for some unwanted spying eyes when downloading torrents via βTorrent (or maybe even any other WebRTC-capable WebTorrent client)?

I appreciate your opinion and advice.

@DiegoRBaquero : which VPN (if any) do YOU use?

Hi!

It should be system wide. I really doubt an extension will do WebRTC proxying.

If you are not using a system wide VPN, yes, that IP will be shared to WebRTC peers.

I don't use any.

Thanks Diego. Interestingly, in Waterfox, after installing and activating WebRTC Leak Shield by Hoxx Vpn, then activate anonymous VPN via Best Proxy Switcher, whoer.net does NOT display my real IP anymore (and not even my LAN IP), and web torrents still download. Needless to say I won't bet on if that also means that my real IP will NOT be shared to WebRTC peers...

Cool, that's interesting, wonder how they do it. Im gonna close this, feel free to continue discussion :)

You can download the add-on
https://addons.mozilla.org/firefox/downloads/file/820050/webrtc_leak_shield-1.0.4-an+fx-mac.xpi
open with pretty much any unarchiver because this is just a .zip renamed to .xpi and look at the source code (should be plain java script)

I don't mind sharing my IP :p :D


Do eks

Issue #, if available:
with the addition of get_metrics in the base fmbench_predictory.py class file the inference in notebook 3 and furthermore the dependent notebook 4 are not working together correctly.
Description of changes:
I have added my deploy and inference scripts for reference. please note that this solution requires the terraform script to be ran and HF token to be at the top. you will be able to complete the inference but fail when trying to call the predictor.get_metrics and further dependent data frames that require the df_metrics_list
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Got past the get_metrics issue, worked with @madhurprash to reach an issue in notebook 4. all deployment and inference is fixed.


Use url::Url or allow passing strings to Redirect

Description of the feature
It would be ideal if Poem did not use its own Uri and insead used the url crate for type-interoperability with other crates, like reqwest, etc.
Code example (if possible)
At the moment I have to do this:
    // Takes `poem::Uri`, results in URL being validated needlessly
    Ok(Redirect::temporary(poem::http::Uri::from_str(redirect.as_str())?))

I would prefer:
    // Takes `Url` type
    Ok(Redirect::temporary(redirect))

    // Takes `String` type
    Ok(Redirect::temporary(redirect.to_string()))

You are right, so now the type is changed from Uri to impl Display.

Released in v1.2.44

Thank you!


Offcanvas on start and end not showing

When implementing the Offcanvas in my component and activating it using a button.
My screen greys out but my offcanvas does not appear.
The HTML is present but the show anymation is not triggerd (As far i can see).
This only happens when placement is "start" or "end".
placement "top" and "bottom" work fine.

v. 1.4.2
OS: Windows
Browser: Chrome (Also tested in Firefox)
using the coreui-free-react-admin-template

TestComponent whit the bug:
import React from 'react'

const {
  CButton,
  COffcanvas,
  COffcanvasHeader,
  COffcanvasTitle,
  COffcanvasBody,
  CCloseButton,
} = require('@coreui/react')

const TestCase = () => {
  const [visible, setVisible] = React.useState(false)
  return (
    <>
      <CButton onClick={() => setVisible(true)}>Toggle right offcanvas</CButton>
      <COffcanvas placement="end" visible={visible} onHide={() => setVisible(false)}>
        <COffcanvasHeader>
          <COffcanvasTitle>Offcanvas</COffcanvasTitle>
          <CCloseButton className="text-reset" onClick={() => setVisible(false)} />
        </COffcanvasHeader>
        <COffcanvasBody>
          Content for the offcanvas goes here. You can place just about any Bootstrap component or
          custom elements here.
        </COffcanvasBody>
      </COffcanvas>
    </>
  )
}

export default TestCase

@wout-junius please update @coreui/coreui to v4.1.3

I updated to v4.1.3
Still the same issue
my packages installed:
    "@coreui/chartjs": "^3.0.0",
    "@coreui/coreui": "^4.1.3",
    "@coreui/icons": "^2.1.0",
    "@coreui/icons-react": "^2.0.1",
    "@coreui/react": "^4.1.2",
    "@coreui/react-chartjs": "^2.0.3",
    "@coreui/utils": "^1.3.1"

Do you have this issue in the latest version:
<EMAIL_ADDRESS><EMAIL_ADDRESS>


fix pricing for oss tech

Deleted everywhere where it is not necessary, web sites lead to github mainly.

👍


Update to latest eventing

Links to https://github.com/cloudevents/sdk-go/pull/675

I tested with the patch https://github.com/cloudevents/sdk-go/pull/675 locally and it works

@slinkydeveloper there is still an issue w/ the e2e tests for this

The lint failure is not related to this PR, i would prefer to avoid fixing it here


Add labelledby element

Adds a hidden labelledby element into the DOM for  each Maths expression to be spoken in continuous read mode by some SRS.

I'm merging this for now.  We can deal with the ignore-class stuff later.

Is there a change of double quote ever being a delimiter?
Since people get to set their own delimiters, I guess the answer is "yes", but it is a slim chance.

But it should still not matter, since the quotes are just the string quotes of the textContent element which I understand are not really part of the DOM. Otherwise (at least for the mathspeak rules) it's only alphanum and dash in the speech strings. So that should not be problematic.
Of course authors can add/change speech rules... but that might take a while until anyone does.

the quotes are just the string quotes of the textContent element which I understand are not really part of the DOM

I'm not sure what you mean by "not part of the DOM", because setting the textContent field deletes any child nodes and then adds a text node containing the text.  So there is definitely a text node in the DOM with the specified text.  I checked that to make sure.
That means that things like tex2jax will scan that text, and if the delimiters are set to things in the string, MathJax will process them.  I checked that, too, and it really does happen.
Of course, the chance that the delimiters will be set to plain words is very small.

I mean the double quotes are not part of the DOM, as in given explicitly.
The textContent is of course part of the DOM. E.g., if you inspect a text node in Chrome it looks for example like  "blurb"  , but in FF it would look like blurb

One thing that I noticed that is an issue, however, is that if existing math is re-typeset (e.g., via MathJax.Hub.Rerender(), or by using an jax's jax.Text() method to change the math expression), a second  div with the same id will be created, and the old one will remain in place.  This makes for invalid HTML, and the first element takes precedence over the second, so the label that is found will be the old value rather than the new one.  It could also lead to the page growing larger and larger if this is done more than once.
A related issue is that jax.Remove(), which is supposed to remove the output for the jax, doesn't remove this label div.  Also, if the page dynamically changes the content so that the math is removed, these divs will still be in the page.  They are hidden, so it is not crucial, but in a situation like an editor preview (which replaces the preview on every keystroke and serenaders the contents), this could lead to lots of extra detached labels.
One thing that would help would be to check if an element with the given id already exists, and reuse it rather than create a new one.  That would take care of the re-render problem.
Another thing that would help would be to place the label div within the element's -Frame span rather than the end of the document, like is done with the assistive MathML output.  That would mean that if the math is removed from the page (via the page setting innerHTML for a container element of the math, for example), then these divs will be remove along with the math and its output.
To take care of jax.Remove(), you would have to override the jax Remove() method with a new method that calls the old one, and then clean up any additional DOM elements that you have added.  That is awkward, but would do the trick.  But Remove() is not used very much, so it may not be worth it.
In 3.0, maybe there should be a means of registering DOM elements that would need cleaning up.  (This is not a use case that we considered in the original design.)  Alternatively, I have been planning to have a container element for each expression that contains all the nodes used by MathJax for that expression (something like the -Frame element, but that doesn't currently contain the script or preview elements; I want something that contains everything used by MathJax for that expression, so it is easier to locate that stuff, and to remove it.  Currently there are complicated and fragile routines for doing that.


State of Layout is lost when using HOC appWithTranslation

🐛 Bug Report

HOC appWithTranslation cause Layout lost it own state

To Reproduce
Setting next-i18next according the documentation:

Setup configuration

//My next-i18next.config.js
module.exports = {
	i18n: {
		defaultLocale: "en",
		locales: ["en", "de"],
	},
};

//My next.config.js
const { i18n } = require("./next-i18next.config");

const nextConfig = {
	reactStrictMode: true,
	swcMinify: true,
	i18n,
};

module.exports = nextConfig;

Setup app with layout

// My _app.js
import "../styles/globals.css";

import { appWithTranslation } from "next-i18next";

const MyApp = ({ Component, pageProps }) => {
	const getLayout = Component.getLayout ?? ((page) => page);

	return <>{getLayout(<Component {...pageProps} />)}</>;
};

export default appWithTranslation(MyApp);

Create sample Layout

import React from "react";

function Layout() {
	const [state, setState] = React.useState(0);
	return (

LayoutState: {state}

			<button onClick={() => setState(69)}>SetState</button>

	);
}

export default Layout;

Setup 2 pages with layout switch between to watch the state is losing

// index.js
import Link from "next/link";
import Layout from "../Layout";

export default function Home() {
	return (

			<Link href="/another-page">Go to another page</Link>

	);
}
Home.getLayout = (page) => {
	return (
		<>
			<Layout />
			{page}
		</>
	);
};

// another-page.js
import React from "react";
import Layout from "../Layout";
function AnotherPage() {
	return
another-page
;
}

export default AnotherPage;

import { serverSideTranslations } from "next-i18next/serverSideTranslations";

export const getServerSideProps = async ({ locale }) => {
	return {
		props: {
			...(await serverSideTranslations(locale ?? "en", ["common"])),
		},
	};
};

AnotherPage.getLayout = (page) => {
	return (
		<>
			<Layout />
			{page}
		</>
	);
};

Existing code in project
This is my code in _app.tsx

This is my test layout with state for this issue

Demo:
https://user-images.githubusercontent.com/62019809/199281870-b0185f24-f059-41a8-8ceb-5ec2466bf5c5.mp4
Expected behavior
A state of layout will not change when switching route
Environment

runtime version: i.e. node v16
i18next version: i.e. 12.1.0
os:  Windows

Please create a minimal reproducible example that shows no issue without the use of next-i18next.

Please create a minimal reproducible example that shows no issue without the use of next-i18next.

I attached the demo video including an example without using the appWithTranslation HOC

That's a video and not a reproducible example...
and additionally, it is not minimal... it seems to use hydration etc... stuff.

I updated the minimal reproducible example again, please help me check it out xD.
Besides, turn out i found that it was not caused from HOC appWithTranslation but the serverSideTranslations in this issue.

Currently afk.
Please create a new GitHub repository and add your code there, so I can clone it and test afterwards.

I found that this problem is not caused by AppWithTranslation HOC.
It cause by serverSideTranslations when i'm passing it down to the current page and when i redirect into another page which i'm not using that serverSideTranslation function.
Btw, this is my repo for testing this problem: https://github.com/thnam1410/i18next-rerender-layout-test

Yes, using serverSideTranslation is correct.
But honestly, I don't know why it is not persisting the state when not using serverSideTranslation, in theory it should also persist the state.

@adrai can we reopen this issue, the same issue happens with me
when there is two pages one with serverSideTranslations and the other is without serverSideTranslations the layout lose it's state when transition between the pages

@adrai can we reopen this issue, the same issue happens with me
when there is two pages one with serverSideTranslations and the other is without serverSideTranslations the layout lose it's state when navigating between the pages

For sure we can reopen it, but still don't know how to solve this... 🤷♂


Use more robust coordinates machinery in get_skycoord

This is basically a move of this comment: https://github.com/astropy/astroplan/issues/382#issuecomment-441892022, which turns out to be separate from the "real" issue #382 . @adrn pointed out there that the get_skycoord function could probably be updated to use some of the astropy machinery that concatenates sky coordinates in a way that's probably more likely to remain stable/without warnings since it would defer more of the "hard work" to astropy itself.

This needs doing carefully, because astroplan allows (and there are real world uses of) very heterogeneous bunches of coordinates - i.e ICRS, GCRS, some with distances and some without.
For this reason Skycoord.concatenate won’t work.


pull request for e2e test at 12/06/2016 12:56:01

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


Follow redirects on GET requests

Follow up to #604 and fixes #632
Redirecting on the main page (#604) doesn't fix everything on the affected networks, as some GET requests (particularly the getUserID function) return an object with the proper URL they should redirect to.

Thanks!


split the setting view in different view

Problem
Our Setting View is not showing a very nice UI because is developed in the wrong way!
Backgbround
This task required reworking the Setting View by splitting the sub-view in different Widget as suggested in this issue https://github.com/dart-lightning/lndart.clnapp/pull/35#discussion_r944285954
Goal
Make the setting view code clean to read and also easy to work on it
Object of this tasks

Show that you are able to understand how Flutter works, and not just make some code that works
Show that you are able to work with git, and follow the contributor guideline https://github.com/dart-lightning/lndart.clnapp/blob/main/docs/dev/MAINTAINERS.md
Show that you can propose and improve the application with your own idea from a general one

Please assign me to this issue.

Please assign me to this issue.

Done, please consider to test also the solution that you provide with core lightning up and running

For sure! also can you provide me some documentation on what are the node id, host, lambda server and rune are?

For sure! also can you provide me some documentation on what are the node id, host, lambda server and rune are?

I should provide this information, sorry about that! But this required a little bit of background on core lightning, maybe you can run the desktop version of the app, and use the UNIX socket? The UNIX socket path is .lightning/testnet

I should provide this information, sorry about that! But this required a little bit of background on core lightning, maybe you can run the desktop version of the app, and use the UNIX socket? The UNIX socket path is .lightning/testnet

How do I communicate with the socket? I haven't worked with sockets till now.

for the use case that I described we use https://github.com/dart-lightning/lndart.cln but the app already use it.
You use the socket :) with HTTP requests, the cln socket is just a special one that is inside the .lightning/testnet/lightning-rpc as all in linux is just a file

I only have .config file inside the .lightning/testnet/