IRGen: Use argmemonly for a couple of readonly runtime functions

One of this functions must do more than just read from the pointer array to get the metatype arguments ... I am seeing failures in optimized mode.

Build failed
Swift Test OS X Platform
Git Sha - 7438c52d04795c447015f0c669503cec33767d28

Build failed
Swift Test Linux Platform
Git Sha - 7438c52d04795c447015f0c669503cec33767d28

Or some code that initializes the arguments is inlined in llvm ir then you could have
Type *args[5];
pointer->field = // inlined initialization
arg[1] = pointer
argmemonly(args)

It would be legal for the optimizer to reorder to:
Type *args[5];
arg[1] = pointer
argmemonly(args)
pointer->field = // inlined initialization

Hm, that shouldn't happen, but it's probably not worth spending much time on right now.

I filed https://bugs.swift.org/browse/SR-6889


Correct usage of third body species

Fix #613. The full writeup is in the issue, but ensuring that the species object in the list of products and reactants contains information on whether something is a third body by copying the value from the species array during a read of a configuration file allows the process set to properly detect if a species is a third body or not and fixes our usage of third body species

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

Project coverage is 93.51%. Comparing base (c74099c) to head (4155437).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #614      +/-   ##
==========================================
+ Coverage   93.48%   93.51%   +0.03%
==========================================
  Files          49       49
  Lines        3500     3502       +2
==========================================
+ Hits         3272     3275       +3
+ Misses        228      227       -1

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

Looks good! One question here, Can "tracer_type": "CONSTANT" be always converted to "tracer type": "THIRD_BODY"?

I don't know. I don't think we really respect CONSTANT right now

        "tracer type": "THIRD_BODY",

causes musica.micm_solve() to crash.
"tracer type": "CONSTANT",
does not.


Rating Bar Inside RecyclerView

I implement this rating as items inside recyclerview, but only last item received the value from array. have you ever tried it?

Same issue here. Found any solution ?

I have not successfully implemented this library as recyclerview items, so for now I change to this https://github.com/techery/ProperRatingBar @abhilucks93

Hello @iffanmajid @abhilucks93 ,
I'll try to figure out this problem in this week, thanks you guys!

Hello @iffanmajid ,
Do you mean your OnRatingChangeListener not working?
If so , my RecyclerView works fine.
Could you put some of your implementation for me?
Thanks again!

您好，本人非常喜欢您能把这个控件分享出来，这个问题我也遇到过了RecyleView，应该所有星星都有显示，但只显示了 最后一排的星星 。求解决
代码如下：

效果如下：

Any update on this @ome450901 ?

Hi guys, I have found the problem and trying to repair it.
Sorry I don't have much time to coding. (because my military service orz)
I'll fix this issue in next release ASAP, thank you all for the help.

ok，Thanks a lot brother！！！

Awesome .. Thanks :D


Adjust shadows on landing page

I guess we should have shadows on the bottom right? As with other components, we already have it them that way

like in chain selector for example

👋

hey @yangwao, you want to add box-shadow for the navbar button only or all buttons(e.g., SEE MORE) on the landing page?

hey @yangwao, do you want to add box-shadow for the navbar button only or all buttons(e.g., SEE MORE) on the landing page?
besides, if we change styles for the navbar it would affect every page.

I've just meant just for boxes with text.
No need to do nothing with navbar here :)


Help

Please give your discord i need help :)

This software does not have a Discord. See #543 for a discussion.

I dont need discord server,please give me your discord

I don't take support requests over Discord. If you have a problem to report, please open an issue here on GitHub.


✨Feature Request: Add Containerized Development environment.

Hi all I am creating this PR as a Proof of Concept / place for discussion about whether the Ghost development team (or community) would support maintaining an out-of-the-box containerized development environment. I appreciate that the forums are where we should be adding new feature requests but this is dev specific and I wanted to show some code / effort along with the idea so show it's been thought through somewhat. If this isn't the right place for this discussion let me know and I can move it.
Rationale
While the Ghost Install From Source documentation is thorough it contains a large number of manual steps to scaffold up a new project. In addition (especially for Open Source developers) the overhead of having to manage global dependencies, and multiple node versions, through tools like nvm can be a burden.
By providing a containerized environment for development users can:

More easily sandbox their relative dependencies
Have an easier onboarding developer experience
Perform less manual setup
Have a known working environment from which to start development

Proposed Solution
Create a .devcontainer in the project root for developers to leverage Developing in Containers with VS Code. This is a purely additive feature that does not preclude developers from running an install on their host machine. The proposed solution scripts out basic one-time setup for a user and configures Ghost to be usable from a container.
Current development container workflow

Follow steps from Installing From Source to fork Ghost, Ghost-Admin and Casper.
On your host machine clone the Ghost repository as per the following command git clone --recurse-submodules git@github.com:TryGhost/Ghost && cd Ghost.
Open the repository in VSCode
Install the extension Remote - Containers.
In the .devcontainer folder create a new file called ghost.env adding the following content.

GHOST_FORK_NAME=git@github.com:<YOURUSERNAME>/Ghost.git
GHOST_ADMIN_FORK_NAME=git@github.com:<YOURUSERNAME>/Ghost-Admin.git
GHOST_CASPER_FORK_NAME=git@github.com:<YOURUSERNAME>/Casper.git

Select Reopen Folder inside Container from the Command Pallete`.
Wait (a fairly long time) for the containers to be pulled, built, and the container to pull down all relevant dependencies...Seriously go get a ☕
Once the log says Ghost is configured to listen on  <IP_ADDRESS> run a grunt dev like normal and code away in the container without needing any dependencies on your host machine. Nice!

Known caveats of current example

The dev container uses a root user, this doesn't bother me personally but Linux best practice would be to create a non-root user to do this all I guess.
The command grunt master won't work. The current work flow assumes that you'll dev in the container but still do your git stuff on the host machine. Everyone's git setup is different so perhaps an addendum to pass in ssh keys or something if you like in doco would be sufficient, I think trying to cater to all use cases here would be tricky...
A first time install will override the config.development.json file with the correct listen host. I am not sure if this is desirable or not.
To save going overboard with configuration I have left the current setup script is fairly simple. For example if you delete your node_modules but have set your upstreams the setup file will crash. It should ideally self heal or just ignore errors to make rebuilding a little easier if you need to update packages or something...?

Feedback
In general I would love feedback on whether this is something the Ghost team would be willing to support or if there was other community interest in it. If this is something the community sees as valuable I'd love some discussion around some best-practice, where you expect to see uplift in the current solution and any other advice you can provide.

@iamscottcab Thank you for this PR! I apologise for the delay but this is on my list to review ASAP.

This idea is beyond brilliant.  I have just been wrestling with the development environments.
I think there is some hubris on the part of senior developers.  It is not that hard to set this stuff up.  I am a senior developer, but it is very hard for me to set it up.  Why?  Well I am a senior Python developer, all this node stuff is new to me.   An alphabet soup of technologies which are all new to me.  Plus I just have too much on my plate. Which OS should i be using.  Debian?  coreOS, Freebsd?  Too much complexity on my plate whatever you can do to simplify it is all good.
It making a developer's life easier your top priority?  Probably not  That is why we all still use....(name he technology you hate the most, I do not want to start a flame war. )


Using ZFS in combination with suspend to disk may cause filesystem corruption

Describe the bug
Suspend to disk / hibernation in combination with ZFS may lead to filesystem corruption.
To Reproduce
The problem only shows up occasionally about once on every 30 startups.
I had been using suspend to disk on my laptop without problem for several years.
After changing from BTRFS to ZFS the import of the ZFS pool  occasionally stalls when resuming from
hibernate. A message says that the pool is corrupted and gets reconstructed.
Once the pool was corrupted  irreparably.
Root cause
ZFS does not support filesystem freeze and thaw under Linux at the moment.
(see https://github.com/openzfs/zfs/issues/260)
When going into hibernation the pool is not exported/unmounted.
(The system is not properly shut down at all and this is same situation for all filesystems).
The problem is that when resuming from hibernation the initrd tries to import the pool
(it also tries to fix a corrupted pool and effectively modifies the pool/partition).
This happens before checks for a possible resume from hibernation
and before the system returns to the state in which it had been suspended.
This means that after returning from suspend to disk
the partition with the zpool is no longer in the state
it was when the system was suspended.
Suspend to disk is a kind of hack which leaves (or may leave)
the filesystem on the hard disk in an inconsistent state.
Discarding the suspended system state or mounting the file system from an other installation may cause problems.
On the other hand before I had changed to ZFS I did not experience any problems
and this filesystem corruption caught me without warning.
Expected behavior
It may be possible to fix the problem by reordering the sequence of actions in initrd.
Unless there is a fix for this NixOS should disable the combination of ZFS and hibernation.
(More precisely, initrd should either contain the setup scripts for ZFS or the scripts for
return-from-hibernate).
At least there should be a warning.

BTRFS is native to Linux, so its suspension mechanism and Linux's were (presumably) always set up to work with each other. On the other hand, ZFS's suspension code has not yet been hooked up to Linux's properly: https://github.com/openzfs/zfs/issues/260.
I agree that—

[…] NixOS should disable the combination of ZFS and hibernation. […] At least there should be a warning.

One potential problem seems to be that there seem to be multiple packages that can cause suspension. In my NixOS configuration, I tell logind and GDM not to suspend, but I don't know whether that's everything that might cause suspension.

@8573 Thanks for adding the link.
I've added a corresponding line of warning here: https://nixos.wiki/wiki/NixOS_on_ZFS

Do you suspend to disk with a zvol swap?  Because I'm running with root on ZFS on my laptop and I have never encountered problems with suspend to disk, but I'm also using a separate regular swap partition.

I for one use a regular swap partition. I recall hearing that swap-on-ZVOL is dangerous independently of hibernation.

Would it make sense to automatically add boot.kernelParams = [ "nohibernate" ]; in cases where zfs is in use?

This. A note in wiki or manual will be good too.

Let's see if we can close this with https://github.com/NixOS/nixpkgs/pull/171680

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/filesystem-recommendations/28486/14


[New Sample] Local server generate elevation profile

This PR is for a new Local Server sample, using a geoprocessing package authored in ArcGIS Pro. The gpkx file (downloaded by the sample from ArcGIS Online) was created using the Interpolate Shape geoprocessing tool, with an input string (pointing to a raster) and feature (polyline). The workflow requires a raster dataset (downloaded by the sample from ArcGIS Online to local samples-data folder), and a user drawn polyline on the scene view. An elevation profile is drawn where the polyline sits over the raster (the polyline is interpolated onto the z-values of the raster).

thanks Mark and Mike! @colinanderson this is ready for your review now :)

@Rachael-E Do you need to add any data credits/attribution to https://arcgisruntime.maps.arcgis.com/home/item.html?id=831cbdc61b1c4cd3bfedd1af91d09d36 ?


remove DeprecationWarning

hi all
I just upgrade my grpc to 0.12.0 and grpcio to 0.12.0b0, when I start the server, I had some warnings.
DeprecationWarning: the alpha API (includes this package) is deprecated, unmaintained, and no longer tested. Please migrate to the beta API.

I looks like protobuf gen code warning. How to remove that? (upgrade protobuf python package or grpcio?)
Thanks.

Coming soon to a release branch near you: #4755 was merged to master and addresses this (insofar as symptoms go - the alpha code is still in the runtime but will be removed Soon™). The protoc plugin is currently distributed in a different way, and so an updated version would come from wherever you got the gRPC core (but, it too will soon be shoved into a PyPI package Soon™).
As far as I know, the only current way to get the above niceties is by working off of the master branch. They should all be present come version 0.13.0, however.
Feel free to re-open if I missed anything!


Session Tracking

We have been tracking session info in the clients for some time but we need to expand on this area some more to provide session dashboards. This will allow us to:

Display dashboards with users affected
List all sessions and allow you to dive into them to see what occurred during that session.
Update the exceptions page with a view of what happened in that session right before the error occurred.

Server

[ ] Add new event plugin step to lookup or create a new session start event. Cache value would be the document id for the start event. Allowing us to continually update the document with last activity.
[ ] Batch last activity document updates so that we don't relentlessly update the start event. Update once every X seconds.
[ ] Create a job or cache behavior that creates a session end for any expired sessions that are still in the cache. This will allow us to query for active sessions and sessions that have ended. This will update the session start event with the session duration and set last activity to the end time of the session.
[ ] Should we have an upgrade plugin that drops environment info and user info for existing events? Need to make sure that data does not change between events and if it does change, we would need to break the data out into stuff that is specific to the session and stuff that is specific to the event.
[ ] Add new project_users  Denormalize the user identity data into a new elastic index (keyed on user id & project id)?

[ ] These should be long lived, but maybe we should consider cleaning them up after a certain amount of inactivity.
[ ] Do we need to keep it in sync with user info on session start event?
[ ] Would we update the user last active time on the heart beat or only on session end?

JavaScript Client

[ ] Make the script load async from cdn and collect any runtime errors. We need to promote this as the best option so we can rev the script faster. We should move bower and npm install methods to a side note.
[ ] Better track session start, maybe set a session guid and time of client creation time and submit it when the first event is submitted. We should check a cookie or local storage for an existing session id.
[ ] Send heartbeat from the client as well as performance metrics from window.performance. We'd also have to track the Node process startup/runtime time as part of the metrics.
[ ] Only the session start event should contain environment & user info.

What should happen if someone calls EventBuilder.setIdentity()?

.NET Client

[ ] Better track session start, maybe set a session guid and time of client creation time and submit it when the first event is submitted.
[ ] Submit Session End on exit.
[ ] I'm not sure if we need to send a heartbeat for this.
[ ] Only the session start event should contain environment & user info.

What should happen if someone calls EventBuilder.setIdentity()?

.NET Client Updates

https://github.com/exceptionless/Exceptionless.Net/commit/e06373e6493b96dafa7c24e27ea7ed5b6c5372d2
https://github.com/exceptionless/Exceptionless.Net/commit/90da26fd86f8b3a363370f4dd8d1caa0cd8d5bf9
https://github.com/exceptionless/Exceptionless.Net/commit/26c0d7f26341b8f0c0d26cacd2c30ffd3c40fd89

https://github.com/exceptionless/Exceptionless/tree/session

NET Client
Seems that we would want the ability to send a heartbeat only on wpf and winform clients. We may even want to specify a default session id that is used by default on these clients (would use this session id even when the user is set and only changed when the user is changed), There are two scenarios of which we would want to send heat beats:

User specified a sessionid
Set the user on the config object.
a. What happens if the there is no user on startup?
b. What happens if the user is changed at runtime or via EventBuilder? Should a session stop and start event be triggered if so how? There are no events that get fired when this happens...

I think this could be solved by doing the following steps.

Move the UseSessions() Extension method to only the winforms and wpf client.
Since it's a single session the session manager will continue to work as expected on only these clients.
Create a Heartbeat manager or event that is fired when default data is changed. Then we can flow these changes through (unregistering existing heartbeat and registering a new one..)

New concept for linking together events..
Let's assume I have a session start… it has a single reference id… I have a page view event… it has a single reference id...
Now I have an event that happened inside of the page view and it sets 2 extended properties… 1 that points to the page view event and 1 that points to the session event. From the client I just call SetEventReference(“session”, sessionRefId)  SetEventReference(“view”, viewRefId)


Create conda pkg for juliacall at conda-forge

I am very surprised that PythonCall supports conda so well, but its python module juliacall itself is not available on conda (at least so far I couldn't find anything on anaconda or the installation documentation).

Indeed releasing on Conda Forge would be nice but I haven't had time to do so. Would be a welcome contribution.

I was actively looking on julia dependent packages on Conda and actually currently it is a bit difficult to depend on conda julia or conda juliaup. Both don't have support for arm64 as of now.
My target architecture is mostly arm64, so I would wait for this to stabilize first. Or create a conda package without a julia dependency... also not ideal.

Also see https://github.com/JuliaPy/PythonCall.jl/issues/378#issuecomment-1755354797 for related PRs. Also very eager to see a conda-forge implemented for juliacall so I can move my libraries from pyjulia (but sadly no time to do this myself).

Now available as conda-forge::pyjuliacall FYI, feel free to close


Battle shows wrong armor and shields for stacks

Dont multiply the armor and shields for stacks

The armor should be 45, because it's a base frigate. What should the shield be?

The code just displays the design's shield amount, not the stack shields

You should see the stack as this is more interresting if you want show the Stack and (ship).


Upgrade attestation v1.1.0 on baklava (external validators)

confirmed all baklava validators upgraded with celocli identity:current-attestation-services --node https://baklava-forno.celo-testnet.org


Support at least one other test framework

Generalise logic to load test adapter
Adapter should be loaded from project output dir

Added support for NUnit v2, NUnit v3, xUnit vX

Fixed with release v<IP_ADDRESS>.


Magerun plugin

I would like to create a Magerun plugin (for Magento, https://github.com/netz98/n98-magerun) with the following features:

Magerun binary (usually magerun or n98-magerun, or n98-magerun.phar locally), for cache:flush, sys:setup:run, config:set path value etc.
Command to download a dump of the media folder (media:dump path/to/zip --strip) or the database (db:dump --strip="@stripped") to a local path.
Deploy strategy for magerun or something?

Any tips on the best approach or example plugin to follow? Probably similar to the Laravel plugin?
For downloading files, is there a best practice? Just write to a tmp folder inside the application folder en use the SSH connection to get the file?
And should I focus on Rocketeer 3 or Rocketeer 2? Don't know if there is an ETA or anything, or is the upgrade relatively easy?
And last question, should I put this in the Rocketeer namespace (+ rocketeers/rocketeer-magerun package)?

FYI, it deploys fine already, I can just manually run the commands in the after hooks and the git+composer also is okay. Just wanna try to make it a bit easier :)

While the plugin would be doable for Rocketeer 2, Rocketeer 3 made plugins far more powerful, see per example the Laravel plugin (which is now decoupled from core in R3). Ultimately it's your choice, you can even probably do both.
I don't have an ETA for R3 yet but a massive part of the planned features are done, so hopefully soon.
As for the namespace I can create a repository on the organization and give you access (like it is done per example for Flysystem plugins).


Logging and Exception Handling

Hello All,
What is the best practice to be followed for Bots, to

check the health of the bot application
log or handle errors errors for bot applications

/Naren

We recommend using Application Insights - for example:
http://aihelpwebsite.com/Blog/EntryId/11/Using-Application-Insights-With-Microsoft-Bot-Framework
@jameslew did you have another article about using App Insights for Bots?

@willportnoy Application Insights seems like a bad solution for node.
It sends strange .net exceptions instead of node traces

@Stevenic do you have experience using App Insights with node?

@willportnoy no I don't. we have some new logging stuff we're working on but not sure it's going to help with this. We're working to add support for streaming a sessions logs to the emulator for debugging purposes but you want more on going monitoring. There are a couple of 3rd parties like http://botanalytics.co/ which have integrated their tools with the Node SDK. That might be an option.


SBP-4433: update plugin config in sb-kong-vagrant

Update the plugin config and workflow in the package to include the jwt, rate-limiting and jwt-claims-headers plugins

bleh, again accidentlay added this to the upstream... cannot do an upstream PR until some of this code is abstracted out


Creating a class with implicit PK, does not create a PK

If I take a model with no attributes and use CreateTable with CreateFlags.ImplicitPK or CreateFlags.AllImplicit, the PK does not seem to actually get created.
If I inspect Table.Table, there is no PK specified. In addition, if I use InsertOrReplace, it inserts duplicate rows (all fields are identical).

My tests also show that this isn't working..
If you take the Customer test class and remove the Id property
    public class Customer
    {
        [MaxLength(64)]
        public string FirstName { get; set; }

        [MaxLength(64)]
        public string LastName { get; set; }

        [MaxLength(64)]
        public string Email { get; set; }
    }

and create a table with CreateFlags.AutoIncPK
conn.CreateTableAsync<Customer>(CreateFlags.AutoIncPK).Wait();
you'll see that in in SQLite.cs -> public int CreateTable(Type ty, CreateFlags createFlags = CreateFlags.None)
this query is produced:

create table if not exists "Customer" (\n"FirstName" varchar(64) ,\n"LastName" varchar(64) ,\n"Email" varchar(64) )

Your Customer class must contain an int property called Id.
The CreateFlags.ImplicitPK and AutoIncPK simply allows you to avoid the [PrimaryKey, AutoIncrement] attributes.


Adding/ modifying

I'm i required to modify or add a .txt document to run the fabric and java/Curse-forge mods together. i would apricate quick response :)

What the hell are you trying to do

Not more than making a server where I use some Fabric and curse-forge mods

I am not sure what you mean, this is a mod template for architectury, a tool to compile a mod to both fabric and forge. This repo that you are commenting on is mainly mod developer-facing.
I will close this issue now, but you are free to continue discuss here, we will try our best to help.

Can u just say if this is a mod i put in a mod folder if I want to use forge and fabric mod at once. since I'm confused about waht this dose when u are saying for developer-facing

Architectury does not let you use fabric and forge mods at the same time.
It allows mod developers to more easisly maintain mods for both fabric and forge.

Do u know of a mod that dose ?

No, patchwork kinda did but it doesn't work. Give up on that.

sadness :(


Secret Management: Can't mount the secret inside the container

Rancher Versions:
Server: v1.4.0-rc5
healthcheck: v0.2.3
ipsec: rancher/net:v0.8.7
network-services: rancher/network-manager:v0.4.5
scheduler:rancher/scheduler:v0.6.1
kubernetes (if applicable):
Docker Version:
1.12
OS and where are the hosts located? (cloud, bare metal, etc):
Ubuntu 16.04
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Single node Rancher
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:

Run the secret catalog driver.
Create a new secret.
Create a new container and specify the secret you just created with any name.

Results:
The secret is not mounted under /run/secrets/name however the secret driver is showing the following error:
1/30/2017 8:58:04 PMtime="2017-01-30T18:58:04Z" level=error msg="Unsuccessful request: 500 Internal Server Error"
1/30/2017 8:58:04 PMtime="2017-01-30T18:58:04Z" level=info msg=mount.response mountpoint="/var/lib/rancher/volumes/rancher-secrets/872b4eb3d9bbbce9b2ee2e694cb57f16011b04485923753b421fdd5ad2184622"

The next release of the agent will have the key fix.

on v1.4.0-rc6, whenever i try to add a secret to a container, i am getting the following error
Error (Scheduling failed: volume [73] must be one of pool(s) [4], valid host(s) [1])

This issue is seen when  attempting to create container with secrets in a set up with more than 1 host where we see the container fail to start with  Error (Scheduling failed: volume [73] must be one of pool(s) [4], valid host(s) [1]) error.
Subsequent attempt to create another container with secrets may succeed, if it ends up
ends up in 1 particular host (not sure which one) in the setup . After this , we are able to create multuple containers with secrets successfully as long as these containers land on this 1 host.

Tested with rancher-server version - v1.4.0-rc8
Had a rancher-server setup with 3hosts.
Able to start containers with secrets on all 3 hosts of the setup. Also able to access the secrets from with in the container.

Rancher version 1.4.0-rc8
steps:

Register 3 agents with Rancher
create a secret with value foo=bar
deploy the secrets catalog template
create a global service where a container will run on each host and select the secret "foo" with the name "foo"
exec into the container on each host and make sure that /run/secrets/foo has the content "bar"

I was able to validate the steps with both localkey and vault backend


⚠ Roblox TextFilter Api has degraded performance

In c4c8978, Roblox TextFilter Api (https://textfilter.roblox.com) experienced degraded performance:

HTTP code: 200
Response time: 13625 ms

Resolved: Roblox TextFilter Api performance has improved in a53857d.


chore(apps/prod/tekton/configs/triggers): disable hotfix tags to trigger builds

Signed-off-by: wuhuizuo<EMAIL_ADDRESS>

/approve


ORDER BY DESC queries should return an error

queries that have any order other than ORDER BY ASC should return an error. These will be wired up at a later date, but until that time we should tell the user that it doesn't work (rather than returning incorrect results).

This change will need to be reverted some time soon.

@otoolep is this issue obsolete now?

Yes, all fixed right now -- thanks @dgnorton


Programming exercise: Detailed test-results not displayed

Describe the bug
Detailed messages on why a test failed are not shown in Artemis.
The role this is executed with, makes no difference. Additionally it is seen in the exercise details view as well as from students perspective who is working on the exercise.
This is very urgent as the detailed messages are a core part of my exercises (and the research for my thesis) and my students can hardly work without them.
To Reproduce

Go to 'Exercise Details' (of an exercise which's template tests have not run in a while)
Click on 'Test result'.
See detailed messages for every failed test case.
Return to details page and run tests for template repository.
See the bug: Some test cases have no details at all others are marked as 'not executed'.

Expected behavior
Artemis is expected to always show the detailed messages.
Screenshots
Template results before rerunning tests:

Template results after rerunning tests:

Which version of Artemis are you seeing the problem on?
6.9.2
What browsers are you seeing the problem on?
Chrome
Additional context
No response
Relevant log output
During the descripted steps nothing is logged in console.
However when reloading the page the following log is shown:

[Violation] Listener added for a synchronous 'DOMNodeInserted' DOM Mutation Event. This event type is deprecated (https://w3c.github.io/uievents/#legacy-event-types) and work is underway to remove it from this browser. Usage of this event listener will cause performance issues today, and represents a risk of future incompatibility. Consider using MutationObserver instead.

analyser.js:485 [Deprecation] Listener added for a synchronous 'DOMNodeInserted' DOM Mutation Event. This event type is deprecated (https://w3c.github.io/uievents/#legacy-event-types) and work is underway to remove it from this browser. Usage of this event listener will cause performance issues today, and represents a risk of future incompatibility.  We only support the most recent Surefire report format, requiring a plugin version of  3.0.0 or higher. Please update your version in the pom.xml file.
To avoid issues like this, activate the Upgrade Template option when importing Maven exercises from old courses. This option will automatically update your dependencies.


I can't sign in to freeCodeCamp through GitHub or Twitter account, Please help

I used my GitHub account to sign in to freeCodeCamp but now I can't sign in. I also connected my twitter account with freeCodeCamp but it is not working as well.
I feeling very confused and stressed since I have completed a lot of challenge.
Please help me get my account back.
User Information:
Full Name: Shuvongkor Barman
User Name: shuvongkor-barman
GitHub User Name: shuvongkor-barman
Twitter User Name: shuvongkor1
Issue Description

Browser Information

Browser Name = Opera
Version: 52.0
Operating System: Windows 10
Desktop

Your Code

Screenshot

@shuvongkor-barman (note the msg from freecodecamp) https://github.com/freeCodeCamp/freeCodeCamp/issues/14669#issuecomment-341918303 :

@raisedadead commented on Nov 4, 2017
Yes, we have disabled new signups via GitHub, you can expect to see this button gone soon too(when we release next version).
If you are signing up for the first time, please do so via the email, and then link up GitHub through the settings on your profile.

Simply create a freecodecamp account using the same email you used to create your github account.
After that, log into freecodecamp using that email, then go to https://www.freecodecamp.org/settingsand you can link your github account, and your progress should still be there.
Good luck

I did exactly what you said but although it is showing that I have completed 112 challenges, on the top it was showing only 1 and when I completed some now it is showing 4. And in the Map it is showing that I did not complete the first 112 challenges.
Why is that?
And please help me solve this problem.

-try https://forum.freecodecamp.org/c/support bc it seems this is a known issue
-Also try reaching someone at freecodecamp https://github.com/orgs/freeCodeCamp/people like @raisedadead @QuincyLarson

@shuvongkor-barman apologies for the delay in getting back to you. please send an email to<EMAIL_ADDRESS>from your email that you have with us.
We will try and get this fixed.
/cc @QuincyLarson

@raisedaddead I can't sign in either. I have already sent you<EMAIL_ADDRESS>without any response.
cc @QuincyLarson

Hi @shuvongkor-barman @mabras Quincy is currently away until Monday for a work trip. He may not have had access to emails.
He is the sole person at the moment, who can access the DB.

Hi @shuvongkor-barman I understand the frustration and apologise for the delay. Unfortunately, as I said, we will have to wait for Quincy to be back. The write access to DB is limited for obvious security reasons.
We know the issue that you have duplicate accounts which need to be merged. Rest assured, your progress is saved. Merging them to your email account with us will get this working again.
May I suggest that you do not wait, and keep working through the next challenges, and save the solutions offline somewhere? This way when we have merge your old account with your email, all you would have to do is just re-submit them?
Quincy, reaches US on Monday just so that you know.
Thanks and sorry about the inconvenience.

Okay Then. I will wait for the fixing. But I will continue my practice and save my project somewhere online or offline.
Thanks.

@shuvongkor-barman This should now be working on learn.freecodecamp.org


create users for a specific time

Where can i find an overview of all endpoints from the Unifi API?
Do you know if there is an endpoint for creating users for a specific period?

Currently there's no official documentation covering (all) the UniFi controller API endpoints. That being said I believe the API client here is the most extensive one around, partly thanks to all the contribtions from the community.
Second, I don't quite understand what you mean by "creating users for a specific period". Can you elaborate on what you are trying to achieve?

Oké thanks for the answer.
Second, i mean with "creating users for a specific period" that if there is a way to create "login accounts" on UniFi where we can say that this account is only useful between 2 sprecified dates.

OK, login accounts for accessing the controller or the wifi network? If it's the network, the only way to implement accounts (user name/password combinations) for wifi access is WPA2-Enterprise with a Radius back-end. That is currently out of scope for the API,  at least for the moment.
Or do you mean to authorize a device for guest wifi access between two certain data/times?
If it's for access to the controller, there is no endpoint in the API that supports that.

i'm sorry, maybe i dumb and i didn't quite get it, but...
scenario: i open the unifi interface, and create voucher there (from date to date, and probably using "note" to put the room number).
Is there any way to do the same through the API? I couldn't find this info anywhere, but, if you look here in unifi forum at the rcourtney post, he says

Keep in mind these fields can be auto filled and then can be modified
in your application software before calling the APIs.

As to him, this was possible...
What do you think about it?

I see where you're coming from, but there is no feature in the UniFi controller to create a voucher with a start date and end date, only expiration time (I believe this starts at moment of activation):

All parameters entered in that form can also be applied to a voucher creation request through the API with this function/method:
https://github.com/malle-pietje/UniFi-API-browser/blob/master/phpapi/class.unifi.php#L859-L887


Image loaded is corrupted

I'm trying to do this:

Steps:

Clone https://github.com/QuantumInformation/Density-Wars
run npm i
run webpack
inspect /public to see corrupted file:

I think I know what's going on. In the current code you have inline file-loader!./assets... kind of definition. file-loader gets evaluated there first. After that it goes through url-loader due to the file size (less than 15k). As a result it emits both a file (first definition) and transforms the output through url-loader.
Based on that I would say it's a configuration problem and not a bug.
Here's a little diff where I worked around it:
diff --git a/client/Ground.ts b/client/Ground.ts
index f963b15..ec096bd 100644
--- a/client/Ground.ts
+++ b/client/Ground.ts
@@ -1,5 +1,7 @@
 import { common } from './Common'
 import Texture = BABYLON.Texture
+
+const url = require('./assets/img/background.png')
 /**
  *
  * The ground (actually a grid in space) that the game sits upon
@@ -24,8 +26,8 @@ export default class Ground {
     this.mesh.position = new BABYLON.Vector3(0, 0, 0)
   }
 }
-var url = require("file-loader!./assets/img/background.png");

+console.log(url);

 /*
 let img = new Image()
diff --git a/package.json b/package.json
index 1c1c8b3..6462e3c 100644
--- a/package.json
+++ b/package.json
@@ -6,7 +6,7 @@
   "scripts": {
     "lint": "tslint client/**/*.ts",
     "test": "echo \"Error: no test specified\" && exit 1",
-    "build": "webpack --watch --watch-poll && cp node_modules/babylonjs/babylon.js server/public",
+    "build": "webpack && cp node_modules/babylonjs/babylon.js server/public",
     "start": "npm run build & node server/bin/www"
   },
   "repository": {
diff --git a/webpack.config.js b/webpack.config.js
index 18999e8..678df9e 100644
--- a/webpack.config.js
+++ b/webpack.config.js
@@ -71,7 +71,6 @@ const commonConfig = merge([
 const productionConfig = merge([
   parts.loadImages({
     options: {
-      limit: 15000,
       name: '[name].[ext]',
     },
   }),
diff --git a/webpack.parts.js b/webpack.parts.js
index 790ad44..b203cfc 100644
--- a/webpack.parts.js
+++ b/webpack.parts.js
@@ -7,7 +7,7 @@ exports.loadImages = ({ include, exclude, options } = {}) => ({
         exclude,

         use: {
-          loader: 'url-loader',
+          loader: 'file-loader',
           options,
         },
       },

Great that works!


Add cop_helper to packaged files

spec/support/cop_helper.rb is required for testing the https://github.com/lovewithfood/rubocop-rspec-focused gem.

All for this change. rubocop-rspec and rubocop-cask could remove their respective submodule dependencies as a result :+1:

I'm fine with the change, but it'd be nice if you added some comment explaining why we've done this.


filter @ Not working

Hello! How are you?
I don't know if this issue has been addressed or if it is just me, but is it normal that in your demo #7, @xxxxx is not filtered?
Thank you Very much in advance!

Odd - that example looks exactly right to me. Which tweet beginning "@username" do you see in that list? And which browser are you using?
-Steve

Filter problems here as well. Problem only occurs in Internet Exlorer. For a client I am trying to filter only the tweets that are made by the account user self and not by a follower or someone following. This works as a charm in Chrome/Firefox but it almost looks like I get the opposite effect in IE 8 & 9 (didn't test 7). I put an alert/console.log in the tweet function of tweet.js: console.log(tweet.item.from_user); In Chrome/FF, I get the expected users (correctly filtered), in IE I get none of the expected users (incorrectly filtered).

Strange. I'll take a look when I get chance, but if you figure out what's wrong in the meantime, any clues would be appreciated.
(PS. Henk - is your use case something that can't be done using the "query" option instead, e.g. "from:accountowner sometag"?)

Thanks for your reply!
The client I have wants to display the last tweets of 4 different
Twitter accounts and then only the messages tweeted by the users of
those accounts. I will check if I can use the query option instead.
Quoting Steve Purcell
<EMAIL_ADDRESS>
Strange. I'll take a look when I get chance, but if you figure out
what's wrong in the meantime, any clues would be appreciated.
(PS. Henk - is your use case something that can't be done using the
"query" option instead, e.g. "from:accountowner sometag"?)
--
Reply to this email directly or view it on GitHub:
https://github.com/seaofclouds/tweet/issues/186#issuecomment-2574544

The best alternative might be to use the "list" option, which would allow your client to change the set of 4 accounts later if they wish.

I fired up a Windows machine to test this, but in IE8, the example works exactly as intended; the @replies are correctly excluded from the list, so I'm confused. Is the problem maybe that the description of the example made you think that only the @replies would be shown?

This is the init I use:

4 accounts are loaded from a database and they seem to work. I then
needed to filter out everything that was not tweeted by the account
user's themselves. I hacked the following into the filter function in
tweet.js:
filter: function(tweet) {
if ((s.username.join("")).indexOf(tweet.item.from_user) >= 0){
return true;
}else{
return false;
}
I made fetch bigger than count (4x). This works fine on Chrome and FF,
there only tweets of the users themselves are displayed. On IE, no
tweets are displayed at all.
Quoting Steve Purcell
<EMAIL_ADDRESS>
I fired up a Windows machine to test this, but in IE8, the example
works exactly as intended; the @replies are correctly excluded
from the list, so I'm confused. Is the problem maybe that the
description of the example made you think that only the @replies
would be shown?
--
Reply to this email directly or view it on GitHub:
https://github.com/seaofclouds/tweet/issues/186#issuecomment-2575640

Henk: I'm guessing that's because of the trailing comma in the resulting array of usernames, which I believe IE will complain about. It probably breaks the entire script on that platform.

Oh, and yes - the "list" option is definitely your best choice. Set up a list on twitter which follows those 4 users, then configure tweet.js just like in the list example (#5), and set the "retweets" option to false.

It was the trailing comma. I should have been aware of that because I
have encountered that issue before with a different code. It works
perfectly now.
The list sounds indeed like a better option and I will certainly try it.
Quoting Steve Purcell
<EMAIL_ADDRESS>
Henk: I'm guessing that's because of the trailing comma in the
resulting array of usernames, which I believe IE will complain
about. It probably breaks the entire script on that platform.
--
Reply to this email directly or view it on GitHub:
https://github.com/seaofclouds/tweet/issues/186#issuecomment-2576427

@Henk - Great! Glad it's working now.
@coder20938 - I think that you were perhaps surprised that a tweet containing "@xxxxx" was present in example 7, but I am now assuming that the tweet began with "RT @xxxxx" or otherwise did not begin with "@xxxxx". The example code is designed to filter out only the tweets beginning with that pattern.
Since it seems like everything is working as expected on all platforms, I'll go ahead and close this issue.


Frontend project port number change

Purpose
Changed the frontend project's localhost port number, to be different from the sensor service, to be able to run the sample in local dev machine.
Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

cc @bradygaster


Opera blocking amplitude by default

Well, I've just found that new Opera has built-in adblock and when it is activated it blocks most of the tracking scripts, including Amplitude ofc, Sentry, Google Analytics etc.
Do you have by any chance some workaround for it? Or do you know whats the exact reason (some script characteristics or whatever) behind that?

@Andarist unfortunately it seems like the only way to circumvent this is to track events yourself (to your server), and then send the data server-side to Amplitude via our HTTP API

@djih do you know whats the exact reason (some script characteristics or whatever) behind the issue?

@Andarist not sure, depends on how the adblocker in Opera works (I have no idea how it works). For example, Adblock uses like a blacklist, and one of the common blacklists is EasyList: https://easylist.to/easylist/easyprivacy.txt. I can see our JS SDK on the blacklist, as well as google analytics, and Mixpanel.
If it's simply blacklisting a script name, maybe what you could do is download our script and host it yourself, but rename it to something like my-site.analytics.js or something, and see if that circumvents the blocker.


PPO algo performs badly

The PPO algorithm is difficult to converge，and the gripper always move up and away from the table.Could you please give me some hint about it.Sincerely appreciate it!

I would recommend using SAC. It performed the best.

I would recommend using SAC. It performed the best.

Thanks,SAC indeed did a wonderful job!


validate project settings

Received this email from Microsoft, need to validate this & other IBM extensions:

Hi,
This is Kai (@kieferrm) from the VS Code team. Thanks so much for creating your VS Code extension! We very much appreciate you being part of our extension community and all your hard work to make users happy.
We have been rolling out changes to how we handle extension icons and images in the README and CHANGELOG files. Your extension is effected by those changes. We only allow images with resolved https addresses. We also restrict the usage of SVGs. We don't allow SVGs as extension icons and we'll only allow SVGs as badges or as images in your README or CHANGELOG file when they come from the following list:
https://vsmarketplacebadge.apphb.com/
https://travis-ci.org/
https://david-dm.org/
https://ci.appveyor.com/,
https://badges.gitter.im/
https://www.bithound.io/
https://img.shields.io/
https://gemnasium.com/
https://isitmaintained.com/
https://coveralls.io/
https://api.travis-ci.org/
https://codeclimate.com/
https://snyk.io/
https://badges.greenkeeper.io/
https://travis-ci.com/
https://badge.fury.io/
https://badges.frapsoft.com/
https://cdn.travis-ci.org/
https://marketplace.visualstudio.com/
https://codecov.io/
These changes will go fully into effect when we release the next version of VS Code in approximately 1 week from today. To make sure that your extension continues to look great on the marketplace and within the VS Code extension list, please make the appropriate changes to your extension and republish it. There are a number of free tools available online to convert SVGs to PNGs that are a great help.
If you have any questions, please let me know.
Keep up the good work!
Cheers, Kai

updated readme for compliance, and pushed a new build to vscode market.


feat: operation type to report

Explanation
Add operation type to report.

/cherry-pick release-0.2


update proxy of master to raftstore-proxy

What problem does this PR solve?
Issue Number: ref #4982 close #6228
Problem Summary:
update proxy to raftstore-proxy
Proxy PR:
Including:\nSubmodule contrib/tiflash-proxy 2d3399cbe..e1b8e44c6:

Fix pprof unwind error on dwarf (#213)

What is changed and how it works?
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/cc @breezewish @flowbehappy @JaySon-Huang @Lloyd-Pottiger

/run-all-tests

/merge

/run-integration-test


Changed Volatility of age_cos, age_tan and age_cot from STABLE to IMMUTABLE as they give same result for the same argument values.

Changes: -
Updated the volatility category of functions age_cos, age_tan, and age_cot from STABLE to IMMUTABLE.
Reasons for modification: -

As these functions only depend on their argument, it doesn't modify the database.
These functions don't do database lookup.
Guaranteed to give the same result for the same argument values.

@Pratikkapsime Please fix your commit log entry for the PR to be descriptive of the change. Once this is corrected, let us know so that we can review the PR for merging.

@jrgemignani  I have updated. Please let me know if you want more changes.


Update Names for the Automation Manager features to match the Ui

Update names and descriptions for the Automation Manager features to match the Ui
Links
https://www.pivotaltracker.com/story/show/138739009

@miq-bot  add_label enhancement

@dclarizio , @Fryguy  -please review
@miq-bot assign @Fryguy

👍 from me, now matches the UI tabs.


config: replace time parameters by cycles like ticks

To facilitate synchronization of all activities by a relative metric instead of a fixed time.
This way I could enable realtime dormant capabilities, by killing dead-time spaces, and separating with fixed cycles the relation between fragile sequenced steps like shepherdizing, reallocation, etc.
Examples of this are now, the impossibility to open all the configuration params. for user management.
All times are fixed and are unsafe to change without breaking some obfuscated logic of behaviour, that indeed is logically consistent, but the problem is not being able to acknowledge it clearly from parametrization, and maintain a "learnt" kwnowledge base of numbers (millis) in the config class.

done


Range Calendar outline ring gets cropped in range selection

Package name(s)
Lab (@salt-ds/lab)
Package version(s)
No response
Description
focus ring gets cropped in range inside range calendar
Steps to reproduce
in the calendar range example, select a range and navigate trough keyboard
Expected behavior
the ring should not get cropped
Operating system

[X] macOS
[ ] Windows
[ ] Linux
[ ] iOS
[ ] Android

Browser

[X] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

Are you a JPMorgan Chase & Co. employee?

[X] I am an employee of JPMorgan Chase & Co.

@mark-tate should this be addressed in your PR?


problem with loading canFam3ToHg38.over.chain

I have a Windows system with 32GB ram and LiftOver("canFam3ToHg38.over.chain") took all of the memory and did not finish. cProfile.run of the same command with a portion of the chain file showed that most of the time was spent in add_interval function and another related function. Can there be a way to do this?

Hmm. Although the canFam3ToHg38 is a large file with 38M blocks, I don't see how those wouldn't fit in 32G (also, I'm trying to load it on a machine with even more memory and it seems to be stuck).
This might be related to the overly naive implementation of the intervaltree data structure being used, however it's pretty hard to debug (especially given that other chainfiles have worked fine so far).
I'll need to try a more decent intervaltree implementation (e.g. something with rebalancing), however I'm not sure when I'll get the time to look into it.

So, apparently, the problem is not in the algorithm, but indeed in the memory consumption (the 64GB machine could handle the file).
I tried to add a couple of changes which should considerably reduce the memory consumption. It still takes forever (several hours) to load, though. Try the newer version and see whether it helps. In principle, once you load the file (if it succeeds), you may try pickling the resulting LiftOver object - I suspect it would be faster to unpickle it than re-read and re-index everything.
There's also a new flag for the LiftOver constructor which might help a bit. Namely, show_progress=True will display a progress bar. You need to do pip install tqdm for this to work, though.

Thanks for your answer. I have been thinking about the possibility of speeding up by using something like Cython or numba. Have you tried already, or do you have any thought on this?

I tried using ncls (https://github.com/hunt-genes/ncls) (numpy needed). It seems to work. For canFam3ToHg38.over.chain, it takes about 2 minutes and 7.3 GB of memory to load the chain file. If you are interested, the modified file is at https://drive.google.com/open?id=1OA65_8EUrk9zZi-iQqMKYQASEAN27mA1.

No, I haven't tried neither Numba nor Cython because the initial aim was to have a super-simple pure-Python tool (also, numba did not even exist when this was first written, I believe). And it does work fine for the "common" use-cases (i.e. hg18-to-hg37, etc), that canFam3 mapping is more of an exception in terms of the amount of re-mapping.
Things like numba/cython/psyco/pypy are in general reasonable directions to look for speed-ups, however I suspect getting a considerable effect might not be straightforward. Also, I'd prefer to have a tool which does not require compilation or heavy dependencies.
I tried running the current version on a Linux server with 32G memory, and the results are as follows:

Loading & indexing data: 13:35 (min:sec)
Pickling: 13:14
Unpickling: 4:38
Pickled object size: 5.2GB.

Thus, if you need to just get things done, for now I'd suggest to either try a Linux machine (somewhy it runs faster than on Windows, also note that Python 3.7 seems about 1.5-2x the speed of 2.7) and, in case you need to load the file multiple times, pre-pickle it.
Alternatively, perhaps just try the standard liftover binary.

(I somehow missed your last comment before posting mine, now saw an email notification though).
Nice, I'll check the ncl option and will probably replace my data structure with that one if it is faster.


Feat: Added Discord server info

I think I did it right... Feel free to correct me.

Thanks, will look into tests (never used them at all). I could see from the fails it was prettier but had no idea why. Thanks learned how to do a pull request anyway, I'm calling that a win 😆


Cannot restore from dump on cloud; Error about postgresql.auto.conf

EdgeDB Version:"5.2+ddd8aa4"
EdgeDB CLI Version:EdgeDB CLI 5.1.0+7c5764f
OS Version:Windows 10, Ubuntu WSL

Steps to Reproduce:

run edgedb restore --all -I edgedbcloudorgexample/mydatabase ./mylocaldump
obtain the following error message:
edgedb error: InternalServerError: could not fsync file "postgresql.auto.conf": Invalid argument
Server traceback:
edb.errors.InternalServerError: could not fsync file "postgresql.auto.conf": Invalid argument

Schema:

The issue here is likely that cloud does not support doing CONFIGURE INSTANCE on certain configuration settings that directly affect the underlying postgres instance.
We should produce a proper error message, though.
I think it should be easy to workaround by editing init.edgeql in the dump directory to remove the offending configures

What configuration values is init.edgeql trying to set?

Ahhhhh, it looks like session_idle_transaction_timeout is always included in DESCRIBE INSTANCE CONFIG because during bootstrap, we execute CONFIGUREs for any backend settings with default values: https://github.com/edgedb/edgedb/blob/df39a451f8c49efc2a332fb4f4072d7b20b9807c/edb/server/bootstrap.py#L1761-L1782
We don't currently have a way to distinguish that from the case where it has been explicitly set.
As a related issue (with a TODO) in that code, we don't have the correct default for session_idle_transaction_timeout when on a backend (like our cloud) that doesn't support CONFIGURE INSTANCE.
We have code that is supposed to produce a proper error message when doing a backend setting CONFIGURE INSTANCE, but for some reason it isn't firing in the cloud setting: https://github.com/edgedb/edgedb/blob/master/edb/pgsql/compiler/config.py#L53-L58
I think that for backend params with default values, we are going to need to give up on the distinction between using default and set to default, but in the opposite direction than we currently are.
(Another option would be to always store our configs for these things in database metadata potentially alongside as system configs, which we can use to disambiguate these cases. If we want to bother with that, it might be a longer term fix)
Action items here:

[ ] Figure out why the has_configfile_access check isn't working
[ ] For backend settings with a default value, ignore CONFIGURE INSTANCE if it is setting it to the default value and we don't have configfile access. (Or maybe just always.) This will fix restoring from existing dumps
[ ] Make DESCRIBE CONFIG not include any backend_settings that have their default values
[ ] Stop doing CONFIGURE INSTANCE at bootstrap and instead configure those settings to the default when connections are created and there is no explicit setting

(Note that the third thing is still necessary even once the fourth is done, because absent other config system overhauls, configuring the settings then will make it seem like they are session configs)

@msullivan It worked commenting out the first line about timeout in the init file. Then I had to wipe the main branch, but then restore worked as expected including data.


fix: run npm audit fix

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Network speed notification not correct, speed notification is slower than in reality

Hello,
I use android 5.1 phone.
I have a 8 Mbit Internet connection, but the gasflow app shows a maximum of 1.3-1.4 Mbit download-speed.
I install OpenVPN for android, the same problem happens. With the VPN Software i see the download-speed 4.5 Mbit, but the gasflow app shows 1.2 Mbit.
Do you have a solution?

Well i tested and the information displayed is what the OS is giving through the traffic manager.


Import System for rsp startup check

Cannot reference Environment without System, so the startup check for RSP fails.

Thanks, but the error has already been fixed in dev branch. dev branch branch will be merged into the master branch at release time.
https://github.com/lilxyzw/lilToon/blob/dev/Assets/lilToon/Editor/lilStartup.cs#L5

Missed that. Thanks.


Update-MarkdownHelpModule should not require dashes in all cmdlet names

I've following code:
New-MarkdownHelp -Module $Module -OutputFolder $PSScriptRoot\..\docs
Update-MarkdownHelp $PSScriptRoot\..\docs
New-ExternalHelp $PSScriptRoot\..\docs -OutputPath $PSScriptRoot\..\docs -Force

I run it to update docs, but also create new word docs when I add new functions. I get errors (as the files exists)

I don't want to use -Force as I guess the file will be overwritten and any changes to it as well. Yet I need an option to create new docs as I add more functions.

I don't quite follow the scenario you have. Are you using one of the common setups?
It sounds like you run New-MarkdownHelp in CI, which suggests that source of truth is not in markdown. But then why would you run New-ExternalHelp?

@vors Interesting find... Update-MarkdownHelpModule and I was using Update-MarkdownHelp. What is the difference and why there are 2? :-)

And the other problem is, while it did found new file, created it. But then I rerun and I get an error. Only on that new file that was just created (and not on the rest that are untouched).

What is the difference and why there are 2? :-)

One is for HelpModules which are folders of docs with a index markdown doc with some metadata that's needed to create .cab files and such. In this example it talks exactly about your use-case https://github.com/PowerShell/platyPS/blob/master/docs/Update-MarkdownHelpModule.md#example-1-update-a-markdown-help-module

Right... but sti.ll

Gets me this:

It's funny thou because it recognized all other files and skipped them properly. It creted Testing.md when I run it the first time but subsequent try results in this errror.

Oh, I see so it's the naming convention really - we should not assume that everything without a - is a module page for the module update scenario.

As a workaround for now you can use Update-MarkdownHelp and pass the list of commands to it explicitly, since it doesn't have this limitation about the dash.

Do you have "best" way to go forward with a fix? I mean I don't use this command lately so I don't need it but seeing as I already found a case I may as well fix it. Just not sure what would be your approach (unless there's bigger issue here...)

New-ExternalHelp has this exact same issue since it also calls GetMarkdownFilesFromPath. As a result, when I run New-ExternalHelp on my module (consisting solely of one word functions without dashes) I get absolutely no output.
Is there any workaround one can implement in that scenario to generate the external XML file?

@lordmilko it's a matter of tweaking GetMarkdownFilesFromPath. If you want to take a stab at it, perhaps you can simply avoid filtering there for the Verb-Noun pattern and add a new switch parameter -RelaxCmdletNameCheck or something.

Adding a new switch should be simple enough, but the question is whether it's a good idea. Discoverability of this new switch wouldn't be really straight forward.

And if you notice there's already switch that's not used anywhere it seems IncludeModulePage that when used would solve the issue.

This should already be fixed in Microsoft.PowerShell.PlatyPS v1. The workflow is going to be different in the new release, so even if there is a problem, a new issue should be opened against the new version once it is released.


Default value of ignore_eos

System Info
Now, ignore_eos default value is set to lazy_mode.
Whereas default value in normal transformers is False.

This results in different values for Accuracy and Perf in Gaudi when compared with GPU.
Also, In summarization tasks, What's the need for setting ignore_eos to True. Stopping when we reach eos should be there for summarization, right?

Right now, only way to set the flag is from generation_config. In case of default generation config, We have to update and that will cause issue with CI accuracy and perf which were right now running with ignore_eos as lazy_mode (which is True in most cases).

Shouldn't there be an extra flag to provide along with command line? So that each model can set it based on the need?

Information

[ ] The official example scripts
[ ] My own modified scripts

Tasks

[X] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction
pytest -s -v tests/test_encoder_decoder_text_summarization.py
Expected behavior
There should be a way to set ignore_eos value without impacting other runs and models

@bhargaveede The generation config should be enough to specify it no? Maybe we can set ignore_eos to False in the generation config in the example scripts for summarization and translation. WDYT?

@regisss Yes, we can specify in the config.
But every model has it's own default generation_config.json. (https://huggingface.co/facebook/bart-large-cnn/tree/main)
(or) some models generate in the on the fly from model.config (https://huggingface.co/t5-3b/tree/main)
For regression testing, having separate generation_config.json for every model with extra flag(ignore_eos) set to False will be hard to maintain. Everytime, new model is supported/ existing model config is modified, We need to add/modify generation_config locally rather than depending on the model file in hugging face repos.
Rather, I suggest having an extra flag in the command flag "--ignore_eos False" which seems better.
With that, Whatever changes to generation_config will be taken care of without the need to modify local configs.

@regisss I have created a PR to update generation_config for Bart.
Can you please take a look at same.
Also, who's responsible for the model PRs in hugging face?
https://huggingface.co/facebook/bart-large-cnn/discussions/58

I would rather avoid modifying the generation config of BART as ignore_eos is not defined in Transformers.
Adding it as a new training arg sounds good to me, feel free to open a PR and I'll review it :slightly_smiling_face:

@bhargaveede Can we close this issue now that #469 has been merged?


Partition issue

I was at first trying to use cryo utilities to increase my swap size but that didn’t work so I tried to do it threw kde partition manager. From there it was basically unusable whatever I did or changes I thought I would be doing wouldn’t take and would remain the same. Any ideas to fix this or is it something to be patched later

Cryoutilities is not supported and there is no swap file by default. The ability to go back to a swap file is in Bazzite Portal (Called Legacy Memory Config and labeled as not recommended), and you can customize the size of it via just.

I figured it out I had to edit it had to edit it from a bootable disk while the disk was unmounted


[Windows] - Problems with linking Alby Account

I had two users that had this problem, and I can replicate on my machine.
When I have Alby Account not linked to any node, and try to link this (Windows) Alby Hub to my Alby Account, I'm (and other users) getting info that it couldn't be linked

In fact, on getalby.com/node it tells me that Alby Hub is linked
But the Hub still doesn't recognize it

Only after refresh (and on Windows there is no 'refresh' button like in browser, so the only thing to check things is to shut the app down and open it again (another issue?), it says it is linked to another wallet

trying to send via this lightning address

Alby Dashboard shows the transactions (receive via invoice works ofc), but doesn't show the balance

Creating the invoice via Alby Dashboard gives such error

Sending to Lightning Address from Alby Dashboard works though

The "Payment aborted: Recipient server error" is something that I have seen when the Alby web account is not linked to the Alby Hub.


Green nudge doesn't recognize shift key.

When holding down the right/left mouse button on the green nudge button, all inputs are accepted (W,A,S,D,SPACE) except for the SHIFT key. I am unable to nudge the selection down.
I am using Windows 7 64 bit.
By the way I mean no offense by this but this UI and its controls are horrible.

Nevermind I figured out the issue. It wasn't that SHIFT wasn't working, it was that the bottom of my selection was at the bottom of the world. Either a notification of this or just cutting off the bottom of the selection (that's what I meant to do) would be fine.

What would you suggest changing in the UI?

First off these option dialogs need either a "Cancel" button or an "x" in the corner to close them without making changes. Also clicking on the white space outside the dialog should close it without accidentally using the current tool you have equipped.

Also my selection/brush shouldn't be moving in the background as I'm mousing over these dialogs, because that makes me think I'm accidentally going to select something or add some blocks when I click on one of the checkboxes etc.
These random shapes/colors in the NBT Explorer mean nothing to me as a person who's never seen this program before. I wasn't even sure if the triangles are expandable sections or if they just represent some other data type.

Normally these kinds of indicators at least have letters so you don't have to memorize an arbitrary color code. Also + and - are global indicators of expandable sections.

Why are some checkboxes to the right of their labels:

... and others to the left?

This may seem like a minor issue but consistency is one of the key factors of usability and inconsistency frightens new users and makes them lose trust in the application.
The clone menu doesn't have the X, Y, and Z nudge fields labeled.

 You have to click the top and bottom arrows. Also, you can't hold the top/bottom arrows down to keep moving. You have to keep clicking. So attempting to scroll is useless here. At least the mouse scroll wheel works, but not everyone has one of those.

Selections are confusing to look at, because you can't tell where they intersect with blocks. It's useful to be able to see them through blocks, but the portion of the selection that is covered up by blocks should be in a different color or darker opacity so you have a better visual indication of what's going on. I'm aware that the blocky texture around the selection only shows up above ground in this instance, but it's really difficult to see. The lines at the edge of the selection are the primary indicators of what's going on, and should have partial opacity when obfuscated from the view.

I can't tell you how many times I've messed up a clone because I didn't select what I thought I selected.
Also in most editing programs I expect the "Esc" key to be used as a cancelling option, such as:

Removing/deselecting the current selection
Closing/cancelling dialogs.

Why does the current selection still display while I'm selecting a new one? This is a strange behavior and as far as I can tell it doesn't have a purpose other than blocking my view. The moment you click to begin a new selection the old one should disappear, still allowing you to Ctrl + z to get it back.
There should be keys to lock your selection to specific axes when still holding the left mouse down while making a selection. For example, holding down the "z" key while selecting would restrict movement on the z axis, only allowing movement along the x and y axis. This is a common feature of 3d modeling programs.
I don't think you should see entities through walls by default. I'm sure this is useful in some cases, but most of the time it's distracting when the bulk of your work in this tool is moving blocks around. Also maybe have them fade out with distance when you can see them through walls, so you get a better visual indication of what's going on.

The "Hidden Ores" checkbox is not clear that it governs all of the checkboxes below it. Normally this would be indicated by an indent but everything is right aligned.

Also there are many menus/dialogs that don't appear to have any clear pattern of organization.

In general center-aligned forms are not scannable and look kind of hectic. There is a benefit to having your labels left aligned so you don't have to look left and right to scan for a word as well as up and down. It makes for a good eye exercise, though, I suppose.

Here is a small article for explanation:
https://www.back40design.com/blog/m.blog/83/simple-web-form-design-techniques
There's more I could go over but I've already posted a miniature essay at this point.

Wow, thanks for that! That would be super useful and we'll try to do it all... What is the problem with Esc? It does remove selections and remove forms. About the selection remaining while start selecting a new one is a feature that exists because it's very common to start selecting a new area accidently and clicking Esc then, would keep the old selection. I think somone already fixed the scroll bar thing already, it was a bug, I think. About the not organized menus, we're aware, it's just not so simple to fix but we'll try to get to it later. And do you suggest always aligning menus to the left? Wouldn't it make it look worse since most of the right side would be empty?

These random shapes/colors in the NBT Explorer [...]

Dunno what happened to the normal bullets. You may want to restore the default setting to use them, or verify your MCEdit installation.
What you see is a fallback...

At least the mouse scroll wheel works, but not everyone has one of those

But any decent keyboard have arrow and page up/down keys...
By the way, this scroll bar bug appear only in widgets having huge amount of item to display
and is already fixed in the dev version.

Also in most editing programs I expect the "Esc" key to be used as a cancelling option, such as:

Removing/deselecting the current selection

Ctrl-D by default. Changeable. Just remember Esc is used to display the main menu while editing a world...

Also there are many menus/dialogs that don't appear to have any clear pattern of organization.

Any suggestion?
Thank you for the link. I think most of us are aware of that.
I don't know if you already tried to program UIs for applications, but it is not always as easy as for web forms...
About the controls, remember that every program has its own control system. To use them properly, you'll always have an adaptation period.
Last, MCEdit is not a finished program. Work is on progress, and we are a few people working on it as a leisure, not a professional dev army.

Some good points here, just going to go through them.
"First off these option dialogs need either a "Cancel" button or an "x" in the corner to close them without making changes. Also clicking on the white space outside the dialog should close it without accidentally using the current tool you have equipped."
Yeah, they could use a cancel button, we just haven't gotten around to the details like that. As for the clicking outside the window, the editor's UI library Albow isn't that advanced and was designed with games in mind, so like many games such as lets say Transport tycoon it was designed to allow interacting with the world while the menu was open. May get changed depending on if someone wants to rewrite that behavior.
Also my selection/brush shouldn't be moving in the background as I'm mousing over these dialogs, because that makes me think I'm accidentally going to select something or add some blocks when I click on one of the checkboxes etc.
Another limitation of albow, needs further redoing to fix and since it hasn't been causing actual issues it hasn't really been messed with.
"These random shapes/colors in the NBT Explorer mean nothing to me as a person who's never seen this program before. I wasn't even sure if the triangles are expandable sections or if they just represent some other data type.
Normally these kinds of indicators at least have letters so you don't have to memorize an arbitrary color code.."
I didn't even know there were real icons for this as they don't work for me either, but the icon's text in LaChal's and your screenshots are so small I'm not sure how they're much better than colors, I'd think either large colored letters or differing icons (Like the windows registry editor) instead of many near identical file icons with letters would be better than both existing systems.
"Also + and - are global indicators of expandable sections"
In many web UIs and such, but i've seen the triangle often enough in UIs not limited to text only it's pretty obvious what it is.
"Why are some checkboxes to the right of their labels:"
Yeah, oversight, whoops.
"The clone menu doesn't have the X, Y, and Z nudge fields labeled."
Also an oversight.
 You have to click the top and bottom arrows. Also, you can't hold the top/bottom arrows down to keep moving. You have to keep clicking. So attempting to scroll is useless here. At least the mouse scroll wheel works, but not everyone has one of those."
The scrollbar is custom since Albow doesn't support them out of the box, and the click and drag is fixed for next build. We've only had the scroll function for a month or two so it'll get there :P.
"Selections are confusing to look at, because you can't tell where they intersect with blocks. It's useful to be able to see them through blocks, but the portion of the selection that is covered up by blocks should be in a different color or darker opacity so you have a better visual indication of what's going on. I'm aware that the blocky texture around the selection only shows up above ground in this instance, but it's really difficult to see. The lines at the edge of the selection are the primary indicators of what's going on, and should have partial opacity when obfuscated from the view."
Definitely true, but since no one really understands our renderer at a low level, this will likely not be fixed.
"Also in most editing programs I expect the "Esc" key to be used as a cancelling option, such as:
Removing/deselecting the current selection
Closing/cancelling dialogs.

"
Esc should close dialogs, agree there. after that as mentioned Albow is based on a game UI so opening the menu is pretty normal, not sure I'd change that part.
"Why does the current selection still display while I'm selecting a new one? This is a strange behavior and as far as I can tell it doesn't have a purpose other than blocking my view. The moment you click to begin a new selection the old one should disappear, still allowing you to Ctrl + z to get it back."
This has been useful behavior to me if I need to use the old selection as a frame of reference for making a new one, although it's really what people want, indifferent here.
"There should be keys to lock your selection to specific axes when still holding the left mouse down while making a selection. For example, holding down the "z" key while selecting would restrict movement on the z axis, only allowing movement along the x and y axis. This is a common feature of 3d modeling programs."
Making selections by axis would be nice, so you don't have to fight it when selecting something of an odd shape. MCEdit 2 has something like this in place already as it selects a plane then depth all the time.
"I don't think you should see entities through walls by default. I'm sure this is useful in some cases, but most of the time it's distracting when the bulk of your work in this tool is moving blocks around. Also maybe have them fade out with distance when you can see them through walls, so you get a better visual indication of what's going on."
This one is highly subjective and may be useful for command block enthusiasts and CTM trapmakers, should have an option with old behavior being default.
"The "Hidden Ores" checkbox is not clear that it governs all of the checkboxes below it. Normally this would be indicated by an indent but everything is right aligned."
Wanted to, but I couldn't figure out how.
"Also there are many menus/dialogs that don't appear to have any clear pattern of organization."
If you have any ideas on how to arrange the top bar feel free to suggest, but none of the functions really seem related.
"In general center-aligned forms are not scannable and look kind of hectic. There is a benefit to having your labels left aligned so you don't have to look left and right to scan for a word as well as up and down. It makes for a good eye exercise, though, I suppose."
True in some ways, but to really make this proper we'd likely need to redo the giant font size and such as that would leave a lot of dead space on the right. The fields still wouldn't be lined up, I think a 2 column approach may be best. Like this: https://www.google.com/search?q=map+editor&biw=1265&bih=680&source=lnms&tbm=isch&sa=X&sqi=2&ved=0ahUKEwjLpPWs8J_MAhWDJh4KHQJ1AVoQ_AUIBygC#tbm=isch&q=ingame+map+editor&imgrc=sCOMYsjZVgyjlM%3A

What is the problem with Esc? It does remove selections and remove forms.

It always opens the main menu for me. I haven't changed any of the default settings.

And do you suggest always aligning menus to the left? Wouldn't it make it look worse since most of the right side would be empty?

Yes, the point of aligning left is making the form scannable, e.g. easy to find what you're looking for. This doesn't mean you can't put fields side-by-side like the H/L/W above. Having all of your white space to the right is much better than having it scattered about in random places.

Dunno what happened to the normal bullets. You may want to restore the default setting to use them, or verify your MCEdit installation.

Huh. I installed from scratch. Good to see you've already done something about it.

By the way, this scroll bar bug appear only in widgets having huge amount of item to display
and is already fixed in the dev version.

Good!

I don't know if you already tried to program UIs for applications, but it is not always as easy as for web forms...

I've messed with stuff like Java swing. As a consumer of the application I'm not concerned with how difficult it is to code something, just making a mention.

Last, MCEdit is not a finished program. Work is on progress, and we are a few people working on it as a leisure, not a professional dev army.

As with most projects on github. Don't take my criticism of the application personally; I'm not attempting an ad hominem here. Overall the tool functions well and serves its purpose. I haven't encountered any map breaking bugs, which was the most important thing to get right.

In many web UIs and such, but i've seen the triangle often enough in UIs not limited to text only it's pretty obvious what it is.

Yes the triangle is common for expanding sections like an accordion. Not debating that, but its placement here looks like it's just another icon indicating a different type. Doesn't matter apparently, because this was already changed.

This has been useful behavior to me if I need to use the old selection as a frame of reference for making a new one, although it's really what people want, indifferent here.

Maybe fade out the old one a bit while selecting a new one? I'm just used to editing programs like Photoshop, GIMP, Blender, where the instant you click to make a new selection the old one disappears. The way it is now it appears that you're making an additional second selection.

If you have any ideas on how to arrange the top bar feel free to suggest, but none of the functions really seem related.

Is the undo checkbox something you really need on the main menu or can that just be a setting? I'm guessing it's for people with not a lot of memory on their computer so they can turn it off, but why is it so highly accessible?
Chunk View vs. Overworld: One button shows the state you're switching to, the other shows the current state. I suggest you make "Chunk View" work like "Overworld" as a dropdown with 2 items, it will be consistent.
I think even doing something as simple as organizing by the visually common elements will improve the readability of this menu.

If we were to change it I'm thinking more like

since that way we can use lesser used functions to break up the UI and make it look less dense.
Also don't forget we're designing under the assumption that it will be used under the default resolution of 1152x864 worst case 1000x680, so we can't add any dead space between buttons. Undecided on the undo checkbox, not sure how often map makers use it if at all.

I know the UI is quite messy :smile:
Efforts have already been made to make it more usable and scalable, create new widgets and enhance existing ones.
The fact is we use an internal UI library based (a long time ago) on A Little Bit of Widgetry...
Efforts will be made to enhance the UI more and more.
I don't take your criticism personally. Sorry if my post let you think so. Using a forein language is not always easy...
In my opinion, a good critic is an reasoned one. Your is of this type; so, welcome :smile:
Concerning the NBT bullets, the Nbtsheet.png file used to display them has been added more than a year ago, almost in the same I added the tree widget...
Thank you to point this issue. We'll have a look at.
By the way, these bullets are displayed when running on Linux.
You may want to have a look to the tool options (right click on the tool icon), and verify that Use bullet style and Use bullet images options are checked on, and the field alongside the Bullet Images File point to Nbtsheet.png (and, of course, this file is in your MCEdit folder).
The Nbtsheet.png comes directly from Minecraft Wiki (excat link to the picture http://hydra-media.cursecdn.com/minecraft.gamepedia.com/d/d0/Nbtsheet.png). It looked like the 'official representation of NBT tags' to me.
If someone wants to create another file, it'll be fine!

(Like the windows registry editor)

Beuark... (IMO)
About the Esc key, it is used to display the main menus, except when you have other windows opened (except the tool options). If you have selected a tool, hitting Esc{ will close this tool, and fall back to the Selectone. When selecting, hittingEsc` cancels the operation.

Chunk View vs. Overworld: One button shows the state you're switching to, the other shows the current state. I suggest you make "Chunk View" work like "Overworld" as a dropdown with 2 items, it will be consistent.

In Minecraft, there are three worlds: Overworld, Nether and The End. The actual solution is the more compact one.

Also my selection/brush shouldn't be moving in the background as I'm mousing over these dialogs, because that makes me think I'm accidentally going to select something or add some blocks when I click on one of the checkboxes etc.

This is not exactly related to ALBOW, but to how MCEdit handles events. And obviously need refactoring.

Just note though this is supposed to be in game editor and not 3d modeler, so the interface will differ.

Right, MCEdit is a save-game editor. No intention to turn it into a clone of Blender on my side :wink:

I think even doing something as simple as organizing by the visually common elements will improve the readability of this menu.

I don't agree. We must mess it up more. It is to much usable.

"If we were to change it I'm thinking more like
untitled since that way we can use lesser used functions to break up the UI and make it look less dense.
Also don't forget we're designing under the assumption that it will be used under the default resolution of 1152x864 worst case 1000x680, so we can't add any dead space between buttons. Undecided on the undo checkbox, not sure how often map makers use it if at all."
Khroki the picture I sent was based on the assumption that some actions would be left alignd and others right aligned. It takes up the same amount of space as today, it's just the empty space in the middle shrinks on smaller screens.

Oh I know, it just feels more cramped to me having all of the buttons pushed together.
Also I knew you'd say that about the icons lachal :P The current icons do the job, I'm not too bothered.

What you have at top serves pretty much the same purpose as a toolbar menu. Maybe they would feel less cramped to you if they looked like menu items instead of buttons?

The show menu has been updated. http://i.imgur.com/9bT7aRs.png

Well, I have one more point concerning the UI: some strings - which are complete sentences - contain a period(full stop), but some don't.  :[


CTA bugfixes.

Changes

We don't want to accidentally float CTAs, since this can cause display issues if the following element doesn't clear floats. Removes unnecessary width property (CTAs are block elements, so already are 100% width).

For review: @DoSomething/front-end

:sheep: it!


Update sites.yml

This PR is:

[x] Adding a new domain
[x] Updating existing domain size
[ ] Changing domain name
[ ] Removing existing domain from list
[ ] Website code changes (512kb.club site)
[ ] Other not listed

[x] I used the uncompressed size of the site
[x] I have included a link to the GTMetrix report
[x] The domain is in the correct alphabetical order
[x] This site is not an ultra lightweight site
[x] The following information is filled identical to the data file

I confirm that I have read the FAQ section, particularly the two red items around minimal pages and inappropriate content and I attest that this site is neither of these things.

[x] Check to confirm

- domain: akgbhat.github.io
  url:https://akgbhat.github.io/
  size: 4.97
  last_checked: 2023-10-06

GTMetrix Report: https://gtmetrix.com/reports/akgbhat.github.io/M2ctw6zw/

Thanks for adding your site! Please use the uncompressed size of your site, as defined in our FAQ.

Updated.

Thanks!


Add priority option

An option should be added to prioritise some tasks and I can work on it.

Don't you think it's a duplicate of #144?
Do you have anything different in mind ?


Codecov integration and badge

Closes #133.
I have never hooked up codecov before. I think this should do the trick.

Is there any reason this won't work for us?

That doesn't appear to be the method that is documented and suggested: https://github.com/codecov/example-node

I was able to remove the extra job from Travis and use one of the existing jobs to report coverage, because I realized that the existing jobs were already each generating coverage data.
I think the reason to still keep the Matrix, instead of a single after_success hook, is because we only need one of the jobs to report coverage, not all of them.

@davidtheclark See https://docs.codecov.io/docs/about-the-codecov-bash-uploader.  Since codecov is a CI specific dependency, I'd prefer just using the bash script and not add a dev-dependency.
If I am being too finicky about this, please go ahead and merge it.

See https://docs.codecov.io/docs/about-the-codecov-bash-uploader.

Ah, thanks! Works like a charm.

My pleasure @davidtheclark!


Add support for pull_request_target

Add support for the new pull_request_target event.
https://docs.github.com/en/actions/reference/events-that-trigger-workflows#pull_request_target

Thanks @robdodson !
I will merge it now.
I just have to run build scripts and commit modified dist/index.js.
Then it will be released as v2.2.1.

Tagged and released as<EMAIL_ADDRESS>

great thank you!


Fix compilation on ARM

Which probably means nobody's tried compiling lodepng on ARM since rust 1.5, yay!
https://github.com/rust-lang/rust/issues/29867

Thanks

Let me know once a new crate is published. Thx.

Done

Great, now little ARM kiddies like me will be able to run some official rust benchmarks :)


Build failing with @purge-icons/generated

In development it's working perfectly fine but while building the build is failing and raising -
[vite-ssg] An internal error occurred.
[vite-ssg] Please report an issue, if none already exists: https://github.com/antfu/vite-ssg/issues
node:internal/errors:464
    ErrorCaptureStackTrace(err);
    ^

Error [ERR_MODULE_NOT_FOUND]: Cannot find module<EMAIL_ADDRESS>imported from C:\Users\Priyam\Documents\vue-learning\.vite-ssg-temp\main.mjs
    at new NodeError (node:internal/errors:371:5)
    at finalizeResolution (node:internal/modules/esm/resolve:416:11)
    at moduleResolve (node:internal/modules/esm/resolve:932:10)
    at defaultResolve (node:internal/modules/esm/resolve:1044:11)
    at ESMLoader.resolve (node:internal/modules/esm/loader:422:30)
    at ESMLoader.getModuleJob (node:internal/modules/esm/loader:222:40)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:76:40)
    at link (node:internal/modules/esm/module_job:75:36) {
  code: 'ERR_MODULE_NOT_FOUND'
}

I have tried to dig into issues but couldn't find any, If I missed anything then please redirect me to the respective issue. Thanks!

After some digging and taking help from Icons website repo, solved it. Seems to be deps error, sigh


Unable to disable telemetry

Describe the bug
Unable to disable telemetry feature
To Reproduce

Using os.environ['DEEPEVAL_TELEMETRY_OPT_OUT'] = "YES" at the top of my script
Running deepeval test run test_xx.py over test cases

Expected behavior

I would expect any telemetry behavior to be disabled. Instead I am still seeing attempted connections (which my system is blocking), see screenshot below

Screenshots

hey @ecatkins, can you quickly print the env variable to see if it is indeed "YES" and resolves to true?

@ecatkins I'm assuming this is no longer an issue?


sessionTimeout in dolphin.properties not working in JBoss AS

Defining the property has no effect

Maybe this is working but JBoss uses a different way to define a timeout. Looks like while a long polling session is on the server no timeout happens.

Once we have an event bus with a timeout this issue will be solved automatically.

Checked it in the customer project and the property is working in JBoss.


MSID: 101302 Version: 1 DOI: 10.1101/2<IP_ADDRESS>6775

MSID: 101302
Version: 1
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>6775
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=101302
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>6775

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/101302v1
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>6775
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/101302/v1/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

error on preview due to missing preprint pub date in DocMap. Added details to spreadsheet

chased

OK to proceed


Add ability to test performance of code fixes with StyleCopTester

Currently StyleCopTester does a great job with performance testing analyzers, but it does not run any fixes. Since batch fixing is the most likely to cause problems in the IDE, it would be nice to have a /fix command line option which applies the fixes for reported diagnostics.

:warning: Make sure to not write the changes to disk during the fix operation. The fixes should be fully applied, but only in memory.

Tentatively assigning to @pdelvo as the likely implementer. :smile:

:warning: Make sure to not write the changes to disk during the fix operation. The fixes should be fully applied, but only in memory.
Means, that the user has to save all documents at the end in Visual Studio? Are all affected documents opened (for review) then, or how could the workflow look like? When will the changed documents be saved?
Second, applying the changes on disk directly is the current implementation in Visual Studio 2015, is that correct? Is that the point, where all the SaveFile dialogs pop up?
Applying in memory sounds great, testing performance with StyleCopTester as well! Please consider undo operations in this context as well!

Please let me back-link to /dotnet/roslyn/issues/2637 [https://github.com/dotnet/roslyn/issues/2637].

@robinsedlaczek StyleCopTester is a tool this team uses to accurately profile the analyzers and code fixes we develop to ensure the best user experience possible. During these profiling passes, it's important that we be able to test the behavior on any number of projects without having to worry about it overwriting the project files on disk. Within Visual Studio, the code fixes will continue to automatically save the results when they are applied to a project or solution.

@sharwell Thanks a lot for your explanation. Got it. :)

Fixed in #1317


[CS] A couple of minor diagnostic improvements

Avoid diagnosing overrides for invalid decls
Fix paren fix-it logic for non-optional weak
Better diagnose conformance failures for EnumElementPatterns

@swift-ci please test


feat(primitives): increase minimum pruning distance

https://github.com/paradigmxyz/reth/blob/8b2ff4f652d7d8e6d678dc2c5eb7c4952af8966d/crates/primitives/src/prune/target.rs#L8-L13

@shekhirin Is this a breaking change and thus we need to update our nodes?

@kaliubuntu0206 I think it's not, but we still need to test it.
The node should run just fine with new minimum pruning distance, and start persisting more blocks in database for the full node. As for the custom configured pruned node, this change may break the existing configuration, but shouldn't require a resync anyway.

Can't select them in the PR, but there are a few comments in the docs saying that pruning distance is 128. should be changed or replaced with the actually var, so we can change it in the future without changing the docs.
regex:
.*run.*128

testing on existing full node


Test on php8

Closing as relies on upstream docker image with test resources which is only built to PHP7.2 and would require PHPUnit to be upgraded. Decide in time whether to maintain this fork or use upstream with patches