request for BatchNormalization3D

Is chainer.links.BatchNormalization can only support 2D feature map ? How to write BatchNormalization3D in chainer?
Furthermore, pytorch has already support the BN3D here:
https://pytorch.org/docs/master/nn.html?highlight=batchnorm3d#torch.nn.BatchNorm3d
it is 5D input (a mini-batch of 3D inputs with additional channel dimension)

Hi, L.BatchNormalization supports axis option, which specifies axes along which we take averages. I guess BN can accept images other than 2D by using that.

@delta2323 no, L.BatchNormalization does not have axis option.  following
https://docs.chainer.org/en/stable/reference/generated/chainer.links.BatchNormalization.html?highlight=batch normalization

axis option merged at #4671 will be released on next v5.0.0b1

Ah, sorry. Yes, as @disktnk pointed out, it is to be introduced in the next major version.

I've replied to your question on SO.
https://stackoverflow.com/questions/50303841/how-to-use-batchnormalization3d-in-chainer


Template stuff

Added html template.
Input payload:
{
    "personaliaSegment": {
        "firstName": "John",
        "lastName": "Doe",
        "email"<EMAIL_ADDRESS>        "phone": "12345678",
        "dateOfBirth": "2022-10-31T23:00:00.000Z",
        "gender": "Male",
        "address": "Streetname 123",
        "address2": "",
        "zipCode": "1234",
        "city": "New york",
        "country": "United States",
        "summary": "I am a cool guy.",
        "profilePicture": null
    },
    "educationSegment": {
        "educationList": [
            {
                "school": "School of Technology",
                "degree": "Pro programmer",
                "startDate": "2024-02-29T23:00:00.000Z",
                "endDate": "2024-08-31T22:00:00.000Z",
                "isStudying": false
            }
        ]
    },
    "experienceSegment": {
        "experienceList": [
            {
                "company": "Best Company",
                "position": "Pro Developer",
                "startDate": "2024-08-31T22:00:00.000Z",
                "endDate": "2024-09-10T16:19:37.257Z",
                "isWorking": true,
                "description": "Worked on coding and stuff"
            }
        ]
    },
    "skillsSegment": {
        "input": "",
        "skillList": [
            {
                "skill": "programming"
            },
            {
                "skill": "coding"
            }
        ]
    },
    "certificationsSegment": {
        "certificationList": [
            {
                "name": "Certified Pro",
                "issuer": "Myself",
                "startDate": "2024-08-31T22:00:00.000Z",
                "endDate": "2024-09-01T22:00:00.000Z",
                "isNotExpiring": true
            }
        ]
    },
    "coursesSegment": {
        "courseList": [
            {
                "name": "How to program",
                "instructor": "YouTube",
                "completionDate": "2024-09-10T16:19:37.257Z",
                "duration": "Hours"
            }
        ]
    }
}

Output PDF
example_cv.pdf

cancel me

Good job, but mind the language. Two commits contain inappropriate words.

What changes do you want?

Please change the names of commits 382761c and 7cfac2c


Enable toggling Sorbet LSP options without needing a full restart

Motivation

Test plan

See included automated tests.

one request for a test
@jez as in one that updates assertDiagnostics etc etc?

👍


Eddystone beacons wrongly identified as iBeacons

Hello,
while using this library on a raspberry pi with 2 sample beacons, I encountered a problem, where after a short while only iBeacons got detected although eddystone beacons were present.
After a short inspection of the code i noticed that in the parser.js the check for iBeacons comes before the check for eddystone beacons. I also noticed that the check for those two kinds of beacons is different. While an eddystone beacon is detected by a unique ServiceUUID, the detection of iBeacons relies on manufacturer data.
After some checks, I realised, that some of the beacons matching the criteria for eddystone (ServiceUUID == "feaa") were classified as iBeacons, because those criteria was also matched.
By rearranging the order of matching (first eddystone then iBeacon), I think I solved the problem. Because my raspberry pi now detects iBeacons and eddystone beacons in roughly the same proprtion.

Thank you so much.


Crash test mode

Context
Some simulations are pretty heavy, requiring a lot of nodes and time while, unfortunately, some problems only manifest in such very large scenarios.
In order to improve the experience when testing new circuits, features or debugging a failure we introduce a new "Crash test" mode.
The new mode runs a simplified version of the circuit:

Cells feature a single section
Connections (between 2 cells) feature a single synapse (to the only section)

Therefore, while results are meaningless, the vast majority of Neurodamus internal machinery runs very quickly, allowing debug sessions to be orders of magnitude faster.
Scope

Added new CLI option --crash-test
Defined a new full-python cell featuring a single section, with all required attributes
Implement a new create_single() function in Connection

Testing
Ran sonataconf-sscx-O1 blueconfigs test (1000 cells) in a single rank:
real	5m37.478s
user	3m44.838s
sys	0m3.816s

including 80 seconds for simulating 100 ms
Review

[ ] PR description is complete
[ ] Coding style (imports, function length, New functions, classes or files) are good
[ ] Unit/Scientific test added
[ ] Updated Readme, in-code, developer documentation

LGTM. Thanks for implementing this new feature.


Documenting nested models

This might not be a bug but the lack of knowledge. But this is my last resort to find out how to do this.
Let's say I have 2 models:
type Item struct {
    ID int `readonly:"true"`
    Name string
    Tags []Tag
}

type Tag struct {
   ID int
   Name string
}

Problem:

POST /tags will expose "id" when we only want to expose "name". But we can't set "id" to readonly because we need to specify tag.ID in Items.Tags.
POST /items should be enough to specify the ID for tags (since they are created above), but since we want to be able to create a tag we need to have the tag.Name as NOT readonly.

So POST to /items looks like this for the generated swagger doc (via UI).
{
  "name": "string"
  "tags": [
    {
      "id": 0,
      "name": "string" // <-- Would like this to be hidden, since it was already set by POST /tags
    }
  ]
}

And for posting to /tags:
{
  "id": 0, // <-- Would like this to be hidden, since it's generated server side.
  "name": "string"
}

Is there a way to handle this scenario?

type Item struct {
    ID int `readonly:"true"`
    Name string
    TagIDs []int
}

type Tag struct {
   ID int `swaggerignore:"true"`
   Name string
}

type Item struct {
    ID int `readonly:"true"`
    Name string
    TagIDs []int
}

type Tag struct {
   ID int `swaggerignore:"true"`
   Name string
}

That seems to work with one change and that would be to set Tag.ID to readonly instead of swaggerignore. However, I'm using gorm as an ORM for my project, hence I'm using relational mapping which makes me want to use []Tag instead of an integer array. I guess I'm out of luck in such case.

Models are shared, behave the same with different APIs.
You have to define different models for your case.

Models are shared, behave the same with different APIs.
You have to define different models for your case.

You mean define models just for documentation?

Models are shared, behave the same with different APIs.
You have to define different models for your case.

You mean define models just for documentation?

Yes, I usually do that myself.

Ok, I guess that's the only way forward. Thanks.


ADD NUMBER TO QUESTIONS TO FIND THEM EASILY WHEN TALKING TO OTHERS

1.1
2.1
3.1
4.1
5.1

Would it be in the format:
<section-number>.<question-number>

Or:
<section-number>.<sub-section-number>.<question-number>

That last version would be the easiest, I think.
And also in the "Appendix"-page I assume?


Tooltip: Doesn't display properly on popout windows

Describe the bug (actual behavior)
"If I pop out my widget into another window, then minimize the main window (important), the iTwin-React tooltip does not disappear when I move the mouse away from the item that initiates its appearance."
Expected behavior
"The Tooltip should disappear when you move the mouse away from the Button"
Reproduction
Link to a minimal repro: https://codesandbox.io/s/cranky-darkness-4z3l1q?file=/src/App.tsx
Steps to reproduce

Open popup window.
Hover over button. Notice that the tooltip appears and disappears.
Minimize original window.
Hover over button again. Now the tooltip will only appear and never disappears.

Additional information
See additional info: https://github.com/iTwin/appui/issues/279
Might have to do with tippy? Maybe related to https://stackoverflow.com/questions/52381990/javascript-tippy-js-not-working-for-dynamic-content?

Duplicating portions of latest comments from https://github.com/iTwin/appui/issues/279 for context:

So, in essence, I'm not sure we can control this behavior as it appear to be a browsesr resouce saving mechanism.

We are also starting work on our next major release very soon, and plan to replace Tippy with floating-ui, so this issue might just go away?
In either case, it seems like a very uncommon and minor issue, plus the proper behavior is returned when the window is restored. So I would say it doesn't need to be prioritized.

For what it's worth, I agree, I don't think it will bother users very much. I mostly just wanted to make the teams aware of it. I'll wait for the upgrade and see if the issue is still present.

Here's what I think we should do: migrate from tippy.js to floating-ui (next major release) and try to reproduce this issue again with the new Tooltip. If it's reproducible, then investigate further. If not, then close, assuming it's fixed.

fixed in v3 via #1311. https://codesandbox.io/s/nervous-platform-vgcf4j?file=/src/App.tsx


How to download

I dont understand how to download .json files.

wdym? Just click code at the top and download as zip

oh ty

Emergent Garden has a video for install instructions if you need some help.
Install tutorial link
Please try to close the issue if it's fixed. ❤


Development

Functionality for second assignment.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

296 of 332 (89.16%)  changed or added relevant lines in 13 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-4.5%) to 91.435%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

repository/IngSoft2-Model/Ship.class.st
73
76
96.05%

repository/IngSoft2-Model/Board.class.st
68
101
67.33%

Files with Coverage Reduction
New Missed Lines
%

repository/IngSoft2-Model/Board.class.st
1
74.05%

Totals

Change from base Build<PHONE_NUMBER>:
-4.5%

Covered Lines:
395

Relevant Lines:
432

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

296 of 332 (89.16%)  changed or added relevant lines in 13 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-4.5%) to 91.435%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

repository/IngSoft2-Model/Ship.class.st
73
76
96.05%

repository/IngSoft2-Model/Board.class.st
68
101
67.33%

Files with Coverage Reduction
New Missed Lines
%

repository/IngSoft2-Model/Board.class.st
1
74.05%

Totals

Change from base Build<PHONE_NUMBER>:
-4.5%

Covered Lines:
395

Relevant Lines:
432

💛  - Coveralls


Disable signature transformation for NTSTATUS

Fixes: #2656

I don't think there are enough STATUS_PENDING returning functions to warrant degrading the developer experience here. Is there something preventing us from specializing the return transformation for NTSTATUS?

I think the issue is that there are many many more success codes when it comes to NTSTATUS compared with say HRESULT.
The alternative would be to return Result<NTSTATUS, Error> but that would be very different to the way we handle error transformations today and just seems like it would cause more confusion than anything else.

The status quo relies on CanReturnMultipleSuccessValues being accurately applied (or not) to all NTSTATUS returning functions. Otherwise the experience is not just degraded, it's unusable for certain scenarios. But I think we simply don't have the people to ensure that the metadata 100% accurately applies the attribute. Maybe that'll come in time but we're not there yet.
So, all else aside, I think something has to change.

Agree we'd just be applying CanReturnMultipleSuccessValues to functions as we find them, but in the absence of more data demonstrating a large swath of functions that will be better off, we may want to consider it over regressing the experience across the board. I recognize, though, getting that data is probably not possible in the short term and concede as a result.

I guess there are alternatives but they'll require more work to get started on. So maybe the opposite attribute for NTSTATUS would be more useful (e.g. CanReturnSingleSuccessValues)? Have the least destructive behaviour by default and only apply Result ergonomics for functions that have been reviewed as only returning one success value.

I just don't think any of these attribute-based approaches are scalable.

Unrelated, the build is failing on the diff check for libs. That's probably because GitHub updated their MSVC toolchain and now the libs have a different version stamp again. I really need to get that version stamp wiped out.

Unrelated, the build is failing on the diff check for libs. That's probably because GitHub updated their MSVC toolchain and now the libs have a different version stamp again. I really need to get that version stamp wiped out.

Yup, will pick this up (removing the compiler ID per last discussion).

Manually updated the libs to unblock this PR.


Test PR on Jenkins do not merge

Looks much better, it also got https://github.com/apache/camel-quarkus/pull/47 run successfully so I'd go for disable travis for the moment


More accurate physics for motors

The SimMotor currently uses a very simplistic model of a maximum output force, and an input signal. This works fine for demos and low-fidelity simulations, but we could probably do better.
This issue involves investigating and potentially buidling a new Motor class that allows you to set maximum free speed, stall torque etc and take into account the mass of the robot, diameter of the wheels etc, to calculate a more accurate force value for each wheel.
We might want to make this a plugin to the SimRobotDrivetrain so that we can swap physics models if necessary. Additionally, this might open the doors to us simulating current flow through the motor, for more advanced simulations.

Superseded by #90


improve node version compatibility

file https://github.com/aheckmann/mpath/blob/634a0fa0f97bf1d00791647e3094273ba360a9ed/lib/stringToParts.js wrote with let & const that supported from node >= 6
see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/let#browser_compatibility
but in package.json only required node >= 4
https://github.com/aheckmann/mpath/blob/634a0fa0f97bf1d00791647e3094273ba360a9ed/package.json#L11

in the perf-optimization PR I changed the whole code to strict mode and it works in node 4 without a flaw.


Add HttpRoutes and HttpApp constructors for middleware

We're adding convenience constructors for middleware shapes for various Http types on an ad hoc bases in #2402 and #2429.  It would be nice to have a more general solution.  There is discussion on #2429.
A couple options that have been floated are a trait for the middleware companions or syntax based on the neglected Middleware type.
This is a resumption of the post-approval discussion on #2429.

I can't decipher my own rambling here.


@Auth can't remove read access for specific fields of a model

Which Category is your question related to?
AppSync
Amplify CLI Version
4.21.1
What AWS Services are you utilizing?
Cognito, Appsync, Pinpoint, Lambda
I have a User model in my graphql schema and am having auth issues. I want some fields (like username and name) to be accessible to other users but other fields (like email) not to be. I initially added an @auth header to the model for { allow: private, operations: [read]} thinking I could add individual lines for the fields I want to hide like { allow: private, operations: []}, but unfortunately that doesn't work because you have to specify an operation and I'm trying to remove all operations. The other option is to leave out the auth header from the model and define { allow: private, operations: [read]} for the lines I want to be accessible to other users, but if you leave the header from the model then you can't retrieve user properties through other models and you can't list users anymore. For example If I want to get the username of a Post's author, I can no longer query the user through the user connection of the post. Essentially either all of the fields have to be accessible or none of them.
Are there any work arounds or functionality I'm missing? Or can this be taken as a feature request? I think this is really crucial to security for apps built on Amplify. Thanks!

Hi @julien-tamade
you can also post this question on amplify discord community for quick answer :  https://discord.com/invite/jWVbPfC and Other users working on same usecase will reply on that.

Hi @julien-tamade ,
As an example below you could create an explicit rule to restrict access to a certain group by defining a field level auth. You can find documentation for you reference here
type Todo
  @model
{
  id: ID! @auth(rules: [{ allow: private, operations: [read]}])
  owner: String @auth(rules: [{ allow: private, operations: [read]}])
  email: String! @auth(rules: [{ allow: groups, groups: ["ForbiddenGroup"], operations: [] }])
}

The issue I'm experiencing is that I need to be able to allow general access to the model for private but remove access for everybody to specific items. If I don't declare general access then I can't query this model through another model. For instance getting a user's username when querying a post. So i add [{ allow: private, operations: [read]}] to the whole model, but then I'm trying to figure out to remove that read access for private users because [] doesn't seem to cancel out the [read] access outlined before.
Does that make sense? I haven't tried in 2 weeks though so not sure if you have made upgrades.
Here's what I'm hoping would work, but private users still can access email and subscriptionId.
type User
  @model(subscriptions: null)
  @auth(
    rules: [
      { allow: owner, ownerField: "id", identityClaim: "username", operations: [read, create, update] }
      { allow: public, provider: apiKey, operations: [read, create, update, delete] }
      { allow: public, provider: iam, operations: [read, create, update, delete] }
      { allow: private, operations: [read]}
    ]
  ) {
id: ID!
username: String
firstName: String
email: String @auth(rules: [{ allow: private, operations: [] }])
subscriptionID: String @auth(rules: [{ allow: private, operations: [] }])

Another option could be to remove the global auth header and just add access to the desired lines
type User
  @model(subscriptions: null)
  @auth(
    rules: [
      { allow: owner, ownerField: "id", identityClaim: "username", operations: [read, create, update] }
      { allow: public, provider: apiKey, operations: [read, create, update, delete] }
      { allow: public, provider: iam, operations: [read, create, update, delete] }
    ]
  ) {
id: ID! @auth(rules: [{ allow: private, operations: [read] }])
username: String @auth(rules: [{ allow: private, operations: [read] }])
firstName: String @auth(rules: [{ allow: private, operations: [read] }])
email: String
subscriptionID: String

But then I can't query through other models, Like a user through a post to get their username
Thanks!


ActiveDirectory Find Users

Question
A clear and concise description of your question.
I have create a page where we can search AciveDirecotry for users, Get-ADuser $username -Properties.
The issue is, that it only find/shows some accounts.
When searching for accounts i Powershell directly, the accounts exists, but can de found/shown on the Pode site.
I have tried to compare the users, which could be found, with no indication on what should be different.
Any idea what could be the issue.

UPDATE
The users failing in org. question can be found by the following code:
Add-PodeRoute -Method Get -Path '/api/users/:userId' -ScriptBlock {
Write-PodeJsonResponse -Value @{
#Name = $WebEvent.Parameters['userId']
UserId = (Get-ADUser $WebEvent.Parameters['userId'] -Properties *)
}
} -PassThru |
Add-PodeOAResponse -StatusCode 200 -Description 'A user object' -ContentSchemas @{
'application/json' = (New-PodeOAObjectProperty -Properties @(
#(New-PodeOAStringProperty -Name 'Name'),
(New-PodeOAIntProperty -Name 'UserId')
))
}

Hi @RonniHartung,
From https://badgerati.github.io/Pode/Getting-Started/KnownIssues/#activedirectory-modul, the usual culprit for AD module issues is the module isn't being imported first; one to check and ensure 😃
If it is being imported, could it be:

An access issue with the user that running your Pode site?
What happens if you use the full LDAP path for the users that can't be found?


Noisy DEBUG messages

Hello,
When org-journal-enable-agenda-integration is turned on, org-journal logs a number of DEBUG: ... messages into *Messages* buffer, but this is noisy. I suppose you added it for development. Have you ever considered suppressing these messages in normal use?

Thank you for the bug report!
I'm sorry, I forgot a debug message in the last release! I just fixed it in 1.14.3, which should hit melpa and melpa stable in a few minutes. Again, sorry.

@bastibe This issue looks like that can be closed.


WSL cannot read cmd arguments on Windows 11

Version
Microsoft Windows [Version 10.0.22000.708]
WSL Version

[X] WSL 1

Kernel Version
Linux version 4.4.0-22000-Microsoft<EMAIL_ADDRESS>(gcc version 5.4.0 (GCC) ) #653-Microsoft Wed Apr 27 16:06:00 PST 2022
Distro Version
Ubuntu 20.04
Other Software
No response
Repro Steps

Update Windows 11 to version 22000.708
Try to upgrade WSL1 to WSL2
You will get a failure

https://user-images.githubusercontent.com/86933883/172475331-51ce7710-547b-4959-99c2-f22268222ff6.mp4
Expected Behavior
I expected to see wsl1 upgrade to wsl2
Actual Behavior
wsl does not accept command line arguments for updating.
Diagnostic Logs
Before i try re-enable VirtualMachinePlatform via dism
PS C:\Users\x>  dism.exe /online /enable-feature /featurename:Microsoft-Windows-Subsystem-Linux /all /norestart

Deployment Image Servicing and Management tool
Version: 10.0.22000.653

Image Version: 10.0.22000.708

Enabling feature(s)
[==========================100.0%==========================]
The operation completed successfully.
PS C:\Users\x> dism.exe /online /enable-feature /featurename:VirtualMachinePlatform /all /norestart

Deployment Image Servicing and Management tool
Version: 10.0.22000.653

Image Version: 10.0.22000.708

Enabling feature(s)
[==========================100.0%==========================]
The operation completed successfully.

What SKU are you running?

Can you make sure you have this patch installed?
https://support.microsoft.com/en-us/topic/may-24-2022-kb5014021-os-build-20348-740-preview-2b180bd4-dceb-4c49-b8cf-402b342ebc84

@keklick1337 based on your video clip, you're trying to convert Ubuntu-20.04 from WSL1 to WSL2 while being in use. You need to shutdown WSL first, just use wsl --shutdown, then convert it to WSL2.

For WSL2 you need to enable the virtual machine platform optional component, and ensure that your machine has virtualization configured in the bios.

Make sure you have added the Windows Subsystem for Linux Windows component, using old skool Control Panel/Programs and Features/Turn Windows features on or off. Not sure if this applies even on the most current versions of Windows, but I just factory reset a machine, and the behaviour of the wsl command out of the box, before installing that component, was to act as if it could not see any command line parameters I was giving it.


Request access to closed API

EPIC for closing APIs so that API is published in catalog, but it only shows limited information and has a e.g. button for the developer / user to request access to the API or a method for API owner to invite users to the API.
This task requires feedback from the team on possible options to allow only certain users to see/use the API with the smallest possible implementation.
Field level visibility of the API in catalog.
Group of allowed users
Method for getting the access: email invititation from owner, administraor can manually add users (by email address) or request button.

Feedback from @philippeluickx Does this require a new user role: API user that is tied to  particular API(s) to which user has been granted access?

We might want to create a seperate epic with user roles functionality. This requires the definition and management of roles and we have to implement permissions as well.

1.0  Only allow API owner to  limit catalogue visibility for users on Apinf side.
Estimation: 5
(Possible before MIndTrek)
2.0 Allove API owner to control developers who can make calls on proxy level:
Estimation: ?
Study story needed.


Video size displayed is wrong because of VideoTrack not found

I have a dvd vob file and it's played with wrong ratio. I've checked you have VideoLockCallback method in VlcPlayer.Events.cs to determine the size. With the line var videoMediaTracks = tracks.TakeWhile(t => t is VideoTrack).Select(t => t as VideoTrack).ToList();
But in my video I have audio track first, and video track as the second element. So, videoMediaTracks is null always, so video is not resized.
I've changed it to var videoMediaTracks = tracks.OfType<VideoTrack>().ToList(); and it works fine now.

let me create pull request for it

Thanks for your contribution.


Default quit message version is static (version that was running when the user was created)

My secondary ZNC which is trying to run the latest tagged release has two users. One has manually set quit message and the other has default.
I think I updated to 1.6.3-rc1 on the next day from it being tagged (I think it was tagged during Europe/Helsinki night).
Today I updated ZNC and noticed that the user with default quit message quit with version 1.6.2 which is what the ZNC was running when that user was created. I think it should have shown the same as the user with %version% as quit message.
2016-02-17 20:14:54+0200 <-- Nikashi (Nikashi@unaffiliated/nikashi) has quit (Quit: ZNC 1.6.2 - http://znc.in)
2016-02-17 20:14:54+0200 <-- Ciblia (Mikaela@unaffiliated/mikaela) has quit (Quit: ZNC 1.6.3-git-15-b49a92b  https://mikaela.info/)

ZNC 1.6.3-git-15-b49a92b - http://znc.in
IPv6: yes, SSL: yes, DNS: threads, charset: yes
Debian GNU/Linux testing (stretch)

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/30952962-default-quit-message-version-is-static-version-that-was-running-when-the-user-was-created?utm_campaign=plugin&utm_content=tracker%2F1759&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F1759&utm_medium=issues&utm_source=github).

(Obvious) Workaround:
2016-02-18 09:49:25+0200 < Mikaela> get quitmsg Nikashi
2016-02-18 09:49:25+0200 < *controlpanel> QuitMsg = ZNC 1.6.2 - http://znc.in
2016-02-18 09:49:42+0200 < Mikaela> set quitmsg Nikashi ZNC %version% - http://znc.in
2016-02-18 09:49:42+0200 < *controlpanel> QuitMsg = ZNC %version% - http://znc.in


[question] how to initialize db = getFirestore(...)?

How do you handle loading data server side and/versus handling with Firestore db in the browser?
Server side... I guess I will have to initialize the app within admin.ts and import it in  +page.ts. But as long as I am in the "browser" how to I handle queries locally?
Here is my "broken" code, btw: https://github.com/CaptainCodeman/sveltekit-example/compare/master...fredguth:sveltekit-example:master

I have some articles that focus on the client-side querying & lazy-loading on my blog. Just like with auth, I usually wrap things in a store as the subscription mechanism aligns well with the Firestore subscription:
https://www.captaincodeman.com/lazy-loading-and-querying-firestore-with-sveltekit

I understand from your post that you use Firebase only in the client, not in the server.  Nothin server side, right?
why not generic collection and document stores? See example https://github.com/fredguth/sveltekit-example/blob/master/src/lib/stores/fireStores.ts

I understand from your post that you use Firebase only in the client, not in the server. Nothin server side, right?

I use either or both, but generally it's easier to explain something specific if you don't try to include everything. Examples are meant to demonstrate some technique, not be a fully comprehensive blueprint.

why not generic collection and document stores? See example https://github.com/fredguth/sveltekit-example/blob/master/src/lib/stores/fireStores.ts

Use whatever approach makes sense for you. I have also used wrapper / decorator stores to add things like pagination, but so much depends on the app your building and the structure of the data. Unless you're trying to create a generic svelte+firebase library, it's usually diminishing returns and added complexity trying to make everything re-usable vs just implementing what you need for each collection as you need it. If there is duplication, extract those pieces out if they are repeated enough times and big enough to warrant it.
Readability and maintainability is an important feature to keep in mind.


How to add dependencies with plugins?

Hi! Thanks a lot for this tool!
I am trying to add https://github.com/dry-python/returns support (because we have quite a lot of complex plugins).
Code:
    Project(
        location="https://github.com/dry-python/returns.git",
        mypy_cmd="{mypy} returns",
        pip_cmd="{pip} install .",
        expected_success=True,
    ),

returns is based on poetry, not pip. But pip is able to install everything from pyproject.toml locally.
The next problem is that our plugin is not found:
old mypy
> /tmp/mypy_primer/old_mypy/venv/bin/mypy returns --python-executable=/tmp/mypy_primer/projects/_returns_venv/bin/python --no-incremental --cache-dir=/dev/null
UNEXPECTED FAILURE
        setup.cfg:153: error: Error importing plugin 'returns.contrib.mypy.returns_plugin': No module named 'returns'
        Found 1 error in 1 file (errors prevented further checking)
----------

Looks like the problem comes from a fact that mypy and returns use different venvs. But, I am not sure how to fix this.

Yeah, mypy_primer doesn't yet support plugins — you'll notice mention of dry-python/returns at https://github.com/hauntsaninja/mypy_primer/blob/7c00c135fd2cacd2e08bd2fc81500d22b0fe3923/mypy_primer.py#L1249 :-) — and I'm not sure what the best way to support them is either.
The difficulty, as you mentioned, is that mypy and the projects are installed in different venvs. This allows us to reuse the same mypy venv across projects / easily change the mypy version independently.
I think one (possibly overkill) way of doing this is if we can get the project specific venv to "inherit" the mypy installation from the mypy venv. I bet there's some fun .pth hack way to do this.

you'll notice mention of dry-python/returns at

@hauntsaninja yeah, this was a pleasant surprise! 🙂

I think one (possibly overkill) way of doing this is if we can get the project specific venv to "inherit" the mypy installation from the mypy venv. I bet there's some fun .pth hack way to do this.

I think that it is also possible to install mypy on top of the returns (for example) venv. Like returns_env/bin/pip install /tmp/mypy_venv/path/to/mypy/setup.py

Yes, it works after a quick experiment:
» /tmp/mypy_primer/projects/_returns_venv/bin/pip install /tmp/mypy_primer/old_mypy/mypy_master
...
Successfully installed mypy-0.820+dev.a503132749eb179eec8d1e97f6b8e372daab6ce7

» /tmp/mypy_primer/projects/_returns_venv/bin/mypy returns/returns --ignore-missing-imports
Success: no issues found in 112 source files

Yeah, that's possible, it's just a little tricky to get right. E.g., you'd need to prevent "old mypy" and "new mypy" from running in parallel on the same project. There are some other gotchas too (bisection, using mypy pypi wheels).

Maybe we can add something like
[mypy_primer]
build_command = 'make && make install'
# possibly `remove_command` as well

to plugins' mypy.ini / setup.cfg / pyproject.toml?
This will just contain the command we need to install this in mypy_primer's env.
So, this will be an opt-in feature for plugin authors.
For example, I am very worried about my projects are not tested (because the most complex ones break the most). And I would totally opt-in.

Would specifying the plugin by path instead of module name work like what edgedb does right now? edgedb/edgedb#2627 seems to suggest that doing that makes installing the plugin unnecessary.

I just wanted to mention on this issue that I think I did figure out an approach to make mypy_primer support plugin dependencies in #121


.github: wip

Description of the change:
Motivation for the change:
Reviewer Checklist

[ ] Implementation matches the proposed design, or proposal is updated to match implementation
[ ] Sufficient unit test coverage
[ ] Sufficient end-to-end test coverage
[ ] Docs updated or added to /docs
[ ] Commit messages sensible and descriptive

Closing - this was just for testing purposes.


Optimize arithmetic in VM.

Nice performance improvement. Binary size unchanged.

Benchmark
Perf (sec)

3d-cube.js
1.138 -> 1.038 : +8.777%

3d-raytrace.js
1.252 -> 1.171 : +6.487%

access-binary-trees.js
0.638 -> 0.621 : +2.690%

access-fannkuch.js
2.842 -> 2.741 : +3.544%

access-nbody.js
1.313 -> 1.165 : +11.278%

bitops-3bit-bits-in-byte.js
0.624 -> 0.586 : +5.942%

bitops-bits-in-byte.js
0.919 -> 0.876 : +4.696%

bitops-bitwise-and.js
1.501 -> 1.460 : +2.718%

bitops-nsieve-bits.js
1.953 -> 1.954 : -0.010%

controlflow-recursive.js
0.458 -> 0.435 : +4.920%

crypto-aes.js
1.228 -> 1.223 : +0.368%

crypto-md5.js
0.786 -> 0.774 : +1.586%

crypto-sha1.js
0.751 -> 0.730 : +2.854%

date-format-tofte.js
0.919 -> 0.913 : +0.661%

date-format-xparb.js
0.507 -> 0.498 : +1.844%

math-cordic.js
1.604 -> 1.553 : +3.170%

math-partial-sums.js
0.877 -> 0.814 : +7.224%

math-spectral-norm.js
0.669 -> 0.614 : +8.206%

string-base64.js
2.163 -> 2.112 : +2.346%

string-fasta.js
2.021 -> 1.993 : +1.384%

Geometric mean:
+4.082%

LGTM from my side although my knowledge of that part of the code is still fairly limited :)

Nice improvement, LGTM

LGTM


parser: add TableOptimizerHint TIDB_HASHAGG and TIDB_STREAMAGG

What problem does this PR solve?
Add new optimizer hints for TiDB PR: https://github.com/pingcap/tidb/pull/11364
What is changed and how it works?
Change parser and a related Restore interface.
Check List
Tests

Unit test

PTAL @crazycs520 @kennytm

PTAL. @lamxTyler


BaseStringHelper::truncateHtml adds suffix regardless of the string length

Hi,
I am using StringHelper::truncateWords (Yii2 v2.0.6) and found out that $suffix always gets added to returned string if $asHtml is set to true, regardless of fact that the string was not actually truncated. This differs from the behaviour of the same function when $asHtml is set to false.
I would expect it to behave consistently and return the same string if it was not truncated (either with suffix, or without).
$str='lorem ipsum';

StringHelper::truncateWords($str, 3, '...', true);
// returns "lorem ipsum..."

StringHelper::truncateWords($str, 3, '...', false)
// returns "lorem ipsum"

For now I have overridden BaseStringHelper::truncateHtml(...)
return $generator->generateFromTokens($truncated) . $suffix;

to
return $generator->generateFromTokens($truncated) . ($totalCount >= $count ? $suffix : '');

which seems to do the job.
Maybe should be considered in upstream.

Hi @mattheobjornson
I've created PR with fix provided by you. Thanks!


[DOC-45] Feat/add partner docs

Looks great content wise, but I caught a typo, have you run this through a spell and grammar checker? 👀

Looks great content-wise, but I caught a typo, have you run this through a spell and grammar checker? 👀

Good suggestion. I have to get used to run Grammarly on all the docs.

I've fixed several typos
Also changed some stuff, grammar-related, based on Grammarly suggestions

Please check it out again.

I made my edits here. https://github.com/yearn/yearn-devdocs/pull/19

Superseded by #19

@lehnberg I cloed the PR by mistake. Would you review this please?


GpxTrackSegment is missing a default constructor

GpxTrackSegment is missing a default constructor in order to use the WithFoo extensions.

Fixed in 0.2.1, which is available on NuGet now.


test: Update PHPUnit version to run tests in PHP 8.2

Can't merge this since fix https://github.com/phingofficial/phing/pull/1685 has not been released  yet.


data-turbo-method="delete" does not maintain data-target nor dispatch turbo:clik

the dynamic change of a link in a form does non hold the target attribute:
dummyl#captureClick" class="btn btn-danger px-3" href="/article/1">delete
become:
<form data-turbo=\"true\" action=\"http://<IP_ADDRESS>:3000/article/1\" method=\"delete\" data-turbo-frame=\"modal\" aria-busy=\"true\" hidden=\"\"></form>
moreover the  event turbo:clik is not dispatched by the form, i think this is by design but here in origin was intended to be a link so to me click should be dispatched.
Thanks

Hi @wljgo, further enhancements to data-turbo-method are not planned. I suggest converting this to a native browser form element with a submit button.


New Package: ente-io.ente.Auth version 2.0.50+250

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[x] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Resolves #147679
Resolves #147677

Microsoft Reviewers: Open in CodeFlow

@denelon - Another package failing dependencies due to VCRedist not having a user scope
I could have sworn that this was never an issue for dependencies in the past

Automatic Validation ended with:

2024-04-08 15:33:52.735 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\RepositorySource.cpp(757)\WindowsPackageManager.dll!00007FF8CC96C11F: (caller: 00007FF8CC6F8D6F) LogHr(1) tid(10f4) 8A15005E     Msg:[D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\Rest\Schema\HttpClientHelper.cpp(30)\WindowsPackageManager.dll!00007FF8CC872B08: (caller: 00007FF8CC91BC20) Exception(1) tid(10f4) 8A15005E ]
2024-04-08 15:33:52.735 [REPO] Failed to open available source: msstore
2024-04-08 15:34:00.530 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\RepositorySource.cpp(757)\WindowsPackageManager.dll!00007FF8AC24C11F: (caller: 00007FF8ABFD8D6F) LogHr(1) tid(1794) 8A15005E     Msg:[D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\Rest\Schema\HttpClientHelper.cpp(30)\WindowsPackageManager.dll!00007FF8AC152B08: (caller: 00007FF8AC1FBC20) Exception(1) tid(1794) 8A15005E ]
2024-04-08 15:34:00.530 [REPO] Failed to open available source: msstore
2024-04-08 15:34:01.749 [CLI ] No suitable installer found for manifest Microsoft.VCRedist.2015+.x64 with version 14.38.33135.0
2024-04-08 15:33:29.126 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\RepositorySource.cpp(757)\WindowsPackageManager.dll!6836535B: (caller: 68261A80) LogHr(1) tid(1bc8) 8A15005E     Msg:[D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\Rest\Schema\HttpClientHelper.cpp(30)\WindowsPackageManager.dll!683AB5DD: (caller: 684412BF) Exception(1) tid(1bc8) 8A15005E ]
2024-04-08 15:33:29.126 [REPO] Failed to open available source: msstore
2024-04-08 15:33:36.881 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\RepositorySource.cpp(757)\WindowsPackageManager.dll!545B535B: (caller: 544B1A80) LogHr(1) tid(b94) 8A15005E     Msg:[D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\Rest\Schema\HttpClientHelper.cpp(30)\WindowsPackageManager.dll!545FB5DD: (caller: 546912BF) Exception(1) tid(b94) 8A15005E ]
2024-04-08 15:33:36.881 [REPO] Failed to open available source: msstore
2024-04-08 15:33:29.654 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\RepositorySource.cpp(757)\WindowsPackageManager.dll!6A0C535B: (caller: 69FC1A80) LogHr(1) tid(1c14) 8A15005E     Msg:[D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\Rest\Schema\HttpClientHelper.cpp(30)\WindowsPackageManager.dll!6A10B5DD: (caller: 6A1A12BF) Exception(1) tid(1c14) 8A15005E ]
2024-04-08 15:33:29.654 [REPO] Failed to open available source: msstore
2024-04-08 15:33:37.485 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\RepositorySource.cpp(757)\WindowsPackageManager.dll!55C8535B: (caller: 55B81A80) LogHr(1) tid(d98) 8A15005E     Msg:[D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\Rest\Schema\HttpClientHelper.cpp(30)\WindowsPackageManager.dll!55CCB5DD: (caller: 55D612BF) Exception(1) tid(d98) 8A15005E ]
2024-04-08 15:33:37.485 [REPO] Failed to open available source: msstore
2024-04-08 15:33:38.733 [CLI ] No suitable installer found for manifest Microsoft.VCRedist.2015+.x86 with version 14.38.33135.0

(Automated response - build 876.)

Automatic Validation ended with:

2024-04-08 15:33:52.735 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\RepositorySource.cpp(757)\WindowsPackageManager.dll!00007FF8CC96C11F: (caller: 00007FF8CC6F8D6F) LogHr(1) tid(10f4) 8A15005E     Msg:[D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\Rest\Schema\HttpClientHelper.cpp(30)\WindowsPackageManager.dll!00007FF8CC872B08: (caller: 00007FF8CC91BC20) Exception(1) tid(10f4) 8A15005E ]
2024-04-08 15:33:52.735 [REPO] Failed to open available source: msstore
2024-04-08 15:34:00.530 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\RepositorySource.cpp(757)\WindowsPackageManager.dll!00007FF8AC24C11F: (caller: 00007FF8ABFD8D6F) LogHr(1) tid(1794) 8A15005E     Msg:[D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\Rest\Schema\HttpClientHelper.cpp(30)\WindowsPackageManager.dll!00007FF8AC152B08: (caller: 00007FF8AC1FBC20) Exception(1) tid(1794) 8A15005E ]
2024-04-08 15:34:00.530 [REPO] Failed to open available source: msstore
2024-04-08 15:34:01.749 [CLI ] No suitable installer found for manifest Microsoft.VCRedist.2015+.x64 with version 14.38.33135.0
2024-04-08 15:33:29.126 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\RepositorySource.cpp(757)\WindowsPackageManager.dll!6836535B: (caller: 68261A80) LogHr(1) tid(1bc8) 8A15005E     Msg:[D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\Rest\Schema\HttpClientHelper.cpp(30)\WindowsPackageManager.dll!683AB5DD: (caller: 684412BF) Exception(1) tid(1bc8) 8A15005E ]
2024-04-08 15:33:29.126 [REPO] Failed to open available source: msstore
2024-04-08 15:33:36.881 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\RepositorySource.cpp(757)\WindowsPackageManager.dll!545B535B: (caller: 544B1A80) LogHr(1) tid(b94) 8A15005E     Msg:[D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\Rest\Schema\HttpClientHelper.cpp(30)\WindowsPackageManager.dll!545FB5DD: (caller: 546912BF) Exception(1) tid(b94) 8A15005E ]
2024-04-08 15:33:36.881 [REPO] Failed to open available source: msstore
2024-04-08 15:33:29.654 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\RepositorySource.cpp(757)\WindowsPackageManager.dll!6A0C535B: (caller: 69FC1A80) LogHr(1) tid(1c14) 8A15005E     Msg:[D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\Rest\Schema\HttpClientHelper.cpp(30)\WindowsPackageManager.dll!6A10B5DD: (caller: 6A1A12BF) Exception(1) tid(1c14) 8A15005E ]
2024-04-08 15:33:29.654 [REPO] Failed to open available source: msstore
2024-04-08 15:33:37.485 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\RepositorySource.cpp(757)\WindowsPackageManager.dll!55C8535B: (caller: 55B81A80) LogHr(1) tid(d98) 8A15005E     Msg:[D:\a_work\1\s\external\pkg\src\AppInstallerRepositoryCore\Rest\Schema\HttpClientHelper.cpp(30)\WindowsPackageManager.dll!55CCB5DD: (caller: 55D612BF) Exception(1) tid(d98) 8A15005E ]
2024-04-08 15:33:37.485 [REPO] Failed to open available source: msstore
2024-04-08 15:33:38.733 [CLI ] No suitable installer found for manifest Microsoft.VCRedist.2015+.x86 with version 14.38.33135.0

(Automated response - build 876.)

I've suggested we remove the "msstore" source from our validation environment to reduce the log load here. Digging into the dependency issue, it seems like we're trying to get a scope match on the package. I'm not sure if we ended up with a newer version of the dependency in our validation environment or something else. We're still investigating.

I've suggested we remove the "msstore" source from our validation environment to reduce the log load here. Digging into the dependency issue, it seems like we're trying to get a scope match on the package. I'm not sure if we ended up with a newer version of the dependency in our validation environment or something else. We're still investigating.

@denelon - In case it helps, here is an example of a PR where https://github.com/microsoft/winget-pkgs/labels/Validation-Unattended-Failed and https://github.com/microsoft/winget-pkgs/labels/Validation-Installation-Error weren't applied, even though VCRedist is specified as a dependency and Scope: user is present - https://github.com/microsoft/winget-pkgs/pull/110135

@wingetbot run

[Policy] Blocking-Issue


Long ToCs not shown fully

Description
A good example of the bug is in the documentation itself, here. When the table of contents is really long, it is impossible to see all the options until you scroll to the bottom of the page.
Expected behavior
One relatively simple solution I can think of, is to make the ToC scrollable, independent of the main page's scroll.
Screenshots

In the above screenshot, it is not possible to see the text below "Firebase" until you scroll to the bottom of the page.
Desktop (please complete the following information):
Even though I don't think it's applicable here, I'll fill it regardless.

OS: Windows 11 23H2, OS Build 22361.3447
Browser: Edge
Version: 124.0.2478.80 (Official build) (64-bit)

The table of content definitely should NOT be scrollable independently of the main page's content. It would look very bad.

My proposed solution was just the first thing that came to mind. I am open to other solutions.

Hi,
Even if we are a minority, may we disagree on the bold (subjective) statement "(...it) should NOT be scrollable (...) It would look very bad" ?!
Typing "Best Practices" + "Table of Contents" in your preferred search engine may show you how many (different) opinions you can have about the "how to do things properly".
That being said, the purpose of a ToC is to provide a way to be able to "jump" quickly anywhere in your document (online / offline ... or even printed = your fingers doing the same job as your browser 😅).
So, even if a majority disagree, some of us may be allowed to think that making it available only partially (I agree the configuration page of Blowfish website is not that big 😅 ... but it was just an example) is not the best/expected behavior.
The same way, @nunocoracao may not give a damn about it, think it's too much work for a too small audience ... or have other priorities 🤷
In worst case, we'll have to switch to another (rare) theme providing this kind of feature, mainly for docs (like Doks) or learning (ex.: Relearn), but, on a more subjective point of view, most of them are not as nice/polished as Blowfish 😅
On a more constructive note (having filled the issue #1455, marked as closed a little too fast 😇, for the very same reasons), may I propose: (in order of priorities, as I see them)

at least a smartTOCPosition configuration entry, so those producing big documents can benefit from a "full ToC" placed at the beginning/end, the same way Blowfish handles posts series = the behavior / responsiveness / etc. being already implemented, it should be easily adapted 🤔

to keep the ToC "in sync" with the main content (the anchor in the main panel should be visible in the ToC lateral panel)

a smartTOCBehavior configuration entry, allowing collapsible and/or scrollable ... for those who like it that way 😅

May I end by saying that the choice should always be given to users. But time being such a rare commodity, as always, @nunocoracao will have the final say ... thanks for considering this 🙇

Even if we are a minority, may we disagree on the bold (subjective) statement "(...it) should NOT be scrollable (...) It would look very bad" ?!
Typing "Best Practices" + "Table of Contents" in your preferred search engine may show you how many (different) opinions you can have about the "how to do things properly".

I have never seen a scrollable table of contents placed in similar way like the one in Blowfish. I suppose that's for a reason, because it would be an objectively  bad design choice. If you think otherwise then show some examples on other websites and implement it in Blowfish as a proof of concept.

I'm not here to convince people who own the absolute truth to be more open-minded or take a better look at what is available outside.
And, no matter this too often passive-aggressive attitude we bump into, when discussing different opinions, and the way of presenting your own as the only possible way of thinking, since I don't pretend to own it myself but merely consider a constructive approach to (start with) fill(ing) issues and try improving things by doing so, maybe we should let @nunocoracao answer this one by himself ?!
And as I said previously / many times in other issues, if he thinks it's not appropriate to fix/add this feature to this project/theme ... that's more than ok 🤷
Blowfish is already a very good theme and we are all thankful for his time and the fact he is providing it freely to the community (and can also add thanks for you 2 commits last year and the other ones who helped him, no matter how (much) they've contributed) 🙇
For those who may be unhappy with his answer and/or the availability/knowledge/whatev to implement this feature, they'll still have the possibility to search for a better-fitting theme 🤷
Hoping for an answer soon so we can stop wasting time/energy talking about it and more building/producing things 🤷
Best regards to you all. 🙇

I'm not here to convince people who own the absolute truth to be more open-minded or take a better look at what is available outside.
And, no matter this too often passive-aggressive attitude we bump into

There is nothing passive-aggressive about my simple request for a proof of concept when suggesting a change. That is just how open source software development works.
How to contribute to Blowfish: New features & enhancements

@maciejopalinski Do you agree that not being able to see the full ToC is a problem? If not, why not? If yes, then do you have any suggestions that are not, in your words, "an objectively bad design choice"? Since I am not a designer, and I suspect @Usual-Coder is not either, could you please enlighten us about some good design choices?

That looks cool, but it would require to move the ToC to the left. I don't think it necessarily fits the theme. I think the theme was designed mainly with blogs in mind in which you won't really find such long ToCs.

Hi,
@wermos thanks for your additional input. I decided to cut my losses with Hugo/theming/etc. (too much time lost with the tooling instead of producing content) = at least for now, I'll use another solution :shrug:  But, still, I think Blowfish is probably one of the best themes for Hugo :bow: (after all, it's just not fitting our needs / decisions had to be made :shrug:)
To try ending on a constructive note, I'm providing a link to a French website behaving as I expected initially (ToC on the right being scrollable and really kept in sync with content):

https://cours.brosseau.ovh/tp/typescript/api.html
Global navigation on the left (as described by MDN for nav element)
Document "navigation" -aka "ToC"- on the right (no matter how it's translated in HTML/CSS, a generic div as it's done right now, an aside element as described from the very same MDN, or any adequate element / styling to achieve this goal)  ... as expected by anyone writing/producing docs/webpages/whatev with AsciiDoc when :toc: right is set
additionally, the linked website / ToC has a very useful (discreetly/nicely integrated ?!) indicator (the vertical bar in front of the displayed/actual entry) making it easy to know how far you are in the document ... easing things from the reader point of view :thinking:

I also still understand that @nunocoracao may not have time to implement that (making his smartTOC scrollable, at least as an option, and kept in sync with the main content). As I said, it's his prerogative to determine if it's bug or enhancement ... and how to handle it :shrug: I hope he will fix/add this feature :crossed_fingers:
Best regards to you all

https://cours.brosseau.ovh/tp/typescript/api.html

This one convinced me, I didn't even realize it was scrollable at first, but then I checked and it really is. I apologize for misunderstanding the idea that you suggested. It would definitely be a nice fix/addition to have this indicator and make it scrollable (preferably auto-sync with content; without the scrollbar visible of course).

@nunocoracao ... if you consider implementing this after all, from the UX designer point of view, to handle sublevels (not presented in the linked URL), may I also suggest replacing (or providing a SmartTOC*** option) your underlined style (some may say we could call it a horizontal bar :smirk:) with the vertical bar shown in the linked website.
By doing so, you could "expand" it in front of the subentries and handle sublevels, adding a similar (expandable, if needed / children) but "sub-leveled" bar, using a darker color from your palette depending on the sublevel (the most vivid being the deepest one ... aka "keep the focus where it's needed + make the rest useful but discrete" :smirk:).
I'm about to go out right now, if my explanations are not good enough, I may search for an example + add a comment with a link later. Just ask.

@Usual-Coder thanks for the example. I can use this to deconstruct how Vitepress does it and perhaps open a PR in Blowfish to implement the same feature. Vitepress's default ToC works exactly how I envisioned the feature to be.

I think Blowfish is probably one of the best themes for Hugo

I wholeheartedly agree. @nunocoracao has done some amazing work here.
The

"sub-leveled" bar

idea sounds pretty interesting to me. I'll play around with it and see if I can implement it along with the Vitepress-style scrollable ToC.
Do you also have an idea for what darker color from the palette could be used here? More specifically, could you help me figure out which color to use for the default blowfish color scheme and the other available skins?

...if my explanations are not good enough, I may search for an example + add a comment with a link later. Just ask.

It would be nice if you could find a site that uses the "sub-leveled" bar so that I have a visual example to go by.

@wermos: I didn't do much ... that design (scrollable/synced right ToC) is quite common 🤷
To stay around Hugo:
https://docs.gethyas.com/install/auto/#starter-templates

The guys from Hyas are the ones behind the Doks theme for Hugo
Some would also notice the very nice idea 😇 of the "tabbed" code block (allowing to switch between variants, also used on Hugo website ... btw, even if it's not that long/useful there, the right ToC is also scrollable ... if it's good for Hugo's daddy maybe it could be good for us 🤷😅) = that could also be a killing feature for Blowfish 🤐😇😅

Another one I came across while dealing with what is actually requiring my attention:
https://elysiajs.com/patterns/websocket.html (behaving the exact same way as the French blog linked previously ... but I prefer the homogeneity of the linked Blog, nav/toc sharing the same design)

btw, (as shown by the first entry) an option to handle the text overflow of entries too large with an ellipsis could also be an idea to explore 😇 ... but (unlike them) with a mouseover animating the text in place instead of popping a basic tooltip ... as always, with design considerations, the devil / wow effect is in the details / how seamlessly everything is integrated (and/or configurable one would argue)

Right now, I don't have time to search specifically for sublevel design examples, but if I stumble upon a website having such a design, I'll try to remember to provide you with a link ✔
Btw, I can't help you with colors/palette considerations. First, because I think it's @nunocoracao prerogative to answer that one. And second, it would require me to read Blowfish code ... which is the very reason I decided to steer away from Hugo/theming/etc., having already lost too much time ("(...) with the tooling instead of producing content (...)", as I put it earlier 😇).
No matter what, wanting to contribute to Blowfish code base is a very constructive way to handle this issue ... big respect for that 🙇 ... but may I suggest starting with a quickwin™ until @nunocoracao comes in the loop ... and gives his point of view on the subject?! 🤐😇
In the meantime, I would stick with a basic/simpler version like the one available here (getting the inspiration from the left panel/nav): https://bun.sh/docs/bundler/loaders#json

Take a good look at the indicatorS ... couldn't it be a good compromise until you hear back from @nunocoracao about how far you'd be allowed to alter Blowfish design?! 🤐😇🤷

At the end of the day, it's your time, feel free to push the envelope ... that's how great things are built 😅🙇

Oh geez... this was a long one. A couple of comments here (sorry if I jumped over some parts this was a long thread to catch up with)

thanks @wermos for raising this, I do think is a valid problem and by explaining the pain you're feeling I can think of a solution - either the scroll or something else. And I would be really happy to review a PR with your proposal.
thanks @Usual-Coder thanks for keeping the discussion in check and positive all the time. And I am sorry that you are not using Blowfish any longer :P - even though I 100% get your comment.
thanks @maciejopalinski for being passionate about this issue

Now, my take on this discussion. I do believe this should be implemented - I will need some time though :D
A couple of points on how I see the solution

In my head, this is an edge case and people having the problem are ok with a "scrolling" solution and I think there are ways of implementing one that makes sense for the rest of the theme - @Usual-Coder provided some good suggestions here.
I don't think this should be another-config - the theme is getting bloated with too many configurations and it seems we are discussing a solution for a problem that doesn't have "a" solution at the moment.
I would implement this in a way that current use cases stay the same i.e. no breaking changes for existing users
...and in a way that it changes when needed to accommodate long content i.e. without configs

If the above is not possible, aka it requires redesign than the second best option is to add another config...
Were you planning on working on a PR or do you want me to take this on? - to be clear, it will take me a while to have time to focus on this particular one

Thanks for the response, @nunocoracao!

Sorry for not getting back to any of you sooner.

No worries. Better late than never 😄

Were you planning on working on a PR or do you want me to take this on?

I would feel better if you took this on. I'm only beginning to familiarize myself with the repo, so I'm more comfortable with bug fixes and the like. I spent some time looking at the existing ToC implementation, and I think that fixing this would require me to rewrite the layouts/partials/toc.html file.

@wermos I'm sorry, I didn't have time to find an example + I'm probably going to unsubscribe soon = won't receive further messages. Having only a draft on how to derive/create a Hugo theme (finishing it is now at the very bottom of my TODO = no task/time will be allotted, at least for a few months), I can't help you further. Sorry. I hope you'll find your own solution.
To end this part on a constructive note / trying to help you contribute ... or build your own solution:

If you're not familiar with the merging process used by Hugo (aka how it handles everything / its virtual filesystem, even for imported themes), it could be a good way to start trying to fix/enhance Blowfish = you can overwrite every file of Blowfish(/any) theme, to test the impact of your modifications on-the-fly + using (Hu)Go modules (instead of messing with git submodules = take a look at "Use Hugo Modules"  and "Theme components" sections) will probably help you work more efficiently on a solution ... and its distribution ... until PR are accepted ... or not 🤐😇
To improve the on-the-fly part, using a containerized Hugo would probably help with the trial/error process.
No "official" image being maintained/documented, if you're familiar with containers, I would STRONGLY suggest building your own. If not, it pains me (I don't like recommending things I'm avoiding myself 😏) to say that HugoMods images are probably the most useful images available, at the moment 🤷. Don't get me wrong, like everyone giving resources freely to the community, I have great respect for the guy behind HugoMods 🙇. The pain is coming from how such kind of images are built! 🧐😱 In this very case, while fully automated, relying on an exotic/unmaintained syntax/process + the lack of privileges de-escalation, etc. ... aka the why you should do it by yourself, if you know how to (actually I wasted a lot of time there too, to the point I had to contribute to docker/buildx bake, learning Go in the process, while trying to create an alternate pipeline relying on an up-to-date/maintained solution = yep I got upset wasting so much time 🤬🤯, I thought wasting even more, trying to fix hardcore stuff / building the missing piece, was the good solution 😅 ... let's just say cutting my losses with Hugo/Blowfish&co has been painful in many ways 🤐😞)

Besides those pointers, I'm sorry I can do much more + I probably won't contribute to Blowfish anyway (if I come back near Hugo / these kinds of time-consuming topics, it will probably mean creating a new theme / everything else to fit my needs) = still I think it's one of the best theme around worth your time / contribution(s) 🤷 Have fun playing with it 😅
@nunocoracao thanks for your answer and don't get me wrong on the "I would prefer to create my own" = I was hoping to fill a gap with Hugo/Blowfish until I had a proper LMS, up and running (some of my needs are related to courses/learning process = I can't afford not having tables of contents behaving the way they should + missing other useful features = the fixing/adding time has to be put in doing it directly in the LMS) = I'm mean it, great job with this theme 🙇. Hope you'll be able to find the time to keep it evolving ... and hopefully the necessary help/contributors 🤞
Best regards to you all 🤘

@wermos @maciejopalinski @Usual-Coder feel free to have a look at https://github.com/nunocoracao/blowfish/pull/1529
A preview link should be there soon to see the differences... this was wayyyyy harder than I expected.
Thanks for suggesting this though got to learn some sutff

@wermos: I thought about creating an example design for you (not having time to read Blowfish code, opening something like Figma seemed rapidly overkill / too far fetch) ... but, since we are never safe from an unexpected/pleasant surprise 😅 ... while answering @nunocoracao I stumble on an example of "sublevels handling" (with vertical indicators) 😅🎉 from one of the previously linked examples = https://elysiajs.com/eden/treaty/overview.html ... on the left (used for global navigation ... but you can extrapolate it for a ToC 😅). I personally like minimalistic designs, so I would consider keeping it even more simple = trying to show only the deepest vertical bar + hide the highest one (replacing it with the same vertical indicator in front of the opened entry). I wouldn't add/mess with the background color (you could consider it useful when trying to keep the info about the main stuff, as it's done on this example ... but I would keep a low/discrete profile for the ToC). (Instead) I would also play with the alpha component of the font (aka opacity) = darkening out of reach entries (obviously restoring full white/black on mouseover) + bold lettering for the parent entry, when opened, and the current one (the child/shown entry) = yep collapsible entries are definitively a must-have with "big docs" ... but that part may also require a lot of work 😇 Remembering, those are only suggestions + opinionated/subjective considerations = I don't own any truth ... feel free to contribute while ignoring them completely 🤷 @nunocoracao is paving the way for many possibilities, don't be afraid to try new things 😉
Just found another example showing something very similar to what I just wrote (ignore the ToC, inspiration could be taken from the navigation on the left): https://clerk.com/docs/authentication/social-connections/gitlab. And this incredible variant (take a look at the ToC on the right), so simple but efficient: https://clerk.com/docs/custom-flows/user-impersonation#revoke-an-actor-token-from-the-backend-api 😱🤩
@nunocoracao (even if I'm not coming back to Hugo/Blowfish, I took a rapid look at your files changes = too many modifications = don't have time to check the PR technically ... Sorry 😞) I'm impressed by your commitment to improve this theme. Great respect for that 🙇 You deserve the interest your theme is picking. I hope you'll get the traction it really deserves 🤞
Now, concerning the PoC/preview you've provided in the PR (doing such a thing/producing this kind of effort is really ~amazing™~ mesmerizing), the following remarks are made per your request (if something is not clear, ask for additional precisions 😅😇):

Per UX fundamentals , given an optional component (scrollbar), it shouldn't be displayed when it's not needed / could be confusing (like 2 scrollbars too close) ... aka "keep the UI clean as much as possible" + "show only what's necessary". Examples linked above don't show any scrollbar = the scrollable area is triggered depending on mouse position = It's not technically needed ... but, as always (it's your prerogative), you could argue you prefer it your way for other reasons (design/whatev)?! 🤷 Your main scrollbar is not shown when it's not needed (ex.: authors page) ... if you want to keep the ToC scrollbar, shouldn't you apply the same logic everywhere?!

On a more subjective/opinionated note, I would have it removed completely + the underline in the Toc is too strong (for my own taste) = I would replace both with vertical indicator(s) in front of the entry(ies), as shown in linked examples.

Improving UX (would probably require even more additional rework 😅😇): a ToC must answer the "where you are" but it could also show the "what is (actually) covered" (like https://bun.sh/docs/bundler/loaders#toml) or point/indicate the "where you go" (useful in learning/teaching context, aka tutorials, courses, processes, etc.), like https://clerk.com/docs/quickstarts/fastify. For that last variant, you could already achieve something quite similar with your Timeline shortcode, for the main page = the ordered ToC is the only missing component 🤐😇

Hope this feedback will help your reflection. 🤷

Thanks @Usual-Coder.
Regarding the design perspective I will definetly play around with it. I also don't like the huge blocks - it came with the lib.
I relied on the markdown sample article and reduced the height of the window to test this. Behavior should be, no scroll when TOC is smaller then height of window - otherwise scroll becomes visible... are you seeing something different?

Short answer
"no scroll when ToC is smaller than the height of window" may be good enough, for now = it will fix the issue = take the (quick)win 🤷 (and start a discussion "somewhere else" about design + UX considerations with users willing to have constructive contributions / being a force of proposal ... or simply sharing design inspirations 🤷).
Not so short answer 😏
It may not be that simple, depending on what you want to do with it 🤷 I started a detailed answer ... but this issue, already quite long 😅, would have exploded 😇 = obviously (theoretically) you'll always have: ToC (aka component) max-height < (...) < window height (aka presentation area). The (...) representing (optional) things you probably didn't consider/think to implement (for now 🤷).
But, as always, the devil is in the details 😏
Let me give you an example outside of considerations regarding the ToC itself. If we call the ToC a component (not the worst idea, some would say 😅😇) and its parent container a panel (avoiding considerations related to the layout, for the sake of this example / to keep it short 🤐), you may put other components in that panel = a (quick)search form/entry at the top, additional/useful links at the bottom, etc. (take a look at the links) ... you could even have other components right in the middle of the ToC (we are talking evil here ... let's consider ads for those who may try that to monetize their blog / would still think it's possible/sufficient to cover their costs with that kind of annoying things 🤐) = you may have to deal with the height distribution in different ways 🤷 (start having playful usage of CSS variables and other things like that 😏)
Now, talking about the ToC only, you may try to make it really smart™ 😏 aka dynamic ... (line-)height + fading/folding/animating out of reach / unnecessary entries ... even better, not showing the full ToC at all 😱 (may come handy for VERY long docs, procedures, etc.), but adapting displayed entries (depending on the mouse position 🤔) ... etc.
= I could fill your (full-time) backlog for weeks (if not more) just on that topic  ... but, without knowing your (real) motivations + where you want to go with your work here (the somewhere else / those discussions may be organized outside GitHub = try building a community around your work, etc. 🤷 yep ... let's just say, I'm a pushing the envelope kind of person 😅), I think it's better to stick with the Pareto principle = fixing this issue, you've probably done enough for your main (aka the 80% 😉) audience 🤷 (and I must go back to my own troubles with very similar topics 😅)
Again, I hope it will help your reflection.
I wish you the best for the rest ... and thanks for the time / fixing this one 🙇

feel free to have a look at #1529

Sorry I wasn't able to get to it sooner, I was a bit busy with some other things.
I took a look at it now, and I am quite satisfied with the result. It definitely addresses this issue in its entirety.
I also think that @Usual-Coder is correct, but further tweaking can always be done in later PRs. I think it's good enough for now. Great work!


warning ILT0005: Type '*' from assembly 'Windows' was not included in compilation, but was referenced in the application. There may have been a missing assembly, or a dependency on a more recent Windows SDK release.

It would be great to also include troubleshooting for following too
Sometime, the warning becomes errors.
8>C:\Program Files (x86)\Microsoft SDKs\UWPNuGetPackages\microsoft.net.native.compiler\2.2.8-rel-28605-00\tools\Microsoft.NetNative.targets(801,5): warning : ILTransform : warning ILT0005: Type 'Windows.UI.Xaml.Controls.Primitives.GeneratorPosition' from assembly 'Windows' was not included in compilation, but was referenced in the application. There may have been a missing assembly, or a dependency on a more recent Windows SDK release.
8>C:\Program Files (x86)\Microsoft SDKs\UWPNuGetPackages\microsoft.net.native.compiler\2.2.8-rel-28605-00\tools\Microsoft.NetNative.targets(801,5): warning : ILTransform : warning ILT0005: Type 'Windows.UI.Xaml.CornerRadius' from assembly 'Windows' was not included in compilation, but was referenced in the application. There may have been a missing assembly, or a dependency on a more recent Windows SDK release.
8>C:\Program Files (x86)\Microsoft SDKs\UWPNuGetPackages\microsoft.net.native.compiler\2.2.8-rel-28605-00\tools\Microsoft.NetNative.targets(801,5): warning : ILTransform : warning ILT0005: Type 'Windows.UI.Xaml.Duration' from assembly 'Windows' was not included in compilation, but was referenced in the application. There may have been a missing assembly, or a dependency on a more recent Windows SDK release.
8>C:\Program Files (x86)\Microsoft SDKs\UWPNuGetPackages\microsoft.net.native.compiler\2.2.8-rel-28605-00\tools\Microsoft.NetNative.targets(801,5): warning : ILTransform : warning ILT0005: Type 'Windows.UI.Xaml.DurationType' from assembly 'Windows' was not included in compilation, but was referenced in the application. There may have been a missing assembly, or a dependency on a more recent Windows SDK release.
8>C:\Program Files (x86)\Microsoft SDKs\UWPNuGetPackages\microsoft.net.native.compiler\2.2.8-rel-28605-00\tools\Microsoft.NetNative.targets(801,5): warning : ILTransform : warning ILT0005: Type 'Windows.UI.Xaml.GridLength' from assembly 'Windows' was not included in compilation, but was referenced in the application. There may have been a missing assembly, or a dependency on a more recent Windows SDK release.
8>C:\Program Files (x86)\Microsoft SDKs\UWPNuGetPackages\microsoft.net.native.compiler\2.2.8-rel-28605-00\tools\Microsoft.NetNative.targets(801,5): warning : ILTransform : warning ILT0005: Type 'Windows.UI.Xaml.GridUnitType' from assembly 'Windows' was not included in compilation, but was referenced in the application. There may have been a missing assembly, or a dependency on a more recent Windows SDK release.
8>C:\Program Files (x86)\Microsoft SDKs\UWPNuGetPackages\microsoft.net.native.compiler\2.2.8-rel-28605-00\tools\Microsoft.NetNative.targets(801,5): warning : ILTransform : warning ILT0005: Type 'Windows.UI.Xaml.Media.Animation.KeyTime' from assembly 'Windows' was not included in compilation, but was referenced in the application. There may have been a missing assembly, or a dependency on a more recent Windows SDK release.
8>C:\Program Files (x86)\Microsoft SDKs\UWPNuGetPackages\microsoft.net.native.compiler\2.2.8-rel-28605-00\tools\Microsoft.NetNative.targets(801,5): warning : ILTransform : warning ILT0005: Type 'Windows.UI.Xaml.Media.Animation.RepeatBehavior' from assembly 'Windows' was not included in compilation, but was referenced in the application. There may have been a missing assembly, or a dependency on a more recent Windows SDK release.
8>C:\Program Files (x86)\Microsoft SDKs\UWPNuGetPackages\microsoft.net.native.compiler\2.2.8-rel-28605-00\tools\Microsoft.NetNative.targets(801,5): warning : ILTransform : warning ILT0005: Type 'Windows.UI.Xaml.Media.Animation.RepeatBehaviorType' from assembly 'Windows' was not included in compilation, but was referenced in the application. There may have been a missing assembly, or a dependency on a more recent Windows SDK release.
8>C:\Program Files (x86)\Microsoft SDKs\UWPNuGetPackages\microsoft.net.native.compiler\2.2.8-rel-28605-00\tools\Microsoft.NetNative.targets(801,5): warning : ILTransform : warning ILT0005: Type 'Windows.UI.Xaml.Media.Matrix' from assembly 'Windows' was not included in compilation, but was referenced in the application. There may have been a missing assembly, or a dependency on a more recent Windows SDK release.
8>C:\Program Files (x86)\Microsoft SDKs\UWPNuGetPackages\microsoft.net.native.compiler\2.2.8-rel-28605-00\tools\Microsoft.NetNative.targets(801,5): warning : ILTransform : warning ILT0005: Type 'Windows.UI.Xaml.Media.Media3D.Matrix3D' from assembly 'Windows' was not included in compilation, but was referenced in the application. There may have been a missing assembly, or a dependency on a more recent Windows SDK release.
8>C:\Program Files (x86)\Microsoft SDKs\UWPNuGetPackages\microsoft.net.native.compiler\2.2.8-rel-28605-00\tools\Microsoft.NetNative.targets(801,5): warning : ILTransform : warning ILT0005: Type 'Windows.UI.Xaml.Thickness' from assembly 'Windows' was not included in compilation, but was referenced in the application. There may have been a missing assembly, or a dependency on a more recent Windows SDK release.

[Enter feedback here]

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

ID: 63e28f14-43fd-3fa0-21f7-404877de279d
Version Independent ID: 416cae79-ea3a-3f3c-7485-f7b14fe94cbb
Content: .NET Native General Troubleshooting
Content Source: docs/framework/net-native/net-native-general-troubleshooting.md
Product: dotnet-framework
GitHub Login: @gewarren
Microsoft Alias: gewarren

@gewarren Should we transfer this issue to https://github.com/MicrosoftDocs/windows-uwp ?

This issue was moved to MicrosoftDocs/windows-uwp#3099


Support for svcutil /wrapped parameter

Some of WSDLs in the wild (especially from Java world) cannot be consumed without it. For example:
https://wasstt.infomonitor.pl/931154b7-682c-4560-b0fe-0e5a825694b5/ws/BIG?wsdl
Combined WSDL/XSD *.wsdlschema file for TypeProvider:
https://gist.github.com/orient-man/4c1dc0402a7a1171a3d9bebeafbf9448
WSDL TypeProvider fails on this one. I added Wrapped static parameter and now this works:
type Big = WsdlService<ServiceUri = "https://wasstt.infomonitor.pl/931154b7-682c-4560-b0fe-0e5a825694b5/ws/BIG?wsdl", Wrapped = true>

@orient-man Thanks for this contribution, apologies it took so long to merge.


Added a dedicated "Terms and Conditions" Page

Hey @PSS2134

Issue fixes #463

I have added dedicated Terms and conditions page and link it to the footer

Please take a look and review it!

Hey @PSS2134 pls review this PR

Keep gap b/w all the points and a noticeable gap b/w last point and footer @MastanSayyad

Keep gap b/w all the points and a noticeable gap b/w last point and footer @MastanSayyad

Okay I will do it right away!

Hey @PSS2134 I made the changes


[ML] Update README to specify that users should also install azure-identity

Description
Related to #29226
azure-identity was removed as an explicit dependency of azure-ai-ml leading up to its GA release, but versions >=1.0.0,<1.3.0 depended on it implicitly due to opencensus-ext-azure.
This pull request updates the READMEs in this repository to make it clear that the intended installation process for azure-ai-ml is:
pip install azure-ai-ml
pip install azure-identity

Which should be in alignment with the other Azure Python SDKs.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


Add tutorial working with workspaces

Added tutorial for workspaces (basics: how to initialize, how to make cross contract calls and use cli to deploy/call + how to share interfaces -- closing #679 )

Preview is available here:http://developers-pr1106.previews.kube001.services.stellar-ops.com

Preview is available here:http://developers-pr1106.previews.kube001.services.stellar-ops.com

Preview is available here:http://developers-pr1106.previews.kube001.services.stellar-ops.com

Preview is available here:http://developers-pr1106.previews.kube001.services.stellar-ops.com


Moving DataLake* to master branch

Since this is still a hyak project, it should be in master and not AutoRest

Hi @begoldsm, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (begoldsm). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;


Update corrupt data file handling

This PR updates the way corrupt data file is handled.
For duplicate person, the first is kept and subsequent data are discarded.
For person data containing illegal value, that person will be discarded but the remaining person data with no illegal value will be read as per normal.
closes #75

LGTM! One nit comment easy fix. Handling duplicates and corruption works fine from my testing, but in the future we should add a message upon boot up to inform the user that data has been tampered with. Otherwise, the user will lose data (upon making changes) and wouldn't know it. Maybe can add this is a later milestone.

Thanks for the suggestion! Actually this has been implemented in the form of logs. If you are referring to the GUI, then yes we definitely can explore that.


build(prerelease): change branch name for prerelease

:tada: This PR is included in version 1.0.0-alpha.18 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[$250] Bank account - Avatar icon for the owner is not legible in dark mode

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.16-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Action Performed:
Precondition:

App is in dark mode

Go to staging.new.expensify.com
Go to workspace settings > Workflows
Click Connect to bank account
Proceed to bank account set up
On Part 4, select Yes on the first step "Do you own 25% or more of Alberta Bobbeth Charleson?'
Select Yes on the next step "Do any individuals own 25% or more of Alberta Bobbeth Charleson?"
Enter all the required information (name, DOB, SSN and address)
Click Confirm on the confirmation page
Select No on the next step

Expected Result:
The avatar icon for the owner should be legible in dark mode
Actual Result:
The avatar icon for the owner is not legible in dark mode
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/user-attachments/assets/7a8f49e9-b3c8-421a-8e40-9921712a3dcc

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01f1b46ddf957e4ce7
Upwork Job ID:<PHONE_NUMBER>779219098
Last Price Increase: 2024-08-05
Automatic offers:

nyomanjyotisa | Contributor | 103407542

Issue OwnerCurrent Issue Owner: @mollfpr

@greg-schroeder  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

PR is on staging. Awaiting deploy to prod

Oops - automation failed and this was deployed to prod on 8/15 or 8/16 I think. With that said, I'll work on processing payments now.

Payment summary:
Contributor: @nyomanjyotisa - $250 - Paid via Upwork
C+: @mollfpr - $250 - You can make a manual request via ND

$250 approved for @mollfpr


Investigate loading SDK without blocking page-load

What
We should investigate if we can load the SDK async without side-effects.
If we can and found no reason not to do it, we need to adjust docs an the in-app instructions in the App O11y plugin

Did some first rough  tests.
Works fine with async anddefer attributes added.
Note 1:
I struggled a bit with initializing the React Instrumentation that way but did not put too much time into it. So I need to double check this specific case.
_(I think that a lot of the users, even if they have an spa, do not want to add any 3rd parties to their initial bundle and rather lazy load them later after delivery to the browser. _
Note 2:
The async attribute is sometimes added alongside the defer attribute as a fallback for older browsers which do not support defer. It's important to know that the execution order of scripts differ between the two. Defer keeps the relative order of scripts while async does not. Even that this shouldn't be any problem it needs to be double checked.

@codecapitano @cedricziel any action we can take on this? or has Marco's answer satisfied this ticket?


Cube map image quality on iphone

Hi Matthew
I've noticed a significant difference in the quality of an image when using cubemaps on an iphone when compared to android or a PC for that matter.
This first screen shot is from an iPhone 6s iOS 12.1.2 safari browser.  The left hand image is a 4096 pixel wide equirectangular.  The right hand side the same image, but 5376 pixels wide as a cube map. But it actually looks a lot worse.

The same images can be seen here on an android phone.  You can see that the image quality from left to right is similar.

Here's another good example from the iPhone where the left hand side is again a 4096 pixel wide equirectangular, and the right hand side a cube map image originally 8192 pixels wide.

This time you can see the benefits of the higher resolution cube map in nearly everything but in the horizontal detailing of the van doors.  On an Android device or PC, you don't see this pixelization.
Any ideas as to what might be causing the degradation of the cub map image on the iPhone? Or is it something iPhone users will have to live with?
Best wishes
Derek

I may have fixed this pixelation issue in 59893bcea4629be059ad3e59c3fa5bde438d292b, although I can't be sure in this case, as I was not able to reproduce it. See https://github.com/mpetroff/pannellum/issues/725#issuecomment-482757714.

Hi Mathew
Just tested the fix, it works beautifully, thank you.
Best wishes
Derek

On 13 Apr 2019, at 01:11, Matthew Petroff<EMAIL_ADDRESS>wrote:
I may have fixed this pixelation issue in 59893bc, although I can't be sure in this case, as I was not able to reproduce it. See #725 (comment).
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi Matthew
I thought I would share a screen shot of the same view as above after the fix you introduced.

Again from an iPhone 6s, where the left hand side is again a 4096 pixel wide equirectangular, and the right hand side a cube map image originally 8192 pixels wide.
Both images have been dramatically improved, though the higher resolution image is obviously clearer.
I'm really pleased and impressed.
Many thanks
Derek


[WIP] bootstrapped TypeScript declarations

cc @lukeed @jbucaran
Hi, as discussed.
Also i am wondering how we could describe node related functions like read write that works with Buffer object ? Should we pass @typings/node as peerDependency ?

@hzlmn Woah, this is awesome thanks!

Should we pass @typings/node as peerDependency ?

What would that be used for? I thought the typings are mostly  to help code editors offer better completion, help type linters to check for type mismatches, etc.

Nice @hzlmn! I'm not sure what the answers are to your questions. I don't use TS but can help you find the answer if you need help.
Also, do you mind restructuring a little bit? Can we move this to the project root as fly.d.ts?
Then you also have to add fly.d.ts to the files entry of the package.json, or else your hard work won't be downloaded.

@jbucaran Yes, they are. But as fly is node tool it use some Node APIs like Buffer, for example, that is unknown for TypeScript by default,  but there are separate declarations for Node @typings/node
@lukeed Yes sure, restructuring is good, that's why i create this as a WIP
Possible solutions is to add @types/node as a peerDependency like here or just define any type for Node related stuffs but it is not very convenient.

Thanks!
Hmm.. It's probably not ideal, but I think the any type might be better here, because we don't want to tell every downloader that they're missing a peerDep for @types/node. I, for one, would be annoyed by that since I don't use TS.

Sorry, forgot about it. Updated


When I turn the batch_size up or change to the bert_large model, I get the following error.

Unfortunately, these max batch sizes for BERT-Large are so small that they will actually harm the model accuracy, regardless of the learning rate used. We are working on adding code to this repository which will allow much larger effective batch sizes to be used on the GPU. The code will be based on one (or both) of the following techniques:Gradient accumulation: The samples in a minibatch are typically independent with respect to gradient computation (excluding batch normalization, which is not used here). This means that the gradients of multiple smaller minibatches can be accumulated before performing the weight update, and this will be exactly equivalent to a single larger update.Gradient checkpointing: The major use of GPU/TPU memory during DNN training is caching the intermediate activations in the forward pass that are necessary for efficient computation in the backward pass. "Gradient checkpointing" trades memory for compute time by re-computing the activations in an intelligen
t way.However, this is not implemented in the current release.

how to solve this problem


popV crosswalk for heart lists hepatocyte (liver)

See https://github.com/hubmapconsortium/hra-workflows-runner/blob/main/crosswalking-tables/popv.csv#L88
See:
| Organ_ID	| Annotation_Label |	Annotation_Label_ID |	CL_Label	| CL_ID	| CL_Match |
|---|---|---|---|---|---|
| heart	| UBERON:0000948	| hepatocyte	| PV:0000077	| hepatocyte	| CL:0000182	| skos:exactMatch |
This has ramifications for these two datasets from the same team:

https://entity.api.hubmapconsortium.org/entities/a48ab0bf5d8084da24859c4e64336e9c
https://entity.api.hubmapconsortium.org/entities/d0f164331f53f745cfa3e67bf3feb22d

I did a quick investigation and discovered that hepatocyte is present in some of the popv algorithm's heart models. So this seems to be a problem with popv itself rather than our processing.

Working with Supriya on 2DFTU manuscript, I asked to see which organs had cells in common with each other.  For the most part the common cells were typically immune cells (B cells, T cels etc), connective tissue cells (fibroblasts) and in the case where nested FTUs exist nephron<--glomerulus, nephron<--tubules (this includes all the parts of the tubules that we have as separate 2D FTUs).  I also then noticed this outlier of hepatocyte in heart and liver, which was WRONG. Tracking it back, I found it in the popv crosswalk file, but that was derived from a list provided from Bruce which I'm not sure where he obtained his list.
I will upload a new popv crosswalk to revise/fix this mistake.
Reviewing this file filtered-CTs-with-datasets-with-organ.csv I saw "hepatocyte associated with heart" which is clearly incorrect, but was able to determine from this file it was enriched from popV crosswalk.  This crosswalk was published in the 7th release in HRA-KG-->digital objects--> ctann as well as being used in the hra-workflows-runner github repo for CTAnn/HRApop work.

The main reason we can NOT have a table that lists "Need list of cell types that are not supposed to be in organ model" because some cell types like various immune cells will always appear in all organs---different numbers or specific immune cells.
The hepatocyte example is the only one we found because it's such an obvious mistake---hepatocytes should only be in liver not heart or lung.
The other possible issue is if a sample block had contaminating tissue from surrounding organs---example: liver is in close proximity to both heart and lung. In this case, one would expect that the number of these contaminating cell types in a dataset should fall below the threshold one would expect of cells that natively exist in the primary tissue. This is one of the preprocessing steps for single cell RNA sequencing analysis--- remove cells that are 3 or below. This threshold can be modified during the analysis.

From Katy:
Please omit the entire organ from popV CTann.
Docu all well in code and paper.

@bherr2  pushed a change that maps all hepatocyte => cell for the heart/popV.

Update: @bherr2 pushed a commit to remove heart popV cell summaries

@andreasbueckle @axdanbol @bherr2 Tabula sapiens version 2 just came out to cellxgene and bioxiv paper, although popV looks like they are still testing changes based on branches in their repo.
<img width="1143" alt="Screenshot 2024-12-12 at 3 49 06 PM" src="https://github.com/user-attachments/
Old cellxgene for 2022 published tabula sapiens using popV

New cellxgene for tabula sapiens 2 Dec 4, 2024 (no more hepatocytes showing up)