Behavior of wasm_bindgen on non wasm target

Was trying to look up the behavior of wasm_bindgen attribute if i am not using wasm32-unknown-unknown target. But couldnt find in the documentation. Will this be ignored or will some action be done. The intention is to write some code that is common between wasm and mobile.

Not sure about any technical implications such as.. for example.. does it affect the final binary size at all.. but you can still compile to other targets so you can have code that is common between your WASM target and another target.
You'll panic if you try to call any functions that expect a DOM to be there or anything of course. So on mobile you wouldn't be able to call functions that create DOM elements etc.

It sounds like you might want a cargo workspace with a mobile crate and web crate that both use your common functionality crate. If you like your common functionality crate can expose a feature(s) and your web and mobile crate can enable this feature(s) for conditional compilation.
More here https://doc.rust-lang.org/1.9.0/book/conditional-compilation.html#cfg_attr

Thank you. The question was more about common objects(POJO types) to be used by both. Can i used wasm-bindgen on them and expect to work with both.


fix llvm-cov path and add framework coverage

Added framework target handling and fixed the path for llvm-cov missing a /

Thank you @petester42. I'm going to check it out and merge this.

Hi, looks fine to me. I've merged this into the feature-profdata branch, so that it gets automatically referenced into https://github.com/venmo/slather/pull/92.
Thanks!


Task dashboard

Allow the operator/user to visualize the current stack of tasks being worked on by the agent.

This is possible by looking at the interactions table, however there is no information about agents in this context.


emoji: Change rendered emoji image to .png.

Currently, :smile: gets rendered to
, and /static/generated/emoji/images/emoji/smile.png is a symlink to /static/generated/emoji/images/emoji/unicode/1f604.png.
It would be better to instead have :smile: render directly to , which doesn't tie us to our current set of names. This is important for being able to freely add/remove emoji shortcodes, i18n names, etc.
Places to look:

zerver/lib/bugdown.Emoji is where the rendering happens, we just need to change the src variable in that code block.
tools/setup/emoji/build_emoji is where the emoji mapping should be computed (build_emoji is run during provisioning). dump_emojis is the relevant function. I would add a dictionary to emoji_codes_file with the mapping from names to codepoint, that bugdown/ can then read from. Don't worry about the redundancy with the names array for now, or the redundancy with emoji_map.json.

After this change, all the .png symlinks in /static/generated/emoji/images/emoji/ will be obsolete, though we still need to keep them for backwards compatibility with old messages. So let's leave them in place for now.
@HarshitOnGitHub

I am little confused here. Do we have to create a dictionary named emoji_codes_file within dump_emojis or do we have to create a file named emoji_codes_file similar to emoji_map.json?
Thanks

@jajodiaraghav FYI, I am working on this one!!

@HarshitOnGitHub I didn't realize the issue has been assigned to you. Sorry about that. 😄

Generally GitHub should be authoritative on who's working on what; I guess the approach we took of mentioning Harshit was a bit ambigious though; @rishig you may want to take a different strategy next time.
(BTW we're working on a GitHub bot that manages labels to make this a bit easier to deal with)

yeah, definitely my bad; I thought our policy was the "help wanted" means up for grabs (mod claims in comments). Will be more explicit next time!


Wrong cloud-init datasource used. Nodes get stuck at bootstrapping phase.

Describe the bug
My understanding is that the bootstrapping nodes in CAPVCD leverages the VMware GuestInfo Cloud-Init datasource.
However, during our testing we see that VMs cloned from our TKG OVA template (ubuntu-2004-kube-v1.21.14+vmware.2) use the OVF Cloud-Init datasource, causing the nodes to get stuck in the bootstrapping phase.
root@ubuntu-2004-001:~# cloud-init status --long
status: done
time: Tue, 18 Oct 2022 09:42:29 +0000
detail:
DataSourceOVF [seed=com.vmware.guestInfo]
root@ubuntu-2004-001:~# DEBUG_LEVEL=2 DI_LOG=stderr /usr/lib/cloud-init/ds-identify --force
Checking for datasource 'OVF' via 'dscheck_OVF'
Found guestinfo transport.
check for 'OVF' returned found
Checking for datasource 'VMware' via 'dscheck_VMware'
check for 'VMware' returned found
Found 2 datasources found=all: OVF VMware
If we manually disable OVF as a Cloud-Init datasource in the cloned VM and then reboot, we see that the VM is then successfully bootstrapped as expected.
root@demo-work-pool-1-78f97b8f69-2mj82:~# date
Tue 18 Oct 2022 11:49:05 AM UTC
kubectl describe vcdmachines demo-work-pool-1-ptsrt
Last Transition Time:  2022-10-18T11:25:17Z
Status:                True
Type:                  Ready
Last Transition Time:  2022-10-18T11:25:17Z
Status:                True
Type:                  BootstrapExecSucceeded
Last Transition Time:  2022-10-18T09:39:28Z
Status:                True
Type:                  ContainerProvisioned
Reproduction steps

Upload TKG OVA as an OVF template to a Cloud Director Catalog
Deploy a CAPVCD cluster referencing the TKG OVA
Observe that the Cloud-Init datasource used is OVF and not GuestInfo.

Expected behavior

Upload TKG OVA as an OVF template to a Cloud Director Catalog
Deploy a CAPVCD cluster referencing the TKG OVA
Cloned VM uses the GuestInfo Cloud-Init datasource by default so no manual intervention is needed to bootstrap the nodes.

Additional context
No response

This is fixed in CAPVCD 1.0.1.

Closing this issue based on the above comment


Specifications for training/testing in easy mode

I was wondering how the curves of Figure 13 from the paper are obtained. The paper's Appendix D indicates that there is only one worker for the easy mode, but that would only provide us the "Train" results. Do you simply use "mpiexec np 2" and assign the second worker as a test_worker or is there another approach? Thanks.

I believe there is always 1 test worker, so your approach should be fine.

Awesome, thanks @christopherhesse !


Tutorial 2. Data reduction

Tutorial 2. Data reduction
Would you like to participate in the writing of this tutorial?
Or do you have a question about this tutorial?
Let us know here!
Description
This tutorial focus on dimensionality-reduction techniques (PCA, ICA, etc.) that can provide useful data preprocessing when the number of variables exceeds the number of samples.
Useful references

Section 5.2 CCA for neuroscientists: Wang, Hao-Ting, et al. "Finding the needle in a high-dimensional haystack: Canonical correlation analysis for neuroscientists." NeuroImage (2020)

Following #4, I propose we just use simulated data for this issue and issue #8 potentially?
I am happy fore people to take the CCA part and data simulation from here:
https://github.com/htwangtw/cca_primer/blob/master/cca_notebook.ipynb
@diiobo has experience with PLS before! I will let her talk about what she can help here :smile:

Following #4, I propose we just use simulated data for this issue and issue #8 potentially?
I am happy fore people to take the CCA part and data simulation from here:
https://github.com/htwangtw/cca_primer/blob/master/cca_notebook.ipynb
@diiobo has experience with PLS before! I will let her talk about what she can help here 😄

I think it's @fBeyer89 who had experience with PLS (definetly not me =)) )

Damn sorry!!!!!! Jump on issues if people think they can take up something :P

Is this section about using dimension reduction as a preprocessing step for CCA/PLS, or is it meant to also stand alone?

I am not sure if its the best to have this section a standalone - there are many ways to achieve this. You can do it as part of the preprocessing (e.g. Smith 2015) or integration with the CCA/PLS algorithm. I would personally pick a type of workflow to prevent complication.

@fBeyer89 I was thinking of showing the impact of using dimension reduction as a preprocessing step for CCA/PLS in this tutorial, but I am happy to change the roadmap and include that on another tutorial that focus on a specific pipeline! Maybe like the one in Smith 2015 as @htwangtw proposed?

I personally will keep thing simple.


Dwarf unwinding results in truncated stacks

I'm trying to make a replacement for perf script with linux-perf-data, framehop, and  wholesym. Thank you for providing the building blocks!
Currently I'm running into an issue with framehop giving me truncated stacks when using ModuleUnwindData::DebugFrame. It works great with ModuleUnwindData::EhFrameHdrAndEhFrame, which is how it works out of the box for rust programs with samply.
For Rust I'm getting TruncatedStackMarker, but for Go I'm also seeing just short stacks with a couple of frames and that's it.
My reference is perf script linked against libdw (libunwind likes to truncate as well, so I'm avoiding it).
I'm on aarch64-linux (via qemu VM on macOS).

I'm trying to make a replacement for perf script with linux-perf-data, framehop, and wholesym. Thank you for providing the building blocks!

Excellent! This was one of the applications I had in mind when building out these pieces. I wanted a  fast replacement for perf script for the Rust flamegraph utility in particular, but I'm sure it would be useful in lots of other cases, too.
My first attempt at this was in the linux-perf-stuff repo, and the process of writing it informed the API of the wholesym crate. I haven't touched the linux-perf-stuff repo in a while, and you may already have a solution that's further along.
By the way, KernelSymbols::new_from_running_kernel is another piece that you may want to use in your tool, and it's not packaged in a separate crate yet.

Currently I'm running into an issue with framehop giving me truncated stacks when using ModuleUnwindData::DebugFrame. It works great with ModuleUnwindData::EhFrameHdrAndEhFrame, which is how it works out of the box for rust programs with samply.

Hmm, that's interesting. I haven't put the ModuleUnwindData::DebugFrame path through a whole lot of testing; I only really tested it with Go binaries on macOS arm64. It was working mostly fine on those, except that it couldn't unwind from the stack-switching-for-FFI code because of incorrect DWARF CFI on arm64 for those functions, from what I recall.

For Rust I'm getting TruncatedStackMarker, but for Go I'm also seeing just short stacks with a couple of frames and that's it.
My reference is perf script linked against libdw (libunwind likes to truncate as well, so I'm avoiding it).
I'm on aarch64-linux (via qemu VM on macOS).

Okay. So then I'll try to reproduce this by making samply prefer .debug_frame over .eh_frame, and then running samply load perf.data on a perf profile of a Rust binary with debug information. And then once that works I can take a look at the Go case.


[#55] Enhanced Slack notifications from Jenkins for main branches (develop and master) depending on previous and current build status (except successful after successful)

This closes #55: this adds filtering for slack messages (only messages for develop and master branch with R->G, G->R, R->R states would be sent)

@akharlamov title has been updated


GetAccessToken bounces with 'invalid request'

Using the V2 endpoint when trying to get an access token made getAccessToken() bounce with ‘invalid request’.
AAD Sign-in monitor says "Status: Success".
Using a fresh V1.4.2 on PHP 5.6 and using the tag V1.4.2 authorization code flow example.
Below, newlines have been inserted for clarity and the domain name redacted:
Fatal error: Uncaught exception 'League\OAuth2\Client\Provider\Exception\IdentityProviderException' with message 'invalid_request' in [my domain]/vendor/thenetworg/oauth2-azure/src/Provider/Azure.php:314
Stack trace:
#0 [my domain]/vendor/league/oauth2-client/src/Provider/AbstractProvider.php(628): TheNetworg\OAuth2\Client\Provider\Azure->checkResponse(Object(GuzzleHttp\Psr7\Response), Array)
#1 [my domain]/vendor/league/oauth2-client/src/Provider/AbstractProvider.php(537): League\OAuth2\Client\Provider\AbstractProvider->getParsedResponse(Object(GuzzleHttp\Psr7\Request))
#2 [my domain]/vendor/thenetworg/oauth2-azure/src/Provider/Azure.php(58): League\OAuth2\Client\Provider\AbstractProvider->getAccessToken('authorization_c...', Array)
#3 [my domain]oauth2.php(108): TheNetworg\OAuth2\Client\Provider\Azure->getAccessToken('authoriza in [my domain]/vendor/thenetworg/oauth2-azure/src/Provider/Azure.php on line 314
and the URL displayed in the address bar is:
https://[my domain]/oauth2.php?code=0.AR8AGLUax2QgTUuUzRcYxIL_5CChkoZ-J7dLoMiFibsDl3cfALY.AQABAAIAAAAGV_bv21oQQ4ROqh0_1-tA20o0T9ogi0J75vs2tc549OpU7l1MELzeb5PTpg1Ba6pFW9eKG1lgRhlwFDC-L53oMsE-p3NArscwgz2njqiDg6rHzs5cXuFy9CashfViPa8rQNMgJl5f6IC9mq1HGMsoJUm0WdNRFQ1fpXlHriK47Fs9nbUVMacgDaFLD2qtK1hHKZcc5QpUOuL2lEghSQtSfu25U6Agkf5ouIaaV8gx2p08BsHr8h6HC31F0dzjxJO88aJKb_rbeiFVYrsVg4YCkjhapHS_hOBzMtdkTbQK_35BzxWpd62rgG3LyPyWW7qEAuhCrSBhkBPPC28qeaWat7YHFExiKol5PAMoN__B9iz8MMBPO2CgozWVAMSlQxcNwI0zaKyumAoQrUiubhTyKnaQViSdz9-IbWUtvDaPdjJh7nyauLoERLtkPaLLtaprObLFxOtrgy1w7EIXfY66HFMH0pwGcnUyAvLD1UZ-JnIyNTzAf2IOQBou-RuZcX80M_tPshUYnlckUlCuKx9jTwPN97V0KrByvpZg-hCLiQHjqRAIUpPYGjJRxX-fU_39_Z2VFfXjQk7F7nRu7h9Y3Jhk9Wi0AW86-cTo7f0-UyAA
&state=ac2f34739c9dafe563633e016917ff90
&session_state=c2a9e945-4351-46c8-9d54-1326a22dd572
My calling sequence is:
$provider = new TheNetworg\OAuth2\Client\Provider\Azure([
'clientId'          => [my client ID],
'clientSecret'      => [my client secret],
'redirectUri'       => [name of my calling module]
]);
$provider->urlAPI = "https://graph.microsoft.com/";
$provider->resource = "https://graph.microsoft.com/";
$provider->scope = "openid  SMTP.Send Mail.Send offline_access";
[My AAD permissions for the MSFT Graph endpoint specify these same scopes]
I have also trial-edited Azure.php with the following changes but with no improvement:
lines 20 and 22: inserted V2.0/ after /oauth2/
lines 30, 149, 240 and 243: replaced windows.net by microsoft.com
Any suggestions pls?

Hey @decomplexity how did you solve this pls?

To be honest, I cannot remember!
But almost certainly the solution will be in my PHPMAILER/PHPMAILER Wiki Microsoft OAuth2 SMTP issues where I summarised the MSFT auth problems and their resolution that I had encountered when trying (eventually successfully) to get MSFT OAuth2 apps to work reliably. .

@decomplexity was able to solve it, added an extra option to the method
$accessToken = $client->getAccessToken('client_credentials', [
                'scope'    => 'https://graph.microsoft.com/.default',
            ]);


[fix] #1640: Generate genesis.json and consolidate generation into one tool

Description of the Change
Can now use iroha_docs binary to generate sample genesis.json
Issue
Closes #1640
Relates to #1992

Benefits
Can now generate and check if the genesis.json needs updating.
Possible Drawbacks
None
Usage Examples or Tests [optional]
cargo run --bin iroha_docs -- --genesis > /tmp/genesis.json && diff /tmp/genesis.json  configs/peer/genesis.json

I can see, that the test with roles I wrote is stalking your PRs. You haven't changed anything related to it but it's failing. Is this test flaky?


Showcase Submission for code/6-objects/3-constructor

Thank you Nathan Rudasill for your contribution! A member of the Coding Train team will review it shortly.

Sudoku Solver
Nathan Rudasill

Wow! I would love to do this as a coding challenge! I'm happy to merge this here but I wonder if it might make sense to add to https://thecodingtrain.com/challenges/171-wave-function-collapse instead?

If anyone wants to help move this showcase to WFC, please go ahead!

I have moved to WFC

Thank you @kfahn22!


After the training, the following error occured.

I user my own dataset.

Thank you. I have modified this before, but this error disappeared in the new training.


Using Scikit-learn APIs directly

Almost yesterday (August 5 at 7 PM, Iran/Tehran time), I had a short conversation with @sonichi about this, and in general it is better to provide such features more easily to the users... Maybe you (FLAML maintainers) don't have any contest, but you should have features so that more developers will use your product...
Anyway, the files I imported from them:

Pre-processing: https://github.com/scikit-learn/scikit-learn/blob/main/sklearn/preprocessing/init.py
Model selection: https://github.com/scikit-learn/scikit-learn/blob/main/sklearn/model_selection/init.py
Metrics: https://github.com/scikit-learn/scikit-learn/blob/main/sklearn/metrics/init.py

I don't think that there is a need to write a test, but we should be careful that a method (class and/or function) is not deprecated...
A quick example that shows how to use these APIs:
import pandas as pd
from flaml.utils import preprocessing
from flaml.utils import model_selection
from flaml.utils import metrics
from flaml import AutoML

# Loading data (nothing changed)
df = pd.read_csv('<a_random_dataset_that_needs_preprocessing.csv>')
X = df[['field_no1', 'field_no2', 'field_no3', 'field_no4']]
y = df['field_no5']

# Preprocessing
le = preprocessing.LabelEncoder()
X['field_no3'] = le.fit_transform(X['field_no3'])
y['field_no5'] = le.fit_transform(X['field_no5'])

# Seperating the train and test data
X_train, y_train, X_test, y_test = model_selection.train_test_split(X, y, test_size=.2)

# Training phase (nothing changed)
automl = AutoML()
automl.fit(X_train, y_train, task='classification')

# Measuring accuracy
y_pred = automl.predict(X_test)
print(metrics.classification_report(y_test, y_pred))

Or:
from flaml.utils import (
    LabelEncoder,
    train_test_split,
    classification_report,
)
from flaml import AutoML

Or even:
from flaml import (
    LabelEncoder,
    train_test_split,
    classification_report,
    AutoML,
)

Dude, contributing to Microsoft projects is really painful! :))

Dude, contributing to Microsoft projects is really painful! :))

The CLA signing is a one-time procedure. You won't need it for every PR.
Some checks failed: https://github.com/microsoft/FLAML/runs/7702402957?check_suite_focus=true#step:5:51

@sonichi Looks like we need to upgrade to a higher version of Sklearn (1.1.2 based on PyPI) Do you know any other solution, except removing the items that give errors?!

@sonichi I don't have enough experience in such situations, can you help?

@sonichi Looks like we need to upgrade to a higher version of Sklearn (1.1.2 based on PyPI) Do you know any other solution, except removing the items that give errors?!

I would suggest using version check of sklearn to decide what to import. Only import what is available. Do not increase the sklearn version in setup.py.

@sheikhartin I figure you and @sonichi already discussed this in your conversation, but any chance you could briefly explain here the benefit of adding these? Assuming a user installed flaml (and should therefore also have a compliant version of sklearn), any utilities provided by sklearn are already available to the user for import and usage.

Very good question @ZviBaratz! But, you should ask your question like this, why did PyCaret 🥕 become so famous and popular? In short, because it literally automates the process!
I just wanted to tell the FLAML developers that the game is not over, and simplicity does not necessarily mean weakness! I hope you understand that my purpose of doing this contribution was not to get the logo of the Microsoft organization on my profile, and I was thinking about the real growth of this project...
Anyway, you can easily block this PR... I will not be upset at all 🙂🧡

IMHO this addition does not improve existing functionality and mostly clutters up the module's namespace.

I'm sorry @ZviBaratz, I may have explained a little badly, I didn't mean to offend you or other contributors! 🙏 You are right, the functionality does not improve, maybe this PR was a mistake on my part as I thought it would help attract more people...

@sheikhartin no offence taken, and I hope it's clear I wasn't trying to be offensive either. I think the discussion regarding the differences between pycaret and flaml is somewhat out of scope, but in the context of this PR my opinion is that enabling users to import unmodified sklearn code from flaml will not constitute an attractive feature or promote the usage of flaml over other AutoML solutions in any way.
In any case, the thought and the effort are appreciated :pray:

I agree with @ZviBaratz here that this improves any existing functionality. It doesn't seem like the imported utils are used anywhere throughout the project so why is there a need to import them? I think it would be best for us to import only what we need
For example, will there be a use for
from ._search import GridSearchCV
from ._search import RandomizedSearchCV
from ._search import ParameterGrid
from ._search import ParameterSampler

Guys, can you delete this PR completely? Thank you, I learned a lot, but I don't want to leave any traces of this 🙏
@sonichi @ZviBaratz


Create jobs for devices for notifications

So we can keep track if a particular device failed in sending notification

merged here 825b1ffca6f068bf039d816e4ee6b2289a569bf0


On Android, services-speech-sdk SPXERR_AUDIO_SYS_LIBRARY_NOT_FOUND

Tested versions
godot4 dev6 c#
System information
android 11 api 30
Issue description
On Android, I have already granted the recording permission. When I use the cognitive-services-speech-sdk for recording, I encounter the error code: 0x38 (SPXERR_AUDIO_SYS_LIBRARY_NOT_FOUND). How can I resolve this issue? Is speechsdk not supported, or is there another reason?
Steps to reproduce
SPXERR_AUDIO_SYS_LIBRARY_NOT_FOUND
Minimal reproduction project (MRP)
    public async Task<PronunciationAssessmentResult> FromMic(SpeechConfig speechConfig)
    {
        var pronunciationAssessmentConfig = new PronunciationAssessmentConfig(
referenceText: "I am tom.",
gradingSystem: GradingSystem.HundredMark,

granularity: Granularity.Word,
enableMiscue: true);
        pronunciationAssessmentConfig.PhonemeAlphabet = "IPA";
        pronunciationAssessmentConfig.NBestPhonemeCount = 5;

        speechConfig.SetProperty(PropertyId.Speech_SegmentationSilenceTimeoutMs,3000.ToString());

        using var audioConfig = AudioConfig.FromDefaultMicrophoneInput();
        using (var speechRecognizer = new SpeechRecognizer(
        speechConfig,
audioConfig))
        {
            pronunciationAssessmentConfig.ApplyTo(speechRecognizer);

            var speechRecognitionResult = await speechRecognizer.RecognizeOnceAsync();

            // The pronunciation assessment result as a Speech SDK object
            var pronunciationAssessmentResult =
                PronunciationAssessmentResult.FromResult(speechRecognitionResult);

            // The pronunciation assessment result as a JSON string
            var pronunciationAssessmentResultJson = speechRecognitionResult.Properties.GetProperty(PropertyId.SpeechServiceResponse_JsonResult);
            return pronunciationAssessmentResult;
        }
    }

You used "godot4 dev6 c#"? That's quite old and not even a stable version... Could you try with Godot 4.2.2 or 4.3, please?

TT1.zip
godot 4.4 dev1


Docs/update

GitHub does not make commit conform to conventional commits, so I had to amend the commit message.


Adds optional flag to allow partial results when querying for jobs

We want to be able to query a particular cook cluster for a set of uuids, where some of them may not belong to that cluster, and get back the data for those that do match. This change allows for that, instead of always returning 404 if any of the uuids is invalid.

Passing build: https://travis-ci.org/twosigma/Cook/builds/229260098

@shamsimam Any more feedback on this?

Can we merge this?

@shamsimam @pschorf Any more feedback on this?


Describe the bug
EntityManager.populate does not work with lazy scalar properties.
To Reproduce
import { Entity, JsonType, MikroORM, PrimaryKey, Property } from '@mikro-orm/core';

@Entity()
class A {

  @PrimaryKey()
  id!: number;

  @Property({ type: JsonType, lazy:true })
  lazyProp = {
    a: 0,
    b: 1,
    c: 2,
    d: 3,
  };

}

describe('GH issue 1479', () => {

  let orm: MikroORM;

  beforeAll(async () => {
    orm = await MikroORM.init({
      entities: [A],
      dbName: `:memory:`,
      type: 'sqlite',
    });
    await orm.getSchemaGenerator().createSchema();
  });

  afterAll(() => orm.close(true));

  test(`GH issue 1479`, async () => {
    const fixture = new A();
    await orm.em.persistAndFlush(fixture);
    orm.em.clear();

    const found = await orm.em.findOneOrFail(A, fixture.id);
    const canPopulate = orm.em.canPopulate(A, 'lazyProp');

    await orm.em.populate(found, ['lazyProp'], undefined, undefined, true); // refresh: true

    expect(canPopulate).toBe(true);
    expect(found.lazyProp).toEqual({ a: 0, b: 1, c: 2, d: 3 });
  });
});

Expected behavior
EntityManager.populate should load lazy properties the same way EntityManager.find does. Or it should fail the same way it does for non-populatable props.
Additional context
Mikro-orm explicitly ignores the loading of the lazy properties.
Versions

Dependency
Version

node
v15.9.0

typescript
^4.1.5

mikro-orm
master

your-driver
?

Can confirm. Using version 4.4.4. Seems to be only findOne() issue though, cannot reproduce with find().
What's more on lazy properties... when you define the fields to select (including lazy prop), then MySQL gives unknown column error. So there's no way to query for say ID and a lazy property only.

I don't really see this as a bug - em.populate is about loading relations, and lazy property is not a relation - populating it means reloading the entity from the first parameter - which is not what em.populate does, it only loads its relation properties.
But agreed this should be working somehow, when using refresh: true (that will be probably required).

What's more on lazy properties... when you define the fields to select (including lazy prop), then MySQL gives unknown column error. So there's no way to query for say ID and a lazy property only.

Please provide repro for that.

I think where this would be especially useful is when using something like graphql-fields-to-relations's fieldsToRelations.
When someone makes a graphql query:
query {
  myEntity {
	myLazyEntity { id other }
    myNormalField { id other } // e.g., a json column?
  }
}

fieldsToRelations will turn that into a list of
['myLazyEntity', 'myLazyEntity.id', 'myLazyEntity.other', 'myNormalField', 'myNormalField.id', 'myNormalField.other']
Which will cause an error because myNormalField is just a regular column. It would nice if those fields were just ignored.

1、find/findOne can populate: ['lazy'] but can't use em. populate sounds wired. they all are populate right?
2、I'm enter the issue when i'm using graphql
schema:
@Entity()
class Article {
  @PrimaryKey()
  id!: number;

  @Property({ lazy:true })
  lazyProp: string;
}

graphql
type Mutation {
  saveArticle(data: SaveArticleInput!): Article!
}

type Article {
  id: String!
  lazyProp: String!
}

input SaveArticleInput {
  id: String
  lazyProp: String
}

I use em.flush() before response and lazyProp is a field resolver
The life cycle:
user request -> mtation saveArticle(update or insert) -> filed resolver lazyProp -> em.flush() -> response

The issues:

If i load content via find/findOne with populate: ['lazyProp'], em.flush() will not update/insert any thing, because Article is refreshed.
Only when requested graphql query has lazyProp field then server should populate lazyProp
So there is no way i process lazyProp in my field resolver, I can only process it in my query, and code is ugly.
Even em.populate can't populate lazy fields, it must has some other method to do the same thing.

I personally didn't understand much, but perhaps you could provide more detailed output of the code you're running.


Update Tox file solving Travis CI broken tests

Today, the Travis CI tests are broken. This commit solves this, updating the tox.ini file and solve some linter issues.
Fix #59

Waiting if PR https://github.com/kytos/storehouse/pull/52 already solve this problem.


ERR_EMPTY_RESPONSE

We've installed QMC Utilities as described - service is running, manually added qmculogin.html to content library, path to certificates as described.
However there is no response from QMC Utiilities https://localhost:9945/qmcu/ - after a long time waiting we got ERR_EMPTY_RESPONSE error.
upd. Found a cause. We use a non-standard port for https in QMC proxy. If change to default - QMCU works. Could you check if it's possible to add parameter for https or read it from QMC proxy settings?
Or as a temporary solution - which line to correct in source code?

@goldbergjeffrey Where can we hardcode a different port for the QMC? At least middleware/auth.js:69, but possibly more locations.

ok, I get it.  Now I see why auth.js.  Let me have a look and I'll post a reply.


create: remove :x11 dep from new formula template

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew tests with your changes locally?

IMO brew create mostly caters to core, and :x11 dep isn't something we encourage in core. Here's a list of core formulae with an :x11 dep, along with the commit and date they were introduced:

List
$ grep -rl :x11 Formula | while read f; do f=$f:r:t; printf '%s: ' $f; brew log --pretty='format:%h: %ci' $f | tail -1; print; done
at-spi2-core: 7ecda193fa: 2013-02-02 18:47:06 -0600
cairo: b901cd391c: 2010-04-27 09:34:30 -0700
diff-pdf: 36257d1eb7: 2014-05-18 23:23:45 -0500
ghostscript: bf7a82e66d: 2009-09-10 19:23:03 +0100
giflib: 2201bf8b23: 2009-11-08 15:21:14 +0000
giflossy: 9aee557512: 2015-06-18 02:18:15 +0100
gifsicle: e990d03870: 2010-05-01 09:41:07 -0700
gnu-smalltalk: 76de754e70: 2009-09-19 11:35:43 +0200
gnuplot: b1679ccfff: 2009-09-22 19:56:22 +0100
goocanvas: ca086fcfd1: 2013-08-05 21:50:19 -0700
gource: 4c23d4c47f: 2010-05-08 10:46:59 -0700
gpac: 150cac8cb0: 2010-08-12 07:42:04 -0700
graphicsmagick: fa2a4c818c: 2009-11-07 18:22:36 +0000
graphviz: b8f509f85a: 2009-09-29 23:52:17 +0100
gst-plugins-good: 6b7bbf0e4d: 2010-08-14 16:16:31 -0700
harbour: a44a1dc048: 2014-08-16 19:20:20 -0700
imagemagick: 4018e05d35: 2009-06-28 16:56:15 +0100
imlib2: a6eb8c316e: 2010-10-20 21:30:20 -0700
jed: eb204999b2: 2010-07-14 07:20:41 -0700
lesstif: 6ea6ca3a0a: 2010-09-27 09:24:02 -0700
libcaca: a8810fdc99: 2010-04-10 17:40:51 +0200
libspnav: 28f05afd48: 2015-08-14 21:42:00 +0100
links: b562b5e48b: 2009-09-25 00:00:55 -0400
lsh: 59d13f3f23: 2014-11-30 17:04:40 +0000
mal4s: 72e74dfbad: 2014-01-05 14:16:16 +0000
mit-scheme: 3bf0dbb1f6: 2011-04-14 13:58:54 -0700
mjpegtools: a93f067f67: 2011-10-27 17:27:36 -0700
modules: 9c0cc2a53b: 2014-04-17 05:43:09 -0700
mpv: 2d56414704: 2015-12-26 21:47:14 +0000
mtools: ee4c35199d: 2011-12-30 20:20:41 -0600
ngspice: 6de6676db1: 2010-10-31 16:44:12 -0700
ocaml: bf8cc3aa37: 2015-08-14 08:42:07 +0100
ode: cb6a4298ab: 2010-04-23 21:21:38 -0700
pango: 2c3ddb15fe: 2010-02-07 12:18:10 -0800
pike: b0093804dd: 2012-11-11 16:58:22 -0800
plan9port: e343b98e78: 2014-02-02 10:07:29 +0000
plotutils: 568d4adf63: 2010-10-30 18:25:02 -0700
plplot: abbb233139: 2013-01-04 11:43:39 -0800
python: 16727d3937: 2009-07-31 00:28:38 +0100
python3: 77a8b25638: 2010-07-09 20:38:14 -0700
ruby: 4f0d14b994: 2009-09-14 20:33:46 +0100
sdl: 9faf689de1: 2009-08-31 01:46:19 +0100
suil: 7d1206acd9: 2015-12-12 14:04:48 +0100
swftools: 06b5f23927: 2009-11-07 18:22:35 +0000
swi-prolog: 7e8a284932: 2010-01-30 14:18:04 +0100
tesseract: 67fe99f417: 2010-01-30 14:11:29 +0100
vim: 8319b6db13: 2012-10-19 19:07:59 -0700
wine: 6570815c23: 2009-10-04 18:36:44 +0100
wordnet: dd98285287: 2009-10-12 17:26:44 +0100
x11vnc: be23806907: 2013-11-16 11:06:51 -0800
xplanet: 6b9c5fd1b8: 2012-02-20 19:30:18 -0800
zbar: b53133e37a: 2010-06-18 10:49:40 -0700

As you can see, not a single :x11 formula was added to core in 2016. Therefore, :x11 no longer deserves a special place in the new formula template.
(Even outside core, I'm not convinced that :x11 is more important than, say, :java.)

I agree here. X11 is kind of a legacy technology, and I wouldn't expect many new :x11-using formulae to be created from now on.

X11 is kind of a legacy technology

Yeah, even the Linux world is moving away from X11.

Good call 👍


Refactor database access code to provide a central access point, abstracting underlying operations

Code completed, pending code review


not receiving link to reset password

This is a template, please remove any non-relevant details to your issue
If you are competition participant:

This Github repository is about the platform itself.
If you problem is specific to a competition, please contact directly its organizers.

If you are having trouble using the site:

What browser and version are you using?
What is the URL of the problem? Codalab is an open source project, we may not be supporting the instance you are using!

Fix soon to be merged #1615


Deprecate.dd: clear error column for sort/remove deprecation

Follow-up to https://github.com/dlang/dlang.org/pull/1672

@CyberShadow "auto-merge" requires a formal "approval" ...

I blame https://status.github.com/ :)


Fix GP unit test

Summary: Turning off fast computations in BoTorch broke this test. The MVN covar had 1 more batch dimension than loc, which the approximate log_prob was happy with but the exact log_prob did not like.
Reviewed By: Balandat
Differential Revision: D42194245

already committed


Not more than 60leds are glowing even if programmed for 256 leds.

Have you tried changing it in both the python and the Arduino/pi?
What platform are you using?

Closing due to inactivity


AddedIntakeSubsystem

shouldn't mock intake go in tests/java/frc/robot/mocks/subsystems directory?

I'll fix this when I refactor the code. I think I'll move the existing mock code into the main code, so we can use the mocks with F5 run as well as unit testing.


Embedded go_proto_library not being indexed

Description of the bug:
In Bazel plugin in Goland, when go_library refers to embedded go_proto_library, source files in the go_library cannot refer to definitions in the go_proto_library.
What's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.
For this snippet of code, IDE should index translators.go to be able to refer to types declared in fooproto_go_proto. Please see the repro in the attached PR.
go_proto_library(
    name = "fooproto_go_proto",
    compilers = ["@io_bazel_rules_go//proto:go_grpc"],
    importpath = "github.com/bazelbuild/intellij/examples/go/with_proto/proto",
    proto = ":fooproto_proto",
    visibility = ["//visibility:public"],
)

go_library(
    name = "proto",
    srcs = ["translators.go"],
    embed = [":fooproto_go_proto"],
    importpath = "github.com/bazelbuild/intellij/examples/go/with_proto/proto",
    visibility = ["//visibility:public"],
)

Which Intellij IDE are you using? Please provide the specific version.
2023.2
What programming languages and tools are you using? Please provide specific versions.
Goland, golang
What Bazel plugin version are you using?
Version 2<IP_ADDRESS>.1-api-version-233
Have you found anything relevant by searching the web?
No response
Any other information, logs, or outputs that you want to share?
No response

@blorente Can you review this diff? https://github.com/bazelbuild/intellij/pull/5567 We're very interested in getting this patched and unblock developers.

@tingilee Reviewed the diff, left some comments. Thanks for the contribution!


Update curl to 7.87.0

Click to expand!
Issue Type
Feature Request
Have you reproduced the bug with TF nightly?
No
Source
source
Tensorflow Version
2.11.0
Custom Code
No
OS Platform and Distribution
Windows
Mobile device
No response
Python version
3.10
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
Security Vulnerabilities fixed in curl 7.87.0

Standalone code to reproduce the issue
NA

Relevant log output
No response

Curl has already been updated to 7.87.0 in master branch: https://github.com/tensorflow/tensorflow/pull/59164

Closing as part of #59164. Thank you @yongtang !


Uncaught Error: tunneling socket could not be established, cause=getaddrinfo ENOTFOUND proxy.repl...

[Enter steps to reproduce:]

...
...

Atom: 1.24.0 x64
Electron: 1.6.16
OS: Mac OS X 10.13.3
Thrown From: Atom Core
Stack Trace
Uncaught Error: tunneling socket could not be established, cause=getaddrinfo ENOTFOUND proxy.reply.it proxy.reply.it:8080
At stream.js:74

Error: tunneling socket could not be established, cause=getaddrinfo ENOTFOUND proxy.reply.it proxy.reply.it:8080
    at ClientRequest.onError (/app.asar/node_modules/tunnel-agent/index.js:177:17)
    at Object.onceWrapper (events.js:290:19)
    at emitOne (events.js:96:13)
    at ClientRequest.emit (events.js:188:7)
    at Socket.socketErrorListener (_http_client.js:309:9)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:188:7)
    at connectErrorNT (net.js:1025:8)
    at _combinedTickCallback (internal/process/next_tick.js:74:11)
    at process._tickCallback (internal/process/next_tick.js:98:9)

Commands
     -2:24 core:save (input.hidden-input)
  2x -1:55.1.0 core:backspace (input.hidden-input)
  2x -1:53.9.0 core:confirm (input.hidden-input)
  2x -1:47.8.0 core:backspace (input.hidden-input)
  2x -1:46.9.0 core:confirm (input.hidden-input)
  2x -0:53.6.0 core:backspace (input.hidden-input)
  2x -0:52.2.0 core:confirm (input.hidden-input)

Non-Core Packages
atom-jade 0.3.0
busy-signal 1.4.3
intentions 1.1.5
language-javascript-jsx 0.3.7
linter 2.2.0
linter-js-standard 4.2.0
linter-jsonlint 1.3.0
linter-ui-default 1.6.10
pigments 0.40.2
project-manager 3.3.5

Thanks for the report - the error seems like a networking/DNS issue rather than an issue with Atom so we'll go ahead and close the issue.
Is the error reproducible for you?


[Bug]: YOUTUBE_API_KEY not found - Different key names

What happened?
I had a problem using YT Helper because there's a mix-up in the names. YT is trying to find YOUTUBE_API_KEY, but the utils.py writes CLOUD_API_KEY in the configuration file. Also when using fabric --setup, it doesn't put a new line after OPENAI_API_KEY=someKey.
Version check

[X] Yes I was.

Relevant log output
`yt 'https://www.youtube.com/watch?v=someId'
Error: YOUTUBE_API_KEY not found in ~/.config/fabric/.env`

Relevant screenshots (optional)
No response

We're working on this one, thank you.


newest release and source code are different, can you update?

newest release and source code are different, can you update?

https://github.com/projectdiscovery/fastdialer/releases/tag/v0.0.16


WebSocket is already in CLOSING or CLOSED state. ERROR

I'm faced with new error a few hours later. If I open my website, then connection will open. But If I switch tabs in my browser after 5 minutes, I'll get this error WebSocket is already in CLOSING or CLOSED state in my webbrowser's console and lose connection
I'm surfed the Internet and found a solution, that I should set PingTimeOut option on a client side. But how can I do this...
Also I seen Timer in the Workerman, but I'm doubt about relating this to my issue, because no information.
Similar Issue https://github.com/socketio/socket.io/issues/3259

Some gateway, routing nodes, such as firewalls will close the TCP connection that does not communicate for a long time(for example, 60 seconds), which leads to application communication failure. So we need to add a heartbeat to the application layer to keep the connection from being disconnected. In JS, setInterval() can be used to send some data to the server regularly through the connection, such as the following.
ws = new WebSocket('ws://example.com');
ws.onopen = function () {
    var t = setInterval(function(){
        if (ws.readyState != 1) {
            clearInterval(t);
            return;
        }
        ws.send('{type:"ping"}');
}, 55000);};

When the server receives heartbeat data, it can ignore it and do nothing. Heartbeat interval less than 60 seconds is better, such as 55 seconds.

Some gateway, routing nodes, such as firewalls will close the TCP connection that does not communicate for a long time(for example, 60 seconds), which leads to application communication failure. So we need to add a heartbeat to the application layer to keep the connection from being disconnected. In JS, setInterval() can be used to send some data to the server regularly through the connection, such as the following.
ws = new WebSocket('ws://example.com');
ws.onopen = function () {
    var t = setInterval(function(){
        if (ws.readyState != 1) {
            clearInterval(t);
            return;
        }
        ws.send('{type:"ping"}');
}, 55000);};

When the server receives heartbeat data, it can ignore it and do nothing. Heartbeat interval less than 60 seconds is better, such as 55 seconds.

May be It will help. But If I open my website(websocket) in the same browser with two similar tabs and start clicking, then I'll get this error too. I changed interval for a 5 seconds and it didn't affect. WebSocket is already in CLOSING or CLOSED state

May be It will help. But If I open my website(websocket) in the same browser with two similar tabs and start clicking, then I'll get this error too. I changed interval for a 5 seconds and it didn't affect. WebSocket is already in CLOSING or CLOSED state

WebSocket is already in CLOSING or CLOSED state means that the connection has been disconnected and data cannot be sent through this connection. In addition to the lack of heartbeat and being disconnected by the network node, it is also possible that the client or server called the close method, and there may be a bug in the business code.

WebSocket is already in CLOSING or CLOSED state means that the connection has been disconnected and data cannot be sent through this connection. In addition to the lack of heartbeat and being disconnected by the network node, it is also possible that the client or server called the close method, and there may be a bug in the business code.

The problem is not with connection already. The problem with multiple tabs. When 1 user open 15 tabs, then connection will lose. I found a answer on stackoverflow about this issue. Link https://stackoverflow.com/questions/59176767/socket-io-with-multiple-tabs-not-working-and-when-refreshing-the-page
I check connection variable and found out, that workerId has notunique id. And it possible affected on this error.

The problem is not with connection already. The problem with multiple tabs. When 1 user open 15 tabs, then connection will lose. I found a answer on stackoverflow about this issue. Link https://stackoverflow.com/questions/59176767/socket-io-with-multiple-tabs-not-working-and-when-refreshing-the-page
I check connection variable and found out, that workerId has notunique id. And it possible affected on this error.

Is the problem solved?

Is the problem solved?

Yes, the problem was with my code

Yes, the problem was with my code

please my webterm shows disconnected...terminado and goes on to show websocket closed, how do i fix this?

please my webterm shows disconnected...terminado and goes on to show websocket closed, how do i fix this?

Get sample from repository
Don't use SESSION
Check SQL queries
Check logs errors

how may you help me who has just started a software engineering course and i still don't know how to write a "c" or "js" script

unable access git hub resources,. its said disconnected or closed
[WebSocket is already in CLOSING or CLOSED state.]
[WebSocket is already in CLOSING or CLOSED state.]

how do i get back to where i stopped when the web socket closes automatically?


Having to add resolve node packages to avoid polyfill?

Hi,
I had to add buffer and other node packages to resolve otherwise it was converting it to ArrayBuffer. I use node libraries in the renderer.
 renderer: {
              resolve() {
                  return [
                      "bw-casclib", "events", "util", "fs/promises", "stream", "buffer", "string_decoder"
                  ]
              }
          },

I find my case to be strange because I believe vite no longer automatically polyfills, so I must be doing something wrong? I'm additionally getting issues around this kind of thing and I'm not certain why.

Above I will also now need to include url and zlib to the resolves. Is this as designed or am I doing something wrong?

seems like I needed the esmodule but I don't know why. probably just for the last one libnpmsearch.
        esmodule(["minipass-fetch", "make-fetch-happen", "libnpmsearch"]),

Can you provide a minimal reproduction Demo?

I will get one up


os.capture

Is your feature request related to a problem? Please describe.
I'm porting over a CMake file for a Node Addon.
In CMake I can reliably get the include directory using require("node-addon-api").include which is the suggested method.
I'm not sure if I can do this in xmake
Describe the solution you'd like
Could this be solved by adding os.capture to the description scope?
Describe alternatives you've considered
No response
Additional context
No response

please get it in script scode.
target("xxx")
    on_load(function (target)
         local includedirs = target:get("includedirs")
    end)

add_requires("zlib")
target("xxx")
    add_packages("zlib")
    on_load(function (target)
         local includedirs = target:pkg("zlib"):get("includedirs")
    end)

I'm trying to get it programatically (as node-gyp and cmake-js do e.g. here, since the project is dependent on node.js
I have node and the node-addon-api npm package  in my path,
in my xmake.lua, I can call and get output fine when I run this on windows:
on_load(function(target)
    os.execv('node', { "-p", '[...require("node-addon-api").include].filter(ch => ![10,13,34].includes(ch.charCodeAt())).join("")' })
end)

using the same call that the standard tool for Node C++ Addons uses to find the include dir
it will output the location of the npm package the node instance is using: PROJECT_DIR\node_modules\node-addon-api
which contains the headers I need.
on_load(function(target)
 local node_install_dir = os.capture('node', { "-p", '[...require("node-addon-api").include].filter(ch => ![10,13,34].includes(ch.charCodeAt())).join("")' })
 print(node_install_dir)
--- include somehow
end)

I think you're looking for os.iorunv.
example with git:
	before_build(function (target)
		local branch = "unknown-branch"
		local commitHash = "unknown-commit"
		try
		{
			function ()
				import("detect.tools.find_git")
				local git = find_git()
				if (git) then
					branch = os.iorunv(git, {"rev-parse", "--abbrev-ref", "HEAD"}):trim()
					commitHash = os.iorunv(git, {"describe", "--tags", "--long"}):trim()
				else
					error("git not found")
				end
			end,

			catch
			{
				function (err)
					print(string.format("Failed to retrieve git data: %s", err))
				end
			}
		}
-- ...

You can use os.iorunv instead of os.execv . https://xmake.io/#/manual/builtin_modules?id=osiorunv

That works @SirLynix @waruqi!

Now I just need to add to includedirs in the script scope

You can use target:add("includedirs", include_dir)


Cleaner closure of a multi-serial protocols communication.

Is your feature request related to a problem? Please describe.
On the file /src/robot/Communication/Protocol.hpp, function void recv_frame(const atomic_bool&, function<void(GlobalFrame)>: the ETHERNET communication creates N detached threads on all the lifetime of the communication.
At the end, when we want to stop it, the atomic_bool& is set to false and threads would exit, but only when they receive a frame on the associate socket (in order to exit the UDP blocking recv_from function).
In order to avoid sending an ampty frame to all UDP connections but only one, a for(thread t : pool) { pthread_cancel(t); } loop is used at the very end of the program; it isn't a dangerous feature because it does not compromise the communication itself or the core.
I would like a better ending function to stop all threads properly.
Describe the solution you'd like

Create N receiving threads
When a reception ends, a AbstractSerialProtocol<P>::ReceptionAborted exception is thrown so we need to catch it (maybe using the std::exception_ptr object
Send an empty frame at all the other working threads to force them exit their receiving blocking function

Have you tried taking a look at promise and futures ? There is an example in the repo for the Can Interface application. See CanListener::mainLoop.


Memory leak in ubuntu 20.04 on 1.60

 yes
Prerequisites

[x] Put an X between the brackets on this line if you have done all of the following:
[x[ * Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
[x]* Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
[x]* Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
[x]* Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
[x]* Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
 atom seems to have a memory leak in 20.04
Steps to Reproduce

 open atom and watch on top/htop/btop/bpytop/the new btop as it slowly climbs in ram

Expected behavior:
 atom to not take stupid levels of ram on the render thread and app asar thread
Actual behavior:
 after only 5 minutes of running atom takes up 18 gigs on the render thread and 4 gigs on the app asr thread
Reproduces how often:
 always, though values might be slightly higher or lower not to sure
Versions

Atom    : 1.60.0
Electron: 9.4.4
Chrome  : 83.0.4103.122
Node    : 12.14.1
ubuntu 20.04.4
kernal 5.13.0-35

it grows at roughly 13 Mega Bytes a second. roughly, going up and down periodicly


Release 1.1.0

This is a draft of changelog for the 1.1.0 release.
Here you can find all related translations changes: 1.0.9...main.

⚠ Do not merge this pull request manually ⚠
You should approve this pull request instead. After approving, I will automatically update it (release date needs to be adjusted), merge, and tag a new release.

Show release notes preview
1.1.0

General changes:

Updated Flarum core translations.

Added support for new extensions:

archlinux-de/flarum-click-image
clarkwinkelmann/flarum-ext-formatted-banner
flarum/nicknames
migratetoflarum/fake-data
v17development/flarum-support

Updated translations for extensions:

flarum/emoji
flarum/flags
flarum/markdown
flarum/pusher
flarum/subscriptions
flarum/suspend
flarum/tags

All changes: 1.0.9...1.1.0.

@ram0ng1 Just as a note, this is an automated PR generated by bot. The idea is to simplify release process for language pack maintainers - all you need is to approve this pull request and bot will handle the rest. You can read more about the process at forum: https://discuss.flarum.org/d/20807-simplify-translation-process-with-weblate/76 . If you don't want to use this feature, let me know.

Success! Now you can announce new release on forum:
## Version [`1.1.0`](https://github.com/flarum-lang/brazilian/releases/tag/1.1.0)

**General changes**:

* Updated Flarum core translations.

**Added support for new extensions**:

* [`archlinux-de/flarum-click-image`](https://github.com/archlinux-de/flarum-click-image)
* [`clarkwinkelmann/flarum-ext-formatted-banner`](https://github.com/clarkwinkelmann/flarum-ext-formatted-banner)
* [`flarum/nicknames`](https://github.com/flarum/nicknames)
* [`migratetoflarum/fake-data`](https://github.com/migratetoflarum/fake-data)
* [`v17development/flarum-support`](https://extiverse.com/extension/v17development/flarum-support)

**Updated translations for extensions**:

* [`flarum/emoji`](https://github.com/flarum/emoji)
* [`flarum/flags`](https://github.com/flarum/flags)
* [`flarum/markdown`](https://github.com/flarum/markdown)
* [`flarum/pusher`](https://github.com/flarum/pusher)
* [`flarum/subscriptions`](https://github.com/flarum/subscriptions)
* [`flarum/suspend`](https://github.com/flarum/suspend)
* [`flarum/tags`](https://github.com/flarum/tags)

All changes: [1.0.9...1.1.0](https://github.com/flarum-lang/brazilian/compare/1.0.9...1.1.0).

In order to update run these commands:

```console
composer update flarum-lang/brazilian
php flarum cache:clear
```


ck-list add @ link item select dropdown shows no highlight

First of all, thanks for making this theme. The default Dark theme wasn't polished enough for me and your theme was exactly what I wanted.
Just pointing a CSS you might have overlooked. The @ add link list isn't showing a highlight when you go through the list with the keyboard.
--active-item-background-color: #033c5;
should be:
    --active-item-background-color: #0784B5;
To make it look like this:

Thanks again for your work on this theme.

@code-ink-space An issue report bundled with a fix, thank you very much. I'm glad you like it and appreciate the issue.
Yeah I don't use this feature (i should start!) so missed this. Was afraid their might be missing coverage due to features I don't use. Fixed in latest, added some extra style on top to match the rest of Midnight.

Oh, and well developing this I also found a bug in the dropdown in question and reported it to Trilium:
https://github.com/zadam/trilium/issues/2703


spec: Define the property-specification syntax

And add platform annotations.
By formally defining our syntax, we do a better job of making that syntax enforcable.  We can also make platform-scoping very obvious, and no longer need to rely on “For Linux-based systems the process supports…” guards and similar.
The only intentional semantic change is that we now allow, platform-named properties that don't match platform.os.  For example:
{
  "platform": {
    "os": "linux",
    …
  },
  "windows": {
    …
  },
  …
}

and even:
{
  "platform": {
    "os": "linux",
    …
  },
  "windows": "foo",
  …
}

are both legal now.  This rolls back the “MUST NOT be set otherwise” requirements which had landed in #673 to make those properties consistent with our general:

Implementations that are reading/processing this configuration file MUST NOT generate an error if they encounter an unknown property.

and:

The state MAY include additional properties.

extensibility policies.  The alternative would be cross-platform specification of those properties which then forbid the properties on non-matching platforms, but then the cross-platform definition would not match the platform-specific Go type.  For example:
Linux *Linux `json:"linux,omitempty" platform:"linux"`

Blocking the non-matching-platform properties didn't seem to be worth the potential confusion of the Go platform tag not matching the Markdown platforms tag.

On Wed, May 10, 2017 at 02:14:28PM -0700, Mrunal Patel wrote:

Closed #747.

Without the ABNF and associated docs 1, there's nothing connecting
(for example) to unpack:
foo (object, REQUIRED) Something about foo…
into:
foo values MUST be objects.  Configurations MUST include the foo
property.  Something about foo…
I think we want something to help with that unpacking, even if it
doesn't use an ABNF (although I think ABNF is a nice, compact way to
clarify the syntax).  Do we have any alternative plans for clarifying
the syntax?


Dark mode

Describe the feature
A clear and concise description of what the feature is. Be warned : I will refuse requests with no or little informations
Screenshots / Draws
If applicable, add screenshots and/or draws to help explain your feature request.

Hello @ 👋
Thanks for your issue!
But...
...Every time someone does not respect the issue's template, a little mouse dies.

Learn how

Please edit your issue description to respect the issue template as much as possible :

https://github.com/isontheline/pro.webssh.net/blob/master/.github/ISSUE_TEMPLATE/bug_report.md
https://github.com/isontheline/pro.webssh.net/blob/master/.github/ISSUE_TEMPLATE/feature_request.md

Duplicate, already planned in #7


Menu Navigaion RTL

How can i move  the menu navigation to the right?

Hi,
you can check my fork where i made a good rtl support. A Demo to see if it satisfy you.
All you have to do is to include this css  in your page heade.
If you find an element doesn't work for you then replace all pull-right with pull-left in your page


Update-Pfa2Support requires 'array_admin' role to enable Remote Assist

Update-Pfa2Support requires 'array_admin' role to enable Remote Assist where as the current array management interface and the CLI allow the 'storage_admin' role to enable Remote Assist.
To my knowledge only the local 'pureuser' account can be assigned the 'array_admin' role.  For Update-Pfa2Support to be useful in our automation work this cmdlet needs to allow the 'storage_admin' role to perform this function too.

Hi Craig,
Actually, you can have any user with the role of "array_admin" as long as that user is defined with that role on creation (local user) or if that user is placed in the proper LDAP group (see documentation reference below).
The limitation of the permissions required for this cmdlet to function are set by the REST API, not by the PowerShell cmdlet. As stated in the referenced doc below, Remote Assist functionality is considered by the API to be considered as "array-wide changes dealing with global and system configurations." The Storage Admin role only deals with storage of the array, such as administering volumes, hosts, and host groups. For this particular cmdlet, as well as all others that deal with global or system modifications, then the Array Admin role is required.
As per the FlashArray User Guide --> Settings --> User Panel -

All users in the array, whether created locally or added to the array through LDAP integration, are assigned one of the following roles in the array:
Read-Only. Users with the Read-Only (readonly) role can perform operations that convey the state of the array. Read Only users cannot alter the state of the array.
Ops Admin. Users with the Ops Admin (ops_admin) role can perform the same operations as Read Only users plus enable and disable remote assistance sessions. Ops Admin users cannot alter the state of the array.
Storage Admin. Users with the Storage Admin (storage_admin) role can perform the same operations as Read Only users plus storage related operations, such as administering volumes, hosts, and host groups. Storage Admin users cannot perform operations that deal with global and system configurations.
Array Admin. Users with the Array Admin (array_admin) role can perform the same operations as Storage Admin users plus array-wide changes dealing with global and system configurations. In other words, Array Admin users can perform all operations.
For local users, the role is set during user creation. For LDAP users, the role is set by configuring groups in the directory that correspond to the FlashArray user roles.

I hope that clears things up.

Hi Mike,
I understand fully what you are saying, but that is only true when calling the Update-Pfa2Support cmdlet.
From the CLI or the GUI management interface as a 'storage_admin' account, I can enable Remote Assist. Shouldn't the CLI and the GUI be using the same rules? Anyway, this what I'm wondering.
Thanks, Craig
Sent with ProtonMail Secure Email.
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On Monday, December 20th, 2021 at 12:34 PM, Mike Nelson @.***> wrote:

Hi Craig,
Actually, you can have any user with the role of "array_admin" as long as that user is defined with that role on creation (local user) or if that user is placed in the proper LDAP group (see documentation reference below).
The limitation of the permissions required for this cmdlet to function are set by the REST API, not by the PowerShell cmdlet. As stated in the referenced doc below, Remote Assist functionality is considered by the API to be considered as "array-wide changes dealing with global and system configurations." The Storage Admin role only deals with storage of the array, such as administering volumes, hosts, and host groups. For this particular cmdlet, as well as all others that deal with global or system modifications, then the Array Admin role is required.
As per the FlashArray User Guide --> Settings --> User Panel -

All users in the array, whether created locally or added to the array through LDAP integration, are assigned one of the following roles in the array:
Read-Only. Users with the Read-Only (readonly) role can perform operations that convey the state of the array. Read Only users cannot alter the state of the array.
Ops Admin. Users with the Ops Admin (ops_admin) role can perform the same operations as Read Only users plus enable and disable remote assistance sessions. Ops Admin users cannot alter the state of the array.
Storage Admin. Users with the Storage Admin (storage_admin) role can perform the same operations as Read Only users plus storage related operations, such as administering volumes, hosts, and host groups. Storage Admin users cannot perform operations that deal with global and system configurations.
Array Admin. Users with the Array Admin (array_admin) role can perform the same operations as Storage Admin users plus array-wide changes dealing with global and system configurations. In other words, Array Admin users can perform all operations.
For local users, the role is set during user creation. For LDAP users, the role is set by configuring groups in the directory that correspond to the FlashArray user roles.

I hope that clears things up.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.***>

Craig,
I'm confused. I just created a user that has the Storage Admin role and logged in to the UI with that user ID. The Remote Assist and Phone Home options are greyed-out for that user ID (screenshot attached) I then tried to run the CLI command to enable Remote Assist with that user ID. It failed as it only allows that user to do a 'purearray remoteassist --status'.

Well, I wonder why our environment is different.
We can do a WebEx/Zoom together to prove that a storage_admin role can do it on any of our arrays regardless of Purity/FA version.
Sent with ProtonMail Secure Email.
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On Monday, December 20th, 2021 at 1:23 PM, Mike Nelson @.***> wrote:

Craig,
I'm confused. I just created a user that has the Storage Admin role and logged in to the UI with that user ID. The Remote Assist and Phone Home options are greyed-out for that user ID (screenshot attached) I then tried to run the CLI command to enable Remote Assist with that user ID. It failed as it only allows that user to do a 'purearray remoteassist --status'.
2021-12-20_15-13-58
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.***>

Craig,
If you could email me directly with your SE's or AE's name and we can set something up. My email is mnelson-at-purestorage.com.
Thanks.

Resolved & closed.


Restore faster-pr-diff-options feature on PRs

Originally disabled on PRs because of https://github.com/sindresorhus/refined-github/issues/2291

This feature was applied to two places:

PR file header: now disabled because of #2291
commit header: it still works

I had suggested some alternatives to replacing the original "PR diff options" dropdown:

dropping its "apply and reload" button (saves one click)
perhaps opening the dropdown on hover (saves one more click)

There's already a keyboard shortcut to toggle the whitespace.
Originally posted by @fregante in https://github.com/sindresorhus/refined-github/issues/2291#issuecomment-517539452

I use the "No Whitespace" button all the time (wish it was just ALWAYS enabled by default) and really miss it now.

Sadly there's not really space for it:

We could change the label on the button from "No whitespace" to something
like "-w" or an icon
On Tue, Dec 17, 2019, 15:57 Federico Brigante<EMAIL_ADDRESS>wrote:

Sadly there's not really space for it:
https://user-images.githubusercontent.com/1402241/62343178-ec00eb00-b513-11e9-9f55-30cb56fc2712.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sindresorhus/refined-github/issues/2597?email_source=notifications&email_token=AAI2U3OQNZCF6KA2B7DB4M3QZE4MPA5CNFSM4JW4L46KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHD5V7I#issuecomment-566745853,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAI2U3PR453AIXLG6KN6UILQZE4MPANCNFSM4JW4L46A
.

Still not enough space if all of GitHub’s widgets are visible and the title is there too

At what screen width does it break?

All of them. Look at the last screenshot I posted. Add the “Commit suggestions” widget and voila: no space left.

As a compromise, it could hide the 'files viewed' indicator maybe? Or we could reduce the amount of space the title has. Or we would increase the width the content has on the header.
I'm happy to mock up visualisations of these if you're interested.

it could hide the 'files viewed' indicator maybe?

We shouldn't hide a useful feature (the feature's utility is debatable, but...)

Or we could reduce the amount of space the title has.

I don't wanna play that game. We tried it for the past 2 years and we've been just moving the problem and never resolving it.

Or we would increase the width the content has on the header.

This could work, but the compromises are:

only works when the window fits it
it might not look great since the header would be slightly longer than the content width

In the end, I think these are the safest to implement and they still bring down the total number of clicks to one (+ hover)

dropping its "apply and reload" button (saves one click)
perhaps opening the dropdown on hover (saves one more click)

+1 for dropping apply and reload, especially as only two settings live on that menu. Not sure about the latter, is the hover behaviour implemented anywhere else?

is the hover behaviour implemented anywhere else?

It used to be implemented on the reactions popup, but it stopped working and we never readded it.

Can you at least re-add the shortcut? :slightly_smiling_face:

PR welcome for that. Most of the feature code can be dropped: https://github.com/sindresorhus/refined-github/blob/master/source/features/faster-pr-diff-options.tsx

Maybe something changed as I see plenty of space for those icons with my resolution (1920x1200)

Also when scrolled down (though in this mode the icons may need to be shorter)

Even on smaller resolutions there is enough space because GH removes some of the filters conveniently leaving room for the no-whitespace icon. (width: 640)

@fregante I see now. That last situation the gui is super tight.
I would be ok with ok with 2 toggle icons though it's not ideal due to vagueness of the their use. Perhaps a hover hint may be enough to clear up their purpose.
Some other ideas:

Move Commit suggestions
imo this is the problem. The PR name is almost removed with only 1.5 words showing, and the filters are all jammed together. Perhaps we can add an extra line to space things out a bit? or move the Comment Suggestions or files viewed beneath the Review Changes.

shrink the Commit suggestions
I'm struggling how this might be shrunk and still be useful, but if it wasn't so big it would likely solve our issue.

Add when enough space
Add the nice icons only when there's enough space. Say width > 1000 for the collapsed header and maybe always for the non collapsed header. I'm always reviewing on a large screen so this would be perfect for me.

Could you open new issue to clean up the file review header? There's indeed room for improvement to make more room.

I added another suggestion in my first issue to restore the feature almost as it was, with the exception that:

the icons appear in a slightly different place
the icons only appear on larger screens

This should avoid the wrapping issue on small resolutions.


Cannot find module './web-view/web-view-attributes' in 0.28.0

In 0.28.0 there are a couple of new issues that show out of the box. You can see these simply by opening the default app and then opening Dev tools. The errors are detailing in the screenshot below:

It looks like atom.asar is missing some code. The size of atom.asar is 111kb in 0.28 but is 123kb when I built master yesterday. Replacing atom.asar with a earlier copy resolves the issues for me.

It looks like something happened in the build process for 0.28 and something happened to atom.asar. I just built 0.28 from master and all is well.

Is this 32-bit or 64-bit Windows? No repro with the released OS X version

Sorry I am on Win 64. Rebuilding from master solved the problem with atom.asar missing code.

Fixed in v0.28.1.


Introduce clippy linter

Shall we update README as well?

We don't have a CONTRIBUTE file yet so adding this to the read me would not have much value. Any dev making changes to the repo would need to cargo fmt and clippy check anyway and CI will also provide feedback after this PR.


Desktop developer docs update

Summary
Changes to refresh the develop documentation for the Desktop App

Still a WIP, but hoping to get a bit of feedback ahead of time.

@BenLloydPearson @cwarnermm Requested re-reviews for content (and grammar :P)


Recommended test setup?

Hi,
I am writing an API client wrapper which I need to test. What is the recommended way to use guzzle services + mocking + history (request) tracking?
Does anyone have an example API client wrapper available?
I read through the Guzzle docs (link) and tried to find a solution in this repo('s issues), but it's not clear to me.
- Sander

Closing since stale - I'm doing a cleanup across the whole organization, to enable us to focus on the tickets that are still live and have interest. Please feel free to open a new issue if this still is important to you. :)


Whitepaper: PoS -> PoA

Proof of authority is a better descriptor for what we are doing than proof of stake.

this is fixed, closing


Change to netflix2 version 0.0.7

This fixes #10

Whoops, forgot to push the change here. Thank you.


fix(runtime-dom): enable set form attr to null on form-elements (#2840)

Close #2840

I thought about this a bit more, and I think this solution is good for this individual issue, but there are more readonly properties out there that could fail like this.
Example: #2766
I think the core reason is that the last line of shouldSetAsProp only checks for the existance of the key, but not wether it has a setter:
https://github.com/vuejs/vue-next/blob/92d7ad18296fe43a01d10f38e03b983188dfba5a/packages/runtime-dom/src/patchProp.ts#L112
Instead of adding new exceptions for individual properties like we do here, we could do something like this:
return key in el && !!Object.getOwnPropertyDescriptor(el.__proto__, key)?.set

But I'm not 100% sure if thats a) performance and b) safe for all elements. It works for not the button.form and textarea.type scenarios.

We could still cache it for perf reasons but I wonder if there isn't an exception in a particular browser that makes the property readonly without setting the proper flag (you never know with JS!)
I don't find having a list of exception that bad unless it begins to get too big

agreed. I sent a seperate PR for the textarea thing. This one here is again good to go.

Removed tag check in 180310cb
The form tag list adds to the baseline size and isn't really worth it.


Corrupted JSON not error handled in History class

I have been using the Jquery terminal as a part of a project that I have been working on, and I believe that I have encountered a bug.
In the file jquery.terminal-0.8.8.js, on lines 677-678
var data = $.Storage.get(storage_key);
data = data ? $.parseJSON(data) : [];

I am still trying to reproduce the issue, but essentially, I was receiving an error regarding illegal JSON when I attempted to initialize my terminal on the page load. This happened in my firefox browser, but not in chrome, which led me to believe that something was corrupted in my local storage. Sure enough, when I added the following try/catch....
var data = $.Storage.get(storage_key);
try {
    data = data ? $.parseJSON(data) : [];
}
catch(e) {
    data = [];
}

...the terminal was able to initialize without a nasty illegal JSON message. This left me wondering how corrupted JSON could be entered into the local storage in the first place. I noticed on line 696,
$.Storage.set(storage_key, $.json_stringify(data));

It appears that a custom JSON.stringify() function has been defined to stringify the inputted commands.
As I said, I unfortunately am still working on reproducing the issue with a variety of inputs, but I wanted to at least let you guys know that I had experienced this issue. Once I am able to reproduce, I will send screenshots of the error, as well as the offending input or sequence of input/browser actions.

Thanks for reporting the bug, I use custom stringify because when I was using original version I've got invalid results when I was using the function with Prototype library that was used by BiwaScheme.
Maybe the problem is when you have history that are larger then the limit of LocalStorage.

That would be a good place to start. I'll do my best to break things again and let you know what I come up with. I realize I haven't given you much to go on, other than the quick fix that worked for me.

I can reproduce this bug on Firefox by pressing F12 while the terminal is focused. Nothing happens (the developer console doesn't open, and there is no visible change in the terminal), but the line saved to local storage contains a \0 for each occurence of F12 in the input. This breaks the JSON parser.

@archtimmy Fixed in devel branch.


extracors issue

Describe the bug
Following installation, recieved the following while executing the parse_single_from_code.py
No extractors passed: initializing default configuration.
No combinedScorers passed: initializing default configuration.

To Reproduce
after connected to the server, run parse_single_from_code.py(or other example codes)
Expected behavior
all possible answers of the 5w1h in the example text should be printed
Log
No extractors passed: initializing default configuration.
No combinedScorers passed: initializing default configuration.

This issue is similar to #63. The code was configured and updated in the latest version, but error arose still
Versions (please complete the following information):

OS: MacOS Big Sur 11.4
Python Version 3.8.10 base:conda
Giveme5W1H Version 1.018
Stanford CoreNLP Version not sure

have the same issue


Configuration file issue

Thank you for creating this SRS milter.  I installed it using the amd64 rpm on Rocky Linux 9.4 with sendmail 8.16 but I have been facing two issues.
The first issue is minor and related to finding the configuration file.   I kept getting the error:
Jun  9 10:45:02 server1 srs-milter[1520879]: lvl=crit msg="error reading config file" err="Config File "srs-milter" Not Found in "[/etc/srs-milter /]"
But the file, provided by the rpm, did exist at /etc/srs-milter/srs-milter.yml
Initially I thought the issue was related to selinux permissions (because there were many selinux messages) so I set selinux to permissive mode and yet the configuration file issue persists.
Permissions were 750 so I decided to try 755 as an experiment.  This worked, although I now get the warning:
Jun  9 10:58:51 server1 systemd[1523231]: ConfigurationDirectory 'srs-milter' already exists but the mode is different. (File system: 755 ConfigurationDirectoryMode: 750)
The second issue is more serious and relates to which messages have their return paths rewritten.  I am using the default srs-milter.yml file except that I changed logLevel to 4 (srsDomain and srsKeys are the only other options set).  Locally delivered e-mails are having their returns paths rewritten but forwarded messages are not.  Of course, it's the forwarded messages that need to be rewritten.  As a result forwarded messages to GMail are still bouncing back.
Please advise.  Am happy to perform additional debugging steps with guidance.  I would love to get this milter working.

Hello @locutius
first issue:
Sorry I have not tested the RPMs myself. What does systemctl cat srs-milter output?
It might be that your systemd does not handle the DynamicUser=true https://github.com/d--j/srs-milter/blob/52f7d0f5d72d62ab4188cf4b86368b6c3a8a5d8d/packaging/srs-milter.service#L6 or SupplementaryGroups https://github.com/d--j/srs-milter/blob/52f7d0f5d72d62ab4188cf4b86368b6c3a8a5d8d/packaging/srs-milter.service#L9
they are needed so that srs-milter can read the configuration file (since it can be accessed by the group nogroup/nobody)
The ConfigurationDirectory settings are commented out in the service: https://github.com/d--j/srs-milter/blob/52f7d0f5d72d62ab4188cf4b86368b6c3a8a5d8d/packaging/srs-milter.service#L13-L14
second issue:
increase the log level to 4 https://github.com/d--j/srs-milter/blob/52f7d0f5d72d62ab4188cf4b86368b6c3a8a5d8d/packaging/srs-milter.yml#L19-L21 and provide the log output

Hello @d--j
Thank you very much for the suggestions.  While I was waiting for your reply, I made the decision to migrate from sendmail to postfix in order to use the postsrsd package.  But, if you feel solving the couple issues I found would be beneficial to other users of srs-milter, I am happy to keep debugging with you.  I provide the details you requested below.
For Issue #1, here are the settings you requested, followed by the configuration file details:
[root@server1 postfix]# systemctl cat srs-milter
# /usr/lib/systemd/system/srs-milter.service
[Unit]
Description=Mail filter handling SRS address rewriting
After=network.target

[Service]
DynamicUser=true
User=srsmilter
Group=srsmilter
SupplementaryGroups=nobody
ExecStart=/usr/bin/srs-milter -systemd
Restart=always
RestartSec=10
ConfigurationDirectory=srs-milter
ConfigurationDirectoryMode=750
#ProtectProc=invisible
PrivateDevices=true
ProtectHostname=true
ProtectClock=true
ProtectKernelTunables=true
ProtectKernelModules=true
ProtectKernelLogs=true
ProtectControlGroups=true
RestrictAddressFamilies=AF_UNIX AF_INET AF_INET6 AF_NETLINK
LockPersonality=true
RestrictRealtime=true
SystemCallFilter=@system-service
SystemCallErrorNumber=EPERM
SystemCallArchitectures=native

[Install]
WantedBy=multi-user.target

[root@server1 postfix]# ls -l /etc/srs-milter/srs-milter.yml
-rw-r--r-x. 1 root nobody 1481 Jun 10 11:03 /etc/srs-milter/srs-milter.yml

For Issue #2, I had already set the logLevel to 4.  The messages log was quite large so I have provided the last 1000 lines matching "srs-milter" (domains redacted).  The result is attached here:
zz2.txt
Let me know if this information helps and/or if there is any other information you require.  Again, I'm moving to postfix with postsrsd but I'm happy to help if you think it is worthwhile.

here is one gmail destination mail:
Jun 20 09:01:55 server1 srs-milter[856]: lvl=dbug msg=start sub=milter qid=45KD1tYR1972485 user=<EMAIL_ADDRESS>Jun 20 09:01:55 server1 srs-milter[856]: lvl=dbug msg="to is not one of our SRS addresses" sub=milter qid=45KD1tYR1972485 user=<EMAIL_ADDRESS>Jun 20 09:01:55 server1 srs-milter[856]: lvl=dbug msg="to is remote" sub=milter qid=45KD1tYR1972485 user=<EMAIL_ADDRESS>transport=esmtp
Jun 20 09:01:55 server1 srs-milter[856]: lvl=dbug msg=SRS sub=milter qid=45KD1tYR1972485 user=<EMAIL_ADDRESS><EMAIL_ADDRESS>Jun 20 09:01:55 server1 srs-milter[856]: lvl=dbug msg="did not touch MIME headers because of DKIM" sub=milter qid=45KD1tYR1972485 user=
Jun 20 09:01:55 server1 srs-milter[856]: lvl=info msg=done sub=milter qid=45KD1tYR1972485 user= dur=639.259µs actions=sender:user<EMAIL_ADDRESS>
This mails return path does not get rewritten because domain1.com is the srsDomain. It is assumed that the srsDomain actually is a local domain that does not need rewriting. Your mail server should accept emails to the srsDomain (otherwise you would not get any bounces).

Thank you for your reply.  Unfortunately, I am having difficulty following your reasoning because it doesn't seem to match our setup.  Perhaps it will help if I provide some more details ...
You are correct that domain1.domain is the srsDomain and that it doesn't need rewriting.  Our setup is as follows.   domain1.domain receives e-mails that are both delivered to the domain1.domain mailbox AND forwarded to<EMAIL_ADDRESS> The return paths for the domain1.domain e-mails are being rewritten by srs-milter but the ones forwarded to gmail.com are not.  This seems backwards to me.
I have a perfect example to share because I received your last comment as an e-mail from github.   I received the e-mail both at domain1.domain AND at gmail.com.  Here are the two return paths in the headers I received
On domain1.domain I received:
Return-Path<EMAIL_ADDRESS>On gmail.com I received:
Return-Path<EMAIL_ADDRESS>Although Gmail accepts the forwarded e-mails from github, e-mails from other domains (which presumably have more stringent requirements) are rejected by Gmail.  My hope is that SRS will solve this problem.
I really do appreciate your efforts on this but, if any any point, you think it's too much work let me know.  If I'm the only one to have brought this to your attention, perhaps it's not an issue for the majority of your users.  Still, if you want to solve this mystery, I'm happy to work through it.


[BuildSystem] Guard killAfterTimeoutThread in LaneBasedExecutionQueue

TSAN tests show a potential data race in cancellation. Add a guard
around the killAfterTimeoutThread variable to protect against it.
rdar://problem/40707314

@swift-ci please smoke test


Fix unnecesery removing

The error was next:
If in one day the strategy has return, but the benchmark has no data, df will have next view (for example): 0.1 na
The old method removes the return of strategy.
In result, if we compare the strategy to different benchmarks, we will result to different numbers in strategy return. Its frustrating.

I understood that i forgot to cubstract risk-free rate...


vstest.console fails to run CPP UWP Test

Description
vstest.console fails to run CPP UWP Test on win10 14393 emulator
Steps to reproduce

Create new project. CPP->Windows Universal->Unit Test App
Store-> Create App package; Choose option; not to publish to store;generate app bundle=Never;
Launch developer cmd prompt elevated.
go to folder where .appx is generated
Use cmd like this to register .cer in root store on the local machine certutil -addstore root UnitTestApp1_<IP_ADDRESS>_Win32_Debug.cer
run tests on installed emulator using emulator.runsettings

vstest.console UnitTestApp1_<IP_ADDRESS>_Win32_Debug.appx /settings:emulator.runsettings

Expected behavior
Unit test run successfully
Actual behavior
Starting test execution, please wait...
Warning: Using Isolation mode to run unit tests for Windows Phone apps. Use the /InIsolation parameter to suppress this warning.
Error: Failed to initialize client proxy: could not connect to test process .

Error: Could not connect to net.tcp://<IP_ADDRESS>:8020/RemoteTestExecutorService. The connection attempt lasted for a time span of 00:00:21.0192488. TCP error code 10060: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond <IP_ADDRESS>:8020

Environment
Win10 14393 + VS 2017 26224.1

To be able to run the tests - the unit test app need internetClientServer capability instead of 'internetClient' capability in Unit Test App package.appxmanifest . It can be done by change through the dialog in visual studio e.g.

or adding the following in package.appxmanifest
<Capabilities>
    <Capability Name="internetClientServer" />
  </Capabilities>
</Package>

This also applies for C# and VB UWP tests if they are run with 14393 emulator or device.


refactor: htsget-lambda and misc deployment changes

Closes #258
Changes

Use the htsget-axum router for htsget-lambda, avoiding the need for most of the htsget-lambda library code.
Update tests to include testing some more error conditions.
Fix the Dockerfile.dockerignore and use htsget-axum in the Dockerfile.
Update the deployment to allow creating stateful buckets which are populated with the data in the data directory.

The docs are updated to clarify the difference between s3BucketResources and the new createS3Buckets.

Updates

Deployment now has a dedicated example_deploy.toml, with options to create a bucket and copy test data.

The deployment should work out of the box, and is public by default.
Add warnings in README.md and printed log message about the public deployment.

The only umccr-specific thing left to remove is the hosted zone domain name. I can't think of a way to get rid of this other than just not using a hosted zone, and instead using the ApiGW default endpoint URL.


Enable-8 create a better character counter widget

This page currently has the following text for the character count section:

The character counter uses a JavaScript library to implement it. Below is the HTML markup needed for it to work, as well as instructions on how to load the library in your own projects.

Are we still using a JavaScript library? If so, I think we should mention which library it is. If not, then this text needs to be changed.

This page currently has the following text for the character count section:

The character counter uses a JavaScript library to implement it. Below is the HTML markup needed for it to work, as well as instructions on how to load the library in your own projects.

Are we still using a JavaScript library? If so, I think we should mention which library it is. If not, then this text needs to be changed.

Hi @zoltan-dulac , I've decided on going with adding/removing an exclamation to the end of the text to force Chrome to "see a difference" on each key press.
For example:

1st key press = "Character Count: 10/100."
2nd key press = "Character Count: 10/100.!"
3rd key press = "Character Count: 10/100."

This approach works for Safari, Firefox and Chrome with VoiceOver on Mac.

If you were wondering, here are other methods I've tried:
Did not work ❌:

Adding or removing a whitespace at the end of the announcement text on each key press.
Doing everything in the KeyUp event (removal, timeout, populate).

aria-atomic

"true" without removing the <p> element.
"true" while also deleting the <p> element and appending a new <p> element to live region each time.

aria-relevant

Set to 'text'.
Set to 'additions'.
Set to 'additions removals' -> ⚠ works but VoiceOver "double-speaks" — it tries to read out the removal and immediately is interrupted by the addition. Even with no timeout there is a noticeable interruption. As a result, don't think this is acceptable.

aria-live

Set to 'assertive'.

aria-busy

Setting to "true" on keydown and then "false" on key up (before or after .textContent).

Other Methods

Force re-building between <p> and <div>.
Force re-building between <p> and <span>.
Repeating 9 and 10 with 'aria-atomic' as well.
Using only <div> without the inner <p> element and changing .textContent for that single <div>.
Removing the element around the text on every other keypress.
Adding a brand-new <p> element within the ARIA live region on every key press.
Remove the inner <p> element, use .textContent for <div> and appending to it on every key press. This sort of works but VoiceOver will repeat the entire string, which has the previous counts as well. ❌ Not acceptable.


Permission denied creating a new pod in statefulset

What did you do?
I upgraded to the newer version of prometheus-operator (v0.11.0) and tried to have my pods use persistent storage using volumeClaimTemplate by adding the following to the bottom of my Prometheus yaml (but formatted correctly):
  storage:     volumeClaimTemplate:       spec:         selector:           matchLabels:             app: "prometheus-persistent-volume-0"         resources:           requests:             storage: 50Gi
and following the example for persistent volume here: https://github.com/coreos/prometheus-operator/blob/master/Documentation/user-guides/storage.md#manual-storage-provisioning
What did you expect to see?
I expected that my pods would be created and that they would have nfs mounts as specified in my config.
What did you see instead? Under which circumstances?
From running kubectl describe statefulset prometheus-shared:
  1m	1m	1	kubelet, k8s-telex-scroll.ny2.etsy.com	spec.containers{prometheus}	Warning	Failed		Failed to start container with id d27851b41aa20f12b75a4081a15b1402dac17bee4095ae0fd758d22a06f61d11 with error: rpc error: code = 2 desc = failed to start container "d27851b41aa20f12b75a4081a15b1402dac17bee4095ae0fd758d22a06f61d11": Error response from daemon: {"message":"mkdir /var/lib/kubelet/pods/75fc59dd-7301-11e7-b843-7cd30ad56ea4/volumes/kubernetes.io~nfs/prometheus-storage/prometheus-db: permission denied"}   1m	1m	1	kubelet, k8s-telex-scroll.ny2.etsy.com					Warning	FailedSync	Error syncing pod, skipping: failed to "StartContainer" for "prometheus" with rpc error: code = 2 desc = failed to start container "d27851b41aa20f12b75a4081a15b1402dac17bee4095ae0fd758d22a06f61d11": Error response from daemon: {"message":"mkdir /var/lib/kubelet/pods/75fc59dd-7301-11e7-b843-7cd30ad56ea4/volumes/kubernetes.io~nfs/prometheus-storage/prometheus-db: permission denied"}: "Start Container Failed"
Environment

Kubernetes version information:

Client Version: version.Info{Major:"1", Minor:"6", GitVersion:"v1.6.2", GitCommit:"477efc3cbe6a7effca06bd1452fa356e2201e1ee", GitTreeState:"clean", BuildDate:"2017-04-19T20:33:11Z", GoVersion:"go1.7.5", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"6", GitVersion:"v1.6.2", GitCommit:"477efc3cbe6a7effca06bd1452fa356e2201e1ee", GitTreeState:"clean", BuildDate:"2017-04-19T20:22:08Z", GoVersion:"go1.7.5", Compiler:"gc", Platform:"linux/amd64"}

Kubernetes cluster kind:

Custom chef deploy on bare metal CentOS 7

Manifests:

insert manifests relevant to the issue

Prometheus Operator Logs:

insert Prometheus Operator logs relevant to the issue here

I'm unsure this has to do with the Prometheus Operator. Could you try to mount that particular volume through a PVC with an extra Pod?

@brancz you were right. This was actually resolved by just fixing the permissions of the directory on the storage host. Thanks!

Always happy to help! Feel free to open issue whenever you have a problem!


Definition for rule 'fp/no-arguments' was not found

I've got the latest version and all seems to be working well except for this. Is this definition not in the latest version of the npm package yet?

You're right, I hadn't publish yet.
I just published v2.1.0 where it should be available, along with no-rest-parameters.

Sorry, hadn't checked the closed issues! :P

Easy mistake to make :smile:


IPv6 support?

Is your feature request related to a problem? Please describe.
The lbrynet daemon doesn't seem to listen for or accept IPv6 connections. The only exception is that there seems to be a connection between the lbrynet daemon and lbry desktop client that is made using the IPv6 loopback address.
Describe the solution you'd like
Ideally, the lbrynet daemon should take advantage of IPv6 where available, to create a much healthier P2P network.
Describe alternatives you've considered
Using IPv4 and port forwarding when behind a NAT is feasible for someone who wants to contribute to the P2P network, but IPv4 addresses are in such short supply now that many people end up behind a CGNAT/double-NAT, which makes this impossible.
Additional context
n/a

Since this is about lbrynet, you should close the issue here and move to https://github.com/lbryio/lbry-sdk/issues, the repository for lbrynet aka lbry-sdk.

Alright, I'll move it over there. Wasn't 100% sure as I didn't notice any descriptions that explicitly pointed out lbry-sdk as being the dedicated daemon for LBRY peer-to-peer stuff. The name itself makes it sound like, well, an sdk, not a daemon/service.


ci(chore) - various updates to replace deprecated action notices and red builds

Hey @uglyog
Noted the CI workflows were throwing a few deprecation notices, and the release workflow was failing on any jobs that were for the ffi, verifier cli, or mock server cli, so this PR add's a few changes to clean up the build

Update actions/checkout@v2 to actions/checkout@v3
Switch deprecated actions-rs/toolchain@v1 with dtolnay/rust-toolchain https://github.com/dtolnay/rust-toolchain as original unmaintained - see issue
Replace marcopolo/cargo@master also unmaintained, and not sure using an action for this adds value based on what we are using from it, with a cargo rust/test/build and setting the working-directory as part of the gh action step value
Remove choco install -y llvm from windows build-chain, llvm is already installed on windows runners https://github.com/actions/runner-images/blob/d8d4d924ad21a8ba4931b2988f74bbb376cad021/images/win/toolsets/toolset-2022.json#L400
replace nightly-2022-12-01 with nightly - fails with updated toolchain actions update, it looks like the underlying reason for it being pinned has been fixed https://github.com/rust-lang/rust/issues/105886
Skip upload release assets on the release workflow, where the tags don't correlate with a step to build artefacts.

Nice one!


flexget: create nixos service module

This module adds support for defining a flexget service.
Due to flexget insisting on being able to write all over where it finds its configuration file, we use a ExecStartPre hook to copy the generated configuration file into place under the user's home.
It's fairly ugly and I'm very open to suggestions.
Things done:

[X] Tested using sandboxing (nix-build --option build-use-chroot true or nix.useChroot on NixOS)
Built on platform(s)

[X] NixOS
[ ] OS X
[ ] Linux

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

This looks good to me. Copying the configuration file is a sensible approach, IMO.

One alternative to copying into the homedir is to copy the configuration file into a volatile location, like RuntimeDirectory, but seeing as this runs under a non-login user anyway, it hardly makes a difference.

Merging, given the lack of objections.

Ach, I realized this was opened against the wrong branch

Please always use master or staging unless you're backporting something.

Sorry about. Do you need me to rebase?

Yeah, just rebase onto master and open a new PR.

@joachifm , any chance we can get this into 16.03? Nothing depends on it so the impact should be limited.

Sounds reasonable to me. Please open a PR against release-16.03.


build: Update GraphDB-SE version

resolves #1522

@benjamingeer let's hope the tests run better with GraphDB 9.0.0 :-)

Thanks for the review :-)