tagger: allow arbitrary string as tags, encode them

Tags should be normalized to be used as function name and filename.

In your tests, you use tag names inside emotions.j2. Is it possible to grab them in another template? I would like to do somethings like this:
{% for tag in resource.meta.tags|sort %}
{{ tag.name|e }}
{% endfor %}

Of course, I would need to replace tag.name by something else. But what?

If I understand your question right, the following should work:
{% for tag in resource.tags|sort %}
{{ tag.name|e }}
{% endfor %}

Note the use of resource.tags instead of resource.meta.tags.
The tag objects themselves along with any metadata in the config are assigned to the resource.tags attribute. See here:
https://github.com/hyde/hyde/blob/master/hyde/ext/plugins/tagger.py#L120-122
Hope that answers your question.

Yep, thanks !


What is the kiara store?

Ultimately, I think this discussion needs to end up in a big conceptual docs page which addresses things including but not limited to

why does kiara have a store, why can't it just work with your files on disk? relate this to data lineage and maybe caching?
how does onboarding/import relate to the store?
what does this mean for your computer's storage requirements etc?
what do you need to think about when deciding whether you should store something? When should you store a thing, when shouldn't you
What can you store, what metadata can/should go with the stored item (aliases?), an example of how to do that via the python API
how does the store relate to contexts? there's a separate one per context?
what is the kiara store on a technical level (its a database? what does it do with caching, does it ever clean itself up etc) Where does it live on your disk, what can you do if things go wrong with it

There also needs to be a small summary and a link to this on the glossary page.
see also https://github.com/DHARPA-Project/kiara-website/pull/11#pullrequestreview-1734226183
I don't know the answer to any of these questions. Are there any other things I didn't know to ask about, are there any existing docs about this that I didn't find

Just a lingo clarification on my front: are aliases the equivalent of variables in python? Or something different? (Trying to translate into my 'knowns' to also try and explain beyond this)

Yeah, roughly equivalent I'd say.


feat: add primary config to support missing pages

🔗 Linked issue
This Targets issue #745
❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[x] 👌 Enhancement (improving an existing functionality like performance)
[x] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
This PR will primarily add the pages suggested by next-auth module here
📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have added tests (if possible).
[ ] I have updated the documentation accordingly.
[x] I have added the types required
[ ] I have added the verifyRequest page routing logic
[ ] I have added the error page routing logic

Thoughts
I just started with simple and stupid lines of code.
If this PR would add value to the module, I think we can do something cool.

pnpm issue:

Downloading registry.npmjs.org/next/13.5.6: 17.52 MB/17.52 MB, done
Downloading<EMAIL_ADDRESS>37.18 MB/37.18 MB, done
 ERR_PNPM_EPERM  EPERM: operation not permitted, rename 'C:\Users\Nasr\Desktop\nuxt-auth\node_modules\.pnpm\vue-demi<EMAIL_ADDRESS>e_modules\vue-demi_tmp_584' -> 'C:\Users\Nasr\Desktop\nuxt-auth\node_modules\.pnpm\vue-demi<EMAIL_ADDRESS>
Progress: resolved 1358, reused 1255, downloaded 0, added 0

Nasr@DESKTOP-NCEJSCU MINGW64 ~/Desktop/nuxt-auth (feat/add-missing-pages)
$ pnpm -v
8.15.6


Reduce lifecycle hooks and use of private methods

The componentDidUpdate() lifecycle method was only needed to work around a TypeError: Cannot read property 'detach' of undefined being thrown when a dynamic sheet is added via HMR.
This PR removes componentDidUpdate() and does all its work in componentWillReceiveProps() instead, so it won’t be able to be merged until react-jss stops attempting to detach the nonexistent previous dynamic sheet.
I also removed the call to createState() and instead restore the state that was created in the constructor. One fewer private method being called and it fixes the actual problem more directly.

Relies on cssinjs/react-jss#281


Try a more informative title

Continuing from #1, this is a stab at a better title.
Reminders:

[ ] Run make format and make check to make sure the code follows the style guide.
[ ] Add tests for new features or tests that would have caught the bug that you're fixing.
[ ] Add new public functions/methods/classes to doc/api/index.rst and the base __init__.py file for the package.
[ ] Write detailed docstrings for all functions/classes/methods. It often helps to design better code if you write the docstrings first.
[ ] If adding new functionality, add an example to the docstring, gallery, and/or tutorials.
[ ] Add your full name, affiliation, and ORCID (optional) to the AUTHORS.md file (if you haven't already) in case you'd like to be listed as an author on the Zenodo archive of the next release.

I really like this title!
Unix ❤


Think more carefully about keyboard key identifiers

There needs to be a straightforward way to specify keys (for the keyboard input device, custom tasks with key handling, etc.). Right now strings work ('w' for the w key) but this is limited and probably doesn't work well with other keyboard layouts. Obviously using Qt key IDs would work but I'd really rather not have to import Qt anywhere in user code.

Idea: look at how pytest-qt does this.


sbt.ResolveException with sbt 0.11.2 and sbt-idea-plugin 0.11.1-SNAPSHOT

I recently updated sbt to 0.11.2, and I wanted to use some of the bug fixes that have been applied to sbt-idea-plugin (e.g. using FSC), so I'm using 0.11.1-SNAPSHOT.
In ~/.sbt/plugins/build.sbt I have:
addSbtPlugin("com.github.mpeltonen" % "sbt-idea" % "0.11.1-SNAPSHOT")
When running gen-idea, the idea project is generated, but at the end I'm getting an error:
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	::          UNRESOLVED DEPENDENCIES         ::
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	:: com.github.mpeltonen#sbt-idea;0.11.1-SNAPSHOT: not found
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn]
[warn] 	Note: Some unresolved dependencies have extra attributes.  Check that these dependencies exist with the requested attributes.
[warn] 		com.github.mpeltonen:sbt-idea:0.11.1-SNAPSHOT (sbtVersion=0.11.2, scalaVersion=2.9.1)
[warn]
[error] {file:/.../}.../*:update-sbt-classifiers: sbt.ResolveException: unresolved dependency: com.github.mpeltonen#sbt-idea;0.11.1-SNAPSHOT: not found

This is duplicate of #119


Installing in Angular 4 project

I'm having problems building a project with Angular4.
When installing with npm install --save ngx-restangular it has dependencies in angular5:
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.

But I cannot use angular5 because of other dependencies, any idea how to solve this?
Thank you for this great tool!

I've realized that installing a previous version works fine, but, it don't want to stay too much behind of master release. This is not an issue, of course, but I would like to read an answer!
I installed the 1.0.13 release. To help others with this issue:
npm install --save<EMAIL_ADDRESS>

Hi @byhoratis, thanks for quick reaction :) We've changed respective documentation chapter


Don't hardcode UIDs in securityContext

Signed-off-by: Zbynek Roubalik<EMAIL_ADDRESS>
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[x] A PR is opened to update our Helm chart https://github.com/kedacore/charts/pull/271
[x] Changelog has been updated and is aligned with our changelog requirements

Fixes #3012

/run-e2e


support mmCIF files

Is support for .mmCIF files planned any time in the near future?

It would be a good feature to add. Currently it is not planned for the near-term though.

I think this can be closed :)

hah, good catch!


Add Suicidal and ContentWarning fields

Still need to wire it up to Admin

Could we add stories with these flags set to the preview DB so we can see what they look like in the preview build?


Consider using codecov.io instead of coveralls.io

I've been becoming increasingly frustrated with coveralls. It has some uptime issues, doesn't always pick up reports, and the tooling is not great. codecov.io seems to have more features and a really slick browser plugin that integrates coverage data right onto github. It also seems to have taken some initiative on the tooling front and doesn't require you to conform to their api, they transform standard scoverage reports to work in their system (which means we have one less plugin to rely on and thus one less point of failure). I can't speak to the uptime yet because I haven't been using it very long.
I think we should consider moving in a version or two, especially if we keep having issues with coveralls. I'd encourage you to look at this branch on my fork and let me know what you think. Also definitely check out the browser plugin.

Looks cool! Thanks @rpless! Does it report code coverage statistic per PR? @travisbrown what do you think about switching from Coveralls?

Yes and its configurable for the amount of information you want. Have a look at https://github.com/rpless/finch/pull/2#issuecomment-105517281. Also its opt-in so you have to explicitly tell codecov to do it.

One annoying thing though is each time it gets a build it updates the comment. So for every build in our matrix we would get a notification. So this is my github newsfeed right now:

I'll probably file an issue with them to see whats up with this, but I see it as a con.

We should probably also change the badge in the README file. I just checked out: http://shields.io has badges for codecov.

I'm open to this move, given all the trouble we've had with Coveralls in this and other projects, but ideally we'd be consistent across Finagle org projects (and Twitter projects generally).
The one concern I have is that we'd be relying a little more heavily on a third party with this move. The details of the scoverage reports have been changing a lot (relative paths, etc.), which sometimes breaks Coveralls reporting, but at least we can fix the breakage via the open source sbt-coveralls plugin (or bother the maintainers to fix it). It's handy to be able to rely on Codecov to handle scoverage reports without a plugin, but it also means that if they decide they don't want to support something that scoverage starts doing, we're stuck writing our own plugin from scratch.

I made the README updates.
@travisbrown I think that's a valid concern. Codecov not being open source is definitely a con for us. I'm still on the fence about switching to it, but I thought I'd at least bring it up and see how others felt about.
I talked to their support about not spamming notifications and it seems like its just a github behavior. Rather than repeatedly comment on a PR, codecov will update the comment for each build, each sends notifications.

@travisbrown maybe we should go with codecov fo now? It believe, it's pretty easy to switch back in case of any troubles. At least, codecov reports code coverage per PRs, which I'm missing very much.

Sure, that's fine with me—it'll be a good experiment.

Great! I'm merging this then.

I've just approved the application access request, so we should be ready to go.

The badge currently reads "unknown coverage". Looks like I didn't properly form the shields link. The correct badge should be:

I can make a PR when I get home, unless one of you just wants to slip this into master within the next hour or so.


Add sync command which combines pull and push

In order to sync repositories at the moment, one has to type several commands in a row:
./invoke.py fetch && ./invoke.py set && ./invoke.py pull_all && ./invoke.py push_all

There could be a single command sync/sync_all to implement all steps:
./invoke.py sync_all

Commit 108e15c80fef7d685a32402afae635c60ebcfb8e finished adding sync_all command.


New package: Vcov v0.1.0

Registering package: Vcov
Repository: https://github.com/matthieugomez/Vcov.jl
Created by: @matthieugomez
Version: v0.1.0
Commit: 4840c9f4b953c26cadd4b722048fe023814de1ec
Reviewed by: @matthieugomez
Reference: https://github.com/matthieugomez/Vcov.jl/commit/4840c9f4b953c26cadd4b722048fe023814de1ec#commitcomment-40693652

Closing due to inactivity, please comment if this is still relevant.

No, I would still like to register this package, even though it is only 4
letters.
On Thu, Sep 24, 2020 at 1:10 AM Fredrik Ekre<EMAIL_ADDRESS>wrote:

Closed #17830 https://github.com/JuliaRegistries/General/pull/17830.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JuliaRegistries/General/pull/17830#event-3801941479,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABPPPXIEOOPUAILED6DKZ3TSHL5GJANCNFSM4OX67LHA
.

Just to give some context. This package does not export anything: the types
can be constructed using VCOV.robust() or VCOV.cluster(). In spirit, it is
similar to the api CSV.file().
On Thu, Sep 24, 2020 at 9:47 AM Fredrik Ekre<EMAIL_ADDRESS>wrote:

Reopened #17830 https://github.com/JuliaRegistries/General/pull/17830.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JuliaRegistries/General/pull/17830#event-3804167270,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABPPPXMLZVHJWZF3V22RL53SHNZ2XANCNFSM4OX67LHA
.


Watch node_modules for changes

I want ts-node-dev to watch node_modules/@namespace for changes (those are local symlinked dependencies).
But README says "by default /node_modules/ are ignored".
I tried the following, which is not working:
ts-node-dev --watch node_modules/@namespace -- my-script.ts

Force watching the whole node_modules folder doesn't seem to work either:
ts-node-dev --watch node_modules -- my-script.ts

Any help appreciated.

My bad, resolved with node-dev' flags (--no-deps for own files and linked modules, --all-deps for all dependencies.

My bad, resolved with node-dev' flags (--no-deps for own files and linked modules, --all-deps for all dependencies.


No precise instructions regarding signing keys creation and alternatives

After assigning claims mapping policy to your service principal you will experience the following error during authentication procedure:
"AADSTS50146: This application is required to be configured with an application-specific signing key. It is either not configured with one, or the key has expired or is not yet valid. Please contact the application's administrator."
There is no valid instructions how to pass this error.
Actually there are two options available...
The first option is to create such a key using Azure AD Graph API:
Create a certificate with the private key in PFX format
Convert the PFX file to base-64 encoded file: $fileContentBytes = get-content "file.pfx" -Encoding Byte [System.Convert]::ToBase64String($fileContentBytes) | Out-File "pfxbytes.txt"
Sign-in to Azure AD Graph API using the following link: https://graphexplorer.azurewebsites.net/
Execute the following method: PATCH https://graph.windows.net/myorganization/servicePrincipals/<ObjectID_of_the_service_principal_of_your_app>
{
        "keyCredentials":

[{
              "startDate": "2018-02-22T01:10:00Z",

              "endDate": "2019-02-22T01:10:00Z",

              "type": "X509CertAndPassword",

              "usage": "Sign",

              "keyId": "100C8EC2-0011-490c-86A2-3BF89A708456",

              "value": "Content of pfxbytes.txt"

        }],

        "passwordCredentials":

[{
              "startDate": "2018-02-22T01:10:00Z",

              "endDate": "2019-02-22T01:10:00Z",

              "keyId": "100C8EC2-0011-490c-86A2-3BF89A708456",

              "value": "Password for the PFX file"

        }]

}
Note that "startDate" and "endDate" must match the real dates of the certificate. "keyId" must be the same for both "keyCredentials"and "passwordCredentials" (you can use any GUID generator to provide its value).
The second option is to verify your application domain:
Add and verify your application domain in “Custom domain names” blade of Azure AD.
Domain verification means that your domain must be accessible through the Internet and you must publish a special TXT file with the secret key on this domain, so that Azure AD can check that this domain belongs to you.
Then you should set the switch "acceptMappedClaims": true in your app registration manifest.
Not that it is also possible to use your application without publishing it to a valid Internet domain.
Azure AD always has the default verified domain that belongs to the directory itself
Registering this domain in your “hosts” file will allow you to use your Intranet app.
See https://social.msdn.microsoft.com/Forums/azure/en-US/2507cb82-97a4-4fad-8266-0fffcdbb0642/azure-ad-applicationspecific-signing-key?forum=WindowsAzureAD

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

ID: 195c79a3-ccc5-8c6a-233e-95e344a9b4df
Version Independent ID: 401c7557-5817-bbd3-5bd8-8c97ef5a28cc
Content: Claims mapping in Azure Active Directory (public preview) | Microsoft Docs
Content Source: articles/active-directory/active-directory-claims-mapping.md
Service: active-directory
GitHub Login: @billmath
Microsoft Alias: billmath

@NicklausBrain Thank you for your feedback! We will investigate and get back to you with our findings.

@NicklausBrain  Thank you for your feedback.  I have created a work item for this and will be incorporating your feedback into the re-design of this page.  Currently we are working  on replacing this page with a new version but I do not have an eta on when it will be ready.
Let me know if there is anything else.  Otherwise I will go ahead and close this issue later today.  thanks!

@MohitGargMSFT #please-close


meta-virtualization: Merge with upstream scarthgap branch

This is the periodic currency merge with upstream scarthgap branch until this commit as the next commit fails build.
WI: AB#2468923
Testing

[x] bitbake packagefeed-ni-core
[x] bitbake packagegroup-ni-desirable
[x] bitbake package-index && bitbake nilrt-base-system-image
[x] Installed BSI on a VM and verified it boots

Note to maintainers
Please complete this merge manually to avoid upstream hashes being changed by GH.

@amstewart Don't merge this PR yet - after a merge with scarthgap branch (that brought in more recent commits), build is failing. Investigating this.

@amstewart Don't merge this PR yet - after a merge with scarthgap branch (that brought in more recent commits), build is failing. Investigating this.

Reverted PR to the commit before https://git.yoctoproject.org/meta-virtualization/commit/?h=scarthgap&id=54fdc5149a57b1f614385dbcfc561e2d25b1ea62 as that one fails builds.
I'll create a WI for addressing build failure from that commit. This PR is ready to merge.


Fix license headers and use php-dev-tools

Questions
Answers

Description?
Fix license headers and use php-dev-tools through composer

Type?
new feature

BC breaks?
no

Deprecations?
no

Fixed ticket?

How to test?

Thanks @matks


Inference Error

I managed to run video_demo.py and it is tracking very well but when it reaches frame 201 it crashes with this error:
`
Traceback (most recent call last):
File "/homeMixFormerV2/tracking/video_demo.py", line 53, in
main()
File "/homeMixFormerV2/tracking/video_demo.py", line 48, in main
run_video(args.tracker_name, args.tracker_param, args.videofile, args.optional_box, args.debug,
File "/homeMixFormerV2/tracking/video_demo.py", line 21, in run_video
tracker.run_video(videofilepath=videofile, optional_box=optional_box, debug=debug, save_results=save_results)
File "/homeMixFormerV2/tracking/../lib/test/evaluation/tracker.py", line 230, in run_video
out = tracker.track(frame)
File "/homeMixFormerV2/tracking/../lib/test/tracker/mixformer2_vit_online.py", line 98, in track
out_dict = self.network(self.template, self.online_template, search, softmax=True, run_score_head=True)
File "/home/pc4193/miniconda3/envs/mix2/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
return forward_call(*input, **kwargs)
File "/homeMixFormerV2/tracking/../lib/models/mixformer2_vit/mixformer2_vit_online.py", line 325, in forward
template, online_template, search, reg_tokens, distill_feat_list = self.backbone(template, online_template, search)
File "/home/pc4193/miniconda3/envs/mix2/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
return forward_call(*input, **kwargs)
File "/homeMixFormerV2/tracking/../lib/models/mixformer2_vit/mixformer2_vit_online.py", line 230, in forward
x = torch.cat([x_t, x_ot, x_s, reg_tokens], dim=1)  # (b, hw+hw+HW+4, embed_dim)
RuntimeError: Sizes of tensors must match except in dimension 1. Expected size 1 but got size 2 for tensor number 1 in the list.`
I believe it is doing sth wrong when the template is updated but I could not figure it out, would you help me please with this ?

What's your running script?
I guess it is mainly because that current version of code does not support multiple online templates, so you should set the online_size parameter as 1.

Hi, I have fixed some setting, you can git pull to update the code. And you can refer the new script in run_video_demo.py.
Keep the --param___online_size 1. I'm examing multi-templates version and will release it when ready.

Thanks for your fast reply, it is working now.
I will be waiting for your update


Remove pipeline from matcher initialization API

Issue #, if available:
Need to align some details between matcher and predictor before fully merging them.
Description of changes:

Remove pipeline from the matcher initialization API.
Remove pipeline from infer_metrics(). We still need to know whether it's a matching task. But no longer require the data modalities in matching.
For multiclass and regression problem types, we will use spearman correlation as suggested in papers https://arxiv.org/pdf/1908.10084.pdf, https://arxiv.org/pdf/2010.08240.pdf, and https://arxiv.org/pdf/2004.09813.pdf. This PR adds a metric placeholder for such problem types. Will have following PRs to support the metric in matching.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Is it a temporary solution? We should later unify the design of problem_type.

Is it a temporary solution? We should later unify the design of problem_type.

According to our general design discussions, predictor may need the presets argument to better support various presets. This can also avoid the conflicts between some new problem types and traditional problem types.

Is it a temporary solution? We should later unify the design of problem_type.

PR updated. Now matcher uses only problem_type to specify image_similarity, text_similarity, and image_text_similarity.

We can refactor the implementation further after https://github.com/autogluon/autogluon/pull/2578


fix: exclude top and running queries itself

CI Results
Test Status: ⚠ FLAKY
📊 Full Report

Total
Passed
Failed
Flaky
Skipped

134
133
0
1
0

Bundle Size: ✅
Current: 78.94 MB | Main: 78.94 MB
Diff: 0.07 KB (-0.00%)
✅ Bundle size unchanged.

ℹ CI Information

Test recordings for failed tests are available in the full report.
Bundle size is measured for the entire 'dist' directory.
📊 indicates links to detailed reports.
🔺 indicates increase, 🔽 decrease, and ✅ no change in bundle size.

Follow up issue: https://github.com/ydb-platform/ydb-embedded-ui/issues/1490


field input width characters cropped while writing

hello,
i have an input field with url mask while i am writing a long url
the text  in the field is fixed to the start.
is this a bug or something i can configure?

http://jsfiddle.net/pobo609e/2/   for example


Calling does not work/connect with proxy

The proxy was connected and the user (located in a country where WhatsApp is banned) can Text/Voice Msg but when she calls it does not connect through (can't hear her voice properly).

VOIP (Voice Over IP) is not supported in this release.

Thanks, you mean its not supported by this github method of creating an IP that could be used in WhatsApp OR in the proxy feature that WhatsApp recently added to its app?

VIP simply means making phone calls over the internet. In other words, you won't be able to make phone calls using the WhatsApp proxy in the current version. I believe that feature will be added to future releases of the proxy.

Thank you very much


French Subdivision holidays missing descriptions

More information
Most holidays of subdivisions in France, e.g. the 10th of June in Guyane are missing descriptions.

Hey @XSpielinbox,
I just took a quick look and I would say that not only the descriptions are missing but also the subdivisions are wrong? Should they not be like here, https://de.wikipedia.org/wiki/ISO_3166-2:FR, since 2016?
Auvergne-Rhône-Alpes 	FR-ARA
Bourgogne-Franche-Comté 	FR-BFC
Bretagne 	FR-BRE
Centre-Val de Loire 	FR-CVL
Grand Est 	FR-GES
Hauts-de-France 	FR-HDF
Île-de-France 	FR-IDF
Normandie 	FR-NOR
Nouvelle-Aquitaine 	FR-NAQ
Okzitanien 	FR-OCC
Pays de la Loire 	FR-PDL
Provence-Alpes-Côte d’Azur 	FR-PAC
And also the Départements are no longer written with numbers.


Create PDF viewer

Now, this is maybe quite complicated because we will need to detach from Expo. But it's needed so we can show PDF of the electronic ticket.

investigate possible solution
if it's needed to detach - pick the best PDF viewer first
let's detach in separate issue

WebView seems to be fine for this use-case. I tried it previously but it didn't work. Showed up that it was because of wrong usage of the FileSystem.

Not now.


TextFormField overrides keyboard selection for multiline keyboards

There is no way of using the default 'text' keyboard for multiline text fields. Sometimes this is the wanted behavior, for apps that do not want to allow new lines in multiline text.

[✓] Flutter (Channel master, v0.5.2-pre.18, on Mac OS X 10.13.4 17E202, locale en-US)
    • Flutter version 0.5.2-pre.18 at /Users/vik/Projects/flutter
    • Framework revision 758711c85d (8 hours ago), 2018-05-31 10:33:15 +0200
    • Engine revision d174c4ff01
    • Dart version 2.0.0-dev.59.0.flutter-ff815d05a5

[!] Android toolchain - develop for Android devices (Android SDK 27.0.3)
    • Android SDK at /Users/vik/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-27, build-tools 27.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b08)
    ✗ Android license status unknown.

[✓] iOS toolchain - develop for iOS devices (Xcode 9.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.3.1, Build version 9E501
    • ios-deploy 1.9.2
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    • Dart plugin version 171.4424
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b08)

[✓] Android Studio (version 2.3)
    • Android Studio at /Applications/Android Studio - old.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] IntelliJ IDEA Community Edition (version 2017.2.5)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 20.0.2
    • Dart plugin version 172.4343.25

[✓] Connected devices (1 available)
    • iPhone 6 • 5DE7BCCC-8793-413A-9924-3D413DE12FCD • ios • iOS 9.0 (simulator)

! Doctor found issues in 1 category.

I think this is a dup of https://github.com/flutter/flutter/issues/17525? You basically want t customize the return key to say "Done" in a multiline text field, right?
Please re-open with additional information if it's different from that bug.


Jesse: Watch conveyor.hydraulic.dev

Jesse requests the watch of the watch_keyword conveyor\.hydraulic\.dev. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

conveyor\.hydraulic\.dev has been seen in 2 true positives, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


docs(guide): add Dart forms guide

I don't intend to merge this until next week, but do let me know if you find any problems. In particular, I'm wondering if my approach to the examples (including a copy of every single version) is OK. If not, it'll require some code contortions that'll result in a Frankenstein example that doesn't work correctly.
To see the text diffs from the TS version, go to https://github.com/kwalrath/angular.io/commit/084e4a40a8fe581cccf613c06df5761fae5db63a#diff-80199a70f2fd234f9b3fb0dade4031b0.

@wardbell Is it OK to have many copies of the sample? I'll ask some other questions in comments on the author guide.

I'd prefer that we do not do that. It's pretty confusing. It's also a bear to maintain.
Didn't I avoid that in the TS version?
I have finessed this elsewhere by creating a single sample with a shell page, router, and links to navigate among the variations.  Maybe we should talk about this first?

@wardbell: This might be ready to go.

Hi Kathy. Are you expecting me to do something here? I don't feel competent to merge DART content

I'm happy to merge it if you're happy (enough) with the text style. The content is just a Darty version of the TS content, so I believe it's much better than nothing. I suppose I could ask @yjbanov to review it, but he's probably very busy, so I don't think that's a showstopper.

I thought what I read before was fine and now you've improved it. I say "go for it!"

Great, thanks!


MCO-258: inplaceupgrader: handle multiple upgrade versions

Compared to self-driving OCP, there is an additional scenario to consider: since the ConfigMap contents are synced separately, those will change on the fly in the pod, meaning that we could have a scenario where there are multiple queue'ed updates, in which case the MCD will just jump straight to the latest version.
This will cause the MCD to softlock, so let's make sure for those unavailable nodes, we are also update their desired configuration.
I've tested this alongside https://github.com/openshift/hypershift/pull/1757, so if we want, I can add this commit to that PR, but these should be completely separate issues.

/retest-required

I should have been more clear in my initial message I think, but the process this is trying to fix is this:
let's say I have a NodePool of 3 nodes, currently on config hash aaa and I create a new MC: test-MC-1, which generates config bbb
while the nodepool is updating, let's say I have the following scenario:
1 node is updated to bbb
1 node is updating, and currently Unavailable
1 node is not yet updated, still on aaa
Now I create a new MC test-MC-2, generating ccc. The MCS regens the secret, which we sync into the desired-config-configmap. Now, we no longer have bbb anywhere
Without this PR, what actually happens is:
1 node is updated to bbb
1 node, the updating node, directly updates to ccc instead! But it's desiredConfig is still bbb according to the nodepool controller, since it never "finished" going to bbb and isn't part of the candidate nodes anymore
1 node is still on aaa
And this will lock the nodepool controller into waiting for node 2 to go to bbb, which it never did, since it doesn't know what bbb is anymore, since it's not in the configmap
With this PR, what happens is, the nodepool controller will also update node 2's desiredconfig to ccc instead of waiting on bbb. In the eyes of the nodepool controller, instead of going from aaa->bbb, it's redirecting it to aaa->ccc. The daemonset is able to handle this, and then we complete the update, bypassing the scenario above.

Just curious @enxebre , what happens to the replace upgrade in the scenario above?
I assume you'll probably end up in a scenario like:
4 nodes (1 surged) on: bbb, bbb<-surged new node, aaa, aaa <- to be deleted
into
4 nodes (1 surged) on: bbb, bbb, aaa<- to be deleted, ccc <- surged new node
as in, the nodes don't all go from aaa->bbb first. The surges go directly to the latest, and the old ones are just removed, even ones that were just surged into the (now old) bbb version?

Hmm, did I break the test? Just to check
/test e2e-aws

/test e2e-aws

Thanks @yuqi-zhang I see this is accounting for additional bumps during an already in progress upgrade, I still think we should model it with getCapacity, getAlreadyUnavailableCandidates, getAvailableCandidates ish

I still think we should model it with getCapacity, getAlreadyUnavailableCandidates, getAvailableCandidates ish

Sure. I will refactor that, and try to add some more unit testing.
I originally built this and tested on https://github.com/openshift/hypershift/pull/1757. And since that's the direction we want to go, I'm also fine with waiting for that to get reviewed, before refactoring this and re-testing.
What do you think @enxebre

I've split up the functions and tested this standalone. This should be good to go. The unit tests we have for the overall function should provide coverage of all the use cases.

/test e2e-aws
failure seems unrelated

thanks!
/lgtm
/approve


 For example, the ASCII value of the character 'A' is 65. The ASCII value of the character 'a' is 97. The ASCII value of the character '0' is 48. The ASCII value of the character ' ' is 32.
Input  : 65 66 67
Output : ABC

How to contribute

Save the solution in print_string_of_ascii_values.go file
Add print_string_of_ascii_values.go file in print-string-of-ascii-values folder

!assign

Hey @Chirag8023,
This issue has been assigned to you for more than 15 days.
We are unassigning you from this issue.
If you are still interested in contributing to this issue, please get auto-assigned on the issue by commenting !assign again.
Thanks for your contributions :)


Add mapping from multiple classes

I want to be able to map from more than one class:
mapper.add(FromClassA, FromClassB, ToClassC)

target_obj = mapper.multimap(obj1, obj2)
# or
target_obj = mapper.to(TargetType).multimap(obj1, obj2)

I don't have specific test cases for this and not sure should this should work without producing unexpected behaviour.


[feat] Support dragging pop-out windows back

Currently, the view gets attached to the containerEl. It has the effect that when dragging the sidebar view outside of the main window or into the root split of the main window, that there is no way to drag it back.
I would have made a PR, but as is visible in the second picture, the resizing is off when using contentEl and would need to be adjusted.
Current containerEl implementation:

When using contentEl:

Should be easy to fix, resizing is probably off due to the size of the containerEl being taken instead of contentEl (though will have to check that), will be adding it soon!

I should be a bit more careful with my hubris, it took me a while to find the culprit
Solved the sizing issue and added the contentEl in the latest commits, but will only be releasing once I've triple-checked that the new security measures work, should be later today... if nothing goes wrong

Thank you :purple_heart:

Thanks for implementing it!


Unable to find where to delete default items..?

I'm unable to find where I need to go to delete the default items that mythicdrops spawns. I want to make my own custom common, uncommon, etc items without having pollution of the normal items. I'm going to assume that the fact that I can't find it is just a user error. I'd be surprised if this wasn't possible, after all of this time put into the plugin. Anyways, what I'm wanting to do is delete ALL of the default items and create about 300+ or so of my own. If this isn't possible, then consider this a suggestion. I looked through all of the configs and directories/couldn't find anything, but I'm hoping that I just happened to overlook this feature, or perhaps I'm misunderstanding something. That's all I'm looking for. Thanks.

Are you looking for handcrafted items or randomly generated tiers?

I'm looking to remove all of the defaultly generated items and create my own tiers with my own items to each tier.


PO User Journey Mapping

Describe the issue
Service design need to map the user journey with our PO users, to be able to get the full picture of our user service journey. we decided with  our PO to run it in the PO CoP on the 23rd of August.
Which Sprint Goal is this issue related to?

Note: 'Milestone' is a ZenHub term that we use synonymously with 'Sprint'.

The 'Milestone' description (created in ZenHub) should clearly list Sprint Goals. This section should indicate which 'goal' this issue is related to.
Additional context
the Journey map will help us to:

Visually see our user service journey
Allow us to gather insight on what need to be automized in our service
or enhance some user touchpoint if needed
and it is a start to build on our service blue print

Definition of done

organize a mural board to engage POs in the event
revise and analyze the insights
share with PO

started the PO journey in the PO community of practice and will continue with it when i am back from vacation


All Browsers - Mobile Menu Tab Order

Please name the task using the following information: Browser - Name. Give this issue the "Bug" label.
Description
Tabbing through the mobile menu doesn't currently loop properly - you can tab through the menu and continue down the page when you should be able to cycle through the menu items and the close menu button.
Steps to Reproduce
Steps to reproduce the behavior:

Tabbing on mobile views.

Additional Context
N/A

Looks good. For some reason my tabbing is going into the search history when I get to the address bar. But I'm going to assume that's a browser thing.


Add Rename mapping

This PR adds the option to rename the JSON to avoid naming conflicts regarding reserved names, illegal characters etc.
Would resolve #74

Coverage decreased (-0.5%) to 99.182% when pulling f241dcd86ae8677534497997307849818687b641 on custom-mapping-middleware into 888d8fae86a64714477b71491a965dae8ba97e00 on develop.


GetDelegatedResourceAccountIndexV2 not included (important)

https://developers.tron.network/reference/getdelegatedresourceaccountindexv2-1
GetDelegatedResourceAccountIndexV2 is not included as far as I can tell. Its important and provides easy info: all addresses that delegated TO the requested address AND all addresses that RECEIVED from requested address.
Its important to have this for a resource focused application

Hi! Indeed it can be added. PRs welcome if I not get to it soon. But before that: do you know that any request not "supported" by tronpy is actually supported? There's a generic request func to issue any request possible. That can serve as a base for the convenience method to be added

Hi @MrNaif2018
Thank you for the info. Can you give me more info on how to do that with a generic request func?

See an example here:
https://github.com/andelf/tronpy/blob/c8ebc870ac64afc80dcf764f59668b065555f63f/tronpy/tron.py#L856-L866

@MrNaif2018
thank you. Okay I think my python skills are not sufficient to figure out how to turn this into GetDelegatedResourceAccountIndexV2.
I'll let the professionals handle this 😄
I also didnt intend to spam jfyi, I just thought Id be better for every request to have its own thread.

Yeah, no problem. Just that issues were getting repetitive so it could be turned into a task list really

Hi, I am Oleksandr on upwork

Hi, I am Oleksandr on upwork

So what?

master cricket property surface tooth million wash nominee priority wolf turtle notice,TXNLWRknggM7aq6W8z1YentbuqGV2tuDJg
USDT is frozen. I am angry. It is impossible to give USDT to Tether. If you have the ability, take it.


what would be best way to show plot on fast api?

any progress?
what would be best way to show plot on fast api?

I have not had time to put into this project (I am astounded that github tells me I did this 2 years ago!).
I think this is still a solid proof-of-concept but needs a lot of work before it would be ready for actual use.

I see

If you would like to help with this project I think the first step would be to re-factor the code into a composable router and then sort out how to make sessions work.


[BUG] Better List type repr in dataframes Fixes#994

This removes the intermediate column name and directly uses the result column name which skips the addition of the child field name such as _local_list in the case of list aggregations. This completely omits the display of the local op performed.

Thanks @khandelwalarvind26!
I am a little concerned that removing the intermediate column name might affect the results of the query. Could we not modify the repr logic of the List type directly instead?


pod5 mirroring of fast5 directory

Currently, the pipeline is just outputting one pod5 file. It would be nice for the pod5 directories to mirror the output of the sequencer, i.e. separated into fast5_pass, fast5_fail, and per barcode, as this is often necessary for downstream analysis and totally trivial to parse out of the combined pod5. This could be done with something like the following:
outdir = "/path"

for d in ["fast5_pass", "fast5_fail"]:
    for barcode in os.listdir(d):
        indir = os.path.join(d, barcode)
        call_pod5_recursively(indir, os.path.join(outdir, indir))

This'd be overkill for the following reason:
the sequencer's basecalling is usually using a fast model (and even if it's HAC/super we cannot assume it's the same model as the guppy version on our node). If we adhere to this this could result in a following structure:
sequencer - fast5_pass, fast5_fail
pod5 - pod5_pass, pod5_fail (as defined by the sequencer).
basecalling would then need to be executed on both pod5 directories separately, resulting in a structure like this:
pod5_pass_fastq_pass
pod5_pass_fastq_fail
pod5_fail_fastq_pass
pod5_fail_fastq_fail
One could convert all of them to pod5, and split them up later on. Though I'd opt to keep them as is for the moment. We can revisit this when some actual use case occurs.
Feel free to reopen if I misunderstood.


Update CI python versions

Summary by Sourcery
CI:

Update the CI configuration to test against Python versions 3.9, 3.10, 3.11, 3.12, and 3.13, removing support for versions 3.7 and 3.8.

@sourcery-ai review


UI: Energy is displaying incorrectly

The energy cost displayed on the card description has a bug where it shows the total energy cost plus the required energy cost. This should be fixed as follows:
Current:

Fixed:

Acceptance criteria:

Energy cost should be displayed with the generic energy cost followed by the required energy cost.

Cierren esto tambien


^^^^ expected &Dom, found &&Dom

I want to use the deploy action, but got following error:
 error[E0308]: mismatched types
  --> src/cli/translate/mod.rs:51:47
   |
51 |     let callbody = rsx_rosetta::rsx_from_html(&dom);
   |                    -------------------------- ^^^^ expected `&Dom`, found `&&Dom`
   |                    |
   |                    arguments to this function are incorrect
   |
   = note: expected reference `&rsx_rosetta::Dom`
              found reference `&&html_parser::Dom`
note: function defined here
  --> /home/runner/.cargo/git/checkouts/dioxus-1e619ce595d3799d/f652474/packages/rsx-rosetta/src/lib.rs:10:8
   |
10 | pub fn rsx_from_html(dom: &Dom) -> CallBody {
   |        ^^^^^^^^^^^^^

For more information about this error, try `rustc --explain E0308`.
error: could not compile `dioxus-cli` (lib) due to previous error
warning: build failed, waiting for other jobs to finish...
error: failed to compile `dioxus-cli v0.3.1 (https://github.com/Dioxuslabs/cli#10c9f3db)`, intermediate artifacts can be found at `/tmp/cargo-installW2gE1t`
Error: Process completed with exit code 101.

Link to the build

I think I need upgrade cli version


React & Redux group project

Set up Kanban board
This is the link  to the GitHub project.
https://github.com/EmekaEmmanuel/space_travel/projects/1
This group includes two people:
@jugosack  @EmekaEmmanuel

Hi Team,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Highlights 💯

Descriptive info✔

Optional suggestions
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Make photostream report API consistent with other APIs

We have a convention for mod report API endpoints of /api/v3/:resource/:id/report. In the SiteUI those routes become /:resource/report/:id. The generalized report handlers in Tricordarr don't work with the :id and report URL path components reversed in this API. Since this is a brand new controller I think we should be consistent with what we've done in the past.
This also expressly includes the PhotostreamController in the list of controllers in https://docs.twitarr.com

Need to update the link on line 26 of Sources/swiftarr/Resources/Views/Photostream/photostream.html

@hendricksond  I thought so too initially and it broke when I did. That is POSTing to the Site UI handler, not the API handler.

Need to update the link on line 26 of Sources/swiftarr/Resources/Views/Photostream/photostream.html

@hendricksond I thought so too initially and it broke when I did. That is POSTing to the Site UI handler, not the API handler.

Ahh, gotcha! Should both be updated, for consistency? If not... approved!

They are both consistent with their respective patterns. Which are not consistent with each other 🤣


Ship placement

Set-up of karma and jasmine
Add tests to check that :

Ship's placement is within boundary
Ship isn't overlapping with the ships of the map

Add function resetMap() to bs.map

This pull request is canceled (I pushed directly on the branch ship_placement)


Create references to templates and specializations referenced in type names. [phab:D761]

Created by pcc at 2016-03-09 00:26:16:

pcc wrote at 2016-03-09 01:51:20:

Actually this seems a little simpler and retains instantiations

zarko wrote at 2016-03-10 20:55:14:
Thanks!

zarko accepted this revision at 2016-03-10 20:55:14

zarko closed via commit 0c9aa956b5557591425fc5da44e557c792343bf4 at 2016-03-10 20:55:33


UK phony number normalization doesn't handle cc correctly

In reality:
Phony.normalize('+44 7700 900853', cc: '44') == "44447700900853"

But I expect it to be:
Phony.normalize('+44 7700 900853', cc: '44') == "447700900853"

Duplicate of #270.


<EMAIL_ADDRESS>hash check failed

It seems that the verification information needs to be updated.
Log:
Installing 'SubRenamer' (0.5.0) [64bit] from ygguorun_scoop-bucket bucket
SubRenamer.v<IP_ADDRESS>1-7-10.exe (995.0 KB) [======================================================================] 100%
Checking hash of SubRenamer.v<IP_ADDRESS>1-7-10.exe ... ERROR Hash check failed!
App:         ygguorun_scoop-bucket/SubRenamer
URL:         https://github.com/qwqcode/SubRenamer/releases/download/0.5.0/SubRenamer.v<IP_ADDRESS>1-7-10.exe#/SubRenamer.exe
First bytes: 4D 5A 90 00 03 00 00 00
Expected:    669b31e82f7fda1cd146407acc5645c7a90a0985e625c123560c22ad1908bdb9
Actual:      bf68db2f2d8f1e429cf9451c03b55ac3b810d7e318d8d75456d91d069cf6d43d

Thanks for your feedback.


separation of single/multigene phenotypes

Kim pitched another idea !
He thought we could combine multi and single allele phenotypes (still keep cellular vs population split), and  have a set of filters to display
Only single/ multi
and then other filters as appropriate (conditions), allele type for single gene, etc
We weren't sure  how this would look but it might be worth a try. At present all of the phenotype data is  displayed as a single blob.
Kim- probably the best thing to do here is to  all of the other parts of streamlining the display
and leave the  multi-single split until the end?
I like the idea of being able to see the single and multi gene mutants for a single phenotype simultaneously, but the display will  need to be a bit cleverer.

I don't think i like that idea, I think it will look confusing in some cases.
e.g.
yfg1-delta = sensitive to MMS
yfg2-delta = normal growth on MMS
yfg1-delta + yfg2-delta = normal growth on MMS
in this case, I'm sure that some users will get confused when they land on the yfg1 page and the first thing they see is that it is annotated to normal growth on MMS.
I KNOW they can filter etc, and it won't be a problem for our power users. But I think more users need the information more spoon fed and less in a big table.
I could be wrong but I think what we think is obvious is not obvious to many people (for example, spotting that not all alleles are deletion mutants, seeing that different annotations have different conditions attached etc it seems like mixing the single and multi gene phenotypes might add more confusion to the mix.
Personally I don't mind, and if you want to go with that then that's cool, but I think it might make it harder-to-get for some people (though arguably maybe they should just spend a bit of time learning to navigate the pages better...)

I'm still now sure.
I can see there could be occasions where it  would be useful  to see single gene  and multigene to the same term  simultaneously....
I was hoping that Kims  suggestion would make this possible in a way that would be clear (the multigene filter would need to be as prominent as the current suumar/full  filter for this to worm.
and...I'm still not sure.
Maybe we will be able to judge better  once the other section changes are in place (i.e Kim can leave the splitting  single-multi until last, unless this over complicates the display).

I'm not sure maybe separate sections are the only way to keep these manageable.
Look at the "full" phenotypes for cdc2
http://pombase2.aska.gen.nz/gene/SPBC11B10.09
(its 99 screens in my browser!).
I know this will come down as Kim jiggles with the formatting....but these are still going to be HUGE lists

OK we decided to keep separate sections (I think, at least for now)
Filtering will allow users to look at subsets of data (i.e particular alleles or terms) from both single and multigene simultaneously. If they want to  do any more than simultaneously   then they probably don't require our help ....


add a few packages to the docker

this should make the docker container kinda more useful by itself.
Size grows from ~.25Gi to ~1.8Gi but that's ignorable I'd say.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (9497ad2) to head (ea582ed).

Additional details and impacted files
@@            Coverage Diff            @@
##            master       #75   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           38        38
  Lines          579       579
=========================================
  Hits           579       579

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


Reinstate tests

As the 0.6.5 release showed, we need CI tests (back). Figure out how to do that best.

0f69481e2b5f4550b3cbc0e41392ebaf10e49fd1 has removed the existing tests.


What happens next?

Hi, great article.
I would like to know how will the data be populated in the storage location.
Will it be a copy of the Entity Store.
Will it allow external data also (like other databases?)
Or will it be exactly the same just bigger and faster?
Thanks in advance :)

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

ID: a344a5de-68fa-e9cf-28eb-51269c33b9b3
Version Independent ID: 1b3ede62-3954-574b-5778-8cdee0ededfe
Content: Make Entity store available as a Data Lake
Content Source: articles/fin-ops-core/dev-itpro/data-entities/entity-store-data-lake.md
Service: dynamics-ax-platform
Product: ****
Technology: ****
GitHub Login: @MilindaV2
Microsoft Alias: milindav

@TheMartha, thanks for the feedback. If you need immediate assistance, here are your Support contact options. You might also want to go to the Dynamics 365 community page to get help with your questions from the Dynamics community.
@MilindaV2, please review. Thanks!

Bug 424471 has been logged to track this issue.

Hi team,
I have been in the Dynamics 365 pages - that is why I am asking all this questions :)
Very interested in the topic

Hi, great article.
I would like to know how will the data be populated in the storage location.
Will it be a copy of the Entity Store.
Will it allow external data also (like other databases?)
Or will it be exactly the same just bigger and faster?
Thanks in advance :)
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: a344a5de-68fa-e9cf-28eb-51269c33b9b3
Version Independent ID: 1b3ede62-3954-574b-5778-8cdee0ededfe
Content: Make Entity store available as a Data Lake
Content Source: articles/fin-ops-core/dev-itpro/data-entities/entity-store-data-lake.md
Service: dynamics-ax-platform
Product: ****
Technology: ****
GitHub Login: @MilindaV2
Microsoft Alias: milindav

Once the use of Data Lake in Tier 1 environments is configured, is it possible to activate the Power BI embedded in one box environments?

@MilindaV2 can you please take a look?

We have provided additional information in recent articles under "export to Data lake" section below
https://docs.microsoft.com/en-us/dynamics365/fin-ops-core/dev-itpro/data-entities/azure-data-lake-overview
You can also see demos and an overview of the roadmap in our MBAS session below
https://mymbas.microsoft.com/sessions/46c51e7e-f1bb-4424-8772-13f89a5e3c3b?source=sessions


Printer stopped sending status update events

I have an XPrinter XP-80C printer, plugged into my compputer via USB and set up via a virutal COM port. I have an app that uses the printer, with the following code for setup:
_printer = new SerialPrinter(portName: _port, baudRate: _baudRate);
Thread.Sleep(500);
e = new EPSON();
// Only register status monitoring once.
if (!_hasEnabledStatusMonitoring)
{
_printer.StatusChanged += StatusChanged;
_hasEnabledStatusMonitoring = true;
}
_printer?.Write(e.Initialize());
_printer?.Write(e.Enable());
if (enableStatusBackMonitoring)
{
_printer.Write(e.EnableAutomaticStatusBack());
}
I'm currently developing the app, which means switching it on and off constantly, so the above code runs over and over again. It's been working fine so far (for about a week), but yesterday the status events just stopped coming. (printing still worked as before)
I restarted my app several times but the StatusChanged event was simply never raised. In the end I had to turn the printer off and on again using the power button. This helped, and it's an acceptable solution for development, but if this happens in production I have a serious problem. Also, due to the nature of this issue it's impossible to reliably reproduce it.
Is there anything I can do to make sure this doesn't happen?

You would have to debug the code while facing the said issue.
You can start putting a break point in this method.
Or if the method is never called, go up from there and see where it stops getting the status update and maybe you will figure out why that is happening.


Docs: Missing html files in navigation

Two pages I can find in code and go to html by using url but they are not in nav. Is this desired behavior?
Page URL: https://www.typescriptlang.org/docs/handbook/2/type-declarations.html and https://www.typescriptlang.org/docs/handbook/2/understanding-errors.html
Issue: Two documentation pages missing in nav
Recommended Fix: Probably edit generateDocsNavigationPerLanguage.js file.

Yeah, I think maybe the type-declarations one could live in the declaration files section of the docs, but it was hard to find a space for it and we never came back for it
The understanding error sone never got fleshed out the idea was interesting but it's not good enough for the docs
So neither being in the nav is intentional right now


Fix Javadoc

It is just a small part of fixes in Javadoc (and corresponding code, usually methods signatures). The initial idea was to cleanup everything to remove Maven profile default-jdk18-profile, but there are a lot of javadoc issues.
I'm not sure that I will fix all of them, so it will be useful to hear some feedback about this work. Is it useful or not? I can continue a little bit.

Hi @vbauer, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, AZPRBOT;

:+1: There are somes changes that are not javadoc related, but the changes all look good. Thanks @vbauer

Ok, so I will continue to work on javadocs.

:+1:

I've just pushed additional fixes, but it looks like never-ending story. :) I just want to notify, that I will not be able 16.04-01.05 (vacation). Maybe it will be better to merge current changes to prevent obsolescence. What do you think?

It makes sense to split into multiple change batches and also get in to avoid merge conflicts. I haven't checked latest changes but the first batch looked good.

I've just updated patches and merged conflicts.

I updated patch (squashed it) and merged conflicts.

@vbauer hmm maybe your master is not up-to-date? There are still 2 other commits.

@serious6 Fixed.

:+1: thanks for the contribution.

Thank you for merge!


Add unified email-x render component

Extracts the reusable rendering logic for EmailX ads into a new component.

 Add creativeId to Mindful Compatibility API


Update free-courses-id.md

Hacktoberfest notes:

due to volume of submissions, we may not be able to review PRs that do not pass tests and do not have informative titles.
please read our contributing guidelines
be sure to check the output of Travis-CI for linter errors
if this is your first open source contribution, make sure it's not your last!

What does this PR do?
Add Resource(s)
For resources
Add Kursus Javascript 8 jam untuk pemula,15 Projek Javascript dengan vanilla javascript and Pemrograman berbasis objek Javascript
Why is this valuable (or not)?
This course really help you which is still beginner and unfamiliar with javascript language.
I ensure after you watch this course and do some exercise you will get mid-end javascript skills
How do we know it's really free?
It's public video
For book lists, is it a book? For course lists, is it a course? etc.
It's a video course
Checklist:

[🗸] Not a duplicate
[🗸] Included author(s) if appropriate
[🗸] Lists are in alphabetical order
[🗸] Needed indications added (PDF, access notes, under construction)

Heads up, you can just replace the space with an x in the Pull Request template to make it a checkbox!
There should be two line breaks above new sections, please see the build errors on Travis or read the contributions guidelines for more information there.
Coould you also change this so it's it alphabetical order?

in the Pull Req

Heads up, you can just replace the space with an x in the Pull Request template to make it a checkbox!
There should be two line breaks above new sections, please see the build errors on Travis or read the contributions guidelines for more information there.
Coould you also change this so it's it alphabetical order?

what do you mean about replace space with x ?

Heads up, you can just replace the space with an x in the Pull Request template to make it a checkbox!
There should be two line breaks above new sections, please see the build errors on Travis or read the contributions guidelines for more information there.
Coould you also change this so it's it alphabetical order?

what do you mean about replace space with x ?

[ ] Unchecked * [ ]
[x] Checked * [x]
[🗸] What you did * [🗸]

Hacktoberfest notes:

due to volume of submissions, we may not be able to review PRs that do not pass tests and do not have informative titles.
please read our contributing guidelines
be sure to check the output of Travis-CI for linter errors
if this is your first open source contribution, make sure it's not your last!

What does this PR do?
Add Resource(s)
For resources
Add Kursus Javascript 8 jam untuk pemula,15 Projek Javascript dengan vanilla javascript and Pemrograman berbasis objek Javascript
Why is this valuable (or not)?
This course really help you which is still beginner and unfamiliar with javascript language.
I ensure after you watch this course and do some exercise you will get mid-end javascript skills
How do we know it's really free?
It's public video
For book lists, is it a book? For course lists, is it a course? etc.
It's a video course
Checklist:

[ x ] Not a duplicate
[ x ] Included author(s) if appropriate
[ x ] Lists are in alphabetical order
[  ] Needed indications added (PDF, access notes, under construction)

For youtube, we prefer "playlist" links and the name of the producer/creator. Please no shortened links.

no shortened links, please

no shortened links, please

where is the shortened link part ?? i don't add any shortened link

youtu.be links

can you tell me where it is?? i can't find it

in the react section


xt::linalg::dot doesn't compile when used with xt::pytensor objects

When I use xt::linalg::dot on xt::pytensor objects (from xtensor-python) I get two compilation errors for lines 750 and 841 in xlinalg.h:

xtensor-blas/xlinalg.hpp(841): error C2398: Element '2': conversion from 'const _Ty' to 'T' requires a narrowing conversion
with
[
_Ty=npy_intp
]
and
[
T=size_t
]

This is where a resize is done in xlinalg.hpp:
result.resize({t.shape()[0], o.shape()[1]});
I could temporarily solve it by explicit casting to size_t on those lines 841 and 750, but not sure what the proper solution would be?
This is with xtensor-blas 0.17.1, on Windows with msvc 2017 v15.9:  cl: Microsoft (R) C/C++ Optimizing Compiler Version 19.16.27034 for x64

I think that adding these static_cast is the proper solution. linalg::dot should work with any kind of expressions, and you cannot guarantee that the type used for the size and the shape is always unsigned. Would you like to open a PR?

From here https://github.com/scikit-learn/scikit-learn/wiki/C-integer-types:-the-missing-manual it seems that though generally npy_intp should have the same size as size_t (64-bit on a regular 64-bit unix or windows system) there's no guarantee for that. So there might be a small risk (on more exotic platforms?) to actually narrow the type?
But it seems shape in numpy is always signed so it would be limited to half the size of an (unsigned) xtensor shape anyway, so casting to size_t should be OK, correct?

I guess my question would also be: where is the correct place for this cast: here in xtensor-blas, or somewhere in xtensor-python (so that doing xt::pyarray.shape() has the same return type as xt::xarray.shape()

(By the way: kudos to QuantStack, it's a really nice library)

But it seems shape in numpy is always signed so it would be limited to half the size of an (unsigned) xtensor shape anyway, so casting to size_t should be OK, correct?

Indeed, I don't think there is any issue in static casting to size_t on most platforms. For the exotic ones, let's see if someone opens issue ;)
I definitely think that the cast should go in this library, the shape of pyarray must hold (and return) values of the same type as those of numpy array's shape since pyarray allows to manipulate a numpy array in place.

Ok! Thanks. Will create a PR.
And then try to solve blas linking problems (using conda and cmake,
installed openblas but cmake doesn't find blas for the moment) :-)
On Mon, Mar 9, 2020, 07:20 Johan Mabille<EMAIL_ADDRESS>wrote:

But it seems shape in numpy is always signed so it would be limited to
half the size of an (unsigned) xtensor shape anyway, so casting to size_t
should be OK, correct?
Indeed, I don't think there is any issue in static casting to size_t on
most platforms. For the exotic ones, let's see if someone opens issue ;)
I definitely think that the cast should go in this library, the shape of
pyarray must hold (and return) values of the same type as those of numpy
array's shape since pyarray allows to manipulate a numpy array in place.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/xtensor-stack/xtensor-blas/issues/155?email_source=notifications&email_token=ALSEKY6BCCTWGQ4G5W6LBSDRGT3LDA5CNFSM4LD7T5Z2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOHMQMA#issuecomment-596559920,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALSEKY2S2XORFZ4WENUKK3TRGT3LDANCNFSM4LD7T5ZQ
.

I had this issue too, for an unknown reason conda installs openblas 0.3.6 while it used to install 0.3.7. Forcing the version to 0.3.7 fixred the issue in travis-ci.

Created PR but doesn't seem to be automatically linked to the issue


Added a readme to Introduction to the API and microservices projects

Added a Readme for the introduction to the APIs and Micro services projects to give a base introduction to the projects and learnings on APIs

:sparkles: :sparkles: LGTM. :sparkles: :sparkles:
Thanks @wordaddict
Happy Contributing!


fix(create-vite): target dir contains special characters

Description
refer https://github.com/vitejs/vite/pull/16306

Description
refer https://github.com/vitejs/vite/pull/16306


ApacheDirectoryStudio-2.0.0.v20161101-M12

This fixes the formula, which had been failing before with a 404 error, and updates to a newer version.

[✓] brew cask audit --download Casks/apache-directory-studio.rb is error-free.

$ brew cask audit --download Casks/apache-directory-studio.rb
==> Downloading http://www.us.apache.org/dist/directory/studio/2.0.0.v20161101-M12/ApacheDirectoryStudio-2.0.0.v20161101-M12-macosx.cocoa.x86_64.dmg
Already downloaded: /Users/marca/Library/Caches/Homebrew/Cask/apache-directory-studio--2.0.0.v20161101-M12.dmg
==> Verifying checksum for Cask apache-directory-studio
audit for apache-directory-studio: passed

[✓] brew cask style --fix Casks/apache-directory-studio.rb} reports no offenses.

$ brew cask style --fix Casks/apache-directory-studio.rb

1 file inspected, no offenses detected

[✓] The commit message includes the cask’s name and version.

This formula had been previously failing with a 404:
$ brew cask install apache-directory-studio
==> Downloading http://www.us.apache.org/dist/directory/studio/2.0.0.v20151221-M10/ApacheDirectoryStudio-2.0.0.v20151221-M10-macosx.cocoa.x86_64.tar.gz

curl: (22) The requested URL returned error: 404 Not Found
Error: Download failed on Cask 'apache-directory-studio' with message: Download failed: http://www.us.apache.org/dist/directory/studio/2.0.0.v20151221-M10/ApacheDirectoryStudio-2.0.0.v20151221-M10-macosx.cocoa.x86_64.tar.gz
The incomplete download is cached at /Users/marca/Library/Caches/Homebrew/Cask/apache-directory-studio--2.0.0.v20151221-M10.tar.gz.incomplete

With this, it works:
$ brew cask install apache-directory-studio
==> Downloading http://www.us.apache.org/dist/directory/studio/2.0.0.v20161101-M12/ApacheDirectoryStudio-2.0.0.v20161101-M12-macosx.cocoa.x86_64.dmg
Already downloaded: /Users/marca/Library/Caches/Homebrew/Cask/apache-directory-studio--2.0.0.v20161101-M12.dmg
==> Verifying checksum for Cask apache-directory-studio
==> Moving App 'ApacheDirectoryStudio.app' to '/Applications/ApacheDirectoryStudio.app'
🍺  apache-directory-studio was successfully installed!

formula

We call them casks. They’re difference from formulas, with a difference structure, purpose, and capabilities.
Thank you for the contribution.


fix: Don't show suggestions which are already added when allowUnique is true

fixes https://github.com/react-tags/react-tags/issues/728

:tada: This PR is included in version 6.7.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


The cursor will jump to the front when the value changes

    const updateContent = (): void => {
      editor.value.set(content.value);
    };

    watch(() => props, updateProps, {deep: true});

    watch(() => props.json, updateContent, {deep: true});

Hi!
The problem is solved.
Changes were also made to the data bindings from v-model:json/v-model:jsonString to v-model. Older bindings are still supported for backwards compatibility as well. See updated documentation: https://github.com/bestkolobok/vue3-jsoneditor/blob/main/README.md


SyntaxError: Unexpected token at Parser.pp$5.raise

Input Code

ref project: https://github.com/willin/esnext-pipeline-biolerplate
function doubleSay(str) {
  return `${str}, ${str}`;
}
function capitalize(str) {
  return str[0].toUpperCase() + str.substring(1);
}
function exclaim(str) {
  return `${str}!`;
}

let result = "hello" |> doubleSay |> capitalize |> exclaim;

result |> console.log;
//=> "Hello, hello!"

Babylon/Babel Configuration

{
  "plugins":[
    "transform-pipeline"
  ]
}

https://github.com/SuperPaintman/babel-plugin-syntax-pipeline/blob/master/src/index.js
Expected Behavior

transform pipeline operators
Current Behavior

ref: https://github.com/SuperPaintman/babel-plugin-transform-pipeline/issues/1
/Users/willin/.config/yarn/global/node_modules/babel-core/lib/transformation/file/index.js:590
      throw err;
      ^

SyntaxError: /Users/willin/Desktop/b2/index.js: Unexpected token (11:22)
   9 | }
  10 |
> 11 | let result = "hello" |> doubleSay |> capitalize |> exclaim;
     |                       ^
  12 |
  13 | result |> console.log;
  14 | //=> "Hello, hello!"
    at Parser.pp$5.raise (/Users/willin/.config/yarn/global/node_modules/babylon/lib/index.js:4452:13)
    at Parser.pp.unexpected (/Users/willin/.config/yarn/global/node_modules/babylon/lib/index.js:1761:8)
    at Parser.pp$3.parseExprAtom (/Users/willin/.config/yarn/global/node_modules/babylon/lib/index.js:3748:12)
    at Parser.pp$3.parseExprSubscripts (/Users/willin/.config/yarn/global/node_modules/babylon/lib/index.js:3492:19)
    at Parser.pp$3.parseMaybeUnary (/Users/willin/.config/yarn/global/node_modules/babylon/lib/index.js:3472:19)
    at Parser.pp$3.parseExprOp (/Users/willin/.config/yarn/global/node_modules/babylon/lib/index.js:3433:42)
    at Parser.pp$3.parseExprOps (/Users/willin/.config/yarn/global/node_modules/babylon/lib/index.js:3406:17)
    at Parser.pp$3.parseMaybeConditional (/Users/willin/.config/yarn/global/node_modules/babylon/lib/index.js:3379:19)
    at Parser.pp$3.parseMaybeAssign (/Users/willin/.config/yarn/global/node_modules/babylon/lib/index.js:3342:19)
    at Parser.pp$1.parseVar (/Users/willin/.config/yarn/global/node_modules/babylon/lib/index.js:2340:24)

Your Environment

software
version

Babylon
6.18.0

node
8.4.0

npm
5.3.0

Operating System
os x 10.12.6

Hey @willin! We really appreciate you taking the time to report an issue. The collaborators
on this project attempt to help as many people as possible, but we're a limited number of volunteers,
so it's possible this won't be addressed swiftly.
If you need any help, or just have general Babel or JavaScript questions, we have a vibrant Slack
community that typically always has someone willing to help. You can sign-up here
for an invite.

https://github.com/babel/babylon#faq
If pipeline is at least a Stage 0 proposal we would accept a PR

Babylon only supports standard JavaScript, official proposals, and some popular JavaScript extensions (JSX, Flow and TypeScript).
The pipeline operator isn't in an official proposal yet (it is not listed at https://github.com/tc39/proposals), so Babylon doesn't handle it.

https://github.com/tc39/proposal-pipeline-operator

Yeah it hasn't been proposed at the last meeting and will be at the next one (I'l be there), so we would accept a PR https://github.com/babel/proposals

Babel/Babylon only supports stage 0+ proposal.
The fact that that proposal lives in the TC39 org doesn't mean it is a stage 0 proposal*. You can find the full list here: https://github.com/tc39/proposals/blob/master/stage-0-proposals.md
* I may be wrong, but that is the repo where officials proposals are listed and the pipeline operator isn't there*

Didn't see @hzoo response 😂

All good, you weren't at the meeting or know about it. It was supposed to be presented here but no time https://github.com/tc39/agendas/blob/master/2017/07.md

minor version update should be compatible, this plugin written in 6.13.0

extends babylon like https://github.com/SuperPaintman/babel-plugin-syntax-pipeline/blob/master/src/index.js
cannot work for 6.13.0+ versions? how to make it work, is there any docs to ref? @hzoo


Server-side update operation response should return proper resource location.

Server-side update operation response should return "See Other" code with
proper resource location.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 9 Oct 2010 at 12:34

A 200 return code is probably more approprite, as per discussion with Sven.

Original comment by<EMAIL_ADDRESS>on 21 Oct 2010 at 8:36

Changed title: Server-side update operation response should return proper resource location.

Original comment by<EMAIL_ADDRESS>on 30 Oct 2010 at 4:45

Changed state: Fixed


Why RTCMultiConnection is not gonna be updated?

Muaz hello.  I read today that the project will no longer be updated:
'Please Try 3rd Party Codes Instead...
Try RTCMultiConnection at your own risk!
It is not gonna be updated from now on.
4/17/2019' https://github.com/muaz-khan/RTCMultiConnection/blob/master/README.md
You have done a great job.  Volunteers use your library in our project. Thanks you!
What happened?  Are there vulnerabilities and security issues in the code? Are you closing your project?

Default demos do not work without a TURN-server. That's the issue. Maybe I should provide only github codes and remove herokuapp demos.
The actual code works. But relies on TURN-servers. You know that.
I'll update readme and remove all demos soon.

Thanks for your reply.
Did I understand your problem correctly, that the demo traffic goes through the TURN-server and the service support is expensive?
If so, I understand the problem.

@muaz-khan thanks for this awesome project! 👏 👏
I have a question about TURN-servers and costs. I want use your project for my startup, but I want know more about costs because we used a lot of open source videoconference projects but this is very interesting for the P2P connection between many participants in the video, meaning less costs.
In the description of repository says:

WebRTC JavaScript library for peer-to-peer applications (screen sharing, audio/video conferencing, file sharing, media streaming etc.)

I understand that the main functionality is provide a multi P2P connection with a signaling server for discover connection between users. But is strictly necesary use a TURN server (connection.iceServers)? I mean, this project have the difference between others because not necessarily use TURN servers for videoconferences and can work only with signaling server? I know that the firewalls and networks problems can cause use TURN servers, but I'm very interested in multi P2P connection!
And finally only if you can and want :satisfied: say me the average cost you pay for the TURN servers used in the demos? :pray:
We don't started using it yet but I want to take it in consideration for the new develop! :clap:
Tell me that is possible only with signaling server, because I talk every day about your project! 😄
Can I email you? :love_letter:
Thanks you and keep it up! :muscle:

I want contribute to this project with a new wiki and also create a manage open collective profile! What do you think @muaz-khan ? 😄

In my experience, less than 50% of the transmissions needs a TURN-Server, but nevertheless it's good to have.
In case someone is in need: It runs well on 5$ machines from digital ocean and other providers. At least for development usage.

Default demos do not work without a TURN-server. That's the issue. Maybe I should provide only github codes and remove herokuapp demos.
The actual code works. But relies on TURN-servers. You know that.
I'll update readme and remove all demos soon.

Hi Muaz,
You have done great job all the journey Webrtc and it is being sad that there will be no more updates from you now. Your contributions and efforts are remarkable, and I also wonder why not there is donate button using paypal, we all are happy to contribute as yours demos are really unbeatable


Add CRD OpenAPI validation for Pipeline, PipelineResource, Task

Changes
Generated the OpenAPI validation schema for the

Pipeline CRD
PipelineResource CRD
Task CRD

Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[x] Includes tests (if functionality changed/added)
[x] Includes docs (if user facing)
[x] Commit messages follow commit message best practices

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task and TaskRun to build and release this image

Reviewer Notes
If API changes
are included, additive changes
must be approved by at least two OWNERS
and backwards incompatible changes
must be approved by more than 50% of the OWNERS,
and they must first be added
in a backwards compatible way.
Release Notes
OpenAPI Validation schema added for the Pipeline CRD.

/ok-to-test

@googlebot I signed it!

/ok-to-test

/test pull-tekton-pipeline-integration-tests

Thank you @bobcatfish for your thoughts on this proposal:

did you generate these with a tool,
yes, using the kubegen openapi generator via operator-sdk .

Since params can be either array type or string type, maybe OpenAPI validation can't handle that kind of syntax?

Yes, that scenario did need some manual fixing.
I feel irrespective of whether generators can do a 100% good job - we should use some openapi validation in the CRD because

It helps the user look at the API and understand what it needs
It helps clients/servers validate the contract before creating the object. Today we could potentially create an object with any structure?
Given that we have a pretty neat test suite now, we needn't have a full openapi validation spec for corner cases. We could leave out specific nested areas as object

For attributes of ArrayOrString I removed the validation information from the spec from those fields because there's no good way to do so with the openapi validation spec today.

Thanks for the detailed explanation @sbose78 !
I'm on board with this if you could also include in this PR a doc that explains how to update these (basically the description you already gave above) - probably https://github.com/tektoncd/pipeline/blob/master/DEVELOPMENT.md#adding-new-types would be a good place for it.
Also I think we should:

Get some other OWNERS opinions + @sbwsg @dibyom @abayer @dlorenc @vdemeester @ImJasonH @afrittoli
Make a follow up issue to add some linting around this (so we can catch cases where it should be updated and isnt)

I am also onboard with this but I really wish we could tool a bit of it :angel:

Quick ping: @sbose78 is this something you want to keep working on? (no worries if not!)

I will clean this up later this week, sorry this had gone off my radar. If anyone wishes to take a look at it in the meantime, I'm good as well.

Closing in favor of #1776
/close


feat: add fish_path option

This PR is a continuation of https://github.com/mtoohey31/cmp-fish/pull/2.
This PR consists of multiple, self-contained commits for ease of review.

Awesome, thanks for splitting things up like this! The code looks great and the documentation is really thorough. I've tested this with nvim v0.10.0 and lazy.nvim, and everything works as expected. Merging now!


Convert TileSet Atlas Merge input images to RGBA8 to match output, if needed

closes: https://github.com/godotengine/godot/issues/80870

This fixes a problem that has been bugging me for a while.

Yay

Cherry-picked for 4.1.3.

@YuriSizov is this not included in 4.1.3?
I see you removed the cherrypick label. I'm on 4.1.3 stable and i am still having issues with the atlas merge tool.

Idk why the cherrypick has been removed, the fix was working very well for me.

The label is removed when after the commit is cherry-picked, so it is included in the release. Are the issues you experience the same as fixed by this PR?

@KoBeWi Yes, just experienced the same issue today that was demonstrated in the videos linked as clones of this issue. Trying to merge 2 atlases resulted in a blank atlas.

I tested the MRP from the issue. The bug happens in 4.1.2, but it's fixed in 4.1.3. Your problem is something different, open a new issue if it doesn't exist.

Are you sure your actually using 4.1.3?

Yes.

I meant that for griphen


Control flow gradient problem with tf.float16 datatype

Environment info
Operating System:
Linux
If installed from source, provide

The commit hash (git rev-parse HEAD)
3cb39956e622b322e43547cf2b6e337020643f21
The output of bazel version

Build label: 0.3.1
Build target: bazel-out/local-fastbuild/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Fri Jul 29 09:09:52 2016<PHONE_NUMBER>)
Build timestamp:<PHONE_NUMBER>
Build timestamp as int:<PHONE_NUMBER>

Steps to reproduce
# Minimal example to reproduce the error.
import tensorflow as tf
from tensorflow.contrib.layers import conv2d, batch_norm
sess = tf.InteractiveSession()
is_train_var = tf.placeholder(tf.bool)
# If `tf.float32` is used here, it works flawlessly. Alternatively,
# if a constant instead of a variable is used for `is_training`,
# everything works as expected.
inp = tf.placeholder(tf.float16, [1, 5, 5, 3], name='data')
conv1 = conv2d(inp, 1, 5, padding='VALID',
               normalizer_fn=batch_norm,
               normalizer_params={'is_training': is_train_var})
optimizer = tf.train.MomentumOptimizer(learning_rate=0.1, momentum=0.9)
optimizer.compute_gradients(conv1)

This results in the error
Traceback (most recent call last):
  File "repr.py", line 10, in <module>
    optimizer.compute_gradients(conv1)
  File "/lustre/home/classner/git/tensorflow/_python_build/tensorflow/python/training/optimizer.py", line 253, in compute_gradients
    colocate_gradients_with_ops=colocate_gradients_with_ops)
  File "/lustre/home/classner/git/tensorflow/_python_build/tensorflow/python/ops/gradients.py", line 478, in gradients
    in_grads = _AsList(grad_fn(op, *out_grads))
  File "/lustre/home/classner/git/tensorflow/_python_build/tensorflow/python/ops/control_flow_grad.py", line 69, in _SwitchGrad
    return merge([good_grad, zero_grad], name="cond_grad")[0], None
  File "/lustre/home/classner/git/tensorflow/_python_build/tensorflow/python/ops/control_flow_ops.py", line 361, in merge
    return gen_control_flow_ops._merge(inputs, name)
  File "/lustre/home/classner/git/tensorflow/_python_build/tensorflow/python/ops/gen_control_flow_ops.py", line 153, in _merge
    result = _op_def_lib.apply_op("Merge", inputs=inputs, name=name)
  File "/lustre/home/classner/git/tensorflow/_python_build/tensorflow/python/framework/op_def_library.py", line 437, in apply_op
    raise TypeError("%s that don't all match." % prefix)
TypeError: Tensors in list passed to 'inputs' of 'Merge' Op have types [float16, float32] that don't all match.

What have you tried?
I am currently using constant variables as a workaround as described.

Should be fixed at HEAD by now.

Closing since it should be fixed. Feel free to open a new github issue if the problem still persists in recent versions.


Add a UDP server to gRPC

Preparation for experimentation with QUIC as a transport.
ctiller: This is the result of our discussion over email. Structure is very similar to the existing tcp_server code.
This pull request isn't quite ready for review: the test_receive test hangs at the destroy_pollset call. I'd appreciate it if you could take a look at how I've wired things up, and see if you can spot what's going on.
I'm also unclear if the read callback/write methods in udp_server.h look sensible, PTAL.
Thanks!

Can one of the admins verify this patch?

My local gitconfig was incorrect, wrong email address. I think this is just going to cause problems with associating with my account, and the CLA bot checks. Closing this pull request, and will open a new one with the correct email. Sorry for the noise.


Add new dependency for openslide 4.0.0 on libdicom

xref: https://github.com/conda-forge/openslide-feedstock/pull/25
cc: @kaczmarj
Checklist

[ ] Used a static YAML file for the patch if possible (instructions).
[ ] Only wrote code directly into generate_patch_json.py if absolutely necessary.
[ ] Ran pre-commit run -a and ensured all files pass the linting checks.
[ ] Ran python show_diff.py and posted the output as part of the PR.
[ ] Modifications won't affect packages built in the future.

Was this dependency also added to the package somewhere?

Also thanks for looking into this Mark! 🙏

Yes, new dependency in version 4.0.0, but we didn't have a run_export in the package. nor a test in the binary to catch it.

Ah ok. That context helps. Thanks Mark 🙏
So the steps were:

Add a run_export to libdicom ( https://github.com/conda-forge/libdicom-feedstock/pull/5 )
Patch the existing openslide packages

Should we add libdicom as a run dependency in openslide as well (just in case an older libdicom package without the run_export is used for the build)?

Should we add libdicom as a run dependency in openslide as well (just in case an older libdicom package without the run_export is used for the build)?

no a test is being added instead.

Ok thanks! Updated the summary above. Does that look better

yes.


Environment overrides are logged and don't follow zap config

We consolidate all logs to Datadog for analysis, this requires formats to be ingestible and there are no secrets/personal information in logs
In testing rk-boot, any environment or flag based config overrides are always logged out as an info log, this exposes any secrets being pushed down from the environment
https://github.com/rookie-ninja/rk-entry/blob/fb172059dd7f7f78e34a1bc53ab1def544b1067b/entry/util.go#L352
At the very least this should be a dev only level log.
They also ignore any Zap log configs as they are shipped before the log configuration is parsed. This is good during failures so the config can be diagnosed but bad as it breaks standardised parsing. This would be less of an issue if they weren't created in non-development environments.

@dave-lang Sounds reasonable. The log level should be dev only as you mentioned. I will fix this issue.

@dongxuny Thank you


Wrong App import in App.story.js

Within App.story.js App is wrongly imported.

import { App } from '../components';

Should be:
import App from '../components';

fixed


Dns e2e

Checklist

I applicable, I acknowledge that I have:

[ X] Read the contributing guide
[ X] Ran terraform fmt on all modified files
[ X] Regenerated the relevant README.md files using tools/tfdoc.py
[ X] Made sure all relevant tests pass

Skipped e2e test for peering example due to the dependency with additional VPC.


IP forwarding setup failing in the container

I've just set up the container in my home network and noticed that while I can connect to the container shell from other devices in my Tailscale network, I can't access the TS network from devices in my home network. Traceroute shows that packets reach the container and get stuck there:
traceroute to <IP_ADDRESS> (<IP_ADDRESS>), 64 hops max, 52 byte packets
 1  <IP_ADDRESS> (<IP_ADDRESS>)  3.070 ms  2.043 ms  3.423 ms
 2  <IP_ADDRESS> (<IP_ADDRESS>)  5.508 ms  3.917 ms  3.819 ms
 3  * * *

I've connected to the container and executed tailscale status to check what's happening. The healthcheck there reports the following issue:
# Health check:
#     - router: adding [-i tailscale0 -j MARK --set-mark 0x40000/0xff0000] in v4/filter/ts-forward: running [/sbin/iptables -t filter -A ts-forward -i tailscale0 -j MARK --set-mark 0x40000/0xff0000 --wait]: exit status 1: iptables: Invalid argument. Run `dmesg' for more information.

While it doesn't seem possible to access dmesg there, I verified that iptables are, indeed, quite empty:
rt1:~# iptables -L
Chain INPUT (policy ACCEPT)
target     prot opt source               destination
ts-input   all  --  anywhere             anywhere

Chain FORWARD (policy ACCEPT)
target     prot opt source               destination
ts-forward  all  --  anywhere             anywhere

Chain OUTPUT (policy ACCEPT)
target     prot opt source               destination

Chain ts-forward (1 references)
target     prot opt source               destination

Chain ts-input (1 references)
target     prot opt source               destination
RETURN     all  --  <IP_ADDRESS>/23      anywhere
DROP       all  --  <IP_ADDRESS>/10        anywhere
ACCEPT     all  --  anywhere             anywhere

Let me know if I can provide any more data to help debug the issue.
Thanks!

Hi,
that's correct. I can reach other Tailscale devices from within the Tailscale container, and seemingly home network devices from other Tailscale-connected devices, but traffic from non-Tailscale devices on the home network doesn't reach the Tailscale network.

Hi,
That's by design... see https://tailscale.com/kb/1019/subnets
Subnet routers act as a gateway, relaying traffic from your Tailscale network onto your physical subnet. You would need to install Tailscale on any device intending to access the Tailnet.
Otherwise you would have non-authenticated devices accessing your secure Tailnet.

Hm, in that case, what's the purpose of the route to the Tailscale network?
/ip/route/add dst-address=<IP_ADDRESS>/10 gateway=<IP_ADDRESS>

Routing return traffic from the non-Tailscale devices back to the tailnet.
For example, if a tailscale host pings a non-tailscale device on the home network, the non-tailscale device sees the source address as 100.x.x.x and returns an ICMP response via the default gateway on your router. The above route enables the router to send the response packet to the container as the next hop.


Cleanup

Is this type of cleanup you asked?... GetFromJson deserialize fails tho...

Hi mate, I just left few comments as suggestions - tell me what you think. Otherwise perfect, thanks a lot :)


Audio download gets throttled to 31 KB/s when the download is done with the URL for Itag = 140

Hi,
I am testing pytubefix and I am facing the same issue existing at pytube 15.0.0.
If I extract the URL to download the audio (ITAG = 140) to download the media with my browser, the download speed gets throttled by Youtube to 31 KB/s.
The problem does not happen to download videos (ITAG = 22).
Here is the code to test the issue:
from pytubefix import YouTube    # https://pypi.org/project/pytubefix/

video_link = f'https://www.youtube.com/watch?v=0FX4jVAL1eg'
yt = YouTube(video_link)
title = yt.title
description = yt.description
duration = yt.length
thumbnail = yt.thumbnail_url

print(f'Video:       {video_link}')
print(f'Title:       {title}')
print(f'Duration     {duration} seconds')
print(f'Thumbnail:   {thumbnail}')
print(f'Description: {description}')
print(f'--------------------------------------------')
ITAG = 22
link = yt.streams.get_by_itag(ITAG).url
print(f'link to download with the browser (itag = {ITAG}): \n{link}')

ITAG = 140
link = yt.streams.get_by_itag(ITAG).url
print(f'\n\nlink to download with the browser (itag = {ITAG}): \n{link}')

The code will generate two links to download.
Click the links and observe the download speed for both cases.
Any idea on how to fix?

I think I may have found a workaround, and it's really easy:
You have to specify a range parameter at the end of the URL provided by pytubefix.
I honestly have no idea why it works, but it works.
the URL provided by this script worked for me and the download speed was back to normal:
from pytubefix import YouTube

yt = YouTube(
	url='https://www.youtube.com/watch?v=0FX4jVAL1eg',
	use_oauth=True,
	allow_oauth_cache=True
)

url = yt.streams.get_by_itag(140).url

print(f'{url}&range=0-999999999999')

Thank you for the hint, @andrebore04 !
I am testing and for me it requests to go to a page and enter a code, and then requires login + password for a google account.
That is something that does not works for my needs.
I guess the same is happening for you, right?
And if I remove the auth code use_oauth=True, allow_oauth_cache=True from the Youtube object, then it gets throttled even with the &range=0-999999999999 at the end of the URL.


Support TOC links

GFM autogenerates clickable anchors for each header based on the name of that header. We should do this as well.

Just use the builtin extension.
markdown.extensions.toc.TocExtension(permalink=True)


[Feature] RSS Feed Page

I'd like to have the ability to create a page from an RSS feed.
Just like we have a custom page for "Home", we should have the ability to select a page for RSS and provide an injected value.
Once that's done it should build out a post listing like any other page.

Possible solutions: https://docs.n8n.io/nodes/n8n-nodes-base.ghost/#example-usage


(Event).Commit has value of "HEAD" instead of SHA-1

Thanks for this awesome library! I wish it would get more attention and more stars because it is definitely helping us out a lot at Sourcegraph lately.
I was hoping you could guide me on what the issue could be, because sometimes in my post-push hooks I receive events that have Commit set to "HEAD" and it is causing me some issues (documentation says it should be the SHA-1).
Would you be able to provide some pointers as to why this happens and how it could be fixed?

@AaronO I kept investigating this in the git documentation for receive-pack and can't find the situation where the commit ID would have a value of HEAD. Shouldn't this be resolved automatically to the correct SHA-1? Some guidance here would be helpful.

@gbbr Sorry I missed your first message. I'm happy to hear that it's useful to you at Sourcegraph !
We use this extensively at GitBook, along with a lot of other git related stuff I built. (This component is just a small part of our production git server)
I'm happy to fix this, do you think you can provide with a repo with your hooks, so that I have something to test against ?

No problem! Thanks for getting back to me. That's exactly the issue - I don't know when that happens or how to reproduce it, but I can see in some of our logs that the reason is Commit having value of HEAD and not an SHA-1. I've found a workaround for now and it seems to be ok. Although, I'm not sure if this behavior is normal. I'm glad to hear you guys are also using it and thanks for sharing!
Sorry for not being able to provide a good test case. If I ever run into this again and figure out why it happens, I'll be happy to start a PR :)


Store ImageResizer settings in JSON format

Summary of the new feature/enhancement
ImageResizer settings are currently implemented using the WinForms Application Settings which store the settings in a user.config file, which is XML. To match the other PowerToys, it should be shifted to a settings.json file.

Marking this resolved as it has been shifted to the settings.json file. Creating a separate issue (#1413) for tracking the registry entry used for enable/disable.


Watch for report file changes

look in src/main/resources for any .jrxml files, and add them to the watched paths for hot deployment

Signed-off-by:Nathan Erwin<EMAIL_ADDRESS>

Fixes #70

Please add a test

I might need some help on that...  I've been testing it by adding log messages, then running the integration project to see what I expect in the log...  but I'm not sure how to test the hot reload in the deployment project since it doesn't have any HTTP endpoints to call to force a reload.

@nderwin the Integration test has REST endpoints exposed?  Run mvn quarkus:dev and hit it from SwaggerUI.

Yeah...  for some reason, I was thinking I needed the test in the deployment project, makes more sense in the integration-test project.

Oh, so Cannot mix both io.quarkus.test.QuarkusDevModeTest based tests and io.quarkus.test.junit.QuarkusTestExtension based tests in the same run - I guess we can't test the hot reload in the integration-tests project then...

well the first thing i would do it try it manually

in integration test run mvn quarkus:dev
Hit the REST endpoint in the browser
Change the JRXML
Hit the rest endpoint again does it hot reload?

@melloware Manual testing - yes, I've done that (with log messages) to prove that it restarts when I hit the REST endpoint because the .jrxml file changed.
I (and probably @gastaldi) was hoping to add some automated tests though.  I'm ok with forgetting about that for now, but I'm not in charge here... :grin:

I will take care of these @nderwin


Update README.md

https://www.npmjs.com/package/buffer
You don't need to use require('buffer').Buffer, it can just be require('buffer');

@ORESoftware I'm pretty sure you have to do require('buffer').Buffer because of the way it's exported. See this line from the buffer package (pasted below): https://github.com/feross/buffer/blob/master/index.js#L14
exports.Buffer = Buffer

I think you are right, actually, sorry

@ORESoftware no problem :)


Requires old version of pytest

Attempting to install timefhuman fails with the following error:
  Because timefhuman (0.0.5) depends on pytest (3.7.1)
   and no versions of timefhuman match >0.0.5,<0.0.6, timefhuman (>=0.0.5,<0.0.6) requires pytest (3.7.1).
  So, because gematria38 depends on both timefhuman (^0.0.5) and pytest (^5.2), version solving failed.

Please allow a version of pytest greater than 3.7.1 to be installed for using timefhuman.

@metaperl @StanczakDominik Ah thanks, haha :P I will push out a release soon (hopefully today).

Any update on when this might get to pypi? Thanks


Icon search loses icon when filtering

Not persisting the icon id in the form means submitting a filter will cease to search by icon at all

Fixed by #1248