Create labels with color if not exists

Create labels with color is they are not exists in the repo
size labels
verified label
github code for create label: repo.create_label(name='test', color="f29513")
gitlab: need to check.

Done for github.


Cast media with media_play service

Cast music or video to the TV
Need further tests to determine all compatible file types
Example
service: media_player.play_media
{
  "entity_id": "media_player.samsungtv",
  "media_content_type": "url",
  "media_content_id": "FILE_URL",
}

Replace FILE_URL with the url of your file.
I tried to make it work with the camera  stream service but I never managed to play it on the TV. The video codec may not be compatible with samsung TVs. If someone has an idea I still included it in my code.

Looks ok, sorry for the late reply, I'll test it on my 2 TVs and merge it if it works on them. I estimate 1-2 days.

This worked flawlessly for me on my 2 TVs. I tested with:
http://commondatastorage.googleapis.com/gtv-videos-bucket/sample/BigBuckBunny.mp4
From the list:
https://gist.github.com/jsturgis/3b19447b304616f18657

Thanks!

@giefca Regarding your question about codec support. I've actually worked with casting through DLNA, Chromecast, Airplay and browsers a lot.. TV codec support is generally a problem.. it is highly likely that your stream would need to be transcoded / transmuxed in order to play it on the TV.
Is it HLS? Could you make a browser page to play it with a video player and load that URL with the Tizen Browser maybe?


Firmware for compliance testing (RF) (IDFGH-3654)

Hello everyone!
I'm facing a RF test for my project with ESP32 to get a national standard certification by ANATEL (Brazil).
I need to do a few configurations on my device, for each test.
I read the IDF user guide, but i can't found out a way to set the bluetooth modulation between GFSK and 8-DPSK.
It has a kind of default firmware for compliance testing? Or, a way to set bluetooth modulation by API?
Please help me.

I found it at espressif idf tools.
https://www.espressif.com/sites/default/files/tools/ESP32%26ESP8266_RF_Performance_Test_EN.zip

Hi @renanfgpascoal, Did the ESPRFTestTool help you with the certification requested by ANATEL?

do we have the source for the EspRftestTool firmware?
i need it for rf testing in conjunction with other modules enabled

I have the same issues of @mfalconf and @menachg

I have the same issues to set the different types of modulation for Bluetooth Classic, BLE and Wi-Fi  with EspRFTestTool_v3.6 software. Aparently the software not able to choose modultaion. I need this to get certification of my product by ANATEL.
The Certificate of Conformity for ESP32-WROOM-32D in Brazil (ANATEL) show several modulations.

Hello,
@guilhermep-git @menachg @natanael127 @mfalconf can you please share some insights regarding this topic?

Hello @DevxMike,
ESP RF Test Tool worked in my case. It was used by certification laboratory with success.
The ESP RF Test Guide was used as reference.

Hello @guilhermep-git ,
Thank you very much.


Android: Implement poll_request mode (similar to iOS)

Implement this https://ntfy.sh/docs/config/#ios-instant-notifications on Android.
The main benefit (for Android) is that with this, instant delivery can be turned off even for selfhosted servers, if an upstream-base-url is configured, reducing battery consumption to ~0% while protecting message privacy.

Hi, I wanted to ask how is this feature coming along?
I'm asking because on my new phone the ntfy app uses a lot of battery consumption and rn am planing on setting up firebase to mitigate it for now.


Verifying of the document’s meaning in IOPS for SQL managed instance

[Enter feedback here]
I'd like to confirm 3 questions about the document of IOPS.
According to the document (https://docs.microsoft.com/en-us/azure/azure-sql/managed-instance/resource-limits#service-tier-characteristics) in Data/Log IOPS (approximate), General Purpose, "Up to 30-40 K IOPS per instance*, 500 - 7500 per file *Increase file size to get more IOPS",

Which means about "Up to 30-40 K IOPS per instance", 30 - 40,000?, or 30,000 - 40,000?

"Up to 30-40 K IOPS per instance" means that max IOPS per instance is 40 K IOPS, am I right?
For example, if I create 100 x 300 GB files with max 2300 IOPS each, it's simply total 230,000 IOPS (2300 x 100). Can I get 230,000 IOPS per instance? Or I can just get until 40,000 IOPS per instance? Because max IOPS per instance is 40 K IOPS.

Why does the max of IOPS per instance have range of 30-40 K?, why it does not simply say UP to 40 K IOPS per instance?

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

ID: 4dff594a-77ba-dca2-e3ae-f3a7bf0c3e1d
Version Independent ID: b496ce1e-c8e6-8fe4-d41d-6f57606f5034
Content: Resource limits - Azure SQL Managed Instance
Content Source: articles/azure-sql/managed-instance/resource-limits.md
Service: sql-managed-instance
Sub-service: service-overview
GitHub Login: @vladai78
Microsoft Alias: vladiv

Hi, do you have any updates so far?
Thank you,

Hi, @MashaMSFT
I just check the original author of the document (https://docs.microsoft.com/en-us/azure/azure-sql/managed-instance/resource-limits#service-tier-characteristics).
If not, sorry, but if you know something, could you answer my concerns about the document.
Thank you,

Hi @naokitakeda14
Thank you for the feedback. We have removed the confusing part of the wording related to per-instance limits in the table. It looks much cleaner and clearer now - thanks!
All the details and best practices are explained in the "File IO characteristics in General Purpose tier" section of the article - let us know if you have additional questions or comments.

Hi, @vladai78
Thank you for modifying the document.
I still have a concern.
You removed the part of per-instance limits in the table. Which means, max IOPS per instance is not "Up to 30-40 K IOPS per instance".
Hence, for example, if I create 100 x 300 GB files with max 2300 IOPS each, it's simply total 230,000 IOPS (2300 x 100), I can get 230,000 IOPS per instance, isn’t it?

Hi @naokitakeda14 - I've replied to your additional questions via e-mail. Let's continue the conversation in that thread

@vladai78 Thanks for covering this over email.
@naokitakeda14 We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


Pattern with occurrences on aggregation

I have the query like -
"from every e1 = inputStream[ name == 'A']<3>  within 1 min select unionSet(createSet(e1.id)) as id_set, 'rule1' as ruleId insert into outputStream"
I am getting only getting [id1] in id_set, I expect [id1,id2,id3] in id_set, if the pattern support aggregation over within time period.
When I try this with windows.
"from inputStream[name == 'A']#window.time(20 sec) select 'rule2' as ruleId,unionSet(createSet(id)) as id_set insert into outputStream"
I am getting [id1,id2,id3] in id_set.
Is there a way so that  I can get aggregation from the pattern within a time period as we specified in within  clause?

Thanks for raising this.
Siddhi does not support getting e1 as a list as of now.
Here is a workaround to collect all the events as a map, you have to check whether the value of the map is not null when used for future usage.
define stream SweetProductionStream (name string, amount double);

@sink(type='log')
define stream TotalCountStream (a double, b object);

-- Count the incoming events
@info(name='query1')
from every a=SweetProductionStream -> b=SweetProductionStream<2:> -> c=SweetProductionStream[name=='end']
select a.amount as a, map:create(0, b[0].amount ,1,  b[1].amount , 2, b[2].amount ) as b
insert into TotalCountStream;

We will start a discussion on this for a suitable solution.


fix systemd release workflow on main

we were missing permissions for this workflow. This commit fixes the same.
Purpose of PR?:
Fixes #
Does this PR introduce a breaking change?
If the changes in this PR are manually verified, list down the scenarios covered::
Additional information for reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs
Checklist:

[ ] Bug fix. Fixes #
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] PR Title follows the convention of  <type>(<scope>): <subject>
[ ] Commit has unit tests
[ ] Commit has integration tests

Can we check the warnings under Token Permissions in https://securityscorecards.dev/viewer/?uri=github.com/kubearmor/KubeArmor
And handle them in this PR as well?
Thanks for looking into this

I think permissions were already set to restrictive levels via https://github.com/kubearmor/KubeArmor/pull/1623
For release workflows, we will require write permissions.


[Bug]: Loxone AC Controle only showing temperature and on/off

Describe the bug
Hello
I'm not sure this is actually a bug or me configuring not correctly. I have a Daikin AC unit which I'm trying to link to loxone via HomeAssistant.
 It's unclear why I can't control any of the other options? These options are available in the Hass-Daikin-integration.
Regards
Firmware of your Miniserver
<IP_ADDRESS>
HomeAssistant install method
Hassio
Version of HomeAssistant
2024.1.6
Version of Pyloxone
0.6.1
Update pyloxone
already the latest version
Log
Settings for the log can be found on the main page under the section Log Configuration

@MaartenProvo sorry but I I have not implemented (and I do not use) the AC Modul.


Draft for OLS-7 and OLS-8 flyer

Good day @bebatut
Is this issue still available and can a description be given, can I be assigned to it

There is no issue there. I already did the work there


Update documentation on the website to deploy Forseti on GKE

User was not able to do the base installation of Forseti on-GKE by referring the docs on the website.

Roles and API has been automated. Documentation has been updated; Awaiting review.


Update deps to cranelift 0.97 and peg 0.8

The tutorial requires minor changes to compile with the current version of cranelift (0.97).
cranelift_module::DataContext does not seem to exist anymore. It used to wrap a DataDescription instance. The latter is now exported via cranelift_module and using it instead works fine.
This PR makes the necessary changes and also updates the README accordingly.

Resolved conflicts by rebasing for a nice commit history.

Thanks again!


Add rudimentary support for codegen of optional chaining

This doesn't handle foo?.(bar) or foo?.[bar] optional chaining.  That requires changes to the AST and parser.  I'll tackle that in a separate PR.

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

Project coverage is 84.58%. Comparing base (09057df) to head (eb33d02).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #308      +/-   ##
==========================================
+ Coverage   84.57%   84.58%   +0.01%
==========================================
  Files          23       23
  Lines       11450    11458       +8
  Branches     1588     1590       +2
==========================================
+ Hits         9684     9692       +8
  Misses       1766     1766

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


The generate server command sorts the Operations but not the Models.

Problem statement
After the loop creates the array of operations, the array is sorted.  The same is not done for models.  When there are multiple models that fail to generate properly, this results in changes in the reported failures from run to run.
func (a *appGenerator) makeCodegenApp() (GenApp, error) {

    var genMods []GenDefinitions
    for mn, m := range a.Models {
        genMods = append(genMods, *mod)
    }

    var genOps GenOperations
    for on, opp := range a.Operations {
        genOps = append(genOps, op)
    }
    sort.Sort(genOps)

Environment
swagger version: master HEAD

@gregmarr  this one should be fixed by now, right?

Yes, it was fixed by the commit referenced just above.


chore(infra): pin k3d image version

Changes

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#975  👈
#904 : 1 other dependent PR (#920 )
#903
#740 : 1 other dependent PR (#750 )
#949 : 1 other dependent PR (#953 )
#954
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


feat: add vlm docs and simple examples

This PR start to add documentation for visual language models

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

hi @merveenoyan thank you for the review 🙏 I've moved the doc into the tutorial section and addresses your comments above.
Please let me know if I should make any more changes!


Fragment in Activity  use butterknife

increment build  @Bind View Error
throw NullPointerException

@tengfei86 Can you provide any sample codes that can cause this exception? I tested ButterKnife with new @Bind codes added and the result is ok.

I hava a fragment in a Activity ,the Fragment use ButterKnife @Bind a TextView ,throw a exception when use incremental build. But if use ButterKnife in activity use  incremental build is ok.

@tengfei86 I tested ButterKnife with Fragment and the result is ok, too. So, could you provide the sample codes or sample repo that can repeat the crash?

@tengfei86 I figure out this problem.. Fix later.


The SQL bind Dialog can not closed

Prerequisites

[ ] I have checked the FAQ, and issues and found no answer.

What version of pinpoint are you using?
v2.0.1
Describe the bug
The sql bind dialog can not be closed after I open it.
What did you do to trigger the bug?
Steps to reproduce, for example:

use the agent to monitor a app with sql bind methods
View the transaction detail
3, Click the sql bind button

Expected behavior
After I view the sql bind detail, there should one button to close the dialog
Screenshots

Hello, @ChenXiaoTemp  Thank you for the report
But, I've just tested with the latest snapshot. didn't find any problem with closing the dialog
As you can see there is 'x' button on the top right.

Do you think you can check whether this is happening in Local only?

I have same problem,on 13 inch screen.

@binDongKim
care to take a look at this?

@ChenXiaoTemp @linux0x5c
I wasn't able to find any issue about it on my local. Could you two provide your testing environment such as os, browser? Plus, I'd like you to check if any error has been marked on your browser console. If there has, I'd appreciate if you leave them in the comment as well. Thank you.

My browser is chrome 80.0.3987.149
My os is macOS
Actually, I found only click some special button, it occurs, like following:

The close button exist on the right top, but it is very small and covered by the page border

@ChenXiaoTemp
The issue was caused by the incorrect setting of the dialog height, which is resolved by https://github.com/naver/pinpoint/pull/6637. Thanks for your report.


fix(android): skip duplicated Content-Type/Content-Length header to fix tauri-apps/tauri#8857

As title follows, currently Android binding need to pass Content-Type separately when creating WebResourceReponse objects, but the response_headers doesn't skip Content-Type and Content-Length headers, which will be append into auto-generated headers with comma, which causes issue tauri-apps/tauri#8857 (e.g. application/wasm becomes application/wasm, application/wasm)
This PR fixed it by adding single conditional statement to skip these headers inside for loop when putting header into objects.

Thank you


pushing project "Spring.SessionFactoryImplError.Tests" to troubleshoot

Reproduced error described in
https://github.com/spring-projects/spring-net/issues/96#issuecomment-73162888
@sbohlen, please let me know if you have any questions.

merged --- thanks!


Route should accept canDeactivate

Right now the Route type doesn't accept canDeactivate and the https://github.com/router5/router5/blob/master/packages/router5/modules/create-router.js#L117 code here doesn't add it when the route is added. As a declarative router, it would be nice if you could add the canDeactivate function in the same definition as the route.

Feel free to submit a PR.


Add example and notice about task type Stored Procedure

Purpose of the pull request
this pr close: #8019
Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

this pr close: #8019

@hstdream Please resolve conflicts.

@hstdream CI failed, it is seem your have some unused imgs, and some deadlink in your change

Codecov Report

Merging #9651 (e7752c7) into dev (13af2ad) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #9651      +/-   ##
============================================
- Coverage     41.17%   41.13%   -0.04%
+ Complexity     4760     4757       -3
============================================
  Files           855      855
  Lines         34621    34621
  Branches       3826     3826
============================================
- Hits          14254    14241      -13
- Misses        18991    19003      +12
- Partials       1376     1377       +1

Impacted Files
Coverage Δ

...r/plugin/registry/zookeeper/ZookeeperRegistry.java
48.21% <0.00%> (-8.04%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
50.70% <0.00%> (-2.82%)
:arrow_down:

Continue to review full report at Codecov.

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

Hi @hstdream CI still failed, you should remove the images '/img/tasks/demo/procedure_task1.png', '/img/procedure-en.png', '/img/procedure_edit.png', '/img/tasks/demo/procedure_task2.png'  in img directory.
And change screenshot language from Chinese to English

I close this pr due to already implemented in dev branch.


elm publish does not require the bumped elm.json to be pushed to GitHub

My process for publishing a new version of a package is as follows (in Elm 0.19):

Make changes and test them
elm bump
git commit -am "Bump version from xxx to yyy"; git push
elm publish
Execute the two lines printed about updating the tags on GitHub (I have an Emacs keyboard macro that copies one of those lines down in my shell buffer).
elm publish

Sometimes I combine the last changes with the elm bump in a single commit.
This is all well and good, except sometimes I forgot to git push the elm.json with the bumped version number (in step 3). elm publish does not complain, and I end up with a new tag on GitHub with an elm.json containing the previous version number.
This doesn't appear to cause any real problems, but it's annoying. elm publish should error if the elm.json on GitHub doesn't match the local copy.

Somewhat related, I forgot to git add the new elm.json before committing along with other changes, so again elm.json didn't end up on GitHub.

As I understand it you don't have to push any changes to github to publish your local changes.
I have experience of upgrading a package to 0.19, just to find out that the owner had already done it - and published, but had not pushed to github.
It was not a ton of work, so it's not a big deal that way. I'm just surprised something like that was even possible.


Hard Clue Scroll Error ( Search box underground)

With this clue, it shows all the boxes within Yanille to be searched but the only box that needs to be searched is underground. The arrow is also pointing in the wrong way.

Fixed in #1707


Add { withFileTypes: boolean } options type on compiler.inputFileSystem.readdir

Today compiler.inputFileSystem.readdir doesn't include the type information for passing {withFileTypes: boolean} options parameter. This is useful if you are leveraging inputFileSystem for and you want to read Dirint objects instead of string | buffer types.
This is low hanging fruit that anyone could contribute to. Location to code: https://github.com/webpack/webpack/blob/main/lib/util/fs.js#L113-L205

In addition we should try and flesh out the fs API as much as possible. We could look at the graceful-fs types or Node@x types and try to maintain compatibility across node versions.

Need to also update how enhanced-resolve handles these functions (not just a type update).

"I am a new github user, and I want to try to solve this problem and strive to meet the PR standards.". If there is anything that does not meet the requirements, I hope to receive your correction and help. Thank you.  @TheLarkInn
In addition, I have recently been learning the enhancement resolver library. I feel that this library is great, and I very much hope to make some modest contributions to it. The following content is related to my learning notes, which I have placed in some technical communities.
01
02

https://github.com/webpack/enhanced-resolve/pull/374
I think all that remains for this is to bump enhanced-resolve in webpack. @fu1996 we are doing a release of enhanced-resolve later today and someone can open a PR for updating.

Fixed


Error Download when set directory

Thank you for reporting this issue
Does the error still occur?
could you please provide the steps you followed to reproduce the exception?

thanks, it worked fine


benefits layout broken

I think this used to be 3 items per row. somehow the layout is broken to be 4 items in the first row and 2 in the second one

https://github.com/datavisyn/datavisyn.github.io/blob/643b046869511a944bd1391f76b82140499566b3/_sass/_mixins.scss#L2-L13
declare that at screens > 992px an element should use just 25%, there are 6 items, so four int the first one, and 2 in the second
https://github.com/datavisyn/datavisyn.github.io/blob/643b046869511a944bd1391f76b82140499566b3/_sass/jobs.scss#L5
declares that spacing wise the space should be distributed between items -> the two are on the sides and space is between in the second row. space-around or space-evenly would be an option


Naming consistency: BuildpackToml and buildpack_descriptor

We currently have a BuildpackToml type that was renamed from BuildpackDescriptor some time ago. We also have a buildpack_descriptor field in the DetectContext and BuildContext that returns that type.
We should have consistent naming for it. We should either rename the type back to BuildpackDescriptor or change the field name to buildpack_toml.
@schneems made a comment about the fact that it wasnt obvious for him that buildpack_descriptor is the buildpack.toml file. (https://github.com/Malax/libcnb.rs/pull/144#discussion_r751516238)

There are so many overloaded terms in the CNB spec, plus many things without an explicit name - it's quite frustrating.
For example we can't really rename BuildpackToml to BuildpackMetadata, since within the file there is also a metadata table. And similarly, we can't rename it instead to Buildpack, because there's a nested buildpack table too.
I see "Buildpack descriptor" is a term referenced in the docs:
https://github.com/buildpacks/spec/blob/main/buildpack.md#buildpacktoml-toml
...though I agree it's not immediately intuitive.
One option, would be to rename BuildpackToml to Buildpack, and have it accessible from the context via buildpack, and then to avoid the nested buildpack key with the same name, flatten those keys into the top-level during deserialisation, giving something like:
context.buildpack.api
context.buildpack.name
context.buildpack.stacks


psroi_pooling.so not find

(tfgpu) root@00300ef242a2:/workspace/Faster-RCNN-TensorFlow# python ./tools/demo.py -- model ./tools/model/VGGnet_fast_rcnn_iter_70000.ckpt
Traceback (most recent call last):
File "./tools/demo.py", line 13, in
from networks.factory import get_network
File "/workspace/Faster-RCNN-TensorFlow/tools/../lib/networks/init.py", line 8, in
from .VGG16 import VGG16
File "/workspace/Faster-RCNN-TensorFlow/tools/../lib/networks/VGG16.py", line 2, in
from networks.network import Network
File "/workspace/Faster-RCNN-TensorFlow/tools/../lib/networks/network.py", line 5, in
import psroi_pooling_layer.psroi_pooling_op as psroi_pooling_op
File "/workspace/Faster-RCNN-TensorFlow/tools/../lib/psroi_pooling_layer/psroi_pooling_op.py", line 5, in
_psroi_pooling_module = tf.load_op_library(filename)
File "/root/envs/tfgpu/lib/python3.5/site-packages/tensorflow/python/framework/load_library.py", line 56, in load_op_library
lib_handle = py_tf.TF_LoadLibrary(library_filename, status)
File "/root/envs/tfgpu/lib/python3.5/site-packages/tensorflow/python/framework/errors_impl.py", line 473, in exit
c_api.TF_GetCode(self.status.status))
tensorflow.python.framework.errors_impl.NotFoundError: /workspace/Faster-RCNN-TensorFlow/tools/../lib/psroi_pooling_layer/psroi_pooling.so: cannot open shared object file: No such file or directory
How to solve excuse me

I saw there is some error with psroi_pooling.so not found. Have you tried building a cython module according to the instructions of the README?
ROOT =  the source code directory of Faster-RCNN-TensorFlow
cd ${ROOT}/lib
make

I have this issue temporarily closed. If I can't solve your problem, I will re-open it.

Hi, I'm facing the same issue. How did you solve it?

Hi @cbiras , try to compile CUDA code.


Frequent sphere crash in same interval

Hi,
in interval of 1-2 days, the sphere will randomly crash without any informations given.
Only one think that is shown  sometimes before random crash is this.
Unfortunately, there is nothing more to give u as help info, console shows nothing.
Edit: Just find out that before every crash, there was this error i shown to you no longer than 5 minutes before crash.
20:11:WARNING:77:Client crashed. Account: 'Dex'. Data from Crash Report packet:
20:11:WARNING:P=0,0,0,0, ErrorCode=0xC0000005, Description='CRASH', Offset=0x243C65CE, ClientExe='', ClientVer=<IP_ADDRESS>
20:11:WARNING:Char attached. Last server P=5996,93,0,0

Try to execute it with GDB or activate the cores and get us the back trace.

mmm, dont know what both of these means. Also how to activate or execute them.
Maybe i know it with different name :O

He's using sphere on windows, the equivalent would be running the debug
version with the debugger and waiting for the crash
Il lun 29 lug 2019, 10:27 AtillaG1<EMAIL_ADDRESS>ha scritto:

mmm, dont know what both of these means. Also how to activate or execute
them.
Maybe i know it with different name :O
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Sphereserver/Source-experimental/issues/306?email_source=notifications&email_token=ADS2CJEXMTTSBRAK7LUVTWDQB2SY3A5CNFSM4IHPF2YKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD277MYA#issuecomment-515896928,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADS2CJEDTJYK3P2KWMVNL3TQB2SY3ANCNFSM4IHPF2YA
.

Ok thats better, will try it. Ty.

first stoppage
looks like cant get through...

today it crashed again and nothing suspicious to see from logs. Will run debugs again and will see.

https://pastebin.com/0swj7Uu0 dropped twice in 2 hours, last thing from me :D

Closing it, since we couldn't find the source of the crash and i don't know if this is still happening (plus, i fixed some code which could generate exceptions or crashes). Feel free to reopen this if there are news.


Support reading block based log format

PR Description
For closing #246, this pr is set. This pr is used for building basics for the new DIO feature, including:

[New] DataLayout, representing the arrangement on data (records), in alignment or not.
Refactor LogFileFormat / LogFileContext.
Add extra support in stress tool for the new alignment-mode reading.

More uts for covering the abnormal code paths wait to be supplemented.

The pr should hold.
Currently, this design is compatible to aligned records or raw records without alignment. However, it lacks compatibilities to fragmented records in alignment, which shoud be re-considered.

Please hold on.
This pr considers the format of aligned records and raw records without alignment. However, it still lack the compatibility to fragmented records, which should be re-considered.

Please hold on. This pr considers the format of aligned records and raw records without alignment. However, it still lack the compatibility to fragmented records, which should be re-considered.

Please hold on. This pr considers the format of aligned records and raw records without alignment. However, it still lack the compatibility to fragmented records, which should be re-considered.

Done for it.

Agree!
So I refactor the reading strategy and remove the over-designed structure in the previous commit.

so the data of one log batch should stay contiguous.

About the padding detection, u menthioned before:

the only work is to add a padding detection to the existing reader.
When reading a log batch, first peek(offset, end_of_current_block) and construct header. If this slice turns out to be padded with zeros, then go to the next block.

Thanks for your suggestion, making me find a existed bug in the preivous detection strategy. And this bug has been fixed in this commit.

You also need to test the decoding properly. Since the write flow is not implemented, you need to manually assemble a block-based log file content. Some cases need to be covered:

[ batch padding ]
[ batch1 batch2_header ][ batch2_middle ][ batch2_rest ]
[ batch1 batch2_header_part1 ][ batch2_header_part2 ] (this should fail)
[ batch non_zero_padding ] (fail too)

Got it.

The first two case have been tested in the failpoints/test_engine.rs: test_build_engine_with_aligned_datalayout in the previous commit.
The third abnormal cornor case waited to be supplemented.
About the last corner case, I think it should be passed. Is necessary to check the padding whether it is filled with zeros? non-zero paddings can also be skipped as mentioned above.

Corner cases have been supplemented in this commit.
Thanks for reviewing.


Add Maven dependency resolution to the CLI

This commit adds the ability to resolve dependencies in the Smithy CLI. This allows end users to author models, generate code, and other activities from the Smithy CLI without needing to use tools like Gradle or Maven. This lowers the up-front learning curve of most Smithy use cases to author models and generate code. Writing Java library code, model build plugins, or transforms is out of scope of the Smithy CLI. Such use cases require the use of tools like Gradle or Maven.
To add dependencies, update smithy-build.json with a "maven" key-value pair, which takes "dependencies" and "repositories":
{
    "version": "1.0",
    "maven": {
        "dependencies": ["software.amazon.smithy:smithy-aws-traits:1.26.0"]
    }
}

When no "repositories" are present, the CLI assumes Maven Central. When the repository member is set, there are no defaults.
{
    "version": "1.0",
    "maven": {
        "repositories": [
            {
                "url": "https://example.com",
                "httpCredentials": "${USERNAME}:${PASSWORD}"
            }
        ],
        "dependencies": ["software.amazon.smithy:smithy-aws-traits:1.26.0"]
    }
}

To support dependency resolution in the CLI, this commit also introduces the --version option to display the current version.
smithy --version
1.26.0

The smithy clean command was added since it was very useful to test dependency resolution by removing the classpath cache file to force them to be re-resolved.
smithy clean

Using the Smithy CLI directly means that tools like Gradle aren't automatically detecting the model directory and adding sources. To account for this, the sources property was added to smithy-build.json. It acts like imports, but the models contained in sources are considered part of the model being built rather than just a dependency (i.e., the sources plugin will contain these models).
{
    "version": "1.0",
    "sources": ["model"]
}

Other changes:

This commit also prints out more useful exceptions when --stacktrace is on since wrapping things in a DependencyRunnable obscured the underlying issue.
To actually test maven resolution, this commit also adds integration tests for the CLI (./gradlew :smithy-cli:integ).
--disable-dependencies can be passed to make the CLI not resolve dependencies. This is useful when using the CLI with other tools that are responsible for resolving dependencies (e.g., Gradle).
This commit attempts to make GitHub actions run the CLI integration tests when using Java 11+ (not 8 since it relies on using jlink to build the CLI).
The binaries created by the CLI were renamed to match the naming conventions of other popular projects
The CLI wrapper bash script was updated to suppress JAVA_OPTS notifications and to improve its performance. The CLI wrapper script is based on the wrapper used by Gradle.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Converted t draft while I work out the shadow plugin issues and Smithy Gradle plugin issues.

Note that I think publishing the CLI JAR is still not working correctly (based on testing it with publishToMavenLocal at least)


Arbitrary closing

It turns on for a few minutes and then closes itself. HP is not working. may close at any time.

would like to know how you get it to work, mine always has this "bad process event offset"


chore/fix: add CI for more platforms and targets, fix build error on 32bit systems

This adds more CI workflows (adapted from iroh):

Build and test on mac OS
Build and test on windows
Build and test on android (aarch64 and armv7)
Cross-compile and test on i686 (32bit) linux, armv7 linux, aarch64 linux
Run cargo fmt and cargo clippy checks
Use nextest for running tests (runs test in parallel, much faster)
Runs only fuzz tests on nightly, all others on stable

apart from the CI workflow this

fixes build failures on 32bit systems by using u64 not usize in more places
tries to fix some unused import warnings wrt syncify (but failed, added an allow - will have to be revisited)

All green now!

tries to fix some unused import warnings wrt syncify (but failed, added an allow - will have to be revisited)

This is due to how syncify works: you place those syncify_replace annotations on a use, but they get copied into the non-syncified code (i.e., the original version of the code gets copied completely untouched, retaining those annotations).
The principled solution would be to modify the syncify macro to drop all syncify_replace annotations in its async output. So running a VisitMut over input_mod just before line 27 here. I can add that some time later (might be up to two weeks till I can do proper coding again).

This is due to how syncify works: you place those syncify_replace annotations on a use, but they get copied into the non-syncified code (i.e., the original version of the code gets copied completely untouched, retaining those annotations).

Right, makes sense to me now. So for now the allow is the correct solution then, and we can remove it once syncify removes the import (but not a priority IMO).

Re Ord on Entry: the sideloading protocol defines a total order on entries to use (entries "being prior" to others): https://willowprotocol.org/specs/sideloading/index.html#sideload_protocol
The implementation of Ord should implement that order.


Allow 'verbatimModuleSyntax' with transpileModule

Since --verbatimModuleSyntax implies --isolatedModules, it should be allowed in conjunction with transpileModule. However, when both options are specified an error is reported, and transpileModule always explicitly sets isolatedModules with no way to unset the option.
This changes transpileModule to skip any potential redudndant flags when setting defaults for transpileModule.
Fixes #53160

However, when both options are specified an error is reported

I feel like the error should only be reported for verbatimModuleSyntax: true, isolatedModules: false.  This error probably also breaks cases where a base config uses isolatedModules and then something extends it with verbatimModuleSyntax

However, when both options are specified an error is reported

I feel like the error should only be reported for verbatimModuleSyntax: true, isolatedModules: false. This error probably also breaks cases where a base config uses isolatedModules and then something extends it with verbatimModuleSyntax

extends is related to tsconfig.json/tsc, so while your comment may be valid it might be better addressed in a separate issue. transpileModule is a language service API and has little to do with extends.

Yeah, that second sentence was just me musing.  The point was the first sentence.  Having both verbatimModuleSyntax and isolatedModules set is not contradictory so probably shouldn't be an error at all (in which case this PR would be unnecessary).

Yes, it feels like there's not exactly a ton of "harm" - you can set checkJs with allowJs too, though checkJs didn't also imply allowJS previously.

Does anyone have any idea why we're getting all these target=ES3 issues in the tests?

Probably some unit test default still set to ES3?

Probably some unit test default still set to ES3?

Yep, in the test's transpilesCorrectly:
            if (transpileOptions.compilerOptions.target === undefined) {
                transpileOptions.compilerOptions.target = ts.ScriptTarget.ES3;
            }

That probably should have been changed when we changed the default, or something. Hard to say when some tests surely tested things about ES3 support, but, we're not going to change that.
Anyway, probably too late to change this as it'd break cherry-picks.

I don't understand, why didn't all these tests break before?

They did, this PR just adds another column to the settings matrix: https://github.com/microsoft/TypeScript/pull/53240/files#diff-15a7ae1852c1cd6c50a9bf7a2453379ef7fce45691dabc656f157272069782b3R94


Corrected

@Akshit312 Thanks for the PR.
Kindly resolve the conflict.

@Akshit312 Thanks for the PR.
There is some conflict in your PR.
Kindly resolve it and send new PR.
Hope to see the further contribution from you.


Place to work on adding a feedback link

Fixes #742

Waiting on some design of where to integrate into the site.

@mikegehard With #937 merged and #918 in progress, are we ok to close this?

Thanks for the heads up @BrianMMcClain. Is there a place in staging or prod that I can check it out to confirm how it looks?

@mikegehard for #937 it should be live on every guide/blog/video/etc, basically everything but workshops, which you can see at the bottom of each page:

Unfortunately the WIP workshop overlay is about where it was last time we spoke in #918, and if you'd like to leave this open until that's completed that works for me too :)

Since my last day with the Services Strategy team is 6/30, might be best to close this unless you think the workshop stuff will be done by then?
Curious if there's a way to change the test to "Give us feedback"? That allows someone to both leave a bug report but also leave non-bug feedback on ways to improve the articles.


12.21.2023 - Suggested Topic - potential UASF on CTV and vault?

Are we close to a potential UASF on CTV and vault? Not time sensitive, can wait until future meetings as needed.
Mentioned on most recent Bitcoin.Review #57
https://bitcoin.review/podcast/episode-57/
I believe this is the github they reference in that podcast:
https://github.com/jamesob/simple-ctv-vault

Let's do it!


Add CFG framework

Having a CFG framework would allow for performing nice analyses.

Poke @Jacarte, this is still a WIP, but I'm making progress.

Thanks @satabin, I will start from this point.

Next step is to add graph traversal, and dominator computation. I'll try to make progress on this this weekend.

We want to use Souper directly, so, we also need an SSA transformation. We can work on it.

We want to use Souper directly, so, we also need an SSA transformation. We can work on it.

Sure, but independently, a graph you cannot traverse is not really useful :)

Totally agree :)

For the SSA part, you can build it on top of this framework, but not sure whether I want to include it int the core module. Maybe a SSA module that depends on core, or not included at all, as you can use so many different SSA representations possible.

Inside the slumps module perhaps. We are implementing it in a separated module. Using the SSA there can be the first step.

Agree. We can see in the future, whether it makes sense to integrate it. I'll keep you updated with progress here.


okvslite & co

LGTM

I need the PR GUI to make it possible to add comments. Arguably that is an experimen, to see whether discussing via PR messages works in practice.

The GitHub PR GUI? That's fine by me.


Move AggregateExpr, PhysicalExpr and PhysicalSortExpr to physical-expr-core

Which issue does this PR close?

Closes #9921.
Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

Thanks @jayzhan211  and @mustafasrepo 🚀


Enhance AddCommand

Main Changes

Removal of compulsory fields

User is now able to add student/professor with only just the name field.
Example: add o/professor n/Aaron
This example will add a professor named "Aaron" to the UniBook.

Creating a new group for a module

User can create a new group by using the o/group option.
Example: add o/group n/Study Group m/CS2100
This example will add a group named "Study Group" to Module CS2100.

Adding a student to a group upon creation of student

User can add a student to multiple study groups whilst adding a student to the UniBook.
Format: `add o/student n/NAME [p/PHONE] [e/EMAIL] [t/TAG]... [[m/MODULE] [g/GROUPNAME]...]...
Take note that the following format must be strictly adhered to when trying to add student to a group: m/MODULE [g/GROUPNAME]...
This is to allow the program to know which group in which module to add the student to.
Example: add o/student n/John Doe m/CS2103 g/Project Work m/CS2100 g/Study Group g/Assignment Group
This example will add a student named "John Doe" to Group "Project Work" in CS2103 and Groups "Study Group" and "Assignment Group" in CS2100.

To Do
Add unit tests.

Nice job! Functionality, from my testing, is there, just some refinements to error messages shown to user left to do. Lets wait on others to test as well see if they can find any bugs


Tutorials on setting up HikariCP for beginner

Hi @brettwooldridge , I'm great to see the great performance of your HikariCP. I'm a beginner to JavaWeb and I wish to use HikariCP as the connection pool in my project. Is there any video about the step by step to setup the HikariCP on windows platform using eclipse, mysql and tomcat 7 ?

Sorry, there is no such tutorial.  But if you google "eclipse tomcat HikariCP" you might find something.  I seem to remember seeing a blog or stackoverflow post.

Thank u, @brettwooldridge. I would try it out.


[Feature] Support various scales in RRDBNet

Motivation

Current RRDBNet only supports 4x upsampling.
The parameter "radius" in config file of real_esrgan is wrong.

Modification

Add scale parameter in rrdb_net
Edit config file of real_esrgan for supporting various scales
Edit config file of real_esrgan for fixing some errors (radius=50 -> kernel_size=51)

Hi @anse3832, it seems that linting failed. You can first install pre-commit by pip install pre-commit. Then you can type pre-commit install in the root directory of mmediting.
After that every time you commit something, it will check the format for you. Thank you very much for your contributions and we are almost there!

In addition, unittest also failed. You may try to run the test locally to test whether it has any errors~

I'm a member of the team with anse3832.
I helped his commit's lint & pytest.
Can you check it?

I'm a member of the team with anse3832. I helped his PR, lint and pytest. Can you check it?

Hi @nijkah, thank you very much! I will check it.

Hi @anse3832 @nijkah, I think it is almost okay. Could you help adding one more unittest, where scale=3? We want to cover the case when scale is not 1,2 or 4

@ckkelvinchan
I have some concerns about ESRGAN and Real-ESRGAN

The Real-ESRGAN official code (https://github.com/xinntao/Real-ESRGAN) only works when scale is 1,2 or 4. Therefore, we I'm worried about supporting x3 scale.

The ESRGAN and Real-ESRGAN use the same "RRDBNet", but it works slightly different.

Real-ESRGAN uses pixelunshuffle, while ESRGAN doesn't
Real-ESRGAN always uses nearest upsample twice regardless of scale (1, 2, 4), while ESRGAN use different times of nearest upsample according to scale (once in x2, x3 scale and twice in x4 scale)

Considering above concerns, I suggest newly writing the code for "RRDBNet" in Real-ESRGAN as "RealRRDBNet"

When user wants to use ESRGAN, the code calls "RRDBNet" (it supports x2, x3 and x4 scale)
When user wants to use Real-ESRGAN, the code calls "RealRRDBNet" (it supports x1, x2 and x4 scale)
Is it okay to proceed these works? If you have any suggestions, please let me know.

Thank you for your suggestions! What I meant in the previous reply is that we want to cover the following line of code.

That means we are not going to support x3. We just need to make sure in the unittest that an error will be raised. I checked the official code and it seems that x3 is not supported neither.

@ckkelvinchan
Do you mean that the unittest code should be like below? (it will raise error because of x3)

@ckkelvinchan Do you mean that the unittest code should be like below? (it will raise error because of x3)

When you expect an error, you should use with pytest.raises(). You can follow the code here.
By the way, you do not need to separate RealRRDBNet from RRDBNet. That means, we support only 1x, 2x, and 4x only, and for 1x and 2x, we use an additional pixel-unshuffle.
Thanks again!

@ckkelvinchan
In the official ESRGAN code (https://github.com/xinntao/ESRGAN/blob/master/RRDBNet_arch.py), it only supports x4 scale and it does not use pixelunshuffle.
If it is allowed to use pixelunshuffle in ESRGAN, I did not need to separate the code for ESRGAN and Real-ESRGAN. In this case, the code supports for x1, x2 or x4 scale. (Neither of official ESRGAN or Real-ESRGAN supports x3 scale)

RRDBNet (both in ESRGAN and Real-SRGAN): supports x1, x2 or x4 scale and it uses pixelunshuffle

If I should follow the official code, I think it's better to separate the code.

RRDBNet (in ESRGAN): supports x4 scale and it doesn't use pixelunshuffle
RealRRDBNet (in Real-ESRGAN): supports x1, x2 or x4 scale and it uses pixelunshuffle

Yes it is allowed to use pixel-unshuffle in ESRGAN. Since this modification does not affect the original ESRGAN. We can extend it to support x1 and x2.

Hi @anse3832 it seems that CI failed. Could you help have a look? Thanks again for your help.

@ckkelvinchan I think it works this time. 🙏

Check lint failure.
@anse3832  Can you approve the PR?
https://github.com/anse3832/mmediting/pull/4

Your efforts are very much appreciated. Look forward to working with you next time!

Hi all, it seems that this PR accidentally changes file permissions of some config files from 644 to 755, can anybody help to fix that?

@wangruohui
https://github.com/nijkah/mmediting/commit/0d41d2cea9873c9a142f63f9e89573bda77ec1bc
Is it okay to pull request this branch?

@wangruohui
nijkah@0d41d2c
Is it okay to pull request this branch?

It is fixed in #718 by @wangruohui. Thanks anyways!

Hi @anse3832！First of all, we want to express our gratitude for your significant PR in the MMEditing project. Your contribution is highly appreciated, and we are grateful for your efforts in helping improve this open-source project during your personal time. We believe that many developers will benefit from your PR.
We would also like to invite you to join our Special Interest Group (SIG) private channel on Discord, where you can share your experiences, ideas, and build connections with like-minded peers. To join the SIG channel, simply message moderator— OpenMMLab on Discord or briefly share your open-source contributions in the #introductions channel and we will assist you. Look forward to seeing you there! Join us ：https://discord.gg/UjgXkPWNqA
If you have WeChat account，welcome to join our community on WeChat. You can add our assistant ：openmmlabwx. Please add "mmsig + Github ID" as a remark when adding friends：）
Thank you again for your contribution❤ @anse3832


Could not find 'blazorContextMenu.SetMenuHandlerReference' ('blazorContextMenu' was undefined)

Hi Kristoffer,
I followed the steps for integrating the SVG Editor, but during the runtime I get the error below.
I tried also manually to register the service without success. Could you help?
Best regards,
Nedim.
Microsoft.AspNetCore.Components.WebAssembly.Rendering.WebAssemblyRenderer[100]
Unhandled exception rendering component: Could not find 'blazorContextMenu.SetMenuHandlerReference' ('blazorContextMenu' was undefined).
Error: Could not find 'blazorContextMenu.SetMenuHandlerReference' ('blazorContextMenu' was undefined).
at https://localhost:5002/_framework/blazor.webassembly.js:1:328
at Array.forEach ()
at a.findFunction (https://localhost:5002/_framework/blazor.webassembly.js:1:296)
at _ (https://localhost:5002/_framework/blazor.webassembly.js:1:2442)
at https://localhost:5002/_framework/blazor.webassembly.js:1:3330
at new Promise ()
at Object.beginInvokeJSFromDotNet (https://localhost:5002/_framework/blazor.webassembly.js:1:3311)
at Object.Gt [as invokeJSFromDotNet] (https://localhost:5002/_framework/blazor.webassembly.js:1:62569)
at Object.Ii (https://localhost:5002/_framework/dotnet.7.0.10.vzm6zpl6t5.js:5:71974)
at _mono_wasm_invoke_js_blazor (https://localhost:5002/_framework/dotnet.7.0.10.vzm6zpl6t5.js:14:103886)
Microsoft.JSInterop.JSException: Could not find 'blazorContextMenu.SetMenuHandlerReference' ('blazorContextMenu' was undefined).
Error: Could not find 'blazorContextMenu.SetMenuHandlerReference' ('blazorContextMenu' was undefined).

I will get back to you tomorrow once I'm back from vacation. 😊

Thank you for the fast reply Krostoffer!
I solved the problem. In a new Blazor app it works perfectly. In an "older" existing project I had to checkout Blazor.ContextMenu and reference it as a project in your library and now it works. Thank you for the great software !!!
Best Regards,
Nedim.

Cool that it was solved. 😊 If there is any changes to the README that could have made this process easier for you then feel free to make some pointers. 🙌

Sure, I have just started using it.
I will try to extend it in my fork and implement support for adding elements on left mouse button click in a separate toolbar (the functionality that we get from context menu at the moment).

I have made an example of extension-by-wrapping for the component in this repository which customizes the behaviour of the editor to make a general graph editor. https://github.com/KristofferStrube/Blazor.GraphEditor
I think an approach like that would be better than forking the repository since you will be able to get the benefits of any future improvements or features I create. A feature that I think will improve the performance a lot is virtualization which is described in issue #15 .

ok, I did not quite understand what do you mean. I just need the SVGEditor in one of pages. I can not see how should I use GraphEditor or how should I wrap the SVGEditor in order to be able to get future updates.

You mentioned that you have forked the project, which means that you will have to manually re-merge my repository into your repository to keep getting new updates.
The alternative is to use the NuGet package that is available for the SVGEditor and then add custom elements to the SVGEditor's SupportedElements parameter to add any custom functionality.

I have just created a fresh MudBlazor application with using only nuget package for SVGEditor.
Unfortunatelly I can confirm that the library at this point does not work with MudBlazor:
https://imgur.com/jRDx71G
ColorPicker gets open without selected objects, SVG objects can not be resized or modified.
I can send you source code if you like. If you provide me instructions where to look and what to fix I will be glad to do that right now.
Your project is very important at the moment for all of us and I would be glad if I can help in any way.

I would to love to help you debug if you will share the source code. It looks as though you are missing to add some CSS.

that would be more than great :D
here is the code:
https://github.com/nedimm/SvgEditorMudBlazor
thanks for taking the time!!

I will check it out later today when I get off work.

Hey @nedimm, I found the problem.
You need to add the CSS file for the compiled Scoped CSS of your project, which also includes the scoped styles of my library. To fix that, you need to add the following StyleSheet link in your index.html file:
<link href="SVGEditorMudBlazor.styles.css" rel="stylesheet" />

And I also found that the manual inclusion of the ColorPicker JS file is no longer needed so you can remove this:
<script src="_content/BlazorColorPicker/colorpicker.js"></script>

I have updated the README accordingly with these details.

thank you very much Kristoffer!!
That works great. If I can help you in the future in any way please conntact me. My email is as my family name (at) gmail.com
Thanks once again and good night!

Nice. I would love to follow your progress in using the library. You can contact me on x.com (Twitter), Mastodon, or at i (at) kristoffer-strube.dk if you have any questions.


feat: Add Navbar, Sidebar, Toast

Changes

 Add Navbar, sidebar and toast notifs

Codecov Report

Merging #7 (1ea6c00) into main (80fdea0) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main        #7   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines            2         2
=========================================
  Hits             2         2

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 80fdea0...1ea6c00. Read the comment docs.


Add signal proxying to podman run and attach

Also removes sig-proxy from 'podman create', where is does not make sense.
Fixes #372

Could probably use some tests, but it's also the weekend. I'll get to it sometime on monday.

Note to self: does handling SIGWINCH here conflict with #448 - investigate further

Wouldn't you need it in podman start then?  I thought the podman create was just to make sure that the sig-proxy was setup for start?

@rhatdan I don't see --sig-proxy in the docker create manpage. If we want it in start, I would not be opposed to adding the flag there, though, that's easy enough

Answering myself from earlier - Go will forward copies of signals independently to separate channels from signal.Notify() so it will not interfere with #448

I guess it would have to go into podman start --sig-proxy, then.  Not sure why you would do run and attach without start?

Docker doesn't have it, but there's no technical reason we can't do it for start --attach. I'll add it in.

Added sig-proxy to podman start (though flipped from default-true to default-false, because it conflicts with starting multiple containers). Also added integration tests, which are hopefully passing.

Sig-proxy seems broken on travis. Will investigate tomorrow.

Rebased & force-pushed, let's see if Travis passes now

This fails in containers. Still trying to identify the reason

Seems to have passed tests now?

LGTM

LGTM

Guess it could be a flaky test

All Green Merging.


"allowMultiple" check

Make sure that "allowMultiple" option is respected for every operation

options.multiple was not exactly working as expected. it was still possible to operate on multiple objects in some cases. with the (future) latest commit this should be solved.


New user can get stuck with wallet pointing at mainnet

Description
Yesterday, my cousin had a latest wallet which was complaining about incompatibility with mainnet. He had already generated a vega wallet, and this was opening again to attempt to trade on fairground. The wallet was complaining that it was pointing at mainnet, and did he want to swap versions. Under my instruction, he closed this, went back to the website, and attempted to download and run the non-mainnet wallet, which I believe still pointed at mainnet and wanted him to swap versions.
I've yet to prove that I can repro this, it's possible that he simply downloaded the wrong wallet initially. After that, either he ran the wrong wallet again and I didn't notice, or perhaps the fairground wallet doesn't override existing mainnet config in this scenario?
Steps to Reproduce
Unclear
Expected behavior
Downloading the fairground wallet will add the fairground network to available networks.
Screenshots
If applicable, add screenshots to help explain your problem.
Device details
Desktop (please complete the following information):

Mac (Intel).
?
Latest on 31st Jan.

Add the fairground network into the config so it becomes a suggested network in the dropdown so the users can switch easily to a non-mainnet
We will need to do this for the validator network too
Mainnet should be the specified default at first use, users will need to switch

I think we misunderstood the underlying problem raised by this ticket.
I think it comes from the confusion induced by a mix of the network incompatibility popup and the latest connected network.
First, we have to understand all desktop wallets apps use the same application configuration file.
Second, every time we switch network, the last used network is saved in the config file.
Now, since the last network you've connected to is saved in config (let's say "mainnet1"), when you switch app (the one for compatible with "fairground"), it will tell you this is not compatible. Then you switch to "fairground" network and it works, and saved in the app config.
But then, you switch to the "mainnet1" app, and it raises the incompatibility popup because it tries to connect to "fairground", because that was the last one you've connected to.

We might want to improve the user experience and the network selection in this case. We should probable embed designers in it.
In the light on these elements, the planning for this issue should be revised.

It appears the sketch of the options 2 didn't export nicely. It's not mandatory to understand the solution though.
Wallet version compatibility.pdf

In the end, we will mostly likely opt for a third option, which is to ship 2 different softwares, one for testnets and one for mainnets.


Invalid variable name - SELENIUM_JAR_PATH

Hey,
Thanks for making this action, it's a really clever way to avoid verbose calls of nix develop --command
It seems to be choking on one of the environment variables, but the variable name looks ok.
I've run it a couple more times and it seems consistent.
Here's a link to a run with set -x on for debugging, maybe you have some idea about what's happening?
https://github.com/arichtman/www.richtman.au/actions/runs/7361637456/job/20039185453#step:5:185

I had a similar problem, but with a different variable. I am using a shell hook in my flake to initialize my dependencies. In my case, I am using tfswitch to essentially pin my terraform version and allow for quick switching between versions. When you call tfswitch to change versions it emits a message like switched to terraform version "1.5.7". The quotes in this were causing the nix-develop-gha script to mess up. I just made sure to send outputs to dev/null and it resolved the issue.

Thanks, I'll look into that

I'm glad I left this issue open because I just ran into it myself!!
Now that I can reproduce it, I hope that I can fix it.  In the meantime, to anyone else who sees the error <variable name>: Invalid variable name, the cause is probably the shellHook emitting characters to stdout which throw off my script: quotes, escape characters, possibly emoji.

Thank you @arichtman for your report, and @matttrach for identifying the cause.   The issue is now fixed in the latest release.


DB Navigator does not show functions

Ubuntu 20.04.2 LTS
DBeaver Version <IP_ADDRESS>104042040
PostgreSQL 12.6 (Ubuntu 12.6-0ubuntu<IP_ADDRESS>)

DB Navigator does not show functions anymore, cf. screenshot.
But the query in SQL editor returns many functions defined in schema adressen.
select * from information_schema.routines
where specific_schema='adressen' order by routine_name;

Moreover table and views are not separated in different branches in the navigator tree.

@kaelte , you should switch to Advanced view to see mentioned objects.
It could be done using Connection context menu:

Or in the Connection properties:


.SlideAbove style is not working when used as Left view

.SlideAbove style is not working when used as Left view. When using it as Right view, it's working fine. All  the other styles are working fine.
If you can point me in the right direction i'm also willing to offer a fix in a pull request.
Kind regards

This issue seems to have been resolved. If you are still having trouble, please let me know.

@jvanzummeren , I have an issue where the style isn't working on right menu. Could you please help me ?


ThorFinn - pt

Source name
Thorfinn
Source link
https://thorfinn.xyz/
Source language
Português
Other details
Thema: Madara
Acknowledgements

[X] I have checked that the extension does not already exist by searching the extension listing and verified it does not appear there.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a meaningful title with the source name.
[ ] I have admitted that I am a clown by having checked this box, as I have not read these acknowledgements.
[X] I will fill out all of the requested information in this form.

Site only has 1 manga with no chapters.

Site only has 1 manga with no chapters.

Because they've just launched the site, they'll soon be putting up more

Still- with no chapters it's impossible to make an extension. Create another issue when they have at least 1 chapter if you can't reopen this issue.


[BUG] Default disk registration sets Storage Reserved to 0 instead of using storage-reserved-percentage-for-default-disk value (Engine v2 block type only)

Describe the bug
Following the Configuring Defaults for Nodes and Disks, the automatic disk registration functionality works as expected when adding a new node to a Longhorn cluster.
However, when using Engine v2 block type, the Storage Reserved value for the automatically registered disk is incorrectly set to 0, rather than being calculated based on the storage-reserved-percentage-for-default-disk setting.
    enginev2-disk1:
      Allow Scheduling:    true
      Disk Driver:         auto
      Disk Type:           block
      Eviction Requested:  false
      Path:                /dev/disk/by-id/scsi-3600508b1001c95d27f513bddea3acc4b
      Storage Reserved:    0  <-- always 0
      Tags:

To Reproduce

Configure automatic disk registration as described in the Default Disk and Node Config Guide.
Add a new node to the Longhorn cluster.
Verify that the default disk is automatically registered.
Check the Storage Reserved value for the newly registered disk.

Expected behavior
The Storage Reserved value should be automatically calculated based on the storage-reserved-percentage-for-default-disk setting (default: 30%). For example, if the disk has a total capacity of 1TB, the reserved storage should be set to 300GB (30%).
storage-reserved-percentage-for-default-disk                      30

Support bundle for troubleshooting

Environment

Longhorn version: v1.7.2
Impacted volume (PV):
Installation method (e.g. Rancher Catalog App/Helm/Kubectl):Kubectl
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version:v1.23.15

Number of control plane nodes in the cluster:1
Number of worker nodes in the cluster:12

Node config

OS type and version:RHEL 8.8
Kernel version:5.15.27
CPU per node:48
Memory per node:128G
Disk type (e.g. SSD/NVMe/HDD):SSD
Network bandwidth between the nodes (Gbps):10Gbps

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal):Baremetal
Number of Longhorn volumes in the cluster:2

Additional context

Workaround and Mitigation
Manually updating the Storage Reserved value via the UI.
For the workaround to function correctly, ensure that longhorn/longhorn#9842 is applied beforehand. This ensures that necessary webhook validations are properly handled, avoiding errors during manual updates.

Verified pass on longhorn master(https://github.com/longhorn/longhorn-manager/commit/ea698235173162b388c11c95c689bdd0fc886aa8)

Create a 3 nodes cluster
Taint node1 with NoSchedule
On tainted node, perform below command (chagne the node name accordingly)

kubectl label nodes ip-172-31-37-151 node.longhorn.io/create-default-disk=config
kubectl annotate nodes ip-172-31-37-151 node.longhorn.io/default-disks-config='[{"name":"fast-ssd-disk","path":"/dev/xvdb","allowScheduling":true,"storageReserved":1048576, "diskType": "block"}]'

Deploy longhorn, enable v2 data engine.
Set create-default-disk-labeled-nodes to true
Untaint the node in step 2
New nodes.longhorn.io will added, and the disk added accorgindly. storageReserved value is corrent on block disk.

@derekbit do we need storage reservation for v2? I thought 0 was originally intended. It should be just for v1, as Longhorn disks for v1 are file system based instead of block device.

@derekbit do we need storage reservation for v2? I thought 0 was originally intended. It should be just for v1, as Longhorn disks for v1 are file system based instead of block device.

Originally, we intentionally set it to 0, but there are some use cases for non-zero storage reservation.

I see. However, we might need to explain how it works for v2 to clarify the right expectations from users. For example, Whats the expected behavior when disk space usage for V2 is reach the storage availability (-storage reservation)? Is it the same as V1? Or does Longhorn still allow replica operations (related to size) work as usual until using up the whole disk space?

Is it the same as V1?

It is same as v1, because v1 and v2 are using the same replica scheduling logic.


18179 filter drawer

Adds a functional filter toolbar to the right side of the DataPack page

Coverage remained the same at 88.753% when pulling b43107a3c38beb008596124d9a9d0a7ba3cf0b52 on 18179-FilterDrawer into 834e9d662d12ec94e13123b9920864880dd6ae8c on master.

Changes Unknown when pulling 44486b1a198bab74aa105af60682fd8d9413be74 on 18179-FilterDrawer into ** on master**.

Changes Unknown when pulling a97e971c264d771701d3b59fa50bacbebd228437 on 18179-FilterDrawer into ** on master**.


Adding publicRoutes to class definition

Quick fix for https://github.com/blackbaud/skyux2/issues/431
Still working on having tests butI've spent way too long on it and they're still flaky https://github.com/blackbaud/skyux-builder/pull/89

Codecov Report

Merging #90 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #90   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          23     23
  Lines         529    529
  Branches       59     59
=====================================
  Hits          529    529

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 d29b056...6ce6c2a. Read the comment docs.


Query is returning data in debugger, but .then() is never called

I'm successfully sending the GraphQL queries to my server, the data is returning, Chrome debugger shows the response data in the network view, but my invocation of .then() never gets called.  My function that I pass in to call console.log does not run, and the promise always shows as "pending" even after the call returns.
What could be the issue?

May you create an example on JSFiddle or alike that behaves similar and I
can check out.

I can paste the code here:
`gq = graphql("http://localhost:9999/query");
wb = gq(
'query { worldBorders { gid, name } }');
wb().then(function (r) { console.log(r); }).catch(function (e) { console.log(e); });`
That's basically it.  Chrome's DevTools show that the data is being returned.  A sample:
{"worldBorders":[{"gid":"1","name":"Antigua and Barbuda"},{"gid":"2","name":"Algeria"}...]}
It's all there.  But then() is never called.

I decided to try and debug the library myself.  I'd been using the minified version, and so I grabbed the non-minified version instead.  Lo and behold, trying it out before I even modified it to add debugging statements, it worked.
So, whatever's wrong is wrong with graphql.min.js, and not with graphql.js.

I suppose we could close this issue, if there's nothing more to say about it?


Question about Parameter extrinsicTrans  and transformation betwen imu and lidar

Hi, Tixiao.
From the picture about lidar and imu coordinates from readme,  it is shown that the translation bewtween the origin of lidar and imu is a few centimeters. My question is why you set [0,0,0] for the parameter extrinsicTrans.
Second question is about the formula which transform imu pose to lidar pose, where in imuPreintegration.cpp line 74,75:
gtsam::Pose3 lidar2Imu = gtsam::Pose3(gtsam::Rot3(1, 0, 0, 0), gtsam::Point3(extTrans.x(), extTrans.y(), extTrans.z()));;  In your code, you firstly use imuConverter() to align the axis of the two coordinates, and then use lidar2Imu or imu2Lidar to get that.  I use this formula to transform imupose to lidarpose, where the rotation matrix of T_imu_lidar is identity matrix.
T_lidar0_lidarn = T_lidar_imu * T_imu0_imun * T_imu_lidar
And this is different from the imu2Lidar in your code. Please give me some advice! Thank you.

My question is why you set [0,0,0] for the parameter extrinsicTrans.

I changed the mounting position of the IMU a few times in different datasets. But the positions are always close to the lidar. So I didn't bother changing this param every time when I test different datasets. Strictly speaking, my approach is not ideal. This param needs to be provided for better performance.
After imuConverter(), the measurements of IMU should be in REP frame. So just a translation is needed. I am not sure I understand your question. Are you talking about using multiple imu and lidar here?

My question is why you set [0,0,0] for the parameter extrinsicTrans.

I changed the mounting position of the IMU a few times in different datasets. But the positions are always close to the lidar. So I didn't bother changing this param every time when I test different datasets. Strictly speaking, my approach is not ideal. This param needs to be provided for better performance.
After imuConverter(), the measurements of IMU should be in REP frame. So just a translation is needed. I am not sure I understand your question. Are you talking about using multiple imu and lidar here?

Thank you.  I get the your point about the first question.  Second question is hard to express. T_lidar0_lidarn means the transformation from time n lidar to time 0 lidar.


Update deu.ini

Translated! Also i fixed some spelling errors from SalmonDE.

[x] This PR actually submits something - don't use this system as a messaging service!
[x] This PR isn't duplicated - you can check if it is by scanning the small list - link is on the navigation bar for this repository.
[x] This PR includes appropriate markdown for sections - e.g. code blocks for suggested code.
[x] This PR description and comments in code is understandable - feel free to use your native language to write if you are not comfortable with English.
[x] This PR has a single branch for itself in your fork - don't just commit to the master branch of your repository!
[x] This PR has comments written in clear English - please don't write unreadable comments just for your eyes.

Pull Request description
{Type something here}

Optional information
{Type something here}

Do you update deu.ini now?


How to deploy SPA to deno.com?

I read that I can deploy SPA, but I see that API Reference has no Deno.open.
deployctl check mode.ts says the same, and deno.com as well for such code:
import { urlParse } from 'https://deno.land/x/url_parse/mod.ts';

addEventListener('fetch', (event: FetchEvent) => {
    event.respondWith(handleRequest(event.request));
});

async function handleRequest(request: Request) {
    const okFile = Deno.open(import.meta.url);

    const resultResponse = new Response(null, {
       headers: {
          body: okFile,
        },
    });

    return resultResponse;
}

Could you please tell me how can I serve static files, like style and images?
EDIT: Maybe related to https://github.com/denoland/deployctl/issues/20.

Could you please tell me how can I serve static files, like style and images?

Here's a small guide: https://deno.com/deploy/docs/serve-static-assets

const okFile = Deno.open(import.meta.url);

You cannot use Deno.open on Deploy.  Visit https://deno.com/deploy/docs/runtime-api to checkout supported APIs.

When I try to run mode.ts using deployctl with content:
import { urlParse } from 'https://deno.land/x/url_parse/mod.ts';

addEventListener('fetch', (event: FetchEvent) => {
    event.respondWith(handleRequest(event.request));
});

async function handleRequest(request: Request) {
    return await fetch(import.meta.url);
}

I see:
error: Uncaught (in promise) TypeError: scheme 'file' not supported
    return await fetch(import.meta.url);
                 ^
    at handleError (deno:core/core.js:186:12)
    at jsonOpParseResult (deno:core/core.js:356:9)
    at opSync (deno:core/core.js:259:32)
    at Object.jsonOpSync (deno:core/core.js:372:12)
    at opFetch (deno:op_crates/fetch/26_fetch.js:887:17)
    at sendFetchReq (deno:op_crates/fetch/26_fetch.js:1257:44)
    at fetch (deno:op_crates/fetch/26_fetch.js:1379:31)
    at handleRequest (file:///Users/coderaiser/thecat/mod.ts:8:18)
    at file:///Users/coderaiser/thecat/mod.ts:4:23
    at innerInvokeEventListeners (deno:op_crates/web/02_event.js:750:27)
error: Process exited with code 1

I can't serve static locally for now? When deploying everything works good.

@coderaiser serving local static file is not supported yet, but we are working on it at #23

Thanks for contributing by opening an issue.
This project is being archived because you can now use deno run to run your Deploy scripts locally: https://deno.com/deploy/docs/running-scripts-locally/. As such, I will close this issue.
If you have feedback about Deno Deploy, please open an issue on this repo: https://github.com/denoland/deploy_feedback


Update feed.atom

Docs Build status updates of commit 468c926:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/feed.atom
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


signal stops when calling from javascript

Hello,
As soon as I call the javascript function to start streaming, the signals on GUI completely stops. I am new to javascript, so I don't know if this is my code issue or the wifi shield issue. Attached is my block of code to get the data from the board. I would appreciate any help!
I am using macOS Mojave 10.14.6, OpenBCI_GUI 4.1.5
Thank you.

I wanted to add that streaming works perfectly fine with matlab.

I suppose you try to start streaming data from a board to openbci gui and to your js code at the same time. But it will never work as you expect because board sends data via tcp/udp socket without broadcast/multicast so there can be only one receiver.
If you need the same data in the GUI and in your JS code you should read data directly from a board in the GUI and stream it further using GUI networking widget but this SDK doesnt read data from networking widget this SDK reads data from a board. So you will need to write a code to receive it from scratch

Would you kindly briefly tell me how I can implement receiving the data? Are there JS libraries for signal processing or anything for getting the data from OpenBCI type of GUI? When you say I have to write from 'scratch', does it mean I won't be able to use any JS signal processing libraries? Sorry for asking too many questions, but I really appreciate your help!

Try to use this sdk without GUI first so there will be only one data receiver.
If you wanna stream data from the GUI you can try js bunding for LSL or stream data via socket and read data from socket.
This lib has no signal processing algorithms its only for data acquisition, if JS is not crucial for you I would recommend to use brainflow instead https://brainflow.readthedocs.io/en/stable/
Currently there is no JS binding but there are signal processing API and many other languages. And I will add JS and julia later.

Thank you, @Andrey1994 . Your advice helped a lot!


Remove gallery option in product addition mode

Directly go to the camera, now that we have "Share to Open Food Facts"

Hie @teolemon, I am working on this issue.


Licence

Hey, can you add a Licence to your project?

Hi @Malusia.
There is a LICENSE file in the root directory: https://github.com/michaelkrone/generator-material-app/blob/master/LICENSE
If you plan to use this project it is worth waiting for the upcomming Version 1.0.0, which will use the latest material version and makes heavy use of generator composition.

Thanks a lot!


ApiKey should not be limited to UUID format

Why do we limit users to only UUID format of API keys? It seems we can relax this requirement and allow arbitrary String values, so I would suggest we remove UUID parsing from ApiKey construction:
https://github.com/near/near-jsonrpc-client-rs/blob/76fbf595138ed0f92d53494586f4425f536c940d/src/auth.rs#L14

Why do we limit users to only UUID format of API keys?

Pagoda Console only accepts UUIDv4 strings in that position, but yeah, this could be used outside of Pagoda Console so it makes sense to relax that requirement.


CO PILOT stops working after approximately 2 prompts

VS Code Version: 1.89.1
OS Version: Pop!_OS 22.04 LTS
Logs:

After some 2 prompts the co pilot just stops working, if i send any texts in chat it will be showing generating for indefinite amount of time, co pilot works after i re-start vs code.

What version of the copilot chat extension are you using?

I believe this has been fixed. Please let me know if you still see it


make simphony-env directory name does not hint its a virtual-env or give information about the release.

it would be very helpful to change the default simphony virtualenv directory (currently $HOME/simphony) to reflec the version and the fact it is a virtual env:
simphony-"release tag"-python-env
will also allow installing multiple frameworks at the same time.
The Readme does not mention that these can be set from the command line, so updating the Readme should be fine.

The information is already in the README file!


Apply code formatting config to all source files

#163 introduced a .editorconfig with code formatting rules. Reformat all .cs (and .gd?) files at once. Going forward devs should use this file in their IDE if possible, and format changes accordingly. But first let's settle on what the formatting rules should be.
I recommend we follow Godot's own C# style guide, and Microsoft's as a fallback. Discuss formatting here.

https://github.com/C7-Game/Prototype/issues/143 falls under this issue

I need to revisit the .editorconfig, recently my vscode C# got updated (I assume) because it's giving new suggestions that aren't in the editor config and were not previously defaults:
remove "this" qualifier on member variables/methods/properties when they are not shadowing a local non-member identifier
personally, I don't like this default, I prefer members to be qualified with "this" because it makes code easier to read at the expense of being slightly more verbose to type
use explicit type instead of var
I believe this has been our recommended coding style, but I don't think my editor used to complain
use lowercase identifier for builtin types instead of capitalised (ie. double instead of Double)
new expression can be simplified
instead of Label foo = new Label(); this one recommends Label foo = new(); Personally I like this one, since we're opting to not use var on the lhs, I think this is a nice tradeoff of making code less verbose without, imo, reducing clarity
object initialisation can be simplified
here's a real world example from our codebase:
before:
TerrainType c7Terrain = new TerrainType();
c7Terrain.Key = civTerrainKeyLookup[civ3Index];
c7Terrain.DisplayName = civ3Terrain.Name;
c7Terrain.baseFoodProduction = civ3Terrain.Food;
c7Terrain.baseShieldProduction = civ3Terrain.Shields;
c7Terrain.baseCommerceProduction = civ3Terrain.Commerce;
c7Terrain.movementCost = civ3Terrain.MovementCost;
c7Terrain.allowCities = civ3Terrain.AllowCities != 0;
c7Terrain.defenseBonus = new StrengthBonus {
      description = civ3Terrain.Name,
      amount = civ3Terrain.DefenseBonus / 100.0
};

after:
TerrainType c7Terrain = new TerrainType {
  Key = civTerrainKeyLookup[civ3Index],
  DisplayName = civ3Terrain.Name,
  baseFoodProduction = civ3Terrain.Food,
  baseShieldProduction = civ3Terrain.Shields,
  baseCommerceProduction = civ3Terrain.Commerce,
  movementCost = civ3Terrain.MovementCost,
  allowCities = civ3Terrain.AllowCities != 0,
  defenseBonus = new StrengthBonus {
	  description = civ3Terrain.Name,
	  amount = civ3Terrain.DefenseBonus / 100.0
  }
};

The indentation is messed up here but the idea is to assign fields values inside a member initialiser list that's tacked on to the end of the constructor (in this case, it's the default ctor, but this syntax also works for something like new Foo(buz){Bar = bar};

I definitely agree with using 'this' by default. It also helps with autocomplete when typing. Using 'new()' instead of 'var' seems fair.
Object initialization seems cool but I haven't used it enough to have much of an opinion. It does make it easier to hide setters.


Device Users collection

In GitLab by @jasonpagetas on Jul 9, 2024, 03:40
request to be able to see a list of user accounts stored on the local machine as well as a list of users who have logged onto the machine with last logon date/time. This would ideally come under a users tab and should include the ability to remove a user or log a user off if they are still logged on. This would be a security function so that admins can see who is logging onto machines and at what time and if necessary  log a user off or remove access y removing a local account.

marked this issue as related to #61

added 2m of time spent

marked this issue as blocked by nofusscomputing/projects/ansible/collections/centurion_erp_collection#1

nofusscomputing/projects/ansible/collections/centurion_erp_collection#1 being added as blocking due to the inventory script providing the current state of a machine.

added 30s of time spent

@jasonpagetas, this issue was migrated from gitlab to github. During this migration you were not added as the user whom opened this issue. This message will ensure that you receive further updates on this issue as it progresses.
Further information can be found within #215


Feature Request: Add Mac arm64 M1 / M2 support to docs

Is your feature request related to a problem? Please describe.
Update the adding_devices.md and the hzn_cli.md to include Mac arm64 M1/M2 support
Describe the solution you'd like.
No response
Describe alternatives you've considered
No response
Additional context.
History:

https://github.com/open-horizon/open-horizon.github.io/issues/262
https://github.com/open-horizon/anax/issues/3539

I missed a comment at the bottom
Open Horizon is not yet supported on arm-based Mac devices.


Implement functional test for maas deployments

Implement functional tests for maas deployments:
This will:

Deploy MAAS
Setup host as a LXD HOST in MAAS
Add VMs to LXD HOST
Deploy sunbeam

Force push is rebase on main


[flutter_tools] add canvaskit hot reload integration test

Description
Add a hot reload integration test for canvaskit. Currently skipped due to failures.

Landing this with the retries to kick the tree. We're currently closed on web hot restart tests and I want to make sure they're passing at least some of the time.


nextToken in dynamo client in aws-sdk

Note: If your feature-request is regarding the AWS Amplify Console service, please log it in the
official AWS Amplify Console forum

I saw, resolver templates uses a param nextToken for paginated query/scan in dynamo db. Is there a similar provision if we use dynamo client from aws-sdk directly?

yes, but I recommend using dynamodb's "document interface", which handles pagination for you automatically. See note at the end of this section:

The AWS SDKs handle the low-level DynamoDB responses (including the presence or absence of LastEvaluatedKey), and provide various abstractions for paginating Scan results. For example, the SDK for Java document interface provides java.util.Iterator support, so that you can walk through the results one at a time.

Closing this ticket. Should you have any additions questions/issue about the resolver template please feel free to reopen.


add class "md-container-ignore" to md-input-container labels

Bug, feature request, or proposal:
This is kind of a bug and feature request. Floating labels in md-input-containers do not work when material1 and material2 are used at the same time.
What is the expected behavior?
material1 styles should not apply to material2 components
What is the current behavior?
The styling for material1 md-input-container is interfering with material2s md-input-container
What are the steps to reproduce?
load material 1 and material 2 at the same time. Create a material 2 input container:
<md-input-container>
    <input mdInput placeholder="Title" value="">
</md-input-container>

Which versions of Angular, Material, OS, browsers are affected?
Angular 4.0.3 and Angular Material 2.0.0.-beta.3
Is there anything else we should know?
Adding the class md-container-ignore could fix this issue.

nevermind, I found that I can use the mat prefix to avoid this.


[CT-3379] [Bug] Test failures on Windows

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
Running the tests from the 1.7.1 tag on Windows results in some failures:

Failures from pytest
=========================================================================================================== short test summary info ============================================================================================================
FAILED dbt-core-git/tests/unit/test_cli_flags.py::TestFlags::test_from_dict__build - AssertionError: assert 'some/path' in 'some\\path'
FAILED dbt-core-git/tests/unit/test_parse_manifest.py::TestPartialParse::test_partial_parse_file_path - AssertionError: expected call not found.
FAILED dbt-core-git/tests/unit/test_parser.py::GenericTestParserTest::test_basic - AssertionError: assert {'name': 'test_not_null', 'resource_type': 'macro', 'package_name': 'snowplow', 'path': 'tests/generic\\test_1.sql', 'original_file_pat
h': 'tests/generic\\test_1.sql', 'unique_id': 'macro.snowplow.test_not_null', ...
FAILED dbt-core-git/tests/unit/test_proto_events.py::test_extra_dict_on_event - AssertionError: assert {'ENV_KEY': 'env_value'} == {'env_key': 'env_value'}
FAILED dbt-core-git/tests/unit/test_system_client.py::TestUntarPackage::test_untar_package_empty - PermissionError: [Errno 13] Permission denied: 'C:\\Users\\dev-admin\\AppData\\Local\\Temp\\tmpuy2mum1d\\tmpym1l5424\\my-empty-package.29dzv7b
z2.tar.gz'
FAILED dbt-core-git/tests/unit/test_system_client.py::TestUntarPackage::test_untar_package_success - PermissionError: [Errno 13] Permission denied: 'C:\\Users\\dev-admin\\AppData\\Local\\Temp\\tmpi1nor0mc\\tmpz6r8d2nu\\a8vxu09cp.txt'
===================================================================================== 6 failed, 1137 passed, 6 skipped, 6 deselected, 2 warnings in 42.11s =====================================================================================

Expected Behavior
Tests not to fail
Steps To Reproduce
Run the tests on Windows
Relevant log output
No response
Environment
- OS: Windows
- Python: All
- dbt: 1.7.1

Which database adapter are you using with dbt?
No response
Additional Context
No response

Thanks for reporting this @JeanChristopheMorinPerso!
Did you happen to also try running the tests from the 1.7.0 tag on Windows? If so, did they also fail similarly, or did they pass?

Hi, I have not tried. And while I'm thikning about it, there is also 6 tests that fail on all platforms (test__dependency_list, test__model_materializations, test__two_models_package_ref, test__two_models_simple_ref``, test_setup_event_logger_specify_max_bytesandtest__cwd_no_permissions`).

Hi @QMalcolm , I had to skip 6 tests on all platforms:

test__dependency_list
test__model_materializations
test__two_models_package_ref
test__two_models_simple_ref
test_setup_event_logger_specify_max_bytes
test__cwd_no_permissions

I don't have the error messages at hand unfortunately.
Also, in case it matters, I'm running the tests as part of creating a conda package for dbt, see https://github.com/AnacondaRecipes/dbt-feedstock/blob/main/recipe/meta.yaml#L103-L105.

@JeanChristopheMorinPerso thanks for linking to the recipe to that conda package for dbt!
Do you happen to have any instructions we could follow to try running the recipe at recipe/meta.yaml so we can see the errors?
If there are instructions how to use Grayskull to generate dependencies, that could be helpful too.

@JeanChristopheMorinPerso Do you happen to have any instructions we could follow to try running the recipe at recipe/meta.yaml so we can see the errors ourselves?
Without being able to reproduce this ourselves, we won't know what changes to make.