How to get this working with TypeScript?

Anyone know how to get BitBar to run a TypeScript file (.ts), without needing to convert it to JS first? Using ts-node?
I tried the following, in test.10s.ts, with chmod +x:
#!/usr/bin/env /usr/local/bin/ts-node
console.log('Test');

But it gives the error in BitBar: env: node: No such file or directory.

Try this : #!/usr/bin/env -S node -r "ts-node/register"

Nope sorry, same error of env: node: No such file or directory in BitBar, and when run in Bash with ./test.10s.ts, I get: Error: Cannot find module '"ts-node/register"'
My ts-node is located at /usr/local/bin/ts-node.

What does which node outputs ?
Le mer. 8 avr. 2020 à 20:25, garyking<EMAIL_ADDRESS>a écrit :

Nope sorry, same error of env: node: No such file or directory in BitBar,
and when run in Bash with ./test.10s.ts, I get: Error: Cannot find module
'"ts-node/register"'
My ts-node is located at /usr/local/bin/ts-node.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/matryer/bitbar/issues/548#issuecomment-611117101, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AKAMABCF7J4QPLQLOEBQVKTRLS6RZANCNFSM4LYH3F3A
.

which node outputs /usr/local/bin/node

Does it work if you compile the ts file down to js and use node as shebang ?
Le mer. 8 avr. 2020 à 20:28, garyking<EMAIL_ADDRESS>a écrit :

which node outputs /usr/local/bin/node
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/matryer/bitbar/issues/548#issuecomment-611118690, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AKAMABAUNPGASTAVJW54EULRLS66FANCNFSM4LYH3F3A
.

Yep that definitely works. I've been doing that for a while now. I convert from TS to JS, then in the JS, it's this:
#!/usr/local/bin/node
console.log('Test');

Does running the Typescript with shebang as a shellscript works ?
Le mer. 8 avr. 2020 à 20:33, Nicolas Surleraux <
<EMAIL_ADDRESS>a écrit :

Does it work if you compile the ts file down to js and use node as shebang
?
Le mer. 8 avr. 2020 à 20:28, garyking<EMAIL_ADDRESS>a écrit :

which node outputs /usr/local/bin/node
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/matryer/bitbar/issues/548#issuecomment-611118690,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKAMABAUNPGASTAVJW54EULRLS66FANCNFSM4LYH3F3A
.

Nope, like I said, I get the error Error: Cannot find module '"ts-node/register"' when run with ./test.10s.ts.

Yes but with the shebang /usr/bin/env ts-node ?
Le mer. 8 avr. 2020 à 21:05, garyking<EMAIL_ADDRESS>a écrit :

Nope, like I said, I get the error Error: Cannot find module
'"ts-node/register"' when run with ./test.10s.ts.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/matryer/bitbar/issues/548#issuecomment-611136435, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AKAMABCWQGJFXJQGP5BZYNDRLTDGNANCNFSM4LYH3F3A
.

For test.1m.ts, the contents are:
#!/usr/bin/env ts-node
console.log('test');

When run in BitBar, the error is: env: ts-node: No such file or directory
And when run in the shell, with ./test.1m.ts, the output is: test.
When using the full path to ts-node, I get the error as mentioned in the first post of this thread.

Alternatively, consider Deno, an upcoming ts/js runtime from the creator of Node. I've been using it with BitBar and it works great!
brew install deno, then put this in your script:
#!/usr/bin/env deno
console.log('Test');

Happy to help if you have questions about Deno!
(Note: I have no affiliation with the Deno project but I think it's cool and I'm starting to use it here and there in small scripts.)

I'm only vaguely familiar with Deno. My BitBar scripts are complex; they aren't just a simple console.log, but instead import other user modules, which in turn, import Node modules. I assume I'd have to change these imports to URL imports for Deno, for instance. So unfortunately, I can't just easily switch to it.

Ah thanks for the clarification.
Looking at the original error message then:

env: node: No such file or directory.

Where is the path to node? Is it /usr/local/bin/node? Can you try making your script this instead?
#!/bin/sh

export PATH="/usr/local/bin:$PATH"
ts-node path/to/script.ts

@szhu saw it by accident, but also working with deno and bitbar =), deno rocks! I don't think it is deno related because my script works perfectly in bash, but I can't get it to display anything (only seeing the ... in the menubar forever)... what am I doing? Trying to basically display updates I receive from a websocket connection but not really knowing the inner workings of bitbar I suspect that because the script basically never finishes (as it receives constant updates) it never displays something? Any help is appreciated =), here's the code:
#!/usr/bin/env /usr/local/bin/deno run --allow-net

import {
  connectWebSocket,
  isWebSocketCloseEvent,
  isWebSocketPingEvent,
  isWebSocketPongEvent,
} from 'https://deno.land/std/ws/mod.ts';

const endpoint = Deno.args[0] || 'wss://<endpoint>/realtime';

try {
  let current = null;
  let last = null;

  const sock = await connectWebSocket(endpoint);
// subscribe to ws topic
  await sock.send('{"op":"subscribe","args":["<topic>"]}');

  const messages = async () => {
    for await (const msg of sock) {
      if (typeof msg === 'string') {
        const response = JSON.parse(msg);

        if ( Array.isArray(response.data) ) {
          // only use the last item in the data array
          current = response.data.pop().price;

          // only print output when data changes
          if ( last != current ) {
            last = current;
            console.log(`${last}`);
            // also tried combinations like this, to no avail
            console.log(`${last} | terminal=true | refresh=true`);
          }
        }

      } else if (isWebSocketPingEvent(msg)) {
        //console.log('< ping');
      } else if (isWebSocketPongEvent(msg)) {
        //console.log('< pong');
      } else if (isWebSocketCloseEvent(msg)) {
        throw new Error(`closed: code=${msg.code}, reason=${msg.reason}`);
      }
    }
  };

  await Promise.race([messages()]).catch(console.error);

  if (!sock.isClosed) {
    await sock.close(1000).catch(console.error);
  }
} catch (err) {
  throw new Error(`Could not connect to WebSocket: '${err}'`);
}

Deno.exit(0);

see anything I'm missing?

Since I might not be the only one finding this thread: I had trouble executing a JavaScript file because I got the same error as @garyking above (env: node: No such file or directory when using #!/usr/bin/env node).
The example given by @szhu with #!/usr/bin/env deno gave the same error, but also didn't work via the terminal. I then tried #!/usr/bin/env -S deno run which worked on the terminal, but xbar showed the same error again. The only solution that worked for me was:
#!/usr/local/bin/deno run
console.log("Test")

This, of course, depends on Deno being available in /usr/local/bin/ (which should always be the case when installed via Homebrew though). If there's a better solution that uses /usr/bin/env, please let me know!

Oh right. At some point in the past year, before Deno reached 1.0, the shortcut deno my-script.ts was removed, and now you have to run scripts using deno run my-script.ts. The change was recorded here: https://github.com/denoland/deno/issues/4701
So then, the solution is to have:
#!/usr/bin/env deno run
console.log("Test")

I tried it and it seems to work!
Note that having more than two args in the shebang line is not portable, it works on BSD (including macOS) but doesn't work on Linux: https://unix.stackexchange.com/questions/63979 Obviously not an issue here since xbar is a Mac app, but good to know in case you work with Linux in the future.

if you are using ESM, try this
#!/usr/bin/env -S node --loader ts-node/esm

or
#!/usr/bin/env -S node --loader ts-node/esm --experimental-specifier-resolution=node --experimental-import-meta-resolve --no-warnings

if you are using eslint-plugin-n, you need to disable this option for this file below the shebang syntax
#!/usr/bin/env -S node --loader ts-node/esm
/* eslint-disable n/shebang */

Encountered the same problem with ts-node. Fixed it by using pnpm instead to install ts-node on Termux. But this is strange. It worked earlier on Termux using node, I think.
https://github.com/linusjf/LearnJavaScript/tree/main/TypeScript
Been a while since though.. Right now, just need to see ts-node working.
https://github.com/linusjf/LearnTypeScript


datree: 1.7.3 -> 1.8.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/datree/versions.
meta.description for datree is: "CLI tool to ensure K8s manifests and Helm charts follow best practices as well as your organization’s policies"
meta.homepage for datree is: "https://datree.io/"
meta.changelog for datree is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.7.3 with grep in /nix/store/sc2jp91m2f8vn8wy9yv09pxmmjay54cc-datree-1.7.3
found 1.7.3 in filename of file in /nix/store/sc2jp91m2f8vn8wy9yv09pxmmjay54cc-datree-1.7.3

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
datree

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/sc2jp91m2f8vn8wy9yv09pxmmjay54cc-datree-1.7.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A datree https://github.com/r-ryantm/nixpkgs/archive/405f42a28b509624c3c6a90283959275ee7affdd.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/sc2jp91m2f8vn8wy9yv09pxmmjay54cc-datree-1.7.3
ls -la /nix/store/sc2jp91m2f8vn8wy9yv09pxmmjay54cc-datree-1.7.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

datree

Maintainer pings
cc @jceb for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/datreeio/datree/releases.
meta.description for datree is: "CLI tool to ensure K8s manifests and Helm charts follow best practices"
meta.homepage for datree is: "https://datree.io/"
meta.changelog for datree is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.8.0 with grep in /nix/store/fxxx49m619xkq1p3cx5pa4q31qcry0nc-datree-1.8.0
found 1.8.0 in filename of file in /nix/store/fxxx49m619xkq1p3cx5pa4q31qcry0nc-datree-1.8.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
datree

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fxxx49m619xkq1p3cx5pa4q31qcry0nc-datree-1.8.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A datree https://github.com/r-ryantm/nixpkgs/archive/57a985e4c5899d990f1f2053c2a43887f2ce15a4.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fxxx49m619xkq1p3cx5pa4q31qcry0nc-datree-1.8.0
ls -la /nix/store/fxxx49m619xkq1p3cx5pa4q31qcry0nc-datree-1.8.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

datree

Maintainer pings
cc @azahi @jceb for testing.


Documentation update for grapql and apiaudit

Parameterized connect and Read timeouts
ServiceAccountOU as a comma separated list of OUs

@nireeshT @Sbrenthughes @tabladrum @matthewbeasley Please review the documentation update and merge the same when you get some time.


Update fs2-io to 3.2.9

Updates co.fs2:fs2-io from 3.2.7 to 3.2.9.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "co.fs2", artifactId = "fs2-io" } ]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Codecov Report

Merging #359 (8e2c5d2) into master (a4a12c4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #359   +/-   ##
=======================================
  Coverage   84.61%   84.61%
=======================================
  Files          35       35
  Lines         221      221
  Branches       10       10
=======================================
  Hits          187      187
  Misses         34       34

Flag
Coverage Δ

unittests
84.61% <ø> (ø)

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

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 a4a12c4...8e2c5d2. Read the comment docs.


Update basis-size upon changing to a truncated basis

When changing to a smaller basis, variables such as basis size l does not update in system (see below). I don't know if this is outside the intended use of the system-class, but it would be useful when using a truncated basis with SINC-DVR.

What is the reasoning behind calling check_axis_lengths? https://github.com/Schoyen/quantum-systems/blob/0d8c797f469244055a5b4cfd6f4a9491343c336d/quantum_systems/system.py#L94
If it is to ensure that spin-doubling should only occur once, could a solution be a flag spin_orbitals = True/False which has to be False for change_to_spin_orbital_basis to run, while at the same time letting l be the number of 'whatever-orbitals'? Or does this put to much responsibility on the user, since the user would have to specify whenever a restricted system was being setup (assuming spin_orbitals = True was the default, it wouldn't have to be)

Yeah, it was included to make sure that a system is not spin-doubled more than once (it can be questioned how smooth it is to crash instead of give a warning...). Your solution to use a flag is in my opinion equally good. This flag can be determined based on the value of the keyword argument add_spin wherever this occurs.

Thats a good idea, then a warning can be raised whenever the add_spin-flag doesn't correspond to the spin_orbital-flag.
Should there be a way to reduce the size of the system, i.e. remove spin doubling? Is there a case where this is relevant? I can imagine it being done by manually setting the matrix-elements and system size, but should one then also be able to set the spin-orbital-flag to False.

Hmm, the reverse can be achieved by using the spin-dependent slices. Incidentally, these slices (o_a, o_b, v_a, v_b) only needs to be created in case of spin_orbital=True.


support Electron (or not?)

I'll convert barbar/riothamus to TypeScript and couldn't decide if I should push the Electron-specific edits back to Vortigern or fork Vortigern to create another boilerplate for Electron.
Any ideas?

Keep in mind that some parts of the Vortigern like the isomorphic render are not needed in an Electron project. With that in mind, I think best course is to forking and simplifying Vortigern.

As you said, we can do it in a separate repo. It would be great to have similar configurations and structure. I was also considering to prepare a simplified, only client-side rendered version of vortigern for smaller (or toy) projects.

It's settled then. I'll merge the PR I have opened here, after that I'll start working on this.


co(fn*) can also take an iterator object as an argument

I've been coming across cases where developers are passing the iterator object returned from the fn* into co, rather than passing in a fn*. Here is an test example of what I mean: https://github.com/jonahglover/co-ex
Is this an intended use case for the co function? If so, I think it should be added to the API docs.

added a PR here: https://github.com/tj/co/pull/290


Replication stopped working after changing AF instance and upgrading

My Environment

ArangoDB Version:        3.7.14
Storage Engine:             RocksDB
Deployment Mode:       Active Failover
Deployment Strategy:   Manual Start
Configuration:               1 agency, 2 single instances with enabled replication
Infrastructure:               AWS
Operating System:        Ubuntu 20.04
Total RAM in your machine:        15GB each
Disks in use:        SSD
Used Package:              Ubuntu .deb

Component, Query & Data
Affected feature:
Replication
Steps to reproduce

Change one of the single instances
Upgrade to latest version
??? (I'm not sure if this is reproducible but that's what I did)

Problem:
I had to replace one of the instances (in this case it happened to be the follower at the time) for a new one. While doing so I noticed there was a newer version (3.7.14) so I upgraded all three instances from to it from 3.7.11. I first detached the old instance that was going to retire from the AF environment, upgraded it and cloned it. Then I upgraded the Agency and the Leader, in this order. The AF cluster was running back to normal with just the Leader in place. I attached the new Follower to it - I saw it appear in the Replication section of the web UI on the Leader so it gets detected correctly but looking at it's log file it appears the replication is failing with this message HTTP 401: Unauthorized - {"error":true,"errorNum":11,"errorMessage":"not authorized to execute this request","code":401}. I disabled authentication on both instances and the Follower started replicating and synced fully with the Leader. Re-enabling authentication on the Leader triggers the same error on the Follower. Technically it's not a problem to leave it like that as the cluster is running in a closed network but I'm annoyed that it was working before and now it doesn't. I had similar issues when I first set up the AF configuration a while ago but back then cycling through disabled/enabled authentication got it working eventually (or at least that's what I did and it fixed itself so I assumed that was the cause) but now it appears that's not enough.
Expected result:
Follower to correctly replicate from the Leader with authentication enabled on both instances.

@mtzonev Can you share the configuration of the nodes?

Sorry for the delayed reply, I was away.
Below are the configurations files of each instance and the agency. I've removed the default stuff that is commented out and only left the active values. I've replaced the IPs with placeholders but I made sure the original values were correct and matching before I masked them out.
Instance 2 (Leader)
[database]
directory = /var/lib/arangodb3/single

[server]
endpoint = tcp://[::]:8529
storage-engine = auto

authentication = false

statistics = true

uid = arangodb

[javascript]
startup-directory = /usr/share/arangodb3/js
app-path = /var/lib/arangodb3-apps

[log]
level = info
file = /var/log/arangodb3/arangod.log

[cluster]
my-address = tcp://INSTANCE_2_IP:8529
my-role = SINGLE
agency-endpoint = tcp://AGENCY_IP:8531

[replication]
automatic-failover = true

Instance 3 (Follower)
[database]
directory = /var/lib/arangodb3/single

[server]
endpoint = tcp://[::]:8529
storage-engine = auto

authentication = false

statistics = true

uid = arangodb

[javascript]
startup-directory = /usr/share/arangodb3/js
app-path = /var/lib/arangodb3-apps

[log]
level = info
file = /var/log/arangodb3/arangod.log

[cluster]
my-address = tcp://INSTANCE_3_IP:8529
my-role = SINGLE
agency-endpoint = tcp://AGENCY_IP:8531

[replication]
automatic-failover = true

Agency
[database]
directory = /data/arangodb3/agency

[server]
endpoint = tcp://[::]:8531
storage-engine = auto

authentication = false

statistics = true

uid = arangodb

[javascript]
startup-directory = /usr/share/arangodb3/js
app-path = /var/lib/arangodb3-apps

[log]
level = info
file = /var/log/arangodb3/arangod.log

[cluster]
my-role = AGENT

[agency]
my-address = tcp://AGENCY_IP:8531
size = 1
supervision = true
supervision-grace-period = 15
activate = true
endpoint = tcp://AGENCY_IP:8531

Right now I've disabled authentication everywhere but as soon as I enable it on the active lead the follower starts throwing the authentication error. Both instances are identical - the follower is a clone of the leader with only the IP changed in its config.
The exact same configuration was somehow working previously with enabled authentication, just one of the instances was a different machine (INSTANCE_1 which is now winding down and no longer used but has the exact same configuration and has been replaced by INSTANCE_3).

@Simran-B Did you ever had the chance to take a look at this?

I forgot to update this. I got it working in the end. The way to correctly pass the JWT secret via the configuration file is as follows:
[server]
authentication = true
jwt-secret = YOU_SECRET_HERE

This needs to be set on all instances - both agencies and singles.
Everything else in the snippets above is correct. I still have no idea how the original replication worked without this, maybe something changed between updates.


UnicodeEncodeError when displaying DataFrame with Python 2.7

Environment

Operating System:
Python Version: 2.7.6
How did you install Qgrid: pip
Python packages:  qgrid==1.0.5, jupyterlab==0.32.1, notebook==5.3.1
Jupyter lab packages (if applicable):  jupyterlab-manager v0.35.0, qgrid v1.0.5

Description of Issue

What did you expect to happen?
I expected to view the DataFrame similar to __repr_html__.

What happened instead?
An error from qgrid.grid._update_df when it calls .map(str)

'ascii' codec can't encode characters in position 16-18: ordinal not in range(128)
Reproduction Steps
import qgrid
x = pd.DataFrame({'alias': '柏林市'}, index=[0])

display(x)
qg = qgrid.show_grid(tmp, grid_options={'editable': False})

What steps have you taken to resolve this already?
I tried setting 'editable' : False hoping that that would not call _update_df.

Thanks for the report...this seems to be working as expected for me, unless I'm missing something.  I tested on both python 2 and 3.   For both, I see the following:

Sorry, it works when '柏林市' is a string but fails with u'柏林市' . I've updated the reproduction steps to:
import qgrid
x = pd.DataFrame({'alias': u'柏林市'}, index=[0])

display(x)
qg = qgrid.show_grid(tmp, grid_options={'editable': False})

Thanks.  This is fixed in the latest beta (1.0.6b6) which you can install from pypi with pip install qgrid --pre.  I'll close this issue once the beta is officially released and available on conda-forge.


docs(courses): fix italian course page

What does this PR do?
Improve repo
For resources
Description

[X] Formatted the authors' name writed wrong
[X] Removed the university name from the parenthesis:
before * [Name](url) - F. Corno (Politecnico di Torino) after * [Name](url) - F. Corno, Politecnico di Torino
[X] Removed/changed broken link: #5470
[X] Replaced unaccepted links (urls with useless parameters...)
[X] Moved/deleted resources

Notes

https://staff.polito.it/claudio.fornaro/CorsoC/: move to books
https://riptutorial.com/it/erlang: move to books (#6153)
http://www.vitoantoniobevilacqua.it/wiki/index.php?title=Pagina_principale#Dispense_di_Algoritmi_e_Strutture_Dati_in_Java: bad resource
https://www.nodeacademy.it: blog
http://latemar.science.unitn.it/segue/index.php?&site=2008Programmazione2&section=86&action=site: it isn't a course
https://www.youtube.com/playlist?list=PL4L8OWDC99_d46jXlR02dCWzTwNISywwU): 64 videos are hidden

Checklist:

[X] Read our contributing guidelines
[X] Search for duplicates.
[X] Include author(s) and platform where appropriate.
[X] Put lists in alphabetical order, correct spacing.
[X] Add needed indications (PDF, access notes, under construction)

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Removed the university name from the parenthesis

tell me if it's okay

@mcicolella take a look. Formatting of the repo's lists is becoming more uniform to make the new search capability more useful.


Incorrect this type when casting method signature.

TypeScript Version:  2.5.0-dev.20170619
Code
class MyTable {
    foo(): MyTable {
        return new MyTable()
    }
}
type ReturnsSelf = <T>(this: T) => T;
const x = new MyTable();
const y: MyTable = (x.foo as ReturnsSelf)() // void not assignable to MyTable
// `this` is actually bound correctly at runtime, even with the brackets.

Expected behavior:
No error.
Actual behavior:
above error
I searched for this type method type assertion  and method type cast this type and didn't find anything.
Some background as to why I'm even trying to do the above:
declare class M4 {}
declare abstract class Curve {
    abstract transform(m4: M4): Curve;
}
declare abstract class XiEtaCurve extends Curve {
    transform<T extends XiEtaCurve>(this: T, m4: M4): T;
}
declare class L3 extends Curve {
    transform(m4: M4): L3;
}
declare const curve: XiEtaCurve | L3
declare const m4: M4
const res1 = curve.transform(m4) // Cannot invoke an expression whose type lacks a call
// signature. Type '(<T extends XiEtaCurve>(this: T, m4: M4) => T) | ((m4: M4) => L3)'
// has no compatible call signatures.
const res2: XiEtaCurve | L3 = (curve.transform as (<T>(this: T, m4: M4) => T))(m4)
// Type 'void' is not assignable to type 'XiEtaCurve | L3'.

maybe I'm overthinking this...

The problem you're seeing is that there is no this parameter from which to infer - in other words, you're calling a method that's been orphaned from the instance, so TypeScript assumes the type is void.
Really, you may be better off with an explicit type assertion on curve itself.

Explicit type assertion on curve is indeed what I've gone with.
I don't entirely follow what you mean by "orphaned from the instance". ([1,2,3].slice)(1,2) == [2], i.e. the array is still bound to this when calling slice. Wouldn't it make sense that in (x.foo as ReturnsSelf)(), TS would infer the type of this from x?

Ah you're right - I just filed #16673.
In any case, at the point that you have a type assertion, I don't think we have any other places where after a type assertion, the type system dives back into the original expression to get some useful information. That would be strange behavior, since the assertion tells TypeScript to override the existing type.

Well arguably the underlying behavior is strange, usually you'd expect: (whatever)() and const foo = whatever; whatever() to be equivalent. As it seems a pretty outlandish edge case, do you want to close this a Bug/Won't fix?

Automatically closing this issue for housekeeping purposes. The issue labels indicate that it is unactionable at the moment or has already been addressed.


[FP]: Multiple CVEs reported against camel-activemq

Package URl
<EMAIL_ADDRESS>CPE
cpe:2.3:a:apache:activemq:3.18.0:*:*:*:*:*:*:*, cpe:2.3:a:apache:camel:3.18.0:*:*:*:*:*:*:*
CVE
Multiple
ODC Integration
{"label"=>"CLI"}
ODC Version
7.1.1
Description
This is a tricky one I think...
Apache Camel is an open source integration framework that empowers you to quickly and easily integrate various systems consuming or producing data.

CVE-2016-3088 applies to ActiveMQ (server)
CVE-2014-3576 applies to ActiveMQ (server)
CVE-2018-11775 applies to the ActiveMQ client (org.apache.activemq activemq-client) but camel-activemq doesn't seem to be affected
CVE-2010-1244 applies to ActiveMQ (server)
CVE-2013-3060 applies to ActiveMQ (server)
CVE-2020-13947 applies to ActiveMQ (server)
CVE-2020-13920 applies to ActiveMQ (server)
CVE-2012-5784 applies to the ActiveMQ client (perhaps activemq-client) but camel-activemq doesn't seem to be affected
CVE-2011-4905 applies to ActiveMQ (server)
CVE-2012-6551 applies to ActiveMQ (server)
CVE-2015-7559 applies to the ActiveMQ client but camel-activemq doesn't seem affected
CVE-2012-6092 applies to ActiveMQ (server)
CVE-2013-1879 applies to ActiveMQ (server)
CVE-2013-1880 applies to ActiveMQ (server)
CVE-2010-0684 applies to ActiveMQ (server)

The dependency I include is<EMAIL_ADDRESS>but I think all the above CVEs, when they refer to client side vulnerabilities, they apply to activemq-camel/client which from what I can see are still transitive dependencies but the version that is imported is 5.17.1 which should not be vulnerable according to the CPEs for each CVE.
So it looks there are multiple issues here at play:

Both server and client CVEs are matched against the client
CVEs that could apply to activemq-camel/client (which is on version 5.x.x) are flagged for camel-activemq (which is on version 3.x.x)
If it is flagging these CVEs against the transitive dependency is matching those CVEs against the wrong version (e.g. I have 5.17.1 installed and CVE is for up to 5.6.0)

It's not as tricky as you think, dependency-check is well prepared to handle the case of activemq being a transtive dependency of a library falsely identified as being part of activemq.
The suppression as suggested (suppression of the apache activemq CPE on this library) is fully appropriate. The transitive dependencies towards activemq libraries will be found and assessed by dependency-check on their own merits (using the proper version of activemq (5.17.1) to evaluate against the vulnerable version range). If any CVEs still apply to ActiveMQ 5.17.1 they will be reported (dependency-check does distinguish between the client- and server-side of activeMQ, as there is only a single CPE for both within the NIST NVD, so server-side issues can be expected to be reported when only using the client-side libraries)

Hi @aikebah, thanks for the information.
Does this mean this can/should be put into a release?
If so, when do you think this could be done?
Thanks

@mpreziuso yes, thanks for pinging me on this. Will try to get it in our hosted suppressions file.
approved


Can't get install pages except for .apk and .ipa files

Summery

I would deploy mapping.txt file to bitrise with using this step.
I could get public install page for my .apk files, however I couldn't get it for my mapping.txt
I attach log below.
I read source code and found the reason why.

Reason

When this step upload files except for .apk and .ipa files, it call DeployFile.
In DeployFile, finishArtifact will call with "no" argument.
finishArtifact function always checks if isEnablePublicPage is "true", but now isEnablePublicPage parameter is "no". So, finishArtifact returns "", nil even if artifactResponse.PublicInstallPageURL is not empty.

Question

Why finishArtifact's 7th argument in DeployFile is always "no" ?
If it is mistake, I want to fix it.

Hi @BirmacherAkos
Thanks replying rapidly.
Okey, I understood we can't get download pages for files except fir .apk and .ipa files.
I'll try your recommend way to get it.
And, I hope my request will accept soon!


Voxelization

Hey there,
I wanted to know why was this line commented out
https://github.com/mit-han-lab/bevfusion/blob/db75150717a9462cb60241e36ba28d65f6908607/mmdet3d/ops/voxel/voxelize.py#L119
the [::-1] part
when it's not in the official documentation
https://mmcv.readthedocs.io/en/latest/_modules/mmcv/ops/voxelize.html

This is because we are using a different coordinate system than mmdetection3d.


status_server: add glibc, pthread, libgcc to the blocklist

Signed-off-by: YangKeao<EMAIL_ADDRESS>What problem does this PR solve?
Problem Summary:
If the pprof-rs sample arrives and the current program is fetching a backtrace, the program will panic. The reason is that there is a lock inside the dl_iterate_phdr, which is called while unwinding the backtrace.
A more detailed description about this problem can be found in https://github.com/tikv/pprof-rs/pull/85#issuecomment-954446008 . In tikv, when an Error is generated, it's likely to get a backtrace and cause this problem.
After adding these libraries to the blocklist, it will also be able to turn on the heap profiling and pprof-rs cpu profiling at the same time, but I haven't modified (and tested) it yet.
What is changed and how it works?
What's Changed:
Skip all samples inside the glibc, pthread and libgcc.
Related changes

PR to update pingcap/docs/pingcap/docs-cn:
PR to update pingcap/tidb-ansible:
Need to cherry-pick to the release branch

Check List
Side effects
The sample in the libc, libgcc and pthread will not be collected in the profiling result.
Release note
None

/cc @BusyJay
/cc @crazycs520
/cc @sticnarf

Have you confirmed profiling works after blocking these symbols?

Have you confirmed profiling works after blocking these symbols?

Yes. It actually works.

@sticnarf PTAL

/merge

/merge

/merge

/merge


Updating services hangs: (Waiting: container starting [container:1i267])

Rancher versions:
rancher/server: v1.6.6
rancher/agent: v1.2.5
Infrastructure Stack versions:
healthcheck: v0.3.1
ipsec: v0.11.7
network-services: v0.7.7
scheduler: v0.8.2
Docker version: (docker version,docker info preferred)
17.03.1-ce
Operating system and kernel: (cat /etc/os-release, uname -r preferred)
Ubuntu 16
Environment Template: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:
Updating a container with Gitlab w/ cdrx/rancher-gitlab-deploy
Results:
Updating fails because the service hangs on updating. Deleting the service and updating keeps producing the same error. Really need some help!
 This did not help. Still having the same problem.

@DirkWolthuis sorry for the silence here... Some question, is this still happening? If you launch the container manually on these hosts, do they go to running?

some sanitized docker/rancher compose files could also help.

Hi! I've deleted the host and re-added it. The problems stayed. Strangely the same admin has a other host with the same specs that hasn't had the same problems. Updating the admin did not resolve the problems with the not working host. I've given up since and waiting for rancher 2.0. This was my own server and I hadn't have time for any personal projects so I have no time to test it again.

Please let us know in a comment if you can still reproduce the issue, and we'll reopen it


[BUG]: drizzle-kit cannot handle pgSchema

What version of drizzle-orm are you using?
0.28.5
What version of drizzle-kit are you using?
0.19.13
Describe the Bug
If I use pgSchema for my tables, drizzle-kit push:pg, studio, etc does not work anymore.
export const users = pgTable('users', {
  id: text('id').notNull().primaryKey(),
  name: text('name'),
  email: text('email').notNull(),
  emailVerified: timestamp('email_verified', { mode: 'date' }),
  image: text('image'),
});

is working fine, I changed it to:
export const $schema = pgSchema('app');

export const users = $schema.table('users', {
  id: text('id').notNull().primaryKey(),
  name: text('name'),
  email: text('email').notNull(),
  emailVerified: timestamp('email_verified', { mode: 'date' }),
  image: text('image'),
});

Then, the pg:push command initially prints the message [✓] Changes applied, but actually does nothing. Afterwards, it always prints [i] No changes detected regardless of schema changes.
Also, drizzle studio doesn't work properly, with this error on requests:
error: relation "app.users" does not exist
    at Parser.parseErrorMessage (.../node_modules/drizzle-kit/index.cjs:40722:98)
    at Parser.handlePacket (...n/node_modules/drizzle-kit/index.cjs:40563:25)
    at Parser.parse (.../node_modules/drizzle-kit/index.cjs:40487:34)
    at Socket.<anonymous> (.../node_modules/drizzle-kit/index.cjs:40763:44)
    at Socket.emit (node:events:513:28)
    at addChunk (node:internal/streams/readable:324:12)
    at readableAddChunk (node:internal/streams/readable:297:9)
    at Readable.push (node:internal/streams/readable:234:10)
    at TCP.onStreamRead (node:internal/stream_base_commons:190:23) {
  length: 108,
  severity: 'ERROR',
  code: '42P01',
  detail: undefined,
  hint: undefined,
  position: '22',
  internalPosition: undefined,
  internalQuery: undefined,
  where: undefined,
  schema: undefined,
  table: undefined,
  column: undefined,
  dataType: undefined,
  constraint: undefined,
  file: 'parse_relation.c',
  line: '1371',
  routine: 'parserOpenTable'
}

Expected behavior
pg:push and studio should work well with pgSchema
Environment & setup
No response

Any update on this ?.

Same here. I have to run migrations by hand.

I can no longer run studio after switching my schema file to use pgSchema instead of pgTable - is there any other workaround besides reverting back to pgTable?

Just ran into the issue, after carefully looking at the docs, it looks like this could be related to the fact that drizzle kit doesn't handle multiple schemas where the tables have the same name.
https://orm.drizzle.team/kit-docs/faq

Related to https://github.com/drizzle-team/drizzle-orm/issues/1773
You have to add schemaFilter to drizzle.config file and include your schemas there. It will fix the issue with No changes detected. But issue with drizzle studio still persists

just delete all the migration files and run the migration again, for me this solved it

I noticed drizzle studio will not launch if there is not at least one table existing in the public schema. You will see a pgTable error in the browser debug console. I have multiple schemas as well and I wasn't able to get studio to launch until I just added a single table to my public database schema. Then it worked and studio displayed my other schemas as well.

schemaFilter does fix the original problem reported here but is still somewhat annoying. Would be much nicer if the schema(s) were inferred from the user of pgSchema and push just worked as expected.
Much larger problem though is the studio bug here though. Postgres by default makes it so only DB Owner can create anything on public and even goes so far as to mention no one should use the public schema in their docs (see Portability section). As such I have created my entire app under a schema I have created which just completely breaks Drizzle Studio to the point that the entire page fails to load.

Yeah, definitely an annoying issue that we have to use public to get drizzle studio to work. Since I'm just using drizzle studio on my local machine, I have the following migration setup so that it will only create a table in public on my local database, just so drizzle studio will work and show me my other schemas. The following user account doadmin exists in my deployed/production digital ocean database, so when deployed it just ignores it and only creates it for my local machine/docker database. Kind of a hacky workaround, but it works for the meantime.
000_drizzle_studio_migration.sql
SET search_path TO public;

-- Extension Creations

DROP EXTENSION IF EXISTS "uuid-ossp";
CREATE EXTENSION "uuid-ossp" SCHEMA public;

-- Only create public.test table on local machine, if rolname 'doadmin' exists that means it's not local
DO
$do$
BEGIN
IF NOT EXISTS (SELECT FROM pg_catalog.pg_roles WHERE rolname = 'doadmin') THEN
  -- Create a test table just to have a table exist in public schema

  CREATE TABLE "test" (
    "id" uuid PRIMARY KEY DEFAULT public.uuid_generate_v4()
  );

END IF;
END
$do$;

I ran into this issue, when trying to build multi-tenants with same table structures in different schemas and drizzle studio stops working and throws weird init, cors and ssl erros. After hours of headScratching, I found that the problem pops up when you have some references in your tables.
What we expect is that table will create those references in the same schema but it connects to the public schema and tries to connect to tables in the public schema. Probably, we are writing the code wrongly. We might need to fix the schema references.
To fix the original issue, you need to add schemaFilter to push the changes.
For drizzle studio, it will work fine , if your tables in schemas don't have any references.
I'm still figuring out drizzle, so will keep updated if I found the solution to build a multi-tenant multi schema app.

We should not have this issue now on latest. Please create another ticket if the issue still persists. I'll close this ticket as long as the original issue was fixed.

Working with mysql db. I am too facing such issues. schemaFilter is only for Postgres. Any workaround for MySQL DB. I am using latest drizzle version.

I'm currently using the t3-turbo monorepo with drizzle-orm(0.31.2)  & drizzle-kit(0.22.8).
To handle multiple schemas you must to define in the schemaFilter those schemas that you need. The 'public' schema is generated automaticly after you create a database, you don't need to use it if you don't want it, but it is also required by drizzle to work.
import type { Config } from "drizzle-kit";

if (!process.env.POSTGRES_URL) {
  throw new Error("Missing POSTGRES_URL");
}

const nonPoolingUrl = process.env.POSTGRES_URL;

export default {
  schema: ["./src/schemas/public/schema.ts", "./src/schemas/auth/schema.ts"],
  dialect: "postgresql",
  verbose: true,
  schemaFilter: ["auth"],
  dbCredentials: { url: nonPoolingUrl },
} satisfies Config;


Update navbar.elm html example

Hi there,
I've noticed that the example for the view refers to state.basicState but should be model.navbarState.
Thanks!

Cheers! From the looks of it, the custom example is also incorrect. Would you mind adding that fix to this pr too ?


Added Chrome 47 WebRTC update

R @jpmedley

@samdutton PTAL

@jpmedley PTAL

@jpmedley PTAL.

@jpmedley ping :)!

@jpmedley are you happy to merge this?

This is nearly four weeks old, and a bit out of date now :/.
If we can get an LGTM, let's publish and then I'll update it.

Id update it, bump up the publish date so it's on the top of the updates list tbh.

I would agree with Matt's take on this one.

I've updated this — now also includes codelab updates, which I'd already started committing a couple of weeks ago. Not ideal that these aren't separate, but both need merging as soon as possible.
@jpmedley PTAL


Option: keep cached options

Please make sure these boxes are checked before submitting your PR, thank you!

[x] Make sure you follow Element's contributing guide (中文 | English | Español | Français).
[x] Make sure you are merging your commits to dev branch.
[x] Add some descriptions and refer relative issues for you PR.

While working with select component with remote, filterable, deleting and allow-create the cachedOptions causes issues when showing the selected value.
If we remove an element from selected values on the change event, the options will be empty and the select won't have the cached options to take the label from, so the selected values will display a wrong label.
There is no harm on keeping the cache.

Duplicate of #17396

Perfect, thanks


Fix zero stage 1 norm computation - WIP

As per our Discord discussion:
Right now, in ZeRO stage 1, each GPU will compute a grad norm across all of its shards - but only a single shard has the correct numbers (after NCCL's reduce-scatter).
So what we should do instead is compute norm squared per shard that the device owns and then reduce (sum) them up across devices, take a sqrt and only then call AdamW kernel to update the params.

WIP - need to convince myself this is correct and run tests :)


router.post() is not working

when router.post() is used
const { Router } = require("express");
const express = require("express");
const router = express.Router();
// controller
const { createOrUpdateUser } = require("../controllers/auth");
router.post("/create-or-update-user",
createOrUpdateUser);
module.exports = router;

when router.get() is used
const { Router } = require("express");
const express = require("express");
const router = express.Router();
// controller
const { createOrUpdateUser } = require("../controllers/auth");
router.get("/create-or-update-user",
createOrUpdateUser);
module.exports = router;

Hey, you are trying this from the browser and all the requests from the browser are Get Requests and not Post. This is also clearly indicated in the error as well 'Cannot GET ....'. You need either Postman/similar tool or browser APIs to make the POST request.

@nikitaghule98, please use correct HTTP method for testing your endpoints.
Closing as this is not express-related issue.


[Question]: Customizing RancherOS builds

Just a quick question. I know this isn't documented yet...
What is involved in customizing the RancherOS build(s)? Specifically I'm looking at wanting to customize the AWS AMI build for our own deployments and take advantage of the /var/lib/docker/preload I read about in the documentation for pre-packing docker images

I haven't run it, but have you looked at usr/sbin/rancheros-install -h (run from within rancher)? I learned about it from rancher/os-installer.

Yeah I haven't tried this tool yet from whin a live rancherOS instance; however I'm looking for more of a way to customize the prebuilt images; specifically the AMI()s).

@prologic if you want to build your own custom AMI, it's absolutely possible to use image preloading. Just use rancher/os:v0.4.0-dev container to install RancherOS files to a fresh EBS volume attached to any Linux instance with Docker 1.8.2. You can consult https://github.com/rancher/os/blob/master/scripts/installer/README.md#usage for instructions.
Basically, you just need to run the container with the
-f /source/path/archive.tar.gz:/var/lib/docker/preload/archive.tar.gz. Be sure to also provide -t amazon-ebs.

Thanks @imikushin (/cc @fud)

@prologic If you have more questions, please re-open.


Connect to emulator?

Hi,
Is it possible to use this tool to connect to the firestore emulator?
I have a docker compose which includes firestore emulator.
  firestore:
    image: ridedott/firestore-emulator:latest
    ports:
      - "8690:8080"

I tried setting FIRESTORE_EMULATOR_HOST env variable but I don't think that worked.
~/optitext-link-tracking  ⎇ 193453-opt-out *  set FIRESTORE_EMULATOR_HOST optitext-link-tracking_firestore_1:8080
~/optitext-link-tracking  ⎇ 193453-opt-out *  echo $FIRESTORE_EMULATOR_HOST         Fri  1 Sep 11:13:50 2023
optitext-link-tracking_firestore_1:8080
~/optitext-link-tracking  ⎇ 193453-opt-out *  firestore-cli ui -p "ff"              Fri  1 Sep 11:13:54 2023
2023/09/01 11:14:01 rpc error: code = PermissionDenied desc = Cloud Firestore API has not been used in project ff before or it is disabled. Enable it by visiting https://console.developers.google.com/apis/api/firestore.googleapis.com/overview?project=ff then retry. If you enabled this API recently, wait a few minutes for the action to propagate to our systems and retry.
error details: name = ErrorInfo reason = SERVICE_DISABLED domain = googleapis.com metadata = map[consumer:projects/ff service:firestore.googleapis.com]
error details: name = Help desc = Google developers console API activation url = https://console.developers.google.com/apis/api/firestore.googleapis.com/overview?project=ff

Actually I think firestore is exposed locally as 8690 not 8080.
But setting FIRESTORE_EMULATOR_HOST to <IP_ADDRESS>:8690 does not help:
 !  ~/optitext-link-tracking  ⎇ 193453-opt-out *  nc -vz <IP_ADDRESS> 8690
Connection to <IP_ADDRESS> port 8690 [tcp/*] succeeded!
 ~/optitext-link-tracking  ⎇ 193453-opt-out *  set FIRESTORE_EMULATOR_HOST <IP_ADDRESS>:8690
 ~/optitext-link-tracking  ⎇ 193453-opt-out *  firestore-cli ls -p "ffff"
2023/09/01 11:36:18 rpc error: code = PermissionDenied desc = Cloud Firestore API has not been used in project ffff before or it is disabled. Enable it by visiting https://console.developers.google.com/apis/api/firestore.googleapis.com/overview?project=ffff then retry. If you enabled this API recently, wait a few minutes for the action to propagate to our systems and retry.
error details: name = ErrorInfo reason = SERVICE_DISABLED domain = googleapis.com metadata = map[consumer:projects/ffff service:firestore.googleapis.com]
error details: name = Help desc = Google developers console API activation url = https://console.developers.google.com/apis/api/firestore.googleapis.com/overview?project=ffff

@thomas-optimove Sorry for the late response, completly missed it.
Have added a docker compose file to the project and have the cli working with emulator host set FIRESTORE_EMULATOR_HOST=localhost:8200 ./firestore-cli --project dummy-firestore-id ls
Don't know what port your emulator is running on, but you can see an example in the docker compose file.


narrow: Window title shows user's name, in private message narrows.

Modified by punchagan to correctly handle narrowing to huddles, and to fix a
couple of tests.
Closes #5876
Supersedes #8532

@rishig what are your thoughts on what we want the window titles to say?  I think it's clear the existing thing isn't right, but I'm not sure if this is the right change to address this.

good question. I think
Tim Abbott - (for PM with Tim)
Tim Abbott, Rishi Gupta - (for group PM with Tim and Rishi)
Private messages - (for the /is/private narrow)
would be in line with how we handle stream and topic narrows.

Thanks @rishig! I've updated the PR to change the titles as requested.
I'm still not sure what the group-pm-with narrows should be title. They are currently just titled as private group.
Does Group messages with Tim Abbott work?
Also, if I understood the code correctly, we do support adding multiple email addresses when searching for group-pm-with, right? @timabbott ?

How about
Tim Abbott and others -
for the group-pm-with case.

How about
Tim Abbott and others -
for the group-pm-with case.

That seems reasonable.
@timabbott I think this PR is now ready for a review. Please take a look when you have some time. Thanks!

This is great, merged, thanks @punchagan!

Thanks for merging!
Sorry, I didn't have the time to respond to your review comments from earlier. I was going to look into it in a couple of days. :see_no_evil:
@timabbott Did you test the case with invalid email addresses?

Oh, no I didn't; that's probably broken (somehow I didn't see that comment when looking at this again; maybe a GitHub caching thing since it looked like the last reply was your message above...)  Can you investigate?

Yeah, I'll take a look!

@timabbott I've a fix for this in https://github.com/zulip/zulip/pull/12665


Project Path Capitalization

What version of mystjs are you using?
0.1.24
How did you install myst?
npm
What operating system are you using?
Mac
Which area is this feature request for?
Command Line Interface
Describe the Bug
When the local path is capitalized, and the slug is not, there is a mismatch in the folder that is generated and uploaded to a CDN. This is generally fine on unix machines but is not fine in, for example, a storage bucket. Regardless, there is a mismatch somewhere.
For example works locally but doesn't deploy properly to a CDN indicating that the case is incorrect.
Expected Behavior
No response
To Reproduce
No response

Capitalizing slug seems to have done the trick https://github.com/alanlujan91/SequentialEGM/blob/751eadcb22b850d09fbfdaf23e69d76374eedeee/myst.yml#L8

my setup
curvenote v0.8.7
mystjs v0.1.20


add gtag for data managemet

so ive added to the home and projects page a gtag so that we can conversion metrics our google ads.
ive also added some pages in the /extras folder, merging this and viewing the directory on the site will provied insite to what ive been working on for the past week and a half
i'm still workingg on the login page anbd shit so give me some time before it works fully

why make it for mobile when the content probs wont scale correctly?
ive emulated the admin page and for some reason it doesnt work for mobile
in terms of looking good
On Mon, 18 Sept 2023 at 14:46, sourena kazemi @.***>
wrote:

@.**** commented on this pull request.
there should also be another link in the mobile-sidebar component for
login page link in mobile
also why is it _blank...make it like others
—
Reply to this email directly, view it on GitHub
https://github.com/OpenStudioCorp/NewOpenStudioCorpSite/pull/30#pullrequestreview-1630275438,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4EUQ4LJJ57I3WD4FY6M4ITX27G4DANCNFSM6AAAAAA44D6J44
.
You are receiving this because you authored the thread.Message ID:
@.***
com>


[BUG] Test of local channel backup fails

Describe the bug
The local channel backup fails.
Location in guide
Link to the corresponding page and section on https://raspibolt.org
Expected behavior
In the logs of scb-backup, according to the guide, I should see:
Success! The file is now remotely backed up!

Screenshots
Instead, these are the logs:
May 24 18:27:18 raspberrypi systemd[1]: Started SCB Backup daemon.
May 24 18:27:18 raspberrypi scb-backup[1542]: Setting up watches.
May 24 18:27:18 raspberrypi scb-backup[1542]: Watches established.
May 24 18:36:46 raspberrypi scb-backup[1542]: /data/lnd/data/chain/bitcoin/mainnet/channel.backup OPEN
May 24 18:36:46 raspberrypi scb-backup[1541]: channel.backup has been changed!
May 24 18:36:46 raspberrypi scb-backup[1541]: Local backup is enabled
May 24 18:36:46 raspberrypi scb-backup[1541]: Copying backup file to local storage device...
May 24 18:36:46 raspberrypi scb-backup[1541]: /mnt/static-channel-backup-external/channel-20220524-183646.backup
May 24 18:36:46 raspberrypi scb-backup[1604]: cp: cannot create regular file '/mnt/static-channel-backup-external/channel-20220524-183646.backup': Permission denied
May 24 18:36:46 raspberrypi systemd[1]: scb-backup.service: Main process exited, code=exited, status=1/FAILURE
May 24 18:36:46 raspberrypi systemd[1]: scb-backup.service: Failed with result 'exit-code'.

Environment

Hardware platform: Raspberry Pi 4
Operating system: Raspberry Pi OS 64bit
Version: Debian GNU/Linux 11 (bullseye)

It looks like you don´t have permissions on the local folder (/mnt/...).
Make sure you set the right GID and UID for the user lnd as described here: https://raspibolt.org/guide/lightning/channel-backup.html#option-1-local-backup


Post release automated changes for entra releases

Post release automated changes for azure-functions-authentication-events

/check-enforcer override


Gitea dump error "Failed to include data directory: open /data/ssh: permission denied"

Gitea version (or commit ref): Gitea version 1.13.2 built with GNU Make 4.3, go1.15.7 : bindata, timetzdata, sqlite, sqlite_unlock_notify
Git version: git version 2.26.2
Operating system: Alpine 3.12.3

Gitea running in Kubernetes cluster
Gitea installed using helm chart from https://gitea.com/gitea/helm-chart/
Database (use [x]):

[X ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No

Log gist: https://gist.github.com/senthilrch/30c2e949e998f909fdeefd5a0bd42455#file-gitea-dump-log

Description

I am running Gitea in a Kubernetes cluster. Installed it via helm chart from https://gitea.com/gitea/helm-chart/. I am following the document in https://docs.gitea.io/en-us/backup-and-restore/ to take a backup of gitea using "gitea dump" command. Steps I followed:-

kubectl exec -it  -- bash
su git
cd /tmp
gitea dump -c /data/gitea/conf/app.ini

Receiving following error (complete log in https://gist.github.com/senthilrch/30c2e949e998f909fdeefd5a0bd42455#file-gitea-dump-log):-
...
...
...
2021/03/18 09:18:18 ...m.io/xorm/core/db.go:286:afterProcess() [I] [SQL] SELECT "id", "issue_id", "project_id", "project_board_id" FROM "project_issue" [] - 663.219µs
2021/03/18 09:18:18 ...m.io/xorm/core/db.go:286:afterProcess() [I] [SQL] SELECT "id", "name", "repo_count", "created_unix", "updated_unix" FROM "topic" [] - 603.999µs
2021/03/18 09:18:18 ...m.io/xorm/core/db.go:286:afterProcess() [I] [SQL] SELECT "repo_id", "topic_id" FROM "repo_topic" [] - 390.503µs
2021/03/18 09:18:18 ...m.io/xorm/core/db.go:286:afterProcess() [I] [SQL] SELECT "id", "version" FROM "version" [] - 339.919µs
2021/03/18 09:18:18 cmd/dump.go:290:runDump() [I] Adding custom configuration file from /data/gitea/conf/app.ini
2021/03/18 09:18:18 cmd/dump.go:303:runDump() [I] Custom dir /data/gitea is inside data dir /data, skipped
2021/03/18 09:18:18 cmd/dump.go:310:runDump() [I] Packing data directory.../data
Failed to include data directory: open /data/ssh: permission denied
2021/03/18 09:18:18 cmd/dump.go:171:fatal() [F] Failed to include data directory: open /data/ssh: permission denied
bash-5.0$ ls -l /data/ssh
ls: can't open '/data/ssh': Permission denied
total 0
bash-5.0$ exit
exit
bash-5.0# ls -l /data/ssh
total 32
-rw-------    1 root     root          1381 Mar 16 12:38 ssh_host_dsa_key
-rw-------    1 root     root           602 Mar 16 12:38 ssh_host_dsa_key.pub
-rw-------    1 root     root           505 Mar 16 12:38 ssh_host_ecdsa_key
-rw-------    1 root     root           174 Mar 16 12:38 ssh_host_ecdsa_key.pub
-rw-------    1 root     root           399 Mar 16 12:38 ssh_host_ed25519_key
-rw-------    1 root     root            94 Mar 16 12:38 ssh_host_ed25519_key.pub
-rw-------    1 root     root          1823 Mar 16 12:38 ssh_host_rsa_key
-rw-------    1 root     root           394 Mar 16 12:38 ssh_host_rsa_key.pub

Screenshots

This is related to the helm-chart configuration, in the future please post there.
Tracking this now at https://github.com/go-gitea/gitea/issues/15024


[WIP] Project: Project View and Actions

[x]  Project view and Project dashboard
[x]  Projects: delete/list project accounts etc.
[x]  Project invitations: list/delete/updat
[x] Project resource tab in the project detail view

@rhtyd @svenvogel
This is the design of the Project View and Actions. Please let me know your comments.

Project dashboard

Projects account:

Project invitaion:

@utchoang couple of issues: cc @svenvogel

account tab not show after I create a project, and see the project's detailview
how do I add/update project invitation by token (updateProjectInvitation? the old UI allow entering project ID and token)
in the project dashboard's account tab, the account list does not update after adding a new account
Can you use this.$parent etc. to find and add the badge to the project invitation, see if it's feasible to avoid changing the AutogenView too much without requiring customisation in the general purpose component?

Can you address those, I'll do a thorough review later tomorrow.

@utchoang cc @rhtyd Rohit wrote the technical things from me only a design suggestion...
we should a add more filter and an search bar

i am also thought about tags but they are not so useful i think

@utchoang Along with the raised issues that you'll address, kindly also add the missing resources tab in the project detail view that allows the admin/owner to configure the limits of resources allowed for that project:

@rhtyd cc @svenvogel

how do I add/update project invitation by token (updateProjectInvitation? the old UI allow entering project ID and token)

In the old UI that I got, I cannot find the option to add token when accepting or declining an invitation. Could you please send me the UI where the token could be added?

@utchoang this is on 4.13 version:
In the project invitation list view the Enter token button is seen here:

Clicking it opens this dialog:

Upon entering the details, the following API call is made:
command: updateProjectInvitation
response: json
projectid: some-uuid
token: some-token
_:<PHONE_NUMBER>606

I did another test, the resources tab to set the limits is only shown to admin @utchoang

@utchoang fyi - I've merged latest master (was doing some tests), will continue the review/test once you can confirm the fixes. Thanks.

@utchoang update - I've fixed permissions issues now, that should fix the issue of when to show the tabs or the action button, and added feature discovery which can be used to determine the: cc @svenvogel
    "allowusercreateprojects": true,
...
    "projectinviterequired": true,

Kindly rebase the branch.

@rhtyd cc @svenvogel : I've completed the fixes:

In the popup to add/update project invitation by token, the Autogenview is set to get the field type of projectid as listbox, thus, I've changed the project type to input, not listbox.

@svenvogel https://github.com/apache/cloudstack-primate/pull/73#issuecomment-567068116
I think there are some problems for this issue as below

The parameter keyword of API listProjectInvitations seems not to work. I did try to add/change keyword but the search results did not change accordingly
Additional filter: do you mean you'd like to have multiple filters for column project and domain? If multiple filters are used, the pagination would not work correctly. The reason is the API does not support search project list and domain list. Please check and let me know your idea.

Alright I'll review this in a while @utchoang - for the mapping, the allowed type are limited (I'll document that later on). To get automatic listing of projects, the projectid mapping shouldn't be defined.

@utchoang quick review comments:

The resource tab under projects view does not work where input buttons are not seen, however it works via dashboard view:

The badge on action does not work or show pending invitations
User dashboard (when not in a project, but default view) is broken

@rhtyd Sorry for this problem. When I merged from the master, it was lost.

@utchoang sure no problem, I'll review and get back to this PR shortly

@utchoang can you perform an end-to-end testing of all the features/cases. I found a few you can try to fix:

Non-admin user's dashboard no longer shows the cards:

When owner of a project switches to a project, the account tab is not visible: (this is a non-admin user account)

The pending invitation badge is not shown? (perhaps a merge issue)

I've merged latest master, kindly rebase and continue testing/fixing. Thanks.

@rhtyd

I have fixed for https://github.com/apache/cloudstack-primate/pull/73#issuecomment-567812849
for https://github.com/apache/cloudstack-primate/pull/73#issuecomment-567788261, I understand and I make change by importing another component

Overall LGTM, except for few edge case issues (like refresh inside project resource view fails when account tab is opened). It needs a bit of refactoring, and allowing projectid to be any kind of text rather than a select dropdown only. Good effort @utchoang, if you've time please work on the raised issues, otherwise try and handle those next week.

Additional filter: do you mean you'd like to have multiple filters for column project and domain? If multiple filters are used, the pagination would not work correctly. The reason is the API does not support search project list and domain list. Please check and let me know your idea.

@rhtyd you are right. hmm okay let me think about that. can we use "tags" i mean add a tag "domain with...." and a "tag with project name ...."?
the search bar should not be a problem. right?

Overall LGTM, except for few edge case issues (like refresh inside project resource view fails when account tab is opened). It needs a bit of refactoring, and allowing projectid to be any kind of text rather than a select dropdown only. Good effort @utchoang, if you've time please work on the raised issues, otherwise try and handle those next week.

@rhtyd Could you please explain the last idea about projectid a little more? The field for projectid could be either text input or dropdown selection. Do you mean to allow input search by projectid?

Hi @utchoang ignore my last comment around the projectid/input type. But follow my last change (around handling changing resource to handle fetchData/refreshing the view). Can you perform an end-to-end functional testing of the feature once? I've made some changes and I think it's ready for merging after a final round of testing and review. (I'm doing code re-review, but LGTM from my side on functionality).

Hi @rhtyd The issues mentioned in PR are fixed and the refactoring is also done. Functional testing is now completed. Please check.
Hi @svenvogel

https://github.com/apache/cloudstack-primate/pull/73#issuecomment-568190912 Regarding to the tags as in your idea, I have the below suggestion

The search bar: The parameter keyword of API listProjectInvitations seems not to work. I did try to add/change keyword but the search results did not change accordingly

Additional filter: do you mean you'd like to have multiple filters for column project and domain? If multiple filters are used, the pagination would not work correctly. The reason is the API does not support search project list and domain list. Please check and let me know your idea.

@utchoang - @rhtyd wrote this above. i dont know if this works?
good suggestion.there are any questions for me and maybe we can change it like this.

does this tags working?
if we use tags and they are working should we remove the filter from state and use only tags for "domain" and "filter". what i understood from @rhtyd was if the api dont give us a filter element it will not work. @rhtyd is this true?
i think tags and domain will be the same and maybe we should not do that. i thought like this.
https://ant.design/components/select/

can we use it as one element "filter" with tags and "search" bar? is this possible or do we need to get it from the api?

Additional filter: do you mean you'd like to have multiple filters for column project and domain? If multiple filters are used, the pagination would not work correctly. The reason is the API does not support search project list and domain list. Please check and let me know your idea.

@utchoang - @rhtyd wrote this above. i dont know if this works?
good suggestion.there are any questions for me and maybe we can change it like this.

does this tags working?
if we use tags and they are working should we remove the filter from state and use only tags for "domain" and "filter". what i understood from @rhtyd was if the api dont give us a filter element it will not work. @rhtyd is this true?
i think tags and domain will be the same and maybe we should not do that. i thought like this.
https://ant.design/components/select/

can we use it as one element "filter" with tags and "search" bar? domain and state together? @rhtyd  is this possible or do we need to get it from the api?
@svenvogel cc @rhtyd

As what I found from the source code http://cloudstack.apache.org/api/apidocs-4.13/apis/listProjectInvitations.html, that the listProjectInvitations requests "keyword" parameter seems not to work. I tried with different value and it always returns the same result.

If requesting "keyword" parameter works, filter based on selecting tags won't be needed as we can search filters by entering domain name or project name.

@utchoang it's possible that some list APIs don't honour or implement the keywords based searching, in which case you may file a bug here: https://github.com/apache/cloudstack/issues/new
Agree on the tags comment cc @svenvogel - maybe we can revisit this once the searching is supported. I think I can live with the current implementation and iterate in future.
I've started reviewing/testing it now, hopefully will merge in next few hours.

@rhtyd @utchoang okay. lets leave it without tags. if its implemented in future we can add this. lets move to the next one.

@utchoang LGTM, can you address the following issues in this PR and other PRs:

Whenever any action/button needs to be added in a custom action/form/tab, always check if the user has access to execute the API using apiName in this.$store.getters.apis or similar
Deleting a user/account from a project, or making another account the project admin does not cause refreshing of the accounts tab

Without blocking this further I'll merge this, but please send another PR to fix issues in this PR @utchoang  Thanks.


fix(payment): BOLT-60 fixed an issue with checkout step header with large font-size

What?
Fixed an issue with checkout step header with large font-size
Why?
Because of the task:
https://jira.bigcommerce.com/browse/BOLT-60
And because of the issue:
https://github.com/bigcommerce/checkout-js/issues/703
Testing / Proof
Here is a screenshot of the result:

Can you please provide more screenshots with different font sizes?

Can you please provide more screenshots with different font sizes?

Done


Change Filename during upload

Hi @dr-dimitru
I was trying to give fileName:'manutest.jpg', , Its not working
      var file = template.find('.image-upload-class').files[0];
      if (file) {
        var uploadInstance = Images.insert({
          file: file,
		  fileName:'manutest.jpg',
          streams: 'dynamic',
          chunkSize: 'dynamic'
        }, false);

How can I specify the file name during insert?, Is there is any way out? I want to change the filename to different name and don't want to keep default name.
Any Possibility?
Thanks and Best Regards,
Manu

Implemented in dev branch, please test it on your end

Wow!, That's Great, Please allow me sometime, I will get back on this soon.

Hi @dr-dimitru ,
How to check dev version actually I have never done it, Can you let me know the steps
Thanks and Best Regards,
Manu

In meteor's project dir (btw it's better to Google it):
mkdir packages
cd packages
git clone https://github.com/VeliovGroup/Meteor-Files.git
cd ../
meteor update && meteor

Thanks, I will check it and get back with my update,

@manusharma1 any news from your end?

Hi @dr-dimitru ,
I have tested it and found that it is working but I was expecting the filename to be saved as the given filename on the disk, Right now it is saving as the ID and extension, It is possible that filename if given as : fileName: 'manutest.jpg' , then on the disk aslo it will be saved as the same name.
Thanks for your quick replies and I really appreciate your nature of reminding for the issues, Keep it up!
Thanks and Best Regards,
Manu

Hi @manusharma1 ,
I wouldn't recommend to change the way lib naming files in FS, but this is possible via config.namingFunction in Constructor

Thanks for your quick replies and I really appreciate your nature of reminding for the issues, Keep it up!

Appreciated, thank you for using this package and your contribution.


Rename to tapyrus

This PR changes the word 'bitcoin' to 'tapyrus'

@Yamaguchi can you rebase this?


[response needed] Deactivate Selectbox replacement

Deactivate Selectbox replacement when the css-class "no_style" is set

Dear Contributor,
Thank you for your contribution to Shopware. We value your engagement.
Unfortunately we have to close all existing Pull Requests.
This is due to the new Shopware 5 Major Release and a restructuring of the branch names.
If your contribution is still relevant for Shopware 5, a new pull request is highly appreciated.
We are very sorry for this inconvenience.
Greetings from the Shopware Team.


WindowsError: [Error 2] - on file finder

Windows 7, Python 2.7.8
To reproduce:
git clone https://github.com/peritus/bumpversion bumpversion
cd bumpversion
python.exe setup.py sdist

C:\Users\<user>\Documents\scripts\python\_other\bumpversion_clean2>python.exe setup.py sdist
C:\Python27\ArcGIS10.2\lib\site-packages\setuptools\dist.py:282: UserWarning: Normalizing '0.5.4-dev' to '0.5.4.dev0'
  normalized_version,
options (after parsing config files):
options (after parsing command line):
option dict for 'aliases' command:
  {}
option dict for 'sdist' command:
  {}
option dict for 'wheel' command:
  {'universal': ('setup.cfg', '1')}
running sdist
Distribution.get_command_obj(): creating 'sdist' command object
running egg_info
Distribution.get_command_obj(): creating 'egg_info' command object
writing bumpversion.egg-info\PKG-INFO
writing top-level names to bumpversion.egg-info\top_level.txt
writing dependency_links to bumpversion.egg-info\dependency_links.txt
writing entry points to bumpversion.egg-info\entry_points.txt
Distribution.get_command_obj(): creating 'build_py' command object
Distribution.get_command_obj(): creating 'build' command object
error: [Error 2]
Traceback (most recent call last):
  File "setup.py", line 39, in <module>
    'Programming Language :: Python :: Implementation :: PyPy',
  File "C:\Python27\ArcGIS10.2\lib\distutils\core.py", line 151, in setup
    dist.run_commands()
  File "C:\Python27\ArcGIS10.2\lib\distutils\dist.py", line 953, in run_commands
    self.run_command(cmd)
  File "C:\Python27\ArcGIS10.2\lib\distutils\dist.py", line 972, in run_command
    cmd_obj.run()
  File "C:\Python27\ArcGIS10.2\lib\site-packages\setuptools\command\sdist.py", line 40, in run
    self.run_command('egg_info')
  File "C:\Python27\ArcGIS10.2\lib\distutils\cmd.py", line 326, in run_command
    self.distribution.run_command(command)
  File "C:\Python27\ArcGIS10.2\lib\distutils\dist.py", line 972, in run_command
    cmd_obj.run()
  File "C:\Python27\ArcGIS10.2\lib\site-packages\setuptools\command\egg_info.py", line 180, in run
    self.find_sources()
  File "C:\Python27\ArcGIS10.2\lib\site-packages\setuptools\command\egg_info.py", line 207, in find_sources
    mm.run()
  File "C:\Python27\ArcGIS10.2\lib\site-packages\setuptools\command\egg_info.py", line 291, in run
    self.add_defaults()
  File "C:\Python27\ArcGIS10.2\lib\site-packages\setuptools\command\egg_info.py", line 323, in add_defaults
    rcfiles = list(walk_revctrl())
  File "C:\Python27\ArcGIS10.2\lib\site-packages\setuptools\command\sdist.py", line 19, in walk_revctrl
    for item in ep.load()(dirname):
  File "C:\Python27\ArcGIS10.2\lib\site-packages\setuptools_scm\__init__.py", line 77, in find_files
    return do(ep.load(), path).splitlines()
  File "C:\Python27\ArcGIS10.2\lib\site-packages\setuptools_scm\utils.py", line 48, in do
    out, err, ret = do_ex(cmd, cwd)
  File "C:\Python27\ArcGIS10.2\lib\site-packages\setuptools_scm\utils.py", line 33, in do_ex
    HGPLAIN='1',
  File "C:\Python27\ArcGIS10.2\lib\subprocess.py", line 710, in __init__
    errread, errwrite)
  File "C:\Python27\ArcGIS10.2\lib\subprocess.py", line 958, in _execute_child
    startupinfo)
WindowsError: [Error 2]

Contents of setup.py
import re
from setuptools import setup

description = 'Version-bump your software with a single command!'

long_description = re.sub(
  "\`(.*)\<#.*\>\`\_",
  r"\1",
  str(open('README.rst', 'rb').read()).replace(description, '')
)

setup(
    name='bumpversion',
    version='0.5.4-dev',
    url='https://github.com/peritus/bumpversion',
    author='Filip Noetzel',
<EMAIL_ADDRESS>    license='MIT',
    packages=['bumpversion'],
    description=description,
    long_description=long_description,
    entry_points={
        'console_scripts': [
            'bumpversion = bumpversion:main',
        ]
    },
    classifiers=(
        'Development Status :: 4 - Beta',
        'Intended Audience :: Developers',
        'License :: OSI Approved :: MIT License',
        'Operating System :: OS Independent',
        'Programming Language :: Python',
        'Programming Language :: Python :: 2',
        'Programming Language :: Python :: 2.7',
        'Programming Language :: Python :: 3',
        'Programming Language :: Python :: 3.2',
        'Programming Language :: Python :: 3.3',
        'Programming Language :: Python :: 3.4',
        'Programming Language :: Python :: Implementation :: PyPy',
    ),
)

Error 2 means "file not found"
https://msdn.microsoft.com/en-us/library/windows/desktop/ms681382(v=vs.85).aspx
But which file? There is no information.
When i delete setuptools_scm pip uninstall setuptools_scm - all is ok, no more error.

Bitbucket: https://bitbucket.org/pypa/setuptools_scm/issue/27
Originally reported by: lansman
Originally created at: 2015-05-20T11:48:19.794

File finders again

Original comment by: Ronny Pfannschmidt

print exception on failed/broken file finding, fixes #27
→ <>

Original comment by: Ronny Pfannschmidt


Simplify/generify GeneratorTask incrementality

As a follow up to #506, we should further simplify the incrementality of the Visual Studio IDE tasks and hoist it up into GeneratorTask so that it can be used by any of the other IDE tasks that extend from this class.  To do this, we should have the tasks generate the xml (or other content) in memory and then compare it to what's on disk, only writing the file if the contents are different.

Mostly implemented in https://github.com/gradle/gradle/pull/4638
Need to add coverage for the other IDE generator tasks.


URL is not a Constructor

Please Help, im trying to get access token that will be use for firebase dynamic link rest.
the probel that i got is :
before iam using google.auth.JWT
it get me an error URL is not custructor message.
now iam using google auth library to give me the access token.
const client = new JWT(
serviceAccount.client_email,
null,
serviceAccount.private_key,
scopes
);
but it still give me the same response like before, any suggestion?
Environment details

OS:
Node.js version: 9.0.0
npm version: 5.5.1
google-auth-library version: latest one
Thanks!

Greetings!  Node.js 9.x is not gonna be a supported version.  Can you try this with the latest version of 10?

Greetings! Node.js 9.x is not gonna be a supported version. Can you try this with the latest version of 10?

it work now, thanks

ok this issue has been solved, thanks for the response justin


Add version v20170701

What this PR does / why we need it: Adds new version v20170701
Release note:
The new version v20170701 exposes:

Multiple agents pools with public port configuration and choice of OSType.
orchestrator version control
SwarmMode renamed to DockerCE
Master VMSize
ManagedDisks
Bring your own VNET, including specification of first static ip for master.
OSDisksize for Master and Agent

This change is

Personal pet peeve. Our version is never in a header. So the PR should be named adding version v20170701or v20170630 if we want to stay consistent with the end of the month versioning we have done so far. s never on a header. I'll try to look through this to make sure you are validating the right things etc later today.

Thanks Jack, I've fixed the wording in the title and the descriptions.

Reviewed 9 of 14 files at r1.
Review status: 9 of 14 files reviewed at latest revision, 5 unresolved discussions.

examples/v20170701/kubernetes.json, line 18 at r2 (raw file):

        "count": 1,
        "vmSize": "Standard_D2_v2",
        "availabilityProfile": "AvailabilitySet"

Do we expose availabilityProfile?

pkg/api/v20170701/types.go, line 39 at r2 (raw file):

	WindowsProfile          *WindowsProfile          `json:"windowsProfile,omitempty"`
	ServicePrincipalProfile *ServicePrincipalProfile `json:"servicePrincipalProfile,omitempty"`
	CertificateProfile      *CertificateProfile      `json:"certificateProfile,omitempty"`

is it exposed?

Comments from Reviewable

pkg/api/v20170701/types.go, line 39 at r2 (raw file):
Previously, weinong (Weinong Wang) wrote…
is it exposed?

certificateprofile is now removed

Comments from Reviewable

Reviewed 2 of 5 files at r4.
Review status: 10 of 15 files reviewed at latest revision, 7 unresolved discussions.

pkg/api/converterfromapi.go, line 87 at r4 (raw file):

i.Plan != nil {
v20170701CS.Plan = &v20170701.ResourcePurchasePlan{}
convertResourcePurchasePlanToV20170701(api.Plan, v20170701CS.Plan)
}
i wonder why do we have this?

pkg/api/v20170701/types.go, line 23 at r4 (raw file):

	Location string                `json:"location,omitempty"`
	Name     string                `json:"name,omitempty"`
	Plan     *ResourcePurchasePlan `json:"plan,omitempty"`

@amanohar do we still need this?

Comments from Reviewable

Reviewed 2 of 3 files at r3, 3 of 5 files at r4.
Review status: all files reviewed at latest revision, 7 unresolved discussions.

Comments from Reviewable

@weinong  we don't need it (ACS  doesn't use it). It's standard ARM API model from their doc so we can leave it in there.

pkg/api/converterfromapi.go, line 87 at r4 (raw file):
Previously, weinong (Weinong Wang) wrote…

i.Plan != nil {
v20170701CS.Plan = &v20170701.ResourcePurchasePlan{}
convertResourcePurchasePlanToV20170701(api.Plan, v20170701CS.Plan)
}
i wonder why do we have this?

purchase plan is part of ARM and can be passed from ARM

Comments from Reviewable

pkg/api/v20170701/types.go, line 23 at r4 (raw file):
Previously, weinong (Weinong Wang) wrote…
@amanohar do we still need this?

Talked with ARM team, and we can remove this, I have submitted https://github.com/Azure/acs-engine/issues/810 for removal

Comments from Reviewable

pkg/api/v20170701/types.go, line 23 at r4 (raw file):
Previously, anhowe (Anthony Howe) wrote…
Talked with ARM team, and we can remove this, I have submitted https://github.com/Azure/acs-engine/issues/810 for removal

Done.

Comments from Reviewable

examples/v20170701/kubernetes.json, line 18 at r2 (raw file):
Previously, weinong (Weinong Wang) wrote…
Do we expose availabilityProfile?

good catch, removed

Comments from Reviewable

Review status: all files reviewed at latest revision, 7 unresolved discussions.

pkg/api/converterfromapi.go, line 87 at r4 (raw file):
Previously, anhowe (Anthony Howe) wrote…
purchase plan is part of ARM and can be passed from ARM

Done.

Comments from Reviewable

Review status: all files reviewed at latest revision, 7 unresolved discussions.

pkg/api/v20170701/const.go, line 9 at r1 (raw file):
Previously, anhowe (Anthony Howe) wrote…
thanks, added back

Done.

Comments from Reviewable

Review status: all files reviewed at latest revision, 7 unresolved discussions.

pkg/api/v20170701/types.go, line 23 at r4 (raw file):
Previously, anhowe (Anthony Howe) wrote…
Done.

Done.

Comments from Reviewable


Create a variable for class properties in scope instead of deleting t…

…he key
Fixes https://github.com/babel/babel-eslint/issues/308

Lets hope there's no issues 😛

Can you do a release so that I can import it inside of fb codebase? Thanks!

Yep already done all that 😄

you are too fast!

probably got that from @kittens actually


:boom: Rename image.name => image.repository

What does this PR do?
Rename helm value from image.name to image.repository
It's just a renaming, no change on the behaviour.
Motivation
It's in fact a repository path and it's needed for Renovate compatibility.
See #625 & https://github.com/renovatebot/renovate/issues/7010
More

[x] Yes, I updated the tests accordingly
[x] Yes, I ran make test and all the tests passed

Friends, Its not just a rename, This broke our production system.
It needs backwards compatibility !

@juanitoddd when releasing we follow semver. We released this by incrementing a major version (X.0.0), this means it will include breaking changes.
As mentioned before, we try not make these changes unless we have to. I'd recommend checking the traefik/Changelog.md and GitHub Release before updating a major version in the future to ensure you're not affected.


Add SSL Support to PostgresKVStore

Description
Adds the following options to from_params and from_uri:

sslmode
sslcert
sslkey
sslrootcert

More info on ssl modes here: https://www.postgresql.org/docs/current/libpq-ssl.html#LIBPQ-SSL-SSLMODE-STATEMENTS
This conditionally requires psycopg3 if using ssl. This is due to an ongoing issue with asyncpg making it unusable for AWS RDS deployments: https://github.com/MagicStack/asyncpg/issues/952
Fixes # N/A
Version Bump?
Did I bump the version in the pyproject.toml file of the package I am updating? (Except for the llama-index-core package)

[x] Yes
[ ] No

Type of Change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Added new unit/integration tests
[ ] Added new notebook (that tests end-to-end)
[x] I stared at the code and made sure it makes sense

Suggested Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added Google Colab support for the newly added notebooks.
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[x] I ran make format; make lint to appease the lint gods

hey why is this closed?
I am unable to connect my potgres without SSL :(

Apologies for lack of communication here. I need to get additional approvals through work before I can do anything else with this PR. I'll re-open once I'm given the green light.. @dhanushreddy291


Google Workspace search google drive files functionality

Extension – Google Workspace
Author:

Describe the feature and the current behavior/state.
Allow search google drive files that search files that are in shared drives as well.
Who will benefit with this feature?
Google drive user who would like to search for files within different shared drives.
Any Other info.
That is all. Thank you for your time.

+1, same request.