Consistent AssetPath Format

What problem does this solve or what need does it fill?
Currently AssetPath is backed by Path, which is great if the goal is to support arbitrary filesystem paths across operating systems:

Windows: \foo\bar, C:\foo\bar
Everything Else: /foo/bar, /mnt/c/foo/bar

However that is not the goal of AssetPath. AssetPath exists to be an opinionated "cross platform virtual asset filesystem with support for multiple arbitrary asset sources and asset labels, which serves as a unique canonical asset identifier".  Ideally, you can compare two "absolute AssetPaths", and if they are not equal, they are not the same asset.
This mismatch in goals can create problems. For example, when we create AssetPaths from hot-reload events on Windows, they come in the form \foo\bar.png. However users tend to use /foo/bar.png (and we encourage this pattern everywhere), which can create problems when considering paths as strings (like it did here #10500). As a cross-platform canonical path format, AssetPath has no business supporting windows-only drive letter syntax, as that location is not directly expressible on other operating systems. AssetPath already has a cross platform way to express other mounted filesystem locations (some_asset_source://).
What solution would you like?
We should build a new, consistent path abstraction for AssetPath and replace the current Path usage with that. It should support only / and be designed with the explicit goal of providing consistent canonical identifiers.
Like the current AssetPath impl, it should avoid allocating when a &static str is used to construct it, and it should have true copy on write behavior (#9729).
This will create a new problem: users that want to support absolute paths (ex: C:\path\to\image.png) can no longer directly use the default AssetSource (which is rooted at the local asset folder)). We need a way to support this, in the interest of building apps like "file editors" or loading scenes from arbitrary locations on disk.
I believe this should be accomplished via a combination of two approachs:

App developers can already choose what the "default" asset source is. They can mount the root of the C drive on windows (and / on every other platform), enabling absolute paths to work by default (without the windows drive syntax).
For windows "drive syntax", App developers can mount those drives as asset sources with equivalent names (ex: The AssetPath A://some/path.png would map to the windows path A:\some\path.png). We could then build AssetPath::from(Path) in a way that explicitly handles windows drive syntax and converts it to the equivalent AssetPath. We could also consider building "automount" functionality that will mount windows drives when they are requested.

One issue with (2) is that Bevy Asset V2 does not currently support mounting new asset sources at runtime. We'd almost certainly want to support that, as new drives can be connected at arbitrary times when an app is running.

Windows supports both paths separators since Windows 7 IIRC. For instance, even on MS-DOS, I can navigate fine using forward slash:

So even things like cd / or cd C:/Users/ also works fine.
The problem is the returned path from Win32 API, which due to naming conventions, uses backslash \.
There are exceptions to this, mostly UNC, like:

Volume names paths: \\?\Volume{26a21bda-a627-11d7-9931-806e6f6e6963}\
Remote paths: \\my-network-server\something
WSL paths: \\wsl.localhost\Ubuntu\home\my-user
Serial ports: \\.\COM56
Paths longer than MAX_PATH: \\?\C:\A\Really\Long\Path\With\Many\Sub\Dir\Subdirectories\And\BIGGER_FILE_NAME.AND_EVEN_BIGGER_EXTENSION

All of these can be solved on user side, by having a mounting letter on Windows. Let's say, for some reason, I want an asset from inside my WSL, which is located at /home/afonsolage/assets/, I could just do in MS-DOS: net use w: \\wsl.localhost\Ubuntu and from Bevy asset, use w:/home/afonsolage/assets/.
With all that being said, we probably still have to do something if we want to support paths returned from Win32 API, like when reading all files from a folder, WinAPI will use \ on returned paths, so we must take care of them. Also on future bevy_editor, this will have to be taken care. I saw too many times mixed paths like C:\Users\afonsolage/some_folder/path.
So as long Bevy new AssetPath take care of always converting \ to / and users don't use UNC directly, but through mount letter, we shouldn't require Bevy to support both forward and backward slash.

Internally, Path is an OSString - would the new non-Path-based AssetPath still keep this instead of regular String? I assume so.
The approach I was contemplating was to replace both Path and PathBuf with Bevy-specific types, although you might not need Path, instead AssetPath can just use an OSString directly. The resolve() method will need something akin to PathBuf, but it can be much, much simpler (resolve already does most of the work, we really just need push and pop).
Here's how I would proceed on this:

Identify what the API should look like. We want to change the API as little as possible, but some things will likely have to change.
Clearly delineate the places where we convert filesystem paths to AssetPaths and vice versa. Make sure we have comprehensive unit tests for those cases.
If we need an AssetPathBuf for constructing paths from components, implement that and write tests for it.
Replace Path with OSString in AssetPath. Re-write methods such as .parent() which need to scan the path for separators. Methods which need to return a Path will have to add conversion logic.

@cart It's still not clear to me whether the internal representation of a path should be str or OsStr. There are arguments in favor of both:

Most of the time, AssetPath is constructed from a str, and parse_internal operates on a str buffer. So this would argue in favor of it being str.
However, at some point the path is actually going to be converted into a filesystem path, which argues in favor of using OsStr.

Implementation-wise, str is probably slightly easier, but only slightly. There are basically 14 places in the code that need to be changed, all in one file.


[bug] strange characters exist when vim ':terminal'

environment

Have you reproduced with a minimal vimrc: yes
if you are using terminal:
terminal: kitty, gnome-terminal (i only installed the two terminals)
$TERM variable: xterm-kitty, xterm-256color
color configuration (:set t_Co?): no
if you are using Neovim:
does it happen in Vim: not test

Reproduce Procedures
vim-airline is latest.
" $ uname -r
" 5.13.8-arch1-1
" $ has nvim
" ✓ nvim 0.5.0
" $ has python
" ✓ python 3.9.6
" $ vi test.vim
set runtimepath=$VIMRUNTIME,~/.local/share/nvim/repos/github.com/vim-airline/vim-airline
let g:airline_mode_map = {
			\ 'n'    : '█',
			\ }
" $ vi -u test.vim

open vim:

press :terminal<CR>

then :tabnew<CR>

press gt

strange characters exist in airline.
in fact, I want to my airline mode is my cursor shape, like this:
let g:airline_mode_map = {
			\ 'i'    : '▏',
			\ 'n'    : '█',
			\ 'R'    : '▁',
			\ }

so i use this special character not 'N'.
thanks.

this is most likely a problem of your terminal, that cannot display some of the characters.

I use gnome-terminal and kitty. The weird <e2> always appear.

can you try with xterm? How about gvim?

uxterm and gvim are same.

this only happens with :term, right? Do you have anything unusual in your shell settings? Fancy prompts or something? Please try with a minimum airline configuration (no fancy characters there) and a minimum shell configuration (no fancy characters there).

" $ uname -r
" 5.13.8-arch1-1
" $ has nvim
" ✓ nvim 0.5.0
" $ has python
" ✓ python 3.9.6
" $ vi test.vim
set runtimepath=$VIMRUNTIME,~/.local/share/nvim/repos/github.com/vim-airline/vim-airline
let g:airline_mode_map = {
      \ 'n'    : '█',
      \ }
" $ vi -u test.vim

The figure of this bug on kitty is in https://github.com/vim-airline/vim-airline/issues/2421#issue-970802837

This only happens with :term. I have deleted zshrc and zprofile. the vimrc only source airline and change g:airline_mode_map

if the terminal is in the top window, can you pelase show the output of :echo airline#statusline(1)  where the 1 is the window number counting from the top, starting at 1

:echo airline#statusline(1)
%{airline#check_mode(1)}%#airline_a# %#airline_a_bold#%{airline#util#wrap("!",0)}%#airline_a#%{airline#util#append(airline#extensions#term#termmode(),0)} %#airline_a_to_airline_b#%#airline_b#%#airline_b_to_airline_term#%#airline_term# /usr/bin/zsh%=%#airline_term_to_airline_y#%#airline_y#%#airline_y_to_airline_z#%#airline_z# %#airline_z_bold#%{g:airline_symbols.linenr}%l%#airline_z#%#airline_z_bold#/%L%{g:airline_symbols.maxlinenr}%#airline_z#

:echo airline#statusline(1)
%{airline#check_mode(1)}%#airline_a# %#airline_a_bold#%{airline#util#wrap("TERMINAL",0)}%#airline_a#%{airline#util#append(airline#extensions#term#termmode(),0)} %#airline_a_to_airline_b#%#airline_b#%#airline_b_to_airline_term#%#airline_term# /usr/bin/zsh%=%#airline_term_to_airline_y#%#airline_y#%#airline_y_to_airline_z#%#airline_z# %#airline_z_bold#%{g:airline_symbols.linenr}%l%#airline_z#%#airline_z_bold#/%L%{g:airline_symbols.maxlinenr}%#airline_z#

as far as I can see, that looks fine, no weird characters there. Don't know why this would happen then.

do you have anything in your terminal configuration that could cause this? Thinking about fancy prompts etc?

Fancy prompts? I think no, because when I don't use any shell rc or tmux rc and just use xterm, the bug still exist. Just open nvim, :term, then press alt-tab to switch another window then switch back to nvim, the strange character will exists in airline.
The bug still exists nowadays. The test.vim is same as https://github.com/vim-airline/vim-airline/issues/2421#issuecomment-988882336

ah so this may be the FocusGained autocommand? Did you try:
  let g:airline_focuslost_inactive = 0

Or even :set eventignore=FocusGained,FocusLost?

I found if add set eventignore=FocusGained,FocusLost can avoid this bug.
let g:airline_focuslost_inactive = 0 cannot.
The step to reproduce:

vi -u test.vim
:term
press i<C-\><C-N> to enter normal mode for vim terminal.
press <alt-tab><alt-tab> to switch back to vim.
I have tried again, and the following is the minimal test.vim can avoid this bug:
" $ uname -r
" 5.13.8-arch1-1
" $ has nvim
" ✓ nvim 0.5.0
" $ has python
" ✓ python 3.9.6
" $ vi test.vim
set runtimepath=$VIMRUNTIME,~/.local/share/nvim/repos/github.com/vim-airline/vim-airline
let g:airline_mode_map = {
      \ 'n'    : '█',
      \ }
set eventignore=FocusGained
" $ vi -u test.vim

But if change FocusGained to FocusLost, the bug will occur.

If comment the following line can avoid the bug:
      " if exists('*timer_start') && exists('*funcref')
      "   " do not trigger FocusGained on startup, it might erase the intro screen (see #1817)
      "   " needs funcref() (needs 7.4.2137) and timers (7.4.1578)
      "   let Handler=funcref('<sid>FocusGainedHandler')
      "   let s:timer=timer_start(5000, Handler)
      " else
      "   autocmd FocusGained * call <sid>on_focus_gained()
      " endif

And I notice
      autocmd VimResized * call <sid>on_focus_gained()

And try resize the terminal emulator, and the bug will occur!

If comment this line can avoid this bug:
function! s:on_focus_gained()
  if airline#util#try_focusgained()
    " unlet! w:airline_lastmode | :call <sid>airline_refresh(1)
  endif
endfunction


[BUG] Install Prometheus stack using argocd. dashboard Resource unavailable

Rancher Server Setup

Rancher version: 2.7.8
Installation option (Docker install/Helm Chart):

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): RKE2

Proxy/Cert Details:

Information about the Cluster

Kubernetes version: 1.25.13 +rke2r1
Cluster Type (Local/Downstream):

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider): Custom

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom)

If custom, define the set of permissions: Owner/Cluster

Describe the bug
When installing Prometheus using Argoсd, the dashboard does not work. Resource unavailable.
To Reproduce
make a configuration to install Prometheus with the following parameters
kind: Application
metadata:
  name: monitoring
  finalizers:
    - resources-finalizer.argocd.argoproj.io
spec:
  destination:
    namespace: cattle-monitoring-system
    server: https://kubernetes.default.svc
  source:
    path: charts/rancher-monitoring/102.0.1+up40.1.2
    repoURL: 'https://git.rancher.io/charts'
    targetRevision: release-v2.7
    helm:
      releaseName: monitoring
      values: |-
        global:
          cattle:
            clusterId: <cluster-id>
            clusterName: production-cluster
        prometheus:
          prometheusSpec:
            enableAdminAPI: true
            evaluationInterval: 1m
            externalLabels:
              provider: production-cluster
            resources:
              limits:
                cpu: 5000m
                memory: 28000Mi
              requests:
                cpu: 5000m
                memory: 28000Mi
            retention: 14d
            retentionSize: 50GiB
            scrapeInterval: 1m
            storageSpec:
              volumeClaimTemplate:
                spec:
                  accessModes:
                  - ReadWriteOnce
                  resources:
                    requests:
                      storage: 70Gi
                  storageClassName: longhorn
        rke2ControllerManager:
          enabled: true
        rke2Etcd:
          enabled: true
        rke2Proxy:
          enabled: true
        rke2Scheduler:
          enabled: true
  sources: []
  project: infrastructure
  syncPolicy:
    syncOptions:
      - ServerSideApply=true
      - PruneLast=true
  ignoreDifferences:
  - group: monitoring.coreos.com
    kind: ServiceMonitor
    jqPathExpressions:
    - .spec.endpoints[]?.metricRelabelings[]?.action
    - .spec.endpoints[]?.relabelings[]?.action

Result
The stack is installed, but the dashboard is not available
Expected Result
The stack is installed, but the dashboard is working
Screenshots

Additional context
This problem occurs due to the lack of a helm release. If you add the helm release manually, then everything works.
https://rancher-url/k8s/clusters/clusterid/v1/catalog.cattle.io.apps/cattle-monitoring-system/monitoring Error 404

I hit the same issue.
To make the dashboards works in Rancher UI I had to apply the following manifests in addition to the ArgoCD app:
apiVersion: catalog.cattle.io/v1
kind: App
metadata:
  name: rancher-monitoring-crd
  namespace: cattle-monitoring-system
spec: {}

---
apiVersion: catalog.cattle.io/v1
kind: App
metadata:
  name: rancher-monitoring
  namespace: cattle-monitoring-system
spec: {}

It would be useful, if Rancher would discover the integrations installed via other GitOps tools.
We mainly use ArgoCD to maintain all applications installed on the clusters and it's not convinient to manually click out the Monitoring stack from Rancher.
I couldn't find any other way to install it except the UI. Adding the dummy App manifests works, but looks like a hack.


PLT-8210: Only refresh changes from other tabs

Ticket Link
PLT-8210
Checklist

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit and component tests passed

This must solve the problem based on this paragraph of the localforage-observable documentation https://github.com/localForage/localForage-observable#cross-tab-change-detection


Getting different width in chrome vs Firefox

Current State
On very first click in chrome width of pop layout is small as compared to width of content in Firefox. In chrome if second time same pop-up is triggered (colorbox) then the width/height are correct and width/height is same as in Firefox
After deep analysis it is found that this part of function is inconsistent and gives different results.
$loaded.hide().appendTo($loadingBay.show())// content has to be appended to the DOM for accurate size calculations..css({width: getWidth(), overflow: settings.get('scrolling') ? 'auto' : 'hidden'})
		.css({height: getHeight()})// sets the height independently from the width in case the new width influences the value of height.
		.prependTo($content);

		$loadingBay.hide()

Here getWidth function is giving different results in chrome as compared to firefox
Chrome Browser Info:
Google Chrome	69.0.3497.100 (Official Build) (64-bit)
Revision	8920e690dd011895672947112477d10d5c8afb09-refs/branch-heads/3497@{#948}
OS	Linux
JavaScript	V8 6.9.427.23

Jquery version: 3.3.1
Colorbox version: 1.6.4

After further analysis, it was found that as while $loaded was hidden getWidth & getHeight method was being called, which somehow did not calculated correct width and height in chrome.
So the solution of this problem is append content first and keep showing '$loaded' till width and height is not calculated. Afterwards $loaded.hide(); can be called.
So here is solution for this problem:
 $loaded.appendTo($loadingBay.show()).prependTo($content);
        $loaded.show(); // keep showing so that after rendering width/height can be calculated correct
        $loaded.css({width:getWidth(),height: getHeight(),overflow: settings.get('scrolling') ? 'auto' : 'hidden'});

        $loaded.hide(); //now can be called hide
        $loadingBay.hide();

@jackmoore   Please review and take this solution.
Thanks
Issue can be closed


Add (optional) properties to edges

Until now, Grafito offers properties/data only for nodes.
However, adding this possibility for edges too would be really interesting and useful (most graph databases support, plus it could open the way for things like "weights graphs", etc), so... let's see how this goes! 🚀
P.S. Syntax-wise, especially in queries, this might get a tiny bit complicated, but remains to be seen!

And... ready to merge! 🚀

What we've managed in this PR
In a few words
Now, not only nodes by edges may have properties/data as well
How do we set set this data?
Let's say we have our helpers in place (e.g. lemma, etc):
lemma ~> 'means .:[as: "noun"] someMeaning

(.: is the part where we are setting the edge's - means's - properties, using a weird bug-or-feature of Arturo: empty attributes - yes, .: is actually the "" attribute being set to [as:"noun"] ⚠ )
How do we query this data?
lemma [
    means: |> [][
        as: "noun"
    ]
]

Word of caution: The above (the syntax mainly) may change. So, don't trust the above 100%. Plus, the whole project is still in in-the-making stage, so different things could change. Sure thing is it needs an extreme cleanup & re-organization, which I hope to find the time for soon! 🚀


[setup.py] monkeypatch py_compile to halt compilation on syntax errors

This is an attempt to fix issue https://github.com/behdad/fonttools/issues/290.
It uses a trick found here which monkeypatches the py_compile.compile function used by distutils, so that it will raise an exception instead of a warning on any errors found while bytecompiling the modules.
With this the make all command will properly fail.

the Python 3 builds on Travis were failing because of mixed tabs and spaces (in Python 3 this raises a TabError). I fixed it a subsequent commit.
Shall I open a separate PR for that, is it ok like this?

btw, it was actually thanks to the patch that the TabError in merge.py could be found.. I guess one more reason to have make return failure on compilation errors.

Thanks Cosimo.
I also added you as a collaborator on the repo so you can push trivial patches directly.

Thank you!


All passive tracking learning devices randomly disappear

Unfortunately I haven't found a way to reproduce it and this is the only time I was able to capture the log messages.
...
2018-04-17 17:03:04 [INFO] [PID-5268] server.go func1:978 <IP_ADDRESS>:50470 POST /passive 19.691983ms
2018-04-17 17:03:10 [WARN] [PID-5268] analysis.go AnalyzeSensorData:117 [qg] nb1 classify: problem preparing SQL: database is locked
2018-04-17 17:03:12 [WARN] [PID-5268] analysis.go AnalyzeSensorData:117 [qg] nb1 classify: problem getting key: database is locked
2018-04-17 17:03:12 [INFO] [PID-5268] fingerprints.go updateCounter:87 have 9 new fingerprints for 'qg', re-calibrating since last calibration was 2562047h47m16.854775807s
2018-04-17 17:03:12 [INFO] [PID-5268] server.go func1:978 <IP_ADDRESS>:59900 POST /passive 7.742035674s
2018-04-17 17:03:13 [INFO] [PID-5268] calibration.go dumpSensorsToCSV:399 [qg] dumping 317 fingerprints to /home/find/go/src/github.com/schollz/find3/server/main/data/m0gbo9ga.csv
2018-04-17 17:03:14 [INFO] [PID-5268] server.go func1:978 <IP_ADDRESS>:53874 POST /passive 22.710843ms
2018-04-17 17:03:20 [INFO] [PID-5268] server.go func1:978 <IP_ADDRESS>:59420 GET /ws?device=all&family=qg 95.342µs
...

Server version: c787ee806f46016ee17b007d5cc690914496df6f
Go version: go1.10.1 linux/amd64

What do you mean the passive tracking learning devices randomly disappear?

You mean the passive scanning devices? Or the devices being tracked?

The devices that show up in the Current learning table.

I'm adding them manually with httpie if that matters.

Hm, I don't know why that would happen. Those should persist no matter what since they are written into the database. I'd like to know if there is a way to reproduce.

Yeah. It will be given in the message when you make any POST to the passive settings endpoint.
http POST https://cloud.internalpositioning.com/api/v1/settings/passive family=FAMILY

This will not change any settings, but just tell you the current settings.

 That could introduce something weird. Are you sure you are on https://github.com/schollz/find3/commit/c787ee806f46016ee17b007d5cc690914496df6f ?

Yes, I'm pretty sure. I recompiled it yesterday to test if the latest go version would help. Previously I was using golang package provided by Fedora: golang-1.9.5-1.fc27.x86_64.
$ git log -1
commit c787ee806f46016ee17b007d5cc690914496df6f (HEAD -> master, origin/master, origin/HEAD)
Author: Zack Scholl<EMAIL_ADDRESS>Date:   Sat Apr 14 19:08:01 2018 -0700

    close lockfile after closing database

$ stat main
  File: main
  Size: 22821928        Blocks: 44576      IO Block: 4096   regular file
Device: fc01h/64513d    Inode: 29691       Links: 1
Access: (0775/-rwxrwxr-x)  Uid: ( 1000/    find)   Gid: ( 1000/    find)
Context: unconfined_u:object_r:user_tmp_t:s0
Access: 2018-04-17 14:48:22.160248016 +0300
Modify: 2018-04-17 14:48:13.409126345 +0300
Change: 2018-04-17 14:48:13.412126387 +0300

Hm okay I'l have to investigate - it must be something to do with too many inserts too quickly.

@hit0ri Can you send me this database?

Closed until I receive more information


client_affinity is referred to as cluster_affinity and therefore doesn't work

https://github.com/cloudposse/terraform-aws-global-accelerator/blob/1d5bdc41409408c308fd311d757d165c970a8989/main.tf#L31
Simple PR, but I'm too tired to do it up tonight...

I appreciate the quick response on this!


feat(remix-dev): upgrade unstable_cssModules config option to v2_cssModules

Follow-up of @markdalgleish's #5800

I'm not sure if we want to introduce churn around this flag just before v2. Do you mind if we close this for now and potentially come back to it?

@markdalgleish As pointed out in @brophdawg11's #5829, the way to go is to upgrade unstable_* flags to v2_* once the API is becoming stable, so that's why I created this PR

@markdalgleish As pointed out in @brophdawg11's #5829, the way to go is to upgrade unstable_* flags to v2_* once the API is becoming stable, so that's why I created this PR

This is generally the idea, but I think with the CSS stuff it might not be worth bumping to v2_ for brand new features that don't break or impact current APIs. Matt's comment was more about how we approach new major-version changes to APIs in the future. New experimental/unstable features can stabilize at any point and be merged before a major release so long as nothing is breaking!

So basically unstable_cssModules should actually be released in dev instead of v2 in future scenarios, but for this time the team opted to do it in v2?


Invalid composer data reported by packagist

The mhetreramesh/flysystem-backblaze package of which you are a maintainer has failed to update due to invalid data contained in your composer.json. Please address this as soon as possible since the package stopped updating.
It is recommended that you use composer validate to check for errors when you change your composer.json.
$ composer validate

  [RuntimeException]
  require.mikey179/vfsStream is invalid, it should not contain uppercase characters.
  Please use mikey179/vfsstream instead.

https://github.com/gliterd/flysystem-backblaze/pull/29


Add aggregation operator.

This PR adds aggregation operator used by minBy and maxBy. In consequent PRs, this operator will be shipped to our backend and evaluated directly in the database.
It's composed of three separate patches, so it might be better to go one by one.
I've made several ugly typing hacks (namely return op.apply(iter) as AsyncIterable<T>; in _makeTransQueryIter function), but I've spent too much time on trying to make it type-correct. Similar hack already exists with the Select operator.
If any of you are interested in helping me making it type-safe, the main issue is that up until this point (modulo select operator), our operators were mappings Op: T -> T, where T extends ChiselEntity. With aggregations, this changes as aggregation ops are AggrOp: T -> T[K] meaning that our Operator class needs to be templated as Operator<Input, Output>.
That is trivially doable except for the fact that Operators are recursive and they contain inner field of type Operator<InputInner, Input> and recursively. Expressing that in TypeScript seems to be quite hard. I think that to make it work, ChiselCursor would have to be templated by the whole sequence of Input/Output parameters that are used by consecutive operators or something.

@BearLemma @dekimir Let's move this forward so that it doesn't bit-rot.


Implement diagnostic reporting in package resolution

Purpose

Implement diagnostic reporting in package resolution
Fixes #31211
Fixes #32767
Fixes #34221
Fixes #34819

Approach

When a version incompatibility is found, the node is added to the dependency graph as an error node. This node will not be resolved from that point, but is carried forward until the end of the graph creation.
Ones the metadata graph is generated, when creating the resolved package graph, the grapsh is created by keeping the error nodes as unresolved
The compilation is continued as usual and cannot resolve module diagnostic is expected to be printed

Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Created the PR to the stage branch.


Add Compare button to Test tab too

Also in Test tab it can be handy to compare two reports

But this is already possible for rerunning reports and then comparing the results. Do you also want to be able to compare two separate tests?

@MatthijsSmets is het zo duidelijk? Kan hij naar Todo of naar In Progress?


ALTO output from Abbyy11r8 contains  between

Perhaps this is not an error.
Kind regards,
J. Barth

Can you provide sample data and how you ran the tool?

I guess you output the ALTO files directly from ABBYY, because we don't yet provide a transormation from ABBYY to ALTO. Then this should be an example: https://digi.bib.uni-mannheim.de/~stweil/ocr-praxis/Testseiten/alto/417576986_0031.xml
The <SP> stands AFAIK for space and it should validate in this form.

Yes, I'll try to find out if  (=space) is really necessary between s in ALTO.

I guess that it still validates without the SP tags. Moreover, most of the information (HPOS, WIDTH) can be calculated from the line above and below, but if the width of a space is important for some application, then it might be easier to have this data directly. I don't know what the VPOS information for a space says or whether it is also determined by some other values.

On ALTO 2.1 .xsd it looks like this:
  \<xsd:sequence maxOccurs="unbounded"\>
    \<xsd:element name="String" type="StringType"/\>
    \<xsd:element name="SP" minOccurs="0"\> ...
    \</xsd:element\>
  \</xsd:sequence\>

So strictly speaking it seems that <SP> is not necessary, but the <sequence> seems to imply it.

but the  seems to imply it.

Not sure. I only see here, that, if <SP> occurs, then it has to occur after a <String>.

The ALTO documentation says "A TextBlock is divided into lines and those are divided into strings, spaces and hyphens". I don't interpret that as a strict requirement that spaces are required, and nor does the .xsd. It's clear that spaces are required if the strings are given without HPOS and WIDTH attributes, but I think it is redundant if those attributes are available.

The ALTO spec itself needs to clarify this issue.

Clemens has created an issue for that: https://github.com/altoxml/schema/issues/54 (thank you).

Thanks for flagging this, I will put it on the agenda for our next ALTO board call which will be held November 29th.

To chip in, I've interpreted the standard that the <SP><String> alternation is mandatory and that whitespace should never occur inside a <String> and this is how I implemented it.

If a <String> never contains whitespace, then <SP> is completely redundant. Does ALTO allow overlapping words in a row? If yes, does that require a separating space with negative width? :-)

If a  never contains whitespace, then  is completely redundant.

Why? Whitespace is a character like any other and personally I would've taken the decision to encode it explicitly using <String> if the standard wouldn't heavily imply that you shouldn't do that. Of course, you can throw away the data and let people compute inter-word spacing implicitly provided through word bounding boxes but it isn't like tesseract, kraken or any other sequence classification based OCR engine doesn't output a label for whitespace (and the boundaries of that activation can almost certainly differ from the boundary of the activations of the adjacent letters). I'd rather not throw away metadata that some weird subdiscipline in the humanities that only the 8 people participating in it have ever heard about might need.

Does ALTO allow overlapping words in a row?

ALTO luckily allows overlapping elements in constrast to PageXML.

Then how would you encode two overlapping words if you are forced to put a <SP> between them?

Just have overlapping bounding boxes? Presumably there is still a reading order that determines the ordering of the <String> tags. But yeah it helps that I decided a long time ago that words are a waaay to squishy concept and arbitrarily defined anything bounded by whitespace is a separate word/segment for serialization purposes (not only for ALTO). Of course, I you want to encode a proper tokenization, this data model shouldn't be used. On the other hand, I'm of firm conviction starting to do that in a raw OCR serialization format is only going to lead to madness.

Just to follow up - I'm afraid a quick resolve is not really around the corner...the issue was discussed in the last ALTO board call, with the core elements of the discussion summarized here.
While the general feeling was that the use of <SP> is not mandatory, some more research into ALTO's history is required to determine the original authors exact intentions.
An expansion of the <SP> tag with a width attribute has been identified among board members as a possibility to create more useful future applications for the <SP> tag.
If one really wants to be on the safe side, the quick solution right now would be to indeed include <SP> in the output of any ALTO export implementation as it is also straightforward to remove in post-processing.

ALTO's history is required to determine the original authors exact intentions.

As a note, most of the character-based classification systems common at the time ALTO was originally specified didn't treat whitespace as a proper glyph, i.e. whitespace is just something bordered by other glyphs and is never seen by the classifier as such. This at least explains the existence of a separate <SP> tag.

Thank you, @cneud, @mittagessen and the ALTO board.
As the current DFG viewer expects the <SP> tags, I think that programs like ocr-transform should produce them, too. Pull request https://github.com/tesseract-ocr/tesseract/pull/2117 adds the tags to Tesseract's new ALTO output, so that output is now compatible with the DFG viewer.

The addition of the <SP> should be handled uspream in the corresponding transformation. Currently, we use hocr2alto and page2alto. We can keep this issue here open as a reminder.

According to the ALTO XSD the SP tag is optional - minOccurs="0"
And I do not see a way how to reliably calculate HEIGHT/WIDTH/VPOS/HPOS attributes from the hOCR data for the SP tag.
IMHO - proper handling of optional SP tag should be fixed by DFG viewer.

If the <SP> is not mandatory, we have to "ignore" it in the styles of the fulltext view and always make a space after a <STRING>.
This is what I've done in the DFG-Viewer styles now. Please have a look at the current master of the DFG-Viewer at test.dfg-viewer.de.
Please compare the example from above in current master and in version 5.0 of DFG-Viewer and report change requests.

@albig IMHO the second one seems better from user perspective - it is more readable/compact and if you need to copy it and ie. quote the text in your paper/text you will need to remove the blanks manually.

@albig IMHO the spacing looks better now (in master), but the linebreaks seem a bit random...


tree.treeModel.expandAll() Not working for  'angular2-tree-component'

Hi @adamkleingit
I have been enjoying using angular2-tree-component for my tree implementation.
I haven't seen any problems until I tried to expandAll()
Here is the webpackbin URL with issue reproduce
https://www.webpackbin.com/bins/-KuZsrLVxHHv40hpsVlk
I am using 'angular2-tree-component' v3.1.0
I found
 #tree.treeModel.expandAll()

not working properly for async node structure when I import TreeModule from 'angular2-tree-component'
import { TreeModule } from 'angular2-tree-component';

Ref
import { BrowserModule } from '@angular/platform-browser';
import { NgModule } from '@angular/core';

import { TreeModule } from 'angular2-tree-component';

import { AppComponent } from './app.component';

@NgModule({
  declarations: [
    AppComponent
  ],
  imports: [
    BrowserModule,
    TreeModule
  ],
  providers: [],
  bootstrap: [AppComponent]
})
export class AppModule { }

expandAll() working fine when I import TreeModule from  'angular-tree-component'
import { TreeModule } from 'angular-tree-component';

I m using Chrome Version 60.0.3112.113
I have to use 'angular2-tree-component' in my project,  also if you could explain what is difference between 'angular2-tree-component' and 'angular-tree-component' as I found both are working with angular 2 and 4.
Can you please help me here
Thanks!
Regards
Abhi

'angular2-tree-component' is the old module, then it was renamed to angular-tree-component. You should use angular-tree-component because this is updated to version 5.0.0 at 2017-01-09 (see the changelog). That is the reason why import TreeModule from angular-tree-component works!

Thanks @crivadavi
I have updated my code everything working fine now !

@AbhiThakare - You could close the issue if it is fixed. Saves the maintainer a little work.

v7.2.1, these do not work:
ngAfterViewInit() {
    this.tree.treeModel.expandAll(); // works first time
  }

(initialized)="onInitialized(tree)"


Formulas raise type mismatch if condition contains variable with same name as key

Hi there,
Many thanks for this super useful library!
I found a potential bug or at least unexpected behaviour in formulas. The problem happens when using a variable that has the same name as the key/column it's applied on.
Here's some example code:
import datamancer
let df = toDf({ "x" : @[1, 2, 3, 4, 5], "y" : @["a", "b", "c", "d", "e"] })

# works as described in docs: echo df.filter(f{ `x` < 3 or `y` == "e" })
# the below produces a type mismatch:

let y="e"
echo df.filter(f{ `x` < 3 or `y` == y })

This can be easily avoided by renaming the variable to something else, but it might be worthwhile pointing this out in the docs (which I realize are thin in formulas right now).
Thanks again,
Andreas

Hey! Thanks for the kind words and the issue!
That is indeed just an oversight. I guess I never encountered the problem or forgot to investigate if I ever did. Previous to #66 we were just not adjusting the identifier that is associated to the Tensor as part of the formula loop body in the case where the user used accented quotes. If you wrote:
echo df.filter(f{ `x` < 3 or idx("y") == y })

instead for example, it would produce a yT variable for you instead. Now, we make sure to produce unique identifiers in all the cases.
And yes, sorry that the docs are still a bit short. You can find a bit more here:
https://scinim.github.io/getting-started/basics/data_wrangling.html
but I should probably write even more about formulas in particular.
Just while you're here, I would mention a couple of things:

in your formula, the code deduces the type information for the "x" column from the 3 and the "y" column from the y variable. There can be cases where the code doesn't manage to deduce the type correctly in more complex expressions (more on that below)
as mentioned above, you can explicitly specify if you wish to access a full column or an index of a column using col("foo") or idx("foo"). This should be more robust in case you ever stumble over other bugs like this one
the col and idx syntax can also be used to overwrite the type, if the type deduction fails. I.e. you could do:

echo df.filter(f{ idx("x", int) < 3 or idx("y", string) == y })
to be explicit about the type information (this is in addition to the type hints you can put at the beginning of the formula, i.e.
```nim
f{string -> int: <formula>}

for example. The latter is convenient, but of course not sufficient if you have multiple different types in an expression.
Hope those points help a bit. And feel free to reach out in case something else is unclear (or broken!)! :)

What an awesome answer. Thanks so much!


#2105 Update tests according to changes in Claim functionality

#2105
Update tests according to changes in Claim functionality

Also, i've merged tests with the latest master and have reverted changes to disable claim queue.
Please, adjust your code according to the comments and re-check how they're running locally


cannot install

When trying to run "sudo su -c "bash <(wget -qO- https://cutt.ly/PjNkrzq)" root" I get: chmod: cannot access '/usr/local/sbin/gopro': No such file or directory
What do I have to do ?

Were you able to solve this issue?
Thanks!


Invoking  gives an error.

When I try to use the formio-grid to display my submissions I get the following error:
To call the grid:
<formio-grid src="https://<my-project>.form.io/proyecto"></formio-grid>
The error:

EXCEPTION: Uncaught (in promise): Error: Error in ./LoginComponent class LoginComponent - inline template:1:15 caused by: Cannot read property 'hasOwnProperty' of undefined↵Error: Error in ./LoginComponent class LoginComponent - inline template:1:15 caused by: Cannot read property 'hasOwnProperty' of undefined↵    at ViewWrappedError.ZoneAwareError (http://localhost:3000/index.js:103075:33)

The error goes on and on.
Not sure what am I doing wrong, any help would be appreciated.

You must also pass in a 'query' object with optional keys (limit & skip). Your html to call the grid would look like.
<formio-grid src="https://.form.io/proyecto" [query]="{}">

Closed due to no activity. If the issue is still relevant, please reopen


cmake: classify libraries in /usr/lib as system libraries

Motivation for this change
This change is only relevant for App Bundle builds on macOS. For an App Bundle
every non-system runtime dependency gets copied recursively into the App Bundle
to make a standalone distributable package.
The replacement of every occurence of '/usr' by '/var/empty' by Nix thwarts App
Bundle builds, because libraries in /usr/lib do not get correctly classified as
system libraries.
This patch reverts this single path name back to '/usr/lib', so that packaging
App Bundles on macOS works. FYI: within CMake the occurence is also guarded by
an 'if(APPLE)' statement.
The substituteInPlace matches only a single line: https://gitlab.kitware.com/cmake/cmake/-/blob/master/Modules/GetPrerequisites.cmake#L522
Let me know, if a proper patch is preferred instead.
Draft, because I'm still building ;)
Edit: Building the world succeeded
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

/rebase-staging

/rebase-staging

@hannesweisbach please rebase on staging.

@hannesweisbach please rebase on staging.

Would not this make builds less hermetic on Darwin (especially when not using sandbox)?

Would not this make builds less hermetic on Darwin (especially when not using sandbox)?

@hannesweisbach please rebase on staging.

I hope I did that right. "cimg: 2.9.1 -> 2.9.2", commit
a85ad18 had a conflict, I'm not sure why …

@hannesweisbach please rebase on staging.

I hope I did that right. "cimg: 2.9.1 -> 2.9.2", commit
a85ad18 had a conflict, I'm not sure why …

@hannesweisbach you forgot to change the base branch. I did that for you.

@hannesweisbach you forgot to change the base branch. I did that for you.


Disable checkstyle plugin in sample build process. Fix #152

Codecov Report

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

@@            Coverage Diff            @@
##             master     #154   +/-   ##
=========================================
  Coverage     41.65%   41.65%
  Complexity      146      146
=========================================
  Files            59       59
  Lines           773      773
  Branches         46       46
=========================================
  Hits            322      322
  Misses          421      421
  Partials         30       30

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 14475d1...3fb6bea. Read the comment docs.


scrolling in the settings moves slightly to the left and back again

Windows Terminal version (or Windows build number)
1.9.1942.0
Other Software
No response
Steps to reproduce
150% scale
1920 x 1980
open settings, go to profiles or somewhere, and scroll
Expected Behavior
No response
Actual Behavior
moves left then back again as per recording
https://user-images.githubusercontent.com/51363864/126472778-29d410d0-e8e7-4ca2-81bc-cd58a889fb47.mp4

Are you scrolling with a mouse wheel, or a trackpad?

With a mouse wheel. It's a Logitech MX Master. Doesn't happen with (precision) touchpad.

Out of curiosity, does this repro in the XAML Controls Gallery app? That's a good sample app for checking if UI bugs are Terminal-specific, or WinUI specific. In that app you should be able to go to the "NavigationView" page, which should have some samples of UI like the Terminal settings.

@zadjii-msft nope, doesn't repro in XCG.

This might be due to the ScrollViewer introduced to constrain the bounds of the settings and reverted in #10716.
Tentatively closing as a /duplicate of #10664.


Refactor UOps match logic to UPat instead of dicts

Refactors the _match function to use UPats. I currently added a from_dict method on UPat so that this PR can do the main refactor without also including all the changes to rewrite the patterns from dicts to UPats. If this looks good I'm happy to add those here or in a followup PR (and remove the from_dict method).
Speed benchmark:
# master (236390aa)
***** model forward in  31.49 ms
***** model schedule in  10.47 ms
***** model lower in 2608.46 ms
# this pr
***** model forward in  31.40 ms
***** model schedule in  10.41 ms
***** model lower in 1556.09 ms

@alecchendev What backend did you benchmark against?
I ran PROFILE=0 python3 test/external/external_benchmark_schedule.py on both an M1 mac and tinybox green - This diff causes a regression.
eg. tinybox green
master:
***** model forward in  51.64 ms
***** model schedule in  16.26 ms
***** model lower in 2097.72 ms

this branch:
***** model forward in  52.52 ms
***** model schedule in  16.24 ms
***** model lower in 2141.94 ms

What backend did you benchmark against?
I ran PROFILE=0 python3 test/external/external_benchmark_schedule.py on both an M1 mac and tinybox green - This diff causes a regression.

Huh, interesting, I ran the same command on my M1 macbook (air) to get my results.

Oh but actually I'm noticing if I run the test many times I get results closer to yours. Not sure why the first time I had run the test yesterday and today it showed the opposite

Oh this is a much better way to do the bounty!

Merged. @alecchendev the bounty is yours once you merge the second PR.

Awesome! should get around to the next PR within a few days


Remove CkVec from ck-ldb

[x] Test with NAMD
[x] Test with ChaNGa
[x] Verify that stats message is being cleared/there is no memory leak in CentralLB and HybridBaseLB

Related to #2108 which is probably out of date after the new framework merges.

Does this still need to be tested with NAMD and ChaNGa as the commit msg body says?

Fix for NAMD: https://gitlab.com/tcbgUIUC/namd/-/merge_requests/51
Fix for ChaNGa: https://github.com/N-BodyShop/changa/pull/93

Does this still need to be tested with NAMD and ChaNGa as the commit msg body says?

It's ready for merge now, it's been tested with NAMD and ChaNGa and they've both merged patches to support this change.


[WIP] Django 1.9

f0c6dc5 Implements various models cleanup:
Registering -> Registering of models (reversion, watson) are done in appconfig.ready(), when Django says the application is ready.
Signals -> signals.py (hooks, e.g. post_save, post-delete)
Tasks -> tasks.py (tasks which modifies a model instance)
Release notes for Django 1.9: https://docs.djangoproject.com/en/1.9/releases/1.9/
ToDo:

[ ] #1376

Great work!

The django-chunks fix requires migrations to be faked when ran.

Actual tests failing this time. :+1: It has to do with templates, and as Django swapped out its own template system for a multitude of choices (including its own), that's probably the root cause. I won't have time to look at it, any takers?

Fixed original errors. Now we're getting lessc errors.
This is likely because of the following code in base.py:
COMPRESS_PRECOMPILERS = (
    ('text/less', 'lessc --include-path=%s {infile} {outfile}' % DJANGO_WIKI_STATIC),
)

Not sure why the tests didn't fail before and I don't really know how to fix this.

:tada:

:+1:

Django-filebrowser does not support Django 1.9 yet. I just made a pull request so let's hope it gets merged.


fix: fuzzy search match each char only once

See this issue

See https://github.com/ukraine-taskforce/request-frontend/pull/25


Release binaries

Generate release binaries with gox and push them to GitHub Releases when a commit is tagged.

Closed via 21bbdaff605381fa60239c119007b8ae24b28200


Support Amazon Music

Possible library to support Amazon Music API access: https://github.com/Jaffa/amazon-music

This is sort of the same as the other ones, we should prioritize the other platforms and then see if more people are interested in using this


Percona DB CrashLoopBackOff

BUG REPORT
What happened:
Launched Percona DB (as described in docs) but after some time it CrashLoopBackOff
What you expected to happen:
Expected it to keep running.
How to reproduce it (as minimally and precisely as possible):
Did not try to reproduce.
Anything else we need to know?:
We used 3 nodes K8S cluster on AWS with OpenEBS enabled.
Environment:

kubectl get nodes

$ kubectl get nodes
NAME                            STATUS    AGE       VERSION
ip-172-20-37-159.ec2.internal   Ready     1d        v1.7.4
ip-172-20-43-216.ec2.internal   Ready     1d        v1.7.4
ip-172-20-44-172.ec2.internal   Ready     1d        v1.7.4

kubectl get pods --all-namespaces

$ kubectl get pods --all-namespaces
NAMESPACE       NAME                                                             READY     STATUS             RESTARTS   AGE
cattle-system   core-services-dns-1-d0b8e21b                                     2/2       Running            0          14h
cattle-system   core-services-dns-2-32c16b6d                                     2/2       Running            0          14h
cattle-system   core-services-healthcheck-1-2faeb0af                             2/2       Running            0          14h
cattle-system   core-services-healthcheck-2-b4d33f33                             2/2       Running            0          14h
cattle-system   core-services-metadata-1-d2a54033                                2/2       Running            0          14h
cattle-system   core-services-metadata-2-1cd927c8                                2/2       Running            0          14h
cattle-system   core-services-network-manager-1-87d33b4c                         2/2       Running            0          14h
cattle-system   core-services-network-manager-2-f1bc67ab                         2/2       Running            0          14h
cattle-system   kubernetes-support-kubectl-shell-1-1900c090                      2/2       Running            0          14h
cattle-system   rancher-agent-68j34                                              1/1       Running            0          14h
cattle-system   rancher-agent-jng74                                              1/1       Running            0          14h
default         maya-apiserver-3416621614-39fgh                                  1/1       Running            0          1d
default         openebs-provisioner-4230626287-35ddg                             1/1       Running            1          1d
default         percona                                                          0/1       CrashLoopBackOff   178        15h
default         pvc-fc58f246-c495-11e7-9677-0a4685f6f814-ctrl-2168308135-9mjz6   1/1       Running            1          15h
default         pvc-fc58f246-c495-11e7-9677-0a4685f6f814-rep-1455769431-6w3c3    1/1       Running            1          15h
default         pvc-fc58f246-c495-11e7-9677-0a4685f6f814-rep-1455769431-z0g75    1/1       Running            0          15h
kube-system     dns-controller-3497129722-mk925                                  1/1       Running            0          1d
kube-system     etcd-server-events-ip-172-20-43-216.ec2.internal                 1/1       Running            0          1d
kube-system     etcd-server-ip-172-20-43-216.ec2.internal                        1/1       Running            0          1d
kube-system     kube-apiserver-ip-172-20-43-216.ec2.internal                     1/1       Running            0          1d
kube-system     kube-controller-manager-ip-172-20-43-216.ec2.internal            1/1       Running            0          1d
kube-system     kube-dns-479524115-nblgb                                         3/3       Running            1          1d
kube-system     kube-dns-479524115-x90g9                                         3/3       Running            3          1d
kube-system     kube-dns-autoscaler-1818915203-vrwkk                             1/1       Running            0          1d
kube-system     kube-proxy-ip-172-20-37-159.ec2.internal                         1/1       Running            1          1d
kube-system     kube-proxy-ip-172-20-43-216.ec2.internal                         1/1       Running            0          1d
kube-system     kube-proxy-ip-172-20-44-172.ec2.internal                         1/1       Running            0          1d
kube-system     kube-scheduler-ip-172-20-43-216.ec2.internal                     1/1       Running            0          1d
kube-system     kubernetes-dashboard-1319779976-7x9zm                            1/1       Running            0          1d

kubectl get services

$ kubectl get services
NAME                                                CLUSTER-IP       EXTERNAL-IP   PORT(S)             AGE
kubernetes                                          <IP_ADDRESS>       <none>        443/TCP             1d
maya-apiserver-service                              <IP_ADDRESS>    <none>        5656/TCP            1d
pvc-fc58f246-c495-11e7-9677-0a4685f6f814-ctrl-svc   <IP_ADDRESS>   <none>        3260/TCP,9501/TCP   15h

kubectl get sc

$ kubectl get sc
NAME                 TYPE
default              kubernetes.io/aws-ebs
gp2 (default)        kubernetes.io/aws-ebs
openebs-cassandra    openebs.io/provisioner-iscsi
openebs-es-data-sc   openebs.io/provisioner-iscsi
openebs-jupyter      openebs.io/provisioner-iscsi
openebs-kafka        openebs.io/provisioner-iscsi
openebs-mongodb      openebs.io/provisioner-iscsi
openebs-percona      openebs.io/provisioner-iscsi
openebs-redis        openebs.io/provisioner-iscsi
openebs-standalone   openebs.io/provisioner-iscsi
openebs-standard     openebs.io/provisioner-iscsi
openebs-zk           openebs.io/provisioner-iscsi

kubectl get pv

kubectl get pv
NAME                                       CAPACITY   ACCESSMODES   RECLAIMPOLICY   STATUS    CLAIM                     STORAGECLASS      REASON    AGE
pvc-fc58f246-c495-11e7-9677-0a4685f6f814   5G         RWO           Delete          Bound     default/demo-vol1-claim   openebs-percona             15h

kubectl get pvc

$ kubectl get pvc
NAME              STATUS    VOLUME                                     CAPACITY   ACCESSMODES   STORAGECLASS      AGE
demo-vol1-claim   Bound     pvc-fc58f246-c495-11e7-9677-0a4685f6f814   5G         RWO           openebs-percona   15h

kubectl get deployments

$ kubectl get deployments
NAME                                            DESIRED   CURRENT   UP-TO-DATE   AVAILABLE   AGE
maya-apiserver                                  1         1         1            1           1d
openebs-provisioner                             1         1         1            1           1d
pvc-fc58f246-c495-11e7-9677-0a4685f6f814-ctrl   1         1         1            1           15h
pvc-fc58f246-c495-11e7-9677-0a4685f6f814-rep    2         2         2            2           15h

kubectl get jobs

$ kubectl get jobs
NAME          DESIRED   SUCCESSFUL   AGE
sql-loadgen   1         1            15h

OS: Ubuntu

$ lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 16.04.2 LTS
Release:	16.04
Codename:	xenial

$ more /etc/os-release
NAME="Ubuntu"
VERSION="16.04.2 LTS (Xenial Xerus)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 16.04.2 LTS"
VERSION_ID="16.04"
HOME_URL="http://www.ubuntu.com/"
SUPPORT_URL="http://help.ubuntu.com/"
BUG_REPORT_URL="http://bugs.launchpad.net/ubuntu/"
VERSION_CODENAME=xenial
UBUNTU_CODENAME=xenial

Kernel:

$ uname -a
Linux ip-172-20-43-216 4.4.0-66-generic #87-Ubuntu SMP Fri Mar 3 15:29:05 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux

Install tools:
--> kubectl version

$ kubectl version
Client Version: version.Info{Major:"1", Minor:"7", GitVersion:"v1.7.4", GitCommit:"793658f2d7ca7f064d2bdf606519f9fe1229c381", GitTreeState:"clean", BuildDate:"2017-08-17T08:48:23Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"7", GitVersion:"v1.7.4", GitCommit:"793658f2d7ca7f064d2bdf606519f9fe1229c381", GitTreeState:"clean", BuildDate:"2017-08-17T08:30:51Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"linux/amd64"}

--> docker version
$ sudo docker version
Client:
 Version:      1.12.6
 API version:  1.24
 Go version:   go1.6.4
 Git commit:   78d1802
 Built:        Tue Jan 10 20:38:45 2017
 OS/Arch:      linux/amd64

Server:
 Version:      1.12.6
 API version:  1.24
 Go version:   go1.6.4
 Git commit:   78d1802
 Built:        Tue Jan 10 20:38:45 2017
 OS/Arch:      linux/amd64

Others:
--> Logs of percona: logs-of-percona-on-aws[CrashLoopBackOff].txt
--> Logs of pvc-fc58f246-c495-11e7-9677-0a4685f6f814-ctrl-con: logs_of_pvc-fc58f246-c495-11e7-9677-0a4685f6f814-ctrl-con_on_aws[percona-CrashLoopBackOff].txt
--> Logs of pvc-fc58f246-c495-11e7-9677-0a4685f6f814-rep-con: logs_of_pvc-fc58f246-c495-11e7-9677-0a4685f6f814-rep-con_on_aws[percona-CrashLoopBackOff].txt
--> Yaml file used to run Percona: demo-percona-mysql-pvc.yaml
--> Yaml file used to genrate load: sql-loadgen.yaml
--> Rancher UI [Screenshot]:

@a4abhishek , here is the analysis :

There has been a network loss for a period greater than 60s between the controller and replica pods which has lead to the iscsi target being stopped.
This has led to the volume bring re-mounted as RO on the application node (i.e., node where percona app has been scheduled).
Subsequent failed DB writes on this mount has caused the application crash. Every restart thereafter has failed due to the inability to write into /var/lib/mysql (where the vol is mounted)

Details of this behaviour can be found here : https://github.com/openebs/openebs/issues/1260
This issue will be tracked by the above issue.
Please note the logs illustrating the above :

Ctrl.log

08/11/2017 20:46:47 time="2017-11-08T15:16:46Z" level=error msg="write tcp <IP_ADDRESS>:3260-><IP_ADDRESS>:51720: write: broken pipe"
08/11/2017 20:46:49 time="2017-11-08T15:16:48Z" level=info msg="connection is connected from <IP_ADDRESS>:53136...\n"
08/11/2017 20:46:51 time="2017-11-08T15:16:50Z" level=info msg="Listening ..."
08/11/2017 20:46:54 time="2017-11-08T15:16:53Z" level=error msg="write tcp <IP_ADDRESS>:3260-><IP_ADDRESS>:51720: write: broken pipe"
08/11/2017 20:46:54 time="2017-11-08T15:16:53Z" level=error msg=EOF
08/11/2017 20:46:57 time="2017-11-08T15:16:57Z" level=error msg="write tcp <IP_ADDRESS>:3260-><IP_ADDRESS>:51720: write: broken pipe"
08/11/2017 20:46:59 time="2017-11-08T15:16:58Z" level=error msg="write tcp <IP_ADDRESS>:3260-><IP_ADDRESS>:51720: write: broken pipe"
08/11/2017 20:46:59 time="2017-11-08T15:16:59Z" level=error msg=EOF
08/11/2017 20:47:59 time="2017-11-08T15:17:58Z" level=info msg="stopping target iqn.2016-09.com.openebs.jiva:pvc-fc58f246-c495-11e7-9677-0a4685f6f814 ..."
08/11/2017 20:47:59 Closing Listener ...

rep.log

08/11/2017 20:48:01 time="2017-11-08T15:18:00Z" level=info msg="Closing volume"

percona.log

chown: changing ownership of '/var/lib/mysql/ib_buffer_pool': Read-only file system
chown: changing ownership of '/var/lib/mysql/ibdata1': Read-only file system
chown: changing ownership of '/var/lib/mysql/private_key.pem': Read-only file system
chown: changing ownership of '/var/lib/mysql/ibtmp1': Read-only file system


Auth working OK, but queries returning 403

Hi,
I've been working for some days on a project and it basically does a couple things right now:

Login
Retrieve data from a collection and show it in a table

It was all working great, but since yesterday I get a 403 Forbidden message. I tought about auth not working well, but I get this trace from Strapi:
[2020-11-06T10:23:40.208Z] debug OPTIONS /auth/local (0 ms) 204
[2020-11-06T10:23:40.641Z] debug POST /auth/local (422 ms) 200
[2020-11-06T10:23:40.660Z] debug OPTIONS /users/me (1 ms) 204
[2020-11-06T10:23:40.715Z] debug GET /users/me (49 ms) 200
[2020-11-06T10:23:42.885Z] debug GET /devices (15 ms) 403
[2020-11-06T10:23:42.892Z] debug GET /devices/count (19 ms) 403
[2020-11-06T10:23:42.907Z] debug GET /devices?_limit=10&_start=0 (12 ms) 403

it seems auth is working but then I get a 403 forbidden when calling the devices endpoints. Same queries using Postman work just well:
[2020-11-06T10:29:48.071Z] debug POST /auth/local (103 ms) 200
[2020-11-06T10:29:51.901Z] debug GET /devices (41 ms) 200
[2020-11-06T10:29:55.013Z] debug GET /devices/count (26 ms) 200
[2020-11-06T10:30:00.022Z] debug GET /devices?_limit=10&_start=0 (35 ms) 200

I also tried using anonymous chrome window, in case something was getting catched, but I got exactly the same.
I've implemented auth using auth Nuxt module, following this tutorial: https://strapi.io/blog/strapi-authentication-in-nuxt-js
Any ideas on what can be happening?
Thanks!

Forget about this issue. While refactoring the this.$strapi.login line was blown away so it worked while data was cached.
Sorry!

Forget about this issue. While refactoring the this.$strapi.login line was blown away so it worked while data was cached.

Hi!
Could you post your working login snippet please? I'm having trouble making it work…
Thanks!

Hi, my solution isn't fully working as I still get 403 responses right after logging in. If I then refresh the page, it works ok, no idea why.
I have a login.vue page where I put a form and this method for logging in:
methods: {
    async onLogin(evt: any) {
      evt.preventDefault();
      console.log(`Username: ${this.username}, Passowrd: ${this.password}`);
      try {
        let data = {
          identifier: this.username,
          password: this.password,
        };
        let result = await this.$auth.loginWith("local", { data });
        await this.$strapi.login(data);
      } catch (e) {
        console.log(e);
      }
    },
  },

I configured nuxt's auth module like this:
// Auth Configuration (https://auth.nuxtjs.org/schemes/local.html#usage)
  auth: {

    strategies: {
      local: {
        endpoints: {

          login: {
            url: `${process.env.API_AUTH_URL}/auth/local`,
            method: 'post',
            propertyName: 'jwt'
          },

          user: {
            url: `${process.env.API_AUTH_URL}/users/me`,
            method: 'get',
            propertyName: false
          },

          logout: false
        }
      }
    },

    redirect: {
      login: '/login',
      logout: '/login',
      callback: '/login',
      home: '/'
    }

  },

I guess the best solution would be to create a  custom scheme for the auth module using just the strapi client for logging in instead of what I'm doing. I didn't get so far as my project was just a quick MVP but you have more info about this here: https://auth.nuxtjs.org/guide/scheme.html#creating-your-own-scheme
It would be great if you get it working and share a really working solution :)

I have not done a lot of testing since last night but I think I finally understood something:
when using the strapi module, we don't need the auth module (nor the axios module).
I don't know why I thought I had to install the auth module, maybe because of the strapi-nuxt-auth tutorial 😬
So, here's what's working for me right now (very basic poc):
// page/login.vue

export default {
  data() {
    return {
      email: '',
      password: '',
    }
  },
  methods: {
    async login() {
      await this.$strapi.login({
        identifier: this.email,
        password: this.password,
      })
      this.$router.push('/private')
    },
  },
}

I don't have anything particular in nuxt.config.js, the strapi module actually takes care of everything (auth routes, jwt tokens, etc.)

Yes, thats works... for login.
The problem with that approach (and the reason I ended up using the auth module) is that you have to handle all logib/logout logic by yourself (redirecting on logout/token expiring, redirecting after login, public/private pages, etc...).
I still think the best approach is using the auth module with a custom schema.
An "official" manual about this from the module owner would be great to have! :)


Local pull request success status testing

Docs Build status updates of commit afedfaa:
:clock10: Pending: waiting for processors (13 builds ahead of you)
:warning: Docs Build is busy, currently there are 13 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

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

File
Status
Preview URL
Details

E2E_DocsBranch_Dynamic/index.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.


Webhook

support mutatingwebhookconfiguration to manipulate the placementrule and placement to disable the original controller, including creation/deletion
hack the microshift issue to support inject service-ca into mutatingwebhookconfiguration
move crds to the top-level pkg/testdata folder
remove unused console job role and rolebinding
fix potential security vulnerabilities for helm/v3.

ref: https://github.com/stolostron/backlog/issues/25858

/retest

/retest

/retest


Commented out unused variables, per pixlet lint

Description
Initial commit for msteamsstatus app. Please insert appropriate hash of client ID & secret for Tidbyt's AAD tenant on lines 29 and 30.
Copilot

🤖 Generated by Copilot at 3f99bae
Summary
📄📝📱

This pull request adds a new app called msteamsstatus to the Tidbyt community repository. The app shows the user's Microsoft Teams presence on a Tidbyt device, and includes a manifest file and a README file.

msteamsstatus app
manifest and README files
autumn leaves of code

Walkthrough

Create a new app called msteamsstatus that shows the user's Microsoft Teams presence on a Tidbyt device (link, link)
Add a manifest file that defines the app's metadata and package name (link)
Add a README file that explains the app's purpose, features, and credits (link)

I have read the CLA Document and I hereby sign the CLA

@noahpodgurski Updated with your suggested change. Thanks.

Hey all - any updates here? Is there something I need to do to move this forward?

Sorry for the delay. Just a pain to click through all the microsoft things to get credentials out. Let's try with the creds from outlookcalendar for now.


add logger

Add logging to the application in a given path,  /tmp should be a good option.
Use environment variable (viper prefix env var, take a look at it) to decide the level.
Make sure the environment variable is embedded in the package, must not be visible to the end user, only in developing mode it's visible

I think it would be too much work for the actual value, sticking to simple print statements, and any error can be seen with the cobra.CheckErr in main.go

I think it would be too much work for the actual value, sticking to simple print statements, and any error can be seen with the cobra.CheckErr in main.goI think it would be too much work for the actual value, sticking to simple print statements, and any error can be seen with the cobra.CheckErr in main.go


Is it working with new ASP.NET 5?

I have to use Identity Server 3 in ASP.NET 5 with Asp Identity. Is it possible to use this nuget? Or there is other approaches? Thanks

This repo only provides a lightweight base implementation of the IUserService. If you're using ASP.NET Identity 3, then I suggest implementing your own IUserService.


Drop log table disappears when given a blank (only 100% items) drop, if those items are toggled off.

Describe the bug
When doing slayer (specifically only tested at Kurasks so far) and you get bones only drop the table will disappear in its  entirety, while preserving the number killed and value.

Cry

Expected behavior
Table will disappear. Following this if gaining any other drop it will reappear -- seems to be a purely visual bug.
Screenshots
This
Environment (please complete the following information):

Windows 10
Runelite v.1.4.24
Launcher v.1.6.0

Small note: Saw another issue similar but it claimed to be from Ring of Wealth equip. I can confirm either this is an entirely separate issue or not at all caused by the ring of wealth as this bug happens with or without the ring equipped -- this is almost certainly caused by the 100% drop being toggled off, as when I toggled it back on and got a 100% drop it did not occur, whereas otherwise, it had a 100% occurrence rate.

Duplicate of #6240


Issue with Saddles Being Incorrectly Assigned to Camels and Horses

The current version of the mod seems to have an issue where horse saddles are being equipped on camels and camel saddles on horses.

Fixed in commit  #


Polish the Azure.ResourceManager.MarketplaceOrdering

/azp run net - marketplaceordering - mgmt

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.MarketplaceOrdering


Automation

Automatically move your cards to the right place based on the status and activity of your issues and pull requests.

done


Send data back to remote

The examples in the documentation shows that one should use one-way data binding from the remote to the dialog. But assume that the dialog constructs some data that needs to be passed back to the remote, what is the recommended way to do that?

You can also two-way bind the data property.


Make refugee Lothemar scale from Sylvanis exp

Make refugee Lothemar scale off Sylvanis exp
Make refugee Rommath scale off Anastearian exp

gonna leave this one for now, its long


chore(deps): update terraform terraform-ibm-modules/cos/ibm to v7.3.0

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/cos/ibm (source)
module
minor
7.2.2 -> 7.3.0

Release Notes

terraform-ibm-modules/terraform-ibm-cos (terraform-ibm-modules/cos/ibm)
v7.3.0
Compare Source
Features

added support to create a single site bucket using new variable single_site_location(#514) (dc9711e)
added the ability to set force_delete for bucket deletion (#514) (dc9711e)
added the ability to set hard_quota (#514) (dc9711e)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

:tada: This PR is included in version 1.1.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Should my subscription be removed from the list?

My wikivncopy subscription was added in https://github.com/iorate/ublacklist/pull/315
https://iorate.github.io/ublacklist/subscriptions#vietnamese
Now I decided to archive it. Should it be removed from the list?

I think so, but do as you like.

do as you like

So I think I should archive this.

Did you remove it yet?


Request: How to get the list of location code to use?

Hello,
I tried the link given in description of the command parameter "location" but I am not able to see any code list from there.
Is there a way to get a list of usable location I can have with my azure ad to be able to install this application?
Many thanks

I get it also looking at other posts. I think documentation could be updated with the following link, on "location" parameter comments with :  https://azure.microsoft.com/en-gb/global-infrastructure/locations/

if you run the Powershell azure command : az account list-locations -o table
you get something like below (note that the location parameter is the Name column) :
DisplayName               Name                 RegionalDisplayName

East US                   eastus               (US) East US
East US 2                 eastus2              (US) East US 2
South Central US          southcentralus       (US) South Central US
West US 2                 westus2              (US) West US 2
Australia East            australiaeast        (Asia Pacific) Australia East
Southeast Asia            southeastasia        (Asia Pacific) Southeast Asia
North Europe              northeurope          (Europe) North Europe
UK South                  uksouth              (Europe) UK South
West Europe               westeurope           (Europe) West Europe
Central US                centralus            (US) Central US
North Central US          northcentralus       (US) North Central US
West US                   westus               (US) West US
South Africa North        southafricanorth     (Africa) South Africa North
Central India             centralindia         (Asia Pacific) Central India
East Asia                 eastasia             (Asia Pacific) East Asia
Japan East                japaneast            (Asia Pacific) Japan East
Korea Central             koreacentral         (Asia Pacific) Korea Central
Canada Central            canadacentral        (Canada) Canada Central
France Central            francecentral        (Europe) France Central
Germany West Central      germanywestcentral   (Europe) Germany West Central
Norway East               norwayeast           (Europe) Norway East
Switzerland North         switzerlandnorth     (Europe) Switzerland North
UAE North                 uaenorth             (Middle East) UAE North
Brazil South              brazilsouth          (South America) Brazil South
Central US (Stage)        centralusstage       (US) Central US (Stage)
East US (Stage)           eastusstage          (US) East US (Stage)
East US 2 (Stage)         eastus2stage         (US) East US 2 (Stage)
North Central US (Stage)  northcentralusstage  (US) North Central US (Stage)
South Central US (Stage)  southcentralusstage  (US) South Central US (Stage)
West US (Stage)           westusstage          (US) West US (Stage)
West US 2 (Stage)         westus2stage         (US) West US 2 (Stage)
Asia                      asia                 Asia
Asia Pacific              asiapacific          Asia Pacific
Australia                 australia            Australia
Brazil                    brazil               Brazil
Canada                    canada               Canada
Europe                    europe               Europe
Global                    global               Global
India                     india                India
Japan                     japan                Japan
United Kingdom            uk                   United Kingdom
United States             unitedstates         United States
East Asia (Stage)         eastasiastage        (Asia Pacific) East Asia (Stage)
Southeast Asia (Stage)    southeastasiastage   (Asia Pacific) Southeast Asia (Stage)
East US 2 EUAP            eastus2euap          (US) East US 2 EUAP
West Central US           westcentralus        (US) West Central US
South Africa West         southafricawest      (Africa) South Africa West
Australia Central         australiacentral     (Asia Pacific) Australia Central
Australia Central 2       australiacentral2    (Asia Pacific) Australia Central 2
Australia Southeast       australiasoutheast   (Asia Pacific) Australia Southeast
Japan West                japanwest            (Asia Pacific) Japan West
Korea South               koreasouth           (Asia Pacific) Korea South
South India               southindia           (Asia Pacific) South India
West India                westindia            (Asia Pacific) West India
Canada East               canadaeast           (Canada) Canada East
France South              francesouth          (Europe) France South
Germany North             germanynorth         (Europe) Germany North
Norway West               norwaywest           (Europe) Norway West
Switzerland West          switzerlandwest      (Europe) Switzerland West
UK West                   ukwest               (Europe) UK West
UAE Central               uaecentral           (Middle East) UAE Central
Brazil Southeast          brazilsoutheast      (South America) Brazil Southeast

@JPialot76 the documentation states where you can obtain the codes, see here - https://azure.microsoft.com/en-gb/global-infrastructure/services/?products=logic-apps,automation&regions=all
You don't have to use the short name, you can use the full name of the location, just make sure it supports Logic Apps and Azure Automation.

@JPialot76 are you sorted now?
Thanks

Why does UKsouth not work?


feat: add array/base/broadcasted-quinary5d

Resolves #3172 .
Description

What is the purpose of this pull request?

This pull request:

This pull request proposes a new package to the array/base/* namespace: @stdlib/array/base/broadcasted-quinary5d.

Related Issues

Does this pull request have any related issues?

This pull request:

resolves #3172

Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

@kgryte Can you please check if it's done?

@RiyaChy072 use backticks while naming packages.
Ex:-  array/base/broadcasted-quinary5d

Also your tests are failing.
Just a point to add try running them locally using :- make TESTS_FILTER="./array/base/broadcasted-quinary5d/." test locally to know the errors. It did help me resolving bugs.

@Neerajpathak07 Thanks for the advice

/stdlib lint-autofix

/stdlib help

/stdlib merge

@RiyaChy072 Thanks for working on this. Looks like this PR needs a bit more attention before we can provide further review. Once CI failures have been resolved, feel free to request another review.

@Neerajpathak07 Can you help me a little bit with these please?

@RiyaChy072 Firstly refer this pr:- https://github.com/stdlib-js/stdlib/issues/3171 as it is the 4d implementation of your current PR.
Your Test.js file is incorrect change var v to var out  for every single file where you made that change.
Next your shapes do not match with the values like in this case :-
    ✔ returns expected value
/home/runner/work/stdlib/stdlib/lib/node_modules/@stdlib/array/base/broadcasted-quinary5D/lib/main.js:236
            x2 = x3[ j3 ];
                   ^

TypeError: Cannot read properties of undefined (reading '0')

Most of the errors should be resolved once you have fixed the /test file.
You will be having a few lint errors but you can work on that on a separate commit.

@Neerajpathak07 Thanks

@RiyaChy072 are you able to solve this issue or you still need some help?

@Neerajpathak07 I'm trying to figure things out. I'll let you know if i encounter any issues

@Neerajpathak07 CI report has been cleared. Thank you very much for helping me

@kgryte CI failures have been resolved. Can you please check?

@RiyaChy072 It seems you have added changes into [array/base/broadcasted-binary5d](https://coverage.stdlib.io/pr-3229/array/base/broadcasted-binary5d/index.html) but these are not necessary in this particular PR.
I suggest you to remove the file from the commit. So that we only have code coverage for array/base/broadcasted-quinary5D

@Neerajpathak07 Maybe i did that by mistake

@RiyaChy072 LGTM just resolve the merge conflicts and the pr should be ready.

@Neerajpathak07 Did what you said. Now do i need to do anything else?


Added a check in dispute.rs

Hi @grunch ,
this is for #220 from @Catrya .
Added a check like this in dispute.rs
fn check_order_status_for_dispute(status: &str) -> bool {
    match Status::from_str(status).unwrap() {
        Status::Success => true,
        _ => false,
    }
}

I just check that order is in success state to block dispute creation for now, feel free to add other states.

Hi @arkanoider, if users canceled the order they can also open a dispute, so I thought that mostro could also verify that the order was not cancelled.
but speaking with @grunch he says that to open a dispute what we must do is check if the status of the order is active or fiat-sent, which is a much better idea than mine

Sounds good! Wiill fix.

@Catrya @grunch
Dispute should be available only for Active and FiatSent orders now.


Produce nt only in jenkins

Slight change such that nt/rdf is not produced from run.py merge by default, but only in Jenkins pipeline

Okay thanks @deepakunni3 - one thing to keep in mind is that we'll now need to make changes in bothmerge.yaml and merge_jenkins.yaml

Yes but that is temporary. Probably a week or so.


Custom Domain not using serverless-domain-manager plugin

Hi @DavidWells, we use serverless-manifest-plugin in many of our repos, and it's great.  However, we stopped using serverless-domain-manager plugin for various reasons, but liked the integration with that plugin as it changed the URLs if a custom domain was used.  We can use the postProcess to overwrite our domain, but was curious if you'd accept a PR that allows a serverless-manifiest-plugin parameter to set the domain name, instead of a deep integration with serverless-domain-manager.
Thanks,
Nick Willan
Trend Micro

I've moved away from the serverless-domain-manager as well.
I'm open to this 😃
The other thing I'm trying to figure out is how to populate appSync URLs if the stack has them... that's for another issue though


initial changes for 1.13 support

Q
A

Bug fix?
no

New feature?
yes

API breaks?
no

Deprecations?
no

Related tickets

License
Apache 2.0

What's in this PR?

Changes needed to support Istio version 1.13.
Checklist

[x ] Implementation tested
[ x] Error handling code meets the guideline
[ x] Logging code meets the guideline
[ x] User guide and development docs updated (if needed)

To Do

[ ] Check compatiblity with SMM

 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.Kristof Gyuracz 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.


Logging of traffic variables with capital letters

Hi Joost,
When I try to use the CRELOG functionality I am not able to log variables that have capital letters.
Examples (using bada):
This works:
MYLOG ADD traf.alt
MYLOG ADD traf.perf.fuelflow
This does not work:
MYLOG ADD traf.M
Syntax error: MYLOG: Variable TRAF.M not found
MYLOG ADD traf.perf.Thr
Syntax error: MYLOG: Variable TRAF.PERF.THR not found
Wkr, René

Hi Joost,
This is my own solution for making the logger work with case sensitive letters. The problem is in the varexplorer.
I added the following class to the varexplorer.py which takes a dictionary and makes it case insensitive when searching the dictionary
from collections import Mapping

class CaseInsensitiveDict(Mapping):
    def __init__(self, d):
        self._d = d
        self._s = dict((k.lower(), k) for k in d)
    def __contains__(self, k):
        return k.lower() in self._s
    def __len__(self):
        return len(self._s)
    def __iter__(self):
        return iter(self._s)
    def __getitem__(self, k):
        return self._d[self._s[k.lower()]]
    def actual_key_case(self, k):
        return self._s.get(k.lower())

and updated line 102 and 103 in the following way:
if obj and name in CaseInsensitiveDict(vars(obj)):
     return Variable(obj, varset[-2][0], CaseInsensitiveDict(vars(obj)).actual_key_case(name), index)

I don't know if this solution will generate a speed hit.

Hi René,
I've solved it somewhat differently, by adding a case-sensitive 'word' argument type. You can now distinguish, e.g., between traf.M and traf.m. See 93fdd6b.


Use conformance access paths in IRGen

This series of patches — primarily the last — teaches IRGen to use conformance access paths to derive protocol conformances instead of a hacky recursive algorithm.
For now, we're just trusting the AST to figure out the best path, but it shouldn't be too hard to get the AST to give us multiple paths if they exist and then figure out which is best based on the type information currently in use within the function.

@swift-ci Please test.

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - a7834851135f83291ea0dff9363ed44a9c904363
Test requested by - @rjmccall

@swift-ci Please test.

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - a7834851135f83291ea0dff9363ed44a9c904363
Test requested by - @rjmccall

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - a7834851135f83291ea0dff9363ed44a9c904363
Test requested by - @rjmccall

Fantastic! The AST doesn't currently track all of the requirement sources for conformances (which would be needed to enumerate all possible paths and pick the best one), but it does some basic keep-the-shortest-path logic.


formatDate: month is off by one

formatDate in date.ts formats 2021-07-11 as 11 August, etc.

🤦♂

@aapeliv you can do this if you can't wait for me to do it in around 10-12 hours. I just did monthNames[date.month()] instead of `monthNames[date.month()-1]|.

@aapeliv you can do this if you can't wait for me to do it in around 10-12 hours. I just did monthNames[date.month()] instead of `monthNames[date.month()-1]|.

Fixed in 8d40298d


Ruby/YARD support

Hi!
Super cool to see that you've ported another extension to Nova :)
If I look at the documentation, the format is not very different from the one used by YARD.
The entire Documentation API can be found here: https://rubydoc.info/gems/yard
Would it be possible to add YARD support to DocBlockr? 😄

I think I can add at least some basic support. Although there seems to be a variety of commenting styles for ruby. May take some time 😉

If you add some basic support I'll try to help and expand on it 😄
JS isn't my cup of tea, but it's doable with a base to work with.
We're expecting a baby, so take your time. I'm glad you're willing to do this 👍🏼

Hey, I hope all is well. I just updated the develop branch with added ruby support. You may want to check it out.

Awesome, thanks! I'll check out the develop branch and try it out locally ❤

Wow, you implemented both RDoc AND YarDoc, thanks!
It works perfectly, thanks a bunch!


Control

Currently, applicative and monad cannot be aligned because the seq_left and seq_right classes changed to require the discarded computation to return punit. This is not a major problem, but there is some category theory in mathlib that uses applicative and monad and so the two barely-different versions would need to persist in mathlib4. I think backporting the change would be simple (here is a starting point: https://github.com/dselsam/lean3/commit/7e8ead2c0d371ddacef7db30e7dd68d52f4b768f) but Mario thinks it would be a change for the worse (https://leanprover.zulipchat.com/#narrow/stream/270676-lean4/topic/mathport/near/226098511) and wonders if the rationale for the change even still applies (https://leanprover.zulipchat.com/#narrow/stream/270676-lean4/topic/mathport/near/226098127).

It seems Mario was right: https://github.com/leanprover/lean4/pull/311

leanprover/lean4#311 was merged and alignment went smoothly


Generate Appliance ResourceConnector dotnet SDK

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
API changes are not detected in this pull request.

/azp run net - resourceconnector - mgmt - ci

/azp run prepare-pipelines


[Feature][MLops] Support MLflow Projects to train machine learning model

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Support MLflow Projects to train machine learning model
MLflow Projects

[x] BasicAlgorithm: contains LogisticRegression, svm, lightgbm, xgboost
[x] AutoML: AutoML tool，contains autosklean, flaml
[ ] Custom projects: Support for running your own MLflow projects

Use case
No response
Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Please submit the document when submitting the new function. @jieguangzhou

Please submit the document when submitting the new function. @jieguangzhou

I have modified the document for PR.
docs/docs/en/guide/task/mlflow.md
docs/docs/zh/guide/task/mlflow.md


signalbackup-tools: 20230603-2 -> 20230607

Description of changes
Update to latest version. No changes that affect build.
Diff: https://github.com/bepaald/signalbackup-tools/compare/20230603-2...20230607
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[x] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.


Fix issue #42

Added a test for that case

This change is

Review status: 0 of 3 files reviewed at latest revision, 1 unresolved discussion.

src/org/unc/lac/javapetriconcurrencymonitor/monitor/PetriMonitor.java, line 81 at r1 (raw file):

			anyThreadSleepingforTransition[i] = new AtomicBoolean(false);
		}
//		Arrays.fill(anyThreadSleepingforTransition, new AtomicBoolean());

Please remove the commented line. The problem was Arrays.fill filled the array with the same object reference and you need a new atomic boolean for every transition, right?

Comments from Reviewable

Review status: 0 of 3 files reviewed at latest revision, 2 unresolved discussions.

src/org/unc/lac/javapetriconcurrencymonitor/monitor/PetriMonitor.java, line 81 at r1 (raw file):
Previously, juanjoarce7456 (Juan Jose Arce Giacobbe) wrote…
Please remove the commented line. The problem was Arrays.fill filled the array with the same object reference and you need a new atomic boolean for every transition, right?

OK

test/org/lac/javapetriconcurrencymonitor/test/resources/timedPetri04.pnml, line 73 at r2 (raw file):

200
Just asking: Is this interval in milliseconds or seconds?

Comments from Reviewable

Review status: 0 of 3 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Reviewed 2 of 3 files at r1, 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

test/org/lac/javapetriconcurrencymonitor/test/resources/timedPetri04.pnml, line 73 at r2 (raw file):
Previously, juanjoarce7456 (Juan Jose Arce Giacobbe) wrote…

200
Just asking: Is this interval in milliseconds or seconds?

milliseconds

Comments from Reviewable

Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from Reviewable

Reviewed 2 of 3 files at r1, 1 of 1 files at r2.
Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from Reviewable


Skip bad zip files

I think the error should be explicitly raised to remind users to repair these files.


C++14 rework

Upgrade some program elements such as replace enable:if with enable_if_t or conditional with conditinal_t. Also see if constexpr works all the way down to C++14, as it seems like most of it should.

It appears as though C++14 and C++17 constexpr-ness won't work with out (at least) writing a local version of std::array. This is beyond the scope of this issue at the moment, and we will stick with C++20 constexpr-ness.