Manual Restart?

I always used to restart manually, as and when convenient. Is this still an option? It isn't clear from this page nor from the options offered on upgrade.

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

ID: 0184809a-63ff-ecc2-ab67-9c9f4df900f8
Version Independent ID: ac25ee6c-4c57-58b3-1313-e4a53f53c5a1
Content: Manage device restarts after updates (Windows 10) - Windows Deployment
Content Source: windows/deployment/update/waas-restart.md
Product: w10
Technology: windows
GitHub Login: @jaimeo
Microsoft Alias: jaimeo

@arcarley can you advise?

@arcarley Please let me know if independent contributors can help with this issue in any way. Thanks.

I'm going to close this issue. I think Aria answered the question: you have to restart after updating, and there are multiple ways of getting there. This article is specifically about managing that experience in an organization. While manual restart is always an option, that's not something you can manage at scale. This article covers the basics of the end-user experience: https://support.microsoft.com/windows/manage-updates-in-windows-643e9ea7-3cf6-7da6-a25c-95d4f7f099fe#WindowsVersion=Windows_11


Translation Support - Challenge History and Change Detection

With our various translations coming on-line soon, we're going to start to run into issue with keeping those translations in sync as changes are made to the curriculum.  Luckily we already have a system in place for tracking changes in the challenges which we can leverage to help.
What is in Place Now
When challenges are read into the database via the node seed command, there is a section of code which calculates a 32 bit checksum based on the contents of the English Title, Description, Challenge Seed, and Tests.  These checksums are appended to the Google Analytics submissions we make as challenges are completed.
Code Here:
https://github.com/FreeCodeCamp/FreeCodeCamp/blob/staging/seed/index.js#L45
Requirements
We need to be able to do the following:

Subscribe and react to GitHub Webhooks for changes on the challenge *.json files.
Read in the latest JSON files on staging and/or master and calculate the current checksum for each challenge.
Keep a history, by challenge ID, of seen challenge checksums.
When a checksum change occurs:

Alert a list of subscribers that a change has occurred (Via e-mail, etc.)
Display a "Diff" of a challenge between any two given checksums

As a side effect, we should have a searchable history of all changes made to all challenges over time.  It should be possible to analyze past commits to challenge .json files and show the state of a challenge at that time.
There will need to be a way to manage e-mail subscriptions, or only subscribe to certain types of changes.  No idea what this would need to look like.
Opportunistic:  Data visualization stuff for changes to challenges over time.  It would be nice to visualize the changes (and change size) to a given challenge file.

Rollout
This should probably be a stand alone, web based tool.  It will need to be web based just to respond to webhook requests.  We can potentially run this on an FCC "utility" server.  It makes sense to have one server so the comparisons only need to be one once per commit.

Thank you for helping.
IMHO using gettext would be easier.
The sources of the application are changed to send the strins that require translation through a function (typically the function t or _t), that function will return the same string but in the language that must be presented (that depends on user preference, route and/or req.lang or whatever criteria is decided).
A tool (xgettext typically or another one tailored) checks the sources and puts every string that could require translation in a file (e.g locale/fcc.pot).  The same tool (or tools) can be run every time the sources are updated to detect changes in strings.
For each target language, another tool (msginit or msgmerge) should be run also every time that locale/fcc.pot changes, to udpate the strings to translate in each target language (e.g lcoale/es/fcc.po, locale/pt_BR/fcc.po, .....).  These tools will leave intact translations that don't require updating, but will "mark" strings that require initial translation or updating.
Each translation team checks regularly their file, looking for strings marked as needing update (or inital translation) and translate those strings (or some of them according to their available time).  Whenever a team updates its file another tool (msgfmt) is run to check and to generate a "compiled" version of the translation that is binary, and has indexes that makes finding a translation extremely fast for the function _t or t used in sources.
Besides being very standard and well known procedure for translators, there are many tools that help, there are graphical and web interfaces developed for doing these processes (some are paid, some are open source).

I am not familiar with gettext at all.
My primary concern with this proposal is simply to provide an alert to and a history of changes to the "mainline" English tile and description, as well as the challenge seed and tests.  It's possible that some of these changes would not require a corresponding update to the translation, but I figure it's important to highlight all changes.
@bugron's bot, it seems, could be updated to trigger on changes to the *.json files and run an analysis engine.
The key here is to automate the process of tracking changes, removing the need to manually run tools or manually inspect the files.  Are you saying that you have already created tools to automate this process?  What is xgettext?

All these tools already exist.  I propose to use them.

Well find way to use them.

@SaintPeter is this still something you're interested in implementing?


Make attributes visible in c-tips chapter

Fixes #211

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @andre-richter (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

In my browser (Firefox) they are rendered:

I also see them render OK in Chrome; what browser are you using @austinglaser ?

Firefox 70.0.1, on ubuntu. I don't have any custom styles enabled.
If you build from my branch, does the chapter mis-render on your browsers?

Firefox 70.0.1, on ubuntu. I don't have any custom styles enabled.
If you build from my branch, does the chapter mis-render on your browsers?

Same browser here, to the last digit. Ubuntu 64bit.

If I look at the page source for one of the offending examples, I see:

#![allow(unused_variables)]
fn main() {
/// In your top-level lib.rs

[cfg(feature="FIR")]
pub mod fir;

[cfg(feature="IIR")]
pub mod iir;
}

What do you see there on your system?

I see the same behavior (and the same static html, including boring spans) when building master locally with mdbook v0.3.4. Do I need to provide any additional arguments/configuration to mdbook other than mdbook serve in the root directory?
I've tried explicitly clearing my browser's cache, and I see the same page.
I'm looking at https://rust-embedded.github.io/book/c-tips/index.html. Is there another hosting location for the book that could cause this confusion?
Are there other things I could check? Does mdbook use rustdoc internally -- could that be a source of version mismatches? The rendering you're seeing seems like it's likely to come from a newer version of the tool which is smarter about attributes -- but the fact that I see this on the public site would seem to indicate that it's unlikely to be solely an issue with tooling versions on my system.

Ah, I'm looking at https://docs.rust-embedded.org/book/c-tips/index.html which renders OK, whereas your link to rust-embedded.github.io does not render correctly (with the attributes missing).

There it is! Your link renders fine for me as well.
That probably means we should hold off on merging this until verifying that the changes render correctly when they eventually get pushed to the rust-embedded.org site.
Also, I've found a couple other places in the book where attributes are mistakenly getting swallowed -- I'll add them to this PR as well.

This repository is configured to serve this repository's gh-pages branch at the rust-embedded.github.io link, and the gh-pages clearly has the boring spans: https://github.com/rust-embedded/book/blob/gh-pages/c-tips/index.html
Meanwhile the docs repository's gh-pages branch is hosted at docs.rust-embedded.org and also has CI build this repository, which does not have the boring tags: https://github.com/rust-embedded/docs/blob/gh-pages/book/c-tips/index.html
I suspect the problem is that docs uses mkbook 0.2.1:
https://github.com/rust-embedded/docs/blob/master/ci/install.sh
while book uses latest:
https://github.com/rust-embedded/book/blob/master/ci/install.sh

I've opened a PR to make the mdbook  on docs the same as book. Additionally, I've fixed what I think are all the occurrences of erroneously-swallowed code in examples (at least rg boring book after a build doesn't turn up any offenders

It looks like this is fixed in mdbook v0.3.5: https://github.com/rust-lang/mdBook/pull/1088

Thanks for chasing this down, @austinglaser!


vSphere Node Template -> Deploy from template: Content Library -> "Name" -> items not showing..

What kind of request is this (question/bug/enhancement/feature request):
bug / question
Steps to reproduce (least amount of steps as possible):
Rancher v2.5.7
vSphere v7.0.0
In vSphere:

Create a new Content Library, in our case we used "Frames UBUNTU 2004" as name
Import the following item "https://cloud-images.ubuntu.com/focal/current/focal-server-cloudimg-amd64.ova"
I also manually create a VM from that ova, never turned it on and converted it to a template which was loaded into the same content Library
I also loaded another item: https://cloud-images.ubuntu.com/focal/current/focal-server-cloudimg-amd64.vmdk - just to test everything

In Rancher:

Create a new Node Template with the vSphere Node Driver
Select a vSphere Cloud Provider that connects to vSphere with the previously setup Content Library
Creation Method: "Deploy from template: Content Library"
Select the Content Library
The Library Template should be empty and not showing the vmdk, ova or vm template

Result:
We can't use the Content Library feature for deployments, which makes it much harder to automate the template management.
Other details that may be helpful:
Environment information

Rancher version: 2.5.7
Installation option: HA - 3 VMs on RKE
vSphere Version: 7.0.0

Cluster information

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

(paste the output here)

Docker version (use docker version):

(paste the output here)

Thanks for the report @hornet83. Can you confirm whether the VM template stored in the Content Library is showing when you select the creation method Deploy from template: Datacenter ?

Hi, yes it is showing there. Not sure if it gets that from the CL though, as that template is just sitting in a vm folder and the CL is a link to it, from how I understand it.

@hornet83  I am closing this as duplicate of https://github.com/rancher/rancher/issues/32120
Please subscribe to that issue to follow along.


shared mailbox

Are licenses required for shared mailboxes (normally, shared mailboxes do not require a license)

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

ID: 85937c83-35db-0882-e226-a4f2f732de73
Version Independent ID: ac0b646b-0fbf-eb24-6768-07cfcd51076b
Content: Exchange Multi-Geo
Content Source: Enterprise/multi-geo-capabilities-in-exchange-online.md
Service: o365-solutions
GitHub Login: @chrisda
Microsoft Alias: chrisda

@gnerd Yes, licenses are required for shared mailboxes in a Multi-Geo scenario, according to this article.

Licensing
USD $2/user/month for users in Satellite Geos
Resource mailboxes (Rooms/Equipment) and Shared mailboxes need to be licensed
Office 365 Group Mailboxes moved to Satellite Geos will need not be licensed for Multi-Geo

Although you need to keep this in mind, which was mentioned in the comments of the mentioned article:

"Legacy" Shared Mailboxes are essentially "Shared" mailboxes that are created from the EXO Admin Center. These aren't tied to a Modern Group in Office 365. PowerShell shows the value for "RecipientTypeDetails" as "SharedMailbox"
Shared mailboxes are "Groups"...this is where a Mailbox is tied to a Modern Office 365 Group. PowerShell shows the value for "RecipientTypeDetails" as "GroupMailbox".
From a Multi-Geo licensing perspective, the mailboxes with the value "SharedMailbox" need to be licensed.

@gnerd  - Thank you for submitting feedback.
From our understanding, the issue you raised has been answered by VLG17 so we will close this issue.
Thank you for your contribution to make the docs better! Much appreciated!


Possible bug: braces does not work

I expect './dir/*.{js,ts}' to only match ./dir/a.js and ./dir/b.js like fast-glob does, but it matches everything in the working directory.
https://stackblitz.com/edit/node-dkpzkw?file=index.js
This probably a bug?
Also it would be nice if there's a explanation which syntax is supported on the README like picomatch does.

that's weird, it's supposed to work. will check

ah, i see now. tinyglobby only supports an array of patterns right now, using plain strings is not supported. it works if you change it to an array

to avoid this very easy to do mistake in the future, i've just added support for plain strings as patterns in 1c7069e

Oh, I should definitely use TypeScript 😅 Thanks for adding support!


add license for admin dependency micrometer

Make sure that:

[x] You have read the contribution guidelines.
[x] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

associated pr https://github.com/apache/shenyu/pull/4336

add license for admin dependency micrometer
set management endpoints default enable false

Codecov Report

Merging #4409 (d9e03ee) into master (1278156) will decrease coverage by 0.13%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #4409      +/-   ##
============================================
- Coverage     68.09%   67.96%   -0.13%
+ Complexity     7483     7476       -7
============================================
  Files          1021     1021
  Lines         28913    28902      -11
  Branches       2584     2580       -4
============================================
- Hits          19688    19644      -44
- Misses         7681     7709      +28
- Partials       1544     1549       +5

Impacted Files
Coverage Δ

...che/shenyu/sync/data/http/HttpSyncDataService.java
70.40% <0.00%> (-19.39%)
:arrow_down:

...yu/sync/data/http/refresh/AbstractDataRefresh.java
68.96% <0.00%> (-13.80%)
:arrow_down:

.../client/server/consul/ShenyuConsulConfigWatch.java
80.85% <0.00%> (-6.39%)
:arrow_down:

...a/org/apache/shenyu/common/utils/VersionUtils.java
21.27% <0.00%> (-6.39%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
52.12% <0.00%> (-4.26%)
:arrow_down:

...yu/common/dto/convert/rule/Resilience4JHandle.java
71.60% <0.00%> (-3.71%)
:arrow_down:

...a/org/apache/shenyu/plugin/mock/util/MockUtil.java
77.55% <0.00%> (-2.05%)
:arrow_down:

...rg/apache/shenyu/plugin/hystrix/HystrixPlugin.java
75.86% <0.00%> (-0.81%)
:arrow_down:

...henyu/admin/service/impl/UpstreamCheckService.java
66.09% <0.00%> (-0.58%)
:arrow_down:

...yu/admin/service/impl/PluginHandleServiceImpl.java
75.47% <0.00%> (-0.46%)
:arrow_down:

... and 10 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


"waterfall plots"

Reported by hankin on 23 Oct 2000 14:32 UTC
"Waterfall" visualizations could be done in a single command if the "back sweep"
could be avoided. A fairly straightforward enhancement would be to creat a
3-argument version of the PLOT command
PLOT/BREAK xpts, ypts, brkpts
where (say) a missing value in the breakpoints array meant that point was the
end (or beginning) of a new line.
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/177

Comment by<EMAIL_ADDRESS>on 27 Oct 2000 22:17 UTC
Rather than achieving this as a mod to PLOT+ it might be nicer to create a new
EF such as ADD_BREAKSX(var, break_mask), which outputs data on an ABSTRACT X
axis the same length as the original data plus the number of breaks missing
values)in the mask

Comment by<EMAIL_ADDRESS>on 21 Oct 2003 20:16 UTC
Moving from the<EMAIL_ADDRESS>user to the<EMAIL_ADDRESS>user.

Comment by<EMAIL_ADDRESS>on 3 Nov 2003 18:52 UTC
Moving to Ferret tasks; reassign to Ansley

Comment by<EMAIL_ADDRESS>on 10 Nov 2003 21:35 UTC
See plot/step , need reordering
Something like
plot/vs/sweep=z

Comment by<EMAIL_ADDRESS>on 5 Nov 2004 19:20 UTC
Changing Component of bugs in "Ferret Tasks" from All to Main


Notification should be received without page refresh

Bagisto Version: Master
Steps To reproduce--
1. Login as a user.
2. Create an order.
3. Login as admin.
4. Try to view current notifications.
Expected result:  Notification should be received without page refresh
Actual result: Notification is not receiving without page refresh. Please find the screenshot Here.

@prateek-webkul
Any Update ??

Information

Setup configuration from the pusher.
After that run queue listener command i.e. php artisan queue:work --queue=broadcastable
Done :100:


update README.md with project info

added some info to the readme and included the slack link.

@durandom


DOC: Update Readme.md

What's wrong with the existing documentation
I've noticed your project lacks a Installation guide and Readme learn md, crucial for sharing essential project details and fostering collaboration. As a GSSoC 2024 participant, I'm keen on contributing meaningfully.
I'm dedicated to crafting a thorough README for the project's benefit. Thanks for considering. Eagerly awaiting your response.
I've done a readme file for another project, you can check my profile, i always give my best.
Can you assign me this task?
Thank u @Dev-tanay
Add ScreenShots
No response
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

duplicate #275


Move integration test config to separate file

An extension on the work added in #106, this PR extracts the integration-test specific Gradle configuration to a separate file. Simply write the following in your build.gradle to enable integration tests for that module:
apply from: "$rootDir/gradle/integration-tests.gradle"

Codecov Report

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

@@            Coverage Diff            @@
##             master     #114   +/-   ##
=========================================
  Coverage     86.38%   86.38%
  Complexity      247      247
=========================================
  Files            25       25
  Lines           536      536
  Branches        110      110
=========================================
  Hits            463      463
  Misses           23       23
  Partials         50       50

Continue to review full report at Codecov.

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


Research how modern parking reservation services work

Research other apps like Parkster, Fahrradspeicher, ...

Link to the Google Document:
https://docs.google.com/document/d/1zMTnv8nLG9b6wZZAPSHkYIzQxaOXx7VUVMiliqxjFjw


Example workflow fails without initialization if external modules present

The example workflow shown in this repository fails if you add it to a repository as is, if there are external modules present. This includes modules from the public registry e.g. terraform-aws-modules/acm/aws or any other modules.
Since this standard workflow is recommended by GitHub via the add code scanning tool wizard in a repository, it would make sense to have a default workflows that doesn't fail in many cases.
Adding a step with - run: terraform plan before calling the tfsec/tfsec-sarif-action would already make it much more resilient (even though private registries / additional config would still not be present).

I agree. This would be much more valuable if it were able to perform an init in the invocation.

We can't run an init for you because we don't know which version of terraform you're running. I would suggest adding a step in your action workflow to do an init for the appropriate terraform version and then run the sarif action


Delete integration samples from SDK

Goal
Delete unused integration verification code for 7.0. These are technically public so we won't merge this until we are ready for 7.0

#1287  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @bidetofevil and the rest of your teammates on  Graphite

Addressed by another PR so closing


Building rkt inside a vm created by the provided Vagrantfile fails

Running scripts/build-rkt.sh fails with:
(1) /usr/local/go/pkg/tool/linux_amd64/link: running gcc failed: fork/exec /usr/bin/gcc: cannot allocate memory
(2) an error indicating that libacl1-dev should be installed
(3) mmap failing (#1875)
I'll submit a PR that fixes this issue shortly.

+1, I have the exactly same issues in my environment!

On my branch I was able to follow the instructions of getting-started.md. Works for me I'd say.
@zhq527725 I have just pushed my changes to https://github.com/fourcube/rkt. You could try using that until the problems are fixed in the main repository.

Thanks @fourcube , it works for me now!


Move exercises to subdirectory

The problems api (x-api) now supports having exercises collected in a subdirectory
named exercises.
That is to say that instead of having a mix of bin, docs, and individual exercises,
we can have bin, docs, and exercises in the root of the repository, and all
the exercises collected in a subdirectory.
In other words, instead of this:
x{TRACK_ID}/
├── LICENSE
├── README.md
├── bin
│   └── fetch-configlet
├── bowling
│   ├── bowling_test.ext
│   └── example.ext
├── clock
│   ├── clock_test.ext
│   └── example.ext
├── config.json
└── docs
│   ├── ABOUT.md
│   └── img
... etc

we can have something like this:
x{TRACK_ID}/
├── LICENSE
├── README.md
├── bin
│   └── fetch-configlet
├── config.json
├── docs
│   ├── ABOUT.md
│   └── img
├── exercises
│   ├── bowling
│   │   ├── bowling_test.ext
│   │   └── example.ext
│   └── clock
│       ├── clock_test.ext
│       └── example.ext
... etc

This has already been deployed to production, so it's safe to make this change whenever you have time.

I like this - really tidies things up.
I think the only thing we have to be careful about is out automated build...


cmd/gofmt: formatting should sort and cluster imports in a standard way

What version of Go are you using (go version)?

go version go1.16.5 windows/amd64

Does this issue reproduce with the latest release?
Yes
What did you do?
The order of the imports is not something that "matters" to developers, yet the developers will comment during code reviews to order them the right way. Why? There is literally no reason for having them in a specific order other than the "look" of them.
Furthermore, by having them manually fiddled with, it causes avoidable merge issues.
Currently, the imports are ordered but by block only. If I separate my imports in different blocks, they will be all left untouched. It will take the solo import lines and sort them correctly. So there is some kind of logic applied so it "looks" good, but it should go all the way opiniated and not allow the dev to order them the way they want.
What did you expect to see?
import (
	"flag"
	"fmt"
	"log"
	"net/http"
	"os"
	"strings"
	"time"

	_ "github.com/fake-a/package"
	"github.com/fake-b/package"
	fakeZ "github.com/fake-z/package"
}

What did you see instead?
import (
	"fmt"
	"log"

	"flag"
	"net/http"
	"os"
	"strings"
	"time"

	fakeZ "github.com/fake-z/package"

	_ "github.com/fake-a/package"
	"github.com/fake-b/package"
}

100% agree on the idea that the imports should be forcefully formatted to the One True Way.  I don't know that I actually care about what way that is; only that I don't want to argue over it.
This exact issue is why our team has a different formatter, that wraps the go fmt library and then reformats the imports to the team standard.

Based on your code examples, it seems like you're implicitly proposing a particular algorithm but haven't stated exactly what it is. Here's how I understand the algorithm implied by your example; please correct me if this doesn't match what you intended.

Take the set of all ImportSpec in the file and then categorize them into two disjoint sets: packages from the standard library, and all other packages.
Generate a single import declaration which groups together all of the import specs. In it:

First, write out the packages from the standard library entries in lexical order, one per line with no blank lines.
If both sets are non-empty, write one blank line to separate them.
Finally, write out the all other packages entries in lexical order, one per line with no blank lines.

I expect that in the above the definition of "in the standard library" is probably the most debatable sub-question, though presumably the Go toolchain already has some sense of that in order to avoid attempting external module package installation for those, and so I'd expect it to follow the same rules.
(FWIW, I don't have a strong opinion on whether or not to do this, just wanted to attempt to clarify what exactly the proposal is.)

I'm the same as @jfesler
I wouldn't infer my algorithm here. Just come up with one and I won't argue. I don't care about the order but only a true forced order.
I chatted a bit on a Gopher slack and people definitely use different libraries to fix this problem. So maybe look at what is being done in these libraries.

Fair enough! I guess then my previous comment is an example of one possible algorithm that fmt could enforce.
Some other questions that come to my mind when I think about this are:

If there are comments adjacent to or inside any of the existing import declarations, what should happen? Do they get preserved, and if so does the presence of the comments have any impact on the chosen ordering?
Are the ImportSpecs ordered by the import path or by the package name? Does that decision vary depending on whether there's an explicit package name or not?
Is "packages belonging to the same module as the current file" another category worthy of having a separately-sorted list of import specifications, distinct from both packages from the standard library and all other packages?

Again, I don't have a strong opinion either way on whether to do this, but it seems like whether it would be accepted depends at least in part on finding reasonable answers to these questions that can produce reasonable results for most existing source files, and so I'm thinking here about the sorts of edge cases I've seen which seem to have motivated "non-standard" import ordering in source files I regularly interact with. Unfortunately the burden of introducing "one true way" is having to decide what that way is, even if you don't actually care very much! (which is also true for me) :grinning:

It's is sometimes necesary to order them "the way the dev wants" in order to make sure the init() functions of the imported packages are called in the right way with the right order.
This is for example needed for Gui libraries in which drawing is only allowed on the main OS thread, and where the GUI llbrary has to call LockOsThread() in init() to be able to execute on that thread, before any other package, especially if that package also uses LocksOsThread().
That's why go fmt currently leaves separate blocks separate, to keep this possible.

@beoran This is quite interesting what you bring up here. I hope this doesn't prevent this issues to go forward anyway.
If you think about it, if you have
import "flag"
import "fmt"

import (
	"log"
)

Then format, you get
import (
	"flag"
	"fmt"
	"log"
)

But if you had ordered "fmt" prior because you "think?!" you have an init() that has to run before flag:
import "fmt"
import "flag"

import (
	"log"
)

You still get
import (
	"flag"
	"fmt"
	"log"
)

So now, being able to order ONLY within an import () block rather than an import line, sounds even more like a hidden feature. And from what I heard (my knowledge of Go is still limited), people say using init() being an anti-pattern.
As mentioned, I'm happy you bring this up so that we are aware of what is going on, but I hope a consensus is reached.
People seem to be wanting this to be fixed, but most do not care on how.

I also don't care how this is solved but there must remain a way to allow manual ordering as well.
And init() is normally an anti-patteren. But init() is essential in some specific cases,  such as the example of a GUI or game library,  that must lock its goroutine to the primary thread, before main is called.

See some previous discussion at #9922 and #18905.

May be import blocks should be processed separately by the tool without merging or splitting block together, so order of initialization can be defined by order of import blocks.
While imports inside block should be reordered, empty lines added/removed, so any order with any number of empty lines would turn into some canonical representation.
For comments  inside import block, if they are not in the form of single line comment after package name, I'd move them out of import block below closing ")". If comment is absolutely required for some import and does not fit on single line after it, then this import can be separated into separate import block and commented outside that block. Or simply comment can be written with package mentioned in the text.

May be import blocks should be processed separately by the tool without merging blocks together, so order of initialization can be defined by order of import blocks.
While imports inside block should be reordered, empty lines added/removed, so any order with any number of empty lines would turn into some canonical representation (for example ordered by import path with single empty line between different sites seems what most find beautiful, but any representation will be great. If block has only single import, then parenthesis must be either always removed or always added)
For comments  inside import block, if they are not in the form of single line comment after import path, I'd move them out of import block below closing ")". If comment is absolutely required for some import and does not fit on single line with it, then this import can be separated into separate import block and commented outside that block. Or simply comment can be written with package mentioned in the text.


UI Setting Tile

Setting Tile Completed

Isme Color direct bhi use howe hain uski wajah yeh k humne aik theme extension banayi hai issue #298 , to yeh jb complete hogi to is me changing hogi

pushed a commit.

post ui screenshot

you can merge it.


small example added to _.extendOwn

That doesn't demonstrate the own nature of extendOwn. We'll need something a little more interesting.

this demonstrate?
https://github.com/jashkenas/underscore/pull/2091

He wants something that uses a prototype:
var Class = function() {};
Class.prototype.own = false;

var obj = _.extendOwn({}, {own: true}, new Class());
obj.own; // => true

I had not Understood. Thanks.


added docstrings for  functions - check_repo_link, check_repo_link, check_commits, check_mechanics

Thanks Sukhdeep, I believe you forgot to pull the changes that we made yesterday. The pull request contains the workflow folder and also It does not contain the changes that Rafael made.

having merge conflicts and creating a new PR


Fix login with Safe

@assimovt try the login flow with gnosis safe

@adam-maj can u resolve conflicts? then we can merge

Moved to other branch


<EMAIL_ADDRESS>hash check failed

Hi there!
I have an issue installing Webex:
First did an "scoop update", and afterwards "scoop install extras/webex".
It is telling me the hash check is incorrect:
Installing 'webex' (<IP_ADDRESS>753) [64bit] from extras bucket
Webex.msi (177.9 MB) [===============================================================================================] 100%
Checking hash of Webex.msi ... ERROR Hash check failed!
App:         extras/webex
URL:         https://binaries.webex.com/WebexTeamsDesktop-Windows-Gold/Webex.msi
First bytes: D0 CF 11 E0 A1 B1 1A E1
Expected:    a35cd8a2ac33ee9c5360d94d082aaa8b2acba166a0fd1dcad6115a0b2c7fe3ce
Actual:      28be553594ca8fede1333685e4f6f17104179e77f176323b2cf205ccb8cd3a38
Please try again or create a new issue by using the following link and paste your console output:
https://github.com/ScoopInstaller/Extras/issues/new?title=webex%4<IP_ADDRESS>753%3A+hash+check+failed

Wow, what an extremely fast solve!
thx a lot!


Question - Do we still need this delay in android?

In the file ImageCropper.cs we have this condition
if (DeviceInfo.Platform == DevicePlatform.Android)
{
//Delay for fix Xamarin.Essentials.Platform.CurrentActivity no MediaPicker
await Task.Delay(TimeSpan.FromMilliseconds(2000));
}
Is it still required for MAUI?
Those 2 seconds make the app look unresponsive.

v<IP_ADDRESS>


Add SF4 support

@darthf1 - Thanks for the PR - this has been merged and a new release has been tagged.


Added basic localization support.

Adds basic localization tool and exposes it to ux namespace.

Do we want to output a warning when someone tries to set a non-existing language?


Added example and new syntax for the constructor

Hi dqminh! I added a few things here. Now, the Harvest class requires a app_factory function to be provided. That function is called for each scenario. That allows each scenario to run in a complete empty state.
There is also a working example packed together.
A few things were removed though. For those, I wrote some description in each commit. Anyway, would love to hear your opinion.
ps: about flask0.8 changes, how do you think things should be worked out? Should a new branch for 0.7 be created and then proceed with development for a version of flask-lettuce compatible with flask 0.8?

looks good. i'm not on my laptop right now, so i cannot merge it immediately. I will merge it as soon as possible.
By the way, i think it's still good to keep before_each_app and after_each_app as hook options instead of removing them in commit #603f036. Why do we need to remove them ?

Hi Daniel. Well, before_each_app was removed because I couldn't find a good use case for it. No other reason. Lettuce docs did not inspire me in any way either, they just cite it as means for populate the database, which is not very helpful, so I thought it would be nice to re-add it when the "need" arise. I could be very wrong here, but the app_hook could be replaced with other hooks with little to no prejudice.

Hi Daniel! How are things? Could you take a look at flask-lettuce pull request? I think flask-lettuce could make it to an approved extension after this.

sorry for the long delay. I have just merged your changes.
I have also authored you as the contributor, so i think that you can also merge your changes without waiting for me.

Thanks! I'll just take a good look at the code and at flask.extensions docs and request official plugin support. Do you have any plans for features before that?


Use skopeo instead of docker to upload images

For running Nomad jobs (run-nomad-job.nix), docker is used to upload images to the registry. This is a bit annoying though, because one needs a working docker daemon to be able to use it.
An alternative to the docker command is skopeo, which can interact with Docker registries directly, without a daemon.
The basic primitives needed are:

skopeo inspect docker://${registry}/${repo}/${imageName}:${imageTag} to get info on a specific image
skopeo copy docker-archive://$outPath docker://${registry}/${repo}/${imageName}:${imageTag} to upload a Nix-built Docker image
--dest-creds ${username}:${password} for authentication

I previously used skopeo in this way for building and uploading Docker images from CI, which could be used as reference/inspiration: https://github.com/niteoweb/nix-docker-base/blob/ea0e70cbd075f0bd209b7452ee327f6055fee5fe/scripts/image-update
Ping @manveru
Ping @NinjaTrappeur who earlier had to enable docker, add his user to the docker group and restart the display manager to be able to deploy

That sounds awesome, i'd be more than happy if you wanna make a PR. Don't have time right now, but I'll put it on my todo-list anyway if you don't get to it first.

That sounds awesome, i'd be more than happy if you wanna make a PR. Don't have time right now, but I'll put it on my todo-list anyway if you don't get to it first.

This was implemented in https://github.com/input-output-hk/bitte/commit/8c822e79e3a4d6e07f8ccbd05aa809e41d5f3142

This was implemented in https://github.com/input-output-hk/bitte/commit/8c822e79e3a4d6e07f8ccbd05aa809e41d5f3142

Nice! I believe https://github.com/input-output-hk/bitte/blob/2e82563fdb31ffb5b5c30a786db2cd80a35290c4/lib/run-nomad-job.nix#L100-L104 can be removed as well, removing the need for docker in the arguments completely

Nice! I believe https://github.com/input-output-hk/bitte/blob/2e82563fdb31ffb5b5c30a786db2cd80a35290c4/lib/run-nomad-job.nix#L100-L104 can be removed as well, removing the need for docker in the arguments completely

Also, skopeo fails to upload images without /etc/containers/policy.json:
FATA[0000] Error loading trust policy: open /etc/containers/policy.json: no such file or directory

The default policy is just insecure though: https://github.com/containers/skopeo/blob/master/default-policy.json
We can pass --insecure-policy to make it work without having to have that json file.

Also, skopeo fails to upload images without /etc/containers/policy.json:
FATA[0000] Error loading trust policy: open /etc/containers/policy.json: no such file or directory

The default policy is just insecure though: https://github.com/containers/skopeo/blob/master/default-policy.json
We can pass --insecure-policy to make it work without having to have that json file.


Use SLF4J logging system for the library

Instead of porting the Log class from HTSJDK I propose to use SLF4J instead to let the API user to choose the logging framework that they want.
The configuration will be to add slf4j as a compile dependency and some implementation for testing (e.g., org.slf4j:slf4j-simple). Promatically setting verbosity should be done by downstream application or passing a org.slf4j.Logger that can be configure by the caller. This can also allow to switch off the verbosity for the library and output only the logs for the downstream application.
Any objections @samtools/htsjdk-next-maintainers?

This seems like a good idea to me in theory.  It does add additional dependencies, but ultimately almost every downstream project is going end up with a logging framework in addition to the htsjdk Logger class.
As long as:

We're able to package a default logger implementation so that people who import only htsjdk get log messages without having to do additional configuration.
It's capable of being adjusted programmatically by downstream projects.
The SLF4J dependency itself isn't super massive with a million of it's own dependencies.

Then I think it should be good.
@tfenne I feel like you're the most likely to object, what do you think?

For your requirements, I suggest to include an module (htsjdk-logging or htsjdk-logging-utils), which will fix your concerns as following:

The module can have custom implementation or a standard one (I recommend logback), so people importing the complete htsjdk artifact does not need to add additional stuff. An advance user can remove completely the dependency and add it's own
The package can have methods to set the verbosity promatically. Assuming a concrete implementation bundled (e.g., logback), a method can implement the setLogLevel with its verbosity levels and getting the root logger. It is a pity that such methods does not exits in slf4j, but that was a design decision that won't change in the near future (I was always thinking to do a small project for each binding to add the functionality, but I've never got the chance).
If I remember correctly, the API version 1.7.25  has no extra dependencies and the jar is 40KB. If we would like to have a very small dependency overload, the slf4j-simple is only 14KB. If we want to have a more flexible default, logback is the native implementation, which requires both logback-core and logback-classic dependencies, whose jars are 460KB and 283KB respectively. Other bindings are also available, so we can check sizes and number of other dependencies allowed.

That sounds good to me.  A few hundred kb doesn't seem like a problem to me.

Cool - once we have the build system (#14) I will add the proposed module.


Add fix for deploy action failure

One of my manual cherrypicked PRs has a check failing for deploying the preview: https://github.com/redhat-developer/red-hat-developers-documentation-rhdh/actions/runs/8648296395/job/23711771598?pr=161

It seems to be happening occassionally for other PRs also, so I tried to look into a fix for it. Based on https://github.com/peaceiris/actions-gh-pages/issues/339 I am proposing that updating the variable here from github_token to deploy_key may fix the issue, but I am not an expert on GH actions, so will need someone to sanity check 🙂

/cherrypick rhdh-1.0

/cherrypick 1.1.x

/cherrypick rhdh-1.0

/hold


Possible to shoot ships through thin surfaces if they are in contact with it

I tested this in Observation Deck and was able to damage and kill another ship through the windows on the deck. It also worked on the window behind super on Black Forest Ham. Tested on Overloaders Chicago and Colorado.
https://user-images.githubusercontent.com/17466775/169447406-a0acc5ab-db71-41c1-b7ba-c29e4a37f430.mp4

I'd honestly not expect this to get fixed, but I'll leave it open in case someone is able to work a miracle.


UUID as column type and Indexing

It's been advised that add_index is not safe during deployment migration.
I wonder it this applies too on implicit column creations with :uuid type in Rails:
add_columns :notebooks, :user_id, :uuid

I think it's only an issue if you add a column with a default but we have a documented work around for creating the column then backporting UUID values for all the existing data in batches.


PowerShell object offers no way to reset/clear scripts

Summary of the new feature/enhancement
PowerShell's Invoke() and its derivatives run commands and scripts that have been added.
If I want to reuse the instance of PowerShell I can clear the commands with ps.Commands.Clear(), but there's no counterpart for the scripts.
Proposed technical implementation details (optional)
Please add/or document if it exists how to clear the scripts as well as commands.

As far as I can tell, scripts are considered to be part of the .Commands property as well. Scripts are added with ps.Commands.AddScript(script) and the entry is removed from ps.Commands when you call ps.Commands.Clear()

Just to add more info a script and command are represented by Command and the only difference is a script has IsScript -eq $true. Just to verify
$ps = [Powershell]::Create()
[void]$ps.AddScript('1')
$ps.Commands.Commands | Select *

# Parameters                           : {}
# CommandText                          : 1
# IsScript                             : True
# UseLocalScope                        : False
# CommandOrigin                        : Runspace
# IsEndOfStatement                     : False
# MergeUnclaimedPreviousCommandResults : None

[void]$ps.AddScript('2')
$ps.Commands.Commands | Select *

# Parameters                           : {}
# CommandText                          : 1
# IsScript                             : True
# UseLocalScope                        : False
# CommandOrigin                        : Runspace
# IsEndOfStatement                     : False
# MergeUnclaimedPreviousCommandResults : None

# Parameters                           : {}
# CommandText                          : 2
# IsScript                             : True
# UseLocalScope                        : False
# CommandOrigin                        : Runspace
# IsEndOfStatement                     : False
# MergeUnclaimedPreviousCommandResults : None

$ps.Commands.Clear()
[void]$ps.AddScript('3')
$ps.Commands.Commands | Select *

# Parameters                           : {}
# CommandText                          : 3
# IsScript                             : True
# UseLocalScope                        : False
# CommandOrigin                        : Runspace
# IsEndOfStatement                     : False
# MergeUnclaimedPreviousCommandResults : None

yep, just wrote some code to check this as well and indeed command.Clear also clears the scripts. The relationship between ps.Commands content and ps content was the point of confusion for me - I'd assumed the commands are for commands only and didn't look for scripts in there. @jborean93 comment clears it up, thanks.


Include the query task status in the QueryInfo struct

Previously, the task manager was modified to keep the query status so it
could track which queries were running and which ones were killed.
In those previous versions, we removed a task from the process table
as soon as it was killed and did not remove it after it had finished
executing. This meant there could be zombie goroutines running in the
background that were impossible to see.
When the task manager was updated to track the task status, we forgot to
expose the status in the public interface so consumers could see the
task status.
Backport of #9778.

Updated the first comment with the backport PR number.


PgRepository

Repository implementation for PostgreSQL database.
Implements Repository interface.
This should be implemented so that the SQL queries can be customized by providing custom queries in the constructor. However, that should be optional. This would make it possible to use the module for any existing database schema.
Tests should go to hgrs's testing.
See also

MemoryRepository
Sample code from PgPersister.
Repository
MyRepository

We have deprecated the simple repository format, closing this ticket.


Minor optimization of conv_ops.

If data_format is NCHW, and the computation is done on the GPU, the temporary tensor transformed_output is not needed. One can use directly the output tensor. This
leaves more memory for the cudnn scratch allocator, possibly improving performance
of the convolution algorithm.
I also fixed several style issues in conv_ops.cc reported by clang-format.
I did not add any tests because the change is covered by python/kernel_tests/conv_ops_test.py.

Can one of the admins verify this patch?

@zheng-xq you're far more familiar with these kernels, can you take a look?

Nagging Assignee: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @zheng-xq: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @zheng-xq: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Reviewer @zheng-xq: It has been 14 days with no activity and the awaiting review label was assigned. Can you please take a look?

Nagging Reviewer @zheng-xq: It has been 16 days with no activity and the awaiting review label was assigned. Can you please take a look?

Nagging Reviewer @zheng-xq: It has been 123 days with no activity and the awaiting review label was assigned. Can you please take a look?

Nagging Reviewer @zheng-xq: It has been 138 days with no activity and the awaiting review label was assigned. Can you please take a look?

Nagging Reviewer @zheng-xq: It has been 153 days with no activity and the awaiting review label was assigned. Can you please take a look?

Nagging Reviewer @zheng-xq: It has been 168 days with no activity and the awaiting review label was assigned. Can you please take a look?

Nagging Reviewer @zheng-xq: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 183 days with no activity and the awaiting review label has been applied.

Nagging Reviewer @zheng-xq: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 198 days with no activity and the awaiting review label has been applied.

Nagging Reviewer @zheng-xq: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 213 days with no activity and the awaiting review label has been applied.

Nagging Reviewer @zheng-xq: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 228 days with no activity and the awaiting review label has been applied.

Nagging Reviewer @zheng-xq: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 243 days with no activity and the awaiting review label has been applied.

Hi @vrv,
Yes, if you are interested, I want to get it committed. This change followed my investigation into a bug, where I discovered that memory usage is not optimal for data format NCHW. From what I see, this optimization still applies to the current conv_ops.cc.
Anyway, in the meantime I changed countries, and I lost the GPU that I used for dev. So I need a few days to find a GPU card and set up a new dev environment.

Hi @vrv,
I resolved the conflicts, but the checks are not being run. Please, what should I do?

I made a small syntax / comment edit, but LG.  Running tests (again).  We have to trigger the tests, unfortunately.

Hi @vrv, @wt-huang,
I think the tests passed. Is there anything else I should do?

I clicked some other buttons that apparently were necessary but I'm awaiting next steps too...


DEPRECATION NOTICE

There is no specification for the object. There is also no specification for the YouTube Video ID.

https://web.archive.org/web/20120306111051/http://osdir.com/ml/youtube-api-gdata/2009-10/msg00237.html

no more we can do with this goat rope.
--QA

agreed.


Divide by zero error when selecting a random book from an empty collection

Running the server in a folder without .epubs and clicking on "Random Book" triggers a runtime error, likely because the collection count is zero.
/usr/local/go/src/net/http/server.go:1721

OK, thanks for reporting this issue.


Wrong behavior in isNetworkIncludes method

IP network <IP_ADDRESS>/8 is generally considered private, and it is obvious that IP address <IP_ADDRESS> with mask <IP_ADDRESS> will enter this network and will also be a private address. But the IP address <IP_ADDRESS> with a mask, for example, <IP_ADDRESS>, will also enter this network and be considered private.


'short' key is not used in choice dictionaries

Howdy,
Loving the module; it's super helpful to have an easy-to-use question interface for building tooling scripts. I did notice that despite the README that mentions the 'short' value should be displayed after a choice is made in a checkbox or list inquiry, but it looks like it's not actually used.
It'd be great to add that in. In some cases I want to display a fair amount of contextual information when the user is making a selection, but printing those post-selection is a bit too verbose. I'm a bit swamped right now, so if you've got the bandwidth/desire it'd be nice to get that fixed, but once I'm out of the woods I can certainly whip up a PR
Either way, thanks for making this module and making it available to everyone!

Hello, thanks for this. Its our TODO list. We'll surely be glad, if you'd help us with. But in the main time, I'll try to take time and study it.

I just ran into this issue. I'll see if I, or someone on my team, can fix this and send you a pull request.

I think I found the issue here:
https://github.com/CITGuru/PyInquirer/blob/f4c0338dbcd4432d43e9b1dd164b1c75e1304bc1/PyInquirer/prompt.py#L85
Looks like answer will always be answer['name'] instead of allowing for answer['short'] if it is defined for choices in a list.

Cool @timharris777


const variables can't be inspected

If I drop a breakpoint in my code, I can't inspect const variables.
void main() {
  const a = 5;
  var b = 4;

  print('test');
}

Put a breakpoint on the print statement. You can inspect b but not a. The variables are actually fine at runtime and work as expected, they just can't be inspected.

Dart SDK version: 2.19.6 (stable) (Tue Mar 28 13:41:04 2023 +0000) on "windows_x64"

This has also been tested on 3.1.0-46.0.dev (ref b0dacad) and was still not working.
This was tested in both VS Code and IntelliJ
Debugging images from various attempts:

I'd say this is currently working as intended.
Local const variables exist in compile time only. Effectively when you write const name = 5; print(name) it is entirely equivalent to print(5), there is no variable name in scope.
CFE completely erases them, e.g. if you write:
void main() {
  const a = 'Hello!';
  final b = 'Hello!';
  print(a);
  print(b);
}

You get out of CFE:
...
  static method main() → void {
    final core::String b = "Hello!";
    core::print(#C1);
    core::print(b);
  }
...
constants  {
  #C1 = "Hello!"
  ...
}

I can see how this can be a usability issue, especially because you seemingly can't expect const variables at the IDE level either.
@scheglov @bwilkerson  any idea why IDE like VS Code does not show concrete constant values on hover?

I find it hard to believe that "can't inspect variables" was a conscious purposeful intended decision


Add Gitpod configuration for quick setup of development environments

With this change, any developer can simply open a development environment in Gitpod. The environment has docs, web, and binary being built on every code change.
Also included the vscode extensions for Go and Docker.
Try it here:

Codecov Report
Base: 66.19% // Head: 66.19% // No change to project coverage :thumbsup:

Coverage data is based on head (265af01) compared to base (2b8ae40).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #540   +/-   ##
=======================================
  Coverage   66.19%   66.19%
=======================================
  Files          36       36
  Lines        3795     3795
=======================================
  Hits         2512     2512
  Misses        923      923
  Partials      360      360

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Well. That was a bust.

Oops, fixed the link. Please try again

Thank you for your contribution.
I tried it and it worked. It's certainly a quick way to get up and running, though I have no idea how anyone would seriously work like that. I'm going to merge it, but remove the "in your face" references in the README, and make the gitpod references more subtle in the developer docs. It almost reads like an ad right now :-D


Updated Readme.md

The SBT plug-in is no longer required as it is included in the Scala plug-in. Also the Android plug-in is necessary or else IDEA won't actually show any file in the imported project. See http://stackoverflow.com/questions/29719394/how-to-get-intellij-idea-14-working-with-scala-on-android-sbt for details.

Thanks, it should be obvious that the intellij android plugin would be required...


[Safe Apps] Onboarding / Risk education

User Feedback:

There is a scary disclaimer about me being responsible about which Safe app I use that I have to accept the first time I use apps, and then I'm brought directly to the interface of an app (Compound), while I'd have expected first to have to "authorize" an app before having it available for use. In general, it's very unclear what the risks of interacting with a Safe app are. Does it mean that since I visited the page with the Compound app then if the Compound app webpage is compromised it means anyone can create transactions on the interface of my Gnosis Safe multisig? Does it mean that Compound now knows my wallet address? How can I stop this from happening in the future, if I suddently won't trust Compound anymore?

We need to better communicate the risks associated with Safe Apps to users.
Details: TBD

closing due to inactivity and not being placed on the roadmap


Please release a 0.2.15 for parking-lot-core to include the rand bump from 0.4 to 0.5

From the commit history I couldn't figure out why parking-lot is still depending on rand 0.4 ( on crates.io ) but it is.
When you get time could you please release a new version for core that definitely has the rand bump?

Done.


bump compact bound to Julia 1.8

This package is using Base.@constprop in many places, which does not exist in Julia 1.7, rather it'll be a 1.8 feature if I understand correctly.

That change will be backported to 1.7

Hm. Do you know when that'll happen?

I thought it was already in RC1, but it needed to be backported manually. Should be in RC2.


Add proposer/sponsor/etc. to vote page, highlight them on ellipse

Per Howard's recommendations, we should automagically match proposer/sponsor data from a rollcall to voter data in a rollcall and then display them in both the text and in the ellipse, highlighted.

I will work on this
On Jan 26, 2017 8:51 AM, "Aaron Rudkin"<EMAIL_ADDRESS>wrote:

Per Howard's recommendations, we should automagically match
proposer/sponsor data from a rollcall to voter data in a rollcall and then
display them in both the text and in the ellipse, highlighted.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/JeffreyBLewis/WebVoteView/issues/151, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AS6Qi_x8pEACje9nIdDr4Ll_9fcD6A9Yks5rWM77gaJpZM4Lu3SS
.

@aaronrudkin I can set sponsor to the icpsr or id (etc) of the member. What field would make it easiest for you?

I put in the sponsor's icpsr, currently running 1-105. Easy to change it to id if that's preferable. Still to do: 106+

I think id is preferable so I changed it. {sponsor: 'MH0120123'}

ICPSR would be preferable since I think the vote data has ICPSR. I can write the mapping for free anyway when I do the join in downloadVotes so it's no problem if this is more convenient for you in any other case.

Sponsors are now noted on vote pages if present. Will do the highlights in a second.


AnnotatedMesh object

During the course, it became clear how often and students want to apply masks to meshes and then reference synapses and or skeletons.  One idea to support these use cases more naturally, is to extend the mesh class to initialize it with a skeleton and with a set of spatial annotations.  Then, you can apply a mask to the entire object and you'll naturally filter both the skeleton and the annotations (i.e. synapses), and be able to naturally express mesh, skeleton or synapse queries in this masked space.

Casey has been working on this on a branch... decorated_branches that isn't yet ready for release but looks very promising.


Updated translation.

Coverage remained the same at 24.341% when pulling 57708f7185ac553539ddda14c18081d04079882a on Myrmece:patch-5 into 445374242fcd1f74793ae3253de15d8b71bbeb94 on NyaaPantsu:dev.


ffmpeg 3 by default

Switches over to ffmpeg 3 by default.
The reverse dependencies have been tested here: https://headcounter.org/hydra/jobset/profpatsch/ffmpeg_3-test
and rolled back to manual ffmpeg 2 overrides where still necessary.

Can you make the commit order so that there will be no breakage (think git bisect)?

@bjornfor @dezgeg Squashed them.

I'm not certain about rebuild impact; maybe it would be better to go through staging.

@vcunat Rebuild impact has been tested, as mentioned in the note. What else do you need to confirm?

I did notice you found which need version 2, but I meant I don't know what's the amount of transitive dependencies rebuilt as result of this PR (to decide whether it's suitable to go to master directly).

but I meant I don't know what's the amount of transitive dependencies rebuilt as result of this PR

Exactly 108.
I diffed nix-env -f ~/nixpkgs -qaP --drv-path before and after and created a hydra build release for those packages with the changes on the headcounter hydra.

@vcunat Shame on me. The release I created only contained the first package of every subattrset, so only amarok made it into the release (out of all kde packages). Working on fixing it, sorry.

Update: Not as many packages broke as I feared, working on fixing them.

That's not a big deal at all, especially if you or someone find solution(s) to these within a couple days.

Spent way too much time on this for the two packages that were affected. :( https://github.com/NixOS/nixpkgs/pull/16682


Pretaining for 2D Backbone, with CMU Dataset

Hi, referencing #75. It seems that because the CMU dataset is looking for a different set of joints, if I would like to train using the CMU dataset, I will need to retrain the 2D backbone but using the CMU dataset instead. Is this correct?
Also, how did you guys train the 2D backbone? Are there any instructions on doing so?
Of course, it would be even better if you could provide the pretrained backbone weights for the panoptic studio dataset!
Thank you!

@Samleo8 did you understand how do the authors train the 2D backbone? Thanks


git commit -m " Add "

I need just one pull request kindly accept it thanks in advance

Thanks for your valuable contribution 😊. Don't forget to ✨ our Repository.
If you want to contribute to our new Ideas Project -https://github.com/KapilKumar7/CreativeAppIdeas


Submariner in different AZs?

We tried running submariner in our cluster which spans three different AZs on AWS. It looks like the route agents fail to add the route to the destination cluster CIDRs when using as a default gateway a local node that sits in a different AZ. This is to be expected since you cannot route from one AZ via another in AWS since they are not directly attached. How to solve this?

We are looking at a solution to fix this, like using VXLAN to pass local cluster traffic instead of using ip routing.

Eventually, we decided to run a submariner per AZ and a broker per AZ. So the cluster becomes the AZ. As we had to run the submariner agents in different namespaces, we had to patch the code as the namespace was hard-coded. We now have it working.

#154


Issue 61

Closes #61

Can you please fix a travis and do a rebase with upstream master?

We are about to make Content bundle deprecated. There is no need to do anything here..


Run nncp target before crc_bmo_setup

This would ensure ironic using the nncp configured static ip for the default interface rather than the dhcp configured one when attaching the interface.
Also reverts the change to use $MAC_ADDRESS when updating default network to unblock ipv6 work.

Aha, I figured we had done this already.
/lgtm
/approved


Birds of a feather blog post

Added a new blog post

Sorry for the repeated and probably silly question, but being on this on and off I am not yet fully convinced of what's the process of adding a new blog post.
Once merged in main, if I remember correctly we are supposed to open a pull request from main to production, but doing so I see many other changes and not only the blog post added.
Any insight?


feature request: support for multi-character constants

I tried to update the code to support the full international morse code standard with what I thought were single characters (minus CH) to the unordered map kMorseCode in ggmorse.cpp, but I got multi-character character constant warnings when compiling and bigger problems when running. Perhaps this is why you left these out to begin with. The symbols in question, some which are shared by other non-latin symbols and pose another interesting issue, are:
    { "0101",    'Ä',  },
    { "01101",   'À',  },
    { "1111",    'CH', },
    { "00100",   'É',  },
    { "11011",   'Ñ',  },
    { "1110",    'Ö',  },
    { "0011",    'Ü',  },

I tried converting the unordered map to strings instead of characters, but it just turned into a big mess. I'm thinking that's the way it probably should be though in order to fully support all possibilities.

Multi-byte characters are currently not supported by ggmorse.
I think the simplest solution that you can do is keep the internal map as it is and extend it with a few extra single-byte characters and then perform the mapping to your multi-byte characters outside of ggmorse.
For example:
// in ggmorse.cpp
const TAlphabet kMorseCode = {
    { "0101",    128,  },
    { "01101",   129,  },
    { "1111",    130,  },
    { "00100",   131,  },
    { "11011",   132,  },
    { "1110",    133,  },
    { "0011",    134,  },
    { "01",      'A',  },
    { "1000",    'B',  },
    { "1010",    'C',  },
    { "100",     'D',  },
...
};

```c++
// in your code
// use this map to convert the resulting single-byte text to multi-byte text
const std::unordered_map<char, std::string> {
    { 128,   "Ä",  },
    { 129,   "À",  },
    { 130,   "CH", },
    { 131,   "É",  },
    { 132,   "Ñ",  },
    { 133,   "Ö",  },
    { 134,   "Ü",  },
    { 'A',   "A",  },
    { 'B',   "B",  },
    { 'C',   "C",  },
    { 'D',   "D",  },
...
};

Maybe a flag could be implemented that would output the morse code sequences instead of mapping them to characters. That way, a follow-on program could handle the mapping with whatever characters are needed without having to worry about updating the mapping inside of ggmorse.


Bookstore: initialize project with components

💯 Achievements

Use React Router.
Use React components.
Use correct GitFlow.
Ensure no react-redux linter errors.
Proper project documentation.

Awesome stuff @TedLivist  🥂 .


.NET Automatic Instrumentation page title

Originally posted in https://github.com/open-telemetry/opentelemetry.io/pull/1015#discussion_r784668409
@cartermp - I'd like to suggest that we use the same title and linkTitle here as the corresponding page in the Java docs, namely:
title: Automatic Instrumentation
linkTitle: Automatic

/assign @cartermp

So this one might be a little point of contention. My understanding is it is inaccurate to call it automatic instrumentation because it documents how to initialize instrumentation libraries. That said, I have seen a lot of people also refer to this as automatic instrumentation anyways, so it's a question of technical accuracy vs. familiarity in my mind.

What might be missing from that page so warrant it being about automatic instrumentation? Let's discuss this at the next Comms meeting.

Was discussed during 2022/01/27 Comms meeting. There was a general consensus for using "Automatic Instrumentation".

@cartermp @chalin This change is incorrect. Automatic instrumentation is well defined in the spec repo : https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/glossary.md#automatic-instrumentation
In case of Java (https://opentelemetry.io/docs/instrumentation/java/automatic/) the term is correct, as it is talking about the actual auto instrumentation. For .NET, auto-instrumentation is work in progress https://github.com/open-telemetry/opentelemetry-dotnet-instrumentation#status

Hi @cijothomas - thanks for the information and link. I've added an entry to the agenda for discussion at our next Comms/Website meeting this coming Thursday. In case you'd like to join, the meeting info is available in the minutes doc.
In the meantime, feel free to respond here @cartermp @tedsuo @austinlparker if you prefer.

Hi @cijothomas - thanks for the information and link. I've added an entry to the agenda for discussion at our next Comms/Website meeting this coming Thursday. In case you'd like to join, the meeting info is available in the minutes doc.
In the meantime, feel free to respond here @cartermp @tedsuo @austinlparker if you prefer.

I think this is something to be discussed in Maintainers/Spec meeting. The fundamental question I am asking is "If one uses instrumentation libraries to get automatic spans from the library, is that called "Auto-Instrumentation" ?


US 1583733 Add missing language IDs - 12

US 1583733 - Fix CATS P1 issues by adding missing language IDs to code blocks.

Used "output" for program output examples.
Used "csharp" for ADO.NET connection string examples.
Used "text" for examples of the relation between DataSet components.

Contributes to #2192

@mairaw Conflicts and changes have been resolved. This should be ready to merge. Thanks!


Python3 Grammar, refactor for a better structure

@MegaIng This is mid-work, but maybe you'd like to review it, to stop me if I'm going in the wrong direction.


Write a better answered request view

Currently, answered requests use the same "full request" view that active requests do. This is not terribly friendly for reviewing. Create a view that has both history and notes, with the final request text at the top, and the history/notes below, going from most recent to least recent. Provide a "jump to bottom" button/link to enable the user to easily read from beginning to end, instead of vice versa.

Fixed in 0.8.5


Resource Group Operation Error

This is an Ansible related issue. https://github.com/ansible/ansible/issues/59251
File \"/var/lib/awx/venv/ansible/lib/python2.7/site-packages/azure/mgmt/resource/resources/v2017_05_10/operations/resource_groups_operations.py\", line 144, in create_or_update\n    request = self._client.put(url, query_parameters, header_parameters, body_content)\nTypeError: put() takes at most 3 arguments (5 given)\n"

I see that the related Ansible issue is already closed due to wrong msrest client version, so I am closing this one.


Pull ‘current_schema_version’ in Data Portal backend and frontend to disable publish

Add to 'partial redeploy' so we can pull current schema version from cli

Won't Do; no longer necessary based on implementation (see associated issues)


Update Minimap2 to 2.21

The pipeline currently uses ViralMSA wrapping around Minimap2 2.17, but Minimap2 2.18 was recently released:
https://github.com/lh3/minimap2/releases/tag/v2.18
Not urgent at all (seems to be many minor bug fixes, but I doubt anything that would significantly impact ViralMSA), but it would be good to update some day in the future (not urgent/necessary whatsoever)

2.21 was released recently:
https://github.com/lh3/minimap2/releases/tag/v2.21

2.22 was recently released as well:

GitHub: https://github.com/lh3/minimap2/releases/tag/v2.22
Preprint: https://arxiv.org/abs/2108.03515

It seems as though this update is primarily focused on improving how Minimap2 handles long indels (e.g. in long-read mapping), so not critical for our purposes (but the other improvements since v2.17 may be relevant)

https://github.com/lh3/minimap2/releases/tag/v2.23

https://github.com/lh3/minimap2/releases/tag/v2.24


[SCB*675] Gen proto from swagger

Coverage increased (+0.06%) to 86.395% when pulling f25f7355a86af4a0627e65e6ccc50a32a0410f01 on wujimin:gen-proto-from-swagger into abdb21adee72717f8fadd6f5fa2a89ebe4e00920 on apache:master.


Mobile app can create post without a topic, when permissions are set not to allow that.

Mobile app users can still create "no topic" posts even if permissions are set to disallow it and desktop users cannot.  Tested on iOS.

Adding to this, it looks as if the mobile client actually can't specify any topic at all, and if there is no topic in the stream yet then I think it's able to create the first post, which falls into (no topic).
Users really should be able to create topics in the mobile app.

Adding to this, it looks as if the mobile client actually can't specify any topic at all, and if there is no topic in the stream yet then I think it's able to create the first post, which falls into (no topic).
Users really should be able to create topics in the mobile app.

OK adding again, users CAN create new topics on mobile but it is completely not obvious how to do it.  You have to start writing a message in an existing topic, then EDIT the topic (that's what it looks like you'll be doing anyway) and post your message.  It will actually create a NEW topic and not edit the old one.
This is extremely unintuitive and really needs some improvement to make it crystal clear how to post a new topic on a mobile device.

OK adding again, users CAN create new topics on mobile but it is completely not obvious how to do it.  You have to start writing a message in an existing topic, then EDIT the topic (that's what it looks like you'll be doing anyway) and post your message.  It will actually create a NEW topic and not edit the old one.
This is extremely unintuitive and really needs some improvement to make it crystal clear how to post a new topic on a mobile device.


Refactor codes for regexp

Coverage decreased (-0.2%) to 92.401% when pulling 628412a7dbc9e58bdfead33b388d20d558d6d737 on Zhangxinguo:2.1.1 into fac8d6380e99d34973b5846476c35e5c5a7e831f on dangdangdotcom:2.1.1.


⚠ Bofrost.lu has degraded performance

In ca10648, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2048 ms

Resolved: Bofrost.lu performance has improved in e46336d after 23 minutes.


unstructured header page information

As in the header information section ,

the difference between the logo and all the sections like home , leading blogs and all
the boundaries between the home, leading blogs, start writing, categories ,about , contact us , give feedback with proper styling make it more attractive
the difference between the alignment of login icon and day night toggle button

@ANSHIKA-26 kindly assigned this issue to me .i will surely solve it

this is not what our navbar looks like , kindly check again !

this is what its like and its perfect , thanks


Added check command generator

Added feature #28 with doc and tests.

Thanks @Gman98ish  @tylerwoonton

This needs rebasing onto master. I had a go but there were merge conflicts with it. Any chance you could have a look @tylerwoonton?

This needs rebasing onto master. I had a go but there were merge conflicts with it. Any chance you could have a look @tylerwoonton?

Sure, I'll take a look @miff2000.


cannot install

when trying to install with "pip install PyTextNow".
i receive answer "Could not find a version that satisfies the requirement PyTextNow (from versions: )
No matching distribution found for PyTextNow"

What OS are you using?
Also, are you installing globally or in a virtual environment and what version does it say is installing?

ubuntu 18.04. i install globally.
Le mar. 29 juin 2021 à 21:28, SilverStrings (Matt) @.***>
a écrit :

What OS are you using?
Also, are you installing globally or in a virtual environment and what
version does it say is installing?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/leogomezz4t/PyTextNow_API/issues/27#issuecomment-870819641,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK53Y7ROQH6J3BAYJW3GOX3TVIGDNANCNFSM47QYVKUA
.

ubuntu 18.04. i install globally. Le mar. 29 juin 2021 à 21:28, SilverStrings (Matt) @.***> a écrit :
…
What OS are you using? Also, are you installing globally or in a virtual environment and what version does it say is installing? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#27 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AK53Y7ROQH6J3BAYJW3GOX3TVIGDNANCNFSM47QYVKUA .

Try installing in a virtual environment and see if that works. I just tried installing in virtualenv and it installed fine with
pip3 install PyTextNow

when installing with git, it ask me login and password
[image: Mailtrack]
https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality5&
Sender
notified by
Mailtrack
https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality5&
29/06/21,
21:35:55
Le mar. 29 juin 2021 à 21:33, Yossef Haim @.***> a écrit :

ubuntu 18.04. i install globally.
Le mar. 29 juin 2021 à 21:28, SilverStrings (Matt) <
@.***> a écrit :

What OS are you using?
Also, are you installing globally or in a virtual environment and what
version does it say is installing?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/leogomezz4t/PyTextNow_API/issues/27#issuecomment-870819641,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK53Y7ROQH6J3BAYJW3GOX3TVIGDNANCNFSM47QYVKUA
.

Huh, That's weird @leogomezz4t did you change the permissions on the repo?
Did pip3 in a virtual environment not work then?

ok pip3 works, thanks.
Le mar. 29 juin 2021 à 21:36, SilverStrings (Matt) @.***>
a écrit :

ubuntu 18.04. i install globally. Le mar. 29 juin 2021 à 21:28,
SilverStrings (Matt) @.***> a écrit :
… <#m_-3398716948144469152_m_-371784464060253732_>
What OS are you using? Also, are you installing globally or in a virtual
environment and what version does it say is installing? — You are receiving
this because you authored the thread. Reply to this email directly, view it
on GitHub <#27 (comment)
https://github.com/leogomezz4t/PyTextNow_API/issues/27#issuecomment-870819641>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK53Y7ROQH6J3BAYJW3GOX3TVIGDNANCNFSM47QYVKUA
.
Try installing in a virtual environment and see if that works. I just
tried installing in virtualenv and it installed fine with
pip3 install PyTextNow
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/leogomezz4t/PyTextNow_API/issues/27#issuecomment-870824920,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK53Y7TSY7HH2P7MAMSYUITTVIHD5ANCNFSM47QYVKUA
.

[image: Mailtrack]
https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality5&
Sender
notified by
Mailtrack
https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality5&
29/06/21,
21:42:25

ok pip3 works, thanks. Le mar. 29 juin 2021 à 21:36, SilverStrings (Matt) @.> a écrit :
ubuntu 18.04. i install globally. Le mar. 29 juin 2021 à 21:28, SilverStrings (Matt) @.> a écrit : … <#m_-3398716948144469152_m_-371784464060253732_> What OS are you using? Also, are you installing globally or in a virtual environment and what version does it say is installing? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#27 (comment) <#27 (comment)>>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AK53Y7ROQH6J3BAYJW3GOX3TVIGDNANCNFSM47QYVKUA . Try installing in a virtual environment and see if that works. I just tried installing in virtualenv and it installed fine with pip3 install PyTextNow — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#27 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AK53Y7TSY7HH2P7MAMSYUITTVIHD5ANCNFSM47QYVKUA .
[image: Mailtrack] https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality5& Sender notified by Mailtrack https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality5& 29/06/21, 21:42:25

No problem


Merging Width and Depth Pruning

Added Support for Retaining KV heads while pruning, incase of GQA. Added eval on mmlu and wikitext2. Added some basic logging to keep track.

A couple of things before @shwu-nyunai reviews, the code needs to be blackformatted, no print statements (stick to python logger if needed), class/function naming should follow PEP 8 convention.

Can we also modularise convert_to_hf.py, right now there are absolute azure paths instead of a argparser.

@Nahush26 let's clean the absolute paths and maybe mark these args as required. and add args if missing.


CB-19939  Remove unused com.sun.xml.ws lib

See detailed description in the commit message.

Sadly the recently introduced "uk.co.jemos.podam" is using it.


Add "env" support

I changed a bit your initial implementation @jgkim from #16, because the configuration from the specific environment should be merged into the default configuration instead of overriding it.

[Current coverage][cc-pull] is 100% (diff: 100%)

Merging [#19][cc-pull] into [master][cc-base-branch] will not change coverage

@@           master   #19   diff @@
===================================
  Files           1     1
  Lines          21    30     +9
  Methods         0     0
  Messages        0     0
  Branches        0     0
===================================
+ Hits           21    30     +9
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update [849ebcb...f7481b5][cc-compare]
[cc-base-branch]: https://codecov.io/gh/tleunen/eslint-import-resolver-babel-module-alias/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/tleunen/eslint-import-resolver-babel-module-alias/compare/849ebcbb601ee151ee566d4e391dfdce959e85ad...f7481b5aa2a5b14e7e634cffb3cbde8a481427d0?src=pr
[cc-pull]: https://codecov.io/gh/tleunen/eslint-import-resolver-babel-module-alias/pull/19?src=pr

[Current coverage][cc-pull] is 100% (diff: 100%)

Merging [#19][cc-pull] into [master][cc-base-branch] will not change coverage

@@           master   #19   diff @@
===================================
  Files           1     1
  Lines          21    30     +9
  Methods         0     0
  Messages        0     0
  Branches        0     0
===================================
+ Hits           21    30     +9
  Misses          0     0
  Partials        0     0

Diff Coverage
File Path

•••••••••• 100%
src/index.js

Powered by Codecov. Last update [dc8c17f...198b1e2][cc-compare]
[cc-base-branch]: https://codecov.io/gh/tleunen/eslint-import-resolver-babel-module-alias/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/tleunen/eslint-import-resolver-babel-module-alias/compare/dc8c17fa25452f9495a580c9076e62e0a7ac7f57...198b1e25313742f56df788bf803b234942d4cd3d?src=pr
[cc-pull]: https://codecov.io/gh/tleunen/eslint-import-resolver-babel-module-alias/pull/19?src=pr

Thank you for informing me, @tleunen.


MultiSeg Parameter File Saving/Loading Issue

Within MultiSeg Parameter Files, the section Registration-parameters-DTI is nonrecognizable for loading and isn't correctly saved:
<Registration-parameters-DTI>
<Software-used bool="1"/>
<First-modality metric="CC" weight="1" radius="2"/>
<Second-modality metric="CC" weight="1" radius="2"/>
<Iterations J="30" K="20" L="10"/>
<Transformation type="SyN" gradient-step-length="0.125" number-of-time-steps="2" delta-time="0.01"/>
<Regularization type="Gauss" gradient-field-sigma="5" deformation-field-sigma="5"/>
<Mask brain-mask="0" smoothed-brain-mask="0"/>
<Resources Number-of-Registrations="1" Number-of-Cores="1" Number-of-GB="4"/>
<Container-Id number=""/>
</Registration-parameters-DTI>

Is not loading DTI parameters when it is executed in the command line?

When the parameter file with this section included is opened it comes up with an error that says Registration Parameters DTI is unrecognizable. If the section is deleted from a created or generated parameter file it works fine.
Mary Enloe
University of North Carolina at Chapel Hill, '22
Computer Science | Major
Philosophy, Politics, and Economics and Social and Economic Justice | Minors
p:<PHONE_NUMBER>
LinkedIn: https://www.linkedin.com/in/rachelbybee12018/ www.linkedin.com/in/mary-enloehttp://www.linkedin.com/in/mary-enloe

From: Juan Carlos Prieto<EMAIL_ADDRESS>Sent: Thursday, April 11, 2019 4:10:57 PM
To: NIRALUser/NeosegPipeline
Cc: Enloe, Mary Amelia; Author
Subject: Re: [NIRALUser/NeosegPipeline] MultiSeg Parameter File Saving/Loading Issue (#12)
Is not loading DTI parameters when it is executed in the command line?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/NIRALUser/NeosegPipeline/issues/12#issuecomment-482293121, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ArBhxbrQqZVrAnx_cPId4qLEprWDJXAkks5vf5bRgaJpZM4cqurE.


Feature request: fallible version of BytesMut::unsplit (i.e. make BytesMut::try_unsplit public)

I'm using BytesMut::unsplit in a project and would like to raise an error when the bytes cannot be unsplitted, instead of silently copying. An Err(_) would let me know I messed up somewhere, so I can go fix the problem.
Reading the source code, I came across try_unsplit, which seems to do exactly what I need, but is private. Have you considered making it public? Or is there a suitable workaround for people like me, who want to guarantee that no data is ever copied from one BytesMut to the other?
In case it's relevant, below is the code for try_unsplit that I think could be made public:
https://github.com/tokio-rs/bytes/blob/cb7f8449b5efc7022dc592b3a1d7dd33079f4c8f/src/bytes_mut.rs#L908-L926

Duplicate of #287.

Just in case: the linked issue seems to be about Bytes, so I assumed it didn't apply to BytesMut.


Fix bugs in multiline Text

This PR fixes 3 critical bugs on Text when using the multiline feature (when you have a constraining width and height).

bug 1: a small Text that fit in one line but contains '\n' is not handled as a multiline.
bug 2: the measuredSize.width was inconsistent between singleline and multiline implementation. It's now uniformed with a max on the lines[i] width.
bug 3: the last item of measuredSize.lines[i].width is not a number but a {width} . it makes a multiline text NOT rendering the last line.

I'd like to see this PR checked on!


Problems with configuration

I am configurating this plugin and I got it like this:

and when I try to login I got this:

My APP on Azure Portal look's like this:

hello @IvernBurner ,
try adding https://glpi.hormipresa.com/

to your URI Application IDs


⚠ Bofrost.de has degraded performance

In a338a9b, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1622 ms

Resolved: Bofrost.de performance has improved in a841262 after 14 minutes.


Bump library-go to get encryption in-process migrator error handling

/retest

@sttts I think we need https://github.com/sttts/cluster-kube-apiserver-operator/pull/1

/test e2e-aws-operator

/lgtm


WhatsApp unexpected websocket close

After some time following error is shown:
time="2019-02-24T09:28:54-05:00" level=error msg="unexpected websocket close: websocket: close 1006 (abnormal closure): unexpected EOF" prefix=whatsapp

Reported by @wsdjeg on #734. As @tjird said: it is already mentioned in https://github.com/Rhymen/go-whatsapp/issues/106

Quickfix provided by @jasontkh in https://github.com/Rhymen/go-whatsapp/issues/106#issuecomment-465915538
To make it bulletproof we should restart the connection on error.

Blocked by https://github.com/Rhymen/go-whatsapp/issues/111 and https://github.com/Rhymen/go-whatsapp/issues/112

[ ] import those PRs to go-whatsapp in the matterbridge fork
[ ] test
[ ] release

I think this can be closed in favor of #783 ?

Agree.


Security docs updated to contain generic references

fixes https://github.com/samsung-cnct/k2/issues/567

LGTM @joejulian can you finish your review so we can merge this?


Simplified Chinese Translation of v2.18-latest

As I said before, this is not made by me but our team.

Thanks to all of you!


[Outlook] (preview) Add notification message with actions

Docs Build status updates of commit c136484:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference/objectmodel/preview-requirement-set/outlook-requirement-set-preview.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Upgrade HMAC from 128 Bit Keys to 512 Bit Keys

“No description provided.” Please provide a rationale for this proposed change explaining the reason for the divergence from the upstream Bitcoin 0.16.3 cloneparent code.


Update Ember-Cli-Babel

@rwjblue  Maybe we have to update ember-cli-babel to version ^6.8.2 ? Is that right?

Yeah I've been getting
warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>
in Yarn for ages.

See #110


raw graph cannot pass data from excel to create Arc Diagram

@Ltongtaa it seems that your data is not in UTF8 format. Try to save the file as an .xlsx file and copy/paste the data to rawgraphs


Consider merging opencensus integration

This fork adds Opencensus integration. Would there be any interest in a PR adding this back here?

I am very much interested in supporting tracing, but I prefer to wait until OpenTelemetry is ready. Meanwhile I am working on better context propagation support.

Good to hear! From the OpenTelemetry website:

I want to start instrumenting production code today. What should I use?
We are still working on the first production-ready release of OpenTelemetry. For those who want to start instrumenting production code immediately, use either OpenCensus or OpenTracing. OpenTelemetry will provide compatibility bridges and there is no need to wait for production-ready OpenTelemetry APIs in your language.
So maybe doing the integration sooner rather than later would still work out?

There is now opentelemtry support in go-redis. This should probably be closed.

Yes, there is also basic OpenCensus integration in https://github.com/go-redis/redisext/tree/master/rediscensus


add a test for task sequencing

Add a test to confirm set_sequence() on tasks works as expected, affecting the order tasks are executed. This leverages that by default Goose executes tasks in the order they are declared, combined with a task that sleeps for the duration of the test after which no other tasks run. Without sequences all tasks run, as the delay task runs last. With sequences only two tasks run, as the delay task runs sooner.
Also confirms that sequences do not affect test_start and test_stop functions, which should always run regardless.

This worked locally, changing parameters I can unexpectedly duplicate -- apparently needs work.


Preserve fields with empty array values when converting json to toml format?

Hello, when we use the stringify function to convert json to toml format, we find that some fields with a value of [] are sometimes lost (as shown below), and we hope to retain these fields. smol-toml is the latest version

That's a weird behavior, I'll investigate this and make sure they're not lost, as this shouldn't be happening.

This has been fixed in v1.1.4


Script modifications for linux

Fix for #11
These revisions allow killrvideo to launch on Centos 7. I also added a command-line option to setup_docker.sh and create-environment.sh that allows the user to specify the loopback IP, which also avoids creating the loopback alias.
./setup_docker -l <IP_ADDRESS>

Intra-container port access was being blocked by firewalld. I'm not a docker wiz, so I turned off firewalld and things are working, for the most part.
Video suggestions are not working. Not sure why.

I tested and merged in the prior pull request separately.
This pull request mostly affects the create-environment.sh script, which gets pulled in from killrvideo-docker-common repo via a subtree pull. I made a similar (but simpler) change to the script in killrvideo-docker-common and then executed a subtree pull in dependent repos (-node, -java, -generator, -web).

Hopefully that all makes sense @bsmith94. Thanks and let us know if you have more feedback.


Only depend on antlr4-runtime at runtime

By default, the Gradle ANTLR plugin depends on the full ANTLR4 binary, which significantly bloats the dependency size where only the ANTLR runtime is required - see https://github.com/gradle/gradle/issues/820. This PR removes the ANTLR dependency from the api configuration used for Maven repository POMs, and adds a dependency on antlr4-runtime.

Nice catch; thanks!


Remove companies, add co-chair

We've decided that the company model isn't as important as we originally
thought.

I figured we should keep them as a reference until we've decided what to replace them with


NotFound: 404 Not Found (error code: 10062): Unknown interaction

Sentry Issue: CHARBOT-1Q
NotFound: 404 Not Found (error code: 10062): Unknown interaction
  File "charbot/programs/shrugman/modal.py", line 102, in on_submit
    await interaction.response.defer(ephemeral=True)

Ignoring exception in modal "<GuessModal timeout=None children=1>":

Have had this issue pop up a few times in a few programs, not sure the cause yet, help diagnosing would be appreicated.

https://github.com/Bluesy1/CharB0T/blob/main/charbot/programs/shrugman/modal.py#L94-L102 as the line ref

This hasn't appeared in a while, closing for now


Question regarding fine tuning

I am trying to fine tune on a custom image dataset. Using base 224 patch 16, I am setting the all params to false and changing the head output dims to 2 (binary classification). When trying to train this model, I get the error "cannot set a hook on a tensor that does not require grads".
I simply added a conditional statement in vit_lrp.py:
if self.train and x.requires_grad:
    x.register_hook()

if self.train and attn.requires_grad:
    attn.register_hook()

Just making sure this isn't breaking anything as this does fix my issue to fine tune.

Hi @MichaelMMeskhi, thanks for your interest!
The second hook you attached is originally used to calculate the gradients of each attention head (as our method averages across the heads using each head's gradient as weights).
By not collecting the gradients, the method will not work (I think it will return None as the gradient information and then you should see an exception).
However, since you do not need the relevance maps when training fine-tuning, this should have no effect on your visualization results after completing the fine-tuning. In fact, you can even fine-tune your model using the simple no LRP code as appears in the original implementation and then just save your weights and replace the url link in our code for visualization with your checkpoint.
I hope this helps, but please let me know if you have any follow-up questions.
Best,
Hila.

Thank you for your reply @hila-chefer! Yes, I did this originally for fine-tuning. Thank you, I managed to get things working.
Best,
Michael


Add docker compose

Notes:

I have altered the test script to pick up environment variables
I have altered the entrypoint script to execute the incoming command
The original setup.sql gave Query limit exceeded errors, so I've added docker-setup.sql which breaks the insert statements into smaller chunks.
Added dev.Dockerfile which includes dev dependencies, I had to add python to the image
I set port 3002 to prevent conflicts with current docker setup
Initial runs set up a database, runs after it use the existing database. If the data should be cleared, docker-compose down -v can be used. This can be unclear for non-compose users.
There are many more available options available (watch mode for dev/testing, ci setup, etc), I've kept it basic for now. I've also not altered the existing docker(file) setup. EDIT: I have altered entrypoint script. Please check whether this is undesirable.

Thank you @bvkatwijk for the detailed and well made notes with PR. I understand less about docker-compose and k8s - learning them now.
Can you please mention why python is included and how changing entrypoint changes  behaviour (if any).

While installing dev dependencies of npm I got an error that python is required. Hence the python install. Same for g++ and make packages.
I recommend the docker docs about entrypoint and command mechanics, but loosely in practice entrypoint is what should always happen (e.g. waiting for database), and cmd is a default action (starting the api generation, or running tests, or maybe other actions). Hence I've removed the api generation command from the entrypoint script, and it can be added to the dockerfile (same as in dev.Dockerfile).


cargo fmt on cargo

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

:umbrella: The latest upstream changes (presumably #3700) made this pull request unmergeable. Please resolve the merge conflicts.

Thanks for the PR! I'm a little alarmed though at the +4k lines and I'm not super thrilled about the prospects of reviewing all 4k lines in this PR. Was there a specific change that motivated this PR?  Could it perhaps be broken up into more bite-sized chunks?
IIRC rustfmt isn't quite ready for prime-time just yet so there's likely to be many bugs inside here that'll need to be fixed up manually.

This PR just contains changes of rustfmt. I think this should be done when things are ready. Closed 😄


[SignIn.jsx] is completed !

The SignIn.jsx is completed with responsiveness !
video of proof:
https://github.com/Curious-Ecosystem/Curious-Connect/assets/94531691/cb34c994-499d-4259-a609-b7c7a1531e52
made some changes:

Checklist

[x] My code follows the style guidelines of this project
[x]  I have performed a self-review of my own code
[x]  My changes generate no new warnings
[x]  My changes does not break the current system and it passes all the current test cases.

routing is not happening @sarthu210

routing is not happening @sarthu210

Did you added routing to the button in homepage @sarthu210

I will add it by today!

Did you added routing to the button in homepage @sarthu210

I will add it by today

Did you added routing to the button in homepage @sarthu210

done


Document merge process

As long as this project is active, it's important to stay relatively up to date with upstream changes. To make this a little easier when I get out of the mindset of this project (and to help others contribute), document the process of merging from upstream.
First merge issue is here, and was straightforward.

Done, here.