How do I organize my Unit/Integration Tests in a Dotnet Aspire Project

I have a dotnet Aspire project and I have added a web API project to the AppHost Project, I have written the Unit/Integration Test for the Web API in the same solution which contains the AppHost and Web API. Is it a good way or is there any better way to organize the Unit/Integration tests? Here is my folder structure.

Transferring this to the Aspire repo as it's not specifically about a sample.

Personally, I don't think this queston is about a specific tech but rather a philosophical question. And it is somewhat a like "should I drink tea or coffee".
There several layers to this questions:

"virtual" solution organition - i.e., how you organise your solution view in the VS, and
physical layout of files and folders on the hard drive.

Let's start with the latter.
For simpler solutions (e.g., not something like .NET SDK) the following folder structure could work. It's generally understood by vast majority of developers:
- src
  - ProjectA
  - ProjectB
- tests
  - Benchmarks
    - ProjectA
    - ProjectB
  - UnitTests
    - ProjectA
    - ProjectB
  - integrationTests
    - ProjectA
    - ProjectB

Some projects like to flatten the folder hirarchy and do something like this:
- benchmarks
  - ProjectA
  - ProjectB
- integrationTests
  - ProjectA
  - ProjectB
- src
  - ProjectA
  - ProjectB
- unitTests
  - ProjectA
  - ProjectB

Personally, I find the latter organisation less organised as I have to hunt down the "right" folder.
The "virtual" solution representation should mirror the physical organisation IMO, otherwise it creates a significant mental overhead and a barrier for contributors to navigate the solution and effectively contribute to it.
Beond that - organise the way that works for you and your team :)


Calendar

1 fluentui_calendar : Moved the declare-styleable attributes to fluentui_core attrs.xml & enum attributes to fluentui_calendar attrs.xml to reuse at the module level. Attributes are also prefixed with 'fluentui_' to avoid conflict with other UI library attributes

Merging fluentui_calendar


[Event Request] T167 Job - procedure OnInsert()

trigger OnInsert()
begin

....................

----------------------- ADD EVENT ----------------------
IsHandled := false;
OnInsertOnBeforeUpdateDefaultDim(Rec, IsHandled)
if not IsHandled then
----------------------------------------------------------
  DimMgt.UpdateDefaultDim(
    DATABASE::Job, "No.",
    "Global Dimension 1 Code", "Global Dimension 2 Code");
....................

[IntegrationEvent(false, false)]
local procedure OnInsertOnBeforeUpdateDefaultDim(var Job: Record Job; var IsHandled: Boolean)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


[WIP] Build custom argsort for GPU quantile sketching.

Customize cub radix sort.
Use argsort for quantile sketching. Merge sort is replaced by radix sort.
Support float16 without transformation.

This is to optimize GPU memory usage for GPU input with QuantileDMatrix and RMM. The idea is to use argsort instead of value sort for quantile sketching. In XGBoost GPU-based GK-sketching, we need to sort the input data according to its feature index and value. During the sorting process, we need to copy out the value and feature index, which costs 8 bytes per element, 4 bytes for value, and another 4 bytes for the feature index. Then efficient parallel sort requires a double buffer, leading to a total of 16 bytes overhead per element.
This PR introduces argsort to replace the value sort. By limiting the size of each batch to the maximum of uint32_t, we can use std::uint32_t as the sorted index. During sorting, we write only to the sorted index and fetch the data in-place without altering it. This way, with a double buffer, the overhead becomes 8 bytes per-element (uint32_t costs 4 bytes), hence halving the peak memory usage inside XGBoost (without accounting for the original input).
The optimization is only useful when input is on GPU (cupy, cudf) while using QuantileDMatrix, and RMM is enabled. For normal DMatrix, the data needs to be copied anyway unless it's constructed on GPU, not much optimization can be done there. When RMM is not used, XGBoost can split the data into small batches based on the amount of available memory, although the splitting might negatively affect the sketching result.
An additional benefit is that we can sort with custom iterators like transform iterator, which may or may not return a type that's supported by cub sort, and may cost arbitrarily large memory. I think this can be useful for other projects as well.
The initial benchmark shows some performance degradation using this argsort (~30%) even with merged sort replaced by radix sort, likely due to inefficient global memory access as we are now fetching data from global memory in a completely random manner. Without the argsort, we can almost always fetch contiguous data. For XGBoost, the cost can be justified as we run the sketching only once during training, and it's the bottleneck for memory usage.
I want to eventually upstream the changes to cub, depending on the developers there. At the moment, the customized radix sort is a drastically slimmed-down version of onesweep sort in cub.
Changes in the cub radix sort:

Remove specialization of single tile sorting. It requires an implementation of two-sweep sort, which is more difficult to customize.
Add custom digit extractor for composite types. (Entry in XGBoost)
Add support for the custom key iterator. The original implementation accepts only pointers.

TODO:
Benchmark results.

A related PR merged in cub https://github.com/NVIDIA/cub/pull/671 .


simplify synchronization object creation logic

@bors r+

:pushpin: Commit 20d8fb1798fd56af2011b52a0ae33ac78923b51c has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 20d8fb1798fd56af2011b52a0ae33ac78923b51c with merge f6a68eee84bab73bb4d49628936c788bd2873ab9...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing f6a68eee84bab73bb4d49628936c788bd2873ab9 to master...


MathJax v3 incompatibilities with WebKit of MacOS 10.9.5

[edit]The issue is finally not limited to Cocoa integration, but Mathjax v3 compatibility with WebKit of MacOS 10.9.5 which does not support all modern Javascript[/edit]
This issue is very special.
I use MathJax as a rendering engine in a Cocoa WebView of a MacOS application. The WebView is provided by the Webkit.framework.
Under MacOS 10.15, it works well with both MathJax v2 and MathJax v3, with both a (deprecated) WebView or a (recommended) WkWebView.
However, I have been told (and observed myself) that under MacOS 10.9.5 (which use legacy WebView, WkWebView is unavailable), the MathJax v3 version does not work as expected in this application-embedded context.
To be more precise, the Cocoa application establish a bridge between the WebView and the Objective-C code, and can call javascript methods or receive javascript callbacks.
With MacOS 10.9.5 and MathJax v3, a callback that should be called to mark the "ready" status of MathJax is never intercepted, as if it was not emitted at all.
Of course, I have built a minimal sample code to exhibit that.
http://chachatelier.fr/temp/TestMathJax-src.zip
And a few screenshots :
http://chachatelier.fr/temp/mathjax-v2-macos-10_15.png
http://chachatelier.fr/temp/mathjax-v3-macos-10_15.png
http://chachatelier.fr/temp/mathjax-v2-macos-10_9_5.png
http://chachatelier.fr/temp/mathjax-v2-macos-10_9_5.png

I don't have access to a iOS 10.9 system to check this out, so I can only give some general advice.
I notice that the finished-loading messages are wrapped inside a test for window.myDocument being defined.  I'm wondering if the window.myDocument.mathjaxDidFinishLoading_ is being skipped because window.myDocument is not there.  Could it be that MathJax is running before window.myDocument has been set up by the WebView?
Alternatively, could the line
self->webView.mainFrameURL = url.absoluteString

that loads a new document cause myDocument to become undefined?  It might be that the older version of WebKit acted differently when a new document is loaded.
I don't know much about WebKit, but would it be possible for your startup.ready() function to cause a change in the displayed DOM to verify that ready() is running (even if myDocument isn't defined)?  I.e., set some text in the window, or set the color of some text.  Then you could tell if MathJax is running its ready() function or not.
That's the best I can offer.  Hope that moves you forward with the process.

PS, your compileError and typesetError are in the wrong place in your configuration, and so are not being used. They need to be in the options section.

Thanks for the info.
I have fixed the misplaced compileError and typesetError
I have added a  element to see the DOM being modified
I have changed the way Mathjax is loaded to defer it to a explicit call to preload()
The behaviour is still the same :
under MacOS 10.15, no problem
under MacOS 10.9.5, the MathJax startup is not called (the "logger" does show the "configure" logs, but does not show the "startup callbacks" logs)
But the problem is not visible in Safari when hitting the "preload()" button, it is only exhibited by the Cocoa application when calling algorithmically the preload() method.
It also means that the problem is not the "myDocument" bridging variable since it is not involved in the preload() call.
New src and screenshots:
http://chachatelier.fr/temp/mathjax-v3-macos-10_15-deferred.png
http://chachatelier.fr/temp/mathjax-v3-macos-10_9_5-deferred.png
http://chachatelier.fr/temp/TestMathJax-src.zip

OK, thanks, that does suggest that MathJax is not running.  I wonder if you could add an onerror handler for the script that loads MathJax and see if that fires (and you might add an unload handler as well, so that one or the other should fire; old version of WebKit didn't run those handlers, but that was long ago).  Something like
script.onload = function () {
  document.getElementById('logger').innerHTML += 'MathJax loaded
';
}
script.onerror = function (err) {
  document.getElementById('logger').innerHTML += 'MathJax failed to load: ' + err.message + '
';
}

Perhaps that will tell you something.

'MathJax loaded' is displayed, 'MathJax failed to load: ' is not (even on 10.9.5). So the script loads, but the ready() callbacks are not called.
Is there another place I can add a useful log ?

Is there another place I can add a useful log ?

Well, one possibility would be to add try { at the beginning of the tex-svg.js file, and
} catch(err) {
  document.getElementById('logger').appendChild(
    document.createText((err.message + '\n' + err.stack)
  );
}

at the end of that file, and turn the logger element into a
 so the newlines will be honored, that might tell us something.
If you are willing to install the developer tools for mathjax-full and recompile and rebuild the components, we could do more granular testing, but the above might give a sufficient clue to what is happening that that might not be necessary.

I haved switched the  by a  (and it still works with \n rather than , as expected)
I have added that try/catch to tex-svg and tex-svg-full
But no more log is visible after loading, so no exception is raised, and the problem remains out of scope for now.
I can spend time with MathJax-src if necessary.

OK, thanks for trying that.  Here's something else to try before editing the code itself:
Use the configuration

Can you add that to your HTML file and see if that does the trick?  See the polyfill.io site for details, in case you need to have a local copy.

I have added polyfill-3.52.1.min.js, and the Map error is gone, but there is still a loading failure :
`preload_v3
>configureMathJax
<configureMathJax
exiting big try/catch
MathJax loaded

loader ready
>defaultReady

Loader failed: 'undefined' is not a function (evaluating 'this._configuration.find(function(t){return t.name===Q})')`

OK, _configuration is an array, and so this is using Array.prototype.find().  Apparently that version of WebKit is missing the find() method, and the polyfill hasn't added it.  The es6 polyfill is supposed to do that, but perhaps you need to specify it explicitly in your case. Try
https://polyfill.io/v3/polyfill.min.js?features=es6%2CArray.prototype.find

and see if that fixes it.  Otherwise, you may want to  add
Log(Array.prototype.find.toString());

to your startup logging and see if you get undefined or the definition of a function.

even after adding ?features=es6%2CArray.prototype.find, logging Array.prototype.find.toString() will just raise exception : TypeError: 'undefined' is not an object (evaluating 'Array.prototype.find.toString')

My mistake, it works with https://polyfill.io/v3/polyfill.min.js?features=es6%2CArray.prototype.find, while it fails when using a local polyfill.min.js.
 I have to investigate

I'm glad that adding the Array.prototype.find allows it to work.  I'm not sure how to capture the result into a local file.  I wonder if you could use XMLHttpRequest inside the WebView to get the file, and print it to the screen where you could copy and past it into a file.
There does seem to be an npm library (polyfill-library) so it may be possible to use that to get the file you need.
If you figure it out, please let us know!

I used curl, it is now OK with the local file. I did not understand at first sight how polyfill was distributed. This is a very dynamic build !

Thanks for all the answers, I would have never figured that out myself.

Thanks for being able to work through the debugging process.  That saved me trying to get a MacOS 10.9 virtual machine running!
Good luck with the project.  Glad you were able to work out a solution.

For information, my project is here : , and MathJax was really the only solution I found to render maths properly in such a context

I reopen the issue and change its title for the next step, new problem : MathJax.typeset() does not work with 10.9.5
error'null' is not an object (evaluating 't.body')body@file:///Users/chacha/Desktop/TestMathJax.app/Contents/Resources/Web/mathjax/v3.0.5/tex-svg.js:2:549062 assistiveMml@file:///Users/chacha/Desktop/TestMathJax.app/Contents/Resources/Web/mathjax/v3.0.5/tex-svg.js:2:541714 assistiveMml@file:///Users/chacha/Desktop/TestMathJax.app/Contents/Resources/Web/mathjax/v3.0.5/tex-svg.js:2:531874 assistiveMml@file:///Users/chacha/Desktop/TestMathJax.app/Contents/Resources/Web/mathjax/v3.0.5/tex-svg.js:2:542391 file:///Users/chacha/Desktop/TestMathJax.app/Contents/Resources/Web/mathjax/v3.0.5/tex-svg.js:2:352724 renderDoc@file:///Users/chacha/Desktop/TestMathJax.app/Contents/Resources/Web/mathjax/v3.0.5/tex-svg.js:2:352956 render@file:///Users/chacha/Desktop/TestMathJax.app/Contents/Resources/Web/mathjax/v3.0.5/tex-svg.js:2:355058 typeset@file:///Users/chacha/Desktop/TestMathJax.app/Contents/Resources/Web/mathjax/v3.0.5/tex-svg.js:2:3164585 typeset@file:///Users/chacha/Desktop/TestMathJax.app/Contents/Resources/Web/calculator-v3.js:21:27 onclick@file:///Users/chacha/Desktop/TestMathJax.app/Contents/Resources/Web/calculator-v3.html:18:51

The error is occurring in the assistive-mml extension that inserts hidden MathML that can be read by screen readers that handle MathML.  I'm not sure if that applies to your WebView or not.  Does VoiceOver work with your application?
One solution for now would be to disable the assistive-mml extension using
MathJax = {
  options: {
    renderActions: {
      assistiveMml: []
    }
  }
}

in your configuration.  Looking at the code, the adaptor.body() call is being made on the results of an adaptor.parse() call that is supposed to be parsing a serialized MathML string into the MathML DOM nodes.  That uses a DOMParser object, which might not be implemented in that version of WebKit, or might not be operating as expected.
If you want to enable the assistive MathML extension, we could do some experimenting to see if we could identify the issue.
[BTW, your calculator looks pretty cool.  I will definitely play around with it when I get some spare moments.]

Setting "assistiveMml: []" works ! Thanks.

Hi,
I have reports of my application not rendering anymore under MacOS 10.9.5 after I upgraded it to MathJax 3.1.4.
I have no logs yet, but do you already have hints of breaking changes in 3.1.3 or 3.1.4 that could have triggered new problems with the poor javascript support of MacOS 10.9.5 ?

I am re-closing the bug.
I can't reproduce it with 3.1.4

OK, thanks.  I'm glad 3.1.4 works for you.


Problem with the yum repository?

Hi,
I'm trying to install Aquilon using "yum -y install aquilon-postgresql" as stated here; http://www.quattor.org/documentation/2012/10/31/install-aquilon.html
Unfortunately, The download of http://yum.quattor.org/aquilon/aquilon-protocols-1.10-1.noarch.rpm fails with the following error : [Errno -1] Package does not match intended download
I've tried cleaning yum's cache but without success.
Is there a problem with the repository?
Julien

Yes, there is a problem with the way the package is generated, the version number is unfortunately hard-coded and a later version of the RPM overwrote an older one.
It's on my to-do list to fix.

I've just realized it's a duplicate of #16
Closing my ticket.


Ability to download release artifacts from self hosted github

Provide a detailed description of the proposed feature
The downloading code seems built for cloud-hosted github, not private installs at different URLs (https://github.com/Homebrew/brew/blob/master/Library/Homebrew/utils/github.rb)
Even with a custom strategy like GitHubPrivateRepositoryReleaseDownloadStrategy, updating the download_url does not seem to be effective.
What is the motivation for the feature?
private homebrew tap's on private self-hosted github
How will the feature be relevant to at least 90% of Homebrew users?
It wont.  But there would be more enterprise usage of homebrew if this was supported
What alternatives to the feature have been considered?
/shrug

I guess its not too bad to monkey patch
module GitHub
  API_URL = "https://api.whatever-I-want.com"
end

This feature sounds very similar to #11218.
I think the answer here is the same: https://github.com/Homebrew/brew/issues/11218#issuecomment-824759529

Could we add a variable for the api url instead of closing this ?
On Sat, Apr 24, 2021 at 9:49 AM Carlo Cabrera @.***>
wrote:

This feature sounds very similar to #11218
https://github.com/Homebrew/brew/issues/11218.
I think the answer here is the same: #11218 (comment)
https://github.com/Homebrew/brew/issues/11218#issuecomment-824759529
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Homebrew/brew/issues/11217#issuecomment-826112166,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIWWJQAOGT4GIZAVVJZYKTTKLSBNANCNFSM43LIJ7KQ
.

Sorry, but that's not how we use our issue tracker. As noted in the answer I linked to, we'll review a pull request for this.


fix: add the --follow parameter to the git log command

What is the purpose of this pull request?

[X] Bug fix
[ ] New feature
[ ] Other

Description
The previous createdTime is imprecise, add --follow parameter to the git log command so that it continues listing the history of a file beyond renames

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 50.304%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
697

Relevant Lines:
1253

💛  - Coveralls

I can't understand what's wrong, bro?

Two failing checks, I can't understand what's wrong, bro?

It seems that those check tasks restarted automatically, now all checks have passed.
But I still unable to merge the PR, because The base branch restricts merging to authorized users.
@Mister-Hope, @meteorlxy


Version Packages

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to master, this PR will be updated.
Releases
<EMAIL_ADDRESS>Patch Changes

#3037 41be6f1e Thanks @yonathan-pineda! - ---
Calendar

add rule on calendar styles

[ ] ⚠ If major release, don't forget to check if peerDependencies needs to be also updated

@toptal-anvil ping reviewers


Introduce support for named parameters in query using @name syntax.

Use a lexer to handle parameters
Fixes #54 by handling comments and strings.
Closes #60 by adding support for named parameters

Hm, something isn't quite right. I ran this with a much larger query and it's complaining about not enough arguments.

Looks like the error is coming from here I''ll look into how to make the sql library happy.

With this PR @param, @PARAM, and @Param would be 3 different variable names while SQL is usually not case sensitive. Should I normalize them all to upper or lower case before passing them out of the lexer? Or do you think having @Param appear as @PARAM would be more confusing?

@mlh758, first off, some beautiful and clean code there. IMHO, internally normalizing them into the same thing (upper or lower case, your call) would be nicer. I don't think those are generally case sensitive. I've got a tight schedule today, so probably won't be able to review it until tomorrow.
Quick question, is there a test case for using named parameters with prepared statements? I'm pretty sure the internal handling by the sql library would be exactly the same.
Also, please make sure to add your name to a "Special Thanks" section in the doc. There are several people that have done yeoman's work in adding new features and identifying bugs that are memorialized.

I'll normalize them all to upper case, and fix these tests I just broke.
Do you mean have I added a test to the driver_test file that actually hits the database? If so, I have not but I'm sure I could. I've been doing my end-to-end testing against a Vertica database at work.
How would you feel about adding a build tag of // +build integration to driver_test.go and renaming it driver_int_test.go? That way it would only hit a database when you ran it as go test --tags=integration. I wouldn't do that in this PR since I have enough changes as it is and it would require a change to your CI. The advantage would be that contributors could run go test ./... as normal and run all the unit tests in this package without needing to set up a vertica database for development.

Added some integration tests for the named parameters to driver_test.go I apologize if this leads to some commit spam, I can't run them locally.
@huebnerr where is the special thanks section?

Also, thank you for the compliment about the code quality but I can't take all the credit. I based the lexer's structure off a talk by Rob Pike.

Looks great. Thanks for the very valuable contribution!


data / ib - fetch futures data

Currently, the IB data fetcher class only pulls in stock data.
Let's add another method (keeping it somewhat uniform so we have a solid data fetcher interface for other data feeds) for futures data.
A couple things to keep in mind,

[x] the calendar that calculates RTH will be different since we'll fetching data from a different exchange

[ ] not all US futures have the same RTH. Eg, /MES and /ZC and /M6E have different market hours. If possible, let's have the rth field account for this. This is not possible based on the current support from trading_calendars, see https://github.com/quantopian/trading_calendars/issues/139.

[ ] Is there any easy way to test this?

I ran accros this randomly and just wanted to let you know that a fork I made of the great Quantopian prject has support for different market hours for futures: https://github.com/rsheftel/pandas_market_calendars

@rsheftel score. I was thinking I couldn't be the only one who might need something like https://pandas-market-calendars.readthedocs.io/en/latest/calendars.html. Thanks!


[core-http] Add api-extractor support

All non auto-generated packages in this repo run the api-extractor tool as part of build to create the api view. This issue is to do the same for core-http

already done in #8536

I found that like 2 seconds ago and was about to comment here!!


DDEV project isn't starting based on a case sensitivity issue

Describe the bug
From time to time a project isn't starting stating that the path to the root of the project has changed but in fact it hasn't.
To Reproduce
I doubt it will be possible to reproduce, even I am unable reliably. It just "happens" from time to time. Steps I apply which lead to the problem and the measure I apply to "solve" it are:

ddev start
The output says:

Starting drupal88alpha...
Failed to start drupal88alpha: project drupal88alpha project root is already set to /Users/myuser/sites/drupal88alpha, refusing to change it to /Users/myuser/Sites/drupal88alpha; you can `ddev rm --unlist` and start again if the listed project root is in error

I open ~/.ddev/global_config.yaml
Change the lower case of sites to the upper case Sites or the other way around for the particular site path as it is needed. Afterwards ddev startworks again flawlessly

I am running DDEV on MacOS 10.13.6 with APFS case insensitive. In the finder Sites is uppercase but in the terminal I am able to get to the Sites directory using the lowercase way with 'cd ~/sites`.
Expected behavior
I would hope DDEV wouldn't refuse to start based of case insensitivities if possible?
Version and configuration information (please complete the following information):

Host computer OS and Version: macOS High Sierra 10.3.6
Docker version information (use docker version) and insert it here.

Client: Docker Engine - Community
 Version:           19.03.4
 API version:       1.40
 Go version:        go1.12.10
 Git commit:        9013bf5
 Built:             Thu Oct 17 23:44:48 2019
 OS/Arch:           darwin/amd64
 Experimental:      false

Server: Docker Engine - Community
 Engine:
  Version:          19.03.4
  API version:      1.40 (minimum version 1.12)
  Go version:       go1.12.10
  Git commit:       9013bf5
  Built:            Thu Oct 17 23:50:38 2019
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          v1.2.10
  GitCommit:        b34a5c8af56e510852c35414db4c1f4fa6172339
 runc:
  Version:          1.0.0-rc8+dev
  GitCommit:        3e425f80a8c931f88e6d94a8c831b9d5aa481657
 docker-init:
  Version:          0.18.0
  GitCommit:        fec3683

ddev version information (use ddev version)

DDEV-Local version	v1.11.2
commit            	v1.11.2
db                	drud/ddev-dbserver:v1.11.0-10.2
dba               	drud/phpmyadmin:v1.11.0
ddev-ssh-agent    	drud/ddev-ssh-agent:v1.11.0
docker            	19.03.4
docker-compose    	1.24.1
os                	darwin
router            	drud/ddev-router:v1.11.0
web               	drud/ddev-webserver:v1.11.0

config.yaml contents for the misbehaving project
not limited to a single project. but the project file to which it happened in the example above is:

APIVersion: v1.11.2
name: drupal88alpha
type: drupal8
docroot: ""
php_version: "7.2"
webserver_type: nginx-fpm
router_http_port: "80"
router_https_port: "443"
xdebug_enabled: false
additional_hostnames: []
additional_fqdns: []
mariadb_version: "10.2"
nfs_mount_enabled: false
provider: default
use_dns_when_possible: true
timezone: ""

Do you have any custom configuration (nginx, php, mysql) in the .ddev folder? If so, have you tried without them?
no

Yeah, ddev works across many platforms, and as you know even APFS has a case-sensitive mode, and almost all linux systems use case-sensitive filesystems.  So ddev doesn't try to work around this problem.
You have many options though:

Change the name of the project to match the case you usually use.
Use bash autocomplete to navigate to the directory you want to use; it will never use the wrong case.

In general, in the world of computers... use the case that the operating system suggests. The alternative is way more pain than ddev alone is giving you right now.

ahhhhhhh I guess I've figured out what lead to the issue in the first place. Your second recommendation did the trick. Both works on my operating system, navigating to the same directory:
[myuser@mycomputer][~]
$> cd Sites
[myuser@mycomputer][~/Sites]
$> ..
[myuser@mycomputer][~]
$> cd sites
[myuser@mycomputer][~/sites]
$>

I must have accidentally used the upper case once for the sites directory. The file system might be case insensitive but the case actually matters for starting ddev. Meaning the case of the path I am currently in in the terminal has to be identical with the one stored in the according paths in ~/.ddev/global_config.yaml. Set all path cases to lower case in the yaml file and as a fail safe will add an alias to the terminal as well as the auto complete suggestion you've made. Now i understood what was causing the issue as well as leading to it. Thanks!


Only works for 1.6

Please update to the newer versions of Tundra Exploration

Please update to the newer versions of Tundra Exploration


Unable to access Grafana with running unraid unifi-poller template

I have installed the unraid template for unpoller (golift-unitifpoller) on my unraid server.
I've completed the variables in the template for the unifi (UDM) controller and influxDB. Once started, this works fine and I can see metrics logged to the DB in the logs from the docker instance.  I am unable to access through any grafana instance however.
I have installed the official grafana instance from the community apps on unraid and tried the unraid IP host address and receive a bad gateway error when I attempt to add the influxdb host ip as the new datasource. I suspect this is a docker networking issue but unsure how to resolve.  Any assistance on what I may be doing wrong?  Also, I thought this all in one docker for unpoller contained an instance of grafana but when this is running, no external port to host mappings exist exposing any grafana instance.
thanks

https://unpoller.com/docs/install/grafana

The important part is http://influxdb1:8086. As long as your containers are on a custom bridge, you can use the container name as the host name (through docker DNS).

Networking is currently set to Bridge.  So If I change this to custom (br0) , assign a IP for the unpoller docker, I can then use the address of http://influxdb1:8086 as the http url within the grafana docker?
also, I assume I would have to change the influxDB host url in the main unpoller docker as well to accomodate the networking change?

I don't know what br0 is, but probably not, unless you made it.
https://docs.docker.com/network/bridge/

also, I assume I would have to change the influxDB host url in the main unpoller docker as well to accomodate the networking change?

Did you try that hostname in Grafana?

Yes I did, and received the same error. I suspect its something I am doing wrong with the networking.
in my unraid docker section, I have 2 dockers. (unifi-poller and grafana)
The network in both of them was set to bridge and when I originally tried adding the http://influxdb1:8086/ as the http url in the grafana docker I receive the bad gateway message. When I set it to http://ip-of-myunraid-server:8086, I received a "cannot fetch" error... so thought I was progressing but maybe not.
in my unraid template, I have the following networking options outside of bridge. (host, none, or custom (br0) , where br0 refers to the main host NIC)
Ill have a look at the docker network bridge link you have set.

Do unifi-poller logs indicate it's writing metrics to influx?

yes, have just put back the unpoller container network back to bridge and in the logs I can see the following. (occurring every 30 secs)
2022/03/20 22:18:08.949749 updateweb.go:213: [DEBUG] Found 1 site(s): default
2022/03/20 22:18:08.954989 logger.go:17: [INFO] UniFi Metrics Recorded. Site: 1, Client: 37, Gateways: 1, UAP: 4, USW: 4, IDS/Event/Alarm/Anomaly: 0/0/0/0, DPI Site/Client: 0/0, Point: 136, Fields: 4431, Err: 0, Dur: 5ms

Yup looks good. If this app can reach it, grafana should be able to as well.

Just had a look in the Grafana docker logs and can see the following.
logger=data-proxy-log userId=1 orgId=1 uname=admin path=/api/datasources/proxy/11/query remote_addr=<IP_ADDRESS> referer=http://<IP_ADDRESS>:3000/datasources/edit/e5c2iEPnk t=2022-03-20T22:26:47.23+1300 lvl=eror msg="Data proxy error" error="http: proxy error: dial tcp: lookup influxdb1 on <IP_ADDRESS>:53: no such host"
Looks like a docker dns error. Ive not changed anything in that area so maybe there is some other step I have missed ?

You should put your containers in a custom bridge. You make one with the docker network create command documented on the link above.

closing as this looks resolved


Error on Dial, should it raise a Ruby exception?

Should this not raise a Ruby exception to do a begin/rescue claus around, or should we be parsing the event that returns?
[2012-02-07 16:49:23] DEBUG Punchblock::Connection::XMPP:
<presence<EMAIL_ADDRESS>id="1ae0d37c-ffaa-438c-9e5e-13d52c15f148"<EMAIL_ADDRESS>  <end xmlns="urn:xmpp:rayo:1">
    <error>Could not negotiate call</error>
  </end>
</presence>

[2012-02-07 16:49:23] DEBUG Punchblock::Client: Handling event #<Punchblock::Event::End reason=:error, headers_hash={}, call_id="xejfqve-lx7-tt9zt738yiay", component_id=nil> with source .

This only shows an error in the event, and does not raise a Ruby exception.

I'm aware of our failure to handle this, and have some ideas, both for
Adhearsion and Rayo. I will work these into the Rayo spec ASAP and
swing back round to Adhearsion later.
Enviado via iPad
Em 8 Feb 2012, às 01:52, Jason Goecke
<EMAIL_ADDRESS>escreveu:

Should this not raise a Ruby exception to do a begin/rescue claus around, or should we be parsing the event that returns?
[2012-02-07 16:49:23] DEBUG Punchblock::Connection::XMPP:
'''xml

Could not negotiate call

 '''
[2012-02-07 16:49:23] DEBUG Punchblock::Client: Handling event #<Punchblock::Event::End reason=:error, headers_hash={}, call_id="xejfqve-lx7-tt9zt738yiay", component_id=nil> with source .

Reply to this email directly or view it on GitHub:
https://github.com/adhearsion/adhearsion/issues/92


RDS Aurora Serverless v2 support

CloudFormation added support for Aurora Serverless v2 on Oct 5, 2022.
https://aws.amazon.com/about-aws/whats-new/2022/10/amazon-aurora-serverless-v2-supports-aws-cloudformation

The only thing I saw in the CloudFormation release notes was ServerlessV2ScalingConfiguration which was implemented in this PR #2077 (in a release likely today). Was there anything else you know of needed?

Oh, excellent. I was not aware it has been implemented already.
I've found that I also need to remove EngineMode from AWS::RDS::DBCluster for serverless-v2, that's working properly though.
Thanks! I'll wait for the release.


CI: add separate workflow for REUSE compliance

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>5:
0.0%

Covered Lines:
1367

Relevant Lines:
1367

💛  - Coveralls


Ideas about new data chart.

I think that the new data chart is a very good thing.  I'm going to purpouse some ideas.
1º vertical values. would be a good idea  put the vertical charts

2º The value in the center. there is an option to put the center line in the midle with the value.

3º The font size of actual values could be big?
4º multiple values in the same chart? I don't know if this is interesting for anybody.
Question about moving average. what happens with you mooving average if you put true wind values of 5º and 355º give averages of 180º??
Mxtommy kip is practically perfect! thank you for your wrork.

Hi. Thank you for the feedback. This was step one for getting to what you asked in the other issue 😉
Angle average calculation for degrees is not done. Kip works in Degrees by default and I need to check why it's not in base radian.

I have done Exponential Moving Average (EMA) and Double EMA calculations but not on angles, Angles should use calculations from link found in the Issue with tang2().

I had to get to a few bugs and it was good as a v1 to get feedback!

If you want to PR some changes you are more than welcomed!
To keep widgets somewhat easy to use for users, their benefits accessible and maintain in code. We will need different ones. The Data Chart is about visual trends over time and so the values are not that critical, hence the smaller size.
For racing/sailing charts, I think we need different widgets with their own specific layouts and config: vertical with larger numbers, multiple series, polar data maybe, different scales, custom settings, etc. I would do new widgets for that purpose. What do you think?

I have this ideas,

Are you planning to add new widgets?

Note that EMA and DEMA are not yet exposed as I need to add angles average. Too much stuff to do!

I'm not programmer, I'm trying your widget. i only can make that for you. and if I say that you can cretate a data chart widget the minimum is try the widget and look the maximum posibilities with this widget.
yes, with this charts we only have the problem with the wind direction magnetic.... and only with values close to 0 or 360.
I'm thinking in things as:

Chart of open cpn, are the right side charts. polar performance and SOG

a chart with VMG and porlar VMG performance

Note that EMA and DEMA are not yet exposed as I need to add angles average. Too much stuff to do!

I'm aware that EMA and DEMA haven't been revealed yet, but I believe it's important to understand the impact of turning the Moving Average on or off. This can be observed in the chart you've made. The differences are around 30 degrees, but with the Moving Average enabled, it essentially becomes a straight line.

Are you on Discord? I found user petsi but not certain it is you.
I'm no math expert and any help or advice would be great. If you are Discord, after I'm done with a complex bug, maybe we can setup a live call so we can talk and I can better understand?

I'm putting out an improved average for directions.

Working with the moving average in Node-RED, I realized that different situations require a different number of values in the moving average. For example, if you want to know stable wind values, you can include a high number of values in the moving average, like 40 values. If what you want to know is the apparent wind angle, it's fine to make the average with the last 10 values. If you want to know the current performance, you use relatively small average values, like 10. If you want to know how performance varies over time, for example with a new trim, you can use higher values, like 40. By high or low values, I mean the number of elements you will take into account in the moving average. Higher values generate more stable values but with a lot of delay to changes, smaller values generate less delay but the values are more unstables.
I want say with this, that put the values that you want assign is important.

To implement your strategy effectively, you would:
Implement the moving average to calculate average values for the data chart. This involves selecting a suitable number of data points for each average calculation, based on the specific requirements of the data you're analyzing. For more stable values, such as average wind speed over a longer period, you might choose a larger number of data points. For more responsive values, like sudden changes in wind direction, you might choose fewer data points.
Create a plugin to Apply the moving average directly to the environment.wind.angleApparent data path. Since many derived data values depend on the apparent wind angle, stabilizing this value using a moving average can significantly improve the reliability and usefulness of the calculated derived data path values. By smoothing out the fluctuations in apparent wind angle, you ensure that the derived data, which might include calculations related to sail settings, navigation, or performance metrics, are based on more consistent and reliable input data.

A dataset gives that. something is bad...

This appears ok I put in the display paths the paths that I'm testing.

and here the problem that says. a value of -170 and +170  is not 0

It because this widget does not support rolling (wrap-around) scales. You'll not get what you want with this widget. You can check the calculations if you want.
As for data, when working with time scales, all the data has to be in sync for both time and frequency to create stats and data points, on time points. The way plugins are built at the moment, it's not really possible or easy. They all come at different times and different rhythm. Try and plot multiple paths and create an average at a specific regular time  interval. You'll see the challenge.
I think we need a performance plugin that does that and sends data in sets.

Working with the moving average in Node-RED, I realized that different situations require a different number of values in the moving average. For example, if you want to know stable wind values, you can include a high number of values in the moving average, like 40 values. If what you want to know is the apparent wind angle, it's fine to make the average with the last 10 values. If you want to know the current performance, you use relatively small average values, like 10. If you want to know how performance varies over time, for example with a new trim, you can use higher values, like 40. By high or low values, I mean the number of elements you will take into account in the moving average. Higher values generate more stable values but with a lot of delay to changes, smaller values generate less delay but the values are more unstables.
I want say with this, that put the values that you want assign is important.
the data set make that... I think...
I'm analazing a file with mxtommy kip and When you tack in the boat, the values of the data chart take about 10 seconds to detect that you are making the tack, more or less.

Yes. That's what a simple moving average (SMA) is. Data Chart uses a 0.25 ratio of data points. We need different widgets for different purpose. Remember Kip has different goals then Grafana. We just need to have what sailors need to see, presented in a "marine" format. So if we explore outside that scope, we'll find gaps. Grafana has its place there and can be integrated into Kip. Just saying...


[WIP] Log only to file - not console [Finishes #153173585]

... following experience of miner hung on lager.
Also console logging is not needed.

I plan to use it locally for a while before proposing it for inclusion.

I see two reviewers approved. CI passed.
I locally merged recent master and: ran make test (passed), and build production release and checked logs produced.
Merging.


Issues found

Hi,
Great idea! I have however picked up the following two issues:

AJAX calls fail
Copy paste into form fields don't work (not sure if this is intended)

Thanks , I'll check up on it 😄 😄 😄

any update on this? I would love to use this package but I have AJAX calls which are a must

Sorry for my laziness… , I fixed Ajax Error :smile: :smile:

I had a feeling it was due to the collapsewhitespace. My other issue post is null then. Unless you choose to make it optional in config for those that want collapsewhitespace and don't use Ajax.

@HDVinnie , I changed some code ... it will only work on GET ...

do you plan to add POST support?


Adding sign out warning dialog to the home screen

Adds a sign out warning dialog informing users their unsaved data will be lost.

Fixes #1683

Adds a sign out confirmation screen to Ground, warning users that unsaved data will be lost on sign out. Currently this is only implemented for sign outs from the home screen fragment.

[x] Adds sign out dialog, fragment and viewmodel
[x] Replaces navigation call on home screen with launch of sign out
[ ] Copy sign off
[ ] Sign out from other screens

Sign out of ground from the hamburger menu and verify that the screen comes up.
Clicking cancel should exit the dialog.
Clicking sign out should sign the user out.

https://github.com/google/ground-android/assets/77077870/fc4ea211-5796-4802-92bb-79e12d20d683
@shobhitagarwal1612   PTAL?

I'm OOO till next week. @shobhitagarwal1612 could you PTAL?

@amegantz what do you think about the copy?

Unsaved data
If you sign out, your unsaved data will be discarded.
[Cancel] [Sign out]


Setup bash/zsh command-line completion for nargo

Problem
Tab-completion in bash and zsh for nargo would be helpful so that typing partial commands like nargo co<TAB> or nargo n<TAB>, the commands are completed to nargo compile, nargo new, respectively.
Happy Case
Setup tab completion for the commonly used shells bash and zsh
Workaround
Yes
Workaround Description

Use "short" versions of commands where available (e.g. check, execute) or else enable them for other commands
Type the whole command

Additional Context
No response
Project Impact
Nice-to-have
Blocker Context
No response
Would you like to submit a PR for this Issue?
None
Support Needs
No response

Kev mentioned there are some addons we can use for clap to improve the CLI experience: https://github.com/noir-lang/noir/issues/3159
Specific to this issue there is clap_complete.


Front-end Utils Refactor

Background
Remove bad utils (e.g. toArray, equalsLowerCase, etc...(
Linked Issues & Documentation
Definition Of Done
Describes in 1-2 sentences what the output of the Task is, so that it can be verified by another team member easily. “Feature X is implemented as described and can be verified by using it”
Tasks

[ ] Implement across app
[ ] QA Functionality

@just-a-node @jwtong Let's plan 5pm CET Monday 13th of may for deploy.

@just-a-node @jwtong Let's plan 5pm CET Monday 13th of may for deploy.

@jwtong provide me with the link for QA


Ignore page.goto timeout

Steps to reproduce
Tell us about your environment:

Puppeteer version: 5.5.0
Platform / OS version: macOS
URLs (if applicable):
Node.js version: 12.18

What steps will reproduce the problem?
page.goto(url, { waitUntil: "domcontentloaded" });
Page loads (but still loading due to many subsequent calls on the page but due to network issues it takes a long while to finish loading). puppeteer then throws an error for being too long.
What is the expected result?
page loads the html with the selector I need, and I expect it not to throw error so I can proceed with my evaluate calls.
What happens instead?
page loads the html with the selector I need, but it throws an error so I cannot proceed with my page.evaluate calls.
How can I ignore the timeout and proceed with my javascript calls? I guess what I want to know is, I don't want goto to dictate whether a page is timedout or not but to use selector to determine it.

I think you have a couple options:

page.goto(url, { timeout: <longer>, waitUntil: "domcontentloaded" });
try/catch around goto

Since you already have the earliest event for goto (domcontentloaded), I would recommend increasing the timeout.

try networkidle2 instead of dom load.


Proper usage of script/build for a user-level install?

Building from source was not a problem for me up to (and including) v1.10.2.
Since then I have been met with multiple problems and errors while trying to build. Today I attempted to build v1.12.6 and received an error identical to #12549.
Reading through the documentation it seems the workflow is to install at the system level. This doesn't work for me as I always install to "$HOME/.local" so that I replicate my build on servers where I don't have admin (root) access.
Previously I would do the following.

./script/build
./script/test
./script/grunt install --install-dir "$HOME/.local"

Every version since v1.10.2 (v1.11.* and v1.12.*) have failed to build on my system (step 1). I am unsure how to continue since the documentation only includes directions for a system-level install.

/cc @nathansobo

System info:

OS: Ubuntu 15.10 (64 bit, Virtual Box VM)
node: v6.9.1
npm: v3.10.9
gcc: v5.2.1

https://github.com/atom/atom/issues/13240?

@Ben3eeE Related, but not the whole issue.
Just remembered about the script/clean script. Why is this not documented in "docs/build-inscructions/linux.md"?

Open a PR?

Closing this issue because documenting script/clean and issue #13240 is what needs to be done.


Add settings/options

Move "ignore pinned" tabs to settings
Remember title vs URL setting

All original suggestions implemented


Stacked documentation on spark-design page

Added some documentation on the design/implementation of the Stacked class to the spark-design page

Accidentally included the filter bug fix in this PR


ApacheBenchmark testing yields null reference error

I just tried out your sample application in Yeoman for creating Web API application.
When I run an ab test it yield a null exception error after a few thousand requests.
ab -n 10000 -c 4 http://localhost:5001/messages

yields:
System.NullReferenceException: Object reference not set to an instance of an object
  at Microsoft.AspNet.Server.Kestrel.Networking.UvHandle.ReleaseHandle () [0x00000] in <filename unknown>:0
  at System.Runtime.InteropServices.SafeHandle.Dispose (Boolean disposing) [0x00000] in <filename unknown>:0
  at System.Runtime.InteropServices.SafeHandle.Finalize () [0x00000] in <filename unknown>:0

You should file this bug on https://github.com/aspnet/KestrelHttpServer


Text is not selectable and text only pdf is 11mb

I guess this module creates a raster of every page instead of a proper pdf?

Interested in getting a sample file and looking at it. Please upload and share link.

@cozzbie I will be able to upload in 2 days. I am traveling at the moment.
RUN npm i && npm cache clean

COPY fonts/* /usr/share/fonts/

COPY . .
RUN ./node_modules/.bin/gulp

The above will output a 100k pdf with selectable text.

I have the same problem. Simple HTML file
Testing
 comes out as single image when run on OS X but works when run on Ubuntu.
test-darwin.pdf
test-ubuntu.pdf
Versions:

html-pdf 1.4.0
PhantomJS 1.9.19
Ubuntu 14.04
OS X 10.10.5

Same problem here
according to this issue in markdown-pdf https://github.com/alanshaw/markdown-pdf/issues/37 the last version of phantomjs fixed the issue
i guess it's time to bump up phantomjs #106

Fixed in v2.0.0 by upgrading to phantomjs v2

I'm seeing this issue on Ubuntu Trusty 14.04, but runs perfectly on OS X 10.11.6. Would you be able to look into this @marcbachmann?
Versions
html-pdf: 2.1.0
phantomjs-prebuilt: 2.1.13
Examples
Generated by OS X 10.11.6
Generated by Ubuntu Trusty 14.04

Hi @marcbachmann,
I'm also running into the same issue. When html-pdf runs on Ubuntu, the PDF contains only images and no selectable text. Any idea why this may occur? Thanks!

Install libfontconfig. That should help.

@marcbachmann thanks for the suggestion! Appreciate the speedy response. Seems like I have libfontconfig installed on my Linux server. Have you heard of any other potential solutions?

Any news about this issue? Experiencing the same problem.

I managed to identify the problem and solve it.
For anyone in the same situation: The problem happens probably because you are using custom fonts in your document, and loading them remotely (i.e: Google fonts). Loading the truetype or web font from the remote URL causes PhantomJS to rasterize the font in the PDF instead of adding text as just text.
So the solution is to include the fonts locally. Create a fonts directory where your templates or .html files are, and then include the local fonts. The size reduction will be effective and the PDF will be a "text PDF" also, instead of a rasterized image PDF.
So change this:

    <meta charset="utf-8">
    <title>Your title</title>
    <link href="https://fonts.googleapis.com/css?family=Lato" rel="stylesheet">
    ...

For this:

    <meta charset="utf-8">
    <title>Your title</title>

And it will work on your remote server as well.

Cheers @DigitalLeaves for the excellent workaround 👍

Worth mentioning that woff2 fonts aren't supported. Use ttf instead.

In the end, I had to actually install the fonts locally in /usr/share/fonts of Amazon Linux to finally get the PDF text to be selectable. Including the ttf font files locally and linking to them made the text show up in the right font, but it wasn't selectable/editable in the PDF. Only when installed as a system font followed by sudo fc-cache was the text selectable.

@DigitalLeaves Thanks, that worked!


[metal] Use provided provider id in config if defined

Do not perform project lookup by name when projectID is defined in the config.

Ah cool,
Thanks for cleaning this up!


[Bug] [Major] "Предложи нова кампания" feature is not working

Describe the bug
after filling all the information about the campaign the "Изпрати" button is not woking
To Reproduce
Steps to reproduce the behavior:

Go to 'Homepage'
Click on 'Дарителство'
Click on  'Предложи кампания' https://dev.podkrepi.bg/campaigns/create
Fill all the information about the campaign
Tick all 2 checkboxes
Click on "Изпрати" button

Expected behavior
The campaign is sent successfully
Actual behavior
After clicking on the "Изпрати" button nothing happens
Desktop
OS: macOS Catalina 10.15.7

Browser: safari
Version [15.4]

Is this bug still relevant? I managed to reproduce it, but I don't find any link which can lead me to this page as it is described in the retro steps above. I had to manually type the URL in trhe browser in order to reach the page and try to create a new campaign.
So I wonder whether this feature is still something we need or it was temporary drooped out of the current release?

The ticket seems to be outdated.
Currently we do not support the page.
I will close the ticket. If we enable the page again in the future and the issue is still valid, please feel free to reopen it.


[feature] New verify command in the Docker-based builder

Describe the solution you'd like
Currently to use the Docker-based builder only has build and dry-run commands that take as input the source, the docker imager, etc. We'd like to have a new command (e.g., verify) that takes as input in generated BuildDefinition or provenance, builds the binary the same way the build command does, and verifies that the generated binary has the same digest specified in the provenance.
Describe alternatives you've considered
Alternatively we could just pass in the BuildDefinition, but that does not contain the binary digest and cannot be used to do a meaningful verification.
Additional context
This could be used locally to verify that the provenance generated on GitHub is valid.

I'm curious if you want the input to be a provenance file,  in the future?

This could be used locally to verify that the provenance generated on GitHub is valid.

what do you mean by "valid"?


docs(chips): fix docs showing deprecated class name

Describe pull-request
The deprecated class name sdds-chip__small was showing ing docs.
This restores sdds-chip-sm

Is this added in the migration document? Otherwise, LGTM.

Yes!


Variable definition problem

Code error message:
These two variables are defined after the reference.
"multiLMs-lang-neutral-subnets\helpers\data_processor.py", line 47, in init
num_examples = int(len(tokenized_text)/ block_size)
UnboundLocalError: local variable 'block_size' referenced before assignment
"multiLMs-lang-neutral-subnets\helpers\data_processor.py", line 67, in init
logger.info("Saving features into cached file %s", cached_features_file)
UnboundLocalError: local variable 'cached_features_file' referenced before assignment

Hi, thanks for pointing out the issue. It's already resolved!


I have add a GetHession function to this library. Does anyone know how I should submit to the library owner

I recommend creating a PR against this repository with your changes for review.

I've already submitted PR, and if you're considering branching into base, you might need to change some of the instructions in README.MD


Avoiding anti-aliasing when rendering bitmap fonts on a canvas

Dear Tom
First of all, thank you very much for this library; it is already of great use to me.
I am reaching out to you because I am currently struggling with rendering a small (5x7) BDF font on a canvas: The font is hard to read due to anti-aliasing effects (see screenshot below). Do you have any experience with this, or do you know how to get a pixel-perfect bitmap rendering? I already tried to disable canvasContext.imageSmoothingEnabled without any luck.

Background: I want to render the website on a low-resolution LED matrix.
Thank you very much for any ideas.
Cheers and have a nice day, remolueoend

I have found a working solution in the meantime. Applying following CSS rules to the canvas solves this issue:
canvas {
  image-rendering: -moz-crisp-edges;
  image-rendering: -webkit-crisp-edges;
  image-rendering: pixelated;
  image-rendering: crisp-edges;
}


Make empty array Criteria parameters invalid

Edit: Added another deprecation PR here https://github.com/shopware/platform/pull/1421
1. Why is this change necessary?
Consider the following code:
/** @var string[] $ids */
$ids = $this->getIds();
$result = $this->productRepository->search(new Criteria($ids), $context):
// do something with results

This code seems rather unsuspicious, until $ids is an empty array. In that case, ALL entities are loaded.  This shows that it currently is easy to load all entities without actually planning to, resulting in unexpected behaviour.

2. What does this change do, exactly?
This change distinguishes between an implicit not-setting some ids, and an explicit setting of an empty id array. The former makes Criterias behave like before, just calling new Criteria() doesn't change any behaviour.
3. Describe each step to reproduce the issue or behaviour.
What changes is the behaviour when doing something like new Criteria([]). Up to now, this is equivalent to new Criteria(), my change will throw an InconsistentCriteriaIdsException in that case - just like it currently would in case of new Criteria([null]).
4. Please link to the relevant issues (if any).
5. Checklist

[X] I have written tests and verified that they fail without my change
[X] I have squashed any insignificant commits
[X] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[X] 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.

I think this needs to target the major branch

I think you're right, changed and rebased!

Reminds me on a bug where a tool loaded all the products when the cart was empty (hence the ids in the criteria as well)...

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-11909
Please use this issue to track the state of your pull request.

Hey @soebbing ,
With the renaming of our branches. This PR got closed. Can you create a new one pointing to trunk?


Update django_select2.js

"formset:added" EventListener for InlineForms and discard Parent Element with class name: ".empty-form"

"formset:added" EventListener for InlineForms and discard Parent Element with class name: ".empty-form"

Hi @jasmanysanchez, I appreciate your willingness to contribute. However, it is hard for me to review changes without any description. Please consider opening an issue before submitting a pull-request. Make sure your submissions have tests as well to accelerate the review process. Best Joe


chore(StakeManager.spec): add invariant sumOfMultiplierPoints

Checklist
Ensure you completed all of the steps below before submitting your pull request:

[ ] Added natspec comments?
[ ] Ran forge snapshot?
[x] Ran pnpm lint?
[ ] Ran forge test?
[x] Ran pnpm verify?

I'm going to hold back merging this one until we have CI green again for formal verification. Otherwise things like https://github.com/logos-co/staking/pull/54#issuecomment-1976040945 will slip through the cracks.

This one can be closed as it turns out to be obsolete.


feat: Configurable callback URL for argo-workflows #11252

Fixes #11252
Motivation
To allow custom callback URL for SSO Login
Modifications

Updated server/auth/sso/sso.go to include new function GetCallbackURLPath that will return default callback path as /oauth2/callback or the callback path from the configured redirectURI in workflow configuration.
Updated function getRedirectUrl in server/auth/sso/sso.go to refer new function GetCallbackURLPath for callback path
Updated Interface in server/auth/sso/sso.go
Updated server/apiserver/argoserver.go - function newHTTPServer to refer new function GetCallbackURLPath for callback path.

Verification
Verified this change by running argo-workflow on my AKS cluster with configurable callback url.
Configuration
sso:
    issuer: https://sts.windows.net/xxxx-xxxx-xxx-xxxx/
    clientId:
      name: argo-workflow-oidc-secret
      key: client-id
    clientSecret:
      name: argo-workflow-oidc-secret
      key: client-secret
    redirectUrl: http://workflows.my.org.com/workflows/callback
    scopes:
    - "groups"
    - "emails"
    rbac:
      enabled: true

Workflow Redirects post initiating SSO Logins

Controller cannot start up. You can find logs in https://github.com/argoproj/argo-workflows/actions/runs/5612630483/job/15209661684#step:20:1


Refactor Parser code and remove unused code

What type of PR is this：

[ ] bug
[ ] feature
[x] enhancement
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Change the Analyzer from the original new object to a static class
Remove the Relation return value of the Analyzer
Delete InsertRelation, QueryAnalyzerV2
Use the Visitor mode instead of if-else

[FE PR Coverage check]
:heart_eyes: pass : 436 / 464 (93.97%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/qe/StmtExecutor.java
0
1
00.00%

:large_blue_circle:
com/starrocks/analysis/ShowStmt.java
0
1
00.00%

:large_blue_circle:
com/starrocks/sql/analyzer/ViewAnalyzer.java
1
2
50.00%

:large_blue_circle:
com/starrocks/sql/ast/AstVisitor.java
9
15
60.00%

:large_blue_circle:
com/starrocks/analysis/Subquery.java
5
7
71.43%

:large_blue_circle:
com/starrocks/http/rest/TableQueryPlanAction.java
8
10
80.00%

:large_blue_circle:
com/starrocks/sql/analyzer/ShowStmtAnalyzer.java
5
6
83.33%

:large_blue_circle:
com/starrocks/sql/analyzer/QueryAnalyzer.java
276
289
95.50%

:large_blue_circle:
com/starrocks/sql/InsertPlanner.java
34
35
97.14%

:large_blue_circle:
com/starrocks/analysis/DropWorkGroupStmt.java
1
1
100.00%

:large_blue_circle:
com/starrocks/analysis/AnalyzeStmt.java
1
1
100.00%

:large_blue_circle:
com/starrocks/sql/parser/SqlParser.java
6
6
100.00%

:large_blue_circle:
com/starrocks/sql/analyzer/Analyzer.java
27
27
100.00%

:large_blue_circle:
com/starrocks/sql/analyzer/SelectAnalyzer.java
2
2
100.00%

:large_blue_circle:
com/starrocks/analysis/BaseViewStmt.java
1
1
100.00%

:large_blue_circle:
com/starrocks/sql/parser/AstBuilder.java
9
9
100.00%

:large_blue_circle:
com/starrocks/analysis/CreateTableAsSelectStmt.java
1
1
100.00%

:large_blue_circle:
com/starrocks/sql/analyzer/CreateTableAnalyzer.java
1
1
100.00%

:large_blue_circle:
com/starrocks/sql/StatementPlanner.java
6
6
100.00%

:large_blue_circle:
com/starrocks/sql/optimizer/transformer/SubqueryTransformer.java
8
8
100.00%

:large_blue_circle:
com/starrocks/analysis/InsertStmt.java
5
5
100.00%

:large_blue_circle:
com/starrocks/sql/analyzer/AggregationAnalyzer.java
1
1
100.00%

:large_blue_circle:
com/starrocks/analysis/CreateAnalyzeJobStmt.java
1
1
100.00%

:large_blue_circle:
com/starrocks/sql/analyzer/CTASAnalyzer.java
2
2
100.00%

:large_blue_circle:
com/starrocks/sql/analyzer/InsertAnalyzer.java
4
4
100.00%

:large_blue_circle:
com/starrocks/sql/analyzer/AST2SQL.java
1
1
100.00%

:large_blue_circle:
com/starrocks/sql/ast/QueryRelation.java
1
1
100.00%

:large_blue_circle:
com/starrocks/analysis/CreateWorkGroupStmt.java
1
1
100.00%

:large_blue_circle:
com/starrocks/sql/ast/SelectRelation.java
2
2
100.00%

:large_blue_circle:
com/starrocks/sql/analyzer/ExpressionAnalyzer.java
13
13
100.00%

:large_blue_circle:
com/starrocks/statistic/StatisticExecutor.java
3
3
100.00%

:large_blue_circle:
com/starrocks/analysis/AlterWorkGroupStmt.java
1
1
100.00%


feat(arr/webhook): add path parsing for risky sonarr season folders

adds a property to the searchee for the source of where the searchee was "requested" from - could be useful down the road
when a path is sent via webhook and matches the IGNORED_FOLDERS regex, this will allow risky/partial matching to parse the path for the parent folder.

this allows for sonarr libraries with season folders to be parsed, as well as enabling season packs sourced from usenet to be searched upon completion with an import script where otherwise "Season 01" would be ignored entirely or searched erroneously as "Season 01" yielding non-matching results in 99% of cases.

adds a property to the searchee for the source of where the searchee was "requested" from - could be useful down the road

how are you thinking about this given that searchees can be involved in multiple sources over the course of their life, given that they are persisted to the DB? for example, a searchee could be created via a webhook for the first time and then be used for a periodic search later.

adds a property to the searchee for the source of where the searchee was "requested" from - could be useful down the road

how are you thinking about this given that searchees can be involved in multiple sources over the course of their life, given that they are persisted to the DB? for example, a searchee could be created via a webhook for the first time and then be used for a periodic search later.

the only fields I'm aware of being stored for the searchees are the name and timestamps, neither the torrent or searchee table appears to store anything that this would affect, this is just dynamic during the searchee usage after that particular generation if I'm not mistaken

adds a property to the searchee for the source of where the searchee was "requested" from - could be useful down the road

how are you thinking about this given that searchees can be involved in multiple sources over the course of their life, given that they are persisted to the DB? for example, a searchee could be created via a webhook for the first time and then be used for a periodic search later.

I've removed the searchee source stuff from this PR, will update the PR description stuff. I still have that local if it ever becomes necessary/wanted.


Remove JSDoc annotations in TS files

This PR can be seen as a continuation of #4135.
I removed the JSDoc annotations in .d.ts files.
Before:
/**
 * Some docs.
 * @param {number} a
 * @param {number} b
 * @returns {number}
 */
export function add_u32(a: number, b: number): number;

After:
/**
 * Some docs.
 */
export function add_u32(a: number, b: number): number;

I did this because JSDoc annotations (so @param and @returns) are just unnecessary noise. They just repeat the function signature and contain zero information.
Note that this only affects the generates TS files. The JSDoc annotations are wanted in JS files and are remain unchanged.

With this PR, I would actually consider the form of generated .d.ts files to be good. At least for my use case, they are now exactly what I want.

Here's an SO post explaining doc comments in TypeScript. The top answer explains it quite well.


Dutch translation

Hi Sarah,
I've added the dutch translation (Netherlands) for this project. If I missed something, please let me know :)
Happy new year and thanks for this project!

Thank you Glenn! This is so great! Happy new year!


Testing compare_cumulative_psf.py

Using the example on https://gammasim.github.io/gammasim-tools/applications.html#module-compare_cumulative_psf , I run in the following problem:
python applications/compare_cumulative_psf.py --tel_name North-LST-1 --model_version prod4 --pars lst_pars.yml --data PSFcurve_data_v2.txt

INFO::db_handler(l508)::_getSiteParametersYaml::Reading DB file /workdir/external/simulation-model-description/configReports/parValues-Sites.yml
INFO::ray_tracing(l188)::simulate::Simulating RayTracing for offAxis=0.0, mirror=0
INFO::telescope_model(l396)::exportConfigFile::Writing config file - /workdir/external/output/simtools-files/compare_cumulative_psf/model/CTA-prod4-North-LST-1_compare_cumulative_psf.cfg
INFO::simtel_runner(l169)::run::Running (20x) with command:/workdir/sim_telarray/sim_telarray/bin/sim_telarray -c /workdir/external/output/simtools-files/compare_cumulative_psf/model/CTA-prod4-North-LST-1_compare_cumulative_psf.cfg -I../cfg/CTA -C IMAGING_LIST=/workdir/external/output/simtools-files/compare_cumulative_psf/ray-tracing/photons-North-LST-1-d10.0-za20.0-off0.000_compare_cumulative_psf.lis -C stars=/workdir/external/output/simtools-files/compare_cumulative_psf/ray-tracing/stars-North-LST-1-d10.0-za20.0-off0.000_compare_cumulative_psf.lis -C altitude=2147. -C telescope_theta=20.0 -C star_photons=10000 -C telescope_phi=0 -C camera_transmission=1.0 -C nightsky_background=all:0. -C trigger_current_limit=1e10 -C telescope_random_angle=0 -C telescope_random_error=0 -C convergent_depth=0 -C maximum_telescopes=1 -C show=all -C camera_filter=none /workdir/sim_telarray/run9991.corsika.gz 2>&1 > /workdir/external/output/simtools-files/compare_cumulative_psf/ray-tracing/log-North-LST-1-d10.0-za20.0-off0.000_compare_cumulative_psf.log 2>&1
INFO::psf_analysis(l93)::readSimtelFile::Reading SimtelFile /workdir/external/output/simtools-files/compare_cumulative_psf/ray-tracing/photons-North-LST-1-d10.0-za20.0-off0.000_compare_cumulative_psf.lis
INFO::ray_tracing(l341)::exportResults::Exporting results to /workdir/external/output/simtools-files/compare_cumulative_psf/ray-tracing/results/ray-tracing-North-LST-1-d10.0-za20.0_compare_cumulative_psf.cvs
d80 in cm = 3.4536989970304424
Traceback (most recent call last):
  File "applications/compare_cumulative_psf.py", line 205, in <module>
    dataToPlot['measured'] = loadData(dataFile)
  File "applications/compare_cumulative_psf.py", line 98, in loadData
    data = np.loadtxt(datafile, dtype=dType, usecols=(0, 2))
  File "/root/miniconda3/envs/gammasim-tools-dev/lib/python3.8/site-packages/numpy/lib/npyio.py", line 1139, in loadtxt
    for x in read_data(_loadtxt_chunksize):
  File "/root/miniconda3/envs/gammasim-tools-dev/lib/python3.8/site-packages/numpy/lib/npyio.py", line 1067, in read_data
    items = [conv(val) for (conv, val) in zip(converters, vals)]
  File "/root/miniconda3/envs/gammasim-tools-dev/lib/python3.8/site-packages/numpy/lib/npyio.py", line 1067, in <listcomp>
    items = [conv(val) for (conv, val) in zip(converters, vals)]
  File "/root/miniconda3/envs/gammasim-tools-dev/lib/python3.8/site-packages/numpy/lib/npyio.py", line 763, in floatconv
    return float(x)
ValueError: could not convert string to float: 'PSF'
(gammasim-tools-dev) [root@009f5c1daf7b gammasim-tools]#
(gammasim-tools-dev) [root@009f5c1daf7b gammasim-tools]# cat lst_pars.yml
mirror_reflection_random_angle: '0.0075,0.15,0.035'
mirror_align_random_horizontal: '0.0040,28.,0.0,0.0'
mirror_align_random_vertical: '0.0040,28.,0.0,0.0'

log-North-LST-1-d10.0-za20.0-off0.000_compare_cumulative_psf.log.gz

Do I have an older version of PSFcurve_data_v2.txt ?
There is no commenting at the top, it looks like:
cat ./verification-process/dataFromInstrumentTeams/LST/optics/psf_LST_1/PSFcurve_data_v2.txt
PSF taken at Zd = ~20[deg], with Vega
radial distance [mm] differential_value	integral_value
0.0		0.0		0.0
1.1601550664045037<PHONE_NUMBER>3333333<PHONE_NUMBER>3333333
2.3203101328090074<PHONE_NUMBER>333333<PHONE_NUMBER>666666
3.480465199213511<PHONE_NUMBER>999997	1288093.7666666664
4.640620265618015<PHONE_NUMBER>999988	2254551.016666665
5.800775332022519	1035933.4166666674	3290484.4333333327

I get the same error and my PSFcurve_data_v2.txt file also has no comments on the top. I guess @RaulRPrado you commented those lines in your local directory and didn't update GitLab?

I get the same error and my PSFcurve_data_v2.txt file also has no comments on the top. I guess @RaulRPrado you commented those lines in your local directory and didn't update GitLab?

Yes, that is right.
I just updated the gitlab version and the problem is gone.


When the delay parameter of k_delayed_work_submit is K_FOREVER, the system will crash

Describe the bug
If the k_delayed_work_submit function is called with the K_FOREVER parameter, the system will error
To Reproduce

Use any board
Call the k_delayed_work_submit function and set the parameter to K_FOREVER, then call k_delayed_work_submit through another k_delayed_work

Expected behavior
Calling k_delayed_work_submit with K_FOREVER can directly return an error, or directly return..
Environment (please complete the following information):

OS: Any
Toolchain Zephyr SDK
Commit SHA 40ee38a9450598f8ddc384cf891fc55a44ded38f

@andyross we are likely missing a test for this particular usage of delayed work?

This actually never worked correctly (well, "correctly" -- it's a pathological case after all), and AFAICT the behavior didn't change with the timeout patch.  What actually happens in practice is that the delayed work timeout gets set for -1 (== K_FOREVER) ticks into the future, which means it fires immediately at the next timer interrupt.
I can't reproduce a crash, btw; it just fires normally but unexpectedly.  Maybe your app is hitting an unexpected situation and crashing?
As far as a fix: we can either document this as a "don't do that", or (what I think I'll submit) add a check to z_add_timeout() which noops if a K_FOREVER reaches down that far.


Inconsistent Font Formatting for Script Functions.

Script function commands are in bold, which is inconsistent. Please use smaller fonts, similar to the "Installation" and "Usage" sections.

Resolved this issue in PR #7

@PayalLakra Thank you for fixing this, I am closing this issue now.


Add fuzzing tests for SPIFFE certificate validator.

In order to promote this extension to robust_untrusted.., we need to have fuzzing tests. ref: https://github.com/envoyproxy/envoy/pull/17708

cc @azdagron, I won't be able to work on this soon-ish, but will ping you to update security_posture when I get to it!


feat: add frameworks titles

Fixes https://github.com/netlify/framework-info/issues/133

I take it we're already too late to change name for id without breaking a whole lot of things and requiring a whole lot of updates correct? 😅

I actually think we could do that as a breaking change.


[13.0] ISPN-13500 CLI install

https://issues.redhat.com/browse/ISPN-13500

@pruivo the install command is a feature. It should be disabled downstream

@pruivo the install command is a feature. It should be disabled downstream

why are we backporting to 13.0.x?

@pruivo because waiting to 14 to have it is too long

well... I guess it is too late to request changes :)


Fix GATT derive visibility

I noticed that when deriving gatt_client, the methods would always be private. I implemented the same logic as for gatt_server, meaning the methods will be public if the field itself is public.
Edit: forgot to mention that I also noticed a single method on gatt_service not having the expected visibility, so I changed that as well.

Makes sense, thank you! :)


⚠ Bofrost.at has degraded performance

In 2257b5b, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1379 ms

Resolved: Bofrost.at performance has improved in caca3db after 7 minutes.


Notifications Improvements

Summary of Changes
This PR contains a set of improvements for the notifications work.

Instead of passing the resource_url using window.location.href to get the url of the resource to send along in the email, the option has been renamed resource_path. The code that sends the notification is responsible for generating the resource_path -- e.g. /table_detail/.../.../.... The logic in python that generates the email content will create the final url by concatenating the app.config['FRONTEND_BASE'] with the resource_path to generate the full url.

This change was made because we wanted to have explicit control over the generation of the url.
In order to achieve this we now pass the full tableMetadata to RequestMetadataForm and generateOwnerUpdateRequest so those entities can generate the resource_path.

The piece of application state -- notification that controls the state of the RequestMetadataForm has a new parameter checkedInputs. The actions to toggle the form open can now take a list of names of which inputs will checked by default. This allows us to auto-select "Table Description" or "Column Description" when opening the form.

If the user opened RequestMetadataForm from the column description, the comment becomes required and we prompt the user to enter which columns they would like better descriptions for. This is a stop gap requested from Product to push users to enter which columns they need, until we can re-evaluate what kind of UI might be better.

Changing the recipients input from type="email" to type="text" honors the spacing between emails that we want. It seems on type="email" the space is stripped, and type="text" currently works for our use case as we aren't leveraging any of the extra features available with type="email".

Updated RequestMetadataForm styles to make the text in the textarea darker.

Tests
TODO: Some tests to update after implementation for some of the bigger changes are agreed upon
Documentation
N/A. No new functionality or features have been added that need to be documented.
CheckList
Make sure you have checked all steps below to ensure a timely review.

[x] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[x] PR includes a summary of changes, including screenshots of any UI changes.
[ ] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[x] In case of new functionality, my PR adds documentation that describes how to use it.

All the public python functions and the classes in the PR contain docstrings that explain what it does

[] PR passes all tests documented in the developer guide

Codecov Report

Merging #301 into amundsen-notifications will decrease coverage by 2.8%.
The diff coverage is 68.42%.

@@                    Coverage Diff                     @@
##           amundsen-notifications     #301      +/-   ##
==========================================================
- Coverage                   71.36%   68.56%   -2.81%
==========================================================
  Files                         143      117      -26
  Lines                        3653     2497    -1156
  Branches                      451      348     -103
==========================================================
- Hits                         2607     1712     -895
+ Misses                       1013      780     -233
+ Partials                       33        5      -28

Impacted Files
Coverage Δ

..._application/static/js/ducks/notification/types.ts
100% <ø> (ø)
:arrow_up:

...dsen_application/static/js/fixtures/globalState.ts
100% <ø> (ø)
:arrow_up:

..._application/static/js/interfaces/Notifications.ts
100% <ø> (ø)
:arrow_up:

...ication/static/js/components/TableDetail/index.tsx
0% <ø> (ø)
:arrow_up:

...ts/TableDetail/DetailList/DetailListItem/index.tsx
0% <0%> (ø)
:arrow_up:

...pplication/static/js/ducks/notification/reducer.ts
100% <100%> (ø)
:arrow_up:

...tion/static/js/ducks/tableMetadata/owners/sagas.ts
100% <100%> (ø)
:arrow_up:

...nents/TableDetail/RequestDescriptionText/index.tsx
100% <100%> (ø)
:arrow_up:

...ation/static/js/ducks/tableMetadata/api/helpers.ts
90.9% <100%> (-9.1%)
:arrow_down:

...nents/TableDetail/RequestMetadataForm/constants.ts
100% <100%> (ø)
:arrow_up:

... and 28 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cd07c42...27c6d04. Read the comment docs.

Will cleanup this branch then change PR to merge into master

Not sure how this got closed.

@danwom already merged amundsen-notifications down to master and tried to clean this one up. Commit history is all over the place but the current diff is at least accurate.
Still need changes reviewed before finishing the unit tests.


99% Of alarm feature implemented, may have minor bugs

The updated PR : Couple of things coming in a future PR :)
This adds an alarm feature to WOPR (sorry had todo some refactoring in the main file)
If there is a V2 board

Let it have RTC
Let it have some form of perm storage (ram or sdcard)
Let it have some "extra" side buttons to toggle up/down
Let the reset be easier to get to
Let it have a rear 3d print case that could hold a battery

Hey mate, so sorry I never saw this!
So after a review, as cool as this all is, it's changed too much in the base display code for me to merge this in... especially considering I also have newer code I'm working on that has none of your refactoring.
Maybe a better idea is for you to just release your fork of this? I'll link to it on the store pages so folks know about it and can use it instead of mine if they like?
Cheers,
Seon

Oh, also, re your rev 2 list... you can already store stuff on the ESP32. It uses a system called SPIFFS - there are examples in Arduino in the ESP32 section. Let's you save and load files like a file system.

All good , you have been busy
I am planning on changing it for the "new" version with back buttons - Hopefully that will result in less core changes
(Some of the core changes mine help re-used the core features )
Ill close this PR

Yeah, you did some nice changes and I'v already done some similar ones, though not to the same extent. Putting menu names  into an array etc. I'm also using enums now for state as well, and have SPIFFS implemented for user data, so extending that for your alarm stuff should be super simple.


[FEATURE]: Fix column type and expose data such as length

Describe what you want
I have a table like follow:
const resources = pgTable('resources', {
  id: varchar('id', { length: 30 }).primaryKey(),
  title: varchar('title', { length: 50 }).notNull(),
});

I want to access the title length to better format my inputs, for example:
db.insert(resources).values({
  id: '123',
  title: title.slice(0, resources.title.length),
});

This code works as the length indeed available under the title column, but I get a type error:
Property 'length' does not exist on type 'PgColumn<{ name: "title"; tableName: "resources"; dataType: "string"; columnType: "PgVarchar"; data: string; driverParam: string; notNull: true; hasDefault: false; enumValues: [string, ...string[]]; baseColumn: never; }, {}, {}>'.

We need to make resources.title.length available within the type system.

this is a regression due to type API changes, will be fixed

Any update?

Using this method for now:
/**
 * This is a fix until drizzle-orm supports length for varchar.
 * See here: https://github.com/drizzle-team/drizzle-orm/issues/1441
 */
export const getColumnLength = (column: PgColumn) => {
  const { length } = column as { length?: unknown };

  if (typeof length !== "number") {
    throw new TypeError(`Column ${column.name} does not have a length`);
  }

  return length;
};


little wrong

the line 95 in wcm_zeroshot.py need to be modify, the test_pred has the wrong shape

Thanks for pointing it out. Solved in PR #24 .


refactor: COLOR_MODE to OnceCell

Refactor COLOR_MODE to be easier for testing and for using features in the standard library come v1.71.0.
Addresses: #368

Codecov Report
Attention: Patch coverage is 64.28571% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 63.72%. Comparing base (f688e38) to head (e6f0127).

Files
Patch %
Lines

src/encode/writer/console.rs
64.28%
5 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #365      +/-   ##
==========================================
+ Coverage   63.42%   63.72%   +0.30%
==========================================
  Files          25       25
  Lines        1572     1577       +5
==========================================
+ Hits          997     1005       +8
+ Misses        575      572       -3

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Looks like CI is failing on this one still...

I see what you're saying but I don't think we get any test benefit with that change (which may be okay because of the integration tests). That said, I don't think I agree that using Lazy is any more lightweight. Both will be initialized once and return a reference on access if I'm reading the docs correctly.

@bconn98 by lightweight i mean not having the following copy pasted at every callsite. There are other options if you prefer, you could wrap the Lazy in a struct and in cfg(test) mode internally it could simply be a Mutex that you can mutate as necessary.
            let color_mode_init = {
                let no_color = env::var("NO_COLOR");
                let clicolor_force = env::var("CLICOLOR_FORCE");
                let clicolor = env::var("CLICOLOR");
                get_color_mode(no_color, clicolor_force, clicolor)
            };

I tried a couple different versions of using mutex.

Locking the whole tests
Like you suggested a struct with a mutex locked getter (using deref and force)

But no avail, the Lazy will not execute a second time thereby causing tests to fail. I wouldn't want to put the actual ENV values inside of mutex's as that would invalidate the majority of the test.

I found a solution that I think is satisfactory. By Option wrapping all the args, we can dependency inject during testing or otherwise just pass Nones and pull from the env. I would like this MR to merge in the initial migration, than a follow on MR will come to bump our MSRV and use the solution that is now part of the std lib.


Webclient redesign

WebClient redesign
Resolves:

#6669
#7223
#7078
#7242
#6774
#7138 for both HTTP/1 and HTTP/2

Follow up issue: #7269
Notable changes:

single API that can run any HTTP version
upgrade done through HTTP/1.1 client always (implemented both for WebSocket and HTTP/2)
HTTP client SPI (HTTP/1.1 and HTTP/2 implementations)
Client SPI (websocket implementation)
Cookie manager support for HTTP/1.1 and HTTP/2
WebClientService support for HTTP/1.1 and HTTP/2
HttpClientResponse.request(Class) now returns a typed entity response, HttpClientResponse.requestEntity(Class) returns only the response entity (and throws if non-success status family)

Updated #7269  with unresolved comments from this PR (except for the inconsistent CR/LF as I cannot find what is wrong)


Disable Markdown rendering in Vue-in-Markdown elements

Bug report
Markdown in Vue-in-Markdown components gets rendered.
Steps to reproduce
# Hello!

<my-component>Works!</my-component>
<my-component>
  Doesn't work!
</my-component>
<my-component>
  # Hello
</my-component>

<button># A regular HTML button works just fine</button>

What is expected?

Hello!

<my-component>Works!</my-component>

<my-component>Doesn't work!</my-component>

<my-component># Hello</my-component>

<button># A regular HTML button works just fine</button>

What is actually happening?
Gets rendered to:

Hello!

<my-component>Works!</my-component>

<my-component>
Doesn't work!
</my-component>

<my-component>
Hello
</my-component>

<button># A regular HTML button works just fine</button>

Other relevant information

Result of running saber -v: latest
Browser version (optional):
Is Saber a global or local install? local
Which package manager did you use for the install? yarn
Does this issue occur when all plugins are disabled? ye

That should be possible by modifying the html renderer in markdown-it, I haven't tested it tho.


fix TypeError in case LinkBranch has no length information

Hi,
I think it should be a ismissing() instead of isnan() here and here,  otherwise there will be a "non-boolean (Missing) used in boolean context" TypeError.
Example:
> mytree = parsenewick("((A,B),C);")
> mytree.branches[1]

Thank you for the great package!

Thanks @dies-und-lenes - sorry for the delay in picking up on this, I've been really busy!


Go build error.

Dear:
The project can run normally, but when me go build reports an error, like this：
D:\GOPATH\src\yisa\trasffer_same_household _registration>go build -o trasffer_same_household_registration
github.com/mattn/go-oci8
......\pkg\mod\github.com\mattn\go-oci8@v0.0.8\connector.go:14:10: undefined: Connector
......\pkg\mod\github.com\mattn\go-oci8@v0.0.8\connector.go:20:18: undefined: Connector
......\pkg\mod\github.com\mattn\go-oci8@v0.0.8\connector.go:21:9: undefined: Driver
......\pkg\mod\github.com\mattn\go-oci8@v0.0.8\connector.go:25:18: undefined: Connector
......\pkg\mod\github.com\mattn\go-oci8@v0.0.8\connector.go:30:11: undefined: Conn
D:\GOPATH\src\yisa\trasffer_same_household _registration>

Would check the readme and other issues. Probably something to do with the setup needs to be changed.

All good? Could you close this?


Check email link text color

It's light grey for some reason! 🤷♂
It should be white.

I will be working in this issue

Add
.usa-section-dark a {
  color: #fff !important;
}

to custom.css and you are good to go.

Hmm... we could... but using !important is not a good approach generally in CSS. It means if we ever need to overwrite that style we'd have to use yet another !important later in the css. I'd rather find a solution to why that button is gray and the other is white.

That button is gray because it is defined like that in the uswds main.css

But why is the other button not gray? In any case, we can override the styles without using !important, it's really a nuclear option that should be avoided.

Dug into this for a minute and, despite Chrome Inspector's ordering of applied CSS rules, it appears that this rule is the dominant:
.usa-section-dark a {
  color: #d6d7d9;
}

That rule appears toward the bottom of main.css and has the same or slightly higher specificity as other rules applying color to this element. 🙃

Fixed in a39b566 by adding:
class: page-home

...to the index.md file and the following to custom.css:
.page-home .usa-section-dark .usa-button {
  color: #fff;
}

🇺🇸


"Invalid alias in frontmatter"

🐛 Describe the bug
Whenever I save a file or when I try to wikilink another file, I get :
Invalid alias value found in frontmatter for /home/simon/4_Notes/uni/Mathe/Mitschrift 19.02.2024.md. Expected string, found table.

Even after quite  a bit of digging, changing my Linter settings etc, I still get the error and I am absolutely clueless. This may very well be an issue on my side, however since I have no issues at all in Obsidian itself, I feel like this might also be a bug.
Any help would be appreciated either way.
Config
 154   │   {
 155   │     "epwalsh/obsidian.nvim",
 156   │     version = "*", -- recommended, use latest release instead of latest commit
 157   │     lazy = true,
 158   │     ft = "markdown",
 159   │     dependencies = {
 160   │       "nvim-lua/plenary.nvim",
 161   │       "hrsh7th/nvim-cmp",
 162   │       "epwalsh/pomo.nvim"
 163   │     },
 164   │     opts = {
 165   │       workspaces = {
 166   │         {
 167   │           name = "Notes",
 168   │           path = "~/4_Notes",
 169   │         },
 170   │       },
 171   │     },

Environment
NVIM v0.9.5
Build type: Release
LuaJIT 2.1.1702233742

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/share/nvim"

Run :checkhealth for more info
[obsidian.nvim (v2.10.0)] Commit SHA: 32de09d0c737d314ff629068902c4657c244d55b
[plenary.nvim] Commit SHA: 4f71c0c4a196ceb656c824a70792f3df3ce6bb6d
[nvim-cmp] Commit SHA: 04e0ca376d6abdbfc8b52180f8ea236cbfddf782
ripgrep 14.1.0%

Hey @DefinitelyNotSimon13, I think the error is pretty clear, no? There's an issue parsing the frontmatter for the note at /home/simon/4_Notes/uni/Mathe/Mitschrift 19.02.2024.md.

If this is a parser issue let me know with a reproducible example and I'll reopen. Otherwise it seems like you have invalid YAML in the frontmatter for one of your files.


tuple index out of range

hi
I have some problems using it, the error message is as follow

Could you help me thanks

The library currently does not support conv statistics other than conv 2d. Please make sure that there are no other convolution layers in your network.


Overlay network problem when swarm nodes have non-unique hostnames

Summary:  I have a Swarm cluster with a master and two nodes.  All three hosts have the same hostname "vagrant-ubuntu-trusty-64".  I cannot ping one container's overlay IP from a container running on the other node.  However, if I manually give each node and the master unique hostnames, then the overlay networking will work normally.
Is this the intended behavior?

Here's a concrete example.  Master is at <IP_ADDRESS>:3375.  I start and nginx container and a /bin/sh container and shown below:
$ docker -H=tcp://<IP_ADDRESS>:3375 run -d --name web --net myStack1 nginx
7bef07de1ba1fb403b1f469face7867f57e07b300d81c855fc8abe099c25555e
$ docker -H=tcp://<IP_ADDRESS>:3375 run -itd --name shell1 --net myStack1 alpine /bin/sh
536cbea76ad18a56a8d1f7b25437c26f915792d7e7025102a684d649111d6014
$ docker -H=tcp://<IP_ADDRESS>:3375 ps
CONTAINER ID        IMAGE               COMMAND                  CREATED             STATUS              PORTS               NAMES
276979ed81a3        alpine              "/bin/sh"                11 seconds ago      Up 10 seconds                           vagrant-ubuntu-trusty-64/shell1
4b32eba5d660        nginx               "nginx -g 'daemon off"   17 seconds ago      Up 16 seconds       80/tcp, 443/tcp     vagrant-ubuntu-trusty-64/web

(By sshing into each node and doing a local docker ps, I can tell that nginx is running on one node and /bin/sh container on the other node.)
Now I attach to the shell container (<IP_ADDRESS>) and try to ping the web container's IP (<IP_ADDRESS>):
$ docker -H=tcp://<IP_ADDRESS>:3375 attach shell1

/ # ping web
PING web (<IP_ADDRESS>): 56 data bytes
[...wait a minute seconds here...]
^C
--- web ping statistics ---
12 packets transmitted, 0 packets received, 100% packet loss

Other network communication like curl, nc and traceroute also fail.  However, networking is fine if the same procedure as above is followed but with master and the two nodes all having unique hostnames.

This looks like a routing problem. /cc @sanimej .

Confirmed with Santhosh that server name needs to be unique in libnetwork. libnetwork uses Serf in underlayer where hostname is used to identify a node.

@dongluochen Thanks for investigating.  I'm going to close this issue and open one on libnetwork instead.

Just


GEODE-6198: Add geode-all-bom for dependency versioning

All modules refer to the BOM to determine dependency versions.
Consequently, all JavaCompile tasks depend on the BOM.
Remove explicit versioning from dependency declarations
Unify plugin / classpath styles.
Use RELEASE of Sprint Dependency Management, not RC.
Also move eclipse configurations outside of dependency blocks, as they
are unrelated.

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

There are multiple "cleanups" that could be part of this or a near-future PR.  Part of me wanted to kick off testing sooner rather than later, though.
(Probably yet part of this PR:)

Multiple same-version dependency lines should be collapsed to a dependencySet.
The application of the dependencyManagement { imports {... could be part of a "plugin" and applied via apply from: ....  This should also subsume the remaining dependency-versions.properties to keep them all in one place.  The JavaCompile task dependency should live there as well.

(Probably future PRs:)

The version of project components should not be explicit in the POM and also specified in the BOM.  Stripping versioning from project dependencies was found in itself to be nontrivial.
How we're handling Jetty and Tomcat should be examined before it is clear if or how they should be part of the BOM.

@dickcav Upon further inspection, the Wiki says that the build runtime is in fact meant to be in the bundle.  So no foul on that front!


Add script for manual tests

Summary
Created a script to automate the manual smoke test we were doing before merging PRs. Next step is to call this using a github workflow so that we can automate testing as soon as a PR is created.
Description
Created a script which takes profiles.yml options as parameters. Another option is specified to decide if we're testing against software or cloud. These parameters will be fixed in our workflow, so no need to memorize the order of them to run the script manually.
Related Issue
https://github.com/dremio/dbt-dremio/issues/35
Additional Reviewers
@ArgusLi
@jlarue26

General comment: can we utilize more indenting for readability?
Here is a good guide, although I don't expect conformance: https://lug.fh-swf.de/vim/vim-bash/StyleGuideShell.en.pdf

General comment: can we utilize more indenting for readability?
Here is a good guide, although I don't expect conformance: https://lug.fh-swf.de/vim/vim-bash/StyleGuideShell.en.pdf

I tried adding indenting, but it breaks the script for some reason.