refactor(tools): move most tools to the engine

Description
So this PR looks big and scary but it's nothing ground breaking - I didn't implement any features here.
I just moved most of the tools in Tesseratos to the engine, under the directory tools.
Previously, Tesseratos was a library, which meant we could just use its utilities in games (such as Scraps vs Zombies) by linking Tesseratos as a dependency.
With 0.3, Tesseratos became an actual executable, which meant its tools became inaccessible.
The goal here is to simply move the tools which can be useful even in a deployed game to the engine library itself, while keeping actual editor stuff, such as the asset explorer and scene editor, on the editor itself.
I had to change a lot of boring stuff, such as the namespaces of everything from tesseratos:: to cubos::engine::, and also the respective documentation. Also used the opportunity to reword some stuff better, and simplify the naming of some plugins.
Also, to make things easier for users, I added a toolsPlugin which simply adds all of the plugins in the new directory.
Checklist

[x] Self-review changes.
[x] Evaluate impact on the documentation. (I had to change a lot of stuff!)
[x] Ensure test coverage. No new features.
[x] Write new samples. No new features, although I added the tools to the CuboSurfers plugin.
[x] Add entry to the changelog's unreleased section.

@Scarface1809 don't forget to approve with your main account


Error: Failed to load colors from Figma

Tool version: 0.38.0
This week, I encountered a very strange bug. Everything was exporting fine except for the colors, and the terminal simply output the following error:
Error: Failed to load colors from Figma

The design team thoroughly checked everything, but there were no results.
I decided to experiment with figma-export.yaml and added nameValidateRegexp specifically for colors. Before, it was commented. Here's what I got in the terminal:
Error: ❌ Bad asset name «some_color_1»
❌ Bad asset name «some_color_2»
❌ Bad asset name «some_color_3»

I just fixed the regex to accept names that we use in Figma, and the problem was resolved.
I don't understand how it could be that with zero restrictions, it fails to export, but when I add restrictions, it works. Perhaps the library applies some default regex validation under the hood when we don't specify anything, but I couldn't find any information about this.
Another question is why it doesn't display any error message, when regex validation is commented.
I want to note that export works perfectly fine on other projects with the same config, and it also worked okay before on this project too. I'm not sure if this is a bug or not, and looking for a piece of advice.
UPD: There is no issue on 0.37.6, looks like the error is related to 0.38.0

The error «Failed to load colors from Figma» occurs when the library gets a network error or a Figma API error. It is strange that adding nameValidateRegexp parameters fixes the issue. Could you please give me access to your Figma file so I can reproduce the issue?

Perhaps the library applies some default regex validation under the hood when we don't specify anything, but I couldn't find any information about this.

No, the library doesn't apply default regexp validation.

Another question is why it doesn't display any error message, when regex validation is commented.

Because the library unable to get colors from Figma API. I agree with you that it is not clear from the error message what the cause of the error is. It should print Figma API error message at least.

Hey @subdan
I just run into similar issue of  Error: Failed to load colors from Figma after bumping tool version to 0.38.0. On any earlier version exporting our figma project works fine, but on the 0.38.0 it throws an error as above.

Unfortunately that is something I cannot share outside of our company :(

@kiddden, @robertmusialek Which Figma plan do you have?

There is no Premium plan. To export variables you must have Enterprise plan.
https://www.figma.com/pricing/

I'm sorry, we use Professional Team plan

Sync variables to code via REST API only available for Enterprise plan

Do you use Variables or Color Styles for colors?

We use color styles

Did you try to use 0.37.6?

@kiddden did you set variablesColors in you figma-export.yaml file?

No, we haven't touched color variables at all


⚠ Jenkins Plugins Artifactory has degraded performance

In d2a346c, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1284 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 5869002 after 25 minutes.


Write a C++ program to check upper case

Description
Write a C++ program to check upper case
Check if the given character is upper case (A-Z).
Input  : A
Output : Upper Case

How to contribute

Save the solution in CheckUpperCase.cpp file
Add CheckUpperCase.cpp file in check-upper-case folder

!assign


How can you use CreateColorKeyFrameAnimation for animating Xaml Control Background?

Hi
The new Composition API looks great but I was wondering if there is a sample / instruction on applying a CreateColorKeyFrameAnimation on a Visual's Background property.
This is what I have so far but I keep getting a incorrect parameter error:
    private void AnimateBox(Visual targetVisual, Vector3 finalOffset)
    {
        var animation = _compositor.CreateColorKeyFrameAnimation();

        animation.Duration = TimeSpan.FromSeconds(1);

        animation.InsertKeyFrame(0.0f, Colors.White);

        animation.InsertKeyFrame(1.0f, Colors.Green);

        targetVisual.StartAnimation("Background.Color", animation);

    }

Hey @AudioW,
Thanks for reaching out with your question. Unfortunately, there isn't a property called Background off of Visual that you can animate. That is the reason for the incorrect parameter error. (check out this page for the list of properties off Visual).
That said, if you want to animate color, you can do so by calling StartAnimation() off of the SpriteVisual.Brush and targeting the color property - I put a short code snippet below.
    void ColorAnimation(SpriteVisual visual)
    {
        var colorAnimation = _compositor.CreateColorKeyFrameAnimation();
        colorAnimation.InsertKeyFrame(0.0f, Colors.Green);
        colorAnimation.InsertKeyFrame(1.0f, Colors.Cyan);
        colorAnimation.Duration = TimeSpan.FromSeconds(3);

        visual.Brush.StartAnimation("Color", colorAnimation);
    }

So in your code above, you need to change your StartAnimation to be off of a SpriteVisual's Brush property and then in the quotes, target the Color property. Is there a reason you were looking for a property called Background?

@AudioW To clarify what is going on here, Control.Background and other XAML Brush properties are not currently represented in the Visual returned by ElementCompositionPreview.GetElementVisual(). This is something we will look to improve in the future!
In the meantime, you can either:

Add a SpriteVisual to the tree using ElementCompositionPreview.SetElementChildVisual() and use the technique that @patsiu mentioned above.
Just animate the background brush using a ColorAnimation.

Option 2 is probably a bit easier, but option 1 will let you use other new Composition API's if you want!

Thank you for all the instructions!


Update primer/css version

In dotcom we have "@primer/css": "15.2.0", and in the Rakefile to build the docs we have<EMAIL_ADDRESS>which resolves to https://unpkg.com/<EMAIL_ADDRESS>
Ah, I see what's going on here.
The docs examples load Primer from a public CDN, not using the version we serve in dotcom.
We really need to update the PVC docs site to use the version of Primer CSS we're using on dotcom. Perhaps we could prioritize that ahead of this PR?
Originally posted by @joelhawksley in https://github.com/primer/view_components/pull/269#discussion_r582185654

@joelhawksley I think we are running on matching versions. Am I missing something?

@joelhawksley I think we are running on matching versions. Am I missing something?

I see that dotcom is using `"@primer/css-next": "0.0.0-6a1daf3"., which is the release candidate for v16.

Interesting! How should / could we keep things up to date? I can update the unpkg link manually now but this is going to break again...

Looking again, https://github.com/primer/css-next is a private mirror. I'm not sure that's going to work for this project...

Maybe @simurai could help us here?

@colebemis would it be possible to package and serve Primer CSS in the PVC doctocat site?

@joelhawksley I'm sure it's possible, but it might be tricky to scope the styles so they don't conflict with Doctocat styles.
I think the easiest solution would be to use https://unpkg.com/<EMAIL_ADDRESS>which should give us the same version that dotcom uses.
When https://github.com/primer/css/pull/1131 is merged, we can go back to using<EMAIL_ADDRESS>

Aha, I was not expecting the unpkg link for css-next to exist given the repo is private ;). I made that switch in https://github.com/primer/view_components/pull/278

Done via https://github.com/primer/view_components/pull/278


Git migration UX improvements

This PR aims to fix some bugs and improve overall migration UX.
Closes #9096
Bugs

GitHub token authentication had a bug where private repositories would not get their release assets correctly because the code was trying to retrieve them without auth.
GitLab's API wants a very specific format for their repo name, which would probably fail more often than not.

While munging the clone address we retrieved the "project path", however it retained .git at the end which would cause the client to fail

If you would like me to split out the bugs into a separate patch, I could probably take out the GitLab one. The GH one, however, mostly relies on other changes in this PR to get the downloader inside the uploader for retrieving assets with auth.
Enhancements

No more "username is a token" magic

If the git service supports a token, the UI will disable basic auth fields and handle it appropriately on the backend

Migration options are always shown, however they are disabled if no authentication is provided
Authentication remains optional for all service types, however just as before items will not be available

Overall I hope the UI is a little clearer as to what is expected from the user.
This PR was tested with a private repository on both GitHub/GitLab, and was able to fully migrate everything in both cases.
I was also able to do a plain migration from private repositories on BitBucket as well as Gitea.
UI Example

Codecov Report

Merging #12619 into master will increase coverage by 0.00%.
The diff coverage is 44.15%.

@@           Coverage Diff           @@
##           master   #12619   +/-   ##
=======================================
  Coverage   43.43%   43.44%
=======================================
  Files         645      645
  Lines       71288    71304   +16
=======================================
+ Hits        30967    30978   +11
- Misses      35309    35316    +7
+ Partials     5012     5010    -2

Impacted Files
Coverage Δ

modules/auth/repo_form.go
42.34% <ø> (ø)

modules/migrations/base/downloader.go
23.36% <ø> (ø)

modules/migrations/git.go
46.66% <0.00%> (-3.34%)
:arrow_down:

modules/migrations/gitea.go
7.07% <0.00%> (+0.35%)
:arrow_up:

modules/migrations/gitlab.go
1.14% <0.00%> (-0.59%)
:arrow_down:

modules/migrations/github.go
83.77% <61.90%> (+2.92%)
:arrow_up:

modules/migrations/migrate.go
24.40% <66.66%> (+2.56%)
:arrow_up:

modules/structs/repo.go
65.51% <72.22%> (+10.97%)
:arrow_up:

routers/repo/repo.go
32.71% <100.00%> (+0.17%)
:arrow_up:

models/repo_mirror.go
2.38% <0.00%> (-11.91%)
:arrow_down:

... and 16 more

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 69b3dd6...b686294. Read the comment docs.

Could you consider my propose on https://github.com/go-gitea/gitea/pull/9096#issuecomment-622860040

:rocket:

had a look how this afect the API ... and the only downside I noticed is that the GitServiceType is an integer
to change this to a string would require just too mouch ... the beter way is to document it and warp it on the Client sides
so the answer of this pull is: https://gitea.com/gitea/go-sdk/pulls/392

@6543 it can be made as enum in API also

@lafriks if we do so, we should cange it with in v1.13 to not inteoduce a api break

@6543 yes, that's right


System.Web namespace vs .net core

Summary
It's not a bug as much as a deep need. The System.Web namespace from ASP.NET is gone, and the individual replacements for the various things under it aren't documented for what replaces them in .net core. We need a document or something to explain what the replacement namespaces are.
Motivation and goals
1-2 paragraphs, or a bullet-pointed list. What existing pain points does this solve? What evidence shows it's valuable to solve this?
In scope
A list of major scenarios, perhaps in priority order.
Out of scope
Scenarios you explicitly want to exclude.
Risks / unknowns
How might developers misinterpret/misuse this? How might implementing it restrict us from other enhancements in the future? Also list any perf/security/correctness concerns.
Examples
Give brief examples of possible developer experiences (e.g., code they would write).
Don't be deeply concerned with how it would be implemented yet. Your examples could even be from other technology stacks.

@mjrousos should have some helpful pointers here.

Hi @rhires, I agree this is a space where more guidance is needed. I think ideally Upgrade Assistant would automatically replace some of those System.Web APIs (the ones with straightforward replacements) and alert the user (via warnings during execution and in a report afterwards) of other APIs that will need replaced manually.
Epic #55 is tracking work items related to implementing those sorts of analyzers and code fix providers. It's just a matter of getting them implemented.


Support configuring bottlerocket admin container image option

Issue #, if available:
#4050
Description of changes:
Patch from https://github.com/abhay-krishna/cluster-api/pull/3
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


Support Arduino standard and contributed libraries

Hi @ivankravets,
Do you think possible to import all the standard and contributed libraries referenced here on the official website?
For instance, I'm currently using the Wire and I had to manually copy sources into the lib/ folder of my project. I think it'd be cool to import such "official" libs using platformio's library management.

Hi @ncolomer,
The standard libraries are automatically bundled into Arduino Framework. Do you have any problems?

You're absolutely right... sorry for this dumb issue and thanks for your answer!
What about contributed libraries (such as MsTimer2)? If relevant, I can give a hand to import them into platformio.

Sure, I can add them. Can you help me with it? Because I don't have time for it now. As you can see, I'm busy with PlatformIO 1.0.0 release.
See example of library.json config https://github.com/ivankravets/platformio-libmirror/blob/master/configs/adafruit/arduino-ht1632.json
I propose you to use "pull-request" to owner of repo. For example, open https://github.com/PaulStoffregen/MsTimer2 then click on [+] near "Branches MsTimer2 / " and type name of file library.json. Then, you can grab example of config and edit it for the specific repo. if you will be ready, please inform me, I will look into your fork and correct something if it need.
What do you think about it? Thanks a lot for helping! :+1:

For sure! Thanks for the detailed procedure, I'll do that this weekend.

(currently working libs json)
Refering the initial subject. I observed that the EEPROM.h (seems a standard library too) is not automatically bundled into Arduino Framework.
Is it expected?

@ncolomer Do you use the latest develop branch?

See here how to install development version of PlatformIO.

I'm now running 0.11.0-dev
In addition to a more verbose output (lot of warning), the upload target still fail with the following error:
scons: warning: Two different environments were specified for target /Users/ncolomer/Developer/Workspace/perso/arduino/nrf24-server/.pioenvs/arduino_nano/libRadioHead_ID124.a,
	but they appear to have the same action: $AR $ARFLAGS $TARGET $SOURCES
$RANLIB $RANLIBFLAGS $TARGET
File "/Library/Python/2.7/site-packages/platformio/builder/tools/platformio.py", line 107, in BuildLibrary
.pioenvs/arduino_nano/RadioHead_ID124/STM32ArduinoCompat/HardwareSPI.cpp:6:20: fatal error: wirish.h: No such file or directory
 #include <wirish.h>
                    ^
compilation terminated.
scons: *** [.pioenvs/arduino_nano/RadioHead_ID124/STM32ArduinoCompat/HardwareSPI.o] Error 1

Surprisingly, the build target seems to only output warnings (ends with a success).
I took care to run a cleanbefore those tests.

Could you report this issue to PlatformIO repo with src details? I will test it tomorrow. Thanks :+1:

Will do this noon
Thanks a lot!

If you still have any problems with the latest 1.1.0 release, please open new issue in PlatformIO repo. Thanks :+1:


Update README.md

As a developer,
I need to update ReadMe file and add information,
so that I can show new configurations like PostSQL and Kubernetes.


[BUG] After update Dashy is not authenticating and guest mode is always on

Environment
Self-Hosted (Docker)
System
Docker  20.10.18
Version
2.1.1
Describe the problem
When opening my dashboard I noticed that I wasn't logged in (I couldn't change anything) but I could see my links, even if I tried manually setting enableGuestAccess: false on config.yml.
I tried hideForGuests: true in a couple sessions too, that cause the whole browser screen to disappear.
Rolling back to 2.1.0 fixed the problem.
Additional info
No response
Please tick the boxes

[X] You have explained the issue clearly, and included all relevant info
[X] You are using a supported version of Dashy
[X] You've checked that this issue hasn't already been raised
[X] You've checked the docs and troubleshooting guide
[X] You agree to the code of conduct

Same bug on 2.1.1, revert to 2.1.0 and authentications works again

The issue is that if a user is not logged on in 2.1.1 it will allow one to go to the base / URL and it won't enforce /login like 2.0.1.  I reverted for now this is an unreliable auth method in the meantime.

I am experiencing this, as well.

Same problem here, working fine when I roll back to 2.1.0

I had the same issue updating to 2.1.1, but rolling back to 2.1.0 did not resolve the issue. I had to roll back to 2.0.0 to get the login working again. It had been a while since the docker was updated, maybe there was some intermediate change to the config that was not checked in 2.1.0 that leads to a similar issue as this 2.1.1 bug.

Any update on this?

I got a similar issue on 2.1.1 - adding the appConfig.auth section with one user and setting enableGuestAccess to false doesn't do anything - I don't get any login screen, manually going to /login redirects to the dashboard, and all links are visible, even those that are hidden for guests.

I just noticed that the whole authentication thing seems to work when setting enableGuestAccess to true, but not when it's set to false

Unfortunately not for me. All versions higher than v2.1.0 break the login for me, no matter what enableGuestAccess is set to.

Have the same problem!
Hope it will be be resolved soon!

Same issue.
Hope to release a new version to fix it.

I believe I found the issue - see https://github.com/Lissy93/dashy/issues/1290#issuecomment-1884249018 for a full writeup.

Hi
This problems seems to be related to dashy not automatically rebuilding on changes, see #1479 and https://github.com/Lissy93/dashy/issues/1290#issuecomment-1884249018
A new release has just been published, which should fix this issue, so please upgrade to the latest version.
If this is not the case just respond back to this ticket and I'll reopen this issue.
Thanks and have a nice day.

Docker not yet updated?
% sudo docker compose up -d && sudo docker compose logs -f [+] Running 1/1  ✘ dashboard Error                                                                                1.8s  Error response from daemon: manifest for lissy93/dashy:2.1.2 not found: manifest unknown: manifest unknown

Docker not yet updated? % sudo docker compose up -d && sudo docker compose logs -f [+] Running 1/1 ✘ dashboard Error 1.8s Error response from daemon: manifest for lissy93/dashy:2.1.2 not found: manifest unknown: manifest unknown
I tried latest as well but that still gives me 2.1.1

https://github.com/Lissy93/dashy/discussions/1484#discussioncomment-8658525


Unexpected behaviour in laravel 5.3

First of all, thanks for creating such a nice package.
I have come across an unexpected behaviour in Laravel 5.3.
FYI: I am using latest release of this package
Only can sender load messages for a conversation from the db, the receiver cannot load messages from the db so I dig into the source code.
Found that the getMessagesById function in ConversationRepository.php file has the following query for the eager loading relations of messages.
return Conversation::with(['messages' => function ($query) use ($userId, $offset, $take) {
            $query->where(function ($qr) use ($userId) {
                $qr->where('user_id', '=', $userId)
                    ->where('deleted_from_sender', 0);
            })
            ->orWhere(function ($q) use ($userId) {
                $q->where('user_id', '!=', $userId)
                    ->where('deleted_from_receiver', 0);
            });

            $query->offset($offset)->take($take);

        }])->with(['userone', 'usertwo'])->find($conversationId);

which translates to
select * from messages where conversation_id in ($conversationId) and (user_id = $userId and deleted_from_sender = 0) or (user_id != $userId and deleted_from_receiver = 0).
Actually the query should look like this
select * from messages where conversation_id in ($conversationId) and ((user_id = $userId and deleted_from_sender = 0) or (user_id != $userId and deleted_from_receiver = 0))
and the code for the passed callback function for the message relations do the trick
$query->where(function ($qr) use ($userId) {
        $qr->where(function ($q) use ($userId) {
            $q->where('user_id', '=', $userId)
                ->where('deleted_from_sender', 0);
        })
        ->orWhere(function ($q) use ($userId) {
            $q->where('user_id', '!=', $userId)
                ->where('deleted_from_receiver', 0);
        });
    });
});

I don't know if it was the issue of Laravel 5.3. Please review. Thanks.

please see the Talk Example
https://github.com/nahid/talk-example


Spoofing Farm Added to Memphis TN[Addition]:

Hotspot b58 Addresses
11RW9hd1pNroSMWBq7HfmtrLYqBniWccv5JpnVDtNyFpcUdJ3mR
11Tgfx2msZ1VszPEj3FKh762XjcWiHpet9UGd9Fa512npcvocsA
Discord Handle
Corbin #7922
Reason(s)
On April 22 a wallet onboarded 22 miners in north memphis. Around the same time another wallet next to this group onboarded another 15 or so miners. These miners only talk to each other and do not interact with any of the normal miners there before this. They seem to only witness and send beacons to each other missing my miner and others well within range. Could you please look at these. I have attached TWO hotspot addresses one attached to each wallet. Their rewards show that they are only witnessed by themselves it looks like.

You need to attach the wallets associated with these hot spots. Not the hot spot address

It is more than 22, more like 100.  I reported it in ticket 4313 but I don't know if I got all the wallets or not.
Here are the wallets I included.  Maybe you can find more.
13xW3G39SvBEfAKfBy4rdaDMeqW7DVu3kYJ68KfMZ8JJPP1y8eg
13q7vx7Y71PXrstrm9YcixmrRsJKJNqv2iJdaJ51ah7nrejsMft
13xW3G39SvBEfAKfBy4rdaDMeqW7DVu3kYJ68KfMZ8JJPP1y8eg
13q7vx7Y71PXrstrm9YcixmrRsJKJNqv2iJdaJ51ah7nrejsMft

how long does it take to investigate and pull the spoofing farm down and those are the wallets i could find as well

As many as they have to investigate I am sure it will take a while.  Sooner the better.

i agree i have noticed a drop in rewards ever since it went up

As many as they have to investigate I am sure it will take a while. Sooner the better.

are
you in the memphis area

I am downtown on the island.  Since they that spoofing cluster went in I went from a transmit scale of .48 to .42 and everything else in the area seems to have dropped too.

I don't see that group on FB.  Can you send me a link to it.

its a brand new group so invite anyone you know in the helium community
https://www.facebook.com/groups/516805070151903/?ref=sharehttps://www.facebook.com/groups/516805070151903/?ref=share&exp=4a9f

11RW9hd1pNroSMWBq7HfmtrLYqBniWccv5JpnVDtNyFpcUdJ3mR

Wallet addresses are the addresses that should be listed. We try not to use whole wallets where possible.

I can confirm that you've identified a clear cluster.

Additional info:
ISP data for the majority is in Australia.
Asserted with 0.7 days.
Same maker ratio 1.0.
Hotspots
1126WXY5m96BfXUdKmRx4BDRBShrvLcaUdBCfT7CWYWsBfDNCey5
11RW9hd1pNroSMWBq7HfmtrLYqBniWccv5JpnVDtNyFpcUdJ3mR
112B24jG1J5cKZAsaXQKz38JJpthskzwbTHgLR5w2JhtaK3wd6Q9
112B85aynhbASfbLc3RJtkKgJgERp48yZL7AfXcKTdUQgSaN98HA
112BymFVhHpoMkZRuw585XxUEwdUHeJLd5mvd5xCydS81swPjdny
112Dz6tEH5xB2xK1p5H3iiM5mYBJ9GneVyoCPHBo113uceWbbcJw
112MQ8dc9khCtqM57ov19yJ5Gi3wq2W9bXvM2mHCZaqYg2VMKuHk
112uSLN8ZQ48NuJ4ty6VKhibRDWt82zWrQRMak5JaSkL6eR5mMZr
116teuZPY1TFx8zYoQ4sCBWNrX4LC9i3dnwH7rQSXqGBeFA64gf
117YwZTyRpHHe53VzZQbovjvqpnoMHFdKeDVXh6UcfXufSo4qR2
11DnU5FNNb9sS7jUe3ufib8iAwjJ9bku7sTLeFUs5de1BTT4tc1
11E3iqTovManrWzQ4SpVEbmVHArjocyU6w98M6H6g8YWXHPQWer
11Es4KutAESRBEzmPATt8Q5fNdu4ZvcPkZoFmxPJ68NUD5reyu4
11GR6cV88jNeJS9d6Rh3UnZyYooS5Sw9YeJYKFehEpYgwvhEU6u
11HP33JtKvNjGRL9apLQygeA64J5SAKvi3RV8pAzxCdwFEQ4AGa
11JznWebnVPLowpQQfFBHcHmfsRTojYxbA5uDxsj7Pn9FFaHHQ3
11TBiVUNPUDcAZXZUnu1kQHRnK2Qniou1BaWzYG5YarqjE9C5MQ
11Tgfx2msZ1VszPEj3FKh762XjcWiHpet9UGd9Fa512npcvocsA
11e8vF4zVtyjXZBkURZsrDSw3UY1gkNup9zVp8mutDnwp7DZfik
11eSiBVBW3WAXenihi6kFF5r98cAorwfbJLnkpWFhbYZR14Renx
11nM2ogdYpTUeSvKnD3boAqmqrAVDXH99Z8J4rroFCJsnYyCWos
11sNQN4TgyiQzp51H9PKjaAdCPKJBzKpHmgTWALKg9uLVAG8eWw
11ug8wqX37WKuedubdMX4ibK8GUkGDCjUbyjVaykFm4manmVZqM
1127eFSor6JL84ESei6uv4YMcRVAxyJrBcKQsY33rw1Le9Ywy383
112K4cJ4S4cyetiybc2g4jDsxQfKxPdpeCuyTiwfSSFc2c3FV3zE
112KwMXLhdfH8QhDUhJDjNdoBE3T1rrRSirGGuPSca77DY8tYaXe
112MzBEuah8KkAdUqw88ukW2yNB8Fmo9p25BR4VD4YwYF6mjL9fW
112U9rpNNMh7W5QigwvnWa7ZuqX8T88LbSorwKupKufbABhuf5Jx
112WPMewf9WuufeQYZKsCYwZcC5hv9ZZHV2zW2yWYXd9WG8JMcsj
112XQWJR9Pp2eH4djYQjzgTZKZmic5gYpoj5bB3kiNeXDLvyr6Bu
112aK9nnudzbRq3DEdaL8tZNFe2nTDkDP9LnfDtCv9yRb1kQpCZt
112bn8WLe3pVKFD6cHK9dqkbmct9LxgGcka8AXdVQYUHsxzvkPo
112f6Ui5BDprq6z9zihdpBU8iXpU5sXmUy4y5hdC1yw91hZZsyXC
112hAw9yW5FudnFm4UL3o3phsffd6nKArQnvCNX5BYsfpCVo6WsX
112hJFYeCYADAkiigKFHzysLcM3JmYaBc4btsfTbyvHi8VnMVdYP
112hVzyiCYTk2TGaGe5DMEG94RaJtDeRWxAfmnbd7XcmXV1iCX2G
112mcQWjhxwHYhT4f7uQUds3Sv1atBkE81knTy32NkaNDC8MRyH4
112pJnAi4psZweojYd6gSiHYm5Q1pa9JvYzumHL3efxyTWFYd8Dj
112qzZNLo7PAP9MpgqXBFNr4V1UGzcxmutW99c2s6jQDXVaptCZz
112tn2UJVQU2CKWeJnUt1tYs8Da2LN2vF1YjdGQ9MAvZfUv52kLN
1135ZpgztDaF2qTatZKku2thinqt9SWJAwv1ZMndYkFR2rWmg18
11HaSXgBpswv6fMVdyyUGMyG3GV8g2fpi4PTV3QF4hR7gRvUkn6
11Lyy91FqgHeHfzqYSaFvz5DTyJttQwgQkjhVGjAjnCZ5sC7pz7
11N1j5TRpckTJb9XxaLm7u1AMo6c7QeFLoKrvEx7PQyWnzeqFMc
11R7LU8ZdDyVjajDPAfEe23pu4YJtsPurWPHrXanY5ScjKpRvkH
11XUPsoPtAmKbBRfmXFc3m3yKWKeVdxxQ4SzzVUTePFpvk9VfLm
11Zr7d5bAXxXET3D9UZ5DYvKgbiDDbQ1d9hDmdseatg9eg5pg4E
11ag7d3F69mBo513niwGoJ4Ek7gXCNdc1uC84UQJDpD1bs15LDH
11ijbbeCpwnPxtMF5L3TwiFozMi2QQjnVZv3JEF5NSE4o6nuJSm
11jXnfGwCVYiaQ2Q7MBP46F5PUfBGJK2Nv4hdQsvAKCZ7tvUn6Q
11qvGU1GwhWsnncYcH9JrPFwYUnLCJyjiJwvTqwL5GXmJecrATJ
11sJu2fmm5WdXAqcA1ZVzQxNFxg3h9ebcniSvBYCjEXwwauE7pZ
11vxvFgotsNv8PrhyKEWPGUbeqHMLRP29bLk2hevWb4r9qiahpz
11ysko5edihd2fujC2U7LVLcouWcJvTWhfQxX6WyNVKcpqgGJNi

Does that mean that all of these hot spots will be put on the denial list? How long does that take?

There are 3 more wallets that are part of that cluster that were missed that I have found so far.  Can you please investigate these too and deni them
13xW3G39SvBEfAKfBy4rdaDMeqW7DVu3kYJ68KfMZ8JJPP1y8eg
13q7vx7Y71PXrstrm9YcixmrRsJKJNqv2iJdaJ51ah7nrejsMft
137LkftHz8t6Ya9T8FuNssb4Rg33pbcQyTCYZiKA3xiMgBJtxFN

There are 3 more wallets that are part of that cluster that were missed that I have found so far. Can you please investigate these too and deni them
13xW3G39SvBEfAKfBy4rdaDMeqW7DVu3kYJ68KfMZ8JJPP1y8eg 13q7vx7Y71PXrstrm9YcixmrRsJKJNqv2iJdaJ51ah7nrejsMft 137LkftHz8t6Ya9T8FuNssb4Rg33pbcQyTCYZiKA3xiMgBJtxFN

The hotspots are from those wallets.
13xW3G39SvBEfAKfBy4rdaDMeqW7DVu3kYJ68KfMZ8JJPP1y8eg
13q7vx7Y71PXrstrm9YcixmrRsJKJNqv2iJdaJ51ah7nrejsMft
137LkftHz8t6Ya9T8FuNssb4Rg33pbcQyTCYZiKA3xiMgBJtxFN
results-memphis.csv

abhay, why did you close this.  There are still many hotspots from this active.  The wallets are listed above.  Do you need me to get every hotspot on those wallets?  It is posted above.

@mmhoward1 Thank you for reporting the original issue. I’d suggest raising a new issue with the remaining hotspots. Not wallet accounts as the denylist uses hotspot addresses. The original hotspots and clusters connected to those were added to the denylist. In the comment above there is a csv that shows the hotspots added and wallet accounts they’re connected too.

mmhoward1    hey do you need help getting these hotspots and did you raise a new issue yet

Cheeto33,  I think it has all been taken care of here.
https://github.com/helium/denylist/issues/4810
Thanks

Thank you man. I have already seen my transmit scale start to go up some. That was the  biggest farm i have seen in memphis so far and can believe it was from Australia.

还有3个钱包是这个集群的一部分，我已经找到了到目前为止。你能不能也调查一下这些并否认它们
13xW3G39SvBEfAKfBy4rdaDMeqW7DVu3kYJ68KfMZ8JJPP1y8eg 13q7vx7Y71PXrstrm9YcixmrRsJKJNqv2iJdaJ51ah7nrejsMft 137LkftHz8t6Ya9T8FuNssb4Rg33pbcQyTCYZiKA3xiMgBJtxFN

热点来自那些钱包。 13xW3G39SvBEfAKfBy4rdaDMeqW7DVu3kYJ68KfMZ8JJPP1y8eg 13q7vx7Y71PXrstrm9YcixmrRsJKJNqv2iJdaJ51ah7nrejsMft 137LkftHz8t6Ya9T8FuNssb4Rg33pbcQyTCYZiKA3xiMgBJtxFN
results-memphis.csv

Dear friend, this is the hot spot that I and two friends bought for the first time from a Chinese businessman, many rules are not understood, and now I know that I need to deploy on the ground, we will quickly correct it and carry out field deployment, which has brought a bad experience to many miners, sorry！


Fountain Outline Failing to Find any Headings

Outline plane remains empty regardless of changes to headings in the .fountain file. Currently running Atom 1.23.3 and MacOS Mojave 10.14.6
Have tried the previous solution of selecting the .fountain file but no luck with that.
Additionally, it seems that the pdf preview is including the # characters and the text following them, where they should be omitted in the output file.

Can you provide a small snippet of the script that causes the problem?


how do i perform a json dump?

I tried querying with to_json, json_build_object or row_jo_json but had an error of function not available

Try again with v0.0.4


Reserved Instance Utilization

One of my customers would love to see a RI utilization component to the IaaS aspect of the CCO. There are APIs (https://docs.microsoft.com/en-us/rest/api/billing/enterprise/billing-enterprise-api-reserved-instance-usage) for this capability. Ideally there would also be a chart that illustrates how much IaaS is leveraging RI vs Pay as you go as well (by subscription or management group).
-Chris<EMAIL_ADDRESS>

Hi @chrislittle
We're curently investigating how to extract this data using other APIs because the Billing API only allows us to retrieve data using the enrollment number.


Python symbol for other ImageNet models

Thanks for such a great contribution with this repo!
This question is about training with different ImageNet models besides the default resnet_v1_101 that is set up here. In particular, it is about generating a python symbol file for other ImageNet models (eg. http://data.dmlc.ml/mxnet/models/imagenet/ ). It seems that .‘param’ and ‘.json’ files are available for many ImageNet models, but the python symbol files are harder to find. Moreover, the symbol files that I find online seem different from the symbol files in this repo in the ‘rfcn/symbols’ folder.
For example, the symbol file for a generic resnet model has the get_symbol function that looks like this:
def get_symbol(num_classes, num_layers, image_shape, conv_workspace=256, dtype='float32', **kwargs):
While the same function in symbol file in this repo for resnet_v1_101 in ‘rfcn/symbols’ looks like this
def get_symbol(self, cfg, is_train=True):
So, how can I find or create python symbol files for other ImageNet .param and .json files?
(This is my first time using mxnet; my experience is mainly r-fcn and faster r-cnn in caffe. I will appreciate your patience with my learning curve.)
Thanks again. Any help is much appreciated.
HashSolo

You can use symbols from MXNET examples (image-classification) https://github.com/apache/incubator-mxnet/tree/1.0.0/example/image-classification/symbols
Simply replace get_resnet_v1_conv4() with the feature for RPN and get_resnet_v1_conv5() for ROI pooling.
Pretrained models can be found here: http://data.mxnet.io/models/imagenet/


Not working for me.

I get no syntax colors when using this.
Atom V.1.9.9
Theme V.1.3.1

U need to upgrade Atom version to ^1.13.0, or downgrade package version to 1.2.0 apm install<EMAIL_ADDRESS>http://flight-manual.atom.io/using-atom/sections/atom-packages/


Automated cherry pick of #4862: fix: 仅kvm需要去判断是否需要关机调整配置，公有云私有云默认不需要

Cherry pick of #4862 on release/2.11.
#4862: fix: 仅kvm需要去判断是否需要关机调整配置，公有云私有云默认不需要

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11@b561ae3). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.11   #4868   +/-   ##
==============================================
  Coverage                ?   7.09%
==============================================
  Files                   ?     480
  Lines                   ?   78233
  Branches                ?       0
==============================================
  Hits                    ?    5552
  Misses                  ?   72215
  Partials                ?     466

Flag
Coverage Δ

#aFlag
7.09% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/guestdrivers/azure.go
7.81% <ø> (ø)

pkg/compute/guestdrivers/base.go
0% <0%> (ø)

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 b561ae3...84ca5cd. Read the comment docs.

/lgtm

/approve


nm, bond: Report only non-default settings

Except the mode bond option, report only the non-default option
values.
This change is avoiding an undesired merge of default values to the
desired state, which in turn causes defaults to be explicitly set on the
NM profile. Setting some defaults explicitly may cause validation errors
that are introduced by NM.

Coverage remained the same at ?% when pulling 22091a66a1d23f9da182a363c6bea0afa88a0b76 on EdDev:bond-options-defaults into 0f0995ef37cbbcb357df6ccdae0cf14d6ae21eb5 on nmstate:master.

Coverage remained the same at ?% when pulling 22091a66a1d23f9da182a363c6bea0afa88a0b76 on EdDev:bond-options-defaults into 0f0995ef37cbbcb357df6ccdae0cf14d6ae21eb5 on nmstate:master.

I don't like the approach.
Then on-disk profile different the one activated, this show command might showing the
incorrect data.
Regarding enforcing the default explicitly, I think we had this talk a while back: why we
create new profile instead of update changed values: everything should be explicitly
created in case default changes or on-disk profile does not match with run time.

@cathay4t , this change does not avoid setting the defaults explicitly if the user requests to do so. It does not complement defaults on merge nor reports them at all.
If someone change these values on the kernel, they will get reported and therefore explicitly set on-disk.
Per NM behavior, I see no other choice at the moment. NM does not allow setting explicitly all the defaults.
What scenario you see here as problematic?

Sorry. You are using NM.SettingBond.get_option_default() which should have no problems.
The change request would be:
Please don't use _get_bond_profile_setting(nm_device) to the NM.SettingBond which might provides None. You should use NM.SettingBond.new() instead, which is sufficient. For example:
NM.SettingBond.new().get_option_default('miimon')

Please don't use _get_bond_profile_setting(nm_device) to the NM.SettingBond which might provides None. You should use NM.SettingBond.new() instead, which is sufficient. For example:
NM.SettingBond.new().get_option_default('miimon')

Done, thanks.


Allow to re-send verification email from admin

Also add a filter to see users with unverified emails


Crypto wallet and currency tracker

Hi @Sulagna-Dutta-Roy ,
I hope you're doing well. I've addressed the issue #1590 and created a pull request to resolve it.
Could you please review the PR and let me know if any adjustments are needed?
Best regards,
Tanisha Garg

@Sulagna-Dutta-Roy Please review my last made PR.

@taneeshaa15 any update?

Mam actually in my github, directory is fine so I cant change it.
I am so sorry for that, I cant do anything more in this issue.
Still I can just request you to merge these issuesand I will take the utmost care from next time.
It is already changed as you said.

I didn't said it's changed, if it's changed than I should approve this earlier. So I commented for the update.
Anyway I am closing this PR because can't accept this without a change. If you want to make another PR for this then I will check. Thanks @taneeshaa15


fix: Disable no-descending-specificity lint rule

Disables the Stylelint no-descending-specificity rule because:

Stylelint cannot access our HTML code and therefore can only interpret usage from our CSS; and
This rule is more applicable to the old ways of writing CSS where selectors and descending specificity was more or less the way to go. Considering that we use SCSS and that we have much better and more modern methods of selecting elements, this rule is, in my opinion, a big time waster when trying to make the linter happy. This rule is outdated.

In my PRs on this project, I've spent more time than I should trying to make the linter happy by moving around blocks of code that I know will work. Moreover, you often find yourself in a situation where moving one block above another, as the linter suggests, just causes it to throw another error telling you to move it back. While I could refactor some of my UI by adding classes and writing CSS in the old style, this basically eliminates one of the biggest and most compelling features of SCSS and is, therefore, in my opinion, a major impediment to progress in this context.

@xplato IMO this is a great example of how to approach disabling a linter rule with a well explained justification!


Add token blacklist

We currently have a mock token blacklist and we need to add the real ones.

We are waiting for railgun to update their sdk so we can use the assertNotBlockedAddress or isBlockedAddress helper methods

Took a look at isBlockedAddress it seems like it is targeting the OFAC list. These aren't necessarily token addresses as far as I can tell.
https://github.com/Railgun-Community/shared-models/blob/183adb0ff23761a1c34396d714cb0af8dadac212/src/models/blocked-address.ts#L3


Error logging for POSTs to notary server

Adds error-level logging for 400 and 500 status codes during an update call to notary server, which can be caught by bugsnag.
Signed-off-by: Riyaz Faizullabhoy<EMAIL_ADDRESS>

Can we actually log the errors themselves too for 500's?  Other than that LGTM :)

@cyli: great idea, done :)

LGTM!  Thanks for adding the logging!


Update SDK to 219

pulls in https://github.com/specklesystems/speckle-sharp-sdk/pull/192

Note: now includes host app changes https://github.com/specklesystems/speckle-sharp-connectors/pull/456


Document CLI for job debug trace

https://cockroachlabs.atlassian.net/browse/CRDB-7076

As a note @livlobo and I discussed that the majority of debug commands are for internal use as mentioned in a section on each external-facing debug command in the docs. It seems like debug trace could be useful on the support side, but maybe not externally? Thinking about this in comparison to the cockroach debug backup [subcommand] cli tool (PR here #11841)

https://github.com/cockroachdb/cockroach/pull/65324


Add missing template file.

Add CSV template used by symposion's schedule_list_csv view. Without
this template, the view will return an error.

I think that this is actually a bug and not an enhancement - I think this is the only template for a symposion view that is not included in PinaxCon.
Please feel free to let me know if there are any changes that I can make to increase the chance of this being merged.

@paltman Any thoughts?


fix(#104): fix fs undefined in node < 12

I brought it back ?

@chrisdickinson , We need your opinion 😊

@viktor-ku  well, after reverting and adding some new stuff , it seems got messy !

@zacanger 👍 that's better

@zacanger that's good 👍

@ceejbot already added with #216   , the only different thing now is #211 , so I don't if shall I revert all the previous commits then push again to resolve those conflicts ,
#104 not decided yet


Implement pagination.

See issue #15
This works without any changes to linux-store-backend but by doing so may download unnecessary data when user doesn't view all pages.
I've opted for 25 entries per page, but it's easy to change if chosen otherwise.

Thanks for your contribution Antonio. I'll try it and talk with the designers in order to merge it

I think also the second solution might be the best. Maybe this could help developing it https://v5.material.angular.io/cdk/scrolling/overview


nullable return type for getConfirmedTransaction and getTransaction

I was trying to implement some logic in Laravel(Lumen) project to get confirmed transactions, however if tx not confirmed yet it may return null value according to documentation: https://docs.solana.com/ru/developing/clients/jsonrpc-api#gettransaction
Even thought I enclose my code in try/catch block I could not handle rising Laravel exception:
In Connection.php line 40:
  Tighten\SolanaPhpSdk\Connection::getConfirmedTransaction(): Return value must be of type array, null returned

So after I changed PHPDoc section and signatures for getConfirmedTransaction() and getTransaction() calls it start working properly.
Maybe I encounter very Laravel-specific case, but I still thing it's better to have nullable return-types as it one of the possible return value for mentioned calls.

Good catch! I'll look into it

oh.. so seems I can close PR now?


All slides visible with perPage: 1

I have the following configs:
galleryOptionsMain: {
        rewind: true,
        perPage: 1,
        perMove: 1,
        heightRatio: 0.66,
        pagination: false,
        arrows: false
},

galleryOptionsThumbnails: {
        rewind: true,
        gap: '1rem',
        pagination: false,
        arrows: false,
        fixedWidth: 110,
        fixedHeight: 70,
        cover: true,
        focus: 'center',
        isNavigation: true,
        updateOnMove: true
},

However my gallery slides seem to all have the class 'is-visible' applied to them, making them vertically stack. I'm using my own css but using the splide min css and can see Splide has correctly set a fixed width/height inline style on each.

Did you update CSS too?

Ah no I didn't thank you, working now https://github.com/Splidejs/vue-splide/issues/1


HSEARCH-4050 Remove restriction on exists() predicate

https://hibernate.atlassian.net/browse/HSEARCH-4050
Removing the restrictions seems to work fine. Need to check with Yoann to see what additional tests we should add.

I've added two more index bindings and removed assumeFullMultiIndexCompatibilityCheck(); from those modified tests. They seem to work fine for both Lucene and elasticsearch, so I wasn't sure if we needed to keep the assumption or not ..


Elevate entry_points.txt to being a PyPA interoperability specification

@jaraco, @takluyver has proposed that we elevate the entry_points.txt file format to being a PyPA interoperability specification, rather than being an internal implementation detail of setuptools: https://github.com/pypa/python-packaging-user-guide/pull/390
I'm inclined to say yes, since it will better allow new publishing tools like flit to declare plugins for existing applications that use pkg_resources for plugin management, and also allow new projects like entrypoints to experiment with alternate approaches to reading and caching the runtime lookup entry point metadata, without risking any compatibility breaks for current pkg_resources users.
However, I'd like to get an explicit +1 from you before we go ahead and do that. The main consequence is that if you later decided you wanted to change the entry points file format, that would now have to go through the PEP process, rather than being a purely internal setuptools design discussion.

The background is that flit already allows specifying entry points. I figured that the format was unlikely to suddenly change underneath me. But now that flit is gradually shifting from my experimental guerilla packaging tool to a serious piece of software that people recommend to other people, I felt the need to make it a bit more official that entry points could be written and read by different tools.
At present, entry points are also almost required to make packages with command-line interfaces that work across platforms, so packaging tools like flit have little choice but to write them. Donald is keen to redefine scripts into standard metadata, but popular tools like pytest and pygments use entry points to find plugins, so I'd still want to support them.

I'm enthusiastically +1.  The offer inversion of control at a very important layer (packaging). I'm happy to accept the consequences of elevating it to an interoperability standard. I believe this elevation was intended anyway when pkg_resources was separated and prepared for inclusion in the stdlib.


[FLINK-9599][rest] Implement generic mechanism to access uploaded files

What is the purpose of the change
This PR extends the existing multipart handling to also support mixed multipart message (i.e. requests  containing both JSON and files), and generalizes the FileUpload handling to provide access to all handlers extending AbstractHandler
Handlers may access uploaded files via HandlerRequest#getFileUploads. File uploads must be explicitly allowed by returning true in MessageHeaders#acceptsFileUploads.
The files and JSON payload are forwarded to the handler by the FileUploadHandler via channel attributes. If a JSON payload is forwarded this way a handler will ignore the content of the received HttpRequest.
This PR only covers the server-side; the RestClient remains unchanged. This will be done in a follow-up.
Brief change log

add MessageHeaders#acceptsFileUploads to signal that a handler accepts file uploads (default=false)
add FileUploads class as a container for uploaded files
extend FileUploadHandler to

properly accept multiple files in one request
also accept a JSON payload
forward both files and json via channel attributes

extend AbstractHandler to retrieve files/json from channel attributes

remove special case for JarRunHandler in AbstractHandler

extend HandlerRequest to accept a FileUploads object
add OkHttp dependency to flink-runtime for testing purposes
update JarUploadHandler/Headers

Verifying this change
added tests:

FileUploadsTest
FileUploadHandlerTest
manually verified via WebUI job submission

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (not documented)

@tillrohrmann I believe I've addressed all your comments.

Checkstyle errors:
18:31:10.780 [INFO] There are 2 errors reported by Checkstyle 8.4 with /tools/maven/checkstyle.xml ruleset.
18:31:10.788 [ERROR] src/main/java/org/apache/flink/runtime/rest/handler/FileUploads.java:[56] (regexp) RegexpSingleline: Trailing whitespace
18:31:10.804 [ERROR] src/test/java/org/apache/flink/runtime/rest/FileUploadHandlerTest.java:[99] (regexp) RegexpSingleline: Trailing whitespace

@tillrohrmann Ready for another review.

will change the logging and merge afterwards.


[SPARK-39707][SQL][DOCS] Add SQL reference for aggregate functions

What changes were proposed in this pull request?
Currently, Spark have a lot of built-in aggregate functions.
But the SQL Reference cannot find these aggregate functions directly and cannot describe the special syntax for PERCENTILE_CONT and PERCENTILE_DISC.
Why are the changes needed?
Add SQL reference for aggregate functions
Does this PR introduce any user-facing change?
'Yes'.
Users can find these aggregate functions directly and the special syntax for PERCENTILE_CONT and PERCENTILE_DISC.
Before this PR, the part 'Data Retrieval Statements' of 'SQL Syntax' show below.

After this PR, the part 'Data Retrieval Statements' of 'SQL Syntax' show below.

The page 'Aggregate Functions' show below.

How was this patch tested?
N/A

ping @cloud-fan
https://github.com/apache/spark/pull/37150 merged, please review this PR again.

@cloud-fan @dongjoon-hyun @MaxGekk Thank you !


For Quasar V2?

Hi there, they are a update for Quasar v2?

I currently have no plans to update it to V2 of Quasar, but would happily merge a PR if the work was done!


fix(ci): enable BUILD_HERMES_SOURCE on rn tester ios

Summary
Fix CircleCI build for rn tester.
Changelog
[Internal] [Fixed] - Fix rn tester app CI
Test Plan
CI should be green.

Thanks for doing this @matinzd 👍 Let me take a closer look but we should be good merging this if the CI is green

Thanks for doing this @matinzd 👍 Let me take a closer look but we should be good merging this if the CI is green

I converted it to a draft since there are some problems that need to be fixed. I will let you know when it's ready.


Disocvery should use async socket

Updated require statement to refer to cosock.socket instead of socket

Hello, I'd like to get this driver updated so that the discovery module is using cosock wrapped sockets since using unwrapped sockets will cause strange behavior in a driver.


Changed to age based fatality.

Looks good.  As I mentioned in slack, access to variant parameters will need to be updated after my PR #16.


talk: All metadata on 1 line (list simple)

Purpose
This is a proposal to have all "talk metadata" (i.e., event, location, date) on one single line under the title, rather than currently in two lines, on the talk_li_simple widget. This behaviour seems logical and more consistent with the presentation of publications in the equivalent widget.
It changes:

Location is added (not present before)
Order is changed towards: Event — Location, DateTime
Separators are added (— systematic after event and , between location and date IF there is a location)

Screenshots

I like this layout. Could we also flip the ordering of the talks? Currently, it is in reverse chronological order. But, I would prefer the talks be chronological order. Thank you for this amazing work!

@abyvinod: Thanks! Regarding the reversed order, I doubt it might become the default option (because currently the descending chronological order is more consistent with the way Publications are [traditionally] ordered in academic pages), but you can easily customize your own by modifying the file themes/hugo-academic/layouts/partials/section/talk.html at line 22, changing:
{{ range sort .Pages ".Params.time_start" "desc" }}

to:
{{ range sort .Pages ".Params.time_start" }}

These changes should also be carry forwarded to talk_li_detailed widget?

Only if you use it somewhere to display talks and want the same behaviour.

Note: in case a better way to display complex data items (typically: title, parent-item (event/company/...), location, date) is adopted globally, for instance as @gu-stat is proposing in #547 and #546 ("four corners" approach: respectively line1-left-aligned, line2-left-aligned, line2-right-aligned, line1-right-aligned), it would also be a great way to display Talk information. I think we need to choose on a global way to display such complex data.

Thanks for the suggestion. In 7eb10f2  , the format for talk metadata in the simple list is now all on one line.


Cannot declare fields on Codable class without initial values

Goals
Declare a Decimal128 on my Realm Object.
Expected Results
Schema discovery should succeed. If I add default values, schema discovery is successful, but for many Codable types, this feels redundant.
Actual Results
@objcMembers class Account: Object, ObjectKeyIdentifiable, Codable {
    dynamic var currency: String
    dynamic var balance: Decimal128
    dynamic var id: String

    override class func primaryKey() -> String? {
        "id"
    }
}

Steps for others to Reproduce
Declare the above model, open a Realm.
Version of Realm and Tooling
Realm framework version: 10.7.4
Xcode version: 12.5
iOS/OSX version: 14.5

This is not just for schema discovery. Reading data from the Realm also requires that we be able to create objects.


Correct interpolation in GEFS reader

Description
This resolves an issue where the GEFS reader could misinterpolate both the swdown and the lwdown fields using the budget_bilinear/conservative method.
This also fixes a bug in the temporal interpolation of swdown by initializing zdoy right before calling zterp.
Reforecast2 type

Operational type

Hi @jvgeiger:
Thank you for checking these fields and making these corrections.   I have reviewed and agree with your updates made.

Thanks @karsenau.

I placed a copy here: /discover/nobackup/jvgeiger/for/eric/gefsreforecast2/

Thanks @jvgeiger
Code compiles and runs w/o issue, and I reproduced your results.  Running with strict checks revealed no errors.  Since @karsenau concurs, I will accept the PR.


Rpc improvements for the indexer

Rpc improvements for the indexer:

Return list of transactions hashes to newHeads subscription
Add taraxa_getNodeStats to implement indexer consistency check without test rpc

prnet cleaned-up successfully


Add Event Overlay Support

We have a PR out for the Yelp version, but in the meantime we can add it here.
This adds event overlay support.
r? @sjung-stripe

the functionality looks good to me, but could we update the doc as well? this is a fairly complex nested structure and could use an example

Oh, derp. I couldn't find those docs. Thanks!

Docs added @sjung-stripe. Back atcha!

lgtm


Publish Surelog builds to PyPi?

As Surelog has a complete Python API, does it make sense for Surelog to be available via PyPi? If so it would be great to automatically publish PyPi packages.

Although Surelog Python API has some use for writing pre-processing and parser AST-level rules (Following the Antlr grammar),
the most value would be to have a UHDM Python binding.
The last changelist only converts the Tcl generator into a Python generator for the Surelog Python API. What we would need is a UHDM Python generator.
I'll let @hs-apotell decide if that is a task he can undertake.

@alaindargelas Need more information on what exactly do you mean by UHDM Python binding? You want to expose all of UHDM thru' parallel classes in Python?

Correct, that should be done automatically by code generation like the rest of UHDM.

Let me investigate this. I have never done extensive work on python bindings. I know AWS Boto3 does a pretty decent job. I will study that as an example. Do you know of any other libraries/projects that does anything similar (not necessarily in context to HDL)??

Surelog does C to Python binding itself for the antlr AST api using the python you just ported along with swig. it should be a very similar approach, there is no need for additional dependency in my mind. From the UHDM model, you auto generate what is required for swig to do the binding.

If you create a file uhdm.i and saveit  in UHDM/build/generated
/* File: uhdm.i */
%module uhdm
%include "typemaps.i"
%include "std_string.i"
%include "std_vector.i"
%include "uhdm/package.h"
%include "uhdm/uhdm.h"

cd UHDM/build/generated
swig -c++ -python -I. -o uhdm_wrap.cxx uhdm.i
uhdm/package.h:41: Error: Syntax error in input(1).
If we use syntax in the C++ headers of UHDM that Swig understands, then the Python wrappers come for "free".

A more complete .i file produces a Python wrapper that makes sense, it contains the classes enumerated.
You have to remove all the C++ syntax SWIG does not understand like all the RTTI stuff to get SWIG to produce the Python wrapper.
/* File: uhdm.i */
%module uhdm
%include "typemaps.i"
%include "std_string.i"
%include "std_vector.i"
%include "uhdm/BaseClass.h"
%include "uhdm/scope.h"
%include "uhdm/instance.h"
%include "uhdm/package.h"
%include "uhdm/uhdm.h"

The other part Surelog that sould be wrapped in Python is the design parsing and command line options "The main".
a Python session should be:
   surelog.init()         << specify and compile the design
   surelog.addFile "... "
   surelog.parse()
   design = surelog.compile()  << get UHDM design object
   foreach  module in design
      ...

Appreciate the pointers. I will try the approach. It might not be for the next few weeks though. Current focus is on LS and Linter implementation.

No problem, I don't think there is any urgency. For SWIG to be able to read the UHDM headers "as-is", It think some simple
#ifdef SWIG
....
#endif

in a few places like some of the RTTI class members, and the "final" keyword for classes will be enough. So the same exact headers can be used for the C++ compilation and the SWIG python binding generation.

BTW, are you going to contribute back the Linter or the LS? Have you noted the UhdmLinter and SynthSubset optional classes in UHDM?

Moving the LS conversation to https://github.com/chipsalliance/Surelog/issues/1712

@hzeller FYI

With the following SWIG file uhdm.i ran in UHDM/include:
%module uhdm
%{
#include "sv_vpi_user.h"
%}
%include <sv_vpi_user.h>

swig -I. -python -o uhdm_wrap.c uhdm.i
gcc -fPIC -I. -I/usr/include/python3.9/ -c uhdm_wrap.c
gcc -shared uhdm_wrap.o -o uhdmpython.so
You can produce the VPI API in Python. Link uhdmpython.so and uhdm.so in Python, plus a little wrapper for loading the surelog.uhdm and you should have the python binding of the "C" VPI UHDM. The C++ is much more work as described above.

FYI - @proppy / @hzeller / @kgugala

https://github.com/chipsalliance/Surelog/issues/2635#issuecomment-1077018391
I try to follow the provided instruction but it failed :
>>> import uhdmpython
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ImportError: uhdmpython.so: undefined symbol: vpi_register_assertion_cb
>>>

I found suspicious that even a dummy implementation of vpi_register_assertion_cb is not provided by UHDM.
But also that
nm uhdmpython.so | grep vpi_scan
return nothing ...
Is the proposed swig file correct ?

Hello
In my trial to follow:

You can produce the VPI API in Python. Link uhdmpython.so and uhdm.so in Python, plus a little wrapper for loading the surelog.uhdm and you should have the python binding of the "C" VPI UHDM. The C++ is much more work as described above.

UHDM.i:
%module uhdm
%{
#include "sv_vpi_user.h"
#include "swig_main.h"
%}
%include "std_vector.i"
%include "sv_vpi_user.h"
%include "swig_main.h"

%template(vpiHandleVector) std::vector<vpiHandle>;

swig_main.h:
#include <vector>
#include <uhdm/uhdm.h>

std::vector<vpiHandle> read_uhdm(std::string filename);

swig_main.cpp:
#include "swig_main.h"

std::vector<vpiHandle> read_uhdm(std::string filename){
  UHDM::Serializer serializer;
  return serializer.Restore(filename);
}

The swig cpp wrapper does not compile at my end with a weird error: related to this line of code %template(vpiHandleVector) std::vector<vpiHandle>; in my interface file:
/home/tom/prog/git/Surelog/third_party/UHDM/dbuild/python/CMakeFiles/swig_example.dir/UHDMPYTHON_wrap.cxx:3359:76: error: ‘type_name’ is not a member of ‘swig::traits<unsigned int>’
 3359 | urn traits<typename noconst_traits<Type >::noconst_type >::type_name();
      |     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~

This error seems to me an inconsistency in swig generated code...
I am using Ubuntu : Ubuntu 22.10, python 3.10.7, swig 4.0.2.
I see in swig release note that support to python 3.9 to python 3.11 and support of c++ 17 has only been added in swig 4.1.0 release in October 2022.
I should retry after updating swig to a later version.
But I am wondering if swig is still the right way to build this python wrapper ? (Availability of this swig version in workstation distribution, availability in the CI build system ...)

Problem remain with swig 4.1.0

@hs-apotell
I push the material here : https://github.com/Thomasb81/UHDM/tree/python_wrapper
The python/CMakeLists.txt with swig_add_library is not clean and have dependency issue on generated files. So from scratch if generate files does not exist it does not work.
So actually I work on this with
1. make debug
2. uncomment last line of the root CMakeLists.txt : #add_subdirectory(python)
3. cd dbuild
4. cmake .. -DBUILD_SHARED_LIBS:BOOL=ON -DCMAKE_BUILD_TYPE=debug
5. make swig_example

namespace std {
  %template(UIntVector) vector<unsigned int>;
}

This code does not make sense to me ... but solve the compilation problem.
Thanks

In the vpi_user.h we have following function prototype:
PLI_INT32 vpi_vprintf (PLI_BYTE8 *format,va_list ap);
PLI_INT32 vpi_mcd_vprintf (PLI_UINT32 mcd, PLI_BYTE8 *format, va_list ap);

This prototype produce following error at compilation time in there respective wrapper generated code:
UHDMPYTHON_wrap.cxx: In function ‘PyObject* _wrap_vpi_vprintf__SWIG_1(PyObject*, Py_ssize_t, PyObject**)’:
../UHDMPYTHON_wrap.cxx:10237:12: error: invalid array assignment
10237 |       arg2 = *temp;
      |       ~~~~~^~~~~~~

with in the generated code:
SWIGINTERN PyObject *_wrap_vpi_vprintf__SWIG_1(PyObject *SWIGUNUSEDPARM(self), Py_ssize_t nobjs, PyObject **swig_obj) {
...

 va_list arg2 ;
...
 void *argp2 ;
...
  res2 = SWIG_ConvertPtr(swig_obj[1], &argp2, SWIGTYPE_p_va_list,  0  | 0);
...
  va_list * temp = reinterpret_cast< va_list * >(argp2);
  arg2 = *temp;
...
  result = (PLI_INT32)vpi_vprintf(arg1,arg2);
...
}

It seems to me that the generated C++ code is wrong as claim by the compiler.
Not very clear how example and documentation should help me.
Support of va_list in swig 4.0
What is the difference of vpi_printf vs vpi_vprintf. It seems to me that both are variadic function.
Do we need to support both of them in a python wrapper ? Apparently support of vpi_printf seems ok.

You can comment out both functions in vpi_user.h, just leave a comment that they pose an issue with the SWIG generation, neither have an implementation in C at this point. They are not really needed. That are part of the VPI standard put one can print object content any way they choose to.

I find an elegant way to exclude them  with #ifndef SWIG / #endif arround declaration definition

I am confuse... does SV API suppose to be usable to modify or create uhdm db or is the intent is limited to get api ?

The VPI API (C API) is ready only (as it is supposed to be).
We can add a couple of new functions (Should not be in vpi_user.h) to

create (With Surelog also linked) the UHDM db
or
load (Only link UHDM) the UHDM db

I missed some comments here.
I think most users of this would rely on a built model and use the model in Python. Elaboration (-elabuhdm command in Surelog) is done in C++, you don't want to perform UHDM elaboration through python... Or if you want you can expose a simple function that invokes the optional uhdmelab.
Yes, there could be some use cases for mutation, but that also could be accommodated by creating a few new "C" routines that edit the model.
At any rate, we will know soon enough if https://github.com/chipsalliance/UHDM/pull/895 succeed.
If it does, there are still users who would rather use the Standard documented VPI "C" API rather than the C++ one. Basically exposing sv_vpi_user.h to Python.

Xref https://github.com/conda-forge/staged-recipes/pull/22208


spike: review Neovim scrollback pager comment on Kitty issue

Review https://github.com/kovidgoyal/kitty/issues/719#issuecomment-952039731 there is a ton of good information.

[ ] add ack for set title trick
[ ] add as an alternative

:tada: This issue has been resolved in version 1.1.10 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


README: update install steps

1.7 => 1.8 update

It's an example. It would be better if you could find a way to change or refine it to encourage users to find the latest version. That way we won't have to update this for every release.


In High-Contrast mode, the Title Bar should change color when the window is inactive

Describe the bug
As indicated here: https://docs.microsoft.com/en-us/windows/uwp/design/shell/title-bar

Do make is obvious when your window is active or inactive. At a minimum, change the color of the text, icons, and buttons in your title bar.

This is even more true when High-Contrast is activated to improve accessibility.
The Calculator currently uses the same color when the window is active and inactive.
Steps To Reproduce

Activate High-Contrast
Launch the Calculator
Switch between applications
--> the Title bar color doesn't change.

Expected behavior

the title bar should change color

Screenshots

Device and Application Information (please complete the following information):

OS Build: 10.0.18362.0
Architecture: X64
Application Version: 10.1812.10048.0

I just noticed the titlebar has also an issue with padding on the right:

Good catch. Outside of HC, we have acrylic/no acrylic, but we should definitely handle HC case as well.

Good catch. Outside of HC, we have acrylic/no acrylic, but we should definitely handle HC case as well.

The PR I sent only modify the UI when in HC, but we should probably also modify the color of the title when Acrylic is not activated (Battery Saver, Transparency Effects disabled by users in Settings > Personalization > Colors, ... ), I can include that in my current PR if you are ok.
Impossible to see that the Calculator doesn't have the focus:

we should probably also modify the color of the title when Acrylic is disabled... I can include that in my current PR if you are ok.

Good point. Yes, we can cover that as part of this PR.

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.

@grochocki: ok, I will try to update the PR on the following days (It's our annual Hackathon this week)

It appears this was fixed as part of #442, but was not caught by automation.


Miss linked event on document

At document
Events and Evaluating Javascript is same linked

yep I did that on purpose for now so its easier to find how to eval js, and to evaluate js you need to read about events


[FEATURE] Support queries on DB's e.g. SQLite and MySQL among others like https://python.langchain.com/docs/use_cases/qa_structured/sql

Is your feature request related to a problem? Please describe.
I want to query SQLite and MySQL databases using an AI Chatbot but don't want to integrate with Python
Describe the solution you'd like
Desire something like the mechanisms provided by https://python.langchain.com/docs/use_cases/qa_structured/sql
Describe alternatives you've considered
Python integration with LangChain

Implemented in https://github.com/langchain4j/langchain4j/pull/1056


(SERVER-2591) Display auth extensions for certificates

Something I ran into here is that if there are no alt names for a cert the whole field is skipped on display, which is more of an issue now that there's a potential third field after. If the output is meant to be parseable at all then each field needs to be displayed for all certs or none of them.

I don't think the default output is really meant to be parseable. We have a ticket for adding JSON formatted output: https://tickets.puppetlabs.com/browse/SERVER-2252.

Current output:
Signed Certificates:
    localhost                             (SHA256)  73:0E:9A:FD:14:1E:F0:9C:FA:F0:65:42:9B:B0:14:91:F5:3D:43:47:64:83:10:80:97:73:73:FC:37:A0:EB:C1	alt names: ["DNS:puppet", "DNS:localhost"]	authorization extensions: [pp_cli_auth: true]
    maggie                                (SHA256)  E4:1A:49:5C:70:D5:77:51:E0:02:7B:D8:C4:E3:CC:49:17:19:C5:27:0D:A4:D8:5C:1E:F4:B3:C1:59:B4:5F:36	authorization extensions: []
Revoked Certificates:
    testing       (SHA256)  5C:10:41:AA:02:8A:6B:B8:15:2A:4A:05:0A:1A:F4:41:7C:8E:90:42:13:AF:7A:2D:7D:B8:DA:FB:88:8A:85:13	authorization extensions: []

So looking at that, alt names aren't displayed when there aren't any, but auth exts are. I think we should pick one and be consistent. Since this out put is supposed to be human-readable, I think I lean towards not displaying it when it's empty. And then maybe we can also do the JSON ticket I mentioned some time soon.

+1 on removing the pry stuff from the Gemfile, since they're just developer tools.

Okay it's updated per feedback. I do think it might be worth having the discussion about output a little more broadly because though I've changed it here I don't think that I agree with it. As a heavy command line user I just want my software to output text that I can consistently parse with the basic unix cli tools, and json is not a solution to that.

Also fine putting that off.

Fair enough. What format makes the most sense to you? (We can either ticket it or fix it here, no strong opinion)

Also @justinstoller you filed the original format flag tickets. Thoughts on this?

Happy to have that conversation, @pcarlisle .
Originally, I had it in my head that the default CLI output should be a "pretty" formatted thing, somewhat based on the original output of puppet cert. This was informed by UX research that we did with puppet cert a long time ago (see the ticket and the UX test results and eventual PR). I think that sentiment gels with the kind of CLI output decisions that are happening elsewhere in the company (I'm thinking of Eric0's recent relay demo, specifically).
I tried to find current guidance that we may have for CLI tools and I didn't find any. I found the old puppet-nogui/patterns/cli_interaction.md, but it contains both "The default approach to the CLI should be non-interactive" & "Build output for humans first" which seem to somewhat contradictory. And the whole thing is really old.
I think that those guidelines I mentioned earlier are nice, but probably not company policy anymore and we can could be pretty free to do what we feel is best, as long as it's defensible. Assuming we're not going to add the --format flag anytime soon (which could support json and/or line oriented output), I don't have a strong opinion on the direction the default formatting should go now.


Datasources.txt, issues loading latest data

@marvinm2 your last update to the datasources.txt file is creating some issues.
It seems to me to be related to the number of fields you added.
Can you check, please?

https://github.com/bridgedb/BridgeDb/blob/master/org.bridgedb.bio/resources/org/bridgedb/bio/datasources.txt
Should be fine now! It had spaces instead of tabs, my mistake with pasting

@marvinm2, please run the test suite before you make a pull request ;)

I didn't know about that, it is why I asked for a review before the merge. I hope it is solved now anyway.

@marvinm2, next time, please give precise instructions of what you expect. For clarity, no worries, these things happen.
BTW, I think I've isolated what went wrong. I do normally compile things, and in the past I learned that Maven doesn't install a bundle if it has test fails. But that never was or no longer is the situation, and since things installed, I assumed the test suite ran fine. It didn't.

@marvinm2, please see the README for explanation on how to run the unit tests.
You almost fixed the problems :)


feat: pagination component on profile page EXLM-1797

Please always provide the Jira Issue your PR is for, as well as test URLs where your change can be observed (before and after):
Jira ID: https://jira.corp.adobe.com/browse/EXLM-1797
Test URLs:

Before: https://main--exlm--adobe-experience-league.hlx.live/en/profile/bookmarks-achievements?martech=off
After: https://feat-pagination-comp--exlm--adobe-experience-league.hlx.live/en/profile/bookmarks-achievements?martech=off

Please update the URL and rerun the aem-psi-check


Support for flex, flex-shrink and flex-grow

I sometimes have the need to set the flex property of an element. Right now I'd have to extend the Flex component using Styled Components or use inline styles (which don't get prefixed).
I was wondering if there is a reason that the flex property is not supported right now. If there's no reason than I'll gladly create a PR that implements this feature :).

It's not in the readme yet, but I just added a flex prop in last night – it should work on any of the components. E.g. <Box flex='1 1 auto' />

Ah, right. Thanks :)


(WIP) Backend API v2

Description
@bnjmnt4n (Benjamin Tan) and @qreoct (Jefferson Lim) are the Backend team for SA2122. We are doing a v2 refactor of the Backend API. This branch is to accommodate all the changes and used for us to make sure the frontend & backend endpoints work together.
Current backend API changes can be found in the cadet repository.
Type of change

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

How to test
No 'automated' testing so far other than integrating both frontend and backend in development. This DOES NOT work with current (v1) backend.
We have to test it on a staging build before this change can be pushed out.

Pull Request Test Coverage Report for Build 682543348

1 of 16 (6.25%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 28.383%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/features/storySimulator/StorySimulatorRequest.ts
0
1
0.0%

src/commons/sagas/RequestsSaga.ts
1
15
6.67%

Totals

Change from base Build 679073259:
0.0%

Covered Lines:
2246

Relevant Lines:
7253

💛  - Coveralls

We have completed backend changes and awaiting finalization there.
Current discovered issues with integration:

Toggling publish status of assessments is not reflected in UI nor backend. Definitely something to do with POST admin/assessments/{id}
Certain calls to GET /assessments/0 crashes a React component (regardless of authorization status) upon login. Have to investigate further.

Update:
Both issues have been fixed. 1 was due to changes in the /assessments endpoints, and 2 was due to a weird edge case where users do not have a game room ID. (I'm guessing this would only occur for admin/staff or in dev only since SA users would typically have a game associated with them?)


Use java8 type in snapshot tests

Have to investigate the missing property as a separate bug. Will merge this PR for now.


Opening of endless chrome tabs

I'm having issues adding a torrent from an RSS fee on the app.
Every time I select an RSS torrent in the app, it opens chrome and starts opening an endless amount of tabs which seem to try and download the file (figures it was endless, max count I saw before i stopped it was about 23). I've tried it a few times, but keep getting the same result.
Am I doing something wrong?
Reported by seltzermx in qbforums

Extra info provided by seltzermx:
After testing with the site itself, other sites, and other RSS feeds, I have narrowed the issue down to a specific site/feed.
The feed/site that fails is nyaa.se. Selecting an item in the feed, or even selecting the magnet or torrent from the site itself in Chrome sends Chrome into an endless loop of trying to download the file but being unable to. Sorry for not having tested thoroughly before posting this.
For reference, I'm using an unlocked HTC 10 running MM.
I still have my older M8 sitting around. I'll try if there this evening and see if the behavior is the same.

Extra info provided by seltzermx:
After testing with the site itself, other sites, and other RSS feeds, I have narrowed the issue down to a specific site/feed.
The feed/site that fails is nyaa.se. Selecting an item in the feed, or even selecting the magnet or torrent from the site itself in Chrome sends Chrome into an endless loop of trying to download the file but being unable to. Sorry for not having tested thoroughly before posting this.
For reference, I'm using an unlocked HTC 10 running MM.
I still have my older M8 sitting around. I'll try if there this evening and see if the behavior is the same.


PSTH grouped on non-categorical column

It is currently possible to group PSTH on a non-categorical column. Here, I am grouping based on RT, reaction time, which is continuous, so the groups are very small and the PSTH is awkward. @magland how are you determining which columns to make available for grouping? Is there any logic to exclude continuously valued columns?
https://flatironinstitute.github.io/neurosift/?p=/nwb&url=https://dandiarchive.s3.amazonaws.com/blobs/d0c/495/d0c495db-37d5-4257-90d4-ff22847921fa

@bendichter I don't think I restrict it at all. Do you have a suggestion for how to determine whether it's categorical? Count the number of unique values?

@magland here is what I use in NWB Widgets, though I will admit it's a bit of a hack:
https://github.com/NeurodataWithoutBorders/nwbwidgets/blob/9885afc44083dda78b3f0fbadf00dfd92aaed455/nwbwidgets/utils/dynamictable.py#L23

Okay, I implemented a similar logic for picking out the categorical columns. One difference is that I still include columns with only one category... I don't think there's a problem with that.


Add ability to include custom middlewares for http server, refactored tracing middleware

Do not merge yet, still in development

Added related SpanKind to spans to differentiate them:

request / publish -> SpanKind.SERVER
listen -> SpanKind.CLIENT
Refactored span name to reflect exact action
Added ability for request/publish span to follow tracing parent span
Optional verbose flag for verbose events (with request/response data)


[CIT-352] Add tedge-dm-agent service file

Signed-off-by: Lukasz Woznicki<EMAIL_ADDRESS>Proposed changes
Add service file for tedge-dm-agent
Types of changes
What types of changes does your code introduce to thin-edge.io?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Documentation Update (if none of the other choices apply)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING doc
[x] I have signed the CLA (in all commits with git commit -s)
[ ] I ran cargo fmt as mentioned in CODING_GUIDELINES
[ ] I used cargo clippy as mentioned in CODING_GUIDELINES
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Codecov Report

Merging #233 (e588828) into main (6d3fff6) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #233   +/-   ##
=======================================
  Coverage   64.85%   64.85%
=======================================
  Files          56       56
  Lines        3224     3224
=======================================
  Hits         2091     2091
  Misses       1133     1133

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 6d3fff6...e588828. Read the comment docs.


Jekyll-Scholar and Al-Folio: undefined method `exists?'

Hello, (new user here)
I decided to create a new issue to reference the problem described in https://github.com/alshedivat/al-folio/issues/1116#issuecomment-1383387483 as this is a separate bug from what was originally mentioned in that thread.
In short, when I run bundle exec jekyll serve, I get the following error, which I suspect is due to jekyll-scholar not being able to compile the bibliography.
 Liquid Exception: undefined method `exists?' for File:Class in C:/Users/shrey/Documents/Website/al-folio-master/_layouts/about.html
                    ------------------------------------------------
      Jekyll 4.3.1   Please append `--trace` to the `serve` command
                     for any additional information or backtrace.
                    ------------------------------------------------
C:/Ruby32-x64/lib/ruby/gems/3.2.0/gems/jekyll-scholar-7.1.1/lib/jekyll/scholar/utilities.rb:471:in `scholar_source': undefined method `exists?' for File:Class (NoMethodError)

        return source if source.start_with?('/') && File.exists?(source)
                                                        ^^^^^^^^
Did you mean?  exist?

Here's a few things I tried:

I went into scholar/utilities.rb and tried changing exists?(name) ---> exist?(name). After this, I was able to run bundle exec jekyll serve without errors. Unfortunately, my bibliography did not compile and as a result, I wasn't able to get my publication page to load.

To check if the problem was with jekyll-scholar, I tried running a different jekyll theme. But my bibliography seemed to compile okay there.

Updated bundle, rubygems, and jekyll.

Other details: Windows computer.
I'm stuck now and I would really appreciate some help on dealing with this issue. Thanks a lot!

same issue here

I was able to fix this by going into C:/Ruby32-x64/lib/ruby/gems/3.2.0/gems/jekyll-scholar-7.1.1/lib/jekyll/scholar/utilities.rb and replacing all instances of


8kHz version?

Is it possible to also run this with 8kHz models? Either by compiling it separately, or by doing both in the same VM?

Let's try! I bet we can use the same VM.
On 04/03/2016 09:35 AM, Florian Metze wrote:

Is it possible to also run this with 8kHz models? Either by compiling it separately, or by doing both in the same VM?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub https://github.com/srvk/eesen-transcriber/issues/8

--
Eric Riebling      Interactive Systems Lab
<EMAIL_ADDRESS>  407 South Craig St.

i think we probably want to create a different branch in the repository, then we make a few changes (re-sampling to 8kHz) and then we just need to load a different set of models (trained on switchboard), which we have, but which we cannot make available for public download, i think.
F.

On Apr 4, 2016, at 9:29 AM, riebling<EMAIL_ADDRESS>wrote:
Let's try! I bet we can use the same VM.
On 04/03/2016 09:35 AM, Florian Metze wrote:

Is it possible to also run this with 8kHz models? Either by compiling it separately, or by doing both in the same VM?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub https://github.com/srvk/eesen-transcriber/issues/8

--
Eric Riebling Interactive Systems Lab
<EMAIL_ADDRESS>407 South Craig St.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub https://github.com/srvk/eesen-transcriber/issues/8#issuecomment-205296400

Florian Metze http://www.cs.cmu.edu/directory/florian-metze
Associate Research Professor
Carnegie Mellon University

8khz models now supported, so long as we remember the 'secret URL' to give folks who are entitled to download the 8k models


DRC PL.2_3.3V mixes gate length rules with gate width rules

Expected Behavior
W=0.22um, L=0.28um (the minimal PCell Size) shouldn't fail DRC
Actual Behavior

Steps to Reproduce the Problem

Version: 11770e5e5bae2a2072899e7ef282c080e6c5d762

Instantiate a PCell with the default parameters
Run Klayout DRC

Possible Cause
Rule PL.2_3.3V in gf180mcu.drc doesn't seem to check width and length separately (i.e. it assumes device width is always larger than length)
References
gf180mcu PDK Physical Verification
@msaligane

/cc @atorkmabrains

@atorkmabrains This issue has been addressed in Efabless/PV repo

Thanks @FaragElsayed2


multiprocessing problem in from_root_file.py

Hello,  I encountered a problem when I  fit some HAWC private data on my laptop. I have successfully installed threeML and hawc_hal. When I did the fit using  fitModel.py,  It crashed at line 249 in from_root_file.py  (https://github.com/threeML/hawc_hal/blob/cd2b4b5071c2e8e574c3938b942ade52ee364036/hawc_hal/maptree/from_root_file.py#L249). It seems that the problem is produced by the multiprocessing module. Also, It seems that that module starts more than one process("Srarting 3ML" occurred 3 times). If I run the same script on a HAWC server, it can work smoothly.
My laptop information:
MacBook Pro
Apple M2 Max
macOS Ventura version 13.5
Xcode Version 15.0.1
miniconda https://repo.anaconda.com/miniconda/Miniconda3-py311_23.11.0-2-MacOS-x86_64.sh (I alse tried arm64, it also has the same problem )
I also set environment when installing threeML:
export OMP_NUM_THREADS=1
export MKL_NUM_THREADS=1
export NUMEXPR_NUM_THREADS=1

19:20:33 INFO      Starting 3ML!                                                                                init.py:35
(Some warning )
19:20:44 WARNING   No fermitools installed                                                            lat_transient_builder.py:44
…………………………
Performing likelihood analysis...
Specifying ntransits currently does not work in hal, this is all @chad's fault.
Loading as MODEL file: new_source.model
……………………………………
Using analyis bins [……………………]
19:20:37 INFO      Using transits contained in maptree                                                                  HAL.py:79
INFO      Reading Maptree!                                                                         from_root_file.py:214
()
…………………………………………
19:20:42 INFO      Starting 3ML!                                                                                init.py:35
(Some warning information)……………………
19:20:44 WARNING   No fermitools installed                                                            lat_transient_builder.py:44
Performing likelihood analysis...
Specifying ntransits currently does not work in hal, this is all @chad's fault.
Loading as MODEL file: new_source.model
(Some warning information)……………………
Using analyis bins [……………………]
19:20:46 INFO      Using transits contained in maptree                                                                  HAL.py:79
INFO      Reading Maptree!                                                                         from_root_file.py:214
Traceback (most recent call last):
File "", line 1, in
File "/Users/zwang/software/hawc_software_test/miniconda3/envs/hal_analysis/lib/python3.11/multiprocessing/spawn.py", line 122, in spawn_main
exitcode = _main(fd, parent_sentinel)
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/zwang/software/hawc_software_test/miniconda3/envs/hal_analysis/lib/python3.11/multiprocessing/spawn.py", line 131, in _main
prepare(preparation_data)
File "/Users/zwang/software/hawc_software_test/miniconda3/envs/hal_analysis/lib/python3.11/multiprocessing/spawn.py", line 246, in prepare
_fixup_main_from_path(data['init_main_from_path'])
File "/Users/zwang/software/hawc_software_test/miniconda3/envs/hal_analysis/lib/python3.11/multiprocessing/spawn.py", line 297, in _fixup_main_from_path
main_content = runpy.run_path(main_path,
^^^^^^^^^^^^^^^^^^^^^^^^^
File "", line 291, in run_path
File "", line 98, in _run_module_code
File "", line 88, in _run_code
File "/Users/zwang/work/source_ana/new_source/fitModel.py", line 515, in
like = HAL("HAWC", map_tree, det_res, roi, options.pixelwidth) #,n_transits)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/zwang/software/hawc_software_test/miniconda3/envs/hal_analysis/lib/python3.11/site-packages/hawc_hal/HAL.py", line 88, in init
self._maptree = map_tree_factory(
^^^^^^^^^^^^^^^^^
File "/Users/zwang/software/hawc_software_test/miniconda3/envs/hal_analysis/lib/python3.11/site-packages/hawc_hal/maptree/map_tree.py", line 26, in map_tree_factory
return MapTree.from_root_file(map_tree_file, roi, n_transits, n_workers)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/zwang/software/hawc_software_test/miniconda3/envs/hal_analysis/lib/python3.11/site-packages/hawc_hal/maptree/map_tree.py", line 64, in from_root_file
data_analysis_bins, transits = from_root_file(
^^^^^^^^^^^^^^^
File "/Users/zwang/software/hawc_software_test/miniconda3/envs/hal_analysis/lib/python3.11/site-packages/hawc_hal/maptree/from_root_file.py", line 249, in from_root_file
with multiprocessing.Pool(processes=n_workers) as executor:
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/zwang/software/hawc_software_test/miniconda3/envs/hal_analysis/lib/python3.11/multiprocessing/context.py", line 119, in Pool
return Pool(processes, initializer, initargs, maxtasksperchild,
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/zwang/software/hawc_software_test/miniconda3/envs/hal_analysis/lib/python3.11/multiprocessing/pool.py", line 215, in init
self._repopulate_pool()
File "/Users/zwang/software/hawc_software_test/miniconda3/envs/hal_analysis/lib/python3.11/multiprocessing/pool.py", line 306, in _repopulate_pool
return self._repopulate_pool_static(self._ctx, self.Process,
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/zwang/software/hawc_software_test/miniconda3/envs/hal_analysis/lib/python3.11/multiprocessing/pool.py", line 329, in _repopulate_pool_static
w.start()
File "/Users/zwang/software/hawc_software_test/miniconda3/envs/hal_analysis/lib/python3.11/multiprocessing/process.py", line 121, in start
self._popen = self._Popen(self)
^^^^^^^^^^^^^^^^^
File "/Users/zwang/software/hawc_software_test/miniconda3/envs/hal_analysis/lib/python3.11/multiprocessing/context.py", line 288, in _Popen
return Popen(process_obj)
^^^^^^^^^^^^^^^^^^
File "/Users/zwang/software/hawc_software_test/miniconda3/envs/hal_analysis/lib/python3.11/multiprocessing/popen_spawn_posix.py", line 32, in init
super().init(process_obj)
File "/Users/zwang/software/hawc_software_test/miniconda3/envs/hal_analysis/lib/python3.11/multiprocessing/popen_fork.py", line 19, in init
self._launch(process_obj)
File "/Users/zwang/software/hawc_software_test/miniconda3/envs/hal_analysis/lib/python3.11/multiprocessing/popen_spawn_posix.py", line 42, in _launch
prep_data = spawn.get_preparation_data(process_obj._name)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/zwang/software/hawc_software_test/miniconda3/envs/hal_analysis/lib/python3.11/multiprocessing/spawn.py", line 164, in get_preparation_data
_check_not_importing_main()
File "/Users/zwang/software/hawc_software_test/miniconda3/envs/hal_analysis/lib/python3.11/multiprocessing/spawn.py", line 140, in _check_not_importing_main
raise RuntimeError('''
RuntimeError:
An attempt has been made to start a new process before the
current process has finished its bootstrapping phase.
    This probably means that you are not using fork to start your
    child processes and you have forgotten to use the proper idiom
    in the main module:

        if __name__ == '__main__':
            freeze_support()
            ...

    The "freeze_support()" line can be omitted if the program
    is not going to be frozen to produce an executable.

    To fix this issue, refer to the "Safe importing of main module"
    section in https://docs.python.org/3/library/multiprocessing.html

19:20:52 INFO      Starting 3ML!                                                                                init.py:35
WARNING   WARNINGs here are NOT errors                                                                    init.py:36
WARNING   but are inform you about optional packages that can be installed                                init.py:37
WARNING    to disable these messages, turn off start_warning in your config file                          init.py:40
WARNING   Multinest minimizer not available
……………………

Hello @Catbohe
Seems that the problem is related to try to install package that are only available for x86 arch, and not for arm64 arc.
I installed threeml and HAL in a Macbook pro with M3 chip and read the HAWC maptree using multiprocessing and did not find any errors.
As for now, astromodels is not available for arm64 arch, and when trying to install using conda, the system will complaint that there is not match between the astromodels and threeml available versions. And trying to install using x86-specific packages do not help as they are optimized for another arch.
Please try the following to check if you can solve the problem:

Erase your current virtual environment
Reinstall conda (miniconda) for arm64 arch
Use the following instruction to create the virtual environment:
conda create -y --name hal_env -c conda-forge numpy scipy matplotlib ipython "numba<0.57" reproject root python=3.10 "uproot<5.2" jupyter notebook
Please note that the python version is fixed to 3.10. The same goes for uproot (uproot<5.2)
Then, activate the virtual environment and install additional packages if needed (ebltable, naima, etc)
Install HAL using pip:

python -m pip install git+https://github.com/threeml/astromodels
python -m pip install git+https://github.com/threeml/threeml
python -m pip install git+https://github.com/threeml/hawc_hal

Let me know if you find any other potential errors.

Hi

Hello @Catbohe Seems that the problem is related to try to install package that are only available for x86 arch, and not for arm64 arc.
I installed threeml and HAL in a Macbook pro with M3 chip and read the HAWC maptree using multiprocessing and did not find any errors.
As for now, astromodels is not available for arm64 arch, and when trying to install using conda, the system will complaint that there is not match between the astromodels and threeml available versions. And trying to install using x86-specific packages do not help as they are optimized for another arch.
Please try the following to check if you can solve the problem:

Erase your current virtual environment
Reinstall conda (miniconda) for arm64 arch
Use the following instruction to create the virtual environment:

conda create -y --name hal_env -c conda-forge numpy scipy matplotlib ipython "numba<0.57" reproject root python=3.10 "uproot<5.2" jupyter notebook

Please note that the python version is fixed to 3.10. The same goes for uproot (uproot<5.2)

Then, activate the virtual environment and install additional packages if needed (ebltable, naima, etc)
Install HAL using pip:

python -m pip install git+https://github.com/threeml/astromodels
python -m pip install git+https://github.com/threeml/threeml
python -m pip install git+https://github.com/threeml/hawc_hal

Let me know if you find any other potential errors.

Hi,
Thanks for your reply.
I followed your instructions but it still does not work.

Hi @Catbohe
Can you provide text file with the error?
BTW, I think the only thing I have different now, it is my configuration file where I set to use parallel by default, but I remember to ran the example without that.

About the multiple init messages, that's the expected output, as the script is running in n number of cores you specified to read the map tree.


No license

I'd like to be able to use this as a foundation for other work, but without a license present on this repo, I can't do that. Since some of your other repos are MIT, I was wondering if you could license this MIT as well.
Thank you.

hey! thanks for pointing this out! I've added the MIT license!


Memory profiler

Profile most intensive functions: GRM, pooling LOF, etc.

Implemented. Could use some improvements, ask Sacha what would be best for him.


psycopg2 or psycopg2-binary

In the requirements we ask for psycopg2 which will be usually fine if you install titiler-pgstac on an environment where you have PG installed but it can be difficult in light environments like AWS Lambda where you would prefer installing psycopg2-binary
https://github.com/stac-utils/titiler-pgstac/blob/84ebde28afbdcdb128537436438902869f54f17a/setup.py#L12
See https://github.com/developmentseed/eoAPI/blob/master/deployment/dockerfiles/Dockerfile.raster#L3-L14 (basically I had to install PG, install titiler-pgstac, remove psycopg2 files and then install psycopg2-binary
I'm not sure what's the best way to have option to install either psycopg2 or psycopg2-binary 🤷♂

A couple notes:
The psycopg2-binary package says:

The binary package is a practical choice for development and testing but in production it is advised to use the package built from sources

One option is to require the end user to choose, by not installing either dependency with the base bundle, then having titiler.pgstac[psycopg2] and titiler.pgstac[psycopg2-binary] extras

🙏 thanks @kylebarron
I kinda like the environment variable approach as in https://github.com/dbt-labs/dbt/blob/45fe76eef4b4b82ae1442f00310e0c6a121774f2/plugins/postgres/setup.py#L22-L73

That's cool. I like that. On the flip side... what happens if you install both packages? Which takes precedence? I think the environment variable approach gets annoying if you become a dependency of another package and then the env var switching isn't obvious. 🤷♂
It would be kinda nice if you could install both packages and then the non-binary would take preference.

I'm not sure about precedence. I think the safest way for now is to remove psycopg2 and add a note in the Readme.