Feat/credit delegation

Some feedback:

Usage of SafeERC20
Don’t see much reason to not hardcode all the assets data in a pure function, and on constructor() only doing the approvals. The reason is that the assets are already constants (no setter), so this only adds more gas consumption.
To be consistent, I would say better V3_POOL https://github.com/bgd-labs/V2-V3-migration-helpers/blob/feat/credit-delegation/src/contracts/MigrationHelper.sol#L17
This is never used in the contract. Any reason externally? https://github.com/bgd-labs/V2-V3-migration-helpers/blob/feat/credit-delegation/src/contracts/MigrationHelper.sol#L16
Is not necessary to check lengths consistency, at least between certain input params? https://github.com/bgd-labs/V2-V3-migration-helpers/blob/feat/credit-delegation/src/contracts/MigrationHelper.sol#L57 Maybe grouping in structs
There should be a document with security assumptions, for example, that we consider the assets involved non-reentrant. Or high-level on why the contracts should never have leftovers. Or high-level on how the isolation between users is managed (one guy flashing and not affecting anybody else, unless “giving” funds)
Maybe checking for consistency with the RepayInput inputs? https://github.com/bgd-labs/V2-V3-migration-helpers/blob/feat/credit-delegation/src/contracts/MigrationHelper.sol#L78
For what is this exactly? Dark enough to add a comment https://github.com/bgd-labs/V2-V3-migration-helpers/blob/feat/credit-delegation/src/contracts/MigrationHelper.sol#L127
No need of init to 9, that is already default https://github.com/bgd-labs/V2-V3-migration-helpers/blob/feat/credit-delegation/src/contracts/MigrationHelper.sol#L179
How can it be that the assets to flash are the same length as the interest rates to flash? Is there some double flashing of the same underlying happening or something like that? https://github.com/bgd-labs/V2-V3-migration-helpers/blob/feat/credit-delegation/src/contracts/MigrationHelper.sol#L180
Same as before, no need https://github.com/bgd-labs/V2-V3-migration-helpers/blob/feat/credit-delegation/src/contracts/MigrationHelper.sol#L198
Why the assembly? https://github.com/bgd-labs/V2-V3-migration-helpers/blob/feat/credit-delegation/src/contracts/MigrationHelper.sol#L221
Could there be any problem with precision here? With withdrawn being more than balance https://github.com/bgd-labs/V2-V3-migration-helpers/blob/feat/credit-delegation/src/contracts/MigrationHelper.sol#L159

@eboadom

Yes, will be used
We are still thinking about this; will save some gas (but not much in %), but add complexity and manual work
Fixing
Fixing
It is not necessary to check length consistency, because there could be a mix of approvals/permits (or no permits at all)
Yes
We do not see the necessity for this check, in case something is wrong transaction will fail
Ok, will add
Fixing
We do not know the length in advance, so we init arrays with the maximum possible length and then squeeze the array using mstore()
Fixing
To squeeze the arrays
No, there shouldn't be a problem, we've rechecked impl of withdraw today

3, 4 - POOL and ADDRESS_PROVIDER are used to implement IFlashloanReceiver


[iOS][InputFiedl - Keyboard] [Japanese keyboard is not working properly with ACRTextField input field]

Platform
What platform is your issue or question related to? (Delete other platforms).

iOS

Author or host
Host - Team iOS
Version of SDK
1.2.7
Details
Default Japanese Keyboard font changes to English font inside ACRTextField input fields. Works fine with ACRTextView input fields.
Repro steps:

Add Japanese keyboard as one of your keyboards
Tap on multi line input field and try changing language to Japanese -> This works fine
Tap on single line input field and try changing language to Japanese -> Notice a different Japanese keyboard pops up this time with default English script.
PFA the screenshots of the two different keyboards.

Expected
Actual

@jwoo-msft can you help me understand the nature of changes involved here? Thanks!


Create a STAC tool

Create a tool to make STAC queries.
As a starting point, we can move the tool in this private repo here: https://github.com/developmentseed/haystac - of course, that's a very bare-bones starting point - we will want to explore implementing more complex queries, teach it CQL2 syntax, etc.
cc @kamicut @geohacker

I can do this

@srmsoumya any updates on how this is going?
cc @zacharyDez

@batpad Sajjad has most of the code implemented in the haystac repository, we have to port this to the LLLLM toolkit.
@geohacker will you have time this week to port this to LLLLM?

@srmsoumya I think if you have time, please go ahead. I don't want to hold this up and frankly have no idea when I'll have time in the next two weeks


Consider The RDCA-DAP: the data interrogator Use case for some examples

Integrating a lot of data source such as

Clinical trial data
Registry data
Upcoming:

EHR
Genomics data

"implementing ontologies"
easy cleaning and formatting
records are tagged
What are all the different options for "dosage field"
access to metadata from the get go: making them FAIR
Metadata catalog <- showed the search. Direct matches only.
Ontology annotation development underway for greater automation

Final step

input data ranges on all levels of curation:

full curated
minimally curated
other data coming

common semantic model

convert into SDTM on export

Basic data integration tools to "understand the data better"
What makes this platform different? - Ontologies

access to robust metadata
Cpath spends more energy on aggregation of data
Searching for similar disease
Being able to search for things that are not explicitly mentioned

R shiny interrogator
exploration across diseases
data dictionary

These are notes from a seminar?

yea


NLOHMANN_JSON_SERIALIZE_ENUM potentially requires duplicate definitions

As many others have said this is a pretty fantastic library.
I'm trying to serialize some enums and this works (with the right to_json and from_json functions defined but not shown here):
struct Foo_t
{
  enum class color
  {
      none = -1,
      black = 0,
      red,
      blue,
      green
  };
  int number{0};
  color foo_color{color::none};
};

NLOHMANN_JSON_SERIALIZE_ENUM(Foo_t::color,
  {
    {Foo_t::color::none, nullptr},
    {Foo_t::color::black, "black"},
    {Foo_t::color::red, "red"},
    {Foo_t::color::blue, "blue"},
    {Foo_t::color::green, "green"}
  })

however if I want to use those string definitions independent of json then I have to define another mapping, something like
static const std::unordered_map<Foo_t::color, const std::string> color_map =
{
    {Foo_t::color::none, ""},
    {Foo_t::color::black, "black"},
    {Foo_t::color::red, "red"},
    {Foo_t::color::blue, "blue"},
    {Foo_t::color::green, "green"}
};

and unfortunately this doesn't work:
NLOHMANN_JSON_SERIALIZE_ENUM(Foo_t::color, color_map)

because the macro is expecting something that can be converted to an array of std::pairs.
Is there way to not have to duplicate the definitions?

I would like to use NLOHMANN_JSON_SERIALIZE_ENUM  in combination with magic_enum https://github.com/Neargye/magic_enum as follows:
NLOHMANN_JSON_SERIALIZE_ENUM( MyEnum, magic_enum::enum_entries<MyEnum> )
The required change would be minimal, but would be of great value for this kind of combination of libraries.


TransferUtility.UploadDirectoryAsync has incorrect URL when ServiceURL is set (i.e. when using LocalStack)

When TransferUtility is used to copy a folder to an S3 bucket and the ServiceURL is set the transfer fails with "No such host" because the URL that is generated is incorrect.
So if you set the ServiceURL to "http://localhost:4572" (i.e. the ServiceUrl of LocalStack's S3 service) call UploadDirectoryAsync where the Bucket is called "MyBucket" and watch the AWS HTTP call using a tool like Fiddler you will see that the URL that the AWS SDK tries to use is something like this:-
http://MyBucket.localhost:4572/Sccm file test.csv
but it should be:-
http://localhost:4572/MyBucket/Sccm file test.csv
The call to UploadDirectoryAsync fails with "No such host".
Expected Behavior
The call to UploadDirectoryAsync should be successful.
Current Behavior
The call to UploadDirectoryAsync  fails with "No such host".
Possible Solution
Line 130 of AmazonS3PostMarshallHandler.cs incorrectly concatenates the bucket name and the ServiceURL's host name:-
ub.Host = string.Concat(bucketName, ".", ub.Host);
Steps to Reproduce (for bugs)
Create a .NET Core Console application with the following code and add the AWSSDK.S3 NuGet package. Run the program and the error occurs:-
using System;
using Amazon;
using Amazon.S3;
using Amazon.S3.Transfer;
namespace AWSSDKS3TransferUtilityTest
{
class Program
{
static void Main(string[] args)
{
TransferUtility transferUtility = CreateTransferUtility();
        TransferUtilityUploadDirectoryRequest transferUtilityUploadDirectoryRequest = new TransferUtilityUploadDirectoryRequest
        {
            Directory = @"C:\Temp",
            BucketName = "MyBucket"
        };

        transferUtility.UploadDirectoryAsync(transferUtilityUploadDirectoryRequest).Wait();
    }

    private static TransferUtility CreateTransferUtility()
    {
        AmazonS3Config config = new AmazonS3Config()
        {
            RegionEndpoint = RegionEndpoint.GetBySystemName("eu-west-1"),
        };

        config.ServiceURL = "http://localhost:4572";

        IAmazonS3 amazonS3 = new AmazonS3Client(config);

        TransferUtility transferUtility = new TransferUtility(amazonS3);

        return transferUtility;
    }
}

}
Context
It is not possible to use LocalStack's support of S3 with the AWS SDK.
Your Environment

AWSSDK.Core version used: <IP_ADDRESS>
Service assembly and version used:
Operating System and version: Windows 10
Visual Studio version: Visual Studio 2017
Targeted .NET platform: .NET Core 2.1

config.ForcePathStyle  = true should help you

Thanks @klaytaybai , that does indeed solve the problem.


Modal dialogs should reduce noise around dialog

Related to https://github.com/microsoft/vscode/issues/122520
We should do something to the entire workbench when a modal dialog is open: either fade out, grayscale, blur, etc. Something that indicates to the user that everything but the modal dialog should be currently ignored.

Blur

Grayscale

Blur + Grayscale

Brightness

Here's what GH does:

Agree, I vote for brightness.

This is https://github.com/microsoft/vscode/issues/122393

closing as duplicate


Download correct ddl for 64 system on windows with Plexon

Currently plexon download is fixed to use the 32 bits architecture. This PR adds code that checks the architecture and selects the appropriate file to download based on that query.
There is a warning that I think should be removed in the code. Let me know what you think.

Hello @h-mayorquin! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file neo/rawio/plexon2rawio/plexon2rawio.py:

Line 84:1: W293 blank line contains whitespace
Line 91:100: E501 line too long (111 > 99 characters)
Line 93:81: W291 trailing whitespace
Line 95:100: E501 line too long (112 > 99 characters)


Make the icon in the task bar more visible in some cases

Description of the new feature/enhancement
I have dark mode enabled on my Win10.
As some might know the task bar is some kind of translucent, so the desktop wallpaper influences the exact shade of "black" for the task bar.
If I have a black wallpaper, the taskbar is very dark and the grey terminal icon is visible.

But if the wallpaper is bright grey, the task bar becomes a little brighter and makes the terminal icon hard to see.

Proposed technical implementation details (optional)
Add some kind of board to the application icon

Maybe the HC icon can be used in this situation. But I think a user would need to manually choose this in a setting (easiest way to fix the issue.)

Changing "accent color" and showing on/off accent colors in "start, taskbar, and action center" might help your issue. Check Settings->Colors in the control panel, please.

I don't believe changing the icon in this way follows design guidelines. I'd recommend playing with the high contrast settings and the accent colors, as mentioned above. 🙂 I'll close out this issue since we won't be changing the taskbar icon.


docs: Deployment instructions for Platform.sh

We recently created a template project for Probot on Platform.sh, and we wanted to share it along with general deployment instructions.
Let me know if I'm missing anything! All the best.

View rendered docs/deployment.md

we are about to release a new version of Probot which makes it easier to work with in a serverless environment:
https://github.com/probot/probot/issues/1286
Is that relevant for platform.sh? Sorry to make you wait for so long, we had to focus on getting this out first, which will also in updates of the deployment documentation. If you have any feedback on the APIs, now is the best time

closing due to inactivity. Feel free to reopen a new pull request if you are still interested in adding the docs for platform.sh


Using specific fonts

I am using pdfbox for an application, the fontbox package is using too much space, is there a way to reduce the number of imported fonts.

Duplicate of #103. In short, If you know what kinds of encoding will be used in the PDFs you will be dealing with, you should be able to remove some of the FontBox resources for other encoding types.


Add fit_transform?

See discussion at #16

I think for transformers it would make sense to require exactly two methods:

fit_transform
transform

No need for a separate fit implementation for transformers. When an ML pipeline is first fit, all of the transformers have to do a fit and a transformation, so I don't think fit needs to be separate. And of course using fit_transform allows for optimizations, as @davidbp mentioned.
The signature of fit_transform would probably look like this:
fitted_transformer, Xout = fit_transform(transformer, Xin)

Thanks for that suggestion.
| No need for a separate fit implementation for transformers.
I dunno.  It  seems like a non-trivial complication to the API. No separate fit means some special-casing in model composition. In MLJ we expect every model to have a fit and this is pretty central to the learning network stuff. (If you can stomach it, see our paper here). Conceptually, predict and transform are treated very similarly - they're just functions depending on a learned parameter that you generate with fit.
If fit_transform is just sugar for transform(fit(...)) then I don't think it's justified in a basic interface. Every name we add to the namespace should work hard to justify it's existence. Can you think of a use case where the optimisations gained are significant? I can imagine one avoids some data conversions (like DataFrame -> matrix -> DataFrame), but with the right "data front end" (which I'm still thinking about) this issue would have a workaround.
[I know some have argued for just one "operation" (predict, say) for added simplification, but this goes too far, in my view. In the current LearnAPI proposal, predict is distinguished by the fact that the ouput is a (proxy for a ) "target",  a general notion we make reasonably precise in the docs, and we enable dispatch on the type of proxy.  transform need not have this interpretation, but can have an "inverse". As we see from sk-learn and MLJ, allowing algorithms to implement more than one operationpredict /  transform /inverse_transform is both natural and useful.]

Okay, here's a variation on your idea that doesn't require adding to the namespace. Each transformer implements one transform and one fit:
Case 1: static (non-generalizing) transformers
fit(strategy, X) -> model # storing `transformed_X`
transform(model) -> model.transformed_X

with a convenience fallback
transform(strategy, X) = transform(fit(strategy, X))

Case 2: generalizing transformers:
fit(strategy, X) -> model # storing `transformed_X` and `learned_parameters`
transform(model, Xnew) -> transformed_Xnew # uses `model.learned_parameters`

with a convenience fallback
transform(strategy, X) = transform(fit(strategy, X).transformed_X)

I'm not sure we'd want to keep a reference to an intermediate transformed data set in a trained transformer. That would prevent the garbage collector from freeing that memory as long as the pipeline is still around.
It also feels conceptually a little muddy, but that's just a feeling that I haven't been able to put into more concrete terms yet. :)

Case 2: generalizing transformers:
fit(strategy, X) -> model # storing `learned_parameters` and any inspectable byproducts of algorithm
transform(model, Xnew) -> transformed_Xnew # uses `model.learned_parameters`

with a convenience fallback
transform(strategy, X) = transform(fit(strategy, X), X)

Hmm, that form doesn't allow for optimizations, but, as you said, maybe there's not really that many fit-then-transform cases that have a large benefit from optimizations.

In #30 an implementation can explicitly overload transform(strategy, data) to provide a one-shot method with no issues. I think providing a universal fallback is a bad idea, as it could lead to type instabilities, and confusion in debugging ("hidden knowledge").

On dev, a learner can implement transform(learner, X), as shorthand for transform(fit(learner, X), X) or transform(fit(learner), X) for "static models".


Revert #5689 to fix build

This change reverts #5689 which broke validatesrunner postsubmits (see https://github.com/apache/beam/pull/5689#issuecomment-398908901)
R: @aaltay @lukecwik
CC: @alanmyrvold

Is there a bug to re-apply the reverted change?

Added a comment to https://issues.apache.org/jira/browse/BEAM-4592.
On Wed, Jun 20, 2018 at 3:59 PM Ahmet Altay<EMAIL_ADDRESS>wrote:

Is there a bug to re-apply the reverted change?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/beam/pull/5706#issuecomment-398923796, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABAZn-O2_qo7u6LQp1Fo1LDrsNfMGR0gks5t-tPQgaJpZM4UwOdA
.

Looks like the revert fixed some tests, but the test_wordcount_fnapi_it ValidatesRunner test is still failing.
Before this change: FAILED (errors=4, failures=1) https://builds.apache.org/job/beam_PostCommit_Python_Verify/5368/consoleFull
After this change: FAILED (errors=1) https://builds.apache.org/job/beam_PostCommit_Python_Verify/5379/consoleFull
Logs:
======================================================================
ERROR: test_wordcount_fnapi_it (apache_beam.examples.wordcount_it_test.WordCountIT)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/jenkins/jenkins-slave/workspace/beam_PostCommit_Python_Verify/src/sdks/python/apache_beam/examples/wordcount_it_test.py", line 79, in test_wordcount_fnapi_it
    on_success_matcher=PipelineStateMatcher()))
  File "/home/jenkins/jenkins-slave/workspace/beam_PostCommit_Python_Verify/src/sdks/python/apache_beam/examples/wordcount_fnapi.py", line 128, in run
    result = p.run()
  File "/home/jenkins/jenkins-slave/workspace/beam_PostCommit_Python_Verify/src/sdks/python/apache_beam/pipeline.py", line 395, in run
    self.to_runner_api(), self.runner, self._options).run(False)
  File "/home/jenkins/jenkins-slave/workspace/beam_PostCommit_Python_Verify/src/sdks/python/apache_beam/pipeline.py", line 408, in run
    return self.runner.run_pipeline(self)
  File "/home/jenkins/jenkins-slave/workspace/beam_PostCommit_Python_Verify/src/sdks/python/apache_beam/runners/dataflow/test_dataflow_runner.py", line 48, in run_pipeline
    self.result = super(TestDataflowRunner, self).run_pipeline(pipeline)
  File "/home/jenkins/jenkins-slave/workspace/beam_PostCommit_Python_Verify/src/sdks/python/apache_beam/runners/dataflow/dataflow_runner.py", line 350, in run_pipeline
    super(DataflowRunner, self).run_pipeline(pipeline)
  File "/home/jenkins/jenkins-slave/workspace/beam_PostCommit_Python_Verify/src/sdks/python/apache_beam/runners/runner.py", line 170, in run_pipeline
    pipeline.visit(RunVisitor(self))
  File "/home/jenkins/jenkins-slave/workspace/beam_PostCommit_Python_Verify/src/sdks/python/apache_beam/pipeline.py", line 436, in visit
    self._root_transform().visit(visitor, self, visited)
  File "/home/jenkins/jenkins-slave/workspace/beam_PostCommit_Python_Verify/src/sdks/python/apache_beam/pipeline.py", line 791, in visit
    part.visit(visitor, pipeline, visited)
  File "/home/jenkins/jenkins-slave/workspace/beam_PostCommit_Python_Verify/src/sdks/python/apache_beam/pipeline.py", line 791, in visit
    part.visit(visitor, pipeline, visited)
  File "/home/jenkins/jenkins-slave/workspace/beam_PostCommit_Python_Verify/src/sdks/python/apache_beam/pipeline.py", line 791, in visit
    part.visit(visitor, pipeline, visited)
  File "/home/jenkins/jenkins-slave/workspace/beam_PostCommit_Python_Verify/src/sdks/python/apache_beam/pipeline.py", line 794, in visit
    visitor.visit_transform(self)
  File "/home/jenkins/jenkins-slave/workspace/beam_PostCommit_Python_Verify/src/sdks/python/apache_beam/runners/runner.py", line 165, in visit_transform
    self.runner.run_transform(transform_node)
  File "/home/jenkins/jenkins-slave/workspace/beam_PostCommit_Python_Verify/src/sdks/python/apache_beam/runners/runner.py", line 208, in run_transform
    return m(transform_node)
  File "/home/jenkins/jenkins-slave/workspace/beam_PostCommit_Python_Verify/src/sdks/python/apache_beam/runners/dataflow/dataflow_runner.py", line 586, in run_ParDo
    input_step = self._cache.get_pvalue(transform_node.inputs[0])
  File "/home/jenkins/jenkins-slave/workspace/beam_PostCommit_Python_Verify/src/sdks/python/apache_beam/runners/runner.py", line 286, in get_pvalue
    return self._cache[self.key(pvalue)]
KeyError: (u'read/Read/Impulse', None)
-------------------- >> begin captured logging << --------------------
root: DEBUG: Connecting using Google Application Default Credentials.
oauth2client.transport: INFO: Attempting refresh to obtain initial access_token
root: DEBUG: Connecting using Google Application Default Credentials.
oauth2client.transport: INFO: Attempting refresh to obtain initial access_token
root: ERROR: Error while visiting read/Read/Split
--------------------- >> end captured logging << ---------------------

I believe the other failure is being caused by: https://github.com/apache/beam/pull/5685


cmd/update: adjustments for setup-homebrew in brew CI

Fix the master of brew being used in brew PRs.

test default formula is always a good test for this. Maybe should make it fail or something if it doesn't do as expected.

Thanks for cleaning up my mess here, @Bo98.

Maybe should make it fail or something if it doesn't do as expected.

@Bo98 great idea.


Server does not run with Symfony 4.1

I'm want to run the server in a Symfony4 application.
Launching the server does not work.
I wonder if other people are running into the same issues?
This is the source code of my project: https://github.com/murtho/SuperSocket

I am running this bundle under Symfony 4.1 more less without major issues. What do you mean by "launching the server"? Could you share your output while trying to start the WebSocket server?

When I run the command php bin/console gos:websocket:server from my terminal nothing happens. there is no output at all
I need to push Ctrl + C to exit the command.

@murtho Hmm... that's strange. That suggests something might be wrong with the configuration. I'll clone your repo and look at it further when I'll be available.

In order to get output in your terminal, the monolog logger needs to be installed. I also managed to find out that in a virtual machine with vagrant/puphpet you'll need to configure your firewall to open the port you're listening to and the IP-address in the configuration needs to match the IP-address on the javascript library


Added example of ClientBuilder customization to readme

Closes #251

Thank you, your suggestion makes sense to me.


Change deprecated findOneAndRemove to findOneAndDelete (mongoose v8 compatibility)

Fix #112

Hi @santiq
Thanks for contributing. There's one more issue that needs to be fixed until we're compatible with v8 and it's changing count to countDocuments when counting the number of records matching given filters:
https://github.com/SoftwareBrothers/adminjs-mongoose/blob/master/src/resource.ts#L71
The problem is that countDocuments usage is a bit different and we need to make sure that it works with the provided filters.
Also, please bump mongoose version in devDependencies and peerDependencies to v8. We can make it a breaking change.

@dziraf
According to mongoose documentation

The countDocuments() function is similar to count(), but there are a few operators that countDocuments() does not support.

Below are the operators that count() supports but countDocuments() does not, and the suggested replacement:
$where: $expr
$near: $geoWithin with [$center] (https://www.mongodb.com/docs/manual/reference/operator/query/center/#op._S_center)
$nearSphere: $geoWithin with $centerSphere

It is safe to merge since people would need to upgrade to Mongoose v8 to use this anyway.

This deprecated method is already breaking many apps which uses mongo v8 so this should be fixed ASAP. Had to manually change findOneAndRemove to findOneAndDelete in node_modules for existing apps to function correctly.


[5.1] Filesystem::put accepts config array

This fixes #8788
Retain default setting of sending a string to determine visibility option, but allow other config values to be set.
Higher functions look for these values to be optionally overridden, so now you can send along config options to override defaults.

@GrahamCampbell Did notice an issue with the logic so changed it slightly.
In your suggested version, if someone send an array but omits the visibility option, then visibility won't be set and later functions will fail.

Use getDriver to get the Flysystem driver if you need more customization.


Add 2019 Honda Insight

Related to closed PR #417 due to fp issue. Was able to get fw ids from the query script.

Do you have a route we can use for the regression tests? Please put one in test_car_models with a fixed fingerprint: like https://github.com/commaai/openpilot/blob/master/selfdrive/test/test_car_models.py#L293

And you also need to add the fingerprint to the list of fingerprints since that is still used in the is_ecu_disconnected fingerprint. To make sure it doesn't interfere with other cars fingerprinting add a list with IGNORED_FINGERPRINTS like https://github.com/commaai/openpilot/blob/master/selfdrive/car/toyota/values.py#L269

@pd0wm Hey, thanks for the additional comments, I added the actual fp and added it to ignored list.
I will provide a route later today.

The route doesn’t matter

Pushed a fix to the fingerprint test to master:https://github.com/commaai/openpilot/commit/6eca88e9bcc4c6d32304444c6565ca6be0d77bbb
Can you rebase?


Feature Request: Add normalizer_fn to sequence_numeric_column.

System information

Have I written custom code: no
OS Platform and Distribution: macOS Sierra 10.12.6
TensorFlow installed from: binary
TensorFlow version: 1.8.0
Python version: 3.6.5
Bazel version: N/A
GCC/Compiler version: N/A
CUDA/cuDNN version: N/A
GPU model and memory: N/A
Exact command to reproduce: N/A

Describe the problem
Currently, sequence_numeric_column does not support a normalizer_fn, wondering if this could be added in like in numeric_column.

Added a PR #19649 for the support.


Use functools.cache more

It'd be great to stick a cache on dask.array.svg.svg and normalize_chunks.
These seem like highly cacheable functions, and would help with https://github.com/pydata/xarray/issues/8902. The svg would help with the ds._repr_html_() of that dataset.

normalize_chunks is a bit weird since it accepts all kinds of input, some of them are mutable and therefore not hashable

If it's always a tuple in xarray we can likely offer a "cachable" version (or we can cache on a best effort basis)

We are indeed sending dask tuples always!
https://github.com/pydata/xarray/blob/1bb867d573390509dbc0379f0fd318a6985dab45/xarray/core/dataset.py#L246-L258
preferred_chunks comes from the storage library is always a tuple of ints.
chunk_shape can be either a tuple of ints or with some "auto" sprinkled in : ('auto', 721, 1440)


Add makefile & Update README

Sorry for missing this, I've been busy for the last few days.
This is great! Thanks a lot for cleaning up the code and adding some build scripts.


Hashtag Pattern UTF (Feature)

Thanks you very much for this project.
I was taking a look into your implementation and i think that you could update the Hashtag regular expression to accept non english words etc..
Example:
#(\w+) fails with #Mañana
Solution:
([#＃]+)([0-9A-Z_]*[A-Z_]+[a-z0-9_üÀ-ÖØ-öø-ÿ]*)
Hope to help you work on it soon.
Jeff.

@jeffersonlicet You're very much welcome. Thanks for the regex input, I will need to create some instrument tests to make sure that pattern will work on any case.
In the meantime, do you have more test cases? I'm sorry I'm not familiar with Spanish naming customs.

We use accent marks everywhere.
For a better internalization I allowed accent marks at the start of the word:
([#＃]+)([0-9A-Z_À-ÖØ-öø-ÿ]*[A-Z_]+[a-z0-9_üÀ-ÖØ-öø-ÿ]*)
I think all spanish words will work, for example:
#Mañana
#CreaciónDivina
#TúVendrás
#CuatroArtículos
#MásNadaQueda
#SeFueÉste
#ÉsteNoEra
#Ñame
You can play with it here:
Regexr

I've added RegexTest which for some reason passes with #(\\w+). While ([#＃]+)([0-9A-Z_À-ÖØ-öø-ÿ]*[A-Z_]+[a-z0-9_üÀ-ÖØ-öø-ÿ]*) only captures Creación out of CreaciónDivina.
I honestly don't know why the result from Regexr can be different with junit test. Would you kindly run the test on your computer and see if it passes?

Sorry, i used the i flag that means it's not case sensitive. Change it to:
static final Pattern PATTERN_HASHTAG = Pattern.compile("(?i)([#＃]+)([0-9A-Z_À-ÖØ-öø-ÿ]*[A-Z_]+[a-z0-9_üÀ-ÖØ-öø-ÿ]*)");
Where (?i) disables the case-sensitive engine.

Okay that works but I'm thinking of using (?i)[#＃]([0-9A-Z_À-ÖØ-öø-ÿ]*[A-Z_]+[a-z0-9_üÀ-ÖØ-öø-ÿ]*) to prevent multiple hashtags. What do you think?

Ohh. Yes. That's perfect.

Sorry for late response, version 0.12.0 has been published with that regex pattern. The patterns are also customizable now with static methods.
Thank you for your help and let me know if you have any other improvement or fix! :)


Make Razor.Design a transitive dependency

By making Razor.Design a transitive dependency of Blazor.Build we can
avoid the need for users to care which version of Razor we depend upon,
and take it out of the templates.

This linked to Issue #662 isn't it ?

Yes that's right.

Note for #662 - this will still be a problem until we pick up 2.1.0-rc of Razor. This change https://github.com/aspnet/Razor/pull/2277 is needed.


out of date requirements?

"Download and Install PackageManagement PowerShell Modules Preview from here."
The PackageManagement module is available by default on most windows installations now.  It would only be required for a much older OS.  And there is no preview version of it any more.

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

ID: 67be3bb5-b150-e197-08fb-eaa3bf8f7a0b
Version Independent ID: 5e2b6dc2-992c-5b76-955d-9cb57da9fbcc
Content: Improve Office 365 sign-in for Yammer with auto-acceleration policy
Content Source: yammer/manage-yammer-users/auto-acceleration-policy-for-signing-in.md
Service: yammer
GitHub Login: @IrenePasternack
Microsoft Alias: v-irpast

@officedocsbot assign @mypil

@jordanmills - Thank you for submitting feedback.
I will get this issue over to the O365ITPro writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

Dear @jordanmills ,
The PackageManagement module is available by default on most windows installations now. It would only be required for a much older OS. - it is available by default only in PowerShell 5.0 and newer (not depending on OS), so those who are using PowerShell 3.0 or 4.0 still need PackageManagement PowerShell Module to be installed.
And there is no preview version of it any more. - yes, current version is 1.3.1, so the link should be updated, I am going to create PR
Thank you

I updated the topic.
From: MaratMussabekov<EMAIL_ADDRESS>Sent: Thursday, April 11, 2019 4:43 AM
To: MicrosoftDocs/OfficeDocs-O365ITPro<EMAIL_ADDRESS>Cc: Irene Pasternack (Steyer Associates Inc<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/OfficeDocs-O365ITPro] out of date requirements? (#416)
Dear @jordanmillshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjordanmills&data=02|01|v-irpast%40microsoft.com|6f143233a16d487b48f708d6be72e207|72f988bf86f141af91ab2d7cd011db47|1|0|636905797966200771&sdata=RRiSWW7o%2B9PlAL407nX7DKIwuLr8wvkCTfPUPTeklcQ%3D&reserved=0 ,
The PackageManagement module is available by default on most windows installations now. It would only be required for a much older OS. - it is available by default only in PowerShell 5.0 and newer (not depending on OS), so those who are using PowerShell 3.0 or 4.0 still need PackageManagement PowerShell Module to be installed.
And there is no preview version of it any more. - yes, current version is 1.3.1, so the link should be updated, I am going to create PR
Thank you
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FOfficeDocs-O365ITPro%2Fissues%2F416%23issuecomment-482080726&data=02|01|v-irpast%40microsoft.com|6f143233a16d487b48f708d6be72e207|72f988bf86f141af91ab2d7cd011db47|1|0|636905797966200771&sdata=pR5GaG9usKlQg4n2vuKXRAVGYzEksMRJXLjeMx1nUqA%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAkqBo20XOS5a0byCfHz6ujC1Z3reLajNks5vfx_SgaJpZM4cn8v8&data=02|01|v-irpast%40microsoft.com|6f143233a16d487b48f708d6be72e207|72f988bf86f141af91ab2d7cd011db47|1|0|636905797966210768&sdata=tkv90CdCYP1BXEIgFgbZ8syIFx1quK6pzNat6GDDmO4%3D&reserved=0.

Dear @IrenePasternack ,
I think there is some kind of mistake in the updated version of the topic. Could you please take a look to the wrong link at the second prerequisite?
Thank you

Thank you Marat. I have updated the topic so the link appears properly.
From: MaratMussabekov<EMAIL_ADDRESS>Sent: Saturday, April 13, 2019 9:01 AM
To: MicrosoftDocs/OfficeDocs-O365ITPro<EMAIL_ADDRESS>Cc: Irene Pasternack (Steyer Associates Inc<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/OfficeDocs-O365ITPro] out of date requirements? (#416)
Dear @IrenePasternackhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FIrenePasternack&data=02|01|v-irpast%40microsoft.com|cb17b2ecdb3743919e7908d6c029437f|72f988bf86f141af91ab2d7cd011db47|1|0|636907680810010398&sdata=I3GNsOBVeYeDCPZxL%2BjsoQD1dIinHWwyNJ4Vb4n6ok0%3D&reserved=0 ,
I think there is some kind of mistake in the updated version of the topic. Could you please take a look to the wrong link at the second prerequisite?
Thank you
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FOfficeDocs-O365ITPro%2Fissues%2F416%23issuecomment-482827345&data=02|01|v-irpast%40microsoft.com|cb17b2ecdb3743919e7908d6c029437f|72f988bf86f141af91ab2d7cd011db47|1|0|636907680810020392&sdata=TOVQwNgo7sUa52PZ%2BsMRFgGPYfvuGk9Fb6PSJcx4gqU%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAkqBo6WEQFFQibMkNQmVkD4axvSn7wH8ks5vgf9NgaJpZM4cn8v8&data=02|01|v-irpast%40microsoft.com|cb17b2ecdb3743919e7908d6c029437f|72f988bf86f141af91ab2d7cd011db47|1|0|636907680810020392&sdata=FxjPUcTN25f2G8McAgwX8apDIml1zhTnDE9HooPksiM%3D&reserved=0.

@jordanmills - From our understanding, the issue has been resolved. If you feel it hasn't been resolved, please re-open this issue.
Thank you for your contribution to make the docs better! Much appreciated!

@officedocsbot close


Input Number 'box' not visible in UI in 0.79.0

Home Assistant release with the issue:
0.79.0
Last working Home Assistant release (if known):
0.78.2
Operating environment (Hass.io/Docker/Windows/etc.):
Ubuntu 18.04 under ESXi 6.5
Component/platform:
Frontend UI
Description of problem:
For input number components with mode set to 'box', the entry UI is hidden from the frontend. Listing icon and name can be seen, but the textbox and controls are missing. Issue occurs on Safari/Chrome on macOS Mojave, and Safari/iOS app on iOS 12.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
        time_offset:
            name: 'Time Offset'
            mode: 'box'
            initial: !include ../../../var/input_number/time_offset.yaml
            min: 0
            max: 60
            unit_of_measurement: !include ../../includes/units/time/minutes.yaml

Traceback (if applicable):

Additional information:

I've been tearing hass apart trying to figure out why my control went messing. I thought it was something I changed.

Same issue in 0.79.1, and label for input_number with type: slider is also missing.
How it should appear:

How it appears:

Same problem here

Seeing this same issue.

Same:

box does not show. Slide show
click on the icon does nothing as on another input_* show the entity detail

Same issue in 0.79.3

Same issue stated above, value can be set via "set state".

yep, same here (tested in 0.79.1, .2 and .3)

Can anyone elaborate on their setups?  Server OS, Client OS, Browser.  Just curious if there is a common thread…

Server OS: Ubuntu 16
Python: 3.6
Intallation type: HomeAssistant in virtual env
HomeAssistant: 0.79.3
Client OS: Windows 10
Browsers: Chrome and Firefox (last version)

Server OS: Debian 9
Python: 3.6
Intallation type: Hassio on virtual debian server
HomeAssistant: 0.79.3
Client OS: Windows 10 / Android
Browsers: Chrome

Server OS: Raspbian GNU/Linux 9.4 (stretch)
Python: 3.5.3
Installation type: Hassbian
HomeAssistant: 0.79.3
Client OS: macOS High Sierra / iOS 11 / iOS 10
Browsers: Chrome/Safari

So three radically different use cases. Chalk that up as no to a common thread. ¯_(ツ)_/¯

Is there anyone out there not having this issue?

Same issusr

Well there is a common thread I think, aren't we all using a virtual environment?
pplucky --> Hassibian --> Virtual environment
Myself --> Virtual environment --> hass.io --> docker layer
SubniC --> Virtual environment
jamieshaw --> Hass.io --> docker layer
etc...

Nope. Not me; I like to be complicated.
Reverse Proxy (Server 1) -> Hass (PIP) -> Ubuntu 18.04 -> VMware ESXi (Server 2)

Same problem ....

The same issue - box is not displayed, number on the right side of slider is not displayed.
Running Homeassistant - HassOS - RPi 3

I am also behind an HTTPS -> HTTP reverse proxy, on raspbian (custom install).

I got the same problem

Can you describe your system, so that the Devs can get an idea about whats going on.

FYA @home-assistant/core

FYA @home-assistant/core

Is it @home-assistant/core command?

FYA @home-assistant/core

Is it @home-assistant/core command?

FYA @home-assistant/core

Is it @home-assistant/core command?

Permission denied (user root)

Does anyone have an idea what about it?

I'm not a dev so i don't know, maybe somebody could quote a dev in ? I'm on my phone atm.

Seeing the same issue.
HassIO dockerized on a Debian VM.
Version: 0.79.3

How can I contact developers?

@RomLab In theory; this issue tracker should do the trick; maybe message one of them directly on here or Twitter?

I don't know, the issue is on page 3 of issues. Can always start a pull request to get their attention alas.

I don't know, the issue is on page 3 of issues. Can always start a pull request to get their attention alas.

Someone would need to code up a fix first.

My observation:
If Lovelace UI is used, there are sliders displayed with a number on the right side and one box displayed correctly, rest of it not
Classic UI

LovelaceUI

I know, it's a "bad" way of getting attention. So I don't condone it.
And I forgot to mention that I am not using Lovelace UI. Tomorrow I'll try reverting to an old backup and see if the problem persists.

Can confirm this issue is still present in 0.80.0b5 - I've raised this in the Home Assistant Polymer Repo as it looks to be more frontend related.

Any news with the version 0.80.0 ?

I have the same issue since updating from 0.77 to 0.79.3 and now still in 0.80.
Ubuntu server 16.04.

On my other setup, with 0.80 on lovelace the inpout box is back. Weird.

I would need to solve this problem, but I don't know how to do it. I think it is big problem-

Issue persists for me in 0.80.0

seeing the same thing.  /following

Just converted over to Lovelace, and everything shows up like it's supposed to.  To add to this display issue, I had a group displayed that the control for it was hidden, after converting to Lovelace, the control was showing up again.

Issue persists after update to 0.80.0. It works while on Lovelace.

Rolled back to 78.3 so I can use input numbers again. Disappointed that this issue isn't being addressed in the updates since it's a very basic component that's clearly been broken by a bug. Especially since there seems to be a fix available.  https://github.com/home-assistant/home-assistant/issues/9779

Same on a fresh install of Hassio. Should not be hard to reproduce. Does anyone that uses a Hass > 0.78 on a Rpi or a docker not have this issue?

Can also confirm same issue for box. Docker on Synology NAS. input number slider works. Same for input list and text.
When using Lovelace frontend it does show without any issues.

I'm on latest version (0.80.3) and still the same.

A fix was merged into the home-assistant-polymer repo, should be in one of the next beta builds or release.

Works again since 0.81.0 :tada:

Indeed ! Thanks a lot!!

0.81.0 - is is back as is should be. Thanks a lot.


feat: split workflows

-
-

Issue
--

Dependencies
--

Decisions
Split workflows by concerns. On each push, we want to test building the docker image. On each pull request to the main branch, we use the actions @rafaeldamasceno  and @afonsojramos found to log into the registry, extract metadata and build plus push the image. I used the official releases/versions of these actions rather than commit hashes.  Consider this more of a proposal to make this easier to read and simpler.

I think that we could reduce this PR to its main goal, which is simply changing the commit tags to version tags for the actions (which might introduce some bugs in the future if the actions change, but in the end I agree with the change), and the creation of the separate Docker Build workflow. It should NOT remove step names as they are useful nor should it remove workflows from running on a PR.

That was not the main goal, the main goal was to simplify the workflows. So instead of if logic to filter a specific step in the workflow, essentially having two "logical flows", create two separate workflows (one for build one for push).
Changing the tags to versions relies on an official release, but allows them to publish new versions that are retro compatible (any non-retro compatible change will mean the major portion of the release version is updated according to semver).
So I guess, to continue:
1 - why are step names that relevant (all I see is instead of, for example, run: cargo fmt i see  Formatting code
2 - why do I want an workflow to run on a pull request, and not just on a push (since any pull request means a push at least)

So I guess, to continue:
1 - why are step names that relevant (all I see is instead of, for example, run: cargo fmt i see Formatting code
2 - why do I want an workflow to run on a pull request, and not just on a push (since any pull request means a push at least)

@joao-conde Please check out the answers above.


Polymorphic Relationships in acceptance tests

Hi there,
First of all, great library! Saved us a ton of time mocking everything.
We're running into an issue with polymorphic relationships and ember-data though. This is the situation:
Step model:
DS.Model.extend({
  unit: DS.belongsTo('unit', { polymorphic: true, async: false }),
  unit_type: DS.attr('string'),
});

Unit model:
DS.Model.extend({
  bucket: DS.belongsTo('bucket', { async: false })
});

Lecture model (this is a unit):
Unit.extend({
  step: DS.belongsTo('step', { async: false })
});

Now, in an acceptance test, we're doing the following:
let course = server.create('course', { stage_ids: [1] }),
      stage = server.create('stage', { course_id: 1 }),
      lecture = server.create('lecture', { step_id: 1 }),
      step = server.create('step', { stage_id: 1, unit_type: 'Lecture', unit_id: lecture.id }),
      url = `/studio/courses/${course['id']}/steps/${step['id']}`;

Whatever I do though, I cannot get the unit (a Lecture in this case) using model.get('unit'); in an Ember route or controller (the code with a real API works, the tests with mirage don't).
How should I model a polymorphic relationships in Ember? Am I doing something wrong? I'm just creating factories for each model like in the above code and I hoped the unit would resolve automatically to the right type according to the unit_type field. This is unfortunately not happening.

This does not have first-class support yet, unfortunately. I'll have time to look at this more later, but for now you can do something like
server.create('unit', { step_id: 1, unit_type: 'lecture' });

as a temporary workaround. Sorry about the inconvenience!

Thanks for the quick reply!
Unfortunately, this doesn't seem to be working either. model.get('unit') is still null in the route.
I'm doing the following now:
let course = server.create('course', { stage_ids: [1] }),
      stage = server.create('stage', { course_id: 1 }),
      // lecture = server.create('lecture', { step_id: 1 }),
      lecture = server.create('unit', { step_id: 1, unit_type: 'lecture' }),
      step = server.create('step', { stage_id: 1, unit_type: 'lecture', unit_id: lecture.id }),
      url = `/studio/courses/${course['id']}/steps/${step['id']}`;

Do I need to create new factories? I have defined Unit, Lecture and Step already... so I can't think of anything. In the route, in renderTemplate, I print the following:
console.log('new model is', model.get('unit_type'), model, model.get('unit'));
The result is:
new model is lecture Class {id: "1", store: Class, container: Container, _internalModel: InternalModel, isError: false…} null

Also, can I just define the shorthand route this.get('/steps/:id'); or do I need to add something myself?

FYI - I figured it out after a few hours of searching:
I needed the following:
let step = server.create('step', { stage_id: 1, unit_type: 'lecture', unit: unit.id })
(note unit instead of unit_id)
Not sure why unit is working and unit_id is not. :)

FYI - this is still not working the way it should..
let unit = server.create('unit', { step_id: 1, unit_type: 'lecture', wistiaId: '1' });
This does return the right unit now, but the wistiaId property is always null. Any clues?

Should it be wistia_id? Sorry you've been running into issues, I'll have some time to look at this tonight.

It's not wistia_id. We have a Rails backend and use ActiveModelSerializers and wistia_id on the backend is automatically camelCased to wistiaId on the front-end afaik. In any case, I just tried wistia_id as well, and it gives me undefined :/
Thanks for the help so far. Hopefully we can figure this out :)

FYI - if I write my own get('step/:id') endpoint I get the following error:
this.get('/steps/:id', function (db, request) {
    let step = db['steps'].find(request.params.id),
        unit = db['units'].find(step['unit_id']);

    step['unit'] = unit;
    console.log(step, unit);

    return {
      step: step
    };
});

returning { step: step, unit: unit } doesn't work either. The console log prints the unit with the correct wistiaId in this case.
Error:
Assertion Failed: Ember Data expected a number or string to represent the record(s) in the unit relationship instead it found an object. If this is a polymorphic relationship please specify a type key. If this is an embedded relationship please include the DS.EmbeddedRecordsMixin and specify the unit property in your serializer's attrs object. Error: Assertion Failed: Ember Data expected a number or string to represent the record(s) in the unit relationship instead it found an object. If this is a polymorphic relationship please specify a type key. If this is an embedded relationship please include the DS.EmbeddedRecordsMixin and specify the unit property in your serializer's attrs object.

@samselikoff sorry to bother you, but do you have any clue or an idea what we could do here? :)

Sorry about the delay, had a busy week!
There's a lot going on here. Can you be specific about the exact API response you're expecting, and the response you're getting now, and we can work from there?

Sorry for the late reply!
I started tracking the exact responses in my Chrome Network tab, and used these to build up the dynamic responses in my mocked Mirage API, and it works now! Thanks a lot for the help and putting me in the right direction of looking at my actual API responses :)
This can be closed.

@philipdesmedt Hey - curious to hear more about your implementation on the Mirage side.
On ember I have mode's setup like this.
// Notifications Model (with polymorphic relationship)
export default DS.Model({
  subject: DS.belongsTo('subjectable', { polymorphic: true })
});

// Subjectable Mixin
export default Mixin.extend( {
  notifications: DS.hasMany('notification'),
)};

// Booking model
export default Model.extend(Subjectable, {
  notifications: DS.hasMany('notification'),
)};

How would I represent subjectable mixin inside Mirage?


Add SAHETI High School

@EMcConkey1 First stage of verification passed, official domain is saheti.co.za as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@EMcConkey1 Pull request merged. Thank you.


feat: add VS code notifications for autostop

part of the resolution for #7360
the rest will follow after https://github.com/coder/coder/pull/7989 is merged up

Approved. I am just not sure if every 5 seconds is not too frequent to keep calling getWorkspaces

Yeah, I feel the same way. It's the cadence we use in Dashboard so I went with it here.

Closing in favor of https://github.com/coder/vscode-coder/pull/111


Only a newly created intent can be deleted

https://github.com/warden-protocol/wardenprotocol/assets/59832414/5593959c-057a-4b84-b136-80cba7a693f0

not a bug; we cannot remove saved intents


fix generator SyncTables embedded entities "Entity has no assigned table"

Is there any way you can include a test for this use-case, so we can catch it in the future. If not I'll do it by myself, but a bit later (currently on vacations).
You can run tests via starting Docker compose in tests.


Vanna with qdrant using old endpoints from qdrant v.1.0.0 or before

Describe the bug
I am using vanna.ai with a qdrant vector db running in a docker container, and when you try to verify if collection exists using "GET /collections/sql/exists" I get this error in the qdrant container logs "GET /collections/sql/exists HTTP/1.1" 404 which means, the endpoint does not exist, and I think bcs you are based on an old version of qdrant 1.0.0 or before, but the new version uses GET "/collections/{collection_name}"
this
To Reproduce
Steps to reproduce the behavior:

use vanna.ai with newest version of qdrant database
try the example code given in the docs to use vanna with qdrant database

Expected behavior
to access qdrant database and find/create the needed collection
Error logs/Screenshots
If applicable, add logs/screenshots to give more information about the issue.
Desktop (please complete the following information where):

OS: [e.g. docker]
Version: [e.g. 20.04]
Python: [3.9]
Vanna: [2.8.0]

Additional context
also the endpoints used to create new collection changed

@Anush008 could you kindly confirm?

I was using the version 1.6.1 of qdrant, I ll update it to latest, thank you


YM2413+AY8910 to ymf262 error

My conversion step is

Convert ym2413+ay8910 to ymf262+ay8910
ymf262+ay8910 converted to only ymf262

When I use vgm-conv 0.8.0, there is no problem in the whole process, but when I use 0.10.0, the converted ymf262 sounds a lot wrong.

It seems that we got unexpected result if ay8910 to ymf262 conversion is given while the ymf262 part already exists.
May be similar to #63.

It seems that we get an unexpected result if ay8910 to ymf262 conversion is given while the ymf262 part already exists. May be similar to #63.

Under Windows 10 system, I cannot use the following statement:
vgm-conv -f ay8910 -t ymf262 -i 01\ Opening.vgz | vgm-conv -f ym2413 -t ymf262 -o test.vgm
It seems that Windows 10 does not allow multiple statements at the same time.

Please see this comment.


fix: yaml-cpp compatability with CMake < 3.18

Description

Fixes yaml-cpp compatability with CMake < 3.18

See #1322 for details

Corrects the minimum required version to 3.15

Required for list(APPEND)

Related issues

closes #1322
Cherry-pick to

5.11 (old stable)
5.12 (current stable)

@KerstinKeller I would like to get a final approval from you 😀


Suggestion: Host sand spiel from github.

actual thing:
recently my districts public education committee decided to get a lot stricter on internet policies and they recently blocked your website categorizing it as adult content... some of my favorite chromebook games migrated to hosting on github to get around this. (like CelesteClassic and adarkroom) is there anyway you could also host your game on a github.io link like you did with the prototype?
one more thing:
I love how you completely revamped the biological/ecosystem aspect from danball. how ever I would love to add mechanical concepts like wireworld and langton's ant, or something of my own that can be described as a 2d-minecraft-redstone-piston-automata. if you are interested please let me know.
one one more thing:
I tried making something to do with automata with nothing but my heavily restricted school chromebook and I can't resist sharing my results
also, the founder of autodesk made a web automata sandbox thats not as interactive but it's still very neat. https://www.fourmilab.ch/cellab

super cool work with your CA system!! I will keep this in mind and maybe host on github in the future!

thanks for the nice message. Sorry your laptop is so locked down. have you tried the game i'm working on?
https://maxbittker.github.io/alchemy-online/

I would aslo like it to be hosted on github. By any chance, is there a way to run it on repl.it?

I would aslo like it to be hosted on github. By any chance, is there a way to run it on repl.it?

You can run it on Replit. In fact, you can run it anywhere. Wait a moment.

Here. sp-client.html lets you look at Sandspiel, sandspiel.html lets you do everything else.
sandspiel-offline.zip

did you make that client? nice work!

did you make that client? nice work!

Made the sp-client.html file. As for the main sandspiel part, I just edited the JS files to point to relative directories and used the URL for my own copy of the WASM file.

how do i work it? I unziped it, but it wont load.

can you try running it then send me a link to the project?

Open https://sandspiel.adith08.repl.co/sandspiel.html or do mv sandspiel.html index.html

Okay, could you show the output of ls?

ls?

https://replit.com/join/mjsrmnylob-adith08 use to colab on repl.it

https://replit.com/join/mjsrmnylob-adith08 use to colab on repl.it

Okay

https://replit.com/join/ztvewpqdve-adith08

Yep.

also, browsing doesnt work, but overall, i like it. Than you.

https://sandspeil2.adith08.repl.co/ link for anyone who wants it.

actual thing: recently my districts public education committee decided to get a lot stricter on internet policies and they recently blocked your website categorizing it as adult content... some of my favorite chromebook games migrated to hosting on github to get around this. (like CelesteClassic and adarkroom) is there anyway you could also host your game on a github.io link like you did with the prototype?
one more thing: I love how you completely revamped the biological/ecosystem aspect from danball. how ever I would love to add mechanical concepts like wireworld and langton's ant, or something of my own that can be described as a 2d-minecraft-redstone-piston-automata. if you are interested please let me know.
one one more thing: I tried making something to do with automata with nothing but my heavily restricted school chromebook and I can't resist sharing my results also, the founder of autodesk made a web automata sandbox thats not as interactive but it's still very neat. https://www.fourmilab.ch/cellab

https://sandspeil2.adith08.repl.co/

actual thing: recently my districts public education committee decided to get a lot stricter on internet policies and they recently blocked your website categorizing it as adult content... some of my favorite chromebook games migrated to hosting on github to get around this. (like CelesteClassic and adarkroom) is there anyway you could also host your game on a github.io link like you did with the prototype?
one more thing: I love how you completely revamped the biological/ecosystem aspect from danball. how ever I would love to add mechanical concepts like wireworld and langton's ant, or something of my own that can be described as a 2d-minecraft-redstone-piston-automata. if you are interested please let me know.
one one more thing: I tried making something to do with automata with nothing but my heavily restricted school chromebook and I can't resist sharing my results also, the founder of autodesk made a web automata sandbox thats not as interactive but it's still very neat. https://www.fourmilab.ch/cellab

https://sandspeil2.adith08.repl.co/

I'll deploy a wget -r'd copy of https://sandspiel.club/school and rewrite it a bit.

ok. thnak you so much.

new link https://sand.adith08.repl.co/

Want a P2P link?

Sure

that would be great!

Could someone else on Windows MacOS, or any device that can run https://beakerbrowser.com/install/ help? I need someone else to help host the P2P version. It will work on school computers, if that's all you have.

Want a P2P link?

Yes please!

but how do i run it?

but how do i run it?

So, what's your OS?

im on a chromebook

and i cant eneble linux, if thats what you're thinking.

and i cant eneble linux, if thats what you're thinking.

Hmm. Can you open https://copy.sh/v86/ ? If so, you might be able to do a persistent (or semi-persistent) Ubuntu 10 VM and build Beaker from source.

blocked

blocked

Ugh. I'll launch a mirror.

@Adithr123 What browser extension?

securely

securly

securely

Then there's an easy fix. https://sheeptester.github.io/longer-tweets/securly-bypass/ 🙂

@SheepTester figured that out.

nice!

so what now?

@Adithr123 Can you install any browser extension?

no.

no.

What about https://chrome.google.com/webstore/detail/ublock-origin/cjpalhdlnbpafiamejdnhcphjbkeiagm ? You might like it. I noticed you use ABP, which is extremely CPU-intensive and ineffective. https://github.com/gorhill/uBlock/wiki/uBlock-vs.-ABP:-efficiency-compared

abp?

i cant install any extensions.

its installed by admin...

its installed by admin...

Huh

i uninstalled it. I meant that i can only install apps that the admin allows.

i uninstalled it. I meant that i can only install apps that the admin allows.

Huh

???

???

Interesting that someone would let people use that but not a better extension.

ye.

ye.

Have you ever used dirc?

@Adithr123

no

no

Okay. Does the letter P have any significance to you?

no.

should it?

no.

Okay. Just was wondering.

ok.

i have to go to swim, but I will be back online in about 2 hrs.

i have to go to swim, but I will be back online in about 2 hrs.

Okay.

I still have my phone , and I have until 5:20

So what else do u need? To make the p2p?

Also, do u have any other securly bypasses?

Also, do u have any other securly bypasses?

No. :( This may sound weird, but check your email. I have a theory, and I want to test it.

Never mind

Ok

How do u know my email?

How do u know my email?

I don't. At least, I think I don't.

<EMAIL_ADDRESS>

Did not ask...

0K

Delete that, or you'll start getting spam.

Way ahead of u

Deleted it as soon as u said u didn't want it

Gtg

Way ahead of u

LOL. Just had a theory. Also, I realized it's impossible.

I'll be back

What?

theory

What

theory

What

Never mind

what happened to the p2p?

@hello-smile6

@hello-smile6

Not sure

ok

ok

Do you have python experience? I need help with a server,

im decent

im decent

Okay. What's your replit username, again?

adith08

<EMAIL_ADDRESS>Don't wreck my server. All I need or want is CORS support.


Duotone Theme Color contrast is not up to WCAG standards for comments

First of all, thanks for the work, great project 👏
While using the Duotone themes, I noticed that the comment style was under the WCAG contrast ratio for Dark Mode and Light mode
Chrome Dev Tools contrast ratio tool:

Visible Code:

Chrome Dev Tools contrast ratio tool:

Visible Code:

As you can see, they're quite under what's recommended for accessibility - especially with light mode, you can see just how difficult it is to read here.
I'm not sure if there's a different setup for updating themes, but I found these CSS rules / Colors to be WCAG compliant and somewhat on theme.
Also this functions as a workaround for other users with this issue:
.cm-s-duotone-light span.cm-bracket,
.cm-s-duotone-light span.cm-comment {
  color: #6f6e6a !important;
}

.cm-s-duotone-dark span.cm-bracket,
.cm-s-duotone-dark span.cm-comment {
  color: #a7a5b2 !important;
}

Looks like:
Light:

Dark:

Adopted your colors in attached patch.


New package: Overseer v0.1.1

Registering package: Overseer
Repository: https://github.com/louisponet/Overseer.jl
Created by: @louisponet
Version: v0.1.1
Commit: 7f4bc4714ac98bafb4fa7be0629266405f69e762
Git reference: master

Your new package pull request does not meet all of the guidelines for automatic merging.
Specifically, your pull request does not meet the following guidelines:

Version number is not 0.0.1, 0.1.0, or 1.0.0

If you want to prevent this pull request from being auto-merged, simply leave a comment.
(If you want to post a comment without blocking auto-merging, you must include the text [noblock] in your comment.)

It is important to note that if your pull request does not meet the guidelines for automatic merging, this does not mean that your pull request will never be merged. It just means that your pull request will require manual review by a human.

These guidelines are intended not as requirements for packages but as very conservative guidelines, which, if your new package or new version of a package meets them, it may be automatically merged.

On a separate note, I see that you are registering a release with a version number of the form v0.X.Y.
Does your package have a stable public API? If so, then it's time for you to register version v1.0.0 of your package. (This is not a requirement. It's just a recommendation.)
If your package does not yet have a stable public API, then of course you are not yet ready to release version v1.0.0.

[noblock]

Your new package pull request does not meet the following guidelines for auto-merging:

Version number is not 0.0.1, 0.1.0, or 1.0.0

Note that the guidelines are only required for the pull request to be merged automatically. However, it is strongly recommended to follow them, since otherwise the pull request needs to be manually reviewed and merged by a human.

If you want to prevent this pull request from being auto-merged, simply leave a comment. If you want to post a comment without blocking auto-merging, you must include the text [noblock] in your comment.


Aeroportoradea - Missing "Targetare.ro" link

ENVIROMENT: production
URL: https://scrapers.peviitor.ro/src/aeroportoradea/index.html
Browser: Chrome
Device: laptop
OS: Windows 10
STEPS TO REPRODUCE:
1.Open URL in browser
2. Go to company logo
3. Check for Targetare.ro link
EXPECTED RESULTS:
Targetare.ro link is visible and opens
ACTUAL RESULTS:
Targetare.ro link for Aeroportoradea company does not exist
ATTACHMENTS:

duplicate #504


[web]: ClipPath widget in iOS Web scroll page height calculation error

Flutter 2.1.0-12.2.pre • channel beta
Tools • Dart 2.13.0 (build 2.13.0-116.0.dev)
iOS 14.5.1
Safari latest
--web-renderer html
Preview img: normal、 after page scroll error
Preview code link : https://codepen.io/kerwin/pen/mdWPWvy

Verified this on latest master using both renderers on iOS simulator safari browser and see below:
https://user-images.githubusercontent.com/67046386/118254010-3aecfa00-b4c8-11eb-8222-e7d21a537595.mov
When we scroll, the clippath widget doesn't retain it's size.

flutter doctor -v

[✓] Flutter (Channel master, 2.3.0-2.0.pre.203, on Mac OS X 10.15.4 19E2269
    darwin-x64, locale en-GB)
    • Flutter version 2.3.0-2.0.pre.203 at
      /Users/dhs/documents/fluttersdk/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision c533ab6515 (3 hours ago), 2021-05-13 23:04:02 -0700
    • Engine revision 0507f4a9a7
    • Dart version 2.14.0 (build 2.14.0-114.0.dev)

[!] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    ! CocoaPods 1.9.3 out of date (1.10.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin
        code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see
      https://guides.cocoapods.org/using/getting-started.html#installation for
      instructions.

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.55.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (3 available)
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729 • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                               •
      web-javascript • Google Chrome 90.0.4430.212

! Doctor found issues in 1 category.

Verified this on latest master using both renderers on iOS simulator safari browser and see below:
80418.mov
When we scroll, the clippath widget doesn't retain it's size.
flutter doctor -v

Is it a bug to be fixed?

This appears to have become worse, this is a side by side comparison of
Chrome | Safari on Desktop | Safari on iPhone

This appears to be an engine issue relating only to Safari on iOS, not sure, @yjbanov WDYT?

Although, when I run the codepen link from OP directly on my iPhone, the issue reproduces as reported above. Could this be a codepen issue as well in the case where the clip is distorting?

Here it is in a dartpad: https://dartpad.dev/?id=a84521958ef17d734ad385a3316b142c&null_safety=true
Checking a couple of different devices to rule out codepen

Ok, it reproduces on Dartpad on iPhone, not on Android.


user_uploads/... cannot be found - Pasted images

Hello,
We have two zulip servers being load balanced by an HAPROXY device.  We are receiving an error message that pasted images cannot be located.
2019-04-18 16:39:13.598 WARN [django.request] Not Found: /user_uploads/2/bd/4noXNJxUV_mIuK0yivK7r8FT/pasted_image.png
2019-04-18 16:39:13.607 WARN [django.request] Not Found: /user_uploads/2/9a/nbEb38yVVFo4s_PCNXSNCkHB/pasted_image.png
We have modified settings.py file on each server to locate the files on a shared NFS share that each server has access to:
LOCAL_UPLOADS_DIR = "/mnt/zulip/uploads"
Looking at the shared location I can see the file in the folder and the server has the correct access to it:
root@zulip2:/mnt/zulip/uploads/files/2/9a/nbEb38yVVFo4s_PCNXSNCkHB# ls -la
total 140
drwxr-xr-x 2 zulip zulip   4096 Apr 11 13:18 .
drwxr-xr-x 3 zulip zulip   4096 Apr 11 13:18 ..
-rw-r--r-- 1 zulip zulip 132921 Apr 11 13:18 pasted_image.png
Looking at the permissions for the mounted folder I see:
root@zulip2:/mnt/zulip# ls -la
total 12
drwxr-xr-x 3 zulip zulip 4096 Apr 17 13:29 .
drwxr-xr-x 3 root  root  4096 Apr 17 15:39 ..
drwxr-xr-x 5 zulip zulip 4096 Apr 18 08:14 uploads
What I don't see is anything that references the path "/user_uploads"
Does anybody have any clues on this?

@cwhitsbm that /user_uploads path is an agreement between Zulip's nginx configuration and the URLs it uses; zerver/lib/upload.py has the code that does the mapping, but in short, basically accessing those paths is done by joining the LOCAL_UPLOADS_DIR to the file type (e.g. uploads) and the stuff after /user_uploads in the URL.
I'm guessing something is wrong with your NFS setup that is resulting in files not being synced between the two servers properly.


Stamina timer is not triggered by drinking a Stamina mix

Stamina timer is not triggered by drinking a Stamina mix

I'll work on this in the next couple days. At a glance it looks like all mixed potions have been missed by the game timers and item stats plugins.

I noticed this too while killing mithril dragons which drop various mixes that hovering my mouse over them in my inventory does not show much much stats they will restore/boost.

For reference, the chat string for drinking a stamina mix is "You drink the lumpy potion.", which is the same as any other mixed potion. :anguished:


init: chime unstable-2023-03-06

Added chime
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Please rename the PR and commit to python3Packages.chime: init at unstable-2023-03-06 as explained in CONTRIBUTING.

Please rename the PR and commit to python3Packages.chime: init at unstable-2023-03-06 as explained in CONTRIBUTING.

Sorry about that. Should be fixed.

nixpkgs-reviewd workflow started
Status and logs: https://github.com/nixosbrasil/nixpkgs-reviewd/actions/runs/12467316961

nixpkgs-reviewd workflow finished
Status and logs: https://github.com/nixosbrasil/nixpkgs-reviewd/actions/runs/12467316961
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 367491

x86_64-darwin

:white_check_mark: 4 packages built:

python311Packages.chime
python311Packages.chime.dist
python312Packages.chime
python312Packages.chime.dist

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 367491

aarch64-darwin

:white_check_mark: 4 packages built:

python311Packages.chime
python311Packages.chime.dist
python312Packages.chime
python312Packages.chime.dist

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 367491

x86_64-linux

:white_check_mark: 4 packages built:

python311Packages.chime
python311Packages.chime.dist
python312Packages.chime
python312Packages.chime.dist

Please rename the PR and commit to python3Packages.chime: init at unstable-2023-03-06 as explained in CONTRIBUTING.

Sorry about that. Should be fixed.

No need to be sorry. The best thing of code review heavy projects is that you don't need to worry so much because there will be more eyes on the code to find issues. I may not have a deep understanding about the domain of this package but I can unblock the people that has by solving, or guiding to solve, the most common issues.

BTW what a cool piece of software you got here.
Just tested it locally on my x86_64-linux machine and works very well.
I'm wondering if I could bring the sounds of Windows XP there 🤔.

Yeah, it is cool!


[ROS2 Porting] Meeting 2020-10-15

Go over actions from previous meeting:

https://github.com/tier4/AutowcreArchitectureProposal/issues/72

Report about any blockers

Use of timer has been updated https://github.com/tier4/AutowareArchitectureProposal/blob/ros2/docs/porting-to-ROS2.md#replicating-rostimer
Use of tf_listeners with timeout (tf2::BufferCore vs tf2::BufferClient) https://github.com/tier4/Pilot.Auto/issues/23

@mitsudome-r Because these issues come up repeatedly during reviews, I want to propose to agree in the meeting on how to uniformly handle
composition
port existing main vs registering components as in https://index.ros.org/doc/ros2/Tutorials/Composition/
ament_auto
shorter cmake files, less repetition vs keeping things mostly as they are
@nnmm perhaps you come up with more topics?

Yes, it would be great to have a written list of things that we discussed to do now vs later. My current understanding is
Now

Make each node compile and launch without issues (obviously)
Use ament_auto and stricter warning flags (recommended by Chris/Dejan in Slack)
Remove superfluous dependencies from package.xml (hasn't been discussed yet)
Fix code style issues in the lines/parts that are changed anyway (hasn't been discussed yet, but clear win)

Later (but before the AutowareAuto integration)

Update README.md files (hasn't been discussed yet
Convert to components (I think this was decided to be a not-to-do for now)
Create custom messages to replace the deprecated std_msgs::Float32 etc. (hasn't been discussed yet)
Rename files, e.g. _core files to _node, .h to .hpp (hasn't been discussed yet)

Adding Linter

Adding lint to packages is quite easy. https://github.com/tier4/Pilot.Auto/pull/30
However, fixing the lint issues might require many changes on source code that would be outside of scope, which might make review difficult at this stage.
my suggestion is to not to add lint check for now. We can add lints to all packages after porting is done.

multi_object_tracker blockers
Waiting for port of unique_identifier package to ROS2/Foxy: https://github.com/ros2/unique_identifier_msgs/issues/4#issuecomment-708932079

Waiting for port of unique_identifier
https://lowrey.me/guid-generation-in-c-11/

https://lowrey.me/guid-generation-in-c-11/

@fred-apex-ai Ah, this is what I was referring to in the meeting earlier. I can implement this for the time being while the UUID package is being ported. (The UUID ROS message itself is already ported so I think this should serve has an immediate solution to the blocker - I just wasn't sure if there were any design decisions around using the ROS UUID library vs the C++11 standard library)

I just wasn't sure if there were any design decisions around using the ROS UUID library vs the C++11 standard library)
Of course I wasn't there but I would think I'd rather use a well tested external package than Implement it myself and possibly make subtle mistakes. @jilaada


Command still hasn't been written into connection!

We are experiencing what at first glance seems like network related errors using the latest version of redisson pro. However we noticed that this error only occurs on LPOP commands.
2021-07-26 09:31:05,547 [trace=,span=] ERROR                    h:221  - Command still hasn't been written into connection! Try to increase nettyThreads setting. Payload size in bytes: 0. Node source: NodeSource [slot=4753, addr=null, redisClient=null, redirect=null, entry=null], connection: RedisConnection [entry=[commandsBatch=830, entry=[freeSubscribeConnectionsAmount=0, freeSubscribeConnectionsCounter=value:47:queue:0, freeConnectionsAmount=24, freeConnectionsCounter=value:64:queue:0, freezeReason=null, client=[addr=rediss://xfv.gnfyko.euw1.cache.amazonaws.com:6379], nodeType=MASTER, firstFail=0], commandsBatchSemaphore=value:0:queue:2, sendWorkers=0, connectionsAmount=2], redisClient=[addr=rediss://xfv.gnfyko.euw1.cache.amazonaws.com:6379]], command: (LPOP), params: [prv::V1.0::{playlist.collection}:write-behind-queue] after 3 retry attempts
org.redisson.client.RedisTimeoutException: Command still hasn't been written into connection! Try to increase nettyThreads setting. Payload size in bytes: 0. Node source: NodeSource [slot=4753, addr=null, redisClient=null, redirect=null, entry=null], connection: RedisConnection [entry=[commandsBatch=830, entry=[freeSubscribeConnectionsAmount=0, freeSubscribeConnectionsCounter=value:47:queue:0, freeConnectionsAmount=24, freeConnectionsCounter=value:64:queue:0, freezeReason=null, client=[addr=rediss://xfv.gnfyko.euw1.cache.amazonaws.com:6379], nodeType=MASTER, firstFail=0], commandsBatchSemaphore=value:0:queue:2, sendWorkers=0, connectionsAmount=2], redisClient=[addr=rediss://xfv.gnfyko.euw1.cache.amazonaws.com:6379]], command: (LPOP), params: [prv::V1.0::{playlist.collection}:write-behind-queue] after 3 retry attempts

Increasing netty threads has no effect on this, also increasing timeout config did not improve the errors.
Also when deploying our code (on a test environment without load) we see a heavy increase in cpu usage on redis master nodes. The load increases by 50% and drops very slowly in about 1 hour to <5%
We are using the RedissonClusteredSpringLocalCachedCacheManager (but the same occurs when using RedissonClusteredSpringCacheManager)

try to set performanceMode = HIGHER_THROUGHPUT setting

Also when deploying our code (on a test environment without load) we see a heavy increase in cpu usage on redis master nodes.

Can you set trace logging level for org.redisson to examine sent redis commands?

please update to 3.16.0-7 version as well it contains connection leak fix https://github.com/redisson/redisson/commit/34d2e4de3a8b6cc9170fe7b5d82d40498fe60f1e

Hi mrniko,
i've updated the version to 3.16.0-7 but it didn't fix the issue. As you suggested I've tried to enable trace logging on the org.redisson package, however after 30sec of logging it left me with a file of 17GB. I can see the commands that are executed but have no clue why the amount is so large. For what it's worth I've attached a small sample of the logfile.
redisson-log-sample.txt
Setting performance mode to HIGHER_THROUGHPUT did not have any noticable effect.
However I've updated my code to replace RedissonClusteredSpringLocalCachedCacheManager with RedissonSpringLocalCachedCacheManager and the problems are gone when using the latter cache manager. Sadly this is not a fix as we do need the data partitioning of the clustered cache manager.
On attempting to simulate the problem on a test project I've come across something else which I'm not sure if related. It seems that updates do not get propagated to the cache after some time. I've attached a test to reproduce this, the test is expected to update it's local cache every 10 seconds, but already after 20 seconds the old value keeps coming back from cache and never gets updated. When you replace the RedissonClusteredSpringLocalCachedCacheManager with the RedissonSpringLocalCachedCacheManager everything just works. Also when removing the namemapper everything works as expected.
redisson-test.zip

Thanks for shared code and logs. Will try to run it.

Unable to acquire connection! RedissonPromise [promise=ImmediateEventExecutor$ImmediatePromise@64639164(failure: java.util.concurrent.CancellationException)]Increase connection pool size. Node source: NodeSource [slot=8616, addr=null, redisClient=null, redirect=null, entry=null], command: (BLPOP)

I see you use executorService which causes connection starvation, due to blpop command usage. BLPOP command allocates connection and can't share it with other commands. It's worth to note that test application doesn't contain executorService usage.
Did you try to set masterConnectionMinimumIdleSize = 500 and masterConnectionPoolSize = 500 ?

Also MapWriter didn't define but writeBehind settings were defined.

Unable to reproduce the issue using cache with your config:

        final LocalCachedMapOptions<Object, Object> localCacheOptions = LocalCachedMapOptions.defaults()
                .storeMode(LocalCachedMapOptions.StoreMode.LOCALCACHE_REDIS)
                .cacheProvider(LocalCachedMapOptions.CacheProvider.CAFFEINE)
                .evictionPolicy(LocalCachedMapOptions.EvictionPolicy.LRU)
                .writeMode(MapOptions.WriteMode.WRITE_BEHIND)
                .timeToLive(10000, TimeUnit.MILLISECONDS)
                .maxIdle(0)
                .cacheSize(10)
                .writeBehindBatchSize(100)
                .writeBehindDelay(3000);

        final LocalCachedCacheConfig cacheConfig = new org.redisson.spring.cache.LocalCachedCacheConfig(
                2000,
                0,
                localCacheOptions);
        cacheConfig.setMaxSize(10);
        final HashMap config = new HashMap<String, org.redisson.spring.cache.LocalCachedCacheConfig>();
        config.put("test", cacheConfig);

        RedissonClusteredSpringLocalCachedCacheManager m = new RedissonClusteredSpringLocalCachedCacheManager(redisson, config);

        for (int i = 0; i < 10000; i++) {
            m.getCache("test").put(i, i);
        }

Did you resolve the issue?


The guest user cannot review previously analyzed plots

The error is "Someone else has analyzed this plot"

go to plot also seems like its not working. "This plot has already been analyzed."


Consider removing isolated scope

Can't apply on element that already asks for isolated scope.
Functions can be extracted with $parse. See http://luxiyalu.com/angular-directive-click-elsewhere/

Fixed, made the whole thing more in line with how angular handles events.  See https://github.com/angular/angular.js/blob/g3_v1_2/src/ng/directive/ngEventDirs.js#L54

Thanks!


Add tags support

Hello there! Working on a ConvertKit integration and needed tags support as well as non-global API key configuration.
This PR has that and a few other small things:

add .env.local.sample file which can be copied to get a working .env.local which is needed for testing purposes
pull out some of the hard coded values in the tests into those env variables
accept api_key and api_secret in client initialization
default api_key and api_secret to nil so that when making a request the string 'api_key' and 'api_secret' aren't sent to convertkit (this is a problem when you dont need/want to specify api_secret, and that string is sent along anyway)
there was some mixing of tabs and spaces, i think i caught them all and made them spaces
bump to 0.0.2
removed the need to make 2 requests per test (use same response to check success and body)

I hope this looks good!

I thought the tests might fail since a .env.local is required now. I wonder how it's making API calls at all though since API_KEY and API_SECRET env variables were needed all along.

@rfunduk I had a convertkit account, and I used the api_key and api_secret on Semaphore to run the tests. I'l merge this PR if you can confirm all tests are passing on your local machine w/ a screenshot.

@rfunduk 💯  got the tests to pass!

🍠  Awesome!


use terraform, 30 qemu are linked clone and started in 1 minute, and some errors are reported

Sorry, English is not my mother tongue.
I have a proxmox cluster with six physical machines and have done Ceph.
I need to linked clone qemu through terraform.(many qemu, for example 60 qemu) （Sufficient memory、cpu）（SSD）
but，The first few are very successful. The more later they are, the more likely they are to fail.
I want to know how many link clones are suitable for batch startup, and what are the main determinants of startup quantity.
error(s)  ,proxmox version 6.3 , terraform-provider-proxmox_v2.7.4 plugin:
1、request qemu/430745715/status/start   response 200 ok
2、request tasks/UPID.../status			 response 200 ok    data {"type":"qmstart","status":"running"}
3、request tasks/UPID.../status			 response 200 ok    data {"type":"qmstart","status":"running"}
4、request tasks/UPID.../status			 response 599 Too many redirections
5、no exit，continue
6、request qemu/430745715/status/start   response 200 ok	UPID update
7、request tasks/UPID...（new）/status	 response VM 430745715 already running  data {"type":"qmstart","status":"stopped"}
8、no exit，continue
9、request qemu/430745715/status/start   response 200 ok	UPID update
10、request tasks/UPID...（new）/status	 response VM 430745715 already running  data {"type":"qmstart","status":"stopped"}
11、terraform error  VM 430745715 already running
12 exit

599 Too many redirections
2021/08/31 14:22:05 >>>>>>>>>> REQUEST:
GET /api2/json/nodes/bdlabhost02/tasks/UPID:bdlabhost02:001F29A8:0285AAD9:612DCA8B:qmstart:240426906:root@pam:/status HTTP/1.1^M
Host: <IP_ADDRESS>:8006^M
User-Agent: Go-http-client/1.1^M
Accept: application/json^M
Cookie: PVEAuthCookie=PVE:root@pam:612DCA82::f7AoOw8tCazlM+Z+lH2JOlQ+SINUud1wQdqU/UxuRXTK0FbHCQxMhAAoJlUymOpVQXySzJuQc4Ue0CiZvFxXlt4Icum3hE1OHnTPwuoqaty+9IcMp7/9FTXvcHBWg/pEgtp+go9uCOtjfx9eMCig8eOf1oYT1+YxnMrYi0lRGO0FP7ZExRey0Tklc7NOzm+6e6V3vcw4sA9CNEZCIPJBfwaQbQ9fQxvlOHKEP87MarP+9N7Yq4qOvXuBHy9DqsLc46Swn0zR4xxv+5FmdtD5dBJSzTXmqjf1Syv4wWZ12GHKslcUL0Qfa40dDaQglQk6l/5VMw9YD71cQY1efzzv7g==^M
Csrfpreventiontoken: 612DCA82:YLzREnnGZlgxJ56ky5KtZia7NpWKBTpkapNWhqSBHRg^M
Accept-Encoding: gzip^M
^M
2021/08/31 14:22:08 <<<<<<<<<< RESULT:
HTTP/1.1 599 Too many redirections^M
Connection: close^M
Cache-Control: max-age=0^M
Date: Tue, 31 Aug 2021 06:22:08 GMT^M
Expires: Tue, 31 Aug 2021 06:22:08 GMT^M
Pragma: no-cache^M
Server: pve-api-daemon/3.0^M

VM 582360253 already running
2021/08/31 14:25:31 >>>>>>>>>> REQUEST:
GET /api2/json/nodes/bdlabhost05/tasks/UPID:bdlabhost05:001EFC3C:027FB215:612DCB5B:qmstart:582360253:root@pam:/status HTTP/1.1^M
Host: <IP_ADDRESS>:8006^M
User-Agent: Go-http-client/1.1^M
Accept: application/json^M
Cookie: PVEAuthCookie=PVE:root@pam:612DCB4C::2ta+IPC6+DR+hs7l+inkk9BAzzdo1YE7+8KouDZLo4Z+EJMvQGriVDOxPfsq2PPpBwkQ2ZiQ4/eF8XiH1o0IIA67RhT0V9XQ8H1X1nUqVgdmDumptOcd0ij/nNH9/epEXubblmPHbMmAIre+8UKtjIEo7HTAyfoybKYuGR72cHdsmZQW1Ub+opLQCA6lEwZYGaHRSs03Du/F/ffFARcIbspBw6rczsWnxyW1PHnoI91jFPqYBPld6k1ufC+gA6bcBfkQCuLYTjaf+oqIglPVDLWX3kxV8gAkEMba1kGDqCHTE9Bf4pd5TJRsPi9K49n6Nu3WXi7s/+3atozp7MSxhA==^M
Csrfpreventiontoken: 612DCB4C:StItrBMRQCJrmw6lXN/Rf+7XbZlSV47+oRGHVjULe2k^M
Accept-Encoding: gzip^M
^M
2021/08/31 14:25:31 <<<<<<<<<< RESULT:
HTTP/1.1 200 OK^M
Content-Length: 282^M
Cache-Control: max-age=0^M
Connection: Keep-Alive,Keep-Alive^M
Connection: Keep-Alive^M
Content-Type: application/json;charset=UTF-8^M
Date: Tue, 31 Aug 2021 06:25:31 GMT^M
Expires: Tue, 31 Aug 2021 06:25:31 GMT^M
Pragma: no-cache^M
Server: pve-api-daemon/3.0^M
^M
{"data":{"user":"root@pam","status":"stopped","pstart":41923093,"exitstatus":"VM 582360253 already running","starttime":1630391131,"pid":2030652,"type":"qmstart","node":"bdlabhost05","id":"582360253","upid":"UPID:bdlabhost05:001EFC3C:027FB215:612DCB5B:qmstart:582360253:root@pam:"}}

I also have this issue. It's very frustrating that this terraform provider doesn't work properly.

@dapeng235 are you able to fix this provider issue with any workaround?


Not rendering the hello world example in 'webpack-plus-reactjs-and-django

I have a react app working in django and am looking to use webpack to re-factor and streamline its ongoing development and maintenance.
Using the blog entry, all went well until I got to the actual presentation in Django.
I've copied the code pretty much verbatim save for putting it in a 'front_end' folder (paths etc. updated appropriately). The app.jsx file exports the class object fine but the render method on the object is never called (inserted console.log does not appear). In my other app example I use ReactDOM.render rather than React.render but apart from that it looks pretty similar.
The rest of the template comes up fine and it has included the bundle js correctly.
As the React debugger does not come up in Firefox and everything is mushed into the bundle (without any issues), I'm not sure where to go from here in terms of debugging.

OK - old school debugging sorted it slowly...
The React object does not have a render method any more so index.jsx must use the react-dom. To address this; npm install react-dom and change the main.js import/call to use react-dom appropriately.
Then we find that the jsx is not being handled by babel so still nothing gets rendered. This took me to  where I followed and used the instructions there to bring things up to date. This required installation of babel-preset-react.
Thus package.json became:
{
  "name": "your name",
  "repository": {
    "type": "git",
    "url": "your repo"
  },
  "version": "1.0.0",
  "description": "react front end to django for xxx",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "author": "whippy",
  "license": "ISC",
  "devDependencies": {
    "babel": "^6.5.2",
    "babel-cli": "^6.10.1",
    "babel-core": "^6.10.4",
    "babel-loader": "^6.2.4",
    "babel-preset-es2015": "^6.9.0",
    "babel-preset-react": "^6.5.0",
    "react": "^15.1.0",
    "react-dom": "^15.1.0",
    "webpack": "^1.13.1",
    "webpack-bundle-tracker": "0.0.93"
  }
}

webpack.config.js became:
var path = require("path");
var webpack = require('webpack');
var BundleTracker = require('webpack-bundle-tracker');

module.exports = {
  context: __dirname,

  entry: './assets/js/index', // entry point of our app. assets/js/index.js should require other js modules and dependencies it needs

  output: {
      path: path.resolve('./assets/bundles/'),
      filename: "[name]-[hash].js"
  },

  plugins: [
    new BundleTracker({filename: './webpack-stats.json'})
  ],

  module: {
    loaders: [
      {
        test: /\.jsx?$/,
        exclude: /node_modules/,
        loader: 'babel-loader',
        query: {presets:['es2015', 'react']}}
    ]
  },

  resolve: {
    modulesDirectories: ['node_modules', 'bower_components'],
    extensions: ['', '.js', '.jsx']
  }
};

index.jsx became:
//noinspection JSUnusedLocalSymbols because React is needed implicitly by babel's expansion.
var React = require('react');
var ReactDOM = require('react-dom');
var App = require('./app');

ReactDOM.render(<App/>, document.getElementById('react-app'));`

Thanks @pwhipp. This should help others. I'll try to update the blog post soon to show latest version of babel


feature/13-add-criteria-for-static-caches

Cache footprint, max size, usage penalty

OBE. Implemented as part of #39


Adding a new metric ingest/notices/queueSize to give more visibility into supervisor notices queue size.

While I was trying to gather more info for #11414, I realized we do not have much visibility into the notices queue size inside Kafka stream supervisor. Hence adding a new metric to increase that visibility.
Testing done
I tested it locally with the logging emitter

@jihoonson The same tests that failed in #11415 are failing here as well. Can u retrigger them when u get a chance and also review the code when u have time?

would it make sense to emit a metric to compute the time it takes to process each notice, this would also give us a sense of which notices take more time to process if we add a tag by notice type

would it make sense to emit a metric to also compute the time it takes to process each notice, this would also give us a sense of which notices take more time to process if we add a tag by notice type

Added one


Let %defaultDatasource use a widget to accept user's account information

This can avoid the issue of plain password saved in a notebook.

yea we should totally do something like that.

what are you all doing as w work around right now? I was just considering reading in password from an environment variable as a workaround... although it isn't obvious to me how to build the connection string that way. Is there an existing good practice?

One workaround is to define your own SQL kernel (JSON file) in which you can story your password.


Center photo grid on mobile

First Image click is zoomed in, rest is fine afterwards @victorwu95

Moved to new issue


Update MFST_xFireFox to pull latest version by default and use HTTPS

Hi @gregjhogan, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;


consistent naming with slides on graduation

people will change their project name over time - this is fine! But if they register as MyAwesomeProjectDotCom at the start, and graduate as AwesomeProj, it's hard to guess who it is sometimes and make sure our internal knowledge management is consistent.
What can we do to make this easier for us?
Tagging @Tajuddeen1 @iramosp

I can share something that didn't work for Nebula: adding a field for project name in the Civi registration form. It wasn't a required field because experts and facilitators used the same form, so some people left it blank. Team members entered different project names even when we insisted that they coordinate to enter the same one.
Is the main challenge here keeping updated project metadata (project title, team, etc) or is it tracking who graduated?


Fix the CI error fo "perform CodeQL Analysis"

modify go version from 1.22 to 1.22.0 so that the following error could be avoided.
go: download go1.22 for linux/amd64: toolchain not available, Error: Encountered a fatal error while running "/opt/hostedtoolcache/CodeQL/2.16.3/x64/codeql/go/tools/autobuild.sh"
Changes have been tested with https://github.com/yanjing1104/memtierd/pull/7, codeql-scan passed successfully.

Thanks, Jing!


New features and bug fixes

Features:

Client side rendering.

an unit can be rendered using AJAX call to /{context}/unit/{unit-full-name}?param1=?value1&param2=value2 URL.

Anonymous units (similar to anonymous pages)

"isAnonymous": true in unit's definition allows that unit to be rendered even there is no user session.

Dynamic error pages.

you can set error pages (for each HTTP status or default one for all) in 'app-conf.json'.

Customizable SSO intermediate page ("now you are being redirect to the IDP server" page).

Bug fixes:

In page inheritance.
In SSO module.
Better modularized architecture.
Some performance enhancements for LESS compiling and rendering.

smoke tested with IoT server pack on 2015/01/06. no issues found.

accidentally closed, hence re-opened.

Missing license headers are added in PR #21.


GEOMESA-1659 Docs: Update discussion of Kryo registration for Spark

Signed-off-by: Matthew Zimmerman<EMAIL_ADDRESS>

@tkunicki can you review this PR?


add prior that takes arguments from a dictionary

If we train a new prior, rather than hardcoding and renaming each prior file, write up a new form that reads parameters from dictionary and sets things up that way

done


🐛 [Bug] Torch-TensorRT static initialisation order

This isn't directly a bug in itself, but rather a suboptimal design of the core_runtime library that can lead to bugs (such as the one discussed here).
Bug Description
In register_trt_op.cpp, the core_runtime library registers the execute_engine function in torch, using the TORCH_LIBRARY macro.
// declaration of execute_engine added here for readability
std::vector<at::Tensor> execute_engine(std::vector<at::Tensor> inputs, c10::intrusive_ptr<TRTEngine> compiled_engine);

TORCH_LIBRARY(tensorrt, m) {
  m.def("execute_engine", execute_engine);
}

This macro results in a bunch of static object initialisation. But more importantly, it relies on the TRTEngine type to be known by torch already! This registration is happening in TRTEngine.cpp (note that this is a different translation unit!) with the instantiation of a static torch::class_:
static auto TORCHTRT_UNUSED TRTEngineTSRegistrtion =
    torch::class_<TRTEngine>("tensorrt", "Engine")

Static initialisations order across translation units is known to be undefined. Relying on that order is referred to as static initialisation order fiasco and is a bad idea in general.
To Reproduce
Steps to reproduce the behavior:

Switch the order of the files TRTEngine.cpp and register_trt_op.cpp in core/runtime/BUILD, recompile, and try to run anything.

cc_library(
    name = "runtime",
    srcs = [
        "CudaDevice.cpp",
        "DeviceList.cpp",
        "register_trt_op.cpp",
        "TRTEngine.cpp",  # <-- put this one after "register_trt_op.cpp"
        "runtime.cpp"
    ],

Proposed changes
There are a few solutions to improve the design of core_runtime, the most straightforward one being to group the static initialization in the same translation unit, so that the order is actually defined.

Thanks, restructured the runtime lib to hopefully avoid this