@lit-labs/virtualizer package migration from PolymerLabs/uni-virtualizer

This PR brings in the @lit-labs/virtualizer package from the https://github.com/PolymerLabs/uni-virtualizer repo.  All other files from uni-virtualizer repo that were imported were not kept.
I prepared a migration script so I could refine this until it worked exactly the way I expected with commit history and everything in its place in the lit monorepo.  The migrate.sh script and its output can be found in this gist if interested:
https://gist.github.com/usergenic/226e876af976905ba557f12c8f4c886f
Verifying commit history is present, here is snippet from output of git blame on the package's README:
$ git blame packages/labs/virtualizer/README.md
e03b71c1e packages/lit-virtualizer/README.md  (Gray Norton     2021-05-07 15:00:26 -0700   1) # @lit-labs/virtualizer
9db07ef51 packages/lit-virtualizer/README.md  (straversi       2019-06-05 14:22:52 -0700   2)
6261458f8 packages/lit-virtualizer/README.md  (Gray Norton     2022-03-22 13:31:49 -0700   3) `@lit-labs/virtualizer` provides viewport-based virtualization (including virtual scrolling) for [Lit](https://lit.dev).
9db07ef51 packages/lit-virtualizer/README.md  (straversi       2019-06-05 14:22:52 -0700   4)
2754a144f packages/lit-virtualizer/README.md  (Gray Norton     2022-03-22 13:08:55 -0700   5) ⚠ `@lit-labs/virtualizer` is in late prerelease. Its API is intended to remain quite stable going forward, but you should expect (increasingly minor) changes before 1.0. Some of these changes may be technically breaking, but we anticipate that they will be mechanical and straightforward to make.
ff7f2ca07 packages/lit-virtualizer/README.md  (straversi       2019-06-25 15:42:39 -0700   6)
0af8bb7e8 packages/lit-virtualizer/README.md  (straversi       2019-06-05 15:07:31 -0700   7) ## Getting Started
9db07ef51 packages/lit-virtualizer/README.md  (straversi       2019-06-05 14:22:52 -0700   8)
0af8bb7e8 packages/lit-virtualizer/README.md  (straversi       2019-06-05 15:07:31 -0700   9) Get this package:
9db07ef51 packages/lit-virtualizer/README.md  (straversi       2019-06-05 14:22:52 -0700  10)
0af8bb7e8 packages/lit-virtualizer/README.md  (straversi       2019-06-05 15:07:31 -0700  11) ```
e03b71c1e packages/lit-virtualizer/README.md  (Gray Norton     2021-05-07 15:00:26 -0700  12) npm i @lit-labs/virtualizer
0af8bb7e8 packages/lit-virtualizer/README.md  (straversi       2019-06-05 15:07:31 -0700  13) ```

With @aomarks help I managed to set up the build related scripts to use "wireit", though at this stage I have not yet added wireit files/output details for optimized wireit-ing.

@augustjk I'm totally confused by this delete. Thank you for calling it out... The entire set of commits in this PR is automated and the result of a script I wrote, none of which is intended to specifically remove anything...
https://gist.github.com/usergenic/226e876af976905ba557f12c8f4c886f
I will try to rerun script to investigate this more thoroughly soon.

@augustjk I'm totally confused by this delete. Thank you for calling it out... The entire set of commits in this PR is automated and the result of a script I wrote, none of which is intended to specifically remove anything... https://gist.github.com/usergenic/226e876af976905ba557f12c8f4c886f
I will try to rerun script to investigate this more thoroughly soon.

Ah, yes. I think the culprit is the --hoist option to npx lerna bootstrap in line 90 here https://gist.github.com/usergenic/226e876af976905ba557f12c8f4c886f#file-migrate-sh-L90
It's listed as a dependency by relative path here https://github.com/lit/lit/blob/ee864b6dd6616d6cb72924089581af220196c894/packages/labs/analyzer/package.json#L59 which I think the --hoist then tries to move it to it to the top node_modules directory. I can reproduce by running npx lerna bootstrap --hoist locally on main branch as well.
I think npx lerna bootstrap doesn't really need any flags anymore as ci and sort are default modes per the doc.

@augustjk oh wow, that's a great catch.  I followed that lead and removed hoist and after some reworking of the migrate script to convert scripts to wireit before bootstrap runs, I skip past the problem hoist had previously solved for me.
Running a few more tests and then I'll have this branch force pushed with the fix.  I'll update the gist with the new migrate code after I verify everything is working.

@augustjk PTAL

Great catch @augustjk! We need to fix the wireit config for analyzer.
@usergenic LGTM! Just some comments about moving things to our monorepo standards over time. Todo issues would be great.

Great catch @augustjk! We need to fix the wireit config for analyzer.

@justinfagnani it was fixed as part of https://github.com/lit/lit/pull/2982

Having resolved the comments, filed issues, and otherwise fixed problems with wireit and testing, I'm going to force push the latest migration script results to this branch.  Also updated the gist with the migration script and the output.


Prefix search doesn''t work in some cases.

There is a document in my index with the title 'Reconnaissance' and when I try to find it, using prefix. It is found from 'r' to 'reconnaiss' but if I try to find 'reconnaissa' nothing is found. Did I missed something?
Same thing with vandalism => from  vandal on there is no result anymore

Same issue here, see ticket. It happens both with and without wildcards.

This sounds likely to be caused by stemming.
> lunr.stemmer(new lunr.Token("reconnaissance")).toString()
'reconnaiss'

That is, only the text 'reconnaiss' is actually indexed by Lunr. This works when performing non prefix search because the search term will also be stemmed before doing the lookup. Stemming is not possible when doing a prefix search, and so once you have a prefix longer than the stemmed word stored in the index there will be no matches.
You can disable stemming for the index, or alternatively perform both a prefix search and a non-prefix search together.
idx.query(function (query) {
  // prefix search, no boost
  query.term("reconnaiss", { wildcard: lunr.Query.wildcard.TRAILING, boost: 1 })

  // 'exact' match, boosted
  query.term("reconnaiss", { boost: 10 })
})

The above will combine both a prefix search with an exact search, exact matches will typically rank higher given the boosts.


Add Line Split capabilities

It may look weird but we have a case where a line can hold 2 different records, the position of the second record length may be provided in the first record itself.
Example:
HEADERMy Title  26        EmplJoe1      Smith     Developer<PHONE_NUMBER>2009
CatSnowball  20200103
EmplJoe3      Smith     Developer

The first line contains 2 records:

"HEADERMy Title  26        " where 26 define the index of the next record
EmplJoe1      Smith     Developer<PHONE_NUMBER>2009

PR is coming.

Merged, thank you!


Getting a 404 with using a connection string to azure container

I think this is potentially related to #39 and #21
Delta scan on a delta table locally with node js duckdb:
I downloaded my small test delta table from my azure blob container locally
const { Database } = require("duckdb-async");

async function main() {
	const db = await Database.create(":memory:");
	await db.run(`force install delta from core_nightly;LOAD delta;
				INSTALL azure;LOAD azure;`);
	const rows = await db.all(`
		CREATE TABLE testdata AS
		SELECT *
		FROM delta_scan('./hundredThousandPoints')`);
	console.log(rows);
	console.log("🦆 DuckDB initialized 🦆");
}
main();

Code runs and outputs as expected:
[ { Count: 100000n } ]
🦆 DuckDB initialized 🦆

Delta scan on delta table in azure:
My delta table is stored in a Blob Container (ADLS Gen2). I am using a connection string for my storage container. Something that looks like this in a .env file:
DefaultEndpointsProtocol=https;AccountName=myaccount;AccountKey=122342314fsdfsdf;EndpointSuffix=core.windows.net

const { Database } = require("duckdb-async");
require('dotenv').config();

async function main() {
	const db = await Database.create(":memory:");
	await db.run(`force install delta from core_nightly;LOAD delta;
				INSTALL azure;LOAD azure;
				CREATE SECRET secret1 (TYPE AZURE, CONNECTION_STRING 'abfss://${process.env.AZURE_CONN_STR}');`);
	const rows = await db.all(`
		CREATE TABLE testdata AS
		SELECT *
		FROM delta_scan('abfss://myaccount/testdata/us_points/hundredThousandPoints')`);
	console.log(rows);
	console.log("🦆 DuckDB initialized 🦆");
}
main();

I then get an error:
[Error: IO Error: Hit DeltaKernel FFI error (from: While trying to read from delta table:
'abfss://myaccount/testdata/us_points/hundredThousandPoints/'):
Hit error: 8 (ObjectStoreError) with message (Error interacting with object store:
Generic MicrosoftAzure error: Error performing list request:
Client error with status 404 Not Found: <?xml version="1.0" encoding="utf-8"?><Error>ContainerNotFound<Message>The specified container does not exist.

I downloaded my data locally from azure so I know it exists. Are connection strings supported or should I use a service principal credential? Or is there something silly I am missing?

As a sanity test I added the delta table to a different blob storage container that I have more control over that I created (not one that my team uses). After doing so I was able to use a connection string and read the delta table with no issues
I could be missing something but it may be worth adding a connection string test case inside of test/sql/cloud/azure tests but thank you for your hard work on this extension!!

FYI I figured it out the issue was:
This abfss://myaccount/testdata/us_points/hundredThousandPoints needed to be changed to
abfss://testdata/us_points/hundredThousandPoints with the account named removed since the connection string includes the account name


Restic on mac: don't use an smb mount

I set up Restic on my Mac and noticed that it's very slow over an SMB connection. I have 400+GB of data to backup and the daily backup took well over 2 hours to run even when no files were changed. And it made my wifi slow.
For some reason, using --force didn't help (but in addition rendered my Mac slow). referring: https://github.com/restic/restic/issues/386#issuecomment-247751400
After investigating, I'd say this is probably an issue with Apple's SMB implementation and/or my network drive. I switched the repository access from an smb-mount to sftp, and things sped up significantly. Now the daily backup takes ~5 mins when the files are (mostly) unchanged:
...
[4:20] 76.55%  1.235 GiB/s  321.004 GiB / 419.338 GiB  130504 / 162423 items  0 errors  ETA 1:19
[4:30] 76.65%  1.191 GiB/s  321.441 GiB / 419.338 GiB  133528 / 162423 items  0 errors  ETA 1:22
[4:40] 88.13%  1.320 GiB/s  369.574 GiB / 419.338 GiB  152324 / 162423 items  0 errors  ETA 0:37
[4:47] 100.00%  1.320 GiB/s  419.338 GiB / 419.338 GiB  162423 / 162423 items  0 errors  ETA 0:37

duration: 4:47, 1491.71MiB/s

So, it seems to me that the way Restic communicates with the repository does not work well with Apple's SMB. For me, a valid workaround was to use sftp. For someone else, a metadata cache might be helpful https://github.com/restic/restic/issues/29.
(not sure if adding an issue is correct for this kind of nice-to-know-info, but I was so astonished to find the speed difference that had never occurred to me in other use. please feel free to close this issue)
For the sake of documentation, I wrote down notes about my Restic-setup: http://www.jussihuotari.com/2017/01/14/computer-backup-using-restic-on-mac/

Hey, thanks for reporting this odd behavior! Let's make sure I understood your setup correctly: You're creating a backup of local data stored on your machine to a repository located on a network drive mounted via SMB. That is slow. Accessing the same data via SFTP is much faster. Correct?
In this case I suspect it's indeed Apple's SMB implementation. During a non-initial backup restic accesses a lot of small parts of the pack files stored in the repo at random. So that's something which stresses the SMB implementation a bit.
I suggest closing this issue so that people can find it, apart from adding the metadata cache (which will surely speed this up a bit) there's nothing we can do. What do you think?

Yes, that's a correct description of the setup. FWIW my network drive does SMB2 (not smb3).
And yes, agreed, this issue can be closed. Thanks!


Github Enterprise Support

This solves https://github.com/rtfd/readthedocs.org/issues/2194.

@humitos yes you need to change that, however in some places the github.com was hard coded into the code, I've replaced in these cases to instead read it from from allauth.socialaccount.providers.github.provider import GitHubProvider  (also I've already mention you need to do the change in there at: https://github.com/gaborbernat/readthedocs.org/blame/38ee82afe90b2a0af455f93b7baa699db4b8cf50/docs/custom_installs/github_enterprise.rst#L32 - this is a config change)
We could expose a main level variable in config if you guys think it would make it easier.
To have the connect account feature work you need to setup the site setting correctly (e.g. point to your domain and not example.com as it's by default - which is the first change in the script).
As for having both exists do we have a strong need for that, as would probably complicate the implementation.

@humitos made the requested changes, ready to discuss supporting an enterprise instance with github.com in parallel.

Made the requested changes. By the way running isort would modify a lot of existing source files :thinking: are we sure it's configured correctly?

Made the requested changes. By the way running isort would modify a lot of existing source files thinking are we sure it's configured correctly?

Yes. It's correctly configured but it's something new and we didn't run to the entire project yet. We were running it only the files that we modify on each PR.
So, you could run it only on those files.
At a higher level, we have pre-commit configured (which will run only on your modified fields) but we are not forcing the community PR to run this command yet.

As for having both exists do we have a strong need for that, as would probably complicate the implementation.

We don't really need this for the community site since we don't support GHE in .org but I was asking you to know if this could be applicable in the readthedocs.com site. There, we will need to support:

multiples GHE accounts
GHE and regular GH connections living together
accessing to a private network of GHE instace

Anyway, I was checking if we could advance in one of those point in this PR but it seems that we won't be able to do this yet.
Currently, this is only useful for private installations of RTD source code.

@humitos tox organization has a fix-lint target to setup and run automatically the pre-commit maybe we should have it here too, but that's part of a different PR/issue
Indeed this PR is useful for private installations of RTD. We can look into supporting more part of further subsequent PRs. I don't have access to multiple instances of these, hence why was not focusing on taking a look at it. But most likely it's something that can be done.
Additionally made the required change (this way we keep previous settings - gitlab included, only upgrade the github URL):
from .dev import CommunityDevSettings
_SETTINGS = CommunityDevSettings()

SOCIALACCOUNT_PROVIDERS = _SETTINGS.SOCIALACCOUNT_PROVIDERS
SOCIALACCOUNT_PROVIDERS['github']['GITHUB_URL'] == '{{github_url}}'

You're heard, @gaborbernat. Four days (and over the weekend) isn't a lot of time for us - we'll get to this when we can.

I'm happy that someone is able to take this on. Thanks for the contribution @gaborbernat !
I'll have more feedback when I can focus on this more, but for starters: this is touching a lot of important code and has no tests. So, fixing that is a good place to continue this work.
I'm going to target a future release, as this isn't quite on our roadmap. I think this is mostly close though. Apologies for any delays responding, off-roadmap PRs have lower priority for us.

Just an update. I definitely plan to continue to work on this. Just other things have come up. The fact that the actual code has not tests at the moment means I first need to figure out how this part is intended to be tested. Which makes picking up this non-trivial.


plz thx

Hello, if I need to add a new resolution (3440 * 1440), can I directly change the default resolution (2560x1440) or do I need to change other places at the same time

Actually, you just need to set the size of the QTE detection based on the real scaling of the corresponding resolution.
To do this, you can get a screenshot containing a QTE in the corresponding resolution. Then find a proper size of a square that should be a little larger than the QTE circle. eg. 200*200 or 250*250. Finally, you can set the crop_w, crop_h of  corresponding resolution as below,
https://github.com/En73r/DBDEazyQTE/blob/02b47d68183e43c40c6e40ae40c78deedf5dc8e0/DBDEazyQTE.py#L525-L530

After I do this, the engine will still explode. I don't know why. Maybe it is the curved screen?


Issue in the server console

I am using VillagerRespawn-1.16.5-<IP_ADDRESS> on my server with forge 36.2.30. I got this in my server console: https://pastebin.com/Qek1dYtS.
Can i ignore that?

The pastebin is apparently expired.
What did you see?
Sorry for the delay responding- been working on my taxes.

I am gonna test this night and tell you the result.

Thank you!   I appreciate your feedback and engagement!


How to customize the color Palette used for plottables

ScottPlot Version: 4.1.14
Operating System: Win10
Application Type: WinForms
Questions:

Is there a way to reset the Colors index for a plot back to 0? In #1032 last comment, you can see that because I add a red Vertical Axis line at the begining, the next signal gets an Orange color (index 1) instead of Blue (index 0).
Is there a way to add custom signals color array (and maybe styles) inside ScotPlot? So ScottPlot will loop around those colors and styles.
Edit: I have seen this Cookbook example which is ok for one plot, but for multiple plots becomes difficult to keep track of the color index for each plot. This is why I believe something internal to ScottPlot is more elegant.

Hi @EmanuelFeru, thank you for this question!
Plot.Palette() is Broken
There is a Palette system which manages default colors for new plottables, but it looks to be incompletely implemented and not documented well. Plot.Palette() exists but doesn't seem to work as intended.
I think I'll convert it from a method to a property, then add some tests to ensure it works as expected and some cookbook examples demonstrating how to use it. I'm pretty sure getting this working will fully address your issue, and give everyone access to very nice looking alternative default color sets.
Related #481
Manually Manage Colors
An immediate fix for your problem is to manage the colors yourself by assigning line colors manually. Colors for the default palette (Category10) and more palettes can be accessed like this:
var pal = ScottPlot.Drawing.Palette.Category10;
for (int i = 0; i < pal.Count(); i++)
{
    System.Drawing.Color color = pal.GetColor(i);
    Console.WriteLine($"[{i}] = {color}");
}

Hi @swharden, you got my point, that is what I am looking for to be able to define a custom ColorSet.
Until that becomes available, I will take your suggestions to manually manage the colors. Thank you!

And for resetting the color index back to 0 do you have any suggestions? Or I do it by manually managing colors?

resetting the color index back to 0

This isn't really feasible with the current design. Currently the default color for all new plottables is the Nth color in the palette where N is the number of plottables that have been plotted previously.

manually managing colors

To reset at zero, managing colors manually is the way to go, but you can get help from the palette system:
var pal = ScottPlot.Drawing.Palette.Category10;

// start at zero
Color first = pal.GetColor(0);
Color second = pal.GetColor(1);
Color third = pal.GetColor(2);

// "reset" to zero
Color fourth = pal.GetColor(0);
Color fifth = pal.GetColor(1);
Color sixth = pal.GetColor(2);

I added cookbook examples demonstrating how to use colormaps... although I might adjust the architecture a bit before the next release. It was a little clunky, and I'd love to smooth it out before locking-in the API here. This is a great step in a positive direction, and I'm reminded that some of the palettes optimized for dark backgrounds are very useful!

This info is sufficient for now. So, I will close the topic. Thank you Scott.

Thank you Scott

Sure thing! Thanks for calling attention to this so we can make it easier for future users. It looks like this will be in the next release:
string[] customColors = { "#019d9f", "#7d3091", "#57e075", "#e5b5fa", "#009118" };
plt.Palette = new ScottPlot.Drawing.Palette(customColors);

for (int i = 0; i < 5; i++)
{
    double[] xs = DataGen.Consecutive(100);
    double[] ys = DataGen.Sin(100, phase: -i * .5 / 5);
    plt.AddScatterLines(xs, ys, lineWidth: 3);
}

I love it!!! Thanks!


Enable dependabot for github-actions

It will enable github-actions dependency updating automatically with sent PRs.

Hi @cketti , could you help to review this PR? Thanks.

Thanks for your pull request.
However, I don't think dependabot works with our custom mechanism to manage dependency versions. I'd rather do this manually anyway.

@cketti Thanks for reviewing. I understood it. But I also want to recommend dependabot for GitHub Aciton's version again. For example, current k-9 GitHub Workflow uses actions/checkout@v1 to check source code, but official recommended version is v2. And configuration enabled by this PR can send a PR to update it automatically. Current configuration will not enable normal Gradle dependency updating.

Oh, I see. Sorry, I misunderstood what this is for.
Is the directory: "/" line necessary? If so, shouldn't it be directory: "/.github/workflows"?

For GitHub Actions, set the directory to / to check for workflow files in .github/workflows.

Official documentation recognizes / for .github/workflows. See https://docs.github.com/en/code-security/dependabot/dependabot-version-updates/configuration-options-for-the-dependabot.yml-file.

Thanks :+1:


Add OPTIONS-method as a shortcut

Axios is missing OPTIONS-method as a shortcut. This PR will implement it

Coverage remained the same at 93.979% when pulling 2e0adc1cae76eb214800ce64cfb5d27c032c8556 on theikkila:master into d982cf99324124582721d1cd15a99cb975e93716 on mzabriskie:master.

Thank you for the PR! What's the use case for this?

This is for making the API consistent across all standardized HTTP-verbs. Options is sematically used ie. when fetching dynamic options for some endpoint.
Usually the response can tell the allowed or required fields.

I would like to use this to test that my API is returning the correct CORS headers.

I don't think we need to add a shortcut method for OPTIONS as preflight requests are normally sent by the browser. For test purposes, you can use the request method as follows:
axios.request({
  method: 'options',
 url: '/foo'
})

Thank you!

OPTIONS are not just related to preflight requests but also for actually inspecting meta-data about some REST api endpoint. OPTIONS-shortcut is not for preflight requests but as I said  ie. to gather some options about the validation.
According to RFC-2616 / 9.2 OPTIONS:
https://www.w3.org/Protocols/rfc2616/rfc2616-sec9.html

The OPTIONS method represents a request for information about the communication options available on the request/response chain identified by the Request-URI. This method allows the client to determine the options and/or requirements associated with a resource, or the capabilities of a server, without implying a resource action or initiating a resource retrieval.

Real world example is Django Rest Framework, it uses the OPTIONS-verb exacly for that purpose.
Please reopen this PR.

Coverage decreased (-0.3%) to 93.687% when pulling 2e0adc1cae76eb214800ce64cfb5d27c032c8556 on theikkila:master into d982cf99324124582721d1cd15a99cb975e93716 on mzabriskie:master.

Coverage decreased (-0.3%) to 93.687% when pulling 8cdfcd02849057d901b2140b7d398f34a0e2685e on theikkila:master into d982cf99324124582721d1cd15a99cb975e93716 on mzabriskie:master.

Fixed reviewed problems. Also updated the documentation from instance methods to contain the new added alias.


[=] code format

Follow the code style in nginx source code, format char* as char *.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Yang Fang seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

LGTM


fix Cython extension importing error.

It seems that pip will not install an empty init.py into submodule folder. I made this fix and it works well on my Fedora 23.

It's weird!
following is my test for python3.4:
`
(py3sdk) [hcheng@hcheng-dev pykrige]$ nosetests ./test.py --with-coverage --cover-package=pykrige
.....................................
Name                          Stmts   Miss  Cover
pykrige.py                        9      0   100%
pykrige/core.py                 227      1    99%
pykrige/kriging_tools.py        128     14    89%
pykrige/lib.py                    1      0   100%
pykrige/ok.py                   286     81    72%
pykrige/ok3d.py                 283     76    73%
pykrige/uk.py                   480     99    79%
pykrige/uk3d.py                 391    104    73%
pykrige/variogram_models.py      16      0   100%
TOTAL                          1821    375    79%
Ran 37 tests in 6.810s
OK
(py3sdk) [hcheng@hcheng-dev pykrige]$ git branch

issue#20
master
(py3sdk) [hcheng@hcheng-dev pykrige]$ cat lib/init.py
all = ['cok', 'lapack', 'variogram_models']
(py3sdk) [hcheng@hcheng-dev pykrige]$

`

@wind-chh Sorry for the later reply, and thank you for this Pull Request!
I restarted the Travis CI build and now it apparently passed (not sure why the random failure in the first place).
Would you have any references as to why adding an __all__ to __init__.py is needed for the cython extensions? As far as I understand, __all__ is only used if were to do a from pykrige.lib import * ; it should not affect what pip installs or don't install, shouldn't it?
I tried to install PyKrige on a clean Fedora 23 install (with system python 3.4), and the current master branch did build and runs fine, so I was not able to reproduce the issue #20 .
Happy to merge this PR (as it doesn't hurt anyway to have a __all__ there) and if it fixes the issue on your system, but I'm not convinced this actually affects Cython extensions.


Value predication: Training failed with model builder error after changing the feature column 's Data type from "String" to "Single".

System Information (please complete the following information):
Windows OS: Windows-11-Enterprise-21H2
ML.Net Model Builder 2022: <IP_ADDRESS>1801 (Main Build)
Microsoft Visual Studio Enterprise: 2022(17.3.6)
.Net: 6.0
Describe the bug

On which step of the process did you run into an issue:
Training failed with model builder error after changing the feature column 's Data type from "String" to "Single".

Data Source
https://testpass.blob.core.windows.net/test-pass-data/iris.mdf
To Reproduce
Steps to reproduce the behavior:

Select Create a new project from the Visual Studio 2022 start window.
Choose the C# Console App (.NET Core6.0) project template.
Add model builder by right click on the project.
Select Value predication scenario.
On Data page, select the data source and choose fare_amount column as label.
Change "rate_code" or "passenger_count" or "trip_distance" column's Data type from "String" to "Single".
On the Train page, you will see that training failed with model builder error "Splitter/consolidator worker encountered exception while consuming source data".

Expected behavior
Training successful.
Screenshot

Additional context:

The bug has been fixed on the latest main build: <IP_ADDRESS>3101

The bug is repro again on the latest main build: <IP_ADDRESS>5803

@v-Hailishi do we have a consistent repro? Can we please validate whether it's been fixed in the latest build. Thanks!

The bug has been fixed on the latest main build <IP_ADDRESS>1401


Research Item for Azure Function App Artifact Validation.

Note
Issues in this repo are for tracking bugs, feature requests and questions for the tasks in this repo
For a list:
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
If you have an issue or request for the Azure Pipelines service, use developer community instead:
https://developercommunity.visualstudio.com/spaces/21/index.html )
Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Feature
Enter Task Name: AzureFunctionAppv1
list here (V# not needed):
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
Check if we have a way to check the artifact directory structure when deploying Azure Functions Apps.

As discussed, we should try to handle it as part of post deploy steps.


ci: use admin token for auto shipit

It's required to push to main.

:rocket: PR was released in v0.1.3 :rocket:


Support Flutter 3.10.0

Add support for latest flutter 3.10.0.
Use flutter_lints as pedantic deprerated

@MacDeveloper1 thanks for the PR! We are quickly checking why the checks are failing:)

@MacDeveloper1 Looks like your commits are not signed. To address the 3.10.0 issue, I changed the dependencies in the meantime in this PR. Would be cool if you could open another PR with signed commits to replace pendantic with flutter_lints!

@edhom sorry for delay. Should this PR be closed?


feat: Next 12 upgrade

This update Next 12, and all node deps. As a part of bumping deps, bumped Node to v16 (current LTS, a few packages complained about)

Need to downgrade ES Lint, it'll be fixed in Next 12.0.5, but that's still in canary:
https://github.com/vercel/next.js/pull/29865


Fix PublisherBasedStreamMessageVerification failures

Need to understand why these timeouts occur.
Gradle suite > Gradle test > com.linecorp.armeria.common.stream.PublisherBasedStreamMessageVerification.required_abortMustNotifySubscriber FAILED
   java.lang.AssertionError: Did not receive expected element within 10000 ms
   Caused by:
      java.lang.AssertionError: Did not receive expected element within 10000 ms

Gradle suite > Gradle test > com.linecorp.armeria.common.stream.PublisherBasedStreamMessageVerification.required_spec317_mustSupportACumulativePendingElementCountUpToLongMaxValue FAILED
    java.lang.AssertionError: Could not subscribe org.reactivestreams.tck.TestEnvironment$ManualSubscriberWithSubscriptionSupport@213f411f to Publisher com.linecorp.armeria.common.stream.PublisherBasedStreamMessage@41584659 within 200 ms

Should be fixed by #874. Repeating the build to ensure the fix.


DX: Add 7.3 to Travis

What is Tidelift for an integration? Maybe we can combine the 2 PR's and merge, look good now :)

discussed in person


[material-ui] Not working correctly with Next.js 'edge' runtime

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Currently, the material-ui doesn't work correctly with the NextJS edge runtime.  There are warnings in a terminal and an error in the HTML.
I created a sample repository here.
Steps:

Create a new Nextjs project: npx create-next-app@latest (I selected the app router and typescript),
Install Material-UI: npm install @mui/material @emotion/react @emotion/styled,
Create a ThemeRegistry.tsx and use it in the app/layout.tsx. I followed the guide here,
Set the edge runtime in the app/layout.tsx: export const runtime = "edge";,
Build the app,
Run the app,
You will see the <html id="__next_error__" error on refresh, and warnings in the terminal: [TypeError: Cannot read properties of undefined (reading 'fontWeightBold')]

Note: the error lasts less than a second, so if you want to see it easier, set the Network to Slow 3G or similar.

Npm packages:
"dependencies": {
    "@emotion/react": "^11.11.1",
    "@emotion/styled": "^11.11.0",
    "@mui/material": "^5.14.12",
    "next": "13.5.4",
    "react": "^18",
    "react-dom": "^18",
    "sharp": "^0.32.6"
  },
  "devDependencies": {
    "@types/node": "^20",
    "@types/react": "^18",
    "@types/react-dom": "^18",
    "eslint": "^8",
    "eslint-config-next": "13.5.4",
    "typescript": "^5"
  }

Current behavior 😯
When I refresh the page, I see the error on the <html> tag like on the screenshot:
<html id="__next_error__"

I can also see the following warnings in the terminal:
[TypeError: Cannot read properties of undefined (reading 'fontWeightBold')]

Expected behavior 🤔

No __next_error__
No warnings in the console

Your environment 🌎

npx @mui/envinfo
System:
    OS: macOS 14.0
  Binaries:
    Node: 18.16.0 - ~/.nvm/versions/node/v18.16.0/bin/node
    Yarn: 1.22.19 - ~/.nvm/versions/node/v18.16.0/bin/yarn
    npm: 9.5.1 - ~/.nvm/versions/node/v18.16.0/bin/npm
  Browsers:
    Chrome: 117.0.5938.149
    Edge: Not Found
    Safari: 17.0
  npmPackages:
    @emotion/react: ^11.11.1 => 11.11.1
    @emotion/styled: ^11.11.0 => 11.11.0
    @mui/base:  5.0.0-beta.18
    @mui/core-downloads-tracker:  5.14.12
    @mui/material: ^5.14.12 => 5.14.12
    @mui/private-theming:  5.14.12
    @mui/styled-engine:  5.14.12
    @mui/system:  5.14.12
    @mui/types:  7.2.5
    @mui/utils:  5.14.12
    @types/react: ^18 => 18.2.25
    react: ^18 => 18.2.0
    react-dom: ^18 => 18.2.0
    typescript: ^5 => 5.2.2

Browser: Chrome

I debugged and found that the created theme has theme.typography as undefined and also theme.palette only has {mode: "light"} and no other color palettes resulting in the error.
Addings logs in node_modules files is not logging anything so I am still trying to figure out the root cause. I am guessing there is some API that is not available in edge runtime resulting in the above behaviour.

I'd like to add that the <CssBaseline /> component also breaks edge

+1 on this issue
Getting the following error when adding edge to a SSR page using MUI.
▲  Error: Could not find the module<EMAIL_ADDRESS>in the React Client Manifest. This is probably a bug in the React Server Components bundler.
▲  at ek (/helphub/node_modules/next/dist/compiled/next-server/app-page.runtime.prod.js:75:12617)
▲  at /node_modules/next/dist/compiled/next-server/app-page.runtime.prod.js:75:24092
▲  at Array.toJSON (/node_modules/next/dist/compiled/next-server/app-page.runtime.prod.js:75:25185)

Hi @brijeshb42,
Any news on this issue?
Thanks!

@brijeshb42  @DusanStojanovic-gowago
This is a duplicate of https://github.com/mui/material-ui/issues/36574
We would really need a solution for this problem it's currently blocking the use of edge runtime

Same issue here also, now our only blocker for moving to edge runtime

@DusanStojanovic-gowago @jordanvector Could you verify if edge runtime is working with the latest version of @mui/material or not ?

@brijeshb42 I confirm the edge runtime is working with the latest version of @mui/material. Thank you!

@brijeshb42 yes I can confirm latest version of @mui/material is working with edge runtime. Great work!

@brijeshb42 I have edited your comment to mention the exact version from which the bug is fixed, it will help developer upgrade to the right version


IotHubTrigger --> EventHubTrigger so many problems with upgrade from 3.1 to .net 6 isolated.

I have a legacy/older running 3.1 azure function IOThubTrigger running using  .net 3.1. Trying to upgrade the project to .net 6 Isolated. Running into so many issues with reading the EventData objects.
Old Function Signature which is working 100%
[FunctionName(nameof(IoTHubMessageHandlerFnc))]
        public async Task Run(
            [IoTHubTrigger("messages/events", Connection = "Service:IoTHubConnectionString")]EventData[] iotHubMessages )

When the function executes from the listener I get the following payload:

     local.settings.json file
     "Values" :{
     "Service:IoTHubConnectionString": "EventHubCompatibleEndpointFoundOnAzureIotHubApplicationIncludingEntityPath",```

<PackageReference Include="Microsoft.Azure.WebJobs.Extensions.EventHubs" Version="4.1.0" />
    <PackageReference Include="Microsoft.Extensions.Configuration" Version="3.1.0" />
    <PackageReference Include="Microsoft.NET.Sdk.Functions" Version="3.0.1" />

.Net6 Isolated 4.x azure function
The Problem:
Trying to run the application results in all EventData being empty. I subscribe to the same IOTHub endpoint using Microsoft.Azure.Functions.Worker.Extenstions.EventHubs V4.3 and the eventData is completely empty. I get the message batches but cannot read or parse them at all as it seems they are empty upon arrival??!

Attempted resolutions:
Upgraded Microsoft.Azure.Functions.Worker.Extenstions.EventHubs to V5.1.0
Result:
Exception is thrown on the Event Hub-compatible endpoint (inlcudes EntityPath),
with the following message
Azure.Messaging.EventHubs: The path to an Event Hub may be specified as part of the connection string or as a separate value, but not both.  Please verify that your connection string does not have the EntityPath token if you are passing an explicit Event Hub name. (Parameter 'connectionString').
Omitting EntityPath from the connection string results in
'The messaging entity 'sb://path.servicebus.windows.net/messages/events' could not be found. To know more visit https://aka.ms/sbResourceMgrExceptions.  (messages/events)'
There does not appear to be any documentation on how to use a different connection string since Connection is still the param in the Ctor of EventHubTrigger
        /// Gets or sets the optional app setting name that contains the Event Hub connection string. If missing, tries to use a registered event hub receiver.
        /// </summary>
        public string? Connection { get; set; }

Attempts at using Shared Access Policy for IOTHubOwner also fail along similar lines.
Service:IoTHubConnectionString: "HostName={hostname};SharedAccessKeyName={};SharedAccessKey={}"    Error:     Azure.Messaging.EventHubs: The connection string used  for an Event Hub client must specify the Event Hubs namespace host, and either a Shared Access Key (both the name and value) or Shared Access Signature to be v alid. The path to an Event Hub must be included in the connection string or specified separately. (Parameter 'connectionString')..... :(
SETUP
.csproj
 <PropertyGroup>
    <TargetFramework>net6.0</TargetFramework>
    <LangVersion>preview</LangVersion>
    <AzureFunctionsVersion>v4</AzureFunctionsVersion>
    <OutputType>Exe</OutputType>
    <ImplicitUsings>enable</ImplicitUsings>
    <Nullable>enable</Nullable>
  </PropertyGroup>
  <ItemGroup>
    <PackageReference Include="Azure.Messaging.EventHubs" Version="5.7.2" />
    <PackageReference Include="Microsoft.Azure.Cosmos" Version="3.30.1" />
    <PackageReference Include="Microsoft.Azure.Functions.Worker.ApplicationInsights" Version="1.0.0-preview1" />
    <PackageReference Include="Microsoft.Azure.Functions.Worker.Extensions.ApplicationInsights" Version="1.0.0-preview4" />
    <PackageReference Include="Microsoft.Azure.Functions.Worker.Extensions.EventHubs" Version="5.1.0" />
    <PackageReference Include="Microsoft.Azure.Functions.Worker.Extensions.Http" Version="3.0.13" />
    <PackageReference Include="Microsoft.Azure.Functions.Worker.Extensions.Timer" Version="4.1.0" />
    <PackageReference Include="Microsoft.Azure.Functions.Worker.Sdk" Version="1.7.0" OutputItemType="Analyzer" />
    <PackageReference Include="Microsoft.Azure.Functions.Worker" Version="1.8.0" />
    <PackageReference Include="Microsoft.Azure.WebJobs" Version="3.0.33" />
    <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="6.0.0" />
    <PackageReference Include="Microsoft.Extensions.Logging.ApplicationInsights" Version="2.21.0" />
  </ItemGroup>
  <ItemGroup>

.cs function
  [Function(nameof(IoTHubMessageHandlerFnc))]
        public async Task Run(
            [EventHubTrigger("messages/events", ConsumerGroup = "vscode",  Connection = "CapService:IoTHubConnectionString")] EventData[] iotHubMessages)
        {

local.settings.json
...
"CapService:IoTHubConnectionString": "Endpoint=sb://{name}/;SharedAccessKeyName={keyName};SharedAccessKey={AccessKey}};EntityPath={EntityPath}",
....

I am at a complete loss at this point on how to get my currently running event into my isolated .net6 project. I can't find any EventHubTrigger code samples that refer to dropping EntityPath from the connection string (Note this only breaks in V5.1.0 of Extensions.EventHubs)
Any advice or help would be greatly appreciated.

I found this link with exactly the same issues as I am having, however the resolution is not working for me. Replacing the EventHubName with whatever is in EntityPath ... in my case is application name iothb-ingress-pr resulted in the function working but the events are still all empty of any content. Running the same app as 3.1 give s me the messages.  System Properties is just a single kv of ContentType , "application-json".

I was able to figure out from another ticket to change my inputs body from EventData[] to Byte[][] or string[] . This has enabled me to obtain the payloads I need to continue working BUT it means I have lost all features of EventData . I feel like either something is very wrong here or this needs to be better documented somehow.

Binding to the EventData type is not yet supported in the isolated model. But you can bind the messages to string array or an array of POCO (which has a matching structure of the message payload sent to the event hub).
Obtaining the meta properties of the message is also supported. Please refer the event hub trigger example in our samples. Let us know if you run into any issues.


Template Requests

My original goal for this project was to create a bunch of templates that showcase what you can do with bulma.
If you have any ideas or suggestions for a new template, post a comment in this issue and I will do my best to accommodate everyones requests.

In the blog section, You've the "View more posts" href.
Can you create the template for "view more posts"?
I could do it by my own. But your designs are simple and impressive!
Thumbs up!

@xdevnull Thanks! I have been working on finishing the blog and blog post templates, I will add this to my todo list and get it included in the latest updates.

Maybe a dark theme for Bulma? Like a dark admin.

@gpetrov Thanks! I will add it to my todo list.

These are awesome templates! Thanks @dansup!

@dansup Thanks! Hope to see more awesome templates soon ;)

Thanks for all the templates, can you include some sidebar templates? thanks man!

Thanks for the requests, I have added them to my list of ideas for new templates.


Adding return to iterate.onItem to increment "recordCount" only if it  returns "undefined" (keep compatibility) or "true".

Because IndexedDB is limited on filtering, I had to use iterate and manually check each item with custom filters.
But I still need to limit the quantity of records returned, so I don't iterate the whole store, I know that there is the param "limit" the problem is that IDBWrapper doesn't know how many records passed my filter.
To solve this, I changed the "onItem" function to return a boolean and increment "recordCount" only if it returns "undefined" (keep compatibility) or "true".

Thanks for the PR!
The idea of allowing the user to reject a result in the onItem callback is nice :)
However, there's two things that still need to be done:

This might break backwards compatibility, as we don't know what existing code might return in the callback. So, it needs to be guarded by an extra flag passed in the options argument, something like options.allowRejection or options.allowItemRejection.

Test(s) need to be added for this feature.

After these tasks are done, I'd happily merge this. Feel free to do these, or, I can do them. Would be fine with me, but might take some time.
Out of curiosity: What did you try to do that couldn't be achieved with Indexes and KeyRanges? Personally, I find IndexedDB's filtering capabilities quite sufficient, so I'd love to know what limitations you ran into :)

Hi Jens,
Thanks for the answer!
I handled the backward compatibility by verifying if the return is undefined or true, so existing code that doesn't actually return something, would keep working as before, but the idea of a parameter is nice too.
About tests, you are right!
I am going to do these as soon as I get some free time =)
About what I wanted to do.
A simple SQL like "%searchterm%" case insensitive.
There is an input in my app that uses angular-ui's typeahead and I want to search the text the user is typing anywhere in the item.
I couldn't find a way to do this with indexes and KeyRanges, if you know a way to do it, I would be glad to hear ;)

so existing code that doesn't actually return something, would keep working as before

That's the point – if, for whatever reason, existing code returns boolean false, it would change IDBWrapper's behavior. Adding the flag will make sure everything stays safe :)

A simple SQL like "%searchterm%"

Ah, yeah, IndexedDB has no solution to this. Makes me wonder if it would be a good idea to add a custom matcher or filter function one could pass to query() which would be run for every match in the query to further reduce the result set…

Got it, you are right, it might break someone's code :)
The matcher idea sounds great!
It would be almost the same thing I did, but with a more explicit API and would not mess up with already existing code.
If you want, I can do it when I get some free time.

Well, if we had the rejection code in the iterate(), then the query() could make easily use of it by passing the filter/matcher as the onItem callback – so, let's get this PR into master, and then onwards to extending query() :)

Alright! As soon as I can, I will create the new flag and add some tests.

Hey! I just added the flag and created 2 new tests.
If there is anything missing, please tell me.

Great, thanks! I'll merge this and add some cosmetics to it. Will be released in 1.7.
Thanks again for the PR! :)


How to set cover image for each article

Excuse me, it is possible to set cover image for each articles rather than randomly using the default1-4.jpg? If if possible, how should I set it?
By the way, I really like the theme, thank you for creating it!

Definitely can. Just edit the cover field to customize it.

And thx for loving it.~ ❤

Definitely can. Just edit the cover field to customize it.

And thx for loving it.~

Thank you, I do it. By the way, what is the meaning of the images field, and where I can find the explaination of all fields at the top of .md files? I can't find it in hugo official doc.

You can find all definitions there: https://gohugo.io/content-management/front-matter/.

You can find all definitions there: https://gohugo.io/content-management/front-matter/.

Thanks!


Find Independent Sets in tree decomposition

Depends on #9

[ ] Example tree decomposition
[ ] Find max. Independent Set

probably a duplicate of #27


Use tmux to benefit long SSH remote runs

Is it possible to wrap current remote SSH commands into a tmux session? I think it would be good for long SSH remote runs in an unstable network connection.

When Guild runs a remote operation, it first starts the operation on the remote server as a background process. Guild then watches that operation so you can see its output. However, the operation continues to run even if Guild is disconnected. E.g. you can type Ctrl-c after the remote operation has started and the operation continues to run.
Guild uses the watch command for this. You can do it yourself this way:
$ guild run train --background   # runs operation in background
$ guild watch  # watch background operation output - Ctrl-c doesn't kill operation

Oops. Sorry, I didn't know that feature. I'll close the issue. Thanks!


script permission

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@0a69d2c). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             main       #34   +/-   ##
========================================
  Coverage        ?   100.00%
========================================
  Files           ?         3
  Lines           ?       168
  Branches        ?         0
========================================
  Hits            ?       168
  Misses          ?         0
  Partials        ?         0

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 0a69d2c...d555880. Read the comment docs.


SPARK-9487][Tests] Begin the work of robustifying unit tests , start with ContextCleanerSuite

…tests
What changes were proposed in this pull request?
This is a work in progress pull request whose goal will be to robustify the unit tests, to kick this off I will focus on one module at a time, for now I will focus on robustifying all unit tests around ContextCleanerSuite
(Please fill in changes proposed in this fix)
Changed ContextCleanerSuite to use local[4] instead of local[2]
How was this patch tested?
I ran unit tests locally for this suite and everything passed locally, of course this doesnt mean anything till we see how Jenkins deals with these changes
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)
Please review http://spark.apache.org/contributing.html before opening a pull request.

Test build #71021 has started for PR 16498 at commit b338ca9.

Test build #71021 has finished for PR 16498 at commit b338ca9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71021/
Test PASSed.

Can you explain why this makes it more robust?

@rxin long story here, the goal of this jira is to create robust unit tests where appropriate and in doing so use the same number of local instances in the spark config both in the Scala and Python code which is local[4].   I am making this change individually and creating pull requests, in some cases the tests will fail in Jenkins but work locally, in these cases I will be working backwards from Jenkins and trying to make the unit tests more robust and in doing so independent of the number of local instances chosen.   Make sense?

To be clear, we're not going to commit this one test case at a time. The best strategy is to get a logical subset working, at the least, like for a whole module. It may entail some tricky debugging if you can't reproduce the Jenkins failures locally, though I think I flagged some potential fixes in the last PRs already.
To be honest I don't know that this is worth our time. It's not making the tests better, just changing them to be more consistent -- which, as a side effect, reveals a few tests which are a bit dependent on local[2] vs local[4] for exact results. This isn't such a big deal.

@srowen I'm ok closing this given the time/effort and energy I've put in, I am more than willing to do challenging debugging for every module that we wish to make unit tests better but if you don't feel like its worth the effort I can pick a different JIRA to tackle.  Let me know how you'd like to proceed.

As before, I think this is around in circles. This PR just changes 1 file, now, not even a module's worth.
If you aren't progressing work on this and haven't been able to resolve why your previous change failed, and there's no strong need to do this anyway, I think we should not keep repeating this discussion.


Update teams-together-mode.md

Correcting spelling of word in step 15.

Acrolinx Scorecards
We do not currently enforce a minimum score for content merged to the default branch. This will change in the future.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/apps-in-teams-meetings/teams-together-mode.md
90
26
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit fef7295:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/apps-in-teams-meetings/teams-together-mode.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


python310Packages.yfinance: 0.1.68 -> 0.1.69

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.yfinance is: "Yahoo! Finance market data downloader (+faster Pandas Datareader)"
meta.homepage for python310Packages.yfinance is: "https://aroussi.com/post/python-yahoo-finance"
meta.changelog for python310Packages.yfinance is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
Warning: no invocation of /nix/store/gcfvqq3lpnzv4066zm8asznb07mrvwrk-python3.10-yfinance-0.1.69/bin/sample had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/gcfvqq3lpnzv4066zm8asznb07mrvwrk-python3.10-yfinance-0.1.69/bin/.sample-wrapped had a zero exit code or showed the expected version
0 of 2 passed binary check by having a zero exit code.
0 of 2 passed binary check by having the new version present in output.
found 0.1.69 with grep in /nix/store/gcfvqq3lpnzv4066zm8asznb07mrvwrk-python3.10-yfinance-0.1.69
found 0.1.69 in filename of file in /nix/store/gcfvqq3lpnzv4066zm8asznb07mrvwrk-python3.10-yfinance-0.1.69

Rebuild report (if merged into master) (click to expand)

14 total rebuild path(s)

7 package rebuild(s)

7 x86_64-linux rebuild(s)
7 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
python310Packages.qiskit-aqua
python310Packages.qiskit-finance
python310Packages.yfinance
python39Packages.qiskit
python39Packages.qiskit-aqua
python39Packages.qiskit-finance
python39Packages.yfinance

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/gcfvqq3lpnzv4066zm8asznb07mrvwrk-python3.10-yfinance-0.1.69 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.yfinance https://github.com/r-ryantm/nixpkgs/archive/9fd02431f813e32b9df99ff1b517213723e7b32c.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/gcfvqq3lpnzv4066zm8asznb07mrvwrk-python3.10-yfinance-0.1.69
ls -la /nix/store/gcfvqq3lpnzv4066zm8asznb07mrvwrk-python3.10-yfinance-0.1.69/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages failed to build:

python310Packages.qiskit-aqua
python310Packages.qiskit-finance
python39Packages.qiskit

4 packages built:

python310Packages.yfinance
python39Packages.qiskit-aqua
python39Packages.qiskit-finance
python39Packages.yfinance

Maintainer pings
cc @drewrisinger for testing.

#154969

Also included in https://github.com/NixOS/nixpkgs/pull/154811


check package dependency check

@pbalcer reported test package broken on Ubuntu 18.04 and created a PR for pkg-config, which I later reverted. There are two reasons: 1. I checked the variables set after calling pkgconfig for check, which didn't seem to be doing all the right things here, and it breaks build on some other platform. 2. The CMake convention seems to prefer writing your own find_package routine instead of using pkg-config.
We still need to address the build problem on Ubuntu 18.04, this ticket will track that effort. We should explore a way that is consistent and robust on all platforms, which will require some digging and talking to folks.

You are right that the variables set by pkg-config didn't match the ones the FindCheck.cmake script sets - but it's arguable that it's the pkg-config that does the wrong thing.
I think upstream projects should implement both pkg-config as the primary source of information and find_package as backup. This ensures that it's the library author (or the distro) that ultimately controls how the code is linked with an application through the use of .pc files that are installed alongside the -dev package.
The immediate problem that I wanted to solve with my patch was to make sure that distro-installed libcheck also pulled in dependency on libsubunit - which the existing FindCheck.cmake didn't do (and it probably shouldn't...).


DFS

DFS (depth-first search) is an alternative method for Pauli propagation. It runs a lot slower than merging BFS, but requires close to O(1) memory. IBM's utility experiments were simulated using a parallelized DFS approach (paper).
I think we never want to start off with DFS, but it may be optimal for continuing the search after merging BFS has saturated the RAM that we want to use.
Alternatives could be writing parts of the paths to disk and still use merging BFS, but that seems like it should not be natively supported (requires write access to the user's machine).

We will consider this closed for now until we believe we need DFS.


Allow setting a default file format

Most programs, at least to start, have config files in ~/.<prog> or ~/.<prog>rc, neither of which have extensions. Those programs can't use this library unless there's a way to set the default file format.
The API for this could be pretty simple, something like pyautoconfig.set_default_extension('toml') where set_default_extension modifies a global and adds a leading '.' if necessary.

Closed in https://github.com/jyn514/py-autoconfig/commit/301cfe8fac0304a285cfc716377a54b87aa7f859


Add IntersectionJoiner

Joins loaded geo data with regions (assigns objects to regions based on intersection)

If memory becomes the problem (multiplied geometries) then we can use dedicated dataclass object with 3 members: regions gdf, features gdf and multiindexed gdf with only intersection geometry. Then embedder might be prepared to use this dedicated class instead of a gdf.
Additionally this approach won't break the "single geometry" rule of proper geodataframe usage.


Add random-assign to poule.yml

Randomly assign incoming pull requests to one of the (Docker Inc employed) maintainers of the project.
The role of the assignee is not necessarily to review the pull request herself/himself, but to be accountable for moving it forward.
Cc @vieux @thaJeztah.

LGTM 👍

Closing this as we're having a debate with @tonistiigi about giving it a threshold before assigning (e.g., if it's still opened after 1 week).


[Bug] Language C highlights "class" as token

Reproducible in vscode.dev or in VS Code Desktop?

[X] Not reproducible in vscode.dev or VS Code Desktop

Reproducible in the monaco editor playground?

[x] Reproducible in the monaco editor playground

Monaco Editor Playground Link
Monaco Editor Playground Link
Monaco Editor Playground Code
const value = `typedef struct {
	xchar class;
} foo;`;

// Hover on each property to see its docs!
const myEditor = monaco.editor.create(document.getElementById("container"), {
	value,
	language: "cpp",
});

Reproduction Steps

Go to https://microsoft.github.io/monaco-editor/
Select Language c
Write

typedef struct {
    int class;
}

Actual (Problematic) Behavior
See that class is highlighted as keyword
Expected Behavior
class is no keyword in C, it should be highlighted like any other variable
Additional Context
No response

It seems like not just "class" but all the keywords which where added in C++ but not present in C are highlighted as keyword.
I looked into it and it seems there is no special file made for C language in basic languages folder. The keywords returned for .c extension is same as the ones returned for .cpp
Take a look at this

What we can do is make a new folder for C language in basic languages.

Created PR to address this issue


BinderHub + JupyterHub diagrams

Hey all - we've had need several times now for some simplified Binder + JupyterHub diagrams. I took a shot at riffing off of @yuvipanda's more complex kubernetes diagram. You can find a draft here:
https://docs.google.com/presentation/d/1t5W4Rnez6xBRz4YxCxWYAx8t4KRfUosbCjS4Z1or7rM/edit#slide=id.g25dbc82125_0_53
This has a ways to go most likely (and may be incorrect in some places). It'd replace the diagram here and could be re-used in workshops / demos / etc.
LMKWYT and please do add edits / comments  etc!  cc @willingc @yuvipanda @minrk

closing this as I think these are in a fairly steady-state now!


Fix node creating and database properties

Closes #312

/**

Class NodeParentRequest represents an incoming DTO for adding a common type of node.

Please change description from Class to Interface here and same NodeParentResponse interface (com.reckue.post.transfers.nodes [line:15])


Automated cherry pick of #4107: Fix Antrea Octant plugin build

Cherry pick of #4107 on release-1.7.
#4107: Fix Antrea Octant plugin build
For details on the cherry pick process, see the cherry pick requests page.

/test-all

/test-conformance


Flex support for imported layers

Not sure if I'm not using it properly but I can't seem to get this to work with layers imported from sketch. Attached a zip with sketch file and framer project.
https://dl.dropboxusercontent.com/u/4037263/_code/flexerTest.zip

Flexer can't be installed just by copying the .coffee file, due to the dependency with css-layout. It requires to be installed through npm. Follow the instructions on the readme file. Citing:
3-Step Installation
In order to install it, you will need node/npm.

From your Framer project folder, type npm install flexer.
Create a file npm.coffee in your modules folder, with this line: exports.flexer = require "flexer"
Lastly, add the following line in app.coffee (or your main file): { flexer } = require "npm"

Actually, it works perfectly when I create new layers... I already have css-layout installed globally.
So the same file with this code works perfectly:
container = new Layer
	flex: true

layer = new Layer
	flex: 0.5
	superLayer: container
	backgroundColor: "#333"

layer2 = new Layer
	flex: 0.5
	superLayer: container
	backgroundColor: "#ccc"

Trying to sketch.box1.flex = 0.5 that to a layer from sketch does not adjust it's width or height. Although now I also see I somehow zipped up the wrong file.... Take a look at this one instead, I put both the working and non working example to ensure that Flexer is loading properly.
https://dl.dropboxusercontent.com/u/4037263/_code/FlexerTest2.zip

Any idea why this isn't working for Sketch layers but is for new layers?

I found the problem cause already, but I need to come up with a solution. Will update soon (hopefully tonight)

Anything I can help with? What is the problem?

Thanks for the offering! I manage to fix it, but you can definitely help testing it :+1:
The problems were actually two:

Whenever the flexible properties were set after layer creation, the layout tree wasn't calculated properly (there were missing layers). This had nothing to do with the importer, it was a general problem
When importing, the layer structure is as follows:

Screen layer
|-- Import layer (this one is added by Framer)
|    |-- bg
|    |    |-- box1
|    |    |-- box2

The second problem was that the "Import layer" had no specific flexible property, so its calculated height was 0. Now every flexible layer has {flex: 1} by default so that should fix it.
Let me know if it works. I understand there's no need to update the npm package for now, as you are getting the .coffee file straight from github, right?

Quick test shows it working!  Cool, now to start tinkering with it.
Thanks for looking into it, I can imagine many cool ways to use this.

Minor issue seems that layer order is reverse of expected, which then lays out layers on opposite sides. In sketch I follow a top down / left to right ordering for my layers. Like these comments, comment 1 on top, comment 2 below it in the layer panel etc.  Then inside of a comment group, we would have an avatar group for the image, and comment box group for the comment detail, with avatar above comment in the layer panel.
When importing and flexing, the layers are in reverse order, comment 1 is on the bottom, comment 2 is on the top, and then avatar is on the right, comment on the left.
I can setup a demo file if that doesn't quite make sense.

Mmm... it makes sense, but it's hard to know without the sketch file + Framer code :)

Sure, here you go. To keep it a bit simpler I just have one list item with an avatar image placeholder and a box to the right with some text. The result is that the image is on the right and text is on the left, and the text scales up to fix (which I don't want).
https://dl.dropboxusercontent.com/u/4037263/_code/flexerTest3.zip

Thanks for taking the time to test it and upload the files :)
I discovered many things by checking your example:

The imported order of layers is reversed somehow. But css-layout offers the row-reverse and column-reverse for flexDirection (i didn't know!). So just by using this, it should work.
I have changed the way the flex layout tree is created. By default, all layers will keep their original size, even if their parents are flex. Once you setup any Flexer attribute, these fixed measures will be deleted. This fixes the problem with the text scaling up that you were mentioning.
Due to this addition, whenever you import something from Sketch you need to set the top-most layer of the imported group to flex if you want the design to resize with the screen (see first line of the code below)

This is how your example looks now:
# Hidden superLayer, added by Framer
# We need this if we want the design to adapt when resizing the screen
sketch.bg.superLayer.flex = true
sketch.bg.flex = true

# Keep the original height
sketch.topBox.fixedHeight = sketch.topBox.height
# We can use reverse!
sketch.topBox.flexDirection = "row-reverse"

# We can use this if we want the image to be square
sketch.image.fixedWidth = sketch.topBox.fixedHeight

sketch.txt.justifyContent = "center"
sketch.txt.flex = true # or any number, it's the only flex layer at its level as image has fixedWidth
sketch.txt.paddingLeft = 20

Hope this helps!

I did know of the reverse settings, and while it does work it poses a couple problems. For one, it requires the reversal on all groups that have children that are flex... Basically any time I want to add layer.flex I also need to add layer.row-reverse, making more code.
Also it's generally counter-intuitive. Is there any other way to determine layout position instead of layer order? If layer1.x < layer2.x then it means that layer likely should go first if they layout is column. Same for y and row. Then it won't matter what the layer order is in Sketch, but it will adjust based on position of the layers themselves.
I can tinker with that if you don't mind pull requests.

Also, this caused me some confusion coming from css flexbox. Flexer layers are defaulting to column (rather, layer.flexDirection prints as 'undefined').
In css flex-direction has an initial value of "row".
Any idea where that is getting set?

About the imported layers order: I agree it's bothering to add row-reverse to every layer that is added. The problem relies on the way Framer arranges the layers exported from Sketch.

For example, this is how your example gets exported:
bg
--topBox
----txt
------Group
----image

The solution would be to check if we're dealing with regular layers or imported ones, and change the nodes order in case the layers are imported. I'm happy to take PRs in, of course.

On the default flexDirection value, I think it's more a css-layout module problem. If I'm not wrong, this line sets the default value as column. I will investigate it further and open a PR if needed.


Bug with really long command in container's consoles prompt

Description
The bug is complex. When you typing a long command in the prompt of a container's console, before having no more space on the line, the prompt come back at the beginning of the same line, and just write over the first line...  But if you copy/paste a long command, no pbm.
Another thing, if you continue to type your command and you press enter, the entire command will be send in the prompt. It's purely esthetic but that's blow my mind sometimes when I must type any long commands.
Example:

Steps to reproduce the issue:

Open a prompt
Type long command
Take a beer

Any other info e.g. Why do you consider this to be a bug? What did you expect to happen instead?
Technical details:

Platform (windows/linux): Windows 10
Browser: Chrome / Firefox / Edge

Cannot reproduce, this could have been fixed when solving #1012 or #932.


Paper

Did you published paper on journal or conference based on this work?

Sorry for the delayed response, I just saw this.
A publication was planned, but scrapped before it was finished. Do tell if the work here ends up being useful to you!


append failing test reproduction

Failing test for #2584
Simplified example from #2584, converting a file to a AVAudioPCMBuffer, and trying to append to the file, throws the same error I was seeing in my app. Added a failing test per request.

So, this test is appearing to pass, I'm confused.


[rdc] out-of-clockedge for MainPok reset

This commit shifts the timing of the main pok reset. Previously, main_pok asserts at t0 time at the same time as clock. Now, main_pok remains high for two cycles then is asserted.

LGTM - should we add a comment to the file explaining why the POK signal is delayed with respect to the clock signals below?

The comment already explained that (prior in the comment)


REQUEST_CHANNEL Interaction Type

Some use cases have come up that may warrant another interaction model. The current 4 models all support a single REQUEST frame only. Bi-directional communication is happening under the covers, and can be achieved by an application using the 4 different REQUEST types in both directions. However, this does mean certain use cases would require the application to associate unidirectional streams, which somewhat defeats the point of having interaction models on top of a multiplexed bi-directional transport.
The two use cases I've come across both have similar semantics as follows:

client requests a stream of data that initially bursts the current view of the world
deltas/diffs are emitted from server to client as changes occur
client wants to update the subscription over time to add/remove criteria/topics/etc

The goal is to avoid re-fetching the initial burst of all data again each time the client changes the subscription.
Using just request/stream would mean the client cancels and re-requests each time, and gets a new burst of data on each change.
Another option is to maintain multiple separate request/stream connections that are more granular. This can indeed work in some cases, but if there is overlap amongst the criteria, it will still require canceling and re-requesting in some cases that are not purely additive.
The most efficient would be a custom application protocol on top that uses REQUEST_RESPONSE to send criteria, then REQUEST_SUBSCRIPTION to receive the events. This however is building a non-trivial behavior again on top of ReactiveSocket. It's no different than using HTTP GET to mutate server-side state and HTTP SSE to receive the changing output.
Thus, the proposal is to add a 5th interaction type, REQUEST_CHANNEL that allows multiple request frames and multiple response frames (potentially infinite or finite).
From an application perspective, it would look like a channel with the ability to write and read ReactiveSocket Frames.
// server
handleRequestChannel(Publisher<Frame> requests) {
       // subscribe to stream of incoming requests ... write Frames to output
}

This is almost easy to add ... the one trick I can think of is thus far we have only had to have REQUEST_N semantics flowing from RESPONDER to REQUESTER, and Lease behavior going from RESPONDER to REQUESTER.
In this case, it would make more sense to have REQUEST_N semantics go both directions, since it is a ReactiveStream Publisher that would represent the input. The Lease would control if REQUEST_CHANNEL can occur, but once the channel is established, then REQUEST_N would be used in both directions.

:+1:

I have added an implementation of this to reactivesocket-java in https://github.com/ReactiveSocket/reactivesocket-java/pull/21

Language has been added to the spec for REQUEST_CHANNEL.


Listen NetInfo connectionChange event (change deprecated)

Fixes #313

@geektimecoil bump. Should be a quick fix/release.  Thanks!

Any ETA on merging this?

It'd be great to merge this in and release a new version, the yellow box is quite annoying!:)

Still needs to get published on npm.

@avishayil Can you publish this update to npm?

Hi @avishayil and @jkasten2, any update on getting this released now that the fix was merged? ☺

+1 we need release with this

+1 for npm update

Just got into the same problem after upgrading my app, hope it will be merged someday in the future  🙇

@adriantombu, it's already merged.

@ujwal-setlur but after the merge it was not published to NPM as new version.

I thought I just checked last night, and it was there. Will check again

hi, any news on this ? still getting this message
thx


Fix nightly build error

The nightly job failed over here. It looks like it is a syntax error in int_clearinghouse__station_agg_five_minutes, probably due to the initial full refresh templating in differently from what is in the incremental logic. @britt-allen would you be able to take a look?

Taking a look

I am not seeing an extra parentheses anywhere, but will keep digging

Yeah I looked at the macro in dbt cloud and couldnt see where the syntax error would be for a full refresh. I am trying to find the nightly job query in Snowflake and am having trouble there too 😓

Figured out where to see the compiled code in dbt cloud. Also figured out how to see it in Snowflake with Ian's help :)


ASYNC_MASTER not supported with dledger?

does dledger supports ASYNC_MASTER? I use rocketmq 4.8, one leader and two followers, after leader selected, the leader brokerRole is changed to SYNC_MASTER showed in rocketmq-console webpage, not ASYNC_MASTER I configured in conf/dledger/broker-nx.properties.

As each node can be selected as leader, I configured different brokerId for each node, same brokerRole=ASYNC_MASTER  in conf/dledger/broker-nx.properties.

As each node can be selected as leader, I configured different brokerId for each node, same brokerRole=ASYNC_MASTER  in conf/dledger/broker-nx.properties.

When using Dledger mode, replication depends on the Raft protocol, so configuring ASYNC_MASTER has no effect.

When using Dledger mode, replication depends on the Raft protocol, so configuring ASYNC_MASTER has no effect.


[HOW TO CONTRIBUTE]

Hi,
If you are reading this, you want to contribute to our mission of creating the future of design systems, enabling everyone to build beautiful web apps with utility first customizable components; thank you!

Hi @HassanBahati, thank you for sharing such a wonderful resource looking forward to contributing through for the development of beautiful web apps with utility first customizable components.

Hi @HassanBahati, thank you for sharing such a wonderful resource looking forward to contributing through for the development of beautiful web apps with utility first customizable components.

Hi @rukundob451, much appreciated. Looking forward to the contributions. Please checkout our Contributing Guide and issues section.

Hey @HassanBahati
I would love to contribute over this issue

Hey @HassanBahati I would love to contribute over this issue

hi @maddy020 I am excited that you want to contribute.
we are building the website lives here in this directory
website tech stack

next.js
tailwindcss

you may also find the figma design here
we are building the documentation lives here in this directory
docs teach stack

react.js
docusaurus v2

Thank you for this @HassanBahati.


Issue with docker-compose and entrypoint: volume not detected

Hi, I am having issues configuring cayley with mongodb using docker-compose. I get the following error:
graph_1          | ERROR: logging before flag.Parse: E0305 08:20:48.563422       1 cayley.go:142] The assets flag is deprecated as assets are provided as part of the binary.
graph_1          | I0305 08:20:48.563529       1 command.go:805] Cayley version: v0.7.x-dev (9c843edfeaa4)
graph_1          | Error: open  data/cayley.yml: no such file or directory

This is my docker-compose file:
version: '3.7'
services:
  db:
    image: mongo:latest
    ports:
      - 27017:27017
    volumes:
      - mongodb_data:/data/db
  mongo_express:
    image: mongo-express:latest
    environment:
      - ME_CONFIG_OPTIONS_EDITORTHEME=ambiance
      - ME_CONFIG_MONGODB_SERVER=db
      - ME_CONFIG_MONGODB_PORT=27017
      - ME_CONFIG_MONGODB_ENABLE_ADMIN=true
      - ME_CONFIG_MONGODB_AUTH_DATABASE=admin
    ports:
      - 8081:8081
    links:
      - db
  graph:
    image: cayleygraph/cayley:latest
    ports:
      - 64210:64210
    volumes:
      - ./data:/data
    links:
      - db
    entrypoint: ["cayley", "http", "--host=:64210", "-c data/cayley.yml"]
volumes:
  mongodb_data:
  data:

the file cayley.yml contains the following configuration:
store:
  backend: mongo
  address: <IP_ADDRESS>:27017
  ready-only: false
  options:
    database_name: "cayley"

I have tested the cayley.yml directly from the docker image (without using docker-compose) and it works fine.
I found a similar issue in the docker project [1]. Anyone experiencing similar problems?
Thanks!
[1] https://github.com/docker/compose/issues/1574

Hey, I think it has to do with the location of the configuration file and the syntax of command-line arguments. Please try changing graph's entrypoint to: ["cayley", "http", "--host=:64210", "-c=/data/cayley.yml"]

Closing for no comment. @narko please reopen if the solution does not work for you.


Should spec constants be uppercased by build rather than CSS?

It's a bit odd that text at e.g. NormalCompletion has a step that looks like "Return Completion Record { [[Type]]: NORMAL, [[Value]]: value, [[Target]]: EMPTY }." but copies as "Return Completion Record { [[Type]]: normal, [[Value]]: value, [[Target]]: empty }." (because text-transform "has no effect on the underlying content, and must not affect the content of a plain text copy & paste operation").
It would probably reduce surprise (and increase the value of copy–paste interactions for purposes such as test262) to perform uppercasing at build time such that it is part of the resulting HTML source:
-{ <var class="field">[[Type]]</var>: <emu-const>normal</emu-const>, <var class="field">[[Value]]</var>: <var>value</var>, <var class="field">[[Target]]</var>: <emu-const>empty</emu-const> }
+{ <var class="field">[[Type]]</var>: <emu-const>NORMAL</emu-const>, <var class="field">[[Value]]</var>: <var>value</var>, <var class="field">[[Target]]</var>: <emu-const>EMPTY</emu-const> }

The alternative is for them to copy as ~empty~, with tildes added in, to better match the source rather than the display. See https://github.com/tc39/ecmarkup/issues/550.

Ah yeah, that's probably better.


Merge pull request #11 from ronaldchan333/branch1

Merge pull request #10 from ronaldchan333/main

23-12-5-8


RPC: UDFs not serialized

Description
  2023-09-01T17:32:35.206140Z ERROR testing::slt::cli: Error while running test `sqllogictests/psql`, error: test fail: statement failed: Execution error: failed to execute physical plan on remote service: Remote session error: error while executing physical plan: status: Unknown, message: "Error during planning: There is no UDF named \"pg_table_is_visible\" in the registry", details: [], metadata: MetadataMap { headers: {"content-type": "application/grpc", "date": "Fri, 01 Sep 2023 17:32:35 GMT", "content-length": "0"} }
[SQL] SELECT n.nspname AS "Schema",
    c.relname AS "Name",
    CASE c.relkind WHEN 'r' THEN 'table'
    WHEN 'v' THEN 'view'
    WHEN 'm' THEN 'materialized view'
    WHEN 'i' THEN 'index'
    WHEN 'S' THEN 'sequence'
    WHEN 's' THEN 'special'
    WHEN 't' THEN 'TOAST table'
    WHEN 'f' THEN 'foreign table'
    WHEN 'p' THEN 'partitioned table'
    WHEN 'I' THEN 'partitioned index' END AS "Type",
    pg_catalog.pg_get_userbyid(c.relowner) AS "Owner"
FROM pg_catalog.pg_class AS c
    LEFT JOIN pg_catalog.pg_namespace AS n ON n.oid = c.relnamespace
    LEFT JOIN pg_catalog.pg_am AS am ON am.oid = c.relam
WHERE c.relkind IN ('r', 'p', 'v', 'm', 'S', 'f', '')
    AND n.nspname <> 'pg_catalog'
    AND n.nspname !~ '^pg_toast'
    AND n.nspname <> 'information_schema'
    AND pg_catalog.pg_table_is_visible(c.oid)
ORDER BY 1, 2;
at /Users/sean/Code/github.com/glaredb/glaredb/testdata/sqllogictests/psql.slt:4

@scsmithr I've been looking into this one & It seems like there is significantly more work than I anticipated. Initially I thought we could just throw them into datafusion's FunctionRegistry and call it a day.  While this works for most of them, several rely on session_vars.
This is pretty straightforward for functions that don't rely on the context such as pg_get_userbyid and pg_table_is_visible, but ones that are dependent on it such as current_database and current_schema are a bit trickier.
I was thinking of implementing a physicalExpr for those functions that takes in the ctx vars as function arguments, then implementing create_physical_expr in our planner. The PhysicalPlanner is the only point during execution/planning that we have access to the sessionvars, so i think we'd have to do the mapping there.
Was wondering if you had any thoughts on other potential solutions?

I don't really have any thoughts on this. But I figured there might be some tricky bits due to that dependency.
Implementing a physical expr seems like it'd make sense. I say we go with a quick proof-of-concept for one of the functions that we're having trouble with, and see how that plays out.

closed via #1755


hangouts: replace facebook with telegram

Reasons:

no one interested in monero uses facebook
company with the worst reputation for privacy ever
not even bitcoin.org lists facebook
telegram isn't listed at all although it's very active and has a good portion of the community
on telegram people can find immediate help

We list telegram in the footer, not in hangouts. I don't remember the details, but it was decided to not put in on hangouts because of the large amount of spam.

no one interested in monero uses facebook

This is sadly not true. A portion of the community thinks that we have to reach people where they are and facebook is one of the main social media. I too agree that feels weird to list it, but i don't think this is going to change, as many in the active community are willing to make heavy compromises in order to onboard more people.

We list telegram in the footer, not in hangouts. I don't remember the details, but it was decided to not put in on hangouts because of the large amount of spam.

Yeah, I saw it. In fact, I was wondering what on earth the telegram group was doing in the footer of every page. There is Hangouts for a reason.

This is sadly not true. A portion of the community thinks that we have to reach people where they are and facebook is one of the main social media. I too agree that feels weird to list it, but i don't think this is going to change, as many in the active community are willing to make heavy compromises in order to onboard more people.

I can understand, however I don't think it's appropriate to jeopardize the Monero reputation just to get it out there as quickly as possible. This can raise a big red flag for both those who are looking to join the community and investors. And besides, in no way would Bitcoin be taken as seriously if it had a facebook profile. Just no way.

There is Hangouts for a reason.

Well you are not wrong. If we list it, should be placed in the appropriate section.


Changing the default “subject” field for the verification email in laravel 5.7

I am trying to change the default subject field in the verification email that come with laravel 5.7. Curently it is Verify Email address. How and where can i change it? I have searched all over the place by myself, and online, but because it is brand new, i could not find an answer. Can you help me please? Thanks!

Please link/post the solution from StackOverflow and close the issue.

The solution is here: https://stackoverflow.com/questions/52408042/changing-the-default-subject-field-for-the-verification-email-in-laravel-5-7


createMany() does not exist when a model has an Unsupported() field on PostgreSQL

I'm using postgreSQL, still facing the same issue.
datasource db {
  provider = "postgresql"
  url      = env("PG_CONNECTION_STRING")
}

model node_locations {
  NodeId String @unique(map: "node_locations_id_idx") @default(dbgenerated("gen_random_uuid()")) @db.Uuid
  ExternalId String @unique(map: "node_locations_ext_id_idx")
  Name String
  Description String
  Geometry Unsupported("geometry(Point, 4326)")
  Meta Json
  Provider String
  Species String[]
  Status String
  LastPing DateTime

  @@index([Geometry], name: "node_locations_geometry_idx", type: Gist)
}

Prisma version: 5.5.2
Note: prisma db push is used for code generation

Originally posted by @rs-nikic in https://github.com/prisma/prisma/issues/11507#issuecomment-1791963064

Thanks! I could reproduce this, only when using the Geometry Unsupported("geometry(Point, 4326)") field, after removing it, createMany was available.
generator client {
  provider        = "prisma-client-js"
  previewFeatures = ["postgresqlExtensions"]
}

datasource db {
  provider   = "postgresql"
  url        = "postgresql://postgres:password@localhost:5492/21756"
  extensions = [postgis]
}

model node_locations {
  NodeId      String                               @unique(map: "node_locations_id_idx") @default(dbgenerated("gen_random_uuid()")) @db.Uuid
  ExternalId  String                               @unique(map: "node_locations_ext_id_idx")
  Name        String
  Description String
  Geometry    Unsupported("geometry(Point, 4326)")
  Meta        Json
  Provider    String
  Species     String[]
  Status      String
  LastPing    DateTime

  @@index([Geometry], name: "node_locations_geometry_idx", type: Gist)
}

And I randomly found a Postgis docker file on my computer to try it with docker compose up -d
version: "3.6"
services:
  pgDB:
    image: postgis/postgis:13-3.1-alpine
    restart: always
    ports:
      - "5492:5432"
    volumes:
      - db_data:/var/lib/postgresql/data
    environment:
      POSTGRES_PASSWORD: password
      POSTGRES_DB: geoexample
volumes:
  db_data:

Thanks for the confirmation @Jolg42 Any other work around until it gets addressed?

@rs-nikic So this is actually expected and somehow documented, though we will try to update the docs a bit in the future
https://www.prisma.io/docs/reference/api-reference/prisma-schema-reference#unsupported
https://www.prisma.io/docs/concepts/components/prisma-schema/data-model#unsupported-types
You can make your Unsupported field optional as a workaround like this
Geometry    Unsupported("geometry(Point, 4326)")?

The create* and update* methods would work again.
About the Unsupported field you will have to use raw queries, see:
https://www.prisma.io/docs/concepts/components/prisma-client/raw-database-access#unsupported-types

I will close this since it's expected.
Follow the PostGIS support issue for Prisma Client https://github.com/prisma/prisma/issues/2789


chore(versions): Upgrade minimum python version

(Redo of #1461, which got messed up when I was trying to rebase development)
As of October, 2024, Python 3.8 is ~out of support~ end of life. Upgrading syntax to target Python 3.9. Adds builds for Python 3.13.

@bcantoni Apologies, the other PR got closed when I was trying to fix my branches. This one has the same changes successfully rebased on development. I have noticed that the tests appear to run considerably faster with this branch and the syntax upgraded.

 Was that a scripted change you made? (I.e. how confident are we nothing broke?)

This was a scripted change. In a year when the minimum version is bumped to 3.10, the volume of change should be lower. It is a lot of change, but I have noticed that some areas of the codebase use f-strings, some use .format, and some use the older python2 style %s string substitution. So while it is a lot of change, hopefully it brings more consistency in these small details across the codebase. Personally, I find the f-strings easier to read. f-strings are also faster than .format or %s, but not in a way that will be meaningful with the main purpose being around API calls.

By upgrading the syntax to target 3.9, will the library still work for 3.8 or will this be a breaking change?

This would be a breaking change, but not because of the f-strings. The main syntax change I see that will cause it to not work for 3.8 would be the change from Dict to dict, List to list, Set to set, and Tuple to tuple.
My comment about 3.8 being out of support is that the Python Software Foundation will stop releasing security fixes for 3.8 in October 2024, as it is considered "End of Life." Obviously we could continue to support older versions of python past their end of life, but I think it would hold back adopting newer syntax, features, and quality of life changes.

@jacalata I went through and fixed the f-strings the script missed. Should be good to go now.


Schema registry support for kafka.pubsub component

Describe the feature
New to DAPR and really eager to leverage it at my company. I was surprised that there is no Schema registry support or Avro/protobuf serialization.
Is this on the roadmap?
The way it could work is enabling configuring schema registry URL/Authentication.
You could add metadata to the subscription to add specific overrides (e.g. schema name)
Release Note
RELEASE NOTE: ADD New feature in Dapr.

Seems there was a mention in https://github.com/dapr/components-contrib/issues/8#issuecomment-1191225512 but the issue was never created...

It's not supported or on the roadmap, but we will accept contributions for this!

So updating this issue with findings.
This work is not trivial as the current sarama Kafka Go implementation doesn't support Schema registry...
Based on convo with @berndverst, we cannot switch to the Confluent Kafka client as it's leveraging a C library librdkafka and C bindings are not allowed in Dapr.
So the solution might be to re-implement the kafka.pubsub component as a pluggable component, which doesn't have the same limitations...
Doesn't seem trivial either but maybe easier?

Hi, so getting ready to help on this issue. I'm looking into adding Schema Registry to the kafka-pubsub component by using the https://github.com/riferrei/srclient library. I'm not experienced in Go but it seems to be pure Go and therefore compatible.
Worst case scenario I could just re-implement the SchemaRegistryClient and leverage the underlying github.com/linkedin/goavro/v2 library for serialization/deserialization.
I would focus on avro first, since this is our use case.
In terms of interface, I would:

Add the following settings to the component:

schemaRegistryUrl: Schema Registry URL
schemaRegistryApiKey: Schema Registry credentials API Key
schemaRegitryApiSecret: Schema Registry credentials API Secret
autoRegisterSchemas: Whether the schema will get auto registered if missing upon publishing (default=True)

Add the following metadata to the subscription metadata

valueSchemaType: Schema Type for value (None, AVRO only for now)

So a subscription would look something like this:
{'pubsubname': 'pubsub',
                      'topic': 'deathStarStatus',
                      'route': 'dsstatus',
                      'metadata': {
                          'valueSchemaType': 'AVRO'
                      } }

Assumptions:

This will NOT be compatible with CloudEvents as while There is a Cloud Event Avro version, Schema Registry doesn't support dynamic types, so the data node will not be able to be passed -- apart from a string, negating any of the schema benefits...

Would love some feedback on the above @berndverst and others

@ItalyPaleAle @yaron2 should this issue receive the 1.13 label?
I want to make sure it's included in upcoming version. I assume it will automatically be but just in case...

@ItalyPaleAle @yaron2 should this issue receive the 1.13 label? I want to make sure it's included in upcoming version. I assume it will automatically be but just in case...

Added. We do go through all PRs as part of the final hygiene however, and add labels as needed :)


With dropdown, add a tag that is not in the dropdown but whose value is included in one of the dropdown

I use the dropdown but I also accept tags that are not in the dropdown. But there is a problem if I want to add a tag that is not in the dropdown but whose value open the dropdown.
For example if the dropdown accepts "Hello", "World", "foo", "bar" and if I want to add "hell" as a tag, the dropdown will open and show "Hello" and I won't be able to add "hell" as a tag :

if the dropdown is open, I can only add a tag from the dropdown,
if it is closed, the input has not the focus and I can't add the tag.

Hi @sancaruso,
what version are you using?

Hi,
I am using 0.8.4.
I observe the same problem in the demo http://www.buompris.co/ng2-tag-input/ (for example in the "Tags within an autocomplete component" I am not able to add "Script" as a tag because it is part of "Typescript"...)

Do you have an updated changelog, or can you tell me if I risk of having to change my code if I update the version ? (It happend last time I updated the package, the way of dealing with object-tags had changed and I had to change my code a lot.)

Most should be compatible, with the exception of separatorKeys (now accets strings) and separatorKeyCodes (basically the old separatorKeys).
Obviously I cannot be 100% sure, but the best shot you have is to always update.

Ok, so after updating I confirm the same problem.

Not sure i'm reproducing correctly, but it works on my end (Chrome/OSX)

So I made the test on several browsers/OS with the demo http://www.buompris.co/ng2-tag-input/, but it never worked.
What I exactly do: in the "Tags within an autocomplete component", I tried to add the tag "Script" without adding the tag "Typescript".
I tried on Windows : Chrome, Chromium and Firefox,
on Ubuntu : Chrome, Chromim and Firefox,
on OSX : Chrome and Firefox.

the demo is outdated - try 0.9.10 locally

Well, it works on Ubuntu, but it doesn't work on Windows. (With Chrome/Chromium)

ah I see - never tried on that unfortunately (usually Ubuntu or Mac).

I am sorry! Actually it works: I didn't build the dll so I actually was using the old version...
You can close the issue.

cool :)


[SPIR-V] Some Input or Output variables in mesh shaders and amplification shaders don't have Location assignments

Hi @sparmarNV
It looks like some mesh payload variables that are Input or Output variables to the SPIR-V module do not have a Location assignment. As a result, the recent update to SPIRV-Tools validator causes test failures in DXC.
These are the test failures:
[  FAILED  ] FileTest.MeshShadingNVMeshTriangle
[  FAILED  ] FileTest.MeshShadingNVMeshBuffer
[  FAILED  ] FileTest.MeshShadingNVAmplification
[  FAILED  ] FileTest.MeshShadingNVAmplificationFunCall

In all of these cases, we have an OpVariable with storage class of Input or Output without a Location assignment.
I need go ahead with updating the SPIRV-Tools version inside DXC in order to address several other issues. In the meantime, I will disable running the validator on the above tests. Once this issue is resolved, we can restore running the validator for these tests.

oops. This is the wrong place to file this.

See https://github.com/microsoft/DirectXShaderCompiler/issues/3006 :)