Plugin fails to build

Prerequsities

[x] I'm on the latest version of the plugin
[x] I've installed the required dependencies
[x] I've run :h gitlab.nvim.troubleshooting and followed the steps there

Setup Configuration and Environment
return {
	"harrisoncramer/gitlab.nvim",
	dependencies = {
		"MunifTanjim/nui.nvim",
		"nvim-lua/plenary.nvim",
		"sindrets/diffview.nvim",
		"stevearc/dressing.nvim", -- Recommended but not required. Better UI for pickers.
		"nvim-tree/nvim-web-devicons" -- Recommended but not required. Icons in discussion tree.
	},
	enabled = true,
	build = function() require("gitlab.server").build(true) end, -- Builds the Go binary
	config = function()
		require("gitlab").setup({
			debug = {
				request=true,
				response=true,
				gitlab_request=true,
				gitlab_response=true,
			},
			log_path = "D:/gitlab.nvim.log"

		})
	end,
}

OS: Windows
Bug Description
When installing the plugin, it does not build the server binary. At the same time, building manually via go build -o bin.exe works (running in gitlab.nvim/cmd directory).
My hypothesis: it tries to build the binary in the wrong directory.
Reproduction Steps

Check out the feature branch
Open Neovim
Observe error:

Could not install gitlab.nvim!

Observe that there is no binary bin.exe in gitlab.nvim folder.

Screenshots

Interesting. I'm not a Windows user so this would be hard for me to debug, as you've already determined there's something going wrong here:
  local file_path = u.current_file_path()
  local parent_dir = vim.fn.fnamemodify(file_path, ":h:h:h:h")
  state.settings.bin_path = parent_dir
  state.settings.bin = parent_dir .. (u.is_windows() and "\\bin.exe" or "/bin")

  if not override then
    local binary_exists = vim.loop.fs_stat(state.settings.bin)
    if binary_exists ~= nil then
      return
    end
  end

  local cmd = u.is_windows() and "cd %s\\cmd && go build -o bin.exe && move bin.exe ..\\"
    or "cd %s/cmd && go build -o bin && mv bin ../bin"

  local command = string.format(cmd, state.settings.bin_path)
  local null = u.is_windows() and " >NUL" or " > /dev/null"

  -- If you add this line we can see the command that's actually being run
  print(command .. null)

  local installCode = os.execute(command .. null)
  if installCode ~= 0 then
    u.notify("Could not install gitlab.nvim!", vim.log.levels.ERROR)
    return false
  end
  u.notify("Gitlab.nvim installed successfully!", vim.log.levels.INFO)
  return true

To debug this, I'd probably want to know the exact command the plugin is trying to run. You can see that if you change into the plugin directory and edit that function in lua/gitlab/server.lua to add a print before the command is actually executed.

This is something to do with your setup.
With an EC2 instance running Windows and with the basic requirements needed for this plugin (Lazy, Neovim, Go, and the bare bones configuration from the Readme), the plugin installed the bin.exe in the correct location and everything worked as expected.

Hey @kostya9 are you able to provide the above information?

Sorry for the long delay.
The command looks fine, but it fails for some reason. When I copy this command and run manually - it works. Now that I think about it - this may be connected to the fact that I use PowershellCore as terminal instead of the default things, and maybe my vim terminal settings are causing this issue.

The build command starts to work if I change it to this (executing commands via vim.system):
![image](https://github.com/user-attachments/assets/9daf622a-1219-4e40-a42e-260d4b3b2645)

That's hugely helpful, thank you.
I've opened up an MR that should fix this: https://github.com/harrisoncramer/gitlab.nvim/pull/419
Do you mind checking out this branch and tell me if the server builds correctly? You should be able to check out a specific branch with Lazy, then run this to force a rebuild:
:lua require("gitlab.server").build(true)

Hi @harrisoncramer, How do I know if I built it correctly?
I now get an error:
   Error  21:04:17 notify.error gitlab.nvim: panic: could not find emojis at ./config/emojis.json
goroutine 1 [running]:

   Error  21:04:17 notify.error gitlab.nvim: github.com/harrisoncramer/gitlab.nvim/cmd/app.CreateRouter(0xc000252000, 0x1?, 0xc00011fd88, {0xc00011fdf0, 0x3, 0x5c35a7?})
	D:/Users/ding.zhao/AppData/Local/nvim-data/lazy/gitlab.nvim/cmd/app/server.go:94 +0x3b4b
github.com/harrisoncramer/gitlab.nvim/cmd/app.StartServer(0xc000252000, 0xc00023a678, {{0xc000174ff0, 0x42}, {0xc000175010, 0xb}, {0xc00017501c, 0x12}, {0xc00023e000, 0x24}})
	D:/Users/ding.zhao/AppData/Local/nvim-data/lazy/gitlab.nvim/cmd/app/server.go:26 +0x156
main.main()
	D:/Users/ding.zhao/AppData/Local/nvim-data/lazy/gitlab.nvim/cmd/main.go:45 +0x397

   Error  21:04:17 notify.error gitlab.nvim: Golang gitlab server exited: job_id: 9, exit_code: 2

Ah, okay I think that was due to me using forward slashes in the emoji location, I've made this fix:

Could you pull down the latest for this branch and try again: https://github.com/harrisoncramer/gitlab.nvim/pull/419

Hi @harrisoncramer, I tried it out and it's running well. However, I encountered this issue: I can't close any of the gitlab.nvim pop-up windows using esc or q. Is this normal?

I'm not sure why your diagnostics wouldn't close. But both of these are separate issues from this original issue, if you are unable to resolve them please open up a separate ticket. I'll close this one out now.

Thank you so much for your support! This is a great plugin.

Checked on my machine, it now works too, thanks!


Version error

Because image >=3.2.1 depends on xml ^6.0.1 and image >=3.2.0 <3.2.1 depends on xml ^6.0.0, image >=3.2.0 requires xml ^6.0.0. And because every version of image_picker_plus depends on image ^3.2.0, every version of image_picker_plus requires xml ^6.0.0. And because excel 2.0.0-null-safety-3 depends on xml ^5.0.2 and no versions of excel match >2.0.0-null-safety-3 <3.0.0, image_picker_plus is incompatible with excel ^2.0.0-null-safety-3. So, because myfeats depends on both excel ^2.0.0-null-safety-3 and image_picker_plus ^0.5.5+1, version solving failed. pub get failed (1; So, because myfeats depends on both excel ^2.0.0-null-safety-3 and image_picker_plus ^0.5.5+1, version solving failed.) exit code 1
I am getting this error after adding the package

You can check the last branch #55
You can add the package from github like:
image_picker_plus:
git:
url: https://github.com/AhmedAbdoElhawary/image_picker_plus


[docs-infra] Fix bug on API prop copy experience

I have noticed this while I was looking at #39704:

When I click on prop links in dev mode http://<IP_ADDRESS>:3000/material-ui/api/dialog/#Dialog-prop-classes

⬇

When I click on prop links in production is replaces my clipboard, I didn't ask him to.

The simplest solution seems to remove this behavior. If we really want to keep it, we would need to use clipboard-copy instead.

@danilo-leal Maybe it comes to a matter of preferences, why I think it's better UX overall not to have it:

It's how most other website trains people for what to expect. From the benchmark of https://www.notion.so/mui-org/Docs-infra-83ba469842104927bb13606b1e859ef4?pvs=4#7647eb75870c42a79dc5bdd85cc412e2, only 2 of the 5 docs have this pattern. When looking at websites more broadly (ones that are not so focused on UX), I find it hard to find other examples (so far, I can only find these two do it).
I don't like surprises, I heavily depend on the content of my clipboard, and it's frequent for me to copy, close the content source, check a few things, and paste.
I'm using the anchor links more than for sharing links: copying the content of headers, which with #39603 could create problems, and sometimes just reset my scroll.
Why is it only the API props that have this behavior and not all the docs.

@danilo-leal I could get behind Stripe's behavior. Meaning implementing this for all headers and with a post-notification for affordance. I suspect that I would personally enjoy it more without copy but I also didn't research this much more than what I shared before, so a #suggestion+.

A new benchmark: https://platform.openai.com/docs/guides/prompt-engineering/split-complex-tasks-into-simpler-subtasks, in the same "class" as https://developers.google.com/maps/documentation/javascript/maptypes. They never change the anchor in the URL.
I'm not sure how it feels. It makes me dubious that it actually work, the first thing I want to try is open a new window to check that the link does work 🤔

Yeah... Open AI's docs are weird in that sense because it's similar to Stripe's, where the icon doesn't necessarily mean that the URL will be copied, but then the scroll position + URL is not changed at all. Maybe a good place is one where, by hovering a heading, you see two icon buttons: one for anchoring and another for copying the URL.


Broken website?

Hi,
Not sure if you're already aware but:

https://www.boringgenerators.com/ fails to load.
http://www.boringgenerators.com/ goes to a parking page.

Andy

Thanks, @andyw8 😊 . I have fixed the issue and the website.

Seems to be down again?


Limit alert notifications

Description:
Sound alert is currently playing every time it's triggered by new datapoint. This is annoying some users.
We will add setting under Alert Notifications in App Settings to disable sending multiple alert notifications (notification popup and sound).
Limit alert notifications
Settings will limit any enabled alert logic for sending/playing alert notifications in app (Bluetooth alerts only).
Default setting: On
User story:
User has enabled temperature alert for Bluetooth 10,1 alert triggers alert, soon value changes to 10,2.
When this setting is enabled there will be no new notification (notification popup, sound alert) shown for user.
New notification triggered after 1 hour if limit is still over threshold.

iOS card: https://github.com/ruuvi/com.ruuvi.station.ios/issues/1505

@andreevdenis notice we changed order of the menu items on iOS due to footer. Let's change the order similarly on Android.

@andreevdenis I signed in to Cloud and enabled Cloud mode, then also enabled limit alert notifications. Looks like I am not receiving only 1 alert from cloud as well.
Expected: limit only local Bluetooth alerts, this setting should have no effect on Cloud alerts.

Rechecked will pass this card


Runtimes: Added pypy-5.3.0, mv python-2.7.11rc1 python-2.7.12rc1

Runtimes:

Added pypy-5.3.0
Moved python-2.7.11rc1 to python-2.7.12rc1

I think I may stop releasing the RCs. No one ever uses them.

I am happy to drop the RC if you want.  I saw 11 there so I updatedi it to 12. The finals should be here in a week anyway.

Thanks! This is on my todo list.

Closed in favor of PR https://github.com/heroku/heroku-buildpack-python/pull/313


Correct Kvrocks spells and new repo location

cc @vmihailenco

Wow, looks like a big step for kvrocks - congratulations!

@vmihailenco thank you!
FYI - I'm one of the mentors of Apache Kvrocks (Incubating) and trying to bring go-redis to build its integration tests (TCL scripts are hard to debug, lol).

@vmihailenco https://github.com/apache/incubator-kvrocks/pull/806#issue-1359235614
FYI - so far, so good :)


Reword license mentions

Summary
Remove the header from the plugin settings and reword the hard requirement on an E20 license in the README.
Ticket Link
--
Checklist

[ ] Telemetry updated
[ ] Gated by experimental feature flag
[ ] Unit tests updated

@jasonblais: Just confirming that the terminology is correct (i.e. "Mattermost Professional" as opposed to "The Professional Edition of Mattermost".)

"Mattermost Professional" is correct, but I've asked Jamie to review the text as part of her other license reviews for this repackaging effort.

@jasonblais, @jamiehurewitz, do you think we can make that review after we merge this PR? Or should we wait?


Perform download tasks in the background; add a progress bar (ooooh)

E.g. this sort of approach: https://stackoverflow.com/questions/15644964/python-progress-bar-and-downloads

Done in 58508cbc7c2c6a95740a16ea880806e71736ce60.


Add ability to reformat specific tables with different options enabled

Specifically regarding reorder_keys and reorder_arrays, it can be that part of the toml file can have these reorders, and other parts should not.
The typical case I run into is Cargo.toml, where the rust tools have a preferred order of the keys in the package table, but it is desirable to reorder the keys in all of the other tables such as features, dependencies, etc.  https://github.com/Electron100/butane/pull/40 is an example of using reorder_keys on Cargo.toml files.
IMO it would be great if the fmt command CLI allows patterns like the get command, to specific which part of the file to apply the formatting to.

You can use rules for that:
[formatting]
reorder_keys = true

# Do not reorder keys within a package or a dependency.
[[rule]]
formatting = { reorder_keys = false }
keys = ["dependencies.*", "package"]

My question would be: Is it possible to define a non-alphabetical ordering? And should this be part of a schema? The concrete example would be that within the package table, I would like to use the following ordering:

name is required and first
version is required and second
authors is required (and next)
license is required (etc)
edition is required
rust-version is optional (and next)
publish is optional (etc)
keywords is optional
categories is optional
readme is optional
etc


⚠ Bofrost.si has degraded performance

In 83015fa, Bofrost.si (https://www.bofrost.si) experienced degraded performance:

HTTP code: 200
Response time: 3316 ms

Resolved: Bofrost.si performance has improved in dd153ec after 6 minutes.


How to make round border Rect?

Feature Request
I want show a round border rect (with also so said "borderRadius") but I am not able to find a way to make
Why it is needed
Designers' request
Possible implementation
<Svg width="200" height="60">
  <Rect
    x="25"
    y="5"
    width="150"
    height="50"
    fill="rgb(0,0,255)"
    strokeWidth="3"
    stroke="rgb(0,0,0)"

    **borderRadius**={5}
    **topLeftBorderRadius**={15}
    **bottomRightBorderRadius**={30}

  />
</Svg>

I found it. use a rx and ry props to get round corners.
but not able to set each corner to different values.

I believe it is a duplicate of https://github.com/react-native-svg/react-native-svg/issues/1393. Could we close it to keep the discussion in one place?


Create a new tag

Could you please tag a new version. The latest tag is 4 months old.
It would be also great to tag all submodules to the same tag name, like glslang-v2.6.

All versions are tagged
https://github.com/DiligentGraphics/DiligentCore/tags
Last tag is from 13 days ago.

Ou, great, thanks!
Could you please also tag the 3rd party libs?
Volk, glslang, SPIRV-* and Vulkan-Header

We update third party modules infrequently, usually before releases. They are not related to the Diligent API version. Breaking changes do happen, but they are rare.
You can find the third-party commits from the tagged Diligent Core commit versions.


Newtonsoft.Json version (10.0.1) that ships with PowerShell 6.0 Beta 8 has Concurrent Collections deserialization bug.

Update to Newtonsoft.Json >= 10.0.3 will fix this issue.  Unfortunately, even if a PSCmdlet references the newer version, the version of Newtonsoft.Json that ships with PowerShell 6.0 is loaded instead of the version referenced by the PSCmdlet library because PowerShell 6.0 owns the AppDomain and configuration.
See Azure/azure-powershell#4488 where this issue has been shown to affect Azure PowerShell Core.
See JamesNK/Newtonsoft.Json#1294 where this was first shown to affect Concurrent Collections deserialization in Newtonsoft.Json for .NET Core 2.0.
See Newtonsoft.10.0.3 release notes which identifies that the fix went into Newtonsoft.Json.10.0.3.
Steps to reproduce
Install-Module AzureRM.Netcore
Save-AzureRmContext
Import-AzureRMContext

Expected behavior
Deserialization of a ConcurrentDictionary works in PowerShell 6.0.
Actual behavior
It does not.

Note: it appears as though this change has been made in the repo.  Feel free to close when Beta 9 ships.

Confirmed that building master you get 10.0.3


collect2: error: ld returned 1 exit status

Hi,
I use https://github.com/ekg/wfmash/pull/116 for compliation, but it throw errror
Error messages:
/usr/bin/ld: CMakeFiles/wfmash.dir/src/yeet/yeet_main.cpp.o: in function `dsets::DisjointSets::find(unsigned long) const':
/wfmash/src/common/dset64-gccAtomic.hpp:101: undefined reference to `__sync_bool_compare_and_swap_16'
/usr/bin/ld: /wfmash/src/common/dset64-gccAtomic.hpp:101: undefined reference to `__sync_bool_compare_and_swap_16'
/usr/bin/ld: CMakeFiles/wfmash.dir/src/yeet/yeet_main.cpp.o: in function `dsets::DisjointSets::unite(unsigned long, unsigned long)':
/wfmash/src/common/dset64-gccAtomic.hpp:136: undefined reference to `__sync_bool_compare_and_swap_16'
/usr/bin/ld: CMakeFiles/wfmash.dir/src/yeet/yeet_main.cpp.o: in function `dsets::DisjointSets::find(unsigned long) const':
/wfmash/src/common/dset64-gccAtomic.hpp:101: undefined reference to `__sync_bool_compare_and_swap_16'
/usr/bin/ld: CMakeFiles/wfmash.dir/src/yeet/yeet_main.cpp.o: in function `dsets::DisjointSets::unite(unsigned long, unsigned long)':
/wfmash/src/common/dset64-gccAtomic.hpp:143: undefined reference to `__sync_bool_compare_and_swap_16'
collect2: error: ld returned 1 exit status
gmake[2]: *** [CMakeFiles/wfmash.dir/build.make:136: bin/wfmash] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:156: CMakeFiles/wfmash.dir/all] Error 2
gmake: *** [Makefile:149: all] Error 2

Def files of singluarity.
Without -march=native, it might not work. However, you're probably doing that to be sure you make a portable binary.
I'm looking into a solution. It's not technically necessary to use atomics here, so it should be easy to work around.

-march=native come from the dockerfile of pggb (https://github.com/pangenome/pggb/blob/master/Dockerfile)

@baozg would you please try the branch again? I've just pushed up code that should resolve this issue.

BTW the issue is that the sed script is removing the -march=native and then we don't have access to the special atomic compare and swap for 128bit integers that the lock-free union-find / disjoint sets code uses. But, we don't need the lock-free version because the union find is single threaded.

Thanks. It worked. I test it with the LPA.subset.fa.gz.


#Stage 1 : Using git and github

1. Stage one for using github

[ ] install git
[ ] git initialize
[ ] first commit
[ ] create a github account
[ ] add account into project contact me
[ ] created a new branch with local names
[ ] add yourname.conf file under conf folder
[ ]  make a PR (pull request )

[√]  install git
[√ ] git initialize
[√ ] first commit
[√] create a github account
[√]  add account into project contact me
[√] created a new branch with local names
[√ ] add yourname.conf file under conf folder
[√] make a PR (pull request )

[x]  install git
[x]   git initialize
[x]   first commit
[x]   create a github account
[x]   add account into project contact me
[x]   created a new branch with local names
[x]   add yourname.conf file under conf folder
[x]   make a PR (pull request )

[ ]   install git
[ ]   git initialize
[ ]   first commit
[ ]   create a github account
[ ]   add account into project contact me
[ ]   created a new branch with local names
[ ]   add yourname.conf file under conf folder
[ ]   make a PR (pull request )

[ ] install git
[ ] git initialize
[ ] first commit
[ ] create a github account
[ ] add account into project contact me
[ ] created a new branch with local names
[ ] add yourname.conf file under conf folder
[ ] make a PR (pull request )

[ ]  install git
[ ] git initialize
[ ] first commit
[ ] create a github account
[ ] add account into project contact me
[ ] created a new branch with local names
[ ] add yourname.conf file under conf folder
[ ] make a PR (pull request )

[ ] install git

[ ]  git initialize

[ ] first commit

[ ] create a github account

[ ] add account into project contact me

[ ]  created a new branch with local names

[ ]  add yourname.conf file under conf folder

[ ] make a PR (pull request )


Multi instance unscope handle message

What?
HandleMessage should work for all instance ids
...
Why?
We need this for multiple instance support in beams
...

CC @pusher/mobile

Codecov Report

Merging #135 into master will decrease coverage by 1.46%.
The diff coverage is 89.3%.

@@            Coverage Diff             @@
##           master     #135      +/-   ##
==========================================
- Coverage   91.75%   90.29%   -1.47%
==========================================
  Files          52       56       +4
  Lines        2815     3152     +337
==========================================
+ Hits         2583     2846     +263
- Misses        232      306      +74

Impacted Files
Coverage Δ

Sources/ServerSyncJob.swift
73.68% <0%> (-1.06%)
:arrow_down:

Sources/Persistence/PersistenceConstants.swift
100% <100%> (ø)

Tests/IntegrationTests/ReportEventTypeTests.swift
100% <100%> (ø)
:arrow_up:

...IntegrationTests/MultipleInstanceSupportTest.swift
100% <100%> (ø)

...rationTests/MultipleClassInstanceSupportTest.swift
100% <100%> (ø)

Sources/Persistence/Metadata.swift
100% <100%> (ø)
:arrow_up:

Tests/Persistence/DeviceStateStoreTests.swift
100% <100%> (ø)

...ts/Persistence/InstanceDeviceStateStoreTests.swift
100% <100%> (ø)

...shNotifications/PushNotifications/InstanceId.swift
100% <100%> (ø)

Tests/EventTypeHandlerTests.swift
92.1% <100%> (+0.43%)
:arrow_up:

... and 30 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 e65a6f1...d158127. Read the comment docs.


Community features

Google groups?
Gitter?

https://groups.google.com/forum/#!forum/chevrotain

starting with just google groups for now.


[TreeView] Add customization demo

The new demo is heavily inspired by https://www.react-spring.io/docs/hooks/examples by @drcmda. The difference is that the new demo is accessible :).
Preview: https://deploy-preview-16785--material-ui.netlify.com/components/tree-view/#customized-tree-view

Closes #16784

Perhaps we should refine the core components before working on customisation? It makes more work if we have to update this as a result of component changes...

@mbrookes Your "more work" concern is completely valid. It remembers me when we added a customization demo to the slider in #15478 to later revamp it.
From this past experience, I think that the existence of the customization demo has been a beneficial constraint. It forced us to make sure we would keep customization easy or improve them.

we should refine the core components

Any specific problem you think we should address in the core components?

Any specific problem you think we should address in the core components?

Nothing major, it works well. Let's see what the feedback is like.
However I'm wondering if there's a way to address the need for manual node-ids. It's similar to the id concern in components like tabs. Can we automatically generate sequential ids without too much overhead, and without causing a conflict where there is more than one instance of a component?

@mbrookes So, do you approve the merge of the pull request? Should we wait for #16814?

However I'm wondering if there's a way to address the need for manual node-ids.

This is an interesting prior art: https://github.com/reach/reach-ui/blob/master/packages/auto-id/src/index.js. With this approach (+ the usage of useLayoutEffect), we would have to sacrifice the SSR output to make the nodeId optional. Why not. It could be an acceptable tradeoff. But in practice, does it make a difference? I would expect people to have raw data they hydrate with the TreeView / TreeItme, not like it's done in the demos 🤔.
💡 We could add a demo for raw data hydration :)

I’m not sure we need to auto-generate IDs. It only seems weird in the demos due to the test data. Ideally any data you use should have a way to uniquely identify an element. Auto-generated ids will potentially make it hard to determine what node is selected because all they would get would potentially be an ID

Should we wait for #16814?

Personally I'd rather wait, and have the customisation demo follow (if there's anything in the stripped back changes that will impact this PR), but your call.

OK, I will wait.


API Design

harfbuzz-gem: https://github.com/jslabovitz/harfbuzz-gem
Lua-C interactions:

http://lua-users.org/wiki/UserDataWithPointerExample
http://lua-users.org/wiki/BindingWithMetatableAndClosures
https://john.nachtimwald.com/2014/07/12/wrapping-a-c-library-in-lua/

The example from harfbuzz-gem is interesting. Especially font face functions (it could be more efficient to cache it, instead of creating new face with font data for each word, as it happens now).
And also buffer.guess_segment_properties, which guess script, language and direction from the text. I want to do something similar in Lua callback, which would process the nodes before shaping and set correct language and direction depending on unicode class they belongs. So Latin text in Arabic paragraph wouldn't need to be denoted in some macro which sets correct direction, font and language.

Here is a preliminary sketch of API that I am thinking of in code form. This will evolve, and ofcourse things like features and segment properties will need to be incorporated.
local harfbuzz = require 'harfbuzz'

-- report version
print(harfbuzz.version)

-- list shapers
print(harfbuzz.shapers)

-- initialize blob
local blob = harfbuzz.Blob.new(…)
print(blob:length())

-- initialize buffer
local buf = harfbuzz.Buffer.new(…)
buf:add(…)
print(buf:length())

-- initialize font
local font1 = harfbuzz.Font.new(file,index)
local font2 = harfbuzz.Font.from_blob(blob,index) -- Maybe just use this, and have a from_file method in Blob
print(font1:upem())

-- shape
glyphs = harfbuzz.shape(font, buf, options)

On Wed, Dec 2, 2015 at 1:07 PM, Deepak Jois<EMAIL_ADDRESS>wrote:

Here is a preliminary sketch of API that I am thinking of in code form.
This will evolve, and ofcourse things like features and segment properties
will need to be incorporated.

local harfbuzz = require 'harfbuzz'

-- report versionprint(harfbuzz.version)
-- list shapersprint(harfbuzz.shapers)
-- initialize bloblocal blob = harfbuzz.Blob.new(…)print(blob:length())
-- initialize bufferlocal buf = harfbuzz.Buffer.new(…)
buf:add(…)print(buf:length())
-- initialize fontlocal font1 = harfbuzz.Font.new(file,index)local font2 = harfbuzz.Font.from_blob(blob,index) -- Maybe just use this, and have a from_file method in Blob
Maybe just use something like

font, index = harfbuzz.Font.new(data_string)?

After the initial repo setup, I just finished coding up the first type in the list above, the harfbuzz.Blob, along with tests etc.
LuaRocks packaging is also done. So now it’s just a matter of wrapping all the other Harfbuzz types with Lua bindings. Hopefully, it will be done soon.

I had forgotten about the hb_face_t type in the API design spec above. I have added it as harfbuzz.Face.
local f = harfbuzz.Face.new('fonts/notonastaliq.ttf')
print(f:get_upem())

Not really api related, but I've setup simple test file for LuaLaTeX, see  https://github.com/michal-h21/luaharfbuzz-luatex-test
The output from test string used in Makefile is

I am not sure whether it's correct, given the fact that we don't take direction nor language and script into account yet.

This is pretty accurate. You can easily compare this output with Harfbuzz’s output by running Harfbuzz’s hb-shape tool.
Here is what I ran
hb-view  --font-funcs=ot --shapers=ot  --output-format=png --output-file=out.png  ~/Downloads/NotoNastaliqUrdu-unhinted/NotoNastaliqUrdu-Regular.ttf "یہ"

And here is what I got

Good stuff!

Yeah, it seems that values of x_offset and y_offset can be used
directly in xoffset and yoffset node fields, after dimension
conversion. Width, height and depth are saved by fontloader, so we probably
don't have to set them explicitly?
On Thu, Dec 3, 2015 at 4:40 PM, Deepak Jois<EMAIL_ADDRESS>wrote:

This is pretty accurate. You can easily compare this output with
Harfbuzz’s output by running Harfbuzz’s hb-shape tool.
Here is what I ran
hb-view  --font-funcs=ot --shapers=ot  --output-format=png --output-file=out.png  ~/Downloads/NotoNastaliqUrdu-unhinted/NotoNastaliqUrdu-Regular.ttf "یہ"
And here is what I got
[image: out]
https://cloud.githubusercontent.com/assets/5342/11564997/06b978bc-9a02-11e5-836a-5d4da6a7b13e.png
Good stuff!
—
Reply to this email directly or view it on GitHub
https://github.com/deepakjois/luaharfbuzz/issues/1#issuecomment-161680133
.

The width, height and x/y-bearings come from the glyph extents. We can control whether to return it via some sort of options flag. That way, clients that need it can ask for it, and others can choose not to.

With 932e2f7, the basic API is now complete. Ofcourse there is lots more to be done, like shapers, additional member functions etc. But it is possible to do basic shaping that matches the output of the hb-shape tool. So I am closing this ticket, and tracking API additions in separate tickets.
@michal-h21 Note that the harfbuzz._shape is deprecated now. It is possible to do the same thing using the wrapped API types and the harfbuzz.shape method (see example). Another thing is that the library should to be loaded using require 'harfbuzz' and not require 'luaharfbuzz' (as some of your sample code does). I will keep harfbuzz._shape around for a while, but I hope to remove it as soon as possible. So let me know once you have modified your code.


Web page or docs for language bindings

I am working on a LuaJIT binding to this library.  It would be nice to have a place to put a link to the binding.
https://github.com/wiladams/LJIT2s2n

Added in https://github.com/awslabs/s2n/commit/55c3231b710d037242379de8398e66aba690ffc8 :)
Note, you've not got a license declared for LJIT2s2n yet.


The table has some bugs

NextUI Version
2.4.8
Describe the bug
1.When a switch component exists in the table and showSelectionCheckboxes is true, the single-machine multiple selection box will change the switch state to enabled at the same time.

2.In addition to using nextui's own buttons, using other tags will trigger multiple selections
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
1.Set showSelectionCheckboxes=true
2.Add Switch Component
3.Click checkbox
Expected behavior
1.Clicking the checkbox should not cause the switch state to change.
2.Clicking other custom buttons will not trigger the selection.
Screenshots or Videos
No response
Operating System Version
windown
Browser
Chrome

duplicate - https://github.com/nextui-org/nextui/issues/2959


Update torch_xla r2.3 test for v2.3.0-rc6

Description
Update 2.3 tests to use 2.3.0rc6 image/whl
Tests
WIP test locally
Checklist
Before submitting this PR, please make sure (put X in square brackets):

[x] I have performed a self-review of my code.
[x] I have necessary comments in my code, particularly in hard-to-understand areas.
[x] I have run one-shot tests and provided workload links above if applicable.
[x] I have made or will make corresponding changes to the doc if needed.

Test setup for rc6 is verified. Internal tracker: /b.corp.google.com/issues/331987634


Roll engine 21342863aa60..9e8f8d0889ec (1 commits)

https://github.com/flutter/engine/compare/21342863aa60...9e8f8d0889ec
git log 21342863aa6018af53c09af2a3e7aa432a6cc844..9e8f8d0889ec04b54def0bf917321a27006623b0 --no-merges --oneline
9e8f8d088 Drop android_sdk_downloader in favor of cipd (flutter/engine#8087)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (mklim@google.com), and stop
the roller if necessary.

Auto-roller completed checks. About to merge.


*Added the ability to determine a UIView's view controller without selection the view controller's view delegate.

This is more of a convenience thing in most cases. It does become quite useful when debugging parent and child view controllers and is quite helpful (personally) when messing with apps and injecting FLEX using Cydia Substrate. _viewControllerForAncestor is quite an old method and has been around since iOS 4.2 (November 2010) so it can be assumed that it is as safe _viewDelegate which was introduced in 4.0, I believe. It is, however, more robust in that the view controller can be determined from any view contained within a given view controller.
I left the old method as is, even though this new method should be a suitable replacement. I'll leave such changes up to the main contributors' discretion going forward.

Cool! Thanks for the contribution! Sorry for making you deal with the object explorer class hierarchy/customization mess. I've been meaning to refactor that but just haven't found the time.

I actually have a few other additions/changes that I've implemented that I'm not sure whether or not you're interested in. I'm not sure of their usefulness outside of my main focus - development of tweaks for jailbroken iOS, but I guess they could be useful to some. My modifications currently exist in the form of a tweak and swizzle methods (this was simply to allow me to update the git repository without having to worry about resolving conflicts and whatnot). With some work, I can probably adapt them for use in the main project.
Namely:

Inspecting a class from a string.
Listing the origin of the class (e.g: UIView originates from UIKit.framework).
Overview of protocols inherited by classes.The code itself traverses the class hierarchy and lists protocols inherited by each superclass.
A button to dismiss the keyboard if the explorer window is presented with the keyboard visible.
A controller for viewing all classes loaded by every loaded image.
A controller for viewing the names of all methods for all classes in all images.
A controller for viewing the names of all methods for all classes in a selected image.
A function for simulating a crash.
I also modified some code to allow inspection of UIAlertViews and UIActionSheets (or UIAlertControllers). This is achieved by enabling the FLEX window to be just over the level of such controllers and then to be reduced upon presenting a view controller.

I've got these implemented in a way that would probably cause app rejection if left in a published build. If you'd like any of these changes implemented, let me know and I'll see what I can do to hide them using the methods that you've used throughout FLEX.


Diagnostic page is "empty" if php5-gd is not installed

Detailed description of the problem
I made a new installation of Lyche without installing the dependencies.
Steps to reproduce the issue
Install lychee. But do not install the Debian package php5-gd.
Output of the diagnostics (Settings => Diagnostics)
The page is mostly empty and contains only:
Diagnostics
-----------

I get an error in Apache error.log
PHP Fatal error:  Call to undefined function Diagnostics\\gd_info() in /home/rousseau/Documents/github/Lychee/plugins/Diagnostics/index.php on line 56, referer: https://images.xxx/Lychee/

The diagnostic should be smart enough to detect that php-gd is missing and display this information.
After installing php5-gd I have a filled Diagnostic page.

The diagnostic should be smart enough to detect that php-gd is missing and display this information.

That's correct. PR welcome.


HDDS-7224. Create a new RocksDBCheckpoint Diff utility.

What changes were proposed in this pull request?

Add a RocksDB diff utility. We want to identify the set of SST files that are different between two different RocksDB checkpoints.
Integrate this utility with Ozone

The changes here include basic changes to track differences in SST files across RocksDB compaction. More changes will follow.
What is the link to the Apache
https://issues.apache.org/jira/browse/HDDS-7224
How was this patch tested?
Manual Testing.
Automated testing will be added in subsequent patches.

Find bugs warnings are in UnitTest Class and can be ignored for now.  integration test failures are unrelated.

+1

Thanks @nandakumar131 for the review. I have addressed all the comments and got a green CI.
Pls give a +1 via the "Review changes" button if you feel like the latest changes are good.

Merging this PR as base set of changes for RocksDiff. We will continue to make more changes on top of this.

Thanks @prashantpogde for the original PR. Thanks @nandakumar131 for the review.


new binding and small fix for new flipperzero-firmware v0.85.2

Apologies for missing that this PR was updated. In the meantime the Flipper SDK continues marching on. Superceded by #93.


feat: replace use of enzyme with react-testing-library

added @testing-library/react to graphiql package
added @testing-libary/jest-dom to graphiql package
removed usage of ENZYM env variable in test script in graphiql packaget
removed enzyme adapter setup from enzyme.config.js file and renamed it
to test.config.js
removed use of enzyme and replaced with react-testing-library in all
graphiql package test files and updated tests to use
react-testing-library patterns
moved mockStorage to own file so it can be reused in other test files
and added other methods to it
added a CodeMirror mock to be used in tests
mocked out codemirror addon modules

completes: https://github.com/graphql/graphiql/projects/4#card-30693199
This PR gets rid off all uses of Enzyme in the GraphiQL package and replaces them with @testing-libary/react.

@ryan-m-walker ok so i merge the react 16 upgrade PR so you'll just need to rebase it looks like?

@acao ok, I rebased

@ryan-m-walker nice that was quick, thank you!

oh drat, i just merged the storybooks PR and so another conflict here

@acao no worries, i'll try to fix later tonight

@acao there's a minor change I want to make anyways

@acao should be good now


Milan - Email translation and translated gifs upload

Uploading the email for translation and the translated gifs for the offline codes email
Spanish and Portuguese language


Update scheduling commands for new schema and procedural goals

This PR updates the scheduling goal commands to better reflect the schema changes that separated goal definitions from plans. It also adds support for procedural goals and should only be merged / released when procedural scheduling is released.
Interesting changes:

added venv and .idea to gitignore. I don't know if ignoring venv is recommended, but it did have a lot of changes after installing the cli.
renamed the scheduling subcommand to goals. This is because the new schema separates goal definitions from specifications, so I don't think it makes sense to manage entire specifications like before. I also deleted the batch commands using a spec file for the same reason. It should still be possible to manage goals that way using something like xargs.

Re: "scheduling" vs. "goals", another option would be to refactor the goal management commands into a sub-command (e.g., aerie-cli scheduling goals upload [...]) if we reach a point where we need to segment goal management from any other scheduling constructs in the future. Until then, I'm mostly ambivalent towards either term, and so prefer to avoid unnecessary breaking changes unless there's a stronger case for the name swap.

We don't need to change the subcommand name, it should be fine to still call it scheduling. I only changed it because of the shift from managing specs to managing individual goals.


Delete temporary files suffixed with -wal and -shm

What
When deleting a database from disk, delete all temp files too
How
Delete files suffixed with -wal and -shm which are sqlite temp files for the journalling mode we use.
Testing
No unit tests due to the difficulty in reliably making sure that the files exist before deleting.
Issues
Fixes #272

+1


Remove great_expectations dependency

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (2e59922) to head (5396385).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #458   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           94        94
  Lines         2454      2454
  Branches       215       215
=========================================
  Hits          2454      2454

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


support for Event Service

The event service allows to send/receive custom messages from the micro:bit. It also enables all the built-in blocks like "alarm" , "remote control" etc...
https://lancaster-university.github.io/microbit-docs/ble/profile/#about-the-event-service

Hi @pelikhan,
Do you have time to work on a pull request for this? You can use a similar patter to the lib/*-service.js files.

Closed via #7.


Conflict with the Splashscreen plugin when starting the app offline

Hi,
I'm testing an iOS app, using the cordova-plugin-splashscreen to control when the splashscreen goes away programmatically. The problem is that when the app starts offline I can't hide the asplashscreen, even if I set the configuration to have "AutoHideSplashScreen" to "true", or if I call the hide method:   navigator.splashscreen.hide();
I've tried this both using the default offline page and a custom one.
When I reconnect the device to wifi the splashscreen hides and the remote site is shown correctly.

Hi @pauloya  it sounds like we are not firing an event that the cordova-plugin-splashscreen is looking for.  So for clarity, the splash screen shows instead of the offline page, not on top of it correct?  And it fails if it is set to auto and if it is set to timed delay?
Also, does it fix it if you add this js to the offline page:
window.setTimeout(function () {     navigator.splashscreen.hide(); }, 0);

I'm attaching macOS Safari debugger to the iOS device, and I see that the offline page is there.
My custom offline page loads the cordova.js
<script type="text/javascript" src="cordova.js"></script>
So when I try to call from the console:
navigator.splashscreen.hide();
nothing happens, I tried setting a breakpoint in the splashscreen ObjectiveC code and that doesn't semm to get hit when I call hide()...
I think the issue may be related with the fact that the offline page is not correctly bound with the cordova APIs...

Just to clarify, is there anything different about the offline page that prevents cordova APIs to work correctly? I noticed that when hooking the Safari devtools with the app there are 2 different instances of browser that show up, is this because the offline page is rendered inside the non standard cordova webview?

The reason why we use two different webviews is so that when you go back online, we can just switch webviews and keep you in context.  With iOS we would often get a page refresh where we would loose the session info.
Your assumption is correct, we don't have the cordova environment on the offline page and we should for this scenario.   If you are using the local plugin structure, you should be able to just solve it by hard coding a link to local js files.

I'm sure you know this better than me, but a quick look at the code indicates the offline view is a [UIWebView] which, perhaps doesn't bind access with Cordova even though I include it. [CDVViewController] should work instead.
Also, I think you would need to inject the plugins JS in the offline page for this to work.

Also seeing it. If network status changes after hosted url has been loaded then offline page is shown, but If I start the app while offline then i'm stuck on splashscreen

Hi @pauloya I do remember some issue with offline, splashscreen and cordova APIs.  @f2bo , @DaveVoyles  or @jasonshortphd     did any of you help fix a problem like this?

Looks like since url in <content src="..." /> never loads and we are not loading local files where we can do navigator.splashscreen.hide() there's no event that hides splashscreen,

Hi @pauloya I do remember some issue with offline, splashscreen and cordova APIs. @f2bo , @DaveVoyles or @jasonshortphd did any of you help fix a problem like this?

@boyofgreen Yes. We did fix an issue (https://github.com/manifoldjs/ManifoldCordova/issues/59) with offline in iOS, but it was a different scenario.
@pauloya Just to confirm, do you mean you see the splashscreen instead of the offline page while the device is offline?

We found a workaround for this.

@pauloya Just to confirm, do you mean you see the splashscreen instead of the offline page while the device is offline?

Yes, the problem was that the splashscreen stayed on when the device was offline.
The workaround is to open a local HTML that does a document.location.href = "..."; to redirect to the remote host.
We looked at the splashscreen code and it relies on a pageload event to start any logic of hiding the splashscreen even when using a delay splashscreen. If you use the splashscreen plugin and there is no page load the splashscreen will never go away.
We set up a delay of 10s on the splashscreen and we call navigator.splashcreen.hide() from the hosted web app. It's not the ideal solution because of the delay of 10s when the device is offline, but we can live with it.

This is my work around:
https://github.com/danzel/cordova-plugin-splashscreen/commit/39741188784c5a166c987754173558ce62d283e6


Update Azure Storage swagger descriptions.

Update the descriptions in the Azure Storage swagger file.

This change is

Hi @robinsh, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (robinsh). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

Merged

 https://github.com/Azure/azure-sdk-for-python/pull/851

 https://github.com/Azure/azure-sdk-for-node/pull/1958

 https://github.com/Azure/azure-sdk-for-ruby/pull/539


cleanup: revert changes inside the point-of-sale-app folder to match main

Description

In an effort to bring the source code for the sample application as a first class item we are moving everything inside the point-of-sale-app directory into a directory called src in the root. (https://github.com/GoogleCloudPlatform/point-of-sale/pull/115)
This restructuring is done by branching off of the branch called v2-sql where changes to the code has been made to support mysql
But after doing the restructuring since the changes that were made in v2-sql branch have been copied into the newly created src directory, we must now revert the point-of-sale-app directory to match its state in the main branch.
This ensure, that when we finally merge v2-sql into main the point-of-sale-app would have not changed at all - so everythign that references the repo should work as is
After we do the merge, and update all the content that references point-of-sale-app to point to the new src directory we can safely delete point-of-sale-app

What changes were done:

Just copy everything from the point-of-sale-app directory in main into point-of-sale-app directory

How to ensure its the same as the main branch:
Follow the steps below:
git clone https://github.com/GoogleCloudPlatform/point-of-sale main-br
git clone https://github.com/GoogleCloudPlatform/point-of-sale restructure-br

cd restructure-br
git checkout restructure2

diff -r point-of-sale-app ../main-br/point-of-sale-app

Only output you see should be related to license header change which I had to do to get the CI passing:
diff -r point-of-sale-app/inventory/src/main/java/com/google/abmedge/inventory/dao/InMemoryStoreConnector.java ../main-br/point-of-sale-app/inventory/src/main/java/com/google/abmedge/inventory/dao/InMemoryStoreConnector.java
1c1
< // Copyright 2022 Google LLC
---
> // Copyright 2021 Google LLC
diff -r point-of-sale-app/service-sdk/src/main/java/com/google/abmedge/dto/Payment.java ../main-br/point-of-sale-app/service-sdk/src/main/java/com/google/abmedge/dto/Payment.java
1c1
< // Copyright 2022 Google LLC
---
> // Copyright 2021 Google LLC

:zap:  Two deployments have been created for the Point-of-Sale application. You may access and test them at: :zap:

MySQL DB backed deployment: <IP_ADDRESS>
Embedded H2 DB backed deployment: <IP_ADDRESS>


[5.2] Account for __isset changes in PHP 7

PHP 7 has fixed a bug with __isset which affects both the native isset and empty methods.  This causes specific issues with checking isset or empty on relations in Eloquent.  In PHP 7 checking if a property exists on an unloaded relation, for example isset($this->relation->id) is always returning false because unlike PHP 5.6, PHP 7 is now checking the offset of each attribute before chaining to the next one.  In PHP 5.6 it would eager load the relation without checking the offset.  This change brings back the intended behavior of the core Eloquent model __isset method for PHP 7.
For reference, please check the following link, specifically Nikita Popov's comment (core PHP dev) - https://bugs.php.net/bug.php?id=69659

This is a change in behavior. Checking if the method exists and is a relation is not the same as checking if it is loaded.

@taylorotwell I see no change in behavior when testing it against all the cases I can see. Because right now in PHP 7 if you do isset($this->relation->exists) it will always return false if the relation hasn't been loaded yet, even if that relation does exist in the database.  If you do the same thing in PHP 5.6 isset($this->relation->exists) then it will return true in the exact same situation.  The reason is that PHP 5.6 (and below) had a bug that it would just load in that relation just by checking it.  It would not first check if the object offset existed. So by doing the isset check you are actually loading it in PHP 5.6.  You may have not intended for it to do that, but it works that way in PHP 5.6 (and below)   And it appears that they see this as a bug fix so they won't be reverting it.

@taylorotwell After some more looking into it, this may not be the way to tackle this for the whole community but I'm still not sure the best way.  In fact, my "fix" doesn't even work unless I move the (method_exists($this, $key) && $this->$key instanceof Relation) to the first thing in the return so it can load the relation for the rest of the checks.  I would love your thoughts on this.  For now I will just overload that method in our code as we do a lot of checks for existing relation fields that weren't loaded previously.


Can u pls help me

I am trying to do this but im on windows and i got no idea how docker works

Hi @K0ha1 , it's not necessary to use docker if you just want to build a simple website with LiveTerm - could you specify what you were trying to do?

im just trying to make a github pages website with the terminal style u showcased in the repo

Unfortunately LiveTerm is built as a dynamic template, and AFAIK github pages can only be used to host static pages. I would recommend using vercel/heroku/netlify to deploy your website instead - they are all free for personal usage.
In any case though, Docker usage is not required - it is only intended for those who are familiar with it to facilitate deployments with Docker.


Light mode and theme switcher

Add a light theme and a way to switch between the themes ( via a new property in styleProperties )

hello, so how its going?

This bug only appears on the iOS emulator.
Everything works fine on iPhone.

Implemented. And theme switcher is implemented as well in epub_reader's main branch.


ETTm2 scripts

Hi, I’m trying to run the experiments on ETTm2. I found in this repository the scripts for ETTh2. Are you using the same settings for ETTm2? If not, would you please add the scripts for ETTm2?

Hi, thanks for your interest.
We use the same forecasting setting on ETT datasets. You can change the data_path to ETTm2 directly, and tune p_hidden_dims and p_hidden_layers for different models.


Isolate (/b/s/w/itSs9tBI/dart_fuzzRJYDGA) FFI NO-FP : KBC-CMP-SRC-SLOWPATH-DET-ReleaseIA32 - JIT-DET-DebugX64: !DIVERGENCE! 1.31:90302744 (output)

no output

Seems we are over allocating. Comparing JIT IA32 vs X64, for example, we see the first difference happening in x0.foo0_1 which causes actual output differences.
< Exhausted heap space, trying to allocate<PHONE_NUMBER> bytes.
---
> Exhausted heap space, trying to allocate<PHONE_NUMBER> bytes.

Two improvements

We need to use this code (note the "lib"):

 ffi.DynamicLibrary ffiTestFunctions =
    ffi.DynamicLibrary.open("libffi_test_functions.so");

also, I propose to add the following lines to main() in every generated fuzz program when ffi is enabled

// The following throws an uncaught exception if the ffi library is not found.
if (ffiTestFunctions == null)
    print('Did not load ffi test functions');


Add static analysis

Added static analysis to Orchard.Core project by:

SonarAnalyzer
StyleCop
Analysis is disabled by default. Run msbuild with StaticAnalysisEnabled parameter set to true in order to enable code analysis during build

msbuild /p:StaticAnalysisEnabled=true Orchard.sln

@IronBy,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla2.dotnetfoundation.org.
It will cover your contributions to all .NET Foundation-managed open source projects.
Thanks,
.NET Foundation Pull Request Bot


'Buienradar' icon set

Would it be possible to enhance this component with icons linked to the Dutch 'Buienradar' integration?
I am using sensors from Buienradar, which works well after some tweaking and finding out which sensor to use.
Only the icons aren't working well enough. Some icons do, but thats because they happen to have the same name as one of the other 3 integrate icon sets (openweathermap, climacell, darksky).
I know a lot of Dutch people use Home-Assistant, and it would be great that we can use the dutch information provided by Buienradar.
While searching thru the files on this repo, I saw the 3 files where the right names for the different integrations where put.
I am willing to help finding the correct names for the different weather conditions.
I am using this integration: https://www.home-assistant.io/integrations/sensor.buienradar/

Of course it's possible.
Give me some time to implement it.
Thanks
Renato

Of course, take the time.
In the meantime, I found the names used by the 'buienradar' integration.
I tried linking a possible icon used by your integration to most of the conditions. Couldn't find a matching icon for 'hail' and 'exceptional'. Also couldn't find one for lighting-rain, might aswell be 'thunder' icon too.

Name used by Buienradar integration  --  Possible icon

cloudy  -- cloudy
fog  --  fog
hail  --  ?? Could be: 'rain-and-sleet-mix' or 'sleet'
lightning  --  thunder
lightning-rainy  --  https://cdn.buienradar.nl/resources/images/icons/weather/116x116/S.png
partlycloudy  --  cloudy-day-3
pouring  --  rainy-6
rainy  --  rainy-3
snowy  --  snowy-4
snowy-rainy  --  rain-and-snow-mix
sunny  --  day
windy  --  wind
windy-variant  --  wind
exceptional  --  ?? Could be: 'severe-thunderstorm' or 'tropical-storm'

Hi, what do you think about the following map?
 Seems like every line is starting with the name used by the 'Buienradar' integration, and then followed by the icon name? Is that right?
If so, I am not sure if all names you used are also provided by the 'Buienradar' integration, but could be me.

I used the conditions name found here: https://www.home-assistant.io/integrations/sensor.buienradar/
It is right?

Didn't see that the conditions where stated on that page, I got them from a part of the code from the github repo of that integration.
Guess your code looks good then, small type error (listed below).
I think it's worth a try, the next couple of days are mixed weather conditions of sun, rain, heavy rain, thunderstorms etc. So good to test it out.
 It's in the version just released. Try if it works.
Thanks,
Renato

Thank you. Seems all the time I was answering in the wrong issue thread :/
So, just trying it again.
The icons are working for the forercast. But the icon for the current condition is not showing up.
I am getting these errors in browser console:
ha-card-weather-conditions.js:381 {card_config: {…}}
3:1 GET https://<PRIVATE_URL>/local/community/ha-card-weather-conditions/icons/static/undefined.svg 404
ha-card-weather-conditions.js:270 Icons issue. States: icons_model=buienradar - isDay=true - condition: na.

The last line is popping up multiple time a second and keeps going on

I'm having the exact same problem. It started after i had to switch the base url. There's also a warning in the devtools "use HACS path to retrieve icons". I don't know who this is adressed at. It's for all icons, not only partly cloudy. Funny enough the glyphs for moon phase work fine and didn't before.


[WIP] SOLR-17040 UpdateLog refactor to use read/write locks

https://issues.apache.org/jira/browse/SOLR-17040

Description
PR for review/discussion. see Jira for current benchmark results.
Solution
Please provide a short description of the approach taken to implement your solution.
Tests
Please describe the tests you've developed or run to confirm this patch implements the feature or solves the problem.
Checklist
Please review the following and check all that apply:

[ ] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[ ] I have created a Jira issue and added the issue ID to my pull request title.
[ ] I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
[ ] I have developed this patch against the main branch.
[ ] I have run ./gradlew check.
[ ] I have added tests for my changes.
[ ] I have added documentation for the Reference Guide

closing. benchmarks do not show improvements on this change.


Uptate the documentation to use lazy API a bit more

The user guide still uses eager api in most of its code snippets. This PR moves a few  straightforward examples to lazy API
Also removed one  occasion  of setting isVisible on a non consumeable configuration
Contributor Checklist

[ ] Review Contribution Guidelines.
[ ] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[ ] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[ ] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team.
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective.
[ ] Provide unit tests (under <subproject>/src/test) to verify logic.
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes.
[ ] Ensure that tests pass sanity check: ./gradlew sanityCheck.
[ ] Ensure that tests pass locally: ./gradlew <changed-subproject>:quickTest.

Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

Thank you for your contribution!

The team responsible for this area will need to confirm those changes.
While moving from all to configureEach is most likely fine, using ConfigurationContainer.register may not be that interesting or even desired.

Thanks, I've added this to our list for 8.13


Fix Issue 430

This implementation should fix #430 and all actuator related endoints.
Breaking Change: all actuator endpoints are moved to /actuator!

Coverage remained the same at 66.631% when pulling 5826c42780df4b43e003d4148946f2c3c6ce8b74 on fix_issue_430 into d89c599f75b52c94ff418c76244fb3f0f5d05889 on master.


After notification sent -> no more voice recognition?

Hi, i try to use notifications to control other modules with the "smart mirror" hotword. He does send the notification correctly, but after that, he doesn't go back into idle mode. Do i have to set the voice recognition back after i received the notification manually?
0|MagicMir | [ASSTNT] Snowboy Activated
0|MagicMir | [ASSTNT] Command recognized: main page
0|MagicMir | { text: 'Here is the main page',
0|MagicMir |   option: { language: 'en-US', useAlert: false },
0|MagicMir |   originalCommand: 'main page' }
0|MagicMir | [ASSTNT] Speak:  Here is the main page
HERE IT STUCKS - other modules are still running.

	getCommands: function(commander) {
	  commander.add({
	    command: 'news page',
	    description: 'You can change to news page with this command.',
	    callback: 'command_showNewsPage'
	  })
	  commander.add({
	    command: 'main page',
	    description: 'You can change to main page with this command.',
	    callback: 'command_showMainPage'
	  })
	},

	command_showNewsPage: function(command, handler) {
	  this.showNewsPage()
	  var option = {
	    language: 'en-US',
	    useAlert: false
	  }
	  handler.response("Here is the news page!", option)
	},

	showNewsPage: function() {
	    this.curPage = 1;
		this.updatePages();
	},

	command_showMainPage: function(command, handler) {
	  this.showMainPage()
	  var option = {
	    language: 'en-US',
	    useAlert: false
	  }
	  handler.response("Here is the main page", option)
	},

	showMainPage: function() {
	    this.curPage = 0;
		this.updatePages();
	},

Currently I have no idea about what's wrong. It looks not bad in your code.
Test defaults command like 'list all commands'. Does it work?

yes, there is no problem. Tried it also on the default news app... it stucks there also...
	getCommands: function(commander) {
	  commander.add({
	    command: 'news more details',
	    description: 'You can show the actual article in full size.',
	    callback: 'command_ArticleMoreDetails'
	  })
	  commander.add({
	    command: 'news less details',
	    description: 'You can hide the actual article from full size.',
	    callback: 'command_ArticleLessDetails'
	  })
	},

	command_ArticleMoreDetails: function(command, handler) {
	  this.showArticleDetails()
	  var option = {
	    language: 'en-US',
	    useAlert: false
	  }
	  handler.response("Ok, here is the article", option)
	},
	command_ArticleLessDetails: function(command, handler) {
	  this.hideArticleDetails()
	  var option = {
	    language: 'en-US',
	    useAlert: false
	  }
	  handler.response("Ok, hide article!", option)
	},

	showArticleDetails: function() {
			this.config.showDescription = !this.config.showDescription;
			this.config.showFullArticle = !this.config.showDescription;
			clearInterval(timer);
			timer = null;
			Log.info(this.name + " - showing " + this.config.showDescription ? "article description" : "full article");
			this.updateDom(100);
	},
	hideArticleDetails: function() {
			this.resetDescrOrFullArticleAndTimer();
			Log.info(this.name + " - showing only article titles again");
			this.updateDom(100);
	},

On "list all commands" is no issue...

If you got some front-end error from other modules when MM is activating, the error might prevent MMM-Assistant's working.
Check with npm start dev.

I allready checked with pm2 log mm no error.
on ppm start dev also no error.
[ASSTNT] Snowboy Activated
[ASSTNT] Command recognized: news page
{ text: 'Here is the news page!',
  option: { language: 'en-US', useAlert: false },
  originalCommand: 'news page' }
[ASSTNT] Speak:  Here is the news page!

It stuck with the "speech logo" (the red one with the "messenger" sing insight)

IDK, but could it be that your notification is not unique? (Other modules or even MM core using the same?)


Request to add a fingerprint for ABL ZigBee Switch

This is a new project for Samsung LED.
It's a ZigBee Switch for ABL(Juno Connect) and the model no(VID). is ABL-LIGHT-Z-401.
The capabilities are 'switch', 'switchlevel'.
Please review it

 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.


Merge NCMP code into main

Signed-off-by: Wood<EMAIL_ADDRESS>

[ ] Function Documentation as docstrings within the function definition. NA
[ ] Examples demonstrating major functionality, which runs successfully locally.

Functionality

[ ] Installation: Installation or build of the code succeeds.
[ ] Functionality: Any functional claims of new code have been confirmed.
[ ] Automated tests: Unit tests cover essential functions for a reasonable range
of inputs and conditions. All tests pass on your local machine.
[ ] Packaging guidelines: New code conforms to the project contribution
guidelines.

Final approval (post-review)
The author has responded to my review and made changes to my satisfaction.

[ ] I recommend merging this request.

 The comment says round/truncate, but actually you truncate. Why not round? Is this so 59.5 to 59.99 month olds aren't included?
Line 27: SDG 2-1-1 isn't limited to under 5s.Does this code contribute to that indicator? If not, do we also want to write code for that?
you overwrite the same dataframe multiple times (clean_NCMP) - if someone runs the code in one go, they will be less likely to see errors, than if you created a new dataframe each time, or used a pipe.
I don't think this is worth fixing as it is just for the suppressed data, but worth bearing in mind. Dropping over 5s also isn't cleaning the data as such so this could maybe be named differently to make it clearer.
line 31 - could give this check to a variable and print or save a warning if it doesn't give the expected result
line 33 - again we may want to produce a report that would tell us how many rows we have lost from cleaning, so could save this as a value
line 34 - This calculation shouldnt include the loss of under 5s as this isnt data cleaning as such
line 39 - Could be improived by using case_when
lines 39 to 50 works fine but could be written more efficiently as below. Note that I changed the name of the df and the variable to accurately reflect what it now is so if you use this, the anthro_prevalence arguments would need to be updated

# create a lookup dataframe with the old column and a new set of values that will replace them,
#   then join to the original data
decile_to_quintile <- data.frame(schoolindexofmultipledepriv = c(1:10), school_imd_quintile = c(rep(1:5, each = 2)))
NCMP_with_quintiles <- merge(clean_NCMP, decile_to_quintile, by = "schoolindexofmultipledepriv" )
unique(NCMP_with_quintiles$school_imd_quintile)

line 56-63 - I get an error because schoolgovernmentofficeregion has been read in as a factor for me. Probably safest to explicitly change all the factors to characters

line 58 - apparently it is good practice to put TRUE rather than T (as T and F can refer to other things so bugs can be introduced). Not an issue for this code, but thought it was worth mentioning.

I would be inclined to do write.csv immediately after running the anthro_prevalence function. That way there is less chance of something going wrong when they run the script, and we also keep all the data (you never know, we might want to look at those dropped rows and columns at some point). You can then write a separate script for us to run once we get the data in.

Before I wrote the point above I had gone a bit further, but haven't gone to the end because I think everything from here on should be in a new script that also puts the data into the format we want for our csv file.

line 72 - not needed as you have a column select a couple of lines further on

line 75 - na.omit removes rows where there are any NAs. Do you want to do this, or just remove rows that only contain NAs. If the latter you can do this: test[rowSums(is.na(test)) != ncol(test), ]

Thanks Emma!
Very useful comments and well spotted improvements - I will integrate most of these.
I create the csv with the final data extracted as will be needed for the publication because PHE will be delivering this directly with the ad-hoc publication - we won't actually be sent a file "behind the scenes". Also, the line numbers you refer to after the decile_to_quintile code suggestion match references assuming that code has been used to replace the original, so I couldn't quite place it correctly to align afterwards, but it's kind of obvious which lines you refer to.
Now, I assume I should close close this pull request until the code is updated in line with the comments?

I create the csv with the final data extracted as will be needed for the publication because PHE will be delivering this directly with the ad-hoc publication - we won't actually be sent a file "behind the scenes".

I will have a look at your code post long-function again today then. I think there are a couple of things I would do differently but not sure whether your way or mine is better for our purposes. Maybe I'll write the code I would do quickly then we can discuss it?

I'm happy to do a new version that integrates your suggestions, test it, and then we can discuss it in a call? Not likely to get round to it today though....
In terms of this pull request.. maybe it could be merged - because this one here has no changes and is simply a copy of the original with comments, right? Your main currently doesn't have the NCMP folder at all, and this is a request to include it in its current form. So maybe it should be merged, and then I can create a branch of it (from main) with the proposed changes integrated?
Alternatively, can work from the NCMP_2-1-1_and_2-2-1 branch itself (i.e. create branches from that branch) and after it's finalised merge it into your main, in which case this pull request can be closed for now.

@ANikolova22  It just occurred to me - where does year come into this? Will they need to run the code on each year separately?

@ANikolova22 It just occurred to me - where does year come into this? Will they need to run the code on each year separately?
Edit: just realised the answer is yes (it is going to take soooo long to run!)

Yes... I could have wrapped it all in one function and have year as an argument, but I think this would be a disaster... also because they couldn't possibly shove all the data in a single csv file, it's millions of rows just for one year. So yes, just hoping that they have fast machines. If it works like mine does, it should be around 30 mins per year, more or less...

Ok, I updated the code based on comments (hope I didn't miss anything), but only the WHO_code_test.R script. Haven't touched the WHO_code_toRUN.R yet, as it should reflect the final changes we agree on.
The final csv output was exactly the same (good news!) and it took again exactly 26 minutes to run

Ok, I updated the code based on comments (hope I didn't miss anything), but only the WHO_code_test.R script. Haven't touched the WHO_code_toRUN.R yet, as it should reflect the final changes we agree on.
The final csv output was exactly the same (good news!) and it took again exactly 26 minutes to run

@ANikolova22 Perfect :) I can't see anything you have missed. I have made a couple more queries/suggestions just now using the add comment functionality in the code. Which way do you prefer? I think it is worth switching away from setwd() as it assumes stuff about the file setup, and makes it more complex than necessary. I also think putting all the configs at the top will make it easier. The rest are less important :)

Yes, I agree with all the points and will make another update with these taken into account. Looks like it may be the final one, so I will reflect those changes in the WHO_code_toRUN.R, which is the final version for delivery.
I like the in-code comments because it kind of opens individual discussions for each specific intervention, though in the end the comment page may look very cluttered. I'll see what happens if I click in "Resolved conversation" and if it makes it look cleaner...
Thanks again for reviewing!

Okay! I have edited the test file and the final "toRUN", plus some small additions to the readme. Running it produced the same output, so it's all good. Hopefully ready for merge now and I will send the main file to PHE. I will check with them that the raw data doesn't have any suppressions or missing values.


SLinkedList bug

when code write is
var s = new SLinkedList();
foreach (var item in s)
{
Console.WriteLine(item);
}
function MoveNext  _current is null
public bool MoveNext()
{
_current = _current.Next;
 return (this._current != null);

}

What exactly bug fixing is needed?


Bad Release (1.2.370) on PyPI

QwikContext
The 1.2.370 on PyPI does not contain what's tagged in 1.2.370-master or 1.2.730-devel.  Specifically we noticed that the function update_from_repo_url is missing from iDRACUpdate.py.
Details
We're trying to use the iDRACUpdate's update_from_repo_url function, however when you pip install omsdk and it installs via the PyPI package, that function is not there (even though it is present in the git tag).
This leads me to believe that it was a bad release to PyPI.
Repro Steps
Adjust the grep command as needed depending on where your packages are installed to.
pip install omsdk
grep update_from_repo_url ~/.pyenv/versions/2.7.15/envs/testvenv/lib/python2.7/site-packages/omdrivers/lifecycle/iDRAC/iDRACUpdate.py

Notice that there is no result.

git clone -b v1.2.370-devel https://github.com/dell/omsdk.git
cd omsdk
sh build.sh 1.2 370
cd dist
pip install omsdk-1.2.370-py2.py3-none-any.whl omdrivers-1.2.370-py2.py3-none-any.whl
grep update_from_repo_url ~/.pyenv/versions/2.7.15/envs/cyclops-ansible/lib/python2.7/site-packages/omdrivers/lifecycle/iDRAC/iDRACUpdate.py

Notice how the function is there.

Impact
What areas, configurations and use cases does this issue impact
Is there a data loss? No
NeedBy
We would just like a properly working release pushed to PyPI.
Severity
Low

1.2.370 on PyPi maps to latest "Master" release of omsdk ie 1.2.370_master. Where as update_from_repo_url is an recent devel changes as part of 1.2.387


audit logs

adds support for all audit log stuff (that i know of)
only partially tested (everything besides options and changes)
fixed a small typo in the route path too

thank


Community Discourse

Over the years, we've discussed and deliberated on the best approach for providing a way for community members to engage with one another.
Currently, Discord is the status quo. However, we did enable GitHub Discussions a while back and Matrix, along with others, have been suggested to adopt a more open source platform.
The Discord server provides real-time chat for anyone that wants to be involved, but is rather lacking in asynchronous features, such as a forum.
Do we need async? I believe so, this would enable people to request help with problems that stand more chance of getting a response, jobs forums, sharing demo videos for show and tell; and much more.
However, I do believe we continue to need real time too.
Let us know what you would prefer and what you want from the community and this will allow us to craft a CDR to improve the status quo.

I totally agree, that we're missing the async part in the community. I imagine having a forum like in the old days, which also helps in "timezone independent communication". Two tools I think would be worth a try:

https://www.discourse.org/ - "battleproofen" and well known
https://flarum.org/ - we would have to try it out

While I agree that an 'async by default' channel (aka forum) might be needed/wanted, I would like to keep Discord along with it.
Subjectively, Discord works in both, sync/async ways for me, and while I get that this isn't the case for everyone, I value the (rare) real-time chats on Discord most about the community.

I would keep Discord, but drop all channels except

the text channel #general
the voice channel #coffee-bar

When people need ad-hoc help, they can create a forum post and link it in #general or we use a thread in Discord to discuss things in detail.

We could probably get an integration between forum <-> Discord up and running which posts those links automagically, right? That'd be a neat solution, imo.

Closing as per #127


misc: drop testdir replacement in skeleton_config

The str.format is not used anymore for ceph.conf.template
Signed-off-by: Kyr Shatskyy<EMAIL_ADDRESS>

@susebot run deploy


Render time styles overrides over props

Based on the conversation in #6130 to a achieve an ethereal level of override possibilities, we need a way to override any css rule over the props. Its different from overrides and variants in the theme, because those need to be defined in advance and are considered as a predefined standard cases.
This issue is about an override target on just one specific element for the case it needs to be very unique and you don't want/can't predefine its styles in advance.
Example
  const theme = {
    root: X,
    label: Y
  }
  <Button theme={theme}>My Special button</Button>

Now what is X and Y? As of right now, we can't render dynamic styles efficiently, so right now we can go for just a classes map:
  const theme = {
    root: 'your-root-override-class',
    label: 'your-label-override-class'
  }
  <Button theme={theme}>My Special button</Button>

Another option would be available once dynamic sheets lands in JSS, so you can pass styles objects, like the theme:
  const theme = {
    root: {
      borderRadius: 0
    },
    label: {
      color: 'red'
    }
  }
  <Button theme={theme}>My Special button</Button>

And finally we still have inline styles option, which is of course not to recommend, but yet available, it could look basically the same like the previous example, just attached to the node using style instead of className.

I have given more thought on that issue, I'm having a hard time finding a situation where we might need it. We can achieve something very similar with the classes property. I believe that the classes solution offers more flexibility.
I'm removing it from the v1.0.0-beta milestone. I'm even tempted to close it.

Now what is X and Y? As of right now, we can't render dynamic styles efficiently, so right now we can go for just a classes map:

I have missed that part, that's the solution we have picked up. So we are good to close 😄 .


Code section not fully displayed

Description
in the code section of this article, only class and method definition lines and comments are visible. The code inside the definitions are not visible, and there is no white spaces in the places of the missing code. This bug is observed on Chrome on an Android phone.
(Optional:) Please add any files, screenshots, or other information here.
No response
(Required) What is this issue most closely related to? Select one.
Other
Internal issue ID
3b9d4105-5d36-470f-8ba6-54178ceac3c2
Paper URL
https://arxiv.org/html/2407.04620v1
Browser
Chrome/<IP_ADDRESS>
Device Type
Android

(a screenshot of the bug

seems like text color is causing the problem


Develop upstream sync 230123

https://github.com/ROCmSoftwarePlatform/frameworks-internal/issues/3430

ERROR: /workspace/tensorflow/compiler/xla/mlir_hlo/BUILD:1931:11: Compiling tensorflow/compiler/xla/mlir_hlo/tools/mlir_interpreter/framework/registration.cc failed: (Exit 1): gcc failed: error executing command

something happened in mlir to hlo...

retest Ubuntu-GPU-single please!

I suggeste we can disable device_tracer_test, as iRohit is already investiging it and it's getting more and more frequent (my previous 2 PRs are failed many times with no pass).

I suggeste we can disable device_tracer_test, as Rohit is already investiging it and it's getting more and more frequent (my previous 2 PRs are failed many times with no pass).

Agreed. Let's disable that one for now

https://github.com/ROCmSoftwarePlatform/frameworks-internal/issues/3430


No add new plugin option in the wordpress admin page.

I need to install an SSO plugin.
But there is no option for this.
That is strange.
Maybe our WordPress is not the full version?

@mhassany Is there any idea for this bug? thanks!

there may be a problem with the wordpress or the account you used to login.
as described here, there should be an option in the side menu for installing new plugins.
first double check your username, then your wordpress user management section.


Keep views in sync during wallet lock/unlock

After the routing refactor, we need to address keeping the full page and popup views in sync when the wallet is locked and unlocked. It was previously handled with renders, but now we need to use proper messaging via the background script to keep the views in sync.

@Eshwari007 originally reported in #2142

Similar behavior but when the popup wallet is unlocked the windows wallet is yet locked despite being refreshed.
Only upon inputting the password, the windows wallet was visible with all the accounts
Perhaps this is the intended behavior but we should ensure for the locking and unlocking process should be consistent amid the popup wallet and windows.


[QuickAccent] Set selection when character change

Summary of the Pull Request
Set selection to first character when the key change.
Repro on issue.

PR Checklist

[x] Closes: #22421
[ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

Trigger QuickAccent
Trigger QuickAccent with another character
First character of the list should be selected

@davidegiacometti looks good and works nice. Thank you for your contribution!


Fixes settings

#342 in part added settings as a kwarg to be passed to the initial DataChain, but it wasn't working in some cases due to not being passed everywhere.

Not sure why the example test fails for examples/get_started/json-csv-reader.py. That example runs locally for me and the test passes if I force it to run locally. @mattseddon @iterative/datachain Any ideas?

Not sure why the example test fails for examples/get_started/json-csv-reader.py. That example runs locally for me and the test passes if I force it to run locally. @mattseddon @iterative/datachain Any ideas?
Edit: There is also a Studio test failure but it appears to be unrelated since I get the same error on main.

It looks like there is an error with datamodel-code-generator (new version released 3 days ago).
By running the code, we end up with TypeError: Cannot recognize type ForwardRef('List[str]'). Extracting List[str] from the ForwardRef doesn't help as we don't support List in PYTHON_TO_SQL.
Do you know the history of why we are using datamodel-codegen? Looks suspect.

Not sure why the example test fails for examples/get_started/json-csv-reader.py. That example runs locally for me and the test passes if I force it to run locally. @mattseddon @iterative/datachain Any ideas?
Edit: There is also a Studio test failure but it appears to be unrelated since I get the same error on main.

It looks like there is an error with datamodel-code-generator (new version released 3 days ago).
By running the code, we end up with TypeError: Cannot recognize type ForwardRef('List[str]'). Extracting List[str] from the ForwardRef doesn't help as we don't support List in PYTHON_TO_SQL.
Do you know the history of why we are using datamodel-codegen? Looks suspect.

More likely:


Provide a way to set the (min-)widths of the Filter sub-components

Now that Button min-width isn't used, we should provide a way to manipulate the button widths otherwise host projects will not be able to align things properly.

See: https://www.flowdock.com/app/rulemotion/resin-frontend/threads/wbkfrXc517e3Zx6harkn59v_u7b

I suggest that we add an optional property on the HOC that allows you to set properties on the sub-components.
<Filters
    ...
    topLeftButtonProps={{ w: 30, mr: 20 }} />

:+1:
Will that be typed as StyledSystemProps or do we need something broader?

I think we'd just use ButtonProps here


Data loaded upon startup

The data loaded upon startup at the moment only consists of 4 users, which is a little sparse.
In addition, all default "groups" must be changed to read like groups instead of tags.

For first point, it makes sense for our app to have little to no users at the startup? Unless this is a special requirement for testing


Custom week starts

Live example here.
I've been thinking about the moment + locale feature. The calendar was definitely opinionated before since it used isoWeek, and is technically less opinionated now (since people can just override the 'en' locale in the global scope), but if people wanted to use locales and custom weekStarts they would experience unexpected behavior. I honestly haven't gone over the locales code at all, so I don't really know how to leverage it. I'm guessing it doesn't matter much at this point, maybe just something to keep in mind for the future.
Also, tests, need tests...

Change the weekStart prop:
0 for Sunday
1 for Monday
2 for Tuesday
3 for Wednesday
4 for Thursday
5 for Friday
6 for Saturday

This PR is out of date. Lets fix this so we can see if this is merge-able or not.

I rebased

Nice, what about some tests?

Sorry, I don't have time right now to set up any tests.

I've had to make some changes since the whole example structure got changed, but you PR has been merged! Thank you so much for your effort!

(BTW: I postponed the tests because I have trouble running them locally, there's something strange going on with node/iojs/jest/some other packages, which is really frustrating. I'm looking into alternatives for Jest)


AudioKit shows "HeyMic" (Bluetooth mic) as output device.

I'm not able to connect a bluetooth mic as the input to my App which uses AudioKit. My goal is to capture audio through a bluetooth mic and play it back on a bluetooth speaker. How can I go about it?

This is a question and not an issue, probably better suited for StackOverflow.


Fixed package names

HI Patrice,
I've found some disalighment in the package names, which I've corrected it.
Cheers
Luca

Hi! The idea is not to have a org.grobid.ner package, similarly as for grobid-quantities and the other grobid "extensions". Is it creating a problem?

OK, there is no problem. In the grobid-ner directory structure there is a 'ner' directory. To maintain the
consistency it should be removed, in order to match the package directive.

Ah yes now I remember - you have already told me ;)
In grobid-quantities, we modified the directory structure, not the package names. I guess we should do the same here (and I will have to update other grobid additional stuff similarly).

👍


BETA: No battery in energy view -> calculation for home energy wrong

Hi there,
I have set up the power wheel card with the beta battery function.
In power view everything is fine.
In energy view, a value for battery "movement" (prod/cons) is not used in calculation. Therefore the value for home energy is wrong.
It has to be: "calculated value" - "battery movement"
Thanks a lot.
Yours, Michael

The battery is implemented in the power view only for the moment. So I interpret the issue you are reporting as a feature request to implement the battery to the energy view. But then you have to be able to supply input parameters to the card for that. Which candidate input sensors for energy (in kWh) to you have present for the battery?


Parse & recognize --extract Foo.,Bar.Baz.

This would significantly improve the usability and we could replace the myriad of --extract options with a single one.

This can be achieved by:

copy/pasting the parser at https://github.com/FStarLang/kremlin/blob/master/parser/parser.mly#L40
copy/pasting the first few lines of https://github.com/FStarLang/kremlin/blob/master/src/Bundle.ml#L4
copy/pasting the first function o f https://github.com/FStarLang/kremlin/blob/master/src/Bundles.ml#L11


installation

Make the application installable in every device (Microsoft, MacOs, Unix).

@SohamG934 I cannot choose you to solve this issue, until  you accept to collaborate to the project. Is the email sent to your inbox?

sent it :)

@SohamG934 You can solve if you want. thanks


Transcoding to Wav file causes problems with Duration

Version information

fluent-ffmpeg version: 2.1.2
ffmpeg version: 4.1
OS: Mac OS Mojave

Code to reproduce
[Node.js] Codeconvert to wav
const ffmpeg = require('fluent-ffmpeg');
const ffmpeg_static = require('ffmpeg-static');

const command = ffmpeg()
  .setFfmpegPath(ffmpeg_static.path)
  .input(inputStream)
  .outputOptions('-f wav')
  .outputOptions('-ac 1')
  .outputOptions('-ar 11025')

[Command line] convert to wav
$ ffmpeg -i input -ac 1 -ar 11025 output.wav

Expected results
I am sending this to external API.
I thought this would work.
It worked on the Wav file generated on the command line.
Observed results
But it did not work.
Although it is possible to play sound without problems. . .
The result of executing the wav file generated by fluent-ffmpeg with the ffmpeg command.
ffmpeg version 4.1 Copyright (c) 2000-2018 the FFmpeg developers
  built with Apple LLVM version 10.0.0 (clang-1<IP_ADDRESS>)
  configuration: --prefix=/usr/local/Cellar/ffmpeg/4.1 --enable-shared --enable-pthreads --enable-version3 --enable-hardcoded-tables --enable-avresample --cc=clang --host-cflags= --host-ldflags= --enable-ffplay --enable-gpl --enable-libmp3lame --enable-libopus --enable-libsnappy --enable-libtheora --enable-libvorbis --enable-libvpx --enable-libx264 --enable-libx265 --enable-libxvid --enable-lzma --enable-opencl --enable-videotoolbox
  libavutil      56. 22.100 / 56. 22.100
  libavcodec     58. 35.100 / 58. 35.100
  libavformat    58. 20.100 / 58. 20.100
  libavdevice    58.  5.100 / 58.  5.100
  libavfilter     7. 40.101 /  7. 40.101
  libavresample   4.  0.  0 /  4.  0.  0
  libswscale      5.  3.100 /  5.  3.100
  libswresample   3.  3.100 /  3.  3.100
  libpostproc    55.  3.100 / 55.  3.100
[wav @ 0x7fec37004a00] Ignoring maximum wav data size, file may be invalid
[wav @ 0x7fec37004a00] Estimating duration from bitrate, this may be inaccurate
Guessed Channel Layout for Input Stream #0.0 : mono
Input #0, wav, from 'output.wav':
  Metadata:
    encoder         : Lavf58.20.100
  Duration: 00:00:02.04, bitrate: 176 kb/s
    Stream #0:0: Audio: pcm_s16le ([1][0][0][0] / 0x0001), 11025 Hz, mono, s16, 176 kb/s
At least one output file must be specified

I think this output is a problem.
This is not output in the wav file generated by the ffmpeg command.
[wav @ 0x7fec37004a00] Ignoring maximum wav data size, file may be invalid
[wav @ 0x7fec37004a00] Estimating duration from bitrate, this may be inaccurate

The wav file generated by fluent-ffmpeg is displayed on Mac's Finder and Duration becomes 0.
I think this also represents a problem.
As with command line execution, what is necessary to run with fluent-ffmpeg?

I found a pattern that works
const command = ffmpeg()
  .setFfmpegPath(ffmpeg_static.path)
  .input(inputStream)
  .outputOptions('-f wav')
  .outputOptions('-ac 1')
  .outputOptions('-ar 11025')

@gvsro
first line says that stream is Buffer
why stream can be an argument  in ffmpeg.pipe(stream, { end: true}) ?
and this not works for me...

@345ml This works for me.
stream = Buffer.alloc(0)
let mediaURL = '../../../Downloads/bbb_sunflower_1080p_60fps_normal.mp4'
ffmpeg(mediaURL)
  .outputOptions('-f wav')
  .outputOptions('-ac 1')
  .outputOptions('-ar 11025')
  .pipe(stream, { end: true })
  .on('data', function(chunk) {
      console.log('I got a Chunk')
      stream = Buffer.concat([stream, chunk])
  })
  .on('end', () => {
      console.log(stream.length)
  });

I try this but still got problems with duration
const data = [];
ffmpeg(amr_audio_url)
  .format('wav')
  .audioFrequency(16000)
  .audioChannels(1)
  .pipe()
  .on('data', (chunk) => {
    data.push(chunk);
  })
  .on('end', () => {
    const wav = Buffer.concat(data)
    // if I fs.writeFileSync this as output.wav, it can be played correctly
    // but duration is missing (00:00, mismatch with original one)
  });

I tried to compare the 2 files with https://www.npmjs.com/package/wavefile.
It looks the difference is the chunksize.
I didn't find a fix yet.
It does sound reasonable that ffmpeg cannot set the chunk size when streaming.

I found an app called "RIFFVIEW", The missing data is "waveform data".
Check the note at the end of this section.
ffmpeg cannot write the length while streaming. It should be done only after the file is completed.
It's interesting to try to fix the file by using wavefile module

Hey guys, I had the same issue and created a solution for that. Here's the package I developed and am actually using. https://www.npmjs.com/package/@wpdas/wave-header


installation error with step python3 manage.py migrate

I try to install the project to my debian 12 VM. So I follow your instructions. I hope I  installed all depedencies but I crash with the step python3 manage.py migrate.
This is my output in my CLI:
(pytition_venv) root@attac-petition:/var/www/petition/www/pytition/pytition# python3 manage.py migrate
Traceback (most recent call last):
File "/var/www/petition/www/pytition/pytition/manage.py", line 22, in
execute_from_command_line(sys.argv)
File "/var/www/petition/pytition_venv/lib/python3.11/site-packages/django/core/management/init.py", line 442, in execute_from_command_line
utility.execute()
File "/var/www/petition/pytition_venv/lib/python3.11/site-packages/django/core/management/init.py", line 382, in execute
settings.INSTALLED_APPS
File "/var/www/petition/pytition_venv/lib/python3.11/site-packages/django/conf/init.py", line 102, in getattr
self._setup(name)
File "/var/www/petition/pytition_venv/lib/python3.11/site-packages/django/conf/init.py", line 89, in _setup
self._wrapped = Settings(settings_module)
^^^^^^^^^^^^^^^^^^^^^^^^^
File "/var/www/petition/pytition_venv/lib/python3.11/site-packages/django/conf/init.py", line 217, in init
mod = importlib.import_module(self.SETTINGS_MODULE)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3.11/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "", line 1206, in _gcd_import
File "", line 1178, in _find_and_load
File "", line 1149, in _find_and_load_unlocked
File "", line 690, in _load_unlocked
File "", line 936, in exec_module
File "", line 1074, in get_code
File "", line 1004, in source_to_code
File "", line 241, in _call_with_frames_removed
File "/var/www/petition/www/pytition/pytition/pytition/settings/config.py", line 41
STATIC_ROOT = /var/www/petition/www/static
^
SyntaxError: invalid syntax
What can I do?
Thanks

Hello @mightynico
The issue is that you need to put quotes or double quotes around the strings in your setting file:
STATIC_ROOT = '/var/www/petition/www/static'

Another thing: don't install using the root user, create a non-priviledged pytition user.

Thanks, I will test it


Pin Bokeh to 0.12.7

This PR pins Bokeh to version 0.12.7.  See PR #687 and issue #686 for further discussion.

Looks like PR #689 resolves #686 . See https://www.ospc.org/ccc/882/


Fix for insights

Reworked insights manager workflow

Coverage increased (+0.1%) to 34.62% when pulling 09995d2dce992e0f18575f77e3372e510b2e89c9 on insight_fixes into ae243cb7817f02b3859a4b1815f702757c566be0 on development.

Coverage increased (+0.1%) to 34.62% when pulling 0cc3ddf78d762a6b617f3cb95c8a5bc4a08fdb9b on insight_fixes into ae243cb7817f02b3859a4b1815f702757c566be0 on development.

Coverage increased (+0.3%) to 34.781% when pulling 971711b55e489cd4ac33572ef5dfeb8b3eedded2 on insight_fixes into ae243cb7817f02b3859a4b1815f702757c566be0 on development.


Broken in Grafana v11.3

I just updated my Grafana setup to v11.3.0+security-01 and the static data source is broken. I'm using it in the variables:
Grafana v11.2.3

Grafana  v11.3.0+security-01

As you can see there are no values anymore.

@Salzi Do you use the latest version 4.3.0 of the Business Input (Static data source) ?
I tested behavior on v11.3.0+security-01, main branch and can't reproduce the issue.

@mikhail-vl Thank you for your response—you were absolutely right. The plugin just needed an update to the latest version. I’m fairly new to Grafana and hadn’t realized that plugins require separate updates. I appreciate your help, and apologies for the oversight!

@Salzi We keep our plugins up to date with the latest Grafana updates.
In the latest Grafana 11.3, you can see which plugins needs to be updated in a separate tab.
Let us know if there is anything else.