"disableDeploy" not available in launch_schema.json

I'm using 2019-16.1.3, but the last piece of code-block in this page doesn't seem to have IntelliSense to describe it.

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

ID: 001834ce-31d2-e768-1df1-4a776aa9ef65
Version Independent ID: 45f15fc0-e5c7-34a1-2469-f38d304ef5bd
Content: Deploy, run, and debug your C++ Linux project in Visual Studio
Content Source: docs/linux/deploy-run-and-debug-your-linux-project.md
Product: visual-cpp
Technology: cpp-linux
GitHub Login: @mikeblome
Microsoft Alias: mblome

Is there a complete example in Github?
"disableDeploy": false,
"deployDirectory": "~\foo",
"deploy" : [
{
"sourceMachine": "<IP_ADDRESS> (username=example1, port=22, authentication=Password)",
"targetMachine": "<IP_ADDRESS> (username=example2, port=22, authentication=Password)",
"sourcePath": "~/example.cpp",
"targetPath": "~/example.cpp",
"executable": "false"
}
]
The above doesn't work.  It is complaining that  disableDeploy wants to be a string.

@infinite-persistence If you put false in quotes, does it work as expected? Is that the only problem you are having?

I couldn't get the entire "deploy to another target" to work.  Most likely I misunderstood the partial instructions, hence the request for a full example.
I ended up installing nfs on the target and making manual copies, since I couldn't get VS2019 to deploy it from the Linux build machine to the Linux target.

@infinite-persistence We have a work item on our internal backlog to provide a better tutorial for this scenario. I'm going to close this issue since you solved your immediate problem.


Confused by DBSCAN min points

I modified the sample and all the points are marked as noise if you increase the min points value to 3:
var dataset = [
    [1,1],[0,1],[1,0],
    [10,10],[10,13],[13,13],
    [54,54],[55,55],[89,89],[57,55]
];

var clustering = require('density-clustering');
var dbscan = new clustering.DBSCAN();
var clusters = dbscan.run(dataset, 5, 3);
console.log(clusters);
console.log(dbscan.noise);

I'm not sure whether this is the intended behavior for DBSCAN. I think this should be fixed or documented. What do you think?
I guess this could be fixed arround here: https://github.com/LukaszKrawczyk/clustering/blob/master/lib/DBSCAN.js#L68

@awenger Thanks for your comment!
As far as I understand the algorithm, this is correct behaviour.
According to algorithm description, minPts is a minimum number of points in ε-neighbourhood around the core point to form a cluster.
In your case, ε is was too small to form a cluster containing at least 3 points, so all points were marked as a noise.
Yes, I agree with you it may be confusing and all parameters needs brief explanation.
I've got two ideas:

write more detailed explaination it in JSDoc comments included in the code
add API.md containing list of exposed functions and detailed explaination

What do you think?
If you want to help, feel free to submit pull request!

@LukaszKrawczyk I'm also not 100% sure. But if I look at wikipedia it describes the parameters as: DBSCAN requires two parameters: ε (eps) and the minimum number of points required to form a dense region [1]
The pseudocode for regionQuery also includes the point that was used to query the neighborhood [1]:
regionQuery(P, eps)
   return all points within P's eps-neighborhood (including P)

I guess this is different in this DBSCAN implementation: https://github.com/LukaszKrawczyk/clustering/blob/master/lib/DBSCAN.js#L184
What do you think?
[1] http://en.wikipedia.org/wiki/DBSCAN

Yes... there was a similar discussion on including core point in OPTICS algorithm some time ago:
https://github.com/LukaszKrawczyk/clustering/issues/4.
I checked how it's implemented in other libraries (such as e.g. SciKit), and I'm more and more confident we should change the algorithm to include core points, both in DBSCAN and OPTICS.
I will modify code and update package on NPM.

Thanks a lot for the update


Support for React < 18

Can we expect support for React < 18 in the future? I see that useId hook is being used and it is only available in React 18.

@JonathanAriass If we can support <18 without useId, I think we can accomplish this task.

@jaywcjlove Can we use 'useId' for React 18 and only support client-side for React < 18?
refer https://github.com/react-component/util/blob/7aaa1d88174b30d7fb9c94b41b7a34cea5fc37f3/src/hooks/useId.ts#L43)

@EliazTray  If this is the only issue, I can handle it easily.
import { useRef } from 'react';

function useIdCompat() {
  const idRef = useRef(null);
  if (idRef.current === null) {
    idRef.current = 'custom-id-' + Math.random().toString(36).substr(2, 9);
  }
  return idRef.current;
}

export default useIdCompat;

@JonathanAriass  @EliazTray Upgrade v2.0.0-alpha.27

@JonathanAriass @EliazTray Upgrade v2.0.0-alpha.27

It took effect  on<EMAIL_ADDRESS>!
Thanks for your quickly reply.


False positive requireValidation check with minProperties

Description
{
    "type": "object",
    "minProperties": 1,
    "properties": {
        "phoneNumber": { "type": "string" },
        "emailAddress": { "type": "string" }
    },
    "additionalProperties": false
}

This schema will fail with [requireValidation] if properties is used, required should be specified.
The requirement in this case is not necessary because minProperties is specified and additionalProperties is set to false, so at least one of phoneNumber and emailAddress is already required without explicitly using the required keyword.
Related code
https://github.com/ExodusMovement/schemasafe/blob/81459f3c7162137c19e19d9b869f9e41d0bf8dde/src/compile.js#L1158

In the additionalProperties: false + properties + minProperties case, this behavior does seem to be intentional though.
I'll look closer into that.

Personally I can't think of any situations where minProperties + additionalProperties: false behavior can be unintentional. The alternative way of doing this would be to use anyOf + required, which is much more verbose, has oblique error messages and presumably compiles to longer code. Numbers larger than one can't be expressed in this way either.
The workaround of require: [] is also not immediately obvious and confusing to future readers. My thoughts after submitting this issue were, "Oh, I guess I just have to disable this check in the mean time then". And since requireValidation can't be trivially overrided in strong mode, I'm sure some people will be lazy enough to disable stronge mode entirely (!).


In Cloudflare v4.41, using an expression with "in {item1 item2}" in cloudflare_rule causes terraform to fail on an Terraform Apply. Issue does not exist in v4.18

Confirmation

[X] This is a bug with an existing resource and is not a feature request or enhancement. Feature requests should be submitted with Cloudflare Support or your account team.
[X] I have searched the issue tracker and my issue isn't already found.
[X] I have replicated my issue using the latest version of the provider and it is still present.

Terraform and Cloudflare provider version
Terraform v1.7.5 on linux_amd6
cloudflare/cloudflare v4.41.0 (self-signed, key ID C76001609EE3B136)
Affected resource(s)
cloudflare_ruleset
This error impacts v4.41 of the Cloudflare Terraform provider but not v4.18
Terraform configuration files
resource "cloudflare_ruleset" "dynamic_restrictions" {
  account_id  = var.cloudflare_account_id
  name        = "Rules for development environment"
  description = "Restrict traffic to dev environment"
  kind        = "custom"
  phase       = "http_request_firewall_custom"

 rules {
    action      = "block"
    expression  = "(not ip.geoip.asnum in {XXX YYY} )"
    description = "Block all traffic not originating from SEI ASN"
    enabled     = true
  }
}

Link to debug output
NA
Panic output
Error: error updating ruleset with ID "8f9137da29324de2812f46fc0db33e16"
with module.acct_firewall.cloudflare_ruleset.dynamic_restrictions,
on modules/acct_firewall/dev_environment_ruleset.tf line 2, in resource "cloudflare_ruleset" "dynamic_restrictions":
2: resource "cloudflare_ruleset" "dynamic_restrictions" {
'not ip.geoip.asnum in {17276 14056})' is not a valid value for expression
because the expression is invalid: Filter parsing error (1:36):
not ip.geoip.asnum in {17276 14056})
^ unrecognised input
(20127)
Expected output
module.acct_firewall.cloudflare_ruleset.dynamic_restrictions: Modifying... [id=6ec095c441e14043bb26b9505f5cf2d5]
Actual output
Error: error updating ruleset with ID "8f9137da29324de2812f46fc0db33e16"
with module.acct_firewall.cloudflare_ruleset.dynamic_restrictions,
on modules/acct_firewall/dev_environment_ruleset.tf line 2, in resource "cloudflare_ruleset" "dynamic_restrictions":
2: resource "cloudflare_ruleset" "dynamic_restrictions" {
'not ip.geoip.asnum in {17276 14056})' is not a valid value for expression
because the expression is invalid: Filter parsing error (1:36):
not ip.geoip.asnum in {17276 14056})
^ unrecognised input
(20127)
Steps to reproduce
create a rule with expression that contains "in {item1 item2}" and run it with v4.41 of the Cloudflare provider and you will get an error.  Run it with v4.18 of the Cloudflare provider and the rule will deploy successfully.
Additional factoids
No response
References
No response

Still an issue On Oct 13, 2024, at 9:04 PM, github-actions[bot] @.***> wrote:
Marking this issue as stale due to 30 days of inactivity. This helps our maintainers find and focus on the active issues. If this issue receives no comments in the next 7 days it will automatically be closed. Maintainers can also remove the lifecycle/stale label.
If this issue was automatically closed and you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thank you!
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


TypeScript: everytime I complete a suggestion an error shows up in the console

/cc @jrieken @ramya-rao-a

From the gif, it doesnt look like the error is showing up for completing suggestion.
It shows up after you type () when calling a method.
And so is the same as #46852


doc(hep): add managed dhcp hep [CI SKIP]

The proposal of the managed DHCP feature
Related Issue:
#4428

Please check https://github.com/harvester/vm-dhcp-controller for the implementation.

Per the concept of SDN, we can enhance and simplify the architecture:

The architecture looks interesting. I will certainly take a look and see how we can proceed in the future. But please note that the renewal and lease timeout have not been implemented in the current version of vm-dhcp-controller. Thank you.


Can't run app because errors from onesignal

My app worked fine yesterday but all of a sudden today android wont run without me doing anything
the error :

what should I do about it?

@matan335 Have you tried updating your project's compileSdkVersion...?


Use Kanban to visualise proposal status

This was proposed as an action at the CF User Meeting in 2020. Is anybody interested in working on this? I would join them at the 2022 User Meeting to hack on it!

Hi Daniel, one question on this: quite what form of Kanban is intended here? Would GitHub Projects suffice, since that can be used effectively as a Kanban board I believe, and it is very accessible directly here in GitHub which we already use for most things relating to discussion, organisation maintenance of the CF Conventions?
I guess there are some points in the CF Meeting 2020 notes relating to this proposal. Would you mind pointing me /  us generally to those so I can see a bit more? Thanks.

@sadielbartholomew yes, I think GitHub Projects would be the tool of choice - it's good to have everything in one place. There's a project associated with this issue that contains the follow-up to the 2020 meeting - you can find everything that I noted down at the time there. As always, your ideas a very welcome :)

Dear Daniel @erget, @sadielbartholomew
Is there a prospect of some progress on this in the near future? If not, I suggest we could close it as dormant for now, since it's been quiet for a couple of years. It could be reopened later, of course.
Best wishes
Jonathan

@JonathanGregory this was used at the 2022 meeting and I think it was useful in the context of hackathons but I haven't gotten the impression it's been a feature in hot demand for the CF organization as a whole between hackathons. It's easy enough to setup that we might use it next time but I don't think that it's something that we need to pursue at this time for moving the issues - we seem to have achieved good flow as it is for the moment. Happy to close this now.


Add MIRAI annotations and fix a bug

Motivation
Add MIRAI annotations that are needed to document the structure invariants that are relied upon by the code.
Also fix a cut & paste error that I noticed along the way.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
cargo xtest

@bors-libra r=dariorussi

:pushpin: Commit 1d9a0fa has been approved by dariorussi

:hourglass: Testing commit 1d9a0fad9006ad188b348a91dd90573f060cb57a with merge 0462ee2cf2bd79ed549c305268dfc1e4c9913b59...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: dariorussi
Pushing 0462ee2cf2bd79ed549c305268dfc1e4c9913b59 to master...


Freeze tokens before migrating to semantic tokens as an external dependency.

@janiekyu, I think this is good to go.

Removed npm scripts that pulled and built tokens locally
Removed script files used by the npm scripts
Bumped Node version specified in .nvmrcs to v18 so that the proper React Native dependencies could be installed

(Note that this doesn't add the new tokens as a dep, it only freezes the old tokens as-is to avoid accidental breakage.)


Module 1 exercises Kabelo A .Monametsi

Please "drag & drop" exercise 1 pdf file below.

Please "drag & drop" exercise 2 pdf file below.

Please UPDATE THE TITLE of this "Issue" in t
exercise 1.pdf
exercise 2.pdf

he title bar above with your name.

If you haven't done so, please go back to Module 1 and make sure to take
the post-training survey. Participating in surveys is required for receiving
the "GALUP Certificate" by the University of Florida.

Well done, Kabelo. You clearly grasped the material outlined in Module 1 and successfully applied it to these exercises. Thank you for your participation!


Add powershell core 6+ support (pwsh)

PowerShell core (6.x and newer) have quite some differences. For the shell plugin most notably is the way that the sourcing of the ps1 script works and the different name of the executable.
Issue: #678

Not sure if you thought this was ready for merging yet, but given that it's a new plugin that can't break anything previous, I've merged it anyway :P


Keep unused import and pass statement

Make it possible to provide CLI args to keep autoflake from removing useless pass (--keep-useless-pass) and unused import (--keep-unused-import)
Closes https://github.com/myint/autoflake/issues/13 https://github.com/myint/autoflake/issues/10

Coverage increased (+0.03%) to 98.253% when pulling 14677f6f362b61b183863bad63919be54825a5c6 on adhikasp:modular into 459f5f53cf10109020fce0470b1cd456d4bfec46 on myint:master.

I'm not sure I want to add the --keep-useless-pass option. autoflake currently depends on temporarily replacing unused imports/variables with pass and then removing the useless ones in a second pass. This option would result in things like the following, where we end up leaving the code in a bit of a mess.
--- original/foo.py
+++ fixed/foo.py
@@ -1,3 +1,3 @@
 def foo():
-    x = 1
-    y = 2
+    pass
+    pass

I think to properly supporting this might involve significant restructuring of autoflake. Currently, I'm fine with supporting the 99% use case where we happen to remove useless pass statements. Though, if someone came up with an elegant solution to this, I'd be fine with that as well.

Yes I agree this could make the resulting code clutters and the approach in this PR is too straightforward. I will close it in the meantime.


Fixed article title

When selecting an edit and scrolling down in the article the title is supposed to be fixed according to the style guide currently it is not

The style guide was largely a suggestion and the implemented site style has diverged from it.


[Search v2.1] Adding semicolon to search filter leads to page not found

Coming from here, adding a ; to the keyword search filter leads to a page not found.
cc @289Adam289

We're awaiting for a volunteer to take on this issue, given that https://github.com/Expensify/App/pull/46799 was merged.

Still awaiting for a volunteer to take on this issue, given that https://github.com/Expensify/App/pull/46799 was merged.

I think this pr from issue will solve this problem.

Ah yea, I'll close this issue in favor of https://github.com/Expensify/App/issues/47095


Maybe two bugs

I'm trying to add days of week, but RecurrenceRule.WeekdayNum is not public:
List<RecurrenceRule.WeekdayNum> days = new ArrayList<RecurrenceRule.WeekdayNum>();
days.add(RecurrenceRule.WeekdayNum.valueOf("SU"));
days.add(RecurrenceRule.WeekdayNum.valueOf("TH"));
rule.setByDayPart(days);
I'm trying to add month days to rule, but I get an IllegalArgumentException:
rule.setByPart(Part.BYMONTHDAY, 2, 3);

Yikes! You're right. I was so focused on the parser and iterator that I forgot to test the builder!


Use Eslint instead of lint-stagged

Fixes many files according to the rules besides the following to warning for now so we can address them in time:

"@typescript-eslint/no-explicit-any": "warn",
"@typescript-eslint/no-unused-vars": "warn",
"@typescript-eslint/no-non-null-assertion": "warn",
"@typescript-eslint/no-empty-function": "warn",

Replaced with https://github.com/konveyor/tackle2-ui/pull/139


GZip Deflater support

Plz assign to me

Fixed in https://github.com/apache/camel-quarkus/commit/72e2d3c5a8798336b6de970e3286778c836d23d3.


Remove @types node dev dependency from botframework-streaming

Fixes #1663
Description
Goal: remove our dependency on @types/node

Removed public types from node's net module from botframework-streaming
Replaced node types with mock interfaces of node classes

Specific Changes

Added helper method to create Servers (createNodeServer(), getServerFactory())

Created interfaces:

IEventEmitter
INodeServer

Added to the following mock interfaces, to be the proper shape to replace node types:

INodeBuffer
INodeSocket -- includes signatures from Duplex, so we didn't have to rely on stream types

Pull Request Test Coverage Report for Build 108800

11 of 15 (73.33%)  changed or added relevant lines in 3 files are covered.
6 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.05%) to 81.708%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

libraries/botframework-streaming/src/namedPipe/namedPipeServer.ts
4
6
66.67%

libraries/botframework-streaming/src/utilities/createNodeServer.ts
6
8
75.0%

Files with Coverage Reduction
New Missed Lines
%

libraries/botframework-streaming/src/namedPipe/namedPipeServer.ts
6
79.49%

Totals

Change from base Build 108742:
-0.05%

Covered Lines:
7882

Relevant Lines:
9271

💛  - Coveralls

@stevenic

moved the import statement below the comment
and as per in person discussion, we're not making createNodeServer() return a promise
I didn't export the 2 interfaces used only within INodeSocket still, but let me know if you want that done. Otherwise I'll leave it as is

@johnataylor it's this P2 that needs review/approval
other one can be punted


pruning obsolete branches

cc/ @tarzzz @chriddyp @jackparmer @bpostlethwaite @rmoestl
Here's a list of branches that look obsolete.
Please, could you go through the list?
If you find that a branch is yours and you don't need it any longer,
please, could you tick it here and I'll delete it?
If you see any branches ticked for removal that you'd like to keep,
please, comment here.
Branches to keep:

2.4-onprem (used for On-Prem 2.3.0)
2.2-release (used for On-Prem 2.2.0)

Branches to prune:

[ ] add-authentication
[x] add-download
[x] allow-oauth-uri-auth-exception
[x] auto-add-to-allowed-users
[x] circle2
[ ] electron-oauth-update
[x] eslint-fixes-in-app
[x] eslint-remaining-fixes
[ ] fix-appveyor
[ ] livy
[x] merge-jps-thing
[x] migrate-webpack
[x] mssql-options-old-build
[x] n-riesco-travis-zip-only-installer
[x] onprem-2.3-fixes
[x] release-v2.4.0
[ ] setup-hadoop-spark-test-db
[x] tests-improve-error-messages
[x] trying-to-api
[x] usePlotlyDataSetsInDB2
[x] v2.3-onprem
[x] v2.3.2-onprem
[x] yarn-sql-editor
[x] yo-gaba-gaba

i don't need any of my branches, prune away!

Prune away! Thanks for tidying up the place!
On Fri, Dec 15, 2017 at 12:49 PM, Chris Parmer<EMAIL_ADDRESS>wrote:

i don't need any of my branches, prune away!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/plotly/falcon-sql-client/issues/325#issuecomment-352068931,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABx4alfAMfEpkXAAuDKH_mC1sUSvdazLks5tArEZgaJpZM4RDaJI
.

@bpostlethwaite I'm going to delete usePlotlyDataSetsInDB2 (since it's been already merged).

@tarzzz can you confirm you don't need the following branches that have unmerged commits?

add-authentication
setup-hadoop-spark-test-db
livy


Avoiding private registry hostname in FROM directive of Dockerfile

Say I have an image FOOBASE in a private registry (hostname:port PRIVREGHOSTNAME:5000)
I want to use this image in a Dockerfile for a new image FOONEW. Currently, it is done as follows:
Dockerfile
FROM PRIVREGHOSTNAME:5000/FOOBASE
...
That is, the hostname of the private registry is in the Dockerfile.
Now say, I want to change the registries. I have to change the Dockerfile of FOOBASE, rebuild it, and rebuild the image FOONEW that depends on FOOBASE. Obviously, changing the Dockerfile is not desirable. The registry hostname should not be in the Dockerfile
Instead, a better solution is to define a new parameter --registry-url for docker build. E.g.,
docker build -t . --registry-url PRIVREGHOSTNAME:5000
So by default, the image referenced in Dockerfile is searched first in the registry-url and then in Dockerhub.

oh, if only you could have written something like:
FROM ${registry:-myregistry.com}/foobase
...

docker build --build-arg registry=yourregistry.com

:-)

This is a long outstanding issue :) I bet there are other issues around about this problem

Are you going to answer to this issue? Working with private issues in Docker is really annoying, and confirming that there are many issues regarding it is probably not enough for users...

FYI: https://github.com/docker/docker/pull/31352

This opportunity is valuable for me. The only way I can do this is to preprocess dockerfile template file with envsubst like this:
$ cat Dockerfile.template
FROM ${REPO_NAME}:${tag}
$ export REPO_NAME=myrepo:5001
$ envsubst '$REPO_NAME' < Dockerfile.template > Dockerfile
$ cat Dockerfile
FROM myrepo:5001:${tag}

Am I missed any other solutions?

You can use --build-arg to pass a registry name.
ARG REGISTRY
FROM $REGISTRY/my/img
...

And then docker build --build-arg REGISTRY=myreg:5000 ...
Closing since this this can be done via build args.

You can use --build-arg to pass a registry name.

Right! I forgot to pass --build-arg REGISTRY=... to build command thinking it will be passed automatically. Thanks!

You can use --build-arg to pass a registry name.
ARG REGISTRY
FROM $REGISTRY/my/img
...

And then docker build --build-arg REGISTRY=myreg:5000 ...

But this does not work if you don't specify any registry!
In that case docker build . is effectively resulting in FROM /my/img and gives me

Step 1/10 : ARG REGISTRY
Step 2/10 : FROM $REGISTRY/my/img
invalid reference format

However it seems Docker knows Bourne Shell parameter expansion a bit (I did not find anything usefull in the docs though).
Just tested with version 19.03.8 on MacOS, where this works for both situations:
ARG REGISTRY
FROM ${REGISTRY:+$REGISTRY/}my/img
...


SQL Stats query fails if you have zero stat related column expressions specified

I've created a query and table against a SQL Stat (e.g. CPU)
The Query is Date Time between year() and now()
If I add a column expression add(123,45) and run the query, I get an SQL error.
Then I add another column ${Node} and re-run, the Query works OK.

Tested in V7-beta.46 and this bug is present


toctree lists header level information lost/mixed up.

Hi all,
first, congrats to a great project! Very useful!
I've noticed that the header level information from the .ipynb files are not consistently transferred to .html.
E.g. this the result of running python3 setup.py build_sphinx locally on a clone of this library https://github.com/spatialaudio/nbsphinx:

And this is what it should like (please only look at the structure):

Do you see how the levels were changed? I noticed this bug for the first time on my own project, so it seems to be consistent on my system (Python 3.5.0):
docutils==0.12 ipdb==0.8.1 ipykernel==4.3.1 ipython==4.2.0 ipython-genutils==0.1.0 ipywidgets==5.1.5 jupyter==1.0.0 jupyter-client==4.3.0 jupyter-console==4.1.1 jupyter-core==4.1.0 nbconvert commit q64aeeefa872be8c8885437b20fa1f0f6c45b8d58 pypandoc==1.1.3 Sphinx==1.4.2 sphinx-bootstrap-theme==0.4.9 sphinx-rtd-theme==0.1.9
Thanks for any ideas!

This is indeed very strange!
For me, the heading levels have always worked, both locally and online on RTD.
Does this occur only with nbsphinx or also with "pure" (only using RST files) Sphinx projects?
I have no clue what's causing this, but as a first step I'd probably try to install the minimum amount of packages in a virtualenv and see if the error still occurs.
It might also have something to do with pandoc, because it's used to process the Markdown cells, I'm currently using version <IP_ADDRESS>.
You could probably try to run only the content of a single Markdown cell (that shows the error) through your local pandoc ...
I that leads nowhere, you can also try to have a look at the internal RST representation, see https://github.com/spatialaudio/nbsphinx/issues/22#issuecomment-188702787.

Thanks for the ideas!
The problem when running python3 setup.py build_sphinx still persists (on Mac OSX) with

pandoc updated to 1.17.1 (Compiled with texmath <IP_ADDRESS>, highlighting-kate 0.6.2.)
on a fresh virtualenv (Python 3.5.0) with these installations:

 pip install git+https://github.com/spatialaudio/nbsphinx
 pip install sphinx_bootstrap_theme
 pip install jupyter-client

[btw, jupyter-client seems to be needed for the nbsphinx package, but not automatically installed with pip ?]
such that pip freeze results in:
alabaster==0.7.8
Babel==2.3.4
decorator==4.0.10
docutils==0.12
entrypoints==0.2.2
imagesize==0.7.1
ipython-genutils==0.1.0
Jinja2==2.8
jsonschema==2.5.1
jupyter-client==4.3.0
jupyter-core==4.1.0
MarkupSafe==0.23
mistune==0.7.2
nbconvert==4.2.0
nbformat==4.0.1
nbsphinx==0.2.8
Pygments==2.1.3
pytz==2016.4
pyzmq==15.2.0
six==1.10.0
snowballstemmer==1.2.1
Sphinx==1.4.4
sphinx-bootstrap-theme==0.4.9
traitlets==4.2.1
```

On the other hand, this problem does not occur in a pure .rst setting, nor does it impair the header structure of toctreed .rst files in a .rst/.ipynb mix setting.

So I'm not sure what to try anymore...

Update: I believe I might have found a fix (though not obvious):
If I run a notebook manually (from within jupyter notebook), the header structure seems to transferred alright even to the Sphinx docs.
For example, I've manually run code-cells.ipynb and raw-cells.ipynb, and the header structure now looks as follows:

So, it seems there is a workaround, though this behavior is unexpected?

The behavior is definitely unexpected!
It is really strange that executing a notebook with nbconvert seems to have an influence on Markdown cells.
Can you try to execute one of the notebooks with nbconvert?
python3 -m nbconvert --to <format> --execute code-cells.ipynb

You can try html, notebook and probably pdf for <format>.
You could also try the save-notebooks branch, which puts a copy of the executed notebook into the HTML output directory.

jupyter-client seems to be needed for the nbsphinx package, but not automatically installed with pip?

Hmm ... it indeed is needed, but you'll also need a kernel, don't you?
nbsphinx doesn't know which kernel you want to use, but any kernel should itself depend on jupyter-client, shouldn't it?
Or is that not the case for the non-Python kernels?
How did you install your kernel?
In the example requirements.txt, ipykernel is listed, which in turn should require jupyter-client, that's why I never realized the missing dependency.
If it is needed anyway, #56 might help?

@elkeschaper Any news? Is it still not working?

Ping @elkeschaper?
I had some problems with the version of pandoc installed on RTD, see #69.
Those problems went away when I changed to the version of pandoc provided by conda-forge.
Probably this also works in your case?

No new info here, just confirming that this is still occurring expanding on channel strategies.
I concur that header level information is being lost for files passing through nbsphinx (all ReST files are fine). Also concur that a work-around is to use the conda-forge pandoc. Thus if installing entirely from conda-forge, there's no problem — conda create -n docenv nbsphinx jupyter_client ipykernel -c source-forge as one of those automatically pulls in pandoc 1.19 from c-f. If using defaults as primary and c-f as secondary, have to install pandoc explicitly; this gets pandoc 1.15 from defaults and headers go awry. Can get a working build with defaults as primary and c-f as secondary by (at time of writing) forcing channel by version — conda create -n docenv nbsphinx jupyter_client ipykernel pandoc=1.19 -c defaults -c source-forge. Below is a working version set.
    docutils:         0.12-py35_2    defaults
    ipykernel:        4.5.2-py35_0   defaults
    ipython:          5.3.0-py35_0   defaults
    ipython_genutils: 0.1.0-py35_0   defaults
    jupyter_client:   5.0.0-py35_0   defaults
    jupyter_core:     4.3.0-py35_0   defaults
    nbconvert:        5.1.1-py35_0   defaults
    nbformat:         4.3.0-py35_0   defaults
    nbsphinx:         0.2.13-py35_0  conda-forge
    pandoc:           1.19.2-0       conda-forge
    python:           3.5.3-1        defaults
    sphinx:           1.5.1-py35_0   defaults

Thanks for the update and thanks for confirming that an inappropriate version of pandoc is the reason for this strange behavior!
Is there anything that can be added to the docs to help other people not run into the same problem?

Most of my nbsphinx docs consulting has been through web searches, and so this was actually the easiest problem to locate and rectify. In retrospect, the usage page is pretty clear about the importance of c-f for pandoc, though I suppose it could emphasize c-f for pandoc and nbsphinx together, rather than pip, since the latter is now conda packaged. So no fault toward the docs.
Thanks for a very useful project, page looks good in the end, and if you should ever need an example of nbsphinx+sphinx-build+CMake+conda-build, let me know.

@loriab I mentioned conda-forge (among some other packages) in #103, is this OK?

@loriab I've also made some changes to the CSS in #104 because your info and warning icons were covered by text. Can you please check if this works for you?

I've also run into the same issue. Adding conda-forge channel and installing pandoc 0.19 fixes the problem to me.

I'm closing this because I think in the meantime everyone is using conda-forge.
If anybody is still having problems with this, please comment!


Add debug middleware & update dependencies.

What's this PR do?
This pull request adds a LogMemoryUsage middleware that prints out how much memory was used by the request to see if we can drop our memory limit even more & get more concurrent requests per dyno. I've added this as a "debug" log message so we can toggle it on and off with APP_LOG_LEVEL.
Here's an example of what these log messages look like from my local:
[2018-07-19 15:13:14] local.DEBUG: memory_usage {"mb":3.274952,"path":"v2/users/5b50a5fb9a892007c52df162"} {"user_id":"5b50a5fc9a892007c52df163","client_id":"oauth-test-client","request_id":null}
[2018-07-19 15:13:16] local.DEBUG: memory_usage {"mb":3.731272,"path":"v2/users"} {"user_id":"5b50a5fc9a892007c52df163","client_id":"oauth-test-client","request_id":null}

It also updates dependencies (including Webpack 2 → 4) since builds were failing on my machine.
How should this be reviewed?
👀
Relevant Tickets
DoSomething/devops#383
Checklist

[ ] Documentation added for changed endpoints.
[ ] Tests added for new features/bug fixes.
[ ] Post a message in #api if this includes something that causes a rebuild!

@DFurnes is that memory limit per request? So is what we're doing here getting a feel for how big a request could be to see if we can safely lower that limit?

Yep, exactly! Right now we allocate a maximum of 32MB per-request, which Heroku uses to set our limit to 80 concurrent requests per performance dyno (w/ 2.5GB RAM). Dropping that further (say, to 16M or 8M) might let us handle more requests per dyno (and save some moolah).

so cool @DFurnes ! One q - if we log every request, will that fill up the log limit?

It probably would! The plan is to only set APP_LOG_LEVEL to debug for a little bit so we can get a good sample of memory usage, and then toggle off this log message by swapping it to info.

ah I see! Love it, @DFurnes !

@DFurnes this is really cool I can't wait to see the results!


Salsa cross TS-JS find all references does not work

Testing #2218
producer.js:
/**
 * @param {number} one
 */
exports.hi = function(one) {
	return 5;
}

exports.ciao = 'ciao';

consumer.ts

import * as p from './producer';

class A {
	public something: boolean;
	public foo(): void {}
}

var x = p.ciao;

This issue was moved to Microsoft/TypeScript#6647


Where is the support when "Your account is temporarily under review." ??? ( Request #5043135)

Request #5043135
I have a very strong problem with the support of SendGrid, The button "Continue" under the text "What can we help you with?" doesn't work, in this way my account is temporarily under review, my web-store cannot send important notifications, and the support where is???
I sent an email to the support 1 day ago to ask some strange activity I looked on Activity Feed, They didn't reply yet!
Please support me!!!

Expected content
Actual content
Link to page:

Hello @loginformweb
We're sorry for the trouble. Do you have a support ticket number I can reference?

yes dure the support ticket is #5043135

I have contacted Support, and they are looking into the issue. Thank you for your patience.

Please, let me know about my ticket #5043135


"Technical Error" / HTTP 500 when submitting quiz answers in tutorial steps

Tutorials: https://developers.sap.com/tutorials/cp-explore-cloud-platform.html

Tutorial: View the SAP BTP from 10,000 Meters
Step: STEP 2 What does "as a Service" mean?
Action: Selecting quiz answers and submitting them via "Check answer" button
Behavior: Display of "Technical Error" (due to HTTP 500 after POST to /developers.sap.com/bin/sapdx/tutorials/forms/validate)
Expected behavior: Visual feedback on quiz answers
Additional notes: Check for correct answers could potentially use client-side JS as answers are described in the next step anyways.
Browser: Google Chrome Version 112.0.5615.49 (Official Build) (x86_64), Safari Version 16.3 (186<IP_ADDRESS>.6)

This seems to be an issue with multiple (all?) quizzes

Hey @jmmargo I'm not sure why this is assigned to me - can you please clarify? Thanks!

Christoph, I am unable to reproduce the errors that you have reported.  There might have been a glitch on the day you were testing, but they all seem to work as expected.  I am sorry for the delay in responding to this issue.  Please reopen this if you are still having an issue.


Improve service status handling and systemd support

This PR covers 2 things:

The service state should be set to the value of the consul_service_state variable, but the "restart consul" handler should always be restarted.
systemd logs are collected via journald, so the systemd unit shouldn't explicitly forward logs to a file (removing sh -c also prevents the service from forking)

If you agree I'll do the merge on this one

Sure thing, I can also rebase my commit if you want.

That would be great, thanks for the PR.

@savagegus done

Thanks!


Avoid throwing error in html reporter

I'm not sure why this is happening, and perhaps someone else knows of a better fix.  This one feels kludgey, like a condition that should never happen, and thus is indicating an error elsewhere which should be fixed in a different way.
Nevertheless, this makes my html reporter stop throwing an error on my machine, and doesn't appear to have any other negative side effects.

@isaacs
Are you generating the original coverage data with nyc or using istanbul directly. Your fix here looks like a problem we've had when using the --cache feature in nyc and applying source-maps. We think it's fixed, but if you can provide steps to reproduce, I will dig in.

@jamestalmage I was using nyc, but not with the --cache feature.  It was on a wip branch in progress, and I can't seem to reproduce now, so it's possible that I ended up changing things to avoid triggering the issue.

@isaacs I'm going to start winding down support for istanbul master in favor of istanbul 1.0 that I hope to get out "soon". Right now things are in a weird limbo where the 1.x branch is almost ready but not quite.
In that branch, I have fixed this case by replacing the annotated source code with the stack track, should an exception occur.
https://github.com/istanbuljs/istanbul-reports/blob/master/lib/html/annotator.js#L210


[Question] How to get Laravel Mix(npm) code assist in VSCode?

In Laravel Mix series, you get code assist all the way for Laravel mix, when you reference mix variable.
It doesn't work in my VSCode. Do I need some packages installed and what to get that? I also went through VSCode series, you didn't mention JS intellisense anywhere, besides ES Lint, but I don't think that is related.

Because there no typing files for this package

How does Jeff then gets code completion inside VS Code?


Fix the build on OpenBSD.

We were assuming the wrong types for f_iosize and f_ffree in struct
statfs on OpenBSD.
Fixes #1125

@kpcyrd could you please test this PR?

bors r+


Update proXPN Cask version, all urls and add license

The download of the latest url only contains version 4.0.2.
Based on appcast version 4.0.3 should be already out.
It is possible to use the domain with com, org and biz.
Now it should be consistent and ssl is preferred.

Looks good, thanks! :+1:


New Website Logo: Ubisoft

Hi,
We would like to add Ubisoft to the list of Friends of TypeScript. Here are the info:

Name: Ubisoft
Homepage url: https://ubisoft.com
Brand Guidelines/Licensing:
Logo: https://www.ubisoft.com/en-US/company/press.aspx. You must take the institutionnal color logo from the press kit (download).

Can you confirm that we can ask you to remove the logo at any time please.
Tx,
Francois Martel

Can you confirm that we can ask you to remove the logo at any time please.

Yep, just log a new issue for that. Thanks!


JUnit Local/Remote wording for OperatorExtension is not meaningfull to me

Each time I look back to the JUnit extension, i'm wondering what Local and Remote means.
In both cases, testing is done against a cluster that can be local or remote depending the current kube config.
The real difference between the two is that Local is doing maven in-process testing while Remote is deploying a builded image to the cluster to run the operator to be tested.
I think that InProcessOperatorExtension and KubernetesDeploymentOperatorExtension better describe the actual behavior.

recently this was renamed to LocalOperatorExtension and ClusterOperatorExtension. Does this sound better also for you?

Maybe LocallyRunOperatorExtension and ClusterDeployedOperatorExtension would be even clearer?

I'm personally fine with both, current state (for me sounds expressive enough), also these new ones sounds good.


Start/Resume a User’s Playback

Since a few days it's possible to start playing using the Web API again:
https://developer.spotify.com/web-api/start-a-users-playback/
Do you plan to implement it in the near future?

Even though now not covered by tests and probably a little messy code, I also need exactly that and started implementing it, see here: https://github.com/pzzz/spotify-web-api-java/tree/player-endpoints

Hi, can anyone help me? I m looking for an endpoint to play a users playlist. Is possible at moment? thanks in advance

Check my repo:
https://github.com/Kaiser1989/spotify-web-api-java

Yes, I check for User Play request but it does a request to the playback player. Is there any method on Spotify api to play specific playlist? I don't think so..

Use a Playlist identifier as context uri

Thank you I solved my problem. But I noticed that at first boot of Spotify client app there is not a playback to resume or start, so the first api call to play/resume return Forbidden message. Is that a way to solve it without start manually a playback at first boot? Thank you a lot

Will be part of version 2 on which I'm currently working.

I'll close this. Please follow #102 for updates.

Implemented all Player endpoints in 40c16770c0148cada236b12a32e8853e776d2b7c.

Version 2.0.0-RC1 was just released.


Add process page for JavaScript

For ES6 we pretty much just want to point to airbnb. For ES5 and prior, mention JS the Good Parts and ESLint defaults.

The generator project we have is based on AirBnb settings, but we use 4 spaces instead of two. I would link to https://github.com/atsid/generator-ats-node-webapp/blob/master/generators/eslint/templates/eslintrc

airbnb's js styleguide is MIT, maybe just pull the whole thing and tweak the indention?

That's basically what we did

i mean the docs, not the eslint file

I would prefer to link to the docs and call out where we differ from them

But YOLO, if we want to copy it whole-hog, that's fine too

I had thought we'd just have a blurb about how quality and consistency is important, then link to the airbnb docs.

:+1:
maybe if we end up deviating more or end up adding technology specific guides like react/angular we can fork/copy theirs or something like these people: https://github.com/airbnb/javascript#in-the-wild

It would be nice to include a sample eslint file too, or maybe a comment that the generator pulls down eslint that has the settings we want.

:arrow_up: obviously I did not read Chris's very first comment.

I liked this quote on the Shopify JS guides repo. Lots of people are pretty opinionated about style, but this sums it up well:

Why? All code in any code-base should look like a single person typed it, no matter how many people contributed. If we all follow along with the rules detailed below, we can focus our efforts on more meaningful problems.

I totally agree ... that everyone starts an 'else' line with 'else {'. :-)
I often fantisize about having 'more meaningful problems'.
On Thu, Mar 17, 2016 at 8:31 PM, Nathan Evans<EMAIL_ADDRESS>wrote:

I liked this quote on the Shopify JS guides repo. Lots of people are
pretty opinionated about style, but this sums it up well:
Why? All code in any code-base should look like a single person typed it,
no matter how many people contributed. If we all follow along with the
rules detailed below, we can focus our efforts on more meaningful problems.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/atsid/guides/issues/11#issuecomment-198144271

--
Kind Regards,
Greg Kedge
202.603.0648
<EMAIL_ADDRESS>


Cut 1.6.2

As said on IRC, I'd suggest doing this patch release and looking into 1.7 then without being rushed.
@beorn7 @grobie

Well, given 1.5.3 is out, it doesn't make sense to not release 1.6.2.
:+1:

@fabxc Oops, didn't know that another minor release was coming. I still wanted to do the remote write fix (the error handling bug you discovered). But maybe that can wait until the next one...

@juliusv there was a discussion of stuff to get into 1.7 – just thought we want to have a few days to collect stuff for it.
We also have the k8s namespace configuration pending, which is pretty important now k8s has auth enabled by default. There are also some compression changes in the remote path IIRC. So there's nothing really blocking doing a 1.7 pretty soon.

@fabxc Great.


ZkTable js component needs an optional numeric column at the first place like in excel.

2021.11.22


iox-#1030 Speed up posh tests

Pre-Review Checklist for the PR Author

[x] Code follows the coding style of CONTRIBUTING.md
[x] Tests follow the best practice for testing
[x] Changelog updated in the unreleased section including API breaking changes
[x] Branch follows the naming format (iox-123-this-is-a-branch)
[x] Commits messages are according to this guideline

[x] Commit messages have the issue ID (iox-#123 commit text)
[x] Commit author matches Eclipse Contributor Agreement (and ECA is signed)

[x] Update the PR title

Follow the same conventions as for commit messages
Link to the relevant issue

[x] Relevant issues are linked
[x] Add sensible notes for the reviewer
[x] All checks have passed (except task-list-completed)
[x] All touched (C/C++) source code files from iceoryx_hoofs are added to ./clang-tidy-diff-scans.txt
[x] Assign PR to reviewer

Notes for Reviewer

This PR reduces the test time by more than two minutes. Some CI jobs are now even four minutes faster compared to the latest master build.
This is also getting a bit out of hand since the faster shutdown of RouDi uncovered some issues in the examples which happened previously only sporadically.
Checklist for the PR Reviewer

[x] Commits are properly organized and messages are according to the guideline
[x] Code according to our coding style and naming conventions
[x] Unit tests have been written for new behavior
[x] Public API changes are documented via doxygen
[x] Copyright owner are updated in the changed files
[x] All touched (C/C++) source code files from iceoryx_hoofs have been added to ./clang-tidy-diff-scans.txt
[x] PR title describes the changes

Post-review Checklist for the PR Author

[x] All open points are addressed and tracked via issues

References

Closes #1030
Closes #1359

I think I found the reason (or at least one) for sporadic failures of the integration tests. It seems there was a race between launch_testing and RouDi trying to terminate the applications. I fixed the issue at the cost of additional 200 lines to review.

@mossmaurice I would keep the bindings for the v3 release and only move them afterwards. In general it does not cost us much and especially if we agree on the reduced scope of the v3 release it would not be much extra work. It might even be less work than to remove them. At least from some comments in some issues I have the feeling some people are using Cyclone DDS with the latest iceoryx master and for them it would be nice to have the bindings in the v3 release. Doing the move right after the v3 release would also result in a better transition path for these users.
cc @dkroenke

I would keep the bindings for the v3 release and only move them afterwards. In general it does not cost us much and especially if we agree on the reduced scope of the v3 release it would not be much extra work. It might even be less work than to remove them. At least from some comments in some issues I have the feeling some people are using Cyclone DDS with the latest iceoryx master and for them it would be nice to have the bindings in the v3 release. Doing the move right after the v3 release would also result in a better transition path for these users.

@elBoberido That makes sense to me.
Could please still create the issue for the move to a separate repository and assign it to the v4.0 project? Thanks :pray:

@mossmaurice done with #2030


Add guidance on pair programming best practice.

Adds new guidance around pair programming; what it is, benefits, working effectively, reviewing code.

Thanks MrW! I'll let others jump in with a review or two but just a reminder to pop a link to this page in the nav. The software dev section the best fit imo.


onNodeDragStart method is executed on Node Click

onNodeDragStart method gets executed with onElementClick as well. not able to distinguish between node drag start and node click

We already thought about this and we didn't find a good solution. In the moment when you click on a node we don't know if you will start to drag it or not. When you just drag a node we don't fire the onClick event because we know that you just dragged the node. You could try to set a timeout on drag start and cancel it with the click handler.
I will close this issue because the behavior is intended.

We already thought about this and we didn't find a good solution. In the moment when you click on a node we don't know if you will start to drag it or not. When you just drag a node we don't fire the onClick event because we know that you just dragged the node. You could try to set a timeout on drag start and cancel it with the click handler.
I will close this issue because the behavior is intended.


Roadmap, features etc.

It would be awesome to see what is done and what is to come. Some kind of a checkbox feature list in the readme file.
As example, I have seen that there was some work done on fee, base_max_size, base_min_size and order types like GTC, FOK etc. Some might be easy to implement while others aren't. I am not sure if there is a lot of community interest and if you are willing to accept PRs for certain planned features?
Also do you have any benchmark or maybe a diagram like in the old repo? What information is redis storing?

To do:

Time in force policy support: GTT, Fok, IOC
Performance test
Architecture document
Integrate a digital currency Wallet
ratelimit

Welcome to submit PR, especially web page, or digital currency wallet integration, KYC integration.
Redis is currently used to save ticker, orderbook snapshot, access token and Pub / sub pipeline. Saving orderbook snapshots in redis is not a good solution, because redis is not suitable for storing big value data. However, in order to keep the project deployment simple, redis is used first

In addition, a management panel is required

Would you like to keep things separate?
You have build API and websocket into the engine. How would scaling in that case work?
Currency wallet and KYC also shouldn't be part of the engine but an extra microservice?
What about the engine consuming order events and producing the results and another service writes to the database. Balance management should stay in the engine (risk engine check) and if you have a currency wallet service then balance should get transferred from currency wallet to the engine.
Separation would also make it a bit easier to maintain if the project gets bigger and gains more contributors.
I understand that this would require a few changes to the existing project but I am curious what the goal is?

You're right. It's necessary to split the project into small modules. Therefore, I have split some independent modules into different directories, such as account, order, product, user, OpenAPI, feed and matching engine. These modules should be deployed independently. It is also necessary to split these modules into separate Maven projects, but it will increase the debugging and deployment complexity of the project. I'll consider taking them apart.

I am not a java expert but maybe there is also the possibility of a mono-repository sharing things like user, order model etc.?
So that it is possible to only start certain services or all together using one repository?

I am not a java expert but maybe there is also the possibility of a mono-repository?
So that it is possible to only start certain services or all together using one repository?

Yes, just use Maven module.


Error local variable 'grads' referenced before assignment

Hello!
Can you help me resolve an error?

(py35_rl_gpu) alexshim@AS-PC:~/py_packs/atari-rl$ python main.py --game BreakoutI tensorflow/stream_executor/dso_loader.cc:135] successfully opened CUDA library libcublas.so.8.0 locally
I tensorflow/stream_executor/dso_loader.cc:126] Couldn't open CUDA library libcudnn.so.5. LD_LIBRARY_PATH:
I tensorflow/stream_executor/cuda/cuda_dnn.cc:3517] Unable to load cuDNN DSO
I tensorflow/stream_executor/dso_loader.cc:135] successfully opened CUDA library libcufft.so.8.0 locally
I tensorflow/stream_executor/dso_loader.cc:135] successfully opened CUDA library libcuda.so.1 locally
I tensorflow/stream_executor/dso_loader.cc:135] successfully opened CUDA library libcurand.so.8.0 locally
/home/alexshim/py_packs/gym/gym/init.py:22: UserWarning: DEPRECATION WARNING: to improve load times, gym no longer automatically loads gym.spaces. Please run "import gym.spaces" to load gym.spaces on your own. This warning will turn into an error in a future version of gym.
warnings.warn('DEPRECATION WARNING: to improve load times, gym no longer automatically loads gym.spaces. Please run "import gym.spaces" to load gym.spaces on your own. This warning will turn into an error in a future version of gym.')
2018-05-21 17:05:30 MainThread: Creating network and training operations
Traceback (most recent call last):
File "main.py", line 145, in
tf.app.run()
File "/home/alexshim/anaconda3/envs/py35_rl_gpu/lib/python3.5/site-packages/tensorflow/python/platform/app.py", line 44, in run
_sys.exit(main(_sys.argv[:1] + flags_passthrough))
File "main.py", line 111, in main
trainer = Trainer(create_config())
File "/home/alexshim/py_packs/atari-rl/agents/training.py", line 16, in init
self.global_step, self.agents = factory.create_agents()
File "/home/alexshim/py_packs/atari-rl/networks/factory.py", line 57, in create_agents
global_step, train_ops = self.create_train_ops()
File "/home/alexshim/py_packs/atari-rl/networks/factory.py", line 104, in create_train_ops
for grad, var in grads:
UnboundLocalError: local variable 'grads' referenced before assignment

I am trying to train gpu version.
I will be very grateful.

The GPU branch is WIP and doesn't currently work as you have seen
Please use master


refactor: pipeline mode is purely automated

@Tailp look at how we can even remove more if conditions

Nice :), I have I lot to learn. Thank you very much for showing me this :) .


registry needs a configured S3 endpoint to enable HA mode

Hi.
I'm having the following problem when installing kuik with ReadWriteMany and replica > 1
Error: execution error at (kube-image-keeper/templates/registry-statefulset.yaml:4:3): registry needs a configured S3 endpoint to enable HA mode (>1 replicas), please enable minio or configure an external S3 endpoint
This is my kuik helm values.yaml file (the registry part):
registry:
  affinity:
    podAntiAffinity:
      requiredDuringSchedulingIgnoredDuringExecution:
        - labelSelector:
            matchExpressions:
              - key: app.kubernetes.io/component
                operator: In
                values:
                  - registry
              - key: app.kubernetes.io/name
                operator: In
                values:
                  - kube-image-keeper
          topologyKey: failure-domain.beta.kubernetes.io/zone
  replicas: 2
  nodeSelector:
    intent: system
  pdb:
    create: true
    minAvailable: 1
  persistence:
    enabled: true
    accessModes: "ReadWriteMany"
    size: 1Gi # EFS doesn't have size parameter
    storageClass: efs
  priorityClassName: system-cluster-critical

Have I done something wrong? Or are the docs outdated?
At this link, It says that kuik HA works with ReadWriteMany and EFS.
Also, I'm only using EFS because I'm afraid of transfer costs from S3 to kuik. Does that (transfer) usually happen a lot?

Hi @felipewnp, I'm not the maintainer of this project -- but I did contribute the feature you are talking about :) Are you using the latest version, v1.7.0-beta.2?
The only reason I'm asking is because the old error message used to say the exception you are getting from Helm, whereas if there was a problem it would now say:

registry needs a configured S3 endpoint or a PVC which supports ReadWriteMany to enable HA mode (>1 replicas), please enable minio or configure an external S3 endpoint

Hi @felipewnp , as @thomasjcf29 said, the feature is available in the latest beta version.
Which version are you using ?

Hi @donch @thomasjcf29

[...] the feature is available in the latest beta version [...]

Yep, I missed that. Found in the release notes:

registry-statefulset: added an additionally supported HA mode (50a24dc)

Should I open a PR to mention in the high availability guide that the PVC ReadWriteMany mode is only available in the v1.7.0-beta.2 and above?
I was using the latest v1.6.0 release and will try the v1.7.0-beta.2.

Side note:
Have anyone used S3 persistence with IRSA?
I tried to do that (without setting access_key_id and access_key_secret) but looks like registry doesn't work with IRSA until 3.0.0-alpha.1.
As the issue says, the image registry:3.0.0-alpha.1 indeed works with IRSA but the registry responds with HTTP 404 or HTTP 5XX when a node tries to pull any image, not showing any particularly useful error message in the logs, besides the HTTP 4XX / 5XX codes..


MAINT: remove pyproject.toml

Based on @grlee77's comment.
Also, see https://github.com/scipy/scipy/pull/8735

Codecov Report

Merging #366 into master will decrease coverage by 0.23%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #366      +/-   ##
==========================================
- Coverage   83.78%   83.55%   -0.24%
==========================================
  Files          21       21
  Lines        3251     3332      +81
  Branches      537      567      +30
==========================================
+ Hits         2724     2784      +60
- Misses        466      484      +18
- Partials       61       64       +3

Impacted Files
Coverage Δ

pywt/_extensions/c/wt.template.c
87.38% <0%> (-7.78%)
:arrow_down:

pywt/_extensions/c/common.c
94.44% <0%> (-5.56%)
:arrow_down:

pywt/_extensions/c/wavelets.c
85.74% <0%> (-1.39%)
:arrow_down:

pywt/_extensions/c/convolution.template.c
97.48% <0%> (-1.22%)
:arrow_down:

pywt/_wavelet_packets.py
78.12% <0%> (ø)
:arrow_up:

pywt/_extensions/c/cwt.template.c
97.89% <0%> (+0.04%)
:arrow_up:

pywt/_utils.py
80.76% <0%> (+15.38%)
:arrow_up:

pywt/__init__.py
96.15% <0%> (+19.23%)
:arrow_up:

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 bf7be1f...1ccd66a. Read the comment docs.

Makes sense. These issues will likely be fixed in pip before long; we can then add it back.


There was an error sending this message to your bot: HTTP status code InternalServerError

I found this :
http://stackoverflow.com/questions/42488700/bot-does-not-respond-to-messages
but i have the same issue.
Any help ?

https://github.com/Microsoft/BotBuilder/issues/2258#issuecomment-280506334 might help - you need to configure authentication for outgoing messages too.
In addition, when you see a http server error 500, you should attach your debugger to find the exception.

It works properly when i'm using one user. I can not start botframework simulator twice  ... any idea ?

I don't think the emulator supports multiple bot authentication credentials for the same bot web service endpoint.  is that right @eanders-MS ?

@lonche Can you provide a url to the docs you're quoting? I'm concerned it might be incorrect and would like to fix or clarify it.
@willportnoy It is a limitation of the emulator UI that you can't assign multiple sets of credentials for a single endpoint. As a work around, use multiple entries of the same endpoint, and add a distinguishing query parameter to each, like this:
http://locahost:3978/api/messages?bot1
http://locahost:3978/api/messages?bot2
etc.

Mhm.
I used :

http://locahost:3978/api/bot1/messages

and
http://locahost:3978/api/bot2/messages.
I'l try your solution. Thanks.

@lonche did that work for you? Are you still experiencing this issue, or could you fix it?

No, i didn't fix that. What about you ?

Damn. This is closed again.


update contributing .md

Hey! I would like to add the following in contributing.md
1.Difference between GIT and GITHUB
2.How to clone,fork repository
3.How to create a branch and then use git push to push to repo
4.Create a PR
5.Squash commits in a single issue into one
6, Updating the forked and local repo as the updations are made in the upstream
I would like to work on it as a part of GSSOC'21

can I work on this issue

Can I work on this as a participant of GSSOC21?


Bundler could not find compatible versions for gem "rails-assets-angular-route"

Bundler could not find compatible versions for gem "rails-assets-angular-route":
In Gemfile:
sprangular (= 0.1.0) ruby depends on
rails-assets-angular-route (>= 0) ruby
Could not find gem 'rails-assets-angular-route (>= 0) ruby', which is required by gem 'sprangular (= 0.1.0) ruby', in any of the sources.

@alexandrule Do you have a source setup for https://rails-assets.org? e.g.
source 'https://rails-assets.org'

Thanks @joshnuss for your quick response.
In Gemfile I have source 'https://rails-assets.org'
source 'https://rails-assets.org'
source 'https://rubygems.org'

# Bundle edge Rails instead: gem 'rails', github: 'rails/rails'
gem 'rails', '4.1.8'
# Use sqlite3 as the database for Active Record
gem 'sqlite3'
# Use SCSS for stylesheets
gem 'sass-rails', '~> 5.0'
# Use Uglifier as compressor for JavaScript assets
gem 'uglifier', '>= 1.3.0'
# Use CoffeeScript for .coffee assets and views
gem 'coffee-rails', '~> 4.1.0'
# See https://github.com/rails/execjs#readme for more supported runtimes
# gem 'therubyracer', platforms: :ruby

# Use jquery as the JavaScript library
gem 'jquery-rails'
# Turbolinks makes following links in your web application faster. Read more: https://github.com/rails/turbolinks
gem 'turbolinks'
# Build JSON APIs with ease. Read more: https://github.com/rails/jbuilder
gem 'jbuilder', '~> 2.0'
# bundle exec rake doc:rails generates the API under doc/api.
gem 'sdoc', '~> 0.4.0', group: :doc

# Use ActiveModel has_secure_password
# gem 'bcrypt', '~> 3.1.7'

# Use Unicorn as the app server
# gem 'unicorn'

# Use Capistrano for deployment
# gem 'capistrano-rails', group: :development

group :development, :test do
  # Call 'byebug' anywhere in the code to stop execution and get a debugger console
  gem 'byebug'

  # Access an IRB console on exception pages or by using <%= console %> in views
  gem 'web-console', '~> 2.0'

  # Spring speeds up development by keeping your application running in the background. Read more: https://github.com/rails/spring
  gem 'spring'
end

gem 'spree_core', '2.4'
gem 'spree_api', '2.4'
gem 'spree_backend', '2.4'
gem 'spree_sample', '2.4'
gem 'spree_gateway', github: 'spree/spree_gateway', branch: '2-4-stable'
gem 'spree_auth_devise', github: 'spree/spree_auth_devise', branch: '2-4-stable'
gem 'sprangular', '0.1.0'

I have tried again and now works.


Bun crashes immediately

Describe the bug
Bun crashes immediately when launching
nixpkgs: master @ fab09085df1b60d6a0870c8a89ce26d5a4a708c2
Steps To Reproduce
Steps to reproduce the behavior:

nix run nixpkgs#bun

Expected behavior
Bun launching without error
Additional context
fish: Job 1, 'nix run nixpkgs#bun' terminated by signal SIGILL (Illegal instruction)

Notify maintainers
@DAlperin
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[dogu@nixos:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.89, NixOS, 23.05 (Stoat), 23.05.20230207.fab0908`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.2`
 - channels(root): `"nixos"`
 - nixpkgs: `/nix/store/ismzj4bksv54y4x2b9spfvk9mqhbcwf8-source`

weird,
[dogu@nebula:~]$ nix run github:nixos/nixpkgs?rev=fab09085df1b60d6a0870c8a89ce26d5a4a708c2#bun
Illegal instruction (core dumped)

@usdogu  works for me as well
nix run github:nixos/nixpkgs?rev=fab09085df1b60d6a0870c8a89ce26d5a4a708c2#bun
bun: a fast bundler, transpiler, JavaScript Runtime and package manager for web software.

  run       ./my-script.ts        Run JavaScript with bun, a package.json script, or a bin
  x         bun-repl              Install and execute a package bin (bunx)

  init                            Start an empty Bun project from a blank template
  create    next ./app            Create a new project from a template (bun c)
  install                         Install dependencies for a package.json (bun i)
  add       react                 Add a dependency to package.json (bun a)
  link                            Link an npm package globally
  remove    underscore            Remove a dependency from package.json (bun rm)
  unlink                          Globally unlink an npm package
  pm                              More commands for managing packages

  dev       ./a.ts ./b.jsx        Start a bun (frontend) Dev Server
  bun       ./a.ts ./b.jsx        Bundle dependencies of input files into a .bun

  upgrade                         Get the latest version of bun
  completions                     Install shell completions for tab-completion
  discord                         Open bun's Discord server
  help                            Print this help menu

nix-info -m
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.10, NixOS, 23.05 (Stoat), 23.05.20230209.ae87533`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.2`
 - channels(kirillvr): `""`
 - channels(root): `"nixos"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

@usdogu  have you tried in qemu built using nixos-rebuild build-vm ? that way maybe  something in your configuration vs something with hardware type/version

Fixed with the vm. Closing this since it's an issue in my side

fwiw I am getting this as well using nixos on an i5 using @kilianar example
Thanks
A

@AlexMouton
nixpkgs packages official binaries. try downloading from
https://github.com/oven-sh/bun/releases/tag/bun-v0.5.7
unzip bun-linux-x64.zip
cd bun-linux-x64/
/nix/store/76l4v99sk83ylfwkz8wmwrm4s8h73rhd-glibc-2.35-224/lib64/ld-linux-x86-64.so.2 ./bun
bun: a fast bundler, transpiler, JavaScript Runtime and package manager for web software.

  run       ./my-script.ts        Run JavaScript with bun, a package.json script, or a bin
  x         bun-repl              Install and execute a package bin (bunx)

  init                            Start an empty Bun project from a blank template
  create    next ./app            Create a new project from a template (bun c)
  install                         Install dependencies for a package.json (bun i)
  add       @shumai/shumai        Add a dependency to package.json (bun a)
  link                            Link an npm package globally
  remove    moment                Remove a dependency from package.json (bun rm)
  unlink                          Globally unlink an npm package
  pm                              More commands for managing packages

  dev       ./a.ts ./b.jsx        Start a bun (frontend) Dev Server
  bun       ./a.ts ./b.jsx        Bundle dependencies of input files into a .bun

  upgrade                         Get the latest version of bun
  completions                     Install shell completions for tab-completion
  discord                         Open bun's Discord server
  help                            Print this help menu

if that crashes for you, maybe report issue upstream

thanks @kirillrdy

0.5.7 working

0.5.7 working

and can you reproduce the crash with 0.5.5 ?

also 0.5.7 has landed in nixpkgs
nix run github:nixos/nixpkgs#bun -- --version
0.5.7

does  that work ?

ah wait.
at 0.5.7 bun-linux-64 crashes, bun-linux-64-baseline works.
at 0.5.5...same

anyway, not a nixpkgs thing then

Baseline is built on "Nehalim" support, the non-baseline requires Haswell.
And Haswell is from 4th gen of Intel Core-i$whatever IIRC.
So for any older you probably need baseline. I also think, that nixpkgs should actually use the baseline, as that is much closer on the "do not optimize for a certain microarchitecture" than using haswell.
Alternatively build from source, really using no further micro-architecture related optimisations.


Mupx deploy failed verifying deploment

getting errors on mupx deploy
npm ERR! fetch failed https://github.com/meteor/node-source-map-support/tarball/1912478769d76e5df4c365e147f25896aee6375e
	npm ERR! fetch failed https://github.com/meteor/node-eachline/tarball/ff89722ff94e6b6a08652bf5f44c8fffea8a21da
	npm ERR! fetch failed https://github.com/meteor/node-source-map-support/tarball/1912478769d76e5df4c365e147f25896aee6375e
	npm ERR! fetch failed https://github.com/meteor/node-eachline/tarball/ff89722ff94e6b6a08652bf5f44c8fffea8a21da

Its okay now. As github has recovered. But how to avoid this if github goes down again?

Better warnings that access to the Internet or resources is not working would be nice.

Or maybe download the resources locally so it won't depend on github


Do not handle RuntimeException specifically

Doing so causes RuntimeExceptions to lack the nice wrapper and breaks the build scan plugin tests in dotcom.

@bot-gradle test this please.

OK, I've already triggered ReadyForMerge build for you.

As expected, this is not so simple. The changes now have caused some failures to be reported more verbosely:
* What went wrong:
Failed to notify task execution graph listener.
> Credentials required for this build could not be resolved.
   > The following Gradle properties are missing for 'ivy' credentials:
       - ivyUsername
       - ivyPassword

...instead of:
* What went wrong:
Credentials required for this build could not be resolved.
 > The following Gradle properties are missing for 'ivy' credentials:
     - ivyUsername
     - ivyPassword

@bot-gradle test this

OK, I've already cancelled the old build and triggered a new ReadyForMerge build for you.

I wonder if we should wrap even RuntimeExceptions in BuildOperationInvocationExceptions (all except Errors).

@bot-gradle test QF

OK, I've already triggered QuickFeedback build for you.


Write a clean script that performs fusion

Here: https://github.com/castorini/anserini/blob/master/docs/experiments-covid.md
from trectools import TrecRun, TrecEval, fusion

r1 = TrecRun("runs/run.covid-r1.abstract.query+question.bm25.txt")
r2 = TrecRun("runs/run.covid-r1.full-text.query+question.bm25.txt")
r3 = TrecRun("runs/run.covid-r1.paragraph.query+question.bm25.txt")

fused_run = fusion.reciprocal_rank_fusion([r1,r2,r3])
fused_run.print_subset("runs/run.covid-r1.fusion1.txt", topics=fused_run.topics())

I've been using the above snippet to perform run fusion using RRF, etc. We should codify this into a clean script.
@x65han can you do this?

Yeap, I have a script for this.


Adding 1.8 auth config to test files and removing 1.5

Description
Adding 1.8 authorization to test files and removing 1.5
GitHub Issues
List the GitHub issues impacted by this PR:

GitHub Issue #

https://github.com/dell/csm/issues/885

Checklist:

[x] I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
[x] I have verified that new and existing unit tests pass locally with my changes
[x] I have not allowed coverage numbers to degenerate
[x] I have maintained at least 90% code coverage
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have maintained backward compatibility

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

[x] Ran unit-tests
[x] Successfully installed authorization with Operator using config

have you done anything beyond runing unit tests to verify your changes?

I forgot to include that I successfully installed authorization using the new config. I've included that in the description now.
But also, the only real change in the config file from 1.7 to 1.8 is just the version update. Github is showing the other changes because it thinks I changed the 1.5 config to the 1.8 config but I had actually just removed the 1.5 config and copied the 1.7 config to make the 1.8 config.


Out of the box compile experience with Visual Studio 2015: TypeScript Error

This error is thrown when doing an out of the box compile of Nether from within Visual Studio 2015:
Error	MSB6006	"tsc.exe" exited with code 1.	Nether.Web	C:\Program Files (x86)\MSBuild\Microsoft\VisualStudio\v14.0\TypeScript\Microsoft.TypeScript.targets	214
This problem seem to be fixed (as a work around) by re-installing the TypeScript Tools for Visual Studio

This seems to be resolved with VS 2017


Add updated Cohere Embedding blueprint

Description
[Describe what this change achieves]
Updates Cohere blueprint documentation, merging the two existing blueprints into one, as both were made to integrate the same /embed API call but with different model params. Clarifies some of the steps, and adds steps to optionally add ingestion pipeline.
Issues Resolved
N/A
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

There is also some documentation here: https://opensearch.org/docs/latest/ml-commons-plugin/remote-models/connectors/, I'd like to update it because some of the info is out of date - can someone point me to where that lives?

@Zhangxunmt @ylwu-amzn Went ahead and updated the PR based off your suggestions - let me know if anything else needs fixing. Also just want to make sure my comment about the other set of docs has some visibility.

@ylwu-amzn done


Side panel visual tweaks

What will this achieve?
cuter look

How will success be measured?
both items addressed, design review

Additional information

[x] The scrollbar goes out of view under the footer. Take a look at CreateTearsheet's body scroll for comparison.

[x] On entry the scrollbar appears last and squeezes the content, making the form fields resize.

Fixed in https://github.com/carbon-design-system/ibm-cloud-cognitive/pull/1141

Complete


Indicat that showsScale prop is Apple Maps only

Does any other open PR do the same thing?
no
What issue is this PR fixing?
See #2757
How did you test this PR?
no test

Thanks for the PR. Would you mind also adding * @platform ios within the source code here?

@christopherdro done 😊

@gkueny ❤


BadHttpException not catchable?

I've just tried catching the exception thrown in Accept when the header/version is all wrong. It seems as though this exception is not catchable, either via a custom handler through dingo, or via Laravel.
Any ideas why?

Hm, it should be catchable. Let me take a look see. I'll get back to you shortly.

Hey Kirk it should be catchable now with the app('api.exception')->register() method, it will also, by default, return the response as JSON.
Basically it's now run through the API exception handler.


Multi-citizen assignment box doesn't resize with citizen icons

See below:

By messing with values in PlotInfo.xml I was able to get it to look like this:

I am not very experienced though, so I have no idea if changing those values could have broken anything. I changed some of the icon, numbers, and lines offsets as well as alignments. Here's a link to the changed PlotInfo.xml


Comment says "Define Google OAuth2" in the Twitter strategy file

https://github.com/GeekyAnts/express-typescript/blob/596420ab7f8e47a18afac0fc05ab037e70bdca68/src/services/strategies/Twitter.ts#L2

@HJassar PR #14 fixes your mentioned issue. Please review.


Add Build Instructions to README.md

Changes:

Added instructions of how to build from source to the README.md

To view changes:

https://github.com/Shanedell/daffodil-vscode/tree/issue_23_build_in_readme

Closes #23

What is the status of this?


CRDs fail to deploy on k8s 1.18

I am getting these:
Error: failed to install CRD crds/sparkoperator.k8s.io_scheduledsparkapplications.yaml: CustomResourceDefinition.apiextensions.k8s.io "scheduledsparkapplications.sparkoperator.k8s.io" is invalid: [spec.validation.openAPIV3Schema.properties[spec].properties[template].properties[executor].properties[initContainers].items.properties[ports].items.properties[protocol].default: Required value: this property is in x-kubernetes-list-map-keys, so it must have a default or be a required property, spec.validation.openAPIV3Schema.properties[spec].properties[template].properties[executor].properties[sidecars].items.properties[ports].items.properties[protocol].default: Required value: this property is in x-kubernetes-list-map-keys, so it must have a default or be a required property, spec.validation.openAPIV3Schema.properties[spec].properties[template].properties[driver].properties[sidecars].items.properties[ports].items.properties[protocol].default: Required value: this property is in x-kubernetes-list-map-keys, so it must have a default or be a required property, spec.validation.openAPIV3Schema.properties[spec].properties[template].properties[driver].properties[initContainers].items.properties[ports].items.properties[protocol].default: Required value: this property is in x-kubernetes-list-map-keys, so it must have a default or be a required property]

This happens with both releases spark-on-k8s-operator-1beta2-1.2.1-3.0.0 as well as spark-on-k8s-operator-1beta2-1.2.0-3.0.0
Using:
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.4", GitCommit:"d360454c9bcd1634cf4cc52d1867af5491dc9c5f", GitTreeState:"clean", BuildDate:"2020-11-12T01:09:16Z", GoVersion:"go1.15.4", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"18+", GitVersion:"v1.18.8-eks-7c9bda", GitCommit:"7c9bda52c425d0d56d7b93f1377a826b4132c05c", GitTreeState:"clean", BuildDate:"2020-08-28T23:04:33Z", GoVersion:"go1.13.15", Compiler:"gc", Platform:"linux/amd64"}

This seems to be the underlying issue: https://github.com/kubernetes/kubernetes/issues/91395

I came across this problem also. Has been fixed in a previous release with https://github.com/GoogleCloudPlatform/spark-on-k8s-operator/commit/5b2fe9218108baa6456eb3472196d5eb55b41efd
Essentially to get this working with the chart checked into this repo, find anywhere where
x-kubernetes-list-map-keys:  contains - protocol and add it into the required
e.g.
                          ports:                             items:                               properties:                                 containerPort:                                   format: int32                                   type: integer                                 hostIP:                                   type: string                                 hostPort:                                   format: int32                                   type: integer                                 name:                                   type: string                                 protocol:                                   type: string                               required:                               - containerPort                               - protocol                               type: object                             type: array                             x-kubernetes-list-map-keys:                             - containerPort                             **- protocol**                             x-kubernetes-list-type: map
I have done this on a local version of the chart and been able to get through the validation errors. It is clear from the commit message referenced above that this was a manual fix and perhaps there is a better way to address this. If there is no more permanent workaround and would be useful for me to open a PR to do this for, I am happy to do so.

Ran into this as well and cherry-picking https://github.com/GoogleCloudPlatform/spark-on-k8s-operator/commit/5b2fe9218108baa6456eb3472196d5eb55b41efd does fix the issue thou the latest crd on master is broken.

This has been fixed in https://github.com/GoogleCloudPlatform/spark-on-k8s-operator/pull/1084.


flowey: support multiple rootfs files when building openhcl

This makes it possible to dynamically add files to openhcl without having to write code to concatenate separate rootfs files each time.

Yeah I'll add a build-igvm option in a later change. This is just paving the way.


best ways to modify this theme?

Hi @cboettig , thanks for this theme! I was wondering how best to modify colors and the like -- is there a standard bootstrap variables partial hiding here somewhere? THank you!

You "just" have to edit and recompile the SASS variables, see: https://github.com/cboettig/hugo-now-ui/blob/master/static/sass/now-ui-kit/_variables.scss

ah.  So would that be sass --scss  sass/now-ui-kit.scss  css/now-ui-kit.css? I usually just use node-sass on other people's
projects :-) .

hehe, Iook it up every time too.  Or sass --watch sass:css I think.


add bond tracking?

best place to get data? csv?
treasurydirect? other?
https://eworkpaper.com/myibonds.php ?
https://www.bogleheads.org/forum/viewtopic.php?t=275819
eyebonds.info
and look more...

Since this issue was opened, we've added support for Ibond tracking


Incorrect Usage: flag provided but not defined: -docker-network

I tried to use --docker-network <network-id> to connect to docker network when I use sam local to invoke my lambda function locally. but it said Incorrect Usage: flag provided but not defined: -docker-network. it seems that --docker-network is not a valid OPTION.
when I type sam local invoke -h, I see these options:
USAGE:
sam local invoke [command options]
OPTIONS:
--template value, -t value               AWS SAM template file (default: "template.[yaml|yml]") [$SAM_TEMPLATE_FILE]
--log-file value, -l value               Optional. Logfile to send runtime logs to
--env-vars value, -n value               Optional. JSON file containing values for Lambda function's environment variables.
--event value, -e value                  JSON file containing event data passed to the Lambda function during invoke
--debug-port value, -d value             Optional. When specified, Lambda function container will start in debug mode and will expose this port on localhost. [$SAM_DEBUG_PORT]
--docker-volume-basedir value, -v value  Optional. Specifies the location basedir where the SAM file exists. If the Docker is running on a remote machine, you must mount the path where the SAM file exists on the docker machine and modify this value to match the remote machine. [$SAM_DOCKER_VOLUME_BASEDIR]
Could anyone please help me on this?
Thanks!

Hi @rickypotato, this feature has been added to our development branch, but has not yet been released to NPM.
You can either install SAM Local via the Go instructions on the README, or wait until our next release to NPM (shouldn't be long).


Display latest freshness check for all sources, irrespective of job, in the Data Sources UI

Describe the feature
The Source Freshness UI (when accessed from the Data Sources link in the main dbt Cloud menu) would display the status of each source as of the most recent job that tested its freshness.
Only sources currently in the project manifest need to be included. A link at the top of each environment's main page – https://cloud.getdbt.com/#/accounts/X/projects/X/environments/X/ – would be convenient, too.
Additional context
Currently, data in the Source Freshness UI is populated only for those sources --selected in the most recently completed job containing a snapshot-freshness statement. This means that in projects with several "freshness SLAs", there's no one-stop, comprehensive view of source freshness for the whole project.
Consider a project with the following source freshness SLAs and scheduled jobs:

Source
Freshness SLA
Job
Schedule

source_a
Hourly
job_a: dbt source snapshot-freshness --select source_a
0,30 * * * *

source_b
Daily by 0600
job_b: dbt source snapshot-freshness --select source_b
15 6 * * *

If a user views the Source Freshness UI between (roughly speaking) 0615 and 0630, they'll see the freshness status for source_b but UNTRACKED for source_a. At any other time, source_b will display as UNTRACKED, and only source_a's status will be available.

Describe alternatives you've considered
Browse the Job History to find the last run of each Source Freshness job, and view the results of each job separately.
Who will this benefit?
Any project with multiple source freshness SLAs, especially as node selection syntax enhancements (via #2987) make it easier to define more granular source freshness checks.

@ljhopkins2 Thanks for opening! I totally agree with what you've laid out here.
Today, the Source Freshness UI in dbt Cloud is powered by the sources.json artifact, grabbed from the most recent run of the configured job. In order to grab all metadata from the selected environment, merging information from multiple jobs into a single view, we'd need some kind of Metadata API... the foundations of which are soon to be in GA. (If you want a sneak peak at some of the exciting new documentation: https://github.com/dbt-labs/docs.getdbt.com/pull/713.)
As those docs (will) state:

Please note that at present, queries to the Metadata API exclusively return metadata about the most recent Run for a given dbt Cloud Job. We are continuously expanding the capabilities of the metadata API and we welcome your feedback and suggestions at<EMAIL_ADDRESS>
So the limitation you've described will remain in place for the immediate future. That said, I believe we have the right foundations in place to support exactly the kind of experience you're after.
I am going to close this issue, only insofar as it's not a code change we'll be making in this repo (dbt Core). But it's definitely on our minds (cc @barryaron)


As an admin, I want a bulk import of MARC metadata from Gale/ECCO so that I can view and search each record by its metadata

We want to pull the following fields from the MARC records: title, subtitle, sort title, place of publication, publisher, and pub date

@rlskoeser how is author field formatted in the MARC records? I'm wondering if it's worth pulling author, too, if it's formatted "last name, first name" so that it matches the format of the HT records.

@mnaydan yes, planning to pull author from the MARC metadata as well — it should get us a format closer to what we've got already.

Here's the output from the most recent run of the gale import script.  (@mnaydan I think I'm still using an old version of your CSV without the most recently corrected IDs).
I'm surprised by how many MARC records we're missing — it didn't seem like that many when I was testing, but when I ran the full import again locally I got similar results. We may need to follow up with Gale to find out what could cause this (could it be a matter of when PUL purchased the marc records?)
I think the character parse errors are because I switched back to using binary MARC records instead of JSON — @mnaydan if you can think of any records with special characters to test, that would be helpful. If anything is getting corrupted we can try switching back to the JSON version of the MARC records.
$ ./manage.py gale_import -c /tmp/ECCO_compiled_non-excerpts_fixed_IDs_6.10.21.csv -v 0
MARC record not found for CB0132859609
MARC record not found for CW0112106158
MARC record not found for CW0111720541
MARC record not found for CW0114229753
MARC record not found for CW0114728738
Unable to parse character 0xcc in g0=66 g1=69
MARC record not found for CW0114704032
MARC record not found for CW0110970244
MARC record not found for CW0110970663
MARC record not found for CW0111406390
MARC record not found for CW0109771665
MARC record not found for CW0113390248
MARC record not found for CW0110986375
MARC record not found for CW0110004195
MARC record not found for CW0110433388
MARC record not found for CW0114905944
MARC record not found for CW0115390412
MARC record not found for CW0113836889
MARC record not found for CW0113837303
MARC record not found for CW0114032230
MARC record not found for CW0114032713
MARC record not found for CW0115236604
MARC record not found for CW0115419546
MARC record not found for CW0115420029
MARC record not found for CW0115963418
MARC record not found for CW0119571809
MARC record not found for CW0119572297
MARC record not found for CW0120106050
MARC record not found for CW0120106500
MARC record not found for CW0120576252
MARC record not found for CW0121009324
MARC record not found for CW0120209174
MARC record not found for CW0116527799
MARC record not found for CW0116528246
MARC record not found for CW0117248465
MARC record not found for CW0117333680
MARC record not found for CW0112168756
MARC record not found for CW0112558307
MARC record not found for CW0112927708
MARC record not found for CW0113057109
MARC record not found for CW0113336035
MARC record not found for CW0113336519
MARC record not found for CW0113337031
MARC record not found for CW0113337536
MARC record not found for CW0110165242
MARC record not found for CW0117289150
MARC record not found for CW0114204864
MARC record not found for CW0110667966
Unable to parse character 0x80 in g0=66 g1=69
MARC record not found for CW0117022614
MARC record not found for CW0110636874
MARC record not found for CW0113694901
Unable to parse character 0xcc in g0=66 g1=69
MARC record not found for CW0113017309
MARC record not found for CW0112375148
MARC record not found for CW0112613319
MARC record not found for CW0114850590
MARC record not found for CW0115756532
MARC record not found for CW0114265763
MARC record not found for CW0113449473
MARC record not found for CW0114721722
MARC record not found for CW0110966909
MARC record not found for CW0111124308
MARC record not found for CW0111124419
MARC record not found for CW0115498273
MARC record not found for CW0115953052
MARC record not found for CW0114828122
MARC record not found for CW0112639507
MARC record not found for CB0129094722
MARC record not found for CW0114590198
MARC record not found for CW0115702497
MARC record not found for CW0115702857
Unable to parse character 0xcc in g0=66 g1=69
Unable to parse character 0x80 in g0=66 g1=69
MARC record not found for CW0112348310
MARC record not found for CW0112348834
MARC record not found for CW0113550692
Unable to parse character 0xcc in g0=66 g1=69
Unable to parse character 0x80 in g0=66 g1=69
Unable to parse character 0xcc in g0=66 g1=69
Unable to parse character 0x80 in g0=66 g1=69
MARC record not found for CW0113652049
MARC record not found for CW0115232948
MARC record not found for CW0110975037
MARC record not found for CW0124787241
MARC record not found for CW0111162361
MARC record not found for CW0111646406
MARC record not found for CW0114894389
MARC record not found for CW0115378212
MARC record not found for CW0111185737
MARC record not found for CW0116297776
MARC record not found for CW0100721039
MARC record not found for CW0115687820
MARC record not found for CW0113483218
Unable to parse character 0xcc in g0=66 g1=69
Unable to parse character 0x80 in g0=66 g1=69
MARC record not found for CW0124737709
MARC record not found for CW0119570768
MARC record not found for CW0103274182
MARC record not found for CW0114518411
MARC record not found for CW0115462979
MARC record not found for CW0114695017
MARC record not found for CB0129117138
MARC record not found for CB0128602753
MARC record not found for CB0131704013
MARC record not found for CW0120334675
MARC record not found for CW0124749360
MARC record not found for CW0124748989
MARC record not found for CW0115984903
MARC record not found for CW0124836956
Unable to parse character 0xcc in g0=66 g1=69
Unable to parse character 0x80 in g0=66 g1=69
MARC record not found for CW0114369233
MARC record not found for CW0116719554
MARC record not found for CB0130572662
MARC record not found for CW0110724021
MARC record not found for CB0128465842
MARC record not found for CW0116770337
MARC record not found for CW0111173803
MARC record not found for CW0113482794
MARC record not found for CW0117561907
MARC record not found for CB0128277182
MARC record not found for CW0125896591
MARC record not found for CW0125630970
Error getting item information for CB132045539
MARC record not found for CB0131407349
MARC record not found for CB0131712342
MARC record not found for CB0131407895
MARC record not found for CB0129117517
MARC record not found for CW0110986839
MARC record not found for CW0113825227
MARC record not found for CW0116119464
MARC record not found for CW0111974384
MARC record not found for CW0111974989
MARC record not found for CW0114888110
MARC record not found for CW0113923438
MARC record not found for CW0115683484
MARC record not found for CW0116702915
MARC record not found for CW0116299725
MARC record not found for CW0116298721
MARC record not found for CW0116815787
MARC record not found for CW0116296895
MARC record not found for CW0116296101
MARC record not found for CW0116293725
MARC record not found for CW0116295328
MARC record not found for CW0116702026
Error getting item information for CB128058273
MARC record not found for CW0115693865
MARC record not found for CW0114894562
MARC record not found for CW0118347262
MARC record not found for CW0113617765
MARC record not found for CW0113954378
MARC record not found for CW0100161757
MARC record not found for CW0100964250
MARC record not found for CW0103878094
MARC record not found for CW0101866153
MARC record not found for CW0101866791
MARC record not found for CW0100161262
MARC record not found for CW0100166317
Error getting item information for CB110589871
MARC record not found for CW0115179207
MARC record not found for CW0115953753
MARC record not found for CW0102862950
MARC record not found for CW0102747242
MARC record not found for CW0102803994
MARC record not found for CW0102804392
Unable to parse character 0xcc in g0=66 g1=69
Unable to parse character 0x80 in g0=66 g1=69
Unable to parse character 0xcc in g0=66 g1=69
Unable to parse character 0x80 in g0=66 g1=69
Error getting item information for CB129347115
MARC record not found for CW0115383022
MARC record not found for CW0115348005
MARC record not found for CW0102294848
MARC record not found for CW0113867877
MARC record not found for CW0114448536
MARC record not found for CW0115722544
MARC record not found for CW0113726727
MARC record not found for CW0113330715
MARC record not found for CB0128515985
MARC record not found for CB0128516952
MARC record not found for CB0128516493
MARC record not found for CW0110094668
MARC record not found for CW0110094982
MARC record not found for CW0111145472
MARC record not found for CW0110163815
MARC record not found for CW0110164224
MARC record not found for CW0115526337
MARC record not found for CW0113056418
MARC record not found for CW0113379207
MARC record not found for CW0112646748
MARC record not found for CW0114853467
Error getting item information for CB125450132
MARC record not found for CW0124563996

Processed 1,182 items for import.
Imported 1,177; skipped 0; 5 errors; imported 385,155 pages.

Some special characters are getting corrupted: accented e ("Dubos, abb©♭ (Jean-Baptiste), 1670-1742") and ash ("the fables of ©sop") are the most frequent culprits. Ampersand, parenthesis, asterisks are holding. It's only a handful (maybe 20?), so I feel badly having you switch again... @rlskoeser let me know what you think.
Other than that, searching by metadata in backend and frontend works great (for the items that have MARC records).
I reached out to Ray today about the missing MARC records.

Thanks for checking the special characters, I was afraid of that. It isn't a huge effort for me to switch to JSON and see if that resolves these problems. (I already wrote code that can do it, just have to get it back from the git history!)

@mnaydan updated test with a fresh import of Gale content from your most recent spreadsheet (6.22.21).
I did get a few errors on missing MARC records; LMK if you want me to investigate these further:
MARC record not found for CB0126228345/N009409
MARC record not found for CB0132944166/T078405
MARC record not found for CB0131704013/T032596
MARC record not found for CB0131389492/T032596
MARC record not found for CB0132926890/N034700
MARC record not found for CB0130984232/N002639
MARC record not found for CB0132774697/T053828
MARC record not found for CB0132841903/N029902
MARC record not found for CB0132652546/T097528
MARC record not found for CB0126231877/T005538

@rlskoeser blarg. Thank you. Let me look real quick before you investigate.

@mnaydan for testing, in addition to checking the character encoding problem, please also check that the ESTC id is set as the record id and volume # is set for items that are multivolume.

@rlskoeser all those IDs are correct and contained in both my and Ray's spreadsheet. (Only difference is that the title for CB0126231877 should be "The dunciad. An heroic poem. In three books" but I don't believe you're using title from the spreadsheet.) I'm not sure what would be causing them to error. Might be good for you to investigate.

@rlskoeser all those IDs are correct and contained in both my and Ray's spreadsheet. (Only difference is that the title for CB0126231877 should be "The dunciad. An heroic poem. In three books" but I don't believe you're using title from the spreadsheet.) I'm not sure what would be causing them to error. Might be good for you to investigate.

Thanks for checking — I meant to point out and forgot, I can tell from the error message that we are getting an ESTC id number for each of those. The error is happening when I try to load the MARC record for that id. I also notice now that they are all CB numbers — so all ECCO2, right? Maybe I can generate a list of all 001 identifiers in the MARC records we were given so we can check and ask Ray if necessary.

@rlskoeser they are all CB / ECCO2, yes. I don't know what 001 identifiers are but that sounds good to me. Is it possible MARC records don't exist for those items?

I'm wondering if somehow they were added to ECCO2 after Princeton purchased the MARC records that we're working from — I don't know how Gale operates enough to know if that's plausible or not!  (001 identifier is the primary id in MARC, and corresponds to the ESTC ids that Ray gave us to use)

Special characters fixed!
ESTC is set as record ID in backend. Volume number appears for multivolume works on frontend and backend.
Re-tested search by metadata on frontend and backend (author, title, and fielded searches in keywords) and it works well!
I'm going to keep open until we get answers from Ray about the 10 missing MARC records, though.