Start of conversion to .toURI()

http://stackoverflow.com/a/7700072/613748
Don't use .getAbsolutePath() as it can break when a path has spaces.

This change is

Coverage remained the same at 53.886% when pulling 5c2d8c3268cbf631d9fffb1d91b7cf4e477c06bf on tests-to-uri into 01a888f30f00a4420277b8ba00d49565a41254f2 on master.

Clarification:
A URI should be used with database connection strings, and a Path should be fine for everything else.

Coverage remained the same at 53.886% when pulling 049c2cb545c0947c0b28ae7e9021577e0de4bb9f on tests-to-uri into 01a888f30f00a4420277b8ba00d49565a41254f2 on master.

Reviewed 2 of 2 files at r1, 10 of 10 files at r2.
Review status: all files reviewed at latest revision, 5 unresolved discussions, some commit checks failed.

Common/src/test/java/com/rgi/common/util/jdbc/JdbcUtilityTest.java, line 34 [r2] (raw file):
should import specific modules.

DataStore/src/main/java/com/rgi/store/tiles/geopackage/GeoPackageWriter.java, line 133 [r2] (raw file):
File URI may lead to some confusing output for users, we may want to format the uri to a more human readable format (they will be expecting "C:\yadda yadda"

GeoPackage/src/test/java/com/rgi/geopackage/DatabaseUtilityTest.java, line 66 [r2] (raw file):
i know this will autoclose the connection, but will it delete the temp file as well?

GeoPackage/src/test/java/com/rgi/geopackage/DatabaseUtilityTest.java, line 85 [r2] (raw file):
same as above, is the temp file deleted when the connection is closed?

GeoPackage/src/test/java/com/rgi/geopackage/GeoPackageTilesAPITest.java, line 184 [r2] (raw file):
is testFile global? (reviewable doesnt show item instantiation)

Comments from Reviewable

Review status: all files reviewed at latest revision, 5 unresolved discussions, some commit checks failed.

DataStore/src/main/java/com/rgi/store/tiles/geopackage/GeoPackageWriter.java, line 133 [r2] (raw file):
Reverting back to getPath()

GeoPackage/src/test/java/com/rgi/geopackage/DatabaseUtilityTest.java, line 66 [r2] (raw file):
yes

GeoPackage/src/test/java/com/rgi/geopackage/DatabaseUtilityTest.java, line 85 [r2] (raw file):
yes as well, see TestUtility.getRandomFile().  The file will delete itself once the JVM exits.

GeoPackage/src/test/java/com/rgi/geopackage/GeoPackageTilesAPITest.java, line 184 [r2] (raw file):
no, testFile is just in that test function.

Comments from Reviewable

Reviewed 2 of 2 files at r3.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Coverage remained the same at 53.886% when pulling bcb6497b2f296d8c9030278fb4f1f76e7d9ef1e4 on tests-to-uri into 01a888f30f00a4420277b8ba00d49565a41254f2 on master.


Spotahome Redis operator - fixes

Resolving guide comments after tech-review

@ganochenkodg lemme know when this is good to merge in


Transaction signer parameters are parsed incorrectly

Describe the bug
The signer / authorizer account selection in the transaction submission box is empty, even though it should allow the selection of an authorizer when the prepare block has account parameters.
To Reproduce

Deploy the Hello World contract from https://docs.onflow.org/cadence/tutorial/02-hello-world, i.e. https://play.onflow.org/5b5bd86b-3845-4c16-9a29-626ef6228846 in Account 1
Try to submit transaction 1
No accounts can be selected to sign the transaction
When removing the comment between the import statement and the transaction declaration, which contains the word "prepare", the dialog correctly allows the selection of signer accounts.

Expected behavior

Signer selection should always work, even when comments or unrelated code mentions "prepare"

Screenshots

Desktop (please complete the following information):

OS: Any
Browser: Any

Additional context
The problem seems to be that the code which extracts the parameters from the prepare block, https://github.com/onflow/flow-playground/blob/d4ec5c6676285fdaf66b9cdda12e8338f6048d1c/src/components/CadenceEditor.tsx#L342-L365,
is using ad-hoc parsing to detect a prepare block, but it doesn't account for "prepare" words in comments.

This is really nice catch, @turbolent
I will take a look into it 🙇♂

quickly verified this is still an issue in the current version of Playground. curious if Cadut can suffer the same problem, or how Cadut solved this problem

Fixed


Major improvements & refactorings

What's implemented:

Pure python support
a) For Windows installer from python.org is downloaded and installed
b) For UNIX python-build from pyenv is used for creation
Virtualenv creation from any environment created
Package installation for virtualenv
env from zip creation: downloading archive from specified url, unpacking and preparing to work with
Dynamic-typed code refactored to static-typed
IronPython support
PyPy support
Other improvements

After the scope of the plugin outgrew the conda, we should change the description.
Now it says: "Gradle plugin to create conda envs."


Ignoring port

# graphql get-schema
Downloading introspection from http://api.mysite.dev:8888/graphql
request to http://api.mysite.dev:8888/graphql failed, reason: connect ETIMEDOUT <IP_ADDRESS>:80

{
  "schemaPath": "schema.graphql",
  "extensions": {
    "endpoints": {
      "dev": "http://api.mysite.dev:8888/graphql"
    }
  }
}

Host file:
<IP_ADDRESS> api.mysite.dev

My fault, turns out I needed a '/' after the URL. Unfortunately it took wireshark to figure this out.

@Aloxis was the related to your specific server?


open wiki for detailed set up instructions?

Hi there Akshay @phenax
Thank you for sharing your library.
I set up a working example on a rails app. it was a little bit tricky: i feel some pointers would be useful for the general community -  would you be against opening up the wiki for these rails set up instructions to be documented? If you deem it useful, I can easily add them in.
chrs,
Ben

Interesting idea. However, I think this plugin by itself isn't complex enough for many nuances. Any guide on this would be too specific to either esbuild or elm. But I do think a general guide on setting up an elm project with esbuild would be incredibly helpful. If it can be turned into a blog post on dev.to or somewhere, I would love to link it in the readme!

But I do think a general guide on setting up an elm project with esbuild would be incredibly helpful. If it can be turned into a blog post on dev.to or somewhere, I would love to link it in the readme

Many rails users will be switching to es-build over the next few months. Your plugin will be incredibly useful to this community - and I imagine others:  i can do a rails specific tutorial on setting up elm with esbuild (as well as initialising the javascript), and leave it to your absolute discretion whether to link it or not. There is no loss from my side in doing so, because i have no doubt it will be useful to the wider community.
Thanks again for the library. (I can see that NoRedInk have forked your repo, which speaks volumes for its usefulness)

Closing in favour of #13.


Renaming a script without a supported file extension, hides the script without a warning

Godot version
v3.4.1.stable.official [aa1b95889]
System information
Windows 10 21H1
Issue description
Renaming a script in the FileSystem panel, by removing the file extension .gd
Hides the file from the FileSystem panel, without showing a warning message.
This might seem like it was deleted.
But it's still available in the project directory.
Steps to reproduce
In the FileSystem panel

Create a new script file
Rename it by removing the extension .gd

Observed
The file disappears from the FileSystem panel.
Expected
A warning message should probably be shown.
Stating that renaming a file without a supported extension, will hide the file in Godot.
It will not be deleted, it's still available in the project directory.
Minimal reproduction project
No response

Happens on master branch too.

I've got the logic the detect when such problem will occur.
But I've no idea on what UI is the best to deal with it :

a warning icon at the right of the LineEdit with a tooltip explaining the problem (didn't see such icon in godot so may feel not consistant in editor)
a label appearing below the lineedit updated before confirming with the OK button (this way is used in script creation dialog but in this rename dialog, there is no space dedicated to such label, would change the dialog size when appears). OK button could also be disabled at this stage if we want to prevent people from renaming with unknown extensions.
another confirm dialog after clicking OK, explaining the problem and giving the opportunity to proceed anyway or cancel (coming back to rename dialog).

I wonder if we shouldn't prevent these renaming in dock as there is not option to show hidden files in FileSystem dock.
People who really wants to rename with an unsupported extension could use system file explorer.


Task status symbols abruptly cut-off

Font: Ubuntu Mono
Terminal: zsh
ZSH theme: Spaceship
They work perfectly well with DejaVu Sans Mono but not with any other Mono fonts.

Hmm that might be due to the charachter width limitation. I do not know how to fix that, since it is using an external library to provide the symbols. But I will take a look in the longer run.

Yes, v10.2.4.

I am going to close this for now, if it is okay with you. But it will be back of my mind even though I think this might be caused by the terminal character width.


lib/comrg/cmake documentation

Is there documentation on what bc2h.cmake, DeviceLibs.cmake, and opencl_pch.cmake do?

Hi @mrcsguy
Thanks for reaching out.
I hope this solves your question: https://github.com/RadeonOpenCompute/ROCm-CompilerSupport
Feel free to reach me for any other information too.
Thank you.

Request you to close this ticket if my response satisfies your need.
Thank you.

Am closing this considering that issue is resolved and no response from user.
Thank you.


Can't get sample test with Generic driver to work against Linux system

Hi,
I've been using Netmiko for a while but wanted to give scrapli a try to hopefully take advantage of speed.  My goal is to get this working against Drivenets boxes but my initial attempt using the generic_driver.py example did not go well, as I couldn't even log in (got stuck here: conn = GenericDriver(**MY_DEVICE).    With that said, I tried the same simple script against an Ubuntu box with the hope of just logging in and issue a "pwd".  It gets hung and times out at conn.send_command.  Here is the the example against the Ubuntu system:
from scrapli.driver import GenericDriver

MY_DEVICE = {
    "host": "<IP_ADDRESS>",
    "auth_username": "xxxx",
    "auth_password": "xxxxxx",
    "auth_strict_key": False,
}

def main():
    """Simple example of connecting to an IOSXEDevice with the GenericDriver"""
    # the `GenericDriver` is a good place to start if your platform is not supported by a "core"
    #  platform drivers
    conn = GenericDriver(**MY_DEVICE)
    print(f"conn.transport.open_cmd: {conn.transport.open_cmd}")
    conn.open()
    print("hello2")

#   print(conn.channel.get_prompt())
    print(conn.send_command("pwd").result)
    print("hello3")
    conn.close()

if __name__ == "__main__":
    main()

Here is what happens when I run it:
$ python sample.py
conn.transport.open_cmd: ['ssh', '<IP_ADDRESS>', '-p', '22', '-o', 'ConnectTimeout=15', '-o', 'ServerAliveInterval=30', '-l', 'as2863', '-o', 'StrictHostKeyChecking=no', '-o', 'UserKnownHostsFile=/dev/null', '-F', '/dev/null']
hello2
Traceback (most recent call last):
  File "sample.py", line 34, in <module>
    main()
  File "sample.py", line 28, in main
    print(conn.send_command("pwd").result)
  File "/home/as2863/python-venv/p1_netmiko_exper4/lib/python3.7/site-packages/scrapli/driver/generic/sync_driver.py", line 168, in send_command
    timeout_ops=timeout_ops,
  File "/home/as2863/python-venv/p1_netmiko_exper4/lib/python3.7/site-packages/scrapli/decorators.py", line 408, in decorate
    result = wrapped_func(*args, **kwargs)
  File "/home/as2863/python-venv/p1_netmiko_exper4/lib/python3.7/site-packages/scrapli/driver/generic/sync_driver.py", line 132, in _send_command
    channel_input=command, strip_prompt=strip_prompt, eager=eager
  File "/home/as2863/python-venv/p1_netmiko_exper4/lib/python3.7/site-packages/scrapli/decorators.py", line 278, in decorate
    kwargs=kwargs,
  File "/home/as2863/python-venv/p1_netmiko_exper4/lib/python3.7/site-packages/scrapli/decorators.py", line 335, in _multiprocessing_timeout
    self._handle_timeout()
  File "/home/as2863/python-venv/p1_netmiko_exper4/lib/python3.7/site-packages/scrapli/decorators.py", line 311, in _handle_timeout
    raise ScrapliTimeout(self.message)
scrapli.exceptions.ScrapliTimeout: timed out sending input to device
(p1_netmiko_exper4) as2863@ubuntuP1-SYSlog-S1:~/pythonProjects/p1-automation/scrapli
$

Once I get this simple test working, I'll go back to the Drivenets system.  With Drivenets, the conn.open() failed.  Is this the best place to ask these issues.  I can't get on slack, as it is blocked by company I am consulting at.

hey @allan-silverstein :wave:
for question stuff slack is probably preference, but also the discussions thing here on GitHub is probably a good place. for now this works though.
99% sure you just need to add "comms_ansi": True to your MY_DEVICE dict there. I am probably going to make this the default/standard here shortly as a lot of devices end up plopping ansi chars in the output and its probably "cheap" enough to not worry about having it be toggle-able. With just adding that your script works nicely against an ubuntu host I have handy.
Let us know if that works!
Carl

re being stuck here: conn = GenericDriver(**MY_DEVICE) -- would def need to know what error, that doesn't do anything other than create the connection object (meaning there are no "side effects" -- it doesn't open a connection or anything) so that shouldnt cause any chaos :)

Excellent. That now works so I now know the package should be installed correctly and working.  Back to the Drivenets box.
Here is a very similar simple test:
from scrapli.driver import GenericDriver

#   "host": "au22",
MY_DEVICE = {
    "host": "<IP_ADDRESS>",
    "auth_username": "p1automon",
    "auth_password": "xxxxx",
    "auth_strict_key": False,
    "comms_ansi": True,
}

def main():
    """Simple example of connecting to an IOSXEDevice with the GenericDriver"""
    # the `GenericDriver` is a good place to start if your platform is not supported by a "core"
    #  platform drivers
    conn = GenericDriver(**MY_DEVICE)
    print(f"conn.transport.open_cmd: {conn.transport.open_cmd}")
    print("hello1")
    conn.open()
    print("hello2")

#   print(conn.channel.get_prompt())
    print(conn.send_command("show interface | no-more").result)
    print("hello3")
    conn.close()

if __name__ == "__main__":
    main()

It fails/timesout at conn.open()
$ python sample.py
conn.transport.open_cmd: ['ssh', '<IP_ADDRESS>', '-p', '22', '-o', 'ConnectTimeout=15', '-o', 'ServerAliveInterval=30', '-l', 'p1automon', '-o', 'StrictHostKeyChecking=no', '-o', 'UserKnownHostsFile=/dev/null', '-F', '/dev/null']
hello1
Traceback (most recent call last):
  File "sample.py", line 29, in <module>
    main()
  File "sample.py", line 20, in main
    conn.open()
  File "/home/as2863/python-venv/p1_netmiko_exper4/lib/python3.7/site-packages/scrapli/driver/base/sync_driver.py", line 87, in open
    auth_private_key_passphrase=self.auth_private_key_passphrase,
  File "/home/as2863/python-venv/p1_netmiko_exper4/lib/python3.7/site-packages/scrapli/decorators.py", line 278, in decorate
    kwargs=kwargs,
  File "/home/as2863/python-venv/p1_netmiko_exper4/lib/python3.7/site-packages/scrapli/decorators.py", line 335, in _multiprocessing_timeout
    self._handle_timeout()
  File "/home/as2863/python-venv/p1_netmiko_exper4/lib/python3.7/site-packages/scrapli/decorators.py", line 311, in _handle_timeout
    raise ScrapliTimeout(self.message)
scrapli.exceptions.ScrapliTimeout: timed out during in channel ssh authentication

If you want to try changing the transport to paramiko/ssh2 that could quickly see if that gets past any auth stuff, otherwise will need logs.
You can add the following to get channel logging and the "normal" python logs:
from scrapli.logging import enable_basic_logging

enable_basic_logging(file=True, level="debug")

And add "channel_log": True to your MY_DEVICE.
If you wanted to try paramiko/ssh2 transport you could add "transport": "ssh2" to your MY_DEVICE. You'll need to install that like: pip install scrapli[full]

Ok, looking at the logs, it looks like it does accept the username/passwd but the format of the output may not be what scrapli is expecting.  Can I send you the log files through a different means, as my client may not want these posted....

Here is the tail portion of the scrapli.log file which should help:.  Notice the device prompt all the way to the right margin.
<EMAIL_ADDRESS>password: "
10    | 2021-03-02 18:43:35,161 | DEBUG    | <IP_ADDRESS>:22         | write: REDACTED
11    | 2021-03-02 18:43:35,162 | DEBUG    | <IP_ADDRESS>:22         | write: '\n'
12    | 2021-03-02 18:43:35,162 | DEBUG    | <IP_ADDRESS>:22         | read : b'\nDRIVENETS CLI Loading...                                                                                AUTTX22CWP# '
13    | 2021-03-02 18:44:04,968 | CRITICAL | <IP_ADDRESS>:22         | channel operation timed out, closing transport
14    | 2021-03-02 18:44:04,969 | DEBUG    | <IP_ADDRESS>:22         | closing transport connection to '<IP_ADDRESS>' on port '22'
15    | 2021-03-02 18:44:05,070 | CRITICAL | <IP_ADDRESS>:22         | encountered EOF reading from transport; typically means the device closed the connection
16    | 2021-03-02 18:44:05,169 | DEBUG    | <IP_ADDRESS>:22         | transport connection to '<IP_ADDRESS>' on port '22' closed successfully

You will probably need to either modify your comms_prompt_pattern, see here -- which may not be super desirable as I'm guessing that subsequent prompts are more "normal" (prompt starting at a newline each time)....
Or set auth_bypass -> True and provide a custom on_open (docs) to handle the auth/initial login (if you use ssh2 or paramiko transport the auth part won't happen "in channel" as it will happen in either of those libraries, so then you should just have to deal w/ sending a return and getting past any login banner stuff).
This is because the default pattern is not going to match that as there is no newline before the actual device prompt there.
In any case, I don't see a bug here so I'm going to close this. You can follow this up on slack or in discussions but I'm trying to keep issues for bugs only.
Carl

ok, thx


Use Jenkins source type for all events

Was implemented in f00b261 but got reverted (not intentionally afaik) when #38 was merged

Thanks @olivielpeau


nested REPEAT needs parentheses, otherwise symbols not updated

Reported by Andrew Wittenberg on 29 Jul 2005 20:55 UTC
Not sure if this is related to bug 558.  The inner repeat loop must be enclosed
by parentheses, otherwise the embedded symbols are not re-translated each time
through the loop.  Trivial to work around, once you know what the problem is!
    NOAA/PMEL TMAP
    FERRET v5.81
    Linux(g77) 2.4.18 - 07/14/05
    29-Jul-05 16:46

yes? can mode verify; def sym a 0
yes? rep/l=1:2 rep/j=1:3 (def sym a j; show sym a; say ($a))
A = "1"
0
A = "2"
0
A = "3"
0
A = "1"
0
A = "2"
0
A = "3"
0
yes? rep/l=1:2 (rep/j=1:3 (def sym a j; show sym a; say ($a)))
A = "1"
1
A = "2"
2
A = "3"
3
A = "1"
1
A = "2"
2
A = "3"
3
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1303

Comment by andrew.wittenberg on 16 Nov 2009 23:19 UTC
Formatting was lost in the transition to Trac:
        NOAA/PMEL TMAP
        FERRET v6.31
        Linux(g77) 2.4.21-57.ELsmp - 11/05/09
        16-Nov-09 18:17

yes? can mode verify; def sym a 0
yes? rep/l=1:2 rep/j=1:3 (def sym a `j`; show sym a; say ($a))
A = "1"
0
A = "2"
0
A = "3"
0
A = "1"
0
A = "2"
0
A = "3"
0
yes? rep/l=1:2 (rep/j=1:3 (def sym a `j`; show sym a; say ($a)))
A = "1"
1
A = "2"
2
A = "3"
3
A = "1"
1
A = "2"
2
A = "3"
3

Modified by ansley.b.manke on 5 Jan 2011 20:03 UTC


Security - User authentication

Hi, does Pinpoint support a user authentication module, to allow them to access the tool?
Thank's

Hi @mastiffs1990
Unfortunately pinpoint doesn't provide authentication/authorization. You should implement function ourself. Please ref this issue.


MAINTAINERS: add twister related files to area

Add pytest plugin and related documenation.
Signed-off-by: Anas Nashif<EMAIL_ADDRESS>

What is the policy for adding a collaborator? I propose to add @gchwier also to this section. He's from my team and actively contributes to twister and was developing pytest plugin

What is the policy for adding a collaborator? I propose to add @gchwier also to this section. He's from my team and actively contributes to twister and was developing pytest plugin

done


Clear errors when delete a chat/action

Details

We are not clearing any errors when deleting an action.
Fixed Issues

$ https://github.com/Expensify/App/issues/16852
PROPOSAL: https://github.com/Expensify/App/issues/16852#issuecomment-1497311034
Tests

Same as QA Steps

[x] Verify that no errors appear in the JS console

Offline tests

Can't test in offline
QA Steps

Go to Settings>Preferences and toggle on "Simulate failing network requests"
Open any chat
Send an image attachment
Wait for a few seconds (I think 3 seconds is enough)
Go back to Settings>Preferences and toggle off "Simulate failing network requests"
Go to the previous chat and wait until the error shows
Verify there is a red dot in LHN
Delete the attachment
Verify the red dot in LHN is gone

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

https://user-images.githubusercontent.com/50919443/231949764-ea70fe9e-c975-44c7-a9d5-225b066a4aa3.mov

Mobile Web - Chrome

https://user-images.githubusercontent.com/50919443/231955736-14c6aa2e-fc48-4a4a-8ef2-ff6584deafe6.mp4

Mobile Web - Safari

https://user-images.githubusercontent.com/50919443/231950094-7be391aa-ea47-4d11-9628-e4dc4195eb4d.mov

Desktop

https://user-images.githubusercontent.com/50919443/231949682-3ceaad58-266f-427b-b1eb-d9e50a4de536.mov

iOS

https://user-images.githubusercontent.com/50919443/231949959-b4982fa7-fde4-4440-8c14-0f8596ca5892.mov

Android

https://user-images.githubusercontent.com/50919443/231955751-72d597bc-56b6-4d17-b92e-634aa1aefcc4.mp4

@neil-marcellini @0xmiroslav One of you needs to copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

I decided to not revert the error when the delete request fails because if the request fails, the delete request will have it's own error. This is the same thing we do with delete workspace
https://github.com/Expensify/App/blob/7f06c27d23f2545f38f96b4b682016c30f3c2049/src/libs/actions/Policy.js#L77-L129

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/neil-marcellini in version: 1.3.1-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/neil-marcellini in version: 1.3.1-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/mountiny in version: 1.3.1-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Warnings due to use of gs

When building erlang-serial the following warnings now occur (using OTP/17):
src/terminal.erl:95: Warning: gs:start/0: the gs application has been deprecated and will be removed in OTP 18; use the wx application instead
src/terminal.erl:96: Warning: gs:create/3: the gs application has been deprecated and will be removed in OTP 18; use the wx application instead
[... a lot more of these ...]

Presumably it will not even compile with OTP/18. I would also like to be able to build this on a headless system without any GUI support installed (not even X11).

Good call. Perhaps the GUI part should be split out or even removed. What would you recommend?

I've moved it to the examples/ folder, lacking time to modernise the code.

This solution works fine for me, I'm not especially interested in the GUI terminal.


StageI Training Failure

I'm currently trying to train the network using pre-trained word embedding of bird and flowers. I'm using TensorFlow 1.3 so I did some necessary adaptions. But the result is frustrating:

It seems the generator is a completely mess. Also I checked some loss values and it's really weird that some of them are never changed:

I'm not sure what could be the cause of the problem, is it possible that some TensorFlow core function behave differently in r1.3 (as opposed to r0.12)?
Is there anyone who successfully reproduced the result with new version of TensorFlow? Can you send me a message to<EMAIL_ADDRESS>so I can check some implementation issue with you? Much appreciated guys!

I met the similar problem before, and I modify the condition augmentation function, and it works.

I'm currently trying to train the network using pre-trained word embedding of bird and flowers. I'm using TensorFlow 1.3 so I did some necessary adaptions. But the result is frustrating:

It seems the generator is a completely mess. Also I checked some loss values and it's really weird that some of them are never changed:

I'm not sure what could be the cause of the problem, is it possible that some TensorFlow core function behave differently in r1.3 (as opposed to r0.12)?
Is there anyone who successfully reproduced the result with new version of TensorFlow? Can you send me a message to<EMAIL_ADDRESS>so I can check some implementation issue with you? Much appreciated guys!

do you solve this problem? I also have it, please share the solution.

@Ereebay I meet the same problem, could you tell me how to modify the condition augmentation function?

@Ereebay @chenhuangsong I meet the same problem, and can't find the solution.
Could you help me, or send message to<EMAIL_ADDRESS>Thanks a lot.

I tried to implement this as well. I used
def conv_batch_normalization(inputs, name, epsilon=1e-5, in_dim=None, is_training=True, activation_fn=None, reuse=None):
    return tf.contrib.layers.batch_norm(inputs, decay=0.9, center=True, scale=True, epsilon=epsilon,
                                        activation_fn=activation_fn,
                                        param_initializers={'beta': tf.constant_initializer(0.),
                                                            'gamma': tf.random_normal_initializer(1., 0.02)},
                                        reuse=reuse, is_training=is_training, scope=name) ```

But this did not seem to solve the problem. The generator still produce weird results. Did you do guys do something similar to this?

You need to update the moving mean before the optimizer.
def prepare_trainer(self, generator_loss, discriminator_loss):
    '''Helper function for init_opt'''
    all_vars = tf.trainable_variables()

    g_vars = [var for var in all_vars if
              var.name.startswith('g_')]
    d_vars = [var for var in all_vars if
              var.name.startswith('d_')]

    update_ops_D = [var for var in tf.get_collection(tf.GraphKeys.UPDATE_OPS) if var.name.startswith('d_')]
    update_ops_G = [var for var in tf.get_collection(tf.GraphKeys.UPDATE_OPS) if var.name.startswith('g_')]
    with tf.control_dependencies(update_ops_G):
        generator_opt = tf.train.AdamOptimizer(self.generator_lr,
                                               beta1=0.5)
        self.generator_trainer = generator_opt.minimize(generator_loss, var_list=g_vars)
    with tf.control_dependencies(update_ops_D):
        discriminator_opt = tf.train.AdamOptimizer(self.discriminator_lr,
                                                   beta1=0.5)
        self.discriminator_trainer =discriminator_opt.minimize(discriminator_loss, var_list=d_vars)
    self.log_vars.append(("g_learning_rate", self.generator_lr))
    self.log_vars.append(("d_learning_rate", self.discriminator_lr))

You also need to save the global variables or save the trainable and moving mean and var of the batch normalization. You will use them to restore the stage I model to generate samples for stage II.

Thanks @Ereebay. According to the TensorFlow documentation, tf.contrib.layers.batch_norm has the option trainable that has a default argument set to True, which adds variables (moving mean and moving variance) to the graph collection.
I'm also assuming I need to update my prepare_trainer function in trainer for Stage II as well following what you posted above?

I'm not very sure whether it will add moving_mean into the trainable variable list. I recommend you make a variable list manually. And according to the docs, you need that code to update moving mean. Anyway, this code has been outdated for a long time, I recommend to implement it by the latest version of TF or Pytorch from scratch.

I checked and it was added to the global variables.
('d_net/d_embedd/fc/biases', [128])
('d_net/d_embedd/fc/weights', [1024, 128])
('d_net/d_n1.0/batch_norm/beta', [128])
('d_net/d_n1.0/batch_norm/gamma', [128])
('d_net/d_n1.0/batch_norm/moving_mean', [128])
('d_net/d_n1.0/batch_norm/moving_variance', [128])
('d_net/d_n1.0/batch_norm2/beta', [256])
('d_net/d_n1.0/batch_norm2/gamma', [256])
('d_net/d_n1.0/batch_norm2/moving_mean', [256])
('d_net/d_n1.0/batch_norm2/moving_variance', [256])
('d_net/d_n1.0/batch_norm3/beta', [512])
('d_net/d_n1.0/batch_norm3/gamma', [512])
('d_net/d_n1.0/batch_norm3/moving_mean', [512])
('d_net/d_n1.0/batch_norm3/moving_variance', [512])
('d_net/d_n1.0/conv2d/weights', [4, 4, 3, 64])
('d_net/d_n1.0/conv2d2/weights', [4, 4, 64, 128])
('d_net/d_n1.0/conv2d3/weights', [4, 4, 128, 256])
('d_net/d_n1.0/conv2d4/weights', [4, 4, 256, 512])
('d_net/d_n1.1/batch_norm/beta', [128])
('d_net/d_n1.1/batch_norm/gamma', [128])
('d_net/d_n1.1/batch_norm/moving_mean', [128])
('d_net/d_n1.1/batch_norm/moving_variance', [128])
('d_net/d_n1.1/batch_norm2/beta', [128])
('d_net/d_n1.1/batch_norm2/gamma', [128])
('d_net/d_n1.1/batch_norm2/moving_mean', [128])
('d_net/d_n1.1/batch_norm2/moving_variance', [128])
('d_net/d_n1.1/batch_norm3/beta', [512])
('d_net/d_n1.1/batch_norm3/gamma', [512])
('d_net/d_n1.1/batch_norm3/moving_mean', [512])
('d_net/d_n1.1/batch_norm3/moving_variance', [512])
('d_net/d_n1.1/conv2d/weights', [1, 1, 512, 128])
('d_net/d_n1.1/conv2d2/weights', [3, 3, 128, 128])
('d_net/d_n1.1/conv2d3/weights', [3, 3, 128, 512])
('d_net/d_net/d_embedd/fc/biases/Adam', [128])
('d_net/d_net/d_embedd/fc/biases/Adam_1', [128])
('d_net/d_net/d_embedd/fc/weights/Adam', [1024, 128])
('d_net/d_net/d_embedd/fc/weights/Adam_1', [1024, 128])
('d_net/d_net/d_n1.0/batch_norm/beta/Adam', [128])
('d_net/d_net/d_n1.0/batch_norm/beta/Adam_1', [128])
('d_net/d_net/d_n1.0/batch_norm/gamma/Adam', [128])
('d_net/d_net/d_n1.0/batch_norm/gamma/Adam_1', [128])
('d_net/d_net/d_n1.0/batch_norm2/beta/Adam', [256])
('d_net/d_net/d_n1.0/batch_norm2/beta/Adam_1', [256])
('d_net/d_net/d_n1.0/batch_norm2/gamma/Adam', [256])
('d_net/d_net/d_n1.0/batch_norm2/gamma/Adam_1', [256])
('d_net/d_net/d_n1.0/batch_norm3/beta/Adam', [512])
('d_net/d_net/d_n1.0/batch_norm3/beta/Adam_1', [512])
('d_net/d_net/d_n1.0/batch_norm3/gamma/Adam', [512])
('d_net/d_net/d_n1.0/batch_norm3/gamma/Adam_1', [512])
('d_net/d_net/d_n1.0/conv2d/weights/Adam', [4, 4, 3, 64])
('d_net/d_net/d_n1.0/conv2d/weights/Adam_1', [4, 4, 3, 64])
('d_net/d_net/d_n1.0/conv2d2/weights/Adam', [4, 4, 64, 128])
('d_net/d_net/d_n1.0/conv2d2/weights/Adam_1', [4, 4, 64, 128])
('d_net/d_net/d_n1.0/conv2d3/weights/Adam', [4, 4, 128, 256])
('d_net/d_net/d_n1.0/conv2d3/weights/Adam_1', [4, 4, 128, 256])
('d_net/d_net/d_n1.0/conv2d4/weights/Adam', [4, 4, 256, 512])
('d_net/d_net/d_n1.0/conv2d4/weights/Adam_1', [4, 4, 256, 512])
('d_net/d_net/d_n1.1/batch_norm/beta/Adam', [128])
('d_net/d_net/d_n1.1/batch_norm/beta/Adam_1', [128])
('d_net/d_net/d_n1.1/batch_norm/gamma/Adam', [128])
('d_net/d_net/d_n1.1/batch_norm/gamma/Adam_1', [128])
('d_net/d_net/d_n1.1/batch_norm2/beta/Adam', [128])
('d_net/d_net/d_n1.1/batch_norm2/beta/Adam_1', [128])
('d_net/d_net/d_n1.1/batch_norm2/gamma/Adam', [128])
('d_net/d_net/d_n1.1/batch_norm2/gamma/Adam_1', [128])
('d_net/d_net/d_n1.1/batch_norm3/beta/Adam', [512])
('d_net/d_net/d_n1.1/batch_norm3/beta/Adam_1', [512])
('d_net/d_net/d_n1.1/batch_norm3/gamma/Adam', [512])
('d_net/d_net/d_n1.1/batch_norm3/gamma/Adam_1', [512])
('d_net/d_net/d_n1.1/conv2d/weights/Adam', [1, 1, 512, 128])
('d_net/d_net/d_n1.1/conv2d/weights/Adam_1', [1, 1, 512, 128])
('d_net/d_net/d_n1.1/conv2d2/weights/Adam', [3, 3, 128, 128])
('d_net/d_net/d_n1.1/conv2d2/weights/Adam_1', [3, 3, 128, 128])
('d_net/d_net/d_n1.1/conv2d3/weights/Adam', [3, 3, 128, 512])
('d_net/d_net/d_n1.1/conv2d3/weights/Adam_1', [3, 3, 128, 512])
('d_net/d_net/d_template/batch_norm/beta/Adam', [512])
('d_net/d_net/d_template/batch_norm/beta/Adam_1', [512])
('d_net/d_net/d_template/batch_norm/gamma/Adam', [512])
('d_net/d_net/d_template/batch_norm/gamma/Adam_1', [512])
('d_net/d_net/d_template/conv2d/weights/Adam', [1, 1, 640, 512])
('d_net/d_net/d_template/conv2d/weights/Adam_1', [1, 1, 640, 512])
('d_net/d_net/d_template/conv2d2/weights/Adam', [4, 4, 512, 1])
('d_net/d_net/d_template/conv2d2/weights/Adam_1', [4, 4, 512, 1])
('d_net/d_template/batch_norm/beta', [512])
('d_net/d_template/batch_norm/gamma', [512])
('d_net/d_template/batch_norm/moving_mean', [512])
('d_net/d_template/batch_norm/moving_variance', [512])
('d_net/d_template/conv2d/weights', [1, 1, 640, 512])
('d_net/d_template/conv2d2/weights', [4, 4, 512, 1])
('d_net/g_net/g_OT/batch_norm/beta/Adam', [256])
('d_net/g_net/g_OT/batch_norm/beta/Adam_1', [256])
('d_net/g_net/g_OT/batch_norm/gamma/Adam', [256])
('d_net/g_net/g_OT/batch_norm/gamma/Adam_1', [256])
('d_net/g_net/g_OT/batch_norm2/beta/Adam', [128])
('d_net/g_net/g_OT/batch_norm2/beta/Adam_1', [128])
('d_net/g_net/g_OT/batch_norm2/gamma/Adam', [128])
('d_net/g_net/g_OT/batch_norm2/gamma/Adam_1', [128])
('d_net/g_net/g_OT/conv2d/weights/Adam', [3, 3, 512, 256])
('d_net/g_net/g_OT/conv2d/weights/Adam_1', [3, 3, 512, 256])
('d_net/g_net/g_OT/conv2d2/weights/Adam', [3, 3, 256, 128])
('d_net/g_net/g_OT/conv2d2/weights/Adam_1', [3, 3, 256, 128])
('d_net/g_net/g_OT/conv2d3/weights/Adam', [3, 3, 128, 3])
('d_net/g_net/g_OT/conv2d3/weights/Adam_1', [3, 3, 128, 3])
('d_net/g_net/g_n1.0/batch_norm/beta/Adam', [16384])
('d_net/g_net/g_n1.0/batch_norm/beta/Adam_1', [16384])
('d_net/g_net/g_n1.0/batch_norm/gamma/Adam', [16384])
('d_net/g_net/g_n1.0/batch_norm/gamma/Adam_1', [16384])
('d_net/g_net/g_n1.0/fc/biases/Adam', [16384])
('d_net/g_net/g_n1.0/fc/biases/Adam_1', [16384])
('d_net/g_net/g_n1.0/fc/weights/Adam', [228, 16384])
('d_net/g_net/g_n1.0/fc/weights/Adam_1', [228, 16384])
('d_net/g_net/g_n1.1/batch_norm_1/beta/Adam', [256])
('d_net/g_net/g_n1.1/batch_norm_1/beta/Adam_1', [256])
('d_net/g_net/g_n1.1/batch_norm_1/gamma/Adam', [256])
('d_net/g_net/g_n1.1/batch_norm_1/gamma/Adam_1', [256])
('d_net/g_net/g_n1.1/batch_norm_2/beta/Adam', [256])
('d_net/g_net/g_n1.1/batch_norm_2/beta/Adam_1', [256])
('d_net/g_net/g_n1.1/batch_norm_2/gamma/Adam', [256])
('d_net/g_net/g_n1.1/batch_norm_2/gamma/Adam_1', [256])
('d_net/g_net/g_n1.1/batch_norm_3/beta/Adam', [1024])
('d_net/g_net/g_n1.1/batch_norm_3/beta/Adam_1', [1024])
('d_net/g_net/g_n1.1/batch_norm_3/gamma/Adam', [1024])
('d_net/g_net/g_n1.1/batch_norm_3/gamma/Adam_1', [1024])
('d_net/g_net/g_n1.1/conv2d/weights/Adam', [1, 1, 1024, 256])
('d_net/g_net/g_n1.1/conv2d/weights/Adam_1', [1, 1, 1024, 256])
('d_net/g_net/g_n1.1/conv2d2/weights/Adam', [3, 3, 256, 256])
('d_net/g_net/g_n1.1/conv2d2/weights/Adam_1', [3, 3, 256, 256])
('d_net/g_net/g_n1.1/conv2d3/weights/Adam', [3, 3, 256, 1024])
('d_net/g_net/g_n1.1/conv2d3/weights/Adam_1', [3, 3, 256, 1024])
('d_net/g_net/g_n2.0/batch_norm/beta/Adam', [512])
('d_net/g_net/g_n2.0/batch_norm/beta/Adam_1', [512])
('d_net/g_net/g_n2.0/batch_norm/gamma/Adam', [512])
('d_net/g_net/g_n2.0/batch_norm/gamma/Adam_1', [512])
('d_net/g_net/g_n2.0/conv2d/weights/Adam', [3, 3, 1024, 512])
('d_net/g_net/g_n2.0/conv2d/weights/Adam_1', [3, 3, 1024, 512])
('d_net/g_net/g_n2.1/batch_norm/beta/Adam', [128])
('d_net/g_net/g_n2.1/batch_norm/beta/Adam_1', [128])
('d_net/g_net/g_n2.1/batch_norm/gamma/Adam', [128])
('d_net/g_net/g_n2.1/batch_norm/gamma/Adam_1', [128])
('d_net/g_net/g_n2.1/batch_norm2/beta/Adam', [128])
('d_net/g_net/g_n2.1/batch_norm2/beta/Adam_1', [128])
('d_net/g_net/g_n2.1/batch_norm2/gamma/Adam', [128])
('d_net/g_net/g_n2.1/batch_norm2/gamma/Adam_1', [128])
('d_net/g_net/g_n2.1/batch_norm3/beta/Adam', [512])
('d_net/g_net/g_n2.1/batch_norm3/beta/Adam_1', [512])
('d_net/g_net/g_n2.1/batch_norm3/gamma/Adam', [512])
('d_net/g_net/g_n2.1/batch_norm3/gamma/Adam_1', [512])
('d_net/g_net/g_n2.1/conv2d/weights/Adam', [1, 1, 512, 128])
('d_net/g_net/g_n2.1/conv2d/weights/Adam_1', [1, 1, 512, 128])
('d_net/g_net/g_n2.1/conv2d2/weights/Adam', [3, 3, 128, 128])
('d_net/g_net/g_n2.1/conv2d2/weights/Adam_1', [3, 3, 128, 128])
('d_net/g_net/g_n2.1/conv2d3/weights/Adam', [3, 3, 128, 512])
('d_net/g_net/g_n2.1/conv2d3/weights/Adam_1', [3, 3, 128, 512])
('d_net/g_net/gen_cond/fc/biases/Adam', [256])
('d_net/g_net/gen_cond/fc/biases/Adam_1', [256])
('d_net/g_net/gen_cond/fc/weights/Adam', [1024, 256])
('d_net/g_net/gen_cond/fc/weights/Adam_1', [1024, 256])
('g_net/g_OT/batch_norm/beta', [256])
('g_net/g_OT/batch_norm/gamma', [256])
('g_net/g_OT/batch_norm/moving_mean', [256])
('g_net/g_OT/batch_norm/moving_variance', [256])
('g_net/g_OT/batch_norm2/beta', [128])
('g_net/g_OT/batch_norm2/gamma', [128])
('g_net/g_OT/batch_norm2/moving_mean', [128])
('g_net/g_OT/batch_norm2/moving_variance', [128])
('g_net/g_OT/conv2d/weights', [3, 3, 512, 256])
('g_net/g_OT/conv2d2/weights', [3, 3, 256, 128])
('g_net/g_OT/conv2d3/weights', [3, 3, 128, 3])
('g_net/g_n1.0/batch_norm/beta', [16384])
('g_net/g_n1.0/batch_norm/gamma', [16384])
('g_net/g_n1.0/batch_norm/moving_mean', [16384])
('g_net/g_n1.0/batch_norm/moving_variance', [16384])
('g_net/g_n1.0/fc/biases', [16384])
('g_net/g_n1.0/fc/weights', [228, 16384])
('g_net/g_n1.1/batch_norm_1/beta', [256])
('g_net/g_n1.1/batch_norm_1/gamma', [256])
('g_net/g_n1.1/batch_norm_1/moving_mean', [256])
('g_net/g_n1.1/batch_norm_1/moving_variance', [256])
('g_net/g_n1.1/batch_norm_2/beta', [256])
('g_net/g_n1.1/batch_norm_2/gamma', [256])
('g_net/g_n1.1/batch_norm_2/moving_mean', [256])
('g_net/g_n1.1/batch_norm_2/moving_variance', [256])
('g_net/g_n1.1/batch_norm_3/beta', [1024])
('g_net/g_n1.1/batch_norm_3/gamma', [1024])
('g_net/g_n1.1/batch_norm_3/moving_mean', [1024])
('g_net/g_n1.1/batch_norm_3/moving_variance', [1024])
('g_net/g_n1.1/conv2d/weights', [1, 1, 1024, 256])
('g_net/g_n1.1/conv2d2/weights', [3, 3, 256, 256])
('g_net/g_n1.1/conv2d3/weights', [3, 3, 256, 1024])
('g_net/g_n2.0/batch_norm/beta', [512])
('g_net/g_n2.0/batch_norm/gamma', [512])
('g_net/g_n2.0/batch_norm/moving_mean', [512])
('g_net/g_n2.0/batch_norm/moving_variance', [512])
('g_net/g_n2.0/conv2d/weights', [3, 3, 1024, 512])
('g_net/g_n2.1/batch_norm/beta', [128])
('g_net/g_n2.1/batch_norm/gamma', [128])
('g_net/g_n2.1/batch_norm/moving_mean', [128])
('g_net/g_n2.1/batch_norm/moving_variance', [128])
('g_net/g_n2.1/batch_norm2/beta', [128])
('g_net/g_n2.1/batch_norm2/gamma', [128])
('g_net/g_n2.1/batch_norm2/moving_mean', [128])
('g_net/g_n2.1/batch_norm2/moving_variance', [128])
('g_net/g_n2.1/batch_norm3/beta', [512])
('g_net/g_n2.1/batch_norm3/gamma', [512])
('g_net/g_n2.1/batch_norm3/moving_mean', [512])
('g_net/g_n2.1/batch_norm3/moving_variance', [512])
('g_net/g_n2.1/conv2d/weights', [1, 1, 512, 128])
('g_net/g_n2.1/conv2d2/weights', [3, 3, 128, 128])
('g_net/g_n2.1/conv2d3/weights', [3, 3, 128, 512])
('g_net/gen_cond/fc/biases', [256])
('g_net/gen_cond/fc/weights', [1024, 256])

Appreciate the help. Thank you


Gambian mobile validation

Dears,
Kindly note that we want to accept the Gambian mobile numbers starting with "4".
We are trying to enter the Gambian mobile number: "4121212", but the functions "phoneUtil.isValidNumber(phoneNumber)" is returning false and the function "phoneUtil.getNumberType(phoneNumber)" is returning UNKNOWN.
I am using the version: libPhoneNumber-iOS (1.2.0)
Kindly advise.

Kind reminder


Fixed IBM call to use api key.

Fixes issue #391 by using the IBM api key instead of discontinued username and password.

The only test failing seems to be due to Python 3.3 no longer being available via the mirror, which isn't something I can fix short of modifying the test runner.
"Unable to download 3.3 archive. The archive may not exist. Please consider a different version."

As a random person from the internet I can say that I tried this fix and it worked for me (Python 3.7)

May I suggest you add  'url' as parameter to "recognize_ibm".
It should be defaulted to "https://stream.watsonplatform.net/speech-to-text/api" but can be updated to local service definition i.e. "https://stream-fra.watsonplatform.net/speech-to-text/api" with '-fra' to indicate Frankfurt...

Can this be merged as currently the recognize_ibm is effectively unusable with speech_recognition - @Uberi ?

Thanks for the patch @chrisspen, and sorry for the delay y'all! Merging now

Awesome, thank you!!


Setting up quicklisp without interactive input

Hi there, I am using sbcl and would like to run the quicklisp setup using the instructions on the quicklisp site https://www.quicklisp.org/beta/#installation or the very similar ones here: https://lisp-lang.org/learn/getting-started/
However, I'd like to be able to do this using a script which runs automatically on a test machine where there is no possibility for interactive input to the script.
This fails because ql:add-to-init-file waits for 'Enter' to go ahead and write to the sbcl config file.
Is there some way I can workaround this currently? If not, could the function be changed so that optionally it doesn't need Enter?

See without-prompting and *do-not-prompt* for options to control
prompting behavior.
You also need not use ql:add-to-init-file. It's not a mandatory part of
the installation.
On Sat, Apr 24, 2021 at 6:41 PM jwg4 @.***> wrote:

Hi there, I am using sbcl and would like to run the quicklisp setup using
the instructions on the quicklisp site
https://www.quicklisp.org/beta/#installation or the very similar ones
here: https://lisp-lang.org/learn/getting-started/
However, I'd like to be able to do this using a script which runs
automatically on a test machine where there is no possibility for
interactive input to the script.
This fails because ql:add-to-init-file waits for 'Enter' to go ahead and
write to the sbcl config file.
Is there some way I can workaround this currently? If not, could the
function be changed so that optionally it doesn't need Enter?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/quicklisp/quicklisp-bootstrap/issues/23, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AACPNLKG4F7RWKMZW62VMSTTKNCIPANCNFSM43QS5IYA
.

Worked, thank you very much for the speedy and helpful reply!


Release fixes and bumps

@andreriesco as I need these changes to make some tests this weekend I will approve and merge it as is. But let's discuss these points that I commented monday.


Android - Captcha doesn't hide on back press - Reported by @adeel0202

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:

Go to Payments > Add payment method > Bank account
Select any bank
Tap on the captcha checkbox
When the captcha displays, press back button

Expected Result:
The captcha hides when the back button is pressed
Actual Result:
The captcha is still visible
Workaround:
Unknown
Platform:

Where is this issue occurring?

Android

Version Number:
Reproducible in staging?: need repro (Not able to trigger CAPTCHA)
Reproducible in production?: need repro
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/43995119/191659149-52e8abb1-4489-4009-87bf-b2f379a8752f.mp4
Expensify/Expensify Issue URL:
Issue reported by: @adeel0202
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1661968966210629
View all open jobs on GitHub

I tried to replicate this but I don't get a captcha code. @adeel0202 could you please let me know how you triggered the captcha?

@kadiealexander I don't think there are any specific steps to trigger the captcha. I too get to see the captcha sometimes, and the other times, it just bypasses the captcha. You can try to reproduce it with different accounts and also by connecting to a VPN.

Asking for help to replicate this issue here.

hello @kadiealexander yes this issue exists on back click captcha doesn't hide, but I highly doubt to solve in our code base since this is inside plaid SDK
https://user-images.githubusercontent.com/47522946/192691144-1139f2d5-6db4-458c-b0c2-bf043541426b.mp4

Closing as this is an issue with Plaid as mentioned above.


Add a note about using metrics, params and plot without pipelines

My use case involves the inability to really use pipelines. I have models and data, but I am not (or cannot) do the training. So I don't have experiments per se. Still, I have metrics or other metadata I would want to track.
Apparently I can do this using commands like dvc metrics diff --targets metrics.json. I think this could be highlighted in the documentation that it is not required to have a dvc.yaml file. Although I would prefer that the cmd argument would be optional.
I can do a PR with guidance about where I should put this if wanted. I thought about putting this in a use case (versioning) or in the user guide.
Reference to this proposal is here . cc @jorgeorpinel

Although I would prefer that the cmd argument would be optional.

cc @skshetry

prefer that the cmd argument would be optional

Not sure having stages without an action would make sense. But how about accepting params, metrics, and plots in regular .dvc files? They already accept deps and outs which are the parents of the aforementioned fields.

where I should put this if wanted

The command reference would be the first place that comes to mind.
Already mentioned in:

https://dvc.org/doc/command-reference/params/diff
https://dvc.org/doc/command-reference/metrics/diff
https://dvc.org/doc/command-reference/plots/show
https://dvc.org/doc/command-reference/plots/diff

Not even mentioned yet in:

https://dvc.org/doc/command-reference/metrics/show

Not supported in:

https://dvc.org/doc/command-reference/plots/modify

One strategy would be to make sure the mentions are emphatic enough e.g. instead of parenthesis or grayed out notes, make a paragraph (starting with the 💡 emoji perhaps). Also review that it's mentioned in the --targets options for commands that have those.
Another idea would be to just leave minimal mentions (double check they're consistent), but emphasize in the Options section of each ref: for targets argument create an entry that fully describes the behavior and accepted values, like in https://dvc.org/doc/command-reference/repro#options. For --targets options similarly expand as much as needed.

prefer that the cmd argument would be optional

Not sure having stages without an action would make sense. But how about accepting params, metrics, and plots in regular .dvc files? They already accept deps and outs which are the parents of the aforementioned fields.

But the value of the main yaml is that the information is centralised. If you just want to give general metadata, there should be an easy way to do so without needing to have stages or anything else. A top level option in the yaml would be better IMO.

One strategy would be to make sure the mentions are emphatic enough e.g. instead of parenthesis or grayed out notes, make a paragraph (starting with the 💡 emoji perhaps). Also review that it's mentioned in the --targets options for commands that have those.
Another idea would be to just leave minimal mentions (double check they're consistent), but emphasize in the Options section of each ref: for targets argument create an entry that fully describes the behavior and accepted values, like in https://dvc.org/doc/command-reference/repro#options. For --targets options similarly expand as much as needed.

Reading at the metrics one for instance, I think the explanation is OK. What I was missing is the fact that I could use all these function without the need to have the yaml file. So in Project structure that could maybe be highlighted. Currently it is saying:
dvc.yaml pipelines files define stages that form the pipeline(s) of a project. All stage-based features such as dvc params, dvc metrics, and dvc plots are specified here.

Which can lead to the conclusion that you have to use this yaml in order to use these functionalities.

If you just want to give general metadata, there should be an easy way to do so without needing to have stages or anything else. A top level option in the yaml would be better IMO.

There's a separate discussion around that. See https://github.com/iterative/dvc/issues/3936#issuecomment-692470880
For now file metadata needs it's own .dvc file so I think that's where params/metrics support could be added. If/when tracking data gets incorporated to dvc.yaml that would carry over too.

BTW that part of the discussion should be in the iterative/dvc repo to be considered. Let's see if @skshetry has no objection on operning that issue (thanks).

the metrics one for instance, I think the explanation is OK

Which one? It's not even mentioned in https://dvc.org/doc/command-reference/metrics or https://dvc.org/doc/command-reference/metrics/show.

in Project structure that could maybe be highlighted

Feel free to send a proposed change (via a PR). Thanks

the metrics one for instance, I think the explanation is OK

Which one? It's not even mentioned in https://dvc.org/doc/command-reference/metrics or https://dvc.org/doc/command-reference/metrics/show.

The one for diff. But true it is missing from the general metric discussion.

in Project structure that could maybe be highlighted

Feel free to send a proposed change (via a PR). Thanks

Ok I will try to propose something 😃

Btw, .dvc file does support specifying metrics, params and plots. This is a pre-1.0 compatibility feature though.
We have also plans for supports different sections in dvc.yaml file. Eg:
metrics:
  - metrics.json
stages:
  ...

Merged into #2815.

#2572, rather.


Byte[] ToTealUlong Implementation

Hi,
The following aren't defined, making it impossible to convert to ulong from byte[], or even from string to ulong.
Byte[] ToTealUlong

https://github.com/FrankSzendzielarz/AlgorandVisualStudio/blob/main/ContractDevelopment/PredefinedFunctions.md

There's no method to convert a byte string to a number either.
This change should be compliant with ARC4 ABI, which is just a specification for how to handle methods, more complex types and so forth. In PyTeal, it's just btoi.

https://pyteal.readthedocs.io/en/stable/api.html#pyteal.Btoi
https://arc.algorand.foundation/ARCs/arc-0004

So you could implement this as a simple text-replacement for ToTealUlong with the opcode btoi.  It seems very important to have simple casting of types available.
For now, I will work around it by simply including the arguments pre-expanded instead of being byte-packed into a single array. I imagine there'll be a scenario where it's not possible.
Kind regards.

These are actually defined. Are you sure the nuget package is the latest?

Hi again, not the VSIX but thr Nuget Packages under Dependencies / Packages in the Solution Explorer. The VSIX is out of phase with the compiler NuGet (AlgoStudio)


Fix for electron 3.x.x (Discord Canary) + Fix TLS certificate errors + Add installer logging

The new BrowserWindow object in 3.0.0+ has checks now to see if it is a BrowserWindow using this function:
const isBrowserWindow = (win) => {
  return win && win.constructor.name === 'BrowserWindow'
}

So this patch just ensures the object is still named properly. Without this it has several errors on startup, ED fails to load, and devtools cannot be opened.
Note: This of course will require a rebuild of the installer and reupload to the site

Added a fix for TLS/SSL certificate errors that was more common in Windows 7. Also added logging (both informational and errors) to the installer.


[main] [6368:0130/195648.904:ERROR:CONSOLE(1)] "SyntaxError: Unexpected end of JSON input", source: devtools://devtools/bundled/entrypoints/main/main.js (1)

Describe the bug
When i open devtools i have this message
371 modules transformed.
dist/index.cjs   4.10 KiB
dist/index.cjs.map 9.67 KiB
built in 3359ms.
19:56:48 [main] [6368:0130/195648.904:ERROR:CONSOLE(1)] "SyntaxError: Unexpected end of JSON input", source: devtools://devtools/bundled/entrypoints/main/main.js (1)

To Reproduce
Steps to reproduce the behavior:

run node scripts/watch.js
check console
See error

Provide a sample project with reproduction. I do not see this problem following your steps on the last commit.

Provide a sample project with reproduction. I do not see this problem following your steps on the last commit.

This is your version of it.  The latest version of you. I made a clean git clone just in case.

C:\Users\lord\Desktop\test\vite-electron-builder-main>npm i
npm WARN read-shrinkwrap This version of npm is compatible with lockfileVersion@1, but package-lock.json was generated for lockfileVersion@2. I'll try to do my best with it!

><EMAIL_ADDRESS>install C:\Users\lord\Desktop\test\vite-electron-builder-main\node_modules\playwright
> node install.js

><EMAIL_ADDRESS>postinstall C:\Users\lord\Desktop\test\vite-electron-builder-main\node_modules\esbuild
> node install.js

><EMAIL_ADDRESS>postinstall C:\Users\lord\Desktop\test\vite-electron-builder-main\node_modules\electron
> node install.js

><EMAIL_ADDRESS>postinstall C:\Users\lord\Desktop\test\vite-electron-builder-main\node_modules\simple-git-hooks
> node ./postinstall.js

[ERROR] Was not able to set git hooks. Reason: Error: ENOENT: no such file or directory, mkdir 'undefined\hooks\'
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\dmg-license):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\esbuild-windows-arm64):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"win32","arch":"arm64"} (current: {"os":"win32","arch":"x64"})
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\esbuild-windows-32):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"win32","arch":"ia32"} (current: {"os":"win32","arch":"x64"})
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\esbuild-sunos-64):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"sunos","arch":"x64"} (current: {"os":"win32","arch":"x64"})
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\esbuild-openbsd-64):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"openbsd","arch":"x64"} (current: {"os":"win32","arch":"x64"})
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\esbuild-linux-ppc64le):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"linux","arch":"ppc64"} (current: {"os":"win32","arch":"x64"})
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\esbuild-netbsd-64):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"netbsd","arch":"x64"} (current: {"os":"win32","arch":"x64"})
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\esbuild-linux-mips64le):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"linux","arch":"mips64el"} (current: {"os":"win32","arch":"x64"})
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\esbuild-linux-arm64):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"linux","arch":"arm64"} (current: {"os":"win32","arch":"x64"})
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\esbuild-linux-64):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"linux","arch":"x64"} (current: {"os":"win32","arch":"x64"})
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\esbuild-linux-32):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"linux","arch":"ia32"} (current: {"os":"win32","arch":"x64"})
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\esbuild-linux-arm):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"linux","arch":"arm"} (current: {"os":"win32","arch":"x64"})
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\esbuild-freebsd-arm64):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"freebsd","arch":"arm64"} (current: {"os":"win32","arch":"x64"})
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\esbuild-freebsd-64):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"freebsd","arch":"x64"} (current: {"os":"win32","arch":"x64"})
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\esbuild-darwin-arm64):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"arm64"} (current: {"os":"win32","arch":"x64"})
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\esbuild-darwin-64):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"x64"} (current: {"os":"win32","arch":"x64"})
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\esbuild-android-arm64):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"android","arch":"arm64"} (current: {"os":"win32","arch":"x64"})

added 595 packages from 418 contributors and audited 631 packages in 57.75s

61 packages are looking for funding
  run `npm fund` for details

found 3 vulnerabilities (2 moderate, 1 high)
  run `npm audit fix` to fix them, or `npm audit` for details

C:\Users\lord\Desktop\test\vite-electron-builder-main>node scripts/watch.js
Pre-bundling dependencies:
  vue
(this will be run only when your dependencies or config have changed)
vite v2.7.13 building for development...

watching for file changes...

build started...
transforming (1) src\index.tsvite v2.7.13 building for development...

watching for file changes...

build started...
✓ 2 modules transformed.
dist/index.cjs   0.31 KiB
dist/index.cjs.map 1.41 KiB
built in 2268ms.
✓ 371 modules transformed.
dist/index.cjs   4.10 KiB
dist/index.cjs.map 9.67 KiB
built in 5624ms.
01:16:01 [main] [17212:0202/011601.589:ERROR:CONSOLE(1)] "SyntaxError: Unexpected end of JSON input", source: devtools://devtools/bundled/entrypoints/main/main.js (1)
01:16:01 [main] [17212:0202/011601.782:ERROR:CONSOLE(1)] "Uncaught TypeError: Cannot read properties of undefined (reading 'instance')", source: devtools://devtools/bundled/devtools_app.html?remoteBase=https://chrome-devtools-frontend.appspot.com/serve_file/@2bee2cb92d553708966882df183bb533ac105e7c/&can_dock=true&toolbarColor=rgba(223,223,223,1)&textColor=rgba(0,0,0,1)&experiments=true (1)

I noticed that this error occurs every time I open the developer console

Looking at https://github.com/electron/electron/issues/32699 there is a certain bug, to remove the error it is enough to specify the mode, for example: browserWindow?.webContents.openDevTools({ mode: "detach" });
Interestingly, when specifying other modes, nothing changes, but errors appear. It looks like this is a consequence of this error https://github.com/electron/electron/issues/32702

Looking at electron/electron#32699 there is a certain bug, to remove the error it is enough to specify the mode, for example: browserWindow?.webContents.openDevTools({ mode: "detach" }); Interestingly, when specifying other modes, nothing changes, but errors appear. It looks like this is a consequence of this error electron/electron#32702

The error still appears on the console now in a separate window

The error still appears on the console now in a separate window

And in truth, when you re-open the console, this error pops up.
I think you can ignore this, since it does not affect the work, and the error is already known and will be fixed soon.


string indices must be intergers

sub_graph = interactive.subgraph("g.V().has('year', inside(2014, 2020)).outE('cites')")
当我运行上行代码时出现错误‘’string indices must be intergers ‘’
请问如何解决

[ ] #

Hi @banbsyip,
Thanks for reporting the issue! Could you please provide more details about the setting ? e.g., the version of graphscope that you are working with.

@sighingnow  the data down from the ogbn-mag,the version of graphscope is 0.8,i think it maybe caused by the property default set when I run the add_vertices()

@sighingnow when I add properties=[('year','int')] to the paper's line,it still this error.

@sighingnow the data down from the ogbn-mag,the version of graphscope is 0.8,i think it maybe caused by the property default set when I run the add_vertices()

Hi, this error is because the error format returned by the interactive engine does not conform to the parsing format of Gremlin-Python. Can you upgrade to the 0.10.0 image and try again on the client side? This problem may be solved in the new version

@sighingnow when I add properties=[('year','int')] to the paper's line,it still this error.

Have you tried to upgrade your gremlin_python to the latest version? that might be helpful.

@sighingnow the data down from the ogbn-mag,the version of graphscope is 0.8,i think it maybe caused by the property default set when I run the add_vertices()

Hi, this error is because the error format returned by the interactive engine does not conform to the parsing format of Gremlin-Python. Can you upgrade to the 0.10.0 image and try again on the client side? This problem may be solved in the new version

after i  upgrade to the 0.10.0 image，the problem has resolved, but another problem occured,
the code is : ret1 = graphscope.kcore(simple_g, k=5)
AttributeError: module 'graphscope' has no attribute 'kcore'

Happy to know that~ I'll close this issue for now, and if you have any other questions, feel free to reopen this or open another thread~

@sighingnow the data down from the ogbn-mag,the version of graphscope is 0.8,i think it maybe caused by the property default set when I run the add_vertices()

Hi, this error is because the error format returned by the interactive engine does not conform to the parsing format of Gremlin-Python. Can you upgrade to the 0.10.0 image and try again on the client side? This problem may be solved in the new version

please change the code  'ret1 = graphscope.kcore(simple_g, k=5)' to 'ret1 = graphscope.k_core(simple_g, k=5)' on the https://graphscope.io/docs/zh/getting_started.html if the website managed by yours

Thank you @banbsyip.
@lidongze0629 could you please fix the kcore related typos in documentation in your PR? Thanks!


How to set ssh port when using multiple ssh-able machines for training

I got 2 ssh-able machines for training. but the ssh port isn't 22.  how to set the port ?   tried to add the port in the hosts file like below, didn't work.
-P port1 IP1
-P port2 IP2

hostfile should be like the form ip:port


[Beta] Video doesn't load on Applied Responsive Web Design Projects

Challenge build-a-survey-form has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/55.0.2883.95 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.

Minified React error in console shows:

Unable to find element with ID 52.

I'm guessing this is related to #12656? Doesn't have to be though!
I can't reproduce this locally either, but I do get an https/http error.

We're removing all project videos from our platform and implementing a multi-file editor so campers don't have to use CodePen any more. Please see https://github.com/freeCodeCamp/freeCodeCamp/issues/12753


Add --case-sensitive and allow --in 0 meaning actual prefix

I'm not quite sure about the tweak to the prediction computation, but the rest works!

Exactly what I needed. Thanks!


[Event Request] codeunit 4470 "Record Link Impl." - OnBeforeResetNotifyOnLinks

Can you add the following event to ResetNotifyOnLinks to be able to skip the reset.
I can see that there is already an event after the reset procedure is ran, but then I have to delete and copy them again, and I don't want to do that as the PK uses AutoIncrement.
    local procedure ResetNotifyOnLinks(RecVariant: Variant)
    var
        RecordLink: Record "Record Link";
        RecordRef: RecordRef;
        // >>>>>>>>>>>>
        IsHandled: Boolean;
        // <<<<<<<<<<<<
    begin
        // >>>>>>>>>>>>
        RecordLinkManagement.OnBeforeResetNotifyOnLinks(RecVariant, IsHandled);
        if IsHandled then
            exit;
        // <<<<<<<<<<<<
        RecordRef.GetTable(RecVariant);
        RecordLink.SetRange("Record ID", RecordRef.RecordId());
        RecordLink.SetRange(Notify, true);
        if not RecordLink.IsEmpty() then
            RecordLink.ModifyAll(Notify, false);
    end;

@auzhhv Since ResetNotifyOnLinks is referenced only in CopyLinks how about a symmetrical event OnBeforeCopyLinks(..., SkipReset)?
Moreover, since the code is out here in the repo, how do you feel about implementing it yourself? Should be easy and definitely faster, and I will be here to help if needed!

@auzhhv Since ResetNotifyOnLinks is referenced only in CopyLinks how about a symmetrical event OnBeforeCopyLinks(..., SkipReset)?
Moreover, since the code is out here in the repo, how do you feel about implementing it yourself? Should be easy and definitely faster, and I will be here to help if needed!

That would work. How do I do that? Just make a pull request with the changes?

Yep, a PR in this repo would be great! Looking forward!

This one was actually processed 13-10-2022. Closing issue.


Providers

Can I please know the providers that you guys frequently use ?( So that I can maintain them)

fix movies and sports

Hi Poseidon444, I usually use  sflix and solar, Thanks for everything

English providers: Remote stream, sflix
And the Sports providers

Hi Poseidon444, I usually use sflix and solar, Thanks for everything

^ + remotestream

^ + remotestream

RemoteStream doesn't exist anymore.

aw dang

I don't know what the source is but supernova.to is pretty great

If you want to recommend a website please to it at: mov-cli-films for film sites and mov-cli-anime for anime sites

done.


Updates!

Added support for Python 3 and standard issue PyQt5 (as opposed to PySide)
A few browser settings were sacrificed during the inevitable transition from QtWebKit to QtWebEngine so YMMV.

Excellent, thanks for making these changes.
If I add you as a collaborator and give you publishing rights on pypi, would you be happy to publish the latest version?

We could also publish the new version under the name flask-desktop rather than WebUI, the old name.
What do you think?

Hey @Widdershin, thanks for inviting me to collaborate!
Renaming it to flask-desktop sounds good to me. Perhaps in the future there could be a mobile/touch version as well.

Cool idea :)
I made a project called positron that turns JavaScript apps into Android
apps, so if there was a way of bundling and running Python on Android it
would be viable.
By the way, would you be willing to publish the new package as
flask-desktop? I don't have a Python dev environment set up right now, so I
would really appreciate it.
On Wed, May 17, 2017 at 11:06 AM, Tarocco<EMAIL_ADDRESS>wrote:

Hey @Widdershin https://github.com/widdershin, thanks for inviting me
to collaborate!
Renaming it to flask-desktop sounds good to me. Perhaps in the future
there could be a mobile/touch version as well.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Widdershin/flask-desktop/pull/3#issuecomment-301939753,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAYUHfqsszpmMCSgTe7lsBwJyTEOcxYkks5r6ivYgaJpZM4Nb71H
.

I can refactor webui/WebUI to flask-desktop/Flask-Desktop or alternatively flask-desktop/FlaskDesktop

Sounds good. Does PEP8 recommend a particular style? I haven't done Python
in a while so I can't remember.
If PEP8 has a preference that would be my bet, otherwise I'm happy with
either!
I'm happy to review your refactor :)
On Thu, May 18, 2017 at 5:15 AM, Tarocco<EMAIL_ADDRESS>wrote:

I can refactor webui/WebUI to flask-desktop/Flask-Desktop or alternatively
flask-desktop/FlaskDesktop
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Widdershin/flask-desktop/pull/3#issuecomment-302160206,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAYUHYfqvDpw0nd7YY5cvxnVC0EdKQzmks5r6ysvgaJpZM4Nb71H
.

I can focus on this sometime. I'm second-guessing myself trying to get freezing to work in Python 3.4 and above
Python 3.x + PyQt5 + py2exe/pyinstaller/cx-Freeze
I've had a bunch of trouble with it so in the meantime I haven't been focusing on it a lot at the moment

Great ☺
I appreciate your contributions. Just remember that you're not obligated to
contribute, so you should never worry about doing other things. Just do
what you want ☺
On Fri, May 26, 2017 at 5:50 PM, Tarocco<EMAIL_ADDRESS>wrote:

I can focus on this sometime. I'm second-guessing myself trying to get
freezing to work in Python 3.4 and above
Python 3.x + PyQt5 + py2exe/pyinstaller/cx-Freeze
https://stackoverflow.com/questions/44033069/python-3-x-pyqt5-py2exe-pyinstaller-cx-freeze
I've had a bunch of trouble with it so in the meantime I haven't been
focusing on it a lot at the moment
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Widdershin/flask-desktop/pull/3#issuecomment-304197891,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAYUHaNh7PLghYi1C46YGUIseYdNnBwjks5r9mgmgaJpZM4Nb71H
.


AtlasTexture Region and Margin ignored for Line2D

Godot version:
v3.2.3.stable.mono.official
OS/device including version:
Microsoft Windows 10 Pro, version 10.0.18363 Build 18363
Issue description:
It's all in the title: Line2D nodes don't make use of AtlasTexture Region and Margin properties and instead use the full texture.
This makes using them with sprite sheets a hassle as you can't specify what rect is visible.
It seems that this issue has existed for at least a couple years now, judging from this Q&A page.
Steps to reproduce:

Create a Line2D node in the editor and add 2 or more points
Set its Texture to an AtlasTexture
Modify the Region and/or Margin properties; note how nothing changes on the line

Minimal reproduction project:
AtlasBug.zip

This issue has the same cause as the following areas of the engine:

Polygon2D (https://github.com/godotengine/godot/issues/25049)
MeshInstance and MultiMeshInstance (https://github.com/godotengine/godot-proposals/issues/1708)
Shaders (https://github.com/godotengine/godot/issues/25895)
NinePatchRect (under certain configurations)

And the reason is simple: AtlasTexture is NOT a texture. It inherits Texture but that's only a facade. It barely just holds a reference to a real texture, with a rectangle, and just so happen to work if used on some simple 2D nodes because these nodes happen to generate their simple quads with special UVs.
For anything more complex, it would be a shitshow of geometry workarounds to implement throughout the engine. Basically, it was never made to work together with other types of nodes. It's meant only for nodes that render simple quads.
I believe if we want to solve this, either we have to:

Make it clear that this kind of "texture" can only be used with certain 2D nodes, and that it's not a bug, but an expected limitation
Or make it so this kind of texture is handled at rendering engine level in such a way the sampler2D would be bound to an image region (instead of the whole image), which would make things work naturally without any UV hacks, even with repeat or clamp flags on (does Vulkan allow to do this now?).

@Zylann AtlasTexture is indeed a Texture in users point of view ; how it is implemented should match this expectation. In fact, the only need of the whole source image is only for region capture, after that capture the whole awaited behavior of AtlasTexture is that one of a regular basic texture. It is now clearly a mismatching implementation problem, as this hard behavior link is now commonly recognized by users as true and ancored in usages exoectations.


Missing macOS versions

I don't understand why yt-dlp versions are provided for macOS but not ffmpeg builds.

Coz I don't know how to make them. If you do, feel free to open a PR

The build pipeline uses crosstool-ng, which doesn't support Windows on ARM and macOS.
macOS doesn't support cross-compilation at all, which means that the entire build pipeline must be created from scratch specifically for macOS. No one wants to do that, for obvious reasons. Quoting the author of the build scripts https://github.com/BtbN/FFmpeg-Builds/pull/294#issuecomment-1848398337:

To build for OSX itself, we'd need a properly working cross compiler, and Apple is doing its utmost to prevent that from happening.
I also would have no way to test anything (beyond compiling itself) without spending way too much money on at least two Apple devices.

If you want macOS support, you'd better off figuring out how to make e.g. Homebrew apply the single FFmpeg patch that this repository currently has instead of trying to port the entire pipeline.


Use gcloud auto generated version

Now that gcloud's deploy output includes the version ID of the deployment, we can use this instead of generating our own. (see internal #120830848)

Closed by #660.

Note: the new gcloud is now out. If users update gcloud, they will see the new JSON format.


ShowRizz (a shows app that displays a list of shows and their details. It allows users to view their details, and book tickets for the show)

🛠 Issue (Number)

Closes #850
Description
ShowRizz ,a shows app that displays a list of shows and their details. It allows users to view their details, and book tickets for the show.
Type of change

[ x] New feature (non-breaking change which adds functionality)

Checklist:

[ x] I have made this from my own
[x ] My code follows the style guidelines of this project
[x ] I have performed a self-review of my own code
[x ] I have commented my code, particularly in hard-to-understand areas
[x ] I have made corresponding changes to the documentation
[x ] My changes generate no new warnings
[ x] The title of my pull request is a short description of the requested changes.

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

Home page

Show details page

Book ticket page

About page

@TusharKesarwani , please help me out , why is my pr rejected ?


Update README.md

Wanted to add something a little more to our current README and what we should expect in the future. This is apart of #424, but is still a work in progress.

looking good mate 👍


Re-add affafj to ami-iit.yml

It was removed in https://github.com/icub-tech-iit/outside-collaborators/pull/180#issuecomment-1602258877 after its username changed affafjunaidMOMIN from affafj .
fyi @affafj @DanielePucci @GabrieleNava

"fl-ferr" does not exist ❌

cc @traversaro

Probably, fl-ferr is now @flferretti.
I'll comply to this new username.

I think the new username of fl-ferr is @flferretti Link to his profile

@pattacini @affafj Thanks!

Probably, fl-ferr is now @flferretti. I'll comply to this new username.

Thanks @pattacini, that is true, thanks for handling that.


demo_launcher scrolling slow on Galaxy S5

 Issue by spierce7
Sunday Aug 02, 2015 at 06:44 GMT
Originally opened as https://github.com/flutter/engine/issues/423

I've been closely following this project. At the Dart Summit talk, very high performance numbers were shown. When I load the sample app from the play store onto my Galaxy S5, or any of my co-workers devices I see fairly low frame rate. The frame rate is consistent, but it looks to only be around 15-20 fps. (The asteroids game sample appears to have very good frame rate though).
Do the original performance aspirations of 120 fps still seem feasible? Does the team feel there are optimizations that can be done to the current engine to increase the current performance significantly?

 Comment by abarth
Monday Aug 03, 2015 at 16:36 GMT

Do the original performance aspirations of 120 fps still seem feasible?

Yes.

Does the team feel there are optimizations that can be done to the current engine to increase the current performance significantly?

Yes.  Currently we are drawing every pixel of every frame from scratch.  We'll eventually cache some pixels and re-use them from previous frames.  We're purposefully waiting to add caching to the graphics pipeline because we want to see how far we can push the system in its current configuration.

 Comment by abarth
Tuesday Aug 04, 2015 at 16:06 GMT

c7f9c68a873dc680f5483f4f157d8091f97f425b should improve scrolling performance on the demo_launcher by 3x.  We'll need to check an S5 with the next release of SkyDemo to see if that fixes this issue.

 Comment by abarth
Thursday Oct 22, 2015 at 23:47 GMT

I believe we scroll much faster than when this issue was filed, so I'm going to mark this issue as fixed.  Thanks!

 Comment by spierce7
Thursday Oct 29, 2015 at 23:52 GMT

@abarth You can keep this closed if you want, but scrolling is still pretty laggy on the stock sample with the samsung s5, and also the nexus 5x I just tried it with too. After a while it seems to smooth out, but it takes some time.

 Comment by abarth
Friday Oct 30, 2015 at 00:12 GMT

Ok, I've re-opened this issue.

If it smooths-out after a while, it's probably Dart optimising the code over time.

I just profiled the Stocks app, after a cold start.
Most frames are > 16ms when running in checked mode.

In production mode, here's what I see:

Many frames are < 16, but you can see some that are bigger than 16.
After warming up the VM (scrolling up and down about 6 times), here's what I see:

I only see one long frame (oddly, right after I started tracing) and the rest are around 6ms

We intentionally do lots of expensive work in checked me to catch errors.  We could have an intermediate mode that had type checking but skipped the expensive checks, but that might just be more complexity.
The cold production mode trace looks pretty typical for our system today.  Background compilation will helpfull remove the green blocks, which is where a lot of the variance is coming from.
The warmed up trace look pretty typically too.  Once the VM warms up, we do a lot better.
  That can help you get a sense of scale when looking at traces.
If you use the flutter trace command to generate the traces instead of observatory, you'll also get tracing from the GPU thread, which can help you understand how we're doing on that side as well.

@chinmaygarde is working on getting all the trace events into observatory.  He's got some local patches that make it possible.  I'm not sure what's blocking landing them or what the bug number tracking that work is.

We are tracking this in https://github.com/flutter/flutter/issues/861

Performance should be much better by now.  We also recently introduced --release and --profile modes to flutter run and flutter build.  Unfortunately they only work on Android built from Linux, but iOS and Android from Mac support should be landing within the next couple days.  Regardless, I would expect these performance concerns to have been addressed in the last 6 months. :)
Please re-open if you don't agree!


Compile production copy with different themes, depending on parameter

🚀 Feature request
Command (mark with an x)

- [ ] new
- [x] build
- [x] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Description
 the problem is, that there should be a way to send parameters to the build, serve command to know which themes to use when building a pack for production.
Describe the solution you'd like
 It would be nice the common CSS file should be able to support multiple css files, and while building the application ng build .. --prod [ParameterA or ParameterB or ...], not only for CSS even for specific images or logos on the site.
depending the parameter it will build with a different CSS file. This will be useful when creating a single product for multiple clients, give them a personalized experience and theme

Hi @nickykapur, I think what you are looking already existing and can be achieved by having different build configurations.
Here is a simple extract of how you can do it.
Example
"build": {
  "builder": "@angular-devkit/build-angular:browser",
  "options": {
      ...
  },
  "configurations": {
    "production": {
      ...
    },
    "theme-1": {
      ...
      "assets": [
            "src/favicon.ico",
            "src/theme-1"
       ]
      },
    "theme-2": {
       ...
       "assets": [
          "src/favicon.ico",
          "src/theme-2"
        ]
      }
  }
}

ng build --configuration theme-2
ng build --configuration theme-1

Thanks, I think it correctly describes my issue!


location of .godeps folder

Maybe a little esoteric for most people - but I am trying to build a go project on a read-only filesystem.
Would be great to have command line option to change the location of the .godeps folder.

Would you accept a pull request for this?

Good question, I think I would since the implementation would be simple: I can imagine a use case like source gvp in .other_directory, if you feel like implementing it I think I'll merge it right away.
The thing is: my use case for gvp is now usually solved by simply using export and source
$ export GOPATH="$PWD"/.godeps:"$PWD"

In most cases I don't even do that much and instead just put that same line in the .env file for my projects, and then run source .env to load the GOPATH and any other env variables my app uses.
Maybe this means I'm not the best person to maintain gvp anymore, since I really barely use it, I'll have to think about that.

Actually I didn't realize it's that simple. Seems like I can just do
$ export GOPATH="/tmp/godeps:$PWD"

and be done with it. That indeed questions the need of gvp.
And I am not a big fan sourcein anyway.
I think I prefer the explicit export as well.

Exactly, gvp was written before I started working in that way, but these days I use it only very ocasionally, hence the little effort in improving the tool. :)

@tcurdt Are you still interested in this feature? Should this issue be closed?

I think this should just be properly documented - but for me I am happy to close this issue.

@tcurdt It would be awesome if you'd like to submit a PR for documentation.


Fix event monitor panic.

Fix the event monitor panic.
We should only stop ticker when it is not nil.
Since start and stop can be called at the same time, we need a lock to protect it.
Signed-off-by: Lantao Liu<EMAIL_ADDRESS>

LGTM

LGTM

/lgtm


Crop problem in 5.4

Greetings!
Im on unreal engine 5.4 and when i crop a BPGaussianActor created with importer in the editor preview everything is fine and also cropping the points creates a niagara effect cropped correctly, but when i launch it in game the crop position is moved and i cannot find a way to crop the selected area...
To Reproduce
Steps to reproduce the behavior:

Latest Plugin release for ue 5.4
Imported/trained correctly aBPGaussianActor from an mp4
Cropped and everything visualized fine in the editor preview and in the new cropped niagara asset created pushing on "Crop New Points In this Box" button from Clip Volume Component Box
Launching the game in editor the crop box is putted way away from selected area and if i launch the game in standalone nothing is cropped, even if i select the cropped one as asset...

Expected behavior
Right cropped position also in game.
Btw great plugin!

Failed to build on UE 5.4, would you mind showing how to build the project on UE 5.4.4?


Fatal Error in Main Process

This is on a Ubuntu Focal machine 20.04.1, using 5.9.0 kernel, and Mate Desktop.
yakyak
Starting Yakyak v1.5.10...
using hangupsjs v1.3.10
hangupsjs:: connecting
reconnecting 0
resolving proxy http://plus.google.com
resolving proxy https://plus.google.com
resolved proxy DIRECT
resolved proxy DIRECT
connected 0
Error on YakYak window:
{
msg: "Uncaught TypeError: Cannot read property 'length' of undefined",
url: '/usr/lib/yakyak/resources/app.asar/ui/models/conv.js',
lineNo: 42,
columnNo: 34,
error: TypeError: Cannot read property 'length' of undefined
at /usr/lib/yakyak/resources/app.asar/ui/models/conv.js:42:34
at add (/usr/lib/yakyak/resources/app.asar/ui/models/conv.js:49:9)
at Object._initFromConvStates (/usr/lib/yakyak/resources/app.asar/ui/models/conv.js:522:17)
at Object.init (/usr/lib/yakyak/resources/app.asar/ui/dispatcher.js:54:10)
at doAction (/usr/lib/yakyak/resources/app.asar/node_modules/trifl/lib/action.js:45:68)
at action (/usr/lib/yakyak/resources/app.asar/node_modules/trifl/lib/action.js:21:12)
at Object.init (/usr/lib/yakyak/resources/app.asar/ui/dispatcher.js:39:14)
at file:///usr/lib/yakyak/resources/app.asar/ui/app.js:209:23
at EventEmitter. (file:///usr/lib/yakyak/resources/app.asar/ui/app.js:107:16)
at EventEmitter.emit (events.js:223:5)
}
--- End of error message in YakYak window.
Goodbye

i have this same error. windows 10.

See #1234 for more info and a fix.

See #1234 for more info and a fix.

Hey thanks! This fixed the issue.


Ireland Reunification

LukeSky
Northern Ireland should have a decision to (re)join Ireland if it becomes independent.

The will, if the majority of the population is irish.


Feature Request: Minimum bar size config option

It would be nice if there was an option for this chart type to specify the minimum bar size in pixels. I have a 24 hour timeline and events that only last a minute or two do not show up at all.
The use case I have is to chart all of the events that occur in my house (motion detectors, doors open/close, lights on/off, etc) but for events that only last a few minutes, the bars render to small to even know they are there or to mouse over them.

Do you mean the bar width?
You can use the resize feature from chartjs to resize the chart.

Yes the minimum bar width. Imagine what a 30 second event looks like on a 24 hour time scale (you can't see it at all). Would like to be able to force it to 10px or 20px so that its visible and also so you can hover over it to see its tooltip.

Got it. This feature may added in future but currently no plan. You can change the code var width = end - x; to var width = Math.max(...) in updateElement function to workaround this issue.


fix(radar): symbolRotate does not work for radar series.

Brief Information
This pull request is in the type of:

[x] bug fixing
[ ] new feature
[ ] others

What does this PR do?
Fixes the bug that symbolRotate does not work for radar series.
Details
Before: What was the problem?
Whatever we set symbolRotate, symbol of radar is always facing north.

After: How is it fixed in this PR?
symbol of radar will perform a rotation as we specify.

Usage
Related test cases or examples to use the new APIs
Please refer to test/radar.html.

An early fixing has found in #10349, but I think we should use data.getItemVisual(idx, 'symbolRotate') but not seriesModel.get('symbolRotate').

LGTM


Addresses #3078 (second try)

Fixes #3078
This patches the Reactions code to do the right thing, as we discussed in the issue, and set stereo bonds in the reaction for which we have no stereo information to STEREONONE, instead of STEREOANY, as it was being done.
This fixes the issue with using findPotentialStereoBonds() on reaction products, since these won't have STEREOANY bonds without bond directions anymore.

@ricrogz sorry I'm slow (again) on this one. I will take a look over the weekend.

@ricrogz sorry I'm slow (again) on this one. I will take a look over the weekend.


JOSHUA-286 - Replace old joshua-decoder.org links with joshua.apache.org

Update links to documentation and support to reflect the
move to Apache.
keep Gitignore entry for old website was kept to keep the
repo clean.
Update links to the git repo as well.
old pages in the docs folder unchanged.

@john-hewitt thanks for the PR :)
If you could address the suggested improvements it would be greatly appreciated.

@lewismc Improvements addressed. Happy to help.

Can someone please merge in to master? Thanks @john-hewitt


Lib version

https://github.com/PHPoole/library/blob/d9bb1d17447eb6a6e3745874c4c0c2d375f1ddc3/src/PHPoole.php#L352
getVersion() Should open a text file, like VERSION, that contain the version number.

Fixed by #320.


Make e2e more stable

Description
Make e2e more stable
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[x] Configuration Update
[ ] Bump-up dependent library
[ ] Bump-up library or tool used for development (does not change the final image)

/lgtm

/approve

/override "Red Hat Konflux / ols-e2e-tests-rosa / lightspeed-service"

/override "Red Hat Konflux / ols-enterprise-contract / lightspeed-service"

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.68%. Comparing base (5da37ca) to head (4d77455).
Report is 31 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1421      +/-   ##
==========================================
+ Coverage   92.32%   94.68%   +2.35%
==========================================
  Files          58       58
  Lines        2503     2503
==========================================
+ Hits         2311     2370      +59
+ Misses        192      133      -59

see 6 files with indirect coverage changes


Create project model

Create a representation for projects in the database.

Jack and I are working on this one in addition.


Bug: Update panos_security_rule_group before deleting panos_address_object

Describe the bug
I have multiple panos_address_object referenced inside of destination_addresses in panos_security_rule_group. When I delete one of the panos_address_objects, the provider attempts to delete the resource before updating the panos_security_rule_group.
This results in an error similar to: Error:  rulebase -> security -> rules -> Outbound ADDRESS -> destination
Expected behavior
panos_security_rule_group is updated first with destination_addresses list before deleting panos_address_object
Current behavior
Provider attempts to clean up panos_address_object which is still referenced in the panos_security_rule_group

Related to these:
https://github.com/PaloAltoNetworks/terraform-provider-panos/issues/238
https://github.com/PaloAltoNetworks/terraform-provider-panos/issues/234
https://github.com/hashicorp/terraform/issues/26173
I tried adding a create_before_destroy on the panos_address_objects but got an inconsistent plan error.

This is equal parts #234 and hashicorp/terraform#25631, please see #234 for further info.
Closing this issue out as this is not a provider issue.