TypeError: object.init() takes exactly one argument (the instance to initialize)

When using the latest version of the holidays, getting the following error.
File "/usr/local/airflow/.local/lib/python3.7/site-packages/prophet/hdays.py", line 683, in <module>     class TU(Turkey):   File "/usr/local/airflow/.local/lib/python3.7/site-packages/holidays/registry.py", line 178, in __init__     super().__init__(*args, **kwargs) TypeError: object.__init__() takes exactly one argument (the instance to initialize)
Please help someone to fix this issue in the holidays module

+1 -  v0.25 does't work for me, but v0.24 was OK.

+1 v0.25 doesn't work for me either

+1 Holidays v0.24 worked for me - fixed the prophet issue

This was ddressed in https://github.com/dr-prodigy/python-holidays/pull/1208
Please use v0.24 as a workaround or let me know if you need help w/ class inheritance migration.


tasks/cosa-init: Drop workaround for yum repo gpgkeys

With the inclusion of the distribution-gpg-keys rpm package in SCOS in https://github.com/openshift/os/pull/1032,
it is no longer required to fetch them from the internet.
Blocked by: https://github.com/openshift/os/pull/1032

With https://github.com/openshift/os/pull/1118 merged, I think this is good to go now


Code commit for RT-3.1 changes

Changes done

Typo in policy name , instead of fwdPolicy5 , it was wrongly set to fwdPolicy3, corrected it.
Expected traffic streams to pass are updated.
for IPv4 cases, explicit rule added to allow IPv6 between DUT port 1 and Ixia port1

/gcbrun

@sezhang2  , could you please help in merging the code. Thanks.

@sachendras Can you approve and merge the PR?

Optimized code with icmp6.

To allow NDP between NDP between ixia---port1---dut  an explicit policy term with icmp6 is added.
This ensures NDP stays Up.

Thanks.

I'm going to verify this PR. Can you please rebase it since this seems too old? Thanks.

have done rebase, can you pls try it now. Thanks

Verified on the rebased PR, see failures. Assign to Sach for coordination among vendors.

I will update the test requirements in the readme soon.
On Tue, Jan 31, 2023 at 8:46 PM Prem Singh @.***> wrote:

@.**** commented on this pull request.
In
feature/experimental/policy/policy_vrf_selection/ate_tests/base_vrf_selection/base_vrf_selection_test.go
https://github.com/openconfig/featureprofiles/pull/845#discussion_r1092760458
:

fwdPolicy5.GetOrCreateRule(1).GetOrCreateIpv4().Protocol = oc.UnionUint8(ipipProtocol)

fwdPolicy5.GetOrCreateRule(1).GetOrCreateIpv4().Dscp = ygot.Uint8(42)
fwdPolicy5.GetOrCreateRule(1).GetOrCreateAction().NetworkInstance = ygot.String("10")
fwdPolicy5.GetOrCreateRule(2).GetOrCreateIpv4().Protocol = oc.UnionUint8(ipipProtocol)
fwdPolicy5.GetOrCreateRule(2).GetOrCreateIpv4().Dscp = ygot.Uint8(46)
fwdPolicy5.GetOrCreateRule(2).GetOrCreateAction().NetworkInstance = ygot.String("10")

fwdPolicy5.GetOrCreateRule(3).GetOrCreateIpv4().DestinationAddress = ygot.String(ipv4Address)
fwdPolicy5.GetOrCreateRule(3).GetOrCreateIpv6().DestinationAddress = ygot.String(ipv6Address)
fwdPolicy3.GetOrCreateRule(3).GetOrCreateAction().Discard = ygot.Bool(true)

fwdPolicy5.GetOrCreateRule(3).GetOrCreateIpv6().Protocol = oc.UnionUint8(icmp6)
fwdPolicy5.GetOrCreateRule(3).GetOrCreateAction().Discard = ygot.Bool(false)
fwdPolicy5.GetOrCreateRule(4).GetOrCreateIpv4().DestinationAddress = ygot.String(ipv4Address)
fwdPolicy5.GetOrCreateRule(4).GetOrCreateIpv6().DestinationAddress = ygot.String(ipv6Address)

IMHO, we should follow the test requirements and HLD which requires
matches based on protocol and/or dscp/dscpset. There is no need to use
destination address as same functionality can be achieved using protocol.
—
Reply to this email directly, view it on GitHub
https://github.com/openconfig/featureprofiles/pull/845#discussion_r1092760458,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKWFCUL4WZONPLCDT3HLO4TWVHTD3ANCNFSM6AAAAAASVJPSVY
.
You are receiving this because you were assigned.Message ID:
@.***>

--
Sincerely
Sash

@sachendras  Thanks.  Kindly update the latest in readme. Thank you.

I think this is a stale PR at this moment, waiting for @sachendras to update the README.
TL,DR, matching IPv6 in the VRF selection policy or icmp is not needed for now.

Please update this PR once https://github.com/openconfig/featureprofiles/pull/1107 is merged.

https://github.com/openconfig/featureprofiles/pull/1107 is merged and code is being rewritten per new test case objective/steps. will update PR once script is ready shortly.

Test case objective/ steps are rewritten and merged thru pull#1107.  Hence a new pull#1187 is raised for revised test case.
Closing older pull#845.
https://github.com/openconfig/featureprofiles/pull/1187


I want to add About Us section in the navbar so that if someone comes to the website they can know what is this site about?

Can you please assign this issue to me? @subhadipbhowmik @sanjay-kv

kindly please assign me with this task @subhadipbhowmik i would like to contribute

@subhadipbhowmik you didn't assign this issue


Updated app.css

edited cs code such that Hover state has only one stroke - red. without the gray.

This doesn't achieve the desired effect:

The css-group defines more than just the border, so removing the button entirely causes this regression.

fixed by #82

@masterismail - I'm starting a discord channel to Console OSS devs. This will be used to communicate new tasks, roadmap/ plans and also enable better communication between devs that contribute regularly. If you are interested in being a part of this send me your discord handle to anil [at] akash.network or share it here.

sure ,here's my discord handle : masterismail#7041


Inspect link open an empty window after upgrade chrome.

I read a lot of closed issues about this problem,but none works for me.
before upgrade

Chrome : 45.0.2454.101 (64-bit)
OS: OS X EI Capitan 10.11

after

Chrome : 46.0.2490.71 (64-bit)
OS: OS X EI Capitan 10.11

By the way, I didn't use proguard.
Stetho logs as below
10-21 23:07:19.874 8485-8506/? I/stetho: Listening on @stetho_com.ailian.healthclub_devtools_remote

Seems everything is ok.
When I close the empty window.
10-21 23:08:32.346 8633-10178/com.ailian.healthclub:leakcanary W/stetho: Client closed connection: org.apache.http.ConnectionClosedException: EOF
10-21 23:09:01.180 8485-11518/com.ailian.healthclub W/stetho: Client closed connection: org.apache.http.ConnectionClosedException: EOF

But the strange thing is when I connect to my colleague's PC, it works fine.
I don't know where the problem is.

Screenshot

Sorry, I solved it.
Damn GFW.Waste me a lot of time.
https://github.com/GoogleChrome/ADBPlugin/issues/14


refactor(scalable): :recycle: override difference and implement in PitchClass

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

6 of 6 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
865

Relevant Lines:
865

💛  - Coveralls


RGBW support

I have some bulbs that show up as RGBW:
supported_color_modes:
  - color_temp
  - rgbw
color_mode: rgbw
brightness: 255
color_temp_kelvin: null
color_temp: null
hs_color:
  - 193.176
  - 100
rgb_color:
  - 0
  - 199
  - 255
xy_color:
  - 0.147
  - 0.252
rgbw_color:
  - 0
  - 199
  - 255
  - 0
supported_features: 4

When applying presets to these bulbs, the colors are very washed out as it is applying 100% white as well as color. If I click the bulb in UI, I have to manually lower the "White Brightness" to 0 to get the proper color.
Let me know if you need any other info, I appreciate your work, thank you!

This integration can't fix those RGBW lights of undisclosed type.
All it does is wrap light.turn_on with some comfort features.
Whatever issue it is you might be seeing is entirely unrelated to this


Fixed the host check having an OR-check when it should be an AND-check

A fix for issue #22 .

@jonhattan any chance to get this merged?

The fix is correct for the code, but not for the error message. Anyway merging. I'll fix the error message afterwards.
Thanks!


1276 mwcore navigation

IMPORTANT: Where possible all PRs must be linked to a Github issue
Fixes #1276
Checklist

[x] I have written Unit tests for any new feature(s) and edge cases for bug fixes
[x] I have added any strings visible on UI components to the strings.xml file
[ ] I have updated the  CHANGELOG.md file for any notable changes to the codebase
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
[x] I have built and run the fhircore app to verify my change fixes the issue and/or does not break the app

should we change TracingAndAppointment to Appointment` throughout? I'm not sure if we should be we changed the feature to name

@AbdulWahabMemon thoughts on this?

changes pushed

@AbdulWahabMemon Kindly delete the mwcore configs from the resources package. We are already have the configs in this repository fhir-resources. We have also migrated the common configs to the assets directory update any of your tests that were using configs from the resources directory.

Hey guys, I was trying out the PR and it looks great. I had a few comments:

Change the nav bar title from "Malawi Core" to "Tingathe".
In the nav bar, the feature titled "HIV" should be labelled "Client List" and so should the toolbar title of it's activity.
In the nav bar, change "Appointment" to "Appointments".
In the nav bar, remove "Tracing" and instead add, "Phone Tracing" as one feature and "Home Tracing" as another feature.
Change the "Phone Tracing" icon to the call icon
Change the "Home Tracing" icon to the house icon
Change the "Appointments" icon to the event icon

Thanks @KhumboLihonga for the feedback @AbdulWahabMemon I think you can address most of the comments with configs with few updates in the code.


Ticket 85

This has all the changes requested in Trac Ticket #85, including the internal links.  It incorporates all of Tanya's changes which were in her previous pull request for this ticket.

That looks good. Thanks very much.


[v0.2.28] package.json from github differs from the npmjs one

I noticed that the package.json differs from the one on npmjs.
For example the dependency 'request' is not in the github repo but in the npmjs code
Can you fix this?

The repo is currently several commits ahead of the 0.2.28 release (about 77 at the time of writing this). There were some issues with the 0.2.28 release related to browser compatibility and I think the idea is to get those sorted out before releasing the next version. Progress has unfortunately been stalled at that process so the next version won't be out for a while.
If you need the repo to be at 0.2.28, checkout the appropriate tag: git checkout tags/0.2.28 -b 0.2.28 (creates a new branch called 0.2.28 and reverts to that tag).

Ok, thanks!


Self Video looking negative effect.

Checking on the HTC devices. Self video view looking like negative effect

Hi there,
Thanks for reporting this. Can you give us more details of the device you are using and the version of the SDK?

My mistake, i was on hurry that day.
On Mon, Apr 24, 2017 at 8:09 PM, Roberto Perez<EMAIL_ADDRESS>wrote:

Hi there,
Thanks for reporting this. Can you give us more details of the device you
are using and the version of the SDK?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/opentok/opentok-android-sdk-samples/issues/106#issuecomment-296689484,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMtgsZ8W2k03gMgkj-qDS3bjCG0Wgorjks5rzLQ2gaJpZM4MTvsy
.

--
Thanks & Regards
Vivek Kumar

Got it, closing the issue then.


chore: fix stargazer chart with star-history.com

What's the purpose of this PR
Fix the README stargazer

I have read the CLA Document and I hereby sign the CLA

recheck

Codecov Report

Merging #4441 (d4fc608) into master (6a89ffb) will increase coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #4441      +/-   ##
============================================
+ Coverage     53.38%   53.41%   +0.03%
  Complexity     2688     2688
============================================
  Files           490      490
  Lines         15252    15252
  Branches       1586     1586
============================================
+ Hits           8142     8147       +5
+ Misses         6553     6549       -4
+ Partials        557      556       -1

Impacted Files
Coverage Δ

...mework/apollo/portal/component/PortalSettings.java
65.62% <0.00%> (+7.81%)
:arrow_up:

Continue to review full report at Codecov.

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


Users/ersciple/m105publish

Is the fix already in master?  If not, link the bug tracking that work.
yes
What testing was done?
E2E testing on machines with different PowerShell versions (3, 4, and 5), tested on Windows machines with different locales (en-US and zh-CN), and tested OSX.
Does this fix change the AT/DT contract?
No
Does this fix include any database servicing changes?
No
Were tests added to cover this to make sure it doesn't regress?  If not, link a bug or user story tracking that work.
Yes

This went in with a different PR


App is adding user to Team but not creating issue.

Hi! First, I'd like to say that this is an awesome app! so Thanks! 👏🏼
While testing to implement the app, I noticed that users were getting added to the team I configured, but issues were not getting created at the onboarding repo. I have a feeling this might be a misconfiguration on my side so figured I would ask.
Here is my configuration so I have a few questions:

onboardRepo is meant to be a different repo correct?
issuePath where should this template be located ? within the onboardRepo or within the org-settings? (tried both but neither worked 😅 ).

Any help on what I must be misconfiguring is appreciated
In my Organization the 'Onboard New Org Member' GitHub App has access to 'All repositories'
# Configuration for onboard-new-org-member

# Org to run the bot on (must be lowercase)
orgName: myorg-name

# Settings repo
settingsRepo: org-settings

# Name of team to add new members to (must be lowercase)
defaultTeam: internal-devtest

# Repo to add Onboard Issue to (name it whatever you'd like)
onboardRepo: dev-onboarding

# Phrase/Word to replace with the new member's handle
replacePhrase: NEW_USER

# Path to Issue template
issuePath: .github/templates/on-board-template.md

# Users/Groups that should be cc'ed on the issue. Should be users/groups separated by a space.
ccList: '@user1'

Same for me - defaultTeam addition worked, but not the issue creation.

@amenocal - the issuePath template is in the settingsRepo (org-settings).


[opengl] save settings

Need to save current settings

[x] Paths to ff8
[x] Paths to upscales.
[x] Current path
[x] Current language
[x] Current field

I also am saving the menu toggles.


Update web-protection-overview.md

Updated Custom policy to Customblocklist

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/6859

@JohanFreelancer9 Please copy review

@Dansimp Please merge this PR. Thanks


Unify last updated font size

Description:
Last updated font size is good on Dashboard card
Let's update font to same size elsewhere (history view, full card).

iOS card: https://github.com/ruuvi/com.ruuvi.station.ios/issues/2175

QA passed


Doesn't build on armhf

I'm getting a following when trying to build on armhf:
go build ./apps/docker-slim
# github.com/cloudimmunity/system
../system/system_linux.go:24: cannot use unameInfo.Sysname (type [65]uint8) as type [65]int8 in argument to nativeCharsToString
../system/system_linux.go:25: cannot use unameInfo.Nodename (type [65]uint8) as type [65]int8 in argument to nativeCharsToString
../system/system_linux.go:26: cannot use unameInfo.Release (type [65]uint8) as type [65]int8 in argument to nativeCharsToString
../system/system_linux.go:27: cannot use unameInfo.Version (type [65]uint8) as type [65]int8 in argument to nativeCharsToString
../system/system_linux.go:28: cannot use unameInfo.Machine (type [65]uint8) as type [65]int8 in argument to nativeCharsToString
../system/system_linux.go:29: cannot use unameInfo.Domainname (type [65]uint8) as type [65]int8 in argument to nativeCharsToString

Sorry about that! And thank you for trying to build it on a new platform! Do you have a netbook? Can you share a bit more about your environment?

Sure,
I have armhf all over the place : Chromebook 2 arm and odroid-U3 platforms, all running ubuntu 15.10 and go 1.5.1. Can't beat 9 hours of battery on chromebook :D

This is due to architecture differences... These tickets (in Go and Docker) provide a good overview: https://github.com/golang/go/issues/13318 , https://github.com/docker/docker/pull/18064 Doing the same thing Docker did should fix it :)
The syscall/seccomp functionality needs enhancements to support ARM architectures... That needs to be done in the "system" library (which covers only ArchName386 and ArchNameAmd64 at this point in time).

This should be fixed, but there are a couple of other arm build errors that need to be addressed too:
docker-slim/sensor/monitors/fanotify/monitor.go:30: nd.Mark undefined (type *fanotify.NotifyFD has no field or method Mark)
docker-slim/sensor/monitors/ptrace/monitor.go:91: regs.Orig_rax undefined (type syscall.PtraceRegs has no field or method Orig_rax)
docker-slim/sensor/monitors/ptrace/monitor.go:99: regs.Rax undefined (type syscall.PtraceRegs has no field or method Rax)

@kcq @sokoow
docker-slim/sensor/monitors/fanotify/monitor.go:30: nd.Mark undefined (type *fanotify.NotifyFD has no field or method Mark)
caused by missing bitbucket.org/madmo/fanotify/fanotify_arm.go or bitbucket.org/madmo/fanotify/fanotify_arm64.go
docker-slim/sensor/monitors/ptrace/monitor.go:91: regs.Orig_rax undefined (type syscall.PtraceRegs has no field or method Orig_rax)
docker-slim/sensor/monitors/ptrace/monitor.go:99: regs.Rax undefined (type syscall.PtraceRegs has no field or method Rax)
I did not  found a solution now.

Moving ARM support to the Wishlist. Ping me if you want to help :-)

Ping? :)

I have built and tested out dockerslim on armhf, but had a make a few changes specific to the arm architecture in the ptrace module, as well as some of the packages in the vendor directory. Do you have any regression tests suites to run to ensure that nothing has broken for the x86 builds?

Thank you for working on this Josh! This is great! There's no automated regression test suite at this point in time, but there's a set of sample applications you can minify and confirm everything is still working. The examples are now in a separate repo ( https://github.com/docker-slim/examples ). Checking if the minified ruby_ubuntu, ruby_rails_standard, ruby_alpine, python_alpine and python_centos7_py27_standard apps work is a good way to confirm. I'll be happy to help with the process.


Would it be possible to have Enduring Spells effect Magus weapon enhancements again?

As the title says, really miss it lasting 24h

Really would like that feature back too. That and the same thing to Sword of Heaven. Thanks a lot! :)

This please!

Maybe back with mythic ability(enduring spells for all abilities) or option? I would even agree to that.


TR3 - Lara Texture Corruption

After switching Lara's skin, her right-hand has texture corruption when holding the shotgun, harpoon, MP5, rocket launcher and grenade launcher.

Left hand also affected for flares in London at least.

Resolved under #263.


⚠ Fortnite has degraded performance

In a95561f, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1648 ms

Resolved: Fortnite performance has improved in 0add1b5.


npm audit reports critical vulnerability when @openzeppelin/cli is installed

I've skimmed through the source, I wouldn't say you're affected but you'd probably want to remove the dependency and implement those functions yourself
Advisory: https://npmjs.com/advisories/1507
Usages

https://github.com/OpenZeppelin/openzeppelin-sdk/blob/c68234271a4a6e29b989fe1bac1902a246a298b3/packages/cli/src/models/dependency/Dependency.ts#L58
https://github.com/OpenZeppelin/openzeppelin-sdk/blob/578e4f27702d66dcbc23292950a221ea00b008c8/packages/cli/src/models/config/TruffleConfig.ts#L54

Looking at the dependency itself, I'd suggest implementing exec commands in your sdk and getting rid of the dep as it doesn't seem to be actively maintained.
Dep source: https://github.com/Manak/npm-programmatic/blob/master/index.js
Initially reported on Telegram: https://t.me/zeppelinos/12443

Hi @shark0der!
Thanks so much for reporting it! The project owner will review and triage this issue during the next week.

Thanks for reporting @shark0der. I looked into this and I don't think it's a vulnerability for OpenZeppelin CLI users, because the places where there could be command injection in the user's machine are controlled by the user.

Hi @frangio! Indeed, like I said before, it doesn't seem like there's an impact (unless of course there are specific scenarios for some users), however, having npm audit report a critical vulnerability after the installation doesn't look pretty and might scare some users. That being said, fixing this doesn't need a very high priority but it would be nice to have it fixed sooner than later.


hw_config incompatible with old mbed code

Hi,
I have started comparing the code in this repository with the old mbed code, specifically with this revision: https://os.mbed.com/users/benkatz/code/Hobbyking_Cheetah_Compact_DRV8323/
This is part of an effort to port an existing modified variant over the version in this repository.
Before testing anything, I have spotted several changes to the pinout of the STM32 that left me puzzled:
Different Timer Connections
This is how the timers are currently set up in hw_config.h:
#define TIM_CH_U		TIM_CHANNEL_1		// Terminal U timer channel
#define TIM_CH_V		TIM_CHANNEL_2		// Terminal V timer channel
#define TIM_CH_W		TIM_CHANNEL_3		// Terminal W timer channel

Looking at the .ioc, I get this mapping:

Channel
Pin

TIM1_CH1
PA8

TIM1_CH2
PA9

TIM1_CH3
PA10

The Config/hw_config.h from the old mbed version defines this mapping:
#define PIN_U PA_10
#define PIN_V PA_9
#define PIN_W PA_8

Which means we end up with swapped pins for U and W:

Channel
Old Pin
New Pin

U
PA10
PA8

V
PA9
PA9

W
PA8
PA10

Different ADC Channel use
Similarly, the ADC channels have been swapped as far as I can tell.
This is how the ADC channels are currently set up in hw_config.h:
#define ADC_CH_MAIN		hadc1				// ADC channel handle which drives simultaneous mode
#define ADC_CH_IA		hadc1				// Phase A current sense ADC channel handle.  0 = unused
#define ADC_CH_IB		hadc2				// Phase B current sense ADC channel handle.  0 = unused
#define ADC_CH_IC		0				// Phase C current sense ADC channel handle.  0 = unused
#define ADC_CH_VBUS		hadc3				// Bus voltage ADC channel handle.  0 = unused

Looking at the .ioc, I get this mapping:

Channel
Pin

ADC1_IN10
PC0

ADC2_IN11
PC1

ADC3_IN0
PA0

The hw_setup.cpp from the old mbed version defines this mapping through register configuration:
     ADC1->SQR3 = 0x000000A;                                    // use PC_0 as input- ADC1_IN0
     ...
     ADC2->SQR3 = 0x0000000B;                                   // use PC_1 as input - ADC2_IN11
     ...
     ADC3->SQR3 = 0x00000000;                                   // use PA_0, - ADC3_IN0

So far, so good, the name/pin mappings are identical.
However, the code in Calibration/calibration.cpp shows an interpretation of the ADC channels that is different from the one in the new repository:
    controller->i_b = I_SCALE*(float)(controller->adc2_raw - controller->adc2_offset);    //Calculate phase currents from ADC readings
    controller->i_c = I_SCALE*(float)(controller->adc1_raw - controller->adc1_offset);
    controller->i_a = -controller->i_b - controller->i_c;

So we have the following mismatch

ADC
Old Meaning
New Meaning

1
I_C
I_A

2
I_B
I_B

3
V_BUS
V_BUS

Questions

Are there reasons for these changes or is this a bug?
Do I have to modify my hw_config.h before attempting to run the firmware on a driver that worked with the old mbed firmware?

Sorry for the long post btw, I wanted to properly show where in the code my confusion comes from ;-)

Hello, I would like to ask by the way, the new version I used has changed some parameters, but the motor does not run, and the calibration cannot be carried out. Did your transplanted program start running?

Did your transplanted program start running?
I am still preparing for the first tests, verifying as much as possible before even starting to adapt the new version of the firmware for my needs.
I will post an update here if and when I get it working.

ok，Thanks
------------------ 原始邮件 ------------------
发件人:                                                                                "bgkatz/motorcontrol"                                                                                @.>;
发送时间: 2022年7月14日(星期四) 晚上8:06
@.>;
<EMAIL_ADDRESS>主题: Re: [bgkatz/motorcontrol] hw_config incompatible with old mbed code (Issue #8)
Did your transplanted program start running?
I am still preparing for the first tests, verifying as much as possible before even starting to adapt the new version of the firmware for my needs.
I will post an update here if and when I get it working.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

The naming convention in the original firmware didn't match the solder mask labels on the PCB.  The new firmware was corrected to match the PCB.  Both work fine on the original hardware.

The naming convention in the original firmware didn't match the solder mask labels on the PCB. The new firmware was corrected to match the PCB. Both work fine on the original hardware.

Thank you for confirming again that porting the new firmware to the old boards is essentially safe and possible. I think this Issue can then be closed, I will go ahead with my plan.
Since I have now realized that the board I have to target is actually using DRV8323RH (so not the SPI variant), there are a few things I need to adapt in any case.

Hello, I would like to ask by the way, the new version I used has changed some parameters, but the motor does not run, and the calibration cannot be carried out. Did your transplanted program start running?

@Erick-Zhf After making the required adaptions tor my driver PCB (notably for the DRV8323RH), the calibration worked pretty much instantly. I needed to reduce KI_D and KI_P in the firmware a bit (and of cause update the number of pole pairs and the gear reduction ratio), otherwise most things just worked from the start.
I guess if you are still having trouble, that discussion should move to a new issue.

OK，I have almost the same problem as you, the calibration has always been a problem, I am now starting to debug the old version
------------------ 原始邮件 ------------------
发件人:                                                                                "bgkatz/motorcontrol"                                                                                @.>;
发送时间: 2022年7月25日(星期一) 下午5:11
@.>;
<EMAIL_ADDRESS>主题: Re: [bgkatz/motorcontrol] hw_config incompatible with old mbed code (Issue #8)
Hello, I would like to ask by the way, the new version I used has changed some parameters, but the motor does not run, and the calibration cannot be carried out. Did your transplanted program start running?
@Erick-Zhf After making the required adaptions tor my driver PCB (notably for the DRV8323RH), the calibration worked pretty much instantly. I needed to reduce KI_D and KI_P in the firmware a bit (and of cause update the number of pole pairs and the gear reduction ratio), otherwise most things just worked from the start.
I guess if you are still having trouble, that discussion should move to a new issue.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


Multiple repo widgets on the same dashboard

Hi,
I am trying to add multiple repo widgets on a dashboard. I managed to get 2 repo widgets on the UI however the two widgets are linked now. How can I create multiple repo widgets? Is it a UI change only or would I need to look at the collectors as well?
Any help would be appreciated!
Thanks
Hwee

@HweeWong #2481 - someone just asked this a couple days ago. Please look at the issue list before creating new issues.


Return value as Python type

Should return value e.g. the created account string (which could be put also inside a dictionary) or the matches as Python List:
Could be similar to the following example for returning a python List of dictionaries:
    // Convert MarketOutput to Python List of Dict representing matches
    let pyMatches = PyList::new(py,
        matches.matches.into_iter().map(|mtch| {
            let dict = PyDict::new(py);
            dict.set_item("time", mtch.energy_kwh);
            dict.set_item("bid_id", mtch.bid_id);
            dict.set_item("ask_id", mtch.ask_id);
            dict.set_item("bid_actor", "");
            dict.set_item("ask_actor", "");
            dict.set_item("bid_cluster", 0);
            dict.set_item("ask_cluster", 0);
            dict.set_item("energy", mtch.energy_kwh);
            dict.set_item("price", mtch.price_euro_per_kwh);
            dict.set_item("included_grid_fee", 0);
            dict
        })
    );

As both types are different the account string or the match list could be it self returned within a dictionary with a specific key, i.e. {"account": account_string} or {"matches": pyMatches}
Originally posted by @j-ti in https://github.com/BESTenergytrade/integritee-cli-py/pull/3#discussion_r1302951203

We agreed on returning the PyDict by each command function and just forward it to the dispatching run_cli return

Also Errors should be forwarded, in that turn also commands need to preferably return Error rather than to panic! as these cannot be handled well, e.g. https://github.com/olisystems/BEST-Energy/issues/53


[FEAT]: Create an online folder to save Images for training/validation purpose

Description
Create an online folder in the Cloud to save Images for training/validation purpose
Additional Information
No response

Do i have to create the folder in the project repo ?

Do i have to create the folder in the project repo ?

Hello, no, you don't have to. If it was a simple folder in the repo, I would have done that myself 😄
I was asking @gmacario to create a folder in the cloud to add there the images we will take to train the NN.
Thank you for trying to help!

@gteti based on a preliminary investigation I found a few options to provide you with what you would need.
Please read this article and let me know which option you prefer.

For now, since we don't have all things clear yet, we can use our own storage folder with a "container" where to try to run the code. I've seen on azure calculator that a North Europe machine for 31 days is 79$. What do you guys think ?
https://azure.microsoft.com/it-it/pricing/calculator/

I've also asked more info on my created issue in depthai-ml-training.

What do you guys think ?

I think that one of the reasons why the OpenCV contest organizers decided to grant us 900+ US$ of credits in Azure services is that we should need them to complete Phase 2 😉
That being said, I am fine if you do not want to activate any specific service for the moment.
However, in case any of us would need some in the future, I have created in advance #98 to request your individual Azure account  name (that is, the user name) and a public SSH key so I can grant you the service you will request.
Closing for now, please reopen or (better) create a more specific issue in the future


CIP-0030: API to enumerate avilable wallets

Please provide API to enumerate wallets available for dApp. Right now the only way is to iterate over properties in window.cardano but this lead to duplicated entries if you have Eternl installed as due to avoid breaking changes to existing apps it's exposed twice via: window.cardano.ccvault and window.cardano.eternl

I do something like this https://github.com/ADAPhilippines/Bifrost/blob/203b0a54c2b7ff691274ce45dd2c5747ebe7fb52/src/Bifrost.ts#L34

I do something similar but then ccvault/eternl is twice as it's exposed as window.cardano.ccvault and window.cardano.eternl. And yoroi is not there as they don't have icon property - but maybe that's actually good because they are missing a lot of APIs

Please provide API to enumerate wallets available for dApp. Right now the only way is to iterate over properties in window.cardano but this lead to duplicated entries if you have Eternl installed as due to avoid breaking changes to existing apps it's exposed twice via: window.cardano.ccvault and window.cardano.eternl

Talk to CARDANO DEV via live chat on https://direct.lc.chat/14345826

I do something similar - but much less accurate - https://github.com/GGAlanSmithee/use-cardano/blob/main/src/hooks/use-wallet-providers.tsx#L17. I think this issue is valid, what would the path be to move this to an actual CIP or ammendment? Just make a CPS?

@GGAlanSmithee the last resolution of the CIP editors (it was back when minutes were still being transcribed into the repo, so you can probably find it) was for additional features for CIP-0030 to be submitted as CIPs with new numbers, because (as I remember; @KtorZ may remember in more detail or correct me if I'm wrong):

the CIP-0030 document was getting too big
tagging additional features for the API with CIP numbers would provide a convenient way of indicating support for a new feature set (i.e. indicating "supports CIP-0XXX" rather than having to describe the feature)

I don't work in this area so I can't advise about the right way of enumerating the available wallets.  If there is a straightforward implementation that everyone can agree on quickly to do that, it should definitely be a CIP (a Path to Active that drives adoption by wallet) rather than a CPS (with only a vague incentive for wallets to support the enumeration somehow).

@rphair many thanks for your quick reply. I will try to move this to a CIP then, since I think it could easily be done in the bridge API itself with introspection, much like @Mercurial does it.
Regarding #151 I'm a bit more hesitant because I am not a wallet provider, but rather a consumer and idk the implications for the different implementations. As a consumer, i like how nami does it though.


Improve error message for missing field name in tuple

I think the parsing error for this could be improved.
The last field in my tuple is missing its name:
actor main(env):
    abc = (a="foo", "bar")

kll@Boxy:~/terastream/acton$ actonc examples/foo.act
Building file examples/foo.act

ERROR: Error when compiling foo module: Syntax error
examples/foo.act:2:7:
  |
2 |     abc = (a="foo", "bar")
  |         ^^
unexpected "= "
expecting ':', ';', call arguments, slice/index expression, comma, end of line, if clause, or operator

and it's pointing to what?

@sydow I think this counts as a parser error, like you were looking for in #732, right? :)

Yes, this is certainly a good example of a terrible error message. I will be glad to try to fix it as soon as I know what syntax should be allowed. For example, this is accepted by the compiler:
actor main(env):
abc = ("foo", b="bar")
and abc gets type (str, b : str), following the idea  (for function parameters) that positional elements precede keyword ones. What are the rules for tuples with (some) named fields? Maybe this was discussed at some point, but I certainly don't remember. I am sure @nordlander does.

@sydow I have not been part of any such discussion - I don't think we've discussed the details of it in particular. As far as I know, the design that we've implemented is to allow tuples with either named fields or no names. Thus, mixing by providing names for some fields but not others is invalid and this seems to be enforced now (although perhaps more by happenstance!?). I don't really have an opinion on whether it makes sense to allow mixing or not. Since names are just an alias that get replaced by the index I guess it's technically possible to mix. What are the drawbacks to mixing?
In the current situation with named / unnamed mix being forbidden, the best error message is the one that guides the user to do the right thing and fix their code.
I think this looks great:
ERROR: Error when compiling foo module: Syntax error
  ╭─[ examples/foo.act:2:7 ]
  │
2 │     abc = (a=1, b=2, c=3, 4)
  .            ─┬─  ─┬─  ─┬─  ┬
  .             │    │    │   ╰──── This field does not have a name
  .             │    │    │
  .             ╰────┴────┴──────── These fields have a name
  ╰──
Error: Tuple contains a mix of named and unnamed fields.
Hint: All fields in a tuple must be named or unnamed. Either add names to all fields or remove names to make the tuple unnamed.

granted, it does quite a bit more stuff than our current error messages but I think it's the direction we should move in. This whole Error / Hint thing is one that Postgresql has implemented and it's been sooooo useful so I'd really like to have that. The ascii style drawings of things is inspired by both Haskell's error messages but also Ariadne (see https://github.com/actonlang/acton/issues/490).

Indeed, the idea was that tuples and function arguments would support exactly the same syntax.
Unfortunately, when we built the first Acton parser we attempted to follow Python's official but messy grammar to the letter, and while we've dropped that goal in later revisions I think we're still struggling with traces of Python grammar leftovers in the tuple and argument parser definitions.
Anyway, what I think we want to support is the following. A tuple --  or a function argument list -- consists of (from left to right):

Zero or more positional arguments
An optional star argument
Zero or more keyword arguments
An optional double-star argument

So the expression (a="foo", "bar") is a syntactical error, while ("foo", b="bar") is correct. Well, this is the idea, at least.
In addition, the constraint solver will match tuple types as well as function arguments by applying the following adjustments:

Surplus given positional arguments will count as keyword arguments in their expected order, unless a star argument is also expected
A deficit of given positional arguments will be accepted provided that the missing argument types are all optional
Same for missing keyword arguments

(It is the implementation of these three rules, including the corresponding term rewrites, that I'm working on right now.)
-- Johan

On 2 Oct 2023, at 08:31, sydow @.***> wrote:
Yes, this is certainly a good example of a terrible error message. I will be glad to try to fix it as soon as I know what syntax should be allowed. For example, this is accepted by the compiler:
actor main(env):
abc = ("foo", b="bar")
and abc gets type (str, b : str), following the idea (for function parameters) that positional elements precede keyword ones. What are the rules for tuples with (some) named fields? Maybe this was discussed at some point, but I certainly don't remember. I am sure @nordlander does.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Hej!
Detta stämmer med min vaga minnesbild och är nog vad vi implementerat — men med ett bedrövligt felmeddelande i Kristians exempel. Ska försöka fixa det även om det inte blir så bra som Kristians förslag.
För närvarande blir det litet mindre dåligt om man gör samma fel i en funktionsapplikation:
def f(x,y):
return x+y
actor env(main):
print(f(x=3,y))
ger felmeddelandet
ERROR: Error when compiling tuple module: Syntax error
newexamples/tuple.act:5:17:
|
5 |     print(f(x=3,y))
|                 ^
unexpected 'y'
expecting "**" or a closing parenthesis
Inte så dumt; efter ett kwd-argument kan endast följa ett **-dito.
Björn

On 2 Oct 2023, at 10:19, Johan Nordlander @.***> wrote:
Indeed, the idea was that tuples and function arguments would support exactly the same syntax.
Unfortunately, when we built the first Acton parser we attempted to follow Python's official but messy grammar to the letter, and while we've dropped that goal in later revisions I think we're still struggling with traces of Python grammar leftovers in the tuple and argument parser definitions.
Anyway, what I think we want to support is the following. A tuple -- or a function argument list -- consists of (from left to right):

Zero or more positional arguments
An optional star argument
Zero or more keyword arguments
An optional double-star argument

So the expression (a="foo", "bar") is a syntactical error, while ("foo", b="bar") is correct. Well, this is the idea, at least.
In addition, the constraint solver will match tuple types as well as function arguments by applying the following adjustments:

Surplus given positional arguments will count as keyword arguments in their expected order, unless a star argument is also expected
A deficit of given positional arguments will be accepted provided that the missing argument types are all optional
Same for missing keyword arguments

(It is the implementation of these three rules, including the corresponding term rewrites, that I'm working on right now.)
-- Johan

On 2 Oct 2023, at 08:31, sydow @.***> wrote:
Yes, this is certainly a good example of a terrible error message. I will be glad to try to fix it as soon as I know what syntax should be allowed. For example, this is accepted by the compiler:
actor main(env):
abc = ("foo", b="bar")
and abc gets type (str, b : str), following the idea (for function parameters) that positional elements precede keyword ones. What are the rules for tuples with (some) named fields? Maybe this was discussed at some point, but I certainly don't remember. I am sure @nordlander does.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Ja, för anrop blir felmeddelandet mycket bättre! Och man borde kunna parsa tupler på precis samma sätt, tycker jag. Det som möjligen krånglar till det är att grammatiken tillåter att man utelämnar parenteserna runt en tupel på vissa ställen. Men det borde gå att begränsa om det ställer till problem. Det enda ställe där jag spontant tycker att tupler utan parentes har någon som helst poäng är vid retur av multipla funktionsresultat. Och även där är vinsten ganska blygsam.


Can't resolve components/PoweredBy/PoweredBy.tsx

Describe the bug 🐛
I can't import widgets with webpack, I get the following error:
Failed to compile.

./node_modules/instantsearch.js/es/widgets/powered-by/powered-by.js Module not found: Error: Can't resolve '../../components/PoweredBy/PoweredBy.tsx'

To Reproduce 🔍
Steps to reproduce the behavior:
yarn add instantsearch.js algoliasearch
Add the following to my JS bundle, as explained in the docs:
import algoliasearch from "algoliasearch/lite";
import instantsearch from "instantsearch.js";
import { searchBox, hits } from "instantsearch.js/es/widgets";

Try to bundle with webpack (I don't use TypeScript)
I get the following error:
Failed to compile.

./node_modules/instantsearch.js/es/widgets/powered-by/powered-by.js Module not found: Error: Can't resolve '../../components/PoweredBy/PoweredBy.tsx' in '/home/hans/projects/myproject/node_modules/instantsearch.js/es/widgets/powered-by'
@ ./node_modules/instantsearch.js/es/widgets/powered-by/powered-by.js 3:0-65
@ ./node_modules/instantsearch.js/es/widgets/index.js
@ ./frontend/packs/application.js
@ multi (webpack)-dev-server/client?http://localhost:3035 ./frontend/packs/admin.js

It works if I comment out this line in node_modules/instantsearch.js/es/widgets/index.js
// export { default as poweredBy } from './powered-by/powered-by';

I think you forgot a tsx import ;)
Expected behavior 💭
I should be able to use instantsearch.js with this basic setup.
Environment:

OS: Ubuntu
Browser: Firefox
Version 64

Thank you for reporting the issue.
#3530 should fix this.

Released in 3.1.1.


❓ Questions / Help / Support VAD not working

❓ Questions and Help
I've been trying to run silero vad using golang. First I tried it on windows but the onnxruntime was just not working so I switched to wsl where the runtime was working correctly but the vad isn't working correctly. Using the examples in the docs
	log.Println("Working with vad 5.0")

	sd, err := speech.NewDetector(speech.DetectorConfig{
		ModelPath:            "./silero_vad.onnx",
		SampleRate:           16000,
		Threshold:            0.5,
		MinSilenceDurationMs: 10,
		SpeechPadMs:          30,
	})
	if err != nil {
		log.Fatalf("failed to create speech detector: %s", err)
	}

	if len(os.Args) != 2 {
		log.Fatalf("invalid arguments provided: expecting one file path")
	}

	f, err := os.Open(os.Args[1])
	if err != nil {
		log.Fatalf("failed to open sample audio file: %s", err)
	}
	defer f.Close()

	dec := wav.NewDecoder(f)

	if ok := dec.IsValidFile(); !ok {
		log.Fatalf("invalid WAV file")
	}

	buf, err := dec.FullPCMBuffer()
	if err != nil {
		log.Fatalf("failed to get PCM buffer")
	}

	pcmBuf := buf.AsFloat32Buffer()

	segments, err := sd.Detect(pcmBuf.Data)
	if err != nil {
		log.Fatalf("Detect failed: %s", err)
	}

	for _, s := range segments {
		log.Printf("speech starts at %0.2fs", s.SpeechStartAt)
		if s.SpeechEndAt > 0 {
			log.Printf("speech ends at %0.2fs", s.SpeechEndAt)
		}
	}

	err = sd.Destroy()
	if err != nil {
		log.Fatalf("failed to destroy detector: %s", err)
	}

I'm using the v1.19.0 runtime and the v5.0 silero vad model.
Now it works for this example .wav file I found here but it isn't working for any recording I did myself which are like 14 to 30 seconds long. Although those work in python fsr.
If anyone can help me out thanks!
@streamer45 @snakers4

I even tried decreasing the threshold but that didn't work either also when I try to use another version like 4.0 it gives me this error:
Detect failed: infer failed: failed to run: Invalid input name: state

@fear-the-reaper If you could share a file sample that doesn't work, I could have a better look.

sure @streamer45  I'll send here's a test.wav drive link since I can't upload .wav here. The link. Ty!

Also another question can we run  silero-vad-go on windows? I tried but it kept giving me errors.

Also another question can we run  silero-vad-go on windows? I tried but it kept giving me errors.

Not sure as I don't have a Windows laptop myself so haven't tried. I'd suggest opening an issue at https://github.com/streamer45/silero-vad-go/issues with the failing logs.

@fear-the-reaper The file you shared has a couple of issues:

Samplerate is 48kHz. The library assumes 16kHz
It's stereo audio. The library assumes mono.

  Noob mistake on my part tysm but in python it just works so do they auto convert it?

@streamer45 I'll add logs to silero-vad-go for windows support. Will close the issue now.

in python it just works so do they auto convert it?

As for various formats, in python we use torchaudio to read files, which converts to mono, normalizes and reads the codecs under the hood with sox or ffmpeg. As for 48 kHz we just do naive under sampling - just take each third sample or just average.


Limiter acts like it's blocking the connection entirely.

Hi there,
So I tested the limiter functionality, when I try to limit the Upload/Download speed for a host on my network it acts like it's blocking it from accessing the network not just limiting the speed.
Kind regards,
Ahmed.

Hey!
You didn't - by accident - limit/block the gateway too?
Did you set the limit to a reasonable amount (e.g. 60bit/s would obviously be too low)?
Is IP routing enabled in the "Settings" tab?
Are you connected to a repeater or using DLAN?
If the error still persists try setting the value of IpEnableRouter in the registry (HKEY_LOCAL_MACHINE\SYSTEM\CurrentControlSet\Services\Tcpip\Parameters) to 1 and restart your machine.
Thank you!

Hi There,
Changing IpEnableRouter  fixed the issue.
Thank you!
Ahmed.

Update:
It seems like the IP routing service wasn't able to start properly. The issue should now be fixed.
By downloading the recent version, this error will no longer occur.
Thank you for reporting!


Why does evaluation of tox21 models use not test dataset but validation dataset?

Thanks for this cool library!
The tox21 model is trained with "train" and "val" dataset defined in the following line:
https://github.com/pfnet-research/chainer-chemistry/blob/bc90b1b0e5019e7676e4973965f87f09b4987d2c/examples/tox21/data.py#L81
And the model is evaluated with "val" dataset, not "test", as follows:
https://github.com/pfnet-research/chainer-chemistry/blob/bc90b1b0e5019e7676e4973965f87f09b4987d2c/examples/tox21/predict_tox21_with_classifier.py#L50
Why "test" dataset is not used?

Thank you for your comment.
I remember test data actually does not contain "label" information. I guess this is tox21 contest dataset and test dataset is really a "test" dataset and its label is blind.
So we used validation data for checking model's accuracy in the example.

Thanks for reply!
Test label is here:
http://bioinf.jku.at/research/DeepTox/tox21.html

Thank you for info.
You can also refer molnet dataset, which also contains tox21 data.
Sorry I did not check the test label is contained or not in molnet.


[Feature Request] GLoRA - "One-for-All: Generalized LoRA for Parameter-Efficient Fine-tuning"

I think it would be interesting and useful to implement GLoRA at least as a parameterization of the procedure, even without the evolutionary search piece. It's a generalization over LoRA and several other procedures. People might enjoy the additional flexibility of several additional new methods to play with (VPT, SSF, FacT, AdaptFormer, RepAdaptor...), which GLoRA gives you for free.

https://sites.google.com/view/generalized-lora
https://github.com/Arnav0400/ViT-Slim/tree/master/GLoRA
https://arxiv.org/abs/2306.07967

already working on actually.

the problem is inference

BUT
I think I will implement GLoKr at first, which have more fun to me :P

@dmarx

actually I already done the glora for a while I just forgot it............


Container webhook - This node is not a swarm manager.

Bug description
Container webhooks does not execute properly
{"message":"Error looking up service","details":"Error response from daemon: This node is not a swarm manager. Use "docker swarm init" or "docker swarm join" to connect this node to swarm and try again."}
Swarm is not in use or enabled on device.  Network is standalone
Expected behavior
Webhook should execute properly.
Portainer Logs
Provide the logs of your Portainer container or Service.
You can see how here
Steps to reproduce the issue:
Enable container webhooks
Post container webhook
Technical details:

Portainer version: 2.17.1
Docker version (managed by Portainer): 23.0.2
Kubernetes version (managed by Portainer):
Platform (windows/linux): linux
Command used to start Portainer (docker run -p 9443:9443 portainer/portainer): as posted
Browser: chrome or edge or api
Use Case (delete as appropriate): Using Portainer at Home
Have you reviewed our technical documentation and knowledge base? Yes

Additional context
This was working until a recent set of docker updates

Additional logs
{"_msgid":"09c545a8d5b47de3","payload":"{"message":"Error looking up service","details":"Error response from daemon: This node is not a swarm manager. Use \"docker swarm init\" or \"docker swarm join\" to connect this node to swarm and try again."}\n","topic":"","statusCode":500,"headers":{"content-type":"application/json","x-content-type-options":"nosniff","x-xss-protection":"1; mode=block","date":"Sun, 02 Apr 2023 14:12:46 GMT","content-length":"209","connection":"close","x-node-red-request-node":"b49e328e"},"responseUrl":"http://<IP_ADDRESS>:9000/api/webhooks/CONTAINERID_REDACTED","redirectList":[],"retry":0}
sudo docker swarm leave
Error response from daemon: This node is not part of a swarm

@cdn4lf
Can you provide the following:
[1] Environment:  Docker Standalone Server Local?  or Docker Standalone Agent?
[2] Can you provide reproduction steps?
[3] Can you attach screenshots of your Home page and Errors
[4] What type of repo or registry?
[5] How is the webhook being executed?
[6] Stack Code
Thanks!

@tamarahenson

Docker = Standalone Server.  I have portainer running locally on this machine
Execute webhook.  No special steps here.  Copy webhook and launch using command system of choice (node red/postman as examples)
I included the information I get from Node Red above.  There are no errors in Portainer that I can see
Repo = Docker hub
See #2.  I'm trying to use Node Red.  I'm able to toggle the webhook on / get environment data without issue, but I can't execute the final webhook command to re-create the container and pull the latest image.
This is across all my containers, not a specific container or stack.

@cdn4lf
Can you post the webhook command you are running?
Thanks!

POST http://<IP_ADDRESS>:9000/api/webhooks/CONTAINERID_REDACTED

@tamarahenson
I ran it again this morning after the latest docker updates
curl -X POST http://<IP_ADDRESS>:9000/api/webhooks/6280f452-7e30-40a8-b19b-41d40e9de268 {"message":"Error looking up service","details":"Error response from daemon: This node is not a swarm manager. Use \"docker swarm init\" or \"docker swarm join\" to connect this node to swarm and try again."}

@cdn4lf
Upon running POST, I receive the following without error on my Docker Standalone:
vagrant@docker:~$ http --verify no --form POST https://<IP_ADDRESS>:9443/api/stacks/webhooks/11795589-4d06-4811-8e80-e9bfb889d50c X-API-Key:<key>
HTTP/1.1 204 No Content
Date: Thu, 06 Apr 2023 18:08:32 GMT
X-Content-Type-Options: nosniff
X-Xss-Protection: 1; mode=block

@tamarahenson, I just tried on a different server (connected via agent), with no issues.  So it's clearly my system.
Any ideas where to start looking for the issue?

@tamarahenson I've been doing some testing and have determined that I only receive an error when the webhook is enabled via API.  When I use the GUI to enable to webhook, the curl command / http command works as expected.

@cdn4lf
Thank you for the additional information.  I am going to further investigate.  I will update you as I learn more.
Thanks!

@cdn4lf
Can you post the API commands you are using to create the Webhook?
Thanks!

@tamarahenson , I'm trying to send it via post from Node Red
msg.payload = {}; msg.payload['resourceID'] = msg.data.attributes.id; if ( msg.data.attributes.watcher =='local' || msg.data.attributes.watcher == 'debia')  msg.payload['EndpointID'] = 2  else if (msg.data.attributes.watcher == 'Pi')  msg.payload['EndpointID']=8 else msg.payload ['EndpointID']=10; //msg.payload['registryID'] = 0; msg.payload['webhooktype'] = 1; msg.headers = {}; msg.headers['X-API-Key'] = 'APIKEY' return msg
I'm sending the command via POST to http://<IP_ADDRESS>:9000/api/webhooks
When I do a GET to 'http://<IP_ADDRESS>:9000/api/webhooks' the following is returned:
statusCode: 404, responseUrl: 'http://<IP_ADDRESS>:9000/api/webhooks?EndpointID=2&ResourceID=4c09ed5a24e38b500f3eb77a5430efc9e350d09bec599e05ce5084dd32988f0f'
Payload for the GET call:
 { EndpointID: 2, ResourceID: '4c09ed5a24e38b500f3eb77a5430efc9e350d09bec599e05ce5084dd32988f0f' }, headers: { 'X-API-Key': 'ptr_Bnv7KJGzNsjdyo3GAliBOr8p7eQzUhfDeqkHdZklSL4=' }
When I try to toggle the webhook, created via API, in the GUI I see:

@cdn4lf
I wanted to follow up on this request.
When I toggle the Container webhook on, I am seeing here the webhook type is 2:

I see you are using['webhooktype'] = 1 can you try ['webhooktype'] = 2 and see if that helps move you forward?
Thanks!

@tamarahenson
That seems to have made a difference.  Are the different webhook types documented somewhere?
I'm now having an issue with http://<IP_ADDRESS>:9000/api/webhooks?EndpointID=2&registryID=0&webhooktype=2&ResourceID=904ea21f54f02f060c4af4fc48072e5aa4d7bc4b94d2073d67d4b6ef65ffa9cd returning a 404 error when trying to query an already existing webhook.

@cdn4lf
Thank you for the update.  I am glad to hear you got one step ahead.
I am logging an internal request to document the 3 types.  I am seeing we have a 0, 1 and 2.  I am still working on this.
Interim, here is the documentation for the GET:  https://app.swaggerhub.com/apis/portainer/portainer-ee/2.18.1#/webhooks/get_webhooks
Thanks!

@tamarahenson
RE: Get webhooks.  I can't get in.  I've gone so far as to import the json for the API into Postman and still am unable to.  Where does the resourceID come from?  I haven't found any docs to support where this comes from.

@cdn4lf
The ResourceID is the Container ID as seen here:

Thanks!

@tamarahenson , That's what I thought and what I've been using.
I'm still getting a 404 error whenever I execute the command.
Latest get sent "http://<IP_ADDRESS>:9000/api/webhooks?EndpointID=2&ResourceID=904ea21f54f02f060c4af4fc48072e5aa4d7bc4b94d2073d67d4b6ef65ffa9cd"

@cdn4lf
Can you try this command here?
http://<IP_ADDRESS>:9000/api/webhooks?filters={"ResourceID":"904ea21f54f02f060c4af4fc48072e5aa4d7bc4b94d2073d67d4b6ef65ffa9cd","EndpointID":2}

Thanks!

@tamarahenson you're a miracle worker.  We have success.
Any idea why this isn't documented as such?

@cdn4lf
I am glad to hear you are able to move forward.  I have logged an internal request to have our Webhook API documentation improved.
Thanks!


Some cleanups to global search

Merge UpdateSearch and UpdateSearchWithCaseSensitivity
Fix GlobalSearch request_handler not responding to an expired search job
Check if workspace is Some(_) before even creating the thread. Responds immediately otherwise.

Signed-off-by: Hanif Ariffin<EMAIL_ADDRESS>
[ ] Added an entry to CHANGELOG.md if this change could be valuable to users

Codecov Report

Merging #2048 (0c45352) into master (5b2d412) will increase coverage by 0.05%.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##           master   #2048      +/-   ##
=========================================
+ Coverage    9.00%   9.05%   +0.05%
=========================================
  Files         135     135
  Lines       58831   58501     -330
=========================================
  Hits         5295    5295
+ Misses      53536   53206     -330

Impacted Files
Coverage Δ

lapce-data/src/command.rs
0.00% <ø> (ø)

lapce-data/src/document.rs
0.00% <0.00%> (ø)

lapce-data/src/editor.rs
0.00% <0.00%> (ø)

lapce-data/src/find.rs
0.00% <0.00%> (ø)

lapce-proxy/src/dispatch.rs
0.00% <0.00%> (ø)

lapce-ui/src/find.rs
0.00% <ø> (ø)

lapce-ui/src/search.rs
0.00% <ø> (ø)

lapce-ui/src/tab.rs
0.00% <0.00%> (ø)

lapce-ui/src/app.rs
0.00% <0.00%> (ø)

lapce-ui/src/title.rs
0.00% <0.00%> (ø)

... and 8 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


At the speed of light

Song Name
(New version) (remix) Dimrain47-at the speed of light
Artist Name
DatRealWrona
Source
Direct file download
Youtube Link
No response
Direct File Link
https://www.newgrounds.com/audio/listen/847621
Song ID
467339
Start Offset [ms]
0
Extra
Cataclysm-3979721, Bloodbath-10565740,Bloodlust-42584142,Aftermath-25610878

replacing with songs on newgrounds is not supported yet


Error while excute python setup.py develop

I was running this codepython setup.py develop
while give me error
`During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "setup.py", line 203, in
zip_safe=False)
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/setuptools/init.py", line 165, in setup
return distutils.core.setup(**attrs)
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/distutils/core.py", line 148, in setup
dist.run_commands()
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/distutils/dist.py", line 966, in run_commands
self.run_command(cmd)
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/distutils/dist.py", line 985, in run_command
cmd_obj.run()
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/setuptools/command/develop.py", line 38, in run
self.install_for_development()
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/setuptools/command/develop.py", line 155, in install_for_development
self.process_distribution(None, self.dist, not self.no_deps)
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/setuptools/command/easy_install.py", line 759, in process_distribution
[requirement], self.local_index, self.easy_install
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/pkg_resources/init.py", line 781, in resolve
replace_conflicting=replace_conflicting
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/pkg_resources/init.py", line 1064, in best_match
return self.obtain(req, installer)
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/pkg_resources/init.py", line 1076, in obtain
return installer(requirement)
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/setuptools/command/easy_install.py", line 686, in easy_install
return self.install_item(spec, dist.location, tmpdir, deps)
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/setuptools/command/easy_install.py", line 712, in install_item
dists = self.install_eggs(spec, download, tmpdir)
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/setuptools/command/easy_install.py", line 897, in install_eggs
return self.build_and_install(setup_script, setup_base)
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/setuptools/command/easy_install.py", line 1167, in build_and_install
self.run_setup(setup_script, setup_base, args)
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/setuptools/command/easy_install.py", line 1151, in run_setup
run_setup(setup_script, args)
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/setuptools/sandbox.py", line 253, in run_setup
raise
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/contextlib.py", line 130, in exit
self.gen.throw(type, value, traceback)
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/setuptools/sandbox.py", line 195, in setup_context
yield
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/contextlib.py", line 130, in exit
self.gen.throw(type, value, traceback)
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/setuptools/sandbox.py", line 166, in save_modules
saved_exc.resume()
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/setuptools/sandbox.py", line 141, in resume
six.reraise(type, exc, self._tb)
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/setuptools/_vendor/six.py", line 685, in reraise
raise value.with_traceback(tb)
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/setuptools/sandbox.py", line 154, in save_modules
yield saved
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/setuptools/sandbox.py", line 195, in setup_context
yield
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/setuptools/sandbox.py", line 250, in run_setup
_execfile(setup_script, ns)
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/setuptools/sandbox.py", line 45, in _execfile
exec(code, globals, locals)
File "/tmp/easy_install-_zh_hd6k/mmcv-1.0.2/setup.py", line 219, in
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/setuptools/init.py", line 165, in setup
return distutils.core.setup(**attrs)
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/distutils/core.py", line 148, in setup
dist.run_commands()
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/distutils/dist.py", line 966, in run_commands
self.run_command(cmd)
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/distutils/dist.py", line 985, in run_command
cmd_obj.run()
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/setuptools/command/bdist_egg.py", line 165, in run
self.run_command("egg_info")
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/distutils/cmd.py", line 313, in run_command
self.distribution.run_command(command)
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/distutils/dist.py", line 985, in run_command
cmd_obj.run()
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/setuptools/command/egg_info.py", line 297, in run
self.find_sources()
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/setuptools/command/egg_info.py", line 304, in find_sources
mm.run()
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/setuptools/command/egg_info.py", line 535, in run
self.add_defaults()
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/setuptools/command/egg_info.py", line 571, in add_defaults
sdist.add_defaults(self)
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/distutils/command/sdist.py", line 228, in add_defaults
self._add_defaults_ext()
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/distutils/command/sdist.py", line 311, in _add_defaults_ext
build_ext = self.get_finalized_command('build_ext')
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/distutils/cmd.py", line 299, in get_finalized_command
cmd_obj.ensure_finalized()
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/distutils/cmd.py", line 107, in ensure_finalized
self.finalize_options()
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/setuptools/command/build_ext.py", line 144, in finalize_options
self.check_extensions_list(self.extensions)
File "/home/user/anaconda3/envs/open-mmlab/lib/python3.7/distutils/command/build_ext.py", line 363, in check_extensions_list
"each element of 'ext_modules' option must be an "
setuptools.sandbox.UnpickleableException: DistutilsSetupError("each element of 'ext_modules' option must be an Extension instance or 2-tuple")`
what should i do to fix it?
i am using cuda10.0, pytorch and torchvision as demanded.

Same issue

I am facing the same issue

same issue here

same issue

I solved this by pip install mmcv previously

I solved this by pip install mmcv previously

Thanks, it worked, and for avoiding further errors i run
pip install mmcv==0.4.0


Creeper Healing mod crash on start

Mod Name
creeper healing
Mod Homepage
https://modrinth.com/mod/creeper-healing
Minecraft version
1.20.1
Describe the bug
Crash on start
Steps to reproduce

...
Logs
https://gist.github.com/cococops/16aa82d93d2f620ecb99743830b87dcf
Additional context
Tried it without any other mod, it just crashes on start with code exit: 1

It still does not work on 1.20.1


connectedCallback will have no children when DOM hasn't parsed yet

  get headings() {
    return this.querySelectorAll(this.selector);
  }

Will return an empty NodeList because of:
https://dev.to/dannyengelman/web-component-developers-do-not-connect-with-the-connectedcallback-yet-4jo7

This component requires the DOM to be parsed in order to know what to render, it relies on existing HTML on the page. Please stop spamming my components.

Then make you WC wait for DOM.
You now present us with a car which you first have to push to the fuel station

Are you using type="module" for the script tag? Using that means you can put the script anywhere on the page


Add many torrents?

Can i add many torrents? and mount it in one/different folders ?

@mafintosh I was wondering how this would behave for 400+ torrents. Does it fetch blocks dynamically? e.g. only the ones needed

From the behavior I've observed it fetches everything, but once you start accessing files it will reprioritize the order to fetch the most relevant blocks for your file read first

--lazy will only begin to download files on first access.  I'm not sure if there is logic to handle automatically stopping retrieving blocks of a file that is no longer open.
I'm wondering whether you could tie this in with Linux autofs to invoke torrent-mount dynamically on on chdir, and then find a way to present a parent dir of magnet links.  Hmm

I'm wondering whether you could tie this in with Linux autofs to invoke torrent-mount dynamically on on chdir, and then find a way to present a parent dir of magnet links. Hmm

Exactly!


fix: bug 1661 - more concise filtering dropdown labels & "any"

Added clarity to the drop down labels, and cleaned up the "any" option

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Problem with Autoupdate

It seems like the autoupdate function is not working as aspected.

Seems like the timestamp within the public branch of steam is not changing. Gonna investigate this


ORM: Trysil

Trysil [Delphi] ORM (Object-Relational Mapping) for Delphi. Supports for SQLServer, FirebirdSQL and SQLite.

Added, thanks!


Merged shapes not showing up on iOS using React Native

We are using React Native for iOS and the animated icon only shows as a square and is missing the three triangles. Is this the Merged Path issue on iOS? If so, when do you expect it to be supported and is there a work around? Thanks!

Fill types are not yet supported in iOS. You can track the progress here: https://github.com/airbnb/lottie-ios/issues/43

Sorry to comment on an old issue, think we're running into this with some animations received from a client. They've said that there are "merged shapes with subtracts", which are basically the holes in some map pins. However we're just getting a flat pin in the animation (with no cutout).
Does that seem to fit the bill for this problem? I notice in the issue mentioned by @gpeal it's been resolved as of 2.0 🤔. Cheers


How to filter events by property

I'm sorry, what is the recommended way to filter/search a calendar by a property? For example, "fetch me all the events with 'HOLIDAY' as summary"

With the current version, the simplest way is probably to do a date_search and then filter on the client side.
I should consider to fix better functionality for date_search and todo-search with filtering,

I'm considering to add more parameters to date_search and include a fix in 0.7.  At the other hand ...

The caldav query language is quite arcane
... so arcane that quite some caldav server programmers fail to get it right
the caldav query langauge is also somewhat restricted
I did some research into server-side search when implementing task lists in calendar-cli - but ended up with fetching all tasks and doing sorting and selecting on the client side.

A proper fix will be postponed until 0.8 or 1.0.  However, I have moved logic from the date_search-method into a build_date_search_query method and a search-method, so that it should be theoretically possible to either modify the query or hand-feed the search-method with some query XML.
Remaining work on this issue:

Play with it, see it it's possible to make some queries in XML (string) and get out search results.  Fix some test code for this.
Play with it, see if it's possible to build same queries using the classes under caldav.elements.cdav.  Fix test code.
See if it's practically possible to create queries by modifying the return results from the build_date_search_query-method, and create test code for this
Think out if there are more parameters that are suitable to pass, and other simplified search methods that ought to be added, and ... of course, add test code for it.

I'm working no a new method calendar.search() in the master branch

The list of properties that it should be possible to filter is not complete, and I'm also intending to do quite some work with client side filtering to iron over various incompatibilities with the different calendar servers.  Anyway, I close this issue and will eventually open (an) other issue(s) for that later on.


Use NodaTime for date/time handling

.NET's DateTime type is not ideal for dealing with time zones, parsing, etc. Use it for all data-related date storage; we may still need to convert to DateTime for display (as DotLiquid knows that type), but the storage and calculation will be more explicit.
This will require migration for SQLite and RethinkDB data stores.

This has been completed, and will be part of v2-rc2


Compiler crash due to unsigned enum

While converting an existing .NET library to Bridge, I hit a compiler error caused by the following code:
    public enum TestEnum : uint
    {
        All = 0xFFFFFFFF,
    }

This results in the following compiler crash:
Severity	Code	Description	Project	File	Line	Suppression State
Error		Error: Specified cast is not valid.    at Bridge.Translator.Inspector.VisitEnumMemberDeclaration(EnumMemberDeclaration enumMemberDeclaration)
   at ICSharpCode.NRefactory.CSharp.EnumMemberDeclaration.AcceptVisitor(IAstVisitor visitor)
   at Bridge.Contract.Helpers.AcceptChildren(AstNode node, IAstVisitor visitor)
   at Bridge.Translator.Inspector.VisitTypeDeclaration(TypeDeclaration typeDeclaration)
   at ICSharpCode.NRefactory.CSharp.TypeDeclaration.AcceptVisitor(IAstVisitor visitor)
   at Bridge.Contract.Helpers.AcceptChildren(AstNode node, IAstVisitor visitor)
   at Bridge.Translator.Inspector.VisitNamespaceDeclaration(NamespaceDeclaration namespaceDeclaration)
   at ICSharpCode.NRefactory.CSharp.NamespaceDeclaration.AcceptVisitor(IAstVisitor visitor)
   at Bridge.Contract.Helpers.AcceptChildren(AstNode node, IAstVisitor visitor)
   at Bridge.Translator.Inspector.VisitSyntaxTree(SyntaxTree node)
   at Bridge.Translator.Translator.InspectTypes(MemberResolver resolver, IAssemblyInfo config)
   at Bridge.Translator.Translator.Translate()
   at Bridge.Build.GenerateScript.Execute()	ClassLibrary2

@strygo,
Thank you for reporting this issue.

No problem. Thanks for the quick fix!


Add MarkdownPP artifacts to Pypi

Would be nice to be able to pip install MarkdownPP.

Took some time today to clean up much of the codebase for Python 3 support, so I wouldn't feel bad about putting it on PyPI, and ended up doing a PEP8 cleanup and adding very basic linting/testing.
Cut a new release, 1.1, and uploaded a package to PyPI.  Cheers!


Update DOCS_BASE to v2.9

Current DOCS_BASE points to https://ranchermanager.docs.rancher.com/v2.8
When 2.9 becomes available value needs updating
Need to sync with docs team on when this will be

@btat are there any existing requests to do this bump already? I couldn't find any.

I'll be generating the 2.9 content next week.

https://ranchermanager.docs.rancher.com/v2.9 is live now.

@izaac should this be a manual test candidate?

@momesgin I see this DOCS_BASE variable in the dashboard repository, do we external link them or use them for translations in the UI?
If that's the case perhaps a simple validation to check the UI links to the proper Rancher Docs DOCS_BASE?

@momesgin I see this DOCS_BASE variable in the dashboard repository, do we external link them or use them for translations in the UI?
If that's the case perhaps a simple validation to check the UI links to the proper Rancher Docs DOCS_BASE?

@izaac I believe we're already doing that indirectly by checking the support links on the home page, for example the Docs item on that list gets its value from DOCS_BASE.

We are good if that health check exist. 👍🏽

Tests pass on v2.9-16190ecee0aa3fb4aa570aa0fc932a36d9dcd082-head


Command to jump to the next/previous difference in the diff editor

I don't use the diff editor much, one of the first things I tried to do when I was learning it better now was to find the command to jump to the next change. I found go to next difference (F7) which is almost what I want:

F7 goes to the diff chunk:

I was expecting it to move the cursor/viewport to center around the change in the right editor.

F7 and Shift+F7 navigate through the diff hunks, but they do so in an accessible way. i.e. they are the mechanism through which the diff editor is accessible.

Thanks, this is it:
workbench.action.compareEditor.nextChange
workbench.action.compareEditor.previousChange

They don't have default keybindings.

These keybindings seem to work well for me:
{ "key": "alt+f3",           "command": "workbench.action.editor.nextChange", "when": "editorTextFocus" },
{ "key": "shift+alt+f3",     "command": "workbench.action.editor.previousChange", "when": "editorTextFocus" },
{ "key": "alt+f3",           "command": "workbench.action.compareEditor.nextChange", "when": "isInDiffEditor" },
{ "key": "shift+alt+f3",     "command": "workbench.action.compareEditor.previousChange", "when": "isInDiffEditor" }

Maybe they should default to alt+f5/shift+alt+f5, so it's the same default as workbench.action.editor.*Change when in the diff editor?

^ I would support that. I made a separate issue here, don't know if that was overkill #55481

Btw I'd think that it would be more logical to use modifiers+F7 (next diff change), instead of modifiers+F3 (next search match) 🙂

Or.. it's actually more similar to
{ "key": "alt+f5",                "command": "workbench.action.editor.nextChange",
There are so many commands related to the concept of "next change",  I'm actually a little confused myself 😅

Alright I have it figured out!
These two now have the default shortcuts:
  {
    "key": "alt+f5",
    "command": "workbench.action.editor.nextChange",
    "when": "editorTextFocus"
  },
  {
    "key": "shift+alt+f5",
    "command": "workbench.action.editor.previousChange",
    "when": "editorTextFocus"
  }

The compareEditor.nextChange does the exact same thing as editor.nextChange, just for diff editor instead of normal editor. So there's absolutely no reason not to overload the alt+f3 and shift+alt+f3 with the same functionality. So all we have to do is add:
  {
    "key": "alt+f5",
    "command": "workbench.action.compareEditor.nextChange",
    "when": "isInDiffEditor"
  },
  {
    "key": "shift+alt+f5",
    "command": "workbench.action.compareEditor.previousChange",
    "when": "isInDiffEditor"
  }

Actually the "when": "isInDiffEditor" might strictly be unnecessary, but it's good to have there for clarity.

I'd contribute the above myself, but I have no idea where the default keybindings are in the source 😅


MapPanel.js example with ScrollView (similar to Material Bottom sheets)

First, sorry if this is very related to #35, just wanted to make sure.
Basically, I was trying to mimic Bottom sheets from Material spec using this example MapPanel.js.
But, with one addition, changing content view from the panel here to ScrollView. Then, you cannot interact with it anymore (only internal scrolling).
You can see this is an issue in the real example, on Apple Maps-Style Panel, if you change your device orientation (I try with Android) to landscape, because there is no internal scroll, you cannot see the whole view.
Desired result would be something like:

Hey any success on implementing this? I'm also trying to do it.

I'm also trying to implement this

Hey @abeddow91 I did a bit of a workaround. I've put a bar that's at the top of interactable view that is not scrollable. Below it have a scrollview which normally handles it's events.
This is the code that does it:

And this is the render of ShoppingListView

This <HeaderDisplay /> is the bar at the top of interactable view and behaves like a normal interactable view.
This <List /> inside of <Interactable.InterceptionBlocker /> is a wrapped FlatList (so ScrollView).
Important part is that you add canCancelContentTouches={true} as a prop to your scrollview component. If you don't it won't receive touch events. You can find an example by @talkol in one the issues that's similar to this one.
Also important: <Interactable.InterceptionBlocker /> is needed only for android and you can find it in a separate branch in this repo. There's a really long thread in one of the issues and maybe you can find some ideas there.
Right now I can snap the interactable view only by interacting with a top bar and that's not bad.
I had an idea however that I could remove the top bar and put it inside the scrollview. Then I would listen for onScroll events and when the would be negative enough. (let's say -30-40px) I would manually call .snapTo on the interactable view and close it down. It wouldn't be as smooth as on the video in this issue but it would behave similarly to "now playing" view in Apple music.
Hope I helped.

@urbancvek that is good workaround but still it is not as good as interact only with one view as you have to release your finger to do either scroll or closing the view.

Yup this approach I'm using is far from perfect but it's what this lib can currently do and yeah since it can do almost everything else I had to use it 😄. This is the thread I mentioned where I got of understanding of how to work around it. It will be awesome once it's implemented but as far as I understand it's a hard problem to solve.

Any updates on this?

@urbancvek @abeddow91 @ferrannp any update?

I ended up with just a scrollView above all the content, I wanted to make the top of the scrollview not touchable, I could not get there with just the normal react-native features. I looked and saw a nice library which does just that. https://github.com/rome2rio/react-native-touch-through-view.
import React, { PureComponent } from 'react'
import PT from 'prop-types'

import {
  TouchThroughView,
  TouchThroughWrapper,
} from 'react-native-touch-through-view'
import {
  View,
  Dimensions,
  ScrollView,
  TouchableWithoutFeedback,
} from 'react-native'

import keyboardHOC from '../logic/hocs/keyboardHOC'
import styles from './styles/BottomSheet.style'
// import log from '../logic/shared/helpers/log'
import binder from '../logic/shared/helpers/binder'
import c from '../logic/shared/constants'
// import n from '../constants'

const getSize = () => ({
  width: Dimensions.get('window').width,
  height: Dimensions.get('window').height,
})

class BottomSheet extends PureComponent {
  constructor(props) {
    super(props)
    binder(this, ['_handleScroll', '_close'])
    this.state = {
      opened: false,
    }
    this.y = 0
  }

  _close() {
    this.setState(
      {
        opened: false,
      },
      () => this._scrollView.scrollTo({ y: 0, animated: true })
    )
  }
  _handleScroll(event) {
    if (event.nativeEvent.contentOffset.y === 0 && this.y !== 0) {
      if (this.state.opened) {
        this.setState({
          opened: false,
        })
      }
      this.props.collapsed()
    }

    if (this.y > 0) {
      if (!this.state.opened) {
        this.setState({
          opened: true,
        })
      }
      this.props.opened()
    }
  }
  //TODO remove touchThroughview when scrolled higher
  render() {
    const { height } = getSize()

    const offsetY =
      this.y > this.props.keyboardHeight ? this.y : this.props.keyboardHeight

    const BackDrop = this.state.opened ? View : TouchThroughView

    return (
      <TouchThroughWrapper style={styles.wrapper}>
        <ScrollView
          onScroll={this._handleScroll}
          ref={component => {
            this._scrollView = component
            this.props.scrollViewRef(component)
          }}
          scrollEventThrottle={16}
          alwaysBounceVertical={false}
          bounces={false}
          keyboardDismissMode={'on-drag'}
          contentOffset={{
            y: offsetY,
          }}
        >
          <TouchableWithoutFeedback onPress={this._close}>
            <BackDrop
              style={{
                height: height - c.panelHeight,
              }}
            />
          </TouchableWithoutFeedback>
          {this.props.panelContent &&
            <this.props.panelContent
              direction={this.state.direction}
              {...this.props.panelProps}
            />}
          <View
            style={{
              minHeight: height,
              backgroundColor: '#fff',
            }}
          >
            {this.props.children}
          </View>
        </ScrollView>
      </TouchThroughWrapper>
    )
  }
}

BottomSheet.propTypes = {
  children: PT.oneOfType([PT.arrayOf(PT.node), PT.node]).isRequired,
  keyboardHeight: PT.number,
  collapsed: PT.func,
  opened: PT.func,
  panelProps: PT.obj,
  panelContent: PT.func,
  scrollViewRef: PT.func,
}
BottomSheet.defaultProps = {
  keyboardHeight: 0,
  count: 0,
  collapsed: () => {},
  opened: () => {},
  scrollViewRef: () => {},
}

export default keyboardHOC(BottomSheet)

keyboardHOC
import React, { Component } from 'react'
import { Keyboard, Platform, LayoutAnimation } from 'react-native'

import binder from '../shared/helpers/binder'

export default WrappedComponent => {
  class HOC extends Component {
    constructor(props) {
      super(props)

      this.state = {
        keyboardHeight: 0,
      }

      binder(this, [
        '_keyboardDidShow',
        '_keyboardDidHide',
        '_onKeyboardChange',
      ])
    }

    componentWillMount() {
      if (WrappedComponent.componentDidMount)
        WrappedComponent.componentDidMount()

      if (Platform.OS === 'ios') {
        this.keyboardDidChangeListener = Keyboard.addListener(
          'keyboardWillChangeFrame',
          this._onKeyboardChange
        )
      } else {
        this.keyboardDidShowListener = Keyboard.addListener(
          'keyboardDidShow',
          this._keyboardDidShow
        )
      }

      this.keyboardDidHideListener = Keyboard.addListener(
        'keyboardWillHide',
        this._keyboardDidHide
      )
    }
    componentWillUnmount() {
      if (WrappedComponent.componentWillUnmount)
        WrappedComponent.componentWillUnmount()

      if (Platform.OS === 'ios') {
        this.keyboardDidChangeListener.remove()
      } else {
        this.keyboardDidShowListener.remove()
      }
      this.keyboardDidHideListener.remove()
    }

    _onKeyboardChange(event) {
      if (!event) {
        this.setState({ keyboardHeight: 0 })
        return
      }

      const { duration, easing, endCoordinates } = event
      const keyboardHeight = endCoordinates.height

      if (duration && easing) {
        LayoutAnimation.configureNext({
          duration,
          update: {
            duration,
            type: LayoutAnimation.Types[easing] || 'keyboard',
          },
        })
      }
      this.setState({ keyboardHeight })
    }
    _keyboardDidShow(e) {
      const keyboardHeight = e.endCoordinates.height
      if (e && keyboardHeight) {
        this.setState({
          keyboardHeight,
        })
      }
    }
    _keyboardDidHide() {
      this.setState({
        keyboardHeight: 0,
      })
    }

    render() {
      return <WrappedComponent {...this.state} {...this.props} />
    }
  }

  return HOC
}

@RichardLindhout Sweet implementation. I gave it a go, but I'm unable to make the  Touch through view work on Android in RN 0.50.1. You able to get it running on the latest version?

@kristfal  Thanks! I've only tested it on an iPhone at the moment.

I am closing the issue as a part of an effort to cleanup and revive the project. If this issue persists after v1.0.0, you are more than welcome to reintroduce it.
Thanks for your contribution.

Any updates on this?

@urbancvek Are you able to get the same behavior like Android bottom-sheet? If yes, can you please share your code?
FYI, I want to achieve UI like Uber Lite android App to select Pickup address - https://play.google.com/store/apps/details?id=com.ubercab.uberlite&hl

@neerajemorphis I did use https://github.com/cesardeazevedo/react-native-bottom-sheet-behavior for Android

@neerajemorphis I did use https://github.com/cesardeazevedo/react-native-bottom-sheet-behavior for Android

Even I used the same for Android, but I am stuck for iOS. I need the same bottomsheet behavior for iOS as well.

For iOS you could go with the solution I posted above with the touch trough scrollview.

@RichardLindhout thats not working as expected, no methods to expand / collapse the bottomSheet, in touch through view, as well as collapsed, opened props doesn't seem to be working. Apart from that, need to restrict the height, of the bottomsheet to some specific height, which should not be crossed, when expanded.
I am looking for the same behavior like Android bottom-sheet - https://github.com/cesardeazevedo/react-native-bottom-sheet-behavior.


Feature/opening date event

Hi @dneed-nimble please mark this PR as ready to review once you have fixed the issues preventing the .NET build task from passing.

@dneed-nimble are you happy to merge with the sonarcloud coverage issues?


fix: update_vulnerability in dbStatements_oracle.properties

Added missing out parameter to update_vulnerability in order to avoid java.sql.SQLException: Invalid column index
Fixes Issue
Description of Change
Added missing out parameter for oracle.
Have test cases been added to cover the new functionality?
no

Sorry about this - I'll get an updated version out by Saturday morning (us-eastern) if not sooner.

Thank You!


fix: Constitutive law fixes and refactoring

This PR covers two aspects:

Constitutive laws have been refactored as a sub-module permitting to have separate files for constitutive laws. @mortenengen maybe the file is already big enough to suggest a splitting into different files already?
when the user inputs a float in get_stress and get_tangent methods, a float will be returned. Otherwise a ArrayLike is returned.

The public API is not changed with this PR.

Thanks for the review.
Tried to fix in e6ff596bfb1ea350d98de1c084cda616cfc9a87f

Nice job on this 😃


NodeList does not implement Iterable

In pkg/compiler/lib/src/tree/nodes.dart, dartsj2 defines a NodeList class with an iterator getter that returns an Iterator<Node>. However, it does not declare that it implements Iterable<Node>, nor does it implement (or inherit or mixin) all of the fun methods we know and love from Iterable.
I'm working on adding better type checking around for-in statements. Analyzer right now does absolutely no analysis (!). The spec defines some funny structural typing, which would make the dart2js code in question pass correctly, but is pretty complex to analyze and doesn't seem to fit with the rest of Dart or add a lot of value to users.
My goal is to implement nominal checking so that the iterator in a for-in loop is expected to implement Iterable<T>. As far as I can tell, this is the only real example of code in the entire repo that doesn't already pass that tighter checking.

This is deliberate since most Iterable methods would be slow on NodeList as long as it is based on Link. The real solution here is to change NodeList to use List and then maybe make it implement Iterable by forwarding to the List.

Why would they be slow - or slower than other literables (that aren't also List or Set)?
A linked list is an excellent example of something to iterate through, and the NodeList iterator is efficient, so the Iterable methods are as efficient as can be expected of an Iterable.
Or is the problem that the name suggests that it's a List and therefore someone could mistakenly assume that length is fast?

I've frequently found it annoying that NodeList did not implement the Iterable interface.  We always end up writing a for in loop using the iterator which isn't really any faster than the missing method implementation would be and requires a reader to reason about it to figure out which pattern its implementing.
So I vote yes!

The slow length is the main culprit.

Iterables do not promise to have a non-slow length. Rather the opposite - if nothing else is stated, you should assume it to be linear.
That's why we check if an Iterable is a List, Set or Queue in some places, before assuming an efficient length.
NodeList may be slightly more mislading than average because it's named List, and List has efficient length.

Fixed by: https://chromiumcodereview.appspot.com/1783053005/
Eventually I would like to revert the NodeList part of this CL and change all the for-in sites to use .nodes.
I also want to get rid of Link. There is no size where this is more space-efficient that a fixed-length or unmodifiable list provided you use the same empty list everywhere.

Thanks for fixing this!


[UI] Swap and Pool UI errors

Describe the bug
A user tried to swap USDT to nextUSDT on Polygon and an "index is not defined" error appeared.

Then when adding liquidity to the pool, they got a call revert error similar to this one https://github.com/connext/monorepo/issues/4231

Additional context
User is running metamask on chrome. https://discord.com/channels/454734546869551114/816982956684869654/1110622685635018792

Fixed the "index not defined" error, but it still reverts.  @sanchaymittal do you have some insight on this issue?

A user is getting missing revert data in call exception error when trying to deposit in a pool.

Context:
https://discord.com/channels/454734546869551114/1120292771585065051/1120292911867764928

Could not repro with USDC on any chain - will get more details from the user in discord

closing, stale.


Renamed the producer name from Zookeeper to ZooKeeper

Hi!
This PR just renames the ZooKeeper's producer name from ZookeeperProducer to ZooKeeperProducer because other classes use ZooKeeper.

Thanks for the PR it has been merged. Do you mind closing this?

@davgordo
Thanks!


[Feat]: Adding Best seller button

Description
By enabling best seller button user can quickly access the best seller products and there is a high scop  of buying a product of
best seller .

Please assign me the issue .
i'm a GSsoc-Extended '24 conrtributor.
Screenshots
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I'm a Hacktoberfest'24 contributor
[ ] I am willing to work on this issue (optional)

@Stpchethan As of now, we do not need it. We will let you know in the future if there is a need. Please search for another issue and raise it, which I'm happy to assign to you.