Fix #6 Recovery function

Change:

[x] signature recovery param #6
[ ] signature recovery function #6
[ ] more tests #6
[ ] specify package version

Codecov Report

Merging #7 into master will increase coverage by 2.07%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master       #7      +/-   ##
============================================
+ Coverage      80.8%   82.88%   +2.07%
- Complexity       35       38       +3
============================================
  Files             5        5
  Lines            99      111      +12
============================================
+ Hits             80       92      +12
  Misses           19       19

Impacted Files
Coverage Δ
Complexity Δ

src/Signature/Signer.php
100% <100%> (ø)
10 <0> (+2)
:arrow_up:

src/Signature/Signature.php
100% <100%> (ø)
3 <2> (+1)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f63f4e8...84dbd0b. Read the comment docs.


fix(DataTable): datable controlled sorting #624

You can now provide a click handler on DataTableHeadCell without passing onSortedChange function to manage sort icon of the DataTable fixes #624 and adds some a storybook file as well as some basic examples for the DataTable

Awesome, looks great!


InvalidProgramException; "Common Language Runtime detected an invalid program."

Running my project in Release mode with Tracer.Serilog.Fody weaved in causes this exception on startup. When I remove the weaver from my FodyWeavers.xml or switch to Debug mode, the app continues running just fine.

Could you please run PEVerify on the release mode assembly?

Getting a bunch of The system cannot find the file specified. errors whenever I run PEVerify on the assembly, both with Tracer and without. Don't have a lot of experience using this tool, what specifically are you looking for?

The reason for the invalid program exception is that Tracer messes up the IL code when adding trace functionality. PEVerify tool can check if the IL code and the metadata in the assembly are correct and can point out any problem. PEVerify can be run from developer's command prompt if you're using Visual Studio. I need to figure out what construct in your code causes this problem.

Not finding a lot on PEVerify, so just don't know what is causing all of the The system cannot find the file specified. errors when I run from command prompt. I am getting this both without Tracer weaved in and with Tracer weaved in.

Another option would be to add VerifyAssembly="true" to the Weavers element in the FodyWeavers.xml. It should run this PE verification after weaving and output the problem.

Added that and built, and now have a lot of Unable to resolve token.(Error: 0x80131869) messages. Using ILSpy, I'm able to go in and see those methods and constructors and ensure they exist though.

Should PEVerify, and Tracer.Serilog for that matter, work for UWP projects?

Theoretically yes, but I've never tried it. Are you using some special language construct where this error appears? Could you give some minimal reproduction source code so I can debug into it? Anyway I'll create a small UWP project and check if it can be rewritten with tracer.

No special language construct that I'm aware of. I'll see if I can put something together, but won't have a lot of time this week. Were you able to check Tracer with a small UWP project?

I created a minimal UWP (just some buttons on a screen) and it works both in debug and release mode. One difference is that I used NLog, not Serilog. I'll check if it makes any difference.


REST api - group name starting with a number is invalid

According to the swagger document, the group parameter must contain only alphanumeric characters or underscore. However, if the group name starts with a number, the call returns BadRequest.

Yes, group name must start with alphabetic characters. We will add it to swagger.


Could not load file or assembly 'HtmlAgilityPack, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=bd319b19eaf3b43a' or one of its dependencies.

Version: 1.2.1
URL used: https://www.webtoons.com/en/comedy/boyfriend-of-the-dead/list?title_no=1102
Environment: Windows 10 Professional x64
See the end of this message for details on invoking
just-in-time (JIT) debugging instead of this dialog box.

************** Exception Text **************
System.IO.FileNotFoundException: Could not load file or assembly 'HtmlAgilityPack, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=bd319b19eaf3b43a' or one of its dependencies. The system cannot find the file specified.
File name: 'HtmlAgilityPack, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=bd319b19eaf3b43a'
   at Kea.Main.<GetChapterAsync>d__23.MoveNext()
   at System.Runtime.CompilerServices.AsyncTaskMethodBuilder.Start[TStateMachine](TStateMachine& stateMachine)
   at Kea.Main.GetChapterAsync(String line)
   at Kea.Main.<>c__DisplayClass22_0.<DownloadQueueAsync>b__0()
   at System.Threading.Tasks.Task`1.InnerInvoke()
   at System.Threading.Tasks.Task.Execute()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
   at Kea.Main.<DownloadQueueAsync>d__22.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
   at Kea.Main.<startBtn_Click>d__21.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.AsyncMethodBuilderCore.<>c.<ThrowAsync>b__6_0(Object state)

WRN: Assembly binding logging is turned OFF.
To enable assembly bind failure logging, set the registry value [HKLM\Software\Microsoft\Fusion!EnableLog] (DWORD) to 1.
Note: There is some performance penalty associated with assembly bind failure logging.
To turn this feature off, remove the registry value [HKLM\Software\Microsoft\Fusion!EnableLog].

************** Loaded Assemblies **************
mscorlib
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.4075.0 built by: NET48REL1LAST
    CodeBase: file:///C:/Windows/Microsoft.NET/Framework/v4.0.30319/mscorlib.dll
----------------------------------------
Kea
    Assembly Version: <IP_ADDRESS>
    Win32 Version: <IP_ADDRESS>
    CodeBase: file:///C:/Users/Public/Pictures/Kea.exe
----------------------------------------
System.Windows.Forms
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.4042.0 built by: NET48REL1LAST_C
    CodeBase: file:///C:/Windows/Microsoft.Net/assembly/GAC_MSIL/System.Windows.Forms/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Windows.Forms.dll
----------------------------------------
System
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.4001.0 built by: NET48REL1LAST_C
    CodeBase: file:///C:/Windows/Microsoft.Net/assembly/GAC_MSIL/System/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.dll
----------------------------------------
System.Drawing
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.3752.0 built by: NET48REL1
    CodeBase: file:///C:/Windows/Microsoft.Net/assembly/GAC_MSIL/System.Drawing/v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a/System.Drawing.dll
----------------------------------------
System.Configuration
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.3752.0 built by: NET48REL1
    CodeBase: file:///C:/Windows/Microsoft.Net/assembly/GAC_MSIL/System.Configuration/v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a/System.Configuration.dll
----------------------------------------
System.Core
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.4110.0 built by: NET48REL1LAST_B
    CodeBase: file:///C:/Windows/Microsoft.Net/assembly/GAC_MSIL/System.Core/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Core.dll
----------------------------------------
System.Xml
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.3752.0 built by: NET48REL1
    CodeBase: file:///C:/Windows/Microsoft.Net/assembly/GAC_MSIL/System.Xml/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Xml.dll
----------------------------------------
Accessibility
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.3752.0 built by: NET48REL1
    CodeBase: file:///C:/Windows/Microsoft.Net/assembly/GAC_MSIL/Accessibility/v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a/Accessibility.dll
----------------------------------------

************** JIT Debugging **************
To enable just-in-time (JIT) debugging, the .config file for this
application or computer (machine.config) must have the
jitDebugging value set in the system.windows.forms section.
The application must also be compiled with debugging
enabled.

For example:

<configuration>
    <system.windows.forms jitDebugging="true" />
</configuration>

When JIT debugging is enabled, any unhandled exception
will be sent to the JIT debugger registered on the computer
rather than be handled by this dialog box.

Found the reason. I extracted only the .exe, as all of the other files were hidden.

Found the reason. I extracted only the .exe, as all of the other files were hidden.


Cleanup

Using github to keep individual commits separate and clear

CC @sgjesse
Brining this package into the future a bit

@lrhn can you take a look?

 This library, if we want to support it, should be rewritten to be converters.

Absolutely!
The goal here: clean this up so it's easy to migrate and maintain the tests as we move to pkg/convert


Correct a constructor call in ruby credentials test

pointed out by  a comment in https://github.com/grpc/grpc/pull/10529#pullrequestreview-44137221.
credentials in this test aren't actually used, so the currently incorrect order is working, but looks like a sort of doc issue

*windows kokoro failure of vsprojects/Release/h2_http_proxy_test.exe hpack_size , looks like secure test version of issue in https://github.com/grpc/grpc/issues/9037


Oc client jadeless

This is a short term fix for #297
Given oc-client requires Jade only for its own client-side runtime, and this is going to be removed soon anyway in favour of Pug on #300 (which compiles encapsulated templates so runtime is not gonna be needed anymore) - this should fix some problems as insecure dependencies and npm shrinkwrap. In fact, npm-shrinkwrap is not actually needed anymore in the oc-client as it was a temporary measure to get snyk green.

LGTM 👍


Prepared parametrized statement fails if previous execution has raised error.

Following scenario causes error:

Suppose we have a table Employees with unique column "name"
Prepare parametrized statement that inserts new employee with name as parameter
Bind and execute statement with paramenter "employee-1"
Do the same again. As expected constraint exception is raised which is correct
Bind and execute statement with paramenter "employee-2".  The statement fails with previous error!

Pull request includes test to reproduce the issue.

Yes, this is not unexpected.  You should be resetting your statements after an exception.  Use Statement#reset! after an exception.
https://github.com/luislavena/sqlite3-ruby/blob/master/ext/sqlite3/statement.c#L407

Ah, I see.  No, I think this is a bug.  If I remove the error check from the reset! call, your test passes.  I think calling reset! doesn't clear the last error on the database.


loads without transparency then exits

Hi I have been attempting to get this to work on my pc for a few versions and have hesitated to comment but it seems Hurl fails to load correctly on my PC. It will appear black rather than glass/transparent and when I hover the mouse over the scrollbar it will exit on its own. I'm not sure what is going on or why but I have attempted to install a number of versions of .NET but still no luck unless I have missed one.
Right now Browser Select seems vastly lighter and more stable but I still have hopes for this one since it reminds me of the OG Browser Chooser with a lot more potential.

Rignt now, I already fixed scrollbars crash in unreleased version. and .NET 6 desktop runtime is the only thing you need to have in your system for Hurl to work.
And Both are issues related to the package hurl uses, wpfui which is bit unstable for lot of things in Windows 10. I had to create  patches and PR them to the package for Acrylic to work. So, I was bit experimenting on moving current codebase to either C++ or UWP or WinAppSDK style.
Yeh, Browser Select is a lot lightweight and stable. I intend to support a lightweight version of Hurl, with bunch of stuff like acrylic, single-instances removed for that lightness. but that's a bit extra work. Im in final year of gradution......

Great to hear from you.
Interesting. I'd imagine C++ would be the better option for compatibility and to lighten things up but I am not a programmer at all and just going off basic knowledge.
Whilst Browser Select is pretty cool and lightweight it Hurl can be great also. I have seen numerous projects attempt to replicate Browser Chooser that never quite made it this far or had this much promise. Perhaps a focus on moving the current codebase to something that does not fight you all the way would be better so you can work on it more efficiently but absolutely keep up with your studies. Nothing is more important than your life and future.
I will continue to keep an eye on this project and keep testing it with every new release.
Thanks for the reply and software development.
It's a pretty cool project.

Please do try out the new version v0.6.3 and check if the same happens....

Good news is that it doesn't crash anymore and the window isn't black anymore. It works fine on desktop in terms of transparency but when over the top of a folder the background goes grey. I'm not sure what the three dot menu on each icon for the browser is meant to do but just a dark grey line appears and not anything else.
Big improvement though.

Thanksi 😊

three dot menu on each icon for the browser

it's #36

I should write some code to hide it if empty or not used, like in your case. but it currently seems not possible and The project needs some refactor for that.
Closing the issue btw. but still open to discuss.

I tried both v6.0.3 and v7 alpha and the acrylic effect isn't working, the background is black…
It would be better to have the options show on mouse over under the icon as an extended list (same dimension that just animate to show them and on mouse over of each one of them name it shows a tooltip or bubble with the arguments if this is enabled in the setting)

I tried both v6.0.3 and v7 alpha and the acrylic effect isn't working, the background is black…

please share Screenshot and OS version

mouse over under the icon as an extended list

Intresting design idea. Might need to experiment if this is actually possible, without the edges of tooltip clipping off.


Encrypted rooms are not supported

I'm trying to setup a bridge to an encrypted room. Unfortunately, when I send a message to Matrix, the bridge writes into the log:
Unknown matrix event type: m.room.encrypted

Encryption there is not relevant, but Matrix does not support disabling encryption for a room. So I'm wondering if it is possible to add support of event m.room.encrypted'?

I would be interesting in supported encrypted rooms but I personally have no plans to do so currently.


Fix Alert Baseclass

Codecov Report
Base: 88.49% // Head: 88.55% // Increases project coverage by +0.05% :tada:

Coverage data is based on head (0c288c8) compared to base (9c7a614).
Patch coverage: 93.75% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #299      +/-   ##
==========================================
+ Coverage   88.49%   88.55%   +0.05%
==========================================
  Files          48       48
  Lines        3939     3939
==========================================
+ Hits         3486     3488       +2
+ Misses        453      451       -2

Impacted Files
Coverage Δ

goblet/infrastructures/alerts.py
84.02% <93.33%> (ø)

goblet/tests/test_alerts.py
100.00% <100.00%> (ø)

goblet/infrastructures/infrastructure.py
81.81% <0.00%> (+9.09%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Rainbow best segments

NOTE: I introduce a function visual_timer_cycle into this.  I think it's conceptually sound (a global timer used to allow animation without state), and the implementation is fine, but I would like someone to majke sure the API makes sense


Updated to 4.3.10

This package is updated automatically using ChocoMilk. Therefore we do not take PRs  for version updates. Unfortunately there seems to be an issue with release-monitoring.org. It reports version 4.3.11 to be available but its nowhere to be found. ChocoMilk tries to package this version an fails. The next release should be picked up automatically again.


Change Key Sequence. Grave Accent (`)

How to set the grave accent key (`) as input language switcher, how do you type the grave accent?

You choose your desired keyboard layout here:

If there is no keyboard layout that offers the desired behavior, I assume you need to install additional software like PowerToys Keyboard Manager.


Updating STS version from <IP_ADDRESS> to <IP_ADDRESS>

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.003%) to 41.756%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/dialog/configurePython/configurePathPage.ts
1
57.02%

src/sql/workbench/services/connection/browser/connectionDialogWidget.ts
1
73.44%

Totals

Change from base Build<PHONE_NUMBER>:
-0.003%

Covered Lines:
30826

Relevant Lines:
69082

💛  - Coveralls


Missing EU project reference in the merged PR

We have recently merged pull request #484 which was done as a part of the EU-funded project FORCE. It is my completely my fault that I forgot to mention this, and didn't include any reference to the grant in the source code.
I was wondering if it could be possible to add this short note to the hypervolume.py file?
This work is implemented under the Formulations and Computational Engineering (FORCE) project within Horizon 2020
(`NMBP-23-2016/721027 <https://www.the-force-project.eu>`_).

I would very much appreciate if you could give me a permission to add it.
Thank you!

Sure, if it does not impact the licensing I can't see any problem with this.
Open a PR and we can merge it as long as the file keeps the initial header.


Spotlight Compatabilty

Hi - I see the project is no longer maintained, but hoping someone still might see this. I have a scene with 4 directional lights and 4 spotlights. The directional lights are Mixed - Baked Indirect and the Spotlights are Realtime. I cannot get objects shaded by the toon shader to interact with the Spotlights at all. I've tried all sorts of combinations of modes and settings, but no luck.
Anyone have thoughts? I'm super new to lighting with URP.

The toon shader should work fine with spotlights (at least on 2021 LTS, which is the latest version it was updated for).
I have an idea though: what's the "Per Object Limit" value set in your URP pipeline asset? It might be that your setup hits the limit.


Performance problem when thread number is big

NLog.config.txt
Processor.cs.txt
Program.cs.txt
Hello!
I'm developing the app with big number of the long running threads (900-2000). When thread number is big the program has high cpu usage. After profiling the program I found that NLog eats 30% of all CPU usage in the method WriteAsyncLogEvents. The Blocking GC is high too.
When I have 10 process with 100 threads there is no any problem
So I write simple test for NLog, where all threads process one collection. Processing of each item do 4 call to Logger.Info() with intermediate pause about 80ms. Running this test with 900 threads leads to 30% CPU usage on my Corei5 processor. But when I start 9 processes with 100 threads per process (total is 900 too) the Cpu usage less then 2-3%.
So, I'm trying to find the way to have 900 threads and low cpu usage.

Thanks for the test program!
We will try to look at this, this weekend, but it's a bit busy, so it could be also next week.

Will be next week, as my laptop is really slow after a Windows update :(

ok. will wait!

Those peaks are the problem? As I don't see them much:

Is this tested with NLog 4.2?

Some results, don't look bad IMO

Hello!
I see this on my core i5

Nlog 4.1.2

Will try 4.2
do you use NLog.config with output to file?

The same with 4.2

do you use NLog.config with output to the file? Looks like you do test without output or with async-wrapper with default drop-mode.

I test with your program, no line changed! Only difference that NLog is directly references instead of nuget package. I don't think nuget packages could lead to more CPU ;)
I have a SSD, i7, but's tested on a laptop. (which has already performance issues)

have you tried changing the values of keepFileOpen  / concurrentWrites?

I'm using SSD too.
For the test I've used
keepFileOpen="true" concurrentWrites="false"
(it is in the file attached to the first post)
If I cut these settings CPU increase to 55-60%

Could you send me your project (or binaries for test) to the email?

Tomorrow will run the test on the server with 16 Opteron core.

I don't see any file written here. I will fix that first :)

ok :)

OK confirmed, but now, what to do about it.
Test program is here: https://github.com/NLog/NLog/tree/many-many-thread---and-nice-cpu (src/nlog/NLog.netfx45.sln),

It seems the lock is a issue. Also I saw a lot of datetime.tostring()'s. I have to go, but I will look further into it. You can also check it out with VS 2015:

Removing payment.Date.ToString("yyyy-MM-ddTHH:mm:ss.fffzzzz") change nothing.
The same result with
logger.Info("TL_ID={0}");
instead of huge string with many parameters.
Honestly the whole producer/consumer part can be removed. So the program will have 900 running threads that output 4 string with pause.

Just for comparison here are the results in a third PC (i5, SSD)

I did some CPU profiling and didn't find anything interesting. Most of the time was spent waiting:

The interesting thing here is that 9 process with 100 thread per 1 process don't use cpu so much.

Thanks for testing @epignosisx

Yet another thing. The problem is for Console target too. So I doubt the problem is in the BaseFileAppender only.

Is it the console or the colored-console target?
It would be nice if it's the console target, because the will make the test case smaller.
We're a bit stuck on this, but if you can make the test case smaller (PR on https://github.com/NLog/NLog/tree/many-many-thread---and-nice-cpu), that we shall test it again.

Sure, I'll make it tomorrow.

Could you get changed sources from the my cloud and insert into the project?
https://yadi.sk/d/R0rdYpPFkaEwV
I have some problem with creating correct pull request

Please restore your branch @emanov , then I will create the PR.

for the correct Pr, just choose the correct branch after PR (Xamarin5 here as example)

restored

Do you need to push some stuff? (to Github)

Just created pull request from the web interface.

I think the lock is blocking too much:

before:
1% cpu at console

after:
almost every cpu cycle (96%) is used for the console itself.

sure, we need the lock, but maybe a smaller scope?

From the my point of the view thera some places that should be optimized.

IsInitialized - is not thread safe, so it's used with the lock
///
/// Gets a value indicating whether the target has been initialized.
///
protected bool IsInitialized { get; private set; }
but what if we would use
///
/// Gets a value indicating whether the target has been initialized.
///
protected bool IsInitialized { get {return isInitialized;} set { isInitialized=value; } }
private volatile bool isInitialized;

that way doesn't require lock(syncLock)

constructions with lock often used without prechecking
lock (this.SyncRoot)
{
if (this.IsInitialized)
{
foreach (Layout l in this.allLayouts)
{
l.Precalculate(logEvent);
}
}
}
that would be good to have
if (this.IsInitialized)
before lock too.

Target.PrecalculateVolatileLayouts use lock for all precalculations:
foreach (Layout l in this.allLayouts)
{
l.Precalculate(logEvent);
}
but there is no nessecary in it as I see. Instead of this we should make
var layouts = this.allLayouts;
foreach (Layout l in layouts)
{
l.Precalculate(logEvent);
}
and to protect initializing this.allLayouts
this.allLayouts = new List(ObjectGraphScanner.FindReachableObjects(this));
seems to be good to replace with
Interlocked.Exchange(ref this.allLayouts, new List(ObjectGraphScanner.FindReachableObjects(this)));
and so on. That's all should reduce locks.

Sounds valid. Can you send a PR for this nl

Ok, hope to make PR next week.

That is the PR:
https://github.com/NLog/NLog/pull/1369
I don't understand the logic in the methods: Initialize, Close. But it looks very suspiciously.

A Comment regarding Console writing from multiple threads.
Console.Write is synchronized by the OS on windows, so if you have multiple threads writing to a Console Appender it is slow, not because of NLog, but because of windows. If you try to run the test, with the window minimized, you will probably see that the test runs faster, since Windows detects that the window should no longer be drawn on the screen, and things run faster.
I once had a Perl script that run on windows, to convert  Source Safe into SVN, after 4 hours of running, while it was writing to the console, I stopped the program. I then restarted the program and by accident I minimised the window, and 5 minutes after it was done.

Thanks for the info @bjornbouetsmith!

Has this been resolved in the latest release?

No and yes as there is IMO no problem (in NLog) here.

Tested this with #1663, and now very few garbage collections occur. So if the problem was time spent on garbage collection, then it can be fixed. Work-around would be to activate gcServer in app.config.

Testing with #1700 and #1663 then the CPU usage stays around 10 pct, and never rises to more than 15 pct. (i5)


Note: Contents to be made translatable

[ ] Quest texts: Titles, subtitles, descriptions, task titles, etc (this includes additional tasks that are added halfway)
[ ] Block / item tooltips
[ ] Ponder information and directions
[ ] Custom machinery block name (probably put a placeholder in the kubejs or whatever startup file)

Ponders: need keys from kubejs\assets\ponderjs_generated\lang, and the code used is Text.translate(key).getString()
Other KubeJs scripts (tooltips for example) uses keys from kubejs\lang\ and the code used is Text.translate(key)

Ponders: need keys from kubejs\assets\ponderjs_generated\lang, and the code used is Text.translate(key).getString() Other KubeJs scripts (tooltips for example) uses keys from kubejs\lang\ and the code used is Text.translate(key)

Noted, all contributors will be notified to refer to this issue for future works.

For ponderjs, you actually shouldnt need to change any code; just create the lang file according to the en_us.json version. This will all be added when I'm done with the ponder stuff


mixing T.untyped and T::Boolean in && expressions gives unintuitive results

Input
→ View on sorbet.run
# typed: true
extend T::Sig

sig {params(x: T.untyped, z: T::Boolean).returns(T::Boolean)}
def foo(x, z)
  x && z
end

sig {params(x: T.untyped, z: T::Boolean).returns(T::Boolean)}
def bar(x, z)
  z && x
end

Observed output
editor.rb:7: Expected T::Boolean but found T.nilable(T::Boolean) for method result type https://srb.help/7005
     7 |end
        ^^^
  Expected T::Boolean for result type of method foo:
    editor.rb:10:
    10 |def foo(x, z)
        ^^^^^^^^^^^^^
  Got T.nilable(T::Boolean) originating from:
    editor.rb:5:
     5 |def foo(x, z)
                ^
    editor.rb:5:
     5 |def foo(x, z)
                   ^
    editor.rb:6:
     6 |  x && z
          ^
Errors: 1

Expected behavior
It shouldn't matter which way you write the && expression.  People are surprised by the T.nilable(T::Boolean), even if it is sort of technically correct.

This is a duplicate of #373 (thanks @marianosimone !)


Some Git repositories might benefit from using Git LFS

Since I have a collection of "clone --mirror" git repositories, I can easily check their size. All above 1 GB size or so, I can check if with "git lfs migrate" I can improve their performance if they are under my control and not many clones/forks exist. Otherwise one can inform the maintainer of the respective projects about the situation.

I could probably optimize:

Vegatrek (https://github.com/Trilarion/vegatrek) with 1.2 GB
MurderPD (https://gitlab.com/Trilarion/murderpd) with 270 MB
Radakan (https://github.com/Trilarion/radakan) with 250 MB
Spice Trade (https://gitlab.com/Trilarion/spicetrade) with 240 MB

Other big repositories are:

Speed dreams (https://sourceforge.net/projects/speed-dreams/ SVN) with 49 GB
UFO:AI (https://github.com/ufoai/ufoai) with 3.6 GB
0AD (https://github.com/0ad/0ad) with 3.6 GB
Crossfire (https://sourceforge.net/projects/crossfire/ SVN) with 2.8 GB
...


JsonReaderException while parsing valid json

I'm hitting a JsonReaderException while trying to parse the json in the gist below with the classes in the gist below. I'm using JsonConvert.Deserialize<Tweet[]>() for this. The json validates according to external validators.
https://gist.github.com/onovotny/b85fdee299399e3212c3
Newtonsoft.Json.JsonReaderException occurred
  _HResult=-2146233088
  _message=Error reading string. Unexpected token: StartObject. Path '[7].retweeted_status.user.entities.description.urls[0]', line 1, position 41582.
  HResult=-2146233088
  IsTransient=false
  Message=Error reading string. Unexpected token: StartObject. Path '[7].retweeted_status.user.entities.description.urls[0]', line 1, position 41582.
  Source=Newtonsoft.Json
  LineNumber=1
  LinePosition=41582
  Path=[7].retweeted_status.user.entities.description.urls[0]
  StackTrace:
       at Newtonsoft.Json.JsonReader.ReadAsStringInternal()
  InnerException:

Seems that string[] in Description should have been Url[]. Was working at one time, strange.


[BUG] Not all api docs show their examples.

testOntologyInsert and getTermsForAdvancedSearch do not show their examples in the api docs.

After some testing i found the problem to be that examples with DateTime do not show.


Pod install missing some files.

I just run the pod install but seem to it's missing some files.

I missed this issue. Is it still happening? What errors do you get?

@moozzyk please publish latest commit. its April commit. Also there is one extra field Swift version in podspec file. Need to remove to install it.
I forked and did the same. everything working nicely.

I'm also seeing this issue. several of the files are missing from the pod when running pod install

@mleatherb - are you installing from the repo or from the CocoaPods? Do you see any errors during installation? FYI, I am working on a new version and updated the repo yesterday. Can you check if the issue still reproduces with the version in the repo?

I just pushed version 0.3.0 to CocoaPods. With the following line in the Podfile of my test project the latest version of the client is installed successfully - it compiles without errors and is able to connect to the server.

Closing this for now - a new version was released and it worked when I tried it. Feel free to reopen if you encounter the issue again.


Style contains key with bad entry

Everytime the dialog is shown, I get this errors. BTW, the dialog is shown correctly.
11-28 01:31:39.203: E/ResourceType(2540): Style contains key with bad entry: 0x010102dd
11-28 01:31:39.203: E/ResourceType(2540): Style contains key with bad entry: 0x0101035b
11-28 01:31:40.753: E/ResourceType(2540): Style contains key with bad entry: 0x010102dd
And so on...
Tested on API 8 and 9.

@davidvavra :Thanks a million for this explanation.
I was worried maybe I have some runtime error or bad code.

Blast from past :)


No update since 2 days

Hello everyone,
I did not get any updates since two days. The RKI is updated.
For example:
Germany --> Bundesland: Bayern --> Kreis "Neustadt_ad_Waldnaab
Many thanks

coronavirus-statistics.0
2021-03-18 10:00:25.754
error
at Covid19.onReady (/opt/iobroker/node_modules/iobroker.coronavirus-statistics/main.js:635:5)

coronavirus-statistics.0
2021-03-18 10:00:25.754
error
at processTicksAndRejections (internal/process/task_queues.js:97:5)

coronavirus-statistics.0
2021-03-18 10:00:25.754
error
at runMicrotasks ()

coronavirus-statistics.0
2021-03-18 10:00:25.754
error
at germanyCounties (/opt/iobroker/node_modules/iobroker.coronavirus-statistics/main.js:512:35)

coronavirus-statistics.0
2021-03-18 10:00:25.754
error
(3853) [germanyCounties] error: values.features is not iterable, stack: TypeError: values.features is not iterable

should be solved by 0.6.9
https://github.com/DrozmotiX/ioBroker.coronavirus-statistics/releases/tag/v0.6.9


Multiline docblock comment for variable

Hello!
Php-cs-fixer removes second asterisk from multiline PHP doc comment when this comment is in method:
private function docMethod()
{
    /**
    * @var \stdClass
    */
    $a = $this->getClass();
}

Becomes =>
private function docMethod()
{
    /*
    * @var \stdClass
    */
    $a = $this->getClass();
}

Is it expected?
Thanks!

Yes, you can't have phpdoc for a variable like that. The correct formatting is:
/** @var $a object */
$a = $this->getClass();

Ok, cool, thanks!

But /** @var $a object */ gets fixed to /* @var $a object */ (inline comment).
Is that indended?

Yes.

That is intended if the thing following the phpdoc is not a structural element.

It's illegal according to the proposed PSR spec we're following.

Which PSR spec states docblocks may have a single asterisk for non-structural elements?
The offending fixer, phpdoc_to_comment, is supposed to convert comments that do not contain docblock tags. These comments clearly contain docblock tags and therefore should not be changed.

Which PSR spec states docblocks may have a single opening asterisk for non-structural elements?

none (AFAIK, but also none that does not allow it), that is not what this fixer is about

The offending fixer, phpdoc_to_comment, is supposed to convert comments that do not contain docblock tags.

The fixer converts "invalid" PHPDoc to comments, it should not touch comments.

These comments clearly contain docblock tags and therefore should not be changed to regular single-asterisk comments.

Btw. just noticed;  this https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/1250#issuecomment-110815055
/** @var $a object */
$a = $this->getClass();

is not correct I think, it should be
/** @var \stdClass $a */
$a = $this->getClass();

last case should not be touched by the fixer.
The first sample of this issue also looks OK to me, but I know it is not by spec (PSR5).
Please provide a sample that is fixed the wrong way.

Yes, whatever psr suggest a single /* is wrong, it must be /** (also logically^^)
It also gets parsed this way as structured doc block by tokenizing.
So to be clear:
/** @var \stdClass $a */
$a = $this->getClass();

is the correct one.

Hi @SpacePossum :wave: Thanks for helping out with this issue!
In this simple example we will see the fixer removes the double asterisk.
<?php

class A
{
    public function b()
    {
        foreach ([] as $c) {
            /** @var string $c */
            ++$c;
        }
    }
}

However, placing the annotation before the foreach does not cause this to happen.
<?php

class A
{
    public function b()
    {
        /** @var string $c */
        foreach ([] as $c) {
            ++$c;
        }
    }
}

I believe it is correct to place the annotation before the foreach, however the way the fixer behaves is odd. Instead of removing the asterisk we would prefer the annotation to be moved to the correct line. Of course, that is not the job of this fixer, but if it's not the job of this fixer why is it making a change at all?

Hi :)
Above the foreach is the "best" place as it is the place where the variable is created. That being said, I see value in the PHPDoc on the second location as well (but is not valid following PSR5; see "Example").

the fixer behaves is odd.[...]  why is it making a change at all?

Agreed, I typically don't run this fixer for this reason. It doesn't "fix" the PHPDoc by moving it or fixing the PHPDoc, it changes it into a comment.
@dereuromark

Well, whatever psr suggest a single /* is wrong, it must be /** (not only logically^^)

I don't think there is a PSR that suggest this. The fixer changes PHPDoc's not following PSR5 (either in content/layout or location) into a comment, so not because it must be a comment but because the fixer doesn't like the PHPDoc.

(normalized form)

I think that variant should be supported but I'm not sure, as I do not use this fixer.

Now that I know why this is happening I don't mind it so much... previously I thought it was a bug in the fixer but it seems that, for all cases I looked at so far, it indicates the docblock is placed on the wrong line (or maybe the type and name were transposed like you identified earlier). The only problem is it's not clear for most people, from the way the fixer behaves, what the real issue is.

I have posted a new (open) issue to track this. #2634


Code for Darcy Flow does not work

Hi,
Amazing project! I am able to duplicate the Burgers experiment. However, the code for Darcy Flow does not work.
python3 train_operator.py --config_path configs/pretrain/Darcy-pretrain.yaml
with the following error:
Traceback (most recent call last):
File "train_operator.py", line 133, in
train_2d(args, config)
File "train_operator.py", line 90, in train_2d
num=data_config['n_samples'],
KeyError: 'n_samples'

Hi,
Amazing project! I am able to duplicate the Burgers experiment. However, the code for Darcy Flow does not work.
python3 train_operator.py --config_path configs/pretrain/Darcy-pretrain.yaml
with the following error: Traceback (most recent call last): File "train_operator.py", line 133, in train_2d(args, config) File "train_operator.py", line 90, in train_2d num=data_config['n_samples'], KeyError: 'n_samples'

write n_samples instead of n_sample in configs/pretrain/Darcy-pretrain.yaml file.

Hi,
Amazing project! I am able to duplicate the Burgers experiment. However, the code for Darcy Flow does not work.
python3 train_operator.py --config_path configs/pretrain/Darcy-pretrain.yaml
with the following error: Traceback (most recent call last): File "train_operator.py", line 133, in train_2d(args, config) File "train_operator.py", line 90, in train_2d num=data_config['n_samples'], KeyError: 'n_samples'

write n_samples instead of n_sample in configs/pretrain/Darcy-pretrain.yaml file.

Thanks for your feedback. I just fix that issue and compiled again, a new error appears:
Traceback (most recent call last):
File "train_operator.py", line 133, in
train_2d(args, config)
File "train_operator.py", line 98, in train_2d
pad_ratio=config['model']['pad_ratio']).to(device)
KeyError: 'pad_ratio'
I am wondering if the author accidentally push the run version of the code?

Hey,
Sorry, my bad. I added a new feature to the FNO architecture (padding) and didn't merge it correctly with the main branch.
I just updated the repo.  For Darcy exp, you can run the following to train the operator
python3 train_darcy.py --config configs/operator/Darcy-pretrain.yaml

To test the trained operator on test set, run
python3 train_darcy.py --config configs/operator/Darcy-pretrain.yaml --test --ckpt [path to the checkpoint]

Regards

Thanks for putting this together.
For train_2d_operator() function, what does variable "y" refer to ? I don't see anything in the argument nor in the code. Does it refer to "u"? I got an error by running it saying that "y" is not defined.
for data_ic, u, pde_ic in train_loader:
data_ic, u, pde_ic = data_ic.to(rank), u.to(rank), pde_ic.to(rank)
        optimizer.zero_grad()

        # data loss
        if data_weight > 0:
            pred = model(data_ic).squeeze(dim=-1)
            pred = pred * mollifier
            data_loss = myloss(pred, y)


Bigtable: TimestampRanges must be milliseconds granularity

Closes #4932.
But.  @tswast Should we have this and change it if/when BigTable supports microseconds or should we have the users handle it themselves and just document?

I'm not familiar with how Bigtable handles timestamps. @mbrukman might?

oh.  Here's some related info also bigtable support for microseconds

I'll ask the various teams about this.

I'm going to make the executive decision.  This is for row range filters.  For now, start out to be rounded down to nearest ms, and end needs to be rounded up to the nearest ms.
If and when Cloud Bigtable opts to enable microsecond support, which is not on the near term roadmap, we'll have to do a sweep of all clients for behavior like this.

@tswast yeah, stupid mistake.  I changed the tests to better capture what it should do.


Replace step 3 in "Update API model" with the following...

"In the kubernetes-azurestack.json file, find orchestratorRelease and orchestratorVersion. Select one of the supported Kubernetes versions. For example, for orchestratorRelease use 1.14 or 1.15 and for orchestratorVersion use 1.14.7 or 1.15.10 respectively. Specify the orchestratorRelease as x.xx and orchestratorVersion as x.xx.x. For a list of current versions, see Supported AKS engine Versions <https://github.com/Azure/aks-engine/blob/master/docs/topics/azure-stack.md#supported-aks-engine-versions>. " REMOVE example with aks-engine get-versions command it is confusing to customers since it for Azure Stack it is a smaller set of supported versions. Thanks!

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

ID: e9652b7b-09b9-ace9-4ef7-abc413b9d52c
Version Independent ID: 53c19bb0-3f77-d15c-87e2-33e1d442d73b
Content: Deploy a Kubernetes cluster with the AKS engine on Azure Stack Hub - Azure Stack Hub
Content Source: azure-stack/user/azure-stack-kubernetes-aks-engine-deploy-cluster.md
Service: azure-stack
GitHub Login: @mattbriggs
Microsoft Alias: mabrigg

@walterov Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@walterov Updated. Please see the following PR: https://github.com/MicrosoftDocs/azure-stack-docs-pr/pull/3444

#please-close


Feature/biometrics

Authentication with PIN ans biometrics

@TobiCrackIT - add a list of URLs where you got the icons

@scotopic https://www.flaticon.com/free-icon/padlock_3064197?term=padlock&page=1&position=4&page=1&position=4&related_id=3064197&origin=search
https://www.flaticon.com/free-icon/fingerprint_2313181?term=fingerprint&page=1&position=4&page=1&position=4&related_id=2313181&origin=search

@scotopic, this PR has been updated to make the biometrics feature work as required.

Android bug

Install app from scratch
Create a new wallet
Click settings > Enter pin
Background the app
Foreground the app ( see the pin entry screen )

EXPECTED: No back button is visible
ACTUAL: Back button visible and takes you to the Secret Phrase view

iOS bug

Settings > Enable "Unlock with biometrics"
Once the auth view comes up > click Cancel

ACTUAL: Another view appears to be added on the stack. Can never cancel out to enter the PIN.
EXPECTED: Cancel should remove the biometrics dialog and allow access to the PIN UI

iOS issue:

Settings > Enable "Unlock with biometrics"
Tap on "Unlock with biometrics" to disable and authenticate the biometrics.

EXPECTED: Biometrics is disabled once the user is authenticated.
ACTUAL: PIN UI is shown even after the biometrics is authenticated.

@scotopic , the issues raised have been fixed


FIX Issue #15

@ipapapa
Fixing https://github.com/Netflix/dynomite-manager/issues/15
Cheers,
Diego Pacheco

Good catch, forgot the binding in the OSS...

@ipapapa  tks np.


Privly button weird behaiour

The following circumstances make the privly button act very strange:

[ ] In Facebook chat small window, when the button appears in the form, if the main page is scrolled, the button will change its position according to the scrolling [1]

[ ] In Gmail, when composing a new mail, adding bold text and selecting it will modify the button's position [2]

Closing this since the gsoc-develop branch takes care of these issues.


Project created via api doesnt have labels

My actions before raising this issue

[x] Read/searched the docs
[x] Searched past issues

Steps to Reproduce (for bugs)

Code fragment which inflicts the issue
        data = {
            'name': 'project',
            'labels': [
                 {'name': 'cat'},
                 {'name': 'dog'},
            ]
        }

        response = requests.post(f'{self.url}/api/projects', data=data, auth=self._get_auth_data())
        assert response.ok

Expected Behaviour

Request should create project with labels in it
Current Behaviour

Project created without labels

Possible Solution

Perhaps my labels in wrong format
Your Environment

Git hash commit (git log -1): commit d5f3f93e8177de13659bedb3467208a196d360b7
Docker version docker version (e.g. Docker 17.0.05): 20.10.12
Are you using Docker Swarm or Kubernetes?
Operating System and version (e.g. Linux, Windows, MacOS): Windows 10 Home 22H2
Code example or link to GitHub repo or gist to reproduce problem: https://github.com/arseniymerkulov/cvat-use-cases

Hi. The request looks correct, it is basically the same what UI sends. I couldn't reproduce the problem - sending the request produces a project with 2 labels:
curl -X 'POST' 'https://app.cvat.ai/api/projects'   \
  -H 'accept: application/vnd.cvat+json' \
  -H 'Content-Type: application/json' \
  -H 'X-CSRFTOKEN: <...>' \
  -H 'cookie: csrftoken=<...>; sessionid=<...>' \
  -H 'referer: https://app.cvat.ai/api/swagger' \
  -d '{"name": "test_5667-2", "labels": [{"name": "cat"}, {"name": "dog"}]}'

Please check our tests (1, 2) and our SDK for examples. You can also play with API here (sending this request will require it to be written in one line).

Thanks for your answer. Solved by putting payload in json request field instead of data


[frameit] Add Nexus 6P support.

[x] Run rspec for all tools you modified
[x] Run rubocop -a to ensure the code style is valid
[x] We currently don't accept new actions, please publish a plugin instead, more information in Plugins.md

This pull request adds support for Android device (Nexus 6P). The need of it discussed in https://github.com/fastlane/fastlane/issues/4077.
Nexus 6P is used as one of the most popular and current Android device.

Hey @chezzdev, thanks for the pull request, and your contribution.
We worked on a brand-new version of frameit and just pushed the release https://github.com/fastlane-old/frameit/releases/tag/3.0.0 :rocket:
We currently don't plan on adding Android support for frameit. However this might be something that would be interesting in the future. It's important that the deliver gem doesn't contain any Android specific code, which is something that was modified in this PR.
I'll close this pull request for now, thanks again for your contribution 👍


Deploy Javadocs on github pages

Javadocs and other repo documentation could go in github pages under gh-pages branch /docs subdirectory.
Instructions for building and updating docs should go in master/develop branches' README, and/or point to a README in the gh-pages branch.

So what's your vision for this one @tttomat19? The version of the deployed javadocs in gh-pages always matches the version deployed in master? Such that eventually there would be a an Action that builds the javadocs into gh-pages branch at each release?

I can't merge it without at least a small explanation. Can you put something in README, even just one sentence, explaining how you build it? I would build it by cloning two versions of the repo, one on develop the other on gh-pages, then running a maven command (which?) in the develop to generate the docs, which would then go in the gh-pages clone of the repo. But you probably used a more efficient way?
Remember we are not doing automatisms here yet. Just a description like mine above would be enough.

@zgypa non-automated instructions to update javadocs added at the end of main README.md file
https://github.com/open-ortho/dcm4ceph/pull/6/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5

Basically yes, there would be an action. Triggers for action are discussable. It would automate what is written in instructions now. One latest version is sufficient I think.

I merged this also. Why did you choose to explicitly set docs and readme in .gitignore?

Hi!
The answer about docs and readme in .gitignore is as follows.
So everything except docs and readme is ignored.
The current manual flow for updating javadocs suggest that one switches branches with git switch command or with IDE.
At this moment bunch of files in working directory is not erased.
Then, if one executes git add . there may be some eg. sources from other branch accidentally added to the gh-pages branch, which is unwanted. We would want only updated javadocs to be added to commit.
Merely a convenience thing.


jsoup will not treat an incoming <Alpha as text

Jsoup does not treat this as text
 Jsoup.parseBodyFragment("<mg per week or  )" );

it seems to just discard this entirely, and I don't see a way around it, I was going to try to get jsoup to encode with &lt, but the parser is just discarding it. Also isValid returns true so I'm not sure how I could handle the problem on the fly. I've stepped through the debbuger, there are no children/attributes

<mg per week or  )

becomes

<mg per week or )<></mg>

and this (and
 )
<mg per week or  )

becomes
<mg per week or )< pan></mg>

version 1.9.2

I think my issue may be similar to this one? If I try to JSoup clean If AA I get If AA. I don't think element names are allowed to have spaces in them, but JSoup is parsing this text as an element.

You know, I'd complain about the resolution, because really I'm not sure how I could have encoded this garbage correctly (only from rereading this issue), yes I know it was garbage input, but that was the problem. I mean, how does the last example become < pan>, in what world does that make any sense. Really, I'm just going to complain that this wasn't responded to for 5 YEARS.

To be specific, it gets parsed like:
In:
<mg per week or  )

Out DOM (with verbose attributes for clarity):

  <mg per="" week="" or="" )>="" span=""></mg>

And how chromium sees it (same):

Which makes sense to me as far as it goes, given the constraints of HTML supporting custom tags and boolean attributes.

gross, and still doesn't make sense to me, I wonder how chromium is getting that too. I mean, I understand the <mg></mg> just not how it's creating , that seems like total garbage (though so does my input), and not sure why it was < pan> 60% certain that's a copy/paste.

There is no  element. As I wrote, that is an attribute name. If you are actually interested in how both of these parsers are getting this as the output, it is covered in the InBody section of the TreeBuilder algorithm in the WhatWG spec.


decode_b: Cleanup starting at line 4314

I think I've covered most of the changes in this PR already in my other recent decode_b PRs, so I think I'm just gonna close this one.  There's not much difference left between this and main now.


My game is not translated at all.

As the title said, nothing translated in my KCV, ship names, quests, equip names,...
Pretty sure that my language setting is set to English, b/c other tabs are in English (Overview, Fleet,...etc)
And this only happen on my laptop, my desktop is still fine.

I'm using Windows 10.
Please help.

You probably did not read the FAQ.

Thank, it's a shortcut problem :P


Adding --force flag to delete command

Description
Currently delete operations depend on devstream.state file.
If I accidentally delete the state file, or the apply execution is halfway interruptted, there is no state in the state file at all. At this time, there is actually some  modifications in the environment, but I have no way to delete this data.For example, helm install fails due to the image pull timeout. At this time, there is no information in the a file, but my application is actually installed. But I have no way to uninstall it. So I think we can add a --force flag to implement the uninstall operation no matter what is in the state file.

/assgin @lfbdev


getUnitIds returns only 50 unit ids

Hello,
i need to read all unit ids from the acs server but the function 'getUnitIds', which is responsible to do so, only returns 50 ids.
I used the unittype or profile as filter option.
However i do not understand why i only get 50 unit ids back.
Maybe someone can help me to solve my issue.
Thanks

This should be an easy problem to solve or figure out. But give me some time.


missing dash in install from source at cxxopt added

option --cxxopt="-D_GLIBCXX_USE_CXX11_ABI=0" needs two dashes.

Can one of the admins verify this patch?

CLA signed.
On Wed, Apr 12, 2017 at 8:22 AM, googlebot<EMAIL_ADDRESS>wrote:

Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project. Before we can look at your
pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll
verify. Thanks.

If you've already signed a CLA, it's possible we don't have your
GitHub username or you're using a different email address. Check your
existing CLA data https://cla.developers.google.com/clas and verify
that your email is set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
If you signed the CLA as a corporation, please let us know the
company's name.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/9157#issuecomment-293484867,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFY1A3sfuneJ4Z2Mpcl0NSZl_XWAHBVrks5rvG2rgaJpZM4M6_57
.

Jenkins, test his please.

Jenkins, test this please.


Bug: Italic/Bold font missing

Currently, the font in use does not bold or italicize correctly. Everything is upright font.

I recently changed it to ubuntu font, is this issue with the ubuntu font or the previous roboto font?

Is the demo currently using Ubuntu? Doesn't look like Ubuntu to me.

Ah, I just updated the demo: https://jieiku.github.io/abridge-demo/overview-markdown-syntax/#tables

I do not see Bold/Italic still

Can you tell me is that running it local (zola serve?)
Does it happen on the demo site? https://jieiku.github.io/abridge-demo/overview-markdown-syntax/#tables
If so which browsers does it happen with? I tested with Firefox and Chrome, but my Operating system is Manjaro.
here is my chrome demo:

here is my firefox:

Hopefully we can figure it out the cause, its difficult to fix if I dont know what is causing it. I will see if I can find a Windows box to test from, 90% of my machines are Linux.

In chrome press Ctrl + Shift + i This will open the developer console, then hold down control and press F5 while on the markdown page, see if the font is actually loading:

I am planning to move the demo to netlifly instead of github.

I was on Ubuntu, however, from my work laptop (Windows) I can see the bold/italics.
I'm wondering if, perhaps, since I likely had Roboto installed locally already, there may have been conflict there.

I believe that the issue is because of local font loading. Roboto was a font I've used heavily in the past, and I'm quite certain I had it downloaded (can't check from work).
It seems in the CSS every font is loaded by the same name, regardless of bold/italics:
https://github.com/Jieiku/abridge/blob/2dd1bf071c624197bf4fb064c52db77e521c0a6f/sass/fonts/_Roboto.scss#L14

OH, I bet your right the local("Roboto"),
Perfect, I will install the font locally and test this out! Awesome!

This was the correct fix, it works now. :)


What do you think about this hack?

We use GitHub and Travis CI to let students of Simplon.co practice both JavaScript and GitHub Test Driven Development Workflow.
To check a solution, just open a pull-request on the main repository, and let Travis CI do his job :wink: !

Thanks for the PR and your interest in improving the code so that others can learn from it.
Actually, this repo has configured Travis CI to run tests on each commit or PR. But as a whole repo, not for each individual exercise. This repo is part of the exercism.io platform, and each individual exercise is supposed to be solved in that platform (it's not just solving an exercise, it's the social interaction you get with that: technical feedback, comments from other developers,...).
So, for me, adding a Travis config file for each exercise is not aligned with the goal of this repo, sorry. Maybe, other @exercism/track-maintainers have a word on this. I can't take a decission myself.
Additionally, adding FizzBuzz exercise is a nice idea. It's a well known exercise to learn TDD and it causes some interesting discussions. Sorry, I'm sure there is a document describing how to add a new exercise to a language track (like this repo), but I can't find it (@matthewmorgan can you help me?)

Hmm, Travis would look for a .travis.yml in the root of the repository, right?
So does that mean: For this to work, every exercise needs to be its own separate repository? It would be possible to do that with git-subtree if that is desired. I wonder if it would be difficult to manage all the separate repos that would result! It seems it can be less work to use only one .travis.yml for all of xjavascript track, since the way to test the each problem is similar.

I'm sure there is a document describing how to add a new exercise to a language track

https://github.com/exercism/x-common/blob/master/CONTRIBUTING.md#implementing-a-completely-new-exercise

I think, that testing the repository as a whole, does improve the quality
of commits.
These tests do involve checking for proper configuration of the track as
well as some level of consistency. Especially when upgrading meta stuff
like updating the test runners version needs the complete run.
Also splitting this up would make a track unmaintanable as Peter said.
I do see only pros in testing everything.
Peter Tseng<EMAIL_ADDRESS>schrieb am Mi., 27. Apr. 2016, 10:20:

Hmm, Travis would look for a .travis.yml in the root of the repository
https://docs.travis-ci.com/user/customizing-the-build/, right?
So does that mean: For this to work, every exercise needs to be its own
separate repository? It would be possible to do that with git-subtree
http://stackoverflow.com/questions/920165/howto-extract-a-git-subdirectory-and-make-a-submodule-out-of-it
if that is desired. I wonder if it would be difficult to manage all the
separate repos that would result! It seems it can be less work to use only
one .travis.yml for all of xjavascript track, since the way to test the
each problem is similar.
I'm sure there is a document describing how to add a new exercise to a
language track
https://github.com/exercism/x-common/blob/master/CONTRIBUTING.md#implementing-a-completely-new-exercise
—
You are receiving this because you are on a team that was mentioned.
Reply to this email directly or view it on GitHub
https://github.com/exercism/xjavascript/pull/189#issuecomment-215008246

Thank you for all your comments, I agree that my pull-request is a bit messy :scream:..
So I can create a special branch for adding the FizzBuzz exercise, and keep the other commits of the fork for my personal use :wink:!
P.S. You can find the repository we create for this hack here.

The fizzbuzz exercise is effectively the Raindrops exercise, if curious.

Take a look at https://github.com/exercism/xlua. We're using Travis and a simple shell script to test each exercise independently. While not the most sophisticated setup in the world, it lets us make sure everything works without making the track unmaintainable.

$ jasmine-node *.spec.js --verbose --captureExceptions
count() - 8 ms
    counts one word - 6 ms

Failures:
  1) count() counts one word
   Message:
     Expected 1 to equal { word : 1 }.
   Stacktrace:
     Error: Expected 1 to equal { word : 1 }.
    at null.<anonymous> (/home/travis/build/simplonco/word-count/word-count.spec.js:8:33)

Finished in 0.013 seconds
1 test, 1 assertion, 1 failure, 0 skipped

Also take a look at https://github.com/jasmine/jasmine, to run tests locally :wink: !

@yvan-sraka @kotp @rchavarria This exercise is pretty much the 'raindrops' exercise, I agree.  For that reason, I don't think it makes sense to add it.
But, @yvan-sraka if you did want to add an exercise in the future (which would be awesome) there is a full list of exercism exercises (@kytrinyx where is that list?  I thought it was in x-common but I can't find it), and each language track repo has a list of what is implemented in that language.
So, you would either:

Add a new exercise definition to the master list (one PR), then add it to the track (a second PR), or
Find an exercise on the master list that's not on the track, and add it with a PR.

No need to really stick to the available list either... but it is good to know what is here so you don't duplicate it.  The way to find the missing exercises in language tracks is covered in https://github.com/exercism/discussions/issues/1

To echo what the others have said: Thank you so much for jumping in with suggestions and code! It's much appreciated.
I agree that fizzbuzz would be too similar to raindrops, in this case, to make sense to add it.
I also don't want to add travis config files to each individual exercise, but if you see a need for a way to test things that we don't currently have, definitely open an issue, and we can figure out what the pain point is, and maybe we'll find a way to fix that which is compatible with what we're doing.
The full list of existing problems is in github.com/exercism/x-common (all the *.md files are problem descriptions). We're always looking to add more exercises, so suggestions, pull requests, ideas, etc are all welcome. Sometimes we'll find that an exercise is too similar to one that we already have, but the discussions often spark interesting new things anyway, so it's never wasted.
I'm going to go ahead and close this. Thanks again for taking the time to submit it!


[SPARK-47378][PROTOBUF][TESTS] Make the related Protobuf UT run well in IDE

What changes were proposed in this pull request?
The pr aims to make the related Protobuf UT run well in IDE (IntelliJ IDEA).
Why are the changes needed?
Facilitate developers to debug the related Protobuf UT.
Before:

After:

Does this PR introduce any user-facing change?
No.
How was this patch tested?

Manually test.
Pass GA.

Was this patch authored or co-authored using generative AI tooling?
No.

Thank you, @panbingkun . Merged to master.


Pressure cluster

I tried to implement a pressure sensor endpoint by forking this repository. But I'm ending in Homekit saying the device is not supported. Are pressure sensors not supported in Homekit or am I missing something?
You can have a look here. Essentially I copied endpoint, cluster and attributes from the temperature sensor. It has the same mandatory attributes. The app I'm testing with is working fine with a temperature sensor.

Pressure sensor is not supproted in HomeKit.

Thank you for the information. Is there an overview, what is supported?

Apple supported devices here

Thanks


Can't open Open XR Toolkit in MSFS.

I can't open the app in MSFS.  Ctrl + F2 does not work and the Companion App does not show me what the hotkey is to open the app.  It just shows the function keys for navigating within the app. The companion app show that the toolkit is active (in green).

Can you attach a log as instructed in the Bug report template? Thank you.

Sorry, I don’t know how to attach a log.
Sent from Steve Paradise's  iPhone

On Jan 29, 2022, at 4:08 PM, Matthieu Bucchianeri @.***> wrote:

Can you attach a log as instructed in the Bug report template? Thank you.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.

Take a look at the troubleshooting guide: https://mbucchia.github.io/OpenXR-Toolkit/troubleshooting.html
Once you grab the log file, you can then reply here, and when you reply, you can add an attachment:

Thanks for the link. I’ll follow the instructions in the guide and respond tomorrow.
Sent from Steve Paradise's iPad

On Jan 29, 2022, at 6:26 PM, Matthieu Bucchianeri @.***> wrote:

Take a look at the troubleshooting guide: https://mbucchia.github.io/OpenXR-Toolkit/troubleshooting.html
Once you grab the log file, you can then reply here, and when you reply, you can add an attachment:
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.

I went through everything in the trouble shooting guide. I reset all the settings (Ctrl+F21+F2+F3), restarted in safe mode, log file is attached below, the layer is active BUT the …hand_tracking layer does not appear.  When I switch to VR mode (Ctrl-Tab) in MSFS the “press Ctrl-F2 to bring up the menu” message appears briefly but those keystrokes don’t open the menu.  I tried re-downloading and installing the software and that didn’t work either.
[OXRTK] 2022-01-29 19:16:21 -0500: dllHome is "C:\Program Files\OpenXR-Toolkit"
[OXRTK] 2022-01-29 19:16:21 -0500: XR_APILAYER_NOVENDOR_toolkit layer (Beta-2) is active
[OXRTK] 2022-01-29 19:16:21 -0500: Using OpenXR runtime Windows Mixed Reality Runtime 109.2111.23003
[OXRTK] 2022-01-29 19:16:21 -0500: Ignoring XR_MSFT_hand_interaction for Windows Mixed Reality Runtime 109.2111.23003
[OXRTK] 2022-01-29 19:16:36 -0500: dllHome is "C:\Program Files\OpenXR-Toolkit"
[OXRTK] 2022-01-29 19:16:36 -0500: XR_APILAYER_NOVENDOR_toolkit layer (Beta-2) is active
[OXRTK] 2022-01-29 19:16:36 -0500: Using OpenXR runtime Windows Mixed Reality Runtime 109.2111.23003
[OXRTK] 2022-01-29 19:16:36 -0500: Ignoring XR_MSFT_hand_interaction for Windows Mixed Reality Runtime 109.2111.23003
[OXRTK] 2022-01-29 19:16:36 -0500: Using Direct3D 11 on adapter: NVIDIA GeForce RTX 2080
[OXRTK] 2022-01-29 19:16:36 -0500: Using OpenXR resolution (no upscaling): 2832x2772
[OXRTK] 2022-01-29 19:16:36 -0500: Creating swapchain with dimensions=2832x2772, arraySize=1, mipCount=1, sampleCount=1, format=29, usage=0x21
[OXRTK] 2022-01-29 19:16:36 -0500: Swapchain image descriptor:
[OXRTK] 2022-01-29 19:16:36 -0500:   w=2832 h=2772 arraySize=1 format=27
[OXRTK] 2022-01-29 19:16:36 -0500:   mipCount=1 sampleCount=1
[OXRTK] 2022-01-29 19:16:36 -0500:   usage=0x0 bindFlags=0x28 cpuFlags=0x0 misc=0x0000020800000802
[OXRTK] 2022-01-29 19:16:36 -0500: Creating swapchain with dimensions=2832x2772, arraySize=1, mipCount=1, sampleCount=1, format=29, usage=0x21
[OXRTK] 2022-01-29 19:16:36 -0500: Swapchain image descriptor:
[OXRTK] 2022-01-29 19:16:36 -0500:   w=2832 h=2772 arraySize=1 format=27
[OXRTK] 2022-01-29 19:16:36 -0500:   mipCount=1 sampleCount=1
[OXRTK] 2022-01-29 19:16:36 -0500:   usage=0x0 bindFlags=0x28 cpuFlags=0x0 misc=0xFFFFFFFF00000802
[OXRTK] 2022-01-29 19:17:13 -0500: D3D11Device destroyed
[OXRTK] 2022-01-29 19:17:13 -0500: Session destroyed
Thanks
Steve Paradise
From: Matthieu Bucchianeri @.>
Sent: Saturday, January 29, 2022 6:26 PM
To: mbucchia/OpenXR-Toolkit @.>
Cc: PTSAGCY @.>; Author @.>
Subject: Re: [mbucchia/OpenXR-Toolkit] Can't open Open XR Toolkit in MSFS. (Issue #129)
Take a look at the troubleshooting guide: https://mbucchia.github.io/OpenXR-Toolkit/troubleshooting.html
Once you grab the log file, you can then reply here, and when you reply, you can add an attachment:
https://user-images.githubusercontent.com/69611560/151680866-98860b0c-5548-41cc-915e-6f839e332706.png
—
Reply to this email directly, view it on GitHub https://github.com/mbucchia/OpenXR-Toolkit/issues/129#issuecomment-1025006148 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AXQN4N5X4LNGIZDSZGPD7PDUYRZRRANCNFSM5NDISXMA .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/AXQN4NYT3J7CD5UU6RJAP43UYRZRRA5CNFSM5NDISXMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHUMFURA.gif Message ID: @.*** @.***> >

Are you sure your FlightSim window is in focus? And you are looking through the headset right and not just the FlightSim stereo window?

Yes and yes
Sent from Steve Paradise's  iPhone

On Jan 29, 2022, at 7:55 PM, Matthieu Bucchianeri @.***> wrote:

Are you sure your FlightSim window is in focus? And you are looking through the headset right and not just the FlightSim stereo window?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.

Just in case it's something odd with the keys in particular, can you use the Companion app and try to change the combination? Maybe make it Ctrl+2 instead of Ctrl+F2. This would be helpful to try.

Ok thanks. I’ll try it tomorrow.
Sent from Steve Paradise's  iPhone

On Jan 30, 2022, at 7:42 PM, Matthieu Bucchianeri @.***> wrote:

Just in case it's something odd with the keys in particular, can you use the Companion app and try to change the combination? Maybe make it Ctrl+2 instead of Ctrl+F2. This would be helpful to try.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.

Matthieu -
Thank you so much for working with me to solve this problem. Your latest suggestion worked. I changed the keystroke commands to CTRL + 1,2&3 and it is now working.
Now I have to experiment with the various settings recommended by “Pie in the Sky” in his excellent video to eliminate stuttering.
My system is not as robust as his - I7, 2080Fe, 64G. Any suggestions on which settings to dial back first  to eliminate stutters?
Thanks again for all your help. I really appreciate it.
Regards
Sent from Steve Paradise's  iPhone

On Jan 30, 2022, at 7:42 PM, Matthieu Bucchianeri @.***> wrote:

Just in case it's something odd with the keys in particular, can you use the Companion app and try to change the combination? Maybe make it Ctrl+2 instead of Ctrl+F2. This would be helpful to try.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.

Thank you. I suspect your keyboard might have an Fn key that needs to be pressed in order to access the F1/F2/F3 key.
For settings - there are many settings experts that can help you on the FS forums:
https://forums.flightsimulator.com/t/openxr-toolkit-upscaling-world-scale-hand-tracking-release-thread/493924
Or on the #settings room in Discord:
https://discord.gg/WXFshwMnke

Thank you Matthieu. I suspect you’re right. I’ll have to check. It’s a brand new Logitech and I’m not familiar with it yet.
Thanks again for all your help.
Sent from Steve Paradise's  iPhone

On Jan 31, 2022, at 8:45 PM, Matthieu Bucchianeri @.***> wrote:

Closed #129.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.

Hi Matthieu
I believe I read somewhere that there is a known issue with text in menus
being garbled in VR.  I have this problem.  Many of my menus (bottom of main
screen menus, controls menus) are unreadable or very unclear.  Is this a
known issue and do you have an estimate for when it will be remedied?  The
problem only occurs in VR.
Thanks for all your hard work in bringing us this incredibly valuable tool.
Regards,
Steve Paradise

"Many of my menus (bottom of main screen menus, controls menus) are unreadable or very unclear."?
Are you talking about the OpenXR Toolkit menu?

Thanks for the quick response.
No, toolkit menu is clear. It’s the bottom of the Home Screen titles and the text in the controls menu to a lesser extent. Scenery and aircraft panels are fine, it’s the menus that are a problem and mostly the bottom left portion. It’s only in VR arhat this is happening.
Sent from Steve Paradise's  iPhone

On Mar 25, 2022, at 11:51 AM, Matthieu Bucchianeri @.***> wrote:

"Many of my menus (bottom of main screen menus, controls menus) are unreadable or very unclear."?
Are you talking about the OpenXR Toolkit menu?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.

it’s the menus that are a problem and mostly the bottom left portion.

There's probably not much we can do here - elements in the peripheral vision are naturally going to be lower resolution/clarity in VR due to lens distortion. The only fix for this is to buy a more expensive VR headset I fear... :( or I hear for G2 there is a FOV mod (a physical part replacement) that I hear can help but I have not tried it myself.

I’ll check into that, but it’s not peripheral vision because it stays blurry when I look directly at it.
Sent from Steve Paradise's  iPhone

On Mar 25, 2022, at 1:44 PM, Matthieu Bucchianeri @.***> wrote:

it’s the menus that are a problem and mostly the bottom left portion.
There's probably not much we can do here - elements in the peripheral vision are naturally going to be lower resolution/clarity in VR due to lens distortion. The only fix for this is to buy a more expensive VR headset I fear... :( or I hear for G2 there is a FOV mod (a physical part replacement) that I hear can help but I have not tried it myself.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.

I see. I'm not sure what you mean, but we certainly don't have a magic cure for that unfortunately. The issues you have read about were related to the OpenXR Toolkit menu itself, and these issues are resolved now.

I tried to send you a couple of emails with screen shots of the menus but it got kicked back because the file sizes were too large. Is there another email address I can send them to?
I also have a new maxed out Falcon Talon coming in a few weeks so I’m hoping that might solve the problem.
Sent from Steve Paradise's iPad

On Mar 26, 2022, at m12:11 AM, Matthieu Bucchianeri @.***> wrote:

I see. I'm not sure what you mean, but we certainly don't have a magic cure for that unfortunately. The issues you have read about were related to the OpenXR Toolkit menu itself, and these issues are resolved now.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.


Hotfix/flags

Description
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Change Status

[x ] Complete, tested, ready to review and merge
[ ] Complete, but not tested (may need new tests)
[ ] Incomplete/work-in-progress, PR is for discussion/feedback

How Has This Been Tested?

[ ] Test A
[ ] Test B

Checklist

[ x] My code follows the style guidelines of this project
[ x] I have performed a self-review of my own code
[ ] My code has been reviewed by at least one peer
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ x] There are no merge conflicts

Updated ui on welcome storyboard and main storyboard. Cleaned up some constraint issues on the login page that weren't letting some devices login. Added validation for sign up for each text field. Updated progress tint for requirement cells and bar to better match web. Deleted unused folders and classes for cleaner structure. Followed commands to delete features that weren't in PVD.

Tested on simulator and iPhone 7-Plus successfully.

@vicxruiz , I was able to pull down your branch and test it on my local machine without any build errors. I saw everything on my local machine's simulator that you described in your comment above. The only things I noticed were: (1) on my home screen, I had some trouble knowing where to click to get to the "steps" page for each requirement card because there were no clearly defined borders/shadows on each card, (2) when I navigated to the "steps" page for the 'mock interview' requirement card, upon toggling the checkmark "ON" for a checkbox, I noticed that it resized the card in an odd way that produced extra whitespace. I wasn't sure if those two things were meant to be that way, but I figured that I would at least alert you! Other than those two things, everything else looked great! Nice work, man!

@vicxruiz, I noticed that the semver tag was not increased from "v1.1.1" to "v1.1.2" so I just went ahead and did that. See here => https://youtu.be/9tkAPA7DLOA


Calling DefaultKafkaHeaderMapper constructor with the same ObjectMapper instance may result memory leak

In what version(s) of Spring for Apache Kafka are you seeing this issue?
2.8.2
Describe the bug
DefaultKafkaHeaderMapper is registering a SimpleModule in constructor to the given ObjectMapper instance. Since the SimpleModule's are unnamed modules, IGNORE_DUPLICATE_MODULE_REGISTRATIONS configuration of ObjectMapper doesn't handle multiple registration of this module. This is problematic and can cause a memory leak if the DefaultKafkaHeaderMapper(ObjectMapper mapper) constructor is called with the same ObjectMapper instance multiple times.
To Reproduce
Call the DefaultKafkaHeaderMapper(ObjectMapper mapper) function with same ObjectMapper instance multiple times.
Expected behavior
A new SimpleModule would be registered to the ObjectMapper beacuse of the new SimpleModule().addDeserializer(MimeType.class, new MimeTypeJsonDeserializer()) call.
Solution
This problematic situation can be avoided simply by giving a name to the SimpleModule created as below:
this.objectMapper.registerModule(new SimpleModule("mimeTypeJsonDeserializer", Version.unknownVersion())
						.addDeserializer(MimeType.class, new MimeTypeJsonDeserializer()));

Thanks for pointing this out, but it would be rather unusual to create multiple header mappers.
Contributions are welcome.

Yeah... That's why I always say that it is better to not mutate an externally provided objects:
		this.objectMapper = objectMapper;
		this.objectMapper
				.registerModule(new SimpleModule().addDeserializer(MimeType.class, new MimeTypeJsonDeserializer()));

That's a good point; that code should only be applied to an internally created OM.

Maybe it can now be removed altogether (at least in 3.1); it was introduced a long time ago for backwards compatibility with old, unsupported, versions.
https://github.com/spring-projects/spring-kafka/issues/892

I guess so. Since it is an internal API we probably can remove it without any side-effects.


But does Vespene run on gas?

Sorry, I had to. :satisfied:

Thanks for stopping by! I'm not sure what the Vespene extractor runs on, it never seems to reduce the amount of gas available.


Path issue in Windows

Don't know if this is an issue with the Bourbon example, brackets-sass, or libsass but compiling is broken when the project files are located on my D drive.
This is what app.css.map looks like:
{
  "version": 3,
  "file": "app.css",
  "sources": [
    "C:/D:/bourbon-example-master/scss/app.scss",
    "C:/D:/bourbon-example-master/bower_components/bourbon/dist/_bourbon.scss",
    "C:/D:/bourbon-example-master/scss/partial.scss",
...

Previously reported in issue https://github.com/jasonsanjose/brackets-sass/issues/21#issuecomment-64128971

Same here.
Found out that the issue was caused by this code.
Unfortunately I'm not really experienced with Node.js so I don't know how to debug Node.js code to fix that :
console.log() doesn't work here.

Fixed: #99

FWIW the path that we get from the source map returned from node-sass will typically look like this:
../../../../../../../../../../D:/bourbon-example/scss/partial.scss

The bug happens when the input file and some included path are on different drives.

Closing. Targeting v2.

See test build here https://github.com/jasonsanjose/brackets-sass/pull/100#issuecomment-76787328 with path fixes for both libsass and ruby. Targeting for v2 release.

Re-opening since PR #100 hasn't been merged yet.

@tijtij there's a test build you can try here https://github.com/jasonsanjose/brackets-sass/releases/tag/2.0.1-105. If you have some time, please let me know if that fixes your problem.

@jasonsanjose Works for me on Brackets 1.20-15697.
None of sources paths in *.css.map are using drive letters.
Thanks for the fix.


Spin and Magnus force

Hi,
I'm a researcher and I'm trying to use MuJoCo for the physics simulation of a sport's ball (e.g. in soccer or basketball).
I want to take into account that the Magnus force is acting on a spinning ball and I would like to know how to incorporate this effect in the MuJoCo simulation.
At the moment I do a simple simulation defined by this XML file:
<mujoco>
      <worldbody>
        <!-- Define the ball -->
        <body pos="5.425 0 1" name="ball">
            <joint type="free" name="ball_joint"/>
            <geom type="sphere" name="ball_geom" size="0.11" mass="0.43" pos="0 0 0" />
        </body>
        <!-- Define the field  -->
        <geom type="plane" name="field" size="200 200 0.1" pos="0 0 0" solref="-1000 0"/>
      </worldbody>
      <!-- Enable gravity for the simulation -->
      <option gravity="0 0 -9.81" integrator="RK4" timestep="0.002" density="1.2" viscosity="0.00002" />
    </mujoco>

By setting the density and viscosity I hoped to incorporate the Magnus effect into the simulation. However, according to the docs, it seems like density and viscosity only affect the drag (btw: this way it is also not possible to set some constants for different surfaces, since some surfaces cause a larger force) and no Magnus effect is introduced. Is this correct?
So do I instead have to use fluidshape=ellipsoid in order to describe a spinning ball correctly?

| So do I instead have to use fluidshape=ellipsoid in order to describe a spinning ball correctly?
Yes


Remove module in hack/tools

Description of changes:
I believe this is not needed anymore. It didn't hurt but also it's extra work when bumping go versions.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm
/approve

/override eks-anywhere-e2e-presubmit


docs: add @property to Time

Description

add @property

Checklist:

[x] Securely signed commits
[x] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

@paulbalandan Why cannot we use @property-read?
The phpdoc_no_alias_tag rule replaces it.
https://cs.symfony.com/doc/rules/phpdoc/phpdoc_no_alias_tag.html


[BUG]: PublishSymbols@2 doesn't find all files match the search pattern

New issue checklist

[x] I searched for existing GitHub issues
[x] I read pipeline troubleshooting guide
[x] I checked how to collect logs

Task name
PublishSymbols@2
Task version
No response
Issue Description
I used PublishSymbols@2 with searchPatters "**" try to match all files in the symbol directory. The directory structure is

When I use "**" to match all files, it can't find files in x64 and x86 directory. There are 2 dll files in x64 and 1 dll file in x86.
Move x86 and x64 into tools directory also doesn't work.

It will skip x64 and x86 directory. However, if I move them into native directory, suddenly task can find files in native/x64 and native/x86.
if set the searchpattern to "x64/.dll", task can find the dll files in x64. However, if set the searchPattern to
"x64/.dll
**"
x64 was still skipped.
Environment type (Please select at least one enviroment where you face this issue)

[ ] Self-Hosted
[x] Microsoft Hosted
[ ] VMSS Pool
[ ] Container

Azure DevOps Server type
dev.azure.com (formerly visualstudio.com)
Azure DevOps Server Version (if applicable)
No response
Operation system
Windows
Relevant log output
No error log

Full task logs with system.debug enabled
No response
Repro steps

Closed. The reason probably is publishsymbols is doing deduplication.


Frontend first step

React/typescript app creation
basic move and renaming of files to ease later comparison

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 48.853%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
2746

Relevant Lines:
5621

💛  - Coveralls


Code 128 Barcode Accuracy

I've been testing out this project for using to read barcodes on USPS shipping labels, but I've noticed that the results are sometimes very, very wrong and sometimes are only slightly wrong. Since the result is a 30 character string, it's hard to spot the subtle incorrect ones without going through the entire string, which defeats the purpose of scanning the barcodes.
I dug into the code and found that it is correctly using the checksum (yay) on the Code 128 barcodes, but is still coming up with the wrong result because there are multiple ways to get to the same checksum value.
I've gotten more accurate results by adding a few checks to Code128Reader: I added a final check that no individual code has an error above X, and a check that the checksum code has an error below Y. The rationale behind that is if the checksum is not trustworthy (high error) then it will end up OKing an incorrect value for the barcode.
With those tighter checks the incidence rate for incorrect values goes down. The tradeoff is it takes longer to scan the barcode because the correct spot must be found to get it to read. However, it is basically rejecting incorrect values until it ends up with the correct one, which is what I want. Using a higher resolution (and closer focus) webcam yields better results.
Is it possible to make these configurable into quaggaJS so that they can be tweaked via the config and not need a custom version? I see some hint of configs being supported for readers but no way to provide one when configuring quaggaJS.

Thanks for reporting on that issue. I've also had some troubles with false positives on Code 128 barcodes. Those bugs are usually hard to track, that's why I haven't really had the time to dig deeper.
As soon as I can find myself some free-time, I'll look into that. Do you have a few samples where these problems can be reproduced?

Here is a sample label that produces bad results. When printed so the barcode is around 4 inches wide it will result in about 30% bad results using the live demo and the webcam on a Macbook Pro,

I'm continuing to play with ways to get better accuracy when reading, and have abandoned the original technique mentioned because it was preventing matches from occurring at all.
So far I'm having good results with only accepting a match if it is read 3 times. It seems most extraneous results are unlikely to occur twice, let alone three times, so if it read the same result 3 times then it is likely the correct result. This leads to a high accuracy, but it may not detect at all.
Here's some output from logging which logs the possible results (that got all the way past the checksum check) and how many times they occur:
Object {)(~,zg7M]8Mc3: 1,<PHONE_NUMBER>99907123456123456781: 3}
Object {630957479499907120456120066781: 1,<PHONE_NUMBER>99907123456123456781: 3,<PHONE_NUMBER>9499907123456124457819: 1}
Object {420957479499907123456123456781: 3,<PHONE_NUMBER>99907123696123450580: 1,<PHONE_NUMBER>9499907123456120456781: 1,<PHONE_NUMBER>99907123696123456781: 1}
Object {630357089412177102456120060542: 1,<PHONE_NUMBER>99907123456123456781: 3}

Another possible technique (but more complex) would be to keep track of the 'best match' for each individual code and weight it by the number of times it has appeared. So if a code is read as 98 with a 0.15 error twice, and read once as 42 with a 0.10 error once, the algorithm would choose the 98 (weighted error of 0.075). However, that's more complex because you need to recheck the checksum, and you may end up with a case where the best candidate codes do not create a result which passes the checksum.

In the past few days I've been busy testing the robustness of the Code 128 reader and making some adjustments to the parameters as well as other corrections to reduce the occurrence of false positives. Could you please try again and close this ticket in case it solves your issue?

@serratus, sorry, I didn't see your response until just now. Assuming the live demo is running this latest version, I still get a lot of false positives from it. Holding up a sheet of paper with the sample image I provided will run through a dozen or so different false positive results.

Alright, thanks for the feedback. I'll give it another try in the next few days.

I occasionally get false positives with code128 as well. Usually this happens when the barcode is not lined up properly or if the whole barcode is not visible within the camera.

I like your approach @dsanders11, I'll implement that as I'm experiencing the same issues when playing around with the live demo and I was wondering what approach could be made to make sure we've got the right data to check against.


Fix 3 issues with route table associations

Fixes https://github.com/spulec/moto/issues/5508 - replacing the main association raising an exception.
Fixes https://github.com/spulec/moto/issues/5507 - showing all route tables as having the main association.
Fixes https://github.com/spulec/moto/issues/5506 - the response for replace missing bits.
Existing unit tests fixed and a new one (testing replacing the main
association) added.

Already in via https://github.com/spulec/moto/pull/5509


Dockstore CLI WES --debug mode support

Is your feature request related to a problem? Please describe.
Because the Dockstore CLI can interact with any number of different WES servers, each with their own unique implementation oddities, it would be good to support a debug mode specific to WES calls. This would help with investigating issues that may be WES-server specific or config related.
Describe the solution you'd like
Add a flag, perhaps --debug to WES calls that will print a large amount of additional information, such as:

WES URL
What authorization type was detected
If an authorization value was detected (Probably don't want to show it in the terminal)
A breakdown of the HTTP request that was made (Mainly, for the launch command, we can print out details for each of the WES parameters, such as the workflow URL, attachments, test parameter files, etc...)
I think the WES parameter workflow_engine_parameters supports what mode the WES server executes your workflow in, we could also modify this to run in debug mode on the WES server.
More?

The CLI currently prints out other strings during various WES commands, so those could be moved into the debug mode instead.
Describe alternatives you've considered
Nothing.
┆Issue is synchronized with this Jira Story
┆epic: Dockstore CLI AGC support
┆friendlyId: DOCK-2056
┆sprint: Backlog
┆taskType: Story

For implementer: Currently, the Dockstore CLI does not print out values during WES commands that lend to easy programmatic use. When implementing the --debug flag, anything that is printed out normally (i.e. without --debug) should be easily worked with.
For example, when launching a workflow, just print out the workflow's runId, and no other content. Or, provide pure JSON object responses so tools like jq can be used alongside them. Depending on implementer, this can be split into another feature ticket.


call pageadapter notifydatasetchange will not update the tab layout.

Hi, @jiqimaogou
Thank you for reporting.
It is not supported in the current version.
So, please manually call the SmartTabLayout#setViewPager(...) again.
I'll try to implement in the near future :)

ok, thank you


How to work with pattern match?

I tried the following methods but none of them worked, how should I use pattern matching?
fn fibonacci_u(n: BigUint) -> BigUint {
    match n {
        BigUint::from(0) | BigUint::from(1) => n,
        _ => fibonacci_u(n.clone() - BigUint::from(1)) + fibonacci_u(n - BigUint::from(2))
    }
}

fn fibonacci_u(n: BigUint) -> BigUint {
    let zero: BigUint = BigUint::from(0usize);
    let one: BigUint = BigUint::from(1usize);
    let two: BigUint = BigUint::from(2usize);
    match n {
        zero | one => n.clone(),
        _ => fibonacci_u(n.clone() - one) + fibonacci_u(n - two)
    }
}

const ZERO: BigUint = BigUint::from(0usize);
const ONE: BigUint = BigUint::from(1usize);
const TWO: BigUint = BigUint::from(2usize);

fn fibonacci_u(n: BigUint) -> BigUint {
    match n {
        ZERO | ONE => n,
        _ => fibonacci_u(n.clone() - ONE) + fibonacci_u(n - TWO)
    }
}

At the language level, pattern matching only works with primitive literal values and structural decomposition. Given that BigUint has private fields, and those have even further privacy in Vec fields, you can't directly match BigUint with values.
You can always use plain if conditions like n == zero || n == one or n <= one. With the Zero and One traits, this can also be n.is_zero() || n.is_one(). If you really want to use patterns, you can do that with an interim conversion to a primitive type using TryInto or ToPrimitive:
match n.to_u32() {
    Some(0) | Some(1) => n,
    _ => todo!(),
}

This is a good usage example, thanks for your help

I too came across this issue, where I found the BigUint always matched on the first branch of the match statement.   As described above, using if statements resolved the issue; but I also found that match statements with an explicit comparsion worked fine too:
let v = match n {
    n if n == zero => zero,
    n if n == one => one,
    _ => _fibonacci(&n - &one, cache) + _fibonacci(&n - &two, cache),
};

...where n is of course a BigUint.
This was a little nicer than a tree of if statements in my particular situation.


fix: Disconnecting evm wallet

Thank you for your contribution to the Koda - Generative Art Marketplace.
👇 __ Let's make a quick check before the contribution.
PR Type

[x] Bugfix

Needs Design check

@exezbcz please review

Needs QA check

@kodadot/qa-guild please review

Context

[x] Closes #10635
[ ] Requires deployment <worker/stick/speck>

Did your issue had any of the "$" label on it?

[x] My DOT address: Payout

Screenshot 📸

[ ] My fix has changed something on UI; a screenshot is best to understand changes for others.

I can't reproduce the bug on my side, but I tried to mock the possible case and fix it.
@exezbcz Could you test if the fix works for you? Btw, what evm wallet are you using?

Btw, what evm wallet are you using?

Guess you can replace all your wallets with Talisman :)

yup much faster, thanks

i use metamask, the bug was there when connecting polkadot wallet, then disconnect, connect evm and disconnect


Tensor.nonzero_static fails on GPU inside torch.func.vmap

def foo(a:torch.Tensor, b:torch.Tensor):
    bool_mat = (a & b)
    print(bool_mat)
    return bool_mat.nonzero_static(size=bool_mat.shape[0])
a = torch.zeros((5,5), dtype=torch.bool, device="cuda")
b = a.clone()
res = torch.func.vmap(foo)(a, b)

ErrorCode:
NotImplementedError: Could not run 'aten::nonzero_static' with arguments from the 'CUDA' backend. This could be because the operator doesn't exist for this backend, or was omitted during the selective/custom build process (if using custom build). If you are a Facebook employee using PyTorch on mobile, please visit https://fburl.com/ptmfixes for possible resolutions. 'aten::nonzero_static' is only available for these backends: [CPU, Meta, BackendSelect, Python, FuncTorchDynamicLayerBackMode, Functionalize, Named, Conjugate, Negative, ZeroTensor, ADInplaceOrView, AutogradOther, AutogradCPU, AutogradCUDA, AutogradHIP, AutogradXLA, AutogradMPS, AutogradIPU, AutogradXPU, AutogradHPU, AutogradVE, AutogradLazy, AutogradMTIA, AutogradPrivateUse1, AutogradPrivateUse2, AutogradPrivateUse3, AutogradMeta, AutogradNestedTensor, Tracer, AutocastCPU, AutocastCUDA, FuncTorchBatched, BatchedNestedTensor, FuncTorchVmapMode, Batched, VmapMode, FuncTorchGradWrapper, PythonTLSSnapshot, FuncTorchDynamicLayerFrontMode, PreDispatch, PythonDispatcher].

nonzero_static is not working on CUDA device, there is also a warning on CPU.
It would be interresting to implement this operation as it is the easiest placeholder of the classic nonzero() that is not supported in vmap for its dynamic range.

#256

#256


Mill stuck for genIdea after I canceled it with Ctrl + C

After I canceled it with Ctrl + C and retry gen idea, it got stuck, only work after I clean it with mill clean.
The project is fastparse.

Can you reproduce it? Can you reproduce it with -i option?

@lefou may be the network issue, cannot reproduce it now, let me close it for now.


Website Redesign Review - Case Study

Review designs for Case Study Page

[x] @jpandersen87 Reviewed and Signed off
[x] @penny-lischer Reviewed and Signed Off
[x] @etanb Reviewed and Signed Off

@yudonoodles @jillian-hammer FYI - Reviews are completed from engineering.