Updated dependencies

Updated most of the dependencies, with the exception of
$ npm outdated
Package                    Current  Wanted  Latest  Location     Comment
babel-loader                 7.1.1   7.1.1   7.1.2  maputnik     Locked to 7.1.1 see <https://github.com/babel/babel-loader/issues/505>
jsonlint                     1.6.2     git     git  maputnik     Not required
maputnik-design              0.1.0     git     git  maputnik     Not required
ol-mapbox-style              1.0.2   1.0.2   2.6.6  maputnik     Issues that need resolving with ES6 syntax in the module
stylelint                   7.13.0  7.13.0   8.2.0  maputnik     We need to switch to a scss aware linter
stylelint-config-standard   15.0.1  15.0.1  17.0.0  maputnik     We need to switch to a scss aware linter

I've fixed the failing tests and also removed nodejs v4 for the time being. As it was causing all the tests to fail. There is some odd error with the install in v4 which I'll look into later in the week.

Removed nodejs v4 tests temporarily

The latest branch build is here https://44-84182601-gh.circle-artifacts.com/0/tmp/circle-artifacts.NgEXZgj/build/index.html
@keum you kindly offered to test maputnik over in ticket #181. In this pull request I've updated most of the library dependencies maputnik uses. Given our lack of test coverage it'd be great if you could give it a test before I merge the changes. Basically we're looking for anything that is broken now and wasn't in the current live version at https://maputnik.github.io/
So basically test the app here https://44-84182601-gh.circle-artifacts.com/0/tmp/circle-artifacts.NgEXZgj/build/index.html and leave any feedback in this ticket. Any data will be lost so make sure you export any styles you want to keep.

@orangemug One thing I noticed was that when Mapzen basema, Tilezen option doesn't show on yours compared to https://maputnik.github.io/editor/#12.24/47.3718/8.5481
also both in yours and original maptunik editor (maptunik.github.io/editor) when trying to add in Gallery Styles, Mapbox's basemaps (satellite, bright, basic) won't show up. I'm not sure what i'm doing wrong but I tried to load using Chrome and Firefox quantum giving same no show.
I've added my own vector tiles from DigitalOcean and styled them and export to preview and no problem with it.

One thing I noticed was that when Mapzen basema, Tilezen option doesn't show

Good spot, that's correct, we removed that in #172 because it was no longer functional. We have a ticket to add it back in #153.

Mapbox's basemaps (satellite, bright, basic) won't show up

That'll be due to permisisons of the mapbox key. So not an issue but thanks for raising it.

I've added my own vector tiles from DigitalOcean and styled them and export to preview and no problem with it.

Thats what I like to hear :)
Thanks for testing @keum, the hard work is much appreciated.

For future reference we also removed node.js v4/v5 support, node v5 is no longer supported and v4 was causing issues. Since v4 is in maintenance due to end April 2018 (see https://github.com/nodejs/Release) I decided to remove support.
There were also some breaking API changes to @mapbox/mapbox-gl-style-spec in a minor version release. Basically styleSpec.latest.source_tile was renamed. Due to travis caching of node_modules this didn't initially show up as a broken build. However a fresh install of maputnik failed to build (see https://github.com/maputnik/editor/pull/192/commits/cb2f854dd5bdafdf393bf3753e6b4a8960fccf9c#diff-b901d42e1ea5b8ba1ea7465908999e83L15)


Apply custom message in login dialog on direct access

Fixes https://github.com/FlowFuse/node-red/issues/25
This PR removes the username/password fields from the login dialog when the editor is accessed directly. We get support requests asking what credentials to use - however we only support accessing the editor via the device agent proxy.
With NR 4.0.6 (to be released imminently), I've added the ability to insert a custom message/button on the login dialog when in this state. Older versions of NR will show a blank dialog.

Note: spotted the dashbaord typo when I took this screenshot for the PR... it will be fixed momentarily.

@knolleary not sure if you noticed test fail following removal of type?

@Steve-Mcl test fix pushed

Aesthetically, I do wish the string didn't wrap on the last word (but at the same time I cant think of a better way to phrase it)

Perhaps 'Access to the editor is managed via FlowFuse' would be a suitable (shorter, non wrapping) alternative?

Have updated the message to Access the editor through the FlowFuse platform


Change app icon

Change icone and app name to differentiate yourself from wled-aircookie version. Very cool way to learn c##.

I updated the icon on Android already, iOS and UWP will follow soon! With regards to the name, I wouldn't really know how to name it, but I am open to suggestions!


feat(lwm2m): always publish update message

https://github.com/emqx/emqx-lwm2m/issues/90
NOTE: This PR is for enterprise emqx, and should be migrate to the open source emqx at branch dev/v4.3.0 after merged.

Please make the pr in emqx.git, we'll merge it back to enterprise soon

Please make the pr in emqx.git, we'll merge it back to enterprise soon

OK, then I close this and open another PR on emqx project.


Implement binary format for POIs

Description
We should return a binary endpoint for the POIs, similar to the binary endpoints in the Core Node API.
Motivation

Should save some space.
Requirements

Define format.
Implement it.
Document the endpoint in the Swagger file.

Open questions (optional)

Settle on format.
Investigate file size improvements.

Are you planning to do it yourself in a pull request?
No.

We came to the conclusion that a binary format is not worth pursuing until there's an actual demand for it. A custom format has the potential of getting rid of some redundancies, but there's no empirical data yet about certain usage patterns. JSON on the other hand is an ubiquitous, future-proof, and widely supported format, that better guarantess that such proofs of inclusion can be validated in the future. If storing them is of any concern then clients can still convert and compress them as they see fit using standard algorithms.


Mobile phone code update needed for Malaysia

Just a short notice regarding MY mobile phone codes, currently the mask is as follow:
{ "mask": "+60-##-###-####", "cc": "MY", "cd": "Malaysia ", "desc_en": "mobile", "name_ru": "Малайзия ", "desc_ru": "мобильные" },
But according to one of my customer who leaves in Malaysia (and an article published here), new mobile phone numbers are formatted on 11-digit.
Hope this helps.

@jnaubry ,
Thx, I updated the phone.js


version control

For not changing the version in multiple places, a variable ${AMIDSTver} has been created. This variable is defined in the properties of the parent pom file. This is a valid solution when using the code via github, however, it does not work if you try to use AMIDST through maven. To make it work, this variable should be replaced by a constant before deploying:
sed -i '' 's/${AMIDSTver}/0.4.2/g' */pom.xml
sed -i '' 's/${AMIDSTver}/0.4.2/g' pom.xml
We should look another method for doing the version control without making this replacement.

This variable produced many problems. The amidst version always contains a comment:
        <version>0.6.0</version> <!-- AmidstVersion -->


Update deployments to apps/v1

We need to update the deployments to apps/v1 to work with Kubernetes 1.16 and later
The change is also needed in the chart -> https://github.com/openfaas/faas-netes/tree/master/chart/cron-connector

I can take this.

/assign: me

Related PR which is missing the matchSelectors work - https://github.com/zeerorg/cron-connector/pull/22/files

I added Derek to the repo which was missing.

/assign: Waterdrips


API for changing debounce time, lift off distance and angle snapping

Some mice, for example Sinowealth-based, support changing debounce time, lift off distance and angle snapping in their configuration software. It would be nice if this was possible to do in libratbag.

Angle snapping and debounce time were implemented.
For lift-off distance I don't have any use-case and won't work on it myself, but if anyone sends a PR, I'd be happy to review it.


React 0.13 support?

Currently, there are a few warnings:
Warning: owner-based and parent-based contexts differ (values: `[object Object]` vs `[object Object]`) for key (layoutContext) while mounting Layout
(see: http://fb.me/react-context-by-parent)

This shouldn't really create a problem, in fact this module simulates parent-based context by injecting the layoutContext prop into children. Once owner-based context is a thing of the past then some code can actually be deleted.


actionSheet issue with presentPicker Xcode 11

Hello,
I'm trying to use presentPicker with action sheet, it works but when I return app, title and message stays and can't close the action sheet. Here is my code
@IBAction func navigationTapped(_ sender: Any) {
       let coordinate = CLLocationCoordinate2D(latitude: 52.5162746, longitude: 13.3755153)
       let berlin = Karte.Location(name: "Brandenburger Tor Berlin", coordinate: coordinate)
       let actionSheetVC = UIAlertController(title: "Title", message: "Please Select an Option", preferredStyle: .actionSheet)

       self.present(actionSheetVC, animated: true)
       Karte.presentPicker(destination: berlin, presentOn: actionSheetVC)
   }

When I clicked button it shows up available maps;

When I return app screen looks like this;

Am I using action sheet with wrong way?

Hi 👋
Karte already creates the UIAlertController using the actionSheet style for you, there's no need to create one manually and then try and present Karte on top of that.
Try doing something like the following:
@IBAction func navigationTapped(_ sender: Any) {
       let coordinate = CLLocationCoordinate2D(latitude: 52.5162746, longitude: 13.3755153)
       let berlin = Karte.Location(name: "Brandenburger Tor Berlin", coordinate: coordinate)

       Karte.presentPicker(destination: berlin, presentOn: self)
   }

Hi,
Thank you, your solution worked :)


updated client lib to apiv2

@andhess Tests failed

how do i set env variables in test?

@andhess We use TravisCI, so you can add the env variable to .travis.yml if it's public or through the interface if it's secret


[SPARK-49009][SQL][PYTHON] Make Column APIs and functions accept Enums

What changes were proposed in this pull request?
Make Column APIs and functions accept Enums.
Why are the changes needed?
Enums can be accepted in Column APIs and functions using its value.
>>> from pyspark.sql import functions as F
>>> from enum import Enum
>>> class A(Enum):
...   X = "x"
...   Y = "y"
...
>>> F.lit(A.X)
Column<'x'>
>>> F.lit(A.X) + A.Y
Column<'`+`(x, y)'>

Does this PR introduce any user-facing change?
Yes, Python's Enums will be used as literal values.
How was this patch tested?
Added the related tests.
Was this patch authored or co-authored using generative AI tooling?
No.

The failure seems not related to this PR.

Thanks! merging to master.


Add new test for nested IDs

This PR adds a new test for nested IDs as noted by @Zegnat in #120.

I wonder if there is something to win by also including a "negative" test?

The d should not show up in the microformats output then.
Also wondering if we want to accept tests that make no sense content wise. Not even the microformats of this example make any sense, even if they are completely valid for testing the parser edge-cases. Not sure what our policy on that should be.

I agree that real-world examples (or at least real-world adjacent) examples are more helpful. @Zegnat Want to update your example and I can update the test files?

@Zegnat Wanted to follow up on this. Looking back at my comment above, do you want to update your example and I can update the test files? Or, is this PR in an acceptable place to merge?

@Zegnat Wanted to follow up on this. Looking back at my comment above, do you want to update your example and I can update the test files? Or, is this PR in an acceptable place to merge?


Remove ConfigureSession

cc @Tratcher

:+1: @HaoK if you take care about this https://github.com/aspnet/Session/blob/dev/samples/SessionSample/Startup.cs#L41 too

This is for: https://github.com/aspnet/Session/issues/58
:shipit:

2bd83e15894ddfaff1f96a4e04488cba2424e690


Help page in admin section

Add a help or about page to the admin section contenting information about the hires system possibly including:

Link to github repo
Email to webmaster

Need webmaster radio option in superadmin user page

Could this be on the home page? The home page is not great at the moment.
Perhaps home page could be something like

Introduction to the site
Alphabetized list of the colleges/societies (with crests?)
Information about the system itself

The home page definitely needs a re-deisng, the current one was very much just a place holder! I'll move this to a separate issue.


Module-qualify test IDs

The new test discovery returns IDs for all the tests it discovers but those IDs currently aren’t qualified by the module that contains the tests. We should add the module-qualification. For this, we probably need to implement a build system query that returns the targets a source file is part of.
rdar://125552485

Implemented in https://github.com/swiftlang/sourcekit-lsp/pull/1530


Remove default font-family rule

See: https://github.com/necolas/normalize.css/issues/517.

Fixed in #20.


New --cp option to copy command(s) to another directory

There will likely be a more elegant alternative to this once #124 is finished.


WIP: markupchart plugin controls migration

SUMMARY
remove controls from incubator, remove shims and update version number.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #9806 into master will increase coverage by 6.48%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #9806      +/-   ##
==========================================
+ Coverage   64.31%   70.79%   +6.48%
==========================================
  Files         536      184     -352
  Lines       29109    17863   -11246
  Branches     2806        0    -2806
==========================================
- Hits        18721    12647    -6074
+ Misses      10208     5216    -4992
+ Partials      180        0     -180

Flag
Coverage Δ

#cypress
?

#python
70.79% <ø> (-0.23%)
:arrow_down:

Impacted Files
Coverage Δ

superset/views/database/mixins.py
58.92% <0.00%> (-21.43%)
:arrow_down:

superset/utils/cache.py
45.83% <0.00%> (-20.84%)
:arrow_down:

superset/db_engine_specs/mysql.py
78.26% <0.00%> (-13.05%)
:arrow_down:

superset/views/database/api.py
83.90% <0.00%> (-3.45%)
:arrow_down:

superset/db_engine_specs/postgres.py
97.29% <0.00%> (-2.71%)
:arrow_down:

superset/models/core.py
83.77% <0.00%> (-2.36%)
:arrow_down:

superset/jinja_context.py
84.69% <0.00%> (-1.03%)
:arrow_down:

superset/security/manager.py
88.62% <0.00%> (-0.35%)
:arrow_down:

superset/views/core.py
74.90% <0.00%> (-0.23%)
:arrow_down:

superset/connectors/sqla/models.py
88.41% <0.00%> (-0.16%)
:arrow_down:

... and 350 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 3cc5400...023df7e. Read the comment docs.


Should the special keyword balance be allowed in the species dictionary to indicate the remaining mole fraction?

PyKED/ChemKED version
v0.1.5
Title question suggested by Mike Burke's group at Columbia

My initial thought is no, that the user can do the subtraction and put in a number. This might be prone to typos, but it will be the easiest to handle on our end. Other thoughts?

Yeah this seems like a "lazy" (or, more charitably, "convenience") option that doesn't provide much value—explicit is better than implicit :)
Although requiring the amount to be specified, I think forcing this actually would lead to fewer overall errors, because our validation will check that everything adds up correctly. With a balance option, a typo in another value won't be caught.

With a balance option, a typo in another value won't be caught.

Good point! Although I was doing some work today and found a typo that I made where I did the rounding incorrectly, so YMMV :smile:

Hah ok, well I can't guarantee saving you from all your typos :)

From Mike Burke via email:

Many papers only report an Ar balance, for example, rather than the mole fraction of Ar.  I personally see value in allowing specification of balance rather than a mole fraction value from the standpoint of uncertainty quantification.  Let's say you're evaluating the effect of having one of the reactants at the upper uncertainty limit of their nominal mole fractions, then to get the mole fractions to sum to 1, you'd need to mole fraction of the Ar balance to go down to compensate.  Thoughts?

That's a pretty good point. Is there some way we can incorporate Kyle's comment about catching typos, and this uncertainty idea? Perhaps a new field balance-species in the composition block?


B&W button on the gradient example seems to have counterintuitive effect

To me, the black and white button on the gradient examle should have effect on the entire gradient. Instead, only one colorStop of the gradient is set to b&w. This could be a little bug or just my perception so I’ll leave this in the middle.

It's not a bug it's an effect. The gradient animates from left to right and when you change the color mode it transitions from last color value to the next black and white value. Otherwise it would be a hard transition.


Updated calculation of MRR history to use mrr column

refs https://github.com/TryGhost/Team/issues/1518

Uses mrr column now
Doesn't account for cancel_at_period_end until a migration fixes the MRR values: https://github.com/TryGhost/Ghost/pull/14480

The snapshot of one of the tests failed because one of the test fixtures created a subscription with a MRR set while cancel_at_period_end was true. Depending on the dashboardv5 flag, that could have been a non-zero MRR or not IRL... So tests are a bit hacky here.


NaNs in pandas dataframes for optional fields hamper object construction

Describe the bug
Various optional fields in the FM model input will be represented as nan when exported to a pandas DataFrame.
When constructing a new object from such dataframe, unnecessary validation errors will occur, and the object is not created.
(reported in #199)
To Reproduce
Steps to reproduce the behavior:

Run this script:

from pathlib import Path
import pandas as pd
from hydrolib.core.io.crosssection.models import CrossDefModel
from hydrolib.core.io.dimr.models import DIMR, FMComponent
from hydrolib.core.io.mdu.models import FMModel

# Specify paths
root = Path.cwd()
modeldir = Path(
    "c:/checkouts/HYDROLIB-core_git/tests/data/input/e02/f152_1d2d_projectmodels_rhu/c04_DHydamo-MGB-initialisation/fm/"
)
mdufile = "moergestels_broek.mdu"

fm = FMModel(modeldir / mdufile)

cross_def = pd.DataFrame([cs.__dict__ for cs in fm.geometry.crossdeffile.definition])

definition = cross_def.to_dict("records")

crossdef_new = CrossDefModel(definition=cross_def.to_dict("records"))

At the last line, an exception will occur that follows from our custom enum validation, such as:

'float' object has no attribute 'lower'
  File "C:\checkouts\HYDROLIB-core_git\hydrolib\core\io\ini\util.py", line 48, in get_enum
    if entry.lower() == v.lower():

(variable v is nan here)
3. And, when temporarily switching off this enum validation in CrossDefModel, then still the object creation gives validation errors follows from our custom enum validation, such as:
471 validation errors for CrossDefModel
definition -> 0 -> circ_d0.300 -> frictionType
  value is not a valid enumeration member; permitted: 'Chezy', 'Manning', 'wallLawNikuradse', 'WhiteColebrook', 'StricklerNikuradse', 'Strickler', 'deBosBijkerk' (type=type_error.enum; enum_values=[<FrictionType.chezy: 'Chezy'>, <FrictionType.manning: 'Manning'>, <FrictionType.walllawnikuradse: 'wallLawNikuradse'>, <FrictionType.whitecolebrook: 'WhiteColebrook'>, <FrictionType.stricklernikuradse: 'StricklerNikuradse'>, <FrictionType.strickler: 'Strickler'>, <FrictionType.debosbijkerk: 'deBosBijkerk'>])

Expected behavior
Optional fields read from input may be represented by nan in pandas DataFrame, but subsequent object creation must directly be possible via CrossDefModel(definition=cross_def.to_dict("records")).
Screenshots
If applicable, add screenshots to help explain your problem.
Version info (please complete the following information):

OS: [e.g. Windows]
Version [e.g. 0.1.5]

Additional context
Add any other context about the problem here.

This behaviour is warned for in https://deltares.github.io/HYDROLIB-core/0.2.0/tutorials/dataframe_handling/, although it is not solved in that example.
Fields that are None are automatically converted to NaN by pandas so the column is all numbers, and thus fast to operate on. I'd say it's up to the user when to convert these NaNs (they could be valid).
This is easily done by something like this:
cross_def.where(pd.notnull(cross_def),None).to_dict()

Or, if the user isn't concerned about performance, you create a DataFrame of objects
pd.DataFrame([cs.__dict__ for cs in fm.geometry.crossdeffile.definition], dtype=object)

In essence this is caused by pandas only using 1 value for missing data (NaN) for performance.

The above is a workaround, to fully fix this I propose:

DataFrame method (mixin class?) that sets the correct dtypes for each column on the relevant classes
Expand tutorial with warning for NaNs

@LisaWeijers: can you let us know whether completely skipping the dataframes, and directly making a true copy of the CrossDefModel is also working well for you?, so using @evetion's suggestion above:

Note that DataFrames are not needed for (temporary) copies of objects, you can just use object.copy(deep=True) (see https://pydantic-docs.helpmanual.io/usage/exporting_models/#modelcopy).

This is easily done by something like this:
cross_def.where(pd.notnull(cross_def),None).to_dict()

In #255 I discovered that some scalar float columns were not accepting None, so first the dataframe must be cast into object type, as follows:
df2=df_structures.astype(object)
df2.where(pd.notnull(df2),None, inplace=True)


repair a comment

repair a comment
Motivation:
the comment of [subHealthToRetry] method may be wrong, repair it
Modification:
repair the comment of [subHealthToRetry] method
Result:
repair the comment of [subHealthToRetry] method

Codecov Report

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

@@             Coverage Diff              @@
##             master     #198      +/-   ##
============================================
- Coverage     71.26%   71.22%   -0.05%
+ Complexity      958      957       -1
============================================
  Files           341      341
  Lines         14213    14213
  Branches       2286     2286
============================================
- Hits          10129    10123       -6
- Misses         2891     2898       +7
+ Partials       1193     1192       -1

Impacted Files
Coverage Δ
Complexity Δ

...ay/sofa/rpc/client/AllConnectConnectionHolder.java
63.12% <ø> (ø)
0 <0> (ø)
:arrow_down:

...m/alipay/sofa/rpc/module/FaultToleranceModule.java
66.66% <0%> (-26.67%)
4% <0%> (-1%)

.../main/java/com/alipay/sofa/rpc/tracer/Tracers.java
39.74% <0%> (-2.57%)
0% <0%> (ø)

...n/java/com/alipay/sofa/rpc/common/SofaConfigs.java
83.92% <0%> (-1.79%)
0% <0%> (ø)

...ava/com/alipay/sofa/rpc/config/RegistryConfig.java
89.21% <0%> (-0.99%)
0% <0%> (ø)

.../alipay/sofa/rpc/registry/local/LocalRegistry.java
57.22% <0%> (-0.61%)
25% <0%> (-1%)

...ent/aft/impl/ServiceHorizontalMeasureStrategy.java
86.66% <0%> (+2.22%)
27% <0%> (+1%)
:arrow_up:

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 37283b7...487842c. Read the comment docs.

Why close #197 ?  You can also fix it in this PR.

@ujjboy OK


feat: Insert to File button for code block

Add insert to file button to code block](feat: Insert to File button for code block)
This commit adds a new insertButtonOnSubmit prop to the Chat and CodeBlocks components. This allows passing in a callback function that will be triggered when the insert button is clicked.
The CopyButtonProps interface is updated to have separate callbacks for copy and insert actions. The CodeBlocks component is updated to conditionally render insert and insert new file buttons if the callback props are passed in.
This enables IDE integrations to handle insert actions separately from copy actions.
When Insert to File button is pressed, user will be prompted to select a new file / existing file to insert the code to. See demo video below for details

Test plan

start Cody from this branch
click on "Insert to File" button from code block

https://github.com/sourcegraph/cody/assets/68532117/8ef71945-4c95-467b-ba67-b5462f0d457e

Nice one @abeatrix!
This is great for inserting into a new file, but it feels too awkward when wanting to insert into an existing file. The interaction of choosing a file in the finder window is normally for overriding files, not appending.
Have you seen my proposal for "Insert to File" in https://www.figma.com/file/9112BsKsJc1BpO2j8XYwLL/🤖-Cody-VS-Code-[Main]?type=design&node-id=5110-7877&mode=design&t=Zrf09QUxBQz8XXP2-4? For appending, I think we need a quickpick in the middle that lists any open tabs. Would love to get your feedback and iterate on that together, and then we update this depending on where we land.

@toolmantim yea I agree! I can limit this to "Save to new file" only, and then add the "insert into files" in a different PR since that will be a different button, along with your new design, what do you think?
(I can't open figma on my phone so I'll have to wait til tomorrow to leave feedback on it, sorry! 😆)

@abeatrix it's a single button in my design, so that might not make sense. But we can discuss tomorrow!

Pushed some design updates!
Turns out the icons I had in Figma were the wrong size to be pixel aligned (they should have been 16x16) so I updated them to use the SVGs directly from the codicons repo.
Updated the PR name, description and changelog to let people know that these buttons have moved to the bottom of the code block too

Thank you! I'll make a final run of the PR in a bit and merge it. Thanks for the improvement @toolmantim


syncing from master to master-v2

Fix conflict before syncing master into master-v2

This should be good to go now

@okalouti can you update this PR again and merge?


Intro To Pytorch Download Issues

In the intro to pytorch section, the references to download the MNIST data set fail.  It would appear, this is tied to the pytorch issue, https://github.com/pytorch/vision/issues/1938 .  There is a workaround that is used in the convolutional neural networks section, proposing that this is also used in the intro to pytorch notebooks that download the MNIST data. https://github.com/udacity/deep-learning-v2-pytorch/blob/master/convolutional-neural-networks/mnist-mlp/mnist_mlp_exercise.ipynb

This issue is also relevant to the autoencoder segment, where mnist data is used.

Thank you for the PR.


Refactor librosa compatibility test

The number of unit tests we have in torchaudio has grown a lot. It is critical to keep tests organized so as to make it easy for contributors to write tests for the feature they are working on.
librosa consistency tests are for checking that torchaudio's implementations (functional/transform) are numerically comparable against the librosa's implementation.
The list of functional has grown, so we now have a dedicated test submodule for functional. We would like to move librosa consistency tests for functionals to this module.
Steps

Create a librosa_compatibility_test.py in test/torchaudio_unittest/functional
Move the TestFunctional and TestPhaseVocoder to the new script. (fix import and stuff)

Building and testing
Please refer to CONTRIBUTING for build process. You can check that the tests success with
(cd test && pytest torchaudio_unittest/functional/librosa_compatibility_test.py torchaudio_unittest/librosa_compatibility_test.py)

Temporarily assigning @kychow

Hi @mthrok, @kychow,  I have created a PR for this issue.

@AzizCode92
Thanks for the contribution. The PR looks good but let me wait for @kychow 's response a bit. I would like to stick to first-come first-served principle for fairness.

Sure.
thanks


"Very Fragmented Ephemeral" condemn reason is incorrect

In Condemned reasons for GCs section of GCStats view, there is a column named "Very Fragmented Ephemeral" with an additional "Ephemeral generations are VERY fragmented, doing a full GC" description.
This is however incorrect. This reason internally refers to gen_max_high_frag_e_p condition, which is set if fragmentation of generation 2 (max_generation, see dt_high_frag_p  below) is big enough to fit generation 1:
if (dt_low_ephemeral_space_p (check_only_p ? tuning_deciding_full_gc : tuning_deciding_condemned_gen))
{
	...
	local_condemn_reasons->set_condition (gen_low_ephemeral_p);
	...

	//It is better to defragment first if we are running out of space for
	//the ephemeral generation but we have enough fragmentation to make up for it
	//in the non ephemeral generation. Essentially we are trading a gen2 for
	// having to expand heap in ephemeral collections.
	if (dt_high_frag_p (tuning_deciding_condemned_gen,
						max_generation - 1,
						TRUE))
	{
		high_fragmentation = TRUE;
		local_condemn_reasons->set_condition (gen_max_high_frag_e_p);
	}
}

BOOL
gc_heap::dt_high_frag_p (gc_tuning_point tp, int gen_number, BOOL elevate_p)
{
	BOOL ret = FALSE;
	switch (tp)
	{
		case tuning_deciding_condemned_gen:
		{
			dynamic_data* dd = dynamic_data_of (gen_number);
			float fragmentation_burden = 0;
			if (elevate_p)
			{
				ret = (dd_fragmentation (dynamic_data_of (max_generation)) >= dd_max_size(dd));
			}
			...
		}
	}
	return ret;
}

I suggest changing the column name and description to the "Ephemeral Low with Very Fragmented Gen2" and "We are running low on the ephemeral segment but gen2 is fragmented enough to compensate that, doing a full GC".

@Maoni0 @jeffschwMSFT.   He is complaining about the logic of the CondemnedReasonsHtmlHeader in GCStats.cs.

that's fine...please feel free to submit a PR.

Sure.


Epidemiological sense & purpose

In their '10 requirements for the evaluation of "Contact Tracing" apps' the CCC requires:

The basic prerequisite is that "contact tracing" can realistically help to significantly and demonstrably reduce the number of infections. The validation of this assessment is the responsibility of epidemiology. If it turns out that "contact tracing" via app is not useful or does not fullfil the purpose, the experiment must be terminated.

Is there any validation of this assessment? We know, only 0.9% of the tested people are positive. Thus, it looks like surveillance can hardly reduce the number of infections any further? Also mind that the test's false-positive rate is in the ballpark of 0.2 - 0.5%.
Furthermore, the Stanford department for epidemiology performed a meta study summarizing the results of 24 independent Covid-19 studies. They're concluding that the infection fatality rate is only 0.2% which is in the same ballpark as the seasonal influenza.
So, for multiple reasons, it is most likely that contact tracing cannot "help to significantly reduce the number of infections". On the other hand, the potential for misuse remains. Security experts are warning:

Contact tracing apps have absolutely no value," Bruce Schneier, a privacy expert and fellow at the Center for Internet & Society at Harvard University, told BuzzFeed News. "I'm not even talking about the privacy concerns, I mean the efficacy. Does anybody think this will do something useful? ... This is just something governments want to do for the hell of it. To me, it's just techies doing techie things because they don't know what else to do."

Also, former NSA contractor Edward Snowden warns that the pandemic is misused to expand illegal state surveillance.
All in all, this surveillance app implies massive risks and neglectable gains. Therefore, we should follow the CCC's advice: "the experiment must be terminated".

Well, we haven't even started with the app - it is yet to be released. Also other countries - especially the ones relying on the decentralized approach - have just recently started or will join soon. So a scientifically valid, large-scale assessment/validation has not been possible so far. But as we also stated in our FAQ, the development and further use of the app is being closely accompanied by scientific research. All parties, especially Apple and Google have repeatedly stated that the infrastructure will be disabled as soon as it's no longer needed, see for example the Apple/Google FAQ. For now, we work under the assumption that it will work - and there are also several studies that support this assumption.
Most serious researchers don't see any indication that the pandemic is over, not even in Europe or Germany. In fact, on a world-wide scale it seems to be accelerating again. The Corona-Warn-App is one component out of many to bring down the numbers even more and to avoid that they grow again or even that a 2nd wave will hit Germany.
Moreover, the consequences of the disease are not binary (dead vs. healthy). Recent research shows that many patients will suffer for a very long time from COVID-19, even if they survived it. Thus, even if the fatality rate is that low, we might face thousands, if not millions of long-term sick or impaired people if we don't avoid infections only in Germany.
In addition, I'd like to ask you to support your statement that the Corona-Warn-App is a "surveillance app" with some facts. From our perspective, we have proven that the opposite is true. Therefore, we consider this statement as inappropriate in a professional environment as it's outlined in our Code of Conduct.
Mit freundlichen Grüßen/Best regards,
SW
Corona Warn-App Open Source Team

Many top-notch medical researchers such as these Stanford professors around Dr. Ioannidis published data which fundamentally disagrees with your perspective. Is your assessment of the situation based on scientific publications? Can you share them please? As you know, the RKI reports that last week only 0.9% have been tested positive. Also, the test's false-positive rate is in the ballpark of 0.8%. Thus, at this point, we mostly measure noise.
Furthermore, are there cut-offs to terminate the app? What value do we have to reach to shut it off? What's the goal?

I'd like to ask you to support your statement that the Corona-Warn-App is a "surveillance app"

"Surveillance of patients" is a professional medical term. What is inappropriate about it?

The Stanford article which you are referring to is a preprint, as stated prominently at the top of the article. To quote what this means: "It reports new medical research that has yet to be evaluated and so should not be used to guide clinical practice." Moreover, there are many studies currently being released - with a lot of interesting, but definitely more differentiating results than what you are implying here. That simply shows that there is no simple truth or accepted consensus among scientists at this point in time.
Moreover, the Technology Review article referenced above includes a link to the epidemiological model which supports the introduction of tracing apps. I share the link directly here for your convenience: https://www.research.ox.ac.uk/Article/2020-04-16-digital-contact-tracing-can-slow-or-even-stop-coronavirus-transmission-and-ease-us-out-of-lockdown
Concerning "surveillance" you already changed your working obviously. In the first post you are quoting Edward Snowden about "illegal state surveillance", now you are changing to "surveillance of patients". I kindly ask you to have a look at the code and the documentation - the app is neither the first nor the second.
About your request to share more publications: We are already to start going round in circles. We have nothing more to add than what we already stated in our comment above.
Mit freundlichen Grüßen/Best regards,
SW
Corona Warn-App Open Source Team

Let me simplify my question. The infection rate is already below 0.9% (Source: RKI report). Assuming a false-positive rate of 0.8% is a conservative estimate (Source: Cohen et al.). Therefore, at this point, we hardly find any infections at all.
So, my question is:
What does it mean to "significantly reduce the number of infections"?
Is the target 0.5%? Or is it it 0.1%? What is the condition for termination?

@RobinLinus These are only the people that actually get tested. This app intends to prevent further infections at an earlier time. Also superspreading events are probably easier to detect with an app and these are the majority of infections afaik.

There is no single figure or target. The reason for all these measures is gone when the respective specialists and institutions such as the Robert-Koch-Institute (RKI) or the ECDC lower the risk considerably and/or officially declare the pandemic as under control/over. Right now, there is no indication that this is or will be the case in the near future.

Quote ECDC: "A framework for contact tracing, based on extensive testing, active case finding, early detection of cases, isolation of cases, quarantine and follow-up of contacts, possibly supported by electronic tools and applications." and "A strong risk communication strategy should remind citizens that the pandemic is far from over. People need to be aware that the public health measures to limit the spread of the virus will continue to impact the way we move, work and travel, and our leisure activities for the foreseeable future. This is especially important as Europe moves towards the summer holidays, when changes in people’s behaviour, activities and movements may cause people to switch back to pre-pandemic and potentially risky behaviour patterns."
Mit freundlichen Grüßen/Best regards,
SW
Corona Warn-App Open Source Team

So, you say, the tracking is terminated only when "the Robert-Koch-Institute officially declares the pandemic as over". That is worrisome for multiple reasons:

Your target is very vage. You're working without any scientific measure.
There are already barely any new infections.
Even without an app, the infection rate has been declining rapidly for months.
For the above reasons you cannot measure if the app has any effect at all.
Obviously, any surveillance or tracking technology implies significant risks of misuse. It is ethically wrong to work with such technology without any objectively verifiable constraints.
Passing that hot potato on to the RKI leads to diffusion of responsibility.

Obviously, any surveillance or tracking technology implies significant risks of misuse. It is ethically wrong to work with such technology without any objectively verifiable constraints.

Could you please explain which surveillance or tracking technology you are talking about here? The app shown in these repositories enables users to be notified of a potential exposure. They can decide to get tested or self-quarantine. Also, a user tested positive can decide to notify his contacts through the app. Nobody is forced to do any of these things. And there is no surveillance. In fact the health authorities have already been complaining in the media that there is no new data they can get, apart from more calls and tests from people notified by the app.

Well, the RKI is officially responsible for this application as you can also see in the privacy notice of the app. There is no diffusion of responsibility - it's quite the opposite.
@RobinLinus As said, we are starting to going round in circles. Moreover, you are repeating your unproven accusations. As mentioned, this has consequences with respect to our code of conduct. This is an official warning. If you continue to repeat these false accusations in this organization, this will lead to a temporary or permanent ban.
Mit freundlichen Grüßen/Best regards,
SW
Corona Warn-App Open Source Team

Obviously, any surveillance or tracking technology implies significant risks of misuse. It is ethically wrong to work with such technology without any objectively verifiable constraints.

Could you please explain which surveillance or tracking technology you are talking about here?

Multiple exploitable attack vectors have been discussed before. Furthermore, the ExposureNotification APIs are not auditable. Such critique should be discussed openly. Downplaying the risks of misuse is irresponsible.
In particular, because our government already openly discusses to misuse your app and force people to use it.

@RobinLinus This is an official warning. If you continue to repeat these false accusations in this organization, this will lead to a temporary or permanent ban.

You said:

There is no single figure or target.

But you also say:

There is no diffusion of responsibility

How can anyone be responsible if there is no way to measure any target?

@RobinLinus Please read the previous statements exactly and don't imply things which haven't been written. Institutions such as the RKI or the ECDC include a lot of factors in their decisions when they are assessing the risk of the Coronavirus pandemic. They even explain that in detail. I really recommend reading their statements.
And for the rest it seems that we are going off-topic now. If you want to discuss the "multiple exploitable attack vectors", please open another issue.
Moreover, please understand that we are not commenting political statements or legal discussions here.
Mit freundlichen Grüßen/Best regards,
SW
Corona Warn-App Open Source Team


Quicklook recipe compose tool

Tool to compose recipes for the EMAC quicklooks
Requires #1382
Tasks

[x] Create an issue to discuss what you are going to do, if you haven't done so already (and add the link at the bottom)
[x] Give this pull request a descriptive title that can be used as a one line summary in a changelog
[x] Make sure your code is composed of functions of no more than 50 lines and uses meaningful names for variables
[ ] Circle/CI tests pass. Status can be seen below your pull request. If the tests are failing, click the link to find out why.
[ ] Preferably Codacy code quality checks pass, however a few remaining hard to solve Codacy issues are still acceptable. Status can be seen below your pull request. If there is an error, click the link to find out why. If you suspect Codacy may be wrong, please ask by commenting.
[ ] Please use yamllint to check that your YAML files do not contain mistakes

If you need help with any of the tasks above, please do not hesitate to ask by commenting in the issue or pull request.

Closes #1377

If this is still relevant to anyone, please feel free to pick it up.


[SR-12067] Need better "generic" NSMutableArray support

Previous ID
SR-12067

Radar
rdar://problem/58997127

Original Reporter
@Cyberbeni

Type
Improvement

Status
Resolved

Resolution
Won't Do

Additional Detail from JIRA

Votes
0

Component/s

Labels
Improvement

Assignee
None

Priority
Medium

md5: 662811b5d128b17bc2dcba19db898029

Issue Description:
We are using https://github.com/protocolbuffers/protobuf for our models because we still have a part of our application in Objective C.
This generates Objective C classes with properties like this:
@property(nonatomic, readwrite, strong, null_resettable) NSMutableArray<MyClass*> *propertyName;
In Swift NSMutableArray<MyClass> is invalid and you also can't assign [MyClass]() to this property, which means even though this property has it's "generic" associated type specified, you will lose that type information when interacting with it from Swift, which can cause problems if it changes and you forget that this could be a problem and only check if the app still compiles.
I think allowing NSMutableArray<MyClass> syntax just for build time checks that result in the same code as currently (and maybe a flag to enforce generic objective c type usage from swift) and also allowing automatic conversion of [MyClass] to NSMutableArray would be relatively painless and wouldn't change the behaviour of currently existing code.
Other mutable classes might also have similar issues, didn't check them.

... how NSArray<MyType *> gets safely translated to [MyType] ...

This translation or "bridging" is technically a runtime copy operation (we can often avoid the actual copy, but not always).   There's no technical reason we couldn't do the same with NSMutableArray, it's just that our current bridging logic is based around the idea that every Swift type (such as Array<>) has at most one corresponding Obj-C type (in this case NSArray).  It would require a fair bit of work to redesign it so that Swift's Array<> could interoperate with Obj-C NSArray or NSMutableArray in different contexts.

Modern Objective-C compilers will warn about misuse but Swift compiler won't warn about misuse because the type information is thrown away.

Even a "modern" Objective-C compiler does not in any real way prevent misuse.  The warning is just a warning and all you have to do is pass your NSMutableArray<MyType *> to an un-annotated NSMutableArray to lose even that minimal protection.  Swift aims to make type errors impossible and it can't do that if Objective-C code can break the type rules at any time.  (And there are still a lot of legacy Obj-C frameworks around that have never adopted Obj-C generic annotations.)
So bridging a Swift Array<> to/from an Obj-C NSMutableArray<> would require making a full defensive copy of the array every time that array passed from Swift to Objective-C code or vice-versa.  In this case, that would mean a full array copy every time you read or wrote that Obj-C property from Swift code.  It's certainly possible, but I'd be interested to know whether people think it's really worthwhile.
nsMutableArray = (swiftArray as NSArray).mutableCopy() as! NSMutableArray

One big advantage of the above syntax over the shorter form you propose is that it makes the copy operation obvious.


leave ledger key untouched when deployMutlisig

This PR filters out ledger key from keys that are converted to limited access keys, when user deploys multisig contract.

@mattlockyer looks good, but please add regression test

Merged in #453


[Deprecation] document.registerElement is deprecated and will be removed in M73, around March

jstree has updated to 3.3.7 to fix this, https://github.com/vakata/jstree/issues/2165
package slider already allows this, just need to update and release a new version

Yup, looks like the new jstree fixes the deprecation. Is there a way to upgrade just the jstree addon?

wouldn't this break compatibility with Edge?
window.customElement is not supported in IE/Edge
https://developer.mozilla.org/en-US/docs/Web/API/CustomElementRegistry/define#Browser_compatibility


Inaccessible markdown heading links

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[ ] I have tried the npm run clear or yarn clear command.
[ ] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[ ] I have tried creating a repro with https://new.docusaurus.io.
[ ] I have read the console error message carefully (if applicable).

Description
Heading hash links in Markdown docs are missing the accessible name. Screen reader users who tab through the page by keyboard and focus on the links hear only "Direct link to heading" (the text inside the title attribute of hash links).
Also, when users of screen readers populate a list of links on the page, it is showing the following:

Basically, they do not know where the links will take them since there is no meaningful text content.
Reproducible demo
https://docusaurus.io/docs#fast-track
Steps to reproduce

Go to any docusaurus docs page, for example, https://docusaurus.io/docs
Turn on the screen reader (e.g. NVDA on Windows, or VoiceOver on Mac)
Tab through the heading links:

Fast Track (the first hash link)
Docusaurus: Documentation Made Easy (the second hash link)
and so on

When focused on the link, the screen reader announces something like "Direct link to heading, link"

Expected behavior
The hash links should have an accessible name, ideally the heading content, so screen reader users know where the links are pointing to.
The suggestion for the new HTML structure of a sample link:

  Fast track

The implementation of the # character can be still made on hover, similarly as on MDN.
Actual behavior
The hash links have no accessible name (no unique text content) so screen reader users do not know where the links are pointing to.
The current HTML structure of a sample link:

  Fast Track

Your environment

Public site URL: https://docusaurus.io/docs#fast-track
Docusaurus version used: 2.2.0

Self-service

[ ] I'd be willing to fix this bug myself.

Makes sense, thanks for reporting.

The hash links should have an accessible name, ideally the heading content,

The implementation of the # character can be still made on hover, similarly as on MDN.

Is this mandatory or an official a11y reco?
Many other solutions are not adding the link on the full heading, see for example:

https://squidfunk.github.io/mkdocs-material/getting-started/
https://vitepress.vuejs.org/guide/what-is-vitepress

Personally, I don't find the MDN implementation pleasant to look at for a regular user without any disability, and would prefer to keep the existing pattern and just fix the title, is it fine?

Note we already have a translated anchor title:
<a
        className="hash-link"
        href={`#${id}`}
        title={translate({
          id: 'theme.common.headingLinkTitle',
          message: 'Direct link to heading',
          description: 'Title for link to heading',
        })}>

Do you have a recommendation for the label so that we do not have to "rewrite" all the existing translations from scratch? (I personally can only translate EN + FR)
Is something like "Direct link to heading - {headingLabel}" good enough?

Thank you for your reply!
IMHO it would be a violation of the WCAG success criterion 2.4.4. Link Purpose (In Context) (A), however, the user experience for users of assistive technologies is currently as described, so any approach that results in links having an accessible and meaningful name would be fine. There is a good article about this problematic: https://amberwilson.co.uk/blog/are-your-anchor-links-accessible/.
The text you're suggesting for the title is great. The main problem is we cannot use the title attribute as the only means for setting the accessible name of the link as it is infamously inaccessible (not supported by many screen readers). You can supply the helper text "Direct link to heading - {headingLabel}" into the title, but the link should have the text content (instead of ) in the first place.

What I see in practice is that tools like VoiceOver will announce better if the link has aria-label rather than using title. Isn't this a good-enough solution?
Duplicating the link title in a hidden element feels like a weird solution to me 🤷♂

Will read that carefully when I have time but if anyone can suggest a good-enough solution that does not involve useless hidden child elements duplication + making the whole text the anchor link, I'd be happy to implement it fast.

Can you tell exactly how we violate the spec according to you?
Unfortunately I don't have much time to read in details the whole docs 😅

Thanks for the explanations!

Another approach can be the aria-label way you suggested, which would also work since the accessible name of the link would not be empty space or "#" character, but "Direct link to Fast track":

I guess we can move on and just add this new aria-label then, that looks like a good enough solution

Sorry, I missed your points about the other solutions before

Not saying those linked examples are good accessibility examples, just wanted to illustrate other docs sites preferring not having clickable anchor headings.

Thank you for taking the time looking to this, really. I realize it can be quite tricky and complex. I am currently building a site dedicated to web accessibility using Docusaurus, and I'd greatly appreciate if accessibility issues and difficulties can be handled

We also appreciate having a base of users caring about accessibility, as this helps up improve on topics we may not have expertise on 😄

optionally have control over the markup so I can adjust it.

In any case, you should be able to swizzle headings components and implement your own markup, but we should have good accessibility by default.
If aria-label is not good enough for you, you can swizzle and make it use heading anchor links instead.

Thanks a lot, much appreciated! If aria-label will be there, no need for me to swizzle then :) 💚


[Event Request] - Codeunit CRM Quote to Sales Quote - OnBeforeGetCRMAccountOfCRMQuote

Similar to request # 14892   , can you add event in GetCRMAccountOfCRMQuote Procedure?
procedure GetCRMAccountOfCRMQuote(CRMQuote: Record "CRM Quote"; var CRMAccount: Record "CRM Account"): Boolean
    var
        CRMContact: Record "CRM Contact";
    begin
        //New Event
        OnBeforeGetCRMAccountOfCRMQuote(CRMQuote, CRMAccount,RetValue);
        if IsHandled then
            exit(RetValue);
        //New Event

        if CRMQuote.CustomerIdType = CRMQuote.CustomerIdType::account then
            if CRMAccount.Get(CRMQuote.CustomerId) then
                if CRMAccount.CustomerTypeCode <> CRMAccount.CustomerTypeCode::Customer then begin
                    Session.LogMessage('0000DMO', StrSubstNo(AccountNotCustomerTelemetryMsg, CannotCreateSalesQuoteInNAVTxt, CRMQuote.CustomerId, CRMProductName.CDSServiceName()), Verbosity::Warning, DataClassification::CustomerContent, TelemetryScope::All, 'Category', CrmTelemetryCategoryTok);
                    Error(AccountNotCustomerErr, CannotCreateSalesQuoteInNAVTxt, CRMAccount.Name, CRMProductName.CDSServiceName());
                end else
                    exit(true);

        if CRMQuote.CustomerIdType = CRMQuote.CustomerIdType::contact then
            if CRMContact.Get(CRMQuote.CustomerId) then
                exit(CRMAccount.Get(CRMContact.ParentCustomerId));
        exit(false);
    end;

    //New Event
    [IntegrationEvent(false, false)]
    local procedure OnBeforeGetCRMAccountOfCRMQuote(CRMQuote: Record "CRM Quote"; var CRMAccount: Record "CRM Account"; var RetValue)
    begin
    end;
    //New Event

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 10836.


Calling setPermission on group subjects doesn't always save

As title states, whenever I attempt to call setPermission on group subjects, it doesn't always save to storage. Player subjects have no issues.
Tested with LP 3.2.20

I'm still thinking about the best way to solve this. Rest assured I haven't forgotten, just don't want to rush a crappy (potentially more broken) solution.

Should be fixed.

@lucko just tested build 3.3.20 and this appears to still be an issue.
I'm testing with json storage and running the following command
https://github.com/MinecraftPortCentral/GriefPrevention/blob/master/src/main/java/me/ryanhamshire/griefprevention/command/CommandHelper.java#L482
This doesn't always work properly with groups. I'll have to execute my command multiple times in order to see it work. However, sometimes it works instantly.

Might be an issue with the way i/o via Sponge is being buffered.

OK it seems like if I let it sit for awhile,  the permission is eventually applied. So the issue is, why do group permission set/unsets take much longer than players randomly?

Should be fixed (kinda) by: d60d0ac9c54a052678df037b8e6a1d1de95c5e01
I want to avoid removing the buffering completely, as it completely kills the storage whenever plugins try to make bulk changes to permissions.


Question: Using ISO8601 string format

Hi,
i try to use it inside a nuxt application.
How to work with string values? It looks like it only accepts Date objects.
But in the real world you get data from your'e api in ISO8601 string format.
Can vue-datepicker handle it?
What I tried so far:
import VueDatePicker from '@vuepic/vue-datepicker';
import<EMAIL_ADDRESS>
let time = ref<string>("08:30:00")

<template>
            <VueDatePicker
                format='HH:mm:ss'
                v-model="time"
                time-picker
                :is-24="true"
                text-input.bool='true'
                dark
                :text-input-options="{ 'format': 'HH:mm' }"
                />
</template>

@Kingside88 Please refer to model-type prop.
In your case, you can just add :model-type="'format'" or provide a custom pattern.


A very suspicious reference path from PrivateImplementationDetails

When analyzing Heap Snapshot of nopCommerce, very strange references are shown, for example for CacheEntry instances:

Many comes from <PrivateImplementationDetails>.GUID classes are used for array initializers. For example, the heaviest one 040AAE3FE52CFC22160CC39DFBB7ADE6B14284C7 comes from Microsoft.CodeAnalysis.CSharp.dll:
[CompilerGenerated]
internal sealed class <PrivateImplementationDetails>
{
    // ...
   internal static readonly <PrivateImplementationDetails>.__StaticArrayInitTypeSize=32 040AAE3FE52CFC22160CC39DFBB7ADE6B14284C7;
    // ...
   [StructLayout(LayoutKind.Explicit, Pack = 1, Size = 32)]
   private struct __StaticArrayInitTypeSize=32
   {
   }
    // ...
}

used in Microsoft.CodeAnalysis.CSharp.BuiltInOperators.GetSimpleBuiltInOperators method:
   // Token: 0x060011E8 RID: 4584 RVA: 0x0004C93C File Offset: 0x0004AB3C
   .method assembly hidebysig
  	instance void GetSimpleBuiltInOperators (
		valuetype Microsoft.CodeAnalysis.CSharp.UnaryOperatorKind kind,
		class [Microsoft.CodeAnalysis]Microsoft.CodeAnalysis.ArrayBuilder`1<valuetype Microsoft.CodeAnalysis.CSharp.UnaryOperatorSignature> operators
	) cil managed
   {
   // ...
   /* 0x0004AC3B 8DD3020001   */ IL_00F3: newarr    [System.Runtime]System.Int32
   /* 0x0004AC40 25           */ IL_00F8: dup
   /* 0x0004AC41 D0DC180004   */ IL_00F9: ldtoken   field valuetype '<PrivateImplementationDetails>'/'__StaticArrayInitTypeSize=32' '<PrivateImplementationDetails>'::'040AAE3FE52CFC22160CC39DFBB7ADE6B14284C7'
   /* 0x0004AC46 288504000A   */ IL_00FE: call      void [System.Runtime]System.Runtime.CompilerServices.RuntimeHelpers::InitializeArray(class [System.Runtime]System.Array, valuetype [System.Runtime]System.RuntimeFieldHandle)
  /* 0x0004AC4B 28E7110006   */ IL_0103: call      instance valuetype [System.Collections.Immutable]System.Collections.Immutable.ImmutableArray`1<valuetype Microsoft.CodeAnalysis.CSharp.UnaryOperatorSignature> Microsoft.CodeAnalysis.CSharp.BuiltInOperators::GetSignaturesFromUnaryOperatorKinds(int32[])
   /* 0x0004AC50 A43502001B   */ IL_0108: stelem    valuetype [System.Collections.Immutable]System.Collections.Immutable.ImmutableArray`1<valuetype Microsoft.CodeAnalysis.CSharp.UnaryOperatorSignature>
   // ...
}

It is impossible that such static fields reference Microsoft.Extensions.Caching.Memory.CacheEntry as shown above... Not sure how to interpret that.

It is hard to know if there is a bug here or not.  More information is certainly needed for the issue to be actionable.    I agree it looks suspicious.
Including the trace will help.
It would also be will be useful to also capture a dump of the GC heap when clicking the 'save ETL' checkbox as well.   This uses a very different way of capturing the data, and if they differ, that is more clearly indicates a bug.

Per @vancem's comment, a repro is required here.  Closing.


Limit individual allocation size using the global memory size

@ahesham-arm Should this supersede https://github.com/KhronosGroup/OpenCL-CTS/pull/1172?

@ahesham-arm Should this supersede #1172?

This extracts a "bug fix" portion from #1172, which is less controversial, making it more likely to be accepted and merged.

Merging as discussed in the December 12th teleconference.


Support more advanced relabeling for pod metrics.

We've attempted to implement the managed service, and though it looks good, it could still use some work before it's ready (at least for us).  One of the biggest issues is relabeling.  Right now, you're only able to use simple "scrape from pod labels and add that label to the metric" using:
targetLabels:
  fromPod:
  - from: somepodlabel
     to: someprometheuslabel

As an end user, it would be nice to have access to more advanced features, such as:

Regex - allow the ability to create custom labels using regex, so we can split up things like "instance" or other metrics, into sharper, and easier to use labels
Access to non Label Metadata - values such as _meta_kubernetes_pod_container_name, so we can provide more details in our alerting as to which container is causing issues, as well as filter based on containers.  (this could be done in a combo of port names and regex, but, it would be nicer to have access to more advanced labels).

Thanks for the feedback.
Adding pod and container labels is certainly something I'd consider adding by default as I imagine most users ultimately wanting them.
We are cautious about exposing raw target and metric relabeling capabilities as it is such a common source of error and often hard to reason about. OTOH we see that people have established relabeling configurations and are also used to having the power it provides.
For metric relabeling we are likely going to allow it. For target relabeling I'd still like to gather more use cases to convince us that we cannot cover the vast majority without relabeling.
Do you have any concrete examples aside from pod and container labels?

From the simple testing I've done, pod, container, address are all ones that are the largest ones that we use.  Depending on the team, other things may be valid too, but I haven't gone to deep at this time.

(also use regex or the like to maybe strip off ports off address or the like, depending on how the metric is presented and how it's actually going to be used in the long run).

Thanks. Could you spot other uses of relabeling onto __address__ aside from the blackbox_exporter?
Essentially PodMonitoring should give cluster admins confidence that no user who can create/update a PodMonitoring can initiate scraping of endpoints outside of the given namespace. Being able to arbitrarily modify the address would make this difficult to guarantee.
If the only use case is the blackbox_exporter we may be able to come up with something more dedicated with stronger bounds.
I don't quite understand your second comment on stripping the port from the address. Did you mean stripping it from the instance = "<pod_name>:<port>" label we generate currently?

Blackbox Exporter is one yes, but I was meaning pulling FROM _address to other metrics.

I don't quite understand your second comment on stripping the port from the address. Did you mean stripping it from the instance = "<pod_name>:" label we generate currently?

Yea, or any similar metrics, using a regex to split them for generating labels and groupings is a really nice have.

So as of recent releases we expanded our capabilities a bit. Via spec.targetLabels.metadata one can now set pod, container and node labels. The former will be set by default going forward.
Manual metric relabeling rules are also supported now.
Full API docs are here.
Do these help for your main use cases?

Potentially, yes.. we'll have to look into implementation again and see how it goes.  (No ETA yet as to when I'll be able to test it out).

AFAICT the additions from the linked PRs address the requests here so I'm going to close this.


Add alignment feature in markdown table

I have add alignment while creating markdown table.
The format for alignment will be like this :-
return json2md([
{
table: {
headers: ["a", "b"],
rows: [[1,2], [3,4]],
align : "center"
}
}
]);

Closing due to no reply...


Local time not supported

Per the TOML local time spec, the following expressions should parse:
lt1 = 07:32:00
lt2 = 00:32:00.999999

I receive the following error:
      throw peg$buildException(null, peg$maxFailExpected, peg$maxFailPos);
      ^
SyntaxError: Expected "#", ".", "E", "\n", "\r", "_", "e", [ \t] or [0-9] but ":" found.

Agree, I this it's necessary.

This is a 0.5.0 feature, and docs say "toml-node supports version 0.4.0 the TOML spec". (This is a little confusing because there are some 0.5.0 features that are supported, like string literals as keys.)

Relatedly, the TOML spec allows replacing the T in a datetime like 2021-02-07T23:00:00-08:00 with a space for readability (as does the RFC 3339 spec).  Granted, this is also a 0.5.0 feature.
But at the same time, v0.5.0 was released more than two years ago; we are now at v1.0.0.  It kind of seems like this project is dead or something, despite being the "standard" JS TOML package.


指定了域名服务器不起作用

rules.cfg文件相关内容
 "Pattern": "dl\\.google\\.com$",
 "NameServer": "<IP_ADDRESS>"

nslookup 结果：

nslookup dl.google.com
<IP_ADDRESS>.in-addr.arpa
primary name server = localhost
responsible mail addr = nobody.invalid
serial  = 1
refresh = 600 (10 mins)
retry   = 1200 (20 mins)
expire  = 604800 (7 days)
default TTL = 10800 (3 hours)
服务器:  UnKnown
Address:  <IP_ADDRESS>

DNS request timed out.
timeout was 2 seconds.
DNS request timed out.
timeout was 2 seconds.
DNS request timed out.
timeout was 2 seconds.
DNS request timed out.
timeout was 2 seconds.
*** 请求 UnKnown 超时

指定<IP_ADDRESS>的nslookup结果：

nslookup dl.google.com <IP_ADDRESS>
服务器:  public1.114dns.com
Address:  <IP_ADDRESS>

非权威应答:
名称:    dl.l.google.com
Addresses:  2404:6800:4008:c03::5b
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
Aliases:  dl.google.com

是配置文件的问题解决了

把正确的配置在这里po一下吧，谢谢啦～

就是贴出来的这个，可能是和我其他配置条目冲突了，把其他的删了只留下这个就好了


Correct typos in code in README.md

Hey- I fixed some typos in the code example under "Usage with additional options."

Thanks for that @scmanta !


cygwin, phar missing Zend\Http\Client\Adapter\ctype_xdigit()

PHP version:
PHP 5.6.8 (cli) (built: Apr 16 2015 17:31:50)
Copyright (c) 1997-2015 The PHP Group
Zend Engine v2.6.0, Copyright (c) 1998-2015 Zend Technologies

zs-client's reported versions:
ZendServerWebApi Client version 1.2
Zend Server Client v1.0

error exists both when using the .phar as an executable and after extracting with the phar utility followed by calling index.php.
Arguments which precipitated the error are:
zs-client addTarget --target="ZendLive" --zskey="zs-client"  --zssecret="removed" --zsurl="http://zendserver.local:10081/"

Error output:
PHP Fatal error:  Call to undefined function Zend\Http\Client\Adapter\ctype_xdigit() in /home/dllewellyn/bin/zs-client.unpacked/vendor/zendframework/zend-http/Zend/Http/Client/Adapter/Socket.php on line 2

Fatal error: Call to undefined function Zend\Http\Client\Adapter\ctype_xdigit() in /home/dllewellyn/bin/zs-client.unpacked/vendor/zendframework/zend-http/Zend/Http/Client/Adapter/Socket.php on line 2

this is a bad installation of php - the ctype functions are supplied by a separate cygwin package - php is split into many individual components in cygwin it seems, so I needed to install php-ctype from the cygwin setup utility.


Compile python bindings from source

Context
https://eclipse-ecal.github.io/ecal/_download_archive/download_archive.html#download-archive
Currently there are already compiled python binaries available for different os and python versions. I was wondering if there is any compilation guide for custom ubunntu version with custom python version. (lets say i want to install python package for ubuntu 22.04 and python3.8)
If i can build those binaries from source, it would help me a lot.
Thank you
Proposal
No response
Tasks and updates
No response

Hi @deo-abhijit ,
thank you for reaching out. There are multiple ways, depending on which branch you want to work from, to build the wheel.
1. If you want to compile for the releases that are currently available, you can follow the guide here:
https://eclipse-ecal.github.io/ecal/development/building_ecal_from_source.html
This would be recommended, if you also want to build the whole of eCAL.
2. If you want to just build the wheel for the python version, you can configure the project instead like this:
cmake .. -DCMAKE_EXPORT_COMPILE_COMMANDS=ON -DHAS_HDF5=ON -DHAS_QT5=ON -DHAS_CURL=ON -DHAS_CAPNPROTO=OFF -DHAS_FTXUI=ON -DBUILD_DOCS=OFF -DBUILD_APPS=OFF -DBUILD_SAMPLES=OFF -DBUILD_TIME=OFF -DBUILD_PY_BINDING=ON -DBUILD_STANDALONE_PY_WHEEL=ON -DBUILD_CSHARP_BINDING=OFF -DBUILD_ECAL_TESTS=OFF -DECAL_INCLUDE_PY_SAMPLES=OFF -DECAL_INSTALL_SAMPLE_SOURCES=OFF -DECAL_JOIN_MULTICAST_TWICE=OFF -DECAL_NPCAP_SUPPORT=OFF -DECAL_THIRDPARTY_BUILD_CMAKE_FUNCTIONS=ON -DECAL_THIRDPARTY_BUILD_PROTOBUF=OFF -DECAL_THIRDPARTY_BUILD_SPDLOG=ON -DECAL_THIRDPARTY_BUILD_TINYXML2=ON -DECAL_THIRDPARTY_BUILD_FINEFTP=ON -DECAL_THIRDPARTY_BUILD_CURL=OFF -DECAL_THIRDPARTY_BUILD_GTEST=ON -DECAL_THIRDPARTY_BUILD_HDF5=OFF -DECAL_THIRDPARTY_BUILD_RECYCLE=ON -DECAL_THIRDPARTY_BUILD_TCP_PUBSUB=ON -DECAL_THIRDPARTY_BUILD_QWT=OFF -DECAL_THIRDPARTY_BUILD_YAML-CPP=OFF -DCMAKE_BUILD_TYPE=Release -DPython_FIND_VIRTUALENV=ONLY

then create a python venv, activate and install:
pip install wheel setuptools
pip install ./doc/requirements.txt

then build like in the 1. approach with
cmake --build . --target create_python_wheel --config Release

This will generate the wheel in the build folder under _deploy.
3. On master (in an alpha state), you can build using cibuildwheel:

install build and wheel
pip install build cibuildwheel

build the wheel for your python version
(check https://cibuildwheel.pypa.io/en/stable/options/#build-skip for more options)
cibuildwheel --only cp38-manylinux_x86_64

install the wheel that gets created in ./wheelhouse

I hope you will find this useful. Let us know, if you could build the wheel and use it successfully. If not, we can support further.
Many greetings
Peter

Thanks for the detailed response. I would try it out and let you know!
Also if we could add this information in the readme, it might help others who are looking for the same things as me.
Thanks


Fixed version of ESLint in the react-scripts package causing issues

Is this a bug report?
Yes.
Did you try recovering your dependencies?
Yes.
Which terms did you search for in User Guide?
Not relevant.
Environment
Not relevant.
Steps to Reproduce
See this issue.
Expected Behavior
This error should not be happening.
Actual Behavior
There is an error because of the fixed ESLint version in the react-scripts package.

Question
Why does react-scripts keeps fixed versions of packages such as eslint in the dependency manifest? Why not just specify the major version, like eslint-config-react-app does (5.x, for example)?
https://github.com/facebook/create-react-app/blob/master/packages/react-scripts/package.json#L40
Notice how eslint-config-react-app sets its ESLint version:
https://github.com/facebook/create-react-app/blob/master/packages/eslint-config-react-app/package.json#L17

same problem

Same problem

I am having the same issue.  And now I can't get eslint to work in VSCode (code-server).  I followed the instructions where you remove node_modules, package-lock.json, yarn-lock.json and take out the reference to "eslint": "^5.16.0" out of the devDependencies of the package.json.  But I left the references to eslink-plugins and eslink-config. Then I do a yarn or npm install.
Now I get the error in my output when VSCode tries to lint a file?
module.js:562
throw err;
^
Error: Cannot find module '/home/bitnami/testlint/-e'
at Function.Module._resolveFilename (module.js:560:15)
at Function.Module._load (module.js:487:25)
at Function.Module.runMain (module.js:726:10)
at startup (bootstrap_node.js:207:16)
at bootstrap_node.js:628:3

The maintainers discussed this issue today and we've decided to relax the ESLint version requirement.  cc @ianschmitz


Hidden doesn't display when starting chat with new user

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

Version Number: 1.4.61-5
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @dukenv0307
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1712564640051209
Action Performed:

Go to FAB> Start chat
Enter the user that we haven't chatted with and this user doesn't have a custom display name
Notice the display name in LHN and header
Logout and login again
Notice the display name in LHN and header again

Expected Result:
In step 3, Hidden should be displayed after the OpenReport API is complete.
Actual Result:
In step 3 the user's email is displayed.
After logging out and logging again, it displays Hidden correctly.
Workaround:
unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/38435837/7df27cc4-b721-4869-b1f3-3a8380f76a01

View all open jobs on GitHub

Proposal
Please re-state the problem that we are trying to solve in this issue.
In step 3 the user's email is displayed.
After logging out and logging again, it displays Hidden correctly.
What is the root cause of that problem?
When we search an exist user, BE will return the personal detail of this user with login field and a flag is isFromServerSearch as true. After we create the report, we don't clear the login data so the email is displayed. When we logout and login again, BE returns the personal detail of this user without login field and then Hidden is displayed correctly.

What changes do you think we should make in order to solve the problem?
I've checked and see that back-end will return the login and isFromServerSearch as true if the personal detail list of current user doesn't have the search user. So to fix this issue, we should clear the login and update isFromServerSearch as false in successData if allPersonalDetails?.[accountID]. isFromServerSearch is true
https://github.com/Expensify/App/blob/aed0bf0e015d745f84f2961955f715bde73ec073/src/libs/actions/Report.ts#L715
const settledPersonalDetail = allPersonalDetails?.[accountID] ?? null;
if (settledPersonalDetail && settledPersonalDetail?.isFromServerSearch) {
    settledPersonalDetail?.login = '';
    settledPersonalDetail?.isFromServerSearch = false;
}
settledPersonalDetails[accountID] = settledPersonalDetail;

What alternative solutions did you explore? (Optional)
NA

I don't think this is a bug. If you are the inviter for the user, they will be known to you and not "hidden".

I cannot reproduce this altogether...

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989


rRNA or mtRNA sequences are detected as adapter sequences?

Hi, developer
I used fastp with some RNA-seq data, and found the detected adapter sequences are identical to some rRNA or mtRNA sequences.
For example:
Detecting adapter sequence for read1...
GTTCGATTCCTTCCTTTCTTATTTTACTTTTACATAGGTTGGTTCCTCGAATGTGTGATA
Detecting adapter sequence for read2...
CAAACTCATGCATATCACATAGTTAATCCAAGTCCATGACCATTAACTGGAGCCTTTTCA
When I run blast with these sequences, it says these sequences are 100% identical to mouse mtRNA.
It's that means the RNA-seq sample contain a lot of rRNA and mtRNA? Should I trim these sequences before mapping?

It's normal that RNA-seq data has rRNA.
I think you can disable --detect_adapter_for_PE, since most adapters will be trimmed by PE overlap detection.


Added support for VisionDataset

This adds support for the 'new' VisionDataset superclass for datasets which were added after the original PR #749.

Codecov Report

Merging #838 into master will increase coverage by 0.54%.
The diff coverage is 54.54%.

@@            Coverage Diff             @@
##           master     #838      +/-   ##
==========================================
+ Coverage   53.92%   54.47%   +0.54%
==========================================
  Files          36       36
  Lines        3347     3306      -41
  Branches      549      543       -6
==========================================
- Hits         1805     1801       -4
+ Misses       1409     1372      -37
  Partials      133      133

Impacted Files
Coverage Δ

torchvision/datasets/imagenet.py
22.68% <50%> (+1.47%)
:arrow_up:

torchvision/datasets/celeba.py
16.09% <50%> (+1.35%)
:arrow_up:

torchvision/datasets/sbd.py
31.66% <50%> (+0.08%)
:arrow_up:

torchvision/datasets/caltech.py
19% <62.5%> (-2.14%)
:arrow_down:

torchvision/transforms/transforms.py
83.52% <0%> (+0.67%)
:arrow_up:

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 2686e1a...7e24527. Read the comment docs.


Should we add pre-commit to the docs/requirements.txt and to CONTRIBUTING.md?

Is your feature request related to a problem? Please describe.
We are asking contributors to check they run the required pre-commits which is great, but the contribution guidelines do not mention the pre-commit installation.
Describe the solution you'd like
I think the easiest way would be:

to add pre-commit to the docs/requirements.txt, and
add a pre-commit install command in the CONTRIBUTING.md to install the provided hooks.

Maybe the Github workflow section could also be reorganised. I think I would have first the dependencies installation, next the "github workflow", and last the local build commands.
Describe alternatives you've considered
\
Additional context
\

It all sounds reasonable and you should go for it. I think that contributing guide was written before we added hooks etc, so we just forgot to update it.


DLPX-78719 [Backport of DLPX-73623 to <IP_ADDRESS>] SSHJ remote port forwarding buffers can grow without limits

Almost clean cherry-pick of https://github.com/delphix/sshj/pull/1 on top of the latest release, 0.32.0. The only conflict was the change in the upstream of a method call from putUInt32 to putUInt32FromInt in ChannelInputStream.checkWindow(). The conflict occurred because our change affects the indentation of that line, nothing else.
Published on artifactory as version 0.32.0-dlpx73623-test for now, and running a trunk engine ab-pre-push including upgrade and precommit tests with this version of sshj: http://selfservice.jenkins.delphix.com/job/appliance-build-orchestrator-pre-push/111/
Confirmed that the buffer OOM continues to exist in 0.32.0 by reproducing it using our existing guide: https://docs.google.com/document/d/1Ws8LeiwtRErzwufIzXtYFwyc-SJzbjWMayv7IQ26dW8/edit#
Tested that the bug does not manifest when using this patched version of 0.32.0.

Looks like there were some upgrade failures on the orchestrator build. Not sure if they are related to the changes, can you confirm?


Update the output log message with more descriptive qualities

🗣 Description
This PR resolves issue https://github.com/cisagov/tpt-reports/issues/43 and adds a more descriptive output log message in CLI for the generated report.
💭 Motivation and context
To make the log  message unique to the assessment and location of the output directory.
🧪 Testing
Run the script as usage suggests and verify output references are replaced.
✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All future TODOs are captured in issues, which are referenced
in code comments.
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[x] All relevant repo and/or project documentation has been updated
to reflect the changes in this PR.
[x] Tests have been added and/or modified to cover the changes in this PR.
[x] All new and existing tests pass.

✅ Pre-merge checklist

[x] Revert dependencies to default branches.
[x] Finalize version.

✅ Post-merge checklist

[ ] Create a release.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
0

Relevant Lines:
0

💛  - Coveralls


Synthesis failed for Google.Cloud.SecurityCenter.V1P1Beta1

Hello! Autosynth couldn't regenerate Google.Cloud.SecurityCenter.V1P1Beta1. :broken_heart:
Here's the output from running synth.py:
Cloning into 'working_repo'...
Switched to a new branch 'autosynth-Google.Cloud.SecurityCenter.V1P1Beta1'
Cloning into '/tmpfs/tmp/tmpnxeyjvql/googleapis'...
Note: checking out '88316b63a486002727e14032d104690541179fc9'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b <new-branch-name>

HEAD is now at 88316b63a Generated synth.py files with multiple commits enabled
Note: checking out '7be2811ad17013a5ea24cd75dfd9e399dd6e18fe'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b <new-branch-name>

HEAD is now at 7be2811a fix: Update gapic-generator version to pickup discogapic fixes
Switched to a new branch 'autosynth-Google.Cloud.SecurityCenter.V1P1Beta1-2'
Running synthtool
['/tmpfs/src/git/autosynth/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']
2020-03-30 07:27:53,252 synthtool > Executing /tmpfs/src/git/autosynth/working_repo/apis/Google.Cloud.SecurityCenter.V1P1Beta1/synth.py.
Contents of preconfig file:
{"preclonedRepos": {"https://github.com/googleapis/google-cloud-dotnet.git": "/tmpfs/src/git/autosynth/working_repo", "https://github.com/googleapis/googleapis.git": "/tmpfs/tmp/tmpnxeyjvql/googleapis"}}Extracted repo location: /tmpfs/tmp/tmpnxeyjvql/googleapis
generateapis.sh: line 40: declare: GOOGLEAPIS: readonly variable
2020-03-30 07:27:53,302 synthtool > Failed executing /bin/bash generateapis.sh --check_compatibility Google.Cloud.SecurityCenter.V1P1Beta1:

None
2020-03-30 07:27:53,304 synthtool > Wrote metadata to synth.metadata.
Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/synthtool/__main__.py", line 102, in <module>
    main()
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/synthtool/__main__.py", line 94, in main
    spec.loader.exec_module(synth_module)  # type: ignore
  File "<frozen importlib._bootstrap_external>", line 678, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/tmpfs/src/git/autosynth/working_repo/apis/Google.Cloud.SecurityCenter.V1P1Beta1/synth.py", line 22, in <module>
    hide_output = False)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/synthtool/shell.py", line 39, in run
    raise exc
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/synthtool/shell.py", line 33, in run
    encoding="utf-8",
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/subprocess.py", line 438, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '('/bin/bash', 'generateapis.sh', '--check_compatibility', 'Google.Cloud.SecurityCenter.V1P1Beta1')' returned non-zero exit status 1.
Synthesis failed
Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/git/autosynth/autosynth/synth.py", line 482, in <module>
    main()
  File "/tmpfs/src/git/autosynth/autosynth/synth.py", line 334, in main
    return _inner_main(temp_dir)
  File "/tmpfs/src/git/autosynth/autosynth/synth.py", line 429, in _inner_main
    branch,
  File "/tmpfs/src/git/autosynth/autosynth/synth.py", line 135, in synthesize_loop
    metadata_path, extra_args, deprecated_execution, environ, synthesize_py_path,
  File "/tmpfs/src/git/autosynth/autosynth/synth.py", line 278, in synthesize
    synth_proc.check_returncode()  # Raise an exception.
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/subprocess.py", line 389, in check_returncode
    self.stderr)
subprocess.CalledProcessError: Command '['/tmpfs/src/git/autosynth/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']' returned non-zero exit status 1.

Google internal developers can see the full log here.

Autosynth passed, closing! :green_heart:


login() does not seem to emit any LoginResult in the first time it is called

I used ReactiveFb.login(Activity) and then used a concatMap to catch the loginResult, I don't think it is emitting any or if the onSuccess method is being called internally that's why it does not continue to the next Maybe. Here is how I am using it
ReactiveLogin.login(mActivity)
                .compose(mActivity.bindToLifecycle())
                .doOnSubscribe(disposable -> {
                    if (mIsFacebookConnectOngoing) {
                        disposable.dispose();
                    } else {
                        mIsFacebookConnectOngoing = true;
                    }
                })
                .concatMap(loginResult ->
                        mAuthFramework
                                .signInWithCredentials(FACEBOOK_PROVIDER, loginResult.getAccessToken().getToken())
                                .doOnSubscribe(disposable -> mView.showProgressDialog())
                                .compose(mActivity.bindToLifecycle()))
                .doFinally(() -> mIsFacebookConnectOngoing = false)
                .subscribe(user -> {
                    mView.hideProgressDialog();
                    mAuthFramework.setLoggedIn(true);
                    mView.goToMainScreen();
                }, catchErrorThrowable());

I am using Rx2Firebase  internally for the abstraction of the mAuthFramework.signInWithCredential. I logged it, and it is not even being called the first time this flow is subscribed to

Hi, have you added the onActivityResult to your Activity ? Is still mandatory to catch the facebook sdk callbacks.
@Override
public void onActivityResult(int requestCode, int resultCode, Intent data) {
    super.onActivityResult(requestCode, resultCode, data);
    ReactiveLogin.onActivityResult(requestCode, resultCode, data);
}
`

Hi,
Yes I have added it in my onActivityResult.
On 24 Apr 2017 21:07, "Wassim Beltaief"<EMAIL_ADDRESS>wrote:

Hi, have you added the onActivityResult to your Activity ? Is still
mandatory to catch the facebook sdk callbacks.
@Overridepublic void onActivityResult(int requestCode, int resultCode, Intent data) {
super.onActivityResult(requestCode, resultCode, data);
ReactiveLogin.onActivityResult(requestCode, resultCode, data);
}
`
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/WassimBenltaief/ReactiveFB/issues/7#issuecomment-296661746,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKDQgqHAI_JXXoUhGNO1z9Jy5BmuYTonks5rzJ6cgaJpZM4NFZi1
.

Can you make a small gist with your code ?It's working fine in my side.
On Apr 24, 2017 3:36 PM, "dev-JE02"<EMAIL_ADDRESS>wrote:

Hi,
Yes I have added it in my onActivityResult.
On 24 Apr 2017 21:07, "Wassim Beltaief"<EMAIL_ADDRESS>wrote:

Hi, have you added the onActivityResult to your Activity ? Is still
mandatory to catch the facebook sdk callbacks.
@Overridepublic void onActivityResult(int requestCode, int resultCode,
Intent data) {
super.onActivityResult(requestCode, resultCode, data);
ReactiveLogin.onActivityResult(requestCode, resultCode, data);
}
`
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<https://github.com/WassimBenltaief/ReactiveFB/issues/7#issuecomment-
296661746>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AKDQgqHAI_
JXXoUhGNO1z9Jy5BmuYTonks5rzJ6cgaJpZM4NFZi1>
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/WassimBenltaief/ReactiveFB/issues/7#issuecomment-296670518,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFxwQqtN0Lg9uls6ImIJiopp63i7UM8kks5rzKVrgaJpZM4NFZi1
.


An error occurred during the iteration because the array exceeded the preset maximum array size

When experimenting with experimental data [2] Berkely, the program ran to the train_hdm_un_Bayesian line of code and iterated, and the following error occurred. How can I solve it?

There are a couple of solutions that may be applicable.

Increase the maximum array size allowed by Matlab. There are many posts online on this topic. For example
Reduce the length of synthesized frames per sequence here: https://github.com/rort1989/BH-HSMM/blob/master/code/script_data_synthesis_synthesis.m#L16
Reduce the number of hidden states (which at the cost of reducing the performance): https://github.com/rort1989/BH-HSMM/blob/6d4d5776fd3ae935e596905969f67a7c5048abbf/code/configuration_synthesis_Berkeley.m#L3


This App is Not Available in Your Country or Region (Apple)

Is it possible to fix that and make the App also available in Europe?

Is it not possible?


update error.

i update my package,
-    "react-redux": "^1.0.1",
+    "react-redux": "^2.0.0",

-    "redux": "^1.0.1",
+    "redux": "^2.0.0",

-    "redux-devtools": "^1.1.2",
+    "redux-devtools": "^2.0.0",

but i got a error,
Warning: React can't find the root component node for data-reactid value `.0.$debugPanel.0.1`. If you're seeing this message, it probably means that you've loaded two copies of React on the page. At this time, only a single copy of React can be loaded at a time.

who can tell me why?

If you're seeing this message, it probably means that you've loaded two copies of React on the page. At this time, only a single copy of React can be loaded at a time.

Have you tried verifying whether this is the case?
See the issue described here.
I'm closing because it's unrelated to Redux DevTools—it's duplicate React in node_modules that is the issue.


Describe the bug
command: show reminder, nothing happens
ob version: 0.15.6
reminders version: 1.1.12
Expected Behavior
show reminders board
Steps to reproduce
command: show reminder
Operating system
Windows

the same with
reminders version: 1.1.15
obisidian 1.0.3


Update Podspec

Hey,
could you please update the Podspec to include the newest commits?
Thanks

+1, I'm currently patching my pod files to include the delegate methods swipeTopViewToUp and swipeTopeViewToDown

Sorry for the delay, the podspec has been updated.


feat(pin): Got pinning prototype working

:tada: This PR is included in version 1.1.0 :tada:
Your semantic-release bot :package::rocket:


Blog should have a search feature

As the number of blog posts increases, it will become imperative to have search functionality in the blog . Dependent on #22

Check out this Algolia + eleventy functions


[BUG] Broadcast (ws) messages duplication

I tried this code:
RPC::create('tcp://<IP_ADDRESS>:6001')
    ->call('websockets.Publish', new Request([
        'messages' => [new Message([
            'broker' => 'memory',
            'topics' => ['topic-1', 'topic-2'],
            'payload' => 'message',
        ])]
    ]);

With follow config:
rpc:
    listen: tcp://<IP_ADDRESS>:6001

server:
    command: "php worker.php"
    relay: "pipes"

http:
    address: <IP_ADDRESS>:80
    middleware: [ "websockets" ]
    pool:
        num_workers: 2

websockets:
    pubsubs: [ "memory" ]
    path: "/ws"

I expected to see this happen:
{"topic":"topic-1","payload":"message"}
{"topic":"topic-2","payload":"message"}

Instead, this happened:
{"topic":"topic-1","payload":"message"}
{"topic":"topic-2","payload":"message"}
{"topic":"topic-1","payload":"message"}
{"topic":"topic-2","payload":"message"}

The version of RR used: rr.exe version 2.3.1-beta.3 (build time: 2021-06-16T13:34:27+0000, go1.16.4)

The error does not depend on the number of workers in num_workers config section.
The error changes its behavior if you subscribe to a different number of topics. For example, this is how the result looks for 3 channels:

const ws = new WebSocket('ws://<IP_ADDRESS>/broadcast');

ws.onopen = e => {
    const message = {
        command: 'join',
        broker:  'memory',
        topics:  ['topic-1', 'topic-2', 'topic-3']
    };

    ws.send(JSON.stringify(message));
};

ws.onmessage = e => {
    const message = JSON.parse(e.data);
}

Then send only 1 message with payload "message" into 3 topics:

I've seen this error during 1st implementation. It was solved by the Client. Router abstraction to ensure only unique subscriptions.

I can not reproduce this issue on the latest build.
Connecting to the 4 topics:

Connected:

Sent 1 message to the 3 topics within initial 4:
makeMessage("join", "memory", []byte(data), "foo", "foo2", "foo3")

Received w/o duplicates:

Left 1 topic (foo), received 2 messages w/o duplicates:

Rejoin, sent message, no duplicates:

fixed in the #732, will be closed automatically after PR got merged.


feature: Added provision resource checks before publish

Description
Before publishing, the publishing profile needs to be checked to verify that the provisioning occurred at least once successfully and that all required resources have values from successful provisioning.
To avoid blocking the user unnecessarily , the checks verify the minimum values exist and do not verify values that could be empty for some deployments.
Task Item
fixes #6627
Screenshots
N/A - verification errors are displayed through the standard notification and result messages

@cwhitten - I think this should fix the ABS 'Open in composer' scenario when trying to publish.

Coverage remained the same at 51.389% when pulling e360cb6da5c43db58cc5069a185a0945eeedd6b0 on gcox/feature6627 into 99d7e4c6b17bb34a0f6ea8a8dffa62296cd29d44 on main.


fixed dedicated type, adjusted jenkins guide with path

Work Done

fixed a typo with dedicated word in solutions
added path to Jenkins in solutions page
Set Jenkins in alphabetical order in TOC

Note

If we are not updating dashboard devnet soon, we should remove from manual.dev.grid.tf the Domains and Jenkins apps until dashboard devnet is updated. @xmonader I tag you here to verify this. thanks!

Closing for a simpler PR


Unify recursive types

TODO:

[x] unify generic recursive types

Codecov Report
Attention: 12 lines in your changes are missing coverage. Please review.

Comparison is base (adb0eb2) 82.20% compared to head (0f008a2) 82.19%.

:exclamation: Current head 0f008a2 differs from pull request most recent head 9a09ef2. Consider uploading reports for the commit 9a09ef2 to get more accurate results

Files
Patch %
Lines

src/Escalier.Data/Library.fs
53.84%
6 Missing :warning:

src/Escalier.TypeChecker/Unify.fs
83.78%
6 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #146      +/-   ##
==========================================
- Coverage   82.20%   82.19%   -0.02%
==========================================
  Files          20       20
  Lines        6855     6901      +46
  Branches      926      946      +20
==========================================
+ Hits         5635     5672      +37
- Misses       1220     1229       +9

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


Bundle file mapping is not properly initialized when used with Smartsprite global preprocessor

The bundle file mapping is not properly initialized when used with Smartsprite global preprocessor for cached resources (like Less ressource).
This is due to the fact that the smartsprite global preprocessor generates use fake bundle to retrieve the cacheable css ressource.
When the bundle is then build, the file bundle mapping is not initialized.

This issue has been fixed in the master branch


VPC Block - Nov Release

@prankulmahajan could you also update OCP versions in metadata if this template is supported on ROKS version > 4.15

@ashimagarg27 Added 4.17 support