Reduce size of guide dev server images

Description
This PR suggests to reduce the size of the static/docs/5.3/assets/img/guides/*-dev-server*.png images which were 2280px width.
I've chosen arbitrarily a value of 1000px width (like some in this same directory) and used https://www.iloveimg.com online tool.
This PR can be closed if this topic has already been mentioned/tackled before or if we want to use other values or tools to do that.
If this PR is merged, we probably want to update them in our gh-pages branch.
Motivation & Context
Save some trees.
Type of changes

Enhancement

Checklist

[x] I have read the contributing guidelines

Live previews

https://deploy-preview-37741--twbs-bootstrap.netlify.app/docs/5.3/getting-started/parcel/
https://deploy-preview-37741--twbs-bootstrap.netlify.app/docs/5.3/getting-started/vite/
https://deploy-preview-37741--twbs-bootstrap.netlify.app/docs/5.3/getting-started/webpack/

I'll have a pass at those. None of the tools out there achieve the best compression AFAICT.

...and you can easily see the above from the fact that calibre action optimized them 😛

EDIT2: the gh-pages branch is only updated when a release is made, unless it's something that doesn't apply to main or is super-important.

When I said we probably want to update gh-pages, I talked about the previous versions already released where the guides (and their images) were already there. Not talking about the 5.3 which will be updated when the real stable 5.3 will be released.

Yeah, we don't touch the old versions unless it's something really important, which in this case it isn't.

I can re-export the source at a specific size. I'd suggest a minimum of 2x the widest content width (805px), which is 1610px. That way we keep retina quality 2x. We could do a 1x version though for smaller ones. Thoughts @XhmikosR?

Sounds good to me, @mdo. I'll compress the newly exported images then.


Artifactory ha nginx bugfix

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:

/ok-to-test

/retest

@rjkernick
Test is failing because of : I0501 21:51:27.549]   9m		3m		22	default-scheduler			Warning		FailedScheduling	0/5 nodes are available: 5 MaxVolumeCount.

/retest

/retest

/lgtm


#237 not completed, disable deno color output (#242)

Description
Resolve #242

Now quick fixed, will be improve at other issue & PR.


ES6 modules not found when loaded over network

Issue Type: Bug
ES6 modules can be loaded over HTTP(S) like this:
import { LitElement, html } from<EMAIL_ADDRESS>
This works fine in any modern browser since it is standard ES6 and I find it very convenient for small projects because it allows you to completely avoid the complexity involved with npm and packagers.
However, VS Code complains with a red underline and the mouseover error message:

Cannot find module<EMAIL_ADDRESS>ts(2307)

It would be great if VS Code did not complain about this (since it is possible to find the module) and even better of course, if the module could be resolved correctly like locally-loaded modules.
VS Code version: Code 1.31.0 (7c66f58312b48ed8ca4e387ebd9ffe9605332caa, 2019-02-06T08:51:24.856Z)
OS version: Linux x64 4.20.6-arch1-1-ARCH

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-5600U CPU @ 2.60GHz (4 x 3084)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: disabled_softwaresurface_synchronization: enabled_onvideo_decode: unavailable_offwebgl: enabledwebgl2: enabled

Load (avg)
1, 1, 1

Memory (System)
11.60GB (0.27GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (20)

Extension
Author (truncated)
Version

swagger-viewer
Arj
2.2.0

rest-client
hum
0.21.1

plantuml
jeb
2.10.3

Kotlin
mat
1.7.0

rainbow-csv
mec
1.0.0

python
ms-
2019.1.0

openhab
ope
0.4.1

vscode-template-literal-editor
pli
0.9.0

java
red
0.38.0

vscode-xml
red
0.3.0

LiveServer
rit
5.4.0

es6-string-html
Tob
1.8.6

vscodeintellicode
Vis
1.1.3

vscode-apielements
vnc
0.6.7

vscode-java-debug
vsc
0.16.0

vscode-java-dependency
vsc
0.3.0

vscode-java-pack
vsc
0.5.0

vscode-java-test
vsc
0.14.0

vscode-maven
vsc
0.14.2

vscode-wakatime
Wak
1.2.5

(1 theme extensions excluded)

Not pulling arbitrary http endpoints is intentional. You can declare the module using its URL:
// .d.ts file
declare module<EMAIL_ADDRESS>{
    export const LitElement: any, html: any;
}

// Elsewhere
import { LitElement, html } from<EMAIL_ADDRESS>

Thx for the tip about declaring a module by its URL. It helps, but in order for it to be really useful, you typically have to add quite a bit more type information than any, e.g.
declare module<EMAIL_ADDRESS>{
    export declare class LitElement extends HTMLElement {
        requestUpdate(name?: PropertyKey, oldValue?: unknown): Promise<unkown>;
        ...
    }
    export const html: any;
    ...
}

I still think it would be great if there was an option to turn on pulling http endpoints just like the ES2015 runtime system in a browser does, since it would make the kind of development/prototyping that I described above much easier and even more productive.

@RyanCavanaugh Is there a way to pull in the type definitions while simultaneously relying on a network import? I'm trying to do the exact same thing as @hindsholm. There's no reason to pull in all the code when it can be served via CDN.

Something that I would like to call out is that the package in question is authored in TypeScript and publishes its type declarations alongside the other build output. Why should we re-declare all of those types?
I also arrived at the solution of loading from a CDN after reading the following advice given in a separate thread:

Our general take on this is that you should write the import path that works at runtime, and set your TS flags to satisfy the compiler's module resolution step, rather than writing the import that works out-of-the-box for TS and then trying to have some other step "fix" the paths to what works at runtime.
We have about a billion flags that affect module resolutions already (baseUrl, path mapping, rootDir, outDir, etc). Trying to rewrite import paths "correctly" under all these schemes would be an endless nightmare.

Except taken from this older thread.
To lay out the scope of my problem:

Say I'm trying to write some TypeScript that will be transformed to JavaScript and run in a browser
My code depends on some external modules that are available from npm or unpkg.com
Browsers support full-URL, cross-domain module specifiers at runtime
tsc supports Node.js-style resolution, but cannot re-write Node.js module specifiers in the output
Also, tsc won't recognize modules that are imported from a CDN

So my options to write TypeScript and deploy to the browser are:

Import everything by relative paths in my module and ...

...make sure none of my transitive dependencies depend on Node.js-style module resolution or
...transform Node.js-style name specifiers in my source tree ahead of time

Note: unpkg.com does this for us with the ?module query param

Use a bundler or similar transform to fix module specifiers in the compiled output

...and what happens to my source maps in this case?

It seems like something important is missing here.

(the missing thing may be Import Maps, but those don't technically exist yet)


User blocked the conversation with the bot

I am using the example in your README.md file to test a webhook created with Workflows. I changed the image url, EMAIL and NAME strings to my corresponding company email and name, and a test image url. The workflow is failing and I am getting this error in the failure report:
Request to the Bot framework failed with error: '{\"error\":{\"code\":\"ConversationBlockedByUser\",\"message\":\"User blocked the conversation with the bot.\"}}'.

Any ideas of what is going on or what should I do to make this work?
Thanks in advance.

I just tested my example code from scratch:
from msteamsapi import TeamsWebhook, AdaptiveCard, Container, FactSet, ContainerStyle, TextWeight, TextSize

webhook = TeamsWebhook('https://prod-87.westeurope.logic.azure.com:443/workflows/blabla/triggers/manual/paths/invoke?api-version=2016-06-01&sp=%2Ftriggers%2Fmanual%2Frun&sv=1.0&sig=blabla')

card = AdaptiveCard(title='card title', title_style=ContainerStyle.DEFAULT)
card.add_background(url="https://github.com/ALERTua/msteamsapi/raw/main/tests/background.png")

container = Container(style=ContainerStyle.DEFAULT)

<EMAIL_ADDRESS>'me', add_text_block=True)
mention_tag =<EMAIL_ADDRESS>'this dude')

container.add_image("https://github.com/ALERTua/msteamsapi/raw/main/tests/background.png", "image alt text")
container.add_text_block(
    'multiline\n\ntext\n\nmention 1: %s' % mention_tag,
    size=TextSize.DEFAULT, weight=TextWeight.DEFAULT, color="default"
)

factset = FactSet(('fact 1', 'fact 1 value'))
factset.add_facts(('fact 2', 'fact 2 value'), ('fact 3', 'fact 3 value'))
container.add_fact_set(factset)

card.add_container(container)

for i, url in enumerate(['https://google.com/', 'https://goo.gle']):
    card.add_url_button('url %s' % i, url)

webhook.add_cards(card)
webhook.send()

and everything works like a charm

did you create a webhook for a public channel? a standard one?

Ok. Got it working now. It was because I had the channel restrictions on like this:

I thought the connector was the one posting the actual card, but it seems to be posting the card from my email, so I had to turn that setting off, or I assume giving specific posting permissions to my user would also work.


Add Analog AD559x MFD driver I2C bus access

This PR renames and refactors the existing code and adds I2C support.

I only have access to AD5593 thus have not tested AD5592.
@bbilas would you be able to do this?

I only have access to AD5593 thus have not tested AD5592. @bbilas would you be able to do this?

Unfortunately, I no longer have a device with such a converter, so I won't be able to test it.

@anangl does a review require someone to test this on an AD5592?

There are quite a lot of changes. Reviewing one commit at a time should make the review process less difficult.

@fabiobaltieri would you take a look at it (I know it's not your area but I know you enjoy optimizing the drivers as well)?

wait what? do I?! 🤨

@fabiobaltieri would you take a look at it (I know it's not your area but I know you enjoy optimizing the drivers as well)?

wait what? do I?! 🤨

Wrong person alert.

@jeppenodgaard I have tested driver in your initial version (before review changes) on HW I have got and it seems to work fine.
However please not that due to HW limitations I was able to verify:

GPIO on IO1/6 as an output only,
DAC on IO0 only,
ADC on IO2/3/4/5.

Nevertheless taking scope of your changes into account I think rest of IOs will also work correctly.
CC @bbilas

@jeppenodgaard I have tested driver in your initial version (before review changes) on HW I have got and it seems to work fine.

Thank you so much!
I might am currently trying to figure out how to refactor so e.g. adc wont be part of mfd header, so there will probably come more changes, but they should not be functional changes.

@bbilas @aasinclair I refactored the code and implemented it slightly different than I suggested by using a config flag and a mdf function to get the flag. I also squashed commits to ensure it builds. Please have a look.

Ping @bbilas

@jeppenodgaard You need to resolve the merge conflicts.

CI fail is unrelated, @aescolar can you take a look? west build -p -b nrf5340bsim/nrf5340/cpuapp -T samples/drivers/mbox/sample.drivers.mbox.simu fails but nrf5340bsim/nrf5340/cpuapp is explicitly allowed in samples/drivers/mbox/sample.yaml

CI fail is unrelated..

@fabiobaltieri thanks for the ping, issue https://github.com/zephyrproject-rtos/zephyr/issues/70566, fix is being tested

Could you move cd42487020dd59dbd79712ea9c8aa67af57b1898 and 7475fa0fac704efe9c2b22e8b14d71d3140e4bca into separate PR?

Could you move cd42487020dd59dbd79712ea9c8aa67af57b1898 and 7475fa0fac704efe9c2b22e8b14d71d3140e4bca into separate PR?

Yes:
https://github.com/zephyrproject-rtos/zephyr/pull/70600

@jeppenodgaard please rebase.

@jeppenodgaard please rebase.

Done :)


Add docs for typedef rule (option: arrow-call-signature)

I couldn't find them here.

Looks like the rule's metadata is out of date so a PR to fix that from anyone would be great!


[RFC] Add Retrying Session

Based on #165 and #143
Here is the proposed api:
from pyairtable import Api,retry_strategy

# current
api = Api("key", "base", "table")

# w/ default retry
api = Api("key", "base", "table", retry_strategy=retry_strategy())

# w/ customized retry
api = Api("key", "base", "table", retry_strategy=retry_strategy(total=5)
api = Api("key", "base", "table", retry_strategy=retry_strategy(status_forcelist=[429])

left retry_strategy args as is to avoid duplicating their docs and mapping args names. Docs just references Retry lib docs.
retry_strategy simply generates a default retry strategy but allows easy modification

PR Includes tests + docs
Opening up for feedback!
@@borland667

Codecov Report

Merging #166 (d4b3441) into main (b7bd086) will increase coverage by 0.20%.
The diff coverage is 94.44%.

@@            Coverage Diff             @@
##             main     #166      +/-   ##
==========================================
+ Coverage   87.08%   87.28%   +0.20%
==========================================
  Files          13       15       +2
  Lines         596      629      +33
==========================================
+ Hits          519      549      +30
- Misses         77       80       +3

Impacted Files
Coverage Δ

pyairtable/compat.py
83.33% <83.33%> (ø)

pyairtable/api/retrying.py
88.88% <88.88%> (ø)

pyairtable/__init__.py
100.00% <100.00%> (ø)

pyairtable/api/abstract.py
100.00% <100.00%> (ø)

pyairtable/api/api.py
76.31% <100.00%> (+0.64%)
:arrow_up:

pyairtable/api/base.py
78.37% <100.00%> (+0.60%)
:arrow_up:

pyairtable/api/table.py
100.00% <100.00%> (ø)

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 b7bd086...d4b3441. Read the comment docs.

Hi @gtalarico is there anything I can do to help merging this one?


How to install globally into the Program Files folder? Can the -g syntax be supported to install into the global directory instead of the user directory?

Description of the new feature / enhancement
How to install globally into the Program Files folder? Can the -g syntax be supported to install into the global directory instead of the user directory?
Proposed technical implementation details
No response

If you are talking about using WinGet to install packages "machine" wide, you can use the "--scope machine" argument for package installers that support installing machine wide.


[BUG] Images missing on archived opensearchcon pages.

Describe the bug
It looks like when we relocated some of the partner images in order to redesign the partner page, there were some other pages that were rereferring to those images. In particular, the exhibitor pages for previous opensearchcons.
Expected behavior
Images should appear when viewing exhibitors from archived opensearchcons.
Screenshots

To Reproduce

Visit opensearch.org
Click once on "OpenSearchCon" in the navbar.
Click on 'Archive'
Click on 'OpenSearchCon 2022 North America'
Click 'Exhibitors' from the stack of buttons on the left.
Note the missing images.

Additional Details
No response

Solved via #3184


irc.hashbang.sh -> irc.freenode.net

I'm still against this for the main reason that we're going to have so many people register with nicks that are already registered on Freenode. I think we should have a separate server up until we have a better solution but unfortunately I don't have the time right now to set one up. So long as we reboot the other IRC server and get the Docker container up appropriately we can have as low as a minute of downtime a month. Not the best solution but it can work til we get something better.

We have no nick registration at all right now
We don't have a path to bridge to Riot which is where a lot of users will connect with us who don't know how to use a terminal yet (pile of noobs currently there)
We have not demonstrated we can run a stable IRC server in years, much less services. This lets u focus on other more important things and circle back to IRC later if we really want to.
a number of people are already on freenode and have said they would join our room, but have expressed they don't want to add yet another IRC server to connect with us.
Freenode -has- given us the #! namespace which is pretty great and bridged to Riot already and people are already in both kind of orphaned
We can always run our own freenode server later when we are in better shape and help make freenode stronger.

I would appreciate the opportunity to try and deliver a stable Oragono (ircd + integrated services) instance to you :-)
Oragono is well positioned to answer some of your concerns, but not others. In particular:

I'm not too worried about scalability or high availability: a single instance should scale comfortably to 10,000 users and 2,000 users per channel. Also, conversations with daurnimator and benharri have suggested to me that Oragono can already be deployed for high availability within k8s without any code changes --- I just need to determine the right k8s primitives to use.
If my reading of https://github.com/matrix-org/matrix-appservice-irc is correct, bridging uses the IRC client-to-server protocol, which should work just as well with Oragono as it does on Freenode.
I don't have an answer for people not wanting to add another server to their clients.
I don't have a terribly convincing answer for people who don't want to run a terminal client or a bouncer. Oragono already has server-as-a-bouncer features, and I'm adding more (including persistent server-side history), but client support is lacking and I don't have a timeline for improvements there.


[benchmark] Add MSVC ARM64 support

Adds MSVC ARM64 support to Google Benchmark.
PR sent upstream over week ago with no changes requested, but not merged yet and no information about planned releases.

I think you need to let upstream discuss this https://github.com/google/benchmark/pull/1052

There are two upstreams in this context:

Gbenchmark, which is not very fond of this change due to cmake lacking relevant implementation or expressly stating this is the intended way of handling
Cmake, which had the issue filed 6 years ago and no implementation in sight (though I haven't spent whole lot of time looking for one)

This leaves this PR in a limbo, as I don't intend to work on cmake fix at this point.

@janisozaur My point is, we, vcpkg, are a packaging system, not a "fix everyone's bugs clearinghouse". If upstream looked at the patch and rejected it, it would be wrong of us to go around them and apply it anyway. Note that upstream is likely to be blamed for any bugs in such patches, not us. This isn't adding a header file or something like that, this is adding a whole new feature directly relevant to the upstream authors' core competencies. They probably have strong opinions about how timing stuff is done given that this is a benchmarking library. See https://github.com/microsoft/vcpkg/blob/master/docs/maintainers/maintainer-guide.md#do-not-implement-features-in-patches
As for the cmake change, I agree with other commenters that if that setting for CMAKE_CROSSCOMPILING needs to be set that way, it probably should be set everywhere in our toolchains, not in this individual port. But I don't think we even need to consider that right now because just the C++ patch is troublesome on its own.

@janisozaur
Since this PR requires upstream changes. We're not sure when upstream will complete these work.
I think we can close this PR for now. You can reopen and ping us if these changes are accepted by upstream.
What do you think?

As upstream had comments on the cmake part of the patch, not the actual code changes, I believe this is the correct way to address the compilation problem. The cmake stuff I've now separated out to another PR upstream, which I imagine can get rejected on the grounds that it's a problem with cmake and not something downstream should bother with. As vcpkg is a collection of cmake patches, I'd hope the CMakeLists.txt portion can get merged here until cmake itself gets fixed.
How does that sound to you?

problem with cmake and not something downstream

let me just cite myself https://github.com/microsoft/vcpkg/pull/14021#discussion_r504461244
It is a problem with vcpkg not setting  CMAKE_SYSTEM_PROCESSOR on windows. The linux toolchain is actually setting those values.

I think we should wait on this until we merge the fixes to the Windows toolchain that @Neumann-A has implied.

Sounds ok. Is there pr already I can subscribe to?

Sounds ok. Is there pr already I can subscribe to?

That is: #16111

Since the required PR https://github.com/microsoft/vcpkg/pull/16111 isn't merged, switch this PR to draft.

The upstream has merged both patches provided, so it should be possible to use it directly, without any patching. I will submit a pr later today

So I checked it just now on current master and it seems there's nothing to do anymore?
Upstream took two of my PRs which were both the gist of this PR to vcpkg:
https://github.com/google/benchmark/pull/1090
https://github.com/google/benchmark/pull/1052
Since they were merged, there was at least 1.5.5 release, which is the version currently in use by vcpkg:
https://github.com/microsoft/vcpkg/blob/134505003bb46e20fbace51ccfb69243fbbc5f82/ports/benchmark/portfile.cmake#L9
https://github.com/microsoft/vcpkg/pull/18595
In 18595 arm was also removed from unsupported architectures and the scripts/ci.baseline.txt file was updated in #14246 to not include benchmark anymore.
It seems there are no changes required to get a build of benchmark working for WoA. Please confirm and close this PR or point what I might be missing.

@janisozaur  So can you please resolve the file conflicts first?

So I think the upgrade PR https://github.com/microsoft/vcpkg/pull/18595 fixed this issue?

Used current master to do a test build for arm64-windows and it compiled correctly:
https://github.com/janisozaur/benchmark-arm64-test/runs/3158745801?check_suite_focus=true
name: CI
on: [push, pull_request]
jobs:
  windows:
    name: Windows
    runs-on: windows-latest
    steps:
      - name: Checkout
        uses: actions/checkout@v2
      - name: Install vcpkg and packages
        uses: lukka/run-vcpkg@v7
        id: runvcpkg
        with:
          vcpkgGitCommitId: d781bd9ca77ac3dc2f13d88169021d48459c665f
          vcpkgTriplet: 'arm64-windows'
          vcpkgArguments: 'benchmark'

I can't test it right now, but given all the issues this PR aimed to fix were spotted at compile-time, I believe this PR is no longer required as upstream took my fixes there.


SDKs should have an OSS license

Background & motivation
Right now all of Kurtosis monorepo is under BSL license, that is not an OSS license.
This is fine for Kurtosis Engine and APIC, since they are external dependencies for most systems. But since the SDK is actually incorporated in codebases, ideally it would be OSS compatible to be friendly to other OSS codebases.
This also unblocks a couple of features, like better indexing and documentation features on go.dev, since now they won't index much based on the license we use:

<EMAIL_ADDRESS>Desired behaviour
Leave all core Kurtosis logic under BSL, but API/SDK under an OSS license like MIT
How important is this to you?
Nice to have; this feature would make using Kurtosis more enjoyable.
What area of the product does this pertain to?
SDK: the Software Development Kit libraries - Typescript, Go, etc.

Thanks @victorcolombo - very helpful. Let me circle back internally to see if we can do a quick fix here.

@leeederek  this makes a ton of sense to me, I think we should do this. SDKs should be available under standard OSS


Private variable values being reset with multiple flowcharts

Reported on the forum

I think it's not variable values being reset, but SetVariable command field values.

I tracked this down to a weird behaviour in the inspector code, it was trying to inspect a variable on the wrong Flowchart. Committing the fix now.


Stretching matrices

Dear Pedro,
Can you explain lines 177, 180, 184 in utils.py?
p = np.sqrt(disp_target / disp_source)
target_pow_train['covs'] = np.stack([powm(covi, 1.0/p) for covi in covs_target_train])
target_pow_test['covs'] = np.stack([powm(covi, 1.0/p) for covi in covs_target_test])
You are taking powers of 1/p instead of p, it seems correct because you are computing p as the fraction of the target over source instead of the source over target as shown in paper eq. 16, but why?

Hi, the goal with the stretching operation is to ensure that the dispersion for target is the same as that for source. Of course, I could have done things with p = np.sqrt(dist_source/disp_target) and then used p in the powm operation... it would be simpler and easier to understand :-)


Properly configure containerd/cri-containerd cgroup.

Both containerd and cri-containerd added the flag to put themselves into a specific cgroup:

containerd: https://github.com/containerd/containerd/pull/1443
cri-containerd: https://github.com/kubernetes-incubator/cri-containerd/pull/184

When bootstrap the node, we should put both of them into a runtime cgroup, and pass the cgroup to kubelet via flag.
We should take care of this in our document or any start-up script in the future.
/cc @abhinandanpb

@abhinandanpb We should take care of this in our installer after the first version gets merged.

yes I have a list of TODO's . This is on it.

Moved to next patch release, not affect feature parity.


Ability to completely unbind all events for a view in dynamic situations

I'm hitting an issue that I haven't found a good solution for. My page is extremely dynamic, and I load in views which may bind to the same elements.
I may also create the same view once, create a different view, then recreate the first one. The issue though is the event handlers persist and I will find the event is firing twice.
So my 'flow' would be viewA --> viewB --> vewA which causes the newest A view to double fire events.
It feels like for completely dynamic pages, an application would want the ability to completely 'destroy' or 'unbindAllEvents' for a given view, which I cant seem to figure out. Is there a known solution for this? If not, is it something others feel is needed? I could code it and provide a pull request if others want.
One other note - a view can have multiple views in it, so you would expect it to unbind all subviews as well.
Thanks

jQuery can handle this for you:
$(element).unbind();

$(element).unbind does handle the issue I mentioned above. I want to do this for an entire view.
Now, if I do $(theView.el).unbind() that still doesnt work, I would have to do - $(theView.el).unbind('.delegateEvents' + theView.cid);
But now the problem is that does not unbind all child views, thus an app would have to traverse all child views and manually unbind those as well.

@csmaster2005: Take a look at #682 and think about implementing a baseclass for your views that has a noop destroy method on it that you can call/extend.


smart_tokenizer_and_embedding_resize

Hi,
I have converted llama to HuggingFace format and now I'm trying to recover alpaca weights. However, after using the command
python weight_diff.py recover --path_raw <path_to_step_1_dir> --path_diff <path_to_step_2_dir> --path_tuned <path_to_store_recovered_weights>
it prompts the following error:
ImportError: cannot import name 'smart_tokenizer_and_embedding_resize' from 'train'
I install the libraries in requirements.txt and also installed the train library (version 0.0.5). I could not find the reason for this error. Can anyone give me a clue on that?

I think you need to uninstall the train library since smart_tokenizer_and_embedding_resize is a function in train.py which is in the repo. I think pip uninstall train should do the trick.


nzbget: 21.1 -> 21.4-rc2

About 9 months ago the NzbGet development officially stopped and there were an announcement on reddit:
https://www.reddit.com/r/usenet/comments/z62skp/nzbget_development_officially_abandoned/
The official github repository has been archived to be marked readonly: https://github.com/nzbget/nzbget
However, there's a fork and they are responsive to issues and makes new commits: https://github.com/nzbget-ng/nzbget
This fixes #244142
Description of changes

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

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

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

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

[x] Fits CONTRIBUTING.md.

I am successfully running nzbget from this commit.
https://github.com/KyleOndy/dotfiles/commit/c0883414e1aa2aef2539f0c86d758ebb7ee67180


Pagination doesn't work

There's a bug in the function:
RequestHelper.php:

public static function createRequestData($data, $sourceUrl = null)
{
    return [
        "source_url" => $sourceUrl,
        "data"       => json_encode($data, JSON_FORCE_OBJECT),
    ];
}

Because of JSON_FORCE_OBJECT, bookmarks are encoded incorrectly:
{"0": "Pz9Nakl5TX...YxNmRjNQ=="}
instead of
["Pz9Nakl5TX...YxNmRjNQ=="]
It makes the option invalid and the first page is always being returned.

Wrong request in:
SearchHelper.php

protected static function createSimpleSearchRequest($requestData)
    {
        $dataJson = [
            'module' => [
                "name"    => "SearchPage",
                "options" => $requestData['options'],
            ],
            "options" => $requestData['options'],  // <-- was missed
        ];
        return $dataJson;
    }

I missed one more options param.
Strange, but for pinns search it works fine.

In my case it didn't work for iterating followers.

I see, it didn't work for boards and pinners, only for pins. I'll fix it now.

Try it now.

Thanks!

Looks like it didn't help, bookmarks are still being encoded to JSON object rather than array.

Oh, I see. Sorry, I'll fix it soon.

I think I have fixed it now)


Conditional compilation

This PR fixes #108. It also cleans up all the ugliness in the command-line options because I was looking at it and my eyes bled.

@bors r+


New integration test

Creates an integration test (currently skipped as it fails for a known bug) so we can fix the bug!

@otoolep This PR is ready for review and merge.  While the test that was the primary driver for this PR is currently skipped, it did flesh out a couple of things.

Changing the way we launch the server so that we can test drive it better.
Discovered some bugs on unmarshalling the json
Added a Server.Ready() so we know when the server can accept data.  This needs a closer review by @benbjohnson to know if it is correct or not.  I mostly took an educated guess at what made a server ready.

I'm not sure if the test is actually failing due to a race condition or not, but it doesn't always fail the same way which tells me there are some more race conditions to be addressed yet.
The CLI depends on the JSON bug fixes in this PR, so to take the CLI further, I would like to merge this and come back when we have more stability in some of our race conditions to continue debugging.

Making sense, though I have some feedback. FWIW, if you have some change here that is blocking something else (the CLI) it's often best practise to simply break that out into its own PR. That helps other devs, since the change will be small, and makes the commit history easier to follow.

I don't have a problem with moving the changes to another PR.  I'm not quite sure how to pull them from here though and put them in another one, unless you are suggesting I make them in a different PR, then merge that to master and rebase this branch (which would be easy enough to do).
The other concern I have is that we are moving code really fast, and even 2-3 days can make this branch non-merge-able, and we do need to get these integration tests running.  I'm open to suggestions on how to get this into play sooner than later.

@otoolep I don't think this branch is mergable (or will be made mergable).  I'll create a new PR that will create the heartbeat, and one that will create the integration tests.  We should be able to merge both of those easily.

This has been replaced by PR https://github.com/influxdb/influxdb/pull/1490


Basic Auth returns invalid api_key

Hey guys
I'm trying using Basic Auth and have set api key in my auth.csv file. while I'm passing Basic Auth value(that is base64 encoded of API key) through my request I always get invalid api_key. here is what I have set in my test:
API key in my csv: 06da1afe-16d3-48fe-8ec2-fea8648d737a
Auth Value through my request: MDZkYTFhZmUtMTZkMy00OGZlLThlYzItZmVhODY0OGQ3Mzdh
I wanna know what's the relation between API Key in CSV file and Auth Value in request.

Hi @hnamzian, sorry for the delay. Here is the documentation about api keys: https://docs.quorum-key-manager.consensys.net/en/latest/HowTo/Authenticate/API-Key/
If your api key is 06da1afe-16d3-48fe-8ec2-fea8648d737a. Then you should define it in your CSV file as sha256(06da1afe-16d3-48fe-8ec2-fea8648d737a) = 4fb54cba957c094e0d6c66bd5c39592c19fdd8d1597651f483460593bebaa0be
The goal is that the "real" api key is never written in the CSV file, only the sha256 of the api key in case the CSV file is exposed.
Then, on your calls, you will pass the base64(06da1afe-16d3-48fe-8ec2-fea8648d737a) = MDZkYTFhZmUtMTZkMy00OGZlLThlYzItZmVhODY0OGQ3Mzdh


Update contribution guidelines for approvers

This PR updates docs contribution guidelines:

Removes references to deprecated labels
Removes section for website admins
Adds guidelines for approvers

/sig docs
/assign @kbhawkey

LGTM


Fix Strings.truncate to return the correct length

Describe the change
Fixes https://github.com/piotrmurach/strings/issues/17
Why are we doing this?
I believe we should be returning characters of N display width when specifying Strings.truncate(s, N).
Benefits
Strings.truncate will be more intuitive (e.g. follow the Principle of Least Surprise).
Drawbacks
It is possible that existing usages of this library are coded to expect the width of displayed characters to be strictly < truncate_at, but equal to it. In this case, they will get potentially an extra character.
If this is a concern, we could instead update the documentation here to specifying that truncate_at is not inclusive (e.g. Strings.truncate does not return a character at truncate_at width).
Alternatively, we could introduce an :inclusive option (defaulted to false) which would treat the behavior of truncate_at as inclusive when passed.
Requirements
Put an X between brackets on each line if you have done the item:
[X] Tests written & passing locally?
[] Code style checked?
[X] Rebased with master branch?
[] Documentation updated?

Hi Justin,
I do apologise for not making this clearer in the readme that the intention is to use strings-truncation instead. I do hope this won't discourage you from contributing in the future.

@piotrmurach Thanks for the heads up! Would you like me to add something to the README here so it is clearer for others in the future? All I see currently is a reference to strings-truncate as being independently consumable, not that it is a replacement for this gem.
https://github.com/piotrmurach/strings#4-components
Happy to write up something! 😄

@justingaylor You're right. We could add a note under the truncate section along the lines: "Please note this API will change in the next release and will be replaced by the strings-truncation component" or something similar.


pcre build failure

Host Environment

OS: Windows 10
Microsoft Visual Studio Enterprise 2019
Version 16.11.16
VisualStudio.16.Release/16.11.16+32602.291
Microsoft .NET Framework
Version 4.8.04084

Installed Version: Enterprise
Visual C++ 2019   00435-60000-00000-AA239
Microsoft Visual C++ 2019
ASP.NET and Web Tools 2019   <IP_ADDRESS>28
ASP.NET and Web Tools 2019
Azure App Service Tools v3.0.0   <IP_ADDRESS>28
Azure App Service Tools v3.0.0
C# Tools   3.11.0-4.22108.8+d9bef045c4362fbcab27ef35daec4e95c8ff47e1
C# components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.
Common Azure Tools   1.10
Provides common services for use by Azure Mobile Services and Microsoft Azure Tools.
Cookiecutter   16.11.21196.2
Provides tools for finding, instantiating and customizing templates in cookiecutter format.
IntelliCode Extension   1.0
IntelliCode Visual Studio Extension Detailed Info
Microsoft JVM Debugger   1.0
Provides support for connecting the Visual Studio debugger to JDWP compatible Java Virtual Machines
Microsoft MI-Based Debugger   1.0
Provides support for connecting Visual Studio to MI compatible debuggers
Microsoft Visual C++ Wizards   1.0
Microsoft Visual C++ Wizards
Microsoft Visual Studio VC Package   1.0
Microsoft Visual Studio VC Package
NuGet Package Manager   5.11.2
NuGet Package Manager in Visual Studio. For more information about NuGet, visit https://docs.nuget.org/
ProjectServicesPackage Extension   1.0
ProjectServicesPackage Visual Studio Extension Detailed Info
Python   16.11.21196.2
Provides IntelliSense, projects, templates, debugging, interactive windows, and other support for Python developers.
Python - Profiling support   16.11.21196.2
Profiling support for Python projects.
Test Adapter for Boost.Test   1.0
Enables Visual Studio's testing tools with unit tests written for Boost.Test.  The use terms and Third Party Notices are available in the extension installation directory.
Test Adapter for Google Test   1.0
Enables Visual Studio's testing tools with unit tests written for Google Test.  The use terms and Third Party Notices are available in the extension installation directory.
TypeScript Tools   16.0.30526.2002
TypeScript Tools for Microsoft Visual Studio
Visual Basic Tools   3.11.0-4.22108.8+d9bef045c4362fbcab27ef35daec4e95c8ff47e1
Visual Basic components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.
Visual F# Tools   16.11.0-beta.21514.6+b6c2c4f53ea3a08fa603ea93d6d2f808a62a21d1
Microsoft Visual F# Tools
Visual Studio Code Debug Adapter Host Package   1.0
Interop layer for hosting Visual Studio Code debug adapters in Visual Studio
Visual Studio Tools for CMake   1.0
Visual Studio Tools for CMake
To Reproduce
Steps to reproduce the behavior:
./vcpkg.exe install pcre --triplet x64-windows
Failure logs
Downloading https://mirrors.sjtug.sjtu.edu.cn/msys2/mingw/i686/mingw-w64-i686-pkg-config-0.29.2-1-any.pkg.tar.xz... Failed. Status: 22;"HTTP response code said error"
Additional context
Building on Windows 10 in powershell. Just running the command above after a clean git pull on the repo

MSYS2 is a rolling release project, and vcpkg currently cannot save obsolete binary packages. Please git pull and try again.
This link has expired.  https://mirrors.sjtug.sjtu.edu.cn/msys2/mingw/i686/mingw-w64-i686-pkg-config-0.29.2-1-any.pkg.tar.xz
Or  unzip this
msys-mingw-w64-i686-pkg-config-0.29.2-3-any.pkg.tar.zip
and put it into VCPKG_ROOT/downloads then try again.

I'm getting the same error on curl now.  Is this something that also needs
to be fixed?
B
On Thu, Jul 28, 2022 at 2:26 AM Frank @.***> wrote:

MSYS2 is a rolling release project, and vcpkg currently cannot save
obsolete binary packages. Please git pull and try again.
This link has expired.
https://mirrors.sjtug.sjtu.edu.cn/msys2/mingw/i686/mingw-w64-i686-pkg-config-0.29.2-1-any.pkg.tar.xz
Or unzip this
msys-mingw-w64-i686-pkg-config-0.29.2-3-any.pkg.tar.zip
https://github.com/microsoft/vcpkg/files/9206007/msys-mingw-w64-i686-pkg-config-0.29.2-3-any.pkg.tar.zip
and put it into VCPKG_ROOT/downloads then try again.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vcpkg/issues/26014#issuecomment-1197718218,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABB63VY77MIHP27A4YCDGDTVWIR2HANCNFSM5425DZEA
.
You are receiving this because you authored the thread.Message ID:
@.***>

@GRRedWings
See my https://github.com/microsoft/vcpkg/issues/26015#issuecomment-1198563709 - this is the same issue. The vcpkg version you have to too old and still references msys tools that are not longer available.

I see that my error was that I had a sub module, and while I thought I was pulling the latest that was over writing. Sorry about that


feat: set the program environment variables

This gives the option to provide a custom set of environment variables to use in the Bubble Tea program. When running Bubble Tea in a remote session such as SSH, we need to be able to pass the SSH session's environment variables rather than the server ones.

Two questions with this one:

Will it have any effect on local programs?
How would the user implement it in Wish?

I'd also add a very short example to this PR (and probably the doc comments) so we can reference it later.

Added an example in the code comment

Will it have any effect on local programs?

No, bubbletea programs default to os.Environ() if no environment is set.

How would the user implement it in Wish?

The bubbletea middleware should pass the SSH session's environment to the program. The middleware should include something like:
	var sess ssh.Session // ssh.Session is a type from the github.com/charmbracelet/ssh package
	pty, _, _ := sess.Pty()
	environ := append(sess.Environ(), "TERM="+pty.Term)
	p := tea.NewProgram(model, tea.WithEnvironment(environ)


name() doesn't work properly on slider controls

Putting the name() call after a min() or max() seems to cause problems.
Example: https://jsfiddle.net/ghkhuof3/2/

You mean before min/max. :) Just encounterd this myself.

As soon as .min() and .max() have been called the controller gets replaced with a new instance: https://github.com/dataarts/dat.gui/blob/1f71271c63d1205c70c0c5b2d30c966066e61a09/src/dat/gui/GUI.js#L854

step() is lost too, for the same reason (I guess).

fixed in 		0aec885


Application state

Create a global application state object that can be passed between modules
Remove references to a given module's parent.
More work can be done to remove function calls to a given module's child's functions, but at least the dependencies only go down the tree now.
New events are in ALL_CAPS
Tried to split out bindEvents and listenForActions.  Another check of all the files would be useful, since I may not have updated every file to follow this pattern.

Boom!

Cool. I will try to review this this evening. Can you leave a few comments in the commit through the github line comment system to guide me a bit? I am not going to be able to get to this until tonight.

I mostly want to understand where/how the global object is being passed around.

Also, I really think this is going to be super useful in the "state atom"/saveController https://egghead.io/series/getting-started-with-redux.
The library is 2kb and is made exactly for this.

I put in a number of comments - let me know if you have any other questions.


gloVe performances

Hi all,
it's been a while now that I've been trying to train gloVe on a large dataset (1.2Tb).
The script successfully created the vocabulary but it's been like two month that it's running on the cooccurrence extraction:
$BUILDDIR/cooccur -memory $MEMORY -vocab-file $VOCAB_FILE -verbose $VERBOSE -window-size $WINDOW_SIZE < $CORPUS > $COOCCURRENCE_FILE
I can see the file cooccurrence.bin growing slowly, but I was wondering if it is normal  that it's running for such long time?
-rw-r--r-- 1 lfoppian0 tdm 631G Sep  8 08:08 cooccurrence.bin
Thank you in advance
For information, I'm attaching the modified the script demo.sh. Mainly I changed memory=80Gb, CPUs=72 and modified various paths and file names:
#!/bin/bash
set -e

# Makes programs, downloads sample data, trains a GloVe model, and then evaluates it.
# One optional argument can specify the language used for eval script: matlab, octave or [default] python

make

CORPUS=myCorpus.txt
VOCAB_FILE=vocab.txt
COOCCURRENCE_FILE=cooccurrence.bin
COOCCURRENCE_SHUF_FILE=cooccurrence.shuf.bin
BUILDDIR=build
SAVE_FILE=vectors
VERBOSE=2
MEMORY=80.0
VOCAB_MIN_COUNT=5
VECTOR_SIZE=50
MAX_ITER=15
WINDOW_SIZE=15
BINARY=2
NUM_THREADS=72
X_MAX=10
if hash python 2>/dev/null; then
    PYTHON=python
else
    PYTHON=python3
fi

echo
echo "$ $BUILDDIR/vocab_count -min-count $VOCAB_MIN_COUNT -verbose $VERBOSE < $CORPUS > $VOCAB_FILE"
$BUILDDIR/vocab_count -min-count $VOCAB_MIN_COUNT -verbose $VERBOSE < $CORPUS > $VOCAB_FILE
echo "$ $BUILDDIR/cooccur -memory $MEMORY -vocab-file $VOCAB_FILE -verbose $VERBOSE -window-size $WINDOW_SIZE < $CORPUS > $COOCCURRENCE_FILE"
$BUILDDIR/cooccur -memory $MEMORY -vocab-file $VOCAB_FILE -verbose $VERBOSE -window-size $WINDOW_SIZE < $CORPUS > $COOCCURRENCE_FILE
echo "$ $BUILDDIR/shuffle -memory $MEMORY -verbose $VERBOSE < $COOCCURRENCE_FILE > $COOCCURRENCE_SHUF_FILE"
$BUILDDIR/shuffle -memory $MEMORY -verbose $VERBOSE < $COOCCURRENCE_FILE > $COOCCURRENCE_SHUF_FILE
echo "$ $BUILDDIR/glove -save-file $SAVE_FILE -threads $NUM_THREADS -input-file $COOCCURRENCE_SHUF_FILE -x-max $X_MAX -iter $MAX_ITER -vector-size $VECTOR_SIZE -binary $BINARY -vocab-file $VOCAB_FILE -verbose $VERBOSE"
$BUILDDIR/glove -save-file $SAVE_FILE -threads $NUM_THREADS -input-file $COOCCURRENCE_SHUF_FILE -x-max $X_MAX -iter $MAX_ITER -vector-size $VECTOR_SIZE -binary $BINARY -vocab-file $VOCAB_FILE -verbose $VERBOSE
echo "$ $PYTHON eval/python/evaluate.py"
$PYTHON eval/python/evaluate.py
~

I've never trained glove, so I can't give a definitive answer, but I
inherited this like Aunt May's old antique chair which I need to do
something with.
Taking forever and doing things very slowly sounds like classic thrashing
behavior.  Is it using a lot of swap space and/or maxxed out on the 80G you
gave it?

I understand the situation.
I checked and things seem fine (no swapping, some memory is still free).
However, after checking this I stopped and restarted with verbose = 0 and with more memory, although I don't think the memory was the issue there.

I can confirm that restarting the process with verbose=0 and more memory finished in much less time

Awesome, thanks!
On Sun, Oct 3, 2021 at 10:43 PM Luca Foppiano @.***>
wrote:

I can confirm that restarting the process with verbose=0 and more memory
finished in much less time
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/stanfordnlp/GloVe/issues/196#issuecomment-933160290,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA2AYWMKX6LMNPQH7HD23QDUFE5GXANCNFSM5DTLKARA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Implement IDebugEngine3.SetEngineGuid

This checkin implements IDebugEngine3 in the MIEngine. The reason for this is
to provide support for SetEngineGuid, which allows registering the MIEngine
for an alternate engine GUID without needing to change the code.
The value of this is that, for the SSH port supplier, we can use a different
engine GUID to differentiate between LLDB/GDB/clrdbg and thus offer multiple
types of code debugging using the same gestures that the attach to process
dialog already knows (engine selection).

:shipit:

Do we need to make any changes to this page?

:+1:

:+1:

@chuckries the change to the .cs file now is optional for folks who only distribute the engine pieces, though it doesn't hurt anything to do so.


Show the exit code of inferior processes

Depends on: https://github.com/mono/debugger-libs/issues/10

Implemented in 81413dfbc163efa8469bf0320fb5d82f735ab10a.


flutter 3.10.3 全新项目引入bruno 3.3.0版本编译报错，里面无任务业务代码

问题提交
标题格式：[BUG]: 问题简介
内容需包含：
1、BUG 描述
flutter 3.10.3 全新项目引入bruno 3.3.0版本编译报错，里面无任务业务代码
2、复现步骤
3、期望行为
4、运行环境

运行设备
系统
Bruno 3.3.0
Flutter Doctor 信息

5、附加信息
包括视频、图片等补充信息
/C:/Users/Administrator/AppData/Local/Pub/Cache/hosted/pub.flutter-io.cn/bruno-3.3.0/lib/src/components/navbar/brn_appbar.dart:297:7: Error: No named parameter with the name 'brightness'.
brightness: brightness ?? Brightness.light,
^^^^^^^^^^
/D:/Program/Work/flutter/win/flutter/packages/flutter/lib/src/material/app_bar.dart:177:3: Context: Found this candidate, but the arguments don't match.
AppBar({
^^^^^^
Target kernel_snapshot failed: Exception
FAILURE: Build failed with an exception.

Where:
Script 'D:\Program\Work\flutter\win\flutter\packages\flutter_tools\gradle\flutter.gradle' line: 1201

What went wrong:
Execution failed for task ':app:compileFlutterBuildDebug'.

Process 'command 'D:\Program\Work\flutter\win\flutter\bin\flutter.bat'' finished with non-zero exit value 1

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 12s
Exception: Gradle task assembleDebug failed with exit code 1

#463

thinks


line number and fold/unfold button are clashing with more than 10k lines

VSCode Version: Version 1.2.0
Commit 809e7b30e928e0c430141b3e6abf1f63aaf55589
Date 2016-06-07T08:47:59.517Z
Shell 0.37.6
Renderer 49.0.2623.75
Node 5.10.0

OS Version: Ubuntu 14.04

Steps to Reproduce:

open a file w/ more than 10k lines
the fold/unfold button for arrays and objects is clashing with the line number, see the attached image.

To verify easily, create a file:

on line 1 use var bug = {
on lines 1-100k use \tsomething
on line 100001 use }

great!


chore(parent): Bump Quarkus to Version 3.85

Related-to: https://github.com/camunda/camunda-bpm-platform/issues/4346
Backported commit e4c392c18f from the camunda-bpm-platform repository.
Original author: psavidis<EMAIL_ADDRESS>

I hoped that we get these version bumps by Dependabot.

Currently we have dependabot activated for security issues, only. There are a lot (!) of older dependencies in the repository, so my fear was that Dependabot would swamp us with Pull Requests if we enabled version updates. But it might be a good idea to just try it


Limit does not work properly with m2m

So here is the scenario, I want to select 2 items from each order on the following example: https://bun.uptrace.dev/guide/relations.html#many-to-many-relation
orders := []Order{}
db.NewSelect().Model(&orders).Relation("Items", func(q *bun.SelectQuery) *bun.SelectQuery {
		return q.Limit(2)
	}).Scan(ctx)

The following query only selects first 2 items instead of selecting 2 items per product.

This is a limitation of the current m2m query generator. Perhaps we should check if limit != 0 and issue a separate query for each order.


Road Map

Road Map
To build the system carefully, the road map has been created with different phases.

Initial Setup

[x] Make registration route
[x] Make log in route
[x] Setup tests
[x] Make necessary guards to prevent manipulation of unowned records
[x] Integrate libraries to handle authentication

Account Availability

[x] Make CRUD routes for currencies
[x] Make CRUD routes for accounts

Financial Entry Availability

[x] Make CRUD routes for modifiers invoked by manual input
[x] Make CRUD routes for financial entry caused by modifiers

Frozen Periods Availability

[x] Make CRUD routes for frozen periods

Collections Availability

[ ] Make CRUD routes for collections
[ ] Make view routes to get aggregated info from collection

Formula Availability

[ ] Make CRUD routes for formula
[ ] Integrate the aggregation of formula to collections

Reactive Modifiers Availability

[ ] Make CRUD routes for events triggered manually
[ ] Upgrade routes for modifiers to react on events
[ ] Make route to trigger a manual event

Dependent Modifiers Availability

[ ] Upgrade routes for modifiers to depend on other modifiers

Timed Events Availability

[ ] Upgrade routes for events to handle interval events
[ ] Upgrade routes for events to handle timeout events
[ ] Conceive a way to handle missed schedules

The road has been changed due to new priorities.
Specialized modifiers may be deferred until more realizations come up.


STOR-1390: Rebase v1.29.1 for OCP 4.15

Diff to upstream v1.29.1:
https://github.com/kubernetes-sigs/azurefile-csi-driver/compare/v1.29.1...RomanBednar:azure-file-csi-driver:rebase-v1.29.1
Upstream changelog since v1.28.1 (OCP 4.14):
https://github.com/kubernetes-sigs/azurefile-csi-driver/releases/tag/v1.28.2
https://github.com/kubernetes-sigs/azurefile-csi-driver/releases/tag/v1.28.3
https://github.com/kubernetes-sigs/azurefile-csi-driver/releases/tag/v1.28.4
https://github.com/kubernetes-sigs/azurefile-csi-driver/releases/tag/v1.28.5
https://github.com/kubernetes-sigs/azurefile-csi-driver/releases/tag/v1.29.0
https://github.com/kubernetes-sigs/azurefile-csi-driver/releases/tag/v1.29.1
@openshift/storage

We tested the following features:

add getLatestAccountKey parameter in storage class: passed
add accountQuota parameter: Jira bug(OCPBUGS-23050) to track
support volume cloning: Jira bug(OCPBUGS-23046) which block the test

@RomanBednar May I know if you have any plan for the fix after evaluation, then we can decide if we could let the PR get merged or not.

/hold

Closing in favor of 1.29.2 bump: https://github.com/openshift/azure-file-csi-driver/pull/43


python_qt_binding: 0.4.2-1 in 'noetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository python_qt_binding to 0.4.2-1:

upstream repository: https://github.com/ros-visualization/python_qt_binding.git
release repository: https://github.com/ros-gbp/python_qt_binding-release.git
distro file: noetic/distribution.yaml
bloom version: 0.9.7
previous version for package: 0.4.1-1

python_qt_binding
* pass ROS_BUILD_SHARED_LIBS to use visibility control properly (#89 <https://github.com/ros-visualization/python_qt_binding/issues/89>)
* allow a list of INCLUDE_PATH (#92 <https://github.com/ros-visualization/python_qt_binding/issues/92>)
* use magic $(MAKE) variable to suppress build warning (#91 <https://github.com/ros-visualization/python_qt_binding/issues/91>)

Waiting for other Noetic releases..


NFT Landing Page Template

I have corrected the errors so I think this time it will pass all checks... fingers crossed.

Also, The website is not responsive yet, So It's fine if you don't merge it now.. The design file I used didn't had the mobile version, But I am working on it.


Fail to play stream

load fail if in url no .mp3 extension, but the url is OK.
./go-chromecast -a <IP_ADDRESS> -v load http://stream.rcs.revma.com/ypqt40u0x1zuv
DEBU[0000] (1)sender-0 -> receiver-0 [urn:x-cast:com.google.cast.tp.connection]: {"type":"CONNECT","requestId":1}  package=cast
DEBU[0000] (2)sender-0 -> receiver-0 [urn:x-cast:com.google.cast.receiver]: {"type":"GET_STATUS","requestId":2}  package=cast
DEBU[0000] (2)sender-0 <- receiver-0 [urn:x-cast:com.google.cast.receiver]: {"requestId":2,"status":{"applications":[{"appId":"CC1AD845","displayName":"Default Media Receiver","iconUrl":"","isIdleScreen":false,"launchedFromCloud":false,"namespaces":[{"name":"urn:x-cast:com.google.cast.cac"},{"name":"urn:x-cast:com.google.cast.debugoverlay"},{"name":"urn:x-cast:com.google.cast.broadcast"},{"name":"urn:x-cast:com.google.cast.media"}],"sessionId":"5db49612-aa4c-4ef3-b99d-44002bb9ac6d","statusText":"Default Media Receiver","transportId":"5db49612-aa4c-4ef3-b99d-44002bb9ac6d","universalAppId":"CC1AD845"}],"userEq":{},"volume":{"controlType":"attenuation","level":1.0,"muted":false,"stepInterval":0.05000000074505806}},"type":"RECEIVER_STATUS"}  package=cast
DEBU[0000] (3)sender-0 -> 5db49612-aa4c-4ef3-b99d-44002bb9ac6d [urn:x-cast:com.google.cast.tp.connection]: {"type":"CONNECT","requestId":3}  package=cast
DEBU[0000] (4)sender-0 -> 5db49612-aa4c-4ef3-b99d-44002bb9ac6d [urn:x-cast:com.google.cast.media]: {"type":"GET_STATUS","requestId":4}  package=cast
DEBU[0000] (4)sender-0 <- 5db49612-aa4c-4ef3-b99d-44002bb9ac6d [urn:x-cast:com.google.cast.media]: {"type":"MEDIA_STATUS","status":[{"mediaSessionId":1,"playbackRate":1,"playerState":"PLAYING","currentTime":883.308866,"supportedMediaCommands":274447,"volume":{"level":1,"muted":false},"activeTrackIds":[],"media":{"contentId":"https://an.cdn.eurozet.pl/ant-waw.mp3","streamType":"BUFFERED","contentType":"audio/mpeg","mediaCategory":"AUDIO","duration":null,"tracks":[{"trackId":1,"type":"AUDIO"}],"breakClips":[],"breaks":[]},"currentItemId":1,"items":[{"itemId":1,"media":{"contentId":"https://an.cdn.eurozet.pl/ant-waw.mp3","streamType":"BUFFERED","contentType":"audio/mpeg","mediaCategory":"AUDIO","duration":null},"autoplay":true,"customData":{},"orderId":0}],"repeatMode":"REPEAT_OFF"}],"requestId":4}  package=cast
unable to load media: unknown file extension ""

This seems like a bug. We try and determine the content type of the media file because the chromecast can't always work it out. In the case of a URL we do a HEAD request to try and work out the content type. However, if we can't work out the content type we should still send it to the chromecast in the hopes that it can work it out.

I have pushed a fix for this to master.

now works, thanks !


Global attribute missing from Thermostat cluster

Summary:
Global attribute "AcceptedCommandList" and "GeneratedCommandList" missing from Thermostat cluster.
Spec screenshot:

Execution logs:
tstat11_CL.txt

Fixed in #18209.


Support passing a list of fingerprint templates to the MATCH function

If possible, we would like to support a workflow in our forms where we can pass more than one fingerprint template to the MATCH function and get back the ordered list of match scores for each template.
In our form, we would then use the max() function to decide based on the highest score whether or not the captured template matches one of the stored templates.
One thing I'm not sure about is whether the list of scores received from Keppel could be stored in a single variable in the XLS form / ODK Central (eventually). Is this possible? Or would they need to be split out into separate variables?
For the multi-value return from Keppel, we would expect back (a) the list of match scores (plural), and (b) the captured fingerprint template (singular).
If receiving a list of match scores in the form will be difficult or impossible, we could foreseeably get back only the highest score (and the fingerprint template in the multi-value option), and then we would still be able to reproduce the match scores on the server if ever needed for auditing purposes.

@tobiasmcnulty what sort of flow would the form your envisioning have? I'dl like to get a sense of the problem being solved here.

@tobiasmcnulty what sort of flow would the form your envisioning have? I'dl like to get a sense of the problem being solved here.

The problem solved is to avoid a false negative when the the original fingerprint captures were swapped (i.e., "right" is stored as "left" and "left" as "right" in our database) due to operator error. Below is an excerpt of what this step in the flow might look like. Specifically, "option 2" proceeds to "scan other thumb" instead of "scan left thumb."

The problem solved is to avoid a false negative when the the original fingerprint captures were swapped (i.e., "right" is stored as "left" and "left" as "right" in our database) due to operator error.

In our form, we would then use the max() function to decide based on the highest score whether or not the captured template matches one of the stored templates.

Interesting! I guess then you could theoretically have both thumbs in the existing data and then just have one "match" question where you scan either to verify the person (assuming the match score is tuned high enough in the form) where you scan either. I guess this approach is already possible, it just requires multiple scans followed by the max() logic?

One thing I'm not sure about is whether the list of scores received from Keppel could be stored in a single variable in the XLS form / ODK Central (eventually). Is this possible? Or would they need to be split out into separate variables?

I guess it could work in a way where you return the multiple match scores as a single string and then use string splitting to pull it apart in the form. However, it feels like the form logic could be lighter if the action returned multiple values to a group. I could imagine that the input is something like this (in the group's body::intent):
uk.ac.lshtm.keppel.android.MULTI_MATCH(uk.ac.lshtm.keppel.android.iso_template_1=/data/finger_1,uk.ac.lshtm.keppel.android.iso_template_2=/data/finger_2, uk.ac.lshtm.keppel.android.return_score_1="my_score_1", uk.ac.lshtm.keppel.android.return_score_2="my_score_2")

The scores would then be returned into my_score_1 and my_score_2 fields. An alternative approach here would be to pass the templates in as a single joined string (with a known separator), but I don't think that's better given we end up needing a convention for the return values anyway.
One risk I see here is running into problems with size limits on passing data around in Android - anything over a 1MB will explode. I believe (and please correct me if I'm wrong) that an ISO template is not at risk of that (as it's generally a few hundred bytes), but obviosuly allowing multiple to be passed in would make this possible.

After discussing this with @tobiasmcnulty again, it appears that there isn't really a use for all the scores to be returned - the best one is good enough. The flow we're supporting here is using a single finger to identify a person based on multiple templates, so returning the max score and then checking that against a suitably high threshold will mean simpler forms and less visual noise than my original proposal of returning multiple scores.
The updated action would like this (in the appearance of a question):
uk.ac.lshtm.keppel.android.MULTI_MATCH(uk.ac.lshtm.keppel.android.iso_template_1=/data/finger_1,uk.ac.lshtm.keppel.android.iso_template_2=/data/finger_2)

This would then return a single result to the question of which would be the highest score obtained from matching a scanned finger against iso_template_1, iso_template_2 (and so on).


Dockerfile.t proposed improvements

Dockerfile best practices suggests to minimize layers by chaining more commands together in a RUN.
https://docs.docker.com/articles/dockerfile_best-practices/
Here are a couple suggestions for doing this better in the node Dockerfile.t...
RUN mkdir -p app/{heroku,src,.profile.d}

RUN curl -s http://s3pository.heroku.com/node/v0.10.30/node-v0.10.30-linux-x64.tar.gz \
  | tar vxz -C /app/heroku --strip-components=2 node-v0.10.30-linux-x64/bin/node

Probably could unify the nodejs.sh setup too.
Just a suggestion

@friism you can use bash -c "mkdir -p dir/{one,two}" (if bash is available).

Closing this issue, see Dockerfile best practices for more ideas: https://docs.docker.com/articles/dockerfile_best-practices/


Set CURRENT_PROJECT_VERSION to avoid validation error on submitting to the app store

When you install this library via carthage, CFBundleVersion is missing because CURRENT_PROJECT_VERSION is not set.
This results in "Info.plist file is missing the required key: CFBundleVersion" error when you try to submit your app to the app store.
It looks like there are other libraries having this issue.
https://github.com/Flight-School/AnyCodable/issues/20#issuecomment-493263003

Thank you @kiyot !


How to enable ssh login between mininet hosts

I have created a topology using mininet. It have 16 hosts (h1-h16). Now I want to run a python script from h1. This will will login to each server and run some commands.
But I am unable to ssh login to a host from another host. I have setup the "HostbasedAuthentication no: in the sshd_config, But still it's not working.
Can someone help to understand the procedure in detail?
If you have a question that is not a bug report or a feature request,
please use the documentation at docs.mininet.org, the FAQ
at faq.mininet.org, and the mininet-discuss mailing list.
For bug reports, please fill in the following information in detail,
and also feel free to include additional information such as debug
output from mn -v debug, etc.
Expected/Desired Behavior:
Actual Behavior:
Detailed Steps to Reproduce the Behavior:
Additional Information:

Please do not ignore the github issue template, which clearly states that issues are not a q&a system.
Feel free to write to mininet-discuss or to check the ssh example in examples/

Closing, please read my comment above.

I use paramiko for ssh'ing trough Python,

host = "test.rebex.net"
port = 22.
username = "demo"
password = "password"
command = "ls"
ssh = paramiko. SSHClient()
ssh. set_missing_host_key_policy(paramiko. AutoAddPolicy())
ssh. connect(host, port, username, password)

On Tue, Dec 1, 2020 at 5:43 PM Debobroto Das Robin<EMAIL_ADDRESS>wrote:

Mininet uses GitHub issues for bug reports and feature requests only.
These issues can be viewed at bugs.mininet.org
I have created a topology using mininet. It have 16 hosts (h1-h16). Now I
want to run a python script from h1. This will will login to each server
and run some commands.
But I am unable to ssh login to a host from another host. I have setup the
"HostbasedAuthentication no: in the sshd_config, But still it's not working.
Can someone help to understand the procedure in detail?
If you have a question that is not a bug report or a feature request,
please use the documentation at docs.mininet.org, the FAQ
at faq.mininet.org, and the mininet-discuss mailing list.
For bug reports, please fill in the following information in detail,
and also feel free to include additional information such as debug
output from mn -v debug, etc.
Expected/Desired Behavior: Actual Behavior: Detailed Steps to Reproduce
the Behavior: Additional Information:
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mininet/mininet/issues/982, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA67NFZOYHAZ4SVOEA5KN53SSV5T5ANCNFSM4UJW2NXA
.


More control over when to notify

My suggestions

When file changes is above user specified value notify.
When diff(Insertion + deletion) is above user specified value notify.

Since this was a simple changes I started to work on this. I would like your opinion and guidance.
I took a note to make this extensions a few days back. Then I looked up if there was already one and found yours.

"Someone already thought of that before me. Nice!!! Might as well work with him to make it better."

😆

Hi, those are some pretty nice suggestions. I wasn't sure if there was anything more to add but this is it!
Can you elaborate a little bit on the diff(Insertion + deletion)?
I don't fully understand what you mean by this.
Since you already started on these changes I'm open to pull requests.
Just make sure you follow the eslint rules to keep the code the same and clean.

Hi. Thanks for the reply. I am glad you considered the suggestions.
By diff (insertion + deletion) I mean the overall repo changes. It could be considered separately.
$ git diff --stat
   package-lock.json |  2 +-
   package.json      | 14 ++++++++++++--
   src/extension.ts  | 17 ++++++++++++-----
   3 files changed, 25 insertions(+), 8 deletions(-)

Here overall changes is 33. So user sets a minimum overall changes above which user will be notified.
This is my first contribution. So if you have any step I should follow, things I might miss as a first-timer. Please share.
Do you want create a separate feature branch or commit changes to master?
Before I create a pull request I will share the branch so you can comment. Let me know how to proceed.

Okay, that clears things up. I like that idea too.
As for creating a pull request, you should create a separate feature branch.
It's a tiny repo so it doesn't really matter but it's best practice so lets just do that.

I have created the pull request.
https://github.com/KurozeroPB/vscode-commit-reminder/pull/5
Open for comment.

Published extension version 0.2.0 including your pull request and some other minor changes.
Thank you very much for the contribution!

Awesome🥰


Payment should be possible to a LNURLp on the same instance

In this dialog it should be possible to put in a LNURLp of a wallet from another user on the same instance since you want to play with your friends those might even use your LNbits already.

I was struggling with this, but found a better solution - just enter your wallet-id and the payment will be done internally.

Why limit to the own instance ?

Remember to include the wallet-id in the descr https://github.com/oren-z0/bets4sats/pull/5/commits/5f4f61c824ab992d19150052eef25b6caaf5a09c


Review home page

Add full screen awesome picture
Move login button to home page

this is almost complete right? what are we missing except for the code review comments?

set the proper image and it's done!

cool, I would like this merge to master when the image and code review are done, to start spoting issues in production :)

I did the changes at work so you will have to wait tomorrow ;)

no problem ;)


Getting compiler error while building app with ng build -prod

Hi, I am using datepicker in my app and it works fine with ng build but fails with ng build -prod. I googled and it looks like AOT compiler issue with ControlValueAccessorProviderFactory
https://github.com/angular/angular-cli/issues/4956
Here is error log:
prorey:provenbid proandrey$ ng build -prod
Hash: f7999b29dd0942099ae8
Time: 7383ms
chunk    {0} main.80551e71e70561828313.bundle.js (main) 1.33 kB {2} [initial] [rendered]
chunk    {1} styles.a5e033767d934a9d8d20.bundle.css (styles) 69 bytes {3} [initial] [rendered]
chunk    {2} vendor.c85a90b3d399a6fd167d.bundle.js (vendor) 1.04 MB [initial] [rendered]
chunk    {3} inline.a5d290ac24c45edf489b.bundle.js (inline) 0 bytes [entry] [rendered]

ERROR in Error encountered resolving symbol values statically. Calling function 'ControlValueAccessorProviderFactory', function calls are not supported. Consider replacing the function or lambda with a reference to an exported function, resolving symbol DatePicker in /Users/proandrey/Projects/provenbid/node_modules/angular-io-datepicker/src/datepicker/datePicker.d.ts, resolving symbol DatePicker in /Users/proandrey/Projects/provenbid/node_modules/angular-io-datepicker/src/datepicker/datePicker.d.ts

ERROR in ./src/main.ts
Module not found: Error: Can't resolve './$$_gendir/app/app.module.ngfactory' in '/Users/proandrey/Projects/provenbid/src'
 @ ./src/main.ts 5:0-74
 @ multi ./src/main.ts

I am having the same issue

me too

Please help me. I am having the sample issue!

me too

me too

me too!! any one got any solution ??

try this:  ng build --prod --aot=false

@WillAcon Thanks for the help... it works perfectly as i desired but there is a small problem. I am deploying this code using webpack and not angular CLI. can you tell me that is there any place where i can set configuration for setting "AOT" to false ?
Thanks

@WillAcon  ng build --prod --aot=false it works fine but '--aot=false' increase our file size .. do u have any solution to reduce the size of file
Thanks

ERROR in Error encountered resolving symbol values statically. Calling function 'ControlValueAccessorProviderFactory', function calls are not supported. Consider replacing the function or lambda with a reference to an exported function, resolving symbol DatePicker in /Users/munirajselvam/projects/muniraj/employeetimesheet/app/web/app/agency/node_modules/angular-io-datepicker/src/datepicker/datePicker.d.ts, resolving symbol DatePicker in /Users/munirajselvam/projects/muniraj/employeetimesheet/app/web/app/agency/node_modules/angular-io-datepicker/src/datepicker/datePicker.d.ts


gps: simplify x = x <op> y to x <op>= y

Simplify x += 1 further to x++.
Already open as #2028 but the fork doesn't exist anymore and I can't push changes to it.
Originally submitted by @quasilyte

Ah, right, we're not going to be able to merge this until #2114 is fixed.

Appreciate the vote of confidence @jmank88 but the Google CLA bot won't let us merge this PR despite the fact that both contributors have signed the CLA.


Tidy up order of "Project Type"  and mariadb version suggestions

Recently, I spun a new project and noticed the following long line:
Project Type [backdrop, drupal10, drupal6, drupal7, drupal8, drupal9, laravel, magento, magento2, php, shopware6, typo3, wordpress] (laravel):

2 things came to mind:

"drupal10" should appear after "drupal9"
It reads better.  Is there another sort option that will do this?

do we need to include "drupal6, drupal7"?
Since these project types are EOL, it don't think we need to display them here. Yes, keep support in DDEV and the documentation; I think the line is too long and removing these will shorten it.

$ ddev --version
ddev version v1.19.0-alpha3
``

Thanks for the good eye and the suggestion. Currently it's sorted alpha, as you see, and I don't think there's any kind of sort that could do what you want. But there could be a manual sorting I guess.

And yes, people need old versions all the time. They do upgrades, they revive them, etc.

I'm not familiar with go but you need a "natural sort" method like:
https://github.com/facette/natsort
https://github.com/skarademir/naturalsort

Just noticed in ddev version v1.19.0-alpha4 , database error messages (postgres:10 ... postgres:9)
Incorrect database value: postgres - use something like 'mariadb:10.3 or mysql:8.0. Options are [mariadb:10.0 mariadb:10.1 mariadb:10.2 mariadb:10.3 mariadb:10.4 mariadb:10.5 mariadb:10.6 mariadb:5.5 mysql:5.5 mysql:5.6 mysql:5.7 mysql:8.0 postgres:10 postgres:11 postgres:12 postgres:13 postgres:14 postgres:9]

This is also true of the mariadb versions:
Options are [mariadb:10.1 mariadb:10.11 mariadb:10.2 mariadb:10.3 mariadb:10.4 mariadb:10.5 mariadb:10.6 mariadb:10.7 mariadb:10.8 mysql:5.7 mysql:8.0 postgres:10 postgres:11 postgres:12 postgres:13 postgres:14 postgres:15 postgres:9]
Both the mariadb versions (10.11 immediately after 10.1) and postgresql versions (9 at the end) are messy.

Also checked the supported databases.
Test
$ ddev config --database mariadb:50
...
failed to validate config: the laravel project has an unsupported database type/version: 'mariadb:50', DDEV amd64 only supports the following database types and versions: mariadb: [5.5 10.0 10.1 10.2 10.3 10.4 10.5 10.6 10.7 10.8 10.11], mysql: [5.5 5.6 5.7 8.0], postgres: [9 10 11 12 13 14 15 16]


Merge pull request #1 from abhshkdz/gh-pages

update

This is a misoperation, please refuse the pull request, I am sorry for that!


Update dist

When using this from a non-webpack project I noticed what seem to be a few mistakes in the build files - i've updated the webpack config and rebuild the assets.

Is this approximately the same as https://github.com/jhudson8/react-chartjs/pull/67 ?

It's similar, but that PR inadvertently bundles ReactDOM - you can see from the number of changed lines in the unminified build that something is up.

Thank you!


Check failed: vm. Must be able to initialize the VM

Add Flutter to existing app issue
Check failed: vm. Must be able to initialize the VM.

caused by productFlavors.

@kangwang1988  thx


release/v1.7.4 [Community Feedback]

Tracker for issues related to release v1.7.4 (see https://github.com/PyCQA/docformatter/pull/249). Please report all issues encountered while using the v1.7.4 release candidates here.
To install a release candidate:
pip install<EMAIL_ADDRESS>where the rc1 would be replaced by the release candidate you'd like to use.
Fixes #243
Fixes #245

1.7.4-rc1 fails on this example:
def f():
"""CC.
C.

C c :math:`[0, 1]`.
"""

it produces
def f():
"""CC.
C.

C c

:math:`[0, 1]`.
"""


Add explanation of imports to API reference

Yesterday, while trying to type check my code using the websockets library, I was getting quite frustrated. All of the example code seemed to show import websockets and then accessing everything directly off the package, which doesn't support type checking or code completion because of the lazy import system. I nearly opened an issue about the matter to see what I was doing wrong, when I then went to double check that there wasn't an existing issue about it before filing. Lo and behold, I found https://github.com/aaugustin/websockets/issues/940#issuecomment-874012438 which gave me what I needed to start type checking my code that uses websockets.
I couldn't find this directly provided as an example or in any of the example code I saw in the docs, but while digging around did see this reference to "use the real import path" at the bottom of the API reference index, without really elaborating on what that is. I consider myself a reasonably experienced python dev and was surprised/confused at first to see import websockets not exposing any of the modules to my IDE but from websockets import  exposing all available modules (and thereby giving my autocomplete and letting mypy type check correctly).
So rather than just raise another issue complaining that I couldn't figure it out, I thought I'd add in a couple more lines to the docs to explain what "use the real import paths" actually looks like in code.
I also considered perhaps adding a note to the types page of the docs, but felt that wasn't really the correct place since that's a proper API reference. I also thought about maybe a new docs page in the how-to guides about how to use the websockets library in a way that your IDE and type checkers know what's going on, but thought I'd start with this simple change first.

Thank you 🙌
It surely helped me.

Addressed in 206d7ef. I chose to link to existing documentation rather than duplicate information.

Thank you for the suggestion!


nextflow: 22.10.6 -> 23.10.1

Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

I am uncertain if this is the right place to mention this, but the metadata for the nextflow package is missing, and the source incorrectly points to the builder template rather than the proper file.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-in-distress/3604/94

Closing in favor of https://github.com/NixOS/nixpkgs/pull/339197 .


Ability to use staging ACME environment?

Hi,
Caddy allows the use of -ca option to define the acme URL, allowing the staging environment to be used for testing.
Is a similar thing available in caddy-docker?
Regards,
Andy

Yeah, just pass additional arguments to the container.
docker run abiosoft/caddy -ca <staging url>

Ah thanks so much.

and with a docker-compose config ?


update installation instructions

update installation instructions to specify python 3.7 and use virtualenv instead of venv (so that people have wheel)

Which installation instructions is this referring to?

looks like just in the user install instructions on readthedocs where we give instruction on creating a virtual env

Ok, it says a few lines above to use python 3.7, but I guess we can repeat it

yeah I think also using virtualenv as people in the workshop ran into problems as they had to do pip install wheel first


Issue with BuildServiceProvider() in ConfigureServices()

When setting up the database in the ConfigureServices() function there's a warning that says:

Calling 'BuildServiceProvider' from application code results in an additional copy of singleton services being created. Consider alternatives such as dependency injecting services as parameters to 'Configure'.

Steps to reproduce
Here's following the piece of code that's references to that warning:
services.AddEntityFrameworkNpgsql()
    .AddDbContext<AppDbContext>(options =>
   options.UseNpgsql(Configuration.GetConnectionString(EnvironmentContants.ConnectionString)))
    .BuildServiceProvider();

Further technical details
EF Core version: 3.0
Database provider: PostgreSQL
Target framework: .NET Core 3.0
Operating system: Windows 10 x64
IDE: Visual Studio 2019 16.3.1
Is there any way to fix it?

Moved here since analyzer lives in this repo.

Why are you building the service provider.

@davidfowl I do as the NpgSQL documentation says about their SQL NuGet package at this link.

cc: @roji

Oops, seems like some old instructions... Have corrected the documentation. Thanks @smitpatel.

Yei, solved :D

Why are you building the service provider.

Any pointers as to how one should go about implementing Microsoft.AspNetCore.Hosting.IStartup without building the service provider in ConfigureServices?

It’s required there but IStartup is no longer supported with the generic host for this reason.

I am using the IOptions pattern for strongly typed AppSettings.  In the past, I needed to specifically make a call to GetService so the Settings were built and I could use the settings to configure the rest of my services.  For example, I need to inject a username and password when registering my soap service.  I am now getting this warning.  Is there a better way to get strongly typed settings injected into other service registrations during the ConfigureServices method?
public void ConfigureServices(IServiceCollection services)
{
    // Add App settings to a strongly typed configuration object
    services.Configure<AppSettings>(Configuration);

    // Create an instance of <AppSettings> so we can use it early in the startup process.
    // By default, this service is not instantiated and available until later during the startup process.
    var serviceProvider = services.BuildServiceProvider();
    AppSettings = serviceProvider.GetService<IOptions<AppSettings>>();

    // Add application specific services
    services.AddApplicationAuthentication(AppSettings);
    services.AddSoapService(AppSettings);
    services.AddApplicationServices();
    services.AddAppSwagger(AppSettings);

    services.AddControllers().AddXmlSerializerFormatters();
}

Why don't you have those services resolve IOptions<AppSettings> instead of taking them directly in the Add* methods?

The problem is that when registering the IOptions<AppSettings> using services.Configure<AppSettings>(Configuration); is that that service is not immediately available for dependency injection during startup.

Can you show me an example of why you can’t defer resolution of IOptions<AppSettings> to the construction of the object?

I need an instance of HttpClient to request credentials from a key-value store inside my ConfigureServices method. How can I do that without  building a ServiceProvider?

I'm also getting this message when attempting to AddSwaggerGen:
services.AddSwaggerGen((options) =>
{
	var provider = services.BuildServiceProvider().GetRequiredService<IApiVersionDescriptionProvider>();
	...
}

All the documentation/examples I've seen for this online uses this code. Is there now a better way to do it?

@davidfowl, if I try to use dependency injection, the IOptions<AppSettings> is null.  I could try to resolve it the static methods, but I would need to resolve it 3 different times, once for each static call.
I decided to just bind it manually in the startup.
public Startup(IConfiguration configuration)
{
    Configuration = configuration;
    Configuration.Bind(AppSettings);
}

private AppSettings AppSettings { get; } = new AppSettings();
public IConfiguration Configuration { get; }

public void ConfigureServices(IServiceCollection services)
{
    // Add App settings to a strongly typed configuration object for dependency injection
    services.Configure<AppSettings>(Configuration);

    // Add application specific services
    services.AddApplicationAuthentication(AppSettings);
    services.AddApplicationServices();
    services.AddAppSwagger(AppSettings);

    // Add controllers and additional serializer formatters
    services.AddControllers().AddXmlSerializerFormatters();
}

Create an HttpClient like so:  new HttpClient()

Yes, of course I can instantiate it directly. But as you know the client itself have utilization issues, and I don't want to keep them in a library-level
From: David Fowler<EMAIL_ADDRESS>Sent: Thursday, October 10, 2019 2:13 AM
To: aspnet/AspNetCore<EMAIL_ADDRESS>Cc: Kirill Taran<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [aspnet/AspNetCore] Issue with BuildServiceProvider() in ConfigureServices() (#14587)
Create an HttpClient like so: new HttpClient()
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Faspnet%2FAspNetCore%2Fissues%2F14587%3Femail_source%3Dnotifications%26email_token%3DAEW5ABXGTL7S4GV5E3RSN7LQNZQQXA5CNFSM4I4ALDK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAZXXTI%23issuecomment-540244941&data=02|01||db05d89b06424107ec5408d74d0e4460|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637062595976518261&sdata=emsxKlZcGCwzq6A%2BqcT8RB7lSFwgovRTnH%2FbmqEfjo0%3D&reserved=0, or unsubscribehttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAEW5ABTOPC3BBHSLKPQS5ITQNZQQXANCNFSM4I4ALDKQ&data=02|01||db05d89b06424107ec5408d74d0e4460|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637062595976528266&sdata=FuEKdguOrSdTepkBO3oKHoEaHNAFZeIzRGT7w2shXso%3D&reserved=0.

Yes, of course I can instantiate it directly. But as you know the client itself have utilization issues, and I don't want to keep them in a library-level

It happens once at startup.

@Frankenleg

@davidfowl, if I try to use dependency injection, the IOptions is null. I could try to resolve it the static methods, but I would need to resolve it 3 different times, once for each static call.

It can't be null. Can you show me exactly what code you wrote and what exactly is null?
Take one of those methods and make a minimal example of the pattern you're using.

i'm also geting this messsage when i try get db context to migrate it

@davidfowl please find a simple example with the logging builder when registering logging services.
services.AddLogging(loggingBuilder =>
{
    using (var scope = loggingBuilder.Services.BuildServiceProvider().CreateScope())
    {
        var apiSettings = scope.ServiceProvider
            .GetRequiredService<IOptions<ApiSettings>>()
            .Value;

        Log.Logger = new LoggerConfiguration()
            .ReadFrom.Configuration(Configuration)
            .Enrich.WithProperty("ApiName", apiSettings.Name)
            .Enrich.WithProperty("ApiVersion", apiSettings.Version)
            .CreateLogger();

        loggingBuilder.AddSerilog(Log.Logger, dispose: true);
    }
});

I don't want to create an IOptions instance in the startup method's code just to do this. Is this the only way? Up until netcoreapp2.2 this was never an issue.

You can suppress the warning in that case. It’s an anti pattern in general. You can do it if you understand what exactly is happening and if you’re ok with it but there’s no support for building the container while trying to build it. What I mean is you have 2 containers and one of them will never be disposed.
PS: it was always an issue, we just warn you about it in 3.0

I still don't see a proper solution mentioned here, I am sure others like me are coming from Google and landing here.
There are lots of service collection extensions that take an options object or action. Many times, we do not have control over those (like the case with registering HttpClients, and this users comment here). So we do not have the luxury of injecting IOptions.
@davidfowl
What is the alternative to building the service provider all over again to get strongly typed and validated options to configure services that we register during startup?
Today, I do the following:
var isOptions = Configuration
    .GetSection("IdentityServer")
    .Get<IdentityServerOptions>();

This means I don't have to build the SP, and I get a strongly typed config for use in ConfigureServices. But it has other shortcomings compared to getting the IOptions<T> stuff registered already.


[Airbrake] [Staging] 301 Moved Permanently

Airbrake error: #8988
Airbrake project: ETMoses
Error type: RestClient::MovedPermanently
Error message: 301 Moved Permanently
Occurred at: Mar 23, 2016 08:31:58 UTC
First seen at: Mar 23, 2016 08:31:58 UTC
Occurrences: 1 (1 since last deploy on Mar 16, 2016 15:07:41 UTC)
URL: https://beta-moses.energytransitionmodel.com/testing_grounds/import
File: [GEM_ROOT]/gems/rest-client-1.6.7/lib/restclient/abstract_response.rb
Backtrace:
[GEM_ROOT]/gems/rest-client-1.6.7/lib/restclient/abstract_response.rb:39:in return!
[GEM_ROOT]/gems/rest-client-1.6.7/lib/restclient/request.rb:230:in process_result
[GEM_ROOT]/gems/rest-client-1.6.7/lib/restclient/request.rb:178:in block in transmit
/home/ubuntu/.rbenv/versions/2.1.1/lib/ruby/2.1.0/net/http.rb:853:in start

Import is broken on beta environment.