Examples app login is not working

Doesn't validate correct EOS account names,   tried cuvialabseos
https://www.bloks.io/account/cuvialabseos

Doesn't validate correct EOS account names, tried cuvialabseos
https://www.bloks.io/account/cuvialabseos

Thanks I will get to this soon, I have been creating a digital signature package for SmartQL.

Doesn't validate correct EOS account names, tried cuvialabseos

I deployed SmartQL@6 API feel free to try and use it.

Your account appears to be there.
I have set up a server that has load feel free to use it.
And example in node.js of how to do this would be.
import fetch from "node-fetch"

fetch('https://eos.relocke.io', {
  method: 'post',
  headers: {
    'content-type': 'application/json'
  },
  body: JSON.stringify({
    query: `{
      account_information{
         get_account(account_name: "cuvialabseos") {
             account_name
             core_liquid_balance
              ram_quota
              net_weight
              cpu_weight
              ram_usage
              last_code_update
              total_resources {
                  owner
                  net_weight
                  cpu_weight
                   ram_bytes
               }
              voter_info {
                owner
                proxy
                staked
              }
          }
      }
}`
  })
})
  .then(res => res.json())
  .then(({ data }) => console.log(data))

If you encounter a bug in the graphiql IDE with LHS not loading just hard reset to fix it.
Let me know how you go.


Disable JFR tests on AIX

Fix #2582
Signed-off-by: Sophia Guo<EMAIL_ADDRESS>

Test: https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk11u/job/jdk11u-aix-ppc64-hotspot_SmokeTests/9/console


Add test for capitalization collations

Need to add new tests related to collations based on capitalization. Related to main issue #1535

might be helpful have an example of what you want to test


solved #431  Added quiz on arrays and results

📥 Pull Request
Description
Successfully created the Quiz on arrays and results  displayed
Fixes #431

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
kindly consider as level2 for work done by me

sir,kindly consider this as level2 for the work and efforts made by me

solved both #431 and #433 consider this as level 2 sir and please allot the marks for 2 commits as i made both of them in same pull request instead of creating 2 pull requesrs

thank you


Integration Test - Release - 2021/12/21 08:17:58

Auto create for integration test. IssueCommandFlow

#pri1

#pri2

Just type '#pri3'

#pri3

#assign PRMergerTest1

#assign NonAllowedUser

The following users don't have valid GitHub IDs, or are not collaborators on this repo: nonalloweduser

#reassign PRMergerTest2

#reassign NonAllowedUser

The following users don't have valid GitHub IDs, or are not collaborators on this repo: nonalloweduser

#please-close

GitHub user PRMergerTest2 has been removed from the MicrosoftDocs organization, so they were automatically removed as an assignee.

#unassign PRMergerTest1

#unassign NonAllowedUser


Latest .NET Core version (.net 7)

See if we can implement .net 8 SDK but use the .net 7 template; making sure we do use the latest.

Managed to download .net8 SDK, however only .net 7 templates are present, you can use beta; but not yet active.
The recommendation is to stay with .net 7 for now.


[3733] Import accredited provider id from DTTP

Context
We sync provider records from DTTP. Previously, we only parsed out a few fields. Now we're about to decommission DTTP we want to store all of the fields returned as there are others we will use.
Changes proposed in this pull request

Add a response column to dttp_providers. This is consistent with the other dttp_ tables.
When we sync providers, store the JSON for each provider in this table.

Guidance to review
Important business

Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?

 [M] Import Accredited Provider Id from DTTP


07/12/2018 01:44 : Create Pull Request for branch prTesting_2018071201 -> master

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


EU domain have no RDAP servers

❯ ~/go/bin/rdap europa.eu
Error: No RDAP servers found for 'europa.eu'

Are there any registry servers for .eu TLDs? I can't find any.
https://search.arin.net/rdap/?query=europa.eu
https://www.verisign.com/en_US/domain-names/registration-data-access-protocol/index.xhtml
https://tld-rdap.verisign.com/eu/v1/domain/europa.eu

It's seems that .eu tld does not officially provide RDAP service: they cannot be found on the Iana bootstrap file https://data.iana.org/rdap/dns.json


[SPARK-19433][ML] Periodic checkout datasets for long ml pipeline

What changes were proposed in this pull request?
For a Pipeline including long stages, the iterative fit and transform cause extremely grown query plans and RDD lineages, it takes longer time to finish the fit and transform.
This patch introduces PeriodicDatasetCheckpointer to do periodic checkout for dataset used in fit and transform.
This introduces new param checkpointInterval to Pipeline and PipelineModel. Once it is set, we will do periodic checkout by PeriodicDatasetCheckpointer.
As there is existing trait HasCheckpointInterval which already defines checkpointInterval param. This patch lets Pipeline and PipelineModel extend HasCheckpointInterval.
Benchmark
Run the following codes locally.
import org.apache.spark.ml.{Pipeline, PipelineStage}
import org.apache.spark.ml.feature.{OneHotEncoder, StringIndexer, VectorAssembler}

spark.sparkContext.setCheckpointDir("/tmp/checkpoint")

val df = (1 to 40).foldLeft(Seq((1, "foo"), (2, "bar"), (3, "baz")).toDF("id", "x0"))((df, i) => df.withColumn(s"x$i", $"x0"))

val indexers = df.columns.tail.map(c => new StringIndexer()
  .setInputCol(c)
  .setOutputCol(s"${c}_indexed")
  .setHandleInvalid("skip"))

val encoders = indexers.map(indexer => new OneHotEncoder()
  .setInputCol(indexer.getOutputCol)
  .setOutputCol(s"${indexer.getOutputCol}_encoded")
  .setDropLast(true))

val stages: Array[PipelineStage] = indexers ++ encoders
val pipeline = new Pipeline().setStages(stages)
pipeline.setCheckpointInterval(5)  // only run this line after applying this patch

val startTime = System.nanoTime
pipeline.fit(df).transform(df).show
val runningTime = System.nanoTime - startTime

Before this patch: 1786001 ms
After this patch: 69013 ms
This issue is originally reported at http://apache-spark-developers-list.1001551.n3.nabble.com/SQL-ML-Pipeline-performance-regression-between-1-6-and-2-x-tc20803.html
How was this patch tested?
Jenkins tests.
Please review http://spark.apache.org/contributing.html before opening a pull request.

Test build #72274 has started for PR 16775 at commit 5ed5c2a.

cc @mengxr @jkbradley @liancheng

also cc @MLnick

Test build #72274 has finished for PR 16775 at commit 5ed5c2a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #72275 has started for PR 16775 at commit 7a1b300.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72274/
Test PASSed.

Test build #72276 has started for PR 16775 at commit 32c90dd.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72276/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72275/
Test FAILed.

Merged build finished. Test FAILed.

retest this please.

Test build #72277 has started for PR 16775 at commit 32c90dd.

Test build #72277 has finished for PR 16775 at commit 32c90dd.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72277/
Test PASSed.

Wouldn't it better to Vectorize StringIndexer and OneHotEncoder? Like for instance .na.fill or .na.replace operate over the whole data set at once instead of running it in a loop? I feel like even with this patch this isn't scalable on lets say 1MM covariates (unless I'm missing something) as this process isn't executed at all nodes/cores at the same time.

StringIndexer and OneHotEncoder are just used as example here. The concept is to have a pipeline with enough long stages.

ping @mengxr @jkbradley @liancheng @MLnick May you take a look at this? Thanks.

@viirya I believe this PR meshes with the refactoring and application to pregel GraphX algorithms in #15125. Basically, it moves the periodic checkpointing code from mllib into core and uses it in GraphX to checkpoint long lineages. This is essential to scale GraphX to huge graphs, as described in my comment in the PR, and solves a very real problem for us. Can you take a look at that PR?

I think we can solve this issue by tackling the codes in SQL. So close it for now.


The sync fails if we have a list with special characters in the name.

I have a list named "Cabeça" and i had to remove the "ç" from the name in order to successful sync the list.

Hi @rfarkv, thank you for reporting this issue.
I investigated and it turns out that the problem stems from IMDb itself.
The syncer determines what the name of an IMDb list is by extracting data from the Content-Disposition header.
When I created an IMDb list with the name Cabeça, this is the value I received in the IMDb response:
Content-Disposition: attachment; filename="Cabe�a.csv"

In conclusion, if IMDb doesn't send correctly formatted values within this header, the syncer has no way of knowing what the correct list name is.
To correctly set the value of this header, IMDb needs to encode the value in UTF-8 format.
I doubt IMDb will change that detail anytime soon, if at all. The solution for now is to not use list names that contain special characters 👍


typo2.0 beta release note fix

Proposed changes
Issue Number: close #xxx

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...

LGTM


Android: Crash marking folder read from long press

java.lang.ArrayIndexOutOfBoundsException: length=12; index=-1
  at java.util.ArrayList.get(ArrayList.java:310)
  at com.newsblur.database.FolderListAdapter.getChild(FolderListAdapter.java:336)
  at com.newsblur.fragment.FolderListFragment.onContextItemSelected(FolderListFragment.java:273)
  at android.app.Fragment.performContextItemSelected(Fragment.java:2174)
  at android.app.FragmentManagerImpl.dispatchContextItemSelected(FragmentManager.java:2002)
  at android.app.Activity.onMenuItemSelected(Activity.java:2908)

@dosiecki from a quick look at the code are folders no longer handled correctly after the recent refactoring for saved stories?

The new code for the feed list hasn't even been beta tested yet, so there could certainly be some bugs.  Good catch; I will take a look tomorrow.

fixed in #912


Remove deprecation warnings for SQL module

@seut Note, that the existing setting cannot be read or written. It is broken because it was an internal ES setting that has just been renamed.

I will make a CHANGES entry explaining that the index.recovery.initial_shards had been replaced and has no effect anymore. We just have to decide whether to keep or remove the new setting. According to internals, it is only ever used to recover old cluster state.
See

https://github.com/elastic/elasticsearch/blob/v5.2.2/core/src/main/java/org/elasticsearch/gateway/PrimaryShardAllocator.java#L365
https://github.com/elastic/elasticsearch/blob/v5.2.2/core/src/main/java/org/elasticsearch/gateway/PrimaryShardAllocator.java#L145
https://github.com/elastic/elasticsearch/blob/v5.2.2/core/src/main/java/org/elasticsearch/gateway/PrimaryShardAllocator.java#L361

Apologies, I was wrong about the deprecated setting not being read anymore. It defaults to the new setting but still reads the old setting if present. So let's keep the setting to avoid trouble for users.
I've added a CHANGELOG entry to point out that the setting is deprecated and may be removed in future versions. I've also marked the setting deprecated in the Crate code base.

Codecov Report

Merging #5860 into master will decrease coverage by 0.12%.
The diff coverage is 80%.

@@             Coverage Diff              @@
##             master    #5860      +/-   ##
============================================
- Coverage     83.07%   82.94%   -0.13%
+ Complexity    12137    12122      -15
============================================
  Files          1428     1428
  Lines         46481    46495      +14
  Branches       5717     5721       +4
============================================
- Hits          38612    38565      -47
- Misses         5202     5251      +49
- Partials       2667     2679      +12

Impacted Files
Coverage Δ
Complexity Δ

...main/java/io/crate/analyze/TableParameterInfo.java
100% <ø> (ø)
6 <0> (ø)
:arrow_down:

...executor/transport/TransportShardDeleteAction.java
80.76% <0%> (ø)
10 <0> (ø)
:arrow_down:

.../crate/rest/action/CrateThrowableRestResponse.java
73.07% <100%> (-1.93%)
5 <0> (ø)

core/src/main/java/io/crate/geo/GeoJSONUtils.java
82.23% <100%> (ø)
29 <0> (ø)
:arrow_down:

...executor/transport/TransportShardUpsertAction.java
85.81% <100%> (ø)
72 <0> (ø)
:arrow_down:

.../io/crate/operation/scalar/geo/WithinFunction.java
88.52% <50%> (ø)
26 <0> (ø)
:arrow_down:

.../src/main/java/io/crate/exceptions/Exceptions.java
66.66% <81.25%> (+46.66%)
5 <4> (+4)
:arrow_up:

...n/java/io/crate/exceptions/JobKilledException.java
0% <0%> (-66.67%)
0% <0%> (-1%)

...in/java/io/crate/lucene/match/CrateRegexQuery.java
43.47% <0%> (-30.44%)
4% <0%> (-4%)

...ion/aggregation/impl/GeometricMeanAggregation.java
59.09% <0%> (-22.73%)
11% <0%> (ø)

... and 32 more

Continue to review full report at Codecov.

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


v3 Release: NFT + code cleaning + doc update

⚠ All PRs have to be done versus 'dev' branch, so be aware of that, or we'll close your issue ⚠
From https://github.com/jeromeetienne/AR.js/pull/674#issue-352547279 #674
What kind of change does this PR introduce?
This PR want to test the new jsartoolkit5 from https://github.com/kalwalt/jsartoolkit5/pull/1 with the new changes especially the NFT part.
🖼 This will eventually make AR.js capable of detect and track 2D images.
https://twitter.com/nicolocarp/status/1204841439160684549

Could be raise some bugs and incompatibilties between the code. It is more a test...
Can it be referenced to an Issue? If so what is the issue # ?
From jsartoolkit5 repo:  PR  https://github.com/kalwalt/jsartoolkit5/pull/1
From AR.js issues:

https://github.com/jeromeetienne/AR.js/issues/544
https://github.com/jeromeetienne/AR.js/issues/149
https://github.com/jeromeetienne/AR.js/issues/503
https://github.com/jeromeetienne/AR.js/issues/420
https://github.com/jeromeetienne/AR.js/issues/227
https://github.com/jeromeetienne/AR.js/issues/190
https://github.com/jeromeetienne/AR.js/issues/185

also, a repo cleaning will help for: https://github.com/jeromeetienne/AR.js/issues/433
How can we test it?
Just test the examples and tell us what happens...

Summary

Does this PR introduce a breaking change?
Nope, it will remove old code, but unnecessary. It will add NFT feature.
Please TEST your PR before proposing it. Specify here what device you have used for tests, version of OS and version of Browser
Other information
I uploaded the new jsartookit5 libs and build the AR.js libs with makefile.
Progress

[x] Clean repo before starting the porting
[x] Import new jsartoolkit5 version, coming with NFT
[x] Adapt the code (threejs side) to support NFT (emulate what's already done for fiducial markers)
[x] Fix the rendering of content on AR.js side, for NFT (onMarkerFound function)
[x] Make it work on mobile
[ ] Enhance resolution in both mobile and desktop
[ ] Check if animations work with NFT. If not, or rendering rate is too high, use interpolation
[ ] Fix broken pattern marker (content appears shifted after NFT modifies, or maybe even before during the cleaning of the repo)
[x] Remove all rendering code on the Worker
[x] Adapt the code (aframe side) to support NFT (emulate what's already done for fiducial markers)
[x] Create at least one example for threejs + NFT
[x] Create at least one example for aframe + NFT
[ ] Cable the change of position of the 3D object (x = 100, y = 100) inside the core, so we don't have to replicate it in every example (both aframe/three.js)
[ ] Change documentation (principal Readme, also with NFT Marker generator + Readmes inside threejs + aframe folders)
[ ] release version 3 of AR.js

It now works on aframe side and three.js side (after mergin v3-working branch).
Updated the checklist.
Work in progress 💪🏻

That's a great Progress indeed, let's improve it now!

Can you please re-open this PR on the new dev branch? thank you

sorry had to clean the dev branch


manifest for container-registry-fra.oracle.com/kubernetes/pause-amd64:3.1 not found

Steps to Reproduce:

cd into Container Registry folder.
vagrant up
vagrant ssh registry
/vagrant/scripts/kubeadm-setup-registry.sh --from container-registry-fra.oracle.com
After the credentials entered, it is throwing an error while pulling docker images from oracle registry.
manifest for container-registry-fra.oracle.com/kubernetes/pause-amd64:3.1 not found

Logs:
Pulling images [v1.10.5] from: [container-registry-fra.oracle.com/kubernetes] to: [localhost:5000/kubernetes]
Checking if docker is active ...
Checking for pull access from container-registry-fra.oracle.com/kubernetes registry ...
Trying to pull repository container-registry-fra.oracle.com/kubernetes/pause-amd64 ...
manifest for container-registry-fra.oracle.com/kubernetes/pause-amd64:3.1 not found
[ERROR] docker cannot pull pause-amd64:3.1 from container-registry-fra.oracle.com/kubernetes registry

This came with the recent release of 1.10 in the developer channel.
It is addressed in  #66 which should be merged soon.

How do I deploy the ContainerRegistry with release version 1.9?

#66 has been merged now, please update your local copy and re-try.
It will download latest supported version by default (1.9) and latest developer version (1.10) if you pass --dev as parameter.


Logging module not working after import pynvim

import pynvim
import logging
logfile = '/tmp/coc-jedi.log'
level = logging.DEBUG
logging.basicConfig(filename=logfile, level=level)
logging.debug('abc')

Nothing would write to /tmp/coc-jedi.log, without import pynvim, it works as expected.

Comment out logging change in __init__.py could fix this problem:
# Required for python 2.6
#class NullHandler(logging.Handler):
#    def emit(self, record):
#        pass
#
#
#if not logging.root.handlers:
#    logging.root.addHandler(NullHandler())

we could skip that section for python3+

Should it just be moved to setup_logging()? The pynvim host should set up logging, but pynvim as a lib should leave that to the app using the library.

Is there any workaround for this?

I've found 2 workarounds for this.
First, remove the NullHandler after you've imported pynvim but before configuring your logger:
from pynvim import NullHandler

# Remove the NullHandler from the root logger if it's there.
# see: https://github.com/neovim/pynvim/issues/373
for handler in logging.root.handlers:
    if isinstance(handler, NullHandler):
        logging.root.removeHandler(handler)

# now add your config
logging.basicConfig(level=args.loglevel)

Alternatively, you can setup logging before the import of pynvim:
import logging

# Set up your logging.
logging.basicConfig(level=logging.DEBUG)

# Now, import pynvim
import pynvim

I checked this issue and can confirm that this has fixed in pynvim 0.5.0-devel (unreleased yet). The workarounds above are valid for pynvim 0.4.3 (and would still be needed for 0.4.3).
FYI, it was fixed in ac03f5c (#492) by dropping python2 support and removing the problematic NullHandler lines.


This project has malicious dependency

Please check issue: https://github.com/dominictarr/event-stream/issues/116
Someone transferred the ownership of the event-stream repo, and introduced a malicious payload.
Please remove the event-stream version and bump to a newer version.

On it! Thanks @ubirajaramneto

Downgraded event-stream to exactly 3.3.4 until I can drop it. Thanks!


build using circleci

Status
Ready
Related Issues
fixes: link to the issue
Description
Move to build via circleci. When build completes a comment will be posted with the preview url to the PR.
@csestito @sserrata FYI. This is phase 1. There is still need to support forked builds which will be handled after this is merged.
Screenshots
Paste here any images that will help the reviewer

Test comment


Apply playspace transform to eye gaze data

Overview
Playspace transform was being ignored by EyeGazeDataProvider, but GazeProvider assumed incoming data was in world space.
Changes

Fixes: #9049  .

This fix has the EyeGazeDataProvider transform the platform data, which is relative to the MRTKPlayspace, into world space before passing it to the consuming GazeProvider. Arguments could be made that the EyeGazeDataProvider should pass the data as is, and let the GazeProvider transform it to whatever space it wants.
That approach, however, assumes that all platform data providers will have raw data in the same space, or else that another data provider which might have the data in world space, would have to transform it to playspace, then have the GazeProvider transform it back to world space.
So I have taken the approach which I have, because world space is something we can all agree on.

Verification

As a reviewer, it is possible to check out this change locally by using the following
commands (substituting {PR_ID} with the ID of this pull request):
git fetch origin pull/{PR_ID}/head:name_of_local_branch
git checkout name_of_local_branch

/azp run


TLS cert per cluster for lens-k8s-proxy

Fixes #5042

I'm seeing "All checks have passed" but there are only 2 checks??


[BUG] Deprecate ProximityForest

Describe the bug
ProximityForest is broken and at this point it is easiest to deprecate it and maybe look for someone to reimplement it. See here for a discussion
https://github.com/sktime/sktime/issues/3648
To Reproduce
from sktime.classification.distance_based import ProximityForest
    from sktime.datasets import load_unit_test
    pf = ProximityForest()
    trainX, trainy = load_unit_test()
    pf.fit(trainX, trainy)```

this results in an infinite recursion in _fit leading to stack overflow.
RecursionError: maximum recursion depth exceeded while calling a Python object
this happens on all problems I tried (unit_test, arrow_head, GunPoint, ItalyPowerDemand)
Expected behaviour
It should fit and predict with default parameters on data in an allowed format.
Additional context
Removal previously blocked. The original author of PF is unavailable,

I agree with the removal. The implementation could not complete the vast majority of UCR datasets (it would crash).
I think having the algorithm as part of sktime is a good idea, however.
https://github.com/wildboar-foundation/wildboar has an implementation, perhaps we could interface with it?
Does not have to be done in the same PR as removal.

I think having the algorithm as part of sktime is a good idea

I like the irony in this :)
But I suppose you mean the fork.

Yes 🙂


FIX-#94: Change docs copyright statement to Modin Authors

Signed-off-by: Igoshev, Yaroslav<EMAIL_ADDRESS>What do these changes do?

[x] passes flake8 .
[x] passes black .
[x] signed commit with git commit -s
[x] Resolves #94
[x] tests passing
[x] module layout described at docs/developer/architecture.rst is up-to-date

@devin-petersohn, we agreed to use Modin Authors in our AUTHORS file so I changed it in conf.py docs too.

@YarShev makes sense!


Update update-compliance-get-started.md

Added new pre-requisite that devices must be AAD joined.

Closing pull request. Moved to private repo (6516) @vpaulreed


[📄DM] Add text to make the history button switch option 'searchable'

Describe your suggested feature
So recently i discovered that the history button can be moved again, so i tried to search the related button in the setting typing "history" and the option didn't come out so i searched it manually and i found it.
https://user-images.githubusercontent.com/75091899/176617781-cc8e4892-2046-44bd-bbf0-f6fde774648b.mp4

After i enabled the switcg of the history button that option came out, but onky because i enabled it.

to sum it up, increse the search range of the search bar in the setting like these kind of contexts
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I will fill out all of the requested information in this form.

It's not showing up in search because it's not a setting


Rename group rules to more human readable abbrevations

Currently rules use group ids:

I believe it could be better to use letters instead of group id. After all there is no reason to keep to only numbers.
For example:
tag-with-space / W0601 -> tag-with-space / TAG01
We can also use this as opportunity to split rules more. Currently most of our rules are under "misc" and could be further grouped. Using letters allows us to create more groups than before.

list rules should also list in groups, so it's easier to read.


Login suddenly stopped working

Today @5:55 UTC login stopped working on my site. I did 0 changes in code, so the issue is not on my side.
After clicking "login button" https://websiteurl.ru/?login opens and nothing happens.
It had been working well for 1 year until today. Changing API key won't help. Host - beget.com. The issue exists in all browsers and devices. SteamAuthentication v4.0
Anyone else experiencing the same issue?
I have .ru domain, could this be the problem?

ip ban


invalid XML characters present in abstracts

@martinseul @aphelionz @dtarb
When running check_bag on all resources, the resource below crashed on bag creation, because there are invalid characters in the resource abstract when creating resourcemetadata.xml. Thus the bag for this resource cannot be created. The only way that this could happen is if the UI and/or REST API allowed the resource abstract to contain invalid characters. This was tested on cuahsi-dev-2.hydroshare.org using an image of the production system.
bag bags/1a7dc5d6b9fa4253bca442341eef500d.zip NOT FOUND
metadata_dirty is True
bag_modified is True
Traceback (most recent call last):
  File "manage.py", line 10, in <module>
    execute_from_command_line(sys.argv)
  File "/usr/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 354, in execute_from_command_line
    utility.execute()
  File "/usr/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 346, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/usr/local/lib/python2.7/site-packages/django/core/management/base.py", line 394, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/usr/local/lib/python2.7/site-packages/django/core/management/base.py", line 445, in execute
    output = self.handle(*args, **options)
  File "/hydroshare/hs_core/management/commands/check_bag.py", line 194, in handle
    check_bag(r.short_id, options)
  File "/hydroshare/hs_core/management/commands/check_bag.py", line 48, in check_bag
    create_bag_files(resource)
  File "/hydroshare/hs_core/hydroshare/hs_bagit.py", line 92, in create_bag_files
    out.write(resource.get_metadata_xml())
  File "/hydroshare/hs_core/models.py", line 1949, in get_metadata_xml
    include_format_elements=include_format_elements)
  File "/hydroshare/hs_core/models.py", line 3935, in get_xml
    dcterms_abstract.text = self.description.abstract
  File "src/lxml/lxml.etree.pyx", line 1031, in lxml.etree._Element.text.__set__ (src/lxml/lxml.etree.c:51697)
  File "src/lxml/apihelpers.pxi", line 711, in lxml.etree._setNodeText (src/lxml/lxml.etree.c:23071)
  File "src/lxml/apihelpers.pxi", line 699, in lxml.etree._createTextNode (src/lxml/lxml.etree.c:22931)
  File "src/lxml/apihelpers.pxi", line 1439, in lxml.etree._utf8 (src/lxml/lxml.etree.c:30219)
ValueError: All strings must be XML compatible: Unicode or ASCII, no NULL bytes or control characters

I have modified check_bag to watch for this error and am running it again on all resources to check the extent of the bug.

Report from updated check_bag that doesn't crash:
value error encountered for 1a7dc5d6b9fa4253bca442341eef500d: All strings must be XML compatible: Unicode or ASCII, no NULL bytes or control characters
value error encountered for 31f30d14c88748d986842d278d125a5c: All strings must be XML compatible: Unicode or ASCII, no NULL bytes or control characters.

See https://www.safaribooksonline.com/library/view/python-cookbook-2nd/0596007973/ch01s24.html
for one way to convert Unicode to XML in a lossless manner.

@dtarb @mseul @aphelionz @horsburgh
I analyzed this situation in detail, and there were control characters in each resource's data, including ASCII code 1 (control-A) and ASCII code 11 (control-K). I have written a patch to remove these from the input to resourcemetadata.xml.
But in looking at this, I inadvertently discovered that all formatting is stripped out of the abstract when it is encoded. For example, 1a7dc5d6b9fa4253bca442341eef500d has line breaks that don't make it into resourcemetadata.xml.  This makes things look bad, because the line breaks are not replaced by spaces; they are just missing. This leads to bad breaks like
...Rapid Communications in Mass Spectrometry, 31(1), 75-84.Abstract:RATIONALE: The fast ...

instead of
...Rapid Communications in Mass Spectrometry, 31(1), 75-84.

Abstract:

RATIONALE: The fast ...

Is this intentional?  If not, what should the XML contain?
At the very least, I think I need to substitute spaces for line breaks, e.g.,
...Rapid Communications in Mass Spectrometry, 31(1), 75-84. Abstract: RATIONALE: The fast ...

(The actual filtering is done by the xml tree functions. In order to avoid it, the formatting has to be pre-translated into its XML equivalents.)

@dtarb @aphelionz @martinseul @horsburgh  How is this for a compromise? Replace \n with paragraph symbol  ¶. Replace control characters with spaces. Avoid repeating spaces or paragraph symbols. To wit:
… Rapid Communications in Mass Spectrometry, 31(1), 75-84. ¶ Abstract: ¶ RATIONALE: The fast ... ```

@alvacouch I think we need a solution that is:

Simple
Consistent with general best practices in the field for working with XML files
Is lossless in that it would permit the abstract to be read back in from the XML identical to the way it is initially stored (not necessarily the same as the user input).
Gives the user flexibility to introduce formatting.  Users are always going to want to provide more formatting than is possible.  We should be as accommodating as is reasonable.  If it is possible to have some sort of rich text or markdown as is quite common in web content entry these days we should provide it, but only consistent with 1-3.
I realize that point 4 may lead us down paths that differ from 1-3, and that the definitions of 1 and 2 are imprecise.

I tried this out a bit on dev-2.  See https://cuahsi-dev-2.hydroshare.org/resource/b6aa4bcafb90425eba889c3b4646731e/.
Formatting with new lines (cut and paste from this comment) did not make it into the XML.
.
I am pleased to see that special characters are accepted in the title and in the abstract.  Many of the considerations for abstract will apply to title and perhaps other fields too, though I think that there is no need for new lines in a title (and they appear not possible now - good).
, though the landing page display is OK.

@dtarb So, you've found a case in which the formatting still gets stripped. Will investigate. My current solution only preserves newlines in otherwise bare text.

@dtarb - @alvacouch is right. We could potentially spend a lot of time here. It's an Abstract field - not an entire document. If we find that people need complex formatting to cram stuff into the Abstract field it is likely because we have failed somewhere else in letting them enter what they want - e.g., they want to put it in the Abstract because there isn't another place for it to go.
How much formatting do we really need to support?  Your number 1 isn't really consistent with numbers 3 and 4.

@dtarb @horsburgh You found is a bug in my bag generation. I regenerated the bag and it puts paragraph marks in where you might want.  So, I need to change the code for "new resourcemetadata.xml" versus "existing resourcemetadata.xml".  What I get now is:
1. Simple ¶ 2. Consistent with general best practices in the field for working with XML files. ¶ compositions (δ2H and δ18O values) of soil and sediment ¶ 3. Is lossless in that it would permit the abstract to be read back in from the XML identical to the way it is initially stored (not necessarily the same as the user input). https://cuahsi-dev-2.hydroshare.org/resource/b6aa4bcafb90425eba889c3b4646731e/ ¶ 4. Gives the user flexibility to introduce formatting. Users are always going to want to provide more formatting than is possible. We should be as accommodating as is reasonable. If it is possible to have some sort of rich text or markdown as is quite common in web content entry these days we should provide it, but only consistent with 1-3. ¶ I realize that point 4 may lead us down paths that differ from 1-3, and that the definitions of 1 and 2 are imprecise

My guess is that to reconstruct what was there before, replacing carriage returns with paragraph marks meets your specifications and looks a lot better than what we have now. I would put whitespace before and after to make these prettier, but in spirit, we could reconstruct the original from the formatted copy.
I note that

in the UI, <p> .... </p> are inserted wherever there is a newline.
the resourcemetadata.xml creation code currently interprets the abstract as unicode text without html or xml markup.

@dtarb @horsburgh The bottom line is that there are only two options:

Replace newlines with single spaces.
Replace newlines with some other unicode character, e.g., ¶.

In existing code, newlines are simply ignored. This is unacceptable.
Which will it be?

@dtarb @horsburgh It turns out that unicode has a special character ␤ that is specifically intended to indicate a newline in a stream of text. This is another option.

@dtarb @horsburgh  cuahsi-dev-2 is now working properly. Forgot to restart to flush .pyc usage.

@dtarb @horsburgh FYI: In the UI: isolated newlines are translated to  while sequences of newlines are translated to . So, to be completely compatible with the UI, I would have to use both ␤ and ¶.

@alvacouch please fix as you can right now using the best quick approach.
@horsburgh with respect to what people are trying to put in the abstract, if you just look at how people are trying to put information in it, I think it speaks to a need for greater flexibility.  I am all for keeping the abstract simple, and for fixing what we have now so that bag creation works, but over the longer term as part of advancing our resource data model I think we need to be considering text fields that users can add to describe their resources using richer formatting.
I looked to see what the XML generated from the resource with formatting and special characters would be, and it is

If I cut and paste this into a new abstract I get

Thus it is not clear how a user could recreate the formatted abstract from the encoding that has been embedded in the XML.  For now this may not be necessary, but longer term we need to think this through.  Some websites have view source options that allow tweaking of these sort of things.  We should be looking at what standard practices there are for these things.

@horsburgh @dtarb @aphelionz @martinseul Try http://cuahsi-dev-2.hydroshare.org now.
- Multiple newline sequences are replaced with ¶ (Unicode paragraph symbol)
- Isolated newlines (e.g., a line break) are replaced with ⏎ (Unicode symbol for carriage return)
- These are padded with spaces.
- Either one or the other appears. There are no combinations.

This was in the spirit of the call yesterday.

@dtarb - I'm not questioning the need for enhanced information, I'm saying that the Abstract isn't necessarily the place for it. Many people may be cramming stuff in the Abstract that could/should go in other metadata elements because they either don't know about what is available to them or they don't want to take the time to fill things out. Some best practices here may alleviate the need for complex functionality around what should be a simple text entry field.
Certainly a review of what has been done by users will clue us in on how people are using the Abstract field and where improvements can be made.

@alvacouch what is the status of this.  If you are busy with downloads, should we assign this to someone else.

@dtarb The proposed fix is done and part of the "Large Files" PR. The algorithm is simple: replace two or more newlines with a paragraph; one with a newline character in UML. This matches the way abstracts are shown in the UI: two or more means , while one in isolation means .

@alvacouch @aphelionz not sure if this was supposed to be addressed in 1.16 deployed today, but https://www.hydroshare.org/resource/1fedd80cd20245409a0ea02cc2a1ecf0/ is broken for downloading a bag and for copying.  When attempting to copy the error is

@hyi @sblack-usu @alvacouch i checked and this is still an issue in production

close, fix is implemented with #3586

close, fix is implemented with #3586


elastic-tabstops for table command

Problem
table command right now has hacks to nicely display some of the standard keys
Solution
Let's try to explore
http://nickgravgaard.com/elastic-tabstops/

maybe cannot be directly applied to this shell


HW7

Almost done except 2 error messages and the main function

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 1

Complexity increasing per file
==============================
- Src/BootCamp.Chapter/BalanceStats.cs  7
- Src/BootCamp.Chapter/TextTable.cs  2

Clones added
============
- Src/BootCamp.Chapter/BalanceStats.cs  2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 2

Complexity increasing per file
==============================
- Src/BootCamp.Chapter/BalanceStats.cs  7
- Src/BootCamp.Chapter/TextTable.cs  2

Clones added
============
- Src/BootCamp.Chapter/BalanceStats.cs  2

See the complete overview on Codacy

Im stuck at 3 places and I hope a mentor can give me some hints.

in the biggestLost data there is a empty input. Is it wise to use tryParse now and so set the amount to zero and make that person the person with the biggest lost  ?

I use Stringbuilder now to make the list of persons to display. But as far as I know I cannot know then the number of persons so I can not make a difference between has and have.  Please some hints how I can make that work. I tried it with a counter but that one did not give me the right answers.

On the same tests of part2 the output schoud be  $ -1 according to the test but I see -$1 .
Any hints how that can happen ?

I have debugged and looked a lot but cannot think of a answers to this questions so I hope a mentor can give me some hints so I can proceed.

solved the currency problem like this :
CultureInfo.CurrentCulture = CultureInfo.InvariantCulture;
            NumberFormatInfo noParens = (NumberFormatInfo)CultureInfo.CurrentCulture.NumberFormat.Clone();
            noParens.CurrencyNegativePattern = 1;
            var lowestBalanceString = lowestBalance.ToString("C0", noParens);

I hope you approve it.

sorry. maybe Im stupid or we misunderstood each other complete.
you said this :  take all the code that finds the personal biggest loss and move it to a function. It's simple.
Can  you exact give me the part , you want me to move into the new function. If I do the whole for loop , I cannot refractor it because you said in the chat , the scope is then not complete.

and is this what you mean :
public static string FindHighestBalanceEver(string[] peopleAndBalances)
        {
            if (IsInValidValidInput(peopleAndBalances))
            {
                return InValidOutput;
            }

            var HighestBalanceEver = decimal.MinValue;
            var PersonWithHighestBalance = new StringBuilder();

            for (int i = 0; i <= peopleAndBalances.Length - 1; i++)
            {
                HighestBalanceEver = FindHighestBalance(peopleAndBalances, HighestBalanceEver, PersonWithHighestBalance, i);

            }

            var answer = $"{PersonWithHighestBalance.ToString()} had the most money ever. ¤{HighestBalanceEver}.";

            return answer;
        }

        private static decimal FindHighestBalance(string[] peopleAndBalances, decimal HighestBalanceEver, StringBuilder PersonWithHighestBalance, int i)
        {
            var currentPersonData = peopleAndBalances[i].Split(',');
            var highestAmountOfPerson = FindHighestBalanceOfPerson(currentPersonData);

            if (highestAmountOfPerson > HighestBalanceEver)
            {
                HighestBalanceEver = highestAmountOfPerson;
                var currentOne = PersonWithHighestBalance.ToString();

                if (string.IsNullOrEmpty(currentOne))
                {
                    PersonWithHighestBalance.Append(currentPersonData[0]);
                }
                else
                {
                    PersonWithHighestBalance = new StringBuilder();
                    PersonWithHighestBalance.Append(currentPersonData[0]);
                }

            }
            else if (highestAmountOfPerson == HighestBalanceEver)
            {
                if (i == peopleAndBalances.Length - 1)
                {
                    PersonWithHighestBalance.Append(" and ");
                }
                else
                {
                    PersonWithHighestBalance.Append(", ");
                }

                PersonWithHighestBalance.Append(currentPersonData[0]);
            }

            return HighestBalanceEver;
        }

        public static decimal FindHighestBalanceOfPerson(string[] currentPersonData)
        {
            var highest = decimal.MinValue;
            var allAmounts = currentPersonData[1..];
            foreach (var amountString in allAmounts)
            {
                var amount = decimal.Parse(amountString);
                if (amount > highest)

                {
                    highest = amount;
                }
            }

            return highest;
        }

        private static decimal CheckIfMorePeopleHaveTheSameAmount(decimal highestBalanceEver, StringBuilder personWithHighestBalance, string[] currentPersonData, decimal highestCurrentPersonBalance)
        {
            if (highestCurrentPersonBalance > highestBalanceEver)
            {
                highestBalanceEver = highestCurrentPersonBalance;
                var currentOne = personWithHighestBalance.ToString();

                if (string.IsNullOrEmpty(currentOne))
                {
                    personWithHighestBalance.Append(currentPersonData[0]);
                }
                else
                {
                    personWithHighestBalance.Replace(currentOne, currentPersonData[0]);
                }

            }

            return highestBalanceEver;
        }

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 13

Complexity increasing per file
==============================
- Src/BootCamp.Chapter/BalanceStats.cs  4
- Src/BootCamp.Chapter/TextTable.cs  2

Clones added
============
- Src/BootCamp.Chapter/BalanceStats.cs  2

See the complete overview on Codacy

I hopefully made all the changes you asked me to do. It was a struggle.  If not, can you precise tell me what and where to change because the most struggle was to find out what you exactly wants.
if its allright , I hope you can give me hints to solve the have/has problem. With StringBuilder I cannot see how many elements there in so I can now never decide if I have to use "has"  or "have"

last version and I hope if I can improve things or hints how to solve the has/have problem. please some hints or examples excpecially when I can improve things. Or a good description what is expected from me.

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 5

Complexity increasing per file
==============================
- Src/BootCamp.Chapter/BalanceStats.cs  4
- Src/BootCamp.Chapter/TextTable.cs  2

See the complete overview on Codacy

and what to do with this one :
Introduce a new variable instead of reusing the parameter 'HighestBalanceEver'.

Time to fix: 5 minutes

            if (highestAmountOfPerson > HighestBalanceEver)

            {

                HighestBalanceEver = highestAmountOfPerson;

                var currentOne = PersonWithHighestBalance.ToString();

Looks nothing wrong to me

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 5

Complexity increasing per file
==============================
- Src/BootCamp.Chapter/BalanceStats.cs  5
- Src/BootCamp.Chapter/TextTable.cs  2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 2

Complexity increasing per file
==============================
- Src/BootCamp.Chapter/BalanceStats.cs  7
- Src/BootCamp.Chapter/TextTable.cs  2

See the complete overview on Codacy


Improve docs on authentication

From Discord:

How is one supposed to set response headers with Telefunc?
I've thought on making my authentication through Telefunc, but I need to set a cookie on the response

Currently, the answer is that you should implement authentication endpoints directly on your server without Telefunc. Then follow https://telefunc.com/permissions.
We should improve the documenation to clarify that.
As for being able to implement authentication over Telefunc, I'm not sure the benefit outweighs bloating Telefunc. Comment and arguments welcome.

Let's track progress on documentation at https://github.com/brillout/telefunc/issues/104.


installation instructions don't work on os x

So I'm trying to install from the instructions on the installation page on os x 10.10.4. I can install binaries only (option 2), but installing the dependencies throws an error. option 1 (full install) gives:
## Failed installation for _deepdive_runtime_deps
without any more verbose errors. From source gives me:
make: *** No rule to make targetinstall'.  Stop.`
At first we were thinking there was a permissions issue in getting the dependencies compiled and installed, but now I'm not sure? Initial curl prompt is:
Sandra:Projects mcfrank$ bash <(curl -fsSL deepdive.stanford.edu/install)
### DeepDive installer for Mac
++ curl -fsSL https://github.com/HazyResearch/deepdive/raw/v0.7.x/util/install/install.Mac.sh
1) deepdive
2) deepdive_from_release
3) deepdive_from_source
# Select what to install (enter a number or q to quit)? (23) Failed writing body`

which includes (23) Failed writing body - might result from some issue with text encoding or reading, per this. But when I move the installer locally and then run I still get the same issue... any help appreciated - thanks!
with @mikabr and @mllewis

Sorry for the hassle guys.  It turns out we didn't test carefully enough on vanilla OS X.  The installer script should now work as expected now.

I have similar error on macOS Sierra 10.12.5

tar xzvf deepdive-v0.8-STABLE-Darwin.tar.gz -C /Users/andyck/local
tar: Error opening archive: Failed to open 'deepdive-v0.8-STABLE-Darwin.tar.gz'

Failed installation for deepdive_from_release
Failed installation for deepdive
Install what (enter to repeat options, a to see all, q to quit, or a number)?


Download file from Blob

Hi,
I have a question about if we can download the file from blob using SDK APIs? Just like we have for upload.

@Abhi2109 Can you elaborate on the scenario.  Currently the C sdk does not provide storage apis.

@RickWinter yes. So just like we have an API "IoTHubClientCore_LL_UploadToBlob" to upload the file in blob , can we have an API to download the same file from Blob in C.

@Abhi2109 Thank you for filling this issue and providing clarity.  I've added this to our backlog.

Hi Rick,
Any idea by when this feature will be available?
DO i need to subscribe to any forum for getting the notification like "Ok this feature is available"

Blobs is available in beta
https://github.com/Azure/azure-sdk-for-c/blob/feature/storage-blobs/sdk/inc/azure/storage/az_storage_blobs.h


Represent blocks as JSON objects and change code to support new format

JSON representation of blocks allows easier conversion to XML and better manipulation of blocks layout, useful for future customization of editor.
drawer.js - use dictionary (which maps categories to JSON block objects), convert blocks to XML using toolboxController.js
toolboxController.js  - provides bd.toolbox.ctr functions for changing JSON block objects into XML strings
blocklyframe.html - added script tag linked to toolboxController.js
appengine-web.xml - includes path for toolboxController.js because it is a static file

The core of this functionality was put into the blocks toolkit work. The remaining work is being discussed in #2384

The core of this functionality was put into the blocks toolkit work. The remaining work is being discussed in #2384


Discriminated union not narrowing when discriminator is itself a union

Bug Report

🔎 Search Terms
discriminated union, discriminator, narrowing
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about discriminated unions

⏯ Playground Link

Playground link with relevant code
💻 Code

type DiscriminatedUnion = A | B | CorD;

interface A {
  type: "a";
}

interface B {
  type: "b";
}

interface CorD {
  type: "c" | "d";
}

declare const thing: DiscriminatedUnion;

thing.type;
// ^? const thing: DiscriminatedUnion
thing.type;
//    ^? (property) type: "a" | "b" | "c" | "d"

if (thing.type === "a") throw "No good!";

thing.type;
// ^? const thing: B | CorD
thing.type;
//    ^? (property) type: "b" | "c" | "d"

if (thing.type === "c" || thing.type === "d") throw "No good!";

thing.type;
// ^? const thing: B | CorD
thing.type;
//    ^? (property) type: "b"

🙁 Actual behavior
thing.type at the end has type B | CorD.
🙂 Expected behavior
thing.type at the end should have type B, because it can never be CorD.

This may well be considered a feature that hasn't been implemented yet.
For context, here's the type I'm trying to remove. This is an AST for SQL expressions. I've parsed an expression using this library, which gives me an Expr, but I'm only expecting to handle simpler kinds of SQL expressions, and I'd like to throw on more complex ones and narrow the type. Currently, it's harder than I'd like to kick SelectFromUnion out of the large Expr union, because its type is 'union' | 'union all'. Probably my best bet at this point would be a custom type guard, but it sure would be nice if TS could figure out the narrowing for me.

Duplicate of #31404.


Tests within describe blocks ran via phpstorm do not show output

Describe the bug
Ran a test in a describe block by pressing the green arrow

Then no output is show in phpstorm

Expected behavior
Expected to see test output like so when code is not in a describe block

Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: MacOS 12.6.5
IntelliJ Version: [PhpStorm 2022.3.3
Plugin Version 1.9.2

As per the following thread, it seems you can avoid to use describe() or PR a fix at the moment.
https://github.com/pestphp/pest-intellij/discussions/462

@davideprevosto yep, currently the only workarounds are avoid using it, or run the whole file every time 👍


select linkage update issue

Laravel Version: 5.7
PHP Version: 7.2
Laravel-admin: 1.6.9

Description:
I use select linkage to load options. everything work correctly but after creattion of my model if I want to update my model select linkage didn't fill with old value, and city will be empty if I want to update my model.  I have to select my old province again to load related cities then pick city and update my model.
it should load selected (persisted) city when I want to update my model.
 $form->select('address.province_id', 'Province')
           ->options(Province::all()->pluck('name', 'id'))
           ->load('address.city_id', '/api/city');

 $form->select('address.city_id', 'City');

Steps To Reproduce:

what I need was to call options with callback to get current city id and retrieve my model.
$form->select('address.city_id', 'City')
          ->options(function($id) {
               $city = City::findOrFail($id);
               return [$city->id => $city->name];
           })


TypeError: Cannot read property 'flags' of undefined in getActionsForAddMissingMember during getCodeFixesAtPosition

tsserver version:
hits: 4938
stack:
TypeError: Cannot read property 'flags' of undefined
    at Object.getActionsForAddMissingMember [as getCodeActions] (tsserver.js:75172:31)
    at tsserver.js:74988:33
    at Object.forEach (tsserver.js:1312:30)
    at Object.getFixes (tsserver.js:74987:16)
    at tsserver.js:80109:35
    at Object.flatMap (tsserver.js:1550:25)
    at Object.getCodeFixesAtPosition (tsserver.js:80107:23)
    at IOSession.Session.getCodeFixes (tsserver.js:86515:64)
    at Session.handlers.ts.createMapFromTemplate._a.(anonymous function) (tsserver.js:85423:61)
    at tsserver.js:86669:88
    at IOSession.Session.executeWithRequestId (tsserver.js:86660:28)
    at IOSession.Session.executeCommand (tsserver.js:86669:33)
    at IOSession.Session.onMessage (tsserver.js:86689:35)
    at Interface.<anonymous> (tsserver.js:87881:27)
    at emitOne (events.js:96:13)
    at Interface.emit (events.js:191:7)
    at Interface._onLine (readline.js:241:10)
    at Interface._normalWrite (readline.js:384:12)
    at Socket.ondata (readline.js:101:10)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:191:7)
    at readableAddChunk (_stream_readable.js:178:18)
    at Socket.Readable.push (_stream_readable.js:136:10)
    at Pipe.onread (net.js:560:20)

code has changed since this issue was reported making the issue outdated. closing as won't fix for now.


Working one day, the next not

Yesterday, it was working fine for me. Today, however, when I tried to login, I got failed to access token.
https://cdn.ch1ll.tk/chrome_qSbsnksnm4.png

Your image isn't found, consider uploading it to GitHub so it can be viewed anytime <3

Your image isn't found, consider uploading it to GitHub so it can be viewed anytime ❤

dude it's been over a year 😭
Anyways, it was probably due to rate limits as I was using replit at the time. Now I have my own VPS and have my own session manager.

Your image isn't found, consider uploading it to GitHub so it can be viewed anytime ❤

dude it's been over a year 😭 Anyways, it was probably due to rate limits as I was using replit at the time. Now I have my own VPS and have my own session manager.

Cool stuff ❤, didn't see the time difference oops 😂


Consider to switch mysql charm from 8.0/edge to 8.0/stable

Hi,
The slurm-bundle is currently deploys mysql from 8.0/edge
which could cause unexpected issues on the production deployments due to edge risk nature.
Please consider to switch the channel to 8.0/stable.
P.S. you can find the list of MySQL 8.0/stable releases on https://charmhub.io/mysql/docs/r-releases
Tnx!

Hi @taurus-forever -
Yes! We are going to switch over the stable branch of the MySQL operator. We mostly just used the edge branch in our testing.


Removed 'preview' label

Note: It will be good to include an updated screenshot without the 'preview' label on it.

@merill : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Note that publishing in this repository is suspended until Tuesday, November 2. Merging will resume Tuesday morning. Authors can still review and sign off on PRs in the interim. Thanks.

#sign-off


Cookies not being sent to webview for iOS

Bug description:

Web view should be rendering a web page that has a cookie authentication. Instead it is rendering sign in page. This is happening on iOS after I upgraded to react native 0.73+. It was working properly before the upgrade.
I have both thirdPartyCookiesEnabled={true} and sharedCookiesEnabled={true} enabled. Also in handleNavigationStateChange I reset the cookies for iOS:
async function resetCookies() {
    const cookies = await CookieManager.getAll(Platform.OS === 'ios');
    Object.keys(cookies).forEach(key => {
      const cookie = cookies[key];
      if (cookie.domain === domain) {
        CookieManager.clearByName(url, cookie.name, Platform.OS === 'ios');
      }
    });
  }

Expected behavior:

Web view should render account page, not sign in page
Environment:

OS: Android, iOS
OS version: Android 34+, iOS 14+
react-native version: 0.73.6
react-native-webview version: 13.8.1

Adding this function fixed this issue for me when attempting to login with a webView in iOS
  const synchronizeCookiesiOS = async () => {
    if (Platform.OS === 'ios') {
      try {
        const allCookies = await CookieManager.getAll(true);
        for (const cookieKey of Object.keys(allCookies)) {
          const cookie = allCookies[cookieKey];
          if (cookie) {
            await CookieManager.set('https://' + cookie.domain, cookie);
          }
        }
      } catch (error) {
        console.error('Error synchronizing cookies:', error);
      }
    }
  };

Thanks @thozh. When do you call the synchronize function? Do you call it in the <WebView onLoad={} /> or the `?

why you close this issue? fixed?

@caralin3

In my case this problem was due to sending a custom header in WebView. As soon as I remove the custom header the session cookie is sent again and Website authentication works in WebView.

In WebView 13.10.6 this problem has been fixed. See here: https://github.com/react-native-webview/react-native-webview/issues/3457

Can confirm 13.10.6 fixes this for us as well, unfortunately Expo is stuck at 13.8 at the moment, but hopefully they'll catch up at some point.

@andflett That's why I decided not to use Expo in my project! ;)


Update openstack resources

This PR updates the relevant resources (cluster, nodedeployment, ssh_key) and adds data sources for these elements.

I added required options that have been lately been added to the KKP, i.e. instance_ready_check_period, opa, mla, octavia
I added data sources for cluster, project, sshkey, nodedeployment and the most relevant the kubeconfig. These elements were completely missing from v0.1.0
I added the relevant tests for the added resources and work with this provider for weeks. So far, I didn't run into any crashes. For basic usage, this version is far more stable than the current provider.
Update of a cluster and node deployment works by changing the kubernetes version setting.
I changed the ID definition of Terraform because encoding datacenter and project and cluster ID in a Terraform ID is wrong. Now, the IDs TF uses internally are more clear and not encoded. i.e. cluster_id = cluster_id that TF uses to identify a resource.
Added support for Flatcar image
Several bug fixes that led to critical TF errors

Fixes #36
This PR is an initial step into the right direction to keep the Terraform repo useful. KKP is cool but without proper usage of its API, it's almost not usable in an automated way. Terraform provider is critical such that Kubernetes clusters can be integrated into InfrastructureAsCode deployments.
I would like to see this PR merged and that a new version is released, i.e. 0.2.0
Can you please review this PR such that users can use an upstream supported Kubermatic TF provider.
Thanks in advance!

/assign @kron4eg @zreigz
Have a look at this PR.
I think this is a huge step forward in the terraform-provider-kubermatic repo. I am looking forward to your feedback.
Totally in the spirit of the open-source community, I'd like to share the changes with Upstream such that everybody can support of my improvements. 👍

/ok-to-test

@Happy2C0de thanks a lot for this PR. I have to switch my context to review and test this PR. I will try to do this tomorrow

@Happy2C0de thanks a lot for this PR. I have to switch my context to review and test this PR. I will try to do this tomorrow

Thanks that you will have a look at this PR @zreigz
Sure, come back to me when I can support you with this. Furthermore, I added a section to the CONTRIBUTING.md because it is not very well explained in TF documentation how one can locally build and use a provider.

@Happy2C0de thanks a lot for this PR. I've tested it. Looks good
/lgtm
/approve

@zreigz Thanks for merging this PR! Do you think we could get a new release?

@judge-red yes :+1:


Explain WASM size through Github Action

Goal of this task is to give some more insights into the size of the wasm/native binary.
Note: This is a beginner task because it requires no insight into maplibre-rs. But its probably not tivial.
🤔 Expected Behavior

We want a cargo profile, which includes debug symbols, but is else identical to a release build. This gives a good estimate of actual sizes and allows to debug the usage.
Track wasm size through a custom Github Action. Post diffs in the PR.

MVP: Post size differences of binary output between builds
Future: Breakdown usage by function/crate/....
For linux binaries we can use: https://github.com/RazrFalcon/cargo-bloat
For wasm we must use: https://github.com/rustwasm/twiggy

😯 Current Behavior
No insights.
💁 Possible Solution
There is https://github.com/orf/cargo-bloat-action/tree/master/src
Sadly this does not support WASM. Also it sends data to some cloudflare service in the US.
🔦 Context

💻 Examples

I created a PR which covers the first task of this PR. Create binaries in release mode with debug symbols.

Twiggy has a bug right now: https://github.com/rustwasm/twiggy/issues/326


Add p2p app button

Similar to our DHO button
Website to go to
https://ptm.hypha.earth/

✅ Test Case 1

Go to Explore.
Tap Swap Seeds.
Observe the Login or Account screen.


Cow: Watch tekskillsinc.com

Cow requests the watch of the watch_keyword tekskillsinc\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

tekskillsinc\.com has been seen in 2 true positives, 0 false positives, and 0 NAAs.

Approved by Nisse Engström in Charcoal HQ


Wrong seek position with long videos

Expected behaviour
When seeking from the start of a very long video (I'm using a 2h countdown for example), to a point near the end, the slider thumb should set it's position exactly under the mouse pointer.
Actual behaviour
On Firefox, the thumb is set a little bit to the right when holding down the left mouse button, but then gets placed correctly on mouse up. On Google Chrome, it doesn't even change position after releasing the left mouse button.

Steps to reproduce
Starting with the video paused, go to a point near the beginning of the video, like 1:20, and then move your mouse to somewhere near the end like 1:55:00, hold down the left mouse button and you can see the slider thumb positioned slightly to the right of the pointer. Release the button and it changes position correctly (in Firefox).
Environment

Browser: Mozilla FireFox
Version: 66.0.3
Operating System: Windows
Version: 10

Console errors (if any)
None
Link to where the bug is happening
https://jsbin.com/zovawiloma/edit?html,output
I uploaded a countdown of 2h on my server and used jsbin because I couldn't get it to load on codepen

I’m not sure how much we can do with this. We’re using a standard <input type=“range”> slider so any weird interactions are due to browser behavior more than likely. We simply have a range between 0 and 100 and use that as a percentage to calculate where to seek too.

Alright, thanks for your fast reply. If that's the case I guess I will deal with that. I just want to raise an observation that maybe I should have included in the first text, but I'm not sure it's the culprit since I'm not able to look into the code myself (I don't have the knowledge)... Maybe it's a problem due to the width of the time label shrinking/expanding causing the progress bar to change size during the mouse down event? I did some other testing and I think this could be the cause, since changing the time to the increment view, the slider gets slightly to the left instead of right, because the time label has more digits and the progress bar shrinks to the left (the opposite happens when the number of digits decreases to just minutes).

I can confirm that the problem seems to be caused by the width of the elements. Infact, making the time label width fixed, the slider thumb is positioned correctly.
Proof (it's the same code as before, I just added the css to change the time width):
https://jsbin.com/gecawixeso/edit?html,output


remediation: report cleanup

User Story
let's sync with @skpaterson  + @TheLunaticScripter about the format and content of remediation reports that will need to be ingested:

node id?
removing extra info (e.g. control descriptions)
yml -> json?

goal is to standardize the report format if possible and ensure smallest report size possible
Definition of Done
agreement between all parties re: content of report + format

https://gist.github.com/vjeffrey/d45e43ce828e6de21bed9d8628adf3a6
https://gist.github.com/vjeffrey/86022b037d5f1f4c181330fc822d5612

https://docs.google.com/document/d/1WlYIMaW8w00jDRAgqWaj6wuqrx88piuf-4kQjei37fk/edit?ts=5ed559eb


Look at replacing StringIO usage with simple Reader implementation

See here for original suggestion.
In short, consider moving to a custom implementation of the Reader protocol (or what Pandas would refer to as a "file-like object") in places where StringIO is used.

I am at least a bit curious the degree to which this is preferable to just using StringIO.  Wouldn't mind a bit of discussion and references here if possible before we proceed with anything.

This is one of those things that makes my head tilt like a dog. Why not use a BytesIO when it's a bytes object?

This is one of those things that makes my head tilt like a dog. Why not use a BytesIO when it's a bytes object?

Probably only because I wasn't thinking too hard about it, and the Pandas documentation for read_csv specifically cites StringIO as an example of a file-like.  So it put a valid solution in my head, and I just went with it.
I probably should change that in the #566 review (and then update the issue title and description here; I think the general question probably still applies).

Yeah, I concur, @robertbartel in regard to switching to BytesIO. I must have had a one tracked mind when reviewing! BytesIO didn't come to mind.


Widget: How to provide helpful context around minimal Outputs in the widget?

Feature Request
Packages to improve:

[ ] OWL/SHACL definitions
[ ] ts-sdk
[ ] ts-etl
[ ] spa
[x] widget

Description
in eLife docmaps, there are Actions that do nothing but stamp an existing preprint with a new DOI under a new prefix. They have Outputs that are not more than doi: xyz. You get results like that:

ResearchSquare found this confusing. Should we provide some kind of helper text to clarify the situation? There is no implicit notion of sameness between these two DOIs. Maybe the protocol itself should support such a thing. Support two DOIs? Support something like id: https://[DOI.org url]?
Use case
Viewers will probably find this confusing. RSq did.
Proposed solution
If you have an idea for how this feature could be implemented, please describe it here.
Additional information
Provide any additional information that might be helpful in understanding the feature request.

Interested in your input @andrewedstrom but am happy to call it blocked on design.


false returned for .dng files

This /tmp/c21d67b8-dda0-46db-8386-f04c4a126d8b.dng returns false.
It should return image/dng or image/x-adobe-dng.

https://github.com/jshttp/mime-types#adding-types

https://github.com/jshttp/mime-db/pull/219


hostap: add pwe config for station

Add pwe config for station connect cmd.

Hi @krish2718 @jukkar
Code formatted, could you help to merge pr?

Hi @krish2718 @jukkar Code formatted, could you help to merge pr?

Still showing formatting issues.

Hi @krish2718 @jukkar Code formatted, could you help to merge pr?

 Still showing formatting issues.

Should I also fixed the merged code format, right?

Yeah, please do as a separate commit for existing code.


Would our plug-in be used with those smart shunt with RS485/USB output?

Hi there, as I can see the Victron SmartShunt is quite expensive, it might be interesting or benefit to buy a SmartShunt like this one (or amazon link) then use the Raspberry-Pi + Venus OS + this very nice plug-in to monitoring the simple battery data via Victron VRM web site.
This could be especially useful in case someone's BMS doesn't have a RS485 output then a simple shunt like this might be used.
I also noticed on Youtube someone used this device to read voltage and current info from the RS485 output.
So perhaps it is possible? If so, I might want to order a part like this to try it out. And if you also want to play it out, I can also a similar part from aliexpress and deliver it to you directly :)
I have very limited USB/485 knowledge, but if this can be not difficult I then would like to learn more about it :)
Thank you.

Hi @liufuyang
The short answer is that an external shunt would be a waste of money and is not really useful for Lithium batteries.
In almost all scenarios you would not need a smartshunt or similar when using a Lithium battery, as Lithium batteries require the use of a BMS. And most BMS does their measurements with a shunt already. So you will just add a duplicate.
Shunts are an acurate way to measure current flow. The Victron Smartshunt is a very good shunt (not all are made the same) and it will give you State of charge, some history, the battery voltage and current and is already capabile to talk to the VenusOS without the help of a driver like this. But it does not give you all the data the BMS can.
Lithiums need a BMS for safety. If any cell is overcharged it can cause a fire, so a BMS is used to monitor each cell and stop any charging when a cell reach an overvoltage state. Those are the balancing wires you see going to each cell. This extra information is what is valuable to us and gives the advantange to the BMS over any external measurement device.
Now if you were using Lead Acid or Gel batteries, these do not have a BMS. So you want to add a battery monitor like a BMV or the SmartShunt.

You might be intrested in this thread. That opening poster has both a BMV (the non "Smart" version of the SmartShunt. i.e. it has not bluetooth interface) and compares those with the BMS values. He also has other threads in the same line.
https://energytalk.co.za/t/here-hold-my-beer-have-you-tried-this-before/964/9

Aha, interesting, I think your suggestion makes a lot of sense.  Then I will just focus on waiting my BMS to arrive and see if this driver can work with it, otherwise I can perhaps get a Victron Smartshun. Thank you very much for the detailed info :)

Thanks again, and I will just close this for now 👍
I think my underling question is - how difficult it could be if there is a new device which has RS485 output and we use it with this driver. I wonder how different each manufacturer is doing with their RS485 output. Hopefully in the future all these could be standardised. But this could be asked in some other thread so don't worry if you don't reply here.


feat(SFN Errors): Mark ingest sfn with errors as failed

Need
When an ingest step function runs, if an error occurs and everything is handled using error handler, the step functions is marked as successful in the AWS Console. This can make it difficult to go to find a dataset that failed during a seurat, cxg, or validation step.

If an error is detected that results in a CXG, seurat, or validation failure, the SFN should handle the error and mark it's self as failed.

This will mark the SFN as failed and will be clear when looking at the AWS SFN Console
Approach

Add a step to check if there are errors in the output and set SFN to failed if any were found. This is require modifying the outputs of some steps to pass along the error message.

Definition of Done

An ingest SFN that fails any of the previous step will be marked as failed.

Says "flowchart!" in Minion's voice.


Themes

Is it possible to use the stock sparx theme? This would make it less obvious that i'm using a modded version.

That is not currently an option, but yeah that is probably a good idea lol.
Sadly, I have to focus on GCSEs etc, so I won't be continuing this extension anymore. Feel free to modify any of the code to your liking.

That is not currently an option, but yeah that is probably a good idea lol.
Sadly, I have to focus on GCSEs etc, so I won't be continuing this extension anymore. Feel free to modify any of the code to your liking.

alright no problem gl with ur gcses innit


Magnolia website font is hard to read

While it looks very nice, it's hard to read. Please consider to change it. Thanks

Thanks for the suggestion. So as not to compromise the design, I could do what I did with https://scala.world/ and offer a button to switch to an easy-to-read font.

that would be great

Still ongoing work in progress on the website, but this issue is fixed now. :)

thanks!


Ability to insert dynamic sql statements

  const dynamicStatements = []
  dynamicStatements.push(SQL` WHERE table_1.column_1 = ${x}`)
  dynamicStatements.push(SQL` AND table_1.column_2 = ${y}`)

  await QUERY/* sql */`
    SELECT *
    FROM table_1
    ${dynamicStatements}
  `

Currently this throws an error as it tries to parse the dynamicsStatements array as just a single statement.
My current workaround is as follows:
export function joinStatements(statements: any[], { join = ' ', prefix = ' ' }: { join: string, prefix: string } = { join: ' ', prefix: ' ' }) {
  const tagSymbol = Object.getOwnPropertySymbols(statements[0])[0]

  const values: any[] = []
  const textParts: string[] = []
  for (const statement of statements) {
    const res = statement()
    textParts.push(res.text)
    values.push(...res.values)
  }

  const res = Object.assign(
    () => ({
      text: prefix + textParts.join(join),
      values,
    }),
    {
      [tagSymbol]: true,
    },
  )

  return res
}

It's especially hacky as the tag Symbol isn't exported, so I have to fetch it like so.
If there is functionality to support this and I missed it, please let me know.

The joinStatements function you wrote seems to be vulnerable to sql injection attacks. You can just use reduce function to merge some SQL statements:
  await QUERY/* sql */`
    SELECT *
    FROM table_1
    ${dynamicStatements.reduce((acc, sql) => SQL`${acc} ${sql}`)}
  `

and there is also a SQLS tag function that does the same thing.
Sorry for the late reply.


Unique jobs in queue

Hi, is there any way to check whether job is already in the queue?
I need to add new job in the list, but i dont want to iterate over all of them to be shure that it is not exists.
Unique key on fields(queue, data) will help, but I think it will crush SlmQueue.
Thank you!

There is no supported manner in which to check if a particular Job and a specific payload is already on the queue.
Your suggestion to make the queue / data unique will (afaik) only work in the Doctrine adapter, but should work i'm guessing. You would need to catch exceptions and make sure processed jobs aren't removed from the db. You would be forcing all queue's to behave this way, which might not be what you want.
But to be honest, I believe housekeeping is a responsibility of the developer because it depends on your requirements too much. Keep track of each Job and relevant payload data you push onto a queue and search through that information before adding an new one...
Note that after a push a Job will have been given a unique $job->getId()
Hope this helps

I hope that i miss something and that functional already exists.
So, I'll try to make world better.
Thank you for detailed answer!


RNFetchBlob.config().fetch('GET', uri) always returns valid result

RNFetchBlob.config({}).fetch('GET', uri) function always returns valid result even though I am using invalid image url.(e.g. https://lol.com/lol.png)
React Version : "react": "16.0.0-alpha.6"
React Native Version : "react-native": "0.44.0"
Library Version : "react-native-fetch-blob": "0.10.2"
  return RNFetchBlob.config({
      path: getPath(uri),
  }).fetch('GET', uri);

  fetchImageUri('https://lol.com/lol.png')
     .then(result => {
       console.log('result = ', result);
     })
     .catch((error) => { console.log(error); });

This always returns valid local image url. So I can get the result of fetching image.
It never goes to catch.
How can I get the result of fetching image?
Thank you,

What is fetchImageUri?

https://lol.com/lol.png

Yes?

I don't see what you mean...

Your code sample is incomplete. You return the promise created by the fetch to somewhere outside the screen...

Ah sorry.
    RNFetchBlob.config({
           path: getPath(uri),
        })
        .fetch('GET', 'https://lol.com/lol.png')
        .then(result => {
           console.log('result = ', result);
        })
        .catch((error) => { console.log(error); });
   }

Is there any solution?

No. I wrote a test case for the test package for RNFB for someone to continue from there.
// REQUIRED TEST SETUP: Some tests, e.g. test-fetch.js, require this image
describe('#492 RNFetchBlob.fetch always resolves with valid result', (report, done) => {
  // 404 Not Found
  RNFetchBlob.fetch('GET', TEST_SERVER_URL + '/INVALID')
  .then(result => {
    report(
      <Assert key="Invalid URL should have caused error" expect={false} actual={true}/>,
      <Info key="Response data for fetch error test">
        <Text>{JSON.stringify(result)}</Text>
      </Info>
    )
    done()
  })
  .catch((error) => {
    report(
      <Assert key="Invalid URL causes error" expect={true} actual={true}/>,
      <Info key="Error">
        <Text>{String(error)}</Text>
      </Info>
    )
    done()
  });
})

Result:

Hmm... This can't be the actual solution.

Of course not. I said

I wrote a test case for the test package for RNFB

I don't know how you expect bug fixing to work, the way I know it to work best is to have a test case that's part of ones existing test framework. If you look the RNFB test repo already contains numerous test cases specifically made for issues such es yours, starting with #[issue number]. I'm sorry if you don't see the value and expected an immediate resolution.

@lll000111 , thanks for the great help 👍   let me run the test and figure out what's going on 😄

@bdavid68 @lll000111  I think this is by designed. Quote from OkHttp's document

Note that transport-layer success (receiving a HTTP response code, headers and body) does not necessarily indicate application-layer success: response may still indicate an unhappy HTTP response code like 404 or 500.

So basically, you can check the status in  res.info() then deal with it, instead of try to catch the error.

Thanks @wkh237 !
I will try it.

It seems already present ("react-native-fetch-blob": "0.10.8")


Appstudio update test-component-pac-mtlm

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-mtlm-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
11 seconds
init

✅ Succeeded
13 seconds
clone-repository

✅ Succeeded
50 seconds
build-container

✅ Succeeded
48 seconds
inspect-image

✅ Succeeded
1 minute
deprecated-base-image-check

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
4 minutes
clair-scan

✅ Succeeded
47 seconds
sbom-json-check

✅ Succeeded
1 minute
label-check

✅ Succeeded
10 seconds
show-sbom

✅ Succeeded
17 seconds
show-summary


Nigthly's microphone redirection crashes

I am trying to set up a connection with audio and microphone redirection from Ubuntu 16.04 to Windows 10. Audio works fine, but microphone never worked -- neither in Remmina nor in Ubuntu's default freerdp packages. So I installed the latest Nightly version. It still not works but this time there's also a 100% reproducable crash:
To Reproduce

Execute the following command.
Then in the logged in session: Go to Skype and start an audio chat (e.g. using the Test Echo Call) where the microphone is used, Wait 10 seconds, Audio works but again microphone not. Shadow bytes around the buggy address:
  0x0fefad43f5f0: f9 f9 f9 f9 00 00 00 00 00 00 00 00 f9 f9 f9 f9
  0x0fefad43f600: 00 00 00 00 f9 f9 f9 f9 00 00 00 00 00 00 00 00
  0x0fefad43f610: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 02 f9
  0x0fefad43f620: f9 f9 f9 f9 00 00 00 00 f9 f9 f9 f9 00 00 00 00
  0x0fefad43f630: 00 00 00 00 00 00 00 00 00 00 04 f9 f9 f9 f9 f9
=>0x0fefad43f640: 00 00 04 f9 f9 f9 f9 f9 00 00 04 f9[f9]f9 f9 f9
  0x0fefad43f650: 00 00 04 f9 f9 f9 f9 f9 00 00 04 f9 f9 f9 f9 f9
  0x0fefad43f660: 00 00 04 f9 f9 f9 f9 f9 00 00 04 f9 f9 f9 f9 f9
  0x0fefad43f670: 00 00 04 f9 f9 f9 f9 f9 00 00 03 f9 f9 f9 f9 f9
  0x0fefad43f680: 00 00 03 f9 f9 f9 f9 f9 00 00 03 f9 f9 f9 f9 f9
  0x0fefad43f690: 00 00 03 f9 f9 f9 f9 f9 00 00 04 f9 f9 f9 f9 f9
Shadow byte legend (one shadow byte represents 8 application bytes):
  Addressable:           00
  Partially addressable: 01 02 03 04 05 06 07
  Heap left redzone:       fa
  Heap right redzone:      fb
  Freed heap region:       fd
  Stack left redzone:      f1
  Stack mid redzone:       f2
  Stack right redzone:     f3
  Stack partial redzone:   f4
  Stack after return:      f5
  Stack use after scope:   f8
  Global redzone:          f9
  Global init order:       f6
  Poisoned by user:        f7
  Container overflow:      fc
  Array cookie:            ac
  Intra object redzone:    bb
  ASan internal:           fe
Thread T22 created by T9 here:
    #0 0x7f796bc58253 in pthread_create (/usr/lib/x86_64-linux-gnu/libasan.so.2+0x36253)
    #1 0x7f7969741974  (/opt/freerdp-nightly/bin/../lib/libwinpr2.so.2+0x107974)
    #2 0x7f7969742187 in CreateThread (/opt/freerdp-nightly/bin/../lib/libwinpr2.so.2+0x108187)
    #3 0x7f796a5edcc5  (/opt/freerdp-nightly/bin/../lib/libfreerdp-client2.so.2+0x133cc5)
    #4 0x7f796a533096  (/opt/freerdp-nightly/bin/../lib/libfreerdp-client2.so.2+0x79096)
    #5 0x7f796a535cd0  (/opt/freerdp-nightly/bin/../lib/libfreerdp-client2.so.2+0x7bcd0)
    #6 0x7f796a5d46d4  (/opt/freerdp-nightly/bin/../lib/libfreerdp-client2.so.2+0x11a6d4)
    #7 0x7f7969741ce1  (/opt/freerdp-nightly/bin/../lib/libwinpr2.so.2+0x107ce1)
    #8 0x7f796905a6b9 in start_thread (/lib/x86_64-linux-gnu/libpthread.so.0+0x76b9)

Thread T9 created by T1 here:
    #0 0x7f796bc58253 in pthread_create (/usr/lib/x86_64-linux-gnu/libasan.so.2+0x36253)
    #1 0x7f7969741974  (/opt/freerdp-nightly/bin/../lib/libwinpr2.so.2+0x107974)
    #2 0x7f7969742187 in CreateThread (/opt/freerdp-nightly/bin/../lib/libwinpr2.so.2+0x108187)
    #3 0x7f796a5d8453  (/opt/freerdp-nightly/bin/../lib/libfreerdp-client2.so.2+0x11e453)
    #4 0x7f796a0bad8b  (/opt/freerdp-nightly/bin/../lib/libfreerdp2.so.2+0x14ad8b)
    #5 0x7f796a0b5a2a in freerdp_connect (/opt/freerdp-nightly/bin/../lib/libfreerdp2.so.2+0x145a2a)
    #6 0x442d74  (/opt/freerdp-nightly/bin/xfreerdp+0x442d74)
    #7 0x7f7969741ce1  (/opt/freerdp-nightly/bin/../lib/libwinpr2.so.2+0x107ce1)
    #8 0x7f796905a6b9 in start_thread (/lib/x86_64-linux-gnu/libpthread.so.0+0x76b9)

Thread T1 created by T0 here:
    #0 0x7f796bc58253 in pthread_create (/usr/lib/x86_64-linux-gnu/libasan.so.2+0x36253)
    #1 0x7f7969741974  (/opt/freerdp-nightly/bin/../lib/libwinpr2.so.2+0x107974)
    #2 0x7f7969742187 in CreateThread (/opt/freerdp-nightly/bin/../lib/libwinpr2.so.2+0x108187)
    #3 0x43dacf  (/opt/freerdp-nightly/bin/xfreerdp+0x43dacf)
    #4 0x4079b0  (/opt/freerdp-nightly/bin/xfreerdp+0x4079b0)
    #5 0x7f796929082f in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x2082f)

==31636==ABORTING

Expected behavior

Should not crash
Probably another issue, but microphone redirection should work :) (maybe one should use microphone:sys:pulse instead of microphone:sys:alsa or some Windows config... anyway, due to the Crashing I could never really test)

** Application details

Version of FreeRDP: Today's nightly.
Command line used: See above.
OS version connecting to: Windows 10

Desktop (please complete the following information):
Client is: Ubuntu 16.04 using the Nightly builds.

Duplicate of #4779


Don't show 'Open resource in Storage Explorer' error action on Linux (Codespaces)

If Storage Explorer is required in Linux, an error is still thrown but the action button to "Open resource in Storage Explorer" is no longer shown.

Mac/Windows
Linux/Codespaces

Fixes https://github.com/microsoft/vscode-azurestorage/issues/896

Should we add a link directing the user to the Storage Explorer download page? https://azure.microsoft.com/en-us/features/storage-explorer/

Is it better to add a link directing the user to the Storage Explorer download page instead of completely removing a link? https://azure.microsoft.com/en-us/features/storage-explorer/

I dig it 👍


Downloading two files with the same name

FTP Server OS: Unix
FTP Server Type: Pure-FTPd
Client Computer OS: Windows
FluentFTP Version: 51.0.0
Framework: .NET 8
I get an exception every time I try to download two files with the same name in different directories:
            await asyncFtpClient.GetFiles(new string[] { "20240807.log", "log\\20240807.log" });

. In my business case, I don't control the names, so I can't avoid this.
Logs :

>         Connect(False)
Response: 220 You will be disconnected after 15 minutes of inactivity. [739113,484d]
Status:   Detected FTP server: PureFTPd
Command:  USER ***
Response: 331 User *** OK. Password required [3ms]
Command:  PASS ***
Response: 230 OK. Current directory is / [6ms]
Command:  FEAT
Response: 211 End. [2ms]
Text encoding: System.Text.UTF8Encoding+UTF8EncodingSealed
Command:  OPTS UTF8 ON
Response: 200 OK, UTF-8 enabled [1ms]
Command:  SYST
Response: 215 UNIX Type: L8 [2ms]
Command:  PWD
Response: 257 "/" is your current location [1ms]
>         GetListing(null, Auto)
Command:  TYPE I
Response: 200 TYPE is now 8-bit binary [2ms]
>         OpenDataStreamAsync("MLSD /", 0)
>         OpenPassiveDataStreamAsync(AutoPassive, "MLSD /", 0)
Command:  EPSV
Response: 229 Extended Passive mode OK (|||30001|) [2ms]
Command:  MLSD /
Response: 150 Accepted data connection [2ms]
>         CloseDataStream()
Response: 226 7 matches total [12ms]
>         DownloadFiles("c:\deviceftpclient\download\a", System.String[], Overwrite, None)
       System.ArgumentException : An item with the same key has already been added. Key:

c:\deviceftpclient\download\a\20240807.log
       Stack Trace:
            at System.Collections.Generic.Dictionary`2.TryInsert(TKey key, TValue value, InsertionBehavior behavior)
            at System.Collections.Generic.Dictionary`2.Add(TKey key, TValue value)
            at FluentFTP.Client.BaseClient.BaseFtpClient.RecordFileToDownload(List`1 rules, List`1 results, Dictionary`2 shouldExist, List`1 toDownload, FtpListItem remoteFile, String localFile, String remoteFilePath)
            at FluentFTP.Client.BaseClient.BaseFtpClient.GetFilesToDownload2(String localFolder, IEnumerable`1 remotePaths, List`1 rules, List`1 results, Dictionary`2 shouldExist)
            at FluentFTP.AsyncFtpClient.DownloadFiles(String localDir, IEnumerable`1 remotePaths, FtpLocalExists existsMode, FtpVerify verifyOptions, FtpError errorHandling, CancellationToken token, IProgress`1 progress, List`1 rules)

In my business case, I don't control the names, so I can't avoid this

Ok. Have you looked at the src code to see if there is an obvious change that would fix this? It seems obvious what is happening. Perhaps you would want to contribute? Especially as it seems to detriment your business?
You do have a number of ways to circumvent this yourself, if you are inventive. Don't put files with the same filename part of the path into the same GetFiles API operation, do those in a second (or third or....). You could check this yourself.
Still, I consider this to be a bug to be fixed.

I realised that I do not expect a different way of working. This seems to be in line with the way the FTP listen command works.

I realised that I do not expect a different way of working. This seems to be in line with the way the FTP listen command works.

Not sure what you mean by that. I will reopen this. As soon as someone has time, depending on proirities, it is sure to be looked at in more detail and will be investigated.

Ok, I have analysed the code and the behaviour.
The description of the DownloadFiles API is perhaps a little misleading.
		/// <summary>
		/// Downloads the specified files into a local single directory.
		/// High-level API that takes care of various edge cases internally.
		/// Supports very large files since it downloads data in chunks.
		/// Same speed as <see cref="o:DownloadFile"/>.
		/// </summary>

Note the line: Downloads the specified files into a local single directory.
Repeat: A single local directory
So, if you are downloading, say { "test.log", "log\test.log" } into "C:\destinationdir" he will try to put "test.log" as "test.log" there, and then try to put "log\test.log" there, also as "test.log".
There is no proviso for recursion

Anyway, this is worth closing as is.


on newer aframe versions objects dont show up anymore

Hi,
It looks like it does not work on the newer aframe versions. aframe 0.7.0 is fine. aframe 1.0.0 shows no objects.
Is there a conflict?
Best regards

Up on this. Same issue it shows no object on the new aframe versions

Hi! I have not tested the latest versions of A-Frame. I may or may not get around to it sometime, but I would love to merge a pull request! :^)
You may want to check and see if the mixin or pool APIs have changed. Can you use the inspector to confirm if the element exists in the scene?


dev env: use docker-compose to run bdb/tm integration

[x] provide docker-compose file for single node
[x] provide docker-compose file for a 4 node cluster
[x] document single node and multi node usages, for dev environments
[x] use "custom" dockerfile for tendermint instead of dockerhub one

All tasks done. I assume. Please re-open if needed. Closing for now.


Sean simmons progress/bugfix/precommand

Description

Related Issue

Types of changes

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

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have run the pre-merge tests locally and they pass.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[x] All commits have been signed-off for the Developer Certificate of Origin.

What is the bug this is fixing?


Added entry for endpointUrl in appsettings.json instead of using connectionString twice

What
Added entry for endpointUrl in appsettings.json instead of using connectionString twice
Why
https://skype.visualstudio.com/SPOOL/_workitems/edit/2691366
How Tested
Process & policy checklist

[ ] I have updated the project documentation to reflect my changes if necessary.
[ ] I have read the CONTRIBUTING documentation.

Is this a breaking change?

[ ] This change causes current functionality to break.

Sry, I am kinda late to the conversation, I remember we were discussing we want to use endpoint instead of endpointUrl, any chance we could make that change?


Update docstring for CSVDiagnosticOutput class

In the docstring, the parameter diagnostic_size is listed as being of type int. It actually should be a tuple, which gives the size of the buffer.
https://github.com/NRL-Plasma-Physics-Division/turbopy/blob/5722606c2dc5800aaa850af93e28bb1c90fc63af/turbopy/diagnostics.py#L23
As can be seen in the append function, the first axis of the buffer is the "time", and the second is the "space" (which can be just a single point).
def append(self, data):
    self.buffer[self.buffer_index, :] = data
    self.buffer_index += 1

So diagnostic_size is type (int, int), where the first is the number of time points, and the second is the number of spatial points.

I can do this real quick.

#31 Fulfilled this issue


Is it possible to add chat history to extractor workflow?

I have the following config.yml. Is it possible to add chat history to the template?
writable: true
path: index

embeddings:
  path: sentence-transformers/multi-qa-mpnet-base-dot-v1
  content: true

extractor:
  path: google/flan-t5-base
  context: 3
  output: flatten

workflow:
  search:
    tasks:
      - task: extractor
        template: |
          Chat history:
          {history}
          Answer the following question using only the context below and the above chat history. Say "I don't have data on that" when the question can't be answered.
          Question:
          {text}
          Context:

        action: extractor

I'll have to take a look and try it out. I'm assuming you tried this and it didn't work as expected?

I just checked in a change that should make this possible. You can install txtai from GitHub to give it a try.

Will be released in 6.2.0


Adaptive Card Designer - toolbar buttons missing html type attribute

Similar to https://github.com/microsoft/BotFramework-WebChat/issues/594, the buttons in the adaptive card designer are missing the type="button" attribute which means if it's embedded in a html form tag it tries to submit the form anytime buttons like the "new card" button is pushed.

@dclaux are you able to take a look at this please? Thanks

@golddove is this for Cisco by anychance? If so can you take it with the Cisco label? thanks


Add Table Card

Proposed changes

I'm developing a voice app with Jovo and wanted a Google rich response that includes the new Table card. The Jovo contribute page lists "Table Card" as the first bullet under "Add Google Action Features", so I decided to try adding this new feature myself.
Please verify that all steps of the checklist are fully complete.
Note: The table card feature is currently in developer preview. During the developer preview, you can test responses with table cards in the Action Console simulator, but users will not see the cards in the production version.
Types of changes

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

Checklist

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

Awesome! Thanks @donth77


Ability to clear datepicker by textbox only

Changed event to keydown instead of keypress as backspace was not detected. Date is cleared when editing textbox now until the next enter keypress.

Thanks again. :smile:


terminate on demand configuration problem

Did anyone try on demand termination? I think there is a typo between configuration property and BasicChaosMonkey.java.
In configuration property chaos.properties,the sample property is
//## on-demand termination of chaos monkey
//##simianarmy.chaos.terminationOndemand.enabled = false
while in the java code, it is
Line 123:        String prop = NS + "terminateOndemand.enabled";
check the word "termination" and "terminate".

Closed ... no longer an issue, there are no "terminationOndemand" properties in the source.