text:
Update dynamodb to 2.21.34

About this PR
📦 Updates software.amazon.awssdk:dynamodb from 2.21.29 to 2.21.34
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk", artifactId = "dynamodb" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "software.amazon.awssdk", artifactId = "dynamodb" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #346.

repo: zio/zio-dynamodb


text:
Update logback-classic to 1.4.5

Updates ch.qos.logback:logback-classic from 1.2.11 to 1.4.5.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (1.2.11).
You might want to review and update them manually.
docs/changelog.md

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "ch.qos.logback", artifactId = "logback-classic" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "ch.qos.logback", artifactId = "logback-classic" }
}]

labels: test-library-update, early-semver-minor, semver-spec-minor, old-version-remains, commit-count:1

Superseded by #2714.

source: gharchive/pull-request

repo: zio/zio-quill

license_type: permissive

license_source: github-api


text:
rp/mock: go generate fails

When running go generate ./... on in the project root, it fails with:
prog.go:14:2: no required module provides package github.com/zitadel/oidc/pkg/rp; to add it:
        go get github.com/zitadel/oidc/pkg/rp
prog.go:12:2: no required module provides package github.com/golang/mock/mockgen/model: go.mod file not found in current directory or any parent directory; see 'go help modules'
prog.go:14:2: no required module provides package github.com/zitadel/oidc/pkg/rp: go.mod file not found in current directory or any parent directory; see 'go help modules'
2023/02/17 09:44:05 Loading input failed: exit status 1
generate.go:3: running "mockgen": exit status 1

https://github.com/zitadel/oidc/blob/c8d61c0858de6b503e38afd8798437b916044963/pkg/client/rp/mock/generate.go#L3
There are 2 issues here:

The path to the rp package has changed to github.com/zitadel/oidc/pkg/client/rp;
There is no type Verifier in neither the client or rp package;

Does somebody know what happened? What would be the correct type and package path?

@muhlemmer i believe your on the next branch? i quickly had a look at the generator file and managed to run it with:
//go:generate mockgen -package mock -destination ./verifier.mock.go github.com/zitadel/oidc/pkg/client/rp IDTokenVerifier

and as the next branch will be breaking we will need to change the module to v2 as well: github.com/zitadel/oidc/v2

I couldn't find rp.Verifier on main neither. But if it's IDTokenVerifier it makes sense. The blame says that was changed 2 years ago :joy: . Do you even use the mocker anywhere?
Anyway, thanks for the hint and I'll send a quick fix to next.

source: gharchive/issue

authors: livio-a muhlemmer

repo: zitadel/oidc


text:
Compilation failure with rust-nightly because of external_doc feature

https://github.com/zkcrypto/subtle-ng/blob/73b8a16e5aed0ddccecfd3410c9323c6a0cd35cd/src/lib.rs#L12
The current version (2.4.1) fails to compile with rustc 1.54.0-nightly (35fff69d0 2021-06-06)  because the "external_doc" feature has been removed. For more information, see this PR.

Not sure what the best way to fix this is.
Switching to
#![cfg_attr(feature = "nightly", doc = include_str!("../README.md"))]

doesn't work on stable, because the right hand side of the cfg_attr has unstable syntax, even though it's not taken.
This is supposed to be stable in 1.54, but I think that doesn't land for three weeks or so.

authors: camelop hdevalence

repo: zkcrypto/subtle-ng


text:
Linting OCSP responses

In this bug, Apple shared a very thorough list of lints they apply to OCSP responses from their own issuance infrastructure. It would be useful to implement these in an open-source linter. Are the folks here interested in incorporating an OCSP linter into zlint? Do you think it would be better as a separate project?

This came up once in the past #382. I think the consensus is: Yes! We'd love to have them, but we should structure their addition that they don't make running or maintaining existing certificate lints more difficult.

I think this fits into our sets of lints nicely, we could even exclude OCSP/CRL lints by default.

+1 for being able to linting CRLs too

+1 for being able to linting CRLs too

Hi Maintainers,
I would be happy to work on implementing OCSP linting. The question is:

Given that CAB Forum has made OCSP optional in v2.0.1 (SC63)
Please let me know. We're planning to implement something internally which we can think of contribute to zlint.
Let me know if there is interest?

authors: Legrandin cardonator jsha kowshikRoy zakird

repo: zmap/zlint

license_source: bigquery


text:
Clang Format

This PR runs clang-format using the repository rules, except for I've disabled 80 character line counts because blindly running it makes our code much less readable in many places. There is no change in behavior.

Awesome

repo: zmap/zmap


text:
1.7.10 server bug

Most of the mods used by the server are normal, but the Moon is not compatible with the territory Residence4.7.3.1 plugin, restarting the server does not save the territory (home) and territory items on the moon. The MOD version is the latest version of 1.7.10, Server is Thermos-1.7.10-1614-58-server, I hope to be able to solve it, I am not good at English so Google translation, I am sorry, please understand.

Does Residence4.7.3.1 support dimensions other than vanilla?

Is this still a issue? Please comment if it is and i will reopen if if you are unable? We are closing old issues to see what is still relevant and needs to be fixed.

authors: dmodoomsirius huajimi zmaster587

license: mit


text:
Hideout button

What feature would you like added?
I switched to this from MercuryTrade. I find it a better alternative to MercuryTrade. However, MercuryTrade had a constantly present overlay that provided a hideout button. I'd like to see something similar added to this, as I grew quite accustomed to it and think it's a great feature.
Why do you want this feature added?
This quickly allows you to access your hideout by clicking a button. There are quite a few hotkeys already and it can get cluttered. The less hotkeys I have, the better.

Added a hideout button to the menu bar in v0.4.0.

license: MIT


text:
Install homebrew on Ubuntu 20.04

References:

https://www.digitalocean.com/community/tutorials/how-to-install-and-use-homebrew-on-linux

sudo apt install build-essential

curl -fsSL -o install.sh https://raw.githubusercontent.com/Homebrew/install/HEAD/install.sh

(echo; echo 'eval "$(/home/linuxbrew/.linuxbrew/bin/brew shellenv)"') >> /home/zmoog/.profile
eval "$(/home/linuxbrew/.linuxbrew/bin/brew shellenv)"

$ brew doctor
Your system is ready to brew.

Installed!

repo: zmoog/public-notes


text:
我用 使用环信接收语音消息 用react native fs 下载 后 无法播放本地文件

收到的语音消息的URL   url: "https://a1.easemob.com/1124180905177345/yjxm/chatfiles/210c4610-ff4c-11e8-a35e-4399f0ecd2fd"
下载 过程
const downloadDest = ${RNFS.CachesDirectoryPath}/${((Math.random() * 1000) | 0)}.mp3;
const formUrl = url;
const options = {
fromUrl: formUrl,
toFile: downloadDest,
background: true,
begin: (res) => {
console.log('begin', res);
console.log('contentLength:', res.contentLength / 1024 / 1024, 'M');
},
progress: (res) => {
let pro = res.bytesWritten / res.contentLength;
this.setState({
progressNum: pro,
});
},
};
try {
console.log('12121',options);
const ret = RNFS.downloadFile(options);
ret.promise.then(res => {
console.log('语音下载成功success', res);
console.log('文件大下载地址是 downloadDest);
callback(downloadDest)
}).catch(err => {
console.log('err', err);
callback('error')
});
}
catch (e) {
console.log(error);
}
播放 过程
this.downloadFile(this.props.item.url,response=>{
if (response=='error'){
console.warn('下载失败了,请重试');
} else {
console.log('我已经下载熬好了 文件 可以播放了',response)
let localSong = response;
setTimeout(() => {
//file://var/containers/Bundle/Application/76A725C1-78CF-4B02-8CEC-CE2013EA7077/RNProjectPlayground.app/184.mp3'
console.log('我要播放的是',localSong)
this.sound = new Sound(localSong, '', (error) => {
if (error) {
console.warn(error)
} else {
console.log('播放准备')
this.sound.play((success) => {
if (!success) {
Alert.alert('播放失败');
}else {
alert('1')
}
});
if (!this.state.duration) {
this.setState({
duration: Math.floor(this.sound.getDuration())
});
}
}
});
}, 100)
}

你試試：
path: '/var/containers/Bundle/Application/76A725C1-78CF-4B02-8CEC-CE2013EA7077/RNProjectPlayground.app/184.mp3'
我是使用require()取得檔案，在resolveAssetSource()會的到 'file:///usr/.../{filename}.mp3'
我查看RNSoundModule.java的createMediaPlayer function, 發現有使用File檢查檔案是否存在
但File file = new File(fileName);會將我的路徑變為  'file:/usr/.../{filename}.mp3', 導致路徑錯誤無法取得檔案
進一步查看MediaPlayer的setDataSource, 裡面已經有檢查檔案以及轉換路徑，轉換出的路徑是‘/usr/.../{filename}.mp3’，這是正確的
所以我修改了RNSoundModule.java的createMediaPlayer, 移除了'File file = new File(fileName);'與'file.exists();' ，就可以順利的播放

try it:
path: '/var/containers/Bundle/Application/76A725C1-78CF-4B02-8CEC-CE2013EA7077/RNProjectPlayground.app/184.mp3'
I is use require(), and resolveAssetSource is return 'file:///usr/.../{filename}.mp3'
Read the createMediaPlayer of RNSoundModule.java for Android
RNSoundModule.java:
File file = new File(fileName);  // ->  path convert to file:/usr/.../{filename}.mp3'
file.exists(); // -> will return false, will can't play
In setDataSource of MediaPlayer, has check file is exist, and convert path,
(file:///usr/.../{filename}.mp3 - > /usr/.../{filename}.mp3)
I remove  'File file = new File(fileName);'  and 'file.exists();' of RNSoundModule.java , will pass.
You can try it.

tri it:
/var/containers/Bundle/Application/76A725C1-78CF-4B02-8CEC-CE2013EA7077/RNProjectPlayground.app/184.mp3'
Android:
I is use require(), and resolveAssetSource() is return 'file:///..../xxx.mp3'
In createMediaPlayer of RNSoundModule.java,  has check file exist. (File file = new File(fileName);) This will convert path to 'file:/.../xxx.mp3', but this path is fail.
I read setDataSource of MediaPlayer. The setDataSource already have convert path and check exist. But, this convert path is return '/.../xxx.mp3'. This is correct.
So, Maybe you can modify createMediaPlayer of RNSoundModule.java, or replace 'file://'

@loss99
原來您是在iOS上遇到的問題, 我是在Android上遇到的, 上面所說的解法無法幫助到您,
我對iOS的檔案路徑也不太熟悉, 所以沒有辦法幫助到您
也許你可以看一下iOS Native code
Debug這段程式
RCT_EXPORT_METHOD(prepare:(NSString*)fileName
                  withKey:(nonnull NSNumber*)key
                  withOptions:(NSDictionary*)options
                  withCallback:(RCTResponseSenderBlock)callback) {
  NSError* error;
  NSURL* fileNameUrl;
  AVAudioPlayer* player;

  if ([fileName hasPrefix:@"http"]) {
    fileNameUrl = [NSURL URLWithString:fileName];
    NSData* data = [NSData dataWithContentsOfURL:fileNameUrl];
    player = [[AVAudioPlayer alloc] initWithData:data error:&error];
  }
  else if ([fileName hasPrefix:@"ipod-library://"]) {
    fileNameUrl = [NSURL URLWithString:fileName];
    player = [[AVAudioPlayer alloc] initWithContentsOfURL:fileNameUrl error:&error];
  }
  else {
    fileNameUrl = [NSURL URLWithString: fileName];
    player = [[AVAudioPlayer alloc]
              initWithContentsOfURL:fileNameUrl
              error:&error];
  }

  if (player) {
    player.delegate = self;
    player.enableRate = YES;
    [player prepareToPlay];
    [[self playerPool] setObject:player forKey:key];
    callback(@[[NSNull null], @{@"duration": @(player.duration),
                                @"numberOfChannels": @(player.numberOfChannels)}]);
  } else {
    callback(@[RCTJSErrorFromNSError(error)]);
  }
}

authors: WilsWei laihenyi loss99

repo: zmxv/react-native-sound


text:
BUG: rewrite logpdf implementation tot address numerical instability

Addresses numerical instability reported in #80

Codecov Report

Merging #81 (024bc64) into main (da31ac8) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #81   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines          122       122
=========================================
  Hits           122       122

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 da31ac8...024bc64. Read the comment docs.

Codecov Report

Merging #81 (d05534b) into main (da31ac8) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #81   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines          122       122
=========================================
  Hits           122       122

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 da31ac8...d05534b. Read the comment docs.


text:
Additional file Windows path

Hi,
iam having problem to sync files on windows. I documentation is just an example of Linux path so i tried it like this
"syncSettings.additionalFiles": ["C:\\Users\\Username\\.ssh\\config"]
but it does crate file named C and its empty. Does this feature works on Windows or its just syntax wrong?
Thanks

Yes, there is an issue with static paths like C:\\Users\\Username\\.ssh\\config. The filename of the backup should be c--Users-Username-.ssh-config but the : isn't replaced.
But anyway, you should can use "syncSettings.additionalFiles": ["~/.ssh/config"] which will work even if you username is different.

Now it works. Thanks

authors: Johnymalina daiyam

repo: zokugun/vscode-sync-settings


text:
Readme: Limitations section

The reasons behind this limitation (that you mentioned) and further details (possibility of over-stepping the protected page) could be included in the context-rs repository.

Thanks! I'd wait a bit before merging this because I'm currently rewriting large parts of context-rs including it's stack creation methods.
It won't include segmented stacks afterwards either (since Rust by default only supports fixed-sized stacks), but I'd like to see what changes until then… I could for instance only imagine that the default stack size will change to some dynamic value…
I probably should be done within this week. :smile:

Then this one is assigned to @lhecker .

I think it would be better to rewrite this with a link to context::stack::ProtectedFixedSizeStack doc. Something along the lines of "... share the same limitations and safety guarantees of context-rs ... uses ProtectedFixedSizeStack [link] ...". I'm not able to get the words right, so I'll close this for now. Feel free to steal my words/phrases if necessary.

authors: critiqjo lhecker zonyitoo

repo: zonyitoo/coio-rs


text:
bootstrap4 custom form widgets support

Hello, i'm using custom-radio and custom-checkbox in my project, and to support it i had to override FieldRenderer.
https://getbootstrap.com/docs/4.1/components/forms/#checkboxes
Now something changed and i need to adjust my code, maybe it's better to implement support for those widgets in project?
If it would be useful, i could implement it and make pull-request.

I'd also like to use the custom control inputs as widgets. I especially like the Switch from bootstra 4.2.

Support for the custom file input would also be nice.

Yes, this would be very cool indeed. The Django way would be to use the default widget unless another widget is specified. If a form has a Widget that is recognizable as a Bootstrap4Widget, the FieldRenderer could delegate the HTML rendering to the widget?

@dyve
Probably it can be done this way, but i don't see any widget-rendering delegation inside django-bootstrap4.
Right now in my project i'm using custom field renderer with this mixin:
class AwesomeRenderer:
    """
    Render checkboxes for custom wodgets
    https://getbootstrap.com/docs/4.1/components/forms/#checkboxes
    """
    CUSTOM_CONTROL_CLASS = {
        "radio radio-success": "custom-control custom-radio",
        "checkbox": "custom-control custom-checkbox",
    }

    def __init__(self, *args, **kwargs):
        super().__init__(*args, **kwargs)
        if isinstance(self.widget, forms.CheckboxInput):
            self.widget.attrs.update({"class": "custom-control-input"})

    def add_checkbox_label(self, html):
        return html + render_label(
            content=self.field.label,
            label_for=self.field.id_for_label,
            label_title=escape(strip_tags(self.field_help)),
            label_class="custom-control-label",
        )

    def wrap_widget(self, html):
        if isinstance(self.widget, forms.CheckboxInput):
            html = '
{content}
'.format(content=html)
        return html

    def list_to_class(self, html, klass):
        klass = self.CUSTOM_CONTROL_CLASS.get(klass, klass)
        html = super().list_to_class(html, klass)
        html = html.replace('class=""', 'class="custom-control-input"')
        return html

    radio_list_to_class = list_to_class

And i can update FieldRenderer with same logic (using beatyfulsoup for list_to_class), and add option like "use_custom_select/use_custom_radio". (or maybe select_css_class/radio_css_class) if it makes more sense.
What do you think?

https://github.com/last-partizan/django-bootstrap4/commit/834212046106636aded8dcebdd056fcc695d2079
Here is how it looks like.

I'd like custom-select support too. I'd want the API to look like:
{% bootstrap_field form.type show_label=False addon_after_class='' input_class='custom-select' %}

Hello there, I'd really like to see this implemented in django-bootstrap4 and I'm glad someone has something functional.
Here's a list:

[ ] Checkboxes
[ ] Radios
[ ] Switches
[ ] Select
[ ] Range
[ ] File input

Checkboxes, Radios and Select should be fairly easy but I suggest implementing the rest through their own PR.
I like the idea to be able to set this globally, but should we handle an option like custom_element=True on the bootstrap_field tag?
What's the next step after https://github.com/last-partizan/django-bootstrap4/commit/834212046106636aded8dcebdd056fcc695d2079? Would it be enough for a PR?

I'd like custom-select support too. I'd want the API to look like:
{% bootstrap_field form.type show_label=False addon_after_class='' input_class='custom-select' %}
Is this now supported ? I really need input_class paramater too

I'd like custom-select support too. I'd want the API to look like:
{% bootstrap_field form.type show_label=False addon_after_class='' input_class='custom-select' %}
Is this now supported ? I really need input_class paramater too

authors: SebCorbin aidanlister dennohpeter dyve last-partizan r03 sedrubal


text:
[Troubleshooting tip] using "ZLUX_NO_CLUSTER" to control app-server cluster behavior

On the Zowe servers, the component "app-server" has an environment variable "ZLUX_NO_CLUSTER" which controls whether or not it uses cluster mode.
Cluster mode is enabled by default.
When ZLUX_NO_CLUSTER=1 as an env var, you disable cluster mode.
When you do this, you lose these benefits:
Performance under high user count due to lack of redundant workers
Low-downtime when an unexpected exception occurs (only 1 interrupted request vs about 15 seconds downtime)

But, you gain this benefit:
**There is a limitation in node where it cannot handle a network interface going down. This leads to an infinite socket read loop until the process is canceled**

To have cluster mode on:
In v1, do NOT have "ZLUX_NO_CLUSTER" in instance.env
In v2, do NOT have "zowe.environments.ZLUX_NO_CLUSTER" in zowe.yaml
To have cluster mode OFF:
In v1, have "ZLUX_NO_CLUSTER=1" in instance.env
In v2, have "zowe.environments.ZLUX_NO_CLUSTER=1" in zowe.yaml
On IBM's github, they have stated they have a fix for this so that the workaround will no longer be needed. The fix should be within all versions of nodejs 18, plus versions of nodejs post Q1 2023 for v14 and v16.

Hi Sean, is this still valid? @1000TurquoisePogs

Yes this is still valid. I know its been a long time but that just means its been a long time that we've needed it.

@nellietan And this one as well, will you be taking it on?

repo: zowe/docs-site


text:
Composed properties are always generated ignoring "ImplementPropertyType".

I have a document type called "SeoComposition" that is used by a page.
I was trying to override the MetaDescription so that it could fall back to another property in the case that it hadn't been filled in.
I'm using the Visual Studio plugin to generate code.
Umbraco Version is 7.5.3

The generated class would be something like:
public partial class Page: PublishedContentModel, ISeoComposition
{
    ...
    ///<summary>
    /// Meta Description
    ///</summary>
    [ImplementPropertyType("metaDescription")]
    public string MetaDescription
    {
        get { return SeoComposition.GetMetaDescription(this); }
    }
    ...
}

And my custom half is something like:
    public partial class Page
    {
        [ImplementPropertyType("metaDescription")]
        public string MetaDescription
        {
            get
            {
                // if metaDescription is set return that, otherwise return another processed property.
            }
        }

My issue is that with the ImplementPropertyType on the concrete class for a composed property it is generated when running the tool again.  I've seen similar issues both open and closed but I'm not looking to change the interface signature here.  Is this replicatable by someone else?

Hey,
As soon as you declare that you implement a property in a local partial, we should not generate it. Will look into it.

Ive also have this occurring.
Also the scenario of IgnorePropertyType on the overriding partial is not stopping the property from being generated - assuming this is the expected behaviour.
[IgnorePropertyType("metaDescription")]
public partial class Page
{
...

@callumbwhyte I've run into this problem before, turns out I didn't put my partial in App_Data/Models, as soon as I popped it in there it worked.

Better late than never... I am currently looking into this issue/question. What happens is this: because the MetaDescription property is a composition property, I assumed that if you wanted to change its behavior, you would want to do it at composition level.
The SeoComposition class provides a static getter for the property, which is then used by all classes using the composition. If you provide that getter (see below) we'll detect it, we won't generate it, and it will be used by all class using the composition:
public partial class SeoComposition
{
  public static string GetMetaDescription(ISeoComposition that) => ""..."";
}

That should be the proper way to provide your own implementation of a composition property.

Note: you probably want to avoid providing the entire property implementation (via [ImplementPropertyType] on a SeoComposition.MetaDescription property) as that would disable everything we may generate for the property, in all classes using the composition. Just work with the static getter.

That being said.
Once could argue that your usage of the ImplementPropertyType is perfectly valid. Hey, you may want to provide the implementation of the property for that class only. I did not think of that scenario. Going to look at how hard it would be to support it.

Not too hard, going to fix.

authors: SayTen aDotNetDeveloper daniel-chenery zpqrtbnk


text:
fix regexp for aws_key and slack_webhook

Description:
fixing regex patterns that were present in v7.6.1 for AWS and Slack and worked nicely.
Currently deployed v8.2.1 does not identify the compromised Slack Webhooks and AWS Keys.
Checklist:

[ ] Does your PR pass tests?
[ ] Have you written new tests for your changes?
[x] Have you lint your code locally prior to submission?

@amandre good catch, thanks.

authors: amandre zricethezav

repo: zricethezav/gitleaks


text:
About the source of the relationship table between delta e and perception

Hello, I want to know where the table of the relationship between delta e and perception given in your document comes from? Is there any literature supporting the data in the table?

Hi there,
I just updated the guide to reflect the answer: I built the table from my own tests. An authority source hasn't built a similar table, the best I found during research was a document that a printing company developed that listed their own threshold values, but not quite in the same way.
Perception is quite subjective, and changes depending on which dE algo you're using :-)
Thanks for reaching out!

authors: Bubbleinpit zschuessler

repo: zschuessler/DeltaE

license: unlicense


text:
Test Site Speed

Enable http2 and test performance.
Enable gzip compression and test performance.

Tested the performance. Need to more research on it.

repo: zuhairmhtb/ThePortfolioFrontend


text:
Use symbol instead of symbol name in accelerators in menu items

We should use the symbol instead of symbol name in accelerators in menu items.
This way, we don't have to worry about i18n.
In particular, show Ctrl+, instead of Ctrl+Comma for opening Desktop Settings.
Created as a result of discussion here.
Upstream issue here.

@zulipbot add "blocked-upstream" "Type: Enhancement" "help wanted"

repo: zulip/zulip-electron


text:
pin flutter SDK version via git submodule

Introduce a git submodule to pin the version of flutter SDK under
vendor/flutter.
Use direnv to add vendor/flutter/bin to the current PATH so that
subsequent calls to flutter on the shell uses the our pinned
version of flutter.
Update instructions to use pinned version of flutter.
Pin to 18340ea16c of flutter which matches the current lowerbound
version in pubsec.yaml (3.21.0-12.0.pre.26).

In addition, this PR also includes in a separate commit:

ci-flutter-main job which runs on pushes, to validate the build with the latest main commit. (see https://github.com/zulip/zulip-flutter/issues/15#issuecomment-2016195583 for context)

NOTE: Users can still choose to opt-out and use their own version of
flutter. This just provides a recommended and tested version on our
CI.
Remaining TODOs:

[x] Should envrc / direnv do a check to see if the flutter version matches expectations?
[x] Add Android Studio setup instructions

Closes #15

Trade-offs when using git submodule (this PR) compared to using fvm (#577)
Pros

One fewer dependency and installation since it just depends on git

One less thing to learn/break.

Does not depend on fvm which seems a bit clunky
Easy to make contributions to flutter with the submodule setup once you find your way around submodules
Few changes needed to CI and current setup process.
flutter upgrade just works™ , just need to commit the change of what the submodule commit is pointing to, which is simple.
git tells us if the submodule is in the clean state

Cons

Users need to run additional commands to setup submodule (This ends up being similar to commands to fvm, but we're not dealing with fvm's issues)
Somewhat annoying that if you make any changes in the submodule, your work tree will be deemed "dirty". Perhaps a non-issue if there's some git command workaround.

Related https://stackoverflow.com/questions/31871888/what-goes-wrong-when-using-git-worktree-with-git-submodules indicates that there are some incompatibilities with git worktree.

IMO this ends up being cleaner and still meets the goals of #15 .

Thanks! I agree — I think this plan looks good. The main thing it needs is some more instructions, in particular on that Android Studio / IntelliJ setup we discussed around https://github.com/zulip/zulip-flutter/pull/577#issuecomment-2016431596 .
The instructions for upgrading Flutter will also need some playtesting — I suspect the flutter upgrade step will want some kind of adjustment.

One peculiarity I notice, from playing with this, is that git submodule checks out the submodule at a detached HEAD, rather than at a named branch like main, and as a result flutter --version can't tell what channel it's coming from:
$ vendor/flutter/bin/flutter --version
Flutter 3.21.0-12.0.pre.26 • channel [user-branch] • unknown source
[…]

That contrasts with, for my existing main Flutter tree at a slightly older version:
$ ~/n/flutter/flutter/bin/flutter --version
Flutter 3.21.0-12.0.pre.29 • channel main • https://github.com/flutter/flutter.git
[…]

Not sure what-all downstream effects that might have. If it sufficiently confuses things, we might need to find a way to coax git submodule to keep things on a branch called main, rather than on detached HEADs. That or have a little script in tools/ that runs git submodule update --init and then fixes things up (e.g. with git checkout -B main in the submodule), or something like that.
Hmm, in fact, here's one downstream effect:
$ vendor/flutter/bin/flutter upgrade
Unable to upgrade Flutter: Your Flutter checkout is currently not on a release branch.
Use "flutter channel" to switch to an official channel, and retry. Alternatively, re-install Flutter by going to https://flutter.dev/docs/get-started/install.

I think taking flutter upgrade out of the instructions would be a fine solution to that — instead we can say something like git -C vendor/flutter pull --ff-only. We'll just need some kind of instructions that have been playtested to work.

By adding a manual setup step to do git -C vendor/flutter checkout -B main so that the submodule is using a local main branch, we can let flutter it's on the main channel. This only needs to be done once. Subsequently, everything runs as expected with flutter upgrade. I've updated this PR to address that.
I'm now following up on the Android Studio investigation but it does look like manual instructions are needed.
Will updated the description of this PR shortly.

Updated the PR and addressed the comments.

Thanks again @chrisirhc for all your work on this! Sorry I haven't managed to return for another review for a bit — we've just this week gotten past the peak PR volume from GSoC application season.
I'll be going on vacation shortly for the next couple of weeks. I'll aim to pick this back up the week I'm back, as I catch up with things. Based on where it was at my last review, I imagine we'll be pretty close to merge. This will be a very nice improvement to have!

I've also rebased the PR branch, resolving a small merge conflict (foreshadowed at https://github.com/zulip/zulip-flutter/pull/579#discussion_r1540336928 :slightly_smiling_face: ) and updating the pinned version.

Updated the PR and addressed the comments though I haven't squashed the commits yet.
One other thing to consider is to add a post-checkout git hook that runs setup-vendor-flutter whenever user does a checkout.
I don't want to add too much magic or over-scope this work, so leaving that out for now, but it might be something to consider based on feedback/play-testing.
A quick preview of the message if the setup-vendor-flutter fails:
chris@Chris-Slate ~/c/zulip-flutter (chua/test-change-branch) [1]> ./tools/setup-vendor-flutter                                             (base)
Initializing 'vendor/flutter' submodule
running 'git submodule update --init' to initialize it:
----------------------------------------
error: Your local changes to the following files would be overwritten by checkout:
        bin/internal/engine.version
Please commit your changes or stash them before you switch branches.
Aborting
fatal: Unable to checkout 'a46a7fdc3d622908c24c509ae48c874368c7eb12' in submodule path 'vendor/flutter'
----------------------------------------
Failed to initialize the 'vendor/flutter' submodule, please resolve the issues indicated above.

authors: chrisirhc gnprice

repo: zulip/zulip-flutter


text:
Continuous fetching of messages for some narrows

When I tried to reload and switch to narrows with limited messages, I sometimes get the following warning.
Error: API usage exceeded rate limit

short video: https://drive.google.com/open?id=0Bx5B8iZLs6VBMkpkSnhZaHNnUzg

I have had that with quite an older version. Test or rebase branch to the very latest master and see if it happens.
Also just test the latest master to see if any issues.

yeah, the latest commit fixed this error me 👍 , I could repro the error on previous commit 2baf53a761ca939eb985eb6f683b9c74eefb7fcb

repo: zulip/zulip-mobile


text:
Better status bar behavior

Fixes #282

Automated message from Dropbox CLA bot
@borisyankov, it looks like you've already signed the Dropbox CLA. Thanks!

authors: borisyankov smarx


text:
message editing: Close message edit when re-narrowing.

If a user is editing a message, and renarrows (e.g. by clicking on a link in the left sidebar), we should ask the user if they want to cancel their edit or go back to editing. Currently we have the odd behavior of the half-edited message sticking around until the browser reloads.

@zulipbot claim

Was testing something. Don't mind my reference. Sorry for the disruption.

@PhilSk can you comment so zulipbot doesn't unassign you?

@zulipbot I'm working on it

@zulipbot abandon

@zulipbot abandon

authors: Gauravpurohit409 PhilSk lonerz rishig

repo: zulip/zulip


text:
admin/org settings: Cannot scroll through Notifications stream dropdown menu to select options.

Clicking on the Notifications stream option on the Organizations settings page produces a dropdown menu allowing organization administrators to change the default notifications stream for the organization.

However, if there are more than 8 streams that an admin can select, the other options are cut off from the dropdown menu, and the admin needs to rely on the Filter streams input to find the appropriate stream that was cut off. This behavior potentially misleads admins into thinking that the streams that aren't included at the top of the menu aren't possible candidates for notifcations stream.
To resolve this issue, I propose that the dropdown menu either be implemented as a <select> like input or use a scrollbar to scroll through all options.
(This was discovered while I was documenting the Notifications stream option, and the behavior left me highly confused, and I suspect that new admins would also encounter the same issue as well)

Interesting!  Yeah this seems like a bug.  I think adding a scrollbar would probably be reasonable.  @brockwhittaker thoughts?

Pinging @brockwhittaker about this.

Can't reproduce on both firefox as well as chrome on ubuntu. Both are having a scrollbar:
Chrome:

Firefox:

The only way I can scroll through  the menu is if I focus input and press a down arrow key to move through the rows.

Can you try with #6691?

This presumably is the same perfectScrollbar issue that we have everywhere. Any scrollbar that isn't modified to be perfectScrollbar that is inside another perfectScrollbar won't work. Really frustrating.
This is making me feel less and less good about using perfectScrollbar while also trying to write maintainable code for the site, given that every since possible scrolling object must also have a modified scrollbar.

Idunno, on the other hand, normal scrollbars always look bad on Linux and Windows, so arguably we should be moving them all anyway.

@timabbott Still reproducible on my system while rebased on #6691

authors: HarshitOnGitHub brockwhittaker synicalsyntax timabbott


text:
parse_lsb_release: Use /etc/os-release instead of /etc/lsb-release.

Testing Plan:

[ ] See if this covers Debian as well.

The Ubuntu's case is covered in CI.
See also https://chat.zulip.org/#narrow/stream/3-backend/topic/.2Fetc.2Fos-release.20and.20.2Fetc.2Flsb-release and https://github.com/zulip/zulip/pull/11302#issuecomment-523689914 for context.

While there are some simplicity gains, there is also that "ID" becomes less greppable than "DISTRIB_ID".

Meh, parse_os_release is what you’d want to grep for, which seems fine.

Note: I have addressed any changes that are within the scope of this PR.

Don’t we need to uncapitalize a bunch of 'Ubuntu's and 'Debian's now, since ID is always lowercase?

Don’t we need to uncapitalize a bunch of 'Ubuntu's and 'Debian's now, since ID is always lowercase?

There are several hardcoded capitalized Ubuntu/Debian references in several places. I chose to instead capitalize the ID so that the number of lines changed in this PR is minimal.

I count five lines with capitalized Ubuntu/Debian references, which is hardly more than the number of lines you’ve devoted to the code that capitalizes them.

All right, I am more confident changing them now. Coincidentally, they are all in tools/lib/provision.py.

Hmm, yeah, this is actually pretty reasonable squashed.  I'll do a review of the squashed version.

Squashed and made a few changes:

Fixed the nit Anders mentioned
Added a docstring to parse_os_release providing some detail on what one should consider using from it.
Changing create-thumbor-vevn and create-production-venv error messages to no longer mention vendor in a conditional about family.
Moved the conditional block for debian/ubuntu to make more sense.

@andersk can you give this another look before we merge?

Adjusted the VERSION_CODENAME thing and added an explanation, thanks @rht and @andersk !
For follow-up work, do we need to parse /etc/redhat-release on RHEL, or can we use /etc/os-release there as well?  Might be nicer to get rid of that conditional.

We definitely want to use /etc/os-release on RHEL too; @rht opened #13071 for that.
There are also a handful of installer-related shell scripts still using lsb_release that we should switch to /etc/os-release, and then we can skip installing lsb_release entirely to save an apt/yum round trip.

Great, that's be a nice improvement.

authors: andersk rht timabbott


text:
Fix problem with showing subscription settings after renaming stream.

Before fix it was impossible to expand stream's settings section after renaming stream because data for settings was getting by stream name.
Fixes: #1540.

Automated message from Dropbox CLA bot
@TomaszKolek, it looks like you've already signed the Dropbox CLA. Thanks!

I looked into this, and it turns out the root cause of this problem was actually that a bug in the default streams code meant that stream rename updates were crashing partway through the process; in fact the pre-existing code for this page worked fine (though it would perhaps still be better to refactor it to work based on IDs).  Since the actual bug caused other problems, I'm going to proceed with fixing that root cause issue.

For proceeding with this PR, I think the code is better fetching these things by ID rather than name.  But we do fetches by name in like 10 places in this file.  So I think what I'd be excited about is a change the refactors this to consistently fetch by ID (and not by name), e.g.
-        var stream = sub_row.find('.subscription_name').text();
 +        var stream_id = parseInt(sub_row.attr('data-subscription-id'), 10);
 +        var sub = subs.get_sub_by_id(stream_id)
# access stream name in later code via sub.name

which is a slightly different flow than yours, but basically the same idea.  Then we'd no longer be using .subscription_name at all in the code.

@TomaszKolek can you rebase this and fix the tests so it can be reviewed and merged?

@timabbott @TomaszKolek Simply because this PR has gone a couple months without followup, and it's not fresh in our minds, and there have been subsequent change to the code, I think we should abandon this particular PR.  But we should have a conversation on Zulip about tackling this again.  I have moved more logic into stream_data.js, which I think will make it easier for us to start using stream_id as a primary key in more places.

@showell we've fixed this now and we can close it, right?

I'm not sure.

asdf

authors: TomaszKolek showell smarx timabbott


text:
frontend : fix overflowing of alert-word-status-text when the window narrowed

when a window is narrowed the alert-word-status-text overflow out of the alert-success.
Testing plan: Tested locally
GIFs or screenshots:
Before

After

@ritikchauhan-01 can you post an "after" screenshot to show this change working without a reviewer needing to test manually?  Also, it'd be nice if you fixed the commit message to have a shorter title.
Check out the Zulip commit message guidelines for more details: https://zulip.readthedocs.io/en/latest/contributing/version-control.html#commit-messages

@timabbott can you review it.

Merged, but I think structurally we may want this CSS on that class of component.


text:
Default settings rename

Rename MAX_LOGO_FILE_SIZE, MAX_EMOJI_FILE_SIZE, MAX_AVATAR_FILE_SIZE and MAX_ICON_FILE_SIZE to have
suffix _MIB reflecting size in mebibytes. Follow up for #18576.
Testing plan:
Automated tests, manual testings.

@timabbott This is ready. Do we have any further documentation to alter?

This looks great, merged, thanks @ligmitz!
Since these have never been particularly documented settings (they're more internal constants), there's nothing special we need to do when changing them.

authors: ligmitz timabbott


text:
[WIP] provision: Run provision if provision check fails.

This will helps us improve the developer experience
where they don't have to run provsion everytime.

Fixes:

Screenshots and screen captures:
Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

claim

@Riken-Shah do you remember where this idea was discussed? I'd like to bump that thread and decide if we want to integrate this.

authors: Riken-Shah timabbott xiaowz0502


text:
user_topics: Rename muted_topics.js to user_topics.js.

This renames muted_topics.js to user_topics.js as we'll be switching over to using user_topics instead of muted_topics.

Merged, thanks @akashaviator!

authors: akashaviator timabbott


text:
vq1

Fixes:

Screenshots and screen captures:

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Hey, @Rodz88ck congrats on your first contribution to Zulip. :tada:
I recommend that you edit the title and description of the Pull Request to indicate what changes are being made and if it solves an issue tracked on GitHub you can mention that as well.
You can find more info on that in our guide to make Reviewable Pull Requests. Also give a look at our Commit Discipline.

Thanks @Rodz88ck. In case you want to customize the development environment configuration, you can refer to https://zulip.readthedocs.io/en/latest/development/setup-recommended.html#customizing-cpu-and-ram-allocation.
Closing this as there is no clear description of why the changes were made.


text:
stream_settings_ui: Avoid two scrollbars.

Fixes #26823
Fixes user card not displayed on last item in the subscribers list. This was happening because there wan not enough space below the user name for the popover to be displayed (as far as I understand this).
Regardless of the fix above, this seems like a nice change.

@amanagr can you describe the constraints that motivate writing a bunch of JS code here?
Our goal as a project is to delete resize.js -- we want to be doing all this layout stuff in CSS. @karlstolley FYI.

Here is how I understand it:
Main problem here is setting the max-height of the subscribers list table correctly. Right now, the max-height set for the table is not correct and even if we modified the CSS to fix, it would not account for the different width in which the elements above it will wrap, changing the "correct" max-height of the table.
We could use media queries to get the correct max-height for different widths, to give us an all CSS solution (which will need to be updated if we modify the elements there). This CSS solution would still not work well for different languages since the breakpoints at which point the elements wrap would change.
Can we do some changes in the HTML strucutre to give us an all CSS solution here?
Yes, but not sure we will be able to. We will have to make all the elements above the table (including table headers if we want to show them always) sticky and remove max-height from table so that whole container along with the elements above it are scrollable. That would likely work well without any JS required but is hard to implement given our LIstWidget code for the table.

The tricky bit is the resizing, I think we can merge it for 8.0 and wait for Karl to have good solution for this?

Seems like this might also address #26835?

OK, merged, thanks @amanagr! We can plan to rewrite this region using pure CSS later.

authors: amanagr karlstolley timabbott


text:
portico: Create header dropdown with avatar.

This creates a header dropdown with reference to whether a user is logged in that also shows their avatar.

There's some alignment issues in the dropdown:

A few more small issues:

Clicking on the avatar a second time should hide the menu.
The newly expanded section of templates/zerver/portico-header.html and templates/zerver/landing_nav.html  have a nearly-identical section; maybe we should put that into a new template (e.g. portico-navbar-login-corner) that can be shared by the two files?

Updated.

A few bugs:

@brockwhittaker can you fix the lint errors?
The click targets for the buttons seem to just be the text, not the rows; feels broken.

@rishig what do you think about the look here?

Or this with the Discourse logo as the realm icon (picked at random from our integrations):

I feel like we might need some sort of container around the organization icon.  Maybe do a bit of a pill with the icon on the left and the organization name on the right, with some padding around it?

Fixed both of those issues. I can add the organization name pill next.

Merged, after fixing the linter errors.  Thanks @brockwhittaker!  We should add the pill thing this week.

authors: brockwhittaker timabbott


text:
Slow work of off-canvas menu

Found some strange problem, when trying to work with off-canvas menu is very slow and strange in phones with Android even in official site of F4A in examples of-canvas, seems like dev team don't like phones on android ). On ios it's slow too but not so much.

We do test on Android 4 devices, phone and tablet. What device, version of Android, and browser are you using?

I test on HTC One m7 and XAOMI also on Nexus 5 the version of chrome is 40.

I just tried http://foundation.zurb.com/apps/docs/#!/off-canvas on my Nexus 5 and can't reproduce the problem. Feels fluid.
Android 5.0.1
Chrome 40.0.x

@gakimball @escapedcat - same here. Canvas itself works great, but every interactive element on it - button, checkbox, switch, e.t.c. - works strange and slow - for example checkbox "freezes", and I need to touch it 6-7 times to check/uncheck it.
Lenovo P780, android 4.2

authors: escapedcat gakimball ruid0 unkmas

repo: zurb/foundation-apps


text:
Changed 'queues' to 'cues'

In Types of Disabilities | Visual we mean 'cues' not 'queues'.

Description

Simple SPaG fix

Closes

Motivation and Context

Simple SPaG fix
Screenshots (if appropriate):
Types of changes

[x] Documentation
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing
functionality to change)

Checklist (all required):

[x] I have read and follow the CONTRIBUTING document.
[x] There are no other pull request similar to this one.
[x] The pull request title is descriptive.
[x] The template is fully and correctly filled.
[x] The pull request targets the right branch (develop or support/*).
[x] My commits are correctly titled and contain all relevant information.
[x] My code follows the code style of this project.
[x] I have updated the documentation accordingly to my changes (if relevant).
[x] I have added tests to cover my changes (if relevant).
[x] All new and existing tests passed.

Cool beans, thanks Daniel!

repo: zurb/foundation-sites


text:
This fix #9122 by changing z-index for the tooltip.

I changed the z-index to fix the issue with the tooltip inside the reveal.
Could you review if the fix is good? @abdullahsalem

Thanks @isaak-ordonez for this PR.
Setting up the z-index of the Tooltip to 1006  will work fine. However, in your PR, both Tooltip and Reveal elements have the same z-index order -1006-, so from Stacking Context  perspective, the Tooltip will overlap because it comes after the Reveal element in the HTML hierarchy. Therefore, I would suggest that Tooltip takes a greater number than Reveal to avoid any unexpected scenario in the Stacking Context process.

@abdullahsalem  Thanks for the feedback, I made a change to the z-index from 1006 to 1200.

Thanks, @isaak-ordonez !

authors: abdullahsalem isaak-ordonez


text:
🛑 Asset Delivery is down

In 6ca3566, Asset Delivery (https://assetdelivery.roblox.com) was down:

HTTP code: 404
Response time: 112 ms

Resolved: Asset Delivery is back up in c73e268 after 1 hour, 28 minutes.


text:
added spanish translation for movements

hi dude , please have a look to the changes , more changes coming ...

very nice! 👍

repo: zwacky/pokedex-go


text:
ZTS-110 Z Wave Thermostat cant change temp heat or cool

Describe the bug
Unable to change the temp for Heat or Cool on the thermostat. I have tried from the integration, and cards in HA.
ZTS-110 Z Wave Thermostat
I unplugged from the wall and removed batterys to see if that would help. Also removed and readded the node.
Device information
Which device(s) is/are affected (make/model)?
What are the node IDs?
Node Id 4
Last Known Working Configuration
This was working about a week ago on core-2021.1.5 I believe I am on core-2021.20 now
Supervisor-2021.01.7 and Home Assistant OS 5.10
I had Zwave installed for years and started having issues, so I migrated to Zwave_js. I noticed that there are not as many sensors for the device and it seems there is not a fan entity for this thermostat in zwave_js
Installation information
I installed with integration and add-on Z-wave JS
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '...'
Scroll down to '...'
See error

Additional context
When I try to change the temp on the thermostat it just changed back to what it was set at after a few seconds. I looked in the Log from the add-on section and I see this over and over
Error in driver ZWaveError: Thermostat Setpoint: "setpoint" must be further specified by a numeric property key
at Proxy.ThermostatSetpointCCAPI. (/usr/lib/node_modules/@zwave-js/server/node_modules/zwave-js/src/lib/commandclass/ThermostatSetpointCC.ts:162:10)
at ZWaveNode.setValue (/usr/lib/node_modules/@zwave-js/server/node_modules/zwave-js/src/lib/node/Node.ts:667:14)
at Function.handle (/usr/lib/node_modules/@zwave-js/server/dist/lib/node/message_handler.js:15:44)
at Object.node (/usr/lib/node_modules/@zwave-js/server/dist/lib/server.js:30:91)
at Client.receiveMessage (/usr/lib/node_modules/@zwave-js/server/dist/lib/server.js:61:99)
at WebSocket. (/usr/lib/node_modules/@zwave-js/server/dist/lib/server.js:35:45)
at WebSocket.emit (events.js:315:20)
at WebSocket.EventEmitter.emit (domain.js:467:12)
at Receiver.receiverOnMessage (/usr/lib/node_modules/@zwave-js/server/node_modules/ws/lib/websocket.js:825:20)
at Receiver.emit (events.js:315:20) {
code: 35,
context: undefined,
transactionSource: undefined
}
Additionally here is a dump of from the zwave_js integrations page
zwave_js_dump.zip
Logfile:

Looks like HA is trying to use an incomplete value ID (i.e. setpoint for an unspecified thermostat mode)
@marcelveldt

I don't know if someone fixed this but there was an update to zwave_js this morning, ran this and rebooted the server. Now it changed the temp and stays changed. Not sure if there is still any backend issues but it seems to be resolved on my end.

Thanks, closing this - feel free to reopen if the issue persists.

repo: zwave-js/node-zwave-js


text:
Duewi 05437 - ZW ZS 3500 not pairing with Aeotec ZST10-700

Checklist
I am not using Home Assistant. Or: a developer has told me to come here.
I have checked the troubleshooting section and my problem is not described there.
I have read the changelog and my problem is not mentioned there.

Deploy method : Docker
Z-Wave JS UI version : 8.14.2.c234053
ZwaveJS version : 10.16.0
Firmware Aeotec ZST10-700 : FW: v7.19.2
Describe the bug : Duewi 05437 - ZW ZS 3500 not pairing with Aeotec ZST10-700, the timer in zwavejs just keeps counting down to 0
It does pair with a Aeotec ZW090
To Reproduce : Try to pair ZW ZS 3500 on a Aeotec ZST10-700
Expected behavior : Succesfull pairing of the Duewi 05437 - ZW ZS 3500

Please check that the RF region is set correctly. There were some issues where that magically changed after firmware updates:

If that doesn't help, I'm afraid you're out of options. The EU version of the Aeotec Z-Stick 7 has measurable RF performance issues - maybe the Z-Pi 7 (GPIO module) does too.

RF Region was set to Europe.
And the performance issues, that could be, but why does a Fibaro Plug in the same wall socket pair without problems.
I even had battery powered motion sensors connected 15 meters away wich worked without problems.
Connecting the Deuwi to my  Z‐Stick Gen5 USB Controller works flawlessly.
Also, the Deuwi is the only device that won't connect (i have over 60 z-wave devices)

Connecting the Deuwi to my Z‐Stick Gen5 USB Controller works flawlessly

The Gen5 definitely has a much better antenna.

why does a Fibaro Plug in the same wall socket pair without problems

Maybe the Fibaro has a better antenna than the Deuwi.

Just to be sure, did you exclude the Deuwi from the other network before trying to include it into the new one?
yes, multiple times to be sure.
I’ll try to pair the Deuwi right next to the Z-Pi 7 to rule out the antenna and RF problem.


text:
fix(config): corrected partial paramaters in Philio Technology Corp PST02-C

closes #2575

@zwave-js-bot fix lint

I have some more labels to edit however interested on thoughts on this.  The manual isn't clear if these are actually double negatives or not.  An example: Disable send out BASIC OFF after door closed. (1:Disable, 0:Enable) So do you enable it to disable send basic off?   I assume this is defaulted to 1 due to the order as well.  I have a few assumptions I have to make when reading this manual.

I think this configures whether a Basic Set is sent or not, with the default being "No".
I don't think these are double negatives, just bad labels.

authors: AlCalzone cdnninja


text:
feat: optimize order of node communication during startup

Prior to this PR, communication with nodes on startup was sequenced purely by node ID, which leads to unresponsive devices delaying the "ready" event for responsive devices.
We now use the following heuristic to prioritize devices during this phase:

Fully-interviewed devices come first (often just need a ping or nothing at all)
Always listening devices come first, then FLiRS, then sleeping devices
Among those, the order is determined by the lastSeen flag, more recently active devices come first
Lastly, we sort ascending by node ID

This should ensure that fully-interviewed responsive devices are ready almost immediately on startup, even if they have a high node ID.

@zwave-js-bot automerge


text:
Update Aeotec Nano Dimmer firmware not working

I've downloaded the EU version of the latest Aeotec Nano Dimmer (v2.03) from aeotecs support site. Changed the name of the .ex_ file to .exe.
When I try to update the old device (v1.2?) I get the message: Error while calling api beginFirmwareUpdate: Unable to extract firmware from file: This does not appear to be a valid Aeotec updater (invalid firmware length)!
What am I doing wrong?

@AlCalzone Sugestions here?

@atterdal send me the file please

OTA of Nano Dimmer_EU_C_V2_03.zip

The .exe file's in that zip. Thank you!

I believe that EXE is the standalone firmware update w/ an EXE designed to be used as an executable whilst the Aeotec stick is plugged into your PC. Ie, it is not the raw hex version you would need for a firmware update through a controller like ZWaveJS2MQTT.
Not for this particular one, but I have found there are people on various forums that have extracted the hex firmware out of the exe and posted the file for download to use.
Example:
https://forums.homeseer.com/forum/homeseer-products-services/system-software-controllers/z-flash-software/105783-hex-file-for-aeotec-multisensor-6

@scott7685 zwave-js supports extracting the raw firmware from Aeotec updater exes.

MIND = BLOWN @AlCalzone
That is phenomenal. I have used it to upload the extracted hex but have never tried a full exe

@atterdal seems like that file had another variation that I haven't encountered yet. The linked PR should fix this.

There's a standard for everything I guess. Thank you so much!

There's a standard for everything I guess

Not when it comes to reverse-engineering 😅

@AlCalzone Faced the same with TriSensor, I will try to test with your PR

It seems that it's the same issue with v3.02 EU firmware of Aeotec ZW132. Please see the firmware below:
https://aeotec.freshdesk.com/helpdesk/attachments/6120512641

Found it - just a matter of allowing spaces in the embedded firmware name. Fix is on the way.

I've pulled the latest Docker image (master, updated 8 hours ago) but I still get the same error message.
Is the Docker not in sync with this repo?

@atterdal Docker image is sync with zwavejs2mqtt master branch, and latest zwave-js release

Too be safe I waited for new release and updated to latest master on docker. Still get this message when trying to update:
zwavejs2mqtt     | 2021-02-17 13:42:47.495 INFO ZWAVE: Failed to start the update: Invalid manufacturer or firmware id! beginFirmwareUpdate {
zwavejs2mqtt     |   success: false,
zwavejs2mqtt     |   message: 'Failed to start the update: Invalid manufacturer or firmware id!'
zwavejs2mqtt     | }

Is it related to the node I'm trying to update register as "Unkown product 111" from "Unkown Manufacturer 134"?

Hmm, my test case with current node-zwave-js master still works. So there's probably some version mismatch.

It says "App version: 1.1.1" "Zwavejs Version: 6.4.0"

You did load the exe from the zip file above?
@robertsLando maybe another artefact of the duplicate dependencies due to zwave-js-server?

Yes, same .exe as I sent before. Is it expected that the node shows up as "Unkown product 111" and "Unkown manufacturer 134"?

@robertsLando maybe another artefact of the duplicate dependencies due to zwave-js-server?

I'm waiting for the new beta 7 release, that should be fixed then

@atterdal just to be sure, are you using master or dev tag ?

@robertsLando master

Ok that should have the latest things so. Sincerly I dunno what's wrong here :(

Hi!
@robertsLando - I have a bunch of Heltun Devices that i need to firmware update, and was hoping that
ZWaveJSMQTT would help me to do. I can use my firmware update files in the ZWave.me application, but am unable to update through ZwaveJSmqtt. No issues adding them to my controller, but when i try a OTA update it gives me
"Error when calling API befinFirmwareUpdate: Unable to extract firmware from file: Could not parse HEX firmware file!"

Attached the .ota file zipped if you can see anything that makes the update crash?
TPS03_EU_v2_3_3.zip

Hi!
@robertsLando - I have a bunch of Heltun Devices that i need to firmware update, and was hoping that
ZWaveJSMQTT would help me to do. I can use my firmware update files in the ZWave.me application, but am unable to update through ZwaveJSmqtt. No issues adding them to my controller, but when i try a OTA update it gives me
"Error when calling API befinFirmwareUpdate: Unable to extract firmware from file: Could not parse HEX firmware file!"

Attached the .ota file zipped if you can see anything that makes the update crash?
TPS03_EU_v2_3_3.zip

And - as @atterdal above, my device is also "unknown" manufacturer by zwavejsmqtt.

Is there a way to make zwavejs read the manufacturer xml from the device? Zwave.me can read this. :)

@johanschelin can you open a new issue for this in https://github.com/zwave-js/node-zwave-js please?

For the record, using zwjs2mqtt 5.2 zwjs 7.10, was successful in updating .exe firmware to Aeotec HEM-G5.

authors: AlCalzone atterdal billiaz johanschelin razvanws robertsLando scott7685 tommyjlong


text:
Added the romanian language

Added the Romanian language for zxcvbn

Hey, thanks for adding a new language!
I added a few comments and do you have a real online source for you dictionaries?

authors: MrWook daniesy

repo: zxcvbn-ts/zxcvbn


text:
Added data classes to Zone

Added tests for all these data classes.
Also includes an attempt to get the tests to run on CircleCI.

@manzanotti I am so sorry to have taken so long to review your very useful PR (many reasons).
I can see where you're heading & it looks great!
Main thing is that I have added a branch, which is based upon your #38 - the reason why I did this is because (blah-blah, and) I have no idea how to send a commit direct to you... Oh well.
The code didn't run on my system without error, the main reason being that not all zone types support all schedule types - see my review notes.
And sorry, I did a bit of refactoring - I couldn't stop myself!!

authors: manzanotti zxdavb

repo: zxdavb/geniushub-client


text:
chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

SonarQube Quality Gate Result

Result: :white_check_mark: OK
Triggered by @zxkane on pull_request

Metric
Status
Value
Error Threshold

Reliability rating
:white_check_mark: OK
1
> 1

Security rating
:white_check_mark: OK
1
> 1

Sqale rating
:white_check_mark: OK
1
> 1

Coverage
:white_check_mark: OK
91.10
< 80

Duplicated lines density
:white_check_mark: OK
0
> 30

Blocker violations
:white_check_mark: OK
0
> 0

Bugs
:white_check_mark: OK
0
> 0

Code smells
:white_check_mark: OK
0
> 40

Critical violations
:white_check_mark: OK
0
> 0

Major violations
:white_check_mark: OK
0
> 0

Vulnerabilities
:white_check_mark: OK
0
> 0

View on SonarQube
updated: 7/1/2024, 00:17:44 (UTC+0)

SonarQube Code Analytics
Quality Gate passed

Additional information
The following metrics might not affect the Quality Gate status but improving them will improve your project code quality.
Issues
 0 Bugs
 0 Vulnerabilities
 0 Code Smells
Coverage and Duplications
No data Coverage
No data Duplication

repo: zxkane/snat


text:
chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

SonarQube Quality Gate Result

Result: :white_check_mark: OK
Triggered by @zxkane on pull_request

Metric
Status
Value
Error Threshold

Reliability rating
:white_check_mark: OK
1
> 1

Security rating
:white_check_mark: OK
1
> 1

Sqale rating
:white_check_mark: OK
1
> 1

Coverage
:white_check_mark: OK
91.10
< 80

Duplicated lines density
:white_check_mark: OK
0
> 30

Blocker violations
:white_check_mark: OK
0
> 0

Bugs
:white_check_mark: OK
0
> 0

Code smells
:white_check_mark: OK
0
> 40

Critical violations
:white_check_mark: OK
0
> 0

Major violations
:white_check_mark: OK
0
> 0

Vulnerabilities
:white_check_mark: OK
0
> 0

View on SonarQube
updated: 9/16/2024, 00:17:54 (UTC+0)

SonarQube Code Analytics
Quality Gate passed

Additional information
The following metrics might not affect the Quality Gate status but improving them will improve your project code quality.
Issues
 0 Bugs
 0 Vulnerabilities
 0 Code Smells
Coverage and Duplications
No data Coverage
No data Duplication


text:
prompt "IDASkins detected that this is you first IDA..." all the time in IDA pro 7.1

I just comment this part of the plugin to handle this ...

Platform, Plugin version?

Please check if this issue is resolved when downloading the latest code from master. I merged a fix for macOS about an month ago, however didn’t do a new release yet. I think it might address your issue.

Thx！

repo: zyantific/IDASkins


text:
Micro crashes everytime i try to open a directory

Description of the problem or steps to reproduce
press control + e
type 'open'
type the name of a directory
Specifications

Commit hash:  b9763856
OS: Microsoft Windows 10 Home
Terminal: Windows Terminal CMD

Seems like a Windows issue, I cant reproduce it on
macOS 10.15.7
Version: 2.0.10
Commit hash: b9763856
Compiled on August 07, 2021

Will be tracked with #2666 and is already addressed with https://github.com/zyedidia/go-shellquote/pull/1.

authors: JoeKar Paulbenny22 faraazahmad

repo: zyedidia/micro


text:
now is go gettable and with updated make file and .gitignore to reflect that

After merging, go get -v -u github.com/zyedidia/micro/cmd/micro  should do it
Try it out: go get -v -u github.com/aerth/micro/cmd/micro

I don't think this installs the runtime files to ~/.micro so you won't have syntax highlighting if you install this way. Still might be worth having though. There's no way to have a post install command run after go get completes right? It would be really nice if that were possible.

After go get completes, the entire git repo is located at $GOPATH/src/github.com/zyedidia/micro

yeehaw

authors: aerth zyedidia


text:
slideTo() does not display page

Seems to be a bug:
when using the slideTo function, the tab bar reacts accordingly, but the content of the selected page does not display. The selected page neither shows up in the DOM.
(I'm using lazy loading btw, if that might be of an issue here)
Clicking directly in the tabbar or sliding between tabs works fine though.
Would be nice if you can have a look at this.

I just tried it without lazy loading the tab page, but the issue remains the same.

Experiencing the same issue. Using provider to hold data between two tab pages. When i use slideTo, the page does not rerender with actual data. Sliding works.

Well, managed to work around that using:
@ViewChild(SuperTabs) tabs: SuperTabs;

this.tabs.slideTo(index);
this.tabs.onTabChange(index);

Seems to work with no issues.

Thanks a lot, the this.tabs.onTabChange(index); indeed does the trick.
Nonetheless, I had a look into the code and it seems the slideTo function does not check whether the tab was loaded or not. Some small edit like this pseudo code might already fix that bug.
if(tab != loaded){
    tab.load(true)
}
slideTo(tab)

This should be fixed in the latest releases.

authors: Chris1234567899 FilipMatys ihadeed


text:
Android build error:  no suitable constructor found for Stripe(no arguments)

I installed this plugin today and ran into an error when building my cordova application for Android.
platforms/android/src/com/zyramedia/cordova/stripe/CordovaStripe.java:24: error: no suitable constructor found for Stripe(no arguments)

I notice this error occurs with the Java Stripe SDK v3.0 and 3.1 (both updated in the past 3 days).
As a workaround I can modify platforms/android/cordova-plugin-stripe/appname-CordovaStripe.gradle to specify version 2.1.0:
dependencies {
    compile 'com.stripe:stripe-android:2.1.0'
}

And the build passes.  However this is not ideal as this code is generated.  Is there a better workaround or will I need to wait for a patch?
Thanks!

Yes for now you need to specify the version. It's fixed in the repo but not in the latest npm version. (you can install the plugin directly from the repository instead of npm)
I will update the code to work with their 3.0.0 SDK in the next week or so, and that should fix the problem for sure.

repo: zyramedia/cordova-plugin-stripe


text:
Optional putting post content in RSS feed

Optionally (disabled by default) it could be also possible to put also the whole content of the post in the RSS feed - <content:encoded> (probably also with <media:content>).
Sample feed with content.

I added all of them
params.toml
...
updatePeriod = "" # Possible values: 'hourly', 'daily', 'weekly', 'monthly', or 'yearly'.
updateFrequency = ""
fullContents = false

Thanks, it works in general. However there is some problem with the images - they are not displayed.
For the following markdown:
> Get know about my migration from WordPress to Hugo and the reasons behind it.

{{}}

{{< notice info >}}
This is a modified version of the blog post I also placed as the last one on my old WordPress-based blog.
{{< /notice >}}

## Welcome to my new blog!

I have:

Get know about my migration from WordPress to Hugo and the reasons behind it.

<img data-src="https://preview--solidsoft-blog-hugo.netlify.com/images/posts/wordpress-to-hugo.png" data-caption=""
             src="data:image/svg+xml,%0A%3Csvg xmlns='http://www.w3.org/2000/svg' width='24' height='24' viewBox='0 0 24 24'%3E%3Cpath fill='none' d='M0 0h24v24H0V0z'/%3E%3Cpath fill='%23aaa' d='M19 3H5c-1.1 0-2 .9-2 2v14c0 1.1.9 2 2 2h14c1.1 0 2-.9 2-2V5c0-1.1-.9-2-2-2zm-1 16H6c-.55 0-1-.45-1-1V6c0-.55.45-1 1-1h12c.55 0 1 .45 1 1v12c0 .55-.45 1-1 1zm-4.44-6.19l-2.35 3.02-1.56-1.88c-.2-.25-.58-.24-.78.01l-1.74 2.23c-.26.33-.02.81.39.81h8.98c.41 0 .65-.47.4-.8l-2.55-3.39c-.19-.26-.59-.26-.79 0z'/%3E%3C/svg%3E"
             class="lazyload" width="400px" height=""/>

 This is a modified version of the blog post I also placed as the last one on my old WordPress-based blog.

Welcome to my new blog!

Looking at some SO answers (I'm also not familiar with Atom), I wonder if data-src shouldn't be replaced with just src?

There were some mistakes in the rss.xml. Now it's fixed
Since I can't detect images in the post when making the xml file, I use the image specified in the front-matter.
---
image: "images/..."
---

My feeder doesn't display them. However, I see they are in the feed, so it may be it just cannot used it. Let's leave it as it - the content itself is displayed. Thanks.

authors: szpak zzossig

repo: zzossig/hugo-theme-zzo


text:
Update account-controller.js

@ShrutikaSingh review once.


text:
Release v0.6.3

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

Project coverage is 74.24%. Comparing base (1da5bb0) to head (fb5fe57).

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1026       +/-   ##
===========================================
+ Coverage   63.26%   74.24%   +10.98%
===========================================
  Files          48        3       -45
  Lines        5970      532     -5438
  Branches      154      154
===========================================
- Hits         3777      395     -3382
+ Misses       2193      137     -2056

Components
Coverage Δ

apel_plugin
74.24% <ø> (ø)

auditor
∅ <ø> (∅)

auditor_client
∅ <ø> (∅)

slurm_collector
∅ <ø> (∅)

slurm_epilog_collector
∅ <ø> (∅)

htcondor_collector
∅ <ø> (∅)

priority_plugin
∅ <ø> (∅)

authors: codecov-commenter dirksammel

repo: ALU-Schumacher/AUDITOR


text:
Adding UC01, UC02, UC03

Codecov Report

Merging #32 (1de6742) into master (cc1b941) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #32   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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 cc1b941...1de6742. Read the comment docs.

authors: chetwinlow codecov-commenter


text:
Fix student part in add command manual testing

Codecov Report

Merging #206 (8d5f2eb) into master (d14116c) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #206   +/-   ##
=========================================
  Coverage     76.47%   76.47%
  Complexity      658      658
=========================================
  Files            92       92
  Lines          2104     2104
  Branches        273      273
=========================================
  Hits           1609     1609
  Misses          425      425
  Partials         70       70

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 d14116c...8d5f2eb. Read the comment docs.

authors: codecov-commenter knotstoks


text:
Fix ppp

Codecov Report

Merging #318 (879880e) into master (daf5023) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #318   +/-   ##
=========================================
  Coverage     68.86%   68.86%
  Complexity      713      713
=========================================
  Files           107      107
  Lines          2229     2229
  Branches        266      266
=========================================
  Hits           1535     1535
  Misses          606      606
  Partials         88       88

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 daf5023...879880e. Read the comment docs.

authors: BrendonLau codecov-commenter


text:
Add test class for FreeScheduleCommand

#70 This is completed in this pull request


text:
Fix date comparison code quality

Codecov Report
Base: 63.26% // Head: 63.64% // Increases project coverage by +0.38% :tada:

Coverage data is based on head (d16610a) compared to base (a0017c0).
Patch coverage: 61.36% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #319      +/-   ##
============================================
+ Coverage     63.26%   63.64%   +0.38%
  Complexity      650      650
============================================
  Files           112      113       +1
  Lines          2273     2261      -12
  Branches        266      264       -2
============================================
+ Hits           1438     1439       +1
+ Misses          736      724      -12
+ Partials         99       98       -1

Impacted Files
Coverage Δ

...c/main/java/seedu/address/model/client/Client.java
80.59% <0.00%> (-1.23%)
:arrow_down:

...main/java/seedu/address/model/client/Birthday.java
33.33% <16.66%> (+11.45%)
:arrow_up:

...eedu/address/logic/commands/ListClientCommand.java
41.17% <50.00%> (+5.88%)
:arrow_up:

src/main/java/seedu/address/model/util/Date.java
74.19% <74.19%> (ø)

.../java/seedu/address/model/meeting/MeetingDate.java
80.00% <100.00%> (-2.76%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

authors: codecov-commenter rexong


text:
Name Autocorrect

As it is close to the deadline, I will be bypassing branch protections. @saiutkarsh33 I have created issues based on the issues I raised in this PR.


text:
Add italian localization

Thank you

repo: AbdurazaaqMohammed/AntiSplit-M


text:
4.1.0

coverage: 100.0%. remained the same
when pulling 93c1bb6bce1820b1f32ec8c90a2e88c31bb5786d on 4.1.0
into 3d97b212af252d43079a7c0badefc80d19e366bc on master.

repo: Accruent/robotframework-zoomba


text:
feat: Upgrade to eslint v9

Pull Request Test Coverage Report for Build 10028269491
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 10008886703:
0.0%

Covered Lines:
20

Relevant Lines:
20

💛  - Coveralls

authors: AdiRishi coveralls


text:
4.6.3

:white_check_mark: Build TheIdServer 4.6.3-ci.1+40.build.2180 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/18ee4b507c by @aguacongas)

authors: AppVeyorBot aguacongas

repo: Aguafrommars/TheIdServer


text:
Progress Bar and Audio Fix

This will also solve #23

repo: AlexPolGit/Super-Sorter


text:
Causal loop DSL?

We now have a causal loop DSL for those with polarities


text:
December 4 Weekly Scrum

Work Completed - Allow users to close surveys (#42), Did a quick fix for deleting surveys (#49).
Work to be done - Charts in results view (#54)
Blocking - Nothing

Last week: created PR (in review) for registering new users (issue #57, PR #66). Minor fixes (issues #60 and #58, PR #62).
Next week: letting the user add more options to MCQs (issue #67)
Blocking: none

Last week: I pushed the main changes to #33.
Next week: Working on finalizing #33  and I will start working on one of the open issues. Most likely #55 (not sure yet)
Blocking: PANICKING FINAL WEEK

authors: Andrei486 CamSommer MiniEinsteinD


text:
First Structure 080623

Updating Structure


text:
add GoogleEarghPro

Please have only one application in each pull request, thanks.

authors: dezbracaty probonopd


text:
Modify Project Prosal, add LCD and change diagrams

issue resolved

repo: ArcueidShiki/SmartEntrySystem


text:
[js][openinference-core] sessionId inheritance support

clsoing as finished in #656

authors: Parker-Stafford mikeldking

repo: Arize-ai/openinference


text:
fix font family of the paragraph

Assign me this task


text:
Add CMake commands to install library for find_package.

Codecov Report
Patch and project coverage have no change.

Comparison is base (5dd7e16) 93.97% compared to head (97c3f8d) 93.97%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##           master       #3   +/-   ##
=======================================
  Coverage   93.97%   93.97%
=======================================
  Files          16       16
  Lines         697      697
  Branches       18       18
=======================================
  Hits          655      655
  Misses         24       24
  Partials       18       18

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

authors: LTLA codecov-commenter

repo: ArtifactDB/comservatory


text:
integrate socket.io

:tada: This issue has been resolved in version 12.1.0 :tada:
The release is available on:

12.1.0
GitHub release

Your semantic-release bot :package::rocket:


text:
Test/task status

lgtm!


text:
Support dotted note in tuplet

仕様がよくわからないので保留にします


text:
Create bubble_sort.cpp

Good job !!! Thank you for contributing !!!😊

Hi Ayush,
Don't forget to change the repo tags to *hacktoberfest. *Otherwise, my
along with other contributors' PRs won't be counted towards
Hacktoberfest-22.
I think you were busy with things, so this is just a friendly reminder 😊
Thanks
Girija
On Tue, Oct 4, 2022 at 6:52 PM Ayush kumar @.***> wrote:

Good job !!! Thank you for contributing !!!😊
—
Reply to this email directly, view it on GitHub
https://github.com/Ayush0431/Data-structure-algoritms-hacktoberfest/pull/25#issuecomment-1266989861,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKFZBNBDPEGASJIMOD6R2ELWBQVPVANCNFSM6AAAAAAQ4QU54I
.
You are receiving this because you were assigned.Message ID:
<Ayush0431/Data-structure-algoritms-hacktoberfest/pull/25/c1266989861@
github.com>

--
Girija Nandan Kar
Phone: +91-8371095573
Email: @.***


text:
Investigate whether or not player's lumber icon and name can be made unique to that player

So it's not possible to modify the actual lumber indicator, but it is possible to create a new lumber indicator and shove it on top of the old one.

repo: AzerothWarsLR/WarcraftLegacies


text:
check with Cosmos DB Security team on the new connector

Done.

authors: kushagraThapar moderakh

repo: Azure/azure-sdk-for-java


text:
[EventHubs] Fix mgmt request token format + logging

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-eventhub

authors: annatisch azure-sdk

repo: Azure/azure-sdk-for-python


text:
Add expect100=True for aiohttp + streaming requests

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

/azp run python - storage - ci


text:
[TA] rename abstract -> abstractive

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-ai-textanalytics

authors: azure-sdk kristapratico


text:
[ServiceBus] update pyproto b1 version/changelog/readme

/azp run python - servicebus - ci

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-servicebus

authors: azure-sdk swathipil


text:
README and help tweaks

/check-enforcer override

repo: Azure/azure-sdk-tools


text:
Merge lza-petclinic to lza

@jeffpaineMS, let me know when you want me to merge this.


text:
increased resources for the billing runner

We'll need to look into and fix the fmt and validate check errors.

@wrnu now it's complaining about a missing provider, I doubt this is new, do you have an idea about this?

@wrnu now it's complaining about a missing provider, I doubt this is new, do you have an idea about this?

Yeah the provider is generated, there is some code we have to add to the workflow to generate it for the validation. I think @prabhukiran9999 did that in another repo

authors: MaxWardle wrnu


text:
chore: add initial citadel treasury api support + mocks

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

repo: Badger-Finance/badger-sdk


text:
chore: add citadel proof method to api

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

authors: axejintao eliobricenov


text:
Вторая и третья практики, оболочки

Жду проверки всех работ)

Проверьте, пожалуйста.

Включите свой модуль в сборку

Что сделать? Объясните пожалуйста, я не понимаю. В какую сборку? Программы вроде как рассортированы по папкам в соответствии с заданиями. Это как-то мешает их проверке???

Обновитесь с основного репозитория

Давайте я попробую пересоздать.

authors: EorrDjocker Pastor


text:
Create ci.yml

njk

repo: BismarckY/skills-test-with-actions


text:
Update beta-F248.md to include bug fix 46

Your PR was made after the release of beta-F248 and will be credited differently.

My bad pimp

authors: Kustys prplwtf

repo: BlueprintFramework/web


text:
fix:app crashes when user is null

@logicalahara Good Work

repo: Bookshlf-in/Website


text:
Add metadata, improve SEO

I would like to learn and work on it.

Has Hemant been assigned this issue or is it still open? I too wanted to learn and work on SEO

Since @ORKO06 is working on another issue, @Swatishree-Mahapatra you can take this up! 🚀

@Swatishree-Mahapatra you can find the basic meta tags here ⬇
https://github.com/CIPHERTron/kaagaz/blob/main/components/layout.tsx
and their config from the SiteConfig object in config/index.json
Apart from this, refer to some articles on the internet regarding SEO in Next.js. You'll get some more idea 💪🏻

authors: CIPHERTron ORKO06 Swatishree-Mahapatra

repo: CIPHERTron/kaagaz


text:
Update CI workflow and config test environment

Please do not commit node_modules to the repo, you can add a .gitignore file to your project directory and include the node_modules folder in your exclusion with the following.
node_modules


text:
Anagrams backend

Everything is a OK

Problem with missing files, try and remerge


text:
Fixed CodeQL missing ref

Current dependencies on/for this PR:

main

PR #303   👈

This comment was auto-generated by Graphite.

repo: CatChen/node-package-release-action


text:
Release v1.5.0

abandoning due to inconsistent peer scoring. Will revisit in 1.6.0

repo: ChainSafe/lodestar


text:
[HB-7211] Public Consent Override

yeah we'll need it when we release the 5.x adapters


text:
first

👷 Deploy Preview for warm-horse-5ea874 processing.

Name
Link

🔨 Latest commit
a63714e2988147a5216494d402f1c3bc40340401

🔍 Latest deploy log
https://app.netlify.com/sites/warm-horse-5ea874/deploys/64d571949778680008821c00

license: ISC


text:
Make the startdate of "Variants in Canada over time"  a global vraiable at the beginning of Rmarkdown

will be in the next release

repo: CoVaRR-NET/duotang


text:
Fixed memory error of arena

buggy


text:
add track link in getting started guide

Ah, great catch!

authors: dipamsen shiffman


text:
Show history of the payment request email sent to client

closing task as it is  complete

repo: ColoredCow/portal


text:
Launcher client app status to app context

Closed in support of https://github.com/Cooperation-org/linked-trust-content-mod/pull/32

authors: ArnobChowdhury CatInAHatIsBack


text:
fix burger menu and nav bar

fixed by https://github.com/CovalenceNetwork/covalence.info/commit/4555346df9fbf9b19b2774629bb3a04bb08ce9d5


text:
Add workflow to backup production DB

 Add DB backup workflow for production

repo: DFE-Digital/teaching-record-system


text:
Update Command Line Arguments

In this setup, how would you change e.g. the batch size, while keeping the other default parameters? For example, the value that you assign now to 'model_strides': [(2, 2, 2), (1, 2, 2), (1, 2, 2), (1, 2, 2), (2, 2, 2)], cannot be set through the command line.

Set argument --use_def_model_hp to 1, to override all model architecture-specific command line arguments and use the full set of default hyperparameters. If you want to use some, but not all, then set --use_def_model_hp to 0, and only pass the argument that you wish to change. Also, switching to string representations imo makes it a lot easier to handle and pass these list-of-list arguments, which is what I've done.


text:
Version 1.3.0

Codecov Report

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

:exclamation: Current head 4f2548c differs from pull request most recent head a522a07. Consider uploading reports for the commit a522a07 to get more accurate results

@@            Coverage Diff            @@
##             master       #6   +/-   ##
=========================================
  Coverage          ?   95.95%
=========================================
  Files             ?        7
  Lines             ?      420
  Branches          ?        0
=========================================
  Hits              ?      403
  Misses            ?       17
  Partials          ?        0

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 82e320c...a522a07. Read the comment docs.

authors: DNKpp codecov-commenter

repo: DNKpp/Simple-Vector


text:
make Str::BasicStringRef closer to C++17 string_view()

Please review this quickly, as it prevents the CI to make https://github.com/Unvanquished/Unvanquished/pull/2720 pass, but the real reason is here not in the aforementioned proposal.

The way I've heard the rule of 5 is that you should define either 0 or 5 of those things. We define 0 so we are already in compliance.

Maybe I'm wrong then. I'm too lazy to check on that. Please could you comment on the PR?

The Rule of Five is a modern extension to the Rule of Three. The Rule of Five states that if a type ever needs one of the following, then it must have all five.

copy constructor
copy assignment
destructor
move constructor
move assignment

https://www.codementor.io/@sandesh87/the-rule-of-five-in-c-1pdgpzb04f

Does it makes this PRs bad or good though? I can change title if you want. Sorry, I'm doing a barbecue right now with some aperitif, so can't think too much.
I'd just like to know: is this going to generate regressions or not? If not, is it going to fix problems in the present (see unv's pr) and future?

Ok, I changed title and added a bit of content even, so that this implementation is closer to the C++17 std::string_view, which does allow default construction and copy construction. It also allows construction from a char* + a size.
The 2 implementations I've checked from C++ STL on my computer (LLVM's libc++11 and GNU's libstdc++) do not provide the move ctors, but I took the freedom to add them [because]https://en.cppreference.com/w/cpp/language/rule_of_three):

Unlike Rule of Three, failing to provide move constructor and move assignment is usually not an error, but a missed optimization opportunity.

Looks like I forgot to actually commit before pushing. Oops.

StringRef is intentionally different from string_view. StringRef is supposed to be a C-string, i.e. a pointer to a null-terminated byte sequence. Except with precomputed size information for performance. string_view is an arbitrary sequence of bytes that need not be null-terminated. We have Str::StringView for the exact equivalent of that.

Ok then.
I think this is all a bit confusing, and don't see what this PR would break or make slower, though. But if you really consider this a bad idea I'll close this, no problem.

Yeah I don't think the PR helps anything.

and I think it also breaks an invariant that Str::StringRef are never pointing to nullptr, or that len == strlen(str).

If you ran tests more often, I'd be interested in the reasons why you think that. But I'm not really. This change is not wished for, so it's fine. If only all not wished changes were closed instead of merged, it would be nice.

authors: bmorel slipher

repo: DaemonEngine/Daemon


text:
modified set header implementation

Current dependencies on/for this PR:

main

PR #12

PR #21
PR #19

PR #20

PR #22   👈

PR #18

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

main

PR #12

PR #21
PR #19

PR #20

PR #22   👈

PR #18

This comment was auto-generated by Graphite.

repo: DataDog/envoy-filter-example


text:
messages service and component integration

👍

repo: DeryckJackson/angular-tour-of-heroes


text:
Variable Model updates

Current dependencies on/for this PR:

main

PR #42

PR #43

PR #44

PR #45   👈

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

main

PR #42

PR #43

PR #44

PR #45   👈

This comment was auto-generated by Graphite.

repo: DevCycleHQ/java-server-sdk


text:
Update RadialStatus.theme.css

This will negatively impact users. So no.

repo: DiscordStyles/RadialStatus


text:
Update README.md

TESTING!! DO NOT MERGE!!

repo: DoD-Platform-One/Istio


text:
provide different env.yml file for local installation without cupy and cudatoolkit

i still did not get  why they have errors about cupy and cudatoolkit. Maybe they are using Windows?
In general, there should not be problems installing them.
In the beginning, I planned to have cupy examples, that is why it is there.
cudatoolkit is needed for numba anyhow, so one should not remove.

maybe we could make one yml file for cpu, then people could use that to install locally

yeah i think the failure to install cupy/cudatoolkit was on windows. But i agree that having two env.yml files, one for CPU and one for GPU, makes sense.
But we might want to consider larger changes, like splitting the material into CPU and GPU parts. This is for a later discussion

authors: qianglise wikfeldt

repo: ENCCS/HPDA-Python


text:
use checkout@v2 in github workflow

Oof, my review doesn't count on this repo I suppose.

repo: EasyPost/duo-auth-rs


text:
First Django Exercice

Hola estimado como andas?, me estas pidiendo un PR a mi repo, en este caso tenes que hacerlo a tu repo directamente

repo: ElDwarf/Django-project-vs-template


text:
Background image src eval and handle environment variables

@vinothvino42 finish it up please, this has been pending for a while. See Vu's comment, it's a minor change.

@vinothvino42 finish it up please, this has been pending for a while. See Vu's comment, it's a minor change.

@vusters @kmahmood74 In the screen.dart
1.
Widget renderScreen(ScreenDefinition screenDefinition) {
    // build the data context
    DataContext dataContext = DataContext(
        buildContext: context, initialMap: widget.screenPayload?.arguments);

    PageModel pageModel =
        screenDefinition.getModel(widget.screenPayload?.arguments, dataContext);

2
class ScreenDefinition {
  ScreenDefinition(this._content);
  final YamlMap _content;

  PageModel getModel(
      Map<String, dynamic>? inputParams, DataContext dataContext) {
    YamlMap output = _content;

    /// manipulate the screen definition to account for custom widget
    if (_content.keys.length == 1 && _content['Widget'] != null) {
      output = _getWidgetAsScreen(_content['Widget'], inputParams);
    }

    final evaluatedData = dataContext.eval(output);
    final data = Utils.getYamlMap(evaluatedData);
    return PageModel.fromYaml(data!, dataContext);
    return PageModel.fromYaml(output, dataContext);
  }
}

Here, after evaluating the output (see evaluatedData object). I'm getting null for all the body (code block) of executeCode.

@vinothvino42 did you resolve the issue with @vusters today? Any progress on this one?

@vinothvino42 this is how you evaluate - all is done for you already:
dynamic evaluatedBackground = _scopeManager.dataContext.eval(widget._pageModel.pageStyles?['backgroundImage']);
There should be zero things you have to do. This actually recursively go through the backgroundImage and evaluate all under it.
Regarding evaluating others. Let's not do that yet. The proper solution is to do them all automatically (just like a regular widget), vs go through every single property to evaluate them.

repo: EnsembleUI/ensemble


text:
Actually clone repo

fixes #850


text:
Feature/entry cards

Falta el stroke en el card

authors: Poncka davidquintr

repo: FactorizacionPlus/Pasadita


text:
Update config.yml

https://www.facebook.com

repo: FadelIsmail/setup-java


text:
Add gifs

The gifs are too big; need to fix!!


text:
Create SECURITY.md

pls


text:
cambricon: fix some bugs in tests

LGTM

authors: pangge sethbrin

repo: FlagOpen/FlagGems


text:
minor: Remove unnecessary PHP version constraints

Coverage remained the same at 92.773% when pulling 80c07c5ce18f2b25c8f1007a822b896839817255 on julienfalque:php-version-constraint into a259c0b55877b2e5bbc6329470afa896ee1966a5 on FriendsOfPHP:master.

if all tests are green than RTM from me :shipit:  :+1:
I do recall only running the doc-are-up-to-date tests on some PHP version because the docs can differ based on the PHP version the docs are generated on, however if true we should this on the CI so lets see : )

authors: SpacePossum coveralls julienfalque

repo: FriendsOfPHP/PHP-CS-Fixer


text:
Remove unused 'sumBlockSize' variable

Thank you for this contribution. The variable probably comes from copy&paste from Vulkan Memory Allocator :)

authors: adam-sawicki-a danielkrupinski


text:
[stable/gocd] Bump up GoCD Version to 19.1.0

/close

repo: GaneshSPatil/charts


text:
Table 1 Fix in GENIE Search Analysis

Adding @wesleygoar as a reviewer

authors: jarbesfeld korikuzma


text:
Added DB schema

Yeah, this is for the schema and then I will add db connection stuff later.

repo: GeoscienceAustralia/dea-conflux


text:
bug或者需求能不能列出来,看有没有能帮忙做的?

细节调整的地方太多了，我都是发现一点调整一点

authors: GershonWang student-ice

repo: GershonWang/linglong-store


text:
Make website Gingerish

TODO:

Company name
PGP

Ready for review.

repo: GingerPrivacy/GingerWalletWebSite