Ability to hide/remove all snack bars in the scaffold's queue

Use case
I have a complicated form a user can fill out. They have the option of adding and removing rows of text fields. Every time a user removes a text field row, a snackbar appears with an "UNDO" action. If the user removes multiple rows, then a queue of snackbars is formed for each removed row.
Let's assume the user removes several rows. Right after doing so, they tap "save," which changes the screen to a friendly read-only format and sends the form data to a database. Right now, a snackbar is still visible with an "UNDO" option. Should the user tap this, the page would try to re-insert the row and possibly crash the UI if this scenario wasn't handled properly. Even if the UI did not crash, the user might think they successfully undid their deletion action, but in reality, the database still contains the form in its original state.
While you can say something like scaffoldKey.currentState.hideCurrentSnackBar() whenever there user saves the form, this only removes the currently visible snackbar if there is one, and you have no way of knowing if another snackbar is still in the queue.

Proposal
I have 2 possibilities in mind:
1:
Create a getter [ScaffoldState.snackBars] for the private variable _snackBars so that developers can do something like:
while (scaffoldKey.currentState.snackBars.isNotEmpty) {
   scaffoldKey.currentState.hideCurrentSnackBar();
}

2:
Create two new methods that empties the queue of snackbars.
void removeAllSnackBars({ SnackBarClosedReason reason = SnackBarClosedReason.remove }) { ... }

void hideAllSnackBars({ SnackBarClosedReason reason = SnackBarClosedReason.hide }) { ... }

Final thoughts
The first proposal offers more control to the developer if they only want to pop a certain number of times or check some condition each round, but it may not be as obvious to developers they can do this. The second approach is easier for a developer to use, but offers less control. Perhaps offer both?

cc @Piinks

Hey @Piinks, I've submitted a PR implementing the first case of the proposal. Let me know if the second case also needs to be implemented :smile: . Also, should this be part of a separate PR?

Hey @chinmoy12c thanks for the PR! I think the first case proposed here may have some issues, and the seconds seems ok, but we may need to tweak in order to implement properly.
I don't think exposing _snackbars is a great idea. It leaves open the opportunity to break things in a lot of wild ways I probably can't fully imagine. :) Plus, I don't think the proposed use of it would work right..

while (scaffoldKey.currentState.snackBars.isNotEmpty) {
  scaffoldKey.currentState.hideCurrentSnackBar();
}

hideCurrentSnackBar is meant for the current SnackBar that is already on screen. So I don't know that this would work since SnackBars in a queue are not on screen yet. Only one is on screen at a given time. removeSnackBar might, but then I wonder, what is the SnackBarClosedReason in this case? When removeCurrentSnackBar is called, it completes (ScaffoldFeatureController._completer) with that reason. Should all the SnackBars in the queue be completed? If their completion triggers something else, should that still happen?
My initial impression is that we should provide a separate, single method that clears the queue, and if one is showing currently, triggers a graceful animation out. I am thinking we might want to handle the one that is visible last, because IIRC it is the animation out that triggers the next SnackBar in the queue to show up.
What do you think?

Hey @chinmoy12c thanks for the PR! I think the first case proposed here may have some issues, and the seconds seems ok, but we may need to tweak in order to implement properly.
I don't think exposing _snackbars is a great idea. It leaves open the opportunity to break things in a lot of wild ways I probably can't fully imagine. :) Plus, I don't think the proposed use of it would work right..

while (scaffoldKey.currentState.snackBars.isNotEmpty) {
  scaffoldKey.currentState.hideCurrentSnackBar();
}

hideCurrentSnackBar is meant for the current SnackBar that is already on screen. So I don't know that this would work since SnackBars in a queue are not on screen yet. Only one is on screen at a given time. removeSnackBar might, but then I wonder, what is the SnackBarClosedReason in this case? When removeCurrentSnackBar is called, it completes (ScaffoldFeatureController._completer) with that reason. Should all the SnackBars in the queue be completed? If their completion triggers something else, should that still happen?
My initial impression is that we should provide a separate, single method that clears the queue, and if one is showing currently, triggers a graceful animation out. I am thinking we might want to handle the one that is visible last, because IIRC it is the animation out that triggers the next SnackBar in the queue to show up.
What do you think?

Yes, now that I think about it, you're completely right. The idea of cleaning the queue seems rather clean. I'll implement this tomorrow, thanks for this :+1: :smiley:


Picoprobe not detected when mounted in WSL

Describe the bug
Picoprobe not detected when using ubuntu WSL2 and usbipd
To Reproduce
Steps to reproduce the behavior:

Set up usbipd in windows so that you can attach a usb device to wsl

Find the picoprobe with usbipd list, for me it's BUSID 4-3
Attach the probe to wsl with usbipd wsl attach --busid 4-3
Verify the device is attached in wsl with lsusb, for me I see Bus 001 Device 002: ID 2e8a:0004 Raspberry Pi Picoprobe

Attempt to detect the probe with probe-rs probe-rs-clie list, I get "No devices were found."
Try again as sudo, same result

Expected behavior
I expect that the attached USB device would be detected as an available probe in ubuntu but instead it shows no available probes.
Additional context
I originally was trying to get things functioning in a vscode devcontainer. I got as far as getting the usb device properly mounted but ran into this same issue so I opted to simplify and try to get just WSL working first.

Are you using this probe: https://github.com/raspberrypi/picoprobe?
We unfortunately don't support it, because it's using a custom protocol. The following alternatives should work with probe-rs:

https://github.com/majbthrd/pico-debug
https://github.com/majbthrd/DapperMime/

I wouldn't recommend pico-debug, that is currently not working well for C code or at all for Rust.
The list of probes that have been tested with rp2040 (and instructions for setting up DapperMime) are in the rp2040 docs.
https://github.com/rp-rs/rp2040-project-template/blob/main/debug_probes.md

We unfortunately don't support it, because it's using a custom protocol
Understood, I was originally using probe-run but I opened the issue here because I saw the same behavior in probe-rs-cli and assumed the issue stemmed from here.

I've been able to get picoprobe + rust working well with openocd and gdb fwiw.

Are you using this probe: https://github.com/raspberrypi/picoprobe?
We unfortunately don't support it, because it's using a custom protocol.

Would you mind elaborating on this?
The documentation said that it uses:

A vendor-specific interface for SWD probe data conforming to CMSIS-DAP v2.

Since I'm just getting started with the whole debuggers ecosystem it seems to me that it would be a big win if probe-run is able to work with such a setup (one Pico as debugger using picoprobe).

The current version should be supported, yes. The initial version used a different protocol.

I will add docs soon to probe-run but the gist is this:

You need the special release of uf2 file that supports CMSIS DAP v2 found here:
https://github.com/raspberrypi/picoprobe/releases/tag/picoprobe-cmsis-v1.0.1 (latest version in the time of writing this)
Run probe-run

Interestingly, src/openocd -f interface/cmsis-dap.cfg -f target/rp2040.cfg -s tcl fails:
Info : auto-selecting first available session transport "swd". To override use 'transport select <transport>'.
Info : Hardware thread awareness created
Info : Hardware thread awareness created
Info : RP2040 Flash Bank Command
Info : Listening on port 6666 for tcl connections
Info : Listening on port 4444 for telnet connections
Info : Using CMSIS-DAPv2 interface with VID:PID=0x2e8a:0x000c, serial=E660C0D1C7144034
Info : CMSIS-DAP: SWD  Supported
Info : CMSIS-DAP: FW Version = 2.0.0
Info : CMSIS-DAP: Interface Initialised (SWD)
Info : SWCLK/TCK = 0 SWDIO/TMS = 0 TDI = 0 TDO = 0 nTRST = 0 nRESET = 0
Error: CMSIS-DAP command CMD_DAP_SWJ_CLOCK failed.

Notes: I'm using the picoprobe branch of RP pi openocd found here: https://github.com/raspberrypi/openocd/tree/picoprobe

You shouldn't need a special branch of openocd anymore, cmsis-dap should work with standard openocd now. The picoprobe branch seems a bit outdated.

Yes, thank you for mentioning this, I did try it with the default branch and updated my comment accordingly.
Hopefully it would help other people having the same issues.


prep for 1.8 release

We still need this PR, but I think we can hold off on merging it until after the release is built and published.
Our build-images action uses the GitHub release tag name and not the IMAGE_VERSION environment variable to tag the images. https://github.com/open-telemetry/opentelemetry-demo/blob/main/.github/workflows/build-images.yml#L30


change the order of the columns

hi i had make the table like :

Partner
Col0
Col1
Col2
Col3
computed col

AA
67
45
11
55
0.308988764

AB
11
76
37
36
0.225

AC
82
28
98
73
0.259786477

AD
62
35
87
87
0.32103321

can i change the order of the columns table to

Partner
Col0
computed col
Col1
Col2
Col3

AA
67
0.308988764
45
11
55

AB
11
0.225
76
37
36

AC
82
0.259786477
28
98
73

AD
62
0.32103321
35
87
87

thnks before ...

No, this is not possible.
But you can create 3 computed columns that just reference col1, col2 and col3, and then hide original col1, col2 and col3 columns

A last update, with version 1.8.0, you can now have “filter” links on computed columns that simply reference a column: col0
That allows you to have the column order that you want, without loosing filter capabilities

Question is answered and no feedback since one month, so I close the issue.

Hi fbaligand, could you please provide more detail in your answer? I have the same question as OP (although I have TWO computed columns at the end of my table which should actually come earlier on, instead of just one), and I can't work out how to use your advice to achieve this.
As an example, say I have the computed column col['Computed A'] and I want it to follow col['Col A'] and I have col['Computed B'] and want it to follow col['Col B']. How can I rearrange to make this happen?

Hi,
You can do it like this:

computed column 'Computed A'
then a computed column, with formula: col['Col A'] (so it is a computed column that just display 'Col A' column content as is)
computed column 'Computed B'
then a computed column, with formula: col['Col B'] (so it is a computed column that just display 'Col B' column content as is)
and finally, in hidden columns option: Col A,Col B

My problem is a bit tangential, but basically gets me here.
I am trying to rename the first column when using a Filters agg. Adding a computed column that simply equals col['filters'] allows me to get the name I want (Priority, in this case), but I need the column to be on the left, not the right.
I also use a terms agg to split columns, and the number of terms that comes back is somewhat variable (6 or less). Because the number of columns changes, I can't just replace them all with computed columns.
Thoughts? If there's another way to change the 'filters' column name, I'd be happy to do that. I mucked around in public/agg_types/buckets/filters.js but I only succeeded in breaking stuff.

Hi,
If you just want to rename your "Filters" column label, it is possible using Kibana core, with a label for each filter.
See this screenshot;

That changes each filter's label, but not the label of the column itself

There's a neglected (but open) issue in Kibana regarding the same point https://github.com/elastic/kibana/issues/26737

Ah OK.
So yes, you can do that with enhanced table.
You add a computed column with “col0” as formula. Then you set “0” in “hidden columns”

Yep. Except the new column is on the right, and I need it on the left. Just like this thread was originally about ;)

Indeed, this is not really convenient.
For now, the only way is to create a computed column, for each "normal" column at the right of "Filters" bucket.
But, I admit that this would a nice enhancement.

hi fbaligand, I had issue, if i want to change order columns but its a date histogram columns, we cant create computed columns that just reference the col1 cause we didnt know what date will the column label, can we reference col label too?

want to be this

Hi,
First, I guess that you use "split cols" bucket, right?
Then, if I understand well, you want a computed column per split column, right?
If so, I advice you to check "computed/hidden columns per split column" option and only one computed column. The computed column formula references col0 (if you don't have "split rows" bucket).

By the way, you can reference a column by its label: col['label']

Hi,
First, I guess that you use "split cols" bucket, right?
Then, if I understand well, you want a computed column per split column, right?
If so, I advice you to check "computed/hidden columns per split column" option and only one computed column. The computed column formula references col0 (if you don't have "split rows" bucket).

no i didnt use split cols, cause this count % increase data this day per data yesterday, its cant using split cols by date histogram.
or can we get data yesterday for date today?
and i mean get the label of col, example col0 label is "2020-09-01" than in computed label i get thats label

Thanks for the precision.
Could you tell me what formula you use currently?

Thanks for the precision.
Could you tell me what computed column formula you use currently?

in computed column i use formula (col1-col2)/col1 for H+1 %, (col2-col3)/col2 for H+2 %, (col3-col4)/col3 for H+3 %, (col4-col5)/col4 for H+4 %, (col5-col6)/col5 for H+5 %, and (col6-col7)/col6 for H+6 %.so we can see the increase data per day,
can we use "computed/hidden columns per split column" for this?

Hi,
Given your formula, I guess that you have also first 2 columns that you hide, that are "Split Rows" bucket, right?
For your formula, when I see "H+1" column value, it seems to be: (col3-col2) / col2, right?
I made some tests, and found a way that should answer your need!
This is how you can do:

first, on Data tab, add a "Serial Diff" in Metrics. The custom metric inside is the same metric that the one you currently use (probably Count).
enable  "Computed/Hidden cols per split col"
add only one computed column with this settings:
-- label: H+1
-- formula: diff = col(3) ? col(3) : 0; diff / (col2 - diff)
-- pattern : 0.00%
in "Hidden columns", set 3

You're done!

Hi,
I just published Enhanced Table v1.11.0! And it includes a new feature named "Custom column position".
This new setting enables you to define the target position of the computed column. And it can be any position, including 0!
More info on release here:
https://github.com/fbaligand/kibana-enhanced-table/releases/tag/v1.11.0

Hi,
I just published Enhanced Table v1.11.0! And it includes a new feature named "Custom column position".
This new setting enables you to define the target position of the computed column. And it can be any position, including 0!
More info on release here:
https://github.com/fbaligand/kibana-enhanced-table/releases/tag/v1.11.0


feat: add pog log steps and pod label assertion step

Some of the pod logs steps are difficult to cover with unit tests due to not having way to mock pod logs. If we would like to consider adding e2e tests for kubedog in future, might be a good coverage approach.
Will add docs if changes look good
Steps added:
kdt.scenarioContext.Step(`^the pods in namespace ([^"]*) with selector ([^"]*) has ([^"]*) in logs since ([^"]*) time and times out in (\S+) seconds`, kdt.KubeContext.ThePodsInNamespaceWithSelectorHasThisSentenceInLogsSinceTime)
	kdt.scenarioContext.Step(`^the pods in namespace ([^"]*) with selector ([^"]*) doesn't have ([^"]*) in logs since ([^"]*) time$`, kdt.KubeContext.NoMatchingStringInLogsSinceTime)
	kdt.scenarioContext.Step(`^the pods in namespace ([^"]*) with selector ([^"]*) have no errors in logs since ([^"]*) time$`, kdt.KubeContext.ThePodsInNamespaceWithSelectorHaveNoErrorsInLogsSinceTime)
	kdt.scenarioContext.Step(`^the pods in namespace ([^"]*) with selector ([^"]*) have some errors in logs since ([^"]*) time$`, kdt.KubeContext.ThePodsInNamespaceWithSelectorHaveSomeErrorsInLogsSinceTime)
	kdt.scenarioContext.Step(`^the pod ([^"]*) in namespace ([^"]*) should have labels ([^"]*)$`, kdt.KubeContext.ThePodInNamespaceShouldHaveLabels)

Can you change this to merge into GED-kubedog instead of master?

Yup, can do. My bad.


VM must consume nameservers from virt-launcher pod

Currently, we add only google nameserver to VM and do not consume nameservers from virt-launcher pod
Pod:
# cat /etc/resolv.conf
nameserver <IP_ADDRESS>
search kubevirt-test-default.svc.cluster.local svc.cluster.local cluster.local
options ndots:5

VM:
$ cat /etc/resolv.conf
nameserver <IP_ADDRESS>
nameserver <IP_ADDRESS>
nameserver <IP_ADDRESS>
nameserver <IP_ADDRESS>

FYI @vladikr

dnsReadConfig from https://golang.org/src/net/dnsconfig_unix.go looks promising

Let's also start tweaking the hostname and the subdomain of the pod which we create for the vm. See: https://kubernetes.io/docs/concepts/services-networking/dns-pod-service/#pods-hostname-and-subdomain-fields
We would set the pods hostName to the name of the vm and set a subbomain on the pod which we could simply call virtualmachines or vms or vm. Then we can simply set the hostname of the vm to the name of the vm. Further we will have reasonable dns entries which will point to our vms. For instance a FQDN like myvm.virtualmachines.my-namespace.svc.cluster.local.
Finally the dns server can offer the correct host name (and cloud-init metadata too).

Also related to https://github.com/kubevirt/kubevirt/pull/775#discussion_r171485334


fix a syntax error in setup.py

This fixes this error:
  File "setup.py", line 31
    "classifiers": [
                 ^
SyntaxError: invalid syntax

Not sure if there is a python version where rtslib-fb 2.1.fb55 works, but it doesn't with Python 3.4.2 or 2.7.8 and show the error above. I build the package on Arch Linux with the fix in this PR.

doh! ok, going to do a fb56 with just this.


docs(README): fix Datadog Browser SDK version and dead link from datadog_flutter_plugin

What and why?
datadog_flutter_plugin README is confusing as it mentions that the Browser SDK version to be used is 4.x.x but then says to use the CDN for Datadog Browser SDKs v5.
It also has an internal dead link to "Flutter Web Support", header which was previously removed in version v2.1.0.
Review checklist

[ ] This pull request has appropriate unit and / or integration tests
[ ] This pull request references a Github or JIRA issue

Hi @Gummiees,
Datadog requires all commits to be signed. Can you sign your commit? If not, no worries, I've already fixed this in #556

Hi @Gummiees,
Datadog requires all commits to be signed. Can you sign your commit? If not, no worries, I've already fixed this in #556

Oh didn't know that, sorry! Signed now but the flow still fails, can merge your PR instead and close this one 👍 thanks

Incorporated into #556


Cleaning a bit, mostly PEP8, used os.path.join to "join" paths

Wow, glad someone cared enough to take care of this. <_< I know the code was definitely a mess.
I'll do some testing tonight to make sure it is working as expected. If good, I'll merge the PR.

I'm gonna clean the rest of the code.

You didn't need to close the PR. Any additional commits are automatically
included, so as long as I don't rush to approve, it would have been fine.
Regardless, thanks for your hard work :D
On Thu, Aug 27, 2015 at 10:26 AM, Krzysztof Kwaśniak <
<EMAIL_ADDRESS>wrote:

I'm gonna clean the rest of the code.
—
Reply to this email directly or view it on GitHub
https://github.com/StarryPy/StarryPy/pull/140#issuecomment-135499906.


BREAKING CHANGE: v0.16 will remove MediaMessage class

From v0.16, MediaMessage will be deprecated.
From v0.18, MediaMessage will be removed.
Use Message instead.
Migration Example
- bot.say(new MediaMessage('/image.png')
+ bot.say(new Message('/image.png')

不行 ding-dong 那个二维码报错发布出去啊

Yes, it can't.
Currently v0.15 is in heavy developing, please use it only if you do want to read the source code and help us to find out the bug(with a fix PR will be better).
You can use v0.14 instead and switch to v0.16 after it published.
See also #1158

See: #1200

Code Example can be found at #1092


Using Goofy with a deactivated Facebook account

I deactivated my Facebook account, but opted to keep my Messenger account. I noticed that when starting Goofy, I see the Facebook login and not the Messenger login. Is it possible that Goofy reactivates my Facebook account when logging in?

Goofy uses https://www.facebook.com/messages as the gateway internally so you can easily test it: if there are no complications when using it through there in a browser, you should be good to go.

Dang, it doesn't look like I can log in there without reactivating my account. Any specific reason why the app doesn't use https://www.messenger.com/login?

I'm not sure why that route is used at the moment, it may be something to do with easier integration with the Facebook for Work chat, but it's hard to tell unless @danielbuechele chimes in himself. See https://github.com/danielbuechele/goofy/issues/354 for a related discussion.

The Facebook route is used, because that way Goofy and Goofy for Work can use the same code.
I think not having a Facebook account, but using Messenger is a real edge-case. And if there are no other reasons, I would stick to facebook.com instead of messenger.com

Yeah, I just deactivated my FB but left messenger.  www.messenger.com works fine but Goofy is no longer an option.  Sad.

As in #375, have now switched to use messenger.com


Improve README.md requirements speach

Fix: #90
I had already npcap installed on my computer with wireshark but without compatibility mode... I thought I already have it (and when I read the readme, i didn't understand that I should reinstall this dependency and check "Compatibility mode").

I'm not particularly keen on linking to a 3rd party image like that. IMO the readme is pretty clear as it is.

And with github ?
image


v1.5 extension testing

https://github.com/microsoft/vscode-azureresourcegroups/issues/525#tasklist-block-8274660b-eec1-4c89-8b77-213eb540e858
https://github.com/microsoft/vscode-azureresourcegroups/issues/525#tasklist-block-980e076b-194e-411a-9a1e-f72cde359f1a
https://github.com/microsoft/vscode-azureresourcegroups/issues/525#tasklist-block-427f6d56-12e0-429a-81bd-0aa893df0353

I'll just include this in #390


Refs #28859 -- Fixed "no data found" exception handling with cx_Oracle 6.3+.

https://code.djangoproject.com/ticket/28859
Behavior changed in cx_Oracle 6.3.

buildbot, test on oracle.

buildbot, test on oracle.

I'd use a single line: # cx_Oracle < 6.3 returns None, >= 6.3 raises IndexError.

buildbot, test on oracle.

buildbot, test on oracle.

@timgraham @pope1ni Thanks. I updated PR.


Sl update exercise numbers

Correcting exercise numbers to match the class number
Merging current Hypothesis Space exercises in ex_information_theory_1.pdf
Creating blank file for ex_information_theory_2.pdf.  New exercises will be added in another PR

migrated to lecture_sl


Upgrade to the current pytest

https://pypi.org/project/pytest

Thanks for bringing this to my attention. Why did you delete poetry.lock?

I did not know how to use Poetry when I wrote this...  I have learned since.

I see. I have upgraded pytest with the rest of the deps.


GCE Loadbalancer controller doesn't report events anymore

From logs:
E0111 00:56:27.615692       1 event.go:252] Unsupported event type: 'ADD'
...
E0111 00:57:39.362128       1 event.go:252] Unsupported event type: 'CREATE'

Maybe https://github.com/kubernetes/contrib/pull/358

@bprashanth it's possible, please check:
https://github.com/brendandburns/contrib/commit/cdccdfdafe0a537530433b7467a6de7ac9497512


do not have permission to view this directory or page

The azure website url is http://webapp6.secureenvironment.p.azurewebsites.net/.
I tried to deploy my asp.net mvc web application to azure. But somehow, I could not using web deploy in vs2015 community, I always receive an error:
Web Deploy experienced a connection problem with the server and had to terminate the connection. Contact your server administrator if the problem persists. Learn more at: http://go.microsoft.com/fwlink/?LinkId=221672#ERROR_CONNECTION_TERMINATED. WebApplication6 0.
So I changed to deploy using local git to deploy to azure. I was able to deploy, but when I typed in the url of the web app, I got:You do not have permission to view this directory or page. And I can't access any of the controller I defined in the code.

Note that Web Deploy is not part of Kudu. Please use the forum to discuss those issues.
For the git deployment, can you give more details about your application? e.g. is it classic .NET, .NET core, Node, etc...? If you can share a small repo that shows the issue, that can also help investigation.

I would say it is a classic .net application using Entity Framework. No much difference from the tutorial 's web app :http://www.asp.net/mvc/overview/getting-started/introduction/adding-a-model

MVC apps should work fine in Kudu. Please do share a repo so we can try to understand what's going on with yours.

I took a look at your app, and it's a Core app based on the old RC1 release (previously called ASP.NET 5). This is no longer supported. You have two main choices for a Web App:

Create a regular ASP.NET application
Create a .NET Core application. To do this, get the latest bits from https://www.microsoft.com/net (it was just released on Monday)

Hi I have a project which is a class library and it contains angular js related files, using npm and yarn on my local machine I am able to run application, after publishing in azure as a webapp/Node js empty webapp, its showing "You do not have permission to view this directory or page." , please tell me the reason and how to solve this?

@srinivasgnt start with this.
But unless you are using an outdated Core build, your issue is unrelated to this one.


Fix source download link when no code unit allowed (#3166)

back port from #3166

LGTM

LGTM

LGTM


Fix koparse... again.

Changes
The tag may contain lowercase and uppercase letters, digits,
underscores, periods and dashes:
https://docs.docker.com/engine/reference/commandline/tag/#extended-description
Extend the regex to include all valid characters.
Signed-off-by: Andrea Frittoli<EMAIL_ADDRESS>/kind misc
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ ] Includes tests (if functionality changed/added)
[ ] Includes docs (if user facing)
[x] Commit messages follow commit message best practices
[x] Release notes block has been filled in or deleted (only if no user facing changes)

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

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

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

/cc @vdemeester

/lgtm


pycpp: Use Google style and cpplint

std includes should be sorted and de-duped.

If you're not sure about any of the latter commits, feel free to merge the earlier commits.


Module has not attribute "Env"

Awesome work on this @kylebarron!
After installing and checking out some sample code, I'm getting the error referenced.

I see an Env stub exists in the source, and I can avoid the error by replacing my call with rio.env.Env. I also tried adding the import to get the original statement recognized, but I can't seem to get that working. Any idea what I'm doing wrong there?

This was never completed so some things might work. If it does work when using rasterio.env.Env, it might just be the __init__.pyi import that needs to be fixed.
Can you try
from rasterio.env import Env as Env

in the __init__.pyi instead? I think in stub files anything imported without as is only for internal use, and you need as Env to re-export it.

If that works a PR would be welcome


Updated Engram-Qwerty layout to work with OSX shortcuts

Some shortcuts were not working correctly
ex. command-= will now work using command-shift-=

Thank you, @iklarman!


shell_exec does not work correctly when using "awk" before generating report

Hi Support,
I'm facing the problem with "awk" and "goaccess"
My PHP script:
$cmd = "awk '$7 ~ /schedule/' /var/logs/httpd/access_log* | goaccess -a -o /var/www/html/log/generate_report.html"
$output = shell_exec($cmd);
Error Log:
AH01215: awk: cmd. line:1: (FILENAME=/var/logs/httpd/access_log-20170508 FNR=126) fatal: print to "standard output" failed (Broken pipe)
This issue only happens when output of "awk" is input of "goaccess", because when I use another "awk" instead of "goaccess", it still works well. Ex:
$cmd = "awk '$7 ~ /schedule/' /var/logs/httpd/access_log* | awk '{gsub(/[/]+/,"/",$7); print }'"
So, there is any way to fix this issue? Thank you so much.
Looking forward to knowing your response.

Hi Support,
Thank you so much for your contribute.
I have resolved this issue by myself.
My solution: write filtered record to a new file before generating the report from the new file.
Script:
// Filter logs
$cmd = "awk '$7 ~ /schedule/' /var/logs/httpd/access_log* > /var/logs/httpd/filtered_log"
shell_exec($cmd);
// Generate report
$cmd = "goaccess /var/logs/httpd/filtered_log -a -o /var/www/html/log/generate_report.html"
shell_exec($cmd);
Let me know if you have a better solution for this :)
Thanks again.

Using
$cmd = "awk '$7 ~ /schedule/' /var/logs/httpd/access_log* | goaccess -a -o /var/www/html/log/generate_report.html"
$output = shell_exec($cmd);

what output do you get? Also please try adding a single dash - to let goaccess know it should read from a pipe. e.g.,
$cmd = "awk '$7 ~ /schedule/' /var/logs/httpd/access_log* | goaccess -a -o /var/www/html/log/generate_report.html -"

Hi Support,
Amazing, your suggestion ( a single dash) works perfectly. But, seems when running command directly on the server, it still work correctly even with or without a single dash. Anyway, it's my mistake and thank you so much for your help :). I very appreciate it. I will close this question now.

@nhochong Glad that worked :) Running it through shell_exec and awk probably does not give you a terminal and thus the dash is needed.


'tests' directory in reposit ( zip file)

can you add in reposit/zip file the directory "Tests" ?

@Erimotte What's the point of this?

This is to have all the version of dbf resources and all the tests for possibly a fork. thx


fix: remove unused prop in HeaderBackButton

Hello!
Following the conversation of https://github.com/flow-typed/flow-typed/issues/3574 I'm opening this PR in order remove some props that seems to be unused in the HeaderBackButton component.
Sorry if I missed something, I'm not really familiar with the react-navigation codebase.
Thanks in advance

Thanks for the PR.
Regarding the props, they are not used by default back button, but a custom back button might be using them. Hence removing these will be a breaking change.
I'm closing this since we're not planning to do breaking changes in 1.0.


docs: webpack-merge also applies to rspack

Summary

Checklist

[ ] Tests updated (or not required).
[ ] Documentation updated (or not required).

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.wangjiwei seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Update testing framework and cleanup syntax

This updates the testing framework and does some minor cleanup.

Copy files from caskdata/cdap_cookbook

.codeclimate.yml
.rubocop.yml
.travis.yml
Berksfile
spec/spec_helper.rb

Test latest Chef by adding Ruby 2.3.1 to Travis configuration and Gemfile
Remove Ruby 2.0 from Travis configuration as it was (mostly) redundant with 2.1
Add require 'chef/version' to Berksfile
Add chef_version to metadata
Switch to Ruby 1.9+ hash syntax

Since this just updates the tests and they passed, I'm merging.


Update configuring-nuget-behavior.md

Corrected a dub.

Docs Build status updates of commit 7d397af:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/consume-packages/configuring-nuget-behavior.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@Eshva thanks for fixing that!

No problem.
Karan Nandwani<EMAIL_ADDRESS>20 июня 2019 г. 1:54:57 написал:

@Eshva thanks for fixing that!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Refactor RoutingConfigurationController to use Form Validation and make sure it follows the How-To-Write-Controllers guide

The RoutingConfigurationController needs to be refactored so that it uses spring form validation. Also, make sure that the controller returns the proper responses according to the How-To-Implement-Spring-Controllers when there is an error in the validation.
Validation rules for RuleSet:

Field
Validation

groupRepository
required

routingRules
required

Validation rules for RoutingRule:

Field
Validation

pattern
required

repositories
required

Fixed!


redirect to totp project

Codecov Report

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

@@          Coverage Diff          @@
##           master    #38   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           6      6
  Lines         183    183
=====================================
  Hits          183    183

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 5591879...ae6a682. Read the comment docs.


Implement Natural Language Classifier service SDK

[x] Create NLC class and model
[x] Write comments in the code
[x] Create unit test cases
[x] Document SDK API reference

Finished, closing this issue.


Adding ability to configure conda-store via Traitlets

Considered several options for making conda-store configurable. Ultimately the python configuration that is possible via traitlets allows for heavily customization that would be hard without it.
Packages worth looking at:

dask-gateway
jupyterhub
jupyter-ssh

These can show how to configure traitlets in a way that is convenient to end users.

This will be an issue that will encompass much of the code. Currently cli options are used to configure conda-store. However, over time is has become apparent that there will be too many cli options. Thus is would be nice to use traitlets to configure all of these options.


Pillar 2: Security - Presentation Protocols

I would argue bullets i-iv & vi describe requirements for the implementation of a wallet while only v is about the protocol between wallet and verifier. I suggest to add the following requirements:

protocols shall ensure the integrity of the exchange, e.g. prevent injection, phishing, and session fixation attacks
protocols shall ensure confidentiality of the exchange, e.g. prevent credentials from leakage
protocols shall prevent replay of credential presentations being exchange, e.g. through binding of the presentation to a transaction specific nonce and an audience
protocols shall enable the holder to reliably authenticate the verifier (as a basis for informed consent and potential follow-up actions, such as complaints)

I would also encourage you to add a similar section on issuance protocols.

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."

I think these four additions are valid and useful additions, thanks @tlodderstedt. The first three are fairly generic for issuance and presentation (and other bidirectional interactions).
Would you mind submitting a pull request to this effect, but leave out the words in brackets on the last bullet point as we don't want to specify particular follow-on actions).


Abandoned Industrial - Duplicates

Duplicate containers Loot_Storage_Cooler_Common "Cooler" CONT: 001F9900

Details
000EF7B9 & 000EF7BC Loot_Storage_Cooler_Common "Cooler" CONT: 001F9900 are duplicate coordinates. No BGS Reflection on cell. Not a good idea to disable references if not absolutely needed so I moved 000EF7BC to the ground in front of the handtruck to the following coordinates:
X:-6.928970
Y:35.369999
Z:9.950000
Angle
X:0.0
Y:0.0
Z:352.2321
Game Version
<IP_ADDRESS>
Language
English (EN)
New Game
Yes
Reproduction Steps
Description

Fix:
SFCP_Issue_848.zip

also raised as #304


Debian disagrees about CVE-2015-8618

trying to build rkt on Debian stable and see
checking whether we have a go version without CVE-2015-8618... no
configure: error: *** go version is vulnerable to CVE-2015-8618 (1.5.1)
but according to Debian it's ok
https://security-tracker.debian.org/tracker/CVE-2015-8618

never mind / totally misread it, wondering where my golang came from.


Old IRD data issues (IRD with a 20 cm telescope)

I am now analyzing the Jupiter data as observed by IRD with a 20cm telescope (very old data) and found some issues:

One needs to rotate the detector image in 90 degree to match to the current setting

There are some zero stripes in the detector images. The width is 2 pixel and the interval is 128 pixels.

$$ j = k + 128 i  $$
for
$$ k \in (63, 64), i = 0, 1, \cdots, 15 $$

The issue 2 prevents aptrace. This can be solved by filling the zeros rows by nearby rows, i.e. 63 by 62, 64 by 65.

nap is 21 for H, i.e. only star fiber

mmf1 and mmf2 in trace_mmf are confusing. I propose to rename them (in the next PR).

something wrong...

I found this issue was caused by two factors:

The rotation of image was only applied to Flat image
Need 90 degree rotation and y-axis inversion -- without inversion, identification of Th-Ar emission line will be failed.

I confirmed that the pipeline works with manually rotated images (not yet implementing rotation instance in irdstream). This figure compares spectra from images applied rotation and rotation + inversion. Telluric lines show that the latter (rotation+inversion) is correct.

Thank you! I see, I will try to implement (and refactor) those part.

reproduced! Note that the inverse is 0-th axis. (y=0, x=1?)


app run or clear getting : Plugin with id 'maven' not found.

Describe the bug
Getting below issue while clean and run react native project:

Where:
Build file 'C:\May2023\EPFleetApp\node_modules\react-native-tesseract-ocr\android\build.gradle' line: 23

What went wrong:
A problem occurred evaluating project ':react-native-tesseract-ocr'.

Plugin with id 'maven' not found.

Dependencies (please complete the following information):
"react": "18.1.0",
"react-native": "^0.70.2",
"react-native-tesseract-ocr": "^2.0.3",
To Reproduce
Steps to reproduce the behavior:

react-native run-android
or
./gradlew clean
Expected behavior
run app
Screenshots
If applicable, add screenshots to help explain your problem.
Smartphone (please complete the following information):
emulator

I am having the same issue. I investigated a bit about it and seems that on Gradle 7 and up maven plugin was replaced for maven-publish. I think the repo needs a restructure to support Gradle 7

after change to maven-publish , get different error:
`A problem occurred configuring project ':react-native-tesseract-ocr'.

Resolving dependency configuration 'implementation' is not allowed as it is defined as 'canBeResolved=false'.
Instead, a resolvable ('canBeResolved=true') dependency configuration that extends 'implementation' should be resolved.`

Hi any updates on the issues? I changed maven to maven-publish but now getting an error
A problem occurred configuring project ':react-native-tesseract-ocr'.
> Configuration with name 'compile' not found. * Try:

@green-mint @rizaru-desu did you get any solution I got the same error

No solution, switched to other library for ocr

No solution, switched to other library for ocr

name it plz

No solution, switched to other library for ocr

name it plz

I switched to React Native ML - Text recognition.


switch to offline evaluation paradigm and add save/load

@RealZiangLiu please check my changes to run_single_experiment.py extra carefully because subtle bugs there would be doom

@RealZiangLiu please check my changes to run_single_experiment.py extra carefully because subtle bugs there would be doom

Everything looks good except for the two minor checks needing confirmation above : )


Add WordPress feed urls to TimberSite

Urls for rdf, rss, rss2 and atom are retrieved

Thanks @rpkoller -- this is merged in #721


(configgrpc): fork configgrpc and add memorylimiterextension config option

There is ongoing community discussion about applying a memory limiter extension (https://github.com/open-telemetry/opentelemetry-collector/issues/9591), but no indication of timeline until PR's that apply the extension can be merged. This PR forks configgrpc and applies the PR (https://github.com/open-telemetry/opentelemetry-collector/pull/9673) which allows a grpc server to configure a memorylimiterextension.
We have reliability issues with our collectors that customers are consistently noticing which started out as major issues with the exporterhelper. Forking this code unblocks our continued work to improve collector reliability by allowing us to limit the amount of memory a receiver can hold.

Let's re-open this when we decide to make progress.


增加armdockerfile

增加armdockerfile

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Update footer links and add download app badge

https://artsyproduct.atlassian.net/browse/GRO-129

Updates the footer to reorganize links and use new links where appropriate.
Adds a "Download on the app store" badge to the footer that links out to the app store and tracks clicks on this link.

cc @artsy/grow-devs
Screenshots

#mergeOnGreen

#mergeOnGreen


Report file not generated when no leaks are found in version 8.22.0

Describe the bug
In Gitleaks version 8.21.2, when running a scan that resulted in no leaks, a JSON report file was still generated, containing an empty "report": []. This was useful for consistent report handling in automated workflows.
After upgrading to version 8.22.0, I noticed that if no leaks are found, the JSON report file is not generated at all. This change disrupts workflows that expect a report file regardless of scan results.
Steps to reproduce the behavior

Run Gitleaks version 8.22.0 with the following command:

gitleaks -c "gitleaks.toml" detect --exit-code 0 --no-git --verbose -f sarif -r "gitleaks.json"

    ○
    │╲
    │ ○
    ○ ░
    ░    gitleaks

1:57PM INF scanned ~107085 bytes (107.08 KB) in 96ms
1:57PM INF no leaks found

Ensure that the scan does not detect any leaks.

Expected behavior
A JSON report file named gitleaks.json is generated, containing an empty report array.
Actual Behavior:
No report file is generated when no leaks are found.
Impact:
This change affects automated systems and scripts that rely on the presence of a report file for further processing, even when no leaks are detected.
Request:
Could an option be added to allow Gitleaks to generate an empty report file when no leaks are found, restoring the previous behavior? This would help maintain consistency in automated workflows.
To reproduce previous behaviour

Run Gitleaks version 8.21.2 with the following command:

gitleaks -c "gitleaks.toml" detect --exit-code 0 --no-git --verbose -f sarif -r "gitleaks.json"

    ○
    │╲
    │ ○
    ○ ░
    ░    gitleaks

1:57PM INF scanned ~107085 bytes (107.08 KB) in 96ms
1:57PM INF no leaks found

Ensure that the scan does not detect any leaks.
Check a json is generated with a content like:

{
 "$schema": "https://json.schemastore.org/sarif-2.1.0.json",
 "version": "2.1.0",
 "runs": [
  {
   "tool": {
    "driver": {
     "name": "gitleaks",
     "semanticVersion": "v8.0.0",
     "informationUri": "https://github.com/gitleaks/gitleaks",
     "rules": [
      {
       "id": "1password-service-account-token",
       "shortDescription": {
        "text": "Uncovered a possible 1Password service account token, potentially compromising access to secrets in vaults."
       }
      },
(....) all ids here (...)
      {
       "id": "zendesk-secret-key",
       "shortDescription": {
        "text": "Detected a Zendesk Secret Key, risking unauthorized access to customer support services and sensitive ticketing data."
       }
      }
     ]
    }
   },
   "results": []
  }
 ]
}

Basic Info (please complete the following information):

OS: Red Hat 8.7
Gitleaks Version: 8.22.0

cc @zricethezav

Same problem occurs with us. Since dec-22 our builds using gitleaks fail and this could very well be the issue. Our integration-tests on repo's with actual findings are fine. But all integration tests with no findings fail since we expect a json report-file to upload to our reporting tools

https://github.com/gitleaks/gitleaks/releases/tag/v8.22.1 should be resolved now. Thanks for opening this issue.


Removed support of YDB_LOG_NO_COLOR environment variable

Changed default behaviour of internal logger to without coloring
Fixed coloring (to true) with environment variable YDB_LOG_SEVERITY_LEVEL

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Other information

Codecov Report

Merging #345 (5f15fed) into master (47508b3) will decrease coverage by 23.98%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           master     #345       +/-   ##
===========================================
- Coverage   59.93%   35.94%   -23.99%
===========================================
  Files         203      112       -91
  Lines       22079    15775     -6304
===========================================
- Hits        13233     5671     -7562
- Misses       8037     9883     +1846
+ Partials      809      221      -588

Flag
Coverage Δ

1.14.x
35.53% <0.00%> (-24.09%)
:arrow_down:

1.18.x
35.94% <0.00%> (-23.92%)
:arrow_down:

1.19.x
35.96% <0.00%> (-23.82%)
:arrow_down:

connection
?

database_sql
?

discovery
?

e2e
?

integration
?

macOS-latest
35.96% <0.00%> (+0.01%)
:arrow_up:

ratelimiter
?

scripting
?

table
?

topic
?

ubuntu-latest
35.93% <0.00%> (-23.92%)
:arrow_down:

unit
35.94% <0.00%> (-0.12%)
:arrow_down:

windows-latest
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Migrate docker repository

Migrate docker repository from astj/mackerel-rpm-builder to mackerel/docker-mackerel-rpm-builder.

"c5" is deprecated,  use "c7" to create a package.

Thank you!


Can't Connect to Database

Describe the bug
Can't connect to database.
Steps to reproduce
This morning, I went to VS Code and my connections were not connected. I have two. They had a grey CB icon with a slash thru them. I right clicked on the connection name and clicked Connect to Cluster. On the bottom right of VS Code was a message "trying to connect" that popped up then went away.
What did you expect to see?
I was expecting to see the cluster connection change to a connected state and see the buckets.
What did you see instead?
Nothing changed.
I did create a new user and connect to the database, just to confirm my IP was still correct, and I could connect, etc. So, now I have three connections configured two that don't work, which I created yesterday, and one that does, that I created today.
What version did you use?
VSCode Version:
Version: 1.77.0 (user setup)
Commit: 7f329fe6c66b0f86ae1574c2911b681ad5a45d63
Date: 2023-03-29T10:02:16.981Z
Electron: 19.1.11
Chromium: 102.0.5005.196
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621
Sandboxed: No
Extension Version:
Published
3/29/2023, 10:49:15
Last released
4/4/2023, 15:43:52
Last updated
4/4/2023, 16:10:50
Identifier
couchbase.vscode-couchbase
[v0.2.0]

Hey @cb-davidelliott,
There is a case handled recently while connected to an already existing connection to show Error if connection fails while getting connected to the cluster. It will be updated in the upcoming release and you will be able to see the cause of connection not getting established.

Hey @cb-davidelliott ,
We have already made some changes to the flow while connecting to an old cluster and with that if connection fails then you will be able to see the cause of the ERROR.
In the next release you will be able to figure out the cause of connection failure.


Using a custom User model with SimpleJWT

Hi, I'm trying to use a custom User model/extension of the default Django auth.User model in my application.
I created the User and Manager like so:
import uuid

from django.contrib.auth.base_user import BaseUserManager
from django.contrib.auth.models import AbstractUser
from django.db import models

class CustomUserManager(BaseUserManager):
    def create_user(self, email, company_name, password=None):
        if not email:
            raise ValueError("User must have an email address.")
        if not company_name:
            raise ValueError("User must have a company name.")

        user = self.model(email=self.normalize_email(email),
                          company_name=company_name,
                          )

        user.set_password(password)
        user.save(using=self._db)
        return user

    def create_superuser(self, email, company_name, password):
        user = self.create_user(email=self.normalize_email(email),
                                company_name=company_name,
                                password=password,
                                )

        user.is_admin = True
        user.is_staff = True
        user.is_superuser = True
        user.save(using=self._db)
        return user

class CustomUser(AbstractUser):
    id = models.UUIDField(primary_key=True, default=uuid.uuid4, editable=False)
    username = None
    email = models.EmailField(verbose_name="email", max_length=60, unique=True)
    company_name = models.CharField(max_length=30)

    # The following fields are required for every custom User model
    last_login = models.DateTimeField(verbose_name='last login', auto_now=True)
    date_joined = models.DateTimeField(verbose_name='date joined', auto_now_add=True)
    is_admin = models.BooleanField(default=False)
    is_active = models.BooleanField(default=True)
    is_staff = models.BooleanField(default=False)
    is_superuser = models.BooleanField(default=False)

    USERNAME_FIELD = 'email'
    REQUIRED_FIELDS = ['company_name']

    objects = CustomUserManager()

    def __str__(self):
        return self.email

    def has_perm(self, perm, obj=None):
        return self.is_superuser

    def has_module_perms(self, app_label):
        return True

but when I create users of this model and try to login into my app with them instead of getting a JWT token like before I get a message stating that there are no active users with those credentials...
How can I use SimpleJWT with a custom User model?
Thanks

Try logging into the admin panel and lemme know if that works. I'm guessing you may not have set AUTH_USER_MODEL in settings

I have set AUTH_USER_MODEL to my custom model app_name.CustomUser and in the admin panel it doesn't appear under Authentication and Authorization anymore. And even if I register it admin.site.register(get_user_model()) it appears under app_name and not under Auth, I believe that might be the issue

PS I login to the admin panel using a superuser created with python manage.py createsuperuser that asks for the correct fields for my custom user and actually creates a user that is active, but that might be because superuser overrides the is_active flag.

I have set AUTH_USER_MODEL to my custom model app_name.CustomUser and in the admin panel it doesn't appear under Authentication and Authorization anymore. And even if I register it admin.site.register(get_user_model()) it appears under app_name and not under Auth, I believe that might be the issue

Correct, it should now appear under your project's app which houses the custom user model. This is the proper functionality, and now SImpleJWT should work.

but it doesn't :sweat_smile: I don't know what I'm doing wrong

haha sorry! Yes please check if is_active=True because we do have a(n overridable) check in place for that. The admin has nothing to do with this package; it was just a test to see if settings.AUTH_USER_MODEL was actually done correctly. Do you mind setting up a reproducible repo? That would help!

Hey, I created a repo with the files required to reproduce the issue!

The repo you provided works perfectly fine for me (though I changed database to sqlite3; this should not have any effect).
Steps I've taken:
python manage.py migrate
python manage.py createsuperuser
# Go to http://<IP_ADDRESS>:8000/token/ and post my credentials
# Received tokens.

have you tried with an user created from the admin panel? please read the readme in the repo

@obvionaoe ah my apologies. The problem is in your custom manager, you're not setting the password correctly:
>>> from app.models import CustomUser
>>> CustomUser.objects.all()
<QuerySet [<CustomUser<EMAIL_ADDRESS><CustomUser<EMAIL_ADDRESS>>>> CustomUser.objects.all()[1].password
'asdf'

The password should look like this: pbkdf2_sha256$260000$M1oIrwmo8uPU44Rt5vuy6X$lkVHYCCm3U+4FVzSUs1m0zGMUxP/lS8I1cnkCX8il8A=

This is probably what you're looking for: https://stackoverflow.com/a/69594199

Closing.
Reasoning: devs when using a custom user model need to ensure that when they are registering their models, they continue to use the UserAdmin:
from django.contrib.auth.admin import UserAdmin
from django.contrib import admin
from app.models import CustomUser

# Inherit from UserAdmin and do more stuff
# Tutorial: https://testdriven.io/blog/django-custom-user-model/#admin

admin.site.register(CustomUser, UserAdmin)

I ran into this also, so I wanted to document what was going wrong. Basically what is happening is that the error message is incorrect, and has nothing to do with custom user models. The actual root cause is circular imports, which just so happens to cause Django to fail on fetching the custom user model.
In my specific case, I was using a custom serializer to add some information into the token claims, as shown here: https://django-rest-framework-simplejwt.readthedocs.io/en/latest/customizing_token_claims.html
However in the same file I was also importing some other stuff from rest_framework_simplejwt, e.g. from rest_framework_simplejwt.authentication import JWTAuthentication.
So when this line is executed: from rest_framework_simplejwt.views import TokenObtainPairView
It resulting in a circular import. But instead of just giving a circular import error, it was complaining about the user model not being able to be found. Anyway sorry this doesn't come with a neat reproduction, but hopefully this hint at least helps someone else!


TLOS icon/tip does not open information card

It was because the symbol "TLOS" was missing from community.json "tlos' tag entry.


Fix HTTP endpoint not passing along query string parameters

The above commits will cause the tests to not pass, demonstrating the issue. Once demonstrated, I'll commit a fix.

One question, also thinking we need a version bump in package.json.

I attempted to bump the version, but Travis CI didn't like it.
https://travis-ci.org/ConduitVC/aws-utils/jobs/459859998

@pianomansam -- my guess is that's because we stick to MAJOR.MINOR.REVISION.
I think this calls for a REVISION increment by 1.


[cleanup] Some comments cleanup.

CHANGE TYPE

[ ] Major
[ ] Minor
[x] Patch

IMPACT, DEPENDENCIES AND TEST CONSIDERATIONS
Some not-relevant comments cleanup

This following PR contains the same changes:
https://github.com/GodelTech/GodelTech.Microservices.Core/pull/51


Add missing imports

Adds two imports that become apparent when not compiling the original project, but adding the files directly.

I appreciate @jberkel's suggestions. Any chance you can make the changes before we merge?

With these changes, there is no needed for @import Foundation in SQLite.h then?
https://github.com/stephencelis/SQLite.swift/blob/master/SQLite/SQLite.h

don't think so, you need it for the FOUNDATION_EXPORT macro

Sure, works for me, updated.

Thanks!


UI: Same icon for two actions

There are the same icons for "propose" and "detail", which is really confusing.

Please select different icons.

Duplicate of #139. Still I cannot explain why I let such a simple, yet relatively prominent bug pass to 4th iteration. This can be fixed ASAP. Closing this one.


Include response body in exception when GET requests fail

Looks like there are some issues with sandbox institution, existing requisition started to return 409, and a new one returns 500 on account requests.


Add support for custom settings in a Plugin

Theoretically custom URLs and Team IDs can be used, but I would also like to provide custom authentication credentials in the configuration of my plugin.
Can this be done now? If not, I would love to have the ability in a future release.

Unfortunately the current implementation is at a very early stage but we are working on adding this.
I would suggest that we are using this thread to collect ideas for the implementation.

My thoughts are, add something like the following:
"settings": {
  "SETTING_MACHINE_NAME": {
    "label": "SETTING_LABEL",
    "type": "FIELD_TYPE",
    "default": "DEFAULT_VALUE_OR_BLANK",
    "options": {
      "KEY": "VALUE",
      "KEY": "VALUE"
    }
  },
}

SETTING_MACHINE_NAME: The machine name to be used in plugin integration.
SETTING_LABEL: Human readable label printed on the settings form.
FIELD_TYPE: One of a few pre-defined field types, e.g., textfield, textarea, radio, checkbox, password, etc
DEFAULT_VALUE_OR_BLANK: A default value for the field, or blank if not required.
options/KEY/VALUE: Key value pairs for radio/checkbox/select list field type.
Obviously this is just a super rough draft, but it's the concept that's important as it give enough flexibility to the integration developer.


Removed redundant error from MeshDataTool

Just a simple fix. The error doesnt need to be checked twice.

Indeed, thanks! Not sure why nobody merged it earlier (/me looks at @godotengine/pr-mergers).


Update changelog and gradle properties for upcoming release

@quentin41500

@ekchang Can you also add a little bit of information about the release in README.md ?


Update network name

And removed delayed_node reference since it was deleted on commit: 2131f3a2924dfebfa4780a92ed0668d88ce2eb3a

@TropikalMonkey could you please set the development branch as the target?

@TropikalMonkey could you please set the development branch as the target?

@tedthecreator Oh my. I'm really sorry! I've created a new PR. https://github.com/Revolution-Populi/revpop-core/pull/15
I'm not sure I can actually change the target once the PR is created. I'm actually not very familiar with github but I really wanted to help with the project in any way I could 😅

And we do really appreciate it! Thanks!


Start with a single-node cluster

Users sometimes ask if it's ok to run cockroach with a single node at the start, and then scale to a replicated cluster later. We should clarify that it's definitely ok, as long as you accept no fault tolerance, and show how to do it.
From gitter:

it's fine to run cockroachdb with a single node, as long as you understand that you don't have any fault tolerance. To start a single node, you simply exclude the --join flag from cockroach start: https://www.cockroachlabs.com/docs/stable/start-a-node.html#start-a-single-node-cluster.

Also, cockroachdb expects to replicate data 3 times by default, which obviously isn't possible when running a single-node cluster. So to prevent lots of log messages about under-replication, you can also disable replication as described here: https://www.cockroachlabs.com/docs/stable/common-errors.html#when-running-a-single-node-cluster.

We have closed this issue because it is more than 3 years
old. If this issue is still relevant, please add a comment and re-open
the issue. Thank you for your contribution to CockroachDB docs!


[PUI] Token management

Expose API token management to the API

User settings panel
Users can create and delete their own tokens

Duplicate of https://github.com/inventree/InvenTree/issues/6500?

@matmair thanks, yes this is a duplicate


Player Movement

Player Movement still continue during dialogue.
Requirements

Freeze player movement

Still enable animations

Time.deltaTime = 0f not doable in this case

Pull request open. Player Movement still continue issue solved.


[BUG] Total Effort was replaced with Effort and cell's data label remained "Total Effort" in Issues -> affected applications

Is there an existing issue for this?

[X] I have searched the existing issues

Konveyor version
latest
Priority
Undefined (Default)
Current Behavior
Affected applications - sort by Total Effort test fails because of changed column name - Total Effort was replaced with Effort and cell's data label remained "Total Effort".
Expected Behavior
Data label should be aligned with column name
How Reproducible
Always (Default)
Steps To Reproduce
No response
Environment
- OS:

Anything else?
No response

This is a follow-up to #1846


⚠ sandbox-api.piste.gouv.fr healthcheck has degraded performance

In b47e3d2, sandbox-api.piste.gouv.fr healthcheck (https://sandbox-api.piste.gouv.fr/cassation/judilibre/v1.0/healthcheck) experienced degraded performance:

HTTP code: 200
Response time: 4863 ms

Resolved: sandbox-api.piste.gouv.fr healthcheck performance has improved in c256188.


Updates to the client

Coverage remained the same at 97.761% when pulling 5b04c9d9633f124fc1f279e726dded216c0ceb72 on kijojo:develop into 21b8e2b71fa543ab3dd079f6b1a4946157ba0ca1 on werwolfby:develop.

[Current coverage][cc-pull] is 97.76%

Merging [#88][cc-pull] into [develop][cc-base-branch] will increase coverage by +<.01%

@@           develop        #88   diff @@
=========================================
  Files           33         33
  Lines         2725       2725
  Methods          0          0
  Branches         0          0
=========================================
  Hits          2664       2664
  Misses          61         61
  Partials         0          0

[][cc-pull]

Powered by Codecov. Last updated by 651c7b0
[cc-base-branch]: https://codecov.io/gh/werwolfby/monitorrent/branch/develop?src=pr
[cc-pull]: https://codecov.io/gh/werwolfby/monitorrent/pull/88?src=pr

Coverage remained the same at 97.761% when pulling 49f9c7fd5907d69816378bca6e9746f4438d7c5a on kijojo:develop into 21b8e2b71fa543ab3dd079f6b1a4946157ba0ca1 on werwolfby:develop.

Coverage remained the same at 97.761% when pulling e60c82cb4ae76724cc302bf3cf7523b24f22fe39 on kijojo:develop into 21b8e2b71fa543ab3dd079f6b1a4946157ba0ca1 on werwolfby:develop.

[Current coverage][cc-pull] is 97.76%

Merging [#88][cc-pull] into [develop][cc-base-branch] will increase coverage by +<.01%

@@           develop        #88   diff @@
=========================================
  Files           33         33
  Lines         2725       2725
  Methods          0          0
  Branches         0          0
=========================================
  Hits          2664       2664
  Misses          61         61
  Partials         0          0

[][cc-pull]

Powered by Codecov. Last updated by 3cfef5e
[cc-base-branch]: https://codecov.io/gh/werwolfby/monitorrent/branch/develop?src=pr
[cc-pull]: https://codecov.io/gh/werwolfby/monitorrent/pull/88?src=pr


Make it possible to publish and view edition in Publisher app

Publisher can now make requests to https://frontend.test.govuk.digital/bank-holidays.json, which is necessary for viewing and publishing editions.
More work is required in order to make these editions viewable via the frontend app.

Thanks! I learnt some new Terraform with this change :)


Use multistage builds for the docker images.

first stage -  build inside a golang container
second stage - just copy the binary in an alpine container.
https://docs.docker.com/develop/develop-images/multistage-build/#use-multi-stage-builds

All Go projects should use our standard promu builds, rather than something new.


Fix bracket position issue #62

Fixing issue #62
Change MiscDemo/connect/python/connector.py Line 91
    print("{}\t{}\t{}").format(siteid, citycode, pv)
to
    print("{}\t{}\t{}".format(siteid, citycode, pv))

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

I am not sure if my contributions in this pull request is my original creation.
But it is an issue that should be fixed.
Thanks

Thanks @NewJerseyStyle , can you please sign the CLA?

@DanRoscigno Thanks. I am trying to identify the complete details of the source and of any license or other restriction related to function and bracket in Python syntax. I will agree CLA once the document is ready.


[Metadata Viewer] Introduce metadata viewer

this commit adds Metadata Viewer
ONE-vscode-DCO-1.0-Signed-off-by: jihong-yu<EMAIL_ADDRESS>

@jihong-yu Could you format this files using 'eslint' and 'clang-format' ?

Screenshot

Thanks, it works very well with the example file. :-D

[Metadata Viewer] introduce metadata viewer
Could you update the commit title as'[MetadataViewer] Introduce Metadata Viewer'

From second commit, there is no need to add signed-off-by and title :)

@llFreetimell I think it's by adding a git-hook, just automatically generated... (Same thing happens to me) Could you please check the commit title when you merge this PR?

First commit's title is used :)
Maybe git commit -m "message" would work...?
Anyway, no problem to merge :)

I mean, I changed the PR title but not the commit title, so I asked you to watch out that the current title is being merged..

I use the git commit -m command but still it put the sign-off ☹

@jihong-yu Could you remove those sign-off-by lines of each commit, except for the first commit?

GIF

@smj53 PTAL :-D


net: add EDNS0 support to built-in DNS resolver

issue #6336 unveiled that we need minimum DNS transport support in build-in DNS resolver
for when we cannot rely on underlying stuff such as libc. For now, looks like EDNS0 is
the only missing piece.

+1
Ran into this while debugging a problem with pulling Docker images: we were unable to pull an image due to a DNS resolution error. It turns out our router was forwarding up to 4k DNS responses regardless of the state of the EDNS0 flag, and the Go DNS package was (correctly to the spec) choking on the larger than expected message. The ultimate fix was for the router itself, setting the window to back down to the RFC correct 512 byte size.
Again, this is not a bug with the Go implementation of DNS, but considering how broadly used EDNS0 is for avoiding a fallback to TCP for large DNS responses (which are more and more common), it might be worth implementing the EDNS0 DNS extension if only for that feature.

/cc @mikioh @mdempsky

I still hesitate to say "yup, that's the way to go" because we are already not innocent.

https://lists.dns-oarc.net/pipermail/dns-operations/2013-September/010704.html
http://www.ietf.org/proceedings/87/slides/slides-87-saag-3.pdf
http://u.cs.biu.ac.il/~herzbea/security/13-03-frag.pdf

I also understand that EDNS0 would be pretty useful under the well-configured, managed and secured environment. In addition, DNS stub resolver perhaps might avoid being affected by various attacks based on UDP over IP fragmentation. I mean, with DNSSEC it sounds pretty reasonable, but in other cases...  still thinking.

Related to #13279


[mkldnn-v1.0] Add MKL-DNN LRN

Description
(Brief description on what this PR is about)
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here
add mkldnn lrn

@TaoLv @ciyongch @ZhennanQin @PatricZhao

All operator UTs are kept. Subgrah UTs will be re-enabled after subgraph feature is finished.


Move limitmode to enum

Why has this seemingly changed the HTTP method that api.php requests are made with? Shouldn't they be GET anyway?? What is going on!

I love this.
I will be adding a few more Limitmodes - ASTRAL and MOLE, both of which limit which zones you can adventure in.  Maybe BIRD, since that replaces your combat skills with bird skills. MOLE and BIRD are mutually exclusive. The question is whether ASTRAL and BIRD are.
In any case, this is really good work.

That sounds good! Though from this code it seems that limit modes are defined kol-side and indicated via the API. We'll need to keep that in mind

Well, poo. I may need to rethink that, then. I need the equivalent, for limiting where you can adventure when you are Half-Astral or Shape of Mole.

Well, poo. I may need to rethink that, then. I need the equivalent, for limiting where you can adventure when you are Half-Astral or Shape of Mole.

I think there's a way we can do it, by looking at certain prefs/effects when we get a "0" back from that API key...


Releases

What's a "release" in npmgraph?

Having a semver is semi-useful for tracking regressions in BugSnag.  Also for communicating changes to the URL API.  And maybe at some point for keeping the web app and CLI in sync.
All of that said, I haven't been particularly rigorous or consistent in how/when I bump the semver.  🤷


[Feature Request] Bionic Reading

Checklist

[X] This issue contains only one feature request. I will open one issue for every feature I want to request.
[X] I checked, but didn't find any duplicates (open OR closed) of this issue in the repo.
[ ] This feature is merely an UI/UX update.
[X] This feature is not going to conflict with many of the existing features/options.

Is your feature request related to a problem? Please describe.
Some ppl can't focus while reading, this feature helps some to focus.
What is Bionic Reading?
Describe the solution you'd like
A Setting to Enable/Disable Bionic Reading
How will you/everyone benefit from this feature?
Bionic Reading is a nice to have feature.
Millions of people have also been reacting to bionic reading on social media, and many people with dyslexia and ADHD have said bionic reading helps them focus.
Additional context
No response

Intersting! will take a look and see how to implement this, thanks.


ask git to store version information in tarballs

The added lines make git(hub) add a file named .git_archival.txt into tarballs it enerated for tags that have the following format:
$ cat .git_archival.txt
node: 3cd27f3
node-date: 2023-10-04T12:53:16-07:00
describe-name: v0.6.1
ref-names: HEAD -> main, tag: v0.6.1

(the above is for v0.6.1 tag).
This allows setuptools-scm to deduce the version number, allowing us to build from exported tarballs (which is how conda packages fetch their source).
More information:
https://setuptools-scm.readthedocs.io/en/latest/usage/#git-archives

Change Description

[ ] My PR includes a link to the issue that I am addressing

Solution Description

Code Quality

[ ] I have read the Contribution Guide
[ ] My code follows the code style of this project
[ ] My code builds (or compiles) cleanly without any errors or warnings
[ ] My code contains relevant comments and necessary documentation

Project-Specific Pull Request Checklists

Bug Fix Checklist

[ ] My fix includes a new test that breaks as a result of the bug (if possible)
[ ] My change includes a breaking change

[ ] My change includes backwards compatibility and deprecation warnings (if possible)

New Feature Checklist

[ ] I have added or updated the docstrings associated with my feature using the NumPy docstring format
[ ] I have updated the tutorial to highlight my new feature (if appropriate)
[ ] I have added unit/End-to-End (E2E) test cases to cover my new feature
[ ] My change includes a breaking change

[ ] My change includes backwards compatibility and deprecation warnings (if possible)

Documentation Change Checklist

[ ] Any updated docstrings use the NumPy docstring format

Build/CI Change Checklist

[ ] If required or optional dependencies have changed (including version numbers), I have updated the README to reflect this
[ ] If this is a new CI setup, I have added the associated badge to the README

Other Change Checklist

[ ] Any new or updated docstrings use the NumPy docstring format.
[ ] I have updated the tutorial to highlight my new feature (if appropriate)
[ ] I have added unit/End-to-End (E2E) test cases to cover any changes
[ ] My change includes a breaking change

[ ] My change includes backwards compatibility and deprecation warnings (if possible)

Comparable changes were made in https://github.com/astronomy-commons/hipscat-import/pull/198


Supporting launching MAPDL from WSL

And ofc, connecting to it.
However some guidance it is needed in order to allow WSL to connect to the MAPDL process running at the host.
Close #2315

@PipKat quick check on the grammar and such?


add config options to allow global coord clamping

allows one to set a minimum and maximum x and y to act as boundaries for the camera projection
works for both dragging and zooming

Hello! I've been getting into bevy recently, and I found your camera super helpful. Thank you for open sourcing it! I did find, however, that I wanted to be able to restrict the camera's view window so that I didn't drag or zoom the window beyond the borders of a specific grid. I did some local tinkering and I got it working pretty much as I envisioned. If the change looks good to you, I thought it'd be a nice first contribution to the bevy community to merge this support upstream :)
Video of the implemented behavior:
https://user-images.githubusercontent.com/8944532/189548695-7bd62198-eea7-4fa0-a576-6d1f01b02a2b.mp4

Hey, and thanks for the contribution! It's a difficult task with lots of literal edge cases :)
Overall, it looks like the implementation makes sense, but like you say, it's kind of hard to wrap your head around the equation as it stands, and it adds quite a lot of complexity to otherwise straightforward code. If possible I'd like us to try and make it more readable.

@johanhelsing Apologies, wasn't able to dedicate enough heads-down time dig into the zooming math until the weekend. I'm still not sure I can explain how it works exhaustively, but I've tried to clean up the math to make it more readable (your suggestions were incredibly helpful in doing so!).
I moved the max allowable zoom scale logic to two new functions, max_scale_within_x_bounds and max_scale_within_y_bounds, mostly so that I could add commentary explaining what was going on without it bloating the original function. I'm happy to remove the long function comment if it feels redundant, but I found it personally helpful to write out a number of examples of how the projection, window, and boundary dimensions would interact with one another. Kind of an "if you turn these knobs, this is what will happen" set.
I renamed all the global/pixel stuff to remove the misleading elements. I think I made a lot of bad assumptions when first trying to understand how cameras work in bevy. I'm definitely not to full understanding yet, but I think I'm starting to grok the basic components a bit better.
I also moved the example to the examples folder, I hadn't even known you could execute examples via cargo run --example, cargo is pretty cool! I appreciate your patience and helpful suggestions, I feel like I'm learning a lot about how to write Rust and Bevy 😄 If you wouldn't mind taking a look at the updated code, I'm of course happy to make any others changes that jump out.

Thank you for the PR @johanhelsing , I think that really provided the remaining refinement needed on the scaling functions! I just put up another commit to add some unit tests for the scale functions and hopefully make the clamped_border comments more useful. I think I'm starting to feel pretty good about merging this (assuming it passes CI), but as always am happy to address anything you think might be remaining

Sorry for being slow to get back, it can be hard to find time for side projects when balancing work and home life. But wow, I feel like you cracked the final piece of this with the simplification of the math. Nicely done! I merged your PR to my fork, and am going to see if I can get some unit tests in place. I'm still wrapping my head a bit around Rust unit testing, but I think it's a good opportunity to learn it while also making sure some of my assumptions are correct

No worries. I've also been juggling a lot lately.
I'm happy it turned out to be really simple and readable in the end :) Thanks again!


error messages if not using default provider

Describe the bug

I started creating a Docker container in which anime DL would run, everything works fine except for when I try to change the provider.
To reproduce

root@9db640099248:/home/Anime# anime dl "golden boy" -p animestar

LOG
anime --log-level DEBUG "golden boy" -p animestar

Traceback (most recent call last):
  File "/usr/local/bin/anime", line 11, in <module>
    load_entry_point('anime-downloader==5.0.14', 'console_scripts', 'anime')()
  File "/usr/local/lib/python3.8/dist-packages/anime_downloader-5.0.14-py3.8.egg/anime_downloader/cli.py", line 81, in main
    cli()
  File "/usr/local/lib/python3.8/dist-packages/click-8.0.1-py3.8.egg/click/core.py", line 1137, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/click-8.0.1-py3.8.egg/click/core.py", line 1062, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python3.8/dist-packages/click-8.0.1-py3.8.egg/click/core.py", line 1662, in invoke
    cmd_name, cmd, args = self.resolve_command(ctx, args)
  File "/usr/local/lib/python3.8/dist-packages/click-8.0.1-py3.8.egg/click/core.py", line 1709, in resolve_command
    cmd = self.get_command(ctx, cmd_name)
  File "/usr/local/lib/python3.8/dist-packages/anime_downloader-5.0.14-py3.8.egg/anime_downloader/cli.py", line 51, in get_command
    command = importlib.import_module(
  File "/usr/lib/python3.8/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1014, in _gcd_import
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 973, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'anime_downloader.commands.golden boy'

Dockerfile

FROM ubuntu
RUN apt-get update
RUN apt-get upgrade -y
RUN apt-get install -y curl
RUN apt-get install -y wget
RUN apt-get install -y git
RUN apt-get install -y python3
RUN apt-get install -y python3-pip python3-dev \
  && cd /usr/local/bin \
  && ln -s /usr/bin/python3 python \
  && pip3 --no-cache-dir install --upgrade pip \
  && rm -rf /var/lib/apt/lists/*
RUN cd home \
  && git clone https://github.com/anime-dl/anime-downloader.git \
  && cd anime-downloader \
  && python3 setup.py install \
  && mkdir /home/Anime

uhhh
the command in the code-block is correct
but the command in the log is not

anime --log-level DEBUG "golden boy" -p animestar
it should be anime --log-level DEBUG dl "golden boy" -p animestar


Project is not not set up to allow translations

This was a mistake, going back and replacing all the strings will be a pain so I shouldn't put this off for too long.

These look like promising libraries to use:

Quasar has some specific recommendations too. https://quasar.dev/options/app-internationalization
I'm not really an expert on this, it's possible we may want to stray from what Quasar does here but staying within the ecosystem sounds nice because things are more likely to work seamlessly when it comes time to build Android/iOS apps.

Yes. Happy to follow their choice of package on this. The main thing for me is to not reinvent i18n by hand, when all the corners have been smoothed out by a library. I will probably take a few weeks, maybe 4? Because this is a hobby so time on it will be limited on it. If that is too long then feel free to give this to someone else that is OK with me.

A small proof of concept (not a PR) - https://github.com/mcapodici/headway/commit/5dbec01dcf1f9b78ae024c485f6c3259240b9b74

Are you able to give some hints about places I might find strings that need translating?
The obvious places are in .vue and .ts files. I assume that console.log and console.error do not need translating?
Is there anything in /services that needs dealing with, or just the frontend?

Anything going to the browser console can just be English I think, and the services/ code shouldn't be included in this, though at some point we'll want to think about passing the user's locale to Valhalla somehow to get directions in the correct language.
The one thing I want to mention is that I'm doing a lot of frontend refactoring right now so try to keep the initial PR as small as you can, then we can replace hardcoded strings with translatable ones file-by-file after that. I'm redoing the entire directions screen to surface alternate routes and show directions in a more intuitive way, and I don't want too many merge conflicts :)

This should be unblocked now that #138 is merged.

Might be closable now?

I think so too. Thank you for all the work! 🎉


Add RepositoryExplorer function for presentation-server

Fixed by #3120 - sorry, forgot to include a cross-ref.


Arnold: "opaque" flag on mesh nodes is not disabled for non-opqaue materials

Observed in Maya 2019 (Arnold 5.2):

Create RPK with rule:Init --> i("/ESRI.lib/assets/Webstyles/Vegetation/Realistic/AbiesBalsamea.glb")

Assign the RPK to a quad in a fresh maya scene
Add an Arnold light
(x) note how the leaf cards are not correctly cutout (no sprite rendering)
Workaorund: manually disable the "Opaque" checkbox at pPlaneShape1 -> Arnold

This is automatically fixed in Maya2020 (Arnold 6) as per a behavior change in the Maya Arnold integration: https://docs.arnoldrenderer.com/display/A5AFMUG/Opaque


License and Attributions for datasets are not saved

This code does not save the attributes
dataset.license = "whatever"
dataset.attributions = "whatever"

Thanks for reporting, I'll take a look! 👍

Fixed in 1.1.0 branch

Vamos, thanks!


Added Convex Hull Monotone Chain Algorithm in CPP and updated Readme

Added Convex Hull Monotone Chain Algorithm in CPP and updated Readme

Thanks for the PR :) Merged.


Add missing thread safety annotation

@veblush this is extracted from #26846.

Subsumed by #26866.


Cargo's tests fail on osx when using rustup.rs [migrated from rust-lang/cargo]

@alexcrichton mentioned he's seen this issue but hadn't figured out a way to reproduce it reliably, sooo I guess I found one!
My original report:
rustup --version is rustup 0.6.3 (a0e2132 2016-08-28)
I just tried to run cargo's test suite and I get 7 failures, some of which contain the error message "error: no default toolchain configured", which is pointing at rustup.rs as the culprit.
Further evidence is that when I remove ~/.cargo/bin from my PATH, then run:
$ ./configure --disable-cross-tests --disable-optimize
$ make
$ make test

All of cargo's tests pass. This uses my checkout of rustc installed to /usr/local/bin/rustc, which I just rebuilt today, so it's rustc 1.13.0-dev (1fca1ab0e 2016-09-10).
When I put ~/.cargo/bin in my path before /usr/local/bin so that I'm using rustup.rs, which rustc is ~/.cargo/bin/rustc, and my rustup default is set to nightly, which I updated yesterday, rustc --version returns rustc 1.13.0-nightly (f1f40f850 2016-09-09). rustc -vV runs without error.
With this setup, if I run the same 3 commands above, I get 7 test failures. Full output in this gist; note the error messages I'm seeing in some of the tests that say "error: no default toolchain configured".
I looked around in rustup's issue tracker a bit for potential situations or causes of this problem, and the only thing I found that looked slightly promising was the aside that "rustc -vV should not be failing with that error even if no default toolchain is configured" in an issue's comments.
@alexcrichton's response:

Ah yes I've seen this classic error before, and I believe @brson has run into it. I haven't been able to reproduce it yet, but I think it's now been unlocked! The key difference here is that you're using ./configure && make test vs cargo test (which I use locally).
Cargo tests munge the value of $HOME for subprocesses to prevent polluting or reading the home directory configuration. If you run ./configure && make then this means we're running the rustup rustc binary with $HOME pointing to an arbitrary new location, which greatly confuses rustup. If you run cargo test, however, I believe that the environment variables rustup sets (I think PATH?) end up getting the tests working.
I suppose my preferred solution here would be to update rustup to work if $HOME isn't set (if that's possible) and we can perhaps investigate other alternatives if that doesn't work.

I believe the symptom here is:
$ HOME=/tmp rustc -vV
error: no default toolchain configured

That's essentially what happens with the Cargo tests. They correctly run the rustc binary that is actually rustup, and then HOME is configured to point somewhere else.

What's happening here is that Cargo's makefiles are using its own bootstrap cargo which then in-turn runs test cases that frob HOME weirdly, then invoke the local (rustup) rustc. Normally it isn't a problem for Cargo to screw with HOME because (rustup) cargo will set rustup-specific environment variables to communicate the proper config to (rustup) rustc. In this case though we're mixing rustup tools with non-rustup tools and things go bad. That's why this only happens when running make check and not cargo test.
Since this is a really weird case that cargo's makefiles are inflicting on itself I'm inclined to fix it there. The way to do this is to set RUSTUP_HOME to the right value. Unfortunately there's no definitive way to get that value, but it's pretty likely that if HOME\.multirust (one day HOME\.rustup) exists and RUSTUP_HOME is not set, then that is the right value.

Is this still an issue in modern Cargo?

It does not appear to be, now that Cargo uses cargo test rather than make. Running HOME=/tmp rustc -vV still produces the error message, but that's irrelevant if nothing does that. So I'm closing!
Thank you for pinging this; I totally forgot about it!


Include License File from Original Project

First off, nice work on adding the new routing code into the project. I just started converting my original Angular 2 JumpStart project (that this ng2-minimal project is based on - https://github.com/DanWahlin/Angular2-JumpStart) to use the latest router today and stumbled across this project. I would appreciate if you could do the following:

Please add back the license file provided in the original project given that the ng2-minimal project is based on my project which had a license file.
I'd definitely appreciate and welcome pull requests back to my original project.

Thanks!
Dan

absolutely... FYI I enjoy your videos...
doing now...

in fact, if you want, I can just close down the project and do a re-direct to yours if you have the router now... let me know will this be better?

If you're able to contribute to the original project I'd definitely welcome it. I just started doing the conversion this morning and found your version which is great (haven't had time to study the new code yet but it gives me a "jumpstart"). :-) I'd love to have 1 "source of truth" for the project if possible.

added lic and credits and sure, let me know when you are done and will do a redirect
FYI I recommend you checkout my other 2 projects:
Angular 2 Kitchen sink: http://ng2.javascriptninja.io
and source@ https://github.com/born2net/ng2Boilerplate
as well as:
https://github.com/born2net/studioDashboard
Regards,
Sean

Cool - thanks Sean! I'll try to get that going today and let you know once I'm done. I'll give you credit in the check-in stuff I do. :-)

Hey Sean. I just realized that this is actually based on the Heroes project rather than my full Angular 2 JumpStart (just has some pieces from the jumpstart project). I should've looked closer - sorry to even bug you about it.

ya it's a mix of the 2, no worries...


AMBARI-22755. Show OneFS JMX metrics on the UI (amagyar)

What changes were proposed in this pull request?
JMX metrics of OneFS should be displayed on service summary page. This page comes from ambari-web and cannot be controlled from a management pack therefore the ambari-wab was modifed to support OneFS summary page. This is solution will be available only until we have a better support for controlling the summary page from a management pack.
How was this patch tested?
Visually inspected service summary page under OneFS.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/1038/
Test FAILed.
Test FAILured.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/1044/
Test FAILed.
Test FAILured.

retest this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/1050/
Test FAILed.
Test FAILured.

retest this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/1103/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/1107/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/1123/
Test FAILed.
Test FAILured.


Syntax highlight is broken while adding jsconfig.json & the same package.json under different folders

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

VS Code Version:

Version: 1.74.0 (Universal)
Commit: 5235c6bb189b60b01b1f49062f4ffa42384f8c91
Date: 2022-12-05T16:43:37.594Z
Electron: 19.1.8
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 20.6.0

版本: 1.74.1
提交: 1ad8d514439d5077d2b0b7ee64d2ce82a9308e5a
日期: 2022-12-14T10:35:03.979Z
電子: 19.1.8
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
作業系統: Linux x64 5.4.0-135-generic
Sandboxed: No

OS Version: MacOS BigSur 11.7, Linux Mint 20.1

Steps to Reproduce:

Clone the reproduce repository and install volar
Run Volar (Debug): Write Virtual Files command
Disable Volar Extension in the workspace and reload vscode (or just open by code --disable-extensions)
Inspect src/Hello/World/World.vue.js & src/Bye/World/World.vue.js

There is a simple video to describe the issue:
https://raw.githubusercontent.com/FlandreDaisuki/reproduce-volar-syntax-highlight-bug/master/demo.mp4

@mjbvz Here is a minimal repro: https://github.com/yume-chan/vscode-repro-169486

The import is from '@/a/Foo' but the error mentions '"@/b/Foo"'.

a/foo/foo.js is showing inlay hints, errors and code suggestions from b/foo/foo.js.


Sprites cache store

Firstly got Protocol error here http://stackoverflow.com/questions/18688887/compass-protocol-error-with-import
and founded that there were some errors on windows machines with sass, so sass filesystem cache store has function [1] that fixes cache names that contain "*"
[1] ->
def path_to(key)
        key = key.gsub(/[<>:\\|?*%]/) {|c| "%%%03d" % Sass::Util.ord(c)}
        File.join(cache_location, key)
end

but in compass-rails sass_importer you pass sprockets cache store [2], that do not know about this problem :)
[2] ->
     # Use custom importer that knows about Sprockets Caching
      cache_store = SassCacheStore.new(context.environment)

After commenting this code, compass made sprite normally. So what is this cache for? and is there should be issue for sprockets too?
Temporary solution is to change cache dir for compass, sass and assets.
But at the moment I have not succeeded to set config.cache_store and config.assets.cache_store to any value :( rails don't want to change them.

Thanks a lot for the help guys! @wldcordeiro and @balauru, what I did was setup the assets on an external tmp dir.
   mkdir /mnt/tmp
   chown railsuser:railsuser /mnt/tmp

and then on application.rb
    # tmp dir for cache must be setup outside virtualbox shared folder for sass cached sprites to work
    config.assets.configure do |env|
      env.cache = ActiveSupport::Cache::FileStore.new('/mnt/tmp')
    end

This was quite a cryptic issue to deal with, since it seems like a VirtualBox issue, not rails related. I hope this helps people with the same issue.


Investigate intern (possible Jasmine replacement)

I know we've done a lot of customization of Jasmine in Cesium, but I'm not sure how tied we are to it.  I stumpled upon intern and it looks pretty promising (particularly because it natively supports AMD, coverage, and promises.)  Has anyone looked at this before?  It's also has CI integration and node support, which could help us with automating branches.
CC @kristiancalhoun @pjcozzi @shunter

I did not evaluate it, but tape is another one to consider.

I haven't used tape, but I know some folks who have and like it a lot.  It does require you to declare how many assertions your test is going to do at the top of your test, with a line like: t.plan(2).  It looks like you can alternatively skip the t.plan and call t.end when you're done, but effectively, that's treating every test like it's async when only maybe 10% of our tests are.

Also just came across mocha but did not evaluate it.

I'm going to close this since we have no plans to move to a different framework any time soon.  The landscape is constantly changing so if we do decide to replace it, it's unlikely any information in this issue will be of much help.  Jasmine is still really popular and personally I think it's a solid choice (though our use of it could possibly be improved).


Rename buton-base (etc) to input-button

Now that the code is stabilizing, it's becoming clear that some names just aren't ideal.
In this example, we've been using button-base and button-link for some of the general button-esc code. To parallel some of the new input controls, button-base should actually be named input-button.
I'm undecided what should happen to button-link at this time, but in general any component that is general purpose, and used for receiving inputs, should be prefixed with input-.

This is obsoleted by #1377