Callstack is invalid after new NodeJS deployment

Hi,
I am using the BotBuilder on Heroku (NodeJS) together with the Facebook messenger. Everything really works like a charm. There is only one issue with the conversation state after deployments. Everytime I do a new deployment to Heroku (NodeJS will be restarted and local memory cleared). When a Prompt is started for a user before the deployment and the users answers to that Prompt after the deployment, the session state will be resetted so the answer of the user does not come back into my current dialog flow. I can see a corresponding message in my log file in that situation:
2016-10-18T11:15:32.958546+00:00 app[web.1]: ....Prompts.choice - WARN: Callstack is invalid, resetting session.
Does anyone have a clue how to fix this issue? If you need some more code snippets just let me know.

Are you making significant changes to the bots dialogs?  We do a scan to see if any of the dialogs on the callstack are missing. If they are the bot will eventually hit a dead end so we just clear everything and start over.

No. The bug also occurs when I deploy no changes to dialogs or to anything else that could affect the bot. The issue even occurs when I deploy with exact the same code base :/
Sent from my phone

Am 19.10.2016 um 08:27 schrieb Steven Ickman<EMAIL_ADDRESS>Are you making significant changes to the bots dialogs? We do a scan to see if any of the dialogs on the callstack are missing. If they are the bot will eventually hit a dead end so we just clear everything and start over.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hmm... I just looked at the code and it would only trigger that error if it couldn't find a dialog that's listed as being on the stack.  You're not dynamically adding dialogs to bot are you?  By that I mean do you ever call bot.dialog() in response to some user action or after some sort of async operation completes? That's the only thing I could think of that would cause this behavior.

@Stevenic That was a good hint! Actually I am adding dialogs dynamically and that must be the issue 🙈 you are a genius 😄
I close this issue as I am pretty sure this will be the fix 👍

I just wrote the damn thing is all :)

Additionally... We like hearing all of your issues, even when they're your own so that we can think about changes to the framework that would help avoid this in the future.  I've already made several changes purely because people kept getting hung up in the same spot.


Why yarn.lock is gitignored?

I prefer the host app to define to lock that down.  I realize people have different opinions on this.

I've my opinion that can be wrong! For that reason I'm asking 😄

I heard someone say once, "ignore for libraries", "include for apps".  Made sense to me.


Toward 0.3.0

a few bugfixes: networked-share, Firebase+WebRTC;
expose NAF options for convenience;
update getDirtyComponents to version in networked.js
(which allows selector-based schema components to work,
which is needed for template-based physics)

Lgtm


Use correct Path for Kivy Home Dir

At the moment, Kivy saves all of it's data under ~/.kivy, but this is wrong. Each Operating System has it's own Location to save such files. This PR make Kivy respect the Standard for each OS.
This is especially important for Linux, if you run a Kivy App in a Sandboxed Environment (e.g. Flatpak or Snap) which expects the Apps to save it's Data in a custom Directory (e.g. Flat Flatpak it's ~/.var/<appid>) and restrict access to Home.
For macOS I#m not 100% sure, if I got the right Location, because I never used it, but this is the Location I found that are used by other Programs as well.
Maintainer merge checklist

[ ] Title is descriptive/clear for inclusion in release notes.
[ ] Applied a Component: xxx label.
[ ] Applied the api-deprecation or api-break label.
[ ] Applied the release-highlight label to be highlighted in release notes.
[ ] Added to the milestone version it was merged into.
[ ] Unittests are included in PR.
[ ] Properly documented, including versionadded, versionchanged as needed.

This is a substantial improvement to Kivy's current home directory handling. It'd be great if this could be merged at some point. Thanks for the tremendous volunteerism, @JakobDev! :hugs:

@JakobDev , sorry for the late reply, but as you might have noticed, we were into feature-freeze for 2.2.0 (which is now released).
Some thoughts about this proposal:

Should we need to offer an automated migration path? (We're currently targeting 2.3.0 as the next release, even if some major changes may require a 3.0.0, so I'm not sure about the next release version yet)
How about using https://platformdirs.readthedocs.io? (we can open a PR on platformdirs repo to support iOS if needed) + is a plain python package and will not need a recipe for Android or iOS


Improve description of object details

Description
Improve the description of the object details in the proto definition.
Acceptance criteria

[ ] a user reading the proto/swagger docs must understand which data the fields represent

@adlerhurst Can we close this issue since we are currently talking about our new api?


add spanish translations

I have always been a fan but i have no money for Patreon, so I decided to help by adding spanish translations.

Thank you so much, @SrKomodo ! :)


Introduce API version 2020-06-01

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[x] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.

adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[x] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

/azp run automation - sdk

Can one of the admins verify this patch?

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

Please review Linter errors and warnings. Specific errors below.

❌ R3016 - DefinitionsPropertiesNamesCamelCase
Property named: 'trigger_url', for definition: 'FunctionSecrets' must follow camelCase style. Example: 'trigger_url'.New: Microsoft.Web/stable/2020-06-01/WebApps.json#L21924

❌ R4005 - UniqueXmsEnumName
Must not have duplicate name of x-ms-enum extension , make sure every x-ms-enum name unique. The duplicate x-ms-enum name is managedserviceidentitytypeNew: Microsoft.Web/stable/2020-06-01/WebApps.json#L20350

❌ R4005 - UniqueXmsEnumName
Must not have duplicate name of x-ms-enum extension , make sure every x-ms-enum name unique. The duplicate x-ms-enum name is managedserviceidentitytypeNew: Microsoft.Web/stable/2020-06-01/WebApps.json#L22391

/azp run automation - sdk

@chiragg4u The issue below is not a new issue. It existed in previous API version too. The property is not owned by the App service control plane but the functions runtime. The API surface just fwds the request.
Property named: 'trigger_url', for definition: 'FunctionSecrets' must follow camelCase style. Example: 'trigger_url'.New: Microsoft.Web/stable/2020-06-01/WebApps.json#L21924

/azp run automation - sdk

Please review all linter errors and warnings.

DefinitionsPropertiesNamesCamelCase can be suppressed if it is maintained by function app service. Though it still better to @ them so that they know about it, and can schedule possible fix in next version (or better, even in this same version).

@mathewc The ask here is to change the property name from trigger_url to triggerUrl in the /admin/host/synctriggers fx runtime API. Please add a tracking item for future work.
@weidongxu-microsoft I will add a suppression to unblock for now.

/azp run automation - sdk

@weidongxu-microsoft @chiragg4u Please review and merge.

Change
https://github.com/Azure/azure-rest-api-specs/compare/8f60822e7d75ce9ceba1269bb6da1ba74149c562..d16b3c43b15d03e41172dda4ca0d110d61c7454f?expand=1

While adding new version the recommendation is to copy the existing version into new directory structure for first commit and then push new changes for easy review.

Looks like you've just copied the files into new version in this case no formal review required. In case there are changes where you require ARM API review please update the PR with the above recommendation for easy review,


Variable number of args and kwargs for handler functions

Hi
I couldn't find anything about this in the docs:
Does this library support a variable number of args and kwargs for a handler function?
Example:
@dispatcher.add_method
def foobar(*args, **kwargs):
    return kwargs["foo"] + kwargs["bar"] + args[0]

@michaelimfeld it does support a variable number of arguments, however, with slightly different approach. JSON-RPC standard allows arguments of the function to be sent either as list or dictionary, but not as a mix of them. This library follows the standard and does not allow both *args and **kwargs at the same time. There is a workaround in your case, args could be extra key of kwargs:
@dispatcher.add_method
def foobar(**kwargs):
    return kwargs["foo"] + kwargs["bar"] + kwargs["args"][0]

Payload in this case would have params value:
{"foo": 1, "bar": 2, "args": [3]}

Does it work?

Yes, works!


Nest doorbell stream has no audio

Hi,
I'm unable to add audio to the Nest doorbell camera stream. The original camera entity from the Nest integration does have audio. Currently, I am using the hass-expose-camera-stream-source integration to be able to use this camera in Frigate. Relevant sections of my Frigate config:
ffmpeg:
  output_args:
    record: preset-record-generic-audio-aac

go2rtc:
  streams:
    doorbell:
      - echo:bash /config/custom_components/expose_camera_stream_source/get_stream.sh camera.doorbell
      - "ffmpeg:doorbell#audio=aac"

Tried without the ffmpeg:doorbell#audio=aac also w/o luck.
Relevant log snippet:
2024-02-11 04:10:29.601473759  20:10:29.601 WRN [rtsp] error="streams: codecs not matched: AAC => ANY" stream=doorbell

Error said "codecs not matched: AAC => ANY". But AAC should be matched to ANY codec.
Usually this happened when stream can't start.


How do I revert

hi i used it and it removed the built in administrator account which had the system restore point now i cant use systemrestore pls help

It didn't remove the built in admin account. It only disabled it per Windows 10 V2R1.

Change this to enabled.

That account isn't used by default on most systems. It shouldn't break anything to have this policy enabled.
Can you tell me what exactly is broken?

Are you still having problems @Ferosauce ?

Closing due to lack of response and due to configuration being documented..


Add cumprod to BatchedArray

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

Comparison is base (b9ca85b) 100.00% compared to head (1d9b120) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #537   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           24        24
  Lines         1884      1886    +2
  Branches       263       263
=========================================
+ Hits          1884      1886    +2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Target Folder on SD Card

Describe the bug
LocalSend works well for me, if I leave the default target folder untouched. If I create a folder on my SD card, transfer from PC fails. I get an error like not allowed. But I allowed files and media in apps settings. When I created the folder on my SD card, I also had to allow access. I do not want to store much on the internal storage as I have a huge SD card. The other direction, from tablet to PC works without any problems.
To reproduce
I hope this report is no dublicate.
Expected behavior
Thanks much.
Screenshots
No response
Desktop (please complete the following information)
Latest Debian stable
Smartphone (please complete the following information)
Lenovo Tab8 4th Gen
with Android 13
Additional context
No response

Duplicate of #710


DreamStudio's API Key is not retained and has to be inserted at every StableStudio session

Describe the bug
The Dream Studio API Key is not retained after closing the program, therefore at the next restart it has to be inserted again.
To Reproduce
Steps to reproduce the behavior:
See above
Expected behavior
It would be convenient if the API Key, once inserted as required, could be someway stored locally, better if inside a separate, dedicated file so that the user might eventually, for any reasons, be able to delete it.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 10 Pro
Browser Firefox
Version 113.0.1 (64 bit)

Smartphone (please complete the following information):
N/A
Additional context
Not really a bug per-se but more a usability suggestion.

The Stability API plugin stores your API key in localStorage in your browser, so it should persist as long as you visit from the same origin.
I have just tested this and cannot seem to reproduce your issue.

I didn't save it in the browser, that's why. Thank you.


Venv not working when in a sub directory

Environment data

VS Code version: 1.50.1
Extension version (available under the Extensions sidebar): v2020.10.332292344
OS and version:

Distributor ID:	Ubuntu
Description:	Ubuntu 18.04.5 LTS

Python version (& distribution if applicable, e.g. Anaconda): 3.8 in venv
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): venv
Relevant/affected Python packages and their versions: XXX
Relevant/affected Python-related VS Code extensions and their versions: XXX
Value of the python.languageServer setting: jedi

[NOTE: If you suspect that your issue is related to the Microsoft Python Language Server (python.languageServer: 'Microsoft'), please download our new language server Pylance from the VS Code marketplace to see if that fixes your issue]
Expected behaviour
I expect the python binary inside dev/venv to be used
Actual behaviour
The default interpreter is always selected
Steps to reproduce:

Create dev/ folder
Create venv/ in dev/
Try to use the python interpreter in VSCode from dev/venv/bin/python

@katsar0v
Thanks for the bug report! We investigate issues in order based on priority and severity, which includes the impact it has on your ability to use the extension to do productive work, and the number of people affected. If other users come forward and leave a comment demonstrating they are seeing/reproducing the problem then we will raise this issue's priority.
If you think your issue is more of a question or configuration problem rather than a bug, please ask on Stack Overflow with the visual-studio-code and python labels for help.
Thanks for your understanding and patience!

In the meantime, what happens if you click on "Enter interpreter path..." and type it in there?

I also cannot change my interpreter any more. It doesn't matter which one I select (all different conda environments), it always just stays with the one that it defaults to on startup of VSC.
I have tried reinstalling the extensions and deleting their files from \.vscode\extensions\, but to no avail. The problem should not be related to jedi or pylance as it persists even when none of them are enabled or installed. It is a pretty jarring problem as there seems to be no way to use python in VSC for the moment, safe for changing the one environment I can use every time I want to do something different.
If I use conda activate other-environment it will show the (other-environment) in the terminal, but it won't actually use that environment. At least that is what it looks like because imports from other-environment clearly aren't working.

In the meantime, what happens if you click on "Enter interpreter path..." and type it in there?

I tried to type it, the default interpreter is used instead..

Is this issue still happening with the latest version of the extension?

Yes, a duplicated pythonpath line in the settings.json will still prevent the user from switching python environments. The extension only changes the first occurence, thus the 2nd will undo whatever choice you make. You'll have to find the duplicate and delete it, then it will work again.
Suggestions to fix:

Either delete or comment out all pythonpath lines in the settings except one (sort of intrusive)
When the user changes the environment, change all pythonpath lines except just the first one found in the settings files
Throw a very visible warning when more than one pythonpath line is present

Okay, this is a surprise. There was a doubled pythonpath line in one of the settings files and switching the environments would only change the first occurence

This seems like a different issue than the first one, can you please create a separate issue for this? @Silun
@katsar0v Regarding the original issue, it seems like the dup of https://github.com/microsoft/vscode-python/issues/16288

Because we have not heard back with the information we requested, we are closing this issue for now. If you are able to provide the info later on then we will be happy to re-open this issue to pick up where we left off.

The solution I found for this issue was to manually specify the Python interpreter path in the workspace configuration (.vscode/settings.json):
{
  "python.defaultInterpreterPath": "${workspaceFolder}/dev/.venv/bin/python",
}

@felipecrs you should also be able to browse to environment via the environment picker if you don't want to write it in your settings.

Well, if I don't write in the settings, everyone using the repository would have to do it by hand. Also, it does not show up in the list of python interpreters when I try to select a different one.
The previous behavior of the extension was to properly figure out my venv, and use it by default. This has been changed, as it seems, and thus it requires me to add this new setting in the repository.

Also, it does not show up in the list of python interpreters when I try to select a different one.

This seems unexpected. Can you set python.logging.level to info in User settings and please provide the logs as mentioned in the issue template.
Logs

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

XXX

Output from Console under the Developer Tools panel (toggle Developer Tools on under Help; turn on source maps to make any tracebacks be useful by running Enable source map support for extension debugging)

XXXX

It has nothing useful:
It has nothing useful:

User belongs to experiment group 'pythonaacf'
User belongs to experiment group 'pythonJediLSP'
User belongs to experiment group 'pythonDiscoveryModuleWithoutWatcher'
User belongs to experiment group 'pythonTensorboardExperiment'
User belongs to experiment group 'pythonSurveyNotificationcf'
User belongs to experiment group 'PythonPyTorchProfiler'
User belongs to experiment group 'pythonDeprecatePythonPath'
User belongs to experiment group 'pythonSortEnvs'
Python interpreter path: /bin/python
Error 2021-07-28 22:11:52: Failed to check if file needs to be fixed [EntryNotFound (FileSystemError): Unable to read file 'vscode-remote://dev-container+5c5c77736c245c5562756e74755c686f6d655c66656c69706573616e746f735c7265706f735c636f6d2e6572696373736f6e2e6273732e636f6d6d6f6e2e6369/home/cicd/.config/Code/User/settings.json' (Error: Unable to resolve non-existing file 'vscode-remote://dev-container+5c5c77736c245c5562756e74755c686f6d655c66656c69706573616e746f735c7265706f735c636f6d2e6572696373736f6e2e6273732e636f6d6d6f6e2e6369/home/cicd/.config/Code/User/settings.json')
at _handleError (/vscode/vscode-server/bin/x64/c3f126316369cd610563c75b1b1725e0679adfb3/out/vs/server/remoteExtensionHostProcess.js:84:167834)
at processTicksAndRejections (internal/process/task_queues.js:93:5)
at async y.readText (/home/cicd/.vscode-server/extensions/ms-python.python-2021.7.1060902895/out/client/extension.js:9:341417)
at async p.doesFileNeedToBeFixed (/home/cicd/.vscode-server/extensions/ms-python.python-2021.7.1060902895/out/client/extension.js:59:589732)
at async /home/cicd/.vscode-server/extensions/ms-python.python-2021.7.1060902895/out/client/extension.js:59:588859
at async Promise.all (index 0)
at async p.getFilesToBeFixed (/home/cicd/.vscode-server/extensions/ms-python.python-2021.7.1060902895/out/client/extension.js:59:588805)
at async p.updateTestSettings (/home/cicd/.vscode-server/extensions/ms-python.python-2021.7.1060902895/out/client/extension.js:59:588432)] {
code: 'FileNotFound'
}
Starting Pylance language server.

/bin/python ~/.vscode-server/extensions/ms-python.python-2021.7.1060902895/pythonFiles/testing_tools/run_adapter.py discover pytest -- --rootdir /workspaces/test-repo -s --cache-clear /workspaces/test-repo/python
cwd: /workspaces/test-repo
/bin/python ~/.vscode-server/extensions/ms-python.python-2021.7.1060902895/pythonFiles/testing_tools/run_adapter.py discover pytest -- --rootdir /workspaces/test-repo -s --cache-clear /workspaces/test-repo/python
cwd: /workspaces/test-repo
Error 2021-07-28 22:11:54: Failed to parse discovered Test [r [Error]: Traceback (most recent call last):
File "/home/cicd/.vscode-server/extensions/ms-python.python-2021.7.1060902895/pythonFiles/testing_tools/run_adapter.py", line 17, in
from testing_tools.adapter.main import parse_args, main
File "/home/cicd/.vscode-server/extensions/ms-python.python-2021.7.1060902895/pythonFiles/testing_tools/adapter/main.py", line 9, in
from . import pytest, report
File "/home/cicd/.vscode-server/extensions/ms-python.python-2021.7.1060902895/pythonFiles/testing_tools/adapter/pytest/init.py", line 7, in
from ._discovery import discover
File "/home/cicd/.vscode-server/extensions/ms-python.python-2021.7.1060902895/pythonFiles/testing_tools/adapter/pytest/_discovery.py", line 8, in
import pytest
ModuleNotFoundError: No module named 'pytest'

at ChildProcess.<anonymous> (/home/cicd/.vscode-server/extensions/ms-python.python-2021.7.1060902895/out/client/extension.js:9:435329)
at Object.onceWrapper (events.js:422:26)
at ChildProcess.emit (events.js:315:20)
at maybeClose (internal/child_process.js:1048:16)
at Process.ChildProcess._handle.onexit (internal/child_process.js:288:5)]

/bin/python -c "import pytest"
/bin/python -c "import pytest"
Error 2021-07-28 22:11:54: Python Extension: displayDiscoverStatus [r [Error]: Traceback (most recent call last):
File "/home/cicd/.vscode-server/extensions/ms-python.python-2021.7.1060902895/pythonFiles/testing_tools/run_adapter.py", line 17, in
from testing_tools.adapter.main import parse_args, main
File "/home/cicd/.vscode-server/extensions/ms-python.python-2021.7.1060902895/pythonFiles/testing_tools/adapter/main.py", line 9, in
from . import pytest, report
File "/home/cicd/.vscode-server/extensions/ms-python.python-2021.7.1060902895/pythonFiles/testing_tools/adapter/pytest/init.py", line 7, in
from ._discovery import discover
File "/home/cicd/.vscode-server/extensions/ms-python.python-2021.7.1060902895/pythonFiles/testing_tools/adapter/pytest/_discovery.py", line 8, in
import pytest
ModuleNotFoundError: No module named 'pytest'

at ChildProcess.<anonymous> (/home/cicd/.vscode-server/extensions/ms-python.python-2021.7.1060902895/out/client/extension.js:9:435329)
at Object.onceWrapper (events.js:422:26)
at ChildProcess.emit (events.js:315:20)
at maybeClose (internal/child_process.js:1048:16)
at Process.ChildProcess._handle.onexit (internal/child_process.js:288:5)]

conda --version
/bin/python ~/.vscode-server/extensions/ms-python.python-2021.7.1060902895/pythonFiles/testing_tools/run_adapter.py discover pytest -- --rootdir /workspaces/test-repo -s /workspaces/test-repo/python
cwd: /workspaces/test-repo
/bin/python ~/.vscode-server/extensions/ms-python.python-2021.7.1060902895/pythonFiles/testing_tools/run_adapter.py discover pytest -- --rootdir /workspaces/test-repo -s /workspaces/test-repo/python
cwd: /workspaces/test-repo
Error 2021-07-28 22:14:13: Failed to parse discovered Test [r [Error]: Traceback (most recent call last):
File "/home/cicd/.vscode-server/extensions/ms-python.python-2021.7.1060902895/pythonFiles/testing_tools/run_adapter.py", line 17, in
from testing_tools.adapter.main import parse_args, main
File "/home/cicd/.vscode-server/extensions/ms-python.python-2021.7.1060902895/pythonFiles/testing_tools/adapter/main.py", line 9, in
from . import pytest, report
File "/home/cicd/.vscode-server/extensions/ms-python.python-2021.7.1060902895/pythonFiles/testing_tools/adapter/pytest/init.py", line 7, in
from ._discovery import discover
File "/home/cicd/.vscode-server/extensions/ms-python.python-2021.7.1060902895/pythonFiles/testing_tools/adapter/pytest/_discovery.py", line 8, in
import pytest
ModuleNotFoundError: No module named 'pytest'

at ChildProcess.<anonymous> (/home/cicd/.vscode-server/extensions/ms-python.python-2021.7.1060902895/out/client/extension.js:9:435329)
at Object.onceWrapper (events.js:422:26)
at ChildProcess.emit (events.js:315:20)
at maybeClose (internal/child_process.js:1048:16)
at Process.ChildProcess._handle.onexit (internal/child_process.js:288:5)]

I couldn't find the revelant code sections, but in my opinion this plugin should not only search for Python in the paths /bin/python, /usr/bin/python etc.
Info 2021-07-28 22:23:17: Found: /bin/python --> /usr/bin/python3.9
Info 2021-07-28 22:23:17: Found: /bin/python3 --> /usr/bin/python3.9
Info 2021-07-28 22:23:17: Found: /bin/python3.8 --> /bin/python3.8
Info 2021-07-28 22:23:17: Found: /bin/python3.9 --> /bin/python3.9
Info 2021-07-28 22:23:17: Found: /usr/bin/python --> /usr/bin/python3.9
Info 2021-07-28 22:23:17: Found: /usr/bin/python3 --> /usr/bin/python3.9
Info 2021-07-28 22:23:17: Found: /usr/bin/python3.8 --> /usr/bin/python3.8
Info 2021-07-28 22:23:17: Found: /usr/bin/python3.9 --> /usr/bin/python3.9

but also try to just pick python from the $PATH, which would result in
Info 2021-07-28 22:23:17: Found: python --> /path/to/virtualenv/bin/python

I believe I may have found a workaround: VS Coded seems to be confused by the fact that the python executable is a symlink to the OS interpreter:
ls -l bin/python3.7
bin/python3.7 -><EMAIL_ADDRESS>
If I recreate the virtualenv with the --copies flag
python3.7 -m venv --copies .

that interpreter is a copy, not a symlink
ls -l bin/python3.7
bin/python3.7

and VS Code can successfully detect the interpreter.


Tag latest not found in repository docker.io/my_repo

Not sure what is the problem..:-|
I downloaded the latest docker toolbox.
Trying to add my repo, but got error
Tag latest not found in repository docker.io/my_repo
I also can't connect to docker from command line on my mac, I get:
Cannot connect to the Docker daemon. Is the docker daemon running on this host?
Can you please help?

I would make sure that your repo has a tag of latest - Additionally use the docker quickstart terminal to be able to talk to docker from the CLI.

Thanks! worked for me!


Certain Objects not Fully Synchronized

When pushing a package from one system to another we are noticing that changes are not fully synched. In fact we can just keep pulling and it constantly shows a list of 5 objects:

When we look at the diff we see that it is the description of the class in english which is not being synchronized. This is the diff when we try to push back from the target system:

It shows, in effect, that the target system (on the left) does not have the following:
   <DESCRIPTIONS_CLASS>
    <SEOCLASSTX>
     <LANGU>E</LANGU>
     <DESCRIPT>Fehlerklasse</DESCRIPT>
    </SEOCLASSTX>
   </DESCRIPTIONS_CLASS>

No matter how many times we pull into the target system it always wants to pull the same objects as it notices they are different:

Function Group:

In the case of the TOBJ Object it seems like the description of a table is not being transferred:

One possible explanation is that we moved the table ZPMR_DATE from a package ZPMR_DATE_SRV which is in the Repo to a package ZPMR_DATE_SRV_DB which isn't. It seems SAP did not move the TOBJ correctly across:

I noticed that after and upgraded both ABAPGIT_STANDALONE reports. We are still seeing the endless pull cycle constantly showing diffs:

okay, try reproducing in a public repository, and open two separate issues one for DSYS and one for CLAS.
DSYS: it might have a bug, dont think theres many people using it
CLAS: can be a translation issue. Personally I recommend doing all development in English, and not translate technical descriptions (only translate stuff that affects possible business users)

DSYS: abapGit keeps the version history of the object. If the object was changed in the target system (more times than in the source), then there will be diffs.
CLAS: I tested with https://github.com/abapGit-tests/CLAS branch "german" and there are not diffs. Please provide a reproducible example repo.


fix broken images for token based authentications after doc update

the image links were broken due to the big doc update

@danamlewis can you merge this?


a bug in ./fastreid/modeling/backbones/regnet/regnet.py

Dear Author,
There is a bug in ./fastreid/modeling/backbones/regnet/regnet.py - line 562.
It is currently written as:
'4000x': 'fastreid/modeling/backbones/regnet/regnety/RegNetX-4.0GF_dds_8gpu.yaml',
But 'regnety' should be rectified as 'regnetx', like this:
'4000x': 'fastreid/modeling/backbones/regnet/regnetx/RegNetX-4.0GF_dds_8gpu.yaml',
I guess you may need to fix this error. Many thanks!

Thanks very mush and already fixed it.


board: Run ./tools/refresh.sh --silent all

Summary
Follow up https://github.com/apache/incubator-nuttx-apps/pull/884
Impact
Should same as before
Testing
Pass CI

We had an old issue about disabling some of these commands for security reasons, is this to address the same issue?

We had an old issue about disabling some of these commands for security reasons, is this to address the same issue?

No, this patch try to reduce the image size with default setting when DEFAULT_SMALL is enabled. mb/mh/mw always disable by default regardless DEFAULT_SMALL setting.

apache/incubator-nuttx-apps#884 disable some feature by default, so this PR remove these options from defconfig which make the binary size increase temporarily and make CI fail like this:
arm-none-eabi-ld: /github/workspace/sources/nuttx/nuttx section `.text' will not fit in region `flash'

Please ignore this false alarm.

All right.  I already merged the apps, this needs to be merged too and we need to keep an eye on the pipeline run on master to make sure that everything is as expected.


[CBR-439] Avoid applying uninteresting blocks

Description
See this other PR for the description. It's formally the same as #3682 but it drops the idea of switching to fork based only on the block height but keeps the useful optimisation of skipping uninteresting blocks if we are not within k from the tip.
Linked issue
https://iohk.myjetbrains.com/youtrack/issue/CBR-439
Type of change

[x] 🐞 Bug fix (non-breaking change which fixes an issue)
[ ] 🛠 New feature (non-breaking change which adds functionality)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)
[x] 🏭 Refactoring that does not change existing functionality but does improve things like code readability, structure etc
[ ] 🔨 New or improved tests for existing code
[ ] ⛑ git-flow chore (backport, hotfix, etc)

Developer checklist

[x] I have read the style guide document, and my code follows the code style of this project.
[x] If my code deals with exceptions, it follows the guidelines.
[x] I have updated any documentation accordingly, if needed. Documentation changes can be reflected in opening a PR on cardanodocs.com, amending the inline Haddock comments, any relevant README file or one of the document listed in the docs directory.
[ ] CHANGELOG entry has been added and is linked to the correct PR on GitHub.

Testing checklist

[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

QA Steps

Screenshots (if available)

This is quite dangerous to do willy-nilly, it requires careful consideration. Closing this on for now.


Need help to create XML node for GET request

Hello Michal Vasko!
This is just a request for help because I cannot find and answer in Google or Libyang documentation.
I'm working on o-ran-performance-management.yang and cannot create XML node for YANG node "/o-ran-performance-management:performance-measurement-objects/measurement-capabilitites/epe-stats-objects[measurement-object='TEMPERATURE']/component-class".
The field is of identityref type.

There are few similar identities in iana-hardware.yang. But when I try to create the leaf-list of size 1 with identity "ianahw:chassis" inside GET request callback I'm gettint error 7  (LY_EVALID - Validation failure) from lyd_new_path function.
My code:
struct lyd_node *capabilities_p = NULL;
ret = lyd_new_path(*parent, NULL, "/o-ran-performance-management:performance-measurement-objects/measurement-capabilitites/epe-stats-objects[measurement-object='TEMPERATURE']/measurement-object", "TEMPERATURE", 0, &capabilities_p);
if (ret != LY_SUCCESS) {
	syslog(LOG_ERR, "Failed to add 'measurement-object' field; error %d", ret);
	return SR_ERR_LY;
}
ret = lyd_new_path(capabilities_p, NULL, "/o-ran-performance-management:performance-measurement-objects/measurement-capabilitites/epe-stats-objects[measurement-object='TEMPERATURE']/component-class", "chassis", 0, NULL);
if (ret != LY_SUCCESS) {
    syslog(LOG_ERR, "Failed to add 'component-class' field; error %d", ret);
    return SR_ERR_LY;
}

The first one is created successfully. But the second one not.
Could you please suggest me how to do that (to create the 'component-class' field correctly).

To not require any additional information and be unambiguous, all API functions expect JSON format of all values/paths. That means that you must use the module name instead of import prefix.
iana-hardware:chassis

That works! Thank you very much! Will keep in mind this format.


Ability to open multiple NWB files at the same time

Should be fixed by #17

But we need to improve the UI to make it more usable

I think we should create several specific sub-window (of Box type I think?) to create a proper layout. For example, we could create a sub-window for the tree view of the files and their groups / datasets, so it will be easy to add a scroll bar on this area. The rest of the window could be dedicated to the plots. Maybe we could even create one Box per file and then the plots inside each Box. WDYT @anilbey ?

A very good idea to have a sub-window and scroll bar.


Very slow start on big files with multiprocessing

It looks like all the child processes are spinning up and churning through point generation, and then piling up behind the yield that pipes everything to output.
I'm not sure how to get around that, since ultimately there's only one output file.

Would it be crazy to write to a directory, instead of a single file, and then combine files?


Can't set x/y for a sticker when adding a story

Form

[x] I've searched the Issues
[x] I've read the basic concepts
[x] I'm using the latest version

Question
I'm using the latest version of instagram-private-api. I use a code sample from here: https://github.com/dilame/instagram-private-api/blob/master/examples/upload-story.example.ts
but with a little modification here:
stickerConfig: new StickerBuilder()
			.add(StickerBuilder.hashtag({
				tagName: 'insta',
				x: 0.5,
			}).center())

It doesn't allow me to use x, y, width, height parameters and says following:

Argument of type '{ tagName: string; x: number; }' is not assignable to parameter of type 'Pick<HashtagSticker, "tagName">'.
Object literal may only specify known properties, and 'x' does not exist in type 'Pick<HashtagSticker, "tagName">'.ts(2345)

I have no any changes in your example.
What am I doing wrong?

In your case, setting x wouldn't change anything. You can open a PR and change the behavior. Note: this only affects Typescript users.

@Nerixyz but it doesn't the root of a problem. Because I get same error in this case also:
.add(StickerBuilder.hashtag({
				tagName: 'insta',
				x: 0.5,
			}))


Fix issue with for loop initializers and block binding

We did not correctly take the initializers in the for loop
into account. We need to stop the alpha renaming if the name
we are renaming is in a initializer that is a lexical declaration.
Fixes #1637

@johnjbarton PTAL

LGTM


Tests for model (un)pickle

We should add tests for saving and loading pickled models (something like what we already have for XGBoost) for all algorithms (pogs based, kmeans, tsvd, pca) to see if we can actually save and load all our models.

We don't for gpu kmeans and gpu svd.  xgboost has a special hook so it copies over (in C) any references in python.  We can do the same.

@pseudotensor do we really need that, though? At least for KMeans the only thing we need are the centroids, which we have in Python as a numpy array so we could just pickle and unpickle that. After unpickling we can just pass it (as we do now) to the C backend, no?

For example seems to work out of the box for KMeans (verbose logs to show it's running on GPUs):
>>> import pickle
>>> import h2o4gpu
>>> import numpy as np
>>>
>>> X = np.array([[1.,1.], [1.,4.], [1.,0.]])
>>>
>>> model = h2o4gpu.KMeans(verbose=100, n_clusters=2,random_state=1234).fit(X)

Using GPU KMeans solver with 2 GPUs.

Using h2o4gpu backend.

Using GPU KMeans solver with 2 GPUs.

Detected np.float64 data
2 gpus.
Copying data to device: 1
Copying data to device: 0
Threshold triggered. Terminating early.
  Time fit: 0.00288296 s
Timetransfer: 0.0531921 Timefit: 0.00288296 Timecleanup: 0.00114107
>>> model.cluster_centers_
array([[1., 1.],
       [1., 4.]])
>>>
>>> pickle.dump( model, open( "save.p", "wb" ) )
>>> unpickled_model = pickle.load( open( "save.p", "rb" ) )
>>> unpickled_model.cluster_centers_
array([[1., 1.],
       [1., 4.]])
>>> model.predict(X)

Using GPU KMeans solver with 2 GPUs.

Detected np.float64 data
Detected np.float64 data
2 gpus.
array([0, 1, 0], dtype=int32)
>>> unpickled_model.predict(X)

Using GPU KMeans solver with 2 GPUs.

Detected np.float64 data
Detected np.float64 data
2 gpus.
array([1, 0, 0], dtype=int32)

Yes, should be easy (or already true) that for kmeans easy as only thing fit does is find centroids.

@pseudotensor yes, thought it would work out of the box for all our models since we copy all the necessary data from C to Python, but @wenphan noticed that for POGS based models it was having problems pickling. From the log it had to do something with . CDLL and/or ctypes so maybe for POGS we need to do some more work but hopefully kmeans and svd are already good.

We should move forward on dropping pogs anyways.  I have a gblinear wrapper we can use as base line that does lambda search with warm start.  We can use the rest of the CV fold stuff but do it in python instead of C.  Probably easiest.

@pseudotensor yes once @RAMitchell impl is stable enough I'm 100% for removing POGS altogether from the codebase.


292-Move-to-Tonel-Format-and-use-a-src-directory

Fixes #292

ok, I got green line (hacked for 6.1, but well...)


Add "Report Violation" menu option

Description
Fixes #5024
What changes did you make and why?
Add the "Report Violation" menu option
Tests performed
Tested betaDebug on Mi A2 with API level 29
Screenshots

Unfortunately today I broke my environment and with my slow Internet building it back will take days, but from the screenshots it looks good!

Unfortunately today I broke my environment and with my slow Internet building it back will take days, but from the screenshots it looks good!

You can directly download the generated APKs from here :)
https://github.com/commons-app/apps-android-commons/actions/runs/2796899450

It works thanks!
Would itwbe possible to show the menu also when going via leaderboard?


crash when posting malformed request to upload avatar

TypeError: Cannot read property 'path' of undefined
at ImageProcessor.handleAvatarUpload

@gmarinov please include steps to reproduce


LF-20449: Bumping SDK version

https://livefyre.atlassian.net/browse/LF-20449

Replaced by: https://github.com/Livefyre/streamhub-wall/pull/108


Uglifyjs and Uglifycss

ISSUE TYPE

Feature request

SUMMARY

It will be great to have these tools for minifying and packaging assets for production deployment, node and rpm is bundled. I have tried installing but of course, the installation is lost anytime the machine is restarted
Goal

Package assets for deployment

Until the feature is considered, you can create a bash script inside ~/devilbox/bash/ to install these whenever you spin a new container.
I use this to config/initialize git, and run my composer/yarn/etc init scripts.

Thanks @dimkasta
I have just done this and it works
HOWEVER -- and please correct me if I am missing something.
It will be very helpful to get back into the state that has been setup without doing new installations all the time. A similar example is tasks saves to cron. Do you think @cytopia will consider a way to save and restore state such that installations to the box and configurations like those saved for cron wouldn't need to be setup afresh?
What do you think?

npm, composer etc packages (including their dependency management, versioning and configuration) are project specific, so their place is in the configuration files of your package managers in your project folder.
That's what makes the immutability of docker images work in the first place.

@dimkasta ,
We're not disputing the value of the immutability of docker images.
The context of my questions is that I shut down my development computer every evening when I'm done and restart it every morning when I resume work, I need a simple way to reload the environment without going through any new installation steps.
I have cron jobs that mirror what will be in production.
I use uglifyCss and uglifyJs to compress assets.
The bash scripts work and will serve till the feature is considered, thanks to your recommendation.
But the script is a work around. I am looking at the purpose of devilbox itself, it's built on docker to take advantage of the immutability you speak of but I suggest that it makes sense for one of its value add be restoring the configured development state on restart.
We can have docker immutability as well as a fixed development environment without reinstalls and reconfiguration on each restart

You touch two very different points.
About the tools, even if Cytopia decides not to include them, there is a very easy way to make them persistent. It is to install them locally on your project directory, not globally. That way they are available to the container every time you boot it. You need to re-install them only if you clone your code on a new machine (assuming you do not include them in your source management). That's where tools like NPM, composer, phive etc come very handy.
Cron settings on the other hand is something very different. You are still thinking of state persistence in terms of virtualization where we would save the machine state and restart it next time you needed the machine. The disadvantage in this approach is that state is temporary and local to your machine. If you wanted to start the environment on your brand new laptop or give it to a colleague, you would have to manually re-enter the commands or copy the state itself along with the rest of the project.
Initialization scripts on the other hand is the way to properly do this. It's not a work around and has many advantages. Mostly because it makes environment initialization part of your code and includes it in your source management tool where you can easily and/or automatically reapply or fine tune it. It documents the process and makes it easily reproducible by your or anyone else that might have to set a new machine, for development or for production. Especially in the not-so-near future when you would have forgotten the structure of the project.
A different approach would be to extend devilbox to create your own images that already include the initialization. But it's hardly worth the effort. You would have to manage docker files and images on top of the initialization scripts.

@Alephel @dbd5 You can install whatever tools inside each of the Docker container. If you simply stop them docker-compose stop and do not remove them docker-compose rm -f, then you state is preserved and you can just start it up again and still have all the installed tools available.


Things to fix on 1st/2nd/3rd day of advanced course

Lambda part might be hasty
Slide 243 (lambdas) goes to unique_ptr, custom deleters and cuda whereas smart pointers will be introduced much later.
Template part is probably too complicated as well
Didn't have enough time for RNGs
Why are our spans not const in the "Usage" slide?

During concurrency:

Typeset decltype using \texttt
Mention std::thread::detach? Detached threads are killed after main returns! That's important to know.
Very nice to show the AdePT example for std::async. should we put a simplified version on the slides?
Better example for thread_local? Where it actually has an advantage over a local variable.
thread_local slide: jthread::join() could be replaced by wrapping an appropriate scope around and let the jthread join in dtor.
condition_variable notify_all vs. notify_one

What I collected myself when rehearsing:

explain left vs right fold + associativity
do we ever explain lvalue and rvalue?
add a slide on expansion loci and that we do not need a comma before ...
constexpr if deserves a bit more demonstration of its power

my notes from the first day:

avoid putting a using namespace ...; declaration into a header file, especially not on the global namespace!
Mentioned on slide Pointers vs References: std::optional<T&>. Unfortunately, this is not allowed :/ boost::optional allows it.
range-based loop slide: show what the compiler generates under the hood! cppinsights?
the operator() of a lambda is also inline by default
there was a question about lifetime extension of a temporary by a const reference, maybe add that to the slides?
discussion of = default; on special member functions. it does not force the generation. compiler only generates them if they are not ill-formed
static_assert is available since C++11, only the version without a message is C++17
MyNS is probably not the best choice for a namespace.
Hidden Friends and ADL "Accidental conversion example" is not clear enough. maybe split to another slide and contrast a working/non-working example?
add a link to https://cdecl.org/
Guilherme: s/Real life example/Example/ in this slide (some constexpr slide)
noexcept specifier slide: put guideline (Use noexcept on leaf functions...) in a good practice box
noexcept operator: add an example definition of f above the constexpr bool callCannotThrow = noexcept(f())
we should finally add a slide on value categories (r-value, l-value, x-value)


Error: Couldn’t find Tailwind screen: last

I added prefix last-child in my project but it's have error
StyledNavbar.Link = styled(Link)`
  ${tw`no-underline text-green-400 active:text-purple-600 last:text-purple-600`}
`;

tailwind.config.js
 variants: {
    textColor: ['responsive', 'hover', 'last'],
  },

That's not a Tailwind issue. It is probably due to tailwind.macro being out of date and not maintained anymore.

@hacknug Do you have suggest for using tailwind , style component together?

There's a few starters listed in this repo that might help you. if none of those work for you, try asking on Discord. There's probably someone who will be able to help you with that :+1:

Just saw this on a couple other issues. Maybe they'll be helpful:

https://github.com/aedificatorum/cra-tailwind-emotion-starter
https://github.com/talensjr/gatsby-theme-tailwindcss/blob/master/theme/README.md

Yeah this is better suited for the tailwind.macro project, sorry 😞


add option to override the content class of list/create/edit

This pull request adds the option to override the hardcoded bootstrap classes for the list/create/edit view.
You can override it globally by using
'create_content_class' => 'col-md-12', in the config-file
or override it per crud by using
$this->crud->setCreateContentClass('col-md-12');
list and edit accordingly.
Should be a non-breaking change but very useful to us.

Hey @tabacitu
Any chance to get this merged soon? Would be very helpful for a current project...
Thanks for checking! :-)

Repo gone - closed.
Available here: #1659


chore(release): v1.0.0 🎉

:robot: I have created a release *beep* *boop*
1.0.0 (2022-11-24)
Features

context: define cache at context level, to ensure all fns share same cache for interactions (5cf727b)
init: with-remote-state-caching working for query caching (d74288d)
triggers: support adding invalidation and update triggers to queries (565f1dd)

Bug Fixes

triggers: get valid keys of cache directly from cache (9283643)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ehmpathy/with-remote-state-caching/releases/tag/v1.0.0 :sunflower:


Feature request: Installation of packages

We can provide a command to install packages like Go language itself does.
When we do a go get, it'll run git clone in the backend. While installing ST3 plugins, you have to clone the plugin into a specific folder, we can automate that with a commandline utility which comes by default in Lime.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/36947333-feature-request-installation-of-packages?utm_campaign=plugin&utm_content=tracker%2F15716642&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F15716642&utm_medium=issues&utm_source=github).

There's a thing called Package Control for ST3 that automates a lot of that stuff, but I believe making a package manager first class in Lime would be a good thing.

@erbridge The beauty is that we can store the packages in github, give a repo name to link to the package name
and fetch and install the packages from github, so we don't need to have a website storing packages :-)

Exactly. See https://packagecontrol.io.

If ST already had this feature then why do we need to manually install packages?

In ST3, you first install Package Control, then use the Command Palette to install packages. Ideally Lime would be able to do it without Package Control, IMO.

If ST already had this feature then why do we need to manually install packages?

I'm not sure what you're referring to.

you first install Package Control

So basically you have to install the package manager manually, after which you can use command palette.
Lime would have this feature by default, that's a great news. I honestly feel that softwares these days are platforms of their own, so having a better UX/UI to the part which makes it a platform is very important!
I wanted to install the Go language plugin for sublime, I did not find a package manager in st3. That's what I was referring to.

Ah. Get GoSublime using Package Control.

Yep, I want to contribute more code, at a loss at what to write :-) shall I build this package manager? or build more commands?

This is very low priority. Higher priority is making the QML frontend work. Failing that, you could look at commands, but leave this until later.

To be honest, I tried looking into the front end and I couldn't understand anything :D

This is duplicate of https://github.com/limetext/lime/issues/542.

@zoli We can close that issue, because it should be in the backend repo right?
Also did you get a chance of looking at the issue migration script?

it should be in the backend repo right?

I don't think so, It's not related to backend. If we are going to have a package manager we'll have a separate repository for it and until then lime meta project is a proper place for gathering ideas and discussing features.

Also did you get a chance of looking at the issue migration script?

Not yet.

I thought it'd be a part of the backend.


frdrpc: convert frdrpc into a go module

This commit transforms the frdrpc subpackage into a versioned module. By doing so, projects that rely solely on the generated RPC code from the proto files will benefit from significantly reduced dependencies during compilation.
Pull Request Checklist

[ ] Update MinLndVersion if your PR uses new RPC methods or fields of lnd.

@bhandras, remember to re-request review from reviewers when ready

@bhandras, remember to re-request review from reviewers when ready

@bhandras, remember to re-request review from reviewers when ready

@bhandras, remember to re-request review from reviewers when ready

@bhandras, remember to re-request review from reviewers when ready

@guggero: review reminder


Removing year end messaging

Removing messaging that encouraged candidates to act speedily to make sure they could get their application in before the September deadline.
Trello card
https://trello.com/c/vViTOgMo/1904-update-years-and-confirmation-page-on-get-an-adviser-form

Hi @peteryates - my page rewrites have been blocked - do I just need someone to review?

Looks good to me - both standard content and returner content.
Happy for this to be approved/merged @peteryates if you have finished your changes?


sdk: extend dev docs with data persistence section

Closes: #2157
Short description
First draft. Open for any suggestions. Don't hesitate to do language improvements on your own.

I'm rebasing now to integrate the fixup commits.


DAWN-387 ⁃ port wallet tests

pull request link: https://github.com/EOSIO/eos/pull/1578/
Fix most of the tests cases in wallet tests except transfer related test cases, as the transfer of native contract not working at the moment.

➤ Jiaen Du commented:
output:
Fredriks-MacBook-Pro:build jiaendu$ ./tests/chain_test --report_level=detailed --run_test=wallet_tests
Random number generator seeded to<PHONE_NUMBER>
Running 2 test cases...
3598718ms thread-0   wallet.cpp:180                save_wallet_file     ] saving wallet to file test
3598719ms thread-0   wallet.cpp:180                save_wallet_file     ] saving wallet to file wallet_test.json
3598720ms thread-0   wallet.cpp:180                save_wallet_file     ] saving wallet to file ./test.wallet
3598720ms thread-0   wallet.cpp:180                save_wallet_file     ] saving wallet to file ./test.wallet
3598721ms thread-0   wallet.cpp:180                save_wallet_file     ] saving wallet to file ./test.wallet
3598721ms thread-0   wallet.cpp:180                save_wallet_file     ] saving wallet to file ./test2.wallet
3598722ms thread-0   wallet.cpp:180                save_wallet_file     ] saving wallet to file ./test2.wallet
Test module "Master Test Suite" has passed with:
2 test cases out of 2 passed
42 assertions out of 42 passed
Test suite "wallet_tests" has passed with:
2 test cases out of 2 passed
42 assertions out of 42 passed
Test case "wallet_tests/wallet_test" has passed with:
  13 assertions out of 13 passed

Test case "wallet_tests/wallet_manager_test" has passed with:
  29 assertions out of 29 passed

I've pushed another change to resolve the merge conflict. Please see https://github.com/EOSIO/eos/pull/1578
All the wallet test cases run successfully in my laptop:
Test suite "wallet_tests" has passed with:
2 test cases out of 2 passed
42 assertions out of 42 passed
Test case "wallet_tests/wallet_test" has passed with:
  13 assertions out of 13 passed

Test case "wallet_tests/wallet_manager_test" has passed with:
  29 assertions out of 29 passed


URL-based manifest fetching (instead of on-disk only)

This allows for manifestPath to be a git url, so one can just point to a git repository on a CDK stack.
The git repository is cloned to a temporary directory and then compiled.
/cc @mmalenic

I've also added the ability to specify a branch name when cloning and override the cached tmp directory logic to add more flexibility - and updated corresponding docs for this.

I'm not completely sure what's going on with the CI builds. Can you run npx projen install:ci locally and print the output here? I'm looking into it, but it doesn't look like projen is being installed correctly. I wonder if the problem is all the deprecated GH actions that this repository still uses.

I get:
👾 install:ci | yarn install --check-files --frozen-lockfile
yarn install v1.22.22
[1/4] Resolving packages...
success Already up-to-date.
Done in 0.68s.

It looks like the GH action is trying to use npm rather than yarn? Not sure if that's the issue.
This has added a packageManager field to package.json, that might help.

This is something related with GitHub actions itself. Let me try to figure out, and I'll rebase this PR when I get it working again.

I got the build working in main, but there is a merging conflict with this branch. Can you merge main into this branch and fix the conflict? The release process is still broken, but I'll fix it once this PR is merged into main.

I like this idea. But I'm not totally sold on using the manifestPath this way. An immediate question that comes to my mind is what happens if the manifest in the remote repository is not in the root. I think we need more explicit options just to fetch remote sources.
Since working in this repo is still a little brittle, I'm going to go ahead and merge this contribution as it is. I will be making some changes before a new release to use more explicit options. I'll tag you in a PR with those changes.


Implement changelog plugin

This PR adds changelog plugin initial implementation.
Please check README.md on what this plugin exactly does and how to use it.

Also forgot to mention that it would be great to assign group (e.g. changelog) for each task :)

Thank you for the review and great suggestions 😊
I've addressed all comments and also made it possible to create any rule during the plugin configuration. That's much more simple and flexible: https://github.com/Doist/changelog-gradle-plugin/pull/1/commits/ee34bb21e5e6b15ca34420b9bc76fd57305f36fe

Have you thought about making commitChangelog task dependent on checkChangelog task?

I would leave it for the plugin user to decide when to run the check. For example, if they want to append a pull request link to the pending entries, this could validate some of the rules 😅

Thank you for another review 😊
I've addressed all comments and pushed two commits.

Thanks! Update the code and will now merge 👍


Packagist hates Symfony/Security, expects symfony/security

Crayfish commons uses a capitalized Symfony/Security, when it should be all lower-case.  This happens in a few branches that need to be updated or deleted.

Resolved via https://github.com/Islandora/Crayfish-Commons/commit/3d6e549c9ca0af08a053631694fe7ea04848fce9


Updates for ARM image and MP AMI listing CFT

Updates for ARM image and MP AMI listing CFT

/do-e2e-tests

/do-e2e-tests

Test passing locally and only failed in one region. Failed in that region for subscription issue.

/do-e2e-tests


Skype for business bot picture configuration issue

Bot Info ithelpchatbot

Bot handle:
App ID:
SDK Platform: .NET
SDK Version: C# with Luis
Active Channels: Skype for business
Deployment Environment: Azure Bot Service

Issue Description
I would like to configure a picture in the Skype for business bot to display for the end users. I configured an image in the bot active directory, enabled exchange license and configured the image again but without success
Any workaround to enable this image ? Can we switch this Application user to normal user to configure an image?
Code Example

Reproduction Steps

Expected Behavior

Actual Results

Any ideas how to solve this ?
I escalate to Office 365 but they push back to Azure support..

I will reach out to the sfb team for you today

Did you get an update from them?

No response yet, I'll poke again today.

Thank you @JasonSowers

Any news about this yet? We're experiencing the same problem. It's not vital but it's frustrating because there are some articles clearly showing screenshots from Skype for business where both user thumbnail and user status are clearly shown. (E.g. https://chatbotsmagazine.com/enterprise-chatbots-on-skype-for-business-af635f346f5c)

@EricDahlvang can we please add pointers to relevant docs and resolve?

The Bot Framework Skype for Business Channel is in Developer Preview and does not support bot display pictures at the current time.

From here: https://docs.microsoft.com/en-us/skype-sdk/skype-for-business-bot-framework/docs/bot-common-errors
Bots on Skype for Business are still in Preview.

https://stackoverflow.com/questions/44451476/bot-frameworkskype-for-business-how-can-i-set-a-bots-icon-picture-in-a-chat


Fix(eos_designs)!: Build VRF and iBGP peering in overlay if MLAG peer has services defined

Change Summary
Fix: Build VRF and iBGP peering in overlay if MLAG peer has services defined
Related Issue(s)
Fixes #3804
Component(s) name
arista.avd.eos_designs
Proposed changes

How to test

Checklist
User Checklist

N/A

Repository Checklist

[x] My code has been rebased from devel before I start
[x] I have read the CONTRIBUTING document.
[ ] My change requires a change to the documentation and documentation have been updated accordingly.
[x] I have updated molecule CI testing accordingly. (check the box if not applicable)

@philippebureau @aristanetworks/avd-maintainers
TL;DR: The penalty hit of 70 to 80% in the fact phase just to solve the corner case is not worth it. Use the workaround described in the issue.
So after careful consideration we have decided with @ClausHolbechArista to close this PR and deem it not feasible at the time with the existing code structure.
To be able to know which VRFs are in my MLAG peer I need to compute the filtered_tenants in the fact phase which is taking a lot of extra time. We have observed a 70-80% time penalty during the fact phase on the repo which is not something that we think is a good solution to impose on every AVD users for such a corner case.
I will remove the issue from AVD 5.0 milestone and we will take another stab at it later once the code structure has changed.


[router] Add metric for stacked\failed txns

Type: feature request
Currently to get information about stuck\failed txns i need to perform request to subgraph so i use custom exporter.
This is critical metric, in future routers can be slashed because of failed tnxs.
Currently, if a router fails to submit a user's transaction to chain (for whatever reason), then the user's funds are "stuck" until either manual debugging occurs or the transfer expires (after 72 hours). While we plan to penalize cases like this in the future via slashing, in some cases txs can get stuck simply because of irregular chain conditions.

#572 contains updated metrics to address the number of transactions in given statuses.
The metric of interest would be number of active transactions (i.e. transactions that you should take action on, but haven't for some reason). and that can be counted by:
sender_prepared - sender_fulfilled - sender_cancelled

sender_prepared -- these are transactions where you have won the bid and the user has deposited funds
sender_fulfilled -- these are transactions that are fully completed and require no further action
sender_cancelled -- these are transactions that were completed and should have all funds released


Unsubscribe Error After Record is Deleted

Scenario:
Client A (JS) is subscribed to the a record. Client B (Server, PHP) sends and event to client A which initiates a process that will in part discard the record. After the event is sent, Client B deletes the record.
Client A calls discard, before receiving deleted event from server. By the time discard method gets processed the record does not exist and the server returns an error to the client.
Its possible to work around this, by letting the delete event process instead of calling discard, but the code in my case is used in scenarios were the record is deleted and when its not. So the event would have to differentiate the outcome of the record.
To keep things simpler, it would be better to have the server not send the error at all. Is there a purpose for sending an error on an unsubscribe if the client is already unsubscribed?. It seems to me that due to the asynchronous nature of the system, that we should simply just acknowledge the unsubscribe and just not do anything rather than return an error.
If the message has a purpose with other uses of the subscription registry, I would propose using the silent parameter to squelch the error.

@datasage thanks for raising this. This fix is in version 2.3.0


feat: add Dutch translations

Was not fully sure on the 'force' translation, so I translated it as 'group'.

Thanks for the contribution. I have added your translations to Crowdin. Please direct any further contributions there!


Keep Support For Coffeescript With PhantomJS 2.0

To my horror, PhantomJS 1.9 stopped working for me on a certain website. To my delight, PhantomJS 2.0 fixes it. But then to my horror again, apparently PhantomJS has dropped support for Coffeescript.
https://github.com/ariya/phantomjs/issues/12410
It seems my choices are:

Add add a compile step (with cake, I guess) on deployment. In development, manually compile after every code change (ouch!).
Convert my thousands of lines of existing Coffeescript to JS, and leave Coffeescript behind (double ouch!).

I guess there is a third choice: somehow compile at runtime. I guess I can find a way to do this in my app before calling CasperJS. However, I wonder if there is way to add this step to CasperJS. It would invoke whenever the file is Coffescript and PhantomJS is >= 2.0.
Is that is possible? Is it desired? If desired, where would the code go? I'm guessing somewhere around here?
I would be willing to take this on if it's desired. I must solve it for myself asap.

For workaround number 1 you could use something like grunt-contrib-coffee with grunt-contrib-watch to automatically compile the .coffee files after every code change.

I'm running CasperJS from a Ruby on Rails app. It turns out to be pretty easy because the coffee-rails gem gives me CoffeeScript.compile (via the ruby-coffee-script gem).
I created a little runtime build system that compiles the script and all possible dependencies just before it's executed. It build only once in production and every time in other environments.
It solves my issue nicely, but it would be even nicer if CasperJS would handle it. The most valuable coding work is done with the delete key!

It is less burden for a project such as CasperJS to reduce the amount of dependencies, here namely CoffeeScript, in similar way as PhantomJS did.
Would @timscott be willing to create a PR for documentation against the phantomjs-2 branch, where he could share how his setup was done?

These have been documented with #1510.  Closing issue.
@timscott We would love it if you could share your findings in the new cookbook, or even just drop us a link to the gist so we can integrate it.

Okay, I extracted my code to something reusable: https://gist.github.com/timscott/636f641fe22046695bf0705bd36b92a7


configure agent images without updating controller with the "image" value

I'd like to add custom hooks to the agent image without updating the controller pod. I notice that when I deploy with --set image=my-custom-image:latest, the controller gets redeployed with my image. This was unexpected based on the help text.
--image string                The image to use for the Buildkite agent (default "ghcr.io/buildkite/agent-k8s:latest")
Is that behavior intentional? And is there a recommended approach for customizing the agent images?

You want to use --set config.image instead of --set image. All the Helm values that start with config will go through to the controller's configuration, but anything that's top-level is values for the controller deployment itself. Sorry if that's a bit confusing.

Works as expected with --set config.image. Thanks for the help.
The help text is a bit misleading. "The image to use for the controller" seems more accurate.

The help text from here? I agree, that's a vestige of a time from before the Helm chart, those are the CLI options for the controller binary.

Ok, agent-stack-k8s --help makes more sense now. I though "Options" was a continuation of the "Deploy with Helm" section.

Hey were you able to get your hooks working? I just figured out a way but not sure if I missed something!
https://github.com/buildkite/agent-stack-k8s/pull/258


JDK问题 | Problem With Jdk

ListBuffer为sun.util包中的class但OpenJdk中没有,现今状况下是否应该避免使用Oracle JDK开发转为OpenJdk?
Q1:ListBuffer is a class in the com.sun.util's package and that defined in OracleJdk,But my env was OpenJdk not OracleJdk,So how to solve the problem?
Q2:Why not use the OpenJdk?

davinci-server_3.01的pom文件加一个依赖

I checked my JDK's tool.jar that has the class named ListBuffer

I checked my JDK's tool.jar that has the class named ListBuffer


Button Toggle Not Responding

Describe the bug
The button with the class tabsbar-toggle is not responding as expected when clicked. The button is intended to trigger the toggleHeight function, toggling the display between a chevron-down and chevron-up icon based on the showMaxHeight condition. However, upon clicking the button, there is no observable change in behavior.
To Reproduce
Navigate to the page containing the button with the class tabsbar-toggle.
Attempt to click the button.
Expected behavior
Upon clicking the button, it should trigger the toggleHeight function and update the icon class accordingly. If showMaxHeight is true, the icon should change to 'fa fa-chevron-down'; otherwise, it should change to 'fa fa-chevron-up'.
Screenshots
CircuitVerse.webm

Device Information [optional]:

OS: Windows 11
Browser chrome
Version 120.0.6099.227

Can I work on this @vinayakjaas ?

Is this issue is still open ?
Can i work on this?


Function names collision - Jiterpretter

Since the bootstrapper were upgraded, Uno Monaco Editor is not abble to set some options.
The reason why is that we have a collision between the function updateOption from Monaco and another with the same name introduced by the recent updates.

Nice find!

Credits on @ebariche !


hugolib: Normalize permalink path segments

When constructing permalinks, ensure that most inputs used as path
segments are normalized with PathSpec.MakeSegment instead of
PathSpec.URLize.
The primary exception to that rule is with taxonomy titles in
pageToPermalinkTitle(). The approach taken here is to use URLize for
taxonomy pages. Everything else will use MakeSegment. The reason for
this exception is that people use taxonomies such as "s1/p1" to generate
URLs precisely they way they wish (see #5223). Tests have been added to
check for this case.
Fixes #4926

Just tested the merged commit and everything looks fine on my end.
Thank you! 👍


Android support

Add Android support to the library with the same API call and functionality.

@Sh3rawi thanks for your contribution!


initial frontend work

Initial frontend work for the app. Created frontend page as per design

@chene40 added tamagui design framework you can use for frontend dev here
also if you're working on the home page I added an OverView component and you can paste your component there after this merges


Update README.md: Fixed link to AUR package

Title
Fixed link to AUR package in README

Description

Fixed link in docs to AUR package. Feel free to close if not needed.
Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate)
Types of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My code follows the code style of this project
[ ] My change requires a change to the documentation
[ ] I have updated the documentation accordingly
[ ] I have read the CONTRIBUTING document
[ ] I have read the  CORE VALUES document
[ ] I have added tests to cover my changes
[ ] All new and existing tests passed

Tests failing are unrelated to this PR. Approved and merged.

@inform880 Do you know what happened with AUR and why the package changed name?
@AZMCode @pekkarr @aminvakil

I can't find the comment regarding details of why
https://aur.archlinux.org/packages/spotdl?O=20#comment-964844
But as spotify-downloader is not a python library package to be used by other packages:
https://wiki.archlinux.org/title/Python_package_guidelines#Package_naming


Image Icons not loading reliably

Description

I was about to make this my main explorer, but then I found the image thumbnail generation in Grid view or other layouts are unreliable.
After some time, the file previews will default to this:

This also seems to be corrupting thumbnails elsewhere in the OS, since other software and the default Explorer have troubles after trying to load thumbnails from those folders.
File Explorer in W11 now freezes:

Start11 also doesn't load icons:

Steps To Reproduce

Have a folder of many various image files and folders with varying dimentions (screen captures folder is a good one to test on)
Go to Grid view Layout (currently easiest with the hotkey CTRL+SHIFT+4)
Notice that some thumbnails don't load when scrolling down

Requirements
Sometimes when scrolling to a small size, the thumbnails will generate, then store and work when scaling up. But not always.
File mixtures can be *.jpg, *.mp4, *.png, *.gif, *.kra (krita) and *.blend (blender).
Other codecs installed:

*.blend by Blender.
FastPictureViewer Codecs (for *.exr and others)
https://www.fastpictureviewer.com/downloads/

Other software that may be conflicting with icons and thumbnails (also bugs the icons after trying to load them in Files)

Start11
https://www.stardock.com/products/start11/

Files Version
<IP_ADDRESS>
Windows Version
10.0.22635.3350
Log File
debug.log

Thanks for the report, Files doesn't change anything with other programs, Can you check your GPU VRAM and Memory usage is not running out in Task manager when this happens?

Computer has not spiked while hitting the error.

I have since uninstalled the FastPictureViewer Codecs , seems more reliable now.

Seems I have to re-register the thumbnail codecs for it to work reliably after installing Files. And also make sure I don't replace any JPG or vector codecs - and all good.

This is good to know, thanks for the detailed report and the steps you took to resolve it.

Did this also resolve the issue with the icons refreshing on zooming in/out?

This did not resolve the zooming blink issue. That would be a separate issue. #15057

Note:

Powertoys from Microsoft with *.svg also is having issues.


runNixOSTest fails on Darwin ( tests.testers.runNixOSTest-example fails to build)

Describe the bug
#282401 should have made NixOS Tests available on Darwin. This should include package tests produced via tester runNixOSTest, but the OfBorg build result shows it isn't. An example is tests.testers.runNixOSTest-example
error: a 'x86_64-linux' with features {} is required to build '/nix/store/rsjmz848mv48kil2s92vnzy4n7q53rcz-builder.pl.drv', but I am a 'aarch64-darwin' with features {benchmark, big-parallel, nixos-test}

See the OfBorg test result in #292759.
Steps To Reproduce
Steps to reproduce the behavior on MacOS:

Set up Nix on MacOS
git clone https://github.com/NixOS/nixpkgs.git
cd nixpkgs
Checkout commit
Build tests.testers.runNixOSTest-example
See build failure

Steps to reproduce the behavior on GitHub:

Go to #292759 or any PR targeting the Nixpkgs master branch.
Comment @ofborg build tests.testers.runNixOSTest-example

Expected behavior
Package tests written by testers.runNixOSTest, including tests.testers.runNixOSTest-example, should run on MacOS. Related OfBorg build should be green.
Screenshots

Additional context
OfBorg build log links:

x86_64-darwin
https://logs.ofborg.org/?key=nixos/nixpkgs.292759&attempt_id=932a1ae7-c6fa-4f38-bd08-6f26b23ed12a

Notify maintainers
@Gabriella439

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
output here

Add a :+1: reaction to issues you find important.

The macOS support for NixOS tests only adds support for running them but not building them.  You still need access to a Linux builder to build those tests.

Thanks for your quick response!

The macOS support for NixOS tests only adds support for running them but not building them. You still need access to a Linux builder to build those tests.

What's the difficulty to build NixOS tests on macOS? Isn't it just a QEMU virtual machine?

It is possible to build Linux products on NixOS using something like darwin.linux-builder but we don't yet have a way to do that in the course of a Nix build.  See:
https://nixos.org/manual/nixpkgs/unstable/#sec-darwin-builder

So I'm thinking about adding a config.disableDarwinNixOSTests to nixpkgs to make https://github.com/NixOS/nixpkgs/blob/795c30faca6c5c7f1296696ad8fce17e2865bafd/nixos/lib/testing/nodes.nix#L19 throw for any attempt to build NixOS tests on Darwin, to give CI an escape hatch to return the old behavior
Does anyone have opinions on this?
I'll probably have a PR prepared today or tomorrow

The macOS support for NixOS tests only adds support for running them but not building them. You still need acces to a Linux builder to build those tests.

Does that mean, those derivations for testing firs got built on the host if not on the cache, and then run in the VM?

So I'm thinking about adding a config.disableDarwinNixOSTests to nixpkgs to make

Any progress on that @lilyinstarlight ?


Search by name not working

We are starting to work on migrating to jenkins 2.0 and thought this plugin looked useful to replace a custom global library/global variables we had been using. With a step like the following
ansibleTower(
                    extraVars: '''
                    host_limit: "foo"
                    message: "hello"
                    ''',
                    importTowerLogs: true,
                    jobTemplate: 'HelloWorld',
                    towerServer: 'DevTower',
                    verbose: false
                )

We always receive:
Beginning Ansible Tower Run on Dev Tower
ERROR: Unable to lookup job template Job template does not exist in tower
If instead we use the job template id number, the pipeline works.
Browsing through the code I see the method convertPotentialStringToID in TowerConnector. If I'm following this correctly jobTemplate = 'HelloWorld' which is then Integer.parseInt(idToCheck). Since this isn't an integer we don't locate a job template.
Does searching by name work for you?

Resolved by PR


{CI} Fix test

Description

CI failure as you can see in https://dev.azure.com/azure-sdk/public/_build/results?buildId=971614&view=logs&j=74095127-2a27-5370-37ed-15a4193f243f&t=5cf5f89a-09fb-583c-72e4-4e4427c89fb1
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

It would be better to paste the error message in the PR description for retrospect. The link will be outdated later.


[Android] fetch, xmlhttprequest do not respect caching headers/params

Description
In our application we need to make a lot of requests to our backend. Requests are different in headers.
On ios everything works fine, all requests are sent to server.
On Android request performs first time fine, but on next try response is with status code 204 without sending request to server. After a few seconds request performs fine again.
Reproduction Steps and Sample Code
const headers = {
project_id: project_id,
language: language,
authorization: [auth.tokenType, auth.accessToken].join(' '),
'cache-control': 'no-cache, no-store, must-revalidate',
// or/and
pragma: 'no-cache'
};
axios.request({
responseType: 'json',
method: method,
url: url,
baseURL: apiConfig.url,
params: params,
data: payload,
headers: headers
});
The same is true for fetch with the same headers and/or 'cache' parameter set to 'no-cache' or 'no-store'.
Additional Information

React Native version: 0.43.3
Platform: Android
Development Operating System: MacOS 10.12.4
Dev tools: Intellij IDEA Ultimate 2017.1, Android SDK - 23.0.1

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with Canny! Check out its new home: https://react-native.canny.io/feature-requests/p/android-fetch-xmlhttprequest-do-not-respect-caching-headersparams


APT whitelist request for sbcl

What needs to be done to complete this work ?

Can we get an update on this?


examples/foc: add phase angle observer support (sensorless mode)

Summary

examples/foc: add phase angle observer support (sensorless mode)
This app can work now as sensorless ESC.
Also introduce a cmd line option that force open-loop control
which is useful when tuning angle/velocity observers

Impact
Testing
ESC based on b-g431b-esc

needs https://github.com/apache/nuttx/pull/10972 to pass sim/foc


Incorrect schema name in the search path is called when using the "--defaultSchemaName" on a non-default schema

Environment
Windows, Linux, PostgreSQL
Liquibase Version:
version 4.0.0-beta1
Liquibase Integration & Version:
CLI
Description
When attempting to create a view to a table in a specific schema, Liquibase attempts to apply the "create view" changeSet to query to the table in the default schema instead.
Steps To Reproduce

Create a new postgreSQL database with schemas: public (default schema), lookup.
Create a changeLog with 2 changeSets, "create table" and "create view".  For example:

<changeSet author="SteveZ" id="1">
        <createTable tableName="execSales">
            <column name="id" type="INTEGER">
                <constraints nullable="false" primaryKey="true" primaryKeyName="id_pkey"/>
            </column>
            <column name="name" type="VARCHAR(255)"/>
        </createTable>
    </changeSet>
    <changeSet author="SteveZ" id="2">
        <createView fullDefinition="false" viewName="newView"> SELECT "execSales".id,
    "execSales".name
   FROM "execSales";
        </createView>
    </changeSet>

Run: liquibase --defaultSchemaName=lookup update
Actual Behavior
The will produce the following error:
liquibase.exception.DatabaseException: ERROR: relation "execSales" does not exist
It appears that the statement from changeSet "create view":
SELECT "execSales".id,"execSales".name FROM "execSales";
is trying to query the table in public.execSales.
Since there is no table “execSales” in the schema "public" the error occur.
Expected/Desired Behavior
The following SQL should be in the "lookup" schema search path.
SELECT "execSales".id, "execSales".name FROM "execSales";
Workaround
This can be worked around with adding ?currentSchema=lookup to the end of the jdbc connection string before running the update command.  For example:
url: jdbc:postgresql://localhost:5432/testDB?currentSchema=lookup
However, this will not work well when using a multi-schema project (ie: using --schemas=public,users,lookup,accounts).
Acceptance Criteria

When specified by a user, --defaultSchemaName=<myschema> should behave as expected, guiding liquibase commands such as update, updateSQL, rollback, and more to work on the specified schema

@szandany Thanks for documenting this bug.  We will take a look and see how to fix it.

What we do with defaultSchemaName and how it is presented to the users needs to be better defined and described.
It currently doesn't do what people often expect it to do, and we should fix that as part of the usability work.

We do actually call SET SEARCH_PATH in CommandLineUtils.initializeDatabase() although when I put a breakpoint in  JdbcExecutor.execute() I see it be set, but then the next time that execute method is called, "show search_path" is back to the default. I'm not sure where it is being reset. It will take more tracing to understand what is going on

Hi @mariochampion Where did this one end up?

Hi,
I updated my version from 3.10.0 to 3.10.3 and this new feature introduced a bug.
We use types declared in the 'public' schema (like the hstore and other types), specific schemas (one by domain and eventually migration) and optionally, the parameter in the 'currentSchema' connection string (if the current user does not fit the schema name). Thus, we can use the types declared in the 'public' schema.
The problem with this new feature is that the hstore type is no longer accessible. Is there a workaround?
In my opinion, adding the 'defaultSchema' in the 'search-path' is a good idea. But rather than replacing the existing 'search_path', it should be added to the existing value.
SELECT set_config(     'search_path',     'my_schema,' || current_setting('search_path'),     true ) WHERE current_setting('search_path') !~ '(^|,)my_schema(,|$)';

Hi @ZigZag59 Thanks for bringing this to our attention.  We did make a change to search path recently and that may be why you are now seeing this issue.  We'll take a look at what was fixed and see if we can accommodate both requests for searchpath use cases. Or if there is a workaround.

I'm also hitting an issue related to this.  The issue is caused by a recent change in version 3.10.2.  Specifically, our migration scripts add geometry types from the PostGIS extension library.  Our migration script (probably incorrectly) do not address the geometry data type correctly.  It should probably be public.geometry
 <changeSet author="roryodonnell" id="20190121">

...
        <createTable schemaName="${schema.name}" tableName="tablename">
            ...
            <column name="DATA_POINT" type="geometry(Point,4326)"/>
            ...
        </createTable>

I'm also hitting an issue related to this.  The issue is caused by a recent change in version 3.10.2.  Specifically, our migration scripts add geometry types from the PostGIS extension library.  Our migration script (probably incorrectly) do not address the geometry data type correctly.  It should probably be public.geometry
 <changeSet author="roryodonnell" id="20190121">

...
        <createTable schemaName="${schema.name}" tableName="tablename">
            ...
            <column name="DATA_POINT" type="geometry(Point,4326)"/>
            ...
        </createTable>

...I would think adding the public schema to the SEARCH_PATH is a suggested workaround
INFO  [2021-01-15 12:07:54,433] liquibase.executor.jvm.JdbcExecutor: SET SEARCH_PATH TO <specified schema>, public

FYI: This SEARCH_PATH overriding is fixed in version 4.5.0 with #1864

@PavloTytarchuk retested on this end and found it was still present. I am working with him to clarify. Thanks @piomar123 !

This was reported again in the forum:
https://forum.liquibase.org/t/search-path-is-not-being-set-correctly/6555

I believe this was fixed in #2812 for 4.11.0. I'll see if I can still reproduce the issue or not

Yes, this seems to be working with me in the code going out in 4.11.0


Floodgate's send-floodgate-data option is incompatible with Fabric Proxy Lite's hackEarlySend option

I am running a Velocity proxy server that is connected to two, 1.21.1 Fabric servers. Each of which has Floodgate and Fabric Proxy Lite installed on them. I need to enable the option hackEarlySend to allow for LuckPerms support. However, when this is enabled alongside Floodgate's send-floodgate-data option in the proxy server, it causes Bedrock clients to not be able to connect to the server, with the error message "Unable to connect to server: Your server did not send a forwarding request to the proxy. Make sure the server is configured for Velocity forwarding.".
I found an issue on the Geyser github repository page talking about problem as well. I also found another thread which suggests a hacky patch to the code of Fabric Proxy Lite to always register the QUERY_START event regardless of whether the sendEarlyHack option is enabled. I suppose this isn't really recommended however.
I would like to see some sort of fix as I would love to be able to use LuckPerms and Geyser/Floodgate in my proxy setup. Thank you.

I just saw that this issue is on Floodgate's end so I will close this issue now.


Add search and filter via ssr

Adds search, category, and filter requests to the ssr page
Closes https://github.com/newrelic/developer-website/issues/1855

if anyone wants to continue between now and 1/3/22-- current status: last AC is adding tests


enable support for enum arrays

 Issue by kns98
Tuesday Feb 24, 2015 at 20:54 GMT
Originally opened as https://github.com/gsscoder/commandline/pull/148

kns98 included the following code: https://github.com/gsscoder/commandline/pull/148/commits

 Comment by ChangePlaces
Wednesday Nov 02, 2016 at 19:11 GMT

What's the status on this please?

 Comment by gsscoder
Wednesday Jun 10, 2015 at 04:14 GMT

@kns98 thank you so much for PR. This is for stable branch and will be handled in a further moment.
I'm try to catch up to 2.0 project status as now.
Stay tuned and thanks again.
Regards,
Giacomo

 Comment by rvazarkar
Friday Oct 13, 2017 at 05:52 GMT

Just another request for an update on this