Android - Profile - The new avatar's photo is not displayed immediately

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open the App
Login with any account
Go to Settings ->Profile
Click on avatar -> Upload photo -> Choose a new photo -> Save

Expected Result:
The new avatar's photo should be displayed immediately
Actual Result:
The new avatar's photo is not displayed immediately
Workaround:
Unknown
Platform:

Where is this issue occurring?

Android

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/202584190-b21771c2-cd3e-4ca5-ba45-2603ef901865.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

Checkin in Slack if we're linking this to the overall image performance project

https://stackoverflow.com/c/expensify/questions/14418

[x] The "bug" is actually a bug - yep!
[x] The bug is not a duplicate report of an existing GH (close the GH and add any novel details to the original GH instead) - I don't think this is a duplicate, but it could be part of https://github.com/Expensify/App/issues/10894
[X] The bug is reproducible, following the reproduction steps: I could not reproduce this. I tried in web, mobile web, and mobile app on iOS (all on Production).
[X] If you’re unable to reproduce the bug:
[X] Add the Needs reproduction label.
[X] Comment on the issue outlining the steps you took to try to reproduce the bug, your results and tag the issue reporter and the Applause QA member who created the issue. Ask them to clarify reproduction steps and/or to check the reproduction steps again. Close issue.

@kbecciv I wasn't able to reproduce this! Are there any mores steps or conditions that you could add? If so, please reopen with that!


Standard module per Terraform module practices.

This PR updates this repo from following Gruntwork preferred practices to following the new Terraform module standard.
'cc @brikis98 @mcalhoun

Closing this PR as it is stale


[jemalloc|vcpkg_configure_make] update and some script fixes

lets rebuild the world ;)

Current CI baseline:
jemalloc:arm64-windows=fail
jemalloc:arm-uwp=fail
jemalloc:x64-linux=fail
jemalloc:x64-osx=fail
jemalloc:x64-uwp=fail
jemalloc:x64-windows-static=fail

@BillyONeal: rsocket ICEd here. Please check the baseline.

Will rerun x64-windows-static tests after other tests finished.

Ping @vicroms for review again.

Are there any questions open regarding the vcpkg_configure_make parts or only the jemalloc parts?
The vcpkg_configure_make related changes are crucial for cross compilation and if these require less discussion they could be split into a separate PR to speed things up.

Thanks!


Git setup

We need git setup with post-receive hook.
For development, it will be just a local directory, for prod/staging, it will be git over https.
Git post-receive hook
The post-receive hook should call an /api/webhooks/post-receive endpoint.
POST /api/webhooks/post-receive
Authorization: Bearer {config.git_post_receive_token}

{
    "repo_path": "/home/capstone/git/abcd1234/rajdhani.git",
    "git_commit_hash: "..."
}

Prod Setup
The git-http-server should be run as a fasctcgi process as capstone user.
Try it and document the setup instructions.

With Gito, we won't be needing to initialise git on the starter code and saving it as repo-git.zip (and in fact causing a bug by setting up git twice).
I'm removing the logic to do that. We can directly use repo.zip now instead of repo-git.zip.

The post-receive hook will also need the username and project name.
Would we store that metadata with the repository, or send the gito repo id and fetch username/project from the db?


Update asmtools location

Signed-off-by: Sophia Guo<EMAIL_ADDRESS>

https://ci.adoptopenjdk.net/view/work-in-progress/job/grinder_sandbox/389/console

https://ci.eclipse.org/openj9/job/test.getDependency/12706/


Recipes with the Same Ingredients

As a smart shopper I want to know recipes with the same ingredients so that I don't buy the same ingredient multiple times.
Confirmation:

Enter the recipes to compare
Run the function
Display the list of recipes with the same ingredients

Closed as of d3fade7d85c4867566a47403b32a4a4b0bca62c6.


Emit progress state escape codes

Emit progress state escape codes when drawing progress bar.
Fixes #11432

In addition, do you have the official documentation at hand about this escape code? CodeEmu doc doesn't seem to be official I feel like.

As per microsoft/terminal#8055, the ConEmu style is the implementation reference.

Here is a screen recording of running cargo check with b60dc0d:
https://user-images.githubusercontent.com/74931857/208923478-7ed6192c-cf80-4fdd-b3e7-919110d7029e.mp4
Notice the bar in the taskbar, and the circle getting bigger / fuller at the top left corner.

Thanks for the screen recording!
Unfortunately on macOS it doesn't seem to compile: https://github.com/rust-lang/cargo/actions/runs/3749845063/jobs/6368801382
I am still uncertain the adoption rate of this non-standard escape code. Does the majority of terminal emulators handle or ignore this well? Sorry being so conservative but I've seen some issue around “escape code not recognized” sometimes. That's also why I suggested adding #[cfg(windows)] at the very first, and even on Windows there are a bunch of different emulators. I'd recommend collecting the usage and support data before rolling out this feature :)

Current testing progress:

Terminal
Support

Windows Terminal
:white_check_mark:

Git MinTTY
:white_check_mark:

Alacritty
:x: (Feature seemingly refused)

https://github.com/rust-lang/cargo/pull/11436#issuecomment-1361804049 was a good try. Thank you. I'll probably split Support to two columns:

Support the display of the progress bar or not.
If not supported, would it break things or just ignore the unrecognized escape code?

Marked this as S-blocked as it need more data to ensure it won't break the majority from different terminal on Windows.


Convert key features to jsoncardlist

Currently the key features is an image, this should be converted to json card list with additional information

Fixed in #66


Ensemble nesting suite needs pytz on accessdev

resolution_fixed   |    by<EMAIL_ADDRESS>
The ensemble nesting suite, e.g. u-ap507, has added a function Jinja2Filters/get_members.py which requires the python pytz module on accessdev.

Issue migrated from trac:333 at 2024-01-31 18:31:01 +1100

<EMAIL_ADDRESS>changed status from assigned to closed

<EMAIL_ADDRESS>changed status from new to assigned

<EMAIL_ADDRESS>set owner to mrd599

<EMAIL_ADDRESS>commented

Shouldn't be too hard to add this to the puppet configuration.

<EMAIL_ADDRESS>set resolution to fixed


Sorting by filename

Not sure if you agree with all the changes, but I'll try to explane them.
The original idea was to add standard sorting by filename to the table. I found it might be quite convenient to work file by file.
There were already several variants of data source in the ViewController (dataSource, filteredData, perFunctionTimes, perFileTimes), so adding one more wasn't an option. So I decided to incapsulate all the processing over the data array to a separate class (ViewControllerDataSource), store only one original array of items and apply all the aggregating, filtering and sorting dynamically if necessary. In my opinion it makes code and the logic cleaner and easily lets extend processing functionality (other operations under the data array if necessary).
Unfortunately I had to convert CompileMeasure to an objC class for being able to easily apply an array of sort descriptors. I couldn't find easy and elegant way to apply several sorting functions to swift array without a lot of pain (didn't want to implement something like http://chris.eidhof.nl/post/sort-descriptors-in-swift/). I thought that inheriting the model from NSObject and double casting the array (to NSArray and back to swift array) is not such a big deal considering the ability to use good old sort descriptors (which are built in to NSTableView)
All the basic functionality of ViewControllerDataSource is covered with tests.

Nice work! I like it. We can live with the objc bits so let's bring this in.


Underscore property prefixing is performed before template string expansion

Underscore property prefixing is performed before expanding template strings. This leads to incorrect results when combining the two.
this._dockerPort = (Math.random() * (0xFFFF - 1024) | 0) + 1024
args.push('-net', `user,hostfwd=tcp:<IP_ADDRESS>:${this._dockerPort}-:2375`)

becomes
this.$Host_dockerPort = (Math.random() * (0xFFFF - 1024) | 0) + 1024
args.push('-net', ("user,hostfwd=tcp:<IP_ADDRESS>:" + this._dockerPort + "-:2375"))

(moved from facebook/react#3172)

Thank you for reporting this issue and appreciate your patience. We've  notified the core team for an update on this issue. We're looking for a response within the next 30 days or the issue may be closed.


It's about time

When will bro @Bebebole finish his cooking, he's keeping it like it's some sango_project

He cooked hopefully


Use gender-neutral language

You have to modify the footer_content.html file for us to merge this. Thanks!

Thanks!


Update README, closes #9

Please check out the README.md and let me know what to fix and what to add or delete.
Also what Licence are we using?
@khammami @ChinonsoIg @SofiaBee-W


[BUG] Colorspace not checked on build

Note: for support questions, please use one of the following channels.
Reddit: /r/gbstudio
Discord: Join Chat
This repository's issues are reserved for feature requests and bug reports.

I'm submitting a ...

[x] bug report
[ ] feature request
[ ] support request => Please do not submit support request here, see note at the top of this template.

Do you want to request a feature or report a bug?
Report a Bug

What is the current behavior?
PNGs with odd, unmatching colors will be converted to the right palette for display. This is fine. However, having many different tiles, complex designs or many colors are turned into garbage tiles.

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem

What is the expected behavior?

Rejection at build time or successful down-conversion.

What is the motivation / use case for changing the behavior?
Foolproofing, error catching, and such. I suppose it could also have some use in an edge case, but I seriously doubt it'd be useful.

Please tell us about your environment:

Version: 1.0.0
Platform: [Windows]

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)
Catch PNGs with too many colors at build time and error out on them.

This could also be relatert to the max unique tile limit. Your image has a lot of entropy

The latest builds handle converting the colours to the nearest supported one a lot better but the main issue here is the amount of detail in the image as Thomas says, each image can only have 192 unique 8px tiles as stated in the docs,
If you looked at the build console you'd have also seen this message

perhaps we should just make this a fatal error that stops compilation?

perhaps we should just make this a fatal error that stops compilation?
That's extreme, maybe switch to the log view when you get a warning?
I've successfully stored pictures with more than 192 tiles, and reduced it to only corrupt the text box, Which lets me use them for full screen graphics slides, forcing a fatal error would loose this.

It's absolutely possible to get the Game Boy to display images with more than 256 tiles (this uses sprites but the BG is more than 256):

Thus making this a fatal error is something I'd support if and only if it can be disabled (CLI switch?)

Yeah it's possible but the engine reserves a certain amount of memory for the window buffer and sprites in a scene. 192 is the amount available guaranteed to be available for background images in the engine.

I meant that if people want to hack the engine for this purpose, they should be able to. (Not that I'd suggest them to keep using the engine, at this point, but someone's bound to try eventually :P)

Commit ad7fe5719ed729ee6db15ae3019a0113e30b9701 now automatically switches to the build tab if there are any warnings/errors during build. It also displays all the warnings as the end of the build so they should be more obvious.

yay, one of my "i deliberately broke it, plz fix" issues actually helped
On Tuesday, April 30, 2019, Chris Maltby<EMAIL_ADDRESS>wrote:

Closed #93 https://github.com/chrismaltby/gb-studio/issues/93.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/chrismaltby/gb-studio/issues/93#event-2310801650, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHE6CQJLXF2DIV6WBIRILYDPTC3KLANCNFSM4HJICNOA
.


Add Node classes for assignment types

Adds nodes with a mostly unified interface for lvasgn, ivasgn, cvasgn, gvasgn, casgn, op-asgn, and-asgn and or-asgn nodes.
The idea here is that we have a lot of repeated boilerplate in some cops to handle assignment nodes that are mostly but not completely the same. This will allow all assignment nodes to respond to name and expression regardless of the node type.
I didn't create a node class for masgn because that works differently than everything else and I'm not exactly sure what we'd want there as of yet.

Once this is available I have some cops I'd like to update to make more simple. 😄

Great work 🎉
PR is great as is, but could you also update docs/modules/ROOT/pages/node_types.adoc please?

@marcandre absolutely! I actually had that file open to use as a reference but forgot to update the nodes 😅 Fixed now!

Released 1.10.1. Thanks!


Implementation of pygatt backend

As requested in https://github.com/open-homeautomation/miflora/issues/50, here is a first implementation of a Backend for Blue Giga Bluetooth adapters based on the pygatt library.

Coverage decreased (-9.2%) to 76.38% when pulling 3e303d0a92b039a6cbb31eda53dbc72f9c2f25d3 on ChristianKuehnel:pygatt into 0af1185abfbfe35ed2202b0ebab9e94a646efe4a on open-homeautomation:master.

Coverage decreased (-9.2%) to 76.38% when pulling 9b2d57fdaa6c6be83ad1781db02547f9f2fda9ed on ChristianKuehnel:pygatt into 0af1185abfbfe35ed2202b0ebab9e94a646efe4a on open-homeautomation:master.

Coverage decreased (-9.2%) to 76.38% when pulling 9b2d57fdaa6c6be83ad1781db02547f9f2fda9ed on ChristianKuehnel:pygatt into 0af1185abfbfe35ed2202b0ebab9e94a646efe4a on open-homeautomation:master.

Coverage decreased (-9.2%) to 76.38% when pulling 9b2d57fdaa6c6be83ad1781db02547f9f2fda9ed on ChristianKuehnel:pygatt into 0af1185abfbfe35ed2202b0ebab9e94a646efe4a on open-homeautomation:master.

Coverage decreased (-9.2%) to 76.38% when pulling 9b2d57fdaa6c6be83ad1781db02547f9f2fda9ed on ChristianKuehnel:pygatt into 0af1185abfbfe35ed2202b0ebab9e94a646efe4a on open-homeautomation:master.

Coverage increased (+6.5%) to 92.032% when pulling 6fd36b93cb10ca0024b10925838dcf7a6defe6bc on ChristianKuehnel:pygatt into 0af1185abfbfe35ed2202b0ebab9e94a646efe4a on open-homeautomation:master.


Replace features with cfg and autodetection

Thanks ! I wait for checks before merging.

Yeah there's an ICE, I'll need to update the Rust version in appveyor first.

I have an ICE on gtk. It must be the bad week. I'll wait for your update then. ^^

And the problem is also on travis. Congrats ! haha

Apparently rustc doesn't like bare --cfg and cargo doesn't bother to check.

You're using a deprecated item in build.rs.

join hasn't made it into stable yet so...

erf... So annoying. Let's merge it for now. Thanks a lot for your work !


⚠ Bofrost.lu has degraded performance

In f427b21, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1087 ms

Resolved: Bofrost.lu performance has improved in a1f8602 after 6 minutes.


add macvlan overlay e2e

Host should be access to nodePort address, including ipv4 and ipv6  case 失败了，本地再看看？@Roronoa-Z3r0


chore: update api-hooks

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4

Relevant Lines:
4

💛  - Coveralls


Sequence - what to do with sequence if a prescription contains repeated actions.

Benjamien thinks about it.

solved in #167 (RequestGroup)


Nested Map causes non-default updates to fail with Jason encoder error

Describe the bug
If you have a map that has array's of maps e.g.
    summary: %{
      chat_history: [
        %{"content" => "Default system prompt", "role" => "system"},
        %{
          "content" =>
            "Here's a collection of tweets from the Twitter list 'Web 3 Mid':\nTweet by KhanAbbas201 (2024-08-26 19:40:13.000000Z):\n@Rahatcodes told me I look good wearing the Eth shirt. https://t.co/xBugAt2tDi\nTweet by Rahatcodes (2024-08-26 19:42:55.000000Z):\n@KhanAbbas201 I dont recall saying this\nTweet by KhanAbbas201 (2024-08-26 19:44:08.000000Z):\n@Rahatcodes Damn what happened to your memory bruv?\nTweet by angelinarusse (2024-08-26 19:56:05.000000Z):\n@dabit3 Real degens call it Twitter\nTweet by angelinarusse (2024-08-26 20:13:58.000000Z):\n@hamseth They tried the same in Afghanistan and it didn’t go well for them.\nTweet by KhanAbbas201 (2024-08-26 20:39:53.000000Z):\nTweet by Osh_mahajan (2024-08-26 21:34:08.000000Z):\n@FedericoNoemie 🐾🐾🦘\nTweet by developer_dao (2024-08-26 21:39:59.000000Z):\n@ZwigoZwitscher @ArweaveEco @k4yls Wildly high praise, ty ty. @k4yls is 🔥 with a 🐶\nTweet by developer_dao (2024-08-26 21:41:17.000000Z):\nRT @ZwigoZwitscher : I've been into @ArweaveEco for years – as an interested outsider – and still learned new things in that course 👇. Thanks…\nTweet by angelin...eet by developer_dao (2024-08-26 22:18:09.000000Z):\nRT @jeremykauffman: BREAKING: France has arrested Gonzalve Bich, the CEO of Bic\nTweet by PatrickAlphaC (2024-08-26 22:26:16.000000Z):\n@oxfav @ar_io_network\n",
          "role" => "user"
        },
        %{"content" => "test", "role" => "user"},
        %{
          "content" => "It looks like you're testing the feature. How can I assist you further?",
          "role" => "assistant"
        }
      ]
    }

To Reproduce
A minimal set of resource definitions and calls that can reproduce the bug.
pull down: https://github.com/nheingit/ash_postgres_repo_error
Create the database, run migrations, and then run in IEX
{:ok, resource} = PostgresError.Resource.MapError |> Ash.Changeset.for_create(:create) |> Ash.create()
map = Application.get_env(:postgres_error, :map_bug)
# This will return the Jason Encoder error
resource |> Ash.Changeset.for_update(:bug_here, %{nested_map: map}) |> Ash.update()
# This will run as expected with no issue
resource |> Ash.Changeset.for_update(:update, %{nested_map: map}) |> Ash.update()

Expected behavior
I expected to be able to use my defined update actions to take nested maps and update accordingly.
** Runtime

Elixir version
Erlang version
OS
Ash version
any related extension versions

Additional context
Originally found here: https://elixirforum.com/t/getting-ash-postgres-error-for-map-type-attributes/65736

Thanks for the report! Please update ash_sql to the latest version and try again 🙇


Remove NeoSmart.Unicode (no strong named) dependency form Choice*

We implemented the only piece of logic we used from the NeoSmart.Unicode package within the class.

bool IsEmoji(string letter)
IEnumerable Letters(string text)

I'd prefer those two methods in a separate UnicodeUtils or something, but this looks OK to me.
Waiting for build to pass.

This addresses issue #405


Remove rules_java from MODULE.bazel

Description
This follows up to our previous commit (699c2ce5b7d92476199e9a6ba1d83414c3558016), which introduced the dependency on rules_java as an attempt to support Android build with bzlmod (#1002).
Luckily a subsequent commit (6c61c4f29a966264784d608584000a56596f3ed9) had effectively removed the the dependency on rules_java, but MODULE.bazel has not been reflected yet. Let's also update MODULE.bazel for simplicity and maintainability.
There must be no difference in the final artifacts.
Issue IDs

https://github.com/google/mozc/issues/1002

We have merged your PR.
Thank you for the contribution!


PPTX Export Plugin: Detail slides generation

Generation of detail slides per single ADO item change fully implemented.

Could you please leave the ado-query.wiql in the repository and not putting it into .gitignore

Changing pull as requested.


Update Japanese locale file

まちがい

Are you going to create a new one? There's a lot of good stuff in here!

I didn't hear back from you so I stole your ideas and added them! Sorry and thank you :)


How to better query the system

Hello there,
First of all, let me say that the project is off to a great start and the UI looks really good.
I was hoping to get some guidance on how to improve the prompt engineering for my local CSV data in order to generate better insights. Specifically, I noticed that the current approach could be improved, and I was wondering if you could suggest some alternatives that might be more effective.
I would greatly appreciate any guidance you could provide on this matter. Thank you for your time and assistance.

Hello, thank you for your interest, for this problem I have introduced CSV Agent for more technical or specific requests, normally csv agent can excecute python to answer your question of calculation try to ask csv agent for this kind of request, it is not yet unequal to the chatbot but I am working on it, good day


Add deprecation indicator for URLs when using JSON format.

This is part of the functionality I think I need in order to be able to do a PR checker than can turn red on breaking changes to the migrations or endpoints.
This will enable tagging endpoints as deprecated in urls.py and the PR checker will be able to see if endpoints existed in the latest release but not in the PR, and if they aren't flagged as deprecated in the latest release it will be considered an error.

Look at #2 instead.  The code changes are roughly the same (I improved one little thing, but no functional differences) but it's based on tag 3.0.2 rather than HEAD.


Regression in 2020.11.15 failing to install dependencies defined in setup.py

Be sure to check the existing issues (both open and closed!), and make sure you are running the latest version of Pipenv.
Check the diagnose documentation for common issues before posting! We may close your issue if it is very similar to one of them. Please be considerate, or be on your way.
Make sure to mention your debugging experience if the documented solution failed.
Issue description
I was investigating this issue: https://github.com/pypa/pipenv/issues/4355 and noticed that my project that contains all the dependencies in setup.py and has the Pipfile simply install "." in editable mode is no longer installing anything defined in install_requires in my setup.py. However, this behavior is not present in 2020.11.4.
My setup.py file is defining "install_requires": ["numpy"],
and my Pipfile looks like this:
[packages]
my-thing = {path = ".",editable = true}

Expected result
I expect the lock file to contain numpy and numpy to be isntalled.
Actual result
My package is installed but numpy is missing. No errors visible in any of the verbose output.
Steps to replicate
Provide the steps to replicate (which usually at least includes the commands and the Pipfile).

$ pipenv --support
Pipenv version: '2020.11.15'
Pipenv location: '/home/myuser/.pyenv/versions/3.6.8/lib/python3.6/site-packages/pipenv'
Python location: '/home/myuser/.pyenv/versions/3.6.8/bin/python3.6'
Python installations found:

3.8.5: /usr/bin/python3
3.8.5: /usr/bin/python3.8
3.8.5: /bin/python3
3.8.5: /bin/python3.8
3.7.5: /home/myuser/.pyenv/versions/3.7.5/bin/python3.7m
3.7.4: /home/myuser/.pyenv/versions/3.7.4/bin/python3.7m
3.6.8: /home/myuser/.local/share/virtualenvs/my-thing-sO64gR_A/bin/python3
3.6.8: /home/myuser/.local/share/virtualenvs/my-thing-sO64gR_A/bin/python3.6
3.6.8: /home/myuser/.local/share/virtualenvs/my-thing-sO64gR_A/bin/python3.6m
3.6.8: /home/myuser/.pyenv/versions/3.6.8/bin/python3.6m
3.6.8: /home/myuser/.pyenv/versions/3.6.8/bin/python3.6
3.6.8: /home/myuser/.pyenv/versions/3.6.8/bin/python3
3.6.8: /home/myuser/.local/share/virtualenvs/my-thing-sO64gR_A/bin/python3
3.6.8: /home/myuser/.local/share/virtualenvs/my-thing-sO64gR_A/bin/python3.6
3.6.8: /home/myuser/.local/share/virtualenvs/my-thing-sO64gR_A/bin/python3.6m
3.6.8: /home/myuser/.pyenv/versions/3.6.8/bin/python3.6m
2.7.18: /usr/bin/python2
2.7.18: /usr/bin/python2.7
2.7.18: /bin/python2
2.7.18: /bin/python2.7

PEP 508 Information:
{'implementation_name': 'cpython',
 'implementation_version': '3.6.8',
 'os_name': 'posix',
 'platform_machine': 'x86_64',
 'platform_python_implementation': 'CPython',
 'platform_release': '5.4.0-52-generic',
 'platform_system': 'Linux',
 'platform_version': '#57-Ubuntu SMP Thu Oct 15 10:57:00 UTC 2020',
 'python_full_version': '3.6.8',
 'python_version': '3.6',
 'sys_platform': 'linux'}

System environment variables:

SHELL
LSCOLORS
SESSION_MANAGER
PYENV_HOOK_PATH
QT_ACCESSIBILITY
COLORTERM
PYENV_SHELL
XDG_CONFIG_DIRS
LESS
XDG_MENU_PREFIX
GNOME_DESKTOP_SESSION_ID
PIPENV_ACTIVE
TERMINATOR_DBUS_PATH
GNOME_SHELL_SESSION_MODE
SSH_AUTH_SOCK
DIRENV_DIR
TERMINATOR_UUID
XMODIFIERS
DESKTOP_SESSION
SSH_AGENT_PID
DEVOPS_PYPI_PW
PYENV_VERSION
GTK_MODULES
PWD
LOGNAME
XDG_SESSION_DESKTOP
XDG_SESSION_TYPE
GPG_AGENT_INFO
XAUTHORITY
GJS_DEBUG_TOPICS
WINDOWPATH
HOME
USERNAME
IM_CONFIG_PHASE
LANG
DEVOPS_BUILDS_PW
LS_COLORS
XDG_CURRENT_DESKTOP
VIRTUAL_ENV
VTE_VERSION
INVOCATION_ID
TERMINATOR_DBUS_NAME
MANAGERPID
DIRENV_DIFF
PYENV_DIR
GJS_DEBUG_OUTPUT
XDG_SESSION_CLASS
TERM
ZSH
USER
PIP_DISABLE_PIP_VERSION_CHECK
DISPLAY
APP_NAME
SHLVL
PAGER
QT_IM_MODULE
PYTHONDONTWRITEBYTECODE
KUBECONFIG
XDG_RUNTIME_DIR
PYENV_ROOT
DEVOPS_PYPI_URL
JOURNAL_STREAM
XDG_DATA_DIRS
PATH
GDMSESSION
DBUS_SESSION_BUS_ADDRESS
DIRENV_WATCHES
DEVOPS_PYPI_UN
GIO_LAUNCHED_DESKTOP_FILE_PID
GIO_LAUNCHED_DESKTOP_FILE
OLDPWD
PIP_SHIMS_BASE_MODULE
PIP_PYTHON_PATH
PYTHONFINDER_IGNORE_UNSUPPORTED

Pipenv–specific environment variables:

PIPENV_ACTIVE: 1

Debug–specific environment variables:

PATH: /home/myuser/.pyenv/versions/3.6.8/bin:/home/myuser/.pyenv/libexec:/home/myuser/.pyenv/plugins/python-build/bin:/home/myuser/.pyenv/plugins/pyenv-virtualenv/bin:/home/myuser/.pyenv/plugins/pyenv-update/bin:/home/myuser/.pyenv/plugins/pyenv-installer/bin:/home/myuser/.pyenv/plugins/pyenv-doctor/bin:/home/myuser/projects/things/thing/my-thing/bin:/home/myuser/.local/share/virtualenvs/my-thing-sO64gR_A/bin:/home/myuser/.pyenv/shims:/home/myuser/.pyenv/bin:/home/myuser/.local/bin:/home/myuser/.local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin
SHELL: /usr/bin/zsh
LANG: en_US.UTF-8
PWD: /home/myuser/projects/things/thing/my-thing
VIRTUAL_ENV: /home/myuser/.local/share/virtualenvs/my-thing-sO64gR_A

Contents of Pipfile ('/home/myuser/projects/things/thing/my-thing/Pipfile'):
[[source]]
name = "artifacts"
url = "$DEVOPS_PYPI_URL"
verify_ssl = true

[dev-packages]

[packages]
my-thing = {path = ".",editable = true}

[requires]
python_version = "3.6.8"

Contents of Pipfile.lock ('/home/myuser/projects/things/thing/my-thing/Pipfile.lock'):
{
    "_meta": {
        "hash": {
            "sha256": "2e965c720ff5205f879ac66e310611ee278a76f98b98369f07f25eba3620b70b"
        },
        "pipfile-spec": 6,
        "requires": {
            "python_version": "3.6.8"
        },
        "sources": [
            {
                "name": "artifacts",
                "url": "$DEVOPS_PYPI_URL",
                "verify_ssl": true
            }
        ]
    },
    "default": {
        "my-thing": {
            "editable": true,
            "path": "."
        },
        "install-requires": {
            "hashes": [
                "sha256:d6229486e085a330041d3cc6f47eeb8d2fd0c255cf81a4b899b287c306deb717"
            ],
            "version": "==0.3.0"
        }
    },
    "develop": {}
}

Would you mind providing the content of setup.py. You can redact the secrets of course

@frostming I was able to really narrow down the issue here. It looks like something is parsing my setup.py file instead of executing it… here are a few examples:
I was able to get things working fine with this very basic example:
from setuptools import setup

setup_kargs = {
    "name": "my-foo",
    "author": "someone",
    "author_email"<EMAIL_ADDRESS>    "description": "",
    "license": "",
    "classifiers": [],
    "packages": ["foo"],
    "package_dir": {"": "src/python"},
    "package_data": {},
    "install_requires": ["numpy"],
    "python_requires": ">=3.6",
    "entry_points": {},
}

setup(**setup_kargs)

however, if you pull out the required packages into a variable and use that, it gets ignored:
from setuptools import setup

install_requires = ["numpy"]

setup_kargs = {
    "name": "my-foo",
    "author": "someone",
    "author_email"<EMAIL_ADDRESS>    "description": "",
    "license": "",
    "classifiers": [],
    "packages": ["foo"],
    "package_dir": {"": "src/python"},
    "package_data": {},
    "install_requires": install_requires,
    "python_requires": ">=3.6",
    "entry_points": {},
}

setup(**setup_kargs)

lastly, it seems like the name of the variable actually matters! this example below actually installs numpy:
from setuptools import setup

numpy = ["numpy"]

setup_kargs = {
    "name": "my-foo",
    "author": "someone",
    "author_email"<EMAIL_ADDRESS>    "description": "",
    "license": "",
    "classifiers": [],
    "packages": ["foo"],
    "package_dir": {"": "src/python"},
    "package_data": {},
    "install_requires": numpy,
    "python_requires": ">=3.6",
    "entry_points": {},
}

setup(**setup_kargs)

just to verify it's using the name of the variable for something meaningful, this example below actually throws an error about failing to find a distribution named foo:
from setuptools import setup

foo = ["numpy"]

setup_kargs = {
    "name": "my-foo",
    "author": "someone",
    "author_email"<EMAIL_ADDRESS>    "description": "",
    "license": "",
    "classifiers": [],
    "packages": ["foo"],
    "package_dir": {"": "src/python"},
    "package_data": {},
    "install_requires": foo,
    "python_requires": ">=3.6",
    "entry_points": {},
}

setup(**setup_kargs)

let me know if you need any other examples!

@ms-lolo Thanks for that. I think you have already figured out the workaround.
Either of the following solutions will do the trick:

Use setup(install_requires=install_requires) directly
Define requirements literally in install_requires field.

@frostming I tried applying your suggested fix but I think I ran into some other unexpected behavior and now I'm just even more confused and, unfortunately, troubleshooting this is taking entire days and I can't continue.
I'm guessing my previous version of code was only working by chance and was just not doing what I was defining it to do. All I can do is describe my project structure and hope someone else can spend time to reproduce and troubleshoot because I am out of time here.
I have two python packages in one repo, foo-common and foo-contrib. They both use setuptools-scm and are unreleased, so they are both pre-releases. The foo-contrib package depends on foo-common and installs it as a local editable package. Here are some example snippets:
foo-contrib Pipfile
[packages]
foo-common = {path = "../foo-common",editable = true}
foo-contrib = {path = ".",editable = true}

foo-contrib setup.py
setup(
    name=distribution_name,
    author="…",
    author_email="…",
    description="…",
    license="…",
    classifiers=[
        "Private :: Do Not Upload to pypi server",
    ],
    namespace_packages=namespace_packages,
    packages=pkgs,
    package_dir={"": package_dir},
    package_data={"": ["py.typed"]},
    data_files=generate_data_files(),
    install_requires=[
        "foo-common",
    ],
    python_requires=">=3.6",
    entry_points={},
    use_scm_version={"root": "..", "local_scheme": "dirty-tag"}
)

I am now getting an error about failing to find a version that satisfied my dependencies:
ERROR: Could not find a version that matches foo-common (from -r /tmp/pipenva1ka99_prequirements/pipenv-1y3v0uol-constraints.txt (line 6))
Skipped pre-versions: 0.1.dev218, 0.1.dev219, 0.1.dev220, 0.1.dev221, 0.1.dev222, 0.1.dev223, 0.1.dev224, 0.1.dev225, 0.1.dev226, 0.1.dev227, 0.1.dev228, 0.1.dev229, 0.1.dev230, 0.1.dev231, 0.1.dev232, 0.1.dev233, 0.1.dev234, 0.1.dev235, 0.1.dev237, 0.1.dev238, 0.1.dev239, 0.1.dev240, 0.1.dev241, 0.1.dev242, 0.1.dev243, 0.1.dev244, 0.1.dev245, 0.1.dev246, 0.1.dev247, 0.1.dev248, 0.1.dev249, 0.1.dev250, 0.1.dev251, 0.1.dev252, 0.1.dev253, 0.1.dev254, 0.1.dev255, 0.1.dev256, 0.1.dev257, 0.1.dev260, 0.1.dev261, 0.1.dev263, 0.1.dev264, 0.1.dev265, 0.1.dev266, 0.1.dev267, 0.1.dev268, 0.1.dev269, 0.1.dev270, 0.1.dev271, 0.1.dev272, 0.1.dev273, 0.1.dev274, 0.1.dev275, 0.1.dev276, 0.1.dev277, 0.1.dev278, 0.1.dev279, 0.1.dev280, 0.1.dev281, 0.1.dev282, 0.1.dev283, 0.1.dev284, 0.1.dev285, 0.1.dev286, 0.1.dev287, 0.1.dev288, 0.1.dev289, 0.1.dev290, 0.1.dev291, 0.1.dev292, 0.1.dev293, 0.1.dev294, 0.1.dev295, 0.1.dev296, 0.1.dev297, 0.1.dev298, 0.1.dev299, 0.1.dev300, 0.1.dev301, 0.1.dev302, 0.1.dev303, 0.1.dev304, 0.1.dev305, 0.1.dev306, 0.1.dev307, 0.1.dev308, 0.1.dev309, 0.1.dev310, 0.1.dev311, 0.1.dev312, 0.1.dev313, 0.1.dev314, 0.1.dev315, 0.1.dev316, 0.1.dev317, 0.1.dev318, 0.1.dev319, 0.1.dev320, 0.1.dev321, 0.1.dev322

This happens on 2020-11-4 too, so I guess it's related to avoiding the setup(**config) type code from my previous version? Why is pipenv trying to find a release for my package when Pipfile has it listed as a local editable package? I can't specify --pre because that's very much no ok for all packages in my env, and making this a per-package flag is already an open feature request. But I am super confused about why this is only happening now, when I simply shuffled logic around in my setup file without changing the actual config values.
Sorry if this sounds a bit like a rant, but I ran out of time and can't file a proper ticket for this. I'm adding this comment here because it's all part of my attempt to get 2020-11-15 to be able to work in my environments.

https://github.com/pypa/pipenv/issues/1760 is the feature request preventing me from just hiding from this issue and letting pipenv install the latest version of my dependency, regardless of it being unreleased.

Pipenv uses a "resolve then override" method when there are editable requirements with the same name as the released one. So the release version must exist in the first place to make the resolution work.

In that case, do you think actually packaging a releasing a version v0.1.0 would resolve this? Would it fetch the v0.1.0 release and then override it with my local version before locking?
Any idea why this is not what I am observing with my previous version of setup.py? Is it just this same bug manifesting in different ways on the different pipenv versions?
I think I can give this a try relatively easily tomorrow morning, just trying to make sure I understand before I make more of a mess on my end  :)

was able to test this today and confirmed that I am back to  a functional environment with the workarounds from this thread. https://github.com/pypa/pipenv/issues/1760 is a much more critical feature request for me now because I cannot require the tagging of a fake, stable versions of all local dependencies in my environments.
thanks for all the help getting me running while this bug is open!

#!/usr/bin/env python3

## Modules
import setuptools
from getpass import getuser
import os
from pkg_resources import working_set, Requirement
from setuptools import setup
from subprocess import check_output
import sys
from time import time

## Constants
VERSION = '0.13.7'
REQUIREMENTS = [
        'aws-python',
        'Jinja2'
]

## Funtions
def get_reqs(env, reqs):
    if env == 'prod':
        reqs = [
            f'{r} == {working_set.find(Requirement(r)).version}'
            for r in reqs
        ]
    return reqs

def print_reqs(env, reqs):
    if env == 'prod':
        for r in reqs:
            print(f'+ fixed dependency: {r}', file=sys.stderr)
    else:
        print('+ using latest dependencies', file=sys.stderr)

def get_version(env, version_base):
   # git_id = check_output(['git', 'rev-parse', '--short', 'HEAD']).decode().strip()
    timestamp = int(time())

    if env == 'prod':
        suffix = ''

    elif env == 'dev':
        suffix = f'b{timestamp}'

    else:
        suffix = f'.dev{timestamp}+git{git_id}'

    return f'{version_base}{suffix}'

def get_env():
    env = os.environ.get('ENVIRONMENT')
    if not env:
        try:
            env = getuser()
        except (KeyError, ImportError) as error:
            env = 'local'
    return env

if __name__ == '__main__':
    env = get_env()
    reqs = get_reqs(env, REQUIREMENTS)
    version = get_version(env, VERSION)
    setup(
        name='abc',
        version=version,
        license='BSD 3-Clause',

        packages=['abc'],
        entry_points={
            'console_scripts': [
            ],
        },
        install_requires=reqs,
    )
    print(f'+ version: {version}', file=sys.stderr)
    print_reqs(env, reqs)

Hello all , I am facing issue to release from master version where we have multibranch Jenkins project . I am able to upload succesfully from development branch but while using master branch where ENVIRONMENT=prod facing this issue
AttributeError: 'NoneType' object has no attribute 'version'
while running pipenv install -e .
pipenv --version
pipenv, version 2020.11.15
Is this issue causing because of this bug?
Any help would be appreciate

Fixed on v2021.11.5


Responsive Image

Hi,
Is there a way to make it work with responsive image?
Thank you
JF

What exactly do you mean by responsive image? Would you please modify this demo to illustrate what you mean.
If you want to programmatically update the plugin, call anything zoomer without any options:
// update plugin
$('.zoom').anythingZoomer();

I would like the zoom container or the image width in % so it adjust to the screen size.
Is this possible?

Using a percentage width is not currently possible.
Initially, I was thinking you meant you were using @media css definitions to step-wise change the image dimensions. If that was the case, then you would only need to update the plugin after the window resize event has stopped firing.

I will have to use @media then...
Thank you.


Automate git tagging with GitHub Action

Resolves #33

[ADDED] GitHub Action adds git tag after releasing package to PyPi
[CHANGED] Version number now in separate file

I did extract the version number to its own file to access it more easily in the pipeline and the setup.py. Hope that's ok.


refactor: move all byte conversion logic to lib/shared/units.js

This commit extracts the byte-related conversions from the byte-size
AngularJS module and the FlashStateModel to a re-usable generic
CommonJS module at lib/shared/units.js, than can also be used by the
CLI.
Signed-off-by: Juan Cruz Viotti<EMAIL_ADDRESS>

I've updated the PR to include a byte to megabyte conversion we were doing at FlashStateModel that I had completely forgot about :)

There must be a pre-written NPM module that already does this, surely? .... I did a quick search and found https://www.npmjs.com/package/byte-size
And IMHO the PR title should refer to "byte-size conversion" rather than "byte conversion" :)

...or https://github.com/resin-io/resin-builder/pull/41#discussion_r88424231

Great find! The API is really awkward though. I'll try to find something else, otherwise we can re-use the functionality but still expose it as a reasonable API in lib/shared/units.js.

I couldn't find anything reasonable out there. The "coolest" one is byte-size, but it requires us to add a new dependency, plus going from something like:
exports.bytesToGigabytes = (bytes) => {
  return bytes / 1e+9;
};

to
exports.bytesToGigabytes = (bytes) => {
  return parseFloat(byteSyze(bytes, {
    unit: 'GB',
    precision: 8
  }).value);
};

Which is way more complex, and we even need to convert to an actual number because the module returns a string (!!).

And IMHO the PR title should refer to "byte-size conversion" rather than "byte conversion" :)

Good observation, fixed!

What's the code-size difference between just doing our own units conversion, and creating a 'friendly' wrapper around byte-size?

Not many more lines of course, but those new lines are denser, harder to understand, and the simple division is crystal clear about what it does.
I'll merge, but I'm happy to include a library that provides a simpler replacement for these functions without too much hassle. Or maybe resin.io should publish one? :)

I just did a bit of testing (using https://runkit.com/npm/byte-size), and it seems that with the byte-size NPM module you can't actually explicitly choose which units the input gets converted to, it automatically chooses the 'best' units. So you'll only get GB-sized output, if the input size is actually >= 1GB and < 1TB.
So yeah, the simple division approach is probably easiest for now ;)
(although looking at the code, I think this means the Etcher GUI would currently display a 2TB USB HDD as 2000GB?)

So yeah, the simple division approach is probably easiest for now ;)
(although looking at the code, I think this means the Etcher GUI would currently display a 2TB USB HDD as 2000GB?)

Good call. We can have a bytesToTerabytes() conversion function and have a small GUI utility that chooses the best based on some heuristics (the GB size, for instance). I'm taking notes...

have a small GUI utility that chooses the best based on some heuristics

Um, that's exactly what the byte-size NPM module does ;-)  So maybe it'd be better to simply remove the hardcoded GB from https://github.com/resin-io/etcher/search?utf8=✓&q=gb and update them to just use the byte-size NPM module? (which would also nicely handle the unlikely event of somebody using a very old card with capacity of < 1GB)
And then maybe the same thing could / should be used for https://github.com/resin-io/etcher/search?utf8=✓&q=mb too...

Haha, fair enough :) In this case the extra overhead of the weird byte-size API is worth it.


fix update now from widget

Tested on my mac. I think it's working properly now.

@songgao  --- I think @chrisnojima , @mmaxim  and I all need more context and a fuller understanding of what's happening here.  I don't understand the basic architecture, and this is already a part of the code that is unfortunately complicated and brittle.  The last thing we should be doing is making it more so.

Also cc @cjb

Lets just have a meeting next week

Yes, good idea.

Sure; happy to Meet™. Before that here's some background:
This is called by RPC handler keybase1.config.StartUpdateIfNeeded. This RPC is called when user clicks "Update now" in widget. The bug is after "update now" is clicked, new version app never comes back up. What happens before this PR is:

User clicks "Update now", and we fire keybase1.config.StartUpdateIfNeeded RPC which calls install.StartUpdateIfNeeded.
install.StartUpdateIfNeeded spawns a process running the updater binary.
The updater process pulls the update, and stops the app before starting new one.
When updater process stops the app, it uses a mix of launchd, keybase ctl stop, and if all failed, kill -9. Launchd (I think) sends a signal to the service process to stop.
Since the spawned updater process is in the same process group as the service daemon, the updater process gets terminated too, and never gets a chance to start the new app.

The key here is step 5, where the updater is in the same process group as service, and get terminated together. So this PR just makes the spawned updater process a new process group so it doesn't die when app restarts.
Note that this isn't a problem for keybase update check CLI because in that case the CLI process rather than the service daemon spawns the updater process directly, so it never gets terminated by launchd.


Add rule for setting Sorcerer start cards.

This rule sets the starting cards for the Sorcerer.
A quick copy/paste for testing:
var cards = new List<Rule.SorcererStartCardsModifiedRule.Card>()
{
    new Rule.SorcererStartCardsModifiedRule.Card { Name = AbilityKey.Whirlwind, IsReplenishable = true },
    new Rule.SorcererStartCardsModifiedRule.Card { Name = AbilityKey.Freeze, IsReplenishable = false },
};

var sampleRules = new HashSet<RulesAPI.Rule> { new Rule.SorcererStartCardsModifiedRule(cards) };

Some findings...
Issues with using a Dictionary or List or other:
I started by implementing the rule as generic, taking in a a Dictionary<String, Dictionary<String, bool>>.  That is, a mapping from hero name, to a set of (abilityKey, isReplenishable) values.
The biggest drawback with that is that dictionaries can not contain duplicate keys (e.g., can not specify two of the same cards).  Sure, there are workarounds, like adding a count field, but this felt too hacky.
Alternatively, I considered  Dictionary<List<List<String>>>, with custom parsing.  However, I didn't like the added complexity.
So ... I eventually landed on one rule per hero:
Sure, this results in some duplicated code, but configuring each rule becomes easier by an entire order of magnitude.  This alone may not have been reason enough, except that I had also considered ...
Future feature: Reading rule configuration from a file
Nested structures with a depth of 2+ are more prone to user confusion.  Additionally, it seems considerably easier (and neater) for a user to list starting cards for each hero as a single array.
Consideration for eventually writing to a file was also the reason why I opted to use a struct.  Not only do we get static typing out of the box with structs with no parsing, but it turns out that structs can be nicely de/serialized using one of the libraries that ship with MelonLoader.
We don't necessarily need to go with JSON.  We could abstract the config/parsing and see what all makes sense.
Not necessarily a standard
I still believe rules should be implemented in whatever way it makes sense for the author, and that we will naturally trend towards improvement.

Update:  Implemented the RulesAPI.IConfigWritable interface, so this rule can now be written to and read from config.

Should we add using Datakeys; to RuleMod.cs now that this new ruletype requires it? (Took me a couple of mins to work out why it objected to Name = AbilityKey.Whirlwind in the ruleset definition). Maybe that change comes later when we define some actual rulesets... I think we've got enough things working now that we could define a few of them.

Yeah, I agree - we can add imports as they're required when rulesets that need it make their way in.


Dependency updates, Rubocop update, linter fixes

Follows onto PR #3; as I'm using this in a project I needed to modernize its dependencies a bit. I took a minute to fix its rubocop warnings, too.
I was forced to use TargetRubyVersion: 2.2 in .rubocop.yml; however, I don't believe I broke source compatibility with 1.9.x (not that I think supporting it makes sense).

Again, this does not look like a Travis error related to this PR.

Thank you!


Strategies monitoring - Phase II (non critical)

Product spec to come.
High level:

Not a hard requirement for the first few Convex strategies, but needed for FRAX
Will track a handful of metrics that will trigger alerts with different levels
No dashboard yet, start with email, SMS, and Discord channels

Product spec

Related: #1157

Engineering design doc: https://docs.google.com/document/d/17WXnkAKFEk2QNASmljCT8_P_Bib8svxGdjm-GFxLCuU/edit#heading=h.o2ylpipenap1

I'm going to close this and create new tickets with the remaining work hat we discussed for SP44


set up issue templates

We should set up Issue Templates for this repo. https://github.com/blog/2111-issue-and-pull-request-templates
We should include, at the very least,

tabula version
link to PDF, attached PDF or a promise to email it to us if it's private
platform

thoughts?

Yup. Also, locale of the platform. We get reports about path encoding issues from time to time.

Yep, three today!

A bunch of path encoding issues were resolved in JRuby since 1.7.20, but somewhere along the chain of build steps is using 1.7.15 (see this).

k pienses, @mtigas? Anything else we should add?


operator ibm-block-csi-operator (1.12.0)

not triggering tests for some reason. closing and opening a new PR


Does PHP-VCR work with stream_context_create?

I'm using the expert-sender-api library. The request in the cassette that ends up recorded is almost blank:
    request:
        method: GET
        url: [URL HERE]
        headers:
            Host: [HOST HERE]
            Content-Length: '0'

There should be a request body, and the content-length should not be zero (it's also odd it's a string, no?). Also, the method there is GET, but it should be POST. If I disable PHP-VCR, and perform a live web service call, the request goes out as I expect and then I get a proper response. So something is not right or fully handled by PHP-VCR here.
If I enabled PHP-VCR and let it send the blank request, it captures the response. As you might expect, it's a response to the blank request, but in my case, the response contains 9 headers and an XML body that I'd expect. So that part works.
The expert-sender-api library's query() method here makes use of PHP's stream_context_create I'm wondering if that has anything to do with it. I suppose not, because that's part of HTTP wrapper, which PHP-VCR explicitly states it supports.
If I manually create the cassette request, there's still another problem. Weirdly (this is directly from the php manual):

When using the HTTP wrapper, $http_response_header will be populated with the HTTP response headers. $http_response_header will be created in the local scope.

The fatal error I get with PHP-VCR is:
Undefined variable: http_response_header

That variable is from line 32. Is it possible for PHP-VCR to handle this? I know essentially how PHP-VCR works, and I can't imagine it dealing with that. Searching the source code for http_response_header yields nothing.

Regarding the "there's still another problem". I just ran a different test with PHP-VCR off and not cassette loaded. I received a response using file_get_contents, but PHPUnit still choked, saying Undefined variable: http_response_header. I'm looking more into this mysterious http_response_header thing in PHP, and I'll report back.

Thank you very much. Indeed the stream wrapper interception is only tested with GET requests. Header support was added quite recently. The response headers seem to be indeed not recorded. Definitely some work and tests to be done here :-/ thank you for pointing this out.
This should be fixed. I would be very happy to merge a pull request as I'm again limited on time.

The $http_response_headers could be set by php-vcr, if that's just an ordinary global variable. To be honest I've never used the stream wrappers/contexts this way and wasn't aware that this variable is set.

$http_response_header (note singular) is not global, it's magically brought into existence in the local scope as mentioned in the doc I linked. It's voodoo!
I will work on the stream wrapper with POSTs and submit a pull request.

I've been looking into this. I think the problem I found originally was because the server I was connecting to is sending an intermediate HTTP/1.1 100 Continue header. This stackoverflow answer suggests that we change HttpUtil.php's parseResponse method to look like this in the middle:
list( $rawHeader, $rawBody ) = explode( "\r\n\r\n", $response , 2);
if (strpos($header," 100 Continue") !== false) {
    list($rawHeader, $rawBody) = explode( "\r\n\r\n", $response , 2);
}

Unfortunately, I don't know how to test this in the tests. Doing the above actually throws away the HTTP/1.1 100 Continue header; it doesn't store it in the cassette. I'm not sure if this is robust enough; the cassette doesn't capture the 'whole story'. I'm thinking that's good enough, or at least better than it is now. The alternative is to capture the initial HTTP/1.1 100 Continue as the header, and then everything after would count as the body. It looks pretty gross in the YAML, but you could also modify the cassette format to account for such scenarios.
@adri, I'd love your in-depth PHP knowledge on how to handle the $http_response_header situation. As I noted from the linked documentation, it's a local variable that's magically brought into existence by PHP when using the stream wrappers. How can PHP-VCR invoke this behavior? My googling got me nowhere!

For anyone who is interested in the $http_response_header issue, a bug was filed on 2014-07-13 at https://bugs.php.net/bug.php?id=63897. I wrote some additional comments there. If you want it to get attention, please vote on that page.

This is a hard one. I'm just brainstorming a bit...
It is possible to overwrite $http_response_header with a call to a function defined by php-vcr, much like the other CodeTransforms.There is a high chance for risking false positives though since $http_response_header could be just used as a variable somewhere.
There are projects like PHP-Parser which can be used to parse PHP source code. We could implement a check if some stream wrapper function is called and $http_response_header is referenced in the local scope. Probably this has a big performance impact and should be only done if the PHP file includes $http_response_header at all.
I think it could be solved but it requires work and lots of testing.

As this hasn't been worked on in years, I think now is a good time to close this issue. If anyone wants to work on this, or any other objective, in the future, please open an new issue.


Not sure if it is rocket or the base game or what - Storage Bug

Here is a video of the bug
https://www.youtube.com/watch?v=VsuhF8HiUUs
Note: I have modified the storage values of lockers so not sure if that has anything to do with it
If there are any questions, feel free to ask.

Shutdown does not trigger save, do /save and then /shutdown

Even if I leave the area, not only on server shutdown.


select_pipelines_from_library displays filters multiple times.

On my computer the same filter gets and can be selected multiple times.
On the SSC Jupyter Hub filters get displayed multiple times when rerunning the cell, though only the first filters can be selected.

This happens if the same filter library is added multiple times.


Request setBody not working

Hi,
	$this->response->setHeader('Content-type', 'application/json');
	$this->response->setStatusCode(200)
	               ->setBody(json_encode($json));

Only variables should be passed by reference // ->setBody(json_encode($json));
Working;
$this->response->setHeader('Content-type', 'application/json');
echo json_encode($json);
Message Class not extended, HTTP\Response.

Message Class not extended, HTTP\Response.

Not sure what you mean by this, as HTTP\Response extends HTTP\Message.

Only variables should be passed by reference // ->setBody(json_encode($json));

This isn't the setBody method having an issue, since the setBody method is used elsewhere in the framework to actually output the final views. So it does work. The problem, I believe, lies in trying to use json_encode within the setBody method. Move that out of the argument spot and it should work just fine:
$json = json_encode($json);
$this->response->setStatusCode(200)
	               ->setBody($json);


Docker daemon not honoring "--iptables=false"

Our Docker daemon is not honoring --iptables=false. As soon as I start a container configured to open a port on the host system with e.g . -p 2022:2022 it adds rules to iptables.

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Same problem here! :(

@aboch can you PTAL ?

This is a duplicate of https://github.com/docker/docker/issues/19063 please subscribe to that issue to stay up to date


Schedule job only if not scheduled.

Scheduling job only if not scheduled. To avoid tests failing after the first install of the package in an org. Addresses failures we were seeing in the new Bamboo CI system.
Changes in ADDR_Addresses_TEST.cls are just undoing some minor changes that had been done directly in master branch.

@daveperry ~ could you please review? Thanks!


Browser extension no longer detecting Teams meetings

Describe the bug
When running a Teams meeting in the browser webapp, the LMA browser extension is no longer detecting the meeting platform or meeting name.

With further testing the problem went away. My theory is that my method of testing (using extension running twice in separate incognito tabs) was contributing to interference.  Or there was some other transient DOM change with the teams WebApp that disappeared. Either way, it seems to be working for me now. Will close ticket, but we should  do a multi-way Teams call next week @wchemz to verify.

Also Issue #153 is still open..  Will you be able to follow up with the submitter, @wchemz ?    Maybe after we do some more testing next week?


Support building on Ubuntu (so Travis CI will work)

On Ubuntu, build_deps.sh fails with the following error:
$ ./rebar get-deps
WARN:  Expected /home/travis/build/leveldb-erlang/h2leveldb/deps/qc to be an app dir (containing ebin/*.app), but no .app found.
==> h2leveldb (get-deps)
c_src/build_deps.sh: 67: set: Illegal option -o pipefail
ERROR: Command ['get-deps'] failed!

This is because build_deps.sh assumes /bin/sh on Linux is a symbolic link to bash and does set -o pipefail. But this is not a case on Ubuntu. (Ubuntu Wiki - Dash as /bin/sh - $PIPESTATUS).
I have the same problem on FreeBSD and I am currently not setting pipefail on it (link). Perhaps, I should just explicitly use bash (e.g. #!/bin/env bash) for all platforms.

I changed the shebang of build_deps.sh to #!/usr/bin/env bash and tested it on Ubuntu (Travis CI), FreeBSD and Arch Linux. They look OK and Travis passes now. https://travis-ci.org/leveldb-erlang/h2leveldb
Commits:

https://github.com/leveldb-erlang/h2leveldb/commit/96a74a0288b7ed8112c2da2f5c106cb1dd5dcfcb
https://github.com/leveldb-erlang/h2leveldb/commit/3fa36dff0a3a7c14ef908b58ff515e4d3aa262bb


Error in mapping rules that do not generate triples

For hierarchical data formats (JSON and XML), if a reference in the mapping is not present in the data file, then an error is raised. The intermediate dataframe should have all references from the mapping rule (even when they are not present in the data file) to avoid errors

Fixed in #50


Renamed buildEnvs function as envs

Renamed buildEnvs as envs. Changes have been made to all the func.yaml files. Please check it out. If there is anything i can do differently please let me know.

/kind

Fixes #

Release Note

Docs

@JanaganSaravanan thanks for this contribution! Will you sign the EasyCLA please.


sharp on aws lambda (node 6.10)

For everyone who wants to run sharp on aws lambda, there is a tutorial on the official documentation.
however, I just build sharp using node 6.12 on a aws t2.micro instance which works fine with aws lambda.
sharp-node-6.12.0.zip

Hello, the "latest" Lambda install docs have recently been simplified - see http://sharp.dimens.io/en/latest/install/#aws-lambda - these will be promoted to the "stable" docs after the next release.

I'm kind of a noob with docker.
So I ran docker run -v "$PWD":/var/task lambci/lambda:build-nodejs8.10 npm install. The folder I ran this in remains empty afterwards. Where do I find the files?

Does the folder you're running this in contain the package.json file with sharp included in its dependencies? If so, were any errors generated?


Tracking: basic multi-asset fee support

Is your feature request related to a problem? Please describe.
Users should be able to pay fees in non-native tokens (at a higher price), so that they can use Penumbra just by sending funds into the shielded pool.
Describe the solution you'd like

[ ] #4320 lays out basic multi-asset fee support at the protocol level, but leaves some threads dangling:
[ ] Swap alt fee tokens into the native token prior to fee burn
[ ] Add events for fees
[ ] Change Rust view server to record alt gas prices
[ ] Change Rust view server to report alt gas prices in RPC responses
[ ] Add alt fee token support to pcli

Additional context
Description of the motivation and rationale can be found in #4320

Before the next sprint planning starts, should this be tagged?


Details about training data?

In the report, I did not see the source and scale of the training data. Could you please provide more details?

This part of the information involves our partner organizations and commercialization models, so it is not convenient to disclose it at this time. Thank you for your support and understanding.

We use approximately 35M single-shot clips after filtering, which each clip averaging about 6 seconds.

hi, in the report, image-video joint training is performed. would you mind sharing how many images are used?

@ethanhe42 We used the aes-filtered LAION-5b for training


Fails to load when using bunny 2.6.3 with "Gem Load Error is: uninitialized constant Bunny::Timeout"

Hi,
when using with bunny 2.6.3, I get this error:
Gem Load Error is: uninitialized constant Bunny::Timeout
Did you mean?  Time
Backtrace for gem load error is:

I created a PR to fix this:
pull request
In the meanwhile, you can fix the error by just doing this:
require 'timeout' # before you require bunny-mock
require 'bunny-mock'

Fixed in #28


Pluralize your application title wherever the place in the app

Context
This PR adds the pluralization for "Your application" / "Your applications" in the app

Do you think it's worth testing one application?
It is done on the spec already :)


[BUG] No such export GetDiscordRoles in resource Badger_Discord_API

[script:DiscordAcePer] SCRIPT ERROR: citizen:/scripting/lua/scheduler.lua:881: No such export GetDiscordRoles in resource Badger_Discord_API
What is this? I can't get into the server :( stuck at Deferring Roles
Update... issue resolved. Someone made a syntax error in the config.lua.

What was the error? Having this issue but can't fix


Create a new game.

When creating a new game in the application a game object have to be created on the back-end. The Game object should be sent rules and the player creating the game.

Creating a new game returns the Game object with relation to only one player, namely yourself as the creator of the Game instance.


Dedicated etcd instances (in multiple AZs)

This PR enables dedicated and fault tolerant deployments of etcd with kube-aws.
A new optional config property in cluster.yaml (etcdIPs) controls the number of dedicated etcd instances. They are matched to the subnets that can already be configured for high-availability workers to create them in separate AZs.
If etcdIPs is not specified, etcd runs on the controller instance as before. Dedicated etcd instances use CloudWatch instance recovery and EBS to survive machine failures.
The following section of cluster.yaml allows to run etcd in three separate AZs, allowing for complete loss of one AZ without outages:
subnets:
  - availabilityZone: us-west-1a
    instanceCIDR: "<IP_ADDRESS>/24"
  - availabilityZone: us-west-1b
    instanceCIDR: "<IP_ADDRESS>/24"
  - availabilityZone: us-west-1c
    instanceCIDR: "<IP_ADDRESS>/24"

etcdIPs:
  - <IP_ADDRESS>
  - <IP_ADDRESS>
  - <IP_ADDRESS>

After experimenting a lot with etcd in autoscaling groups and the associated complexity, I find this to be a clean, simple solution to running fault tolerant etcd. From here putting the controllers in an ASG and behind an ELB should be very simple.
(I originally implemented this will full TLS support with client authentication, but since these assets have to be in each of the etcd* cloud-configs, the total stack size quickly exceeds the 52k limit on direct uploads to CloudFormation. When the template is first uploaded to S3 and then referenced there is enough space, any thoughts on this?)

@pdressel Wow, you did a very good job. Thanks for sharing this!
I'm afraid if I'm throwing cold water on your great work but let me share my thought.
I'd like to split subnets into two. One for workers+masters, and the another for etcd.
subnets:
  - availabilityZone: ap-northeast-1a
    instanceCIDR: "<IP_ADDRESS>/24"
  - availabilityZone: ap-northeast-1b
    instanceCIDR: "<IP_ADDRESS>/24"

etcdSubnets:
  - availabilityZone: ap-northeast-1a
    instanceCIDR: "<IP_ADDRESS>/24"

etcdIPs:
  - <IP_ADDRESS>
  - <IP_ADDRESS>
  - <IP_ADDRESS>

and have 2 k8s clusters, one with etcd in 1a and the another for etcd in 1b.
Reasons behind this are:

To achieve resilience in the face of single AZ failure when there is not 3 but only 2 AZs in your region (e.g. Seoul, Beijin, Seigapore, and Tokyo) when...
You want to deploy Multi-AZ aware app on each K8s cluster (if it isn't a requirement for you, you can choose to deploy single AZ k8s cluster on each AZ for hosting single-AZ-aware apps, instead of 1 Multi-AZ k8s cluster for hosting multi-AZ-aware apps. No changes is needed on user side once this PR is merged then)
You can have 2 k8s clusters to limit failure domain. E.g. when 2 AZs 1a and 1b are available in your region, placing etcd in 1a for the first k8s, and in 1b for the second k8s while spreading k8s masters and nodes over 2 AZs. In the face of single AZ failure, one of two k8s clusters will be fully functioning and you can earn time to migrate(maybe scale up/out the nodes in the functioning cluster?).

Maybe am I overthinking?

A 2-AZ k8s cluster containing 2 etcd and a tie-breaker etcd node in another region,
or having single-AZ k8s cluster for each zone,

will suffice?
Btw, regardless of this PR or kube-aws itself supports H/A for regions with less than 3 AZs, this PR is GREAT!!
I'm really looking forward for it it be merged.

@colhom @aaronlevy @philips Sorry for bothering you but I believe you're working on dedicated etcd clusters to land kube-aws too!
Would you mind looking into this?

@mumoshu I agree, we should also have a solution for regions with < 3 AZ. I checked the AWS docs, the following regions have only 2 AZs:

AWS GovCloud
Frankfurt
Singapore
Bejing
Seuol

There are also 5 new regions being built right now, for which the number of AZ is not published yet.
Since it is fundamentally not possible to safely keep a quorum-based service like etcd online with just two failure domains, having at least a soft failure with two clusters seems like a good option. Using a separate region for an etcd arbiter node would work, but AFAIK CloudFormation stacks are limited to a single region, so automatic deployment of this would be complicated. Using the current implementation, couldn't we achieve the two-cluster solution you described with the following configuration?
Cluster A
subnets:
  - availabilityZone: ap-northeast-1a
    instanceCIDR: "<IP_ADDRESS>/24"
  - availabilityZone: ap-northeast-1b
    instanceCIDR: "<IP_ADDRESS>/24"

etcdIPs:
  - <IP_ADDRESS>
  - <IP_ADDRESS>
  - <IP_ADDRESS>

Cluster B
subnets:
  - availabilityZone: ap-northeast-1a
    instanceCIDR: "<IP_ADDRESS>/24"
  - availabilityZone: ap-northeast-1b
    instanceCIDR: "<IP_ADDRESS>/24"

etcdIPs:
  - <IP_ADDRESS>
  - <IP_ADDRESS>
  - <IP_ADDRESS>

Let me know what you think and thank you for the praise!

@pdressel Great. Having 2 k8s clusters and differenciating the subnet in which more than majority of etcd instances placed, for each cluster, as you've shown above would work!
FYI to someone reading these comments in the future. More than majority in this case means 2 of 3 etcdIPs.
In the cluster A, we see 2 10.0.0.x IPs in 1a. In the cluster B, we see 1 10.0.0.x in 1a and 2 10.0.1.x IPs in 1b. A single AZ failure will take down one of those two clusters but the whole system will keep functioning.
It may suffer with degradated performance until the alive cluster scales depend on your workload but I believe it is not what we can fix completely.
AFAIK while the "dead" cluster having e.g. 1 of 3 etcds alive, etcd won't accept writes and therefore k8s is unable to (re)schedule pods or (re)configure services or auto-scale pods or etc. Already running pods will keep running but I don't know a single AZ failure affects network between the internet and the pods. So how much/fast the alive cluster must scale will depend. Though, for me, it is much better than your k8s has a chance to fully stop working until the AZ failure is fixed. And this PR seems to allow us to prepare for single AZ failures regardless of num of AZs in your regions.
Great.
Now this PR is LGTM also in that regard!
Thanks @pdressel
P.S. to folks using the ap-northeast-1 or the us-west-1 region, AWS actually have 3 AZs in it(1a, 1b, 1c) but usually they limit you to 2 AZs. I had asked my SAs if they could expose the third AZ but had no luck. So you may be interested in having 2 k8s clusters like we discussed above, too.

@pdressel I've been travelling last week and working on the exact same thing :(. I just put a WIP PR up- #544.
With respect to the etcd/AZ issue, I simply round-robin etcdCount across all the specified availability zones. Given that there are a fixed number of AZs per region, we can't do any better than this as far as availability goes (until we figure out dynamic etcd cluster management). ref https://github.com/coreos/coreos-kubernetes/pull/544/files#diff-e5ad8b828a5de3fe2229bee5c64ab2a0R187
\cc @mumoshu

@colhom no worries. I think your implementation using the same cloud-config-etcd for all instances is great. It nicely gets around the limitation I ran into when implementing TLS regarding template size. I am happy to close this PR and contribute to yours if you like, maybe for future work we can sync beforehand :) I would really like to help advance this project and can dedicate some resources to it.
I am not sure about the round robin vs predefined fixed IP approach for the instances - using predefined IPs for the instances might make instance replacement for cluster upgrades easier later down the road: a replacement etcd instance could re-use the IP (and thus etcd name and data dir) without runtime reconfiguration. What do you think?

@pdressel what I really want to do is setup a private hosted zone and refer to the nodes by DNS name. etcd1.xxx, etcd2.xxx, etcd3.xxx, kube-controller1.xxx,kube-worker1.xxx,kube-worker2.xxx It definitely complicates the cloudformation, adding cost and complexity.
This way though, we can just throw all the interfaces (etcd, controllers and workers!) on dhcp and not worry about it. Would greatly simplify TLS signing and we offload address allocation entirely. The problem is I haven't figured out how to assign these addresses to members of an autoscaling group without a lambda hook on autoscaling add/remove events (I want to avoid adding more bells & whistles if possible).
\cc @cgag

I'm going to close this now if the consensus is to continue work on this in https://github.com/coreos/coreos-kubernetes/pull/544 and https://github.com/coreos/coreos-kubernetes/pull/608
Please let me know if this should be re-opened


Home Screen Book List

Administrative Info
Monday Board ID: 853642866
Changes
What changes did you make?

Added "New Books" section
Added "All Books" section
Added support for custom font (Gotham Rounded)
Added circle at the top of page

Testing
How did you confirm your changes worked?

Rendered on both Anshul and Aksharan's screens

Confirmation of Change

Did not add comments to the code. Had some questions we wanted to ask you in Wednesday's meeting, but thought you would like to see what we have done so you could give us pointers tomorrow also :)

Did not add comments to the code. Had some questions we wanted to ask you in Wednesday's meeting, but thought you would like to see what we have done so you could give us pointers tomorrow also :)

I'm also getting an error: "VirtualizedLists should never be nested inside plain ScrollViews with the same orientation - use another VirtualizedList-backed container instead."
Sounds like it doesn't like that the entire home screen is a ScrollView, with a FlatList inside of it. That sucks, maybe the root of the home screen can be a  with ListHeaderComponent containing the header, horizontal book scrollers, and text headers?

I'm also getting an error: "VirtualizedLists should never be nested inside plain ScrollViews with the same orientation - use another VirtualizedList-backed container instead."
Sounds like it doesn't like that the entire home screen is a ScrollView, with a FlatList inside of it. That sucks, maybe the root of the home screen can be a  with ListHeaderComponent containing the header, horizontal book scrollers, and text headers?

I think there's a problem with the way we're displaying all the books using ColumnBookList. I ran the app on a few different devices and found that there wasn't much space between books on the same row for smaller devices (ex. iPhone 12 mini)

Currently, if the total number of books is congruent to 2 modulo 3, the last row of the all books grid ends up formatted a bit weirdly.

Currently, if the last row of the all books grid has 2 books then it ends up formatted a bit weirdly.

This bug is known and we are going to fix it in a later version

Yep, I want to check with design to see if those last 2 should be left-aligned, or if they should be spaced evenly, or something else. So I'm fine with just living with it for now.


[Backport 2.x] Maintainer approval check (#11378)

Description
Backport of 87b3011 from #11378
Related Issues

Related https://github.com/opensearch-project/OpenSearch/issues/10613#issue-1942146755

Check List

[ ] ~New functionality includes testing.~

[ ] ~All tests pass~

[ ] ~New functionality has been documented.~

[ ] ~New functionality has javadoc added~

[ ] ~Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)~
[X] Commits are signed per the DCO using --signoff
[X] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
[ ] ~Public documentation issue/PR created~

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Impacted by failure from https://github.com/opensearch-project/OpenSearch/issues/11574


pallet-lottery: add generate_storage_info

Adds generate_storage_info to pallet-lottery.
Related https://github.com/paritytech/substrate/issues/8629
I still need some help with the MR tagging.

Lottery pallet is not deployed to any production runtimes, so it should be fine to merge this without any worries about migrations.

/benchmark runtime pallet pallet_lottery

bot merge


Typo for Zcmop instruction

I think there is a typo for the Zcmop number example the compressed C.sspush is noted as using Zcmop.1 encoding when it is Zcmop.0

The description follows the latest updated Zcmop specification. I believe the quoted version is stale.

ok sorry didn't know there was a new version for Zimop thanks