Network/HTTP Provider

I'm currently using the WASI provider, but AFAIK WASI doesn't support network yet. I'd like being able to just have an HTTP server on top of a WASI module so that I can send REST requests and retrieve computation results.
The wascc-provider could be the solution, but you also have to "accept" its architecture with actors, signing modules, capability providers etc. One may not desire to stick to such architecture. Also, wascc uses wasmi under the hood instead of wasmtime, AFAIK.
So the only remaining alternative would be writing my own provider, right? In particular, I can see at the least 2 big steps are required:

Writing the host code which allows some sort of HTTP capability inside the WASM module, just like a wascc HTTP provider does
Implement the k8s network virtualization in Krustlet, aka supporting CNI 😄 I see there was some work already done months ago https://github.com/deislabs/krustlet/issues/293, but it seems that development stopped. Was there any motivation in particular?
Allow to expose Krustlet "containers" as services

Good questions here. So as for #293, we stopped work on that so we could focus on the CSI stuff this release (0.6). The focus of the next release it getting networking up and running in a way that can work cross-provider. So we'll be picking that back up soon. CSI (and I assume CNI will run into similar issues) took much longer than expected due to conflicting documentation and supporting it cross platform.
As for writing your own provider, I am guessing we won't have support in WASI for networking for a little while yet. However, @technosophos currently has some plans to make a WAGI provider soon that should give you the functionality you are looking for. That is something you could try out once we have it (or maybe even help with once we get it to a state where people can contribute)

Just to add something on top of what @thomastaylor312 already mentioned: implementing CNI won't mean that the WASI (wasmtime) provider will support WebAssembly modules that can open a socket connection - WASI does not have sock_send/sock_connect/sock_recv support at this time, so the runtime will be unable to fulfill that request.
However, it does open up the opportunity to write new Providers that can utilize CNI support. @kflansburg's krustlet-cri project is one example, and @radu-matei has been working on a proof-of-concept of a WASI runtime with Berkeley Sockets support. Implementing CNI means that we could write a new Provider around this proof-of-concept to demonstrate WebAssembly modules with networking support.

(POC of the sockets work mentioned in the previous comment).
Besides WASI implementing the syscalls @bacongobbler mentioned, there is also a discussion around network clients and servers - given that WebAssembly does not support multi-threading just yet (here is the proposal), even if sock_recv was implemented, it would only allow for a single-threaded server, which is far from ideal.
Networking clients, which would just open connections to other servers not running in Wasm runtimes should work, as described in the article above, but are still unsupported, both in standard libraries, and in runtimes.
WAGI is our current workaround for building simple, language-agnostic HTTP handlers compiled to WebAssembly, and a Krustlet provider that used WAGI would be a really nice way to temporarily handle this use case.

I figured out that actually I can workaround by using the exec handler for the time being. It may sound extremely dirty but I think it works and I feel it's better, at least for me, than exploring something like WAGI since I'm not familiar with CGI.
That would also allow to reach the WASM module by using the pod name instead of the node IP (which will probably what I'm using since I have issues with the TLS from the apiserver). Otherwise, as far as I understand it, the WAGI solution still doesn't address how other k8s containers can reach the WASM module.
Thanks for the article btw, I'll have a read tomorrow. You already helped me a lot with the article about Tensorflow and WASM. Actually, I'm trying to make an inference model compiled in WASI reachable from a standard container in the cluster.

An interesting question emerged while I was trying to implement the exec in wasi-provider. Currently it's expected for the WASM module to have a _start function which will run endlessly or to completion. That is reasonable because it's what one would expect from a K8s pod.
However, it's not possible currently to execute any exported function from the WASM module other than _start. I was wondering what /exec could mean for a WASM module and if it makes sense to have a module which doesn't have _start or a noop _start.
I think it does. An example that comes to my mind is dnsutils, such as https://kubernetes.io/docs/tasks/administer-cluster/dns-debugging-resolution/#create-a-simple-pod-to-use-as-a-test-environment where it sleeps upon start. Its purpose it then to allow executing bash commands to debug DNS.
For a WASM module, it may make sense not to have _start and acts in a similar way. In my case, I would have a ML inference model which is called only when an inference is needed. Usually one would connect to a socket and listen for incoming requests, but since networking is still missing in WASI, this could be a good compromise simple enough for me to implement.

How you are thinking of using exec is definitely what we thought about doing, but it is kinda weird how we'd make it work without multithreading. Right now you can only call one at a time within the same running module. Otherwise it is kinda more like a FaaS type call

Just a quick update that we have released an experimental library that allows experimental outbound HTTP for guest modules running in Wasmtime, meaning we could explore adding this to Krustlet.
https://github.com/deislabs/wasi-experimental-http


Process one sample run across multiple lanes/multiple libraries

Hi all,
I'm looking for a way of specify lane/barcode/library information when running bcbio-nextgen from multiple fastq file.
The only info I could find is documented there https://bcbio-nextgen.readthedocs.org/en/latest/contents/configuration.html#multiple-files-per-sample, but it essentially means merging all the files together and thus masking any lane/library specific information.
Is there a way of resolving this?
Cheers
Tim

Hi @tcezard,
Could you tell us exactly how you want to have this information?
Normally we join the fastq files for the same sample together. You have the same samples done in different lanes and library  etc ... ?
I guess you want to detect the effect of the lane/library after the analysis? If it is that, I would run the pipeline for each file, and add in the metadata all variables you want to keep. Then you can compare among lane ... and if everything is ok you can run again merging first.
Is this for RNAseq?
I am just guessing, if you tell us more about the experiment design maybe we have more ideas.
thanks!

Hi Timothy,
There isn't a great way of resolving this; if you think that there shouldn't be anything systematically different between the libraries then merging them together before running makes sense. On the other hand if you think there might be a batch effect, then running them separately is the way to go. If this is the same library run across multiple lanes of sequencing machines then it is pretty safe to assume the batch effect will be minimal and you can merge them together. If they are different library preparations though, then normally you would treat them separately throughout the whole process because the preps can have a large batch effect.
Or what Lorena said.

Thanks for the quick responses.
I actually had resequencing/variant calling in mind although this is also (even more so) applicable to RNA-Seq.
Intrinsically, I like the idea of keeping the metadata alongside the bam file and library and lane have an obvious place there.
From a more technical point of view:

GATK base recalibration will treat reads from different lanes separately considering that they might have different error profiles.
We might have mixture of libraries (that very hypothetical) such as PCR free and Truseq nano and still want to call genotype on the merged dataset keeping the library info intact.

On top of that it seems odd to me to merge fastq files together to then split them to increase alignment speed.
Cheers
Tim

Tim;
This would be good to have, although we haven't prioritized it over other work like build 38 support, better parallelization and improved SV and cancer calling. So unfortunately it's not a likely goal in the near future for us, although happy to support community contributions. It would take some work to decide how to define the inputs, and then be able to manage and merge the samples inside bcbio.
Practically, we haven't found a big benefit in running BQSR (http://bcb.io/2013/10/21/updated-comparison-of-variant-detection-methods-ensemble-freebayes-and-minimal-bam-preparation-pipelines/). I also wouldn't suggest combining multiple technologies and calling in a pool. As Rory mentioned you're going to get hard to resolve batch effects from taking that approach.
Sorry to not have this integrated but hope the explanation helps.

Hi
I am closing this because it seems an old issue. And it seems that Brad explained the reason.
cheers

Would like to add that this would be helpful for us too! Currently working on a big project where pre-processing needs to be done for multiple samples where fastq's are split by lanes.

Thanks for the swift reply Brad! Very helpful to know! Much appreciated,
Alon


fix a spelling typo

Pull Request Test Coverage Report for Build 349

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 95.441%

Totals

Change from base Build 346:
0.0%

Covered Lines:
1926

Relevant Lines:
2018

💛  - Coveralls

Pull Request Test Coverage Report for Build 349

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 95.441%

Totals

Change from base Build 346:
0.0%

Covered Lines:
1926

Relevant Lines:
2018

💛  - Coveralls

Thank you!


Top menu not suppressed in full screen video mode

The Issue
When playing a video in the app full screen, the Edit / Help / Developer menu bar is stiill shown.
Steps to reproduce

Go to claim with video (lbry://@IGNonLBRY/linked-ep5)
Click Play, go full screen
Menu bar visible

Expected behaviour
Video should be in full screen
Actual behaviour
Menu bar shown
System Configuration

LBRY Daemon version:

LBRY App version: 0.12.2.rc9

LBRY Installation ID:

Operating system: Windows 10

Anything Else

Screenshots

https://github.com/lbryio/lbry-app/commit/d8e867aa5ae8e9872b7943a0fd8593f9fc68d826


Populate the ownerAccountID in the allowances fields for owner account infos

Problem
ownerAccountID is left null for all allowance token types listed in the account info for an owner.
hbarAllowances=[HbarAllowance{ownerAccountId=null, spenderAccountId=0.0.1948, amount=10 ℏ}], tokenAllowances=[], tokenNftAllowances=[]}
Solution
Populate this field with the owner account ID
Alternatives
No response

Implemented here: 5092b57b5a88126f84f62679e49c92384917d048


ILI9488 Portrait Mode Help

Hello,
I got the portrait mode to work by adding the following in the ILI9488.cpp
madctl |= MADCTL_COLUMN_ADDRESS_ORDER_SWAP;
The problem now is it says the width is 480 (long side) and the height is 720 (short) side
How do I fix this ?

To enable portrait mode, try removing the landscape mode define in config.h: https://github.com/juj/fbcp-ili9341/blob/486a32e7f23b41d49b0e9da4845a86709f6f9c8a/config.h#L161

I tried both the OP method and jul's method but neither work for making my 3.5" ili9488 screen..


Field Importance

Making the “offer name” high importance is a bad decision. It really hurts non-descriptively named apps and forces people to jump through hoops with their product names to get better positioning. It's even worse for Business Central, where the offer name is the same as the app name, which causes technical issues when the name (which should just be marketing text) is changed.
The short description should be a higher importance, followed by the offer name.

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

ID: d1ff1531-133d-4e3f-9b75-6e150b023d04
Version Independent ID: 24ae69e2-5584-14ba-84a8-ed0f32840ca1
Content: Azure Marketplace SEO guidance
Content Source: articles/marketplace/cloud-partner-portal-orig/cloud-partner-portal-seo-guidance.md
Service: marketplace
GitHub Login: @v-miclar
Microsoft Alias: pabutler

@mhamblin Thanks for your feedback . I  would recommend sharing this feedback directly with the Azure Marketplace team by posting the feedback here.

We will now close this issue. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


chore: Update .github/workflows/audit.yaml in `artichoke/strftime-r…

…uby`
Managed by Terraform.
Contents
---
name: Audit
"on":
  push:
    branches:
      - trunk
  pull_request:
    branches:
      - trunk
  schedule:
    - cron: "0 0 * * TUE"
jobs:
  ruby:
    name: Audit Ruby Dependencies
    runs-on: ubuntu-latest

    steps:
      - name: Checkout repository
        uses: actions/checkout@v3

      - name: Install Ruby toolchain
        uses: ruby/setup-ruby@v1
        with:
          ruby-version: ".ruby-version"
          bundler-cache: true

      - name: bundler-audit
        run: bundle exec bundle-audit check --update

  rust:
    name: Audit Rust Dependencies
    runs-on: ubuntu-latest
    strategy:
      matrix:
        checks:
          - advisories
          - bans licenses sources

    # Prevent sudden announcement of a new advisory from failing ci:
    continue-on-error: ${{ matrix.checks == 'advisories' }}

    steps:
      - name: Checkout repository
        uses: actions/checkout@v3

      - name: Install Rust toolchain
        uses: artichoke/setup-rust/audit@v1

      - name: Generate Cargo.lock
        run: |
          if [[ ! -f "Cargo.lock" ]]; then
            cargo generate-lockfile --verbose
          fi

      - uses: EmbarkStudios/cargo-deny-action@v1
        with:
          arguments: --locked --all-features
          command: check ${{ matrix.checks }}
          command-arguments: --show-stats

migrating all artichoke repositories to artichoke/setup-rust.


weasel: Add version 0.14.3

Closes #3665

Link to English docs?

I'm afraid there isn't an English doc for weasel. The only English resource I could find is the librime repo, and weasel is just a Windwos front-end of it. It's a input method engine made by Chinese and mainly targeting Chinese speakers.

Then this faces the same problem as https://github.com/ScoopInstaller/Main/pull/2938#issuecomment-966830432

this looks fairly reasonable to me, feel free to close it

/cc @issaclin32

Weasel is a fairly wide-used Chinese IME tool. It's a pity that we cannot add it.
However, I still think we should keep the rules. Otherwise we would have to accept softwares in Japanese, Vietnamese, Hindi, Russian, etc.
You can use my version of the manifest by:
scoop bucket add issaclin32 https://github.com/issaclin32/scoop-bucket
scoop install weasel

Or simply start your own bucket.


feature suggestion: toggle for default pause icon or fancy dnd5e pause icon

As the creator of Alternative Pause Icon my module currently doesn't function anymore as the added class makes the CSS of dnd5e of higher specificity meaning personalized pause icon + text is no longer possible using my module (while keeping the module general for any system). So it would be lovely if there was a toggle in the system settings where you can disable this extra CSS.
I know a lot of folks like to personalize this pause message. So i hope this suggestion is entertained.

Can't you just increase the specificity of your CSS a bit? Shouldn't stop it from working on other systems.
#pause#pause {
  /* changes */
}

the problem is the css of dnd5e is higher specificity because of selecting the added dnd5e2 class. I cant add that class by default as it would break it for other systems. But if it is a big problem to add a toggle to the system, I will find a way around it.

Fixed on my end.


Python console is very slow to respond when typing in commands

Description
I ran a script in the python console. After it finished, I continue to type commands in the interpreter , but none of these commands are executed. Even a simple "print a" wouldn't execute. Or, if I reassign a "a=5", the variable value in the variable explorer remains unchanged.
The cursor just moves to the next line with the >>> waiting for the next command.
What steps will reproduce the problem?

Open a python console
Run a script
Print a known variable name

What is the expected output? What do you see instead?
I expect it to immediately execute the command within the console, such as return the variable value. The previous spyder version (2.something) didn't have this issue. I just installed spyder version 3.0
Please provide any additional information below
Version and main components

Spyder Version: 3.0.0
Python Version: 2.7.11
Qt Versions: 5.6.0, PyQt5 5.6 on Windows

Dependencies
pyflakes >=0.5.0 :  0.8.1 (OK)
pep8 >=0.6       :  1.5.7 (OK)
pygments >=2.0   :  2.1.3 (OK)
qtconsole >=4.2.0:  4.2.1 (OK)
nbconvert >=4.0  :  4.1.0 (OK)
pandas >=0.13.1  :  0.14.1 (OK)
numpy >=1.7      :  1.11.0 (OK)
sphinx >=0.6.6   :  1.3.1 (OK)
rope >=0.9.4     :  0.9.4 (OK)
jedi >=0.8.1     :  0.9.0 (OK)
matplotlib >=1.0 :  1.5.1 (OK)
sympy >=0.7.3    :  0.7.5 (OK)
pylint >=0.25    :  1.5.4 (OK)

Thanks for reporting. I have some questions:

Could you provide a simple example of the file that you're trying to execute?
How did you install Spyder?

I updated spyder through Anaconda. In the Anaconda command promt: conda
update spyder.
Just about any code results in the error. I attached a screen shot. Here is
some:
import numpy as np
import matplotlib.pyplot as plt
import matplotlib as mpl
from datetime import datetime, timedelta
import os
import sys
def do_some_math(a):
# stuff for making barbs
return a+300*np.pi
if name == 'main':
a = 5
c = do_some_math(a)
print c
http://kbkb-wx.blogspot.com/ Brian
On Thu, Oct 13, 2016 at 12:00 PM, Carlos Cordoba<EMAIL_ADDRESS>wrote:

Thanks for reporting. I have some questions:

Could you provide a simple example of the file that you're trying
to execute?
How did you install Spyder?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/3533#issuecomment-253589946,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF9cjWriBh9HlamVQ6W77FOdIi8kPA0_ks5qznGvgaJpZM4KWIhK
.

Just out of curiosity: What is the use case for the Python console (as opposed to the IPython console)? I never use it myself. I am wondering whether we should just remove it, or maybe do not show it in the default layout.

@jitseniesen, my plan was to remove for 3.0 but I didn't have time to do it at the end :-)
There are some things that still work better in the Python console:

History browsing while you are on Pdb.
2, The ability to evaluate a file in a dedicated console (i.e. a file per console evaluation model).

The ability to browse environment variables and sys.path contents.

and 3. can be easily migrated to the IPython Console, and we could do that in the 3.x series :-) Then I think we could remove the Python console in 4.0 because it's terribly broken (at least on Windows)

I have also problems with python console. I reported on issue #3623 that I closed by error

@jitseniesen and @goanpeca, the Python console is working so badly on Windows that I think we should disable it for 3.1 (not remove it, just disable it) and ask our juniors to solve points 2. and 3. for the IPython Console (which are not that hard :-).
What do you think?

@ccordoba12 you know I am in on removing it for good.
The only thing we need to get this going is to allow running in a new IPYTHON console in the run preferences, and make sure that process is fast.

you know I am in on removing it for good.

We can't remove it for 3.0 because that'd be a major breakage, but I'm all for removing it for Spyder 4.0 :-)

The only thing we need to get this going is to allow running in a new IPYTHON console

Yep, that's point 2.

the Python console is working so badly on Windows that I think we should disable it for 3.1 (not remove it, just disable it)

Agree wholeheartedly.

If you deactivate the Python console, could you please provide a run option (selectable as default in the preferences) which resets the IPython console namespace before running a script (execute reset -f before run(..)).
This would help in teaching Python to beginners which I am currently doing using the Python console. Using a dedicated IPython console instead of a dedicated Python console is too slow for my usage case.

@wtheis, ok, we'll take that into account for 3.1 :-)

This is (at the end) the same problem as the one on issue #1831, so I'm closing it.


backporting: Properly escape commit message when used as regex

This change escapes the commit message when it is used as a regular
expression. Any special character of the posix extended regex
(.^$*+?()[{\|) is now prefixed with a backslash. This fixes an issue where
git log would crash due to to the string being passed to --grep not
being a valid regex, such as e.g. in the commit messages found in
PR #13674 which contain $, * and {.
Example of a failure when processing commit c0b8a82366f573470afe62bdce4463bd8a56174d:
++ git log -F --since=1year --pretty=%H --no-merges --extended-regexp --grep '^docs: use $API_SERVER_{IP,PORT} vars in kube-proxy-free GSG' origin/master
fatal: command line, '^docs: use $API_SERVER_{IP,PORT} vars in kube-proxy-free GSG': Invalid content of \{\}

With the fix applied:
+ entry_sub='docs: use $API_SERVER_{IP,PORT} vars in kube-proxy-free GSG'
++ sed 's/[.^$*+?()[{\|]/\\&/g'
+ entry_sub_re='^docs: use \$API_SERVER_\{IP,PORT} vars in kube-proxy-free GSG'
++ git log -F --since=1year --pretty=%H --no-merges --extended-regexp --grep '^docs: use \$API_SERVER_\{IP,PORT} vars in kube-proxy-free GSG' origin/master
+ upstream=11282c7cde428a01dd9295c77937350ffb1b9fd7

Additional note: For some reason, even with the fix, check-stable still is not able to correlate the commits in the above PR, but I'm 99% sure this is unrelated to the escaping issue addressed here.
Thanks to the fix in this PR, check-stable now finds the correct upstream commit instead of crashing. But due to differences in line numbers (maybe due to the rebase done by GitHub), the computed git patch-id between the upstream sha and the master sha still don't match.

I'm also marking this for backport to 1.7-1.9, such that users who are initiating backport from a release branch will use the updated scripts as well.

One note here is you could also add an ending $ to the regex, so that we only search for the commit title. This will narrow down the possibility of collisions.

Good point, I added the $ at the end and tested it against the current list of PRs which have needs-backport/1.{9,8,7}, seems to work.
Therefore, I'm marking this ready for merge (no need to run CI). There are potential improvements such as querying the GH API, but I currently lack the cycles to investigate them. Let's merge this as is to avoid more person wasting time on this.


Fix help for odo link

/kind documentation
What does this PR do / why we need it:
Fixes incorrect env variables name in example.
Which issue(s) this PR fixes:
Fixes #5175
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[x] Documentation

[x] I have read the test guidelines

How to test changes / Special notes to the reviewer:

/approve
/lgtm


More info about process

Writer is no logging info about his process

skipped tables
table processes
errors
bulk info

Fixed in 0.1.3 60ccb8dbc7d93dbbe5ef91c862e2e178591219a8


gi-gtk-declarative: unbreak

Motivation for this change
the packages gi-gtk-declarative and gi-gtk-declarative-app-simple both compile, but their test suites fail since they try to
access the display to run.
Things done

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

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

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

/rebase haskell-updates

Hi @LuisChDev, changes to haskell need to be delivered to haskell-updates. It seems you've submitted this PR off the master branch, so the bot wasn't allowed to rebase master to haskell-updates in your fork. I think the best solution here would be to locally do
git checkout haskell-updates
git checkout -b $unique-branch-name
git cherry-pick $hash_of_the_commit_in_this_pr

And then open another pull request to merge into haskell-updates. I hope that makes sens.

@SuperSandro2000 Thanks for trying to rebase this with the action.  I wonder why it didn't work?

Oh god, I'm sorry. I'll close this and send another PR to the appropriate branch.


Correct JSON case for HostDBEntry

Are JSONified HostDBEntries ever sent over the network? This would break compatibility if they are.

good catch. No, they are not sent over the network. This will still break compatibility with old hostdbs though.

Oh, cool!


Load additional configuration file from within configuration

Provide an extraConfigFile option that loads an additional JSON configuration file. To keep things simple, this should only be allowed at the top level of a configuration and will not be recursive. The purpose of this file is to provide additional defaults, such as translations.

Hi,
could you describe an example use case?
I have trouble thinking about how or why someone would use this when one can control the main config itself?
I guess for the standalone version where person A uses person B's config but wants other strings for example? Then person A can override the settings without touching javascript. Do I get this right? I mean person A could just copy, modify the config file of B and use it after that.
Am I missing something?
Cheers!

I had a site-wide configuration in mind. Instead of adding a particular set of options to every configuration, they could be included in one file, which could then be loaded from each configuration. If there's a desire to update this set of options, it could be done by editing one file instead of making identical changes to a whole set of files. It has the downside of requiring an additional round trip to the server, though.

Ah, makes sense. I might give this a shot tomorrow!
Cheers!


Merge master into feature branch

For contributor use:

[ ] Targeted PR against master branch
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work
[ ] Code follows the standards mentioned here
[ ] Updated relevant documentation
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Added appropriate labels

Codecov Report
Attention: Patch coverage is 77.41935% with 7 lines in your changes are missing coverage. Please review.

Project coverage is 39.09%. Comparing base (eac9436) to head (9f44b3f).
Report is 55 commits behind head on feature/interactive-init.

Files
Patch %
Lines

internal/dependencymanager/dependencyinstaller.go
77.41%
6 Missing and 1 partial :warning:

Additional details and impacted files
@@                     Coverage Diff                      @@
##           feature/interactive-init    #1563      +/-   ##
============================================================
- Coverage                     39.93%   39.09%   -0.85%
============================================================
  Files                            52       52
  Lines                          2977     3407     +430
============================================================
+ Hits                           1189     1332     +143
- Misses                         1624     1907     +283
- Partials                        164      168       +4

Flag
Coverage Δ

unittests
39.09% <77.41%> (-0.85%)
:arrow_down:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Added output of posterior summaries for ACNV global model parameters.

Also:
-Moved classes to pulldown package.
-Refactored table columns (changed enum names to singular, replaced string/enum errays with TableColumnCollections, etc.)
-Miscellaneous boy scout activities.
Closes #251.

@davidbenjamin, can you review when you get a chance?  I'm going to branch off of this for the tool for Grace.

@samuelklee Sure.

Looks good, back to @samuelklee.

thanks, back to you @davidbenjamin!

@samuelklee good to go after squashing.


Update prometheus client vendor

Update to the recent release (includes a CVE fix) of the Go prom client library
Signed-off-by: Phil Estes<EMAIL_ADDRESS>

Passes locally on macOS with v1.44.2 golangci-lint and the same config as CI.. so ...


Update FAQ.md

Add RFID standard links

Merged in update 62.

Reopening so I can mark this as merged.


Nancy forero patch 1

This needs to be closed.

I moved all of the updates over to the private repo: https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness-pr/pull/12488
Please close this PR.


Client Error in custom Widget - Uncaught (in promise) TypeError: widget is undefined

Im experiencing a ClientError when communicating with a custom widget.
Uncaught (in promise) TypeError: widget is undefined
    _run_widget_hooks http://localhost:8080/static/lona/lona.js:1467
    _run_widget_hooks http://localhost:8080/static/lona/lona.js:1463
    _show_html http://localhost:8080/static/lona/lona.js:1528
    add http://localhost:8080/static/lona/lona.js:70
lona.js:1467:32

LonaWorker_0                   ERROR    19:23:59.862863 lona.view_runtime client error raised:
Lona.LonaWindow/this._run_widget_hooks/<@http://localhost:8080/static/lona/lona.js:1467:32
Lona.LonaWindow/this._run_widget_hooks@http://localhost:8080/static/lona/lona.js:1463:34
Lona.LonaWindow/this._show_html/<@http://localhost:8080/static/lona/lona.js:1528:19
Lona.JobQueue/this.add@http://localhost:8080/static/lona/lona.js:70:27

Code can be found here: https://github.com/laundmo/northstar-server-webui
You should be able to run this without anything else running. Then just type something into the textbox and press enter to get the error (will take a while until the connection attempt to the socket it tries to communicate with times out).
(in case its not clear, i give you permission to run it even though it currently doesnt have a license)

Hi @laundmo,
I got it starting by running python -m lona run-server --project-root=app -s settings.py in the top level of your project but it says
Multiple exceptions: [Errno 111] Connect call failed ('::1', 9998, 0, 0), [Errno 111] Connect call failed ('<IP_ADDRESS>', 9998)

multiple times per second. Can i disable this to see more log?

GET http://localhost:8080/static/autocomplete-widget.js net::ERR_ABORTED 404 (Not Found)

thats ... wtf? i swear that was working like, just a bit earlier.
thank you, and sorry for not noticing that.

@laundmo no problem!
I happen to have an old example on how to use jquery-ui autocomplete, that i plan on turning into a library at some point. Are you interested in an example how i would implement such a thing? Give me a few minutes and i push a branch for you.

I mean, it works for me the way i have it right now, so theres no immediate need.
Though generally, more pre-made widgets and nodes would be really nice to have.

@laundmo I need this anyway in the future, so it's a good opportunity to collaborate on this :)
https://github.com/lona-web-org/lona/tree/fscherf/autocomplete

Ah, i see. i think i will keep my solution, as it doesn't require constant requests just for the autocomplete

Good point. I should add an option for local completion where all possible results get transmitted right away. My solution was build with a database in mind where completion on the server makes more sense.

Yeah, for me its for a command history, so it doesn't make too much sense.


[DPE-4369] SQL queries exposing passwords on postgresql logs

Issue
config value log_statement = 'ddl' logs queries such as ALTER USER/ROLE, CREATE USER/ROLE that expose users passwords on log files.
Solution
set log_statement='none' when submitting sensitive queries:
BEGIN;
SET LOCAL log_statement = 'none';
CREATE ROLE "backup" WITH LOGIN SUPERUSER ENCRYPTED PASSWORD 'MRiT223dummy2322nHESx';
...
COMMIT;

Test
New integration test failing before the change: https://github.com/canonical/postgresql-k8s-operator/actions/runs/9483057657/job/26129752923#step:21:386
... and passing after the change: https://github.com/canonical/postgresql-k8s-operator/actions/runs/9483490912/job/26131217890?pr=506#step:21:182

@lucasgameiroborges The bug in redis-k8s-operator has been fixed.


frontend/main: HtmlWebpackPlugin.options.publicUrl is not applied

Environment

Deploy method: built from sources
Jitsu version:  1.43.3
OS: Alpine Linux
Cloud Provider: none
Docker Params (if deployed with docker): n/a

Description
frontend/main/index.html contains <%= htmlWebpackPlugin.options.publicUrl %> tags for the favicons, editing craco.config.js does not help. The only way to supply the publicUrl is to modify node_modules/.pnpm/react-scripts<EMAIL_ADDRESS>overriding doesn't work either.
Steps to reproduce
Steps to reproduce the behavior:

Edit main/craco.config.js
Add new HtmlWebpackPlugin({publicUrl: "/configurator/"}) to the webpack plugins.
pnpm build

Expected behavior
main/dist/index.html should contain /configurator/favicon.ico and related links.
Actual behavior
/favicon.ico is being output. htmlWebpackPlugin.options.publicUrl is not baked in.

Found the correct way to do this inside the configure section instead.
const htmlWebpackPlugin = webpackConfig.plugins.find(
  plugin => plugin.constructor.name === "HtmlWebpackPlugin"
)

if (htmlWebpackPlugin) {
  htmlWebpackPlugin.options.publicUrl = "/configurator"
}


Add support for BCM2711 high peripheral mode

High peripheral mode moves the peripherals out of the first 4GB of address space. This restores the full 35-bit address map with peripherals at 0x47C000000. Only works on 64-bit kernels for technical reasons but you gain 64MB of RAM. Userland can be 32-bit or 64-bit.

@trejan , please resolve the merge conflicts.


Prevent possible NPEx

LGTM finding

[X] I hereby declare this contribution to be licenced under the Apache License Version 2.0, January 2004

We prefer a style where Collections and Maps are never null, so you can loop without extra null-check.

Is it safe to remove null-check in https://github.com/apache/maven-compiler-plugin/blob/0d1e77325d8dfb55f1a12068bbcc4be304c33b85/src/main/java/org/apache/maven/plugin/compiler/AbstractCompilerMojo.java#L1347 then?

Replaced with #32


[DEVOPS-673] Build Daedalus bridge with Nix

Provides a daedalus-bridge derivation which can be used when building the Daedalus installer.

Nice :)

OK, removed the code to upload explorer.
Also added daedalus-bridge to the hydra jobset.
I just realised that the buildId argument is quite useless when we start using nix-build. We might need to rework iohk-ops find-installers.

@rvl I'm a bit confused here - windows part is missing?

The windows part stays the same here - it just builds CardanoSL.zip. Then the Daedalus installer downloads it off AppVeyor in input-output-hk/daedalus#789.


New PR after Action is set to required

Check if recent push is only a rebase on the base branch is set to only execute on synchronize, not on opened (https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#pull_request) resulting in blocked state for newly opened PR if set to required:

This check was skipped because the latest action was not a push


feat: Create addAirTableRows function

We needed the ability to write to an Airtable table using default mappings
If accepted, this commit and PR will resolve #10

LGTM 👍


Added BMI Calculator

Description

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of Changes

[ ] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

LGTM ! Thanks for contributing 🤓


Add git hook for auto-format node project

In order to maintain reliability of the code, if more than one developer will modify it, it's necessary to have a universal code style.
The easiest way to do it is with a pre-commit hook for auto-formating.
An example is Prettier with Husky.

PR is welcome :)


Problem with links without http

Describe the bug
When link without http is scanned, "Open" button sometimes appear and sometimes doesn't.
Also when such link is opened, linking don't work correctly. Ie. scanning page.com result in address like qrreaderhost.com/page.com
To Reproduce
Steps to reproduce the behavior:

Open scanner
Scan link with http
Close popup
Scan link without http - "Open" button will be present*
Click "Open" - incorrect webpage will open
*Sometimes "open" button will be persistent, sometimes it disappears after few scans of non-http text

Expected behavior
Links without http should be either not recognized as links or script should check if link have http:// and add it if not (I'd prefer second option).
Desktop (please complete the following information):

W10 1909
Chrome 87

Additional context
Imho ideal way of dealing with this would be by using 3 buttons in popup:

Open as website
//checks if text is starting with http:// or https://, if not - adds http:// at the beginning of text and treats it as hyperlink
Copy to clipboard
//copies text to clipboard
Close
//closes popup without additional action

@Aramil-S Thanks for reporting. I will fix it when I get free time.

@Aramil-S #5 Fixed

@Aramil-S #5 Fixed


docs: add guide on integrating user comments with Giscus

Giscus enables dynamic user comments on static sites hosted on Github pages. This post gives an outline on how to integrate Giscus with AstroPaper.
resolves #250

My bad, done :)

Thanks for your contribution @FjellOverflow


Rewrite

TODO comments and better readmes

neovim has bug which causes there to be a small margin with a white translucent border on the left side


Game doesn't start

When theres 2 players in the arena the countdown goes down as normal, but when it hits 0 it just freezes the Server and you cant use any commands or do anything until you restart the server. pls fix

Hello, im one of the people working on this and im aware of this issue ill fix


Add dockerfile and docker-compose support

Motivation and Context
Adding docker support to deploy into kubernetes or run in local docker environment
Description
Adding docker support to deploy into kubernetes or run in local docker environment.
A new folder "docker" created under root, to build images, from root directory, run
docker build -f docker/webapi/Dockerfile -t repo/chat-copilot-webapi
docker build -f docker/webapi/Dockerfile -t repo/chat-copilot-webapp
A docker-compose file is also created to start whole environment, under docker directory, just run
docker-compose up, it uses .env file to read environment variables, so make sure webapi and webapp directory has .env file
the manifests to deply chat-copilot into kubernetes environment can be found here
https://github.com/huangyingting/flux2-gitops/tree/main/apps/base/chat-copilot
Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows the Contribution Guidelines and the pre-submission formatting script raises no violations
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile:

I had a bit of trouble getting the webapp and webapi talking together so I'll put my .env files here for others to use.
\chat-copilot\webapi.env
AIService__Type=OpenAI
AIService__Models__Completion=gpt-3.5-turbo
AIService__Models__Planner=gpt-3.5-turbo
AIService__Key=YOUR_OPEN_AI_KEY
Kestrel__Endpoints__Http__Url=http://<IP_ADDRESS>:8080
MemoryStore__Type=qdrant

\chat-copilot\webapp.env
REACT_APP_BACKEND_URI=http://localhost:8080/

The localhost and <IP_ADDRESS> were throwing me off.


Multiple command argument in runscript broke on 1.7 upgrades

HI guys,
With 1.6.7 this way of passing multiple arguments to the run function used to work:
python manage.py runscript admin_util --script-args=my_function 1234 456 "hello"

def run(*args):
    print *args        # would print (1234, 456, "hello")

But after upgrading to 1.7.0  we get this error:
usage: manage.py runscript [-h] [--version] [-v {0,1,2,3}]
[--settings SETTINGS] [--pythonpath PYTHONPATH]
[--traceback] [--no-color] [--email-notifications]
[--email-exception] [--fixtures] [--noscripts] [-s]
[--no-traceback]
[--script-args [SCRIPT_ARGS [SCRIPT_ARGS ...]]]
[args [args ...]] script [script ...]
manage.py runscript: error: unrecognized arguments: 4
So with the new release how are we supposed to pass multiple arguments via command line
Thanks in advance.

Also see https://github.com/django-extensions/django-extensions/issues/788#issuecomment-228459700

You should remove the = sign from the command:
$ cat scripts/admin_util.py

def run(*args):
    print args

$ ./manage.py runscript admin_util --script-args my_function 1234 456 "hello"
('my_function', '1234', '456', 'hello')

That worked. Thank you . You are awesome. :-) @trbs

Great and no problem, glad to help.
I also updated the documentation since the example there also used =. Which works if there is only one argument but is misleading for multiple arguments.
If you have some time could you look at the documentation of runscript and see if there is room for improvement ?

Great.
I would be good to make the example into multiple command line arguments.
On Wed, Aug 10, 2016 at 4:55 AM, trbs<EMAIL_ADDRESS>wrote:

Great and no problem, glad to help.
I also updated the documentation since the example there also used =.
Which works if there is only one argument but is misleading for multiple
arguments.
If you have some time could you look at the documentation of runscript
and see if there is room for improvement ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/django-extensions/django-extensions/issues/939#issuecomment-238844779,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AArzdoDcRStHupavWRZXHvV92VOhnC_6ks5qebw8gaJpZM4Jf8Td
.

Would you be able to make pull request for that ? (or post text here so I can copy-past it into the docs)


remove implicit groups

this PR removes support for implicit groups (Zarr groups with no metadata documentation) from the spec. See #291 for the motivation. Besides changing the content of the text, I don't know what else to change, so I'm keeping this a draft for now.
A mergeable version of this PR would include a section explaining what implicit groups are, why they were added previously, and why they are now removed. I am happy to add this if it looks like this PR has legs.
Related discussion: #184

@d-v-b, is this PR ready for review?

@MSanKeys963 yes!

how can we move forward with this? I think it's important to make these changes quickly, before we get skew between implementations.

@zarr-developers/steering-council - this PR has sat without review for 4 months. Can we move it forward please?

@d-v-b: do you want to take a look at the conflict?

@d-v-b: do you want to take a look at the conflict?

done!


Rollback react-dates to compatible version

When upgrading react dates to version 20, we introduced some styling issues:

I think it's easier to simply rollback. We have no intent on supporting this custom date picker further.

@connor-baer Ideally our next step is actually revisiting the date picker usage. react-dates has quite a constricting API which has caused us some issues for a while now.


Modify artifacts definition to include the type attribute

There has been a schema migration to use the same xml element to represent different types of artifacts. Here's the PR that introduced the change - https://github.com/gocd/gocd/pull/4278
<artifact src="foo" dest="bar" />
<test src="baz" />

gets converted to
<artifact type="build" src="foo" dest="bar" />
<artifact type="test" src="baz" />

It would seem this https://github.com/gocd-contrib/gomatic/pull/53 does the trick for this issue?

👍

Awesome thanks to all involved :)


Can't get my book to load

i am using enchiridion v3 but every time i make a book and it will not load on the 3rd time i run the game.

provide proof

https://imgur.com/a/2pc57
is the proof good for you

I would ask for the json but it's been so long since I've worked on Enchiridion I've forgot what it's supposed to look like xD.

lol

if i just make a book it will be ok but when i start editing it and adding more than one page it will load the first time i save and quit the game the second time is iffy but the third time i get on minecraft it will not load

https://dl.dropboxusercontent.com/u/92193552/ThePack.json

Thanks for the report @jackmaster9000 !
This seem to be an issue with Enchiridion support in Progression. Maybe you have removed Progression from your pack?
Closing this, since it is not caused by Enchiridion. I suggest you go report it to Progression at https://github.com/joshiejack/Progression :)


Make HTTP Keep-Alive timeout configurable for backend connections

What does this PR do?
Make the HTTP Keep-Alive timeout for backend connections configurable (right now it is hardcoded to 90 seconds).
An integration test was added to verify that the connections are closed in the specified time.
Motivation
There are a couple of open issues indicating that the ability to configure this would be welcome (#727, #3237). Furthermore, testing in our environment where traefik is used as a k8s ingress proxy indicated that the ability to tweak this setting has a measurable impact on the stability and performance of our workload.
More

[x] Added/updated tests
[x] Added/updated documentation

Done in d558bfdd


tmt test lint does not accept all valid attributes

Problem introduced by commit 5d8c146586ac.
Steps to reproduce (from inside the tmt git repository):
tmt test lint | grep fail

Output:
fail unknown attribute 'coverage' is used
fail unknown attribute 'adjust' is used
fail unknown attribute 'adjust' is used

Thanks for filing the issue. The coverage key is not included in the L1 metadata specification. Why do you think it should be supported? Perhaps you are using it for use cases now covered by the new link key?
Regarding the adjust key: There has been a change in the fmf implementation which means that after applying the rules the adjust key is not remove from the metadata node. So I guess we could now include it among the core (L0) attributes.
@lukaszachy, @jscotka, what do you think?

Thank you for the response.
I don't use the coverage attribute myself; I originally noticed the problem with adjust (in tests produced by tmt import (importing Nitrate relevance)) and only used tmt's own tests to illustrate the problem in a simply reproducible way.
I can't comment on the implementation, but find it surprising that tmt's own tests don't pass lint. Is the coverage usage there mistaken?
adjust is documented as working, so should definitely be supported as far as I understand.

I can't comment on the implementation, but find it surprising that tmt's own tests don't pass lint.

Good point! :) We should ensure that all tmt tests are nice & clean. The coverage attribute was an experimental example (before link key was introduced) and thus should be removed.

Cool. tmt lInt is finally becoming useful. Just specification isn't up-to-date

Thanks for filing the issue. The coverage key is not included in the L1 metadata specification. Why do you think it should be supported? Perhaps you are using it for use cases now covered by the new link key?

Isn't this candidate for lint --fix? Similar to relevancy -> adjust conversion.

Regarding the adjust key: There has been a change in the fmf implementation which means that after applying the rules the adjust key is not remove from the metadata node. So I guess we could now include it among the core (L0) attributes.

Let's make it core (L0) attribute.

@psss I'm not so sure about 'adjust'. If I add it to Test._keys it spoils show() a bit
      adjust {'environment': {'METHODS': 'container local'}, 'when': 'trigger == commit'}
             {'environment': {'METHODS': 'container local virtual'}, 'when': 'how == full'}

I'd prefer to fix lint part now and decide how to proceed with #636 in the future.

I'd prefer to fix lint part now and decide how to proceed with #636 in the future.

Yeah, that makes perfect sense to me.

Isn't this candidate for lint --fix? Similar to relevancy -> adjust conversion.

No, the coverage attribute was really just a future dream example. Let's remove it.


Fix for offline in UWP app.

If UWP app was offline, the app would simply crash.

Hi @toddkitta, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (tkitta). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

Thanks for the fix!


Fix #4947: Retain the structure of PreTransRecordTree.

Instead of only retaining the RecordType.
The structure contains the complete information required to map FieldNames to the record's SimpleFieldNames.

Other than the newly added test, this PR causes 0 changes to the generated JavaScript code.

A higher level comment here, before I dive into the specifics: It seems there are two overall approaches to take to class field / record field mapping:

Dynamic: allocate record field names dynamically when we need them; track them. this is the status quo approach which was broken before and this PR fixes.
Static: fixed mapping between class field name / record field names (based on mangling of names, potentially even later, e.g. use field names in record types).

I was wondering if you have explored the static approach. It seems at least at face value, it would keep the optimizer simpler.

It's a good question. I have thought a bit about it. I don't think there's a strong case either way.
One thing going for the dynamic approach is that it produces less code size in fastLinkJS. I am particularly concerned about expanded Longs, which already often "catch my eye" when I skim through test-suite/main.js.
Regarding complexity, I'm not sure the static approach would really be simpler. Overall, I find that keeping the structure is simpler, regardless of the static/dynamic approach. Or at least it's clearer to me. It is more obvious that it does the right thing too, especially when merging the results of If/Match/Labeled.

Rebased to fix conflicts after #5092 was merged. No other changes.


Added tooltip visibility toggle for Imp repellent

I like how this plugin helps to remind me of common secondaries, but Imp repellent may be a tad out of place.
Could a visibility toggle be added to hide the tooltips for Imp repellent?
Personally, I can't see what potion the secondaries are used for on the resolution I usually play either way.
(screenshot example on a wide resolution window)

That's so funny hahah, I knew this potion would cause problems. I'll probably shorten this list to just "flowers" or something, have it be a special case.

Will be available when this PR is merged: https://github.com/runelite/plugin-hub/pull/4676

That's looking very nice, thanks!


'lens-writer' is not in the npm registry

Registry returned 404 for GET on https://registry.npmjs.org/lens-writer

Didn't think about making Lens Writer available as an npm module, but why not. Will consider this when we ship the next version. At times it's still too experimental to deserve an npm release :)
In the meanwhile...
require('substance')

https://www.npmjs.com/package/substance/


Pitch: Enable Github Actions for ci testing

This also helps with performance tuning.

+1. I think this is a good idea to quickly get a CI process running, unless there are plans to integrate Swift-Numerics into Swift's overall CI story. I recently set up GitHub actions for a SwiftPM package of mine and it turned out to be quite easy. You sign up for GitHub actions and add some config files to the repository.
This is how the configuration looks in my package. Check out the Actions tab to see how the CI reports look:

https://github.com/apple/swift-numerics/commit/ee54cc207583f8b481d8ca3e7357819963c2be40
Nothing sophisticated, but wired up and working. I'd like to get the tests and benchmarks more mature before sinking too much effort into the workflows around them.


Change timezone in calculate_business_date

This change is

Coverage decreased (-0.02%) to 95.101% when pulling 9f856fabc5a99161026f8e7e1b141f6dc844ccc8 on oleksiyVeretiuk:fix_utils into 8553be51774a3fc420532e30c21597e9e97e6fe5 on openprocurement:a499878011598746_separating_awarding.

Coverage decreased (-0.02%) to 95.101% when pulling 9f856fabc5a99161026f8e7e1b141f6dc844ccc8 on oleksiyVeretiuk:fix_utils into 8553be51774a3fc420532e30c21597e9e97e6fe5 on openprocurement:a499878011598746_separating_awarding.


Add apply_file and apply_string to BuilderBase

The idea behind this feature is to be able to load a set of rules and apply them without applying all the already existing rules or adding the rules applied to the existing ones.
It's my first contribution to Kivy, so don't hesitate to tell me if I forgot something.
Thanks.

Is this an issue with travis?
The job exceeded the maxmimum time limit for jobs, and has been terminated.

Yeah, it's a travis issue.
Regarding the PR, it is really nice, however, I'm not sure why this would be needed? I.e. in what real situations is this helpful? If it's for situations where you want to apply a rule to widget but not keep that rule around so that it's applied to other widgets of the same class. Then I'd do it differently,
I'd not add new functions for loading kv strings/files just once and not store them, but rather force having to load the kv string/file and store it like in the past, but instead also add a method which force applies a kv rule (even if the class doesn't match) to the widget. This way, you can still load the rule like normal, but give the kv rukle a special name so it's not applied to any normal class and instead is only applied when you force it on the widget.
This would reduce the amount of new code added, and it'll be a lot more efficient and compatible with the upcoming compiled kv code. As you could precompile the rule, and then apply to whatever widget you like.

My need is to be able to load level in a game (an arkanoid) using kv files. Currently, it would keep the rules and loading a level would reload anything already linked to the widget (you can see it here https://github.com/Gagaro/arkanivy/tree/395187ab92def750b92c5d6aba0b23eb662e2453/arkanivy).
So it is mainly to be able to have one time rules.
Are you suggesting to had something new to the kv language directly? It seems like more work to do (and it would make it more difficult to use). My way re-uses what already exists (to load the file, match the widgets to the rules, ...) and only really add one method (apply_file using apply_string).
How is the compiled kv going to work (at load or before loading, ..) ? Is there something I can read on it somewhere?

So for your example, the problem is that you want to apply the ArkanivyGame rule from level1.kv after the ArkanivyGame object has been created and its ArkanivyGame rule from arkanivy.kv has been applied. But you don't want to read and store the new ArkanivyGame rule from  level1.kv because then it'll affect every ArkanivyGame created, but you don't want that. Right?
If so, what my alternate solution is as follows. Rename ArkanivyGame in level1.kv to ArkanivyGame576698434 and then load level1.kv as normal and rules will be read and saved. Now, when you want to apply ArkanivyGame576698434 to the existing ArkanivyGame, all you'd have to do is call Builder.apply_rule(my_game, 'ArkanivyGame576698434'). This will apply that rule to the widget even though the class names don't match.
This alternate solution requires much less to implement, because all you have to do is locate the rule and apply it.
With regards to compiling kv, this loading and not saving would be quite expensive because you'd have to recompile every time. However, if you load it once and apply when asked it should be quite fast. It's the same with current kv though. The kv compiler is a work in progress, you can see the PR here: https://github.com/kivy/kivy/pull/3456.

Ok I see what you mean now. It sounds better indeed. Thanks for your input, I'll try to rewrite my PR asap.

Done. I had to add a match_widget_name because the match method needs an actual widget to get the matching rules.

Fixed pep8 and renamed widget_name.

Awesome!


SSI: implement #fsize and #flastmod commands

This PR implements #fsize and #flastmod commands for SSI module to better mimic Apache SSI implementation.

Thank you for this pull request. Unfortunately, I can only state the fact that being busy with more frequently requested functionality, we still couldn't found any free resources for a review.
I hope it does happen sooner or later.


Issue logging for Streak_Calendar

It would be good if you had issue tracking for the Streak_calendar as it would show developers that the component is likely to be maintained and therefore more confidence.

It would be good if you had issue tracking for the Streak_calendar as it would show developers that the component is likely to be maintained and therefore more confidence.

Hello, Streak_Calendar is a fork of this project and is not maintained by me.


DW should ignore high_availability_replica_count

Description
An existing property was renamed and I missed adding the ignore for the renamed version.

This checklist is used to make sure that common guidelines for a pull request are followed.

[x ] The PR title and description has followed the guideline in Submitting Pull Requests.

[x ] I adhere to the Command Guidelines.

[x ] I adhere to the Error Handling Guidelines.

sql

sql


2.7.0 (and beyond) - Tracking any/all feature requests

I have created a 2.7.0 branch which I'm thus far using to work on the following new features:

#288 Add ACARDS support
#279 Add HAB LoRa Gateway using dedicate LoRa Radio
#223 + #249 Add GPS with timing data fed to NTP
#248 Generic Feeder for BEAST/MLAT data
#291 Use Kalibrate to calibrate tuners

Please feel free to suggest additional features or functionality to be considered by Joe for inclusion in this release; however please ensure to raise a separate issue thread and link to it rather than making suggestions directly :)

Going to close this one out so your fork is not confused with this project being this one will be going in a totally different direction than it currently is with the upcoming v3.x release. Please feel free to keep your fork running! I love the fact someone has shown so much interest in the project.


creator is not populated correctly

Both OpenTopography and CanWIN have an issue where no creator is stored, making the citations incorrect.
CanWIN example: DataONE representation, dataset landing, schema validation
Citation:
(2023). Dease Strait Mooring Chl-a and Light Data - 2017 and 2019. Canadian Watershed Information Network (CanWIN). [10.34992/jt0y-en67](https://doi.org/10.34992/jt0y-en67), version: sha256:bf5660b425f554166b3a97d1a827792ec62a526925e332a3f805018ce12d0843.

OpenTopography example: DataONE representation, dataset landing, schema validation
Citation:
(2022). Lidar Survey of Dangermond Preserve, CA. OpenTopography. OTSDEM.042022.32611.1, version: sha256:359b20070883f69c5cd5f5b77d7f91c95a6cf31cffaa12fcc05c515946d3c86e.

sonormal (and by extension mnlite) needs to be more adaptable when looking for dataset creators. I think this is set in sonormal.normalize._forceSODatasetLists.

I'm having trouble figuring out where to find this problem. It might actually have something to do with how mnlite serves creator information to the requestor.

What do you mean, to the requestor? mnlite as a member node serves its SO metadata documents in json-ld to the DataONE syncronization system, which grabs an exact copy of the SO document. That is transferred to and stored on the CN in its Metacat object store, and then an event is triggered to queue indexing for that document. The dataone indexer then picks up that queue entry, and attempts to extract the information from the SO JSON-LD document using a JSON-LD subprocessor, which loads the SO document and a number of other vocabularies into an in-memory triple store and runs SPARQL queries on the content to extract values, which are then updated in SOLR.
For example, here's a link to the subprocessor configuration for extracting a single creator from SO to populate the SOLR author field, showing its SPARQL query. Similar queries are used to populate the other SOLR fields.

I didn't understand the simplicity of how mnlite serves records, so that makes sense.
In that case, maybe it makes sense to look at the SPARQL query. I'm not that familiar with SPARQL but I understand some SQL. Here is a CANWIN creator field:
    "creator": {
      "@type": "Role",
      "creator": {
        "@type": "Person",
        "Affiliation": {
          "@type": "Organization",
          "name": "Centre for Earth Observation Science - University of Manitoba"
        },
        "Email"<EMAIL_ADDRESS>        "Identifier": {
          "@type": "PropertyValue",
          "propertyID": "https://registry.identifiers.org/registry/orcid",
          "url": "https://orcid.org/0009-0001-2454-4614",
          "value": "0009-0001-2454-4614"
        },
        "Name": "Yendamuri, Kiran\t"
      }
    },

And here is the SPARQL query:
PREFIX rdf:  <http://www.w3.org/1999/02/22-rdf-syntax-ns#>
PREFIX list: <http://jena.hpl.hp.com/ARQ/list#>
PREFIX SO:   <http://schema.org/>

SELECT (?name as ?author)
WHERE {
    ?dsId rdf:type SO:Dataset .
    ?dsId SO:creator ?list .
    ?list list:index (?pos ?member) .
    ?member SO:name ?name .
}
order by (?pos)
limit 1

@iannesbitt the actual SPARQL query for the origin field, which is what is used for the creator list in citations, is here:
https://github.com/DataONEorg/dataone-indexer/blob/develop/src/main/resources/application-context-schema-org.xml#L304
PREFIX rdf:  <http://www.w3.org/1999/02/22-rdf-syntax-ns#>
PREFIX list: <http://jena.hpl.hp.com/ARQ/list#>
PREFIX SO:   <http://schema.org/>

SELECT (?name as ?origin)
WHERE {
    ?dsId rdf:type SO:Dataset .
    ?dsId SO:creator ?list .
    ?list list:index (?pos ?member) .
    ?member SO:name ?name .
}
order by (?pos)

Note that it assumes list structure. Here's your example doc (slightly enhanced) that does not use a list but rather a creator Role embedded in creator:
{
"@context": "https://schema.org",
"schema:Dataset": {
  "@type": "schema:Dataset",
  "name": "Test dataset",
  "creator": {
      "@type": "Role",
      "creator": {
        "@type": "Person",
        "affiliation": {
          "@type": "Organization",
          "name": "Centre for Earth Observation Science - University of Manitoba"
        },
        "email"<EMAIL_ADDRESS>        "identifier": {
          "@type": "PropertyValue",
          "propertyID": "https://registry.identifiers.org/registry/orcid",
          "url": "https://orcid.org/0009-0001-2454-4614",
          "value": "0009-0001-2454-4614"
        },
        "name": "Yendamuri, Kiran"
      }
    }
  }
}

Here's a SPARQL query to retrieve both the name and email from that. Somehow we need to support these multiple encoding approaches:
PREFIX rdf:  <http://www.w3.org/1999/02/22-rdf-syntax-ns#>
PREFIX list: <http://jena.hpl.hp.com/ARQ/list#>
PREFIX SO:   <http://schema.org/>

SELECT ?name ?email
WHERE {
    ?dsId rdf:type SO:Dataset .
    ?dsId SO:creator $creator .
    $creator SO:creator ?role .
    $role SO:name ?name .
    $role SO:email $email .
}

This produces the following results:
{
  "head": {
    "vars": [
      "name",
      "email"
    ]
  },
  "results": {
    "bindings": [
      {
        "name": {
          "value": "Yendamuri, Kiran",
          "type": "literal"
        },
        "email": {
          "value"<EMAIL_ADDRESS>          "type": "literal"
        }
      }
    ]
  },
  "metadata": {
    "httpRequests": 46
  }
}

In an ideal world we would also capture the ORCID and Affiliation too.

@mbjones should I open an issue or PR in the indexer to track this?


PKG-5563: Update transformers to v4.44.1

transformers 4.44.1
Destination channel: defaults
Links

PKG-5563
Upstream repository
Upstream changelog/diff

Explanation of changes:

Update transformers to v4.44.1

Depends on a newer version of huggingface accelerate

Confirmed working with the updated huggingface_accelerate: https://github.com/AnacondaRecipes/accelerate-feedstock/pull/5


Update ToggleButton.js

Removing height and changed padding to 6px,16px as per the issue

[x] I have followed (at least) the PR section of the contributing guide.

Please check the ToggleButton.js and if everything is good please pull request...

Will this also solve the different sizes? I guess based on the sizes there should be different paddings
https://material-ui.com/components/buttons/#sizes

@rehanmohiuddin @Primajin thanks for your help


Make helpdesk email address and platform URL configurable at runtime

In this PR I suggest changes that extend the configurability at runtime, similar to how it works in the Platform front end (platform-app).
Summary of changes:

Update Dockerfile and adopt format used in platform-app.
Remove environment variables used at build time (REACT_APP_).
The files src/configuration.js and nginx.conf became obsolete and were therefore removed.
Use config.helpdeskEmail for both contact elements in src/constants.js.
Add DEVOPS_CONTEXT_ variables to public/config.template.

I hope you think this is useful.

Thank you @romanhaa for opening this PR! 👍
@carcruz is currently reimplementing the configuration for the genetics-app to match the platform-app. Part of this will be adjusting the configuration to use our profiles-based approach. Once he has opened and merged his PR, I'll ask him to review.


AddImagePart adds Media folder and images to Root instead of the Word sub folder within the document package

Describe the bug
We have been trying to work out why when some 3rd party systems convert a Word document that we have generated using the Open XML SDK that images are not visible and came across other people with the same issue.
We have done quite a bit of investigation and research and it appears to be down to the SDK putting the images and relationships in a media folder in the root of the package, rather than in the word sub folder.
If one of these documents is editing in Word then it detects that this is wrong and moves the images and then the document is fine again.  Someone posted on the internet about this 3.5 years ago and have a work-around which was to insert the images manually and avoid the AddImagePart function but his sample code no longer works in the new versions and it seems to make more sense to resolve the issue at source rather than trying to fudge a way round it.
sample code that shows the behaviour:
ImagePart imagePart = mainPart.AddImagePart(ImagePartType.Png);
imagePart.FeedData(imageStream);
I also tried using an embedded object to see if I could make that work in the meantime and although it didn't do what I wanted it does put the EmbeddedObjects in the correct WORD sub folder so the functionality appears to be inconsistent:
var imagePart = mainPart.AddEmbeddedObjectPart(ImagePartType.Png);
imagePart.FeedData(imageStream);
Screenshots
Word generated document has created the media folder under the word folder:

OpenXMLSDK generated document has created the media folder in root:

To Reproduce
Steps to reproduce the behavior:
ImagePart imagePart = mainPart.AddImagePart(ImagePartType.Png);
imagePart.FeedData(imageStream);
Observed behavior
The image uploads to the \media folder within the package.
Expected behavior
The image should upload to the \word\media folder within the package.
Additional context
If the AddImagePart function could be changed to upload media files into the word sub folder then that would be ideal but if that proposes a problem then having an additional parameter would be a good alternative solution to get the desired result.
Desktop (please complete the following information)
OS: Docker mcr.microsoft.com/dotnet/aspnet:8.0
Office version [e.g. 16.0.15427.20178]
.NET Target: .net8
DocumentFormat.OpenXml Version: 3.0.2

Any news on this? I also have to keep using the old version (2.19.0), because of this issue.

I have the same issue.

This is the code I used as a workaround in the meantime if it helps anyone:
// Add physical image to main document part
UInt32Value useId = ReturnRandomUInt32Value(1000000000,<PHONE_NUMBER>);
// -----------------------------------------------------------------------------------------------------
// This but uploads the image into \media and which Word moves to \word\media if amended so logged with Microsoft
// ImagePart imagePart = mainPart.AddImagePart(ImagePartType.Png);
// imagePart.FeedData(imageStream);
// string relationshipId = mainPart.GetIdOfPart(imagePart);
// -----------------------------------------------------------------------------------------------------
// New code to replace the above - If Microsoft resolve the AddImagePart function can be swapped back
Uri imageUri = new("/word/media/" + useId + ".png", UriKind.Relative);
Uri imageUriInternal = new("media/" + useId + ".png", UriKind.Relative);
var docPackage = mainPart.OpenXmlPackage.GetPackage();
var imagePart = docPackage.CreatePart(imageUri, "image/png", System.IO.Packaging.CompressionOption.NotCompressed);
imageStream.CopyTo(imagePart.GetStream(FileMode.Open, FileAccess.Write));
var mainPartPackage = docPackage.GetPart(mainPart.Uri);
var docRelationship = mainPartPackage.Relationships.Create(imageUriInternal, System.IO.Packaging.TargetMode.Internal, "http://schemas.openxmlformats.org/officeDocument/2006/relationships/image");
string relationshipId = docRelationship.Id;
// -----------------------------------------------------------------------------------------------------


Ss 2670 enhance alias actions

LGTM

Code Coverage is 82.0%, details at https://s3.amazonaws.com/rs-code-coverage/rsc/2015-09-24_447_Coverage-82.0.txt (sometimes the URL can take a few minutes to be available)


Error messages

Bug description
Upon following the README.md to install, successfully running given the examples within python projects. When I run outside of a python project; I get an error on a windows system. It's needlessly verbose.
This isn't just about me testing the fences or being obtuse. Sometimes CI and build servers (particularly in mono-repos) need to switch environments and working directories. This would absolutely impact the case for a windows runner in the wrong folder.
Reproduction steps
python3 -m pip install -U pip-audit
pip-audit

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots and logs
$ pip-audit
WARNING: You are using pip version 20.3.1; however, version 21.3.1 is available.
You should consider upgrading via the 'c:\python38amd64\python.exe -m pip install --upgrade pip' command.

←[?25hTraceback (most recent call last):
  File "c:\python38amd64\lib\site-packages\pip_audit\_dependency_source\pip.py", line 64, in collect
    for (_, dist) in pip_api.installed_distributions(local=self._local).items():
  File "c:\python38amd64\lib\site-packages\pip_api\_installed_distributions.py", line 83, in installed_distributions
    return _new_installed_distributions(local)
  File "c:\python38amd64\lib\site-packages\pip_api\_installed_distributions.py", line 71, in _new_installed_distributions
    for raw_dist in json.loads(result):
  File "c:\python38amd64\lib\json\__init__.py", line 357, in loads
    return _default_decoder.decode(s)
  File "c:\python38amd64\lib\json\decoder.py", line 340, in decode
    raise JSONDecodeError("Extra data", s, end)
json.decoder.JSONDecodeError: Extra data: line 2 column 1 (char 30318)

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "c:\python38amd64\lib\runpy.py", line 193, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "c:\python38amd64\lib\runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "C:\Python38amd64\Scripts\pip-audit.exe\__main__.py", line 7, in <module>
  File "c:\python38amd64\lib\site-packages\pip_audit\_cli.py", line 240, in audit
    for (spec, vulns) in auditor.audit(source):
  File "c:\python38amd64\lib\site-packages\pip_audit\_audit.py", line 60, in audit
    yield from self._service.query_all(specs)
  File "c:\python38amd64\lib\site-packages\pip_audit\_service\interface.py", line 115, in query_all
    for spec in specs:
  File "c:\python38amd64\lib\site-packages\pip_audit\_dependency_source\pip.py", line 81, in collect
    raise PipSourceError("failed to list installed distributions") from e
pip_audit._dependency_source.pip.PipSourceError: failed to list installed distributions

Platform information

OS name and version: Windows 10 64-bit Pro
pip-audit version (pip-audit -V): 1.0.0
Python version (python -V or python3 -V): 3.8.1 😬 (I'll upgrade that separately)
pip version (pip -V or pip3 -V): 21.1.2

Additional context
I Understand I should perhaps use flags.
I expected this to be the default, or for usage to be output if I supply no default command.

[X] pip-audit -l works
[X] pip-audit --help works
[X] pip-audit -r /path/to/requirements.txt works for both existing and non-existing paths

Same behavior when using python 3.10.0 with up to date pip 👍

@Lewiscowles1986
Thanks for the bug report and showing interest in pip-audit!

This isn't just about me testing the fences or being obtuse. Sometimes CI and build servers (particularly in mono-repos) need to switch environments and working directories. This would absolutely impact the case for a windows runner in the wrong folder.

Just to clarify, if you run pip-audit from a different directory, does the behaviour change? I don't think the working directory should affect much as it just queries the output of pip list which you can run from anywhere.
@woodruffw I suspect this is the same as #122 (specifically the stack trace in https://github.com/trailofbits/pip-audit/issues/116#issue-1049107467).

The working directory should not matter for pip-audit -l or in-fact pip-audit (no arguments) in my opinion. That is not the subject of this issue. Operating from a different folder generates the same issue.

Hmm, that JSONDecoderError is fishy — looks similar to #116, which we never fully root caused.
Sent from mobile. Please excuse my brevity.

On Dec 1, 2021, at 10:46 PM, Lewis Cowles @.***> wrote:

The working directory should not matter for pip-audit -l or in-fact pip-audit (no arguments) in my opinion. That is not the subject of this issue. Operating from a different folder generates the same issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

@Lewiscowles1986 Can you give us the output of pip list -v --format=json in both a directory where it works and a directory where it doesn't work?

Umm.. The output is the same regardless of this working...
$ pip list -v --format=json
[{"name": "backports.entry-points-selectable", "version": "1.1.1", "location": "c:\\users\\lewis\\appdata\\roaming\\python\\python310\\site-packages", "installer": "pip"}, {"name": "distlib", "version": "0.3.3", "location": "c:\\users\\lewis\\appdata\\roaming\\python\\python310\\site-packages", "installer": "pip"}, {"name": "filelock", "version": "3.4.0", "location": "c:\\users\\lewis\\appdata\\roaming\\python\\python310\\site-packages", "installer": "pip"}, {"name": "pip", "version": "21.2.3", "location": "c:\\program files\\python310\\lib\\site-packages", "installer": "pip"}, {"name": "platformdirs", "version": "2.4.0", "location": "c:\\users\\lewis\\appdata\\roaming\\python\\python310\\site-packages", "installer": "pip"}, {"name": "setuptools", "version": "57.4.0", "location": "c:\\program files\\python310\\lib\\site-packages", "installer": "pip"}, {"name": "six", "version": "1.16.0", "location": "c:\\users\\lewis\\appdata\\roaming\\python\\python310\\site-packages", "installer": "pip"}, {"name": "virtualenv", "version": "20.10.0", "location": "c:\\users\\lewis\\appdata\\roaming\\python\\python310\\site-packages", "installer": "pip"}]

It is for sure a spicy meatball. Unfortunately debugging in windows is awful. And on Linux it works.

TBF, I will just avoid using this on windows. That is my fix for myself 😄


AWS check if bagged_score.csv exists

It checks if the bagged score file (the last score saved by ramp) exists on the AWS ec2 instance.
It should close #483 and close #499

The error happens because although the screen does no longer exist the score session is not saved.
The outcome of the submission might be:

successful (we can check if the bagged_scores.csv exists)
failed (check if error.txt exists in one of the fold_* directories).

this is not a very nice solution and I don't know how to properly test it using pytest
still.. it might work temporarily...

The error happens because although the screen does no longer exist the score session is not saved.
The outcome of the submission might be:

successful (we can check if the bagged_scores.csv exists)
failed (check if error.txt exists in one of the fold_* directories).

this is not a very nice solution and I don't know how to properly test it using pytest
still.. it might work temporarily...

@tomMoral or @kegl could you please also look at it before merging?
Thanks in advance

@tomMoral or @kegl could you please also look at it before merging?
Thanks in advance

What's the conclusion on this? Can we merge it or not?

I have no clue

Me neither. I guess it depends on the need: is ramp running into this issue nowadays?
This was just a patch, if it's not urgent better solution might be found.

Yes, I run into this one multiple times with follicles because of some test-time error which created the fold0 but not the others.
I would say it is nice to fix this.
But we should finish this and detect the broken state (and report the error).


pip install ffmpeg-python

https://github.com/kkroening/ffmpeg-python

no need 🙅♂

no need 🙅♂


Fix Tests

Test fixes are needed.

Need more utilization of TestFX.

Need a way to utilize DISPLAY environment variable on Linux for tests.


Java install failing on Windows since 1.37

Looks like 1.37 found its way to the supermarket yesterday.
Since then I'm getting errors installing java.
[2016-01-11T16:43:29-08:00] FATAL: Mixlib::ShellOut::ShellCommandFailed: windows_package[Java 8 Update 51 (64-bit)] (java::windows line 80) had an error: Mixlib::ShellOut::ShellCommandFailed: Expected process to exit with [0, 42, 127], but received '-80'
---- Begin output of start "" /wait "c:\windows\temp\chef_temp\local-mode-cache\cache\jre-windows-x64-<IP_ADDRESS>.exe"   /s INSTALLDIR="C:\Program Files\Java\jre1.8.0_51" /INSTALLDIRPUBJRE="" & exit %%%%ERRORLEVEL%%%% ----
STDOUT:
STDERR:
---- End output of start "" /wait "c:\windows\temp\chef_temp\local-mode-cache\cache\jre-windows-x64-<IP_ADDRESS>.exe"   /s INSTALLDIR="C:\Program Files\Java\jre1.8.0_51" /INSTALLDIRPUBJRE="" & exit %%%%ERRORLEVEL%%%% ----
Reverting back to version 1.36.0 works perfectly.

This was the only change was this - https://github.com/agileorbit-cookbooks/java/commit/6f0fad78858425a402866b3555a599b124e613a7#diff-4f50112fe9c68776e2852d8d5e6b887cR75
It looks like this if is bad https://github.com/agileorbit-cookbooks/java/commit/6f0fad78858425a402866b3555a599b124e613a7#diff-4f50112fe9c68776e2852d8d5e6b887cR75
@scottresnik any thoughts on that?

Looks like the nil default being set for the public jre location is breaking the attribute existence check. @erichelgeson would you prefer to have the default removed, or augment the if to check for the attribute existence, and it being not nil?

Ahh I see, the .attribute?() is nil safe but even the existence of the key (being nil) causes it to be true.
I think the second option (augment to check existence and not nil) would be the least surprised. Incase anyone ever wanted to unset that option they could easily do it by setting it to nil

I agree.  I'll submit a new pull request.

v1.38 pushed to supermarket with the fix. Thanks @scottresnik
@ggascoigne - can you re-test and close (sorry i have no windows machines to test with)

Looks like there's a :wq file in the root of the package for 1.38. As a result it's failing to sync on Windows Chef:
  ==============================================================================
==
  Error Syncing Cookbooks:
  ==============================================================================
==

  Unexpected Error:
  -----------------
  Chef::Exceptions::FileNotFound: Cannot find c:/chef/cache/cookbooks/java/:wq f
or cookbooks/java/:wq!

Was editing in vim and accidentally wrote that file out, ha, it has been re-uploaded without it.

Heh, I thought Vim had some part in it... :) Thanks for speedy fix!


Disable asset debug in development

Description
We've been getting complaints that the initial request to local diaper servers takes an extremely long time and can sometimes just 'hang'. This makes it appear that the server isn't working.
I've found that the assets are the primary reason why this is happening. To address this I've set assets.debug=false.
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Test trying to access http://localhost:3000/ before and after this PR. Make sure to clear your assets via:
rm -rf tmp
rails assets:clean

You should see that before this option the server just hangs and then after it should be very fast.

When I try to load now I see this:

No dice for me. Looking at the blame on this file, I wonder if it's an issue with trying to do mail delivery for people without it setup?

I don't think so with this one. The assets are likely the culprit here.

When I try to load now I see this:
I think this is a related but different issue with using sprockets 4.0

I'am going to merge this in for now since I think debug is going to address the initial problem. Then I can focus on figuring out why that is happening (I had this error happen to me before).


old content wont show up for me on alerts and time line for some reason ?

Description
old content from arr all the way to shb wont show up to me on alerts or time line for some reason even tho i have everything enabled
Description of the issue you're seeing.
Additional information
If available, please include any logs or screenshots depicting the issue you're experiencing.

Check https://github.com/quisquous/cactbot/issues/4848 or ask in the ACT discord for more support.


Validating generated csv files

@stijnvanhoey CSV files can be validated with:

Create a datapackage.json file with the following content:

{
  "profile": "tabular-data-package",
  "resources": [
    {
      "name": "vpts",
      "path": "vpts.csv",
      "profile": "tabular-data-resource",
      "format": "csv",
      "mediatype": "text/csv",
      "encoding": "utf-8",
      "schema": "https://raw.githubusercontent.com/enram/vpts-csv/main/vpts-csv-table-schema.json"
    }
  ]
}

Place the datapackage.json file in the same directory as your CSV file. Rename the path value if necessary to point to the CSV file (above named vpts.csv).
Install https://github.com/frictionlessdata/frictionless-py, I'm using v4.40.0
Running in CLI: frictionless validate datapackage.json

handles on frictionless so-called "resource" level in https://github.com/enram/vptstools/pull/19


⚠ Bofrost.be has degraded performance

In e6b8831, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1137 ms

Resolved: Bofrost.be performance has improved in 0ac3122 after 9 minutes.


split the eval of operator into multiple steps to get the circuits

What is the expected enhancement?
Now eval method in the operator includes multiple steps, so intermediate might be useful for the users.
E.g., users want to get the cirucits for the evaluation.
It will be better we split the steps of eval into multiple methods and then users have higher flexible to do that.
We can split it into three steps:

Generate the circuits for evaluation
Run the circuits to get results
Use the result with Operator to calculate the observable.

With this enhancement, we create the potential to bundle the circuits of different parameters in wavefunction during the VQE.
E.g., in SPSA, we need to evaluate Operate against with theta+ and theta-. if we can bundle the circuits of theta+ and theta- together, we can submit them together, rather than run them separately.

resolved by #160 and #161


Limiting Frame Ticks

In the frame tick provider, on this line, it basically caps the tick time if it's more than the maximum.
Instead of doing this, because of the way euler integration works, wouldn't it make more sense to write something like:
while (frameTime > maximumFrameTime) {
     signal.dispatch(frameTime * timeAdjustment);
     frameTime -= maximumFrameTime;
}

This would allow someone to pick a low maximum time (eg. I know I have fast-moving bullets and collision problems, so I pick something like 20ms) and get consistent gameplay.
Does that make sense?

You can write your own TickProvider. But I suppose you does not need that smart calculation for all systems - it is need only for physics. So you could do your "while" calculation only for specific systems.


fix l2 penalty

fix l2 penalty covariance matrix
fix return target optimizer

@Gubba-tech I merged these changes into #14 and fix the test cases I think we can close this pull request without merging it.


Improve view load timer

Add extra filtering and options for measuring load times

Coverage remained the same at 62.036% when pulling 6e2b0b88bca90009e5c0b48b9210efe558403d90 on webpack-load-timer into 66bf22129a950ea800bd5686dad8f2eb0ef16da4 on webpack.

Coverage remained the same at 62.036% when pulling dd7ec9b95eed94c768460b39510e074a494089fd on webpack-load-timer into 66bf22129a950ea800bd5686dad8f2eb0ef16da4 on webpack.

Changes Unknown when pulling 9b20150efbd127b557aedbb7c9fd28e66d17d272 on webpack-load-timer into ** on webpack**.

Coverage remained the same at 62.036% when pulling 9b20150efbd127b557aedbb7c9fd28e66d17d272 on webpack-load-timer into a236fa2cd680871dee1864ecf4daa71c731c1091 on webpack.


PR resource names include build number

Description
Changes the names of the resources created for testing a PR to include the build number in addition to the PR number.
Related issues
Previously we have encountered an error where resource names were conflicting with existing resources. This would require a new PR to be created as the names of the resources was based only on the PR number. This change will change the names of the resources on each build, allowing a rerun of the build to fix this issue.
Testing
Running the PR.
FHIR Team Checklist

[x] Update the title of the PR to be succinct and less than 50 characters
[x] Add a milestone to the PR for the sprint that it is merged (i.e. add S47)
[x] Tag the PR with the type of update: Bug, Enhancement, or New-Feature
[ ] Tag the PR with Azure API for FHIR if this will release to the managed service
Review squash-merge requirements

/azp run


Nested tasks by group

Is your feature request related to a problem? Please describe.
Currently the tasks are listed without any grouping. It would be useful to show the tasks in their respective groups.
Describe the solution you'd like
Show a grouped nesting in the explorer view.

This is fixed in #56. Released with https://github.com/badsyntax/vscode-gradle/releases/tag/2.2.0


tests(stream) - stream connections are also counted

The CI has been failing since the https://github.com/Kong/kong-plugin-prometheus/commit/d125bf35ce2b39426609ce09c94ef21771279314 commit. @javierguerragiraldez Could you look into it and see what is wrong here?

The CI has been failing since the https://github.com/Kong/kong-plugin-prometheus/commit/d125bf35ce2b39426609ce09c94ef21771279314 commit. @javierguerragiraldez Could you look into it and see what is wrong here?

AFAICT, some tests verify that failure cases fail exactly as specified, so I haven't been able to run outside of Pongo, which doesn't help when you are developing on the Kong side too.  (at least I think it requires pulling Kong from an "official" docker image, not from a local directory or an unmerged branch)  I managed to get it passing agains Kong CE, but EE is a different can of worms...

AFAICT, some tests verify that failure cases fail exactly as specified, so I haven't been able to run outside of Pongo, which doesn't help when you are developing on the Kong side too.  (at least I think it requires pulling Kong from an "official" docker image, not from a local directory or an unmerged branch)  I managed to get it passing agains Kong CE, but EE is a different can of worms...

I think the tests faield because the stream_rpc listener is not defined in custom_template. While CE uses 2.1.4 where kong.tools.stream_api is not available, it skipps the stream subrequest. But when running with CE >= 2.3, it will suffer same issue with EE.

https://github.com/Kong/kong/pull/6952


Closed Issues show wrong user information

Gitea version (or commit ref): 1.9.3

Description
The list of closed issues shows the name of the creator of the issue as the closer, even if the closer was different user.

The picture above should say: "opened 3 days ago by cga..., closed 3 days ago by sdg...."
This is the view of the issue:

We're not saving information about the user that closed the issue. Perhaps the message should be something like:

opened 3 days ago by poster, closed 2 days ago

Would be so much more helpful if the closer was shown as well without having to open the issue to see that. (And BTW, wouldn't it be nice if the issue number in the red box was also clickable like the issue title.)

@guillep2k could you confirm that if this has been resolved?

@guillep2k could you confirm that if this has been resolved?

Sadly, no (2677d071f911fb91f382acfedaedc251bd807f70).
URL:
http://mytestsite:3000/test-org/test-repo/issues?q=&type=all&sort=&state=closed&labels=&milestone=0&assignee=0

The issue list shows the wrong info. If I navigate into the issue, I can see the correct user that closed the issue as a comment. The problem is that the issue list (/:userid/:repoid/issues) does't have the closer info at hand in the issue table.


Create new content and update content table

Create new content and update links

Resolved issue: https://github.com/akebu6/Kotlin-Content/commit/6d29bbbc6bdd5ffada5cef0c2984a41bd33191bb


fix(workflows): Throw errors for duplicate workflows

Changes

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1011  👈
#1010
#1009
#1008
#1000
#1007
#1001
#999
#998
#996
#995
#988
#986
#974
#987
#977
#965
#964
#963
#962
#957
#956
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @MasterPtato and the rest of your teammates on  Graphite


Syntax issue ( allowable, but causes error from fqm-execution )

Summary
CQL contains a strange, but allowable syntax
or "Macular edema absent (situation)" in MacularEdemaAbsentNotCommunicated.reasonCode
When that clause is included in the CQL, fqm-execution returns an error ( e is undefined )
Expected Behavior
parsed & executed successfully
Stan Rankins reports

Greg Akins
1 day ago
Do you think the syntax “should” work, or it should through an error from CQL-ELM-Translator (which it’s not)?
Stan Rankins
1 day ago
That's what they are being told to do by Bryn - https://github.com/cqframework/CQL-Formatting-and-Usage-Wiki/wiki/Authoring-Patterns---QICore-v4.1.1#communication-not-done

Version or Commit
1.0.8
Inputs (e.g. Measure Bundle, Patient Bundle, CQL Library)
bundleAndTestcas.zip
Relevant Calculation Options (e.g. Measurement Period, meta.profile Validation)
cql-to-elm Version Used for Measure Logic Translation (if known)
CQFramework 2.7.0
Any Additional Info

Thanks for the issue. It looks like cql-execution is failing to even construct a valid expression based on the ELM provided, so the error is happening before it even tries any execution. Thanks for providing the detailed inputs, I think this will require digging into the ELM and the CQL to see what’s going on. We will look at this. Let me know if you come across anything interesting in the meantime.
The e is undefined message that happens from the minified JavaScript is not super helpful unfortunately. I linked it to my local TypeScript files and got the following:

It's failing on the constructor of the Expression, which tries to recursively build the internal cql-execution model based on the expression tree defined in the ELM.

Related: https://github.com/cqframework/cql-execution/issues/296

Discussed with @brynrhodes at the May 2023 HL7 Connectathon.
Resolving this issue requires an update in both the translator and cql-execution.
For the translator, an update to the QICore modelinfo file is required and a ticket has been filed at https://github.com/cqframework/clinical_quality_language/issues/1168.
Once the translator is updated, a resolution for valueset expressions can be worked in cql-execution.  Track https://github.com/cqframework/cql-execution/issues/296.  fqm-execution will then incorporate the updated `cql-execution'.

@gregory-akins: Originated from MADiE Issue Tracker 1455

Just to follow up on this one, after a lot of discussion in the past couple weeks, it has been determined that the authoring pattern for CommunicationNotDone now explicitly discourages use of the "<direct reference code>" in <alias>.reasonCode syntax:

NOTE: The positive counterpart for this statement (Macular Edema Absence Communicated) uses a direct-reference code, and ideally the negative statement would as well. However, at this time, direct-reference codes cannot be used as the terminology target of a retrieve of a negation profile. The workaround for this issue is to create a value set with the single required code and use the value set negation pattern.

The updated syntax of that authoring pattern is supported as is in fqm-execution only if a reasonCode is directly present on the data. If the data is using the qicore-notDoneValueSet extension, this will not work in fqm-execution until https://github.com/cqframework/cql-execution/issues/296 is addressed. Our team will discuss priorities around that cql-execution issue and how best to move forward.
One key assumption that we've discussed as well is that any canonical to a ValueSet referenced using the notDoneValueSet extension must be resolved prior to doing execution. It should be a ValueSet that is already included in the CQL for the measure logic. So fqm-execution will expect any ValueSet referenced in the canonical of that extension to have already been provided to the engine.
I might suggest closing this issue, as the specific syntax that caused it is no longer of concern, but I will wait for input from others.

Follow https://github.com/cqframework/cql-execution/issues/296 for more progress on this


Zulip email notifications: provide some way to filter by instance

I am now member of two Zulip instances both hosted on zulipchat.com, and I'd like the email notifications for those instances to end up in two different folders in my inbox (to match up the already existing extensive auto-sorting I need to keep up with my email volume).
However, currently there does not seem to be a way to do that: the From email address is the same for both instances, the subject contains the stream name but not the instance name (and adjusting my filters for each stream in both instances is not realistic), and I found no other header field either that would help here. To give a point of comparison, Discourse solves this by using different email addresses per instance (e.g. rust_lang@discoursemail.com). Having something structured at the beginning of the subject (like the instance name in brackets) would also do it (that's how I filter GitHub emails into different folders).

There's a "Notification settings" option to include the instance name in the subject line.
I think we should add an X-Zulip-Organization HTTP header designed for filtering that we can document somewhere easy to find.    (Name suggestions welcome)

@mateuszmandera FYI

(We may eventually want to do unique email addresses for spam-report scoping reasons, but probably we should provide a header in any case)

There's a "Notification settings" option to include the instance name in the subject line.

Thanks for pointing that out! I should have checked the settings.
But it is also not entirely clear what the "organization name" is.

Hmm, yeah, we could show the example for clarity.


fix: Removing the invalid resource path enables pnpm dev to succeed.

PR
PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our Commit Message Guidelines
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[x] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?
fix: #106
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information
packages/modules.jsonuses a non-existent path that causes a compilation error.

同步 #108  代码