PDModal CloseOnEscape="false" should not close the form when the ESC key is pressed

PDModal CloseOnEscape="false" should not close the form when the ESC key is pressed.
Currently it does e.g. I have this modal below. Pressing the ESC key closes it, but should not:
<PDModal @ref="EditDialog"
CloseOnEscape="false"
HideOnBackgroundClick="false"
Size="ModalSizes.Large">

Version 3.1.47.


UML feature bug fixes

Changes made:

Update UML connection to account for rotation, resizing, and unselecting elements. Note that the connection formed on a rotated UML element still has some quirks at certain rotation angles (the line might be a straight line instead)
Update unit test cases to remove some assertions

I think that I will work on a better solution (such as updating the connection anchor point as the UML element rotates) after working on other more important features (eg. deleting of connection lines).

Getting a huge delay between rotating/moving UML elements and the UML connections forming. Is this intended?

We need to comment out assert(checkRepresentation()) to avoid this delay!

Getting a huge delay between rotating/moving UML elements and the UML connections forming. Is this intended?

We need to comment out assert(checkRepresentation()) in Graph.swift to avoid this delay!

Ah ok, thanks!


What algorithm used for secret

What is the specific algorithm used for az ad sp create for the generated password that is returned? How is it attempting randomization and obscurity?

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

ID: e99d25ff-03e2-e015-37f4-a29b59ec50d4
Version Independent ID: 1a8e88ed-72c7-fbe7-b0ad-68f23b3946cd
Content: az ad sp
Content Source: src/azure-cli/azure/cli/command_modules/role/_help.py
Service: active-directory
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

Looks like it is just using a new uuid
https://github.com/Azure/azure-cli/blob/f8aeed58c3af83a2ab8ce57bf060fab29088e81b/src/azure-cli/azure/cli/command_modules/role/custom.py#L1197

Thanks for finding, seems all are using whatever uuid package comes in as some connected dependency as I don’t see any explicit demand in setup
https://github.com/Azure/azure-cli/blob/c83710e4176cf598fccd57180263a4c5b0fc561e/src/azure-cli-core/azure/cli/core/commands/arm.py#L1096


M2 does not compile in 1.7 due to 1.8 annotation

Build fails in 1.7 due to @Repeatable annotation on KafkaListener but documentation states it is 1.7 compatible.
:spring-kafka:compileJava
C:\Projects\spring-kafka\spring-kafka\src\main\java\org\springframework\kafka\annotation\KafkaListener.java:21: error: cannot find symbol
import java.lang.annotation.Repeatable;
^
symbol:   class Repeatable
location: package java.lang.annotation
C:\Projects\spring-kafka\spring-kafka\src\main\java\org\springframework\kafka\annotation\KafkaListener.java:85: error: cannot find symbol
@Repeatable(KafkaListeners.class)
^
symbol: class Repeatable
2 errors
:spring-kafka:compileJava FAILED
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':spring-kafka:compileJava'.

Compilation failed; see the compiler error output for details.

Using this jdk
C:\Projects\spring-kafka>java -version
java version "1.7.0_79"
Java(TM) SE Runtime Environment (build 1.7.0_79-b15)
Java HotSpot(TM) Client VM (build 24.79-b02, mixed mode, sharing)

Hello, @backroadssoftware !
Thank you for feedback!
That's correct.
You have to use Java 8 to compile project.
But from other hand you don't need Java 8 to run applications with spring-kafka.
Unknown annotations are ignored by Java.

Thanks for the info.


Return Riak type (EE/OSS) in /explore/clusters.

Adds the "riak_type" key to the cluster info list.
Checks to see if the cluster is Riak EE by seeing if the
riak_repl_console module is available.
Fixes #93 (in part).  Does not add riak_version as indicated in discussion in #93.

This is the sample output, from Riak EE:
$ curl http://localhost:9000/explore/clusters/default
{"cluster":{"id":"default","riak_node":"riak@<IP_ADDRESS>","development_mode":true,"riak_type":"ee"},"links":{"self":"/explore/clusters/default"}}

From Riak OSS:
$ curl http://localhost:9000/explore/clusters
{"clusters":[{"id":"default","riak_node":"riak@<IP_ADDRESS>","development_mode":true,"riak_type":"oss"}],"links":{"self":"/explore/clusters"}}

Tested out the branch, works great. Thanks Travis! +1 from me.


Date on daily notes

Hello, I'm super new to obsidian and just watched your videos on YT on repeat so I downloaded your vault. Thank you so much! One thing i found on the Daily Notes is that the written date is not automatic, nor is showing the time. Do you know what could be causing it?

Check if your daily note template has been altered accidentally
Make sure your daily note date format follows YYYYMMDD


Some fixes for language checks and additions to Readme

There was an error if a language was specified via e.g. class="lang-JSON", but unsupported by highlight.js (in this case because JSON was written in capital letters, npm module "marked" produces code like this). I added a check that will prevent this by lower-casing the attribute and then checking against highlight.js' listLanguages().
I added some more explanation and examples to the Readme.
I also specified version numbers for dependencies, in case a dependency updates and breaks this package.
Furthermore, the --o / output parameter allows you to write into the same input as output file.

Looks good, thanks!


Change documentation references from topic to topicArn

Description
Minor adjustment to the README to reflect the correct configuration of plugin.
This is needed because topic is not a valid property. It has to be topicArn.
 Configuration error at 'custom.slicWatch': unrecognized property 'topic'

Thanks for finding this and resolving it, @asionek 🙏


Airlock resources - tf scripts

PR for issue
What is being addressed
Adding the relevant resources to support the implementation of the Airlock feature.
How is this addressed
The resources creation are split into two groups:

resources under the TRE core group
resources per workspace

Group 1 - Within the TRE core:

Storage accounts: a) external-import b) external-export c) in-progress-import c) reject-import
The common properties for the above SAs is that they are either with approved data to be exported or still data which was not imported
Event grid topics (and system topics): a) update status b) status changed c) blob created topics for each SA except for the two storage accounts which are the drop location
Event grid subscriptions : between the sb queues and topics
service bus with different queues as the destination for the events from the event grid

Group 2 - per workspace

storage accounts: a) accepted-import b) internal-export (drop point for exporting) c) inprogress-export d) rejected-export
Event grid topics (and system topics): a) accepted-import blob created b) inprogress-export blob created c) rejected-export blob created
event grid subscriptions: a) accepted-import b) inprogress-export c) rejected-export

Notes:

networking was not configured yet. UPDATE: Done
storage accounts which should be protected by the ws networking are created with the ws

Screenshots:
Core resources:

WS resources:

Hi reviewers. PR is ready for review.
I think the best way to review it, is to a) deploy TRE and look at the generated resource b) create a workspace and look at the resources

/test

/test-extended

/test

/test-destroy-env

/test

/test

/test

/test-destroy-env

/test

/test

/test-extended

/test

/test-extended

/test

/test-extended

/test-extended

/test-extended

/test-extended

/test-extended

Fixes #1936


StringView methods that are good to have

substring(int beginIndex)
parseLong(int radix = 10)
parseInt(int radix = 10)

0.60.1


Add a Location tab to the Info pane

Add a Location tab that displays relevant information about the current reticule (crosshairs) location and the last queried (right-click) location.
Candidates:

[x] Lat/Lon and other coordinate systems
[x] Elevation (ft and meters)
[ ] Administrative places
[ ] Sunrise/Sunset

Added a Location tab/panel to the Info view that displays the location of the crosshairs (reticule), the eye/camera location, and terrain data at the crosshairs.  Coordinates, altitudes/elevations and terrain slope are displayed in multiple units of measure.


Course Groups

When we want to know how many times someone has run Hardrock, or what's the overall course record (odd or even year) for the Rattlesnake Ramble, we have no good way of computing that information, because OST does not know that the Hardrock Clockwise and CCW courses are related, or that the Ramble Even and Odd Year courses are related.
We should have a data structure that allows us to relate multiple courses in a group, similar to the way events are related in groups and series.
Which is the better analogy?
Events belong to event groups, meaning that every event has an event group, and an event group has one or more events. However events have a one-to-many relationship with event series, meaning that an event may be a member of zero or more event series, and an event series consists of zero or more (one or more in practical use) events.
A third option would be for a course to belong to a course group optionally, meaning that only certain courses would be grouped.
I think the event series analogy (with the many-to-many relationship) is best. This is the only way a course could belong to more than one course group. I can already think of at least one use case for this, which would be to show best efforts for a single course (Hardrock CCW), a close comparison group (Hardrock CCW and Hardrock CW), and a broad historical group (Hardrock CCW, Hardrock CW, and the original 1992/1993 courses).
What's the best name for this concept?
I'm immediately drawn to course_group or course_series. Not sure if either of these is right though.

This is done!


Move from Waitlist concept in DeepMemory access

🚀 🚀 Pull Request
Impact

[ ] Bug fix (non-breaking change which fixes expected existing functionality)
[ ] Enhancement/New feature (adds functionality without impacting existing logic)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
This PR will remove the references to Waitlist when checking the access to DeepMemory.
Currently, DeepMemory access will be automatically granted to anyone on a Paid Org plan. (Similar to now previously query access was granted)
Please note that some test cases might need to be adjusted accordingly.

Things to be aware of

Things to worry about

Additional Context

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


[HUDI-1806] Honoring skipROSuffix in spark ds

What is the purpose of the pull request
*Honoring skipROSuffix in spark ds
Brief change log

Honoring skipROSuffix in spark ds

Verify this pull request
This change added tests and can be verified as follows:

Tested via HoodieSparkSqlWriterSuite.Test build sync config for spark sql

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Codecov Report

Merging #2882 (7999033) into master (3e4fa17) will increase coverage by 8.77%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #2882      +/-   ##
============================================
+ Coverage     53.06%   61.84%   +8.77%
+ Complexity     3746      335    -3411
============================================
  Files           488       54     -434
  Lines         23466     1997   -21469
  Branches       2503      236    -2267
============================================
- Hits          12453     1235   -11218
+ Misses         9914      640    -9274
+ Partials       1099      122     -977

Flag
Coverage Δ
Complexity Δ

hudicli
?
?

hudiclient
?
?

hudicommon
?
?

hudiflink
?
?

hudihadoopmr
?
?

hudisparkdatasource
?
?

hudisync
?
?

huditimelineservice
?
?

hudiutilities
61.84% <ø> (-7.90%)
335.00 <ø> (-39.00)

Flags with carried forward coverage won't be shown. Click here to find out more.


webapp: Fix the az webapp ssh test that's failing on recording

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior.
Expected behavior
A clear and concise description of what you expected to happen.
Environment summary
Install Method (e.g. pip, interactive script, apt-get, Docker, MSI, edge build) / CLI version (az --version) / OS version / Shell Type (e.g. bash, cmd.exe, Bash on Windows)
Additional context
Add any other context about the problem here.

This test is blocking the PR https://github.com/Azure/azure-cli/pull/8886 that is blocking the other  pending PRs to be merged as well.

The test passed on mac - @patricklee2 not sure why you were not able to record this successfully


Nightscout CGM Import Settings fails with error

Describe the bug
I know Nightscout is not recommended for CGM but I thought I would test it. Trio connects fine to my Nightscout instance but Nightscout Config - Import Settings fails with the following error:
"The data couldn't be read because it isn't in the correct format."
To Reproduce
Steps to reproduce the behavior:

Go to Settings
Click on CGM
Scroll down to Nightscout under Type
After entering URL and Secret, confirm Connect
Tap on Import Settings, accept prompt to allow overwrite of existing settings
See error

Expected behavior
Nightscout settings should be imported.
Screenshots
If applicable, add screenshots to help explain your problem.
Smartphone (please complete the following information):
iPhone SE 3rd Generation iOS 17.4.1
Setup Information (please complete the following information):

Pump type: none yet, plan to use OmniPod Dash
CGM type and CGM app: Nightscout 15.0.2
Trio version, branch and git reference (see Trio Settings): dev 5faf162 Trio v0.1.0

Additional context
Unable to provide screenshot as the Nightscout CGM cannot be deleted and re-added.  Configuring Nightscout seems to be a one-time only process.  It would be nice to have a Delete option to allow a re-configuration of Nightscout CGM.

Does your selection of mg/dl vs mmol match your NS configuration?
My hunch is perhaps your Trio preferences still has mmol set (default starting point) and your NS is using mg/dl.
If this is the case, the fix is to set your preferences in Trio to match NS. It would also indicate that the error message in Trio isn’t clear enough.

Does your selection of mg/dl vs mmol match your NS configuration?

Yes, Preferences - Glucose Units is mg/dL which matches my NS configuration.  I retried again after following your instructions to Delete the Nightscout service and then re-added it, connected, then Import Settings.  It still shows:
Import Error
The data couldn't be read because it isn't in the correct format.

Very strange. I have trouble locating that particular or similar Import error text string  in the Trio code ("The data couldn't be read because it isn't in the correct format.").
A screenshot of the Import error alert would be of great help for further troubleshooting so I can see the exact phrasing.
And also, you wrote "Pump type: none yet, plan to use OmniPod Dash". Do you run a pump simulator? (The reason im asking is that there needs to be a pump simulator or actual pump set up for the import settings to work, since basal settings are written to the pump. If that fails, it should however show another import error message.

Do you run a pump simulator?

Yes, I am using the pump simulator.  I just ran Trio in the Simulator on my Mac, and it is failing in the code above.

Have you used this Nightscout URL with iAPS or Loop before?

And one more thing. You do not need to delete nightscout to be able to do a new import. When it works as it should, the "import setting" can be run anytime you like (if you for instance make changes to the CR/ISF/Basals In nightscout, it can be imported by Trio. However very important to always double check that its imported as expected as stated in the alert

Have you used this Nightscout URL with iAPS or Loop before?

Yes, I am using DIY Loop for several years with my Nightscout URL.
Here is another screenshot of that same file for Import Settings:

A screenshot of the Import error alert would be of great help for further troubleshooting so I can see the exact phrasing.

This is from my iPhone SE

Ok. I think Mike can answer this better, but the alert is a default alert for parsing errors. I see that now. And as stated in the alert and in the log it's a formatting mismatch (maybe because previously saved entries from loop doesn't fit 1:1 with formatting in Trio)

It errors out right after it attempts to decode the JSON.  Here is the parsingError from the above catch statement:
Printing description of parsingError:
▿ DecodingError
▿ typeMismatch : 2 elements
- .0 : __C.NSDecimal
▿ .1 : Context
▿ codingPath : 2 elements
▿ 0 : _JSONKey(stringValue: "Index 0", intValue: 0)
▿ rep : Rep
- index : 0
- 1 : CodingKeys(stringValue: "mills", intValue: nil)
- debugDescription : ""
- underlyingError : nil

And as stated in the alert and in the log it's a formatting mismatch

I guess that means I cannot import settings from Nightscout then?  I just wanted you to know this was occurring and may impact others.  Thank you so much for your help!!!

And as stated in the alert and in the log it's a formatting mismatch

I guess that means I cannot import settings from Nightscout then?  I just wanted you to know this was occurring and may impact others.  Thank you so much for your help!!!

We need to dig deeper in what could or could not be done to fix this.
And as you say, if this affects all users coming from Loop to Trio it is very good to know (and communicate in onboarding documentation) that it's a known limitation with the import function. That you need to have made a first upload of settings to NS from Trio (or previously iAPS) for the import function to work correctly.
But I don't know for sure. We need to look into this more.

I downloaded the profile.json file from Nightscout and jsonlint.com shows that it is valid.  There are several elements with "mills" and they are strings not integers or Decimal (as defined in RawFetchedProfile.swift below).  Could this be the issue?
{
    "_id": "6655fb828f969503514dfdef",
    "defaultProfile": "Default",
    **"mills": "1716910977635",**
    "enteredBy": "Loop",
    "startDate": "2024-05-28T15:42:57Z",,

NightscoutConfigStateModel.swift uses:
let fetchedProfileStore = try jsonDecoder.decode([FetchedNightscoutProfileStore].self, from: data)
Defined in RawFetchedProfile.swift:
struct FetchedNightscoutProfileStore: JSON {
let _id: String
let defaultProfile: String
let startDate: String
let mills: Decimal

Ok, good catch.
I jumped into my own MongoDB library and extracted an (really) old Loop profile file, and a fresh iAPS/Trio profile file and run it through GPT4o to get help in pointing out any differences in fields and formatting, and there are a few mainly regarding number formatting, timezone and units. See attached chat answer if you like.
And as input to further investigations between devs in how to handle the user migration from loop->Trio scenarios in the best way going forward.
https://chatgpt.com/share/99f6aaea-dce3-48d0-8a0b-d1e371a1d782

After I changed mills to a String in RawFetchedProfile.swift, "Import Settings" got further but errored out on:
"carbs_hr" which is a String on Nightscout, but Int in FetchedNightscoutProfileStore,
"Store" which is JSON on Nightscout and String: ScheduledNightscoutProfile on FetchedNightscoutProfileStore.
"delay" is also a String in Nightscout
"created_at" does not exist in Nightscout
.store["default"] does not exist in Nightscout, it is .store["Default"] //uppercase Default
Swift.DecodingError.Context(codingPath: [_JSONKey(stringValue: "Index 0", intValue: 0), CodingKeys(stringValue: "store", intValue: nil), _JSONKey(stringValue: "Default", intValue: nil), CodingKeys(stringValue: "carbs_hr", intValue: nil)], debugDescription: "Expected to decode Int but found a string instead.", underlyingError: nil))
keyNotFound(CodingKeys(stringValue: "created_at", intValue: nil), Swift.DecodingError.Context(codingPath: [_JSONKey(stringValue: "Index 0", intValue: 0)], debugDescription: "No value associated with key CodingKeys(stringValue: "created_at", intValue: nil) ("created_at").", underlyingError: nil))
profile.json:
"dia": 6,
"carbs_hr": "0",
"delay": "0"
    "mills": "1716742417215",
    "units": "mg/dL",
    **"store": {**
        **"Default": {**
            "target_low": [
                {
                    "time": "00:00",
                    "timeAsSeconds": 0,

Changing these in RawFetchedProfile.swift and NightscoutConfigStateModel.swift to match Nightscout resolves the "Import Settings" issue for "enteredBy": "Loop" Nightscout profiles.

iAPS/Trio profile file and run it through GPT4o to get help in pointing out any differences

That is a great idea to use chatGPT for the diff!!  It matched basically what I found in testing.

Changing these in RawFetchedProfile.swift and NightscoutConfigStateModel.swift to match Nightscout resolves the "Import Settings" issue for "enteredBy": "Loop" Nightscout profiles.

One idea could be to make a conditional check for enteredBy: Trio or Loop (or not Trio) when importing and adjust the formatting accordingly. Loop->Trio isn't however the only user migration case. The previous NS profile entries could of course have been made by AAPS or previous iAPS, and there maybe isn't possible to find a proper way to handle all scenarios.
But this issue is registered and lives on this board, and some of our devs can look into this further as soon as the higher prioritized issues right now are resolved.

The previous NS profile entries could of course have been made by AAPS or previous iAPS, and there maybe isn't possible to find a proper way to handle all scenarios.

Yes, and it could be an old version of Nightscout with totally different data, but this scenario probably isn't too likely.  Perhaps we could just re-try the JSON decoder on catch with the Loop data types?  I'm not sure what format AAPS uses but I think it may be the same as Loop?
Yes, I can look into making a PR draft for this issue.

That’s great @kskandis. You should be able to find users here with examples of both Loop and AAPS data in their NS databases.


Integrating Log level / verbose ?

Hello folks,
as the homebridge logfile is the biggest one on my raspberry I‘m asking if it is possible to integrate something like a log level definition or verbose.
Or is there anything like that to tell homebridge how to log and to minimize log file size and with that the SD card wear ?
Thanks, Chris

this is not a direct answer, but if you look "big picture", you can try to set up overlay FS, I followed this link and it is working great. after that your SD card will be mounted in read only mode.
https://www.domoticz.com/wiki/Setting_up_overlayFS_on_Raspberry_Pi

Along these lines I think a better logging library that supports rolling log files / size limits would be helpful.   Homebridge has filled my router's drives with logging due to runaway repeating failures a couple times.


Documentation not accessible

The documentation on the Homepage leads to 404-Error
e.g. http://www.shogun-toolbox.org/doc/en/current/installation.html

@vigsterkr any clues? I thought we had fixed this

@karlnapf ok so the problem is that at the time of 4.1.0 release there was a bug in doxygen config, hence it is 404. as you can see in case of latest it's fixed:
http://www.shogun-toolbox.org/doc/en/latest/installation.html

@karlnapf should we maybe port back that fix into current master?

Ah yes, thanks for explaining. I think it might be a good idea to at least fix the docs online.
On the other hand, we can also just release ASAP and the 4.1 gets outdated anyways?
What do you think is better?

i'd say let's concentrate on the new release... :)

Ok so this will be fixed once we release


redirect failed on windows7

This issue is similar to issue 26. Driver file operation error. DeviceIoControl failed (The device is not connected. Error code = 1167) Redirect of USB device failed

usbdk version is 1.0.22

Is it similar to https://github.com/daynix/UsbDk/issues/103?

no, it is similar to #26

Please collect the log as explained in https://github.com/daynix/UsbDk/issues/103
Upload the log and etl file (to be on the safe side)

Please collect the log as explained in #103 Upload the log and etl file (to be on the safe side)
usb_log.zip

Please upload the etl (the log is not parsed correctly, probably due to win7)

usb_log-20211214.zip
The log has been uploaded, but now I cannot reproduce the problem.

@xiangjiang-cn In the log I do not see any problem, but also I do not see any attempt to redirect USB device.

20220111.zip
20220111-1.zip


Disabling "Let websites provide locally relevant content" is not working

The corresponding key and value are:
"HKEY_CURRENT_USER\Control Panel\International\User Profile" /v "HttpAcceptLanguageOptOut"
which are featured in the Standard xml in this repo.
But when using the xml to auto-install Windows and later checking Settings > Privacy > General, I see "Let websites provide locally relevant content" is actually enabled and not disabled. I need to manually disable it. All other registry settings as far as I remember were respected, though.
Also, by manually browsing the registry, I noticed the key and value were not there at all, as if never created (or erased...)
Tried with the Standard xml from here, and also with a version generated by Schneegans service and modifying it, which uses the approximation of first loading the NTUSER.DAT and then using "HKU\DefaultUser" instead of "HKEY_CURRENT_USER".
Same results.
Tried both in Windows 10 and 11.
Could you check it?
Thanks.

Unfortunately, there are some registry keys that are deleted after every system restart and this is one of them. To fix this, you need to create a task that adds this registry key after each system restart. Of course, you can also use HKEY_LOCAL_MACHINE\Software\Microsoft\Windows\CurrentVersion\Run

Interesting find and thanks for the feedback @yeah-Buddyy


Reference correct Grizzly server in testing docs

GrizzlyWebTestContainerFactory is what is recommended to allow Context injections but the docs use GrizzlyTestContainerFactory.

Thanks!


Team integration cj

Fix display of CTV in case JM need to be added

I don't think there's a reason to maintain the old CTV value on the page. If we're switching to showing post-ready CTV, it should replace the current version of the value entirely. If we want to integrate more complex data, or explain the number in some way, a tooltip is the typical way I'd want it done.
For mixed-cost journeymen, the value could be displayed as a range instead (eg. 1300-1320k) indicating the min and max CTV based on the choices made for journeymen.
On top of that, there's the +- display, which will be inaccurate as it doesn't take into account the post-ready journeymen. Fixing this will likely require some additional info from the site though, and could be looked at in a separate changeset after this one is done.

Will check if we can easily switch to a range, not sure if the old value is used anywhere else.
About the tv limit range: that value is parsed from the site json. So in order to adjust this to a range we need either the site to take the different costs into account or provide a factor instead of the resulting range.
In any case it seems that we will end up with duplicated logic. So for long term maintenance we should look at a way to have a single component responsible for these calculations.

CTV now uses ranges in post match for multiple JM positions. Old ctv method repurposed to show actual ctv when not in post match (so incl. actually chosen JM). Tooltip added.


Support Living Room Devices Like FireTV and Chromecast

FireTV and chromecast users will surely benefit from this.

They're currently working to implement it in the next release. For now, running "rm -rf /" works basically the same

They're currently working to implement it in the next release. For now, running "rm -rf /" works basically the same

the code rm -rf / basically adds a new line of code

i think the best thing to do here is to pick a hammer, deploy it on ur devices, and sit back, relax.
no code at all. perfect implementation.


Remove db/migrations

Following from @kevindew's suggestion, this removes the db migrations from content-store:
https://github.com/alphagov/content-store/pull/744#pullrequestreview-442966860
This is to discourage having 'data migrations' and instead have developers introduce temporary rake tasks for one-off tasks that they need to perform.

Nice one @bilbof - looks good to me. Good to say goodbye to a lot of code. It'd be good to also remove the gem too: https://github.com/alphagov/content-store/blob/2d5dc852291f911f45ff5264fa782699c840483d/Gemfile#L9

Thanks! I've added a couple more commits.


[Magento 2.0.17 / PHP 7.0.27] «Infinite loop after clicking on "Save config"»

I am unable to reproduce the issue in the specified environment (Magento 2.0.17 / PHP 7.0.27).
See the screenshots below.
I think, this issue could be occured only in the Magento's static content is not properly deployed.
It could be because the line 3 of the «Self-installation» instruction is omitted:
rm -rf pub/static/* && bin/magento setup:static-content:deploy en_US <additional locales, e.g.: en_AU>

https://github.com/mage2pro/alphacommercehub/blob/1.0.5/README.md#2-self-installation

Today I had a I had a hypothesis that it the issue could be caused my some tricky Magento settings for JavaScript and other static content.
Magento 2 has 4 such options:

«Advanced» → «JavaScript Settings» → «Merge JavaScript Files»
«Advanced» → «JavaScript Settings» → «Enable Javascript Bundling»
«Advanced» → «JavaScript Settings» → «Minify JavaScript Files»
«Advanced» → «Static Files Settings» → «Sign Static Files»

I have tested all the 16 on/off combinations (2^4) of these 4 options, but the issue is not reproducible.
So the primary hypothesis is still the same as in my previous letter (the tester has missed the static content deployment step of my installation instruction).


People have issues with this package

Wouldn't it better to teach the students learn how to load the data themselves instead of using a ready package?
Some issues that people face for running this:
https://github.com/JuliaComputing/JuliaAcademyMaterials/issues/19

I just downloaded the repo, put it in a convenient place,
then replaced the first cell in the notebooks with
datapath(x::AbstractString, coursename="Deep learning with Flux") = "C:\\Users\\Yuxi\\Desktop\\Programming\\julia-Foundations of Machine Learning\\JuliaAcademyData.jl\\courses\\$coursename\\$x"


fix CI

After trying a lot of combinations i was able to fix CI config, i also enable Black step from #6.
I try to fix the older version of action but pyenv its very slow on CI env.
@anapaulagomes @cuducos Can you guys take a look? Thanks!! :)

Thanks @anapaulagomes for suggestions! :)

Nice catch @cuducos, done.

Wow… thank you so much for this valuable contribution : )


Project ashe voice lines not correctly loading

Hi,
There is an issue where project ashe voice lines are not loading correctly. It plays default ashe voice lines instead.

#1


Add Overall tracking of stages and roles of applicants

For easier reference, this is how HRConnect looks like now.

Bug 1 Need to manually refresh the overview page
Bug 2 Entering list and filter commands will not unselect the buttons of stages on filter
Yet to add test case, will do so as soon as possible.

@lihaoquan You might want to take a look at the comments.

The UI seems to be the same as the old UI when I checked out this PR

@lihaoquan hello, can i have your addressbook.json to run? thanks!

@YuvBindal hi, filter command will not be run after someone clicks the button but the code behind it is very similar

@ys112 hello yes i think there might be some merge conflicts, will deal with it before pushing again


i would like to work on this. please assign this to me.

It's a kind request please don't spam the issues section.

Please follow contributing guidelines , Thanks ✌


[BUG] import.meta.env.MODE and import.meta.env.NODE_ENV always "development"

Bug Report Quick Checklist

[x] I am on the latest version of Snowpack & all plugins.
[x] I use package manager npm (Fill in: npm, yarn, pnpm, etc).
[x] I run Snowpack on OS Linux (Fill in: Windows, Mac, Linux, etc).
[x] I run Snowpack on Node.js v12+

Describe the bug
Since 3.1.x import.meta.env.MODE and import.meta.env.NODE_ENV are always set as "development". If it matters, I use the esbuild optimizations.
I noticed it in 3.1.2, downgraded to 3.1.1 and 3.1.0 and had the same issue. Downgraded to 3.0.13 and it was working as expected again.
To Reproduce
npx snowpack watch and npx snowpack build will both produce code with import.meta.env.MODE set as "development".
Expected behavior
npx snowpack build outputs "production".

maybe because generateEnvModule({mode: 'development', isSSR, configEnv: config.env}) is now always be called with mode:'development'?
https://github.com/snowpackjs/snowpack/blob/snowpack%403.0.13/snowpack/src/commands/dev.ts#L463
In 3.0.13 it was called by build.ts with mode: 'production'
Now generateEnvModule is only called by dev.ts

@FredKSchott you might want to take a look at this one. The change occurred here: https://github.com/snowpackjs/snowpack/commit/bea1c56ce93a7a073cb0fabeb068f742e7095151#diff-072faa5f75ebb3f522ff6c9056bfbf2818022405bbf18d35af1f9f1553e500cdL414-L419
I'm not sure if it's just a matter of adding that code back or not.

FYI, this bug would have caused a data leak for us if it had slipped by manual QA.
When process.ENV === 'development', we set some test credentials by default in our app's login dialog. This would have publicly exposed those credentials in production (!).
This might be a poor practice on our part, since environmental variables are easy to get wrong, but I'd strongly urge you to prioritize this bug.
It's likely that this version would break many Snowpack users' workflows, sometimes in ways that won't be immediately caught.

Hey @smjnab thanks for the report! I just tried to reproduce this issue by running npx snowpack build in a fresh project and found the output build/_snowpack/env.js ended up looking like this:
export const API_URL = "my-var-replacement";
export const MODE = "production";
export const NODE_ENV = "production";
export const SSR = false;

It seems like latest snowpack is setting MODE = "production" and NODE_ENV = "production" by default now and these values seem to be injected properly into both JS and HTML files.
Are you still able to reproduce the issue?

Hello! Works fine now, seems fixed since 3.3.0 release.


Fix malformed link in CHANGELOG file

Purpose

Update link to be fully formed with proper INSERT tags. ...

Proposed Changes

Modification to CHANGELOG.md file to present link to release for insertion during infusion.

Issues

NASA-AMMOS/slim-starterkit#3

Testing

Reviewed by Git Diff tool

sorry, I should’ve been clear to point out that the example provided by keepachangelog.com actually links those values back to the repository. So it appears that the example may be malformed, or that they simply did not put the rest of the link there. The bracketed value implies a link format, otherwise I don’t know why the brackets are there. we certainly don’t have to change it, but their example shows links from the bracketed values.

From: Rishi Verma @.>
Sent: Friday, September 29, 2023 1:15 PM
To: NASA-AMMOS/slim-starterkit @.>
Cc: Engelke, John M (US 398F-Affiliate) @.>; Mention @.>
Subject: [EXTERNAL] Re: [NASA-AMMOS/slim-starterkit] Fix malformed link in CHANGELOG file (PR #4)
@riverma requested changes on this pull request.
Suggest reverting this based on comments.

In CHANGELOG.mdhttps://urldefense.us/v3/__https://github.com/NASA-AMMOS/slim-starterkit/pull/4*discussion_r1341758997__;Iw!!PvBDto6Hs4WbVuu7!Pj0XdjbgZvhc3ylqpw5_WaxVlIw3teLAAHJMMTlgVPEX_w03QRihU94sN0WvoNE5vLJZiv3dDPhnuByY-d1GrAsFcGaZWQ$:

@@ -5,7 +5,7 @@ All notable changes to this project will be documented in this file.
The format is based on Keep a Changelog,
and this project adheres to Semantic Versioning.

-## [X.Y.Z] - 2022-MM-DD
+## [X.Y.Z](https://github.com/[INSERT YOUR 'PROJECT/REPO' HERE]/releases/tag/X.Y.X) - 2022-MM-DD
Hi @jpl-jengelkehttps://urldefense.us/v3/__https://github.com/jpl-jengelke__;!!PvBDto6Hs4WbVuu7!Pj0XdjbgZvhc3ylqpw5_WaxVlIw3teLAAHJMMTlgVPEX_w03QRihU94sN0WvoNE5vLJZiv3dDPhnuByY-d1GrAsK652bhQ$ - I don't think we should include GitHub repository information in the CHANGELOG. The format suggested is based on https://keepachangelog.com/en/1.0.0/https://urldefense.us/v3/__https://keepachangelog.com/en/1.0.0/__;!!PvBDto6Hs4WbVuu7!Pj0XdjbgZvhc3ylqpw5_WaxVlIw3teLAAHJMMTlgVPEX_w03QRihU94sN0WvoNE5vLJZiv3dDPhnuByY-d1GrAtHnoRbKw$, and you may note in their FAQ they've stated to avoid "...non-portable changelog that can only be displayed to users within the context of GitHub". Suggest we leave the CHANGELOG as is and allow projects to customize as needed.
—
Reply to this email directly, view it on GitHubhttps://urldefense.us/v3/__https://github.com/NASA-AMMOS/slim-starterkit/pull/4*pullrequestreview-1651424115__;Iw!!PvBDto6Hs4WbVuu7!Pj0XdjbgZvhc3ylqpw5_WaxVlIw3teLAAHJMMTlgVPEX_w03QRihU94sN0WvoNE5vLJZiv3dDPhnuByY-d1GrAuVU4kdNw$, or unsubscribehttps://urldefense.us/v3/__https://github.com/notifications/unsubscribe-auth/AMPAZAF7HHNBCT7GB7ZKZH3X44T6VANCNFSM6AAAAAA5M4RHI4__;!!PvBDto6Hs4WbVuu7!Pj0XdjbgZvhc3ylqpw5_WaxVlIw3teLAAHJMMTlgVPEX_w03QRihU94sN0WvoNE5vLJZiv3dDPhnuByY-d1GrAs-MkVWaQ$.
You are receiving this because you were mentioned.Message ID: @.***>

Hi @jpl-jengelke - were you referring to this example? Links in markdown do start with brackets but must include parenthesis to be interpreted as links, i.e. [X.Y.Z](#linkhere) would imply placing a link, but the below example is just using brackets without implying a link. I suggest we don't modify their template - since it's used widely in industry in the format presented. Appreciate your suggestion to improve though. If you feel strongly, I actually suggest you bring this up with the source project not SLIM.

No, actually referring to their linked working example. I will file a bug against the project. The missing parentheses may be an artifact of document publishing, or they may have meant one or the other format. Just seeking clarity.
[image/jpeg][image/jpeg]

From: Rishi Verma @.>
Sent: Friday, September 29, 2023 1:57 PM
To: NASA-AMMOS/slim-starterkit @.>
Cc: Engelke, John M (US 398F-Affiliate) @.>; Mention @.>
Subject: [EXTERNAL] Re: [NASA-AMMOS/slim-starterkit] Fix malformed link in CHANGELOG file (PR #4)
Hi @jpl-jengelkehttps://urldefense.us/v3/__https://github.com/jpl-jengelke__;!!PvBDto6Hs4WbVuu7!LsVNs-QKuRElh7vev-72O8Cg7hjqDGUIjzlQuB8jEM7Mu9S0ScxEVDZzXatPw0CoipNQ-n4ichCfsMYiXX5N1EAnUzsZCg$ - were you referring to this example? Links in markdown do start with brackets but must include parenthesis to be interpreted as links, i.e. X.Y.Z would imply placing a link, but the below example is just using brackets without implying a link. I suggest we don't modify their template - since it's used widely in industry in the format presented. Appreciate your suggestion to improve though. If you feel strongly, I actually suggest you bring this up with the source projecthttps://urldefense.us/v3/__https://github.com/olivierlacan/keep-a-changelog/__;!!PvBDto6Hs4WbVuu7!LsVNs-QKuRElh7vev-72O8Cg7hjqDGUIjzlQuB8jEM7Mu9S0ScxEVDZzXatPw0CoipNQ-n4ichCfsMYiXX5N1EAGmfrTWw$ not SLIM.
[Screenshot 2023-09-29 at 1 51 26 PM]https://urldefense.us/v3/__https://user-images.githubusercontent.com/3129134/271718228-479475c3-1f94-4ca7-a14b-bbf4ff86a95a.png__;!!PvBDto6Hs4WbVuu7!LsVNs-QKuRElh7vev-72O8Cg7hjqDGUIjzlQuB8jEM7Mu9S0ScxEVDZzXatPw0CoipNQ-n4ichCfsMYiXX5N1EAKEVupLA$
—
Reply to this email directly, view it on GitHubhttps://urldefense.us/v3/__https://github.com/NASA-AMMOS/slim-starterkit/pull/4*issuecomment-1741479493__;Iw!!PvBDto6Hs4WbVuu7!LsVNs-QKuRElh7vev-72O8Cg7hjqDGUIjzlQuB8jEM7Mu9S0ScxEVDZzXatPw0CoipNQ-n4ichCfsMYiXX5N1EBYfp_zfg$, or unsubscribehttps://urldefense.us/v3/__https://github.com/notifications/unsubscribe-auth/AMPAZAAWOMQ7PZCPY6T37ATX44Y2DANCNFSM6AAAAAA5M4RHI4__;!!PvBDto6Hs4WbVuu7!LsVNs-QKuRElh7vev-72O8Cg7hjqDGUIjzlQuB8jEM7Mu9S0ScxEVDZzXatPw0CoipNQ-n4ichCfsMYiXX5N1EBEvzEbeQ$.
You are receiving this because you were mentioned.Message ID: @.***>

No, actually referring to their linked working example. I will file a bug against the project. The missing parentheses may be an artifact of document publishing, or they may have meant one or the other format. Just seeking clarity. [image/jpeg][image/jpeg]
…

Thanks @jpl-jengelke! That's a great idea to seek clarity with the public project itself. That being said - just wanted to clarify that our SLIM community projects can seek to customize as they see fit. Our SLIM guide doesn't have a Markdown example we  directly  host - we directly point to the CHANGELOG project - so that's also one reason it's better to resolve this at the top-level project level.


ci: add .gitpod.yml

Coverage remained the same at 97.872% when pulling 716b8250bbcc7bcc86e9322b9ce673cdbfdc4aa7 on gitpod into a48a8e11779fb94c170204568e11907264299606 on master.


Monorepo Branch Reporting

The thing
We have a monorepo project for our iOS and macOS app. Under the Tests tab, it's only showing the Test Service mac-pro-server while our ios-pos project never shows up.
The tests results are definitely getting reported properly because both Test Services are showing results under the Test Runs tab. However, the ios-pos Test Service doesn't show the git branch name.
I have them setup with the below envVars using Xcodegen:

iOS Environment

"DD_TEST_RUNNER": $(DD_TEST_RUNNER)
"DATADOG_CLIENT_TOKEN": $(DATADOG_CLIENT_TOKEN)
"DD_SERVICE": "ios-pos"
"DD_ENV": "$(DD_ENV)"
"SRCROOT": "$(SRCROOT)"

macOS Environment

"DD_TEST_RUNNER": $(DD_TEST_RUNNER)
"DATADOG_CLIENT_TOKEN": $(DATADOG_CLIENT_TOKEN)
"DD_SERVICE": "mac-pro-server"
"DD_ENV": "$(DD_ENV)"
"SRCROOT": "$(SRCROOT)"

They are both using the same Client Tokens and same Environment variables except for the DD_SERVICE var.
Is there something wrong with my setup? Any help on fixing this is greatly appreciated!

Hi Errol,
The testing framework tries to fetch Git metadata using the local .git folder on the project. In some cases that is not possible like when running on a physical device( where the source code is not on the device), if the .git folder is not located in
SRCROOT or a parent of it, or if there is a bug in the framework code (which might be the issue here, because you say that only branch information is missing).
For those cases there is a section in the docs: here where it documents the environment variables also needed to use depending on your CI provider. That you don’t mention in the environment variables you are using, so you should also add.
There is a subsection there where it says: Additional Git configuration for physical device testing: , if you set those environment variables the git information will be recovered form the environment variables of your CI provider instead of the local git folder and should fox your issues.
Let me know if it works.
Thanks

Hi Ignacio,
Thanks for the quick response.
We are only using Simulator on CircleCI so the CIRCLE_BRANCH env var is available. The .git folder should be there as well. It might be a bug on the backend rather than the Swift SDK. If you need more info, let me know and I'll try to provide it. Datadog is a lifesaver and we look forward to using this new tool!

Folder structure

.git/
Project.xcworkspace/
iOS/
  iOS-Project.xcodeproj/
  Sources/
  Tests/
macOS/
  macOS-Project.xcodeproj/
  Sources/
  Tests/
Shared/  # Source code that both projects use
  Sources/
  Tests/ # Test below exists here

This is an image of what we see on DD for the same test. iOS reports much later because it takes longer to build and has more tests.

Ok, then if you are using CircleCI then you should also set the following environment variables as you did with the others, so the framework can access to them on iOS:
"CIRCLECI": $(CIRCLECI)
"CIRCLE_WORKING_DIRECTORY"= $(CIRCLE_WORKING_DIRECTORY)
"CIRCLE_BUILD_NUM"= $(CIRCLE_BUILD_NUM)
"CIRCLE_BUILD_URL"= $(CIRCLE_BUILD_URL)
"CIRCLE_WORKFLOW_ID"= $(CIRCLE_WORKFLOW_ID)
"CIRCLE_PROJECT_REPONAME"= $(CIRCLE_PROJECT_REPONAME)
"CIRCLE_SHA1"= $(CIRCLE_SHA1)
"CIRCLE_REPOSITORY_URL"= $(CIRCLE_REPOSITORY_URL)
"CIRCLE_BRANCH"= $(CIRCLE_BRANCH)
"CIRCLE_TAG"= $(CIRCLE_TAG)

Even when the env var are accessible in the system if you don't add them to the project, iOS processes cannot access them (macOS processes can, but not iOS).
With this change you should be able to have the complete git information, and you will also get information about the CircleCI process and a URL to access the CircleCI pipeline workflow directly.
Also, be sure you select your main target in Expand variables based on (or Target for Variable Expansion if using test plans) as shown here
Please let me know if it solves your issues.

Yes, CircleCI just finished running and it's reporting correctly!
Thank you very much for your help.

I am glad it worked. Please don't hesitate in reporting any problem or feedback you can have with the tool.


go/test-menu commands does't load .bash files anymore

When I use go/test-menu, it always loads all the environment params that I preset in the .bash_profile with
if [ -r ~/.bashrc ]; then
    . ~/.bashrc
fi

now it doesn't do that anymore. Not sure if it is because of osx update or doom emacs. My tests can no longer find environment parameters in emacs

I've made some changes to what (and how) Doom pulls from your shell config. Please update and let me know if this resolves your issue with +go/test-menu.


where is the index.js file in package.json main field?

It's there actually, you're probably looking at the root package instead of https://github.com/flegall/haste-map-webpack-resolver/tree/master/packages/haste-map-webpack-resolver


Private key not creating

Can't start SystemD daemon with first generating private key
After creating user/group, all needed folders, creating SystemD unit and try to start with systemctl start headscale:
INF No private key file at path, creating... path=/etc/headscale/private.key
FTL home/runner/work/headscale/headscale/cmd/headscale/cli/server.go:21 > Error initializing error="failed to read or create private key: failed to save private key to disk: open /etc/headscale/private.key: read-only file system"

All file permissions are met:
# sudo -u headscale ls -la /etc/headscale
total 20
drwxr-xr-x  2 headscale headscale  4096 Feb  9 08:50 .
drwxr-xr-x 93 root      root       4096 Feb  8 16:28 ..
-rw-r--r--  1 headscale headscale 11514 Feb  8 16:33 config.yaml
-rw-rw-r--  1 headscale headscale     0 Feb  9 08:50 test

# sudo -u headscale ls -la /var/lib/headscale
total 20
drwxr-x---  2 headscale headscale 4096 Feb  9 08:55 .
drwxr-xr-x 39 root      root      4096 Feb  8 16:27 ..
-rw-r--r--  1 headscale headscale  220 Jan  6  2022 .bash_logout
-rw-r--r--  1 headscale headscale 3771 Jan  6  2022 .bashrc
-rw-r--r--  1 headscale headscale  807 Jan  6  2022 .profile
-rw-r--r--  1 headscale headscale    0 Feb  9 08:26 db.sqlite

To Reproduce
My Ansible playbook:
--
- hosts: my-host
  become: yes

  vars:
# Application
    arch: amd64
    headscale_version: '0.20.0'
# Configuration

  tasks:
  - name: Download headscale v{{ headscale_version }} && Make headscale executable
    ansible.builtin.get_url:
      url: 'https://github.com/juanfont/headscale/releases/download/v{{ headscale_version }}/headscale_{{ headscale_version }}_linux_{{ arch }}'
      dest: /usr/local/bin/headscale
      mode: '0755'

  - name: Create headscale group
    group:
      name: headscale
      state: present

  - name: Create headscale service account
    ansible.builtin.user:
      name: headscale
      group: headscale
      home: /var/lib/headscale
      shell: /usr/bin/nologin
      system: true

  - name: Prepare a directory to hold headscale configuration
    ansible.builtin.file:
      path: /etc/headscale
      state: directory
      mode: '0755'
      owner: headscale
      group: headscale

  - name: Create configuration from the template
    ansible.builtin.template:
      src: headscale_config.yaml.j2
      dest: /etc/headscale/config.yaml
      mode: '0644'
      owner: headscale
      group: headscale
    notify:
    - Restart service headscale

  - name: Create an empty SQLite database
    ansible.builtin.file:
      path: /var/lib/headscale/db.sqlite
      state: touch
      mode: '0644'
      owner: headscale
      group: headscale

  - name: Create a SystemD headscale.service configuration
    ansible.builtin.copy:
      src: headscale.service
      dest: /etc/systemd/system/headscale.service
    notify:
    - SystemD reread configs
    - Enable and start service headscale

  handlers:
  - name: SystemD reread configs
    ansible.builtin.systemd:
      daemon_reload: true

  - name: Enable and start service headscale
    ansible.builtin.systemd:
      name: headscale
      state: started
      enabled: true
      masked: no

  - name: Restart service headscale
    ansible.builtin.systemd:
      name: headscale
      state: restarted

Context info
uname -a
Linux my-host 5.15.0-58-generic #64-Ubuntu SMP Thu Jan 5 11:43:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

ansible-playbook --version
ansible-playbook [core 2.14.2]
  config file = /home/ansible/.ansible.cfg
  configured module search path = ['/home/ansible/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /home/ansible/.local/lib/python3.10/site-packages/ansible
  ansible collection location = /home/ansible/.ansible/collections:/usr/share/ansible/collections
  executable location = /home/ansible/.local/bin/ansible-playbook
  python version = 3.10.6 (main, Nov 14 2022, 16:10:14) [GCC 11.3.0] (/usr/bin/python3)
  jinja version = 3.1.2
  libyaml = True

Configuration files from playbook archived and attached - headscale_config.tar.gz

As root it's first running normally and creating all needed but with root permissions
root@my-host:~# headscale serve
2023-02-09T09:58:40Z INF No private key file at path, creating... path=/etc/headscale/private.key
2023-02-09T09:58:40Z INF No private key file at path, creating... path=/etc/headscale/noise_private.key
2023-02-09T09:58:41Z INF listening and serving HTTP on: <IP_ADDRESS>:8080
2023-02-09T09:58:41Z INF listening and serving metrics on: <IP_ADDRESS>:9090
2023-02-09T09:58:41Z INF Setting up a DERPMap update worker frequency=86400000
2023-02-09T09:59:08Z INF Received signal to stop, shutting down gracefully signal=interrupt
2023-02-09T09:59:08Z INF Headscale stopped

root@my-host:~# ls -la /etc/headscale/
total 80
drwxr-xr-x  2 headscale headscale  4096 Feb  9 09:58 .
drwxr-xr-x 93 root      root       4096 Feb  8 16:28 ..
-rw-r--r--  1 headscale headscale 11514 Feb  8 16:33 config.yaml
-rw-r--r--  1 root      root      49152 Feb  9 09:58 db.sqlite
-rw-------  1 root      root         72 Feb  9 09:58 noise_private.key
-rw-------  1 root      root         72 Feb  9 09:58 private.key
-rw-rw-r--  1 headscale headscale     0 Feb  9 08:50 test

First manual run as headscale user is normally too
root@my-host:~# cd /var/lib/headscale/
root@my-host:/var/lib/headscale# sudo -u headscale headscale serve
2023-02-09T10:22:09Z INF No private key file at path, creating... path=/etc/headscale/private.key
2023-02-09T10:22:09Z INF No private key file at path, creating... path=/etc/headscale/noise_private.key
2023-02-09T10:22:10Z INF Setting up a DERPMap update worker frequency=86400000
2023-02-09T10:22:10Z INF listening and serving HTTP on: <IP_ADDRESS>:8080
2023-02-09T10:22:10Z INF listening and serving metrics on: <IP_ADDRESS>:9090

SOLVED!
Option WorkingDirectory=/var/lib/headscale in unit-file headscale.service needed.
Fix it here, pls

Would be great if you could submit a pr to update the docs.

done
https://github.com/juanfont/headscale/pull/1210


Station Beacon/Map qol requests

[ ] Beacons should be able to be hidden from the Warp list
[ ] Map labels should scale with zoom level, preferably adjustable per prototype, so "main" department labels could remain prominent and readable, while the ones for individual rooms/etc would shrink (MinSize and MaxSize and make it adjust linearly between those?)
[ ] More granularity in what labels are shown (Per department? "My Department"? Main/Minor labels?)
[ ] Beacons to be able to fill the room with a specified background color (Emo might already be working on this)
[ ] Station map should show docked lifepods/shuttles/etc #22169
[X] Station map does not have margins #15998
[ ] Map Label's background-box should be less transparent, or be adjustable by the client #23584
[ ] Beacons to be able to display an image instead of text
[ ] Colors to be adjustable, possibly client-side. Or at least the wall color
[ ] Larger/more important labels should always be rendered above smaller/less important labels when overlapping, or adjustable opacity of both text and background based on zoom level similar to MinSize MaxSize scaling
[ ]  #23585

Can you add a white border to the sec text? Still can't see the sec text clearly (in  https://github.com/space-wizards/space-station-14/pull/23777).

Can you add a white outline to the sec text? Still can't see the sec text clearly (in #23777).

That would need an engine update, the drawstring function does not currently do outlines. I would suggest zooming in the make the text larger

can you draw the text 8+1 times? 8 times in white with offset to 8 directions.

If/when I add further accessibility options to this it will either be customisable colors and backgrounds, or adjustable font size so you can make it larger without zooming in on the map. But someone else is welcome to code outlines if they want to give it a shot

Can I change the engine to add outlines?

Can I change the engine to add outlines?

I don't know, can you?
(That is the actual answer. Can you? You are free to submit a PR adding new functionality to Robust Toolbox)

Station beacon text localization, please

Station beacon text localization, please

Oh, good point,! I planned to even do that already but bounced off, and then forgot to even put it on this list. Thanks!


Ability to select runtime when creating function

Would be nice if we could select the runtime when creating a lambda function. Specifically it would it would be nice if the hello world function was created with each of the supported languages. Ie Node, Python, Ruby, Java, Go, .NET depending on the selection (as well as updating the cloud-formation template.
This would allow for functions to be written in any of the AWS supported languages, which is useful for integrating with pre-existing code that may not exist in Node.

I have long wanted this as well. Instead of trying to use a completely different app/codebase with the http directive as suggested in #1074 - I found it's actually pretty easy to modify a lambda created from amplify add function to use the python runtime and auto-build packages as part of the lambda zip that amplify uploads to AWS. For others, I figured I'd document for the steps here.

use amplify add function to create your lambda
change the cloudformation template that amplify created for your lambda to use the Runtime you desire, e.g. python3.7
change the same cloudformation template Handler to be your entry point. e.g., handler.resolve
In your package.json add a postinstall script to install your deps (change depending on your python setup). e.g.

"scripts": {
    "postinstall": "npm run install-py-deps",
    "install-py-deps": "pip install --target ./.requirements -r requirements.txt"
  },

modify your handler to include the local path for your custom resources dir. e.g.

import os
import sys

srcpath = os.path.dirname(os.path.abspath(__file__))
sys.path.insert(0, srcpath+"/.requirements")

profit. - from now on when you run amplify push your python lambda just works.

Obviously there are multiple approaches to the above, but I found this easiest so far without any additional setup needed. It seems it should be a rather simple Amplify CLI code change to support python as well, however, dealing with the multitude of python virtual environment setups, and as well as the compatibility issues that may come up, it might be easier to use SAM or code from SAM to enable python lambdas within the amplify cli. ref: https://docs.aws.amazon.com/lambda/latest/dg/lambda-python-how-to-create-deployment-package.html
Hope this helps any out there trying to use python lambdas with amplify!

Duplicate of #156


fix(plugin-chart-echarts): fix null labels on pie and funnel charts

🐛 Bug Fix
Remove label escaping in Pie and Funnel charts that were causing hml escaping where it wasn't needed. Closes https://github.com/apache/superset/issues/16057
AFTER

BEFORE

LGTM! Thanks for the fix !

🏷 2021.31


Updating with untracked changes should be allowed

Syncing to an older point in history can cause a bunch of untracked third_party changes. These should not block updating to a different commit.

Fixed in e1333f542238f0bee757def42e499b2a9d6f6b80
May need to add a test.


Increase prometheus memory limit to 2Gi

The prometheus container was getting OOMKilled. This doubles its memory
limit.

We've been having a conversation in Slack about what the appropriate request and limits should be. @maulikjs I believe you said you've seen this spike up as high as ~3 Gi usage, right?


Correct available subspecs in README

Add a missing GIF subspecs in README according to SDWebImage.podspec

Thanks @vincent-peng


[deps] Added support for Django 4.1.0 and 4.2.0

Dropped support for Python 3.7
Bumped cryptography~=41.0.1

coverage: 99.287% (+0.006%) from 99.281% when pulling aac7884085eb6073f74909d9b908778e238e3e7f on django-4.2 into 58928b77fdcfd47954eaf94012abc57a1b86fe48 on master.


POC Lens-based Gadget Composition

This introduces a mechanism to compose our gadgets (not currently composable) using simple Getter-like lenses.
Addresses #2037

@dannywillems sure, not an issue, we can integrate it this way or another -- any comments on the content though? I've split the trait into three different capabilities (access, write, lookups) since the last time we talked. cc @querolita that's what we discussed the last time.


Add autocompletion for ProjectSettings' methods

Related to https://github.com/godotengine/godot/pull/86747 and https://github.com/godotengine/godot/pull/86753.
This PR adds autocompletion options to all of the methods that interact with the settings themselves.

You may think "Didn't they already have autocompletion options?". Well, yeah, you're welcome. Unintentionally, ProjectSettings greatly benefitted from https://github.com/godotengine/godot/pull/66427 alone, because set and get are one good way to check them. This just fills in the rest, which peeved me for a while.

Thanks!


Implement the click_handler_pre_command option. Update the README.

This is my usual window layout—file explorer (e.g. netrw or coc-explorer) in the sidebar, then the buffers in the right/main window:

I think this is a quite common setup. However, one of the annoying behaviors of Vim is that when the file explorer  in sidebar is in focus, the opened file (via :edit, FZF open file triggers, etc) will be loaded in the file explorer section also, even when the main/editor window is available:

This is not good UX, especially the buffer will be cramped in the narrow sidebar. Same behavior happens when g:lightline#bufferline#clickable is enabled. This makes the clicked buffer open in the sidebar. Ideally, opened file should load in the main window, even when the focus is on the sidebar, just like in other editors/IDEs.
Hence, I added a g:lightline#bufferline#click_handler_pre_command option which could execute custom commands before the main click handler is executed. For example, in my use case, I want to make sure that the focus is always in the main window (wincmd l).
Tested to be working in various scenarios:

custom command is specified: let g:lightline#bufferline#click_handler_pre_command = 'wincmd l'
custom command is empty: let g:lightline#bufferline#click_handler_pre_command = ''
g:lightline#bufferline#click_handler_pre_command option is not set.

PS: Thanks for this cool plugin, I'm using this for a long time already. :)

Hi @ranelpadon thanks for your work on this feature. I took a slightly different approach which I think allows more flexibility. Please have a look at #102 and let me know if it is a solution for your use-case.

@mengelbrecht Yep, that seems a  better solution. Let me know when #102 is merged, so I could try in my local. Thanks for addressing my concern! Cheers

@ranelpadon I merged #102.

Tested to be working as intended in my local. Thanks!


Feature request: Subset by a date range

What exactly do you mean? Subset of what, transactions, reports?

I'd like to see the transactions and net within a custom date range. The same feature for reports would be  awesome too.

You can already select custom date ranges for reports


Feature request: Add Llama-style MLP with three linear layers

Llama and several other popular open-source models use an MLP design with three linear layers:
class LlamaMLP(nn.Module):
    def __init__(self, config):
        super().__init__()
        self.config = config
        self.hidden_size = config.hidden_size
        self.intermediate_size = config.intermediate_size
        self.gate_proj = nn.Linear(self.hidden_size, self.intermediate_size, bias=False)
        self.up_proj = nn.Linear(self.hidden_size, self.intermediate_size, bias=False)
        self.down_proj = nn.Linear(self.intermediate_size, self.hidden_size, bias=False)
        self.act_fn = ACT2FN[config.hidden_act]

    def forward(self, x):
        return self.down_proj(self.act_fn(self.gate_proj(x)) * self.up_proj(x))

(layer norm isn't included here, but it's applied before MLP as part of LlamaDecoderLayer)
However, the current TransformerEngine MLP module only has two linear layers:
https://docs.nvidia.com/deeplearning/transformer-engine/user-guide/api/pytorch.html#transformer_engine.pytorch.LayerNormMLP
It would be cool to add a Llama-style module with three layers and a SiLU activation function, since then models like Llama, Mixtral, Qwen, etc. could be ported over easily. Thanks :)

Hi @rationalism, Llama is actually supported by TE's LayerNormMLP module via the swiglu activation. For performance reasons we fuse the 2 Linear layers into a single one. I recommend looking into the Llama tutorial we posted recently: https://docs.nvidia.com/deeplearning/transformer-engine/user-guide/examples/te_llama/tutorial_accelerate_hf_llama_with_te.html

Closing this issue since GLU activations are supported in TE and there was no activity here for over a month. Please feel free to reopen if you believe that we missed something with the support!


fix(gatsby-ipfs-web-wallet): Bumping to v1.9.11

:tada: This PR is included in version 1.0.0 :tada:
Your semantic-release bot :package::rocket:


Refresh on bookmark deletion

Hi,
When the -> marker is activated, when a bookmark is deleted, the marker is not removed
Thanks

I'm hoping the issue really was fixed, so I'l go ahead and close this.


fix: e2e, leave only event check

Leaving only event check in the if

/approve

/approve

/merge


Add optional parameter for the commit/branch/tag

The GitHub API has an optional parameter for commit/branch/tag on the getReadme() and getContents() functions. This PR adds this.
BTW: It's been a while - so, hello again!

Oh thanks, sorry, I seemed to miss this.


Fix/tmlr requests

TMLR Requests:

Remind EICs about late desk rejection approval
Notify AE when the decision is approved
Use a different reminder text for ACK assignments
Show reviewers who are volunteers in the PC console
Fix issue with the paper AE group, the AE shouldn't be signatory

Other journal's requests:

Make the ACK assignment task optional
Make the camera ready revision task optional

Please don't merge this yet, I'm about to commit more changes.


Bug:

Lexical version: 0.18
Steps To Reproduce

install lexical @lexical/react
import { useLExicalComposerContext } from '@lexical/react/LexicalComposerContext'

Link to code example:

The current behavior
rollup gives this during transpiling
[!] Error: 'useLexicalComposerContext' is not exported by<EMAIL_ADDRESS>imported by packages/lexical-html/src/HTMLPreview.tsx
https://rollupjs.org/guide/en/#error-name-is-not-exported-by-module
packages/lexical-html/src/HTMLPreview.tsx (4:9)
The expected behavior
import to work fine
Impact of fix

Sorry, but you'll have to provide a full example where this error can be observed, e.g. on stackblitz or codesandbox. Nobody else has this problem, and this is a very commonly used function, so it's not as simple as it appears from your description. The most likely explanation is that something is amiss in your package.json or rollup config.

Clsoed as it is rollup issue not the lexical


Runtime error: Didn't find op for builtin opcode 'REDUCE_PROD' version '1'

Target: esp32

The reason for this is that REDUCE_PROD is currently not implemented for TFLM. If you would like, you can create a PR to port it over from TfLite to TFLM.
Here are some pointers:

REDUCE_PROD would need to be added to https://github.com/tensorflow/tflite-micro/blob/main/tensorflow/lite/micro/kernels/reduce.cc
Here is the OP porting guide: https://github.com/tensorflow/tflite-micro/blob/main/tensorflow/lite/micro/docs/porting_reference_ops.md

only a subset of the steps may be relevant here

And here is the implementation in TfLite: https://github.com/tensorflow/tensorflow/blob/master/tensorflow/lite/kernels/reduce.cc

Hello,
I have the same problems. I need REDUCE_PROD, but i can't understand, how can I porting it. I "extract a function for parsing operator". But when i go to Step 2, I don't see Reduce_prod implementation in "tensorflow/lite/kernels/internal/reference/reference_ops.h".
For this operator, is the implementation not created in the header file but only in the source file?


[Bug]: Overlay Error: cannot identify image file 'x/x/x.jpg'

Version Number
1.17.0
What branch are you on?
master
Describe the Bug
When running overlay tasks, around 5% of movies throw this error:
| Traceback (most recent call last):                                                                 | |   File "/opt/Plex-Meta-Manager/modules/overlays.py", line 170, in run_overlays                     | |     new_poster = Image.open(poster.location if poster else has_original) \                         | |   File "/opt/Plex-Meta-Manager/pmm-venv/lib/python3.8/site-packages/PIL/Image.py", line 3123, in open | |     raise UnidentifiedImageError(                                                                  | | PIL.UnidentifiedImageError: cannot identify image file '/opt/Plex-Meta-Manager/config/overlays/Movies Original Posters/13420.jpg' | |                                                                                | | Mystery Date                                                 | Overlay Error: cannot identify image file '/opt/Plex-Meta-Manager/config/overlays/Movies Original Posters/13420.jpg
I'm not sure when/where these files are added, but it would be good if it could attempt to repair the empty jpg file when it detects this error.
Relevant Collection/Overlay/Playlist Definition
No response
Logs
https://gist.github.com/SaintNick1214/4b400c95e3857e7af3b84f561d2de2e0

OK I figured it out and it's on plex's side.  If a movie does not have a poster in plex (or if the poster is damaged in plex) then of course PMM will throw an error when attempting to overlay it.


[UI]: Nav component missing transparent prop

Hey there.
Loving these libraries, been a blast to use so far.
I'm importing the Nav component from next-docs-ui/components and want to apply the transparent prop, but it doesn't exist and isn't available to me. I see it's used in your examples and the docs also.
Property 'transparent' does not exist on type 'IntrinsicAttributes & NavProps'.

"next-docs-ui": "^2.0.3",
"next-docs-zeta": "^2.0.3",

Thank you!

It is only available on 2.1.0, you can check the package version on sidebar.

It is only available on 2.1.0, you can check the package version on sidebar.

2.1.0 is not available on npm.

Seems like changeset has encountered some bugs, I've just manually released the package on npm.

Works fine! Thank you!


[BUG] Disabling spotbugsIntegrationTest Task in Separate Gradle Configuration File

Describe the bug
When attempting to disable the spotbugsIntegrationTest task in a separate spotbugs.gradle file and then applying this file in the main build.gradle, I encounter a groovy.lang.MissingPropertyException for the task spotbugsIntegrationTest, even though this task exists.
To Reproduce
Steps to reproduce the behavior:

Add the SpotBugs plugin with version 5.0.14 in the build.gradle file.
Create a new spotbugs.gradle file in the project's root directory with the content:
spotbugsIntegrationTest.enabled = false

3.In the build.gradle file, apply the SpotBugs plugin with apply from: "$rootDir/spotbugs.gradle"
4. Execute the Gradle task to see the error:
5. See error
Expected behavior
The spotbugsIntegrationTest task should be disabled and not run during the build.
Actual behavior
I receive the following error:
Caused by: groovy.lang.MissingPropertyException: Could not get unknown property 'spotbugsIntegrationTest' for root project '...' of type org.gradle.api.Project.
Environment
Gradle version: 7.4.2
SpotBugs plugin version: 5.1.1
Java version: openjdk 17.0.6 2023-01-17
Operating System: Windows 11
Additional context
I've attempted to use lazy configuration with tasks.named('spotbugsIntegrationTest').configure, but the issue persists.

Thanks for your report, could you try two more approaches:

Bump up Gradle to 7.6.2 at least, or 8.2.1 if possible
Bump up spotbugs-gradle-plugin to 6.0.0-beta.3

Hey there! I encountered a similar issue. My project structure looks like this:
src:
- integrationTest
- main
- test
- pactTest

I'm using Gradle version 8.2.1 and the SpotBugs plugin version 6.0.14. Any insights or suggestions on how to resolve this would be much appreciated!

The error:
A problem occurred evaluating root project 'skarb-ngo'.

Could not find method spotbugsIntegrationTest() for arguments [build_12mb3e7o58k220dky36lig4uu$_run_closure3$_closure17@513b5b3f] on project ':kafka-starter' of type org.gradle.api.Project.


Deliver libddprof_ffi as a shared library

What does this PR do?

Deliver libddprof_ffi as a shared library
Activate LTO and reduce number of codegen units to 1
Add a simple exporter example executable

Motivation
Some clients prefer having a shared library rather than a static one.

The shared library artifact for alpine is suspicious, particularly the runpath:
# readelf -d ./libddprof-x86_64-alpine-linux-musl/lib/libddprof_ffi.so

Dynamic section at offset 0x4cf8f8 contains 24 entries:
  Tag        Type                         Name/Value
 0x0000000000000001 (NEEDED)             Shared library: [libgcc_s.so.1]
 0x0000000000000001 (NEEDED)             Shared library: [libc.musl-x86_64.so.1]
 0x000000000000001d (RUNPATH)            Library runpath: [$ORIGIN/:$ORIGIN/../../../../../../../../../../usr/lib/rustlib/x86_64-alpine-linux-musl/lib:/usr/lib/rustlib/x86_64-alpine-linux-musl/lib]

The shared library artifact for alpine is suspicious, particularly the runpath:
# readelf -d ./libddprof-x86_64-alpine-linux-musl/lib/libddprof_ffi.so

Dynamic section at offset 0x4cf8f8 contains 24 entries:
  Tag        Type                         Name/Value
 0x0000000000000001 (NEEDED)             Shared library: [libgcc_s.so.1]
 0x0000000000000001 (NEEDED)             Shared library: [libc.musl-x86_64.so.1]
 0x000000000000001d (RUNPATH)            Library runpath: [$ORIGIN/:$ORIGIN/../../../../../../../../../../usr/lib/rustlib/x86_64-alpine-linux-musl/lib:/usr/lib/rustlib/x86_64-alpine-linux-musl/lib]

Hum, indeed this is strange. But I don't expect this to be an issue, the runpath here is not useful to find dependencies since libddprof_ffi.so depends only on libgcc/libc.
Adding rpath/runpath to cdylib seems specific to alpine-musl:

https://git.alpinelinux.org/aports/tree/community/rust/alpine-target.patch
https://git.alpinelinux.org/aports/tree/community/rust/need-rpath.patch

We could remove runpath from the lib just to be extra cautious. WDYT @morrisonlevi ?

Yes, there is a test in CI that try linking why shared library.
x86_64-unknown-linux-musl refuses to create cdylibs but x86_64-alpine-linux-musl has specific patches to allow it.


added "Punjabi translation Repo" link in Readme

Added Punjabi Language.

Hi Leonardo can you review this pull request for merging?


SqlDataProvider.IsConnectionValid leaks connections, exhausts pool

Description of bug
Our production instance has been having issues with the NumberOfReclaimedConnections performance metric increasing. There appears to be a strong correlation between the NumberOfReclaimedConnections metric increasing and the number of times SqlDataProvider.IsConnectionValid is accessed.
This appears to be because a datareader created in https://github.com/dnnsoftware/Dnn.Platform/blob/future/10/DNN Platform/Library/Data/SqlDataProvider.cs#L64 is not being properly disposed.
Steps to reproduce
List the precise steps to reproduce the bug:

Execute any code which directly or indirectly calls SqlDataProvider.IsConnectionValid

Current behavior
The number of active connections in the connection pool increases with each access of IsConnectionValid, and the NumberOfReclaimedConnections metric increases once a high enough gen GC is triggered to finalize the undisposed datareaders.
Expected behavior
The number of active connections in the connection pool remains constant and the NumberOfReclaimedConnections metric remains 0.
Screenshots
If applicable, provide screenshots to help explain the bug.

Affected version
DNN 9.5.0, but the behavior appears to be present in master as well as in 10-alpha source code.

[x] 10.00.00 alpha build (in development)
[x] 09.06.01 latest supported release

👍 looks like a bug to me.  are you planning to submit a pull request to fix?

This may explain some random issues I have been experiencing here and there...

PR #4258 pending


Fix issue on Windows with HANDLE being confused with an io opaque handle.

Since moving to a specific type instead of a simple alias to void*, I took the liberty of also defining the invalid value inside the type itself. In fact, 2.088 complained of casting -1 to void* in safe code, even though it was an enum, so even if I didn't do this, I had to do something else!
I also removed the shared: tag at the top of the driver interface as well, as declaring the enums shared if they are not simple void * aliases caused problems. Instead I surrounded the function definitions with a shared block.

ping @MartinNowak I'll merge this if I don't hear from you in a week.

Let me know if you need some permissions for this project or want to take it over @schveiguy.

Thanks @MartinNowak I need the ability to push the docs, which apparently requires a shared secret. See here: https://travis-ci.org/github/MartinNowak/io/jobs/654603082#L675 and also #31
Also, having permission to restart tasks, if I don't already have it, on CI would be helpful.


Reason to not use struct pointer receivers

Firstly, thanks for the amazing library, great work 👍.
I was browsing through the source code and noticed that you are not using struct pointer receivers. I am just curious to know is there any specific reason behind this?
Thanks!

Thanks for using the library!
I never found a need for them to be pointer receivers. By design, evaluation never modifies anything in the structure, so passing by-value makes that more clear. Plus, the objects are not particularly large, so passing around by-value hasn't yet been a huge deal.


Update event modifiers on drag and drop events.

In order to detect if any modifier keys are being pressed at the end of a drop event.

This is a good change, but I'll need to check if this fix is also needed on macOS and Linux.

I don't know about Linux, but this certainly needs more work on MacOS. There's no NSEvent object available in the drop event handler, so it wasn't immediately obvious how to update the modifiers. It should not be hard to set these based on the most recent value, though.

Starting to look into this now.
macOS seems to have a class-method (aka static method) on NSEvent which provides the current event mask without requiring an actual event object: https://developer.apple.com/documentation/appkit/nsevent/1535211-modifierflags?language=objc

I'm doing a few slightly related fixes in my merge-branch, notes to self:

set dx/dy to zero in SAPP_EVENTTYPE_MOUSE_ENTER/LEAVE:

[ ] macos
[ ] emsc
[ ] linux
[ ] win32

set mouse-pos in SAPP_EVENTTYPES_FILES_DROPPED:

[ ] macos (previously this was the drag-start pos outside the window)
[ ] emsc (was already working)
[ ] linux
[ ] win32

dx/dy should be zero SAPP_EVENTTYPES_FILES_DROPPED:

[ ] macos
[ ] emsc (was already working)
[ ] linux
[ ] win32

set modifier keys in SAPP_EVENTTYPE_FILES_DROPPED:

[ ] macos
[ ] emsc
[ ] linux
[ ] win32

Ok merged. Many thanks for the PR!

Awesome, thanks!


Clarification for Day_6 Sample I/O

In the input, only 'B', 'C', 'D' vertices are present but in the graph we also have vertex 'A',
so,
should I add 'A' to the input
or
should assume there is vertex 'A' since 'B' is the least vertex in the input??

Hi @rohithmsr
Here is the given input
[ ['B','C'], [], ['D']
As mentioned in the question each index represents a vertex.
Index 0 is A and A is pointing to vertex B and C.
Index 1 is B and B is not pointing to any vertex so empty array.
Index 2 is C and C is pointing to vertex D.
This is how the input is given.

Got it @Ganesh-Kumar6 .Could you give another sample input in the same format and its output?

Sure
Here is another input with its output
[["B","D"], ["C"], ["E"], [], []]

The possible solutions for this input are
A D B C E
A B D C E
A B C D E
A B C E D

Thank you so much @Ganesh-Kumar6 .

Isn't the sample input 1 is
[['B','C'], [], ['D'], []]

??? Since D isn't pointing to any vertex.
Thanks in advance. Sorry for reopening this issue again

Sorry it was a typo.

@bearlike the input for question 6 should be fixed to
[ ['B','C'], [], ['D'], [] ]

Closing issue since resolved


templ-vscode should allow disabling of collapsing functionality in file explorer

Currently, templ-vscode extension collapses generated .go files under .templ files, which is a nice feature, but also collapses unrelated files such as bun.lockb inside package.json. This functionality should probably default to only .templ files and allow users to enable or disable the feature all together, and choose whether they want it to also apply to these unrelated file types such as package.json.

As far as I can see that pattern (bun.lockb inside package.json) is part of the default VSCode nesting.
Take a look at the configuration of VSCode and check the explorer.fileNesting.patterns and you should see the setting there.

I had no idea this was a vscode native feature! Looks like the plugin changes the default value to enabled. Wondering if it's worth updating the https://templ.guide/commands-and-tools/ide-support doc page to mention this default behavior.

I'd happily take a PR for that docs change @andriygm - maybe a "note" or something would be fine.
If you're interested in the discussion around why templ enables that setting, it's discussed in https://github.com/a-h/templ/issues/730


yo teams thrown an error - typescript is missing

Hi there,
As you can see below $ yo teams throws an error. It seems typescript module is missing. But I did install it:
sudo npm install typescript --global
Password:
/usr/local/bin/tsc -> /usr/local/lib/node_modules/typescript/bin/tsc
/usr/local/bin/tsserver -> /usr/local/lib/node_modules/typescript/bin/tsserver
+<EMAIL_ADDRESS>updated 1 package in 4.494s
Nicolass-MacBook-Pro:~ nicoj$ yo teams
internal/modules/cjs/loader.js:582
    throw err;
    ^

Error: Cannot find module 'typescript'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:580:15)
    at Function.Module._load (internal/modules/cjs/loader.js:506:25)
    at Module.require (internal/modules/cjs/loader.js:636:17)
    at require (internal/modules/cjs/helpers.js:20:18)
    at Object.<anonymous> (/Users/nicoj/node_modules/generator-teams/generators/app/index.js:1:2200)
    at i (/Users/nicoj/node_modules/generator-teams/generators/app/index.js:1:186)
    at Object.<anonymous> (/Users/nicoj/node_modules/generator-teams/generators/app/index.js:1:1219)
    at i (/Users/nicoj/node_modules/generator-teams/generators/app/index.js:1:186)
    at Object.<anonymous> (/Users/nicoj/node_modules/generator-teams/generators/app/index.js:1:2785)
    at i (/Users/nicoj/node_modules/generator-teams/generators/app/index.js:1:186)

###Additional Info###
$ yo doctor

Yeoman Doctor
Running sanity checks on your system

✔ Global configuration file is valid
✔ NODE_PATH matches the npm root
✔ Node.js version
✔ No .bowerrc file in home directory
✔ No .yo-rc.json file in home directory
✔ npm version
✔ yo version

Everything looks all right!

Appreciate any help!
Thanks

I fixed this issue by installing 'typescript' locally instead of global:
sudo npm install typescript
Hope this help someone else.

Please report bugs running a generator on the specific generator repository.
The Yeoman team works on the core system and unfortunately cannot maintain the ~5000 publicly available generators. Thanks for your understanding!


Upload Liquity front-end on Buildnet

Objective
To retrieve & deploy front-end on buildnet to test that everything is working well
How to

[ ] Fork & build front-end
[ ] Discuss with Designer if we want to custom things on the FE
[ ] Deploy front-end
[ ] Upload front-end on Massa
[ ] Test that everything is working well (end-to-end) - if you don't know how to, please ask other devs in Massa (ex: Pierre, Damir...)

How to be on liquity front-ends website: https://github.com/liquity/frontend-registry?tab=readme-ov-file

how to develop liquity front end: https://github.com/liquity/dev?tab=readme-ov-file#development

We're waiting on design customisation

0x5fcFBC6681Db668f133e1E741045CEAf71d3fdfF is the registered address as frontend liquity tag

We are waiting on someone to test it


Manually update version number to test autodeploy to dev instance (will still require a patch semver guidance)

Change Description:
see above
Closes GH(s): {{ FILL THIS IN }}
GH-159
UW-Directory Pull Request checklist

[x] I have run poetry run tox
[x] I have selected a semver-guidance: label for this pull request (under labels,
to the right of the screen)

Merging without review, no need to wait for this one.


Opengraph meta tags

For the sake of linking this on more botnet sites it would help if templates/header.html had some opengraph meta tags for clean looking embeds.
I think most sites use this to generate fancy embedded links so these should be all the tags we would need
<meta property="og:title" content="Based Cooking">
<meta property="og:site_name" content="Based Cooking">
<meta property="og:url" content="based.cooking">
<meta property="og:description" content="Only Based cooking. No ads, no tracking, nothing but based cooking.">
<meta property="og:type" content="article">
<meta property="og:image" content="">

The og:image on just needs a full link to an image so no URI's (/path/to/image.png). I usually just use the project logo from my Gitlab repos to not put any undue load on the server.
Reference to available opengraph tags: https://ogp.me/

Okay. I guess the content in description should be whatever description of the recipe there is above the ingredients list. This could be doable.


Autoware /Docker   installation  to DrivePX2

We are DrivePX2  algorithm developper  and trying  to  install  Autoware (Docker)  .
After installation finished with sucsessfully ,  we got  a error masage and
drive works related function (GMSL camera input  or  Lidar point cloud data )
can not work.
It's  seems that  Autoware can not find out driveworks APi at all.
Could you suggest us how to resolve the issue .

Could you tell me the execution steps and full error messages?

Please also tell me the version of Driveworks.

Thanks a lot for your quick response.
1.Driveworks version  is  driveworks-0.6.(SDK <IP_ADDRESS>)
2.Excution steps are following.
/install step is  as   "https://github.com/CPFL/Autoware/wiki/DRIVE-PX2-Docker"   .
/excution  step  is
$ autoware-docker.
$ ~/start_autoware.sh

Okey, in that state only RuntimeManager is up.
Did you run roslaunch autoware_driveworks_interface gmsl_interface.launch ?
cd autoware/ros
source devel/setup.bash
roslaunch autoware_driveworks_interface gmsl_interface.launch

Also, you needs to operate the RuntimeManager for starting the driver.

Deer Sir,
We tried flowing  excution process  but reslove it.
$ autoware-docker
[INFO]:Autoware-Driveworks module is found. Autoware can use driveworks.
[WARN]:Execute script with sudoers.
[sudo] password for nvidia:
[INFO]:The autoware-docker image [tier4cr.io/dpx2/autoware:1.6.2] was ready.
[INFO]:GMSL Camera initialization sequence is start.
.....autoware-docker: line 138: kill: (6604) - No such process
[INFO]:GMSL Camera initialization sequence was end.
ls: cannot access '/media/nvidia': No such file or directory
non-network local connections being added to access control list
access control disabled, clients can connect from any host
nvidia@nvidia:~$ cd autoware/ros
nvidia@nvidia:~/autoware/ros$ source devel/setup.bash
nvidia@nvidia:~/autoware/ros$ roslaunch autoware_driveworks_interface gmsl_interface.launch
[gmsl_interface.launch] is neither a launch file in package [autoware_driveworks_interface] nor is [autoware_driveworks_interface] a launch file name
The traceback for the exception was written to the log file

Does following directory exist in your image?
~/autoware/ros/src/driveworks/packages/autoware_driveworks_interface/launch
Please also refer to the videos below.
how to autoware with docker+px2
https://www.youtube.com/watch?v=3UO1K0lD2Qk
how to use gmsl with docker+px2
https://www.youtube.com/watch?v=-swTFYO3Fuc

detail error message was bellow.
nvidia@nvidia:~/autoware/ros$ rosrun autoware_driveworks_interface gmsl_interface
Initialize DriveWorks SDK v0.6.67
Release build with GNU 4.9.2 from v0.6.0-rc7-0-g51bd3aa against Vibrante PDK v<IP_ADDRESS>
TimeSource: monotonic epoch time offset is<PHONE_NUMBER>951033
TimeSource: PTP ioctl returned error. Synchronized time will not be available.
TimeSource: Could not detect valid PTP time source at 'eth0'. Fallback to CLOCK_MONOTONIC.
SDK: Resources mounted from /usr/local/driveworks-0.6/data/resources
nvrm_gpu: Bug 200215060 workaround enabled.
SDK: Create NvMediaDevice
egl::Display: found 2 EGL devices
egl::Display: use drm device: drm-nvdc
SDK: number of GPU devices detected 2
SDK: currently selected GPU device discrete ID 0
SAL: identified board as DrivePX2-TegraA
SensorFactory::createSensor() -> camera.gmsl, camera-type=ar0231-rccb,csi-port=ab,fifo-size=3,required-framerate=20,slave=0,output-format=yuv
CameraGMSL: required FPS = 20, resolution = 1920x1208
iscRootDevOpen: can't open /dev/isc-mgr.7.a - Device or resource busy
NvMediaISCRootDeviceCreate: Unable to open root device
nvmedia: ERROR: Init: Failed to create NvMedia ISC root device
Driveworks exception thrown: DW_SAL_CANNOT_CREATE_SENSOR: CameraGMSL: cannot connect to camera
Cannot create driver: camera.gmsl with params: camera-type=ar0231-rccb,csi-port=ab,fifo-size=3,required-framerate=20,slave=0,output-format=yuv
Error: DW_SAL_CANNOT_CREATE_SENSOR

ah, current driver setting fixed parameter is ar0231-rccb,csi-port=ab,fifo-size=3,required-framerate=20,slave=0,output-format=yuv.
It's a very bad implementation...;(
Also, I am planning to fix some bugs and implementation problems,
but I think that we can merge immediately.
Could you tell me the parameter for your camera(ov10640)?
I'll send you a libautoware_driveworks.so to set your parameter.

for  driveworks  API, we are using following comand and parameter ,that works well.
./sample_camera_gmsl --csi-port=ab --camera-type=ov10635
and   lidar ,
./sample_lidar_replay --protocol=lidar.socket --params=device=VELO_VLP16,ip=<IP_ADDRESS>,port=2368,scan-frequency=12.5 --show-intensity=true
that's is also works well .
those are our environment.

also   for recording of  driveworks  API  , we use
"params": "camera-type=ov10635,csi-port=ab,camera-count=1,output-format=yuv,async-record=1,fifo-size=6"


Go-like syntax to handle errors

Describe the feature
I suggest to add throw: boolean option so we can return the HTTP error instead of throwing it. This way users can get full type safety after implementation of Typed API definition.
Example:
const { data, error } = await $fetch('/api/v1', { throw: false })

or not throw by default:
const $api = $fetch.create<unknown, ApiDefinition>({
  baseURL: 'http://httpstat.us',
  throw: false
})

const { data, error } = await $api('/500')

error // -> error is typed here based on the ApiDefinition

data - 2xx response if OK, otherwise undefined
error - 5xx, 4xx response if not OK, otherwise undefined
Alternatives
Return array:
const [data, error] = $fetch('/api/v1', { throw: false })

Additional information

[X] Would you be willing to help implement this feature?

I like the idea. Only thinking it might be more suitable for $fetch.raw where we return full object (data body, headers and now possibly new headers). Combining two different return values for $fetch can increase the types complexity at very least but open to ideas.

I fully agree with @pi0.
@enkot, Generally APIs ErrorResponse schema is same for every routes. So, another way to  achieve it would be we expose ErrorResponse interface which will be used by raw to override the response type with ErrorResponse if success is false. ErrorResponse can overwritten via .d.ts file.
Currently the work around will be to let your api return status_code:number or success:boolean & make response a union and use it with ignoreResponseError to get correct typing.

By the way, you could support both syntaxes in the same return object:
const { data, error } = $fetch('/api/v1', { throw: false })
// And:
const [data, error] = $fetch('/api/v1', { throw: false })

Implementation:
export function createIsomorphicDestructurable<
  T extends Record<string, unknown>,
  A extends readonly any[],
>(obj: T, arr: A): T & A {
  const clone = { ...obj }

  Object.defineProperty(clone, Symbol.iterator, {
    enumerable: false,
    value() {
      let index = 0
      return {
        next: () => ({
          value: arr[index++],
          done: index > arr.length,
        }),
      }
    },
  })

  return clone as T & A
}

Source

Good point 👍
One more reference: https://vueuse.org/shared/makeDestructurable/#makedestructurable

@enkot Absolutely! That's the same implementation under the hood. The VueUse one just also has IE11 support, probably because VueUse supports Vue 2 still. :)

@enkot, Generally APIs ErrorResponse schema is same for every routes. So, another way to achieve it would be we expose ErrorResponse interface which will be used by raw to override the response type with ErrorResponse if success is false. ErrorResponse can overwritten via .d.ts file.

Agree, generally the same for all routes, but not 100%:

there are situations when, for example, the project uses gateways (aggregators) to different APIs and error response can be quite different.
an error response is just a different type of response (not 200), so why should it be treated differently in terms of types?


Installed versions are not always displayed in "correct order" with haxelib list

This is a very minor issue, if it can be considered an issue at all. The versions installed don't always show in the "correct order" (oldest first, newest last). Like you can see on this screenshot, it seems to work perfectly for every lib but nape, where 2.0.9 shows up on the very right instead of on the left:

I went ahead and added version sorting using current SemVer.compare


Mismatch in the comment and the code for properties of CohereCallResponseChunk class

In the code below, the comment says that the response property returns the response for the text-generation event type. But the code checks for Stream End (StreamedChatResponse_StreamEnd) event type.
@property
    def response(self) -> Optional[NonStreamedChatResponse]:
        """Returns the response for text-generation event type else None."""
        if isinstance(self.chunk, StreamedChatResponse_StreamEnd):
            return self.chunk.response
        return None

The similar thing goes here:
  @property
    def documents(self) -> Optional[list[ChatDocument]]:
        """Returns the documents for citation-generation event type else None."""
        if isinstance(self.chunk, StreamedChatResponse_SearchResults):
            return self.chunk.documents
        return None

So I believe the first one is actually correct. The StreamedChatResponse_StreamEnd chunk type has a response property that returns the NonStreamedChatResponse instance, which has the same type as the standard text-generation event (and the full response for the entire text-generation stream). Maybe this could be more clear, something like "Returns the full response for the stream-end event type else None."?
The second one should say "search-results event type" I believe, good catch!
Want to open a PR with the fix?


Ranking query is creating performance issues

The ranking query itself takes ~200-300ms to run each time. This is not overall response time for an HTTP request, but rather response time from the database itself.

More importantly, this query is executed for nearly every page view, because even for an individual effort view, we typically show an overall and gender rank.
Given the very high read/write ratio of ranking data, we should experiment with calculating and persisting this data each time split time data is created, updated, or destroyed. Because a change in one effort can affect the ranking for each other effort in the event, this operation will take some time--presumably longer than the existing ranking query. To avoid backups in saving split time data, the new query to calculate and persist ranking data should probably be run asynchronously using a Sidekiq worker, and in the case of batch updates, it should be invoked only after the batch is complete.

This has been fixed by persisting overall performance data and using a new, greatly simplified ranking query to take advantage of that data.


Application crashing While downloading the district wise report.

Example
In tamilnadu while trying to download theni district info report application crahes.

@Rajakumar1988 I guess this issue has been resolved. Can you check again now?

Closing because of inactivity


Domain Activation Error : At least one of your domains has a CAA record that does not include Amazon as an approved Certificate Authority

Before opening, please confirm:

[X] I have checked the custom domains troubleshooting guide.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

App Id
dojms6svkj6ag
Region
us-east-1
Step
Domain activation
Status
Failed
Issue or question
I tried to add the dns configuration, following all the steps of the platform.
But I've got an error on Domain Activation step. I have retried multiple times but in vain.

Error message

As shown in the screen it says that CAA record is misssing but I have addedd a CAA record and tried several times to reverify but it keeps on failing with the same error. You can check the CAA record propagation here
https://www.whatsmydns.net/dns-lookup/caa-records?query=coachbar.io&server=google
Additional information
No response

@hloriii can you please help me with this ?

I was able to solve this issue by allowing a  CAA record  to create a certificate for wildcard domain. I think this should be clearly mentioned in the amazon docs. I had to add a CAA record with issuewild as well to make this work. So folks if you are pointing to a subdomain keep in mind to add a CAA record for both issue and issuewild tags.

Hi @indrajitsinh-rathod 👋 , thanks for your continued patience and apologies for the inconvenience caused due to a lack of documentation on this issue/error.
We are working on updating our documentation for the same.