INVALID_BUFFER_SIZE error in ContextPyopencl

Hi, I am posting this here as to my understanding it is an issue coming from xobjects. Feel free to move the issue to the appropriate repository should I be mistaken.
I am encountering a failure with the ContextPyopencl when increasing the number of tracked turns above a certain threshold. Here is a script that reproduces the issue for me based on the documentation's FODO example:
from typing import Union

import numpy as np

import xline as xl
import xobjects as xo
import xtrack as xt

SEQUENCE = xl.Line(
    elements=[
        xl.Drift(length=2.0),
        xl.Multipole(knl=[0, 1.0], ksl=[0, 0]),
        xl.Drift(length=1.0),
        xl.Multipole(knl=[0, -1.0], ksl=[0, 0]),
    ],
    element_names=["drift_0", "quad_0", "drift_1", "quad_1"],
)

N_PART = 200
STARTING_COORDINATES = {
    "p0c": 6500e9,
    "x": np.random.uniform(-1e-3, 1e-3, N_PART),
    "px": np.random.uniform(-1e-5, 1e-5, N_PART),
    "y": np.random.uniform(-2e-3, 2e-3, N_PART),
    "py": np.random.uniform(-3e-5, 3e-5, N_PART),
    "zeta": np.random.uniform(-1e-2, 1e-2, N_PART),
    "delta": np.random.uniform(-1e-4, 1e-4, N_PART),
}

def track_on_context(
    _context: Union[xo.ContextCpu, xo.ContextCupy, xo.ContextPyopencl],
    starting_coordinates: dict,
    n_turns: int = 10_000,
) -> np.ndarray:
    """
    Convenience function to track as the documentation's simple example on a given context, returning the
    resulting horizontal coordinates.

    Args:
        _context (Union[xo.ContextCpu, xo.ContextCupy, xo.ContextPyopencl]): `xobjects` context to build for.
        starting_coordinates (dict): initial coordinates of particles to track.
        n_turns (int): number of turns to track for, defaults to 10000.

    Returns:
        Resulting X coordinates from tracking, as a `numpy.ndarray`.
    """
    # Transfer lattice on context and compile tracking code
    tracker = xt.Tracker(_context=_context, sequence=SEQUENCE)

    # Build particle object on context
    particles = xt.Particles(_context=_context, **starting_coordinates)

    # Track and return turn-by-turn
    tracker.track(particles, num_turns=n_turns, turn_by_turn_monitor=True)
    return tracker.record_last_track.x

if __name__ == "__main__":
    # macOS machine, using the OpenGL context as Metal is not supported
    context_gpu = xo.ContextPyopencl()
    _ = track_on_context(context_gpu, STARTING_COORDINATES)  # this is fine
    _ = track_on_context(context_gpu, STARTING_COORDINATES, 100_000)  # this is fine
    _ = track_on_context(context_gpu, STARTING_COORDINATES, 300_000)  # this is not

I am on a macbook pro machine (Intel CPU + Radeon Pro Vega 20 GPU). The first two trackings run fine but the third one raises at the tracker.track(...) step, the error message giving
pyopencl._cl.LogicError: create_buffer failed: INVALID_BUFFER_SIZE

Hi @fsoubelet , sorry I had not seen this!
You are trying to allocate memory to save turn-by-turn data for all particles. Probably you run out of memory.
Does it work if you set turn_by_turn_monitor=False. In that case you have only the coordinates at the last turn (directly in particles)


Kadane's Algorithm in C++

💥 Proposal
Implementation of kadane algorithm in C++ . This algorithm find the contiguous subarray of maximum sub.
Have you read the Contributing Guidelines on Pull Requests?
Yes

/assign


create from a server localhost adress

Hello, how can I create an application, with a local internet adress:
http: //localhost:4040/
I have an audio server on my pc and I would like to create the application.
thx

i mean, you can, but only you can use it.
if you want more people to use it, you’ll need to host it on a server hosting company like linode or google cloud, and if you want to still self host it yourself, you can port forward.

I found the solution: You have to add all the complete address, like this for example: http://localhost:4040 /web/index.php/xxxxx/xxx.
The x correspond to the target while being connected to the site with his user account.
Thx


Update README.md

Great job.
General design. Everything is displayed correctly in the terminal.
Functionality. The program is divided into parts, which makes it easier to understand and maintain.
Naming. Variable names correspond to their values, which increases code readability.
Good things. There is no unnecessary code, everything is structured according to PEP 8 (Style Guide for Python Code), which is an important criterion for quality code. Keep it up!


Use the metadata naming convention for files

Current coverage is 79.30%

Merging #1650 into master will decrease coverage by -0.01% as of 13e0087

@@            master   #1650   diff @@
======================================
  Files          539     539
  Stmts        32097   32097
  Branches      8928    8928
  Methods          0
======================================
- Hit          25458   25456     -2
- Partial       5181    5183     +2
  Missed        1458    1458

Review entire Coverage Diff as of 13e0087

Powered by Codecov. Updated on successful CI builds.

:+1:


isthg.jk version 1.7

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.2 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree [company="Chill Paradise"]

@microsoft-github-policy-service agree company="Chill Paradise"


yum updates for ec2 instances

I was a little curious about the reasoning behind the line below. Is this line stating that ec2 instances should not get yum updates?
https://github.com/ansible-lockdown/RHEL7-CIS/blob/devel/tasks/section_1/cis_1.9.x.yml#L9

jlosito,
Thanks for reaching out, that is mainly for our testing pipeline. What we were finding was for some odd reason Ansible failing on that control in an ec2 instance. That control is basically doing an yum update which we found can be run manually and works fine via Ansible on bare metal and VM's (virtual box), but for some reason fails with Ansible against the ec2. With your question, I'm thinking a different name for that var is probably the best way forward.
George

Changed name of var


Wrong Norwegian language code (no)

Package
filament/filament
Package Version
v3.2.111
Laravel Version
v11.22.0
Livewire Version
v3.5.6
PHP Version
PHP 8.3
Problem description
Norway have two written languages, Norsk Bokmål (nb) and Nynorsk (nn).
Laravel lang have both of these which translates your Laravel application.
Since Filament uses "no" you wount benefint from these translation, like the validation messages.
When you set your language to "no" Filament is translated as expected, but the rest of your Laravel application is not, and vice versa.
Expected behavior
Setting language to "nb" would translate both Laravel application and Filament.
Steps to reproduce
Install Laravel, Livewire, Filament and Laravel lang.
Sett locale to "no" or "nb"
Add a user.
Log in.
Go to "Edit user"
Fill in a short password.
Validation message would be in another language than the Filament app.
Reproduction repository (issue will be closed if this is not valid)
https://github.com/Tor2r/novsnb
Relevant log output
No response

Donate 💰 to fund this issue

You can donate funding to this issue. We receive the money once the issue is completed & confirmed by you.
100% of the funding will be distributed between the Filament core team to run all aspects of the project.
Thank you in advance for helping us make maintenance sustainable!

I will gladly pull request a "nb" language to Filament, but I just don't know if that is the correct way to go, since there already is a "no" version.

Hopefully this will be fixed soon.

When using the no language code, which language would you expect, @Tor2r @ekandreas?
Would that be Norsk Bokmål or Nynorsk, or doesn't it make sense to use no at all?
Let me know and I can point you in the right direction.
I think you should be able to add the right codes in a pull request.

It doesn't make sense at all, since Laravel it self doesn't use "no" it uses "nb" and "nn". Many other packages uses "nb" as well.

Alright, let's add the correct "nb" and "nn" locales then.
Should not be breaking if we keep the wrong "no" files around until v4.
Could you work on this, @Tor2r? :)

I'll fix it 👍


Mockup

Screens: splash screen, configuration screen, main screen & settings screen
Icons: Feather Icons
 (SVG)
Font: Montserrat
Splash Screen:

Graphic: Fancy animation
Size: quite small, rectangle or square

Configuration Screen:

Graphic: maybe logo and background
Appears on first startup only
Size: big rectangle
Contains: title (LibreLinkUpDesktop), short description, country chooser, language chooser, e-mail input & password input

Main Screen:

Graphic: trending blood sugar level arrow and gear icon
Background changes in relation to blood sugar level
Size: quite small, rectangle or square
Contains: blood sugar level, trending arrow and gear icon

Settings Screen:

Graphic: background?
Size: big rectangle
Contains: four sections Configuration, Support, About & Libre Licenses

Configuration: theme chooser, country chooser, language chooser, e-mail input & password input
Support: short description and link to manual
About: short description, link to GitHub, link to website & link to Hosted Weblate
Libre Licenses: a list with used libraries and so on

Unclear:

May the user choose the metric unit (mg/dL or mmol/l) or does that come from the API?

Here is a mockup and a prototype.
The logo may be found here.

Initial Release is available now: https://github.com/Crazy-Marvin/LibreLinkUpDesktop/releases/tag/v0.1.2. 🥳


Add validation for non-text responses

Just to get you started, here's 1 possible way to do it:

let Axios pass the response as a buffer:

  responseType: "arraybuffer",

in the data to verify, after the 2 newlines, append the binary string version of the buffer:

data = response.data.toString("binary");
...
const template: string = `${response.status}\n${headers}\n\n${data}`;

let your verifyString function use raw encoding instead of utf-8:

messageDigest.update(data, "raw");

This preserves the binary encoding in the whole chain.
The same will also work just fine for regular JSON responses, but make sure you manually convert the response data to JSON, as Axios won't do this anymore because of the responseType option; something like:
if (response.headers["content-type"] === "application/json") {
  response.data = JSON.parse(response.data.toString("utf8"));
}

Technically, I don't think this last "utf8" is strictly necessary (i.e. could be "latin1" just as well), because the encoded JSON will contain escaped UTF-8 characters in embedded strings, should the response contain UTF-8 (e.g. an emoji in your bank account name).


NOTICK: Allow Crypto functions to be invoked from Java.

Remove Kotlin types from Crypto API signatures and apply @JvmStatic annotations, allowing them to be invoked from Java.
Note also that Java doesn't support Kotlin's default parameters either :wink:

LGTM

Thanks, I'll just check my composite build still works before merging it :wink:


⚠ Turnos 24x7 has degraded performance

In 93a43dd, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 3242 ms

Resolved: Turnos 24x7 performance has improved in 39ac19e.


Type instantiation is excessively deep and possibly infinite

I want to add the createClient return type to a interface, however it gives me this error
declare module 'vitest' {
	export interface TestContext {
		client: Awaited<ReturnType<typeof createClient>>
	}
}

Result when consuming the context:

I don't have a lot of endpoints so I wonder if this error won't appear in other places too

createClient doesn't return a promise, so I don't think you need Awaited there, and the return type of createClient is too dynamic to be used in that way.
I'd recommend to use these types instead of ReturnType;
https://github.com/ardatan/feTS/blob/master/packages/fets/src/client/createClient.ts#L68C4-L68C13

Ohhh my bad, actually removing Awaited made it work!


[BUG] Inability to search all documents or a group of documents with GraphRAG

Description
Whenever you try to use all documents or a group of documents with GraphRAG, you get either
searching in doc_ids []
INFO:ktem.index.file.pipelines:Skip retrieval because of no selected files: DocumentRetrievalPipeline(

for all documents or something similar to
AssertionError: GraphRAG index not found for file_id: ["d6f18887-0b01-4df0-a30d-997c919d60f1", "08bad9e8-ef3f-4e89-abed-21da7d4f9611"]

for a grpup of documents. However, there is no problem searching any of these documents one by one. Considering, that RAG is mostly used to be able to access a large quantity of different docs, it makes Kotaemon unusable unless you stick with File Collections
Reproduction steps
1. Go to Files, upload more than one document
2. Got to Chat, Click on 'Graph Collection', choose "Select All"
3. Send any kind of message
4. Observe an absence of a reference of any documents and completely unrelated response

Screenshots
No response
Logs
use_quick_index_mode False
reader_mode default
Using reader <kotaemon.loaders.pdf_loader.PDFThumbnailReader object at 0x7f1fdbdd19f0>
Page numbers: 4
Got 4 page thumbnails
Adding documents to doc store
indexing step took 0.2428741455078125
Initializing project at
/app/ktem_app_data/user_data/files/graphrag/da6da42a-8eb2-4c0b-afb2-a8a56fc509d7

/usr/local/lib/python3.10/site-packages/numpy/core/fromnumeric.py:59: FutureWarning: 'DataFrame.swapaxes' is deprecated and will be removed in a future version. Please use 'DataFrame.transpose' instead.
  return bound(*args, **kwds)
/usr/local/lib/python3.10/site-packages/numpy/core/fromnumeric.py:59: FutureWarning: 'DataFrame.swapaxes' is deprecated and will be removed in a future version. Please use 'DataFrame.transpose' instead.
  return bound(*args, **kwds)
/usr/local/lib/python3.10/site-packages/numpy/core/fromnumeric.py:59: FutureWarning: 'DataFrame.swapaxes' is deprecated and will be removed in a future version. Please use 'DataFrame.transpose' instead.
  return bound(*args, **kwds)
/usr/local/lib/python3.10/site-packages/numpy/core/fromnumeric.py:59: FutureWarning: 'DataFrame.swapaxes' is deprecated and will be removed in a future version. Please use 'DataFrame.transpose' instead.
  return bound(*args, **kwds)
/usr/local/lib/python3.10/site-packages/datashaper/engine/verbs/convert.py:72: FutureWarning: errors='ignore' is deprecated and will raise in a future version. Use to_datetime without passing `errors` and catch exceptions explicitly instead
  datetime_column = pd.to_datetime(column, errors="ignore")
/usr/local/lib/python3.10/site-packages/datashaper/engine/verbs/convert.py:72: UserWarning: Could not infer format, so each element will be parsed individually, falling back to `dateutil`. To ensure parsing is consistent and as-expected, please specify a format.
  datetime_column = pd.to_datetime(column, errors="ignore")
User-id: None, can see public conversations: False
User-id: 1, can see public conversations: True
User-id: 1, can see public conversations: True
Session reasoning type None use mindmap (default) use citation (default) language (default)
Session LLM
Reasoning class <class 'ktem.reasoning.simple.FullQAPipeline'>
Reasoning state {'app': {'regen': False}, 'pipeline': {}}
Thinking ...
Retrievers [DocumentRetrievalPipeline(DS=<kotaemon.storages.docstores.lancedb.LanceDBDocumentStore object at 0x7f2004dca7d0>, FSPath=PosixPath('/app/ktem_app_data/user_data/files/index_1'), Index=<class 'ktem.index.file.index.IndexTable'>, Source=<class 'ktem.index.file.index.Source'>, VS=<kotaemon.storages.vectorstores.chroma.ChromaVectorStore object at 0x7f2004dcaef0>, get_extra_table=False, llm_scorer=LLMTrulensScoring(concurrent=True, normalize=10, prompt_template=<kotaemon.llms.prompts.template.PromptTemplate object at 0x7f1fd8106290>, system_prompt_template=<kotaemon.llms.prompts.template.PromptTemplate object at 0x7f1fd8106170>, top_k=3, user_prompt_template=<kotaemon.llms.prompts.template.PromptTemplate object at 0x7f1fd8106080>), mmr=False, rerankers=[TeiFastReranking(endpoint_url='http://proxy:3000/v1/rerank', is_truncated=True, model_name='jina')], retrieval_mode='hybrid', top_k=10, user_id=1), GraphRAGRetrieverPipeline(DS=<theflow.base.unset_ object at 0x7f20fe849300>, FSPath=<theflow.base.unset_ object at 0x7f20fe849300>, Index=<class 'ktem.index.file.index.IndexTable'>, Source=<theflow.base.unset_ object at 0x7f20fe849300>, VS=<theflow.base.unset_ object at 0x7f20fe849300>, file_ids=[], user_id=<theflow.base.unset_ object at 0x7f20fe849300>), LightRAGRetrieverPipeline(DS=<theflow.base.unset_ object at 0x7f20fe849300>, FSPath=<theflow.base.unset_ object at 0x7f20fe849300>, Index=<class 'ktem.index.file.index.IndexTable'>, Source=<theflow.base.unset_ object at 0x7f20fe849300>, VS=<theflow.base.unset_ object at 0x7f20fe849300>, file_ids=[], user_id=<theflow.base.unset_ object at 0x7f20fe849300>), NanoGraphRAGRetrieverPipeline(DS=<theflow.base.unset_ object at 0x7f20fe849300>, FSPath=<theflow.base.unset_ object at 0x7f20fe849300>, Index=<class 'ktem.index.file.index.IndexTable'>, Source=<theflow.base.unset_ object at 0x7f20fe849300>, VS=<theflow.base.unset_ object at 0x7f20fe849300>, file_ids=[], user_id=<theflow.base.unset_ object at 0x7f20fe849300>)]
searching in doc_ids []
INFO:ktem.index.file.pipelines:Skip retrieval because of no selected files: DocumentRetrievalPipeline(
  (vector_retrieval): <function Function._prepare_child.<locals>.exec at 0x7f1fbbfe31c0>
  (embedding): <function Function._prepare_child.<locals>.exec at 0x7f1fbbfe32e0>
)
Got 0 retrieved documents
len (original) 0
Got 0 images
Trying LLM streaming
INFO:httpx:HTTP Request: POST http://vllm:8000/v1/chat/completions "HTTP/1.1 200 OK"
Got 0 cited docs
INFO:httpx:HTTP Request: POST http://vllm:8000/v1/chat/completions "HTTP/1.1 200 OK"

Browsers
No response
OS
Linux
Additional information
No response

Additionally, based on this commit, it seems like groups are intended not to work. Maybe the "select all" option is intended not to work as well?


Bugs and features

Q
A

Bug fix?
yes

New feature?
yes

BC breaks?
no

Deprecations?
no

Tests pass?
yes

Fixed tickets
#1234, #5678

License
MIT

Doc PR
api-platform/doc#1234

Please update this template with something that matches your PR
#142 fixed some bugs and added new features, as filtering.

Wow that was fast!

Thanks @devel-pa! @Cruiser13, can you try if this PR fixes your issue? Thanks!

@dunglas ;) Last month work, that's why it was fast, I really tested on my stack and works, so WorksOnMyMachine status.


DOCS -- Inkscape "Setting up The Canvas" should use "Major grid line every: 6" instead of "5"

Browser

[ ] Chrome/Chromium
[ ] Firefox
[ ] Safari
[ ] Edge
[ ] iOS Safari
[ ] Opera
[X] Other/not relevant

Operating system

[ ] Windows
[ ] Linux
[ ] macOS
[X] Other/not relevant

Description
Hi folks,
At your documentation at: https://lucide.dev/guide/design/inkscape-guide#setting-up-the-canvas
The image shows "Major grid line every: 5" I think it should be "6" and the screenshot should highlight the change, similar to my suggestion.
I did start with a template using the suggested settings. But major lines at 5 get annoying really quickly. So I think since the main viewport is 24, 6 would be a much better default setting.

Steps to reproduce
See info in Description
Checklist

[X] I have searched if someone has submitted a similar issue before and there weren't any. (Please make sure to also search closed issues, as this issue might already have been resolved.)

Sure -- Mine looks a bit different. I'm using Inkscape V1.3.2

I think the marking is important. "Major grid line every: 5" is default. If there is no marking imo nobody will see it.

Hi @ericfennis, can you please assign this issue to me? So, I can start working on it.

@poswalsameer Yess ofcourse! Tnx!

Hi @ericfennis, I have created a PR by doing the said change as mentioned in the issue, please approve my PR. Thanks!

Hi @ericfennis, please review this PR so that it can get merged.


github Page build failed!

error msg: Your site is having problems building: Page build failed. For more information, see https://docs.github.com/github/working-with-github-pages/troubleshooting-jekyll-build-errors-for-github-pages-sites#troubleshooting-build-errors.
_config.yml:
remote_theme: https://github.com/cotes2020/jekyll-theme-chirpy

error msg: Your site is having problems building: Page build failed. For more information, see https://docs.github.com/github/working-with-github-pages/troubleshooting-jekyll-build-errors-for-github-pages-sites#troubleshooting-build-errors.
_config.yml:
remote_theme: https://github.com/cotes2020/jekyll-theme-chirpy

Hello dufei99,
I've got same error as yours,
do you solve this problem yet?

error msg: Your site is having problems building: Page build failed. For more information, see https://docs.github.com/github/working-with-github-pages/troubleshooting-jekyll-build-errors-for-github-pages-sites#troubleshooting-build-errors.
_config.yml:
remote_theme: https://github.com/cotes2020/jekyll-theme-chirpy

Hello dufei99,
I've got same error as yours,
do you solve this problem yet?


Define ClockFormatTrait

Add a trait to set the clock display format.
Options are 12-hour, 24-hour, and locale default.

@mrjerryjohns, good to merge? The PR has passed all checks, there's a hiccup between GitHub and Travis—if you click through the pending integration, it passed.


ENYO-1687 : apply accessibility on moonstone checkboxItem

Apply accessibility on moonstone CheckboxItem.
Issue
When CheckboxItem is focused, Only child checkbox is readout.
Cause
The CheckboxItem consists of checkbox and text item. When spotlight is focused on CheckboxItem, only child checkbox component is readout because CheckboxItem does not have checkbox role and aria-checked attribute.
In addition, apply accessibility mixin on moonstone Checkbox and Icon to supporting accessibilityDisabled and removing tabindex of child component.
The enyo component applied accessibility mixin has tabindex at its dom node because enyo control adds tabindex to all components which have content or accessibilityLabel or accessibilityHint.
The CheckboxItem has text item and checkbox as child, so each child component also has tabindex.
The tabindex of child is not neccessary, and If accessibilityDisabled of CheckboxItem is true, accessibilityDisabled of child component also should be true.
Fix
Add checkbox role and aria-checked attribute to CheckboxItem.
Set accessibilityDisabled of child components to true.
Enyo-DCO-1.1-Signed-off-by: Bongsub Kim<EMAIL_ADDRESS>

This PR has dependency on https://github.com/enyojs/enyo/pull/1207


deps update hadoop libs to <IP_ADDRESS>

Proposed changes
Fix native shared library issue
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 45.59 seconds
stream load tsv:          510 seconds loaded<PHONE_NUMBER>9 Bytes, about 139 MB/s
stream load json:         19 seconds loaded<PHONE_NUMBER> Bytes, about 118 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.1 seconds inserted 10000000 Rows, about 343K ops/s
storage size:<PHONE_NUMBER>8 Bytes


Add Sparse Representation Capability to Dataloaders

This solves for #500.

rerun tests

The "conflict" to be resolved here was literally whitespace, so I went ahead and resolved it. (Can you tell I'm excited about getting this PR merged? :smile: )

rerun tests

rerun tests

rerun tests

rerun tests

rerun tests

As far as I can tell, the comments have now been resolved and it's just a matter of getting pylint to stop timing out on CI, yeah?

rerun tests

rerun tests

rerun tests


Feature: login

please make hass usable from the homescreen so i can login. also make windows button a viable option to press plsss

Hello,
Please use the feature template to properly describe what you'd like to request.


SetContent->SetHtmlContent

Merge after aspnet/HtmlAbstractions#19.

Hi @ryanbrandenburg, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

CC @NTaylorMullen @dougbu

:shipit:


Android support (with Android.mk)

Same as https://github.com/mrousavy/react-native-multithreading/pull/3, but I try to use Android.mk here.
I'm trying to import the node_modules/react-native-reanimated/react-native-reanimated.aar file the user has on his system, by following this SO answer
Useful links:

Android Developer: Android.mk docs
Stackoverflow: "How to manually include external aar package using new Gradle Android Build System"

closed in favor of #3


[mdnsresponder] Change compile flag /Zi to /Z7

In order to embed debugging information in the static library, set /Z7 instead of /Zi to sync with vcpkg_configure_cmake.
Fixes #13955.

Thanks for the PR!


Work on the description to the questions added in the REGULARITY page of Skill Assessment Spreadsheet

While writing description of the questions one main question should be kept in mind that:
- What kind of description can be added in the page for the questions so that when interviewee taking interview can judge/grade the interviewer according to the description?

As a description you need to mention a few recent examples of fascinating or unusual topics that can help in describing the questions. The description should be for understanding, explore or discovering the questions demand.
You can also give pictorial representation as description to the questions.
Evaluate the interviewer's response based on the uniqueness and relevance of the topics they have learned about, as well as their ability to articulate why they found these topics interesting.
Description should be given in a way that evaluate the interviewer's response based on the relevance and potential impact of the topic they would like to learn about, as well as the level of genuine curiosity and enthusiasm they express for this topic.
You can add the description in the areas that are empty.

Hello Team,
I have started working on this issue.


[Components: Controls] SourceCodeEditor

Highlighted program code editor. Use existing implementation.
e2e tests required.

QA Notes
import {SourceCodeField} from @haulmont/react-antd;
Use in place of <Field>
e2e test:
test-pup/example-react-app/form-controls/source-code-editor.test.js


Added /pings chat command

I'm not too sure about some stuff but at least it's a few things already done.

So toggle between normal and 10% visible pings? Something like that?

yea, that could be done as well

Is it useful to keep this? I don't think I know enough to create the whole settings thing myself.

yes, I'll merge and work on it when I have time. I prioritized the other PR despite being later as we've been actually thinking on that for a while longer.

Ended up implementing it with a different solution, check the above commit for details. Closing.


Automated broken link checks ?

https://github.com/gaurav-nelson/github-action-markdown-link-check
Maybe worth checking ?

I tried this for rnaseq pipeline and it works reasonable well:
name: Check Markdown links
on: push
jobs:
markdown-link-check:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v1
- uses: actions/setup-node@v1
with:
fetch-depth: 1
- uses<EMAIL_ADDRESS>

@FriederikeHanssen - do you have a PR open for this somewhere?

@ewels not yet, still working on it

Note that we can (a) have these tests on both push and pull_request and (b) limit to only events that change markdown files. See the markdown linting as an example:
https://github.com/nf-core/nf-co.re/blob/59225cbb20445f26a2b60ca79ec35c00d1862d33/.github/workflows/markdown.yml#L2-L6

cool thanks.

In fact, it might make most sense to add this link checking to the same workflow as above. Can just be an additional step / job.

Yes, I agree. Will do that

Hi @FriederikeHanssen, any movement on this issue?

@ewels the proposed package does not support to check for section links. I opened an issue there . So far no reply. I guess we could move forward without this specific check for now. TBH I wanted to finish it right after the hackathon and then got derailed by other stuff. I should have some time on Friday afternoon to finish this up

Maybe we can use remark lint instead. It seems to have two plugins that can handle both soft headings links as well as external links:
https://github.com/remarkjs/remark-validate-links
https://github.com/davidtheclark/remark-lint-no-dead-urls

@FriederikeHanssen Do you have a PR for this already? I must have marked Unwatch on GitHub so I wasn't getting any notifications about it. I am back again now, if you put up a PR (for nf-co.re) we can work together on it and try to configure and use markdown-link-check action.
Also, it does check for internal links and images as well.


Module nonfunctional in Worlds Without Number system

Module completely nonfunctional in the Worlds Without Number 0.9.3 system for Foundry v9.238

Update popup does not appear on module activation
RollTable Configuration menu is not present
UI switches to white text instead of black, buttons are misplaced

could you test with the latest version?

Text color is corrected. The configuration menu remains missing, and the buttons are still sitting under Roll Table Formula. Appreciate the fast response!

is this a duplicate of you other issue or are there differences in the problem you try to describe?

I tested as requested, and the issue is not solved. The text changing from white to black is the only thing that changed in updating.

Oh, these are 2 different systems. Didn't realize. Sorry.


add historyAPI fallback

It seems that history API fallback is not activated.
\ (F5 on a screen, and you will get a 404)
I didn't have the time right now to look into it, sorry. But here an issue :)

I saw this behavior too, but the fallback is activated, it even says so in the output


Updated Spatial Persistence checkouts

XRTK - Mixed Reality Toolkit Pull Request
Overview

Updated Spatial Persistence checkouts

Can you give me a clue where to find the Preview Number version then for the updated Spatial Persistence package?  Still shows as 0.3 in dev @StephenHodgson

package.json

Except the version shown for the spatial persistence service system at present is 0.3.0 (no preview)
https://github.com/XRTK/com.xrtk.spatial-persistence/blob/development/XRTK.SpatialPersistence/Packages/com.xrtk.spatial-persistence/package.json

https://github.com/XRTK/com.xrtk.spatial-persistence.asa/blob/ca8d2b0c73a7a0995dd8366f38d086e4e9f510a1/XRTK.SpatialPersistence.ASA/Packages/com.xrtk.spatial-persistence.asa/package.json#L25

Closing as these already seem to be up to date


Error converting value 0 to type 'Nest.Retries'. Path 'retries', line 1, position 105.

Hi guys,
I'm trying to execute an update by query operation. This is my code:
var response = elasticClient.UpdateByQuery<QualifiedLead>(u => u
					 .Script(s => s.Inline($"ctx._source.userId = userId").Params(p => p.Add("userId", userId))).Conflicts(Elasticsearch.Net.Conflicts.Proceed)
					 .Query(q => q.Term(t => t.Field(f => f.MultiRequestId).Value(requestGuid)))
					);

and I have this beautifull exception:

Error converting value 0 to type 'Nest.Retries'. Path 'retries', line 1, position 105.

I'm using a 5.0.1 Nest version and 2.4.0 Elastic Version
Thank you.

NEST 5.x is only tested and supported against Elasticsearch 5.x while it might work for the most part there are several edge cases like this that you might hit.
The shape in which retries is returned is now an object in 5.x indicating whether the search or the update was retried and in 2.x it was an overall numeric.


refactor Maru#servers/0 for better readability

😃

Thank you @1ternal , it looks better 🙂 merged.
It's not an urgent bugfix, so I won't release a new version for this pr, the change will be in the next release 🙂

@falood understood, Thanks 😃


Feature | V3 Support Multiple Authenticators

With previous versions of Saloon, you could only have one authenticator per connector/request. This meant that if an API utilized multiple ways of authentication like a certificate and an Authorization header you would have to write it manually. You can now change the method signature of your defaultAuth method to an array.
protected function defaultAuth(): array
{
    return [
         new TokenAuthenticator('example'),
         new HeaderAuthenticator('api-key', 'X-API-Key'),
    ];
}

You can also now pass an array of authenticators into the authenticate method.
$connector->authenticate([...]);

This is a backwards-compatible change and it's entirely opt-in.

Update - some feedback is suggesting just introduce the new MultiAuthenticator authenticator which can be used when multiple authenticators want to be used at the same time.


Warnings raised by mkdocs

Is your feature request related to a problem?
Running poetry run mkdocs serve currently raises 7 warnings:
WARNING  -  griffe: safeds/data/image/containers/_image.py:363: No types or annotations for parameters ['Factor']
WARNING  -  griffe: safeds/data/image/containers/_image.py:363: Parameter 'Factor' does not appear in the function signature
WARNING  -  griffe: safeds/data/tabular/transformation/_range_scaler.py:22: No types or annotations for parameters ['Raises']
WARNING  -  griffe: safeds/data/tabular/transformation/_range_scaler.py:22: Parameter 'Raises' does not appear in the function signature
WARNING  -  griffe: safeds/data/tabular/transformation/_range_scaler.py:22: Could not parse line '------'
WARNING  -  griffe: safeds/data/tabular/transformation/_range_scaler.py:22: No types or annotations for parameters ['ValueError']
WARNING  -  griffe: safeds/data/tabular/transformation/_range_scaler.py:22: Parameter 'ValueError' does not appear in the function signature

Looks like the corresponding docstrings are missing some type annotations or exceptions or are otherwise outdated.
Desired solution
Check the docstrings and correct them.
If possible, make the CI fail if warnings are generated by mkdocs.

:tada: This issue has been resolved in version 0.15.0 :tada:
The release is available on:

v0.15.0
GitHub release

Your semantic-release bot :package::rocket:


select ON CLICK sometimes load the default one

Hello.
im struggling with:
sometimes when i click on my , he just open the default one. is there anyway to prevent this to happen?
basically how to force the .init to only show the "design" of the select, when/if the bootstrap-select is being shown?
Thanks

Can you elaborate a bit? What do you mean by the "default" one?
See guidelines for contributing.
Bug reports
A bug is a demonstrable problem that is caused by the code in the repository.
Good bug reports are extremely helpful - thank you!
Guidelines for bug reports:

Use the GitHub issue search. Check if the issue has already been
reported.

Check if the issue has been fixed. Try to reproduce it using the
latest master or development branch in the repository.

Provide environment details. Provide your operating system, browser(s),
jQuery version, Bootstrap version, and bootstrap-select version.

Create an isolated and reproducible test case. Create a reduced test
case.

Include a live example. Use this Plunker debugging template to share your isolated test cases. You can also make use of jsFiddle or jsBin.

A good bug report shouldn't leave others needing to chase you up for more
information. Please try to be as detailed as possible in your report. What is
your environment? What steps will reproduce the issue? What browser(s) and OS
experience the problem? What would you expect to be the outcome? All these
details will help people to fix any potential bugs.
Example:

Short and descriptive example bug report title
A summary of the issue and the browser/OS environment in which it occurs. If
suitable, include the steps required to reproduce the bug.

This is the first step
This is the second step
Further steps, etc.

<url> - a link to the reduced test case
Any other information you want to share that is relevant to the issue being
reported. This might include the lines of code that you have identified as
causing the bug, and potential solutions (and your opinions on their
merits).


Remove redundant switch in m2mFirmware.cpp

M2MFirmware::resource_name makes the switch in M2MFirmware::get_resource redundant
Also "resource" was misspelled.

CLA Submitted

@yogpan01

CLA Accepted

@JanneKiiskila Can this be taken in ?

@yogpan01 guess not.


Update ESP-IDF submodule to fix return value of esp_wifi_deinit

Description
This change fixes implementation of WIFI_Off in wifi port layer. Also fixes failing test case AFQP_WiFiOnOff in Quarantine_WiFi test group.
NOTE: This fix is also required in esp32-s2 branch
Checklist:

[x] I have tested my changes. No regression in existing tests.
[x] My code is Linted.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/bot run checks

/bot run checks


【Maven】master分支，编译失败

[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  02:03 min
[INFO] Finished at: 2022-12-04T17:05:52+08:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal com.diffplug.spotless:spotless-maven-plugin:2.27.1:check (spotless-check) on project iotdb-parent: Execution spotless-check of goal com.diffplug.spotless:spotless-maven-plugin:2.27.1:check failed: Unable to resolve dependencies: Failed to collect dependencies at com.google.googlejavaformat:google-java-format:jar:1.7 -> com.google.errorprone:javac-shaded:jar:9+181-r4173-1: Failed to read artifact descriptor for com.google.errorprone:javac-shaded:jar:9+181-r4173-1: Could not transfer artifact com.google.errorprone:javac-shaded:pom:9+181-r4173-1 from/to alimaven (https://maven.aliyun.com/repository/central): authorization failed for https://maven.aliyun.com/repository/central/com/google/errorprone/javac-shaded/9+181-r4173-1/javac-shaded-9+181-r4173-1.pom, status: 403 Forbidden -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/PluginExecutionException
——————
您好，请问这是我网络的问题还是maven版本的问题？我的maven版本是3.8.6。

可以看一下这个文章
https://blog.csdn.net/yagebu1983/article/details/124144594
也可以考虑把 aliyun maven 源换成 maven 的官方源


Auto cast a unicode response status to a string if a unicode status is provided

This bit me, and I felt like it would be smarter to allow unicode response status inputs that get auto cast. Thoughts?

aside: pull request 2^8. woot

lgtm

lgtm2 ;)


Latest version does not work with UserMin

Getting the following error after setting up the theme an try to open UserMin
Error - Perl execution failed
Undefined subroutine &WebminCore::embed_header called at /usr/share/usermin/authentic-theme/authentic.pl line 29,  line 28.
Usermin 1.670

All ok, was maybe a browser refresh issue.


Impossible pair packings

If one of the grids doesn't have any slots for a shape, don't use that shape to pack the other grid.

Probably replaced by #59, simultaneous packing.


Azure Cloud Agents: cli installs in the wrong path

Environment: Microsoft Azure Cloud Agents: ubuntu-latest

Issue:
cli is installing at /home/vsts/work/[node_modules/.bin/cmf-cli/cmf] when it should be installed at /home/vsts/work/_temp/[node_modules/.bin/cmf-cli/cmf]. The temp path is expected when running the build and pack commands, which is causing pipelines to fail.

Logs:
Debug info did not provide anything useful and a manual Linux install does not present the same issue

Pipeline generator is fixing this by changing the path in the command invocations (this will be in cmf init), but the underlying issue is not solved yet


RemoveRow() Removes the wrong Row. Version 2.5.1

Using a xslx spreadsheet with 10 rows:
RemoveRow seems to have an off by 1 error, but it's weird because the row object is correct, but RemoveRow deletes the wrong row.
_sheet = (XSSFSheet)_spreadsheet.GetSheet("Sheet1");
var row0 = _sheet.GetRow(0);
var row1 = _sheet.GetRow(1);
var row2 = _sheet.GetRow(2);
_sheet.RemoveRow(row0);  -- does nothing.
_sheet.RemoveRow(row1); -- removes row[0]
_sheet.RemoveRow(row2); -- removes row[1]
Looking at https://github.com/tonyqus/npoi/blob/3ddc11ab79c2dbe0fb6fffee075e1eb6c957fd9b/ooxml/XSSF/UserModel/XSSFSheet.cs

Wouldn't rows.Count be wrong here?

duplicate with #450

Hello,
I think the same issue is affecting XSSFSheet (and maybe other implementations of ISheet).
If I try to delete a row on an XSSFSheet, the previous one gets deleted


More precise types in arithmetic evaluation

Previously, we just casted most of our numeric types to Long when doing arithmetic operations in the ValueEvalutor. This PR tries to optimize that by introducing a number of extension functions that under the hood call the necessary Kotlin functions to properly deal with the resulting type based on the bit width.

We may want to add a note to our gh page and say that the evaluation could return different types of numbers. This means that it might be required/useful to transfer the value e.g. via toLong() (this is related to #902 )


Unable to use Order app signals

I wanted to trigger some functionality when an order is placed Then i found Order app some sugnals which we can use. So I was trying to use order_placed signal.
For this I have already forked order app and inside this app order_placed function is created in signals.py. I have also imported signals in config.py but When this signal is triggered the Order instance doesn't have any lines associated with it neither I can see that order in Dashboard. SO I am assuming this signal is triggered before order is fully placed.
So how can I use order_placed signal which is mentioned in docs so that I have order full info in that signal.
Can anyone share any example of oscar signal usage ?
config.py
from oscar.apps.order import config
    class OrderConfig(config.OrderConfig):
        name = 'catalogue.order'

        def ready(self):
               from catalogue.order import signals

signals.py
from django.db.models.signals import post_save
from django.dispatch import receiver

from oscar.apps.order.models import Order

    @receiver(post_save, sender=Order)
        def order_placed(*args, **kwargs):
            """

            :param args:
            :param kwargs:
            :return:
            """
            print("i ma here ----------------------")

Hi there. We use Github only for managing bugs and feature requests, and not for support. We recommend that you ask for help on Stack Overflow using the django-oscar tag, or joining our slack workspace and asking a question there. Thanks.


retest- Last message does not disappear with express checkout

Environment Details
Spartacus: feature/express-checkout-fr1 branch
Browser: chrome
Device: mac
Happens on: http://dev-com-21.accdemo.b2c.ydev.hybris.com:4200/electronics-spa/en/USD/
Steps to Reproduce

Enable express checkout
Login.
Place an order (with 1 saved address and 1 saved payment)
Now go to MyAccount -> Address Book
Delete the address
Add a product to cart, proceed to checkout
Add and save an address and place the order

Actual Result
the "address deleted successfully" message doesn't disappear, even after 8-10s.
Expected Result
The message should disappear. Can't remember if it's configured to be 8 or 10 seconds. Unless this has been changed?

Sometimes we get a "Cart Not Found" error right after, and it doesn't disappear. So looks like the last message displayed is the one that doesn't go away.

error msg: 7s
confirmation msg 3s

Michal tried to reproduce it on develop branch.
With this scenario:
"yeah, then we can select delivery mode and got error Delivery Mode [standard-gross] is not supported for the current cart . this error will not disappear. refreshing page repair problems with not disappearing GMs"
But it's not related to EC.

The easiest way to reproduce that bug is to:
1 try login with bad credentials(error "Bad credentials. Please login again."), then register a new account - "Please log in with provided credentials." will not disappear
2 Or do same in reverse order
If we generate 2 successful messages (eg. in Consent Management) they are going to disappear, same with 2+ error messages (refresh is important).
Conclusions
The problem occurs if we display messages different type than before

Local QA steps:
add that e.g in breadcrumb.component
<button (click)="addConfirmation()">addConfirmation</button>
<button (click)="addError()">addError</button>
<button (click)="addInfo()">addInfo</button>

  addConfirmation() {
    this.globalMessageService.add(
      'msg_' + this.counter++,
      GlobalMessageType.MSG_TYPE_CONFIRMATION
    );
  }
  addError() {
    this.globalMessageService.add(
      'msg_' + this.counter++,
      GlobalMessageType.MSG_TYPE_ERROR
    );
  }
  addInfo() {
    this.globalMessageService.add(
      'msg_' + this.counter++,
      GlobalMessageType.MSG_TYPE_INFO
    );
  }

and click x times for every button.
Messages should disappear in the same order, with delay specified in config - every delay starts when the previous message disappears.

Confirmed messages appearing and disappearing in numerical order


Switch to listing clusters on demand instead of caching them

What this PR does / why we need it:
Relying on the cluster lister can sometimes cause the machine reconcicliation to fail without being able to find a cluster since the machine creation event can be received before the cluster creation event. Normally this doesn't matter because it is retried in a loop. It is, however, causing the unit test to flake about 2% of the time.  This refactor makes the test more reliable (0 failures out of 500). The difference may not seem like much, but will become annoying when PR testing is turned on.
Outside of testing concerns, we would likely have to make sure we fetch the latest version of the cluster anyway like we do for machine elsewhere. This also allows for getCluster to locate the cluster based on the machine in the future when we change how machines and clusters are associated as #609 proposes.
@kubernetes/kube-deploy-reviewers
/lint

/assign @medinatiger @jessicaochen

/lgtm

/unhold
@jessicaochen Nope. That's it.


optimize analytics loading speed

investigate and propose potential fixes

@puncsky
The api's TTFB is too long.


permit underlying conn to implement batch interface directly

I reverted #3236 because I messed up the merge. This PR adds the changes.

@AudriusButkevicius Sorry for the confusion here, this is entirely my fault. Could you please consent as the GoogleBot is asking?
Regarding the release: This is more ad-hoc. Wasn't planning to cut a new release any time soon, unless there's a reason to do so. Would the batch-read something you'd be interested in having released soon?

@googlebot I consent.

Syncthing is stuck on an old version of quic-go as we are wary of breaking connectivity between versions like we did in the past by not reading quic-go release notes about compatability.
I guess now that RFC9000 is out, we should upgrade and keep rolling it forward, as I trust it should now never become incompatible.
Also, as part of that I'd like to pick up the batch reading code, not that I think the optimisations matter much, as we do our own reads and then fake a net.PacketConn anyway (to filter out stun packets via my pfilter package), but more so that the method quic-go uses to read packets would not change in the future release. Namely upgrading to the current latest release will give us RFC9000, which is great, but it still uses ReadUdpMsg (which we implement) but the next release would move to batch reads, meaning pfilter package would have to play catch up. I'd rather quic-go was released, I'd update pfilter to support ReadBatch, and forget about it (atleast until quic-go changes something again).
Ideally, quic-go would provide a way to read packets it does not recognize to be quic packets, that way I would not need pfilter package at all and could give quic-go a real net.UDPConn and trust that I somehow can recover stun packets that arrived on the same net.UDPConn, without really caring how quic-go implemente reads from net.UDPConn

Syncthing is stuck on an old version of quic-go as we are wary of breaking connectivity between versions like we did in the past by not reading quic-go release notes about compatability.

That shouldn't be a problem if you're using tagged releases. As long as the QUIC (draft) version matches, it should be interoperable. Depending on how old your quic-go version is though, we might have dropped support for that particular draft version in a later release.

I guess now that RFC9000 is out, we should upgrade and keep rolling it forward, as I trust it should now never become incompatible

Sounds like a good idea. That version will be supported for a long time.

Depending on how old your quic-go version is though, we might have dropped support for that particular draft version in a later release.

I think that is exactly what happened, and our current version is 19.3, so I think we are going to have a breaking change regardless, as we move towards RFC9000.
Given the releases are ad-hoc, are you waiting for some specific PRs to land before you move forward? I guess I could upgrade us to the latest untagged commit, and assume nothing will break between that and the next future tag.

Given the releases are ad-hoc, are you waiting for some specific PRs to land before you move forward?

Not really, although I'd probably want https://github.com/lucas-clemente/quic-go/pull/3230 to go in (mind giving me a review, should be very straightforward?). Once that's in, I could cut a new release, there've been a couple of important changes since the last one.

Ideally, quic-go would provide a way to read packets it does not recognize to be quic packets, that way I would not need pfilter package at all and could give quic-go a real net.UDPConn and trust that I somehow can recover stun packets that arrived on the same net.UDPConn, without really caring how quic-go implemente reads from net.UDPConn

@marten-seemann does quic-go provide a mechanism like this? I'm not a go dev but unknownPacketHandler looks interesting.

@marten-seemann does quic-go provide a mechanism like this? I'm not a go dev but unknownPacketHandler looks interesting.

It doesn't, mainly because of the implementation complexity: There are a lot of places where we drop packets, for various reasons (grep the code for DroppedPacket for a ~90% complete list), and we'd have to somehow hand back the packet from all of them.

That said, if anyone comes up with a solution that's not super ugly I'd be willing to consider it.


Virtualenv wrapper not working?

Hey, I have installed virtualenvwrapper using pip.I am aware that there is a plugin for using it but it doesn't seem to work for me when I run mkvirtualenv xyz. So are there some configurations I need to do?

How did this turn out? Was it solved?

This worked for me

This worked for me :+1:

it works for me after typing this
source /usr/local/bin/virtualenvwrapper.sh
then mkvirtualenv

The plugin should be updated. The default location of this script changed. Opened a pull request.


DOC+MAINT: optimize: lb and ub in the Bounds constructor are not optional

Also fix a really long line.

Do you want to add a test comparing the repr against the expected string?

The repr can change over time (e.g. it changed when lb and ub were passed through np.asarray before being saved as attributes), so I don't know if an exact match of the repr to a string is what we want.  How about evaling the repr and checking that we get back what we put in?  Something like
b = Bounds([1, 2], 100)
s = repr(b)
b2 = eval(s)
assert_array_equal(b.lb, b2.lb)
assert_array_equal(b.ub, b2.ub)
assert_array_equal(b.keep_feasible, b2.keep_feasible)

(and another with an explicit keep_feasible argument).

Sure. Sounds good.

That was easy... it turns out the tests are already there: https://github.com/scipy/scipy/blob/ac23f85a38e7fa8a4447ff53d7cda079ab09315a/scipy/optimize/tests/test_constraints.py#L171

OK I'll merge when the last test finishes.

I don't know if the last test is going to finish.

I restarted the stalled job. They're all green now.


error 127

[+] Building 2.8s (14/15)                                                                          docker:desktop-linux  => [internal] load build definition from Dockerfile                                                               0.0s  => => transferring dockerfile: 735B                                                                               0.0s  => [internal] load metadata for docker.io/library/python:3.9.16-slim-buster                                       1.3s  => [auth] library/python:pull token for registry-1.docker.io                                                      0.0s  => [internal] load .dockerignore                                                                                0.0s  => => transferring context: 471B                                                                                0.0s  => [1/9] FROM docker.io/library/python:3.9.16-slim-buster@sha256:83c6f59ca894629d710ea03243819b8af63e1b074c5a1bd  0.0s  => => resolve docker.io/library/python:3.9.16-slim-buster@sha256:83c6f59ca894629d710ea03243819b8af63e1b074c5a1bd  0.0s  => [internal] load build context                                                                                0.0s  => => transferring context: 584B                                                                                0.0s  => [3/9] ADD --chmod=755 https://astral.sh/uv/install.sh /install.sh                                              1.1s  => CACHED [2/9] RUN apt-get update      && apt-get install -y      curl     tesseract-ocr     && apt clean     &  0.0s  => CACHED [3/9] ADD --chmod=755 https://astral.sh/uv/install.sh /install.sh                                       0.0s  => CACHED [4/9] RUN chmod +x /install.sh                                                                          0.0s  => CACHED [5/9] RUN /install.sh && rm /install.sh                                                                 0.0s  => CACHED [6/9] WORKDIR /app                                                                                0.0s  => CACHED [7/9] COPY requirements.txt ./                                                                          0.0s  => ERROR [8/9] RUN /root/.cargo/bin/uv pip install --system --no-cache -r requirements.txt                        0.4s ------                                                                                > [8/9] RUN /root/.cargo/bin/uv pip install --system --no-cache -r requirements.txt:                                   0.319 /bin/sh: 1: /root/.cargo/bin/uv: not found                                                                        ------                                                                                2 warnings found (use docker --debug to expand):                                                                        - UndefinedVar: Usage of undefined variable '$PYTHONPATH' (line 14)                                                     - LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format (line 14)              Dockerfile:24                                                                                --------------------                                                                                22 |     WORKDIR /app                                                                                23 |     COPY requirements.txt ./                                                                                24 | >>> RUN /root/.cargo/bin/uv pip install --system --no-cache -r requirements.txt                                    25 |     COPY . ./                                                                                26 |                                                                                --------------------                                                                                ERROR: failed to solve: process "/bin/sh -c /root/.cargo/bin/uv pip install --system --no-cache -r requirements.txt" did not complete successfully: exit code: 127

Getting same error

@jakubszkaradek @sanjaychugh1287 thanks for the heads up, it is fixed now ;)


Semgrep Upgrade and add rules.

I am opening this with just two rules ported so that you guys can see that I am heading in the correct direction. Let me know if these rules can be written better.
I am focusing on porting js and python rules which don't have good/bad lines and whose pattern is not just regex but a proper statement/expression.
❯ time ./tools/lint --only=semgrep-py
./tools/lint --only=semgrep-py  6.69s user 0.16s system 99% cpu 6.874 total

@andersk FYI

Merged as the series ending with 9089fd5b08ec0b4b470df2336246e343754722d3, thanks @amanagr!  I figure since this is a highly iterative process, you can just address the notes above in the next PR :)

Sorry for the trouble. I will keep that in mind next time.


*: decouple FlashGrpcServerHolder from Server.cpp

What problem does this PR solve?
Issue Number: ref #4609
Problem Summary:
What is changed and how it works?
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-all-tests

/run-all-tests

I will move it into anonymous namespace
---Original---
From: @.>
Date: Mon, Aug 1, 2022 18:22 PM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [pingcap/tiflash] *: decouple FlashGrpcServerHolder from Server.cpp (PR #5516)
@SeaRise commented on this pull request.
In dbms/src/Server/FlashGrpcServerHolder.h:
> + +#include <Common/assert_cast.h> +#include <Debug/astToExecutor.h> +#include <Flash/DiagnosticsService.h> +#include <Flash/FlashService.h> +#include <Flash/Mpp/GRPCCompletionQueuePool.h> +#include <Server/RaftConfigParser.h> + +namespace DB +{ +namespace ErrorCodes +{ +extern const int IP_ADDRESS_NOT_ALLOWED; +} // namespace ErrorCodes + +void handleRpcs(grpc::ServerCompletionQueue * curcq, const LoggerPtr & log);
Is this function referenced by other c++ files?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

/run-all-tests

/merge

/merge

/run-all-tests


Bug when used with QGLWidget and overflown toolbar

I have a very specific issue, which I only noticed this week but was present since March 2021.
This issue is triggered

when a toolbar does not have enough space to display all its buttons
when a QGLWidget is inserted in the QMainWindow
In that context, when you press the >> button and try to hover on the extra buttons, the toolbar will close as soon as the mouse is on top of the QMainWindow

EDIT: I know this is very specific, I tried to comment out bits of code until the bug is gone.
I'm still debugging with the code from March and this seems to be the infamous call:
void qCoreAppFixup()
{
    if (!QCoreApplication::testAttribute(Qt::AA_DontCreateNativeWidgetSiblings)) {
        QCoreApplication::setAttribute(Qt::AA_DontCreateNativeWidgetSiblings);
    }
}

So one work around could be:
 for (QToolBar* tbar : findChildren<QToolBar*>())
    tbar->setAttribute(Qt::WA_NativeWindow);

Really sorry for the delayed response. I somewhat missed or forgot about this issue, I apologize.

I'm still debugging with the code from March and this seems to be the infamous call:

Does this block of code cause any issues? If so, it can be removed. This code was added because I read some commercial Qt components source code and some of them use that code to avoid some Qt bugs, but since I've never actually seen such bugs, the code may be unnecessary. If it's causing problems, it definitly should be removed.

So one work around could be:

Make it a native window will cause it go through our native event filter, there will be some behavior change of course.

However when doing so, the "window resizing area" is reduced when hovering on the toolbars

They now will go through our native event filter, so the resizing area is controlled by the event filter, the temporary solution is modify the logic of WM_NCHITTEST, but maybe should add a API for such situations.

I'm still debugging with the code from March

The code from 2021.03 ? It's really outdated, bro. There are many improvements since then. I suggest use or debug the latest code instead.

Hello, The bug was present with latest code as well.
It actually forced me to port from QGLVidget to QOpenGLWidget so the issue is gone.
If your remove the Qt::AA_DontCreateNativeWidgetSiblings related code, then I get RTTI qWarning each time I maximize/restore the window.

It actually forced me to port from QGLVidget to QOpenGLWidget so the issue is gone.

It seems that QGLWidget is quite old and outdated and Qt official also recommends to migrate to the new QOpenGLWidget

You're absolutely right. We can close this issue, maybe had a note in the Readme regarding  incomaptibility with QGLWidget

We can close this issue, maybe had a note in the Readme regarding incomaptibility with QGLWidget

OK. I'll update the README file and then close this issue.

QGLWidget is not supported, closed. Feel free to reopen it if you think it's necessary.


Add metadata

Would it be possible to link a bio.tools entry (or edam terms)? See https://docs.galaxyproject.org/en/latest/dev/schema.html#tool-xrefs
The former would be sufficient.
The Galaxy  (in particular the metagenomics) community tries to add this to as many tools as possible.

https://github.com/qiime2/q2galaxy/pull/54


Make the "Discover services" URL configurable

As someone deploying a multi-user OpenRefine instance I would like to be able to set a new URL for the "Discover services" button so that users can go to documentation regarding internal reconciliation endpoints rather than the Reconciliation service test bench to which anyone can add endpoints.
Proposed solution
Add a setting to OpenRefine allowing one to set a custom URL for the "Discover services" button.
Alternatives considered
One option could be a setting to remove the button as a custom OpenRefine deployment can "pre-set" reconciliation services anyway. However, I could still see cases in which one would not like to "pre-set" all of ones services(multilingual services, etc).

Hmm, storing the setting in a preference doesn't seem ideal for deployment needs.  Needs to be a config file.
So maybe a preference variable read from the refine.ini or it's JAVA_OPTIONS?
Just wondering on the deployment side of things...seems like perhaps the refine.ini (a config file) is the way to make it easier for multi-user deployment?

@thadguidry yeah, refine.ini might be the better place. I do always end up making some configuration using settings anyway, but in this case it would probably be preferred if users can't edit it.

Ah, on that note... what configuration settings do you set and where?  Do we need to open those up to be read from refine.ini to help you more?

I am not too sure about making this link configurable. As a user I would not suspect that the behaviour of this button could be different if I am using OpenRefine from some institutional deployment.
How about making the list of services initially registered in OpenRefine configurable instead? At the moment, my Wikidata recon endpoint is hardcoded, but that is quite bad (for instance because it could go down any time).
The advantage with this solution is that the user would not even have to register the services themselves by copying a URL into a form.

Ah, on that note... what configuration settings do you set and where? Do we need to open those up to be optionally read from refine.ini to help you more?

@thadguidry I currently just set reconciliation endpoints by making a call to to the API which works very well.

I am not too sure about making this link configurable. As a user I would not suspect that the behaviour of this button could be different if I am using OpenRefine from some institutional deployment.

@wetneb could it be that the button should look differently altogether to make the behavior more open-ended? It's quite confusing as it is given that it does not behave like the other buttons in the footer, and that it isn't styled like other external links in the reconciliation dialog.

How about making the list of services initially registered in OpenRefine configurable instead? At the moment, my Wikidata recon endpoint is hardcoded, but that is quite bad (for instance because it could go down any time).
The advantage with this solution is that the user would not even have to register the services themselves by copying a URL into a form.

This is in my opinion already solved as there is a setting which can be set with the API.

I currently remove the button altogether with some CSS but will likely move to an patch to change the URL instead as a multilingual reconciliation service I have follows the same pattern as the Wikidata one resulting in a lot of separate services.

How about making the list of services initially registered in OpenRefine configurable instead? At the moment, my Wikidata recon endpoint is hardcoded, but that is quite bad (for instance because it could go down any time).

@wetneb That was the idea I was driving at.  To set the endpoints made available in the UI by allowing them to be listed/setup in refine.ini to make it easier for administrators at institutions to deploy.  Basically, all such "settings" or "preferences" should be able to be read from a file (and maybe we just eventually move to a TOML file like other tools do for "config file" handling).


[EventRequest] FinanceChargeMemoLine.Table.al (table 303 "Finance Charge Memo Line")

Hi MS,
Please help us in migrating our verticalized solution by adding below Event in FinanceChargeMemoLine.Table.al Table.
Standard Code

Modified Code

Event Signature

Thanks
SUD

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Any update on this event?
Thanks
SUD

HI There,
I think the published event in Master branch have wrong second parameter.

It should be record parameter of Table 302 (Finance ChargeMemo Header).
Kindly Update!!
Thanks
SUD

@bc-ghost
@StanPesotskiy

Please create a new request. Don't forget to mention the precise name of the object and the event.
Please always put all details in text instead of screenshots.


Setting default permission to true is doing nothing.

I'm an admin for my server, I've been trying since I installed sponge to give myself a default permission of true, tried a whole bunch of different ways to more or less give myself access to all commands, and none of them work.
Doing
/pex user tooforgetful def true
does absolutely nothing, and still has plenty of commands locked out for whatever reason.
Creating an admin group with default permissions and adding myself to it has the same effect, sill not giving me access and making me run some commands through the console (some of which cannot work properly through it, making them effectively worthless)

Serious question: Where did you learn that was a command?

Yeah, this is PEX for Sponge that there's an issue with here.
The only thing I can think of being an issue here is that you ran this command before your user had logged in -- when PEX2 does uuid lookups it only uses the local cache since command argument handling is synchronous and we don't want to block the main thread.
Check your permissions.json to confirm if the default was set under your uuid. If it wasn't, delete the existing entry and after having logged in to your server at least once set the default again (confirming that it's stored by UUID).
If that doesn't work, we'll have to investigate further.

It's in the sponge list of commands, and is typically used for assigning default values to groups.
For example, doing
/pex group Default def false
Will set the permissions for every command for the group Default to false, meaning nobody in the Default group can use commands, unless you specifically set them. You can see it with a couple more examples over in this thread over here -> https://forums.spongepowered.org/t/guide-pex-quick-command-reference/8738
As far as the issue itself though, this seems to be an issue with if a user is in a group that has a default param of false. I ended up completely wiping the permissions.json file and starting over from scratch, and making an admin group which defaulted to true for all commands and then adding myself to it seemed to work just fine, but hadn't before when the admin group was created after a default with a false default.
Long story short, it seems like it'll work unless there's a conflicting global default somewhere else, even if you specifically set it different for the user.

Yeah, negative overrides positive of the same weight I guess
What you could do for yourself is /pex user someone def 2
(Permissions are set with int-weighted values -- positive evaluate to true, negative to false, 0 to check parent, and higher numbers override lower ones)

Ahhh! I was looking in the wrong place for commands.  Silly me.


[gdal] build failure

hi,
I installed gdal successfully in other machine, but failed in new machine.
it seems that something wrong in include path.
cc1plus: warning: /home/cuhk/vcpkg/installed/x64-linux/debug/lib/pkgconfig/../../../../include: No such file or directory [-Wmissing-include-dirs]
it does not exist: /home/cuhk/vcpkg/installed/x64-linux/debug/lib/pkgconfig/../../../../include
it exists :             /home/cuhk/vcpkg/installed/x64-linux/debug/lib/pkgconfig/../../../include
Host Environment

OS: ubuntu 20.04
Compiler: gcc 11

To Reproduce
Steps to reproduce the behavior:
./vcpkg install gdal
Failure logs
-Cut and paste the appropriate build messages from the console output.
-Please attach any additional failure logs mentioned in the console output.
Computing installation plan...
The following packages will be built and installed:
    gdal[core,hdf5,netcdf,postgresql,supported-default-features]:x64-linux -> 3.4.1#1
Detecting compiler hash for triplet x64-linux...
Restored 0 packages from /home/cuhk/.cache/vcpkg/archives in 5.379 us. Use --debug to see more details.
Starting package 1/1: gdal:x64-linux
Building package gdal[core,hdf5,netcdf,postgresql,supported-default-features]:x64-linux...
-- Using cached OSGeo-gdal-v3.4.1.tar.gz.
-- Cleaning sources at /home/cuhk/vcpkg/buildtrees/gdal/src/v3.4.1-8905a3e6e8.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source /home/cuhk/vcpkg/downloads/OSGeo-gdal-v3.4.1.tar.gz
-- Applying patch 0001-Fix-debug-crt-flags.patch
-- Applying patch 0002-Fix-build.patch
-- Applying patch 0004-Fix-cfitsio.patch
-- Applying patch 0005-Fix-configure.patch
-- Applying patch 0007-Control-tools.patch
-- Applying patch 0008-Fix-absl-string_view.patch
-- Using source at /home/cuhk/vcpkg/buildtrees/gdal/src/v3.4.1-8905a3e6e8.clean
-- Getting CMake variables for x64-linux-dbg
-- Getting CMake variables for x64-linux-rel
-- Generating configure for x64-linux
-- Finished generating configure for x64-linux
-- Configuring x64-linux-dbg
-- Configuring x64-linux-rel
-- Building x64-linux-dbg
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:158 (message):
    Command failed: /usr/bin/make V=1 -j 9 -f GNUmakefile all
    Working Directory: /home/cuhk/vcpkg/buildtrees/gdal/x64-linux-dbg/
    See logs for more information:
      /home/cuhk/vcpkg/buildtrees/gdal/build-x64-linux-dbg-out.log
      /home/cuhk/vcpkg/buildtrees/gdal/build-x64-linux-dbg-err.log

Call Stack (most recent call first):
  scripts/cmake/vcpkg_build_make.cmake:187 (vcpkg_execute_build_process)
  scripts/cmake/vcpkg_install_make.cmake:26 (vcpkg_build_make)
  ports/gdal/portfile.cmake:317 (vcpkg_install_make)
  scripts/ports.cmake:145 (include)

Error: Building package gdal:x64-linux failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `git pull` and `./vcpkg update`.
Then check for known issues at:
  https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+gdal
You can submit a new issue at:
  https://github.com/microsoft/vcpkg/issues/new?template=report-package-build-failure.md&title=[gdal]+Build+error
including:
  package: gdal[core,hdf5,netcdf,postgresql,supported-default-features]:x64-linux -> 3.4.1#1
    vcpkg-tool version: 2022-02-01-c76e0def147898591d9c35113b95e0b3f2eaa4d4
    vcpkg-scripts version: ce7ee5ac3 2022-02-04 (8 hours ago)

Additionally, attach any relevant sections from the log files above.

build-x64-linux-dbg-err.log
build-x64-linux-dbg-out.log
Additional context
Add any other context about the problem here, such as what you have already tried to resolve the issue.

Can you provide config.log...?
Did you build from a clean install tree? The gdal build relies on proper pkg-config data. You might try to grep  for ../../../.. in debug/lib/pkgconfig/*.pc. If you have a hit, remove the offending package.

@dg0yt  thank you for your reply.
config-x64-linux-dbg-err.log
config-x64-linux-dbg-out.log
I build from a clean install tree, so there is no gdal.pc in pkgconfig.

I really mean the config.log......log files. This is the detailed log from configure.

@dg0yt  there is no config.log  file in buildtrees/gdal.
could you tell me where to find it ?

Originally, it is created as  config.log in each build directory (x64-libux-dbg etc.).

@dg0yt

... because it is moved after successful configure. So look again for config.log-x64-linux-dbg.log next to the other logs.

i found the config.log-x64-linux-dbg.log file @dg0yt
config.log-x64-linux-dbg.log

It is strange, all paths from pkg-config data carry an extra ../, e.g.
EXPAT_CFLAGS='-I/home/cuhk/vcpkg/installed/x64-linux/debug/lib/pkgconfig/../../../../include'
EXPAT_INCLUDE='-I/home/cuhk/vcpkg/installed/x64-linux/debug/lib/pkgconfig/../../../../include'
EXPAT_LIBS='-L/home/cuhk/vcpkg/installed/x64-linux/debug/lib/pkgconfig/../../../lib -lexpat'

Can you please also share installed/x64-linux/debug/lib/expat.pc?
This is mine from a fresh build, and it looks valid:
prefix=${pcfiledir}/../..

exec_prefix=${prefix}
libdir=${exec_prefix}/lib
includedir=${prefix}/../include

Name: expat
Version: 2.4.1
Description: expat XML parser
URL: http://www.libexpat.org
Libs: -L"${libdir}" -lexpat
Cflags: -I"${includedir}"

Are there any environment variables set that modify PKG_CONFIG behaviours? (env |  grep PKG_CONFIG)?

expat.pc:
prefix=${pcfiledir}/../../..
exec_prefix=${prefix}
libdir=${exec_prefix}/lib
includedir=${prefix}/../include
Name: expat
Version: 2.4.1
Description: expat XML parser
URL: http://www.libexpat.org
Libs: -L"${libdir}" -lexpat
Cflags: -I"${includedir}"

I just reinstalled vckpg, and then installed gdal, it has the same problem.

Hm, where does the wrong prefix come from?
The prefix= line is written by vcpkg_fixup_pkg_config at after the actual build and installation:
https://github.com/microsoft/vcpkg/blob/7e7dad5fe20cdc085731343e0e197a7ae655555b/scripts/cmake/vcpkg_fixup_pkgconfig.cmake#L194
(Before fixup, it is an absolute path. You can see it in the build directory.)
Note: this is not an gdal issue. It is an issue with pc files.

which part would produce the wrong path? how can i to debug ?
it is strange that my another computer is ok, they have same system and development environment.

The 'prefix=' key or variable should contain an absolute path!
@insar-dev Most probably your environment is broken. How did you install expat?

@sfhacker  expat  was installed automatically by vcpkg. expat is one of gdal dependencies.
i found all gdal dependencies pc file have same issue.
GEOS.pc
prefix=${pcfiledir}/../../..
exec_prefix=${prefix}
includedir=${prefix}/../include
libdir=${exec_prefix}/lib
Name: GEOS
Description: Geometry Engine, Open Source - C API
Version: 3.10.0
Libs: -L"${libdir}" -lgeos_cd -lgeosd -lstdc++ -lm
Requires:
Cflags: -I"${includedir}"
libpng.pc:
prefix=${pcfiledir}/../../..
exec_prefix=${prefix}
libdir=${prefix}/lib
includedir=${prefix}/../include/libpng16
Name: libpng
Description: Loads and saves PNG files
Version: 1.6.37
Libs: -L"${libdir}" -lpng16d -lz -lm
Requires: zlib
Cflags: -I"${includedir}"

Where have I seen similar issues...

The 'prefix=' key or variable should contain an absolute path!

prefix=${pcfiledir}/../../.. is an absolute path.

How did you install expat?

expat was installed automatically by vcpkg.
i found all gdal dependencies pc file have same issue.

This was visible from config.log.

which part would produce the wrong path? how can i to debug ?

If you do know cmake, you can instrument the section in vcpkg_fixup_pkg_config to trace how the new prefix is derived.

i manually modified pc files.
And now vcpkg can install gdal normally.

It may occur again as soon as you restart from a fresh tree, or pull updates from vcpkg.
So far, it seems to be not reproducable for anyone else.

i manually modified pc files.

I don't think this is a proper fix or solution. Next time you update/upgrade, the same issue will occur. Need to improve our troubleshooting and bug fixing skills.

I have tried to solve the issue, but failed.
Could someone give some more detailed suggestions?

Could someone give some more detailed suggestions?

I suggested this:
If you do know cmake, you can instrument the section in vcpkg_fixup_pkg_config to trace how the new prefix is derived.
If you want more details, you should indicate which prior knowledge we can assume.

@insar-dev Are you sophomore?
Post here what you have tried along with the error messages and we will assist you.

i am not familiar with cmake, i will try my best to do it.

hi,
i added some message to track issue.
finally find that relative_pc_path value is:../../..

relative_pc_path:/home/cuhk/vcpkg/packages/expat_x64-linux/debug
pkg_lib_search_path:/home/cuhk/vcpkg/packages/expat_x64-linux/debug/lib/pkgconfig/expat.pc
modified relative_pc_path: ../../..

Could you please also print ${file} right after foreach?
I expect file to be
/home/cuhk/vcpkg/packages/expat_x64-linux/debug/lib/pkgconfig/expat.pc,
but then we should get pkg_lib_search_path as
/home/cuhk/vcpkg/packages/expat_x64-linux/debug/lib/pkgconfig
which it isn't according to your report.

file is :/home/cuhk/vcpkg/packages/expat_x64-linux/debug/lib/pkgconfig/expat.pc

Can you please clone a new vcpkg and try again?

So it comes down to an issue with cmake_path(GET file PARENT_PATH pkg_lib_search_path). This is pure CMake, and the wrong result is only on your computer. It is a fairly new command, but I didn't find any issue reports about it.
For reference, I already tested this script with CMake 3.21 on my Ubuntu 18.04 computer:
message(STATUS "CMake ${CMAKE_VERSION}")
set(file /home/cuhk/vcpkg/packages/expat_x64-linux/debug/lib/pkgconfig/expat.pc)
message(STATUS "file:                ${file}")
cmake_path(GET file PARENT_PATH pkg_lib_search_path)
message(STATUS "pkg_lib_search_path: ${pkg_lib_search_path}")

from my vcpkg dir with:
$ downloads/tools/cmake-3.21.1-linux/cmake-3.21.1-linux-x86_64/bin/cmake -P test.cmake
-- CMake 3.21.1
-- file:                /home/cuhk/vcpkg/packages/expat_x64-linux/debug/lib/pkgconfig/expat.pc
-- pkg_lib_search_path: /home/cuhk/vcpkg/packages/expat_x64-linux/debug/lib/pkgconfig

Can you please clone a new vcpkg and try again?

@JackBoosY I have done it yesterday.

@dg0yt i will try to reinstall cmake to 3.21.

Are you targeting x64-linux from Windows? Is this WSL?

@dg0yt yes, not WSL, is ubuntun20.4
I reinstalled vcpkg and  reinstalled cmake to 3.21.1, vckpkg can install gdal successfully now.
There is a bug in cmake 3.22.2.

Can anyone report this bug to cmake https://gitlab.kitware.com/cmake/cmake/-/issues?

let me report this bug to cmake.

It is important to include the environment into the CMake bug report. This function is unit-tested.

I reported the bug to cmake.
https://gitlab.kitware.com/cmake/cmake/-/issues/23187

I have changed cmake to 3.21.1,  then there is another issue:
cmake can not find some libraries that build by vcpkg(using cmake 3.22.2)

How to solve it?  Thanks for your suggestion.

I don't see anyhing related to gdal here.
In any case, you should get rid of the cached artifacts which were built with the broken cmake.

I will try it . Thanks.

@insar-dev Can you please follow the cmake mantainer's suggestion and provide the output?
I can't repro this issue using cmake 3.22.2 locally.
Thanks.

@insar-dev The python-pip side has been fixed this, I think you should re-install cmake using pip now.


Add a store of the original function

Otherwise it's impossible to ever unwrap it, which can be required in order to allow more types.
It would nice to also have a fallback where if it's not found in the arg list to just call original, and have a type parameter true/false turn that on/off.

Nevermind, found it.


[New Package] Added MartiCliment.UniGetUI.Pre-Release version 3.1.0-alpha1

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[x] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

https://github.com/microsoft/winget-pkgs/issues/173379
Microsoft Reviewers: Open in CodeFlow

@stephengillie, This package had problems with installing correctly even though it does install correctly, this was the first of UniGetUI, it can be published
https://github.com/microsoft/winget-pkgs/pull/156394

Automatic Validation ended with:

No errors to post.

(Automated response - build 895.)

Verify:

Watching this install reminded me why this package doesn't pass the pipelines - because for some reason, the package manager mis-detects the dependency installer as the main installer, and so it returns and gives exit codes once Microsoft.EdgeWebView2Runtime is installed, but before this package has completed. Or a similar situation.

@wingetbot waivers Add Validation-Installation-Error


Add versions panel for all entities

@Sachin-chaurasiya and @darth-coder00, please add tasks breakdown to this issue.

Closing this as we have added a version panel for all the entities (Table, Topic, Pipeline, Dashboard)


Using own retrieval results

Hello. I am attempting to experiments with using my own retrieval results and assessing the impact on the performance of the model (for the full-wiki setting)
If I understood correctly, the process would be as follows.
Generate my own version of hotpot_dev_fullwiki_v1.json in which the context entry of the JSON has been modified with my own paragraphs. All the other keys should be left identical. I shall refer to this json file as custom_dev_fullwiki.json.
Likewise for the test set, to create custom_test_fullwiki.json
Then, I would run the preprocessing , training and evaluation commands as specified in the README.
To make meaningful comparison of metrics, once I change my retrieval method and generate a 'v2.json' set of files, I should start again from scratch on preprocessing, then train etc.
Is this correct?

Yeah, I think you are right :)


Servant 0.18 and increase upper bounds [CircuitHub's PR #43 rebased on latest master]

@ocharles
We can either merge this PR or you can add commit beb9f11 to your original PR and then we can go ahead and merge it.
I'm okay with either, let me know your preference 😄

Please go ahead and do what you want, I unfortunately don't have time to do more on this I'm afraid!

Okay, no worries.
I don't think I can commit/push to the other PR since the branch is from CircuitHub's fork, but I'll just go ahead and merge this 😄
Do let me know if there are any other changes, updates or enhancements you'd like to see for this project & I can try my best to implement them 👍🏻

Sounds good! Thanks folks :smiley:

@kahlil29 Thanks for all of this work. The last piece of the puzzle would be to upload to Hackage - could you do that?

@ocharles Sure, will do. Should be done early next week 👍🏻

@ocharles Released version 0.1.3 to Hackage.
I'm not familiar with doing these releases so had to do a couple to (hopefully) get it right.
Do let me know if something does not look right and I'll fix it.


[AVM Module Issue]: Make hub-virtual-network-connection a top level resource

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Issue Type?
Feature Request
Module Name
avm/res/network/virtual-hub
(Optional) Module Version
No response
Description
I have a requirement for attaching new virtual networks to an existing virtual-hub. The current sub-module hub-virtual-network-connection is not available seperately to deploy this resource, and is only available when deploying a virtual-hub directly (at least as far as I can determine).
(Optional) Correlation Id
No response

Hey @HexagonSam,
this issue will unfortunately join a pool of several other, very similar, issues that request the child-modules to be published:

https://github.com/Azure/bicep-registry-modules/issues/1390
https://github.com/Azure/bicep-registry-modules/issues/1933
https://github.com/Azure/bicep-registry-modules/issues/2899
https://github.com/Azure/Azure-Verified-Modules/issues/578
https://github.com/Azure/Azure-Verified-Modules/issues/747

The short of it is that we the way the module's are structured is so by design to align with the ProviderNamespace-ResourceType structure of ARM. Much of the CI and processes are designed around that, with the clear goal of eventually enabling the publishing of child-modules again (which are tested through their parent) - just as it is done in the CARML - CI.
I already described in a few places why it is not a straight forward solution to implement as it's effectively made very hard by a manual process we have no control over - but I really hope we can get this sorted rather sooner than later.
That said, by mentioning the other issues in this comment, this issue is effectively now connected to them so that we can easily find it again. For this issue in particular that means however that I'll close it not as closed (because it isn't), but as 'not planned' (even though that's also not entirely accurate). We have an internal backlog item to track progress on this where it will be tracked going forward.
Sorry for not having better news to offer :-/ For the time being, for connections specifically, you'd need to deploy them through the hub module, or by using a copy of the module, or by implementing the raw bicep code.


Pin docutils version

Need to downgrade docutils to v0.17.1 for the rendering with the bibliography block to work (see related GH issue). Upgrading docutils to v0.20.1 doesn't work for now as this results in dependency hell for myst-parser, sphinx and jupyter-book.

Also currently recommended in the official documentation.


Try to get X moving from within LinuxCNC Axis

Enable all axis controllers when not in ESTOP, and connect X pid controller
to servo-x-enable and x-output in the hope that this will allow X control via GUI.

Together with #8 I was able to get the machine out of ESTOP, by doing 'halcmd sets power-on 1' and turning the LinuxCNC GUI "on", followed by trying to move X.  Sadly no movement happened, only an error message about joint 0 showed up.  Did not write down the message.

this is the wrong approach.


retain/report build.log

Is your feature request related to a problem? Please describe.
there are two issues with build.log:

its overwritten each build
is not reported, and deleted in the pip (even when an error occurs

Describe the solution you'd like

maybe label with build time?
report/ write to stdout on error condition

I might be wrong but some community codes (e.g. ph4, simplex ) seem to use C++ MPI bindings that are now obsolete. What is the status on this? amuse v12.0.0 (tag c442d27) still presents these bindings.

simplex still needs to be done #446


Apache2 Reverse Proxy with OpenID Connect result in a bad Request

I used the setup as described above in combination with .net core. This works for most scenarios except when using in combination with OpenID connect. After the authentication, the result is:
Bad Request
Your browser sent a request that this server could not understand.
Size of a request header field exceeds server limit.
Apache/2.4.29 (Ubuntu) Server

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

ID: cfd9a1f0-c1b6-4443-0700-4c5beafeed03
Version Independent ID: c22ca7a0-d1d4-200d-fc36-a9798d350c6e
Content: Host ASP.NET Core on Linux with Apache
Content Source: aspnetcore/host-and-deploy/linux-apache.md
Product: aspnet-core
GitHub Login: @spboyer
Microsoft Alias: spboyer

I read online that Apache has a default of 8K (should be able to confirm with command: getconf PAGESIZE) and that it can be configured via LimitRequestLine and LimitRequestFieldSize. You should be able to set those for the virtual host. However, I'm merely looking at online resources and there are very likely security/perf issues, sooooo ........... drum roll plz 🥁 ........... ping @shirhatti

Thank you guardrex. Adding LimitRequestFieldSize was the solution. The result from signin-oidc is 9.2 KB, so I guess adding this to the documentation would be nice.

Cool.
We'll hear back from Sourabh soon.
If we're covering this scenario, he'll tell  me what the topic should say.

It's hard to give prescriptive guidance especially since the size of the header depends on the number of claims. I'd rather just mention you may need to increase the header size.
See https://github.com/aspnet/Docs/issues/8679 which is in similar vein where certain workloads requires change in configuration of your proxy and we have to be cognizant that we aren't going to cover all scenarios.


confirmMicroDeposits Fix

Fixed an issue in Amount.Equal()
Slightly refactored SQLDepositoryRepo.confirmMicroDeposits()
Rewrote the unit tests for both functions

@adamdecaf

Codecov Report

Merging #247 into master will decrease coverage by 4.11%.
The diff coverage is 16.66%.

@@            Coverage Diff            @@
##           master    #247      +/-   ##
=========================================
- Coverage   60.92%   56.8%   -4.12%
=========================================
  Files          31      31
  Lines        4478    4480       +2
=========================================
- Hits         2728    2545     -183
- Misses       1231    1440     +209
+ Partials      519     495      -24

Impacted Files
Coverage Δ

microDeposits.go
62.7% <0%> (-1.7%)
:arrow_down:

common.go
85% <100%> (ø)
:arrow_up:

accounts.go
10.95% <0%> (-67.13%)
:arrow_down:

internal/database/mysql.go
10.66% <0%> (-50.67%)
:arrow_down:

ofac.go
38.65% <0%> (-35.3%)
:arrow_down:

internal/filetransfer/sftp.go
47.08% <0%> (-24.87%)
:arrow_down:

internal/filetransfer/config.go
65.12% <0%> (-0.52%)
:arrow_down:

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 ec60632...2119ac2. Read the comment docs.

LGTM, thanks for the cleanup!


Window?

Having trouble in running the .yml environment

Having trouble in running the Makefile

Hi,
Seems like some package in yml was outdated, so that anaconda was not able to find it.
I will update yml file and you can try it again. The makefile error was caused by the incorrect yml environment.
You need to activate your env before make Makefile

.yml file still can't work.   The yml file is working under ubuntu 16.04. If you want to run under windows, please install these package manually.
python 3.0+:
pytorch
torchvision
pyymal
numpy
py-opencv
matplotlib
pillow
PIL
scipy
setuptools
cython
easydict
argparse
scikit-image
My code is written under Linux operating system, some file path in my code my not be correct.

I just need to have the proper environment listed to run test.py right? because the makefile can only be run linux

You would need to change to these lines to from model.nms.py_cpu_nms import py_cpu_nms
and change return py_cpu_nms(dets, thresh)
be warned:
NMS Speed will be much slower than native compiled code and GPU NMS (in  my case GPU NMS take 1 ms)

I can have it working in Window.. I will try using orignal script in Virtual Box linux
In VB I should have better performance compare to windows right? @dechunwang

No,
Virtual box will not able to run cuda library.(most virtual environment do not give access to host graphics card, except docker with nvidia-docker module) You need a cuda compatible GPU in order to run on GPU.

You can modify on this code, it is pretty self-explained
`
img = img.transpose(1, 2, 0)
img += cfg.PIXEL_MEANS
box_numpy = box.cpu().numpy().copy()
    for i in range(box_numpy.shape[0]):
        pt1 = tuple(box_numpy[i, 0:2])
        pt2 = tuple(box_numpy[i, 2:4])
        cv2.rectangle(img, pt1, pt2, (255, i, 255 - i))
    cv2.imwrite("test/{}.jpg".format(name), img)`

You would need to change to these lines to from model.nms.py_cpu_nms import py_cpu_nms
and change return py_cpu_nms(dets, thresh)
be warned:
NMS Speed will be much slower than native compiled code and GPU NMS (in my case GPU NMS take 1 ms)

What's the difference between from model.nms.py_cpu_nms import py_cpu_nms and from model.nms.cpu_nms import cpu_nms? Why can't I run the original code that you publish?

model.nms.cpu_nms is written in cython which is c-extension python, and can be compiled into machine depended low-level machine code. It is faster than interpreted python code (py_cpu_nms).  Because you are running on windows, a lot of things  are different than the Linux system.  If you run this project under ubuntu system, you will not experience that much trouble.

You can modify on this code, it is pretty self-explained
`
img = im_data.cpu().squeeze(0).numpy()
img = img.transpose(1, 2, 0)
img = img[:, :, (2, 1, 0)]
img += cfg.PIXEL_MEANS
box_numpy = cls_dets_single
    for i in range(box_numpy.shape[0]):
        pt1 = tuple(box_numpy[i, 0:2])
        pt2 = tuple(box_numpy[i, 2:4])
        cv2.rectangle(img, pt1, pt2, (255, i, 255 - i))
    cv2.imwrite("test/{}.jpg".format(name), img)`

Which script contain this?
I can't find it in test_utils.py

@dechunwang
why do we need to transpose bgr and adding back the subracted mean?
Based on this code it was done in the blob and save in a variable istead of modifying the actual image

Without transposing and adding back the mean

I was able to get this result

I am glad it worked. The previous code comes from another project I am working on, I did not test on this project.

@dechunwang
I edited the comment can you read it again? XD you reply too fast (good thing)

@dechunwang can I add you somewhere else for knowledge sharing purpose?


openssl hostname validation

copying fix from https://github.com/Azure/azure-c-shared-utility/pull/488, only copying adapters/tlsio_openssl.c

closing this one as we decide to update the whole c submodule:
https://github.com/Azure/azure-uamqp-python/pull/200

closing this one as we decide to update the whole c submodule:
https://github.com/Azure/azure-uamqp-python/pull/200


SST dev to use custom toolkitStack(CDKtoolkit stack) and Not update it

I have to customize the customToolKit so i created a customized version using
cdk bootstrap aws://<MY_AWS_ACCOUNT>/us-east-2 --template template.yaml
the issue i am facing is whenever i try to run sst dev
it tries to update the customToolKit i created above. I do not need the updates to ben done

There are some config options for these that we expose: https://docs.sst.dev/configuring-sst#config-options

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble


Added event for default rule "Read SSH Information"

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind documentation

/kind tests

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area commands

/area pkg

/area events

What this PR does / why we need it:
Added event for default rule "Read SSH Information"
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: GLVSKiriti
Once this PR has been reviewed and has the lgtm label, please assign leogr for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

It is also working as expected! Error log is produced on left terminal as expected

Kindly tell me if there are any changes!

/approve
I think in the future we might want to make createSshDirectoryUnderHome more reusable (by eg: taking a new parameter fileName string in order to allow it to be used to create any file in a temporary user folder directory).
For now, it LGTM!

Yess certainly It would be very useful