Simplify SafeArea test for maintainBottomViewPadding to ensure maintainBottomViewPadding is always respected

Recent flutter engine changes enable iOS MediaQuery bottom viewInsets and padding to be !=0 simultaneously (see https://github.com/flutter/engine/pull/29281).
This condition resulted in SafeArea maintainBottomViewPadding being ignored during keyboard animation.
This commit forces use of viewPadding.bottom when maintainBottomViewPadding is true.

Before
After

https://user-images.githubusercontent.com/55881597/152152622-1f9c8400-ca6b-4ea5-b58c-cff7c7b806d9.mp4
https://user-images.githubusercontent.com/55881597/152152697-75ecfeaf-1ea5-4eb0-a224-f4cefa8744d8.mp4

This fixes the following issue:
#97609.
No changes to existing tests.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

I offer the following justification for the much-simplified condition concerning bottom padding:
Assuming maintainBottomViewPadding is true and knowing that padding is calculated...

If viewInsets >= viewPadding (padding = 0), we should use viewPadding (this was the case previously)
If  0 < viewInsets < viewPadding (padding = viewPadding - viewInsets), we should use viewPadding (this represents animating keyboard, and fixes the graphical glitch)
If viewInsets == 0 (padding = viewPadding), there is no difference between padding and viewPadding

In all cases, it is valid to use viewPadding when maintainBottomViewPadding is true. This is also semantically aligned with the property name itself, which does not imply conditional behaviour based on viewInsets/padding.

I assume the failing checks are unrelated to the PR content - is there a simple way to prod the checks to be performed again?


sponsorsLogoMadeVisible

due to the presence of the class name="sponsor-logo" it was making the view of the sponsor image somehow 0x0 pixels.

changing the class name makes it work all fine.

initially there were some #shadow.root style which was making all this, but now unable to see that style attribute.
@championswimmer can you explain it please.

@SilentFlame , I am not finding any new things in the commit . Kindly explain.
Class name changed from sponsor-logo to logo ?

@aayusharora problem is that initially when it was sponsor-logo it have some style files somewhere,
which is


Create tenant - seeding is not happening if same connectionString

If I create a new tenant with the exact connection string as the root tenant
eg : {{url}}/tenants
{
"name" : "tst",
"id":"tst",
<EMAIL_ADDRESS>"connectionString":"same as root"
}
Admin user is not seeded for "test" tenant

Even if i use the same database connection string, the admin is being created.


Mavlink PLAY_TUNE message does not play tune

Describe the bug
Sending a tune string "MFT200CDEFGAB" via a PLAY_TUNE mavlink message does not produce any sound on the drone.
From the fix for #12059 I can produce sound with "tune_control play -m "MFT200CDEFGAB" ".
The PLAY_TUNE mavlink handler is implemented in https://github.com/PX4/Firmware/blob/4831a4b0cfde499eb1a20257cc46b130e6554e3a/src/modules/mavlink/mavlink_receiver.cpp#L1627 and I am sending the correct message because this triggers, but it doesn't produce sound.
To Reproduce
Steps to reproduce the behavior:

Drone switched on '...'
send PLAY_TUNE mavlink message with "MFT200CDEFGAB" in PLAY_TUNE.tune
Listen for sound

Expected behavior
A tune should be played.
Drone (please complete the following information):

Pixhawk2 with latest firmware from git

Hi @wluker , if I issue the command tune_control play -m "MFT200CDEFGAB"  from the mavlink console in QGC the tune plays correctly for me on fmu-v2 and fmu-v5.
I think we can close this issue, but if I am not understanding your issue or this is still an issue for you would you please re-open?  Thanks!

Hi @mcsauder
Thanks for looking into this. This issue is not in tune_control play -m "MFT200CDEFGAB" because that plays for me as well. The issue is that you can't trigger any tune via the mavlink message PLAY_TUNE. https://mavlink.io/en/messages/common.html#PLAY_TUNE
I was trying to send that message via an offboard controller, but no tones were being played even though I could tell PX4 was receiving the message. Maybe I am misunderstanding the intended use of the PLAY_TUNE message.

Fixed in #13158.

In your face @stale bot!
Thanks @julianoes 🎵🎶🎵🎶


pipe separated IDs in 'with' field

I thought this was allowed? I can delete one ID, but I wanted to check if something changed? Or if I should use a different seperator (I'm sure I have used a pipe in this field historically, maybe not?)
with value "UniProtKB:P39986|UniProtKB:Q9HD20" should be in the form "DB:ACCESSION": failed to load qualifier 'aspect=F; term=transmembrane protein dislocase activity; GOid=GO:0140567; evidence=ISO; db_xref=GO_REF:0000024; with=UniProtKB:P39986|UniProtKB:Q9HD20; date=20210305' from SPACUNK4.07c.1:
with value "UniProtKB:P39986|UniProtKB:Q9HD20" should be in the form "DB:ACCESSION": failed to load qualifier 'aspect=P; term=extraction of mislocalized protein from ER membrane; GOid=GO:0140569; evidence=ISS; db_xref=GO_REF:0000024; with=UniProtKB:P39986|UniProtKB:Q9HD20; date=20210305' from SPACUNK4.07c.1:

They need to be comma separated.

I can't see any warnings like this in the logs now.  Can we close this issue?

yup!


AttributeError: 'ClustersOperations' object has no attribute 'create'

Starting to create HDInsight Hadoop cluster hdisamplecluster5305 with Azure Data Lake Storage Gen2
Traceback (most recent call last):
File "C:\Users\Surendra Babu G\OneDrive - Nuvepro Technologies Private Limited\Desktop\Azure\hdinsight-python-sdk-samples\samples\create_hadoop_cluster_with_adls_gen2_sample.py", line 87, in
main()
File "C:\Users\Surendra Babu G\OneDrive - Nuvepro Technologies Private Limited\Desktop\Azure\hdinsight-python-sdk-samples\samples\create_hadoop_cluster_with_adls_gen2_sample.py", line 83, in main
client.clusters.create(RESOURCE_GROUP_NAME, CLUSTER_NAME, create_params)
AttributeError: 'ClustersOperations' object has no attribute 'create'

the examples aren't updated in a while. also there is no version mentioned. I can see that create has been replaced with begin_create
https://docs.microsoft.com/en-us/python/api/azure-mgmt-hdinsight/azure.mgmt.hdinsight.operations.clustersoperations?view=azure-python


print additional column CLUSTERS and STEP for release object

fixes https://github.com/bookingcom/shipper/issues/67

Nice! Two things, both our fault :)

Please rebase on master to get the lint fixes (megacheck was deprecated and we needed to update some things in the CI)
Please also update pkg/crds/release.go. We'll hopefully ditch the plain YAML representations soon; until then we need to keep them in sync.

I took the changes from this one, applied the changes suggested, and opened #80. That now supersedes this PR.


Fix some typo

Was reading the docs and noticed some typo so I fixed them. Then I also did a quick check in the repo README file as well just to be thorough. The only slightly bigger change would be the following (repo README file line 127):
original: lib file containts functions
changed to: lib file which contains functions
just changing containts to contains seems abit off, hence I added  which before it. Hope that's okay :)

Thanks i will accept it :D


Rename "tsconfig.json" to "tpsconfig.json" due to transphobia

Bug Report
The name tsconfig.json is triggering because it sounds like "trans configuration", as if trans people need to be fixed. I propose changing to tpsconfig.json
🔎 Search Terms
tsconfig.json
🕗 Version & Regression Information
Forever
🙁 Actual behavior
tsconfig.json sounds like "trans config", as if trans people are not good enough on their own and need configuring
🙂 Expected behavior
tpsconfig.json which sounds more like typescript and less like trans

Does this really need to be changed? who cares. its a file name. it could be named dsklfjnfkltghfjkltfjklhdjfgkl.json for all i care and i wouldnt make a issue changing it.
:)

This is a poor representation of the trans community. If you really care about the trans community, you wouldn't make issues where they don't exist because transgender individuals have enough on their plate already. TS when said as an initialism or said as a word, is a far stretch from the word trans bar 2 sounds. You could just as easily made this extrapolation with the word trains. You're probably just a troll, but I think this ought to be said regardless.

Does TG mean Transgender to you?

As a trans person myself, this is clearly a bad-faith troll post pretending to be trans to then screenshot and/or point to as an example of how unreasonable trans people are.
The only person who reacted positively (with many reacts, mind you) is a user who proudly admits to being banned for "thinking different" (which is, in my opinion, a telltale sign of someone who arrogantly holds onto objectionable (bannable) beliefs).

I don't think this merits a response.


HyperdriveRegistry deploy

Resolved Issues

fixes https://github.com/delvtech/hyperdrive/issues/922

Description
Since anyone can deploy instances of Hyperdrive, it is difficult to determine and keep track of which instances are known and reputable.  To alleviate this, the HyperdriveRegistry contract is permissioned and includes an additional data field along with the instance address to give additional context.
Review Checklists
Please check each item before approving the pull request. While going
through the checklist, it is recommended to leave comments on items that are
referenced in the checklist to make sure that they are reviewed. If there are
multiple reviewers, copy the checklists into sections titled ## [Reviewer Name].
If the PR doesn't touch Solidity and/or Rust, the corresponding checklist can
be removed.
@jalextowle
Rust

[ ] Testing

[ ] Are there new or updated unit or integration tests?
[ ] Do the tests cover the happy paths?
[ ] Do the tests cover the unhappy paths?
[ ] Are there an adequate number of fuzz tests to ensure that we are
covering the full input space?
[ ] If matching Solidity behavior, are there differential fuzz tests that
ensure that Rust matches Solidity?

@jrhea
Rust

[ ] Testing

[ ] Are there new or updated unit or integration tests?
[ ] Do the tests cover the happy paths?
[ ] Do the tests cover the unhappy paths?
[ ] Are there an adequate number of fuzz tests to ensure that we are
covering the full input space?
[ ] If matching Solidity behavior, are there differential fuzz tests that
ensure that Rust matches Solidity?

LGTM. Nice dude!
One problem that we may run into is that the devnet admin defaults to address(0), which would prevent the registry from being updated on devnet. I think we'll want to update .github/workflows/devnet.yaml so that it has an ADMIN environment variable similar to .github/workflows/testnet.yaml. We can use a Github variable or just put a comment with the private key in the workflow since this is used for developer purposes.

Awesome! Went for the "storing it in github variables" approach. Made the corresponding addition to the workflow as well.


doesn't compile with brunch

Couldn't find how to compile it on a project using brunch ; stopped using it.
Apparently there is some trouble with ES6 syntax.

How can I reproduce the error?
Can you give me your brunch configuration, what node version ur using and error log.


Ffmpeg44

Checklist

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

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

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

Documentation on acceptable licenses can be found here.

@conda-forge-admin please rerender

@conda-forge-admin please rerender

I think this needs: https://github.com/conda-forge/conda-forge-pinning-feedstock/pull/2717/files

@conda-forge-

It seems to need: https://github.com/conda-forge/libcap-feedstock/pull/35
I also added an attr migrator.

@conda-forge-admin please rerender


Update docs on resetting partial record state

Provides information about how to reset the partial record state on a given record instance.
Also clear up the extra costs involved with JIT loads.

@jswymer Please take a look at this.


Add include paths

Codecov Report

:exclamation: No coverage uploaded for pull request head (master@442b8fd).

Continue to review full report at Codecov.

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

Thanks! Actually something I needed a few weeks ago but I was too lazy :-)
I'm wondering if the WSGI_AUTH_INCLUDE_PATHS should be renamed to WSGI_AUTH_PATHS and set it to '/' as default. That makes it perhaps somewhat easier to understand.

Good idea @mvantellingen I renamed WSGI_AUTH_INCLUDE_PATHS to WSGI_AUTH_PATHS.

Neat, thanks!! Will do a manual check asap and then a new release


ARCs for smart contract NFTs and interface detection

This PR includes two ARCs.  I decided to put them together because one refers to the other (specifically, the NFT ARC depends on the interface detection ARC).
The interface detection ARC is nearly identical to ERC-165 with minor adjustments for Algorand.
The NFT ARC is similar to ERC-721.  Besides minor changes to fit Algorand, the most notable change is moving the transfer management features out of the core spec and into an extension.  This makes the core interface significantly simpler.
The biggest drawback of this NFT ARC is that it relies on ARC-22 read-only methods, which are not yet well supported by algod and indexers to view data without publishing an actual transaction.  The most important views of an NFT are ownerOf and metadataURI.  The ownerOf method provides data that is redundant with the Transfer event, so without support for free calling of ARC-22 read-only methods, an indexer can still get this info.  I've considered adding an extra ametadataURIEvent extension for indexers to have an easy way to index metadata as well for NFTs created while ARC-22 support is still poor.

This NFT ARC could possibly subsume the current ARC-72 PR.  ARC-72 is similarly an NFT spec based on ERC-721, but is currently not as fleshed out as this one.

We will also be submitting a spec for an NFT Index interface to pair with smart contract NFTs. People should not be hitting the chain for anything other than verification and/or state change.

Will this work with conduit?

We will also be submitting a spec for an NFT Index interface to pair with smart contract NFTs. People should not be hitting the chain for anything other than verification and/or state change.

Will this work with conduit?

We are only submitting the interface, people can do whatever they want to deliver the interface.

Superseded by #164 and #165.


FIX Gridfiled pagination missing after search

Parent issue

https://github.com/symbiote/silverstripe-queuedjobs/issues/405

Open new issue https://github.com/silverstripe/silverstripe-framework/pull/10828


Add extension: axetroy.vscode-deno

I'd like to use Deno so this will be helpful for adding this extension: axetroy.vscode-deno
Thanks for your consideration!

Thank you for the request! The vscode_deno is now available in the code-server marketplace,


[Suggestion] achievements checker

Hello. I hope this is correct place to write my thoughts, if not please show me a correct place for this topic or move it.

Your project is really great and helpful, you did amazing job.
Is there in feedback link any info about account's achievements or is there a way to extract them (let's say from api or something)? If yes - could you make a achievements search, which would really help to find missing achievements (especially from "wonders of the world" section since they are hidden there).
Thanks in advance for your response.

I don't think we can see achievement from the token sadly

But will make a page with the list of achievements and you can check it manually

Would love it even if we only have that much to be honest ❤


Issue when suspending and resuming the background audio sample

I tried using the background audio sample.
But when I suspend the app and resume it more than 2 times the app crashes.
Sorry I'm not able to give more details about the crash.

Sorry for the trouble. There are a couple known issues in the sample right now and we're preparing an update to address them. If you can provide a callstack at the time of the crash I may be able to offer a workaround in the meantime, otherwise please stay tuned for a future update.


p_tuning layers disappear after loading peft model

After I trained my peft model  (in this case P_Tuning), I save it using save_pretrained method. Later I load the model using:
config.peft_config = PeftConfig.from_pretrained(peft_model_id,
                                            inference_mode=inference)
MODEL_TASK_TYPE = {
    TaskType.CAUSAL_LM: AutoModelForCausalLM,
    TaskType.SEQ_2_SEQ_LM: AutoModelForSeq2SeqLM,
    TaskType.SEQ_CLS: AutoModelForSequenceClassification,
}
base_model = MODEL_TASK_TYPE[self.task_type].from_pretrained(config.peft_config.base_model_name_or_path)
self.model = PeftModel.from_pretrained(base_model, config.prompt_model_name_or_path) # prompt_model_name_or_path is the dir where I save my peft model

When I query self.model.prompt_encoder I only see the embedding layer of the prompt. How can I load the model with all layers of the prompt as was during training?
This question brings another question. How does inference work? I tried tracing the function calls after model(**input) and noticed that the embeddings of the prompt (which are randomly initialized) is concatenated to the input, and the whole thing is embedded by the word embedding layer of the base model and sent to the base model forward function. I am not sure how the trained p_tuning layers are used during inference.

When I query self.model.prompt_encoder I only see the embedding layer of the prompt. How can I load the model with all layers of the prompt as was during training?

Indeed, currently, when we save the p-tuning model, the MLP part (which I think is what you're referring to) is not being saved along with the prompt embedding. My guess is that when this code was written, it was assumed that after initial training, there is no further training step, which makes it unnecessary to load the whole p_tuning.PromptEncoder module. Pinging @pacman100 if he can confirm this.
Do you plan to do further training? If not, then you shouldn't need the additional layers.

Yes, I plan to do further training.

While waiting for a reply, note that you can always use torch.save and torch.load to save and load the whole model the normal PyTorch way. This isn't super nice because the resulting file will be much larger than only the PEFT adapter, but if that's not a limitation for you, you could give it a try.

Also, I just noticed that the classifier layer is tuned, which I am not sure if it should be according to p_tuning paper.

Note that we generally want the methods we implement to be widely applicable, and not to work only for the tasks mentioned in the paper. If you want to apply p-tuning to a sequence classification task (if that's what you mean), a new classification head has to be added and it needs training for it to work properly.

It's clear to me know. Thank you @BenjaminBossan  for your prompt response.


Discord bot not running

[ ] Automate restarting bot on boot-up of cloud server
[ ] Email upon fail or otherwise notify
[ ] Move discord bot to docker

[ ] logging

@Ninjagod1251 should we create a notification via discord message in the except statement?

[ ]  Automate restarting bot on boot-up of cloud server
cloud server is not rebooting, this is an issue with python script failing.

issue was with python script not running after logout. restarted process in a screen terminal multiplexer instance.

I think this issue is closed. The failure doesn't occur because of the script itself, it failed when we closed the remote terminal. If this script begins to fail due to our actual script, we can open this back up.

closing as per @johnson2427


Ironman QOL

The idea that i have for Ironman is something that i encountered today while on my ironman, I havent had an ironman account for long and Gp for early game food for bosses was always a struggle, i checked in my bank today using =b --sv to see what i could Alch for some good gp to buy some potato's with cheese so i could do some zulrah, I looked at my bank value and it was worth nearly 800m and i was shocked i didnt think i had that much, but that amount is set in Bot sale price not Alch value, so i was hoping Anyone not just Ironman can use the =b --av (Alch Value) just so we can have a visual of what we could alch for some gp.
Thanks,
RagingCaucasian.

ye, would be nice. got my support! +1

I don't see why this shouldn't exist, shouldn't be difficult since alch values are already inputted for items. Maybe even reorder the bank to alch value instead of sacrifice value

I don't see why this shouldn't exist, shouldn't be difficult since alch values are already inputted for items. Maybe even reorder the bank to alch value instead of sacrifice value

This ^^^ Should add that too

I don't see why this shouldn't exist, shouldn't be difficult since alch values are already inputted for items. Maybe even reorder the bank to alch value instead of sacrifice value

I completely agree with @wildchickenz on this and I completely love your idea @RagingCaucasian . you are 100% right I remember looking at a price of something, went to alch it and instantly wish I hadn't lol. wasn't worth it. so u got my vote 1+

=b --alv or --av
(Alch value) would be a brilliant addition. I literally alch what I know alched decently in the main game or junk XD
Thumbed up

Added --av, available in next update.


durable persistence not working

Tried restarting and checking the same value several times in cache by putting a break point in
com.romeh.ignitemanager.repositories.impl.IgniteAlertsStore#createAlertEntry
...
alertsCache.put(alertEntry.getAlertId(), alertEntry)

But, I don't see that the value is found in cache, with persistence to disk enabled, shouldn't it be able to find the cache populated on restart?
I even tried:
alertsCache.query(
        new SqlFieldsQuery("select serviceId from AlertEntry")).getAll()
;

now it is working fine , just clean ur local workspace

could you elaborate more please, what am I cleaning and how do I check that persistence is working?

you just need to check out master in a fresh work space folder the do the following :
1- start the application , open the swagger UI where u can call rest APIs there from the UI without rest client
2- add an alert , then call get all alerts to check if it is added correctly
3- restart your java application from ur IDEA
4- call again get alerts , you should see ur old added entry being returned in the response


acquireTokenByUsernamePassword won't work because client_secret is missing

Core Library
@azure/msal-node
Core Library Version
1.0.3
Wrapper Library
Not Applicable
Wrapper Library Version
n/a
Description
new PublicClientApplication(msalConfig).acquireTokenByUsernamePassword(...) can't get acces_token succesfully because authority service required client_secret, which is not possible to pass.
I know that 'acquireTokenByUsernamePassword' is not recomended way to get access token, but for now, in my case, this is the only solution.
I don't know if this is issue of @azure/msal_node or authority service. I can't find out appropriate documentation about client_secret and Azure AD OAuth2 ROPC.
I've tried to modify by hand source files, so I added in
@azure\msal-common\UsernamePasswordClient.createTokenRequestBody(request) {
var parameterBuilder = new RequestParameterBuilder();
parameterBuilder.addClientSecret(request.clientSecret); // << this line added
...
}
and then it works.
Maybe bug is in authority service and this parameter is unnecessarily required? - I don't know.
Error Message
{
"error": "invalid_client",
"error_description": "AADSTS7000218: The request body must contain the following parameter: 'client_assertion' or 'client_secret'.\r\nTrace ID: 7d3b82f2-1456-41ba-ac90-5086654fb102\r\nCorrelation ID: 51a8fbef-b4ee-4aff-8357-4670dee3edaa\r\nTimestamp: 2021-05-05 06:50:34Z",
"error_codes": [
7000218
],
"timestamp": "2021-05-05 06:50:34Z",
"trace_id": "7d3b82f2-1456-41ba-ac90-5086654fb102",
"correlation_id": "51a8fbef-b4ee-4aff-8357-4670dee3edaa",
"error_uri": "https://login.microsoftonline.com/error?code=7000218"
}
Msal Logs
No response
MSAL Configuration
{
    auth: {
        clientId: 'my-client-id',
        authority: 'https://login.microsoftonline.com/my-tenant-name/'
    },
    graphEndpoint: 'https://graph.microsoft.com/',
}

Relevant Code Snippets
import { AuthenticationResult, ClientCredentialRequest, ConfidentialClientApplication, PublicClientApplication } from "@azure/msal-node";

const msalConfig = {
    auth: {
        clientId: 'my-client-id',
        authority: 'https://login.microsoftonline.com/my-tenant-name/'
    },
    graphEndpoint: 'https://graph.microsoft.com/',
};

const tokenRequest = {
    scopes: [msalConfig.graphEndpoint + '.default'],
};

const pca = new PublicClientApplication(msalConfig);

async function getPcaToken(tokenRequest: ClientCredentialRequest): Promise<AuthenticationResult | null> {
    return await pca.acquireTokenByUsernamePassword({
        scopes: tokenRequest.scopes,
        username: 'my-email',
        password: 'secretPWD',
    });
}

getPcaToken(tokenRequest);

Reproduction Steps
All sample is in 'Relevant Code Snippets' section.
Expected Behavior
Exprect no error result from authority service.
Identity Provider
Azure AD / MSA
Browsers Affected (Select all that apply)
None (Server)
Regression
No response
Source
External (Customer)

I'm having this identical issue with msal-react

Hi,
SOLUTION
to deal with that issue you have to check one option:
Microsoft_AAD_RegisteredApps -> yours app -> Authentication -> Advanced settings -> Allow public client flows -> Enable the following mobile and desktop flows: YES
After that it works.
Returned error message is strange and not helping with that situation.


Decide on initial local LLM server option

[ ] Must be able to run on < 12gb RAM

A smaller model could be leveraged for few-shot, single-token or ngram translation suggestions (e.g., 2-3B params). This is the simplest use case and something we might want to leverage first.
A larger model will be needed for chat, and we need some way to figure out which model the user needs based on their source language settings. This is obviously somewhat fragile, but we can begin to support various gateway languages.
We can actually fire this up through txtai right now to keep things simple, though we will also want to use PredictionGuard for an API version, so when we get to more consistently online users we will likely want a proxy that can gracefully navigate between both.

Hindi

Hindi instruction-tuned LLM

Telugu

abhinand/telugu-llama-7b-instruct-v0.1
eswardivi/llama2chat_telugu
PosteriorAI/godavari-telugu-llama2-7B

Kannada

jayavibhav/Llama-2-Kannada

Tamil

Lots of models here

For now we opted to simply use LM Studio. We can reopen this later if we want to mask this from the user too, but it would take a lot of work to rival LM Studio's flexibility at this point.


Client UI not refreshing

Data displayed in modules on the client app are falling out of date, even though fresh data is being pulled from the API.

This issue is blocked pending completion of #10 in order to test against controlled data.

With the completion of #10, now blocked by Azure Functions dev environment mocks.

Completed with version 0.2.3.


Is this code suitable for data in DCM format?

I haven't tried it, you can try it and give me feedback. My personal opinion is to convert to body data, which may be more beneficial for the alignment of the patches.

I haven't tried it, you can try it and give me feedback. My personal opinion is to convert to body data, which may be more beneficial for the alignment of the patches.


DP_ODQA.ipynb giving error "/usr/local/lib/python3.6/dist-packages/deeppavlov/models/doc_retrieval/logit_ranker.py:65: FutureWarning: LogitRanker.call() API will be changed in the future release. Instead of returning Tuple(List[str], List[float] will return Tuple(List[List[str]], List[List[float]]).   ' Tuple(List[List[str]], List[List[float]]).', FutureWarning)"

Hi,
I am trying to run DP_ODQA.ipynb on colab but I don’t get the final answer at the end but the warning below:
2019-10-20 04:40:26.77 INFO in 'deeppavlov.core.data.utils'['utils'] at line 64: Downloading from http://files.deeppavlov.ai/deeppavlov_data/multi_squad_model_noans_1.1.tar.gz to /root/.deeppavlov/multi_squad_model_noans_1.1.tar.gz
100%|██████████| 265M/265M [01:12<00:00, 3.63MB/s]
2019-10-20 04:41:38.894 INFO in 'deeppavlov.core.data.utils'['utils'] at line 216: Extracting /root/.deeppavlov/multi_squad_model_noans_1.1.tar.gz archive into /root/.deeppavlov/models
2019-10-20 04:41:44.319 INFO in 'deeppavlov.models.preprocessors.squad_preprocessor'['squad_preprocessor'] at line 310: SquadVocabEmbedder: loading saved tokens vocab from /root/.deeppavlov/models/multi_squad_model_noans/emb/vocab_embedder.pckl
2019-10-20 04:41:44.597 INFO in 'deeppavlov.models.preprocessors.squad_preprocessor'['squad_preprocessor'] at line 310: SquadVocabEmbedder: loading saved chars vocab from /root/.deeppavlov/models/multi_squad_model_noans/emb/char_vocab_embedder.pckl
WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/deeppavlov/core/models/tf_model.py:38: The name tf.Session is deprecated. Please use tf.compat.v1.Session instead.
WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/deeppavlov/core/models/tf_model.py:223: The name tf.placeholder is deprecated. Please use tf.compat.v1.placeholder instead.
WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/deeppavlov/core/models/tf_model.py:223: The name tf.train.Optimizer is deprecated. Please use tf.compat.v1.train.Optimizer instead.
WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/deeppavlov/core/models/tf_model.py:194: The name tf.train.AdamOptimizer is deprecated. Please use tf.compat.v1.train.AdamOptimizer instead.
WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/deeppavlov/core/common/check_gpu.py:30: The name tf.ConfigProto is deprecated. Please use tf.compat.v1.ConfigProto instead.
WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/deeppavlov/models/squad/squad.py:223: The name tf.placeholder_with_default is deprecated. Please use tf.compat.v1.placeholder_with_default instead.
WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/deeppavlov/models/squad/squad.py:101: The name tf.get_variable is deprecated. Please use tf.compat.v1.get_variable instead.
Using TensorFlow backend.
2019-10-20 04:41:46.705 INFO in 'deeppavlov.core.layers.tf_layers'['tf_layers'] at line 614:
Warning! tf.contrib.cudnn_rnn.CudnnCompatibleGRUCell is used. It is okay for inference mode, but if you train your model with this cell it could NOT be used with tf.contrib.cudnn_rnn.CudnnGRUCell later.
WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/tensorflow/python/ops/init_ops.py:1251: calling VarianceScaling.init (from tensorflow.python.ops.init_ops) with dtype is deprecated and will be removed in a future version.
Instructions for updating:
Call initializer instance with the dtype argument instead of passing it to the constructor
WARNING:tensorflow:
The TensorFlow contrib module will not be included in TensorFlow 2.0.
For more information, please see:

https://github.com/tensorflow/community/blob/master/rfcs/20180907-contrib-sunset.md
https://github.com/tensorflow/addons
https://github.com/tensorflow/io (for I/O related ops)
If you depend on functionality not listed there, please file an issue.

WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/tensorflow/contrib/cudnn_rnn/python/ops/cudnn_rnn_ops.py:121: GRUCell.init (from tensorflow.python.ops.rnn_cell_impl) is deprecated and will be removed in a future version.
Instructions for updating:
This class is equivalent as tf.keras.layers.GRUCell, and will be replaced by that in Tensorflow 2.0.
WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/deeppavlov/core/layers/tf_layers.py:589: MultiRNNCell.init (from tensorflow.python.ops.rnn_cell_impl) is deprecated and will be removed in a future version.
Instructions for updating:
This class is equivalent as tf.keras.layers.StackedRNNCells, and will be replaced by that in Tensorflow 2.0.
WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/deeppavlov/core/layers/tf_layers.py:594: dynamic_rnn (from tensorflow.python.ops.rnn) is deprecated and will be removed in a future version.
Instructions for updating:
Please use keras.layers.RNN(cell), which is equivalent to this API
WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/tensorflow/contrib/cudnn_rnn/python/ops/cudnn_rnn_ops.py:138: calling Constant.init (from tensorflow.python.ops.init_ops) with dtype is deprecated and will be removed in a future version.
Instructions for updating:
Call initializer instance with the dtype argument instead of passing it to the constructor
WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/tensorflow/contrib/cudnn_rnn/python/ops/cudnn_rnn_ops.py:154: calling Zeros.init (from tensorflow.python.ops.init_ops) with dtype is deprecated and will be removed in a future version.
Instructions for updating:
Call initializer instance with the dtype argument instead of passing it to the constructor
WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/deeppavlov/core/layers/tf_layers.py:808: calling reverse_sequence (from tensorflow.python.ops.array_ops) with seq_dim is deprecated and will be removed in a future version.
Instructions for updating:
seq_dim is deprecated, use seq_axis instead
WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/tensorflow/python/util/deprecation.py:507: calling reverse_sequence (from tensorflow.python.ops.array_ops) with batch_dim is deprecated and will be removed in a future version.
Instructions for updating:
batch_dim is deprecated, use batch_axis instead
2019-10-20 04:41:47.977 INFO in 'deeppavlov.core.layers.tf_layers'['tf_layers'] at line 614:
Warning! tf.contrib.cudnn_rnn.CudnnCompatibleGRUCell is used. It is okay for inference mode, but if you train your model with this cell it could NOT be used with tf.contrib.cudnn_rnn.CudnnGRUCell later.
2019-10-20 04:41:48.126 INFO in 'deeppavlov.core.layers.tf_layers'['tf_layers'] at line 614:
Warning! tf.contrib.cudnn_rnn.CudnnCompatibleGRUCell is used. It is okay for inference mode, but if you train your model with this cell it could NOT be used with tf.contrib.cudnn_rnn.CudnnGRUCell later.
2019-10-20 04:41:48.230 INFO in 'deeppavlov.core.layers.tf_layers'['tf_layers'] at line 614:
Warning! tf.contrib.cudnn_rnn.CudnnCompatibleGRUCell is used. It is okay for inference mode, but if you train your model with this cell it could NOT be used with tf.contrib.cudnn_rnn.CudnnGRUCell later.
WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/deeppavlov/models/squad/utils.py:87: calling dropout (from tensorflow.python.ops.nn_ops) with keep_prob is deprecated and will be removed in a future version.
Instructions for updating:
Please use rate instead of keep_prob. Rate should be set to rate = 1 - keep_prob.
WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/deeppavlov/models/squad/utils.py:171: dense (from tensorflow.python.layers.core) is deprecated and will be removed in a future version.
Instructions for updating:
Use keras.layers.dense instead.
WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/deeppavlov/models/squad/squad.py:211: softmax_cross_entropy_with_logits (from tensorflow.python.ops.nn_ops) is deprecated and will be removed in a future version.
Instructions for updating:
Future major versions of TensorFlow will allow gradients to flow
into the labels input on backprop by default.
See tf.nn.softmax_cross_entropy_with_logits_v2.
WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/deeppavlov/core/models/tf_model.py:51: checkpoint_exists (from tensorflow.python.training.checkpoint_management) is deprecated and will be removed in a future version.
Instructions for updating:
Use standard file APIs to check for files with this prefix.
2019-10-20 04:42:01.357 INFO in 'deeppavlov.core.models.tf_model'['tf_model'] at line 52: [loading model from /root/.deeppavlov/models/multi_squad_model_noans/model]
WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/deeppavlov/core/models/tf_model.py:55: The name tf.train.Saver is deprecated. Please use tf.compat.v1.train.Saver instead.
INFO:tensorflow:Restoring parameters from /root/.deeppavlov/models/multi_squad_model_noans/model
2019-10-20 04:42:02.200 INFO in 'deeppavlov.models.vectorizers.hashing_tfidf_vectorizer'['hashing_tfidf_vectorizer'] at line 264: Loading tfidf matrix from /root/.deeppavlov/models/odqa/enwiki_tfidf_matrix.npz
2019-10-20 04:42:02.734 INFO in 'deeppavlov.dataset_iterators.sqlite_iterator'['sqlite_iterator'] at line 57: Connecting to database, path: /root/.deeppavlov/downloads/odqa/enwiki.db
2019-10-20 04:42:02.736 INFO in 'deeppavlov.dataset_iterators.sqlite_iterator'['sqlite_iterator'] at line 112: SQLite iterator: The size of the database is 300 documents
2019-10-20 04:42:02.742 INFO in 'deeppavlov.models.preprocessors.squad_preprocessor'['squad_preprocessor'] at line 310: SquadVocabEmbedder: loading saved tokens vocab from /root/.deeppavlov/models/multi_squad_model_noans/emb/vocab_embedder.pckl
2019-10-20 04:42:03.65 INFO in 'deeppavlov.models.preprocessors.squad_preprocessor'['squad_preprocessor'] at line 310: SquadVocabEmbedder: loading saved chars vocab from /root/.deeppavlov/models/multi_squad_model_noans/emb/char_vocab_embedder.pckl
2019-10-20 04:42:03.984 INFO in 'deeppavlov.core.layers.tf_layers'['tf_layers'] at line 614:
Warning! tf.contrib.cudnn_rnn.CudnnCompatibleGRUCell is used. It is okay for inference mode, but if you train your model with this cell it could NOT be used with tf.contrib.cudnn_rnn.CudnnGRUCell later.
2019-10-20 04:42:04.130 INFO in 'deeppavlov.core.layers.tf_layers'['tf_layers'] at line 614:
Warning! tf.contrib.cudnn_rnn.CudnnCompatibleGRUCell is used. It is okay for inference mode, but if you train your model with this cell it could NOT be used with tf.contrib.cudnn_rnn.CudnnGRUCell later.
2019-10-20 04:42:04.274 INFO in 'deeppavlov.core.layers.tf_layers'['tf_layers'] at line 614:
Warning! tf.contrib.cudnn_rnn.CudnnCompatibleGRUCell is used. It is okay for inference mode, but if you train your model with this cell it could NOT be used with tf.contrib.cudnn_rnn.CudnnGRUCell later.
2019-10-20 04:42:04.374 INFO in 'deeppavlov.core.layers.tf_layers'['tf_layers'] at line 614:
Warning! tf.contrib.cudnn_rnn.CudnnCompatibleGRUCell is used. It is okay for inference mode, but if you train your model with this cell it could NOT be used with tf.contrib.cudnn_rnn.CudnnGRUCell later.
2019-10-20 04:42:17.382 INFO in 'deeppavlov.core.models.tf_model'['tf_model'] at line 52: [loading model from /root/.deeppavlov/models/multi_squad_model_noans/model]
INFO:tensorflow:Restoring parameters from /root/.deeppavlov/models/multi_squad_model_noans/model
/usr/local/lib/python3.6/dist-packages/deeppavlov/models/doc_retrieval/logit_ranker.py:65: FutureWarning: LogitRanker.call() API will be changed in the future release. Instead of returning Tuple(List[str], List[float] will return Tuple(List[List[str]], List[List[float]]).
' Tuple(List[List[str]], List[List[float]]).', FutureWarning)
Can anyone please reply on this issue?

It is just a warning that in future versions of DeepPavlov LogitRanker API may change.
To get final answer just print value of answers variable after running this cell:
from deeppavlov import configs
from deeppavlov.core.commands.infer import build_model

# Download all the SQuAD models
squad = build_model(configs.squad.multi_squad_noans_infer, download = True)
# Do not download the ODQA models, we've just trained it
odqa = build_model(configs.odqa.en_odqa_infer_wiki, download = False)
answers = odqa(["what is tuberculosis?", "how should I take antibiotics?"])


Problem with 10.0.18362.267 - User changes

Hello,
when I start the remote connection, the user selection comes.
I can then log in with the respective user.
for example User A works on the PC
User B logs on remote.
Then the PC is switched to user B, not logged out.
User A sees what User B does.
When user B logs out, automatically fall back to user A.
Problem has been since the last update.
Is there a solution?
Thanks

Yes, I have Teamviewer 13
Why?

I have problem with TeamViewer 13 on an host, if I RDP into one session, my mouse move onto others sessions and I have weird screen. I must close fully TeamViewer on the host before RDP into.
sorry for my english.

I had the same problem.
Have now imported another termserv.ini and remote desktop connection entered the IP address of the PC.
Now it works strangely ...


luci-app-statistics: fix config of ipstatistics

The config view did not return a configSummary resulting in an error:
plugin.form.configSummary is not a function
This commit fixes the error.

Also, the config does not actually get applied into collectd, as the config generation definition is also missing from /usr/share/luci/statistics/plugins/ipstatistics.json
I am currently authoring and testing my own fix.
(I think that I will also remove the unnecessary config submenu, as there are no options to be selected)

Fix for the issue applied as 5da8a0d2


Discuss: enforce mul_by_inverse

Description
It seems that the mul_by_inverse implementation has a soundness issue that the newly allocated d_inv does not need to be the inverse of d but could be any value. This can be a soundness issue as the poly gadgets have used this API.
The speed improvement seems okay for FpVar, but the constraint needs to be manually written (otherwise, mul + inverse or mul + equal always cause two constraints).
It passes the existing tests.

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (master)
[x] Linked to Github issue with discussion and accepted design OR have an explanation in the PR that describes this work.
[x] Wrote unit tests
[x] Updated relevant documentation in the code
[x] Added a relevant changelog entry to the Pending section in CHANGELOG.md
[x] Re-reviewed Files changed in the Github PR explorer

This one would need a little bit more mental effort to review, as it is not straightforward.

I feel like we previously used to enforce these constraints; was there a reason why we stopped, @ValarDragon ?

I think this issue was noticed in the discussion of https://github.com/arkworks-rs/r1cs-std/pull/42#issuecomment-873289100 but we just forgot to follow up with it.

Nope, I found that this one (the one-constraint version) is still not sound.
When self = 0, d = 0, the code is still satisfiable. And this is problematic because this allows double_and_double in NonZeroAffine to go through if the self and other are the same, but it shouldn't since the result will be incorrect.

It is another thing to discuss: do we want to require the caller to mul_by_inverse to ensure that inverse is not zero? Since this soundness issue, with the change in this PR, is only present now when self = 0, d = 0.
This has an effect on non-zero affine gadgets, as it now requires additional constraints to handle the case of adding the point to itself and doubling-and-adding the point to itself.

Given the discussion above, it seems that all the single-constraint implementation of mul_by_inverse would have soundness issues somewhere.
So, I replaced the mul_by_inverse with the naive implementation of getting the inverse and then multiplying the inverse. This change is somehow needed for the soundness of NonZeroAffine.

Hmm it should not be satisfiable; the constraint we want is that self * d == 1, right? In the prime field case, that can be performed by allocating d, and enforcing the foregoing constraint.

If we require the caller to ensure that d != 0, that also restricts us from being in the situation where self == 0, d== 0.
Definitely looks this has always been missing the mul_equals constraint, RIP :/

I think this also suggests we need a general way of having tests identify allocated vars within a gadget, and try randomly mutating one at a time and ensuring failure.

Following Pratyush's suggestion, I think this:
allocate d_inv
allocate prod = self * d_inv
enforce prod * d = self (via prod.mul_equals(d, self))

has two things:
(1) first, it seems that one does not need to allocate d_inv since it is not being used.
(2) it will be satisfiable when self = 0 and d = 0, in which case prod can be any value.

We may consider mul_by_inverse_unchecked where the developer is responsible to make sure d != 0 otherwise the behavior is undefined.

I will merge this one sooner, and we may need to release a new version (0.3.1) + make a security announcement, based on the discussion in Telegram.
I feel that likely we will let mul_by_inverse stay a simple combination of inverse and multiple, and add a new function mul_by_inverse_unchecked as described above.


Custom constructor properties

Is there a way we can add a custom argument to a class constructor? e.g
@injectable()
class ExampleClass {}

@injectable()
class MyClass {
  constructor(custom: CustomInterface, example: ExampleClass) {}
}

const instance = container.resolve(MyClass, { custom }); // not possible now but to illustrate my point

I need to provide a different custom object to the class on each instantiation. Basically this stackoverflow comment describes perfectly what I mean (quite surprised the library hasn't got this ability already tbh!)

Ok I guess you could do something like
const custom: CustomInterface = {};
container.registerInstance("CustomInterface", { useValue: custom });
const instance = container.resolve(MyClass);

// to clear up instances you've registered with registerInstance()
container.clearInstances();

Would be nice if this was in the docs. Right now I don't see registerInstance() mentioned anywhere in the docs (except for when speaking of clearInstances(). I think adding it to docs and mentioning that this is how one can supply different parameters to constructors for each instances. I don't mind submitting a PR for these doc additions? @Xapphire13

Also is the latest commit to master (which adds .clearInstance() to DependencyContainer typings) published. Even though its in source code in github when I pull in tsyringe from npm (latest which is v4.1.0) its not appearing in dist typings folder

Hey! Thanks for the issue, I'll look into it in the next few weeks =].
You're right, I haven't gotten around to punishing the new version yet. I'm aiming on doing that this upcoming week.

I just released v4.2.0 =]

Thanks man. So to clarify, to provide a different custom object to the class on each instantiation the recommended tsyringe way is to do o=something like
@injectable()
class ExampleClass {}

@injectable()
class MyClass {
  constructor(@inject('custom') custom: string, example: ExampleClass) {}
}

const name: string = 'Hello world';
container.registerInstance("custom", name);
const instance = container.resolve(MyClass);
container.clearInstances();

If resolving a new MyClass with a new custom param is it necessary to call .clearInstances() before registering the new custom instance again?

@Xapphire13 Is there not a better way to pass custom parameters rather than registerInstance? I would think something like
const instance = container.resolve(MyClass, { custom: 'Hello world' });

where u pass the custom param as a second argument to resolve would be much better? Surely this is fundamental oop design?

I guess this already works with @autoinjectable?
it("autoinjectable works with additional 'unmanaged' parameters", () => {
  @singleton()
  class Foo {}

  @autoInjectable()
  class Bar {
    constructor(public custom: any, public foo?: Foo) {}
  }

  // const bar = container.resolve(Bar);
  const bar = new Bar({ x: 5 });
  expect(bar.foo).toBeDefined();
  expect(bar.custom.x).toBe(5);
});```

I think inversifyJS uses the @unmanageddecorator for this.
Something like that would help a lot (e.g. for optional unmanaged parameters in the constructor).

@autoinjectable would not work with child containers.
As a workaround, you can register a "factory" class and use that to create specific instances, ex:
const factory = container.resolve(MyClassFactory); // not possible now but to illustrate my point
const instance = factory.create(custom);

But you need to build manually such a factory and it is a bit of a workaround. The ability to pass unmanaged parameters to resolve would be nice. I would use that. 👍

That's an interesting idea - it's possible (probably) to change the resolve() API to be resolve(token, ...args) or something like that. Sort of like autoInjectable. I'd definitely look at a PR for this.

Just started using tsyringe and find the need for container.resolve(MyClass, ...args) also. Thanks for this library and I hoping someone creates a PR.
@injectable()
class ExampleClass {}

@injectable()
class MyClass {
  constructor(public example: ExampleClass, custom?: CustomInterface) {}
}

const instance1 = container.resolve(MyClass, { custom });

const instance2 = container.resolve(MyClass);

Any updates to this?

Any updates for this?


Add workflow for generating a node-path index table.

Added a new Apache Beam workflow for generating a (node,path) index table for hierarchies. The path is a list of ancestors in order for the given node. e.g. (123,"456.789") where 456 is the parent of 123 and 789 is the grandparent of 123. (111,null) where 111 is a root node.
Generated 2 new node-path index tables for all rows in the concept table in the aou_synthetic and synpuf datasets.
I originally wrote a simpler workflow to recursively call BQ to get the next node in the path for each row (see initial commit). This worked for small numbers of rows, but hit BQ quota limits when running against all rows in the concept table for a given domain_id. The current version of the workflow reads in all parent-child relationships and does this recursion in Beam without calling out to BQ again.
Currently the path is a nullable string field in the output BQ table because we don't support the array data type in Tanagra, yet. Once we do, we can change this field to be a repeated integer.

Sorry for the slow turnaround, I will try to get to this later today.

I assumed that you've been referencing Will's previous work on Beam. However I forgot that some of the work he added was in this library: https://github.com/DataBiosphere/tanagra/blob/main/workflow/src/main/java/bio/terra/tanagra/workflow/GraphUtils.java - hopefully you found this (see also, the collocated tests). Using a similar algorithm to the one used for the transitive closure helper may perform better than this solution - but I don't think it can be reused directly and I don't think it's significant enough to justify reworking it. The pattern of pulling out the main pieces into a helper library is definitely a nice pattern for testability.

Yes, I spent a while going through Will's existing Beam pipeline before writing this one. I've never used Beam or Flume before, so I started with his and modified as I studied Beam tutorials. I also saw the GraphUtils and tests, but couldn't immediately see how to modify it to generate this path instead. I think it's possible, and I'm sure there's a better algorithm for generating these paths, but I wanted to prioritize getting something working over performance for now.
Agreed that pulling some pieces of this out into a helper library would help test this. I'll try to tackle that in a follow-on PR.


SAML enabled UAA prohibits local account logins

If the BOSH UAA component has SAML configuration, there is no way to locally authenticate against UAA.  The workflow causes a range prompts call to set a UI AskForPassword which will require a not null entry.  Because the entry is not null, BOSH will force UAA to use SAML as the code was provided.
Users should be able to login with a local account, even in the presence of SAML.

+sree
interesting.
this sounds like a security feature in uaa -- can only use one source of users. can you auth as local user via uaac when saml is on?
even if not, cli follows prompts abstraction established by uaa (dunno and dont care what im asking, just collect user responses and forward them), so if uaa does allow auth from multiple sources (local and users) it should validate necessary prompts in the uaa server (so still transparent to bosh cli). ie  if saml field is empty assume its local user or something like that. or provide a better categorization of prompts so that cli can only show a subset.
Sent from my iPhone

On Dec 1, 2016, at 8:17 AM, Steven Harms<EMAIL_ADDRESS>wrote:
If the BOSH UAA component has SAML configuration, there is no way to locally authenticate against UAA. The workflow causes a range prompts call to set a UI AskForPassword which will require a not null entry. Because the entry is not null, BOSH will force UAA to use SAML as the code was provided.
Users should be able to login with a local account, even in the presence of SAML.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi,
It depends on what UAA is configured for. It can support one or more
providers at the same time.
You can disable internal authentication through the property below:
uaa.disableInternalAuth:
description: "Disables internal user authentication"
default: false
Dmitriy is right about the UAA prompts. If internal auth is disabled we
don't send back the Username and Password prompts in the info response.
Thanks,
Sree Tummidi
Staff Product Manager
Identity - Pivotal Cloud Foundry
On Thu, Dec 1, 2016 at 8:52 AM, Dmitriy Kalinin<EMAIL_ADDRESS>wrote:

+sree
interesting.
this sounds like a security feature in uaa -- can only use one source of
users. can you auth as local user via uaac when saml is on?
even if not, cli follows prompts abstraction established by uaa (dunno and
dont care what im asking, just collect user responses and forward them), so
if uaa does allow auth from multiple sources (local and users) it should
validate necessary prompts in the uaa server (so still transparent to bosh
cli). ie  if saml field is empty assume its local user or something like
that. or provide a better categorization of prompts so that cli can only
show a subset.
Sent from my iPhone
On Dec 1, 2016, at 8:17 AM, Steven Harms<EMAIL_ADDRESS>wrote:
If the BOSH UAA component has SAML configuration, there is no way to
locally authenticate against UAA. The workflow causes a range prompts
call to set a UI AskForPassword which will require a not null entry.
Because the entry is not null, BOSH will force UAA to use SAML as the code
was provided.
Users should be able to login with a local account, even in the presence
of SAML.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cloudfoundry/bosh-cli/issues/59, or mute the thread
https://github.com/notifications/unsubscribe-auth/AALV95ytn8FCkrgKzTPkapTUReBIxS7Gks5rDvMmgaJpZM4LBlcK
.

UAA is configured for both SAML and internal authentication
With both enabled, bosh-cli does not allow login to internal because of AskForPassword
Ruby cli does not exhibit this behavior

Minor clarification--the ruby cli does incorrectly prompt for an otp when both saml and local accounts are enabled, but it's still possible to use local accounts because the ruby cli accepts empty strings for the otp. The golang cli doesn't accept empty strings here, so it's impossible to login in this situation.

...but it's still possible to use local accounts because the ruby cli accepts empty strings for the otp. The golang cli doesn't accept empty strings here, so it's impossible to login in this situation.

ah. nice find! sounds like we can fix that. it's using go-interact library: https://github.com/cloudfoundry/bosh-cli/blob/master/ui/ui.go#L101

@sharms @jmcarp if it's easy can you check if removing interact.Required(...) wrapper from two lines: [1], [2] allows you to log in?
[1] https://github.com/cloudfoundry/bosh-cli/blob/master/ui/ui.go#L104
[2] https://github.com/cloudfoundry/bosh-cli/blob/master/ui/ui.go#L132

@cppforlife: yep, allowing prompt values to be empty lets us log in again. I sent an example in #62.

Closed via PR#62. Next of version of CLI will have it.


Understanding packet representations and emit

I'm working on @Dirbaio's pull request to make an example that uses Tun (in my case a virtual tun, not a file descriptor based tun) since he already separated the IP from Ethernet.
Here are my commits: https://github.com/Dirbaio/smoltcp/compare/ip-interface...lucaszanella:ip-interface?expand=1 I made it compile at least, now I'm working on how to dispatch to the tun interface.
I've come up with a dispatch_ip function that should do what @Dirbaio'sdispatch_ip does but for a tun device, in my example tun.rs, not a tap one (ethernet.rs in his). Here's the function: https://github.com/lucaszanella/smoltcp/blob/c2050c3023f5e78e5f6b3dca4d20bba52d90e109/src/iface/tun.rs#L689 or simply:
fn dispatch_ip<Tx: TxToken>(&mut self, _tx_token: Tx, _timestamp: Instant,
                      _packet: ip::Packet) -> Result<()> {

My question is simple: can I assume that the ip packet _packet already has the payload and everything inside it? Then in order to create a buffer with the entire IP packet, I'd do:
ip_repr.emit(my_buffer, &caps.checksum);
let payload = &mut frame.payload_mut()[ip_repr.buffer_len()..];
packet.emit_payload(ip_repr, payload, &caps);

and then simply send my_buffer to the tun interface?
I'm asking because on dispatch_ip for ethernet.rs:
fn dispatch_ip<Tx: TxToken>(&mut self, tx_token: Tx, timestamp: Instant,
                          packet: ip::Packet) -> Result<()> {
        let ip_repr = packet.ip_repr().lower(&self.ip_config.ip_addrs)?;
        let caps = self.state.device_capabilities.clone();

        let (dst_hardware_addr, tx_token) =
            self.lookup_hardware_addr(tx_token, timestamp,
                                      &ip_repr.src_addr(), &ip_repr.dst_addr())?;

        self.dispatch_ethernet(tx_token, timestamp, ip_repr.total_len(), |mut frame| {
            frame.set_dst_addr(dst_hardware_addr);
            match ip_repr {
                #[cfg(feature = "proto-ipv4")]
                IpRepr::Ipv4(_) => frame.set_ethertype(EthernetProtocol::Ipv4),
                #[cfg(feature = "proto-ipv6")]
                IpRepr::Ipv6(_) => frame.set_ethertype(EthernetProtocol::Ipv6),
                _ => return
            }

            ip_repr.emit(frame.payload_mut(), &caps.checksum);

            let payload = &mut frame.payload_mut()[ip_repr.buffer_len()..];
            packet.emit_payload(ip_repr, payload, &caps);
        })
    }

it does a series of emits. I don't understand what are these representations and emit things. It looks like it's a way to fill the ethernet packet's payload with the IP stuff then the UDP stuff, for example?
I'm trying to understand the design of the stack. For example, if the IP packet is an UDP packet, then this packet can be emitted into a buffer. This emit process will first emit the IP header, then will call and emit the UDP payload. Like this:
        Packet::Udp((_, udp_repr)) =>
                udp_repr.emit(&mut UdpPacket::new_unchecked(payload),
                              &_ip_repr.src_addr(), &_ip_repr.dst_addr(), &caps.checksum),

Why this emit design was chosen?

This design is necessary to ensure that it is never necessary to buffer a packet before it gets into the PHY buffer. The representation is a compact way to represent the properties of the packet in a way that allows computing its length. The stack asks the PHY to allocate memory for the packet using just the representation, and then writes the packet contents, again using the representation, into the buffe.r
A secondary consequence is that illegal states of the wire-level packet fields cannot be a part of the representation. So the high-level TCP code doesn't have to care about quite as many minor details.

Hello! As part of issue cleanup I'm closing inactive or already-answered "question" issues, since they're not actionable.
Note this doesn't mean questions aren't welcome. They still are! It's just that they shouldn't stay open forever.


Issue with case-sensitive file system

When installing homebridge on a case-sensitive file system it fails to start because it cannot find HAP-NodeJS even though it is installed correctly.
This is because HAP-NodeJS is located in node_modules/hap-nodejs and not node_modules/HAP-NodeJS. After hours of debugging I fixed it by just renaming the hap-nodejs folder but this should probably by fixed in code!?

Added a pull request: https://github.com/nfarina/homebridge/pull/275


Maven option for including Applications symlink into DMG

It would be nice to have a configuration option to include symlink to the Applications folder into DMG file.
Something like createApplicationSymbolicLink parameter in this project:
https://github.com/RomanKisilenko/osxappbundle

Thank you for your comment.
I think it is easy to add that feature.
Of course PR is welcome :smile:

Done. Thanks for @iovuio

I dont see symlink being created for my DMG. Can you comment on what configuration needs to added for this
Below is my plugin build configuration. Info.plist is same as what u have in template
  <plugin>
                <groupId>sh.tak.appbundler</groupId>
                <artifactId>appbundle-maven-plugin</artifactId>
                <version>1.2.0</version>
                <configuration>
                    <mainClass>de.andreasgiemza.mangamanager.MangaManager</mainClass>
                    <generateDiskImageFile>true</generateDiskImageFile>
                    <dictionaryFile>osx/Info.plist</dictionaryFile>
                    <iconFile>images/icon.icns</iconFile>
                    <!--Provide JRE When Target Environment doesn' have it-->
                    <!--<jrePath>/Library/Java/JavaVirtualMachines/jdk1.8.0_92.jdk</jrePath>-->
                </configuration>
                <executions>
                    <execution>
                        <phase>package</phase>
                        <goals>
                            <goal>bundle</goal>
                        </goals>
                    </execution>
                </executions>
            </plugin>

Looking into the code the below configuration worked for me
<configuration>
                    <mainClass>de.andreasgiemza.mangamanager.MangaManager</mainClass>
                    <generateDiskImageFile>true</generateDiskImageFile>
                    <dictionaryFile>osx/Info.plist</dictionaryFile>
                    <iconFile>images/icon.icns</iconFile>
                    <includeApplicationsSymlink>true</includeApplicationsSymlink>
                    <bundleName>MMD</bundleName>
                    <!--Provide JRE When Target Environment doesn' have it-->
                    <!--<jrePath>/Library/Java/JavaVirtualMachines/jdk1.8.0_92.jdk</jrePath>-->
</configuration>


Exec tachikoma update<PHONE_NUMBER>1153

:hamster::hamster::hamster:   Strategy bundler
Powered by Bundle Update as a Service - Tachikoma.io Announcement: 2015-06-28 Updated permissions - Tachikoma.io

Coverage remained the same at 10.798% when pulling 1b0a3cb498a01f8ce45c85206f75bc2e1b7e8979 on pazjacket:tachikoma/update-20171130181153 into 71fe5aba06441dbf8dbc1bca6dd38395cb95d015 on Protom512:master.


mgarch not compatiple with scipy > 1.4.1

after upgrading scipy to version 1.6.0 the mgarch optimization fails:
  File "/home/kic/sources/private/projects/pandas-quant/.nox/tests-3-8/lib/python3.8/site-packages/mgarch/mgarch.py", line 164, in fit
    res = minimize(self.mgarch_logliket, (0.01, 0.94, 3), args = D_t,
  File "/home/kic/sources/private/projects/pandas-quant/.nox/tests-3-8/lib/python3.8/site-packages/scipy/optimize/_minimize.py", line 619, in minimize
    return _minimize_lbfgsb(fun, x0, args, jac, bounds,
  File "/home/kic/sources/private/projects/pandas-quant/.nox/tests-3-8/lib/python3.8/site-packages/scipy/optimize/lbfgsb.py", line 306, in _minimize_lbfgsb
    sf = _prepare_scalar_function(fun, x0, jac=jac, args=args, epsilon=eps,
  File "/home/kic/sources/private/projects/pandas-quant/.nox/tests-3-8/lib/python3.8/site-packages/scipy/optimize/optimize.py", line 261, in _prepare_scalar_function
    sf = ScalarFunction(fun, x0, args, grad, hess,
  File "/home/kic/sources/private/projects/pandas-quant/.nox/tests-3-8/lib/python3.8/site-packages/scipy/optimize/_differentiable_functions.py", line 155, in __init__
    self._update_grad()
  File "/home/kic/sources/private/projects/pandas-quant/.nox/tests-3-8/lib/python3.8/site-packages/scipy/optimize/_differentiable_functions.py", line 231, in _update_grad
    self._update_grad_impl()
  File "/home/kic/sources/private/projects/pandas-quant/.nox/tests-3-8/lib/python3.8/site-packages/scipy/optimize/_differentiable_functions.py", line 151, in update_grad
    self.g = approx_derivative(fun_wrapped, self.x, f0=self.f,
  File "/home/kic/sources/private/projects/pandas-quant/.nox/tests-3-8/lib/python3.8/site-packages/scipy/optimize/_numdiff.py", line 448, in approx_derivative
    raise ValueError("`f0` passed has more than 1 dimension.")
ValueError: `f0` passed has more than 1 dimension.

And it also fails for all the 1.5.x versions

Did this ever work for you?  Seems as if there is a fundamental problem with a 2 dimensional matrix in the minimize function which is causing this error.. I do not think this ever worked???

Did this ever work for you?  Seems as if there is a fundamental problem with a 2 dimensional matrix in the minimize function which is causing this error.. I do not think this ever worked???

yes this works fine with scipy 1.4.1

yes this works fine with scipy 1.4.1

I am confused as to why this is failing now?  do you understand why it is failing ?   so your exact same code no longer runs with the newer versions of scipy?  is the _numdiff function different in the 1.5x versions of scipy?

I am confused as to why this is failing now?  do you understand why it is failing ?   so your exact same code no longer runs with the newer versions of scipy?  is the _numdiff function different in the 1.5x versions of scipy?

Hey, thank you for raising the issue. It will fixed asap

Hey, thank you for raising the issue. It will fixed asap

hello.   is the code fixed?

hello.   is the code fixed?

hello.. hope you are well... any updates.. thanks

hello.. hope you are well... any updates.. thanks

Hi…
How are you doing?   Not sure if the fixes you mentioned have been implemented.   Can you please let me know when you have some time.. thanks
Owner
srivastavaprashant commented 21 days ago •
edited
Hey, thank you for raising the issue. It will be fixed asap
Sent from Mail for Windows 10
From: Prashant Srivastava
Sent: Wednesday, January 13, 2021 6:21 AM
To: srivastavaprashant/mgarch
Cc: larryguido; Comment
Subject: Re: [srivastavaprashant/mgarch] mgarch not compatiple with scipy >1.4.1 (#4)
Closed #4.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi..
Please let me know the status.. Do not see any new code saved… Am I missing something.. Please let me know when you can… thanks
Sent from Mail for Windows 10
From: Larry G
Sent: Wednesday, February 3, 2021 7:44 AM
To: srivastavaprashant/mgarch
Subject: RE: [srivastavaprashant/mgarch] mgarch not compatiple with scipy>1.4.1 (#4)
Hi…
How are you doing?   Not sure if the fixes you mentioned have been implemented.   Can you please let me know when you have some time.. thanks
Owner
srivastavaprashant commented 21 days ago •
edited
Hey, thank you for raising the issue. It will be fixed asap
Sent from Mail for Windows 10
From: Prashant Srivastava
Sent: Wednesday, January 13, 2021 6:21 AM
To: srivastavaprashant/mgarch
Cc: larryguido; Comment
Subject: Re: [srivastavaprashant/mgarch] mgarch not compatiple with scipy >1.4.1 (#4)
Closed #4.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hello.
Have you fixed this code?  Let me know
Sent from Mail for Windows 10
From: Larry G
Sent: Thursday, February 4, 2021 7:51 AM
To: srivastavaprashant/mgarch
Subject: RE: [srivastavaprashant/mgarch] mgarch not compatiple withscipy>1.4.1 (#4)
Hi..
Please let me know the status.. Do not see any new code saved… Am I missing something.. Please let me know when you can… thanks
Sent from Mail for Windows 10
From: Larry G
Sent: Wednesday, February 3, 2021 7:44 AM
To: srivastavaprashant/mgarch
Subject: RE: [srivastavaprashant/mgarch] mgarch not compatiple with scipy>1.4.1 (#4)
Hi…
How are you doing?   Not sure if the fixes you mentioned have been implemented.   Can you please let me know when you have some time.. thanks
Owner
srivastavaprashant commented 21 days ago •
edited
Hey, thank you for raising the issue. It will be fixed asap
Sent from Mail for Windows 10
From: Prashant Srivastava
Sent: Wednesday, January 13, 2021 6:21 AM
To: srivastavaprashant/mgarch
Cc: larryguido; Comment
Subject: Re: [srivastavaprashant/mgarch] mgarch not compatiple with scipy >1.4.1 (#4)
Closed #4.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.


Ignore missing app-operator.giantswarm.io/version labels on CAPI clusters

Changes

Fix skipping app namespaces if the app-operator version was already scraped

Skip collecting app versions for apps in the org-* namespaces
The app-operator.giantswarm.io/version label is not mandatory there (CAPI clusters)
so we do not know the version of the app-operator reconciling these apps. We could check
the app-operator in the same, org namespace but then the validation after that that this
part does makes little sense as if we were able to get the version than the opp-operator
obviously does exist.

Checklist

[ ] Update changelog in CHANGELOG.md.

CircleCI build is passing too: https://app.circleci.com/pipelines/github/giantswarm/app-exporter/1202/workflows/ecfc27bb-de0d-40c1-9481-aaf1f0de5995 (invoked via github action)


only allow sending to its own domain

Is there a toggle, i.e., environment variable option, to enable/disable sending to other domains than the mailserver's own domain? Or do I need to make my own Dockerfile which overwrites the postfix configuration files for that?

Hi @hansbogert, you probably need to override postfix configuration, which does NOT require a custom build.

For reference, the following postfix directive can be used:
default_transport = error:outside mail is not deliverable


Sometimes files copying to RDP session doesn't work

Description
Sometimes when i'm trying to copy file from any folder to my RDP session, file isn't copying. I have no 100%-correct reproduce scenario, but every time that issue associated with Ctrl+C -> Ctrl+V shortcuts.
Steps To Reproduce
There are some cases that i tested:

Open RDP session to another Windows machine;

Select any file and press Ctrl+C;

Open RDP window, select any folder and press Ctrl + V;

File won't copy.

Open RDP session to another Windows machine;

Select any file and press Ctrl+C;

Press Ctrl+V in this folder;

Open RDP window, select any folder and press Ctrl + V;

File will copy.

Open RDP session to another Windows machine;

Select any file and press "Copy" icon on the "header-part" of the Files window;

Open RDP window, select any folder and press Ctrl + V;

File will copy.

Expected behavior
Correct copying to RDP session via shortcuts.
Files Version
Версия: <IP_ADDRESS> OS Version: 10.0.22000.795
Windows Version
Windows 11 21h2 22000.795
Relevant Assets/Logs
debug.log
debug_fulltrust.log

Asking @yaichenbaum for approval.

@bapho-bush which RDP client are you using?

Standard MS RDP client - mstsc.exe

Is this still an issue in the preview version? https://files.community/download/preview

@bapho-bush please let us know if this is still an issue after updating to v2.4.67.

Test it on version <IP_ADDRESS>. Now all works fine.

Thank you for checking


Fixing login, service and link reports path

What is the purpose of this change? What does it change?

sync with other test /reports directory
Was the change discussed in an issue?
no

How to test changes?

login, service and link test reports should be generated in reports/ directory

[odo] Authentication required for https://<IP_ADDRESS>:8443 (openshift)
[odo] Username: Password: Login failed (401 Unauthorized)
[odo] Verify you have provided correct credentials.
[odo]  ✗  challenger chose not to retry the request

/retest

https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/openshift_odo/2166/pull-ci-openshift-odo-master-v4.2-integration/281#0:build-log.txt%3A7000
/retest


Loss

Shouldn't it be reduced_mean instead of reduced_sum when computing the vae and reconstruction loss?

Yes, but if you use reduced_sum for all losses consistently, then it shouldn't matter (mean is basically sum, divided by a constant)


Fix ssr rendering

Should merge #112 before
This fixes the server side rendering in node (#107)

lgtm

Thanks!


More usb fixes

Fix a problem with the atsam3u and a generic problem causing CBW packets to get stalled if they occur at the right time.

@mbed-bot: TEST

@mbed-bot: TEST

@mbed-bot: TEST

@mbed-bot: TEST


Update to Beat Saber v1.13.5

@ToniMacaroni Seems your CI script fails at the GetStrippedRefs step
Run wget --no-check-certificate "$FILES_URL" -q -O bsfiles.zip
Error: Process completed with exit code 1.

PR builds fine.

CI script fails because you don't have the "FILES_URL" secret but that's fine.


Decouple VariantLookup and variant-masking from Primer3

It would be convenient if VariantLookup offered a method to mask arbitrary FASTAs and/or sequences based on the variants it contains.
Currently the logic to apply variant masking is contained within Primer3.get_design_sequences, which makes it difficult to reuse. I think this logic could be extracted and refactored into a method on VariantLookup.

Sounds good to me.


[DNM] [rebranch] Correct nullability of free() redeclaration

In the new clang, this declaration ends up influencing the redeclaration chain, altering the nullability of the free(_:) function imported into Swift.
Fixes rdar://71269128. Note that tests may fail unless a clean build is performed.
I'm not yet certain whether I will merge this to rebranch or to main, but I know that I want a PR test against rebranch.

@swift-ci please smoke test

@swift-ci please smoke test

19:35:26
/home/buildnode/jenkins/workspace/swift-PR-Linux-smoke-test@2/branch-rebranch/swift/lib/IRGen/GenCall.cpp:496:35: error: no member named 'SwiftAsync' in 'llvm::Attribute'
19:35:26   b.addAttribute(llvm::Attribute::SwiftAsync);
19:35:26                  ~~~~~~~~~~~~~~~~~^
19:35:26 /home/buildnode/jenkins/workspace/swift-PR-Linux-smoke-test@2/branch-rebranch/swift/lib/IRGen/GenCall.cpp:1538:33: error: no member named 'SwiftAsyncContext' in 'clang::ParameterABI'; did you mean 'SwiftContext'?
19:35:26       case clang::ParameterABI::SwiftAsyncContext:
19:35:26            ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~
19:35:26                                 SwiftContext
19:35:26 /home/buildnode/jenkins/workspace/swift-PR-Linux-smoke-test@2/branch-rebranch/llvm-project/clang/include/clang/Basic/Specifiers.h:347:5: note: 'SwiftContext' declared here
19:35:26     SwiftContext
19:35:26     ^
19:35:26 /home/buildnode/jenkins/workspace/swift-PR-Linux-smoke-test@2/branch-rebranch/swift/lib/IRGen/GenCall.cpp:1541:12: error: duplicate case value 'SwiftContext'
19:35:26       case clang::ParameterABI::SwiftContext:
19:35:26            ^
19:35:26 /home/buildnode/jenkins/workspace/swift-PR-Linux-smoke-test@2/branch-rebranch/swift/lib/IRGen/GenCall.cpp:1538:12: note: previous case defined here
19:35:26       case clang::ParameterABI::SwiftAsyncContext:
19:35:26            ^

Failure is unrelated.

19:35:26
/home/buildnode/jenkins/workspace/swift-PR-Linux-smoke-test@2/branch-rebranch/swift/lib/IRGen/GenCall.cpp:496:35: error: no member named 'SwiftAsync' in 'llvm::Attribute'
19:35:26   b.addAttribute(llvm::Attribute::SwiftAsync);
19:35:26                  ~~~~~~~~~~~~~~~~~^
19:35:26 /home/buildnode/jenkins/workspace/swift-PR-Linux-smoke-test@2/branch-rebranch/swift/lib/IRGen/GenCall.cpp:1538:33: error: no member named 'SwiftAsyncContext' in 'clang::ParameterABI'; did you mean 'SwiftContext'?
19:35:26       case clang::ParameterABI::SwiftAsyncContext:
19:35:26            ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~
19:35:26                                 SwiftContext
19:35:26 /home/buildnode/jenkins/workspace/swift-PR-Linux-smoke-test@2/branch-rebranch/llvm-project/clang/include/clang/Basic/Specifiers.h:347:5: note: 'SwiftContext' declared here
19:35:26     SwiftContext
19:35:26     ^
19:35:26 /home/buildnode/jenkins/workspace/swift-PR-Linux-smoke-test@2/branch-rebranch/swift/lib/IRGen/GenCall.cpp:1541:12: error: duplicate case value 'SwiftContext'
19:35:26       case clang::ParameterABI::SwiftContext:
19:35:26            ^
19:35:26 /home/buildnode/jenkins/workspace/swift-PR-Linux-smoke-test@2/branch-rebranch/swift/lib/IRGen/GenCall.cpp:1538:12: note: previous case defined here
19:35:26       case clang::ParameterABI::SwiftAsyncContext:
19:35:26            ^

Failure is unrelated.

Added a revert commit which should work around that failure.
@swift-ci please smoke test

Added a revert commit which should work around that failure.
@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

We're hitting unrelated bugs in the LLDB configuration, which prevent us from getting useful results from PR tests.
In local testing, I observed that:

stdlib/TestData.swift passes
llbuildSwift still fails to compile, but it fails by crashing in IRGen, not by a use of free(_:) failing to typecheck.

On this basis, I believe this change is good and should be merged to main branch.

We're hitting unrelated bugs in the LLDB configuration, which prevent us from getting useful results from PR tests.
In local testing, I observed that:

stdlib/TestData.swift passes
llbuildSwift still fails to compile, but it fails by crashing in IRGen, not by a use of free(_:) failing to typecheck.

On this basis, I believe this change is good and should be merged to main branch.


Lint Community Scripts

Lints the community scripts and takes care of the ScriptAnalyzer issues
@denelon
@mdanish-kh
Microsoft Reviewers: Open in CodeFlow

@Trenly did you want to update this one, or merge it and work on @mdanish-kh suggestion/question?

@Trenly did you want to update this one, or merge it and work on @mdanish-kh suggestion/question?

Wait a min and I'll see what I can find

cc @denelon


remove idna req from setup.py/requirements.txt

CATEGORY
Choose one

[X] Bug Fix
[ ] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY
This requirement seems to be no longer required, and it seems it was only added originally to pin its version to resolve conflicts that kept coming up. It is currently causing a conflict in our local installation at Lyft (due to interactions with requests and pydruid).
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

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

REVIEWERS
@betodealmeida @john-bodley @mistercrunch

Codecov Report

Merging #8067 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #8067   +/-   ##
=======================================
  Coverage   65.57%   65.57%
=======================================
  Files         469      469
  Lines       22496    22496
  Branches     2446     2446
=======================================
  Hits        14752    14752
  Misses       7624     7624
  Partials      120      120

Continue to review full report at Codecov.

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


How to set DEBUG to analyze information from pool.

Hi all,
My application is having tons of troubles working with knex, I keep getting connection pool errors, despite usage being low:
export function createKnexConnector(): knex {
return knex({
client: 'oracledb',
connection: {
user: DB_USER,
password: DB_PASSWORD,
host: ${DB_HOST}:${DB_PORT},
database: DB_NAME
},
fetchAsString: ['number', 'clob'],
acquireConnectionTimeout: 120000,
pool: { min: 2, max: 500, idleTimeoutMillis: 15000, evictionRunIntervalMillis: 300000 },
DEBUG: true,
} as any);

}
I have tried many settings and even DEBUG true, but I cannot see any info that would help me out.

Have you tried running app with DEBUG=knex:* environment variable it should output information about acquiring and releasing connections.

export function createKnexConnector(): knex {
return knex({
client: 'oracledb',
connection: {
user: DB_USER,
password: DB_PASSWORD,
host: ${DB_HOST}:${DB_PORT},
database: DB_NAME
},
fetchAsString: ['number', 'clob'],
acquireConnectionTimeout: 120000,
pool: { min: 2, max: 500, idleTimeoutMillis: 15000, evictionRunIntervalMillis: 300000 },
DEBUG=knex:*
} as any);

}
It doesnt validate as a correct connection opening, am I doing something wrong ?

It is environment variable, not knex configuration option. You should set it before running your app.


S3upload (does not clean up existing objects

If objects with the same key/prefix exist in s3 bucket clean up prior to uploading.

No longer relevant


Give an example test file for /etc/buildbot/settings.yaml

Hi! I am planning on adding support for cross compiling and then Emscripten as a platform. However I am unsure how to set up a realistic testing environment.
I can run the current master code, but it seems that the buildbot master relies on a file /etc/buildbot/settings.yaml to configure workers.
I am unsure of what the format of the file should be for testing. Could you give an example of what this file should look like please? (I can guess some of the information from settings.py, but I don't know what the configuration looks like for workers)

Hi @ethanhs,
I can try to give you a sample later today but in theory, you don't need it for what you want. You may be able to just configure a local worker:
https://docs.buildbot.net/latest/manual/configuration/workers.html#local-workers
You can modify locally this:
https://github.com/python/buildmaster-config/blob/15552ea4890e4404dab4732d878dc8872ea1572a/master/master.cfg#L92
To point only to your local worker. It has been a while since I did that in the past so I am sure they may be some gotchas but that's how I have iterated in the past when doing modifications to the worker workflow or plugins.

First off, we should probably adjust https://github.com/python/buildmaster-config/blob/15552ea4890e4404dab4732d878dc8872ea1572a/master/master.cfg#L82 to check a PYBUILDBOT_SETTINGS_PATH environment variable for a replacement path so you don't have to stick a local test config file in /etc/buildbot.  From there, your test settings file could just be use_local_worker: true to use the single local worker, and you can add anything else you need as you find you need it :).  The only worker configuration you might need would be something like:
workers:
    smith-emscripten:
        password: somepassword

Ok, thank you! I will try that out. I do think having a configurable path with an example would be good, because it would prevent mistakes like committing temporary changes to master.cfg.


expand explanation of the ml that supports detection in the ia pipeline

https://docs.wildme.org/docs/researchers/ia_pipeline
Under the detection section, we have three OPTIONAL preparation processes that we can detail much better. This should include:
more expressive description of what is being done
species that have benefited from the tech
assessment methods of what would be valuable (Do you have a species with [blank]? contact us!)
Anything else you think sounds super spiffy important
Page restructure to accommodate these changes is totally fine

Three major parts are

background segmentation
Rotational correction
Parts to bodies assignment


1.22 won't run shell tasks with args

VSCode Version: 1.22
OS Version: OS X 10.13.4

Steps to Reproduce:

Create shell task with command args:

{
        "label":"clean build",
        "type":"shell",
        "command":"tsc && rm",
        "args": ["-f", "./logs/*"],
        "presentation": {
          "echo": true,
          "reveal": "always",
          "focus": false,
          "panel": "shared"
         }
}

Run the task.

Does this issue occur when all extensions are disabled?: Yes
Expected output:

Executing task: tsc && rm -f ./logs/*
Terminal will be reused by tasks, press any key to close it.

Actual output:

Executing task: 'tsc && rm' -f ./logs/* <

/bin/bash: tsc && rm: command not found
The terminal process terminated with exit code: 127

This appears to be a new issue in 1.22. The task ran in 1.21
Changing it to this (integrating the command args into the command) works:
{
        "label":"clean build",
        "type":"shell",
        "command":"tsc && rm -f ./logs/*",
        "presentation": {
          "echo": true,
          "reveal": "always",
          "focus": false,
          "panel": "shared"
         }
}

We fixed bugs and added better support for quoting shell commands and args tasks in 1.22 (see https://code.visualstudio.com/docs/editor/tasks#_custom-tasks and https://code.visualstudio.com/updates/v1_22#_tasks). It was a long wanted support. For a while tasks issued warnings that args with spaces aren't supported and that they should be folded into the command itself. But  forgot about the use case you use of having a combined command with &&. I apologize for this. You must either use one command or use args without spaces. In our case:
"command":"tsc",
"args": [
   "&&", "rm", "-f", "./logs/*"
]

I again apologize for not warning about your case. But reverting this and not having proper quoting support is a step backwards.
Is it OK for you to close the issue?

That works for me. Thanks for the clarification!


can't run apt-get update inside Ubuntu 18.04

root@NetworkAutomation-1:~# cat  /etc/os-release
NAME="Ubuntu"
VERSION="18.04.4 LTS (Bionic Beaver)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 18.04.4 LTS"
VERSION_ID="18.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=bionic
UBUNTU_CODENAME=bionic
What kind of request is this (question/bug/enhancement/feature request):
BUG
Steps to reproduce (least amount of steps as possible):
root@NetworkAutomation-1:~# apt-get update
Result:
Err:1 http://security.ubuntu.com/ubuntu bionic-security InRelease
Connection failed [IP: <IP_ADDRESS> 80]
Err:2 http://ppa.launchpad.net/ansible/ansible/ubuntu bionic InRelease
Connection failed [IP: <IP_ADDRESS> 80]
Err:3 http://archive.ubuntu.com/ubuntu bionic InRelease
Connection failed [IP: <IP_ADDRESS> 80]
Err:4 http://ppa.launchpad.net/deadsnakes/ppa/ubuntu bionic InRelease
Connection failed [IP: <IP_ADDRESS> 80]
Err:5 http://archive.ubuntu.com/ubuntu bionic-updates InRelease
Connection failed [IP: <IP_ADDRESS> 80]
Other details that may be helpful: All the urls referred moved permanently ( 301 )
root@NetworkAutomation-1:~#  curl http://ppa.launchpad.net/deadsnakes/ppa/ubuntu

301 Moved Permanently

Moved Permanently
The document has moved here.

Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):
Installation option (single install/HA):

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported):
Machine type (cloud/VM/metal) and specifications (CPU/memory):
Kubernetes version (use kubectl version):

(paste the output here)

Docker version (use docker version):

(paste the output here)

when I try to update it with curl command it gives me error below
root@NetworkAutomation-1:~# apt-get download http://ppa.launchpad.net/ansible/ansible/ubuntu/dists/bionic/main/debian-installer/binary-amd64/by-hash/
E: Unable to locate package http://ppa.launchpad.net/ansible/ansible/ubuntu/dists/bionic/main/debian-installer/binary-amd64/by-hash
E: Couldn't find any package by glob 'http://ppa.launchpad.net/ansible/ansible/ubuntu/dists/bionic/main/debian-installer/binary-amd64/by-hash'
E: Couldn't find any package by regex 'http://ppa.launchpad.net/ansible/ansible/ubuntu/dists/bionic/main/debian-installer/binary-amd64/by-hash'

Please describe what you are trying to and the steps you are following. Both a clean ubuntu:18.04 or the rancher/rancher container does not include the repositories that are listed. If this is running inside a Kubernetes cluster, try to verify apt-get update works when using Docker only, and if that succeeds, use https://rancher.com/docs/rancher/v2.x/en/troubleshooting/ to determine what's wrong in the cluster.


🏗 Markdown card bug

Is your feature request related to a problem? Please describe.
The Markdown card has a a bug (or the integration).
{%- elif type == "ARRIVE_SAFE_ZONE" -%}
    <ha-alert title="Type: {{ type }}" alert-type="success">ID: {{ msgId }}
***{{ (data.tm / 1000) | timestamp_custom('%Y-%m-%d %H:%M:%S') }}***
{{ trash }}</ha-alert>
    Sender: {{ sender.name }}
    Empfänger: {{ receiver.name }}

    {{ type }}
    {{ data.poi }}
    {{ data.city }}
    {{ data.province }}
    {{ data.country_name }}

At least for me the integration doesn't return 'country_name' for the "ARRIVE_SAFE_ZONE" event. (Maybe for others too?)
This causes the Card to not render at all when you have this event. Removing the last line above will solve it.
Describe the feature you want
Either the Sample card should be changed or the integration should provide the "country_name" attribute.
Additional context
No response

The issue is not related to the card itself, but to the sample you provide.
Either the integration is not returning the country_name anymore and the sample is correct or the sample is wrong and the integration never returned this value.
Anybody who will just user your sample as is will probably run into this issue and you can only find it if you use the HA Template function to find the error message.

In fact, that's an old variable.


(CI) Add ability to use #SR_CLOSE() with several arguments.

Issue is closed automatically by CI. Commit: 3364eaf4bc1b8fbb9b04521b3ded14fc810916b2


ios: appstore doesn't show localizations

When I upload my localized react-native-i18n project to the appstore, it doesn't recognize the localizations. It seems I need to do some manual config in the xcode project as per the README. However, when I click to add a localization, I get a blank list of resources and the locale isn't actually added when I finish.

I'm sure I'm missing a step, any help would be warmly appreciated.

@tonygentilcore Hi, can you provides a screenshot of your Project Infos? (similar to the one in the README)

@zoontek Thanks for getting back to me! Here's a screenshot. I suspect the root issue is that there are "0 Files Localized," but given everything non-trivial about this project is in RN, I'm not sure what I can/should choose to localize in Xcode (or exactly how, really).

Thanks again for taking the time to help.

Even if you are not using it, try adding a new locale, and translate your Info.plist: https://stackoverflow.com/a/25736915
You can keep the InfoPlist.strings blank.

That was exactly it. Thanks again.


Rewrite juicy-carousel to starcounter-composition

It seems that <juicy-carousel> https://github.com/StarcounterPrefabs/Images/blob/develop/src/Images/wwwroot/Images/elements/juicy-carousel/juicy-carousel.html
has no special functionality and it's just HTML structure and set of CSS rules applied in shadow DOM.
If we want just declarative Shadow DOM. plus improve performance (get rid of CE & HTML Import overhead),  code clarity and blending. starcounter-composition is the way to go! :)
It seems that we can simply move entire code into composition :)
// cc @diliger
BTW, It's also not a juicy element ;)

https://github.com/StarcounterPrefabs/Images/pull/62
@diliger up?

can be closed now?

Please see https://github.com/StarcounterPrefabs/Images/pull/62#pullrequestreview-19066665

Issues cannot be closed until PRs that implement them are merged

Done (https://github.com/StarcounterPrefabs/Images/pull/62)


Website: Feedback on the stuff above the fold from Joel.

Before:

After:

Verbal approval from @Onurbon after making some changes.


[NEW RESOURCE]

Updating or New:
New
Name of your plugin:
Report
Creator:
Sorbon
Version of my plugin:
1.1
Raw link:

Raw link to your script.
https://pastebin.com/GyPSXimr
Addons Required:
TuSke
Description:
Minimum 4 sentence description of your resource.
Player can report people for any reason.
All players reports get saved and can be viewed in a GUI.
The GUI is very user-friendly, if you click on the reported player, you can see all the reports that the player received with all infos like who reported him, the date of the report and the reason.
You can remove the reports by clicking on them or remove all of them with the barrier at the bottom.

Contributing has been moved back to our discord.


Can you please help with conversion of pcloud_isoncut.cpp file to mex file?

Hello Ataiya!
Thank you so much for your amazing effort on this work. While I ran the matlab code I realize that pcloud_isoncut.mexmaci file that you have provided does not properly work with newer version of matlab! I would be thrilled if you could convert pcloud_isoncut.cpp file to a mex file working with newer version of matlab and share it with me.
I would deeply appreciate your help.
Best,
Salar

Just got through this myself, there are instructions for the setup in the README.txt file that worked for me

Sorry, haven't touch this code in 10 years, and matlab in 5+
This code is mostly reference, there are more robust techniques around now.
e.g. L1 Median Skeleton for point clouds, Mean Curvature Skeletons for
meshes.
Both have code AFAIK
Cheers

Thank you y'all for your response.
Have a great day!!
Salar
On Fri, Aug 14, 2020 at 10:32 AM Andrea Tagliasacchi <
<EMAIL_ADDRESS>wrote:

Sorry, haven't touch this code in 10 years, and matlab in 5+
This code is mostly reference, there are more robust techniques around now.
e.g. L1 Median Skeleton for point clouds, Mean Curvature Skeletons for
meshes.
Both have code AFAIK
Cheers
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/taiya/rosa/issues/2#issuecomment-674131393, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AMHOAPWKN3VGRCK4NGRVG3DSAVKIPANCNFSM4HSEIDUA
.


Data methods doesn't work inside menu dropdowns

Data methods doesn't work in dropdown menus. I.e. if you have a logout link with method:  : delete it will still send a GET request. This has been fixed in v2.1.1 (wip) of the original Bootstrap. It's a small .js issue.
I will issue a pull request to get this fixed in v2.1-wip of bootstrap-sass.

is this broken again in 3.3.3 for anyone?


jump with a comment

Steps to Reproduce
[First Step]
open a php file
<?php

namespace App\Jobs;

use Illuminate\Bus\Queueable;

abstract class Job
{
    /*
    |--------------------------------------------------------------------------
    | Queueable Jobs
    |--------------------------------------------------------------------------
    |
    | This job base class provides a central location to place any logic that
    | is shared across all of your jobs. The trait included with the class
    | provides access to the "onQueue" and "delay" queue helper methods.
    |
    */

    use Queueable;
}

[Second Step]
press Ctrl+R want  jump into a function
[and so on...]
displayed content　is:
/*-----------
/ Job
/ Line:7
/-----------
/ providers
/ Line:14
/-----------*/

Expected behavior: [What you expected to happen]
no data displayed or only class name displayed
Actual behavior: [What actually happened]
a comment displayed
/*-----------
/  providers
/  Line:14
/-----------*/

Versions
atom 1.9.7

Are you using a non-US keyboard layout?  If not, please reopen this on the atom/symbols-view repository.