Odd colour pick with dark theme

Flat Decorator/ControlLook + MoonFlat.
When PonpokoDiff starts the text on the left side is white making it impossible to read highlighted lines in yellow.
As soon as the window is resized, the text turns to black (or a dark tone, I can't tell exactly) and the lines are visible again.
Before (when the app starts)

After resizing the window

It's slightly different than #17 but related.

The text becoming black on resize is #17 , I'd say.
The main issue of the illegibility is due to the fixed colour scheme. I just checked with black and white text, the colours look good with that. But your text colour is a 'darkish' light blue, which doesn't work well with the chosen colours for a dark theme.
We'd need a nice algorithm to choose a good red, yellow, green colour for a given text colour. (And of course a fix for #17...)
Another confusing thing is that the empty space ("colorEmpty") appears always light grey. It uses B_PANEL_BACKGROUND_COLOR, which should be dark in a dark theme, but doesn't appear to change...

Due the lack of a proper and official dark mode on Haiku, We'd need a System-wide algorithm to determine if we are on a dark-ish theme by checking the relevant components, I presume. I.e. if the background colour is on a bright tone and the foreground text colour on the dark end of the spectrum then maybe there's a dark theme in place.
At that point PonpokoDiff or any other app might pick the most appropriate built-in palette.
It's a bit simplistic and yours looks the right way to do it, by the way. A bit more complex, though.

There is a perceptual_lightness function that can be used, if it is higher than 127, the color is "light". This is what we use for the desktop icons text color for example (to make sure the color of the text is always visible on top of the background color)

amazing, this is exactly what every app should use to adjust custom colours to match the system theme. Thanks!

Appreciated. I'll use perceptual_lightness instead of the simple if (bkColor.Brightness() > 100).
That doesn't solve using colours that provide enough contrast with the current text colour. We'd need something like:
rgb_color textColor = ui_color(B_DOCUMENT_TEXT_COLOR);
rgb_color lightColor = {163, 255, 180, 255};
rgb_color darkColor;
darkColor = find_contrasting_color(lightColor, textColor);

That is, find_contrasting_color() should find the equivalent of the fixed lightColor that has a nice contrast to the current textColor.
Anyway, here's a suggestion for better contrasting colours for the MoonFlat light blue colour:

WRT using the loopy Invalidate(), I'd rather we find and fix the underlying issue... :)
Until then, dark theming user will have to wave a window in front. :)

Or you can simply pick a light and a dark green, a light and a dark red, and switch between the two color sets depending on perceptual_lightness of the background color.

That's what I do now.
Thanks for the hint for mixing colours etc. I don't want to complicate things too much. I originally just didn't think about people using quite less bright text colours in their dark theme. My dark background colours therefore were a bit too bright. I've now toned them down some more with PR #19.
@nexus6-haiku, think those colours are OK?


Add openxr SDK

I'm attempting to add the recently released Khronos OpenXR SDK to vcpkg via #6339

It looks like the CI build is failing.  The initial failure was commented on by @NancyLi1013 with the logs from the failures, indicating that I hadn't properly configured python3 as a build time dependency.  However, after fixing that they still seem to be failing, but I don't see any way to look at the failure, short of someone linking a zip file again.  The CI link seems to be behind an authentication wall I don't have access to.
My local build for windows-x64-static and windows-x64 work, as does an x64-linux build I'm doing against a docker image based on the attached dockerfile.
Is there any way to get more visibility into the CI failures?

This has been done, I don't know why it's still open.


Modify version of UG

Codecov Report

Merging #277 (ab72e4e) into master (eca590e) will not change coverage.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##             master     #277   +/-   ##
=========================================
  Coverage     69.34%   69.34%
  Complexity      727      727
=========================================
  Files           112      112
  Lines          2525     2525
  Branches        308      308
=========================================
  Hits           1751     1751
  Misses          666      666
  Partials        108      108

Impacted Files
Coverage Δ

src/main/java/seedu/address/MainApp.java
0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


template project wont build html

The default project created by flambe new <project> can not be built to html. Flash appears to  work fine though.
Versions of stuff:
DeLorean:test-flambe jakob$ haxe -version
3.2.0
DeLorean:test-flambe jakob$ flambe --version
4.1.0

What happens:
DeLorean:haxe jakob$ flambe new test-flambe
New Flambe project created in /Users/jakob/GameDevelopment/Experiments/haxe/test-flambe
DeLorean:haxe jakob$ cd test-flambe/
DeLorean:test-flambe jakob$ flambe build html
Building: build/web
haxe -main urgame.Main -lib flambe -cp src -dce full --no-traces -D html -D js-es5 -D js-flatten --macro flambe.platform.ManifestBuilder.use("build/web/assets") -js build/.cache/html.unminified.js
/usr/local/lib/haxelib/flambe/4,1,0/flambe/platform/html/CanvasTextureRoot.hx:44: characters 28-32 : Array<Int> should be haxe.io.BytesData
/usr/local/lib/haxelib/flambe/4,1,0/flambe/platform/html/CanvasTextureRoot.hx:44: characters 28-32 : Array<Int> should be js.html.ArrayBuffer
/usr/local/lib/haxelib/flambe/4,1,0/flambe/platform/html/CanvasTextureRoot.hx:44: characters 28-32 : For function argument 'b'
/usr/local/lib/haxelib/flambe/4,1,0/flambe/platform/html/CanvasTextureRoot.hx:39: lines 39-45 : Missing return haxe.io.Bytes
/usr/local/lib/haxelib/flambe/4,1,0/flambe/platform/html/HtmlAssetPackLoader.hx:364: characters 16-38 : String should be js.html.XMLHttpRequestResponseType
/usr/local/lib/haxelib/flambe/4,1,0/flambe/platform/html/HtmlAssetPackLoader.hx:367: characters 12-37 : String should be js.html.XMLHttpRequestResponseType
/usr/local/lib/haxelib/flambe/4,1,0/flambe/platform/html/HtmlAssetPackLoader.hx:368: characters 16-42 : String should be js.html.XMLHttpRequestResponseType
/usr/local/lib/haxelib/flambe/4,1,0/flambe/platform/html/HtmlAssetPackLoader.hx:205: characters 12-43 : String should be js.html.XMLHttpRequestResponseType
/usr/local/lib/haxelib/flambe/4,1,0/flambe/platform/html/HtmlExternal.hx:37: characters 42-48 : js.html.Window should be haxe.Function
/usr/local/lib/haxelib/flambe/4,1,0/flambe/platform/html/HtmlExternal.hx:37: characters 42-48 : For function argument 'func'
/usr/local/lib/haxelib/flambe/4,1,0/flambe/platform/html/HtmlExternal.hx:25: lines 25-38 : Missing return Dynamic
``

Same as https://github.com/aduros/flambe/issues/346
tl;dr you use the latest Haxe (3.2) which is not supported officially, only using on the git-version of Flambe. Apply this patch and the project should compile.

Thanks. It works.
:tada: :balloon:
To who ever runs into this issue next, it's done by:
git clone https://github.com/aduros/flambe.git <some-path>
haxelib dev flambe <some-path>/src/


Add ClipTools

NOTE: A similar PR may already be submitted! Please search among the Pull request before creating one.
Types of Changes
What types of changes does your PR introduce? Put an x in all boxes that apply

[x] New addition to list
[ ] Fixing bug in existing item in list
[ ] Removing item from list
[ ] Changing structure (organization) of list

Proposed Changes
I've added to the list ClipTools, is a status menu application that gives you access to a variety of simple clipboard utilities from the right side of the menu bar.
In addition to being able to access older clipboard contents, you can save clips, use powerful Smart Clips, paste the date or time in any format you wish, paste sample Lorem Ipsum text, type calculations and paste the result, or even calculate the result of a formula currently selected in text.
PR Checklist
Put an x in the boxes once you've completed each step. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING guide
[x] I have explained why this PR is important
[x] I have added necessary documentation (if appropriate)

Further Comments
N/A

Thanks for adding
Am I a contributor now ?


Numpy integration and Comphys integration

In task 3, there is a task:
Compare the numpy integration functions to the ones found in compphys
So what really are we comparing? Can you elaborate?

I hope this is clear to everyone by now.


intial RPO loss

from the paper: https://arxiv.org/pdf/2404.19733

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@kashif couldn't get a clean stack trace on multi-gpu/process, but doesn't seem to work with deepspeed:
  File "/root/miniconda3/envs/py3.10/lib/python3.10/site-packages/transformers/trainer.py", line 1885, in train
    return inner_training_loop(
  Fil  "/root/miniconda3/envs/py3.10/lib/python3.10/site-packages/transformers/trainer.py", line 2216, in _inner_training_loop
    tr_loss_step = self.training_step(model, inputs)
    self.optimizer.step()
  File "/root/miniconda3/envs/py3.10/lib/python3.10/site-packages/transformers/trainer.py", line 3250, in training_step
    self.accelerator.backward(loss)
  Fil  "/root/miniconda3/envs/py3.10/lib/python3.10/site-packages/deepspeed/runtime/zero/stage_1_and_2.py", line 1846, in step
  File "/root/miniconda3/envs/py3.10/lib/python3.10/site-packages/accelerate/accelerator.py", line 2117, in backward
    self.deepspeed_engine_wrapped.backward(loss, **kwargs)
  Fil  "/root/miniconda3/envs/py3.10/lib/python3.10/site-packages/accelerate/utils/deepspeed.py", line 175, in backward
    self.engine.step()
    r turn torch.norm(torch.stack(norm_groups), p=norm_type)                                                                                [342/1890]
self.optimizer.step()  File "/root/miniconda3/envs/py3.10/lib/python3.10/site-packages/deepspeed/runtime/engine.py", line 2169, in step
    self._take_model_step(lr_kwargs)
  File "/root/miniconda3/envs/py3.10/lib/python3.10/site-packages/torch/functional.py", line 1611, in norm

  File "/root/miniconda3/envs/py3.10/lib/python3.10/site-packages/deepspeed/runtime/zero/stage_1_and_2.py", line 1846, in step
  File "/root/miniconda3/envs/py3.10/lib/python3.10/site-packages/deepspeed/runtime/engine.py", line 2075, in _take_model_step
    self.optimizer.step()
         root/miniconda3/envs/py3.10/lib/python3.10/site-packages/deepspeed/runtime/zero/stage_1_and_2.py", line 1846, in step
    scaled_global_grad_norm = self.scaled_global_norm()
          oot/miniconda3/envs/py3.10/lib/python3.10/site-packages/deepspeed/runtime/zero/stage_1_and_2.py", line 1793, in scaled_global_norm
    return torch.norm(torch.stack(norm_groups), p=norm_type)
    scaled_global_grad_norm = self.scaled_global_norm()  File "/root/miniconda3/envs/py3.10/lib/python3.10/site-packages/torch/functional.py", line 1611, in norm
    return torch.linalg.vector_norm(input, _p, _dim, keepdim, dtype=dtype)

  File "/root/miniconda3/envs/py3.10/lib/python3.10/site-packages/deepspeed/runtime/zero/stage_1_and_2.py", line 1793, in scaled_global_norm
RuntimeError: linalg.vector_norm: Expected a floating point or complex tensor as input. Got Long
    return torch.linalg.vector_norm(input, _p, _dim, keepdim, dtype=dtype)
RuntimeError: linalg.vector_norm: Expected a floating point or complex tensor as input. Got Long
    scaled_global_grad_norm = self.scaled_global_norm()
  File "/root/miniconda3/envs/py3.10/lib/python3.10/site-packages/deepspeed/runtime/zero/stage_1_and_2.py", line 1793, in scaled_global_norm                                                        return torch.norm(torch.stack(norm_groups), p=norm_type)
  File "/root/miniconda3/envs/py3.10/lib/python3.10/site-packages/torch/functional.py", line 1611, in norm
    return torch.norm(torch.stack(norm_groups), p=norm_type)
  File "/root/miniconda3/envs/py3.10/lib/python3.10/site-packages/torch/functional.py", line 1611, in norm
    return torch.linalg.vector_norm(input, _p, _dim, keepdim, dtype=dtype)
RuntimeError: linalg.vector_norm: Expected a floating point or complex tensor as input. Got Long
    return torch.linalg.vector_norm(input, _p, _dim, keepdim, dtype=dtype)
RuntimeError: linalg.vector_norm: Expected a floating point or complex tensor as input. Got Long

thanks @winglian  checking!

@winglian are you still encountering this issue?


CF deployment bump 12.33

Description

This is a follow up on https://github.com/SUSE/kubecf/pull/409. Major changes include:

Moving buildpack file paths out of values yaml.
Bumping CATs to 12.33
Updating docs related to bump

Motivation and Context

It includes regular maintenance updates and also addresses CVEs.
How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code has security implications.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Build + Deploy + Smokes on Diego and Eirini are passing.
CATs(on diego) are passing: https://concourse.suse.dev/teams/main/pipelines/kubecf/jobs/cf-acceptance-tests-diego/builds/4


Delete ctrl if associated active env is deleted

Codecov Report

Merging #12148 (59ef208) into main (55ca7ac) will increase coverage by 0%.
The diff coverage is 65%.

:exclamation: Current head 59ef208 differs from pull request most recent head 737ae42. Consider uploading reports for the commit 737ae42 to get more accurate results

@@          Coverage Diff           @@
##            main   #12148   +/-   ##
======================================
  Coverage     70%      70%
======================================
  Files        497      497
  Lines      31276    31292   +16
  Branches    4907     4910    +3
======================================
+ Hits       21946    22020   +74
+ Misses      7451     7388   -63
- Partials    1879     1884    +5

Impacted Files
Coverage Δ

...rc/notebooks/controllers/controllerRegistration.ts
83% <65%> (-4%)
:arrow_down:

...c/platform/common/process/pythonDaemonPool.node.ts
78% <0%> (-2%)
:arrow_down:

src/platform/api/pythonApi.ts
70% <0%> (-1%)
:arrow_down:

src/notebooks/debugger/debuggerVariables.ts
69% <0%> (+<1%)
:arrow_up:

...c/platform/interpreter/interpreterPackages.node.ts
74% <0%> (+<1%)
:arrow_up:

...lPythonAndRelatedNonPythonKernelSpecFinder.node.ts
71% <0%> (+<1%)
:arrow_up:

...rnels/raw/finder/localKernelSpecFinderBase.node.ts
80% <0%> (+1%)
:arrow_up:

src/kernels/helpers.ts
73% <0%> (+1%)
:arrow_up:

...tive-window/debugger/jupyter/kernelDebugAdapter.ts
62% <0%> (+1%)
:arrow_up:

src/platform/common/application/extensions.node.ts
86% <0%> (+2%)
:arrow_up:

... and 6 more


Add in-repo-addon for ember create-website command.

Created this command as an in-repo-addon for initial prototyping, and easier extraction into a standalone addon later (we still need to make the template a bit more generic to be useful for all ember-cli users).
The command can be invoked as:
ember create-website --domain=aptible-staging.com --subdomain=dashboard-rwjblue

To build and deploy to this newly created bucket, use the following command:
AWS_BUCKET=dashboard-rwjblue.aptible-staging.com ember deploy --environment staging

Fixes #365.

/cc @fancyremarker for review

This is awesome! Thanks, merging.


VSCode extension

Hey :)
super idea :) I'd love to have this markdown preview in vscode, do you think you would do it ? (or be open to pull requests ?)
Cheerz :)

Hey,
Glad to hear that you loved the tool. Yeah sure I am open to pull requests for vscode extension and thanks for your contribution.


No matching key exchange

Shell opener command was unsuccessful:
Unable to negotiate with <IP_ADDRESS> port 22: no matching key exchange method found. Their offer: diffie-hellman-group-exchange-sha1,diffie-hellman-group14-sha1,diffie-hellman-group1-sha1

In other SSH clients it works fine.

Hmm, xpipe just calls your locally installed ssh client. If you click on edit for the SSH connection, on the bottom left there should be an insights button that tells you what command is getting executed.
Is there any kind of special configuration you use in your user SSH config?

Hmm, xpipe just calls your locally installed ssh client. If you click on edit for the SSH connection, on the bottom left there should be an insights button that tells you what command is getting executed.
Is there any kind of special configuration you use in your user SSH config?

ssh<EMAIL_ADDRESS>-p 22 -oStrictHostKeyChecking=accept-new -oRemoteCommand=none -oNoHostAuthenticationForLocalhost=yes -T

Does that run if you paste it into a terminal?

You should also be able to fix this by creating a custom connection that supports one of these key exchanges like this

You should also be able to fix this by creating a custom connection that supports one of these key exchanges like this

Thanks, this worked. And yeah I was also getting the same problem with the Win Terminal app. Also added HostKeyAlgorithms = +ssh-rsa to work.


Add nominee: The Global Centre for Risk and Innovation (GCRI)

Automatic addition of a new nominee submitted through the online form available at https://digitalpublicgoods.net/submission

Made a follow up with contact person for GCRI project to provide additional information on the following:

SDGs: The project claims relevance to SDGS 1 through 17.  Currently, SDG evidenceText and evidenceURL(optional) are missing for all the above SDGs.
Project category: The project indicates they support all categories namely: software, ai model, data, standard & content. For more complete answers, explicit license URLs are required especially for software.

@lacabra will update on this branch as soon as we get a valid response from the project. Thanks.

Digital solutions that are no longer active, or do not provide the required documentation needed for review, will be removed from the DPG Registry as per the archiving policy.


model:show does not honor composer autoload

Laravel Version: 9.30.1
PHP Version: 8.1.9
Database Driver & Version: pgsql 14

Description:
First off, thanks @jessarcher for your work on the model show command! It's very convenient and I expect to get a lot of mileage from it.
model:show seems to only search for models in the App\ directory (unless there's a config variable I missed) instead of honoring composer's autoload. I'm not sure if this is the desired behaviour or not (for instance, Tinker will auto-alias a model found in autoload), but my application is organized by (sub)domains and it quickly becomes inconvenient to type out the full namespace that's several levels deep.
If this is not a bug, would you please consider honoring autoload (like Tinker) for convenience?
Thanks!
Steps To Reproduce:
Try to list the details for any model not in App\ but available through composer autoloading.
artisan model:show <Model>

Hi there. Right now this is indeed limited to the App namespace. Feel free to attempt a PR to change that.

It's worth noting here that you may pass through the entire namespace, for example, artisan model:show Laravel\\Cashier\\Subscription


HSEARCH-2568 Do no check that the index exists with the NONE schema management strategy

https://hibernate.atlassian.net/browse/HSEARCH-2568
To be discussed, but it feels a bit strange that the NONE strategy does in fact do something (check if the index exists). This PR reverts the NONE strategy to the pre-HSEARCH-2456 behavior of not doing anything, not even checking whether the index exists.
I'm not sure we should backport this to 5.6, though, since it's a behavior change.

What is the value in "not checking" at all - even not for index existance?
It's not that Hibernate ORM can start when there's no database..

What is the value in "not checking" at all - even not for index existance?

The only use case I see is mentioned in the ticket: a user wanting to create the index himself, but after HS started. Strange, but then I guess using NONE is also quite strange to begin with...

It's not that Hibernate ORM can start when there's no database..
True, but that's not particularly because of HBM2DDL. So if we want to mirror that behavior, we shouldn't try to check the index existence inside our strategy.

ok, yes I had merged it after reading Alex Laptseu's comments, but .. I agree it's strange.


The 'tiny_tds' native gem requires installed build tools

I am trying to get SQL 2012 running on a 2012 R2 Core machine. SQL get's installed fine however it exits when trying to install 'tiny_tds' at the end of the server recipe. It is giving the following error:
The 'tiny_tds' native gem requires installed build tools.'

The docs say the following about this.

The TinyTDS gem is also installed as this is used in the SQL Server providers for the database and database_user resources (see the database cookbook for more details).

I'm not really interested in using these providers so I'm wondering if there is a reason why it's being handled in this cookbook as opposed to the database cookbook?

Having the same issue on a 2008 server with 2008 while installing the client tools only.

I've removed the gem in master

Would you be able to push this to the supermarket?


Add micro benchmarks to test suite

I recommend using pytest-benchmark for simple benchmarks. This was already useful for testing potential performance regressions in recent PRs.
This is not meant to replace the existing benchmarking tool for file format performance (examples/filetimes.py).

Sounds good!  We could consider airspeed velocity as well/in addition.


Timeout listing keys if "tokenize" filter used and key has fewer than n tokens

Choice was either

return the key untransformed
do not return the key

I went with 2

This looks fixed, running against 1.0+ it is no longer an issue


In Batch accessibility maps, Prepare and Calculate button should be 2 different one

Having the Prepare button just change it's name to Calculate is confusing and it's hard to see that something has happen. They should be 2 different button, (That are maybe grayed when not available)

This is somewhat related to #477. There should be only one button that does the whole process.

Yeah, that could work.
The 2 steps are in fact
1-upload a file
2-then select fields and settings.

oK, we really need to revamp this UI. As the blue upload button should be uploading, not just analyse the file locally :/
And it's batch processing, we can wait for the full upload before doing the calculation, we are not in a hurry there.


Add basic auth capabilities

Added basic auth flags '-B' and '--web-auth' to the command line arguments
Enabled basic-auth on the flask application for basic security to the UI

Hi @ioneyed ! Could you update your PR according to cgoldbergs suggestions & resolve the merge conflicts? Then I'ld be happy to merge this.

Closing due to inactivity.


bau-cof-r3-config-bugs-wave-4

this PR includes bug fixes for: FS-3163, FS-3165, FS-3167, FS-3170, FS-3173, FS-3171, FS-3172, FS-3176, FS-3178

@gio-karunakaran Just fix the unit tests before you merge.


api: Reintroduce getBug() method

Please reintroduce method that was removed by a2200c2444bf8827850056245f4219b17734643f.
Since then the method was listed in README. IMHO it would be great to have such API call.

Thank you for explanation, I understand. I can prepare PR to update README, If you wish.

Yes that would be good, it doesn't serve anyone to have bad docs!

Thank you, I really like this package! :+1:  I look for this kind of tool for long time...


Returns completed tasks (not null) from BlazorComponent.OnInitAsync

BlazorComponent methods OnInitAsync and OnParametersSetAsync should return Task.Completed and not null.
For example following code will fail with NullReferenceException. And there is FxCop rule to run base methods.
    protected override async Task OnInitAsync()
    {
        await base.OnInitAsync();
        forecasts = await Http.GetJsonAsync<WeatherForecast[]>("sample-data/weather.json");
    }

This PR has following changes:

BlazorComponent methods OnInitAsync and OnParametersSetAsync return Task.Completed.
Replaces ContinueWith with async/await in SetParameters.
Add few unit tests for BlazorComponent to cover cases above.

Oh, I just noticed, there is similar PR #1619

Thanks for this! It's now moved to https://github.com/aspnet/AspNetCore/pull/4790, where it will be merged shortly.


Getting error "Must use import to load ES Module"

Getting this error while trying it with solid js, vite and unocss.
[ERR_REQUIRE_ESM]: Must use import to load ES Module: /home/hp/myWorkspace/solid-capacitor/client/node_modules/unocss-preset-daisy/index.js
require() of ES modules is not supported.

Since node will not run esm directly, you should add "type": "module" in your package.json.

Please try the latest version if you are still encountering issues.
This is a good read on (pure) ESM package: https://gist.github.com/sindresorhus/a39789f98801d908bbc7ff3ecc99d99c


Use admonition and reformat some content

As a pull request owner and reviewers, I checked that:

[X] Appropriate documentation exists and/or is up-to-date:

[X] Content of the docs/source is reflecting the changes

/approve


Material properties exported as integers instead of floats/numbers

Describe the bug
When importing a gltf file created with Blender using a well-known C++ library several material properties were not handled correctly. For example, a metallic factor of 0 was not read at all. This seems to be a result of the exporter writing 0 (and 1) without a decimal point. Without a decimal point, the values are then interpreted as integers, which appears to be a violation of the gltf standard (see for example spec).
To Reproduce
Simply export the default cube with the principled BSDF and a metallic factor of 0.
Expected behavior
number types should be exported using a decimal point.
Additional context
This problem might occur in more places than just the materials.

Here's an example file, created as you suggest:
IntTest.gltf.zip

My understanding of the JSON schema is that the value must be of the correct type, but the decimal is neither required for floats nor prohibited for integers. See: https://json-schema.org/understanding-json-schema/reference/numeric.html. This model passes the official glTF validator, as well: http://github.khronos.org/glTF-Validator/. The two 'info'-level issues it reports are just because I didn't remove unused UVs and nodes in the scene, not related here.
If tinygltf cannot load a model that passes the validation suite and loads correctly in other viewers (like gltf-viewer.donmccurdy.com/) then I would recommend reporting an issue on that repository.
/cc @lexaknyazev

Updated the pending PR: https://github.com/KhronosGroup/glTF/pull/1574.

@lexaknyazev thanks! So it sounds like we should report this as a bug on tinygltf, if real values without a fractional part "should be universally supported"?

I have a project that uses Tinygltf, and I'm having trouble reproducing this issue.  It seems to handle integers in these fields OK.  I don't know if it's the same version etc.
@TimZoet Many JSON writers will skip the fractional part of it's zero, so your glTF reader does need to tolerate that possibility.  This is not a bug in the Blender exporter, so I'll close this issue, but discussion of spec clarifications will continue in KhronosGroup/glTF#1574.


How to create model with dynamic json fieldname

Hello everybody.
I'm new in LoganSquare, and i'm faced with next problem.
I have next json structure:
{
"status": "ok", -> String
"meta": { -> Model with integer value
"count": 1
},
"data": { -> Model with child JSON object
"1466466": {       <------ Dynamic name
"field1": "value1",
"field2": value2,
"field3": value3
}
}
}, where "1466466" is filed with dynamic key. How I can parse it?
Thanks.

My initial reaction to this situation would be to map your data field to a Map<String, Fields> object, where Fields is a @JsonObject class containing your field1 to field3 stuff. Then, you'd expose some more convenient accessors to those fields based on the keySet() of that Map:
@JsonObject
class Outer {
    @JsonField(name = "status")
    String status;

    @JsonField(name = "meta")
    Meta meta;

    @JsonField(name = "data")
    Map<String, Fields> data;

    // ...

    public String getField1() {
        // Of course, you probably want to check for "empty" first, this is just the basic idea
        return data.keySet().get(0).getField1();
    }
}

@JsonObject
class Fields {
    @JsonField(name = "field1")
    String field1;

    // ...
}

Thanks a lot!
A found the similar solution with TreeMap and this works.


samples: sockets: echo_async_select: Add error check

This patch adds error checking routine for socket APIs.
Fixes: #13852
Coverity-CID: 190966
Signed-off-by: Tedd Ho-Jeong An<EMAIL_ADDRESS>

Codecov Report

Merging #14130 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #14130   +/-   ##
=======================================
  Coverage   52.68%   52.68%
=======================================
  Files         307      307
  Lines       45473    45473
  Branches    10530    10530
=======================================
  Hits        23956    23956
  Misses      16640    16640
  Partials     4877     4877

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 f78288b...e45483d. Read the comment docs.


Question: Technical processes in which air is used

Hello
Recently, a friend built a benzene production plant and I decided to calculate its parameters using your script. It turned out this:
- m: industrial coke oven
  tier: HV
  I:
    oak wood: 4
    nitrogen: 250
  O:
    charcoal: 5
    wood tar: 375
  eut: 96
  dur: 3.2
  number: 1
- m: fluid extractor
  tier: MV
  I:
    charcoal: 1
  O:
    ashes: 0.1
    wood tar: 100
  eut: 16
  dur: 1.5
- m: distillery
  tier: MV
  I:
    wood tar: 200
  O:
    benzene: 80
  eut: 64
  dur: 0.8
- m: centrifuge
  tier: MV
  I:
    air: 10000
  O:
    nitrogen: 3900
    oxygen: 1000
  eut: 7
  dur: 80
- m: compressor
  tier: MV
  I:
  O:
    air: 2000
  eut: 2
  dur: 15

The question arose why air was supplied from the outside if its production was prescribed in the process. Did I describe the process incorrectly or is it a feature of the script?

@Eldrinn-Elantey This is a feature, it is meant to reduce the complexity of the problem so it can be solved as a "linear program". While this may make some steps unnatural (you would pull cells from "source" instead of "air"), the recycling still ultimately works and the summary node is still accurate (0 net air).
For technical details - I try and avoid cycles in the graph, which means I run an algorithm that removes "back edges" and forces the graph into being a "directed acyclic graph" before I run anything else. You can see the method for this in src/graph/_preprocessing:95 (removeBackEdges).
I suppose I should make the documentation clearer by explaining a process fully, so I will leave this ticket open until I do that.


Ignore invalid selectors, see #179

This is a partial fix for #179. It simply adds a check on the xpath query return value before attempting to iterate over the result set. If the xpath has failed, it just issues a continue to go to the next iteration.
This fixes the second of the two errors mentioned in the ticket, so it's effectively treating the symptom rather than the cause, but it's better than nothing! I don't know how it would be best to try to prevent the first error - somehow validate the CSS selector fragment before trying to pass it to the xpath converter?

Hi @Synchro, thanks for the PR!
Could you please add a regression test (i.e., a minimal unit test that fails without the fix and that passes with the fix)?
Could you also please prefix the commit message subject with [BUGFIX] (and, if you have more than one commit, squash those commits)? Thanks!

As for #179: Could you please add "Relates #179" to the commit message description part? Then #179 will  be autolinked (without closing it).

Done all those things, rebased, pushed. The test case was a little tricky - needed to create an error handler to trap the warnings. I've arranged it so that it eats the first error type, but lets the second go through. That can be removed when you need a test case for the other error.

Hi @Synchro, I've polished your PR a bit in #194 and just merged it. Thanks for contributing!


HTML comment spacing disrupts rendering

I wanted to write a small tutorial for the team here, and added some HTML comments.
<!DOCTYPE html>

    <title>T-Shirt</title>
    <meta charset="UTF-8">
    <meta name="description" content="">
    <meta name="keywords" content="">
    <link rel="stylesheet" href="./t-shirt.css">

        All the narrow columns have editable content; simply click on an item to edit it.

The T-shirt columns accept either upper- or lower-case input, limited to s, m, l, xl, and xxl.

The story points column accepts any number, but rounds up to the next value that would be appropriate for planning poker.  Anything larger than 100 becomes 100.

                Story	Effort	Value	Points

                <input data-type="work" size="4" value="{{item.work}}">
                <input data-type="value" size="4" value="{{item.value}}">
                <input data-type="points" size="4" value="{{item.points}}">

                Scores

This works as expected.

Then I tried it with a space separating the angle brackets for the "see tshirt.js" comment:
<!DOCTYPE html>

    <title>T-Shirt</title>
    <meta charset="UTF-8">
    <meta name="description" content="">
    <meta name="keywords" content="">
    <link rel="stylesheet" href="./t-shirt.css">

        All the narrow columns have editable content; simply click on an item to edit it.

The T-shirt columns accept either upper- or lower-case input, limited to s, m, l, xl, and xxl.

The story points column accepts any number, but rounds up to the next value that would be appropriate for planning poker.  Anything larger than 100 becomes 100.

                Story	Effort	Value	Points

                <input data-type="work" size="4" value="{{item.work}}">
                <input data-type="value" size="4" value="{{item.value}}">
                <input data-type="points" size="4" value="{{item.points}}">

                Scores

The results were unexpected:

I don't recall any explicit advice in the docs about being careful with HTML comments, but I certainly could have missed something.

Let me follow up by saying that production code should never go out with casually-visible comments in it, so a proper dev stack would make sure they're removed--but just in case someone forgets....

Hey, you're right, this is a bug in Moon. It has already been fixed in v0.10.0, which will be released soon, I'm aiming for Friday 😄
I just have to update the docs, as some of the syntax and scoping has changed. For example, all directives will now be rendered as if they were in a mustache:

Condition is {{condition}}

As you can see, the directive uses a syntax as if it were inside of a mustache, and a normal attribute uses mustache syntax. Along with this, mustaches will be able to have a valid Javascript expression. (That's right! Moon has a built in Javascript tokenizer that can analyze dependencies, in ~100 bytes!)
You will be able to do something like:

{{num * 10}}

By the way, Moon will remove all comments that are parsed, so if you open Developer Tools, you'll see the comment be removed.
Demo link


tripleee: Blacklist abella\Wmayfair

tripleee requests the blacklist of the following keyword(s):

abella\Wmayfair - MS search

abella\Wmayfair has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Here's a better search, 70 hits, all TP.
https://metasmoke.erwaysoftware.com/search?utf8=✓&title=&body_is_regex=1&body=abella[^A-Za-z0-9_]mayfair&username=&why=&site=&feedback=&reason=&user_rep_direction=>%3D&user_reputation=0&commit=Search


Driver can't be loaded on Windows 11 Beta ring

It seems all of the tdp tools and .bat's fail to run properly on the windows 11 beta ring. The error appears on start and shows the driver fails to load.

Did you make sure core isolation is off?
Get Outlook for iOShttps://aka.ms/o0ukef

From: bschmidy10 @.>
Sent: Saturday, May 21, 2022 10:43:24 PM
To: project-sbc/Power-Control-Panel @.>
Cc: Subscribed @.***>
Subject: [project-sbc/Power-Control-Panel] Driver can't be loaded on Windows 11 Beta ring (Issue #26)
It seems all of the tdp tools and .bat's fail to run properly on the windows 11 beta ring. The error appears on start and shows the driver fails to load.
—
Reply to this email directly, view it on GitHubhttps://github.com/project-sbc/Power-Control-Panel/issues/26, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASZ4DJJZNN27KI4QDZY4RYLVLGNMZANCNFSM5WSV532Q.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

You are the 3rd or 4th person to mention this, I’ll dive into it to find out
Get Outlook for iOShttps://aka.ms/o0ukef

From: Chris Moore @.>
Sent: Sunday, May 22, 2022 7:56:23 AM
To: project-sbc/Power-Control-Panel @.>; project-sbc/Power-Control-Panel @.>
Cc: Subscribed @.>
Subject: Re: [project-sbc/Power-Control-Panel] Driver can't be loaded on Windows 11 Beta ring (Issue #26)
Did you make sure core isolation is off?
Get Outlook for iOShttps://aka.ms/o0ukef

From: bschmidy10 @.>
Sent: Saturday, May 21, 2022 10:43:24 PM
To: project-sbc/Power-Control-Panel @.>
Cc: Subscribed @.***>
Subject: [project-sbc/Power-Control-Panel] Driver can't be loaded on Windows 11 Beta ring (Issue #26)
It seems all of the tdp tools and .bat's fail to run properly on the windows 11 beta ring. The error appears on start and shows the driver fails to load.
—
Reply to this email directly, view it on GitHubhttps://github.com/project-sbc/Power-Control-Panel/issues/26, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASZ4DJJZNN27KI4QDZY4RYLVLGNMZANCNFSM5WSV532Q.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

It has been confirmed in a couple discord servers for tools like this. Core
isolation is turned off, correct!
Hyatice says "wmic" is removed in the beta build which he thinks is causing
the problem.
Brendan Schmid
On Sun, May 22, 2022, 7:44 AM project-sbc @.***> wrote:

You are the 3rd or 4th person to mention this, I’ll dive into it to find
out
Get Outlook for iOShttps://aka.ms/o0ukef

From: Chris Moore @.>
Sent: Sunday, May 22, 2022 7:56:23 AM
To: project-sbc/Power-Control-Panel @.>;
project-sbc/Power-Control-Panel @.>
Cc: Subscribed @.>
Subject: Re: [project-sbc/Power-Control-Panel] Driver can't be loaded on
Windows 11 Beta ring (Issue #26)
Did you make sure core isolation is off?
Get Outlook for iOShttps://aka.ms/o0ukef

From: bschmidy10 @.>
Sent: Saturday, May 21, 2022 10:43:24 PM
To: project-sbc/Power-Control-Panel @.>
Cc: Subscribed @.***>
Subject: [project-sbc/Power-Control-Panel] Driver can't be loaded on
Windows 11 Beta ring (Issue #26)
It seems all of the tdp tools and .bat's fail to run properly on the
windows 11 beta ring. The error appears on start and shows the driver fails
to load.
—
Reply to this email directly, view it on GitHub<
https://github.com/project-sbc/Power-Control-Panel/issues/26>, or
unsubscribe<
https://github.com/notifications/unsubscribe-auth/ASZ4DJJZNN27KI4QDZY4RYLVLGNMZANCNFSM5WSV532Q

.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
https://github.com/project-sbc/Power-Control-Panel/issues/26#issuecomment-1133888384,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQ6U4NSNLX5KP7VKQIUPA3VLITZDANCNFSM5WSV532Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

Can you check the debug or log file for rw.exe? Download a fresh copy of the portable version on the website and try to run iy
Get Outlook for iOShttps://aka.ms/o0ukef

From: bschmidy10 @.>
Sent: Sunday, May 22, 2022 10:49:31 AM
To: project-sbc/Power-Control-Panel @.>
Cc: project-sbc @.>; Comment @.>
Subject: Re: [project-sbc/Power-Control-Panel] Driver can't be loaded on Windows 11 Beta ring (Issue #26)
It has been confirmed in a couple discord servers for tools like this. Core
isolation is turned off, correct!
Hyatice says "wmic" is removed in the beta build which he thinks is causing
the problem.
Brendan Schmid
On Sun, May 22, 2022, 7:44 AM project-sbc @.***> wrote:

You are the 3rd or 4th person to mention this, I’ll dive into it to find
out
Get Outlook for iOShttps://aka.ms/o0ukef

From: Chris Moore @.>
Sent: Sunday, May 22, 2022 7:56:23 AM
To: project-sbc/Power-Control-Panel @.>;
project-sbc/Power-Control-Panel @.>
Cc: Subscribed @.>
Subject: Re: [project-sbc/Power-Control-Panel] Driver can't be loaded on
Windows 11 Beta ring (Issue #26)
Did you make sure core isolation is off?
Get Outlook for iOShttps://aka.ms/o0ukef

From: bschmidy10 @.>
Sent: Saturday, May 21, 2022 10:43:24 PM
To: project-sbc/Power-Control-Panel @.>
Cc: Subscribed @.***>
Subject: [project-sbc/Power-Control-Panel] Driver can't be loaded on
Windows 11 Beta ring (Issue #26)
It seems all of the tdp tools and .bat's fail to run properly on the
windows 11 beta ring. The error appears on start and shows the driver fails
to load.
—
Reply to this email directly, view it on GitHub<
https://github.com/project-sbc/Power-Control-Panel/issues/26>, or
unsubscribe<
https://github.com/notifications/unsubscribe-auth/ASZ4DJJZNN27KI4QDZY4RYLVLGNMZANCNFSM5WSV532Q

.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
https://github.com/project-sbc/Power-Control-Panel/issues/26#issuecomment-1133888384,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQ6U4NSNLX5KP7VKQIUPA3VLITZDANCNFSM5WSV532Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

—
Reply to this email directly, view it on GitHubhttps://github.com/project-sbc/Power-Control-Panel/issues/26#issuecomment-1133912538, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASZ4DJLFQ6ASNMPQRW3Z73TVLJCPXANCNFSM5WSV532Q.
You are receiving this because you commented.Message ID: @.***>

Roll back to the latest stable released build. Beta builds cause rw.exe to crash. I alerted windows to this issue.

Can't use it on Win 11 22h2

Yes that’s a known issue, Microsoft disabled the driver for rw. I’m not sure if that will carry forward into the public release
Get Outlook for iOShttps://aka.ms/o0ukef

From: plum @.>
Sent: Tuesday, June 28, 2022 11:23:32 PM
To: project-sbc/Power-Control-Panel @.>
Cc: project-sbc @.>; State change @.>
Subject: Re: [project-sbc/Power-Control-Panel] Driver can't be loaded on Windows 11 Beta ring (Issue #26)
Can't use it on Win 11 22h2
—
Reply to this email directly, view it on GitHubhttps://github.com/project-sbc/Power-Control-Panel/issues/26#issuecomment-1169492180, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASZ4DJICUJ7YYCER5UXERWTVRO6TJANCNFSM5WSV532Q.
You are receiving this because you modified the open/close state.Message ID: @.***>

It seems its already in stable release.
I'm on Win11 22H2 RTM version.

I might have something, can you message me on discord?
Get Outlook for iOShttps://aka.ms/o0ukef

From: plum @.>
Sent: Wednesday, June 29, 2022 11:28:51 AM
To: project-sbc/Power-Control-Panel @.>
Cc: project-sbc @.>; State change @.>
Subject: Re: [project-sbc/Power-Control-Panel] Driver can't be loaded on Windows 11 Beta ring (Issue #26)
It seems its already in stable release.
I'm on Win11 22H2 RTM version.
—
Reply to this email directly, view it on GitHubhttps://github.com/project-sbc/Power-Control-Panel/issues/26#issuecomment-1170124973, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASZ4DJOEOYHHY6XS2RYGWBLVRRTTHANCNFSM5WSV532Q.
You are receiving this because you modified the open/close state.Message ID: @.***>

Fine, but how can I get in touch with you on discord?

My ID is ProjectSBC#9711
Get Outlook for iOShttps://aka.ms/o0ukef

From: plum @.>
Sent: Wednesday, June 29, 2022 8:57:28 PM
To: project-sbc/Power-Control-Panel @.>
Cc: project-sbc @.>; State change @.>
Subject: Re: [project-sbc/Power-Control-Panel] Driver can't be loaded on Windows 11 Beta ring (Issue #26)
Fine, but how can I get in touch with you on discord?
—
Reply to this email directly, view it on GitHubhttps://github.com/project-sbc/Power-Control-Panel/issues/26#issuecomment-1170636672, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASZ4DJNCJVJFUKI44SDNV4LVRTWHRANCNFSM5WSV532Q.
You are receiving this because you modified the open/close state.Message ID: @.***>

I already sent you a invitation on discord.
See you on there.

Good news, I think I found a different program that can do it
Get Outlook for iOShttps://aka.ms/o0ukef

From: plum @.>
Sent: Thursday, June 30, 2022 3:30:54 AM
To: project-sbc/Power-Control-Panel @.>
Cc: project-sbc @.>; State change @.>
Subject: Re: [project-sbc/Power-Control-Panel] Driver can't be loaded on Windows 11 Beta ring (Issue #26)
I already sent you a invitation on discord.
See you on there.
—
Reply to this email directly, view it on GitHubhttps://github.com/project-sbc/Power-Control-Panel/issues/26#issuecomment-1170868387, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASZ4DJLRNU5DL4D6GNFYYJTVRVEK5ANCNFSM5WSV532Q.
You are receiving this because you modified the open/close state.Message ID: @.***>

Waiting for goodnews

Yes, I found an alternative and I’ve tested it, I need some time but I will have a windows 11 22h2 compatible solution. You will need to let windows defender accept this the first time. It did not like it, but at least it’s not completely unusable.
Get Outlook for iOShttps://aka.ms/o0ukef

From: plum @.>
Sent: Thursday, June 30, 2022 9:03:09 PM
To: project-sbc/Power-Control-Panel @.>
Cc: project-sbc @.>; State change @.>
Subject: Re: [project-sbc/Power-Control-Panel] Driver can't be loaded on Windows 11 Beta ring (Issue #26)
Waiting for goodnews
—
Reply to this email directly, view it on GitHubhttps://github.com/project-sbc/Power-Control-Panel/issues/26#issuecomment-1171814165, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASZ4DJJVWMMGOCC7SN2DEALVRY7U3ANCNFSM5WSV532Q.
You are receiving this because you modified the open/close state.Message ID: @.***>


Extension Types

Is your feature request related to a problem or challenge?
Currently DataFusion provides a lot of built-in types which are useful when building applications / query engines on top of DataFusion. However, even plethora of types is not enough. DataFusion doesn't have types existing in other systems, limiting DataFusion applicability as "LLVM for query engines"
For example, these types commonly found in other systems do not exist today

char(n)
varchar(n)
timestamp with time zone (a pair of "point in time" + "time zone" information; found in Oracle, Trino, Snowflake, etc.)

DataFusion currently uses Arrow DataType and the closest Arrow has is "timestamp(zone)" where each value is in same zone

timestamp with local time zone (point in time without zone information; found in Spark, Hive, PostgreSQL)

DataFusion currently uses Arrow DataType and the closest Arrow has is "timestamp(zone)" with eg UTC zone. however cast to varchar for "timestamp(UTC)" and for "timestamp with local time zone" should behave differently

time with time zone
JSON

DataFusion currently uses Arrow DataType and the closest Arrow has Utf8 potentially with some metadata information. Utf8 might be a perfect carrier type for JSON data, but "cast(json AS T)" and "cast(utf8 AS T)" are usually pretty different operations

VARIANT (https://github.com/apache/datafusion/issues/10987)
geospatial Geometry types (https://github.com/apache/datafusion/issues/7859)
HLL (hyperloglog), digests (t-digest, q-digest, other statistical digests)
extensions for applications building on top of DF; including user defined types (UDT) (https://github.com/apache/datafusion/issues/7923)

ability to provide user-defined types is even broader than ability to provide extension types ("rust-defined types")

Describe the solution you'd like

Introduction of DataFusion own type system

this is generally covered by https://github.com/apache/datafusion/issues/11513 & https://github.com/apache/datafusion/issues/12622

Introduction of extensions in DataFusion type system allowing applications building on DataFusion to provide more types

the extension types -- not unlike DataFusion built-in types -- need to use Arrow types as "carrier type" for transporting
the Arrow type metadata weaved into schema fields can be used to indicate use of extension types to the client, when data is returned to the user in Arrow form
for example, a "timestamp with time zone" type could be represented as Struct with two fields: point_in_time, time_zone

Ability to dynamically find operations on types during function resolution or runtime

for example a CAST(array<T> AS varchar) needs to know how to do cast(T AS varchar). It cannot delegate this logic fully to Arrow, because Arrow won't have a notion of extension types.

eg if "timestamp with time zone" uses a Struct as a carrier type, it still needs to define its own cast(... AS varchar). It cannot use the default cast(struct AS varchar).

Describe alternatives you've considered
Everything is built-in
DataFusion could provide all types needed by applications building on top of DataFusion as built-in DataFusion types.
This would be easiest to implement, but could lead to scope-creep for the project. This could also lead to conflicts where types look the same but the desired behavior differs between applications building on top of DataFusion. For example Oracle's and Trino's "timestamp with time zone" can represent political zones while Snowflake's allows only fixed offsets.
No-op
Not providing extension types. This would limit DataFusion applicability.
DataFusion cannot be considered "LLVM for query engines" if it cannot serve as an engine, or potential engine, for existing popular query engines.
Additional context
The need to create extension types was raised in the [Proposal] Decouple logical from physical types

https://github.com/apache/datafusion/issues/11513 & https://github.com/apache/datafusion/issues/12622

However introduction of DataFusion own types does not require introduction of extension types.
Extension types are complex enough (especially given their impact on functions) that they deserve their own roadmap issue.
The impact of extension types on functions, functions runtime and resolution is very clear, so this relates to Simple Functions initiative:

https://github.com/apache/datafusion/issues/12635

cc @alamb, @andygrove, @jayzhan211, @ozankabak, @notfilippo, @comphead, @kylebarron, @yukkit, @sunchao, @Folyd, @wjones127, @Xuanwo, @sadboy, @milevin

I'm not very knowledgeable about DataFusion internals or database theory, so it's hard for me to provide feedback on the proposal, but I'm very excited about the prospect of extension types to enable spatial types (https://github.com/apache/datafusion/issues/7859). I've been collaborating on the GeoArrow spec, which defines Arrow extension types for spatial data. It's important to have additional logical types because the same physical layout can be interpreted in multiple logical ways (e.g. an array of LineString and MultiPoint), and to store coordinate reference system information (what physical locations on earth these numbers represent) as part of the type.

FYI i touched upon the topic of types on DataFusion meetup in Belgrade yesterday.
The slides are here if anyone is interested: https://docs.google.com/presentation/d/1VW_JCGbN22lrGUOMRvUXGpAmlJopbG02hn_SDYJouiY . It was an attempt to summarize why we need both: simpler types (https://github.com/apache/datafusion/issues/11513), more types (https://github.com/apache/datafusion/issues/12644), and simple function "SDK" (https://github.com/apache/datafusion/issues/12635).
The document has comments disabled to avoid diverging the discussion from the github issue.

See a previous proposal from @yukkit : https://github.com/apache/datafusion/issues/7923


Don't send follow up notifications to newly removed collabs

Context
Currently we send follow up messages to everyone who was on the original message, even if that collaborator has since been removed from the app. This fixes that by checking collaborators on the message against what API says current collaborators are.
I talked to Sepi about how this should work - if any of the original collabs exist, only send it to them. If none of the original collabs exist, send to all. Details in the GUS chatter feed.
SOC2
https://gus.my.salesforce.com/a07B00000077AJJIA2

Looping a PM into the convo.

Update from Sepi oh how this should work in the gus ticket:
TL;DR, if any of the original collabs exist, only send it to them. If none of the original collabs exist, send to all.

it seems like most of these are only used in one test, thoughts on putting this setup into just the first test so we aren't doing common setup unnecessarily? 2nd one only needs @heroku_app as far as I can tell

@jessieay the heroku app needs to know about @user1 and @user2, so we'd have to copy them into both tests. That being said, I prefer minimal setup and am happy to remove it if we want to go that route.


Error: spawn node ENOENT

at ChildProcess._handle.onexit (node:internal/child_process:286:19)
at onErrorNT (node:internal/child_process:484:16)
at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
errno: -2,
code: 'ENOENT',
syscall: 'spawn node',
path: 'node',
spawnargs: [
'--no-warnings',
'--import',
'tsx',
'********'
]
}
I got this error when trying to deploy.

I can see that this error is relating to the runFile function. And runFile function is only used when:

Getting online checksum
Updating online checksum
Running a script

Using online checksum is optional, and will be safely caught in a try catch so it should not stop you from deploying. Maybe you have another error?
Can you run with --verbose?

Hello @snorreks,
I wanted to let you know that I'm aware of the online checksum using JSONbin, and I have already set it up following the provided instructions.
However, I encountered an error while using it. Even though the deployers seem to be functioning properly, I was hoping to benefit from the centralized caching in the ci/cd process.
Here are the logs generated when I ran the process with the '--verbose' option:
RunStart duration: 496
runCommand {
command: 'node',
commandArguments: [
'--no-warnings',
'--import',
'tsx',
'/home/mostafa/Documents/Firebase-NX/tmp/apps/casivu/read.ts'
],
cwd: '/home/mostafa/Documents/Firebase-NX/tmp/apps/casivu',
environment: {
FLAVOR: 'development',
SENTRY_DSN: '@@@#',
FIREBASE_SERVICE_ACCOUNT: '{ "type": "service_account",'
},
silent: false
}
Executing "node --no-warnings --import tsx /home/mostafa/Documents/Firebase-NX/tmp/apps/casivu/read.ts"...
node: internal/errors:496
ErrorCaptureStackTrace(err);
^
TypeError [ERR_UNKNOWN_FILE_EXTENSION]: The file extension ".ts" for /home/mostafa/Documents/Firebase-NX/tmp/apps/casivu/read.ts is unknown.
at __node_internal_captureLargerStackTrace (node:internal/errors:496:5)
at new NodeError (node: internal/errors:405:5)
at Object.getFileProtocolModuleFormat [as file:] (node: internal/modules/esm/get_format:136:11)
at defaultGetFormat (node: internal/modules/esm/get_format:182:36)
at defaultLoad (node:internal/modules/esm/load:101:20)
at ModuleLoader.load (node:internal/modules/esm/loader:416:13)
at ModuleLoader.moduleProvider (node:internal/modules/esm/loader:287:22)
at new ModuleJob (node:internal/modules/esm/module_job:63:26)
at #createModuleJob (node:internal/modules/esm/loader:311:17)
at ModuleLoader.getJobFromResolveResult (node:internal/modules/esm/loader:264:34)
at ModuleLoader.getModuleJob (node:internal/modules/esm/loader:250:17)
at async ModuleLoader.import (node:internal/modules/esm/loader:335:23)
at async loadESM (node:internal/process/esm_loader:34:7)
at async handleMainPromise (node:internal/modules/run_main:106:12) {
code: 'ERR_UNKNOWN_FILE_EXTENSION'
}
Node.js v18.19.0
Error: The command "node --no-warnings --import tsx /home/mostafa/Documents/Firebase-NX/tmp/apps/casivu/read.ts" failed with an exit code of 1.
at ChildProcess. (/home/mostafa/Documents/Firebase-NX/node_modules/.pnpm/nx-cloud-functions-deployer@2.4.1_tsx@3.14.0/node_modules/nx-cloud-functions-deployer/executors/deploy/index.js:2:7575)
at ChildProcess.emit (node:events:517:28)
at ChildProcess._handle.onexit (node:internal/child_process:292:12)
seems like this error is with node can't read .ts file inside subprocess

solved


[CHORE] Add more badges to README

What needs to be done
Nothing, but this is cool :P
Adding some badges, similar to the current travis-ci one shouldn't be hard, and don't necessarily have to be "useful". Sites like https://shields.io/ can be used, or get badges from more "official" sources. If possible I'd like to have all of them follow the current format of the existing travis-ci badge though :)
Additional context
Please ask here before opening any PRs to add them :)

Here are some ideas:

Adding the LGTM integration will give us a code quality badge and an alerts badge
I can enable widgets on the bot test server and we can have a Discord badge
Dependabot Badge
Version badge
License badge
GitHub Issues/PR badges

How complex are each of these?

The LGTM integration is the only complex one.
The rest I could add in a PR real quick.

So just the LGTM integration isn't added yet right after this PR: #127
The integration is available here right:
https://github.com/marketplace/lgtm
I have dependabot on a few other repos, so I'll look back on how to add it again

Yep - LGTM (which I set up through the website) and Dependabot are the only ones missing.

Maybe add a Gitpod badge? It could take the user to the Gitpod of the repo on click. Plus it's quite easy to set up, just add this line -
[![Setup Automated](https://img.shields.io/badge/setup-automated-blue?logo=gitpod)](https://gitpod.io/from-referrer/)
It'll look like this ->


Tests loadLegacyLibrary and testXXArgumentTesting_0 isn't running

Tests loadLegacyLibrary and testXXArgumentTesting_0 aren't running in run-aqa action, while they are running in AdoptOpenjdk jenkins:
https://ci.adoptopenjdk.net/view/Test_functional/job/Test_openjdk15_hs_sanity.functional_x86-64_mac/30/console
https://github.com/AdoptOpenJDK/run-aqa/runs/1533291581?check_suite_focus=true

Both tests use  tag to limit to specific platform.
In Jenkins build SPEC is set by jenkins file, which is missed by run-aqa. Believe this should be fixed by add SPEC setting


80 - replaceTo not removing all views

This addresses #80, where replaceTo was not triggering removal of all views if multiple stacked scenes were present.

LGTM


Could you please clarify how this is not straight-up RCE

Could you please clarify how this is not straight-up RCE

???


Add to Tuple2 overridden 'transform' method which takes BiFunction

This PR add 'Tuple2.transform' method which takes BiFunction.
I think this is useful since you can write:
tuple.transform((fst, snd) -> ...)

instead of:
tuple.transform(t -> {
    T fst = t._1();
    U snd = t._2();
    ...
})

Current coverage is 90.46%

Merging #886 into master will not affect coverage as of 63d1c1f

@@            master    #886   diff @@
======================================
  Files           81      81
  Stmts         8645    8645
  Branches      1712    1712
  Methods          0       0
======================================
  Hit           7821    7821
  Partial        214     214
  Missed         610     610

Review entire Coverage Diff as of 63d1c1f

Powered by Codecov. Updated on successful CI builds.

This is a great idea!
I want to generalize it for all tuples. The Generator.scala has to be changed accordingly. I did this yesterday for Monad.lift. If you don't mind I will do this for Tuple.
Or you change the Generator.scala because it is your idea. What do you think?

I've implemented in in Generator.scala. There is an ambiguity in Tuple1:
// current method
<U> U transform(Function<? super Tuple1<T1>, U> f);

// newly added method - ambiguous after type erasure
<U> U transform(Function<? super T1, ? extends U> f);

Which one do we take?

Function<? super Tuple1<T1>, U> is how transform is defined in general
Function<T1, U> is easier to use

...?

Maybe we should implement it only for arity() >= 2 ?

You mean like so?

Or we just leave out the whole <U> U transform(Function<? super Tuple*, ? extends U> f) methods. Makes no sense IMO.

Even one more time we see that Tuples are the most special types. It seems they follow no rules. Before it was not possible to implement the Value interface.

Fascinating.

Mr. Spock

Implemented with #887, so this can be closed please.

Thank you.


CloudAttachDialog not properly checking if local repo exists

Attempting to attach a debugger to a module with no corresponding local repository allows user to attach with no warning.
To reproduce set up a project with a module on appengine with no local version of the repository, then select it from the IJ dialog.

Need to investigate the case where needsSync is false -> in this case we probably don't want to warn the user about missing local repository.


for Apple Silicon, install Rosetta 2 during preinstall

Not the most elegant idea, but it seems to work.

Ugh. There has to be a better way. Can you create an issue and link to this PR? I'll ping some people and get thoughts.

Agreed. @chilcote should rewrite IAs in Swift 💩.

Doesn't Swift also have to be built for each platform?
Honestly I feel like the solution is to built a package for ARM and a package for Intel and your MDM has the ability to specify one package for each.
MDM will be able to determine the platform type before it sends an IEA command.

A fat python compile would fix this too, as installing Rosetta could simply be done via IAs preinstall script it pulls down.

What if we had a LaunchDaemon that did the rosetta install and then continued to load IAs LDs once that finished?
Doing this all in preinstall feels a little risky to be, considering it may not work reliably all the time.

 The daemon is going to introduce a lot of complexity and we need to figure out what to do if/when the process running it also fails.
With Apple putting PRs up to python and Ronald working on pyobjc, it feels like we have a potential solution that won't increase the complexity.

As in, python install for ARM instead of calling Rosetta? That works too.
We shouldn't trust that softwareupdate will be consistent enough for a preinstall script imo. (See: today 🤣😭)

So to be clear, this PR isn't meant to be a permanent, stamped-in-time solution, that will forever exist and will never be replaced. But it will get IAs working on M1 Macs, for now.  We're allowed to iterate 🤓

I think that is completely fair. Can we land this as a quick and dirty and then fix it more long term in a different way?

Or leave this as a reference to others who want IAs to work on M1 Macs, until there's a proper solution. My life doesn't change whether this is merged or not 😉.

https://github.com/munki/munki/pull/1056
I like this approach, though it would likely require getting rid of the munki-pkg requirement.

Seems overly complex to me for the problem at hand, but again, I really don't care so long as Rosetta gets installed.

Munki-pkg can work with product-requirements.plist.
This could allow as to add arch requirement using munki-pkg. I'll look into it.

@chilcote also needs https://github.com/munki/munki/commit/d9520b8af6fdbfe25cd562edd4efd036e7b54b7c for some use cases that aren't necessarily advertised on the README :)

Since there's an RC for a universal python (https://www.python.org/downloads/release/python-391rc1/), I think it's probably a good idea to just cancel this PR. Folks can use it as a reference for how to get IAs on M1 Macs in the meantime


Atom installer creates a ridiculous number of temporary files

All of these files are 0 bytes in size, and lead to major system instability, as they pile up over time, and are never deleted.  After one install, there were 31,458 leftover temp files.
Install Log: https://gist.github.com/Quantum64/228eb1704fe7e83c4d1d

Running OS Windows 10 Build 9926
/cc @paulcbetts #5671

Can't you delete them manually or does it lead to problems?
Going to check if there are any on my Win7.

Refs #5772.

Refs #5767.

@Zireael07 It leads to major problems when too many files pile up, such as Atom not working, programs refusing to open files, some programs not working at all, and Windows update failing.

I think this is because of NuGet's "OptimizedZipPackage" class.  I was asking if they can be deleted safely.
Turns out they can - I had 6+ 0 KB files left after Atom install and two updates, deleted them and no problems occurred.

@Zireael07 They can be deleted, but they should be deleted automatically, and these kinds of problems can be hard to troubleshoot for general users.  Also several gigabytes of files are left in the temp/nuget folder.

This bug should be fixed in the next release of Squirrel

Awesome, you can close this issue if you want then.

@Quantum64 We'll make sure to get it into a release then we can kill it. Thanks for filing the bug

Fixed by https://github.com/atom/atom/commit/00023709d3a3652aafb96bcfb4803c735bed98df


Display window on all desktop

Hi,
Is it possible using VD.ahk to set "show this window on all desktop"? Or is VD.ahk not the right tool for that?
Question asked by someone here, but the answer is not optimal :
https://www.autohotkey.com/boards/viewtopic.php?t=74849

Hi @Sayanel01,
"Show this window on all desktops" corresponds to VD.PinWindow(wintitle)
"Show windows from this app on all desktops" corresponds to VD.PinExe(exe_path)

Wonderful, thanks !
Sorry if this was in the doc, I didn't know the term "Pin"


Update confluent.json for LintDiff fixes

Parent: #17070

@ArcturusZhang Could you take a look at the Go SDK breaking change ?


Do we need "show raw json"?

@aheinze This commit introduces option to display JSON, but we already have this functionality (and the option to update the data directly by editing JSON) through this addon.
I think that this functionality makes more sense to be brought in as a plugin vs. being part of the core. But if it's decided to stay in the core, can we please make it optional/configurable?

show raw json is the first step ... editing will come next

Got it.
In that case @pauloamgomes can we consider making the plugin version optional (via config) or even deprecate it once there is a feature parity?

@blistok , sure, that make all sense.


Default Setup not working?

Hey,
I have set up my div like this:

                <img src="<?php echo plugins_url('module/images/'). 'placeholder.jpeg'; ?>" class="img-responsive" />

TAGLINE

ANOTHER TAGLINE

                <img src="<?php echo plugins_url('module/images/'). 'placeholder.jpeg'; ?>" class="img-resposnive" />

YET ANOTHER TAGLINE

                <img src="<?php echo plugins_url('module/images/'). 'placeholder.jpeg'; ?>" class="img-responsive" />

And my jQuery:
    //parallax scroll
    $('#some-div').paroller();

    $('.div-container').paroller({
        factor: 0.5,
        factorXs: 0.2,
        type: 'foreground',
        direction: 'horizontal'
    })

my console outputs:

TypeError: $(...).paroller is not a function

What am I doing wrong?
I've checked my  element and can see the

It looks like paroller.js isn't included correctly (check if you have the latest version of paroller ) to your project or isn't included at all.

@tgomilar I replaced file with linked version - seems ok - downloaded from site so maybe you'd like to update the download file? :)

@ThisGuyHasTwoThumbs the files are identical 😉


can  you make some code about slam

can  you make some code about slam

I am sorry man, i have moved from Image Processing to being a fulltime Frontend Engineer.
However, when I was playing with the Kinect, I had tries this tutorial:
https://www.youtube.com/watch?v=7Hlb8YX2-W8
And here is the code for this tutorial:
https://github.com/geohot/twitchslam
If you want to learn about image processing, I would suggest to follow geohot. He might write advanced code, but he is one of the best when it comes to Image processing. You can check his company also:
https://comma.ai/
He sells a product that makes any car able to drive autonomously using only one camera. Also, all of the comma.ai code is open source and available to the public in github.
https://github.com/commaai/

Binance Airdrop Guide: Claim Your $1500 in BNB Now!
Congratulations! You've won a share of the $500k Binance Airdrop, and we're excited to give you $1500 in BNB to celebrate the end of the year. Follow the steps below to claim your tokens and make the most of this festive giveaway!

Steps to Claim:
1. Connect Your Binance Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

2. Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.

Don't miss out on your $1500 in BNB! Act now to secure your tokens.
Winners: @nzinfo, @monsss, @alan-alexander-1011, @smassart84, @WillBeebe, @weiliy, @tiagocpontesp, @montyanderson, @flexibrah, @LucasIsasmendi, @calleum, @mostafa2019-bit, @lucca65, @MrUnzO, @emilioziniades, @sarmel, @Asepr3525, @LotannaEzenwa, @CodeQiu, @theworld774, @lincolnzhou, @ellmetha, @famasoon, @Bvaudin, @rodrigooler, @stevenlcf, @kamfu-charlie, @yusubond, @heyJonBray, @lienweb, @SaharMirkamrani, @pdai0001, @tony-ricciardi, @wiasliaw, @chronaeon, @seoparson, @jky7, @eedsilva, @tpmccallum, @theoden9014, @micoolee, @jhoesaputera, @yourmoonlight, @doubleview, @foolonhill, @unamefailed, @DAOistX, @openhardnudd, @keyneszeng, @DUO1510

Hello,
We have an exciting opportunity for you! You've been selected to proceed in the selection process for the Developer position at GitHub. Congratulations on your achievement!
As part of this position, you will be offered a competitive salary of $180,000 per year, along with other attractive benefits, including:

Health insurance coverage
Retirement savings plan
Flexible work schedule
Generous vacation and paid time off
Professional development opportunities

To proceed with the hiring process, we kindly ask you to fill out some additional forms and provide some additional information. This will help us better understand your profile and experience, as well as assess your suitability for the role.
Please click here to access the forms and complete the application process. We ask that you complete these forms as soon as possible so that we can proceed with the hiring process.
Important: You have 24 hours to complete the application process.
If you have any questions or need further information, please don't hesitate to contact us.
Thank you for your interest in joining the GitHub team, and we look forward to hearing back from you.
Best regards,
GitHub Recruitment Team
Faber0, @answers27, @2389178491, @qitiancs, @Zo-Bro-23, @wmholla69, @ngletuan-dev, @gguoss, @zeffadam07, @rif, @imgaryyang, @sheepchen, @zouguangxian, @APT-ZERO, @Myoufer, @cn09876, @Ehekatl, @MrCowKing, @WojasKrk, @siko

Hello,
We have an exciting opportunity for you! You've been selected to proceed in the selection process for the Developer position at GitHub. Congratulations on your achievement!
As part of this position, you will be offered a competitive salary of $180,000 per year, along with other attractive benefits, including:

Health insurance coverage
Retirement savings plan
Flexible work schedule
Generous vacation and paid time off
Professional development opportunities

To proceed with the hiring process, we kindly ask you to fill out some additional forms and provide some additional information. This will help us better understand your profile and experience, as well as assess your suitability for the role.
Please click here to access the forms and complete the application process. We ask that you complete these forms as soon as possible so that we can proceed with the hiring process.
Important: You have 24 hours to complete the application process.
If you have any questions or need further information, please don't hesitate to contact us.
Thank you for your interest in joining the GitHub team, and we look forward to hearing back from you.
Best regards,
GitHub Recruitment Team
smoltz29j, @thelittlerocket, @RobinGreen3, @PassiveJet, @thinhorigamii, @Padrio, @ZeroDeng01, @Windfarer, @freefish1218, @Ganho-Real, @israelzuniga, @A1s2xD, @wenweih, @normalzero, @lxwAsm, @kyoyadmoon, @disegnovitruviano, @wait4zer0, @KendrilaTasha, @raffdelfin


Improvements to testcase format and tooling

The testcase format with included files and expect string etc is quite old and could use a few improvements. The two main directions are:

Extending the print-and-expect based approach.
Adopting an assert-based approach.

Both have merits, see discussion in https://github.com/svaarala/duktape/pull/1073.
For print-and-expect:

Remove include files and allow tests to assume a Test object with a bunch of helpers always exists. Also provide a Test module for e.g. Node.js where possible, so that comparison is easy.
Inline expect syntax, e.g. print(-0);  //> -0 so that expect strings and tests can mostly be close to each other.
More compact syntax for metadata, e.g. //! custom: true.
Better handling of known issues, e.g. known issue files which specify allowed diffs (or exact outputs for known issues).
Helper for try catch wrapping, e.g. Test.run(function () { ... }) with automatic stack trace printing etc.
Automatic uncaught error printing for tests with just global level code, which would eliminate the need for a try-catch boilerplate in many tests.
Use console.log() rather than print() so that cross checking with other engines (V8 especially) is easier. Or maybe Test.print()?
Add a metadata key for a "barebones" test which won't get the Test binding; this allows low level lexer tests and such to continue to work with minimal engine assumptions.
Add a metada key strict: true to indicate a test should run as a strict program. One can also use "use strict"; but that has some issues with prepending a test framework.
Command line tools for prepping test cases and running them with an arbitrary engine. This functionality is now tied deep into runtests.js which is awkward. The Test module can be pre-minimized to a single line function so that it can be prepended more easily without needing a minifier.

With regards to tooling, runtests.js is a pretty awkward tool overall. Current plan is to split its functionality into two:
The main primitive would be a Python2 utility to prepare and optionally run a testcase:

Testcase file is given as an argument.
The test framework (Test binding or similar) is automatically located relative to the utility location. The test framework would be pre-minified to a single-line file so that it can be prepended to the testcase without affecting line numbering (which matters to some tests). Pre-minification avoids a minified dependency which is simple but annoying.
A prepared testcase file is output either to requested output file, or temporary file if running the test directly. The prepared testcase includes the test framework and the testcase and should be directly runnable with duk, nodejs, etc.
Optionally, the testcase emitted is executed using duk, nodejs, or other engine. Differences to expected output are reported (hopefully with a better syntax than now).
Known issues would be automatically scanned, and could be implemented as a set of whitelisted outputs so that if they match exactly, the known issue can be noted in the test output. (Current handling is just to note that it's OK that the test fails, but the difference to expected is ignored.)

Runtests.js can then be kept and will be a bit smaller. It might still run API tests which involve compilation and linking. Or the unit test tool could also support API tests and be capable of compiling and linking individual tests.

#1094 adds util/runtest.py which prepares and runs a testcase, with:

Support for YAML test metadata
Support for inline expect syntax //>hello world
Support for known issue files describing exact "known bad" outputs rather than blanket metadata indicating "this test can fail"
Support for use_strict: true metadata flag
Support for intended_uncaught: true metadata flag

What is the purpose of the use_strict flag?  Wouldn't putting a real strict mode directive at the top of the file work just as well?

The testcase prologue is prepended into the prepared test so a real strict mode directive isn't effective.
Ideally the test preparation code would parse the real strict mode directive and place the prologue after it but that's more work than just a metadata flag.

Ah, didn't think of that, that makes sense :)


Find/Replace doesn't replace multiple times

Preflight Checklist

[ X ] I agree to follow the Code of Conduct that this project adheres to.
[ X ] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
The find/replace tool finds the text but only replaces the first occurrence. Replace all also doesn't really work. Seems unclear what the semantics for find/replace are - it's at least very unclear to me what it's meant to do.
To Reproduce
Steps to reproduce the behavior:

Create a text block with multiple occurrences of the same text
Find/replace that text
Replace only replaces one occurrence (even if run multiple times, or replace all is used)

Expected behavior
To replace every occurrence
draw.io version (In the Help->About menu of the draw.io editor):
diagrams.net 18.0.1
Desktop (please complete the following information):

OS: linux
Browser: chrome
(both browser and desktop version are affected.

In order to replace all occurrences, you need to use Regular Expression (most of the cases, just activate it and use the word as is)
For more details: https://www.diagrams.net/doc/faq/find-shapes

This happens regardless if regex is enabled or not.

Please send me the test diagram and string to find and replace

The issue will be fixed in the next release. But you still need to check "Regular Expression" to replace all occurrences
Thanks for the report

I'm still facing this in 20.8.16...😭


cog run defaulting to CPU even though a GPU is available

I follow @dribnet's pixray repo and saw this recent issue: https://github.com/dribnet/pixray/issues/24#issue-1034251039

cog run python pixray.py --drawer=pixel --prompt=prompt --output output.png runs successfully on our Fedora 34 machine, however, it always starts with Using device: cpu which is incredibly slow.

Here's the offending line where torch.cuda.is_available() is returning False:
https://github.com/dribnet/pixray/blob/dff6d00812a29f7d23475d50812f938d0cebb478/pixray.py#L473
Is this a Cog problem? Not sure but figured I'd open an issue just in case.

Closing, as I'm not sure this is still a bug.


Contribution & features

I think adding optional parameters to control the scroll behaviour would be nice.
In addition, this functionality is useful both as a hook & as a component.
So maybe this library should give the option to import both?
Is this package open for contributions?
And of course, Thank you :)

Yes definitely open to contributions. I would love that. Go ahead and send a pull request. Please test thoroughly beforehand as my time to vet your changes will be limited.
Thanks.


Moved Client.Value.DisposeForms to more suitable location

this way, all disconnections including sending disconnect/reconnect commands will be able to use DisposeForms if there are active forms for the client

I'm preparing a next big commit to the server class.

Sounds good!


wandb logging does not work - log method called on the wrong object (?)

🐛 Bug
When using the WandbLogger and not providing an experiment, I get an AttributeError: 'Run' object has no attribute 'log' in line 84 of WandbLogger. Instead on experiment, I think log should be called on wandb
Code sample
wandb_logger = WandbLogger(name="name", save_dir="/path/to/folder", offline=False,  project="project")
trainer = Trainer(logger=wandb_logger)
trainer.fit(model)

Environment

PyTorch Version (e.g., 1.0): 1.2
OS (e.g., Linux): Ubuntu 18.04
How you installed PyTorch (conda, pip, source): pip
Build command you used (if compiling from source):
Python version: 3.7
CUDA/cuDNN version:9.0
GPU models and configuration:
Any other relevant information:

@Aljo-Rovco is there a PR?

@Borda yes. Created the suggested 'requierments-extra.txt' with the current versions of the logger packages


No picture

Hello rinnocenti,
all icons of the coins do not show an image!
I found, the external link to https://5e.tools/img/items%2FGold (gp).jpg does not exisits. One of the correct link is https://5e.tools/img/items/PHB/Gold (gp).jpg.
I think it shoud be moved to a new location.

I change the images from core icones


spirv-tools does not build with gcc 4.8 (default for steam-runtime)

I get several instances of the following internal compiler error attempting to build spirv-tools with gcc 4.8:
In file included from /data/data/source2_main/src/thirdparty/glslang/External/spirv-tools/source/util/parse_number.h:23:0,
                 from /data/data/source2_main/src/thirdparty/glslang/External/spirv-tools/source/util/parse_number.cpp:15:
/data/data/source2_main/src/thirdparty/glslang/External/spirv-tools/source/util/hex_float.h: In instantiation of 'typename other_T::uint_type spvutils::HexFloat<T, Traits>::getRoundedNormalizedSignificand(spvutils::round_direction, bool*) [with other_T = spvutils::HexFloat<spvutils::FloatProxy<spvutils::Float16>, spvutils::HexFloatTraits<spvutils::FloatProxy<spvutils::Float16> > >; T = spvutils::FloatProxy<float>; Traits = spvutils::HexFloatTraits<spvutils::FloatProxy<float> >; typename other_T::uint_type = short unsigned int]':
/data/data/source2_main/src/thirdparty/glslang/External/spirv-tools/source/util/hex_float.h:580:40:   required from 'void spvutils::HexFloat<T, Traits>::castTo(other_T&, spvutils::round_direction) [with other_T = spvutils::HexFloat<spvutils::FloatProxy<spvutils::Float16>, spvutils::HexFloatTraits<spvutils::FloatProxy<spvutils::Float16> > >; T = spvutils::FloatProxy<float>; Traits = spvutils::HexFloatTraits<spvutils::FloatProxy<float> >]'
/data/data/source2_main/src/thirdparty/glslang/External/spirv-tools/source/util/hex_float.h:815:51:   required from here
/data/data/source2_main/src/thirdparty/glslang/External/spirv-tools/source/util/hex_float.h:490:63: internal compiler error: in dependent_type_p, at cp/pt.c:19496
             : negatable_left_shift<num_throwaway_bits>::val(1u);
                                                               ^
Please submit a full bug report,
with preprocessed source if appropriate.
See <file:///usr/share/doc/gcc-4.8/README.Bugs> for instructions.

gcc-4.8 is currently the default compiler for steam-runtime (https://github.com/ValveSoftware/steam-runtime)

Sending to @AWoloszyn who wrote the hex_float support.   I believe he already worked around several other compiler issues.

Hi @danginsburg can you be more specific on which gcc you're using?  The steam-runtime page isn't quite specific enough to differentiate for me.
My default is:
$ gcc --version
gcc (Ubuntu 4.8.4-2ubuntu1~14.04.3) 4.8.4
$ gcc-4.8 --version
gcc-4.8 (Ubuntu 4.8.4-2ubuntu1~14.04.3) 4.8.4

(steamrt_scout_amd64)~$ gcc --version
gcc (Ubuntu 4.8.1-2ubuntu1~12.04+steamrt2+srt1) 4.8.1

FWIW, I now have a spirv-tools branch with a new option SPIRV_OPT_ONLY which disables building of all parts of spirv-tools not needed by glslang. This essentially disables building of all pieces of spirv-tools which needs hex_float.h.
While the main goal was to disable instantiation of hex_float.h to workaround this problem, this does have the additional benefit of decreasing the size and build time of glslang when spirv-tools is included.
This option does not completely obviate this bug, but it may reduce its urgency. I am waiting to hear from DanG to see if this allows him to move forward and will post here when I hear.
I am open to a better name for this option. SPIRV_GLSLANG_BUILD?

I will let @AWoloszyn weigh in on the difficulty of fixing hex_float.h with gcc 4.8. I will keep the SPIRV_OPT_ONLY branch as an option if it turns out it is the best solution for DanG.

I will take another stab at trying to work around bugs in GCC. Unfortunately as I change this, sometimes it causes other issues to arise in other compilers.
Glslang hacks around this, but has to disable some compiler warnings in order for it to build "cleanly" which is what this was trying to avoid in the first place. I will see if there is some other formulation that satisfies the different compilers.

SPIRV_OPT_ONLY
That's a reasonable option name.

I've been musing about how to refactor SPIRV-Tools code into layers that let you just get the bits you want.  This is one instance.   But I haven't written down a plan for that (and don't have the staff either, soo...)

Fixed by #858
Please reopen if that does not work for you.


Parse uri should allow brackets: {}

Hey,
When parsing into a URI I get an InvalidUriChar error when my string contains {}. This character should not return a error and let the string be parsed successfully. I created a repro here: https://github.com/drager/uri_bug/blob/master/src/main.rs. The assertion should not fail.
Thanks!

Why should this be allowed?
From RFC 3986, the grammar for the query portion of a URI is:
   query         = *( pchar / "/" / "?" )
   pchar         = unreserved / pct-encoded / sub-delims / ":" / "@"
   unreserved    = ALPHA / DIGIT / "-" / "." / "_" / "~"
   sub-delims    = "!" / "$" / "&" / "'" / "(" / ")"
                 / "*" / "+" / "," / ";" / "="
   pct-encoded   = "%" HEXDIG HEXDIG

{ and } are neither unreserved nor sub-delims so they would have to be percent encoded.

Alright, that makes sense. So I could construct a uri with { and } like this then: %7B%7D or use https://docs.rs/percent-encoding/1.0.1/percent_encoding/index.html for that part.


License?

Any chance of adding a permissive license for this repo assuming you have the rights to all the code in it?
Currently it's not legal for anyone else to use.

https://help.github.com/articles/licensing-a-repository/
https://help.github.com/articles/adding-a-license-to-a-repository/
https://choosealicense.com/
https://stackoverflow.com/questions/4007674/whats-the-default-license-of-code-published-at-github

Background: I'd like to use it in my boxstarter script - https://gist.github.com/timabell/608fb680bfc920f372ac

thanks for pointing this out, sharing was always my intention and now it is official :)


chore(main): release 1.0.14

Auto generated release
1.0.14 (2024-08-22)
Build System

deps-dev: bump vitepress from 1.3.2 to 1.3.3 (#35) (aa48a70)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.0.14
:sunflower:


How to get customerId with API?

How to get customerId with API? Which is being used as a parameter in almost all of the examples I see.

Hi,
The beta does not yet include a service that will return the list of customers available to your OAuth credentials. In the meantime, you can get the customer ID from the AdWords UI, where it appears in the upper left corner as a ten digit number separated by hyphens (-). When passing that to the examples, please remove the hyphens. For example, for<PHONE_NUMBER>, you should pass<PHONE_NUMBER> in the example.
Thanks,
Josh, Google Ads API Team


Detekt RC 7-3 checks the wrong Gradle module

Expected Behavior
When running :moduleA:detektCheck, Detekt checks that module
Current Behavior
When running :moduleA:detektCheck, Detekt checks that module, not :moduleZ (the last one alphabetically)
Steps to Reproduce (for bugs)
Have a Gradle project with several modules, and run :moduleA:detektCheck. Verify that it only finds issues, and writes reports to, moduleZ, the last in alphabetical order
Context
Self-explainatory I think
Your Environment

Version used: RC7-2, RC7-3
Operating System and version: macOS 10.13.5
Link to your project: N/A (private project)\

Not sure if it matters, but the project contains a composite build (references another project located in ../other-project)

The problem is that the defaultProfile is a mutable singleton.
Workaround:
Do not use the default profile. Either
subprojects {
    apply plugin: "io.gitlab.arturbosch.detekt"
}

or
subprojects {
    apply plugin: "io.gitlab.arturbosch.detekt"
    profile('myprofile') {
        input = file("src/main/kotlin")
        filters = ".*/resources/.*,.*/build/.*"
    }
}

What is the reason for using a singleton in the first place?

@rock3r can you please test out if the branch changes work for you?

What branch @arturbosch?


New Send screen - Fee in summary section should be converted from groth to beam

If a user enters a fee = 1001000 Groth in the summary section it shows 1 001 000 BEAM
Should be 0.01001 BEAM

checked


Module not found: Can't resolve '@dqbd/tiktoken' in '/vercel/path0/node_modules/chatgpt/build'

Verify latest release

[X] I verified that the issue exists in the latest chatgpt release

Verify webapp is working

[X] I verify that the ChatGPT webapp is working properly for this account.

Environment details
vercel build environment
Describe the Bug
when I build my demo in vercel it always build fail.
Module not found: Can't resolve '@dqbd/tiktoken' in '/vercel/path0/node_modules/chatgpt/build'

This may be related: https://github.com/transitive-bullshit/chatgpt-api/issues/413
@dqbd/tiktoken uses WASM, which may not be handled correctly in some environments. Can you give me as much detail about your "vercel build environment" as possible? e.g., is it a serverless function or edge function? Is this from next.js and if so, what version? Are you using appDir? etc.

Thanks. I  know that the problem is this api can only run in node.js client  not in browser


Change UNKNOWN_CODE_PLEASE_REPORT to identify uknown code

Changes UNKNOWN_CODE_PLEASE_REPORT to include packet.errno in it.
This makes it easier to identify what the code is even if the code is not yet supported:
(node:1) UnhandledPromiseRejectionWarning: Error: UNKNOWN_CODE_PLEASE_REPORT: Query execution was interrupted, query_timeout exceeded

@dougwilson 19aa86d


vsc.unique constraint on list with 1 item raises Boolector Exception

If a randomizable list is constrained with vsc.unique and that list has a length of 1, then VSC throws a BoolectorException during randomize().  This is a problem because my application allows the list length to change between randomize() calls, and I'd rather not check for the special case of list.size == 1.  The example below demonstrates the issue.
import vsc

@vsc.randobj
class Selector:
    def __init__(self):
        self.selectedList = vsc.rand_list_t(vsc.uint16_t(), 3)

    @vsc.constraint
    def list_c(self):
        vsc.unique(self.selectedList)

selector = Selector()
selector.selectedList.clear()
selector.selectedList.extend([0, 0, 0])
selector.randomize()         # 3-item list passes randomization.
print(f"Selected List:  {selector.selectedList}")
selector.selectedList.clear()
selector.selectedList.extend([0, 0])
selector.randomize()         # 2-item list passes randomization.
print(f"Selected List:  {selector.selectedList}")
selector.selectedList.clear()
selector.selectedList.extend([0])
selector.randomize()         # 1-item list throws BoolectorException.
print(f"Selected List:  {selector.selectedList}")

Output from example:
Selected List:  [65292, 63129, 1522]
Selected List:  [45353, 19828]
Exception: selectedList
Traceback (most recent call last):
  File "/usr/lib64/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/lib64/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/users/mcgrathm/.vscode-server-insiders/extensions/ms-python.python-2021.12.1559732655/pythonFiles/lib/python/debugpy/__main__.py", line 45, in <module>
    cli.main()
  File "/users/mcgrathm/.vscode-server-insiders/extensions/ms-python.python-2021.12.1559732655/pythonFiles/lib/python/debugpy/../debugpy/server/cli.py", line 444, in main
    run()
  File "/users/mcgrathm/.vscode-server-insiders/extensions/ms-python.python-2021.12.1559732655/pythonFiles/lib/python/debugpy/../debugpy/server/cli.py", line 285, in run_file
    runpy.run_path(target_as_str, run_name=compat.force_str("__main__"))
  File "/usr/lib64/python3.6/runpy.py", line 263, in run_path
    pkg_name=pkg_name, script_name=fname)
  File "/usr/lib64/python3.6/runpy.py", line 96, in _run_module_code
    mod_name, mod_spec, pkg_name, script_name)
  File "/usr/lib64/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/data/cf/sshot/u/mcgrathm/silicon-validation/experiment/pyvsc/constrDebug18.py", line 56, in <module>
    selector.randomize()         # 1-item list throws BoolectorException.
  File "/data/cf/sshot/u/mcgrathm/silicon-validation/.venv/lib64/python3.6/site-packages/vsc/rand_obj.py", line 168, in randomize
    solve_fail_debug=solve_fail_debug)
  File "/data/cf/sshot/u/mcgrathm/silicon-validation/.venv/lib64/python3.6/site-packages/vsc/model/randomizer.py", line 597, in do_randomize
    r.randomize(ri, bounds_v.bound_m)
  File "/data/cf/sshot/u/mcgrathm/silicon-validation/.venv/lib64/python3.6/site-packages/vsc/model/randomizer.py", line 196, in randomize
    raise e
  File "/data/cf/sshot/u/mcgrathm/silicon-validation/.venv/lib64/python3.6/site-packages/vsc/model/randomizer.py", line 193, in randomize
    btor.Assume(c[1])
  File "src/pyboolector.pyx", line 837, in pyboolector.Boolector.Assume
pyboolector.BoolectorException: [pybtor] Argument at position 0 is not a BoolectorNode

Any case that results in a Boolector exception is a bug and shouldn't require user-level workarounds. I'll have a look. Thanks for reporting!

I've corrected this with the 0.6.9 release. Unique constraints on total lists of <1 now always hold (always eval to 'true')


Dialogflow API: dialogflow.googleapis.com

Note that this API already has proto libraries in external/googleapis, it may need some refactoring.  In addition, it has a bidirectional streaming RPCs:

google.cloud.dialogflow.cx.v3.Sessions.StreamingDetectIntent
google.cloud.dialogflow.v2.Sessions.StreamingDetectIntent

cat  ~/googleapis/api-index-v1.json | jq '.apis | map(select(.services != null)) | map(.services | map(select(.methods != null)) | map(.methods | map(select(.mode == "BIDIRECTIONAL_STREAMING") | [.mode, .fullName]  ))) | flatten ' | grep -v BIDIRECTIONAL_STREAMING  | grep -v beta
[
  "google.cloud.apigeeconnect.v1.Tether.Egress",
  "google.cloud.bigquery.storage.v1.BigQueryWrite.AppendRows",
  "google.cloud.dialogflow.cx.v3.Sessions.StreamingDetectIntent",
  "google.cloud.dialogflow.v2.Sessions.StreamingDetectIntent",
  "google.cloud.mediatranslation.v1alpha1.SpeechTranslationService.StreamingTranslateSpeech",
  "google.cloud.pubsublite.v1.CursorService.StreamingCommitCursor",
  "google.cloud.pubsublite.v1.PartitionAssignmentService.AssignPartitions",
  "google.cloud.pubsublite.v1.PublisherService.Publish",
  "google.cloud.pubsublite.v1.SubscriberService.Subscribe",
  "google.cloud.speech.v1.Speech.StreamingRecognize",
  "google.devtools.build.v1.PublishBuildEvent.PublishBuildToolEventStream",
  "google.firestore.v1.Firestore.Listen",
  "google.firestore.v1.Firestore.Write",
  "google.logging.v2.LoggingServiceV2.TailLogEntries",
  "google.pubsub.v1.Subscriber.StreamingPull"
]

I think this is blocked by #8459