Windows 1809 symlink issues

My Environment

[ ] Windows 7 or below (not truly supported due to EOL - see wiki for details)

[ ] Windows 8

[ ] Windows 8.1

[x] Windows 10

[ ] Windows 10 IoT Core

[ ] Windows Server 2012

[ ] Windows Server 2012 R2

[ ] Windows Server 2016

[x] My Windows installation is non-English.

I'm using NVM4W version:

[x] 1.1.7
[ ] 1.1.6
[ ] 1.1.5
[ ] 1.1.4
[ ] 1.1.3
[ ] 1.1.2
[ ] 1.1.1
[ ] Older
[ ] OTHER (Please Specify)

I have already...

[x] read the README to be aware of npm gotchas & antivirus issues.
[x] reviewed the wiki to make sure my issue hasn't already been resolved.
[x] verified I'm using an account with administrative privileges.
[x] searched the issues (open and closed) to make sure this isn't a duplicate.
[x] made sure this isn't a question about how to use NVM for Windows, since gitter is used for questions and comments.

My issue is related to (check only those which apply):

[ ] settings.txt
[ ] proxy support (Have you tried version 1.1.0+?)
[ ] 32 or 64 bit support (Have you tried version 1.1.3+?)
[ ] Character escaping (Have you tried version 1.1.6+?)
[x] A standard shell environment (terminal/powershell)
[ ] A non-standard shell environment (Cmder, Hyper, Cygwin, git)

Actual Behavior
With Windows 1809 the symlinks seems to have a little delay until they are fully there. Our Build Servers are running the following commands
nvm install x.x.x
nvm use x.x.x
npm install

After the nvm use, the symlink resets and npm is not a known command. After some time the symlink is present in the filesystem and the npm commands works again.
Expected Behavior
After using nvm use x.x.x the command should wait until the symlink is present in the filesystem, to avoid issues with not finding npm and node.
Steps to reproduce the problem:
With Windows 1809 you can open a cmd with these commands
nvm install 10.11.0 && nvm use 10.11.0 && dir

Output: 10.01.2019  10:30    <SYMLINKD>     nodejs [..]

Sometimes the symlink is completely gone. With an delay inbetween the directory symlink appears
nvm install 10.11.0 && nvm use 10.11.0 && timeout 1 > NUL && dir

Output: 10.01.2019  10:35    <SYMLINKD>     nodejs [C:\develop\nvm\v10.11.0]

You can also add npm --version instead of dir to get "the system can not find the path" error
With virtualization the timeout of 1 second is sometimes not enough to get the symlink working.

Closing due to age.
Unfortunately, this is not something the app can detect without adding significant overhead in non-CI cases. The only possibility is to keep the application open in a loop until the symlink is detected.
In CI environments, use a larger timeout or execute the command sequentially (instead of chaining them together with the && notation.
A note has been added to the wiki regarding this issue and workarounds: https://github.com/coreybutler/nvm-windows/wiki/Common-Issues#delayed-changes


Use structured logging

It looks nicer in code and output (if piped through jq), and it's much better for post-processing.
This implies a more common-practice use of context. A "ctx" should be passed almost everywhere, not a "ctx.Done()".
I think I am almost as big fan of structured logging as @pohly  :smiley:, so I'll be happy to do a PR (but that will be a big one).
/kind feature

/assign @uablrek
please go for it, there is also a dry-run feature @npinaeva is talking about that can make use of this

An informative slack thread with @pohly as ref: https://kubernetes.slack.com/archives/C020CCMUEAX/p1706617958386689

/remove-lifecycle stale
(IMO should issues that have a PR not be stale)


Functional Transformations on Streamers

I'd love to have a monadic/functional methods on Streamer and its subclasses. Those include:

map() to apply data transformation like normalization
flatmap() for parsing files, augmenting data
filter() to filter data based on some criteria
shuffle() to shuffle the dataset based on a buffer
group() to make fixed-sized batches of data points
take()/skip() to get the first few or other than the first few data points
and many others listed in the proof of concept implementation

This programming style and nomenclature are from the functional programming idioms, but they are becoming increasingly popular in both server-side asynchronous/event-based programming and data processing. Examples include ReactiveX in many languages, CompletableFuture in Java 8+, every collection type in Scala, RDD in Apache Spark, and most recently tf.data from TensorFlow.
Personally, I think this is the right API to deal with the flow of the large datasets we work on - It makes it simple to compose short and easy-to-debug operations to build a wide range of complex behaviors, and the data is treated immutable making the whole process less error-prone.
I can imagine the following kinds of use cases:

Streamer(filenames).flatmap(tf.python_io.tf_record_iterator) will transform a Streamer of file names to a streamer of tensorflow Example protocol buffers.
We could build something equivalent to tf.parse_single_example where it isn't a TensorFlow operation and we don't have to deal with Graph and Session quirks
Using MUDA's and librosa's functionalities on-the-fly, without requiring us to consume excessive data storage.

The proof of concept implementation is in Python 3 (will break miserably in Python 2 for now) and has an implementation of Streamer class with desired methods, along with example usages. The part 4 shows how this API can perform multiplexing on its own, and part 5 shows how it can be incorporated to the current Class API without breaking it. It relies heavily on generator comprehensions and lambdas to enable lazy evaluation (which makes the examples like the sieve of Eratosthenes possible), but I hope this can be incorporated in a less hacky-looking way.
The naming and arguments for each method and whether or not to include them are up to discussion. Please let me know how you'd think!

Sorry to take so long in responding to this!  (What's 18 months between friends, anyway...)
I like the general idea here, but I'm a little wary of scope.  Much of what you describe applies to any kind of iterable sequence, and could already be achieved using functionality in itertools. I'm not sure we need to do anything here.
The one big exception would be group, which really needs to understand the shape of iterates, and which we already implement as buffer.  (shuffle is a bit of a weird case, but I think you can reasonably implement that as a generic iterator.)


Make 'somatic nervous system' and 'pharyngeal nervous system' subclasses of 'nervous system'

The terms:
pharyngeal nervous system (WBbt:0005440)
somatic nervous system (WBbt:0005760)
should be made subclasses of:
nervous system (WBbt:0005735)

@raymond91125
It seems you've resolved this issue already? Thanks!
I'll close the ticket. Please reopen if needed.


Rename all the things to ambiata

@markhibberd This what you had in mind? /cc @olorin

:shaved_ice:

:+1: looks good. Will rename the repository at some point as well.

Have updated to Ambiata.Cli. Will merge after a green build.


Performance estimator CBPE calculates realized ROC AUC using calibrated probabilities (while it should use raw)

Describe the bug
I have noticed differences between realized roc auc calculated by performance estimator vs. performance calculator.
To Reproduce
Steps to reproduce the behavior:
import pandas as pd
import nannyml as nml

reference_df = nml.load_synthetic_binary_classification_dataset()[0]
analysis_df = nml.load_synthetic_binary_classification_dataset()[1]

estimator = nml.CBPE(
    y_pred_proba='y_pred_proba',
    y_pred='y_pred',
    y_true='work_home_actual',
    timestamp_column_name='timestamp',
    metrics=['roc_auc'],
    chunk_size=5000)

estimator.fit(reference_df)

results_estimation = estimator.estimate(reference_df).data

calc = nml.PerformanceCalculator(
    y_pred_proba='y_pred_proba',
    y_pred='y_pred',
    y_true='work_home_actual',
    timestamp_column_name='timestamp',
    metrics=["roc_auc"],
    chunk_size=5000)

calc.fit(reference_df)

results_monitoring = calc.calculate(reference_df).data

results_monitoring['roc_auc'] - results_estimation['realized_roc_auc']

returns:
0   -0.000224
1    0.000146
2   -0.000257
3   -0.000456
4   -0.000133
5   -0.000272
6   -0.000323
7   -0.000271
8    0.000341
9    0.000237
dtype: float64

The values should be the same so it should return zeros only. I noticed that this is due to the fact that CBPE uses calibrated probabilities for realized performance calculation (a bug).
Expected behavior
CBPE should use raw probabilities to calculate realized performance.

Fixed the issue for both binary and multiclass classification cases.
I've pushed the fix to the main branch, official release is soon to follow.


[WIP] Add RHC Connection Availability Check

This is going to sit for a while until #111 is merged, the org_id <-> acct # service is deployed, and getting time to finish it.
Pretty simple process though - loop through all of the join records and request availability from RHC for each of them. It gets returned from the POST request right in the HTTP response, nice!

/retest

Well - it looks like a bunch of RHC Connection DAO tests are failing now that I added the relationships on the struct(s). It may be due to letting gorm generate the relations I'm wondering.

So it turns out we need this functionality yesterday - I commented out all of the failing tests just because they're valid, but adding the GORM relations on the structs just broke most of these. GORM is creating foreign keys etc that make the tests fail.
For now I would just like to get the RHC availability chekcing code on stage to the satellite folks can validate it.

It looks like an IQE failure, AttributeError: 'IQEVaultReference' object has no attribute 'openapi_types', so probably safe to ignore.


Migrate to the new rule set API

What's done:

@Suppress("Deprecation") annotations removed.

maybe migrate to the old rule set API?


Running MPI with DEVITO_MPI=0 , missing attribute data

I'm trying to run MPI on a python script where each rank calls Devito operations to forward-model a single shot. So I want Devito to run without any awareness of MPI but I'm calling the script with MPI to parallelize the I/O across several shots.  Here's my minimal reproducible (hopefully) example
So before running this I have set DEVITO_MPI=0 as an environment variable.
"mpirun -np 2 python test.py" (note this works if I run mpirun -np 1)
Here's "test.py":

from mpi4py import MPI
from devito import Grid, Function, configuration

grid=Grid(shape=(10,10,10), extent=(100,100,100), comm=None)
vel=Function(name="vel", grid=grid, space_order=2)
vel.data[:]=1.0

print("All done")

I get this error printed twice.
Traceback (most recent call last):
File "test.py", line 12, in
vel.data[:]=1.0
File "/root/devito-cpu/devito-src/devito/devito/finite_differences/differentiable.py", line 130, in getattr
raise AttributeError("%r object has no attribute %r" % (self.class, name))
Notes: It works (although not as I intend) if I set DEVITO_MPI=1. I'm not getting an error if I try to access the "data" attribute of SparseTimeFunction objects.

Update: it looks like you can workaround this by setting DEVITO_MPI=1 AND initializing my grid with comm=MPI.COMM_SELF. Doing so is now giving me the expected behavior.
Not sure if this is expected behavior, but comm=None (as in the example above) does not work, and similarly I can't unset DEVITO_MPI (or set it to 0) while running with more than one rank.

I've added this to my TODO list. Thanks for reporting

Hi @jrt54, this seems to be resolved now.
Unless you have any objections maybe we can close this issue.

Hello,
I'm trying to parallelise an inversion code over the shots. The python code works serially. But when I parallelise it using mpi4py, the same error as  jrt54 occurs. Setting DEVITO_MPI to 1 or 0 creates some new errors. as follows:

Any idea how I could solve this?
Thank you,


Introspect endpoint backwards compatibility

This issue is imported from pivotal - Originaly created at Apr 29, 2019 by Okke Harsta
The https://tools.ietf.org/html/rfc7662 spec dictates that the Resource Server issues a POST for the introspection endpoint. Currently we also support GET requests. Proposal is to follow the spec as implemented in the nimbus library.
Agree?
The introspect endpoint in https://tools.ietf.org/html/rfc7662 spec - and the nimbus implementation - supports two authorization mechanisms: basic authentication with client_id & secret and separate bearer token. Proposal is to only allow the basic authorization.
Agree?

Yes I agree with both points. We do need to rewrite all introspect calls from our internal applications though.
Currently there is no use case to support bearer token authentication for the introspect.  (Bart Geesink - Apr 29, 2019)


Defense in depth SAML Response validation

This PR adds two important safety features to EngineBlock
1. A test to verify we are not vulnerable to the Hackmanit signature validation bypass exploit
This simulates the H01 finding described in detail in [1]. Effectively verifying we are not vulnerable to this exploit.
The test intercepts the SAML Response sent from the IdP and injects malicious XML to the Signature element, exploiting the vulnerability. The response is then sent to EngineBlock to process.
Depending on the presence of the safe xmlseclibs, EngineBlock would then process this assertion without a problem. Allowing the attacker to provide an assertion of their liking. From xmlseclibs 3.0.4, the SAML2 library is no longer vulnerable to this attack. And in this PR, EngineBlock will also provide defense in-depth protection against tampered with SAML Responses.
2. Defense in depth countermeasures
Several checks are performed on the SAMLResponse. This allows us to reject clearly invalid responses before actually processing them. See this as an early, high level validation process.
The tests give insights into the actual validations that are performed in the SamlresponseValidator.
This addresses finding H01 in the Security audit performed by Hackmanit
See:

https://www.hackmanit.de/en/blog-en/82-xml-signature-validation-bypass-in-simplesamlphp-and-xmlseclibs
https://simplesamlphp.org/security/201911-01
https://nvd.nist.gov/vuln/detail/CVE-2019-3465

Target release pending. @thijskh @pmeulen @quartje what releases would you like to see these changes land in?
My gut feeling tells me it should be somewhere post 5.13 but before 6.1

If this is the same as our forked repository, and you've just pushed to this remote just feel free to merge. Would you else elaborate what you want to have reviewed again?

Would you else elaborate what you want to have reviewed again?

I mentioned you as a reviewer, mainly to keep you in the loop on where this will land.


Update common to 0.14.5

Updates io.github.cquiroz.react:common from 0.14.2 to 0.14.5.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.github.cquiroz.react", artifactId = "common" } ]

labels: library-update, semver-patch

Done as part of #59


Activate Project Interpreter to execute recipe

For example: Python virtualenv, Java JDK etc.

@funkyfuture  Fixed with version 0.6.3. If any problem, let me know.

thanks, it works.
then there's just the issue left that the console output isn't colored, which is is sometimes useful when tools encode information by using colors, usually something to catch attention or to distinguish text contents.

@funkyfuture   I disabled color output on Windows because of a bug  :)   I will have a test and fix color output on Windows.

@funkyfuture  Fixed with version 0.6.6.

i have only reasons to despise the Windows OS family. most times i use PyCharm on Ubuntu.

@funkyfuture  I closed this issue. If still some problems, please create an new issue with screenshot. thanks.


Naive chart

https://user-images.githubusercontent.com/82500/131909955-a54ab43e-c505-41c0-b768-5cb23e477954.mp4

This fails on my test env with:
iaso_1     | Traceback (most recent call last):
iaso_1     |   File "/usr/local/lib/python3.6/site-packages/django/core/handlers/exception.py", line 47, in inner
iaso_1     |     response = get_response(request)
iaso_1     |   File "/usr/local/lib/python3.6/site-packages/django/core/handlers/base.py", line 181, in _get_response
iaso_1     |     response = wrapped_callback(request, *callback_args, **callback_kwargs)
iaso_1     |   File "/usr/local/lib/python3.6/site-packages/django/views/decorators/csrf.py", line 54, in wrapped_view
iaso_1     |     return view_func(*args, **kwargs)
iaso_1     |   File "/usr/local/lib/python3.6/site-packages/rest_framework/viewsets.py", line 114, in view
iaso_1     |     return self.dispatch(request, *args, **kwargs)
iaso_1     |   File "/usr/local/lib/python3.6/site-packages/rest_framework/views.py", line 505, in dispatch
iaso_1     |     response = self.handle_exception(exc)
iaso_1     |   File "/usr/local/lib/python3.6/site-packages/rest_framework/views.py", line 465, in handle_exception
iaso_1     |     self.raise_uncaught_exception(exc)
iaso_1     |   File "/usr/local/lib/python3.6/site-packages/rest_framework/views.py", line 476, in raise_uncaught_exception
iaso_1     |     raise exc
iaso_1     |   File "/usr/local/lib/python3.6/site-packages/rest_framework/views.py", line 502, in dispatch
iaso_1     |     response = handler(request, *args, **kwargs)
iaso_1     |   File "/opt/app/iaso/api/instances.py", line 313, in stats
iaso_1     |     df.index = df.index.to_period("M")
iaso_1     | AttributeError: 'Index' object has no attribute 'to_period'


Add imkSushi's Better Densities Mod from SpaceDock

This pull request was automatically generated by SpaceDock on behalf of imkSushi, to add imkSushi's Better Densities Mod to CKAN.
Mod details:
name = /mod/751/imkSushi%27s%20Better%20Densities%20Mod
author = imkSushi
abstract = Planets in KSP are super dense. This fixes it.
license = MIT
Homepage = http://forum.kerbalspaceprogram.com/index.php?/topic/141365-imksushis-better-densities-mod/
description = REQUIRES:
Sigma Dimensions (which requires Kopernicus)
In ksp, all the planets are roughly 10x denser than in real life, such that they have a realistic surface gravity. This mod changes it such that instead of:
Realistic Surface Gravity BUT Unrealistic Density;
You get this:
Unrealistic Surface Gravity BUT (more importantly) Realistic Density;
NOTE:
Do not install this on an install of ksp that has a save you care about without backing up the save.
Credits:
Me -for a decent idea;
Sigma88 - for Sigma Dimensions;
ThomasP - for Kopernicus;
On a more serious note, I do think that it is perfectly viable to do a playthrough with this mod, so if you want to, show me some of your spacecraft that you use.
Please direct questions about this pull request to imkSushi.

Closing in favour of #4223


Use actual log timestamp as logstash timestamp

Based on the grok example from the docs, a Filebeat + Logstash stack will produce the following:

a @timestamp field based on the time that filebeat sends the log to logstash, as timestamp type.
a timestamp field from the Spring Boot log, as string type.

In order to use the Spring Boot timestamp as the real Logstash @timestamp I use the following filters after the grok pattern:
  date {
    match => ["timestamp", "ISO8601"]
  }
  mutate {
    remove_field => ["timestamp"]
  }

Using these filters will result in @timestamp being the Spring Boot's timestamp, as a timestamp type, and will remove the now duplicated (string type) timestamp.

Very good point, thanks!


"Software Error"

This started happening to me sometime yesterday I think.  Since the error asks for the date and time, this  happened around 2016-11-09 10:17:51.543438 Korea time.
Software error:

Could not connect Access denied for user 'johnhol_inject'@'localhost' (using password: YES) at ../../../injection/common.pm line 13.
Compilation failed in require at sale.cgi line 4.
BEGIN failed--compilation aborted at sale.cgi line 4.
For help, please send mail to the webmaster (webmaster@injection.johnholdsworth.com), giving this error message and the time and date of the error.

Injection console shows this error, but it was signing ok a day or two ago.
/Users/timothycosta/Desktop/lingq/iLingQ/iOSInjectionProject/build/Debug-iphonesimulator/InjectionBundle1.bundle: is already signed

*** Could not codesign as 'iPhone Developer: Tim Costa (****)':
/Users/timothycosta/Desktop/lingq/iLingQ/iOSInjectionProject/build/Debug-iphonesimulator/InjectionBundle1.bundle ***

 at /Users/timothycosta/Library/Application Support/Developer/Shared/Xcode/Plug-ins/InjectionPlugin.xcplugin/Contents/Resources/common.pm line 57.

main::error('Could not codesign as 9hone Developer: Tim Costa (SV7EDU5N...') called at /Users/timothycosta/Library/Application Support/Developer/Shared/Xcode/Plug-ins/InjectionPlugin.xcplugin/Contents/Resources/injectSource.pl line 513

*** Bundle build failed ***

By the way, the suggested email bounces.

Thanks for letting me know. I’ll look into it.

The "could not codesign" bit looked suspicious, so I explicitly set that certificate in my project settings and that error went away.  However, even without errors my changes didn't seem to be being applied to the code.  I ended up restarting Xcode for unrelated reasons and the first time I tried to inject it said it was reinstalling my license.  Seems to be working again.

Haven't had issues with "Software Error" since my last message, but I ended up getting the same "already signed" errors again.  In case anybody else finds this:

Set header and framework search paths in your injection bundle project
Set your development team and provisioning profile in the injection bundle project.  Make sure the development team matches the profile.  You may need to use "Other" in the provisioning dropdown, in which case you can just use the "code" portion of the profile  - iOS Developer My Name (CODE)
Set the same provisioning profile in iOSInjectionProject/x86_64/identity.txt

I’ve added —force to the code signing so you shouldn't see this problem again.


TypeError: Invalid Version: 1.3 when running yarn

I am trying to install the theme and when I hit yarn in vuetique directory I got below error.

[Arguments:
/usr/local/bin/node /usr/local/bin/yarn
PATH:
/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/snap/bin
Yarn version:
1.19.0
Node version:
10.16.3
Platform:
linux x64
Trace:
TypeError: Invalid Version: 1.3
at new SemVer (/usr/local/lib/node_modules/yarn/lib/cli.js:3479:11)
at Range.module.exports.Range.test (/usr/local/lib/node_modules/yarn/lib/cli.js:4286:15)
at Function.satisfies (/usr/local/lib/node_modules/yarn/lib/cli.js:4335:16)
at WorkspaceLayout.getManifestByPattern (/usr/local/lib/node_modules/yarn/lib/cli.js:32262:31)
at Function.isWorkspace (/usr/local/lib/node_modules/yarn/lib/cli.js:98062:51)
at PackageRequest. (/usr/local/lib/node_modules/yarn/lib/cli.js:36629:76)
at Generator.next ()
at step (/usr/local/lib/node_modules/yarn/lib/cli.js:304:30)
at /usr/local/lib/node_modules/yarn/lib/cli.js:322:14
at new Promise ()](url)

Hi @rizwan-i-khan,
I've fixed theme version.
BTW, you don't need to run yarn from theme folder, just run it from vue-storefront root.

HI, @dmitry-kozachek I have run yarn from the vue-storefront root and version error is resolved after your latest commit. but I got this error when yarn build can you please help me.
The current yarn version is 1.19.1

ERROR in ./src/themes/vuetique/index.js
Module not found: Error: Can't resolve '@vue-storefront/core/lib/router-manager' in '/var/www/html/magento231-vue-pwa/vue-storefront/src/themes/vuetique'
@ ./src/themes/vuetique/index.js 7:0-72 24:2-15
@ ./core/app.ts
@ ./core/client-entry.ts
@ multi babel-polyfill ./core/client-entry.ts


Update formula to release v1.8.0

/cc @lionelvillard @dsimansk
Created this PR based on https://github.com/knative-sandbox/homebrew-kn-plugins/pull/109

/lgtm
Thanks @nak3! I was too quick with my original PR and had old SHAs. PTAL https://github.com/knative-sandbox/homebrew-kn-plugins/pull/119


Migrate sensor models to functional form

Proposed changes
Depends on #324. This patch migrates sensor models to a functional form. This affords:
beluga::actions::reweight(std::execution::par, sensor(std::move(measurement)))

when reweighting particles.
Type of change

[ ] 🐛 Bugfix (change which fixes an issue)
[x] 🚀 Feature (change which adds functionality)
[ ] 📚 Documentation (change which fixes or extends documentation)

Checklist

[x] Lint and unit tests (if any) pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)
[x] All commits have been signed for DCO

Alright, CI's green, same old same old with codecov.


fix(TDI-40149) : salesforce long running query master

What is the current behavior? (You can also link to an open issue here)
https://jira.talendforge.org/browse/TDI-40149
What is the new behavior?
User can define a timeout  for the salesforce job or disabled it with 0.
Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Pull request test result: success :star:
Sonar results


Update StopWordFilter.php

Fix typo

Awesome thank you @SublimeRaiser! Don't worry about the code style error, it's a bug in PHP 8, but we do need you to sign the CLA before we can merge your changes. If you agree, you can reply with a comment in this thread with the words "I have read the CLA Document and I hereby sign the CLA" and our bot should pick it up.
https://github.com/RubixML/ML/blob/master/CONTRIBUTING.md
https://github.com/RubixML/Signatures/blob/master/CLA.md
Thank you again!

Awesome thank you @SublimeRaiser! Don't worry about the code style error, it's a bug in PHP 8, but we do need you to sign the CLA before we can merge your changes. If you agree, you can reply with a comment in this thread with the words "I have read the CLA Document and I hereby sign the CLA" and our bot should pick it up.
https://github.com/RubixML/ML/blob/master/CONTRIBUTING.md
https://github.com/RubixML/Signatures/blob/master/CLA.md
Thank you again!

I have read the CLA Document and I hereby sign the CLA

I have read the CLA Document and I hereby sign the CLA

Thank you @SublimeRaiser
Feel free to join us in  our Telegram channel if you're not already in there https://t.me/RubixML

Thank you @SublimeRaiser
Feel free to join us in  our Telegram channel if you're not already in there https://t.me/RubixML


Converted Hugo website to Vuejs pwa

This series of commits provides more structure and modularity to the current code base by utilizing the vuejs cli and framework. Most if not all of the current functionality is preserved across this change.
By separating the html and js into components using vuejs it not only invites developers who are familiar with the npm and webpack build ecosystem, but it also makes the codebase more approachable to front end developers whether they have vuejs experience or not because of vue's small learning curve.
In order to deploy this to netlify the publish directory would have to be set to dist instead of public.

Also, the reason the Makefile and the custom.js are in the docs directory is as a reminder to ask the following questions.

With the change to npm, what the purpose of the makefile? If it is still useful then I'm sure we can refactor it to work with the new build system instead of Hugo.
As for the custom.js, I left it in the docs dir to ask if we still need to dynamically determine the circuit edge stroke-dasharray and stroke-dashoffset or if, since the svg is already made, a set value would suffice.

Do you have webpage test results showing the performance impact of making these changes?


Add basic SEO and SoMe information for Topic pages

Acceptance criteria

Can override the page title with a <title> field SEO. If the title field is empty, it should fallback to using the title
Can add a meta description with guidelines for length in Sanity
Can add an image for open graph. If no image is added, a reasonable fallback is provided (most probably by the banner image? To be decided)
The webpage should add <title>, meta description and og: tags for SoMe the web page

In order to ensure good quality we should (Keep the ones we need for this US)

[ ] Perform integration testing
[ ] Perform functional testing
[ ] Extra focus on usability
[ ] Test for portability
[ ] Test for accessibility
[ ] Create automated acceptance test/smoke test
[ ] Perform performance and load test
[ ] Perform user testing
[ ] Perform security testing

In Sanity I have added a fieldset for seo and some in the same way as with the news articles.

Note: The bug with the missing validation trigger for input fields in collapsed fieldsets is still present. Still waiting for Sanity here.
There is no fallback for missing og:image as that depends on the banner (#274) yet.
Other than that, this is ready for testing.
Example before this task/facebook does its best

Example after this task

@petterwildhagen @nicholas-haggerty  This is ready for testing :)

[x] Can override the page title with a  field SEO. If the title field is empty, it should fallback to using the title
[x] Can add a meta description with guidelines for length in Sanity
[x] Can add an image for open graph. If no image is added, a reasonable fallback is provided (most probably by the banner image? To be decided)
[x] The webpage should add , meta description and og: tags for SoMe the web page

[x] Can override the page title with a  field SEO. If the title field is empty, it should fallback to using the title
[x] Can add a meta description with guidelines for length in Sanity
[x] Can add an image for open graph. If no image is added, a reasonable fallback is provided (most probably by the banner image? To be decided) - @wenche Yes the top banner should be the fallback/default if an image is not placed in 'open graph'
[x] The webpage should add , meta description and og: tags for SoMe the web page


parameters lib convert to c++

Once the parameters library is c++ we can make the generated parameters static constexpr. Then we can drop the extra parameters type array (~ 4KB of flash) and explore a few other interesting things.
It would also be nice to get parameters_shmem merged back into the main parameters. The entire SHMEM side is a tangled mess of circular dependencies at various levels, but could be sorted out. Proper dependencies would help with the constant snapdragon breakage.
TODO:

general pixhawk testing
flash based param testing (aero, tap, etc)
snapdragon testing

Rebased.


FIX Safe merging with LoHa and LoKr

There was a small bug when merging the LoHa and LoKr tuners with safe_merge=True due to a missing clone call. This is now fixed.
Furthermore, the test coverage for merging with LoHa and LoKr has been extended, as there were a few tests where these methods were excluded unnecessarily.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Create TLS secret from Azure Key Vault Certificate

This is my Externalsecret manifest to extract tls.crt and tls.key from Azure Key Vault Certificate.
apiVersion: external-secrets.io/v1beta1
kind: ExternalSecret
metadata:
  name: tls-secret
  namespace: ingress
spec:
  refreshInterval: 1h
  secretStoreRef:
    kind: SecretStore
    name: azure-backend
  target:
    template:
      type: kubernetes.io/tls
      engineVersion: v2
      data:
        tls.crt: "{{ .tls  | b64dec | pkcs12cert }}"
        tls.key: "{{ .tls  | b64dec | pkcs12key }}"
  data:
    - remoteRef:
        key: secret/my-certificate-io
      secretKey: tls

When I use this manifest, it retrieves a certificate chain for tls.crt. This chain includes the leaf, intermediate, and root certificates, resulting in a bundle of certificates, like below.
-----BEGIN PRIVATE KEY-----
MIIEvQIBADANBgkqhkiG9w0BAQEFAASCBKcwggSjAgEAAoIBAQCvxGZOW4IXvGlh
 . . .
m8JCpbJXDfSSVxKHgK1Siw4K6pnTsIA2e/Z+Ha2fvtocERjq7VQMAJFaIZSTKo9Q
JwwY+vj0yxWjyzHUzZB33tg=
-----END PRIVATE KEY-----
-----BEGIN CERTIFICATE-----
MIIDMDCCAhigAwIBAgIQabPaXuZCQaCg+eQAVptGGDANBgkqhkiG9w0BAQsFADAV
 . . .
NtFUGA95RGN9s+pl6XY0YARPHf5O76ErC1OZtDTR5RdyQfcM+94gYZsexsXl0aQO
9YD3Wg==
-----END CERTIFICATE-----
-----BEGIN CERTIFICATE-----
MIIDMDCCAhigAwIBAgIQabPaXuZCQaCg+eQAVptGGDANBgkqhkiG9w0BAQsFADAV
 . . .
NtFUGA95RGN9s+pl6XY0YARPHf5O76ErC1OZtDTR5RdyQfcM+94gYZsexsXl0aQO
9YD3Wg==
-----END CERTIFICATE-----

However, I specifically require only the leaf certificate, which is the first one in the chain. How can I extract and use only the leaf certificate, excluding the intermediates and root certificates?
reference:
https://external-secrets.io/latest/provider/azure-key-vault/#creating-external-secret
https://external-secrets.io/latest/guides/templating/#extract-keys-and-certificates-from-pkcs12-archive

Hmm, this will be an ugly solution, but what you can try is to use one of the regex* functions provided by sprig (e.g. regexFind) inside your template. That way you can match by the PEM headers -----BEGIN CERTIFICATE----- and -----END CERTIFICATE-----. However, if the order is wrong this will break.
We do not have a function to extract the leaf certificate out of a cert chain. This would be something we could add to the template function library.

Going to give it a good-first-issue label for someone to pick it up.

I am not sure if this helps. But I have uploaded the pem cert as certificate in azure key vault and then called it via cert/xx  remote ref which pulled the leaf cert only.


Changes for Line 53

Edit to Line 53

@e0i

Hi Johan. I'm quite new to this. Isee you have approved the pull request. What happens next?

@nandans-msft, I just did a copy editor review. It will now be reviewed by the document owner/manager and if approved, it will be merged.

Perfect. Thanks Johan.
From: JohanFreelancer9 @.>
Sent: 28 September 2021 09:02
To: MicrosoftDocs/windows-itpro-docs @.>
Cc: Nandan Sheth @.>; Mention @.>
Subject: Re: [MicrosoftDocs/windows-itpro-docs] Changes for Line 53 (#9980)
@nandans-msfthttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnandans-msft&data=04|01|Nandan.Sheth%40microsoft.com|d668d53b27e04033cdf408d9825633b0|72f988bf86f141af91ab2d7cd011db47|1|0|637684129043344933|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Lbwu%2B2x7bA6KNKx0lsIVa8x0KVEsASxPQTg8br9ie9Q%3D&reserved=0, I just did a copy editor review. It will now be reviewed by the document owner/manager and if approved, it will be merged.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindows-itpro-docs%2Fpull%2F9980%23issuecomment-928951382&data=04|01|Nandan.Sheth%40microsoft.com|d668d53b27e04033cdf408d9825633b0|72f988bf86f141af91ab2d7cd011db47|1|0|637684129043344933|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=D3yAV7lNW2UEqeGQ257mmm0e%2FmodMXccY6NBqCl0VwU%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAV2CTXP2IN3WPY7T5SKZMGTUEFY6HANCNFSM5E3UQMJA&data=04|01|Nandan.Sheth%40microsoft.com|d668d53b27e04033cdf408d9825633b0|72f988bf86f141af91ab2d7cd011db47|1|0|637684129043354888|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Nu0plGOlQKLZf1EFsSUQfSQQL4Vtdic0YBxb0M0%2B6ZM%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|Nandan.Sheth%40microsoft.com|d668d53b27e04033cdf408d9825633b0|72f988bf86f141af91ab2d7cd011db47|1|0|637684129043354888|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=6x3xvZCANahNRjE5QCn%2FTgPx2WfEXG5Zyb5%2F8dE8R6s%3D&reserved=0 or Androidhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|Nandan.Sheth%40microsoft.com|d668d53b27e04033cdf408d9825633b0|72f988bf86f141af91ab2d7cd011db47|1|0|637684129043364845|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=nE%2BlKFCVCpD2mmY5eFIiVATU82Lva02twGZimEQhltY%3D&reserved=0.


feat: add '?' key in order to show help informations

Just like lazygit, when user press '?' key, it prompt a help interface which shows the keybindings,. so I add some code the impliment the similar feature in lazysql. And after testing,  it works in lazysql now .
And also, I modify the README.md to let users know this feature.
Well, I review some of the original code and rewrite some code to make it more readable, but I did't affect any of the major function of lazysql, so it is safe to merge into the main branch

Well?  Could you please leave some advice about what can I do to improve the code quality of closed PR, and why not want to add the feature, and what new features that I can work on.


How about the accuracy of triplet loss training?

I had try to train the model by myself.
I download CASIA-WebFace which had been celan and then follow the instructions in wiki "Triplet loss training" to train the mode.
But it seems the accuracy on LFW, is %88, is it right?

Obviously, it is very hard to train triplet loss nn.  Until now, except for the original google facenet, I have not seen any case with very sussuccful triplet loss training.
So I think the performance it is reasonable although I have not trained it.

Hi
I am a neophyte,I don't understand why batch_number exceeds epoch_size.

Yes, I get 89% Accuracy.

@tonybaigang please look at the wikis, David mentions this


Fix Django 1.7 support

Fix #245

Coverage increased (+0.59%) to 51.56% when pulling aa6937ab7aeb746521eb31c22566929c85bb988e on yakky:django_app into b60e4c3128e023f9ea0de9296aa52d0edb9c5b74 on divio:master.

Coverage increased (+0.59%) to 51.56% when pulling aa6937ab7aeb746521eb31c22566929c85bb988e on yakky:django_app into b60e4c3128e023f9ea0de9296aa52d0edb9c5b74 on divio:master.

Coverage increased (+1.07%) to 52.04% when pulling b1aaeb498aa1603d6599d26d5473e2014b553ade on yakky:django_app into b60e4c3128e023f9ea0de9296aa52d0edb9c5b74 on divio:master.

Coverage increased (+1.07%) to 52.04% when pulling b1aaeb498aa1603d6599d26d5473e2014b553ade on yakky:django_app into b60e4c3128e023f9ea0de9296aa52d0edb9c5b74 on divio:master.

Coverage increased (+1.07%) to 52.04% when pulling b1aaeb498aa1603d6599d26d5473e2014b553ade on yakky:django_app into b60e4c3128e023f9ea0de9296aa52d0edb9c5b74 on divio:master.

Coverage increased (+1.07%) to 52.04% when pulling b1aaeb498aa1603d6599d26d5473e2014b553ade on yakky:django_app into b60e4c3128e023f9ea0de9296aa52d0edb9c5b74 on divio:master.


[Bug]: 0.11.4 continues to exist websockets.exceptions.ConnectionClosedOK: received 1000 (OK); then sent 1000 (OK)

Is there an existing issue for this?

[X] I have searched the existing issues and checked the recent builds/commits

What happened?
Traceback (most recent call last):
File "/usr/local/lib/python3.10/site-packages/uvicorn/protocols/http/h11_impl.py", line 428, in run_asgi
result = await app(  # type: ignore[func-returns-value]
File "/usr/local/lib/python3.10/site-packages/uvicorn/middleware/proxy_headers.py", line 78, in call
return await self.app(scope, receive, send)
File "/usr/local/lib/python3.10/site-packages/fastapi/applications.py", line 276, in call
await super().call(scope, receive, send)
File "/usr/local/lib/python3.10/site-packages/starlette/applications.py", line 122, in call
await self.middleware_stack(scope, receive, send)
File "/usr/local/lib/python3.10/site-packages/starlette/middleware/errors.py", line 184, in call
raise exc
File "/usr/local/lib/python3.10/site-packages/starlette/middleware/errors.py", line 162, in call
await self.app(scope, receive, _send)
File "/usr/local/lib/python3.10/site-packages/starlette/middleware/exceptions.py", line 79, in call
raise exc
File "/usr/local/lib/python3.10/site-packages/starlette/middleware/exceptions.py", line 68, in call
await self.app(scope, receive, sender)
File "/usr/local/lib/python3.10/site-packages/fastapi/middleware/asyncexitstack.py", line 21, in call
raise e
File "/usr/local/lib/python3.10/site-packages/fastapi/middleware/asyncexitstack.py", line 18, in call
await self.app(scope, receive, send)
File "/usr/local/lib/python3.10/site-packages/starlette/routing.py", line 718, in call
await route.handle(scope, receive, send)
File "/usr/local/lib/python3.10/site-packages/starlette/routing.py", line 276, in handle
await self.app(scope, receive, send)
File "/usr/local/lib/python3.10/site-packages/starlette/routing.py", line 69, in app
await response(scope, receive, send)
File "/usr/local/lib/python3.10/site-packages/starlette/responses.py", line 270, in call
async with anyio.create_task_group() as task_group:
File "/usr/local/lib/python3.10/site-packages/anyio/_backends/_asyncio.py", line 597, in aexit
raise exceptions[0]
File "/usr/local/lib/python3.10/site-packages/starlette/responses.py", line 273, in wrap
await func()
File "/usr/local/lib/python3.10/site-packages/starlette/responses.py", line 262, in stream_response
async for chunk in self.body_iterator:
File "/data/web/xiezuo.gptlala.com/aiboxs/api/common.py", line 85, in generate
async for c in massage:
File "/usr/local/lib/python3.10/site-packages/EdgeGPT/EdgeGPT.py", line 168, in ask_stream
async for response in self.chat_hub.ask_stream(
File "/usr/local/lib/python3.10/site-packages/EdgeGPT/chathub.py", line 128, in ask_stream
msg = await wss.recv()
File "/usr/local/lib/python3.10/site-packages/websockets/legacy/protocol.py", line 568, in recv
await self.ensure_open()
File "/usr/local/lib/python3.10/site-packages/websockets/legacy/protocol.py", line 944, in ensure_open
raise self.connection_closed_exc()
websockets.exceptions.ConnectionClosedOK: received 1000 (OK); then sent 1000 (OK)
Steps to reproduce the problem

Go to ....
Press ....
...

What should have happened?
websockets.exceptions.ConnectionClosedOK: received 1000 (OK); then sent 1000 (OK)
Version where the problem happens
python3.10
What Python version are you running this with?
python3.10
What is your operating system ?
No response
Command Line Arguments
websockets.exceptions.ConnectionClosedOK: received 1000 (OK); then sent 1000 (OK)

Console logs
websockets.exceptions.ConnectionClosedOK: received 1000 (OK); then sent 1000 (OK)

Additional information
websockets.exceptions.ConnectionClosedOK: received 1000 (OK); then sent 1000 (OK)
[root@gptlala xiezuo.gptlala.com]# pip list|grep Ed
EdgeGPT                      0.11.4

Traceback (most recent call last):
File "/usr/local/lib/python3.10/site-packages/uvicorn/protocols/http/h11_impl.py", line 428, in run_asgi
result = await app(  # type: ignore[func-returns-value]
File "/usr/local/lib/python3.10/site-packages/uvicorn/middleware/proxy_headers.py", line 78, in call
return await self.app(scope, receive, send)
File "/usr/local/lib/python3.10/site-packages/fastapi/applications.py", line 276, in call
await super().call(scope, receive, send)
File "/usr/local/lib/python3.10/site-packages/starlette/applications.py", line 122, in call
await self.middleware_stack(scope, receive, send)
File "/usr/local/lib/python3.10/site-packages/starlette/middleware/errors.py", line 184, in call
raise exc
File "/usr/local/lib/python3.10/site-packages/starlette/middleware/errors.py", line 162, in call
await self.app(scope, receive, _send)
File "/usr/local/lib/python3.10/site-packages/starlette/middleware/exceptions.py", line 79, in call
raise exc
File "/usr/local/lib/python3.10/site-packages/starlette/middleware/exceptions.py", line 68, in call
await self.app(scope, receive, sender)
File "/usr/local/lib/python3.10/site-packages/fastapi/middleware/asyncexitstack.py", line 21, in call
raise e
File "/usr/local/lib/python3.10/site-packages/fastapi/middleware/asyncexitstack.py", line 18, in call
await self.app(scope, receive, send)
File "/usr/local/lib/python3.10/site-packages/starlette/routing.py", line 718, in call
await route.handle(scope, receive, send)
File "/usr/local/lib/python3.10/site-packages/starlette/routing.py", line 276, in handle
await self.app(scope, receive, send)
File "/usr/local/lib/python3.10/site-packages/starlette/routing.py", line 69, in app
await response(scope, receive, send)
File "/usr/local/lib/python3.10/site-packages/starlette/responses.py", line 270, in call
async with anyio.create_task_group() as task_group:
File "/usr/local/lib/python3.10/site-packages/anyio/_backends/_asyncio.py", line 597, in aexit
raise exceptions[0]
File "/usr/local/lib/python3.10/site-packages/starlette/responses.py", line 273, in wrap
await func()
File "/usr/local/lib/python3.10/site-packages/starlette/responses.py", line 262, in stream_response
async for chunk in self.body_iterator:
File "/data/web/xiezuo.gptlala.com/aiboxs/api/common.py", line 85, in generate
async for c in massage:
File "/usr/local/lib/python3.10/site-packages/EdgeGPT/EdgeGPT.py", line 168, in ask_stream
async for response in self.chat_hub.ask_stream(
File "/usr/local/lib/python3.10/site-packages/EdgeGPT/chathub.py", line 198, in ask_stream
raise Exception(
Exception: InternalError: Unhandled Dependency Failure: DeepLeo

Can't replicate. Please send minimum reproducible code

    bot = await Chatbot.create(cookies=cookies, proxy=proxy)
    massage = bot.ask_stream(prompt=info.messages, conversation_style=info.conversation_style, raw=True, )
    async def generate():
        m_n=n=""
        async for c in massage:
            res = {}
            if c[1]["type"] == 1:
                if "messages" in c[1]["arguments"][0]:
                    if "text" in c[1]["arguments"][0]["messages"][0]:
                        newmsg = c[1]["arguments"][0]["messages"][0]["text"]
                        n=newmsg.replace(m_n,'')
                        m_n=newmsg
                        res = {"content": n,
                        "finish_reason": c[0]}
                    else:
                        res = {"content": c[1]["arguments"][0]["messages"],
                        "finish_reason": c[0]}
            elif c[1]["type"] == 2:
                res = c
            yield (json.dumps(res))
            if c[0] == True:
                return
    await bot.close()

exception websockets.exceptions.ConnectionClosedOK(rcvd, sent, rcvd_then_sent=None)[source]
Like ConnectionClosed, when the connection terminated properly.
A close code with code 1000 (OK) or 1001 (going away) or without a code was received and sent.

but i don't konw why bing closed connection?
Judging from the results, the entire conversation is not completely output, and the connection is disconnected。

except can raise out?

Updating to 0.11.8 version should work

I have the same issue after updating to  0.11.8
  File "C:\12333\Windows-quickstart-go-cqhttp-refs.tags.v2.5.2\python3.11\Lib\site-packages\EdgeGPT\EdgeGPT.py", line 185, in ask_stream
    async for response in self.chat_hub.ask_stream(
              │           │    │        └ <function ChatHub.ask_stream at 0x000002BB6DE13880>
              │           │    └ <EdgeGPT.chathub.ChatHub object at 0x000002BB6E352B50>
              │           └ <EdgeGPT.EdgeGPT.Chatbot object at 0x000002BB6E361910>
              └ (False, 'Searching the web for: `今天中国深市和沪市的股票`\n```json\n{"web_search_results":[{"index":"1","title":"市场数据 - 深交所主页","snippets...
  File "C:\12333\Windows-quickstart-go-cqhttp-refs.tags.v2.5.2\python3.11\Lib\site-packages\EdgeGPT\chathub.py", line 140, in ask_stream
    await wss.send(append_identifier({"type": 6}))
          │   │    └ <function append_identifier at 0x000002BB6DE13740>
          │   └ <function WebSocketCommonProtocol.send at 0x000002BB6DCA87C0>
          └ <websockets.legacy.client.WebSocketClientProtocol object at 0x000002BB6E397F10>
  File "C:\12333\Windows-quickstart-go-cqhttp-refs.tags.v2.5.2\python3.11\Lib\site-packages\websockets\legacy\protocol.py", line 635, in send
    await self.ensure_open()
          │    └ <function WebSocketCommonProtocol.ensure_open at 0x000002BB6DCA8B80>
          └ <websockets.legacy.client.WebSocketClientProtocol object at 0x000002BB6E397F10>
  File "C:\12333\Windows-quickstart-go-cqhttp-refs.tags.v2.5.2\python3.11\Lib\site-packages\websockets\legacy\protocol.py", line 944, in ensure_open
    raise self.connection_closed_exc()
          │    └ <function WebSocketCommonProtocol.connection_closed_exc at 0x000002BB6DCA8AE0>
          └ <websockets.legacy.client.WebSocketClientProtocol object at 0x000002BB6E397F10>

websockets.exceptions.ConnectionClosedOK: received 1000 (OK); then sent 1000 (OK)

I ran into this issue before upgrading to version 0.11.8, is the network latency too high？


text overlapped by plotted data

Having multiple issues with this figure in two different matlab releases
https://dl.dropboxusercontent.com/u/29498615/untitled1.fig
plot was made with pcolor, text written afterwards on top
in r2015a:
The text, which is visible in the figure window, becomes hidden and overlapped by by the plotted data when written to pdf.
print command also suffers this issue
in r2013b:
this plot gets rasterized using export_fig, but not with print command
the text does is not hidden with this release of matlab, for either print or export_fig

Just realized that in r2013b, the single horizontal line gets hidden by the data in pcolor using the print command, but is visible in the figure.

@falkben - export_fig uses print internally so if the problems appear using print then you should report them as bugs to MathWorks (support@mathworks.com), and not here as export_fig issues.
export_fig does try to overcome many such internal bugs of print by post-processing the output file, but this specific ones don't seem to me to be important enough to warrant such extraordinary measures, that should really be reserved for high-impact print bugs.


Handle resumes and reconnects correctly

Original issue: https://github.com/Frederikam/FredBoat/issues/403

Tested this with latest LL and FB code, resumes are not handled optimally.
The bot always leaves the voicechat, music is paused, unpaused on rejoin.
Sometimes it does not rejoin, and therefore does not continue the playback.
Reconnects break it worse: Never rejoins the voicechat, but always keeps playing.

Instead, these events should be handled the same way they are handled when running without LL:
Resumes are, if at all, noticed by a short audio cut out.
On Reconnects the VoiceChannel is left, and upon restoration of the connection, rejoined and the playback continued.

It is odd that it isn't rejoining, as that as explicitly handled on resume and reconnect by the Lavalink class.

This should be fixed in V2


Syntax error

The commands as listed result in the error, The resource namespace ''Microsoft.PolicyInsights'' is invalid. This is caused by the use of single quotes. Removing the single quotes resolves the error. This is with azure-cli (2.0.32)

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

ID: cf5513d0-6c0e-8235-8a19-48ad15a687d0
Version Independent ID: 9b5ab97a-2599-6320-bdd7-a08f29cf49bf
Content: Use the Azure CLI to create a policy assignment to identify non-compliant resources in your Azure environment
Content Source: articles/azure-policy/assign-policy-definition-cli.md
Service: azure-policy
Product: unspecified
GitHub Login: @DCtheGeek
Microsoft Alias: dacoulte

@sancheza Thanks for bringing this to our attention. We are actively investigating and will get back to you soon.

@sancheza , A fix has been deployed to resolve the syntax error. will send a final update once the fix is live.

@sancheza, to clarify, the issue is not with the single quotes but rather an issue was caused by a malformed quote due to a copy/paste prior to publishing the doc. Again this is being fixed along with some other minor updates and the fix will be completely pushed out soon. @DCtheGeek or myself will update this thread once all the fixes are live. Thanks for your patience.

@sancheza The page has now been updated to fix the code issue (which was actually with the dashes, not the quotes) and ensured the rest of the examples are correct.  Thank you again for the feedback!  #please-close

@sancheza We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Remove keystone-all from alternatives

The keystone-all binary doesn't exist in the latest version of keystone
so linking the alternative fails. The implementation has moved on to
support uwsgi deployment instead so there's no reason to keep it around.

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

can i have an "add to whitelist"?

okay to test

ok to test


Make something about python -m pylxd

Would be nice to have something like diagnostic info when executed standalone. Maybe some commands.
$ python -m pylxd
...
No module named pylxd.__main__; 'pylxd' is a package and cannot be directly executed

I personally think to try pylxd for controlling remote container, so inspecting if pyxld can connect it from command line would be nice.

I think this is beyond the scope of the current usage of pylxd. I would gladly review a patch that performed this functionality, should you have a use case, but as the current developers don't have a use case, we won't be focusing on writing this code.


Output SRT file format is invalid.

Example result:
0:02:36,720 --> 0:02:37,879
请各单元组
0:02:37,880 --> 0:02:39,599
认真填写工作日志

Needs indexes
Hours require leading zeroes

Should be:
1
00:02:36,720 --> 00:02:37,879
请各单元组
2
00:02:37,880 --> 00:02:39,599
认真填写工作日志

Mark, I will fix it later.

I have fixed it

I missed it.
Now , I have fixed them all.
Thansk for your reminder.

Thanks! It works for me now


IDVA6-1675 Remove code smells indicated by SonarQube

Link to Jira:
https://companieshouse.atlassian.net/browse/IDVA6-1675
Changes:

Code smells indicated by SonarQube removed

CI: No security warnings found


Intermittent error while generating authorization code

Summary
We are currently using 0.8.3 with Cassandra data store and we are hitting the intermittent issue generating a authorization _code.
We looked at the code and we could not find any reasons why it could fails. The authz code is not passed from access.lua, it is supposed to generate a random code, but is returning nil and the insert is failing because of it and looking at the code there is no way it can return a nil value (It is supposed to return a “UUID without hyphens). Are we missing something. Any pointers to debug it further would help (We are also looking at option to upgrade to 0.9.9 but need a quick fix before we do the upgrade)
Steps To Reproduce

Try a simple CURL command like this below
curl https://localhost:8443/oauth2/authorize
    --data "client_id=XXX"
    --data "response_type=code"
    --data "scope=all"
    --data "provision_key=XXX"
    --data "authenticated_userid=XXX:4000002"
--header "Host: test.com"
--insecure

Additional Details & Logs
2017/02/08 11:35:37 [error] 160#0: *414 lua entry thread aborted: runtime error: /usr/local/share/lua/5.1/kong/plugins/oauth2/access.lua:176: attempt to index local 'authorization_code' (a nil value)
stack traceback:
coroutine 0:
/usr/local/share/lua/5.1/kong/plugins/oauth2/access.lua: in function 'authorize'
/usr/local/share/lua/5.1/kong/plugins/oauth2/access.lua:406: in function 'execute'
/usr/local/share/lua/5.1/kong/plugins/oauth2/handler.lua:12: in function 'access'
/usr/local/share/lua/5.1/kong.lua:188: in function 'access'

Kong version ($ kong version)
Kong debug-level startup logs ($ kong start --vv)
Kong error logs (<KONG_PREFIX>/logs/error.log)
Kong configuration (registered APIs/Plugins & configuration file)
Operating System Ubuntu Linux

@chetankmalur do you see this behavior when running on Kong 0.10.x? 0.8.3 is far out of date; we have made a number of significant refactors and improvements since then.

no recent activity


lnav: 0.9.0 -> 0.10.0

Motivation for this change
ChangeLog: https://github.com/tstack/lnav/releases/tag/v0.10.0
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase 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.

Result of nixpkgs-review pr 132873 at 562749f4 run on aarch64-linux 1

1 package failed to build:

lnav

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.


Error: connect ECONNREFUSED <IP_ADDRESS>:80

After update to latest vs code stable version (1.19.0), Live Server stop working and give the following error when open browser:
Error: connect ECONNREFUSED <IP_ADDRESS>:80
at Object.exports._errnoException (util.js:1050:11)
at exports._exceptionWithHostPort (util.js:1073:20)
at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1097:14)

Please help me. Thanks

Are you sure Port 80 is available? Have you tried to change the port? By default it opens port of 5500.

As far as I seen, this issue is solved for now. If not feel free to tag me and I will reopen it.

Hi,I'm just encountering the same Error in vscode's extension(jupyter),NOT this extension, and I finally solved this problem.You can reference the follow steps i used:

Reinstall the extension
open the 'hosts’ file,which located in ‘C:\Windows\System32\drivers\etc’,and add a sentence like this:
<IP_ADDRESS> http://localhost
then save it .

Thanks.

hey, I am facing the same issue in VS code, can you please specify how to open the "hosts" file? I am on windows.

The host file can be opened with notepad.
This is the file content that showed for me.

Hi,I'm just encountering the same Error in vscode's extension(jupyter),NOT this extension, and I finally solved this problem.You can reference the follow steps i used:

Reinstall the extension
open the 'hosts’ file,which located in ‘C:\Windows\System32\drivers\etc’,and add a sentence like this:
<IP_ADDRESS> http://localhost
then save it .

Thanks.

hey, I am facing the same issue in VS code, can you please specify how to open the "hosts" file? I am on windows.

You can simply right click and open it with  notepad

Hi, you can simply copy the hosts file to your desktop, edit it and update it. Then copy it from the desktop to the original location, overwriting the source file and thus circumventing the file permissions issue.

I had seen all the process to solve the problem. But cannot be able to solve the problem.


Fix initializing required fields with proto3 JSON null values

proto3 JSON spec says this about null values:

If a value is missing in the JSON-encoded data or if its value is
null, it will be interpreted as the appropriate default value when
parsed into a protocol buffer

Normally empty field set represents the default message, but for that representation to work with proto2 required fields, serializers need to check for required fields in the BuilderInfo and for fields that are not in the field set serialize the default value for the field.
Currently we don't do this, which results in generating messages with missing required fields. See #760 for a repro.
Checking missing required fields in serializers should fix the issue, but as a simpler fix, this PR deserializes proto3 JSON null values to field sets with required fields set to defaults. This way serializers do not need to be tweaked.
Fixes #760

I asked protobuf team about this fix. It turns out we have an internal version of Dart protobuf spec that says this about the proto3 JSON format:

When parsing, a “null” value is accepted for all field types. It will be treated as if the field has the default value (the field itself would not be set, but its getter will return the default value for proto3).

This version is not in conflict with the public version, but it has extra information. Namely, it says the field should not be set when we see null.
(The reason why the public version doesn't have this information is probably because it doesn't talk about presence yet, and without presence it doesn't make sense to talk about setting fields)
According to this version, this fix is not correct, because the hazzer for the message-field and the message's fields will return true.
I want to avoid changing serializers because (1) it will incur runtime overhead (2) we need to update each serializer and test them separately.
Maybe we can implement some common code shared between all of the serializers for iterating over fields with values?


Exclude calls to functions that are effectively constants (e.g. .dp in Compose)

When your source files include values that aren't completely constant according to Kotlin (because of the function call) they get picked up as untested when there are reasons why they shouldn't be tested. For instance in Compose, values that represent layout dimensions:
private val IconToTextGap = 10.dp

This is picked up as an untested line

Hi,
the basic approach used in Kover is that any custom code written by the user should be used.
If a constant is added, then it must either be read, deleted, or excluded from the report.
Unfortunately, due to the peculiarities of the language, it is impossible to track the use of const val, but the use of the other val is tracked, even if the value is always the same.

Close as not planned


Update Composer dependencies (2023-02-28-00-09)

Visual regression test passed!
View the visual regression test report


Fix DeprecationWarning by replacing imp module with importlib

Currently while running Sabre using following command
python3 ../src/sabre.py

generates the following DepecrationWarning:
../src/sabre.py:30: DeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses
  import imp

This PR fixes above warning by replacing the imp module used in Sabre with importlib module.

Thanks!


fix(scroll): 修复滚动边界判断错误导致无法滚动

👀 PR includes

🐛 Bugfix

[x] Solve the issue and close #1660

📝 Description
背景:
表格由于是虚拟滚动, 滚动条是模拟的, 所以做了边界滚动检测, 用来做滚动链优化

父/子都有滚动条, 子容器滚动时如果没有滚动到边缘, 不触发父容器滚动
如果滚动到边缘, 不再渲染表格, 优化性能

原因:
使用了 已经废弃的 layerX 和 layerY 来做坐标判断, 在最近的 chrome 版本中, layerY 返回的数值不太对 (表格高度一共 400, 但是 layerY 有时拿到的是 600 多), 导致判断错误
解决:
使用 offsetX 和 offsetY 来替代
🖼 Screenshot

Before
After

🔗 Related issue link

ref #1663
🔍 Self-Check before the merge

[ ] Add or update relevant docs.
[ ] Add or update relevant demos.
[ ] Add or update test case.
[ ] Add or update relevant TypeScript definitions.

:tada: This PR is included in version @antv/s2-v1.26.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-react-v1.24.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-v1.26.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-react-v1.24.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add transmision speed and progress

http-file-server is very usfull for me, and it give me much help of my test.
Could you add transmission speed and progress to the server log, then we can see the translation quality of client.
Thank u.

Now, I can get the speed by curl, which is commited by v1.53. It works fine for me.
Uploading files using cURL
$ ./http-file-server -uploads /=/path/to/serve
2020/03/10 22:00:54 serving local path "/path/to/serve" on "/"
2020/03/10 22:00:54 http-file-server listening on ":8080"
curl -LF<EMAIL_ADDRESS>localhost:8080/path/to/upload/to


Potential issue in wrappers/QuASAR/data_to_hifive.py

This may be a issue specific to me, but I had float values in my contact map which caused similar issues in https://github.com/bxlab/hifive/issues/12 where there were no valid read data.
I approximately fixed it by modifying line 83 of wrappers/QuASAR/data_to_hifive.py to account for float values:
count = int(temp[4]) -> count = int(float(temp[4]))
Additionally, perhaps exceptions could be printed for easier debugging purposes?

I had the same issue and this code change worked for me also. I had generated contact maps from hic files using juicer_tools dump, and the resulting scores are float values.

I had the same issue, since I generated the contact maps using juicer_tools. I wish it was better documented, but to be fair, the example provided by the authors shows counts as integer.
I just wish I had discovered this post sooner!
Thank you

Much aprreciated! I sincerely hope @oursu could modify this part, because most HiC contact normalization generate floats, so that future users won't have to spend hours struggling with where the problem is.


upgrade to resteasy 6

resteasy 6 actually supports jakarta-rs while resteasy 4 is for jaxrs but it looks like some changes need to be made to this lib to support resteasy 6 (test failures)
https://mvnrepository.com/artifact/org.jboss.resteasy/resteasy-jackson2-provider

Yeah it'd be good to upgrade -- I had been wondering where dep to old JAXB stuff came from. But unfortunately test setup is rather fragile so simple plug-n-play won't work. I hope someone has time to figure it out, JAX-RS/Jakarta-RS providers are getting bit out of date (would be great if someone from RESTeasy et al project could help actually).


Are files downloaded via Refile.attachment_url cached?

Hi! I’ve been using Refile on a recent project of mine, and this involves using Refile.attachment_url to set the src attribute of HTML5 <audio> tags and then playing them. I’ve noticed that in some browsers, finally starting playback of the audio file will end up making multiple GET requests. Safari will make anywhere between 3 and 6, Firefox tends to make 2, and Chrome makes one. Normally I wouldn’t worry too much about this, but my files are being stored in S3 so bandwidth is a little bit of a concern for me.
I was wondering if those 6 GET requests to Refile’s Sinatra app actually result in 6 requests to S3 or if the Sinatra app does any sort of caching to prevent requesting files over and over.

From the README:

Important: Unlike other file upload solutions, Refile always streams your files through your
application. It cannot generate URLs to your files.  Serving
files through your app takes a lot of resources and you want it to happen rarely.
Setting this up is actually quite simple, you can use the same CDN you would use
for your application's static assets. This blog post
explains how to set this up (bonus: faster static assets!). Once you've set this
up, simply configure Refile to use your CDN:
Refile.host = "//your-dist-url.cloudfront.net"

Using a protocol-relative URL for Refile.host is recommended.

Ah sorry, I didn't catch that first sentence in that README section. Thanks!


Remove delete

Picked up @haacked's 'remove-delete' branch and finished it up.  You can now list/unlist each version of a package.  You cannot edit the package metadata or actually delete a package through the site.
There were a few bug fixes along the way with how the latest package is determined and displayed.

This is for Issue #136, wherein the ability to delete a package needs to be removed.

Ok, I'm done reviewing. There were a couple things I think should be changed. Otherwise, looks good!

Thanks for the thorough review.  I've pushed some changes that address the feedback.

Ship it!


Books with images stored not in Google Cloud Storage could not be del…

…eted.  Fix that bug.

LGTM


Add ability to show only notebooks in a folder

Use CSS selectors to hide/show regular files.

On second thought, I'd prefer not to do this.


Verify doc examples

Motivation for this change
Test the function examples actually work.
Testing steps:

enter doc/
nix-shell
make doc-tests

It outputs something like:
$ make doc-tests
cd ./functions/library; python3 ./test.py
function-library-lib.attrset.attrByPath-example-value-exists: pass
function-library-lib.attrset.attrByPath-example-default-value: pass
function-library-lib.attrsets.recursiveUpdateUntil-example: pass
function-library-lib.attrsets.recursiveUpdate-example: pass
function-library-lib.attrsets.matchAttrs-example-matching-pattern-deep: pass
5 passed, 0 failed

For each example without a testable program, it prints:
Skipping example function-library-lib.attrsets.zipAttrs-example: no <programlisting role="input">

If a test fails, it outputs instructions on how to test only the failing examples:
function-library-lib.attrsets.recursiveUpdateUntil-example: pass
function-library-lib.attrsets.recursiveUpdate-example: pass
--------  Doc test failure  --------
ID: function-library-lib.attrsets.matchAttrs-example-matching-pattern-shallow
Executing:

      {
        actual = (let lib = import /home/grahamc/projects/nixpkgs/lib; in (

lib.attrsets.matchAttrs
  { cpu = {}; }
  { cpu = { bits = 64; }; }

        ));
        expect = (

  false

        );
      }

Expecting:

  false

Found:
True
----
--------  Doc test failure  --------
ID: function-library-lib.attrsets.matchAttrs-example-matching-pattern-deep
Executing:

      {
        actual = (let lib = import /home/grahamc/projects/nixpkgs/lib; in (

lib.attrsets.matchAttrs
  { host = { cpu = {}; }; }
  { host = { cpu = { bits = 64; }; }; }

        ));
        expect = (
          false
        );
      }

Expecting:
false
Found:
True
----
4 passed, 2 failed
Retry failing examples with:
./functions/library/test.py --lib-dir /home/grahamc/projects/nixpkgs/lib function-library-lib.attrsets.matchAttrs-example-matching-pattern-shallow function-library-lib.attrsets.matchAttrs-example-matching-pattern-deep

These tests will block the Nixpkgs manual from building as well.
The XML isn't super nice to write, but I think isn't too onerous?
Anyway, rendered it looks like this:

I also verified that python3 was already in the build closure.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

Are there any updates on this pull request, please?

@grahamc has this been abandoned?


update the menu items on location search to match the homepage search

Reduces the bundle size by ~ 74kB (the state SVGs are gone)

will implement a different solution, closing!


Missing axiom at KSG sector buildings

Description of the issue
According to Federal Climate Change Act also 1.A.5 is part of KSG sector buildings.
Ideas of solution
Add axiom 'KSG sector buildings' 'has part' 'CRF sector (IPCC 2006): fuel combustion - other sectors'
Workflow checklist

[ ] I discussed the issue with someone else than me before working on a solution
[ ] I already read the latest version of the workflow for this repository
[ ] The goal of this ontology is clear to me

I am aware that

[ ] every entry in the ontology should have a definition
[ ] classes should arise from concepts rather than from words

This was found during last weeks project meeting and I'll implement now.


Add OSX Support to Makefile.

Managed to get pxlart to build on OSX via Homebrew ncurses. The changes in the Makefile will check the operating system using uname -s, and will set the build flags accordingly.

Thanks!


Unable to resolve 'lodash/cloneDeep' import

Not sure if this bug is caused by my setup or how module resolution works in node (it seems like this is an unclear area), so apologies if this is a false report.
But when I try to use @cloudinary/base in my Next.js project, I get an error about failing to resolve lodash/cloneDeep, and then indeed I see that various places import it like lodash/cloneDeep: https://github.com/cloudinary/cloudinary-js-base/search?q=lodash%2FcloneDeep
but in the package dependencies, it is written as lodash.cloneDeep. Should this be switched to import cloneDeep from 'lodash.cloneDeep';?
I monkey-patched the files locally and changing the import statements to lodash.cloneDeep fixed the issue. I also was able to get the package working locally by just installing lodash to my whole project.

Hi,
Would you try and it should resolve this issue.
npm i lodash
Please let me know if this works.
Thanks,
Mo

hey, installing lodash works but it seems like this should be included in the package.json for this package then?

Good idea.  I will pass it on to our Dev team.
Thanks.

@mmmulani thanks for reporting this issue, we've merged a fix and it will be part of the next version:
https://github.com/cloudinary/cloudinary-js-base/commit/41dbf2a691865d353cb2fad2def6fd32a227e8a7


WIP: feat(*): Allow applications to be in maintenance mode

Closes: #3722
Note this is very much a non working wip.
TODO:
[ ] Test the basics, I just wrote the code without even testing if it made syntactical (or semantical) sense. I will try  this later on a computer with more RAM in vagrant
[ ] Add unit tests
[ ] Add documentation for controller API
[ ] Add user documentation
[ ] Change client
[ ] Add custom error/maintenance pages
If someone from the core dev team could tell me this is more or less the right way to implement this feature, let me now

I was more intrested in general remarks

@nathansamson at first glance, I think this is valid as a general approach: set a per-app etcd key that deis-router uses to tailor the HTTP response based on whether maintenance mode is on or not.

Closing, as I tried (but failed) to get a dev environment up & running on my computer (8GB ram).
Even when not running a UI, and do everything from a shell (effectively using almost all ram for nodes), builder never worked properly in my test VM.
I am planning to buy a new computer in the short-term with 16GB (or more) RAM, but for now I'll close the issue...
I might work on it in a few weeks time again, but for now I have no viable option to do so...

@nathansamson most of us deploy Deis on one of the providers. DO's a pretty cheap option for testing changes. You should be able to runthe unit tests on your computer without an issue, but running the entire platform is certainly a resource hog.

After I add documentation + some form of tests I'll reopen


Key metadata

Solves #33

Add method GetKeyMetadata()
Add missing metadata fields to the Key struct

I created another PR #35 against cli-integration
I may close this PR once we merge #35

Replaced by #35


define_connection_info always resetting

I have set up some QEMU VMs with define_connection_info = false, but every plan contains the following, for each vm,
      ~ define_connection_info    = true -> false

I suspect that this value is not saved correctly (either with the proxmox api or in the terraform state)

I think it would help if you'd provide the provider log. But be aware to redact the sensitive information.

Full TRACE level log of a terraform apply with two machines asking to get from define_connection_info = true to false (again, since they should be already in that state): https://gist.github.com/bryanpedini/57d2b98342268081351ff2abe61a8f5a

PS, forgot to mention:
As already stated in #291, this issue's setting constantly changing, means that a new disk is created every single time and substitutes the disk actually currently in use by the VM, not to mention that RAW isn't showing the size in the dashboard.

I am having the same exact issue.


Inserted images in Quill Editor are treated as inline text, causing scrolling issues

Is there an existing issue for this?

[X] I have searched the existing issues

The question
Description:
When inserting an image in the Quill Editor, the image is recognized as a line of text, but its height isn't accounted for in the scrolling behavior of the text field. This results in new text entered after the image not scrolling into view properly. The editor treats the image as a standard line without considering its actual dimensions, leading to text being hidden behind the image.
Expected Behavior:
The editor should account for the image's height and scroll accordingly so that any new text entered after the image is visible and not hidden behind the image.
Actual Behavior:
The editor scroll behavior treats the inserted image as a single line of text, disregarding the image's height. This causes the text typed after the image to be hidden behind it, and the scroll controller does not scroll down to accommodate the new text.
Video:
https://github.com/user-attachments/assets/930fc415-b826-43a1-aed0-fc4c47f6384d

same issue. did you fix it?

same issue on web. did you fix it?

not yet.


DF-229: Replace department letters with descriptions

@TNA-Allan Tests are now fixed, so ready for you to play with now :)


Wrong preprocessor condition used in x509_crt.c for threading

In file x509_crt.c, line 1172, function mbedtls_x509_crt_parse_path()use of a mutex is protected by  the preprocessor condition #if defined(MBEDTLS_THREADING_PTHREAD). All use of concurrency primitives should be protected by #if defined(MBEDTLS_THREADING_C) as the library cannot assume they will always be implemented by pthread support.

ARM Internal Ref: IOTSSL-1102

Fixed in PR's #739, #753 and #754, which have now been merged. This issue can be closed.


fix(native filter): clean deleted parent filter ids

SUMMARY
When a parent filter has been removed, the cascadingParentIds has not updated accordingly.
This regression caused by the redux state mutation migration which remains filterConfigMap in obsolete dependency metadata because the redux action will update state.nativeFilters whereas filterConfigMap references from state.dashboardInfo.metadata.native_filter_configuration
This commit updates the cleanDeletedParents logic to retrieve the updated filterConfigMap data and then pass to the onSave payload in order to sync the latest updates.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
After:
https://github.com/apache/superset/assets/1392866/154684ec-b1da-4814-bd29-cb593ad2c7c7
Before:
https://github.com/apache/superset/assets/1392866/11d11646-3fac-4e06-8d67-01b29f1a5e8a
TESTING INSTRUCTIONS
Create a native filter and links a dependent filter
Hit save and then edit the native filter again
Remove the dependent filter only and then save
mouse over the filter has the dependent filter and see an error
ADDITIONAL INFORMATION

[x] Has associated issue: https://github.com/apache/superset/issues/24729
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Thanks @justinpark for the fix. I pushed a commit which includes a migration to remedy any corrupted dashboards. It would be good to verify said change in your environment.
Note you'll need to rebase your branch (and update the Alembic migration file) as 863adcf72773 is the latest revision on master.

Thanks justinpark for the fix. I pushed a commit which includes a migration to remedy any corrupted dashboards. It would be good to verify said change in your environment.

Confirmed the migration script worked well and the corrupted cascadingParentIds has been cleaned.

FYI @eschutho I wonder if you want to cherry-pick this into 2.1.1. Note there's a DB migration and thus you'll have to handle the version.


Add remember-folds to deprecated packages

Description of the Change
This adds my "remember-folds" package to the deprecated list, as it appears that this functionality works in core as of v1.32.0 (as I could best guess from my analysis).
Following the process shown in "how to mark a package as deprecated", as indicated on the issue on the remember-folds repo, this PR adds it to the deprecated list.
Release Notes

The remember-folds package is now marked deprecated.

This isn't what we intended for the deprecated packages list. It was specifically to deprecate packages that hadn't been updated for the v1.0 release four years ago. Since your package supports v1.0 APIs, it isn't a candidate for this treatment.
My suggestion is to add a notice to the README, publish a new version so that the atom.io page displays that notice, and archive the repo.