New integration 'Home Energy Managment'

Hi,
Can 'state_class' please be added and set to 'measurement' for the appropriate sensor(s) so it can be used in the new Energy Management?
https://developers.home-assistant.io/docs/core/entity/sensor#long-term-statistics
Thanks!

Hi,
I’m currently out for a few days but I’ll change it as soon as I am back :)
Regards
El El mié, 11 ago 2021 a las 9:42, GMTZMU @.***>
escribió:

Hi,
Can 'state_class' please be added and set to 'measurement' for the
appropriate sensor(s) so it can be used in the new Energy Management?
https://developers.home-assistant.io/docs/core/entity/sensor#long-term-statistics
Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dmoranf/home-assistant-wattio/issues/14, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABW6HAJETVVRUCV5PQ6MGWLT4ISW3ANCNFSM5B5YW4DQ
.

Hi!
Im checking the docs because it seem that changing the state_class is not enough to use the new energy management features.
I think the problem is that the measurement we get from wattio is current Kw so it needs to be a power sensor instead of energy sensor (KWH) ... and the statistics will not be accurate since the consumption depends on update interval (defaults to 60s) ... so maybe we need to build a new sensor or template to get the mean value ... :/
We can change the update interval to 1 second but wattio api does not work very well and returns errors on many of them ...
Any ideas?

Hi, thanks for your message.
I had integrated wattio with new  power management feature.
If you need more information, you can view this video https://www.youtube.com/watch?v=gOH2P1ZJ6fE
Thanks.

Good job! I have managed to configure it following the video and integration component official docs at https://www.home-assistant.io/integrations/integration
I'll take advantage of this issue to make some minor changes in the component since the measurement units of power sensors are set to Watt and should be W as Home Assistant needs and maybe i could add some info to the README about how to configure it with the Energy Management Panel
Thank you!

Te contesto en castellano.
Yo he usado el video para configurar los dispositivos de Wattio y de Tuya que tengo, el video es muy intuitivo.
Con Wattio, primero tienes que cambiar las unidades de Watt a “W”, esto se hace en el documento “customize.yaml” que previamente lo tienes que llamar desde “configuration.yaml”.
 sensor.ordenador_marc:
   unit_of_measurement: W
   device_class: power

Tienes que crear sensores nuevos de energía indicando el sensor de origen, lo puedes hacer en el fichero “sensor.yaml”, que también se llama desde “configuration.yaml”.

platform: integration
source: sensor.ordenador_marc
name: energy_ordenador_marc
unit_prefix: k
unit_time: h
round: 2
method: left

De nuevo en el fichero “customize.yaml” se ha de cambiar las unida de medida de los sensores de energía que se han creado.
sensor.energy_ordenador_marc:
unit_of_measurement: kWh
friendly_name: Ord. Marc
Con todo ello, después de reiniciar Hassio y que los dispositivos estén en uso ya podrás agregarlos al panel de energía.
Gracias por tu labor, por la integración de Wattio.
Saludos.
Sent from Mail for Windows 10
From: dmoranf
Sent: martes, 7 de septiembre de 2021 11:11
To: dmoranf/home-assistant-wattio
Cc: GMTZMU; Author
Subject: Re: [dmoranf/home-assistant-wattio] New integration 'Home Energy Managment' (#14)
Good job! I have managed to configure it following the video and integration component official docs at https://www.home-assistant.io/integrations/integration
I'll take advantage of this issue to make some minor changes in the component since the measurement units of power sensors are set to Watt and should be W as Home Assistant needs and maybe i could add some info to the README about how to configure it with the Energy Management Panel
Thank you!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.


[WIP] Prep release

I'd like to get a candidate out we can test headsets against. Let's not merge this until we've done tests.


Parser error recovery seems to be adding invincible nodes with semantic errors.

Description:
type ObjectName object {
    function foo() {
        int i = 0;
    }
};

// extra token `abstract`
abstract type ObjectT object {
    *ObjectName;
};

public function main() returns error? {
}

error: .::main.bal:8:23: invalid token 'ObjectT'
error: .::main.bal:8:23: invalid token 'type'
error: .::main.bal:10:2: missing identifier

Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

What do you mean by an invincible node?

In the above example, what happens is, since abstract keyword is a valid type starting token (abstract object - before the object change), the parser starts parsing that code snippet as a module var-decl, which starts with an object type.

What do you mean by an invincible node?
$missingNode$12 in screenshot.
If a user didn't type that he shouldn't be getting this error right.

Parser sometimes insert nodes to create a resilient tree, upon syntax errors.
We should not log errors for those inserted nodes at the compiler front end.

btw, when I run this on the latest master I only get below errors:
Compiling source
	testMain.bal
error: .::testMain.bal:8:23: invalid token 'ObjectT'
error: .::testMain.bal:8:23: invalid token 'type'
error: .::testMain.bal:10:2: missing identifier

Sorry forgot to specify the issue was detected on slp-preview3.

What puzzles me is why the errors logged on the console are different from the errors shown on vscode.
@nadeeshaan any idea?

What puzzles me is why the errors logged on the console are different from the errors shown on vscode.

The test case is now outdated and couldn't observe this behavior.
We will suppress all the semantic diagnostics causing on top of missing node names with issue #29168. Hence closing this issue.


game crashes when it tries to execute code from this mod

the game is regular with loading other mods but then it tries to load this mod also then it just crashes as soon as it executes the code, is there anyway to fix it or do i need to uninstall the mod?

there's no crashlog so it's impossible to tell what's the issue, but if v1.0.2 didn't fix the crash, please reopen the issue and attach a crashlog.
and yes, you will need to manually uninstall the mod in order to update it (because it crashes on load)

oh sorry i was asleep so heres the error code for you and tell me if i did something wrong  with the download.


fix(frontend): enhance auth logging

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


Store timezone tz info as part of timestamps

StarRocks: We don't store tz info in timestamp. So they will have to handle tz at the application layer. ie. always store timezone in UTC in the db, and convert to user's local tz at retrieval time.

https://www.reddit.com/r/dataengineering/comments/18fxgna/current_data_setup_cannot_handle_userfacing/

https://docs.starrocks.io/docs/administration/timezone/


test: test for checkbox in alert action

Test for checkbox in alert action

@mbruzda-splunk do we still need this PR?


BigQuery: remove circular imports in helper functions that use Row class

With https://github.com/GoogleCloudPlatform/google-cloud-python/pull/4291 the Row class was moved from _helpers.py to table.py. This caused circular references between the two files, because _helpers.py imports Row from Table and table.py imports multiple helper functions from _helpers.py.
To temporarily resolve the import errors, the Row class was imported within the individual helper functions where it was needed. The circular import issue needs to be resolved so that Row does not have to be imported inside functions.

#5550 moved the _item_to_row and _rows_page_start to the table module (their only uses were there).


Dear, could you support A1584. I can make A1584 device in pwndfu.

https://github.com/dora2-iOS/ra1npoc  this is pndfu for a1584. thanks dear

Is it possible?


Please enable Dashboard Native filters in SuperSet 1.3.0

I do not see the option for adding dashboard native filters in SuperSet 1.3.0.

There is "documentation" here:
https://github.com/apache/superset/issues/12148
The feature is behind a "feature flag" as shown above.


Add .editorconfig file

Why
Add .editorconfig https://editorconfig.org to enforce consistent coding styles for everyone who uses a different editor/IDE.
EditorConfig plugins are available for different editor/IDE:

https://marketplace.visualstudio.com/items?itemName=EditorConfig.EditorConfig
https://plugins.jetbrains.com/plugin/7294-editorconfig
https://github.com/sindresorhus/editorconfig-sublime
https://github.com/editorconfig/editorconfig-vim

Who Benefits?
Developers

After research, I found that in Go language we can use go fmt for formatting.


Get the home of the current user

Is this more commonly available than $HOME?

Oh yes
On Sat, Dec 11, 2021, 4:19 AM Ted Hart-Davis @.***>
wrote:

Is this more commonly available than $HOME?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dylanaraps/pure-bash-bible/pull/93#issuecomment-991450704,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AV7BMBYUCQXSE53R5R32OG3UQLGNPANCNFSM4J4BJDEQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Upgrade to go 1.20

CORE-1519

A few more places to consider, there might be good reasons not to upgrade some of them right now.

[ ] ./src/integrations/connectors/snowflake/Dockerfile
[ ] ./src/server/cmd/pachd/signals_unix.go
[ ] etc/testing/circle/rootless_test.sh
[ ] etc/testing/circle/start-minikube.sh
[ ] etc/testing/kafka/Dockerfile
[ ] etc/testing/kafka/go.mod
[ ] etc/proto/Dockerfile
[ ] etc/proto/protoc-gen-zap/go.mod
[ ] etc/testing/spout/Dockerfile
[ ] etc/testing/spout/go.mod

Please take another look.  Explicitly requiring 1.20 for everything looks pretty OK.  I also ran "make proto" locally since I think CI doesn't test that, and it works fine.


Ensure run_handle_stop gives up waiting for the task to complete on subsequent KeyboardInterrupt events

The current implementation will endlessly expect the application to shut itself down based on the stop signal: https://github.com/vkottler/vcorelib/blob/master/vcorelib/asyncio/init.py#L167
Instead, if the stop signal is already set, either return without doing anything else or attempt to shut the event loop down completely.

Should be fixed by https://github.com/vkottler/vcorelib/pull/111 and https://github.com/vkottler/vcorelib/pull/112.


Events Tab - Set to UTC / Local Time

It should toggle the time displayed in the table as well.

In the beta, there are 2 ways this gets handled.

There is a toggle that sets the time selection(date drop downs, slide bar)  to UTC/local time.
There is now a column for UTC and one for local time.

Are there any other aspects that should be added?


Translate "explicit_warning" section into german

uhh… description

on a different note,
how would you recommend translating "Song Setup" into german?

Maybe "Song Einstellungen" (Song Settings)?

thx for helping
lgtm :)


Add allow-list rescan button and display the allow-list numbers on the UI

Tested pretty thoroughly, doesn't seem to cause any issues and should be crash-free. Should update the logic for querying the AllowList section to not use indexes for future-proofing, but unless the UI structure is modified it should be fine for a while.


function table_to_book returns a date if td element contains text "marks" followed by a number

my data has text like "Tally marks - up to 10" and  "Tally marks - up to 20" in td element of my html. When I convert this data to xlsx it returns a date object like this
A4:{t: "n", v: 36586, z:"m/d/yy"}
instead of same text value. I have then tried to figure it out what is causing it. So I concluded that if there is a word "marks" followed by a number then it shows this kind of behavior.
Why is it showing this behavior.

The error comes from the Chrome date logic:
new Date("Tally marks - up to 10") // Sat Mar 10 2001 00:00:00 GMT-0500 (EST)

To avoid matching those, the reader checks for a short month name, and of course "Marks" happens to contain "Mar".
Until it is resolved, passing raw:true bypasses the value guessing logic.

@reviewher should I pass raw:true like this
var worksheet = XLSX.utils.table_to_book(document.getElementById(tableId), {raw : true});
I have tried this but this is not working. let me know if I am doing something wrong.
Also I am writing the file like this
var wopts = { bookType:'xlsx', bookSST:false, type:'binary', raw: true};
var wbout = XLSX.write(worksheet, wopts);

Your first line is correct.  Are you using the latest version? console.log(XLSX.version) will tell you, and the current release is 0.11.2
Just as a sanity check, try going to http://sheetjs.com/demos/table and editing the number 4 to Marks 4, then open the console and run:
XLSX.utils.table_to_book(document.getElementById('table')).Sheets.Sheet1.D3
// {t: "n", v: 36954.041666666664, z: "m/d/yy"}
XLSX.utils.table_to_book(document.getElementById('table'), {raw:true}).Sheets.Sheet1.D3
// {t: "s", v: "Marks 4"}

@reviewher Thank you so much for helping me out here. putting raw: true and updating library to latest version fixed it for me.
God Bless You

Glad that worked :)

I would like to know how to use the raw parameter but at the same time, format the numeric columns in a way that im still allowed to add the values in excel.

Currently that's not possible.
For a customer of our Pro offering, we added a backdoor rawDates option that skips the entire date parsing logic.  We can do something similar here too.

@reviewher Thank you so much for helping me out here. putting raw: true and updating library to latest version fixed it for me.
Best Regards


Add tests for our markdown parser/renderer

This PR makes a couple of changes:

it replaces homebrewed pico-check with jest.
I'm totally unfamiliar with JS ecosystem, but from what I've read Jest seem to be a reliable and popular framework so I decided to go with it. Also it is listed as recommended tool for testing React applications.

it introduces a few initial tests, mostly for mustache inline spans

@calculuschild, any feedback here?

This looks fine to me. I'll merge this so we have the framework in place to add more tests in future PRs.

@calculuschild, merge conflict in package-json.lock is now resolved

Thanks @AlexeySachkov ! Looks good. Merging now.


[SpriteFrames ]Value of FPS field is applied to next selected animation while field is being focused.

Godot version
v4.1.1.stable.mono.official [bd6af8e0e]
System information
Godot v4.1.1.stable.mono - Windows 10.0.19045 - Vulkan (Forward+) - dedicated Quadro T2000 () - Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz (12 Threads)
Issue description
When in the SpriteFrames animation editor, if the user clicks on the FPS field and sets a new value, that value will be applied to the next selected animation while this field remains focused.
Steps to reproduce
[Having installed the sample project]

Double-click on "new_sprite_frames.tres"
In the SpriteFrames editor that pops up, select "animA" in the list on the left. Each of these animations have their FPS value set to 5 by default.
Click on the FPS field, and set the value to 10.
While the FPS field is still focused, select "animB". You will see now that the FPS value from the previous animation has automatically been applied to the new one, and animB will have a value of 10 FPS.

Note that at this point if you select "animC" while the FPS field is still focused from earlier, it will not change the value to 10FPS, indicating that this problem only occurs for the next immediate animation selected.
Minimal reproduction project
godot-bug-animation.zip

I see this is actually a duplicate report and will be fixed. Closing on your behalf; apologies! :)

I see this is actually a duplicate report and will be fixed. Closing on your behalf; apologies! :)

Can you link to the issue in question?

Certainly! https://github.com/godotengine/godot/issues/80575


Don't create expando object literals in TS

Previously, getWidenedTypedFromJSPropertyAssignment was not called for Typescript code. Since property assignments on functions, it is. That meant that property assignments would incorrectly create a JS container for empty object literals in a property assignment, even in Typescript:
const one = () => 1
one.p = {}
one.p.q = {} // should not work in Typescript!

Now empty object literals never create expando objects in Typescript, because getJSExpandoObjectType requires the declaration to be in a JS file.
Fixes #26521

Why does the contextual type matter for an empty object literal?


DB Connection

Just wondering whether it's making a database connection with the every request or does it keep the connection open ?

That's the main reason why this blog post should be deprecated...
From my understanding, it's creating a new connection every time the worker executes the code because there is no cache to actually remember any context from the previous execution.
I should write a new blog post because since this blog post was written, MongoDB released the Atlas Data API which is a better way to talk to MongoDB when using the Driver isn't an option. Which is basically the problem we are trying to work around here.
So yeah. My advice is don't do that in production. It was more a proof of concept than a production ready piece of code.
Use the Atlas Data API instead. This will resolve the connection problem issue.

That's the main reason why this blog post should be deprecated...
From my understanding, it's creating a new connection every time the worker executes the code because there is no cache to actually remember any context from the previous execution.
I should write a new blog post because since this blog post was written, MongoDB released the Atlas Data API which is a better way to talk to MongoDB when using the Driver isn't an option. Which is basically the problem we are trying to work around here.
So yeah. My advice is don't do that in production. It was more a proof of concept than a production ready piece of code.
Use the Atlas Data API instead. This will resolve the connection problem issue.

Thanks for your input.
i noticed, in AWS, if you don’t terminate them and leave them hanging your function will timeout, can set to callbackWaitsForEmtpyEventLoop = false however this feature is not in cloudflare workers
context.callbackWaitsForEmtpyEventLoop = false

In AWS you can have a shared context and reuse the same connection for each lambda execution.


Latencies Overlay with tests

Need an overlay using latencies between clients
Need functionnal tests with this overlay

Use T-Man repository from RAN3D/tman-wrtc

Close because an overlay was added. If needed, reopen this issue.


config.jelly: add "Copy HTML reports in workspace" option

[Current coverage][cc-pull] is 70.20%

Merging [#46][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master        #46   diff @@
==========================================
  Files            22         22
  Lines           574        574
  Methods           0          0
  Messages          0          0
  Branches         85         85
==========================================
  Hits            403        403
  Misses          139        139
  Partials         32         32

Powered by Codecov. Last updated by [e70c0dc...1a31bb6][cc-compare]
[cc-base-branch]: https://codecov.io/gh/web-innovate/bootstraped-multi-test-results-report/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/web-innovate/bootstraped-multi-test-results-report/compare/e70c0dcd62cb8b72ea60e87ce347075b69e0154c...1a31bb62ab692a8b5c0a117cd286f47cb7d5d6d0
[cc-pull]: https://codecov.io/gh/web-innovate/bootstraped-multi-test-results-report/pull/46?src=pr

@bagage two minor things, other than than, this is good to go

Done, thanks @BogdanLivadariu

:+1:  great job, I just hope you have tested it :smile:

Yes I tested it on my side, working great for me ;).

@bagage super, will try to hit the deploy, in the next days, working on a mechanism to make travis deploy the releases when a tag is created, I hope this will not take too long

There is no urgency in deploying for me - I am using my own built .hpi so please take your time ;).


Evaluating cell or selection in Python consoles takes ages

From<EMAIL_ADDRESS>on 2014-08-10T07:46:15Z
Spyder Version:  2.3.0beta2
Python Version:  3.3.5
Qt Version    :  4.8.4, PyQt4 (API v2) 4.9.6 on Windows
pyflakes >=0.6.0:  None (OK)
pep8 >=0.6      :  1.5.7 (OK)
IPython >=0.13  :  2.0.0 (OK)
rope >=0.9.2    :  0.9.4 (OK)
sphinx >=0.6.6  :  1.2.2 (OK)
matplotlib >=1.0:  1.3.1 (OK)
sympy >=0.7.0   :  0.7.5 (OK)
pylint >=0.25   :  1.1.0 (OK)
What steps will reproduce the problem?

I have a short code which I would like to evaluate such as the following function definition
def remove_field_header(sourceFile):
import os
import codecs
input_name = sourceFile
tmp_name = input_name + '.tmp'
with codecs.open(input_name, 'r', encoding='utf8') as fi,
codecs.open(tmp_name, 'w', encoding='utf8') as fo:
for i, line in enumerate(fi):
if i == 0:
line.replace('#Field:', '')
fo.write(line)
fo.write(line)
os.remove(input_name) # remove original
os.rename(tmp_name, input_name) # rename temp to original name

I press F9 after I selected the above lines

What is the expected output? What do you see instead?
I expect fast evaluation of the above.
Instead I see the code is evaluate VERY slowly in the console, about 2 seconds per line.
Please provide any additional information below
. Windows 64Bit 8.1 Enterprise
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=1930

From ccordoba12 on 2014-08-17T10:00:10Z
Please use the IPython console if you want better speed.

From ccordoba12 on 2014-10-31T15:13:25Z
Labels: -MS-v2.3.2 MS-v2.3.3

From ccordoba12 on 2014-10-12T17:25:04Z
It probably has to do with our quite bad unicode support in our Python consoles. I'll see if I can fix it in our next release.
Labels: MS-v2.3.2

From ccordoba12 on 2014-12-15T09:33:53Z
I finally found the problem with this one. It'll be fixed in 2.3.3
Summary: Evaluating cell or selection in Python consoles takes ages  <span class="oldvalue"> (was: evaluate cell or F9 takes ages) </span>
Status: Accepted
Labels: Cat-Console

From ccordoba12 on 2014-12-15T09:22:50Z
issue #1624 has been merged into this issue.

From<EMAIL_ADDRESS>on 2014-12-15T14:16:53Z
Awesome, thank you very much.

From ccordoba12 on 2014-12-15T09:39:23Z
This issue was closed by revision b5b4323b3cf0 .
Status: Fixed


Watcher wont upload when using context

Config file

{
"name": "GeDeLi",
"host": *******************************,
"protocol": "ftp",
"port": 21,
"username": *********************,
"remotePath": "/public_html/gedeli",
"uploadOnSave": false,
"context": "dist",
"watcher": {
"files": "dist/*",
"autoUpload": true,
"autoDelete": false
},
"ignore": [
".vscode",
".git",
".DS_Store",
"node_modules"
]
}

Describe the bug
I am using vue-cli to build a production dist folder. If i use a context, the watcher doesnt seems to upload. If i dont use a context, upload works (but destination is not what i want).
To Reproduce

npm run build
dist folder is created/updated
nothing is uploaded

Expected behavior
The files inside dist folder should be uploaded to the root of the remote path
Desktop :

OS: Win 10
VSCode Version 1.33.1
Extension Version 1.12.5

Extension Logs from Startup
[05-15 14:23:50] [debug] register command "Cancel All Transfer" from "./commandCancelAllTransfer.ts"
[05-15 14:23:50] [debug] register command "Config" from "./commandConfig.ts"
[05-15 14:23:50] [debug] register command "List Active Folder" from "./commandListActiveFolder.ts"
[05-15 14:23:50] [debug] register command "Open Ssh Connection" from "./commandOpenSshConnection.ts"
[05-15 14:23:50] [debug] register command "Set Profile" from "./commandSetProfile.ts"
[05-15 14:23:50] [debug] register command "Toggle Output Panel" from "./commandToggleOutputPanel.ts"
[05-15 14:23:50] [debug] register command "Upload Changed Files" from "./commandUploadChangedFiles.ts"
[05-15 14:23:50] [debug] register command "Delete Remote" from "./fileCommandDeleteRemote.ts"
[05-15 14:23:50] [debug] register command "Diff" from "./fileCommandDiff.ts"
[05-15 14:23:50] [debug] register command "Diff Active File" from "./fileCommandDiffActiveFile.ts"
[05-15 14:23:50] [debug] register command "Download" from "./fileCommandDownload.ts"
[05-15 14:23:50] [debug] register command "Download Active File" from "./fileCommandDownloadActiveFile.ts"
[05-15 14:23:50] [debug] register command "Download Active Folder" from "./fileCommandDownloadActiveFolder.ts"
[05-15 14:23:50] [debug] register command "Download File" from "./fileCommandDownloadFile.ts"
[05-15 14:23:50] [debug] register command "Download Folder" from "./fileCommandDownloadFolder.ts"
[05-15 14:23:50] [debug] register command "Download Force" from "./fileCommandDownloadForce.ts"
[05-15 14:23:50] [debug] register command "Download Project" from "./fileCommandDownloadProject.ts"
[05-15 14:23:50] [debug] register command "Edit In Local" from "./fileCommandEditInLocal.ts"
[05-15 14:23:50] [debug] register command "List" from "./fileCommandList.ts"
[05-15 14:23:50] [debug] register command "List All" from "./fileCommandListAll.ts"
[05-15 14:23:50] [debug] register command "Reveal In Explorer" from "./fileCommandRevealInExplorer.ts"
[05-15 14:23:50] [debug] register command "Reveal In Remote Explorer" from "./fileCommandRevealInRemoteExplorer.ts"
[05-15 14:23:50] [debug] register command "Sync Both Directions" from "./fileCommandSyncBothDirections.ts"
[05-15 14:23:50] [debug] register command "Sync Local To Remote" from "./fileCommandSyncLocalToRemote.ts"
[05-15 14:23:50] [debug] register command "Sync Remote To Local" from "./fileCommandSyncRemoteToLocal.ts"
[05-15 14:23:50] [debug] register command "Upload" from "./fileCommandUpload.ts"
[05-15 14:23:50] [debug] register command "Upload Active File" from "./fileCommandUploadActiveFile.ts"
[05-15 14:23:50] [debug] register command "Upload Active Folder" from "./fileCommandUploadActiveFolder.ts"
[05-15 14:23:50] [debug] register command "Upload File" from "./fileCommandUploadFile.ts"
[05-15 14:23:50] [debug] register command "Upload Folder" from "./fileCommandUploadFolder.ts"
[05-15 14:23:50] [debug] register command "Upload Force" from "./fileCommandUploadForce.ts"
[05-15 14:23:50] [debug] register command "Upload Project" from "./fileCommandUploadProject.ts"
[05-15 14:23:50] [info] config at d:xxxxxxxxxxxx\dist {"remotePath":"/public_html/gedeli","uploadOnSave":false,"downloadOnOpen":false,"ignore":[".vscode",".git",".DS_Store","node_modules"],"concurrency":4,"protocol":"ftp","connectTimeout":10000,"interactiveAuth":false,"secure":false,"remoteTimeOffsetInHours":0,"name":"GeDeLi","host":"","port":21,"username":"*****","context":"dist","watcher":{"files":"dist/","autoUpload":true,"autoDelete":false}}

watcher.files is relative to context. You should use "files": "**/*"

@liximomo ahhh..good thanks for your answer!


replaced list.insert(0, ...) with deque.appendleft(...)

Replaced list with deque because it's faster to use.
Main drawback is memory usage.
import sys
from collections import deque
sys.getsizeof(deque())
# 760
sys.getsizeof([])
# 56

I searched around the github for "def process_view" with "path:middleware", found that on avg people have 1-2 middleware per project.
If it's not a good idea to put deque here, maybe it can be considered to replace other instances of list.insert(0, ...) with deque

If it's not a good idea to put deque here, maybe it can be considered to replace other instances of list.insert(0, ...) with deque

We don't make hypothetical bulk optimization. Each case must be considered separately and proven (with benchmark) that it is actually faster and worth changing. As far as I'm aware, it's not worth changing if it's not a hot path.

I agree with Mariusz here the risk of introducing regressions & increased complexity (albeit small) isn't worth it 👍


Mines obsidian over lava AND blacklisting it if fighting a mob

Steps to Reproduce (as best as you can)
Run @gamer until get to the stronghold task
Expected Behavior
Mine obsidian safely
Actual Behavior (What lead to the bot getting permanently stuck)
Burns obsidian in lava and blacklisting the rest of it
Crashlogs and Screenshots (if applicable)
(you may drag + drop text logs and images here, thanks GitHub)
https://youtu.be/vB7URyaTOdA

Hmm,  this is odd. I'll take a look into this next week.

Began looking into this.

I widely believe that the issue on this bug is rooted deep inside the bots ability to keep track of all the different things going on.
Things to note about the video:

We are in bruhsalt biome so lot's of uneven levels (limited movement paths) surrounded by hostile mobs and dangers
We are mining a tough block
We are (seemingly) limited to only a few paths to the nether portal.

Whenever we see things like "Blacklisted", this means the bot is trying to either avoid getting stuck, or avoid dying. Mining obsidian is a really dangerous task because it takes a long time.
Suggested Fix:
To be honest, block tracking probably needs a refactor or a re-design. At the very least, a look over again. However, the easiest way to overcome this bug now is to simply collect the 10 obsidian needed for a portal in the overworld. It is much much safer and is very unlikely to break since we aren't stuck in the basalt biome.  I have only ever seen this issue happen in the basalt biome. Pretty much everytime we get that biome, altoclef gets messed up. Possible solutions to this are to simply get out of that biome. More research into this is needed.
The suggested course of action here is to collect obsidian prior to entering the nether portal for fast travel. This will protect us from the pitfalls of mining such a tough material in the most dangerous biome in Minecraft.
Resolution difficulty: easy
Timeline: probably a day

Began development on this. The main thing to worry about here is the fix for Fast Travel.
The timeline we are following here is a hard timeline, which is 1-2 weeks.

With 1.18.2 done, I can finally get back to this. I plan to re work this task. Delaying this another 2 weeks.


Fix directory path replacement

Fixes https://github.com/zeppelinos/zos/issues/927. The main problem was that the previous regexp was not precise enough to cover some edge (well, not precisely edge) cases  if having a project directory path such as: foo/bar/bazcontracts, or anything with a contracts at the end of the directory name.
Now, we are using path#dirname instead (thanks @ylv-io!!)
If approved, I will add this to the release/2.3 branch.

🚳
path.dirname
Or I am missing something?

path.dirname
Or I am missing something?

I always forget about that one! thanks @ylv-io


Add IISSS Eugenio Pantaleo

file.zip
Pantaleo

@AlessioExe Add I am sorry, I have to close the request without merging, because you made a mistake in a request and did not specify any filepath. Please ensure that you are familiar with this repository rules located at the bottom of this page and make a new correct request. Thank you for your understanding!


Link status on project page should have left margin

Other link items have margin. Status indicator should too.

v1.0.0


zap+store:// scheme

So that we can link directly to an app and it opens zapstore with that app ready for install

Great idea!
https://pub.dev/packages/app_links

maybe this one https://pub.dev/packages/protocol_handler


Pong Coding Challenge

This is my refactored repo on a pong coding challenge. To be honest, I am not sure whatever all bugs are fixed or not.
Major Changes:-

Everything is relative to the center of the canvas. (even the calculations lol)
Two separate functions "CheckPaddleLeft" and "CheckPaddleRight" merged into a single function "CheckPaddle".

Bugs in the repo:-

If puck and paddle both reach the corner of the canvas, puck stucks and vibrates until the paddle is not moved (even though puck's center is out of the canvas).
If observed on the low framerates, the puck does not bounce directly from the surface of the paddle, it comes a bit inside the paddle to bounce.
When the puck passes the paddle, not only puck but also both the paddle should go back to their original positions.

Work left behind:- Text alignment to show the score.
Anyways, here is my repo:-
Click here

This is great! Thanks for sharing. If you'd like it to be shown on the website you can follow the community contributions guide to add it to the pong webpage.


Migrate from Vitreum to Webpack AND/OR other services

Issue
Vitreum has not been maintained for the last 4 years. Since then we have been running using a fork by @calculuschild, but this might prove like an inefficient idea in the long run.
Conversation has started talking about moving from Vitreum to Webpack or another bundling system, this issue is here to log that project's ideas and conversation.
It uses common tools: React, Browserify, and LiveReload.  so we might need to install other  tools as well.

#3479 might be related to Vitreum

Vitreum is mostly a wrapper for Browserify, which is the bundler, with some default settings baked in. It also includes Babel (tranform new JS into backwards-compatible JS), and includes watchify, livereload, and nodemon which are the "auto-restart/rebuild" tools. Then it kind of outputs it all into a string that launches the app on the client. That's basically it.

Browserify is not getting updated much anymore (occasionally hiccups on newer JS), but we could update it to Webpack.
Babel should be kept. We have overwritten most of the Vitreum defaults in our buildHomebrewery.js script anyway.
Watchify tells Browserify to rebuild when a change is detected. Don't think it works with other bundlers but Webpack should have a similar tool
Nodemon should be kept. Lets us auto-restart the server when rebuild happens, for development
Livereload no longer works with newer browsers (like Nodemon but for restarting the browser page)

Thanks for clarity on this. Livereload I can live without, but it seems some frameworks still have something similar.

Can confirm Webpack has a similar thing, but i think i used a diferent command for it in Axe, not sure.

I have done some initial work on migrating Homebrewery to Browserify et al directly, in order to eliminate the technical debt that is Vitreum.
My working theory is that once we can plainly see how Browserify et al are actually being used by Homebrewery, then we can start to investigate and learn how to migrate from them to other tools, using the guides and articles on the internet at large that we would struggle to find if we were still trapped behind the obfuscation layer that is Vitreum.

Breaking it down, to the best of my knowledge, Vitreum has been providing the following functions to Homebrewery:

headTags.js - provides the Meta React component. Not sure if there's a reason that this should live in the bundler rather than the codebase directly.
transforms/asset.js - a wrapper for an async file copy function
transforms/less.js - a wrapper for less.render
transforms/pack.js - a wrapper for browserify

At this point in time, I have commented out watchFile and liveReload as Dev only tools.

I had not previously realized that we now have two systems for inject Meta tags. So there exists the opportunity to standardize to a single Meta tag system. The Vitreum Meta tag is used on the Share and Edit pages to add <Meta name='robots' content='noindex, nofollow' />, and on the Home page to add <Meta name='google-site-verification' content='[content_redacted]' />.
In theory, at least, the ogMeta tag system used in app.js could be extended to incorporate these additional tags.


Export networks in other formats

GET network returns network in JSON, but there is no way to get in other format, such as SIF or XGMML.

New query parameter added for v1.1.1 release.

/networks/SUID/views/VIEWID?file=filename.sif

where extension is the output file format.


Adds a feature to create a button on the right hand side of the toolbar.

Adds a feature to create a button on the right hand side of the toolbar at the end, right after the settings button.
On the other hand, the add_toolbar_button function adds a button only in the left half of the toolbar.

Hey @srastanka.
Thanks for the pull!
I'll be honest and admit I haven't tested these changes yet in a real gmail-project, but initially scanning the code I see you're following patterns and conventions from the existing code-base. And that's usually a good thing.
In my opinion however, I think there seems to be quite a bit of code-duplication.
From what I can tell the toolbar() and right_toolbar() shares almost everything except for two things:

the DOM selector
the default button-class if no style-class is provided

We can extract what's common into a internal "generic" toolbar function, and just dispatch to that one from the actual API methods. I've outline this in the quasi-code below.
var create_generic_toolbar_button = function(params, cssStyle, selector) {
    // all the common stuff
};

api.tools.add_toolbar_button = function(params) {
    create_generic_toolbar_button(params, left_style, left_selector);
}
api.tools.add_right_toolbar_button = function(params) {
    create_generic_toolbar_button(params, right_style, right_selector);
}

Doing so would eliminate most of the code-duplication, and I think that's a worthy goal. If you could do that, I'll try to take a deeper look and do some actual testing once landed.
Sounds good?

Hi @josteink,
Thanks for checking out my PR and for your comments.
You brought up a very good point about the code duplication. I agree with your comments.
But would like to mention that there are actually three differences:

the DOM selector

the default button-class if no style-class is provided

the base class is also different for the two styles ("T-I J-J5-Ji lS" for the left and "T-I J-J5-Ji ash" for the right)

So I've refactored the code to follow proper coding practice and put the common code in the create_generic_toolbar_button function and then dispatching to that from the actual api methods.
Let me know how does everything look and if you think anything needs modification.
Thanks.

Hi @josteink .
Have you had a chance to look at this and do some testing?

From an initial glance it looks good.
I'll try to get some testing done tomorrow.

Seems to do what it says, and I don't see any obvious problems with how it works either!
I forgot to request one final change though, but if you can do that as well, I'll merge this in an instant without any further delay.
Basically when adding a new function we need to also do two things:

Add it to the documentation (which you did. Good good!)
Add it to the TypeScript type-definition. This is still lacking.

You should pretty much be able to duplicate the following line, rename the function-name in the copy and you should be home free: https://github.com/KartikTalwar/gmail.js/blob/master/src/gmail.d.ts#L650
If you can do that, I'll get this merged once and for all. Thanks for your contribution, and again sorry for the hold-up!

Hi @josteink .
Sorry for the delay. I've added the function to the TypeScript type-definition.
Please check and merge.

if you can do that as well, I'll merge this in an instant without any further delay.

Looks good to me, and as such I'll honour my word.
Thanks for the patch and thanks for contributing to gmail.js!


Annotation for NullableResponse

At the moment, when using NullableResponse you need to do something like:
if (response.HasValue) {
    x = response.Value!;
}

The ! is required because the compiler doesn’t know anything about the relationship between the properties. This PR adds MemberNotNullWhen to HasValue so that the compiler knows that Value is not null if it is true.
I haven’t stated anything about the false case, because when that happens, Value throws if accessed. (This seems incorrect, to me, since the return type of Value is T? but it will never be null. But this can be addressed in a separate PR.)

API change check
API changes are not detected in this pull request.


Added Network to Cluster Config

@statmike please review this request.

as we talked about, these three changes:

separate line for import of google.cloud.compute
add spaces before/after = signs for readability
add logic check for default network before checking for ALLOYDB_NETWORK like is already used in the cluster config


Default to system nameservers if a Resolver is initialized without any.

This change allows Resolver to be created without specifying nameservers.  When no nameservers are specified, the system resolver config is used.
Types of Changes

Quality of Life Improvement

Contribution

[x] I added tests for my changes.
[x] I tested my changes locally.
[x] I agree to the Developer's Certificate of Origin 1.1.

Thanks, this is a reasonable change and a nice improvement.

I have been trying out this change, and I found the (my) implementation lacking.
Specifically, just using the system nameservers is insufficient, as there are other options like search domains, and so on, that are required. I've added Async::DNS::System.resolver which is also exposed by Async::DNS::Resolver.default.


[incubator/tensorflow-inception] add apiVersion

Adding the apiVersion for chart.yaml
to fix the issue: https://github.com/helm/charts/issues/13763
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] title of the PR contains starts with chart name e.g. [stable/chart]

/assign @mattfarina
/assign @maorfr

/lgtm

/retest

/test pull-charts-e2e

see this error in the logs for the pod
tensorflow_model_server: error while loading shared libraries: libcuda.so.1: cannot open shared object file: No such file or directory

maybe @thomasjungblut can help?

The chart installation needs a GPU node in K8s, I reckon that's not the case for the cluster the build runs on. At least it complains about not finding CUDA.
What we can do:

I can publish a CPU version of the docker image and update the chart accordingly
get a gpu node into the cluster and define this as a requested resource
we ignore the e2e test

What do you guys prefer?

would go for 1 @maorfr @thomasjungblut

I am with @cpanato. If you could do this, I think it can be great!

Sure no worries, happy to help!
I've  just rebuilt both images, you can find the cpu one here:

quay.io/thomasjungblut/tfs-inception:tfs-1.8.0-cpu

and for GPU this one (should be still the same):

quay.io/thomasjungblut/tfs-inception:tfs-1.8.0-gpu

You can just swap the tag in the values.yaml, but please leave some hint for the GPU version, which I reckon is the more popular choice of both.

/lgtm
NICE!
thanks @thomasjungblut, that worked like a charm.
i'll merge that, but please let me know if you think the hint that i left is insufficient ;)

@maorfr +1 all good, thanks for testing the whole thing :)


Please open a Security Advisory

I'd like to report a security vulnerability.
Please open an advisory and add me to it.
https://github.com/badges/shields/security/advisories

Thanks for reaching out @JLLeitschuh!
IIUC, only a repo admin has access to be able to add a user/collaborator to an advisory.
@paulmelnikow is this something you have access to do?

I've created one and added you as a collaborator @JLLeitschuh. I assume you would've received a notification with the advisory details but let us know if not

Closing this in favor of continuing the dialog in the advisory


add pre- and post-link text insertion with tests

Hi,
I needed to add 🔗  link character to external links or the link SVG https://commons.wikimedia.org/wiki/File:External_link_font_awesome.svg
The extension should be backwards compatible with your release. It adds 5 parameters for adding text (a) before / after and (b) inside / outside the  tag (c) on external or all links.
Up to you if you want to add it to your already-useful package.
-Andrew

@andrewjhunt Can you please change the parameters?

preOutside -> beforeLink
preInside -> beforeLinkText
postInside -> afterLinkText
postOutside -> afterLink

Hi, parameter names changed as requested.

Hi, parameter names changed as requested.

Hi @andrewjhunt Thank you for the changes. I just noticed that you have directly made the changes in the index.js file which is a compiled file. In actual, changes needs to be done in index.ts file under src folder and then run npm run build to generate the index.js file.

The param changes are applied to TS. Build working and tests passing.
Lots of TS errors from the remarkable package but perhaps that's expected?
BTW, first time using TS for me. Made best effort to get the details right.

@andrewjhunt Thank you for doing all of these changes, It looks good to me. Great job 👍


Model training guide

TF, yaml and python code for Model training guide

@NimJay thank you for all your comments, we will try to fix them soon


Token validation

Based on nov's IdToken#verify, which we unfortunately cannot use out of the box because we'd like to discern between expired tokens and invalid application/audience/issuer.

Ah – can you merge master into this (or rebase if you're feeling sadistic)?

Rebased! :heart:


Remove hard-coded path to test executable.

Such paths are brittle, due to npm deduping, mocha is not guaranteed to be at this location, it may be higher in the tree.
node_modules/.bin is added to the PATH env var for npm scripts so "mocha" on its own is both shorter and will more robustly be able to locate the mocha executable.
Note that is is a problem in the build script as well, for which you could use something like npm-run

Nice, thanks.


How to use contentContinuation for channels

How do I use ChannelInfosResponse's getChannelContentContinuation method? I cannot figure out how to properly pass in ChannelContent

Hello, you have to pass the type of the content that you want to fetch e.g. ChannelInfosResponse.Playlists.self. If you want to see it in action here.

Thank you!


fix ci build

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@513d4bc). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master    #300   +/-   ##
========================================
  Coverage          ?   73.4%
========================================
  Files             ?      22
  Lines             ?    1098
  Branches          ?      91
========================================
  Hits              ?     806
  Misses            ?     275
  Partials          ?      17

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 513d4bc...5dd0138. Read the comment docs.


Consistency between power line datasets

Is your feature request related to a problem? Please describe.
It is related to the consistency between the data downloaded from openstreetmap using earth-osm and the dataset of power lines in PyPSA-Eur. Here is a visualization of the data for Denmark:

Some of the LineStrings significantly differ from one database to another (blue line shows the PyPSA-Eur data). I know that the PyPSA-Eur data comes from ENTSO-E database that is an "illustration" of the power lines but I can still see some major differences between the two (see e.g. the connectivity of lines located at the North-West of Denmark).
Describe the solution you'd like
Since some of these databases are available, it is useful to have them here for comparison.
Describe alternatives you've considered
Having other databases in the package.

Interesting comparison, supporting anything other than osm data is currently outside the scope of this package. Ideally we need something like powerplanmatching for lines. The holy-grail for hv lines is probably a remote-sensing approach (? @pz-max ).
based on the graph it would seem that there are many more additional lines in osm. There is an implementation of filtering lines in pypsa-earth. Could it also be possible that ENTSO-E lines are simplified?

Yes @mnm-matin , ENTSO-E lines are simplified because some TSOs are blocking the GIS exact location for "no-reason" in my perspective.

Thank you @pz-max and @mnm-matin for your helpful comments, and apologies for the late follow-up. Just a couple of additions to my first comment above:
I forgot to extract both power lines and cables from the OSM data. Now I have both with the following code:
import earth_osm.eo as eo
eo.save_osm_data(
primary_name = 'power',
region_list = ['denmark'],
feature_list = ['substation', 'line', 'cable'],
update = False,
mp = True,
data_dir = './earth_data',
out_format = ['csv'],
out_aggregate = False,
)

and the extracted figures look like this:

Thick black lines show cables and thinner lines show lines. I also used osmium to extract the same data. Here's the code after a few iterations with co-pilot:
import osmium
import csv
import pandas as pd

class PowerLineHandler(osmium.SimpleHandler):
    def __init__(self, output_file):
        super().__init__()
        self.output_file = output_file
        self.csv_writer = csv.writer(open(output_file, 'w'))
        self.csv_writer.writerow(['id', 'version', 'visible', 'timestamp', 'uid', 'user', 'changeset', 'latitude', 'longitude', 'type'])

    def way(self, w):
        if 'power' in w.tags:
            if w.tags['power'] == 'line':
                power_type = 'overhead'
                for node in w.nodes:
                    location = osmium.osm.Location(node.lon, node.lat)
                    self.csv_writer.writerow([w.id, w.version, w.visible, w.timestamp, w.uid, w.user, w.changeset, location.lat, location.lon, power_type])
            elif w.tags['power'] == 'cable':
                power_type = 'underground'
                for node in w.nodes:
                    location = osmium.osm.Location(node.lon, node.lat)
                    self.csv_writer.writerow([w.id, w.version, w.visible, w.timestamp, w.uid, w.user, w.changeset, location.lat, location.lon, power_type])

pbf_file = 'earth_data/pbf/denmark-latest.osm.pbf'
csv_file = 'dk_power_line.csv'
handler = PowerLineHandler(csv_file)
handler.apply_file(pbf_file, locations=True)

The figures look identical but there are subtle differences between them. Notably, some lines are missing in the data extracted by earth-osm. I want to share these observations with you and ask your opinion about the data extraction methods above, whether they should lead to the same outcome, and -as always- the consistency issues.

@simulkade can you report which lines are missing?
@mnm-matin mentioned if lines are dropped in earth-osm,
you will get a log message which lines are dropped.
Generally, we use the same data source as osmium, so there should be no difference.

I have verified that the results of both osmium and the latest earth-osm version are the same. Thanks for bringing this to attention. The osmium part will be part of a unit test in future releases to ensure consistency.

Thanks @mnm-matin and @pz-max, and apologies for not coming back to this issue earlier.


All devices are unaivailable / HA core upgrade 2024.5.4

Hi, first of all, many thanks for your awesome integration. I'm from Italy. Starting from a HA core upgrade to 2024.5.4 (the first I tryed in may, same issue with 2024.5.5) all of my sonoff devices, TH, rf bridges and basic, became in unaivailable state. They work correctly in ewelink app (both first and second account I use for integration) but in the debug files I can see an error to log in at ewelink server (I tryed both local and auto mode). Downgrade to older integration versions doesn't work, issue is still there. What can I do? Thx
Local mode debug.txt
Auto mode debug.txt
diagnostics config_entry-hacs-555296aeb62481b4869d95446410c751.json
1st reg error.txt

I have the same problem

Same as me too

Same here, 2024.5.5

Did anyone find a solution to this? I have the same issue. All devices showing in ewelink, but unavailable in HAS version 2024.5.5. Update to this version seems to be the trigger to the failure.

Same here. Wifi devices are all OK, but zigbee not working. I'm using Home Assistant 2024.6.3 version.

I have the same problems, before with the latest version of HA from 2023 everything worked ok, I updated 6-2-2024 and all my Sonoff devices stopped working (62 units), Please help

I have had this happen a few times after upgrading HA. The only thing that seems to work for me is to remove the Integration Entry:

add the Integration Entry again

Remove the Integration Entry and re-add look like it's work for me. It's need to restart HA for about 3 time to make it fully work

First reboot because when I re-add the entry it's only show me about 2 devices (I have about 15~ of them)
Second reboot to make it Sonoff fully work. but some of custom "Automations" it's marked some of them invalided due to invalided devices. Due to Sonoff start up slow and need to reload all the device lists
Third reboot to make Automations back to normal.

Login error: {'error': 406, 'msg': '{"error":401,"msg":"Sign:sign is error"}'}

Questo errore ha avuto origine da un'integrazione personalizzata.

Registratore: py.warnings
Fonte: custom_components/__sonoff/sonoff_local.py:238
Integrazione: __sonoff
Prima occorrenza: 07:37:01 (1 occorrenze)
Ultima registrazione: 07:37:01

/config/custom_components/__sonoff/sonoff_local.py:238: RuntimeWarning: coroutine 'async_load_platform' was never awaited _LOGGER.debug(

@saltabas do you still have the problem?
The others who have posted here have not provided any debugging information.

I just noticed. The issue author has an error in an unclear version of the component:
custom_components/__sonoff/sonoff_local.py:238

Thanks for clearing that up @AlexxIT: as a result a manual deletion and reinstalling brings it back to working (simply hacs remove was not enough for me)


SCB-680 Optimize find instance api

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Format the pull request title like [SCB-XXX] Fixes bug in ApproximateQuantiles, where you replace SCB-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run go build go test go fmt go vet to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Codecov Report

Merging #376 into master will increase coverage by 0.3%.
The diff coverage is 95.74%.

@@            Coverage Diff            @@
##           master     #376     +/-   ##
=========================================
+ Coverage   69.94%   70.24%   +0.3%
=========================================
  Files          25       26      +1
  Lines        3746     3784     +38
=========================================
+ Hits         2620     2658     +38
  Misses        938      938
  Partials      188      188

Impacted Files
Coverage Δ

server/service/dependency.go
73.17% <ø> (-0.43%)
:arrow_down:

server/service/watch.go
70.96% <0%> (ø)
:arrow_up:

server/service/instance.go
68.55% <100%> (+1.14%)
:arrow_up:

server/service/util/find_cache.go
100% <100%> (ø)

server/service/util/instance_util.go
50.45% <80%> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 75c5ba2...e6ed348. Read the comment docs.

Coverage increased (+0.3%) to 73.546% when pulling e6ed348acbe0b4f5bd178b243e42da563b5d1ac2 on little-cui:master into 75c5ba2e034ae553ce94319b5633805cb1ea5cce on apache:master.


Window size restore after maximize

Type: Bug
(1) Open Visual Studio Code. Note the window size and position.
(2) Click the window maximize button in the title bar. The window will resize to the fill the entire screen and the button will change to the restore function.
(3) Click the window restore button in the title bar. The expected behavior would be to restore the window to the size and position from step (1). However, the button does nothing.
(4) Double-click the title bar. This will indeed restore the window size, and is a workaround for (3).

Thanks a lot for your follow-up; please find a screen recording attached. While you see the mouse above the button, I was clicking multiple times to restore the window's original size - without effect.
https://github.com/microsoft/vscode/assets/16840687/d686e12d-9b7e-41e2-8b88-8d7b4ada79f3

@fbertram thanks for the recording, are the other controls responsive in that state (i.e. minimize and close)

yes, the other buttons work.

Hey @sbatten, this issue might need further attention.
@fbertram, you can help us out by closing this issue if the problem no longer exists, or adding more information.

it appears to be fixed with 1.84.2
On Tue, Dec 19, 2023 at 3:25 AM VSCode Triage Bot @.***>
wrote:

Hey @sbatten https://github.com/sbatten, this issue might need further
attention.
@fbertram https://github.com/fbertram, you can help us out by closing
this issue if the problem no longer exists, or adding more information.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/194422#issuecomment-1862583608,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEAPP36A3PJOHDOYWJF4PK3YKF2RTAVCNFSM6AAAAAA5LJMLCWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRSGU4DGNRQHA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Update template.md

Amanda these are excellent ideas.  Do you know what format the data is in?  It is possible to read the data file and convert it to the format that is most convenient.  How much work it takes is only determined by the availability of libraries to do the conversion.  Or write your own solution.  Let me know if you need help with this.


Fix missing subtract_plddt argument in prep_output call

This PR is a very minor bug fix to add args.subtract_plddt to the prep_outputcall which otherwise fails due to a missing argument.
Interestingly subtract_plddt is already defined in the argument parser, but never used in the script.

Thanks!


Modern Cmake

Learn Modern CMake with subdirectories and targets.

CMakeLists compile without any issue.
subdirectories, include_dirs, compile commands and find_library work properly on Linux.

CMakeLists compile without any issue.
subdirectories, include_dirs, compile commands and find_library work properly on Linux.


[Bug] Version 2.0.3，couldn't resolve slot descriptor

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
doris 2.0.3
What's Wrong?
exec sql
select
	count(*) total_records
from
	(with purchase_tmp as (
	select
		t1.seq_no ,
 		t1.superior_com_name ,
		t3.bill_id
	from
		dwt.dwt_purchase_order_detail_wt t1
	inner join app.app_bill_main_status_v t3 on
		t1.order_id = t3.biz_id
		and t3.bill_type = 25
		and t3.bill_status in (20, 40, 45, 50, 60)
  )
	select
		t1.superior_com_name as "companyName" ,
		concat('[', nvl(t2.bill_info, ''), ']') as "billDetail" ,
		round(t2.tot_amt, 2) as "billAmt"
	from
		purchase_tmp t1
	left join (
	select
		t1.bill_main_id,
		GROUP_CONCAT( JSON_OBJECT( 'costName', t1.cost_name /** 费用类型 */, 'supplier', t1.biz_item_extend_name /** 供应商 */, 'payType', t2.std_code_desc /** 支付方式 */, 'amount', t1.cost_amt_rmb /** 费用金额 */, 'bearer', t1.undertake_method /** 费用承担方 */, 'remark', t1.remark_1 ) ) as bill_info,
		sum(nvl(cost_amt_rmb, 0)) as tot_amt
	from
		(
		select
			t1.bill_id as bill_main_id ,
			t3.cost_name ,
			t3.biz_item_extend_name ,
			t3.payment_type ,
			t3.cost_amt_rmb ,
			t3.remark_1 ,
 			'我方承担（对方填写）' as undertake_method
		from
			purchase_tmp t1
		inner join dwt.dwt_sales_order_wt t2 on
			t1.seq_no = t2.seq_no
		inner join dwt.dwt_bill_cost_wt t3 on
			t2.bill_main_id = t3.bill_main_id
			and t3.deleted = 99
			and t3.bill_item_type = 20
			and t3.undertake_method = 20
	union all
		select
			t2.bill_main_id ,
			t2.cost_name ,
			t2.biz_item_extend_name ,
			t2.payment_type ,
			t2.cost_amt_rmb ,
			t2.remark_1 ,
 			'我方承担' as undertake_method
		from
			purchase_tmp t1
		inner join dwt.dwt_bill_cost_wt t2 on
			t1.bill_id = t2.bill_main_id
			and t2.deleted = 99
			and t2.bill_item_type = 20 ) t1
	left join dim.dim_code_std_type t2 on
		t1.payment_type = t2.std_code
		and t2.from_schema_name = 'dwt'
		and t2.from_table_name = 'dwt_bill_cost_wt'
		and t2.from_column_name = 'payment_type'
	group by
		t1.bill_main_id ) t2 on
		t1.bill_id = t2.bill_main_id
  ) total_records_tmp_table

error: couldn't resolve slot descriptor 445
SQL 错误 [1105] [HY000]: errCode = 2, detailMessage = (<IP_ADDRESS>)[INTERNAL_ERROR]couldn't resolve slot descriptor 445, desc: tuples:
Tuple(id=84 slots=[Slot(id=812 type=BIGINT col=-1, colname=bill_main_id, nullable=1), Slot(id=813 type=VARCHAR col=-1, colname=, nullable=1), Slot(id=814 type=DECIMAL128(38, 4) col=-1, colname=, nullable=1)] has_varlen_slots=1)
Tuple(id=83 slots=[Slot(id=810 type=BIGINT col=-1, colname=order_id, nullable=1), Slot(id=811 type=BIGINT col=-1, colname=bill_main_id, nullable=1)] has_varlen_slots=0)
Tuple(id=82 slots=[Slot(id=804 type=VARCHAR col=-1, colname=std_code, nullable=1), Slot(id=805 type=VARCHAR col=-1, colname=from_schema_name, nullable=1), Slot(id=806 type=VARCHAR col=-1, colname=from_table_name, nullable=1), Slot(id=807 type=VARCHAR col=-1, colname=from_column_name, nullable=1), Slot(id=808 type=VARCHAR col=-1, colname=std_code_desc, nullable=1), Slot(id=809 type=TINYINT col=-1, colname=__DORIS_DELETE_SIGN__, nullable=1)] has_varlen_slots=1)
..................

But when I replaced union all with union  or remove count query the query is ok
And in doris-1.2.4 the query is ok
What You Expected?
Can be executed
How to Reproduce?
No response
Anything Else?
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Can you provide the statement to create table?

ddl-sql.txt
query-sql.txt

Can you provide the statement to create table?

ddl-sql.txt
query-sql.txt

Can you provide the statement to create table?

ddl-sql.txt
query-sql.txt

set global experimental_enable_nereids_planner = true;
try again

set global experimental_enable_nereids_planner = true; try again

okey

2.0.9版本，报 [INTERNAL_ERROR]couldn't resolve slot descriptor 417, desc: tuples: 错误


New Academy School

@nfouka Please provide us with:

the school official website URL
the school street address
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

@nfouka You might miss my previous comment. Please react.

@nfouka I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for the understanding!


CMS Pages Not Showing on Front Side

Thank you for your great admin panel, How can i display the backend register CMS pages display on Frontside. When i call the route like (i.e http://localhost:8000/cmspage/terms-and-conditions) give me error like below , Please provide some brief description for that. Here below i have to attached screen shot for the error.
.
Thank You.

Hello @krishu-25, I have added a PR for this issue. thanks for letting us know.
#145

@bvipul  @viralsolani I would really appreciate if you guys will share step-by-step guide to create new CRUD screen along with permssions.

@krishu-25 ok sure. we will soon add that in to our readme file.

@viralsolani @krishu-25 I think I should create a video for that. I think I would do that.

@bvipul  That's great news. When you are able to publish those videos?

@krishu, soon is what I can say. Probably next week

@bvipul @viralsolani thanks a lot, i think i lost my way, need tutorial
thanks guys

Will soon publish the video for this. For now closing this ticket.


Angular Icon Registry does not work on StackBlitz / CodeSandbox

Describe the bug
Registering custom PX Blue svg icons via maticonregistry does not work in stackblitz.
What is the expected behavior?
PX Blue icons should work when following the instructions.
What are the steps to reproduce?

Start a new stackblitz project
Follow the instructions for using svg icons in Angular
Try to use one of the custom icons (https://github.com/pxblue/icons/tree/master/svg#usage)
You'll get an error that the icon cannot be found

Screenshots or links to minimum reproduction example

What is your environment?

Framework

[x] Angular
[ ] React
[ ] Ionic
[ ] React Native

Browser

[ ] Internet Explorer
[ ] Edge
[ ] Safari
[x] Chrome
[ ] Firefox

Operating System

[ ] MacOS
[x] Windows
[ ] Other

Anything else to add?
Doesn't work when using the single icon syntax or the entire icon set syntax. The instructions work as expected in a local project.

I would like to test if this works when importing a project into StackBlitz from GitHub.

Looking closely at the original logs, it looks like te HTTP request returned a 494 (Header too large) response. I tried with a reduced local version of the icons.svg file (with just a single icon definition), and got a new error:

This bug is reproducible on codesandbox as well.

Is this still a problem? @daileytj @emclaug2


add pragmatic testing

We still have the test setup leftovers in the app so we should make them work again. So components can either have a Storybook story, or a dedicated unit test with react-testing-library.
So the current npm test just needs to succeed.
Ideally we get full unit testing coverage, and ave additional integration test runs.
After this is all setup, the branch protections should be updated to not allow any regression of coverage.
Better alternative: add integration testing with Cypress as we had in Commons for the very basic user flows:

publish
search
consume

This is way more valuable to catch errors

Better alternative: add integration testing with Cypress as we had in Commons for the very basic user flows:

publish
search
consume

This is way more valuable to catch errors

Better alternative: add integration testing with Cypress as we had in Commons for the very basic user flows:

publish
search
consume

This is way more valuable to catch errors

Moved this to progress again. I managed to connect an account. It turned out that we needed window.ethereum instead of window.web3. Web3Modal check for both but for some reason only window.ethereumworks.
Still trying to figure out why the Submit stays disabled.

reopen when we have bandwidth for it


LXD can't remove volume provided by LVM backend

Distribution: Ubuntu
Distribution version: 16.04.6 LTS
The output of "lxc info":

lxc info
config:
  cluster.https_address: <IP_ADDRESS>:8443
  core.https_address: <IP_ADDRESS>:8443
  core.trust_password: true
api_extensions:
- storage_zfs_remove_snapshots
- container_host_shutdown_timeout
- container_stop_priority
- container_syscall_filtering
- auth_pki
- container_last_used_at
- etag
- patch
- usb_devices
- https_allowed_credentials
- image_compression_algorithm
- directory_manipulation
- container_cpu_time
- storage_zfs_use_refquota
- storage_lvm_mount_options
- network
- profile_usedby
- container_push
- container_exec_recording
- certificate_update
- container_exec_signal_handling
- gpu_devices
- container_image_properties
- migration_progress
- id_map
- network_firewall_filtering
- network_routes
- storage
- file_delete
- file_append
- network_dhcp_expiry
- storage_lvm_vg_rename
- storage_lvm_thinpool_rename
- network_vlan
- image_create_aliases
- container_stateless_copy
- container_only_migration
- storage_zfs_clone_copy
- unix_device_rename
- storage_lvm_use_thinpool
- storage_rsync_bwlimit
- network_vxlan_interface
- storage_btrfs_mount_options
- entity_description
- image_force_refresh
- storage_lvm_lv_resizing
- id_map_base
- file_symlinks
- container_push_target
- network_vlan_physical
- storage_images_delete
- container_edit_metadata
- container_snapshot_stateful_migration
- storage_driver_ceph
- storage_ceph_user_name
- resource_limits
- storage_volatile_initial_source
- storage_ceph_force_osd_reuse
- storage_block_filesystem_btrfs
- resources
- kernel_limits
- storage_api_volume_rename
- macaroon_authentication
- network_sriov
- console
- restrict_devlxd
- migration_pre_copy
- infiniband
- maas_network
- devlxd_events
- proxy
- network_dhcp_gateway
- file_get_symlink
- network_leases
- unix_device_hotplug
- storage_api_local_volume_handling
- operation_description
- clustering
- event_lifecycle
- storage_api_remote_volume_handling
- nvidia_runtime
- container_mount_propagation
- container_backup
- devlxd_images
- container_local_cross_pool_handling
- proxy_unix
- proxy_udp
- clustering_join
- proxy_tcp_udp_multi_port_handling
- network_state
- proxy_unix_dac_properties
- container_protection_delete
- unix_priv_drop
- pprof_http
- proxy_haproxy_protocol
- network_hwaddr
- proxy_nat
- network_nat_order
- container_full
- candid_authentication
- backup_compression
- candid_config
- nvidia_runtime_config
- storage_api_volume_snapshots
- storage_unmapped
- projects
- candid_config_key
- network_vxlan_ttl
- container_incremental_copy
- usb_optional_vendorid
- snapshot_scheduling
- container_copy_project
- clustering_server_address
- clustering_image_replication
- container_protection_shift
- snapshot_expiry
- container_backup_override_pool
- snapshot_expiry_creation
- network_leases_location
- resources_cpu_socket
- resources_gpu
- resources_numa
- kernel_features
- id_map_current
- event_location
- storage_api_remote_volume_snapshots
- network_nat_address
- container_nic_routes
- rbac
- cluster_internal_copy
- seccomp_notify
- lxc_features
- container_nic_ipvlan
api_status: stable
api_version: "1.0"
auth: trusted
public: false
auth_methods:
- tls
environment:
  addresses:
  - <IP_ADDRESS>:8443
  architectures:
  - x86_64
  - i686
  certificate: |
    -----BEGIN CERTIFICATE-----
    MIIFUzCCAzugAwIBAgIRAKa2GJpBtESWVBUr/5QftiAwDQYJKoZIhvcNAQELBQAw
    NjEcMBoGA1UEChMTbGludXhjb250YWluZXJzLm9yZzEWMBQGA1UEAwwNcm9vdEBq
    b2JwdWx6MTAeFw0xODA2MjAxNDI4NDlaFw0yODA2MTcxNDI4NDlaMDYxHDAaBgNV
    BAoTE2xpbnV4Y29udGFpbmVycy5vcmcxFjAUBgNVBAMMDXJvb3RAam9icHVsejEw
    ggIiMA0GCSqGSIb3DQEBAQUAA4ICDwAwggIKAoICAQDGH57cSdm2CTCUsoR+N9Y3
    JNWoXFKmcGa4F4/ZM1IyhzK7ZpWesXNPw1llmKoeUIzt6041XwOR3tk/pU95WDha
    ZXD2ilNiP8ws/5mHWTgfy/56SNbLZIJe6/WfIl9E7eCZewYjCg5XVWw6l+FMzFJk
    E11k6cPsbkIiUMBRzi/3OsfgoBTe34YO5yN7cTi9Y/EtSuCiY2Xav8TTRZ0w4jf3
    FjUyUKNe8+2oQQXgZRypE9o3tjKtP8tAUq3IS3beasin+K0s7fHTrCvHJijFc2gV
    s9jMU+e4HM+o97jsgxHpIpJTA9LUe5bJNvCgLEOX9MmkReyZjBqE5GVzrj3hE+he
    ByAoPN+D8x8AOagNFYPWsprXnvlUgVAxYqzBw5mUT2yzo3iQ+ECiyYic+U++KFSt
    9MRjp0k53uxoB23DqtFc/boM1A4/BerLm/CxiBMP5cYUJc5IC9x1yQXU+j/TgwfN
    dVejyFAYXgn/rbUO8xKTzvNQ3UcDp+3c0hmzZTursTdjmUJISGChltxmr3HefJqs
    V6dbi0XBSVbO9wYjj4CUBPqAEZ6zT6uxkb1V9NyXM0i1tsyV3CFBIu/8aHt947yZ
    XLhZDP5A6BUCP+itt1LTf2ETVBc3dh76XSv7M6jFbb1xfjUJ4RFPuX9yyr1wC2xh
    RXgD/rNoTwwi3qrUqwp/awIDAQABo1wwWjAOBgNVHQ8BAf8EBAMCBaAwEwYDVR0l
    BAwwCgYIKwYBBQUHAwEwDAYDVR0TAQH/BAIwADAlBgNVHREEHjAcgghqb2JwdWx6
    MYcEW719IocECgAAFIcECgAACjANBgkqhkiG9w0BAQsFAAOCAgEAV3M3/lM88wwf
    nAoZJfp8y3jJG8lk418aZT6Ki3BL6//I2ldslgkEcXMAPc7ovr+Xy9AhbMvdOc6h
    oiiDG8njrgaoHBp5LascU5b5F9heQo47BpWM+UOB7jtp1/r4O4E6yichB9wElwrs
    M2op9AnG2CcnqNKW3XTbdWY7eEuwbml3mjGuUJBKGV29qwT9G8+q2/PB1R+7SnTm
    f5gISCYkwiPlixGqRQh0kX63djN2SCqNy5lzDJDRQS07qtHULcmrNEHxf/wt7P1Y
    rtIjBqr1sS2qT04VtQxApsKhlBvouhWYJjhrbj6SaiMv1R5zSUOv73VdICHawnN8
    wosM7RlL4f8jVFc/daIWe8SIqjObUbYXDF5y74v0xM8bSS7ZbQYN61J8Tr+Sfhxk
    H4dDHIQEPKNJ0tauGnU4ZCs/8htGjoVpiN8E9T8DTuetfOpJno2np1PIDMGJ1dKs
    GvWZS398O/dV8bgwh5n4PJaz9TeosqqFmYv9fxu3f73oFDyWvN83QSbACU+H1P4p
    9g75ktTUq13SMztyxO2x7BTm4MsTmo4m7AAnH93I+VVJU0aMdnM9TBcDra8r8GC+
    rDwjqPFjso66d/F3iRcXUCgFrK1NEswFcirOT6ts2b6f039gFZ1/Dp6Ync/r6LOJ
    9rcXw9i4lanGNXaHoOnVdOcSoipsf9E=
    -----END CERTIFICATE-----
  certificate_fingerprint: b5e4f4aa59774e57312c6ee25c2060dcfd36db54150c988a5456d351d6b56697
  driver: lxc
  driver_version: 3.1.0
  kernel: Linux
  kernel_architecture: x86_64
  kernel_features:
    netnsid_getifaddrs: "false"
    seccomp_listener: "false"
    shiftfs: "false"
    uevent_injection: "false"
    unpriv_fscaps: "true"
  kernel_version: 4.4.0-141-generic
  lxc_features:
    mount_injection_file: "true"
    network_gateway_device_route: "false"
    network_ipvlan: "false"
    network_l2proxy: "false"
    seccomp_notify: "false"
  project: default
  server: lxd
  server_clustered: true
  server_name: cl-node-3
  server_pid: 11886
  server_version: "3.13"
  storage: lvm
  storage_version: 2.02.133(2) (2015-10-30) / 1.02.110 (2015-10-30) / 4.34.0

Issue description
I can't remove volumes provided by LVM storage backend. Also, I can't move containers between nodes: the process of migration fails because it can't delete source LV. For newly created containers problem doesn't reproduce.
Steps to reproduce

volume looks like:

|         TYPE         |                               NAME                               | DESCRIPTION | USED BY | LOCATION  |
| custom               | demo-storage                                                     |             | 0       | cl-node-2 |

i'm trying to delete this volume:

lxc storage volume delete local demo-storage
Error: Could not remove LV named demo--storage

Can you look at your LXD log for a more complete error?

Can you show:
grep containers_prod--docker-1 /proc/*/mountinfo

grep containers_prod--docker-1 /proc/*/mountinfo
output is empty.

Doh, sorry, should be containers_prod--docker--1 instead

output attached
grep.log

Can you show ps fauxww too?
The output above looks like you have a container actively running using that LV.

Now the container which is owner, of this LV is running. But at time of migration, it was stopped
ps.log

Can you confirm this is still going on with LXD 3.14?
I don't remember us fixing anything that would make this problem go away but there were a variety of storage fixes so worth checking that this is still happening.

The problem still exists: i can't remove any LV that was detached from containers.

I think my LVs are blocked by smth. I'll reboot servers this weekends during maintenance and will try to remove LV's and do migration again.

I'm not sure if this is the same problem or not, but I arrived here by searching for the same error message which I'm seeing in a fresh Debian 10 droplet on Digital Ocean.
I first installed snap and then using that installed LXD (3.14). After running lxd init and creating storage with the LVM backend, I do lxc launch ubuntu:18.04 foo. Then, waiting a bit to make sure the new container is not in some intermediate starting state, I do lxc rm -f foo. This results in the container stopping, but I get an error, Error: Could not remove LV named foo. Running the command again a second time removes the container/volume as expected. Running lxc stop and then lxc rm also works as expected.

@syswipe did the reboot unblock things for you?

Hi, @stgraber!
planned reboot was delayed, i'm going to do it ASAP. After that i'll inform you.

@stgraber , i've just rebooted one of the nodes affected. Now i can delete volumes that belong to this node. On the other nodes problem still exists...

now, i rebooted all nodes. problem solved. but the reason remains unclear

Anyone has a solution to this?
Still happening, reboot does solve it but no permanent fix.
The container is just stock Debian 10, nothing running on it.
I did find the issue to be operating system independent.
https://pastebin.com/raw/GQ8rGnXE

LXD reports the container as stopped, but its still mounted?
| lxcc39dbb31 | STOPPED |                    |                                            | CONTAINER | 0         |

If you unmount each one of those repeated mounts that should solve it:
sudo umount /var/snap/lxd/common/lxd/storage-pools/secondary/containers/lxcc39dbb31

You may need to do it inside the snap's namespace, so try also:
sudo nsenter --mount=/run/snapd/ns/lxd.mnt -- umount /var/snap/lxd/common/lxd/storage-pools/secondary/containers/lxcc39dbb31

Repeat until all the mounts for that instance are gone.

If you unmount each one of those repeated mounts that should solve it:
sudo umount /var/snap/lxd/common/lxd/storage-pools/secondary/containers/lxcc39dbb31

You may need to do it inside the snap's namespace, so try also:
sudo nsenter --mount=/run/snapd/ns/lxd.mnt -- umount /var/snap/lxd/common/lxd/storage-pools/secondary/containers/lxcc39dbb31

Repeat until all the mounts for that instance are gone.

Is there an API for this? any way to automate this?
Because this problem happens randomly.
Any plans to fix this?

We would need a reproducer for it as it does not happen randomly normally (in our test suites or experience).

We would need a set of reproducer steps for it to show how it happens as it does not happen randomly normally (in our test suites or experience). As well as details of the host system, kernel version etc.

So, I should ditch LVM? or is this a known bug to other storage backends too?
I can reproduce this bug on several machines, running different ubuntu kernels, apparently the kernel dosen't matter at all.

Where do I need to look, what info do you need?

So, I should ditch LVM? or is this a known bug to other storage backends too?

Its not a known bug.
This is why we would need more information.
Please post over at https://discuss.linuxcontainers.org/ with your system details, and steps you perform (after a fresh reboot) to get into this mount state. Then we can go through it together and if necessary reopen this issue or open a new issue.

So, I should ditch LVM? or is this a known bug to other storage backends too?

Its not a known bug.
This is why we would need more information.
Please post over at https://discuss.linuxcontainers.org/ with your system details, and steps you perform (after a fresh reboot) to get into this mount state. Then we can go through it together and if necessary reopen this issue or open a new issue.

That is the problem, I don't know the machine gets into this state.
The machine works without issues, until you try to delete it.
There is no specific software running inside the container.
Hence I asked, if any log file maybe gives what you need, so I asked where shall I need to look.
But I will for sure post later.

Thank you. I suggest you start and stop the container repeatedly until it happens.
The LXD logs are stored in /var/snap/lxd/common/lxd/logs/lxd.log


Support alternate data formats in transports - yaml/xml and pluggable

Make sure to also support binary formats.
The content type must be stored in the request, map object on write.
Replace Request::data() with an interface that supports the same methods as the current JsonObject.

There's not enough time for this.


import changes directory

Importing the arrayfire module changes the working directory. For example:
$ python
Python 2.7.13 (default, Apr  4 2017, 08:46:44)
[GCC 4.2.1 Compatible Apple LLVM 8.0.0 (clang-8<IP_ADDRESS>)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import os
>>> print os.getcwd()
/private/tmp
>>> import arrayfire as af
>>> print os.getcwd()
/usr/local/Cellar/python/2.7.13/Frameworks/Python.framework/Versions/2.7/Resources/Python.app/Contents/Resources
>>> print af.get_version()
(3, 3, 2)

This causes problems with any file names specified with a relative path.
I'm running on OS X 10.11 with arrayfire 3.3.2 installed with homebrew and arrayfire-python installed with pip.
I don't see any chdir calls in the module, so I'm not sure where the problem is.

@bhawkins can you try testing out the devel branch and see if it is still an issue?

@bhawkins Hmm, I just tried to reproduce the issue with pip version of arrayfire-python and arrayfire 3.5 and there doesn't seem to be an issue. Would it be possible to download arrayfire 3.5 ?

Trying to download an OS X installer from the website is giving me trouble. When I go to https://arrayfire.com/download-splash/ and click "Login to Download" I'm taken to https://arrayfire.com/login/?redirect_to=https%3A%2F%2Farrayfire.com%2Fdownload which just says, "Get access to the ArrayFire download page by logging into your ArrayFire account. It's that easy."  But I'm already logged in, and I can view my profile, etc. I tried Chrome and Safari with the same result. Is there a workaround? Otherwise I'll have to build from source, which seems like a lot of work to debug what seems like a Python problem.
Anyhow, I played around and think that I've got some issue with my Python install.  Here's my test script:
from __future__ import print_function
import os
print('cwd', os.getcwd())
import arrayfire
print('cwd', os.getcwd())
print('module', arrayfire.__file__)

If I run it using my Homebrew python I get
$ /usr/local/bin/python test_af_pwd.py
cwd /Users/bhawkins/Desktop
cwd /usr/local/Cellar/python/2.7.13/Frameworks/Python.framework/Versions/2.7/Resources/Python.app/Contents/Resources
module /usr/local/lib/python2.7/site-packages/arrayfire/__init__.pyc

If I create a virtualenv that uses the system site packages, I don't change directory:
$ /usr/local/bin/python -m virtualenv --system-site-packages af
New python executable in /Users/bhawkins/Desktop/af/bin/python2.7
Also creating executable in /Users/bhawkins/Desktop/af/bin/python
Installing setuptools, pip, wheel...done.
$ af/bin/python test_af_pwd.py
cwd /Users/bhawkins/Desktop
cwd /Users/bhawkins/Desktop
module /usr/local/Cellar/python/2.7.13/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/arrayfire/__init__.pyc

Although the import path is somehow different, it's the same code because site-packages in the framework directory is a symlink to the one in /usr/local/lib:
$ # same inode?
$ ls -i /usr/local/lib/python2.7/site-packages/arrayfire/__init__.pyc /usr/local/Cellar/python/2.7.13/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/arrayfire/__init__.pyc
60161931 /usr/local/Cellar/python/2.7.13/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/arrayfire/__init__.pyc
60161931 /usr/local/lib/python2.7/site-packages/arrayfire/__init__.pyc
$ # yep here's the symlink
$ ls -l /usr/local/Cellar/python/2.7.13/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages
lrwxr-xr-x  1 bhawkins  admin  54 Apr  5 14:46 /usr/local/Cellar/python/2.7.13/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages -> ../../../../../../../../../lib/python2.7/site-packages

So this seems like a problem with me, pip, and/or homebrew, but I'm clueless as to how this happens  or why I've only noticed it with arrayfire.

@bhawkins The latest installer from the ci can be downloaded from here: http://ci.arrayfire.org/userContent/OSX/
Just do not rely on it to always exist :)
@umar456 @mlloreda can you see what's happening with the downloads ?

@bhawkins can you test this with other packages ? For example numpy installed via pip ?

@bhawkins I'm not able to reproduce the download issue on my end. Can you try the following and update with results:

Log out
Log back in
Visit https://arrayfire.com/download/

@pavanky I updated my test script to the following
import os, sys, imp
print('cwd', os.getcwd())
if len(sys.argv) > 1:
    name = sys.argv[1]
    fp, path, desc = imp.find_module(name)
    imp.load_module(name, fp, path, desc)
    print('cwd', os.getcwd())

I browsed pypi for a recently updated module I don't have and came up with pdfconv. It doesn't change directory on import:
$ pip install pdfconv
Collecting pdfconv
  Downloading pdfconv-1.0.0.zip
Building wheels for collected packages: pdfconv
  Running setup.py bdist_wheel for pdfconv ... done
  Stored in directory: /Users/bhawkins/Library/Caches/pip/wheels/f7/f2/9f/a25590d13b1d4141ef1cb966835d2ab7c178d7d8da5b2bba53
Successfully built pdfconv
Installing collected packages: pdfconv
Successfully installed pdfconv-1.0.0
$ python test_import_pwd.py pdfconv
cwd /Users/bhawkins/Desktop
cwd /Users/bhawkins/Desktop
$ python test_import_pwd.py arrayfire
cwd /Users/bhawkins/Desktop
cwd /usr/local/Cellar/python/2.7.13/Frameworks/Python.framework/Versions/2.7/Resources/Python.app/Contents/Resources

@mlloreda Download seems to work now, though I'm currently on a different network. Thanks.


Nest does not work with alternatives

Consider the following examples:
SmartDocRenderer(max_line_width=5).to_str(
  Cat(
    (Line | Space).join("1 2 3 4 5 6 7 8 9".split())))

Space SmartDocRenderer(max_line_width=7).to_str(
  Space / Space / Nest(2,
    (Line | Space).join("1 2 3 4 5 6 7 8 9".split())))

While the first behaves correctly and gives
1 2 3
4 5 6
7 8 9

the second gives
  1 2 3 4 5 6 7 8 9

This one's basically a duplicate of #4, right?

No the problem here is that within a Nest Alternatives don't seem to work at all. I didn't look at the code of Nest too closely, but it seems to have it's own rendering logic for many things? Maybe it entirely bypasses render_with_lookahead?


NLog badly includes Jetbrains resharper attributes

NLog version: 4.5.10
Platform: .NET Core 2

What is the current result?
When trying to use the UsedImplicitly and PublicAPI attributes in a project I get resharper errors pointing out that these are internal. They worked fine until I tried to add NLog to the project and sure enough, NLog has its own version of these that are internal since #292 (this was fixed in a way to allow it to compile at all). But why the heck does NLog embed these? I couldn't believe my eyes. Please reference the jetbrains.annotations package like everyone else.
What is the expected result?
NLog should not interfere at all in any way with the Jetbrains annotations package.
Did you checked the Internal log?
Not relevant
Please post full exception details (message, stacktrace, inner exceptions)
Not relevant
Are there any workarounds? yes/no
It compiles but I can't get rid of the resharper errors/squiggly lines. A suggested workaround here https://stackoverflow.com/questions/24047368/why-does-nlog-dll-contain-a-jetbrains-annotations-namespace is not possible in this net core project.
Is there a version in which it did work?
Not sure
Can you help us by writing an unit test?
Not applicable

Closing as this is an external issue


Feature request: Chatty on Slack?

I was wondering if Chatty can be made available from within Slack, that could provide a better UX for most of the cases. I am not sure if there are privacy and/or governance aspects that have prevented this, up till now.

That would probably only make sense if the model is fed our internal documentation (such as Confluence) as well. Otherwise the UI Slack doesn't make a lot of sense of ChatGPT, does it?
Or give me some examples of what you'd like to achieve, to better demonstrate its usefulness?
(By the way, feeding our docs to the model would be very helpful, but not in scope of Chatty itself.)


Add BlazorState.Redux library to the list

BlazorState.Redux library allows you to use Redux store with your Blazor apps. The programming style closely resembles React-Redux style, with container components and presentational components.

Thank you for submitting. I moved down the item and removed the stars count (we display this starting with 10 stars).


Automated Package Upgrade

:tada: This PR is included in version 25.3.232 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add Railties/Engines

This adds Railties/Engines to remove as much manual configuration as possible when using flipping in a Rails app.

[x] require 'flipper' sets up middleware
[x] require 'flipper-cloud' configures and mounts app
[x] require 'flipper-ui configures and mounts app skipping since it needs some kind of auth
[x] Doc all the things

This is part of #500 to improve the onboarding experience.

For some reason I'm having a hard time picturing how Flipper.configure in an initializer would get things setup soon enough for the rails engine to pickup the changes. But I don't know about load order for stuff like that. Have you tried it in a rails app with different memoize options and stuff?

Thanks for bringing this up. I thought based these docs in rails/application.rb that the Railtie initializers would run after config/initializers/*:

The application is also responsible for setting up and executing the booting
process. From the moment you require "config/application.rb" in your app,
the booting process goes like this:

require "config/boot.rb" to set up load paths
require railties and engines
Define Rails.application as "class MyApp::Application < Rails::Application"
Run config.before_configuration callbacks
Load config/environments/ENV.rb
Run config.before_initialize callbacks
Run Railtie#initializer defined by railties, engines and application.
One by one, each engine sets up its load paths, routes and runs its config/initializers/* files.
Custom Railtie#initializers added by railties, engines and applications are executed
Build the middleware stack and run to_prepare callbacks
Run config.before_eager_load and eager_load! if eager_load is true
Run config.after_initialize callbacks

However, I tested it in a real app and that was not the case. So I've updated the Railtie to explicitly run after config/initializers/*, and I've updated the cloud engine to mount the route after initialize.

@jnunemaker I tested this in a real app and had to rework configuration a little. Before 2bf2e82, Cloud::Engine would instantiate Flipper.instance to decide if the webhooks app should be mounted. This would cause it to attempt an HTTP request to sync on app boot, which is not ideal because it not only slows down the boot process, but could prevent the app from booting if an error occurred.
So as a result of cleaning some of that up, I ended up going full circle on the discussion about configuration and moved a few things back to Rails.application.config. These are only configuration that relate to this Rails integration (env_key, memoize, preload, and cloud_path). Configuring Flipper adapters still happens in Flipper.configuration. This feels better to me since they really only apply for the Rails integration.

This AMAZING. Very excited.
I don't want to block shipping this, but some other things that would be nice to wrap up in rails config eventually:

enabling instrumentation with ActiveSupport::Notifications for cloud and default. I usually use these. Probably good on by default and config to disable ala memoize.
log flipper calls in rails log require "flipper/instrumentation/log_subscriber" if ENV.fetch("FLIPPER_LOG", "0").to_i == 1. This is usually something that I protect with env var but it could be on in dev or on based on config. It would need to set instrumented for dsl instance as well.

I should be done kicking the tires today. Tested a couple apps. I just want to check all the config with the flipper cloud default rails app to ensure things are working.

ooh, good call. 7040006c sets instrumentor in both the Railtie and Cloud::Engine. LogSubscriber already checks if logger.debug?, so seems to me like it's fine to just require in the railtie without any configuration. If someone really wanted to change or disable it, I think they could set Flipper::Instrumentation::LogSubscriber.logger = Logger.new(…) .


[CentralDashboard][Testing] End-to-end test linked as a post-submit test

sibling to https://github.com/kubeflow/kubeflow/pull/4780
fixes https://github.com/kubeflow/kubeflow/issues/4015
About
Added central-dashboard end-to-end test as a postsubmit check within endpoint ready test file. Note:

Should this be a post-submit only test @jlewi?
How can I run this file (seems like I cannot pip install kubeflow.testing like metadata, and I cannot find the relevant docs)? @jlewi or @richardsliu

Meta
/area testing
/area centraldashboard
/area front-end
/priority p0
/assign @avdaredevil
/cc @kwasi

This change is

/cc @kimwnasptd

@jlewi Can we merge this?

@avdaredevil what about my previous question

Since the test depends on IAP, we probably need to run this after the endpoint is ready test to make sure the IAP endpoint is available. Its possible there is a way to do this using pytest but otherwise we may need to modify the Argo graph.

Is there a way to sequence the tests using pytest if not I think you will need to make it a separate step in the argo workflow.

I'd need the following information to fix this:

KF_HOST
CLIENT_ID
SERVICE_ACCOUNT_EMAIL: # The email address for the service account you created @.iam.gserviceaccount.com
SERVICE_ACCOUNT_KEY: # Should be a JSON file downloaded from cloud console service accounts

Meeting Notes:

Plan is to hardcode those values in the test (or still accept them as command line args
Put the gcp_util.iap_is_ready(url) command in the dashboard test to make sure it doesn't run before cluster is ready

/cc @jlewi

To verify the test is working can you enable the test temporarily in the PR? (uncomment the code to skip it). Push it to GitHub to trigger a presubmit run. If the test passes include a link the PR thread as evidence the test is passing.
Then reenable the code to skip the test in presubmit and push it to the cluster.

This is an useful test for functional E2E test
Since we haven't heard back from @avdaredevil , I'll go ahead to add it instead.

@PatrickXYS Feel free to take it over and you can use this as an inference. I will close this PR .


problem in : make boruvka opts bliss lpsolve

C:\pandaPIgrounder\cpddl>make boruvka opts bliss lpsolve
-f was unexpected at this time.
-f was unexpected at this time.
-f was unexpected at this time.
-f was unexpected at this time.
make CC="cc" DEBUG=no -C third-party/boruvka all
process_begin: CreateProcess(NULL, uname, ...) failed.
-f was unexpected at this time.
-f was unexpected at this time.
--exists was unexpected at this time.
-f was unexpected at this time.
-f was unexpected at this time.
-f was unexpected at this time.
--exists was unexpected at this time.
make[1]: Entering directory C:/pandaPIgrounder/cpddl/third-party/boruvka' echo 'int main(int argc, char *arvg[]) { int x = 1; return *(char *)&x; }' >__end.c 'int main(int argc, char *arvg[]) { int x = 1; return *(char *) 'x' is not recognized as an internal or external command, operable program or batch file. make[1]: *** [boruvka/config_endian.h] Error 1 make[1]: Leaving directory C:/pandaPIgrounder/cpddl/third-party/boruvka'
make: *** [boruvka] Error 2

This is not an error in pandaPIgrounder itself. The issue lies in the thrid-party project cppdl that we use. I would advise to contact the author of cpddl (Daniel Fiser) to debug this issue.


Fixes #34 - Phoenix 1.3-rc compatability

credit goes to KazW for leading the effort
addressing the Phoenix alias of :datetime for :naive_datetime source

rebuilding

reopening to restart build

must fix Elixir error


coredump appears in udf development using third-party class libraries

Describe the bug
We noticed the previously updated UDF development  #945 ,In our actual development, we found that when the third-party library is not used, it can work normally. Once the third-party library is introduced, such as "libuuid.so" coredump will appear.
We want to know whether a third-party class library can be used when developing new expressions, and if so, how we should use it.

For implementing new function with third-party lib, a so-called function holder should be used in arrow.
The below PR link should be a good reference to you.
https://github.com/oap-project/arrow/pull/97

@ziyangRen
Is it convenient for you to share the call stack on the coredump? Usually there will be some clues in the call stack. Please also check if all the dependencies are installed on worker nodes.
thanks, -yuan

thanks very much.We have now successfully implemented udf functions.


Performance Improvments

1- changed GetProxy Method In ProxyPool.cs to use a for loop instead of Linq as this is a hot Path and linq is slow
2- commited to use Span in Responce Header Parsing to increase performance and decrease allocated memory from strings.
These 2 changes will lead to good CPM improvement:

I was getting 900 - 1.2 k cpm on that Job.
now 3.5 k .

Hey, did you encounter some issues after this PR? Just wondering why you closed it

no issues really.
I closed it thinking you did not like the changes since it was open for a long time.
sorry for that my just different time zones. my bad.

@RydjGit I was away today ^_^ that's the only reason, I'll merge this in a couple hours


扇形，虚线，带特定标记，有实现Demo吗。。

Sure. I will add the above circular soon to this package. Thanks for supporting