Use bundled version of rubocop instead of latest gems

Hi 👋 Thanks for this action.
Bug Report
Describe the bug
We installed this action, but were not on the latest version of Rubocop. The action installed the latest versions, so we had some issues. It would be much preferable to use the bundled versions.
To Reproduce
Steps to reproduce the behavior:

Have a Gemfile.lock with older rubocop
Have some errors in the latest version
Use action

Expected behavior
The action uses the bundled versions instead of the latest ones.
Stacktrace
If applicable, add the action stacktrace.
Action Version & Workflow File

Version v1.0.3

Workflow File
name: Rubocop

on: [push]

jobs:
  build:
    runs-on: ubuntu-latest
    steps:
    - uses: actions/checkout@v1
    - name: Rubocop Linter
      uses<EMAIL_ADDRESS>      env:
        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

@bbugh Hey there!
So I have had this conversation before, not sure if its been on this repo specifically. I am open to a change to add this functionality in, and probably will myself if I find some time (would love if you take a shot at it though 😄)
This is one of three very similar actions, with more coming, and I would like to keep the code the same as much as I can so I am planning to add it to all three.
I want to have a way for users to toggle between just doing a gem install vs. doing a bundle. Something like this:
jobs:
  build:
    runs-on: ubuntu-latest
    steps:
    - uses: actions/checkout@v1
    - name: Rubocop Linter
      uses<EMAIL_ADDRESS>      env:
        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
      with:
          bundle: true # false would be the default

Then, in entrypoint.sh we would do something like this:
if [ "${INPUT_BUNDLE}" = 'true' ]; then
  bundle install --jobs 4 --retry 3
else
  gem install rubocop rubocop-performance rubocop-rails rubocop-minitest rubocop-rspec
fi

Pretty simple change. One thing I would like to also figure out a good way of doing though is allowing a user to add in rubocop plugins, or replace the main rubocop runner. I haven't figure out a great way of doing that yet but I imagine it will be similar to above.
related to andrewmcodes/haml-lint-action#4

That does seem pretty easy!
I know very little about how the actions work yet, but with the yaml could you use an array for the gems?
    - name: Rubocop Linter
      uses<EMAIL_ADDRESS>      env:
        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
      extensions:
        - rubocop-performance
        - rubocop-rails
        - rubocop-minitest

# somewhere in the ruby code?
ENV['INPUT_EXTENSIONS'] = step_config['extensions'].join(" ")

else
  gem install rubocop $INPUT_EXTENSIONS
fi

I tried that and it didn't work but I was running on no sleep so I will give it another shot

You should be able to do this with the new v1.0.5 release.


standard editor should use CEF repo menu options

The Repo menu VI has standard terminals and need to change them to dynamic.

Double check that there isn't a DD sub vi in there somewhere. I've been tending to use static wrappers around DD functions to enforce some behaviors.


Implement SPI Cmd/Data

To control the ST7789 Data/Command Pin, the SPI Driver flips the MISO Pin as though it was a GPIO:
Here's the implementation for ESP32-C3: esp32c3_board_spi.c
int esp32c3_spi2_cmddata(FAR struct spi_dev_s *dev, uint32_t devid, bool cmd) {
      ...
      /*  This is the Data/Command control pad which determines whether the
       *  data bits are data or a command.
       */
      esp32c3_gpiowrite(CONFIG_ESP32C3_SPI2_MISOPIN, !cmd);

To implement this on BL602 we reconfigure MISO as GPIO on the fly: bl602_spi.c
static int bl602_spi_cmddata(struct spi_dev_s *dev, uint32_t devid, bool cmd) {
  //  MISO is now configured as SPI Pin. We reconfigure MISO as GPIO Pin.
  gpio_pinset_t gpio =
    (BOARD_SPI_MISO & GPIO_PIN_MASK)  //  Get the pin number
    | GPIO_OUTPUT | GPIO_PULLUP | GPIO_FUNC_SWGPIO;  //  Change to GPIO Output
  int ret = bl602_configgpio(gpio);
  ...
  //  Set MISO to High (data) or Low (command)
  bl602_gpiowrite(gpio, !cmd);

  //  After this the caller will transmit data or command.
  //  Then bl602_spi_select() will revert MISO back from GPIO to SPI Pin.
  //  We must revert because the SPI Bus may be used by other drivers.
  return OK;
}

When the SPI Port is deselected (after the SPI operation), we revert MISO back from GPIO to SPI Pin: bl602_spi.c
static void bl602_spi_select(struct spi_dev_s *dev, uint32_t devid, bool selected) {
  ...
#ifdef CONFIG_SPI_CMDDATA
  //  Revert MISO from GPIO to SPI Pin. See bl602_spi_cmddata()
  if (!selected)
    {
      spiinfo("Revert MISO to SPI");
      bl602_configgpio(BOARD_SPI_MISO);
    }
#endif  //  CONFIG_SPI_CMDDATA
}

We must revert because the SPI Bus may be shared by other SPI Drivers. (Like the Semtech SX1262 Driver on PineDio Stack BL604)
(TODO: Test with other SPI Drivers)


Added profile editing.

Added profile editing and image upload. More work needs to be done on the profile page.


ListGroup throws OverflowException

Description
ListGroup intermittently throws OverflowException from the librdkafka bindings. It appears to happen when group membership information reaches a certain size.
How to reproduce
With Confluent.Kafka 2.0.2 on .NET 6:
using Confluent.Kafka;
using System;
using System.Collections.Generic;
using System.Threading.Tasks;

var broker = $"bootstrap:9092";
var groupId = $"test-group-{Guid.NewGuid()}";
var topicName = "test-topic";

var adminConfig = new AdminClientConfig
{
    BootstrapServers = broker,
};

var consumerConfig = new ConsumerConfig
{
    BootstrapServers = broker,
    EnableAutoCommit = false,
    GroupId = groupId,
};

var admin = new AdminClientBuilder(adminConfig)
    .Build();

for (var i = 0; i < 10; i++)
{
    var consumer = new ConsumerBuilder<Ignore, Ignore>(consumerConfig).Build();

    consumer.Subscribe(topicName);
    await Task.Delay(TimeSpan.FromSeconds(5));

    var info = admin.ListGroup(groupId, TimeSpan.FromSeconds(5));
    Console.WriteLine($"{i}: {groupId} has {info.Members.Count} members");
}

After a few members (locally, 3 seems to do it), this program crashes:
0: test-group-496fec82-92e4-4d54-9d51-ce769e149134 has 1 members
1: test-group-496fec82-92e4-4d54-9d51-ce769e149134 has 2 members
Unhandled exception. System.OverflowException: Arithmetic operation resulted in an overflow.
   at System.IntPtr.op_Explicit(IntPtr value)
   at Confluent.Kafka.Impl.SafeKafkaHandle.CopyBytes(IntPtr ptr, IntPtr len)
   at Confluent.Kafka.Impl.SafeKafkaHandle.<>c.<ListGroupsImpl>b__73_3(rd_kafka_group_member_info mi)
   at System.Linq.Utilities.<>c__DisplayClass2_0`3.<CombineSelectors>b__0(TSource x)
   at System.Linq.Enumerable.SelectRangeIterator`1.ToList()
   at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)
   at Confluent.Kafka.Impl.SafeKafkaHandle.<>c.<ListGroupsImpl>b__73_1(rd_kafka_group_info gi)
   at System.Linq.Utilities.<>c__DisplayClass2_0`3.<CombineSelectors>b__0(TSource x)
   at System.Linq.Enumerable.SelectRangeIterator`1.ToList()
   at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)
   at Confluent.Kafka.Impl.SafeKafkaHandle.ListGroupsImpl(String group, Int32 millisecondsTimeout)
   at Confluent.Kafka.Impl.SafeKafkaHandle.ListGroup(String group, Int32 millisecondsTimeout)
   at Confluent.Kafka.AdminClient.ListGroup(String group, TimeSpan timeout)
   at Program.<Main>$(String[] args) in /workspaces/ubuntu/Program.cs:line 35
   at Program.<Main>(String[] args)

Full logs here with Debug = "all" on adminConfig.
The problem is not reproducible with Confluent.Kafka 1.9.3.
Checklist
Please provide the following information:

[x] A complete (i.e. we can run it), minimal program demonstrating the problem: above
[x] Confluent.Kafka NuGet version: 2.0.2
[x] Apache Kafka version: 2.7 (Confluent Platform 6.1.2)
[x] Client configuration: above
[x] Operating system: macOS (M1), Ubuntu 22.10 (x64)
[x] Provide logs (with "debug" : "..." as necessary in configuration): run.log
[ ] Provide broker log excerpts: n/a
[ ] Critical issue.

We encounter the exactly same problem. Would be very helpful to investigate and fix this.

Any update on this?  Thanks!

Thank you for reporting the bug. I am currently investigating the issue.

This should be fixed in the coming release.

When do you plan to release the next version including this fix?


remove old 'tip' about how to adjust hinge angle

Latest emulators have a 3D simulation and dedicated hinge-angle configurators

Shane took care of this in #11


Add Address Element so we can get shipping information

Is your feature request related to a problem? Please describe.
Currently there is no way to get a users shipping information with the current plugin (a very common use case).
It's not ideal for us to create the shipping address element on our own separately  as a major win is to assign the return result from the address element to the payment sheet element. This has the effect of adding the shipping details to the payment intent, and pre-filling billing address.
Describe the solution you'd like
Add 2 new functions:

A "present" function that calls Stripe's Address element.present
Create a listener function for us to get the completed result from shipping address input

Describe alternatives you've considered
Creating the address element on our own, separate to this plugin, however the down side is we can not assign the result to the payment sheet creation function.
Usage Product
Usage in e-commerce related apps

+1
Not sure how people are able to use this plugin without this feature. Makes 1 click checkouts impossible.

This issue has been resolved by https://github.com/capacitor-community/stripe/pull/315
I will close.


Crash if search contains invalid substitution

Steps to reproduce:
[[file]]
src = "..."
search = "{no-such-key}"

tbump crashes with:
:: Bumping from 2.5.1-1 to 2.5.2-1
Traceback (most recent call last):
  File "/mnt/data/dmerej/.local/bin/tbump", line 5, in <module>
    module.main()
  File "/mnt/data/dmerej/src/tanker/tbump/tbump/main.py", line 183, in main
    run(args)
  File "/mnt/data/dmerej/src/tanker/tbump/tbump/main.py", line 103, in run
    bump(bump_options, operations)
  File "/mnt/data/dmerej/src/tanker/tbump/tbump/main.py", line 137, in bump
    executor = Executor(new_version, file_bumper)
  File "/mnt/data/dmerej/src/tanker/tbump/tbump/executor.py", line 50, in __init__
    file_bumper.get_patches(new_version),
  File "/mnt/data/dmerej/src/tanker/tbump/tbump/file_bumper.py", line 198, in get_patches
    change_requests = self.compute_change_requests()
  File "/mnt/data/dmerej/src/tanker/tbump/tbump/file_bumper.py", line 241, in compute_change_requests
    change_request = self.compute_change_request_for_file(file)
  File "/mnt/data/dmerej/src/tanker/tbump/tbump/file_bumper.py", line 278, in compute_change_request_for_file
    to_search = file.search.format(current_version=re.escape(current_version))
KeyError: 'no-such-key

Something I noticed was that currently there are lots of places where .format(...) is called. This makes it tricky to track down all the places where that is called and it is hard to know what replacement keys can be used.
What would you think if I approached this with a bit of a bigger change? My idea is to have a single class that is only in charge of formatting these templates. This class would know the keys & values that could be used and only needs to be given the specific format template. This would make it easy to handle these errors in a consistent way so that the CLI can display the message. It also would standardize the replacement keys that are available and make it easy to add new ones if needed. (Could also address #109 and make it easier to achieve #85.)
This would move a lot of things around since new_version and current_version are currently passed into a lot of different places. So I wouldn't want to dive into that if it would be something that would be unlikely to be merged. Thought?

So I wouldn't want to dive into that if it would be something that would be unlikely to be merged.

I think having a separate class is a really good idea -go for it ! We have plenty of tests so it's unlikely we get regressions - and even then we'll be able to fix them :)


Update ALL the OCID in the terraform scripts of DB Security Livelabs

Hi Team,
With William Masdon, we've updated the livelabs Terraform scripts for the main packages: DB Security Basics (WMS_ID 3241) and Advanced (WMS_ID 11057).
But we have also all the individual DB Security Livelabs Terraform scripts to update to target the new OCID VMs now.
So, please, could you change these Terraform scripts accordingly:

OCID DBsecLab VM Only for:
1644	DB Security - OLS
3262	DB Security - DMS
3242	DB Security - DBSAT
3243	DB Security - NNE
3261	DB Security - Unified Auditing
3260	DB Security - Privilege Analysis
3263	DB Security - ASO
1641	DB Security - DB Vault
5881	DB Security - TSDP
9801	DB Security - Oracle Data Safe for On-Premises DB

OCID DBsecLab + AVS + DBF Vms for:
3460	DB Security - AVDF

OCID DBsecLab + OKV Vms for:
5322	DB Security - OKV

Please contact me if you have any question
Best regards
Hakim

tag @william-masdon-oracle - Could you check and update the other DB Security Livelabs Terraform scripts to target the new OCID VMs.


refactor: use pull-streams

Tests are failing, not sure where the issue is currently.

All tests passing :)

Coverage increased (+0.4%) to 75.0% when pulling 716e7381389fb154b61ac79b345e7104fdb8c2b1 on pull into 0eabb1813971defd3747068d732ba11a0fa68f17 on master.


Destructive swiping behavior, iOS 9

Thank you for making SwipeCellKit. You did a really nice job with the animations. We've been wanting Mail like swiping in Basecamp for a long time but never felt like building it.
Sorry for so many changes in one PR. I can break it up if you don't want to accept all of them.
iOS 9
I made it so the framework can be included in a project that targets 9 and 10. I didn't actually try to duplicate the animation code in 9 but at least we can derive our cells from SwipeTableViewCell and they will function the same as long as we don't set the delegate.
I know it's not great. We're OK with not having perfect feature parity in iOS 9 but we can't drop support with 5% of our customers still on it.
minimumDestructiveWidthRatio
I wanted the destructive swipe to be triggered with more of flick instead of having to drag it most of the way over.
deleteRowOnDestruction
Basecamp does more animations that need to be coordinated when the destructive action is triggered. I think it would be a more Cocoa like pattern to remove my flag and have the delegate or action callback always be responsible for managing the UITableView.
Destructive animation
I wanted to provide better feedback to the user about what they just did. I also needed to do something to make it flow better with our animations.
I didn't realize until I just double checked it that this isn't actually how Mail works. It pans the cell and collapses it at the same time like you had it. I think Mail doesn't make it clear enough what happened. I use Airmail and I like its swiping animation. If your goal is to duplicate Mail then I understand you may not want this behavior.
I'm not sure if my changes handle the mask correctly.

Hi @dginsburg - Thanks for your kind words, and especially for taking the time to put this together. 🍻
There's lots of great stuff here to think about. I'd really appreciate if you could break it into separate pull requests once we decide how to proceed.

iOS 9
I made it so the framework can be included in a project that targets 9 and 10. I didn't actually try to duplicate the animation code in 9 but at least we can derive our cells from SwipeTableViewCell and they will function the same as long as we don't set the delegate.
I know it's not great. We're OK with not having perfect feature parity in iOS 9 but we can't drop support with 5% of our customers still on it.

I like this approach.  And it seems like a good compromise for issue #1.  I'm thinking it would also be nice to include a warning if the delegate is set when running on iOS 9.
Ideally we could break out the "animator" into a separate protocol and provide the iOS 10 implmentation and a "no-op" implementation.  It would get rid of a lot of the #available's in SwipeTableViewCell, but I can like with it for now.

minimumDestructiveWidthRatio
I wanted the destructive swipe to be triggered with more of flick instead of having to drag it most of the way over.

I'm currently working on refactoring some of the expansion logic (see below). As far as I can tell - mainly because my original code here is a bit of mess - this property only applies when flicking from the active state (ie. you need to swipe first to reveal the swipe menu, then flick).  I just want to confirm because I'll probably end up refactoring this a bit with my current changes and I don't want to lose your expected behavior.

deleteRowOnDestruction
Basecamp does more animations that need to be coordinated when the destructive action is triggered. I think it would be a more Cocoa like pattern to remove my flag and have the delegate or action callback always be responsible for managing the UITableView.

I struggled with this decision in my original implmentation. I included the deleteRows inside SwipeTableViewCell because I didn't want the client-side to call it with an incorrect UITableViewRowAnimation option.  If we can provide the Mailbox style of deletion animation  as an option (fully fill, then fade while deleting), then would this parameter still be necessary?

Destructive animation
I wanted to provide better feedback to the user about what they just did. I also needed to do > something to make it flow better with our animations.
I didn't realize until I just double checked it that this isn't actually how Mail works. It pans the cell and collapses it at the same time like you had it. I think Mail doesn't make it clear enough what happened. I use Airmail and I like its swiping animation. If your goal is to duplicate Mail then I understand you may not want this behavior.

Please see my question above.

Ideally we could break out the "animator" into a separate protocol and provide the iOS 10 implmentation and a "no-op" implementation. It would get rid of a lot of the #available's in SwipeTableViewCell, but I can like with it for now.

I learned a lot about @available while doing this. It's not as flexible as I wanted. You can't use it to have different versions of the same function signature or use it on an override. That's why I sprinkled it in so many places.
But, yeah, the behavior could at least be organized in the code to one protocol/extension with a known set of functions. This would probably be a tough merge if we're both working on it. If you know you are going to have a quiet day let me know and I'll make that into a PR.

this property only applies when flicking from the active state (ie. you need to swipe first to reveal the swipe menu, then flick)

Damn it, good catch! I broke that when I changed it to use your new isActive. It should be !state.isActive.
My goal is to be able to trigger the destructive action with your thumb from the default state when holding the phone with one hand.

If we can provide the Mailbox style of deletion animation as an option (fully fill, then fade while deleting), then would this parameter still be necessary?

What we're doing is moving a row from one section to another (the destructive action is for "Mark as read".) If the action callback was inside a .beginUpdates()/.endUpdates() block then I could probably make it work but it would make it harder because I've got code that ties together the model and UITableView changes.
I still think the library needs that flexibility - like how TweetBot uses swipes not to delete cells.

This would probably be a tough merge if we're both working on it. If you know you are going to have a quiet day let me know and I'll make that into a PR.

Thanks, a PR sounds great!

My goal is to be able to trigger the destructive action with your thumb from the default state when holding the phone with one hand.

Got it.  I think we just need a name for the propery that makes this clearer.  I was planning to add a expansionThreshold property to allow customizing the default, so if we could be clear that we're overriding the threshold all together and using distance instead. Maybe even use a enum/struct to easily configure a flexible trigger configuration and assign that to a property.  I'll work on that and run it by you.

If the action callback was inside a .beginUpdates()/.endUpdates() block then I could probably make it work but it would make it harder because I've got code that ties together the model and UITableView changes.
I still think the library needs that flexibility - like how TweetBot uses swipes not to delete cells.

Let me think about this one a bit more. I think we need something flexible enough to handle your requirement, destruction, and also the feature request in issue #14.  There's a lot of different permutations to consider.

I'll make a PR just for iOS 9 next week.

Hi @dginsburg - I worked on a more flexible expansion API this weekend.  You can take a look at where I'm going with it on the expansion branch.  The CompletionAnimation is still a work in progress and will change.  But the expansion trigger logic is a lot cleaner now.
I switched the SwipeExpansionStyle from an enum to a struct, and allows for more customization. The existing styles of none, selection, and destructive still exist (as backwards compatible static vars) but you can now customize your own SwipeExpansionStyle  with flexible options.  In your case, you could write an extension of SwipeExpansionStyle  as follows:

extension SwipeExpansionStyle {
   static var customDelete: SwipeExpansionStyle { return return SwipeExpansionStyle(target: .edgeInset(30),
                                                                                additionalTriggers: [.overscroll(0)],
                                                                                completionAnimation: .fill(.afterFill(delete: true))) }
}

The afterFill case still needs to handle deleting or resetting the cell at later point via a completionHandle or something in order to support #14.

Hi @dginsburg - I finished up the expansion API.  It has changed a bit since my last comment.  The main difference is now you can completely configure the expansion style from beginning to end and cooridinate deletion with your other row animations.  I think your configuration would now look like:
static var customDestructive: SwipeExpansionStyle { return SwipeExpansionStyle(target: .edgeInset(30),
                                                                   additionalTriggers: [.overscroll(0)],
                                                                  completionAnimation: .fill(.manual(timing: .after))) }

With .manual fulfillment, you need to call action.fulfill(style: .delete) from inside your action handler.  That essentially calls the tableView.deleteRows and configures the animation, so you can coordinate calling it with the rest of your row animation.
I'm interested to know if this works for you!

Sorry I had to put this off. Looks like you and others done a lot in the last two weeks! I'll resync and see if I even need to fork anymore.

No worries!  This pull request led to some good changes.  Thanks again for putting it together!  🍻
Full iOS 9 support has now been merged into develop. Let me know how it and the other expansion changes work for you.  I'm going to close this PR, but I think it still might be valuable to add your changes where the cell works as normal if the delegate has not been set.  Thoughts?
Feel free to create a new PR for that if you're interested.


Incompatible with Java 9

Java 9 support willy require upgrading to Spring Boot 2. See https://github.com/spring-projects/spring-boot/wiki/Spring-Boot-with-Java-9


LambdaTest: Modal does not overflow correctly

Bug description
The modal for licenses is too long and overflows not correctly.
Screenshots

Should be an easy fix. Wait a minute.


Azure Support

Is there any plan to support Azure or just release Azure X  as a service for us. That would help a lot.

And maybe we could just base on HDInssight to do that.

Thanks @BinZone for raising the request, may you be more specific on what's kind of support is needed here? A sample scenario would be great help. Thanks.

@BinZone OpenPAI can run on Azure. You can create a set of VMs, and deploy OpenPAI to the VMs as if it is an on-prem cluster. For the Azure X as a service, I assume you would like to have something like Azure quickstart template? As @scarlett2018 says, it would be great to elaborate more in this regard.


Refactor GraphQL File Upload Implementation

Purpose
Fixes: #2838
Examples
N/A
Checklist

[x] Linked to an issue
[x] Updated the changelog
[x] Added tests
[x] Updated the spec

LGTM! I hope there are no breaking changes here.


Command-line support

See the specification: https://mathewsachin.github.io/Captura/CmdLine/
ToDo

Check for any possible issues with this approach in Chocolatey.
Implement More arguments.
Support specifying filename for the captured file.
Testing

Really awesome. Thanks you guys really appreciate.

Thank You @randomwalk1225.
Any suggestions you have are also welcome!

Can I start record, pause, stop video with command line?

Currently pausing has not been implemented. You can start and stop recording.
Here's a dev build to test: https://ci.appveyor.com/api/buildjobs/5w2h4x69m9he481g/artifacts/Output%2FCaptura-Release.zip
Please see the specification above.
e.g. Record 20 seconds
captura start --length 20


Sort branch

we asked to solve the bubble-sort algorithm. we used the yield and block passing method to solve the algorithm.

STATUS: APPROVED :dancing_women: :partying_face:
Hello team @Addis0943 and @chasscepts   👋🏾 👋🏾 great job on this project
Highlights :white_check_mark: :white_check_mark:

[ ]  Well written README file, simple to follow and understand
[ ]  Clean well written code

However, before marking the project as complete please fix the following

[ ] Incoropate rubocop linters on your project. You can find setup instructions
[ ] You don't need Rspec testing at this point you will come back to it in the next milestones. So check on it

That's all :smile: kindly submit another review when done.
Feel free to reach out to me on slack if you have any questions or something is unclear, or just comment on this PR and tag my name  (@mimipeshy) and I will be glad to help
Regards
Peris Ndanu
Regards
Peris Ndanu

STATUS: CHANGES REQUIRED 🔴 ♻
Hello team @Addis0943 and @chasscepts   👋🏾 👋🏾 great job on this project
Highlights :white_check_mark: :white_check_mark:

[ ]  Clean well written code

However, before marking the project as complete please fix the following

[ ] Incoropate rubocop linters on your project. You can find setup instructions
[ ] You don't need Rspec testing at this point you will come back to it in the next milestones. So check on it
[ ] In your README file, in getting started section, after cd Bubble_sort in the next bullet point add the command on how to run your project

That's all :smile: kindly submit another review when done.
Feel free to reach out to me on slack if you have any questions or something is unclear, or just comment on this PR and tag my name  (@mimipeshy) and I will be glad to help
Regards
Peris Ndanu
Regards
Peris Ndanu

Thank you for your feedback. We fixed all the feedbacks.
kindly review and approve it, please.


Function Generator: Cancel dialog shows wrong save directory

Cancel directory selection leads to displaying "0" as the directory, even though the internal setting seems correct.

How to replicate: "Choose save directory", then click "Cancel". If the current function name doesn't exist yet, "save current function" still creates a valid function -- otherwise it creates an error.

Fixed with commit ba743cbff67be8f43448d8454f0de4356da86ff7


SoundCloud not working

It's not been working for quite a while now and I'm not sure if it's only me.

As a starter you can try the troubleshooting steps here:
https://abertschi.github.io/ad-free/troubleshooting/troubleshooting.html
If the issue persists post the following pieces of information:

your phone model
version of android
version of soundcloud
version of ad-free

As stated in the troubleshooting steps under 6.2 Record notifications for a new detector, enable the recorder for soundcloud and make sure you capture the ad. Send me the capture file. The storage location is displayed when you enable the soundcloud recorder. It is in ./Android/data/ch.abertschi.adfree/files/adfree-soundcloud.txt on your internal phone storage. With the capture file we can further investigate your issue.

Sorry for the late response. Always when I tried to test it I didn't get any ads.
I've activated the experimental detector for SoundCloud and it's working again. I'm not quite sure but I think it's like that since I've updated my OnePlus 3T to Android 10. Before that I didn't have to activate that feature.

Oh I see. You updated ad-free as well. Before version 2.0 the experimental soundcloud detector was not opt-in; there was no feature implemented to toggle active detectors in the app.


Bump to new version

I think ruleset is outdated, bump it to new version

@EVAST9919 will not able to update the ruleset for around 2 months for personal reason.
You can tag me in the discord channel if you need that ruleset.

Fixed


openOrJoin and checkPresence function not working with SSEConnection  in php

Hi, you wrote that connection.socketURL is VERY_important but i don't have node.js, so i cant specify socketURL.
Is it openOrJoin SSEConnection ?
Please help me.
What i ought to do?

Yes. openOrJoin can be added to dev/SSEConnection.js as well.
I'll add this method in the next commit, later today.

When i using open and join functions separately everything working fine, but with openOrJoin connection cant be establish, i cant find out what i am doing wrong

Also checkPresence function not calling callback function
This is not working with SSEConnection
connection.checkPresence(room_id, function(isRoomExist, roomid) {
if (isRoomExist === true) {
connection.join(roomid);
} else {
connection.open(roomid);
}
});

Yes. openOrJoin relies on checkPresence. So both methods will be implemented in the next commit.

Can you say please when are you going to commit sources?

Please check this commit: https://github.com/muaz-khan/RTCMultiConnection/commit/b9bf7ff9ee6b25b56a6cab43acfe1447e20b4ab9
checkPresence is now supporting SSEConnection.
Added new file: demos/SSEConnection/checkPresence.php

Sorry, but it does not working, i check you demo you added new button "Open or Join Room", your demo is not working to, i clicked that button, then i open new incognito window in chrome and  clicked "Open or Join Room" button and connection does not established.
checkPresence always return false

screen sharing with audio not workig
connection.session = {
audio:'two-way', //here i added
screen: true,
oneway: true
};
connection.sdpConstraints.mandatory = {
OfferToReceiveAudio:true, //here i change to true
OfferToReceiveVideo: false
};
connection.videosContainer = document.getElementById('videos-container');
connection.onstream = function(event) {
connection.videosContainer.appendChild(event.mediaElement);
event.mediaElement.play();
setTimeout(function() {
event.mediaElement.play();
}, 5000);`
}
screen sharing work fine but audio not working please check this code what's wrong.
Thanks in advance


Remove 'simple service' and 'explore capabilities' widget styling

Description

description text...
RHCLOUD-31879

Screenshots

Before:
After:

Checklist ☑

[ ] PR only fixes one issue or story
[ ] Change reviewed for extraneous code
[ ] UI best practices adhered to
[ ] Commits squashed and meaningfully named
[ ] All PR checks pass locally (build, lint, test, E2E)

[ ] (Optional) QE: Needs QE attention (OUIA changed, perceived impact to tests, no test coverage)
[ ] (Optional) QE: Has been mentioned
[ ] (Optional) UX: Needs UX attention (end user UX modified, missing designs)
[ ] (Optional) UX: Has been mentioned

/rebase


Make VS Code use ".js" instead of ".jsx" when auto-generating import for ".tsx" file

Given a file foo.tsx and bar.tsx, when importing bar from foo, I'd like a VS Code setting that will make it auto-import as import Bat from './bar.js';

The auto-import behavior will try to match whatever would work at runtime. Please post a complete repro (including tsconfig, package.json, and the actual files)

Hey @RyanCavanaugh. So it's my custom runtime in https://github.com/sdegutis/immaculatalibrary.com, where all .ts or .tsx files are compiled by sucrase and turned into .js files within the runtime environment. I currently have an ugly hack to just replace all .jsx with .js during the compilation phase, but it would be nice if VS Code had an option to just allow me to specify auto-imports as .js since that's what .ts and .tsx could reasonably compile to in other environments as well. This is also part of my plan for this week of creating universal modules that work the same both at build time (for my ssg) as well as client-side, which means my "compiler" would be generating .js files that can be imported in the browser, where .jsx isn't a reasonable extension to generate, plus it has no JSX in it (compiled away by sucrase).

On a related note, why hasn't the JS community standardized on just transforming JSX expressions to this? Seems like it would solve like every problem.
interface JSX {
  _isJsx: true,
  tag: string | Function,
  attrs: Record<string, any>,
  children: any[],
}

It sounds like you want to set jsx to react-native, which preserves JSX for the next step in the build chain, but tells TS that the final file extension will be .js

Yep that does exactly what I need. Maybe it should be renamed to be more generic? But minor issue


ui: Implement editing and deleting secrets

Please see individual commits for details.

Design question: editing is handled with dialogs, right? What's the philosophy behind choosing those instead of edit routes as is done for organizations, products and repositories?

Design question: editing is handled with dialogs, right? What's the philosophy behind choosing those instead of edit routes as is done for organizations, products and repositories?

Well, it felt maybe a bit excessive in this case to create a secrets/$secretName route for that reason only, and to redirect to a separate page. I felt it gave a more cohesive feel for both the actions for the buttons in the end of the row to result in a similar flow and view (although of course you could argue that it's more cohesive for all edits to behave the same way throughout the application). I also thought this might be ok, as the table itself already breaks the pattern, as the items in it don't have their own page where to perform the edit and delete actions. But yeah, feel free to disagree, and we can discuss it more when you get back if needed.

But actually now that I'm working on the infrastructure services, I realize it would probably be best to create an edit route instead, because the infrastructure service refers to a couple of secrets, and it would be good to be able to link to an edit page for those secrets straight from the infrastructure service details.


COL-770 changing Perfil to right arrow

Swapping button copy out with a fontawesome icon for COL-770

:+1:


Build and deploy to AWS

[x] Create and configure release-manager app in AWS EB
[x] Use shared configuration and code quality checks
[x] Deploy a beta version manually (hollowverse/hollowverse#167)
[x] Test locally
[x] Add health checks

[x] Report unhealthy if fetching environments using AWS SDK fails

[x] Create new IAM user with read-only access to EB environments

[x] Get and encrypt user secrets

[ ] Write deploy script
[x] Enable Travis builds for this repo

[x] Add required secrets to Travis settings

[x] Configure TypeScript to build to ./dist
[x] Write Dockerfile

[x] Test Docker image

Closes hollowverse/hollowverse#186
Closes hollowverse/hollowverse#167

Closing this PR because it was based on a branch of the base repo. I opened a new one started from my fork.


Add: Good TV [KR]

Channel Name
Good TV
Alternative Names (optional)
No response
Network (optional)
No response
Owners (optional)
No response
Country
KR
Subdivision (optional)
No response
City (optional)
No response
Broadcast Area
c/KR
Languages
kor
Categories (optional)
religious
NSFW
FALSE
Launched (optional)
No response
Closed (optional)
No response
Replaced By (optional)
No response
Website (optional)
http://home.goodtv.co.kr/
Logo
http://home.goodtv.co.kr/images/main/logo_goodtv.png
Notes
No response

Please update your request:

logo URL must use the HTTPS protocol

https://github.com/iptv-org/database/blob/master/CONTRIBUTING.md#channels


No way to target default token color in specific language

monaco-editor version: 0.X.Y
Browser: Google Chrome
OS: Mac
Playground code that reproduces the issue:
const theme = {
  base: "vs",
  inherit: true,
  rules: [
    {
      token: "",
      foreground: "#ff0000"
    },
    {
      token: ".html",
      foreground: "#00ff00"
    }
  ]
};

monaco.editor.defineTheme("theme", theme);
monaco.editor.setTheme("theme");

let editor = monaco.editor.create(document.getElementById("container"), {
  value: "
Hello
",
  language: "html"
});

I'd like to target the default token color for just the html language. I've tried every variant of "html", "*.html", etc. I can think of no way to do this. Could such a thing be added (or mentioned here)? 🙏
Unclear if this is a bug or feature request...?

The theme itself is language agnostic so if a token has no type (for example, hello in your case), it will only use the token color defined in
rules: [     {       token: "",       foreground: "#ff0000"     } ]
One way to solve this problem from your end is extending the HTML grammar https://github.com/microsoft/monaco-languages/blob/master/src/html/html.ts#L68 to give it a default token like invalid.

Thanks for the info. This is a feature request then 😃.


chore: create installers (but don't publish) on every pull request

This allows us to catch changes that break our installer builds before
merging the problematic changes.
As a way to simplify the CI configuration files, this commit introduces
an installers-all Makefile target that builds all installers.
This commit also replaces all the cp -rf calls with cp -RLf in
Makefile to avoid some weird hard link Appveyor issues.
See: https://github.com/resin-io/etcher/pull/1078
See: https://github.com/resin-io/etcher/pull/1354
Signed-off-by: Juan Cruz Viotti<EMAIL_ADDRESS>

And the culprit was a wget call in one of the CLI build scripts :)

Still failing on Appveyor, and apparently there's a conflict in Makefile :-(
Also, a minor nit, but could you change the commit / PR title to "chore: create installers (but don't publish) on every pull request" ? The current title makes it sound like you're not creating installers when publishing ;-)

Probably worth merging #1378 first, and then rebasing this one afterwards?

Still failing on Appveyor, but at least it's a slightly different error this time ;-)

What do you think about this @lurch ?

CI tests seem to be passing, but are printing:
><EMAIL_ADDRESS>install /Users/travis/build/resin-io/etcher/release/node-darwin-x64-dependencies/node_modules/lzma-native
> node-pre-gyp install --fallback-to-build
node-pre-gyp ERR! UNCAUGHT EXCEPTION
node-pre-gyp ERR! stack Error: Unsupported target version: 6.10.3

?

The issue is gone! Looks like passing -f to npm install (which we did because of optional dependencies), causes npm to continue even if errors are encountered along the way. I got rid of this option and everything is back to normal.

"back to normal" with all the CI builds failing again? :rofl:

Why is gyp trying to fetch iojs-1.6.6 (https://travis-ci.org/resin-io/etcher/jobs/233069196#L6131)?

"back to normal" with all the CI builds failing again?

Precisely :P

Why is gyp trying to fetch iojs-1.6.6 (https://travis-ci.org/resin-io/etcher/jobs/233069196#L6131)?

Hm, it should be downloading Electron 1.6.6 headers instead. I'll take a look.

Hm, the target is correctly set to electron (https://travis-ci.org/resin-io/etcher/jobs/233069196#L6123), and our npm dependencies script is configured exactly as https://github.com/electron/electron/blob/master/docs/tutorial/using-native-node-modules.md :/

:tada:

Hm, the target is correctly set to electron (https://travis-ci.org/resin-io/etcher/jobs/233069196#L6123), and our npm dependencies script is configured exactly as https://github.com/electron/electron/blob/master/docs/tutorial/using-native-node-modules.md :/

@jviotti did you find out what caused this?


Implement GreaterThan instruction

LGTM

The CI isn't working. I'll run the checks on my side and merge.

 Support gt instruction


QuRT: src/platform/qurt changes

Changes to support QuRT intrgration with DSPAL and move from simulator to real HW.
Signed-off-by: Mark Charlebois<EMAIL_ADDRESS>

Travis :)

The Travis error seems odd and unrelated to the patch:
The command "rvm 1.9.3 --fuzzy do $CASHER_DIR/bin/casher add $HOME/.ccache" failed and exited with 1 during .


Fixing double quote formatting in code block

fixing double quote formatting 🚀

@tabs-not-spaces : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Looks good; thanks.


Have on-error event on firebase-document on getting and saving event

Description
Would it be better if there's an event handler when saving/reading data in firebase-document?
Based on firebase documentation, set(), update(), transaction(), on() has the ability to catch errors and show them. But firebase-document doesn't have yet the capability to throw events when we get an error.

Expected outcome
when firebase call set, and there's an error, it should fire an error event

Actual outcome
no event is called

Live Demo

Steps to reproduce

Browsers Affected

[x ] Chrome
[ x] Firefox
[x ] Safari 9
[x ] Safari 8
[x ] Safari 7
[x ] Edge
[x ] IE 11
[x ] IE 10

+1 this is absolutely needed

Also, an event listener for isNew, maybe onIsNew="someFunction".

Someone right a patch because it would take the Polymer team the better half of the century to get on this, haha.


Service 'app' failed to build : Build failed

i am on arm64 ubuntu
using docker compose, just ran sudo docker compose up -d
https://pastebin.com/J6P97sbD
heres the .env

Hey thansk for bringing this to my attention. Seems that pnpm updated their lock file format.
I've updated the pnpm-lock.yaml file in the main branch. If you pull a fresh copy you should be good to go to try the docker-compose command again.

Closing this issue for now, feel free to open a new one. If you do though, please include more error output :blush:


Sunmi V2S - attempt to invoke interface method on null object reference

Thanks for this package - it looks great.
When I try to use it (through EAS build on Expo) I'm getting:
Attempt to invoke interface method 'void com.sunmi.peripheral.printer.SunmiPrinterService.printText' on a null object reference
Am I missing something? My code is pretty basic:
import SunmiPrinter from '@heasy/react-native-sunmi-printer';

SunmiPrinter.printerText('Hello hello hello');

I haven't used it in Expo, I can't answer it for you, it is recommended not to use Expo

Its not Expo specific - I'm using EAS build which is the same as using Android studio. Its just a different build system but it has complete parity.
Have you ever tried/tested the library on a V2S? What hardware are you using?

I see Sunmi development documentation and there is no v2

See: https://www.sunmi.com/en-US/v2s/

@ryanmitchell See the latest document, wait a moment, I will update

@ryanmitchell Please try it with v2

@ryanmitchell Is it ready to use, I'm going to close this.

@ryanmitchell Is it ready to use, I'm going to close this.

Struggling to get a build as I cant target the branch - it needs to be a release

Struggling to get a build as I cant target the branch - it needs to be a release

👌

I just tested against 1.6.2 and the build now fails...  is not allowed in the manifest element - see:
https://developer.android.com/guide/topics/manifest/manifest-element
Should it maybe be  as per: https://android.googlesource.com/platform/frameworks/base/+/master/packages/PackageInstaller/AndroidManifest.xml#5

@ryanmitchell You try again.

@Surile thanks. Unfortunately I'm back to getting the null object reference error so the change doesn't seem to have worked. Do you have any other ideas?

@ryanmitchell
package android:name="woyou.aidlservice.jiuiv5"

You try to put that code in AndroidManifest.xml under Android project

I saw the Chinese document, the document said that v2 and v2plus need to add this

Can you send the link to the document so I can investigate?

Documentation

Thanks, I see now, it needs added to queries.
I've got it working - I'll open a PR with the config needed for expo incase anyone else needs it.

Thank you!

谢谢，我现在明白了，它需要添加到查询中。 我已经开始工作了——我会用博览会所需的配置打开一个 PR，以防其他人需要它。

Ok, thank you

I think the best thing might be to add something to the docs:
For Expo users, you need to add a config plugin as follows:
const { withAndroidManifest } = require("@expo/config-plugins")

module.exports = function androiManifestPlugin(config) {
  return withAndroidManifest(config, async config => {
    let androidManifest = config.modResults.manifest

    androidManifest.queries.push({
        'package': {
            '$': {
                'android:name': 'woyou.aidlservice.jiuiv5'
            }
        }
    });

    return config
  })
}

Add this to a file called android-manifest.plugin.js in your app root, then add a reference to this file your expo.plugins array in app.json... eg.
		"plugins": [
			"./android-manifest.plugin.js"
		]

Hello guys,
I see that you have solved this issue of V2s for EAS, but what about Android build? Is there any opportunity to make it work?
Thanks

Hello guys,
I see that you have solved this issue of V2s for EAS, but what about Android build? Is there any opportunity to make it work?
Thanks

it should be ok

Yes, it works now. Thanks

Im having the same issue. Tried multiple versions of React-native. V2s and not using EAS or Expo.
Attempt to invoke interface method 'void com.sunmi.peripheral.printer.SunmiPrinterService.printText' on a null object reference


PluginManagerSetsCacheBackendRule false positive

reviewing some warnings I found a false positive from the plugin manager cache rule.
The triggering code looks something like this:
  /**
   * Constructs a MagicPluginManager object.
   *
   * @param string $type
   *   The plugin type, for example source.
   * @param \Traversable $namespaces
   *   An object that implements \Traversable which contains the root paths
   *   keyed by the corresponding namespace to look for plugin implementations.
   * @param \Drupal\Core\Cache\CacheBackendInterface $cache_backend
   *   Cache backend instance to use.
   * @param \Drupal\Core\Extension\ModuleHandlerInterface $module_handler
   *   The module handler.
   */
  public function __construct($type, \Traversable $namespaces, CacheBackendInterface $cache_backend, ModuleHandlerInterface $module_handler) {
    parent::__construct("Plugin/magic/$type", $namespaces, $module_handler, $this->pluginInterfaces[$type], $this->typeAnnotations[$type]);
    $this->alterInfo('magic_' . $type);
    $this->setCacheBackend($cache_backend, 'magic_' . $type . '_plugins');
  }

As you can se, the cache backend is actually set.
Stepping through the rule though something interesting happens on this line.
https://github.com/mglaman/phpstan-drupal/blob/ffdf3686f3cc3d8c3b3cfece7896dfc5e77d9e51/src/Rules/Drupal/PluginManager/PluginManagerSetsCacheBackendRule.php#L68-L70
Instead of String_, this type is actually \PhpParser\Node\Expr\BinaryOp\Concat because, its concatenating strings.

I wonder if this might be providing some false positives or false successes in some of these other places too.
src/Rules/Deprecations/GetDeprecatedServiceRule.php:        if (!$serviceName instanceof Node\Scalar\String_) {
src/Rules/Deprecations/StaticServiceDeprecatedServiceRule.php:        if (!$serviceName instanceof Node\Scalar\String_) {
src/Rules/Drupal/PluginManager/PluginManagerSetsCacheBackendRule.php:                if (!$cacheKey instanceof Node\Scalar\String_) {
src/Rules/Drupal/PluginManager/PluginManagerSetsCacheBackendRule.php:                            if (($item->value instanceof Node\Scalar\String_) &&
src/Rules/Drupal/RenderCallbackRule.php:        if (!$key instanceof Node\Scalar\String_) {
src/Type/ContainerDynamicReturnTypeExtension.php:        if (!$arg1 instanceof String_) {
src/Type/DrupalServiceDynamicReturnTypeExtension.php:        if (!$arg1 instanceof String_) {

This code is a NIGHTMARE that was written 3 years ago. It needs to be revamped to modern PHPStan

Main fix:
$cacheKeyType = $scope->getType($cacheKey);
$is_not_empty_string = $cacheKeyType->value();

The code just needs to be updated to latest PHPStan types, that was written before they existed.

I did bang on this a bit over the weekend and the check was easy... but the effects where problematic. Directly after the string check, it tries to get the value and validate it against the tag list. When its _String, but when its Concat, or a variable or... it crashes. I got stalled trying to find a generic way to resolve the value.
I'm afraid its a lost cause though because if say the variable is a constructor argument(like my example) there is no final string.


Stratus Does Not Use Instance Profile On EC2

What is not working?
When trying to run Stratus Red Team in an EC2 Instance, we found that it wasn’t able to leverage the instance credentials allocated to the instance. It gives us the following error:
Error: error configuring Terraform AWS Provider: no valid credential sources for Terraform AWS Provider found.

Please see https://registry.terraform.io/providers/hashicorp/aws
for more information about providing credentials.

Error: NoCredentialProviders: no valid providers in chain
caused by: EnvAccessKeyNotFound: failed to find credentials in the environment.
SharedCredsLoad: failed to load profile, .
EC2RoleRequestError: no EC2 instance role found
caused by: RequestCanceled: EC2 IMDS access disabled via AWS_EC2_METADATA_DISABLED env var

We are calling Stratus programmatically as follows:
import (
...
stratusrunner "github.com/datadog/stratus-red-team/v2/pkg/stratus/runner"
...

)
stratusRunner := stratusrunner.NewRunner(ttp,stratusrunner.StratusRunnerNoForce)
...
stratusRunner.Detonate()

When we try to run Stratus as a binary on the EC2 instance to execute the same technique, we run into the same error message:
root@ip-10-128-184-185:~# ./stratus detonate aws.defense-evasion.cloudtrail-stop
2023/06/05 04:11:07 Checking your authentication against AWS
2023/06/05 04:11:07 Warming up aws.defense-evasion.cloudtrail-stop
2023/06/05 04:11:07 Initializing Terraform to spin up technique prerequisites
2023/06/05 04:11:09 Applying Terraform to spin up technique prerequisites
2023/06/05 04:11:10 Error during warm up. Cleaning up technique prerequisites with terraform destroy
2023/06/05 04:11:11 unable to run terraform apply on prerequisite: unable to apply Terraform: exit status 1

Error: error configuring Terraform AWS Provider: no valid credential sources for Terraform AWS Provider found.

Please see https://registry.terraform.io/providers/hashicorp/aws
for more information about providing credentials.

Error: NoCredentialProviders: no valid providers in chain
caused by: EnvAccessKeyNotFound: failed to find credentials in the environment.
SharedCredsLoad: failed to load profile, .
EC2RoleRequestError: no EC2 instance role found
caused by: RequestCanceled: EC2 IMDS access disabled via AWS_EC2_METADATA_DISABLED env var

  with provider["registry.terraform.io/hashicorp/aws"],
  on main.tf line 9, in provider "aws":
   9: provider "aws" {

This is because both the binary and the SDK are leveraging the same terraform configuration for the AWS provider:
provider aws {
    skip_region_validation      = true
    skip_credentials_validation = true
    skip_get_ec2_platforms      = true
    skip_metadata_api_check     = true
}

When we unset the skip_metadata_api_check above, Terraform was able to run just fine using the assigned instance profile.
We noticed that the change was introduced very early here.
These changes specifically stop terraform (and thereby Stratus) from being able to discover the instance profile and use it on an EC2 instance.
Our questions for you:

Were these flags introduced into the terraform by design?
Would you be okay with us providing a PR to remove these checks if they weren’t intentional?

 But I 100% agree that there should be a way to have Stratus work when using instance roles

Hey @christophetd thanks so much for the swift reply.
We tried setting the variable AWS_EC2_METADATA_DISABLED=false and then running it, it doesn't help (IMDS is still unaccessible).
How best do you suggest we proceed with getting Stratus to work with instance roles?

I ran a few tests, and setting skip_metadata_api_check = false (default value) in the provider configuration seems to have little to no impact on the warmup time. I tracked down this parameter in the Terraform provider Go code, and it seems to only end up controlling whether the IMDS endpoint is used (at all) for credentials when nothing else is available.
See

https://github.com/hashicorp/terraform-provider-aws/blob/main/internal/provider/provider.go#L493-L499
https://github.com/hashicorp/aws-sdk-go-base/blob/main/aws_config.go#L353-L358
https://docs.aws.amazon.com/cli/latest/userguide/cli-configure-envvars.html#envvars-list:~:text=AWS_EC2_METADATA_DISABLED

Based on this, I believe it would take more time only when no credentials are configured at all when Terraform is run. But since we perform an authentication check before running Terraform, this should literally have 0 impact.
So we're good to go to remove this flag! Great catch. PR appreciated, otherwise I will look into fixing it in the coming weeks!

Great, thank you! I'll raise one for you today :)

I've raised https://github.com/DataDog/stratus-red-team/pull/368 🙂

Will be released shortly as part of v2.5.7. Thanks for the contribution!


Removing timeout for dnu publish

Sometimes on slower machines that it takes more than this time especially when -no-source is turned on.
@Tratcher

:shipit: I hope this only affects music store. 60 seconds would be prohibitively long in any of the tests I want to run.

that's right. I observed this issue on the cross plat CI for MusicStore.


Warn that logging out will lose E2E keys

Once we have megolm export wired up, we should obviously prompt the user to export

Done in https://github.com/vector-im/riot-ios/issues/950


Fix path to chr-1.fasta in readfasta.py

The path to the FASTA file in this code has a broken link. Kindly fix it @Shuyib

Acknowleged. Made the necessary changes.


How to use SV without submodules?

Hi,
Any possibility of replacing the use of submodules by a combination of cloning to a folder (which could be /sv, which should be in .gitignore)?
Submodules introduce extra complexity and can cause some issues with existing CI/CD pipelines and with multi-platform dev environments.
thx

Long term we are going to look into consuming Unity dependencies through UPM compared to submodules. Are you specifically having CI/CD issues with the native projects (vcpkg and gamecapture repos)? These projects would not change with the move to UPM.
How would your CI/CD pipelines resolve these dependencies in an ideal world? Would you have to manually piece together some sort of resources package that contains these dependencies and is installed on your CI/CD machine?
Have you considered creating a standalone SV package with the tools\scripts\CreateUnityPackage.bat command and using that package for automation compared to the repo + submodules?

We will give UPM a try.
Traditionally we have preferred a more static approach, where the CI/CD process doesn't introduce any updates besides what is committed.

Hi,
Can this be done with MRTK 2.3 and Unity 2019.3?
I've managed to build the unity package but have errors regarding unity xr packages when adding it to the project as a local package. I suspect this only works with Unity's new XR plugin system. Is this correct? If it is, is there an earlier commit on the master branch that works for the Unity 2019.3-MRTK 2.3 combo?
Cheers!

Hey @Nuno-Monteiro we managed to get that combination to work, so yes it is possible. It was not an easy process though, as we already had QR and Spatial libraires in our project. We took the SpectatorView.Unity/Assets folder out of the rep straight into our project and then had to spend an hour cleaning up missing references and commenting out code snippets... Not an elegant solution at all, but yes, we had Spectator View running from one Hololens 2 to another.

Hey @Nuno-Monteiro we managed to get that combination to work, so yes it is possible. It was not an easy process though, as we already had QR and Spatial libraires in our project. We took the SpectatorView.Unity/Assets folder out of the rep straight into our project and then had to spend an hour cleaning up missing references and commenting out code snippets... Not an elegant solution at all, but yes, we had Spectator View running from one Hololens 2 to another.

We either used Master or the Last release, will have to get back to you on that one, but for sure not an earlier version than that.

@Nuno-Monteiro I'm not sure how friendly MRTK 2.3 will be with Azure Spatial Anchors v2.2.1. Last time I looked into moving forward both Azure Spatial Anchors and MRTK they referenced conflicting ar foundation versions. Azure Spatial Anchors also isn't tested against Unity 2019.3, so I am not sure if things will break.

due to some sensitive reasons i can not get the CI/CD process done. not only the network(i've tried ssr/clash/astrill, proxy mode or global mode),but also the confusing generating process as well as bunch of missing parts of this SV thing.
please do help me with one single package or just send me a Mega link.


Opening a beginner clue nest increments the step count

Opening a fresh beginner clue nest should put the count at 0.
Instead, it makes it one higher.
Open a nest -> count now 1.
Drop clue.
Open another nest -> count now 2

Hmm, yeah this is the same issue as with juggling clues, and i dont really have a good solution for it atm


Manual testing of Release 2023.03 Part 3

Initial set-up
Add link to Plugin build #***

[x] Sarif report + Problems view
[x] Cancellation of generation (with Generate / Generate & Run)
[ ] Java docs styles
[ ] Sandboxing
[ ] Logging
[x] Hanging test timeout, check with Fuzzing/Symbolic
[ ] Overflows, check Summaries
[ ] Context estimator, check nightly runs
[ ] Bugs list

Security testing (sandboxing)
Basic security

[ ] Project with JDK 8 + manual scenario from PR #625 + default settings
[ ] Fuzzing only
[ ] Symbolic execution only
[ ] Project with JDK 11 + default settings
[ ] Fuzzing only
[ ] Symbolic execution only
[ ] Concrete executor is invoked from Symbolic execution
[ ] Project with JDK 17 + default settings

With permissions

[ ] add PropertyPermission in ~.utbot\sandbox.policy - check test for property is not in sandbox, execute
[ ] add "permission java.io.FilePermission  "<>", "read";" - check test for read not in sandbox, execute
[ ] add AllPermission  in ~.utbot\sandbox.policy - check NO tests are in sandbox, try to execute (carefully!!!)
[ ] corrupt permission - check how UTBot fails  - try scenario from #796
[ ] empty file
[ ] remove ~.utbot\sandbox.policy file - check generated tests are in sandbox

Test rendering

[ ] JUnit4
[ ] JUnit5
[ ] TestNG
[ ] JUnit5 with parametrization
[ ] TestNG with parametrization

Code structures

[ ] in private method
[ ] bug to check: #791

Reproducing:

#1314


Update swiss person dataset

The swiss (de-ch) person dataset seems to have quite untypical names (first names) in it. Also the list of last names is quite short.
The Federal Statistical Office has a large list with firstnames and how common they are (where in 2018): Source
The swiss post provides a large collection about the last names: opendata.swiss, swisspost
Should I create a pull request with adjusted lists for this data? If so, how many names to include for first names (women / man) and last names? Should I use the most commonly used or a random sample?

Great idea! I think 8k names (or less) and 8k  (or less) surnames will be enough. The main condition is that names must be valid and that's it.
I'll be waiting for your PR.


Document button and bar ConfigNode formats.

I need to write up some documentation on the formats that the button and bar configurations are expected to be in. They're not particularly complicated, and their format could probably be deduced by looking at the configs provided, but it's still a good idea.

Done. Documentation is on the wiki.


Added Right-To-Left (RTL) support to ViewPager

This modification allows user to set ViewPager layout direction to Right-To-Left (RTL). Since top tabs and bottom navigation have RTL support by default, no change is required for those elements.
Important note: user must add fragments in reverse order at StepAdapter:createStep() as suggested here:
http://stackoverflow.com/questions/18781944/how-to-make-my-viewpager-swipe-from-right-to-left

Codecov Report

Merging #111 into master will decrease coverage by 0.35%.
The diff coverage is 33.33%.

@@             Coverage Diff              @@
##             master     #111      +/-   ##
============================================
- Coverage     46.26%   45.91%   -0.36%
  Complexity      109      109
============================================
  Files            22       22
  Lines           871      882      +11
  Branches         84       86       +2
============================================
+ Hits            403      405       +2
- Misses          404      412       +8
- Partials         64       65       +1

Impacted Files
Coverage Δ
Complexity Δ

...main/java/com/stepstone/stepper/StepperLayout.java
33.65% <33.33%> (-0.57%)
17 <1> (ø)

Continue to review full report at Codecov.

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

Hi @Yasir-Ghunaim,
Thanks for the pull request.
I'll definitely have a look during the week and get back to you!

@zawadz88, I'm glad to help in supporting this great library. Please, let me know if you need further clarification.

My response to above comments:

when opening the Activity in RTL mode you can see the transition to the last step (this should happen without animation on start)

This has been addressed in this commit https://github.com/stepstone-tech/android-material-stepper/pull/111/commits/882706740aec1270f4290690e499083089942e3f
Further details are found in upcoming comments.

There are issues when your app supports both RTL and non-RTL configurations, e.g. changing locale could cause an error when a wrong step is displayed.

Do you mean changing locale while application is running and displaying a step?

I Googled this and there is an Open Source library which might handle this case: https://github.com/diego-gomez-olvera/RtlViewPager although I haven't tried it myself.

Personally, I don't think you should go for a third-party solution as long as a a cheaper solution exist using native ViewPager.

Hi @Yasir-Ghunaim,
I agree that using a 3rd-party library in not the best option if there are other solutions there.
However regarding:

There are issues when your app supports both RTL and non-RTL configurations, e.g. changing locale could cause an error when a wrong step is displayed.

I've noticed that this still isn't fixed.
Steps to reproduce:

Open the Default tabs sample
Click once on the Taps: 0 button
Go to Settings and change locale to e.g. Arabic
Go back to the app

Expected result:
A correct fragment should be displayed and its state saved. Other fragments should also be set correctly.
Actual result:

A correct fragment is shown on the 1st step however its state is not saved (button label should be Taps: 1 but is Taps: 0).
When you go the 3rd step now you will see the 1st fragment with its saved state, but the expected fragment here is the 3rd fragment.

So I had a look and found a much simpler solution using a custom PageTransformer which basically for RTL reverses  the direction, which seems to work since the ViewPager in this library doesn't support swiping between steps.
I'll try to have something production ready in a couple of days.
Thanks for this PR though!

Hello @zawadz88, regarding below comment:

There are issues when your app supports both RTL and non-RTL configurations, e.g. changing locale could cause an error when a wrong step is displayed.

When reproducing your experiment, I found out the problem to be in the way FragmentPagerAdapter handle fragments. Let's consider this scenario:

Default tabs sample is started in English locale
AbstractFragmentStepAdapter.getItem() is called for position 0 and 1 which will link these positions with Fragment1 and Fragment2 respectively
At this point user decides to switch to Arabic locale
When user returns to app, AbstractStepperActivity.onCreate() is called
mStepperLayout.setAdapter() is called with a starting position of 0
However since mRtlEnabled is true,  mPager.setCurrentItem() is called with a position of 2 as a result of RTL transformation
Now, AbstractFragmentStepAdapter.getItem() is called for position 2 and link that position with Fragment1
But, no further calls are made to AbstractFragmentStepAdapter.getItem() for position 0 and 1 since these positions have been populated with fragments already

I found this article to explain this issue really well:
Article Summary
Full Article
Path forward:
In StackOverflow, many developers have pointed out similar issues faced when using FragmentPagerAdapter. Some try to overcome this by switching to FragmentStatePagerAdapter which doesn't maintain hard link to attached fragments. Although, I'm not sure whether you want to go for that approach. There are some solutions to work around the way FragmentPagerAdapter is structured such as:

http://stackoverflow.com/questions/7723964/replace-fragment-inside-a-viewpager
http://stackoverflow.com/questions/13391048/swapping-fragments-in-a-viewpager
http://stackoverflow.com/questions/18588944/replace-one-fragment-with-another-in-viewpager

I'm still testing different methods to find out a way to shuffle fragments positions in PagerView which we can use at the moment of locale changes.

So I had a look and found a much simpler solution using a custom PageTransformer which basically for RTL reverses the direction, which seems to work since the ViewPager in this library doesn't support swiping between steps.
I'll try to have something production ready in a couple of days.
Thanks for this PR though!

I remember seeing this solution a few months ago but didn't get the chance to try it out. Please, let me know if you get a working version out of this.

Hi @Yasir-Ghunaim,
I've made the changes in #115 so I'm closing this PR.
Please have a look and let me know if they work for you :)


SPY-350 Don't let Master remove RUNNING Applications

SPARK-2425 introduce LOADING -> RUNNING ApplicationState transition and prevent Master from removing Application with RUNNING Executors

@markhamstra: do we still need this PR?

No this has already been fully integrated and through another generation of development upstream, and the fix is already present in csd-1.1 and csd-1.2.


Setting proxy_read_timeout per server block by default disallows per ingress settings via configuration-snippet

Every server block contains its own proxy_read_timeout directive set to 60s. This should be on the global http block, so that we can override proxy_read_timeout with the configuration-snippet annotation.
More preferable would be annotations for the various proxy_{read,send,connect}_timeouts

@tanner-bruce this already exists. I'm writing the docs. Thanks for the report.


Enforcer should have an acceptable "base" rule-set to provide decent initial coverage

This should be rolled into #58 so that we don't have multiple rules for enforcer and scanner.

Closing this because it should be taken care of with #58.


Add support for zig's official package manager

Description
Zig 0.11.0 ships with an official package manager. This adds a build.zig.zon and exposes the zig module & sqlite3 c library to the package manager in build.zig. Additionally, it removes the gyro.zzz file, as gyro is now archived. This does not modify the files in place for zigmod, but I don't know if those are necessary to keep around anymore.
This adds some documentation in README.md as well for installing with zig's package manager.
Checklist

[ ] I added tests for my changes and they pass

I did not add any tests as this just modifies the documentation and build system.

This is great, thanks a lot!

Maybe I'm doing something wrong but I can't build a test program that uses this new module.
I tried with my zig-sqlite-demo program (build.zig, build.zig.zon) and it fails to build:
thread 62163 panic: unable to find artifact 'sqlite'
/home/vincent/local/lib/zig/std/debug.zig:374:22: 0x37ad19 in panicExtra__anon_46632 (build)
    std.builtin.panic(msg, trace, ret_addr);
                     ^
/home/vincent/local/lib/zig/std/debug.zig:349:15: 0x343199 in panic__anon_14221 (build)
    panicExtra(null, null, format, args);
              ^
/home/vincent/local/lib/zig/std/Build.zig:1550:18: 0x2f5cbd in artifact (build)
            panic("unable to find artifact '{s}'", .{name});
                 ^
/home/vincent/dev/perso/projects/zig-sqlite-demo/build.zig:27:36: 0x2f3520 in build (build)
    exe.linkLibrary(sqlite.artifact("sqlite"));
                                   ^
/home/vincent/local/lib/zig/std/Build.zig:1639:43: 0x2e22b4 in runBuild__anon_7175 (build)
        .ErrorUnion => try build_zig.build(b),
                                          ^
/home/vincent/local/lib/zig/build_runner.zig:297:29: 0x2de082 in main (build)
        try builder.runBuild(root);
                            ^
/home/vincent/local/lib/zig/std/start.zig:574:37: 0x2c998e in posixCallMainAndExit (build)
            const result = root.main() catch |err| {
                                    ^
/home/vincent/local/lib/zig/std/start.zig:243:5: 0x2c9471 in _start (build)
    asm volatile (switch (native_arch) {
    ^
???:?:?: 0x4 in ??? (???)
Unwind information for `???:0x4` was not available, trace may be incomplete

do you an example that works ?

You're referencing the first commit of this pull request in your build.zig.zon which doesn't have have the sqlite artifact, I think the latest 6e0d6b7 should work

Oh right, duh. Thanks!

Thanks again for your work.


Rework custom heading renderer in apify-marked, update marked to fix vulnerability

We're not using the current way to pass custom IDs to headings anywhere right now, because the markup in the required format renders badly if not using apifyMarked.
This PR reworks the way custom IDs are passed to this:
 ### [](#custom-id) Heading text

becomes

Heading text

The link is rendered empty, so it takes no space, but in Docs we're using it to render the "Copy to clipboard" link and icon, and we had to add it everywhere manually up until now.
The apifyMarked renderer is not used anywhere but in Docs, so this should not break anything, but I'm releasing this as a breaking change just in case.

The third one, with the space, is sorta by design / just how Markdown works - [](#custom vibes) is not a legal markdown link, so it does not get rendered as such, and then the regular expression does not parse it properly. We could add another regular expression dealing with cases like this, but then it breaks the invariant that the rendered markdown should look the same when rendered through apifyMarked as when rendered without it, e.g. through GitHub (because there the broken link would not be parsed and would be rendered just as [](#custom vibes).
The fourth one, without the fragment, I'm not so sure about. The whole point of this is that the markdown part before the heading text should be a legal link pointing to the heading, and it is pointing to the heading only with the # symbol at the start, so only then it should be considered a custom ID.
I'd keep both of these behaviors as is, I think it's conceptually cleaner.

@fnesveda alright then :)


Add lib64 EGL libGLES_mali.so

In main MainActivity.java program. libopenclSoPaths have /system/lib/egl/libGLES_mali.so in 32bit.
Android ≥ 9 in 64bit Exynos SoC that equiped mali GPU have /system/lib/egl/libGLES_mali.so

Pull Request #80 Add this file path on this issue.

PR merged

@krrishnarraj Thanks for making such great software!


Union not working?

Just trying to implement the simple example from the Query Builder docs:
$first = DB::table('users')
->whereNull('first_name');
$users = DB::table('users')
->whereNull('last_name')
->union($first)
->get();
    $first = DB::table('content')
    ->where('type' , '=', 'webcast');

    $second = DB::table('content')
        ->where('type', '=', 'video')
        ->union($first)
        ->get();

All I end up being returned is the results of the second query. Both queries individually return the expected data.

I would like to know the answer to this as well if anybody cares to respond....

It's been several months on this. I would like to get results from several different collections that are not related. How might I do that? union should be the solution, but it's not working apparently.

Same here.
The only way I managed to make it work is not having the second query in the same fluent statement.
Using the OP example :

 $first = DB::table('content')
    ->where('type' , '=', 'webcast');

 $second = DB::table('content')
      ->where('type', '=', 'video');

$results = DB::table('content')->union($first)->union($second)->get();

+1, not working here. The union() statement are simply not compiled into the query.

It turns out that union was not what I needed anyways. To combine the two queries I ended up using merge, such as the following:
$cards = Card::all();
$stores = Retail::all();
$result = $cards->merge($stores);
return $result;

That chains the two together. I believe union would be if I needed to reference one from the other.


Update CONFIG to Foundry VTT 0.7.9

Which should be updated?
CONFIG
For which version should the class be updated?
0.7.9

Fixed by #265


Fix source / target mappings from MSH segment

Latest WiFIS 2.0.1 defines:

source: MSH-4:HD.1
target: MSH-6: HD.1

At least, the following classes must be fixed:

WiFIS.V201.BS.Service
WiFIS.V201.DT.HL72XML

Added to release 2.7


kie-issues#413: Dry run of backend changing headers of ts and js files

Guys, I'm running a small dry run of the header rename tasks just on backend folder for js and ts files.
Could you please take a look?
Please don't merge this. I'll send another one with full repo changes.
apache: format -> https://www.apache.org/legal/src-headers.html
ON the next Prs, I'll send a dry run for Java files.

Closing, as this will evolve to other PR's.


Lab 6: Deep learning for text data

NLG task that builds a language model - something like this or this.

Closed with lab 6


[fix] release: add changelog to release body with changelog-builder-action

Motivation for changes:
github releases have empty body with no notes or changelog on new versions
Detailed changes:

replace run step with changelog builder action

To Do:

verify github release notes
test changelog workflow for wiki changelog

Not sure how this will behave compared to our own changelog builder, but happy to find out. Thanks!

how does it currently generate the changelog for the wiki? just reading the commit messages since the last release? I believe this action does the same.

It only adds lines from commit messages that were prefixed by one of the compatible tags in brackets.
https://github.com/Flexget/Flexget/blob/f55093c976a26e0fd1e46d8cd43143717c3b536e/update-changelog.py#L13-L17

Plus, the wiki page can be edited manually to add changes that weren't in commit messages, or edit them. (Although I don't think we've ever done a manual edit 😝 )

looks like github releases can also be edited after the fact.

It would be nice if we could have one changelog that got copied in one
direction or another on release, but something is better than nothing, so
I'm not too concerned. If we find a clean way to embed log from wiki into
release in the future, is like to switch to that.
On Wed, Oct 19, 2022, 22:08 ksurl @.***> wrote:

looks like github releases can also be edited after the fact.
—
Reply to this email directly, view it on GitHub
https://github.com/Flexget/Flexget/pull/3576#issuecomment-1284808838,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABNV7IXHZ6JETXFGTZONELWECSQXANCNFSM6AAAAAARJL6VEU
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

I will look at it tomorrow but I think it's possible to wrangle this to handle both and replace the wiki changelog workflow


[Discussion] Changing the names and order of Theme options

As per #3460(Comment), we can discuss if we need to change to names and order for theme options or not.

Continuing discussion from the PR:
Quotes from https://github.com/commons-app/apps-android-commons/pull/3460#issuecomment-596098017:

This particular suggestion about "not showing the 'Default' option at all" is based on how Signal handles this. They recently added supported to handle the system theme on Android 10+ devices. When I use the Signal app with that change, I only see the 'Light' and 'Dark' options in the Theme setting. If it's of any help, here's the related commit: signalapp/Signal-Android@4e7a926

I used custom android skin for my Android 9 device and I liked the way app selects the theme automatically, so IMO it is better if we keep it.

I understand that this would benefit users who use a custom theme. But we would also have to think about those don't use a custom theme. For those other users, the "Default" means "Light" and that could be confusing. If there's a reliable way to identify if an Android 9 or lower device has support for a custom dark theme, then we can show the option only for those devices. Is that possible?

My preference would be for 1 if it could be done easily. Otherwise, we could just go with 3 despite the surprise I mentioned. Share your opinions/suggestions too, including possibly new ones. 🙂

I agree with you, we can educate them about this, many dark theme lovers will keep the system-wide dark theme on and it won't be a surprise for them. Also recently Whatsapp introduced dark theme for the app and after the update, it automatically changed my theme from Light to System Default (Dark) and I loved it.

I think you misunderstood what I meant by educating users. I should have been more clear, sorry. When I said "educating users", I was imagining that after the app upgrade we would show a screen that would clearly communicate to them that we are adding a "System default" option to the "Theme" option and that we would be changing the default to the newly introduced option. This way the user wouldn't be surprised when the app theme changes to their system default. FWIW, Signal introduces (educates about) new features this way to its users and I love it.
That said, it's arguable that this is overkill for what's a simple change to the default value of a preference which is actually for the better. And I agree. I just wanted to share my thoughts ☮

Github Android app also have theme options
Dark
Light
System Default
Which is quite similar to one we are using. I am not sure about pre-Android 10 devices.

Follow System works for me

Which is quite similar to one we are using.

"Follow system" sounds a lot better but whether it's quite similar to "Default" is debatable.

I am not sure about pre-Android 10 devices.

Yeah they show the same thing there too.
FWIW, WhatsApp shows just "Light" and "Dark" as options for the theme in my device (Android 5.1.1).

Rather than bikeshedding this further, I believe it would be better if we just go with the following:

Light
Dark
Follow system

Note the order there. What do you think?

I will make the change if @macgills and @misaochan approves :)

Approved

Sounds good to me.


optimize tests crashing Python in pre_release_deps_source_dist Azure job

In all recent PRs, the pre_release_deps_source_dist Azure job is failing because of multiple Python crashes that occur when the optimize tests are run.  This is the job that uses the "nightly" numpy wheel and Python 3.7.  Is the cause of these crashes known?  I created a similar build locally, and I can recreate the crashes, but the crashes occur as far back as v1.6.3, so this is apparently not the result of a recent change in SciPy.

NumPy didn't merge any substantive changes after branching recently?
I think I pinged Matt about these failures, but it probably wasn't clear from a quick glance.

In case anyone else tries to reproduce the issue: the crash appears to be triggered in the L-BFGS-B code.  Either of these produces the crash for me:
res = minimize(lambda x: (x - 2)**2, 1, method='L-BFGS-B')

res = fmin_l_bfgs_b(lambda x: (x - 2)**2, 1, approx_grad=True)

I didn't see the ping if this is not gh-14014.

This problem isn't gh-14014.  The tests in the pre_release_deps_source_dist job are causing a hard crash of the Python interpreter, e.g. locally I get
double free or corruption (out)
Aborted (core dumped)

or
munmap_chunk(): invalid pointer
Aborted (core dumped)

A git bisect with numpy shows that the commit where the crashes start occurring is https://github.com/numpy/numpy/commit/8992459df6f8558c85d5036951799b757d74b3d5.


Add uploading via proxy

I currently have to go through my college proxy to connect to the internet. This breaks the imgur upload as the proxy is not specified. Kindly make an option to specify proxy while uploading.
A better option would be to create an option to specify exact command line arguments too.

Hi @cs-mshah. What type of proxy do you use in college? Http, socks or something else?
Also, what did you mean by cmd line args?

A better option would be to create an option to specify exact command line arguments too.

Obsidian is an electron-based app. Electron has cmd-line args for setting proxy globally for the whole app. Could you please try this and tell me if it will work out?
https://github.com/electron/electron/blob/v20.1.0/docs/api/command-line-switches.md#--proxy-serveraddressport

We use Http proxy. I tried to launch obsidian using obsidian --proxy-server=address:port but the image still wasn't uploaded.

Yes, existing images are visible, which means that Obsidian does respect the proxy, but it seems that while uploading to imgur, proxy isn't used.

I am surprised with such behavior.
I am surprised, because I do not use anything special to upload images. It's just a regular fetch API provided by Chromium. If it respects proxy for loading images, why it does not respect it for uploading them? 🤔

So I got some surprising results:

Some of the old images were not visible. I tried again after connecting with proxy and they seem to appear good as before.
The upload also worked this time.

I feel that there is some issue of imgur with proxy, as I searched and found that several proxies block imgur. So if there are some domains that are free from the blocks then those could be used.
There doesn't seem to be an issue after all, but the results vary.
Can you support picgo core for upload? As I already have a tested configuration which works. All it requires is a command line argument and you can upload. It supports proxy too.

I think the reason you've seen some images behind proxy is that they were cached. Some old images you did not open for a long time were unable to load.

I tried again after connecting with proxy and they seem to appear good as before.

How did you specify the proxy settings for Obsidian?

The upload also worked this time.

Good to hear 👏

I feel that there is some issue of imgur with proxy, as I searched and found that several proxies block imgur. So if there are some domains that are free from the blocks then those could be used.

I think good VPN should work well too

Can you support picgo core for upload?

Looks like there is already a fork which supports PicGo: https://github.com/renmu123/obsidian-image-auto-upload-plugin
If PicGo is a self-hosted solution, then what is the benefit of it comparing to default pasting images to the vault directory?


InfluxDB - create database via CURL command is failing

Hi Team,
I'm trying to setup InfluxDB - 2.0.8 using Docker image. I was able to spin-up InfluxDB but not able to create database using following CURl command.
CURL command:
curl -i -XPOST http://localhost:8086/query --header "Authorization: Token MY_INFLUX_TOKEN" --data-urlencode "q=CREATE DATABASE selenium"

Error:
HTTP/1.1 200 OK
Content-Type: application/json
Date: Wed, 18 Aug 2021 17:41:43 GMT
Content-Length: 76

{"results":[{"statement_id":0,"error":"not implemented: CREATE DATABASE"}]}

How to resolve this error?

@100josearun InfluxDB 2.0 does not support creating a database from a query. "Databases" and "retention policies" have been replaced by "buckets." The following link provides information about creating a bucket:

Create a bucket

You can use InfluxQL to query data from InfluxDB 2.0, but InfluxQL is read-only (mostly) in 2.0, so you can't perform any administrative tasks like creating databases (buckets) or managing users. InfluxQL in 2.o also requires a bit of setup. Here are some links related to InfluxDB that may help:

Query data with InfluxQL
/query 1.x compatiblity API


fetches produced blocks by validator ID

Fetches produced block data by validator ID that includes relay, fee recipient, and amount


Modified copyright in license header (resolves #989)

Change copyright owner from UCSC Computational Genomics Lab to Regents of the University of California and updated date range from 2015 to 2015-2016.

Continues #989. Amended commit to add back the version.py symlink at the project root.


Update UnRegister.tsx

The documentation mention keepValid instead of keepIsValid for the unregister options, so I fixed it by putting the right option name (keepIsValid) in the code examples and the description.

thank you.

thank you.

I beg you


Segfault with an extension shipped with setup-php

#157 revealed that the TestTimeout_module test segfaults with the default installation of setup-php. Disabling all extensions shipped with setup-php fixes the issue.
It would be nice to identify the faulty extension (by re-enabling them one by one) and fix report (or better, fix) the issue.

Actually, this isn't related to extensions: https://github.com/dunglas/frankenphp/pull/165


Do escaping of HTML characters in titles

E.g. & should be converted to &...

Done as of 3232b83b210a591c154a84fae6cc1b9642040ff9


Add redirects

It seems search engines are pointing to older links so it makes sense to add redirects.
How to test
$ npm run link-docs
$ npm run build # npm run start won't work since redirect plugin only work in production
$ npm run serve

Then if you open http://localhost:3000/docs/latest/builtin-gadgets/trace/open you will be redirected to http://localhost:3000/docs/latest/gadgets/builtin/trace/open
ref: https://docusaurus.io/docs/api/plugins/@docusaurus/plugin-client-redirects

Thanks for the review!

Examples provided by users:


I'm looking for training

Hi Guys
I'm a c# developer but most of my experience is Winforms, and I have to develop an integration between our ERP and Magento
I'm looking for someone that can provide some training and/or develop some basic class that read and write data in Magento, then I'll provide to cut and copy in my solution
Feel free to email me<EMAIL_ADDRESS>for any question or proposal
Regards

Hi Sandro. We don't provide any training or custom development. If you want to develop some additional functionality and provide a PR we'll evaluate it and try to merge it in.

Hi Sandro. We don't provide any training or custom development. If you want to develop some additional functionality and provide a PR we'll evaluate it and try to merge it in.


(fspath: py.path.local) argument to BlackItem is deprecated

When running pytest on the test file src/wf_psf/tests/train_test.py, I encountered the following error:
`_______________________________________________________________________________________ ERROR collecting src/wf_psf ________________________________________________________________________________________
../.local/lib/python3.10/site-packages/_pytest/nodes.py:152: in _create
return super().call(*k, **kw)
E   TypeError: BlackItem.init() got an unexpected keyword argument 'path'
During handling of the above exception, another exception occurred:
../.local/lib/python3.10/site-packages/pluggy/_hooks.py:501: in call
return self._hookexec(self.name, self._hookimpls.copy(), kwargs, firstresult)
../.local/lib/python3.10/site-packages/pluggy/_manager.py:119: in _hookexec
return self._inner_hookexec(hook_name, methods, kwargs, firstresult)
../.local/lib/python3.10/site-packages/pytest_black.py:27: in pytest_collect_file
return BlackItem.from_parent(parent, fspath=path)
../.local/lib/python3.10/site-packages/_pytest/nodes.py:661: in from_parent
return super().from_parent(parent=parent, fspath=fspath, path=path, **kw)
../.local/lib/python3.10/site-packages/_pytest/nodes.py:275: in from_parent
return cls._create(parent=parent, **kw)
../.local/lib/python3.10/site-packages/_pytest/nodes.py:167: in _create
return super().call(*k, **known_kw)
../.local/lib/python3.10/site-packages/pytest_black.py:47: in init
super(BlackItem, self).init(fspath, parent)
../.local/lib/python3.10/site-packages/_pytest/nodes.py:718: in init
super().init(
../.local/lib/python3.10/site-packages/_pytest/nodes.py:616: in init
path = _imply_path(type(self), path, fspath=fspath)
../.local/lib/python3.10/site-packages/_pytest/nodes.py:108: in _imply_path
warnings.warn(
E   pytest.PytestRemovedIn8Warning: The (fspath: py.path.local) argument to BlackItem is deprecated. Please use the (path: pathlib.Path) argument instead.
E   See https://docs.pytest.org/en/latest/deprecations.html#fspath-argument-for-node-constructors-replaced-with-pathlib-path

Hi @jeipollack, does the automatic testing with pytest use a different Python version (3.10.5) compared to WaveDiff (3.10.12) ?

Hi @jeipollack, does the automatic testing with pytest use a different Python version (3.10.5) compared to WaveDiff (3.10.12) ?

Are you speaking about the CI testing? You can check the logs to see what versions of python was used.

Hi @jeipollack, does the automatic testing with pytest use a different Python version (3.10.5) compared to WaveDiff (3.10.12) ?

Are you speaking about the CI testing? You can check the logs to see what versions of python was used.

The pytest tests are currently running, but some files are failing the black test in the CI testing, which is not the case locally.

Okay, do you think you can figure out and resolve why they are failing or do you think you need my help?

Yes, I think I need your help because I didn't make any changes to these files, so I'm unsure why I'm encountering this error.

This is a good case to put into practice our issue template. You should share the error messages you're getting in this issue, so I or someone can have a look.

Ok this issue #118 was solved by fixing the pytest version to 7.4.4 so I will open another one to describe the new error.

It's good you found a working version, but that doesn't really answer the original question of whether the version of pytest-black available in PyPi.org is up to date with its the codebase (in its original repository). This is what Sam asked for you to check.
Basically, check if the codebase is being maintained by looking at the last updates, reported issues, the version of python or pytest. You can even try to look at the last release of the plugin to spot the cause of the error.
I've given you some clues, as a learning experience into diagnosing and debugging.
If you have questions or don't understand let me know by replying to this message.

Hi @nadamoukaddem, checking in to see how you're progressing with this check.  Do you have any updates?

okay, I see it's cumbersome. I will try to have a look, too.

Actually I don't see this message that pytest-cases is not compatible with pytest.  And the idea of you checking the distributions is not necessarily to identify a solution but to understand what the problem is caused by and then we discuss collectively solutions for which you're welcome to propose a solution if you have one.

Yes, maybe it's just the fixture that is not compatible.(link).
Also, in this link, I didn't see version 8 of pytest, and the latest release of pytest-black was in October 2020, so it's been quite a while.

The plugins pytest-cases, emoji, and xdist have been updated with their codebases. However, pytest-black has a pre-release label for the latest version (and all previous versions), while the raises plugin and pytest-cov have tags but no formal releases.

Thank you for your updates.
I had a look myself and the failure is due to the last release of pytest-black no longer being compatible with pytest 8.x. Additionally, it's concerning to note that the plugin is no longer maintained, which may pose further challenges for us in the future. Regarding fixture incompatibility, this is not the cause of the issue as the CI tests for 3.9 and 3.10 passed successfully.
I discussed this matter with Sam, and it's clear that we need to address this issue to ensure the stability and compatibility of our codebase.
At the next meeting, we can discuss options below in our plan of going forward:
Temporary Workaround: Bypass the pytest-black check in our CI workflow or using an older version of pytest like 7.4.4. This will prevent the failures from blocking our testing process while we work on a more permanent solution.
Long-term Solution: We'll need to explore alternatives to pytest-black that are compatible with pytest 8.x and actively maintained. This may involve finding a replacement plugin or possibly contributing to pytest-black's development ourselves if feasible.


Fix "spcified" to "specified" in index.md

Thank you for finding this. I think this file gets pulled from the cordova-cli repo so this typo should be fixed here: https://github.com/apache/cordova-cli/blob/b57fb4543b7a9c8a912a922f3a34d88dc5fb65a2/doc/readme.md?plain=1#L536


Add zkTree recursion circuits

There are three types of proofs in a zkTree. User proof is a zk proof to be included in the zkTree. A user proof can be generated from any circuits with any different zk types or configurations. A leaf proof is acting like a wrapper to recursively prove different types of user proof into a uniform zkTree leaf proof type. A node proof is used to recursively prove multiple leaf/node proofs and all node proofs are produced by the same zkTree node circuits. Besides verifying the children proofs, zkTree node/leaf circuits also compute the hashes of the public inputs and the hashes of circuit hashes of the children circuits. The circuit hash and input hash computed in the root node serve as the merkle roots of all user circuits and user proofs.
More implementation details can be found in this paper: https://eprint.iacr.org/2023/208

Hello!
This PR has been open for a long time, please review and update as following:

If this PR is no l longer relevant, close it.
If this PR is relevant but not read, mark it as Draft and make a comment of what else needs to be done.
Fix any issues and get it merged / reviewed by 10/27/2023.

If there's no update by 10/27/2023 this PR will be closed.

Closing per last message.


Policy fixture provisioning does not fill example data

Running go run ./service provision fixtures does not have any errors on the client side but calling ListAttributes or ListSubjectMappings after the provisioning does not return anything. The issue exists when running in CI as well. https://github.com/opentdf/platform/actions/runs/9057319882/job/24881138541?pr=759#step:42:156
Locally in the postgres logs I see
opentdf-v2-poc-opentdfdb-1   | 2024-05-13 04:29:17.189 UTC [137] ERROR:  relation "goose_db_version" does not exist at character 32
opentdf-v2-poc-opentdfdb-1   | 2024-05-13 04:29:17.189 UTC [137] STATEMENT:  SELECT tstamp, is_applied FROM goose_db_version WHERE version_id=$1 ORDER BY tstamp DESC LIMIT 1
opentdf-v2-poc-opentdfdb-1   | 2024-05-13 04:30:34.495 UTC [253] ERROR:  relation "goose_db_version" does not exist at character 32
opentdf-v2-poc-opentdfdb-1   | 2024-05-13 04:30:34.495 UTC [253] STATEMENT:  SELECT tstamp, is_applied FROM goose_db_version WHERE version_id=$1 ORDER BY tstamp DESC LIMIT 1

@elizabethhealy looks like runMigrations is false https://github.com/opentdf/platform/actions/runs/9057319882/job/24881138541?pr=759#step:42:35

@elizabethhealy looks like runMigrations is false https://github.com/opentdf/platform/actions/runs/9057319882/job/24881138541?pr=759#step:42:35

@jrschumacher thats just for the authorization namespace cause it doesnt have a db client, the attributes service registration runs them https://github.com/opentdf/platform/actions/runs/9057319882/job/24881138541?pr=759#step:42:52, as well as the subjectmapping service, namespace service etc

That's strange. runMigrations is a global config option maybe there's another defect there on the logs.

@jrschumacher @elizabethhealy
The fixtures look to be getting provisioned against the wrong schema.
2024/05/13 14:48:28 INFO 📦 running migrations in schema schema=opentdf
2024/05/13 14:48:28 INFO running migration up schema=opentdf database=opentdf

We are pulling the schema from the db config and during service registration we prefix with namespace now.
https://github.com/opentdf/platform/blob/main/service/internal/fixtures/db.go#L23
https://github.com/opentdf/platform/blob/main/service/pkg/db/db.go#L72

https://github.com/opentdf/platform/pull/794