KLD client is running on top the development mode of webpack. It is better to give configuration and documentation to guide users to understand how they are running development using web pack and how is the steps to deploy KLD in a web server or running built-in Node.js server

We will be going to add script to build JS files and save output into a directory that allows the user can copy and paste KLD client on a web server such NGINX or Apache

The current run script "npm start" in KLD client will run webpack development mode that allows developers running KLD client locally for testing but this is not recommended for demo purpose. we should provide built-in web server in Node.js that serves KLD client which is written in Angular for users who want to use a simple Node.js web server within the project rather than running NGINX or and Apache.   By the way, providing predefined scripts for this improvement should be managed in package.json, so developers and users can choose the right deployment for KLD client.

This is a Apache configuration we use it before in our recent testing server.
Template
<VirtualHost *:80>
    ServerName {domain}
    ServerAdmin webmaster@localhost
    DocumentRoot {assetPath}

    ErrorLog ${APACHE_LOG_DIR}/{whiteLabelAppName}.error.log
    CustomLog ${APACHE_LOG_DIR}/{whiteLabelAppName}.access.log combined

    RewriteEngine On
    RewriteCond %{DOCUMENT_ROOT}%{REQUEST_URI} -f [OR]
    RewriteCond %{DOCUMENT_ROOT}%{REQUEST_URI} -d
    RewriteRule ^ - [L]
    RewriteRule ^ /index.html

With <IP_ADDRESS> IP example:
NameVirtualHost <IP_ADDRESS>
<VirtualHost <IP_ADDRESS>:80>
ServerName <IP_ADDRESS>
DocumentRoot /var/www/KLD/client/asset.default
RewriteEngine On
RewriteCond %{DOCUMENT_ROOT}%{REQUEST_URI} -f [OR]
RewriteCond %{DOCUMENT_ROOT}%{REQUEST_URI} -d
RewriteRule ^ - [L]
RewriteRule ^ /index.html

Run KLD Client.   There are 3 ways you can run KLD client in different contexts.
Run KLD Client in Development Mode
With Development Mode, you will run a simple web server that watches all KLD client source files. Each source file will be watched by the KLD build system and entire KLD client web app will be built automatically and the result of the change will be updated on the web browser in real-time when developers make a change on files under src folder.
cd KLD/client
npm start
Open http://localhost:9000
Run KLD Client using Built-In Node.js web server
With Built-In Node.js web server, you will run KLD client using a built-In Node.js based web server.  Yuri have the build KLD client source files using “npm run build”, which the built files will be saved in asset.default folder. You can start the web server using “npm run server” command line.
cd KLD/client
npm run build
npm run server
Open http://localhost:3000
Run KLD Client on an Apache Web Server
You can reuse Apache Web Server to run KLD client which allows you configure advanced web application deployment.
cd KLD/client
npm run build
use “asset.default” folder as an Apache document root folder
If you want to setup an Apache virtual host, the following is the example of KLD client virtual host setting. To assume that your server IP is <IP_ADDRESS> and your KLD GIT repo is relocated at /var/www/KLD
NameVirtualHost <IP_ADDRESS>
<VirtualHost <IP_ADDRESS>:80>
ServerName <IP_ADDRESS>
DocumentRoot /var/www/KLD/client/asset.default
RewriteEngine On
RewriteCond %{DOCUMENT_ROOT}%{REQUEST_URI} -f [OR]
RewriteCond %{DOCUMENT_ROOT}%{REQUEST_URI} -d
RewriteRule ^ - [L]
RewriteRule ^ /index.html


Made uses of Rad more generic using Into<Rad>

This normalizes the use of Rad<S> to the same as the perspective function.

Thanks!


Test Issue created at Mon, 06 Dec 2021 02:22:34 GMT

Test issue body  Mon, 06 Dec 2021 02:22:34 GMT

Comment on issue at Mon, 06 Dec 2021 02:22:35 GMT


Replace vienna-rss.org URLs with vienna-rss.com

Closes #800. Only skipped the CHANGES document.

Thanks!!


Strict templating of values

Describe the solution you'd like
Is there any way to enforce strict templating of values? For example currently I'd like to template an Alertmanager configuration. However this is a template that has other templated values that should not be templated by External Secrets - these are templated by Alertmanager itself.
Secret:
apiVersion: external-secrets.io/v1alpha1
kind: ExternalSecret
metadata:
  labels:
    argocd.argoproj.io/instance: monitoring
    tanka.dev/environment: ac01f6e2b7516fbe884c27716a1be2e96137a278ac0bbbdc
  name: alertmanager-config
  namespace: monitoring
spec:
  data:
  - remoteRef:
      key: kv/data/monitoring/alertmanager
      property: slackApiUrl
    secretKey: slackApiUrl
  dataFrom: []
  refreshInterval: 60s
  secretStoreRef:
    kind: ClusterSecretStore
    name: vault
  target:
    name: alertmanager-config
    template:
      templateFrom:
      - configMap:
          items:
          - key: template.yaml
          name: alertmanager-config-template

The template referenced:
apiVersion: v1                                                                                [0/71]
data:
  template.yaml: |
    global:
      resolve_timeout: 5m
      slack_api_url: >-
        {{ .slackApiUrl | toString }}
...
    receivers:
      - name: "null"
      - name: slack
        slack_configs:
          - color: '{{ template "slack.color" . }}'
            title: '{{ template "slack.title" . }}'
            text: '{{ template "slack.text" . }}'
...

Give us examples of the outcome
could not apply template: could not execute template: unable to execute template at key template.yaml: unable to execute template at key template.yaml: template: template.yaml:26:28: executing "template.yaml" at <{{template "slack.color" .}}>: template "slack.color" not defined

I'd expect it to only template the slackApiUrl that's set in the external secret and not all fields.

Hi @adinhodovic ! Did it work? :smile:

Hey @gusfcarvalho, I think so - at least the inline templated version (I've been having issues with templateFrom - I think the templateFrom fix was not in 4.2.2). Will try v0.5.0 with the templateFrom fix.

Hi @adinhodovic ! I hope things went well when testing it with 0.5.0! I'm closing this issue now. Feel free to reopen it, though!


Get more info about connection failing with Kodi/XBMC

I can access my Kodi station running on Raspberry using Sybu Kodi remote controller but not with Kore or Yatse. Is there a way to get more log info about why the connection fails within the Android Smartphone Application ?

Only if you're willing to build a debug version, and then look at the logcat output...
I wouldn't go that far. That is usually a connectivity issue, so i would make sure that you have the necessary ports correctly specified and open (80 or 8080 and 9090) on your network.

@cmoulliard are you still unable to connect with Kore? If you need help compiling a debug version just let me know.


Refined tracks

This PR adds an option to save formant tracks rather than formant points when using the refined formant tracks.

this will fix #218

Hmm, there seems to be some sort of issue calling analyze_segments within Celery. It worked previously using formants.py from the SPADE repo but doesn't seem to work with ISCAN. Have you encountered this before @mmcauliffe  ?
ps-worker           | [2019-06-12 17:51:56,454: WARNING/ForkPoolWorker-2] output_columns:
ps-worker           | [2019-06-12 17:51:56,454: WARNING/ForkPoolWorker-2] ['F1', 'F2', 'F3', 'B1', 'B2', 'B3', 'A1', 'A2', 'A3', 'Ax', 'A1A2diff', 'A2A3diff']
ps-worker           | [2019-06-12 17:51:56,456: WARNING/ForkPoolWorker-2] michael ae: 1 of 8: 4 tokens
ps-worker           | [2019-06-12 17:51:56,476: WARNING/ForkPoolWorker-2] Traceback (most recent call last):
ps-worker           |   File "/site/proj/ISCAN/iscan/models.py", line 1011, in run_enrichment
ps-worker           |     output_tracks=output_tracks
ps-worker           |   File "/site/proj/PolyglotDB/polyglotdb/acoustics/formants/refined.py", line 119, in analyze_formant_points_refinement
ps-worker           |     multiprocessing=multiprocessing)  # Analyze the phone
ps-worker           |   File "/site/env/lib/python3.6/site-packages/conch/main.py", line 200, in analyze_segments
ps-worker           |     call_back, stop_check)
ps-worker           |   File "/site/env/lib/python3.6/site-packages/conch/multiprocessing.py", line 161, in generate_cache
ps-worker           |     manager = Manager()
ps-worker           |   File "/usr/lib/python3.6/multiprocessing/context.py", line 56, in Manager
ps-worker           |     m.start()
ps-worker           |   File "/usr/lib/python3.6/multiprocessing/managers.py", line 513, in start
ps-worker           |     self._process.start()
ps-worker           |   File "/usr/lib/python3.6/multiprocessing/process.py", line 103, in start
ps-worker           |     'daemonic processes are not allowed to have children'
ps-worker           | AssertionError: daemonic processes are not allowed to have children
ps-worker           | [2019-06-12 17:51:56,489: INFO/ForkPoolWorker-2] Task iscan.tasks.run_enrichment_task[e890462b-fe77-4038-9c17-11589aeb36d8] succeeded in 4.214987319999636s: None
ps-app              | [12/Jun/2019 17:52:03

nvm, fixed it in PolyglotDB

Excellent! @mmcauliffe ?


Fix building with GCC 10

GCC 10 defaults to -fno-common which requires us to explicitly mark global variables with extern in header files.

Any feedback here, please?

At this point in time we are not looking at GCC10 due to the fact that it is still in preview. Thank you for bringing this to our attention and we'll address it once it is released. Be sure to follow #80 for GCC10 related issues.

Fixing that will not cause any issues on building with older versions of the gcc.


Allowing UIButton bckground image overlap.

Find UIButton's private backgroundImageView and add it into self.viewsAllowingOverlap. Solving testing failure: Bottom right corner of <UIImageView: 0x7fb8c3d9dba0; frame = (0 0; 44 44); clipsToBounds = YES; opaque = NO; userInteractionEnabled = NO; layer = <CALayer: 0x7fb8c3d9cd80>> overlaps upper left corner of <UIButtonLabel: 0x7fb8c61521c0; frame = (4.33333 11.3333; 35.6667 21.6667); text = 'Button'; opaque = NO; userInteractionEnabled = NO; layer = <_UILabelLayer: 0x7fb8c61555a0>>. If this is intentional, you should add one of the views to viewsAllowingOverlap.

There's actually a built in method for doing this: viewClassesAllowingSubviewErrors in the config. If we add UIButton there, it should fix this problem.

Working on a fix now.

Actually, would like to use your test code.  Let me get a PR ready first for you to take a look at.

https://github.com/linkedin/LayoutTest-iOS/pull/16

Merged the other pull request

@Candyroot Any update? Wanted to give you credit for the tests, but I can copy over the code instead if you'd prefer.

In the meantime, if you point your pod to HEAD, you should get the fix you're looking for. Once I merge the tests, I'll release a new version.

Fixed in #18


Webapp:Setting connection string from file

Is your feature request related to a problem? Please describe.
I want to be able to sync settings between slots. On of the part is to set connection strings from file as it is possible with app settings ('az webapp config appsettings set')
Describe the solution you'd like
I want to be able to run this command:
az webapp config connection-string set --name hadar --resource-group TheCodeManual --settings<EMAIL_ADDRESS>where con.json is a file like:
[
  {
    "name": "Deault",
    "slotSetting": false,
    "type": "SQLAzure",
    "value": "<RealConnectionString>"
  }
]

Describe alternatives you've considered
I can parse file in powershell and set connection string one by one.
Additional context
I wanted to check if this is possible even if it is not clearly stated in documentation (like it is for instance for --slot-settings

However, command stuck for more than 15 minutes and I had to break it. I checked portal bot all values remained as they were.

routing to appropriate team

I was trying to achieve the same result as @kmadof, i.e. I wanted to set the value of deployment slot setting for a connection string using: az webapp config connection-string set
However it  worked for me when I supplied the following slot settings parameter , like so:
az webapp config connection-string set --resource-group $(resourceGroup) --name $(appServiceName) --connection-string-type SQLAzure --settings web-db='$(stagingDBConnectionString)' --slot $(deploymentSlot) --slot-settings web-db=true

Looks to still be broken even though there is an example here:
https://learn.microsoft.com/en-us/azure/app-service/configure-common?tabs=cli#edit-connection-strings-in-bulk
az webapp config connection-string set --name app-my-app --settings<EMAIL_ADDRESS>

the following arguments are required: --connection-string-type/-t
Examples from AI knowledge base:
az webapp config connection-string set --resource-group MyResourceGroup --name MyUniqueApp --connection-string-type mysql --settings mysql1='Server=myServer;Database=myDB;Uid=myUser;Pwd=myPwd;'
Add a mysql connection string.
az webapp config connection-string set --connection-string-type MySql --ids {ids} --settings mysql1='Server=myServer;Database=myDB;Uid=myUser;Pwd=myPwd;'
Update a web app's connection strings. (autogenerated)
https://docs.microsoft.com/en-US/cli/azure/webapp/config/connection-string#az_webapp_config_connection_string_set
Read more about the command in reference docs

Is there an update on this as I can still see the issue?

reviewed. --connection-string-type should not be a required parameter. will follow up on this.

This is still an issue. Just ran the command with JSON file and it is prompting for
the following arguments are required: --connection-string-type/-t

Still an issue in the latest CLI.   Error:
the following arguments are required: --connection-string-type/-t
if providing the argument (which should be taken from the array in the file), the error is:
ValueError: not enough values to unpack (expected 2, got 1)

Is there really no fix for this yet? At least the docs should be updated.

Here is my workaround while this is getting fixed:
$ConnectionStringsJson = ConvertFrom-Json $ConnectionStrings
for($i = 0; $i -lt $ConnectionStringsJson.Count; $i++)
{
	$settingName = $ConnectionStringsJson[$i].name
	$settingValue = $ConnectionStringsJson[$i].value

	az webapp config connection-string set --resource-group "rgname" --name "appname" --connection-string-type $ConnectionStringsJson[$i].type --settings "$settingName=""$settingValue"""
}

I needed this long-winded way for ""$settingValue""" so the key vault reference string won't lose last bracket (@Microsoft.KeyVault(VaultName=...;SecretName=...) <=

resolved by https://github.com/Azure/azure-cli/pull/27195

Resolved by linked PR and now available in latest CLI version.

Resolved by linked PR and now available in latest CLI version.

Still not working. Command I run:
az webapp config connection-string set --resource-group $StepAzureResourceGroupName --name $StepAzureAppServiceName --settings<EMAIL_ADDRESS>Error I get: argument --resource-group/-g: expected one argument


These changes make the code more concise and easier to read.

The changes I made include:

Removed unnecessary single quotes around the keys and values.
Used a list instead of a square bracketed array for the files section.
Used a hyphen instead of curly braces for the files item.
Aligned the keys and values to make the code more readable.

Checklist:

[x ] I have read and followed the contribution guidelines.
[ x] I have read and followed the how to open a pull request guide.
[x ] My pull request targets the main branch of freeCodeCamp.
[x ] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

Dear FreeCodeCamp Team,
I am writing in response to the rejection of my pull request, which was labeled as spam. I wanted to clarify that the changes I made were intended to improve the quality and readability of the code, and were made in good faith.
In the pull request, I removed unnecessary single quotes around keys and values, used a list instead of a square bracketed array for the files section, used a hyphen instead of curly braces for the files item, and aligned the keys and values to make the code more readable. These changes were intended to make the code more efficient and easier to read.
I understand that there may have been some confusion regarding my changes, and I am more than happy to provide additional context or examples to help you understand why I made the changes I did. I also want to express my openness to feedback and my willingness to learn and grow as a developer.
Thank you for your time and consideration.
Sincerely,
MrShadow


Allow installation from non-root users

What kind of change does this PR introduce?
At this moment, it's impossible to install this extension under non-privileged user because "C" language is untrusted.
What is the current behavior?
create extension pg_graphql;

ERROR: permission denied for language c (SQLSTATE 42501)

What is the new behavior?
create extension pg_graphql;
CREATE EXTENSION

Additional context
Changes at pgx template
tcdi/pgx#1056

@vadim2404 sorry for the extreme delay getting this merged, I didn't realize you were with neon. It'll be in the next release
If you run into any issues with any supabase extensions please open issues or reach out to me directly and we'll be sure to prioritize them


Stricter date format constraints

This adds tests for the date format to ensure that the month day-length is permitted following the day restrictions, there is 2 tests for each month (one at last day and one negative at day after). February has two sets of tests for normal and leap years.

5.7. Restrictions
The grammar element date-mday represents the day number within the
current month.  The maximum value varies based on the month and year
as follows:
  Month Number  Month/Year           Maximum value of date-mday
  ------------  ----------           --------------------------
  01            January              31
  02            February, normal     28
  02            February, leap year  29
  03            March                31
  04            April                30
  05            May                  31
  06            June                 30
  07            July                 31
  08            August               31
  09            September            30
  10            October              31
  11            November             30
  12            December             31

Appendix C contains sample C code to determine if a year is a leap
year.

There contains a test for invalid months too.

Thank you! Think these all seem great. Sincere apologies for the delay.


Make the construction of the endpoint regex pattern stable.

It is possible to query the endpoints that your Django app provides.
from django.core import urlresolvers
urlresolvers.get_resolver()

One can then walk that tree to enumerate each of the regex patterns that went into constructing it.
The regex pattern that is constructed for an endpoint is not stable. For instance, in my project I have seen it construct each of the following:
^admin/^(?P<app_label>social_django|authtoken|queues|jobs|files|users|clusters|appps|auth|vdcs)/$
^admin/^(?P<app_label>social_django|auth|users|vdcs|appps|jobs|queues|authtoken|clusters|files)/$
^admin/^(?P<app_label>authtoken|appps|social_django|clusters|users|queues|jobs|vdcs|auth|files)/$
^admin/^(?P<app_label>authtoken|appps|social_django|clusters|users|queues|vdcs|jobs|auth|files)/$
This precludes the use of patterns in unit tests.
Make the construction of the patterns stable so that subsequent walks of the tree will be consistent.

Tracking ticket: https://code.djangoproject.com/ticket/28703#ticket

Seems okay, can you add a test?

By default, Python uses a random seed for its hashing algorithm. This seed is constant for the life of the execution, but differs between executions. This means that during a single execution (i.e., the life of a test) the hashing will be stable (but not predictable). It is between executions that the ordering will vary. As such, I don't think that one would be able to write a single test that detects variances in ordering.
https://docs.python.org/3/reference/datamodel.html#object.__hash__
The use of sorted() is to guarantee a consistent ordering. There is no need for the ordering to be sorted; just that it is consistent. One could just as well implement this using sorted(valid_app_labels, reverse=True). Or, any other method that would guarantee a consistent ordering.
Given that, the best I can think of would be to write a test that verifies the values end up in sorted order. But, that would be a weak test in that if it was found that reverse=True was better (perhaps it is faster) the test would break because it is testing the wrong property (sorted versus consistent).
What's your opinion? Am I overthinking it? :-)

We have other tests for uses of sorted() in Django. Most likely, a random ordering won't be alphabetical. The test may not act as a regression test all of the time but should act as one most of the time.

I've added a test. The location of the test is somewhat arbitrary. I'm happy to move it elsewhere if you think there's a more appropriate location for it.
Also, I'm pretty new at this, so I'm very happy for any feedback on improving the test.

I'd put the test in tests/admin_views/test_adminsite.py.

It's not a use-case I ever envisaged, but it looks good to me (bar any qualms about where the test resides) and the cost isn't exactly high (unless you're querying the ModelAdmin.urls property manually on purpose, the sorted() is essentially run once at include() time, right?)

I have been considering reverting this change. Tim made a good point that for our use case we could filter away the non-stable regex, so it looks like there may not be any use cases where this is needed. If it is not needed, I'd prefer to not implement it. What is your opinion?

If there's no need for it (and yours was the first I'm aware of) then I'm not tied to seeing it land just for the sake of it :)

Retracting this, as it is not absolutely needed.


Is osmnx being used for testing?

https://github.com/pyansys/pyaedt/blob/017e4c582a25842a991a0421d2c535f5248f3b89/requirements/requirements_test.txt#L8

@akaszynski Yes it is. For instance we are using it coupled with SBR+ to simulate signal in streets.
We have some tests for those features in our testing suites.
osmnx is part of our extra-requirement too and because people might not have installed those one on their machine to perform tests.
We thought about adding it in the requirements_test to make sure it is not disrupting local testing. Same goes for the documentation.


vimPlugins.markdown-preview-nvim: fix plugin dependencies

Description of changes
Reverting package.json changes to the previous version, because it breaks the plugin.
Current version of package.json installs incorrect dependencies, the plugin expects app dependencies instead of root.
node:internal/modules/cjs/loader:1078
  throw err;
  ^
Error: Cannot find module 'tslib'
Require stack:
- /nix/store/rg7nwn36rm5xc15x48y2baidlqy3w857-vimplugin-markdown-preview.nvim-2022-05-13/app/lib/app/index.js
- /nix/store/rg7nwn36rm5xc15x48y2baidlqy3w857-vimplugin-markdown-preview.nvim-2022-05-13/app/index.js
    at Module._resolveFilename (node:internal/modules/cjs/loader:1075:15)
    at Module._load (node:internal/modules/cjs/loader:920:27)
    at Module.require (node:internal/modules/cjs/loader:1141:19)
    at require (node:internal/modules/cjs/helpers:110:18)
    at Object.<anonymous> (/nix/store/rg7nwn36rm5xc15x48y2baidlqy3w857-vimplugin-markdown-preview.nvim-2022-05-13/app/lib/app/index.js:3:17)
    at Module._compile (node:internal/modules/cjs/loader:1254:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1308:10)
    at Module.load (node:internal/modules/cjs/loader:1117:32)
    at Module._load (node:internal/modules/cjs/loader:958:12)
    at Module.require (node:internal/modules/cjs/loader:1141:19) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    '/nix/store/rg7nwn36rm5xc15x48y2baidlqy3w857-vimplugin-markdown-preview.nvim-2022-05-13/app/lib/app/index.js',
    '/nix/store/rg7nwn36rm5xc15x48y2baidlqy3w857-vimplugin-markdown-preview.nvim-2022-05-13/app/index.js'
  ]
}
Node.js v18.16.0

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Current version of package.json installs incorrect dependencies, the plugin expects app dependencies instead of root.

That's right.
The change seems to be fine.

Result of nixpkgs-review pr 229258 run on x86_64-linux 1

1 package built:

vimPlugins.markdown-preview-nvim

can we add a test to prevent this kind of regression ?

can we add a test to prevent this kind of regression ?

@teto 👌

rather than testing implementation details, wouldn't it be possible to test that the command in neovim works ? like you start neovim and run a :MarkdownPreview command and check that it doesn't raise any issue ? It would look more robust IMO

I've just tested it, and I'm happy to report that markdown-preview-nvim works perfectly on NixOS! I'll keep this open for now because of the suggestion above.


Template Haskell hangs

:ext TemplateHaskell
:ext OverloadedStrings

import Frames
import Frames.CSV

declareColumn "day" ''Double
:i day

This code hangs in both the console, the ihaskell jupyter notebook, and ob-ipython. Going line by line in the console, declareColumn "day" ''Double is the culprit.

I tried this and you're right that it crashes the kernel, but the stderr output suggested enabling DataKinds and then FlexibleContexts, and it seems to work after that:
:ext TemplateHaskell
:ext OverloadedStrings
:ext DataKinds
:ext FlexibleContexts

import Frames
import Frames.CSV

declareColumn "day" ''Double
:i day


help wanted: add solution to challenge 30

find the challenges directory, open it and search for the challenge 30
try to figure out the best way to answer it in whatever language you're most comfortable with
after that, add your solution to the solutions directory
don't forget to link your PR to this issue

Hi! I'd like to take this up

Thank you for your interest, let me review your PR and see if we can have it merged


web test pipelines are taking up to 1 hour

The API test pipelines are finishing quickly (within 15 to 20 minutes). But some web test pipelines are taking up to 1 hour to finish.
The web smokeTest pipeline is taking about 1 hour. That is too long.

split the full web tests into more parts (that will reduce the elapsed time of each pipeline)
look at how many web smokeTest scenarios there are. Either split that pipeline into 2 or 3 parts, or reduce the number of smokeTest scenarios.

split tests into more parts done in PR https://github.com/owncloud/ocis/pull/2353

Note: the "split into parts" thing is not smart at all. It counts the number of test suites that exist and splits them between pipelines without any analysis of their content. It is supposed to be a "reasonable rough way" to make a set of pipelines that finish in a similar time.
The problem is that not every test suite has a similar number/length of scenarios. And in the case of testing with OCIS, a lot of sharing tests are skipped - all the scenarios that test with "sharing to the root folder" are not applicable to OCIS. That results in silly outcomes like:
https://drone.owncloud.com/owncloud/ocis/6227/54/6
+ ./tests/acceptance/run.sh
run.sh: running acceptance-tests-drone
waiting for backend server to start
yarn run v1.22.11
$ cucumber-js --retry 1 --require-module @babel/register --require-module @babel/polyfill --require tests/acceptance/setup.js --require tests/acceptance/stepDefinitions --format node_modules/cucumber-pretty /srv/app/web/tests/acceptance/features/webUISharingInternalUsersToRoot /srv/app/web/tests/acceptance/features/webUISharingInternalUsersToRootBlacklisted /srv/app/web/tests/acceptance/features/webUISharingInternalUsersToRootCollaborator -t 'not @skipOnOCIS and not @skip and not @notToImplementOnOCIS and not @federated-server-needed'
0 scenarios
0 steps
0m00.000s
Done in 4.46s.
Checking expected failures in /drone/src/tests/acceptance/expected-failures-webUI-on-OCIS-storage.md
runsh: There were no unexpected failures.
runsh: There were no unexpected success.

There were 3 test suites to run in that pipeline. But none of those test suites had any test scenarios relevant to OCIS. So that pipeline did nothing useful.
We could think of ways to improve the balancing of the run-time of each pipeline.

Even with my comment above, the test scenarios have got slower. We need to investigate why, and fix whatever waiting/wasted-time is happening.

In oCIS, we used to have similar drone config as web for UI tests. Maybe that might help.
But it would be better if we can equally split tests into pipelinnes

We can explicitly specify the web test suites to be run in each pipeline in oCIS, and organize those so that they almost equalize the run-time of each pipeline. The problem we had with that in the past is that when new test suites are added (or test suites are renamed...) then we have to remember to update the pipelines of web tests in oCIS. It is not such a big thing. Maybe it is worth going back to doing it that way, because the current 10 or 20 parts have big differences in run-time.
But whatever we do about spreading the test suites between pipelines, we also have to work out why the tests are now a lot slower!

The issue lies in a new implementation of appSideBar.js::activatePanel() in the web acceptance test suite. We had accordions in the past and now have a new sidebar UI/UX concept implemented. I'm digging into the timing issues right now. Will open a PR in web soon.

PR in web is here https://github.com/owncloud/web/pull/5635, but it looks like I tried to optimize too much. Definitely found the reason for the major CI time increases.

PR in web https://github.com/owncloud/web/pull/5635 is merged and brought down the CI time to 35 minutes. Should help in ocis as well.

The run-time of many web pipelines in oCIS is very different. It would be good to think of a way to make this smarter, and each pipeline have a more similar number of test scenarios.

We know that there are various web test suites that have zero scenarios for the oCIS case. Those are the "sharing to root" test suites.
We could enhance the "run parts" logic run.sh in owncloud/web so that the caller can specify a list of test suites that are not to be run. The script that gets the list of available test suites can filter out the "not to run" suites and then split the remaining suites into parts, like it does now.
That will remove the useless "zero scenario" suites from the oCIS test run and might be enough to balance out the pipelines.
Maintainenance of the list of "do not run" test suites will be low. When we add new test suites in web, they will automatically get run in oCIS.


Question about speed.

Hi
Saw you on youtube her: (great video)
https://www.youtube.com/watch?v=rg_GiOZ5Owk
Postgresql looks like a lot of FUN, for the smaller applications I want to make, sqlite will prb be enough.
PLv8 is that faster or slower then PLPGSQL?
I,m pretty sure Im going to prefer the PLV8, but nice to know if its is a lot slower, or faster than the options shipped with postgresql.

Hey sorry, I didn't get a notification about this! PLV8 executes the same, I think, as it would for Node etc. I haven't seen anything about speed vs. PLPGSQL.


Remove the depenency on warning as it's currently unused

There's some commented out code that does use it, but it's better to remove this for now.

Well, it's there but commented out. I'd worry about what happens as soon as someone uncomments it.
FWIW, warning is really tiny<EMAIL_ADDRESS>

oops, did this manually already. Thanks!


Replace . with decimal point for strtod

Fixes https://github.com/ruby/prism/issues/2638

Thanks, nice fix!


Tool Throws a Failure for Alternate MIL Off

A third party controller on one of our trucks is using "Alternate MIL Off" instead of the convention off, the tools isn't recognizing this as a valid MIL off:
12:18:27.1351 18FECA2F [8] 00 00 00 00 00 00 FF FF
DM1 from Suspension - System Controller #1 (47): MIL: alternate off, RSL: alternate off, AWL: alternate off, PL: alternate off, No DTCs
FAIL: <IP_ADDRESS>.c - Non-OBD ECU Suspension - System Controller #1 (47) did not report MIL off or not supported

Yes, please See #957 for prior observation.


Add description of the equation symbols

Adds tables with the descriptions of the symbols of the equations described in the docs.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Additional details and impacted files

:loudspeaker: Thoughts on this report? Let us know!


docs: prevent collision in operation names in IdentityHub management aPI

What this PR changes/adds
Prevent API operation name collision in merged OpenAPI file for Management API.
Why it does that
Some API Gateway such as Azure API Management does not support multiple API operations having the same name in the same openapi file.
Further notes
List other areas of code that have changed but are not necessarily linked to the main feature. This could be method
signature changes, package declarations, bugs that were encountered and were fixed inline, etc.
Linked Issue(s)
Closes # <-- insert Issue number if one exists
Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.

I am OK with renaming method names, but I think the better approach here is to explicitly set the operationId on OpenApi annotations like so:
    @Tag(name = "KeyPairResources Management API")
    @Operation(description = "Finds a KeyPairResource by ID.",
            operationId = "findKeyPairResourceById", <-- ADD THIS
            responses = {
                    // responses
            }
    )
    KeyPairResource findById(String id, SecurityContext securityContext);

That way we can decouple method names from the API spec and we prevent subtle bugs in the future, that no-one detects until the API is actually published using Azure API GW.
We should do both, the method renaming and the explicit declaration.

updated


Set drawable for submit button

[x] I have verified there are no duplicate active or recent bugs, questions, or requests
[x] I have verified that I am using the latest version of Easy Video Player.
[x] I have given my issue a non-generic title.
[x] I have read over the documentation (before asking questions on how to do something).

Include the following:

Easy Video Player version: 0.2.12
Device OS version: 6.0
Device Manufacturer: GenyMotion

Hi, I was hoping if you can make a quick implemention from you to your library, I wanted to be able to set drawable for the submit button, i truly need it.
thanks for you good library

I intend to keep it as text.


Support for external virtual keyboards and QWERTY input.

¡Great, that's working much better than my version!
I think it would be better for the keyboard button to be on the side of the screenshot button, so i would no need to press the 3-Dots button all time when i would use it, that's why i put a shortchut in there
¿Will it be merged on original J2ME Loader aswell?

Doesn't a shortcut appear for you?

Will it be merged on original J2ME Loader aswell?

I don't know - Nikita manages J2ME Loader.

Doesn't a shortcut appear for you?

No

I don't know - Nikita manages J2ME Loader.

¿Can i commit your code in that pull request?

@JoseskVolpe Yes, I will merge it.

yay merged ^w^


Make dev build faster

rework webpack configs to on do whats needed for the dev build

webpack config refactor made this quick


Refactor version update process

Codecov Report

Merging #65 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #65   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           9      9
  Lines         270    270
=====================================
  Hits          270    270

Coverage remained the same at 100.0% when pulling fcec2c90b949cf7a011f52b07b4d13aa782145cd on refactor-update-version into 7028cac07596d0ed44176b85b43b73fa68866214 on master.

Coverage remained the same at 100.0% when pulling fcec2c90b949cf7a011f52b07b4d13aa782145cd on refactor-update-version into 7028cac07596d0ed44176b85b43b73fa68866214 on master.

Coverage remained the same at 100.0% when pulling fcec2c90b949cf7a011f52b07b4d13aa782145cd on refactor-update-version into 7028cac07596d0ed44176b85b43b73fa68866214 on master.


docs(schema.ex): update incorrect variable names

Description
Fixes incorrect variable names in the documentation from date to value.
Checklist

[x] I added tests that cover my proposed changes.
[x] I updated the documentation related to my changes (if applicable).

Thanks!


Added more data to the table view.

Added call sign, speed, vertical speed, and number of messages to the
table view.
Instead of being a map from ICAO -> AirplaneCoor, Airplanes is now a
map from ICAO -> state vector, where the state vector (actually an
AirplaneState struct) holds coordinates, squawk code, call sign,
speed, vertical speed, etc.
The rows are starting to get a bit long; Eventually maybe there could
be a detail view with more data (autopilot, position accuracy, etc.).

Awesome, what an improvement.


unknown option: channel-commit-interval

Background
Trying to run lnd with this option in lnd.conf
Your environment
lnd v0.12.1-beta
Ubuntu 20.04
Bitcoin Core 0.21.0
Steps to reproduce
Create lnd.conf with the following:
[Application Options]
channel-commit-interval=200ms
and run lnd
Expected behaviour
lnd should start up successfully
Actual behaviour
Error output: "unknown option: channel-commit-interval"

It looks like perhaps this is no longer a real option, but it has vestigal code in config.go and still exists in the sample lnd.conf

Aha, it's the opposite and I shouldn't have been looking at the sample conf in master


cm-chessboard: Figure out why some piece sets have cache: false

In the demo “Example: Different styles and piece sets, consistently the piece set chessboard-sprite-staunty.svg has no cache setting, but the other piece set specifies cache: false:
sprite: {url: "../assets/images/chessboard-sprite.svg", cache: false}

I should figure out why. (Ask the developer.) I may learn something about (a) what I should do and (b) how this might affect alternative piece sets.

Closed as out of scope for this repository.


Number of threads used by spateo

Congratulations for the great tool.
I am running a script within a pipeline to perform cell segmentation of ST datasets.
Running the very same pipeline on different machines, I get consistent results but ten fold different running times.
Upon inspection, I found that on one machine (ubuntu 22.04) spateo runs using all the available threads (24).
On other machines, (an HPC cluster with a CentOS installation, a 24.04 Ubuntu workstation) spateo runs using only a single thread.
In all cases I am running spateo 1.0.2 in a conda environment with python 3.10.16, scipy 1.12.0, matplotlib 3.7.1 and a few other packages that should not be relevant (scanpy louvain, pyvista) in the choice of the number of threads.
How can I get spateo to run on parallel threads on all the machines?
Is there a configuration option I am missing to explicitly set the n_threads parameter?
Thanks for your kind help,
Valerio

I switched to release 1.1.0 of spateo, and the issue is not reproducible with the latest release.
Since this is not an issue in the current release, from my side, this is fixed.
I apologize for the inconvenient, it would have been wiser from my side to check the latest release before opening an issue.
Best,
Valerio


Do not emit errors if 'browser.quit' fails

/cc @sipayRT @j0tunn

Coverage decreased (-14.06%) to 66.083% when pulling 1885b4163e2a402c8b3730707cda9eccf2b21ec7 on fix/free-browser into 499ffb3b9cd36a01a7c1392b3ae6db8aafaafebe on master.

Coverage decreased (-14.06%) to 66.083% when pulling d1f8e29e9e8fc84dd757e455c8fadc654b1b615b on fix/free-browser into 499ffb3b9cd36a01a7c1392b3ae6db8aafaafebe on master.

Coverage decreased (-14.06%) to 66.083% when pulling 99abc6dbf328e721548560f824bb7d3b2174a122 on fix/free-browser into 499ffb3b9cd36a01a7c1392b3ae6db8aafaafebe on master.

Coverage increased (+0.02%) to 80.169% when pulling 2d0eae5501f4eab046c737f4cfaecec4df480489 on fix/free-browser into 499ffb3b9cd36a01a7c1392b3ae6db8aafaafebe on master.

Coverage increased (+0.02%) to 80.169% when pulling 7cc07fcf43478af362a7ab07cc4e28867128144c on fix/free-browser into ee66eceba84f4045e255566e7d0c0fbb90478ebb on master.

Проверил интеграционно – работает.

🆗

/ok

Coverage decreased (-0.02%) to 80.127% when pulling aa28848da3a35345e551d50ea108e24827efae80 on fix/free-browser into ee66eceba84f4045e255566e7d0c0fbb90478ebb on master.

/merge

Coverage decreased (-0.02%) to 80.127% when pulling 07394d8ab0a35dfa4d081ab92074f5ca9c85420a on fix/free-browser into 3b876bc76e8b1f1ee9a612cf70a9f822b70a7bc3 on master.


[bug] SignedTransaction and SignedTransactionCandidate should not implement Decode / Deserialize in no_std

Originally posted by @mversic here
How to test
Honestly I don't think it's worth spending time on tests like this.
Before changes
Currently it's possible to create a Rust package which will link to iroha_data_model with default-features = false enabled. That will disable std feature. After that you should be able to decode / deserialize SignedTransaction with wrong signatures, because they won't be checked at no_std
After changes
You should get a compile error if you try to decode / deserialize SignedTransaction from no_std environment.

This is not possible to implement as-is because it breaks the validator (see https://github.com/hyperledger/iroha/pull/3686#issuecomment-1628935427, TLDR: validator decodes SignedTransaction for ffi)
Even if it was implemented, it's additionally harder to make it a compile-time error, as the versioned! macro used on the SignedTransaction requires Deserialize and Decode, reducing the utility.
The only really actionable item is documenting it clearly I guess


How to cite this project?

I would like to cite this project in an upcoming publication. So far I have put together this bibtex entry:
@misc{G4HepEm,
	author = {Mihaly Novak and Jonas Hahnfeld and Ben Morgan and Stephan Hageböck},
	title = {The G4HepEm R\&D Project},
	year = {2022},
	publisher = {GitHub},
	howpublished = {\url{https://github.com/mnovak42/g4hepem}}
}

Could you please tell me if this is appropriate for you?
In the long run, it may be beneficial to add a CITATION.cff file to your repository, so people can find this information faster.
Thank you!

It's fine at the moment.
We are working on a paper describing the project that will be the main reference later.

OK great, thx! Good luck with your paper!


Provide capability to subscribe for standup

Provide capability to subscribe for standup meetings via Google Calendar.
This allows folks to just pull in the agenda in their calendar client and see if there is a meeting or not, for example when it's cancelled due to Thanksgiving.
Relates to https://github.com/kedacore/keda/issues/472
Relates to https://github.com/kedacore/keda/issues/502

LGTM


The vagrant feature “Synced folders” is not working on RancherOS… missed rsync?

Is that all that is needed?  Just SSH+rsync?   We don't have rsync today, but we could add it.

yes :) I think based the below error that only "rsync" should be add into rancherOS
==> default: Machine booted and ready!
"rsync" was not detected as installed in your guest machine. This
is required for rsync synced folders to work. In addition to this,
Vagrant doesn't know how to automatically install rsync for your
machine, so you must do this manually.

No problem.  We should do a smaller release within the next release and we'll include rsync.

@ricardsonwilliams v0.1.2 was just release and it now includes rsync.  Could you please try and see if the vagrant synced folders works now?
ping @cloudnautique

@ibuildthecloud did you update the Vagrant Box Image? ( http://cdn.rancher.io/vagrant/x86_64/prod/rancheros_virtualbox.box )

We did update. Need to do the merge, but url is "http://cdn.rancher.io/vagrant/x86_64/prod/rancheros_v0.1.2_virtualbox.box"

Good! I update mine Vagrant's file but the rsync option still show a error:
Error message:
==> default: Rsyncing folder: /Users/ricardsonwilliams/GitHub/Vagrant2/rancherOS/ => /vagrant
There was an error when attempting to rsync a synced folder.
Please inspect the error message below for more info.
Host path: /Users/XXXX/GitHub/Vagrant2/rancherOS/
Guest path: /vagrant
Command: rsync --verbose --archive --delete -z --copy-links --no-owner --no-group --rsync-path sudo rsync -e ssh -p 2222 -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null -i '/Users/XXXXX/GitHub/Vagrant2/rancherOS/.vagrant/machines/default/virtualbox/private_key' --exclude .vagrant/ /Users/XXXXXX/GitHub/Vagrant2/rancherOS/ rancher@<IP_ADDRESS>:/vagrant
Error: Warning: Permanently added '[<IP_ADDRESS>]:2222' (RSA) to the list of known hosts.
rsync: This rsync lacks old-style --compress due to its external zlib.  Try -zz.
rsync error: syntax or usage error (code 1) at main.c(1554) [server=3.1.1]
rsync: connection unexpectedly closed (0 bytes received so far) [sender]
rsync error: error in rsync protocol data stream (code 12) at /SourceCache/rsync/rsync-45/rsync/io.c(453) [sender=2.6.9]

I try to disable the option "vb.functional_vboxsf     = false"
==> default: Mounting shared folders...
default: /vagrant => /Users/XXXXX/GitHub/Vagrant2/rancherOS
Failed to mount folders in Linux guest. This is usually because
the "vboxsf" file system is not available. Please verify that
the guest additions are properly installed in the guest and
can work properly. The command attempted was:
mount -t vboxsf -o uid=id -u rancher,gid=getent group rancher | cut -d: -f3 vagrant /vagrant
mount -t vboxsf -o uid=id -u rancher,gid=id -g rancher vagrant /vagrant
The error output from the last command was:
mount: mounting vagrant on /vagrant failed: No such device

We definitely don't have the guest additions yet.  We are working on that one.

@ricardsonwilliams The latest os-vagrant/Vagrantfile has the vboxsf disabled. It ends up looking like
config.vm.provider "virtualbox" do |vb|
     vb.check_guest_additions = false
     vb.functional_vboxsf     = false
     vb.memory = "1024"
  end

We should have something up soon that will setup an rsync folder to /opt/rancher from the current directory.

@ricardsonwilliams rancherio/os-vagrant Vagrantfile now has configuration to enable an rsync folder. It looks like the error we are seeing with the --compress flag, at least with OS X Yosemite, is that the version of rsync is really old (version 2.6.9). The RancherOS version is 3.1.1 which is the latest release.

@cloudnautique Thank you! Works pretty well. Nice you already update the os-vagrant readme file.

I've got the following error on Windows 7. I installed cwRsync free version. If I type in the command prompt: "rsync". I've got it run:
$ rsync
rsync  version 3.1.1  protocol version 31
Copyright (C) 1996-2014 by Andrew Tridgell, Wayne Davison, and others.
Web site: http://rsync.samba.org/
Capabilities:
    64-bit files, 64-bit inums, 32-bit timestamps, 64-bit long ints,
    socketpairs, hardlinks, symlinks, IPv6, batchfiles, inplace,
    append, ACLs, no xattrs, iconv, symtimes, prealloc

$ vagrant up
Bringing machine 'rancher-01' up with 'virtualbox' provider...
==> rancher-01: Checking if box 'rancherio/rancheros' is up to date...
==> rancher-01: Clearing any previously set forwarded ports...
==> rancher-01: Clearing any previously set network interfaces...
==> rancher-01: Preparing network interfaces based on configuration...
    rancher-01: Adapter 1: nat
    rancher-01: Adapter 2: hostonly
==> rancher-01: Forwarding ports...
    rancher-01: 22 => 2222 (adapter 1)
==> rancher-01: Running 'pre-boot' VM customizations...
==> rancher-01: Booting VM...
==> rancher-01: Waiting for machine to boot. This may take a few minutes...
    rancher-01: SSH address: <IP_ADDRESS>:2222
    rancher-01: SSH username: rancher
    rancher-01: SSH auth method: private key
    rancher-01: Warning: Connection timeout. Retrying...
    rancher-01: Warning: Remote connection disconnect. Retrying...
==> rancher-01: Machine booted and ready!
==> rancher-01: Configuring and enabling network interfaces...
==> rancher-01: Rsyncing folder: /d/VM/rancheros/ => /opt/rancher
==> rancher-01:   - Exclude: [".vagrant/", ".git/"]
There was an error when attempting to rsync a synced folder.
Please inspect the error message below for more info.

Host path: /d/VM/rancheros/
Guest path: /opt/rancher
Command: rsync --verbose --archive --delete --copy-links --chmod=ugo=rwX --no-perms --no-owner --no-group --rsync-path sudo rsync -e ssh -p 2222 -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null -i 'd:/VM/rancheros/.vagrant/machines/rancher-01/virtualbox/private_key' --exclude .vagrant/ --exclude .git/ /d/VM/rancheros/ rancher@<IP_ADDRESS>:/opt/rancher
Error: Warning: Permanently added '[<IP_ADDRESS>]:2222' (ECDSA) to the list of known hosts.
rsync: connection unexpectedly closed (0 bytes received so far) [Receiver]
rsync error: error in rsync protocol data stream (code 12) at io.c(226) [Receiver=3.1.1]
rsync: connection unexpectedly closed (0 bytes received so far) [sender]
rsync error: error in rsync protocol data stream (code 12) at io.c(235) [sender=3.1.1]

Can we use smb to sync folders between Windows and RancherOS under Vagrant?

awesome! Is there any integration for docker-machine instead of vagrant?

@gregory Are you looking to launch RancherOS with docker-machine? If so, this is already supported and documented: http://docs.rancher.com/os/running-rancheros/workstation/docker-machine/
Or are you looking for something else?

yes, that's actually what i used, but unfortunately it doesn't mount the volumes on OSX.
You can reproduce this easily: docker run -it --rm -v $(pwd):/code busybox ls /code


Kivy - Complex shapes with vector or matrix

Versions

Python: 3.6.1
OS: Ubuntu 16.4
Kivy: 1.10.0
Kivy installation method: pip

Description
I'm trying to create with vector the figure below in Kivy, however I'm not able to create it. I only find instructions for basic shapes.

Could someone help me with this problem or clarify to me if it is possible to create more complex shapes in Kivy?
Thank you

I'm not really sure what are you asking. For drawing such a shape you can use e.g. line. If it's about vector graphics, then there's SVG you have to create beforehand.

Thank you for clarifying me. As I learn more about Kivy some concepts are becoming clearer.


Timelock support uniswap

Security Checklist

[ ] 1. Re-Entrancy
[ ] 2. Arithmetic Over/Under Flows
[ ] 3. Unexpected Ether
[ ] 4. Delegatecall
[ ] 5. Default Visibilities
[ ] 6. Entropy Illusion
[ ] 7. External Contract Referencing
[ ] 8. Short Address/Parameter Attack (off chain)
[ ] 9. Unchecked CALL Return Values
[ ] 10. Race Conditions / Front Running
[ ] 11. Denial Of Service (DOS)
[ ] 12. Block Timestamp Manipulation
[ ] 13. Constructors with Care
[ ] 14. Uninitialized Storage Pointers
[ ] 15. Floating Points and Precision
[ ] 16. Tx.Origin Authentication
[ ] 17. Address.isContract Re-Entrancy via Constructor

⚠ NOTES ⚠
Make sure to think about each of these exploits in this PR.

Merged in https://github.com/para-space/paraspace-core/pull/402


Add LGPU with single GPU backend

Before submitting
Please complete the following checklist when submitting a PR:

[x] All new features must include a unit test.
If you've fixed a bug or added code that should be tested, add a test to the
tests directory!

[ ] All new functions and code must be clearly commented and documented.
If you do make documentation changes, make sure that the docs build and
render correctly by running make docs.

[x] Ensure that the test suite passes, by running make test.

[x] Add a new entry to the .github/CHANGELOG.md file, summarizing the
change, and including a link back to the PR.

[x] Ensure that code is properly formatted by running make format.

When all the above are checked, delete everything above the dashed
line and fill in the pull request template.

Context:
Add Ligtning-GPU to Pennylane-Lightning repo
Description of the Change:
Benefits:
Possible Drawbacks:
Related GitHub Issues:

Thanks @mlxd and @vincentmr for reviewing this 15k PR!


05.02.01 - Allow follows

Focus
Habit Following and Sharing
Partial US
As a doer, I want to grant or deny another doer permission to follow all my public habits.
Reason
Doers should control who can and cannot follow them.
Story Points
5
Risk Level
Medium

Finished as of #137


refractor refractor DataTypeArray from_string

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.

refractor DataTypeArray from_string, make it more clear;
support ',' and ']' inside string element, for example: ['hello,,,', 'world][]']

Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[x] No
[ ] I don't know

Has unit tests been added:

[x] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[x] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[x] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 38.63 seconds
load time: 573 seconds
storage size:<PHONE_NUMBER>1 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221102125658_clickbench_pr_38523.html


Start a project

Signed-off-by: Meg McRoberts<EMAIL_ADDRESS>Simple instructions to start a project using either the Keptn Bridge or the command line.

specified to that command >> required for that command perhaps?
This shipyard would break the webhook screen in the bridge and cause an invalid shipyard warning because it doesn't contain sequences and corresponding tasks blocks for each stage.
It's fine to show it like this as long as we immediately link to some information telling users that after initial project creation, they must then decide on sequences and tasks. I like that flow because it reinforces that stages are static but sequences and tasks can be changed at will.
apiVersion: spec.keptn.sh/0.2.3
kind: "Shipyard"
metadata:
  name: "shipyard-sockshop"
spec:
  stages:
    - name: "dev"
    - name: "hardening"
    - name: "production"

specified to that command >> required for that command perhaps?

done

added rules for project name; put the configuration parameter in the shipyard reference page -- only need that in one place and it isn't when they are learning how to create a project.
Not sure what to do about the config parameter for service name max length -- I don't think a service name is specified in shipyard and I don't have a reference page for services -- probably need one.

Added link to "Define Keptn Projects" for information about how to code sequences and tasks.  Right now, that section is just a dumping ground but hopefully I will improve the structure very soon.


Asset Lists v1

@jonator

Think we could place this in a FigJam or Figma instead? We could then jam on the design a bit more efficiently there.


Support configurable tunnel port for TrafficEncapMode

This PR supports configurable tunnel port for TrafficEncapMode. When
deployed on the public cloud, the tunnel type and tunnel port used by
the underlying network may be the same as the antrea container network,
so it needs to support the modification of the tunnel port to prevent
conflicts.
Signed-off-by: Wu zhengdong<EMAIL_ADDRESS>

/test-all


Add Swift 4.1 / Xcode 9.3 support

Do you plan add support Swift 4.1/Xcode 9.3 ?

Hey @kiokumicu , sooner or later. Currently swamped with work.

Support was added

Fixed


Objects

completed code challenge 06. test failed to run for challenge 03 and 04.

None of your challenges passed. Please complete and resubmit for additional credit. Thanks!

Correction --- 4 of your challenges passed. Good Job!


is it possible to render Panda3D context within Hello Imgui?

Hello,
First of all, great job for the bundle. It is amazing.
I was wondering if it is possible to render Panda3D context within Hello ImGui, Kind of like matplotlib figures. I have been trying to achieve it but I was not lucky. I am newbie with Panda3D.
If it is possible, any hint?
Thanks In advance.

Hello,
First of all, great job for the bundle. It is amazing.

Thanks a lot, it is always nice to receive positive feedback!

I was wondering if it is possible to render Panda3D context within Hello ImGui, Kind of like matplotlib figures. I have been trying to achieve it but I was not lucky. I am newbie with Panda3D.
If it is possible, any hint?

I am not familiar at all with Pandas3D. Based on what I've quickly found, It is a full engine for realtime 3D game.
Given this, there are two possible directions, which depends on a choice you have to make by answering this question: who is responsible for creating and handling the main app window, and handling events (mouse, keyboard, etc)?
If Dear ImGui Bundle is responsible for creating the main window
In that case, you would need to:

have a way to render Pandas3D to a frame buffer, and use this frame buffer as the background of your application.
have a way to transmit all needed events from ImGui to Pandas3D

Please note that in this case you would be using the standard rendering of ImGui Bundle (provided by HelloImGui).
It uses OpenGL3 and glfw and this cannot be changed.
As far a 1. (Render Pandas3D to a frame buffer) is concerned, here is see an example where the background is provided by an OpenGL shader:

Online demo
Python source code
C++ source code. This code is only if you are using C++. The Python code I provided before is enough in your case.

As far as 2. (transmit events to Pandas 3D) is concerned, you might interrogate imgui.get_io() to get info about the events.
If Pandas3D is responsible for creating the main window
In that case

You cannot use the app handling part of ImGui Bundle (i.e "immapp.run", or "hello_imgui.run")

you will need to integrate ImGui inside pandas3D: this means that you will need to:

manually render ImGui's "DrawData".
For a C++ example you can look at panda3d_imgui and especially this part
https://github.com/bluekyu/panda3d_imgui/blob/e42a14c62b4ee7c59871f73fc41e2a4e45d6ae51/panda3d_imgui/panda3d_imgui.cpp#L358-L437

Transmit events from Pandas3D to ImGui when needed

You may take some inspiration from some existing pure python backends, available here

At a first glance, it is not a very easy endeavor.  Anyhow, good luck, and keep me informed if you make progress in either direction!

First of all, @pthom, thank you for your prompt response.
I am aiming to achieve the first option: rendering Panda3D within Hello_Imgui. You've done an outstanding job incorporating a variety of functionalities into the bundle, such as viewports, imPlot, and ImNodes, which has saved me a significant amount of work.
That being said, I am looking to integrate a graphics engine like RenderPipeline, which is based on Panda3D, into your bundle. While I am actively exploring ways to accomplish this, I haven’t yet found a straightforward method for integrating Panda3D. I’ll continue my efforts and will share any progress in this thread.
Additionally, I have a question: would it be possible to use ModernGL with your bundle?
Thanks in advance for your help!

would it be possible to use ModernGL with your bundle

I know that some people tried, but they did have to render from Python.
On my side I did not use ModernGL, So I don't know what is possible. Anyhow, I think that if you are rendering with modernGL to a texture, you could then use this texture in imgui bundle.

Keep me posted about your progress, I am interested!

In my case, I'd like to do kind of the opposite with ModernGL,
I'm already too deep with the glfw package and the moderngl_window's classes in my code, and would like to see an integration directly of imgui_bundle how's it currently done in mgl [1] [2], drawing on top of a fbo currently in use
I'll tackle on this eventually per https://github.com/BrokenSource/ShaderFlow/issues/6, as imgui is holding me on cp311. But the linked PRs and resources here smells good to me on it being possible, perhaps we/I can make the integration official!
I'm really looking forward to use your package for the future!

@Tremeschin
I originally worked and contributed to pyimgui (which you mentioned), before developing imgui bundle.  So the API of both are very similar, except that ImGui bundle bindings are auto-generated (and thus maintenable and maintained) documented, and include lots of other libraries.
Again, you can take inspiration from the pure python backends.

I'm really looking forward to use your package for the future!

Welcome to the community! I do like the motto of your project (" Imagine ShaderToy, on a Manim-like architecture"), and I'm interested in it. If possible, please keep me informed of the status here, or post updates in the gallery.

Welcome to the community! (...) and I'm interested in it (...) updates in the gallery

Thanks @pthom, happy to get into the community! These resources will also definitely help 🙂
It'll be first used as an internal GUI for controlling actions inside the scene, states, dev menus, switching scenes in real time. But it might expand to more advanced stuff in the future, like rendering options within, potential use for basic games or interactive demos. Whenever I have something worthy I'll surely add to the gallery
Humbly linking DepthFlow, given your interest on the parent project! Much more traction, and a actual final product. Will be the first to receive any GUI :)

A, someone beat me to it in PR https://github.com/moderngl/moderngl-window/pull/197 (unmerged), I had a local version which was rendering correctly just needed to proxy the events to your package, can confirm it was very easy to convert the code and worked pretty much flawlessly in my code base with moderngl 🙂

@Tremeschin : I tried https://github.com/moderngl/moderngl-window/pull/197, and it works fine. Thanks for the info


Simplify TimeoutSpec

Even though resources should be managed by processes, they aren't necessary for what we're proving here.  And that Strategy.sequential seems shady.  Hoping this fixes the intermittent timeouts I have introduced to master.

That's not the resource handling failing.  That's the timeout itself failing.  We're asserting that the 204 response got preempted by the timeout response.

I mean the error is quite long it looks like it started the delivery, and half-way through was disconnected, but returned a 204 instead of a 500 and the Blaze stage was very displeased with how it was handled.
If that is the correct behavior I think the test will need changes to match that we now expect a 204, but why would a timeout yield a 204 rather than a 500?

Hmm.  The blaze mess is a 304, not a 204.  I don't know what's doing a 304 in our tests.  I think that might be output from a different test running concurrently?
The "timeout" is returning a 204 because it isn't timing out, I think.  If the example service returned something other than a 204, with this bug, that's what we'd see instead of the 500.  The question is why the timeout isn't winning the race.
I think this started when I simplified the way the timeout response is generated, but maybe it's just the first time this test was on that branch.  It's all a bit of a blur that I haven't had time to sort out yet.

Locally:
[info] Timeout Middleware should
[info]   + have no effect if the response is not delayed (73 ms)
[info]   + return a 500 error if the result takes too long (73 ms)
[info]   + return the provided response if the result takes too long (73 ms)
[info]   + handle infinite durations (4 seconds, 56 ms)
[info]   + clean up resources of the loser (280 ms)

Travisally:
[info] Timeout Middleware should
[info]   + have no effect if the response is not delayed (3 seconds, 157 ms)
[info]   + return a 500 error if the result takes too long (3 seconds, 137 ms)
[info]   + return the provided response if the result takes too long (3 seconds, 103 ms)
[info]   + handle infinite durations (6 seconds, 993 ms)
[error]   x clean up resources of the loser (3 seconds, 153 ms)
[error]    the response status '204 No Content' is not equal to '500 Internal Server Error' (TimeoutSpec.scala:54)

Look at those time differences!

Wish I had my head around this a little better, but I think it largely boils down to Travis only has 2 cores and is more likely to block on access to disk or CPU.  Combine that with running a bunch of specs in parallel and having a million (give or take) pooled threads floating around and interacting with one another, and things get pretty bogged down.

Build failure was addressed by #1126 and looks like TimeoutSpec passed on 2.11 and 2.12 after the increase.  Worth running another build (or restarting the old one) to get a second data point?

If that solved it, that would be nice. Definitely worth investigating, but I know better than to restart another mans build.

We're all friends here.


[★𝐍𝐎𝐖★] 𝐊𝐮𝐥𝐡𝐚𝐝 𝐏𝐢𝐳𝐳𝐚 𝐂𝐨𝐮𝐩𝐥𝐞 Video 𝙻𝚎𝚊𝚔𝚎𝚍 Viral 𝘖𝘳𝘪𝘨𝘪𝘯𝘢𝘭 Clips LINK On Social Media X Ｔｗｉｔｔｅｒ

20 sec ago [★𝐍𝐎𝐖★] 𝐊𝐮𝐥𝐡𝐚𝐝 𝐏𝐢𝐳𝐳𝐚 𝐂𝐨𝐮𝐩𝐥𝐞 Video 𝙻𝚎𝚊𝚔𝚎𝚍 Viral 𝘖𝘳𝘪𝘨𝘪𝘯𝘢𝘭 Clips LINK On Social Media X Ｔｗｉｔｔｅｒ
..
..

..

..

..
..
𝐊𝐮𝐥𝐡𝐚𝐝 𝐏𝐢𝐳𝐳𝐚 𝐂𝐨𝐮𝐩𝐥𝐞, a young and talented digital creator, recently gained widespread attention on social media platforms with her viral video. The video quickly became a trending topic across various platforms, sparking a significant amount of discussion among viewers. As a rising star in the digital world, 𝐊𝐮𝐥𝐡𝐚𝐝 𝐏𝐢𝐳𝐳𝐚 𝐂𝐨𝐮𝐩𝐥𝐞's creativity and content have captivated audiences, contributing to her growing popularity on platforms like X and Twitter.
This viral moment has sparked conversations about the impact of digital content on social media trends. 𝐊𝐮𝐥𝐡𝐚𝐝 𝐏𝐢𝐳𝐳𝐚 𝐂𝐨𝐮𝐩𝐥𝐞's video, while generating attention, highlights the growing influence of young content creators who are reshaping the landscape of online media.


The sidebar in mobile view

Describe the bug
the sidebar in mobile view covers all the page and make the site looks less interactive. by opening the navbar whole screen is covered and it's a bad user experience
current behaviour

Expected behaviour
i want to make it look like a side panel opened and does not cover the whole screen providing good user experience.
example image

Screenshots
No response
Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

@amanjaiman1 please assign me this, so i can work on it
one more suggestion we can add one thing in the home page that's present on majority of websites
let our numbers do the talk.
example

let me know your view on this, if you approve i will make a issue and start workinh on it.


What are some of the strategies I can use to avoid any egress charges to my Log Analytics in different region than my azure service?

Can you provide some recommendation for avoiding any egress charges for sending data to log analytics service in different region? E.g. In Azure Gov cloud, Log Analytics is only available in US Gov Virginia region and services from another region might be using the Log Analytics which could be subject to egress charges.

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

ID: 521c75f5-5d71-10b5-df71-a8a6c0204a2e
Version Independent ID: 4343bd59-d34e-8d99-a723-292fc27b60bd
Content: Log Analytics FAQ
Content Source: articles/azure-monitor/platform/log-faq.md
Service: log-analytics
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@Anand-Moghe Thanks for the comment. Your feedback has been shared with the content owner for further review.

I'm going to include @DaleKoetke from the PG on this issue who can best answer it.

@DaleKoetke or @MGoedtel Any update on this?

@Anand-Moghe - Can you be more specific as to what the scenario/use-cases are and the resources in Azure Gov Cloud that you would configure to send data from?

@MGoedtel Lets say I have VM or App Service setup in region such as "US Gov Arizona", "US Gov Texas" or "US Gov Iowa" and I am using Log Analytics workspace in "US Gov Virginia".
Currently in Azure Government Cloud in USA, only "US Gov Virginia" has Log Analytics.
So now when my environment generates telemetry data from these regions different from Log Analytics workspace, what are the implication to egress as this data is now travelling out of regions where my services are configured. Wouldn't there be  egress cost to store these telemetry data in the Log Analytics workspace in different region?

Hi Anand,
As described in the Azure Bandwidth pricing page, data transfer between Azure services located in two regions charged as outbound data transfer at the normal rate. Inbound data transfer is free. However, this charge is very small (few %) compared to the costs for Log Analytics data ingestion. So controlling costs for Log Analytics needs to focus on your ingested data volume, and we have guidance to help understand that here.
Let me know how else I can help.
Thanks
Dale

@DaleKoetke Thank you!

This article indicates the egress is on a "region" level (e.g. Europe) not a sub-region level (e.g. Europe covers West Europe, North Europe, etc):- https://techcommunity.microsoft.com/t5/Azure-Sentinel/Best-practices-for-designing-an-Azure-Sentinel-or-Azure-Security/ba-p/832574#linkend%23 %23linkstart%23https:%2F%2Fdocs.microsoft.com%2Fen-us%2Fazure%2Fazure-monitor%2Fplatform%2Fdesign-logs-deployment%23access-mode%23linkend%23 %23linkstart%23https:%2F%2Fazure.microsoft.com%2Fen-gb%2Fblog%2Fprotect-virtual-machines-across-different-subscriptions-with-azure-security-center%2F.
Is this true?
@tianderturpijn


flagged this file as malicious

https://www.virustotal.com/gui/file/e4337a72d86e3f8d63b41a2604fab97729c78ad17d5593f6ac1a23f6c2654a75
Security vendors' analysis
Bkav Pro
W32.AIDetectMalware
Cynet
Malicious (score: 100)
Elastic
Malicious (moderate Confidence)
MaxSecure
Trojan.Malware.300983.susgen
SecureAge
Malicious
Acronis (Static ML)
Undetected

It's a false positive because I use PureBasic. Known problem:
https://www.purebasic.fr/english/viewtopic.php?t=78612
https://www.purebasic.fr/english/viewtopic.php?t=77459
https://www.purebasic.fr/english/viewtopic.php?t=72466
https://www.purebasic.fr/english/viewtopic.php?t=76051
https://www.purebasic.fr/english/viewtopic.php?t=69872
https://www.purebasic.fr/english/viewtopic.php?f=7&t=69166
http://forums.purebasic.com/english/viewtopic.php?t=75600
https://www.purebasic.fr/english/viewtopic.php?t=77433
https://www.purebasic.fr/english/viewtopic.php?t=61372
... and so on.
Just open the project in PureBasic and compile it yourself if you don't trust it. People wrote so much malware using the PureBasic standard library functions that every compiled executable gets flagged.

But thank you for trying to keep everyone safe regardless. 👍


Wrong colors displayed in spatial-js-demo example

When running the spatial-js-demo example, the following logs are seen in the JS Console:
three.module.js?88cf:8141 THREE.Color: Unknown color #8080
three.module.js?88cf:8141 THREE.Color: Unknown color #ff

Also some colors look wrong, e.g. extruded shape is white rather than teal. I fixed this by prepending extra zeros in Colors/rgbToString(), but need to make sure this does not break anything else.

Thanks, I though that I've fixed that. But maybe I am wrong, so I will check again.

Confirmed, fixed.


u32split parser does not output correct VM instruction sequence

As mentioned in the comment here current implementation of u32split instruction parser produces an incorrect instruction sequence. We could fix it as suggested in the comment, but there might be a better way.
In general, we want u32split to leave 1 at the top of the stack if all 4 top stack elements are u32 values and 0 otherwise. Thus, we can test if a given element is a u32 (leaving 1 at the top of the stack if it is) and then use AND operation to combine these values. Thus, we don't need to have NOT operations after EQZ and then we can replace OR operations with AND. This should also reduce the cycle count needed for this operation by 4.
So, for example, to test the first element, we'd do:
span_ops.push(Operation::Dup3);
span_ops.push(Operation::U32split);
span_ops.push(Operation::Swap);
span_ops.push(Operation::Drop);
span_ops.push(Operation::Eqz);

Instead of the current:
span_ops.push(Operation::Dup3);
span_ops.push(Operation::U32split);
span_ops.push(Operation::Swap);
span_ops.push(Operation::Drop);
span_ops.push(Operation::Eqz);
span_ops.push(Operation::Not);

And for the second, we could do:
span_ops.push(Operation::Dup3);
span_ops.push(Operation::U32split);
span_ops.push(Operation::Swap);
span_ops.push(Operation::Drop);
span_ops.push(Operation::Eqz);
span_ops.push(Operation::And);

Instead of the current:
span_ops.push(Operation::Dup3);
span_ops.push(Operation::U32split);
span_ops.push(Operation::Swap);
span_ops.push(Operation::Drop);
span_ops.push(Operation::Eqz);
span_ops.push(Operation::Not);
span_ops.push(Operation::Or);

current implementation of u32split instruction parser produces an incorrect instruction sequence
...
In general, we want u32split to leave 1 at the top of the stack if all 4 top stack elements are u32 values and 0 otherwise.

I think it may need to be clarified that this is for the parse_u32testw assembly op (which uses the U32split machine op), and not for the parse_u32split assembly operation

Ah yes! Thank you! Fixed!

Fixed by #72 and #91


Connection message missing in Realtime-API documentation

The Realtime-API documentation is missing a reference to the connection message that must be sent once before any other message to establish the WebSocket connection, otherwise the server will always respond with "Must connect first" message.
The connection message that I found watching those exchanged by the Rocket.Chat web client is:
{
	"msg": "connect",
	"version": "1",
	"support": ["1", "pre2", "pre1"]
}

This is the server response:
{
	"msg": "connected",
	"session": "daEkMjtYpGHhDuqRy"
}

This is instead the message that the server returns to every request if you do not send the connection message first:
{
	"msg": "error",
	"reason": "Must connect first",
	"offendingMessage": "[the-message-you-sent]"
}

Same problem here, it shoud be written in the docs :)

We welcome any pull requests to add this to the documentation @Davide95 👍


MobX integration best practices

Is anyone using this library with MobX?
So far I'm having trouble how can I integrate with the [zoom] array for example. If I do
zoom={mapUIStore.zoom.toJS()}

where mapUIStore.zoom is an observable.array, it doesn't work. By this I mean that

setting zoom to a new value -> works
scroll zooming the map to a different value
setting zoom to the same value -> doesn't work, as it's already set to the same value.

When [zoom] is used with setState is the state updated behind-the-scenes?
At this point, what do you recommend me to do? Should I just use the native zoomTo and forget about the auto-syncing [zoom] array magic?

I've been using Uber's react-map-gl to apply viewport updates reliably, and then continuing with react-mapbox-gl for its layer components: See #572


Download image resources as local resources

Download the latest version of image resources from online resources to local resources

merge latest dev into binbinCheng:dev-picture-local


FR: Being able to copy the text or at least make the text selectable

Hi! Nice plugin but it would be nice to be able to select  and copy the text too. Thanks!

Hi! Thank you for using it :) Please try v0.4.0 now you can drag and drop the text

thanks! i tried the update but you can't really select the text (or i am doing something wrong ?), when you click on the transcription and drop it onto the page  it copies a small arbitrary portion of the text. If making the ui of the transcript freely selectable is hard, why not just make a command that insert the transcript in the current page or a new page so that users can extract what they want from the transcript or search through it ...?

v0.5.0 ready, try right click on the text and choose 'copy all' option

perfect! thanks!
btw idk if you want me to create another ticket but it would be nice to be able to trigger the command without having to have a perfect selection of the whole url, just put the cursor on the string of a youtube url and that it would recognize and extract that string using obsidian editor and a regex.
Here's an example you could adapt from another plugin
https://github.com/zolrath/obsidian-auto-link-title/blob/18e6a809eb/editor-enhancements.ts

Thanks for new ideas!
Because plugin is still in young age for me it is ok to have here all your ideas.
Please check v0.6.0 if your cursor is inside url and nothing else is selected then url should be properly fetched.

Hi thanks! Did it work for you when you tried ? It's not working for me, idk why but getSelectedText seems to return a empty string. It's weird i tried with the obsidian-auto-link-title plugin and it works, not sure why it doesn't work in your case

Hi can you paste a screenshot where you have a link in the editor and where you have a cursor when using the command?

Sure here it is! I show you that i'm on the last version, that it doesn't work (twice) and that it work with obsidian-auto-link-title plugin.
https://user-images.githubusercontent.com/51007222/213266405-9d1e8cd4-0e4d-4334-baff-870adcffd2b7.mp4
But does it work for you? Because i'm looking at the debugger and getSelectedText return an empty url. Here's what i found:
i think this might have to do with
    return {
      start: { line: cursor.line, ch: match.index ?? 0 },
      end: { line: cursor.line, ch: match.index ?? 0 + match[0].length },
    };

why not just
      start: { line: cursor.line, ch: match.index },
      end: { line: cursor.line, ch: match.index + match[0].length },

like in obsidian-auto-link-title plugin. ?
In your case if match.index == 0

it returns 0 even for ch: match.index ?? 0 + match[0].length
Which put the wordBoundaries.end at 0 and this is why you get an empty string

Thank you very much @nicolasmarc1 for your detailed description. I hope version 0.6.4 will statisfy your needs

I'll close this issue because initial bug was fix and this new problem will be tracked here #4

Unfortunately copying text using 'Copy all' looses nice formatting.
this formatting is lost:

and copied text looks more like this:

Workaround is to use Developer Tools, copy whole element and convert output from html to markdown. Pasting retrieved transcript directly into the current note would be a nice option.

I'd like to solve this one


getting user location and the weather in it

Can you share more about how weather/location affects the purpose of this website? @PromiseTheCat


Update to go-git v5.1.0

Update to go-git v5.1.0.
Fix #9938
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

I suspect that we should probably backport this to 1.12 and possibly 1.11 as #9938 badly affects our FreeBSD users.

I can confirm the fix is there, @kevans91's PR https://github.com/go-git/go-git/pull/39 is included in 5.1.0 release of go-git per the release notes: https://github.com/go-git/go-git/releases/tag/v5.1.0 (and I can also see the code change in the vendor dir)

"backport" -> #11941

ping LG-TM


[NO-ISSUE] Prevent directory escape bypass through repeated URL decoding

What is this PR for?
This PR addresses an issue in NotebookService where the notebook path validation only performs a single decoding pass.
This allowed a malicious user to bypass validation by double-encoding the ".." token.
By implementing the repeated decoding, we can prevent this bypass.
Additionally, to prevent excessive decoding attempts, a maximum limit on the number of decoding attempts has been added.
What type of PR is it?
Hot Fix
How should this be tested?

CI

Questions:

Does the license files need to update? No
Is there breaking changes for older versions?

There may be minor compatibility issues if a user relies on multiple encoded paths, but this is unlikely in realistic scenarios.

Does this needs documentation? No

cc @jongyoul

@pan3793 @Reamer
When you have a moment, could you check this PR? Appreciate it!

@Reamer Thank you for the feedback! I've added a commit based on your feedback.
I agree that using five attempts feels somewhat arbitrary. However I couldn't think of a better alternative, so I went with this value.
If you have any other suggestions, I'd love to hear them!

I read the change and understand what it does, but I don't understand why should this change.
And please create a JIRA ticket for it, if you think this involves CVE which is not suitable to discuss publicly, you can write a detailed message to<EMAIL_ADDRESS>

I agree with @pan3793 . Please create a Jira ticket and adjust the pull request description accordingly.


artifact-registry-yum-plugin for el9, arm64

Build artifact-registry-yum-plugin for el9, arm64 and release it in OS specific repo

/lgtm


How to pass metadata to QdrantClient

I see that due to #19 and #20 that **kwargs has been removed from the QdrantDocumentStore __init__.
Is there a workaround to pass metadata to the QdrantClient?
Currently the following code doesn't work:
document_store = (
    QdrantDocumentStore(
        url=f"{QDRANT_APP_URL}",
        port=443,
        index=f"{QDRANT_INDEX}",
        embedding_dim=768,
        recreate_index=True,
        timeout=120,
        similarity="cosine",
        metadata={"Authorization": f"Bearer {get_user_token()}"}
    )
)

@MathiasSpanhove It might not be the most elegant solution to that, but could you please verify if that works?
document_store._client._rest_headers = {"Authorization": f"Bearer {get_user_token()}"}

It seems to be an ugly workaround, but I can't see a different way.

I could also create a PR with something like the following:
@classmethod
def from_qdrant_client(cls, client: qdrant_client.QdrantClient, **kwargs):
    cls.client = client
    return cls(**kwargs)

And add a check in the __init__ to make sure that the client hasn't already been provided?
This way all current implementations would keep working.
Let me know what you think and if you would like a PR!

I think we can expose the metadata parameter, as long as it doesn't break the YAML pipelines. Would you like to contribute?

I've made a PR that exposes the metadata parameter and doesn't break the YAML pipeline.


Sessions potentially were not released after response with TIMEOUT code

SESSION_BUSY errors were observed during a training of one DC shutdown. Small research showed up that these errors correlate with TIMEOUT errors.

All TIMEOUT error has typical issues:
Caused by: tech.ydb.core.UnexpectedResultException: Failed to execute query, code: TIMEOUT, issues: [#1030 Type annotation (S_WARNING)
  1:5 - 1:5: At function: RemovePrefixMembers, At function: Sort, At function: PersistableRepr, At function: SqlProject (S_WARNING)
  6:14 - 6:14: At function: Filter, At function: Coalesce (S_WARNING)
  38:17 - 38:17: At function: And (S_WARNING)
  9:17 - 9:17: At function: SqlIn (S_WARNING)
  9:17 - 9:17: #1108 IN may produce unexpected result when used with nullable arguments. Consider adding 'PRAGMA AnsiInForEmptyOrNullableItemsCollections;' (S_WARNING)]
	at tech.ydb.core.Status.expectSuccess(Status.java:78)

It all seems like the service doesn't release the session after sending the TIMEOUT error and the next request with this session fails with SESSION_BUSY error

@spuchin, @gridnevvvit could you take a look on that?


Allow ignoring certain folders/extensions

For many projects, .gitignore is not enough, we also want to explicitly ignore certain folders of autogenerated code while keeping them in version control.
For example, in our project we have:
client/
    js/
        vendor/
             ...       // we want to ignore everything in here, but keep it in our VCS

server/
    build/
        ...            // we also want to ignore all autogenerated code in here, while keeping it in our VCS
    src/
        ...            // but we do want to autocomplete any files from here

Suggested Fix:
Allow adding a .tabnineignore file, just like .gitignore, .dockerignore, .npmignore, etc.
This can be excluded from VCS or added, it's up to the user.

tabnineignore will be added in the next release. In the meantime, you could try adding a .ignore file.


Add skip files parse param

The get_files_types method is the most CPU intensive part of parsing an APK and on some bigger APKs can take minutes depending on your machine.  This param lets you optionally skip it.

It is probably not even necessary to have this flag, just remove the self.get_files_types() from __init__ - the file types should then be generated on the first call of get_file_types.
What do you think?

Yea that works too, I had just assumed you would want to keep the default behavior.
I can update the PR with that change if it's what's preferred.

@reox Force updated the branch.  How does that look?

looks good, thanks!


Implement a template engine

A templating engine needs to be implemented, it needs to check the provided template.html file for variables.
I've decided after some research that I initially am going to implement Pebble for this.

I know that there is a vert.x library for this, but this library requires the templates to be present in the file system, while I want to store the templates in the database.


VM Size test does not recognize params in object vars

Run the ttk on this sample:
https://github.com/Azure/azure-quickstart-templates/tree/master/application-workloads/sharepoint/sharepoint-adfs
It looks like we may have regressed something in #682

@StartAutomating

@bmoore-msft got repro working.

@StartAutomating @bmoore-msft
We are publishing our product, and this issue caused the test to fail.
I'am look forward to you fixing this problem as soon as possible.
  VM Size Should Be A Parameter
    [-] VM Size Should Be A Parameter (61 ms)
        VMSize parameter must be declared for the parent template

We've encountered an issue with this as well, we have a top level template and inner template using a parameter name of size instead of vmSize, but it maps into the hardwareProfile.vmSize property of the inner template, but it is reporting a failure even though the vmSize is coming from a parameter.

Due to this, our publish  plan has been delayed by more than a month.

We had a rather poor experience.😠😠

We ran into the same issue, see https://github.com/Azure/arm-ttk/issues/712， which required huge changes to our templates (https://github.com/oracle/weblogic-azure/pull/215) .

It seems that many people have similar problems, please speed up.

I also with the problem, please fix it asap, I hope your efficiency like AWS.

This appears to be back.
`D:\a\1\s\arm-ttk\testcases\deploymentTemplate\VM-Size-Should-Be-A-Parameter.test.ps1 : VM Size for resourceType '  '
named '  ' must be a parameter
At D:\a\1\s\arm-ttk\Test-AzTemplate.ps1:253 char:21

                . $myModule $TheTest @testInput 2>&1 3>&1

                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,VM-Size-Should-Be-A-Parameter.test.ps1`

On code where the VM size is declared as;
var referenceImage = {   windows: {     publisher: 'MicrosoftWindowsServer'     offer: 'WindowsServer'     sku: '2019-Datacenter'     version: 'latest'     VMsize: 'Standard_D4s_v3'   } }

@GeordieGuy Can you provide a template where we can replicate this issue so we can determine what is causing the error? Thanks!

@GeordieGuy it seems that you are hardcoding the VMSize to 'Standard_D4s_v3'. The VMSize property should take in a parameter.

@GeordieGuy it seems that you are hardcoding the VMSize to 'Standard_D4s_v3'. The VMSize property should take in a parameter.

No. Btw, this issue is 5 months old. I barely remember what it was.

@GeordieGuy it seems that you are hardcoding the VMSize to 'Standard_D4s_v3'. The VMSize property should take in a parameter.

If I think back to 5 months ago when I was having this problem, declaring a variable shouldn't require me to populate it with other parameters. ARMTTK thinks this declaration is creating a VM with a hardcoded SKU, it isn't, it's defining a variable.


fix(select): make invalid selector more specific

The selector for the touched/invalid style was too general, so it was picking up when the parent form was "touched" rather than the control itself.  This PR ensures it only applies touched/invalid styles when its own control is touched/invalid.
Fixes #2147.
r: @jelbourn

It still the same....all the md-select will turn red even without touching the selector

@dreweird This change isn't merged into master.

LGTM

Anyone know when it will be released?


Better Email Validation

Problem statement
The current email validations are provided by the https://github.com/asaskevich/govalidator project. It is a complex and custom regex that does not cover a lot of valid emails. (notably, it does not cover the .mil domain at all which is getting in my way) I recommend that a different validator be used for the go-swagger project. There is an example on the issue here: https://github.com/asaskevich/govalidator/issues/131 of how to use the govalidator project with a different validator. The maintainer is reluctant to switch away from the validator they have built. I think the simple one provided by https://www.regular-expressions.info/email.html would be a good change. In the end, it's best not to be too strict in validating emails, if you need to validate an email you should send it a confirmation link.

feel free to send a PR with a better regex

I'm running into this too, I think because I'm using a new(ish) TLD. I would like to fix this to fix this, but I have no idea where the email validation is taking place. Could someone tell me where this validation is taking place?

my 2c on actual email validation would be that it would need to use an actual mime address parser because if you go through the rfc for this stuff it's pretty complex what you can put in an email address.
https://golang.org/pkg/net/mail/#Address
Here's the implementation of the email type: https://github.com/go-openapi/strfmt/blob/master/default.go#L393-L473
And here's where the regex is registered.
https://github.com/go-openapi/strfmt/blob/master/default.go#L138-L139
A validator is a func(string) bool

From the email addresses listed in the asaskevich/govalidator#131
The following don't parse with the golang mail package, but they are also very esoteric.
<EMAIL_ADDRESS>very.unusual."<EMAIL_ADDRESS>very."(),:;<>[]".VERY."very@\\ <EMAIL_ADDRESS>much."more\<EMAIL_ADDRESS><EMAIL_ADDRESS>allen@[IPv6:0:0:1]

Maybe use the same regex as html5 input-tag type=email: https://www.w3.org/TR/html5/forms.html#valid-e-mail-address ?


M03-LAB01: Tip

Contact Details
No response
What happened?
LP 3 lab 1 ex 1 task 5.16
Description of issue:
This is displayed as optional:
Note: For the purposes of this lab you could select Windows Event Logs. These selections can be revised later.
In LP 7 you'll need the windows event log which can take some time to sync. It would be easier for the students if this would be listed as a mandatory action this early in the course.
Lab
Lab 03 Exercise 01 Enable Microsoft Defender for Cloud
Relevant screenshots
paste here 😉
Do you want to help us? 👏

[ ] Create a Pull Requestt by following this project's Contribution Guide

I 'll have this ready for Friday.

These labs are completely revised, and I believe this no longer an issue.


[Bug] Mode: token recieve nuxt error

Describe the bug
When I try to go to login page it gives me nuxt 500 error like this [nuxt] A composable that requires access to the Nuxt instance was called outside of a plugin, Nuxt hook, Nuxt middleware, or Vue setup function. This is probably not a Nuxt bug. Find out more at https://nuxt.com/docs/guide/concepts/auto-imports#vue-and-nuxt-composables.
at useAppConfig (./node_modules/nuxt/dist/app/config.js:38:41)
at Module.updateAppConfig (./node_modules/nuxt/dist/app/config.js:46:22)
at setupDefaultTokenStorage (./node_modules/nuxt-auth-sanctum/dist/runtime/plugin.js:30:25)
at process.processTicksAndRejections (node:internal/process/task_queues:105:5)
at async ./node_modules/nuxt-auth-sanctum/dist/runtime/plugin.js:68:101
at async setup (./virtual:nuxt:/home/casper/Documents/projects/AlbaniaDanceFederation/albania-dance-federation/cms/.nuxt/plugins/server.mjs:45:116)
at async Object.callAsync (./node_modules/unctx/dist/index.mjs:72:16)
at async applyPlugin (./node_modules/nuxt/dist/app/nuxt.js:146:25)
at async executePlugin (./node_modules/nuxt/dist/app/nuxt.js:183:9)
To Reproduce
Steps to reproduce the behavior:

Copy starter pack from minibile theme from theme forest
npx nuxi@latest module add nuxt-auth-sanctum
Try going to localhost:3000 error shown

Expected behavior
Log me in
Screenshots
If applicable, add screenshots to help explain your problem.
Module information

Version:  "nuxt-auth-sanctum": "^0.4.13"
Complete configuration of sanctum from your nuxt.config.ts

export default defineNuxtConfig({
    modules: ['nuxt-auth-sanctum'],

    sanctum: {
    baseUrl: 'http://localhost:7305/', // Laravel API
    mode: "token",
    userStateKey: 'sanctum.user.identity',
    redirectIfAuthenticated: true,
    redirectIfUnauthenticated: false,
    endpoints: {
      csrf: '/sanctum/csrf-cookie',
      login: '/v1/auth/login',
      logout: '/v1/auth/logout',
      user: '/v1/auth/me'
    },
    csrf: {
      cookie: 'XSRF-TOKEN',
      header: 'X-XSRF-TOKEN',
    },
    client: {
      retry: false,
    },
    redirect: {
      keepRequestedRoute: false,
      onLogin: '/',
      onLogout: '/login',
      onAuthOnly: '/login',
      onGuestOnly: '/',
    },
    globalMiddleware: {
      enabled: false,
      allow404WithoutAuth: true,
    },
    logLevel: 3,
  },,
});

Nuxt environment:

Version: "^3.6.5"
SSR Enabled:  no
Environment: local

Laravel environment:

Version: 11
Sanctum installed via Breeze:  no
Checklist completed: yes
Session domain from your config/session.php: .domain.test yes
List of stateful domains from your config/sanctum.php: ['domain.test', 'www.domain.test'] yes
CORS settings from your config/cors.php yes

return [
    'paths' => ['*'],
    'allowed_methods' => ['*'],
    'allowed_origins' => [
        env('FRONTEND_URL', 'http://localhost:3000'),
    ],
    'allowed_origins_patterns' => [],
    'allowed_headers' => ['*'],
    'exposed_headers' => [],
    'max_age' => 0,
    'supports_credentials' => true,
];

Hey @filiptalev, I tried to reproduce the behavior but it worked fine without any error.
I noticed that you have the Nuxt version as 3.6.5, can you try upgrading it to the latest one?
Also, could you provide a code sample of your Vue component/service where you are trying to call the login function of the nuxt-auth-sanctum module?

+1 I have the same issue, it does not appear to be an issue with nuxt. I had to downgrade the package to "nuxt-auth-sanctum": "0.4.11" make it work.
This is how i am calling the login.
<template>

Login

        <UInput class="mb-2" v-model="email" placeholder="Email" type="email" />
        <UInput v-model="password" placeholder="Password" type="password" />
        <UButton class="mt-4" @click="click">Login</UButton>

</template>

Hi @shreejalmaharjan-27, thanks for the code sample, could you also provide nuxt.config.ts and your nuxt version?

Hi @shreejalmaharjan-27, thanks for the code sample, could you also provide nuxt.config.ts and your nuxt version?

My nuxt version is "nuxt": "^3.13.2",
Nuxt config
// https://nuxt.com/docs/api/configuration/nuxt-config
export default defineNuxtConfig({
  compatibilityDate: '2024-04-03',
  devtools: { enabled: true },
  modules: [
    "@nuxt/ui",
    "nuxt-auth-sanctum",
    "@nuxtjs/i18n",
    "@nuxt/content",
    '@nuxt/image',
  ],

  sanctum: {
    baseUrl: 'http://localhost/api', // Laravel API
    endpoints: {
      user: '/user',
    },
    mode: 'token',
  },
  icon: {
    provider: 'server'
  },

  i18n: {
    vueI18n: './i18n.config.ts',
    locales: [
      {
        code: 'en',
        iso: 'en-US',
      },
      {
        code: 'fr',
        iso: 'fr-FR',
      },
      {
        code: 'es',
        iso: 'es-ES',
      },
      {
        code: 'de',
        iso: 'de-DE',
      },
      {
        code: 'pt',
        iso: 'pt-PT',
      },
      {
        code: 'it',
        iso: 'it-IT',
      }
    ],
    defaultLocale: 'en',
  },
})

@shreejalmaharjan-27 @filiptalev
I have just published a new version 0.4.14, please try upgrading and let me know if it helps!

@shreejalmaharjan-27 @filiptalev I have just published a new version 0.4.14, please try upgrading and let me know if it helps!

All good on my side! Thank you for the quick fix

IT WORKS!!!!!!!!!! I love you people <3 @manchenkoff  @shreejalmaharjan-27

ITTT WOORKKSSS!!!


Link to original thread

I cannot just read the full contents of a reply by just clicking on it.  I have to go to the original post and to the replies section to see the full contents of the reply.
Here is a screenshot

Could you please test and close if it okay.
Android Apk Download link: https://install.appcenter.ms/orgs/esteem.app/apps/esteem/distribution_groups/beta_testers
IOS Ipa Download link:
https://install.appcenter.ms/orgs/esteem.app/apps/esteem-1/distribution_groups/beta_testers
Thank you. @horpeyloaded


fix!: move c.stream* to helper

ref: #1843
Author should do the followings, if applicable

[x] Add tests
[x] Run tests
[x] yarn denoify to generate files for Deno

@sor4chi
Is this ready to review?

@yusukebe Yes, please.

Hi, @yusukebe
Thanks, I fixed.