COMMAND_CODE_DEBUG_INFO is unused

This structure is defined here: https://github.com/01org/TPM2.0-TSS/blob/master/sysapi/sysapi_util/debug.c but is unused in the code AFAIK. Sweep this up.

Nuke it.

This should be fixed by #188


Fix page headers and margins

Summary of the pull request

Page titles should be 20 pt text (SubtitleTextBlockStyle) rather than 28 pt text (TitleTextBlockStyle)

#567

FeedbackPage.xaml: Fix code indentation
Thickness.xaml:

Remove NavigationViewPageContentMargin. It was not used in Dev Home or WinUI 3 code.
Adjust NavigationViewHeaderMargin so that if a default header were used (we don't use any currently) it would be displayed in the same location our custom headers are displayed.
Adjust ContentPageMargin so the page content is higher up

In design specs / #869

ShellPage.xaml: Remove unnecessary extra Grid. Round corner here, instead of in WhatsNewPage.xaml, as suggested in https://github.com/microsoft/devhome/pull/1258#discussion_r1253655793
ShellPage.xaml.cs: Change ShellInfoBar to display with 0 height. It is not currently used anywhere, but the 5 px height was pushing down the content of every page. If the service is used in the future, this sizing can be tweaked.
WhatsNewPage.xaml: Remove rounded corner in favor of above. Rest of change is whitespace, LF newlines got converted to CRLF (these should all be changed eventually)

Old look:

New look:

Validation steps performed
Tested locally.
PR checklist

[x] Closes #567
[x] Closes #869
[ ] Tests added/passed
[ ] Documentation updated

Is there a place where we've said that we should only be using LF for the project? I wonder if we can detect it during development with linting.

@bbonaby We're developing something for Windows, we should use Windows line endings. #1384


adjust menu layout for iPad layout

The top menu is squished when viewed at this size:

Appears to be collapsing correctly now at 1036 x 1366


Implement capability to use a validator config file

This commit adds a ValidatorConfig object, an example
validator config file, and updates the validator cli to
check for a config file and combine it with any command
line options set.
Important to note the public-uri is no longer "required" to
be set on the command line, but if it is not set by either
the command line or the config file the validator will
shutdown.
Signed-off-by: Andrea Gunderson<EMAIL_ADDRESS>

Updated to include the above requested changes, except for changing the --join option as a better name could not be decided on.


read env variable to set GA tracking ID

[updates #103597472]

This will need to pull in the latest version of money-to-prisoners-common once the template change has been merged and the version has been bumped.


⚠ BCR Bot has degraded performance

In efd05d2, BCR Bot (https://bcr.alternativasinteligentes.com/BCRApi/web/chatbot) experienced degraded performance:

HTTP code: 200
Response time: 856 ms

Resolved: BCR Bot performance has improved in 77952f7.


Typos in Chapter 2

I am very new to Vue, which is why I bought this book.  From what I can tell the following has errors in it and it does not reflect the source file it refers to.   This is from page 31 of the PDF and it is the same in the online version at packtpub.com.

Vue has dominion over the element it mounts on and any child node. For our project so far,
Vue could manipulate the div with the header class, but it could not manipulate
the div with the toolbar ID. Anything placed within this latter div will be invisible to
Vue.
index.html:

...

...
...

In this example code given the div that has a header class is closed with a  tag, which doesn't make sense to me.  It is also completely different from the actual code in the index.html file, which is the following:

vuebnb

There is no div with a class called header.  The class is called container.  I am confused enough.  These typos don't help.

Yeah, you're right, that code snippet should be:

...

...

  ...

Yes, and the text should be referring to the container class, not the header class.
I am happy to report I am now part way through Chapter 4 and I have found no more typos.   Enjoying the process so far.


Inquiry about good first issues

Greetings!
I have lots of interest in this project, especially the following two parts:

Convert DSL to MLIR.
Improve performance of a specified operation or a model.

I'm trying to learn and participate in it but I have found there is few good first issues. Could you please give me some guides or some new good first issues to start up? Many thanks!
Best regards,
Rinne

Hi Rinne,
Thank you for your interest in our project!

Regarding the conversion of DSL to MLIR, we currently have a basic example (https://github.com/buddy-compiler/buddy-mlir/blob/main/examples/README.md#dsl-examples) where the language from the MLIR Toy tutorial is translated to MLIR through our custom frontend. Our further plans include designing and implementing VectorDSL, with a preliminary task to develop extended vector abstraction. We are in the process of drafting a research paper on this topic, aiming to complete it by the third quarter of this year. Following that, we expect to finalize the implementation of VectorDSL.

To improve the performance of operations or models, this area is more accessible to new contributors. You might start by running examples such as Bert (https://github.com/buddy-compiler/buddy-mlir/tree/main/examples/BuddyBert) or LLaMA (https://github.com/buddy-compiler/buddy-mlir/tree/main/examples/BuddyLlama), examining the dumped MLIR files, and identifying specific operators for optimization to enhance performance.

Personally, I recommend the second topic to get started. If you run into any problems or ideas for improvement along the way, please feel free to let me know!
We also invite you to join our Slack channel (https://join.slack.com/t/buddycompiler/shared_invite/zt-13y6ibj4j-n6MQ8u9yCUPltCCDhLEmXg), where you can discuss in real-time with our team and other community members!
Additionally, our team is recruiting interns, and there will be opportunities available this summer through the OSPP summer project (https://summer-ospp.ac.cn/). If you are interested, please let me know.
Thanks,
Hongbin

@zhanghb97 Thank you for your reply! I'll follow your guide to start up and learn the implementation of buddy-mlir.

We also invite you to join our Slack channel

I have joined the channel. :)

Additionally, our team is recruiting interns, and there will be opportunities available this summer through the OSPP summer project (https://summer-ospp.ac.cn/). If you are interested, please let me know.

OSPP seems attractive to me! Is there any pre-task I could take before applying for it? I noticed this page that contains some tasks but I'm not sure of which is suitable to take since the OSPP projects haven't been published. I'll appreciate it if you could give me some suggestions about that. I have great interest in the path of DSL->MLIR->LLVM, and then the performance improvement of operation/model. :)

As for this year's OSPP, our projects will focus on performance improvement.
You can run Bert (https://github.com/buddy-compiler/buddy-mlir/tree/main/examples/BuddyBert) and LLaMA (https://github.com/buddy-compiler/buddy-mlir/tree/main/examples/BuddyLlama) E2E examples. Then, you can find some operations as the target to perform optimization with vector dialect, which can be a good start. Before the OSPP, we will also give some pre-tasks for specific projects.

Thank you for your suggestions. Performance improvement is also attractive to me. However I will be more than happy if there could be a few projects related with DSL&MLIR&LLVM. I'm looking forward to the start of OSPP. :)

@zhanghb97 I ran the LLaMA example but got an segmentation fault. I tried to compile it under debug mode and run it in GDB but failed to get its back trace.  Is there any suggestion to debug the error, please?


Nativescript 8 - postclone: "path" argument null

Hi,
I was attempting to create a new Nativescript 8 plugin using this plugin seed. However, I get this error:
The "path" argument must be of type string. Received null
I'm on Ubuntu 20.04.
Is this the right plugin seed to use?
Following the stack I get:
NativeScript Plugin Seed Configuration
prompt: What is your GitHub username (used for updating package.json)? Example: NathanWalker:  andreav
prompt: What will be the name of your plugin? Use lowercase characters and dashes only. Example: yourplugin / google-maps / bluetooth:  wifi-aware
Using WifiAware as the TypeScript Class name..
Will now rename some files..
prompt: Do you want to include a "TypeScript NativeScript" application linked with your plugin to make development easier (y/n)?:  (y)
prompt: Do you want to include an "Angular NativeScript" application linked with your plugin to make development easier (y/n)?:  (n)
Creating 'TypeScript' application from latest published template...
You are using the deprecated nsconfig.json file. Just be aware that NativeScript now has an improved nativescript.config.(js|ts) file for when you're ready to upgrade this project.
The "path" argument must be of type string. Received null
TypeScript-NativeScript application created at: ../demo
Adjusting scripts..
Installing plugin to ../demo ...
Error: Command failed: cd ../demo && tns plugin add ../src
/bin/sh: 1: cd: can't cd to ../demo

    at ChildProcess.exithandler (node:child_process:326:12)
    at ChildProcess.emit (node:events:369:20)
    at maybeClose (node:internal/child_process:1067:16)
    at Socket.<anonymous> (node:internal/child_process:453:11)
    at Socket.emit (node:events:369:20)
    at Pipe.<anonymous> (node:net:666:12) {
  killed: false,
  code: 2,
  signal: null,
  cmd: 'cd ../demo && tns plugin add ../src'
}

Thank you

I am getting the following output instead:
Creating 'TypeScript' application from latest published template...
No matching version found for<EMAIL_ADDRESS>
Seems that this has not been updated for 8.0 yet (which is a pity).

I am receiving the same error as above
Creating 'TypeScript' application from latest published template...
No matching version found for<EMAIL_ADDRESS>Is this going to be fixed or is this now abandonware?

same error


Unstoppable error flood freezes editor after trying to generate a Concave Collision Shape

 (Done)
Godot version:
3.1 Stable
OS/device including version:
Windows 10 Intel HD graphics 520
Issue description:
When you try to create a concave collision shape from this menu

You will get this error:
ERROR: build: Condition ' O == __null ' is true. Continuing..:
At: core/math/quick_hull.cpp:402
ERROR: build: Condition ' !F2 ' is true. Continuing..:
At: core/math/quick_hull.cpp:428
Sometimes it will stop and will generate a concave mesh but some times it won't stop at all and you will be forced to exit godot.
Steps to reproduce:
(I think this happens in objects that are made out of separated meshes)
*Create an object in some editor, import it into godot
*Select its meshinstance
*From the mesh menu select create concave staticbody
*It freezes.
Minimal reproduction project:
Try to generate concave staticbodies for these:
objects.zip

Looks like an old problem has been revived - https://github.com/godotengine/godot/issues/21141

I have the same problem when I try this on a CapsuleMesh. Using 3.1.1 Stable Mono.

Also happens with Convex collision shapes, on master (76678b260).
example.zip

Interestingly, mine fails on convex but is fine for trimesh.

Still valid in 3.2.4 beta3

Also happens with Convex collision shapes, on master (76678b2).
example.zip

Present in 3.3.2.stable.
Fixed in 3.4.beta3 (most likely by #48533).
Didn't check master (but most likely fixed by #48916).

Closing per @kleonc's comment.


Send email notification/adaptive card when a Team request is rejected

Is it possible to change the flow so it will send an email and/or display a notification to the original requester with the reasoning why the Team was rejected?
I know this information is available in the 'My Requests' section in the app but it would be nice if it could be sent via email so they can then correct the issue, without them contacting an admin a few days later asking where it is

@nmunkley good idea, I have now added this to our roadmap 👍

Is it possible to change the flow so it will send an email and/or display a notification to the original requester with the reasoning why the Team was rejected?
I know this information is available in the 'My Requests' section in the app but it would be nice if it could be sent via email so they can then correct the issue, without them contacting an admin a few days later asking where it is

Hi,
Add a 'Send an email' step after the 'Update status to Rejected' step. The 'To' field is the SharePoint 'created by email' in dynamic content list.  For consistency I have then copied out the body from the 'Send Team Created Email' in the logic app flow and amended it to read as a rejection message advising the requester to open the  Request-a-Team app to review the rejection and re-submit (with a link back to the App in the Team).

Is it possible to change the flow so it will send an email and/or display a notification to the original requester with the reasoning why the Team was rejected?
I know this information is available in the 'My Requests' section in the app but it would be nice if it could be sent via email so they can then correct the issue, without them contacting an admin a few days later asking where it is

Hi, Add a 'Send an email' step after the 'Update status to Rejected' step. The 'To' field is the SharePoint 'created by email' in dynamic content list. For consistency I have then copied out the body from the 'Send Team Created Email' in the logic app flow and amended it to read as a rejection message advising the requester to open the Request-a-Team app to review the rejection and re-submit (with a link back to the App in the Team).

Where can we control the From field?

Is it possible to change the flow so it will send an email and/or display a notification to the original requester with the reasoning why the Team was rejected?
I know this information is available in the 'My Requests' section in the app but it would be nice if it could be sent via email so they can then correct the issue, without them contacting an admin a few days later asking where it is

Hi, Add a 'Send an email' step after the 'Update status to Rejected' step. The 'To' field is the SharePoint 'created by email' in dynamic content list. For consistency I have then copied out the body from the 'Send Team Created Email' in the logic app flow and amended it to read as a rejection message advising the requester to open the Request-a-Team app to review the rejection and re-submit (with a link back to the App in the Team).

Where can we control the From field?

Hi, there's options at the bottom of the action box:


Update Regex for AccessLog

Fixed regex parser to handle cases where Bytes, Referer and User-agent fields are empty.
Change(s):

Updated the regex parser to handle cases where Bytes, User-agent, and Referrer fields are empty.

Reason for Change(s):

To ensure the regex parser can handle log lines with Bytes as -, and empty User-agent and Referrer fields.
To handle negative timezone offset ex: -0600

Version Updated:

N/A

Testing Completed:

Tested the updated regex parser with sample log lines containing empty fields.
Verified that the regex parser correctly captures all required fields even when some fields are empty.
Checked the validity of the parsed log data against the log line structure.

Checked that the validations are passing and have addressed any issues that are present:

Yes. The regex parser now handles empty fields without any issues.

Hi @0ccupi3R, we tried to check the parser functionality, but since don't have the latest sample data, we aren't able to test the functionality. Can you please attach here the latest sample data?

 Although we wanted you to update the sample data here

@v-rbajaj the logs are updated at #8618

@v-rbajaj the logs are updated at #8618

Ill

@v-rbajaj the logs are updated at #8618

I have requested changes on #8618 PR

This discussion regarding this PR is on going, please refer https://github.com/Azure/Azure-Sentinel/pull/8618

Discussion on going on https://github.com/Azure/Azure-Sentinel/pull/8618 this PR.

@v-rbajaj I have tested the new changes with sample logs. Logs are getting parsed ok. Please test the other content like Analytics Rules etc. They should not be breaking post these changes.


[6.0 cherry-pick][KeyPath] Fix regression in == on keypaths

Explanation: A recent change in KeyPath implementation (https://github.com/apple/swift/pull/72472) that was done for Embedded Swift and was supposed to be an NFC for regular Swift accidentally commented out an important check inside the implementation of == in AnyKeyPath. This caused a regressions in the equality of keypaths and they were misreporting being equal in some cases when they're not supposed to. https://github.com/apple/swift/pull/73347
Scope: Code that uses KeyPaths and specifically comparing them for equality.
Risk: Low -- a removal of the accidental changes that reverts back to the implementation of == that has been in use for years.
Testing: Added test case.
Issue: rdar://127296261
Reviewer: @Catfish-Man on https://github.com/apple/swift/pull/73347

@swift-ci please test


Maint: test meilisearch

Test meilisearch for pyaedt docs. # WIP

@MaxJPRey could we merge this PR?

@MaxJPRey could we merge this PR?

@Samuelopez-ansys Merging 🚀


Trigger 2.2/2.7

Signed-off-by: xuezhaojun<EMAIL_ADDRESS>

/assign @haoqing0110

/assign @haoqing0110

can we use this PR to trigger https://github.com/stolostron/clusterlifecycle-state-metrics/pull/93 ?

can we use this PR to trigger #93 ?

Sure, we can.

close as already triggered by https://github.com/stolostron/clusterlifecycle-state-metrics/pull/93


Restore pipenv package vuln check

Motivation and Context
Pipenv package vuln check was failing with an API key error.
What has changed
Implemented workaround.
How to test?
Run make check.
Links
Trello: https://trello.com/c/AcwAmLZ4

 Reinstate pipenv checks once the current API issue is resolved


Enhance Live Location on Google Maps for Driver and Ambulance

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
I would like to enhance the live location tracking on Google Maps to display the real-time location of drivers and ambulances. This feature should update live location data on the map for users, including the ambulance and driver’s positions. It would function similarly to real-time tracking on Google Maps, providing continuous location updates.
Use Case
This feature will allow users and hospital staff to track the exact position of ambulances and drivers in real-time. It can improve response times during emergencies and ensure better coordination between patients, drivers, and hospitals.
Benefits

Increased efficiency in managing and tracking emergency vehicles.
Better coordination for patient pickups and hospital visits.
Real-time updates provide users with transparency and reliability.

Priority
High
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

@mansiruhil13 can you assign the issue and add labels to this issue.

@mansiruhil13 can you assign the issue and add labels to this issue.

https://github.com/mansiruhil13/Bobble-AI/issues/1#issuecomment-2380723339


Allow specifying allow list of trusted runner labels

Let's say one has custom GitHub hosted runners they would have other labels, we need to allow list those
The self hosted runner rule should honor this list in addition to hardcoded regex

Draft of config file
include:
- path: .poutine
ignoreForks: true
rulesConfig:
  pr_runs_on_self_hosted:
    trusted_runner_labels: [ "my-paid-hosted-runner" ]


Load test process workflow fails intermittently

Description:
When the process workflow try to merge the result PR, sometimes it does not wait for the codecov status check and tries to merge before that passes. So the merge step fails.
And also need to do several improvements regarding branch deletion and checking for the repo owner.

Closing since this is completed


How to ignore an appointed struct field?

I have a orm model struct, I want to use it for @param's json body type. but there only two fields is needed.
How to ignore a field like go-swagger by using swagger:ignore?

You can.
https://github.com/swaggo/swag/blob/50660570fd78f4658c71f2612a970123ae36711b/testdata/simple/web/handler.go#L32


Improve racer#ShowDocumentation()

I added empty lines to before/after code snippets and before section title. It makes output more readable as following.
Before this PR

After this PR

And I added small refactorings. Nested substitute() is so unreadable...

Thank you


Update the version of all packages in pubspec yaml, and make sure project is compatible with latest version of flutter

@namanpopli Can you pick this up, I guess you are managing this repo

@apgapg @Sam1kumar UI Kit is compatible with the latest version of flutter.

@namanpopli report the same on discord group too


🐛 [Seafile] Cannot change or download files

Which addon?

Addon name : Seafile
Addon version : v9.0.14-pro

Describe the bug

To Reproduce
If I connect locally or over the configured url (over firefox, chrome, safari), I cannot change or download files.
If I try to save the file the save button just showing the loading circle.
If I try to download a file I get the error ERR_CONNECTION_REFUSED.
Full addon log
Starting...
/etc/cont-init.d/00-banner.sh: executing
-----------------------------------------------------------
 Add-on: Seafile
 High performance file syncing and sharing, with also Markdown WYSIWYG editing, Wiki, file label and other knowledge management features
-----------------------------------------------------------
 Add-on version: v9.0.14-pro
 You are running the latest version of this add-on.
 System: Home Assistant OS 9.4  (amd64 / generic-x86-64)
 Home Assistant Core: 2022.12.8
 Home Assistant Supervisor: 2022.11.2
-----------------------------------------------------------
 Please, share the above information when looking for help
 or support in, e.g., GitHub, forums
 https://github.com/alexbelgium/hassio-addons
-----------------------------------------------------------
/etc/cont-init.d/01-custom_script.sh: executing
[13:00:41] INFO: Execute /config/seafile.sh if existing
[13:00:41] INFO: ... no script found
/etc/cont-init.d/92-local_mounts.sh: executing
/etc/cont-init.d/92-smb_mounts.sh: executing
/etc/cont-init.d/99-run.sh: executing
[13:00:41] INFO: Setting variables
PGID='1000'
PUID='1000'
SEAFILE_ADMIN_EMAIL='redacted-email'
SEAFILE_ADMIN_PASSWORD=******
TZ='Europe/Berlin'
URL='redacted-url'
data_location='/share/seafile'
database='sqlite'
[13:00:44] INFO: Setting data location
... check /share/seafile folder exists
... setting permissions
... copy media files
... correcting official script
[13:00:44] INFO: Defining database
[13:00:44] INFO: Starting app
2022-12-28T13:00:44+01:00 [Entrypoint] Local time set to Europe/Berlin
2022-12-28T13:00:44+01:00 [Entrypoint] Checking permissions
2022-12-28T13:00:44+01:00 [Entrypoint] Adjusting identifiers
usermod: no changes
2022-12-28T13:00:44+01:00 [Entrypoint] Running launch script
2022-12-28T13:00:44+01:00 [Launch] Making symlink to latest version
2022-12-28T13:00:44+01:00 [Launch] Linking internal configuration and data folders with the volume
2022-12-28T13:00:44+01:00 [Launch] Waiting for db
2022-12-28T13:00:44+01:00 [Launch] Launching seafile
Starting seafile server, please wait ...
** Message: 13:00:44.794: seafile-controller.c(621): No seafevents.
Seafile server started
Done.
LC_ALL is not set in ENV, set to en_US.UTF-8
./seafile-server-latest/seahub.sh: line 236: warning: setlocale: LC_ALL: cannot change locale (en_US.UTF-8): No such file or directory
Starting seahub at port 8000 ...
Seahub is started
Done.
2022-12-28T13:00:53+01:00 [Launch] Done
2022-12-28T13:00:53+01:00 [Entrypoint] Waiting for termination

Full addon config
PGID: 1000
PUID: 1000
SEAFILE_ADMIN_EMAIL: "!secret seafile_username"
SEAFILE_ADMIN_PASSWORD: "!secret seafile_password"
TZ: Europe/Berlin
URL: "!secret seafile_uri_1"
data_location: /share/seafile
database: sqlite

System

Supervisor version: 2022.11.2
Host system version: Home Assistant OS 9.4  (amd64 / generic-x86-64)

Issue still present

Hi, thanks it wasn't up to date and some config were missing. You should see two new options to configure : the server ip (your HA ip/domain) and it's port (8082 by default)
I'll build it now and test it

Actually it doesn't seem to work. But you can change manually the server address by going top right in System Admin > Settings, and define SERVICE_URL as your ip

I've pushed a new version where the addon options could work

This is still present for me... I've just installed the add-on, but I'm unable to configure it properly.
My server is accessible via http://hass.local:8123, and thus the add-on config is as follows:
CONFIG_LOCATION: /config/addons_config/seafile/config.yaml # nothing here but the standard comments
FILE_SERVER_ROOT: http://hass.local:8082
PGID: 1000
PORT: "8082"
PUID: 1000
SEAFILE_ADMIN_EMAIL: ***
SEAFILE_ADMIN_PASSWORD: ***
SERVER_IP: hass.local
TZ: America/Sao_Paulo
data_location: /media/backup # this is the same path I can use to access my external drive from the SSH add-on
database: sqlite
url: seafile.example.com # not really sure what this is? that's the default value
localdisks: sdb1

The path it generates to download a file is something like http://hass.local/seafhttp/files/«hash/file.ext», but the one I got working (from trial and error) should be http://hass.local:8082/files/«hash/file.ext».
I've already tried the actual IP under SERVER_IP, as well as adding the port or protocol. The port even shows up correctly, but nothing (not even a leading slash) could remove the seafhttp from the path.


Switch off stalebot

The stalebot actions keep sending emails to me for unfixed issues such as #1682. It think it would be much better to switch off the stalebot.

Thank you for your feedback, we have discussed this internally and we see the value of the stale bot. I apologize for repeated notifications when something becomes stale. But for us, this is an important tool to prioritize. Most of the time, users lose interest in tickets they created. By requiring users to leave a quick comment a la "not stale", we know that they have in fact not lost interest and the issue is still relevant. This is common practice on most open-source projects and in my opinion a very small price to pay for being able to use cutting-edge software completely for free. Thanks for your understanding.

You might want to change the message of the stalebot to ask uses for this reaction. Also changing the timing might help.

You might want to change the message of the stalebot to ask uses for this reaction

Great idea! I've updated the message to make it a bit more friendly and explain the reasoning behind why the bot is marking issues as stale.


Build Mac installer

We'll use this issue to talk through options for the Mac installation experience.

Target Users

Internal VFSForGit developers (for this sprint - ending 11/01/2019.)
Office team, Mac users.

Requirements

A shell script that creates the VFSForGit Mac installer. This script can be used in the CI build machines. (for this sprint)
VFSFotGit Mac installer - installs VFSForGit Mac assemblies & PrjFSKext.kext.
VFSFotGit Mac installer - could be invoked from the command line using Terminal or a shell script. Would also provide a Installation Wizard UI (like VFS For Git Windows installer.) (for this sprint)
Nice to have - Uninstaller script.

Dependencies

Git for Windows installer on the Mac.
GCM installer for Mac.
Java 6 (for GCM).
During the prototyping this sprint, I will experiment and see if it is possible to combine the installers for dependencies into a single metapackage installer.

VFSGit Mac installer - What to install

All VFSForGit assemblies in the Publish folder produced by the CI build scripts (for this sprint.)
PrjFSKext.kext (for this sprint)
All scripts there is today needed to load PrjFSKext.kext and clone a repo (for this sprint).
GCM - using GCM installer for Mac
Git for Windows - using Git for Windows installer.

Goal for this sprint
Build a prototype installer – the goal is to help identify the right tools, alternative options & uncover any hidden issues. The installer would install all artifacts in the “Publish” directory and PrjFSKext.kext along with the existing scripts needed to load the Kext and clone a repo. The prototype installer will be for internal use only and not be released in GitHub.

During the prototyping this sprint, I will experiment and see if it is possible to combine the installers for dependencies into a single metapackage installer.

It is definitely possible (and in fact standard practice) to place one or more .pkg installer packages produced by pkgbuild into a scripted, customised installer .pkg using productbuild. I'm not sure what happens if you try to nest one of these into another .pkg, however.

What to install
[…]
2. PrjFSKext.kext (for this sprint)

Installing kexts isn't really any different from installing anything else below /Library/ with a pkg created with pkgbuild. If you have any questions, I might be able to help though as I've built quite a few kext installer packages over the years.

Thank you Phil. Could you please take a look at this PR https://github.com/Microsoft/VFSForGit/pull/438. It installs the Kext as well. Appreciate your feedback.


Add Text Summarization

Text Summarization
Feature Overview:

Purpose: To help students quickly understand and extract key points from long articles, research papers, lecture notes, and other textual materials.
Functionality: Users can input or upload text, and the system generates a concise summary.

Implementation Ideas:

Summarize Uploaded Documents: Allow students to upload documents (PDFs, Word files) and get summarized versions.

Highlight Key Points: Provide the option to highlight key sentences or paragraphs within the original text, helping students focus on important information.

Summarize Web Articles: Enable students to input URLs of web articles or academic papers and get summaries of the content.

Customizable Summaries: Offer different lengths for summaries (short, medium, long) based on user preferences or the context of use.

Multi-language Support: Provide summarization for documents in different languages, catering to a diverse student body.

Topic Extraction: Extract and list the main topics covered in the text, giving students a quick overview of the content's themes.

Integration with Study Tools: Combine summarization with other study tools like flashcards, where the summary can be broken down into key points or questions for review.

User Scenarios:

Quick Revision: Before exams, students can upload their lecture notes and get summarized versions to review quickly.
Research Assistance: When conducting research, students can summarize multiple academic papers to identify relevant studies faster.
Efficient Reading: Summarize lengthy readings assigned in courses, helping students grasp essential information without reading every word.

Example Interface:

Upload Section: An area to upload documents or input text.
Summary Output: A space where the generated summary is displayed, with options to copy, download, or save the summary.
Customization Options: Settings to choose summary length and language.
Highlight and Extract: Tools to highlight key points and extract topics.

By integrating text summarization into College.ai, you can significantly enhance the platform's value, making it easier for students to manage and digest large amounts of information efficiently.

@SurajSanap can you assign me this issue !

Hi @Shraman-jain, This feature is already available on the 'Ask to PDF' page, where you can upload your PDF and simply give the text command 'Summarize the given context in ___ words'.

@SurajSanap Generally, pdf are not easy to edit . Every time user has to edit the pdf and write that command.

@SurajSanap Any other specific feature you want to add ? You can tell , I will try to work on it.

Okay @SurajSanap


Sort Rooms stream by last updated

Hi,
I am streaming rooms with code below
FirebaseChatCore.instance.rooms()
I would like to show a room with the most recent activity at the top.
Is there a work around to sort rooms based on activity/last updated

Hi @mashegoindustries, please see this release v1.1.1 (specifically find a documentation comment I am referring to, it should give you the idea)


Cache, CachedData taking up quite some disk space

Mentioned on twitter that the ~/Library/Application Support/Code folder was taking up disk space.

On my machine, only MongoDB and Spotify take up as much space.
Is there anything we can do here to periodically clean?

Yes, we do clean it up and delete every thing older than 3 months: https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/electron-browser/nodeCachedDataManager.ts#L69. @chrisdias Are you saying that isn't working? Who tweeted this?

For insider we now delete everything older 8 days, for stable older 3 months


Persist and rehydrate ids of completed issues

The current application in this repo does not persist and rehydrate ids of completed issues. Completed issues are marked with a green check. Since the id of the completed issue is not persisted and rehydrated (when revisiting the page), subsequent page visits do not show the green check to indicate the issue had been previously completed.

Fixed in #37


No config file available

I can't find a configuration file to add the api keys for censys and security trails

You must create environnement variables:
censys: CENSYS_UID and CENSYS_SECRET
security trails: SECURITY_TRAILS_API_KEY
I'll add this in the readme, thank you.


Provisioning upgrade

Fix php 7.0 installation
Add required by composer php extensions and packages
Add composer global installation

@domnikl, what about merging this PR?
Any more comments?

Merged, thanks for your contribution!


cilium: populate wildcard src->dst policy for transparent encryption

Policy/State rules are populated when we receive kvstore or k8s
events. On egress we must match the (dst_addr, spi) tuple via the xfrm
policy and then link this to a state using the tmpl including the
same (dst_addr, spi). This results in a pair of policy/state rules
per node on the host. The above rules also specify a src address
so that we only match Cilium managed traffic. This includes the
IP address on cilium_host and the endpoint allocation range.
On the ingress side we have been doing the same but swapped. The
tuple (src, dst, spi) is used in the policy to match traffic and
then linked to the state via the tmpl. These are also added on
update events. However, this creates more rule than are actually
needed. By wildcarding the source on ingress traffic we can
match all traffic for the node with a single rule. Further the
existing scheme is buggy and dst-ip/spi pairs can collide.
The fix is to insert a wildcard decrypt rule when the local node
update event is received. Then on node update events only
add the egress rule.
Signed-off-by: John Fastabend<EMAIL_ADDRESS>

This change is

Lets get CI test in then merge this so we get CI coverage.

test-me-please

Added CI patch to this series on my test boxes with this patch CI tests pass reliably.

Coverage decreased (-0.02%) to 42.986% when pulling ae58bc04f43922e35d4605d664ed6db9879768d9 on xfrm-fix-src-collision into e3abc9c7f1146d3c587bca347d32724ab52f5731 on master.

OK this is passing now. I'm going to run it a few times more to check for any flakes.

test-me-please

test-me-please

Lots of noise today but try again without net-next and see if we still hit the e1000e restart kernel bug.

test-me-please

test-me-please

test-me-please

Run time monitor test failed this seems unrelated will retry

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

test-me-please

OK removed failing patches until we get a resolution. For now pushing minimal fixes. And will create a new PR for the failing patch so we don't hold these up.

test-me-please

Tracking failure here https://github.com/cilium/cilium/issues/7282 this seems unrelated we don't even have encryption tests enabled here.

test-me-please

FQDN flake also seems unrelated.

test-me-please

test-me-please

Passing now just to recount, these were seen:

https://github.com/cilium/cilium/issues/7282
https://github.com/cilium/cilium/issues/7190
? https://github.com/cilium/cilium/issues/7128


Fix Flask 2.0+ deprecation warning related to using jinja2.Markup, now it uses markupsafe.Markup

Hi,
I recently updated a project to use Flask 2.0 which came with Jinja 3.0 as a dependency.
After doing that upgrade I started to get this warning from Flask-WTF:
python3.9/site-packages/flask_wtf/form.py:133: DeprecationWarning: 'jinja2.Markup' is deprecated
and will be removed in Jinja 3.1. Import 'markupsafe.Markup' instead.
    return Markup(

This PR takes that advice and imports Markup from markupsafe instead of jinja2.

The CI failure looks maybe unrelated to this PR, any guidance on how to make it pass?

The CI failure looks maybe unrelated to this PR

It seems related to https://github.com/sphinx-doc/sphinx/issues/9216.

I created #437 as a temporary fix for building the docs.

Thank you!


update FHEM platform to the current version

lots of internal cleanups
more supported device types
query of values to fhem is nearly gone, initial device state (and longpoll) is used instead
optimized caching

edited title to reflect what the PR was updating "to the current version"


paging size not calculated accurately

Currently on moving to next page by clicking next button or the page button, 1 element is always getting skipped.
A pen demonstrating the issues  (on moving to page 2 the id seems to be starting from 5 instead of 4)
http://codepen.io/opavader/pen/vGZdRZ
I tried to fix this on my own fork and it seems the the size calculated @ https://github.com/Swimlane/angular-data-table/blob/master/src/components/DataTableController.js#L130 doesn't need to be padded with 1,
Doing that I was able to get the expected behaviour, as shown here http://codepen.io/opavader/pen/QNgQxz
Will be happy to submit a pull request if doesn't breaks anything.

Hmm interesting. I know there was a reason we did that. :S

Might have been for ensuring that page size is never less than 1 (which will make number of pages infinite) but the ceiling function does that any way. Ideally even that should be floor so that even the last element completely fits in.
IMHO this should do
this.options.paging.size = Math.max(Math.floor(
      this.options.internal.bodyHeight / this.options.rowHeight),1);

any updates on this?


Update the installed package with a newer version

Describe the problem/challenge you have
When a newer version of a package is available on the cluster, Package consumers should be able to install the new version on their cluster.
Describe the solution you'd like
Package consumers should be able to update the InstalledPackage CR to point to the newer version of the package in order to deploy it.
Anything else you would like to add:

In progress on the dev-packaging branch.


New refactoring

Mostly files from the XML pipeline
Added a call graph to understand module/function dependencies
Needs more work on really complex and long functions, documentation, simplification, etc

No changes in logic, probably did a couple of minor changes that might need checking, and as it's in a different directory there's no "takeover" as nothing actually changed.
The idea here is to make the code less complex, more Python-like and easy to understand. There are some really large functions that might need to be broken down and refactored. Maybe we can find efficiencies here and there.
As we planned, I can do more stuff after going over the code with you, then I can suggest larger changes.

For src/xml_processing/code/lib/get_pubmed_xml.py
basically I added all comments after the import to a DOCSTRING on the top of the file, just to organize.
Yes, the idea is that to change anything, I would need to understand what is going on, and I don't want to do anything without you checking first. Everything so far were style changes and some simplification of the code in different places that don't interfere with the original as they are in different dir.
And anything on crud will be documentation and minimal style changes, like consistency with double quotes instead with a mix double and single.


Issues Datachile Sprint @ Wed Dec 27, 2017

compare with https://github.com/Datawheel/datachile/issues/293 in case of duplicated todos

[x] Add 404 page
[x] Search R+D survey, check methodology and add note about "just private companies on this sample" or so
[x] Fix formats on treemaps (@palamago) (i.e exports by location and # of occupations by workers, check images below)
[x] Fix decimals vs thousands on treemaps (@palamago) (check images below)
[x] FIx diferent NA types on visas data
[x] Check Araucanía 2014 health access data
[x] Country profile (@frabarz): add "source note" component (right, @palamago?) + comment about + remove "array topics"  (again, right, @palamago?)
[x] Geo profile (@palamago): check images in sub continents or regions (i.e Africa, West)

This was checked before Jan, 8th


run agent occurred an error by prometheus

When i run agent, it failed ,the error message is:
# go run ./cmd/agent/main.go
github.com/uber/jaeger-lib/metrics/prometheus
../../uber/jaeger-lib/metrics/prometheus/factory.go:143:12: cannot use hv.WithLabelValues(f.tagsAsLabelValues(labelNames, tags)...) (type prometheus.Observer) as type prometheus.Histogram in field value:
	prometheus.Observer does not implement prometheus.Histogram (missing Collect method)

when i locate the code ,It is really an error:

didn't find other issue mentioned about this，Given that I do not really familiar with prometheus，I haven't find a way to resolve this problem

I can reproduce the problem about tls.Config. From the latest master (a4706c0d), these are my changes:
diff --git a/glide.lock b/glide.lock
index 15dfad6..972af16 100644
--- a/glide.lock
+++ b/glide.lock
@@ -60,7 +60,7 @@ imports:
 - name: github.com/go-openapi/validate
   version: a762d5dd38f7c7e9e1a13c3e98344cfbe4aa15d9
 - name: github.com/gocql/gocql
-  version: 4d2d1ac71932f7c4a6c7feb0d654462e4116c58b
+  version: 181004e14a3fb735efcc826a4256369d0c96747b
   subpackages:
   - internal/lru
   - internal/murmur
diff --git a/glide.yaml b/glide.yaml
index 687a535..613f869 100644
--- a/glide.yaml
+++ b/glide.yaml
@@ -5,7 +5,7 @@ import:
   subpackages:
   - lib/go/thrift
 - package: github.com/gocql/gocql
-  version: 4d2d1ac71932f7c4a6c7feb0d654462e4116c58b
+  version: 181004e14a3fb735efcc826a4256369d0c96747b
 - package: github.com/opentracing/opentracing-go
   subpackages:
   - ext

Then:
$ rm -rf vendor
$ glide install
$ make test

The last command yields:
# github.com/jaegertracing/jaeger/pkg/cassandra/config
/mnt/storage/jpkroehling/Projects/src/github.com/jaegertracing/jaeger/pkg/cassandra/config/config.go:136:10: cannot use tls.Config literal (type tls.Config) as type *tls.Config in field value

Probably because of incompatible upgrade of gocql driver. It's on our to do list to test an upgrade for perf impact

It's on our to do list

Is there a GitHub issue for this task? If it's only a matter of updating the gocql, I can probably do it.

Upgrading is the easy part, we need to deploy it and see the impact at scale. @black-adder claims he tried it once and saw perf drop

The fool I was I didn't dig into it, if you're willing to bump it and submit a PR, I can run it internally and make sure the metrics look fine.

@jmhon08 this could be the perfect first issue for you! Would you like to send a PR? I can help you with that.

Looks like this is not an issue anymore.


Unable to download sbt-native-packager plugin on Ubuntu with OpenJDK 9 headless (certificate issue)

Partly out of curiosity, was trying to build my sbt project using openjdk-9-jdk-headless on our CI machine. It fails with what seems to be a certificate issue.
I understand that once JDK 9 was still being developed, but it's been out for a while now, and the behaviour was surprising to me. I don't know if this is directly an sbt issue.
problem
The build stops with:
01:05:20[warn] [FAILED ] com.typesafe.sbt#sbt-native-packager;1.3.2!sbt-native-packager.jar: sbt-plugin-releases: unable to get resource for com.typesafe.sbt#sbt-native-packager;1.3.2: res=https://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/com.typesafe.sbt/sbt-native-packager/scala_2.12/sbt_1.0/1.3.2/jars/sbt-native-packager.jar: javax.net.ssl.SSLHandshakeException: sun.security.validator.ValidatorException: No trusted certificate found (176ms)

This seems to be an issue known to StackOverflow: link, but it seems weird now that JDK 9 has been officially out.
expectation
Updating the libraries and plugins should work, on both JDK 8 and JDK 9.
notes
sbt version: 1.0.4

Ah!  I was hitting this as well.  I was able to get sbt to work by upgrading the version specified in project/build.properties


Updating element discovery to fix loading on MacOS

I found that in the latest version of Stream Deck (on MacOs, at least), propertyInspectorDidAppear is invoked before onDocumentLoaded. This means the input elements are not initialised when the settings are loaded, so everything just comes up blank all the time.
I've also updated the logging from console.log (which wasn't giving me anything) over to this.logMessage, and removed sensitive information from the log output.
The refreshPlugin script also got a couple of MacOS specific updates: one for the pkill parameter (the application name has changed, looks like) and a slightly cleaner method of launching the app which is agnostic to its location.

Thanks for your contribution, I'll take a look at it later.

I am having the same problem. Great that you already prepared a fix!

I've updated the JS plugin: https://github.com/JetBrains/intellij-streamdeck-plugin/blob/main/releases/com.jetbrains.ide.streamDeckPlugin , please give it a try. It should be released next week on Stream Deck Store.


Add missed functionality

New:
* add ssl certKey;
* file upload;
Fixed:
* bind ssl vserver;

Too much changes which will brake backward compatibility. =(


Add ignore-tls flag

Hi,
it would be nice to have an ignore-tls flag or something similar to test against https with an self-signed certificate. Or is there some way to add the self-signed certificate to the trusted certs?

the flag is in #138 (as --https-insecure) with follow up in #141 for providing custom cert/ca


reset historical.nodeSelector default values to {}

hello guys, I think setting nodeSelector = {size: historical} as default value will make historical workload find no node matched the selector to run itself.
  nodeSelector:
    size: historical

we should set it {} by default.
It must be the crucial issue to make e2e test failed.

Oh wow, good catch, I missed this when I ported over the code. I missed it when I looked over the diff. Thanks!


Payment for #53003

https://github.com/Expensify/App/issues/53003
Issue OwnerCurrent Issue Owner: @Pujan92

@Kalydosos

Closing, original job worked.


Uncaught TypeError: Cannot read property 'value' of undefined at decorators.ts

Using monaco editor with https://github.com/abraham/reflection/ throws Uncaught TypeError: Cannot read property 'value' of undefined

Original issues:

https://github.com/abraham/reflection/issues/630
https://github.com/suren-atoyan/monaco-react/issues/237

@softmarshmallow I think the vscode decorator implementation is correct. Here is the TS documentation: https://www.typescriptlang.org/docs/handbook/decorators.html#method-decorators
Notice how the descriptor is not documented or typed to ever be undefined:

Since this NPE does not reproduce without abraham/reflection, I believe this is a problem with that package, which perhaps misses a case (or something).

@softmarshmallow I think the vscode decorator implementation is correct. Here is the TS documentation: https://www.typescriptlang.org/docs/handbook/decorators.html#method-decorators
Notice how the descriptor is not documented or typed to ever be undefined:

Since this NPE does not reproduce without abraham/reflection, I believe this is a problem with that package, which perhaps misses a case (or something).

I'll take another deep look, but reflection is not that a complex package and i don't see any malicious implementations.
I'm humbly requesting to check if the vscode side can allow descriptor to be null?
Thanks :)

I'm humbly requesting to check if the monaco side can allow descriptor to be null?
:+1: Yes, that's what I've done. I've looked at the code which has been slightly modified on main, but I can see that this particular descriptor called memoize is only ever used on methods in classes. I've then proceeded to read about their constraints and they should always be invoked by TypeScript with the descriptor argument.

This can also be seen in the generated JS. Here is an example of using that particular decorator in TypeScript. And here is the generated JS:
var __decorate = (this && this.__decorate) || function (decorators, target, key, desc) {
    var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
    if (typeof Reflect === "object" && typeof Reflect.decorate === "function") r = Reflect.decorate(decorators, target, key, desc);
    else for (var i = decorators.length - 1; i >= 0; i--) if (d = decorators[i]) r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
    return c > 3 && r && Object.defineProperty(target, key, r), r;
};
define(["require", "exports", "vs/base/common/decorators"], function (require, exports, decorators_1) {
    "use strict";
    Object.defineProperty(exports, "__esModule", { value: true });
    exports.parseLinkedText = exports.LinkedText = void 0;
    class LinkedText {
        constructor(nodes) {
            this.nodes = nodes;
        }
        toString() {
            return this.nodes.map(node => typeof node === 'string' ? node : node.label).join('');
        }
    }
    __decorate([
        decorators_1.memoize
    ], LinkedText.prototype, "toString", null);
    exports.LinkedText = LinkedText;
    //...
});

From my reading of that method (__decorate), it looks like desc is passed in as null, but then it gets assigned to desc = Object.getOwnPropertyDescriptor(target, key). So desc is always defined. If a global called Reflect is defined (which is what I guess this other package is doing), then decorating gets delegated to that global using Reflect.decorate(decorators, target, key, desc). IMHO at that point desc is always set. I don't know how this other npm package implements Reflect, but I would expect it should carry over the desc argument and pass it on to the decorator.

Ok, Thanks for the detailed explanations :)


Rope

The rope will be waving all the time at minVelocity.
Character can increase rope velocity when attached by pushVelocity or decrease in held the axis in the other direction.
Character can Jump out.
Character can fall down/dismount.
Character grab the last segment

[x] CharacterActionRope (CharacterAction)

hand offset
grab box

[x] Rope (Tiles)

minVelocity
maxVelocity
pushVelocity
maxAngle (angle will be doubled Range: 0-180)
Segments

Sprite
Length

[x] Climb up/down -> grab any segment

[x] Transition to the "grab" segment, during transition can't jump or dismount.

Future

[ ] Breakable ropes

Forgot animation

[ ] Play rope animation
[ ] use rope rotation (flag)


openlayers map client

make like for like openlayers map in react app

Working basemap, rough around the edges: https://github.com/bcgov/WetPaddlers/tree/conor-openlayers

Merged in: https://github.com/bcgov/WetPaddlers/pull/21


Investigate the default http connector

Currently the default http connector is Prebuilt: https://github.com/awslabs/smithy-rs/blob/bb5bbeb5cff2afb4b3664e95f5c63cae3a524379/aws/rust-runtime/aws-config/src/lib.rs#L463-L466
This means that timeout settings at a service level will be ignored. This should probably be ConnectorFn but the implications need to be considered.

Fix released in https://github.com/awslabs/aws-sdk-rust/releases/tag/release-2023-03-30.


Allow alternative frontends

I'm uncertain if this makes sense, but maximum flexibility would be to have the option to implement alternative frontends.
Requirements

Key bindings
Widgets

Table (for the file list)
Scrollbar
Text field (for the URL)
buttons
Labels
Menu (context menu)
Pop-up windows

Looking at https://www.pythonguis.com/faq/which-python-gui-library/, it seems that PyQt6 or PySide6.
https://www.pythonguis.com/faq/pyqt6-vs-pyside6/ : I would need to use GPL if I use PyQt6.

https://stackoverflow.com/questions/tagged/pyside : 3500 questions
https://stackoverflow.com/questions/tagged/pyqt : 17,000 questions

We could have a base class:
from abc import ABC, abstractmethod

class Frontend(ABC):
    def __init__(self, cfg):
        self.cfg = cfg
        self.width = None
        self.height = None
        self.x = None
        self.y = None

    def set_geometry(self, width: int, height: int, x=0, y=0) -> None:
        self.width = width
        self.height = height
        self.x = x
        self.y = y

    def set_title(self, title: str) -> None:
        self.title = title

    def bind(self, event, callback) -> None:
        self.root.bind(event, callback)

    @abstractmethod
    def run(self):
        pass

and classes implementing it:
from tkinter import Tk, Label
import tkinter as tk
from tkinter import messagebox, ttk
from tkinter.simpledialog import askstring
from flitz.frontends.base import Frontend
from flitz.config import Config
from pathlib import Path

class TkinterFrontend(Frontend):
    def __init__(self, cfg: Config):
        self.cfg = cfg

        self.root = Tk()
        self.set_geometry(width=self.cfg.window.width, height=self.cfg.window.height)
        self.root.configure(background=self.cfg.background_color)

        self.root.rowconfigure(0, weight=0, minsize=45)
        self.root.rowconfigure(1, weight=1)
        self.root.columnconfigure(0, weight=0, minsize=80, uniform="group1")
        self.root.columnconfigure(1, weight=85, uniform="group1")
        self.root.columnconfigure(2, weight=5, uniform="group1")

        self.style = ttk.Style()
        self.style.theme_use("clam")  # necessary to get the selection highlight
        self.style.configure(
            "Treeview.Heading",
            font=(self.cfg.font, self.cfg.font_size),
        )
        self.style.map(
            "Treeview",
            foreground=[
                ("selected", self.cfg.selection.text_color),
                (None, self.cfg.text_color),
            ],
            background=[
                # Adding `(None, self.cfg.background_color)` here makes the
                # selection not work anymore
                ("selected", self.cfg.selection.background_color),
            ],
            fieldbackground=self.cfg.background_color,
        )

        # Set window icon (you need to provide a suitable icon file)
        icon_path = str(Path(__file__).resolve().parent / "../icon.gif")
        img = tk.Image("photo", file=icon_path)
        self.root.tk.call("wm", "iconphoto", self.root._w, img)  # type: ignore[attr-defined]

    def set_geometry(self, width, height, x=0, y=0):
        self.width = width
        self.height = height
        self.x = x
        self.y = y
        self.root.geometry(f"{self.width}x{self.height}")

    def set_title(self, title: str) -> None:
        self.title = title
        self.root.title(self.title)

    def bind(self, event, callback) -> None:
        pass

    def run(self):
        label = Label(self.root, text="Tkinter GUI")
        label.pack()
        self.root.mainloop()

The problem is how to set the layout. The tk.Frame / pack concept of tkinter likely doesn't translate properly to other GUI frameworks

The next issue is that some front-end elements have direct connections to the application state which again should trigger other front-end components to change:

User enters path in URL bar -> current_path changes -> details view needs to be reloaded
User clicks on up-button -> current_path changes -> URL-bar + details view needs to be reloaded.

I've introduced a on_current_path_change to trigger all changes, but that currently is in the main application. I could pass a reference to the main application to the frontend... I'm not sure if that is a good idea.
Instead, I could introduce a "controller".


[WIP] Change dist server to run on port 4000

I think it makes sense to want to build while dev server is running(I do this periodically). Unfortunately, the dist and dev server run on the same port so this fails. This PR changes the port the dist server runs on so both can run at the same time.
Questions

Is this a desired change? I'm not sure if there was a reason for having them on the same port in the first place
Is port 4000 ok? I'm happy to change to something else.

Issues
Executing yarn start or npm start will remove the dist folder. This is not good if the dist server is running and its files are getting deleted. My solution is to not execute remove-dist when the dev server is started but that depends on why it currently happens.

Hi @fayimora. I see your point. Great idea. The only reason dist is being deleted is to remind people to run dist before committing. I'd be okay with eliminating dist removal during npm start.
@nickytonline @kwelch thoughts?

@fayimora @coryhouse changing the port seems fine and we can remove the deletion of the dist folder, but I don't generally run into this scenario since my CI would generate the build artifacts (dist).  Having said that, I don't see it harming anything since CI will still work (fresh install each time). @kwelch?

I am good with this all around. 👍

Great. @fayimora if you submit an updated PR that eliminates the remove-dist step I'll get it merged. Thanks!

@coryhouse done!

Looks great! Thanks so much @fayimora! 👍

My pleasure! 🙇


Broken scss variables import

Describe the bug
When importing  src/utils/variables.clarity.scss in Angular component styles in order to (re)use clarity variables one must also import grid variables, mixins and layers as they are unresolved dependencies of variables files.
@import<EMAIL_ADDRESS>@import<EMAIL_ADDRESS>@import<EMAIL_ADDRESS>@import<EMAIL_ADDRESS>
Omitting grid variables is causing

Omitting mixins is causing

While omitting layers causes

How to reproduce
Stackblitz repo.
Steps to reproduce the behavior:

Open the application and you will see the bug
Bug is caused by import in src/app/app.component.scss

Expected behavior
In order to use scss variables, importing src/utils/variables.clarity.scss should be enough. Variables file should not have unresolved dependencies.
Versions
App

Angular: 8.0.0
Node: 12.3.1
Clarity: 2.0.1

Device:

Type: Dell XPS
OS: Ubuntu
Browser Chrome
Version 74

Additional notes
This seems to be randomly changing from version to version. It was working in v1. Related to https://github.com/vmware/clarity/issues/2442

@meeroslav
What is the intended end result? Is this an attempt to theme the clarity components or are you trying to reuse the clarity variables in your own SASS files without using the rest of the library?

Just wanted to bring my own comment on the same topic https://github.com/vmware/clarity/issues/2442#issuecomment-501149800
I also have an open ticket https://github.com/vmware/clarity/issues/3444
I still owe you an answer @mathisscott. The reason I have not commented yet is that I was awaiting on answer from my frontend dev for all of the reasoning. It wasn't a blocker since we were actually able to stick with the 1.x branch but now it turns out we need Angular Universal ... which for Node.JS 12 needs Angular 8. Which means we need the 2.x branch.
Anyway ... I'd appreciate any work to make 2.x variables as flexible (or more) as 1.x.

@Rush
Thanks for circling back to us on this.
#3444 is a different issue from this one. I'd prefer to keep the issues relevant to their individual topics so as to not engage in scope creep or conflating seperate bug reports or feature requests into one issue.
I'll comment there on the state of maps and variables in the SCSS.
For this issue (the addition of a dependencies.scss file so people can import individual clarity SCSS files more easily), I'm still unclear on the end goal.
Is the intent to reuse Clarity variables in a product's SCSS?
Is the intent to modify Clarity variables in a manner we would consider theming?
Once I understand what @meeroslav is trying to accomplish, I can try to diagnose either a workaround or what a convenience dependencies import would need to pull in.

Sorry @mathisscott, I somehow missed your reply.
My intent is to reuse the clarity variables in my custom angular component's SASS files to maintain the same visual style.

As we stabilize and lock Clarity Angular's features, this shouldn't be an issue in the future. Files did move from time to time, which could have broken import chains. With Clarity Core, we don't anticipate this issue because theming should resolve the need to build custom versions of CSS. I'm going to close this as stale and not something that should continue to affect folks in future updates.

As we stabilize and lock Clarity Angular's features, this shouldn't be an issue in the future. Files did move from time to time, which could have broken import chains. With Clarity Core, we don't anticipate this issue because theming should resolve the need to build custom versions of CSS. I'm going to close this as stale and not something that should continue to affect folks in future updates.


Module cannot be initialized

Hey,
I've found an issue where after addition of packages.config with just Cake.BuildSystems.Module to ./tools/Modules is breaking the Cake build process.
packages.config
<?xml version="1.0" encoding="utf-8"?>
<packages>
    <package id="Cake.BuildSystems.Module" version="0.3.0" />
</packages>

Error with Diagnostic output:
Preparing to run build script...
Running build script...
Verifying assembly 'Cake.MyGet.Module, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'.
Verifying assembly 'Cake.TeamCity.Module, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'.
Verifying assembly 'Cake.TFBuild.Module, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'.
Verifying assembly 'Cake.TravisCI.Module, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'.
Verifying assembly 'Cake.MyGet.Module, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'.
Verifying assembly 'Cake.TeamCity.Module, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'.
Verifying assembly 'Cake.TFBuild.Module, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'.
Verifying assembly 'Cake.TravisCI.Module, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'.
Registering module Cake.MyGet.Module.MyGetModule...
Registering module Cake.TeamCity.Module.TeamCityModule...
Registering module Cake.TFBuild.Module.TFBuildModule...
Registering module Cake.TravisCI.Module.TravisCIModule...
Registering module Cake.MyGet.Module.MyGetModule...
Registering module Cake.TeamCity.Module.TeamCityModule...
Registering module Cake.TFBuild.Module.TFBuildModule...
Registering module Cake.TravisCI.Module.TravisCIModule...
Error: System.TypeLoadException: Method 'RunTargetAsync' in type 'Cake.MyGet.Module.MyGetEngine' from assembly 'Cake.MyGet.Module, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' does not have an implementation.
   at Cake.MyGet.Module.MyGetModule.Register(ICakeContainerRegistrar registrar)
   at Cake.Composition.ModuleLoader.LoadModules(IContainer container, CakeOptions options)
   at Cake.Program.Main()

The problem with module exists in Cake 0.29.0 only. The module is working fine with Cake 0.28.1 or older.

I think that's because in 0.29, they changed the interface ICakeEngine (see https://github.com/cake-build/cake/issues/2094)

@SheGe : As @flcdrg correctly pointed out, that was a change introduced in a later version of Cake than the module was built for. If you upgrade to Cake 0.30.0 and the module to 0.3.1 you should not get this error anymore.

@agc93 thanks, after updating to Cake 0.30.0 and module to 0.3.1 everything is working as expected.


[release-4.6] Install CRI-O from OBS

Use published packages built on OBS instead of Fedora's bodhi and
modularity.
Fixes https://github.com/openshift/okd/issues/463

/override ci/prow/e2e-aws-upgrade
stableinstall fails, would be fixed by this PR


Added mistral provider documentation

Summary

Added initial documentation of mistral ai documentation

Curious why we chose to refer to Mistral AI as Mistral AI Platform. Did you find that that was the way they typically refer to themselves?

Curious why we chose to refer to Mistral AI as Mistral AI Platform. Did you find that that was the way they typically refer to themselves?

Curious why we chose to refer to Mistral AI as Mistral AI Platform. Did you find that that was the way they typically refer to themselves?

Curious why we chose to refer to Mistral AI as Mistral AI Platform. Did you find that that was the way they typically refer to themselves?

No reason actually. I think it sounds better without Platform.


feat(sms): support for templates for the sms message

Before submitting this PR:
Checklist

[x] No breaking changes
[x] Tests pass
[x] New features have new tests
[x] Documentation is updated

Breaking changes
Avoid breaking changes and regressions. If you feel it is unavoidable, make it explicit in your PR comment so we can review it and see how to handle it.
Tests

please make sure your changes pass the current tests (Use the make test or the make watch command).
if you are introducing a new feature, please write as much tests as possible.

Documentation
Please make sure the documentation is updated accordingly, in particular:

Workflows. Workflows are Mermaid sequence diagrams
Schema. The schema in a Mermaid ER diagram
Environment variables. Please adjust the .env.example file accordingly
OpenApi specifications. We are using inline JSDoc annotations

Conventional commits
Versioning and changelog are generated following conventional commits. Please do your best to follow the convention, but don't worry about it too much. We will be most likely moving away to changesets soon.
Closes #196

@elitan can you check this?

@elitan @plmercereau any ideas when this could be reviewed, merged and shipped?

@elitan @plmercereau any ideas when this could be reviewed, merged and shipped?

We are needing this PR merged too, we cannot send SMS in english.

@elitan Can we prioritize merging this?

Sorry for the delay, I will review this PR next week. I've put time in my calendar for it.


Fix dep warning and prune unneeded code

Fix dep warning due to "github.com/golang/protobuf" is not
direct dependencies.
And pruning is now performed automatically by dep ensure.
So update the vendor/ and prune unneeded code.
fixes: #127

LGTM!

Hi @laijs - could you add that Fixes #127 to one of the commits to keep checkcommits happy?
That's an impressive reduction in code btw! ;)

done

LGTM

lgtm


add hydrogen suppression option

I added a small adjustment to the substrucDepict ipython notebook that lets you chose if you want all of the hydrogens depicted. The molecules look a little crazy with the hydrogens, but it makes it easier to check smirk strings where you want to tag a hydrogen.

Thanks!


save_path error while using API[Bug][windows]

Issue
Api giving save_path error even though destination path is mentioned
Code
from spleeter.separator import Separator

separator = Separator('spleeter:2stems')
separator.separate_to_file('./song.wav',
                           destination='./output')

Error Log
Traceback (most recent call last):
File "D:/Work/Python/test/extract.py", line 4, in
separator.separate_to_file('D:/Work/Python/test/song.wav',
File "D:\Work\conda_env\music\lib\site-packages\spleeter\separator.py", line 378, in separate_to_file
sources = self.separate(waveform, audio_descriptor)
File "D:\Work\conda_env\music\lib\site-packages\spleeter\separator.py", line 321, in separate
return self._separate_librosa(waveform, audio_descriptor)
File "D:\Work\conda_env\music\lib\site-packages\spleeter\separator.py", line 265, in _separate_librosa
sess = self._get_session()
File "D:\Work\conda_env\music\lib\site-packages\spleeter\separator.py", line 240, in _get_session
saver.restore(self._session, latest_checkpoint)
File "D:\Work\conda_env\music\lib\site-packages\tensorflow\python\training\saver.py", line 1291, in restore
raise ValueError("Can't load save_path when it is None.")
ValueError: Can't load save_path when it is None.

Hi @Coder-Pranav,
this does not seem to be linked to the destination path, but to the path where the model is supposed to be stored.
The default path where  the model are stored is ./pretrained_models (otherwise, you can set it with the MODEL_PATH env variable).
Can you check that the model was correctly downloaded and that you indeed have a ./pretrained_models/2stems path with some files in it?
If the model wasn't downloaded properly, you can try to download it manually from https://github.com/deezer/spleeter/releases/download/v1.4.0/2stems.tar.gz and put it in ./pretrained_models/.
I was not able to reproduce you bug on Linux, so it could be a Windows specific issue (that may be linked to the latest release). What is the spleeter version you use?

Yes thank you, you are right it was the issue with the model.


feat(build): arm64 support

PR adds multi-arch / arm64 support to the linkerd dev tooling stack.

Notes
There are a few stages that should be FROM apt-base as {stage name} but have been changed to FROM docker.io/library/rust:alpine as {stage name} (effectively). This is because the upstream sources these stages are pulling binaries from do not supply arm64 binaries. Luckily though, they're all rust codebases, so cargo install --git='repo url' --tag='existing version tag argument' takes care of the issue as a simple proof of concept.
The repositories in question are:

j5j
hokay
cargo-action-fmt
cargo-deny

There is an open (but as-of-yet-unmerged) PR to add arm64 builds to cargo-deny's releases. PR #659 to cargo-deny was merged 2024-05-03.
The other three repos (j5j, hokay, and cargo-action-fmt) all belong to @olix0r and so hopefully won't be as much of an ordeal to add arm64 support to 😅.

IMPORTANT: while this PR should be entirely safe to merge without the altered stages being restored to their original FROM apt-base as {stage name} forms, they do impose a non-zero but realistically negligible amount of overhead to build times. If the decision is made to have the three repos mentioned above supply multi-arch binaries of their own, the only real changes to the Dockerfile that should persist should be the additions of the sed expressions that take care of formatting upstream urls to match the platform of the build target.

Screenshots

Closing this to get it off my list of open PRs, and because it doesn't seem like there's any interest in it outside of @kflynn 🫤. Please feel free to re-open it at your discretion though 🙂.


apache check failed, https://whimsy.apache.org/pods/project/dubbo

It uses curl curl http://dubbo.incubator.apache.org, and gets the html text, then checks it.
But now the html text is just a javascript code, so all check failed. We need to make sure the text right.

The site has a redirect to specific language(zh-cn or en-us), can it change to curl http://dubbo.incubator.apache.org/zh-cn/ or curl http://dubbo.incubator.apache.org/en-us/?

Maybe can not change. If not, what can we do to solve this problem?

add check rules in redirect.ejs


Move turbo cartocss

cc @rochoa

I'm missing the turbo-cartocss replacement for named maps, apart from that ... LGTM! :+1:

Finally, integration of turbo-cartocss for named maps is done. It took more time than expected, I had to dedicate several hours to understand how templates, named-layers and named-maps work behind the scenes.
:tired_face:

:shipit:


input :valid & :invalid pseudo-classes not being set appropriately

Expected Behaviour
The CSS3 :valid and :invalid pseudo-classes are applied correctly when using Angular in-built form validation. CSS rules that apply to these elements work out of the box for angular forms that are using .ng-valid and .ng-invalid. I would expect angular-schema-form to manage these states correctly too.
Actual Behaviour
When using input validation with angular-schema-form, inputs appear to always be :valid, even if a field is invalid.
To demonstrate, see the basic form on schemaform.io.

Cause the validation on any of the fields to fail.
Call $('input:valid') in the developer console.
Observe that the invalid (with .ng-invalid) is still returned.

Unfortunately this means that CSS frameworks that require :valid, :invalid states are not natively supported by angular-schema-form.
I am happy to tackle this issue if someone can point me in the right direction.
Thanks!

@oskanberg thanks for the example. I'd love some help in finding out what angular does to handle it.

I'm afraid I am not familiar with how Angular sets this property. I have had a quick dig in the source to find it, but it's a difficult topic to search for.
I know there is a HTML5-spec API for setting form validity that is probably what we need, but would need a polyfill for older browsers.

This is what is required as linked above
element.setCustomValidity('I make it invalid')
element.setCustomValidity('') // I make it valid
Still not sure why Angular isn't handling it with ASF as Angular has a wrapper for this:
$setValidity(validationErrorKey, isValid);