image not show with a base64 encoded byte array source.

I have something like this:
<template>

</template>

From the page source I see HTML block seems to be correct.. but can't display the image and log an error Failed to load resource: net::ERR_INVALID_URL in the console.

Hi, thanks for your interest but Github issues are for bug reports and feature requests only. You can ask questions on the forum, the Discord server, gitter or StackOverflow.


Type guards do not understand unions of array types

Try the following:
function workWithXArray(array: x[]) { }
function workWithYArray(array: y[]) { }

function workWithArray(array: (x|y)[]) {
    if (array.length > 0) {
        let value = array[0];

        if (value instanceof x) {
            workWithXArray(array);
        } else if (value instanceof y) {
            workWithYArray(array);
        }
    }
}

It will throw error TS2345: Argument of type '(x| y)[]' is not assignable to parameter of type 'x[]'

The following code can be compiled by tsc version 1.8.5:
class X {}
class Y {}

function workWithXArray(array: X[]) { }
function workWithYArray(array: Y[]) { }

function workWithArray(array: (X|Y)[]) {
    if (array.length > 0) {
        let value = array[0];

        if (value instanceof X) {
            workWithXArray(array);
        } else if (value instanceof Y) {
            workWithYArray(array);
        }
    }
}

Which version of tsc are you using?

By the way, I am not sure if (X|Y)[] is the intended type of the "array" parameter. I think it should be X[]|Y[]. The former one appears to be an array of the union type of X and Y while the latter one means either an array of X or an array of Y.

I am not sure if (X|Y)[] is the intended type of the "array" parameter. I think it should be X[]|Y[]. The former one appears to be an array of the union type of X and Y while the latter one means either an array of X or an array of Y.
The following code appears to work:
class X {
	public x;
}
class Y {
	public y;
}

function workWithXArray(array: X[]) { }
function workWithYArray(array: Y[]) { }

function workWithArray(array: X[] | Y[]) {
    if (array.length > 0) {
        let value = array[0];

        if (value instanceof X) {
            workWithXArray(array as X[]);
        } else if (value instanceof Y) {
            workWithYArray(array as Y[]);
        }
    }
}


chore(devtools): RHIF-13 default proxy to stage env

Codecov Report

Merging #30 (f04c502) into master (8a991c6) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #30   +/-   ##
=======================================
  Coverage   11.86%   11.86%
=======================================
  Files          34       34
  Lines         337      337
  Branches       51       51
=======================================
  Hits           40       40
  Misses        288      288
  Partials        9        9

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 8a991c6...f04c502. Read the comment docs.


Refactor ProductList Logic into Separate Component

📘 Description
The logic for ProductList needs to be refactored into a reusable component to improve maintainability and structure. The products should be moved into a separate file for better data management, and a dedicated folder for type definitions should be created to ensure consistency across the application.
🔢 Steps

Move the ProductList logic (currently in the Marketplace page) to a new file named product-list.tsx inside the components/marketplace directory.
Create the components/marketplace folder if it doesn’t already exist.
Move all products data to a new file named placeholder-data.ts inside the lib directory.
Create a new folder named types inside the app directory and add a file index.ts.
Copy the Product interface into types/index.ts.
Update all relevant imports to use the refactored component, product data, and type definitions.
Validate that the application functions as expected after the changes.

✅ Acceptance Criteria

The ProductList logic should be fully refactored into a reusable component.
Product data should be organized in the placeholder-data.ts file.
Type definitions should be centralized in types/index.ts.
The application should function correctly after the changes.

🌎 References
Not applicable.
📜 Additional Notes
⚠ Do not apply until the ODHack begins.

Can I take up this issue? I have experience working on various typescript(and some react) projects. I will follow all the steps and make a valid PR.

Hey! I'm SimplementeCao. 👋
I have contributed before to other projects like:

Cairo Book
Starknet Book
Starknet Docs
Starknet Security Challenges Factory
Starkane

Can I work on this issue?
Check my OnlyDust, I've contributed to many projects: OnlyDust Profile,
and my GitHub

Plan to Solve the Issue:

Analyze the codebase to understand the current structure and functionality.
Review the problem description that needs to be solved by thoroughly reading the issue details.
Leverage all provided resources within the issue to get a better grasp of the task at hand.
If I face any challenge, I will ask questions promptly in the relevant support group or communication channel to move forward efficiently.
Once resolved, I will open a PR and wait for your feedback.

Let me know if I can start working on this! 🚀

I would like to work on this issue

I am a Full Stack Developer with a strong background in blockchain and extensive experience in Next.js/React, TypeScript, and Rust. I’ve made over 49 contributions to over 17 projects in the OnlyDust ecosystem, solving complex issues and delivering efficient, scalable solutions.
I can handle this task.
ETA is 3days.

Hi, i am Emmanuel Dorcas, and i will like to tackle this issues
I am software developer in React js and Typescript, I have three years experinece
my estimated time is 2days
how I will tackle this:
I will Move the ProductList logic to a new file named product-list.tsx
I will Create the components/marketplace folder if it doesn’t already exist.
i will Move all products data to a file named placeholder-data.ts inside the lib directory.
i will create the Product interface  inside a file called index,ts in types folder
I will update all relevant imports to use the refactored component, product data, and type definitions.
i will makes sure to confirm that the application functions are working accordingly

Hii! My name is Mario Araya, I’m a Software Developer with 2+ years of experience. I have worked with backend technologies such as Cairo, Java, and C#, as well as frontend development using React, NextJS, and JavaScript/TypeScript. I’ve made contributions to open-source projects, completed a Starknet Bootcamp, exercises on NodeGuardians, finished Starklings, and participated in multiple hackathons. I’m also a member of the Dojo Coding community here in Costa Rica.
I will  start by creating a new product-list.tsx file inside a components/marketplace directory and move the relevant logic from the Marketplace page into this file, ensuring it is reusable by accepting props like products. Next, create a placeholder-data.ts file in a lib directory to store and export the product data separately. Then, centralize type definitions by creating a types folder in the app directory with an index.ts file, where the Product interface will be defined and exported. Update all relevant imports in the application to use the new component, data file, and type definitions, and validate that the application works correctly after these changes.

@danielcdz thanks hommie


Update JSON schema to move Annotations to an array directly under the Document

Depending on the outcome of the SPDX Spec issue https://github.com/spdx/spdx-spec/issues/517 this PR will update the JSON schema to move annotations from properties under SpdxElement to a separate array directly under SpdxDocument (similar to how we treat Relationships).
Signed-off-by: Gary O'Neall<EMAIL_ADDRESS>

This is resolved in PR #167


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 875 ms

Resolved: Bofrost.at performance has improved in b538c10 after 9 minutes.


Giving error when passing "environment" option to compose.

After giving the adding environment: VERSION: "win7" to docker it gives me this error: PATH_TO_GIT_REPO/docker-compose.yml: (root) Additional property environment is not allowed. Here's the docker-compose file if needed:
version: "3"

services:
  windows:
    image: dockurr/windows
    container_name: windows
    devices:
      - /dev/kvm
    cap_add:
      - NET_ADMIN
    ports:
      - 8006:8006
      - 3389:3389/tcp
      - 3389:3389/udp
    stop_grace_period: 2m
    restart: on-failure

environment:
  VERSION: "win7"

I don't have any experience with docker. So, this might be a pretty silly mistake. Please don't get angry if so.
Thanks.

Ok... It now works, Thanks.


Feature: reload endpoint for hot load changes from config file

In this PR an endpoint is added for loading hot changes without restarting the container for #346 . Just making POST request to /-/reload latest config changes are available in metrics.
Let me know if any changes are required in this.

@antoniomerlin If you get this fixed till Thuersday morning, I get this into the next release.
Else I schedule it for next week :)
Greetings :)

Hi i removed that reload as separate function and added directly inside http handlefunc, as for change requested to move this function inside config.go currently the way i am doing this is loading the config file again and calling the scrape function for making the config changes available in next metric scrape.
log.Println("Parse config..")
if err := config.Load(configFile); err != nil {
	log.Fatal("Couldn't read ", *configFile, ": ", err)
}
if *decoupledScraping {
	s.decoupled(ctx)
} else {
	s.scrape(ctx)
}

What are your thought about this approach


Fix/error redirect

Fixes #1574

[x] server redirect on errors
[x] server redirect on 404
[x] client redirect on errors
[x] client redirect on 404
[x] client redirect only on browser (edgy)

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpx changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

I'm trying to handle the "edgy" case, which is client redirection on start, when no SSR is active.
It happens here:
https://github.com/sveltejs/kit/blob/c3ca3fbb05b208af4a30c8b311074fd95d161304/packages/kit/src/runtime/client/renderer.js#L191-L196
But in the end, i don't believe this is a "real" case in this PR, since by that point, the node is already loaded correctly with the work done in the previous commits.

yeah, it looks like you've probably covered all the cases. basically just need to rebase and add tests at this point

The nested __error.svelte page i added for redirections is not taken into account, basics/routes/__error.svelte is instead, which makes my simple test fail, obviously...

Actually, #1576 is the reason why my test does not work :/

One thing I realized is that people can actually handle the 404 case themselves today. You can just create a file like src/routes/[zzz_catchall].svelte:
<script context="module">
	/**
	 * @type {import('@sveltejs/kit').Load}
	 */
	export async function load() {
		return {
			status: 307,
			redirect: '/'
		};
	}
</script>

One thing I realized is that people can actually handle the 404 case themselves today. You can just create a file like src/routes/[zzz_catchall].svelte:

That's indeed an alternative. But I think in my case I'd rather handle it the __error page.

All tests should be passing now. I'm not sure why deploys are failing.

Deploy failures are unrelated to your change. We need to fix. I think Netlify is caching old versions of the Netlify adapter

Thank you so much for all the effort you put into this PR. I talked about it with the rest of the maintainers at today's maintainer's meeting and the consensus was that isn't something folks wanted to add. I put more details about that on the issue here: https://github.com/sveltejs/kit/issues/1574#issuecomment-919336290
I'm terribly sorry as I know this must be a disappointing outcome given all the effort you've put into this PR and it would have been a lot nicer to have clarified that up front. I really appreciate everything you did to move this forward and help us get to a resolution on it though! Even though this didn't get merged it is nice that we now have an answer for folks on how they can accomplish this and we were able to knock an item off the 1.0 list thanks to the discussion this spurred

I'd be lying if i said i'm not a little bit disappointed, but I'll get over it :)
Happy to help, for sure I'll try to find another issue to knock down, maybe something more consensual :D

that would be awesome. I went through and labeled the issues with "help wanted" where I think we're fairly agreed on a solution and it just needs to be implemented
thanks again for all the help in looking at this one!


pypi2nix: 2.0.1 -> 2.0.2

Since the source distribution of the package on pypi2nix does not contain unittests,
we removed the checkPhase command that was present in earlier versions of the package.
Motivation for this change
A new version of pypi2nix was released to PyPI.  This pull request updates nixpkgs to distribute that version.
Closure size grew slightly since jsonschema was introduced as an additional dependency.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

@Lassulus thank you :)


Possible mistake at Insert records from a path into an existing table section

Under "Insert records from a path into an existing table" there is a note in the Important section:
"The bucket path where you create a table cannot also be used to read or write data files"
I think it is wrong: data is copied from a file into an existing table. New table is not created in this case, so this statement above does not make sense under this section

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

ID: 11f523ea-346d-42a9-da71-cd5779e9d5ff
Version Independent ID: 11f523ea-346d-42a9-da71-cd5779e9d5ff
Content: Create tables - Azure Databricks
Content Source: databricks/data-governance/unity-catalog/create-tables.md
Service: azure-databricks
GitHub Login: @mssaperla
Microsoft Alias: saperla

@mihaly-hazag
Thanks for your feedback! We will investigate and update as appropriate.

@mihaly-hazag
I'm going to assign this to the document author so they can take a look at it accordingly.
@mssaperla
please can you review this.

Thanks for providing feedback that helps improve our documentation. We've created an internal work item (DOC-9221) to address your feedback. The timeline for resolution varies based on resourcing.
#please-close


[Day One] Day One CLI reported as missing even after installing

Extension
https://www.raycast.com/AntonNiklasson/day-one
Description
I installed the Day One extension, and when I attempted to use its one command ("Add Entry"), I get an error:

After installing the Day One CLI , the error message persists.
Steps To Reproduce
Apple M1 Max, macOS Sonoma 14.1.2 (23B92)

Install Day One extension
Attempt to run Add entry command
Get error to install Day One CLI
Install Day One CLI
Still get error.

Current Behaviour
See above.
Expected Behaviour
An entry should be able to be created.

Thank you reporting! I'll try to take some time tomorrow to fix this.


ORC-1016: Use<EMAIL_ADDRESS>in GitHub Action MacOS CIs

What changes were proposed in this pull request?
This PR aims to recover GitHub Action by using<EMAIL_ADDRESS>because GitHub Action macOS image does not ship openssl recently.
Why are the changes needed?
GitHub changed its openssl install version.

[MacOS] Do not install latest OpenSSL
Updating readme file for macOS-10.15 version 20210927.1

How was this patch tested?
Pass the GitHub Action.

cc @autumnust , @guiyanakuang , @williamhyun , @omalley

This is still Draft because we need to verify this via GitHub Action run on this PR.

This is ready for review. Could you review this please?

+1 LGTM

Thank you so much, @omalley and @williamhyun !


fix Err.Timestamp

fix Err.Timestamp

fix: https://github.com/RobotsAndPencils/buford/issues/37

Thanks for submitting a pull request.

merged. thanks again!


Functions being wrapped in Brackets

Issue Type: Bug
When typing functions native to SQL Server (I haven't tested with User Scalar functions, as I don't tend to use them) and using Tab to complete the name, the function is being wrapped in Brackets ([]), instead of having parentheses (()) added at the end and the cursor being put inside them.
Wrapping them in Brackets invalidates the function, as it would mean it is treated as an object name, not a function.
You can see this in the below animated image:

Edit: Note this was not a "feature" (bug) on previous versions of ADS. I did have ADS 1.8 installed on Windows, however, as a test I updated it to 1.12.1 (released yesterday at time of edit) 1.12.0 and it now too has the above "feature".
Azure Data Studio version: azuredatastudio 1.10.0 (5fdb967ffcea41df0335ec17cfa396a694bb174e, 2019-08-14T18:03:49.463Z)
OS version: Linux x64 5.0.0-31-generic

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8700 CPU @ 3.20GHz (12 x 4299)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwareoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: disabled_softwareskia_deferred_display_list: disabled_offskia_renderer: disabled_offsurface_synchronization: enabled_onvideo_decode: unavailable_offviz_display_compositor: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
2, 1, 1

Memory (System)
15.58GB (6.42GB free)

Process Argv
--unity-launch

Screen Reader
no

VM
0%

Extensions (7)

Extension
Author (truncated)
Version

vsc-material-theme
Equ
30.0.0

admin-pack
Mic
0.0.2

agent
Mic
0.43.0

dacpac
Mic
0.6.0

profiler
Mic
0.9.0

material-icon-theme
PKi
3.9.0

sql-search
Red
0.3.4

Seems to have no been fixed in 1.12.1 (I had installed 1.12.0 when I made the above edit, not 1.12.1)

I'm reopening this issue, as it appears to be intermittent, and I am getting the behaviour in 1.12.1.

@LarnuUK I haven't seen this before.  Do you happen to have any extensions, such as Postgres, installed?  Or any customized editor settings?

Hi @kburtram , all the extensions installed are listed in my original post (which I believe are the same both on Ubuntu and Windows. PostgreSQL isn't one of those though.

I have the same issue  in 1.13.0
I have 3 extentions:

Extension
Author (truncated)
Version

import
Mic
0.12.0

azdata-sanddance
msr
1.3.9

poor-sql-formatter
WSR
0.1.0


feat: Exclusion in collector pods

What this PR does / why we need it:
Collector pods reference excluded configmap to before adding images to final list of images for imagecollector-shared.
Which issue(s) this PR fixes (optional, using fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when the PR gets merged):
Fixes #274
Special notes for your reviewer:

LGTM, going to rebase then merge when checks pass
Actually, now that I think of it, we could use an e2e test with scanner disabled that checks the imagecollector-shared to ensure excluded items don't show up there.


use nrgba instead of rgba image

Commit 111554b fixed this for decoding, but the encoding Go code still says:
// extract pixel
c := m.At(x, y)
c_r, c_g, c_b, c_a := c.RGBA()
px := pixel{byte(c_r >> 8), byte(c_g >> 8), byte(c_b >> 8), byte(c_a >> 8)}

The c.RGBA() means that you're encoding premultiplied alpha (as per the Color.RGBA godocs), not non-premultiplied alpha (what QOI uses).


Running StructureBoost.fit gives "Buffer dtype mismatch, expected 'long' but got 'long long'"

On my computer, when I run the notebook LN200806_StructureBoost_Intro, code cell 22
# We use the `eval_set`, `early_stop_past_steps` and `eval_freq` to define early stopping
stboost_CA_def.fit(X_train, y_train,
                   eval_set = ((X_valid, y_valid)),
                   early_stop_past_steps=1,
                   eval_freq=20)

gives the following error:
i=0, eval_set_log_loss = 0.47018127601084786
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-22-77dc75dea0eb> in <module>
      3                    eval_set = ((X_valid, y_valid)),
      4                    early_stop_past_steps=1,
----> 5                    eval_freq=20)

structure_gb.pyx in structure_gb.StructureBoost.fit()

structure_dt.pyx in structure_dt.StructureDecisionTree.fit()

structure_dt.pyx in structure_dt.StructureDecisionTree._process_tree_node()

structure_dt.pyx in structure_dt.evaluate_feature()

structure_dt.pyx in structure_dt._evaluate_feature_numerical()

structure_dt.pyx in structure_dt._evaluate_numerical_splits()

structure_dt.pyx in structure_dt.get_bin_sums_c()

ValueError: Buffer dtype mismatch, expected 'long' but got 'long long'

I ran this notebook on my computer today.  Also today (8/6/20), I set up a new Anaconda environment with a fresh install of StructureBoost and associated packages.  My computer is a virtual desktop installation of Windows.
I also ran this notebook on Colab but it did not have a problem.

Hmmm... that's a bummer.  I've not tested at all with Anaconda / Windows.  This is an issue with Cython -- it is picky about the "variant" of integer.  It is expecting np.int32 or np.int64, but is getting a format to handle (much) larger integers.  Python handles these "upgrades" elegantly, but the C part of Cython does not.
Can you run
df_ca_PRCP.info()
and tell me the results?
You can also try running the lines
df_ca_PRCP['month'] = df_ca_PRCP['month'].values.astype(np.int64)
df_ca_PRCP['rained'] = df_ca_PRCP['rained'].values.astype(np.int64)
immediately after loading in the data set and seeing if that helps

I just pushed a new version - 0.0.11.  It should fix that particular bug, though something else may come up.  There may be other platform dependent issues.

Let me know if it works now - otherwise we (I) can keep digging.

Brian,
Thanks for this.  Will do.  I have a couple pressing tasks, but should be
able to do in the next couple days.
Mike
On Fri, Aug 7, 2020 at 3:28 PM blucena<EMAIL_ADDRESS>wrote:

Let me know if it works now - otherwise we (I) can keep digging.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/numeristical/resources/issues/1#issuecomment-670676789,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMJKAUJ2UFYSJONXFNGDW73R7RIWPANCNFSM4PW6QPOA
.

--
Best Regards,
Michael Herzog
(908) 809-3903

Brian,
I finally got a chance to look at this.  I installed a new version of structureboost and reran the notebook.  I got a different but similar error:
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-20-77dc75dea0eb> in <module>
      3                    eval_set = ((X_valid, y_valid)),
      4                    early_stop_past_steps=1,
----> 5                    eval_freq=20)

structure_gb.pyx in structure_gb.StructureBoost.fit()

structure_dt.pyx in structure_dt.StructureDecisionTree.predict()

structure_dt.pyx in structure_dt.get_prediction()

ValueError: Buffer dtype mismatch, expected 'long' but got 'long long'

Running stb.__version__ gives '0.0.11'.
Here is the output of the commands you requested:
df_ca_PRCP.info():
<class 'pandas.core.frame.DataFrame'>
RangeIndex: 415930 entries, 0 to 415929
Data columns (total 7 columns):
 #   Column       Non-Null Count   Dtype
---  ------       --------------   -----
 0   county       415930 non-null  object
 1   month        415930 non-null  int64
 2   DATE         415930 non-null  object
 3   STATION      415930 non-null  object
 4   PRCP         415930 non-null  float64
 5   rained       415930 non-null  int64
 6   county_DATE  415930 non-null  object
dtypes: float64(1), int64(2), object(4)
memory usage: 22.2+ MB

I tried running the commands you suggested:
df_ca_PRCP['month'] = df_ca_PRCP['month'].values.astype(np.int64)
df_ca_PRCP['rained'] = df_ca_PRCP['rained'].values.astype(np.int64)

before running the .fit cell, but it still failed with the same error.

Version 0.12 fixed the problem for me.  The notebook worked flawlessly.  Thank you!


Auto scroll on sortable sort

This is a (multiple allowed):

[ ] bug

[x] enhancement

[ ] feature-discussion (RFC)

Framework7 Version: LATEST

Platform and Target: All

It would be nice if the sortable list will auto scroll the page if there are more items than the page height and you would like to drag an item to a location which isn't visible.
I hope @nolimits4web understand ;)

@nolimits4web
Yeah I have the same problem. Would be nix of you improve that :)

Closing as it is done in v2


Question regarding self-checkout queues

I understand that that customers only queue at the first self-checkout counter, but what is the max queue size of this counter?
It this qmax? or is this qmax times the number of SCC's?

the way i designed my queue was that when looping through your queue, looping until the first self check out and handling the queues just at this first self check out. the q max would just be the same as the rest not times the number of SCCs


Add a build ARG for installer download source

Currently the installer dockerfile is hard-coded to always use pre-release.racket-lang.org as the source website of installers and /installers as a root resource for specially formatted installer filenames. This should be configurable so the images can test Racket's regular releases and historical installer archives in addition to the prerelease installers.

Yes, this would be good because some things are fixed on HEAD.


[Doc]:  Add Kubernetes (K8s) Deployment Documentation for vLLM with GPUs

📚 The doc issue
Context:
I am currently managing H100 GPUs using Kubernetes (K8s). However, I’ve noticed that the vLLM documentation only provides deployment instructions for Docker, which is quite different from K8s. This creates a gap for users like me who rely on K8s for managing our infrastructure.
Issue:
Lack of Kubernetes (K8s) deployment documentation for vLLM.
Existing documentation focuses solely on Docker, leaving out crucial information for K8s users.
The differences between Docker and K8s make it challenging for users to adapt the Docker-based instructions for a K8s environment.
Proposal:
I suggest expanding the current vLLM documentation to include detailed instructions for deploying vLLM in a Kubernetes (K8s) environment. This should cover:
Setting up the environment to manage GPUs using K8s.
Step-by-step deployment instructions tailored for K8s.
Configuration and optimization tips specific to K8s.
Additional Details:
As a K8s user managing H100 GPUs, I am willing to contribute by sharing more details on how to handle these tasks. This could include:
Sample YAML files for deployment.
Configuration tips and best practices.
Troubleshooting common issues encountered during deployment in K8s.
Suggest a potential alternative/fix
No response
Before submitting a new issue...

[X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

Contribution welcomed!

@simon-mo PTAL the related PR for deploying_with_k8s guide


Fix create pr

This PR does some more cleanup and adds the ID and guiurl to a newly created or updated PR. I needed to add some minimal JSON marshaling and unmarshaling to parse the ID and guiurl.
I removed unused files.
I also updated the documentation and commandline options to mention bitbucket cloud configuration options and moved our documentation to the correct template section.

reviewers note: to be merged after the cleanup PR https://github.com/asecurityteam/multi-gitter/pull/5

Now that we include the ID we can see it after creating a PR:
 Megabees/kvstore-aws-accounts #140
Previously it would only show 0 and not the ID of the created PR.


Create HelpSuggestionCommand and HelpSuggestionCommandTest

Closes #203
This PR works on:

[x] HelpSuggestionCommand
[x] HelpSuggestionCommandTest

Hey @johannagwan , I don't think you need path as a field in your HelpSuggestionCommand. Otherwise, looks good

What I mean is you can delete your path field and everything will still work fine


Fix typo in JobDetail interface

In submitting this contribution, I agree to the current Software AG contributor agreement as referred to here:
https://github.com/quartz-scheduler/contributing/blob/main/CONTRIBUTING.md
This PR...
Changes

Name of the method isConcurrentExectionDisallowed to isConcurrentExecutionDisallowed in JobDetail

Checklist

[x] tested locally
[x] updated the docs
[x] added appropriate test
[x] signed-off on the above mentioned SoftwareAG contributor agreement via git commit -s on my commits.
(If you're not using command-line, you can use a browser extension )

Fixes

#953
A typo in the name of method isConcurrentExectionDisallowed under JobDetail interface

Unfortunately, due to the change of ownership of this project, the terms of contribution have been updated, and this PR will need to be re-created, agreeing to the new terms.


Numba issues while importing UMAP

I have installed umap-learn and all its requirements, such as numba. However, when importing umap it raises an error related to numba experimentals. I am not being able to solve it.
The error is displayed in the image.
Thanks in advance

After installing UMAP with condaforge,I got a similar problem :
import umap
Traceback (most recent call last):
File "", line 1, in   import umap
File "/home/fenaux/anaconda3/lib/python3.7/site-packages/umap/init.py", line 2, in  from .umap_ import UMAP
File "/home/fenaux/anaconda3/lib/python3.7/site-packages/umap/umap_.py", line 32, in <module|> import umap.sparse as sparse
File "/home/fenaux/anaconda3/lib/python3.7/site-packages/umap/sparse.py", line 12, in   from umap.utils import norm
File "/home/fenaux/anaconda3/lib/python3.7/site-packages/umap/utils.py", line 39, in  @numba.njit("i4(i8[:])")
File "/home/fenaux/anaconda3/lib/python3.7/site-packages/numba/core/decorators.py", line 218, in wrapper    disp.compile(sig)
File "/home/fenaux/anaconda3/lib/python3.7/site-packages/numba/core/compiler_lock.py", line 32, in _acquire_compile_lock
return func(*args, **kwargs)
File "/home/fenaux/anaconda3/lib/python3.7/site-packages/numba/core/dispatcher.py", line 776, in compile
def _rebuild(cls, uuid, py_func, locals, targetoptions, impl_kind,
File "/home/fenaux/anaconda3/lib/python3.7/site-packages/numba/core/sigutils.py", line 24, in normalize_signature
parsed = _parse_signature_string(sig)
File "/home/fenaux/anaconda3/lib/python3.7/site-packages/numba/core/sigutils.py", line 14, in _parse_signature_string
return eval(signature_str, {}, types.dict)
File "", line 1, in
File "/home/fenaux/anaconda3/lib/python3.7/site-packages/numba/core/types/abstract.py", line 187, in getitem
return Array(dtype=self, ndim=ndim, layout=layout)
File "/home/fenaux/anaconda3/lib/python3.7/site-packages/numba/core/types/abstract.py", line 66, in call
inst = type.call(cls, *args, **kwargs)
File "/home/fenaux/anaconda3/lib/python3.7/site-packages/numba/core/types/npytypes.py", line 411, in init
super(Array, self).init(dtype, ndim, layout, name=name)
TypeError: super(type, obj): obj must be an instance or subtype of type

I can't say for sure, but I suspect you need a newer version of numba (0.51 or newer should do the job). I can't say why that didn't get handled by the requirements / dependencies listed for umap-learn, but it is the most likely explanation.

Thank for your answer
Here are the package installed by conda-forge
llvmlite-0.34.0      | 297 KB    | ##################################### | 100%
certifi-2020.12.5    | 143 KB    | ##################################### | 100%
umap-learn-0.5.0     | 122 KB    | ##################################### | 100%
ca-certificates-2020 | 137 KB    | ##################################### | 100%
numba-0.51.2         | 3.6 MB    | ##################################### | 100%
pynndescent-0.5.1    | 41 KB     | ##################################### | 100%
libllvm10-10.0.1     | 26.4 MB   | ##################################### | 100%
So numba version satisfies your recommendation.
If I try to install numba with conda numba0.52 is not installed. Probably no fit my environment.

It looks like conda-forge is doing the right thing, so that's all fine. Admittedly the conda-forge error you posted is a little different, so perhaps we are getting closer to getting it working for you. It looks like it is having trouble compiling the norm function. I'm not sure why that should be the case -- it has been in the code that way for a long time, and this is the first time I've heard of an error like that. Regardless, it is clearly stopping things working for you. One potential approach is to remove the signature on that particular line. I.e. edit the file: /home/fenaux/anaconda3/lib/python3.7/site-packages/umap/utils.py at line 39 from:
@numba.njit("i4(i8[:])")

to
@numba.njit()

and see if that helps. This may be simply pushing the problem to elsewhere. You may also want to try asking the numba people if they have seen this sort of error before. I would reccommend their gitter channel, where you may be able to get a relatively swift response -- at least as to whether they've seen anything quite like this before.

I got this error
ImportError: cannot import name 'tau_rand_int' from 'umap.utils' (/home/fenaux/anaconda3/lib/python3.7/site-packages/umap/utils.py)

It seems like there is something broken with your installation generally then :-( I am really not quite sure what the best way to diagnose it is. There are some issues right now related to numpy, but it should not be affecting numba in this way. I admit that I am at a loss, and don't really have any more suggestions right now. Sorry.


enter-chroot: Unmount /sys/fs/selinux, remount fake directory

Fixes #2655.
We first unmount /sys/fs/selinux, which fixes a number of issues
(but not all).
Then, we create a fake, read-only, /sys/fs/selinux tmpfs, with
its enforce flag set to 0. This is necessary for precise to be able
to add users.
@rickyz: Any comment on this?

Anecdotally, on the noxephyr (rebased on top of the wayland branch), I was running into strange password prompts when enter-chroot was attempting to run groupadd to create groups.  This patch fixed it.

Oops sorry, I've been slow on this - sounds reasonable to me, thanks for the fix!


Pcos Detection

Pull Request for ML-Crate 💡
Issue Title: [PROJECT ADDITION]- PCOS DETECTION

Info about the related issue (Pcos Detection ) : helps to detect that the person is suffering from Pcos or not.
Name: Payal SInha
GitHub ID: https://github.com/payalsinha12
Idenitfy yourself: A Web Developer and Graphic Designer

Closes: #624
Describe the add-ons or changes you've made 📃
I made a project on Pcos Detection using the dataset taken from Kaggle.
Type of change ☑
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested? ⚙
Describe how it has been tested
Describe how have you verified the changes made
Checklist: ☑

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[ ] I have commented my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added things that prove my fix is effective or that my feature works.
[ ] Any dependent changes have been merged and published in downstream modules.

https://github.com/abhisheks008/ML-Crate/tree/main/ASELSAN Stock Prices
Follow the project structure and put your files.

you told me to add different visualization images inside the README file. the images present inside the model folder is actually present in the README file now you are asking to delete the pictures what should i do ?? and i removed the desktop.ini file @abhisheks008

you told me to add different visualization images inside the README file. the images present inside the model folder is actually present in the README file now you are asking to delete the pictures what should i do ?? and i removed the desktop.ini file @abhisheks008

If you remove the images from the Model folder and put them inside the Images folder, you can still hyperlink the images and use them inside the README irrespective of the folder.

@abhisheks008 hey please review this

You haven't fixed this.

You haven't fixed this.

Create a table for showing the accuracy scores, it's not feasible to show the accuracy scores like this. It's a minor markdown change you need to do.

please tell in which folder i have to create the table suggest the proper path for it @abhisheks008

@abhisheks008 please review it and merge it

Images are not visible.

please review it asap @abhisheks008

No need to add drive link. You can just hyperlink the images from the Images folder to the README.

but what is the need of changing again if that link is working ultimately we want to see the pictures and it is visible then why i have to change it . please assign me as those links are also working now

Hi @payalsinha12 sorry for the inconvenience caused from my end. Please share your email address for further communications.
I have added your points in the excel sheet, no worries now.

Hi @payalsinha12 sorry for the inconvenience caused from my end. Please share your email address for further communications.
I have added your points in the excel sheet, no worries now.

<EMAIL_ADDRESS>Thank you for the help


how can i recognize the page leave?

how can i recognize the page leave?   i now have a requirement  like   if the page shows , there is a notification appears from top.  then i need the notification hide when the pages leave , because i assume i made every notification view for each page.

and i am now using  the rznotification view，  I have a special case :  i need present the notification in A page, then i tried to let it stay there until i click it.   but when the page B was pushed, i need the notification of A  hide.  then i manually invoke the rznotificationview hide  method in A 's viewdiddisappear ， but b's notification was dismissed as well. how can i correctly use it?

@ivanyang1984 Looks like a lifecycle relevant issue, what about hiding the rznotifcationView in A's viewWillDisappear?


Add grade snapshots

I need a way for students to see the current distribution of overall (final, sum, or total) grades.

Grading will be different for each class. e.g. participation (engagement) may we worth less in one class than another.
Grading will vary by semester.

This should be something like showing their sum total grade and then having an API through which they can view the distribution. Since these will be computed in batches, we will want students to get the distribution that corresponds to their current grade.
So, we might create something like
CREATE TABLE sum_grade (
    id int not null primary key,
    user_id int not null references user(id),
    round int not null,
    created_at timestamp with time zone,
    description text not null,
)

So, there would be "rounds". Usually this will be sum_grades computed at the same time, e.g. week 5 of of a class. But, I might give it a textual description too, e.g. (after exam). See existing quiz grade and assignment grade distributions.

Closed by 47bf080 and 58d21dc.


Media Player: missing option for volume up/down

Describe the bug
I have a media player entity that juts support volume up/volume down, but not set volume. So when I use the media player card and want to control the volume via the slider, I get an error message that its not supported.
To Reproduce
Steps to reproduce the behavior:

Have media player without slider volume control
Use Bubble Card Media Player
Try to edit volume

Expected behavior
Possibility to toggle between slider and volume up/down buttons
Informations (please complete the following information):

OS: all OS
Browser: all Browser
Bubble Card version 2.0.0
Home Assistant version: 2024.6

Additional context
Not applicable
Thank you! 🍻

Hi! You can add sub-buttons for that with tap_action set to call-service and media_player.volume_up / media_player.volume_down.


AbpLoginManager doesn't support SuccessRehashNeeded

Hi,
With implementing advansed PasswordHasher we can't use PasswordVerificationResult.SuccessRehashNeeded status in expected way.
We don't think that override of  complex method LoginAsyncInternal is the best solution here.
So, could you add appropriate handler in LoginAsyncInternal method of AbpLogInManager class ?
if (verificationResult == PasswordVerificationResult.Failed)
{
	if (shouldLockout)
	{
		if (await TryLockOutAsync(tenantId, user.Id))
		{
			return new AbpLoginResult<TTenant, TUser>(AbpLoginResultType.LockedOut, tenant, user);
		}
	}

	return new AbpLoginResult<TTenant, TUser>(AbpLoginResultType.InvalidPassword, tenant, user);
}
// ....
return await CreateLoginResultAsync(user, tenant, verificationResult );

protected virtual async Task<AbpLoginResult<TTenant, TUser>> CreateLoginResultAsync(TUser user, TTenant tenant = null, PasswordVerificationResult passwordVerificationResult)
{
//...
            return new AbpLoginResult<TTenant, TUser>(
                tenant,
                user,
                passwordVerificationResult,
                await UserManager.CreateIdentityAsync(user, DefaultAuthenticationTypes.ApplicationCookie)
            );
}

Thank you. I hadn't know that. I'll implement for the next version.


IE11 W7 doesn't wrap large .row > .col elements

BS4-beta2 IE11 on W7
When using the basic equal column width example from docs:
https://getbootstrap.com/docs/4.0/layout/grid/#equal-width
and one column contains large un-wrappable content (eg text without spaces or a replaced element like image), then IE11 does not wrap the remaining columns. Instead IE11 lets the long col1 overflow over the top of col2 and col3.

        col1WithReallyLongContentThatCantWrapOrIsALargeReplacedElementEgAnImage

        col2

        col3

When reducing the BS4 CSS down to the bare essentials we can demonstrate the problem with this codepen: https://codepen.io/oschonrock/pen/NXxXEj (test in IE11 and shrink your codepen window to see the effect).
The current BS4 css has flex-basis: 0; for .col. This doesn't work in IE11 as shown in bottom example row in the codepen, col1 overflows onto col2 & col3. If we change it to flex-basis: auto; for .col it does work as the top row in the code-pen shows.
Works as expected in recent FF or Chrome. ie both rows in codepen, wrap as expected.
Is this an IE flexbox bug or an issue with the BS4 .col CSS?

This is currently an undocumented flexbug: https://github.com/philipwalton/flexbugs/issues/170. We cannot change flex-basis to auto though as it will cause our columns to stack in IE11 vs layout as you intend. Note that your first column isn't equal-width to the second and third in your "good column" example.
Flexbox is still buggy in IE, and it has quirks in every other browser, too. There are usually a few options for every situation, but nothing that's uniformly perfect. Sorry!

Yeah, I realise flex-basis: auto is not a generic fix. It works in my specific (original) case, but layout is certainly different for the general case.
Thanks for looking into it. I feel like IE11 is the new IE6, and corporates will be very slow to upgrade away from W7, hence IE11 will be here for a long time... sigh
At least this broken behaviour now has one more piece of Googlebait, and people will realise what's happening.

Yeah it’s a bit of a pain for sure :). Hopefully we can get there with time—IE11 isn’t that big overall.

Just ran into this myself, and after a few hours of tinkering I was looking at the col source and realized there is also a col-auto that fixed this in ie11 for me.


fix tracy frame marker placement (after preseting all windows)

Objective
The frame marker event was emitted in the loop of presenting all the windows. This would mark the frame as finished multiple times if more than one window is used.
Solution
Move the frame marker to after the for-loop, so that it gets executed only once.

bors r+


bug: truncate text on status bar

This ensure that the call tree can still be interacted with
when the error statuses are very large.
The message is truncated to 80 characters and the full text show in a tooltip.
fixes #39

@kjonesffdc wait to merge this one.
I'm going to change it to a pure CSS implementation.
It will will the space in each status pill better and show more of the message on larger screens.

@kjonesffdc Ok finished.
This should handle it better and when the view is resized it will shrink/ grow showing more of the text or less.

I had to resolve some conflicts on this so you may want to check over.


Is this project still alive?

Description
I wanted to have a plugin similar to Mason, using which we can manage many themes that could be found online and could be configured with just a few keystrokes. Surprisingly I didn't find any solution and decided to create my own, and later stumbled upon this repository, so I was just curious are you still working on this project?
Additional Information
You can check out Mason here: Mason

Yeah, this project is still alive, just haven't really been working on it due to starting a new job and wanting to just relax after work.  I am thinking about getting back to this sometime soon just don't really know when yet.  Funny enough I was in your spot when I thought of this project after graduating last year and used Mason and Packer as heavy inspiration.
Something worth noting is that there was a bug that I ran into when developing where colorscheme does not reset when changing themes.

Great to know this, thank you so much @Kirbitz. I would try to use this plugin for a day or two, so I could help you with maybe documentation or development. I just started my Co-op after my 3rd Semester and I am getting a bit of time to work on this fun stuff, and joyed that I was not alone who is looking for a Plugin Manager for Themes haha.
Right now I am closing this issue.


Implementing COMMAND DOCS

[x] Added support for COMMAND DOCS command

All supported command docs are located in Garnet.server/Resp/RespCommandsDocs.json

[x] Added support for specifying docs path in command registration using the REGISTERCS command
[x] Added support for specifying command docs when registering command programatically
[x] Updated CommandInfoUpdater tool to import existing RESP command docs and unify them with Garnet-specific command docs

All Garnet-specific command docs are located in CommandInfoUpdater/GarnetCommandsDocs.json

[x] Updated RespCommandInfo to use "flattened-style" RespCommand enum for sub-commands
[x] Fixed COMMAND INFO to return sub-command info when called with sub-command name
[x] Added COMMAND DOCS tests & updated website docs

Examples:

Autocompletion of Garnet-specific commands:

Autocompletion of custom commands (requires restarting the client for it to refresh the docs):

Please check the binary size compared to main, and if it looks reasonable, we can merge.

Added a tool to compress the json files (pre-build) and reduce the overall size of Garnet.server.dll from 897KB to 650KB (with both resources embedded)

Please check the binary size compared to main, and if it looks reasonable, we can merge.

I've moved the json files to a separate dll that is being loaded (and unloaded) dynamically. The size of Garnet.server.dll is now down to 635KB. Let me know if you like the current solution. The loading & unloading of the assembly would happen at most twice (once for command info init & once for command docs init, on an as-needed basis) and is synchronized as to not cause any issues.

Please check the binary size compared to main, and if it looks reasonable, we can merge.

I've moved the json files to a separate dll that is being loaded (and unloaded) dynamically. The size of Garnet.server.dll is now down to 635KB. Let me know if you like the current solution. The loading & unloading of the assembly would happen at most twice (once for command info init & once for command docs init, on an as-needed basis) and is synchronized as to not cause any issues (Look at RespCommandDataCommon.cs for implementation)
Note: I packed and tested the nuget package locally and it works.

LGTM


Google Assistant missing redirect_uri field

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.56.2
Python release (python3 --version):
3.5.3
Component/platform:
Description of problem:
After Setting up Google Actions for Assistant i am getting a warning in the log file and it doesn't work , i don't see anything in the Google Assistant home control
Expected:
Problem-relevant configuration.yaml entries and steps to reproduce:

Traceback (if applicable):
2017-10-30 23:40:49 WARNING (MainThread) [homeassistant.components.google_assistant.auth] missing redirect_uri field

Additional info:

MOre than likely, it is an incorrect config. Did you change your project-id there?

This is the correct project id
I have tried to create a new project and i had the same error.
It is happening when i am trying to enter the http
https://XXX.duckdns.org/api/google_assistant/auth

I'm facing the same issue, can you please write what was the incorrect setting ?

Make sure your configuration.yaml file uses exactly the same id that is listed at https://console.actions.google.com/project/PROJECTNAME/settings/general
I added mine with a capitol letter by mistake and it gave an "missing project_id in redirect_uri" error message. Replacing the capitol letter with a lower case and then restarting home assistant made the testing step work as expected.

Thanks !
Apparently I just had to wait a few minutes for the project-id to get updated or something, it just worked after 5 minutes of waiting


Change cloud config placement to allow SSL usage

@neilstevenson also take a look, please, if I understood you correctly.

Closes #168, #200


Hand Gesture Recognizer unable to use GPU inference

Have I written custom code (as opposed to using a stock example script provided in MediaPipe)
No
OS Platform and Distribution
Nvidia Tegra Orin aarch64
MediaPipe Tasks SDK version
0.10.2
Task name (e.g. Image classification, Gesture recognition etc.)
Gesture Recognition
Programming Language and version (e.g. C++, Python, Java)
Python
Describe the actual behavior
I have been using Mediapipe's Gesture Recognition library in Python, and the CPU inferencing works all right. However, I noticed that version (0.10.2) of the Gesture Recognition module does not support GPU acceleration, which could significantly enhance its performance, especially on devices with GPUs.
Describe the expected behaviour
Integrate GPU support into the Gesture Recognition module, enabling users to benefit from enhanced performance. Or if it's already available, please point out where to configure that in the library since changing the delegate to GPU only does not help utilizing the GPU.
self.options = GestureRecognizerOptions(
            base_options=BaseOptions(model_asset_path=self.model_path, delegate="GPU"),
            running_mode=VisionRunningMode.VIDEO,
        )

Standalone code/steps you may have used to try to get what you need

Built a GPU-compatible whl file for aarch64 architecture.
Tested the whl file on my system and observed performance improvements in other modules.
Verified that the Gesture Recognition module does not currently support GPU acceleration.

Other info / Complete Logs
I20230821 16:28:38.537245 25612 gl_context.cc:342] GL version: 3.2 (OpenGL ES 3.2 NVIDIA 35.3.1), renderer: NVIDIA Tegra Orin (nvgpu)/integrated
Camera rgb: 7096 Frames [11:50, 10.66 Frames/sW20230821 16:28:38.773761    28 gesture_recognizer_graph.cc:128] Hand Gesture Recognizer contains CPU only ops. Sets HandGestureRecognizerGraph acceleration to Xnnpack.
I20230821 16:28:38.776032    28 hand_gesture_recognizer_graph.cc:249] Custom gesture classifier is not defined.
I20230821 16:28:38.783733    28 gl_context_egl.cc:84] Successfully initialized EGL. Major : 1 Minor: 5

@JehanJaye,
Currently, We do not offer any official support, libraries, modules to build into Jetson. There are plugin from the community is quite up-to-date which might help in this context But For now, We do not have a plan to support Jetson and does not provide support in how to use them.
Just a note, We have recently rolled out the support of Raspberry OS 64-bit in v0.10.3 as noted and example can be seen here.
Thank you

@kuaashish
Thank you for your response. I'd like to add a bit more detail to my issue regarding my experience with mediapipe-gpu on Jetson.
I've actually managed to successfully build and utilize the mediapipe-gpu ( v0.10.2 ) library on my Jetson.
While I understand that the official Gesture Recognition vision module doesn't currently support GPU acceleration as of version 0.10.2, my intention is to see if there's any potential plan or possibility to extend GPU support to the Gesture Recognition module in future releases.
Any further guidance or insights you can provide on this matter would be appreciated.

@JehanJaye,
Please go through the comment(4707) for better understanding. Thank you

So the mediapipe GPU building was successful and works. However, since the gesture recognition binary graph for GPU is not currently available, Hand Gesture Recognizer only runs with the CPU. Also, note that the other modules run seamlessly with the GPU in my platform.
@kuaashish When do you plan to release the GPU version for Hand Gesture recognition?
Thank you

@schmidt-sebastian  When do you plan to release the gesture_recognizer_graph with GPU ops?

I am also seeing that the hand gesture solution cannot be ran GPU yet.
Is there a plan to make a gesture_recognizer_graph with GPU ops?
Hand Gesture Recognizer contains CPU only ops. Sets HandGestureRecognizerGraph acceleration to Xnnpack.
When setting delegate=BaseOptions.Delegate.GPU it seems to try make a GPU delegate, but then reverts to CPU since the ops do not exist.
I0000 00:00:1720855483.067228 2544335 gl_context_egl.cc:85] Successfully initialized EGL. Major : 1 Minor: 5
I0000 00:00:1720855483.156018 2544497 gl_context.cc:357] GL version: 3.2 (OpenGL ES 3.2 NVIDIA 555.58.02), renderer: NVIDIA GeForce RTX 4090/PCIe/SSE2
W0000 00:00:1720855483.156332 2544335 gesture_recognizer_graph.cc:129] Hand Gesture Recognizer contains CPU only ops. Sets HandGestureRecognizerGraph acceleration to Xnnpack.
I0000 00:00:1720855483.156959 2544335 hand_gesture_recognizer_graph.cc:250] Custom gesture classifier is not defined.
INFO: Created TensorFlow Lite delegate for GPU.
INFO: Created TensorFlow Lite XNNPACK delegate for CPU.
W0000 00:00:1720855483.212475 2544520 inference_feedback_manager.cc:114] Feedback manager requires a model with a single signature inference. Disabling support for feedback tensors.
W0000 00:00:1720855483.212555 2544520 inference_feedback_manager.cc:114] Feedback manager requires a model with a single signature inference. Disabling support for feedback tensors.

Well I feel silly now.
After reading through this issue I see that both delegates being created is expected when specifying delegate=BaseOptions.Delegate.GPU https://github.com/google-ai-edge/mediapipe/issues/4711
At first I didn't see any performance differences between them, but it turned out to be an issue with my camera having a low frame-rate...
For testing I sent both delegates a static image and now see a bout a 5.5x performance increase on the GPU Delegate.
Also confirmed it was using the GPU with nvidia-smi.

Hi @JehanJaye,
Our Gesture Recognizer Task API supports GPU in Python for macOS and regular Ubuntu starting from version https://github.com/google-ai-edge/mediapipe/releases/tag/v0.10.8. Could you please adjust as per the comment https://github.com/google-ai-edge/mediapipe/issues/4712#issuecomment-2227178861 and update us on the status?
Thank you!!

Hi @kuaashish,
The issue is fixed now. Thanks!


nextPageToken empty string catch

Fix for issue outlined here: https://github.com/delta-io/delta-sharing/issues/526

Is there a way to test it?


Add connection messages

The goal of connection messages are to deal with several potential use cases

general handshake and authentication messages (ex: you are authenticated, but you are quarantined)
ensuring event logs have not been reset or otherwise changed between connections (ex: if a database was wiped)
communicating what event type ids and schema versions are compatible between client and server

Implemented by https://github.com/sparkmeter/sink/pull/8


fix: addressing indeterminate Map orderings in tests

Description
Using Nondex when running command mvn edu.illinois:nondex-maven-plugin:1.1.2:nondex in the tests directory after installing all dependencies, several tests in CSVSaverWithMultiOutputsTest.java and EnsembleExportTest.java produce flaky results (nondeterministically pass or fail). The reason is that the outputs in the tests are dependent on iterators for hashmaps, which does not guarantee a fixed order by default. (From the Javadoc: "HashMap class makes no guarantees as to the order of the map; in particular, it does not guarantee that the order will remain constant over time.") For example, in the function save(Path csvPath, Dataset<T> dataset, Set<String> responseNames) in CSVSaver.java, the code is using the iterator defined for ImmutableFeatureMap that deals with a Collection of values in a HashMap. Therefore, there is no guaranteed order for the entries in headerLine[], which leads to possible test flakiness.
To resolve the problem, I changed the code to construct a sorted ArrayList for all keys in the referenced HashMap to ensure determinate iteration order. I also let ImmutableLabelInfo.java and ImmutableMultiLabelInfo.java deal with LinkedHashMap to ensure deterministic element orders. All tests pass without flakiness after such changes.
Motivation
The change is necessary as related functionalities / tests may fail when, for example, the Java version upgrades in the future, or when the code is run in a different environment.
OCA
https://oca.opensource.oracle.com/api/v1/oca-requests/6710/documents/6730/download

Hi there, thanks for the response!
The LinkedHashMap approach is to deal with 2 flaky tests in EnsembleExportTest.java: testHeterogeneousClassificationExport and testHeterogeneousMultiLabelExport. Currently the constructors for ImmutableMultiLabelInfo and ImmutableLabelInfo are using Map.Entry<String,MutableLong> e : labelCounts.entrySet() to setup the 2 HashMaps idLabelMap and LabelIDMap. We need to make these two maps order deterministic to avoid flakiness in the 2 tests.
Currently nondex is not reporting flakiness in the tests directory other than CSVSaverWithMultiOutputsTest.java and EnsembleExportTest.java, so are the other flaky tests you mentioned in modules other than tests?
Also if the LinkedHashMap approach is problematic I can definitely first update the PR without including those changes~

Sure, but I'd prefer the code not to rely upon iteration order in the output infos, as it isn't an invariant guaranteed by the output info contract (and we had a bug in v4.0 and v4.1 due to issues where the regression code was depending on iteration order when it shouldn't have been).  I ran nondex on an internal repo we use for validating Tribuo releases which has all the public Tribuo tests in a single project, and hit the errors you found as well as some others. This branch fixes some of the flakyness, but I've not had chance to run down all of it (https://github.com/Craigacp/tribuo/commit/d9e5825ff450097f84eb7fd88cbc91bdbf63a83f). There were some failures in the way that the single label tests were run, K-Means reproducibility test, and those in the ensemble export you found which haven't been fixed that way, which suggests there is a deeper bug or we're depending on label order when we shouldn't be.

I updated a commit with a fix without switching to LinkedHashMap.
I used some breakpoints to find the cause of flakiness in EnsembleExportTest. Bascially the bug shall just be label order issues (rather than something deep):

the setup of idLabelMap(line 63 in ImmutableMultiLabelInfo.java) used values retrieved from an iterator from entrySet() and then used it as the value corresponding to the integer key counter. Then as entrySet() allows permutation, idLabelMap can be something like [1 -> a, 2 -> b] in one run and [1 -> b, 2 -> a] in another run.
in line 177, the test calls WeightedEnsembleModel.createEnsembleFromExistingModels(). Then in lines 197 and 203 in WeightedEnsembleModel.java, the current code calls the iterator of ImmutableOutputInfo<T> to add elements into a list. However, outputInfo is an instance of ImmutableInfoIterator (which implements the ImmutableOutputInfo interface), and its defined iterator is inherited from an iterator of an unordered entryset of a hashmap (line 198 in ImmutableMultiLabelInfo.java. Therefore, the iterator does not guarantee any order, so there can be order permutations in the lists firstList and comparisonList, leading to the exception thrown in line 207 of WeightedEnsembleModel.java. This issue can be resolved if we change the lists into sets.
Also I updated the solution to flakiness in CSVSaver.

I wonder if I understand your message correctly - is it undesirable to sort the keyset?

I agree that changes to ONNX shall be nontrivial, so I updated this pr to only include changes for CSVSaver. Appreciate your time and patience looking into this!

Thanks for flagging the iteration order issues. I've got a fix for the ensemble export, and some label ordering flakiness in the other tests which I'll make a PR for shortly.

No problem! Glad it works out


Linter returns "No default export found in module" for configureStore.js

Node version: 6.3.1
npm version: 3.10.3
Operating system: Ubuntu 16.04 LTS
Command line used: Terminal
Steps to reproduce:
The default config returns an error while linting, pointing to file configureStore,js. The error is:
✖  12:8  No default export found in module  import/default
pointing to line:
12: import configureStore from './store/configureStore';
I guess the problem is the if statement in said file:
if (process.env.NODE_ENV === 'production') {
  module.exports = require('./configureStore.prod');
} else {
  module.exports = require('./configureStore.dev');
}

and the fact that there's no default statement placed in it directly.
Any idea how to fix it?

It sounds like you have altered the kit. This rule is disabled in index.js to supress this error.
https://github.com/coryhouse/react-slingshot/blob/master/src/index.js#L1

Thanks Cory, you're right - I accidentally removed that line when I was pasting initial config from a separate file. All's working fine now.


issue with tangle hooks

Hello,
I have a hook defined for org-babel-post-tangle-hook
  :hook
  (org-babel-pre-tangle . mb/yank-project-from-org-file)
....

But when auto tangle is enabled, I am receiving these errors:
error in process sentinel: async-when-done: Symbol’s function definition is void: mb/yank-project-from-org-file
error in process sentinel: Symbol’s function definition is void: mb/yank-project-from-org-file

Apparently, the async process does not have access to the variables in the main process. Apparently, there is a :vars option, so I added this to my org-file:
#+auto_tangle: vars:load-path

and I have added the path to my hook function to the load-path, but it is still not picking up. Could you advise where I might be doing this wrong?
P.S I tried adding both el file and the path as such:
"/home/michael/.doom.d/literal/org" "/home/michael/.doom.d/literal/org/org-babel-helpers.el"

I have the same problem.

I have the same problem with org-babel-post-tangle-hook

I think the issue is that the async process is run as an emacs with an init file of just (require "org) and the variables in the local buffer.
The function name would be known but the actual function definition is unknown to the new process.
I suspect if you make hooks lambdas rather than functions it would work. In my case I also need other functions so can do as such here.


[SPARK-45835][INFRA] Make gitHub labeler more accurate and remove outdated comments

What changes were proposed in this pull request?
The pr aims to make gitHub labeler more accurate and remove outdated comments.
Why are the changes needed?
The functions mentioned in the comments have been released in the latest version of Github Action labeler.
https://github.com/actions/labeler/issues/111
According to the description of the original PR (https://github.com/apache/spark/pull/30244/files), after 'any/all' is released in the official version of Github Action labeler, we need to make subsequent updates to better identify the code label.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Continuous manual observation is required.
Was this patch authored or co-authored using generative AI tooling?
No.

cc @kbendick @HyukjinKwon

Merged to master.


[Help]How to hide the sidecar console window?

I wrote and packaged a file in Python. When I double-clicked to open the file, no console window appeared. However, when I used the sidecar function to start the file,
a console window appeared.s shown in my video
https://github.com/user-attachments/assets/3a326286-74bc-49f2-8db7-f72472bd2794
I use these codes to start this sidecar
import { Command } from "@tauri-apps/plugin-shell";
const sidecar_command = Command.sidecar("bin/MangovidAPI");
await sidecar_command.execute();

And this situation only occurs after packaging, it is normal during development, that is, when the yarn tauri dev command is running

Did you by any chance remove this line from your main.rs file? https://github.com/tauri-apps/tauri/blob/dev/examples/api/src-tauri/src/main.rs#L6

Did you by any chance remove this line from your main.rs file? https://github.com/tauri-apps/tauri/blob/dev/examples/api/src-tauri/src/main.rs#L6

Thanks, I noticed I was missing one line of code!

using pnpm tauri build --debug generated result seems to have the same issue.


[Global Styles] Mixin paragraph is empty

How to reproduce this bug:
visit page http://foundation.zurb.com/sites/docs/global.html
The last Paragragh, Mixins, contains only

We use these mixins to build the final CSS output of this component. You can use the mixins yourself to build your own class structure out of our components.

any other information is missing

@ncoden @IamManchanda is this worth keeping open?

It's a problem with Foundation Docs.


fix: fix configmap issue

This pull request does:

Add a Makefile to build the application and the image for initContainers
Create a named binary on go build
Add flag to set ConfigMap name --config-map-name
Add remove hook to remove configmap on uninstalling

Ping @mcruzdev

Hi guys, sorry for delay!
I changed the pull request!


(🎁) Add a shortcut to set the IDE vertical rule / line length / 'Hard wrap at'

Not many people actually know about this setting, I think it's a good one.
A "Set in IDE" button could be added here:

And it would update the IDE's project level settings to match the selection.
If fact, this should be heavily emphasized.

I think this is out of scope for this plugin.

Okay, but in my opinion, this plugin is "add black support to Idea", and if black is specifying a line length, then it makes sense that the plugin would change Idea to reflect that.
If I implemented this would you accept the change?


CLI improvements #321 #314

Fixes the rest of  https://github.com/theislab/sfaira/issues/321 and https://github.com/theislab/sfaira/issues/314

Thanks! Did you check if this here https://github.com/theislab/sfaira/pull/334 speeds up the individual commands?

Thanks! Did you check if this here #334 speeds up the individual commands?

Sure. Everything that speeds up the general sfaira import speeds up the CLI. As mentioned above, the CLI has to import the whole sfaira package to get the version.

Ok, could you check if it's a bareable import time for the first two commands though?

If not we can try to address this now as well.

@davidsebfischer it's about 5 seconds. I like my CLIs to start in <0.5s, but 5s is borderline acceptable.

Agree!

We will probably be able to close the gap to <1s when we refactor data into a separate repo in the future, for now this is good.

@davidsebfischer could you please quickly glance over it so that we can merge this one and I can tick it off my mind?

Thanks!!


Change personal website to birthdate and "projects" to "about me"

Hello,
The title should be self-explanatory :).
I am currently writing my resume with your nice template but I was wondering if it was possible to :

Remove the "personal website" in the sidebar and put a birthdate instead (since I don't have any personal website !) ?
Replace the "projects" section in the sidebar by another one, for instance an "about me" section or another skills overview diagram ?

Thank you.

@Mataivic Of course! Feel free to edit the twentysecondscv.cls file as necessary.

Ok cool ! I have never tried to modify such a file, can I have a few hints to know where to start ?

I recommend the official LaTeX tutorial. Once you get the hang of basic LaTeX concepts, you can get your hands dirty with changes to the .cls file 😃


Change "Github Pull Pipenv" to "Github Pull Requests" in contributing.rst

Pretty straight forward change. This seemed like a typo to me. I'm assuming this file is used to generate https://docs.pipenv.org/dev/contributing/ as well.

Thanks for this @sup!  Always appreciate the community chipping in to help with silly mistakes like this!


bug: Can't see camera preview.

Plugin(s)

[X] Barcode Scanning
[ ] Face Detection
[ ] Face Mesh Detection
[ ] Selfie Segmentation
[ ] Translation

Version
6.1.0
Platform(s)

[X] Android
[ ] iOS
[ ] Web

Current behavior

I trigger the scanning
Scanning successfully starts
The camera preview is not present
If i put the phone in front of a qr code it scans correctly
The value returned is correct

Expected behavior

I trigger the scanning
The camera preview is visible
The scan i successful

Reproduction

Steps to reproduce

Other information
Can't share the complete code as is a work related project but if needed i can share specific parts if needed for debugging this issue
The project started as a React + Vite + TS then i added capacitor because the specs changed mid development.
Adding capacitor was successful and the app works great
I just can't see the camera preview
Tried all the steps mentioned in the docs, even the css rules but still the camera isn't shown.
This project uses TailwindCSS and NextUI
Theres no IONIC stuff in this.
In the now deprecated capacitor/barcode-scanner GHRepo it was mentioned something about dark mode bugging the interface, so i obviously tried without dark mode and nothing changes.
As previously mentioned, the scanner , scans perfectly.
If more context or stuff is needed i will gladly provide it.
Capacitor doctor
💊   Capacitor Doctor  💊
Latest Dependencies:
@capacitor/cli: 6.1.2
@capacitor/core: 6.1.2
@capacitor/android: 6.1.2
@capacitor/ios: 6.1.2
Installed Dependencies:
@capacitor/ios: not installed
@capacitor/cli: 6.1.2
@capacitor/core: 6.1.2
@capacitor/android: 6.1.2
[success] Android looking great! 👌
Before submitting

[X] I have read and followed the bug report guidelines.
[X] I have attached links to possibly related issues and discussions.
[X] I understand that incomplete issues (e.g. without reproduction) are closed.

This is most likely a bug in your implementation and not a bug in the plugin. I have just tested the latest version and everything worked without any problems. For this reason, I am closing this issue. Feel free to create a discussion or book a consulting hour.


Docs: incorrect useMatch argument description

Describe the bug
In the docs on useMatch at https://tanstack.com/router/latest/docs/framework/react/api/router/useMatchHook#usematch-options it says:

The useMatch hook accepts a single optional argument, an options object.

But the options object according to the type definitions is not optional. I used tanstack/react-router 1.19.2

Your Example Website or App
https://stackblitz.com/edit/github-14t2wa?file=src%2Fmain.tsx%3AL15
Steps to Reproduce the Bug or Issue
Use useMatch  in a component and do not specifiy any arguments.
Expected behavior
Either the hook should work without args or the docs should be fixed.
Screenshots or Videos
No response
Platform

TanStack Router 1.19.2
TypeScript 5.3.3

Additional context
No response

yeah docs need to be updated. can you open a PR?

opened pr #1282. I also tried to clarify the usage of opts.from and opts.strict as I think, it's not allowed to set from when strict: false and on the other hand its not allowed to skip fromwhenstrict: true`.
please let me know if I should change anything.

btw: how can I recive the current Match in a component? I thought it was just useMatch() but but that doesn't work, if opts is required.
Now I use
  const matches = useMatches();
  const current = matches[matches.length - 1];

Is this correct?

don't you know the current route you are in?

no, i don't know the current route. I'm in a "generic" layout component that needs access to the current route' context.


Added onClick prop to button and fixed justifyContent prop in Box

Added onClick prop to button
Fixed justifyContent prop in Box component to be taken into account.

Thank you!

My pleasure! Will push later tonight another one with a basic Input component done as well 👍

I wonder how you can follow so many changes I made. I think we are almost there. The last API seems to be the right.


Use Aff (fixes #15)

Changes HandlerM to use Aff (asynchronous effects) instead of Eff

Adds a type parameter (e) for the effect rows to HandlerM to get rid of
unsafeInterleaveEff. Normally when writing top-level handlers one can
write:
myHandler :: forall e. Handler e

which is a type alias for
myHandler :: forall e. HandlerM (express :: EXPRESS | e) Unit

Removes capture as it's no longer needed. Async code can be wrapped with
Aff and Aff can be lifted to HandlerM with liftAff.

@anttih I've rebased your changes in my the use-aff branch of my own fork. Can you pull that into your own and check everything's ok? (It does already pass the tests on Travis)

I've pushed your branch here. Everything's looking good. Thanks a lot for working on this :+1:
I'm in the process of rebasing the plain-ffi stuff on top of this (in the other PR).
@dancingrobot84 want to take a look at this?

:cool: Thank you very much for the work! I will merge it as soon as you fix a couple of minor issues.

Fixed now.

Great! Merged


Changed to allow for sub-path in url

Added a bit of extra knowledge in so that the urls are adjusted to have the full original path in, rather than assuming its hosted in /

Resolved in #37.


Fix non billable filter value

Notion card
https://www.notion.so/saeloun/The-billing-status-filter-is-not-working-on-Timesheet-entry-report-819d384d69994a4fb78104ab4d8ae00f
Summary
Preview

Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to
not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

Checklist:

[ ] I have manually tested all workflows
[ ] I have performed a self-review of my own code
[ ] I have added automated tests for my code

@apoorv1316 Please retest the bug

@shalapatil Supriya has a billed entry for the 25th of may but why it's not showing when I applied billed filter on reports

@shalapatil Can you please check review comments from apoorv Tiwari

I tested this  and its working fine for me. Could you please check again?

@apoorv1316 Did you connect with Shalaka for this bug ?

@harshaanNihal Please review this PR .


Invalid type NoneType for attribute 'fastapi.route.operation_id' value

Description
Hi there.
When using FastAPI and Logfire, for each request, I have this strange log:
16:35:25.519   FastAPI arguments
Invalid type NoneType for attribute 'fastapi.route.operation_id' value. Expected one of ['bool', 'str', 'bytes', 'int', 'float'] or a sequence of those types

The endpoint works as expected.
How can I get rid of this?
Thanks.
Python, Logfire & OS Versions, related packages (not required)
logfire="1.3.0"
platform="Linux-6.10.4-linuxkit-aarch64-with-glibc2.36"
python="3.12.7 (main, Oct 19 2024, 18:29:53) [GCC 12.2.0]"
[related_packages]
requests="2.32.3"
pydantic="2.9.2"
fastapi="0.115.3"
protobuf="4.25.5"
rich="13.9.3"
executing="2.1.0"
opentelemetry-api="1.27.0"
opentelemetry-exporter-otlp-proto-common="1.27.0"
opentelemetry-exporter-otlp-proto-http="1.27.0"
opentelemetry-instrumentation="0.48b0"
opentelemetry-instrumentation-asgi="0.48b0"
opentelemetry-instrumentation-fastapi="0.48b0"
opentelemetry-instrumentation-sqlalchemy="0.48b0"
opentelemetry-proto="1.27.0"
opentelemetry-sdk="1.27.0"
opentelemetry-semantic-conventions="0.48b0"
opentelemetry-util-http="0.48b0"

Released a fix in 1.3.1, thanks for the report

Thanks!
FYI, this line still appears after the upgrade: 12:03:51.153   FastAPI arguments
Not sure if it's intended or not.

Yes, that's intended, see https://logfire.pydantic.dev/docs/integrations/fastapi/#endpoint-arguments-and-validation-errors


Expose ability to customize JSON serialization

Removed public ObjectMapper references which let users customize JSON serialization.
This was due to Azure/azure-cosmosdb-java#153 need a better option.

After reading the following blog post: https://devblogs.microsoft.com/cosmosdb/difference-between-null-and-undefined/  I was wondering how to achieve this with the Java SDK.
The default ObjectMapper is configured to always serialize null fields.
I was stumbling across this when adding a ttl field to my entity class. I want to set a custom TTL when updating a particular Cosmos DB item, the majority of items will never need a custom TTL and can simply use the container TTL. Therefore, the field would be null.
But if I set the ttl field to null I get the following error:

The input ttl 'null' is invalid. Ensure to provide a nonzero positive integer less than or equal to '2147483647', or '-1' which means never expire.

The documentation here states that setting ttl to null works, although not quite sure if null has the same meaning there.

After reading the following blog post: https://devblogs.microsoft.com/cosmosdb/difference-between-null-and-undefined/  I was wondering how to achieve this with the Java SDK.
The default ObjectMapper is configured to always serialize null fields.
I was stumbling across this when adding a ttl field to my entity class. I want to set a custom TTL when updating a particular Cosmos DB item, the majority of items will never need a custom TTL and can simply use the container TTL. Therefore, the field would be null.
But if I set the ttl field to null I get the following error:

The input ttl 'null' is invalid. Ensure to provide a nonzero positive integer less than or equal to '2147483647', or '-1' which means never expire.

The documentation here states that setting ttl to null works, although not quite sure if null has the same meaning there.

@kushagraThapar I'd like to see some solution here, as this really limits some uses of the SDK. So maybe you can move it out of the "parking lot" again and assign it a higher prio?
For example, you cannot use Kotlin classes as https://github.com/FasterXML/jackson-module-kotlin is not registered in the static ObjectMapper hidden in the SDK. This forces you to annotate Kotlin classes manually as a very ugly workaround to get deserialization of immutable data classes working:
data class Email @JsonCreator constructor(
    @JsonProperty("email") val email: String // note the redundant specification of "email"
)

Or do some hack like calling .registerModule(new KotlinModule()) on that static ObjectMapper instance at an early point of the application lifecycle.

@xinlian12  - thoughts on this issue ?

I don't know if this is still being actively looked into, but there is an example of this in azure-search-documents. There is an interface hierarchy in azure-core with ObjectSerializer as the base interface with a sub-interface JsonSerializer, and an experimental interface AvroSerializer in azure-core-experimental, which allows for a customer designed serializer to be passed into an API call or service client. For azure-search-documents the service can return JSON documents which can have a few different handlings on the service client:

Document is retrieved with no specified Java type, a generic JSON Map<String, Object> call SearchDocument is returned.
A document type is specified but JsonSerializer wasn't configured in SearchClientBuilder, JacksonAdapter (or effectively JacksonAdapter) is used to deserialize the JSON document to the type.
A JsonSerializer was specified in SearchClientBuilder and a document type is specified, the passed JsonSerializer is used to deserialize the customer's document.

There are two implementation packages for JsonSerializer in azure-core-serializer-json-gson and azure-core-serializer-json-jackson which a customer could use out of the box to pass their GSON or Jackson serializer to handle their JSON document type. Customers can also roll their own implementation if they choose to and there is also the ability to use service provider interfaces to load their own implementation from the classpath similar to what is seen with HttpClient implementations.

@alzimmermsft Without having looked into details, am I getting it right that using a custom JsonSerializer is currently only possible with SearchClientBuilder and you're mentioning it here to achieve such a customization for users of the SDK? Or is this actually a general solution and that issue can be closed?

@alzimmermsft Without having looked into details, am I getting it right that using a custom JsonSerializer is currently only possible with SearchClientBuilder and you're mentioning it here to achieve such a customization for users of the SDK? Or is this actually a general solution and that issue can be closed?

It would be the former, where some libraries offer support for JsonSerializer. I was mentioning it here as a possible solution going forward for the Cosmos libraries, so this issue should remain open.

Bumping this thread.  I have run into an issue where the Cosmos SDK failed to serialize a LocalDate within a repository query (resulting in a missing jsr130 module exception).  I tried the hack that neiser mentioned above which partially resolved things, but would rather let the customer specify their own serializer as they can with the ACS SDK as mentioned by alzimmermsft.  Happy to open a new issue to track, if that makes sense.

@mmcdermottMS - FYI, I have recently added the jsr310 module to Java SDK. You should be able to leverage it in the latest release of azure-cosmos Java V4 SDK.

Hi, I think this is a related-enough issue to post it here: I'm struggling using GeoJson points in Spring Data for Cosmos. It looks like there's a class designed for just plugging in and using, com.azure.core.models.GeoPoint, but the ObjectMapper set up for serializing for Spring Data defined in com.azure.spring.data.cosmos.core.convert.ObjectMapperFactory doesn't have the serializers for it configured. (In addition to other weird choices like not turning off SerializationFeature.WRITE_DATES_AS_TIMESTAMPS even though ISO-format is the preferred format for storing in Cosmos, but that's a side note.) There's a quite comprehensive separate com.azure.core.implementation.jackson.ObjectMapperFactory that looks to be used internally, and which includes adding a GeoJsonSerializer & GeoJsonDeserializer, but all of those classes are package-local and so can't be easily used when defining my own cosmosObjectMapper.
I wrote a terrible hack using reflection to call the GeoJsonSerializer.getModule() anyway to use while defining my cosmosObjectMapper (though I don't know if I actually want to check it in), but it seems like having the SDK expose better options for customizing is really the way to go here. Thanks.

@petercooperjr-spls - I think it would be better to create a separate issue for the problem you are facing. We can better investigate it that way.

Another hack would be to use your own serializer and persist the json tree.
        val jsonTree = myObjectMapper.valueToTree<JsonNode>(myKtDataClass)
        val itemResponse = cosmosContainer.createItem(jsonTree)

Just wanted to update this issue, we are building functionality to customize JSON serializability - https://github.com/Azure/azure-sdk-for-java/pull/38997
Apologize for the delay in this development of this feature.

This has been done. Implementation details in this PR - https://github.com/Azure/azure-sdk-for-java/pull/38997


Blank lines removed between Python function and special keywords

Note: this was ran from commit bbceeeb
It appears that blanks line are removed between the end of a Python function and special Snakemake keywords.
Given this Snakefile:
from os.path import exists
from os import mkdir

def foo():
    print("Hello World!")

workdir: "data"

localrules: run_all

onstart:
    if not exists("logs"):
        mkdir("logs")

rule run_all:
    output:
        my_output="output.txt",
    shell:
        "touch {output.my_output}"

onsuccess:
    print("Workflow finished, no error")

onerror:
    print("An error occurred")

snakefmt formats it this way:
from os.path import exists
from os import mkdir

def foo():
    print("Hello World!")
workdir: "data"
localrules:
    run_all,
onstart:
    if not exists("logs"):
        mkdir("logs")

rule run_all:
    output:
        my_output="output.txt",
    shell:
        "touch {output.my_output}"

onsuccess:
    print("Workflow finished, no error")
onerror:
    print("An error occurred")

Shouldn't the 2 blank lines separating the foo() function and the workdir keyword be kept?
Also, should the special handlers onstart, onsuccess and onerror be treated like rules and have 2 blank lines before and after?

Yes, please ensure two blank lines between every item.

cc @mbhall88 @bricoletc

great spot, i'd noticed this too and unhappy with it.
@johanneskoester are you saying there should be two blank lines before workdir but also between workdir and localrules?

I disagree that there should be two lines between a python function and workdir. Python functions, by convention, should have one line spacing and workdir feels to be more like a normal variable assignment, so
from os.path import exists
from os import mkdir

def foo():
    print("Hello World!")

workdir: "data"

something else

Should be
from os.path import exists
from os import mkdir

def foo():
    print("Hello World!")

workdir: "data"

something else

PEP8 states clearly "Surround top-level function and class definitions with two blank lines" and "Method definitions inside a class are surrounded by a single blank line". In the example above we are talking about a top level function, so I believe 2 blank lines are required between the end of the foo() function and the workdir directive.
from os.path import exists
from os import mkdir

def foo():
    print("Hello World!")

workdir: "data"

...

Right you are.

Yes, I agree, two blank lines feel right.

Great. Thanks a lot for bringing this up @enixmail !
Running from 9919e4b the diff from the snakefile you gave at the top of this issue is:
- localrules: run_all
?            --------
+ localrules:
+     run_all,

So the double spacing you suggested is in place.
I'll keep this open for now as it might fuel more discussion.


Better document setting oneof fields

The documentation states:

For a oneof field, the protobuf compiler generates a single field with an interface type isMessageName_MyField.

And so it states that the generated structs look like this:
type Profile struct {
        // Types that are valid to be assigned to Avatar:
        //      *Profile_ImageUrl
        //      *Profile_ImageData
        Avatar isProfile_Avatar `protobuf_oneof:"avatar"`
}

However, then in the example code it goes on to set it this way:
p1 := &account.Profile{
  Avatar: &account.Profile_ImageUrl{"http://example.com/image.png"},
}

Note that the field name is Avatar not isProfile_Avatar.
Am I missing something with my understanding of golang or this package?

I think you have read the ordering of field and type backwards in the struct? The generated struct code says the field is Avatar and is of type account.isProfile_Avatar, and then the assignment assigns into the field name Avatar the actual concrete value of a complex literal of type *account.Profile_ImageURL which implements account.isProfile_Avatar.

Oops, I think you are right.


[stable/prometheus-adapter] Defining a variable for listen port

What this PR does / why we need it:
Defines a variable for listen port of the application so it can be changed to the purpose.
Which issue this PR fixes

addresses https://github.com/DirectXMan12/k8s-prometheus-adapter/issues/134#issuecomment-439904042
addresses https://github.com/kubernetes-sigs/metrics-server/issues/45#issuecomment-541110986

Special notes for your reviewer:
Changing the port for GKE deployment is the easiest approach since the firewall rules in GCP can't be easily updated through IaC and require extra software to be deployed.
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @mattiasgees


Support for clock muxes and clock dividers

This is a summary of the discussion at the Hurricane-2 debrief meeting.
Clock muxes and clock division registers should have first-class support in Firrtl.  Not sure if this needs an accompanying Chisel issue.
The related issue here is that the designer may wish to specify a library cell to use as a clock mux or divider (to ensure balanced rise/fall times on a clock signal, for example) rather than simply defining a mux construct in Verilog and giving the synthesis tools free reign.  It's unclear what the API for this would look like.

What's wrong with using black boxes for this feature?  You can use syntactic sugar to make it look nice (e.g., ClockMux(Bool, Clock, Clock) can instantiate a black box).

I'm using the black box approach. I think it's 100% sufficient + some SDC constraint that only allows the tools to upsize/downsize the mux. There's a synthesis version that does sel ? opt1 : opt0 and there's a real implementation version that just uses a foundry mux. I don't think it needs to be a Firrtl pass; you just need to provide a utility in Chisel to support it -- (that's populated by some foundry-specific template -- so maybe a little bit of Firrtl magic involved).

@aswaterman - My only reluctance about blackboxing is that it might be nice for Chisel/Firrtl to still understand that the thing is a Mux.  I'm not certain about the resolution of ucb-bar/chisel3#551, but if Scala-based testing is still a goal, then blackboxes break that functionality.  Unless we can define functional behavior for blackboxes now?

Yea, I think you'd need to special case them for FirrtlInterpreter support. I haven't been using it for a while courtesy of limitations w/ multiclocking, etc., but it is useful for fast simulations.
Is the point of FirrtlInterpreter to be fully compatible with all potential "advanced" features though?
When thinking about physical design/larger scope chips, I think it's possible to request a lot of funky things that "dirty" the Firrtl codebase. People keep mentioning not wanting to bring the bad aspects of Verilog into Chisel/Firrtl, but some of these sketchy things -- like non-synthesizable Reals -- are extremely useful to mixed signal chips... Are we supposed to be special casing each instance of these sketchy but super useful features? Where's the boundary between Chisel/Firrtl + pure Verilog/SystemVerilog?

For other reasons, black-box linkage will become a mandatory feature for Scala-based testing to be generally useful.  When using the Verilator backend, it should just be a matter of providing an additional list of Verilog sources.  It's a much harder problem for the Firrtl interpreter, where we'd need a functional Firrtl model of the black box.

Similar to the synchronizer discussion, I've been leaning towards the
blackbox solution for this issue (well, similar variants). The projected
plan for retaining FIRRTL analytical capabilities is annotations. The
general justification is similar to synchronizers where you generally want
to fine-tune the implementation for these things rather than letting the
synthesis engine do whatever.
This could be a good candidate for these sorts of blackboxes being
'grayboxes' similar to how LLVM has intrinsic functions. They would
basically blackboxes with spec-defined behavior. So, some passes could
EITHER treat them like any other blackbox OR try to interpret 'over them'
using the spec-defined behavior others (like when clock analysis). (I kind
of wanted mid-low firrtl memories to be like this.)
On Wed, Apr 12, 2017 at 9:24 PM, Andrew Waterman<EMAIL_ADDRESS>wrote:

For other reasons, black-box linkage will become a mandatory feature for
Scala-based testing to be generally useful. When using the Verilator
backend, it should just be a matter of providing an additional list of
Verilog sources. It's a much harder problem for the Firrtl interpreter,
where we'd need a functional Firrtl model of the black box.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ucb-bar/firrtl/issues/540#issuecomment-293776909, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA9_-Gt6H948ONSaCQ9_bjdd1k-ggsw7ks5rvaNngaJpZM4M8Oi9
.

The graybox thing is a little bit like how chisel2 blackboxes worked, where in the module body you'd write a behavioral description, used for the C++ backend.

This is good example of the utility of gray box concept


compile tsx file

Laravel 8
Node Version v16.13.0
NPM Version 8.1.0

How can I compile tsx file ?
let's say .. example.tsx which include some import modules

They're just react typescript file.  Since it's typescript, have you try mix.ts(your file) ?


Is it possible to avoid nested transactions for create_database?

https://github.com/Kerollmops/heed/blob/b235e9c3e9984737c967b5de1014b48f125dc28b/heed/src/env.rs#L347
Currently I can't use
env_builder.flag(heed::flags::Flags::MdbWriteMap);
because that flag doesn't work with nested transactions and create_database uses nested transactions internally

It looks like this is a duplicate of #95


move tatara recorder for circular import

Current dependencies on/for this PR:

main

PR #44   👈

This stack of pull requests is managed by Graphite.


Keep the Clear Terminal button on the terminal toolbar

Please have the clear button return in the terminal toolbar or have an option to have it visible in the terminal toolbar
https://github.com/microsoft/vscode/issues/203955

is there a reason you cannot use the keybinding for this vs clicking it in the toolbar?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!