Import vuejs component

Sails version:1.0.2
Node version:8.11.2
NPM version: 5.6.0
DB adapter name:sails-postgres
DB adapter version:1.0.1
Operating system:Linux

Expected behaviour
Hi
Is it possible in parasails.registerComponent file to import class ES6 from other existing component ?
I create a component and it depend on another component VueJS so I need to import it and add as a component on my Sails app.
If you look at this gist https://gist.github.com/craigh411/c274a29ca331041fa6f3666756339634, I want to import vue component to enrich my form on page by adding select2 with a new existing component.
Ore like this another component https://vue-treeselect.js.org/ which requiert import ES6. It could be good if I can use it with import.
Thanks.

ok, fixed!

ok, fixed!

ok, fixed!

ok, fixed!

ok, fixed!

ok, fixed!

New example for my issue.
I available polyfill and babel. Not working as "import should be at top of module" error is prompt.
Then I only try require. Is it possible to transpile parasails front component ?
/**
 * <tree-select>
 * -----------------------------------------------------------------------------
 * A treeSelect dialog pop-up.
 *
 * @type {Component}
 * -----------------------------------------------------------------------------
 */
var treeSelectCss =<EMAIL_ADDRESS>var TreeSelect = require('@riophae/vue-treeselect/Treeselect')

parasails.registerComponent('treeSelect', {

  // register the component
  components: { 'Treeselect' : TreeSelect },

  //  ╔═╗╦═╗╔═╗╔═╗╔═╗
  //  ╠═╝╠╦╝║ ║╠═╝╚═╗
  //  ╩  ╩╚═╚═╝╩  ╚═╝
  props: [ ],

  //  ╦╔╗╔╦╔╦╗╦╔═╗╦    ╔═╗╔╦╗╔═╗╔╦╗╔═╗
  //  ║║║║║ ║ ║╠═╣║    ╚═╗ ║ ╠═╣ ║ ║╣
  //  ╩╝╚╝╩ ╩ ╩╩ ╩╩═╝  ╚═╝ ╩ ╩ ╩ ╩ ╚═╝

  data: function () {
    return {
      // define default value
      value: null,
      // define options
      options: [ {
        id: 'a',
        label: 'a',
        children: [ {
          id: 'aa',
          label: 'aa',
        }, {
          id: 'ab',
          label: 'ab',
        } ],
      }, {
        id: 'b',
        label: 'b',
      }, {
        id: 'c',
        label: 'c',
      } ],
    }
  },

  //  ╦  ╦╔═╗╔═╗╔═╗╦ ╦╔═╗╦  ╔═╗
  //  ║  ║╠╣ ║╣ ║  ╚╦╝║  ║  ║╣
  //  ╩═╝╩╚  ╚═╝╚═╝ ╩ ╚═╝╩═╝╚═╝
  mounted: function () {

  },

  //  ╦╔╗╔╔╦╗╔═╗╦═╗╔═╗╔═╗╔╦╗╦╔═╗╔╗╔╔═╗
  //  ║║║║ ║ ║╣ ╠╦╝╠═╣║   ║ ║║ ║║║║╚═╗
  //  ╩╝╚╝ ╩ ╚═╝╩╚═╩ ╩╚═╝ ╩ ╩╚═╝╝╚╝╚═╝
  methods: {

  }

});

ReferenceError: require is not defined[En savoir plus]

@chadyred Are you intending to extend a component in a bunch of places? If so, it seems you could register the code as a service then call it within the components that use it. If not, it seems best to just copy the code directly into the component

Hi,
Most of the devs must have resolved this issue so far. But posting an answer as I was not able to find any answer anywhere on this issue.
I was able to import the VueSelect component (with 3 more dependent components) in Parasails.
Here are the steps:
Step 1:
Copy the content of JS file from Component's CDN path.
Step 2:
Create a folder in your assets/js/components folder by the name of the component (try to create name in kebab case format). And copy the content of JS file (from Step 1) in this component.
Step 3:
Register the component locally in the required file by defining it in the components property, like so-
parasails.registerComponent('nameOfLocalComponent' , {
components: {
'v-select': VueSelect.VueSelect, (this is the name provided in exports declaration or how to import via cdn as explained in the npm page of the component)
},
Yes, I was able to do it only for a given npm package component but guess it will work for other components also. Hope it helps.
Thanks

Thanks a lot...but you said
Copy the content of JS file from Component's CDN path.
Is it possible do import it with require ? Look at my previous post with TreeSelect and require.

No, somehow parasails is not currently supporting require and import.

To import 3rd party components using the default asset pipeline as it comes, the simplest approach is to include the 3rd party component as an already-compiled bundle of js+css.  (That’s what my team does on our projects and it works well for us.  That said, it’s definitely not as sophisticated as it could be... but I’d still recommend at least giving it a go once and seeing what you think, if you haven’t already tried that)
If that doesn’t suit, here are two other solutions I’ve heard other teams say have been successful for them:

Use webpack instead and uninstall sails-hook-grunt
Install one of the grunt plugins that compile .vue files and hook that up in your tasks/ folder

That’s a big bummer!
However… For those who want the sophisticated solution, check out this article:
https://medium.com/@Noitidart/adding-code-split-react-frontend-without-losing-vue-to-sails-1-x-app-5642131ed8a0

I was able to import a component called Vue Color into my project the following way.

Add the dist js file to /assets/dependencies (in my case vue-color.js)

Add a new file to assets/js/components (in my case I called it color-picker.component.js) and register the component using Parasails:
parasails.registerComponent('colorPicker', VueColor.Chrome);

Vue Color exports its components into the window.VueColor object hence the VueColor.Chrome

Then when I tried to use  I got the following error:

If you're unsure, or you're deliberately attempting to use a Vue.js feature that relies on having a top-level property named %propertyName% then please remove this check from the parasails.js library in your project

I searched for this error message in parasails.js (line 250 for me), commented out the line responsible and the component now loads.

Not bad so I give it a try soon and I come back later !

@manas009in ,
Following steps described by you, i tried it for component, Vue-Multiselect https://vue-multiselect.js.org
But not sure what the name should be in this case .
Vue.component('vue-multiselect', window.VueMultiselect.default)
can u please correct me for this case?
PFA

Thanks

@hussainkhan try:
parasails.registerComponent('vueMultiselect', VueMultiselect.Multiselect);

@hussainkhan try:
parasails.registerComponent('vueMultiselect', VueMultiselect.Multiselect);

Thanks @madisongiles , it works fine.

It's seems to be challenging to figure out component names,
in order to register component vue-jquery-mask as mentioned by @madisongiles
i have tried below name combination  but still getting error  - did you register the component correctly?
VuejQueryMask.vueMask, VuejQueryMask.VueMask ,VuejQueryMask.Vuemask
can someone please correct me for this library name, any hint to get this name

For that plugin you'd use VuejQueryMask
I've found a lot of component names by typing "window.vue" into the Chrome Developer Tools Console and looking at the object it returns. Sometimes the component object will be down a level as is the case with VueMultiselect.Multiselect and VueColor.Chrome

For that plugin you'd use VuejQueryMask
I've found a lot of component names by typing "window.vue" into the Chrome Developer Tools Console and looking at the object it returns. Sometimes the component object will be down a level as is the case with VueMultiselect.Multiselect and VueColor.Chrome

Thanks @madisongiles


Documentation for angularjs use

Hi, where is the docs for angular js using configuration? Thanks

How to get the swiper instance?


Page disappears from history.

Steps to reproduce:

Open SAFE browser and navigate through a few pages. Eg 1 > 2 > 3. History reads 1,2,3
Click back on browser. 3 > 2. History still reads 1,2,3
Click on a different page 2 > 4

Expected result:
History contains all pages visited in browsing session (1,2,3,4)
Actual result:
History does not list page user clicked back from (1,2,4)
Browser history issue.pdf

closing as duplicate of #461
Related to #557

Moving to done as story already exists for this issue (duplicate report here).


[Core JS][Date Object] Obscure bug when instantiating Date with 'm-d-yyyy' across build configs

// Build config = Debug
console.log( new Date('4-4-2016') )
"Mon Apr 04 2016 00:00:00 GMT-0700 (PDT)"
// Build config = Release
console.log( new Date('4-4-2016') )
"Invalid Date"

I suspect this is because you're using the Chrome debugger while using the Debug build config. Can you make sure you're using the on-device JS VM and see if this issue repros?

@ide, whoops, correcting the description.

It's been a few months so my hypothesis here is that this was indeed an issue of using the on-device JS VM. I'm closing this issue but if this does repro with the right VM then feel free to reopen it with the new info.


Save minecraft-server configuration in etc.

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if adding a new cask:

[ ] Named the cask according to the token reference.
[ ] brew cask install {{cask_file}} worked successfully.
[ ] brew cask uninstall {{cask_file}} worked successfully.
[ ] Checked there are no open pull requests for the same cask.
[ ] Checked the cask was not already refused in closed issues.
[ ] Checked the cask is submitted to the correct repo.

@reitermarkus I've added a java caveat and rebased to fix the travis error.
fatal: Invalid symmetric difference expression 4149d9716dba5807ce814c23877eef3ee5115d45...2065585d7b3ea6c1e2b21f9d82490dcf483b7ee6
``

Thanks, @commitay!


Text with English and Japanese characters is identified as Galician or Basque

The following test: BizteX cobitの使い方 is identified as Galician and Basque:
Language [eu]
probability: [0.8410933444100416]

Language [gl]
Probability: [0.13353105016279743]

@adblancod can you share a snippet of code you used to get these? I tried with https://github.com/optimaize/language-detector/issues/86#issuecomment-638818158 and got:
detectedLanguages = {ArrayList@1320}  size = 1
 0 = {DetectedLanguage@1325} "DetectedLanguage[eu:0.7259662120805258]"
detectedLanguagesNormalised = {ArrayList@1321}  size = 2
 0 = {DetectedLanguage@1328} "DetectedLanguage[eu:0.8410933444100412]"
 1 = {DetectedLanguage@1329} "DetectedLanguage[gl:0.13353105016279732]"

It looks like you're either manually normalising input, or using some Optimaize method which does normalization for you (which seems very important for CJK, but wasn't happening for me and another user in #86 ).
I don't think I can help with your accuracy though - perhaps the string is just too short for Optimaize. Google Translated detects Slovenian.


(POST-BETA) What minimal PHP version to choose - PHP7 vs PHP5.6)?

My development and all projects done using PHP 7, but there is no strict typing in framework and components so spiral can work starting with 5.5+. Should i keep minimal version as 5.6 or up it to 7?

Keep 5.6


Add builtin playground plugin

Summary
Add builtin playground plugin.
Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

Surpassed by #75


Extension Host is now responsive.

Extension Name: debug-auto-launch
Extension Version: 1.0.0
OS Version: Windows_NT x64 10.0.17134
VSCode version: 1.28.0

We have written the needed data into your clipboard. Please paste:

Duplicate of #60379


login: Allow user to specify a browser path

Sometimes I like to use multiple browsers for logging into AWS (for example, when I want to compare two different environments, like prod and stage).
It would be useful to be able to specify that I'd like to log in via Safari (which is not my default browser) instead of Chrome for a particular session.  I'm an OS X user.

@otterley aws-okta is just using https://github.com/skratchdot/open-golang/, which is wrapper around open.
This would be pretty major surgery on the login subcommand.

The workaround here has been merged, that may work for your use case

Good enough, thanks!


Consider removing dbt_user from dbt's Docker image

Describe the feature
The Dockerfile definition for dbt creates dbt_user as the user to run subsequent commands under. This can cause issues when mounting volumes where the host filesystem is owned by root. The specific situation I'm thinking of is using the dbt Docker image as a build step in a CI environment, it's common for the files/directories in these environments to be owned by root, but the expectation is that build steps are still able to create directories and write files. Using the dbt Docker image in this context results in permission issues.
I'd like to suggest removing dbt_user from the image definition and using the default root user instead. Most Dockerized CLI tools I've come across do not specify an explicit user and use root as the default (refer: gcloud, doctl, busybox).  Users using the image locally would be able to override the user at runtime using the --user flag if there was a desire or need to do so.
Describe alternatives you've considered
Our particular issue was using the dbt Docker image in Cloud Build. We were getting errors that dbt was unable to create the target/ and logs/ directories due to permission issues. We ended up having to run chmod -R 777 ./ on the working directory prior to running dbt to make things work. This was fine for us as the filesystem is ephemeral, but in other situations you obviously wouldn't want to do this (you'd also have access to the --user flag, which isn't an available option in Cloud Build).
Additional context
N/A
Who will this benefit?
This change would benefit users who use the dbt Docker image as part of their CI/CD pipeline. Having the image run under dbt_user imposes constraints that require workarounds that could ideally be avoided.

This sounds really reasonable to me. I think the permissions change is just a regression from switching build systems, but I don't think there's any good reason to use dbt_user. It seems to cause a lot of trouble for end users and the upside is limited. I think the main advantage is it doesn't change all my local files to be owned by root when I run it in a mount, but I don't really use it enough to justify that.
@drewbanin how do you feel about me making this change for 0.17.1? It should be a pretty straightforward change to the dbt-release project's dockerfile. I could also cut an rc3 with just this change if we wanted to test it out.

@beckjake sure thing, I'm supportive of making a change like this in 0.17.1

@christippett 0.17.1rc3 (just released) should run as root now. I would love it if you could give it a try sometime and let me know if it works as you expect.

Will do. Thanks for being so receptive to my suggestion and turning this around so quick 😊

Hi there @beckjake,
Chris and I worked together so I thought I'd give you an update.
I've upgraded to 0.17.1rc3 and my testing would indicate that the issue has been resolved. @christippett might wanted to do testing on his end too to confirm my findings.
Cheers, Meurant

I just tested 0.17.1 and I was able to remove some work-arounds I had for permissions thank to the usage of root.
However, I'm seeing that fishtownanalytics/dbt:0.18.0b1 is using dbt_user again, any reason for it?

0.18.0b1 was released on June 8th, 0.17.1 was released on July 20. The next 0.18.0 prerelease will use root.

@beckjake apologies for taking so long to get back to you. I can confirm I'm currently using the the dbt Docker image (tag 0.17.1rc4) in Cloud Build with no issues.

Closing this since the Docker image has been updated and there’s been no issues (as reported here) for several releases.


Is this module still working?

Hi, I wonder if the module is still working?
I am trying to install the package using pip3 install afterhours, and the package is installed and I can tell that by the message in the console "Requirement already satisfied: afterhours in /Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/site-packages (0.2.1)"
However, when I tried to import the module by from afterhours.afterhours import AfterHours, python complains "ModuleNotFoundError: No module named 'afterhours'". BTW, I am usnig python 3.8
Thanks.

Exact same issue here


[Linter panic] IPykernel Debugger Module

A minimal code snippet that reproduces the bug.
https://github.com/ipython/ipykernel/blob/1270d5a6e17845e6b22361e43325dd2de59d6392/ipykernel/debugger.py

The command you invoked (e.g., ruff /path/to/file.py --fix), ideally including the --isolated flag.
ruff --fix

The current Ruff settings (any relevant sections from your pyproject.toml).

[tool.ruff]
target-version = "py37"
line-length = 100
select = [
  "A",
  "B",
  "C",
  "DTZ",
  "E",
  "EM",
  "F",
  "FBT",
  "I",
  "ICN",
  "ISC",
  "N",
  "PLC",
  "PLE",
  "PLW",
  "Q",
  "RUF",
  "S",
  "SIM",
  "T",
  "TID",
  "UP",
  "W",
  "YTT",
]
ignore = [
  # Allow non-abstract empty methods in abstract base classes
  "B027",
  # Ignore McCabe complexity
  "C901",
  # Allow boolean positional values in function calls, like `dict.get(... True)`
  "FBT003",
  # Use of `assert` detected
  "S101",
  # Line too long
  "E501",
  # Relative imports are banned
  "TID252",
  # Boolean ... in function definition
  "FBT001", "FBT002",
  # Module level import not at top of file
  "E402",
  # A001/A002/A003 .. is shadowing a python builtin
  "A001", "A002", "A003",
  # Possible hardcoded password
  "S105", "S106",
  # Q000 Single quotes found but double quotes preferred
  "Q000",
  # N806 Variable `B` in function should be lowercase
  "N806",
  # T201 `print` found
  "T201",
  # N802 Function name `CreateWellKnownSid` should be lowercase
  "N802", "N803",
  # C408 Unnecessary `dict` call (rewrite as a literal)
  "C408",
  # N801 Class name `directional_link` should use CapWords convention
  "N801",
  # SIM105 Use `contextlib.suppress(ValueError)` instead of try-except-pass
  "SIM105",
  # S110 `try`-`except`-`pass` detected
  "S110",
]
unfixable = [
  # Don't touch print statements
  "T201",
  # Don't touch noqa lines
  "RUF100",
]

[tool.ruff.per-file-ignores]
# B011 Do not call assert False since python -O removes these calls
# F841 local variable 'foo' is assigned to but never used
# C408 Unnecessary `dict` call
# E402 Module level import not at top of file
# T201 `print` found
# B007 Loop control variable `i` not used within the loop body.
# N802 Function name `assertIn` should be lowercase
# F841 Local variable `t` is assigned to but never used
# EM101 Exception must not use a string literal, assign to variable first
# PLR2004 Magic value used in comparison
# PLW0603 Using the global statement to update ...
# PLW2901 `for` loop variable ...
# PLC1901 `stderr == ""` can be simplified to `not stderr` as an empty string is falsey
# B018 Found useless expression. Either assign it to a variable or remove it.
# S603 `subprocess` call: check for execution of untrusted input
"ipykernel/tests/*" = ["B011", "F841", "C408", "E402", "T201", "B007", "N802", "F841", "EM101",
    "EM102", "EM103", "PLR2004", "PLW0603", "PLW2901", "PLC1901", "B018", "S603"]

The current Ruff version (ruff --version).  276

panicked at 'range start index 2 out of range for slice of length 1', crates/ruff/src/rules/isort/block.rs:159:36
Backtrace:    0: <unknown>
   1: <unknown>
   2: <unknown>
   3: <unknown>
   4: <unknown>
   5: <unknown>
   6: <unknown>
   7: <unknown>
   8: <unknown>
   9: <unknown>
  10: <unknown>
  11: <unknown>
  12: <unknown>
  13: <unknown>
  14: <unknown>
  15: <unknown>
  16: <unknown>
  17: <unknown>
  18: <unknown>
  19: <unknown>
  20: <unknown>
  21: <unknown>
  22: <unknown>
  23: <unknown>
  24: <unknown>
  25: <unknown>
  26: <unknown>

Thanks, will take a look!

If you need an immediate workaround, you can remove the # isort: skip comments. But I'll make sure this is fixed in the next release.

We're fine to ignore the warning for now, thanks!


Make remote status bar item flush with activity bar

This PR fixes #96952 and improves visual consistency when the remote item is visible in the status bar.
Before:

After:

@misolori To make the final call, I'm fine with it.

Hey, thanks for taking the time to make this PR. I already commented on the original issue (https://github.com/microsoft/vscode/issues/96952#issuecomment-624871143) but unfortunately this is not something we are interested in adopting. Additionally, this PR hardcodes a width for the remote indicator which breaks when used in context:

In the future, it's best to wait for the team to label issues that will be fixed. If you'd like to help with other issues, please see the issues marked as help-wanted. Thanks again.

Thanks for looking into this. I responded to #96952 and fixed the overflow truncation:

The width is still hard-coded because it looks that's the norm within the activity bar styles, but the overflowing has been fixed.


Minor fix to downscaling metadata from feedback

PR fixing bugs a team review caught in downscaled output metadata attrs.

Got the "this looks good" elsewhere. Merging.


feature: subscription override field

Overview/Summary
Adds an azure_subscription_id variable that defaults to the connected subscription if left blank. This can be used to target a subscription other than the one connected to for the bootstrap resources.
A future iteration could update this field to use the subscription_id_management field in the starter module, but would require updates to the ALZ PowerShell module to achieve that in a robust way.
This PR fixes/adds/changes/removes

#15

Breaking Changes
None. Defaults to the connected subscription if none is supplied, which is the existing behaviour.
Testing Evidence
Will require a draft release to test. TODO
As part of this Pull Request I have

[ ] Checked for duplicate Pull Requests
[ ] Associated it with relevant issues, for tracking and closure.
[ ] Ensured my code/branch is up-to-date with the latest changes in the main branch
[ ] Performed testing and provided evidence.
[ ] Updated relevant and associated documentation.

Successfully tested with development version of ALZ PowerShell module, but waiting on this PR to be able to use this. It is fine to merge this PR though. Will tag and create a release once the ALZ PowerShell module is released.


travis build fails with sharp as native dependency

I'm trying to build an Electron application with electron-builder via Travis-CI for Windows, Mac, and Linux and have sharp as a dependency. The electron-builder documentation just points to a sample Travis build for reference. My app uses a modified version based on the sample. While building the executable for the app on my own Windows machine is successful, I'm greeted with the following error when running the Travis-generated executable:

Likewise, I get the following error when installing the Travis-generated Debian package:
Error: libvips-cpp.so.42: cannot open shared object file: No such file or directory
    at process.module.(anonymous function) [as dlopen] (ELECTRON_ASAR.js:172:20)
    at Object.Module._extensions..node (module.js:598:18)
    at Object.module.(anonymous function) [as .node] (ELECTRON_ASAR.js:186:18)
    at Module.load (module.js:503:32)
    at tryModuleLoad (module.js:466:12)
    at Function.Module._load (module.js:458:3)
    at Module.require (module.js:513:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (/opt/zoom/resources/app.asar/node_modules/sharp/lib/constructor.js:10:15)
    at Object.<anonymous> (/opt/zoom/resources/app.asar/node_modules/sharp/lib/constructor.js:259:3)

Based upon the research I've done thus far, it seems that the error is related to having native dependencies (sharp) which aren't "linked" properly in the Travis build process.
Thanks for your help!

"Perhaps it's because I'm trying to build using Wine inside of a Docker container, and I need to build my application on a native machine?"

Yes, perhaps try https://www.appveyor.com/

According to the electron-builder documentation:
From the electron-builder documentation:

"You cannot build for Windows using Docker if you have native dependencies and native dependency doesn't use prebuild."

Does sharp meet that criteria? Otherwise this issue can be closed. I'm trying out AppVeyor right now.
Thanks!

After shifting the Windows build process over to AppVeyor, my app now runs without error on Windows. However, I have yet to test the binary for Linux, which also threw an error related to sharp's native dependencies. I will close this issue once I test the Linux binary soon.

Linux binary still has an error with libvips-cpp.so, but it appears that it is an electron-builder issue. Closing this issue.


Failing to connect to OVMS after running ~5 pipelines

Version
main
Hardware configuration
List Hardware components used.
Ex.
Issue found with the following:

Intel® xeon Platinum 8351N with Intel® Data Center GPU Flex 140
12th Gen Intel® Core™ i7 with Intel® Arc™ A770M Graphics

Describe the bug
After running 5 pipelines (object_detection or any other), the consequent pipelines cannot connect to OVMS.
The following error appear on the results log:

RuntimeError: Requested model: ssd_mobilenet_v1_coco, version: latest, has not been found or is not in available state

It appears to be a race condiction. There is a fix when increasing the InitWaitTime to 20s.
To Reproduce
Steps to reproduce the behavior:

run camera simulator
run 5 or more pipelines:
DEVICE=CPU PIPELINE_PROFILE="object_detection" RENDER_MODE=1 sudo -E ./run.sh --workload ovms --platform dgpu.0 --inputsrc rtsp://<IP_ADDRESS>:8554/camera_0
observe results logs and after 5 pipelines, the rest of them will fail.

notes:

based on the error log, this issue seems to relate to the readiness of models from OpenVINO model server instance that is used for the pipeline application
more pipelines launched, the system resource is more consumed and thus the launching time for OVMS server takes longer and hence the readiness of the model takes also longer

number of pipelines to run successfully can depend on user's system capability.


Migrating WebviewViewProvider to the web

[wip]

Codecov Report

Merging #10013 (04eef9c) into main (580dd68) will increase coverage by 0%.
The diff coverage is 100%.

:exclamation: Current head 04eef9c differs from pull request most recent head 03bde7f. Consider uploading reports for the commit 03bde7f to get more accurate results

@@          Coverage Diff           @@
##            main   #10013   +/-   ##
======================================
  Coverage     63%      63%
======================================
  Files        211      211
  Lines      10090    10097    +7
  Branches    1610     1611    +1
======================================
+ Hits        6442     6448    +6
  Misses      3131     3131
- Partials     517      518    +1

Impacted Files
Coverage Δ

src/platform/common/platform/errors.ts
98% <ø> (ø)

src/platform/common/platform/fileSystem.node.ts
75% <100%> (ø)

...c/platform/common/platform/fileSystemUtils.node.ts
15% <100%> (ø)

src/platform/common/variables/environment.node.ts
95% <100%> (ø)

src/platform/common/utils/misc.ts
69% <0%> (-3%)
:arrow_down:

...ommon/process/environmentActivationService.node.ts
69% <0%> (-1%)
:arrow_down:

src/platform/common/platform/fs-paths.ts
84% <0%> (ø)

src/platform/common/utils/localize.ts
98% <0%> (+<1%)
:arrow_up:

src/platform/api/pythonApi.ts
69% <0%> (+<1%)
:arrow_up:

src/platform/errors/errorHandler.ts
63% <0%> (+<1%)
:arrow_up:


Failed to bind to the service on Android

Hi, everyone.
I installed this package yesterday. But there is an error I couldn't fix. It works on iOS properly. But on Android throw below error while using checkNeedsUpdate method:

Error: sp-react-native-in-app-updates checkNeedsUpdate error: Error: com.google.android.play.core.appupdate.internal.zzy: Failed to bind to the service.

"react-native": "0.67.3",
"sp-react-native-in-app-updates": "^1.4.0",
Device: Redmi Note 12
I found thrown place of error in package's native code:
appUpdateInfoTask.addOnFailureListener(err -> {
    resolutionPromise.reject("Exception", err.toString());
});

Here is code when I used package:
import {Platform} from 'react-native';
import SpInAppUpdates, {
  IAUUpdateKind,
  StartUpdateOptions,
  IAUInstallStatus,
} from 'sp-react-native-in-app-updates';

export const checkForUpdate = async () => {
  const inAppUpdates = new SpInAppUpdates(
    false, // isDebug
  );
  try {
    await inAppUpdates.checkNeedsUpdate().then(result => {
      try {
        if (result.shouldUpdate) {
          let updateOptions = {};
          if (Platform.OS === 'android') {
            updateOptions = {
              updateType: IAUUpdateKind.FLEXIBLE,
            };
          }
          if (Platform.OS === 'ios') {
            updateOptions = {
              title: 'Update available',
              message:
                'There is a new version of the app available on the App Store, do you want to update it?',
              buttonUpgradeText: 'Update',
              buttonCancelText: 'Cancel',
            };
          }
          inAppUpdates.addStatusUpdateListener(downloadStatus => {
            console.log('download status', downloadStatus);
            if (downloadStatus.status === IAUInstallStatus.DOWNLOADED) {
              console.log('downloaded');
              inAppUpdates.installUpdate();
              inAppUpdates.removeStatusUpdateListener(finalStatus => {
                console.log('final status', finalStatus);
              });
            }
          });
          inAppUpdates.startUpdate(updateOptions);
        }
      } catch (error) {
        console.error(error);
      }
    });
  } catch (error) {
    console.error(error);
  }
};

Please help me to fix it and continue work. Thanks for in advance!

did you fixed it ?, i got same error

did you fixed it ?, i got same error

no, can't fix, owners haven't responded yet

Are you using real device? or simulator?

Are you using real device? or simulator?

both

well, you should test on real device not on simulator.

well, you should test on real device not on simulator.

Real device returns same error, I toasted error.

well, you should test on real device not on simulator.

Real device returns same error, I toasted error. Device has Google Play service

also just to confirm you building  release apk not dwbug right?

I came across this today, here is what i have done to make it working.

install package and react-native-device-info
opened in simulator and i got the same error you have got
Seen this post,then went back to package document where found some additional info.
Then build the app for relaease
Uploaded aab for internal sharing, install or update
Then made some changes, uploaded new aab
Checked on play store that update is available but did not hit update
opened the app and  updated from app.

I came across this today, here is what i have done to make it working.

Installed the package and react-native-device-info.
Opened the app in the simulator and encountered the same error.
Referred to the package documentation and found additional information.
Built the app for release.
Uploaded the AAB for internal sharing, installed, or updated the app.
Made some changes and uploaded a new AAB.
Checked on the Play Store that an update was available but did not hit update.
Opened the app and updated it from within the app.

Well done, it worked after doing that guidline


Fix class not found error for SupportedHdfsAclProvider and SupportedH…

What changes are proposed in this pull request?
Please outline the changes and how this PR fixes the issue.
Why are the changes needed?
Please clarify why the changes are needed. For instance,

If you propose a new API, clarify the use case for a new API.
If you fix a bug, describe the bug.

Does this PR introduce any user facing changes?
Please list the user-facing changes introduced by your change, including

change in user-facing APIs
addition or removal of property keys
webui

@shangxinli I have reviewed this, I think it's good. Do you need @beinan to double check it.

Sounds good! I will have a try. Do you know the official release tarball was generated with that flag on?

Sounds good! I will have a try. Do you know the official release tarball was generated with that flag on?

I'm afraid not, It's turned off by default.


NIFI-5839: Support case insensitive membership decisions in LdapUserGroupProvider

NIFI-5839:

Introducing case insensitive group membership.
Ensuring user identity and group name mapping is applied when the user identity or group name is inferred through group membership decisions.

+1 Code looks good and tested the various scenarios using Apache Directory Studio's LDAP, going to merge, thanks!


How to use custom class with amf_socket?

Hello,
Thanks so much for providing amf_socket. I see the way how to send and receive Objects between client an server. But how do I send back and force custom class objects? sendRequest() and sendMessage() functions can't be used to send custom class objects?

@jimmymjing it sounds like you are asking how to use the class mapping feature of Flash. If so, you can find a very basic example on the amf_socket homepage:
https://github.com/chadrem/amf_socket#class-mapper
I haven't worked with custom classes in a long time, but last I remember you can find more information in Adobe's docs by searching for the registerClassAlias function.


Enable authoring and use of external  tools for models that support it.

This is more or less ready to go from our side.
Due to a bug in Ollama tools do not work properly when the response is streamed as we do in oterm.  Once this is fixed we can proceed to release tool support.
https://github.com/ollama/ollama-python/issues/279


Maintain releases in GitHub

It would be nice to maintain all releases also in GitHub: https://docs.github.com/en/free-pro-team@latest/github/administering-a-repository/managing-releases-in-a-repository so that one gets a quick overview of the history of releases and the last release version

Releases are currently all in Maven Central here: https://repo1.maven.org/maven2/com/adobe/aem/aemanalyser-maven-plugin/

releases are visible at https://github.com/adobe/aemanalyser-maven-plugin/releases

Added the missing releases

And again: New releases don't show up

@kwin are you aware of a way to automate this?

https://github.com/marketplace?type=actions&query=github+releases


Add "species" to pet model

DB migration to add "species" field to the pet model. Species can be things like "cat", "dog"

I can do this!

@nsiwnf are we including exotic pets like: "rabbit" "rodent" "lizard" "snake"?

@nsiwnf are we including exotic pets like: "rabbit" "rodent" "lizard" "snake"?

Hmm, let's just do "cat" and "dog" for now


[SPARK-40314][SQL][PYTHON] Add scala and python bindings for inline and inline_outer

What changes were proposed in this pull request?

Adds Scala and Python bindings for SQL functions inline and inline_outer
Why are the changes needed?

Currently these functions can only be used via SQL or through expr functions. This makes it a little easier to use them with the DataFrame APIs.
Does this PR introduce any user-facing change?

Exposes new functions directly instead of only through SQL.
How was this patch tested?

Updated existing inline tests to use the new Scala binding instead of being called through SQL expressions

Can one of the admins verify this patch?

also, what about adding some tests in python/pyspark/sql/tests/test_functions.py?

also, what about adding some tests in python/pyspark/sql/tests/test_functions.py?

Thought I found all the places there were explode tests to add inline as well but missed these somehow, I'll add one

Merged into master, thanks all!


Disable lui-mode line wrapping regardless of :feature spellcheck

I don't see how those are related. Or am I missing something?

I don't recall why they were grouped together, but you're right, it seems odd that they are.
Would you kindly rebase 5703435 onto develop? A few orphaned commits were caught in your PR during a push --force (sorry!).

Re-submitted as #651.


Cassandra_scaler improvement

Signed-off-by: ithesadson<EMAIL_ADDRESS>Current Code: Port information can be assigned in 2 different ways.
a- in clusterIPAddress
b- in port
Change: Removed assignment of port information in clusterIPAddress. Port information can only be entered in the port.
Why is this change necessary?
In case clusterIPAddress contains port information, in previous version of the code there was an unnecessary and incorrect check in clusterIPAddress, it is better to simplify it and only ask to be entered in port variable.
If the clusterIPAddress contains port information, checking if the port information is entered in the clusterIPAddress will cause some errors.
Error example in current code: User enters the following values.
clusterIPAddress: "https://cassandra.test"
port: "9042"
->Expected clusterIPAddress : https://cassandra.test:9042
The actual clusterIPAddress: https://cassandra.test
CheckList

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)

I think we should avoid breaking changes

https://github.com/kedacore/keda/issues/4110

@ithesadson please don't break the behaviour and keep the original one. Just fix the problem please.

closing in favor of https://github.com/kedacore/keda/pull/4162


st-util problem

I've set up everything and here what shows up:
setting new configuration (0 -> 1)
2015-05-17T21:26:23 WARN src/stlink-usb.c: libusb_set_configuration() failed
I have fedora 21 machine and nucleo stm32f411re. what's wrong

I have the same problem with Ubuntu 15.04 and a Nucleo-F401RE. Do you have any idea, what to do, to fix this?

Probably you are not in the dialout or plugdev linux group as this is required for raw access because of libusb.

@vmishev and @framlin could you test with latest tools from the master branch and verify if it works or is still broken?

Closing due to inactivity of original issuer. Please test with latest v1.2.0 or master. Feel free to open a new issue if there are still problems. Thank you for your contribution.

@vmishev @framlin: I know this is old, but can you tell us, if you could manage to get it working with Release v1.2.0?

As it appears, there is a feasible solution to this problem in #427.
Since the provided information in this issue is not sufficient and there is no possibility to reproduce, we can not investigate this any further. Thus I am closing this ticket now.


Sending Windows Event Collector logs to graylog

Hello,
I want to setup Windows Event Collector on one machine and then from that machine send all the logs to graylog with an agent as noted in this readme. Is this something that is possible to do with the agents?
https://docs.microsoft.com/en-us/windows/win32/wec/windows-event-collector

We are using GitHub issues for tracking bugs in Graylog itself, but this doesn't look like one. Please post this issue to our discussion forum or join the #graylog channel on freenode IRC.
Thank you!


Batch: Add Changelog for Data Plane

Description
Add a changelog for the Track 2 Azure Batch Java SDK changes
Thie changelog is for this recently merged PR (forgot to add the changelog for it): https://github.com/Azure/azure-sdk-for-java/pull/37319
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


"Killed" happening while fitting large dataset

I'm getting terminal killed print; not sure why yet (memory usage seems good, but CPU usage is pretty much maxed out).
Here's the snipped:
import pacmap
import numpy as np

# shape is (9268108, 3584)
# dtype is float32
X = np.load("X.npy", mmap_mode="r")

n, dim = X.shape
n_neighbors = 10

scaled_dist = np.ones((n, n_neighbors)) # No scaling is needed

scaled_dist = scaled_dist.astype(np.float32)

# shape is (9268108, 2)
# dtype is int32
pair_neighbors = np.load("pair_neighbors.npy", mmap_mode="r")

embedding = pacmap.PaCMAP(n_components=2, n_neighbors=n_neighbors, MN_ratio=0.5, FP_ratio=2.0, pair_neighbors=pair_neighbors, verbose=True, apply_pca=True)

X_transformed = embedding.fit_transform(X, init="pca") # <-- killed here

Wondering if there's anything obvious I'm doing or not doing would run afoul of the fitting process.. It's not hitting "iterations" at all or reporting any loss...
It's also not lost on me that this is a rather large dataset. Assuming the memory requirements are simply too large here and i can do nothing about it, how could I calculate an appropriate embedding dimensionality to downcast the dataset into? (whether by PCA autoencoder or something else)

It's hard to tell -- the dataset is very big and the dimensionality is also high here. I only tried dataset as large as ~2M. Nevertheless, the memory consumption should still be manageable. I noticed that you set verbose=True when initializing PaCMAP instance. Do you know which stage your script was killed?

It's hard to tell -- the dataset is very big and the dimensionality is also high here. I only tried dataset as large as ~2M. Nevertheless, the memory consumption should still be manageable. I noticed that you set verbose=True when initializing PaCMAP instance. Do you know which stage your script was killed?

Thanks for the reply!
It happens before any of the verbose prints, so before preprocess_X() (the X is normalized print) as far as I can tell. Adding a whack of prints to try and determine the kill point now.
I'd be forever thankful if you could give me any tips re: this use case. Will give a more detailed crash report ASAP!

Oh I see. I guess the problem is within the PCA initialization, where the memory consumption can be huge (O(N^2)). Unfortunately PCA is currently hard-coded into PaCMAP so I don't have a solution that's immediately ready -- perhaps you can try downsampling your data by sample 1/10th data points out of it.

Looks like the killing line was at the start of PaCMAP.fit():
X = np.copy(X).astype(np.float32)

https://github.com/YingfanWang/PaCMAP/blob/3637bc5735f21c50718c90b914eec71f173a9654/source/pacmap/pacmap.py#L877
I'm guessing this was a defensive measure? Commenting this out allows me to start prepossessing X function, so will wait to see how this goes and report back with results! (the PCA will probably take a while before possibly failing).
RE: subsets vs pre-dimensionality reduction
Taking a subset of the data is definitely an option but i'd probably prefer to downsample the embedding dim first. I can certainly bring my own PCA model (would embedding my dataset into a lower dim PCA and then putting it through PaCMAP with init="pca" turned off essentially be equivalent to having it turned on?). I'm also interested in other kinda of dim-reducers.
Any idea what a safe or optimal dimension to aim for might be in general? Any idea how to estimate memory usage given a dataset length and component count?
PaCMAP looks like the highest quality dimensionality reducer so I'm going to give it my effort to make it work for this case! (advice always appreciated!)

Yes, this is a defensive measure as our functions are set to work only for fp32. It could be possible that your machine is running out of memory since your data is huge.
Regarding the dimension, I would suggest try to do dimension=100 for a first trial, but the number of dimension really depends on the type of data you are working with, so additional experiments are required.

Yes, this is a defensive measure as our functions are set to work only for fp32. It could be possible that your machine is running out of memory since your data is huge.
Regarding the dimension, I would suggest try to do dimension=100 for a first trial, but the number of dimension really depends on the type of data you are working with, so additional experiments are required.

Managed to get the job done in the end, and in fairly reasonable wall clock time...
Tricks:

prepare the annoy tree before hand (the longest part of the process)
multiprocess-compute top-k-neighbors for the whole dataset with the annoy tree
precompute neighbor/MN/FP pairs
precompute an incremental PCA and get the embedding initialization (Yinit)
Pytorch-ify the final pacmap optimization loop (pacmap_grad(), update_embedding_adam(), pacmap())

Running the final optimization loop with large batch size on GPU via pytorch was the biggest time save (turns the task from hundreds of hours to tens of minutes!). Some of these pre-compute steps aren't absolutely necessary memory-wise (eg: I could still compute a PCA/top-k-neighbors/neighbor pairs at run time to get the init, but memory has to be carefully managed and cleaned, and if something goes wrong or the process needs to be restarted, the wait time became extreme.
Having all the bits nicely pre-prepared also means that I can re-run the optimization steps with different hyperparams to try and improve results (20 minutes on an H100).
I still have some experimenting to do with those various hyper-params (eg. n_neighbors, lr, iterations-per-phase, etc); not yet sure how everything ends up impacting the memory and wall-clock performance of the final loop, but it's surprisingly lean for this dataset once all the heavy lifting is precomputed.
Got any tips or insights that might be relevant to my big data use-case?
Here's what 9.6M embedded text documents ended up looking like (2-Dim pacmap, colored by a 3-dim PCA transformation)

Closing due to issue resolved (described above)


elasticClient.Index() or Bulk() when index doesn't exist throws exception - no such index

NEST/Elasticsearch.Net version:
Elasticsearch version:
Description of the problem including expected versus actual behavior:
in elasticsearch 2.3.4, calling elasticClient.Index() or Bulk() with a new index when index doesn't exist throws exception - no such index, whereas in early version of elasticsearch e.g. 2.3.1, elasticsearch will automatically create index if it doesn't exist
Steps to reproduce:

call elasticClient.Index() or Bulk() for a new index
receive server error  - no such index
Index() or Bulk() will automatically create index if it doesn't exists, with early version elasticSearch e.g. 2.3.1

Provide ConnectionSettings (if relevant):
Provide DebugInformation (if relevant):

Describe the feature:

Hey @mikewanguk, apologies it taken some time to respond to this. I've just tried this running NEST 2.5.0 against Elasticsearch 2.3.4 and 2.3.5 and don't see this issue. Here's a reproduction
void Main()
{
	var pool = new SingleNodeConnectionPool(new Uri("http://localhost:9200"));
	var defaultIndex = "default-index";
	var connectionSettings = new ConnectionSettings(pool)
			.DefaultIndex(defaultIndex)
			.PrettyJson()
			.DisableDirectStreaming()
			.OnRequestCompleted(response =>
				{
					// log out the request
					if (response.RequestBodyInBytes != null)
					{
						Console.WriteLine(
							$"{response.HttpMethod} {response.Uri} \n" +
							$"{Encoding.UTF8.GetString(response.RequestBodyInBytes)}");
					}
					else
					{
						Console.WriteLine($"{response.HttpMethod} {response.Uri}");
					}

                    Console.WriteLine();

					// log out the response
					if (response.ResponseBodyInBytes != null)
					{
						Console.WriteLine($"Status: {response.HttpStatusCode}\n" +
								 $"{Encoding.UTF8.GetString(response.ResponseBodyInBytes)}\n" +
								 $"{new string('-', 30)}\n");
					}
					else
					{
						Console.WriteLine($"Status: {response.HttpStatusCode}\n" +
								 $"{new string('-', 30)}\n");
					}
				});

	var client = new ElasticClient(connectionSettings);

    if (client.IndexExists("bar").Exists)
        client.DeleteIndex("bar");

    if (client.IndexExists("baz").Exists)
        client.DeleteIndex("baz");

    client.Index(new Document { Field = "foo" }, i => i.Index("bar"));

    client.Bulk(b => b
        .IndexMany(new[] { new Document { Field = "foo" } })
        .Index("baz")
    );
}

public class Document
{
    public string Field { get; set; }
}

which returns
HEAD http://localhost:9200/bar?pretty=true

Status: 404

------------------------------

HEAD http://localhost:9200/baz?pretty=true

Status: 404

------------------------------

POST http://localhost:9200/bar/document?pretty=true
{
  "field": "foo"
}

Status: 201
{
  "_index" : "bar",
  "_type" : "document",
  "_id" : "AVmGRX0EVdpPHEynTpAb",
  "_version" : 1,
  "_shards" : {
    "total" : 2,
    "successful" : 1,
    "failed" : 0
  },
  "created" : true
}

------------------------------

POST http://localhost:9200/baz/_bulk?pretty=true
{"index":{"_type":"document","_id":null}}
{"field":"foo"}

Status: 200
{
  "took" : 282,
  "errors" : false,
  "items" : [ {
    "create" : {
      "_index" : "baz",
      "_type" : "document",
      "_id" : "AVmGRX4vVdpPHEynTpAc",
      "_version" : 1,
      "_shards" : {
        "total" : 2,
        "successful" : 1,
        "failed" : 0
      },
      "status" : 201
    }
  } ]
}

------------------------------

Is it possible that automatic index creation has been set to false on the cluster?

Closing this due to failure to reproduce, @mikewanguk still looking forward to hear back from though! 👍


Add methods for calling the delta link in GraphResponse for Collection

Collections may return a response with a delta link. GraphResponse should have a method for calling the next delta link when the inner value is a Collection.

This has been implemented previously.


Update documentation for Amp* functions

All combinators and helpers should be properly documented.

Closing this one, will create smaller issues.


Disallow multiple clusters backing up into the same bucket (and prefix)

We should forcibly not allow multiple clusters to back up into the same bucket. Once we have subdirectory/prefix support, multiple clusters can share the same bucket, but they can't share the same subdirectory.
We could use the uid of the default namespace in each cluster, as it's impossible to delete that namespace.
@heptio/product-team @heptio/bluesteelsquad

@rosskukulinski if we only allow a single cluster to write to (bucket,prefix), it seems ok to share a bucket.

Closing this in favor of an epic #726


Z-Wave Climate Device not reporting State

Home Assistant release with the issue:

0.74.1
Last working Home Assistant release (if known):
??
Operating environment (Hass.io/Docker/Windows/etc.):

Ubuntu
Component/platform:

zwave climate
Description of problem:
ZWave Climate does not report state despite state being available in attributes.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
zwave:
  usb_path: /dev/zwave_stick
  network_key: "XXXXXX"

Additional information:
As you can see below, the device state is "Cool" but the operating_state in the attributes is "Idle". My device is actually Idle. For this device, the state never changes regardless of the operating_state of the device.

The state is set to operation_mode, operating_state is just what the thermostat is currently doing and is not the current state of the device.  Zwave climate devices are working as expected.

Your thermostat is set cool operation_mode, but since current_temperature 73 is lower than temperature 75, its operating_state is Idle
It looks perfect.
I am going to close it. Feel free to reopen it if you have any question.

This is not how generic_thermostat works. Generic Thermostat uses operating_state for the state of the device. Additionally, there's a green bar in the graph for these devices to indicate when the device is actively cooling. In the case of my zwave thermostat, it appears to ALWAYS be cooling... even when it's idle. And generic_thermostat doesn't have a operating_state attribute at all.

If you believe that it is generic_thermostat and the UI code for the green car that are both incorrect instead of the zwave climate code, I will file a new bug against generic_thermostat instead.


add *transferAllAssets tests

These replace concludePushOutcomeAndTransferAll.test.ts and pushOutcomeAndTransferAll.test.ts.

I'm going to try and rewrite history here to make the diff easier to read.

Partial success. Note to reviewer: just look at fb34ba69934fd559a20c1c8c0e079a0c5d02cfe9 (the only thing missing is the filename change).

Partial success. Note to reviewer: just look at fb34ba6 (the only thing missing is the filename change).

That's great! Thanks.

I read this, and made sense of it as I went, but there's a fair amount of detailed implementation in here (parsing data from loosely typed objects, etc). I think it's good to merge this in but keep in mind that we may have to circle back when getting the target branch running successfully against Circle.

Yeah we will need to get a ✅  on #3624 before merging that in to master. These tests pass for me locally, so I am hoping when it all comes together it just works or only requires some small tweaks.


Rotating player and enemies

Be able to rotate the player so it can shoot in any direction
Be able to rotate enemies to have them moving in different direction

Player rotates but the shooter doesn't. Also the attached UI rotates with the player (weird how the UI is attached but the shooter isn't).
Need to add rotation configurations for the setting menu


[$250] Chat - Plus/FAB icon menu can't be opened by pressing Enter on keyboard

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

Version Number: 1.4.63-7
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Issue found when executing PR https://github.com/Expensify/App/pull/40425
Action Performed:

Launch new Expensify
Open any chat and click on the text box for a new message
Press SHIFT+TAB on the keyboard to highlight the Plus icon
On the Desktop app press ENTER on the keyboard

Expected Result:
The Plus icon on chats can be highlighted and pressed using just the keyboard and TAB or SHIFT+TAB
Actual Result:
The Plus icon on chats can be highlighted on Desktop or Chrome but when pressing ENTER the menu appears and is immediately dismissed. On Safari using the keyboard and TAB or SHIFT+TAB does not get the plus icon highlighted
It's the same on the main FAB icon
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/74f809d6-de69-484d-a52f-6701677beb99
https://github.com/Expensify/App/assets/78819774/eb5d6828-b6c4-4730-848f-ffa933f5d075
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01f7e1c8f66a91018d
Upwork Job ID:<PHONE_NUMBER>766507520
Last Price Increase: 2024-04-24

Issue OwnerCurrent Issue Owner: @sakluger

@sakluger  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #vip-vsp

Asked Rory in Slack about the expected behavior here: https://expensify.slack.com/archives/C01GTK53T8Q/p1713811802815369

It sounds like this same issue has already been fixed several times (ex. https://github.com/Expensify/App/pull/40642 and
https://github.com/Expensify/App/pull/39924). Let's make sure that whatever solution we go with is a global solution.
Accordingly, I'm going to increase the bounty back to $250.

@dragnoir, thanks for your latest proposal but @bernhardoj proposed the same thing just yesterday over here, after preparing the upstream PR and getting it merged. So I'm going to just put this on HOLD for https://github.com/Expensify/App/issues/33502 for now and let @bernhardoj finish upgrading react-native-web.

Moving to weekly while it's on hold.

Still on hold.

Still holding for https://github.com/Expensify/App/issues/33502 which is blocked on an upstream issue.

@miljakljajic assigning you to help watch this issue while I'm OOO through May 31. I'll take back over if it's still open when I return.
All that needs to be done for now is to check on the hold issue https://github.com/Expensify/App/issues/33502 weekly to see if it's made any movement.

Removed @miljakljajic since I'm back.
The PR linked to the hold issue is merged and on Staging. @roryabraham does that mean that we can start working on this one now, or should we wait until it's on prod?

@dragnoir good catch. Any idea why updating RN fixed open behavior but did not fix the close behavior?
As for payment, no payment is due for the linked PR because payment is being handled in another linked issue.

@dragnoir friendly bump - do you know why the close behavior wasn't fixed by updating RN?

@sakluger hi, I don't think it's possible to highlight the FAB anymore to close it. Please see the video below. The TAB-bing focus is trapped within the FAB menu (it's the behavior for all popover/modal)
https://github.com/Expensify/App/assets/50919443/12d2c825-c400-4d0b-b4df-31b6a320c3b1

Hey @bernhardoj, good point - thanks for pointing that out! We can close this issue out now.
Since the linked PR was paid out on another issue, no payment is due here. For any questions, please bring it up in Slack. Thanks!


Fix typo

Fix minor typo in an exception.
Really I'm just making this PR so a new build kicks off and runs against the latest pyasn1 and pyasn1-modules since it's been months since the last build, to double check that they're safe to upgrade to the latest versions.

Oh man, is that the trick? Not do anything and trick people into contributing because they want to see the CI run? ;). Thanks!

@hynek You could add a daily or weekly cron job to Travis CI :)


Is there a way to detect UTM parameters ?

utm parameters are marketing parameters. Once in the analytics dashboard, you may start to see which ads campaigns worked well, and which did not. As far as I can see, in active_analytics, "referrer" is here, but not  "source" "campaign" or "medium" that are often see in classical analytics dashboard - these data comes from utm parameters.
BTW thanks for this great gem.

Thanks. Not yet.


Live search, s-replace: div results is not replaced correctly

Question
I have included a sprig component to my site:
{{ sprig('_partials/search-query', {
            query: craft.app.request.getParam('query')})
}}

And in my _partials/search-query file, I have:
{% if sprig.isInclude %}
    {% set query = query ?? '' %}

    <input sprig s-trigger="keyup changed delay:200ms" s-replace="#result" type="text" name="query" value="{{ query }}">

        {% if query %}
            {% set entries = craft.entries().search(query).orderBy('score').all() %}
            {% for entry in entries %}
                {{ entry.title }}
            {% endfor %}
        {% endif %}

{% endif %}

So when I test this, it triggers the sprig search function (ajax):
URL :
http://mysite.test/index.php?p=actions/sprig/components/render&query=test&sprig%3AsiteId=3efdba262d9eeb8df219de7ad3f88f764a1b121279be29c506693f28c98ca41f3&sprig%3Atemplate=2da19ff12bba17c7ad759e55f1ac15e08d93a0626c3d3a9258ead498e81aae6e_partials%2Fsearch-query&sprig%3Avariables[query]=dfaafcea991bb3710a99f62d0a7d54a8fd3ededca5625c840de62d0309bc4634
And I get nothing on the response.
I saw that the expected result, as in the demo, should be something like:

    <input sprig="" s-trigger="keyup changed delay:300ms" s-replace="#results" type="text" name="query" value="s" autocomplete="off" placeholder="Search" hx-get="https://putyourlightson.com/index.php?p=actions/sprig/components/render" hx-trigger="keyup changed delay:300ms" hx-select="#results" hx-target="#results" hx-swap="outerHTML">

Sprig

Snaptcha

Sherlock

Plugin Sales

Amazon SES

CP Style Guide

Sendgrid

Sitemap

But instead, I'm getting nothing (not even the

 with empty results).
Am I doing something wrong?
Additional context
Sprig version: 1.6.0
Craft CMS version: 3.6.16

By wrapping your entire component in {% if sprig.isInclude %}...{% endif %}, you are saying that the contents should only be output when the component is included in its parent template and not in AJAX requests.
I suggest you start out with the sample live search component at https://putyourlightson.com/sprig/cookbook#search and tweak as necessary from there.


RxMoya linker error for watchOS via project & carthage (Xcode 9.3.1)

When building RxMoya via Carthage and in the Xcode project with "Generic watchOS device", I get a linker error.  The version in question is Moya 11.0.2 in Xcode 9.3.1, and I've also attempted and failed to build with current master too.  From playing around with the project, it also affects ReactiveMoya.
The full linker error is:
Undefined symbols for architecture armv7k:
  "_OBJC_CLASS_$_UIImage", referenced from:
      objc-class-ref in SignalProducer+Response.o
ld: symbol(s) not found for architecture armv7k

I do have 2x separate workarounds as things stand, neither of which I feel viable for contributing to the codebase:

Adding Image.swift to the RxMoya target (as well as Moya).
Adding these imports to the top of Observable+Response.swift and  Single+Response.swift:

#if os(watchOS)
    import UIKit
#endif

The impression I get is tvOS and iOS automatically brings in UIKit.framework, whereas watchOS doesn't unless you explicitly import it in your target.

Hey @simonrice, thank you so much for a very detailed issue! 💯
So this is something I noticed as well when I started migrating to Xcode 9.3 in #1656. I actually did what you also suggested in second point, importing UIKit.UIImage where possible. I think it fixed the original problem you encountered but I've got stuck on some other error that happened after the fix, probably now connected more to the Carthage than Moya, but still I didn't finish the job, unfortunately.
If you have some experience in Carthage and want to help out in finishing the PR, please hesitate to do so 😉

Fixed by #1656.

Oh, thanks for closing it @simonrice, totally forgot about the issue 😄 We still need to prepare a new major version to get this through, but in the meantime you can use development branch.


community: Fix LlamaContentFormatter to match actual input/output formatting

Description: The LlamaContentFormatter in the langchain_community.chat_models.azureml_endpoint does not work because it doesn't actually comply to the output of Llama models. The input/output formats are documented here:
Deploy Llama2 Chat Models - Chat API
This PR creates a LlamaContentFormatter that works with the documented API.
Issue: N/A
Dependencies: N/A
Twitter handle: parkerdhancock

closing as i believe this was resolved in #14560, let me know if i'm missing something


Fix two sensor graph footer issues

Before:
After:

If a (non-open) footer isn't full width, there is a gap between the .info and .footer elements that gets filled with the background-color (which is also what determines the inner-color color, so we can't just set it to black).

Fixed by using flexbox to make the footer element (but not the graph inside) grow to fill the gap.

The graph draws on top of the inner corner, causing subtle discoloration.

Fixed by using overflow: hidden to clip the graph to the black region.

I originally discovered the first issue by creating a horizontal stack of two sensors at the end of a vertical stack. I noticed the second issue while investigating the first.
I may have put these rules in a more specific rule than would be ideal, but as far as I know this is the only situation with this issue. Graphs as headers have completely different brokenness that I'm less interested in personally investigating.
Test case:
type: sensor
graph: line
entity: sensor.husky_battery_level
card_mod:
  class: middle-right
layout_options:
  grid_columns: 3
  grid_rows: 3

Excellent! I can probably employ this method in other card types, as well. It's way less janky than some of the fixes I've implemented for similar issues.


Wrong type hint used in \Kiener\MolliePayments\Service\OrderService - EntityRepository instead of EntityRepositoryInterface

In order to support EntityRepository decoration like e.g. Shopware does internally for some of there repositories, you should use EntityRepositoryInterface instead of EntityRepository.
It provides all the same methods.
current code:
    public function __construct(
        EntityRepository $orderRepository,
        EntityRepository $orderLineItemRepository,
        LoggerInterface $logger
    )
    {
        $this->orderRepository = $orderRepository;
        $this->orderLineItemRepository = $orderLineItemRepository;
        $this->logger = $logger;
    }

I had to decorate the repository in a project and now this fails.

@AndreasA you are correct. Thanks for your feedback. We will replace this in an upcoming release.

@AndreasA This will be fixed in the release of this week.

Thanks 👍


MQTT subscribe not working since 0.69

Home Assistant release with the issue: 0.69
Last working Home Assistant release (if known): 0.68
Operating environment (Hass.io/Docker/Windows/etc.): Docker image
Component/platform: Custom
Description of problem:
When I want to subscribe in a custom component I now get this since 0.69:
homeassistant.exceptions.HomeAssistantError: topic needs to be a string!
Additional information:
This used to work perfectly. I used to do loader.getcomponent and then mqtt, but that gave an error. So I changed it to "mqtt = hass.components.mqtt". Then I got the error that my topic isn't a string.
Doesn't matter what I enter there, even "test" as topic gives the same error. When I use the async function (Which seems to be called in the source later) I get no error, but it doesn't work either.
So what got changed that broke this?

I recently did a change to MQTTs topic validation, but it's well tested and the error doesn't seem to be related to the change from what I see. Could you share your code snippet?

So this is how I get my mqtt object:
mqtt = hass.components.mqtt
Then I want to subscribe to a topic:
mqtt.subscribe(hass, "lala", message_received)
Where hass is the hass object from the setup function.
With 0.68 and earlier I used to do this:
mqtt = loader.get_component(‘mqtt’)
to get the mqtt object, but that also gave and error (get_component() missing 1 required positional argument: ‘comp_or_platform’), and I believe to do it correctly with the new way?
I have the idea that the parameters are somehow not right, because when I add two more parameters (the QOS and encoding parameter) I get an error that the subscribe function needs 5 parameters, and not  6, while I really only entered 5!

Went back to 0.68 for the time being. My custom components work perfectly now.


cleanup: Mark uncommon CMake options as "advanced".

This removes the options from the CMake UIs, and makes it easier for our
users to discover the interesting tweaks.

Rebased to pickup the C++17 configs.


REST conventions for web API

Hello @Ali-Razmjoo ,
I was thinking that all of our api does not follow the REST conventions which are like the standard guidelines to be followed when designing the Web api's
for eg : in all of the api's we have shell methods=["GET", "POST"] but they should be just get ,as they are just reading the databases and giving the values
if they modify the database,for example store the username or some attribute then it should be POST.
and also the names of the URI does not follow REST conventions
for example
like it should be segregated into /network and /honeypot and then the URI names should also be shortened
top_ten_ips_in_network_events_by_date should be something like /network/top_ips_by_date

I followed these links for reference : https://restfulapi.net/resource-naming/ and https://blog.miguelgrinberg.com/post/designing-a-restful-api-with-python-and-flask
What do you think @Ali-Razmjoo ? based on the feedback i would work on it.

fixed in https://github.com/zdresearch/OWASP-Honeypot/pull/37


New version: Aserto.Topaz version 0.31.8

Pull request has been created with komac v2.1.1 :rocket:

For moderators
This account is automated by Github Actions and the source code was created by CoolPlayLin. If you have any questions about any pull request, don't hesitate to ping Exorcism0666.

[!warning]
With the new version of Komac, numerous Pull Requests may contain bugs or unwanted elements. If such issues arise, please tag russellbanks so that these bugs can be addressed in future updates.

[!important]
Please carefully review these Pull Request before merging. If it is a Pull Request for removing incorrect content and the URLs are issue, free upon manual checking, please close this Pull Request directly. (It is best to inform Exorcism0666 of the closure.)

良い一日をお過ごしください！


[SMALLFIX] Supply the name of variable to Preconditions.checkNotNull in /alluxio/job/server/src/main/java/alluxio/master/job/JobCoordinator.java line111

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2751/
Test FAILed.


Implementing all 4 typical "messages" as in proftpd and other commercial FTP servers

The FTP protocol allows for (and many open source as well as commercial FTP servers support) custom messages to be sent to the client in 4 specific moments of the session:

upon client connection
upon successful user auth
upon failed user auth
upon receipt of the QUIT command (just prior to closing the connection)

We noticed that ftpserverlib only supports point number 1, therefore we would like to contribute some code to implement the other 3.
Our notes:

Implementing point number 4 was quite straightforward by means of an "extended" interface (like you did with several other things)
You may not like our implementation of point number 2 and 3 here above, because it breaks backwards compatibility by changing the AuthUser method prototype; this was the easiest way to implement it, but if you want to do it in a more backward-compatible way, by all means, feel free to totally reject our implementation

Regardless of the implementation details, we see there's value to this functionality, so we hope you'll consider including it into your excellent library.
Thank you,
kind regards.

Hello,
thank you for this contribution.
I took a quick look at your patch. Do you need to set different messages based on the user?
If the messages are global to the server maybe you can just add some new settings and thus keep backward compatibility. Not sure if I missing something

Having those messages returned based on the user can be very useful indeed. Some enterprise customers may use variables or even need to return a totally different message based upon who's logging in, hence the need to build those messages in real-time, in code, based upon the user.
Of course I do understand the reluctance for breaking backward compatibility, I promise I don't take this lightly. If there is a way to do it without breaking backwards compatibility I am all for it.
I am simply reinforcing that the functionality - in and by itself - would be extremely useful to have in a user-dependent fashion.

Guys, I've seen the latest codecov reports, but I am not sure they are correct (unless I am missing something obvious, which is entirely possible). In fact, I have added both of my changes to the library's test suite, so I am not sure why codecov now says that those lines are "not covered by tests" (when indeed they are). Would it be possible for you to take a quick look and let me know what I'm missing (if anything)?
Thanks!

I have also fixed the generic HASH function... it was using strings.SplitN which fails if the path contains spaces, and substituted it with a custom advSplitN func that correctly splits strings in case there are quoted substrings that contain spaces (as the FTP protocol RFC requires).

I have also fixed the generic HASH function... it was using strings.SplitN which fails if the path contains spaces, and substituted it with a custom advSplitN func that correctly splits strings in case there are quoted substrings that contain spaces (as the FTP protocol RFC requires).

Thanks for this fix. I think it should be a separate PR and writing a test cases should be quite simple.
As for the original issue, I have added to my TODO to write the required test cases, but I'm very busy at the moment, it might take some time


Is usage of the SSIS "Table lock" checkbox desirable or not? when creating a new "OLE DB destination" component in SSIS.

I am simply wondering: Is usage of the SSIS "Table lock" checkbox desirable or not?
This checkbox is checked .. by default .. when creating a new "OLE DB destination" component in SSIS.
We have several SSIS processes running.
At certain times of the day, trying to just refresh the list of tables in SSMS for a database gives me message "Lock request time out period exceeded. (Microsoft SQL Server, Error: 1222)"
Am thinking this is related to this SSIS option ?
I will go with the default setting for now.

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

ID: 2f2c8c53-6371-b844-3b7b-8e93b8601683
Version Independent ID: a5a62a2f-e91c-d2bd-e489-a612ea049b1a
Content: Prerequisites for Minimal Logging in Bulk Import - SQL Server
Content Source: docs/relational-databases/import-export/prerequisites-for-minimal-logging-in-bulk-import.md
Product: sql
Technology: data-movement
GitHub Login: @MashaMSFT
Microsoft Alias: mathoma

Hi @foral66 , thanks for contacting Microsoft! Let me see if I'm understanding your issue correctly -

You have a number of SSIS processes running, and you have the Allow Table Lock option enabled (since it's enabled by default).
During the day, you are using Object Explorer in SQL Server Management Studio, expanding Databases > Tables, and when you do so,  you're seeing the error "Lock request timeout period exceeded"
You believe that the error is because your SSIS processes are running, and causing your tables to lock.

Am I understanding correctly?
I suppose you could try unchecking to box to see if that resolves the behavior. However, there may be other consequences for choosing that option, and there's a reason why it's checked by default. Since this form of contact is intended explicitly for issues with the documentation,  your best bet would be to engage the community, or open a support ticket directly with Microsoft. More information can be found here:  https://docs.microsoft.com/en-us/sql/sql-server/sql-server-get-help?view=sql-server-2017.
My apologies that the SQL docs team is unable to assist you further with your support issue. If you need any thing else from the content perspective, please let us know. Otherwise, we'll proceed with closing out this Git Issue in a few business days.
Thanks again and hope you have a wonderful rest of your day!


Fix missing chains results

Previously, the patch was overwriting the results from the template-build pipeline. This change makes it so results are appended to the list of existing results.
Before this change:
🐚 oc kustomize pipelines | yq -P .spec.results
- name: JAVA_COMMUNITY_DEPENDENCIES
  value: $(tasks.build-container.results.JAVA_COMMUNITY_DEPENDENCIES)
---
null
---
- name: REPORT
  value: $(tasks.verify.results.REPORT)
---
- name: JAVA_COMMUNITY_DEPENDENCIES
  value: $(tasks.build-container.results.JAVA_COMMUNITY_DEPENDENCIES)
---
- name: IMAGE_URL
  value: $(tasks.build-container.results.IMAGE_URL)
- name: IMAGE_DIGEST
  value: $(tasks.build-container.results.IMAGE_DIGEST)
- name: CHAINS-GIT_URL
  value: $(tasks.clone-repository.results.url)
- name: CHAINS-GIT_COMMIT
  value: $(tasks.clone-repository.results.commit)

After this change:
🐚 oc kustomize pipelines | yq -P .spec.results
- name: IMAGE_URL
  value: $(tasks.build-container.results.IMAGE_URL)
- name: IMAGE_DIGEST
  value: $(tasks.build-container.results.IMAGE_DIGEST)
- name: CHAINS-GIT_URL
  value: $(tasks.clone-repository.results.url)
- name: CHAINS-GIT_COMMIT
  value: $(tasks.clone-repository.results.commit)
- name: JAVA_COMMUNITY_DEPENDENCIES
  value: $(tasks.build-container.results.JAVA_COMMUNITY_DEPENDENCIES)
---
null
---
- name: REPORT
  value: $(tasks.verify.results.REPORT)
---
- name: IMAGE_URL
  value: $(tasks.build-container.results.IMAGE_URL)
- name: IMAGE_DIGEST
  value: $(tasks.build-container.results.IMAGE_DIGEST)
- name: CHAINS-GIT_URL
  value: $(tasks.clone-repository.results.url)
- name: CHAINS-GIT_COMMIT
  value: $(tasks.clone-repository.results.commit)
- name: JAVA_COMMUNITY_DEPENDENCIES
  value: $(tasks.build-container.results.JAVA_COMMUNITY_DEPENDENCIES)
---
- name: IMAGE_URL
  value: $(tasks.build-container.results.IMAGE_URL)
- name: IMAGE_DIGEST
  value: $(tasks.build-container.results.IMAGE_DIGEST)
- name: CHAINS-GIT_URL
  value: $(tasks.clone-repository.results.url)
- name: CHAINS-GIT_COMMIT
  value: $(tasks.clone-repository.results.commit)

Signed-off-by: Luiz Carvalho<EMAIL_ADDRESS>

@stuartwdouglas @psturc - as part of @psturc verifying the component based kcp flow on jvm-build-service that @stuartwdouglas recently sorted out, it may make sense for @psturc to first verify with current main branch, and then re-verify with either this PRs changes in his local env / producing a bundle with these changes, or re-verify the flow after this PR merges.

FWIW, without this change, you'll see that the image is signed and an attestation is created. However, only the TaskRun is attested, not the PipelineRun. This means the attestation image contains a single layer, instead of the expected two. Let me know if you need any help verifying this behavior in the kcp flow.

Merging since this is clearly a fix.


Revert "Add new "setupTestFramework" option (#4976)"

This reverts commit dbfd7877c450058cd7485cbc2530e8f9b1fa6dba. I had to manually revert because the Github UI seems broken.

This is related to the reversion of #4970. Since it got also reverted, this out-of-the-sandbox option is not needed anymore (at least for now), so it's better to get back to the original setup.

Codecov Report

Merging #5025 into master will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5025      +/-   ##
==========================================
- Coverage   60.42%   60.37%   -0.05%
==========================================
  Files         198      198
  Lines        6640     6637       -3
  Branches        3        4       +1
==========================================
- Hits         4012     4007       -5
- Misses       2628     2630       +2

Impacted Files
Coverage Δ

packages/jest-config/src/valid_config.js
100% <ø> (ø)
:arrow_up:

packages/jest-config/src/index.js
23.8% <ø> (ø)
:arrow_up:

packages/jest-jasmine2/src/index.js
5.97% <ø> (+0.25%)
:arrow_up:

packages/jest-config/src/normalize.js
92.94% <ø> (ø)
:arrow_up:

packages/jest-validate/src/warnings.js
44.44% <0%> (-55.56%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0ce82e6...98a897d. Read the comment docs.

Alright, let's do this for now.


About the worlddreamer agent render FPS

Hello,thank you for your great work!
I wonder the FPS about the worlddreamer agent's render speed!
Thank you for your impressive work a lot!

The inference process requires approximately 20 seconds per frame with 50 sampling steps.

The inference process requires approximately 20 seconds per frame with 50 sampling steps.

OK  thanks for you reply and your great work!


Empty resupply crate

Version: develop https://github.com/LordGolias/antistasi/commit/8c0ea9bdacffc86ebd00146e22bf80d16e31a147
Mod: CBA, TFAR
Multiplayer dedicated server
Side: tested both vs East and vs West
Called in esupply t = crate was empty

This has no information whatsoever on what is the error and how to reproduce it. Please provide more information please.


queries don't seem to support comparison operators ?

Hi, I'm trying to get my head around the fact that comparison operators don't seem to work on queries, I'm pretty sure that this is me being silly but I can't figure it out some here I am.
this works
entries:get({ where = { timeStamp = os.time() } })

this doesn't work
entries:get({ where = { timeStamp < os.time() } })

🤔 the syntax is not yet supported I guess.
If I'm not mistaken you can try something like { where = { timeStamp = "<" .. os.time() } }

🤔 the syntax is not yet supported I guess.
If I'm not mistaken you can try something like { where = { timeStamp = "<" .. os.time() } }

if I try
entries:get({ where = { timeStamp "<" os.time() } })

I have the same issue, which is lua complains that timeStamp cannot be found and I can't run the script.
however I tried this
entries:get({ where = { "timeStamp" < "os.time()" } })

which seems to be doing something but the result is not at all relevant so I guess we can't do this neither.
surely there is a way to have a query where I can use comparison operators, I'm trying to look to some code from other plugins that use sqlite.lua and it seems like some of theme are injecting strings with the full sqlite queries to get results but I'm yet to decipher all of it.

if I try
entries:get({ where = { timeStamp "<" os.time() } })

I have the same issue, which is lua complains that timeStamp cannot be found and I can't run the script.
however I tried this
entries:get({ where = { "timeStamp" < "os.time()" } })

which seems to be doing something but the result is not at all relevant so I guess we can't do this neither.
surely there is a way to have a query where I can use comparison operators, I'm trying to look to some code from other plugins that use sqlite.lua and it seems like some of theme are injecting strings with the full sqlite queries to get results but I'm yet to decipher all of it.

Yes, injected string, that's why you need to set the value to os.time() value while at the same time preappending < so that the interface gets "< val".
Previously I tried to play around with sugar syntax unfortunately lua doesn't allow room to do that. The best we could do is something like that age = (strftime("%s", "now") - strftime("%s", "timestamp")) * 24 * 60, which all result in a string.
you may do something like where = { timeStamp = strftime("%s", "now) < strftime("%s", "timeStamp")} .
you need first to local strftime = sqlite.lib.strftime

Yes, injected string, that's why you need to set the value to os.time() value while at the same time preappending < so that the interface gets "< val".
Previously I tried to play around with sugar syntax unfortunately lua doesn't allow room to do that. The best we could do is something like that age = (strftime("%s", "now") - strftime("%s", "timestamp")) * 24 * 60, which all result in a string.
you may do something like where = { timeStamp = strftime("%s", "now) < strftime("%s", "timeStamp")} .
you need first to local strftime = sqlite.lib.strftime

I tried this and it always returns an empty table.
res = entries:get({ where = { timeStamp = strftime("%s", "timeStamp") < strftime("%s", "now")} })

I tried different things as well such as
res = entries:get({ where = { timeStamp = strftime("%s", "timeStamp") .. " < " .. strftime("%s", "now")} })

I also changed the type of timeStamp in my table, previously I was using "timestamp" as a type and I change to "date" but it didn't make any difference.
how would the function strftime get access to the field timeStamp of my table? is it some magic under the hood?

I tried this and it always returns an empty table.
res = entries:get({ where = { timeStamp = strftime("%s", "timeStamp") < strftime("%s", "now")} })

I tried different things as well such as
res = entries:get({ where = { timeStamp = strftime("%s", "timeStamp") .. " < " .. strftime("%s", "now")} })

I also changed the type of timeStamp in my table, previously I was using "timestamp" as a type and I change to "date" but it didn't make any difference.
how would the function strftime get access to the field timeStamp of my table? is it some magic under the hood?

how would the function strftime get access to the field timeStamp of my table? is it some magic under the hood?

It doesn't.
All strftime fn is return a string. it is syntax magic you can say.


API docs: Argument $field is missing from the Docblock of getField

I am getting several dozen errors in the API documentation of the form:

Argument $field is missing from the Docblock of getField

When it is clear that the field is actually defined in the documentation and, in fact, the description of the method in the class summary includes type information from the doc comment.
This appears to be Issue phpDocumentor/phpDocumentor2#1476 , present in 2.8.0 and 2.8.1. A fix is present in the 3.0 branch but it has not been back-ported.

Underlying issue has been closed and will be in the next 2.8.x release. Will maintain the issue here until the fix appears in a regular build and is verified.


[GSSoC'23] Create a movie recommendation bot with python

Genre-based movie  Recommendations with Integration with Movie Databases.

Sir please assign me this issue. I will resolve it.

Hi
I would like to work on this. Please can you assign it to me?

Hey I have resolved this issue. Here is the link below.
https://github.com/amankumar100/Bot-Collection/tree/main/Movie Recommendation Bot1.

Please explain how will I get points on Girlscript Summer of Code via this contribution.

I would like to work on this issue
Here is how I plan to do it:

User will input his preferred genre of movie (comedy, action, fantasy, etc)
I would use a public dataset like MovieLens dataset
After cleaning and preprocessing , calculate similarity between the movies (cosine similarity technique)
Give the output after data analysis using pandas in python

If you liked the idea , please assign this issue to me.


ibmq_qasm_simulator backend error

Informations

Qiskit Terra version: 0.7
Python version: 3.6.8
Operating system: Debian Linux

What is the current behavior?
When using the backend ibmq_qasm_simulator for a qaoa script, two different errors are encountered.
The first one is just ignored and the script continues running, the second causes it to stop.
Steps to reproduce the problem
I have only encountered this a few times, but it seems sort of random as to when it occurs.
I am testing some QAOA scripts using both IBM Quantum Experience and Rigetti Simulator, all of the current code is on this directory - https://github.com/epelofske/quantum_optimization/blob/master. To reproduce, running combined_qaoa_test.py will likely get the error eventually. I have only tested using the backend ibmq_qasm_simulator, but it might occur when using other backends.
What is the expected behavior?
The following two errors are encountered:

Got a 502 code response to /api/Jobs/5c23aaa40fc4f2005836cb6f/status: 502 Bad Gateway: Registered endpoint failed to handle the request.

Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/urllib3/connection.py", line 141, in _new_conn
(self.host, self.port), self.timeout, **extra_kw)
File "/usr/lib/python3/dist-packages/urllib3/util/connection.py", line 83, in create_connection
raise err
File "/usr/lib/python3/dist-packages/urllib3/util/connection.py", line 73, in create_connection
sock.connect(sa)
TimeoutError: [Errno 110] Connection timed out
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/urllib3/connectionpool.py", line 601, in urlopen
chunked=chunked)
File "/usr/lib/python3/dist-packages/urllib3/connectionpool.py", line 346, in _make_request
self._validate_conn(conn)
File "/usr/lib/python3/dist-packages/urllib3/connectionpool.py", line 852, in _validate_conn
conn.connect()
File "/usr/lib/python3/dist-packages/urllib3/connection.py", line 284, in connect
conn = self._new_conn()
File "/usr/lib/python3/dist-packages/urllib3/connection.py", line 150, in _new_conn
self, "Failed to establish a new connection: %s" % e)
urllib3.exceptions.NewConnectionError: <urllib3.connection.VerifiedHTTPSConnection object at 0x7ff51940e898>: Failed to establish a new connection: [Errno 110] Connection timed out
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.6/dist-packages/requests/adapters.py", line 449, in send
timeout=timeout
File "/usr/lib/python3/dist-packages/urllib3/connectionpool.py", line 639, in urlopen
_stacktrace=sys.exc_info()[2])
File "/usr/lib/python3/dist-packages/urllib3/util/retry.py", line 388, in increment
raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='quantumexperience.ng.bluemix.net', port=443): Max retries exceeded with url: /api/Jobs/5c23cc6f5259d20051b060d3/status?access_token=CENSORED (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7ff51940e898>: Failed to establish a new connection: [Errno 110] Connection timed out',))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/root/all/qiskit/providers/ibmq/ibmqjob.py", line 267, in status
api_job = self._api.get_status_job(self._job_id)
File "/root/all/qiskit/providers/ibmq/api/ibmqconnector.py", line 674, in get_status_job
status = self.req.get(url)
File "/root/all/qiskit/providers/ibmq/api/ibmqconnector.py", line 357, in get
**self.extra_args)
File "/usr/local/lib/python3.6/dist-packages/requests/api.py", line 75, in get
return request('get', url, params=params, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/requests/api.py", line 60, in request
return session.request(method=method, url=url, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/requests/sessions.py", line 533, in request
resp = self.send(prep, **send_kwargs)
File "/usr/local/lib/python3.6/dist-packages/requests/sessions.py", line 646, in send
r = adapter.send(request, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/requests/adapters.py", line 516, in send
raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPSConnectionPool(host='quantumexperience.ng.bluemix.net', port=443): Max retries exceeded with url: /api/Jobs/5c23cc6f5259d20051b060d3/status?access_token= CENSORED (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7ff51940e898>: Failed to establish a new connection: [Errno 110] Connection timed out',))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "combined_qaoa_test.py", line 80, in
print(combined_test2('agave_ibmqx_no_noise.txt'))
File "combined_qaoa_test.py", line 72, in combined_qaoa_test
x = solve_ising_qubo(G, max_clique_qubo_matrix_ibmqx, optimizer, a)
File "/root/all/ibmqx_ising_qubo_qaoa.py", line 65, in solve_ising_qubo
result = qaoa.run(quantum_instance)
File "/root/all/qiskit_aqua/algorithms/quantum_algorithm.py", line 103, in run
return self._run()
File "/root/all/qiskit_aqua/algorithms/adaptive/vqe/vqe.py", line 264, in _run
self._solve()
File "/root/all/qiskit_aqua/algorithms/adaptive/vqe/vqe.py", line 208, in _solve
opt_params, opt_val = self.find_minimum_eigenvalue()
File "/root/all/qiskit_aqua/algorithms/adaptive/vqe/vqe.py", line 348, in find_minimum_eigenvalue
variable_bounds=bounds, initial_point=initial_point)
File "/root/all/qiskit_aqua/components/optimizers/slsqp.py", line 87, in optimize
options=self._options)
File "/usr/lib/python3/dist-packages/scipy/optimize/_minimize.py", line 611, in minimize
constraints, callback=callback, **options)
File "/usr/lib/python3/dist-packages/scipy/optimize/slsqp.py", line 403, in _minimize_slsqp
g = append(fprime(x), 0.0)
File "/usr/lib/python3/dist-packages/scipy/optimize/optimize.py", line 293, in function_wrapper
return function((wrapper_args + args))
File "/usr/lib/python3/dist-packages/scipy/optimize/slsqp.py", line 63, in approx_jacobian
jac[i] = (func(((x0+dx,)+args)) - f0)/epsilon
File "/usr/lib/python3/dist-packages/scipy/optimize/optimize.py", line 293, in function_wrapper
return function(*(wrapper_args + args))
File "/root/all/qiskit_aqua/algorithms/adaptive/vqe/vqe.py", line 291, in _energy_evaluation
result = self._quantum_instance.execute(to_be_simulated_circuits)
File "/root/all/qiskit_aqua/quantum_instance.py", line 156, in execute
has_shared_circuits=self._shared_circuits)
File "/root/all/qiskit_aqua/utils/run_circuits.py", line 268, in compile_and_run_circuits
results.append(job.result(**qjob_config))
File "/root/all/qiskit/providers/ibmq/ibmqjob.py", line 203, in result
job_response = self._wait_for_result(timeout=timeout, wait=wait)
File "/root/all/qiskit/providers/ibmq/ibmqjob.py", line 210, in _wait_for_result
job_response = self._wait_for_job(timeout=timeout, wait=wait)
File "/root/all/qiskit/providers/ibmq/ibmqjob.py", line 389, in _wait_for_job
while self.status() not in JOB_FINAL_STATES:
File "/root/all/qiskit/providers/ibmq/ibmqjob.py", line 273, in status
raise JobError(str(err))
qiskit.providers.exceptions.JobError: "HTTPSConnectionPool(host='quantumexperience.ng.bluemix.net', port=443): Max retries exceeded with url: /api/Jobs/5c23cc6f5259d20051b060d3/status?access_token=CENSORED (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7ff51940e898>: Failed to establish a new connection: [Errno 110] Connection timed out',))"
Suggested solutions
I am not familiar enough with network error handling to know what solution could work. In the meantime, I am trying just using local simulators, and also eventually an IBMQ NISQ device to see if the same error occurs.

I have only encountered this a few times, but it seems sort of random as to when it occurs.

Unfortunately it seems to be indeed due to network issues - could it be that you are running several jobs against IBMQ  (ie. against an online device or simulator) concurrently, or in a short time span? There are some network limits and availability constraints that might be in play in your current scenario.


Invalid cache when building with Carthage

Check List
Thanks for considering to open an issue. Before you submit your issue, please confirm these boxes are checked.

[x] I have read the wiki page and cheat sheet, but there is no information I need.
[x] I have searched in existing issues, but did not find a same one.
[x] I want to report a problem instead of asking a question. It'd better to use kingfisher tag in Stack Overflow to ask a question.

Issue Description
What
I'm getting Invalid cache found for Kingfisher, rebuilding with all downstream dependencies when building Kingfisher with the --cache-builds parameter.
I tried clearing the Carthage cache (rm -rf ~/Library/Caches/org.carthage.CarthageKit) and removing all Carthage related files before installing, but the error will still up, when running the carthage bootstrap command a second time (after the cleanup).
Reproduce
Running carthage bootstrap --platform iOS --cache-builds will produce the bug every time.
Other Comment
I'm not entirely sure with which version this started happening, because we were running a rather old version of Kingfisher (4.10.1) before upgrading to 5.8.1 and now 5.8.2.

Good catch!
I'm not too familiar on how the headers are added. Is that something that Carthage is doing or can the headers be added in the `Kingfisher's build phase?

@EmDee @igorkulman
Thanks for the feedback.
Removing the header files was an intended change in Kingfisher v5.8, since the framework cannot be used in ObjC and there should be no need to generate it. However, it seems that triggering a side-effect (or say bug) of Carthage. See https://github.com/Carthage/Carthage/issues/2882
Carthage fixed it in its master branch, but it is now not clear when they could tag it and release a new version.
As a temporary workaround, I will merge #1308 later today and release a new version. I guess we can remove the ObjC header again later.

@onevcat That'd be great! It will probably take Carthage to release a version, considering their last release was in April.

Done. https://github.com/onevcat/Kingfisher/releases/tag/5.8.3

Just tested – works! Thanks again.


Support pulling images by digest & fix broken tests

@rohansingh @davidxia

hrrm this works locally for me... will have to dig into why tests are failing still

CircleCI's docker client, which is 1.6.2-circleci as of this writing, doesn't pull from public registry v2. So we can't use image hashes as tags. https://github.com/spotify/docker-client/pull/249/files

:+1:


Add Metrics, Console, Logs support to OSO proxy

What does this PR do?
Support metrics, console, logs URL along with API URL in OSO proxy.  Now, metrics URL also dynamically configured with traefik by OSIO provider.
When any request arrived to OSO proxy

if its api, metrics request then it will forward to api or metrics cluster
if its console, logs request then it will redirect request to console cluster

Fix - https://github.com/openshiftio/openshift.io/issues/2342
Motivation
Direction towards removing all direct OSO call from OSIO UI.
More
OSIO provider integration test enhance to consider metrics url.
New component test added which test OSIO middleware without traefik.

This PR is just duplicate of other PR https://github.com/fabric8-services/fabric8-oso-proxy/pull/16.
PR-16, had lot of commits from other PR and closed and this new PR created for better review and merge process.

Replaced by https://github.com/fabric8-services/fabric8-oso-proxy/pull/32


[Fixes #1542] Pdr Dashboard v0.5 - Move calculate_tx_gas_fee_cost_in_OCEAN() to app startup #1542

Closes #1542.

Got the latest changes from main branch and I'm running into an error that crushes the app, I don't know if this was an issue before. I had no conflicts when merging the latest changes.


[fix] elasticserach: Rspec, set released val in mapping

Coverage remained the same at 84.563% when pulling 30e860a1387fde8ae045329e1dcc622be6b5d102 on fix/elasticsearch_rspec into b10b60daf2783e6cabd32e3d093a7cc12ccfacfe on master.