Activity tab shows incorrect ammount

this tx shows as 600K ada tx in my activity tab.
https://cexplorer.io/tx/85101faf0babb24201204c595d7d207c10ee8cb5a6f1114a36f77f96849c5a15#data
I think its because its a smart contract tx. but I think the wallet should just show just the amounts that go to / from my wallet address.

basically same as #473


Code execution (interpreter) support

It's easy enough today to implement an "eval" action on an agent and use it to run arbitrary code in python or the shell, but given the power of code execution in general, I want to explore how it can be supported as a first class feature.
Following the React app updates, I plan to explore how this could be more directly supported.
In the meantime, this is all it takes to create a simple code-interpreter-like agent if you'd like to play around with this concept.
@action
def eval(self, code: str):
    """Evaluate python code within this agent's context"""
    eval(code)

I've decided that this feature will be addressed in a new independent project dedicated to creating a new UI for experimentation and development. See the discussion here for more details.
If you'd like to sign up to be emailed when a prototype is ready, please click here.


Please add note about AppServiceFileAuditLogs

From Ruslan's blog
https://ruslany.net/2020/01/how-to-monitor-azure-app-service-content-changes-and-publishing-activity/
The file change audit logs (AppServiceFileAuditLogs) are only available for app services in Premium, PremiumV2 and Isolated App Service Plans. You will need to scale up the app service plan in order to get those logs.
but we didn't mention this limitation in azure document, please add note about it, in case Cx gets into problem.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: bfc316af-0d30-b449-a04f-3019b7a16bc1
Version Independent ID: 01c4c435-fda9-374a-70b3-f1d7f13c7651
Content: Enable diagnostics logging - Azure App Service
Content Source: articles/app-service/troubleshoot-diagnostic-logs.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

@rachelxj-ms, Thanks for the feedback! We are taking a look into this and will get back to you soon.

@rachelxj-ms, The section "Supported log types" in the document, highlights "File changes made to the site content; only available for Premium tier and above", hope this clarifies Or let us know if you still think more clarity on this necessary.

@AjayKumar-MSFT Thank you, highlight is better, PG is going to fix this in portal

@rachelxj-ms, Thanks for the update.
Since document update is not required, we will close this out, but if you feel you need more information please just let us know.


Suggested to update with no admin rights then fails to launch

Preflight Checklist

[X] I have installed the latest version of Storage Explorer.
[X] I have checked existing resources, including the troubleshooting guide and the release notes.
[X] I have searched for similar issues.

Storage Explorer Version
1.32.1
Regression From
No response
Architecture
x64
Storage Explorer Build Number
No response
Platform
Windows
OS Version
Windows 10
Bug Description
I agreed to update and now I am not able to use an app at all.
Steps to Reproduce

Agree to update without admin rights
Be unable to use the app anymore

Actual Experience
App suggested to update to new version, with a message I can install just for my account, so no admin rights are needed. It removes the previous version first, then tries to install new version, and only then checks I need .NET to be updated to v6.0. For which I need admin rights, now I am not able to run previous version, as it was deleted, and cannot run new version as it requires .NET 6. I agreed to update and now I am not able to use an app at all.
Expected Experience
Add a message you need admin rights for an update before the previous version deleted. Add an option to download previous versions, to be able to use an app in case user has no admin rights.
Additional Context
No response

Ok, thank you for the link @MRayermannMSFT


quick fixes for two issues: nil pointer and type mismatch

This PR fixes two issues:

type mis-match for "Description" field of "AccessPackageResource", reference: https://learn.microsoft.com/en-us/graph/api/resources/accesspackageresource?view=graph-rest-beta
This is introduced recently.
For AccessPackageResource, when there is no match on Azure, AccessPackageResources will be an empty array, hence the fix

@kaovd @manicminer can you please take a look and merge this quickly? Thanks.
They are all quick simple bugs/fixes to me.

Thanks @manicminer. The golang-lint workflow error here https://github.com/manicminer/hamilton/actions/runs/3238793336/jobs/5324839976 seems not related to this PR, but it does reveal an excessive check on my local, which I've fixed in the additional commit.

Rebased for test fixes


User not logged out after restart docker

Environment

Ruby 2.5.5
Rails 6.0.2
Devise 4.8.0

Current behavior
Hello guys, I have a problem with my app, the app is build with docker and the rememberable option is not configurated in user.rb file, when i stop the container and start it again the user is still logged in the system. Cleaning the cookies or cache doesn't work, the user is still logged in.
user.rb
class User < ApplicationRecord
audited
devise :database_authenticatable, :registerable,
:recoverable, :validatable, :trackable
Expected behavior
When the container restarts the user should be logged out and asked to sign in again into the system.

There's no reason for the user to be logged out if the containers are restarted, as the session cookie may still be alive for that browser session -- that's what you'd control whether they're logged in or not (despite rememberable being off).
However, clearing the cookies should log them out I believe. You still having problems with that?


Project creation results in files with read only permission (444) as opposed to read-write (644)

Files should be writable by owner (644)

Hi @petejkim,
Can you give us the exact command you typed and which files have the wrong permissions?
Thanks!


Configure debug info to DWARF v4.

As done by @thegecko in https://github.com/lancaster-university/microbit-v2-samples/commit/b0ab5ffeafe6b3a47a11d65f611fe289feff4df0.
This should also help with some of the issues I've seen with bloaty parsing DWARF v5 files produced by arm-none-eabi-gcc v12.3.

@Johnn333 could you apply the same dwarf flags to the LLVM PR and ensure they compile correctly?

BTW, if you set the build mode to regular 'debug' you will also need these flags set:

CMAKE_C_FLAGS_DEBUG
CMAKE_CXX_FLAGS_DEBUG
CMAKE_ASM_FLAGS_DEBUG

Looking at the docs it looks like CMAKE_FLAGS_DEBUG_INIT initialise the flags for CMAKE_FLAGS_DEBUG, so that should be fine?

This didn't work for me, perhaps because versions

@Johnn333 could you apply the same dwarf flags to the LLVM PR and ensure they compile correctly?

 I'm hoping to spend some time at some point getting the GCC built libraries to be compatiable with LLD (currently Clang has to build all of CODAL and use those libraries), but with some of the changes to CODAL its much closer to being able to digest the default GCC ones. Wouldn't worry too much about the LLVM build :).

Okay, so essentially the CMAKE_<LANG>_FLAGS_<CONFIG>_INIT CMake variables didn't seem to be working, and they weren't overwriting the default values for CMAKE_<LANG>_FLAGS_<CONFIG>.
More info can be found in:

https://github.com/lancaster-university/codal-microbit-v2/issues/373

Since we have to support older version of CMake anyway, I've replace CMAKE_<LANG>_FLAGS_<CONFIG>_INIT with CMAKE_<LANG>_FLAGS_<CONFIG> (setting the variable, rather than appending to it, as we want these to be the starting values).
With this change and comparing the before and after this patch, there are significant changes in the build. That's because until now we've been building with -O2 instead of -Os.
As a quick test, I've change the optimisation level on top of this patch to O2, built again, and compared that with a build from master.
With this patch + O2, doing a diff of this map file and the map from master, and the only difference is the size of the debug strings, everything else is the same, so that's a decent test that this PR is working as expected.
Before we can merge this PR with -Os, we should have a look at the affected components, as listed in https://github.com/lancaster-university/codal-microbit-v2/issues/373#issuecomment-1712037165 to see if anything that might be performance critical could be negatively affected.
@JohnVidler could you have a look at the list there and confirm if we are relatively safe to introduce performance issues?


Idea: General syscall ABI

Right now the Linux syscall interface uses a7 for the syscall number and a0-a6 for integer arguments.  This is quite adequate for Linux, where syscalls have at most 7 integer arguments; but I think it'd be cleaner to have a general syscall ABI defined with the same functionality as the base ABI.
Straw proposal: syscall number in t0 or t1 (TBD; t1 is friendlier to millicode) before ecall, everything else as for the base calling convention.

Making syscalls look as much like function calls as possible is a reasonable goal.  In one respect, they are quite different, though: syscalls preserve all registers, except a0.  I assume we want to keep it that way.
I'm fine with t1, since syscalls could be in millicode routines that would then have to juggle the return address.

This might have been premature…

Linux is not fully representative in returning only one register from syscalls using the negated errno trick.  The BSDs typically return two values, errno-or-return and which-was-it (the latter is usually the carry flag, but it's $a3 on MIPS).  We can pretend this is a struct, but since we defined C return for small types as "exactly as passing one argument", there's a single obvious way to extend the calling convention to multiple return values.

L4-ish systems like to pass and return several registers unchanged (by treating them as fixed in the kernel; L4s like to treat synchronous IPC as the kernel hot path).  This seems to be mostly a special case of "multiple return values", though.

I'm not necessarily opposed to declaring one or more registers as possibly-clobber.  There's design space out there for future/alternate privileged architectures to implement ECALL without the full sscratch dance; if you want to, let's say, take a system call with interrupts automatically enabled, "sepc" has to be rerouted somewhere else, and it could be a GPR or a CSR.  Not something I want to persue myself any time soon.

The biggest advantage of this proposal is that there's one obvious way to extend it.  If someone wants 8 or 9 arguments, they can, and there's no question about how.  (9 requires reading user sp, but i386 kernels already do that regularly.)  Everyone on x86_64 uses argument registers a bit differently, which would be nice to avoid.

The biggest disadvantage is that anything that's not already broken by the calling convention changes and the ELF flag changes will no longer be able to make system calls.

Number 5 is a non-trivial concern to me, since it falls to me to do all the work.

I don't follow number 4.  Either strategy supports more arguments pretty seamlessly, right?

I don't think 3 is worth considering for this privileged architecture.

The L4 approach seems roughly compatible (preserve all arguments that would not be clobbered by the syscall itself), yeah?

The errno-return style the MIPS ABI employed never made much sense to me, but treating it as an a0/a1 struct seems compatible.  The trick here is the entire ABI has to know about it; presently, all glibc syscalls are treated as clobbering only a0.

The situation I was trying to avoid is: I'm porting some niche/homebrew OS to RISC-V, and I follow the Linux convention because Schelling point.  Then I get to a syscall with 8 arguments.  Do I (a) pass the eight argument on the stack, (b) move the syscall number somewhere else, (c) put the eight argument in a non-a-register?
Anyway, it's probably not worth changing Linux at this point.  Leaving final discretion about whether to close this up to you.

@palmer-dabbelt any thoughts?  I'm leaning towards "it doesn't really matter much, so let's avoid the hassle."

There are a few other nice effects of having a common ABI between function calls and syscalls, one of which is uniformity of debugging, and another of which is emulation. The two are connected - as an example, on Exherbo seccomp mode 2 is used with SECCOMP_RET_PTRACE to sandbox package building.
In addition, a common ABI between functions and syscalls makes it entirely feasible to replace a (addi t1, zero, NR; ECALL) sequence with (AUIPC t1, table; JALR whatever, t1, NR) or some similar sequence at relocation time in order to substitute a non-privileged syscall implementation.
This is immensely useful for anything akin to "zones", etc, and would make foreign binaries vastly easier to support - currently, the midipix project builds a custom musl in order to sub out syscalls so Linux programs can run on Windows. Using a common ABI could reduce that to nothing more than an ELF loader patch.

This ship has already sailed. Linux does one thing and FreeBSD does another (more sensible IMO, putting the syscall number in t0 rather than wasting an argument register). Besides, this is an OS-specific concern that does not affect toolchains.


Add docs to demonstrate adding new route

Describe about adding new route /contact-us

https://www.reactpwa.com/docs/en/add-new-route.html


Encoding issue on windows when saving a table

Description
When saving a table with the save message, an exception occurs in Windows.
Reproducible example

Run pandas/jupyter on a windows maching
create a simple table
try saving e.g. df.pipe(GT).save('test.pdf')

The bottom of the stack trace shows this
File D:\Python\Python312\Lib\site-packages\great_tables\_export.py:310, in save(self, file, selector, scale, expand, web_driver, window_size, debug_port, _debug_dump)
    303 with (
    304     tempfile.TemporaryDirectory() as tmp_dir,
    305     wdriver(options=wd_options) as headless_browser,
    306 ):
    307
    308     # Write the HTML content to the temp file
    309     with open(f"{tmp_dir}/table.html", "w") as temp_file:
--> 310         temp_file.write(html_content)
    312     # Open the HTML file in the headless browser
    313     headless_browser.set_window_size(window_size[0], window_size[1])

File D:\Python\Python312\Lib\encodings\cp1252.py:19, in IncrementalEncoder.encode(self, input, final)
     18 def encode(self, input, final=False):
---> 19     return codecs.charmap_encode(input,self.errors,encoding_table)[0]

UnicodeEncodeError: 'charmap' codec can't encode character '\u2212' in position 7916: character maps to <undefined>

Note that the error occurs due to CP1252 encoding which is the default. I fixed it by adding encoding='utf-8' ib line 309 of the _export.py file:
    # Write the HTML content to the temp file
    with open(f"{tmp_dir}/table.html", "w", encoding='utf-8') as temp_file:
        temp_file.write(html_content)

(Sorry I'm not comfortable with pull requests I don't want to ruin anything)
Expected result
A file test.pdf should be created
Development environment

Operating System:  Windows 10, Python 3.12.4
great_tables Version: 0.11.0

Encountering the same issue when trying to utilize a Unicode character in the heading section (in lieu of an image). Unable to figure out a workaround yet without just removing the character.

I just tested it, with my fix in place that works without issues (windows
10, python 3.12.4)
On Sat, Sep 14, 2024 at 8:58 AM HansMellman @.***>
wrote:

Encountering the same issue when trying to utilize a Unicode character in
the heading section (in lieu of an image). Unable to figure out a
workaround yet without just removing the character.
—
Reply to this email directly, view it on GitHub
https://github.com/posit-dev/great-tables/issues/440#issuecomment-2350982021,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEOWAQKP6H4EUPFPID7H263ZWQXHFAVCNFSM6AAAAABODZ26S6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJQHE4DEMBSGE
.
 Thank you for that - did the trick perfectly! Hopefully they'll be able to implement the fix natively.

I'll close this issue as the Unicode problem has been resolved. GT.save() now includes an encoding parameter, which defaults to utf-8 and allows users to specify other encodings if needed.


Adding types and dfn handling

Add new group "html"
Adding new types to types list
Add "for" handling for all dfns

Opened issue #547 on me to finish cleanup in the future.


MASS should not require user to supply a peak time in microseconds

Original report by Joseph Fowler (Bitbucket: joe_fowler, ).

@christinegpappas reminded @oneilg today that it's stupid that the user needs to tell MASS the pulse peak time in microseconds before it can even do a summarize_data() step. (I'm sure she said it in a much nicer way.) This is an artifact of the very earliest days of MASS, and we can definitely get rid of it.
@oneilg has shown that one can infer the peak time by a simple examination of the most common peak time among all records in the first data segment.
I propose to learn the peak time this way, and then remove the peak_time_microseconds arguments from the relevant methods' argument lists in the TESGroup and MicrocalDataSet objects. I realize that this will break some user scripts. Perhaps this should be part of a new version 0.6 that's worth announcing to all users.

Original comment by Galen O'Neil (Bitbucket: oneilg, GitHub: oneilg).

For inspiration, look at
https://bitbucket.org/oneilg/pope.jl/src/d105b6752b294d3431dbd996d01d8045ea705144/scripts/make_preknowledge.py?at=master&fileviewer=file-view-default
in functions estimate_peak_index_ds, estimate_peak_time_microsec_ds, calc_cuts_from_noise

Original comment by Joseph Fowler (Bitbucket: joe_fowler, ).

Well, fine, maybe we can continue to accept a non-default argument, if it's a real number, while setting the default now to be None.

Original comment by Galen O'Neil (Bitbucket: oneilg, GitHub: oneilg).

I guess the other option is to deprecate the old function, so just make it throw an error with a useful error message describing the new way. Or maybe combine them, let the old way work, but make it print out a bunch of stuff about how you shouldn’t do it, and sleep for 10 seconds so it’s slower and you can see the messages.

Original comment by Joseph Fowler (Bitbucket: joe_fowler, ).

Here's the plan:

TESGroup.summarize_data(...) and MicrocalDataSet.summarize_data(...) both take a peak_time_microsec argument. Set its default to None, but let the argument still exist.
Remove the corresponding argument from MicrocalDataSet._summarize_data_segment(...) because the value should be stored as an object attribute (probably as a sample number, not as microseconds).
If argument is None, we'll notice this in the first call to the per-segment method and use Galen's approach to choose a non-None value.
Ultimately this will change the argument mass.core.analysis_algorithms.compute_max_deriv(...ignore_leading=X...)

Original comment by Galen O'Neil (Bitbucket: oneilg, GitHub: oneilg).

I'd suggest not breaking scripts, either made peak_time_microsec be calculated automaticaly (and stored) when the argument is None, or "auto", or make a new function.


avg.sh best exp/deepspeech2/checkpoints 1  时报错  too many indices for array: array is 1-dimensional, but 2 were indexed

IndexError: too many indices for array: array is 1-dimensional, but 2 were indexed
Failed in avg ckpt!

可以使用pbd或者dump定位一下报错的具体位置。目前猜测可能是由于ckpt模型文件不存在。


Add Cambodia holidays

Proposed change

Add Cambodia holidays (en_US, km, th localization).
Closes #1160.
Type of change

[x] New country/market holidays support (thank you!)
[ ] Supported country/market holidays update (calendar discrepancy fix, localization)
[ ] Existing code/documentation/test/process quality improvement (best practice, cleanup, refactoring, optimization)
[ ] Dependency upgrade (version update)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (a code change causing existing functionality to break)
[ ] New feature (new python-holidays functionality in general)

Checklist

[x] I've followed the contributing guidelines
[x] This PR is filed against beta branch of the repository
[x] This PR doesn't contain any merge conflicts and has clean commit history
[x] The code style looks good: make pre-commit
[x] All tests pass locally: make test, make tox (we strongly encourage adding tests to your code)
[x] The related documentation has been added/updated (check off the box for free if no updates is required)

coverage: 99.955% (-0.04%) from 99.991% when pulling 56ab3dc1965012f398685cfcda72c616825388a0 on PPsyrius:cambodia_holidays into 0ec26da847010a61d157c7b3a0d0d0563ade7b03 on dr-prodigy:beta.

Oof, forgot to check that the new normalized tuples doesn't work prior to #1296 getting merged - will revert my changes for now.
That and waiting for #1295 changes to holiday group in general before I attempt migrating _ThaiLunisolar for that since Laos will likely ended up using that too for their 10+ Buddhist holidays that are dependent on this.

ThaiCalendarHolidays is now implemented and should hopefully simplifies future #1215 implementations down by a lot. 🙏🇹🇭🇱🇦🇰🇭

Seems good now 🎉

Hey @PPsyrius thanks for taking care of another entity.
It's great to see your persistent work on quite complicated holiday calendars!


'_body_enter_tree: Condition "!E" is true' errors when a scene containing an Area is removed and re-added to the tree

Godot version:
3.2.2, but also reproduced in upcoming 3.2.3, in commit 9bf5a0b791faf8e960bdb07009b5fe8003206e4f on the 3.2 branch
OS/device including version:
Ubuntu Linux 16.04, 64-bit
GeForce GTX 970 with NVIDIA 384.130 driver
GLES3 backend
Bullet Physics
Issue description:
If you remove a scene containing an Area overlapping a few bodies, then re-add the same scene strange errors appear in the error log.
E 0:00:02.384   _body_enter_tree: Condition "!E" is true.
  <C++ Source>  scene/3d/area.cpp:123 @ _body_enter_tree()
  <Stack Trace> Reproducer.gd:23 @ _input()
E 0:00:02.384   _body_enter_tree: Condition "!E" is true.
  <C++ Source>  scene/3d/area.cpp:123 @ _body_enter_tree()
  <Stack Trace> Reproducer.gd:23 @ _input()
E 0:00:04.217   connect: Signal 'tree_entered' is already connected to given method '_body_enter_tree' in that object.
  <C++ Error>   Method failed. Returning: ERR_INVALID_PARAMETER
  <C++ Source>  core/object.cpp:1515 @ connect()
E 0:00:04.217   connect: Signal 'tree_exiting' is already connected to given method '_body_exit_tree' in that object.
  <C++ Error>   Method failed. Returning: ERR_INVALID_PARAMETER
  <C++ Source>  core/object.cpp:1515 @ connect()
E 0:00:05.167   _body_exit_tree: Condition "!E" is true.
  <C++ Source>  scene/3d/area.cpp:140 @ _body_exit_tree()

Steps to reproduce:

Run the reproducer
Press space when any of the kinematic or rigid bodies in the scene is inside the semi-transparent red Area
Wait a second
Close the window
Check the error log

Minimal reproduction project:
Physics-RemovedNodes.zip
The Reproducer scene contains a number of physics bodies:

A green CSGBox providing a static collision
A semi-transparent red box, demarcating the boundaries of the Area in question.
A red bouncy ball (a RigidBody), set up to bounce in and out of the Area
A white capsule (a KinematicBody) scripted to move back and forth through the Area
A brown beam (a RigidBody on a HingeJoint) that is not relevant to this issue

When you press space, the scene is scripted to:

Pause the tree
Remove itself from the tree root
Wait 0.1 seconds (which may not be necessary?)
Then re-add itself to the root and unpause the tree

CC @madmiraal


feat: add subscription detach sample

Closes #1149.

Looks like it's possible if you follow these instructions

Are we publishing docs with this feature @pradn ? If not, I'd prefer to hold off merging until the feature is GA

Thank you for adding the sample!


Count number of open connections in Pool

Due to a bug in our code, I accidentally opened 1000s of connections to redis. In order to avoid this in the future (and to be able to track the number of connections), I want to be able to know how many connections are in the pool right now.
As I see, this is just a matter of exposing len(pool), right? If you'd be willing to accept a MR (and my approach is correct, since I'm kind of new to golang), I'd gladly do it.
Just need a "go" from you ;)

Hi! Thanks for submitting the issue, I guess there's some points to clarify first:

Yes, exposing len(pool) would allow someone to see the number of connections in the pool. kind of. In reality, that would give the number of connections that are available to be used at any given moment, it does not take into account connections which have been gotten but not yet put back. So a more accurate description of len(pool) would be "Number of available connections in the pool".

The number of available connections in the pool will always be less than the size parameter passed in when the pool is created. In the case you're describing, where you accidentally created thousands of connections, it wouldn't actually help you to know the number of available connections, since that number wouldn't grow with all the new connections you're making.

While being able to see the number of available connections might be useful, I don't think it would help in this case so I'd like to hold off until a better use-case presents itself. If I could ask, what was the exact bug which caused you to create all those connections?

Ok, you're right, under these circumstances, len(pool) instrumentation wouldn't have seen the bug:
for {
  redis := pool.Get()
  defer pool.Put(redis)

  // do stuff and sleep
}

Rather dumb bug cause by developer dumbness ;-)

lol, yeah that would be a really tough one to catch. Depending on what // do stuff and sleep entails, you might be able to use the Cmd method, which does the Get and Put for you. Unless you specifically need pipelines or transactions it's often the best way to go, cause it prevents small bugs like this :P
I'm gonna go ahead and close this out, feel free to re-open it if you have any more questions :)


[NIFI-413] PutKafka Compression and Batching Support

Adding properties to PutKafka processor to support asynchronous producing with configurable batches.
Also added user-defined control over compression codec and compressed topics per NIFI-413.
Producer Type remains synchronous by default.

Brian,
Thanks for the update! I left several comments inline - generally just small tweaks in order to keep the property naming scheme consistent with the typical conventions that we use in NiFi.
Thanks!
-Mark

Thanks for the feedback, @markap14 !  Updating now...

@markap14 PR is now updated based on your feedback. Thanks for guiding this processor toward a more user-friendly experience.

Happy to provide feedback! Tested it out and all looks good. There were some checkstyle failures, so I formatted the code to fix those. Otherwise, all is working well. Pushed to develop.
Thanks for catching and correcting this!


Please add ability upgrades to the API

The Ability Upgrade array includes information about timing for ability/level and is something we are very interesting in using.

You can get this data from the Steam WebAPI (GetMatchDetails call)  The OpenDota project only stores the ability upgrades themselves (dropping the timing data) in order to save space.

Please reconsider this idea. you have timing information a large array of topics but it sucks having to query another API (the steam API sucks) and join that with yours only to get level up/ability timings.

Will consider it, but even if it's added it will only be available for future matches.  Backfilling this data for the 1.4 billion old matches will take months.

Thank you and future forward is completely understandable.


Add Audio Wave preview.

Goal:
When loading an audio within engine or editor an audio preview would be available with the following data points:

Length of audio in milliseconds.
Volume at each moment within the track.
Data useful for visualization of each audio track for play and preview.

Moving to Summer 2024 milestones/sprints.
Is an IDE/Editor feature that is useful, but not ultra priority for engine and/or export options. Very nice visual feature to have tho. Requires each audio module to store or be able to render/stream audio data.
Tested in SDL_Mixer_Bark, but not in raylib or other modules yet..


v0.13 breaks setup using tracing_subscriber_layer

Up until v0.12.1 the tracing integration was working ok. However, the v0.13 update results in no events being displayed by the widgets.
I suspect this is related to move_events now being called in a background task that's initialized when calling init_logger. However I can't use init_logger since that'll try to override (and error) the tracing_log integration I have already set up.
Is there a workaround for this that allows me to keep using tracing_log, i.e. to send events to the widgets without setting up tui-logger as the log's global logger?

Thanks for the report. I assume the line 500 in lib.rs is problematic:
log::set_logger(&*TUI_LOGGER).map_err(|err| {TuiLoggerError::SetLoggerError(err)})

Could you please try by commenting this line out ? Without this line init_logger will still start the background thread but will not call log::init_logger().  If this works, then that line can be disabled by conditional compilation, if the tracing feature is enabled.


New thermal camera integration test failure for Edifice

After forward porting some changes from ign-rendering4 to main, a new integration test failure appeared for the thermal camera. More information can be found in #246.
Environment

OS Version: MacOS
Source build (commit https://github.com/ignitionrobotics/ign-rendering/commit/52c4ab0d5a57a793d90171cbc54f324e8aac37d4, main branch at the time of this writing)

Description

Expected behavior: test/integration/thermal_camera should pass on MacOS.
Actual behavior: test/integration/thermal_camera failed on MacOS.
test/integration/thermal_camera does not fail for MacOS on the ign-rendering4 branch: https://github.com/ignitionrobotics/ign-rendering/pull/246#issuecomment-778477528

Steps to reproduce

Clone the repo: git clone https://github.com/ignitionrobotics/ign-rendering.git
Complete the build so that tests are available:

cd ign-rendering
mkdir build
cd build
cmake ..
make

Run the thermal camera integration test:

# the following steps assume you are already in the ign-rendering/build directory
cd bin
./INTEGRATION_thermal_camera

Output
Here is the original failure from CI in #246:
/Users/jenkins/workspace/ignition_rendering-ci-pr_any-homebrew-amd64/ign-rendering/test/integration/thermal_camera.cc:233
The difference between boxTemp and temp is 14, which exceeds boxTempRange, where
boxTemp evaluates to 310,
temp evaluates to 296, and
boxTempRange evaluates to 3.

It seems like there are related test failures in https://github.com/ignitionrobotics/ign-gazebo/pull/622. I'm guessing that the thermal camera behavior is different in Edifice.

Another instance of this test failure: https://build.osrfoundation.org/job/ignition_rendering-ci-ign-rendering5-homebrew-amd64/17/
I noticed many instances of the following logged error (I'm not sure if it's relevant):
[Err] [Ogre2RenderEngine.cc:467] Unable to find Ogre Plugin[/usr/local/opt/ogre2.1/lib/OGRE-2.1/RenderSystem_GL3Plus]. Rendering will not be possible.Make sure you have installed OGRE properly.

I'm guessing that the thermal camera behavior is different in Edifice.

The test has been flaky on all versions, so I don't think the issue is version-specific. Here are the histories:

ign-rendering3 - has the most failures
ign-rendering4 - surprisingly no failures
ign-rendering5 - few failures
main - again, no failures

https://github.com/osrf/buildfarmer/issues/224


WHY ?

why doesnt work on mobile phone?

because whatsapp-web.js uses puppeteer, while puppeteer doesn't work on mobile phones :<


global: inclusion request api change

InclusionRequest.get returns None on fail.

Signed-off-by: Krzysztof Nowak<EMAIL_ADDRESS>

Can you add a release commit so we get it published on PyPI immediately as well.

Coverage decreased (-0.08%) to 67.347% when pulling 29c5e50066e6b8d1b31d1208031c021ba3061edc on krzysztof:refactor into dae341c26f681fb82a77ce6f59dee24967a87de7 on inveniosoftware:master.

As far as I can tell :shipit:


gattc: Fix unnecessary status overwrite in read by type err

This status change is not needed. With this patch it is possible to pass TC_GAR_CL_BI_07_C.

Good catch.  Thanks, Michał.


add roberta support

Add RoBERTa support in squad_torch_bert config.
The changes concern the shape of token_type_ids specific for RoBERTa

Now distilbert, roberta, xlm, bart and longformer-like models can be trained correctly (confirmed on a GPU)


kona_ev / grandeur / grandeur hev add

Choose one of the templates below:
Fingerprint
kona_ev fingerprint add
grandeur fingerprint add
grandeur_hev fingerprint add
sonata fingerprint add
sonata20 fingreprint add
sonata_hev , sonata20_hev make
Car support
grandeur / grandeur hev interface add
sonata / sonata20 / palisasde interface add
Feature
This pull requests adds feature X
Description
sonata20 use crc code
sonata under 2020 not use crc code
Testing
Explain how the feature was tested. Either by the added unit tests, or what tests were performed while driving.

thanks @crwusiz
could you please add info to README.me file?

thanks @crwusiz
could you please add info to README.me file?

ok.. i added readme.me


Contributors.md

The list of original contributors to the project. If we decide to list all contributors here, no problem. Just add more names.

Could you add me also for the stabilizer simulator?

Done


Add new test and fix for 3309

Test that conf_remap can override float settings (e.g. proxy.config.http.background_fill_completed_threshold) successfully

Why did you merge this in to your master?

Thought it was normal when closing. The PR back to the open source repo is still off the branch, though.


Remove docs from the develop branch

Docs are exist in two incarnations: in the root of gh-pages branch and in docs directory of develop. Let's remove docs dir from develop branch. Why this should be done? Because it is very hard to read diffs of PRs containing docs update now.

I'm fine with this. Can you open a PR?


Adds optional dependencies to features when they are enabled indirectly

When Cargo sees a dependency feature opt_crate/feat for an optional dependency opt_crate,
it implicitly adds opt_crate to the feature set for the current crate.
For example, num-rational crate relies on this behavior for importing num-bigint, which is optional.
This patch lets crate2nix mimic this behavior.
Main changes are in crate2nix/templates/nix/crate2nix/default.nix
I also modified test cases(crate2nix/templates/nix/crate2nix/tests/packageFeatures.nix) so that they match what Cargo does.
Closes #146, #182.

Woah, thank you, @pandaman64!
Prime example of a PR I love.

You clearly write what you want to achieve and what you changed. (that significantly reduces the time that I need to review it)
You have nice inline comment that also explains it.
You adjusted the unit tests.

:fireworks:

Thanks! Just confirming that it works for us too, I tried it today.


How can i make the modal overlay non blocking?

Hello, we do have a prop to disable closing the modal on overlay click and we can hide the overlay using css / styling. But is there a way to make the background / overlay non blocking, meaning that the user should be able to interact with the background (currently the overlay blocks the background)

What you want is not a modal...modal is a blocking activity.

hmm true. The concept for the modal is a bit different than what I want to achieve.
Thanks btw 👍, giving it the same height as the content and unsetting top makes it non blocking (i'm doing this only for mobile screens because I want the modal to behave like a bottom sheet).


Fixed error when removing file selection

When selecting a file and then removing the selection onChangeEvent.target.files[0] was undefined, and therefore accessing the name of the first file through "cannot read property 'name' of undefined". Now the change function checks whether there is selection

thanks for this @maryum375

Thank you! :)
BTW, Why didn't it assigned me as a contributor to the repo?

I honestly have no idea why ...

I just merge another pull request and the same happened, strange ...


Russian translation (2)

Code in actions.py is unchanged

Thanks for updating the submission!
Sorry I didn't notice this before, but the changes from the ardublockly/msg/ru.js file should have been added to ardublockly/json/ru.json instead. The ru.js is automatically generated by the Blockly build process, which combines the content from blockly/msg/messages.js and ardublockly/json/ru.json.
Could you this change? There is no need to remove the content from the ru.js as I will have to build Blockly after your submission, but they need to be in the ru.json file or the content will be lost.


Added more details on HoloLens 2 compatibility for Holographic Remoting

Docs Build status updates of commit a214b8b:
:warning: Validation status: warnings

File
Status
Preview URL
Details

mixed-reality-docs/add-holographic-remoting.md
:warning:Warning
View
Details

mixed-reality-docs/add-holographic-remoting.md

Line 18: [Warning] Illegal link: `<a href="index.md#news-and-notes">coming soon</a>` -- missing bookmark. The file index.md doesn't contain a bookmark named news-and-notes.

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.


tapaal: init at 3.9.3

Description of changes
Added new derivation for TAPAAL (Petri-Net tool), version 3.9.3.
(My first nixpkgs PR)
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage

(this package does not have dependencies)

[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

you have to add yourself to maintainers/maintainer-list.nix


Add Extensions Guidance

Add comprehensive extensions guidance to each verb.
Update extensions to be simple types rather than objects.

I think this leaves very comprehensive instructions for the users. All of this metadata is crucial in collecting the right information for DoD applications. Adding this data along with other profiles in the works will set a foundation for AI use in instruction.


Type and usage-type are not tracked correctly by the UnRedoCtrl-Class

New values in type and usage type are not tracked when switching to another object and no other properties have been changed in the selected object after changing the type or usage type

This issue is not directly issue of the undo redo functionality.
The same problem occurs when after typing a new value it wont be directly added to the dropdown-list.
When opening the dropdown it will use the first entry to set the text property.
Probably the same effect occurs when selecting another object.

Investigated the issue further:
Turns out the gui (dropdown) doesnt report value changes until another field has been selected.
There for the value isnt stored in the objects type property and therefor the list wont show the new value until something else has been selected.
Might need another property to properly track all changes

Issue solved:
Links used to resolve the issue:
https://github.com/LukasWilllinFHNW/MInventory/issues/27
http://stackoverflow.com/questions/32620739/java-8-datepicker-and-editable-combobox-behavior-change-between-8u51-and-8u60#32629396
By definition the valueProperty is only changed after pressing enter or taking away the focus. When enter is pressed or focus moved the value of ComboBox.getEditor().textProperty() is commited to the ComboBox.valueProperty()
We now only use the ComboBox.getEditor().textProperty() to track changes


CRDs do not get upgraded

Hi 👋🏼  The way crds are handled in this repo Helm does not upgrade them when the user upgrades the chart.
https://helm.sh/docs/chart_best_practices/custom_resource_definitions/#some-caveats-and-explanations

There is no support at this time for upgrading or deleting CRDs using Helm. This was an explicit decision after much community discussion due to the danger for unintentional data loss. Furthermore, there is currently no community consensus around how to handle CRDs and their lifecycle. As this evolves, Helm will add support for those use cases.

What I've seen most chart developers do it have a installCRDs option in the helm values and template them out, take a look at cert-manager and rook-ceph charts for an example on how they handle CRDs.
I would suggest a option in the values.yaml and to move the crds folder into the templates folder and wrap it in an if statement.
operator:
  create: false
  installCRDs: true
  autoRestart: false

+1
I am currently in the process of deploying 1Password Connect and Operator into our infrastructure using:
operator:
  create: true

However, I would like to use the latest Connect and Operator to leverage the latest feature: "set type on OnePasswordItem resources, which can be any of the Kubernetes secret types" as discussed here.
I have tried upgrading it via the values.yaml by setting under operator version: "1.5.0" like below but there is an error: unknown field "type" in com.onepassword.v1.OnePasswordItem which I believe is related to the CRDs not upgrading. I am using ArgoCD + Helm to deploy.

I managed to upgrade 1Password Connect Server to 1.5.4 and Operator to 1.5.0  using the latest chart revision 1.8.0 that released 2 days ago. I went from revision 1.7.1 to 1.8.0 using ArgoCD App template + Helm source.
However, I am getting the error from the operator when trying to take advantage of the new type feature and doing type: kubernetes.io/dockerconfigjson. I already tried redeploying 1Password and the same issue occurs.
Error: "error":"Failed to retrieve item: need at least version 1.3.0 of Connect for this function, detected version 1.2.0 (or earlier). Please update your Connect server"

I created a new issue to track my issue: https://github.com/1Password/connect-helm-charts/issues/105

Thanks for raising this issue. @onedr0p feel free to make a PR with the suggested changes

I bring that up again.
Today I patched to the current chart version 1.7.1 and suddenly I had issues to apply new 1PasswordItems because of the error onepassword.com/v1, Kind=OnePasswordItem): .type: field not declared in schema.
After trying some stuff I decided to run
kubectl apply -f https://raw.githubusercontent.com/1Password/connect-helm-charts/main/charts/connect/crds/onepassworditem-crd.yaml manually.
Since then it looks good again.
Is there an issue in the current chart release?
We had no issues in earlier versions like 1.5.x or 1.6.x.


Support range-based charge queries from SMARTS

Description:
This would be [+{1-}], etc.

I like the  idea - is it 'standard' SMARTS (whatever that might mean!)?

Nope. These are RDKit extensions: https://github.com/rdkit/rdkit/blob/master/Docs/Book/RDKit_Book.rst#range-queries
CACTVS has something similar (at least that's where the inspiration came from).

Thanks Greg - that's an interesting read!
Any plans to support $$()-style recursive queries (e.g. https://www.xemistry.com/docs/html/tcl_reference-49.htm) - "Standard recursive SMARTS does not know about atoms and bonds already matched in upper levels - the complete structure can be matched by the atoms in the recursive expression. This variant blocks all atoms and bonds already matched in any previous recursion level."?

I'm not even sure I know what that means. ;-)
Seriously though: sounds like one of those "someone had a use for it, once" things.

I can't think of a good example at the moment, but a (very) trivial example would be for e.g. the SMILES CC, then the SMARTS [C][C;$(CC)] would match, whereas [C][C;$$(CC)] would not, because the 1st C atom has already meen matched. I think!
You can probably come up with a use for e.g. looking for substitution patterns on disubstituted aromatics.


Import map diagnostics warning

deno task test says this:
Import map diagnostics:
  - Invalid top-level key "tasks". Only "imports" and "scopes" can be present.
  - Invalid top-level key "compilerOptions". Only "imports" and "scopes" can be present.

looks like importing www/deno.json from deno.jsonc caused this warning:

"importMap": "./www/deno.json",

Yeah, there isn't proper inheritance yet and fresh is kind of twisting our arm here. We're also two minor versions behind on Fresh but I haven't had the time to look into why an upgrade messes everything up.

Ok I'll look into fresh upgrade.
Also I think it might be better to manage import maps separately.

Thanks for looking into it. I think separating the import maps could work, but then we couldn't run deno task www from the root if we didn't specify exactly the same dependencies.

I thought so too, but when I tried it, it worked. It seems that deno.json in the same directory as the main entry point is looked up first.

FYI: Regarding the issue where the design collapses due to fresh upgrade. The problem seems to be that the twind plugin is not loaded in dev.ts immediately after the upgrade. main.ts works fine. marvinh is aware of the problem and I think it will be fixed eventually.


Sockets stopped connecting with SNI-related error (python2.7)

Running the websocket client test code from the readme, I'm getting a SNI-related connection error now. It seems that Coinbase has enabled SNI on their servers and python2.7 clients may not be able to connect anymore. This problem is REALLY hard to troubleshoot for python2.7.6 users! I'm sort of locked to an OS that runs on python2.7 for the moment so I need to get around this somehow.
How can I write a simple tool that will run under python3 here, just connect to a websocket print out the lines to the command-line? Can anyone help?
For reference the error I'm seeing under python2.7.6 now is this:
Traceback (most recent call last):
  File "/usr/lib/python2.7/threading.py", line 810, in __bootstrap_inner
    self.run()
  File "/usr/lib/python2.7/threading.py", line 763, in run
    self.__target(*self.__args, **self.__kwargs)
  File "/home/myusername/.local/lib/python2.7/site-packages/cbpro/websocket_client.py", line 40, in _go
    self._connect()
  File "/home/myusername/.local/lib/python2.7/site-packages/cbpro/websocket_client.py", line 72, in _connect
    self.ws = create_connection(self.url)
  File "/usr/local/lib/python2.7/dist-packages/websocket/_core.py", line 487, in create_connection
    websock.connect(url, **options)
  File "/usr/local/lib/python2.7/dist-packages/websocket/_core.py", line 211, in connect
    options.pop('socket', None))
  File "/usr/local/lib/python2.7/dist-packages/websocket/_http.py", line 77, in connect
    sock = _ssl_socket(sock, options.sslopt, hostname)
  File "/usr/local/lib/python2.7/dist-packages/websocket/_http.py", line 182, in _ssl_socket
    sock = ssl.wrap_socket(sock, **sslopt)
  File "/usr/lib/python2.7/ssl.py", line 487, in wrap_socket
    ciphers=ciphers)
  File "/usr/lib/python2.7/ssl.py", line 243, in __init__
    self.do_handshake()
  File "/usr/lib/python2.7/ssl.py", line 405, in do_handshake
    self._sslobj.do_handshake()
SSLError: [Errno 1] _ssl.c:510: error:14077410:SSL routines:SSL23_GET_SERVER_HELLO:sslv3 alert handshake failure

relevant issue in ws4py implementation of websockets, which I also debugged for ws-feed.pro.coinbase.com websocket connection.


Fix logic error.

if f'wlan{wlanNum}' not in (rcconf and rcconflocal):
Assuming rcconf is not None, only rcconflocal is checked if it contains
wlan{wlanNum}. Concatenate lists with + to check both.
if f'wlans_{card}=' in (rcconf or rcconflocal):
Only rcconf is checked if it's not None. Concatenate lists with + to
check both.

I can pull that in but netcardmgr to automate is deprecated in the later version. This code will be removed soon.

Thank you for merging, anyway :)


Direct Line error on intializing

The Microsoft Bot Framework team prefers that "How To" questions be submitted on Stack Overflow. The official Bot Framework GitHub repository  is the preferred platform for submitting bug fixes and feature requests.
Bot Info

SDK Platform: Node.js
SDK Version: 3
Active Channels: Direct Line
Deployment Environment: Azure Bot Service

Issue Description
Since about one hour ago I started getting error on direct line application
Code Example
var directLineClient = rp(directLineSpecUrl)
    .then(function (spec) {
        // Client
        return new Swagger({
            spec: JSON.parse(spec.trim()),
            usePromise: true
        });
    })
    .then(function (client) {
        // Obtain a token using the Direct Line secret
        // First, add the Direct Line Secret to the client's auth header
        client.clientAuthorizations.add('AuthorizationBotConnector', new Swagger.ApiKeyAuthorization('Authorization', 'Bearer ' + directLineSecret, 'header'));
        return client;
    });

Reproduction Steps

initiate directline script
error on initialization

Expected Behavior
Run normally
Actual Results
2018-08-17T18:48:13.369Z - error: Error initializing DirectLine client StatusCodeError: 404 - "The resource you are looking for has been removed, had its name changed, or is temporarily unavailable."
at new StatusCodeError (/home/people/mbot/node_modules/request-promise-core/lib/errors.js:32:15)
at Request.plumbing.callback (/home/people/mbot/node_modules/request-promise-core/lib/plumbing.js:104:33)
at Request.RP$callback [as _callback] (/home/people/mbot/node_modules/request-promise-core/lib/plumbing.js:46:31)
at Request.self.callback (/home/people/mbot/node_modules/request/request.js:185:22)
at Request.emit (events.js:182:13)
at Request.EventEmitter.emit (domain.js:442:20)
at Request. (/home/people/mbot/node_modules/request/request.js:1157:10)
at Request.emit (events.js:182:13)
at Request.EventEmitter.emit (domain.js:442:20)
at IncomingMessage. (/home/people/mbot/node_modules/request/request.js:1079:12)
at Object.onceWrapper (events.js:275:13)
at IncomingMessage.emit (events.js:187:15)
at IncomingMessage.EventEmitter.emit (domain.js:442:20)
at endReadableNT (_stream_readable.js:1091:14)
at process._tickCallback (internal/process/next_tick.js:174:19)

Hi @10p30pl3
Please share the
-BotHandle
-MicrosoftAppId
-Where does the DirectLineClient code reside? (what data center)

Calling:
https://docs.botframework.com/en-us/restapi/directline3/swagger.json
I'm getting:
The resource you are looking for has been removed, had its name changed, or is temporarily unavailable.
Swagger endpoint has changed for directline V3?

@10p30pl3 Thank you for reporting this.  I've escalated this issue to the docs team, and I'll report back once I hear from them.

@10p30pl3 This has been fixed.

Thank you

The swagger file is again available at: https://docs.botframework.com/en-us/restapi/directline3/swagger.json
What sort of errors are you seeing?

It is initializing ok. But when I try to send any message
        // Send message

        client.Conversations.Conversations_PostActivity(

            {

                conversationId: conversationId,

                activity: {

                    textFormat: 'plain',

                    text: input,

                    type: 'message',

                    from: {

                        id: directLineUserId,

                        name: directLineUserId

                    }

                }

            }).catch(function (err) {

                console.error('Error sending message:', err);

            });

I just get error

Error:
{ url: 'https://directline.botframework.com/v3/directline/conversations/LS7eaamiZAU4IwZ2NzlkM5/activities',
method: 'POST',
headers:
{ 'cache-control': 'no-cache',
pragma: 'no-cache',
'content-length': '116',
'content-type': 'application/json; charset=utf-8',
expires: '-1',
server: 'Microsoft-IIS/10.0',
'x-powered-by': 'ASP.NET',
'strict-transport-security': 'max-age=31536000',
date: 'Fri, 17 Aug 2018 23:11:18 GMT',
connection: 'close' },
errObj:
{ Error: Forbidden
at Request.callback (/home/people/mbot/node_modules/superagent/lib/node/index.js:698:17)
at IncomingMessage. (/home/people/mbot/node_modules/superagent/lib/node/index.js:922:12)
at IncomingMessage.emit (events.js:187:15)
at IncomingMessage.EventEmitter.emit (domain.js:442:20)
at endReadableNT (_stream_readable.js:1091:14)
at process._tickCallback (internal/process/next_tick.js:174:19)
original: null,
response:
Response {
domain: null,
_events: {},
_eventsCount: 0,
_maxListeners: undefined,
res: [IncomingMessage],
request: [Request],
req: [ClientRequest],
links: {},
text: '{\r\n  "error": {\r\n    "code": "BadArgument",\r\n    "message": "Security token not valid for this conversation"\r\n  }\r\n}',
body: [Object],
files: {},
buffered: true,
headers: [Object],
header: [Object],
statusCode: 403,
status: 403,
statusType: 4,
info: false,
ok: false,
redirect: false,
clientError: true,
serverError: false,
error: [Error],
accepted: false,
noContent: false,
badRequest: false,
unauthorized: false,
notAcceptable: false,
forbidden: true,
notFound: false,
charset: 'utf-8',
type: 'application/json',
setEncoding: [Function: bound ],
redirects: [] },
status: 403 },
status: 403,
statusText: '{\r\n  "error": {\r\n    "code": "BadArgument",\r\n    "message": "Security token not valid for this conversation"\r\n  }\r\n}',
obj:
{ error:
{ code: 'BadArgument',
message: 'Security token not valid for this conversation' } },
data: '{\r\n  "error": {\r\n    "code": "BadArgument",\r\n    "message": "Security token not valid for this conversation"\r\n  }\r\n}' }

I am able to get a new token and conversation ID.
But whenever I try to use this conversation ID I get "Security token not valid for this conversation".
I tried to regenerate keys, but no success.

Anyone else with this problem??

https://docs.botframework.com/en-us/restapi/directline3/swagger.json is again redirecting to another url
https://docs.microsoft.com/en-us/azure/bot-service/?view=azure-bot-service-4.0 and bot having an error initializing

Thanks for reporting this.  I'll look into it and get back to you.

The same issue we are facing when we can expect resolution as it is impacting our live application and we have not alternate solution. Either suggest some alternate solution or please do the needful ASAP.

System down again.
Getting error on initialization
06022019-09:49:35:142 - error: Error initializing DirectLine client SECRET.... SyntaxError: Unexpected token < in JSON at position 0
Same problem again??

could you please share the swagger.json file

email?

email?

<EMAIL_ADDRESS>

sent

hey. I also have the same problem. could you please send me a swagger file?
email<EMAIL_ADDRESS>Thank you.

@10p30pl3 thanks, i've put it in a gist: https://gist.github.com/EricDahlvang/ab4197b85dba47212df804d6245a5061 until we can find out where this file is hosted now.
Note: It is probably a good idea to not take an external dependency on a file located in the docs.

The directline swagger file can now be found here: https://github.com/Microsoft/BotBuilder/blob/master/specs/botframework-protocol/directline-3.0.json
The path or filename might change in the future, so please make a copy if you are planning to use it at runtime.

sent

Thanks a lot... your shared file is working perfectly.

Having the same problem.
@EricDahlvang any update on the problem, please?

The latest swagger file is here: https://github.com/Microsoft/BotBuilder/blob/master/specs/botframework-protocol/directline-3.0.json (note: for it to work with swagger-client you will need to set the tokenParameters required to false https://github.com/Microsoft/BotBuilder/pull/5289/files )
Please download and host it locally, rather than retrieving it at runtime.

Thanks, @EricDahlvang


Pressure Sensitive support

Hello. As you may know, all pressure sensitive related controls have been completely stripped out from this port, which makes sense.
In general, you could play MGS3 on PC (on PCSX2 or RPCS3) with pressure sensitive controls working with DsHidMini (https://github.com/nefarius/DsHidMini). One example of it being implemented directly into one emulator is this: https://github.com/RPCS3/rpcs3/blob/master/rpcs3/Input/ds3_pad_handler.cpp
I don't know enough to guess how hard this would be to reimplement, and it could be out of scope for a project like this, but I wanted to ask anyway.
What do you think?

https://www.pcgamingwiki.com/wiki/Controller:DualShock_3#reWASD
reWASD already lets you use pressure sensitive controls with ANY game.

https://www.pcgamingwiki.com/wiki/Controller:DualShock_3#reWASD
reWASD already lets you use pressure sensitive controls with ANY game.

Some stuff wouldn't be doable with only reWASD though. For example, the behavior while aiming has been changed in the 360 version so that you keep aiming even when you're not pressing anything.

Self plug, but I had something worked up for the '03 PC release, could be adapted to work in MC possibly? None of the addresses are going to be the same, so you'll have to do some digging to find where the virtual pad is mapped at (preferably with SigScanning for any patches that roll out), presuming they still use a virtual pad for input handling.
https://github.com/UltimateNova1203/mgs2_input/blob/main/Input-Map.md

Awesome. I'm praying for a way to re-enable the pressure sensitive R1 codec lines.


[WIP] Descriptive statistics

Hackday project in need of feedback.
Working with data you're interested in:

Shape of data, which could be your schema
General case, which is addressed by bigSampler
Edge cases, which this PR tries to tackle

It's inspired by summary from R.
Todo:

[ ] Property and unit tests
[ ] Support booleans
[ ] Support floating point numbers
[ ] Support for different formats (protobuf)?

Hey, thanks for taking the initiative! We have some internal stuff that overlaps a bit. I've been thinking a lot about the future of that and might be good to make sure we're on the same page.

@idreeskhan may I ask, were data profiling tools opensourced since then?

Sorry this comment got lost in email while I was on vacation back in June. They have not been open sourced but we are hesitant to merge this in. Internally the data profiling tools fit our needs and if we merge this it means we are taking over maintenance which we don't really want to do at the moment.


不能自动判断编码是 UTF8 或 GB2312

VSCode Version:1.18.1
OS Version:Win10

Steps to Reproduce:
1.打开编码为GB2312的html文件
2.看到中文乱码

是啊，我也有这个问题

Please file this issue in english with reproducable steps so we can react on it

this soft is not support to analyze file encoding format by itself. such as: gb2312.

VS Code supports a wide range of file encodings. You can see the full list when you click on the lower right status bar where the encoding is shown and then choose to either reopen the file with a specific encoding or save the file with a specific encoding:

There are two options how to configure the encoding for files
Configure the encoding as a setting for all files or files of specific type
We provide a files.encoding setting that you can use to set an encoding for all files. You can also configure this encoding for specific file types, e.g. like this:
"[markdown]": {
    "files.encoding": "utf8"
}

Configure VS Code to automatically detect the encoding when opening files
We provide a files.autoGuessEncoding setting that you can enable to let VS Code detect the encoding of every file automatically. Note that this approach is known to not be accurate in all cases. This guessing of encoding is rather a heuristic that can produce wrong results (that is also why this setting is not enabled by default).
Please also check out issues that are already reported with the files-encoding label: https://github.com/Microsoft/vscode/issues?q=is%3Aopen+is%3Aissue+label%3Afile-encoding


Old ownership panel was better

The new ownership panel feels clunkier and less intuitive than the old one. Can we do a bit of design work to re-evaluate it?
attn: @jtomeck

Closed by #942


8184.1 DEV show dashboard page

https://github.com/Shippable/pm/issues/8184
once the login is complete, show the page that lists all system configs
mockup if this is available in corresponding heap issue
https://github.com/Shippable/admiral/issues/9 should be completed before showing this page

Update from Devashish:

add dashboard view. Should be an angular app

dashboard loads post login and calls GET /systemConfigs route to get system configs
it shows the database config from the env variables

Front page on admin UI should show db information and system configs. Details in Mockup section of https://github.com/Shippable/heap/issues/659


Autoware parks inappropriately on parking lot

Checklist

[X] I've read the contribution guidelines.
[X] I've searched other issues and no duplicate issues were found.
[X] I'm convinced that this is not my fault but a bug.

Description
In this example, I have set the goal point perpendicular to the initial position (in red), so a maneuver is required. Autoware starts its maneuver but kind of stops halfway, as it considers the goal has been reached as soon as the XY matches:

I guess this is because the goal heading error tolerance is too high. For lane driving, it is not a problem as we can't maneuver, but when the vehicle is on a parking lot, Autoware should park precisely.
I guess this should be fixed for cargo ODD.
Expected behavior
Autoware parks precisely on parking lot
Actual behavior
heading error is too big on goal.
Steps to reproduce
Run simulator with a parking lot and ask autoware to park somewhere.
Versions
No response
Possible causes
No response
Additional context
No response

Thanks for the report.
The parking planner has not been changed for some time since it was created as a prototype.
The free space planner doesn't generate stable trajectory, so when it stops at a certain place, we create the lanelet.
It may be better to review the design from the beginning if necessary this time with cargo ODD or bus ODD.

@VRichardJP Which planning algorithm did you choose? A-star or RRT?

@NorahXiong A* in this example

I did a similar test with the default settings and I got a pretty good result.
@VRichardJP Have you repeated the test? Did it stop halfway everytime?

I did a similar test with the default settings and I got a pretty good result.
@VRichardJP Have you repeated the test? Did it stop halfway everytime?

In my case, I added a few obstacles to make the parking maneuver less trivial. You can see the 2 buses left and right of the parking spot in my screenshot. I only tested a couple times with this setup.

I am using the same map you are showing.
Starting position:

After struggling with the maneuver, going back and forth many times just a few meters, autoware drove very quickly close to the target goal and got stuck:

Another again. Start position:

After goal is reached:

As @yukkysaito said, it seems that Autoware considered the goal has been reached if it is stopped around the goal, even if the maneuver is not finished. This occurs typically when Autoware moves back and forth around the goal to adjust its orientation

Hi, @VRichardJP
I updated my environment and reproduced your issue. I reduced the "vehicle_shape_margin_m" value to 0.2(m) and the parking procedure succeeded.
I checked the "vehicle_shape_margin_m" parameter changelog and I found no one modifying the default value recently. The problem may come from the perception module. Maybe algorithms and logic changed in the perception module and it results in unstable/incorrect distance measurement.
Anyway, hope my information could help you!


ENOENT: no such file or directory, scandir

Trying to run zzAPI on a Windows machine, by keeping the zzb file in the E: directory. Getting the below error while running the request.
A system error occurred (ENOENT: no such file or directory, scandir '/e:/Tools/zzAPI')

This was a bug in a previous version. Please check your version that is installed. The bug is fixed in v0.10.6.

This is resolved in the latest version. Thank you for the prompt response.


Adding currency csv test

Note that MSVC build is broken due to an MSVC bug in 17.12.1 that I reported and was fixed last night, but has not been released yet.


Rehash passwords when needed

Fixes https://github.com/aspnet/Identity/issues/17
cc @divega @GrabYourPitchforks @blowdart @suhasj

Merged 69ac9abcc5bdd831026bb31983b9e4f2979915e5


How to allow search engine indexing on web public streams

Hey!
While browsing a web-public stream, I've noticed that <meta name="robots" content="noindex,nofollow"> is defined. As a result, I don't think our web-public streams are indexed by search engines.
Would it be possible to provide an option or to enable it on web-public streams?
Example: https://asciidoc.zulipchat.com/#narrow/stream/335216-announce-.F0.9F.93.A2
Thanks!

Closing as a duplicate of #21881.

Ooops sorry, I searched for "robots" and "noindex,nofollow" but found no result 😬

It looks like Alya's close-as-duplicate didn't go through; trying now.


Hugo's JS minification removes legal comments

Describe the bug
Hugo's minify function does not preserve legal comments. The medium-zoom license notice /*! medium-zoom 1.0.6 | MIT License | https://github.com/francoischalifour/medium-zoom */ isn't present in the minified script, which is probably a license violation.
To Reproduce
Steps to reproduce the behavior:
Inspect the minified main JS bundle on a production build to see that the medium-zoom license notice is missing.
Expected behavior
Comments beginning with /*! should be preserved during minification.
Technical details:

Link to your GitHub project: https://github.com/CVMuAlphaTheta/cvhsmath.org
Wowchemy Version (from your go.mod): v0.0.0-20210526223849-81ba17522966
Hugo Version (run hugo version): 0.83.1
Browser/OS: Firefox/Fedora
Wowchemy Template (e.g. Academic): Research Group

The minify option for js.Build minifies the code without removing legal comments, but esbuild moves the comments to the end of the file by default and Hugo doesn't seem to provide an option to change that.

This isn't the Hugo repo...


Unprotected Access in All()

https://github.com/jftuga/TtlMap/blob/1e9219bd31794f520e2f2138ffa13f20a6e1898d/ttlMap.go#L118-L120
The All() method returns a direct reference to the internal map (m.m).
Any caller can then modify the map without acquiring the mutex, leading to potential race conditions.
Returning a deep map copy instead of a direct reference could be a solution.

It seems you are right. Copied map can be returned by using locks. I think using RLock and RUnlock will be more suitable.

Could you please review this commit to see if this fixes the problem?
Thanks.


When will mml3 be fully supported?

Documentation(http://docs.mathjax.org/en/latest/options/MML3.html?highlight=mml3) states it is experimental support only. https://github.com/mathjax/MathJax-docs/issues/78
It would be great if some information is provided on this.

It is in our backlog but not scheduled to be worked on any time soon.

For completeness: the slow progress in elementary math support is primarily an issue of resources  (i.e., developer time).


DocumentSelector instead of scheme(s) for registerTimelineProvider

registerTimelineProvider takes as its enablement clause a string or array of strings for the schemes to which it applies. It would be nice if this were instead the DocumentSelector that we use in other providers, as that offers more flexibility (search editors for instance are defined by language rather than scheme: { language: 'search-result' } .

@JacksonKearl @jrieken and I talked about using a DocumentSelector vs scheme and we felt that scheme was more appropriate here, because a TimelineProvider is Uri-based, not document-based. For example, the Uri could be a folder or workspace.

Yeah, I still think that scheme-only is correct because the timeline is per resource. Maybe, simplify the register-function to only accept a single string (like other scheme-only providers) so that this doesn't look like a doc-selector.

Without allowing multiple schemes, it will make it more complicated to use. For example, the Git provider registers file, git, and gitlens-git right now. Although, I could go back to using * instead.

Yeah - we could allow * but calling register three times or in a loop shouldn't be too much a hassle

Even though it is non-standard, I really like being able to target multiple schemes with 1 provider registration, having multiple registrations seems like needless ceremony imo.
See the usage here:
https://github.com/microsoft/vscode/blob/28dbe0755ec2a103ea500f4c40d7229c435720a6/extensions/git/src/timelineProvider.ts#L77


what can i do if i want to add this datapack into my 1.20.1 world

my world is in 1.20.1 version, forge. and i have a mod witch haven't update to any higher version yet. plz tell me what can i do

That is not possible. The data pack relies on the experimental flag, which wasn't available in 1.20.1. If you want to use the 1.21 blocks in 1.20.1, you'll have to look for an actual mod that backports the blocks.


WordEmbedding Why train.step with batchsize 1

Description
In scripts/word_embeddings/train_glove.py:319:             trainer.step(batch_size=1)
and scripts/word_embeddings/train_sg_cbow.py:240 trainer.step(batch_size=1)
why the batch_size set to 1?  why not average the loss first? or batch_size set to the real batchsize?
when the batch size is large, it would cause the loss large too.
Thank you for help!

The batch_size in step() affects gradient rescaling. As an adaptive optimzer is used (actually blockwise adaptive, see https://arxiv.org/pdf/1905.09899.pdf), the rescaling isn't essential. The gradient is already normalized based on the history of gradients.
https://github.com/apache/incubator-mxnet/blob/b914d0a6296b5773499a1232dd8d4a3851b6a842/src/operator/contrib/optimizer_op-inl.h#L120-L136
Setting the batch_size to the actually used batch_size seems not to help much: the gradient is sparse and many of the non-zero rows only depend on a small number of elements in the batch. Thus rescaling by batch_size is not correct for them (though to some degree this is counteracted by the adaptivity of the optimizer).
If you like, you can change the batch_size parameter and tune the learning rate. If the results match, let's update the code. But if I remember correctly, I got better results with batch_size=1.
I'm closing the issue for now, but please feel free to reopen if you like to discuss this further or have obtained good results with the suggested code change.


Use only the data required for builds in jobs

Somehow this was lost during the switch to ActiveJob.

:+1:


Adding "no doctor found" when there is no doctor similar to search name.

Describe the solution you'd like
Adding "no doctor found" when there is no doctor similar to search name.
when the searched doctor is not in the database then i will show "no doctor found"
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
please assign this to me under gssoc-ext

@ratnesh-maurya I want to contribute to this project on this particular issue. Please assign it to me!

sure go ahead @Hamza1821


Added 1.9 enchantments

Added 2 new enchantments from 1.9: Mending and Frost Walker
Added 'Rarity' enum instead of the 'weight' magic value
Added 'treasure' property to enchantments

Thank you for your contribution!


Gutter Marker - Multiline Support

Hi, great work on CodeMirror! I'm currently integrating it into Gitblit (https://github.com/gitblit/gitblit) to support in-browser editing of repository files.
The code sample on gutter notes (https://codemirror.net/doc/upgrade_v3.html#gutters) gave me a thought that this would be a useful way to implement and present code review and gitblame pages.  However, it would require line ranges to be defined and it looks as though setGutterMarker only accepts a single line.
Would a PR be accepted for allowing multiline gutter markers in a similar way to the markText functions where a from-to range can be provided?  Or can you recommend a way to do this already using the markText functionality?

What would multiline gutter markers do, exactly?

@marijnh setting a gutter marker across a range of lines along with an alignment option for placing the passed element either to each line, or align within the group i.e top, middle, bottom line.
cm.setGutterMarker(from: {line}, to: {line}, gutterID: string, value: Element, align: string (each|top|middle|bottom) )  → LineHandle[]
The lineInfo function would then include all gutters which are defined on or that cross over the given line.
cm.lineInfo(line: integer|LineHandle) → object
For example building upon the v3 gutter sample, a note relevant to lines 1 through 5 could be placed with align:middle, so only one note is shown.  However, if lineinfo is called on any line between 1 and 5 it will return the associated note.
In-addition multiple markers could be placed on the same line in the same gutter, allowing support of multiline markers whose ranges cross.  Currently it appears only one item per line and gutter type pair is supported.
Perhaps this is an application specific use case better handled outside the core codemirror source.  What do you reccommend?

Nope, that doesn't really fit the current model, where lines (including their gutter content) are rendered separately and independently, and doesn't seem that much of a win.


Uniform the usages of gradients

[ ] Rename y -> energy and dy -> gradient
[ ] Update the loaders and trainer

Fix #116

Superseded by https://github.com/torchmd/torchmd-net/pull/118


BUG: stable docs site fails to load JQuery

Describe your issue.
https://docs.scipy.org/doc/scipy/ (currently showing version 1.12.0) shows "failing to find JQuery" errors in the browser console. One consequence of this is that the version switcher doesn't work.
History
Following similar decisions in Bootstrap v5 and Sphinx v6, the pydata sphinx theme dropped JQuery inclusion in version 0.13.0rc1 (2023-02-03).
Status
The site footer says the v0.12.0 docs were built with Sphinx 7.2.6 (so sphinx won't be providing Jquery), but it looks like they're still being built with v0.9.x of the theme (which ought to still provide it) so I'm not 100% clear what went wrong.  I also haven't investigated the SciPy docs folder carefully yet.
FWIW, the theme offers some transition guidance here: https://pydata-sphinx-theme.readthedocs.io/en/stable/community/topics/bootstrap.html#dropping-jquery
cc @12rambau who might have ideas.
Reproducing Code Example
go to https://docs.scipy.org/doc/scipy/ and open the browser console (`F12` on Firefox)

Error message
Uncaught ReferenceError: $ is not defined
    <anonymous> https://docs.scipy.org/doc/scipy/:183
    <anonymous> https://docs.scipy.org/doc/scipy/:221
[scipy:183:5](https://docs.scipy.org/doc/scipy/)
Uncaught ReferenceError: jQuery is not defined
    <anonymous> https://docs.scipy.org/doc/scipy/_static/scripts/pydata-sphinx-theme.js?digest=92025949c220c2e29695:1
    n https://docs.scipy.org/doc/scipy/_static/scripts/pydata-sphinx-theme.js?digest=92025949c220c2e29695:1
    <anonymous> https://docs.scipy.org/doc/scipy/_static/scripts/pydata-sphinx-theme.js?digest=92025949c220c2e29695:32
    n https://docs.scipy.org/doc/scipy/_static/scripts/pydata-sphinx-theme.js?digest=92025949c220c2e29695:1
    <anonymous> https://docs.scipy.org/doc/scipy/_static/scripts/pydata-sphinx-theme.js?digest=92025949c220c2e29695:26
    n https://docs.scipy.org/doc/scipy/_static/scripts/pydata-sphinx-theme.js?digest=92025949c220c2e29695:1
    <anonymous> https://docs.scipy.org/doc/scipy/_static/scripts/pydata-sphinx-theme.js?digest=92025949c220c2e29695:1
    <anonymous> https://docs.scipy.org/doc/scipy/_static/scripts/pydata-sphinx-theme.js?digest=92025949c220c2e29695:1
[pydata-sphinx-theme.js:1:927](https://docs.scipy.org/doc/scipy/_static/scripts/pydata-sphinx-theme.js?digest=92025949c220c2e29695)
Uncaught ReferenceError: $ is not defined
    <anonymous> https://docs.scipy.org/doc/_static/versionwarning.js:13

SciPy/NumPy/Python version and system information
n/a, this issue is about the website.

Thanks @drammock, we actually have started a discussion regarding that JS 😅 #19961 TLDR is that I would like to just remove this and this specific issue would go away doing so.

I think we're hoping that this will have been fixed by upgrading the theme and using the version switcher from there. So should work for 1.12.1 / 1.13.0 ?

@lucascolley what do you mean? JQuery is being dropped from the theme. The code which calls some JSQuery is on our side.

Yeah makes sense, I was just going off https://github.com/scipy/scipy/issues/18031#issuecomment-1771789663 which suggested that upgrading the theme was needed to fix the version switcher, but sounds like we need to remove the JSQuery on top of what's already been merged.

The code which calls some JSQuery is on our side.

Not entirely, because those docs were still being built with theme v0.9.x, so some of the calls to JQuery are in theme code (you can see that in the errors)
honestly the more I think about this it's probably the theme's fault (or at least, we could have prevented this). When we learned that Sphinx 6 would remove JQuery support, we should have added a pin to keep sphinx at or below v5 (and then later removed that pin once our own code had purged JQuery usage).
Regardless, I think it will go away once #16660  is backported, so maybe nothing needs to be done.

ah right I was looking at the last message only 👍

in case you want to add jquery back in the build you can also have a look to https://github.com/sphinx-contrib/jquery
This extention have been created when JQuery was dropped from Sphinx and allows you to postpone searching where the requirement is coming from.

FWIW, the version switcher is still broken for me on https://docs.scipy.org/doc/scipy/ on firefox. When I look at the dev tools, I see this where JQuery should be used
Uncaught ReferenceError: $ is not defined

I know that we are hoping that the version switcher will be fixed when 1.13.0 is released, now that we have upgraded to a newer version of the theme.

I said this a few times, we should consider dissociating the doc release process from the release of the library itself. (And doing so move away from the Enthought server...). The current process is really making our lives complicated for nothing and also adds to the release process complexity. The doc team should have the agency to update the doc without a full blown release.

Agreed, it would be nice to be able to see if the fix has actually worked before the release goes out and we have to wait until the next release to fix things.

Also we are constraining ourselves a lot with the CI credits etc. Like the issue we had for the last year not being able to update. That was just a timeout in the end. So if I would have had access to the doc infra, I would just have uploaded the new version. We don't have to match the repo state 100% for the infra part.


remove fields: style xml spec, prereq for #89

in my naive days, i thought treating the xml entities like records would be best. how wrong i was; need to treat them as a stream. order matters. so let's abandon the record format completely.


Root access for joining a cluster

Running lxd init interactively to join an existing cluster requires root access.
However, when using a preseed file for initialization and running cat <preseed-file> | lxd init --preseed, root access is not required.
Originally, running lxd init needed root access to introspect the available storage engines, but this changed a while ago.
So maybe root access isn't needed for lxd init anymore, and the requirement should be removed when running lxd init interactively?

Basically when you use the preseed, you feed LXD with all the data needed in one go, including data that would normally require root access to get.
When not using the preseed,lxd init will open server.crt and server.key in order to connect to the prospective target cluster (during a cluster join operation). This cannot be done by non-root users.

Thanks for confirming @stgraber


Custom .pot generation script

Adds a custom script to convert our en.json to a .pot file. This gives us more control than using a library, for example I'm adding a "(Do not translate words in curly braces)" message to strings with variables inside.
Please review the script, the output, and the gettext standard:
white-space
#  translator-comments
#. extracted-comments
#: reference…
#, flag…
#| msgctxt previous-context
#| msgid previous-untranslated-string
msgctxt context
msgid untranslated-string
msgstr translated-string

our example output:
# hero.caption.content (Do not translate words in curly braces)
msgctxt "hero.caption.content"
msgid "Put a word or phrase inside quotes. For example, {link}."
msgstr ""

todos

[ ] make better
[x] #152
Make placeholders compatible with GlotPress, and escape double quotes (")
[ ] #153
move to somewhere that makes sense in the project structure
decide where to store the .pot file
[ ] convert pluralized strings to the format that is acceptable in GlotPress (see the comment below)
[x] #159
It would be amazing to able to add the 'reference' to the file in a gettext reference comment (example: #: https://github.com/WordPress/openverse-frontend/blob/main/src/components/FooterSection.vue#L23). I think that would be incredibly helpful to translators, and basically for each line we'd just need to grep for the full.i18n.string and find the component + line number for it to build the GitHub URL.
[ ] Add metadata to the top of the .pot file, maybe?
[x] #158
[ ] Pre-commit hook + PR check to see if en.json is dirty, and if so, generate a new .pot file

Translate.WordPress.org includes some warnings and checks that might be relevant here.

for example I'm adding a "(Do not translate words in curly braces)" message to strings with variables inside.

GlotPress already has warnings for %s and other printf-style friends, but on WordPress.org we've also got one for ###NAMEHERE### which could be extended for this use-case if wanted, or the string could even be mangled to/from that format I guess.. but let's not do that..
https://github.com/WordPress/wordpress.org/blob/master/wordpress.org/public_html/wp-content/plugins/wporg-gp-custom-warnings/wporg-gp-custom-warnings.php#L230-L240
Eg, we could alter that to something like this, which I'm happy to facilitate here:
if ( 'meta/openverse' === $project_slug ) {
    $placeholder_regex = '@(\{[a-zA-Z-]+\})@';
} else {
   $placeholder_regex = '@(###[A-Z_]+###)@';
}

One of the reasons I'm suggesting this, is because we will 100% have translations being submitted that do not respect the placeholders, either from automated translation tools that some translators use, or just from some missing it at first. Adding warnings to catch these makes the translation experience better for all due to less mistakes.

header.notification.text

This string is purely placeholders, that probably shouldn't be a translated string? I guess it's probably because the translate & placeholder functionality is one and the same though?
It's also present with two different source strings?

photo-details.reuse.credit.licensed

This string contains no placeholders, but appears to be used between them {asset} is licensed under {license} would be better for translation purposes.

photo-details.reuse.credit

In fact, that entire section appears to be awkward to translate as it appears there's probably a bunch of placeholders missing and short sharp phrases that need to be translated with the context of the others around it.

browse-page.audio-result-count
"no audio results|{localeCount} audio result|{localeCount} audio results"

This is an awkward string for GlotPress. In GlotPress it should really be two strings using a :

a singular string: no audio results
a pluralised string: {localeCount} audio result & {localeCount} audio results
eg:

# browse-page.audio-result-count
msgctxt "browse-page.audio-result-count"
msgid "no audio results"
msgstr ""

# browse-page.audio-result-count
msgctxt "browse-page.audio-result-count"
msgid "{localeCount} audio result"
msgid_plural "{localeCount} audio results"
msgstr[0] ""
msgstr[1] ""

The photo-details.reuse.credit.text string is complicated, and I think we will have to add a message to translators in the POT file manually. The string contains only placeholders, so there is no need to translate anything, but it is important to rearrange the placeholders in accordance with the language's grammar rules:
English
    1        2           3             4         5
"{title}{creator}{marked-licensed}{license}{view-legal}"
Mars by Olga is licensed under a Creative Commons Attribution 4.0 International License. View legal terms.

Turkish
     2       1         4           3             5
"{creator}{title}{license}{marked-licensed}{view-legal}"
Olga isimli yazarın Mars başlıklı eseri bu Creative Commons Atıf 4.0 Uluslararası Lisansı ile lisanslanmıştır. Yasal bilgilerine bak.

I suggest we add a 'translator comment' with # Please, rearrange the curly braced names to match the sentence structure in your language: '<WorkTitle>< by Creator >< is licensed under a ><Creative Commons Attribution 4.0 International License.>< View legal terms.>' Do not translate the words in curly brackets!
I'm not sure if it's clear enough, any suggestions are welcome!

Thank you for your detailed analysis, @dd32 !
I have addressed the placeholder replacement in #152.
Next on my list is updates to the singular and pluralized strings. Thankfully, we only have one of them :)
I have a question about pluralization in GlotPress, and hope you know the answer or someone who knows the answer, @dd32  :) Are pluralization rules added to PO files automatically, or do we need to add them?
What I mean is this line at the top of a Russian PO file that helps to select the correct plural form:
"Plural-Forms: nplurals=3; plural=((((n%10)==1)&&((n%100)!=11))?(0):(((((n%10)>=2)&&((n%10)<=4))&&(((n%100)<10)||((n%100)>=20)))?(1):2));\n"

Are pluralization rules added to PO files automatically, or do we need to add them?

The Plural-Forms header will be added automatically, based on the nplurals and plural_expression fields present in https://github.com/GlotPress/GlotPress-WP/blob/develop/locales/locales.php
I'm unsure if the plural form will be the format that you need though, as it's a gettext plural form, and not a CDIR (as mobile apps need)... but based on your example here.. I think it's close enough that it'll work for you.
You can export any / inspect any WordPress.org .po/.mo fileset to see what it would generate, all GlotPress generated files will be the same :)
I've imported the test.pot file from this PR into https://translate.wordpress.org/projects/meta/openverse/ as a basis for translations, some locales noticed and translated some.. so you've got some head start there and real-life strings to work with. Just enter a language from that page, then hit the Export link at the bottom

The photo-details.reuse.credit.text string is complicated, and I think we will have to add a message to translators in the POT file manually.
    1        2           3             4         5
"{title}{creator}{marked-licensed}{license}{view-legal}"
Mars by Olga is licensed under a Creative Commons Attribution 4.0 International License. View legal terms.

That string being made up of the other smaller string chunks is harder to translate, as the "By Olga" text might be used in multiple contexts/etc.
A string of {title} by {creator} is licences under a {license-name}. View legal terms. is how most of the WordPress translators would be more used to it. It does end up with multiple overlapping strings, as you'd probably also now have a By {creator} string by itself, but as they're used in different contexts it provides a more context to each string.
Keeping it like this is "fine", there just needs to be a comment to explain the full string to the translator, so that they can properly re-organise it without having to go looking at the code to find what text is contained within each placeholder.
(Please note: I speak only one language - English; All of my experience with translations comes from bending to the will of the WordPress translation teams and making things work, I flag things early but they're not always something that must change, just be noted.)

Thank you for all of your amazing comments and suggestions, @dd32 , and for uploading the locale files!

A string of {title} by {creator} is licenced under a {license-name} License. View legal terms. is how most of the WordPress translators would be more used to it. It does end up with multiple overlapping strings, as you'd probably also now have a By {creator} string by itself, but as they're used in different contexts it provides a more context to each string.
Keeping it like this is "fine", there just needs to be a comment to explain the full string to the translator, so that they can properly re-organise it without having to go looking at the code to find what text is contained within each placeholder.

We would have to look into the Vue-i18n implementation to see if it is possible to organize the sentence the usual WP way. If not, we would have to add a comment-explanation for the translators.

@dhruvkb and @obulat as per the changes in #176 I would recommend po-files/openverse.pot for the generated filename, does that sound okay?

Okay @obulat and @dhruvkb file renamings are complete so this is ready for final review 😄

Howdy @dd32, this i18n PR for Openverse is merged! Here's some info for ya:

Final .pot path is https://github.com/WordPress/openverse-frontend/blob/main/src/locales/po-files/openverse.pot
Could you make sure Openverse is tagged as "active" here: https://translate.wordpress.org/projects/meta ? There was some confusion in #polyglots about this
And remove the "(TRANSLATORS: Please note that the strings here are not finalised, and is being worked on in this PR. This message will be removed once the strings are being auto-imported.)" comment in the Openverse translation project description?

Thank you SO MUCH for the guidance and help here—it made a substantial impact on the Openverse project for the better.

Final .pot path is https://github.com/WordPress/openverse-frontend/blob/main/src/locales/po-files/openverse.pot

Updated!

Could you make sure Openverse is tagged as "active" here: https://translate.wordpress.org/projects/meta ? There was some confusion in #polyglots about this
And remove the "(TRANSLATORS: Please note that the strings here are not finalised, and is being worked on in this PR. This message will be removed once the strings are being auto-imported.)" comment in the Openverse translation project description?

Updated!

and enable all available Rosetta sites for Openverse?

Added! However, I think there's some work that still needs to be done to pass the locale through to the Openverse embed?
For example, https://nl.wordpress.org/openverse/ works but the embed is the same.
It looks like we just need to pass the locale of the site through wp-content/themes/pub/wporg-openverse/js/iframe_nav.js?
The WordPress locale is available through the get_locale() PHP function, and I can make the JS respect that, if someone can tell me the query param / path / etc that's needed? (I looked at the Nuxt config, but couldn't immediately tell)


QR Code Reading Problems with Android Wallet App

A community member using the Android mobile wallet (Android version: 8.0 Galaxy Note 8 Qtum wallet version 0.12(113)) reported problems reading the address QR code on the Upbit exchange. Here is a QR code from the Upbit exchange:

Which gives this error “Invalid QR Code” on the Android wallet app:

The Android wallet correctly reads the address QR code on the Explorer (explorer.qtum.org).

As a reference, the Qbao wallet also produces and error for the Upbit QR code, with the message “Please scan the correct QR code” and correctly reads the Explorer QR code and the QR image for qtum-qt – Receive – RECEIVE A PAYMENT.

Using a generic QR code scanner (https://itunes.apple.com/us/app/free-qr-code-reader-barcode-scanner-for-iphone/id903799541?mt=8) , for the Upbit image, the scanner reads the Q address only, for the Explorer image the scanner reads “qtum:QSyiND…?label=QTUM Mobile Wallet” where QSyiND… is the full Q address, and the scanner cannot read the QR image from qtum-qt.

Please post this in the android app repo.


Flavors not picked up correctly

Problem: The plugin builds APKs for the correct flavor, but tries to upload the wrong flavor.
I think the issue lies within FirebaseTestLabPlugin.kt:527, but I might be wrong.
How to reproduce:

Define 2 flavors, called first and second.
run firebaseTestLabExecuteSecondDebugInstrumentation - you should get an error saying that the APK in path /outputs/apk/first/debug/*.apk was not found.

README.md mentions something about testBuildType selection, but it's not present in the plugin.
Let me know if you need more input.

@AndroideRob there indeed was a problem with the line you mentioned. It's fixed in unreleased version 2.6.0 and you should be able to test it by loading the latest version of the plugin from Jitpack instead of the released version.

@zbynek @AndroideRob Gradle Plugin portal just approved. https://plugins.gradle.org/plugin/firebase.test.lab

@AndroideRob Can you please let me know if the issue is solved?

Thanks guys, the issue is solved.


Add mewz

Pre-submission checklist:
Please check each of these after submitting your pull request:

[x] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[x] Is your project closed source or, if it is open source, does your project have at least 300 GitHub stars?
[x] Have you picked the single best (existing) category for your project?
[x] Does it follow the other guidelines from the new entries section?
[x] Have you added your SVG to hosted_logos and referenced it there?
[x] Does your logo clearly state the name of the project/product and follow the other guidelines?
[x] Does your project/product name match the text on the logo?
[x] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?

@alabulei1 can you make this in A-Z order? otherwise looks OK

Fixed. Thanks @caniszczyk


Docker blocked in 0x3

Describe the bug
I have an Assetto Corsa Manager server on my Portainer, and when I start it, I get stuck in a reconfiguration state.
To Reproduce
Steps to reproduce the behavior:
4. See error
[ 98%] Downloading update (59523 of 59523 KB)...
[100%] Download complete.
Steam Console Client (c) Valve Corporation
-- type 'quit' to exit --
Loading Steam API...OK
"@sSteamCmdForcePlatformType" = "windows"
Logging in user 'USER' to Steam Public...[  0%] Checking for available updates...
OK
Waiting for client config...OK
Waiting for user info...OK
Update state (0x3) reconfiguring, progress: 0.00 (0 / 0)
Update state (0x3) reconfiguring, progress: 0.00 (0 / 0)
Update state (0x3) reconfiguring, progress: 0.00 (0 / 0)
Update state (0x3) reconfiguring, progress: 0.00 (0 / 0)
Error! App '302550' state is 0x2 after update job.
Work thread 'CJobMgr::m_WorkThreadPool:1' is marked exited, but we could not immediately join prior to deleting -- proceeding without join
Desktop (please complete the following information):

OS: seejy/assetto-server-manager

Fixed by creating my own docker https://github.com/SylvainSorlin/Docker-AC-Server


Enable WIFI_LoRa_32_V3 macro

The LoRaWAN library needs this one.

This seems wrong, pin_arduino.h already defines this.


Fix image build from golden

When the image is compressed, the imagePath should not be the config image path, as the target/destination of the compression should be different.

:white_check_mark: Build windows-openstack-imaging-tools 1.0.187 completed (commit https://github.com/cloudbase/windows-openstack-imaging-tools/commit/c137157474 by @ader1990)


Feature Request: Stream transactions for small number of accounts

What problem does your feature solve?
The transaction streaming functionality is very useful, but only allows for streaming all transactions for all accounts, or transactions for a single account. This makes it difficult to stream transactions for a small number of accounts that together make up a contract, such as a payment channel, and still retain some guaranteed ordering.
What would you like to see?
I'd love to see the ability to include a list of accounts, up to some small limit such as 2 or 4 accounts, with the streaming functions, specially the transaction streaming endpoint.
What alternatives are there?
@bartekn suggested merging on the client side and ordering, using a delay period to try and guarantee that all transactions are seen, although he noted that this is not 100% reliable as there's no guarantee we'll receive transactions for one account in time to be interleaved with the other account.

Would something like:
GET /transactions?account_ids=GA...,GB...,GC....
Accept: text/event-stream

be the expected API?