Add a guided attention loss

Add a guided attention loss implementation for sequence-to-sequence models where there is a direct correspondence between inputs and outputs, such as Grpaheme-to-Phoneme and Text-to-Speech.
The implementation is based on this paper
https://arxiv.org/abs/1710.08969
and inspired by the R9Y9 implementation of DeepVoice3:
https://github.com/r9y9/deepvoice3_pytorch
However, unlike the R9Y9 implementation, which contains nested loops and must be optimized with Numba, this proposed implementation relies on PyTorch vectorization instead.

@mravanelli LGTM, maybe we could merge.

The only thing that should be done before merging it is to add an example in the docstring.

@mravanelli : Will add shortly

@mravanelli : The example is done

Thank you for this contribution @flexthink! We should try guided attention for speech recognition as well to figure out if that might help convergence or not.


custom domain resets after build

If this was just a one-time occurrence, close this.
Reported at https://webapps.stackexchange.com/questions/164534/github-pages-resets-custom-domain-after-github-action-build.

Problem occurred again today after the last commit.


Consider encrypting remaining minor Wallet metadata properties

Some minor metadata on Wallets like isInViewOnlyMode, dateThatLast_fetchedAccountInfo, and dateWalletFirstSavedLocally are not currently stored encrypted (see wallet_persistence_utils.js). I had figured these weren't important but don't know that for sure.

This is now effectively closed by encrypting the whole document.


Add Support for 3rd Party DLL's

Tracking for http://tridion.stackexchange.com/q/12724/9, currently only the plugin project's DLL is actually packaged with the A4T file that is built, and any 3rd party DLL's must manually either be put into the Alchemy's bin, the GAC, the plugin's bin, or merged with ILMerge.
Need a way to manage 3rd party DLL's across plugins that may share libraries but package different versions of them.

This has been pulled into the 0.8.1 release.

http://www.alchemywebstore.com/help/how-to/Packaging-DLL-References


Update google-cloud-trace to 2.24.0

About this PR
📦 Updates com.google.cloud:google-cloud-trace from 2.3.7 to 2.24.0
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.google.cloud", artifactId = "google-cloud-trace" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.google.cloud", artifactId = "google-cloud-trace" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #321.


Horizontal scrollbar?

Why is there a navbar on the bottom of all of our discrete panels? I keep our Grafana very updated, so perhaps a recent update broke this (or added it)? It looks terrible.

I am unable to reproduce this... can you post the 'Panel JSON'?  What version of grafana and the plugin?
I'm guessing this is an issue with 5.1 and the changes to how scrollbars work.

HorizontalBar.zip
Grafana v5.1.0 (commit: 844bdc53a)

I think this will be fixed in the 0.0.8 (master) release.  Are you able to easily test that?

What's the best way to install it other than the quick one-liner from Grafana (usually how I install). Yes, I'm a n00b.

I have the same problem in Grafana 5.1.3 and plugin version 0.0.7.
Could you please push the 0.0.8 version out?

Could you please push the 0.0.8 version out?

I wish I had control :(
https://github.com/grafana/grafana-plugin-repository/pull/273

Looks like it's now fixed, thanks for the prompt resolution!


Unable to set the 'response_type' for a message

My messages always get sent as 'ephemeral'. I'm using the response url from the slack request to post back to and need to set the 'response_type' on the slack message as 'in_channel'.
Are you able to add this is so we can explicitly set this.

Pushed version 0.1.8 with this change, many thanks 👍


artisanry, fungose (logical-conflict)

This pull request was generated by the 'mq' tool

/trunk merge


Ef core freezes when exception expected

Ef core freezes when exception expected
Steps to reproduce
Insert two items into a table. Query them both with SingleOrDefault().
Exception expected but nothing happens. Code freezes.

    public class Trailer
    {
        public int Id { get; set; }
        public string SourceId { get; set; }
    }
public class WorkerDbContext : DbContext
    {
        public WorkerDbContext(DbContextOptions options)
            : base(options)
        {
        }
        public DbSet<Trailer> Trailer{ get; set; }
}
async Task Test(WorkerDbContext db){
db.Trailer.Add(new Trailer(){SourceId = "xxx"});
db.Trailer.Add(new Trailer(){SourceId = "xxx"});
await db.SaveChangesAsync();
var ok = await db.Trailer.Where(x => x.SourceId == "xxx").FirstOrDefaultAsync();
var freeze = await db.Trailer.Where(x => x.SourceId == "xxx").SingleOrDefaultAsync();
}

Further technical details
EF Core version: 3.1.0-preview3.19554.8
Database provider: Npgsql.EntityFrameworkCore.PostgreSQL" Version="3.1.0-preview3
Target framework: netcoreapp3.1
Operating system:Windows 10
IDE: Visual Studio 2019 16.4 Preview 5.0

Sorry. It was a problem with not caught exception in generic host service.


build: set defaults for SourceKit on Windows

SourceKit on Windows is ported and the instructions explicitly enable it
since the defaults were not set to enable it by default.  Adjust the
CMake logic to enable it by default for Windows builds as well.  This
reduces the complexity of the default build invocation for Windows.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

CC: @gwynne @akyrtzi

@swift-ci please smoke test

🎉

@gwynne - I think that at this point it may be reasonable to default SourceKit to enabled and if someone wishes to port to a platform where SourceKit does not yet run, then it can be disabled there specifically until it is enabled.  However, I would really rather that be a separate change - I don't feel like I have the authority to make that decision, and think that the SourceKit developers should make that call.


Fix negative keys in properties dictionary when reading GDSII files

Problem
When reading GDSII files using the read_gds function in gdspy, there is an issue with how the properties are handled. The attr value, which is used as the key in the properties dictionary, is directly obtained from the GDSII file without any conversion or checking. If the attr value is greater than 32767, it is interpreted as a negative number due to the signed integer representation.
This leads to a problem where keys greater than 32767 are stored as negative values in the properties dictionary when reading the GDSII file.
It's important to note that when saving properties using the write_gds function in gdspy, the keys are directly written to the GDSII file as unsigned 16-bit integers (2 bytes). The struct.pack function is used with the format string ">5H", where "H" represents an unsigned short integer. This ensures that the keys are correctly stored as unsigned values in the range of 0 to 65535.
However, when reading the GDSII file using the read_gds function, the attr values are interpreted as signed integers, leading to the issue of negative keys for values greater than 32767.
Solution
To fix this issue, the read_gds function has been modified to handle the attr value appropriately. The attr value is now converted to an unsigned 16-bit integer before being used as a key in the properties dictionary.
The relevant code changes are as follows:
# PROPATTR
elif record[0] == 0x2B:
-    attr = record[1][0]
+    attr = record[1][0] & 0xFFFF  # Convert to unsigned 16-bit integer
# PROPVALUE
elif record[0] == 0x2C:
    properties[attr] = record[1]

By performing a bitwise AND operation with 0xFFFF (65535), the attr value is converted to an unsigned 16-bit integer, ensuring that it falls within the range of 0 to 65535.
This change aligns the behavior of the read_gds function with how the properties are saved using the write_gds function, maintaining consistency and avoiding the interpretation of keys as negative values.
Impact
This fix resolves the issue of negative keys in the properties dictionary when reading GDSII files. It ensures that the keys in the properties dictionary are correctly stored as unsigned integers, matching the expected behavior and aligning with how the properties are saved.
Users of the gdspy library who rely on the read_gds function to read GDSII files with properties will benefit from this fix, as it will prevent unexpected negative keys in the resulting properties dictionary.
Testing
The modified read_gds function has been tested with GDSII files containing properties with attr values greater than 32767. The fix has been verified to correctly store the keys as unsigned integers in the properties dictionary.
An existing unit tests has been moddified to ensure the correct behavior of the read_gds function when handling properties with large attr values. Previously having a key ≥2**16 would result in a negative number being loaded.

This is outside the valid range of layers in the GDSII specs: https://boolean.klaasholwerda.nl/interface/bnf/gdsformat.html#rec_layer
We could implement it as an extension, but gdspy is not actively developed any longer. Only bug fixes are accepted.


🐛  YahooDownloader, Can only use .dt accessor with datetimelike values

I run Stock_NeurIPS2018_SB3_MV.ipynb in a docker environment.
And I meet the following error:
  File "demo_StockTrading.py", line 53, in <module>
    df_dji = YahooDownloader(start_date = TRADE_START_DATE,
  File "/home/develop/.conda/envs/finrl/lib/python3.8/site-packages/finrl-0.3.5-py3.8.egg/finrl/meta/preprocessor/yahoodownloader.py", line 76, in fetch_data
  File "/home/develop/.local/lib/python3.8/site-packages/pandas/core/generic.py", line 5487, in __getattr__
    return object.__getattribute__(self, name)
  File "/home/develop/.local/lib/python3.8/site-packages/pandas/core/accessor.py", line 181, in __get__
    accessor_obj = self._accessor(obj)
  File "/home/develop/.local/lib/python3.8/site-packages/pandas/core/indexes/accessors.py", line 506, in __new__
    raise AttributeError("Can only use .dt accessor with datetimelike values")
AttributeError: Can only use .dt accessor with datetimelike values

Maybe something wrong with pandas, So I try pandas 1.1.5, 1.3.0, 1.3.5, but the error is still occur.
FinRL needs pandas>=1.1.5.
yfinance needs pandas>=1.3.0
Desktop

OS: ubuntu 22
python3.8.15
GCC 10.4.0

import sys

sys.path.append("../FinRL")

import itertools

'''Part 3. Download Data'''
TRAIN_START_DATE = '2009-01-01'
# TRAIN_START_DATE = '2015-01-01'
TRAIN_END_DATE = '2020-07-01'
TRADE_START_DATE = '2020-07-01'
TRADE_END_DATE = '2021-10-29'
'''2.4. Create Folders'''
from finrl import config
from finrl import config_tickers
import os
from finrl.main import check_and_make_directories
from finrl.config import (
    DATA_SAVE_DIR,
    TRAINED_MODEL_DIR,
    TENSORBOARD_LOG_DIR,
    RESULTS_DIR,
    INDICATORS,
    TRAIN_START_DATE,
    TRAIN_END_DATE,
    TEST_START_DATE,
    TEST_END_DATE,
    TRADE_START_DATE,
    TRADE_END_DATE,
)

check_and_make_directories([DATA_SAVE_DIR, TRAINED_MODEL_DIR, TENSORBOARD_LOG_DIR, RESULTS_DIR])

df_dji = YahooDownloader(start_date=TRADE_START_DATE,
                         end_date=TRADE_END_DATE,
                         ticker_list=['dji']).fetch_data()
print(df_dji)

This error won't occur when I use google colab to run this code.

Open Google colab, then we use the pip command to list the version of
ccxt
pandas
exchange-calendars
alpaca_trade_api

And reinstall these package to solve this.


Health endpoint

I added a few tests and a very simple, configurable health check endpoint. Can you take a look at this and tell me if I'm on the right track? Or even test it out and see if it does what you want?

Nice, thank you!  And thank you for adding to and improving the tests, too.
This needs just one more thing for our use case: a run-time flag to override the default /health path, like how we have no-preview, simply because that's going to be easier to integrate into the ADO workflow than anything in a config file.
To satisfy everyone posting in https://github.com/CrunchyData/pg_tileserv/issues/131 there should also be a way to specify additional paths to the health endpoint.  But I'm inclined to leave that as an exercise for someone who needs it, and/or me later on volunteer time outside the scope of the project which is funding this bit.  Just adding the flag will satisfy our use case and the majority of other peoples'.

@eldang let me know how that flag suits you!

Perfect, thank you.
I'm going to merge the PR within this repo now, and then would you like to make the PR to the upstream repository?  It seems appropriate for you to get the credit having done the work.

Not deleting the branch because it may be tidier to make the PR to upstream from this than from local-main.

ah, cool, I'll make the PR!


Fetch the list of NFT from a Metamask wallet

I was able to use this library to pull a token using the contract address + the tokenId.
What I want to know now is to only fetch the TokenIds that belong for a given user (by connecting his Metamask wallet) and fetching that information from it.
I tried to look for example but could not find any.
Is this something possible using this library?
Thank you!

Hi, unfortunately this is not an easy thing to do by directly querying an Ethereum node.
But I think it’s something we could add as an optional method in the API, only supported by some fetchers (e.g. OpenSea and a future The Graph API), so I am going to leave this issue opened.

Hi Pierre. I was able to take the Opensea approach and use this instead of your library. I'll keep watching this issue as I might revert back to it if you offer such functionality. You basically need to query the asset end point of OS by passing a Eth address to return all the NFT that belong that that collection.

@cdric - why don't you build it and contribute a PR for it?

The solution I'm using is now very different that the architecture of your component so I would have to rewrite pretty much everything. That being said, if you want, I'm happy to share my reactJs code for you to look at the interaction I'm having between metamask and opensea if this can be helpful?


Fix build issues related to missing grgit-core artifact

Force grgit-core to 4.1.1 to resolve build issues. This is a temporary workaround, the ideal solution is to move the project to the latest gradle and netflixoss plugin versions, but that is a larget undertaking.

LGTM


Thank You message added after submit detail

Description
Added script and for show message prevent to reload page and reset form.
Added thank you message div.
Fixes:  #366

Type of change

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

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

@sk66641 Please check it and merge !
Thank You !


CI: enable e2e test

What problem does this feature solve?
.
What does the proposed API of configuration look like?
.

Plan: enable it on ubuntu, and make ubuntu test run without docker.

https://github.com/web-infra-dev/rspack/pull/3122


Mailchimp

Just curious, are you sure you wanted to merge this here?

😂 I think so, @vzsg

I think it was meant for decisiveweb/miswiftdotcom instead.


In the latest AWS provider, you cannot set the region on an S3 bucket, so this parameter is invalid

Using latest version of the AWS terraform provider (3.18), the cloudtrail s3 bucket configuration is no longer valid.
Specifically, the region is now an output parameter only, not a variable. I have raised this PR so I can continue using this module against the latest version of the AWS terraform provider.
Terraform Plan Output:
$ terraform plan
Error: Computed attribute cannot be set

  on .terraform/modules/account_setup/cloudtrail.tf line 55, in resource "aws_s3_bucket" "cloudtrail_bucket":
  55:   region        = var.cloudtrail_bucket_region

Example Module:
  source = "philips-software/account-setup/aws"
  version = "~> 2.0.0"

  enable_mfa                     = false
  enable_account_password_policy = false
  minimum_password_length        = 8
  max_password_age               = 36500 // 100 years

  enable_read_only_group = true
}```

Thanks, will check asap


"npx only-allow pnpm" is too rogue

it is too rogue to limit package manager into pnpm

This only limits lib contributors, forcing the use of pnpm to ensure a unified package manager.
Doesn't affect people who use this lib.


[raylib] Add new port (fix #4500)

Test project: https://github.com/myd7349/Ongoing-Study/tree/master/cpp/CMake/vcpkg/raylib_examples
Test on:

x86-windows
x86-windows-static
x64-windows
x64-windows-static

Currently, this lib fails on x64-linux. And it doesn't support ARM or UWP. Didn't test it with macOS.
I will try to figure out what happens to x64-linux after the holiday.

Hi @myd7349, thanks for your contribution to vcpkg. Here are the test results from current CI system, please let me know if you need the related logs or other info.

x64-windows
master
test
notes

raylib

Pass
New Port

x64-windows-static
master
test
notes

raylib

Pass
New Port

x64-osx
master
test
notes

raylib

Fail
New Port

arm64-windows
master
test
notes

raylib

Fail
New Port

x86-windows
master
test
notes

raylib

Pass
New Port

x64-linux
master
test
notes

raylib

Fail
New Port

x64-uwp
master
test
notes

raylib

Fail
New Port

arm-uwp
master
test
notes

raylib

Fail
New Port

Hey @myd7349, I tried out x64-linux myself and found that the system packages libxinerama-dev, libxrandr-dev, libx11-dev, libxcursor-dev, and libgl1-mesa-dev are required. I'll update this PR with a systems dependencies message and ensure that these are all found on the build machines so we can get the CI to pass. Thanks for all the work!

Hey @myd7349, I tried out x64-linux myself and found that the system packages libxinerama-dev, libxrandr-dev, libx11-dev, libxcursor-dev, and libgl1-mesa-dev are required. I'll update this PR with a systems dependencies message and ensure that these are all found on the build machines so we can get the CI to pass. Thanks for all the work!

Awesome! Thanks for that. I can remove the WIP tag now.

Oops, may have messed up the commit history a bit :laughing:. I'll fix it momentarily

Hey @myd7349, I modified the condition for the system dependencies message as it is possible for non-UNIX to target UNIX, and the UNIX variable is populated based on the host machine, not the target. Thanks again for the port!!


Add EKS supported versions

Description of your changes

Fixes #
I have:

[ ] Read and followed Upbound's contribution process.
[ ] Run make reviewable to ensure this PR is ready for review.
[ ] Added backport release-x.y labels to auto-backport this PR, as appropriate.

How has this code been tested

/test-examples

@fillerimiz thanks a lot for the contribution! I rebased with main after we merged https://github.com/upbound/configuration-aws-eks/pull/99 to unblock uptest flow


lbry:// hyperlink fails to open sometimes

I've run into this a few time - click a lbry:// hyperlink while the app is closed  - the app opens, but it fails to load the url requested. If I click it again, it works. If I exit out, it's hit or miss on the next try.
System Configuration

Device:
Android version:

Anything Else

Screenshots

unable to reproduce, reopen with more detail

This is still happening randomly - usually on cold open or when the app goes through startup phase (daemon already running)

This is now definitely fixed!


[1.8.9][suggestion] sink to auto make fresh water

i am currently using the 1.8.9 version of CFB and its helps save alot of time. my suggestion is why not let the sink make fresh water when a bucket is available in the fridge/tool rack? will save a ton of crafting time. Awesome mod btw =D

I don't see how this would save a ton of crafting time, you can literally craft a stack of fresh water using two clicks if you have a sink. There might also be people who don't want this happening and your suggestion does not offer a way of control over it (and any possible way of control is likely out of this mod's design scope).

how can i click a stack with 2 clicks? i have to go back and forth with a bucket and the fridge to craft fresh water.... or am i not doing something right?

Cooking Table connected to sink.

then i guess i have a bug report for the 1.8.9 version. i have the sink next to the cooking table and i have no option to make fresh water.


multiple v4l2rtspserver instances on the same machine

First of all thanks for an amazing project, it makes serving rtsp streams a breeze!
My goal is to reliably stream 2 cameras connected to a raspberry pi CM, and I'm 95% there.
I am able to stream both cameras using the same rtspserver by providing the two devices in the command line. For extra reliability I'm launching the server (together with the uv4l initialization) as a service that relaunches after a crash or an improper initialization.
however there are cases where only one stream works and the other one fails (probably because of the uv4l driver), this doesn't trigger a service relaunch because the rtspserver process did not exit. Restarting the service manually fixes the problem.
I figured I should just use two separate services, each will init a different server - one for each camera. Then when a single camera crashes, it will relaunch automatically.
I tried launching two independent servers (on different ports). the first one launches without a problem (no matter which of the servers is first), however the launch of the second one causes the first to stop working (the process does not exit) with the following error in the console:
VIDIOC_DQBUF: Transport endpoint is not connected
BTW, the second stream works great.
Any suggestions?
Does this setup even supported?
If not, and I need to serve both streams from a single server, is there a way of making the server crash when only one stream is faulty?
Thanks!

Hi jonspieg,
As I understand you have 2 video devices managed by uv4l, and there is a misbehaviour when 2 processes access to different video device ?
Do you have the same behaviour using v4l2-ctl something like :
v4l2-ctl --device /dev/video0 --stream-mmap --stream-to=stream1.raw &
v4l2-ctl --device /dev/video1 --stream-mmap --stream-to=stream2.raw &

Maybe you could try to run v4l2rtspserver with the -r option is order to use the read interface (instead of mmap interface)
Best Regards,
Michel.


Added filter property on PineconeStore to enable metadata filtering

Related to #333
I simply added the filter property to the PineconeStore to enable metadata filtering. I set filter?: PineconeMetadata;, using the existing PineconeMetadata type.
My first open-source contribution, so please let me know if I am missing anything.
Thank you for this amazing project!

Hi team, I was totally stuck on my project due to the lack of metadata filtering on Pinecone.
Now that I see this commit has been merged (thank you Trevor), is it true that this feature is accessible on LangchainJS?
I guess the below doc is not updated 🤔 as it says

/* Use as part of a chain (currently no metadata filters) */

Thank you for this amazing project. I'm too hyped about what I'm working on and it's all thanks to Langchain

It seems to be working for me! I pass filter into fromExistingIndex, and then pass the vector store into VectorDBQAChain like normal.

This amazing, will give it a try during the day! Thank you for the actual code you provided @ElektrikSpark

Thanks @ElektrikSpark, this is amazing!
@hwchase17 the docs should be updated to reflect this. This is a massive feature for my use case, so glad I stumbled upon this.

Thanks for your contribution! This should 100% be part of the doc :)

Hello, sorry for the inconvenience, I am using pinecone and langchain in nestJS, I have a problem, the problem is that the metadata filtering is not working, maybe you could give me a hand, I don't know where to look anymore, I can't find any information, it turns out that the filters do not return any matches.
This way I'm storing embeddings with metadata:
`      const docs = await textSplitter.splitDocuments(rawDocs);
// this.#logger.debug('split docs', docs);
if (metadata) {
for (const doc of docs) {
doc.metadata = { ...doc.metadata, metadata };
}
}
const embeddings = new OpenAIEmbeddings();
const index = this.pineConeClient.Index('uk-embeddings');
  await PineconeStore.fromDocuments(docs, embeddings, {
    pineconeIndex: index,
    textKey: 'text',
  });`

In this way I'm performing the query to obtain the documents:
const index = this.pineConeClient.Index('uk-embeddings');       const options: PineconeLibArgs = {         namespace: 'default',         pineconeIndex: index,         textKey: 'text',         filter: {           country: { $eq: 'MEX' }         }       };       const vectorStore = await PineconeStore.fromExistingIndex(         new OpenAIEmbeddings({ modelName: 'text-embedding-ada-002' }),         options,       );       const completionChain = {         model: completion.model,         temperature: completion.temperature,         prompt: completion.prompt,         prompt_qa: completion.prompt_qa,       };       const chain = await this.makeChain(completionChain, vectorStore);
 This is my makeChain function:
 `    const model = new ChatOpenAI({
  temperature: completionChain.temperature,
  modelName: completionChain.model,
});
const chain = ConversationalRetrievalQAChain.fromLLM(
  model,
  vectorStore.asRetriever(),
  {
    qaTemplate: completionChain.prompt,
    questionGeneratorTemplate: CONDENSE_TEMPLATE,
    returnSourceDocuments: true,
  },
);
return chain;`
This is how my embeddings are stored in Pinecone:

And this is the response:

The text response is generated by openai, it is invented by the model, it is not a valid response.
Thanks for your help!


Problem: Cannot specify ingest pipeline.

Solution: Updated to ES5+. Optionally specify pipeline. Major version.

Sorry for delaying.
We're trying to change code for support ES5.x by using Java REST API.

IIUC, this has already been done.

We're trying to change code for support ES5.x by using Java REST API.

Let me close this pull request.


Update vscode-scala-syntax

Use commit hash from vscode-scala-syntax 0.3.5

https://github.com/scala/vscode-scala-syntax/releases/tag/0.3.5

Not necessary. The pull request template you neglected to read explains why.


Fakelag and basic desync aa

It says this in the commit description, but read fakelag.h it is not the most proper way to do it but it works.

I just realized a major mistake I made, in the fakelag it checks for antiaim being enabled and fakelag off. It never returns if antiaim is enabled with 1 tick choke enabled. But that's a small fix, you can do it immediately after merging the PR if you do merge it.

Did you test fakelag?

Did you test fakelag?

Yes, it's working, just read my comment and what I said about using m_choked in the fakelag.h

Just make a separate pull request with only the fakelag and i’ll merge it

Just make a separate pull request with only the fakelag and i’ll merge it

I updated this one.


[dv/common] initial support for shadow register

This PR is implemented as a first step to check shadow register (implemented in PR: https://github.com/lowRISC/opentitan/pull/2293)
This PR will automatically write twice (with the same value and same addr) if it is a shadow register.

Add a tag for shadow register in dv_base_reg
Add a knob to control if user wants to
a). write shadow register twice with the same values
b). only write shadow register once (later use for error case)

Signed-off-by: Cindy Chen<EMAIL_ADDRESS>

Thanks Sri,
Yeah I realized that when you suggested updating the update function. I
will finish them soon.
On Wed, Jun 3, 2020 at 1:02 PM sriyerg<EMAIL_ADDRESS>wrote:

@sriyerg commented on this pull request.
Sorry, I forgot one more thing - you will need to override predict as
well for the writes. You will need to 'stage' the first write. Wait for the
second write; Compare and ensure that they match; only then call the
super.predict. Else ignore the writes.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/2377#pullrequestreview-423887530,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACXPOOITAGZ2AM4EQFXXEPDRU2T6PANCNFSM4NRE4GHQ
.

Hey @cindychip,
thanks for working on this. I just wanted to confirm that I successfully tested this PR with the shadow reg draft on AES #2152 . The CI checks pass over there.

Hey @cindychip,
I just wanted to let you know that I have added a new feature to the shadow register. By performing a read operation, software can now clear the phase tracker (unless the register is read only, then the phase tracker is not affected by a read). So if for example the following sequence happens:

Write 0xFFFF_FFFF
Read
Write 0x1111_11111
The committed register is not updated and no update error is triggered. This feature was asked for to give software the opportunity to go back into a known state at any time.

I am not sure if you want to implement this in this PR yet or maybe at a later point (with the other follow-up items).

Hi Pirmin @vogelpi,
Thanks very much for letting me know. I think that is a good feature. I will wait until  this PR https://github.com/lowRISC/opentitan/pull/2152 get merged, then write a separate PR to implement that.

Hey @cindychip ,
PR #2293 got finally merged. This means we we should move forward with this one before we can ultimately merge also #2152.

Awesome! Thanks for letting me know.
Cindy Chen
On Tue, Jul 14, 2020 at 12:23 AM Pirmin Vogel<EMAIL_ADDRESS>wrote:

Hey @cindychip https://github.com/cindychip ,
PR #2293 https://github.com/lowRISC/opentitan/pull/2293 got finally
merged. This means we we should move forward with this one before we can
ultimately merge also #2152
https://github.com/lowRISC/opentitan/pull/2152.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/2377#issuecomment-658016684,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACXPOOLIOFMEEDPR3SQF563R3QBXJANCNFSM4NRE4GHQ
.

sorry about blocking everyone's progress. I will try to get this merged by tonight. @vogelpi

Thanks DV team for pushing this through and for your support :+1:


Question about adaptation？

Thanks for the  great work！
I want to know if  the weighted mask will be updated in online adaptation phase using a small amount of gt labels？

If weighted mask is not updated in online adaptation，maybe better unspervised loss can bring better performence？

No gt labels are used online. The weighted mask is predicted from RGB data and the small network predicting it is not updated.
Better unsupervised losses can definitely improve performances.


[0.8.4] Right-click Execute of Disabled Op no longer works - empty burn

In addition to an empty burn which only contains the before and after autoops, the stand and end dates and times in spooler history are at the epoch time 1970/01/01 00:00:00.


LlamaCppEmbeddings does not work

Currently, we are employing llamacpp via langchain as documented at https://js.langchain.com/docs/integrations/llms/llama_cpp.
(version: 0.0.204)
It looks like that the embeddings via LlamaCppEmbeddings class (located at: langchain/src/embeddings/llama_cpp.ts) still do not work for us, as the resulted vector's length is quite randomized, for instance:
[
1824,  349,   418,
274, 1402, 28709,
334, 8236, 28828,
28804
]
It is noted that, the different gguf models such as llama-2 or zephyr-ß result the same thing.
Your tips will help us in this case much.
Thank you in advance!

CC @nigel-daniels any ideas here?

@lanluu do you have some more context on what you are trying to do and how the failure manifests?

I too can confirm this issue. The LlamaCppEmbeddings is generating random length encodings which in turn is giving below error when inserting into any DB:
Error processing the file: ClickHouseError: Constraint cons_vec_len for table default.vector_search (e161893e-d050-4cc8-b412-6f8051dc83df) is violated at row 2. Expression: (length(embedding) = 49). Column values: embedding = [512., 13715., 362., 7569., 3062., 259., 13., 30211., 29871., 29906., 29900., 29906., 29906., 10811., 1693., 616., 512., 1890., 4803., 9333., 363., 4831., 291., 29880., 6897., 259., 13., 29906., 259., 13., 3562., 310., 2866., 1237., 259., 13., 23869., 12267., 259., 13., 29909., 15837., 310., 1857., 319., 29902., 2693., 1860., 259., 13., 1576., 5434., 310., 319., 29902., 259., 13., 1168., 10085., 259., 13., 29896., 29899., 29941., 259., 13., 29941., 259., 13., 29946., 29899., 29953., 259., 13., 29947., 259., 13., 11403., 4251., 259., 13., 29947., 259., 13., 11096., 2708., 393., 674., 337., 7922., 319., 29902., 259., 13., 29896., 29899., 29941., 259., 13., 24445., 800., 310., 1857., 319., 29902., 26811., 335., 29885., 448., 21784., 29257., 259., 13., 29941.]

@AnirudhaGohokar the length of the token array will be approximately relative to the length of the response, therefore this will result in a variable length array. The error you shared looks like the error is coming from a breaking table constraint in ClickHouse. Can you share the code you are using as I don't see why the resulting embedding array would map to a database constraint?

Ok got it so the error is the dimension size not the specific array length.

@awijshoff correct I am just passing the results from node-llama-cpp thru. I've started a discussion withnode-llama-cpp's maintainer on where the fix is best applied.

@giladgd thanks for the update, I'll await the node-llama-cpp v3 release then update accordingly.

I think #3509 is related to this. When changing the order of the text (moving "Bye bye" to the first position) HNSWLib returns
this.index.addPoint(vectors[i], docstoreSize + i);
                       ^

Error: Invalid the given array length (expected 4, but got 2).

HNSWLib internally sets this.args.numDimensions to the length of the embeddings of the first element of the passed elements.
    async initIndex(vectors) {
        if (!this._index) {
            if (this.args.numDimensions === undefined) {
                this.args.numDimensions = vectors[0].length;
            }

therefore this never fails for the first element, but in this case fails for the second with an array-lenght of 2. I think other vectorStores work very similar, for example in Faiss:
const dv = vectors[0].length;
        if (!this._index) {
            const { IndexFlatL2 } = await FaissStore.importFaiss();
            this._index = new IndexFlatL2(dv);
        }
        const d = this.index.getDimension();
        if (dv !== d) {
            throw new Error(`Vectors must have the same length as the number of dimensions (${d})`);
        }

Therefore I assume that the vectors/embeddings have to be of equal length for this to work? But that is just my wild guess.
This article supports my theory that every vector/embedding has to have the same length (MongoDB article).
If that is indeed the case, the issue seems to be in node-llama-cpp, because in the following code-example (using node-llama-cpp directly), the val1 val2 and val3 are arrays of unequal length, whereas they should have been arrays of equal lenght - the dimension of the model?
import { LlamaModel, LlamaContext, LlamaChatSession } from "node-llama-cpp";

const llamaPath = "models/mistral-7b-v0.1.Q5_K_M.gguf"; //english

const model = new LlamaModel({
  modelPath: llamaPath,
});

const context = new LlamaContext({ model });

const val1 = context.encode("hello nice world");
const val2 = context.encode("Hello world");
const val3 = context.encode("Bye bye");

@awijshoff Thanks for detailing this. The issue I created is exactly what you've described and am very excited about the update coming down the pipe for it. :)

this is not a bug. the context encode and decode functions are meant to turn text into the tokens actually sent to the llm (it is not sent text), so they will vary with text input length. if your example above happened to match, it's because one or more words used more than one token. it's not 1:1. encode is good for determining if your input text will exceed the context length of the model. for embedding vectors, you will want to use a model designed for generating semantic search vectors. you can get these from Huggingface. in JS you can do something like:
import { HuggingFaceTransformersEmbeddings } from "langchain/embeddings/hf_transformers";
import { env } from '@xenova/transformers';

const modelName = "Xenova/all-MiniLM-L12-v2";
const model = new HuggingFaceTransformersEmbeddings({ modelName });
const prompt = "query or prompt to transform into a semantic vector";
const vector = model.embedDocuments([ prompt ]);

in the case of the Xenova model above, you will always get a vector of dimension 384. then for each of your document chunks being stored in a vector db, you will use the same embed call. that way vectors are comparable. the vector db can take it from there.

@giladgd Hi, we are so excited to be able to work with node-llama-cpp v3 👍. Thank you for your effort. Do you have a hint regarding the release date of V3 for us? :-)

@lanluu I predict it'll take about ~1 month to finish the major features that contain breaking changes for the v3 beta, so the stable release would come a bit after that

I've added support for embedding in the beta version of node-llama-cpp (3.0.0-beta.3); you can find a code example of how to use it here.
Note that since this is a beta version, the API may change before the stable release.

@giladgd that's great, thanks. Looking forward to the 3.0.0 release :)

I've added support for embedding in the beta version of node-llama-cpp (3.0.0-beta.3); you can find a code example of how to use it here.
Note that since this is a beta version, the API may change before the stable release.

I used node-llama-cpp (3.0.0-beta.3) with langchain(2.8.8), but still got TypeError: this._context.encode is not a function
at file:///Users/panjunlin/work/github/chat-mind/node_modules/langchain/node_modules/@langchain/community/dist/embeddings/l lama_cpp.js:48:39, is the langchain version wrong?

I also used node-llama-cpp (3.0.0-bet.14) earlier, but found a new error: file:///Users/panjunlin/work/github/chat-mind/node_modules/node-llama-cpp/dist/evaluator/LlamaModel.js:22
constructor({ modelPath, gpuLayers, vocabOnly, useMmap, useMlock, onLoadProgress, loadSignal }, { _llama }) {
^
TypeError: Cannot destructure property '_llama' of 'undefined' as it is undefined.
at LlamaModel (/Users/panjunlin/work/github/chat-mind/node_modules/node-llama-cpp/src/evaluator/LlamaModel.ts:63:9)

Cannot destructure property '_llama' of 'undefined' as it is undefined.
Ran into this bug as well, solution was to use node-llama-cpp without langchain.


bot detection documentation isssue

Description:
i am following link
https://apim.docs.wso2.com/en/latest/learn/api-security/threat-protection/bot-detection/
Steps to reproduce:
below the image on the portal is not as per YAML file of the worker, the description is mismatched

Content Positioning in Documentation:

Link:

Heading (& Step):

Any other reference:

Optional Fields
Related Issues:

Suggested Labels:

Suggested Assignees:

@nadeepoornima , please share your input with regard to this git issue.

Seems like it there is the wrong description for the AM_DB data source config in the deployment.yaml. The description belongs to the Micro Gateway data source description. The description should correctly like "The data source used for AM_DB which shared from APIM". Others are ok.

Fixed in -

3.1.0 - https://github.com/wso2/docs-apim/pull/1067
Master branch - https://github.com/wso2/docs-apim/pull/1068


Changed link for GAE SDK download

Changed the URL and linktext for the GAE SDK download link. I followed the instructions and downloaded and installed the SDK from the original link, but that installs only the SDK and not the Launcher (GUI). Trying to follow the instructions with that version or flavour of the SDK is impossible.

I signed it!

This should be fixed as of the PR by @GlynnPhillips (#22)

Thanks for taking time out to contribute :+1:


Implement rules around star expressions with different precedence

Summary
This PR adds implementations around the following grammar rules to support the other expression and statement parsing. This PR doesn't update the related expressions and statements as they will be done in their own PRs.

star_expressions
star_expression
'*' bitwise_or

Problem
There are two grammar entries around star expressions:
star_expression:
    | '*' bitwise_or
    | expression

starred_expression:
    '*' expression

The main difference is the precedence at which the expression should be parsed if a * token is encountered. Currently, we parse both rules at the same precedence which makes the parser more lenient. There are various expressions and statements which uses either one of the above rules.
Solutions
Current solution
Add the following new methods to the parser:

parse_star_expression_list which would be similar to parse_expression_list and would match the star_expressions grammar rule
parse_star_expression_or_higher which would match the star_expression or star_named_expression grammar rule
parse_expression_with_bitwise_or_precedence which would match the bitwise_or grammar rule

Here, we could possibly merge the parse_expression_list and parse_star_expression_list into a single function as it only differs in the method it delegates the parsing to. The parameter could be called allow_star_expression. It could be confusing because even if it’s false, we would allow starred expression but at a higher binding power.
This solution closely resembles the actual grammar and is easier to follow in the parser code. It doesn't require a lot of verification by the caller as the precedence parser makes sure to not allow that.
Other solution
Add a new verify_bitwise_or_precedence method which reports an error if the grammar expected a starred expression with bitwise_or precedence:
pub(super) fn verify_star_expressions(
    &mut self,
    expr: &Expr,
    allow_named_expression: AllowNamedExpression,
) {
    match expr {
        Expr::Tuple(ast::ExprTuple { elts, .. }) => {
            for expr in elts {
                self.verify_star_expressions(expr, allow_named_expression);
            }
        }
        Expr::Starred(ast::ExprStarred { value, .. }) => {
            let expr_kind = match &**value {
                Expr::Compare(_) => "comparison",
                Expr::UnaryOp(ast::ExprUnaryOp {
                    op: ast::UnaryOp::Not,
                    ..
                }) => "unary `not`",
                Expr::BoolOp(_) => "boolean",
                Expr::If(_) => "`if-else`",
                Expr::Lambda(_) => "lambda",
                Expr::Named(_) if allow_named_expression.is_no() => "named",
                _ => return,
            };

            self.add_error(
                ParseErrorType::OtherError(format!(
                    "{expr_kind} expression cannot be used here",
                )),
                expr.range(),
            );
        }
        _ => {}
    }
}

For this solution, the parser will need to verify the precedence of a starred expression wherever grammar expected the star_expressions, star_expression or star_named_expression rule.
We could possibly adopt this solution for better error recovery but I think for now having a strict parser is better suited because we don't really know what complete error recovery looks like.
Follow-up
There's a related follow-up issue which I'm working on and is related to starred expressions.
The starred expression is only allowed in certain expressions and context which the parser needs to make sure and report an error if found otherwise.
Test Plan
Tested this locally by changing various expression and statement to use these methods. As mentioned they will be in their own PR.

(Converting to draft because I think it might be better to use the second solution as described in the PR description.)

For future reference, this change is going to be updated to use the "other" solution mentioned in the PR description. Not all will be reverted but instead of having a parse_expression_with_bitwise_or_precedence, we'll allow the starred expression to be parsed with the highest precedence and limit it later by checking if it's allowed as per "bitwise or" precedence and report an error.
I don't want to add the change here as it'll require updating all of the PRs in the stack. I'll link the PR when it's opened.


Use of HTTP

Greetings,
I am a security researcher, who is looking for security smells in Puppet scripts. I found instances where the HTTP protocol is used instead of HTTPS (HTTP with TLS). According to the Common Weakness Enumeration organization this is a security weakness (https://cwe.mitre.org/data/definitions/319.html).  I was wondering why HTTP is used? Is it because of lack of tool support?
I am trying to find out if developers are forced to adopt bad practices due to lack of tool support when it comes to the HTTPS protocol. Maybe it is due to dependency on a resource that uses HTTP?
Any feedback is appreciated.
Source: https://github.com/pcfens/puppet-filebeat/blob/master/manifests/repo.pp (Line 57)

Puppet itself supports HTTPS, but the underlying tools that puppet calls don't always support it (e.g. apt without the apt-transport-https plugin).
The code you referenced is about 2 and a half years old, so I have no idea why it originally didn't use https (maybe Suse at the time didn't). I've gone ahead and changed it - hopefully someone that uses Suse will be able to test before releasing another copy the the forge.


Unit €/MWh should be entirely visible on chart axis

Currently only €/M is visible instead of €/MWh in chart 244: Production costs and price of green gas:

Fixed in 4c1d995.


add pcc64le support

Needs https://github.com/conda-forge/vtk-feedstock/pull/165
Fixes #60.

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

License is not an SPDX identifier (or a custom LicenseRef) nor an SPDX license expression.

Documentation on acceptable licenses can be found here.

@conda-forge-admin, please restart ci

@conda-forge-admin, please restart ci

Ready for review @looooo

Should be closed in favour of https://github.com/conda-forge/occt-feedstock/pull/64 which does native builds compared to cross builds here.


[Feature Request]: Creators Page for website

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
Create a well designed Creators page for the website,
I have already added the routes.
Make sure that page follows home page design patterns
Use Case
helps in listing of all nft creators

Can be used for navigating to their Creations
Useful for creators sharing their pages on social platforms

Benefits
Better web structure along with creators listing
Priority
High
Record

[X] I agree to follow this project's Code of Conduct
[ ] I'm a WOB contributor
[ ] I want to work on this issue
[ ] I'm willing to provide further clarification or assistance if needed.

@4darsh-Dev, could you please assign this issue to me? I would like to contribute.


[BACKEND] Clean up and optimize shared memory allocation for atomic ops

Unnecessary barriers have been identified and removed because the membar pass should have taken care of inserting necessary barriers.

cc @antiagainst @ThomasRaoux , affecting both amd and nvidia gpu backends


Use a real SessionBase object on FooterNoSessionMiddleware

Instead of forcing to be an empty dict {}, we force it to be a real SessionBase object which it's what Django uses internally. This avoid issues where other pieces expects the SessionBase.
Django docs: https://docs.djangoproject.com/en/1.11/topics/http/sessions/#django.contrib.sessions.backends.base.SessionBase

I'm pretty sure this isn't the solution to our problem. We don't want a session on the request, anywhere downstream that thinks it's editing a session, that data won't be saved. We want that to error, and to stop trying to edit the session, not fake out users who think they are really editing it.

Hrm... I implemented this solution because we are already faking it with a dict (allowing get/set values). The comment says that otherwise Auth backend fails.
On the other hand, using this general SessionBase will behave similar to a dict (it's a dict-like object) but actually it will fail with a proper error (NotImplementedError) when calling the methods .save, .delete, .create, etc instead saying that the dict does not have this method.
Finally, for the use case I'm trying to fix solve, it will work since request.session.session_key will return None which it's expected when there is no session.
I'm not 100% sure, but seems to be the right way of having the "same behavior than a regular session object without saving anything and hard-failing if trying to save it".

I know that this affects a lot of requests. So, if we are in doubt we can just go in another direction that we consider safer. Although, at some point this will bite us again.

Good point. I mostly have 2 reservations:

I tried something similar when I wrote this code, but I don't think it was quite this
Like you mentioned, the scale of the change. Though it should be pretty obvious if this breaks, so we can just roll it back.

So I guess I'm +0 on this proposed change, then.

OK. I think we can merge and deploy carefully being prepared to roll it back, as you said. Although, we can have another person reviewing this before merging.


Re-asking when feedback enabled

When feedback is enabled, the user cannot re-ask the question since that button is disabled.
I think best solutoin (which would be also UI improvement when feedback is disabled) is to have another submit button on the "Concepts" headling line, which never gets disabled and is labelled something like "Re-ask with only selected concepts".

Button created!


Update New-StorageTier.md

Made changes to the example to reflect the tier name
fixes #https://github.com/MicrosoftDocs/windows-powershell-docs/issues/2954

CC @scanum

Learn Build status updates of commit f58f36f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2022-ps/storage/New-StorageTier.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 24cb562:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2012r2-ps/storage/New-StorageTier.md
:white_check_mark:Succeeded
View (winserver2012r2-ps)

docset/winserver2022-ps/storage/New-StorageTier.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit fdb1661:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2012r2-ps/storage/New-StorageTier.md
:white_check_mark:Succeeded
View (winserver2012r2-ps)

docset/winserver2016-ps/storage/New-StorageTier.md
:white_check_mark:Succeeded
View (WindowsServer2016-ps)

docset/winserver2022-ps/storage/New-StorageTier.md
:white_check_mark:Succeeded
View (WindowsServer2022-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@tiburd Could you please take a look at this one? Thanks!


florance vision model

https://huggingface.co/microsoft/Florence-2-large/tree/main uses pytorch
https://huggingface.co/spaces/SixOpen/Florence-2-large-ft

This is kind of related to what I am working on. I am trying to bring the Florence-2-base model into ollama (manually).
https://github.com/ollama/ollama/issues/5195

Is there still hope for Florence 2 in Ollama?

@javierxio  Is there any updates?

@dev3py It seems that the florence is a pytorch model and not supported by ollama.
C:\ollama_models\florence-2-base>ollama create flo -f ./Modelfile -q Q5_K_M
transferring model data 100%
converting model
Error: unsupported architecture

Have you tried lm studio?

@javierxio I have no idea. I am not a python dev

@dev3py i mean, have you tried another program (not ollama). check out LM Studio:
https://lmstudio.ai/
They may have florence model available.


Urgent!!! There was an error changing your password-HRESULT: 0x800704F1

Hello everyone.
The application was working for 10 days without problems.
But then I get the following error and searching the Internet says it is an upgrade.
These updates are not installed on my Windows Server 2012r2 (my domain is Windows Server 2012r2)
**** Error Information: The system cannot contact a domain controller to service the authentication request. Please try again later. (Exception from HRESULT: 0x800704F1)

You need to uninstall:
KB3167679
KB3177108
Reboot your machine, then hide these updates.
Check for new updates, then install the latest patches.

Thank you.
But Now i have:
Error Information: The password does not meet the password policy requirements. Check the minimum password length, password complexity and password history requirements. (Exception from HRESULT: 0x800708C5)
[cid:e42ccc76-ed37-44d1-8ffb-32461e9034fc]

From: mdbrauer<EMAIL_ADDRESS>Sent: Friday, October 14, 2016 10:52 AM
To: unosquare/passcore
Cc: marlonc1980; Author
Subject: Re: [unosquare/passcore] Urgent!!! There was an error changing your password-HRESULT: 0x800704F1 (#46)
You need to uninstall:
KB3167679
KB3177108
Reboot your machine, then hide these updates.
Check for new updates, then install the latest patches.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/unosquare/passcore/issues/46#issuecomment-253857420, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AVt9HeRC_8F9gFzQSG2vf7tAjXqMSTVJks5qz7NHgaJpZM4KTxRP.

Is there another work around for this issue?  I am not able to uninstall those updates.

I have the same issue from beginning,

There was an error changing your password
Error Information: The password does not meet the password policy requirements. Check the minimum password length, password complexity and password history requirements. (Exception from HRESULT: 0x800708C5)

we actually need to be a simple password not complex "or with user option" I hope to get help very soon

Please try with the latest version and let us know if this is still a problem. Reopen the issue if necessary.

It is a old post but might be helpfull for the people like me.I was struggling for 4 days and this link solve my problem. Good Explanation.
http://blog.cionsystems.com/?p=907


Panning and zooming

It would be nice to support pan-and-zoom interaction (à la d3-zoom) in dense scatterplots with Plot. Presumably this would work by altering the scale domains and then re-rendering the plot.

It would be nice if zooming and panning were supported in geo contexts as well.

In the meantime, something along those lines is working https://observablehq.com/d/223f34bca10702a3

Thanks for sharing @jcolot!

I would like zooming in all kinds of plots, and the ability to lock either the x or y axis so it cannot be zoomed. This should also work in combination with brushing / selection of a part of the plot, so a suggestion is to use e.g. the Alt-key for zooming. In case you don't know, here is how Plotly does it, although I'm not a fan of the hovering menu they have in the corner of the plot. Thanks!

It looks like you almost have the necessary functionality already, as zooming can be hacked by changing the domain values as shown in this example. This just needs to be hooked up to the mouse, so when the user e.g. holds the Alt-key and clicks and moves the mouse in the plot, it zooms in / out centered on the mouse-pointer. I think that would work great. It also needs to work with brushing / selection of data.

I'm excited for zoom functionality to exist in Plot!
An added consideration:
In my use case, to save on data transfer time, I'm downsampling high-resolution timeseries channels to my plot's resolution on the backend, so when I zoom in, I'll need to fetch more data at some point.
To support this kind of use case, it would be great if the Zoom mark allowed you to pass in a callback that would get called with the new extents when a zoom action completes, so you can react by fetching more detailed data. This would benefit users doing dynamic backend downsampling like I am, or with anyone serving tiled data sets.

In the meantime, something along those lines is working https://observablehq.com/d/223f34bca10702a3

I tried to use this mechanism in a standalone web page and I observed something weird: While the wheel event works fine, the mousemouse does not. More precisely, the first "one pixel" pan works but the next provokes a big jump in the event transform captured by "zoomed" function, which makes is unusable.
I suspect that the mousemove event does not completely stops while still panning, but in the meantime a new plot was rendered, a new zoom created, and I guess the fact that the ongoing mousemove started before the current zoom does not play well...
I "feels" like the the wheel events are a sequence of discrete, disjoint event so that each event starts after the latest plot rendering. And it works fine.
In usual d3, it seems to me that the zoom object is not meant to be recreated after each event. It is long lived and after each event a transform property of some svg can be updated.
But with Plot (at least this example), where the whole chart is re-rendered so after any rendering the zoom must be recreated. As a consequence the transform event is reset at every step, which makes method .scaleExtent() pointless, for example.
Is this description approximately correct ?
How to make it work with mousemove events ?
Naively, I would think if there were a render function with new options on an existing plot, it would work ?
Is there a way to avoid re creating the zoom object at each step ?
Thx in advance for any hint or guidance.

It would be nice if pan and zoom worked with Protomaps base tiles: https://protomaps.com/
Context:

tippecanoe can convert between the Protomaps pmtiles format and geoparquet formats for use in duckdb with the spatial extension
duckdb spatial extension is supported in Mosaic, which in turn is supported in observable framework)

I’ve added brush-to-zoom to the mortgage-rates example:
https://observablehq.observablehq.cloud/framework-example-mortgage-rates/
Source is here:
https://github.com/observablehq/framework/blob/721ead19300031d3cae4ede41d2c85908d20141c/examples/mortgage-rates/src/index.md?plain=1#L134-L169

We are looking to make a switch and hope this feature comes out soon!

this gets a vote from me, as I am in process of moving this app into observable framework as a validation piece
I use Zoom on globe and Pan on Mercator: i have a few workarounds, using controls instead, but the UX might be better with these ported over from D3. I am giving it a go my self, not successful :( i need to learn to program better
https://www.youtube.com/watch?v=I44bHsFPZjU
thanks for the great framework


Add LearningTest lite to the repo

A first proposal is to add LearningTest in the test directory with:
├── cmd
│   ├── python
├── datasets
│   ├── Adult
│   ├── Iris
│   └── Mushroom
├── MTdatasets
│   └── SpliceJunction
├── TestCoclustering
│   ├── Standard
├── TestKhiops
      └── Standard

I'm not sure it is necessary to add all these data. I was using LearningTest to test packages. For this purpose Iris.txt is good enough. We have to decide what is the purpose of LearningTest in CI

We need the python scripts in the Khiops repository for ci tests (mainly to compare results and references). But also on our local machines to run the full LearningTest. I'm not sure it is a good thing to put these scripts on the Khiops repository:

by default, the scripts execute the tests located relatively to its own directory
if we test an old release of Khiops, we must use an old version of the scripts

Why not a separate github project just for the python scripts and a pypi package: it will be available everywhere ?

There may be several levels of tests:

automatic basic test, for commits: only one test with Iris, for both exe MODL and MODL_Coclustering
automatic standard test, for pull requests: TestKhiops/Standard and TestCoclusteringStandard
manual exhaustif test, with all LearningTest

can be used anytime by any developper for massive non regression tests
should be used for important pull requests
must be used before merging on main branch

One first question if where to store:

code for performing tests (LearningTest/cmd and LearningTest/doc)

on a distinct repo?

data for all datasets
data for for test scripts only
data for reference results

Remarks

the volume for basic and standard test is low (some tens of Mb) and consistent with the storage on a public repo
the volume for exhaustif test if very important, and should be store on an internal data store

datasets: ~5 GB
scripts: ~1 Gb (could be reduced if local datasets are externalized)
reference results: ~3 Gb

the scripts and reference results must be versionned
the time to perform exhaustive test is very important

several hours for all tests
between one and two day for all+complete tests

On last question is how to synchrornize the version of the Khiops repo and that of LearningTest

the test scripts must be versioned

beware that one some rare "special" commit on the Khiops repo have an impact on the scripts
keep track of these "special commits" by using tags (including in the dev branch)?
use these tagsto synchornized with the LearningTest repos?

the reference results should be versionned

but they may slight vary whith the version


Pipeline to deploy terraform

As devops engineer I would like a pipeline to automatically deploy terraform to each environment
AC Pipeline that deploys terraform to Sandbox, Dev, Prod

Linking to lsc-sde/lsc-sde#39


restic-rest-server: init at 0.9.7

Package is named as the OpenBSD one: https://repology.org/metapackage/restic-rest-server.
Things done

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

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

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

/cc @mbrgm

@GrahamcOfBorg build restic-rest-server

Success on x86_64-linux (full log)
Attempted: restic-rest-server
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/syvdgziki033n157r57s5pxfd26sr1l1-restic-rest-server-0.9.7-bin
shrinking /nix/store/syvdgziki033n157r57s5pxfd26sr1l1-restic-rest-server-0.9.7-bin/bin/rest-server
cannot find section .dynamic
strip is /nix/store/fzcs0fn6bb04m82frhlb78nc03ny3w55-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/syvdgziki033n157r57s5pxfd26sr1l1-restic-rest-server-0.9.7-bin/bin
patching script interpreter paths in /nix/store/syvdgziki033n157r57s5pxfd26sr1l1-restic-rest-server-0.9.7-bin
checking for references to /build in /nix/store/syvdgziki033n157r57s5pxfd26sr1l1-restic-rest-server-0.9.7-bin...
cannot find section .dynamic
strip is /nix/store/fzcs0fn6bb04m82frhlb78nc03ny3w55-binutils-2.28.1/bin/strip
/nix/store/syvdgziki033n157r57s5pxfd26sr1l1-restic-rest-server-0.9.7-bin

I'd like to mention that the binary is also called restic-rest-server in the case of OpenBSD, but I decided to rather not change it and use rest-server.

Success on x86_64-linux (full log)
Attempted: restic-rest-server
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/syvdgziki033n157r57s5pxfd26sr1l1-restic-rest-server-0.9.7-bin
shrinking /nix/store/syvdgziki033n157r57s5pxfd26sr1l1-restic-rest-server-0.9.7-bin/bin/rest-server
cannot find section .dynamic
strip is /nix/store/fzcs0fn6bb04m82frhlb78nc03ny3w55-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/syvdgziki033n157r57s5pxfd26sr1l1-restic-rest-server-0.9.7-bin/bin
patching script interpreter paths in /nix/store/syvdgziki033n157r57s5pxfd26sr1l1-restic-rest-server-0.9.7-bin
checking for references to /tmp/nix-build-restic-rest-server-0.9.7.drv-0 in /nix/store/syvdgziki033n157r57s5pxfd26sr1l1-restic-rest-server-0.9.7-bin...
cannot find section .dynamic
strip is /nix/store/fzcs0fn6bb04m82frhlb78nc03ny3w55-binutils-2.28.1/bin/strip
/nix/store/syvdgziki033n157r57s5pxfd26sr1l1-restic-rest-server-0.9.7-bin

Success on aarch64-linux (full log)
Attempted: restic-rest-server
Partial log (click to expand)
/nix/store/xnxyaizcsj7k71y1lp1i0njncjnfdl1k-restic-rest-server-0.9.7-bin

Success on x86_64-darwin (full log)
Attempted: restic-rest-server
Partial log (click to expand)
find: '/nix/store/b5rhrj7rvbavnma8hf0spmzz1397zvi2-restic-rest-server-0.9.7': No such file or directory
find: '/nix/store/b5rhrj7rvbavnma8hf0spmzz1397zvi2-restic-rest-server-0.9.7': No such file or directory
find: '/nix/store/b5rhrj7rvbavnma8hf0spmzz1397zvi2-restic-rest-server-0.9.7': No such file or directory
find: '/nix/store/b5rhrj7rvbavnma8hf0spmzz1397zvi2-restic-rest-server-0.9.7': No such file or directory
find: '/nix/store/b5rhrj7rvbavnma8hf0spmzz1397zvi2-restic-rest-server-0.9.7': No such file or directory
find: '/nix/store/b5rhrj7rvbavnma8hf0spmzz1397zvi2-restic-rest-server-0.9.7': No such file or directory
find: '/nix/store/b5rhrj7rvbavnma8hf0spmzz1397zvi2-restic-rest-server-0.9.7': No such file or directory
find: '/nix/store/b5rhrj7rvbavnma8hf0spmzz1397zvi2-restic-rest-server-0.9.7': No such file or directory
find: '/nix/store/b5rhrj7rvbavnma8hf0spmzz1397zvi2-restic-rest-server-0.9.7': No such file or directory
/nix/store/yipn9cayvg3lhzjz3vsldcyhx3bl45m7-restic-rest-server-0.9.7-bin


Remove "Converger" metrics from the metrics docs.

Since the converger is no longer a component and there is nothing emitting those metrics should those not be removed from the doc?

Thanks, @andrew-edgar. Coincidentally, https://www.pivotaltracker.com/story/show/137740383 recently made it to the top of the Diego backlog, so that's done in that story.
Best,
Eric


Update kanela-agent to 1.0.8

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

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.kamon", artifactId = "kanela-agent" } ]

labels: library-update, semver-patch

Superseded by #154.


Fix #66 : show error traceback when pyramid's bootstrap fails

As said in comment I don't like this solution but at least it works and avoid celery to start with wrong configuration values.

Coverage decreased (-2.6%) to 93.836% when pulling 45df08795e172b77f289b7481325cfdc4dc52754 on tonthon:fix_silent_fail into efecf544a5d79be17417725c4c6cece378a29546 on sontek:master.

It seeems to be a travis configuration error:
ERROR: InterpreterNotFound: pypy on travis

Similar content already pushed to the master branch


better landmark docs

Please review and also waiting for #960 being merged ...

Will correct the problems and also add the new prepare.lm.threads config possibility.


Add CORS handling in stream API server

I'm developing a web GUI in VUE to manage streams via API. And I've some problems with CORS. It would be interesting if we can configure CORS on the server-side.
Thanks

Hey @dhinojosac, I've added some CORS fields to the http spec, you can see the preliminary documentation here: https://github.com/Jeffail/benthos/blob/master/website/docs/components/http/about.md
I've taken a best guess at some of the config defaults, the underlying choice of allowed methods is GET, POST, HEAD, DELETE as those are the methods used in the streams API. I've left allowed origins as a required config field, if you need any changes or extra fields let me know and I'll aim to get them in before the next release.

Thanks, @Jeffail . I will test this feature and let you know if we need other fields. It would work for us in advance.

ahhh, currently, I'm using the PUT method to edit stream in a Textarea input in the Vue application. It would be helpful in the next release!

Already test it, and it works fine! Thanks, @Jeffail


Add Error enum to JSON rpc client

Motivation
Add error enum to json rpc client to distinguish between retriable and non-retriable error.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
cargo test

@dmitri-perelman I think the reqwest::Error already includes all three types of error you mentioned already. I'm not against having a more detailed enum though, because one risk we are taking here is that we are making ourselves depend on specifics of reqwest here, may make it hard to replace in the future

@bors-libra r=dmitri-perelman

:pushpin: Commit 7b79692 has been approved by dmitri-perelman

:hourglass: Testing commit 7b7969293e151841039d2fed956a698d1f043bbc with merge b1aaedd3fc0290c136a7266bde38a97a37cf72f7...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: dmitri-perelman
Pushing b1aaedd3fc0290c136a7266bde38a97a37cf72f7 to master...


Upgrade Appium Desktop to v1.1.1 & github hosting

Appium has migrated their code to github. The version
in the current caskroom points to an old binary
hosted on bitbucket.
The project is now
hosted and distributed on github.
I've also changed the cask token name to avoid
confusion between 'appium', the server for mobile
automation, and 'Appium Desktop' the desktop client
that allows for access and interaction with an
appium server.
I've also deleted the outdated cask. This exact situation
wasn't covered in the CONTRIBUTING.md, but seemed like
the right way to go.
Token: appium-desktop
Version: 1.1.1
If there’s a checkbox you can’t complete for any reason, that's okay, just explain in detail why you weren’t able to do so.
After making all changes to the cask:

[ X] brew cask audit --download {{cask_file}} is error-free.
[ X] brew cask style --fix {{cask_file}} reports no offenses.
[ X] The commit message includes the cask’s name and version.

Additionally, if updating a cask:

[ ] If the sha256 changed but the version didn’t,
provide public confirmation by the developer: {{link}}

Additionally, if adding a new cask:

[ X] Named the cask according to the token reference.
See comment regarding disambiguation.
[ X] brew cask install {{cask_file}} worked successfully.
[ X] brew cask uninstall {{cask_file}} worked successfully.
[ X] Checked there are no open pull requests for the same cask.
[ X] Checked the cask was not already refused in closed issues.
[ X] Checked the cask is submitted to the correct repo.

I've also changed the cask token name to avoid
confusion between 'appium', the server for mobile
automation, and 'Appium Desktop' the desktop client
that allows for access and interaction with an
appium server.

Has the appium server been discontinued?

Nope.
I think I may have conflated to requirements, though.
Just to be clear, Appium distributes two "products".
Appium Desktop (a GUI that has a bundled Appium server and some tooling to help configure/discover testing commands.  distributed via releases on GitHub.)
appium (the standalone server, distributed via npm).
This is sort of documented on their home page, but the project itself doesn't make a huge distinction.
That being said, It makes sense to keep the token reference as is.  I'll update and push.

Will do.

Thank you for your contribution.


Request: Ogg Vorbis and Opus

Hi @ClementBeal and thanks for creating this libary!
I am currently try to find a pure dart replacement for metadata god  which has a rust dependency which makes it hard to test and is also not very fast, for MusicPod
Your library works great for MP3s
However some of my users use flac and ogg vorbis and opus.
It would be great if you could add those too :) Thanks in advance and best regards

Hi @Feichtmeier ,
Your comment sparked a little of joy in me!
About your request, I'll check in the week how to fix the issue. It's a strange behavior because I'm using this library to build my custom audio player too and it works 90% of the time correctly with Ogg and FLAC.
I'll keep you updated

Added!  There might be some issues for the duration calculation.
Sometimes it would need to parse all the file to get the correct duration. I'll think about a solution!


Adhere to scss-lint

I was cleaning this file up for my own use and wanted it to adhere to scss-lint. Entirely up to you whether or not to use it.

Merged! Thanks, @CookPete.


Fix cordova auth for IOS and Android

Motivation
WIP

Works for me on both Android and iOS.
Should there be the addresses? Can we use <allow-navigation href="*"/> instead?
As Auth module initializes after cordova "deviceready" event, authentication no longer works for npm run ionic:serve.

@jhellar Awesome work. Thank you for helping me with
@TommyJ1994

Yeah, I think the addresses there can be removed as the issue was server-side in incoming requests.

Yes. Committed in the rush. Going to remove it

Regarding the deviceready trigger for auth, I think we can probably remove the init from the startup/bootstrap and instead just have it on the Authenticate Button

I think this should be kinda canonical solution that encourage people to use lib the way is documented. If we do not init at the beginning would we get valid app token. For the cases when app is launched and you are actually logged in.
I may misunderstood the keycloak.js code but init seems to be crucial part to make everything work.

@TommyJ1994 - I would love to get info what whitelist was added. Curious if we should document that for local development using this template.

@wtrocki Actually I think this is ok about the init() placement.
I added http://localhost:8080/* to the web origins, http://localhost/* was here before, but does not seem to catch it on iOS, only on Android. I can sync up with @jhellar later and experiment more to see if there is a catch-all for all platforms, and if necessary create some docs to handle iOS individially.

@TommyJ1994 It feels like hostname (with port) should be a standard here.


Discord invite expired

The discord invite expired.

Here is one for you, i'll decide later about what kind of invite i put in the readme.
https://discord.gg/Kujn4uaJ (5ppl, 7days)


Bar Heights in Oncoplot Wrong When Copy Number Used

If both SNVs and copy number are plotted, the bar chart only counts samples that have either a SNV or a copy number change, but not both. A minimal example which can simply be copied and pasted for testing:
library("maftools")

mutations <- data.frame(Hugo_Symbol = rep(c("TP53", "TSC2"), c(4, 1)),
                        Chromosome = '.',
                        Start_Position = '.',
                        End_Position = '.',
                        Reference_Allele = '.',
                        Tumor_Seq_Allele2 = '.',
                        Variant_Classification = "Missense_Mutation",
                        Variant_Type = '.',
                        Tumor_Sample_Barcode = c("Adam", "Ben", "Chris", "David", "Edward"), stringsAsFactors = FALSE)

copyNumbers <- data.frame(Gene = c("TP53", "TP53", "TP53", "TP53", "BRAF", "TP53"),
                          Person = c("Adam", "Ben", "Chris", "David", "David", "Zena"),
                          Copy = rep("Amp", 6), stringsAsFactors = FALSE)

MAF <- read.maf(mutations, cnTable = copyNumbers)
oncoplot(MAF)

Notice how TP53 is altered for 83% of samples but the bar's length only reaches to 1 out of 5. The bars above each column have the correct height.

Hi @DarioS ,
Sorry for the delay in addressing the issue, and thanks for reporting.
There is no easy way to represent these. I have annotated them as Complex_Event (Mutation+Amp/Del). Please pull the updated version and let me know.

It's plotted in a reasonably meaningful way in the updated version.


Implemented config directory support

Especially in the case of Rackers, many of us have an extremely large number of environments for supernova. This pull request implements an extra function (get_config_dir()) which attempts to pull from ~/.config/supernova.d/ or ~/.supernova.d/.
If it finds either directory, it will append each regular file the directory contains to the configObj. Currently, it allows for a config file, a config directory, or both.

I like where you're going with this, but some tests have failed. Could you look over the report from Travis CI above?

Hey Major,
Thanks for the feedback! I have to confess this is my first time working with Travis and unit tests, but it looks like I finally have full test coverage, and 100% pass.
Is there anything else you need from me prior to merging?
Thanks!
Russell

Thanks for that, @swyytch.  I'll get a look at this later today.

  I couldn't figure out a way to merge both without some code edits.
Would you have a chance to rebase your commits on top of the current master branch?  You did a great job fixing the tests!

Hey @major ,
Thanks for the feedback! rebasing was a new experience for me as well, but I think I finally got it sorted out. Would you mind taking a look and letting me know what you think?
Thanks!
Russell

Sorry for the long delay, @swyytch.  If you look at the "Files changed" tab, you'll see that your branch reverts some things from the .spec and removes a bunch of blank lines from some files.  Is there any way you can remove some of those changes so we can merge this cleanly?

I think that this PR can be closed in favor of #121.


.net core 3.1 support

Hi!
I found out that AnonymousIdentity can't work with .net core 3.1, because of changes in IdentityServer4 library - some extension methods were move into other namespace
There are some changes to fix this problem

@Sverdel I updated the nuget package. Enjoy =)


is Norconex vulnerable to CVE-2021-44228 ?

Hi
we have Norconex 2.8 installed on our servers, and we would like to know if the log4j Libraries  included in this version are vulnerable to what is reported in https://discuss.elastic.co/t/apache-log4j2-remote-code-execution-rce-vulnerability-cve-2021-44228-esa-2021-31/291476.
many thanks
Mauro

Hello Mauro,
No, your version of HTTP Collector is not affected by CVE-2021-44228.  For a few reasons:

CVE-2021-44228 applies to Log4j version 2.x, and the HTTP Collector version you are running uses Log4j 1.x.
Unless you made it so on purpose, the HTTP Collector does not run as a service and is not accessible from the internet.

Note about upcoming HTTP Collector version 3:
The upcoming version 3 of the HTTP Collector uses Log4j 2 by default but it uses the patched release.  Also, you can replace logging with a different implementation.

Hi Pascal
thanks a lot for the prompt reply.
great to hear our version is safe!
we have installed Norconex as local service which can be executed only from command line and it's not reachable from internet; considering that we have upgraded Elastic Search to version 7.16 (which used patched log4j 2.17) we can consider our environment safe.
many thanks again and wish you a great start of the new year.
Mauro

The upcoming version 3 of the HTTP Collector uses Log4j 2 by default but it uses the patched release. Also, you can replace logging with a different implementation.

Hi! Where can I find the documentation on how to replace logging with a different implementation?
Thanks!


Remove murmur3 from the codebase and use xxh3 for HLL

Currently fails some of the HLL tests. Probably need some input from @apavlo since HLL tests are a mystery to me.

I will look into this tonight / tomorrow

This is what the failure looks like on my machine, Jenkins may provide something a bit different.
[2019-10-09 17:20:58.936] [optimizer_logger] [trace] Estimated[103] <-> Actual[100]
[2019-10-09 17:20:58.955] [optimizer_logger] [trace] Estimated[10091] <-> Actual[10000]
[2019-10-09 17:20:58.972] [optimizer_logger] [trace] Estimated[955] <-> Actual[1000]
/Users/matt/git/terrier/test/optimizer/hyperloglog_test.cpp:22: Failure
Expected: (estimate) <= (static_cast<double>(actual) * (1 + error)), actual: 113 vs 106.5
Error[0.065000000000000002]: Estimated[113] <-> Actual[100]
[2019-10-09 17:20:59.007] [optimizer_logger] [trace] Estimated[113] <-> Actual[100]
[2019-10-09 17:20:59.025] [optimizer_logger] [trace] Estimated[96493] <-> Actual[100000]
[2019-10-09 17:20:59.077] [optimizer_logger] [trace] Estimated[7428] <-> Actual[10000]
[2019-10-09 17:20:59.077] [optimizer_logger] [trace] Estimated[9949] <-> Actual[10000]
[2019-10-09 17:20:59.078] [optimizer_logger] [trace] Estimated[10068] <-> Actual[10000]
/Users/matt/git/terrier/test/optimizer/hyperloglog_test.cpp:22: Failure
Expected: (estimate) <= (static_cast<double>(actual) * (1 + error)), actual: 10395835 vs 1.0325e+07
Error[0.032500000000000001]: Estimated[10395835] <-> Actual[10000000]
[2019-10-09 17:21:11.370] [optimizer_logger] [trace] Estimated[10395835] <-> Actual[10000000]

Also the CheckErrorsBounds stuff in the test, instead of EXPECT_LE and EXPECT_GE I think we can use a EXPECT_NEAR as seen in https://github.com/google/googletest/blob/master/googletest/docs/advanced.md

This also bumps xxHash to yesterday's release.


Module that may fit node-named users

Hello!
We developed a module that replaces/extends node's dns.lookup method. The main goal is to bypass issues with blocking of thread pool. So module caches responses.
I want to ask you is it possible to add link and short description to our module in your project?
Module was tested in production and highload environments. Also we have good unit and functional tests with 100% coverage, multi-records resolving and TTL support. Here it is: https://github.com/LCMApps/dns-lookup-cache
Thank you for reply!

Cool! This project is specifically for serving DNS requests, not looking them up. I can link to it but they're two separate sides to the DNS equation.