get_gravity() doesn't update after move_and_collide()

Tested versions
v4.3.stable.mono.official [77dcf97d8]
System information
Godot v4.3.stable.mono - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce GTX 1060 6GB (NVIDIA; <IP_ADDRESS>44) - AMD Ryzen 5 2600X Six-Core Processor (12 Threads)
Issue description
PhysicsBody2D.get_gravity() is not updating its value after I call move_and_collide()
I'm doing multiple iterations of move_and_collide() in one frame to draw a Line2D path in the direction the object will be going and stop at any collision point. The problem is that get_gravity()'s value doesn't change at all over the course of the iterations, it stays constant the entire time. I'm using Area2D nodes as gravity points, so the gravity value should never be constant for a moving object, and without get_gravity() updating I can never actually predict the path my object will take in the future.
If this is intended behavior, there's no indication in the documentation. It just says Returns the gravity vector computed from all sources that can affect the body which is not the behavior I'm observing.
Steps to reproduce
Use multiple move_and_collide() iterations in one frame whilst the object is inside an Area2D with gravity_point enabled and observe that the value returned from get_gravity() is the same for every iteration.
Minimal reproduction project (MRP)
get_gravity_bug_godot_4.3.zip

I suspect this is due to delays in handling some updates, will test the MRP and confirm but this might be something to document as a limitation

See also:

https://github.com/godotengine/godot/issues/87996

I'm wondering if the implementation in https://github.com/godotengine/godot/pull/83087 would be better, given the issues with get_gravity. @mihe thoughts?

This specific problem wouldn't be fixed by something like #83087. As hinted at by @AThousandShips already, the area overlaps are evaluated as part of stepping the physics simulation, which you'd need to have up-to-date no matter which way you flip this.
I'm not sure if this is really documented anywhere, but this limitation applies to a number of physics-related things in Godot, like the contact reporting and the various signals emitted (e.g. body_entered). I wouldn't really consider this a bug.
The only way I can see this being done (short of custom stepping + rollback, which isn't a thing in Godot, and seems overkill here) would be to use intersect_shape with collide_with_areas set to true, to figure out what areas you'd be overlapping with at that point and then calculate the total gravity yourself based on their priority and gravity_space_override, but that is not trivial.
This is borderline off-topic, but there is another bug lurking in the current implementation of get_gravity, which would be fixed by something like #83087, where if your CharacterBody3D (or some other kinematic body) is just standing perfectly still, and you have some Area3D with gravity_space_override moving around and entering/exiting the body, then the gravity reported by get_gravity will become stale, as gravity is only computed for active/awake bodies, and kinematic bodies are only considered active/awake during the single tick after which they moved.
I need to make a separate issue for that.


Propagate vertical alignment style.

Useful to align markers vertically.
Signed-off-by: Matthew Khouzam<EMAIL_ADDRESS>

Thanks!


[Feature Request]: add password eye button

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
Add password eye button in login and signup page for checking password
Use Case
user can see their password if they have filled it right or not.
Benefits
No response
Priority
Medium
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[ ] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

Please assign this to me with gssoc-extd label. i can do this for you

please assign me this issue under gssoc-ext label

please assign me this issue

@mansiruhil13 I have fixed this issue and have raised the PR. plz check and merge accordingly.

@mansiruhil13 i have fixed the issue and have raised a PR. check and merge accordingly.

can i generate a new pull request

can i generate a new pull request

yes

please assign me this task


Update ciceksepeti.yml

Update company name
Adds or updates the following companies, events, or universities:

Checklist
Company updates

[ ] I have put the most recent relevant date in the "Last Update" column
[ ] I have linked to the article about the change, not the company's homepage (Please put N/A if there is no public post)

Event updates

[ ] I have linked to the article about the change, not the event's homepage

University updates

[ ] I have linked to the article about the change, not the university's homepage

Update Company Name

@krishnatre-siddhartha there's no need for that as the last updated date is already 2019-03-19


CSS  - particularly the "turn" unit

See MDN article here. It looks like currently all browsers support deg, rad, grad and some support turn.

+1

+1

+1

+1
https://css-tricks.com/lets-look-50-interesting-css-properties-values/#angle

+1

+1

Available at https://caniuse.com/#feat=mdn-css_types_angle & https://caniuse.com/#feat=mdn-css_types_angle_turn


Unable to publish new version due to hbmqtt not being installable by Poetry

Wait for https://github.com/python-poetry/poetry/issues/3649 to be fixed by https://github.com/python-poetry/poetry/pull/3656

Fixed by switching to asyncio-mqtt instead.


new optimizer rules

This PR does 3 things:

Fixes a bug in the "constant lookup extractor"  where a lookup of form 0 $ [A, B] in [zero, one] generates a new constraint -1 = 0.
Adds a new rule that removes lookups where the left selector = 0. This is sound because it's a tautology.
Adds a new rule that transforms a permutation with left selector = 0 into a constraint R = 0, where R is the right selector. This is sound because the LHS of the permutation being 0 means both sides should be the empty set, which is guaranteed by the constraint for the RHS.

The new optimization rule causes witgen issues. I'll push the bugfix in a different PR.

Should we merge some of this, or was it all fixed in #1889 ?

Yea theres one bugfix that needs to go in

Closing in favor of https://github.com/powdr-labs/powdr/pull/2004

#2004 only addressed one of the items.

#2004 only addressed one of the items.

The other one causes witgen issues, not sure it's worth pursuing it.


[Bug] Token showing in merge commit message

If you have a remote including username and github token, gittyup will incude it in the commit message.
It must be hidden

I can't reproduce this behavior with my version, which version are you using?

According to a comment in libgit2, this shouldn't happen:
/* We should never be called with multiple remote branches, but handle
 * it in case we are... */

Can you provide a test repository and which branch to merge to reproduce this?


style(challenges): Format Descriptions for Project Euler 1-16

Pre-Submission Checklist

[x] Your pull request targets the staging branch of freeCodeCamp.
[ ] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[ ] Addressed currently open issue (replace XXXXX with an issue no in next line)

Closes #XXXXX
Description

Here are a list of style changes made for problems 1-16:

2: Centered text
6: Centered text and <sup> tags
8: Centered text
9: Centered text, <var>, and <sup> tags
11: Centered text and spacing between number grid and next paragraph
12: Tab spacing (padding-left: 4em)
14: Centered text, <var> tags, and tab spacing (padding-left: 4em)
16: <sup> tags

BREAKING CHANGE: None

Currently working on this PR

@raisedadead, sure, no problem! I'll get on it later tonight.


SecurityEvent Entity Updates Second Half

Change(s):

Updating entity mappings for
non merged /SecurityEvent detections

Reason for Change(s):

Updating entity mappings to be more thorough for future correlations

Version Updated:

yes

Testing Completed:

yes

Checked that the validations are passing and have addressed any issues that are present:

no

Hi @ddamenova,
Please refer to the comments above from @shainw, act accordingly, and also check for KQL validation failures. Thanks

Hi @ddamenova,
Please inform us when you have completed all the suggested changes from @shainw. Thank you.


foundryup errors with Unrecognized archive format

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

[ ] Foundry
[ ] Foundryup

What version of Foundry are you on?
forge 0.1.0 (339ff41 2022-03-09T00:27:43.665618+00:00)
What command(s) is the bug in?
foundryup
Operating System
macOS (amd)
Describe the bug
I am running foundryup to update my Forge version, but it errors out with:
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:-  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:-100   166  100   166    0     0    405      0 --:--:-- --:--:-- --:--:--   412
100     9  100     9    0     0     15      0 --:--:-- --:--:-- --:--:--    15
tar: Error opening archive: Unrecognized archive format

Closing, issue went away when I pulled the latest install script with curl -L https://foundry.paradigm.xyz | bash
`

Running into this at the moment. libusb can't get found although it is installed and used successfully by gpg. And:
foundryup: installing foundry (version , tag )
foundryup: downloading latest forge, cast, anvil, and chisel
################################################################################################################################################################################################################ 100.0%
tar: Error opening archive: Unrecognized archive format
foundryup: downloading manpages
################################################################################################################################################################################################################ 100.0%
tar: Error opening archive: Unrecognized archive format
foundryup: installed - forge 0.2.0 (0398a95 2023-01-01T00:04:54.768126Z)
foundryup: installed - cast 0.2.0 (0398a95 2023-01-01T00:04:54.768126Z)
foundryup: installed - anvil 0.1.0 (0398a95 2023-01-01T00:04:55.458842Z)
foundryup: installed - chisel 0.1.1 (0398a95 2023-01-01T00:04:55.245173Z)
foundryup: done

Ran into the error as well
foundryup: installing foundry (version nightly, tag nightly-ebe1b45187f56b6c024d63c55de2d520cb715639)
foundryup: downloading latest forge, cast, anvil, and chisel
################################################################################################################### 100.0%
tar: Error opening archive: Unrecognized archive format
foundryup: downloading manpages
################################################################################################################### 100.0%
foundryup: installed - forge 0.2.0 (4dc05d2 2023-01-18T00:15:05.61036Z)
foundryup: installed - cast 0.2.0 (4dc05d2 2023-01-18T00:15:05.61036Z)
foundryup: installed - anvil 0.1.0 (4dc05d2 2023-01-18T00:15:56.319353Z)
foundryup: installed - chisel 0.1.1 (4dc05d2 2023-01-18T00:15:56.54533Z)
foundryup: done

+1 - Same issue on MacOS Ventura 13.1 (M1)
foundryup: installing foundry (version nightly, tag nightly-ebe1b45187f56b6c024d63c55de2d520cb715639)
foundryup: downloading latest forge, cast, anvil, and chisel
############################## 100.0%
tar: Error opening archive: Unrecognized archive format
foundryup: downloading manpages
############################## 100.0%
/Users/<username>/.foundry/bin/foundryup: line 139: /Users/<username>/.foundry/bin/forge: No such file or directory

+1 Same error
foundryup: installing foundry (version nightly, tag nightly-ebe1b45187f56b6c024d63c55de2d520cb715639)
foundryup: downloading latest forge, cast, anvil, and chisel
######################################################################################################################################################### 100.0%
tar: Error opening archive: Unrecognized archive format
foundryup: downloading manpages
######################################################################################################################################################### 100.0%
/Users/aman/.foundry/bin/foundryup: line 139: /Users/aman/.foundry/bin/forge: No such file or directory
foundryup: installed -
foundryup: warning:
There are multiple binaries with the name 'forge' present in your 'PATH'.
This may be the result of installing 'forge' using another method,
like Cargo or other package managers.
You may need to run 'rm ' or move '/Users/aman/.foundry/bin'
in your 'PATH' to allow the newly installed version to take precedence!
/Users/aman/.foundry/bin/foundryup: line 139: /Users/aman/.foundry/bin/cast: No such file or directory

hmm, this looks like a curl issue,
could you please try after reinstalling foundryup curl -L https://foundry.paradigm.xyz | bash

i have the same issue - i'm installing currently freshly using:
curl -L https://foundry.paradigm.xyz | bash
and i get the error:
warning: libusb not found. You may need to install it manually on MacOS via Homebrew (brew install libusb).
Then when I run foundryup I get the same error everyone else is reporting

I believe the libusb issue has to do with the script searching for a specific version of it:
if [[ "$OSTYPE" =~ ^darwin && ! -f /usr/local/opt/libusb/lib/libusb-1.0.0.dylib ]]; then
    echo && echo "warning: libusb not found. You may need to install it manually on MacOS via Homebrew (brew install libusb)."
fi

However there is a newer version out; when I run brew link libusb:
Warning: Already linked: /opt/homebrew/Cellar/libusb/1.0.26

seems to be working now...not sure what has changed

Can confirm that the unrecognized archive format is not appearing any more on my machine as well.
But the libusb not found still persists, although it is properly installed.
warning: libusb not found. You may need to install it manually on MacOS via Homebrew (brew install libusb).

Detected your preferred shell is zsh and added foundryup to PATH. Run 'source /Userxxxx/.zshrc' or start a new terminal session to use foundryup.
Then, simply run 'foundryup' to install Foundry.

foundryup: downloading latest forge, cast, anvil, and chisel
################################################################################################################################################################################### 100.0%
foundryup: downloading manpages
################################################################################################################################################################################### 100.0%
foundryup: installed - forge 0.2.0 (d7019d0 2023-02-07T17:33:59.346172Z)
foundryup: warning:
There are multiple binaries with the name 'forge' present in your 'PATH'.
This may be the result of installing 'forge' using another method,
like Cargo or other package managers.
You may need to run 'rm /Userxxxx/.cargo/bin/forge' or move '/Userxxxx/.foundry/bin'
in your 'PATH' to allow the newly installed version to take precedence!
foundryup: installed - cast 0.2.0 (d7019d0 2023-02-07T17:33:59.346172Z)
foundryup: warning:
There are multiple binaries with the name 'cast' present in your 'PATH'.
This may be the result of installing 'cast' using another method,
like Cargo or other package managers.
You may need to run 'rm /Userxxxx/.cargo/bin/cast' or move '/Userxxxx/.foundry/bin'
in your 'PATH' to allow the newly installed version to take precedence!
foundryup: installed - anvil 0.1.0 (d7019d0 2023-02-07T17:34:22.205508Z)
foundryup: warning:
There are multiple binaries with the name 'anvil' present in your 'PATH'.
This may be the result of installing 'anvil' using another method,
like Cargo or other package managers.
You may need to run 'rm /Userxxxx/.cargo/bin/anvil' or move '/Userxxxx/.foundry/bin'
in your 'PATH' to allow the newly installed version to take precedence!
foundryup: installed - chisel 0.1.1 (d7019d0 2023-02-07T17:34:21.903621Z)
foundryup: done

seems to be working now...not sure what has changed

See this comment: https://github.com/foundry-rs/foundry/issues/4295#issuecomment-1421158538
and previous discussion

But the libusb not found still persists, although it is properly installed.

Can you still run the binaries with no problems? We might just not be picking it up during installation


Feature Request: Baby Crying binary sensor

Is your feature request related to a problem? Please describe.
Not related to a problem, just missing
Describe the solution you'd like
A binary sensor for the baby crying feature. As of now i can get a push notification from the Tapo app.

Duplicate of https://github.com/JurajNyiri/HomeAssistant-Tapo-Control/issues/304


Can't create a All Day event today's date?

I am trying to create a All Day event in today's date, but I can't create anymore with your latest fixes.?

do you have any code reproducing it? tests and docs work fine


chore: add sourcemaps for JS and minified files #10999

Changes:

add sourcemaps for js files (foundation.js and all plugins)
add sourcemaps for minified files (foundation.min.js, foundation-*.min.css)
improve documentation for dist:javascript gulp task

Related to https://github.com/zurb/foundation-sites/pull/10998 (@DanielRuf)
Closes https://github.com/zurb/foundation-sites/issues/10999 (@ncoden)

Will check tomorrow.

Will check tomorrow.

Poke @DanielRuf 😄

@DanielRuf Fixed.


Gmail refuses email to multiple gmail recipients, claiming From: field is missing

Hi,
On Monday (5 days ago) answered a group email where most (all but two of the 24 of the 30 ish) recipients where gmail addresses.
I immediately got an "Undelivered Mail Returned to Sender" return from my email provider's mailer daemon, with this, for each gmail user:
<EMAIL_ADDRESS>host gmail-smtp-in.l.google.com[<IP_ADDRESS>]
said: 550-5.7.1 [<IP_ADDRESS> 11] Our system has detected that this
message is 550-5.7.1 not RFC 5322 compliant: 550-5.7.1 'From' header is
missing. 550-5.7.1 To reduce the amount of spam sent to Gmail, this message
has been 550-5.7.1 blocked. Please visit 550-5.7.1
https://support.google.com/mail/?p=RfcMessageNonCompliant 550 5.7.1 and
review RFC 5322 specifications for more information. o6-v6si12588343wmb.167
- gsmtp (in reply to end of DATA command)

I tried to check if the From: field was indeed empty when sending email from K9, but saw this in my sent message's source:
From: axx | axel simon<EMAIL_ADDRESS>I tried to remove the pipe '|' in case that was throwing Gmail's filters but, no difference. I tried again a few days later, in case it was very temporary issue, but still the same error.
I also tried to send an email to only one gmail contact, it went through fine.
On the other hand, I also investigated with my email provider, and they suggested to try sending via they webmail, which worked.
So the situation is a bit weird. It might be a transient issue with gmail, but I thought it was worth reporting.
Expected behavior
Gmail should accept and deliver the email.
Actual behavior
Gmail throws back a en error complaining the message doesn't conform to RFC 5322
Steps to reproduce

Reply all to a message with multiple gmail users

Environment
K-9 Mail version: 5.403 (Fdroid)
Android version: 5.1.1 (I know…)
Account type (IMAP, POP3, WebDAV/Exchange): IMAP

Is this still an issue with the latest K-9 Mail version?

I haven't seen this in a while, but also haven't sent any emails to groups of gmail users. So hard to say, sorry.

It's probably not the From header, but the To header that got incorrectly wrapped.
See issue #822


uorb autogeneration

I've split this out of the new logger work.

Neat! @julianoes Can you build this branch on Snapdragon to make sure it still works?

@LorenzMeier I tested this on the Snapdragon and the ADSP crashes on ekf2 start.

on fmu-v4, I'm getting hardfaults with this branch.  According to main.map, the fault occurs while in DriverFramework::DevHandle::~DevHandle()

Aha that is an interesting pointer, thanks! I'll try to have a look sometime this week.

Does it make any sense that DriverFramework is involved?

The -Wl,--gc-sections change I mentioned in #4365 actually went in here.
If I had to guess...
On Tue, Apr 26, 2016 at 10:04 AM Mark Whitehorn<EMAIL_ADDRESS>wrote:

Does it make any sense that DriverFramework is involved?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/PX4/Firmware/pull/4366#issuecomment-214755935

Enable --print-gc-sections and see what is getting culled?

Yep
Removing unused section
vtable for DriverFramework::DFUIntList
vtable for DriverFramework::DFPointerList
DriverFramework::DFUIntList::DFUIntListNode::DFUIntListNode(unsigned int)
DriverFramework::DFUIntList::DFUIntListNode::~DFUIntListNode()
DriverFramework::DFUIntList::get(DriverFramework::DFUIntList::DFUIntListNode*, unsigned int&)
DriverFramework::DFUIntList::next(DriverFramework::DFUIntList::DFUIntListNode*&)
DriverFramework::DFUIntList::size()
DriverFramework::DFUIntList::clear()
DriverFramework::DFUIntList::empty()
DriverFramework::DFUIntList::erase(DriverFramework::DFUIntList::DFUIntListNode*)
DriverFramework::DFUIntList::pushBack(unsigned int)
DriverFramework::DFUIntList::pushFront(unsigned int)
DriverFramework::DFUIntList::DFUIntList()
DriverFramework::DFUIntList::~DFUIntList()
DriverFramework::DFUIntList::~DFUIntList()
DriverFramework::DisableCopy::~DisableCopy()
DriverFramework::DisableCopy::~DisableCopy()
DriverFramework::DFPointerList::DFListNode::DFListNode(void*)
DriverFramework::DFPointerList::DFListNode::~DFListNode()
DriverFramework::DFPointerList::get(DriverFramework::DFPointerList::DFListNode*)
DriverFramework::DFPointerList::next(DriverFramework::DFPointerList::DFListNode*&)
DriverFramework::DFPointerList::size()
DriverFramework::DFPointerList::clear()
DriverFramework::DFPointerList::empty()
DriverFramework::DFPointerList::erase(DriverFramework::DFPointerList::DFListNode*)
DriverFramework::DFPointerList::pushBack(void*)
DriverFramework::DFPointerList::pushFront(void*)
DriverFramework::DFPointerList::DFPointerList()
DriverFramework::DFPointerList::~DFPointerList()
DriverFramework::DFPointerList::~DFPointerList()
DriverFramework::clockGetRealtime(timespec*)
DriverFramework::clockGetMonotonic(timespec*)
DriverFramework::absoluteTimeInFuture(unsigned long long)
DriverFramework::DevMgr::initialize()
DriverFramework::DevMgr::waitForUpdate(DriverFramework::DFPointerList&, DriverFramework::DFPointerList&, unsigned int)
DriverFramework::DevMgr::getNextDeviceName(unsigned int&, char const**)
DriverFramework::DevMgr::setDevHandleError(DriverFramework::DevHandle&, int)
DriverFramework::DevMgr::finalize()
DriverFramework::SyncObj::waitOnSignal(unsigned long)
DriverFramework::SyncObj::lock()
DriverFramework::SyncObj::signal()
DriverFramework::SyncObj::unlock()
DriverFramework::SyncObj::SyncObj()
DriverFramework::SyncObj::~SyncObj()
DriverFramework::DevHandle::write(void const*, unsigned int)
DriverFramework::HandleObj::~HandleObj()
DriverFramework::HandleObj::~HandleObj()
DriverFramework::initMutex(pthread_mutex_s&)
Sensors::apply_mag_calibration(DriverFramework::DevHandle&, mag_calibration_s const*, int)
Sensors::apply_gyro_calibration(DriverFramework::DevHandle&, gyro_calibration_s const*, int)
Sensors::apply_accel_calibration(DriverFramework::DevHandle&, accel_calibration_s const*, int)

We either need to do something to force it to be kept, or ifdef DriverFramework usage everywhere.

I don't suppose this explains the earlier hardfaults on the newLogger branch.

With the cmake -Wl,--gc-sections change on master I'm not seeing any hard faults and the same DriverFramework sections removed. Let's test and merge that separately. #4373

I'll re-test tomorrow.

@julianoes Is there anything I should look at here for DriverFramework?

Ran the same pixracer for 2 hours and 15 minutes on this branch without a crash. No changes to the source, but did checkout and build/flash other branches before rebuilding and flashing this one.
2000_01_01_00_01_40: [cmd] ARMED by command line
2000_01_01_00_01_41: [blackbox] /fs/microsd/log/sess008
2000_01_01_00_01_41: [blackbox] recording: log001.px4log
2016_04_26_18_34_59: [cmd] DISARMED by arm/disarm component command
2016_04_26_18_35_00: [blackbox] stopped (14889 drops)
2016_04_26_19_23_57: [cmd] ARMED by arm/disarm component command
2016_04_26_19_23_57: [blackbox] /fs/microsd/log/2016-04-26
2016_04_26_19_23_57: [blackbox] recording: 19_23_57.px4log
2016_04_26_19_25_22: [cmd] DISARMED by arm/disarm component command
2016_04_26_19_25_23: [blackbox] stopped (568 drops)
2016_04_26_19_50_46: [inav] GPS signal found
2016_04_26_19_50_47: [inav] init ref: 39.7502382, -105.0979762, 1718.8
2016_04_26_19_52_34: [inav] GPS signal lost
2016_04_26_19_57_01: [inav] GPS signal found
2016_04_26_19_57_37: [inav] GPS signal lost
2016_04_26_20_00_57: [inav] GPS signal found
2016_04_26_20_02_42: [inav] GPS signal lost
2016_04_26_20_09_11: [inav] GPS signal found
2016_04_26_20_13_19: [inav] GPS signal lost
2016_04_26_20_15_29: [inav] GPS signal found
2016_04_26_20_21_58: [inav] GPS signal lost
2016_04_26_20_22_34: [inav] GPS signal found
2016_04_26_20_30_51: [cmd] ARMED by command line
2016_04_26_20_30_51: [blackbox] /fs/microsd/log/2016-04-26
2016_04_26_20_30_51: [blackbox] recording: 20_30_51.px4log
2016_04_26_20_40_00: TAKEOFF DETECTED
2016_04_26_20_40_04: LANDING DETECTED
2016_04_26_20_42_19: TAKEOFF DETECTED
2016_04_26_20_42_22: LANDING DETECTED
2016_04_26_20_43_03: TAKEOFF DETECTED
2016_04_26_20_43_06: LANDING DETECTED
2016_04_26_20_43_52: TAKEOFF DETECTED
2016_04_26_20_43_54: LANDING DETECTED
2016_04_26_20_44_19: [inav] GPS signal lost
2016_04_26_20_46_28: [inav] GPS signal found
2016_04_26_20_48_01: [inav] GPS signal lost
2016_04_26_20_48_30: [inav] GPS signal found
2016_04_26_20_49_32: TAKEOFF DETECTED
2016_04_26_20_49_35: LANDING DETECTED
2016_04_26_20_59_03: [inav] GPS signal lost
2016_04_26_20_59_04: [inav] GPS signal found
2016_04_26_20_59_07: [inav] GPS signal lost
2016_04_26_20_59_37: [inav] GPS signal found

@kd0aij the uorb header #pragma pack(push, 1) is causing each struct to be misaligned.
For example in fw_pos_ctrl the sensor_combined struct was at 0x20021337
I'm trying to use __attribute__((aligned(4), packed)) instead, but there are other problems that GCC now catches like not being able to pass fields of a packed struct as a reference.

Interesting, but wouldn't unaligned accesses either fail completely, or always work?

For me it was always failing in the same place.

Every one of my hardfaults shows a return address on the stack at line 2295
of commander.cpp, but I don't see any connection between that line and orb
topics.
On Wed, Apr 27, 2016 at 8:47 PM Daniel Agar<EMAIL_ADDRESS>wrote:

For me it was always failing in the same place.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/PX4/Firmware/pull/4366#issuecomment-215292595

What's 2295 for you? In this branch and newLogger2 it's a comment block.
https://github.com/PX4/Firmware/blob/newLogger2/src/modules/commander/commander.cpp#L2295

@kd0aij @dagar
Hmm - this might be stack related - you can enable the CONFIG_ARMV7M_STACKCHECK and set a break in the point it vectors to the hard fault from the check preamble.
Nuttx and the PX4 code need to be in agreement on this setting: So - If you uses it do a clean build, after setting or unsetting it.

I was behind a few commits on uorb_autogen. Just committed a workaround which solves my crash: is yours in a different place?

@kd0aij _home is a generated orb struct (home_position_s). Going back a few changes so you can reproduce your hard fault, what is &_home.timestamp? Your work around avoids the problem by not passing something unaligned to hrt_elapsed_time().
Thanks for the input @davids5. I'll give that a try tomorrow.

oh, I didn't realize you had already eliminated the problem by aligning the structs. I reverted my change and no problem so far. Is there any impact on logger efficiency with the new alighment?

and my workaround should go away... I assume you'll prefer to drop it during a rebase.

Just tested this again, and it still segfaults on the DSP side on Snapdragon.

It crashes somewhere in commander, I'm doing some debugging.

I narrowed the problem down but I can't figure out why exactly the crash happens.
The failing line is:
https://github.com/PX4/Firmware/blob/ec4d2fa8de963e1d9379aa32e57b1723619b7b0d/src/modules/commander/commander.cpp#L1740
The pointer and value at &sensors.barometer_failure are correct.
The crash is not happening when hrt_elapsed_time() is used without the volatile keyword:
The declaration is here: https://github.com/PX4/Firmware/blob/ec4d2fa8de963e1d9379aa32e57b1723619b7b0d/src/drivers/drv_hrt.h#L103
Alternatively, you can work around the bug like below but it won't prevent the same thing from happening anywhere else:
const hrt_abstime temp_timestamp = sensors.baro_timestamp[0];
if (hrt_elapsed_time(&temp_timestamp) < FAILSAFE_DEFAULT_TIMEOUT) {

@mcharleb, @LorenzMeier: any idea why Snapdragon would crash when calling the function which uses  has a volatile argument?

@julianoes you said &sensors.barometer_failure was correct, but was it 4 byte aligned? What compiler do you use for eagle? I'm guessing not gcc or clang?
I borrowed this from mavlink to handle packing and alignment, but so far ignored the !GNUC part.
// Macro to define packed structures
#ifdef __GNUC__
#define ORBPACKED( __Declaration__ ) __Declaration__ __attribute__((aligned(4), packed))
#else
#define ORBPACKED( __Declaration__ ) __pragma( pack(push, 1) ) __Declaration__ __pragma( pack(pop) )
#endif

In general I don't like that we're messing with the packing and alignment of uorb structs. This is mainly for logger simplicity and performance so it can do a single copy per message. I'm going to see if we can avoid messing with packing and alignment by sorting the struct by size (already done) and adding some small filler where necessary.

@dagar Why does hrt_elapsed_time crash when given an unaligned pointer, when (now - _home.timestamp) runs OK?

@kd0aij the compiler knows _home.timestamp comes from a packed struct and does assume alignment. When we take a pointer that information is lost and the compiler assumes default alignment.

Would have saved some time if the compiler warned about such things...

It is a longstanding issue. Another problem is with these size lookups; they're really used to order fields in the struct. First off, they mix and match convention of bytes vs bits (char is 1, int8 is 8... what?) second they do not really account for alignment concerns. Only types whose sizeof(type) % 4 == 0 should be promoted first in the struct, everything else should follow. Adding to that, there is no packing applied to the composite structs that we define there, so we can't necessarily assume they are packed unless we explicitly pack them.
The most insidious issue is the timestamp on _home should be word aligned since it comes first in the struct. Even in the case of packed struct members I am surprised to see GCC placing members without word alignment. That just does not make sense to me.

I would very very strongly argue that we should not try to screw around with the compiler's default alignment requirements (optimally ordering members is another story and totally fair game).
Get rid of all aligned/packed attributes and use the python codegen to provide packed serial/deserialization operators for user buffers similar to how mavlink does it. The next gen logger can use those operators. Everything else is going to be a headache game of whack-a-mole.

That's the plan - https://github.com/dagar/Firmware/blob/newLogger2/msg/templates/uorb/msg.cpp.template#L128
Almost working...

@kd0aij @dagar: sorry, I wrote that down wrongly. It is baro_timestamp, not baro_failure.

@dagar Ping me when this is vetted. Thanks for hanging in there!

The new logger and uorb generation are more intertwined than I originally thought, so I'm going to keep them together in the same PR again. https://github.com/PX4/Firmware/pull/4365


no report/excel to be found anywhere

as a user i'm not getting any report back in Excel nor a location where the file is located, I'm only getting the logging output (scan list)

Can you share the console output? DO you se any exception?

seems i didn't get notified of the follow up on this : no error given


How do I block traffic between certain containers on a docker network?

Context: I have a large network for all containers connected to my reverse proxy (traefik), and I'd like to prevent containers from talking to each other, allowing them to only communicate with the traefik container.
How would I set up iptables for this scenario? Both for non-swarm and swarm cases?
I haven't been able to find good documentation on how exactly the networks/vips are set up, so I'm just trying to reverse engineer the setup. It seems that for a non-swarm configuration, it should be as simple as creating some iptable rules in the network namespace of that docker network to block all traffic on its subnet unless the source or destination are the traefik container.
Swarm is a little trickier because of the vips and the load balancing. Also, I have traefik configured with host-mode networking so that it sees the actual source ip of incoming traffic, rather than the internally routed one, so this adds a small layer of complication.
As I'm working through it, I'm hoping someone might have an answer or guide readily available.

Ended up implementing this in https://github.com/kaysond/trafficjam
My architecture.md explains some of how this works.


Multi-file Upload

Is your feature request related to a problem? Please describe.
Current version of PASS supports single file uploads to Solid via HTML input element
Describe the solution you'd like
HTML input allows for multi-file uploads with the multiple attribute. We just need to update these elements and refactor all relevant components and functions

Opening new branch for #77, issue-77/multi-file-uploads

Function for multi-file upload is viable as proof-of-concept. Will keep as back log at this time.

Currently working on new system for uploading files on to Solid Pods. See PR #249.

Think what we've established in recent PR #249 helped us upload multiple files to Solid Pods one at a time and could assign metadata to them individually.
Think there might still be a case where someone wishes to upload a set of related files have them all use the same metadata description, but the concept would need to be flushed out first.


TOC Click changes Map

It works, however, clicking the TOC too much messes up the event listener causing lat long issues. Note I used listen on. Maybe listenTo is the better choice.

@brian-smith723 this might break when the annotation stuff is applied


Modify the retrieveAll() sort direction to match the index.

Closes #285

The changes in this merge request do not modify the results of the retrieveAll() endpoints, except in one case. When retrieving all versions of all collections:
GET /api/collections?versions=all

The sort order of versions within each returned collection is reversed. Previously the versions were ordered oldest first, they are now ordered most recent first. This makes the sort order consistent with the sort order when retrieving all versions of one particular collection, which are already ordered most recent first.


How to activate GPU training?

I tried to find out the GPU training setting. Can you share the code snippet where can I set the GPU for training and testing?

you should be better use google colab and active GPU runtime easily

This is the code snippet:
def get_device():

    return torch.device('cuda') if torch.cuda.is_available() else torch.device('cpu')

I hope this will solve resolve your issues.


Fix SRC3 examples code

Type of change

Improvement
Documentation

Changes
The following changes have been made:

Fixed an obvious typo in SRC3 Example

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

Thanks for the fix!


Run Benchmarks for EIP-2046 & 2666

On this week's AllCoreDevs, the benchmarks for EIP-2046 and EIP-2666 were brought up again. We should run them, and share our numbers with Alex Vlasov (@shamatar, not PegaSys' Alex Vlasov :-)).

Would it be possible to get some data before this week ACD call?

@shamatar where are the other published results for EIP2666 repricing? We want to make sure we're doing an apples to apples benchmark. Thanks!

@timbeiko
There is a public spreadsheet here. When converting execution time to the gas I use a constant 35MGas/second.

On the benchmarking side:

hash functions (both SHA256/Ripemd160 called via precompiles, and Keccak being a VM function) get random bytestrings of fixed length as inputs
BNAdd just gets two random points
BNMul gets 2^256 - 1 as a scalar and random point
BN pairing should get N pairs of random G1+G2 points, but since no repricing is proposed I was lazy to put the results there

There is NO intermediate glue (from EVM side). Function are pure like function(bytes) -> bytes and only this execution time is measured.

Was there a specific machine these are run on?  t3.micro?  MacBook Pro 201x?

It’s 2.9 GHz MBP 2018 (or may be 2.6, can check again tomorrow), but performance of a concrete machine is captured by the conversion constant. I've measured it based on ecrecover and BNMUL performance on my machine with turbo boost being off (assuming these two precompiles are well priced)

My benchmark code - https://github.com/shemnon/besu/pull/3
A run in my 2018 Macbook Pro (2.6 GhZ i7) - https://gist.github.com/shemnon/0ddba91be501fa23291bdec9107fe99a
SHA256 is a Java built in
Keccak and RipeMD are pure java, and hence slower.  A native impl will be faster for both of these two and hopefully we will get bounties out this year for those two.
AltBN are native Rust code (borrowed from SputnikVM).  I'm not sure why the mul is so far off, there's probably room to optimize there.


[Bug] Fix themes module import causing crashes in va-mobile-app

Description of Change

Fixes the TypeError: Cannot read property 'themes' of undefined error we are seeing in the app by using a namespace import instead of destructured import in our hooks file.
Set tokens and assets packages as peerDependencies and devDependencies in the components package.json as the app is now adding tokens and assets directly in their dependencies
Exports the useTheme() hook which I've tested in the app and closes #377.

I could not find a way to keep the destructured import after several attempts of refactoring our tokens exports. I think the namespace import is an acceptable workaround for now.
Things I've tried without success:

Combining colors and themes exports into index.js instead of having them as separate colors.js and themes.js
Combining types/colors.ts and types/themes.ts into index.d.ts
Declaring the @department-of-veterans-affairs/mobile-tokens module in index.d.ts and wrapping it around the color/themes types
Moving index.d.ts to same folder as index.js
Using commonJS exports instead
Setting "type": "module" in package.json and converting style dictionary config files to .cjs
Setting exports in package.json
Adjusting several settings in va-mobile-app's tsconfig.json

I've created a follow-up ticket #390 to determine root cause when we have additional bandwidth.
Testing Packages

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
Screenshots/Video

Testing

[x] Tested on iOS
[x] Tested on Android
[x] Tested on Web

PR Checklist
Code reviewer validation:

General

[ ] PR is linked to ticket(s)
[ ] PR has changelog label applied if it's to be included in the changelog
[ ] Acceptance criteria:

All satisfied or
Documented reason for not being performed or
Split to separate ticket and ticket is linked by relevant AC(s)

[ ] Above PR sections adequately filled out
[ ] If any breaking changes, in accordance with the pre-1.0.0 versioning guidelines: a CU ticket has been created for the VA Mobile App detailing necessary adjustments with the package version that will be published by this ticket

Code

[ ] Tests are included if appropriate (or split to separate ticket)
[ ] New functions have proper TSDoc annotations

Publish

If changes warrant a new version per the versioning guidelines and the PR is approved and ready to merge:

[ ] Merge main into branch
[ ] Merge branch to main
[ ] Verify that Check Component Integrations workflow ran successfully
[ ] Publish new version after merging

Screenshots/Video

Just as a sanity check: your going into the app did go deeper than the login screen, right? It shouldn't matter since main and correspondingly your alpha test branches should have theme used by the Button component, but given how it slipped by the first time, would be good to have confirmation that screens containing Links/Segmented controls were also accessed, didn't crash, and toggled light/dark mode as expected.

Screenshots/Video

Just as a sanity check: your going into the app did go deeper than the login screen, right? It shouldn't matter since main and correspondingly your alpha test branches should have theme used by the Button component, but given how it slipped by the first time, would be good to have confirmation that screens containing Links/Segmented controls were also accessed, didn't crash, and toggled light/dark mode as expected.

Yup, went through the sections of the app with SegmentedControl and Link as well. Toggling light/dark mode works as expected. Our components do seem to toggle between light/dark mode faster than the rest of the app but I think that has always been the case.

Our components do seem to toggle between light/dark mode faster than the rest of the app but I think that has always been the case.

It has, the useColorScheme hook works faster than the app theme wrapper.


WIP AuthZ plugin interface refactoring

There are still a couple of FIXMEs.

Hey @gkleiman thanks for this change - the approach is great!
The one problem I see is spread over a lot of places: the transaction spawned by groupManager.update is not taken into account for consistency checking. Made some comments.

Hey @gkleiman this looks awesome! Thanks!
Please:

repair the tests
remove the frameworeName from the pluginconf trait
squash the commits

and I will merge.

@aquamatthias one test is still red and I haven't removed the frameworkName yet, but you can already review my last commit while I fix the remaining issues.

Please see my (very minor comments).
Squash and I will merge.

Thanks! :+1:

Yay!
                                              ____________
                                --)-----------|____________|
                                              ,'       ,'
                -)------========            ,'  ____ ,'
                         `.    `.         ,'  ,'__ ,'
                           `.    `.     ,'       ,'
                             `.    `._,'_______,'__
                               [._ _| ^--      || |
                       ____,...-----|__________ll_|\
      ,.,..-------"""""     "----'                 ||
  .-""  |=========================== ______________ |
   "-...l_______________________    |  |'      || |_]
                                [`-.|__________ll_|      Incom's T-65B X-wing
                              ,'    ,' `.        `.      Space Superiority
                            ,'    ,'     `.    ____`.    Starfighter (2)
                -)---------========        `.  `.____`.
                                             `.        `.
    By Unknown                                 `.________`.
    Patched by LS              --)-------------|___________|


Fix default Servlet TCK tests

Fix the following TCK tests related to default Servlet functionality

com/sun/ts/tests/servlet/spec/defaultmapping/URLClient.java#defaultservletTest
.com/sun/ts/tests/servlet/spec/defaultmapping/URLClient.java#defaultservletTest1

These tests seem to be passing now.


fix: support different tag association queries for COLUMN object types

This is intended to close #1201 by implementing some logic to handle the slightly different query needed for COLUMN tag associations vs. other object types.
Test Plan

[ ] acceptance tests

[ ] …

References

/ok-to-test sha=be77caa

/ok-to-test sha=be77caa

    tag_association_test.go:45:
        	Error Trace:	/home/runner/work/terraform-provider-snowflake/terraform-provider-snowflake/pkg/snowflake/tag_association_test.go:45
        	Error:      	Not equal:
        	            	expected: "c"
        	            	actual  : "\"a\".\"b\".\"c\""

        	            	Diff:
        	            	--- Expected
        	            	+++ Actual
        	            	@@ -1 +1 @@
        	            	-c
        	            	+"a"."b"."c"
        	Test:       	TestTableColumnName

Hello @rmorris1218! There is a minor issue with the test case TestTableColumnName. If you could please correct then we can get this PR merged.

We are calling ALTER TABLE x ALTER COLUMN y when should be: ALTER TABLE x MODIFY COLUMN y

Thanks for the heads up! Updated the syntax for these.

@sfc-gh-swinkler  any chance this can be reviewed by the Snowflake team? It's a blocker for us and the requested changes seems to be already applied by the owner. Let me know if I can help  with something

/ok-to-test sha=9206993

/ok-to-test sha=0c041916

/ok-to-test sha=0c04191

/ok-to-test sha=0c04191

Sorry this slipped through the cracks. This looks okay to me. what i will do is merge it and resolve any workflow errors myself and get in the next release.


Allow dot character in tag names (#86)

(also increase size of field for raw tag QL)

Could someone review this? @tsegismont / @karelhala / @mtho11


Introduction to Push Notifications

ILT content: https://docs.google.com/document/d/1nc0ADQUNZhEz0xmAyDtvKhTRoplEWKuhWM_cc4Di5Fk/edit#heading=h.gxkloi8vwsa2
What we have today: https://developers.google.com/web/fundamentals/engage-and-retain/push-notifications/?hl=en and https://developers.google.com/web/fundamentals/getting-started/push-notifications/?hl=en
We may not need to migrate this content over if the current section is enough. Assigning to Joe initially to take a pass through ILT.

I've started work on this. Updated content is in the push-hackathon branch.
https://github.com/google/WebFundamentals/tree/push-hackathon

#3556 Merged earlier today.


CI housekeeping

List of changes:

bump GitHub Action / Mix deps
support on the min/max Elixir/Erlang versions in CI
use lint as flag to running linting job with latest Elixir/Erlang version
remove duplicate CI jobs

Thanks!

:partying_face: :partying_face: :partying_face: :partying_face: :partying_face:


feat(contrib/linode): update firewall to allow for cluster expansion.

Adds a parameter to the Linode firewall setup script in order to open ports 2379,2380 when adding nodes to an existing cluster so that etcd can be properly initialized. Without this (after the firewall has been setup on the CoreOS cluster), there is no way to easily add nodes to an existing cluster.

After looking at this a bit, I see that leaving these ports open is necessary for adding members, but when I add members, I have to leave them open (even after CoreOS has been fully installed). I think this has something to do with the additional members being added as proxies in etcd. I'll close this PR and resubmit once I have some more concrete information.


⚠ ClamAV has degraded performance

In 5180009, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 475 ms

Resolved: ClamAV performance has improved in 8664265.


Listeners should have visitErrorNode throw an exception by default

It is possible to get a false positive if errorNodes are encountered. The default implementation is to do nothing. The user can implement an exception to be thrown manually, but I'm concerned that it will be easy to forget and lead to difficult to diagnose bugs.
I'm going to see if this can be built into the generated code ANTLR produces with its "tool" code.

I was having difficulty getting tests in ANTLR to pass with my modifications, although I didn't have a dedicated amount of time to devote to the problem.
A stopgap solution might be to have PDSL log a severe error if it notices any nodes are error nodes.


removed scalar strict type

this method supports more types then string (as per phpdoc)

I do think it should be a string in this case, even if the api might support non-strings. The argument is meant to be the rootElementName, and nothing else. I'll drop into the other issue to give an alternative.

@evert ok then we should change the phpdoc...

I think the phpdoc is mostly correct. I removed all @param lines from phpdocs where they are redundant with the actual typehint. Or is the first paragraph maybe confusing

your file is using strict types. when the caller of your file also uses strict-types you can only pass strings into this method, no matter what the phpdoc additionally defines.

Yes but there is no phpdoc for that parameter. There is only the type hint.

Ohh.. i see... there is only one phpdoc for the middle param. Everything fine then.


chore: change company name (nhnent -> nhn)

ok to test

ok to test

retest this please

ok to test

ok to test

retest this please

retest this please


Feature 5 - Implement contact form

Link for linkedin profile
Link for github account
Send email

Secret key should be changed
Email & password should be stored elsewhere


yq: 3.2.3 -> 3.3.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for yq is: Command-line YAML/XML/TOML processor - jq wrapper for YAML, XML, TOML documents
meta.homepage for yq is: https://github.com/kislyuk/yq
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.3.0 with grep in /nix/store/7p2km3qv711qmhbak1jv2pgqwy1n62lv-python3.11-yq-3.3.0
found 3.3.0 in filename of file in /nix/store/7p2km3qv711qmhbak1jv2pgqwy1n62lv-python3.11-yq-3.3.0

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

44 total rebuild path(s)

44 package rebuild(s)

First fifty rebuilds by attrpath
dart-sass
discourse
discourseAllPlugins
domine
expidus.calculator
expidus.file-manager
firmware-updater
flet-client-flutter
fluffychat
fluffychat-web
flutter
flutter313
flutter316
flutter319
flutterPackages.stable
flutterPackages.v3_13
flutterPackages.v3_16
flutterPackages.v3_19
hover
intiface-central
jekyll
jsonnet
localsend
nuget-to-nix
protoc-gen-dart
python311Packages.flet
python311Packages.flet-runtime
python311Packages.nbdev
python311Packages.yq
python312Packages.flet
python312Packages.flet-runtime
python312Packages.nbdev
python312Packages.yq
quarto
quartoMinimal
rstudio
rstudio-server
rstudioServerWrapper
rstudioWrapper
rustdesk-flutter
shopware-cli
stubby
yq
yubioath-flutter

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/7p2km3qv711qmhbak1jv2pgqwy1n62lv-python3.11-yq-3.3.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/9adf0708a3b60e2a71e40d2fd5e98c8b12d9beeb#yq

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/7p2km3qv711qmhbak1jv2pgqwy1n62lv-python3.11-yq-3.3.0
ls -la /nix/store/7p2km3qv711qmhbak1jv2pgqwy1n62lv-python3.11-yq-3.3.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

71 packages built:

dart-sass
dart-sass.pubcache
discourse
discourseAllPlugins
domine
domine.pubcache
expidus.calculator
expidus.calculator.debug
expidus.calculator.pubcache
expidus.file-manager
expidus.file-manager.debug
expidus.file-manager.pubcache
firmware-updater
firmware-updater.debug
firmware-updater.pubcache
flet-client-flutter
flet-client-flutter.debug
flet-client-flutter.pubcache
fluffychat
fluffychat-web
fluffychat-web.pubcache
fluffychat.debug
fluffychat.pubcache
flutter
flutter313
flutter316
hover
intiface-central
intiface-central.debug
intiface-central.pubcache
jekyll
jsonnet
jsonnet.doc
localsend
localsend.debug
localsend.pubcache
nuget-to-nix
protoc-gen-dart
protoc-gen-dart.pubcache
python311Packages.flet
python311Packages.flet-runtime
python311Packages.flet-runtime.dist
python311Packages.flet.dist
python311Packages.nbdev
python311Packages.nbdev.dist
yq (python311Packages.yq)
yq.dist (python311Packages.yq.dist)
python312Packages.flet
python312Packages.flet-runtime
python312Packages.flet-runtime.dist
python312Packages.flet.dist
python312Packages.nbdev
python312Packages.nbdev.dist
python312Packages.yq
python312Packages.yq.dist
quarto
quartoMinimal
rstudio
rstudio-server
rstudioServerWrapper
rstudioWrapper
rustdesk-flutter
rustdesk-flutter.debug
rustdesk-flutter.pubcache
shopware-cli
stubby
stubby.man
stubby.stubbyExampleJson
yubioath-flutter
yubioath-flutter.debug
yubioath-flutter.pubcache

Maintainer pings
cc @womfoo @SuperSandro2000 for testing.

Apparently upstream broke float parsing: https://github.com/kislyuk/yq/issues/187
This made it to my system and broke my config lol.


tempo & mido-play

Hi! First, thanks for mido - it's great, and I'm especially impressed with how everything I've tried so far has worked well cross platform between my desktop linux system and on Raspberry Pi! I super-appreciate the efforts here.
I am using mido-play to play .midi file data out over a hardware midi interface (via rtmidi) to some hardware midi synthesizers, drum machines, and effects. This is essentially working (even on the Pi!) and is super rad.
This bit of mido-play: https://github.com/mido/mido/blob/stable/bin/mido-play#L65-L67
Seems to indicate that tempo meta events are 'ignored', and so every midi file seems to be played back at 120 bpm.
This seems plausible, since mido-play uses the send_message mechanism, which seems to boil down to a sendMessage in rtmidi (which under alsa looks like: https://github.com/thestk/rtmidi/blob/cc887191c3b4cb6697aeba5536d9f4eb423090aa/RtMidi.cpp#L2330) and seems to be handling the timing stuff 'automatically'. Moreover, I don't see a way through rtmidi to change alsa's notion of tempo (see: "Setting queue tempo" here: https://alsa-project.org/alsa-doc/alsa-lib/seq.html)
Now, I may be missing something, but that seems to add up to mido-play only playing 120bpm music.
Since I am the author of the midi files, I am thinking of working around this by setting the ppq sufficiently high, and inserting a shim that can re-tick a (non-120-bpm) composition to play at the correct speed assuming the above behaviors of mido-play -> rtmidi -> alsa.
Any pointers regarding mido-play and tempo would be appreciated. And regardless, thanks again for your time and effort on this project. Without it I'd likely need to start by writing some lame subset of it. :christmas_tree:

Hello sir,
I'm not sure if you already solved this problem, but in case you haven't, I wanted to let you know that I was just struggling with this same issue. I discovered that you can control the BPM by multiplying the sleep time of each MIDI message. It's a bit of an annoying calculation, but works great for me, as I am just doing it by ear. Hope this is helpful.

Hello. I did eventually solve this, by replacing mido-play with aplaymidi in the system I was building. Good times were had by all, original, unique, generative live performances. Mido is still a great resource for cross-platform python midi stuff, and I'm grateful it exists. Stay rad.


Problems airgapped install oadp-operator 0.4.2 in Openshift 4.7

We have the Problem that we can't install OADP-Operator Airgapped over OperatorHUB. Operator was mirrored airgapped and the following Images are available:
imagecontentsourcepolicy:
spec:
  repositoryDigestMirrors:
  - mirrors:
    - registry.dev.net/olm-oadp-v47/konveyor-velero
    source: quay.io/konveyor/velero
  - mirrors:
    - registry.dev.net/olm-oadp-v47/konveyor-velero-restic-restore-helper
    source: quay.io/konveyor/velero-restic-restore-helper
  - mirrors:
    - registry.dev.net/olm-oadp-v47/konveyor-velero-plugin-for-gcp
    source: quay.io/konveyor/velero-plugin-for-gcp
  - mirrors:
    - registry.dev.net/olm-oadp-v47/konveyor-velero-plugin-for-csi
    source: quay.io/konveyor/velero-plugin-for-csi
  - mirrors:
    - registry.dev.net/olm-oadp-v47/konveyor-velero-plugin-for-microsoft-azure
    source: quay.io/konveyor/velero-plugin-for-microsoft-azure
  - mirrors:
    - registry.dev.net/olm-oadp-v47/konveyor-openshift-velero-plugin
    source: quay.io/konveyor/openshift-velero-plugin
  - mirrors:
    - registry.dev.net/olm-oadp-v47/konveyor-velero-plugin-for-aws
    source: quay.io/konveyor/velero-plugin-for-aws
  - mirrors:
    - registry.dev.net/olm-oadp-v47/konveyor-oadp-operator
    source: quay.io/konveyor/oadp-operator
  - mirrors:
    - registry.dev.net/olm-oadp-v47/konveyor-registry
    source: quay.io/konveyor/registry

catalogsource:
spec:
  displayName: olm-oadp-v47
  image: registry.dev.net/olm-oadp-v47/olm-oadp-v47-community-operator-index-pruned:v4.7.20220210
  publisher: ITZ v4.7.20220210
  sourceType: grpc

If we trigger a install there is a image missing during the operator install:
[root@vleosh3uxmda ~]# oc get pod -n openshift-marketplace | grep -i 922eeb749cba3b514ae88c3c0c90779d63f609efc3e5862a796bd3252edjs8b
922eeb749cba3b514ae88c3c0c90779d63f609efc3e5862a796bd3252edjs8b   0/1     Init:ImagePullBackOff   0          48m

describe to this pod there is a image missing which is not included in imagecontentsourcepolicy:
Events:
  Type     Reason          Age                    From               Message
  ----     ------          ----                   ----               -------
  Normal   Scheduled       50m                    default-scheduler  Successfully assigned openshift-marketplace/922eeb749cba3b514ae88c3c0c90779d63f609efc3e5862a796bd3252edjs8b to vlec2192ud2x.dev3.cloud
  Normal   AddedInterface  50m                    multus             Add eth0 [<IP_ADDRESS>/23]
  Normal   Pulled          50m                    kubelet            Container image "quay.io/openshift-release-dev/ocp-v4.0-art-dev@sha256:6ce678458f6b8bbc8c2bf38c2aa221fb7ad6f6207339735b6fac2aab076da7d6" already present on machine
  Normal   Created         50m                    kubelet            Created container util
  Normal   Started         50m                    kubelet            Started container util
  Normal   Pulling         49m (x4 over 50m)      kubelet            Pulling image "quay.io/openshift-community-operators/oadp-operator:v0.4.2"
  Warning  Failed          49m (x4 over 50m)      kubelet            Failed to pull image "quay.io/openshift-community-operators/oadp-operator:v0.4.2": rpc error: code = Unknown desc = error pinging docker registry quay.io: Get "https://quay.io/v2/": Service Unavailable
  Warning  Failed          49m (x4 over 50m)      kubelet            Error: ErrImagePull
  Warning  Failed          5m41s (x196 over 50m)  kubelet            Error: ImagePullBackOff
  Normal   BackOff         35s (x218 over 50m)    kubelet            Back-off pulling image "quay.io/openshift-community-operators/oadp-operator:v0.4.2"

In my experience, offline mirroring never worked with the community operator.
Install the Red Hat supported OADP operator v1.0.0 from the Red Hat Catalog. It is compatible with offline mirroring and disconnected installation.

Thanks for your Information. It seems to be that it is another Operator which is flagged as „disconnected“ in packagemanifest but doesn‘t work like expected. I give the Redhat based OADP a Chance and try  to install it in our disconnected Cluster.
Best regards,
Steffen

Let us know and feel free to close if 1.0.0 resolves your issue.
Thanks @alexisph for the help!

@stefffhem OADP 0.4.2 does work in disconnected installs if desired even though its a community operator. The logs above seem to indicate the bundle image was never mirrored, hence:
quay.io/openshift-community-operators/oadp-operator:v0.4.2

failing to pull. If you mirrored this image as well this would work yes?


Setup Clippy and other build tools

You can probably copy-pasta a lot from the build.yaml in intermodal: https://github.com/casey/intermodal/blob/master/.github/workflows/build.yaml
There are a lot of steps that you probably don't need, but they can just be deleted.

Oh nice, thanks! That looks quite complete :)


Added advanced pagination UI for media stores that return dozens, hundreds, or thousands of pages

Improves the media manager pagination UI, as it is currently totally unstyled, and for media stores that return dozens of pages, it looks terrible (and breaks).
In fact, listing pages doesn't work passed around 10 pages, and feels ineffective. Considering, the media manager has been updated to have two behaviours...
For media stores that return 10 pages or less, it renders this:

For media stores that render more than 10 pages, it renders this:

Which allows the user to quickly jump to a specific page as follows:

The behaviour is as follows:

The first button is disabled on the first page

It goes back to page 1

The prev button is disabled on the first page

It goes back 1 page

The next button is disabled on the last page

It goes forward 1 page

The last button is disabled on the last page

It goes to the last page

On change of the page dropdown, the media manager changes to that page

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

@chrisdmacrae rather than a tedious pattern to browse media (who uses this?), could we rely on search + progressive loading with interactions observer here?

@chrisdmacrae rather than a tedious pattern to browse media (who uses this?), could we rely on search + lazy loading with an interaction observer?

@DirtyF search is a premature abstraction. I plan on adding a custom search to my store, but this store makes a clear assumption: media will be stored in folders.
I am experiencing one of the few where that is not true. So the advanced pagination is more of an "escape hatch" of the folder-based UI.
In 80% of scenarios, I'm guessing any directory of media will have less than 10 pages. (That's 300 files with the default logic)

After talking with @chrisdmacrae I think we're instead going to make the relevant parts of this UI pluggable so this can be solved in userland (at least for now until we have time to work on an official solution)

After talking with @chrisdmacrae I think we're instead going to make the relevant parts of this UI pluggable so this can be solved in userland (at least for now until we have time to work on an official solution)

#1615

Closing this as it is handled by #1615


[core] Rename WindowShouldClose to IsWindowClosing

Following on from #1226, WindowShouldClose returns a bool and should be consistently renamed with other functions in the library that return a bool by prefixing it with Is like IsWindowClosing. Currently, the naming is confusing.
I see it is following the naming from a function in GLFW where it simply returns the close flag for the window.
I recognize this can break code, so this could by a TODO for a future major raylib release (5?).
If this isn't a good name for the function or I'm misunderstanding what the function does, I apologize.

@wmentha I agree that funtion is not consistent with the naming conventions but unfortunately at this point it is used in thousands of codebases, so, changing it will break lot of code. I'm afarid we should live with it...

I also think the proposed alternative is not quite apt.  IsWindowsClosing suggests agency outside of the program, rather than the program is being signalled to close the window (from perspective of the app).  I suppose the problem is that it could be a forced close (the big X) or a requested close (ESC).  But what is called for is the same in either case: stop drawing, establish a clean state, and exit quickly.


Error: 'ForeignKey' object has no attribute 'rel'

Getting 'ForeignKey' object has no attribute 'rel' on administration view. Model file attached
models.py.txt

I also had this error when visiting the admin page for one of my models.  It appears that field.rel was removed and so the call to field.rel.to.objects.count() in django_baker/admin.py fails.  See https://docs.djangoproject.com/en/1.9/releases/1.9/#field-rel-changes.
If you don't mind losing admin filters, one work around is to implement get_list_filter() in each of your admin classes:
class MyModelAdmin(ExtendedModelAdminMixin, admin.ModelAdmin):
    extra_list_display = []
    extra_list_filter = []
    .....
    formfield_overrides = {}
    readonly_fields = []

    def get_list_filter(self, request):
        return ()


Сallback function if all permissions are received for the first time

Is it possible to call the callback function if all permissions are received for the first time? To thank the user

That is a nice idea.


Added a code to change font size Fixes #2866

Fixes #2866
Describe the changes you have made in this PR -
I have added a button to let user change the font size of label
Screenshots of the changes (If any) -
https://user-images.githubusercontent.com/87171452/151769160-40c75fd0-73b5-41d5-b8b0-ed948f07a6c5.mp4
Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

@tachyons @gr455 please once review this pull request :)

Coverage remained the same at 81.774% when pulling 55e17d4f380145417a812c137f1a51f11c461e85 on Palaksharma23:flexiblefontsize into 29f98c87dd976f9106968ec4277131127f5b9fe0 on CircuitVerse:master.


Feature: Ability to automatically skip null checking for requests DTO in ASP.NET Core

First of all thank you very much for a great library!
Here is a case:

Create an ASP.NET Core project and add RuntimeNullables package
Create endpoint and request data class:

[Post]
public IActionResult Auth(AuthRequest request)
{
    //logic
}

public record AuthRequest(string Something);

Send a request to this gateway:

{ "Something": null }

Server returned 500 error (cause Something cannot be null), but had to return 400 (Bad Request). So, as you see this is kind of a bug actually.

Of course we can add [NullChecks(false)] attribute to the request class and the bug will disappear, but what if we forget to do that? Again 500? And it's actually not so convenient to add NullChecks to requests explicitly all times.
I suggest to add an ability to automatically skip null checking for requests DTO in ASP.NET Core. Let's discuss an implementation here.
PS. It's interesting that NullGuard.Fody just crashed my app at all when I sent an incorrect request like in the example, but RuntimeNullables only throws an exception and generates 500 (and also customizable). So, I fully migrated from NullGuard.Fody to RuntimeNullables. Thanks again for your work!

You don't have to add [NullChecks(false)] - just make the parameter nullable :)

I don't know how I feel about "implicitly" removing null checks from some classes because they are used as an action method parameter. That seems a little too magical and unexpected for me.
The more appropriate resolution here IMO is to fix ASP.NET's response code when an ArgumentNullException is thrown during model deserialization. Given the extensibility points ASP.NET provides I would be somewhat surprised if this wasn't easily achievable.

If you add [NullChecks(false)] then how are you validating the value in the model? You would either need to null check it manually, in which case you can just make the property nullable, or you need to put a [Required] annotation on it. I might be open to something involving removing null checks on properties that have [Required] applied to them, given that it signals you want to use validation to ensure the value is not null and not a normal null check.

That would fix the "bug" and also give you proper model validation, but it would also require remembering to explicitly annotate each non-null property with [Required] to get that behavior so less than ideal IMO.
Really I think the optimal soution is to add a RuntimeNullables.AspNet package that has a simple config extension method that sets up ASP.NET to properly handle ArgumentNullException during DTO deserialization.

FYI: I have an issue open on aspnetcore to address this issue: https://github.com/dotnet/aspnetcore/issues/54197


Discussion: Should parameter SQLServer be of type qualifier Key, Require or Write?

Hello all!
For a while I have questioned myself if the parameter SQLServer should be mandatory (Key or Required),
or that it instead should be a non-mandatory parameter (Write) with a default value of env:COMPUTERNAME.

The type qualifier, [Key], on a property indicates that this property will uniquely identify the resource instance. At least one [Key] property is required.
The [Required] qualifier indicates that the property is required (a value must be specified in any configuration script that uses this resource).
The [Write] qualifier indicates that this property is optional when using the custom resource in a configuration script.

For this discussion I will assume that "InstanceName" parameter is always Key.
Scenario 1
When SQL Server is installed on a single server the SQLServer parameter could be left out of the configuration so that the default value of env:COMPUTERNAME could be used. So for that scenario the parameter could have the type qualifier Write.
Scenario 2
But what if SQL Server is installed in cluster, either as a resource in a failover cluster or as AlwaysOn Availability Groups? Let's say we set up AlwaysOn with two Availability Groups, and we want to create one database in each Availability Group. This configuration will be run on the node acting as the primary replica, let's call it node 1. A second replica is already setup at this point, let's call it node 2.
Each availability group listener has a DNS-record pointing to its IP address; AG1 and AG2.
The databases was created on node 1 using the following configuration (just a mock example).
Configuration Example
{
    node node1
    {
        xSQLServerDatabase 'Database1-In-AG1'
        {
            Name = 'Database1'
            SQLInstanceName = 'TESTDSC'
        }

        xSQLServerDatabase 'Database2-In-AG2'
        {
            Name = 'Database2'
            SQLInstanceName = 'TESTDSC'
        }
    }
}

The above example would work because the parameter SQLServer is default set to env:COMPUTERNAME and both availability groups had node 1 as the primary replica.
But now let say that the first availability group AG1 is primary on node 1, and let's say that the second availability group AG2 is primary on node 2. In this scenario the above example would fail because node 1 is no longer primary replica for both availability groups.
To solve that we need to add the parameter SQLServer to the configuration, to tell which DNS-record each availability group listens on.
Configuration Example
{
    node node1
    {
        xSQLServerDatabase 'Database1-In-AG1'
        {
            Name = 'Database1'
            SQLServer = 'AG1'
            SQLInstanceName = 'TESTDSC'
        }

        xSQLServerDatabase 'Database2-In-AG2'
        {
            Name = 'Database2'
            SQLServer = 'AG2'
            SQLInstanceName = 'TESTDSC'
        }
    }
}

But the parameter SQLServer still does not need to have the type qualifier Required or Key. It is enough for it to have type qualifier Write with a default value of env:COMPUTERNAME.
I have not thought of any scenario yet that needs to have the parameter SQLServer as type qualifier Key.
Can anyone else see a scenario where it would be required?
Thanks,
Johan

After thinking about this extensively, I believe the "SQLServer" parameter can be dropped completely.
To connect to the SQL instance (clustered or standalone), the instance name can be looked up in the registry. Doing this will allow Connect-SQL to dynamically detect the hostname. It is impossible to have more than one instance per node with the same instance name (I tested to double-check).
When connecting to or managing a database that is a member of an Availability Group, it is important to remember that use of a listener is not required. In fact, there are many cases where AGs are used as a 1:1 replacement for database mirroring. Because of this, I do not believe that supplying the hostname for an AG listener makes sense.
However, it might be a good idea for the Database resources to determine if the database is a member of an AG and make a decision about what to do based on that information. Options could include:

Connect to the primary replica and get/set/test the settings
Silently stop processing the resource
Look for a parameter telling the resource what to do. Maybe ApplyOnlyWhenPrimary = $true will silently stop if the database is not currently on the primary replica and ApplyOnlyWhenPrimary = $false will always connect to the AG replica and update the database settings.

I agree completely that these parameters must be standardized across all resources. However, I disagree with @randomnote1 on the topic of removing the parameter pointing at the appropriate host name for the instance. I think automatically determining this information at runtime will prove to be a costly operation that will undoubtedly introduce performance and maintenance issues down the road.
I do like the idea of adding an additional common parameter to only test / set when on the primary node/replica of a cluster or AG. It is interesting when reviewing SQL Server logs and seeing two different nodes making max memory changes! We could implement that as a common function Test-ActiveNode or Get-ActiveNode that each resource could call upfront to determine whether to continue.

I need some parameter name help. I'm trying to determine a concise name that tells the end user that the resource will or will not process/apply when the current node is not actively hosting the SQL Instance. Here are some of my ideas so far:

ProcessWhenNotActiveNode
ProcessOnPassiveNode - I don't like this one because I believe it makes it sound like it is offloading processing to a secondary node.
ProcessOnlyOnActiveNode - This would keep the current behavior of applying the resource no matter what, but would override the behavior when true.

I'm leaning towards using "Process" because it think it describes the Get/Set/Test instructions where I think "Apply" indicates the "Set" instructions.

Since which node is active / primary vs. passive / secondary is dependent on the context of a specific availability group (possibly determined indirectly by a DatabaseName property), I'm leaning toward the terminology that SQL Server itself uses, namely "availability replica", "primary replica", "secondary replica" (Always On Availability Groups). I agree with @randomnote1 that something like ProcessOnPrimaryReplica makes it sound like the processing with occur on the primary node when the configuration is being applied to the secondary, which is undesirable. The requirement could be accomplished with a single property maybe:
WhenReplicaIs = 'any'
WhenReplicaIs = 'primary'
WhenReplicaIs = 'secondary'
WhenReplicaIs = 'primary' might be a better default than the current behavior (WhenReplicaIs = 'any'), since many alter-type functions fail when attempted on a secondary replica. This could actually be combined with the ProcessOn properties as in:
WhenReplicaIs = 'secondary'
ProcessOnPrimaryReplica = $true
or for things like copy-only backups, which can be done on a secondary:
WhenReplicaIs = 'primary'
ProcessOnSecondaryReplica = $true


Update README.md

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

AWS CodeBuild CI Report

CodeBuild project: codebuildgithubautobuildPro-ZQrYvg1Wv7BG
Commit ID: 9092cb8348a87a7b06125f864c8a8780fe5eeba8
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: codebuildgithubautobuildPro-ZQrYvg1Wv7BG
Commit ID: c98448581aa33ddfddf412fd8dcd38a4a894be72
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Train time?

Curious what sort of train times you're seeing with this implementation.
I'm using a GRID K520 GPU (Amazon g2.2xlarge) -- i'm seeing each Epoch take around 1200 seconds, which seems wrong.
From the original paper:
"Data requirements and speed We note that decent results
can often be obtained even on small datasets. Our facade
training set consists of just 400 images (see results in
Figure 12), and the day to night training set consists of only
91 unique webcams (see results in Figure 13). On datasets
of this size, training can be very fast: for example, the results
shown in Figure 12 took less than two hours of training
on a single Pascal Titan X GPU."
Granted I'm not using a Pascal GPU -- which as 2496 CUDA cores, but the g2.2xlarge has around 1500 CUDA cores.  At the current rate 200 epochs would take 3 days, as opposed to the 2 hours quoted in the original paper.
Are you seeing similar train times when running this code?  Wondering why there is such a discrepancy compared to the original paper/Torch implementation

I am investigating this issue.
It took me around 10 hours to run 200 epochs on a Pascal GPU.
One of the crucial bottlenecks is that in this implementation, generator needs to update twice in each iteration, which slows down the training process a lot.

i am seeing 700 sec per epoch on GTX1080

My test with GRID K520 GPU (Amazon g2.2xlarge) using my own dataset shows that pix2pix/Torch runs about 30 times faster than pix2pix/Tensorflow version. Monitoring using 'watch nvidia-smi' shows that the Tensorflow version is not using the GPU at all.

@kaihuchen sorry for the obvious question, but did you install "tensorflow-gpu"?

@kaihuchen I'm sure that I'm training this code with GPU. Can you tell me how you installed tensorflow?
sidenote: 看來您是畢業自台灣清華大學的學長 😄
@eyaler I've updated the codebase alot recently (which gain speed comparable to torch version, will upload later)

@yenchenlin  My bad!  I have many servers and it would seem that I did the test on a server with the CPU version of the tensorflow, and not the GPU one.

@eyaler I also had a tensorflow implementation here. It takes me 43 seconds every epoch (400 iterations of batch=1 on facades dataset) on GTX1080, while the torch version takes 42 seconds.

Thanks @ppwwyyxx for the info!
@eyaler I think currently the code mentioned above works better!
However, I'll still update code here in these 3 days.

@yenchenlin  Any update on this? I do not see any recent commits pertaining to speed. Otherwise, I am tempted be forced to use the code provided by @eyaler.

The code looks clean and straight forward... I really can't get my head around the reason why it's slow... It's pretty much a standard GAN so why is it so slow?! answer to this question has become one of the reasons I check this thread every now and then...

I have one idea.
Feed_dicts are incredibly awefully slows. We should do what Tensorpack does and load say 50 images at a time, keep them in a queue of numpy arrays and then feed them in with a queue runner. This alone might be responsible for the speed difference since it doubles the number of copies needed and causes a lot of expensive switching between Python and Tensorflow C code.

@Skylion007
hmmm... How about the fact that this network is using a fully connected layer in the discriminator... last I checked Tensorpack uses a 1x1 Convolution in the last layer (instead of a fully connected layers)... couldn't it be because of this?

That's another issue, there was a pull request to address this, but it was rejected because it made the edges sightly more blurry. I'm open to try to that and see if it improves the speed. You want to try experimenting with that pull request and see if it yields any results?

My GPU is currently in use by another experiment.
That's exactly my case too! I've been running one for 3 days and last night it started showing acceptable improvements, I really don't want to stop it for at least 3 more days...

The graphs for each network look very different as well. @ppwwyyxx implementation's graph looks like this for instance while the network in this repo seems to have alot of dependencies so much so that the graph looks more like a straight line than tree. A very different appearance from the one below:

Not entirely sure how much of that is due to good Tensorboard formatting and how much of that is a fundamental differences in the architecture between the networks.

@Skylion007 Tensorboard tend to organize ops under the same name scope together, so what you see in the above figure isn't the real architecture but more about summaries and utilities. You can open the "gen" and "discrim" block in the above figure, and they will contain the model architecture for generator and discriminator.

Yeah, I see that now. I am just so confused why the other code is so much faster. I just discovered Tensorboard so I was trying to see what I could gain from it. I will say that the GPU memory use is much higher in this implementation. I am really curious why that would be the case. That could explain why it's slower maybe. Any ideas @ppwwyyxx ? Any special tricks your code is doing?

@Skylion007 it's probably the Fully Connected Layer... Those things takes a lot of memory...

changing the last layer from fully connected to a convolution as in the original pix2pix implementation did not give me any speedup

Thanks @eyaler , it's 2. and 3 IMO.
I'm really sorry that I'm dealing with some other annoying stuffs recently 😭

@eyaler This was an eye opener... I had so many misconceptions about performance in this project! Thanks for the time... please keep going on!

can anyone tell me why we do this:
        self.fake_AB = tf.concat(3, [self.real_A, self.fake_B])
        self.D_, self.D_logits_ = self.discriminator(self.fake_AB, reuse=True)

why do we concat real_A and fake_B and give them BOTH to the discriminator while what we want is to just give them one image (the generated fake: self.fake_B) ?
doesn't this force the discriminator to accepts dual images (one half the real image and the other half the generated one) and double the time needed to process them?

Hello @Neltherion, please see the image from paper:

Hmm... You're right... and just giving the fake images to Discriminator is probably not enough... my bad! thanks for the quick reply...

Normally, conditional GAN will send the conditional data (e.g., class, attribute, text, image)  together with the synthesized image to the discriminator. See this paper for a more complicated discriminator.

some benchmarks for the community:
images/sec:
5.2 phillipi K80/torch/cuda8
1.1 yenchenlib K80/tf0.12.1/cuda7.5
1.2 yenchenlib K80/tf0.12.1/cuda8
1.2 yenchenlib K80/tf1.0/cuda8
2.2 yenchenlib 1080/tf0.12.0/cuda8
2.3 yenchenlib_mod K80/tf0.12.1/cuda7.5
2.5 yenchenlib_mod K80/tf0.12.1/cuda8
2.5 yenchenlib_mod K80tf/1.0/cuda8
4.7 yenchenlib_mod 1080/tf0.12.0/cuda8
4.7 affinelayer K80/tf1.0/cuda8
5.5 tensorpack K80/tf1.0/cuda8
all experiments are on the facades dataset and use cudnn 5.11
phillipi = https://github.com/phillipi/pix2pix
yenchenlin = https://github.com/yenchenlin/pix2pix-tensorflow
yenchenlin_mod = https://github.com/yenchenlin/pix2pix-tensorflow/issues/4#issuecomment-273221087
tensorpack = https://github.com/ppwwyyxx/tensorpack
affinelayer = https://github.com/affinelayer/pix2pix-tensorflow


--disable_metrics has no effect

DebugConf maps the option value instead of actually checking if it was disabled.
Appears to have been the case since April 2015.
    val metricsBehavior = conf.metrics.get.map(_ => new MetricsBehavior(metricsProvider))

should be something like
    val metricsBehavior = if (conf.metrics.get.getOrElse(true)) Some(new MetricsBehavior) else None

Patch is here: https://phabricator.mesosphere.com/D451


live incoming Signal plot with DateTime axis demo

JR emailed code that doesn't work as expected. Fixing this code, or creating a demo which makes this easier to figure out, may be useful.
public double[] data = new double[10000];
public double[] timedata = new double[10000];

void Init() {
signalPlot = wpfPlot1.plt.PlotScatter(data, timedata);
             wpfPlot1.plt.YLabel("Value");
             wpfPlot1.plt.XLabel("Sample Number");
             wpfPlot1.plt.Ticks(dateTimeX: true, dateTimeFormatStringX: "HH:mm:ss");
}

void UpdateData() {
            double randomValue = Math.Round(rand.NextDouble() - .5, 3);
            double latestValue = data[nextDataIndex - 1] + randomValue;
            data[nextDataIndex] = latestValue;
            timedata[nextDataIndex] = DateTime.Now.ToOADate();
            nextDataIndex += 1;
}

Upon closer inspection, it seems the problem is that JR modified the demo to replace a signal plot with a scatter plot:
// original code
signalPlot = wpfPlot1.plt.PlotSignal(data);

// JR's code
signalPlot = wpfPlot1.plt.PlotScatter(timedata, data);

Scatter plots display ever data point in the arrays they are given, and in JR's case most of those values are 0 (since they were never assigned to). This resulted in a plot that had numerous data points at 0.

To modify the demo to display DateTime units on the horizontal axis, set the appropriate sample rate (time span between data points) and xOffset (when the recording starts) and also indicate that ticks are to be displayed as dates.
// original code
signalPlot = wpfPlot1.plt.PlotSignal(data);

// replacement to display horizontal axis as DateTime
double sampleRate = 1.0 / TimeSpan.FromMilliseconds(1).TotalDays;
signalPlot = wpfPlot1.plt.PlotSignal(data, sampleRate, DateTime.Now.ToOADate());

The sample rate seems bizarre, but that's because OADate expresses dates as a number where every day represents 1.0. Therefore the sample period (1 millisecond) is an extremely small number, and the sample rate (inverse of sample period) is an extremely large number. The numbers look weird, but they work out. Dates are confusing in .NET


No module named psycopg2

I got an error when trying to run the first cell in the tutorial
The autoreload extension is already loaded. To reload it, use:
%reload_ext autoreload
ImportError                               Traceback (most recent call last)
 in ()
8 os.environ['SNORKELDB'] = 'postgres:///snorkel-intro'
9
---> 10 from snorkel import SnorkelSession
11 session = SnorkelSession()
12
/Users/arjasethan/OpenSource/snorkel/snorkel/init.py in ()
----> 1 from .models import SnorkelSession
2
3 version = 0.5
/Users/arjasethan/OpenSource/snorkel/snorkel/models/init.py in ()
36     import snorkel.models
37 """
---> 38 from .meta import SnorkelBase, SnorkelSession, snorkel_engine, snorkel_postgres
39 from .context import Context, Document, Sentence, TemporarySpan, Span
40 from .context import construct_stable_id, split_stable_id
/Users/arjasethan/OpenSource/snorkel/snorkel/models/meta.py in ()
41 # We initialize the engine within the models module because models' schema can depend on
42 # which data types are supported by the engine
---> 43 SnorkelSession = new_sessionmaker()
44 snorkel_engine = SnorkelSession.kw['bind']
45
/Users/arjasethan/OpenSource/snorkel/snorkel/models/meta.py in new_sessionmaker()
30     # open, multiple processes, etc.
31     if snorkel_postgres:
---> 32         snorkel_engine = create_engine(snorkel_conn_string, isolation_level="AUTOCOMMIT")
33     else:
34         snorkel_engine = create_engine(snorkel_conn_string)
/Users/arjasethan/OpenSource/anaaconda2/lib/python2.7/site-packages/sqlalchemy/engine/init.pyc in create_engine(*args, **kwargs)
385     strategy = kwargs.pop('strategy', default_strategy)
386     strategy = strategies.strategies[strategy]
--> 387     return strategy.create(*args, **kwargs)
388
389
/Users/arjasethan/OpenSource/anaaconda2/lib/python2.7/site-packages/sqlalchemy/engine/strategies.pyc in create(self, name_or_url, **kwargs)
78                 if k in kwargs:
79                     dbapi_args[k] = pop_kwarg(k)
---> 80             dbapi = dialect_cls.dbapi(**dbapi_args)
81
82         dialect_args['dbapi'] = dbapi
/Users/arjasethan/OpenSource/anaaconda2/lib/python2.7/site-packages/sqlalchemy/dialects/postgresql/psycopg2.pyc in dbapi(cls)
552     @classmethod
553     def dbapi(cls):
--> 554         import psycopg2
555         return psycopg2
556
ImportError: No module named psycopg2

Hi @arjasethan1, it looks like you don't have psycopg2 installed. If you're using pip, you can do
pip install psycopg2

from your shell command line.


Update Core_kernel to Core

Core_kernel is deprecated since Core and Core_kernel have been merged. This PR fixes the open statements to reflect this.

@fortunac Does make test pass for you for wp?


Can't split items into smaller stacks unless bags are shown.

Addon Version
0.3.18
World of Warcraft Version
Classic Era (And SoD/Hardcore)
Describe the bug
I have a stack of 20 mageweave. I want two stacks of 10. I can't get it to work, unless I enable "show bags" and drop the desired amount selected into a specific bag.
Steps to reproduce
There is no step to reproduce, it just doesn't work.

I does work if you dont have merge stacks on. I think the merge stacks just regroups them after you split them.

I does work if you dont have merge stacks on. I think the merge stacks just regroups them after you split them.

No it does not. I disabled merge stacks and still can't split items.

Hmm, very odd. It works for me.


Replace SimpleSegmentRef with FileSegmentRef as default

To avoid out of memory conditions when running larger executions

Additional suggestion: Having a optional NopSegmentRef that just stores the index number but de-allocs the segment is really useful for local perf testing. It would be a nice optional segment-ref to have in zkvm as a option.


scala 2.11 not recommended

scala 2.11 not recommended

In the case of scala 2.11, FatalError may occur in macro expansion when scanning some external libraries.
[error] <macro>:1:1: Symbol 'term org.junit' is missing from the classpath.
[error] This symbol is required by 'class org.scalatest.junit.MyRunListener'.
[error] Make sure that term junit is in your classpath and check for conflicting dependencies with `-Ylog-classpath`.
[error] A full rebuild may help if 'MyRunListener.class' was compiled against an incompatible version of org.
[error] org.scalatest.junit.MyRunListener
[error] ^
[error] <macro>:1:1: Symbol 'term <none>.junit.runner' is missing from the classpath.
[error] This symbol is required by 'class org.scalatest.junit.MyRunListener'.
[error] Make sure that term runner is in your classpath and check for conflicting dependencies with `-Ylog-classpath`.
[error] A full rebuild may help if 'MyRunListener.class' was compiled against an incompatible version of <none>.junit.
[error] org.scalatest.junit.MyRunListener
[error] ^
[error] <macro>:1:1: Symbol 'term <root>.junit' is missing from the classpath.
[error] This symbol is required by 'class org.scalatest.junit.JUnitTestFailedError'.
[error] Make sure that term junit is in your classpath and check for conflicting dependencies with `-Ylog-classpath`.
[error] A full rebuild may help if 'JUnitTestFailedError.class' was compiled against an incompatible version of <root>.
[error] org.scalatest.junit.JUnitTestFailedError
[error] ^
[error] <macro>:1:1: Symbol 'term <root>.junit' is missing from the classpath.
[error] This symbol is required by 'class org.scalatest.junit.MyTestListener'.
[error] Make sure that term junit is in your classpath and check for conflicting dependencies with `-Ylog-classpath`.
[error] A full rebuild may help if 'MyTestListener.class' was compiled against an incompatible version of <root>.
[error] org.scalatest.junit.MyTestListener
[error] ^
[error] <macro>:1:1: Symbol 'term org.junit' is missing from the classpath.
[error] This symbol is required by 'class org.scalatest.junit.JUnitRunner'.
[error] Make sure that term junit is in your classpath and check for conflicting dependencies with `-Ylog-classpath`.
[error] A full rebuild may help if 'JUnitRunner.class' was compiled against an incompatible version of org.
[error] org.scalatest.junit.JUnitRunner
[error] ^
[error] <macro>:1:1: Symbol 'term <root>.junit' is missing from the classpath.
[error] This symbol is required by 'class org.scalatest.junit.JUnit3Suite'.
[error] Make sure that term junit is in your classpath and check for conflicting dependencies with `-Ylog-classpath`.
[error] A full rebuild may help if 'JUnit3Suite.class' was compiled against an incompatible version of <root>.
[error] org.scalatest.junit.JUnit3Suite
[error] ^
[error] <macro>:1:1: Symbol 'term org.apache' is missing from the classpath.
[error] This symbol is required by 'class org.scalatest.tools.ScalaTestAntTask'.
[error] Make sure that term apache is in your classpath and check for conflicting dependencies with `-Ylog-classpath`.
[error] A full rebuild may help if 'ScalaTestAntTask.class' was compiled against an incompatible version of org.
[error] org.scalatest.tools.ScalaTestAntTask
[error] ^
[error] <macro>:1:1: Symbol 'term <none>.apache.tools' is missing from the classpath.
[error] This symbol is required by 'class org.scalatest.tools.ScalaTestAntTask'.
[error] Make sure that term tools is in your classpath and check for conflicting dependencies with `-Ylog-classpath`.
[error] A full rebuild may help if 'ScalaTestAntTask.class' was compiled against an incompatible version of <none>.apache.
[error] org.scalatest.tools.ScalaTestAntTask
[error] ^

Currently in Scaladia, it happens with org.scalatest, org.scalatestplus and akka-stream.
As it may occur in other external libraries, you need to set the following in reference.conf when it occurs.
di.unscanning.package: ["com.failed-package-name"]

Stopped support.


Changes GPU wire_memsync usermessage to net

Usermessages have a limit of 256 bytes, this changes the wire_memsync usermessage to use the net library instead, allowing you to stream a larger amount of data to the GPU safely.

yeah I mentioned in discord that it might need to have some limits added to it. dunno if I will have the time to look at it for a bit tho

Definitely not safe with or without the changes. The CacheMGR and everything that uses it's 'Flush' method need an overhaul to make it safe.


can we support etcd V2?

when i import etcd v2 into kstone, then status was Unknown.
etcd version
etcd Version: 2.3.4
Git SHA: df60227
Go Version: go1.6.2
Go OS/Arch: linux/amd64
and  etcd 2379 port use https,  2380 use http
--initial-advertise-peer-urls "http://xx.xx.xx.xx:2380"
--listen-client-urls 'https://<IP_ADDRESS>:2379'
the kstone logs is
E1227 02:07:06.068092       1 helper.go:70] get new clientv3 cfg failed:tls: failed to find any PEM data in certificate input
E1227 02:07:06.068109       1 helper.go:95] failed to get new etcd clientv3,err is tls: failed to find any PEM data in certificate input
I1227 02:07:06.068244       1 event.go:291] "Event occurred" object="kstone/xxx-etcd" kind="EtcdCluster" apiVersion="kstone.tkestack.io/v1alpha1" type="Warning" reason="EtcdClusterUpdateStatus" message="failed to get cluster status tls: failed to find any PEM data in certificate input"
W1227 02:07:06.077150       1 etcdclusters_controller.go:389] cluster xxx-etcd is not ready

v2 might be deprecated after 3.5. And v2 is default disabled since 3.4.


indicator doesn't shows up

var myApp = new Framework7({
// Hide and show indicator during ajax requests
onAjaxStart: function (xhr) {
myApp.showIndicator();
},
onAjaxComplete: function (xhr) {
myApp.hideIndicator();
}
});
The indicator stills not shows up while post request but works on get request.
$$('.confirm').on('click', function (e) {
e.preventDefault();
// post request here
myApp.alert("complete！");
mainView.back();
});

These two callbacks onAjaxStart/End are only for internal XHR requests, you may just use this, like in kitchen sink:
$$(document).on('ajaxStart', function () {
    myApp.showIndicator();
});
$$(document).on('ajaxComplete', function () {
    myApp.hideIndicator();
});

I tried it, just not working.....

Look for problem on your side or provide a link so i can take a look. It works for everyone

Sorry I cant, my code runs on android. In webview it looks fine!

Then i can't say much. Try to look for problem, as it does work in kitchen sink and for everyone else

Closing for now, if you still have it issue, provide a link or more code examples