ninja cmake ignores compiler override

I want to be able to switch out cl for clang. However when I do
cmake -Bbuild --DCMAKE_C_COMPILER=clang-cl.exe -DCMAKE_CXX_COMPILER=clang-cl.exe
CMAKE_C_COMPILER is not recognized and the compiler tool change defaults to msvc.
-- Building for: Visual Studio 17 2022
-- The CXX compiler identification is MSVC 19.37.32822.0

CMake Warning:
  Manually-specified variables were not used by the project:

    CMAKE_C_COMPILER

Did you clear your CMakeCache.txt? Anyway: This works for me and there seems to be some issue on your side. But it's a general CMake question, nothing Ninja specific.


Playground/CodePen example for datapicker with Bootstrap 4

Need a playground sample for testing and experimentation, like something that runs on https://codepen.io, fiddler, or a similar site.
If you have one please reply and provide a link.  If not, I'll set one up and share it here for others to use.
Btw, I also assume there is no CDN url, other than using the site links:
https://fengyuanchen.github.io/datepicker/css/datepicker.css
https://fengyuanchen.github.io/js/common.js
https://fengyuanchen.github.io/js/datepicker.js
Thanks-

Ok here it is:
https://codepen.io/whitneyland/pen/XeeKjg
This makes is easy to experiment with the control in a playground environment.

Playground: https://fengyuanchen.github.io/datepicker/

That’s a great page as a demo! However would you agree it doesn’t let you use the date picker in a playground environment?
The idea is to make it instantly hackable.  Instantly try out options, styliing, examples.   This let’s people quickly see how they can benefit in their particular situation.


MapRazorPages() runtime exception

The "Routing startup code" > "Razor Pages" section seems incomplete.  I added "MapRazorPages()" to my endpoints as described, but "MapRazorPages" throws an exception at runtime saying the method "set_AllowAreas" is missing from RazorPagesOptions.  What am I missing?

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

ID: 76f39eec-9dd1-6167-56ec-47b26e0b5063
Version Independent ID: f9f2b0f3-01eb-cf58-8767-194fe0fc0e44
Content: Migrate from ASP.NET Core 2.2 to 3.0
Content Source: aspnetcore/migration/22-to-30.md
Product: aspnet-core
Technology: aspnetcore-migration
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Ah, I had the Microsoft.Identity.Web project in my solution which required .NET Core 2.2.  I unloaded that project and did not receive the runtime exception.


Create download directory if it doesn't exist

Create a download directory (~/Downloads by default) if it doesn't exist

It does refuse to start, since exception will be thrown there.

Also there is two places where daemon sets up directories , in init() and _setup_data_directory() , which is unnecessary.

Move the creation of directories to _setup_data_directory()
Move the initialization of stream_info_manager (which requires db_dir) to _setup_lbry_file_manager()


Graph View | Highlight that estimated values are estimations

Currently the user can choose do display estimated values. However, in the graph view the context that the values are estimated are lost. I propose to indicate this by pre-pending estimate values with (~) in the graph view.

Done


Extension host terminated unexpectedly.

I did everything. I deleted all data of Visual Studio code, deleted extensions manually, on reinstalling it works at the very first instance but after turning off my PC and switching back, again same error occurs also my extension fails everytime

You can open a issue in the repository of extension.

Please check if you are seeing this: https://github.com/microsoft/vscode/issues/94858


bench: create a tool to compare benchmark between different revisions

I also wanted to integrated into CI, but I want to get this done first.


Does not seem to work with react 16.8.6

https://codesandbox.io/s/no78qmnkm?fontsize=14
Am I missing something?

never mind, I thought it would render a tooltip but it just render positioned div.

https://github.com/mohsinulhaq/react-popper-tooltip
you can use this library if you want to render tooltips easily


Dependency with xcode build requirement causes formula to be skipped

The qt formula has a :build requirement on :xcode, but recently (possibly since #14), I am unable to build a bottle for a formula that depends on qt ( https://github.com/osrf/homebrew-simulation/pull/261 ) on a build machine that only has the command-line tools installed. The build fails with the following console message:
==> SKIPPING: osrf/simulation/ignition-gui
A full installation of Xcode.app is required to compile this software.
Installing just the Command Line Tools is not sufficient.
Xcode can be installed from the App Store.

It should be fine to build without full Xcode since qt is being installed from a bottle. I can try to help with some guidance. I'm not sure if the problem is here or in brew.

Maybe some logic from https://github.com/Homebrew/brew/pull/2815 could be added to the test-bot?

@scpeters Can you fill out the issue template, please?

I didn't see a template when I clicked on new issue.

I could file in a different repo that has issue templates if you prefer.

Apologies, I misread this @scpeters. https://github.com/Homebrew/homebrew-test-bot/blob/80333911e7130b9542a6fe2b94c1311b52cef319/cmd/brew-test-bot.rb#L554-L578 is the relevant code; I agree that this code could be adjusted to handle built-time requirements for bottles.

No problem; I'll take a look at that code and work on a PR.

I thought https://github.com/Homebrew/brew/pull/3479 would fix this, but it didn't. I still need to make another attempt at #116.


⚠ Samsung has degraded performance

In e30f467, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2590 ms

Resolved: Samsung performance has improved in 041a6d6.


Added Minutes per File Summary #29

Fixes #29

would be great this view could be sorted per default by minutes descending, that way the most expensive workflow would be at the very top


Make SwifterError a LocalizedError

The current SwifterError is not a LocalizedError and calling localizedDescription does not return the expected description.

LGTM 👍


Converting all actors on map fails with error

If you click the convert all tokens on map button, you just get a large number of errors saying:
Error: [jqzbY4CXoA1Ho2yn] Invalid TokenData displayBars which must be a value in CONST.TOKEN_DISPLAY_MODES
    at TokenData._validateField (data.mjs:384:13)
    at TokenData.validate (data.mjs:275:14)
    at ClientDatabaseBackend._preUpdateDocumentArray (foundry.js:10197:18)
    at ClientDatabaseBackend._updateEmbeddedDocuments (foundry.js:10152:33)
    at ClientDatabaseBackend.update (backend.mjs:153:31)
    at async Function.updateDocuments (document.mjs:373:21)
    at async TokenDocument5e.update (document.mjs:456:21)

None of the tokens change.

This is using version 1.0.1 of the module.

Released version 1.0.2 fixing this issue! Thanks for pointing me to it. 💯


fix checkbox (toggle group) event propagation on mobile devices

1. Why is this change necessary?
This fixes the cookie configuration checkboxes on mobile devices when selecting a cookie group (e.g statistics) which will then toggle all checkboxes in that subgroup.
2. What does this change do, exactly?
This event doesn't propagate when using touchstart but will when using click. I don't see the reason for device detection here as touchstart is included in the click event chain (without the trouble).
3. Describe each step to reproduce the issue or behaviour.

Add statistics cookies
Switch to a mobile device (or emulate)
Click on statistics
See that the checkboxes for the individual cookies are not checked by pressing the arrow icon
Repeat on desktop and see that it works

4. Please link to the relevant issues (if any).
5. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-9634
Please use this issue to track the state of your pull request.

Thank you for your contribution @MDSLKTR :tada: :+1:


VC Storage

Where should ASCS (or any other issuer) store the verifiable credentials? The initial idea is that vcs shouldn't be publicly available but be only provided upon valid request.

I think for now we can skip the discussions about under which circumstances ASCS provides a VC to a party which asks for it. For now, let's assume that every company (i.e., VC holder) always provides their VC.
The question here is, where should ASCS store the VCs it issues? A Mongo DB instance could be one option as VCs are in JSON format.


[Alternative] - Solve issue 5 with Colorama.

Fixed issue #5, using Colorama. Notice there is another PR using Hue here => #62 to resolve the same issue. I think both of them work well on Windows 10 with PowerShell and Command Prompt. I have tested both softwares but I have not tested on the Linux machine. Since Colorama is well known as cross OS and my PC crashed yesterday while installing Python 3.7 (Damn it!), I would not worry too much about it but if anyone would like to tested on their Linux distros and MacOS are welcome.

Tnx @jcs090218 , It worked good on MacOS and Linux. I thinks this would be the solution.
@nareddyt, Can you try it on windows and review this PR?

@TheYahya No problem! I am concerned about the readability https://github.com/TheYahya/sherlock/issues/5#issuecomment-451051326 and OS compatibility, I did some research and made a simple list of two color libraries' comparison below.
Hue

Works on Windows 10/Linux/MacOS. (Lose)
Supports bright colors. (Win)

Colorama

Works on Windows/Linux/MacOS. (Win)
No supports bright colors. (Lost)

I personally reckon and prefer Colorama's style more than Hue's style https://github.com/TheYahya/sherlock/issues/5#issuecomment-451051326, and beyond that, OS compatibility is way more important than weather the bright colors is supported or not, in my opinion. As the matter of fact, both of the color libraries are working on their defects. Colorama has the advantage at the moment but maybe it would not be sooner. Hope these information would help!

Works great on Windows Command Line!
@TheYahya Code looks good and is ready to merge. Unfortunately, I do not have permission to add a review on the PR. You would have to add me as a collaborator in the settings for this repo. Feel free to add me if you'd like, no worries otherwise :)
For documentation purposes, here's the output on Windows Command Line with this change.
Before

With this PR

Tnx @jcs090218 😊
And thank you @nareddyt, I add you to collaborators too. you have full access to the repo now, so use it ☺

Tnx @jcs090218 😊
And thank you @nareddyt, I add you to collaborators too. you have full access to the repo now, so use it ☺

Let me know if there's any specific issues / PRs you need help with! As I mentioned to @sdushantha, I don't have much time to dedicate to this project, but I'm certainly willing to help! Also feel free to point any new contributors asking for help with git/github to me, I can walk them through the process 👍

After this update, the coloring no longer works in my Git Bash shell.  I am using Windows 10 with Git For Windows 2.17.0.windows.1 (https://gitforwindows.org/).
It does work fine in my Git CMD window, or a regular Windows command shell.  This is odd, as before the update the coloring only worked in the Git Bash shell, and acted weird in the other shells.
Personally, I was never a fan of the colors.  But I know that there are people that like this feature, so I am mentioning it.

@hoadlck Thank you for reporting this, I will look into it and see if this could be resolved. I am really confused at the moment, and not quite sure there are other alternatives could get around this issue...


Update to RC2

This pull request contains @mkjeff and @bearTM's changes.
Currently only tested that src/Swashbuckle.SwaggerGen and src/Swashbuckle.SwaggerUi restores and builds, and packs normally using dotnet.
Uses netstandard1.5 but contains few wild imports.
Btw, it is advised to use as low "netstandard1.x" version as possible for libraries to achieve maximal portability, so when this is released one has to test & try with lower versions.

I'm pulling my hair out with this RC2 debacle and am tempted to just wait until a stable version is released. I've pulled down these bits but can't get it to build. Lots of errors like this:
The type or namespace name 'System' could not be found (are you missing a using directive or an assembly reference?

Did you come across any build issues - any gotcha's you can share?

@domaindrivendev I feel for you. For some reason I'm not yet bald, I tried the ASP.NET Core when it was beta2 or something, and stopped trying because it always broke.
Where did you get the errors? Also there is a lot of helpful people at JabbR AspNetCore
C:\DepSource\SwashbuckleRC2\src\Swashbuckle.SwaggerUi>dotnet --version
1.0.0-rc2-002498

C:\DepSource\SwashbuckleRC2\src\Swashbuckle.SwaggerGen>dotnet restore
log  : Restore completed in 3092ms.

NuGet Config files used:
    C:\DepSource\SwashbuckleRC2\NuGet.Config
    C:\Users\jarppa\AppData\Roaming\NuGet\NuGet.Config

Feeds used:
    https://www.myget.org/F/aspnetcirelease/api/v3/index.json
    https://api.nuget.org/v3/index.json

C:\DepSource\SwashbuckleRC2\src\Swashbuckle.SwaggerGen>dotnet restore
log  : Restore completed in 2083ms.

NuGet Config files used:
    C:\DepSource\SwashbuckleRC2\NuGet.Config
    C:\Users\jarppa\AppData\Roaming\NuGet\NuGet.Config

Feeds used:
    https://www.myget.org/F/aspnetcirelease/api/v3/index.json
    https://api.nuget.org/v3/index.json

Installed:
    2 package(s) to C:\DepSource\SwashbuckleRC2\src\Swashbuckle.SwaggerUi\project.json

I can try to make this work with tests you've done there also, that I didn't try.

RC2 dropped today.  Is there an ETA on getting this pull request merged?

Looking at the various PR's, it looks like a lot of people are duplicating work - I also have a local RC2 branch in progress. My bad for lack of communication. With two small kids there just aint enough hours in the day - even to just get the appropriate communication out there :(
I won't be accepting any PR's that don't come with a green suite of tests so that makes #88 the likely candidate. I'm going to pull it down tomorrow and compare with my own local branch before merging one of them into master. Either way, I'm targeting 5/28 for an RC2 compatible release to Nuget. See the beta2 milestone here https://github.com/domaindrivendev/Ahoy/milestones. Note the revert to "beta" status - still lots of feature gaps and a config. API that will likely undergo change before we reach a stable release.
Thanks to everyone for chipping in. Please don't get disheartened if I'm slow to respond to PR's. So long as they include tests and a passing suite, I'll get always consider them and strive to respond as soon as I can.
I have one significant architectural refactor that I want to get done after the beta2 release. After this, I'll be looking to open up the project and add additional contributors. For anyone reading this, please let me know if you're interested?
Ahoy!
Richie


[question]How to pre-train? What dataset should we use?

Hi rykov8,
Do you know how to  pre-train the vgg part in ssd network? And what dataset do you use to do that?
Thanks a lot.

@KevinYuk the authors use this model. I'm not sure, but I suppose that it is pre-trained on ImageNet, but you'd better check.

@rykov8 OK. Thanks for your reply. I'll try that.


Deploment failure VS 2019

Everything seemed to go well following these steps. But Deploying fails with te following error:
"An error occurred while trying to deploy the app 'App1.iOS.app'. Details: Error running winiosterminal: 'An error occurred while deploying the application 'App1.iOS.app' into 'brendan's iPhone'. Details: An Afc error occurred. The error code was ObjectNotFound'. Command: 'deploy -u=383dff3c9a96a1414278e25cef7f83deec5fa825 -i=com.bzsoftware.app1 -n=app1.iOS.app'"
The "App1" iOS app works well otherwise if I Build and Distribute it via the AppCenter and install it on the same iPhone 8 device.
SysInfo: Visual Studio 2019 16.5.1, Windows 10, iPhone 8, iOS 13.3.1

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

ID: 8f3eb808-8e6d-4e07-fc75-8d79a7249265
Version Independent ID: dbb2b9a9-87a6-e473-50e6-f88435d49e88
Content: Xamarin Hot Restart - Xamarin
Content Source: docs/xamarin-forms/deploy-test/hot-restart.md
Product: xamarin
Technology: xamarin-forms
GitHub Login: @maddyleger1
Microsoft Alias: maleger

I am getting the same error.  My device is an iPhone 5S.

Hi folks - can you please file this via Help > Send Feedback > Report a Problem in the IDE? Thanks!

i have the same issue, i want to know how to resolve it...

Same issue with iPhone 5S.

Hi,
I have the same issue.
I have the Xamarin.Forms app and use the Xamarin Hot Restart to run the iOS app on iPhone from Visual Studio on Windows. And I also have the deployment issue. But the message is slightly different:
An error occurred while trying to deploy the app 'KC.Mobility.Sales.iOS.app'. Details: Error running winiosterminal: 'An error occurred while deploying the application 'KC.Mobility.Sales.iOS.app' into 'IPR iPhone7+ Test Device'. Details: An Afc error occurred. The error code was EmptyResponse'. Command: 'deploy -u=96346626f88f9f139d81b4d8b9d64dda905679c7 -i=com.yourcompany.KC.Mobility.Sales -n=KC.Mobility.Sales.iOS.app'
Microsoft Visual Studio Enterprise 2019
Version 16.5.4
VisualStudio.16.Release/16.5.4+30011.22
Xamarin <IP_ADDRESS>3 (d16-5@9152e1b)
Xamarin Designer <IP_ADDRESS>0 (remotes/origin/d16-5@681de3fd6)
Xamarin Templates 16.5.49 (0904f41)
Xamarin.Android SDK <IP_ADDRESS> (d16-5/988c811)
Xamarin.iOS and Xamarin.Mac SDK <IP_ADDRESS> (b75deaf)
Device: iPhone 7 Plus.
iOS version 13.4.1

I have the same issue and it doesn't seem to depend on the device I use.  I've tried two Ipads and I get the same error on when trying to deploy to both.

@GaboMendez @aherrick @avorobjovs @nhadro thanks everyone for the feedback - I know this is really frustrating. Unfortunately this is for issues with the documentation itself, not the product!
Instead of reporting here, can you please file this via Help > Send Feedback > Report a Problem in the IDE? Thanks!
We're aware of this problem and there is a fix in the works but you can get the most up-to-date info on Developer Community.

I think what's happening is that if you perform a clean and then build again, you'll find that it's building the app with a .exe extension. Then, when attempting to deploy, it's looking in the debug folder for [your app name].app.  Not sure how to fix that just yet though.


Can I use with RecyclerView?

Hey, in you example you use ListView...
I would like know if is possible use with:

RecyclerView
GridView

Hello @hjJunior
That's a good question, I never tried myself. You can try I guess ^^
Cheers


Fixing onSuccess delete method

fixing https://github.com/vuex-orm/plugin-axios/issues/63

It would be nice if this PR could me merged 🤞 🙏


fix(tslint): add correct semicolon option

The first parameter in the semicolon option in tslint.json should be a boolean.
Please read and mark the following check list before creating a pull request (check one with "x"):

[X] I read and followed the CONTRIBUTING.md guide.
[ ] I read and followed the New Feature Checklist guide.

Short description of what this resolves:
Fixes the typescript linter

Hey @eduhenke, so is it correct that it wasn't working properly before this fix and now it checks for the semicolon presence?

@nnixaa yep, exactly :)

@eduhenke ah, good catch!


Gazebo crashing with violent WAM-V motion

Updates:

I managed to reproduce the crash without hitting any buoys. I did it spinning the WAM-V violently while braking and accelerating again causing some pitching.

While testing #692 I also managed to produce a crash in the acoustic tracking task world while moving relatively slowly. After a number of collisions the WAMV seems to destabilize and eventually tilts sideways under the water and crashes the simulation. A wild guess, but I wonder if this might also have something to do with the tuning of the drag coefficients, as described by @tejalbarnwal in #689 . The WAMV seems to be sitting high on the water and when the thrust is strong its movement seems strange.

See a workaround patch in #706 .

Fixed in #706


Replace KnownIsNot* with *IsPlausible in pyhanabi.h.

This matches what pyhanabi.py was calling and what pyhanabi.cc was implementing.
It applies to both color and rank.
Also, change them (along with *WasHinted) to return bool instead of int since that's what the underlying functions return.

Fix for ColorIsPlausible and RankIsPlausible already added separately: I looked at issues before pull requests. Thank you for also catching this.
Re. bool vs int, I would prefer to leave the use of int-as-bool in the CFFI interface: bool support in CFFI seemed both newer, and a bit rougher compared to support for int. For example, one supposedly up-to-date machine I tested on had cffi 1.9, which does not seem to support C++ bool.
Re. obs_dict, I would prefer to leave the RL interface untouched unless necessary, even if it might not be the most aesthetically pleasing choice. The extended hint information is exposed in both the vectorized observation and the pyhanabi observation. If I was to change the RL interface, my personal preference would actually be to remove almost all of the obs_dict entries as they're duplicates of the HanabiObservation object.

If I was to change the RL interface, my personal preference would actually be to remove almost all of the obs_dict entries as they're duplicates of the HanabiObservation object.

In fact, the duplication will also make the Python interface pretty slow.
In [1]: import rl_env

In [2]: env = rl_env.make()

In [3]:
   ...: def run_actions(env):
   ...:     obs = env.reset()
   ...:     for _ in range(1000):
   ...:         a = random.choice(obs["player_observations"][obs["current_player"]]["legal_moves"])
   ...:         obs, r, d, info = env.step(a)
   ...:         if d:
   ...:             obs = env.reset()
   ...:

In [5]: import random

In [6]: %timeit run_actions(env)
760 ms ± 8.97 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

It's almost 20 times slower compared to the thin Julia wrapper.

Environment
mean time
median time
memory
allocs

HanabiEnv()
37.900 ms
36.825 ms
2.98 MiB
7999

You might see hanabi.jl if you are interested.


Reduce number of replies on MinMds query processing.

UDP packets are received on every listening interface, resulting in single queries being multipled by the interface count and this causes more packets than expected being generated.
This change will only process queries that are originating from the same interface that the mdns endpoint is bound to.

Marking as ready for review as this reduces number of replies in minmdns.
I am still wondering if maybe there is an update potential for minmdns to use less udp sockets and listening ports as I would expect addr_any to cover listening over more interfaces (the existing behavior seems to cover that).

obsolete after #27219

Re-opened as the other PR seems to have issues on OpenIOT. The error of Failed to reply to query that I get there is not helpful, so I am unsure on how long this will take to debug. Maybe once we have better tracing/logging piped through we can actually see what queries and answers are seen.

can we cherry pick this PR on v1.1-branch ?


One of my templates uses a partial which needs some data sent into it.

I am going to create the ability to set some templateDefaults against the options used to create a FormController.

I would recommend using the locals hook on the form controller to do this - https://github.com/UKHomeOffice/passports-form-controller/blob/master/lib/form.js#L87-L89
eg.
MyController.prototype.locals = function (req, res) {
  return { stepNumber: 2 };
};


in python, mapdl.input_strings(cmd), cmd is a very long command. If it is not called in the method, it will be executed normally. If it is put in the method, the error "ansys.mapdl.core.errors.MapdlExitedError: MAPDL server connection terminated" will be reported.

🔍 Before submitting the issue

[X] I have searched among the existing issues
[X] I am using a Python virtual environment

🐞 Description of the bug
in python, mapdl.input_strings(cmd), cmd is a very long command. If it is not called in the method, it will be executed normally. If it is put in the method, the error "ansys.mapdl.core.errors.MapdlExitedError: MAPDL server connection terminated" will be reported.
📝 Steps to reproduce
1，Define a string containing multiple commands
2，Then mapdl calls input_strings(cmd) to execute
💻 Which operating system are you using?
Windows
📀 Which ANSYS version are you using?
2023R1
🐍 Which Python version are you using?
3.11
📦 Installed packages
ansys

Transferring to PyMAPDL repository - pinging @ansys/pymapdl-maintainers


It should not be possible to model inputOutput mappings for gateways

Right now we are able to model camunda:inputOutput on gateways. This is not allowed by the engine and results in an error during deployment.
Gets addressed by https://github.com/bpmn-io/bpmn-js-properties-panel/issues/221.

Waiting for pending properties panel release, cf. https://github.com/bpmn-io/bpmn-js-properties-panel/commit/a7b5d7bfe2fd599f85a942d42eef8e1570a15ae0

Closed via https://github.com/camunda/camunda-modeler/pull/1468.


Error in org.elc upon loading org-mode files

What did you expect to happen?
I use org-roam to manage my notes, I don't usually upgrade at all, but after an upgrade after a while, I'm seeing an error upon either issuing org-journal-new-entry, or loading an orgmode file.
What actually happened?
Error whenever I issue org-journal-new-entry:
org-babel-read: End of file during parsing: /Users/Popeye/.emacs.d/.local/straight/build/org-mode/org.elc
Failed to load ’magit’ package incrementally, because: (end-of-file /Users/Popeye/.emacs.d/.local/straight/build/org-mode/org.elc)
Failed to load ’org’ package incrementally, because: (end-of-file /Users/Popeye/.emacs.d/.local/straight/build/org-mode/org.elc)

Or, if I load an orgmode file:
File mode specification error: (end-of-file /Users/Popeye/.emacs.d/.local/straight/build/org-mode/org.elc)

Additional details:
My configuration:

config.el
packages.el
init.el

Steps to reproduce:

Copy my configurations (above) into ~/.doom.d/
Clone Doom Emacs into ~/.emacs.d.
Install Doom Emacs: ~/.emacs.d/bin/doom install
Load Emacs.
Either issue org-journal-new-entry or open an org-mode file, and observe the errors mentioned above.

System information:

((emacs
  (version . "26.3")
  (features . "JPEG RSVG IMAGEMAGICK GLIB NOTIFY ACL GNUTLS LIBXML2 ZLIB TOOLKIT_SCROLL_BARS NS MODULES THREADS LCMS2")
  (build . "Mar 11, 2020")
  (buildopts "--disable-dependency-tracking --disable-silent-rules --enable-locallisppath=/usr/local/share/emacs/site-lisp --infodir=/usr/local/Cellar/emacs-plus/26.3/share/info/emacs --prefix=/usr/local/Cellar/emacs-plus/26.3 --with-xml2 --without-dbus --with-gnutls --with-imagemagick --with-modules --with-rsvg --with-ns --disable-ns-self-contained")
  (windowsys . batch)
  (daemonp))
 (doom
  (version . "2.0.9")
  (build . "HEAD -> develop, origin/develop, origin/HEAD 95547823c 2020-05-18 20:11:05 -0400")
  (dir . "~/.doom.d/"))
 (system
  (type . darwin)
  (config . "x86_64-apple-darwin18.7.0")
  (shell . "/bin/zsh")
  (uname . "Darwin 18.7.0 Darwin Kernel Version 18.7.0: Thu Jan 23 06:52:12 PST 2020; root:xnu-4903.278.25~1/RELEASE_X86_64 x86_64")
  (path "~/.cargo/bin" "~/.gem/ruby/2.3.0/bin" "/usr/local/opt/gettext/bin" "~/Code/sonar-scanner/bin" "~/.gem/ruby/2.3.0/bin" "/usr/local/bin" "/usr/local/sbin" "/usr/local/opt/cython/bin" "/usr/local/opt/sqlite/bin" "/usr/local/opt/openssl/bin" "/usr/local/opt/libiconv/bin" "/usr/local/opt/e2fsprogs/sbin" "/usr/local/opt/e2fsprogs/bin" "/usr/local/opt/openssl/bin" "/usr/local/opt/gettext/bin" "~/.poetry/bin" "~/.rbenv/shims" "~/.rbenv/bin" "/usr/local/Cellar/ruby/2.6.3/bin" "/usr/local/bin" "/usr/bin" "/bin" "/usr/sbin" "/sbin" "/Library/TeX/texbin" "/opt/X11/bin" "/Applications/Wireshark.app/Contents/MacOS" "/usr/local/bin" "/usr/local/opt/go/libexec/bin" "/usr/local/Cellar/emacs-plus/26.3/libexec/emacs/26.3/x86_64-apple-darwin18.7.0"))
 (config
  (envfile . envvar-file)
  (elc-files . 0)
  (modules :completion company (ivy +fuzzy) :ui deft doom hl-todo indent-guides modeline nav-flash ophints (popup +defaults) (pretty-code +iosevka) treemacs unicode vc-gutter vi-tilde-fringe window-select workspaces zen :editor file-templates fold snippets :emacs (dired +ranger +icons) electric undo vc :checkers (syntax +childframe) :tools (eval +overlay) (lookup +dictionary) (lsp +peek) (magit +forge) pdf :lang data emacs-lisp (javascript +lsp) (latex +latexmk +cdlatex +lsp +fold) (markdown +grip) (org +brain +noter +dragndrop +journal +roam +present) (python +lsp +pyenv +poetry) (sh +lsp) yaml :config (default +bindings +smartparens))
  (packages (emacsql-sqlite3) (org-download) (almost-mono-themes))
  (unpin "n/a")
  (elpa "n/a")))

This suggest that org didn't clone or byte-compile correctly. Try deleting ~/.emacs.d/.local/straight/repos/org-mode and ~/.emacs.d/.local/straight/build/org-mode manually, then run doom sync.

Recompiled, I'm still getting an error:

org-babel-read: End of file during parsing: /Users/Popeye/.emacs.d/.local/straight/build/org-mode/org.elc
Failed to load ’org’ package incrementally, because: (end-of-file /Users/Popeye/.emacs.d/.local/straight/build/org-mode/org.elc)
Failed to load ’org-agenda’ package incrementally, because: (end-of-file /Users/Popeye/.emacs.d/.local/straight/build/org-mode/org.elc)
Failed to load ’org-capture’ package incrementally, because: (end-of-file /Users/Popeye/.emacs.d/.local/straight/build/org-mode/org.elc)
Failed to load ’org’ package incrementally, because: (end-of-file /Users/Popeye/.emacs.d/.local/straight/build/org-mode/org.elc)
Mark set

I'm afraid this error makes little sense to me. I cannot reproduce it. A End of file during parsing error means there's a syntax error, but one in machine-generated, byte-compiled code? I'm stumped.

You could try adding this to ~/.doom.d/packages.el:
(package! org-mode
  :recipe (:host github
           :repo "emacs-straight/org-mode"
           :files ("*.el" "lisp/*.el" "contrib/lisp/*.el")
           :no-byte-compile t))

This will prevent the org package from being byte-compiled.

That didn't help either; the error just started being reported in the equivalent .el file.
If this helps: I seem to no longer have the error if I disable the +roam submodule form the (org +...) module in :lang inside init.el. The error persists if I install org-roam from the github repo instead too.

Can you try updating to the latest commit of Doom (where I've recently bumped org-roam and others)? Perhaps it was fixed upstream.

Caught it. One of the org files inside the org-roam notes repository looked like this:
#+TITLE: subject
#+ROAM_ALIAS: subject1" "subject2" "subject3"

Closing the quote fixed everything:
#+TITLE: subject
#+ROAM_ALIAS: subject1" "subject2" "subject3"

Goes to show how important the cleanliness of the org-roam notes really is.
Thanks for the follow-along and the suggestions, @hlissner.

Ah, that would explain it. I'm glad you figured it out, and thanks for letting me know!


refactor(tooltip): tooltip-v2 - aligned to how popover functions

@pimenovoleg @mrfrac @artembelik
This is an RC version of tooltip-v2 ready to have a lot in common with popover. Works perfect :)
Once you approve it, I will remove the current tooltip and rename tooltip-v2 to tooltip
CAUTION: I've left CloseDirective as it works fine with external triggering enabled. Just to showcase - can be removed.
https://github.com/user-attachments/assets/1de2653c-60c3-4633-9c9b-6bc5889a2183

Nice, this version came out super quickly! :)
While reviewing the examples, I noticed a strange indentation here. Is everything okay with it?
In multiple section, secondary example.

Nice, this version came out super quickly! :)
While reviewing the examples, I noticed a strange indentation here. Is everything okay with it? In multiple section, secondary example.

@pimenovoleg yes, that's ok because the second and the third of the multiple case story have side offset and align offset applied. Maybe I should add an info about it to them, what do you thing? You feel confused w/o that info so the other can also feel the same, I think :)

Nice, this version came out super quickly! :)
While reviewing the examples, I noticed a strange indentation here. Is everything okay with it? In multiple section, secondary example.

@pimenovoleg yes, that's ok because the second and the third of the multiple case story have side offset and align offset applied. Maybe I should add an info about it to them, what do you thing? You feel confused w/o that info so the other can also feel the same, I think :)

@pimenovoleg
would this be ok?

btw @pimenovoleg when I click deployment vercel links I see this:

It would be nice to have an access but w/o it is also ok :)

Nice, this version came out super quickly! :)
While reviewing the examples, I noticed a strange indentation here. Is everything okay with it? In multiple section, secondary example.

@pimenovoleg yes, that's ok because the second and the third of the multiple case story have side offset and align offset applied. Maybe I should add an info about it to them, what do you thing? You feel confused w/o that info so the other can also feel the same, I think :)

Yes, I think adding some information about the side and align offsets would be really helpful. It makes the examples more self-explanatory, especially for someone reviewing them for the first time. Maybe a short comment or a note in the documentation near the example would work well. What do you think? 😊

Yes, I think adding some information about the side and align offsets would be really helpful. It makes the examples more self-explanatory, especially for someone reviewing them for the first time. Maybe a short comment or a note in the documentation near the example would work well. What do you think? 😊

Yes, I definitely agree :)
What about this:

@pimenovoleg would this be ok?

Make sense? :)

btw @pimenovoleg when I click deployment vercel links I see this:
It would be nice to have an access but w/o it is also ok :)

I changed some settings, please try creating a PR from a branch instead of a fork next time.

@pimenovoleg can I continue on this by removing the current tiiltip and replacing it with the v2? Then I wanna create hover card and next that one https://github.com/radix-ng/primitives/issues/236

@pimenovoleg can I continue on this by removing the current tiiltip and replacing it with the v2? Then I wanna create hover card and next that one #236

lets do it 👍

@pimenovoleg ready to CR


could it be possible to add hacktv pipe support?

would this project be able to generate the tbc from ether video or hacktv output and pipe it to the hacktv as input
for use with analouge transmition?
https://github.com/fsphil/hacktv
could this be possible?
thank you.
Sincerly,
Michael

With the latest commit (6e0deca) I was able to do it. Here's the command:
ffmpeg -re -f lavfi -i pal100bars=s=720x576 -f rawvideo -pix_fmt rgb24 - | \
Unai.VITC -s 720x576 -f R8G8B8 -I -i - | \
ffmpeg -f rawvideo -pixel_format rgb24 -video_size 720x576 -i - -f matroska -c:v mjpeg - | \
hacktv - -f 854e6 -g 24 -m b

The first FFmpeg instance converts the input video to a format that Unai.VITC can understand (raw 24-bit RGB).
Then, Unai.VITC embeds the VITC signal to the video (resolution is unchanged) and outputs the result in the same format.
After that, the second FFmpeg instance converts the resulting video stream to a format that HackTV can understand (it uses FFmpeg for decoding anyway, so I guess you could change the parameters if you want).
And finally, HackTV takes that stream and sends it to the HackRF.

Also, HackTV recently added support for VITC. Although it is not as customisable as this program, it's definitely closer to what an actual VITC signal looks like.

thank you so much!
This is actully very helpful to know...
but i was refering to a generated and/or simulated tbc (Time Base Correction) for hacktv's basband composite output.
https://www.digitalfaq.com/forum/video-restore/2251-tbc-time-base.html#:~:text=A 'timebase corrector' or 'time base corrector' --,then it is corrected before being output again.
Would be interesting to see if this is accually possible with hacktv!
@fsphil What Do You Think Phil?
Can it be done?
;)

Oops... I didn't read the TBC part. But I still don't understand what does "generating the TBC" mean.
If you want to insert VITC signals into the baseband output of HackTV (something that looks like this)…:

…I can easily implement that in a future commit.
Now, video signal correction would require a totally different program, since this one only takes care of generating a VITC signal and optionally insert it into a video stream.

@03stevensmi We have had fully software-defined TBC for composite signals for 3 years now and is widely in use via vhs-decode / ld-decode & lastly Composite or cvbs-decode it has replaced using hardware TBCs for a while but LS doesn't really like that so digital FAQ is kinda in this post software era keeping outdated info alive lol.
We have the TBC format in 16-bit unsigned values you can use with SDR hardware, and you can make standard video into TBC files via the ld-chroma-encoder.

@03stevensmi We have had fully software-defined TBC for composite signals for 3 years now and is widely in use via vhs-decode / ld-decode & lastly Composite or cvbs-decode it has replaced using hardware TBCs for a while but LS doesn't really like that so digital FAQ is kinda in this post software era keeping outdated info alive lol.
We have the TBC format in 16-bit unsigned values GREY16 you can use with SDR hardware, and you can make standard video into TBC files via the ld-chroma-encoder (there is also a windows release of the ld-tools suite btw)
Example of the signal in CVBS

VITC on SVHS

@harrypm Thank you so much,
I had no idea this was already a thing thank you.
You mentioned this working with sdr?
How (or would) this be compatible with use on hacktv's fl2k usb2vga output?
If so,
What would I need to do in short?
Once again, thank you for the info.
Really has been a massive help

@03stevensmi Yeah we have been using a modified FL2K driver TBC to FL2K for Composite/S-Video playback it's iffy on PVMs with some files but if run through modern ICs in TV's it's very stable, this originated with a GNU Radio script.
The commands for that are pretty copy-paste, but the TBC format is not 4fsc sampling exactly the specs are:
1135x625 For PAL (17727262 Hz) (25i) (560mbps) (4.2GB/min) (252GB/hour)
910x525 For NTSC (14318181 Hz) (29.97i) (453mbps) (3.4GB/min) (204GB/hour)
(Also the FL2K does not output the proper signal voltage its like 0.5~0.7vpp when CVBS is always 1vpp for live signals it really is a cheap way to do it but it's not the most compliant to the standard)


[bug]  Uncaught (in promise) TypeError: window.TAURI_IPC is not a function

Describe the bug

Reproduction
No response
Expected behavior
No response
Platform and versions
yarn run v1.22.18
$ tauri info

Environment
  › OS: Mac OS 12.4.0 X64
  › Node.js: 16.10.0
  › npm: 7.24.0
  › pnpm: Not installed!
  › yarn: 1.22.18
  › rustup: 1.24.3
  › rustc: 1.61.0
  › cargo: 1.61.0
  › Rust toolchain: stable-x86_64-apple-darwin

Packages
  › @tauri-apps/cli [NPM]: 1.0.0-rc.13
  › @tauri-apps/api [NPM]: 1.0.0-rc.6
  › tauri [RUST]: 1.0.0-rc.14,
  › tauri-build [RUST]: 1.0.0-rc.12,
  › tao [RUST]: 0.9.1,
  › wry [RUST]: 0.17.0,

App
  › build-type: bundle
  › CSP: default-src blob: data: filesystem: ws: wss: http: https: tauri: 'unsafe-eval' 'unsafe-inline' 'self' img-src: 'self'
  › distDir:
  › devPath: http://<IP_ADDRESS>:8080/
  › framework: Vue.js (Vue CLI)
  › bundler: Webpack

App directory structure
  ├─ isolation-dist
  ├─ node_modules
  ├─ public
  ├─ src-tauri
  ├─ .git
  └─ src
✨  Done in 15.37s.

Stack trace
No response
Additional context
No response

Does it still happen if you change "devPath": "http://<IP_ADDRESS>:8080/" to "devPath": "http://localhost:8080/"? If so, can you share a repo we can use to reproduce it?

@lucasfernog https://github.com/wychl/devutil

@wychl I couldn't reproduce the issue. I executed yarn and yarn serve in a terminal window, then yauri tauri dev in a separate terminal, all in the same machine. Can you check what's the value of window.location.href?

@lucasfernog  window.location.origin is  http://localhost:8080, my computer is darwin system

So it should be working. How are you running the app? I can't reproduce it.

are you running the app in firefox? our API doesn't work there.

@amrbashir @lucasfernog run yarn run serve, yarn tauri dev , chrome or firefox  have same problem problem.i use vue-cli-plugin-tauri plugin

You can't open the app url in the browser, it must be in the Tauri window unless you use the tauri-invoke-http crate.


Error when executing notebook

Getting the error - TypeError: <watson_developer_cloud.natural_language_understanding.features.v1.Entities object at 0x7f15027c2dd0> is not JSON serializable

This issue is due to change in the Python Watson SDK api. The issue has been fixed.

please us a PR instead of committing to master directly


Improve Restoration and Update process

Onboarding restoration
onboarding restoration is now responsible to fetch ALL the wallet data including:

transactions + unblind them using masterBlindingKey
assets details from registry
block headers associated with transactions

the onboarding page is loading until that steps are all done.
it closes #457
Presenter layer
This PR also moves the StorageContext logic (all the useEffects used to create the frontend cache data) to application/Presenter (it lets to do not take care about framework stuff like rendering while writing the presentation update).
Mermaid triggers update
The Marina logo at the top-left of the popup becomes clikable on home page -> it triggers a repository update = scan the repos and try to fill holes (assets, transactions, blockheaders etc...)
@tiero please review

In the receive list we have duplicates

Can you split the commits into two PRs? One for the restoration only and one for the change application/Presenter, otherwise is hard to review and properly test changes to a specific layers.


Push notification to open Bitkit

Description
When a user makes a request to open a channel, the user must come back to the app in between the time the channel is ready to be opened and before channel expiry. However, since the transaction can take several hours or even several days some users may forget and miss the time window, ending up with an expired channel order.
The problem is that the BT order is only valid for 2 days. We therefore need to warn the user to open Bitkit.
Ex: https://synonymworkspace.slack.com/archives/C0726U60V6Y/p1721769530317439
Solution
Schedule local notifications telling the user that he must open Bitkit in order to continue the channel opening process.

First notification after 24 hours
Second notification after 40 hours (this notification will not be shown if the channel was opened before then)

Note:
We don't check if the channel purchase transaction actually got confirmed (will be solved with wakeToOpen), so there can be the case where fees are spiking and the transaction doesn't confirm before order expiry. In this case the notifications will still (falsely) appear.

First: after 24h
Second: After 40h

Updated description to better explain the problem and implementation

I received a notification 24h after on my phone. But Once I clicked on it, nothing happened. I needed to open myself Bitkit.

The changes from the stealth mode feature did break this on Android. If we end up reverting that feature this will be resolved.

Yes, during the dev call Johna said we were killing this feature, right?


[Bug]:

Describe the bug
After upgrading to OpenSearch and OpenSearch Dashboards version 2.16.0, we get error when we try to save a Discover search.
To reproduce

Go to OpenSearch Dashboard Discover
Click on 'Save'
Inout any title
Click on 'Save'

Expected behavior
Discover search is saved with no errors
Screenshots

Host / Environment
OpenSearch and OpenSearch Dashboards version 2.16.0
Additional context
No response
Relevant log output
{"type":"log","@timestamp":"2024-09-22T04:16:47Z","tags":["error","opensearch","data"],"pid":1,"message":"[security_exception]: no permissions for [] and User [name=admin, backend_roles=[admin], requestedTenant=null]"}

{"type":"response","@timestamp":"2024-09-22T04:16:47Z","tags":[],"pid":1,"method":"post","statusCode":403,"req":{"url":"/api/saved_objects/search?overwrite=true","method":"post","headers":{"host":"[MASKED]","x-request-id":"cd1b40e738290336e5236bd11c7e11f3","x-real-ip":"<IP_ADDRESS>","x-forwarded-for":"[MASKED]","x-forwarded-host":"[MASKED]","x-forwarded-port":"443","x-forwarded-proto":"https","x-forwarded-scheme":"https","x-scheme":"https","content-length":"599","sec-ch-ua":"\"Chromium\";v=\"128\", \"Not;A=Brand\";v=\"24\", \"Google Chrome\";v=\"128\"","sec-ch-ua-platform":"\"[MASKED]\"","osd-xsrf":"osd-fetch","sec-ch-ua-mobile":"?0","user-agent":"[MASKED]","osd-version":"2.16.0","content-type":"application/json","accept":"*/*","origin":"https://[MASKED]","sec-fetch-site":"same-origin","sec-fetch-mode":"cors","sec-fetch-dest":"empty","referer":"https://[MASKED]/app/data-explorer/discover","accept-encoding":"gzip, deflate, br, zstd","accept-language":"en-US,en;q=0.9,ar;q=0.8","priority":"u=1, i"},"remoteAddress":"<IP_ADDRESS>","userAgent":"[MASKED]","referer":"https://[MASKED]/app/data-explorer/discover"},"res":{"statusCode":403,"responseTime":19,"contentLength":9},"message":"POST /api/saved_objects/search?overwrite=true 403 19ms - 9.0B"}

@atalakey4work What version are you upgrading from? Was this working prior to upgrade? Are you searching on any index pattern in particular?

@cwperks from version 2.16.0 to version 2.17.0.
Note that I’m using the helm chart.

What is the index pattern that the search is pertaining to and what plugins are installed on the cluster?

There are no plugins, other than the out of the box plugins if any, installed on the cluster.
The issue occurs with any index pattern.

[Triage] @atalakey4work Can you provide a little more information about the setup? I am not able to reproduce the issue as described.

@cwperks the issue started after upgrading OpenSearch and OpenSearch Dashboards to version 2.17.0. After the upgrade, I am no longer able to save new searches nor open searches previously saved. I had to downgrade OpenSearch Dashboards to version 2.16.0 to get the saved searches to work but I am still unable to save new searches. I was not able to downgrade OpenSearch as it would not allow me. I suspect it had something to do with the security plugin config so I took the latest copy from the below repo:
https://github.com/opensearch-project/security/tree/main/config

What index patterns are the saved searches on? I have a suspicion that there's a saved search that includes searching on system indices (system indices always start with a dot .). I will try to replicate the 2.16 to 2.17 upgrade and see what steps would be needed for mitigation. In general, if you have a saved search on a broad index pattern (like *), then it should be narrowed to a more specific index pattern.

@cwperks I was able to resolve the error by updating the below in my opensearch_dashboarfs.yml
Before:
opensearch.requestHeadersAllowlist: ["Authorization", "securitytenant"]
After:
opensearch.requestHeadersAllowlist: ["Authorization"]
Now save works but whenever I open a saved search the saerch filters are not populated:
Here is a copy of the kibanaSavedObjectMeta.searchSourceJSON
{
"query": {
"query": "",
"language": "kuery"
},
"highlightAll": true,
"version": true,
"aggs": {
"2": {
"date_histogram": {
"field": "@timestamp",
"fixed_interval": "30s",
"time_zone": "",
"min_doc_count": 1
}
}
},
"filter": [
{
"meta": {
"alias": null,
"negate": false,
"disabled": false,
"type": "phrase",
"key": "kubernetes.namespace_name",
"params": {
"query": "nginx-ingress"
},
"indexRefName": "kibanaSavedObjectMeta.searchSourceJSON.filter[0].meta.index"
},
"query": {
"match_phrase": {
"kubernetes.namespace_name": "nginx-ingress"
}
},
"$state": {
"store": "appState"
}
}
],
"indexRefName": "kibanaSavedObjectMeta.searchSourceJSON.index"
}


feat(display): Allow full configuration of display task

Description

Allow configuration of display task using espp::Task::BaseConfig.
Motivation and Context

Allows users to change the name, core id, etc. for the display task.
How has this been tested?

Building the display_drivers/example and building and running in esp-box-emu project.
Screenshots (if appropriate, e.g. schematic, board, console logs, lab pictures):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update
[ ] Hardware (schematic, board, system design) change
[x] Software change

Checklist:

[ ] My change requires a change to the documentation.
[x] I have added / updated the documentation related to this change via either README or WIKI

Software

[ ] I have added tests to cover my changes.
[ ] I have updated the .github/workflows/build.yml file to add my new test to the automated cloud build github action.
[x] All new and existing tests passed.
[x] My code follows the code style of this project.

https://w.wiki/ACCX

https://w.wiki/ACCX


Android lacks real time feedback on test failures

What happened?
On iOS when a test step fails it provides real time feedback in Jest so that the test can be cancelled early. This results in faster development iterations while developing or debugging.
On Android, Detox is not providing real time feedback during failures. I don't know why the failure occurred until the very end of the test suite. This is not ideal for fast development iterations or real time development.

For example:
13:40:47.510 detox[57896] i lifecycle Buy Again Test: Should signup for an account and add a payment method [FAIL]

should include details of why the failure happened right away so I can cancel the test and keep working.
What was the expected behaviour?
Expect Detox to provide why the test step failed in real time rather than only at the end summary of the test suite
Was it tested on latest Detox?

[X] I have tested this issue on the latest Detox release and it still reproduces.

Did your test throw out a timeout?

[X] I have followed the instructions under Identifying which synchronization mechanism causes us to wait too much.

Help us reproduce this issue!
No response
In what environment did this happen?
Detox version: 20.1.0
React Native version: 0.68.5
Node version: v16.18.0
Device model: Android AOSP
Android version: Android 11 API 30
Test-runner (select one): jest
Detox logs
No response
Device logs
No response
More data, please!
No response

@shamilovtim have you considered using Jest's --bail? In any case, this doesn't sound like anything platform-specific, TBH. We would appreciate any additional evidence (e.g. complete logs) showing that, or otherwise to be able to understand the difference between the errors you've had in the two platforms.


HtmlUtilities::ConvertToText Crash in WinUI 3 App with C++/WinRT

Describe the bug
I have a problem on Windows SDK API HtmlUtilities::ConvertToText in WinUI 3 App with C++/WinRT. Calling it in UWP app is OK, but not in WinUI 3 App. In WinUI 3 App, Calling it after starting the application by Application::Start throws an exception. However, Calling it before starting application has no problem and calling it again after starting application is OK too.
Steps to reproduce the bug

Create a project from WinUI 3 and UWP project template.
Call HtmlUtilities::ConvertToText after Application::Start

Expected behavior
Call HtmlUtilities::ConvertToText anywhere without an exception.
Screenshots
No response
NuGet package version
WinUI 3 - Windows App SDK 1.2.3: 1.2.230118.102
Windows version
Windows 11 (22H2): Build 22621
Additional context
Additional information provided by community
https://github.com/microsoft/WindowsAppSDK/discussions/3401#discussion-4824959

Sounds like this is an issue in the HtmlUtilities::ConvertToText component?  Some code that works fine in UWP doesn't work in a non-UWP context because not all the same objects are available.  Please follow up with owners of that component.  Thanks!


Must, not Need

In the login Notification pop-up:

It's not that you Need to  sign-in, you must sign-in for this to work. Changing the phrase from need to to must makes it less passive, clearer that it's not optional, and removes one extra word from the dialog. :-)

@RMacfarlane  closed because? Fixed? Don't care about it? Stupid suggestion?

@jwargo Fixed, it will be part of the next release. Thanks!


Library Validators - Error URL with "--" in subdomain

In version 0.18.2 URL with '--' in subdomain have problem:

import validators
url = 'https://abc--eddd.repl.co/'
print(validators.version)
print (validators.url(url))
0.18.2
ValidationFailure(func=url, args={'value': 'https://abc--eddd.repl.co/', 'public': False})

_

Describe the bug

Expected behavior

To Reproduce

My Platform

Additional context

Hi, this issue tracker is for issues with the codebase behind https://pypi.org itself, not the projects hosted on PyPI.
You should report this issue to the tracker for the project in question instead.


Release 3.2.0

Summary of the Pull Request
What is this about?
PR Checklist

[ ] Applies to work item: #xxx
[ ] CLA signed. If not, go over here and sign the CLI.
[ ] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request
What does this include?
Validation Steps Performed
How does someone test & validate?

Integration tests passed except known failures from #1263


Converted all to Kotlin, updated all SDKs, handled all Lint important…

… issues

Good, but I am not familiar with kotlin, so I might not consider to convert it to kotlin...

I might update this project when I have time, but Android software development is really annoying...especially compared with Linux software development, and I am not in good state those days. Thank you anyway.

Well this can help you be familiar with it:

The default, more advanced language for Android is now Kotlin, and it's very similar anyway.
You can convert to it usually fine if you have nullability annotations, and thankfully you usually added those.
Speaking about nullability, Kotlin forces you to declare what can be null and what can't, so it helps a lot against NPE.
Usually Kotlin code is shorter than Java

Please do consider adding this:
https://github.com/AlynxZhou/alynx-live-wallpaper/issues/14
I tried to update the project and add it yesterday for hours, but failed. I was hoping it would be easier to do both.
Hope you will feel well.

Well this can help you be familiar with it:
1. The default, more advanced language for Android is now Kotlin, and it's very similar anyway.

I am not an Android program in fact, so I am not interested in which language Google prefer. I write this in Java just because I know Java, if I start to learn Kotlin before write this program, it may never finished...

2. You can convert to it usually fine if you have nullability annotations, and thankfully you usually added those.

Maybe, but if I am not familiar with code converted, it's not good...

3. Speaking about nullability, Kotlin forces you to declare what can be null and what can't, so it helps a lot against NPE.

Not a problem for a C programmer? It might help, but not a lot.

4. Usually Kotlin code is shorter than Java

The same as the second. I may consider to use Kotlin when I decide to learn it, but I also need to say I've tried to learn Kotlin some times, but all ends with "I can not see how Kotlin will help me and it looks unnecessary for me" (and the same for rust XD) ... I am a little bit lazy, sorry.

Please do consider adding this:
#14

OK, if I have time, I'll do it, not a problem.

I tried to update the project and add it yesterday for hours, but failed. I was hoping it would be easier to do both.
Hope you will feel well.

Thank you.

But I have to say that to support cropping video is a little bit hard for me, I don't know how to write it, but I may add a toggle to allow user to change between center crop or keep full content?

You can start in Java each class, and then convert.
I learned it this way on my own apps. I converted various Java classes, and then tested the apps. The most common issue is NPE. The rest is usually rare and eventually you will see how it works.
Java is not written exactly like C. And C has its own things that you need to very carefully take care of (memory...). I've actually come in the far past from C to Java :)
Being lazy is what's supposed to encourage you. It's writing less code (in most cases). Plus you don't need to write ";" anymore :)

About scale-types, I gave you a link to a similar repository that does offer various kinds of them. Please check it out. I also put a ZIP file of it sample, as the original had some issues being opened for some reason.


Make control an explicit capability to overload the traversal

If you get the time, could you also add a basic boundary handler for Control as per the discussion in https://github.com/effekt-lang/effekt/issues/404#issuecomment-2194555200?
(so that we can express an early exit from a function in examples without the need for a return)


Improve error handling

One potential improvement to error handling would be to pass the error instance as an option:
return utils.build.failBuild(`Failed to inline function files due to the following error:\n${err.message}`, { error: err });

This ensures the original error's stack trace is kept in the build logs.

Done in #7.


Geo.JSON does not support PointZ

First of all, thanks for such great library! I am using it to store spherical coordinates with height, but I found that Geo.JSON.encode does not support PointZ. I am new to Elixir and Phoenix, so any suggestion is welcome! I can separate height from longitude and latitude, but that looks not clean since height is part of a coordinate.

Hi! You are correct in it's not supported, but only because I don't know how it should be displayed. I remember looking at the GeoJSON spec and not getting any clues. I'm open to suggestions

Hey @dongli, this was implemented in geo 3.1.0.

Hi there -- sorry to revive this closed issue, but I am wondering if there is any more thought around extending Z-coordinate support more broadly to other geometry types.
If I understand correctly from the this pull request it looks like these are currently handled for Points using a dedicated Geo.PointZ, which gets serialized to GeoJSON as {"type": "PointZ", "coordinates" => [x,y,z]}.
Unfortunately the PointZ type currently produced by the library is not valid GeoJSON -- if you put something like {"type":"PointZ", "coordinates":[100.0,0.0,70.0]} into a GeoJSON linting tool like geojson.io, it will flag it as an error.
I believe in the GeoJSON spec z-coordinates should be handled on all geometries as just an optional 3rd element in the coordinate array, for example {"type": "Point", "coordinates": [1,2,3]}.
I'm curious if there have been other discussions or proposals around extending all the geometry types to support optional 3rd z elements in their coordinates?
I'm not very experienced with elixir so I'm afraid I don't have great advice about the best way to model this in elixir would be -- it might require changing the coordinate types for geometries from a 2-element to a 3-element tuple or a list or something like that. In JTS for example they model it by subclassing the Coordinate class.
Anyway, mostly just curious if something like this is possible, or has been considered before. Thanks!

I'll have to take a deeper look. I originally intended for it to work as with coordinates can be n-tuple because not all projections are the same and there are different dimensions.
There may need to be some rethinking on that. Maybe the workaround for now is to make it so when 3 coordinate are found, it makes it into a pointz.
I'm just thinking out loud right now but will think some more on it and welcome any advice or feedback.
Also thanks for pointing me to geojson.io. I didn't know it existed!

Section 4 of rfc7946 has this to say about it:

An OPTIONAL third-position element SHALL
be the height in meters above or below the WGS 84 reference
ellipsoid.  In the absence of elevation values, applications
sensitive to height or depth SHOULD interpret positions as being at
local ground or sea level.

The problem is that this applies to all geometries. Not only Point geometries. So it would take some slight restructuring of Geo to support this. I'll be happy to join in and do some thinking on how it could be if supporting optional 3rd coordinate on all geometries is something you wish to do.
I do understand that the library is a Geo library but not a GeoJSON library so if you decide not to support optional 3rd element. It's a perfectly valid choice. But even if you decide you not to support that it might be a good idea to change the GeoJSON decoder to support reading 3 element coordinates and simply thowing away the 3rd element. That would eliminate it throwing when reading valid GeoJSON files.

If you choose not to have generic support of 3 element coordinates I would recommend the JSON encoder/decoder to do the following:
Encoder:

%Point{} -> {"type": "Point", "coordinates": [1,2]}
%PointM{} -> {"type": "Point", "coordinates": [1,2]}
%PointZ{} -> {"type": "Point", "coordinates": [1,2, 3]}
%PointZM{} -> {"type": "Point", "coordinates": [1,2, 3]}

Decoder:

{"type": "Point", "coordinates": [1,2]} -> %Point{}
{"type": "Point", "coordinates": [1,2,3]} -> %PointZ{}
{"type": "OtherGeometriesWith3Elements", "coordinates": [1,2,3]} -> discard the 3rd element


Use a single triangle for screen render instead of quad

To avoid oversampling across screen diagonal (similar to postprocessing package)

This is already being done in three.js' FullScreenQuad.


Unfortunate window size

The window size of the gadget seems to be fixed at a certain size, and in relation to the window the actual plot seems to be too big. The result is that the controls are hard to interact with, especially with dropdown menues.
Here's a screencap to see what I mean (mp4)

System: OSX 10.11.3 / Retina MBP 15"
Tested both current GitHub and CRAN version

Which version of shiny are you using?

I just tried to set the plot to a relative size, it seems to work. I will push a bugfix later.
Thanks for bringing this up.

Could you install from the dev branch and give some feedback if the plot has now a correct size on your macbook?
devtools::install_github('calligross/ggthemeassist', ref = 'dev')

Looks better on my 12":
http://www.pic-upload.de/view-29987643/ggthemeassist_dev10032016.png.html

Yep, looks better, thanks!


tracking of files anchored at the root of the repository fails on Windows

Setup
git-lfs/1.4.1 (GitHub; windows amd64; go 1.7; git 2070e4c)
git version 2.9.2.windows.1
Win 7 & Win 8.1

Steps to reproduce
MINGW64 ~/my-repo (master)
$ echo "test" > test1.dat

MINGW64 ~/my-repo (master)
$ git lfs track /test1.dat
Error getting git tracked files

Errors logged to ....
Use `git lfs logs last` to view the log.

Tracking of files anchored at the root of the repository fails on Windows in Git Bash.
Log
git-lfs/1.4.1 (GitHub; windows amd64; go 1.7; git 2070e4c)
git version 2.9.2.windows.1

$ git-lfs track C:/path/to/my/Git/test1.dat
Error getting git tracked files

exit status 128

Thanks for the report. Added to a v1.5.0 milestone. If @ttaylorr or someone else can submit a patch, I can see getting this into a v1.4.x release, but I personally don't have the time now.

Thanks for the ping! I can take a look at this after I finish some work I'm doing on the transferqueue.

Sorry for the late response! I just finished digging into this, here's what I found.
When you type /a.dat at the Windows shell, it turns that into C:/Program Files/Git/a.dat. This is usually outside of the Git repository (unless for some reason you're working in C:/Program Files/Git) so running git ls-files fails.
I don't think it's worth trying to change or guess the path that Windows is prepending, since this will break the expectations of Windows scripts. I do think, however, that we should improve the error message we're getting from git lfs track to include the error that we failed with when running the ls-files command.
Does this sound like a good approach?

Hm... on Linux/OSX /something refers to something directly at the root of the file system. However, Git interprets that as "something at the root of the Git repository". I believe that is similar to what you mean with "break the expectations of Windows scripts", right?
When is /a.dat turned into C:/Program Files/Git/a.dat? I assume you see the latter as parameter in Go? What if we put parenthesis around the parameter (git lfs track "/a.dat")?

The conversion happens at the shell, before it is sent to LFS as an argument. Quoted strings have the same behavior. Should we maybe be responding to \ instead? My preference would be to leave this behavior unchanged, and just improve the output.

You mean a Windows user would write git lfs track "\something"?
In what environment did you observe the conversions? In cmd.exe, in git-bash.exe, or both?

The conversion occurs in git-bash.exe only. cmd.exe works correctly with Git LFS, given a leading /.


Backend Use Nacos cause OutOfMemoryError

What happened?
When backend loses contact with naocs, OOM occurs after a period of time：
Exception in thread "com.alibaba.nacos.client.Worker" java.lang.OutOfMemoryError: unable to create new native thread
	at java.lang.Thread.start0(Native Method)
	at java.lang.Thread.start(Thread.java:719)
	at java.util.concurrent.ThreadPoolExecutor.addWorker(ThreadPoolExecutor.java:957)
	at java.util.concurrent.ThreadPoolExecutor.processWorkerExit(ThreadPoolExecutor.java:1025)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1167)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:750)
Exception in thread "com.alibaba.nacos.client.Worker" java.lang.OutOfMemoryError: unable to create new native thread
	at java.lang.Thread.start0(Native Method)
	at java.lang.Thread.start(Thread.java:719)
	at java.util.concurrent.ThreadPoolExecutor.addWorker(ThreadPoolExecutor.java:957)
	at java.util.concurrent.ThreadPoolExecutor.processWorkerExit(ThreadPoolExecutor.java:1025)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1167)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:750)

How can we reproduce it (as minimally and precisely as possible)?

do not start nacos
backend use nacos

dynamic.config.enable=true
dynamic.config.namespace=sermant
dynamic.config.timeout=30000
dynamic.config.serverAddress=<IP_ADDRESS>:8848
dynamic.config.dynamicConfigType=NACOS
dynamic.config.connectTimeout=3000
dynamic.config.enableAuth=false
dynamic.config.userName=
dynamic.config.password=
dynamic.config.secretKey=

start backend

Anything else we need to know?
No response
Sermant version
2.0.0
OS version
MacOS

We don't need to manage reconnection ownerself when using the Nacos client.

for agent, it is the same, nacos version after 2.2.1（https://github.com/alibaba/nacos/pull/9639）, it add health check logic, it will cause memory/thread leak.

i will try to fix it, by remove nacos reconnect logic

How many time nacosclient manages reconnection？Is it configurable？

The number of reconnections for nacos client cannot be configured, unlimited retries.
See https://github.com/alibaba/nacos/blob/2.2.1/common/src/main/java/com/alibaba/nacos/common/remote/client/RpcClient.java#L280


Uncaught Error: Cannot call function 'getPosition' on missing remote object 42Error: Cannot call...

[Enter steps to reproduce:]

...
...

Atom: 1.39.1 x64
Electron: 3.1.10
OS: Microsoft Windows 10 Home
Thrown From: Atom Core
Stack Trace
Uncaught Error: Cannot call function 'getPosition' on missing remote object 42
Error: Cannot call function 'getPosition' on missing remote object 42
at throwRPCError (C:\Users\Developer\AppData\Local\atom\app-1.39.1\resources\electron.asar\browser\rpc-server.js:145:17)
at EventEmitter. (C:\Users\Developer\AppData\Local\atom\app-1.39.1\resources\electron.asar\browser\rpc-server.js:367:7)
at EventEmitter.emit (events.js:182:13)
at WebContents. (C:\Users\Developer\AppData\Local\atom\app-1.39.1\resources\electron.asar\browser\api\web-contents.js:294:13)
at WebContents.emit (events.js:182:13)
At C:\Users\Developer\AppData\Local\atom\app-1.39.1\resources\electron.asar\renderer\api\remote.js:219

Error: Cannot call function 'getPosition' on missing remote object 42
Error: Cannot call function 'getPosition' on missing remote object 42
    at throwRPCError (~/AppData/Local/atom/app-1.39.1/resources/electron.asar/browser/rpc-server.js:145:17)
    at EventEmitter.<anonymous> (~/AppData/Local/atom/app-1.39.1/resources/electron.asar/browser/rpc-server.js:367:7)
    at EventEmitter.emit (events.js:182:13)
    at WebContents.<anonymous> (~/AppData/Local/atom/app-1.39.1/resources/electron.asar/browser/api/web-contents.js:294:13)
    at WebContents.emit (events.js:182:13)

    at metaToException (~/AppData/Local/atom/app-1.39.1/resources/electron.asar/renderer/api/remote.js:271:17)
    at Object.exception (~/AppData/Local/atom/app-1.39.1/resources/electron.asar/renderer/api/remote.js:219:30)
    at metaToValue (~/AppData/Local/atom/app-1.39.1/resources/electron.asar/renderer/api/remote.js:223:28)
    at Object.remoteMemberFunction (~/AppData/Local/atom/app-1.39.1/resources/electron.asar/renderer/api/remote.js:123:16)
    at AtomEnvironment.getWindowDimensions (~/AppData/Local/atom/app-1.39.1/resources/app/static/<embedded>:1:730051)
    at AtomEnvironment.storeWindowDimensions (~/AppData/Local/atom/app-1.39.1/resources/app/static/<embedded>:1:730466)
    at WindowEventHandler.handleWindowBlur (~/AppData/Local/atom/app-1.39.1/resources/app/static/<embedded>:11:284967)

Commands
     -2:23.9.0 intentions:highlight (input.hidden-input)
  3x -2:23.7.0 core:save (input.hidden-input)

Non-Core Packages
busy-signal 2.0.1
file-icons 2.1.33
highlight-selected 0.14.0
highlight-selected 0.14.0
intentions 1.1.5
language-babel 2.85.0
language-graphql 1.0.0
linter 2.3.0
linter-eslint 8.5.5
linter-ui-default 1.7.1
native-ui 0.27.0
react 0.18.0
sort-lines 0.19.0

Thanks for contributing!
We noticed that this looks like a duplicate of https://github.com/atom/atom/issues/16630 so you can subscribe there for updates if you'd like (we believe that this will be fixed in a future Electron upgrade).

Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


Add more doc about metrics-server

Based on this issue https://github.com/kubernetes-incubator/metrics-server/issues/292 it seems that Rancher documentation about metrics-server on K3S is missing a step (see https://github.com/kubernetes-incubator/metrics-server/issues/292#issuecomment-516382818 in the issue).
I was unable to run kubectl top pods if --kube-apiserver-arg="requestheader-allowed-names=system:auth-proxy,kubernetes-proxy" is not set when starting K3S.
PS: My K3S cluster was upgrade a couple of times and I'm now running v0.10.0.

Those are pretty old issues, there should not be a need to pass extra args to k3s or modify the default metrics-server deployment from https://github.com/kubernetes-incubator/metrics-server/tree/master/deploy/1.8%2B.

I fully reinstalled the cluster and the problem is not present anymore. But maybe is there a problem with the upgrade process ?

Probably, there was a certs refactor in v0.7.0 that may not have upgraded a particular cert correctly. Apologies for that! Would you happen to know the original version of k3s that was installed?

Hum I don't remember exactly but it's around v0.7.0 :/

let me also add that it would be nice to add documentation/examples around HorizontalPodAutoscaler along with metrics-server (since the former depends on the latter).

We will be having metrics-server by default in an upcoming release. Reference: https://github.com/rancher/k3s/issues/990


Broken with<EMAIL_ADDRESS>

Describe the bug
Dashboard tool crashes with sanity package version 3.23.0
To Reproduce
Steps to reproduce the behavior:

npm install sanity@latest
npm run dev
See error

Expected behavior
Desktool should render properly.
Screenshots

Error: Cannot read properties of undefined (reading 'bleed')
TypeError: Cannot read properties of undefined (reading 'bleed')
    at buttonColorStyles (http://localhost:3333/node_modules/.sanity/vite/deps/chunk-2OCGD354.js?v=da9292b9:33404:32)
    at Ne (http://localhost:3333/node_modules/.sanity/vite/deps/chunk-KOT67DZC.js?v=da9292b9:1039:14)
    at e2.generateAndInjectStyles (http://localhost:3333/node_modules/.sanity/vite/deps/chunk-KOT67DZC.js?v=da9292b9:907:19)
    at http://localhost:3333/node_modules/.sanity/vite/deps/chunk-KOT67DZC.js?v=da9292b9:1158:88
    at http://localhost:3333/node_modules/.sanity/vite/deps/chunk-KOT67DZC.js?v=da9292b9:1160:8
    at O2 (http://localhost:3333/node_modules/.sanity/vite/deps/chunk-KOT67DZC.js?v=da9292b9:1164:6)
    at renderWithHooks (http://localhost:3333/node_modules/.sanity/vite/deps/chunk-W4SCQDMF.js?v=da9292b9:12171:26)
    at updateForwardRef (http://localhost:3333/node_modules/.sanity/vite/deps/chunk-W4SCQDMF.js?v=da9292b9:14327:28)
    at beginWork (http://localhost:3333/node_modules/.sanity/vite/deps/chunk-W4SCQDMF.js?v=da9292b9:15934:22)
    at beginWork$1 (http://localhost:3333/node_modules/.sanity/vite/deps/chunk-W4SCQDMF.js?v=da9292b9:19749:22)

Which versions of Sanity are you using?
Run sanity versions in the terminal and copy-paste the result here.
@sanity/cli (global)              3.23.1 (up to date)
@sanity/cli                       3.23.1 (up to date)
@sanity/code-input                 4.1.2 (up to date)
@sanity/color-input                3.1.0 (up to date)
@sanity/components                2.14.0 (up to date)
@sanity/cross-dataset-duplicator   1.2.3 (up to date)
@sanity/dashboard                  3.1.6 (up to date)
@sanity/default-layout            2.36.2 (up to date)
@sanity/default-login             2.36.2 (up to date)
@sanity/orderable-document-list    1.1.0 (up to date)
@sanity/plugin-kit                3.1.10 (up to date)
@sanity/production-preview        2.35.3 (up to date)
@sanity/scheduled-publishing       1.2.2 (up to date)
@sanity/table                      1.0.1 (up to date)
@sanity/ui                         1.9.3 (up to date)
@sanity/vision                    3.23.1 (up to date)
sanity                            3.23.1 (up to date)

What operating system are you using?
Mac OS Sonoma 14.2 (23C64)
Which versions of Node.js / npm are you running?
npm: 10.1.0
node: v20.8.0

Additional context
Setup in sanity.config.js:
definePlugin({
 ...,
  plugins: [
    ...,
    dashboardTool({
      widgets: [
        documentListWidget({
          layout: { width: 'small' },
        }),
      ],
    })
  ]
})

I think the widget is crashing for any parameter passed. I tried with default params and the query params.

Crashing for all 3 options. ( I have published as a field)

Same issue here

Same here had to remove it in order to continue to use sanity


Add meta-data keys command

Implement a buildkite-agent meta-data keys command for listing meta-data keys:
E.g:
buildkite-agent meta-data keys # default delimited by newlines
buildkite-agent meta-data keys --delimiter ';'

You can use it to enumerate meta-data like:
#!/bin/bash
set -euo pipefail

buildkite-agent meta-data keys | while read -r key; do
  echo "$key: $(buildkite-agent meta-data get "$key")"
done

/cc @toolmantim

Amazing! Thanks @lox. I went to give it a test, but I don't think the meta-data keys subcommand exists yet yeah?

Ok @toolmantim this should be good to go.

❤👏🏼 So excited. Can’t wait to give this a test!

Woo, almost works a treat.
I tested it with this:
steps:
  - label: "Set the keys"
    commands: |
      cd ~/Codez/buildkite/agent
      go run main.go meta-data set foo1 bar1
      go run main.go meta-data set foo2 bar2
      go run main.go meta-data set foo3 bar3
  - wait
  - label: "Get the keys"
    commands: |
      cd ~/Codez/buildkite/agent
      echo "+++ List the keys"
      go run main.go meta-data keys
      echo "+++ Read each key"
      go run main.go meta-data keys | while read -r key; do
        echo "$$key: $$(go run main.go meta-data get "$$key")"
      done

The only thing I found is that I think we'll need to output the delimiter after the last item too, as the above outputs:

@toolmantim I dunno, we don't output a trailing \n at the end of meta-data get, why would we for meta-data keys? My take is that a delimiter should occur between records, not between and after.
I think the use-case of printing to stdout is probably pretty rare, generally it's going to be consumed by a tool, for which the trailing delimited might actually cause issues. Thoughts?

I'm going to merge this and we can discuss in master.

@sj26 suggested a nice thing would be to detect if it was a TTY and maybe output a trialing newline?

Did you spot that this didn't actually output all the keys though?
go run main.go meta-data keys | while read -r key; do
  echo "$$key: $$(go run main.go meta-data get "$$key")"
done

That is unexpected 🤔

while IFS= read -r key; do
  echo "$$key: $$(go run main.go meta-data get "$$key")"
done <(go run main.go meta-data keys)

Should work!

Given this is designed for bash, I figured you'd want it to work with read? (like in the PR description?)
i.e. the following only prints "foo1" and "foo2":
printf "foo1\nfoo2\nfoo3" | while read -r key; do
  echo "$key"
done
# foo1
# foo2
# foo3

This shows the same:
while IFS= read -r key; do
  echo "$key"
done < <(printf "foo1\nfoo2\nfoo3")
# foo1
# foo2

Echo works, because it echoes the trailing newline:
echo -e "foo1\nfoo2\nfoo3" | while read -r key; do
  echo "$key"
done
# foo1
# foo2
# foo3

Maybe --delimiter is the wrong name for the argument? 🤷🏼♂ But that's why I brought it up…

Or… you could remove --delimiter altogether and make sure to put a newline at the end too, so it works for both read and just reading it in a terminal? And if people have meta-data keys containing newlines… 🤷🏼♂

I’m inclined to agree and add a trailing newline @toolmantim

I’ll PR it

Yeah I think the --delimiter may have warped my impression, it makes sense to make it newline only, and make sure there's a trailing newline. 🙌

This is a tricky one 🤔 I didn't understand the nuance of read where it returns a false value if it encounters an EOF before a newline.
The correct way to do this with read is:
while IFS= read -r key || [[ -n "$key" ]] ; do
  echo "$key"
done < <(printf "foo1\nfoo2\nfoo3")

This is because even though read returns false, the read still happens and $key is populated.

The extra newline still seems like a more human API to me? 🤷🏼♂

That is what I did @toolmantim!

Was just trying to understand the bash-ness.

See #1040.

Ah, I see! 👍🏼 We can always add a format json/newlines argument for machine reading?

Yeah, I reckon.


Create and serve new API to users.

Approach

Create the initial openapi spec for the curator API
Update the routes for the portal backend to include the new curator API.

Definition of Done

The curator API documentation can be reached at app../curator/

complete


Problems compiling with -DDOCS=ON

Configuring CMake works in the build directory by running something like:
cmake -GNinja -DFLUID_PATH=../../core -DDOCS=ON ..
However, calling ninja afterwards produces this error:
[0/2] Re-checking globbed directories...
ninja: error: 'MAKE_CLI_REF', needed by 'CMakeFiles/CLI_MAKE_DOCS', missing and no known rule to make it

I've torn my hair out a bit searching for how the other environments deal with it and I can't see a difference... Max's counterpart is even directly commented out https://github.com/flucoma/flucoma-max/blob/5dbca51bbab24cd0861456cea5043c887457c3f6/CMakeLists.txt#L96. I would try to solve this one but any chance you take a look for me @weefuzzy . My CMake-fu is pushed beyond its limits 😿

Now uncommented. See if that helps

It does indeed help. New issue incoming though....

oops no:
FAILED: _deps/flucoma-docs-build/CMakeFiles/MAKE_CLI_REF /Users/james/dev/flucoma/cli/build/_deps/flucoma-docs-build/CMakeFiles/MAKE_CLI_REF
cd /Users/james/dev/flucoma/cli/build/_deps/flucoma-docs-build && /opt/homebrew/Cellar/cmake/3.22.0/bin/cmake -E make_directory "" && /usr/local/Frameworks/Python.framework/Versions/3.9/bin/python3.9 /Users/james/dev/flucoma/docs/script/MakeRef.py cli /Users/james/dev/flucoma/cli/build/json /Users/james/dev/flucoma/docs/doc "" /Users/james/dev/flucoma/docs/script/templates
Error creating directory "".
ninja: build stopped: subcommand failed.

Yep I had a dig and thought maybe MakeRef.py was responsible though I see cmake -E make_direcotry “” is called too

On 7 Dec 2021, at 15:28, weefuzzy @.***> wrote:
Error creating directory "". is evidently the problem but will need to
wait until I'm cleverer to work why that's happening...
On Tue, 7 Dec 2021, 15:17 James Bradbury, @.***> wrote:

oops no:
FAILED: _deps/flucoma-docs-build/CMakeFiles/MAKE_CLI_REF /Users/james/dev/flucoma/cli/build/_deps/flucoma-docs-build/CMakeFiles/MAKE_CLI_REF
cd /Users/james/dev/flucoma/cli/build/_deps/flucoma-docs-build && /opt/homebrew/Cellar/cmake/3.22.0/bin/cmake -E make_directory "" && /usr/local/Frameworks/Python.framework/Versions/3.9/bin/python3.9 /Users/james/dev/flucoma/docs/script/MakeRef.py cli /Users/james/dev/flucoma/cli/build/json /Users/james/dev/flucoma/docs/doc "" /Users/james/dev/flucoma/docs/script/templates
Error creating directory "".
ninja: build stopped: subcommand failed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flucoma/flucoma-docs/issues/21#issuecomment-988021775,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGQHIYK4NTBCX7MIKLFP3VTUPYQPPANCNFSM5JPUM46Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/flucoma/flucoma-docs/issues/21#issuecomment-988032331, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADFKURSGIN72XPFBXLZMNTTUPYRZDANCNFSM5JPUM46Q.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Ok, I see the problem. Just testing before I push

You're a gem 💎

Try now (or, like, whenever)

will do later, got a thing now. Exciting! CI might be online 2nite ;)

Cool this solves this problem. I'm now running into an architecture issue which is likely to be an M1 thing 😔 I think that the docs compile is now picking up the architecture properly or something. Here is my error log:
error.txt

Undefined symbols for architecture x86_64: doesn't look promising. Is this starting with a clean cache?

Oh, hold up. I see it. x64 is baked in on Apple 😬 Will remove

All good. I await your precise and surgical Cmake-fu

Pushed. You'll need to clean your cache or whatever before this makes any positive difference

Although, I notice we have the same badness in the other sub-repos. Are you not passing a -DCMAKE_OSX_ARCHITECTURES=arm64 in to cmake?

Since I've been allowing fat binaries for Max'n'stuff no but I should really...

Error arises still even after passing -DCMAKE_OSX_ARCHITECTURES=arm64
FAILED: _deps/flucoma-docs-build/src/clients/BufSines/BufSines
: && /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++ -stdlib=libc++ -O3 -DNDEBUG -arch x86_64 -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX12.0.sdk -mmacosx-version-min=10.8 -Wl,-search_paths_first -Wl,-headerpad_max_install_names  _deps/flucoma-docs-build/src/clients/BufSines/CMakeFiles/BufSines.dir/BufSines.cpp.o -o _deps/flucoma-docs-build/src/clients/BufSines/BufSines  _deps/flucoma-core-build/libHISSTools_FFT.a  -framework  Accelerate  -stdlib=libc++ && :
ld: warning: ignoring file _deps/flucoma-core-build/libHISSTools_FFT.a, building for macOS-x86_64 but attempting to link with file built for macOS-arm64
Undefined symbols for architecture x86_64:

That's flucoma-docs barfing though 😖

😡 let me know how I can help or test things

My bad, insofar as I didn't notice that this was the docs repo to start with (curse all the emails). But now I'm puzzled this doesn't break elsewhere, or is this across Max / PD and CLI?

Yep its everywhere. I'll chec the linked PR now.


Changed maximum number of dependencies to 20. Optimized build times

See https://github.com/Swinject/SwinjectAutoregistration/issues/14

In case of Curry, it is told that compilation performance hits over 21 generic arguments:
https://github.com/thoughtbot/Curry/commit/70f1a2b5ee18865ffdb51e658a025200600877a1
It might be good to measure the number of generic arguments to hit the compilation performance in our case.

@tkohout, I have a question. Is it possible for a user to write unresolvedService and checkResolved for 20 arguments as a kind of an extension?
I thought it might be good to write documentation how to increase the number of arguments by users themselves. If this is possible, the compilation performance is good for users who use only limited number of arguments, and users who need more than 20 arguments can extend by themselves.

Well the problem in curry is not with the number of generic arguments but the curried functions that it produces. If you write curried function with 21 non-generic arguments it won't be able to compile either.
The difference between build times of AutoRegistration with 9 and 20 dependencies is about 3s (5s vs. 8s) and that can be only result of more code that have to be compiled.
Btw I tried also create autoregistration with 50 services and it still builds in 30s so I don't think it is a big problem.

Thanks @tkohout for the detailed info👍
I'll merge this pull request if you really want to provide 20 arguments to all the users. If you actually think 12 or 15 arguments are enough, we can reduce the number of arguments. My point is just API design.

Well, I wanted to set it to a maximum number which I think will be satisfactory for all users. I just want to avoid raising it again when e.g. 12 arguments won't be enough.

I got it👍


Add a search bar to filter the hackers

If anyone wants to contribute other than just adding their names, add a search bar. The search bar must be able to filter out the hackers as a name is typed. Just make sure it does not overflow in mobile view.

@joe733 I have merged the PR regarding caching users in local storage. Now all the data is cached for 48 hours. Hopefully, this would be the solution to the API limit error. A single API is sent for each user => earlier it used to cross the limit if we reload the site 3 times. Currenly, it will call an API only if there is a new contributor.
The variable allContributors stores all the contributors. For the search feature, filter the allContributors array using the search query and call the generateCardGrid function. This will remove the existing card grid and render a new one with the newly passed contributors.


Enhance authority rewrite support to allow port and scheme

Currently, the redirect.authority can take any input, such as new-host, https://new-host:80, etc. However, only host and host:port actually work. For example https://new-host would end up with envoy setting Location to http://https://new-host.
The new Gateway API (https://gateway-api.sigs.k8s.io/v1alpha2/references/spec/#gateway.networking.k8s.io/v1alpha2.HTTPRequestRedirect) has port, hostname, and scheme as valid options. Any combination of these three may be set.
This PR makes redirect.authority more flexible, by natively understanding scheme://host:port, where any of the three are optional, and converting directly into Envoy's support for these in redirects rather than always doing host redirect.
Backwards compatibility:

host form: not impacted
host:port form: not impacted. The envoy config will now be hostRedirect: host; portRedirect: port but the end result is the same
All other forms: before it would generate invalid location like  http://https://new-host, now it works
Therefore, all previously valid configs are still valid

Alternatives:

Modify the API to have port and scheme first class
Do something similar to this PR, but make it internal-facing only so it can only be access by gateway-api objects.

@howardjohn This would make the VirtualService.HTTPRedirect.Authority confusing, according to the rfc, authority does not include scheme.
So prefer make it a first-class API.

I also feel a first class api like scheme may be better and with that we can also support scheme-redirect in Envoy?

ok I can add it as an API. We need port as well
On Mon, Aug 30, 2021, 3:54 AM Rama Chavali @.***> wrote:

I also feel a first class api like scheme may be better and with that we
can also support scheme-redirect
https://www.envoyproxy.io/docs/envoy/latest/api-v3/config/route/v3/route_components.proto#config-route-v3-redirectaction
in Envoy?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/34910#issuecomment-908243438, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXL4KYFNUFJSYQHZX3TT7NPOHANCNFSM5C6E4OFQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-istiodremote-tests_istio
7ca57f86f408e1399a0f1b60116041fcb71c686f
link
/test integ-pilot-istiodremote-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

obsolete: https://github.com/istio/istio/pull/35146


Add depends_on failing tests

Add a test that shows that depends_on() doesn't work as expected.

Thanks for finding this! I'll have a look.

Thanks for digging -- this actually looks fine and is not a bug. The libc dependency to memchr is optional (and not activated), while the quickcheck dependency is dev-only and also not activated.

OK, my baby, sorry 😔


Attribute priority: first wins

Accoridng to the HTML spec when it comes to duplicate attributes the first defined attribute is the one that wins. Subsequent attributes of the same name are ignored:

I just want to confirm that we are adhering to this as well in the SwiftUI client.

It seems that get_attribute already returns the first.
There are specific cases where the last one is used. csrf tokens being the most important example. Should we consider taking the first of those?

I have none. I can dig through the commit history to see if there is a spelled out justification for that, but my intuition tells me that if you have multiple CSRF tokens something has gone terribly wrong.

I agree, I cannot tink of a scenario where there would be multiple tokens rendered. But let's ask Chris today on our call

Yeah, sounds good, if there aren't any concrete instances of attributes showing up with the wrong precedence i'm going to vote to close this issue.


WiP multivar skew normal

#44
Sorry about this being very rough but I found it challenging just to get to this point. I've only implemented sampling so far and, visually, it looks correct.
corr =
[[1, .7],
[.7, 1]]  ) # mat
skew =
[[.5],
[-5]]

I fried my brain trying to do this with a bijector but, in the end, it seemed I had to make a TFP distribution.
todo:

[x] log_p
[x] batching/broadcasting
[x] use a bijector to locate and scale.
[x] wrap with (PyMC) RV
[ ] test

I wonder if this shouldn't go into TF probability instead?

@brianwa84 isn't there a good amount of logic in the code that i submitted that should be in tfp? i feel like, as a distribution implementer, i shouldn't have to deal so much with broadcasting .

Ideally you should not need to do explicit broadcasting in most cases -- TF ops should take care of it for you.
Perusing the code, I think it could benefit from renaming some things to avoid greek (you can still mention it if helpful in a comment # maps to omega from eq 1.x in the paper) and instead express what the variable is doing (skew matrix?)
We do want to eventually get a skew MVN in TFP, so you're welcome to move this PR over to our repo.

@majidaldo Closing this but encouraging you to open it on TFP.


[📚] missing import

On this page there is a missing import
https://firebase.flutter.dev/docs/ui/auth/integrating-your-first-screen
https://firebase.flutter.dev/docs/ui/auth/integrating-your-first-screen
This import is needed to get SignInScreen widget
import 'package:flutterfire_ui/auth.dart';

@graemep-nz
The links you shared are archived and no longer maintained. You should refer below link which shows the import:
https://github.com/firebase/flutterfire/blob/master/packages/flutterfire_ui/doc/auth/integrating-your-first-screen.md#constructing-a-signinscreen-widget
Closing.


add snowpack example and improve comp check

/CC @FredKSchott
Do you have better idea's to find this out pre-runtime? The difference with vite is that here we're relying on babel to run before this plugin so we can check for h or $refreshreg$. If we do this check during runtime the server will think this module is hot since it contains an accept call (in an if clause)

This is probably the best for now. The next move on our end is to handle a conditional accept() call, where the server will have to send the update down to the client but then listen for a response from the client saying "sorry, not actually accepted".


test webclient for bot utilization https://app.coracle.social

against relay server and how from V we can create bot using nostr

is in svelte
https://github.com/staab/coracle


Font weight support

How can I update font weight for barcode text and addontext

Font are specified by their PostScript font name and most fonts have specific variants for different weights.
For example, if you currently have:
textfont=Courier

You can change this to:
textfont=Courier-Bold


Improve Primus/Optimus support

When working on a laptop featuring a combined integrated and discrete GPU, it may be beneficial to split the application across both.
Proposed architecture:

Do not use g_vkComputeDevice for final rendering. Create a separate Vulkan device pointer which may or may not point to the same handle.
Use g_vkComputeDevice for waveform rasterization
Use the new render device handle for all imgui stuff as well as tone mapping
Ensure device handles are propagated in AcceleratorBuffer's and other objects as needed
During tone mapping, if g_vkComputeDevice is not equal to the render device, perform a device to device copy of the grayscale fp32 pixels. This is similar to how glscopeclient's Vulkan-OpenGL transition works, and would effectively be application managed Primus (with low level control of the partitioning)

We still need to look into how PrimusVK manages power. Do we have the ability to power the dGPU on/off on cue? Is data lost / are handles invalidated when we do so?
Also, might it be beneficial to dynamically switch from 100% iGPU to Primus-style iGPU rendering and dGPU compute offload depending on framerate, memory depth, or other metrics? This would improve power efficiency if we are working with smaller datasets where the dGPU is overkill.