Support for OpenType Variable Fonts

Version 1.8 of the OpenType font format specification introduced Variable Fonts and Adobe's Source * Pro fonts using this mechanisms are already available (see the "Releases" section of the GitHub repos) and more fonts using this mechanisms may follow soon.
Using this approach it would be possible to allow arbitrary selection of font weight and (optical) font size.
Will fontspec implement mechanisms supporting OT Variable Fonts?

fontspec only provides an interface to what luaotfload and XeTeX can do. I’m out of the loop as to whether they support OT variable fonts… as soon as they have the capability I can add it to fontspec.

According to upstream in latex3/luaotfload#120, the default node shaper in luaotfload now supports variable fonts as of version 3.16, so fontspec may be able to utilize it.

According to upstream in latex3/luaotfload#120, the default node shaper in luaotfload now supports variable fonts as of version 3.16, so fontspec may be able to utilize it.

It already works under lualatex:
% !TEX program = lualatex

\documentclass{article}

\usepackage{forloop}
\usepackage{fontspec}
\setmainfont{Hundred.ttf} %https://github.com/edward-martyr/Hundred-ASCII/blob/master/Hundred.ttf

\begin{document}

\newcounter{themenumber}
\forloop{themenumber}{20}{\value{themenumber} < 101}{{\addfontfeatures{RawFeature={+axis={wght=\arabic{themenumber}}}}a}}

\end{document}

Result:

There seems to be an issue in picking the correct default values for the font axes:
for instance
\documentclass[a4paper,12pt]{article}
\usepackage{fontspec}
\setmainfont{Source Serif 4 Variable}
\begin{document}
The quick brown fox jumps over the lazy dog.
\end{document}

fails, unless you add a
\defaultfontfeatures{RawFeature={+axis={wght=100}}}

before the \setmainfont.
However, if I am correct, the font fvar table should list a default value for every axis. Another thing that is unclear to me is the following. The font that I have used as an example above has two axes: weight and optical size. The need to explicitly specify a font feature seems to exist only for the weight. Why?


[iter 4]  finished frontend view

fished touches on the frontend for import user view and import project view

shipped


RAW(16) does not show up properly in oracle 11g

System information:

Operating system (distribution) and version
DBeaver version
Additional extensions

Connection specification:

Database name and version
Driver name
Do you use tunnels or proxies (SSH, SOCKS, etc)?

Describe the problem you're observing:
Raw(16) datatype does not show up properly in dbeaver 6.1
Steps to reproduce, if exist:
create table dropme (a int,b raw(32) default sys_guid(), c raw(16) default sys_guid());
insert into dropme(a) values(1);
select * from dropme;
Include any warning/errors/backtraces from the logs

DBeaver supports different binaries representation.
If you need to see it as hex you can switch presentation in preferences:


return terms parameter type mismatch

In the .return docs you say this is valid:
query.return({
  people: {
    name: 'personName',
    age: 'personAge',
  },
})

But Term doesn't allow a Dictionary of Dictionary's. people has to be an array of objects.
https://github.com/jamesfer/cypher-query-builder/blob/master/src/clauses/term-list-clause.ts#L14

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Rework PR #302 in a cleaner way

Replaces https://github.com/DISTRHO/DPF/pull/302

Patch implementation moved to pugl.mm alongside other platform-specific code
Renamed function to follow existing convention
Avoid unnecessary new file and changes in makefiles

This update will become obsolete but safe to keep once DPF provides application bundles for the standalone JACK target.

That looks nice and clean, thanks a lot!


Prepare links for weekend (day 11 and 12)

Prepare links for solutions contributed by the community over the weekend

not sure it wont fail when deploying the website, since the pages are not created yet :s

add empty pages maybe, like in this commit https://github.com/scalacenter/scala-advent-of-code/pull/32/files

not sure it wont fail when deploying the website, since the pages are not created yet :s

Here I see that all pages are there. Is this what you are referring to?

not sure it wont fail when deploying the website, since the pages are not created yet :s

Here I see that all pages are there. Is this what you are referring to?

I didn't notice! thanks


Does it support Linux as a service?

I can use Texpresso on my Ubuntu server.  Now I wanna know whether I can use my Ubuntu server as a service and allow VsCode on Windows to use this service. I am confused about configuring the Path to the Texpresso binary when I want to access the binary file on the Ubuntu server.

I'm not familiar with this setup. You need to put whatever path you need to enter in the Windows command line in order to start texpresso into the "path to texpresso" setting so the extension can call it.


[question] absolute noob starting yazi can't handle compress

I'm incapable to make compress part running, here is what I did so far:

select file to compress in yazi (v)
hit C
hit w to see what happens

There is Run micro plugin 'ouch' shown and 2 left in the status line shown.
When I hit q after a while I'm asked if ouch process ought to be closed.

Are your files large? Have you tried with small files? Instead of v, have you tried space for selection?

Yes, with a javascript file and space then C I see the same effect.

Reinstalled, it works now.


WC.org Site Audit

Type
UX discovery
Objectives
To document current WC.org and WL.org for the navigation user research and project planning.
Tasks

[ ] Audit of current site and identify which pages need to be revisited or merged
[ ] Review with the team
[ ] Create the materials for the card sorting exercise

DrupalContent27102017.xlsx

@jamesgorrie was looking at 404s showing up in GA for .aspx pages (from Drupal's predecessor?). Noting here in case looking at / sorting out these 404s fits with the site audit work


Feature/react refetch

Codecov Report

Merging #359 into master will increase coverage by 0.14%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master    #359      +/-   ##
=========================================
+ Coverage     3.2%   3.34%   +0.14%
=========================================
  Files          52      49       -3
  Lines        1093    1045      -48
  Branches      182     203      +21
=========================================
  Hits           35      35
+ Misses       1057    1010      -47
+ Partials        1       0       -1

Impacted Files
Coverage Δ

...problems-frontend/src/pages/ListPage/ListTable.tsx
0% <ø> (ø)
:arrow_up:

...ms-frontend/src/pages/ListPage/DifficultyTable.tsx
0% <ø> (ø)
:arrow_up:

...lems-frontend/src/pages/TablePage/ContestTable.tsx
0% <ø> (ø)
:arrow_up:

...problems-frontend/src/components/NavigationBar.tsx
0% <ø> (ø)
:arrow_up:

atcoder-problems-frontend/src/utils/Api.tsx
0% <ø> (ø)
:arrow_up:

...ems-frontend/src/pages/UserPage/SubmissionList.tsx
0% <ø> (ø)
:arrow_up:

...ontend/src/pages/TablePage/AtCoderRegularTable.tsx
0% <ø> (ø)
:arrow_up:

...s-frontend/src/pages/UserPage/FilteringHeatmap.tsx
31.57% <ø> (ø)
:arrow_up:

...oder-problems-frontend/src/pages/StreakRanking.tsx
0% <0%> (ø)
:arrow_up:

...coder-problems-frontend/src/pages/ShortRanking.tsx
0% <0%> (ø)
:arrow_up:

... and 14 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 1183929...aef1d19. Read the comment docs.


How can i remove all options from select?

I use setOptions() method without any parameter but it's not rendering. When i scroll into option list, it's rendering.
I tried use reset() setOptions() renderOptions() methods at the same time, like this

But after 2 calls it throws error

I think there should be a clearOptions() method and it remove all options from select in this way select value is empty.

You should call set options method with DOM instance as below.
https://sa-si-dev.github.io/virtual-select/#/methods?id=setoptions

But it's not rendering options. When I scroll in the options div, it's render select.
https://user-images.githubusercontent.com/69236857/185776411-79ea5318-986a-4fcb-ab53-175f18c765a0.mp4

May I know, how do you call setOptions method?
What is fleetBranchSelect here?

Sorry. I'm wrong here


Codeunit 90 modify event on procedure PostItemTrackingLine

Hi,
We need a modify event on Codeunit 90 procedure PostItemTrackingLine :
.....
// > Modify Event
OnBeforeItemJnlPostLine(ItemJnlLine, PurchLine, PurchHeader, SuppressCommit, IsHandled, WhseRcptHeader, WhseShptHeader, TempItemChargeAssgntPurch, TempWhseRcptHeader, PurchInvHeader, PurchCrMemoHeader);
// < Modify Event
            if not IsHandled then
                if PurchLine."Job No." <> '' then begin
                    PostJobConsumptionBeforePurch := IsPurchaseReturn;
                    if PostJobConsumptionBeforePurch then
                        PostItemJnlLineJobConsumption(
                          PurchHeader, PurchLine, OriginalItemJnlLine, TempReservationEntry, QtyToBeInvoiced, QtyToBeReceived,
                          TempHandlingSpecification, 0);
                end;

            OnPostItemJnlLineOnBeforeItemJnlPostLineRunWithCheck(ItemJnlLine, PurchLine, DropShipOrder, PurchHeader, WhseReceive, QtyToBeReceived, QtyToBeReceivedBase, QtyToBeInvoiced, QtyToBeInvoicedBase);

...
[IntegrationEvent(false, false)]
    local procedure OnBeforeItemJnlPostLine(var ItemJournalLine: Record "Item Journal Line"; PurchaseLine: Record "Purchase Line"; PurchaseHeader: Record "Purchase Header"; CommitIsSupressed: Boolean; var IsHandled: Boolean; WhseReceiptHeader: Record "Warehouse Receipt Header"; WhseShipmentHeader: Record "Warehouse Shipment Header"; TempItemChargeAssgntPurch: Record "Item Charge Assignment (Purch)" temporary; TempWhseRcptHeader: Record "Warehouse Receipt Header" temporary; PurchInvHeader: Record "Purch. Inv. Header"; PurchCrMemoHeader: Record "Purch. Cr. Memo Hdr.")
    begin
    end;

..

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Add nodes persistence

scalastyle

You're right @mcsherrylabs. With peer discovery in place we no longer need tryDiscardPeersToFreeUpLimit

@mcsherrylabs Actually on second thoughts I'm not sure if we should remove it. Without this logic other peers may have problems connecting to us. Say we have a limit of 10 peers. 5 handshaked and 5 slots continuously taken by nodes we keep trying to connect to. Without tryDiscardPeersToFreeUpLimit we will just reject all incoming connections (as peer limit is reached). With tryDiscardPeersToFreeUpLimit however, we'll try to disconnect one of the nodes that we keep trying to connect to.
Another solution could be to allow to have more than maxPeers connections for a short period and then, from time to time remove some peers so that they fit the limit.

but then retrying connections can succeed and we end up with more connected peers than the limit

other than my last comment LGTM

change LGTM


Review unit and integration AtomDB tests

Links to EPIC https://github.com/singnet/das/issues/97

@eddiebrissow @andre-senna
https://github.com/singnet/das-atom-db/pull/212#pullrequestreview-2293192886 <-- check this comment.
Those cases must be investigated and fixed when possible. I left some TODO comments to highlight them.
Let's keep this ticket open to remind us.


Authorised example in README is wrong

It should be like this (notice the "Token " string before the token).
$api = Flarum('http://example.com', ['token' => 'Token <insert-master-token>; userId=1']);

Hi, thanks for the report !
The README is indeed incorrect for all versions released, but it isn't worth fixing anymore given the future release (dev-master right now) automatically adds the token prefix:
https://github.com/flagrow/flarum-api-client/blob/98bb9ba3b90f1ccbb1649c0ef1beaf9bbff954ae/src/Flarum.php#L107
Not sure when we'll work on this package again though... Let's keep this issue open in the meantime in case somebody wonders the same thing.

Have you abandoned the package?

@pryley we will publish any security update necessary, but as you can guess from the commit history it's been a while since we've worked on new features for the package.
If beta8 requires us to make changes to the package we will probably release an update.


generate coverage reports

generates coverage reports and uses coveralls.io to track
coverage.

Changes Unknown when pulling 8e9d768103e8930c2b4509d4766d01d14c1c2587 on coverage into ** on master**.


fix image ID tick

image ID had an extra ` . removed. added param name to it.

/lgtm

Rebased

/label tide/merge-method-squash


Switching Sublevel map does not work

Hi Sir,
I have tried to switch the sublevel map (different layers) through shortcut V / B from Carla client, but it does not work.
Town03_Opt and Town01_Opt maps are tested.
CARLA version - 0.9.11 - download from https://carla-releases.s3.eu-west-3.amazonaws.com/Linux/CARLA_0.9.11.tar.gz
By the way, I have also tested this feature from Editor compiled from source,  it does not work either.
Do I need any special configuration or option/argument for launching this feature?
Thank you!
CARLA version - 0.9.11
OS - Ubuntu 16.04.
Python - 3.7

Hi @jimmyqwy,
You can switch between sublevel map with the shortcut "v". When you decide which sublevel you want to Unloaded, press "Shift+b" and b to reload.
When you run the script manual_control.py if you press "h" there is useful info about it.

Hi @jimmyqwy,
You can switch between sublevel map with the shortcut "v". When you decide which sublevel you want to Unloaded, press "Shift+b" and b to reload.
When you run the script manual_control.py if you press "h" there is useful info about it.

Thank you @XGodina , it works!
Just find some names of sub level do not 100 percent match the list levels from client, but it should be fine.
Thanks again.

Thank you @XGodina , it works!
Just find some names of sub level do not 100 percent match the list levels from client, but it should be fine.
Thanks again.

Hello I'm trying to save images applying the sublevel option to the worlds. However, I do not see the expected changes. I have detailed this here. Any help would be very highly appreciated.
https://github.com/carla-simulator/carla/issues/3517#issuecomment-873606472.
I'm running the server in synchronous mode though.


Idea: Shaders

I've seen some interesting usages of shaders in many games. And I think that having a shaders support inside the game engine itself is good.
In my head shaders can be used in many ways to improve the game graphics.
They should be optional, so the game can run without any shader, it's better for my toaster computer

Custom shaders are good indeed, but this will require to refactor the renderer. Not sure when I'll have time for this.

Custom shaders are good indeed, but this will require to refactor the renderer. Not sure when I'll have time for this.

Yes, I thought of it. I also thought about the idea of switching OpenGL to Vulkan.
I wanted to make a game engine using the knowledge of other game engine by view their source code. Like Godot and this. This is the link to my post
https://www.reddit.com/r/rust_gamedev/comments/nbaqqp/a_game_engine/?utm_source=share&utm_medium=web2x&context=3

Closing this in favor of #134 since it adds more wide support for custom rendering.


[SPARK-34868][SQL] Support divide an year-month interval by a numeric

What changes were proposed in this pull request?

Add new expression DivideYMInterval which multiplies a YearMonthIntervalType expression by a NumericType expression including ByteType, ShortType, IntegerType, LongType, FloatType, DoubleType, DecimalType.
Extend binary arithmetic rules to support year-month interval / numeric.

Why are the changes needed?
To conform the ANSI SQL standard which requires such operation over year-month intervals:

Does this PR introduce any user-facing change?
No
How was this patch tested?
By running new tests:
$ build/sbt "test:testOnly *IntervalExpressionsSuite"
$ build/sbt "test:testOnly *ColumnExpressionSuite"

Test build #136530 has started for PR 31961 at commit 92deb16.

Perhaps a bit broader question. When adding new expressiones, should we keep in mind places like PushFoldableIntoBranches#supportedBinaryExpression? Or should they be handled separately?

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41114/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41114/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/41114/

Or should they be handled separately?

@tanelk Separately. There is the JIRA https://issues.apache.org/jira/browse/SPARK-34841 for that. Need to add new expressions + tests.

@yaooqinn @cloud-fan Could you review this PR, please.

Test build #136530 has finished for PR 31961 at commit 92deb16.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/136530/

thanks, merging to master!

late lgtm.


Adding retries for vault-init to come online

In some instances vault-temp can take a moment or two to come online.

ci check this


Remove startup command

Now CustomTrayMenu is already supported run multiple command at once,
startup command is seems redundant.
Maybe add new command alias to run certain custom tray menu from command interface?
alias RunStartupAgain -> get "RunStartupAgain" from CustomTrayMenu ->  run a.exe\nrun b.exe\nrun c.exe

Just delete startup. I'm sure nobody uses command interface. If one really do script, look at stdin interface.


Chrome plugin inserts an additional blank line at the top of Gmail

When using the current Chrome plugin (2.11.3) toggling Mardown mode in a Gmail will insert a space at the top of the email. It effectively looks like an additional line of whitespace, but this line is not removable.
Here is an example, before:

After:

This is on Chrome 40.0.2214.94 on Mac OS X (10.10.2).
Let me know if any more information is needed.

I have noticed this, but I guess I stopped seeing it after a while. I'll fix it in the next release.
You can fix it yourself by adding this to your "Primary Styling CSS" in the MDH Options, below the existing p rule (so, it's fine if you just add it to the bottom):
.markdown-here-wrapper:first-of-type > p:first-of-type {
  margin-top: 0 !important;
}

(I adapted that from the Salutation Styling on our Hints and Tips wiki page.)
Does that fix it for you?

That fix does work, thanks!
Is that something that can be fixed for everyone in future versions of the plugin?

Yes. I think it's an improvement, so I'll add it to the default CSS in the next release. (Unfortunately, people will have to reset their CSS to get the change. Blah. Dumb design. #48, #78.)

In my fix I made an even simpler change -- I just got rid of the top margin altogether. The <p> rule is now:
p {
  margin: 0 0 1.2em 0 !important;
}

(Also note that in the CSS I suggested earlier, the .markdown-here-wrapper:first-of-type is meaningless -- :first-of-type can't be used with classes.)


LICENSE currently refers to no one

https://github.com/Synergile/Synergile/blob/13cded3531007a7680972d20937c7ba6e801cf90/LICENSE#L3
The project is currently licensed under CodeWritten's old username, meaning it isn't licensed to any of us currently. Under the terms of the MIT License we are able to update it. My suggestion is that we update the copyright holder to be the Synergile organization.

I agree

I'm gonna backtrack a little bit, due to how legal enforcement work. In order for the license to have the proper legal effect, and in this case the legal effect is that people other than us are allowed to use any portion of our software and we are not liable for uses of our software by people other than us, we need to have some legally binding names. We may want to consider incorporating the Synergile name (I think one of those suggested something along those lines), or listing our proper names in the license. I normally license my own projects under my real name because of all this, for context.

If adding our names is what it takes, then alrighty! @wakfi @NapkinsXYZ @jedi3r4, in the order I pinged you guys please commit the appropriate changes to LICENSE.md to add your proper names to said file. this issue is to remain OPEN until the changes have been made. I will go first


Combine audits

There is now a summary for all audits, it lists the run audit, total issues for each audit and the file path of the detailed report.

After testing the new code with the resolution bugs I think we are fine now to merge


aws-c-common: 0.9.21 -> 0.9.23

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/aws-c-common/versions.
meta.description for aws-c-common is: AWS SDK for C common core
meta.homepage for aws-c-common is: https://github.com/awslabs/aws-c-common
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.9.23 in filename of file in /nix/store/nrqngb6ji51591jcyz0fwqvfz3f0q8l9-aws-c-common-0.9.23

Rebuild report (if merged into master) (click to expand)

458 total rebuild path(s)

458 package rebuild(s)

First fifty rebuilds by attrpath

apacheHttpdPackages.mod_tile
apacheHttpdPackages_2_4.mod_tile
appvm
arrow-cpp
arrow-glib
attic-client
attic-server
aws-c-auth
aws-c-cal
aws-c-common
aws-c-compression
aws-c-event-stream
aws-c-http
aws-c-io
aws-c-mqtt
aws-c-s3
aws-c-sdkutils
aws-checksums
aws-crt-cpp
aws-sdk-cpp
browsr
bundix
cabal2nix
cached-nix-shell
cachix
ceph
ceph-client
ceph-csi
ceph-dev
chatgpt-retrieval-plugin
cloudcompare
colmena
common-updater-scripts
crate2nix
crystal2nix
databricks-sql-cli
datatrove
dbx
dep2nix
devenv
disko
dub-to-nix
dydisnix
entwine
fusionInventory
gdal
gmt
gplates
grass

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/nrqngb6ji51591jcyz0fwqvfz3f0q8l9-aws-c-common-0.9.23 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A aws-c-common https://github.com/r-ryantm/nixpkgs/archive/dfe570a697931de7c4f3b692531a581ea7d4ce44.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/dfe570a697931de7c4f3b692531a581ea7d4ce44#aws-c-common

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/nrqngb6ji51591jcyz0fwqvfz3f0q8l9-aws-c-common-0.9.23
ls -la /nix/store/nrqngb6ji51591jcyz0fwqvfz3f0q8l9-aws-c-common-0.9.23/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review took longer than 120m and timed out

Maintainer pings
cc @orivej @edolstra @r-burns for testing.

@ofborg build arrow-cpp lix nix


Virtual scroll not working in ion-slides

Short description of the problem:
Without the slider, the list behaves normally, but with the ion-slides, the content is not displayed correctly.
What behavior are you expecting?
Data shown correctly
Steps to reproduce:

Make slider
Make virtual list in slider

    <ion-slides #contentSlider (ionWillChange)="contentListChanged()">
      <ion-slide *ngFor="let contentTab of contentOfNavigationTabs">
        <ion-list [virtualScroll]="contentTab">
          <ion-item *virtualItem="let item">
            {{item}}
          </ion-item>
        </ion-list>
      </ion-slide>
    </ion-slides>

Which Ionic Version? 2-beta0.0.9
Plunker that shows an example of your issue
http://plnkr.co/edit/BgRiHgBEPXLK3G9LRkq2?p=preview
Run ionic info from terminal/cmd prompt:
Cordova CLI: 5.4.1
Gulp version:  CLI version 3.9.0
Gulp local:   Local version 3.9.1
Ionic Framework Version: 2.0.0-beta.9
Ionic CLI Version: 2.0.0-beta.25
Ionic App Lib Version: 2.0.0-beta.15
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Mac OS X El Capitan
Node Version: v5.3.0
Xcode version: Xcode 7.3.1 Build version 7D1014

@jgw96 Any updates on this? It is really important for us that this works..

Hello @elineopsommer i dont have any updates on this at the moment, but i will get back to you as soon as i do. Thanks for using Ionic!

@jgw96 It's been a while since this issue is posted, do you have an update on this?

I'm having this problem too :( , looking forward for update @jgw96

At this time we virtual scroll is not designed to work within ion-slides. I would recommend using a normal ngFor, and to reduce the amount of data trying to be fit within each slide.

Hi, I'm having this problem too, is any solutions now?


Fix Concatenating Strings with Plus Operator tests

According to @SaintPeter's comment, the tests for "Concatenating Strings with Plus Operator" have been updated to check:

If the + operator was used to concatenate the two strings, regardless of quotes used (double vs single quotes)
If the string has been assigned to myStr variable, regardless whether it was assigned on the same line as the declaration (e.g. var myStr = ...) or on a separate line (e.g. var myStr; myStr = ...)

Closes #6028

Thanks, but we already have PR #6031 open to fix this issue!

@ltegman I just commented on PR #6031. It contains one inaccurate test.


Canonical URLs not working

Issue Summary
It seems that setting the canonical url in the meta tags of a post don't change the  in the rendered page later on. (The post's canonical URL from the own domain stays there)
I'm using a self-hosted version of ghost.
EDIT: If this helps: I checked in the database, and the right value is definitely there. It just doesn't seem to be taken into account on the time of rendering the HTML.
To Reproduce

Create post
Change canonical URL in meta tags
Publish post
In my case, the canonical URL is still set to my own domain.

Any other info e.g. Why do you consider this to be a bug? What did you expect to happen instead?
Technical details:

Ghost Version: Ghost-CLI version: 1.10.1, Ghost version: 2.23.1
Node Version: v8.10.0
Browser/OS: Mac, Chrome 74
Database: MySQL

Can you check that your theme is using the v2 API? Canonical URLs are only available if you're using the v2 API

Oooh that was it, thanks! I used a modified version of the Casper theme (to comply with GDPR), but that version apparently didn't use the current API.
Sorry for the bother, it works now 😄


ID Claim

Mod ID: 099
Mod Name: Message Of The Day
Mod Author: Glosb
MOTD / server welcome message mod!
Automatically posts a message in chat whenever a person joins your lobby. Special escape codes can be inserted in the message to be replaced with personalized values - currently (the joined person's) username, overall rank, ping and a count of how many times they have been in your lobby before. Feel free to leave suggestions/ideas what other values to add!
Other features:

10 save slots, so you can switch to a different welcome message for e.g. different play style modpacks or just have some messages to paste in chat with the manual "post message" button.
The automatic message feature can be turned off by disabling the mod in the BPMM mod menu.

Added!


Support for changing fonts per syntax group

Implements part of #126.
@franko I'm not sure whether you want to include this in a 1.16 patch or 1.17 altogether, so I didn't include a changelog entry yet.

What about the function DocView:get_col_x_offset and DocView:get_x_offset_col ?
The cursor positioning on mouse click will not be all wrong is the document's font and the style's fond have different text widths ?

Yeah, I noticed that as well when I was trying out variable-width fonts. I'm afraid that reading tokens on the line will slow down the editor considerably when selecting anything.

Yeah, I noticed that as well when I was trying out variable-width fonts. I'm afraid that reading tokens on the line will slow down the editor considerably when selecting anything.

Well, this problem is why I dismissed the proposition in the first place.

@franko I implemented font changing support in get_col_x_offset and get_x_offset_col. As far as I understand, the latter shouldn't really have any noticable performance impact, as the highlighter's each_token actually returns the tokens it already processed before. I couldn't notice any high performance impact with get_col_x_offset when selecting a lot of lines, but please do some extra testing on your machine as my CPU is maybe too fast to make a noticable difference.

I can rename fonts to syntax_fonts if you prefer that, it does sound more clear.
As for the height thing, I tested with the alternative font being smaller than the main font, and it works fine: the alt font is centered vertically on the line. It does look a bit janky with the alt font being the bigger one, though:

I don't think this is a common enough occurence to warrant an edge case. I can however think of one case where that would be nice to have, namely we could let the user make markdown headers larger if they want to, but currently there is no easy way to apply these alternative fonts to only specific syntaxes.

Well for the syntax.fonts you may leave it as it is if you want.
I will merge this, may be you can just add a changelog entry.

@franko I wanted to, but I'm not sure which version to put it under. I suppose 1.16.8 or 1.17?

It should be 1.16.9 because I need 1.16.8 to hotfix some urgent problems and I cannot yet move to 1.17 as long as I don't fix the plugin versioning stuff.

I added a changelog entry, should be ready to merge (except that changelog conflict which is easy to fix).

I added a changelog entry, should be ready to merge (except that changelog conflict which is easy to fix).

Great, thank you. I cannot merge into the master now because I am preparing the 1.16.8 release. If you want we can merge into the "dev" branch I just re-positioned into the current master.

Alright, I think this should work. I'm not familiar with changing PR branches so fingers crossed this doesn't break anything :crossed_fingers:

Great work!


Handle complex redirect URIs on Python 3

Resolves #2653.
This is one of those annoying changes that's almost impossible to test because of the sheer complexity of our redirect handling code. This also doesn't make it any simpler, sadly.
As to what version we merge this into, I proposed it against the master branch. I don't think it belongs in 3.0.0 (resolve_redirects isn't really part of our public API), but it could definitely break people's stuff. Next minor release feels appropriate, but I'd like to hear what you think @sigmavirus24.

The tests fail miserably on this :(
One way to fix the failing tests is to either change unquote_unreserved to call split with a b'%', or to immediately decode the URL to utf-8, e.g.,
if is_py3 and isinstance(url, str):
    url = url.encode('latin1').decode('utf8')

Personally, I'd like those to be on separate lines to make it easier to find errors in the future, but that's just me =P.
Personally, I'm a bit ... skiddish to include this in 2.8. I think 3.0 is the safer option because we don't regularly test against internationalized domains. Then again, I have to wonder how many people are using requests on Python 3 with redirects that behave like this since this is the first report we've had like this in the 2+ years that we've had this code in place. Perhaps that makes it safe, but I'm not a fan of that logic. =/

Wouldn't it be better to file a bug report on Python 3? I always thought that Python 3 had some kind of "UTF-8 everywhere where possible" policy, so I wonder why they decode with ISO-8859-1.

Wouldn't it be better to file a bug report on Python 3? I always thought that Python 3 had some kind of "UTF-8 everywhere where possible" policy, so I wonder why they decode with ISO-8859-1.

No that's not the policy. Even so, it doesn't make sense in the context of HTTP/1.1 where headers were (until the latest revision of the specification late last year) always supposed to be Latin-1 (a.k.a, ISO-8859-1). http.client is doing the right thing on Python 3 by returning text as was described in the original version of the HTTP/1.1 specification

Yeah, this definitely isn't wrong for Python 3, it's trying to do the best it can, it just happens to be inconsistent. I'll think some more, I think I can restructure the code to get it to work properly.

Ok, I've fixed the broken tests. I agree with you though @sigmavirus24, I think this is scary enough to want to go into 3.0.0, so I'm closing this PR and opening a new one with the correct target.

Please see #2754 for the new version of this change.


update predicates on startup, and on demand via API endpoint

In #141, we put the update of /data/predicates.json and /data/smartapi_specs.json on a cron job that runs every 10 minutes (see updateSmartAPISpecs in https://github.com/biothings/BioThings_Explorer_TRAPI/blob/master/src/controllers/cron/update_local_smartapi.js#L138).  But the cron job will restart the server and interrupt any currently-running jobs, which will increasingly become an issue with long-running jobs.  A possibly better solution would be to run updateSmartAPISpecs on startup, create an API endpoint to trigger updateSmartAPISpecs, and reduce the cron frequency to a daily update.

After some investigation, any time one of the v1 endpoints are hit, a new knowledge graph is generated. For example, for the /v1/meta_knowledge_graph route, it calls the getKG function, which creates a new knowledge graph (See https://github.com/biothings/BioThings_Explorer_TRAPI/blob/main/src/controllers/meta_knowledge_graph.js#L18-L20 and https://github.com/biothings/BioThings_Explorer_TRAPI/blob/main/src/controllers/meta_knowledge_graph.js#L61)
(The other paths seem the same. See https://github.com/biothings/BioThings_Explorer_TRAPI/blob/main/src/controllers/predicates.js#L54, https://github.com/biothings/bte_trapi_query_graph_handler/blob/main/src/index.js#L95)
Because of this, it seems unnecessary to restart the server any time predicates.json or smartapi_specs.json is updated. The problem of the cron job restarting the server and interrupting jobs can be solved by replacing the npm start script with node src/server.js (or nodemon --ignore './data/' src/server.js as Andrew suggested if nodemon is still needed).

Are there any problems that could be caused by not restarting the server after updating these files?
Could any problems be caused by updating the specs while a query is running?
Are there any other parts of the code that

Insights from someone who knows the repo better appreciated!

@ericz1803 on production bte server, we run node src/server.js directly, so no server restart when two smartapi data files are updated. Here in package.json file, it will be just up to how people use npm start in their development flow. I typically just use npm run debug on my local which will auto-restart with debug logs turned on.

And I agree with @andrewsu, a daily refresh should be good enough.


Added cog to handle logging and updated intents in bot.py

See title.
We need to check if we have the all the needed privileged intents enabled (see: https://discordpy.readthedocs.io/en/stable/intents.html). For intents, I added messages, moderation, and members, while keeping message_content intents.
In logging.py, I also added boilerplate for each logging listener that we will flesh out later.
Specifically, note in purging messages, I added on_raw_bulk_message_delete() instead of on_bulk_message_delete(). This is to ensure the failsafe, in case any messages are deleted that do not have a cache. See these links for more details about the difference between them:
https://discordpy.readthedocs.io/en/stable/api.html?highlight=bulk#discord.on_bulk_message_delete
https://discordpy.readthedocs.io/en/stable/api.html?highlight=bulk_message_delete#discord.on_raw_bulk_message_delete
If the implementer of the purging listener feels it would be more fitting to use on_bulk_message_delete(), then feel free to discuss further!

For purging, from a quick glance it looks like while the raw event is better, if the message isn't in the bot's cache, it seems we might not be able to access its content, so the usefulness of the log will be limited. Just something to keep in mind in terms of bot reliability.
https://discordpy.readthedocs.io/en/stable/api.html?highlight=bulk_message_delete#discord.RawBulkMessageDeleteEvent

LGTM! I'll check on the intent stuff, and update any necessary intents.


Function / function change to exclude occurrences beyond the extent of a grid

7: https://docs.gbif-uat.org/b-cubed/1.0/en/#spatial

Occurrences that are located beyond the extent of the chosen reference grid MUST NOT be included.

MRGS and EEA grids are not worldwide.

This was implemented in dcd5a30be3ef5a021015e69541e20a509b000562


Exception handling

ExceptionHandler should redacts auth token headers. #2815
Add XH.handleException 'showWithToast' option, to alert user w/toast instead of modal. #2772
Add support for showing action buttons within toasts.

Hoist P/R Checklist
Pull request authors: Review and check off the below. Items that do not apply can also be
checked off to indicate they have been considered. If unclear if a step is relevant, please leave
unchecked and note in comments.

[x] Caught up with develop branch as of last change.
[x] Added CHANGELOG entry, or determined not required.
[x] Reviewed for breaking changes, added breaking-change label + CHANGELOG if so.
[x] Updated doc comments / prop-types, or determined not required.
[x] Reviewed and tested on Mobile, or determined not required.
[x] Created Toolbox branch / PR, or determined not required.

If your change is still a WIP, please use the "Create draft pull request" option in the split
button below to indicate it is not ready yet for a final review.

Pull request reviewers: when merging this P/R, please consider using a squash commit to
collapse multiple intermediate commits into a single commit representing the overall feature
change. This helps keep the commit log clean and easy to scan across releases. PRs containing a
single commit should be rebased when possible.

Thanks for taking a look Lee!
I've updated the branch with your feedback, including:

New ExceptionHandlerDefaults to set app-wide defaults
Open the exception dialog by clicking on button within toast
showWithToast > alertType

Thanks for getting this done so fast -- looks great -- just a matter of the "best" way to specify the configuration class now -- should it be statics on the main class?    Properties on an anonymous object?

Thanks Lee.
I took another pass at it, and brought the static options onto ExceptionHandler. I ended up moving the stringify method to ExceptionHandler as well, to resolve the circular dependency while still keeping it available for application code.
Also styled and titled the toasts a bit, think they look a bit more structured now:

Looks fantastic!
On Tue, Jan 18, 2022, 7:20 AM Tom Tirapani @.***> wrote:

Thanks Lee.
I took another pass at it, and brought the static options onto
ExceptionHandler. I ended up moving the stringify method to
ExceptionHandler as well, to resolve the circular dependency while still
keeping it available for application code.
Also styled and titled the toasts a bit, think they look a bit more
structured now:
[image: Screenshot 2022-01-18 at 11 47 39]
https://user-images.githubusercontent.com/3017757/149936312-c30b2c78-478e-42f7-8059-d6647cf57810.png
—
Reply to this email directly, view it on GitHub
https://github.com/xh/hoist-react/pull/2865#issuecomment-1015357676, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AARTL2LVZRG2EAXQDQWPXJTUWVLHVANCNFSM5L6WHVDQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>

That does look great - eager to start using it. 👍
Thoughts on going with more user-friendly "Error" vs. "Exception" ?

Yes,  Let's move to 'Error' for consistency.  Happy to merge this.

Yes,  Let's move to 'Error' for consistency.  Happy to merge this.

That showAsError flag has always struck me as a bit confusing, but perhaps it will help to clarify if setting that flag to false has an obvious effect on the toast intent and title.
I would suggest that when false we default to no title and either the warning (orange) or primary (blue) intent. Leaning towards the latter, but defer to you guys re. what you think the most useful default would be.

All makes sense to me - just pushed changes.
I noticed we already have the title / message logic in parseOptions, so I changed it to use the parsed options rather than the exception (keeping the dialog and toasts consistent): https://github.com/xh/hoist-react/blob/develop/core/ExceptionHandler.js#L190-L191
Also picks intent (danger / primary) based on showAsError


Display results

With an obtained analytical representation of the surface (implementation on this branch) it should be possible to implement some further processing steps. Results to be obtained from the surface representation are:

[ ] anisotropic stress
[ ] geodesic distances between minima/maxima of 2nd derivate (= locations of cell junctions)
[ ] degree and order of surface. This shows what contributions of the spherical harmonics base contribute most strongly to the obtained surface.

Duplicate of #27


Upload Crash

I tried to use ionic view but the upload crashed after a view minutes.
$ ionic upload
Uploading app...
Error malformed response: SyntaxError: Unexpected token <
Response: Not Found The requested URL /api/v1/app/upload/***** was n (CLI v1.3.11)
Your system information:
OS: Mac OS X Yosemite
Node Version: v0.10.33
Cordova CLI: 4.1.2
Ionic CLI Version: 1.3.11
Xcode version: Xcode 6.1.1 Build version 6A2008a
ios-sim version: 3.0.0

@nativeapps I haven't seen this error yet.
Does the app run just fine in the simulator/device?

Yes. It runs on all variations (android, iOS, device and simulator) without any problems.
I created another app from tabs template and it works fine.
Maybe there is a bug when a project was updated to current ionic version.

Can you post the contents of your ionic.project file?

I was able to fix this issue by clearing my app_id field in ionic.project file and then uploading. The app_id got filled in by a value from the service I think.

All working now?

For me it is. Not sure about OP.

@nativeapps - can you confirm you dont have an app_id in your ionic.project file? If you do, can you remove and try again?

@jbavari If you check your server logs for 404's hitting https://apps.ionic.io/api/v1/app/upload/[some id that wasn't generated by ionic] you'll probably see other users w/ this issue.

Yep, it's an on-going issue.
Many users open this file, then plop in an ID for app_id, then have issues with upload.

I ran into this issue today, but my app_id was already empty.
I downgraded node to v0.10.33 and the upload went through successfully (v0.12.0 threw the Error malformed response)

I had a similar problem. I repeated about two times, and on the third time the upload was sucessful without any error output.

I'm seeing this issue
OS: Mac OS X Yosemite
Node Version: v0.12.2
Cordova CLI: 4.3.0
Ionic CLI Version: 1.3.20
Xcode version: Xcode 6.3.1 Build version 6D1002
ios-sim version: 3.0.0
ios-deploy version: 1.5.0

Some notes:

Better error message would be good, ran into this, too.
We had a prefilled app_id but haven't used Ionic View previously, so it was filled with something a developer thought would be appropriate.
Not having the key app_id in the ionic.project at all results in an error (because the app_id will result in undefined as string)

Resolution: clear your app_id in your ionic.project but leave an empty string for it!

My app ID was filled in for whatever reason as well. Setting it to an empty string did the trick. Thanks @rainboxx for noting that. A better error message would be really nice.

Setting the app_id to an empty string did the trick for me as well. Thanks a lot!

Simply setting the app_id to empty string did not work for me. I needed to do the following steps to get it working:

set app_id to empty string
updated node to 4.2.1
updated cordova cli to 5.3.3
updated ionic cli 1.7.6
updated ionic framework with bower to 1.1.0
deleted files in ~/.ionic/

I had the same issue: 404 on upload (though I had to hack upload.js to get this message) - BUG plus bad error handling?
I certainly did not fill in the app_id in ionic.project manually but clearing it allowed me to upload.
Is it normal that this URL returns 404:
https://apps.ionic.io/api/v1/app/
How can I see a list of apps I uploaded?


Prompt templates should be pluggable

There's a growing list of prompt templates that LangStream should be able to leverage.
For example, FLARE: https://github.com/devinbost/langchain/blob/f35db9f43e2301d63c643e01251739e7dcfb6b3b/libs/langchain/langchain/chains/flare/prompts.py
(Demo with CassIO is here: https://colab.research.google.com/drive/1FGMmmkvy3PH7gWQdBBr05HFLTsM6PRU6)
and many other prompts are available in this directory: https://github.com/devinbost/langchain/tree/b3a8fc7cb17170b8a272a0a1366b0f18d8f7ab4a/libs/langchain/langchain/chains
(Look at the files named "prompt.py" in the subdirectories.)
If we can leverage these pluggably, then we won't need to spend so much time trying to keep up as new research results in additional prompt techniques.

I think that it is too early to standardize a mechanism to handle prompts, loops and so on.
We need more feedback from the community.
LangChain is trying to build some model around this topic, but we can wait to see this taking more shape.
It would be great to see how people are trying to build this patterns here with LangStream.
Moving from a flat python file to a scalable deployment with async batch processing will be fun and interesting.

After working a little bit on sample applications and demos I realise that we need to at least make "prompts" a first class object, like we have agents, assents and resources.
Also one big problem is the the prompt format highly depends on two factors:

the LLM model you are using (some LLMs are very "chatty" and they interact with an human, other LLMs are more formal, like the "instruct" models)
the LLM API (some LLMS allows you to report some context and examples (like Vertex), other require a list of messages with "roles" (like OpenAI), others only require a simple string (like OpenSource models from Ollama)

My proposal is to add a new file type that contains prompts and in the chat-completion and text-completion agents you can refer to a prompt by id.
Something like:
prompt1.yaml
prompt:
 - id: qa-chatbot-openai-sample-1
   templating-language: mustache
   messages:
         - role: system
            text: |
                    .... .{{documents}}..
        - role: user
          text: {{value.question}}

prompt2.yaml
prompt:
 - id: qa-chatbot-ollama-sample-1
   templating-language: mustache
   examples:
             .....
    text: |
            .... .{{documents}}..

prompt3yaml
prompt:
 - id: qa-chatbot-vertex-sample-1
   templating-language: mustache
   examples:
             .....
   messages:
         - role: system
            text: |
                .... .{{documents}}..
        - role: user
          text: {{value.question}}

We can also go futher and allow prompts to be read from an external system, like a database

@eolivelli since the only place you use the prompt is the completion, and you likely have only one completions agent in the application, what is the value of putting it out of the agent configuration?
the only pros I see is that it could be easier to make it dynamically loaded from an external system, which I think it's not a common usage
Can you expand more on what would the benefit of this change ?

Apart form making it (in the future) loadable from an external system, I see that putting the prompt into separate files will help in this cases:
Developers should focus on the prompt and not on the boilerplate of the pipeline
Prompts are a core concept in a LLM based applications, if you strip them out of the pipeline file you can "focus" on the prompt, given that the pipeline probably is almost the same for a given type of application (Q/A Chatbot...)
You almost always need more than one prompt
Some applications (in really any application that you want to deploy seriously in production 😉 ) require multiple prompts (see the samples above), having to write them inside the pipeline files is pretty awkward and your pipeline files tend to become huge and not readable/maintainable


Add Notifynder

This is a great package:
https://github.com/fenos/Notifynder

Feel free to send a PR :)


Add warnings for 0-ID nodes and duplicate nodes/edges

Stop writing the same edge in multiple chunks, and only warn when we
want to check an input graph for duplicates.
Fix test data to not duplicate edges.
Make sure to serialize dangling edges when only the larger-id node is
in the graph.

Both these errors (0-ID nodes and duplicate edges) were made by the sg2vg tool and not caught, so I figured I should add code to catch them.
This may make importing graphs that do contain duplicate nodes or edges for good reasons unnecessarily spammy.


Bad Sign in Error

Category

[ ] Question
[ ] Typo
[x] Bug
[ ] Additional article idea

we are facing bad sign in error while executing context.GetRequestDigest() and also used context.useragent in the header ,
DLL used:
we are using SharePointPnPCoreOnline.dll - 1.3 version
Reporting an Issue or Missing Feature
The partner returned a bad sign-in name or password error
Expected behavior
should not get the bad sign error
Thanks for your contribution! Sharing is caring.

Hi is this the same issue as your previous ticket #4393?

yes i pasted in wrong forum,sorry @bcameron1231


[MTSRE-729] AM0016 validator ensuring additional catalog source are unique

Signed-off-by: Ankit Kurmi<EMAIL_ADDRESS>AM0016
Description
A new validator ensuring additional catalog source names are unique.

/retest


Update dependencies and migrate to NodeJS v10

Also, change toolchain:

[ ] Add prettier and integrate with Eslint
[ ] Remove jsdoc plugin from eslint
[ ] Extract babel and eslint configs to separated files
[ ] Replace nyc with ava by jest ☢
[ ] Update semantic-release
[ ] Remove bunyan-debug-stream and pipe dev output through bunyan cli ❓
[ ] Remove nsp
[ ] Add yarn.lock and mention yarn in readme as recommended package manager
[ ] Remove tests from precommit (O_o)
[ ] Update husky
[ ] Update dockerfile
[ ] Add .gitlab-ci.yaml

Coverage remained the same at 100.0% when pulling 0ccb428b782fe9e1699f8b7cba653d459a7e7c78 on v10 into caf102cbf0d18973a5ab089c7d7c3c07767c6789 on master.


What are some use cases of this project?

I missed some details in the README file regarding common use cases of this project. It would be nice if luizalabs explains how she is using it in prod right now in order to get more eyes looking at it.

@kfcaio Hi there!
I think it's worth using when you want to store some key/values in memory, but working with same operations expected from any cache database (get, set, delete, increment..) in an abstracted and simple manner.
I work at Luizalabs, and in our squad, we use it in one of our application as a circuit breaker (we are using lasier as a circuit breaker library) state control database (e,g: threshold for when we open, errors count per service and other parameters) for some services we request.
Before it, our application used a common shared circuit breaker among all our app instances, generating too many requests (network calls).
Since  it wouldn't have a significant business impact (of course it's not the ideal) if we had an inconsistency (e.g: our application has two instances running, but one of them is with an open circuit breaker and the other one no). We could reduce significantly
network calls and hence the performance of the application. (By the way, it is uncommon to find inconsistencies among app instances)
AIO_CACHES = {
    'redis': {
     # Redis configs here
}
    'circuit_breaker': {
        'cache': 'shared_memory_dict.caches.aiocache.SharedMemoryCache',
        'size': 1024,
        'name': 'circuit_breaker'
    }
}

So when we need to pass it, it works like a cache implementation, which has all the operations (get, set, increment etc) that's need for the circuit breaker to save its current state.
    @circuit_breaker(
        rule=rule,
        cache=cache,
        failure_exception=OpenCircuitException,
        catch_exceptions=(CustomServiceException, )
    )
    async service_call(
        id
    ):
         network_call(id)


Text mods under Design (and a couple under Community)

As I went through the Documentation, I have implemented minor text and layout modifications, such as adding introductory texts, inserting links to referred posts, and the like. I would appreciate if you could take a look at the modifications and let me know, if there's something you think should be done differently :-)

/reformat_markdown

Yea, the /reformat_markdown command doesn't really work anymore. Or rather, didn't work well to begin with. I'm still working on that issue, it annoys me :laughing:

Haha, I was so hopeful when I tried it out 😄

Me too :laughing:

Let me know when you'd like me to review and merge it in, unless you will keep adding to this PR?

Will do. My plan was to request a review when I had implemented what I believe I can at this point from the issues I created already (which will be this week). However, I can also wait until you have gone through the issues I've created and add those changes to this pr as well before merging?

@signekb no, if I need to modify anything from the issues I'll create another issue for that.

I'm considering redoing this PR to create a clearer commit history to practice what I'm learning by writing the Git guidelines. This became quite messy, and I think that I will be able to structure it a lot better now. This will take some time but will also ease the review process. What do you think?

Haha, I think its fine, you've learned a lesson :P Next one's will follow the practices!

This looks fantastic! I only fixed the way you put the links. In Quarto, links can be referenced relative to the file and to the qmd version rather than the html version, since Quarto will fix them automatically. E.g. [text](introduction.qmd) rather than [text](https://seedcase-project.org/introduction.html).

Thanks! Ah, yeah, I learned that along the way, but I forgot to go back and fix the ones I had already done. Thanks for reviewing this :-)


Where is *.scss files ?

I have cloned the repo but there are no *.scss files to compile.
Just index.scss with this code
@import "primer-support/index.scss";
@import "primer-base/index.scss";
@import "primer-utilities/index.scss";

@import "primer-forms/index.scss";
@import "primer-layout/index.scss";
@import "primer-buttons/index.scss";
@import "primer-alerts/index.scss";
@import "primer-avatars/index.scss";
@import "primer-blankslate/index.scss";
@import "primer-navigation/index.scss";
@import "primer-states/index.scss";
@import "primer-tooltips/index.scss";
@import "primer-markdown/index.scss";

@import "primer-flex-table/index.scss";
@import "primer-truncate/index.scss";

How can i get the rest?

You need to run npm install and build the project https://github.com/primer/primer#build if you're installed via npm, the already compiled source is in build/build.css

Well, Thank you @jonrohan


Update Homepage

Let's tighten up the landing page at facebook.github.io/jest a bit.

[ ] Move "React Testing", "React Native Testing", and "API Reference" out of the red hero banner at the top. This leaves us with a single "Get Started" call-to-action.

[ ] The "Get Started" CTA links to a section further down in the homepage. Let's add the "React Testing", "React Native Testing", and "API Reference" CTAs here.

[ ] Update "Getting Started" introduction. Reader should be be presented the option to "Get Started with React Native Testing", or "Get Started with React Testing", etc (see previous checkbox). Or, user can proceed through "Getting Started" section using the embedded repl.it.

[ ] Update the three featured items at the top. New ones are: "Easy and Familiar", "Performance", "Snapshot Testing". Update the accompanying text as needed.

[ ] Replace snapshot testing section with a link to the new Snapshot Testing guide and some introductory text.

[ ] Replace babel section with a link to the new Babel configuration guide and some introductory text.

@hramos where can i currently find the 'new' guides for react-native and babel setup? i've spent rather too long getting jest to work for a react-native build as the docs (as linked from the homepage) do not seem to reflect the current situation for an updated (0.39) React Native project.

@tijs checkout the new-docs branch.

@thymikee thanks! the react-native setup outlined there seems the same as the current docs though? https://github.com/facebook/jest/blob/new-docs/docs/TutorialReactNative.md
 ...
  "jest": {
    "preset": "react-native"
  },
...

I'm using the preset there as outlined in the docs but i can only get my tests to run when i add a custom .babelrc which adds some presets. which in turn breaks your build so i now have to resort to different presets for different babel presets..
{
	"env": {
	    "development": {
	      	"presets": [
			    "react-native",
		  	]
	    },
	    "test": {
	      	"presets": [
			    "react-native",
			    "es2015-node",
			    "fbjs"
			]
	    }
	}
}

Quite likely this is not the recommended setup. This fixes my iOS builds but i still have trouble with my android builds with this setup. Either way it seems like the docs do not cover the full setup?

Have you tried setting transformIgnorePatterns?

The transformIgnorePatterns option can be used to whitelist or blacklist files from being transformed with babel. Many react-native npm modules unfortunately don't pre-compile their source code before publishing.

By default the jest-react-native preset only processes the project's own source files and react-native. If you have npm dependencies that have to be transformed you can customize this configuration option by whitelisting modules other than react-native:

not sure what i should be ignoring? on android when i setup jest as above the first error i get as the app bundle loads is error unknown named module: 'object-assign'. Which seems to indicate that react-native failed to transform the js code. strangely this is not and issue for iOS which works fine as long as i run my tests with the alternate BABEL_ENV presets. did not mean to hijack this issue btw maybe i should open a new issue for this..

@thymikee just to be sure; this example is also outdated right? the jest-react-native preset used there should now be just react-native and it's no longer needed to install the jest-react-native package as far as i can tell? https://github.com/facebook/jest/blob/master/examples/react-native/package.json

Yes, there's a note on the beginning about deprecating jest-react-native but it's present throughout the tutorial. @hramos we need to address that.
To ignoring: by default whole node_modules directory is not transformed for react-native (ignored).  If i do not add my own ignore rules the react-native code transforms fine (as the preset takes care of that). But it's not enough to test my react-native project as i get the following error then:
 FAIL  src/containers/__tests__/Transaction-test.js
  ● Test suite failed to run

    Invariant Violation: Native module cannot be null.

      at invariant (node_modules/fbjs/lib/invariant.js:44:15)
      at Linking.NativeEventEmitter (node_modules/react-native/Libraries/EventEmitter/NativeEventEmitter.js:32:1)
      at new Linking (node_modules/react-native/Libraries/Linking/Linking.js:119:141)
      at Object.<anonymous> (node_modules/react-native/Libraries/Linking/Linking.js:191:16)
      at Object.get Linking [as Linking] (node_modules/react-native/Libraries/react-native/react-native.js:91:22)
      at Object.<anonymous> (node_modules/react-native-experimental-navigation/NavigationRootContainer.js:15:36)

This then is solved by adding the fbjs preset too (as in the package.json found in the react-native project). My tests now run and my bundle loads correctly for iOS.
For android i get the error about object-assign not being found though.
Even if that is unrelated it seems the default react-native config for jest should mention the fbjs preset too as that seems to be needed for any RN project not just mine?

Sadly I don't have that knowledge about react-native. But looking at their package.json it seems like they ignore lots of fbjs files, so I guess it is OK if Babel preset fixes this for you.
If you have any ideas how to improve this tutorial, feel free to send a PR against new-docs branch.

@thymikee ok will do. As soon as i figure out a repeatable setup i'll make a PR.

@tijs #2272 is up for grabs if you'd like to take a stab at this.

@thymikee @hramos when creating my own simple mock as outlined in a comment to another issues here: https://github.com/facebook/jest/issues/2208#issuecomment-264733133 the default jest setup for react-native with just the react-native preset configured does work correctly. I can now make builds again and run my tests. So i could add some troubleshooting tips to the current documentation for react-native and update the example. But eventually it's up to the react-native maintainers to make sure their code works correctly with jest out of the box i guess which seems to something they are aware off.


Allow readjusting of label points (PolygonLabels)

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
I would like to be able to readjust/add segmentation/label points to an existing segmentation/label. This is because I would like to allow for collaboration with my team members, where they could also readjust labels created by me.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Add the feature as a configuration setting
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Using pan image shows the label points, but the points cannot be moved.
Additional context
Add any other context or screenshots about the feature request here.

This should be possible by tagging those white dots on the image.

Hi @deppp the readjusting of label points won't work if I selected any of the options from this pane:


[BUG]WorkPro:添加好友报异常 Cannot read properties of null (reading 'corpId'

Please make sure you have read the Wechaty Puppet Service: WorkPro from Wechaty Official Website before you continue writing this issue.
IMPORTANT
DO NOT post your token in the issue. If we need to investigate the behavior of certain token, we will provide a private contact route.
Describe the bug
A clear and concise description of what the bug is.
code is：

const friend_phone = await bot.Friendship.search({phone: phone})
await bot.Friendship.add(friend_phone, {hello:msg})

exception is:

node:internal/process/promises:246
triggerUncaughtException(err, true /* fromPromise */);
^
Error: 13 INTERNAL: Cannot read properties of null (reading 'corpId')
at callErrorFromStatus (/wechatydemo/node_modules/@grpc/grpc-js/build/src/call.js:31:19)
at Object.onReceiveStatus (/wechatydemo/node_modules/@grpc/grpc-js/build/src/client.js:192:76)
at Object.onReceiveStatus (/wechatydemo/node_modules/@grpc/grpc-js/build/src/client-interceptors.js:360:141)
at Object.onReceiveStatus (/wechatydemo/node_modules/@grpc/grpc-js/build/src/client-interceptors.js:323:181)
at /wechatydemo/node_modules/@grpc/grpc-js/build/src/resolving-call.js:93:78
at processTicksAndRejections (node:internal/process/task_queues:78:11)
for call at
at ServiceClientImpl.makeUnaryRequest (/wechatydemo/node_modules/@grpc/grpc-js/build/src/client.js:160:34)
at ServiceClientImpl.friendshipAdd (/wechatydemo/node_modules/@grpc/grpc-js/build/src/make-client.js:105:19)
at node:internal/util:360:7
at new Promise ()
at bound  (node:internal/util:346:12)
at PuppetService.friendshipAdd (file:///wechatydemo/node_modules/wechaty-puppet-service/dist/esm/src/client/puppet-service.js:1265:44)
at Function.add (file:///wechatydemo/node_modules/wechaty/dist/esm/src/user-modules/friendship.js:107:39)
at addFriend (file:///wechatydemo/httpServer.js:233:30) {
code: 13,
details: "Cannot read properties of null (reading 'corpId')",
metadata: Metadata {
internalRepr: Map(3) {
'content-type' => [ 'application/grpc+proto' ],
'date' => [ 'Fri, 13 Jan 2023 10:04:37 GMT' ],
'authorization' => [ 'Wechaty undefined' ]
},
options: {}
},
progress: 'PROCESSED'
}

Info

Platform[e.g. Windows/Mac/Linux]: Linux
Runtime version[e.g. node/JAVA]: node
Wechaty version:   1.20.2
Wechaty Puppet version:  1.19.8

To Reproduce
Steps to reproduce the behavior:

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

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

We might need to investigate token log.
Do you have contact juzi.bot? Please send your token and link to this issue to juzi.bot.

@leesolar @hcfw007 该问题已修复，先关闭该issue。


Feature: in SP, pause timer when the game is paused

First of all, thanks for this project! It's great!
So, as the title says, in single-player it'd be great if the timer stopped when we hit ESC. Not sure if it's possible at all with your current memory reader, but thought I'd open this issue.
If it's trivial, feel free to give me some direction and I could do it myself.

Might be possible, first you need to find a memory address that is True when the game is paused and false otherwise. That might be available somewhere in this list: https://github.com/planqi/slashdiablo-maphack/blob/master/BH/D2Ptrs.h
If not, some research to finding that address would be needed.
After that it's simply a matter of mapping it in the "map_ptrs" function found here for each patch version:
https://github.com/oskros/MF_counter_releases/blob/master/memory_reader/reader.py#L180-L194
And finally, a function could be created to monitor this address and hook it up to the UI, which should probably be done in the functions "_update_lap_time" and "_update_session_time" here:
https://github.com/oskros/MF_counter_releases/blob/master/tabs/mf_timer.py#L69-L77
Not trivial, but possible if the memory address can be located.
Also some case handling should be made to not make the application crash if you don't select to have automode on (since this would entail memory readying, and that is only allowed when using advanced automode)

Actually instead of changing "_update_lap_time" and "_update_session_time", it's probably easier to just invoke the "pause" function when the game is paused 👍
https://github.com/oskros/MF_counter_releases/blob/master/tabs/mf_timer.py#L237-L268

I might've found the memory location, but I don't have much time ATM to work on this, so it might take me a while to submit a proper PR...
If you're interested, I'm looking at address D2Client.dll+FADA4. It becomes 1 whenever I'm in the in-game menu, and 0 otherwise. I haven't tested it thoroughly, though, just grabbed it with Cheat Engine in a few minutes of experimenting.
PS: I'm running patch 1.13c.

Cool, it seems to work for me as well with this address.
I have found the addresses for patch 1.14b+c+d as well from another library, so just need to get the one for 1.13d then I will be happy to make the implementation
I am not familiar with cheat engine, so if you wouldn't mind helping out with getting the address for 1.13d then I will be happy to make the implementation

turned out the addresses I found for 1.14b+c+d didnt work. I have however added this feature in the latest commit, but wont release until I have accumulated some more features and also hopefully found addresses for the remaining patch versions
If you would like to help with finding addresses it would be appreciated, if you have time ofc.
closing the issue


Server should automatically add a newly generated secret key to its keyring

When you start a bindle-server without a predefined secret key, it will automatically create one for you. However, it does not add this key to its keyring. It should do this automatically for 2 reasons: First is devex. Purposefully, a locally run bindle server and client share the same keyring for validation. If we don't auto add it, then we end up making the user do it. Second is that a bindle server should always have its own public key in its keyring anyway since we already trust it.

I plan on adding this before we cut 0.9 as it is an important devex change


Telegram APP - tried to update it on the screen

Application version: [e.g. 2.2.1]
Stationversion 2.4.0
Electron24.8.1
Chrome112.0.5615.204
Node18.14.0

v8<IP_ADDRESS>-electron.0

OS: [Windows 10, MacOS 13.4, CentOS 7 ...]
13.2.1 (22D68) - Intel

Steps to Reproduce:
tried to update the app as it asked me to. (the app had a passcode set)
i have tried the passcode aswelll as other ones to no avail.
deleted the app and retried to add, cleared the cookies in chrome.
i have tried to also use the logout button. ive cleared the process and also restarted the laptop.
still shows the same dialogue. only for telegram however. the rest of my apps are ok.
please see the attached screenshot. i dont want to reset my complete config.. just that particular app.
Screenshots

I'm using Telegram App and it works fine and I've never seen this window.
We have to reproduce the bug to fix it.
Do you have additional security settings in Telegram?
Have you seen this screen before?

hello thank you for the response. the only thing i had added in terms of security was a PIN on the app.. however when i attempted to do an upgrade that it had requested in the APP it just went nuts and showed me this. i've removed and reinstalled the plugin and app but nothing available shows what can be changed regarding that setup or cleared in terms of configuration for that particular app. i'd rather not have to completely delete all my apps and rest station. if that makes any sense?


Auth UIKit type not found

In Framework testing, the UI type variables are not found.
https://github.com/firebase/firebase-ios-sdk/pull/5929/checks?check_run_id=812209394

firestore/Podfile.lock

Closing because UIKit has since been imported in authentication/AuthenticationExampleSwift/PasswordlessViewController.swift


SRG SSR hidden and page view event tracking support

As an app developer integrating Pillarbox I want to be able to track page views according to the SRG SSR standards. I also want to be able to track various arbitrary events.
Acceptance criteria

Page view analytics are correctly collected in the demo.
Hidden tracking is possible.

Hints
Identifiers do not need to be made private as they can be sniffed easily and are publicly visible on GitHub since years anyway.
Tasks

[x] Implement page view analytics for comScore.
[x] Implement page view analytics for Commanders Act.
[x] Implement hidden event analytics for Commanders Act.
[x] Handle user id
[ ] Handle from push notifications
[x] Add Unit tests

Should we still send page id build client side?

nonLocalizedApplicationName was used in witch case?
The use case is when you localized the application name but for analytics we only need one application name. The case PLAY is not concerne with that because the application is "localized" but each version is a real analytics application name.
For example
AnalyticsName : MySRG
FR : Mon SRG
DE : Meine SRG
IT: Magica SRG


added support for Device electronic signature

this was talked briefely in #120, i think @hannobraun 's idea is also the easiest to add.
code was indeed copied from stm-32-device-signature, credit to @Disasm

Note that device_id() code for L0 is incorrect: https://github.com/stm32-rs/stm32-device-signature/issues/9

Oh thanks for the heads up, i'll try to fix that

according to the reference manual, it is at least coherent between x0,x1,x2,x3 family. all have the same memory layout.
I think it would be better to have the user give an array to fill then. or other possibility is doing something similar to stm32f7xx
i'll look into it tomorrow.


Refactor: tests do not inherit from stack

Description
Toward making the Stack class final and removing completely the pattern from C# of declaring subclasses of Stack with @Export annotated fields. The key impediment to that is lots of tests written using this pattern. The PR refactors these tests to use public static void init(Context ctx) pattern of declaring stacks that is similar of what we recommend for production use now.

Fixes # (issue)
Checklist

[ ] I have added tests that prove my fix is effective or that my feature works

[ ] I have updated the CHANGELOG-PENDING file with my change

[ ] Yes, there are changes in this PR that warrants bumping the Pulumi Service API version

My 2nd review: https://github.com/pulumi/pulumi-java/pull/601


Adding Russian language on Crowdin

Hello there. Since the game I play (Genshin Impact) supports this language, I thought it would be great to also add it to this launcher. As a Russian native, I could help with the translation, if interested.

Heyo, please make your request in #146 so we can track it better. Thanks!


failure to import the client class is hidden

i made the mistake of using print statements on python3 with a extended django client class
django still started up however raven ceased to work silently

closing to get off my plate on github

Just got a ping about this. The new sdk has a debug parameter which prints crashes to stderr.
We try very hard, both in the raven and the new sdk, to not take down the app even if the sdk crashes and becomes dysfunctional

I couldn't even produce context if I wanted to, shortly after I reported it I left the company to move across the country


change float constants to improve precision and readability

This has been force-pushed and rebased against #23 so it should be merged after it. Just to avoid having to rebase again.


Out of memory

We use this plugin in the TeamCity and it was worked well but since we started using it more building we get the java.lang.OutOfMemoryError: PermGen space exception. We increased the Permsize to 512m in the TC agent but it didn't solved the problem.
Is it possible that the plugin leaks?
ru.yandex.qatools.allure.report.AllureReportBuilderException: java.lang.reflect.InvocationTargetException
	at ru.yandex.qatools.allure.report.AllureReportBuilder.processResults(AllureReportBuilder.java:135)
	at ru.yandex.qatools.allure.teamcity.AgentBuildEventsProvider.runnerFinished(AgentBuildEventsProvider.java:79)
	at sun.reflect.GeneratedMethodAccessor42.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:606)
	at jetbrains.buildServer.util.EventDispatcher.dispatch(EventDispatcher.java:121)
	at jetbrains.buildServer.util.EventDispatcher$2.invoke(EventDispatcher.java:71)
	at com.sun.proxy.$Proxy9.runnerFinished(Unknown Source)
	at jetbrains.buildServer.agent.impl.buildStages.runnerStages.finish.FireRunnerFinishedFStage.doFinishStage(FireRunnerFinishedFStage.java:29)
	at jetbrains.buildServer.agent.impl.buildStages.RunnerFinishStagesExecutor.runFinishStage(RunnerFinishStagesExecutor.java:32)
	at jetbrains.buildServer.agent.impl.buildStages.RunnerFinishStagesExecutor.doBuildFinishStages(RunnerFinishStagesExecutor.java:23)
	at jetbrains.buildServer.agent.impl.buildStages.startStages.steps.RunnerContextExecutor.safeCallFinishStages(RunnerContextExecutor.java:65)
	at jetbrains.buildServer.agent.impl.buildStages.startStages.steps.RunnerContextExecutor.callRunnerStages(RunnerContextExecutor.java:56)
	at jetbrains.buildServer.agent.impl.buildStages.startStages.steps.StepExecutor.processNextStep(StepExecutor.java:25)
	at jetbrains.buildServer.agent.impl.buildStages.startStages.steps.ForEachBuildRunnerStage.executeRunnerStep(ForEachBuildRunnerStage.java:138)
	at jetbrains.buildServer.agent.impl.buildStages.startStages.steps.ForEachBuildRunnerStage.runStep(ForEachBuildRunnerStage.java:123)
	at jetbrains.buildServer.agent.impl.buildStages.startStages.steps.ForEachBuildRunnerStage.executeBuildRunners(ForEachBuildRunnerStage.java:83)
	at jetbrains.buildServer.agent.impl.buildStages.startStages.steps.ForEachBuildRunnerStage.doBuildStage(ForEachBuildRunnerStage.java:44)
	at jetbrains.buildServer.agent.impl.buildStages.BuildStagesExecutor$1.callStage(BuildStagesExecutor.java:31)
	at jetbrains.buildServer.agent.impl.buildStages.BuildStagesExecutor$1.callStage(BuildStagesExecutor.java:24)
	at jetbrains.buildServer.agent.impl.buildStages.StagesExecutor.callRunStage(StagesExecutor.java:78)
	at jetbrains.buildServer.agent.impl.buildStages.StagesExecutor.doStages(StagesExecutor.java:37)
	at jetbrains.buildServer.agent.impl.buildStages.BuildStagesExecutor.doStages(BuildStagesExecutor.java:24)
	at jetbrains.buildServer.agent.impl.BuildRunAction.doStages(BuildRunAction.java:70)
	at jetbrains.buildServer.agent.impl.BuildRunAction.runBuild(BuildRunAction.java:50)
	at jetbrains.buildServer.agent.impl.BuildAgentImpl.doActualBuild(BuildAgentImpl.java:265)
	at jetbrains.buildServer.agent.impl.BuildAgentImpl.access$100(BuildAgentImpl.java:52)
	at jetbrains.buildServer.agent.impl.BuildAgentImpl$1.run(BuildAgentImpl.java:231)
	at java.lang.Thread.run(Thread.java:745)
Caused by: java.lang.reflect.InvocationTargetException
	at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:57)
	at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.lang.reflect.Constructor.newInstance(Constructor.java:526)
	at ru.yandex.qatools.allure.report.AllureReportBuilder.processResults(AllureReportBuilder.java:132)
	... 28 more
Caused by: java.lang.OutOfMemoryError: PermGen space

Hi, I had similar issue today!
For me helped increasing agent memory.
SET TEAMCITY_AGENT_MEM_OPTS_ACTUAL=-Xmx1100m -XX:MaxPermSize=120m

We tried these:
first test:

SET TEAMCITY_AGENT_MEM_OPTS_ACTUAL=-Xmx512m -XX:MaxPermSize=512m
second test:
SET TEAMCITY_AGENT_MEM_OPTS_ACTUAL=-Xmx512m -XX:MaxPermSize=1024m

But these didn't help.  :(

Summon @eroshenkoam

FYI: I set the allure report version 1.4.0 in the Team City plugin then it works well but I set back to 1.4.14 then I get the exception again.

@pdoggyster could you please specify the size of your allure-results folder and count of testsuite.xml files?

file size: 92 713byte
file count: 1
But now we try to increase the xmx parameter.

@pdoggyster hm...weird
could you send me the allure-results folder (charlie at yandex-team.ru)?

I created 3 empty build step in TC, but the allure reporter build feature was turn on and the out of memory already came. So the test suite XML is not important.

@pdoggyster I spend some time to investigate the problem:
Version 1.4.0 PermGen usage

Version 1.4.14 PermGen usage

As you can see there is no big difference in memory usage.
Also all works fine with TeamCity (I configure 64m of PremGen, mac agent, teamcity 9.0.2 running via https://github.com/nskvortsov/teamcity-sdk-maven-plugin).

Thanks for the information. Try to run this build setup 3 or 4 times. If it is reaching the PermGen limit then the Allure 1.4.14 version will freeze but the 1.4.0 will empty the permgen memory and can run on.

@pdoggyster I found the problem (bug in guice 3.0, fixed since 4.0). New version of allure should fix this issue

This is a good news! I'm looking forward to the new version.

@pdoggyster it turned out that there is also problems with groovy (see https://issues.apache.org/jira/browse/GROOVY-7484). So I propose the following solution:

Update Guice version in allure-report-data (https://github.com/allure-framework/allure-core/pull/620)
Release new report builder with excluded sisu-guice dependency (god damn they just copy some classes from guice 3.0, so it affects the report generation too)
Release new version of teamcity plugin and use https://confluence.jetbrains.com/display/TCD9/Build+Runner+Plugin to run report generation beyond the agent JVM.

This steps helps us to avoid OOM for now. In general we should wait while issue https://issues.apache.org/jira/browse/GROOVY-7484 will be solved and then release new version of allure with new version of groovy

The GROOVY-7484 solved. Is it solved this bug? When will you release?

@pdoggyster we are working on this

Okay, Thanks for the efforts!

@pdoggyster for now you can try to use the beta version of new plugin https://github.com/allure-framework/allure-teamcity-plugin/releases/tag/2.1-beta

We will check it. I will inform you about the result.

I found a bug: the allure result directory value is restored to the default value.
Steps: 1. I modified this value
2. Save it
3. Edit
The result: The value is restored to the default value

@pdoggyster yep, I see. Thanks

We finished the plugin of memory leak issue tests. It looks good, the memory usage normal, no leak.
We are waiting for the final release!

fixed via https://github.com/allure-framework/allure-teamcity-plugin/pull/28 . The release and documentation is coming soon.

@pdoggyster https://github.com/allure-framework/allure-teamcity-plugin/releases/tag/allure-teamcity-plugin-2.1


Image-Copy Extension - Argument of type 'NoneType' error

Receiving this error when using the extension.

Getting os disk id of the source vm/image
argument of type 'NoneType' is not iterable
Traceback (most recent call last):
File "/opt/az/lib/python3.6/site-packages/knack/cli.py", line 197, in invoke
cmd_result = self.invocation.execute(args)
File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/init.py", line 332, in execute
six.reraise(*sys.exc_info())
File "/opt/az/lib/python3.6/site-packages/six.py", line 693, in reraise
raise value
File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/init.py", line 306, in execute
result = cmd(params)
File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/init.py", line 167, in call
return super(AzCliCommand, self).call(*args, **kwargs)
File "/opt/az/lib/python3.6/site-packages/knack/commands.py", line 109, in call
return self.handler(*args, **kwargs)
File "/opt/az/lib/python3.6/site-packages/azure/cli/core/init.py", line 420, in default_command_handler
result = op(**command_args)
File "/home/saadmin/.azure/cliextensions/image-copy-extension/azext_imagecopy/custom.py", line 29, in imagecopy
if 'id' not in json_cmd_output['storageProfile']['osDisk']['managedDisk']:
TypeError: argument of type 'NoneType' is not iterable

If i invoke this code manually i am able to view the json of the image.
az --version

azure-cli (2.0.32)
acr (2.0.24)
acs (2.0.32)
advisor (0.5.1)
ams (0.1.0)
appservice (0.1.32)
backup (1.1.1)
batch (3.2.2)
batchai (0.2.2)
billing (0.1.8)
cdn (0.0.14)
cloud (2.0.13)
cognitiveservices (0.1.13)
command-modules-nspkg (2.0.1)
configure (2.0.15)
consumption (0.3.1)
container (0.1.23)
core (2.0.32)
cosmosdb (0.1.21)
dla (0.0.19)
dls (0.0.21)
dms (0.0.1)
eventgrid (0.1.12)
eventhubs (0.1.3)
extension (0.0.13)
feedback (2.1.1)
find (0.2.9)
interactive (0.3.20)
iot (0.1.20)
keyvault (2.0.21)
lab (0.0.22)
monitor (0.1.6)
network (2.1.0)
nspkg (3.0.2)
profile (2.0.24)
rdbms (0.2.2)
redis (0.2.13)
reservations (0.1.2)
resource (2.0.28)
role (2.0.23)
servicebus (0.1.2)
servicefabric (0.0.12)
sql (2.0.25)
storage (2.0.32)
vm (2.0.31)
Extensions:
image-copy-extension (0.0.6)
Python location '/opt/az/bin/python3'
Extensions directory '/home/saadmin/.azure/cliextensions'
Python (Linux) 3.6.1 (default, Apr 30 2018, 23:14:03)
[GCC 5.4.0 20160609]
Legal docs and information: aka.ms/AzureCliLegal

@rohrerb can you rerun the command with the --debug flag and post the full output (it will include all sub commands and their output)?

Getting os disk id of the source vm/image
command: ['/opt/az/bin/python3', '-m', 'azure.cli', 'image', 'show', '--name', 'baseCentos_7.5', '--resource-group', 'CustomImage', '--output', 'json'] ended with output: {
"additionalProperties": {},
"id": "/subscriptions//resourceGroups/CustomImage/providers/Microsoft.Compute/images/baseCentos_7.5",
"location": "westus",
"name": "baseCentos_7.5",
"provisioningState": "Succeeded",
"resourceGroup": "CustomImage",
"sourceVirtualMachine": null,
"storageProfile": {
"additionalProperties": {},
"dataDisks": [],
"osDisk": {
"additionalProperties": {},
"blobUri": "https://.blob.core.windows.net/disks/baseCentos_7.5.vhd",
"caching": "None",
"diskSizeGb": 32,
"managedDisk": null,
"osState": "Generalized",
"osType": "Linux",
"snapshot": null,
"storageAccountType": "Standard_LRS"
},
"zoneResilient": null
},
"tags": {},
"type": "Microsoft.Compute/images"
}
argument of type 'NoneType' is not iterable
Traceback (most recent call last):
File "/opt/az/lib/python3.6/site-packages/knack/cli.py", line 197, in invoke
cmd_result = self.invocation.execute(args)
File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/init.py", line 332, in execute
six.reraise(*sys.exc_info())
File "/opt/az/lib/python3.6/site-packages/six.py", line 693, in reraise
raise value
File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/init.py", line 306, in execute
result = cmd(params)
File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/init.py", line 167, in call
return super(AzCliCommand, self).call(*args, **kwargs)
File "/opt/az/lib/python3.6/site-packages/knack/commands.py", line 109, in call
return self.handler(*args, **kwargs)
File "/opt/az/lib/python3.6/site-packages/azure/cli/core/init.py", line 420, in default_command_handler
result = op(**command_args)
File "/home/saadmin/.azure/cliextensions/image-copy-extension/azext_imagecopy/custom.py", line 29, in imagecopy
if 'id' not in json_cmd_output['storageProfile']['osDisk']['managedDisk']:
TypeError: argument of type 'NoneType' is not iterable

@tamirkamara
Pretty sure its a managed image.   The image was generated using this command.

az image create -g $rg -n $imageName --os-type Linux --source https://$storname.blob.core.$vhdblobendpointname.net/disks/$vhdblobname

Updated issue: the extension doesn't support copying images backed by a vhd disk stored as blob.
It will be added soon.


Chrome browser fork not supported for live preview

Prerequisites

[x] Can you reproduce the problem with Debug -> Reload Without Extensions? yes
[x] Did you perform a cursory search to see if your bug or enhancement is already reported? yes
[x] Did you read the Troubleshooting guide? yes

Description
While I don't have the Google Chrome browser, I have a browser which is forked from Google Chrome. I see no reason the program couldn't use it, but it won't allow me to attempt to set it up.
Steps to Reproduce

Don't have Google Chrome but install Comodo Dragon
Attempt to use live preview

Expected behavior: allow me to select where it was
Actual behavior: requested I download Google Chrome
Versions
Windows 10
Brackets 1.11.0-17524

@boredboardboared Agree with you on having an option to choose which browser to use while launching live preview. Unfortunately with the current set of things in Brackets, Chrome is a requirement for live preview to work even if live preview is opened in another browser. We are hoping to improve this. But as a workaround please follow the steps to enable multi browser preview.

Install Chrome.
Enable experimental multi browser live preview from File->Enable Expermental Live Preview.
Launch live preview by clicking on the lightning button at the right hand corner of Brackets.
This will launch Chrome. Now you can copy the URL and paste the same in Comodo Dragon.
Note that not all features of Live preview, working in Chrome, will work with other browsers.

Let me know if you are facing any difficulties.


Help output on destroy remote kills is out of date

ran: testtrack destroy remote_kill --help and got the below output which refers to unset_remote_kill which doesn't work
Unsets a remote kill, allowing users of affected apps to see whatever variant
of the split they would otherwise see.
Example:
testtrack unset_remote_kill my_fancy_experiment
Usage:
testtrack destroy remote_kill split_name reason override_to first_bad_version [fixed_version] [flags]
Flags:
--force       Force creation if split isn't found in schema, e.g. if split is retired
-h, --help        help for remote_kill
--no-prefix   Don't prefix split with app_name to refer to legacy splits

Tried to open a PR but can't push to this repo. here's a diff that can be applied:
diff --git a/cmds/create_remote_kill.go b/cmds/create_remote_kill.go
index 6128359..ec51bdc 100644
--- a/cmds/create_remote_kill.go
+++ b/cmds/create_remote_kill.go
@@ -68,7 +68,7 @@ func createRemoteKill(split, reason, overrideTo, firstBadVersion, fixedVersion *
 		return err
 	}

-	// These validations are the difference between remote_kill and unset_remote_kill which is why they're inline
+	// These validations are the difference between remote_kill and destroy_remote_kill which is why they're inline
 	err = validations.Presence("override_to", overrideTo)
 	if err != nil {
 		return err
diff --git a/cmds/destroy_remote_kill.go b/cmds/destroy_remote_kill.go
index a800098..0534128 100644
--- a/cmds/destroy_remote_kill.go
+++ b/cmds/destroy_remote_kill.go
@@ -9,12 +9,12 @@ import (
 )

 var destroyRemoteKillDoc = `
-Unsets a remote kill, allowing users of affected apps to see whatever variant
+Destroys a remote kill, allowing users of affected apps to see whatever variant
 of the split they would otherwise see.

 Example:

-testtrack unset_remote_kill my_fancy_experiment
+testtrack destroy remote_kill my_fancy_experiment reason
 `

 func init() {


Correct multidex typo

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@ened thanks for making this change! as the bot points out, you will either need to add a test for this, or request an exemption on discord.


curator tag trying to tag language for external subrip files

I'm not sure what curator thinks it can do in this case?
$ chrt -b 0 curator tag -s all -t language Risk\ \(2017\).en.srt
...

I wouldn't normally try to tag an SRT file of course, but if you're using some shell glob you will inevitably get one. This results in this plan:

There is nothing to tag in this case because it's just a text file. If the subrip is inside an mkv container then it makes sense...

My bad, I tried to move away from using extensions to detect the format type, but forgot to add checks to ensure .SRT files are skipped.
Fixed in https://github.com/AlexAltea/curator/commit/8af45b6f47fda68192c246af6f83feeb166d7647.


Example in README doesn't produce indicated output

If you run the example as given, you don't get the indicated output, because the default direction for a Node is ".Column" instead of ".Row".

Good catch, thanks!


Django 1.11 support

Hello, when did this package drop support for django 1.11?

That must have been since before I took over maintainer ship, but latest in version 3.0.
I am sure you are aware that Django 1.11 has reached the end of extended support about a year ago.

That must have been since before I took over maintainer ship, but latest in version 3.0.
I am sure you are aware that Django 1.11 has reached the end of extended support about a year ago.

ah, yeah. I found the commit that broke our stuff.
I was hoping you could point me to some table or release note that indicates which django versions are support in each release.
No biggie. We reverted to 3.14.2 and we're all good now.

ah, yeah. I found the commit that broke our stuff.
I was hoping you could point me to some table or release note that indicates which django versions are support in each release.
No biggie. We reverted to 3.14.2 and we're all good now.


Trying to add a 2nd Opponent character (a dad2 if you will) however, i broke it

Describe your problem here.
I set out to add a 2nd Opponent* character on dad side in without having to replace GF as the character and use GF Sing notes, unfortunately, like every coding project I do, well upon going in and testing out the code, the song its set to have a 2nd dad on gives off an error that reads [file_contents,assets/data//no-villains.json] I've tried changing the variable to state something else, and a few other things to tryto fix it, but it didn't work, so I reversed it all to the original set I had, I'm attaching a link to imgur with the code and error I get. If you can shed any light on how to fix it (or just call me a dumbass your call) I would greatly appreciate any and all help
Imgur post with all the screenshots of code I added and the error: https://imgur.com/a/JDpkmrH
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
other than whats in the imgur post no

SONG = Song.loadFromJson('no-villains', 'no-villains');
You forgot to add the second value which indicates the folder

oh ok yeah so it turns out I am just a dumbass however that causes a new problem, now it forces the characters (both opponent and player) to bf and will not change and also the second character doesn't exist.


Auto Update Outside the Store

I found on the docs that there is a parameter named hours_between_update_checks, but I cannot find something like feedURL as a link that provide auto update functionality. Is it still possible to do auto update without publishing to store (using web page or appcast)?

Hello
Auto update outside the Microsoft store are available via the App Installer feature.
This future can use only by organization that have certificate installed on the organization users machines,
and used that certificate to sign the Msix file.
So if is your situation, you can use App Installer.
About auto update source, for now, its available only from file-share and not from the web:


Use a funcref instead of a string in quickmenu#append()

Hi, awesome plugin :smile:
I'm wondering if it would be possible to use a function reference or lambda for quickmenu#append().
For example:
function! ExampleFunc(arg) abort
  return arg + 1
endfunction

call quickmenu#append('call funcref', funcref('ExampleFunc', [10]))

Thanks

this is mostly so that I can pass an argument (like a dictionary) into the function. A better example might be:
for setting in keys(config_dict)
  " Give the option of a setting for each item:
  call quickmenu#append(
            \ printf('Set: %s.%s', area, setting),
            \ { -> config#set_setting(config_dict, setting, input('New value: ')},
            \ )
endfor

good point, new version released

Works great for me. I'm using it in my plugin here: www.github.com/tjdevries/conf.vim
It makes great menus now. Autogenerated cause of your wonderful plugin.

Having the ability to add descriptions is great for helping people to figure out what the options are. Thanks!

The code changes became very easy with your change.
      call quickmenu#append(
            \ printf('Set: %s.%s', area, setting),
            \ funcref('conf#set_setting_prompt', [
                    \ a:script,
                    \ area,
                    \ setting,
                    \ ]),
            \ conf#setting#get_description(a:script, area, setting)
            \ )


Config dirs files

Added consul_debug variable to toggle debugging/testing options. Defaults to no.
Updated consul_config_path to point to /etc/consul and added consul_configd_path defaulting to /etc/consul.d.
Moved all config related tasks to the tasks/config.yml file.
Added the ACL and TLS parameters to the main config.json file that will be created. Using /etc/consul/config.json for all consul roles makes a few things easier. It also fixes a small bug that prevented the gossip key to be read from normal servers (previously bootstrap only).
If the consul_debug variable is set to yes/true, all config files will be generated in the config dir (eg. /etc/consul/config.json, .../bootstrap.json, .../server.json, .../client.json)
The custom config and acl rules will be added to the config.d dir and are prefixed with the number 50. So it uses the common numbering convention to change load order.
Updated all init files to use the new config file/dir layout. Best to have a good look as I'm not 100% familiar with all the different init systems and there files/syntax.
Exposed consul_node_role as a fact. ( Fixes #55 )
Closes #53

👍 It looks good, thanks!


Audio decoding glitch

audio.zip
All generation works fine except the audio one—no matter from which data type.
It looks like there are some problems with audio decoding. Here is the output sample.

Have you loaded the audio checkpoint [CoDi_audio_diffuser_m.pth]? https://huggingface.co/ZinengTang/CoDi/tree/main
Sorry if this is not included in the jupyter notebook.

Thank you for this checkpoint!
Everything works fine now!