Texts isn't rendering correctly on Windows

We are using rich to shows rich and colored text on console but it isn't working properly on Windows.
I have turned on unicode support on my windows. Though on windows it looks like following:
┌────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┐
│                                                                CVE BINARY TOOL                                                                 │
└────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┘

 • cve-bin-tool Report Generated: 2020-07-03  17:58:41
┌───────────────┐
│ NewFound CVEs │
└───────────────┘

but on linux or modern Windows terminal it renders correctly:
╔══════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════╗
║                                                                     CVE BINARY TOOL                                                                      ║
╚══════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════════╝

 • cve-bin-tool Report Generated: 2020-07-03  17:39:48
╭───────────────╮
│ NewFound CVEs │
╰───────────────╯

On Windows github actions it is even unable to render in ASCII (Here is how columns of panel displays on Windows github actions):
\u250c\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2510 \u250c\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2510
\u2502CVE-2018-5729\u2502 \u2502CVE-2018-5730\u2502
\u2514\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2518 \u2514\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2518

On linux everything renders perfectly even in github actions.

You might want to send me a screenshot, because the Windows version you posted above looks fine. Unless its the lack of double borders you are referring to?
The GH Actions on Windows I've seen before, and can be fixed by setting the environment var PYTHONIOENCODING="utf8" .

Yes I am referring to bold borders and rounded box. I have viewed the other issue where you have mentioned to use legacy_windows=False on Console constructor and it is working properly now.

The double borders and rounded borders don't render on legacy windows when you have raster fonts selected, but work fine with truetype fonts. Unfortunately there is no way to detect which font you are using so they are disabled by default on legacy windows.
Detecting legacy windows is not perfect, it looks at env vars. Here's the code:
def detect_legacy_windows() -> bool:
    """Detect legacy Windows."""
    return "WINDIR" in os.environ and "WT_SESSION" not in os.environ

Works fine with terminals on your desktop, but may fail on other environments like Github Actions. I haven't found a better way unfortunately.
What config did you use for the screenshot above? There are some blank lines in the top box I can't explain.

I am using powershell with utf-8 support turned on. I am using furacode (powerline patched firacode) truetype font.

I guess there can be some registry value that stores which font is being set as default for console.

I guess there can be some registry value that stores which font is being set as default for console.

Good point, but tbh I'm not sure I want to go to such lengths. The fallback doesn't look too bad, and its not unreasonable to sacrifice a little visual appear when running on an ancient terminal. If you are sure you'll never use raster fonts you can set safe_box=False on the Panel and Table classes.
I wonder if the box characters on that font aren't truly the same size as other characters. Would you mind trying the following, and tell me if both lines look the same width.
>>> print("═"*50+"\n"+"W"*50)

Ah wait, so in that screenshot you had legacy_windows=False ?
Rich subtracts 1 from the terminal width on legacy windows. This is because a new line character at the end of the line is treated as having a width of 1 and wraps the line (on other terminals it is treated as 0 width).
If you really really want all the box options, setting safe_box=True is your only option at the moment.

Yes both looks same width.
══════════════════════════════════════════════════
WWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWW

I guess there can be some registry value that stores which font is being set as default for console.

I have found registry path that stores current console fonts of current user: Computer\HKEY_CURRENT_USER\Console
For cmd it is under: Computer\HKEY_CURRENT_USER\Console\%SystemRoot%_System32_cmd.exe
and
For powershell 32bit it is under: Computer\HKEY_CURRENT_USER\Console\%SystemRoot%_System32_WindowsPowerShell_v1.0_powershell.exe
For powershell 64 bit it is under:
Computer\HKEY_CURRENT_USER\Console\%SystemRoot%_SysWOW64_WindowsPowerShell_v1.0_powershell.exe
Here, fontface contains name of the font and there is also some meta data about fonts under this registry path.
If somehow we can know current terminal is powershell or cmd and font set is truetype or raster than we can surely tell that texts is going to render or not.

If somehow we can know current terminal is powershell or cmd and font set is truetype or raster than we can surely tell that texts is going to render or not.

It would require knowing in advance which fonts render and which don't. You might be right most of the time, until you find one user who installs a custom raster font. I also suspect that those registry keys aren't guaranteed to be the same in the future (v1.0 is a clear indication of that).
Since you can't know with 100% confidence, I think it is safer to disable by default.

Ah wait, so in that screenshot you had legacy_windows=False ?
Rich subtracts 1 from the terminal width on legacy windows. This is because a new line character at the end of the line is treated as having a width of 1 and wraps the line (on other terminals it is treated as 0 width).
If you really really want all the box options, setting safe_box=True is your only option at the moment.

Where am I supposed to add that safe_box=True, As I can see neither Markdown nor Console have optional kwargs safe_box ?
Here is my code:
        console.print(
            Markdown(
                textwrap.dedent(
                    f"""
                    # CVE BINARY TOOL
                    - cve-bin-tool Report Generated: {now}
                    """
                )
            )
        )

It's on the Panel class, used by Markdown. It's not exposed via the Markdown constructor.

I'll add a parameter to the Console constructor in the next update.

I've added a safe_box parameter to the Console constructor in v3.0.3

Thank you!

I have installed latest rich but Panel is still not rendering as expected on Windows. I am still getting those extra lines.
After some tinkering, I get to know that if we keep expand=False, or True with custom width (smaller than max width of window) then Panel will render correctly but if expand is True like in case of Markdown then it will show extra lines.

I can't reproduce that here. I'll need you to help me find out what is different.
How have you configured the Console? Have you set legacy_windows?
What version of Windows? What terminal?
Does the problem rectify itself with a different font?
What happens when you run this?
from rich.console import Console
c = Console()
print(c.size)
import shutil
print(shutil.get_terminal_size())

What does this give you?
python -m rich.panel

Yes I have set legacy_windows=False, safe_box=True and
from rich.console import Console
c = Console()
print(c.size)
import shutil
print(shutil.get_terminal_size())

produces
ConsoleDimensions(width=79, height=24)
os.terminal_size(columns=80, lines=24)

output of panel module
❯ python -m rich.panel
<__main__.Panel object at 0x000001A320D61BC8>
┌────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┐
│┌────────────────────────────┐                                                                                │
││                            │                                                                                │
││        Hello World!        │                                                                                │
││                            │                                                                                │
│└────────────────────────────┘                                                                                │
└────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┘

I believe problem is due to incorrect console dimention.

If you're on legacy windows, the width needs to be reduced by 1. By forcing legacy windows to be False, this isn't happening, which is why you are getting those blank lines.

If I keep legacy_windows=None then newlines aren't being printed but rounded box is being replaced by normal box.

Can there be any intermediate solution that let windows user get benefit of rich text if it's supported in his system?

Set safe_box=False if you are happy that Windows users with raster fonts may see broken visuals.

Can there be any intermediate solution that let windows user get benefit of rich text if it's supported in his system?

cmd.exe is over 30 years old. The default setting in Rich ensure the output looks not bad, but you can't expect the same results as more modern software.
Use the new Windows Terminal and you'll have the full experience.
There really isn't any more to say on this issue.


.NET Runtime version 4.0.30319.0 - The profiler was loaded successfully

1022
0
0
Keywords

-1089492812
Application
RD00155D2AA144

.NET Runtime version 4.0.30319.0 - The profiler was loaded successfully. Profiler CLSID: '{324F817A-7420-4E6D-B3C1-143FBED6D855}'. Process ID (decimal): 5544. Message ID: [0x2507].

What is the issue here?


Turbolinks disabled for POST requests that redirected via GET

I am using a new Rails 4.0.2 application, and the turbolinks gem.
When I create/update a RESTful resource, Turbolinks are disabled even if the resource was successfully created. For any other GET requests, Turbolinks work just fine.
This is my controller code:
def create
  @comment = Comment.new(comment_params)

  if @comment.save
    redirect_to @comment, notice: 'Comment was successfully created.'
  else
    render action: 'new'
  end
end

Instead of redirecting to the comment via turbolinks, it does a full loading of the page.

Take a look at this gem: Turboboost


Test requirements not documented

pandas is now required for test/test_utils.py but is not included in requirements-dev.txt like xarray is.
certifi is required for test/test_network.py but is only included in install_requires in setup.cfg.

pandas is now required for test/test_utils.py but is not included in requirements-dev.txt like xarray is.

That sounds like a good idea to be more explicit.

certifi is required for test/test_network.py but is only included in install_requires in setup.cfg.

Where would be the most helpful location to put the requirements?

I would personally also recommend to use those optional test dependencies behind a pytest "skip if not installed" (you can use eg pytest.importorskip, to be read as "import or skip")

Where would be the most helpful location to put the requirements?

tests_require or extras_require: {'test': []} in setup() and/or requirements-tests.txt are likely good options.

@sebastic, what about certifi?

As certifi is not only required for tests install_requires is probably sufficient.


User Story 3: Gene Expression and Diff Exp should work on spatial mode

Context:
User Story 3:
As a data consumer, I want to easily compare gene expression data from different spatial regions within the same tissue image, so that I can identify unique or shared expression patterns.
DoD:

Gene Expression and Diff Exp should work on spatial mode
Add corresponding e2e tests

Hey team! Please add your planning poker estimate with Zenhub @kaloster @seve

I'm pointing with the assumption that it'll just work after User Story 1 is done, so this ticket is really just to verify that the feature does indeed work

e2e test covered in https://github.com/chanzuckerberg/single-cell-explorer/pull/854


added some tabline plugins

Sorry but both of these are already in the list.

Oh sorry I didn't read it.


Fix Pdop and Sdop shorthands

The previous code was nonsense - len(args[0]) == 1 and isinstance(args[0],Symbol) was never true, as Symbol.__len__ is undefined.
This also makes it illegal to pass extra ignored arguments to Pdop.
Finally, this deprecates Pdop({}, ga=ga) as an alias for Pdop({}, ga=ga) - there's no reason to have two ways to spell the same thing, and the second is more natural.

@utensil, I made a typo in the commit message here - don't merge after you approve, I'll merge myself once you sign off


Add override specifier to inherited library class functions

What does this Pull Request accomplish?
Add the override specifier to library class functions that exist in the interface class it inherits from.
Why should this Pull Request be merged?
Although we own the parent class, it's generally good practice to use override when you override a method so that you get a compiler error if there's no function to override (usually because the signature of the function in the parent class changes).
What testing has been done?
Built it locally

Carrying forward with this in #995 to get past merge conflicts and see if the tests are looking better now.

Abandoning this one in favor of the new one (#995)  based on newer version of main.


[Feature Request] Allow special characters for the column name

Feature request
Overview
Currently special characters like [ ,;{}()\n\t=] are not allowed for column name when writing the delta table. Is there special concern for this?
Motivation

Further details

Willingness to contribute
The Delta Lake Community encourages new feature contributions. Would you or another member of your organization be willing to contribute an implementation of this feature?

[ ] Yes. I can contribute this feature independently.
[ ] Yes. I would be willing to contribute this feature with guidance from the Delta Lake community.
[ ] No. I cannot contribute this feature at this time.

Thanks, this makes sense


Load map in custom coordination

Hi dear
I want to load map in custom zoom or location
I add world shale layer to map 1 on form load i want when form load map view in custom location or custom zoom
How can i do it ?

Dim exts As Extent = New Extent(10, 45, 15, 55)
map.ViewExtents = exts
~agp


Unit tests for essential API requests

Keep track of things breaking at either end. To consider:

these spurious requests will need to be identified and discarded to avoid polluting the dataset

See https://github.com/OAButton/backend/issues/165#issuecomment-232775024

This can be closed as not relevant to the extension. API is tested via the backend. Plugin can do nothing about them being unavailable anyway, so no use unit testing them within the plugin.


Support for New NodeJS 4.3.2 Lambda API

I am having issues running tests using the new Lambda API.  The error I see is
TypeError: cb is not a function
It appears this may be due to using an older version of lambda-wrapper.
I am doing something wrong or does lambda-wrapper version need to updated?

You are correct about the lambda-wrapper version.  I just cloned https://github.com/SC5/lambda-wrapper, and the latest version is 0.0.6 - Just link it to this plugin, etc and you should get past that error.

Ok, but shouldn't this be updated in the package.json of the plugin?  How
would I go about linking to a different version?
On Wed, May 4, 2016, 7:09 PM Christopher Lam<EMAIL_ADDRESS>wrote:

You are correct about the lambda-wrapper version. I just cloned
https://github.com/SC5/lambda-wrapper, and the latest version is 0.0.6 -
Just link it to this plugin, etc and you should get past that error.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/SC5/serverless-mocha-plugin/issues/9#issuecomment-217044081

To link do the following:

Clone the lambda-wrapper repo, it will have the latest changes.  cd into that directory run "npm link"
Clone the serverless-mocha-plugin. cd into that directory and run "npm link lambda-wrapper" then run "npm link" in the same directory.
Go to your "serverless" project, the one you are deploying, not the framework.  Double check your package.json to see if the mocha-plugin is in your deps, not necessary, but more of a sanity check you are in the right directory/project.  run "npm link serverless-mocha-plugin"

Hope I put the correct info down in the above steps, as you can see there are a few of them.

Ok, thanks for the information.  Hopefully, we can also get the version
updated so we don't require the extra steps.
On Wed, May 4, 2016, 8:52 PM Christopher Lam<EMAIL_ADDRESS>wrote:

To link do the following:

Clone the lambda-wrapper repo, it will have the latest changes. cd
into that directory run "npm link"
Clone the serverless-mocha-plugin. cd into that directory and run
"npm link lambda-wrapper" then run "npm link" in the same directory.
Go to your "serverless" project, the one you are deploying, not the
framework. Double check your package.json to see if the mocha-plugin is in
your deps, not necessary, but more of a sanity check you are in the right
directory/project. run "npm link serverless-mocha-plugin"

Hope I put the correct info down in the above steps, as you can see there
are a few of them.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/SC5/serverless-mocha-plugin/issues/9#issuecomment-217056779

The dependencies have now been fixed to point to newer lambda-wrapper versions and a new package (0.5.3) has beem published.


Windows 10 client cannot be found under 'Nearby devices'

Describe the bug
I have localsend on android, MacOS and Windows 10.
But I can't send anything to Windows because it is not shown under sending. Also using IP adress doesnt work.
Only downloading over browser works.
Receive port is running. but had to set it to a bit higher port than the standard port because the standard port was already reserved.
Dont know why Windows client cannot be seen by Android and by Mac
Is there something like an error log?
Are there specific firewall settings needed to be done?
Anything else?
To reproduce

Go to 'Send'
Click on 'Nearby devices' or 'Ip address'
Windows 10 device not shown

Expected behavior
Windows client should be shown
Screenshots
No response
Desktop (please complete the following information)
No response
Smartphone (please complete the following information)
No response
Additional context
No response

Hi @wadid , thanks for opening this issue. If you changed the port on one of your devices, you need to change it as well on all the other devices you want to share files with. We are aware that it is quite inconvenient and are working on a better streamlined solution.
Could you try this and comeback with a reply if it succeeded/failed ?

I'm having the same problem. Same configuration: Mac/Android/Win10. Win10 installation does not appear to other devices, and vice versa. All devices have matching ports.
LocalSend v1.14.0 (45) on Windows.

Hi,
Can you try using the Autofix functionality?
You can find it under Send > Troubleshoot > Fix automatically.
!https://cdn.discordapp.com/attachments/1106328604687728643/1251726160530772069/image.png?ex=6672ec14&is=66719a94&hm=27b3d5458baf1a4f399d5f5475c191efc87683acc4be64d2d9d5a78f845be2f3&

Can you try using the Autofix functionality?
You can find it under Send > Troubleshoot > Fix automatically.

Tried that. No effect. Although I'm not sure what was supposed to happen. Clicking [Fix Automatically] launched cmd.exe, which populated with...
Ok.
Ok.
Press any key to continue . . .

I pressed [Enter], and cmd.exe closed.

All my devices are on the same wi-fi (I even checked that the are all on the 5.0GHz band of the network)

Mac, iPhone, and 2 Android phones can find, send and receive

Win10 & Chromebook can't find any other devices

Chromebook does appear on other devices, but Win10 does not

On Win10 > Windows Defender Firewall, I found 2 inbound rules for LocalSend that were blocked

I allowed them

Didn't help

Figured out how to make a rule in Defender for LocalSend on port 53317
✓ That worked

Any idea why these extra steps were required? I don't have any special security settings on this device.
Also, any suggestions for fixing this problem on the Chromebook?
Thanks!

@Caesarovich re-replying with the last question from my previous comment:
Any idea why these extra steps were required? I don't have any special security settings on this device.
Also, any suggestions for fixing this problem on the Chromebook?

Hi @Robio , I am not too familiar with the way the Windows Defender interacts with the Windows firewall. The Fix automatically button creates "allow inbound" rules in the Windows firewall. It is usually sufficient to fix the firewall issues if such rules were not created at install. So I don't really know what happened in your case for you have to manually edit the firewall rules.
For your Chromebook I have no idea as well as to my understanding Chromebooks cannot install any application. We have a fair number of Chromebooks at my workplace and these only contain the Chrome browser. So I am really curious to know how you managed to install LocalSend on it.
But if LocalSend doesn't work properly on your Chromebook even after successfully installing it; I don't think we can help you as Flutter (and thus LocalSend) doesn't have any support for ChromeOS. Furthermore LocalSend relies on libraries and APIs that are specific to the Windows/Linux/android/apple platforms and which might not exist on Chrome OS.
I'm sorry to not be able to help you further on these issues.

@Caesarovich , Chromebooks can install most Android apps, as Chrome OS is, for the most part, a bastardized Android OS. They come with the Play Store pre-installed and everything. How well they run is a different matter, since most developers are developing with phones and tablets in mind.
Our household Chromebook is running — successfully for the most part — Aqua Mail, UpNote, Enpass, Simplenote, Twitch, GSAM, Comixology, Floating Notes, Spotify, and others.
I'd love to see LocalSend work on Chromebook, but until that glorious day, we'll limp along using Google Drive. :)

This worked for me actually. Both devices same port. But I needed some
tries to find a free port. 65000 seemed to work for me.
Am Di., 18. Juni 2024 um 00:22 Uhr schrieb Caesarovich <
@.***>:

Hi @wadid https://github.com/wadid , thanks for opening this issue. If
you changed the port on one of your devices, you need to change it as well
on all the other devices you want to share files with. We are aware that it
is quite inconvenient and are working on a better streamlined solution.
Could you try this and comeback with a reply if it succeeded/failed ?
—
Reply to this email directly, view it on GitHub
https://github.com/localsend/localsend/issues/1421#issuecomment-2174534999,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4GOBDLPWU5URAST6XDG5DZH5OTZAVCNFSM6AAAAABJN2MV56VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZUGUZTIOJZHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

This worked for me actually. Both devices same port. But I needed some tries to find a free port. 65000 seemed to work for me. Am Di., 18. Juni 2024 um 00:22 Uhr schrieb Caesarovich < @.>:
…
Hi @wadid https://github.com/wadid , thanks for opening this issue. If you changed the port on one of your devices, you need to change it as well on all the other devices you want to share files with. We are aware that it is quite inconvenient and are working on a better streamlined solution. Could you try this and comeback with a reply if it succeeded/failed ? — Reply to this email directly, view it on GitHub <#1421 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA4GOBDLPWU5URAST6XDG5DZH5OTZAVCNFSM6AAAAABJN2MV56VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZUGUZTIOJZHE . You are receiving this because you were mentioned.Message ID: @.>

The issue you were experiencing was most likely due to the fact that the port range above 49152 is for dynamically allocated ports, which can cause issues when it comes to apps like Localsend which expect a hardcoded port on both ends. Its best to use a range between 1024 to 49151 because these are not dynamic ports and don't require privilege escalation


module 'torch.library' has no attribute 'custom_op'

When I'm trying to
from hqq.utils.patching import prepare_for_inference

I got this error
AttributeError                            Traceback (most recent call last)
Cell In[3], line 1
----> 1 from hqq.utils.patching import prepare_for_inference

File ~/hqq/hqq/utils/patching.py:11
      9 from ..backends.torchao import patch_hqq_to_aoint4
     10 from ..backends.marlin import patch_hqq_to_marlin
---> 11 from ..backends.bitblas import patch_hqq_to_bitblas
     14 def patch_linearlayers(model, fct, patch_param=None, verbose=False):
     15     base_class = model.base_class if (hasattr(model, "base_class")) else AutoHQQHFModel

File ~/hqq/hqq/backends/bitblas.py:20
     17 from ..core.peft import HQQLinearLoRA
     18 from ..core.utils import cleanup
---> 20 @torch.library.custom_op("hqq::matmul_bitblas", mutates_args=())
     21 def matmul_bitblas(x: Tensor, W_q: Tensor, scale: Tensor, zero: Tensor, out_features:int, eng_tag:str) -> Tensor:
     22 	origin_x_size = x.size()
     23 	x = x.reshape(-1, origin_x_size[-1])

AttributeError: module 'torch.library' has no attribute 'custom_op'

Yeah this is something that just happened over the weekend, it was working fine on Friday, they changed something in the torch nightly, Will check with the torch team

I didn't get an answer from the torch team. For the moment, I removed the bitblas dependency, so it now the import should simply print a message and will not crash:
https://github.com/mobiusml/hqq/commit/e57104fdcac92bab2a3eed01e75bd30a837c1cfe


set-default-builder should suggest builders

If I run set-default-builder with the help flag, or without providing a builder, I'd like a hint at which builders I could potentially use. Whether we display that in the help text, or from another command specific to list suggested builder, that information would be useful for users who already know they need to set a default builder but don't know what builders are available.

I gave this a fair try but ended up having a slew of issues trying to get it into the --help output so created the PR to only include it when builder name is not provided.

cobra doesn't call PersistentPreRun for --help, probably with good reason making the logger remain uninitialized. I could have created a new one but that defeats the purpose of DI. The other option was to create a "default" logger until the properly set one was initialized but again didn't really think that idea was ideal.
The default help text (aka template) is constructed by private cobra methods. I worked around this by holding a reference to the original HelpFunc and calling it inside the custom HelpFunc but that just felt wrong.

Ultimately, I propose that we don't cram it in --help but instead add another command. For example, list-builders which for now simply suggests builders but in the future could be provided centrally from a server. In the meantime, builders to that list could be added via PR.

@jromero I agree that a separate command to list the suggested builders is a good idea. I would propose suggest-builders instead of list-builders. I worry that if a user runs create-builder followed by list-builders and doesn't see their builder in the list, that could be confusing. suggest-builders implies that the list is not comprehensive


visual studio compilation error

I am compiling libconfuse with visual studio 2015.
I get an error compiling fmemopen
fmemopen.c(161): fatal error C1189: #error:  Sorry, this platform currently has no fmemopen() replacement.
If I modify windows/msvs.net/config.h to add "#define HAVE_WINDOWS_H 1", it compiles.

I have never tested libConfuse with Visual Studio, but I suspect that if Windows has fmemopen(), this could be a simple fix to the configure script. Care to give it a go?

I did a bit of compiling and research and determined that visual studio does not provide an fmemopen. Modifying the config.h for the windows project works. I'll put a brief pull request together.

I created a pull request with the modification
#118

Great work! On most sane platforms we generate config.h based on the probe results from the configure script ...


Unable to use "either::serde_untagged", instructions unclear

I am providing a reprex for the use of either::serde_untagged.
I have not been able to use this feature, because
this line:
#[serde(with = "either::serde_untagged")]
fails to compile, and I cannot figure out why.
//main.rs
use serde::{Serialize, Deserialize};
use either::Either;

fn main() {
    #[derive(Serialize, Deserialize, Debug)]
    #[serde(transparent)]
    struct IntOrString {
        #[serde(with = "either::serde_untagged")]
        inner: Either<Vec<String>, u8>
    };
}

//Cargo Toml deps
[dependencies]
serde = { version = "1.0", features = ["derive"] }
serde_json = "1.0"
either = "1.6.1"

I am grateful for any help

You also need the "serde" feature for either:
[dependencies]
serde = { version = "1.0", features = ["derive"] }
serde_json = "1.0"
either = { version = "1.6.1", features = ["serde"] }

Thank you, that worked!
Maybe it would be helpful to include this information in the docs? (if I haven't missed it)


feat: Adds synthesis rules for AWS entities out of logs

Relevant information
This PR adds entity synthesis rules out of logs for the following entity types: AWSRDSDBINSTANCE, AWSS3BUCKET, AWSALB, AWSELB, AWSNLB, AWSLAMBDAFUNCTION, AWSDCLOUDFRONTDISTRIBUTION, AWSSQSQUEUE, HOST (EC2).
The Logs pipeline will infer the aws.arn attribute and include it together with the entity.name and entity.type attributes to the log prior to the synthesis step. This is why they are used as regular attributes to name the entity and in the synthesis rule conditions. These attributes, together with the resulting entity.guid, will be included in the log sent downstream for the materialization process to happen.
Checklist

[x] I've read the guidelines and understand the acceptance criteria.
[x] The value of the attribute marked as identifier will be unique and valid.
[x] I've confirmed that my entity type wasn't already defined. If it is I'm providing an explanation above.

Blocked by internal concerns please reach out if there's doubt on moving this one forward.

Closing PR for now, since we'll most probably follow a different approach.


Ignore mouse over option

YoutubeTV works fantastic with your app but with it floating anytime your mouse crosses over it the YouTube interface comes up and you need to wait for it to go away.  It would be nice if you could ignore all mouse over activity and require you to actually click or right-click on it to interact with it.  Thanks!

Related to #60


[minor] Update alert text

I found the "Mark All" alert a bit strange.. this pull requests changes that to "OK" instead
From https://developer.apple.com/ios/human-interface-guidelines/ui-views/alerts/

Give alert buttons succinct, logical titles. The best button titles consist of one or two words that describe the result of selecting the button. As with all button titles, use title-style capitalization and no ending punctuation. To the extent possible, use verbs and verb phrases that relate directly to the alert title and message—for example, View All, Reply, or Ignore. Use OK for simple acceptance.

Also this changes "Ok" to "OK" in other places
Thx for the app!

Would you accept "Mark All Read" ?

@dkhamsing sure! I like that.

@rnystrom updated


Build python wheel artifacts for Windows

This patch changes appveyor config.
Added new msvc target for win32 platform.
Wheel artifacts are created after build stage for msvc release targets.
python-package\wheel\xgboost-0.6-py2.py3-none-win32.whl
python-package\wheel\xgboost-0.6-py2.py3-none-win_amd64.whl

Slightly related to #1551

Codecov Report

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

@@            Coverage Diff            @@
##             master    #2941   +/-   ##
=========================================
  Coverage     43.19%   43.19%
  Complexity      207      207
=========================================
  Files           155      155
  Lines         11606    11606
  Branches       1159     1159
=========================================
  Hits           5013     5013
  Misses         6265     6265
  Partials        328      328

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 76f8f51...c71d1c5. Read the comment docs.

Sorry, I didn't notice that build for the Win32 platform is unstable. Fixed.

Pre-build python wheel package available as appveyor artifact for following tagrets:

msvc, ver=2015, generator=Visual Studio 14 2015 Win64, configuration=Release
mingw, generator=Unix Makefiles

I've lately working on binary wheel distribution on PyPI, and this should be of great help. Thanks!


fix(dev): improve local development experience

PR Fixes:

Improves the seed command to not fail when executed once more
Uses process.env.LOCAL_CMS_PROVIDER to provide mock users in middleware and signup

Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I assure there is no similar/duplicate pull request regarding same issue

Also, @hkirat
Can I add a demo video which shows how to setup cms in the readme?

Also, @devsargam @hkirat , can we add more data in the db while seeding? Have a gist for this

Yeah would be really great

/bounty $25


Delete reconciler of Compass Runtime Agent after migration to Application Connector

Description
After the Compass Runtime Agent was migrated to the Application Connctor, the old reconciler for the Compass Runtime Agent is obsolete and we have to disable it.
AC:

[ ] Remove the Compass Runtime Agent from the list of components the old reconciler is maintaining (TBC: is this really needed as it's the last item on the list)
[ ] Configure the Compass Runtime Agent as disable component in the mothership reconciler (https://github.com/kyma-project/control-plane/blob/main/resources/kcp/charts/mothership-reconciler/values.yaml#L158)

Obsolete - handled in #212


fixed issue with running mocha tests that include this library

If I include this library with browserify and try to run my unit test, I get this error:
    Swiper.prototype = {
    ^
ReferenceError: Swiper is not defined

This is because most of the use cases of using the variable Swiper assume a browser.  I have slightly modified the code so that the initial creation uses var Swiper = ... instead of window.Swiper = ....  At the end of the main file, I then do a window.Swiper = Swiper; to make sure it is available globally like it is now.  I also updated the amd script to use window.Swiper instead of just Swiper (which was just inferring the window. anyways).
I tested some of the examples (since there don't seem to be able unit tests) and everything worked fine.  If this could be merged and a new version pushed to npm that would be great (otherwise I will have to push my own version to npm to continue on with my project since I need my tests to pass).
Let me know if there is anything I need to change.

Doesn't it break use cases with Require js or in node.js or in components or browserify?

I don't see how this would break any of them (I can guarantee it does not break browserify since I am currently using this exactly code with browserify).  All I am doing is instead of directly assigning Swiper to the window global variable at the top of the file I am assigning it is a scoped variable at the top and then assigning it to the window global variable at the bottom of the same scoped function.  Assuming the window variable is available (which the current code does) prevents this code from being able to be included outside the context of a browser (like within mocha unit tests).
Bear in mind that I know Swiper itself will not function properly outside the context of the browser however without this change, I can't even load the code for my mocha tests (without removing the inclusion of this npm module within my code which then defeat the purpose of being able to install this package as a npm module).

Any update on this?
Again I know it works with Browserify/NodeJS and don't see any reason it would not work with other loading systems like RequireJS or Components.  I have never used RequireJS or Components and since there are no unit tests I can't confirm that this change does not break them.  I don't have the time to learn those system just to test this change.
If you don't want to merge these changes in because you don't know if they will break other loading systems and you don't have the time to test them, please close this PR and let me know that.  If that is going to be the case then I will just have to remove the library from my build process and manually include it (not ideal but it would be the only other option for me).

Yes, it is because i don't know will it affect other loading systems or not, and also don't have much time now to test it too. I will have a bit more time on next week, so let's keep it open for a while

Just a little update as I was doing more testing, there where some other locations that was assuming window in the global context (like doing new Date(...) or new Image(...)) and added another commit that adds window. to the beginning of those statements.

Seems to be everything fine


Fix/Update zh-Hant localization

Fix those using Simplified Chinese terms in Traditional Chinese characters but not properly translated.

Please mark this as ready for review once the changes are ready to be merged, thanks.

I will merge what's here and further changes can be made in a new PR.


Delete match relations when deleting match

Deleting a match from backend does not cause its relations (rows stored in rikki_heroeslounge_team_match) to be deleted as well. So from a teams point of view, that match looks like it still exists, but it wont be found when accessed.
Probably fixable with a small tweak in the relation definition.

Going to have a shot at this. Do you also want the relations between match and caster removed when this happens? (rows stored in rikki_heroeslounge_match_caster)

There should be no more references to the object anywhere in the database, so yes.


kbpages: add MySQL backed storage for activity based stats

This allows us to deploy kbpages without having to worry about maintaining the storage on the instance. The cert provision process stills relies on the local storage, but it's fine to lose them since once it's on a new instance it'll just reprovision (as long as it's not too frequent to hit Let's Encrypt's rate limit). Although it does still prevent us from running multiple instances. Will look into moving that too.

Just to be sure, the mysql package won't be pulled into the normal keybase or kbfsfuse binaries, right? I don't think libpages gets pulled into those, but just wanted to make sure we agree.

That's correct; libpages is only used by kbpagesd. Even kbpagesconfig doesn't use it -- it just imports libpages/config.


Replace the tasks in nimble with a custom tool.

Create a tool to replace the tasks in the nimble file.

[ ] Fix dependencies check for fusion when running nimble build
[ ] Run watcher remove the watchexec dependency.
[ ] Build host
[ ] Build guest
[ ] Initialize Unreal project

[ ] .uproject, enable plugin
[ ] add a reminder to build to generate Definitions or better yet trigger the build

[ ] Run tests

Nimble will be kept for building the nim files.  It's just easier.  Initializing an Unreal project can be done via a plugin. The watcher can put put in the hostwatcher.nim. fusion can be removed.


Fix NoSuchElementException for klabel

Can we have a test?

I added a commit with the test.. It fails on master, passes with this PR.


Linking error when compiling stm32duino project

When compiling a stm32duino project the linker complains about missing references. I tried all branches. The output is:
/home/user/local/arduino/arduino15/packages/STM32/tools/xpack-arm-none-eabi-gcc/9.2.1-1.1/bin/../lib/gcc/arm-none-eabi/9.2.1/../../../../arm-none-eabi/bin/ld: lib_arduino_lib_core.a(startup_stm32yyxx.S.obj): in function LoopFillZerobss': (.text.Reset_Handler+0x2a): undefined reference to SystemInit'
/home/user/local/arduino/arduino15/packages/STM32/tools/xpack-arm-none-eabi-gcc/9.2.1-1.1/bin/../lib/gcc/arm-none-eabi/9.2.1/../../../../arm-none-eabi/bin/ld: lib_arduino_lib_STM32duino_Low_Power.a(STM32LowPower.cpp.o): in function STM32LowPower::begin()': STM32LowPower.cpp:(.text._ZN13STM32LowPower5beginEv+0x4): undefined reference to LowPower_init'
/home/user/local/arduino/arduino15/packages/STM32/tools/xpack-arm-none-eabi-gcc/9.2.1-1.1/bin/../lib/gcc/arm-none-eabi/9.2.1/../../../../arm-none-eabi/bin/ld: lib_arduino_lib_core.a(HardwareSerial.cpp.o): in function HardwareSerial::write(unsigned char)': HardwareSerial.cpp:(.text._ZN14HardwareSerial5writeEh+0x2a): undefined reference to uart_enable_tx'
/home/user/local/arduino/arduino15/packages/STM32/tools/xpack-arm-none-eabi-gcc/9.2.1-1.1/bin/../lib/gcc/arm-none-eabi/9.2.1/../../../../arm-none-eabi/bin/ld: HardwareSerial.cpp:(.text._ZN14HardwareSerial5writeEh+0x46): undefined reference to serial_tx_active' /home/user/local/arduino/arduino15/packages/STM32/tools/xpack-arm-none-eabi-gcc/9.2.1-1.1/bin/../lib/gcc/arm-none-eabi/9.2.1/../../../../arm-none-eabi/bin/ld: HardwareSerial.cpp:(.text._ZN14HardwareSerial5writeEh+0x52): undefined reference to uart_attach_tx_callback'
/home/user/local/arduino/arduino15/packages/STM32/tools/xpack-arm-none-eabi-gcc/9.2.1-1.1/bin/../lib/gcc/arm-none-eabi/9.2.1/../../../../arm-none-eabi/bin/ld: lib_arduino_lib_core.a(HardwareSerial.cpp.o): in function HardwareSerial::HardwareSerial(void*, HalfDuplexMode_t)': HardwareSerial.cpp:(.text._ZN14HardwareSerialC2EPv16HalfDuplexMode_t+0x52): undefined reference to pinmap_pin'
/home/user/local/arduino/arduino15/packages/STM32/tools/xpack-arm-none-eabi-gcc/9.2.1-1.1/bin/../lib/gcc/arm-none-eabi/9.2.1/../../../../arm-none-eabi/bin/ld: HardwareSerial.cpp:(.text._ZN14HardwareSerialC2EPv16HalfDuplexMode_t+0x5c): undefined reference to pinmap_pin' /home/user/local/arduino/arduino15/packages/STM32/tools/xpack-arm-none-eabi-gcc/9.2.1-1.1/bin/../lib/gcc/arm-none-eabi/9.2.1/../../../../arm-none-eabi/bin/ld: lib_arduino_lib_core.a(HardwareSerial.cpp.o): in function HardwareSerial::enableHalfDuplexRx()':
HardwareSerial.cpp:(.text._ZN14HardwareSerial18enableHalfDuplexRxEv+0x28): undefined reference to uart_enable_rx' /home/user/local/arduino/arduino15/packages/STM32/tools/xpack-arm-none-eabi-gcc/9.2.1-1.1/bin/../lib/gcc/arm-none-eabi/9.2.1/../../../../arm-none-eabi/bin/ld: lib_arduino_lib_core.a(board.c.o): in function init':
board.c:(.text.init+0x2): undefined reference to hw_config_init' /home/user/local/arduino/arduino15/packages/STM32/tools/xpack-arm-none-eabi-gcc/9.2.1-1.1/bin/../lib/gcc/arm-none-eabi/9.2.1/../../../../arm-none-eabi/bin/ld: lib_arduino_lib_core.a(wiring_time.c.o): in function millis':
wiring_time.c:(.text.millis+0x2): undefined reference to `getCurrentMillis'
collect2: error: ld returned 1 exit status
CMake Error at .scripts/LinkScript.cmake:286 (message):
Linking test_deepsleep failed!!!

I tried the example build with stm32duino, and it seems to be working well. So, I think it might be some simple issue that you are facing. In your case, many of the undefined references point to SrcWrapper library that comes with STM32 platform.
Please ensure that you add target_enable_arduino_upload(test_deepsleep) in your code. Although seems unrelated, it is a workaround for linking some additional files in stm32duino. If you still face issue, please share the verbose build log (make VERBOSE=1).

Thank you very much for the quick help.
The solution was the missing line "target_enable_arduino_upload(test_deepsleep)" in my CMakeLists.txt.
The compilation and linking of all libraries works now perfectly.


Fix reponse serialization if response body is empty.

The newline after the headers was missing if the response body was empty.

Thanks, @jbaron-gingco


Add test for single node network

Depends on https://github.com/mafintosh/dht-rpc/pull/57

Will leave here some invalid tests in case it's useful in the future. They're invalid because two nodes are required for pinging each other to get their own stable address back, etc. Although, some changes can be done to make it work but it's probably not intended anyway

Tests
test.skip('first persistent node with no host given', async function (t) {
  const bootstrap1 = new DHT({ bootstrap: [], ephemeral: false, firewalled: false })
  await bootstrap1.ready()
  t.is(bootstrap1.address().host, '::')

  t.ok(await makeServerAndClient([{ host: localIP(), port: bootstrap1.address().port }]))

  t.absent(await makeServerAndClient([{ host: '<IP_ADDRESS>', port: bootstrap1.address().port }]))

  await bootstrap1.destroy()
})

test.skip('first persistent node but binds to IPv6 localhost', async function (t) {
  // Note: anyway will not work due peer.id ipv4 encoding
  const bootstrap1 = new DHT({ bootstrap: [], ephemeral: false, firewalled: false, host: '::1' })
  await bootstrap1.ready()
  t.is(bootstrap1.address().host, '::1')

  t.ok(await makeServerAndClient([{ host: '::1', port: bootstrap1.address().port }]))

  await bootstrap1.destroy()
})

test.skip('first persistent node but binds to IPv4 localhost', async function (t) {
  const bootstrap1 = new DHT({ bootstrap: [], ephemeral: false, firewalled: false, host: '<IP_ADDRESS>' })
  await bootstrap1.ready()
  t.is(bootstrap1.address().host, '<IP_ADDRESS>')

  t.ok(await makeServerAndClient([{ host: '<IP_ADDRESS>', port: bootstrap1.address().port }]))

  await bootstrap1.destroy()
})

test.skip('first persistent node with no host given but binds to local host address', async function (t) {
  const bootstrap1 = new DHT({ bootstrap: [], ephemeral: false, firewalled: false, host: localIP() })
  await bootstrap1.ready()
  t.is(bootstrap1.address().host, localIP())

  t.ok(await makeServerAndClient([{ host: localIP(), port: bootstrap1.address().port }]))

  await bootstrap1.destroy()
})

test.skip('ephemeral node enters persistent mode, and later goes back to ephemeral', async function (t) {
  // This should test when a node is adaptive and becomes persistent naturally
  // but then later goes back into 'ephemeral' mode (i.e. due sleep, etc)
  // Currently, hyperdht is not destroying the persistent class object when it goes back to eph
})

Depends on https://github.com/mafintosh/dht-rpc/pull/58

Just cleaning up, reopen if relevant still


Improvement: 单据处理手动节点表单更换插件

Codecov Report

Merging #902 (d902899) into v2.6.x_develop (4a09933) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@                Coverage Diff                 @@
##           v2.6.x_develop     #902      +/-   ##
==================================================
- Coverage           60.85%   60.85%   -0.01%
==================================================
  Files                 466      466
  Lines               29664    29669       +5
==================================================
+ Hits                18053    18056       +3
- Misses              11611    11613       +2

Impacted Files
Coverage Δ

itsm/ticket/validators/field.py
32.77% <0.00%> (+0.14%)
:arrow_up:

itsm/openapi/ticket/views.py
47.55% <0.00%> (+0.16%)
:arrow_up:

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

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


Joss review

I forgot this wasn't merged.  I should have made that last commit on this branch @sdhutchins


Go components returning error for WAGI executor

The go examples have been updated to use the spin golang sdk. This changes the interface that is being used and triggers WAGI to return an error when hitting the defined golang http routes.
Purposed solution

Remove WAGI executor for golang examples in spin.toml

PR incoming.
There is another issue with the go-static example when pulling in the static assets. I will consider opening a new issue to track progress towards that.


chore: update snapshot

Update snapshot of test

#5626  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @alexghr and the rest of your teammates on  Graphite

Benchmark results
Metrics with a significant change:

note_trial_decrypting_time_in_ms (8): 63.8 (+80%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit ee234153 and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
676
676
676

l1_rollup_calldata_gas
6,412
6,412
6,412

l1_rollup_execution_gas
585,745
585,745
585,745

l2_block_processing_time_in_ms
1,298 (-1%)
4,765
9,351

note_successful_decrypting_time_in_ms
259 (+11%)
634 (-2%)
1,036 (+1%)

note_trial_decrypting_time_in_ms
:warning: 63.8 (+80%)
42.7 (-23%)
33.7 (+2%)

l2_block_building_time_in_ms
11,950 (-1%)
43,388 (-1%)
85,258 (-1%)

l2_block_rollup_simulation_time_in_ms
11,915 (-1%)
43,268 (-1%)
85,027 (-1%)

l2_block_public_tx_process_time_in_ms
6,219 (-1%)
24,392 (-1%)
48,869 (-1%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
13,982 (-3%)
27,061 (-1%)

note_history_successful_decrypting_time_in_ms
1,216 (-6%)
2,344 (-7%)

note_history_trial_decrypting_time_in_ms
44.1 (-36%)
79.8 (-27%)

node_database_size_in_bytes
18,612,304
35,012,688

pxe_database_size_in_bytes
29,859
59,414

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
181 (-1%)
44,377
26,164

private-kernel-ordering
163 (-1%)
50,830
39,026

base-parity
3,913 (-2%)
128
311

root-parity
1,119 (-1%)
1,244
311

base-rollup
14,302 (-1%)
110,464
861

root-rollup
49.7 (-3%)
4,359
725

private-kernel-inner
218 (-1%)
71,744
26,164

public-kernel-app-logic
120 (-2%)
47,695
40,661

public-kernel-tail
164 (-2%)
53,372
7,125

merge-rollup
9.75 (-12%)
2,568
861

public-kernel-teardown
117
47,695
40,661

public-kernel-setup
116
47,695
40,661

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
512 leaves
1024 leaves
2048 leaves
4096 leaves
32 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.1 (-1%)
16.2 (-1%)
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.8
31.6
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.590 (-1%)
0.499 (-1%)
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
46.1
72.8
232
448
871 (-1%)
1,739
N/A

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
96.0
159
543
1,055
2,079
4,127
N/A

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.472
0.448
0.421
0.418
0.413 (-1%)
0.415
N/A

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
54.6
107 (-1%)
340
662
1,313 (-1%)
2,632
N/A

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
105
207
691
1,363
2,707
5,395
N/A

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.481
0.481 (-1%)
0.461
0.455
0.453 (-1%)
0.456
N/A

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
61.4 (-1%)

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
109

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
0.535

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
40,372
496,534

Transaction size based on fee payment method

Metric
native fee payment method
fpc_public fee payment method
fpc_private fee payment method

tx_with_fee_size_in_bytes
905
1,161
1,377

Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes
2 new note hashes

tx_pxe_processing_time_ms
1,696
1,090
5,329 (-1%)

Metric
1 public data writes
2 public data writes
3 public data writes
4 public data writes
5 public data writes
8 public data writes

tx_sequencer_processing_time_ms
579 (-2%)
400
1,065
522
1,743 (-1%)
584

Merge activity

Apr 9, 5:57 AM EDT: @alexghr merged this pull request with Graphite.


failed to build x86 windows

HI!
I compiled the library for x86 Windows according to BUILD-WINDOWS.md, but it failed.
It indicated  that some Linux interfaces cannot be compiled.

This is a known issue: Windows support has been broken since version 2 of the library (with server support) was released.  We are hoping some kind soul will port the library to Windows and submit a PR.

yeah,I would try to do it

That would be very nice of you! I've seen @amoldeshpande start doing this already, perhaps you can reuse some of his work.

My need for lsquic did not pan out, so I have not spent much time on it.
my fork may be a bit of a mess right now since I was also trying to get rid of openssl dependencies by using the Win32 crypto library.  hopefully, that's controlled by a #define, but I don't remember.
Theoretically, it would be better to just use LLVM clang instead of Visual C++ so you don't end up with the code diverging this much again.  It works quite well these days now that they can generate proper PDB files. Hopefully the ABI compatibility will work with the 3rd party libs that lsquic needs.
So modifying the CMake setup to use LLVM (not clang-cl, the full clang)  should take you 90% of the way there.
Either way, I wouldn't undertake fixing lsquic without also making a CI pipeline for Windows.  That would make future divergence easier to catch.

Thank you for the pointers! Making the library compile with Visual C++ was a good learning experience last time I did it. I haven't had the time (and the heart) to do this again.
Version 1 of the library, when it was called lsquic-client, was using AppVeyor; it should be pretty easy to revive it. The breaking of the Windows build with version 2 was conscious: we wanted to release the server support but did not have the resources to port the server to Windows.

I should caveat that using LLVM with cmake on Windows is a royal pain in the backside and I was never able to figure it out in a way that felt clean (without mucking up  Visual Studio install). Which is why I basically gave up on it.
I don't know why CMake still doesn't support it right out of the box, but that's the state of C++ cross platform toolsets these days, half-assed, over-complicated and impossible to actually understand.
A simple gnumake makefile would be so much easier.

Windows is now supported on master (2.16.0).  Closing.


Adding IsDisposed checks to Span getter and GetArray/Pointer Internal

cc @KrzysztofCwalina, @pakrym, @davidfowl
Addressing comments:
https://github.com/dotnet/corefxlab/pull/1465#discussion_r112050277
https://github.com/dotnet/corefxlab/pull/1465#discussion_r112050352

in general looks good.

@KrzysztofCwalina  We need to figure our OwnedBuffer recycling and fix dispose implementation therer.


Make boolean values accessible in react, e.g. "isWriteable" and "canMakeReadonly"

All other types than boolean is being returned to react, so I just added a type check for primitive booleans.

It's awesome, thanks for contribution!

closes #284

It's awesome, thanks for contribution!

Any time, thanks for a great nfc lib!
Would it be possible to publish a patch with the merged changes to NPM?

Hi @clrsustainas sure, v2.1.7 just published

Hi @clrsustainas sure, v2.1.7 just published

Much appreciated.
I used the wrong name in one of the functions though, so I've made a pull request here, that will fix it https://github.com/whitedogg13/react-native-nfc-manager/pull/288 .


Disabling Certain Properties on analytics.page

The call to page includes url, referrer and path by default. Sometimes the URL has sensitive information that we do not want to surface in integrations like Intercom. Is the proper way to exclude this information is to set these values in the properties hash to null? If so, I would move to get that added to the documentation.

Unfortunately, this isn't currently possible.
I think we could support this with middlewares. Tracking internally in JIRA https://segment.atlassian.net/browse/LIB-145.

You can override the default properties(url, referrer, search, ...) when calling page with passing an object as the second parameter of the page function.
https://segment.com/docs/sources/website/analytics.js/#default-properties


ESP32 BLE driver is crashing

After the PR https://github.com/apache/nuttx/pull/7160 ESP32 BLE driver is crashing.
@gustavonihei figure-out that the issue is caused by the change with the source pointer _data_start_btdm_rom
And he come up with this small fix:
--- a/arch/xtensa/src/esp32/esp32_ble_adapter.c
+++ b/arch/xtensa/src/esp32/esp32_ble_adapter.c
@@ -451,8 +451,8 @@ extern uint8_t _bss_start_btdm[];
 extern uint8_t _bss_end_btdm[];
 extern uint8_t _data_start_btdm[];
 extern uint8_t _data_end_btdm[];
-extern const uint8_t _data_start_btdm_rom[];
-extern const uint8_t _data_end_btdm_rom[];
+extern uint32_t _data_start_btdm_rom;
+extern uint32_t _data_end_btdm_rom;

 extern uint8_t _bt_bss_start[];
 extern uint8_t _bt_bss_end[];
@@ -2027,11 +2027,11 @@ static void btdm_controller_mem_init(void)

   /* initialise .data section */

-  memcpy(_data_start_btdm, _data_start_btdm_rom,
+  memcpy(_data_start_btdm, (void *)_data_start_btdm_rom,
          _data_end_btdm - _data_start_btdm);

   wlinfo(".data initialise [0x%08x] <== [0x%08x]\n",
-         (uint32_t)_data_start_btdm, (uint32_t)_data_start_btdm_rom);
+         (uint32_t)_data_start_btdm, _data_start_btdm_rom);

   /* initial em, .bss section */

@xiaoxiang781216 could you please helps us to understand what is going on after that PR?

You can test it on NuttX this way:
nsh> bt bnep0 scan start
nsh> bt bnep0 scan stop
nsh> bt bnep0 scan get

could you print out all memcpy argument before and after revert?


Support dynamic properties in entities

One of the selling points of NoSQL databases are their flexible entities. As far as I can see, this is not supported by Catacumbo -- all entities will have all of its properties set upon insert, including possibly NullValues if the field has not been set. (One could specify different entity classes with different properties, having the same kind, but this doesn't feel right.)
Here are two suggestions for supporting different set of properties on the same kind. (Let me know if you want to split them up.)
1. Optional properties
Add an option to not write the property at all if it is null. For example using an annotation specifying that the property should not be created if the field is not set. (My suggestion is @Optional.)
Example class:
@Entity
public class Foo {
  @Identifier
  private Long fooId;

  @Embeddable
  public static class Embed {
    @Optional
    private String embeddedProperty;

    [... getters and setters]
  }

  @Embedded
  @Optional
  private Embed exploded;

  @Embedded
  @Imploded
  @Optional
  private Embed imploded;

  @Optional
  private String optionalProperty;

  [... getters and setters]
}

entityManager.insert(new Foo()) will currently result in an entity with three properties: embeddedProperty, imploded and optionalProperty (all set to NullValue). I would like to have this code not insert any properties at all. (It seems like the unmarshaller handles missing properties well.)
Still with this change, all possible property names must be known at compile time, so my second suggestion is:
2. Dynamic properties
Add an option to create a dynamic set of properties. This can be implemented in several ways, here are two suggestions:
a. Using Maps
Allow properties of type Map<String, Object> to have the @Exploded annotation. When this field is inserted, each entry maps to a separate property, where the key specifies the name, and the value (after applying mappers) specifies the value. When the entity is loaded, properties that are not accepted by any other field of the entity should be inserted into this map. (A problem here might be to decide which mapper to use, but see below.) And care must be taken if keys collide with other fields during an insert (ignore the value or throw run-time exception?).
It might be reasonable to accept a more specific value type in the map, and for instance put all String values in a Map<String, String> field, and all Long values in a Map<String, Long> field (if present). This will solve the problem of selecting mappers.
b. Generalize Mappers
Currently, a custom Mapper can only map fields to properties 1-to-1. It might make sense to expand this functionality to support mapping one field to many properties, just like the @Exploded fields do now. For example like:
public interface ExplodingMapper {
  public void toDatastore(Object input, BaseEntity.Builder<?, ?> entityBuilder);

  // Perhaps wrap the input in a state object that remembers all properties that have been read
  // so far by other mappers.
  public Object toModel(BaseEntity<?> input);
}

Then 2a (and the @Exploded handling) can be implemented in terms of this general mapper, and clients are free to define classes that explode in any way they like. Again, behavior must be defined if property names collide.
This last suggestion might be too complex, but I would really like to see some way to have dynamic properties in Catatumbo, and this is the most general solution.

What do you think? Are any of these suggestions reasonable and possible to implement?

@bnordli
Optional Properties
This makes sense, but one needs to understand that there is no way to find out the entities that are missing a specific property. What would be the pros/cons of storing null vs. not storing the property altogether? A couple of benefits I could see are -

Viewing the Entities in GCD console keeps it clean
Less storage/cost

Are there any other benefits to it that you would like to point out?
Dynamic Properties
I always wanted to have this from the very beginning and reserved an Enum with the name Unmodeled (not in the domain model). It is currently unused. However as I was adding features and started supporting Maps, I did not see a need for this any more. One could simply define the below:
@Entity
public class Item {
  @Identifier
  private long id;
  private String description;
  @Decimal(10, 2)
  private BigDecimal price;
  //Dynamic or user defined properties (not developer defined)
  private Map<String, Object> customProperties
  }
}

Now you can add all your custom properties to the Map (e.g. color, size for a Shirt item, capacity, memory, processor, color for a Phone item).
The one difference between your request and this is - the Map is stored in a single field, not in exploded form. Do you see any difference in the way the Map is stored (Imploded/Exploded)? How does it affect your application?
Also, since the the type of Value in the Map is Object, the values are mapped using CatchAllMapper, which supports the following types:

Long
Double
Boolean
String
DatastoreKey
GeoLocation

Of course, you can have more than one map to handle specific value types, but I think the basic types listed above should suffice in most cases.
Let me know your thoughts and we will go from there.

Optional properties: Yes, you are right about the search restrictions, but I believe most searches for optional properties go towards entities that have a specific value, not all entities that are missing this value (and otherwise you need to actually store null). It is mostly a matter of keeping the data -- and perhaps especially indices -- clean.
I am not hugely worried about storage costs, but if you have a property that has (a small amount of) data in only 1% of entities (and null on the rest), and you decide to index it, the index will be 100x larger than actually needed. (And each entity (with a long id) uses at least 60 bytes of data in the index, in addition to the property and kind name.)
Summing up, I think this is a nice feature to have, but not a very important one.
Dynamic properties: Nice that you already have an annotation for this! (I see you mention in the code that it only makes sense to have one Map per entity with this annotation.)
Actually, we are already using Map<String, String> in one of our entities, and yes, we are getting problems because of the imploded form: When any of the key values contain ., we get the following error: com.google.cloud.datastore.DatastoreException: property.name contains a path delimiter, and the entity contains one or more indexed entity value. We have identified four solutions to this problem:

Turn off indexing for the property: This works, but makes the property very hard to work with.
Escape all dots (for instance a.b -> a_b): This kind of works, but needs all sorts of special casing both during inserts, loads and queries.
Manually explode into subentities: We started implementing this, but stopped when we figured out there was no way to handle the two keys a and a.b in the same entity. (Also, catatumbo could not handle maps as values inside maps, but that is a smaller problem.)
Explode the map into separate properties: Experimenting in the Cloud Console, indexed properties can  have dots in their names, and there can be properties with names a and a.b in the same entity. And the entity looks better in the Console.

We are currently doing 1, but would very much like to go with 4, otherwise we have to fall back to 2. So this was a little more background for our request.
I think the CatchAllMapper is a good solution for retrieving dynamic entities -- more mapping possibilities could be added later if needed. (As discussed above, maybe supporting Maps could be an acceptable solution for some clients.)

Not knowing what exactly you are trying to accomplish, I would like to ask - why is it so important for you to use a dot (.) in the property name of an EmbeddedEntity? How is it related to/how does it help if Catatumbo supports Map of Maps? Perhaps a more realistic example would help understand your requirements so we do this right.
On the optional properties, we can avoid persisting properties that have a null value. What about if the fields of the entity are primitives, example int. Do we need to handle this as well and not store the property if the value is zero or have the developer specify a default value (e.g. don't store the property of my int field is 100)? Perhaps it is best to support @Optional on non-primitives? Any thoughts here?

We are receiving an external data stream with some associated metadata. Different streams have different metadata keys, but some of them are common across streams. As we don't have control over the keys, we don't want to mess too much with their names (some have .s and some have _s), but we do want to group streams by metadata, or look up what kind of metadata we are receiving.
If Catatumbo supported map of maps, we could explode the map into a entityValue like this:
{ "properties": {
    "FOO": { "entityValue": { "properties": { "BAR": { "stringValue": "value for foo.bar" } } } },
    "BAZ": { "stringValue": "value for baz" }
} }

and still be able to select WHERE customProperties.FOO.BAR = "value for foo.bar". Yes, this works, and is the reason dots are not allowed as property names in indexed properties in the first place.
We currently have no metadata where one key is a prefix of the other (FOO and FOO.BAR), but as we don't have control over them, we want to make sure to handle this case, which looks impossible to represent correctly.
Yes, @Optional only makes sense on reference types, IMO.

I think we should be able to support map of maps (nested to any level) when using a raw map. I will give that a try and let you know. I will create another ticket for the Optional fields.

Here is something that seems to do the trick. Would you please try this and let me know.
catatumbo-156.txt

Yes, this works nicely, thanks!
With a small hack, representing {"FOO" -> "x", "FOO.BAR" -> "y"} as {"FOO" -> { "_" -> "x", "BAR" -> "y"}} in the common prefix case, this is an acceptable work around for us at the moment. (Inserting and loading works correctly, while querying has to be aware of the possibility of a renamed key.)
I still believe an exploding map has merit, but I also realize that requires much more work than these other small changes, and will let you prioritize accordingly.
(Just nitpicking, in CatchAllMapper.java, the new code returns early from the method, while all the other cases assigns to a return variable which is returned at the end.)

That sounds fine with me! Solves the problem, but is also more flexible!

Optional properties are implements as part of #157


Lines hit just by test

Hi, guys!
Maybe it is not an issue, maybe it is me but I have a PORO class instantiated in my controller. This instance variable is used to call a method. This method is not covered by my Minitest::Spec and, even though, simplecov report consider this 1 line method covered with 1 hit. If I remove this method call from the controller, I got the red line on the HTML report.
Is there a way to consider the hit just by the tests ? Calling all methods in my controller and not creating the class spec I guess is not the correct way to consider my file 100% covered.
Thanks,
Paulo

Hi there,
well the line must be covered somewhere in your tests because otherwise it wouldn't show up green. With covered I mean, some time during your test execution that controller must be hit calling that method and therefore it is covered. SinpleCov and the underlying library really just track which lines of code have been executed at all - especially if you do End2End testing that will be a lot :)
If you just want to see what's covered from your unit tests - only run your unit tests and then look at the report :)
Hope that helps + cheers,
Tobi


TypeError: null is not an object (evaluating 'cards[index]')

TypeError: null is not an object (evaluating 'cards[index]')
Warning: Failed %s type: %s%s, prop, The prop cards is marked as required in Swiper, but its value is null.,

I keep getting TypeError: undefined is not an object (evaluating 'value.getValue')
Could these be related?

Could you please provide a more detailed description of the issue? Especially when does it occur? I might know how to fix a similar bug but I don't know if it is the same one

shoot, how did not see this reply til now 🙈 my b @nofacez
I must've fixed that particular issue back then somewhere

Just solved the above error with getValue issue..
made a patch-package react-native-deck-swiper+2.0.17.patch
`diff --git a/node_modules/react-native-deck-swiper/Swiper.js b/node_modules/react-native-deck-swiper/Swiper.js
index ba9fafe..15dff26 100644
--- a/node_modules/react-native-deck-swiper/Swiper.js
+++ b/node_modules/react-native-deck-swiper/Swiper.js
@@ -57,8 +57,16 @@ class Swiper extends Component {
this._animatedValueX = 0
this._animatedValueY = 0

this.state.pan.x.addListener(value => (this._animatedValueX = value.value))
this.state.pan.y.addListener(value => (this._animatedValueY = value.value))

this.state.pan.x.addListener(value => {

 if(value && value?.value !== undefined){

   this._animatedValueX = value.value

 }

})

this.state.pan.y.addListener(value => {

 if(value && value?.value !== undefined){

   this._animatedValueY = value.value

 }

})
this.initializeCardStyle()
this.initializePanResponder()
@@ -646,7 +654,7 @@ class Swiper extends Component {
this.getCardStyle(),
{
zIndex: position * -1,

 transform: [{ scale: this.state[`stackScale${position}`] }, { translateY: this.state[`stackPosition${position}`] }]

 transform: [{ scale: this.state[`stackScale${position}`] !== undefined ?this.state[`stackScale${position}`] : 1 }, { translateY: this.state[`stackPosition${position}`] !== undefined ? this.state[`stackPosition${position}`]:0 }]

},
this.props.cardStyle
]
@@ -761,22 +769,27 @@ class Swiper extends Component {
return cardIndex
}

pushCardToStack = (renderedCards, index, position, key, firstCard) => {

pushCardToStack = (index, position, key, firstCard) => {
const { cards } = this.props
const stackCardZoomStyle = this.calculateStackCardZoomStyle(position)
const stackCard = this.props.renderCard(cards[index], index)
const swipableCardStyle = this.calculateSwipableCardStyle()
const renderOverlayLabel = this.renderOverlayLabel()

renderedCards.push(

return (
<Animated.View
key={key}
style={firstCard ? swipableCardStyle : stackCardZoomStyle}
{...this._panResponder.panHandlers}

 >

   {firstCard ? renderOverlayLabel : null}

   {stackCard}

  >

    {firstCard ? renderOverlayLabel : null}

    {stackCard}
 </Animated.View>

)

}
renderStack = () => {
@@ -790,9 +803,10 @@ class Swiper extends Component {
while (stackSize-- > 0 && (firstCard || showSecondCard) && !swipedAllCards) {
const key = this.getCardKey(cards[index], index)

 this.pushCardToStack(renderedCards, index, cardPosition, key, firstCard)

 firstCard = false

  renderedCards.push(this.pushCardToStack(index, cardPosition, key, firstCard))

  firstCard = false

 if (index === cards.length - 1) {
   if (!infinite) break

@@ -802,6 +816,7 @@ class Swiper extends Component {
}
cardPosition++
}
+
return renderedCards
}
`


Merge pull request #1572 from lukas-bednar/test-lifecycle

What this PR does / why we need it:
Some test requires to work with compute node - these are counting with
fact that all nodes are compute, which is wrong assumption.
Signed-off-by: Lukas Bednar<EMAIL_ADDRESS>Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Cherry-pick of #1572
Release note:

NONE

ci test please


Add free disk space

This PR is tackling #32. The code should work fine on POSIX platforms.

Bars are supported too.

Looks good to me!


[FEATURE] Api implentation for SME signup and signin

Api implentation for SME signup and prevent duplicte registration through authentication

done


Codespace verbose dollop v4rggqwjrq73x79q

What type of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Refactor
[ ] Documentation
[ ] Not Sure?

Does this PR introduce breaking changes?

[ ] Yes
[ ] No

List any relevant issue numbers:
Description:

Must be a mistake. Closing.


add metrics to Estimator ( not canned estimators)

This template is for miscellaneous issues not covered by the other issue categories.
For questions on how to work with TensorFlow, or support for problems that are not verified bugs in TensorFlow, please go to StackOverflow.
If you are reporting a vulnerability, please use the dedicated reporting process.
For high-level discussions about TensorFlow, please post to<EMAIL_ADDRESS>for questions about the development or internal workings of TensorFlow, or if you would like to know how to contribute to TensorFlow, please post to<EMAIL_ADDRESS>

I just changed my code from tf.contrib.learn.Estimator to tf.estimator.Estimator in TensorFlow version 1.11.0. In previous version there was an option to use other metrics like accuracy while evaluating the estimator by passing metrics to evaluate function. In current version if there is a canned estimator, we can define the metrics with tf.contrib.estimator.add_metrics, but when I define my estimator and want to use add_metrics it says there is no such wrapper. The error is as follow:
raise TypeError('{!r} is not a Python function'.format(func)) TypeError: <method-wrapper '__call__' of type object at 0x7fff9cb37530> is not a Python function
I don't know whether it's because I'm not using canned estimator or something else is wrong with my code. I want to know how I can add metrics to estimator which is not a canned estimator.


Reformatted all the code with ruff

As per our discussions, I ran ruff . --fix on the entire code base.
I had to change a few things to get the tests to still run:
[tool.ruff.lint.per-file-ignores]
"**/{tests,docs}/*" = ["ALL"]
"**__init__.py" = ["F401"]

The first line just disables linting/error fixing on the tests.
This was necessary, because some tests do stuff like importing List rather than using list, because it wants to be backwards compatible.
The line "**__init__.py" = ["F401"] prevents ruff from removing unused imports in __init__.py files, which are used to make those modules publicly available.

Happy to merge this. Just wanna do:

Push current to pypi first so there's a safe version up there.
Merge this. Then push again to pypi

Ok, I've checked all the notebooks as discussed in the Discord.
Merge at your leisure.


Google Sign In connection bug sometime

Steps to reproduce

create a Google Sign In connexion
on some android device the sign in failed

Expected results
the google connection give me a token and finish the connection
Actual results
In some tests on Smatphone (Android or Iphone ). We receive sometime the following error:
PlatformException(sign_in_failed, x0.b: 10: , null, null)
I didn't found this error code x0.b: 10: and I don't think that is a problem of misconfiguration because it's work on the majority on our test devices the connection succeded
Code sample
  final GoogleSignIn _googleSignIn = kIsWeb
      ? GoogleSignIn(clientId: Constants.GOOGLE_WEB_CLIENT_ID, scopes: Constants.GOOGLE_SCOPE)
      : io.Platform.isAndroid
          ? GoogleSignIn(
              scopes: Constants.GOOGLE_SCOPE,
            )
          : GoogleSignIn(clientId: Constants.IOS_GOOGLE_CLIENT_ID, scopes: Constants.GOOGLE_SCOPE);

GoogleSignInAccount? googleValue = await _googleSignIn.signInSilently();
googleValue ??= await googleSignIn.signIn();

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
PlatformException(sign_in_failed, x0.b: 10: , null, null)

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.24.1, on macOS 14.6.1 23G93 darwin-x64, locale
    fr-IL)
    • Flutter version 3.24.1 on channel stable at /Applications/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 5874a72aa4 (4 weeks ago), 2024-08-20 16:46:00 -0500
    • Engine revision c9b9d5780d
    • Dart version 3.5.1
    • DevTools version 2.37.2

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/acherklein/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /Users/acherklein/Library/Application
      Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/222.4459.24.2221.1012163      9/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 16.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 16A242d
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Users/acherklein/Library/Application
      Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/222.4459.24.2221.1012163
      9/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.93.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.96.0

[✓] Connected device (3 available)
    • iPhone de Acher (mobile) • 00008110-00160CDE1E01401E • ios            •
      iOS 17.6.1 21G93
    • macOS (desktop)          • macos                     • darwin-x64     •
      macOS 14.6.1 23G93 darwin-x64
    • Chrome (web)             • chrome                    • web-javascript •
      Google Chrome 128.0.6613.138

[✓] Network resources
    • All expected network resources are available.

• No issues found!

I think this is related to:
https://github.com/flutter/flutter/issues/155429#issue-2537349829

@Acher1234 Are you only seeing this issue in release build? If so, this could be similar to https://github.com/flutter/flutter/issues/75224.
Otherwise, if you see this on debug mode as well, can you provide the entire crash stack trace with a minimal & complete sample code? (You can try on plugin example to see if crash occurs there or not)

hey,
I'm not using firebase and It's bug on realease but not every time. so it's not the same error than #75224 .
the error it's not the same too .
I tried what is write in #155429 + I add stack trace in the log :)

I tried what is write in https://github.com/flutter/flutter/issues/155429 + I add stack trace in the log :)

I don't see the updated logs on your original comment. Please update it.
If it's not similar to https://github.com/flutter/flutter/issues/75224 and you can see it locally, you should try to reproduce with a minimal sample code and find a pattern then share it here so that will be helpful to investigate further.

No sorry I add the trace back in our log, because that the error is random I didn't see her again (but the next time the log will be save on our server and I will update the tracelog).
I can't reproduce it locally it's error that our client had sometime .
We send each error not catched in our server so next time I will have the trace back.
thanks

so next time I will have the trace back.

Please provide us this trace once you have it, that will probably help to analyze this further, because we currently can't debug further.


chore(zetaclient): improve logging by adding and moving data to attributes

We see a lot of "RPC Status Check error: RPC down?" with error "not found".

make the log messages more specific about what exactly is failing
add consistent block_number log attribute whenever possible

Summary by CodeRabbit

New Features

Enhanced logging functionality for RPC status checks, providing more detailed error messages and context.
Improved clarity in logs for both successful and failed RPC calls, including specific method names and additional structured information.

Bug Fixes

Updated logging for block age checks to include relevant details like block number and age in seconds, aiding diagnostics.

superseded by https://github.com/zeta-chain/node/pull/2751 which does at least add more descriptive error messages.


libssw.so: undefined symbol: cigar_int_to_len

I was have an issue your answer before about ssw_wrap.py. To solve the problem, I copied the ssw_wrap.py from this repo: https://github.com/mengyao/Complete-Striped-Smith-Waterman-Library
Now I am getting: libssw.so: undefined symbol: cigar_int_to_len.
When I get back to the code, I found this function is in  ssw_cpp.cpp. Why this function is not included in the lib file?

Hi,
You should not have to directly copy ssw_wrap.py from other repos! It is already included in the Cogent installation.
Please make sure you have installed Cogent per installation wiki here and that you have the appropriate variables configured:
export PATH=$HOME/anaconda2/bin:$PATH
source activate anaCogent
export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:<your_dir>/Cogent/Complete-Striped-Smith-Waterman-Library/src
export PYTHONPATH=$PYTHONPATH:<your_dir>/Cogent/Complete-Striped-Smith-Waterman-Library/src

I forgot to use python install.
Thanks!

Hi, I am also facing the similar issue. I installed Complete-Striped-Smith-Waterman-Library from the link in the tutorial (https://github.com/mengyao/Complete-Striped-Smith-Waterman-Library).
But after installion when i executed run_mash.py it gave following error:
Traceback (most recent call last):
File "/home/user/anaconda2/bin/run_mash.py", line 4, in
import('pkg_resources').run_script('Cogent==3.6', 'run_mash.py')
File "/home/user/anaconda2/lib/python2.7/site-packages/pkg_resources/init.py", line 658, in run_script
self.require(requires)[0].run_script(script_name, ns)
File "/home/user/anaconda2/lib/python2.7/site-packages/pkg_resources/init.py", line 1438, in run_script
exec(code, namespace, namespace)
File "/nfsroot/home/user/anaconda2/lib/python2.7/site-packages/Cogent-3.6-py2.7.egg/EGG-INFO/scripts/run_mash.py", line 6, in
from Cogent.sanity_checks import sanity_check_mash_exists
File "/nfsroot/home/user/anaconda2/lib/python2.7/site-packages/Cogent-3.6-py2.7.egg/Cogent/sanity_checks.py", line 3, in
from Cogent import splice_align
File "/nfsroot/home/user/anaconda2/lib/python2.7/site-packages/Cogent-3.6-py2.7.egg/Cogent/splice_align.py", line 4, in
from ssw_wrap import Aligner
ImportError: No module named ssw_wrap
So i copied the ssw_wrap.py and copied it and did python ssw_wrap.py install. But it gave following error.
Traceback (most recent call last):
File "/home/user/anaconda2/bin/run_mash.py", line 4, in
import('pkg_resources').run_script('Cogent==3.6', 'run_mash.py')
File "/home/user/anaconda2/lib/python2.7/site-packages/pkg_resources/init.py", line 658, in run_script
self.require(requires)[0].run_script(script_name, ns)
File "/home/user/anaconda2/lib/python2.7/site-packages/pkg_resources/init.py", line 1438, in run_script
exec(code, namespace, namespace)
File "/nfsroot/home/user/anaconda2/lib/python2.7/site-packages/Cogent-3.6-py2.7.egg/EGG-INFO/scripts/run_mash.py", line 6, in
from Cogent.sanity_checks import sanity_check_mash_exists
File "/nfsroot/home/user/anaconda2/lib/python2.7/site-packages/Cogent-3.6-py2.7.egg/Cogent/sanity_checks.py", line 3, in
from Cogent import splice_align
File "/nfsroot/home/user/anaconda2/lib/python2.7/site-packages/Cogent-3.6-py2.7.egg/Cogent/splice_align.py", line 4, in
from ssw_wrap import Aligner
File "/home/user/2018/tools/Complete-Striped-Smith-Waterman-Library-master/src/ssw_wrap.py", line 267, in
class PyAlignRes(object):
File "/home/user/2018/tools/Complete-Striped-Smith-Waterman-Library-master/src/ssw_wrap.py", line 282, in PyAlignRes
cigar_int_to_len = libssw.cigar_int_to_len
File "/home/user/anaconda2/lib/python2.7/ctypes/init.py", line 379, in getattr
func = self.getitem(name)
File "/home/user/anaconda2/lib/python2.7/ctypes/init.py", line 384, in getitem
func = self._FuncPtr((name_or_ordinal, self))
Complete-Striped-Smith-Waterman-Library-master/src/libssw.so: undefined symbol: cigar_int_to_len
Please help me proceeding.
Thank you

Did you use pip to install the cogent?
That helped me to solve the problem. But I did not use anaconda, you may want to try conda install?

Hi, thanks for response. Setting path for the script installed solved my problem.


Jobergum/add ranking models

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

👍 Thanks @freva


Update audit-log-retention-policies.md

Added guest as well int he note as the audit log is retained only for 90 days for guest users.

@JohanFreelancer9 Please copy review

@JohanFreelancer9 Please copy review

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/3885

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/3885

@msbemba This change implies that we retain audit records for auditable activities performed by unlicensed guest users. Did you confirm that with engineering or PM?

@msbemba This change implies that we retain audit records for auditable activities performed by unlicensed guest users. Did you confirm that with engineering or PM?

I have not confirmed. But tested it on my test tenant.
On Wed, 13/01/2021 at 23:32 Mark Johnson - MSFT<EMAIL_ADDRESS>wrote:

@msbemba https://github.com/msbemba This change implies that we retain
audit records for auditable activities performed by unlicensed guest users.
Did you confirm that with engineering or PM?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/microsoft-365-docs/pull/3902#issuecomment-759620303,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK63OZG2EN7NQ2AOKPTK7RLSZXN2LANCNFSM4WBCSXQQ
.

--

Sriraman Srinivasan
If Plan "A" Doesnt work there are 25 other letters:) Stay Cool !

I have not confirmed. But tested it on my test tenant.
On Wed, 13/01/2021 at 23:32 Mark Johnson - MSFT<EMAIL_ADDRESS>wrote:

@msbemba https://github.com/msbemba This change implies that we retain
audit records for auditable activities performed by unlicensed guest users.
Did you confirm that with engineering or PM?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/microsoft-365-docs/pull/3902#issuecomment-759620303,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK63OZG2EN7NQ2AOKPTK7RLSZXN2LANCNFSM4WBCSXQQ
.

--

Sriraman Srinivasan
If Plan "A" Doesnt work there are 25 other letters:) Stay Cool !

@msbemba Curious, what activities performed by guest users returned audit records? Please send to my internal acct. Also, please send me your email address so that we can take this (and other issues) offline. Thanks!

@msbemba Curious, what activities performed by guest users returned audit records? Please send to my internal acct. Also, please send me your email address so that we can take this (and other issues) offline. Thanks!