include/lib/mtimer: prevent overflow with large delays

Thank you! Apparently I still don't know all of C..


xeval.io website changed

the website is now a chatbot instead of the legislation website, remove the link

closed in https://github.com/mrciolino/portfolio/commit/5db1b9339ca16271456bc8f06420df7ad1c5ff11


terminate called after throwing an instance of 'std::out_of_range' when calling blueprint_library

I am running tutorial.py and the for loop at the end, which spawns 10 cars, causes the blueprint_library to sometimes throw the error:
terminate called after throwing an instance of 'std::out_of_range'
  what():  _Map_base::at

it doesn't happen every time which is strange.
There are some warnings at the start which may be relevant:
WARNING: Version mismatch detected: You are trying to connect to a simulator that might be incompatible with this API
WARNING: Client API version     = <IP_ADDRESS>-30-g31fff28b
WARNING: Simulator API version  = <IP_ADDRESS>-30-g31fff28b-dirty

I saw another issue with a similar problem but it was with the traffic manager.
Operating System: Ubuntu 18.04 LTS
I followed the guide last week to build carla, so it should be the latest version.
Also, is there a way to catch this error in python so that the finally clause runs to clean everything up?

Hi,
Thanks for pointing this out, we will be looking into this issue and solve it!

Seems to be fixed in CARLA 0.9.11, thanks guys!

Seems to be fixed in CARLA 0.9.11, thanks guys!


How to use weights on HuggingingFace?

Is it possible to use weights on HuggingingFace, e.g https://huggingface.co/google/gemma-2b-it ?
BTW - does it work on Rocm(AMD)?

interested as well!

It's the same model but this repo cannot use those weight formats directly. We have an internal export script that transforms other formats into our simple blob format, but this is not yet ready for production/open-sourcing.
Please comment on #11, which discusses which input formats would be useful.
We do not use Rocm, this repo is currently CPU-only, see #18.

@purefire adding an export script is a high priority to unlock model variations / fine tuning. Please chime in https://github.com/google/gemma.cpp/issues/11 on the source format that would be most useful.
Hi @julien-c :wave: :)

Consolidating weight export discussions to the issue https://github.com/google/gemma.cpp/issues/11 . We're working on making a script available soon - feel free to chime in with further comments there. Thanks!


Form validation using

Bug, feature request, or proposal:
Unable to show different validation messages for each field.
What is the expected behavior?
To show different error messages for each field like for "first name" would like to show
"required", "max length", "min length" error messages.
What is the current behavior?
currently able to show only error one message for each field.
What are the steps to reproduce?
currently creating a form using this plunker
https://plnkr.co/edit/y6akTq?p=preview
referring:- https://obsessiveprogrammer.com/validating-confirmation-fields-in-angular-reactive-forms-with-angular-material/
What is the use-case or motivation for changing an existing behavior?
Using ng-if with multiple mat-error tags.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular v4.4.6
Material v5.2.1
TypeScript v2.7.1
Is there anything else we should know?
no

And like this ? https://stackblitz.com/angular/lmmgqqjbxggx?file=app%2Fform-field-error-example.ts

yes, like this.
it is working fine for "email" but not for "maxlength"

Please keep GitHub issues for bug reports / feature requests. Better avenues for troubleshooting / questions are Stack Overflow, Gitter, mailing list, etc

Seems fine to me: https://stackblitz.com/edit/angular-t655tq?file=app/form-field-error-example.ts

Well, thats an option, but why this one is working fine:
                    <mat-error *ngIf="itemForm.get('status').hasError('required')">
                        {{ 'form.required' | translate }}
                    </mat-error>
                    <mat-error *ngIf="itemForm.get('status').hasError('pattern')">
                        {{ 'form.validation.digits' | translate }}
                    </mat-error>

And this one doesn't?

                <mat-error *ngIf="itemForm.get('signature').hasError('required')">
                    {{ 'form.required' | translate }}
                </mat-error>
                <mat-error *ngIf="itemForm.get('signature').hasError('maxLength')">
                    {{ 'form.validation.field.too_long' | translate }}
                </mat-error>

Component:

form = this.fb.group({
    id: [''],
    signature: new FormControl('', [Validators.required, Validators.maxLength(3)]),
    status: new FormControl('', [Validators.required, Validators.pattern('^[0-9]+$')]),
    last_action: ['']
});


fix(kit): prevent memory leak

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[x] Refactoring
[ ] Build or CI related changes
[ ] Tests related changes
[ ] Documentation content changes

@vladimirpotekhin please, review me


25/7 Frontend - Tackling User Feedback on Nav

Left dashboard menu on navigation has been feedbacked to contain too many elements, suggested to only keep the important functional pages. The routing could be better if the landing page was the feedback page based on user feedback.

Users also feedbacked that the UI/UX can be improved by adding a subtitle to let people know of the significance of the numbers

Closed with rectification of user feedback


UI Test : API Publisher readonly mode

The test verifies the functionality for read-only user introduced by the following issue : API Publisher readonly mode
Verifies that user should not be able to edit/create/publish APIs and only be able to view the API configurations.
API policies menu could not be verified due to this bug

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


Modify the WebLink to point to the matrix root if there are more than one device

Previously, if a matrix had multiple devices, the link would take you into the details of the first device.
Expected:

Actual:

Now, if there's a single device, it will take you directly to the execution. But if there are multiple devices, it will take you to the root matrix page which lists all the devices.
So, for ONE device, you'll still see the execution as before ( second screenshot above ). But for MULTIPLE devices, you'll see the matrix base screen ( first screenshot above ).

Codecov Report

Merging #380 into master will increase coverage by 0.05%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #380      +/-   ##
============================================
+ Coverage     79.83%   79.88%   +0.05%
  Complexity      476      476
============================================
  Files            66       66
  Lines          1542     1546       +4
  Branches        232      234       +2
============================================
+ Hits           1231     1235       +4
  Misses          173      173
  Partials        138      138

Great idea! Merged.


[joss] Default graph layout does not handle disconnected graphs well

The default layout used by the plotting function mpl_draw does not handle disconnected graphs very well. The distance between the components is very large, making it impossible to see the structure of the components themselves.
mpl_draw(rx.undirected_gnm_random_graph(20,15, seed=1234))

In mpl_draw we use spirng_layout with default values which isn't always ideal. With a little experimentation, you can get a nicer output:
graph = rx.undirected_gnm_random_graph(20, 15, seed=1234)
pos = rx.spring_layout(graph, k=1.0, seed=1234)
mpl_draw(graph, pos=pos)


Does this module support windows？

System information

OS Platform and Distribution:Win11

Modin version :0.13.2

Python version:3.8.8

Code we can use to reproduce:
import modin.pandas as pd
from modin.config import Engine
Engine.put("dask")
Engine.put("ray")
data = pd.read_csv(file_path, dtype=np.float, header=None).to_numpy("float32").astype('float32')

Describe the problem
Not work on Win11
Source code / logs
using dask:
UserWarning: Dask execution environment not yet initialized. Initializing...
To remove this warning, run the following python code before doing dataframe operations:

    from distributed import Client

    client = Client()

Traceback (most recent call last):
  File ".\test_read.py", line 45, in <module>
    print((test_modin_pd() == test_pd()).all())
  File ".\test_read.py", line 30, in test_modin_pd
    data = pd.read_csv(file_path, dtype=np.float, header=None).to_numpy("float32").astype('float32')
  File "C:\Users\wqh\anaconda3\lib\site-packages\modin\pandas\io.py", line 135, in read_csv
    return _read(**kwargs)
  File "C:\Users\wqh\anaconda3\lib\site-packages\modin\pandas\io.py", line 60, in _read
    pd_obj = FactoryDispatcher.read_csv(**kwargs)
  File "C:\Users\wqh\anaconda3\lib\site-packages\modin\core\execution\dispatching\factories\dispatcher.py", line 180, in read_csv
    return cls.__factory._read_csv(**kwargs)
  File "C:\Users\wqh\anaconda3\lib\site-packages\modin\core\execution\dispatching\factories\factories.py", line 207, in _read_csv
    return cls.io_cls.read_csv(**kwargs)
  File "C:\Users\wqh\anaconda3\lib\site-packages\modin\core\io\file_dispatcher.py", line 151, in read
    query_compiler = cls._read(*args, **kwargs)
  File "C:\Users\wqh\anaconda3\lib\site-packages\modin\core\io\text\text_file_dispatcher.py", line 1007, in _read
    splits = cls.partitioned_file(
  File "C:\Users\wqh\anaconda3\lib\site-packages\modin\core\io\text\text_file_dispatcher.py", line 314, in partitioned_file
    outside_quotes = cls.offset(
  File "C:\Users\wqh\anaconda3\lib\site-packages\modin\core\io\text\text_file_dispatcher.py", line 192, in offset
    outside_quotes, _ = cls._read_rows(
  File "C:\Users\wqh\anaconda3\lib\site-packages\modin\core\io\text\text_file_dispatcher.py", line 386, in _read_rows
    for line in iterator:
TypeError: 'LocalFileOpener' object is not iterable

using ray:
UserWarning: Ray execution environment not yet initialized. Initializing...
To remove this warning, run the following python code before doing dataframe operations:

    import ray
    ray.init()

Traceback (most recent call last):
  File ".\test_read.py", line 45, in <module>
    test_modin_pd()
  File ".\test_read.py", line 30, in test_modin_pd
    data = pd.read_csv(file_path, dtype=np.float, header=None).to_numpy("float32").astype('float32')
  File "C:\Users\wqh\anaconda3\lib\site-packages\modin\pandas\io.py", line 135, in read_csv
    return _read(**kwargs)
  File "C:\Users\wqh\anaconda3\lib\site-packages\modin\pandas\io.py", line 56, in _read
    Engine.subscribe(_update_engine)
  File "C:\Users\wqh\anaconda3\lib\site-packages\modin\config\pubsub.py", line 213, in subscribe
    callback(cls)
  File "C:\Users\wqh\anaconda3\lib\site-packages\modin\pandas\__init__.py", line 128, in _update_engine
    initialize_ray()
  File "C:\Users\wqh\anaconda3\lib\site-packages\modin\core\execution\ray\common\utils.py", line 183, in initialize_ray
    ray.init(**ray_init_kwargs)
  File "C:\Users\wqh\anaconda3\lib\site-packages\ray\_private\client_mode_hook.py", line 105, in wrapper
    return func(*args, **kwargs)
  File "C:\Users\wqh\anaconda3\lib\site-packages\ray\worker.py", line 918, in init
    _global_node = ray.node.Node(
  File "C:\Users\wqh\anaconda3\lib\site-packages\ray\node.py", line 215, in __init__
    self.start_head_processes()
  File "C:\Users\wqh\anaconda3\lib\site-packages\ray\node.py", line 911, in start_head_processes
    self.start_redis()
  File "C:\Users\wqh\anaconda3\lib\site-packages\ray\node.py", line 715, in start_redis
    self.get_resource_spec(),
  File "C:\Users\wqh\anaconda3\lib\site-packages\ray\node.py", line 344, in get_resource_spec
    self._resource_spec = ResourceSpec(
  File "C:\Users\wqh\anaconda3\lib\site-packages\ray\_private\resource_spec.py", line 149, in resolve
    num_gpus = _autodetect_num_gpus()
  File "C:\Users\wqh\anaconda3\lib\site-packages\ray\_private\resource_spec.py", line 241, in _autodetect_num_gpus
    lines = subprocess.check_output(cmdargs).splitlines()[1:]
  File "C:\Users\wqh\anaconda3\lib\subprocess.py", line 411, in check_output
    return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
  File "C:\Users\wqh\anaconda3\lib\subprocess.py", line 489, in run
    with Popen(*popenargs, **kwargs) as process:
  File "C:\Users\wqh\anaconda3\lib\subprocess.py", line 854, in __init__
    self._execute_child(args, executable, preexec_fn, close_fds,
  File "C:\Users\wqh\anaconda3\lib\subprocess.py", line 1307, in _execute_child
    hp, ht, pid, tid = _winapi.CreateProcess(executable, args,
FileNotFoundError: [WinError 2] can not find that file.

Hi @wqh17101 thanks for the report! Modin works on Windows.
What type of object did you pass as file_path? It seems like that might be causing the issue.

hi, file_path="./x.csv", the same code can work on Linux @devin-petersohn

@wqh17101 is this a test file you can share? I have tried to reproduce this, but I haven't been able to.

Sure,it is a normal csv
test_Y.csv

@wqh17101 can you share the full script? I wasn't able to reproduce the issue with your code and file. Are you doing os.chdir or something like that? That could be contributing here.

sure， it is a common script.
#!/usr/bin/env python
# _*_coding:utf-8_*_
"""
@Time   :  2022/3/5 22:20
@Author :  Qinghua Wang
@Email<EMAIL_ADDRESS>"""
import time

import pandas as pd

file_path = r"./test.csv"

def test_modin_pd():
    import modin.pandas as pd
    from modin.config import Engine

    Engine.put("ray")  # Modin will use Ray
    start = time.time()
    data = pd.read_csv(file_path, dtype=np.float, header=None).to_numpy("float32").astype('float32')
    print(data.shape)
    print(f"modin cost {time.time() - start}s")
    return data

if __name__ == '__main__':
    test_modin_pd()

oh , i check the func code where it raised exceptions.
C:\Users\wqh\anaconda3\Lib\site-packages\ray_private\resource_spec.py
def _autodetect_num_gpus():
    """Attempt to detect the number of GPUs on this machine.

    TODO(rkn): This currently assumes NVIDIA GPUs on Linux.
    TODO(mehrdadn): This currently does not work on macOS.
    TODO(mehrdadn): Use a better mechanism for Windows.

    Possibly useful: tensorflow.config.list_physical_devices()

    Returns:
        The number of GPUs if any were detected, otherwise 0.
    """
    result = 0
    if sys.platform.startswith("linux"):
        proc_gpus_path = "/proc/driver/nvidia/gpus"
        if os.path.isdir(proc_gpus_path):
            result = len(os.listdir(proc_gpus_path))
    elif sys.platform == "win32":
        props = "AdapterCompatibility"
        cmdargs = ["WMIC", "PATH", "Win32_VideoController", "GET", props]
        lines = subprocess.check_output(cmdargs).splitlines()[1:]
        result = len([x.rstrip() for x in lines if x.startswith(b"NVIDIA")])
    return result

It seems like it want to run some cmd, i run it in command line and get
WMIC is not a command in my system
so is there anything i need to install first?

When i comment that branch it triggered to continue,then i get
PS G:\naic_csi_semi_final\task2> python .\test_read.py
UserWarning: Ray execution environment not yet initialized. Initializing...
To remove this warning, run the following python code before doing dataframe operations:

    import ray
    ray.init()

Traceback (most recent call last):
  File ".\test_read.py", line 53, in <module>
    test_modin_pd()
  File ".\test_read.py", line 30, in test_modin_pd
    data = pd.read_csv(file_path, dtype=np.float, header=None).to_numpy("float32").astype('float32')
  File "C:\Users\wqh\anaconda3\lib\site-packages\modin\pandas\io.py", line 135, in read_csv
    return _read(**kwargs)
  File "C:\Users\wqh\anaconda3\lib\site-packages\modin\pandas\io.py", line 60, in _read
    pd_obj = FactoryDispatcher.read_csv(**kwargs)
  File "C:\Users\wqh\anaconda3\lib\site-packages\modin\core\execution\dispatching\factories\dispatcher.py", line 180, in read_csv
    return cls.__factory._read_csv(**kwargs)
  File "C:\Users\wqh\anaconda3\lib\site-packages\modin\core\execution\dispatching\factories\factories.py", line 207, in _read_csv
    return cls.io_cls.read_csv(**kwargs)
  File "C:\Users\wqh\anaconda3\lib\site-packages\modin\core\io\file_dispatcher.py", line 151, in read
    query_compiler = cls._read(*args, **kwargs)
  File "C:\Users\wqh\anaconda3\lib\site-packages\modin\core\io\text\text_file_dispatcher.py", line 1007, in _read
    splits = cls.partitioned_file(
  File "C:\Users\wqh\anaconda3\lib\site-packages\modin\core\io\text\text_file_dispatcher.py", line 314, in partitioned_file
    outside_quotes = cls.offset(
  File "C:\Users\wqh\anaconda3\lib\site-packages\modin\core\io\text\text_file_dispatcher.py", line 192, in offset
    outside_quotes, _ = cls._read_rows(
  File "C:\Users\wqh\anaconda3\lib\site-packages\modin\core\io\text\text_file_dispatcher.py", line 386, in _read_rows
    for line in iterator:
TypeError: 'LocalFileOpener' object is not iterable

Hi @wqh17101!
Can you reproduce TypeError: 'LocalFileOpener' object is not iterable problem with following reproducer?
import fsspec

file_path = r"./test.csv"
file = fsspec.open(file_path).open()

for line in file:
    print(line)

For reference: problem with Ray is reported here.

Thanks @wqh17101! This looks like a problem with Windows 11 support on fsspec side (which Modin use for reading files, just like pandas). Can you create an issue there?

@anmyachev
ffspec
v0.7.4 not work
v2022.5.0 work now
so you should add a minimum version of ffspec

@wqh17101 thanks for working on that! We should think about upgrading minimum version of fsspec thinking about compatibility with pandas.
@jbrockmendel doesn't the problem affect pandas too? Maybe you faced with this?

This doesn't look familiar, but I can confirm that pandas 1.5.0 (coming soon) bumps the fsspec minimum supported version to 2021.5.0

thanks @jbrockmendel!
@wqh17101 could you check fsspec v2021.5.0 with Windows 11 (I haven't the OS)? If this works, then a future release of Modin that supports pandas 1.5.0 will also work for you.

One piece is a bug in Ray, and another one is handled by https://github.com/modin-project/modin/issues/4855
I'm going to close this tracker in favour of other two which are more specific.


Unbranded CodeGen should ignore the generation of unsupported authentications

We should skip the generation of unsupported authentications like below:
Feel free to close this item if we didn't generate anything for them today.
@service({
  title: "Todo App",
})
@useAuth(BearerAuth | ApiKeyAuth<ApiKeyLocation.cookie, "session-id">)

@useAuth(NoAuth)

Please move this to the TypeSpec repository and rescope.   We should not be tracking work here.

Close it as I recreated the issue in https://github.com/microsoft/typespec/issues/5355


chore(main): release 1.1.0

:robot: I have created a release beep boop
1.1.0 (2024-11-29)
Features

version-debt: add totalDependencies to version debt reporting (3334ef9)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.1.0
:sunflower:


chore: move dag validation to packet handler

Purpose
fixes: #1323
How does the solution address the problem

Changes made

prnet cleaned-up successfully


Wrong Chapter Referenced for Installing VS Code and Extensions

In Chapter 2, section 2.1.3 of COBOL Programming with VSCode.md, it says:

To see a more detailed instruction on installing this extension, visit Chapter 5 "Install VSCode and Extensions".

However, Chapter 5 is Data Division according to the Table of Contents.  Chapter 3 is titled Installation of VSCode and extensions according to the Table of Contents.

Resolved in #77


backtracking array BinarySearch

Please fill all the details and checkbox, so we can merge PR faster.
Pull Request Template
What have you Changed(must)
Write here what you have changed in codebase
Issue no.(must) -
Pr will be closed and marked as spam. If issue number not found or issue was assigned to someone else.
Marking as spam can block your account from HacktoberFest.
Self Check(Tick After Making pull Request)

[ ] This issue was assigned to me if not it will marked as spam.
[ ] My file is in proper folder
[ ] I am following clean code and Documentation.My code is well linted and I am using flake8 standard for python
[ ] I have added title and what program will do first in the file

acknowledgment

[x] I have tested the script and distributed it under an MIT license.

README - How to Contribute

@aditi10701, It's good practice to create an issue before creating a PR.


Grid, Tab, Button, and some other elements require attributes that are not required by docs

Not sure why and when it started but some components started to require optional component attribute to be set
The documentation says it's not required and default "div" works perfectly fine.
https://material-ui.com/api/grid/
https://material-ui.com/api/tabs/

Tab requires href. I think it's coming from Button/ButtonBase

[x] This is not a v0.x issue.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Tech
Version

Material-UI
v4.3.0

React
"^16.8.6"

Browser
chrome

TypeScript
no

IDE
newest webstorm

OS
macos 10.14.6

half of the code is with a warning because of these issues :/

Is this a duplicate of https://github.com/mui-org/material-ui/issues/16652 ? Looks like a bug has been raised on JetBrain’s side.

Closing this since this is a bug with webstorm/your IDE and a duplicate of #16647 or #17035.


Changes needed for "Automatic" splitting in CRAB

These are changes needed for "Automatic" splitting in CRAB. In detail:

Allow to specify the runtime limitation parameter maxSecondsUntilRampdown in the pset (CMSSW_7_4_0 and later)

this allows us to run a splitting "probe" on the schedd that will return after a specific amount of time
allows to cap the runtime of regular jobs to have more of a "task completion time" guarantee (ASO excluded)

Adjust the dashboard API to accept non-integer job IDs

the "capped" regular jobs have completion jobs with IDs of the form 1-1, 1-2,… to complete unprocessed lumis from job 1 etc…

Allow to set a hard job limit for splitting, raising an exception when the total number of jobs exceeds this limit

the splitting probe runs the splitting algorithm in the the postjob on the schedd. When splitting parameters turn out to be unreasonable, creating more jobs than we ask for will bog down the schedd. Returning with an error state before this happens prevents the postjob from consuming excessive amounts of memory.

Hi @matz-e , in case you still want to have this merged into WMCore, can you please rebase and squash the commits? Thanks

@amaltaro did that…

@ticoann I think we can merge this one. I didn't test it, but code looks Ok and should be transparent for production. (branch PR was merged already)


Collect potential image that can be useful in website (licenced under CC0 or CC BY-SA)

Acceptance criteria

[ ] I have at least two images that can be used in the design.

Tasks

[ ] Brainstorm the tentative types of images, which will fit as per design perspective.
[ ] Collect good open licensed images that represent women and data.

@nikeshbalami  we still need an image on the home page. ( I guess images from the last conference will be good.)

Some of the selected images from last year conference and side events have been collected here: https://drive.google.com/drive/folders/17izagPd8QfQtf19vRI_OIQeRJd4tLOQH?usp=sharing

closing as DONE.


Test generate app

Fixes #68

Codecov Report

Merging #69 (6a9df20) into main (bd83c1d) will increase coverage by 10.94%.
The diff coverage is 83.33%.

@@             Coverage Diff             @@
##             main      #69       +/-   ##
===========================================
+ Coverage   69.77%   80.71%   +10.94%
===========================================
  Files          24       24
  Lines         526      529        +3
===========================================
+ Hits          367      427       +60
+ Misses        159      102       -57

Impacted Files
Coverage Δ

fooddata_vegattributes/app/generate.py
96.55% <ø> (+96.55%)
:arrow_up:

fooddata_vegattributes/utils/random.py
89.47% <83.33%> (+8.22%)
:arrow_up:

fooddata_vegattributes/food.py
100.00% <0.00%> (+5.55%)
:arrow_up:

fooddata_vegattributes/utils/general.py
100.00% <0.00%> (+100.00%)
:arrow_up:

fooddata_vegattributes/utils/terminal_ui.py
100.00% <0.00%> (+100.00%)
:arrow_up:

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 bd83c1d...6a9df20. Read the comment docs.


Payload Settings Language No other language found

Link to reproduction
No response
Payload Version
beta
Node Version
v20.11.1
Next.js Version
15.0.0-rc.0
Describe the Bug
No other languages shown
export default buildConfig({
  admin: {
    user: Users.slug
  },
  collections: [Users, Media],
  editor: lexicalEditor(),
  secret: process.env.PAYLOAD_SECRET || "",
  typescript: {
    outputFile: path.resolve(dirname, "payload-types.ts")
  },
  db: mongooseAdapter({
    url: process.env.DATABASE_URI || ""
  }),
  sharp,
  localization: {
    locales: ["en", "zh-TW"],
    defaultLocale: "en",
    fallback: true
  },
  // 繁體中文
  i18n: {
    fallbackLanguage: "zh-TW"
  },
  plugins: [
    // storage-adapter-placeholder
  ]
})

https://github.com/payloadcms/payload/assets/58339015/5beab146-e1fb-45bc-aa2c-bbdc919f14db
Reproduction Steps
Enter http://localhost:3000/admin/account
Adapters and Plugins
No response

There is also an error here

In some places, the type
zh,zh-Hant
zh,zh-TW
zh-CN,zh-TW

Hey @sao-coding you should actually not be able to clear the language selection in the account view. I have a PR out that will fix this.
As for adding more languages you will need to import them from payload/i18n like so:
import { de } from 'payload/i18n/de'
import { en } from 'payload/i18n/en'
import { es } from 'payload/i18n/es'

and then define them in the config like so:
i18n: {
  supportedLanguages: {
    de,
    en,
    es,
  },
}

See example in our test suite: https://github.com/payloadcms/payload/blob/e7159c033e5b80dcbdc3f3f2f56b7b5e30d8dc2b/test/buildConfigWithDefaults.ts#L172-L177


Put Back Ability to Send in Store as a Prop to Connected Component in (v6 restricts this now)

This nonsense calling the ability to pass in store via as legacy and forcing people down one path.
I use shallow when I TDD connected components. Sounds like we can't still send in store as a prop from a shallow test to a connected component.  That's bad because I do not want isolated tests all be broken now that this no longer works and furthermore, I rely on my style of testing to test this way.  There's no reason that should be prohibited if you don't use context API.  And again contextAPI forces me to use mount with enzyme which forces me to use provider.  We can't use shallow anymore as things with React and Redux progress this way.
This directly relates to testing because to test a connected component with Context API, you have to mount() and reliant in your tests on using a provider.  With enzyme, I do not want to use mount or provider in my tests, just shallow and simply passing in a dummy store.  My tests don't test re-renders or stuff that's integration testing like that. I don't rely on integration tests so the way I test is via TDD with shallow.  Preventing passing in a store as a prop is a big problem for devs doing so and I know others who also TDD this way and drive their design and inject stores this way through isolated or smaller component tests.
We don't all view context API as a good thing. Not being able to keep my tests simple by passing in the store is ridiculous which is a major reason I don't have provider in my tests to begin with. Simplicity! To be forced down this past is a very bad play on the React and Redux team. My tests should not be coupled to the implementation this much forcing one way to do it all.
Keep the simplicity. Stop trying to add more and more magic.

https://blog.isquaredsoftware.com/2018/11/react-redux-history-implementation/#implementation-notes-1

Please don't be mean to other users.

Note: There was an additional comment after @btnwtn's, but it will no longer be visible as Dave has been blocked from this project. Sorry you had to deal with a rude user, Brandon!


MustacheJS rendering nested(>2 level) JSON using partials

This is not an issue. Its a question basically.
I am trying to render my nested(can be multiple level) JSON using Mustache partials. It renders only till second level it does not third & greater. As per definition partials can be used to render recursively.
Am I doing it wrong ? or is there any other way to achieve the same using mustache ?

Data:

var data = {
    product: [{
        productName: "Category1",
        product: [{
            productName: "Windows"
        }, {
            productName: "Windows Mobile"
        }]
    }, {
        productName: "Category2",
        product: [{
            productName: "SubCategory 1",
            product: [{
                productName: "Nexus 4"
            }, {
                productName: "Nexus 6"
            }]
        }, {
            productName: "SubCategory 2",
            product: [{
                productName: "ipod"
            }, {
                productName: "iphone"
            }]
        }]
    }]
};

$('body').html( Mustache.render( productList.html(), data, {recurse :recursiveList.html()}) );

Output (missing products: Nexus 4, Nexus 5, ipod, iphone)

Category1
Windows
Windows Mobile
Category2
SubCategory 1
SubCategory 2

Your "recursive list" template doesn't actually have a recursive call, so it can only render one level deep. Since it's identical to the product list, with the exception of the missing recursion call, just use the product list:
$('body').html(Mustache.render(productList.html(), data, {recurse: productList.html()}));

If you do that, though, you'll get a "maximum call stack size exceeded", which you probably wouldn't expect. It's because, by structuring your data and templates the way you did, you have created infinite recursion. In the tail products (like Nexus 4, Nexus 6, iPod, etc) the template will look for a "product" property and not find it. So it will walk back up the context stack, and check each level until it finds something with a "product" property. You can stop this by making sure the leaf products have a "product" property with a value like false, null or [].
You can find more on how mustache resolves variables here: https://github.com/bobthecow/mustache.php/wiki/Variable-Resolution

:clap: Awesome Thanks a lot @bobthecow. It worked!


v1.6 backports 2020-06-30

#12328 -- Fix bug where identity allocation wouldn't cancel from api timeouts (@joestringer)
#12343 -- envoy: Update to 1.13.3 (@jrajahalme)

Once this PR is merged, you can update the PR labels via:
$ for pr in 12328 12343; do contrib/backporting/set-labels.py $pr done 1.6; done

test-backport-1.6

Hit known flake: https://github.com/cilium/cilium/issues/10446
Otherwise, this should be good to go.


Close bash command quotation mark

Remove the empty spaces.
Close the bash command with missing quote mark (can see in syntax highlighting on the doc page it isn't closed too)

Great catch @tommeier! Thanks for taking the time to PR 😊🙏🏻


Reads name  invalid error

Hi,
Amazing CELL article and this software! I'm trying to add the sequence information of insertions to the vcf file. my samples were sequenced using ONT and aligned using NGLMR. when I run the Iris, it told me that my reads' name was invalid.  Below is my .bam file looks like:

What should I do to fix this problem? Thanks for your help!

Hi,
Thanks for your interest in the software! Those alignment entries look fine, so an invalid read names error is likely referring to the VCF file.  What did you use to generate your SV calls?  Iris requires an RNAMES INFO field which indicates a list of supporting reads for each variant (I just updated the README file to indicate this).  If you are using Sniffles for variant calling, this field is enabled with the parameter "-n -1".  If you already have this field present in all of your variants and this is not the issue, could you please provide an example of the error message, and if you are able to share it, an example of an insertion-type variant call which you are passing to Iris?
Thanks!
Melanie

Hi,
Glad to tell you that adding flag '-n -1' of Sniffles worked for me. Thanks a lot!
Shangzhe

Hi Shangzhe,
I'm glad to hear that!  I'll go ahead and close this Github issue, but please don't hesitate to reach out if you have other problems!
Best,
Melanie


Add custom gtm events to onboarding

What is included in this request?

[ ] Bug Fix
[ ] New feature
[X] Changes to an existing feature
[ ] New extension
[ ] Changes to an existing extension

Describe your changes
(If this is a new extension or feature, please describe in detail what the extension or feature does)
Adding custom GTM events to symphony onboarding
Are you adding any dependencies or calling external APIs?

[X] No
[ ] Yes (Please list the dependencies or APIs below and how they're being used)

:tada: This PR is included in version 1.24.0-dev.6 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.24.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Error on second withdrawal

Hi,
i withdrawed some Beer to my own address to get utxos.
First i withdrawed 1 Beer and after that I tried to withdraw another one and the second one failed with this error
Unhandled Promise Rejection
Error: Couldn't create withdrawal transaction:
{ txid: '0000000000000000000000000000000000000000000000000000000000000000',
txfee: 1000,
complete: false }
at errorWithObject (app://-/bundle.js:90100:46)
at app://-/bundle.js:90357:11
at Generator.next ()
at step (app://-/bundle.js:90089:191)
at app://-/bundle.js:90089:361
at

HyperDEX 0.1.0-alpha.1
Electron 2.0.0
win32 10.0.16299
Locale: de

Closing this as it's a duplicate of https://github.com/lukechilds/hyperdex/issues/201


Missing step does not exit with non-zero CLI code

CLI ouput like:
Feature: Checkbox
  Scenario:                                    # tests-behat/checkbox.feature:3
    Given I am on "form-control/checkbox.php"  # Behat\MinkExtension\Context\MinkContext::visit()
    ...
    Then Toast display should contain text '...'

1 scenario (1 undefined)
8 steps (2 passed, 2 undefined, 4 skipped)
0m1.86s (12.32Mb)

 >> <snippet_undefined><snippet_keyword>main</snippet_keyword> suite has undefined steps.
Please choose the context to generate snippets:</snippet_undefined>

  [0] None
  [1] Behat\MinkExtension\Context\MinkContext
  [2] Atk4\Ui\Behat\Context

--- Behat\MinkExtension\Context\MinkContext has missing steps. Define them with these snippets:

    /**
     * @Then Toast display should contain text :arg4
     */
    public function toastDisplayShouldContainText($arg1, $arg2, $arg3, $arg4): void
    {
        throw new PendingException();
    }

silently exists with zero CLI exit code making it very hard to fail CI.

This is already partially possible in Behat. The --strict option will consider that skipped or pending scenarios should make the run use a failure exit code.
Maybe we need a third way of interpreting results which would allow skipped scenarios but reject pending ones.

Thank you very much for meantioning the --strict option - it does exactly what I want.
I personally would make it default, as CI should fail in case of undefined steps.


fix: ci build

I downgraded flutter_inappwebview to version 5.8.0 as it prevented the build pipeline from completing successfully.
This PR also replaces #34
Please upgrade your build pipeline to use Flutter 3.13.x as it is the recent version.

:tada: This PR is included in version 1.4.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Uplift requests

This is moves the API requests into the Card component and makes
the button a pure UI component with no side effects.
Still WIP 🏗

refactor(ButtonWater): W.I.P. Moves requests up
refactor(BtnWaterGroup): Is deprecated
refactor(Types): Add User type from auth0-spa-js
reafactor(ButtonRound): Rename toggle to onClick

Looking at this code getting 🧼🧽🧹 makes me horny 🤤

Great improvements here! Thanks a lot! Merging this branch is going to be a pain though. We both did consequent changes in the same files. Maybe a pair programming session is necessary. 🤓🤓

Great improvements here! Thanks a lot! Merging this branch is going to be a pain though. We both did consequent changes in the same files. Maybe a pair programming session is necessary. 🤓🤓

Just push your changes and I will try to sort out the confilcts

Just push your changes and I will try to sort out the confilcts

You wish it was that easy :P My changes are pushed but there is still work to do here and there and bugs might be remaining.


uu_more fails to build on Windows

Setup
Compiling on Windows 11 using rust nightly:
rustc 1.59.0-nightly (c5ecc1570 2021-12-15)

Error
error[E0069]: `return;` in a function whose return type is not `()`
   --> src\uu\more\src/more.rs:266:25
    |
266 |                         return;
    |                         ^^^^^^ return type is not `()`

For more information about this error, try `rustc --explain E0069`.
error: could not compile `uu_more` due to previous error

Fixed here
https://github.com/uutils/coreutils/pull/2772


Fix <* scaladoc

Codecov Report

Merging #112 (d9843ca) into main (454127f) will increase coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #112      +/-   ##
==========================================
+ Coverage   94.45%   94.59%   +0.13%
==========================================
  Files           6        6
  Lines         758      758
  Branches       77       77
==========================================
+ Hits          716      717       +1
+ Misses         42       41       -1

Impacted Files
Coverage Δ

core/shared/src/main/scala/cats/parse/Parser.scala
95.05% <ø> (+0.15%)
:arrow_up:

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 454127f...d9843ca. Read the comment docs.

Thanks for the fix!


Visual Design for basic table content types

😯 Problem to Solve
Provide designs for basic content types outside of what was already completed from the initial design

#883

In table cells

Numeric values (Related dev task: #1011)
Hyperlinks (Related dev task: #1012)
Icons (Related dev task: #1013 )
DateTime (Related dev task: #1014)

In table header cells

Icon
Text and Icon

💁 Proposed Solution
🤔 Open Questions

What size should the icons be?
Do we need default alignments for different data types?

🖼 References/Prior Art
Design for basic table

#883

@RickA-NI here are a couple prototypes of right-aligned numeric columns. They reveal some interesting questions for you to provide guidance on:

For numeric columns, should we right align the values, both the values and the header, or neither?
If we right align the values, should we also set font-variant-numeric: tabular-nums to request that the browser render digits with equal widths? Does our font support this setting? (I couldn't see a difference when I played with it)

My initial reaction is that headers should match their data. I'm not sure what the ramifications are for all the stuff (sorting direction, menu icons, grab handles, etc.) we've talked about stick up there but visually it looks better and keeps the header from becoming dissociated from the data as in your example.
Source Sans Pro renders digits with equal widths by default. You shouldn't need to turn that on unless we're trying to force it into a monospaced like mode where the decimal places and commas take up equal space to a didgit.

I've always thought....strings to the left, numbers to the right...but it is challenging to visually connect data when you don't have borders in the headers to group columns, like we've got.

I found some blog on table design and it had three (and a half) rules for alignment that seem to jive pretty well with what we're all aligning on here:

Ahhhh...the good ole "some blog" :) Just kidding, the No.3 is new to me but does appear to be a pretty standard practice and something I've, admittedly, not thought about too much. Learn something new everyday. 🌈⭐
No. 3 1/2 is interesting as their could be some cases (not necessarily our use cases) where centering the value along a separator might be preferable like the resolution and ratio columns in the image below. Not sure if modern CSS supports this well yet and probably isn't a priority for us right now, but you can learn more at: https://ux.stackexchange.com/questions/24066/what-is-the-best-practice-for-data-table-cell-content-alignment

Here's a link to another some blog that reiterates 1-3 of the simple rules to follow...with a fun little animation to show the impact. https://www.darkhorseanalytics.com/blog/clear-off-the-table/

how is that content aligned in the resolution and aspect ratio columns? I get the intent to align by the midpoint symbol (x or :) but the data in those columns is not left right or center aligned it's like some bespoke hand layout stuff. If you couldn't do a custom layout and had to pick one then I'd think matching the header to the one pick is still the right choice.

@RickA-NI, just for clarity, the aspect ratio column is using an existing CSS mechanism to provide that alignment behavior. Basically, CSS would look like this:
.resolution {
    text-align: 'x' center;
}

.aspect-ratio {
    text-align: '.' center;
}

aspect ratio column is using an existing CSS mechanism

Support for this particular CSS feature doesn't look so great. Looks like we still need vendor prefixes and a non-vendor prefix version might be deprecated/obsolete.
https://caniuse.com/mdn-css_properties_text-align_block_alignment_values
Support as of 02/02/2023

@jattasNI @nate-ni @leslieab I think I've got these recommendations ready to go. I just want to run them by everyone here before I add examples to the visual spec Storybook is pulling in.
These three rules I think are widely accepted norms and good guidelines for all column types:

Numeric data is right-aligned
Textual data is left-aligned
Headers are aligned with their data

For Numeric value columns I think the above guidelines and Jesse research has pretty much laid out what we do here. We right-align and use tabular numbers. The font is standard body copy. Source Sans Pro has tabular numbers by default so we get that for free.
For Hyperlinks columns we use our standard anchor link control. We treat them as textual data and left-align.
For Icon columns we can have icons with text or without. Either way we treat them as textual data and left-align. Text when needed is standard body copy and offset from icon using our standard sizing token.
DateTime columns are mostly numeric data but seem to be usually treated like textual data and left-aligned. Again we use our standard body copy font. There is a standard format set by ISO 8601 and suggested for use by W3C on how to format this data. We should look into adopting it at least as a default. Here is a summary:

For a more full breakdown on this standard you can see this page here written by the author.
https://www.cl.cam.ac.uk/~mgk25/iso-time.html

The only one that we may have to think more about is #3..."Headers are aligned with their data" specifically for right-aligned content (numbers). All other direction seems pretty solid to me. Since we intend to have controls and icons (for sorting) in the header, we may need to explore how that impacts that particular guidance.
As an example, GH is in a similar situation and has maintained left-alignment in their headers for numeric, right-aligned, columns. While this doesn't dictate our solution, it is a reference to see the impact of that solution.

I know we don't have final designs for those fully featured headers yet but the initial prototype may give us something to work from.

I agree, I think right-aligning headers is going to be problematic. And due to sort/grouping iconography and the menu button, the text will never vertically right align with row cell content anyway.

@leslieab just brought up the points I was about to. So...I have nothing. Gosh leslie!

Here are my column alignment mockups. I think this covers all of the relavent scenarios. Let me know if you like to see something else.

Thanks for posting these, @RickA-NI!! It is really helpful.
If our tables were static (non-interactive) displays of data...or maybe just didn't have multiple elements in the header; I think we would align the header content with the column content...but they aren't. I think we should start with keeping the header content left-aligned.
This might just be a place to start, I think this is a decision we can change relatively easy and with low impact.
Having all the controls in the same order across headers will also likely make things a little less complicated when we get to supporting keyboards and screen readers.

One more case to consider: would numeric data with units still be right aligned? We lose some benefit of right aligning if the units have inconsistent widths. e.g.:
Voltage
    101.0V
     1.3mV
    25.7kV

Here are my column alignment mockups. I think this covers all of the relavent scenarios. Let me know if you like to see something else.

Sounds like @nate-ni's vote is to go with "Numeric headers left aligned, values right-aligned" to begin with. I'm ok with this as our initial approach but want to note that, to me, it feels a bit harder to associate the header with the data than other examples in this thread because of our lack of vertical lines separating the columns. While I recognize it's not my area of expertise or decision to make, is there anything we could do visually to increase the association between the column and its header in that case? e.g. Subtle background colors? Always showing the header dividers? @RickA-NI

We decided in a team meeting to start with  "Numeric headers left aligned, values right-aligned". The concern above will likely be moot if users size their columns to match their data; there won't be large amouts of whitespace which cause headers to be far from their content. We can always revisit this if it turns out to be a faulty assumption.

Note from Rick in response to a question from Molly in a team meeting: if the content is mixed data type (e.g. an SLE tag column with strings and numbers) then it should be left aligned.

@jattasNI I know that this has been closed but I did come across an pretty indepth breakdown of how to format a table for scientific and technical publication. This is for print so there are some things we might do differently, but this section seems pertinent to your last question:


Tracking token usage?

I see that the API supports .message_token_len() for an individual ChatMessage; it would be nice to be able query total token usage over the course of a conversation for cost tracking purposes.
I'm not entirely sure the best way to handle it - maybe like a .next_message_tokens_cost(message: ChatMessage) that would return the total prompt tokens (system + function defs + chat history) plus the tokens in message that would be incurred? If it could be done over the course of a chat (accumulating after each full round) maybe something like .conversation_history_total_prompt_tokens() and .conversation_history_total_response_tokens() so a user could compute a running chat cost?
Thanks for considering, and for developing Kani! It really is the 'right' API interface to tool-enabled LLMs in my opinion :)

Thanks for the kind words! I should note that often times the internals of the LLM providers (in particular OpenAI) are a bit of a mystery, so Kani's token counting is really just a best guess to within a couple of percent.
You have a couple options if you want to track tokens as accurately as possible, which I'll lay out here:

Overriding Kani.get_model_completion - this is the method that the Kani instance uses to go the underlying LLM, and it returns a Completion, which includes the prompt token len and completion token len as returned by the engine. You could, for example, add tokens_used_prompt and tokens_used_completion attributes in a subclass of Kani and increment those after a super call; this has the disadvantage of being post-hoc counting though. I use a similar approach in one of my projects here: https://github.com/zhudotexe/kanpai/blob/cc603705d353e4e9b9aa3cf9fbb12e3a46652c55/kanpai/base_kani.py#L48

You could also use an estimation like sum(self.message_token_len(m) for m in await self.get_prompt()) + self.engine.token_reserve + self.engine.function_token_reserve(list(self.functions.values())) if you wanted a token estimation before sending it to the LLM. The instance caches the token lengths so this won't result in a major slowdown.

Use an external gateway - in our lab we've been trying out Helicone for token counting. If you're using OpenAI, you can integrate it with Kani pretty easily by specifying the api_base and headers when constructing an OpenAIEngine. I've also been interested in Cloudflare AI Gateway, though I haven't used it yet. These solutions also require a bit more engineering though, and I believe they're also only post-hoc.

I'll have to think a bit more about how to implement an official token counting interface if we decide to - maybe Kani.prompt_len_estimate(msgs: list[ChatMessage]) -> int to perform the estimation detailed above?

Wonderful, thank you! Post-hoc is fine for my case, I used your first suggestion and it works great. I did need to remember to update the counts manually when calling out to sub-kanis. (Maybe engine-level counting?)

Hello again :) I've started playing with the newer streaming functionality, and it's been nice! However, it looks like get_model_completion isn't called when streaming, so the override described above isn't processing token counts. Perhaps the new get_model_stream could be overridden similarly? I suppose I could increment the completion token count as they are yielded, but I'm not sure how to get the prompt token count accurately.
Thanks as always-

Good call out - this is a little less elegant with streaming. The get_model_stream method is a little lower-level (returning a mixed iterator which is managed by a StreamManager) with no guarantee that a Completion will be yielded, or that each yield is exactly one token.
In the current version (1.0.1), your best option is probably to look at await stream.completion(), which should have the prompt_tokens and completion_tokens attributes set, with some caveats:

for API models, these attributes will only be set if the underlying API returns them in the stream (which was not the case for OpenAI models until recently, I'll patch this in)
this is a different interface than for the non-streaming case, which is kind of ugly

example:
stream = ai.chat_round_stream("What is the airspeed velocity of an unladen swallow?")
async for token in stream:
    print(token, end="")
completion = await stream.completion()
prompt_tokens = completion.prompt_tokens
completion_tokens = completion.completion_tokens
# ...
# msg = await stream.message()

In v1.0.0 I added a private _add_completion_to_history method that acts like add_to_history, but is only called on model completions with a Completion rather than on each message. I'll make this method public since it's called on both stream and non-stream completions and token counting is a good use case for it.
I'll update this thread with new code snippets (probably later today?) once that's done.

As of v1.0.2, Kani.add_completion_to_history is called after each completion for both streaming and non-streaming generations. You can implement token counting by overriding it like so:
class TokenCountingKani(Kani):
	# ...

    async def add_completion_to_history(self, completion):
        prompt_tokens = completion.prompt_tokens
		completion_tokens = completion.completion_tokens
		# ...
        return await super().add_completion_to_history(completion)

Note that completion.[prompt|completion]_tokens might be None for user-implemented engines and llama.cpp streams (https://github.com/abetlen/llama-cpp-python/issues/1498), but it should be present for all kani-bundled OpenAI, Anthropic, and Hugging Face engines.

Wonderful, I will give it a try, thank you!


python311Packages.hatasmota: 0.7.1 -> 0.7.2

Diff: https://github.com/emontnemery/hatasmota/compare/refs/tags/0.7.1...0.7.2
Changelog: https://github.com/emontnemery/hatasmota/releases/tag/0.7.2
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 254608 run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.tasmota
python310Packages.hatasmota
python310Packages.hatasmota.dist
python311Packages.hatasmota
python311Packages.hatasmota.dist

Result of nixpkgs-review pr 254608 run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.tasmota
python310Packages.hatasmota
python310Packages.hatasmota.dist
python311Packages.hatasmota
python311Packages.hatasmota.dist


Meeting1 tweaks

Meeting changes from discussion w NGUX, 2023-01-10

Replaced by https://github.com/scitools-classroom/iris-mesh-tutorial/pull/3


Query/Question

Query/Question
A clear and concise ask/query.
Why is this not a Bug or a feature Request?
A clear explanation of why is this not a bug or a feature request?
Setup (please complete the following information if applicable):

OS: [e.g. iOS]
IDE : [e.g. IntelliJ]
Version of the Library used

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Query Added
[x] Setup information Added

Originally posted by @myrp26 in https://github.com/Azure/azure-sdk-for-ios/issues/519

Closing since this issue is blank.


Remove references to ORM

Thank you @malukenho I wanted to do this for quite some time but kept forgetting :D


HaveIbeenPwned API breaks the plugin

This great plugin doesn't seem to be working anymore when the API needs to be used. Does anyone recommend any solution to this issue?

If you mean the API changes on HIBP to Version 3 you can try to download the plgx from my pull request #83 , especially if you already have a valid api key, as I was unable to test with a working api Key.
I think you can find further details in #80 as it started to discuss the issue before the changes went live.

Fixed in #83


Update Build Configuration if any of post-clone, pre-build, or post-build scripts gets added or removed.

Describe the solution you'd like
It is a known issue (https://docs.microsoft.com/en-us/appcenter/build/troubleshooting/general#when-do-i-have-to-update-my-build-configuration-manually-through-save--build) that build configurations do not update when custom build-phase scripts are added or removed.  This is particularly problematic for those of us leveraging your service for CI/CD.  I would like to not have to use the web portal any time a build-phase script is added or removed.  It should just happen automatically.
Describe alternatives you've considered
Another option would be to provide in the CLI or API some mechanism for manually triggering a refresh of the build script detection.
Additional context
I added a post-clone script to my repo and triggered a CLI build.  It failed because it didn't run the script.  I went to the portal, opened the config, saved it, and built again.  The build ran the post-clone script as expected.  I only knew that this was necessary after some 2 hours of debugging and ultimately turning to your troubleshooting docs in desperation.  I was one step away from an email with our AppCenter contact. :)
Let's see some of that customer obsession in action! 💯

Thanks for the feature request here @MichaelMackendMS. Related to #265, we can make it clearer that you need to save the build configuration for scripts to be triggered. We'll take a deeper look into improving this experience.


XML parsing issue with Entity Reference

Summary
There is an issue on function: https://github.com/LadybirdBrowser/ladybird/blob/c04297129323904f15c743eb83a3ff934769b14d/Userland/Libraries/LibXML/Parser/Parser.cpp#L754
I was able to figure out it is related to the parsing of Entity Reference. There may be an issue in there. Additionally, there is an Entity Reference grammar rule that seem not be implemented, namely [69] here https://www.w3.org/TR/2006/REC-xml11-20060816/#NT-Reference:
PEReference | ::= | '%' Name ';'
It's similar to the EntityRef one, so maybe easy to implement.
Maybe that's the cause of the issue.
Side note, just wondering, is it worth to develop our own XML parsing lib? Maybe it's better to use an existing, well-mantained FOSS one?
Operating system
Linux
Steps to reproduce

Go to https://yewtu.be/watch?v=5OYawYN9ZCs (or any Invidious video page really)
There seem to be an issue when parsing XML related to DASH. See here:

636.040 WebContent(5257): Unhandled JavaScript exception: [TypeError] Cannot access property "setVolume" on undefined object
636.050 WebContent(5257):     at set
    at https://yewtu.be/videojs/video.js/video.js?v=d8b893e:26371:21
    at <unknown>
    at https://yewtu.be/videojs/video.js/video.js?v=d8b893e:4412:20
    at <unknown>
    at https://yewtu.be/videojs/video.js/video.js?v=d8b893e:4411:29
    at https://yewtu.be/videojs/video.js/video.js?v=d8b893e:5078:11
    at <unknown>

:1: parser error : EntityRef: expecting ';'
ndwidth="130429" codecs="mp4a.40.2"><BaseURL>/videoplayback?expire=1730611585&ei
                                                                               ^
:1: parser error : EntityRef: expecting ';'
4a.40.2"><BaseURL>/videoplayback?expire=1730611585&ei=IbUmZ-HvFaaG6dsPuabvmAs&ip
                                                                               ^
:1: parser error : EntityRef: expecting ';'
mZ-HvFaaG6dsPuabvmAs&ip=2603%3Ac020%3A8002%3Aaa00%3A2711%3A7e46%3Aba4f%3Ac4a7&id
                                                                               ^
:1: parser error : EntityRef: expecting ';'
2711%3A7e46%3Aba4f%3Ac4a7&id=o-ACsSloeXB0SKqSw1-sxjdPdph5NzLAvhjxSxHfRjsjc0&itag
                                                                               ^
:1: parser error : EntityRef: expecting ';'
%3Aba4f%3Ac4a7&id=o-ACsSloeXB0SKqSw1-sxjdPdph5NzLAvhjxSxHfRjsjc0&itag=140&source
                                                                               ^
:1: parser error : EntityRef: expecting ';'

...
[dash @ 0x563d6ef169c0] Unable to parse '' - missing root node

Expected behavior
The thing should parse correctly and without error
Actual behavior
Parsing error
URL for a reduced test case
Not vailable
HTML/SVG/etc. source for a reduced test case
Couldn't extract a minimal case but the Invidious site still provide plenty

Log output and (if possible) backtrace
See above

Screenshots or screen recordings
No response
Build flags or config settings
No response
Contribute a patch?

[ ] I’ll contribute a patch for this myself.

+1 to adopting a mature OSS library for XML parsing


MINREV will incorrectly handle intermediate releases

Revision such as x.y.z-rc1 will lead to an error.

Opened by mistake, MINREV works perfectly for this case!

Second thought, the behavior should be thought about, i.e. x.y.z-rcQ is not yet x.y.z, but x.y.z-XX-YYYY is after x.y.z. What should we do?

For now, consider x.y.z-qqq is >= x.y.z


Support for data from the R10 flowcells

Hi,
As the R9 flowcells are soon be discontinued by ONT, I was wondering would there be any support for the R10 flowcells in the future for NanoRepeat?
Many thanks!

it supports sequence data generated from R10 flowcell.
On Thu, May 2, 2024 at 4:21 AM Y-Pei @.***> wrote:

Hi,
As the R9 flowcells are soon be discontinued by ONT, I was wondering would
there be any support for the R10 flowcells in the future for NanoRepeat?
Many thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/WGLab/NanoRepeat/issues/12, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABNG3OEVPD57VISUPINZO6LZAHZQXAVCNFSM6AAAAABHDH6EXWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGI3TIOJQGQYDENQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

The description of the -d doesn't seem to have an option for R10 though?

Please use -d ont_q20 if your read accuracy is Q20+ or above.


Add option to remove all whitespace surrounding binary operators

First of all, thank you for all the work you've put into making this a great piece of software!
Right now there is no way to have the CS fixer remove spaces surrounding binary operators. (1 + 2 -> 1+2).
Actually, I'm only looking to get rid of them for multiple exception catches. It's part of the style guide of the project I'm currently working on that there should be no spaces between exception type declarations, i.e.
<?php

try {
} catch (FooException|BarException $e) {
}

Is this something you'd accept? I'd have time to work on it on Friday, so if you give me the go signal I'll get it done by then.

Hi @ocrampete16 ,
Thanks for the kind words, means a lot to hear someone enjoys the result of this great community!  :)
Currently it is indeed impossible to configure the fixer to remove any space around binary operators. Before the multiple catch was introduced I didn't see a need for it, but now you brought it forward I kind of do. I would even like the fixer to remove spacing around a | for flag type of expressions (json_encode comes to mind with something like JSON_NUMERIC_CHECK|JSON_PRETTY_PRINT).
For ease of implementation I would suggest to allow a new strategy for the operators option on the binary_operator_spaces, called something like no_space.
Before I or someone else starts working on a PR I would like to hear some more opinions on this topic :)

As it would be against PHP 12 (yes, it's in review, but we are going this way):

All binary and ternary (but not unary) operators MUST be preceded and followed by at least one space.

I'm 👎 for this one.

closing as https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/3833 has been merged


control wrapping of type signatures

Signatures are best wrapped before or after:

the :: following the name;
the => following the type-class constraints; or
a -> separating argument types.

This wrapping is undesirable:
get :: Accessible a => TypeRep b -> String -> a -> Maybe
b

This wrapping is preferable:
get :: Accessible a => TypeRep b -> String -> a ->
Maybe b

To achieve the desired wrapping we replace SPACE with NO-BREAK SPACE in positions where wrapping should not occur. For example:
get :: Accessible_a => TypeRep_b -> String -> a ->
Maybe_b

Underscores represent U+00A0 in the example above.
I'd :heart: a review from @benperez. I believe controlWrapping to be :ram:worthy, though I'd appreciate a review of the logic as this project lacks tests.

I love that you were able to make this happen: R.map(R.map(R.map(R.map(nbsp)))) in real code. One question I have is which of these 2 alternatives is more readable:
R.pipe(
  R.map(foo),
  R.map(R.map(bar)),
  R.map(R.map(baz)),
  R.map(R.map(R.map(quux))),
  fizzbuzz
)

R.pipe(
  R.map(R.pipe(foo,
               R.map(R.pipe(bar,
                            baz,
                            R.map(qux))))),
  fizzbuzz
)

I love that you were able to make this happen: R.map(R.map(R.map(R.map(nbsp)))) in real code.

:grinning:
@scott-christopher also suggested nested pipes to avoid nested maps. I find nested maps significantly more readable, as the triangular shape of the code makes it easy to see which "joining" transformation undoes each "split". In a sense R.map(R.map(R.map(R.map( is just indentation:
R.pipe(R.split(' :: '),
             R.split(' => '),
                   R.split(/([(][^()]+[)])/),
                   R.append(''),
                   R.splitEvery(2),
                         R.over(R.lensIndex(1), nbsp),
                   R.unnest,
                         R.split(' -> '),
                               nbsp,
                         R.join(' -> '),
                   R.join(''),
             R.join(' => '),
       R.join(' :: '))

Splitter
Joiner

R.split(' :: ')
R.join(' :: ')

R.split(' => ')
R.join(' => ')

R.split(/([(][^()]+[)])/)
R.join('')

R.splitEvery(2)
R.unnest

R.split(' -> ')
R.join(' -> ')

I find your arguments compelling @davidchambers and I award you one :ram: for maniacal yet effective use of ramda in a pull request.

Thank you. I will treasure it always!

I tested this on Sanctuary. It works an intended. Wrapping can still occur in the middle of ->, but that's a separate issue (which could be addressed by using → instead).


Download Latin.unicharset along with radical-stroke.txt

Need another PR to add Inherited.unicharset after https://github.com/tesseract-ocr/langdata_lstm/pull/41/ is merged

All unicharset files for scripts are potentially needed, starting with Arabic.unicharset and ending with Thai.unicharset.
I usually get the required ones to satisfy the error message(s), but still don't know what happens if they are missing.

I added only Latin and Inherited unicharsets in this list because these are required in almost all cases, even though they don't stop processing like missing radical-stroke.txt.
We could add another optional variable for SCRIPT_UNICHARSET, downloading it when it is non-blank.

still don't know what happens if they are missing.

I think some characters e.g. Arabic accents get dropped in the generated unicharset by unicharset_extractor.  That was the reason I built the Inherited.unicharset.

A list of all required unicharset files can be extracted from unicharset:
cat $(OUTPUT_DIR)/unicharset | sed s/.*0,0,0.// | sed 's/ .*//' | sort | uniq | grep "^[A-Z][a-z][a-z]*" | grep -v common

A list of all required unicharset files can be extracted from unicharset:
cat $(OUTPUT_DIR)/unicharset | sed s/.*0,0,0.// | sed 's/ .*//' | sort | uniq | grep "^[A-Z][a-z][a-z]*" | grep -v common

Thanks for the suggestions @stweil and the hint to get the list of required unicharsets from  $(OUTPUT_DIR)/unicharset.
I am having a hard time putting it together in a separate Makefile target using the list. Would appreciate if you can make the required change.

Thanks for the suggestions @stweil and the hint to get the list of required unicharsets from  $(OUTPUT_DIR)/unicharset.
I am having a hard time putting it together in a separate Makefile target using the list. Would appreciate if you can make the required change.

@kba Could you pls. have a look at the change request and maybe come up with a proposal?

@kba Could you pls. have a look at the change request and maybe come up with a proposal?

I added  sed '/Common/d' | sed '/Inherited/d' | sed '/Joined/d' to the script suggested by @stweil because there are no unicharsets for Common and Inherited . Joined was being picked up accidentally.
A simpler way maybe asking the user to specify a script and download that.

I added  sed '/Common/d' | sed '/Inherited/d' | sed '/Joined/d' to the script suggested by @stweil because there are no unicharsets for Common and Inherited . Joined was being picked up accidentally.
A simpler way maybe asking the user to specify a script and download that.

A simpler way maybe asking the user to specify a script and download that.

I have tried that in the new Makefile-font2model
I think that is a much cleaner way of doing this.

A simpler way maybe asking the user to specify a script and download that.

I have tried that in the new Makefile-font2model
I think that is a much cleaner way of doing this.

Included as part of https://github.com/tesseract-ocr/tesstrain/pull/230


GetNeighbourObjArr missing, breadcrumbs tree empty

Hey
I never got Breadcrumbs to work, now troubleshooting it.
A sample note with

Parent:: [[Cooking MOC]]

As the first line
DataView installed and enabled
Breadcrumbs plugin installed and enabled
Tried debugging it in the DEBUG mode, but console shows no errors. Interestingly, there's also no GetNeighbourObjArr when I refresh the index.
What's the next step?

Hey, thanks for checking the debug logs :)
It may be that you need to add the parent field to your Hierarchies. It isn't recognized by Breadcrumbs by default

True, that was it, thanks. It didn't strike me it would not be set up by
default especially that I believe some of the examples used "parent".
Anyway, thanks a lot for a quick response.
śr., 26 sty 2022 o 06:24 SkepticMystic @.***>
napisał(a):

Hey, thanks for checking the debug logs :)
It may be that you need to add the parent field to your Hierarchies. It
isn't recognized by Breadcrumbs by default
—
Reply to this email directly, view it on GitHub
https://github.com/SkepticMystic/breadcrumbs/issues/290#issuecomment-1021879469,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAL7IZ2GJTCPSAUW6OQP57LUX6APFANCNFSM5MZIEW2Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


Not Really An Issue - using Lumberjack from a .framework - where to embed it?

New Issue Checklist

[x] I have read and understood the CONTRIBUTING guide
[x] I have read the Documentation or CocoaDocs
[x] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
iOS

Platform Version
latest

CocoaLumberjack Version
latest

Integration Method
cocoapods

Xcode Version
latest

Repro rate
100%

Demo project link
not worth it?

Issue Description and Steps
This isn't really an 'issue', this is me, a newb, trying to figure out Xcode and Cocopods and Lumberjack... go ahead and close this, if you've read this far and are immediately annoyed.
Here's the goal, and you don't spell this out in your docs: I am trying to make an iOS framework, that includes Lumberjack. I've gotten far enough in cocoapods that LJ is included and the project compiles. (This is in Swift). Now I want to make an iOS app that consumes the framework I just made, and when I try and run it, it crashes and tells me Lumberjack isn't loaded. Somewhere, I need to include the LJ framework or binary, or whatever it's called. But I don't know where I need to do this, or how. I tried including the LJ framework in my general settings for the .framework I am building. It wouldn't compile when I did that, because it can't find the Lumberjack.framework. I don't know if I need to link against it, or what. It's pretty weird, since because it is swift, it seems to need to build the LJ framework from scratch?
Anyhow, some kind of message about how to do this would be great. I'm not seeing how to do this in the GitHub code, even the examples. Too complicated for me to follow!
some help would be great...
thanks

Hey... think I figured something out... In my framework that I am building, I am trying to add to General/Frameworks the CocoaLumberjack.framework, and turning on embed... if I don't select embed, the project compiles, but crashes when I run it, because it can't find LJ.If I turn on embed, it says,
error: /Users/ericfrazer/Projects/FrameworkTestMaster/FrameworkTest/DerivedData/FrameworkTest/Build/Products/Debug-iphonesimulator/CocoaLumberjack.framework: No such file or directory (in target 'FrameworkTest' from project 'FrameworkTest')
well I guess I wasn't looking closely enough in Finder. If I go to finder, to Debug-iphonesimulator, I see CocoaLumberjack, a folder, and FrameworkTest.framework (a folder), and Pods_FrameworkTest.framework. Okay, what's the outlier here? The CocoaLumberjack folder doesn't have a .framework in the name. If I open it up, there you go, there is a CocoaLumberjack.framework folder in it! If I copy & paste this SUBFOLDRE parallel to the other .framework folders, lo & behold, the project now builds, with the embedded LJ framework. Plus, added bonus, it runs w/o crashing.
I'm thinking for whatever reason, the LJ.framework folder is building one level too deep! I'm so new at Xcode, I'm not even sure where to look for the answer to WHY. What specifies the output folder in a Cocoapods project for where the framework builds to?!

Looking into my Pods project, under Targets/CocoaLumberjack, the user-defined CONFIGURATION_BUILD_DIR's debug and release paths both end in /CocoaLumberjack.  How does CONFIGURATION_BUILD_DIR get set, and boy, this was sure confusing.

Seems like this has been resolved. Sorry for not being of any help here...


Update munit, munit-scalacheck to 0.7.29

Updates

org.scalameta:munit
org.scalameta:munit-scalacheck

from 0.7.28 to 0.7.29.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta" } ]

labels: test-library-update, semver-patch

Codecov Report

Merging #131 (95bf006) into master (f3ce324) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #131   +/-   ##
=======================================
  Coverage   98.27%   98.27%
=======================================
  Files           1        1
  Lines          58       58
  Branches        2        2
=======================================
  Hits           57       57
  Misses          1        1

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 f3ce324...95bf006. Read the comment docs.


Feature request: add virtio-fs support (alternative to virtio-9p)

Description of problem
virtio-fs is an alternative to virtio-9p with better performance and POSIX compatibility.  These properties make it useful as a drop-in replacement for virtio-9p.
The virtio-fs website is here: https://virtio-fs.gitlab.io/

@liwei Hi, why was this closed?  There is a pull request regarding this feature here: https://github.com/kata-containers/runtime/pull/1016


Willpower Skill Focus doesn't show

Skill focuses on Willpower do not show up. This is both full and block mode.
If the same focus is added a second time I get the error: Focus is not added to character - it already owns a Focus named "SELF-DISCIPLINE"
So its there as far as the code is concerned, but the UI is not displaying it.

Hi! I tested on my end adding SELF-DISCIPLINE as a Willpower focus and it worked fine. Could you confirm the no modules are active when you experience this issue? Also, you could also hit F12 and print the latest messages from console log.

So I created a brand new world, made a test actor and test focus and...it shows.
So whatever is causing the bug must be specific to that World. .....i'll begin the process of migrating my items/journals/etc over to the new world I created and see if any of them break it again LOL.

Oh boy... it seems you got your previous world corrupted. I hope you are able to port everything to a healthy world!
I will close the ticket now, but you can re-open it if the problem returns.


Add "lite" sub string matching

This pull request is opened for a discussion about how best to go forward with this merge given that this repo is public and used in npm.
The change is simple. We have added an additional flag  "lite" to the substrings option property. This will use a separate words file for substring matching in order to not be so strict. A further options property subStringList allows for a custom "lite" word list to be specified.
If I get a green light on this approach I will update the README.

@pazdera


Manifest "strict" validation mode

As an operator, I get  fast feedback on invalid configuration properties in my manifest for a given job
Use case
An operator mistypes an optional property or adds an unsupported property for a job and once deployed is expecting the system to have a certain behaviour.
Todays solution
Today we can add logic into the templates to check for unexpected properties and throw an error.
This is resulting in a lot of boilerplate code that gets written.
Bonus points
Release authors should be able to write spec files for repeating structures. This is a major shortfall in spec files today.

Duplicate/semi-related to #1568

Closing in favor of #1568


Cards do not load correctly

I am not sure what is causing this:

I checked the past couple of PR previews they all seemed to be fine. Maybe a redeploy could fix it?

cc: @jordinl83 to investigate

Seems like it was caused by https://github.com/cncf/landscapeapp/pull/770, which has been reverted.


[vcpkg] Add -w dupbuild=warn for ninja

Update ninja to 1.10.0, if we build the ports with ninja 1.10.0, it will fail due to this error:
ninja: error: build.ninja:225: multiple rules generate tmx.lib [-w dupbuild=err]
There was a change made to ninja between v1.8.2 and v.1.9.0:
when multiple rules are found for building the same binary, lib or file:
prior to ninja v1.9.0 it was a warning, the build continues, but since ninja v1.9.0 it is an error, the build stops.
This should be a ninja bug.
Related references:
https://devzone.nordicsemi.com/f/nordic-q-a/46041/build-error-after-upgrading-to-ninja-release-1-9-0
https://gitlab.kitware.com/cmake/cmake/issues/18900
https://bugs.launchpad.net/kicad/+bug/1829082
Related issue #10887
Fixes #11959

This was intentionally changed in Ninja by https://github.com/ninja-build/ninja/pull/1406

When I did vmss that updated ninja for our builds to 1.10. The ports portaudio, and sundials were affected.

@ras0219-msft indicates that it would be better to do this in the 2 affected ports rather than everywhere.

Closes #11959

I think that sundials and portaudio need to have their builds fixed and we should not paper over this correctness fix Ninja is helpfully pointing out for us. @ras0219 suggests that this might be because those ports are trying to build their static bits and their DLL bits with the same extension.

This is most likely caused by the problematic ports building a shared and static form simultaneously with the same output name. On non-Windows, this is fine because they have different file extensions (.so/.dylib vs .a), however on Windows the import lib and static lib conflict.
The correct fix for this is to patch the builds so they do not try to build both binaries at the same time, which is goodness all around. Until then, I think we want to keep them as "fail".

Please dont just keep them as fail this literally breaks building any project using libmariadb, devil and others when using Ninja Makefiles and packages with PREFER_NINJA. Atleast make it an option to have this as warning.


Readme quick start guide

Dear Nelson,
Do you mind writing in the readme a quick start guide, especially the view and url implementation? It is hard to guess from your code how to integrate exactly your lib in django.
Best regards

Hello Nelson,
like Yannik I believe it's could be great to have a quick starter in order to help us to use you code.
Thank for you help
Regards

Hi! Sorry for the late response. I’ve not been to active on GitHub. I will try to create a quick guide on how to use it on the weekend. This was something I created really quick just to resolve my problem I’m not sure if still works with the latest version of flow. Anyway I will put the version that is working and write some issues that I found.
--
Nelson Monteiro
Skype: nelsonmonteiro.net
Email<EMAIL_ADDRESS>Mobile: (+47) 405 31 317
On 12 March 2016 at 09:30:32, Jonathan CHARDON<EMAIL_ADDRESS>wrote:
Hello Nelson,
like Yannik I believe it's could be great to have a quick starter in order to help us to use you code.
Thank for you help
Regards
—
Reply to this email directly or view it on GitHub.

Also does this support joining the chunks back together? I ask this as i wanted to use this as a reference.

Hi! Yes, this joins the chunks together after download. I have updated the file README.md with some instructions.

Thanks for that. I had one more doubt, how will the POST work in my view. If you could walk me through it, it'd be great. As the sample view which you mentioned isnt for that correct?

And Installation steps are missing in the README.md


LanceDatasink for integration with ray doesn't support None

Passing None to nullable fields crashes ray jobs.
import lancedb
import pyarrow as pa
import ray.data
from lance.ray.sink import LanceDatasink

uri = 'testdb'
table = 'test_table'

schema = pa.schema([
    pa.field("a", pa.list_(pa.string()), nullable=False),
    pa.field("b", pa.list_(pa.string()), nullable=True),
    pa.field("c", pa.list_(pa.string()), nullable=True),
])
data = [{"a": ["foo"], "b": None, "c": None}] * 100

db = lancedb.connect('testdb')
db.create_table('test_table', schema=schema, exist_ok=True)

# init datasink
datasink = LanceDatasink(
    uri=f"{uri}/{table}.lance",
    schema=schema
)

ds = ray.data.from_items(data)
ds.write_datasink(datasink)

# Read data from table
table = db.open_table('test_table')
print(table.count_rows())

Same thing is happening for other datatypes. For example if datatype is pa.string() passing None still crashes ray job. Using pa.null() instead works for primitive datatypes but doesn't work for pa.list_(pa.string())

Here's the error.  It appears the pyarrow -> arrow-rs conversion is failing for null arrays:
(Write pid=278398) thread 'lance_background_thread' panicked at /home/pace/.cargo/registry/src/index.crates.io-6f17d22bba15001f/arrow-data-53.3.0/src/ffi.rs:311:9:
(Write pid=278398) assertion failed: !self.children.is_null()
(Write pid=278398) stack backtrace:
(Write pid=278398)    0: rust_begin_unwind
(Write pid=278398)    1: core::panicking::panic_fmt
(Write pid=278398)    2: core::panicking::panic
(Write pid=278398)    3: arrow_array::ffi::ImportedArrowArray::consume_children
(Write pid=278398)    4: arrow_array::ffi::ImportedArrowArray::consume
(Write pid=278398)    5: arrow_array::ffi::ImportedArrowArray::consume_children
(Write pid=278398)    6: arrow_array::ffi::ImportedArrowArray::consume
(Write pid=278398)    7: arrow_array::ffi::from_ffi_and_data_type
(Write pid=278398)    8: <arrow_array::ffi_stream::ArrowArrayStreamReader as core::iter::traits::iterator::Iterator>::next
(Write pid=278398)    9: tokio::runtime::task::raw::poll
(Write pid=278398) note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
(Write pid=278398) thread 'lance_background_thread' panicked at /home/pace/dev/lance/rust/lance-datafusion/src/utils.rs:24:35:

A workaround for the above script (not sure if it works on your real data) is to supply the schema when you create the table so you never get null arrays.
Change ds = ray.data.from_items(data) to ds = ray.data.from_arrow([pa.Table.from_pylist(data, schema=schema)])

Thanks for the fast response! I am reading data from parquet, performing transformations and assigning None's when needed. Job ends with .map where I pack data into dict ... smth like this:
def f(row):
    return {
        'a': row['a']
        'b': None,
        'c': None
    }

ds = ray.data.read_parquet(input_file)
ds = ds.map(f)
ds.write_datasink(datasink)

write_datasink fails in this case.


Security Definitions

Hmm, this is a tough one. I don't actually think this is possible but I'll check at least.
I'd like to somehow generate SecurityDefinition entries in the swagger spec.
Looking at how authentication is handled, there're all these filters that we don't have access to. We can get @Context annotation in Resource, but no way to describe what form of security is required.
Thoughts? I'll probably simply introduce an additional build step before swagger-codegen to inject the required json at this stage.

Puh, this would be hard to get...
It would be possible to read a potentially existent web.xml (or other deployment descriptors), but as a) many app servers offer non-standard security features and more important b) a lot of enterprise setups use a proxy server or authentication portal upfront (without any change to see that in the EE source) I would leave it out for now.
Too much required work for a small chance to get it right (in a generally usable way).

Anyway, there might be a possibility to manually add that information to the generated doc (by adding extra configuration in the pom or via CLI). Depending on the options Swagger offers of course, but I haven't looked at Swagger's SecurityDefinitions yet TBH.
So feel free to investigate in that direction -- but I wouldn't bloat the configuration too much, only if there is a really easy way...

The swagger spec regarding security definitions is quite significant. Highly doubt it will be something configurable. Each resource path can be arbitrarily defined as required 1-many security levels and scopes.
If I figure something out I'll drop another line here. Could be useful for other users in the future.

If anyone else comes around in the future, here's how I'm solving it (at a basic level) using Gradle:
// in your jaxrsanalyzerdoc task

doLast {
        def json = new groovy.json.JsonSlurper().parse(file(outputFile))
        json.securityDefinitions = [
            "auth": [
                "type": "basic",
            ],
            "jwt" : [
                "type": "apiKey",
                "name": "Authorization",
                "in"  : "header",
            ],
        ]

        json.security = [
            [ "auth": [] ]
        ]

        file(outputFile).write(groovy.json.JsonOutput.toJson(json));
    }

@sdaschner so I had a dream (literally, I dream about this shit in my sleep somehow), and thought of a configuration option which you can point to another json/yml spec which you can tell jaxrsanalyzer to deep-merge in. Removal can simply be "null" entries.
This really should be a general purpose utility people make  (I'll probably do it for my build separately for example), but thought I'd suggest it as an option if you want to consider it.

literally, I dream about this shit in my sleep somehow

I know that feeling :-)
Yes, I see what you mean, however this is a really complex change and targets more to post-processing output files (like you would on AsciiDoc for instance). So probably a better target for a subsequent plugin call like a modify-something-plugin which modifies your output files, i.e. the swagger output by merging another json/yml snippet in.
So would be a cool extra but probably too much and not the right scope for the Analyzer.


[release-23.05] firefox-{,bin-}unwrapped: 115.0.3 -> 116.0; firefox-esr-102-unwrapped: 102.13.0esr -> 102.14.0esr; firefox-esr-115-unwrapped: 115.0.3esr -> 115.1.0esr

Backport #246454
Untested.
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Looks good.  x86_64 NixOS 23.05:

firefox-bin and firefox-esr started and displayed some webs
firefox is what I switched to yesterday


Hi, will it have a CPU version impl?

Hi, will it have a CPU version impl?

We focus on reducing GPU memory reads/writes to speed up attention & save memory. The bottlenecks on CPUs are likely to be different.

Could have been great for the sake of having uniform code and only change the device...

Sure would have been great, someone just needs to write the code


Bundle icons

Should probably bundle some core icons. On Slackware this is not an issue, but I have noticed some issues with other distros where the system icon theme is broken(-ish).

done


Clarify variable groups, secure files, and agent pools protected resources

The documentation lists the following resources as "protected":

variable groups
secure files
agent pools

The documentation notes that "protected" means:
"You can run additional manual or automated checks every time a pipeline uses one of these resources."
However, variable groups, secure files, and agent pools do not support "checks".

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

ID: d40fccec-c358-843c-10d5-7ea367f6e406
Version Independent ID: 45390d5d-c1d7-c90e-ef3f-2768d406ba7b
Content: Pipeline resources - Azure Pipelines
Content Source: docs/pipelines/security/resources.md
Product: devops
Technology: devops-cicd-security
GitHub Login: @juliakm
Microsoft Alias: jukullam

@poltj18 Thanks for pointing this out. Variable groups, secure files, and agent pools support checks in a recent update. You can see a detailed overview of permissions in the About resources article. I'll add a link to it from this one.


Warnings should be optional (or removed entirely)

They take up precious real estate, and some of these warnings are extremely opinionated and I may choose to never address them. I may have a really long function and I can't or won't ever do anything about it, so I should not be forced to lose visual real estate. You've provided no way to toggle this stuff off or adjust it. And frankly, this doesn't even seem like the responsibility of YOUR package, it's just causing bloat and complexity.
If you REALLY want to provide intuitive linting in symbol view (which I don't really think is necessary since I can see linting errors in my scrollbar, gutter, and minimap) then you should instead simply count how many linter errors and warnings are in each function, etc, and simply give me a warning or error icon on the symbol that contains such issues with a count of how many errors or warnings are found.
However, I ultimately feel this entire feature just needs to be removed. Completely superfluous and not worth the time to work on since it's done in such a limited manner.

Now warnings are optional and can be disabled/enabled in settings (thanks to @srguiwiz )


feat(storage): add persistent storage permission check

Browsers have internal limits for storage that may be cleared when under storage pressure. However, a recently-added permission allows the storage to be permanent.
This currently asks for the permission on every load (which might be overly aggressive).
See https://developers.google.com/web/updates/2016/06/persistent-storage for details.
Testing

Serve from http:// (not localhost) and verify that there are messages in the logs stating that persistent storage is not permitted in an unsecure environment
Firefox: Serve from https:// (not localhost) and verify that the user is prompted for a persistent storage permission

Verify that declining the permission produces a warning in both the toast alerts and in the log
Verify that accepting the permission produces a successful log message and that navigator.storage.persist() resolves to true afterward

Chrome: Serve from https on a hostname that you don't hit often

Verify that it asks for notification permissions, and that accepting the permissions results in navigator.storage.persist() resolving to true

This build is failing due to .travis.yml referencing a file that doesn't exist yet on this branch.  Get the latest from master to resolve.

@schmidtk I added some detection on whether the result of the promise was a user decision or automatically resolved based on timing. Also added request for Notification permissions in Chrome.

navigator.storage isn't writable, so mocking it for tests is difficult. I could add code to allow passing in a mock StorageManager for tests. Would that be preferable or would we rather avoid the test-only code?


Added back to top button #151

Issue #151
https://github.com/user-attachments/assets/9131a7b4-fa6f-4be1-9469-f92f7e68afa8
This PR will add a back to top button on all the pages for consistency and better user experience
i have uses best codding practice and added the button on all the pages by including in app.jsx
and also maintain the folder structure by adding this button on shared folder to mantain consistency
After adding animation
https://github.com/user-attachments/assets/1ec692cb-b11f-44e9-a3e3-def6396fcd5e

@Sawan-Kushwah can you make this more attractive via adding simple animation to it?

@Shiva-Bajpai Sure i can

@Shiva-Bajpai Added animation you can see in video i have uploaded