Error parsing test ids containing paths during pytest test discovery

Issue Type: Bug

Behaviour
Expected vs. Actual
Expected: Extension successfully discovers tests when clicking the test icon
Actual: Error occurs. The error is of this form (I've removed information I am not sure I can disclose), and occurs on line 184 of pythonFiles/testing_tools/adapter/pytest/_pytest_item.py:
fullname: path/to/file.TestClass.TestName
testfunc: TestClass.TestName
parameterized:

So the general issue is that fullname contains the path of the file containing the test, and testfunc does not, so the check fails and raises the should_never_reach_here exception.
Simply changing the raise statement on line 185 to pass fixes the problem for me; all tests, including those authored under the library cited below, are discovered successfully, can be run, etc.
Steps to reproduce:

Use the https://github.com/wayfair/pytest-eucalyptus library to author tests and author some
At least in my case, I get an error (described above). Apologies I can't provide a more precise test case, I am not a Python expert nor an expert on this library, and I'm not sure what I can disclose, but I do have a solution described above as well as a general form of the error I'm getting.

Diagnostic data

Python version (& distribution if applicable, e.g. Anaconda): 3.8.10
Type of virtual environment used (e.g. conda, venv, virtualenv, etc.): Pyenv
Value of the python.languageServer setting: Pylance

User Settings

defaultLS: {"defaultLSType":"Pylance"}

downloadLanguageServer: true

envFile: "<placeholder>"

venvPath: "<placeholder>"

venvFolders: "<placeholder>"

condaPath: "<placeholder>"

pipenvPath: "<placeholder>"

poetryPath: "<placeholder>"

languageServer: "Pylance"

linting
• enabled: true
• cwd: "<placeholder>"
• Flake8Args: "<placeholder>"
• flake8Enabled: false
• flake8Path: "<placeholder>"
• lintOnSave: true
• banditArgs: "<placeholder>"
• banditEnabled: false
• banditPath: "<placeholder>"
• mypyArgs: "<placeholder>"
• mypyEnabled: false
• mypyPath: "<placeholder>"
• pycodestyleArgs: "<placeholder>"
• pycodestyleEnabled: false
• pycodestylePath: "<placeholder>"
• prospectorArgs: "<placeholder>"
• prospectorEnabled: false
• prospectorPath: "<placeholder>"
• pydocstyleArgs: "<placeholder>"
• pydocstyleEnabled: false
• pydocstylePath: "<placeholder>"
• pylamaArgs: "<placeholder>"
• pylamaEnabled: false
• pylamaPath: "<placeholder>"
• pylintArgs: "<placeholder>"
• pylintPath: "<placeholder>"

sortImports
• args: "<placeholder>"
• path: "<placeholder>"

formatting
• autopep8Args: "<placeholder>"
• autopep8Path: "<placeholder>"
• provider: "autopep8"
• blackArgs: "<placeholder>"
• blackPath: "<placeholder>"
• yapfArgs: "<placeholder>"
• yapfPath: "<placeholder>"

testing
• cwd: "<placeholder>"
• debugPort: 3000
• nosetestArgs: "<placeholder>"
• nosetestsEnabled: undefined
• nosetestPath: "<placeholder>"
• promptToConfigure: true
• pytestArgs: "<placeholder>"
• pytestEnabled: true
• pytestPath: "<placeholder>"
• unittestArgs: "<placeholder>"
• unittestEnabled: false
• autoTestDiscoverOnSaveEnabled: true

terminal
• activateEnvironment: true
• executeInFileDir: "<placeholder>"
• launchArgs: "<placeholder>"

experiments
• enabled: true
• optInto: []
• optOutFrom: []

insidersChannel: "off"

tensorBoard
• logDirectory: "<placeholder>"

Extension version: 2022.0.1814523869
VS Code version: Code 1.64.2 (f80445acd5a3dadef24aa209168452a3d97cc326, 2022-02-09T22:00:58.347Z)
OS version: Darwin x64 21.3.0
Restricted Mode: No
Remote OS version: Linux x64 5.10.76-linuxkit

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyl392:30443607
pythontb:30283811
pythonvspyt551:30345470
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
pythondataviewer:30285071
vscod805cf:30301675
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30433507
vsclayoutctrc:30437038
vsrem710:30416614
dsvsc009:30440023
pythonvsnew555cf:30442237
vsbas813:30436447
vscscmwlcmt:30438805
helix:30440343
vsnot107:30443614

Hi @davidpcaldwell, thank. you for reaching out!
This is a known shortcoming of our current code for parsing test data, we will address this as part of #17242.

I also have an issue with failing pytest discovery that produced the following message:
The Python extension has run into an unexpected situation
while processing a pytest node during test discovery.  Please
Please open an issue at:
  https://github.com/microsoft/vscode-python/issues
and paste the following output there.

nodeid: utils::recursiveupdate
kind: ('function', False)
class: Function
name: test_recursiveupdate
fspath: /home/lik/repos/mp/maggma/tests/test_utils.py
location: ('tests/test_utils.py', 21, 'test_recursiveupdate')
function: <function test_recursiveupdate at 0x7f6e30da05e0>
markers: []
user_properties: []
attrnames: ['_ALLOW_MARKERS', '__annotations__', '__class__', '__delattr__', '__dict__', '__dir__', '__doc__', '__eq__', '__format__', '__ge__', '__getattribute__', '__gt__', '__hash__', '__init__', '__init_subclass__', '__le__', '__lt__', '__module__', '__ne__', '__new__', '__reduce__', '__reduce_ex__', '__repr__', '__setattr__', '__sizeof__', '__slots__', '__str__', '__subclasshook__', '__weakref__', '_check_item_and_collector_diamond_inheritance', '_fixtureinfo', '_getobj', '_initrequest', '_nodeid', '_obj', '_prunetraceback', '_pyfuncitem', '_pytest_diamond_inheritance_warning_shown', '_report_sections', '_repr_failure_py', '_request', '_store', 'add_marker', 'add_report_section', 'addfinalizer', 'cls', 'config', 'extra_keyword_matches', 'fixturenames', 'from_parent', 'fspath', 'funcargs', 'function', 'get_closest_marker', 'getmodpath', 'getparent', 'ihook', 'instance', 'iter_markers', 'iter_markers_with_node', 'keywords', 'listchain', 'listextrakeywords', 'listnames', 'location', 'module', 'name', 'nextitem', 'nodeid', 'obj', 'originalname', 'own_markers', 'parent', 'path', 'reportinfo', 'repr_failure', 'runtest', 'session', 'setup', 'stash', 'teardown', 'user_properties', 'warn']

extra info:
fspath:   /home/lik/repos/mp/maggma/tests/test_utils.py
fileid:   ./utils

I figured I shouldn't create a new issue if there is one with a similar title.

Hi @jmmshn,
Since the root cause of your problem is unclear, please open a separate issue with repro steps.
Thanks!

Looks like release of this is imminent. https://github.com/microsoft/vscode-python/issues/17242#issuecomment-1604417199

Hello! Thank you for reaching out, you are correct in that we have just finished our testing rewrite and are beginning the rollout to users. To try this yourself you need to be on vscode insiders and then add this setting to your users settings.json "python.experiments.optInto": ["pythonTestAdapter"]. Let me know if this works for you now! Thanks

Hi @eleanorjboyd, I don't think I can move to Insiders given the constraints I'm under, though I'll think harder about it (is there documentation on how to switch back if needed?). In the meantime, what the process for landing this experimental code (presumably behind the setting) on the stable build?

Hi! So insiders is a whole different application (so you can download it along side vscode stable). We will release the experiment to stable this week so once that release comes out (likely thursday) you should be able to try it on stable. Thanks!

OK, so I can confirm (so far) that when combined with my patches, the new parsing works with my test case.
Next I'll remove my patches and see whether it still works with more of an out-of-the-box setup.

Great! Yes, let me know without your patch if it works. Thanks for trying this and keeping me updated

Hi @eleanorjboyd -- we've got multiple users of VSCode (some of whom are not VSCode nerds like me). We're discussing whether and when we can remove our (ugly) patch to the Python extension that worked around this, and gathering data about what the current behavior looks like.
I see that with 1.81, 25% of users have been opted into the new test adapter. Is there a way we can see whether a particular installation is in the 25% or not?
Thanks, and we'll keep working to figure this out and let you know.

Hi @davidpcaldwell, sorry it has been a minute! Each user can check if they are enabled through the steps below, I don't think there is a way to check for someone else's machine id. The rewrite is now at 50% of all stable users and we will be bumping it up soon. Are there any outstanding problems you see? Just wanted to check in and see how things were looking. Thanks!
You can check if you have the rewrite enabled by setting your log level to trace, via the Developer: Set Log Level command in the command palette. Then check to see if Experiment 'pythonTestAdapter' is active is in your Python logs.

closing due to inactivity, please comment if you have more you want to discuss and I can reopen, thanks!


Github menus have changed

Organization account settings page has changed over on github.com.  SSO settings are now found under "Authentication Security".

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 156e3923-74d6-432f-c5a0-b905520d4e7d
Version Independent ID: aeca7d10-15e2-9e42-74bc-5309a5108c05
Content: Tutorial: Azure AD SSO integration with a GitHub Enterprise Cloud Organization - Microsoft Entra
Content Source: articles/active-directory/saas-apps/github-tutorial.md
Service: active-directory
Sub-service: saas-app-tutorial
GitHub Login: @jeevansd
Microsoft Alias: jeedes

@QeexoJim
I'm going to assign this to the document author so they can take a look at it accordingly
@jeevansd
Can you please check and add your comments on this doc update request as applicable.

Updating the document with latest UI.
#please-close


⚠ Bofrost.de has degraded performance

In 6f34534, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 919 ms

Resolved: Bofrost.de performance has improved in 42c6afd after 28 minutes.


Fork choice: minor formatting change

Non-substantive changes:

Ensure a comment is above the corresponding line of code.
Add an additional comment

Shouldn't we check parent state exists before copy the parent state?

I was hesitant to move beyond "making sure the spec is accurate" territory into "optimizing away copies", but I don't mind either way. If we're going to move it we should put it right before the state is modified.

So we do the assert before the state lookup because otherwise (as seen in a failed test), the state lookup in block_states throws if a block with unknown parent is passed in

So we do the assert before the state lookup because otherwise (as seen in a failed test), the state lookup in block_states throws if a block with unknown parent is passed in

Got it! Ordering has been changed :)


🍱 Adds Forgejo

Forgejo is a community maintained soft-fork of Gitea. Read more about it here.

My bad, I assumed ImgBot took care of that. Added now.


semantic should be 2.2.13 and rtl maybe??

can you add an option for rtl = both in the config
i did it manually copied from another install
but not sure if everything would work
didnt get that far yet to test

Bower will automatically install the last version of Semantic when you run bower install.
I don't think you can install Semantic UI with RTL with Bower. You have to install it with npm and set the rtl option to true in semantic.json:
$ npm install semantic-ui --save

# Then set "rtl" option to "true" in semantic.json

$ cd semantic
$ gulp build

Bower will automatically install the last version of Semantic when you run bower install.
About RTL:
Apparently, if you downloaded Semantic UI with Bower, you have to run npm install in the semantic folder to install the dependencies for gulp tasks. Then you have to copy the semantic.json.example to semantic.json and set the "rtl" option to "true". Then copy src/theme.config.example to src/theme.config. Finally, run gulp build to generate RTL CSS files.
A better solution would be to install Semantic UI with npm:
$ npm install semantic-ui --save

# Then set "rtl" option to "true" in semantic.json

$ cd semantic
$ gulp build

saw you removed bower
better since they no longer recommend using it
it has been replaced by yarn or npm to most packages
great
closed !!

rtl has 3 states
true fals and both
to use both change
"rtl": "both",
its diffrent needs extra quotes


Update README.md add dependency badge

Added badge for dependencies

Conflicts from another change. I'm going to add the badge directly. Thanks.


Improved documentation and more useful default for wrapper dir

Documentation how the username is determined.
Use the config directory instead of /usr/local/bin as default for wrapper. This has two main advantages:

guaranteed to be writeable for the user
not in $PATH, so gopass_wrapper.sh won't block autocompletion.

Review comments adressed.


Add collaborators to this repo

Comment below to be added as a collaborator
We will all be using this repository today. Even though it's public right now, you will not be able to make changes until you're given the correct permissions. We've automated this process via the GitHub API. Once you comment, we will add you as a collaborator.
You'll start to receive a lot of emails. (❗) You can slow these down immediately by clicking the Unwatch button at the top of the page and selecting either the Participating and @mentions or Ignore option.

Test

Test!

Test

This is just a test

Ping

test

wooo test

Test

Testing

test

Test

test

Test

test

test


neutronless member tests need to clean up after themselves

Title: neutronless member tests need to clean up after themselves
Attachments:
Details:

Suggested Issue Type
Test-Error
Traceback
$ py.test --symbols ../symbols.json.example ./member/ ./loadbalancer/
================================================ test session starts ================================================
platform linux2 -- Python 2.7.12, pytest-2.9.1, py-1.4.34, pluggy-0.3.1
rootdir: /home/ubuntu/git/f5-openstack-agent, inifile:
plugins: symbols-0.1.0a3, cov-2.2.1, f5-sdk-2.3.3
collected 19 items

member/test_single_member.py .E.E.E.E
member/test_single_member_opflex.py .E
member/test_single_member_weight_updates.py .
loadbalancer/test_f5_common_networks.py .s
loadbalancer/test_loadbalancer.py ..
loadbalancer/test_loadbalancer_admin_state.py .
loadbalancer/test_loadbalancer_disconnected.py ..
loadbalancer/test_loadbalancer_opflex.py ..
loadbalancer/test_loadbalancer_snat.py ...
loadbalancer/test_snat_common_network.py .

====================================================== ERRORS =======================================================
_________________________________ ERROR at teardown of test_create_single_member_up _________________________________

cls = <class 'neutronless.bigip_interaction.BigIpInteraction'>

    @classmethod
    def check_resulting_cfg(cls):
        """Check the current BIG-IP cfg agianst previous Reset upon Error

            This classmethod will check the current BIG-IP config and raise if
            there are any changes from the previous snap-shot.   Error: Invalid value [u'demo_policy'] for tag lbaas_policy
Pseudo-terminal will not be allocated because stdin is not a terminal.
Warning: Permanently added '<IP_ADDRESS>' (ECDSA) to the list of known hosts.
Warning: Permanently added '<IP_ADDRESS>' (RSA) to the list of known hosts.
--------------------------------------------- Captured stderr teardown ----------------------------------------------
Pseudo-terminal will not be allocated because stdin is not a terminal.
Warning: Permanently added '<IP_ADDRESS>' (ECDSA) to the list of known hosts.
Warning: Permanently added '<IP_ADDRESS>' (RSA) to the list of known hosts.
Warning: Permanently added '<IP_ADDRESS>' (ECDSA) to the list of known hosts.
Warning: Permanently added '<IP_ADDRESS>' (RSA) to the list of known hosts.
Saving active configuration...
Current configuration backed up to /var/local/ucs/cs_backup.ucs.
Product : BIG-IP
Platform: Z100
Version : 12.1.0
Edition : Hotfix HF1
Hostname: host-<IP_ADDRESS>.openstacklocal
Installing --full-- configuration on host host-<IP_ADDRESS>.openstacklocal
Installing configuration...
Installing the license file...
Post-processing...
usermod: no changes
Reloading License and configuration - this may take a few minutes...
________________________________ ERROR at teardown of test_create_single_member_down ________________________________

cls = <class 'neutronless.bigip_interaction.BigIpInteraction'>

    @classmethod
    def check_resulting_cfg(cls):
        """Check the current BIG-IP cfg agianst previous Reset upon Error

            This classmethod will check the current BIG-IP config and raise if
            there are any changes from the previous snap-shot.   Error: Invalid value [u'demo_policy'] for tag lbaas_policy
Pseudo-terminal will not be allocated because stdin is not a terminal.
Warning: Permanently added '<IP_ADDRESS>' (ECDSA) to the list of known hosts.
Warning: Permanently added '<IP_ADDRESS>' (RSA) to the list of known hosts.
--------------------------------------------- Captured stderr teardown ----------------------------------------------
Pseudo-terminal will not be allocated because stdin is not a terminal.
Warning: Permanently added '<IP_ADDRESS>' (ECDSA) to the list of known hosts.
Warning: Permanently added '<IP_ADDRESS>' (RSA) to the list of known hosts.
Warning: Permanently added '<IP_ADDRESS>' (ECDSA) to the list of known hosts.
Warning: Permanently added '<IP_ADDRESS>' (RSA) to the list of known hosts.
Saving active configuration...
Current configuration backed up to /var/local/ucs/cs_backup.ucs.
Product : BIG-IP
Platform: Z100
Version : 12.1.0
Edition : Hotfix HF1
Hostname: host-<IP_ADDRESS>.openstacklocal
Installing --full-- configuration on host host-<IP_ADDRESS>.openstacklocal
Installing configuration...
Installing the license file...
Post-processing...
usermod: no changes
Reloading License and configuration - this may take a few minutes...
______________________________ ERROR at teardown of test_create_single_member_down_up _______________________________

cls = <class 'neutronless.bigip_interaction.BigIpInteraction'>

    @classmethod
    def check_resulting_cfg(cls):
        """Check the current BIG-IP cfg agianst previous Reset upon Error

            This classmethod will check the current BIG-IP config and raise if
            there are any changes from the previous snap-shot.   Error: Invalid value [u'demo_policy'] for tag lbaas_policy
Pseudo-terminal will not be allocated because stdin is not a terminal.
Warning: Permanently added '<IP_ADDRESS>' (ECDSA) to the list of known hosts.
Warning: Permanently added '<IP_ADDRESS>' (RSA) to the list of known hosts.
--------------------------------------------- Captured stderr teardown ----------------------------------------------
Pseudo-terminal will not be allocated because stdin is not a terminal.
Warning: Permanently added '<IP_ADDRESS>' (ECDSA) to the list of known hosts.
Warning: Permanently added '<IP_ADDRESS>' (RSA) to the list of known hosts.
Warning: Permanently added '<IP_ADDRESS>' (ECDSA) to the list of known hosts.
Warning: Permanently added '<IP_ADDRESS>' (RSA) to the list of known hosts.
Saving active configuration...
Current configuration backed up to /var/local/ucs/cs_backup.ucs.
Product : BIG-IP
Platform: Z100
Version : 12.1.0
Edition : Hotfix HF1
Hostname: host-<IP_ADDRESS>.openstacklocal
Installing --full-- configuration on host host-<IP_ADDRESS>.openstacklocal
Installing configuration...
Installing the license file...
Post-processing...
usermod: no changes
Reloading License and configuration - this may take a few minutes...
______________________________ ERROR at teardown of test_create_single_member_up_down _______________________________

cls = <class 'neutronless.bigip_interaction.BigIpInteraction'>

    @classmethod
    def check_resulting_cfg(cls):
        """Check the current BIG-IP cfg agianst previous Reset upon Error

            This classmethod will check the current BIG-IP config and raise if
            there are any changes from the previous snap-shot.   Error: Invalid value [u'demo_policy'] for tag lbaas_policy
Pseudo-terminal will not be allocated because stdin is not a terminal.
Warning: Permanently added '<IP_ADDRESS>' (ECDSA) to the list of known hosts.
Warning: Permanently added '<IP_ADDRESS>' (RSA) to the list of known hosts.
--------------------------------------------- Captured stderr teardown ----------------------------------------------
Pseudo-terminal will not be allocated because stdin is not a terminal.
Warning: Permanently added '<IP_ADDRESS>' (ECDSA) to the list of known hosts.
Warning: Permanently added '<IP_ADDRESS>' (RSA) to the list of known hosts.
Warning: Permanently added '<IP_ADDRESS>' (ECDSA) to the list of known hosts.
Warning: Permanently added '<IP_ADDRESS>' (RSA) to the list of known hosts.
Saving active configuration...
Current configuration backed up to /var/local/ucs/cs_backup.ucs.
Product : BIG-IP
Platform: Z100
Version : 12.1.0
Edition : Hotfix HF1
Hostname: host-<IP_ADDRESS>.openstacklocal
Installing --full-- configuration on host host-<IP_ADDRESS>.openstacklocal
Installing configuration...
Installing the license file...
Post-processing...
usermod: no changes
Reloading License and configuration - this may take a few minutes...
_____________________________ ERROR at teardown of test_lbaas_service_one_opflex_member _____________________________

cls = <class 'neutronless.bigip_interaction.BigIpInteraction'>

    @classmethod
    def check_resulting_cfg(cls):
        """Check the current BIG-IP cfg agianst previous Reset upon Error

            This classmethod will check the current BIG-IP config and raise if
            there are any changes from the previous snap-shot.   Error: Invalid value [u'demo_policy'] for tag lbaas_policy
Pseudo-terminal will not be allocated because stdin is not a terminal.
Warning: Permanently added '<IP_ADDRESS>' (ECDSA) to the list of known hosts.
Warning: Permanently added '<IP_ADDRESS>' (RSA) to the list of known hosts.
--------------------------------------------- Captured stderr teardown ----------------------------------------------
Pseudo-terminal will not be allocated because stdin is not a terminal.
Warning: Permanently added '<IP_ADDRESS>' (ECDSA) to the list of known hosts.
Warning: Permanently added '<IP_ADDRESS>' (RSA) to the list of known hosts.
Warning: Permanently added '<IP_ADDRESS>' (ECDSA) to the list of known hosts.
Warning: Permanently added '<IP_ADDRESS>' (RSA) to the list of known hosts.
Saving active configuration...
Current configuration backed up to /var/local/ucs/cs_backup.ucs.
Product : BIG-IP
Platform: Z100
Version : 12.1.0
Edition : Hotfix HF1
Hostname: host-<IP_ADDRESS>.openstacklocal
Installing --full-- configuration on host host-<IP_ADDRESS>.openstacklocal
Installing configuration...
Installing the license file...
Post-processing...
usermod: no changes
Reloading License and configuration - this may take a few minutes...
======================== 18 passed, 1 skipped, 1 pytest-warnings, 5 error in 409.35 seconds =========================

Impacted Tests:
Only neutronless tests.  Nightly and Smoke are impacted here.  Smoke (not unit-based smoke/Travis tests) tests are currently disabled, though.
OpenStack Release
All Supported (liberty, mitaka, newton)
Agent Version
All
Operating System
nightly

Fixed.


Incorrect rendering of "flipped text"

Attach (recommended) or Link to PDF file here:
Vendor whitepaper download form
Configuration:

Web browser and its version:

Firefox release 121.0
Firefox nightly 125.0a1 (2024-03-04) (64-bit)

Operating system and its version:

Windows version: 10.0.22631.3235

PDF.js version:

latest web at https://mozilla.github.io/pdf.js/web/viewer.html

Is a browser extension:

both yes and direct load from here

Steps to reproduce the problem:

Open the file

What is the expected behavior? (add screenshot)
As rendered in Joplin:
Joplin 2.14.17 (prod, win32)

Revision: 094175c

Admonition markdown extension: 1.1.0
Automatic Backlinks to note: 3.0.3
Better Markdown Viewer: 2.0.4
Custom CodeMirror .vimrc: 1.0.6
Embed Search: 2.0.0
Folding in Code Mirror Editor: 2.0.1
Link Graph UI: 1.1.3
Note list and sidebar toggle buttons: 1.0.3
Quick Links: 1.2.2
Slash Commands: Datetime & More: 1.2.1
Templates: 2.2.0
Untagged: 2.2.0

What went wrong? (add screenshot)
As rendered on https://mozilla.github.io/pdf.js/web/viewer.html

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):
n/a

I have found the following direct link to the PDF file (since we don't want contributors to have to unnecessary fill out details in a form): https://lp.entro.security/hubfs/Gated Assets/Onboarding-offboarding-checklist.pdf
I can reproduce the issue with the current master branch on Linux.


Refactor: review name of processor packages and respective config name

Before releasing first version we should review the name of all processor packages and more importantly the respective name used in the config. Example: we probably should rename tracesamplerprocessor to probabilisticsampler and have its entry in the config be named "probabilistic-sampler".

Assign it to me, please.


How skip User model?

 PG::DuplicateTable: ERROR:  relation "users" already exists
: CREATE TABLE "users" ("id" serial primary key, "username" character varying, "role" character varying DEFAULT 'client', "email" character varying, "slug" character varying, "password_digest" character varying, "auth_token" character varying, "password_reset_token" character varying, "parent_id" integer, "password_reset_sent_at" timestamp, "last_login_at" timestamp, "created_at" timestamp, "updated_at" timestamp, "site_id" integer DEFAULT -1)
Is it possible skip an existing model?

you can do it overwriting the class in a initializer, see a sample to overwrite a class here:
http://camaleon.tuzitio.com/documentation/category/40757-developer-docs/customize-post-type-routes.html
So, you need to overwrite this class:
https://github.com/owen2345/camaleon-cms/blob/master/app/models/camaleon_cms/user.rb#L9
like:
CamaleonCms::UniqValidatorUser.class_eval do
  def validate(record)
    # here my custom validation for user
  end
end

@apradillap Had this problem recently after attempting to upgrade camaleon.
The schema_migrations table in production only had the most recent timestamp. I had to fix it by adding the schema timestamps that were missing from schema_migrations.

You can use db_prefix for table names, to do this, modify this in config/system.json
db_prefix: 'my_prefix'


Best practices for editing Main account category values

Dear team,
I work in Microsoft Support for Dynamics 365 Finance and Operations.
A customer raised a support ticket, ID<PHONE_NUMBER>001887, reporting that, after changing the Main Account Categories, the Row Definition for default Financial Reporting reports was not updated automatically.
I know that this is not possible since the values on those default FR reports are static (text fields), not dynamics fields. However, can we add a note on this documentation to officially state the Financial Reporting Report Definitions will have to be manually changed when you edit the Main Account Category name.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 847735ec-bba5-3d0c-e68c-20b457b07708
Version Independent ID: 930742dc-0d0b-a5dc-15e5-4079450ac367
Content: Set up main account categories - Finance | Dynamics 365
Content Source: articles/finance/general-ledger/tasks/set-up-main-account-categories.md
Service: dynamics-365-finance
Product: ****
Technology: ****
GitHub Login: @aprilolson
Microsoft Alias: aolson

Bug 797255 has been logged to address this issue.

@gebaluta, thank you for the feedback. The topic has been updated and changes will be live later today.


compileOnly dependencies aren't properly added to classpath when using clover

I had a working project that integrated clover and notice that when I change some of the dependencies to compileOnly, the compile fails.  Removing the clover integration or reverting the compileOnly will make it work, but it would be nice to have both rather than one or the other!

compileOnly is a Gradle 4.x feature. We have not made this compatible with the new Gradle 4.x features yet.

OK, thanks... it was driving me crazy since some projects were working and other's weren't...

I will try to unravel the new way dependencies work and see how it intersects with this plugin. Perhaps all I need to do is detect the new configurations and if present add them to the classpath. We use the Ant tasks from Clover to compile so the classpath needs to be configured explicitly.

According to this -
https://blog.gradle.org/introducing-compile-only-dependencies, you should
now be using compileClasspath property for the sourceset instead of the
compile property to get all the relevant classpath references...
On Tue, Aug 29, 2017 at 2:01 PM, Alex-Vol<EMAIL_ADDRESS>wrote:

I will try to unravel the new way dependencies work and see how it
intersects with this plugin. Perhaps all I need to do is detect the new
configurations and if present add them to the classpath. We use the Ant
tasks from Clover to compile so the classpath needs to be configured
explicitly.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bmuschko/gradle-clover-plugin/issues/99#issuecomment-325802932,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AANgOLfkj_u78nC_R9tm8_0QslmPZAt7ks5sdHwqgaJpZM4PE6co
.

@fbazbo The new version 2.1.3 works with Gradle 4.1
@Alex-Vol-SV Thank you for your fix! I was facing the same problem. When will the latest version be available on plugins.gradle.org?

I requested from Ben Muschko to publish the new version on Bintray and the plugins.gradle.org. He is busy so it may take a day to happen. Thanks for the feedback.
There is still some work to be fully 4.x compatible. In order for that to happen I may have to break compatibility with earlier versions though. I will probably bump the major version of the plugin when this happens.


Some package files have a date way in the past like Jan 31 1980

I see that in a few pkgs some files are way in the past:
C:\pt\scoop\apps>fd --changed-before 30years | sbperl0 -lnE "s{\.*$}{}; our %h; $h{$}++; END{ say qq[$ => $h{$_}] for keys %h }"
jbang => 10
gcloud => 20595
kotlin => 108
kotlin-native => 1864
gradle => 11013
,
I wonder what can trigger that. All pkgs are java-related
Anybody see that?
thanks

this means they are missing the timestamp.

Yes that is what I think happens. Yet whats surprises me is how and why.
If I package some project with some tool like tar o 7z, it not easy to fiddle with the timestamp except if somehow the tool cannot read the timestamp and decides to put for all those files with unreadable timestamps an epoch tiemstamp or whatever is the version of the epoch is for the tool.
Another possibility is that somehow the tool used for extraction by scoop cannot read some timestamps and thus spits out his version of the "epoch".
kotlin-native ships 2 o 3 times per year. don't you find it weird that an active project have such timestamps...
I hope you don't find me too peaky. I love scoop! :)
I will investigate a bit more...
hth


[TF] Conform ShapedArray/ShapedArraySlice to Hashable.

Fix latent bug in TensorBuffer.withUnsafeBufferPointer.

The body closure argument should not take an inout argument because
UnsafeBufferPointer is not mutable.
This bug caused errors in ShapedArray equality checking.

Conform ShapedArray/ShapedArraySlice to Hashable.
Add tests for Equatable and Hashable conformances.
Minor gardening.

Some more info about the withUnsafeBufferPointer bug: the program below had incorrect behavior.
import TensorFlow
let x = ShapedArray(shape: [2, 3], scalars: Array(0..<6))
x.withUnsafeBufferPointer { ptr1 in
  x.withUnsafeBufferPointer { ptr2 in
    print(ptr1, ptr1.count)
    print(ptr2, ptr2.count)
  }
}
// UnsafeBufferPointer(start: 0x00007faae0a00c60, count: 6) 6
// UnsafeBufferPointer(start: 0x00000001166d9520, count: 0) 0
// This bug causes `ShapedArray.==` to be incorrect.

@swift-ci Please test tensorflow


Use of newlib's memcpy/memmove functions in the kernel is unstable.

When building the kernel with newlib, rlib.rs is not compiled. This usually provides implementation for memcpy, memmove and the like. Instead, the implementations provided by newlib are used.
This is an issue, since newlib is not build for the kernel target, but a user one. Newlib is allowed to use the floating point unit, and as such memmove uses SSE. Using SSE in the kernel-context is an obviously bad idea.
I can somewhat reliably reproduce a BorrowMutError in my filesystem benchmark code:
[0][!!!PANIC!!!] src/scheduler/mod.rs:326: already borrowed: BorrowMutError

I am not entirely sure what causes this, but have tracked it down to the SSE usage inside memmove. Maybe something like:

userspace thread accesses FPU for the first time, exception is thrown and fpu_switch() is called.
fpu_switch() borrows current owner, switches state to new owner
fpu_switch() does self.current_task.clone(), which under certain circumstances calls memmove.

memmove again uses the FPU, causing another call to fpu_switch(), and a double-borrow occurs -> panic()

Log:
I am running 6 threads on a single core. First [ ] column contains rdtsc value. Unfortunately, adding more logging makes the bug disappear.
[0x112aeaebc][0][INFO] FPU Switch caused by PC 0x807b1b (userspace uses `fild    qword [rbp-0x8]`)
[0x112ce00e4][0][INFO] Switching FPU owner from task 6 to 7
[0x112eda4e4][0][INFO] FPU Switch caused by PC 0x807b1b (userspace uses `fild    qword [rbp-0x8]`)
[0x1130a30c0][0][INFO] Switching FPU owner from task 7 to 5
[0x11329b468][0][INFO] FPU Switch caused by PC 0x807b1b (userspace uses `fild    qword [rbp-0x8]`)
[0x11350c854][0][INFO] Switching FPU owner from task 5 to 4
[0x1137a506c][0][INFO] FPU Switch caused by PC 0x807b1b (userspace uses `fild    qword [rbp-0x8]`)
[0x113a09900][0][INFO] Switching FPU owner from task 4 to 6
[0x113c9f9b8][0][INFO] FPU Switch caused by PC 0x807b1b (userspace uses `fild    qword [rbp-0x8]`)
[0x113f05938][0][INFO] Switching FPU owner from task 6 to 3
[0x114196350][0][INFO] FPU Switch caused by PC 0x807b1b (userspace uses `fild    qword [rbp-0x8]`)
[0x1143b2428][0][INFO] Switching FPU owner from task 3 to 2
[0x114685c44][0][INFO] FPU Switch caused by PC 0x8aee0a (userspace uses `_calloc_r`, which uses `xorps xmm0, xmm0`)
[0x1148a0fc0][0][INFO] Switching FPU owner from task 2 to 7
[0x11538e0f4][0][INFO] FPU Switch caused by PC 0x8aee0a (userspace uses `_calloc_r`, which uses `xorps xmm0, xmm0`)
[0x11559bc94][0][INFO] Switching FPU owner from task 7 to 6
[0x1159dd6fc][0][INFO] FPU Switch caused by PC 0x8aee0a (userspace uses `_calloc_r`, which uses `xorps xmm0, xmm0`)
[0x115c01718][0][INFO] Switching FPU owner from task 6 to 2
[0x115e5f898][0][INFO] FPU Switch caused by PC 0x8b7b10 (kernel? in `memmove`, which does `movups xmm1, [rcx]`)
[0x116079b10][0][INFO] Switching FPU owner from task 2 to 5
[0][!!!PANIC!!!] src/scheduler/mod.rs:326: already borrowed: BorrowMutError

Solution
The use of newlibs optimized memcpy also had an unexpected performance penalty during my testing. We should stick to the implementations inside rlib.rs, even when building with newlib.
This can be done by simply removing the #[cfg(target_os = "hermit")] annotation from mod rlib;. Not entirely sure why that compiles rather then complaining about duplicate symbols though.
The resulting binary only contains the rlib version, which are non-optimized. This means userspace also gets the performance hit of not using SSE instructions in their copies. But this seems preferable to a race condition.
I can not think of a nice way to have both implementations with the current build system.

I hope that I fix this issue. Can you check it? It works on my test system.

Haha, love the solution of just renaming the kernel symbols. I don't have a nice setup to reproduce the original bug with the new master kernel, but have compiled a test c program and disassembled it. It has two copies of each mem* function, the libc one and the kernel_mem* one. All kernel functions call only the kernel_ one, everything else goes to the newlib mem*, so the issue should be fixed.


initialize the RSA struct to 0.

This helps with program code linked against static builds accessing a uninitialized ->engine pointer. (like e.g. steam)

+1

Any thoughts when this PR will be merged?

no thoughts; needs another team member to review.

+1

commit 6180c0f  closing.  thanks.


Add Docker image link in README

Do you know pip install --user manuale?

@szepeviktor Yes, what is your point?

I think --user provides separation.
But I am not against a container, I do things in Docker also.

Thanks!

Thanks!


chore: add SUPPORT information

more infos: https://help.github.com/en/articles/adding-support-resources-to-your-project

@Thomaash @yotamberk and every one else here: Could you please review this, I would like to introduce this to the other repositories in the visjs-family as well...If you like it..

LGTM


Support of mixing decimal ValueType with int or double or ...

If you use/mix decimal properties with properties of other valuetypes (like int) within expressions, one of the value cannot be converted.
Example:
{{ if obj.decimal_value > 0 }}
-> throws exception:
Scriban.Syntax.ScriptRuntimeException:  Unable to convert type System.Int32 to System.Decimal

Good catch, this should be fixed by commit 34bd596 and available in 1.2.6+


Add support for keyboardType: TextInputType.none to keep soft keyboard hidden but still allow input

Is there an existing issue for this?

[X] I have searched the existing issues

Use case
I am using QuillEditor on an Android tablet where the user will have a physical keyboard connected to the tablet. I want the user to be able to type inside the QuillEditor but I want to disable the on-screen soft keyboard from popping up when the editor is in focus.
Proposal
Flutter 3.21.0-0.0.pre changes the behavior of keyboardType: TextInputType.none to be exactly this, as mentioned here. I'd like to achieve the same behaviour with QuillEditor.
Thank you for this amazing Flutter package!

Hi, can you share a video that shows before and after adding the support for this feature?

all what you want is to hide the software keyboard when using the physical one when using keyboardType: TextInputType.none, right?

Yes.

This PR that solves the issue in Flutter uses native code (Java in Android) and this project doesn't use any native code, it's all pure dart except the plugins that are being used

so I'm not sure how to support it, maybe we should wait until Flutter 3.21 is released because right now the project only supports the stable branch of Flutter and only that to simplify the merging process
I see, tricky to solve indeed if you want this to remain a Dart-only package.
Maybe I could fork this at that point and try handling it on my own just like that PR does, but the maintainers of this repository are much smarter than me so it would be great to find a way to support this within this package. If someone comes up with a solution that is agreed upon, I'd be happy to take over the implementation for it and contribute in this package itself.

all what you want is to hide the software keyboard when using the physical one when using keyboardType: TextInputType.none, right?

Yes.

This PR that solves the issue in Flutter uses native code (Java in Android) and this project doesn't use any native code, it's all pure dart except the plugins that are being used
so I'm not sure how to support it, maybe we should wait until Flutter 3.21 is released because right now the project only supports the stable branch of Flutter and only that to simplify the merging process

I see, tricky to solve indeed if you want this to remain a Dart-only package.
Maybe I could fork this at that point and try handling it on my own just like that PR does, but the maintainers of this repository are much smarter than me so it would be great to find a way to support this within this package. If someone comes up with a solution that is agreed upon, I'd be happy to take over the implementation for it and contribute to this package itself.

Users already expect this to be a dart package only with some plugins that are being used
Unfortunately, most of the maintainers, including me, are off this project and are available for a short time for things like publishing, accepting PRs without reviewing it too much, and adding support for future versions of flutter
We really need someone who has both the time and the passion to complete this project and clean the mess in it


Implement the two-output division

Having mul_rev_to_pair as defined in section 10.5.5 of the standard or possibly taking advantage that Rust has enums
pub fn div2(self, rhs: Interval) → Union2;
pub enum Union2 {
    Single(Interval),
    Union(Interval, Interval),
}

would be useful (e.g. for Newton).  What do you think?

How about just returning a pair of Intervals, i.e., (Interval, Interval)?

Returning a pair (as in the standard) is Ok.  However, the caller has to remember which one is empty in case a single interval is returned so “in the abstract” using algebraic data-types is a better design.  That being said, for Newton, one has to check that N(x) ∩ x ≠ ∅ anyway so a function returning a couple of intervals may even be easier to use.  So I'm fine with returning (Interval, Interval).

I think this is complete, so closing.


Connecting to lnd from docker-compose

Background
We are attempting to run integration tests with docker-compose.
On Linux, we successfully connect to lnd instances running in docker, but for some reason, the test on windows unsuccessfully connect to LND.
Your environment

Latest LND version.
Windows 10 64bit, home.
Latest btcd and ltcd nodes.
Docker toolbox latest version.

Steps to reproduce
doceker-compose: docker-compose file
This sciprt runs the docker-compose : script
run: npm run test:int
these are the tests: lnd test
Expected behaviour
Test should pass, test should be abble to conenct to LND
Actual behavior
I get these errors:
1) LndClient        LndClients should connect:      Error: Timeout of 30000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (C:\Users\*****\Documents\Projects\walli-server\test\integration\lightning\LndClient.spec.ts)
2) LndClient        LndClients should fund get funds:      Error: Timeout of 10000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (C:\Users\****\Documents\Projects\walli-server\test\integration\lightning\LndClient.spec.ts)
3) LndClient        LndClients should open a channel to eachother:      AssertionError: expected undefined not to be undefined

I don't think this is the right place to report this as we don't support any of these files. If the test succeeds on other platforms, that would point to an issue with the test/docker setup on Windows itself. lnd should work just fine on all of the platforms that we provide releases for.
If you do happen to find that the culprit is some edge case within lnd that only happens when running under Windows, we'd be more than happy to re-open this.


bug with chart editor

Describe your bug here. If you are modding, try testing your bug in a clean version of the engine instead. Also, if you're using an older version of JSE, please try the latest version/action build. Also, be sure to check the pinned 'JS Engine: Known Issues' issue, to check if your issue hasn't already been found!
this error is one i didnt know it existed, but heres how it happens if you undo something but you are not in the same section of the notes it crashes the game
Command Prompt/Terminal/Crash logs (if existing)
Uncaught Error: Null Object Reference
editors/ChartingState.update() [line 3026]
flixel/FlxState.tryUpdate() [line 239]
flixel/FlxGame.update() [line 749]
flixel/FlxGame.step() [line 679]
flixel/FlxGame.onEnterFrame() [line 558]
openfl/events/EventDispatcher.__dispatchEvent() [line 426]
openfl/display/DisplayObject.__dispatch() [line 1398]
Please report this error to the GitHub page: https://github.com/JordanSantiagoYT/FNF-JS-Engine
The engine has saved a crash log inside the crash folder, If you're making a GitHub issue you might want to send that!

Have you identified any steps to reproduce the bug? If so, please describe them below in as much detail as possible. Use images if possible.
step 1 go to any song in the chart editor
step 2 place some notes
step 3 go to the next section
step 4 try ctrl + z
it should crash the game its a little random
if it doesnt try some more times it happened to me again on the second try

Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
nothing
If you use 1.12.0 or earlier, did you have Optimized Chart Loading turned on?
No
Did you check for any similar issues to what you're reporting? Check, then come back here. If there is a similar issue, then do not report the issue, otherwise it will be marked as a duplicate.
Yes

what version is this?

the lastest release, also im using vortex editor

apparently tried modifying bpmTxt's text while it's null

the only way for me to avoid this is not to use undo and remove the notes manually


action type when providing initialHandlers for createReducer

Description
Hey @piotrwitek
Just try out the latest version and it's a real improvement. I like this idea, it's very flexible and it simplify more the way to organize file.
I just have a small issue with the initialHandlers approach, the type of the action is not provided anymore. Works perfectly with the handleAction approach.
Steps to Reproduce
Consider we have this action:
export const ExampleStoreActions = {
  First: createStandardAction("ExampleActions/First")<{
    foo: string
    bar: string
  }>(),
}

export const ExampleStoreReducers = createReducer({foo: ''})
    .handleAction(ExampleStoreActions.First, (state, action) => {
        action.payload.bar; // Works perfectly
        return {
            ...state
        }
    });

createReducer({foo: ''}, {
    [getType(ExampleStoreActions.First)]: (state, action) => {
        // action.payload. // no typing hint for action
        return {
            ...state
        }
    }
});

createReducer({foo: ''}, {
    "ExampleActions/First": (state, action) => {
        // action.payload. // neither here
        return {
            ...state
        }
    }
});

CodeSandbox to help you reproduce issue in isolation
https://codesandbox.io/s/64ow9qxl3z
Expected behavior
Having the action correctly type like for handleAction.
Suggested solution(s)
Can investigate.
Correct me if I'm wrong, for the getType approach, we need to wait the fix from TypeScript.
But for explicit constant approach, should display the correct typing, shouldn't it?
Project Dependencies

Typesafe-Actions Version: 4.1.0
TypeScript Version: 3.4.3

PS: Good job for this release 👍

Hey @MasGaNo
Thanks for the report, it doesn't work because you have not followed tutorial.
If you're not using RootAction extension method, you need to add generic type arguments explicitely:
export const exampleStoreFooReducer = createReducer<
  { foo: string },
  ActionType<typeof ExampleStoreActions>
>({
  foo: '',
})
  .handleAction(ExampleStoreActions.First, (state, action) => {
    // action.payload.foo // Works well
    return {
      ...state,
    };
  })
  .handleAction(ExampleStoreActions.Second, (state, action) => {
    // action.payload.example // Works well
    return {
      ...state,
    };
  });

export const exampleStoreBarReducer = createReducer<
  { bar: number },
  ActionType<typeof ExampleStoreActions>
>(
  {
    bar: 42,
  },
  {
    [getType(ExampleStoreActions.First)]: (state, action) => {
      // action.payload. // No completion
      return {
        ...state,
      };
    },
    'ExampleActions/Second': (state, action) => {
      // action.payload. // Same with this approach
      return {
        ...state,
      };
    },
  }
);

@MasGaNo Sorry my bad, I just noticed it is there :)

Yeah, sorry, I tried to summarize, but yeah, I provided the RootAction, and this is my point when I said it's a good idea to keep it flexible with the RootAction approach or the generic one 🙂
But yeah, which both approach, I face the same problem for action type.

@MasGaNo you're right I have reproduced it and added a new test case, I think it should work but cannot confirm right now I need some time to investigate.

Sure no problem 🙂
I have some time before my next meeting, I'm trying to figure out something also 😉

Quick workaround:
// typesafe-actions/dist/create-reducer.d.ts

// ...
declare type GetAction<T extends Action, P extends T['type']> = T extends { type: P } ? T : never;

declare type InitialHandler<TState, TRootAction extends Action> = {
    [P in TRootAction['type']]?: (state: TState, action: GetAction<TRootAction, P>) => TState;
}

export declare function createReducer<TState, TRootAction extends Action = RootAction>(initialState: TState, initialHandlers?: InitialHandler<TState, TRootAction> /*Record<RootAction['type'], (state: TState, action: RootAction) => TState>*/): Reducer<TState, TRootAction> & {
    handlers: {
        readonly [x: string]: (state: TState, action: any) => TState;
    };
    readonly handleAction: HandleActionChainApi<TState, TRootAction, TRootAction>;
};

I replaced the Record type for initalHandler by what I used in my implementation, and it works for constant, but we come back with the problem of getType and the implicit any:

@MasGaNo thanks it works great 👍 Fix is on the way!
Regarding getType, we need to open a separate issue tracking a previously known TS compiler bug: Microsoft/TypeScript#29718

I think a way to get ride the need of getType (and so, the problem of using it in the handlerMap approach) is to use the future Symbol.toPrimitive and/or Symbol.toStringTag:
Microsoft/TypeScript/issues/4538 and Microsoft/TypeScript/issues/2361
Could be interesting to keep an eye on it.


Just In Time Provisioning (JITP) only works the first time then not anymore

Describe the bug
I am trying to use JITP scenario for provisioning devices in our company.
We have already tried it on python and it has worked only once after which gives an error that is fully described in this issue and happens to be the same here with the device client: https://github.com/aws/aws-iot-device-sdk-python-v2/issues/531
To Reproduce
Steps to reproduce the behavior:
With a JITP provisioning template created as well as a CA registered in the aws iot console, run the aws-iot-device-client where the config has fleet provisioning turned on.
Expected behavior
The execution of aws-iot-device-client should lead to a thing creation
I should be able to create many key and cert files and provision a thing for each of them.
However when I go to the iot webconsole under Security->Certificates I can see that a new certificates appeared.
However unlike the first one, it sits on pending instead of active.
If I manually set it to active and attach a policy and then rerun the code from above, everything works fine.
However the beauty of JITP should be that it automatically attaches the policy and activates the certificate.
Actual behavior
After the device client is setup with fleet provisioning enabled and the CA is registered in the aws iot web console, running the aws-iot-device-client will give an error like this:
2023-11-28T12:43:29.082Z [ERROR] {SharedCrtResourceManager.cpp}: MQTT Connection failed with error: libaws-c-mqtt: AWS_ERROR_MQTT_UNEXPECTED_HANGUP, The connection was closed unexpectedly.
2023-11-28T12:43:29.082Z [ERROR] {SharedCrtResourceManager.cpp}: *** Did you make sure you are using valid certificate with recommended policy attached to it? Please refer README->Fleet Provisioning Feature section for more details on recommended policies for AWS IoT Device Client. ***
2023-11-28T12:43:29.082Z [ERROR] {SharedCrtResourceManager.cpp}: Failed to establish shared MQTT connection, but will attempt retry...
2023-11-28T12:43:29.082Z [DEBUG] {Retry.cpp}: Retryable function returned unsuccessfully, sleeping for 10000 milliseconds

This error happens before fleet provisioning is ever attempted.
Possible Solution
Probably I misunderstood something about how to use JITP and self signed certificates.
But also the first attempt works the first time as expected.
Environment (please complete the following information):

OS: Ubuntu
Version: 20.04
Architecture: x86-64

Additional context
No response

Hello @DumitruTudor , I believe your Fleet Provisioning template does not specify to set the certificate status to Active after provisioning the thing. Please refer to this sample template over here and update your provisioning template to set the certificate status to Active after provisioning/creation. https://github.com/aws/aws-iot-device-sdk-python-v2/
.
.
.
...
"Resources": {
    "certificate": {
      "Properties": {
        "CertificateId": {
          "Ref": "AWS::IoT::Certificate::Id"
        },
        "Status": "Active"
      },
      "Type": "AWS::IoT::Certificate"
    },
    "policy": {
      "Properties": {
        "PolicyName": "FPCertPolicy"
      },
...
.
.
.

Once you set it to active, the service will create the certificate and set it to active by default for all things which are provisioned using your provisioning template.
The documentation itself is self explanatory but if you still have any questions for us then please feel free to reach out to us over here.
Regards,
Harsh Gandhi

Hello @DumitruTudor , its been a while since we heard back from you. We will close this issue now. I hope the previous comment answers your question and you are unblocked now.
If you have any other question regarding Device Client or if you are having any trouble using the software then please feel free to reopen this issue or create a new one.
Regards,
Harsh Gandhi


Call for Discussion - Fix a bug when animating icons within a button

Currently, if you add the cf-icon__spin class to an icon within a button, it spins everything, the padding, border, and margin due to those properties being attached to the icon pseudo element.

In order to spin the icon without spinning the other properties, we have to separate the two. This change won't affect icon usage anywhere else, only this specific case (meaning we would only need the empty spans here).
Changes

Updated buttons with icons styles and markup pattern

Testing

Clone the this branch to ~/Downloads/capital-framework.
cd ~/Downloads/capital-framework and npm install && npm run cf-link
Clone the gh-pages branch to ~/Downloads/cf-site.
cd ~/Downloads/cf-site and npm install && npm run cf-link && npm run build && npm start
Navigate to http://localhost:3000/components/cf-buttons/#button-with-an-animated-icon

Review

@caheberer
@Scotchester
@cfarm
@contolini
@anselmbradford
Anyone else that has had/has an opinion about icons and atomic organization

Screenshots

Notes

As much as I dislike using an empty element, and I was part of the push to remove them for icons, this is a bit of a complex situation. @Scotchester and I discussed the option to remove the border for animated icons and offsetting the rotation to account for the padding to avoid using an empty span. While short term I agree this keep things tidy and keeps from having a breaking change, long term it restricts the possibilities for animating the icon, especially when we move to svg icons and the possibilities for animation are opened up.
This also brings up a good question of should we be animating icons within buttons and what are the use cases beyond loading data.
See https://github.com/cfpb/design-manual/issues/227 for back discussion.

Todos

I didn't include a changelog entry because this is technically a breaking change. At the same time, none of v4 has been implemented yet, so I don't know that there's a risk of marking it as a minor change.
Discuss the speed of the spin animation, it's really slow. @caheberer, do you have input on this?
Move to SVG icons. If you look closely you can see the icon wobbles a bit do to the fact that it's a font icon and has a slightly offset baseline.

Checklist

[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

This also brings up a good question of should we be animating icons within buttons and what are the use cases beyond loading data.

For the Before You Claim project, we felt keeping the icon close to or within the button was important so people could know that the page had changed and their interaction was registered. Once the button is clicked, the tool makes calls to SSA's website; it usually takes less than a second on a high-speed connection, but sometimes takes as long as five or six seconds. The length of time might be even longer for folks on slower connections. The icon is serving as an indicator to the user that something is happening in the background, even if it takes ten seconds.
There's a bit more info on our internal GitHub, on the bugs-and-issues repo for this tool. Search for "spinner" and look for a closed issue.

Discuss the speed of the spin animation, it's really slow. @caheberer, do you have input on this?

We leaned towards slower partially to not unknowing trigger any neurovestibular disorders that are sensitive to spinning, and partially to counteract the potential long length of time the spinner would be going.

we felt keeping the icon close to or within the button was important so people could know that the page had changed and their interaction was registered

I had assumed this was the reason but thank you for verifying it. We should include something about this in the button specific content in the DM.

We leaned towards slower partially to not unknowing trigger any neurovestibular disorders that are sensitive to spinning, and partially to counteract the potential long length of time the spinner would be going.

Ah, that's a good use case, we should include something about that in the animation content in the Design Manual. I'll be sure to ping you when I open that PR for feedback.
Realized I should have included this for anyone that doesn't know what the icon spin speed is currently https://cfpb.github.io/capital-framework/components/cf-icons/#animated-icons.

we felt keeping the icon close to or within the button was important

Do you think that having a state icon (loading spinner, confirmation check, etc.) next to a button would be equally as effective as within the button?

Do you think that having a state icon (loading spinner, confirmation check, etc.) next to a button would be equally as effective as within the button?

I know we explored that as an option when designing this solution. If I remember correctly, we were worried about visibility, that folks might still miss it and also then needing to show the button in an inactive state (or visually different) while the loading process was happening. @Scotchester is next to a more desired solution for you? If so, just curious to hear your rationale.
Separate but similar issue we might want to add (because we use the animation in a button and in a page-level alert):
Make a distinction that when a button loads content within the same page, an icon inside a button is used, and when a button loads a new page, a loading icon within a page-level alert is used.

also then needing to show the button in an inactive state

Now I wonder if we should do this even with the icon inside the button so it's clear that you can't keep clicking the button to make it go faster (especially since that'll probably have the inverse affect)

Looking at our typical form example, I don't know that adding a current state icon between the two buttons is a great option, it feels very awkward and it's not clear which button it's attached to, where as the icon within the button looks far more refined.

(ignore the border color, I added the styles inline to make this one work from the inspector)

Thanks @jimmynotjim for your latest thoughts. Now that you've posted it here, I remember our team encountering that exact issue. For this reason, I think inside the button is better from a ux perspective @marteki agree?
It's also the most common convention I've seen outside the Bureau.

Yeah, I also agree. We'd want to make it really clear which action the spinner was related to. Having the spinner appear as a part of the action button meets that need, and doesn't introduce as much uncertainty about what the UI is trying to say to the user.
Also: on the Retirement example shown, I believe we either set the button inactive or rejected further responses on the backend while the API call was still ongoing. We could have a more explicit indicator on the UI side. ("DON'T CLICK THIS ANYMORE, IT'S TRYING TO DO THE THING FOR YOU" but obviously not just saying that.) Do users need a more explicit indicator?

I think making it inactive with the animation is plenty of hints to the user. I've made that update in the doc example.

@jimmynotjim Your code example doesn't show the a-btn__disabled modifier.

@Scotchester I guess I should push that eh?

So it sounds like a consensus about the icon staying inside the button. @Scotchester do you have any objections or can we move forward here?

I'd still rather narrow the exception to standard icon handling only to animated icons within buttons, and leave standard buttons with static icons alone. Perhaps we should get another FEWD or two to weigh in on that issue.
Whichever way we go on that, as long as we agree it's not necessary to bump to 5.0.0 for this, I'm good with it. (Don't forget to update the change log when we have consensus on that point.)

One last possible scenario, say we start with a static icon in a button, like the save icon, then switch to an animated update icon, then a static approved or error icon.
Why change the markup simply because it's animated in one of those three states when we could just change the class names? It's far simpler to maintain future code with the suggestions laid forward here.

@Scotchester and @jimmynotjim it seems like there is a disagreement on implementation, but this looks like a reasonable solution. While I there is certainly merit to both sides of the discussion, I like the simplicity of having a single pattern for buttons, rather than having to document exceptions or modify markup with JS. I'd recommend merging this, but creating a new CF issue documenting this as less than ideal. That moves us forward while leaving it open to future improvement should we have an idea for a different implementation.
What do you two think?

Jimmy's final example scenario was enough to convince me, for now, I just forgot to post that.
I agree with merging, again provided we call this 4.2.0 or whatever it is, not 5.0.0.

4.2.0 or whatever it is, not 5.0.0.

👍


my side of the story

Hi! Thanks for your submission.
Unfortunately it seems that there were changes in the main that conflict with yours.
Could you please perform a merge as described in Tutorial 5 slides 82-96 and integrate those changes?
We do not want to lose any of them, so you should keep what others have written and add yours.
You're free to reorganize the content blocks so that it still makes a story, but please do not modify
the content of what others have written.
Please do as soon as possible and if you can't make it please let me know.
Add a comment when you are so that I can progress with the next student!

Fixed the compatibility issue

I wonder what kind of creatures will spawn after this!


RRFS_A graphic for full and cutout domains.

The RRFS_A full and cutout domain (conus, ak, hi, pr) are available on JET:
/mnt/lfs4/BMC/public/data/grids/rrfs_a/rrfs_a.YYYYMMDD/HH/control for hourly deterministic forecast
Could you setup plotting for those domains?
Also, CONUS domains results from  RRFS A ensemble forecast at 00/06/12/18Z  are available:
/mnt/lfs4/BMC/public/data/grids/rrfs_a/rrfs_a.YYYYMMDD/HH//single_physics

Additions to RRFS_A NA 3km graphics workflow to support the AK cutout is now working.
Still working toward HI and PR.

AK rolled out and is available on the web now.
HI is going to be a bit more complicated. The cutout is a Mercator project, but similar to lat/lon projection. Need some adjustments for a single set of lat/lon values.


[rcore][GLFW] Wayland surface isn't the right size

Please, before submitting a new issue verify and check:

[x] I tested it on latest raylib version from master branch
[x] I checked there is no similar issue already reported
[x] I checked the documentation on the wiki
[x] My code has no errors or misuse of raylib

Issue description
Same as glfw/glfw#2643, wayland windows are exactly 37 pixels taller than they should be. I believe that this is a Raylib issue, since it doesn't happen with the glfw examples.
Environment

Platform/OS: Linux/NixOS
Compositor: Wayland/Hyprland
OpenGL version: 4.6.0 (I think?)
GPU: Nvidia

Issue Screenshot

Code Example
#include <raylib.h>

int main(void) {
  InitWindow(200, 200, "raylibtest");
  SetTargetFPS(10);

  while (!WindowShouldClose()) {
    BeginDrawing();
    EndDrawing();
  }

  return 0;
}

Tested it with Ubuntu and Labwc and everything is working correctly here. Probably a NixOS or Hyprland issue.

Logs

~$ lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 22.04 LTS
Release:        22.04
Codename:       jammy

~$ echo $XDG_SESSION_TYPE
wayland
~$ labwc --version
labwc 0.7.1

~$ unzip raylib-master.zip
[...]
~$ cd raylib-master/src/
~/raylib-master/src$ vim.tiny Makefile
[...]
~/raylib-master/src$ cat Makefile | grep GLFW_LINUX_ENABLE_WAYLAND
GLFW_LINUX_ENABLE_WAYLAND  ?= TRUE
        ifeq ($(GLFW_LINUX_ENABLE_WAYLAND),TRUE)
~/raylib-master/src$ make PLATFORM=PLATFORM_DESKTOP_GLFW
[...]

~/raylib-master/src$ cd ../examples/
~/raylib-master/examples$ vim.tiny core/core_basic_window.c
[...]
~/raylib-master/examples$ cat core/core_basic_window.c
#include <raylib.h>
int main(void) {
    InitWindow(200, 200, "test");
    SetTargetFPS(60);
    while (!WindowShouldClose()) {
        BeginDrawing();
        ClearBackground(RAYWHITE);
        EndDrawing();
    }
    return 0;
}
~/raylib-master/examples$ make PLATFORM=PLATFORM_DESKTOP_GLFW core/core_basic_window
[...]

~/raylib-master/examples$ ./core/core_basic_window
INFO: Initializing raylib 5.5
INFO: Platform backend: DESKTOP (GLFW)
INFO: Supported raylib modules:
INFO:     > rcore:..... loaded (mandatory)
INFO:     > rlgl:...... loaded (mandatory)
INFO:     > rshapes:... loaded (optional)
INFO:     > rtextures:. loaded (optional)
INFO:     > rtext:..... loaded (optional)
INFO:     > rmodels:... loaded (optional)
INFO:     > raudio:.... loaded (optional)
Couldn't open plugin directory: No such file or directory
No plugins found, falling back on no decorations
WARNING: GLFW: Error: 65548 Description: Wayland: The platform does not provide the window position
INFO: DISPLAY: Device initialized successfully
INFO:     > Display size: 1600 x 900
INFO:     > Screen size:  200 x 200
INFO:     > Render size:  200 x 200
INFO:     > Viewport offsets: 0, 0
WARNING: GLFW: Error: 65548 Description: Wayland: The platform does not support setting the window position
INFO: GLAD: OpenGL extensions loaded successfully
INFO: GL: Supported extensions count: 174
INFO: GL: OpenGL device information:
INFO:     > Vendor:   Intel
INFO:     > Renderer: Mesa Intel(R) HD Graphics 3000 (SNB GT2)
INFO:     > Version:  3.3 (Core Profile) Mesa 24.0.9-0ubuntu0.1
INFO:     > GLSL:     3.30
INFO: GL: VAO extension detected, VAO functions loaded successfully
INFO: GL: NPOT textures extension detected, full NPOT textures supported
INFO: GL: DXT compressed textures supported
INFO: GL: ETC2/EAC compressed textures supported
INFO: PLATFORM: DESKTOP (GLFW - Wayland): Initialized successfully
INFO: TEXTURE: [ID 1] Texture loaded successfully (1x1 | R8G8B8A8 | 1 mipmaps)
INFO: TEXTURE: [ID 1] Default texture loaded successfully
INFO: SHADER: [ID 1] Vertex shader compiled successfully
INFO: SHADER: [ID 2] Fragment shader compiled successfully
INFO: SHADER: [ID 3] Program shader loaded successfully
INFO: SHADER: [ID 3] Default shader loaded successfully
INFO: RLGL: Render batch vertex buffers loaded successfully in RAM (CPU)
INFO: RLGL: Render batch vertex buffers loaded successfully in VRAM (GPU)
INFO: RLGL: Default OpenGL state initialized successfully
INFO: TEXTURE: [ID 2] Texture loaded successfully (128x128 | GRAY_ALPHA | 1 mipmaps)
INFO: FONT: Default font loaded successfully (224 glyphs)
INFO: SYSTEM: Working Directory: ~/raylib-master/examples
INFO: TIMER: Target time per frame: 16.667 millisecond

Screenshot:

I should have tested that sooner, yeah, Hyprland is at fault.
I'll wait a bit before closing these issues however.

Closing, see hyprwm/Hyprland#8495 for more details.

Interestingly enough, I'm still seeing this with the latest Hyprland (0.46.2) @diniamo. Is it fixed for you now?

Yes, it has been since. You can always use the SDL backend though


BookInfo sample docs need update RE: GATEWAY_URL

Our instructions ask users to issue the following two commands to set GATEWAY_URL:
$ kubectl get ingress -o wide
NAME      HOSTS     ADDRESS                 PORTS     AGE
gateway   *         <IP_ADDRESS>          80        1d
$ export GATEWAY_URL=<IP_ADDRESS>:80

However, when running on GKE, I see something like:
$ kubectl get ingress -o wide
NAME      HOSTS     ADDRESS                                                                       PORTS     AGE
gateway   *         <IP_ADDRESS>,<IP_ADDRESS>,<IP_ADDRESS>,<IP_ADDRESS>,<IP_ADDRESS>   80        17m

Setting the GATEWAY_URL to any of the addresses listed (with port 80) fails, as follows:
$ curl -o /dev/null -s -w "%{http_code}\n" http://$GATEWAY_URL/productpage
000

If I do the following, however, it works:
$ kubectl get svc istio-ingress -o wide
NAME            CLUSTER-IP      EXTERNAL-IP     PORT(S)        AGE       SELECTOR
istio-ingress   <IP_ADDRESS>   <IP_ADDRESS>   80:30872/TCP   37m       istio=ingress
$ export GATEWAY_URL=<IP_ADDRESS>:80
$ curl -o /dev/null -s -w "%{http_code}\n" http://$GATEWAY_URL/productpage
200

this is what I stepped on in https://github.com/istio/manager/issues/635

I think we just need to set a field here: https://github.com/istio/istio/blob/master/kubernetes/istio-install/istio-manager.yaml#L11 to be
ingressService: istio-ingress

Has this been fixed?

Will submit PR soon, it's a documentation issue.

Doc changes are in istio/istio.github.io/pull/146


Converting extension to use a language server?

I had a look at the tailwindcss-intellisense package for vscode and how they do it and found that they developed an LSP for this. It is currently nested in the project (not a separate repo as one might suggest):
https://github.com/tailwindlabs/tailwindcss-intellisense/tree/master/packages/tailwindcss-language-server
Further they also provide a tailwindcss language service:
https://github.com/tailwindlabs/tailwindcss-intellisense/tree/master/packages/tailwindcss-language-service
It could make sense to evaluate this option so the extension for nova wouldnt need to reimplement the definitions based on the documentation.
Have you seen this? Would it make sense to you?
Cheers
Originally posted by @gobijan in https://github.com/jasonplatts/tailwindcss-nova-ext/issues/21#issuecomment-993622586

The above suggestion was made by @gobijan.
This previous issue I posted gives some idea of my thought process and why I haven't yet gone that direction, https://github.com/jasonplatts/tailwindcss-nova-ext/issues/2.
However, I am completely open to looking at again and would love some input. There are some obvious benefits.

As an aside, I would like to add additional features, but there is a lot of work involved in this extension. Unfortunately, I don't have as much time to spend on it as I would like.
If anyone is interested in contributing, please do. PRs are always welcome. I would also be open to adding some maintainers if someone wants to become more involved.

Hey @jasonplatts,
I have a few vacation days now. In case I can cook something up I'll get back to you. I just can't promise anything at the moment :)
I read the older issue you created. One quick question: so one needs a full tailwind installation to get it going?
Cheers
Bijan

No worries. If there is anything I can do to help, just let me know.
It's been a while since I looked, but last I knew the Tailwind VS Code extension required a full installation of Tailwind. I don't think that's changed.

Does anyone find the extension's sidebar documentation useful as a reference?

I do :) Love it!

Thanks! Great to know that others find it helpful.
It would be really nice if the Tailwind documentation was available as an API. Also, if Nova provided a way of searching and filtering sidebar items. I requested that feature be added to the Nova extension API, but Panic hasn't included it yet. Hopefully it's on the way.

It has occurred to me that there may be another issue with handling the Nova extension the same way as the VS Code extension. Some projects using frameworks, such as Ruby on Rails (tailwindcss-rails gem), are now using the Tailwind executable instead of the typical Tailwind Node.js installation method. There is some information about it here, https://tailwindcss.com/blog/standalone-cli.
Maybe it is possible to implement the Nova extension in a way that makes use of the new Tailwind executable?


Duplicate schema resources are not detected if the resources are different types

The work performed in conjunction with #230 changed the schemaResources to a Set (in the code now residing in DefaultSchemaResourceGraphQlSourceBuilder).
However, set semantics only work for Resource implementations of the same type.
If all resources are loaded via classpath scanning, they would likely have the same type and there would not be an issue with duplicate detection.
However, if the same resource is registered via classpath scanning (via Spring Boot's GraphQlAutoConfiguration) and explicit registration (via a GraphQlSourceBuilderCustomizer), the resources can be of different types. For example, a classpath scanned resource may be a FileSystemResource; whereas, a user might supply a ClassPathResource when using the customizer.
When the latter scenario occurs, you end up with an exception due to an attempted redefinition of the same schema types since the equals() implementations of FileSystemResource and ClassPathResource are not compatible.

I'm not sure how configuring the same resource twice in a different form can be considered a valid use case in Spring for GraphQL. We're also advising users to define their own GraphQlSource if they want to take full control over the schema.
Marking this for team discussion.

@joshlong, I believe this is the issue you encountered, but please correct me if I'm wrong.

I'm not sure how configuring the same resource twice in a different form can be considered a valid use case in Spring for GraphQL.

Understood.
My main goal was to bring attention to the issue, since Josh ran into this at one point (though his issue got resolved via proper classpath scanning support within a native image).

We've discussed this as a team and we believe that this case was mainly driven by a workaround for native support. We think that any other use case involving duplicate schema sources using different types of resources would be invalid.
I'm closing this as a result. Thanks!


jetbrains: update

Description of changes
Update the jetbrains IDEs. The new CLion release comes with LLDB, which has to be swapped out or the build fails. All IDEs launch.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

This PR is the same as https://github.com/NixOS/nixpkgs/pull/186263

Goland, works.
Datagrip, starts, but has at least one bug, that is not present in stable.
On 'Test Connection' to a PostgreSQL Database, this error occurs:
Driver class 'java.lang.constant.Constable' not found.

Driver Version 42.4.0 automatically downloaded by datagrip. I also tested 42.2.2, same error.
With a sqlite database this error doesn't occur.

Closes #187881

just curious how long these usually take to land. Is it waiting on review?

I've been using GoLand, WebStorm, PyCharm, and IntelliJ from this branch for five days. No problems so far.
@thyol The previously downloaded PostgresSQL driver was failing for me too, but worked by just removing and re-adding it:

Cool, it's ready to merge then !

What do we need to do to get this into the nixos-unstable branch?

I've been using GoLand, WebStorm, PyCharm, and IntelliJ from this branch for five days. No problems so far.
@thyol The previously downloaded PostgresSQL driver was failing for me too, but worked by just removing and re-adding it:

I just tried this trick, but it won't work for me. I guess I'll have to wait a bit until the driver is updated?

It seems like SFTP deployment is broken in the new versions (see #189298)

@thyol The previously downloaded PostgresSQL driver was failing for me too, but worked by just removing and re-adding it:

This did not resolve the issue. Which JDK are you using? https://github.com/NixOS/nixpkgs/issues/189298#issuecomment-1234639377

The one included on the derivation, I haven't changed it:
DataGrip 2022.2.1
Build #DB-222.3345.126, built on July 26, 2022
Runtime version: 11.0.13+0-adhoc..source amd64
VM: OpenJDK 64-Bit Server VM by Oracle Corporation
Linux 5.15.64
GC: G1 Young Generation, G1 Old Generation
Memory: 1962M
Cores: 12

Current Desktop: XFCE

Rider does not launch anymore due to it being unable to start its .net backend process, see https://github.com/NixOS/nixpkgs/pull/189412

Does anyone have LLDB debugging with rust working in CLion? I only get the error LLDB library not found: /nix/store/3zwqzysn0sm698pmp2aswm5lm7h7sfgx-clion-2022.2.1/clion/bin/lldb/linux/lib/liblldb.so.
Setting the LLDB path in the settings does not seem to have any effect (or all the values I tried to input are rejected). The console shows Jetbrains UI DSL - Row is created with empty label and if I navigate back to to the setting after clicking OK the field empty again.

I think this is because the lldb and its libraries are separate derivations, so when the bundled lldb gets replaced it doesn't have access to the libraries. Can you see if #192144 fixes the issue?


Update axios to latest 0.26.1

Description
axios is being used today in bolt-js for @slack/web-api and @slack/bolt packages. There is a vulnerability detected documented at https://github.com/advisories/GHSA-pw2r-vq6v-hr8c. This is due to a package follow-redirects.
This is a medium vulnerability and hence please update the axios dependency to at-least 0.26.0 that solves this. Check the release tag here
What type of issue is this? (place an x in one of the [ ])

[x ] bug
[ ] enhancement (feature request)
[ ] question
[ ] documentation related
[ ] example code related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[ x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[ x] I've read and agree to the Code of Conduct.
[ x] I've searched for any related issues and avoided creating a duplicate issue.

Bug Report
This is a security issue of type moderate risk. More details about the risk here.
Reproducible in:
package version:
node version:
OS version(s):
Steps to reproduce:

Expected result:
What you expected to happen
Actual result:
What actually happened
Attachments:
Logs, screenshots, screencast, sample project, funny gif, etc.

Hi @msrivastav13, thanks for reporting this issue! I just submitted a pull request fixing this: https://github.com/slackapi/bolt-js/pull/1366 The next release will rely on only 0.26.1 or newer version of axios.


Files with unexpected EOF cannot be processed.

The "assert" statement causes the process to fail.  Desired behavior is to process all complete packets.

Addressed in PR #99


Q: re, Example does not release model reference

In the "Using SketchUp Live C API" example ...
https://github.com/SketchUp/sketchup-live-c-api/blob/master/docs/using-live-c-api.md#example
The ruby_num_faces function does not release it's reference to the "live" model.
The primer page for standalone C API use has an example comment that says to always release the model reference, viz:
  // Must release the model or there will be memory leaks
  SUModelRelease(&model);

Q: Is releasing the model reference not needed for the "live" C API use ?
Q: What other differences are there from what is described in the standalone use primer section ...
Memory Management in the SketchUp C API ?

Q: Is releasing the model reference not needed for the "live" C API use ?

No, the model reference should not be released because the model is owned by SketchUp. In standalone API usage you need to do  The SUCreate* function mention you need to release the model you create with them - because you will be owning them.
SUApplicationGetActiveModel doesn't mention anything about ownership, we can add a note about that in its documentation to make it clear that the application owns the model returned.

Q: What other differences are there from what is described in the standalone use primer section ...
Memory Management in the SketchUp C API ?

None really, you are only supposed to release the objects you own.

Add to docs or create FAQ page, whatever is best.


[ADnD_2E_Revised] New feature: Psionic QoL and better rearranging of repeating sections

Submission Checklist
Required

[X] The pull request title clearly contains the name of the sheet I am editing.
[X] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[X] The pull request makes changes to files in only one sub-folder.
[X] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description
All changes described on the forum: https://app.roll20.net/forum/permalink/11361587/

Character Sheet Info Roll20 Internal Use only.


Seeking CallKit / iOS 13 advice for VOIP push notifications

Hi there! Less of a true "issue," but I was wondering if anyone from the twilio video team has any advice on the following scenario or any insights into your team's suggested workaround.
In iOS 13, Apple removed the ability to use VOIP pushes for anything other than reporting a new call. Previously, our app used VOIP pushes to cancel ongoing calls for certain cases: caller ends call before callee answers, callee answers on another device, callee declines call, etc.
In Apple's documentation, they specifically say we should not use VOIP pushes for this scenario:

After sending the initial push notification, don’t send additional push notifications to cancel the call or communicate new details to your app. Instead, communicate with the app directly over the network connection you established between it and your server. Using an existing network connection is generally faster than sending a push notification, and if network conditions are poor, APNs may be unable to deliver push notifications to the device anyway.

Since we are using twilio instead of a network connection to our servers directly, how do you suggest we communicate the cancel in these situations? I noticed the twilio voice team added a TVOCallInvite object that receives the updates in these cases (I'm assuming through a network connection to twilio's servers), but that case is a little different because I believe that product hosts the push notification service instead of this case where we are responsible for the push service.
I've thought about using regular pushes, but the potential delay and difficulty executing code reliably while your app is in the background (like it is when you are displaying the CallKit UI) are making that strategy difficult. I'm also considering following the twilio voice team's hack shown here by continuing to use VOIP pushes and reporting the call before ending it the next line down, but it was proposed as a fallback by the voice team in case the new push registration did not occur, and I'm worried about relying on that strategy long term.
Thanks so much in advance for any advice/insights!

@rwbayer Thanks for reaching out.
Our iOS Voice SDK ran into the same issue last fall. We followed Apple guidelines and shipped Voice iOS SDK version 5.x to support iOS 13 requirements. Your observation is correct, our Voice SDK creates a persistent connection with our infrastructure once the incoming call push notification is received. If the call is canceled by the Caller before called party could answer or reject, the called party receives Cancel via the its persistent connection with our infra.
Since Video SDK follows Room based model, you can use one of the following route to have incoming call feature -

Implement your own signaling using VoIP push notification - with this approach, you will have to make sure when a push notification is received, your app reports the received push to CallKit as a new incoming Call, and your app should create a persistent connection with your server to support cancel use case.

Use Voice SDK and Video SDK side by side in the same. Use Voice SDK to receive incoming call - once the called party take action on the incoming call, you should disconnect the Voice SDK Call. Some of our Video customers are using this work around to receive an incoming calls.

Let me know how it goes.

I just wanted to add a note here that option (2) above is not a solution but a workaround. Managing two SDKs for one call adds lot of  complexities in the app code, I would try to avoid it and keep it as the last option.

@piyushtank thanks for the ideas! We're going to try a polling approach first. If that doesn't work, we'll try out a long poll/socket with our infra. As a last resort, I'll keep the Voice SDK option in mind. Thanks again for your help, I'll close this for now since that advice was exactly what I was looking for


Bump Flink and Spark job server container base imager to tumerin-11

openjdk images have been long deprecated: https://hub.docker.com/_/openjdk
Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

PVR Spark Batch permared due to https://github.com/apache/beam/issues/30512#issuecomment-2441997749, not related to this change

assign set of reviewers


Coordinate Conversion does not support the ; and : as coordinate pair separtors when input is DDM and DMS (ArcGIS Pro)

The Coordinate Conversion addin supports the colon and and semi-colon characters (this is not supported by Pro/ArcMap). These separator characters work for input in DD format, but not DDM and DMS.
The following DD coordinates which have ; and : work:

45.0N;100.0W
45.0N; 100.0W
45.0N ; 100.0W
45.0N:100.0W
45.0N: 100.0W
45.0N : 100.0W

The following DDM and DMS coordinates which have ; and : do not work:

45 0.0N;100 0.0W
45 0.0N; 100 0.0W
45 0.0N ; 100 0.0W
45 0.0N:100 0.0W
45 0.0N: 100 0.0W
45 0.0N : 100 0.0W

@dfoll @topowright FYI. I have this noted in the Word doc that will serve as the xmetal content.

@topowright and I believe this should be closed

@dfoll @topowright I just confirmed that the Supported notation formats topic lists : and ; in DD but not for DDM and DMS, so yes it was okay to close.


enable documentation testing

Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

What does this PR do?
Fixes #232
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

@awaelchli do you remember what was the issue with running doctest in Sphinx and failing to import PyTorch?

@Borda sorry, did not see  this message until now. No, I don't remember exactly. It was something about mocked requirements, but I don't know anymore what the import issue is.
We have to find the old PR's and check the discussions there

the actual failer is following:
/opt/hostedtoolcache/Python/3.7.9/x64/lib/python3.7/site-packages/recommonmark/parser.py:75: UserWarning: Container node skipped: type=document
  warn("Container node skipped: type={0}".format(mdnode.t))

Warning, treated as error:
autodoc: failed to import module 'callbacks' from module 'pl_bolts'; the following exception was raised:
Traceback (most recent call last):
  File "/opt/hostedtoolcache/Python/3.7.9/x64/lib/python3.7/site-packages/sphinx/ext/autodoc/importer.py", line 32, in import_module
    return importlib.import_module(modname)
  File "/opt/hostedtoolcache/Python/3.7.9/x64/lib/python3.7/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1006, in _gcd_import
  File "<frozen importlib._bootstrap>", line 983, in _find_and_load
  File "<frozen importlib._bootstrap>", line 967, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 677, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 728, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/home/runner/work/pytorch-lightning-bolts/pytorch-lightning-bolts/pl_bolts/callbacks/__init__.py", line 4, in <module>
    from pl_bolts.callbacks.data_monitor import ModuleDataMonitor, TrainingDataMonitor
  File "/home/runner/work/pytorch-lightning-bolts/pytorch-lightning-bolts/pl_bolts/callbacks/data_monitor.py", line 5, in <module>
    import torch
  File "/opt/hostedtoolcache/Python/3.7.9/x64/lib/python3.7/site-packages/torch/__init__.py", line 190, in <module>
    from torch._C import *
RuntimeError: _ARRAY_API is not PyCapsule object


⚠ Samsung has degraded performance

In faaf488, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2007 ms

Resolved: Samsung performance has improved in 6f016e0.


Popup close link not work

Thanks of #1, #5，I got this project running now.

Another issue:  the popup close button not work
<f7-link close-popup>Close</f7-link>
If I add the class manually, it works:
<f7-link close-popup class="close-popup">Close</f7-link>

yes, i saw this problem too. probably it's about component.

Try <f7-link :close-popup="true">Close</f7-link>

It's working now.

None of the examples above seems to be working for me.
This framework is a joke.


Redstone dupe bug w/ Me Interface & Formation plane

This bug appears when AE2 is used with AE2 Stuff, also really easy to reproduce.
By simply placing a single redstone in the ME interface, the formation plane just keeps spitting out redstone:
http://imgur.com/a/O3iS2
See issue here:
https://github.com/AppliedEnergistics/Applied-Energistics-2/issues/2900
This is a bug with core AE2 items that appears to be caused by this mod.

NVM, not related to this, that link was wrong, closing this.


ShareDirectoryClient.Exists() may throw an exception if directory does not exist (ErrorCode: ParentNotFound)

Describe the bug
For a non-existing directory, the behaviour of ShareDirectoryClient.Exists() varies:

For a path "dir_does_not_exist" it returns False
For a path "parent_dir_exists\sub_dir_does_not_exist" it returns False
For a path "parent_dir_does_not_exist\sub_dir_does_not_exist" it throws an exception

Expected behavior
It seems reasonable to assume that an Exists() method always returns True or False under normal conditions. Hence not occasionally throw an exception for a non-existing directory, depending on the part of the path that does not exist.
We did get such consistency earlier, from the CloudFileDirectory.ExistsAsync() method in the v9.x SDK (WindowsAzure.Storage package) and the v11.x SDK (Microsoft.Azure.Storage.File package).
Actual behavior (include Exception or Stack Trace)
See description above, full exception is:
Azure.RequestFailedException: The specified parent path does not exist.
RequestId:adf8bd21-501a-001e-0d40-980676000000
Time:2020-10-01T22:14:40.7697655Z
Status: 404 (The specified parent path does not exist.)
ErrorCode: ParentNotFound

Headers:
Server: Windows-Azure-File/1.0,Microsoft-HTTPAPI/2.0
x-ms-request-id: adf8bd21-501a-001e-0d40-980676000000
x-ms-client-request-id: 06861a3e-6caf-45b9-a296-d538afe4556d
x-ms-version: 2019-12-12
x-ms-error-code: ParentNotFound
Date: Thu, 01 Oct 2020 22:14:40 GMT
Content-Length: 224
Content-Type: application/xml

   at Azure.Storage.Files.Shares.FileRestClient.Directory.GetPropertiesAsync_CreateResponse(ClientDiagnostics clientDiagnostics, Response response)
   at Azure.Storage.Files.Shares.FileRestClient.Directory.GetPropertiesAsync(ClientDiagnostics clientDiagnostics, HttpPipeline pipeline, Uri resourceUri, String version, String sharesnapshot, Nullable`1 timeout, Boolean async, String operationName, CancellationToken cancellationToken)
   at Azure.Storage.Files.Shares.ShareDirectoryClient.GetPropertiesInternal(Boolean async, CancellationToken cancellationToken, String operationName)
   at Azure.Storage.Files.Shares.ShareDirectoryClient.ExistsInternal(Boolean async, CancellationToken cancellationToken)
   at Azure.Storage.Files.Shares.ShareDirectoryClient.Exists(CancellationToken cancellationToken)
...

To Reproduce
Invoke Exists() or ExistsAsync() on a ShareDirectoryClient object that has been created for a path like "parent_dir_does_not_exist\sub_dir_does_not_exist"
Environment:

Name and version of the Library package used: Azure.Storage.Files.Shares v12.x
Hosting platform or OS and .NET runtime version (dotnet --info output for .NET Core projects): not relevant
IDE and version : not relevant

Thank you for your feedback.  Tagging and routing to the team best able to assist.

Hi,
Thanks for bringing this issue to our attention. This is a reasonable ask, you're right, we should be also catching that error code. We will look to update the exists method soon. I will label this as a bug.

Thanks for the quick response. Let me add that ShareFileClient.Exists() behaves similar to ShareDirectoryClient.Exists().


In the ZestGuidance Class, there may be a potential issue in the calcalation of this.branchCount within the handleEvent function.

Orignoal soucre code is following:
   // fuzz/src/main/java/edu/berkeley/cs/jqf/fuzz/ei/ZestGuidance.java
    protected void handleEvent(TraceEvent e) {
        conditionallySynchronize(multiThreaded, () -> {
            // Collect totalCoverage
            ((Coverage) runCoverage).handleEvent(e);
            // Check for possible timeouts every so often
            if (this.singleRunTimeoutMillis > 0 &&
                    this.runStart != null && (++this.branchCount) % 10_000 == 0) {
                long elapsed = new Date().getTime() - runStart.getTime();
                if (elapsed > this.singleRunTimeoutMillis) {
                    throw new TimeoutException(elapsed, this.singleRunTimeoutMillis);
                }
            }
        });
    }

The optimization of short-circuit evaluation can result in this.branchCount not being incremented.

The short-circuiting is intentional. When runStart is null, we are not within a test method, so we don't want to count branches.

Thank you for your explanation, got it.


RATIS-1143. Return exception of submitClientRequestAsync to client

What changes were proposed in this pull request?
Return exception of submitClientRequestAsync to client
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/RATIS-1143
How was this patch tested?
new ut.

@szetszwo Could you help review this patch ? Thanks a lot.

@szetszwo I have updated the patch. Could you help review it again ? Besides,  I think with the commit https://github.com/apache/incubator-ratis/commit/05eaa70128dfa12dcfed459684983267cb079e92, CI is easier to pass, I think the previous failed reason is port conflict, I will continue to find the root cause.