Icon Render Android

Environment
Android Simulator 5.1.1

Description
The icons are being rendered as shown in the below screenshot. I tried installing it manually twice, then I used react-native link. I'm using the latest version.
When using with "md-", for example "md-menu", it doesn't give me any erros, but then this happens:

When using only the icon's name, it gives me this warning:

Reproducible Demo
Am I missing something?

<TouchableOpacity onPress={this.props.onLogout}>
  <View style={styles.drawerItem}>
    <Icon
    name='md-log-out'
    size={30}
    color='#aaa'
    style={styles.drawerItemIcon}
    />
    <Text>Sign Out</Text>
  </View>
</TouchableOpacity>

Yes I also found same issue..
import Icon from 'react-native-vector-icons/FontAwesome';

<Icon name='user' style={{alignSelf:'center', fontSize:30}}/>

I have downgraded the version to this one, and now it works:
 "react-native-vector-icons": "^4.2.2"
The "menu" icon still renders like that crossed square thing tho...


jobs: Refactor and fix food timer

Fix food timer by split _updateFoodBuff into _showFoodBuff, _canShowFoodBuff and _shouldShowFoodBuff
_canShowFoodBuff decides if cheese icon can show now.
_shouldShowFoodBuff decides show cheese icon now or later if it can show.
_showFoodBuff directly make cheese icon shown.
Fix a bug that in-combat status not updated soon enough.

Fixes #4032
Known issues: bug within a bug

When changing zone, a fake "gainEffect" log with fake duration will update food timer incorrectly.
When you dispel your food buff, food timer will not update.
When using a max level job in leveling dungeons, cheese icon still showing.

Unrelated issues found in debug: bug within a bug within a bug

When AOE combo action hit nothing, comboTimer still considers combo established.
In patch 6.1, some 2.0 dungeons are remaked and got a new ZoneID, which makes CN/KO version cannot match these dungeons.

In patch 6.1, some 2.0 dungeons are remaked and got a new ZoneID, which makes CN/KO version cannot match these dungeons.

I think for now we can ignore this, as the 6.0 Korean patch is happening in a few days.  If more dungeons get remade, I'll try to remember to add old entries for them like I do with the unreals that get removed.

small sidenote, KO 6.0 will patch is tomorrow the 10th Mai

Done. Also fixed dispel wellFed issue.
As for issue of level, I believe there are a related block lost in previous refactor, maybe fix them in later pr.

Thanks again for taking the time to fix this (and finding other bugs)!  Much appreciated.  :)


feat(select): Add select header to the md-select

Bug, feature request, or proposal:
Feature request
Yesterday I filled a pull request ( #2797 ) for adding a search bar in the md-select component. As @jelbourn pointed out this wasn't the right way to add this feature, so now I'm willing to do it the right way.
What's the best way to address this? Just implement the same functionality as material does it?
That will mean something like this is possible:
<md-select placeholder="Favorite food">
    <md-select-header>
        <input [ngModel]="searchTerm" type="search" placeholder="Search for a vegetable..">
    </md-select-header>
        <md-option *ngFor="let food of foods | filter " [value]="food.value">
            {{ food.viewValue }}
        </md-option>
</md-select>

The idea is pretty much the same as the AngularJS version. One of the things holding us back from working on this has been that we're not sure what the appropriate accessibility story for this feature is. The most common use case the the select header is a filter input, but the standard role="listbox" doesn't have anything like a associated filter. Adding it would require some research as to whether there is something we can do to make this interaction work for screen-reader users.
As far as API and implementation details go, @kara will likely have some input.

Is it possible that I port the AngularJS version for now and add the whole role="listbox" part later when this is clear?

@Eernie go for it

@Eernie, FYI, another use case of md-select-header is a check all / uncheck all (button(s) or tri-state checkbox) if the options are multi-select.

What's the status of this? Is this merged or not?

I was wondering if there was any progress on this?

@JasonAHeron the related PR (#3211) was blocked for a while until we decide how to handle the accessibility of the header. It should be unblocked once https://github.com/angular/material2/pull/6856 gets in.

Any news on this issue?
It looks like #6856 was merged

There's a pending pull request at https://github.com/angular/material2/pull/7835.

Hi I was wondering if 'mat-select-header' is available yet?

@andplusblaw unfortunately not yet. Please leave feedback at https://github.com/angular/material2/pull/7835 if you have any.

when can we expect this feature to be work ?

@jrood thank you

Still no? More than 1 year passed

+1

+1

Closing as a duplicate of #5697 because that one has more votes (even though this one is older)


Overflow when adding durations during pop

Opening a bug raised by @RobinKa in https://github.com/metadsl/egglog-python/issues/53 that sometimes when calling pop an overflow is raised by adding the durations. Relevent part of the traceback copied:
thread '<unnamed>' panicked at 'overflow when adding durations', library/core/src/time.rs:914:31

[...]

  21:     0x7f22ff5419e0 - <core::time::Duration as core::ops::arith::AddAssign>::add_assign::hf0b4e48d4fab5e6b
                               at /rustc/eb26296b556cef10fb713a38f3d16b9886080f26/library/core/src/time.rs:921:17
  22:     0x7f22ff399742 - egglog::RunReport::union_times::h38ef9263e9e7347a
  23:     0x7f22ff399812 - egglog::RunReport::union::h651a29516887af3d
  24:     0x7f22ff39ab81 - egglog::EGraph::pop::hec17bdef39ad741e
  25:     0x7f22ff3a372a - egglog::EGraph::run_command::h1a909c7d32e16c7c
  26:     0x7f22ff3a70b6 - egglog::EGraph::run_program::h3cdc3518c784f89b

This didn't use to happen, but may have been introduced by https://github.com/egraphs-good/egglog/pull/225 ?

Good catch! Can you confirm this is fixed by #241?

Yep can confirm all my tests pass now, thanks for the quick fix!


feat: add @zeit/next-bundle-analyzer

Analyze both your front end bundle and back end bundle with webpack-bundle-analyzer in next.js

A Gif of it in action

@timneutkens Rebased w/latest.

@jimthedev cleaned up the code, it did a lot it didn't have to do 👍If you could review my changes I'll merge after 👍

@timneutkens Awesome. Reviewed. LGTM.


Sort results of collect_list test before comparing since it is not guaranteed

Fixes #9311.  Order of items returned in collect_list is not guaranteed.  Originally test was modified to try to use a stable sort, but some Spark platforms were still returning a different order.  Changed this to use an explicit sort_array which necessitated treating maps separately, as Spark cannot sort maps.

build


[UI Issue]: Taskbar Icon needs to be a keyboard icon for remap/shortcut windows

Right now its a slightly different PT logo. Confusing for user.

Duplicate issue. #2869 dealt with this.


Universal mode duplicates requirements lines

I just noticed the --universal flag and it looks like just what we need with trio! However, when we try this out (though it's not shown in that PR), we run into an issue where requirement lines are duplicated.
Here's a reproducer, I can't explain this well. x.in:
pylint
sphinx

And here's what is spit out in my terminal when I run uv pip compile --universal --python-version=3.8 x.in:
Resolved 42 packages in 40ms
# This file was autogenerated by uv via the following command:
#    uv pip compile --universal --python-version=3.8 x.in
alabaster==0.7.16
    # via sphinx
alabaster==0.7.13
    # via sphinx
astroid==3.2.2
    # via pylint
babel==2.15.0
    # via sphinx
certifi==2024.7.4
    # via requests
charset-normalizer==3.3.2
    # via requests
colorama==0.4.6 ; sys_platform == 'win32'
    # via
    #   pylint
    #   sphinx
dill==0.3.8
    # via pylint
docutils==0.20.1
    # via sphinx
docutils==0.21.2
    # via sphinx
idna==3.7
    # via requests
imagesize==1.4.1
    # via sphinx
importlib-metadata==8.0.0 ; python_version < '3.10'
    # via sphinx
isort==5.13.2
    # via pylint
jinja2==3.1.4
    # via sphinx
markupsafe==2.1.5
    # via jinja2
mccabe==0.7.0
    # via pylint
packaging==24.1
    # via sphinx
platformdirs==4.2.2
    # via pylint
pygments==2.18.0
    # via sphinx
pylint==3.2.5
    # via -r x.in
pytz==2024.1 ; python_version < '3.9'
    # via babel
requests==2.32.3
    # via sphinx
snowballstemmer==2.2.0
    # via sphinx
sphinx==7.3.7
    # via -r x.in
sphinx==7.1.2
    # via -r x.in
sphinxcontrib-applehelp==1.0.8
    # via sphinx
sphinxcontrib-applehelp==1.0.4
    # via sphinx
sphinxcontrib-devhelp==1.0.6
    # via sphinx
sphinxcontrib-devhelp==1.0.2
    # via sphinx
sphinxcontrib-htmlhelp==2.0.5
    # via sphinx
sphinxcontrib-htmlhelp==2.0.1
    # via sphinx
sphinxcontrib-jsmath==1.0.1
    # via sphinx
sphinxcontrib-qthelp==1.0.7
    # via sphinx
sphinxcontrib-qthelp==1.0.3
    # via sphinx
sphinxcontrib-serializinghtml==1.1.10
    # via sphinx
sphinxcontrib-serializinghtml==1.1.5
    # via sphinx
tomli==2.0.1 ; python_version < '3.11'
    # via pylint
tomlkit==0.12.5
    # via pylint
typing-extensions==4.12.2 ; python_version < '3.11'
    # via
    #   astroid
    #   pylint
urllib3==2.2.2
    # via requests
zipp==3.19.2 ; python_version < '3.10'
    # via importlib-metadata

See how the sphinx lines are duplcated! Note that essentially everything above (x.in, command run) is minimized.
I'd love to poke at uv's internals a bit and maybe fix this! nevermind, I don't think I'm able to given some investigation, as I think this requires a larger change.
Platform information:
$ uv --version
uv 0.2.22
$ python --version
Python 3.12.2
$ python -m platform
Linux-6.1.0-21-amd64-x86_64-with-glibc2.36

I'm running Debian Linux but I also reproduced on Windows.

Ok this doesn't happen in uv==0.2.18. The bad change is bisected to https://github.com/astral-sh/uv/commit/d9f389a58d748f217dfb35d11b63706ce53a5be7 ... which isn't too surprising TBH.

Thank you for bisecting, that's really helpful. cc @charliermarsh

Based on some investigation: when uv hits pylint, it forks into <3.11 and >=3.11 (or something like that). This means it then processes sphinx with >=3.8,<3.11 and >=3.8,>=3.11: the second simplifies to >=3.11. Then uv processes the sphinx dependency, with >=3.11 first it finds sphinx==7.3.7 but with >=3.8<3.11 it finds sphinx==7.1.2 (because sphinx>=7.2.0 dropped Python 3.8).
I think what uv should be doing here is adding markers that apply to everything within the forked state. But ATM it's not clear to me how to do that.

Yeah there should already be markers on the forked state that get propagated here. I'll take a look!

Will prioritize this of course, hopefully fixed today but I have to travel in the afternoon.

I'll probably revert if we don't figure it out today -- better to be strict than wrong.

Yeah based on my investigation a very trivial fix is to comment out narrowing the forked state's Python version. (Which is essentially reverting any improvements)

Is this supposed to be fixed for uv 0.2.24?
I still see it, but in a very complex example, so I wanted to check before trying to create a more minimal reproducer.

I believe so — feel free to file an issue. There are some known errors that we’re working on already, it may end up being a dupe but it’s totally fine.


freezed - Compatibility issue

Hello,
I was using the freezed package sample code from documentation. But it is not able to serialize properly and giving empty object in return.
Documentation code:
import 'package:dart_mappable/dart_mappable.dart';
import 'package:freezed_annotation/freezed_annotation.dart';

part 'union.freezed.dart';
part 'union.mapper.dart';

@freezed
@MappableClass(discriminatorKey: 'type')
class Union with _$Union {
  @MappableClass(discriminatorValue: 'data')
  const factory Union.data(@MappableField(key: 'mykey') int value) = Data;
  @MappableClass(discriminatorValue: 'loading')
  const factory Union.loading(int value) = Loading;
  @MappableClass(discriminatorValue: 'error')
  const factory Union.error(int value, [String? message]) = ErrorDetails;
}

void main() {
  var data = Union.data(42);

  var dataJson = data.toJson();
  print(dataJson); // {"mykey":42,"type":"data"}

  var parsedData = UnionMapper.fromJson(dataJson);

  print(parsedData); // Union.data(value: 42)
}

void main() {
  var data = Union.data(42);

  var dataJson = data.toJson();
  print(dataJson); // {"mykey":42,"type":"data"}

  var parsedData = UnionMapper.fromJson(dataJson);

  print(parsedData); // Union.data(value: 42)
}

Result:
{}
Unhandled exception:
MapperException: Failed to decode (Union): Cannot instantiate class Union, did you forgot to specify a subclass for [ type: 'null' ] or a default subclass?
#0      UnionMapperElement.fromMap (package:cricinfo_modules_flutter/domain/video/viewmodel/union.mapper.dart:50:7)
#1      UnionMapperElement.decode.<anonymous closure> (package:cricinfo_modules_flutter/domain/video/viewmodel/union.mapper.dart:46:20)
#2      checkedType (package:dart_mappable/src/mapper_utils.dart:78:14)
#3      UnionMapperElement.decode (package:cricinfo_modules_flutter/domain/video/viewmodel/union.mapper.dart:37:30)
#4      TypeSwitcher.apply.<anonymous closure>.<anonymous closure> (package:type_plus/src/type_switcher.dart:103:22)
#5      TypeSwitcher.apply.$params (package:type_plus/src/type_switcher.dart:87:22)
#6      TypeSwitcher.apply.<anonymous closure> (package:type_plus/src/type_switcher.dart:101:17)
#7      TypeSwitcher.apply.$args (package:type_plus/src/type_switcher.dart:58:22)

Project dependencies:
dependencies:
  dart_mappable: ^2.0.3
  equatable: ^2.0.5
  freezed_annotation: ^2.2.0

dev_dependencies:
  build_runner: ^2.3.3
  dart_mappable_builder: ^2.0.2
  freezed: ^2.2.0

Framework specs:
dart --version
Dart SDK version: 2.19.3 (stable) (Tue Feb 28 15:52:19 2023 +0000) on "macos_arm64"

flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.7.6, on macOS 12.6 21G115 darwin-arm64, locale en-IN)
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
[✓] Xcode - develop for iOS and macOS (Xcode 14.2)
[✓] Android Studio (version 2022.1)
[✓] VS Code (version 1.76.0)
[✓] Connected device (1 available)
[✓] HTTP Host Availability

Can you please help

Ok seems like encoding using the supertype (here Union) is broken in v2.
You can currently fix this by casting to a concrete subtype (here Data) like this:
var data = Union.data(42) as Data;
Also this is fixed in the v3 prerelease. There isn't really an easy fix I can do for v2 though.


Fixing lint errors in pkg/gcp

Fixes https://github.com/GoogleCloudPlatform/k8s-multicluster-ingress/issues/44
Ref #61
Fixing lint error for pkg/gcp
All sub packages in pkg/gcp have similar changes. The pattern was already reviewed for url map in https://github.com/GoogleCloudPlatform/k8s-multicluster-ingress/pull/180.
Have hence combined all the changes in this single PR.
Apologies if its too big and hard to review :)
cc @G-Harmon

phew, that was long!

Reviewed 22 of 22 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Thanks @G-Harmon :)


my game stops at process factory

launch 1.8.9 -commands -lwjgl -paulscode -lookup -jndi -noout -keep -exit
Launching version 1.8.9, 0985783d-fb22-4842-9950-a3f74d22990d
[08:03:07] [main/INFO] [Log4jPatcher]: Patching lookup in org/apache/logging/log4j/core/lookup/Interpolator
[08:03:07] [main/INFO] [Log4jPatcher]: Patching lookup in org/apache/logging/log4j/core/lookup/Interpolator
[08:03:07] [main/INFO] [Log4jPatcher]: Patching lookup in org/apache/logging/log4j/core/lookup/JndiLookup
[08:03:07] [main/INFO] [Log4jPatcher]: Patching lookup in org/apache/logging/log4j/core/lookup/JndiLookup
[08:03:08] [main/INFO] [ProcessFactory]: Game will run in C:\Users\Curry\AppData\Roaming.minecraft

Its because you use -noout.


Unable to get Tensofflow.NET working:  Source, Nuget, and  Docker Image

Hello, My team and I am clearly still missing something but I've personally spent a day on trying to get these libraries working and no luck. Any assistance getting past this would greatly be appreciated.  At the moment I just trying to get team started using the stack and failing miserably; Given we cant even get "Hello World" executing.
We have already gone in and set build platform targets to x64 as issue https://github.com/SciSharp/TensorFlow.NET/issues/252 suggests. Unfortunately this has not resolved the issue. We still are met with unable to find dll for UnitTests.
From source( using stated build platform target ) unit tests fail:

When I tried a simple console app (.NET Core 3.1 using latest NuGet Packages):

Existing Examples (.NET Core 3.1 using latest NuGet Packages)

When debugging while referencing to local build of assembly Both of those fault immediately when attempting to create a new instance of tensorflow object.
Docker Latest ScSharpCube(.NET Core 2.2):

If there is additional supporting documentation needed to determine cause let me know.  Ill post it straight away.

@Lithonite Can you setup a TeamViewer to allow remote control? I might connect to help.

Sure I up for using team viewer to address the issue. By chance do you have Teams or some other means to screen share?

This is a result of the compiled instructions not being supported on this series following series of processors.
http://www.cpu-world.com/Sockets/Socket 1366 (LGA1366).html
If you are running into this issue. Time to upgrade your machine(s).

Related to https://github.com/SciSharp/TensorFlow.NET/issues/340


运行的时候报错 Module not found: Error: Can't resolve '@netless/cursor-tool' in

问一下，这个指令是在哪个目录下运行的

在项目的根目录执行，如果还是不行，请把终端的输出保存成 log文件发一下，我们会协助你进行排查

I am also getting this error.
No idea of how to fix it.

@future-mine You have to build all packages before running the main app. Simply yarn && yarn lerna run --parallel build:lib.

And to run this,
I should to go to whilteboard folder, right?
And I should run yarn dev, I suppose.

This is what I am getting right now.

That's because your operating system is Windows and the regex to match this file of @netless/slide-prefetch failed at / with \:
https://github.com/netless-io/whiteboard-demo/blob/8042704b6f31227b266d80f3cba9d9a13cf305e9/whiteboard/webpack.config.js#L66-L73
You can manually edit this webpack config to replace \/ in the regex with [\/\\] to fix that.

I will check.
So eventually, I want to replicate the current whiteboard to my application.
Is it possible to use it without lerna?
For example, Can I install the bunch of @netless/... modules just by npm install or yarn add?

Yes you can. These packages are published to npm. BTW, it is also possible to write these components by yourself to make them suit your needs.

That sounds good to me.
Thank you.

I don't understand what I should as the env variables.
Can anyone help me, please?

And is there any live agora/netless community such as discord?

The variables you quoted are used in the upload-button package.
You can find our Slack group at https://github.com/netless-io/flat.


feat(A2): add TypeId< and TypeIdVector<

Hi, i also developped a tool that does pretty much the same but i think having a single repo which we rely on could be a great thing for the ecosystem.
Sadly, as this issue  states, the events.json file isn't complete enough to read a message.
This PR aims to fix this by adding two new types along with Vector :

"TypeId" means the entity we have to read can be identified by its id that we get reading a short from the buffer
"TypeIdVector<Short, BaseEntity>" means the buffer contains a vector with a length encoded in a short, and for every item of the array we have to read a short to get the id of the matching entity.

I can clarify any point if needed !
PS: I'm working on a datafus-parser npm library 🤫

Hello @ledouxm ,
First of all thank you for your contribution, it looks promising. I don't have a lot of time to work on the project. I will take a look at the PR asap.


Preference page : Allow to exclude specific yml/property files

Add support for excluding property files, not just the properties names as it is today.

Add new table to the preference page to store filenames of property files that should be excluded
Check if the file is excluded in PropertiesPropertyPlaceholdersSmartCompletion::isValidExtension and YamlPropertyPlaceholdersSmartCompletion::isValidExtension

So the question is, should we allow regex as filename pattern? Right now it's implemented as String.contains


Is it dead?

Hi @yaniswang!
Thanks for that library, I think it solved a lot of issue for many people.
But for now it seems to be mostly abandoned, with lots of unclosed PRs and issues, outdated dependencies.
Is HTMLHint dead?

@yaniswang there are likely members of this community that can assist as collaborators.

@yaniswang If you do not plan to maintain this project that's fine, but please do not leave the community hanging, we can help with the maintenance as many of us depend on this package.

Absolutely no. ^_^

Very disappointing. Especially seeing that VSCode was looking to recommend this project. There's so many open PRs too, lots of people interested in helping. At the least, it would be nice to see the existing work migrated to another project that enables more community participation.

Some alternatives can be found here

@AndrewCraswell Just needs someone willing to take it on. =]

Hey @everyone! I know the issue is old, but I just wanted to tell you that I sent an email to the owner of the repo to ask if he could add me as a collaborator or give the ownership of the project. I'm willing to help the project to go further and work as everyone is expecting! 😉

First of all, it could stay with HTMLHint and become a fork of this repo. Then you can ask at npm if the old package can be replaced: https://docs.npmjs.com/misc/disputes
I would be very interested in helping to build a new HTMLHint or HTMLLint (hobby time)
Whereby a Hinter only provides annotations and a Linter also fixes and offers more configuration.
I would not call it HTML5*, because HTML* could be synonymous for HTML4 XHTML or HTML6, why directly limiting?
In a new Repo we can create new Proposals and build up from stretch maybe

I totally agree with having a "Linter" and not just a "Hinter". About HTML5, I thought about that too... Let see what other people think about.
Regarding the fact to build up from scratch, I would rather to start with something existing and updating it because it could take less time I think... but I may be wrong.

Ok, I reviewed GitHub a bit and found an existing Linter https://github.com/htmllint/htmllint
I do not know this linter and have not tried it
I think we now have 3 options:

Write a new (competing) HTMLLint
Make a fork of this HTMLHint, leave it behind and try it to stay compatible with the old config + extend the config to further new formats
Cooperate with the HTMLLint and push it (for example, there is no VSCode extension to support this linter)

In all 3 options I would like to rewrite it in TypeScript because I think that helps to find bugs and it is easier to maintain in the long term

@ArmorDarks I think the goal is not to go too far in a first time, but upgrading the existing. So the option 2 seems the most reasonable. And regarding TypeScript, even I love it too, plain JavaScript is the best in a first time, small steps for a biggest impact.

Agreed
Let me know when you start, so that I can help

Seems like no one mentioned markuplint so I guess I'll just leave it here in case it's helpful.

@hacknug, I evaluated markuplint for my angular application and it didn't go well when I plugged it in.  It did not seem to care for custom angular components, and had some of the indecipherable error messages.

@vort3xxx @hacknug I experienced the same thing
markuplint seems not to be ready in any way, it is heavily under construction

Hi @everyone!
In 8 days, if we don't have any news from @yaniswang, I'll be added to the list of contributors for HTMLHint! 🍾
It's going to be a lot of work, going through all bugs and PR... But I still believe, for now, this is the best thing to do. Moving to something else is probably too soon. First, reduce the among of issues, after, we can all decide the best direction to take!

@thedaviddias have you talked to Github?

I'm sorry, i have no time for it.
Who can take over HTMLHint?

Hi @yaniswang - great to hear from you! I'd love to be a contributor. I've been a HTMLHint user for many years and I'm very active on GitHub and manage and contribute to many open source projects.

Who can be the owner of the project?

I'd be very happy to be the project owner, but I'm fine with @thedaviddias being project owner if I can be on the projects team. :-)

Welcome @yaniswang! I would vote for @thedaviddias as well given his enthusiasm and action thus far.

Hey @yaniswang! Awesome to hear from you, NPM and I were sending you various emails since a month. You can put me as owner of the project, and I'll add people that have been around to help in correcting bugs and issues we have today. @coliff and @tdmalone, thanks for the support, I eager to work with you on that project!

Probably last message on this issue, @yaniswang just transferred to me the ownership of the project. I'm going to create a Gitter channel for us to be able to exchange and update "Projects" on Github to see what needs to be done, prioritize etc... @ArmorDarks I think  that you can close this ticket 🤗

@thedaviddias please give me your npm account

This is my NPM account: https://www.npmjs.com/~thedaviddias

Done

Do you have godaddy account? i transfer the domain htmlhint.com to you

I don't have but you can eventually send me by email the code for the transfer I think... (my email<EMAIL_ADDRESS>

i had send you the transefer code from email.

this issue can be closed


@conda-forge-admin, please rerender

Issue:

Environment (conda list):

$ conda list

Details about  conda and system ( conda info ):

$ conda info

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/libspatialite-feedstock#35.


if elseif on same bool

This line https://github.com/ACEsuit/ACE1pack.jl/blob/b1505b8d8e5ac500fcce8e8216b4c3fd41dfe223/src/read_params.jl#L20 seems to use the same logical test for the initial if and the final elseif, which seems wrong (unless julia had rather different elseif syntax compared to other languages, in which case feel free to close this issue).

That's a typo, thanks for pointing this out.


mlx - minor fixes in nn.py

Codecov Report
Attention: Patch coverage is 0% with 2 lines in your changes are missing coverage. Please review.

:exclamation: No coverage uploaded for pull request base (mlx@c84d052). Click here to learn what that means.

Files
Patch %
Lines

keras/src/backend/mlx/nn.py
0.00%
2 Missing :warning:

Additional details and impacted files
@@          Coverage Diff           @@
##             mlx   #19572   +/-   ##
======================================
  Coverage       ?   53.32%
======================================
  Files          ?      506
  Lines          ?    45810
  Branches       ?     8466
======================================
  Hits           ?    24429
  Misses         ?    19793
  Partials       ?     1588

Flag
Coverage Δ

keras
53.32% <0.00%> (?)

keras-numpy
53.32% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Allow to register custom object formatter in error messages #455

I would like to use this feature, so I put together a quick prototype. If you rather not expose Representation, I'm open to every other approach.

Cool, will look into it soon but first I'm preparing 2.4.0/3.4.0 releases

No need to hurry.
Nice to hear that 2.4.0/3.4.0 will be released soon. :smiley:

I think this is on the right track, I'm going to continue it, I want to refactor the Representation hierarchy and introduce a few things:

a way to register a representation globally (not having to specify it for each assertions)
a way to register in the current representation the description of a specific type without having to create a new representation (I think it is only possible using Java 8 lambda though)

Great, these enhancements will make the feature much more powerful. :+1:

Was merged and greatly improved upon by @joel-costigliola ! Thanks!
See https://github.com/joel-costigliola/assertj-core/issues/455 for details.


A callback that publishes the uid of the latest run to a PV

@klauer I think you have one of these already?  Can that get pushed upsteam?
Do we want to split the call-backs out into their own repo?  I suspect we will want to do that eventually but not sure if it is worth it now.

Yea eventually, but I would say it is still too soon.
On Thu, Nov 5, 2015 at 8:46 AM Thomas A Caswell<EMAIL_ADDRESS>wrote:

@klauer https://github.com/klauer I think you have one of these
already? Can that get pushed upsteam?
Do we want to split the call-backs out into their own repo? I suspect we
will want to do that eventually but not sure if it is worth it now.
—
Reply to this email directly or view it on GitHub
https://github.com/NSLS-II/bluesky/issues/220#issuecomment-154062135.

I'm sure you'll want to do things differently, but one or more of these classes could potentially serve as a starting point: https://github.com/NSLS-II-HXN/hxntools/blob/bs/hxntools/scan_monitor.py

Can you make a PR out of the broadcaster?

There are no callback modules in bs that depend on ophyd currently (scientific_callbacks.py, broker_callbacks.py, callbacks.py). Before I make a PR, am I correct in guessing that you'd like a separate ophyd_callbacks module?

Actually, can you put the PR into ophyd?

If you can give me a hint as to where it belongs there...

I would say create a top-level module callbacks.py in ophyd.
The suspenders should also move to ophyd as well.

Thoughts regarding ophyd module location, @dchabot?

Closing as this is been merged over on ophyd.


Add better handling for url between hash and internal

Close #2
This should also remove the need to do any encoding ahead of time when adding presigned URLs

@cholmes made updates which should fix the initial load and sample buttons
I changed the code to manipulate the value property of the input elements rather than defaultValue since that's a little better practice (I think)

Looks really great, thanks!


Updated scheduler to properly use runOnce intervals.

Fix #1444
Signed-off-by: Brandon Forster<EMAIL_ADDRESS>

Codecov Report

Merging #1480 into master will increase coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1480      +/-   ##
==========================================
+ Coverage   17.75%   17.78%   +0.03%
==========================================
  Files          88       88
  Lines        9068     9069       +1
==========================================
+ Hits         1610     1613       +3
+ Misses       7307     7306       -1
+ Partials      151      150       -1

Impacted Files
Coverage Δ

internal/support/scheduler/router.go
0% <ø> (ø)
:arrow_up:

internal/support/scheduler/interval.go
32.16% <ø> (-0.03%)
:arrow_down:

internal/support/scheduler/schedulecontext.go
76.47% <100%> (+1.47%)
:arrow_up:

Continue to review full report at Codecov.

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


Implement health check for kafka

Description
Previous attempts at implementing a health check for kafka have not been successful. We should make a new attempt at this.
A working hypothesis is that the background services (kafka consumers) might be getting in the way of the health checks.
Additional Information
No response
Tasks
No response
Acceptance Criterias
No response

Canceled


.NET 8 Investigation

This PR is not intended to be merged, but instead lets us experiment with .NET 8 release candidates within the GitHub Actions environment prior to its official release.

Closing this experimental PR without merging, now that the experiment is complete and upcoming .NET 8 adoption is well understood.


Separate analysis deps

Current builds fail analysis when using withDeps: true, separating with
and without Deps analysis test cases to have batter overview on what is
actually broken.

Detail on withDeps Task error:
errors:
    - severity: Error
      description: '[Analyzer] technology-usage.mvc-01220: Malformed version: ${spring-framework.version}'
    - severity: Error
      description: '[Analyzer] technology-usage.mvc-01220: Malformed version: ${spring-framework.version}'
    - severity: Error
      description: '[Analyzer] technology-usage.embedded-framework-08400: Malformed version: ${spring-framework.version}'


Remove the account services url from the header

What is the context of this PR?
Remove the account services url/"My Account" link from the header.
Amend the text on session expiry page so that it matches the design here.
How to review
Open any schema and check that the link no longer appears in the header.
Check to make sure that the text on the session expiry page has been amended to match the design above.

 Remove account_services_url from header


[react-native-hms-map] Add support for custom behavior on cluster click

Description
I am using a react-native-hms-map to show orders on a map. Sometimes there are double orders (two at one point) and due to the standard behavior of clicking on the cluster (centering) it is impossible to select these orders even at the maximum zoom.
Expected behavior
Ability to use custom behavior onClusterClick
Current behavior
Standard behavior of clicking on the cluster (centering)
Environment

Platform: React-Native
Kit: Map
Kit Version 6.0.1-304

Hi @ilpwork,
According to our tests the issue is caused by the native Android Huawei Map SDK. Huawei React Native Map Plugin is a wrapper of native Android Huawei Map SDK thus inherits the behaviours of it. For any question and suggestions please submit a ticket online.
Thank you for your interest.


Poor performance writing layered HFBs

Some context

A LayeredHorizontalFlowBarrier was introduced, renamed now to SingleLayerHorizontalFlowBarrier as it only worked for single layers.
In the LHM, and other regional models, multiple layers can be assigned the same set of lines (same GEN file).

Issue
User testing from @luitjansl showed that writing the steady-state LHM takes ~2 minutes without HFBs and ~25 minutes with HFBs, which is quite killing for user's productivity.  The reason is that a lot of calls are made to xu.snap_to_edges.
This is because:

A call needs to be made for every linestring, as there is no support for summation of resistances in xugrid
A call needs to be made for every single layer, as we are dealing with SingleLayerHorizontalFlowBarriers

Potential solutions
I see three potential optimizations, in order of perceived difficulty to implement, low to high:

Support in iMOD Python that lines assigned to multiple layers are snapped to cell edges only once. Either by creating a LayeredHorizontalFlowBarrier and combine all hfb systems that point to the same GENfile into it. Or by caching snapped edges for each file.
Add support in xugrid to snap and aggregate to edges.
Create separate optimized snapping functions for structured grids

Final note
This is not so much of a problem for the HorizontalFlowBarrier with a ztop and zbottom. There snapping only is done once for all layers automatically.

UPDATE: Xugrid 0.11.0 supports aggregating multiple linestrings now
https://deltares.github.io/xugrid/changelog.html


Try to enforce openblas feature.

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

More efforts to force openblas rather than mkl.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Not clear here whether the package should explicitly declare the openblas feature here or whether requiring the package is enough. The conda documentation of this has disappeared, so it's hard to know.


Allows multiple keywords as conditions

Allows multiple keywords as group visiable conditions, it is very useful when a parameter shares same keywords.

This function will be refactored into ShowIf(propName, comparisonMethod, value) in the future

Great! looking forward to it.


Notes that repeat annually

For the purpose of creating holidays that happen on the same day every year.

Good idea, I will look into adding this.

I added the ability to repeat notes weekly, monthly or yearly. This will be part of the 1.1.x release which should hopefully be ready in the next few days.

This has been added in version 1.1.0


[Bug][API] Creating folders with multiple threads will result in multiple identical folders

Multiple threads calling the API to create folders will result in multiple identical folders。
The API to create folders is expected to be thread-safe
dolphinScheduler version is 1.3.1

Who wants to fix it？

Who wants to fix it？

I will  fix it

How is the progress of this issues now

https://github.com/apache/incubator-dolphinscheduler/pull/3697

Is this Issue still Open?!!... wanted to work on it if it is still open

The problem has been resolved


[Dashboard] DE-1062 update website validation

Codecov Report

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

@@           Coverage Diff           @@
##             main      #18   +/-   ##
=======================================
  Coverage        ?   83.51%
=======================================
  Files           ?       41
  Lines           ?     1377
  Branches        ?      370
=======================================
  Hits            ?     1150
  Misses          ?      225
  Partials        ?        2

Flag
Coverage Δ

unittests
83.51% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


Added HttpSesion mock for the session API tests

Changelog
Added HttpSession mock inside XskApiSuiteTest.java
Changed
Fixed the nullpointer for the Session API test

Related to https://github.com/SAP/xsk/pull/231


Animation broken. Following wrong bone

Looks like animation is broken. This is basically the example plus the 2 lines copying the animation from the original mesh. The skeletal mesh works fine is it's just static without the animation.
The video attached is using the skeletalmesh and animation from the content examples. I've tried with other skeletal meshes and same issue.
This is tested with 4.27.2
void ADuplicateSkeletalMesh::Duplicate()
{
	TArray<FMeshSurface> OutSurfaces;
	TArray<int32> OutSurfacesVertexOffsets;
	TArray<int32> OutSurfacesIndexOffsets;
	TArray<UMaterialInterface*> OutSurfacesMaterial;

	USkeletalMeshComponent* SkeletalMeshComp = SkeletalMeshActor->GetSkeletalMeshComponent();
	FRuntimeSkeletalMeshGenerator::DecomposeSkeletalMesh(
		SkeletalMeshComp->SkeletalMesh,
		OutSurfaces,
		OutSurfacesVertexOffsets,
		OutSurfacesIndexOffsets,
		OutSurfacesMaterial);

	USkeletalMeshComponent* comp = FRuntimeSkeletalMeshGenerator::GenerateSkeletalMeshComponent(
		this,
		SkeletalMeshComp->SkeletalMesh->Skeleton,
		OutSurfaces,
		OutSurfacesMaterial);

	comp->AnimationData = SkeletalMeshComp->AnimationData;
	comp->SetAnimationMode(EAnimationMode::AnimationSingleNode);
}

Skeletalmesh on the left is the original and the right is the duplicated one.
https://user-images.githubusercontent.com/70504591/173758191-6dccc622-046f-4ea2-8370-65561c03f5e5.mp4

Found the bug and requested merge request for fix.


Several changes that deprecate the old written tutorials in favour of Lessons section

Good stuff! Do you also want to replace the "tutorial" link in the main doc menu with a "lessons" one? http://take.ms/L1V2g. On that note: Why is the "Lessons" navigation item in the main top menu highlighted with a blue border even if I'm not on that page? This might have been a measure to highlight the existing "Training" item above the others (questionable as well), but that only works IMHO when highlighting a single entry, not multiple ones. My suggestion is to remove the highlights on both entries. /cc @camfindlay

Yeah I'd be open to making the language consistent. "Lessons" is what we are calling things and it is easy enough to change the folder name to rejig the nav. Just make sure to put in a 301 redir too.

The idea is that the "Tutorials" section of the docs will no longer exist, and we'll set up 301s from all the old links to go to the appropriate lesson. This pull request is a bit of an interim measure. It deprecates most of them, but leaves them available for users who may want to still use them for whatever reason. More importantly, not all of the topics have been covered by Lessons at this point, so we can't do a full hatchet job quite yet. Sig is keen to get to a mostly-deprecated state for the UK trainings, which is why this is getting ushered through a bit prematurely.

Where are we at with this? Good to merge?

Merged - sorry for the delay, and thanks for your hard work on improving docs, Aaron :)
Does anybody know what the deal is with the double highlight in the menu though? Looks odd.

@chillu highlight I think gets set in the CMS, can be changed, what are you proposing?

@camfindlay As I said above: "My suggestion is to remove the highlights on both entries."

I agree only having 1 highlight makes more sense, as for dropping both... I think that was @clarkepaul 's design call... happy to only highlight lessons (as this is legitimately something we want to get people into) :)

Lessons was added after the IA was made that I did so I haven't been involved how we present these areas. I think the idea was that lessons would be different enough from tutorials to complement each other rather than compete. Happy to discuss with someone if need be.

Cheers @clarkepaul the new lessons are set to replace the old tutorials. We also had that training link in the same space. I'd be keen to drop the highlight on training and keep on lessons as really only one call to action should be highlighted and I think lessons adds the most value for new community members so should be easy to find (I'd actually be keen to swap the homepage call to action from "demo" to the lessons).


Add --skip-recursive-build-dependents flag

This is based on the discussion at Homebrew/brew#12455.
My intention is for workflows in Homebrew/core to always pass
--skip-recursive-build-dependents, perhaps with a label that causes
the workflow to omit the flag.
I'm tempted to not make this an option at all, but this changes testing
behaviour in external taps too, and I'm a little wary about forcing
those changes when they haven't been asked for.

I'm tempted to not make this an option at all, but this changes testing
behaviour in external taps too, and I'm a little wary about forcing
those changes when they haven't been asked for.

I would say: just do it. Anything involving large numbers of dependents is unlikely to be hit by Not Us and this repository is primarily used by us and not really supported otherwise.

I would say: just do it. Anything involving large numbers of dependents is unlikely to be hit by Not Us and this repository is primarily used by us and not really supported otherwise.

Ok, done.


@tsed/di overrides native metadata from 'reflect-metadata'

Description
I have been actively working on integrating the Automock library (https://github.com/automock/automock) with various DI frameworks, including Ts.ED 🎉 . During the exploration of @tsed/di metadata mechanism, I came across a potential limitation concerning the @Inject decorator.
Presently, when using the @Inject decorator, it overrides the native metadata key of the parameter see here. This behavior can have unintended consequences, particularly when integrating with external libraries like Automock, which heavily relies on accurate metadata to automatically mock class dependencies. See:
https://github.com/tsedio/tsed/blob/83bc31d70aa58c672df3aea05daab38045b3d704/packages/di/src/common/decorators/inject.ts#L47
To address this concern and ensure seamless integration with any library, I propose the addition of an extra metadata key, named original:paramtypes, in the @tsed/di infrastructure. This new key will store the original parameter types before applying any decorator modifications. By preserving the original metadata, the @Inject decorator can continue to function as intended, while also enabling 3rd party libraries to access the unaltered parameter types when needed.

:tada: This issue has been resolved in version 7.35.0-beta.5 :tada:
The release is available on:

v7.35.0-beta.5
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 7.35.0-rc.1 :tada:
The release is available on:

v7.35.0-rc.1
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 7.36.0-rc.1 :tada:
The release is available on:

v7.36.0-rc.1
GitHub release

Your semantic-release bot :package::rocket:

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

v7.36.0
GitHub release

Your semantic-release bot :package::rocket:


SAP HANA, express edition, extract installation files and execute script

This does not work. This only installs the server. The XSA does not get installed. I think the parameters are missing or The script file needs to be changed. See at the bottom of this log. Even though I selected all, on the parameter is saying no and my Hana install is successful but the LCM and other components don't get installed.
Component [all]: all
Enter local host name [sshxe]: sshxe
Enter SAP HANA system ID [HXE]: HXE
Enter HANA instance number [90]: 00
Password must be at least 8 characters in length.  It must contain at least
1 uppercase letter, 1 lowercase letter, and 1 number.  Special characters
are allowed, except \ (backslash), " (double quotes), and ` (backtick).
Enter HDB master password:
Confirm "HDB master" password:
Do you need to use proxy server to access the internet? (Y/N): n
SAP HANA Interactive Education, or SHINE, is a demo application that makes
it easy to learn how to build applications on SAP HANA Extended Application
Services Advanced Model.
Install SHINE? (Y/N) : y
The SAP HANA smart data streaming option processes high-velocity, high-volume
event streams in real time, allowing you to filter, aggregate, and enrich raw
data before committing it to your database.
Install SAP HANA Smart Data Streaming? (Y/N) : y
##############################################################################
Summary before execution
##############################################################################
HANA, express edition installer : /home/srini/HANA_EXPRESS_20
Component(s) to install       : HANA server, Application Function Library, and Extended Services + apps (XSA)
Host name                     : sshxe
HANA system ID                : HXE
HANA instance number          : 00
Master password               : ********
Proxy host                    : N/A
Proxy port                    : N/A
Hosts with no proxy           : N/A
Install SHINE                 : Yes
Install SDS                   : Yes
Proceed with installation? (Y/N) : y
Installing HDB server...
SAP HANA Lifecycle Management - SAP HANA Express Edition <IP_ADDRESS>.1491294693

Start reading from input channel...
... Done.
Summary before execution:
SAP HANA Express Edition System Installation
Installation Parameters
Remote Execution: ssh
Use single master password for all users, created during installation: Yes
Database Isolation: low
Installation Path: /hana/shared
Local Host Name: sshxe
SAP HANA System ID: HXE
Instance Number: 00
Local Host Worker Group: default
System Usage: development
Location of Data Volumes: /hana/shared/data/HXE
Location of Log Volumes: /hana/shared/log/HXE
Directory containing custom configurations: /home/srini/HANA_EXPRESS_20/DATA_UNITS/HDB_SERVER_LINUX_X86_64/configurations/custom
Certificate Host Names: sshxe -> sshxe
System Administrator Home Directory: /usr/sap/HXE/home
System Administrator Login Shell: /bin/bash
System Administrator User ID: 1001
ID of User Group (sapsys): 79
Restart system after machine reboot?: Yes
Inter Service Communication Mode: ssl
Software Components
SAP HANA Database
Install version <IP_ADDRESS>.1491294693
Location: /home/srini/HANA_EXPRESS_20/DATA_UNITS/HDB_SERVER_LINUX_X86_64/server
SAP HANA AFL (incl.PAL,BFL,OFL,HIE)
Install version <IP_ADDRESS>0.1491308763
Location: /home/srini/HANA_EXPRESS_20/DATA_UNITS/HDB_AFL_LINUX_X86_64/packages
SAP HANA Database Client
Do not install
SAP HANA Smart Data Streaming
Do not install
SAP HANA XS Advanced Runtime
Do not install
Finally this is the log where the install stops
SAP HANA Lifecycle Management - Update Repository 2.1.25

Start reading from input channel...
... Done.
Importing delivery units...
Importing delivery unit HDC_SYS_ADMIN
Importing delivery unit HANA_SYS_ADMIN
Update Repository done
Log file written to '/var/tmp/hdb_HXE_UpdateRepository_2017-06-03_04.37.16/hdbupdrep.log' on host 'sshxe'.
Enable statistics server...

10: authentication failed SQLSTATE: 28000

Failed to enable statistics server.

It looks you are installing the server-only version. With server-only version, XSA will not be installed.

Enable statistics server...
10: authentication failed SQLSTATE: 28000
Failed to enable statistics server.

What special character(s), if any, do you have in the password?
Are you able to login to the HANA server through hdbsql after the failure? Login to "sshxe" as "hxeadm" and run:
hdbsql -i 90 -d SystemDB -u SYSTEM -p


Ensure full control over back button and close button

User reports not able to hide close and back buttons. Need to either reproduce this or close as confirmed working.

Hey! Could you add more details about this issue?

Sure. A user of this library expressed a concern that the onBack and onClose props may not be used or exposed consistently. This issue was opened to remind us to look into that concern.
The onBack prop is called anytime the "back" button is clicked inside a child view. Looking for onClose, I see this prop is not part of the API, so that may just be a misunderstanding by the user.
Actually, I am going to close this issue, since the original reporter later said they could not confirm it themselves. If there is a known case where onBack is missing or not working, I am happy to reopen. Thanks!


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/120964

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/121430

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/121432


Update the README for networking changes.

Current coverage is 98.31% (diff: 100%)

Merging #330 into development will not change coverage

@@           development       #330   diff @@
=============================================
  Files               38         38
  Lines             2852       2852
  Methods              0          0
  Messages             0          0
  Branches           136        136
=============================================
  Hits              2804       2804
  Misses              43         43
  Partials             5          5

Powered by Codecov. Last update 54839d4...2192e07


Video/audio integration not working

if i click on a video chrome opens only the preview picture of the video in chrome.
running os x 10.11.6 beta, chrome 52 beta and yakyak 1.2.3
really nice client keep up the great work.

Right, thank you.
Does not look very difficult to do. Someone wants to try ?


Initial upload

Travis tests have failed
Hey @liyasthomas,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.
TravisBuddy Request Identifier: bc5c5210-6411-11ea-b857-47bc0e3c428e

Hey @liyasthomas,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 2cd18920-6412-11ea-b857-47bc0e3c428e

Travis tests have failed
Hey @liyasthomas,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.
TravisBuddy Request Identifier: 70a94880-6413-11ea-b857-47bc0e3c428e

Travis tests have failed
Hey @liyasthomas,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.
TravisBuddy Request Identifier: 0a8c6cc0-6414-11ea-b857-47bc0e3c428e

Hey @liyasthomas,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 3bb2b600-6415-11ea-b857-47bc0e3c428e

Hey @liyasthomas,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: fa772280-6459-11ea-b857-47bc0e3c428e

Travis tests have failed
Hey @liyasthomas,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.
TravisBuddy Request Identifier: d53a4d20-645f-11ea-b857-47bc0e3c428e

Travis tests have failed
Hey @liyasthomas,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.
TravisBuddy Request Identifier: 793c3b90-6460-11ea-b857-47bc0e3c428e

Travis tests have failed
Hey @liyasthomas,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.
TravisBuddy Request Identifier: 37bd6e10-6464-11ea-b857-47bc0e3c428e

Hey @liyasthomas,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: db321b30-6465-11ea-b857-47bc0e3c428e

Hey @liyasthomas,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: cc1b5b30-6469-11ea-b857-47bc0e3c428e

Travis tests have failed
Hey @liyasthomas,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.
TravisBuddy Request Identifier: 8f8c5b40-646b-11ea-b857-47bc0e3c428e


1.22 Docker builds failing

openzipkin / docker-zipkin is failing on Travis. The logs, along with what I can see on https://quay.io/repository/openzipkin/zipkin, make me think that Quay's build trigger is not working.
We're excited to get our hands on the 1.22 container image.

Trying to rebuild to see if Quay wasn't responding during the last build

So looking at the build more, someone with quay.io access will most likely need to finish this build.
cc @openzipkin/devops-tooling

Honestly I don't know why a build-trigger on zipkin repo ended with a noop build on quay. I followed the RELEASE instructions for a manual build/publish and all good now. It wasn't automatic as I needed to manually run the build-trigger for the zipkin image. I commented out everything in release.sh before synchronizing tags, and that ran it.

ps thanks for opening this! I'm not sure the next release will run, but there's a chance there was something off about this particular tag/image.. we'll see and re-open if it sticks

had to manually run the build trigger on the zipkin image while the release tag was going ... it picked up and proceeded. https://quay.io/repository/openzipkin/zipkin?tab=builds
I'm wondering if the web hook is screwed up

note the same thing happened for zipkin-dependencnies.. the build trigger didn't go

I had to simplify the tag regex to ^tags/[0-9]+.*.. now good


The arch figure doesn't comform to the code

I found that in the arch figure of gated CNN block, the concat part is missing, and the gate branch and conv should share the same linear layer. This adjustment would bring the figure closer to the intended inference.

Hi @kytimmylai ,

the conv can be standard depthwise convolution or depthwise conv on partial channels (like InceptionNeXt), controlled by the conv ratio.
the gate branch and conv branch do not share the same linear (linear + split).


Expose Finality Certificates via the Lotus API

I.e., some method like:
func ChainGetFinalityCertificates(from Instance) (<-chan FinalityCertificate, error)

@Stebalien Can we close this? It seems like we already have implemented a temporary one in Lotus?

Yeah, we want some way to subscribe to them.

@Stebalien do we want to ship this, or is the pooling API sufficient?
Or do we want to introduce new v2 API that is the equivalent to HeadChange where the head changing means there is a new ticket?

I think the polling API is fine and yeah, in the future, I think we should have a head-change API that uses finality certificates.


More Deployment details

Hi
I have built a running app on your framework. Thank you for your work, its been very helpful.
I am a bit unsure about the deployment steps for Azure however.
I have 'dotnet publish' running cleanly.
Once this is run the client dist folder is updated.
On a clean IIS azure website what exactly needs to be deployed? Just this dist folder or the whole code base (for the .net core element)? I am manually copying to the FTP folder on Azure.
Could we please get a little more documentation?
Thank you!

Did it end up working? You ideally want everything that dotnet publish builds up there yes!

Yes I have it deployed but I do have something wrong still.
Maybe you could help me with a quick look please?
Check out http://tilecasev3.azurewebsites.net/TileCase
The JS version works very well but the non JS version just renders and empty page.
This did work during testing but I have changed something in the code that is stopping it from rendering. There are no errors or exceptions thrown.
Any ideas?
Thank you.

From: Mark Pieszak<EMAIL_ADDRESS>Sent: 09 July 2017 15:43:50
To: MarkPieszak/aspnetcore-angular2-universal
Cc: Ben; State change
Subject: Re: [MarkPieszak/aspnetcore-angular2-universal] More Deployment details (#332)
Did it end up working? You ideally want everything that dotnet publish builds up there yes!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/MarkPieszak/aspnetcore-angular2-universal/issues/332#issuecomment-313927557, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFtLSeZllbGwZsXqV-ZBIuzAgOHU1n-sks5sMPU2gaJpZM4OSEZd.

It might be your base href, you have it as just <base href="/" />, but it looks like you're serving from a /TitleCase directory? Give that a shot!

I did change the routes recently. Would it be possible that could have broke it? I will give it a try shortly.
Out of interest would you like access to the code base?
This one is killing me, I'm about to start cutting code out until I can find what change broke it.

From: Mark Pieszak<EMAIL_ADDRESS>Sent: 10 July 2017 15:10:10
To: MarkPieszak/aspnetcore-angular2-universal
Cc: Ben; State change
Subject: Re: [MarkPieszak/aspnetcore-angular2-universal] More Deployment details (#332)
It might be your base href, you have it as just , but it looks like you're serving from a /TitleCase directory? Give that a shot!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/MarkPieszak/aspnetcore-angular2-universal/issues/332#issuecomment-314135882, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFtLSfrv5krDlBKkTmIkcSKjAeGrc9F2ks5sMj7EgaJpZM4OSEZd.

I have it sorted now. It was an issue in one of my components. It was hard to find as there was no error messages being thrown up. Thanks for the ideas though.
By the way is there a way to stop the flick/render between the server side and client side versions in the browser?
My app uses geolocation. The server side version uses a default location (for seo) while the client side uses the clients location. This leads to different data being shown in each mode.
How would you solve this problem? I'm looking for ideas.

From: Ben Cameron<EMAIL_ADDRESS>Sent: 10 July 2017 15:30:34
To: MarkPieszak/aspnetcore-angular2-universal
Subject: Re: [MarkPieszak/aspnetcore-angular2-universal] More Deployment details (#332)
I did change the routes recently. Would it be possible that could have broke it? I will give it a try shortly.
Out of interest would you like access to the code base?
This one is killing me, I'm about to start cutting code out until I can find what change broke it.

From: Mark Pieszak<EMAIL_ADDRESS>Sent: 10 July 2017 15:10:10
To: MarkPieszak/aspnetcore-angular2-universal
Cc: Ben; State change
Subject: Re: [MarkPieszak/aspnetcore-angular2-universal] More Deployment details (#332)
It might be your base href, you have it as just , but it looks like you're serving from a /TitleCase directory? Give that a shot!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/MarkPieszak/aspnetcore-angular2-universal/issues/332#issuecomment-314135882, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFtLSfrv5krDlBKkTmIkcSKjAeGrc9F2ks5sMj7EgaJpZM4OSEZd.

You might just need to put some kind of CSS overlay on it potentially for the server render, and remove it when the client finishes?
Are there other http calls that aren't using HttpTransfer, because that also can cause a flicker sometimes since calls are being ran twice. Common issue with all SSR solutions unfortunately.

npm run build:AoT can usually help give you errors in the command line as to where those errors are coming from.

Great, I like the sound of the CSS solution. Is there an an example of this out on the web somewhere?

From: Mark Pieszak<EMAIL_ADDRESS>Sent: 11 July 2017 12:58:21
To: MarkPieszak/aspnetcore-angular2-universal
Cc: Ben; State change
Subject: Re: [MarkPieszak/aspnetcore-angular2-universal] More Deployment details (#332)
You might just need to put some kind of CSS overlay on it potentially for the server render, and remove it when the client finishes?
Are there other http calls that aren't using HttpTransfer, because that also can cause a flicker sometimes since calls are being ran twice. Common issue with all SSR solutions unfortunately.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/MarkPieszak/aspnetcore-angular2-universal/issues/332#issuecomment-314435104, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFtLSSV6a9_NeZf7aLBvYy5QiqmCHo6sks5sM3FtgaJpZM4OSEZd.

Also I'm not sure if it got merged, but there's a current bug for lazy loaded routes flickering between server & client renders, not sure if that's what you're doing in this case, but that could be the issue as well :)

Cool, thanks for your help on this, I know I have lots of questions. I will try it shortly.
And the server side version will still render okay?

From: Mark Pieszak<EMAIL_ADDRESS>Sent: 11 July 2017 13:07:52
To: MarkPieszak/aspnetcore-angular2-universal
Cc: Ben; State change
Subject: Re: [MarkPieszak/aspnetcore-angular2-universal] More Deployment details (#332)
Nothing crazy just use [ngClass]="{ 'server-overlay': someBooleanFlag }" and switch that boolean flag when you're truly ready to render.
Also I'm not sure if it got merged, but there's a current bug for lazy loaded routes flickering between server & client renders, not sure if that's what you're doing in this case, but that could be the issue as well :)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/MarkPieszak/aspnetcore-angular2-universal/issues/332#issuecomment-314437477, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFtLSYccaGWaXdojFsqs3ACIdSHQ9JCgks5sM3OogaJpZM4OSEZd.


How to build a snap using this action?

How to build a snap using this action and upload to snapcraft store?

This action only supports bundles that tauri-cli / @tauri-apps/cli supports since it uses the cli to build&bundle the app the same way you'd do locally.
There are other actions like https://github.com/snapcore/action-build that can help you but you probably still need to set up the snap config yourself.


Port ToMe25's PR for sawing factories

Changes proposed in this pull request:

Port of #5327 I figured it would be quicker to manually port the PR rather than explain the slight differences done to the upgrading system and other parts of mekanism.

@ToMe25 if you could double check this branch works as intended that would be great. I inlined a helper method of yours and also added the missing blockstate files as the blocks and item variants were untextured.
I still need to test this more myself, and I am not quite sure about how clear it will be to people that if they want to multiprocess using the sawing factories the secondary outputs of all items need to be the same or one stack won't be processed. Though at least on the surface everything seems to be working properly and looks good.

Everything seems to work properly to me, so I believe this can be merged once a second set of eyes reviews it.


Disable swagger badge when route contains localhost

Useful, otherwise it shows "error" suggesting there's a mistake in the generated API docs.

Good point - but this issue belongs to hapi-swaggered-ui. hapi-swaggered does not have any control about that. (But one of them could offer a built-in validator.)
I opened an issue z0mt3c/hapi-swaggered-ui#5 to add the missing configuration options. But i'm not sure if should really toggle it based on "localhost" or the validation should just be off as default. I guess most apis are running private and not on localhost (except during development). What do you think?

Ah, true. That's UI stuff :)
Well, maybe we should check if NODE_ENV === production and if so, display the validator? Obviously, user should be able to explicitly overload that config by passing enableValidation boolean or something similar. How that sounds?

Seem the current swagger-ui doesnt even contain a default validation url anymore. Follow up in z0mt3c/hapi-swaggered-ui


How do you actually get the FK for a FSMKeyField Field?

How do I actually get the FK Model if I need to access it and get field values in the related model?

Please don't use issue tracker as Q&A website.
Here is http://stackoverflow.com/ for that.


Improved Hold notes for Venturers

These changes make them act like individual characters usually would act like on Hold notes.


how to make api call with https request

example:
api https://domain.com/api/rest/tokens

You can use api step to make API calls. Methods such as GET/PUT can be chosen, and headers / body can be supplied. More details here under API section - https://github.com/kelaberetiv/TagUI#api

I did the same. but it is throwing me the error as "FAIL Error: NETWORK_ERR: XMLHttpRequest Exception 101".

there is a need of disabling "ssl certification verification". Please suggest

It could be because TagUI's api step web engine is based on PhantomJS and not accepted by the web service. As TagUI supports integration with Python, you can consider using Python requests package and py step to make and save the request response. After that use TagUI load step to retrieve and processing accordingly using JavaScript, eg JSON.

@kensoh, we are trying as below.
Script:-
py begin
import requests
respose = requests.post('https://ipaddress:port/v3/auth/tokens',api_config.body)
py finish
Still, it is not working. Please suggest

api_config is a variable object in TagUI which is not accessible in Python.
For simple data, you can use py_step() to do that (eg in readme under cheat sheet pro steps).
For complex data, you can try saving the object JSON.stringify() using dump step. Then from Python parse that file using json library, before giving to requests.

Let me know again, if you run into any issue!


E21: Cannot make changes, 'modifiable' is off

The Problem
When I add the following to my vimrc:
" bottom terminal
function OpenBottomTerminal()
    " don't open multiple terminals:
    if &buftype !=# 'terminal'
        belowright split | term
        " don't highlight words in the terminal as red:
        :syntax off
    endif
endfunction
autocmd FileType * if (&ft!="gitcommit" || &ft!="vim-plug") | call OpenBottomTerminal() | endif

I get:
Error detected while processing function <SNR>2_install[1]..<SNR>2_update_impl:
line   54:
E21: Cannot make changes, 'modifiable' is off
E21: Cannot make changes, 'modifiable' is off
Press ENTER or type command to continue

when I run :PlugInstall
Removing the above code block from my vimrc resolves the error. What I'm trying to do is open a bottom terminal if the ft I'm opening isn't gitcommit (in other words, always open a terminal except for when opening the git commit message editor). My debugging efforts have only brought me to that codeblock above; I can't figure out why vim-plug is complaining about buffers.
If I run :set ma in all open buffers before running :PlugInstall, I still get the modifiable error.
:version
NVIM v0.4.4
Build type: Release
LuaJIT 2.0.5
Compilation: /usr/bin/clang -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -DNDEBUG -DMIN_LOG_LEVEL=3 -Wall -Wextra -pedantic -Wno-unused
-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wmissing-prototypes -Wimplicit-fallthrough -Wvla -fstack-protec
tor-strong -fno-common -fdiagnostics-color=auto -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM
_UNIBI_HAS_VAR_FROM -I/tmp/neovim-20201103-57912-svovf1/neovim-0.4.4/build/config -I/tmp/neovim-20201103-57912-svovf1/neovim-0.4
.4/src -I/usr/local/include -I/tmp/neovim-20201103-57912-svovf1/neovim-0.4.4/deps-build/include -I/Library/Developer/CommandLine
Tools/SDKs/MacOSX10.15.sdk/usr/include -I/usr/local/opt/gettext/include -I/tmp/neovim-20201103-57912-svovf1/neovim-0.4.4/build/s
rc/nvim/auto -I/tmp/neovim-20201103-57912-svovf1/neovim-0.4.4/build/include
Compiled by brew@Catalina

Features: +acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/local/Cellar/neovim/0.4.4/share/nvim"

Run :checkhealth for more info

Type:

[ ] Bug
[ ] Enhancement
[ ] Feature Request
[x] Question

OS:

[ ] All/Other
[ ] Linux
[x] OS X
[ ] Windows

Vim:

[ ] Terminal Vim
[ ] GVim
[x] Neovim

and after receiving the modifiable error, 2 more buffers are opened (and the original file loses syntax coloring):

and if I:

comment out the code block that causes the error (so no terminal is opened on start)
run :belowright split | term
run :PlugInstall

everything works perfectly

If I change belowright split | term to belowright split, the :PlugInstall command works (although with too many splits):

this resolves the issue:
" bottom terminal
function OpenBottomTerminal()
    " don't open multiple terminals:
    if (&buftype !=# 'terminal' && &ft !=# 'vim-plug')
        belowright split | term
        " don't highlight words in the terminal as red:
        " :syntax off
    endif
endfunction
autocmd FileType * if &ft != 'gitcommit' | call OpenBottomTerminal() | endif


Remove Google Analytics

Would you be willing to remove this Google Analytics 'functionality' from this theme? There are better alternatives if you must keep tabs on users' activity.
Some reputable alternatives can be seen here: https://switching.software/replace/google-analytics/

Yes, I'll replace it with some of the alternatives you mentioned when I get some time on my hands. Thanks for the feedback.


Update to Cadence 5a06fec82b4a

For contributor use:

[x] Targeted PR against master branch
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[x] Code follows the standards mentioned here.
[x] Updated relevant documentation
[x] Re-reviewed Files changed in the Github PR explorer
[x] Added appropriate labels

@turbolent what is with the tests?

@turbolent what is with the tests?

Cadence is now using some features that only exist in Go 1.18, so to make this update work we are going to have to update the sdk to 1.18 as well.

Oh I see. Yeah that should be updated then. Should we create a PR or will you? I'm OOO now for a while so can't do that. If nothing else we should have an issue.

@turbolent what is with the tests?

Cadence is now using some features that only exist in Go 1.18, so to make this update work we are going to have to update the sdk to 1.18 as well.

Oh I see. Yeah that should be updated then. Should we create a PR or will you? I'm OOO now for a while so can't do that. If nothing else we should have an issue.

I was going to do it in this PR unless the version upgrade requires a lot of changes.


ogre_helpers: Fixing mesh memory leak (untested cherry-picked to indigo)

MAJOR leak: no meshes are ever destroyed without removing the mesh from
the mesh manager. This gets really bad when drawing meshes with 50K
triangles at 10Hz (leak @ ~60MB/sec).
NOTE: THIS IS AN UNTESTED CHERRY-PICK

massif graphs from hydro branch: https://gist.github.com/jbohren/c19375ba3cf8e8355554

This should be fine to cherry-pick since it is not used by rviz directly but it might affect MoveIt plugins in Indigo if they changed from Hydro. However, I find that unlikely.
I will do a release of indigo now, so that this change can get into shadow-fixed for some soak time. It would be great if you could test this out on Indigo before it is synced with public.

@jbohren Your commit message is pretty unprofessional. I'll take the patch, but I'm reverting the commit and redoing it with an appropriate commit message.

Well I hope nobody was offended, it was merely me claiming victory over the memory leak that taunted me all afternoon.

I can only speak for myself, but I'm not offended. I didn't consider it aimed in any particular direction, it's just not appropriate for the commit history.

I'll stick to this one for memory leak patches from now on.
   /~~~~~\~\__                                ________--------~~~~~~~~~~~~~=
  /          @`\            ___--------~~~~~~~                            _-~
 |              ~)~)~)~)~O~~                                               ~=
 |       /      /--------'~~-__                                           -~_
  \_____/ `----'               ~~=__                                       _=
     |    \__\___                   ~==__                                  _=
    /         /"\"\     ,----.           ~~==__                            =_
   |      ____\_/_/    |,----.|                ~~--__                        =
   |            /"\"\  |`~~~~'|                      ~~==_                  ~_
  /|           |   | | |MEMORY|                           ~~==__            _=
M'  \___________\_/_/  |______|                                ~~===_________=


[FR] Most of the links redirect to the English version of the website.

This is a Bug Report
Problem:
With the French language, most of the links that reference this website are not prefixed with '/fr/'. When I'm in a French version, I automatically switch to the English version when using a link to another section of this documentation.
Proposed Solution:
Prefix all the unprefixed links on the French pages with '/fr' to redirect to a French version.
Page to Update:

776 results over 115 files.
This potentially represents a lot of files, so a lot of pages.

Do you see this as a bug ?
If yes, can I propose a pull request to fix it for pages that have a French version?

/area localization


2021-09-21 equity dashboard chart data update

Equity dashboard stats updates in this PR may be reviewed on staging - here.
After reviewing, if all looks well, approve and merge this Pull Request.
If there are issues with the data:

Note concerns or issues here by commenting on this PR
Work with Triston directly to resolve data issues
Alert the COVID19 site team in Slack (in the Equity page channel)

Good morning everyone - Below is my preliminary data review. @tomasleon, off to you!
:heavy_check_mark: Everything seems normal, and I observe no major red flags!
:bulb: Noticeable highlights:
:bulb: Black COVID Deaths/100K is the only category that shows a decrease in death rate (7.4/100K to 6.6/100K). This decrease is following 5 consistent weeks of increase.
:bulb: Unlike last week (statewide & all income brackets’ case rates decreased), $0-40K & $40-60K case rates increase this week. However, all other income brackets as well as statewide case rate decrease. Values are reasonable, and decreasing trends of mid-higher income brackets align with R&E trends. Statewide case rate is between $40-60K and $60-80K bracket cases/100K rates.
:bulb: Updated data shows that Quartile positivity reached a “spike” around Sep. 5 (highest it has been since ~Aug. 25),  reaching a value of 4.7 (statewide = 3.9) before decreasing to 3.8 as of Sep. 12.
Thank you!
1. Blue Box Updates:
:heavy_check_mark: All percentage values are appear reasonable —

Death rate for Latino people compared to statewide (Blue Box 1) decreases by 1% to 19% after 3 weeks at 20%
Case Rate for Pacific Islanders (Blue Box 2) remains identical to last week, at 43%
Death rate for Black people compared to statewide (Blue Box 3) increases by 1% to 14% (the highest this value has been since March 9th, 2021); this increase is consistent with recent increasing trends
Case rate for communities with median income (Blue Box 4) remains identical to last week, at 33%

2. Death Rate/100K by R&E:
:heavy_check_mark: Statewide COVID Deaths/100K increases to 5 after two weeks at 4.
:bulb: Black COVID Deaths/100K is the only category that shows a decrease in death rate (7.4/100K to 6.6/100K). This decrease is following 5 consistent weeks of increase.
:heavy_check_mark: All other categories show rate increases. Multi-Race continues to be the lowest COVID Deaths/100K rate.
3. Case Rate/100K by R&E:
:heavy_check_mark: Like the last two weeks, this week shows decreases in all Case/100K rates. Statewide decreases from 527 to 466.
:heavy_check_mark: All values appear reasonable. NHPI Cases/100K remains the highest rate. Multi-Race Cases/100K remains the lowest rate.
4. Testing Rate/100K by R&E:
:heavy_check_mark: Unlike last week (mixed, category-dependent changes), testing rate/100K returns to  increases in all Testing/100K rates. All values appear reasonable. NHPI Testing/100K remains the highest rate, Multi-Race Testing/100K remains the lowest rate.
5. Sanity Checks
:heavy_check_mark: Mortality rate/100K < Case rate/100K for all categories (as expected).
:heavy_check_mark: % of state population remains consistent across all categories for Cases/Deaths/Testing by R&E
6. Community case rate/100K by median annual household income bracket:
:bulb: Unlike last week (statewide & all income brackets’ case rates decreased), $0-40K & $40-60K case rates increase this week. However, all other income brackets as well as statewide case rate decrease. Values are reasonable, and decreasing trends of mid-higher income brackets align with R&E trends. Statewide case rate is between $40-60K and $60-80K bracket cases/100K rates.
7. HE Metric:
:heavy_check_mark: HE Quartile positivity continues to be greater than Statewide Positivity; the difference between Quartile & Statewide positivity decreases to 0.2 this week. X-Axis (Dates) successfully extended. Both Statewide & HE Quartile Positivity appear to decrease, consistent with recent trends.
:bulb: Updated data shows that Quartile positivity reached a “spike” around Sep. 5 (highest it has been since ~Aug. 25),  reaching a value of 4.7 (statewide = 3.9) before decreasing to 3.8 as of Sep. 12.
8. Data completeness (DC):
:heavy_check_mark: DC values did not show significant changes (≥0.6%) this week, except for a 0.7% increase in SO Cases DC.


Task9react

Add Stubs
100%
https://github.com/amitsiliguri/easy/issues/9

Can not directly merge to main


Install For All Users

Fixes #218 [Need Verification of functionality when upgrading from single-user installation to multi-user installation before merging it in.]

Looks good to me

Good to me.


SpecDrill is not logging

sometimes no logs are produced even if log4net.config file is present in same folder as specDrillConfig.json file.

fixed with commit e26f4750


refactoring the 'update_authors' function to try to update author rows as much as possible

Perform case-insensitive comparisons of author names after trimming leading and  trailing whitespace
Update existing author records to maintain author IDs as much as possible by comparing the key (combination of name, last_name, first_name, & first_initial), orcid ID, combination of lastName and firstInitial.
Handle updates to affiliations and ORCIDs without deleting the entire author record
Update author names if the order remains the same to maintain author IDs
Update author names if the order is changed by using temp order mapping

@azurebrd Thanks for checking this out.  Yes, it is complicated.
The end result from the author update should exactly match what PubMed has so I think it should be okay.
We will not update entries flagged as first_author, corresponding_author, or with a PERSON connection unless we are certain about the changes. This can be redefined later when the process is ready.
I have tested all PubMed updates over the past five weeks for all papers in ABC to make sure all cases are caught.

That's awesome, thanks @sweng66 !


Responsive FAQ template

Is your feature request related to a problem? Please describe.
Yes. I will be adding FAQ template which can be beginner friendly and responsive.
Describe the solution you'd like
This project will be build using Html,Css,Javascript.
Describe alternatives you've considered
NA
Additional context
Add any other context or screenshots about the feature request here.

sure @Sulagna-Dutta-Roy pls go ahead also add one clip while making a PR


musicbrainz integration is very imprecise

i love the fact we can search for tags in musicbrainz (really love it) but i need to say that more than 50% of the times it picks the wrong record, even if i choose all songs from an album  (that maybe are parts of various compilations ond so on, the app will probably choose the compilation over the original album, that maybe is already in tags. Oh, and it's also seriously slow
Anyway to correct this? :)

The way the Download MusicBrainz Metdata function works is Tagger uses the chromaprint fingerprint for the file and sends a query to AcoustId, which returns a list of MusicBrainz records, ranked from 0-100 (closer to 100, closer the match).
Tagger then picks the highest ranked record and sends a query to MusicBrainz to get all the information it needs for the tag.
On top of that, Tagger will also query CoverArtArchieve to get an album art for the album provided by MusicBrainz and will download that image and set it as the album art.
I will try to compare our MusicBrainz metdata action to that of MusicBrainz Picard and see how Picard picks a record and try to improve the accuracy of picking a record

As far as being slow, from the above description you can see that each file makes about 5 internet requests (AcoustID Query, MusicBrainz Record Query, MusicBrainz Release (Album) Query, CoverArtArchive Query, Downloading Cover Art Image)
This starts to pile up fast, for example, running the 'Download MusicBrainz Metadata' action of 5 files is about 25 internet requests made, which depending on your connection can be slow.
The best way that I can think of now to improve the speed is to maybe run up to 3 files on different threads handing about 15 requests at a time instead of 1 at a time per file.

Another side note, V2022.10.0 provides a new option in preferences called 'Overwrite Tag From MusicBrainzin which you can turn off and whatever fields already in the tag will not be overwritten/changed by theDownload MusicBrainz Metadata` function.

Ok, I may have found a way to speed up and improve the accuracy of the function.
AcoustId is able to provide musicbrainz data within the query basically reducing the number of internet requests per file.

Speed and accuracy should be improved in V2022.10.1-next1
Please test and let me know how it goes.
Side Note: Download MusicBrainz Metadata action has been moved to a new Web Services button next to the Tag Actions button

hello, this was much faster i need to say, only problem is, since not all files as the acustid, some files are missing
as for the speed itself it was much faster, need to do further tests anyway

To fix the issue of a file not being found with AcoustId, i added a feature in  V2022.10.1-beta1 called 'Submit to AcoustId'.
This will allow you to select a file and associate it with a MusicBrainz Recording Id that you can get from online.


[api-docs] Update Reference.md (#4810)

A new commit was made into the api-docs repo: https://github.com/discord/discord-api-docs/commit/af8066a8d22d9fad64c92dd0f6c232d82b442779
Changed so that the same filename is used throughout the example.
This is a bot created issue.

Changed so that the same filename is used throughout the example.

Can we close this?


Collapse Permissions Panel By Default

Don't show identity permissions by default (should require accordion expand to view)

This issue has been resolved


ycmd-eldoc: Use GetTypeImprecise if applicable

This is a small optimization for ycmd-eldoc.
GetTypeImprecise  does not reparse the buffer like GetType and since we parsing the buffer on many other events,  I guess it is totally fiine to use this. Ask the server first it the subcommand is supported.
I think this should also go into emacs-ycmd 1.2

Nice catch! Fixed!

Awesome, thanks!


blue exhibit main painting on display

@funnyboy044

Asset: https://drive.google.com/file/d/1U93IOK3O-aqY116xnPutc_-SaiRzJn0J/view?usp=sharing
image below is resized to 291 x 404 px

credit to Vio!


Modify MultiplayerSpawner scene path error checking.

Fixes #90903

Thanks! And congrats for your first merged Godot contribution :tada:


FALCON-1596 Spring shell based CLI for falcon

Hi All,
Have completed the code and dev testing for spring shell please review this,had a made a executable jar and gave it to few people for field testing too as so many permutation and combinations are there.
As this is a big pull request and will keep on going out of sync like earlier please help me get it through quickly and if there is some feature pending will add it as enhancement.
@ajayyadava  @sriksun @bvellanki  please have a look.
Thanks
Praveen

@prongs  Can you also do a review ?

@PraveenAdlakha This is huge! Thank you for working on it. Apart from some minor nits the patch looks good to me. However, it will be nice if you can confirm the following as I haven't been able to verify them.

There is no intended/unintended change in existing CLI functionality. It will be nice to test a couple of error cases and happy scenarios.
All commands in CLI are now also available in shell and there is no change in the behavior of the commands(except the syntax of course)

Look forward to the docs(I noticed you have already created another JIRA for that).

video link : http://showterm.io/845935d36f61f897cbf7a

I have tested the happy scenarios for shell and some users have used it to run there pipeline from fresh and its working fine.Let me know incase there are any concerns.

+1


Expose P# as a framework

We would like to have full VS support, which would dramatically increase productivity. This is a really complicated matter though, especially due to how the P# parsing currently is done in a three phase approach. There are some suggestions:
(1) Directly extend Roslyn AST
(2) Extend the P# parser to handle advanced C# expressions/statements
(3) Embed P# as a C# framework
I am more inclined to do (1) or (2) as (3) would be too verbose and maybe harder to control? Have to think more about this ...

I'll add my suggestion.
(4) Use C# to C# translation.

Provide dummy C# classes and some templates (examples) such that it is possible to specify machines in C# succinctly.
Use Roslyn to parse and visit the AST of the succinct C# and generate the actual C# code that will be executed.

This gives you several things for free:

VS support (Intellisense, errors shown in VS, etc.)
C# error detection.
Some basic error detection for mis-use of the P# API.

Example:
    using static Events;
    using static Server.States;
    using static P;

    public enum Events { Ping, Pong, Unit, Timeout };

    // "Machine" is a dummy class to mark the fact
    // that this is a machine.
    // It may have dummy methods too.
    // The real Machine class will have a different name or be in a different namespace.
    // The actual Server class will be generated by the visitor (perhaps in a different namespace again).
    class Server : Machine
    {
        // STATES
        public enum States { Playing, Waiting };

        // FIELDS
        private readonly Machine client;

        // STATE TRANSITIONS

        // Init state
        public Server()
        {
            // This will not actually be the constructor of the generated
            // Server machine, but this seems like a good way to
            // specify the Init state.

            // Use of "new" on a machine type will be replaced with code that creates the machine.
            client = new Client(this);
            Raise(Unit);
        }

        // Use of attributes is one way to specify (with Intellisense) state transitions.
        // There might be a better (cleaner) way.
        // The visitor will check the structure of the method and that every event is handled, etc.
        [State(Playing)]
        public void PlayingBody()
        {
            Send(client, Pong);

            On(Ping).GoTo(Playing);
            // Or:
            On(Ping, Playing);
            // Or (named parameter):
            On(Ping, GoTo: Playing);
        }

    }

And a variation without comments:
    using static Events;
    using static Server.States;
    using static P;

    public enum Events { Ping, Pong, Unit, Timeout };

    class Server : Machine
    {
        public enum States { Playing, Waiting };

        private readonly Machine client;

        public Server()
        {
            client = new Client(this);
            Raise(Unit);
        }

        [State(Playing)]
        public void PlayingBody()
        {
            Send(client, Pong);

            On(Ping, Playing);
            On(Pong, Waiting);
        }

        // ...

    }

The visitor would generate a class for each event specified, etc. It would generate a ServerMachine class, which would include bits of code parsed from Server. Calls like "Send" and "On" will be replaced with the respective P# API call. The visitor would have to enforce the various constraints. E.g. you must specify the events, you must handle all events at the end of a "state transition method", etc.

This is what I am experimenting with right now:
using System;
using Microsoft.PSharp;

namespace PingPong
{
    internal class Server : Machine
    {
        MachineId Client;

		[Initial]
		[OnEventGotoState(typeof(Unit), typeof(Playing))]
		class Init : MachineState
        {
            Server Fields;

            protected override void OnEntry()
            {
                (this.Machine as Server).Client = this.CreateMachine(typeof(Client), this);
                this.Raise(new Unit());
            }
        }

		[OnEventDoAction(typeof(Unit), "SendPong")]
        [OnEventDoAction(typeof(Ping), "SendPong")]
        class Playing : MachineState
        {
            protected override void OnEntry()
            {
                this.Send((this.Machine as Server).Client, new Pong());
            }
        }

        void SendPong()
        {
            this.Send(this.Client, new Pong());
        }
    }
}

I quite like the use of attributes to declare transitions and action handlers, which will then be picked up by the P# compiler and translated to the intermediate C# representation. I am not entirely happy though with the States. Currently to access a field from the machine you have to do (this.Machine as Foo).Field, which is quite verbose ... I am trying to think a clean solution for this. Btw, internally states are just nested classes to a machine, which makes sense to have them as classes even on the frontend. Any thoughts/suggestions?

Some more thoughts. This could end up being quite easy to use and not too verbose:
using System;
using Microsoft.PSharp;

namespace PingPong
{
    internal class Server : Machine
    {
        MachineId Client;

		[Initial]
        [OnEntryDoAction("InitOnEntry")]
        [OnExitDoAction("InitOnExit")]
        [OnEventGotoState(typeof(Unit), typeof(Playing))]
        [DeferEvents(typeof(Foo))]
        [IgnoreEvents(typeof(Bar), typeof(Baz))]
        class Init : MachineState { }

		void InitOnEntry()
        {
            this.Client = this.CreateMachine(typeof(Client), this);
            this.Raise(new Unit());
        }

		[OnEventDoAction(typeof(Unit), "SendPong")]
        [OnEventDoAction(typeof(Ping), "SendPong")]
        class Playing : MachineState
        {
            protected override void OnEntry()
            {
                this.Send((this.Machine as Server).Client, new Pong());
            }
        }

        void SendPong()
        {
            this.Send(this.Client, new Pong());
        }
    }
}

You can still write a state like before (which is the actual translation):
[Initial]
[OnEventGotoState(typeof(Unit), typeof(Playing))]
class Init : MachineState
{
    protected override void OnEntry()
    {
        (this.Machine as Server).Client = this.CreateMachine(typeof(Client), this);
        this.Raise(new Unit());
     }
}

But if you want less verbosity you have the above option using attributes!

Done exposing a cleaner framework for now. The static checking (to make sure someone does not write whatever he wants) needs a bit more work but will expose and merge meanwhile so we can start using this.

I have implemented some static checking too the last few days. Closing this issue for now.


Set Up Github Actions for BDD

As a developer
I need Github actions enabled
So that I can ensure that we integrate CI/CD into our dev pipeline
Details and Assumptions

Github workflow yml file will have to be created to configure github actions
Assume test cases work before doing this

Acceptance Criteria
Given Github Actions have been properly implemented
When I submit a pull request
And the PR will indicate the code coverage

Hey team! Please add your planning poker estimate with Zenhub @bartconsedine @EmiliePourchet @LotteLa @marianaruizallende @nilesh1401 @Victoria-Obasa


Remove legacy, pre-0.8.3 code

Remove checks for CAMERA=auto.
Remove checks for obsolete variable names.
Merge functionality of scripts/filename.sh into config/config.sh (almost every script that sources in config.sh also sourced in filename.sh).

Fixes #1004

@AndreasLMeg FAN_DATA_FILE is a new setting in config.sh that points to the optional file that contains the fan speed and percent.  It's set to "" in config.sh;  the "-v" checks if it's not set to anything, i.e., it doesn't exist.  If so, the user has an old config.sh file.
I learned about "-v" today.  It takes the NAME of the variable, not the variable contents, so "[ -v FAN_DATA_FILE ]" is correct.
Do you think you'll have a chance to test this PR?  I'm running it without any problems.


Use updated query hash to fetch extra info from UserFollowers

What does this PR do?
The most up to date version of the Instagram uses this endpoint to fetch the list of followers for a user. Notably, the result includes the is_private field for each follower.
Why was this PR needed?
It's a nice improvement and it follows the current Instagram API more closely.
What are the relevant issue numbers?
(List issue numbers here.)
Does this PR meet the acceptance criteria?

[x] Passes flake8 (refer to .travis.yml)
[x] Docs are buildable
[x] Branch has no merge conflicts with master
[ ] Is covered by a test

Not really sure why Travis is failing on flake8 for such a trivial change?


Fixes #74 - Use localized time range format

Use localized time format with MomentJS and trim AM/PM only if end is within 12h from start and AM/PM is in used by the localized result.

Any input on this? 😄

yup sorry, thanks for the PR, busy this weekend. ping me again on Tuesday if I don't get to it :P

@jquense ping

long time coming but i've just gone and removed the extra logic around AM/PM formats. If folks want that behavior they can customize it :) but not doing that is a better default