Feature: connect_to_server() from a channel

Fixes #200
Add the ability for a user to connect to a running DPF server by providing a channel. This enables advanced scenarios such as DPF running behind a gateway, and a channel with interceptors injecting metadata or credentials. This also allows to address DPF by a DNS name instead of an IP.
This will also be the basis for using DPF with PyPIM (in a further PR)

This will need a different approach to be compatible with future pydpf versions, closing.


Change 424 bus from inner express to local

Summary of changes
Asana Ticket: ❗🚍 Better Bus | Remove 424 from hard-coded list of express routes for Sept 1
This changes the 424 from an inner express bus to a local. I considered making the code changes detailed in the ticket, but on investigation decided that it was more complex than I wanted to do for something with a hard deadline like this.

Assigned to: @meagonqz

@meagonqz Note that @aayesha pointed out that the 448 and 449 are being cut by the Better Bus Project, so on her suggestion I'm removing them from the list too.


Touchables not working on CardB of Card Stack (Calls CardA's Touchables instead)

React Native Version
0.57.1
Description
If any Touchable is included into a Card and stacked on top of one another, Every 2nd Card's Touchable will not work, but instead calls the Touchable of the Card below it.
E.g.
Top
______ <-- Card A
______ <-- Card B
______ <-- Card C
In this scenario, each card (A, B, C) has the same Touchable. At this point pressing the Touchable on the top will trigger onPress() from Card A. Upon removing Card A from the Stack....
Top
______ <-- Card B
______ <-- Card C
Pressing the Touchable on the top at this point will trigger onPress() from Card C, even though the expected behavior is to trigger onPress() from Card B.
Problem
Relevant Issue: https://github.com/facebook/react-native/issues/1076#issuecomment-97923186
The behavior occurs due to positioning of CardA and CardB in CardStack.js. Whichever card being placed below the other in the code will determine the which card's the Touchable is clickable.
In the current code, CardA will be rendered below CardB, allowing CardA's Touchable to be on the top, as shown below:
Top
______ <-- CardA
______ <-- CardB
Once a card is removed, CardB will be rendered on top whereas the next card will be rendered as the new CardA and be placed below CardB according to the zIndex:
Top
______ <-- CardB
______ <-- CardA <-- Touchables take precedence
But since the code still renders CardA below CardB, CardA's Touchables will still take precedence over CardB's Touchables, irregardless of the zIndex value of both cards,
Proposed Solution
Render CardA and CardB sequentially based on the topCard variable in render(). In doing so ensures that each Touchable at the top of the stack can be interacted with.

The problem is completely solved in here.


(Astra DB) Explicit projection when reading from Astra DB

Summary
In view of upcoming changes in the Astra DB Data API, this PR explicitly sets a projection every time a find command is executed, in order to ensure all necessary fields of the document are returned by the API.

Hello @homanp , anything I can do to help completing this PR to merge? :)

Hello @homanp , anything I can do to help completing this PR to merge? :)

Merged! Thanks!


If license file is not provided for I3DRSGM the program crashes on selection

Should the user attempt to select the I3DRSGM from without a license present in the application then the program will crash.

Added license check to make the select unavailable if the license is not found


Export mail download link is not working

Export mail download link is not working
https://ac-397.easycloudbooks.in/sharedfile?guid=397-zmqYOdMKUk6kTJCvHYUNbg


AnimatedSprite animation with .TRES files rockets memory usage!

Godot version:
3.0.6
OS/device including version:
Windows 10 64 bit, Intel(R) HD Graphics 4600
Issue description:
When adding .TRES files to an AnimatedSprite animation, memory usage jumps CONSIDERABLY!
Steps to reproduce:
*Note: These steps, and the zip file included may increase your memory usage an additional ~8GB, which can lock your computer up (as it did mine with many .tres files).

Add TexturePacker plugin to project, and import your .tpsheet and .png spritesheet (2048x2048) created by TexturePacker. This creates the .TRES resource files.

Add an AnimatedSprite to the scene, and then add ~192 .TRES files to an animation.

Observe memory usage for the Godot editor.

Minimal reproduction project:
Bug.zip

It needs to be fixed in 3.1 first before a backport can be considered.

@vspin Nice job isolating it.
Using your project I confirmed this in:  d064b8e0554954af675ca76481388fb4aca4ed0d
From a stopwatch it takes about 46 seconds to launch the scene, and there are editor hang up issues as well that may be related.
On my end at least, using Win10 64-bit, it took a long time to open the project, and at some point after running the main scene and closing it, the editor got hung up for a few minutes and crashed.

I hope this ends up being a trivial thing to fix.  We're definitely going to run head first into this one pretty soon.  In the end I can imagine ourselves having a few dozen atlases and thousands of .tres files.
We've also invested in using Texture Packer, and rely on the plugin to save a lot of time to organize and manage things into TextureAtlas .tres files.  So I'm glad you caught it and it is officially on the 3.1 radar now.  X)

I hope this ends up being a trivial thing to fix.

Well it doesn't seem so easy to me... (but maybe it is for someone else, heh)
This seems to be the function that loads the texture referenced by the external tres files:
https://github.com/godotengine/godot/blob/a2b6be23ada5e7dc6f5815236d7d2b3bb41ab2db/scene/resources/texture.cpp#L463
Maybe it can be changed to load only the relevant rectangular fragment of the texture instead of the whole thing or perhaps to cache the texture?

Maybe it can be changed to load only the relevant rectangular fragment of the texture instead of the whole thing?

Sorry - that would not make any sense. The idea of sprite sheets is to have 1 texture with all the sprites in it to reduce the number of draw calls required for rendering the scene. What you propose is going back to a bunch of single sprites...
Godot has to make sure that the texture is loaded only once and shared across the .tres files.

@avencherus :) Thank you. I was also experiencing hang ups, and crashing as well.
Yeah, I'm really looking forward to 3.1. If your game is multiplayer, you should really have a look at this is well: #16452

@vspin It is an option I will explore soon.  I'm mainly interested in a simple 1 to 1 connection through Steam in the distant future of this project, but I re-designed the game recently to be focused around single player.  If the multiplayer things aren't working out, it'll get cut.
It looks at least this disconnect option you're interested in appears to have been merged already:  https://github.com/godotengine/godot/pull/18070
Apologies for going off topic.  X)

So it is actually possible to achieve the expected result right now...
(I also renamed the addon folder to disable the plugin so it doesn't reimport everything broken again)
Bug_workaround.zip

@piratesephiroth Interesting, seems to load the project up in reasonable time.  Maybe a tweak to the plugin will solve it for now.
Though in the project provided there is a whole mess of other errors when trying to run it.  Perhaps only caused by one texture dependency problem, but haven't had much time to go any further than testing it out.

At runtime on Win10 64-bit (1b63dd8) , it will throw these errors.  Interesting, was that it had a previously resolved socket error.  Maybe some regression has happened.

That's a weird error becuse all tres files load the exact same texture. Maybe some race condition?

It would seem Internal Resources are the culprit based on @piratesephiroth workaround. Maybe someone more astute could have a look at #18801 to see if it shares related issues, beginning at:

It calls VariantWriter::write_to_string (line 1611) for the image property of the ImageTexture. This is where it should write image = SubResource( 1 ), but instead writes image = null and prints

This could explain why the png is loaded with every tres file.

OK, turns out it was a silly problem with the plugin indeed...
It was intentionally denying the caching of the texture by setting the p_no_cache argument of ResourceLoader.load.
https://github.com/CodeAndWeb/texturepacker-godot-plugin/blob/e9912d574e3c624bc545aff39f8a1a0e76326c16/addons/codeandweb.texturepacker/image_loader.gd#L54

Nice, I hope that's all it was.

Nice, I hope that's all it was.

I suppose it is all.
The project loaded fine (took minutes to import all the assets but it did not display any errors)

Thanks for finding this issue. I've updated the importer plugin in the asset library... it's now waiting for approval.
@akien-mga Can you please make the update available to the public? Thanks!

Done :)

@akien-mga Thanks. You rock!

Is this issue fixed?

Is this issue fixed?

yes, it was a tiny little mistake in the plugin


Changing servers per page

I was curious as to how I would go about changing the amount of servers displayed per page. I know some basic PHP and JavaScript so if you point me in the right direction I should be able to figure it out myself.

I have not read the code, but I bet it's SQL-side 'limit', so you should
look for that query.
śr., 15.03.2017, 07:08 użytkownik Ethan Gallant<EMAIL_ADDRESS>napisał:

I was curious as to how I would go about changing the amount of servers
displayed per page. I know some basic PHP and JavaScript so if you point me
in the right direction I should be able to figure it out myself.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Pterodactyl/Panel/issues/349, or mute the thread
https://github.com/notifications/unsubscribe-auth/AFWMcJwoizIhOFWGYMTT5ScfROPoI6-Qks5rl4BZgaJpZM4Mdhh9
.

Here you go.

You actually want this unless you're on a pre release.

Thanks!


Enable Alfresco Tomcat remote debugging

Option to start Tomcat with remote debugging enabled

What benefit is there to enabling remote debugging?  This seems like an option geared towards developers and not necessarily deploying to a server.

@gui81 I totally agree also. There is no real benefit in adding it.


HDFS-15590. namenode fails to start when ordered snapshot deletion feature is disabled

please see https://issues.apache.org/jira/browse/HDFS-15590

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 39s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
34m 34s
trunk passed

+1 :green_heart:
compile
1m 40s
trunk passed with JDK Ubuntu-11.0.8+10-post-Ubuntu-0ubuntu118.04.1

+1 :green_heart:
compile
1m 16s
trunk passed with JDK Private Build-1.8.0_265-8u265-b01-0ubuntu2~18.04-b01

+1 :green_heart:
checkstyle
0m 48s
trunk passed

+1 :green_heart:
mvnsite
1m 21s
trunk passed

+1 :green_heart:
shadedclient
19m 19s
branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 51s
trunk passed with JDK Ubuntu-11.0.8+10-post-Ubuntu-0ubuntu118.04.1

+1 :green_heart:
javadoc
1m 30s
trunk passed with JDK Private Build-1.8.0_265-8u265-b01-0ubuntu2~18.04-b01

+0 :ok:
spotbugs
3m 17s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
3m 15s
trunk passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 14s
the patch passed

+1 :green_heart:
compile
1m 12s
the patch passed with JDK Ubuntu-11.0.8+10-post-Ubuntu-0ubuntu118.04.1

+1 :green_heart:
javac
1m 12s
the patch passed

+1 :green_heart:
compile
1m  2s
the patch passed with JDK Private Build-1.8.0_265-8u265-b01-0ubuntu2~18.04-b01

+1 :green_heart:
javac
1m  2s
the patch passed

-0 :warning:
checkstyle
0m 43s
hadoop-hdfs-project/hadoop-hdfs: The patch generated 1 new + 20 unchanged - 0 fixed = 21 total (was 20)

+1 :green_heart:
mvnsite
1m  8s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedclient
14m  2s
patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 47s
the patch passed with JDK Ubuntu-11.0.8+10-post-Ubuntu-0ubuntu118.04.1

+1 :green_heart:
javadoc
1m 23s
the patch passed with JDK Private Build-1.8.0_265-8u265-b01-0ubuntu2~18.04-b01

+1 :green_heart:
findbugs
3m  6s
the patch passed

_ Other Tests _

-1 :x:
unit
97m 15s
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 42s
The patch does not generate ASF License warnings.

190m 42s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.namenode.TestNameNodeRetryCacheMetrics

hadoop.hdfs.server.namenode.TestDecommissioningStatus

hadoop.hdfs.server.sps.TestExternalStoragePolicySatisfier

hadoop.hdfs.TestSnapshotCommands

hadoop.hdfs.web.TestWebHDFS

hadoop.hdfs.qjournal.server.TestJournalNodeSync

hadoop.hdfs.TestFileChecksumCompositeCrc

hadoop.hdfs.TestFileChecksum

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2326/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2326

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux 1286642aaefc 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 6b5d9e2334b

Default Java
Private Build-1.8.0_265-8u265-b01-0ubuntu2~18.04-b01

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.8+10-post-Ubuntu-0ubuntu118.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_265-8u265-b01-0ubuntu2~18.04-b01

checkstyle
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2326/1/artifact/out/diff-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt

unit
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2326/1/artifact/out/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2326/1/testReport/

Max. process+thread count
4242 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2326/1/console

versions
git=2.17.1 maven=3.6.0 findbugs=4.0.6

Powered by
Apache Yetus 0.13.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

How about failing the second user delete request with a "Already marked as deleted" exception instead of changing the edit log loading? It is hard for editlog loading to guess if the command is valid.

Thanks @szetszwo . The problem is just not with user deletes. Let's see a sequence like this:

Let's say we have 2 snapshots s1 and S2 after enabling ordered snapshot deletion
User deleted S2 ---> creates an edit log entry
User deleted S1
Snapshot Deletion GC thread will now delete S2 ---> creates an edit log entry again
Now turn off Ordered Snapshot Deletion and Restart

It will run into the same problem again. We cannot seem to avoid 2 edit log entries for same snapshot delete with Snapshot GC thread running and deleting the snapshot.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 28s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+1 :green_heart:
@author
0m  1s

The patch does not contain any @author tags.

+1 :green_heart:

0m  0s
test4tests
The patch appears to include 2 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
29m 57s

trunk passed

+1 :green_heart:
compile
1m 17s

trunk passed with JDK Ubuntu-11.0.8+10-post-Ubuntu-0ubuntu118.04.1

+1 :green_heart:
compile
1m 11s

trunk passed with JDK Private Build-1.8.0_265-8u265-b01-0ubuntu2~18.04-b01

+1 :green_heart:
checkstyle
0m 49s

trunk passed

+1 :green_heart:
mvnsite
1m 17s

trunk passed

+1 :green_heart:
shadedclient
16m 55s

branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 54s

trunk passed with JDK Ubuntu-11.0.8+10-post-Ubuntu-0ubuntu118.04.1

+1 :green_heart:
javadoc
1m 26s

trunk passed with JDK Private Build-1.8.0_265-8u265-b01-0ubuntu2~18.04-b01

+0 :ok:
spotbugs
3m 24s

Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
3m 22s

trunk passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m  9s

the patch passed

+1 :green_heart:
compile
1m  8s

the patch passed with JDK Ubuntu-11.0.8+10-post-Ubuntu-0ubuntu118.04.1

+1 :green_heart:
javac
1m  8s

the patch passed

+1 :green_heart:
compile
1m  0s

the patch passed with JDK Private Build-1.8.0_265-8u265-b01-0ubuntu2~18.04-b01

+1 :green_heart:
javac
1m  0s

the patch passed

+1 :green_heart:
checkstyle
0m 40s

the patch passed

+1 :green_heart:
mvnsite
1m  9s

the patch passed

+1 :green_heart:
whitespace
0m  0s

The patch has no whitespace issues.

+1 :green_heart:
shadedclient
15m 23s

patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 50s

the patch passed with JDK Ubuntu-11.0.8+10-post-Ubuntu-0ubuntu118.04.1

+1 :green_heart:
javadoc
1m 26s

the patch passed with JDK Private Build-1.8.0_265-8u265-b01-0ubuntu2~18.04-b01

+1 :green_heart:
findbugs
3m 39s

the patch passed

_ Other Tests _

-1 :x:
unit
104m  4s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 38s

The patch does not generate ASF License warnings.

190m 45s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.datanode.TestDataNodeUUID

hadoop.hdfs.server.datanode.TestBPOfferService

hadoop.hdfs.server.namenode.ha.TestHAAppend

hadoop.hdfs.server.blockmanagement.TestReconstructStripedBlocksWithRackAwareness

hadoop.hdfs.TestFileChecksumCompositeCrc

hadoop.hdfs.server.balancer.TestBalancerRPCDelay

hadoop.hdfs.server.balancer.TestBalancer

hadoop.hdfs.server.namenode.ha.TestPipelinesFailover

hadoop.hdfs.server.datanode.fsdataset.impl.TestLazyPersistReplicaRecovery

hadoop.hdfs.TestFileChecksum

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2326/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2326

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux 9fbfa264c0e2 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 18f9d77c46f

Default Java
Private Build-1.8.0_265-8u265-b01-0ubuntu2~18.04-b01

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.8+10-post-Ubuntu-0ubuntu118.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_265-8u265-b01-0ubuntu2~18.04-b01

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2326/2/testReport/

Max. process+thread count
4173 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2326/2/console

versions
git=2.17.1 maven=3.6.0 findbugs=4.0.6

Powered by
Apache Yetus 0.13.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Thanks @szetszwo for the review. I have committed this.


Mass learn and unlearn

As sometimes its easy to click learn on too many things. I noticed I clicked learn on some hits that should not be hits.
Would it be possible to implement "unlearn / forget" and relearn (if I for example use deepstack and install facebox too, I could click a button and it learns facebox with all the images already learned)

The API currently supports this and may solve your issue. There isn't a way to do it from the UI currently.
/api/train/add/:name will add all images from the/.storage/train folder and /api/train/remove/:name will remove all trained images. This only applies to whatever detectors are currently configured at the time of hitting the API.

Was just going to log this as an enhancement but see it's already here. I was thinking you could add this functionality to the 'Files' view. Implement the multiple select (clicking on the image) that is on the 'Matches' view along with the 'Train' dropdown, but with a new dropdown or tick box called 'Remove' (or something like that).

@Haloooch, I was hoping to implement it similar to how you described, but the ability to remove specific images varies per detection service. CompreFace and Facebox both allow you to remove specific images, but DeepStack only allows you to remove the entire trained person, as far as I can tell.
I may just have to allow the multiselect for CompreFace and Facebox and then just have a single button to remove the entire DeepStack name.
Do you think something like that would work?

I use DeepStack so it's a shame to hear it doesn't allow the per image option, I would however still love the ability to simply click a button and remove a person. As such your suggestion sounds like a great enhancement.
I didn't know at least DeepStack saved all trained images, which you have made visible through the 'Files' menu option. Since DeepStack doesn't let you delete one image that's been trained would there be a way to delete person but then re-train using selected images? Trying to work out the UI workflow:

If no images selected and button clicked to delete then that person deleted
If one or more images selected and button clicked to delete person, person deleted but selected images used to recreate person

(As I type this its all starting to sound quite complex and confusing, not sure if it's worth it....)

I like what you are proposing, I was thinking of a similar solution. It does become a little complicated to have to retrain the entire person each time, but that's probably the only way to really do it in an elegant way.
Maybe I'll do CompreFace and Facebox the same way to just keep it consistent. This would be a killer feature if implemented correctly.

100% agree and part of my hesitance of suggesting it was the different workflow between products. But if you think you can do it in the same way across all 3 then perfect!

I'll try to get this into the next release in some form and we can then tweak it from there.

@Haloooch, good news. I've made some good progress on this over the last few days. Hoping to get a beta build of it ready by end of week.

@Haloooch the newest beta build includes a lot of cool features. I would love some feedback from you and any suggestions you may have. Here's a screenshot of what it looks like so far.

Let me try to walk you through how it is currently setup.
The red trash button on the right side untrains / deletes images from multiple subjects. If the image is currently trained it will remove that subject entirely and retrain it with the other trained images.

If the image is untrained by doing the previous step and you click the same button it will then remove it.

So a trained image goes from trained -> untrained -> deleted
If you select a name from the dropdown you can do one of three things.

Upload a new image from your file system. You can even use your phone camera which will allow you to use some high quality photos for training.
Sync and retrain the model. This deletes the current user from the configured detectors and then takes all the images from the /train/${name} folder to retrain the model.
Ability to completely untrain a user. This will not remove the images, but they will go into the untrained state and then can be selected and deleted if wanted.

I completed extensive testing of this last night, pleased to say it 'just works'! Very cool feature, thank you.
When I first read your instructions above it did sound overly complicated, equally when playing with it, I didn't need to think ok which button do I click on now, but after a few seconds it all became intuitive.
I didn't find a single bug.
I do have one possible enhancement:

Navigate to Train Screen
select a trained image and click delete (to untrain)
now re-select that image once untrained
add the ability to train that image for a different person

If I'm correct you cannot currently do this, all you can do is retrain that image to the originally trained person.
Thank you again, really amazing work.

Hey @Haloooch, I'm so happy to hear it's working well for you. There's a little refactoring I need to do, but I tested it pretty extensively as well. Glad to hear you haven't ran into any issues.
I initially didn't have labels on the buttons so it would scale down to mobile and I wouldn't have to worry about it breaking to a new line. For larger screens I added these so it's more clear what the buttons do and I try to make it clear in the confirmation box what will happen too.
I was able to squeeze your enhancement in tonight as well. Let me know if this is what you had in mind. Here's a screenshot of how it works. For any untrained image you can select a name from the dropdown and train it for that user.
The latest beta build should include this.

Just tested and working perfect! I like the idea of adding labels. Thx.

Closing as this is now included in release v0.9.0.


[bug] GetTimeStamp should return time.Time

Now that type generation uses time.Time instead of string for the timestamps, when revising to another version the timestamp should be updated to reflect the type changes.

Hi there!
I've identified the location in the code where the correction is needed. The GetTimeStamp function is implemented in the timestamp.go file within the utils directory. Currently, it returns a string, but to align with recent type changes, it should return a value of type time.Time instead.
Here's a suggested correction:
import "time"

func GetTimeStamp() time.Time {
    return time.Now()
}


Don't remove app/etc/config.php when installing Magento

Hello,
I'm using Paliarush Vagrant box with a custom Git repo instead of the base Magento Git. This means that the repo will have a set of modules, with the purpose of having Vagrant launch a website will several extensions already installed.
I noticed that in scripts/guest-m-reinstall, the app/etc/config.php is being removed, prior to Magento installation. This means that Magento will regenerate it, overriding any disabled modules or other kinds of versioned information.
status "Removing Magento configuration files (env.php and config.php)"
rm -f "${MAGENTO_ROOT}/app/etc/config.php"
rm -f "${MAGENTO_ROOT}/app/etc/env.php"

I'm aware that app/etc/config.php hasn't been kept in Git by Magento itself, due to some issues they have been having in versions 2.0 and 2.1. However, it looks like they are now officially planning to have this file in Git, and to extend it for more uses:

Another 12 Factor App consideration is to treat configuration as code. It should be under source code version management and propagated as a part of your standard deployment process. For this reason, Magento has formalized that config.php is intended to be committed to git and shared between all environments. env.php overrides settings in config.php and is expected to be local per environment. (Environment variables can also be used to override the env.php file.)

So, in order to prepare for Magento 2.2, and to allow custom Git repos to operate successfully out of the box, I think it would be a good idea to not remove config.php? (or maybe make it configurable?)

Thank you @leoquijano for raising this question. Starting from Magento 2.2, config.php is not under .gitignore anymore. I believe in general it should still be removed during re-installation.
But it definitely makes sense to make this behavior configurable for scenarios like yours. Would you be able to contribute such change in the form of pull request? Option can be added to the magento section of config.yaml

Sure :)
I have to upgrade a website I work on to 2.2, and then I can test a new change and submit the PR.


no badge notification counter

there is no badge notification counter in any service.
I am on macos,
Ferdi 5.5.0

Almost same for me.
Some of my services are having counter notification, some don't (especially Whatsapp).
Environment:
macOS BigSur (11.0.1)
Ferdi (5.5.0)

Hello,
None of my service have notification (Discord, Element, Nextcloud, Mastodon, Dispora, Discourse) :-(
Environment:
Windows 10
Ferdi portable (5.6.0-beta2)

Same for me! :(

Same for me!

Just to be more precise :

Discord have notification, but no counter
Element installed with Riot app has no notification and no counter
Nextcloud installed with Custom app has no notification and no counter
Discourse installed with Custom app has no notification and no counter
Mastodon and Diaspora has no notification and no counter

Hi,

Whats App has notification, but no counter.
Asana has notification, but no counter.
Slack has notification, but no counter.

All Services i use the same as this.
Best regards

not stale!

still there is no badge notification in Mac OS. A lot of message that I was not able to see!

With the latest version of Ferdi and the latest recipes, this should no longer be the case. I have a bunch of services and most of them show the unread counter + notification.

Can you please try to upgrade to the nightlies available here AND then upgrading the recipes by following the steps outlined here?
If you are on macos or linux, I have shared some newer builds of the latest source code here

If there is a service that doesn't show a badge notification counter with the latest nightly from: https://github.com/getferdi/nightlies/releases, please open a new, specific issue for it.
Closing since outdated and cannot reproduce.

Also - in such cases if they occur, the better place to open such a service-specific ticket would be: https://github.com/getferdi/recipes/issues


[0.8] host-context improperly shimmed

If I have an element named x-foo, using :host-context in an external stylesheet gets converted to x-foo-context

:host-context is not supported in 0.8+. Instead you should use Polymer's custom properties shim for this type of cross-scope styling.

@sorvell So what would that look like? I was trying to do something like this:
:host-context(.fancy)

Something like this:
In the scope where you have .fancy:
<style>
  x-foo.fancy {
    --fancy: {
      color: red;
    }
  }
</style>

In the element (e.g x-foo)
:host {
  @apply(--fancy);
}

It's not exactly the same, but instead of the element having to anticipate exactly how it might be styled, it delegates this power, where appropriate, to the user.
Fwiw, we're not totally ruling out adding support for :host-context and will consider user feedback, but we'd like to encourage using properties for these use cases.


Apply basic scheduling to untuned kernels

Apply some basic scheduling optimizations to untuned kernels. This is a temporary workaround to get basic scheduling optimizations (such as AutoInline, Unroll) without spending a lot of time tuning computationally light kernels like softmax, reductions.
Ideally MetaSchedule API should be updated to perform these optimizations without spending extensive amounts of tuning for each kernel.

Merged in #89


Player rejoin causes radio 'catch up'

If there are 3 players in a world and 2 of them are talking through a radio and the third person who isn't leaves the session and rejoins they'll hear everything that has been said through the radio from the point at which they left up until they rejoined.
I have a video of what it looks like although github won't let me post it here

Thanks for the bug report! You can try sending the video to me at<EMAIL_ADDRESS>or on discord at @willnottell
I'm not really sure what could be causing this but I'll try and do some testing to see if I can narrow it down in the meantime.


chore(streams): release<EMAIL_ADDRESS>

To be merged July 19.
The latest API docs: https://jsr.io/<EMAIL_ADDRESS>Closes #5004

@crowlKats PTAL


Pod only contains license and readme

The pods project of ObjectMapper is empty after pod update
it only contains the readme and license file.
i've tried a couple of things:

pod 'ObjectMapper', :git => 'https://github.com/Hearst-DD/ObjectMapper.git'
pod 'ObjectMapper'
pod install

podfile:
# Uncomment this line to define a global platform for your project
platform :ios, '10.0'
# Uncomment this line if you're using Swift
use_frameworks!

target 'BGDD Universal Swift' do

pod 'HockeySDK'
pod 'GooglePlaces'
pod 'Alamofire'
pod 'SwiftDate', '~> 4.0'
pod 'CircleProgressView'
pod 'IBAnimatable'
pod 'ObjectMapper', '~> 2.1'
pod 'SwiftyJSON'
pod 'RealmSwift'

end

in pod spec:
s.source_files = 'ObjectMapper/**/*.swift'
should be:
 s.source_files = 'Sources/**/*.swift'

Thanks for the fix @RickPasveer , I have now merged it into master.
I will make a release soon


Jest coverage targets the wrong file

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
Considering this file tree:

app/services/
├── categories
│   ├── Parser.js
│   ├── __mocks__
│   │   └── Parser.js
│   ├── actions.js
│   ├── constants.js
│   ├── reducer
│   │   ├── data.js
│   │   ├── index.js
│   │   ├── list.js
│   │   └── tests
│   │       ├── data.test.js
│   │       └── list.test.js
│   ├── sagas.js
│   ├── selectors.js
│   └── tests
│       ├── Parser.test.js
│       ├── actions.test.js
│       └── selectors.test.js
├── index.js
├── items
│   ├── Parser.js
│   ├── __mocks__
│   │   └── Parser.js
│   ├── actions.js
│   ├── constants.js
│   ├── reducer
│   │   ├── data.js
│   │   ├── delete.js
│   │   ├── fetch.js
│   │   ├── index.js
│   │   ├── list.js
│   │   ├── next.js
│   │   ├── tests
│   │   │   ├── data.test.js
│   │   │   ├── delete.test.js
│   │   │   ├── fetch.test.js
│   │   │   ├── list.test.js
│   │   │   ├── next.test.js
│   │   │   └── update.test.js
│   │   └── update.js
│   ├── sagas.js
│   ├── selectors.js
│   └── tests
│       ├── Parser.test.js
│       ├── actions.test.js
│       └── selectors.test.js
└── router
    ├── reducer.js
    ├── selectors.js
    └── tests
        ├── reducer.test.js
        └── selectors.test.js

And my coverage when I run the npm run jest -- --coverage app/services/items/reducer/tests/list.test.js:
---------------------------------|----------|----------|----------|----------|----------------|
File                             |  % Stmts | % Branch |  % Funcs |  % Lines |Uncovered Lines |
---------------------------------|----------|----------|----------|----------|----------------|
All files                        |     5.98 |     8.41 |    10.75 |     9.14 |                |
 services/categories             |        0 |        0 |        0 |        0 |                |
  Parser.js                      |        0 |        0 |        0 |        0 |1,3,22,27,28,30 |
  actions.js                     |        0 |      100 |        0 |        0 |... 20,27,34,38 |
  constants.js                   |        0 |      100 |      100 |        0 |         1,7,13 |
  selectors.js                   |        0 |      100 |        0 |        0 |1,7,12,20,23,25 |
 services/categories/reducer     |    33.33 |    45.83 |    33.33 |    42.11 |                |
  data.js                        |        0 |        0 |        0 |        0 |... 26,30,32,34 |
  list.js                        |      100 |      100 |      100 |      100 |                |
 services/items/reducer          |       70 |    80.82 |     87.5 |    73.21 |                |
  data.js                        |      100 |      100 |      100 |      100 |                |
  delete.js                      |      100 |      100 |      100 |      100 |                |
  fetch.js                       |      100 |      100 |      100 |      100 |                |
  list.js                        |        0 |        0 |        0 |        0 |... 40,42,44,46 |
  next.js                        |      100 |      100 |      100 |      100 |                |
  update.js                      |      100 |      100 |      100 |      100 |                |
 services/router                 |        0 |        0 |        0 |        0 |                |
  reducer.js                     |        0 |        0 |        0 |        0 |... 32,39,41,43 |
  selectors.js                   |        0 |      100 |        0 |        0 |1,7,12,20,23,25 |
---------------------------------|----------|----------|----------|----------|----------------|

(I removed some unrelated part of the coverage and file tree).
Jest coverage targets the services/categories/reducer/list file when running the tests on services/categories/reducer/list. I import the right file and it is actually testing the right file, meaning that voluntarily changing something in services/items/reducer/list makes the tests fail.
If the current behavior is a bug, please provide the steps to reproduce and either a repl.it demo through https://repl.it/languages/jest or a minimal repository on GitHub that we can yarn install and yarn test.
This comes from a private repository and I don't think I can give you a runnable example 😞
What is the expected behavior?
The coverage should target the right file.
Please provide your exact Jest configuration and mention your Jest, node, yarn/npm version and operating system.
OS: MacOS Sierra 10.12.5 (16F73)
Node: v7.10.0
Yarn: v0.24.6
Jest: v20.0.4
Jest configuration:
"jest": {
    "collectCoverageFrom": [
      "app/components/**/*.{js,jsx}",
      "app/services/**/*.{js,jsx}",
      "app/utils/**/*.{js,jsx}",
      "app/containers/**/reducer.js",
      "app/containers/**/selectors.js",
      "app/containers/**/actions.js",
      "app/containers/**/constants.js",
      "!app/**/*.test.{js,jsx}",
      "!app/*/RbGenerated*/*.{js,jsx}",
      "!app/app.js",
      "!app/routes.js",
      "!app/themes.js",
      "!app/global-styles.js",
      "!app/**/sagas.js",
      "!app/services/form/intlForm.js",
      "!app/services/**/index.js"
    ],
    "coverageThreshold": {
      "global": {
        "statements": 98,
        "branches": 91,
        "functions": 98,
        "lines": 98
      }
    },
    "moduleDirectories": [
      "node_modules",
      "app"
    ],
    "moduleNameMapper": {
      ".*\\.(css|less|styl|scss|sass)$": "<rootDir>/internals/mocks/cssModule.js",
      ".*\\.(jpg|jpeg|png|gif|eot|otf|webp|svg|ttf|woff|woff2|mp4|webm|wav|mp3|m4a|aac|oga)$": "<rootDir>/internals/mocks/image.js"
    },
    "setupTestFrameworkScriptFile": "<rootDir>/internals/testing/test-bundler.js",
    "testRegex": "tests/.*\\.test\\.js$"
  },

Additional informations:

If I add a simple console.log in my app/services/items/reducer/list files, the coverage now targets it as expected. I initially added one to make sure the right file was loaded and it not only showed up but also corrected the coverage.
Running the same command with --no-cache corrects the issue too.

Hey @dispix,
Can you test if this fix resolves your case? https://github.com/facebook/jest/pull/4432

Hi @stipsan !
I was about to test it but got an error when trying to target the master branch from this repository:
devDependencies: {
  "jest-cli": "facebook/jest#master"
}

error /.../node_modules/jest-cli: Command failed.
Exit code: 1
Command: sh
Arguments: -c yarn run build --silent
Directory: /.../node_modules/jest-cli
Output:
module.js:487
    throw err;
    ^

Error: Cannot find module 'string-length'
    at Function.Module._resolveFilename (module.js:485:15)
    at Function.Module._load (module.js:437:25)
    at Module.require (module.js:513:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (/.../node_modules/jest-cli/scripts/build.js:32:22)
    at Module._compile (module.js:569:30)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:503:32)
    at tryModuleLoad (module.js:466:12)
    at Function.Module._load (module.js:458:3)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.

Am I missing something with the way I should target a git repository as a dependency ?

Like @SimenB says the easiest way to test it is to:

Create your local clone.
Run yarn install in the clone (using yarn v0.27.5 or later as jest is using the workspaces feature).
npm link to compile jest so you get the same files available as when you yarn add jest-cli right off npm.
cd to your project and npm link jest-cli, this should setup a symlink for you to jest.

Just tried following the steps to link a local version of jest and this doesn't seem to fix the issue 😞
Using<EMAIL_ADDRESS>and the master branch of this repository.

No need to link, just clone, run yarn and then in your other project, do ../jest/jest

Doesn't change a thing though, both methods still display the wrong coverage.

Very strange. It might still be loading the old packages for babel-jest and jest-runtime since you get the same results.
Any chance you can create a repo that reproduces the problem? It'll help me check for sure if #4432 will fix it, if it doesn't I'll be able to work on a fix using your test case. :)

After upgrading to<EMAIL_ADDRESS>and<EMAIL_ADDRESS>I am no longer experiencing this issue.

I'm having a similar issue with Jest generating random or nonsensical coverage data in #4960


PS  read line

Type: Feature Request
where i have to enable PS read line
VS Code version: Code 1.71.2 (74b1f979648cc44d385a2286793c226e611f59e7, 2022-09-14T21:03:37.738Z)
OS version: Windows_NT x64 10.0.19044
Modes:
Sandboxed: No

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Adding Spanish language for error handling

[X] Added Spanish language file to languages directory

Thanks! :+1:


[Multisend][RPC] Optimisations for MultiSend RPC Functionality

What is the purpose of this pull request (PR)?
[Multisend][RPC] Optimisiations for MultiSend RPC Functionality
Any background context to help the reviewer?
https://github.com/PIVX-Project/PIVX/pull/68/files
Was this PR tested and how?
On Windows 10, compiled using UNIX Subsystem
More Questions:

Does the knowledge base need an update? No

Does this PR add or change dependencies? No

Does this PR change or fork the blockchain  No

Currently causes wallet to become unresponsive, please investigate @plaxton

Okay


Suggestion for Workbook Update Connection

Per #148, an option for updating workbook connections. Having to iterate over each connection individually is sub-optimal, but core REST API only allows one connection update at a time.
This only accounts for connection string updates and not ports, username, or password changes.
Use case is DB migration.
Also - I'm not sure why the two HTML files were included. No edits were made after the fork.

Changes made to reflect @t8y8 comments

Current fixes complete

Looks good to me. At some point we might rethink how our "update" works. It is supposed to figure out what needs to be updated (based on anything that has changed) but it has gotten lax. Thus you need updateConn. If not, you might just call "update".

Thanks!!

I edited your commit messages and merged. Congrats on your first accepted PR, @cmtoomey !!

Yay - thanks @t8y8...now just to wait until it ships


Routes ending with -ves search for models ending in -fe

Steps to reproduce:

Clone https://github.com/cerebris/peeps, bundle, rake, migrate
Add a line such as jsonapi_resources :drives to config/routes.rb (notice the ending, driVES)
rails s

(I did this on an already existing project with the correct resources and controller files. But the peeps project is enough to demonstrate the search error)
Error:
jsonapi-resources-0.7.0/lib/jsonapi/resource.rb:310:in `resource_for':
JSONAPI: Could not find resource 'drives'.
(Class DrifeResource not found) (NameError)

(Notice that for the resource 'drives' jsonapi-resources is looking for 'DrifeResource')

JR uses ActiveSupport::Inflector. I'm guessing this is related to the inflection rules for your system. Could you try the test of
"drive".pluralize

from somewhere in your project. My guess is it will pluralize to drife.

@lgebhardt is right on there. It's bizarre that it convert drives to drife. Add this to config/initializers/inflections.rb to correct the inflection of "drives" and anything else you need to fix.
ActiveSupport::Inflector.inflections(:en) do |inflect|
  inflect.singular /^(drive)s$/i, '\1'
end

Yep, that seems to be the problem. The thing is, there isn't anything specific to my project that would cause such an error. (inflections.rb is empty, too). Looks like this could be a ActiveSupport::Inflector bug then? Bizarre indeed!

I was just browsing the source for the ActiveSupport inflectors and the comment at the heading of it says the list is not complete and it is frozen. It's intended as a starting point for applications and the developer should override as necessary. They won't accept any PRs or fixes because it could break existing applications that rely on incorrect inflections. Although that's what versioning and tests are for... 🙄

Yes, I just came across the same thing - crazy. I'm new to Rails and this really confused me for a bit. I figured that it was some pluralisation rule that was causing the problem, but for a word like 'drive', it is quite surprising! I opened this issue here because I didn't think that a project the size of Rails could possibly have such a bug for a normal word, but I guess I was wrong..
Thanks 👍


Ability to modify isOptedInToDistributeIosAppOnMacAppStore is no longer working

New Issue Checklist

[ x] Updated fastlane to the latest version
[ x] I read the Contribution Guidelines
[ x] I read docs.fastlane.tools
[ x] I searched for existing GitHub issues

Issue Description

Ability to modify isOptedInToDistributeIosAppOnMacAppStore is no longer working. It was working 2 weeks ago, but not now with same code.
https://github.com/fastlane/fastlane/issues/16868
Command executed

Code:
app_from_connect_api = Spaceship::ConnectAPI::App.find("#{app.bundle_id}")
UI.message "GOT app from connect API, setting is_opted_in_to_distribute_ios_app_on_mac_app_store: false"
puts app_from_connect_api.is_opted_in_to_distribute_ios_app_on_mac_app_store  #gives null response
puts app_from_connect_api.name
puts app_from_connect_api.bundle_id
app_from_connect_api.update(attributes: {
is_opted_in_to_distribute_ios_app_on_mac_app_store: false
})
Complete output when running fastlane, including the stack trace and command used

 Error: The provided entity includes an unknown attribute - 'isOptedInToDistributeIosAppOnMacAppStore' is not an attribute on the resource 'apps'

Environment

 [ruby-2.4.5/gems/fastlane-2.169.0/spaceship/lib/spaceship/connect_api/api_client.rb:193:in `handle_response': [!] The provided entity includes an unknown attribute - 'isOptedInToDistributeIosAppOnMacAppStore' is not an attribute on the resource 'apps' (Spaceship::UnexpectedResponse)
	from /Users/isdr/.rvm/gems/ruby-2.4.5/gems/fastlane-2.169.0/spaceship/lib/spaceship/connect_api/api_client.rb:135:in `patch'
	from /Users/isdr/.rvm/gems/ruby-2.4.5/gems/fastlane-2.169.0/spaceship/lib/spaceship/connect_api/tunes/tunes.rb:200:in `patch_app'
	from /Users/isdr/.rvm/gems/ruby-2.4.5/gems/fastlane-2.169.0/spaceship/lib/spaceship/connect_api/models/app.rb:106:in `update'
	from Fastfile:700:in `block (3 levels) in parsing_binding'
	from Fastfile:691:in `collect'
	from Fastfile:691:in `block (2 levels) in parsing_binding'
	from Fastfile:683:in `each'
	from Fastfile:683:in `block in parsing_binding'
	from /Users/isdr/.rvm/gems/ruby-2.4.5/gems/fastlane-2.169.0/fastlane/lib/fastlane/lane.rb:33:in `call'
	from /Users/isdr/.rvm/gems/ruby-2.4.5/gems/fastlane-2.169.0/fastlane/lib/fastlane/runner.rb:49:in `block in execute'
	from /Users/isdr/.rvm/gems/ruby-2.4.5/gems/fastlane-2.169.0/fastlane/lib/fastlane/runner.rb:45:in `chdir'
	from /Users/isdr/.rvm/gems/ruby-2.4.5/gems/fastlane-2.169.0/fastlane/lib/fastlane/runner.rb:45:in `execute'
	from /Users/isdr/.rvm/gems/ruby-2.4.5/gems/fastlane-2.169.0/fastlane/lib/fastlane/lane_manager.rb:47:in `cruise_lane'
	from /Users/isdr/.rvm/gems/ruby-2.4.5/gems/fastlane-2.169.0/fastlane/lib/fastlane/command_line_handler.rb:36:in `handle'
	from /Users/isdr/.rvm/gems/ruby-2.4.5/gems/fastlane-2.169.0/fastlane/lib/fastlane/commands_generator.rb:108:in `block (2 levels) in run'
	from /Users/isdr/.rvm/gems/ruby-2.4.5/gems/commander-fastlane-4.4.6/lib/commander/command.rb:178:in `call'
	from /Users/isdr/.rvm/gems/ruby-2.4.5/gems/commander-fastlane-4.4.6/lib/commander/command.rb:153:in `run'
	from /Users/isdr/.rvm/gems/ruby-2.4.5/gems/commander-fastlane-4.4.6/lib/commander/runner.rb:476:in `run_active_command'
	from /Users/isdr/.rvm/gems/ruby-2.4.5/gems/fastlane-2.169.0/fastlane_core/lib/fastlane_core/ui/fastlane_runner.rb:76:in `run!'
	from /Users/isdr/.rvm/gems/ruby-2.4.5/gems/commander-fastlane-4.4.6/lib/commander/delegates.rb:15:in `run!'
	from /Users/isdr/.rvm/gems/ruby-2.4.5/gems/fastlane-2.169.0/fastlane/lib/fastlane/commands_generator.rb:352:in `run'
	from /Users/isdr/.rvm/gems/ruby-2.4.5/gems/fastlane-2.169.0/fastlane/lib/fastlane/commands_generator.rb:41:in `start'
	from /Users/isdr/.rvm/gems/ruby-2.4.5/gems/fastlane-2.169.0/fastlane/lib/fastlane/cli_tools_distributor.rb:119:in `take_off'
	from /Users/isdr/.rvm/gems/ruby-2.4.5/gems/fastlane-2.169.0/bin/fastlane:23:in `'
	from /Users/isdr/.rvm/gems/ruby-2.4.5/bin/fastlane:23:in `load'
	from /Users/isdr/.rvm/gems/ruby-2.4.5/bin/fastlane:23:in `'
	from /Users/isdr/.rvm/gems/ruby-2.4.5/bin/ruby_executable_hooks:24:in `eval'
	from /Users/isdr/.rvm/gems/ruby-2.4.5/bin/ruby_executable_hooks:24:in `'

Having the same problem, it was working for me just a few days ago. The code hasn't changed.

Yup, i've been getting the exact same error too

Any update on this? can we turn off availability of the app on Mac Store (Apple Silicon) some how from fastlane?

The same here:
#<Spaceship::UnexpectedResponse: The provided entity includes an unknown attribute - 'isOptedInToDistributeIosAppOnMacAppStore' is not an attribute on the resource 'apps'>

Hi,
Do we have any updates on this?

An update would be appreciated

An update would be appreciated

It seems that Apple silently dropped support to this attribute 🧐 I also couldn't find any similar API to perform the same action 😞

It seems that Apple silently dropped support to this attribute 🧐 I also couldn't find any similar API to perform the same action 😞

:'(

:'(

Hi, I am having the same issue and was wondering if there have been any updates. Thanks.


Documentation

Docs

[x] Getting started
[x] Installing
[x] Updating
[x] FAQ

Themes

[x] Overview
[x] Templates
[x] Helpers
[x] Filters
[x] Globals

API

[ ] Overview
[ ] Auth
[ ] Endpoints

Is there anything else you'd like to see in the docs?

Yes, I'd like to see #66 please.


Symbol not found: _clock_gettime

Hey!
Running rails server gives me following error, any suggestion?
dyld: lazy symbol binding failed: Symbol not found: _clock_gettime
  Referenced from: /usr/local/opt/rbenv/versions/2.2.5/lib/ruby/gems/2.2.0/extensions/x86_64-darwin-15/2.2.0-static/eventmachine-<IP_ADDRESS>/rubyeventmachine.bundle (which was built for Mac OS X 10.12)
  Expected in: /usr/lib/libSystem.B.dylib

dyld: Symbol not found: _clock_gettime
  Referenced from: /usr/local/opt/rbenv/versions/2.2.5/lib/ruby/gems/2.2.0/extensions/x86_64-darwin-15/2.2.0-static/eventmachine-<IP_ADDRESS>/rubyeventmachine.bundle (which was built for Mac OS X 10.12)
  Expected in: /usr/lib/libSystem.B.dylib

Here's the end of that "Referenced from"

eventmachine-<IP_ADDRESS>/rubyeventmachine.bundle (which was built for Mac OS X 10.12)

Which makes me think it's a missing dependency for EventMachine (this gem uses EventMachine for a websocket server).
Are you on OSX? Here's a similar bug report where a package from homebrew failed with the same error. It's a bit of a hack, but you could try updating your homebrew, then installing the package mentioned there. Maybe that would cause it to install the missing dependency!

Yey, thanks! [HotLoader] connected 👍
However, is there a way to change the default assets folder ?

does reload JS assets when they change (from /app/assets/javascripts/.{js,coffee})

I tried this
# config/initializers/react_rails_hot_loader.rb
React::Rails::HotLoader::AssetChangeSet.asset_glob = "**/*.{js,rb}*" # I <3 Opal

but still no reload.
Thanks 🙏

You can see here where app/assets/javascripts is provided:
https://github.com/rmosolgo/react-rails-hot-loader/blob/master/lib/hot_loader/asset_change_set.rb#L14
I don't think it's configurable (yet!) but I'm open to a patch if you'd like to improve that :)

Haha ok, thanks for your help 🙌


Define ReadableAirdropStore and WritableAirdropStore interfaces and provide implementations for them

Add the following classes in hedera-token-service-impl:
ReadableAirdropStore interface
ReadableAirdropStoreImpl that would implement ReadableAirdropStore, holding ReadableKVState<PendingAirdropId, PendingAirdropValue> airdropsState as a field and implement the interface's methods
WritableAirdropStore that would extend ReadableAirdropStoreImpl and implement the put and remove methods
When implementing the put method - if there is already an existing airdrop for a user make sure to sub the tokens for that user. E.g. user A sends user B 10 tokens. Then it sends another 5 - the total tokens should be 15.
Define the new readable store in ReadableStoreFactory
Define the new writable store in WritableStoreFactory

For more context read: https://github.com/hashgraph/hedera-services/pull/12792

ticket closed as its now tracked under the correct repo as https://github.com/hashgraph/hedera-services/issues/13938

ticket closed as its now tracked under the correct repo as https://github.com/hashgraph/hedera-services/issues/13938


GitHub tags missing

c.f. https://github.com/ryanmcgrath/twython/issues/354 , tags are missing from v3.3 to current v3.7

ping @ryanmcgrath .
https://build.opensuse.org/request/show/673135 is sort of waiting for this.

Sup.
I don't have time for this library right now, and I'm pretty sure @michaelhelmick is busy with his own life stuff. I'll ping Mike about contributors again, but concurrently, if my glancing over your linked thread is correct, what's wrong with just following master for now?

Takes only a few minutes to add a tag for the most recent release.
That would suffice for my needs.

Sup.
Protip: "takes only a few minutes" is condescending and kinda unwarranted, and your comments in that openSUSE thread also come across as the same. People have lives outside of GitHub and software, and what might seem like only a few minutes can be a completely unnecessary distraction to someone else.
This'll get done whenever another maintainer has time. I'll poke Mike about new ones again.

A new tag has been released, sorry for the wait @jayvdb
https://github.com/ryanmcgrath/twython/releases/tag/3.7.0

Thanks @michaelhelmick 	.  Package resubmitted at https://build.opensuse.org/request/show/689955 .  Hopefully I have better luck this time, so I can move on to fixing the nltk package which uses twython.


fixed a typo

- I fixed a typographical mistake, because, if err == nil, then there is no point of returning anything
- I read the whole code and found that statement fishy, so did the  change, because if there is actually any error the the program execution must stop
- How to verify it
It can be verified by deliberately creating an error
- Description for the changelog
Fixed a typographical mistake, which previously has no effect on the execution of the code. Now, the execution halts on error, and returns the error
- A picture of a cute animal (not mandatory but encouraged)

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "master" git@github.com:chawlanikhil24/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

Sorry I'll have to close this PR because this isn't correct, but thanks so much for contributing, and I hope my explanation makes it a bit clearer.
Feel free to continue the discussion :+1:

Thank you for the information, I assume that , the code is skipping the rest of the code if there's  no error. This statement drew my attention because, majority of the GO code I've seen returns err, if err!=nil , In this case, your suggestion of return nil is actually clearer than return err, in case of err==nil. If we change it to return nil , will cause any problem?

return nil should work

I've replaced "err" will "nil" and even added a comment in the code , for no such conflicts in future. Should I make another PR?

Yes please


part2/Properties: mention that Canonical N ⦂ A ≃ Value N × ∅ ⊢ N ⦂ A

Chapter part2/Properties introduces canonical forms; it is immediately mentioned how those represent well-typed closed values, with a proof of the equivalence split up into three distinct functions.
It turns out that this correspondence can be extended to an isomorphism: Canonical N ⦂ A ≃ Value N × ∅ ⊢ N ⦂ A.
Since the book points out isomorphisms between types very often, it seems a bit strange that it doesn't do that in this case, and I think this might lead the reader to suspect that the two concepts are not completely interchangeable.
I wonder if it would be a good idea to mention the isomorphism explicitly in the book, possibly with an exercise that asks the reader to prove it (maybe reusing the canonical, typed and value functions).

Another doubt I have which is tangentially related is about the use of the canonical function in the proof of progress: is there a particular reason to use that function, given that pattern matching on the Value returned by the recursive call to progress would be enough, and actually spare a nested with abstraction?
The code without using canonical would look like this:
progress : ∀ {M A} → ∅ ⊢ M ⦂ A → Progress M
progress (⊢ƛ _) = done V-ƛ
progress (⊢L · ⊢M) with progress ⊢L
... | step L—→L′ = step (ξ-·₁ L—→L′)
... | done V-ƛ with progress ⊢M
...   | step M—→M′ = step (ξ-·₂ V-ƛ M—→M′)
...   | done V-M = step (β-ƛ V-M)
progress ⊢zero = done V-zero
progress (⊢suc ⊢M) with progress ⊢M
... | step M—→M′ = step (ξ-suc M—→M′)
... | done V-M = done (V-suc V-M)
progress (⊢case ⊢L ⊢M ⊢N) with progress ⊢L
... | step L—→L′ = step (ξ-case L—→L′)
... | done V-zero = step β-zero
... | done (V-suc V-M) = step (β-suc V-M)
progress (⊢μ ⊢M) = step β-μ

Nice observation regarding the isomorphism. Please prove it and enter the answer into the answers repository, then edit the chapter accordingly with proof of the isomorphism added as an exercise. (I'd be reluctant to state it is an isomorphism unless someone actually proved it.)
Regarding progress, did you run that code? When I omit the call to canonical from the code in the book it yields a type error.

Thanks for the response @wadler.
About the isomorphism, I'd be happy to contribute with a solution, but I still don't have access answers repository, and neither do I know where to ask for it 😅.
Regarding progress, I did type-check the code on my setup; one thing to note is that I didn't just omit the call to the canonical lemma, but I also explicitly pattern matched on the evidence that some terms are Values, which the implementation in the book doesn't do:
-- Line from the code above:
... | done V-ƛ with progress ⊢M
-- Corresponding line on the book
... | done VL with progress ⊢M

-- Lines from the code above:
... | done V-zero = step β-zero
... | done (V-suc V-M) = step (β-suc V-M)
-- Corresponding lines in the book:
... | done VL with canonical ⊢L VL
...   | C-zero =  step β-zero
...   | C-suc CL = step (β-suc (value CL))

This is consistent with the fact that Canonical N ⦂ A contains the same information as Value N and ∅ ⊢ N ⦂ A: pattern matching on the parameters of the latter types should have the same effect, without having to construct (redundant) evidence for Canonical N ⦂ A.
Another thing I should point out is that I discovered just yesterday that my version of Agda is a little more recent than the one suggested for the book, which is also a possible explanation for the problem.

@wadler I am now running Agda version <IP_ADDRESS>, which is the one suggested for the book, and the implementation of progress from above still type-checks.

Cool! You are absolutely right. I thought canonical forms was essential, but it is not. I have simplified the proof along the lines you suggest, deleted the section on canonical forms, and added an exercise that defines canonical forms and asks the reader to show that being a canonical form is isomorphic to being well-typed and a value.
I've also invited you to plfa-answers. Thank you!

Thanks @wenkokke for reopening the issue, and thanks @wadler for your response. I'll be happy to submit a solution to the new exercise later this week.
@wadler I am happy with all the changes you have implemented, but I'm worried that a little problem may have gone unnoticed: canonical forms and the proof of progress happen to be mentioned in the next chapter of book (possibly even in later ones?), and those references should probably be updated.
Consider, for example, the following paragraph from the chapter DeBruijn, section progress:

The statement and proof of progress is much as before, appropriately annotated. We no longer need to explicitly refer to the Canonical Forms lemma, since it is built-in to the definition of value:

As another example, the line count in section intrinsic typing is golden may need to be updated.

Ah, yes, that should perhaps be automagically generated.


Video Challenges show 0 brownie points

If I go to any Video Challenge, I see the following:

The jobs board also has the same bug.

I am only adding this, as it might be of help to whomever can fix this, but here is the two different  tags and their content from a regular challenge and a video challenge:

[ 307 ]
<li role="presentation" data-reactid=".11po48uf4<IP_ADDRESS>.0.$5/=11"><a href="/robeerob" class="brownie-points-nav" data-reactid=".11po48uf4<IP_ADDRESS>.0.$5/=11.0"><span data-reactid=".11po48uf4<IP_ADDRESS>.0.$5/=11.0.0">[ <span data-reactid=".11po48uf4<IP_ADDRESS>.0.$5/=11.0.1">0<span data-reactid=".11po48uf4<IP_ADDRESS>.0.$5/=11.0.2"> ]

My best guess is that that spedicific  element in the nav should be stripped down and be given the "brownie-points-nav" class, so that some .js file can work it's js or jquery magic on it.


kepler_orbiters build on macOS fails due to OpenMP

Following up on #266, kepler_orbiters cannot be built on macOS due to the -fopenmp flag which adds -lomp (OpenMP library). OpenMP is not officially supported on macOS (there are ways to do it, but they are hacky and non-robust), so the -fopenmp flag should not be used for macOS builds.

That's with the Xcode compilers, right? I believe with gcc OpenMP works fine?

Yes, you're right, so it only needs to be disabled when using Apple clang. This is, however, the default compiler, so I think it still makes sense to fix it. Using gcc on macOS in my experience has always been and remains to be a hassle; Macports and Homebrew are great in their own ways, but they also introduce some new problems that I personally started preferring to avoid lately. Conda also uses Apple clang by default, and since that is the way I'm currently building on macOS, that was how I got to this point.

Fixing the dependency is of course a great idea, but let's also check if the compiler supports OpenMP and enable it again if it does.

Note that configure checks for openmp and that we have special flags for openmpi inclusion. Probably the kepler_orbiters flag do not use these flags.

Indeed, kepler_orbiters' Makefile hard-codes the -fopenmp flag: https://github.com/amusecode/amuse/blob/master/src/amuse/community/kepler_orbiters/Makefile#L11
So probably that's the only thing to fix then?


DB Scrubber Node.js utility update

feat(scrubber): Updates Node.js MongoDB scrubber utility application
Adds exporter and importer modules that use mongodump and mongorestore, respectively.
Updates code to ES6 syntax.
Expands ENV parameter requirements to account for changes in the updated scrubbing workflow.
Updates README with new instructions.

2 things

Not sure if the version of node we are using in the project supports es6 syntax, I remember trying it at one point and having issues.
For emails we wanted to generate gmail emails that will all end up at the same inbox - there is a pattern to follow for this (can't remember exactly but something like<EMAIL_ADDRESS><EMAIL_ADDRESS>etc). this would allow us to perform end-to-end testing including receipt of notification emails

From what I spoke to with @agehlers. The exporter/importer container environment (which this code would be housed in) will have Node.js 4+ so we should be fine with es6 syntax. I also tested in the containers she provided for development and they worked ok (I guess that's the advantage of having these encapsulated environments).

I can update the scrubber so that instead of simply removing the emails, it replaces them with whichever pattern is suitable for notification testing. At the time I wasn't sure which approach we wanted to go with so I just cleaned them from the data but it is easy to replace them instead.

Since @amanji reviewed this issue with @agehlers, let's leave it the way it is.  If @agehlers runs into difficulty when testing in Production, we can modify that.

@ccoldwell - where do we find the pattern for replacing the emails?  Who do we need to talk to about that?

@Akiff - @ccoldwell will give you express instructions on what to do.  What we want to do is to create a devex Gmail account that we can use to test the Notifications module.

@amanji  - @ccoldwell will give you express instructions on what to do. What we want to do is to create a devex Gmail account that we can use to test the Notifications module.

Let's merge this now, as @agehlers is ready to set up the Staging environment for the app and requires @amanji's utilities.  After this PR goes through, we will talk about setting up the Gmail notifications properly.


Allow the c_foxbin2prg class to be more COM compatible.

Hello Fernando,
I wanted to make processing the foxbin2prg over 7000 times for one of our projects a little less annoying.
FoxPro executables have a quirk. The simplest one that just quits right after it starts, even with SCREEN=OFF in an embedded CONFIG.FPW file will steal the focus, then let it go again.
This leaves the computer unusable for an hour as it processes all the commands I'm running.
Background: I'm converting VSS to Git. Running a VSS command to get files of a specific version from VSS, if it needs to be converted, run the FoxBin2Prg, then run a Git command to stage the appropriate file. Then when the comment changes from VSS, commit the changes to Git.
I wrote a small COM DLL that uses the EXE as source for the c_foxbin2prg class. Set the l_CancelWithEscKey = .F. and n_ShowProgressBar = 0. Then call the Execute method.
I found there are a few commands in the FoxBin2Prg that don't work in the COM environment. ON ESCAPE and MESSAGEBOX. Setting the l_CancelWithEscKey needs a change in the class to skip the ON ESCAPE. llEscKeyRestored is .T. when it reaches the Finally clause to avoid the ON ESCAPE in there.

Until I looked a the size of this pull request, I didn't realize how many lines changed due to code page differences. I'm closing this request, I'll open an issue explaining what I changed.


Vendoring libnetwork 3e31cead05cba8ec20241630d051e6d73765b3a2

Fix a couple of bugs introduced by previous libnetwork vendoring:

in bitseq which prevents to use experimental overlay networking
in docker service ls cli o/p

Add missing http subrouter for newly introduced sandboxes
Fix fragmentation issue on vxlan header addition for overlay network driver
Remove libnetwork test code utilities from vendoring

Relevant PRs:
Move test specific functions to a testutils package. #499
Fix bug for docker service ls #498
Fix bitsequence set() #496
For the endpoints on overlay network set the MTU to 1450 to avoid fra… #491
Do not import testing in the main binary. #484
Signed-off-by: Alessandro Boch<EMAIL_ADDRESS>

LGTM

@aboch as suggested in https://github.com/docker/libnetwork/pull/501, can you please add a sub-router hook for the newly added sandboxes api in https://github.com/docker/docker/blob/master/api/server/server_experimental_unix.go ?
There is a few changes coming to these shortly. But in order to keep the existing experimental features working, i think we should have this fixed now.

@mavenugo Thanks, I will add it shortly.

Taken care of adding sandboxes route to http sub-router.
PTAL when you get a chance. Thanks.

LGTM @mavenugo feel free to merge if you approve

Thanks @aboch. LGTM.


Received a 'math/big: cannot unmarshal ...  into a *big.Int: tx parse  error' error.

Summary of Bug

Version

Steps to Reproduce

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

fixed! thank you for your report👍

Summary of Bug
Version
Steps to Reproduce
For Admin Use

[ ]  Not duplicate issue
[ ]  Appropriate labels applied
[ ]  Appropriate contributors tagged
[ ]  Contributor assigned/self-assigned

Hi, thank you and congratulation for becoming the bug bounty prize winner for the Gravity DEX Competition! Please reply here with your Cosmos Hub mainnet address to receive the prize, by your github ID, until May 21, 23:59.

Hey team!,
OMG this is so amazing.Thank you all. I am extremely happy to be a part of this community and supporting since the white paper days. I truly believe this is what’s going to reinvent the way we deal with assets management on every level of our society. I am a university physics student and the implications for the future excite me beyond anything else besides the expansion of our species to the actual cosmos. This is honestly revolutionary and you all should be extremely happy and proud of yourselves. I would love to be an official cosmonaut and it’s a blessing to even play a part. Thank you again.
My main hub keplr address is:
cosmos1yera5r8ancdhedeyqcumjwgpxe53au95aglgqr
Is there anything else you’ll need?
James Dennis
@.***

On May 19, 2021, at 2:23 AM, Lim Sung-hoon @.***> wrote:
Summary of Bug
Version
Steps to Reproduce
For Admin Use
Not duplicate issue
Appropriate labels applied
Appropriate contributors tagged
Contributor assigned/self-assigned
Hi, thank you and congratulation for becoming the bug bounty prize winner for the Gravity DEX Competition! Please reply here with your Cosmos Hub mainnet address to receive the prize, by your github ID, until May 21, 23:59.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/tendermint/liquidity/issues/245#issuecomment-843918789, or unsubscribe https://github.com/notifications/unsubscribe-auth/AS5JCRDR6YQEHAHGYEJ657TTON7SPANCNFSM44GB47IQ.

Hey Maydany, just following up. Just hope I don't mess anything up and not do something else thats needed.
Thanks
James


fix: replication dropdown max height + minor css

fixes: https://issues.redhat.com/browse/DFMS-209
along with the replication dropdown raised by the QA team.
before

(no padding as u can see)

after

/assign @SanjalKatiyar

@debjyoti-pandit please add before/after screenshots.

@debjyoti-pandit please add before/after screenshots.

Done @SanjalKatiyar

/test mcg-ms-console-e2e-aws

/test mcg-ms-console-e2e-aws

/test mcg-ms-console-e2e-aws


Update dependency tsconfig-paths

A high severity vulnerability in JSON5 was discovered.
tsconfig-paths relies on this package.
Fortunately, a fix was created for v4.1.2.
Request:
Update dependency
"tsconfig-paths": "^3.14.1"

To
"tsconfig-paths": "^4.1.2"

Bump. This is very annoying for me personally because the place I work at, this dependency is actually pretty much at the bottom of our dependency hierarchy so it's impossible to get rid of, and also because npm audit lacks a feature to disable specific packages from auditing. The only way for us to currently work around this is to literally change our automatic audits to --audit-level critical or --omit=dev, neither of which is good or optimal.

You can use the eslint-plugins-i package

Omitting CVE is a bad idea, because you could still trigger the CVE in JSON5 with a different route. So if an other package would depend on json5 the CVE could be very real, although through this package it will not be triggered. So only for this dependency tree it is a false positive.
Also since a few days the CVE is fixed in the v1 of json5 but the CVE db is not yet updated so it is now a real false positive.

You can use the eslint-plugins-i package

We can't, because we're also using eslint-config-airbnb-base which peer-depends on this package specifically.

I have created a fix. Can someone review and merge it?

@Teascade I use .npmrc with legacy-peer-deps=true to get around this issue. In my package.json I have: "eslint-plugin-import"<EMAIL_ADDRESS>

@desaisagar The maintainer of this package unfortunatly does not consider this a bug and will not merge fixes for it. (See all the other related and closed tickets)

Okay, thanks for the information @air2.

@air2 any idea why maintainer is not concerned in helping to fix a CVE security issue?

@gustaff-weldon you may find the explanation in this issue https://github.com/import-js/eslint-plugin-import/issues/2631

It is because it is not a CVE for this package and ask everybody just to ignore this CVE, which is of course a bad idea, because the CVE could also be triggered if another packages uses json5. Also he wants to support node4, because not supporting node4 would be a breaking change, and needs a major version increment, which would be more work for him.

@Teascade I use .npmrc with legacy-peer-deps=true to get around this issue. In my package.json I have: "eslint-plugin-import"<EMAIL_ADDRESS>
I would as well, but for some weird reason this results in an error Cannot read property 'spec' of undefined from npm itself with some projects, not all. Probably a bug, but makes the feature completely unusable for me.

json5 v1.0.2 has already been updated with this fix, and either way, it's not a valid vulnerability.
As is the case with almost every JS CVE, the best course of action is to do nothing until the ecosystem fixes it for you.
This is a duplicate of #2625; a duplicate of #2628; a duplicate of #2626; a duplicate of #2627; a duplicate of #2631; a duplicate of #2632; a duplicate of #2634; a duplicate of #2635; a duplicate of #2636; a duplicate of #2637.
Please stop filing issues about a vulnerability on "not the vulnerable package", it doesn't help.

@ljharb please stop telling to ignore CVE per se!

@mashpie why? it's a good one to ignore, and spreading FUD about it doesn't make anything more secure.

@ljharb with your Package possibly  not vulnerable doesn‘t mean there won‘t be any exploits for this CVE of json5.
ignoring it will ignore all vulnerable attack vectors beside of your package too. Like: chromiim, tap, nuxt, air-bnb, some pm2 etc.
json5 maintainer (@jordabtucker) himself committed to this CVE to be risky and mitigated it in all Majors.
So: Ok, assuming you package won‘t exploit that vulnerabilty, others might do so. Telling your community to ignore this CVE and generalizing it in way like „… As is the case with almost every JS CVE…“ is wrong, missleading and dangerous for the whole ecosystem you - on the other side - relay on to „… until the ecosystem fixes it for you…“
please don‘t get ne wrong: Fixes ARE applied by Json5 in v1 and v2 and it‘s not eslint-Plugin-Import to be blamed. Story finished. Keep CVEs serious and to investigated and mitigated rather than to be ignored.
Thanks! Have a nice 2023.

…sry for spelling: autocorrection on mobile ;)

It was a real CVE until json5 was upgraded, even because it was not triggered by this package does not say it was not a real CVE.  So if i have different dependecies that use json5 it could be a real CVE for me. Ignoring it would let me not see the CVE if i included a package in the future that uses json5 and was vulnerable. The only reason it can now be ignored is because it is fixed in json v1. But that was not the reason you told people to ignore it in the first place.

@mashpie first of all, i have been very explicit in explaining that this code path isn't valid, but either way, Prototype Pollution is a class of CVEs that isn't actually that exploitable in practice - primarily only in theory.
If we all want CVEs to be considered seriously, then the entire system (one designed for boxed software and hardware) needs to be rethought from scratch for open source, which would include considering code paths, so that transitive warnings don't trigger unnecessarily - because in security, false negatives are MUCH WORSE than false positives, because they undermine confidence in the security systems themselves. The most dangerous thing for the ecosystem is treating every CVE as equally critical and scary.

@ljharb Hi man, I just wanted to thank you for the patient explanation given in https://github.com/import-js/eslint-plugin-import/issues/2631.
I've learned something about the npm ecosystem here.
And lastly, excuse us for making so many duplicate issues on your repo for what is, ultimately, out of your control.
See you!

Thanks, I appreciate that.


Fixes in UnorderedHashMap

This change is

crates/cairo-lang-utils/src/unordered_hash_map.rs line 236 at r1 (raw file):

    /// This can be used to convert an unordered map to an ordered map (mostly when the unordered
    /// map was used for intermediate processing).
    pub fn iter_sorted_by_key<TargetKey, F>(&self, f: F) -> vec::IntoIter<(&Key, &Value)>

I'm wondering, why not impl IntoIterator<Item=...> though?


⚠ SSH 🇮🇩 Indonesia IDJ 1 has degraded performance

In a2f9a92, SSH 🇮🇩 Indonesia IDJ 1 (http://ids-1.opensvr.net:8080/) experienced degraded performance:

HTTP code: 400
Response time: 5464 ms

Resolved: SSH 🇮🇩 Indonesia IDJ 1 performance has improved in 6257b21.


Exlude (Empty) Files from PDF Export

[x] I have searched the issues of this repo and believe that this is not a duplicate.
[x]  I have searched the discussions and believe that my question is not already covered.

Feature Request

Hi, I find that keeping "typst-lsp.exportPdf" set often ends up generating blank pdf exports for files that are not meant to be compiled, such as templates or utility files. Having to manually delete these blank files gets tedious, so I'd like a settings interface to exclude files from the pdf export.
I imagine something along the lines of "typst-lsp.exportPdfExclude": "glob string" or even
"typst-lsp.exportPdfExclude": [
    "template.typ",
    "util.typ"
]

Alternatively, a setting for detecting empty files and not generating them would be even more helpful for my own use-case, but is less general, and I'm not sure what the complexity of implementing this would be.
Thanks for the project, it's been working wonderfully for me so far!

An include-list might be more useful since you'd be explicitly listing the files you care about, and you can add any number of helper files without having to change the vscode config. Though I agree, automatically excluding blank PDFs would be the most user-friendly option.

currently there is no good way for lsp to determine which file is the entry point. This may also happen in c/c++ and they use a compile_commands.json to determine where the entrypoint is.
i'd suggest something like a typst-project.toml or package.json to determine where the entrypoint is.
include/exclude list is obviously suboptimal but it's like a quick workaround.

While I agree that a typst project config file of some sorts makes the most sense, I feel like that is a pretty large change that would require a lot of community input, and should probably be standardized in conjunction with the @typst project.
IMO a whitelist setting (I agree w/ @platformer, this seems better than an exclude list) in the LSP config would be a great solution for the time being, and if/when a project format is decided on, the setting can be deprecated.

A very simple approach is to detect your document's title and don't export that documents with empty title.
// main.typ, this document will have pdf exported
set document(title: "...")

// package.typ, this document won't have pdf exported
// set document(title: "")

Related issue with a different solution: #11.

@Marmare314 Are you referring to on-demand compile? I feel that's a workaround at best, not a solution.

Typst packages already use a typst.toml which also has an excludes section, is there a way we could leverage that?


Websocket client does not allow sending 0 byte packets (IDFGH-9316)

https://github.com/espressif/esp-idf/blob/e8bdaf91986a41678adc6be13888fc037b1acb68/components/esp_websocket_client/esp_websocket_client.c#L849
This line prevents the websocket client from sending 0 byte packets, which should be valid, and some implementations use this feature for ping/pong packets, reducing overhead by having no data in the packet.
This seems to be present in all versions of esp-idf, up until 4.4.4. (I cannot find websocket client in 5.x)

Related PR was merged, close the issue, feel free to reopen. Thanks.

@Alvin1Zhang Just remind that v4.x branches do not include this fix.

@Alvin1Zhang Just remind that v4.x branches do not include this fix.

Also on 4.4.7 there is no esp_websocket_client_send_with_opcode so I can't even send ping or pong.


Use friendly app names for Fire TV sources

Breaking Change:
The source and sources_list attributes for Fire TV devices will use friendly app names instead of app IDs (e.g., "Netflix" instead of "com.netflix.ninja"). If you are using these attributes in automations, sensors, etc., you will need to update them.

If you are currently checking that the source attribute of a Fire TV device is a particular app ID, you have two options:

Check the app_id attribute instead
Replace that app ID with the friendly name for the app.

If you are currently checking the sources_list attribute, then you will need to check for friendly app names instead of app IDs.

You can still use app IDs for the media_player.select_source service.
Description:
Use friendly app names instead of app IDs for the sources for Fire TV devices (e.g., "Netflix" instead of "com.netflix.ninja").
Related issue (if applicable): fixes #
Pull request with documentation for home-assistant.io (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

This pull request is ready for review. Also, it needs the following labels:

Breaking change
By code-owner

Thanks for reviewing, @Santobert!


Different authorization scope https://www.googleapis.com/auth/drive.file

Rather than giving full access, I want to set up google-drive-ocamlfuse as a drop point where people drop files into my drive/folder without giving them full access. Is this possible? The drop will be done using sftp and will give an anon access to the user.

I don't think so (file permissions are not enforced). I think your best option would be to put user files in a staging area then periodically push them with something like drive.


Introduce MemoryProf for detecting memory spikes

What is the purpose of this pull request?
Introducing a memory profiler MemoryProf that can help to detect test examples and groups that cause memory spikes. Memory profiling supports two metrics: RSS and allocations.
Examples:
TEST_MEM_PROF='rss' rspec ...
TEST_MEM_PROF='alloc' rspec ...

By default MemoryProf shows the top 5 examples and groups (for RSpec) but you can set how many items to display with:
 TEST_MEM_PROF='rss' TEST_MEM_PROF_COUNT=10 rspec ...

The examples block shows the amount of memory used by each example, and the groups block displays the memory allocated by other code defined in the groups. For example, RSpec groups may include heavy before(:all) (or before_all) setup blocks, so it is helpful to see which groups use the most amount of memory outside of their examples.
What changes did you make? (overview)
MemoryProf has the following structure:

Configuration defines MemoryProf configs and adds a few supporting methods (memory_prof.rb).
Trackers are responsible for tracking memory usage and determining the top n examples and groups (memory_prof/tracker.rb).
Printer assembles and prints the results of a tracker (memory_prof/printer.rb).
RSpec integration (memory_prof/rspec.rb).
Minitest integration (memory_prof/minitest.rb).

There are two types of trackers: AllocTracker and RssTracker. A tracker consists of four main parts:

list - a linked list that is used to track memory for individual groups/examples, and it is an instance of LinkedList (for more info see (memory_prof/tracker/linked_list.rb).
examples – the top n examples, an instance of Utils::SizedOrderedSet.
groups – the top n groups, an instance of Utils::SizedOrderedSet.
track - a method that fetches the amount of memory in use at a certain point.

Checklist

[x] I've added tests for this change
[ ] I've added a Changelog entry
[x] I've updated a documentation

Hey @Vankiru, just wanted to share a neat trick–you should be able enable GitHub Actions in your fork https://github.com/Vankiru/test-prof/actions there should be a button in there or smth like that 😅


Can't switch away and back (AIGLX)

I can start my windowing manager fine (via startgnome, startxfce, etc.), but if I leave (via ctrl-alt-back) and return, I see the Developer Console, and:
(II) AIGLX: Suspending AIGLX clients for VT switch.

I don't know how to get the GUI back.

Hi,
I can only switch between desktops using ctr-alt forward/back and am running into the same error. crt-alt-shift back is not an option. Any ideas?


UnicodeDecodeError in assert_nondirty

Windows 7, Python 2.7.10, bumpversion 0.5.3
[...]: bumpversion patch
<...>
122 files updated, 2 files merged, 64 files removed, 0 files unresolved
(branch merge, don't forget to commit)
Traceback (most recent call last):
  File "C:\Python27\ArcGIS10.2\lib\runpy.py", line 162, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "C:\Python27\ArcGIS10.2\lib\runpy.py", line 72, in _run_code
    exec code in run_globals
  File "C:\Python27\ArcGIS10.2\Scripts\bumpversion.exe\__main__.py", line 9, in <module>
  File "C:\Python27\ArcGIS10.2\lib\site-packages\bumpversion\__init__.py", line 897, in main
    vcs.assert_nondirty()
  File "C:\Python27\ArcGIS10.2\lib\site-packages\bumpversion\__init__.py", line 174, in assert_nondirty
    b"\n".join(lines)))
UnicodeDecodeError: 'ascii' codec can't decode byte 0xf1 in position 77: ordinal not in range(128)

Thanks and good find!
Could you attach the result of running git status for this project ? I suspect you have a non-ascii filename that causes this error, but I'd like to know for sure. If that's the case, that should totally be fixed in bumpversion.

peritus, i don't have git, i use mercurial <3
hg status shows no non-ascii filenames as this moment. Although i've changed project's state since i posted this bug report. There is non-ascii file in my project, but i doubt it was in the hg status output when i got the bug.

@lansman Yes, of course. Also hg status having non-ascii characters could trigger this, I suppose.

I've just checked that problem and yes, there are non-ascii files in lines list in bumpversion/__init__.py
There is simple solution for that, just replace 174 line content:
from
b"\n".join(lines)))

to
b"\n".join(str(l).encode('string_escape') for l in lines)))

And always to avoid unicode problems everywhere, use:
1. When you want convert Unicode string to bytes string with ASCII only
s_uni = u'\u0430\u0431\u0432' # first three letters of russian alphabet

s_uni.encode('ascii') # UnicodeEncodeError, because their codes a far more than >127

s_uni.encode('unicode_escape').encode('ascii')  # OK, unicode_escape gives us byte string with codes <= 127

2. When you want convert Bytes string (which MAY contain bytes with codes > 127, e.g. those strings are returned by various python built-in OS, network etc functions) to bytes string with ASCII only (<= 127)
s_uni = u'\u0430\u0431\u0432' # first three letters of russian alphabet
# cp1251 - 1-byte ANSI encoding of Russian alphabet
s_1251 =  s_uni.encode('cp1251')  # OK

s_1251.encode('ascii') # UnicodeDecodeError, because s_1251 byte string has bytes with codes > 127

s_1251.encode('string_escape').encode('ascii') # OK, string_escape gives us byte string with codes <= 127


Revert conditions for PyPI job

Proposed changes:
Revert the conditions for the PyPI job to their original state.
Status (please check what you already did):

[x] made PR ready for code review
[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog
[ ] reformat files using black (please check Readme for instructions)

@federicotdn do you want to edit with the fix?

Yes! Just pushed them here.


no file menu with python 3

With the current code (commit 03888fd423680a23c0b0584e5c2b95e24cf81443) I have no file menu when I start with python3 bootstrap.py. There are no warnings or errors.
With python2 bootstrap.py there are no problems.

What platform are you working on?

I mean: Windows, Linux or Mac? And also: PyQt4 or PyQt5?

eendebakpt@marmot:~/develop/spyder$ python3 bootstrap.py --no-apport
Executing Spyder from source checkout
NOTICE: Ubuntu Apport exception hook is disabed
Revision 03888fd, Branch: master
01. Patched sys.path with /home/eendebakpt/develop/spyder
02. PyQt5 is detected, selecting
03. Imported Spyder 3.0.0dev
    [Python 3.4.3 64bits, Qt 5.2.1, PyQt5 5.2.1 on Linux]
04. Executing spyder.main()
Bootstrap completed in 00:00:00.3443

I'm sorry, I'm unable to reproduce this problem.

With Qt4 the system is working fine (latest checkout from git)
python3 bootstrap.py --gui pyqt

The system with Qt5 is still missing the file menu. I will check the issue again once I update my system (currently ubuntu 14.04, will go to 16.04).

My configuration is almost the same (PyQt5, Kubuntu 14.04) and I can't reproduce this error.

I seems that I can reproduce it but randomly. It's the same symptoms as when a transparent QWidget is placed at the default position (0, 0), so that the clicks don't pass the the underlying menus.
Maybe it's random because the widgets are not always created in the same order ?

This was fixed in master.


Feature Sugesstion: date_range from end to start

When using pd.date_range we work our way forwards from start to end.  It would be nice to pass a flag or use the dates in reverse to go from end to start when creating the date_range.
#EX:
pd.date_range(end, start)
#or
pd.date_range(start, end, end_to_start=True)

Another example given here:
http://stackoverflow.com/questions/35437733/pandas-date-range-starting-from-the-end-date

reversed?

In [7]: d=pd.date_range(start='2000', freq='Q', end='2002')

In [8]: d
Out[8]:
DatetimeIndex(['2000-03-31', '2000-06-30', '2000-09-30', '2000-12-31',
               '2001-03-31', '2001-06-30', '2001-09-30', '2001-12-31'],
              dtype='datetime64[ns]', freq='Q-DEC')

In [9]: reversed(d)
Out[9]: <reversed at 0x1073aaa20>

In [10]: pd.Index(reversed(d))
Out[10]:
DatetimeIndex(['2001-12-31', '2001-09-30', '2001-06-30', '2001-03-31',
               '2000-12-31', '2000-09-30', '2000-06-30', '2000-03-31'],
              dtype='datetime64[ns]', freq=None)

Not reversed, the order doesn't change, but you work the algorithm from the end date to the start. I'll add an example above to make it more clear.

so pls provide a usecase. Generally you would simply reverse sort to work with things in the reverse order.


Fix cool index

Sorry to open a new PR so soon but, while trying to implement this new attribute with HASS, I noticed that elect and cool weren't keys in the same dict but different dicts in a list

Thanks. That was my mistake. Your fix was perfect, but I like my way better. Do you use my HASS integration? https://github.com/fustom/ariston-remotethermo-home-assistant-v3

Yeah, I thought the same but didn't change in a single dict anyway because maybe there was a reason that I didn't know.
Of course! I'm testing locally and soon will make a new PR in the other repo.
Thanks to you and your project I'm able to control that device, their app is awful...


Anvil reverts with invalid FE opcode even though the opcode is not run.

Component
Anvil
Have you ensured that all of these are up to date?

[ ] Foundry
[ ] Foundryup

What version of Foundry are you on?
anvil 0.2.0 (5be158b 2023-10-02T00:23:45.472182000Z)
What command(s) is the bug in?
Running specific smart contract code
Operating System
macOS (Apple Silicon)
Describe the bug
I'm using cannon to develop some contracts. Cannon uses Anvil to run them.
When I run the code on the Hardhat network, everything succeeds. When I run it on Anvil, I get an InvalidFEOpcode error even though the FE Opcode is not being run. Full error:
 Error: transaction reverted in contract unknown: processing response error (body="{\"jsonrpc\":\"2.0\",\"id\":53,\"error\":{\"code\":-32603,\"message\":\"EVM error InvalidFEOpcode\"}}", error={"code":-32603}, requestBody="{\"method\":\"eth_estimateGas\",\"params\":[{\"type\":\"0x2\",\"maxFeePerGas\":\"0xd09dc300\",\"maxPriorityFeePerGas\":\"0x59682f00\",\"from\":\"0xf39fd6e51aad88f6f4ce6ab8827279cfffb92266\",\"to\":\"0xe661b3f15d0310af885f4d5c6482c0b5a8c0523d\",\"data\":\"0xd7513467000000000000000000000000000000000000000000000000000000000000002000000000000000000000000070997970c51812dc3a010c7d01b50e0d17dc79c8000000000000000000000000f275321e2e6a907f9113f037ef9ecc22b4dd4ff4000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000f42405f63408b72155b13a706288900cf7bc9ad629958e4992a0087324937e2affbfa00000000000000000000000000000000000000000000000000000000000000e000000000000000000000000000000000000000000000000000000000000001200000000000000000000000000000000000000000000000000000000000000004cfae3217000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000041da67e60a74f3985567c34a9a75a40f78bdc0c2924f024a7289639a776b84a7ec7e24181439b2166af1274da4127e41ce452f2add4a77692aa1c35727ed8a77c51b00000000000000000000000000000000000000000000000000000000000000\"}],\"id\":53,\"jsonrpc\":\"2.0\"}", requestMethod="POST", url="http://<IP_ADDRESS>:8545", code=SERVER_ERROR, version=web/5.7.1)

Reproduction Repo
Including a more in-depth explanation is here: https://github.com/CoinageCrypto/anvil-revert-issue

thanks for flagging
https://github.com/foundry-rs/foundry/pull/6140 should fix it
I was unable to run the example, unclear how I can run this against a local anvil instance, only got errors with pnpm t -- --network hardhat
pnpm t -- --network hardhat

><EMAIL_ADDRESS>test /Users/Matthias/git/rust/forge-repros/anvil-revert-issue
> hardhat test "--network" "hardhat"

Downloading compiler 0.8.21
Error HH502: Couldn't download compiler version list. Please check your internet connection and try again.

For more info go to https://hardhat.org/HH502 or run Hardhat with --show-stack-traces
 ELIFECYCLE  Test failed. See above for more details.

could you perhaps check the PR directly?

Downloading compiler 0.8.21
Error HH502: Couldn't download compiler version list. Please check your internet connection and try again.

We've noticed today that compiler cannot be downloaded, that looks like something on hardhat end perhaps?

Yep @noisekit this looks hardhat related—please let us know once you get to test!

Yep @noisekit this looks hardhat related—please let us know once you get to test!

I personally did not have this issue today because all these compilers are already cached on my dev machine (and in CI).


Polling option not picked up by watchpack

Hi, I am reposting this here even though I'm not convinced it's actually a problem with karma-webpack. This is a direct copy+paste from the issue I posted in the webpack issue tracker...

Using webpack v1.9.11, I am trying to enable poll-based watching, in my karma configuration I have
    webpack: {
        watchOptions: {
          poll: true
        },
        devtool: 'inline-source-map',
        resolve: webpackConfig.resolve,
        module: {
          loaders: [buildConfig.loaders.babel]
        }
    }

It seems as if the watchOptions options are not being propagated to watchpack. My other options work. I can add a single line to node_modules/webpack/node_modules/watchpack/lib/DirectoryWatcher.js:
     options.poll = true;

in the DirectoryWatcher function and then polling works.
Any ideas?

still half broken. now polling works when the spec file is updated, but if i update the file that the spec is testing polling doesn't pick up the change


Update TheThingsNetwork.cpp

When the MCU baud rate is different from default it is necessary to "autobaud" again after the module reset. Also, the response message after a reset is not readble if the MCU is working on a different baud rate. For this reason it it neccessary to issue another "autobaud" and a SYS_GET_VER to leave unchanged the behaviour of this function, that is reply with module informations.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for reporting, we'll be reviewing this!

Closes #219

You mean this way?

Well, we don't need the length of the first call. Since we're writing to buffer anyway, it may be clean to at least read it. Can you revert back to your first implementation? That's a bit cleaner than this one

Although it is not generally used I prefer to keep the statement as a reminder that there is a return value...