Support common keyword setup / teardown for a file

Basically what the user guide says isn't supported at the moment. Something like Keyword Setup/Keyword Teardown settings for resource files to set setup/teardown steps for all keywords, which you could then override with individual Setup/Teardown settings in keywords, much like Test Setup and Setup for tests.
My use case is simplifying some bits of session management for RequestsLibrary keywords. I realize this is just syntax sugar and very much a "nice to have", rather than a need or a requirement.

@Brownies maybe https://github.com/sebastianciupinski/robotframework-listenerlibrary is what you are looking for

@Brownies maybe https://github.com/sebastianciupinski/robotframework-listenerlibrary is what you are looking for

If I understood its functionality correctly it's not quite what I'm looking for. I could use Register Start Keyword Listener for every keyword but in that case I might as well use Setup for those keywords and that's what I want to avoid.

The reason this functionality hasn't been implemented is that we haven't considered it too common for all or even most of the keywords in a same suite or resource file to have a same setup or teardown. Do you @Brownies have a concrete example about such usage? Implementing this shouldn't be too complicated and the design with Keyword Setup and Keyword Teardown settings would also be straightforward.

This functionality would become more usable if we'd allow changing the default setup/teardown within a file like this:
*** Settings ***
Keyword Setup        Setup 1    # The initial default setup

*** Keywords ***
Keyword 1
    xxx

Keyword 2
    yyy

*** Settings ***
Keyword Setup        Setup 2    # New setup used with the subsequent keywords

*** Keywords ***
Keyword 3
    zzz

Keyword 4
    ååå

The above doesn't currently because default specified in the Settings section apply for the whole file. That's even true if the Settings section is positioned after the test or keyword section it affects. There have now and then been requests to allow changing test related settings (mainly the template), and although they have been rejected earlier I start to like that idea nowadays. The main benefit I see is that it would simplify implementing new parsers for Robot data (e.g. in C or Rust for performance), because a setting like Test Template at the end of the file wouldn't anymore affect how earlier data should have been parsed.
Although the above is somewhat related to this issue, it absolutely would need a separate issue. Changing the behavior would also be somewhat badly backwards incompatible, so it would need a deprecation period. I may submit an issue about that at some point, but others can submit one already now if you find it important.

Somewhat simplified example is that I have a resource file with a bunch of keywords for a REST API. I want the keywords to be independent in the sense that if the default credentials are good for what I'm doing, then all I need to do in a test file is to import the resource file and call the keyword. I want to use a single RequestsLibrary session per credential set if possible because I don't want to get hundreds of access tokens per test run. So I end up calling Get Token And Create Session as the first step of every keyword in this file.
Note, we haven't actually upgraded to RF 7 yet so there's no [Setup] sections in the keywords.

Do you ... have a concrete example about such usage?

I have.
I have templated testcases - something like this;
*** Settings ***
Resource        ${CURDIR}/../resources/landingPage_test_templates.resource
Suite Setup     Open And Setup New Browser   ${DEFAULT_BROWSER}   ${IS_HEADLESS}

*** Test Cases ***
Language Selection
  [Template]   Language Selection Template
  fi   en
  sv   fi
  en   sv

So, each test case is actually defined as Keyword and some of the arguments are going to a test setup, i can't use [Setup] and [Teardown] on this file where test cases are defined (?) as variable is not yet defined at this point (as its end up as argument for keyword) so the workaround is to use setup and teardown for the keyword like this;
*** Keywords ***
Language Selection Template
  [Documentation]   Verifies language change from/to
  ...
  ...   Args:
  ...   ${lang}:  langugage to change to
  ...   ${locale}:  browser locale.
  [Arguments]   ${lang}   ${locale}
  [Setup]   Navigate To   ${LANDINGPAGE_BASEURL}   fi   ${DEFAULT_BROWSER}   ${IS_HEADLESS}   ${locale}
  # my testcse here ..
  [Teardown]   Tear Down Test Case

Since i have bunch of test templates and only test templates within this file, it would be quite useful for having something like Keyword Setup and Keyword Teardown  in settings section..


Training scripts

Hi, do you plan to open source the scripts used to train flyvec?

Thank you for your question! FlyVec was trained using code developed specifically for the PowerPC architecture that makes it hard to distribute (using similar training code to paper https://arxiv.org/abs/1908.08993). While I would like to re-implement this in PyTorch, I fear I will not have time in the upcoming weeks.
In the meantime, @DimaKrotov has created a demo notebook for example training in numpy that is available here, and there is a new general-purpose Hebbian Learning wrapper around PyTorch Ignite available here (paper).

Past me underestimated the free time current me would have. I built a small wrapper around the training code for v0.3 (317bdac
), though you are on your own to build your own tokenizations. Please let me know if the instructions are unclear


Clarify user documentation (fixes #220)

This fixes #220

Actually, this documentation was removed by the change to move user to the hook context. I'm happy to apply this requested change on top of that PR

The way we would handle this would be to merge this PR in, and then resolve the conflicts with your PR by applying this change in there.

@isaacvetter - Can you suggest something concrete?
Our prior documentation referenced the Patient and RelatedPerson resources -- both of which are not providers. The name of this field is user (or soon to be userId) which further reinforces the expanded intention of this field.
Should this type of clarification be best specified in each hook? For instance, medication-prescribe should probably always be a Practitioner id rather than something more broad, right?

@kpshek @isaacvetter , I agree with Kevin's suggestion that this should be specified more concretely in the hook, I'm happy to add that to the changes there.


python310Packages.unstructured: 0.11.8 -> 0.12.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.unstructured is: Open source libraries and APIs to build custom preprocessing pipelines for labeling, training, or production machine learning pipelines
meta.homepage for python310Packages.unstructured is: https://github.com/Unstructured-IO/unstructured
meta.changelog for python310Packages.unstructured is: https://github.com/Unstructured-IO/unstructured/blob/0.12.0/CHANGELOG.md
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.12.0 with grep in /nix/store/gj2j1d5791jsy41i29d49kg7zyp3prc9-python3.10-unstructured-0.12.0
found 0.12.0 in filename of file in /nix/store/gj2j1d5791jsy41i29d49kg7zyp3prc9-python3.10-unstructured-0.12.0

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
python310Packages.unstructured
python311Packages.unstructured

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/ff954ad2a5d88fa03a19f8568ce2866f565b3796#python310Packages.unstructured

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/gj2j1d5791jsy41i29d49kg7zyp3prc9-python3.10-unstructured-0.12.0
ls -la /nix/store/gj2j1d5791jsy41i29d49kg7zyp3prc9-python3.10-unstructured-0.12.0/bin

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

2 packages built:

python310Packages.unstructured
python311Packages.unstructured

Maintainer pings
cc @happysalada for testing.

Taken care of in a separate PR.


Added Google Search Clone

Solves issue #800
@iamrahulmahato I have added Google Search, Google Image Search and Google advanced search clone.
Please do check it out.
Also please add the respective labels.

@iamrahulmahato can I please get level 2 for this PR? as I have coded more than 200 lines of code ?


ANR when adding email configuration

Checklist

[X] I have used the search function to see if someone else has already submitted the same bug report.
[X] I will describe the problem with as much detail as possible.

App version
6.900
Where did you get the app from?
F-Droid
Android version
14
Device model
Redmi K40S
Steps to reproduce

Open K-9 Mail
Add email configuration
Click “Next” or “Sign in”

Expected behavior
APP continue running
Actual behavior
ANR
Logs
15_05-23-41-18_676.zip
crash-com-fsck-k9-15_05-23-40-50_459.zip

6.901 still ANR

6.901 still ANR
16_05-07-36-02_394.zip
crash-com-fsck-k9-16_05-07-34-17_429.zip

ANR when:

First open K-9 Mail
Click “Start”
Click “Back”
Repeat 2 and 3 several times

This seems to be related to custom tabs that are used for the OAuth sign-in flow. It looks like communication with the custom tabs service takes longer than 5 seconds.
The OAuth library seems to be using the recommended way of doing this.
This issue does occasionally show up in ANRs reported via Google Play. But it's not very common (9 affected users in the last 60 days). Given that, we can't really justify spending more time on this.
Of course, pull requests to fix the problem are welcome (although this probably needs to be fixed in AppAuth-Android).

Thank you for the response.
I am not doing anything to deliberately slow down such communication.
It is strange that it started to have issues after I updated to HyperOS based on Android 14, but there were no problems on MIUI based on Android 13 before the update.
I will try again after the next system update.

Found a temporary solution:
Set the battery saver settings for browsers that use the custom tab feature to “No restricted”.
When you have installed a browser that supports the custom tab feature, regardless of whether this browser is set as the default browser, K-9 Mail will use the custom tab feature of this browser when adding an email account. However, due to unknown system reasons, setting the battery saver of this type of browsers to “Battery saver” can cause K-9 Mail to ANR .

Found a temporary solution:
Set the battery saver settings for browsers that use the custom tab feature to “No restriction”.
When you have installed a browser that supports the custom tab feature, whether or not this browser is set as the default browser, K-9 Mail will use the custom tab feature of this browser when adding an email account. However, due to unknown system reasons, setting the battery saver of this type of browsers to “Battery saver” can cause K-9 Mail to ANR.

Found a temporary solution:
Set the battery saver setting for the browser that K-9 Mail use the custom tab feature to “No restrictions”.
When you have installed a browser that supports the custom tab feature, whether or not this browser is set as the default browser, K-9 Mail will use the custom tab feature of this browser when adding an email account. However, due to unknown system reasons, setting the battery saver setting of this type of browser to “Battery saver” can cause K-9 Mail to ANR.


Add opportunity to change db properties depending on client db server

SMSC add to JIRA

SMSC-260 bind


Also exclude the test crate from packaging.

r? @Manishearth

@bors-servo r+

:pushpin: Commit 8b0246a has been approved by Manishearth

:hourglass: Testing commit 8b0246a with merge 2fdb02f...

:sunny: Test successful - status-travis
Approved by: Manishearth
Pushing 2fdb02fddd26a4445085faa2e5f09ff4c814b68d to master...


Docs on use of matrix.node_version don't match actual output

In the docs it recommends using the following format to display a node version in the name:
strategy:
    matrix:
      node: [ '12', '14' ]
    name: Node ${{ matrix.node }} sample

or
  strategy:
      matrix:
        os:
          - ubuntu-latest
          - macos-latest
          - windows-latest
        node_version:
          - 10
          - 12
          - 14
        architecture:
          - x64
        # an extra windows-x86 run:
        include:
          - os: windows-2016
            node_version: 12
            architecture: x86
    name: Node ${{ matrix.node_version }} - ${{ matrix.architecture }} on ${{ matrix.os }}

When I do this with v2.1.5, my names don't include node version info:

Possible I'm doing it wrong, but following the README doesn't seem to do what it claims.

Names are generated service side (not in this action) but I'll ping someone to see why


When relaunch vscode after aborted vscode update then Uncaught Exception thrown

VSCode Version: <IP_ADDRESS>
OS Version: Microsoft Windows [Version 10.0.14393]

Steps to Reproduce:

Update vscode
While updating launch a second vscode instance
Abort the Installation on Dialog
Start vscode again

Reproduces without extensions: Yes

Please reinstall VS Code. Try not to launch a second instance while it updates. We have it on our backlog to improve this experience.


[autosubmit] Autosubmit should check for the presence of Google Testing.

Request
The autosubmit label should check for the presence of the Google Testing check before allowing pull requests to be submitted on Master / Main.
Context
There are numerous instances where a user submits a pull request by adding the autosubmit label to their PR.
Example
Due to the nature of Google Testing running on a cron job, there are times where autosubmit sees that all checks are green and code review is satisfied, then submits the PR.  Occasionally, this happens in a timeframe where the PR has not had the google testing check attached, which can result in failures being identified in post submit.
Impact
Contributors may be confused as to why a PR that passed all presubmit checks has to suddenly be reverted, or spend time searching for a mistake they made.

CC: @godofredoc @ricardoamador  @christopherfujino

This is a duplicate of https://github.com/flutter/flutter/issues/122543

Thanks @ricardoamador closing this and commenting on https://github.com/flutter/flutter/issues/122543.


Is this actually a plugin?

Isn't it a library? Plugins are normally runtime loaded extensions to pre-existing software, whereas this looks like a development library that is compatible with and depends on vibe.d
Also, the description field in dub.json is still the default, which matters because it's used on code.dlang.org

Uf... sorry for missing your issue. I updated the readme and dub.json file.


Rename insert_sample_with_metadata setting

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en
Category (leave one):

Improvement

Missing full description for changelog. It is backward incompatible change.


New incoming channel - high fees

Something I noticed on my node. Someone opened a 5 M channel to my node and made a 261k sat payment. CLBOSS set the our channel fees to 187313 sat (base fee) and and 3.278% (fee rate). Super high fees resulting from the rebalancing logic which makes sense for the general case but maybe not in this particular case.
What if this was a new user who connected to our routing node, paid for something and now expects to be able to receive some sats back and this is the only open channel ? The sender would pay a hefty fee. Perhaps adding upper bounds on base and fee rates would be a good idea (eg. 50k sats base and 2% fee rate ?)
I don't know if this is an actual issue and if it needs "fixing", just raising something to think about. Feel free to close this if this isn't a problem.
Below is a screenshot from the tool I'm building for C-Lightning showing this channel setup:

Sorry, I just realized I got confused by the comma in 187,313 - it's not a thousand separator as I thought initially . It's 187 sats so nothing earth shattering.


Docker-Content-Digest in GET does not match docker push

Output of docker version:
Client:
 Version:      1.10.3
 API version:  1.22
 Go version:   go1.6
 Git commit:   20f81dd
 Built:        Sat Mar 12 04:07:44 UTC 2016
 OS/Arch:      darwin/amd64

Server:
 Version:      1.10.1
 API version:  1.22
 Go version:   go1.5.3
 Git commit:   9e83765
 Built:        Thu Feb 11 20:39:58 2016
 OS/Arch:      linux/amd64

Output of docker info:
Containers: 6
 Running: 1
 Paused: 0
 Stopped: 5
Images: 6
Server Version: 1.10.1
Storage Driver: aufs
 Root Dir: /mnt/sda1/var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 50
 Dirperm1 Supported: true
Execution Driver: native-0.2
Logging Driver: json-file
Plugins:
 Volume: local
 Network: bridge null host
Kernel Version: 4.1.17-boot2docker
Operating System: Boot2Docker 1.10.1 (TCL 6.4.1); master : b03e158 - Thu Feb 11 22:34:01 UTC 2016
OSType: linux
Architecture: x86_64
CPUs: 1
Total Memory: 996.2 MiB
Name: dev
ID: OFRG:6OKA:ND64:QAPX:UPVB:QIZZ:OVNM:34VN:R33J:SVTU:XIW3:S6QI
Debug mode (server): true
 File Descriptors: 20
 Goroutines: 35
 System Time: 2016-03-25T01:12:13.364344828Z
 EventsListeners: 0
 Init SHA1:
 Init Path: /usr/local/bin/docker
 Docker Root Dir: /mnt/sda1/var/lib/docker
Username: arastra
Registry: https://index.docker.io/v1/
Labels:
 provider=virtualbox

Additional environment details (AWS, VirtualBox, physical, etc.):
VirtualBox 5.0.16 r105871 on Mac OS 10.11.4
Steps to reproduce the issue:

docker push <IP_ADDRESS>:5000/foo/bar:12345
curl -D - http://<IP_ADDRESS>:5000/v2/foo/bar/manifests/12345 | grep Docker-Content-Digest
curl -X DELETE http://<IP_ADDRESS>:5000/v2/foo/bar/manifests/<digest from step 2>

Describe the results you received:
{"errors":[{"code":"MANIFEST_UNKNOWN","message":"manifest unknown"}]}
Describe the results you expected:
Deletion of the image
Additional information you deem important (e.g. issue happens only occasionally):
The same sequence works by specifying the digest from the output docker push, which differs from the output of the GET request at step 2. I couldn't find any other digest that resembles the latter through the API.

Can you open this issue in the https://github.com/docker/distribution/issues issue tracker? This looks more an issue/question for that repository, thanks!


Unable to load template

I've got an error:
throw new Error('Unable to load template: ' + e.message || e);
^
Unable to load template: Cannot find module 'templates/confluence-jsdoc-template/publish'
...

HI hade the same problem and I've used the same solution.
now, on publish.js my 8th line is:
wikiHelper = require('./templateHelper'),

Ok, I trust you. I'll commit it.


Document workaround for --docker-run & firewalld

Docs for  #464
Assuming I don't need to update the CHANGELOG for documentation changes?

Thanks for the PR. Just had one small nit before merging. Thanks!


Differential Privacy

Closes #303

looking good. 2 things
* you're doing it for the weights, not the \delta w (gradients). while for adding noise that can be fine, the clipping unfortunately is not ok on models. that would need to be on gradients. we need to briefly discuss maybe if in FL we want to send gradients or models. without DP, the result in both cases is identical anyway. we might later also want to support byzantine robustness, which also likes to have clipping on gradients, not models. for the moment what you could do is keep sending models, but before you do, the 2 new ops you now added should happen only on the local gradients, not the models

* add a field to the task config where people can switch it on/off andspecify the noise level

I fully agree. At first glance, there doesn't seem to be a way to access the current epoch's gradient, which is why @Nacho114 and I decided to go with the weights. Would taking the difference W_curr - W_prev be correct? This doesn't take into account the current learning rate.
I'll add the task config field for the noise scale 👍

exactly. the diff curr-prev (over the round) is perfectly fine. then clip etc. if we do it like this, the nice thing is that it will work easily for both federated and decentralized

actually you can keep sending weights (model) instead of diffs.
it's just that locally before sending, the client needs to compute their own weight diff since beginning of the round, and then apply noise and clipping. the result can still be the weights (model)
maybe easier this way because then it will be the same in decentralized

actually you can keep sending weights (model) instead of diffs.
it's just that locally before sending, the client needs to compute their own weight diff since beginning of the round, and then apply noise and clipping. the result can still be the weights (model)
maybe easier this way because then it will be the same in decentralized

Oh right, I see

closing this, will open create another branch since @tharvik's latest PR moved many files

exactly. the diff curr-prev (over the round) is perfectly fine
On Fri, May 13, 2022, 11:23 Hugo @.***> wrote:

looking good. 2 things

you're doing it for the weights, not the \delta w (gradients). while for adding noise that can be fine, the clipping unfortunately is not ok on models. that would need to be on gradients. we need to briefly discuss maybe if in FL we want to send gradients or models. without DP, the result in both cases is identical anyway. we might later also want to support byzantine robustness, which also likes to have clipping on gradients, not models. for the moment what you could do is keep sending models, but before you do, the 2 new ops you now added should happen only on the local gradients, not the models

add a field to the task config where people can switch it on/off andspecify the noise level

I fully agree. At first glance, there doesn't seem to be a way to access
the current epoch's gradient, which is why @Nacho114
https://github.com/Nacho114 and I decided to go with the weights. Would
taking the difference W_curr - W_prev be correct? This doesn't take into
account the current learning rate.
I'll add the task config field for the noise scale 👍
—
Reply to this email directly, view it on GitHub
https://github.com/epfml/disco/pull/317#issuecomment-1125838813, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDIMRYGDYPX7EO5YBF7UQDVJYNP7ANCNFSM5V2WY6KQ
.
You are receiving this because you commented.Message ID:
@.***>


Set up application updates for production

2 years old - lets reassess  based upon feedback from @HernanRivasAcosta and users


Check overdue tickets

Create a job that will be run on regular basis to check if there are not unpaid tickets. The default threshold may be 7 days but it should be configurable. Optionally - add possibility to define action - send email, silently cancel order etc.

This functionality has been added in PR #258 how ever there is room for improovement such as issue #259


Add Dockerfile for Backend App to Build and Run in Python

Add a Dockerfile to containerize the backend application, allowing it to be built and run in a Python environment.
•	Use an Alpine-based image for a lightweight container.
•	No need to include or pass any config file, as it is planned to be removed before first release.
•	The application should expose port 8080.

Hello, I would like to have a try with this issue, could you please assign it to me?

Hey @leinadsened  ,
Assigning this issue to you, happy coding
If any assistance required meet our team here : https://join.slack.com/t/theailounge/shared_invite/zt-2ogkrruyf-FPOHuPr5hdqXl34bDWjHjw

HI @leinadsened  any updated on this ?

Hi @agberoz @ashmilhussain
Created a PR #68

@leinadsened, Please go through the review comment

Thank you for the detailed review @agberoz
I had a feeling my understanding of the task was a bit simple 😅 I'm sure I will have some questions, will start working on it later today.

@leinadsened,  I've mentioned two changes.
Once they're done, the PR will be ready to merge.

@agberoz, I'm finished 😄
Thank you for the guidance, it was a great experience!


Tighten preflight check to err on existing namespace

The typical flow for Sonobuoy involves creating a new namespace
and it is uncommon to need to put it into an existing namespace.
When that occurs, cleanup must be manual since Sonobuoy by default
just deletes the namespace.
This tightens the precheck to error on the existing namespace. In
situations where that is still desired, preflight checks can be
skipped.
Refactored the logic some so that tests could be added for preflight
checks.
Fixes #677
Signed-off-by: John Schnake<EMAIL_ADDRESS>Release note:
Preflight checks will not fail if the namespace Sonobuoy is targeting already exists. This helps avoid a situation where a user must manually cleanup a Sonobuoy run in order to avoid destroying the namespace.

Codecov Report

Merging #678 into master will increase coverage by 0.91%.
The diff coverage is 54.83%.

@@            Coverage Diff             @@
##           master     #678      +/-   ##
==========================================
+ Coverage   30.53%   31.45%   +0.91%
==========================================
  Files          57       57
  Lines        3134     3151      +17
==========================================
+ Hits          957      991      +34
+ Misses       2086     2067      -19
- Partials       91       93       +2

Impacted Files
Coverage Δ

pkg/client/preflight.go
53.12% <54.83%> (+53.12%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 922fd03...d02b013. Read the comment docs.

The typical flow for Sonobuoy involves creating a new namespace
and it is uncommon to need to put it into an existing namespace.

Any reason to not support this? Why must the user use heptio-sonobuoy as the namespace? Or run multiple versions on sonobuoy in the same cluster? (i.e. have one to do e2e tests, another for customer plugin, etc)?

Any reason to not support this?

It is still supported, but you just have to provide the --skip-preflight flag, implying that you know there may be consequences. The problem currently is that some users have put Sonobuoy in an existing namespace without realizing that removing the Sonobuoy resources must be a manual process in that case. If they dont realize that and still try sonobuoy delete -n my-existing-namespace then we could end up causing damage to their cluster. I'm not against a more careful deletion if lots of people are requesting it later, but as of now Sonobuoy gains nothing from being in an existing namespace, it isn't the norm, and it doesn't work well with the deletion model that has been in place for a long time.

Why must the user use heptio-sonobuoy as the namespace?

They don't have to use heptio-sonobuoy as the namespace, there is a --namespace flag which allows them to define which to use. The change to the preflight checks is only that we want the namespace to be a new, fresh namespace.

Or run multiple versions on sonobuoy in the same cluster? (i.e. have one to do e2e tests, another for customer plugin, etc)?

Using the namespace flag you can run multiple copies of sonobuoy in your cluster; I actually do this all the time. In particular when a local k8s namespace is failing to delete I just use a different namespace rather than clean it up for my own velocity. In this situation (running multiple Sonobuoy copies) the current preflight checks already fail if the Sonobuoy pod exists in the chosen namespace so you already can't run multiple Sonobuoy copies in the same namespace. In addition, even if you skip preflight checks, the resource creation will fail because of the name conflict (pods/configmaps with same names in same namespaces)


Remove unnecessary using & rename DiagnosticsConfig to JsonConfig

Prerequisites

[x] I have written a descriptive pull-request title
[x] I have verified that there are no overlapping pull-requests open
[x] I have verified that I am following the Nancy code style guidelines
[x] I have provided test coverage for my change (where applicable)

Description
Removed erroneous/no longer needed using statement for Nancy.Diagnostics. In the same file (JsonConfigurationExtensions.cs), Line 9 was renamed (in comment) from DiagnosticsConfiguration to JsonConfiguration in order to match the filenames and for consistency.

Umm. Well this is awkward. I just merged an equal PR. What happened here?


[(LIVE.FREE)] Ottawa Senators vs Boston Bruins LIVE Streams Free NHL On Tv Channel 9 NOV 2024

The Ottawa Senators face the Boston Bruins on Saturday, Nov. 9, 2024 (11/9/24) in a regular season game at TD Garden in Boston, Mass.
➤🔥CLICK HERE TO WATCH LIVE NOW🔴►►
➤🔥CLICK HERE TO WATCH LIVE NOW🔴►►
➤🔥CLICK HERE TO WATCH LIVE NOW🔴►►
How to watch: Fans can watch the game for free via a trial of DirecTV Stream or fuboTV. You can also watch via a subscription to Sling TV, which is offering half off your first month.
'Surreal and super special' - Dabrowska and Routliffe celebrate WTA Finals doubles victory'Surreal and super special' - Dabrowska and Routliffe celebrate WTA Finals doubles victory
Here’s what you need to know:
What: NHL regular season
Who: Senators vs. Bruins
When: Nov. 9, 2024 (11/9/24)
Time: 7 p.m. ET
Where: TD Garden
TV: NHL Network, NESN (regional)
Live stream: DirecTV Stream (free trial), fuboTV (with subscription)

Here’s a hockey story via the Associated Press:
DENVER (AP) — At first, the protective undershirt resembling a mock turtleneck was hot and uncomfortable. It even caused Jayson Megna’s visor to fog up.
But the journeyman hockey player stuck with wearing the long-sleeved base layer designed to protect his neck from a razor-sharp skate blade.
Since the death of Adam Johnson in October 2023 during a game in England, protective equipment has been a prime topic at every level of the sport. The American Hockey League, the top developmental league below the NHL, took a big step this season by requiring all skaters, referees and linespersons to wear some sort of cut-resistant neck protection.
Like the requirement of eye-protecting visors a decade ago, the neckwear could take some getting used to. It is becoming more common for players of all ages — Hockey Canada has mandated neck guard protection for youth players for three decades and USA Hockey made it mandatory for U.S. players under 18 as of Aug. 1 — and now all eyes are on the pros.
“Hopefully that’s what’s going to end up happening as we move forward here, that it’s just going to be a piece of their equipment,” said AHL president and CEO Scott Howson, whose league has long been a testing ground for the NHL, including 3-on-3 overtime and the mandate for cut-resistant socks and sleeves last season. “With the different products out there, hopefully all the players can find something that they can adapt to and eventually like — or, at the very least, not notice when they’re playing hockey.”
The death of Johnson was a horrific reminder of the risks. The former NHL player was cut in the neck during a game and died at a hospital. Guidelines exist for many youth programs in North America and beyond, but not at the highest levels of hockey, where league and players' union leaders have discussed the topic for years.
Neck guards are getting a trial run in the AHL, which starts Friday, with unanimous backing from NHL executives on the competition committee.
“Last year some teams just decided, ‘Well, I can‘t do it in the NHL, but I can do it in the American League,’” Howson explained. “We’ll see where it goes.”
Why not wear neck protection?
The arguments against protective neckwear range from it being too hot to too restrictive to players simply being creatures of habit.
“Obviously, there’s things that people should do, but I think we’re stuck in our ways, also,” said New Jersey Devils forward Jack Hughes, who added he doesn’t wear nearly enough protective equipment. “Got to be comfortable.”
Longtime NHL player Pierre-Edouard Bellemare had planned on wearing a neck guard this season for the Colorado Avalanche before he was let go last week. He exalted the benefits of visors by pointing to the cuts in his beard that could’ve been worse if not for the eye protection.
Because on the ice, anything can happen.
“The game is faster and you have less control, sometimes to your body and the cutting all over the place is not something you want to see,” Bellemare said.
Bellemare simply wanted to pave the way for the next generation, like other players did before him with visors.
“It doesn’t cost me much” to wear protective neck gear, Bellemare said. ”Come on, it’s a neck guard. That’s not going to change my season. I’m not going score 70 because I have a neck guard or not.”
Veteran defenseman Dylan McIlrath, who captained Hershey to back-to-back Calder Cup championships in the AHL, did not experiment with any neck protection during training camp in Washington and concedes players will endure some growing pains with the equipment. But the 32-year-old is not protesting too much.
“To be honest, I think it’s probably the right call,” McIlrath said. “Obviously it’s a younger league and the next generation coming up ... guys can get used to it going into the NHL after playing in the American League a few years, they don’t really notice the changes.”
Testing things out
Megna, who was recently assigned to the AHL’s Colorado Eagles, has tried out different types of neck protection and brands. He’s been wearing one by T.J. Oshie’s Warroad Hockey Apparel that boasts “an adjustable neck collar built from premium grade cut resistant fabric and high-impact foam.” His undershirt has cut-proof sleeves.
Megna also wears cut-proof socks, which prevented a possible calf or Achilles injury last season when a player stepped on him with a skate blade.
“I could feel the skate there — felt like a burning sensation, and the protective sock saved me,” Megna said. “I didn’t even have a scratch. ... Any time you can protect your players and increase safety, it’s a good thing.”
Megna said it didn’t take long to adjust to the new gear — maybe a session or two.
“After that, I haven’t even thought about it,” said Megna, who’s spent time with a half-dozen NHL organizations. “It’s just part of what I wear now.”
NHL resistance
The NHL currently does not have any mandate that players must wear cut-resistant gear. That would require an agreement between the league and union.
St. Louis forward Robert Thomas understands the hesitation.
 “I think they’ve done a good job of having all that available to us. Some guys wear it, some guys don’t. I think giving guys the choice is ultimately the right decision.”

ccc bcbcb


Drop shadow effect for the top menu bar

The menu bar in the API-M 4 docs currently looks as follows.

I think we are missing an effect. For example, see the API-M 3.2.0 docs show below. The drop shadow effect is not there in the API-M 4 doc space.

Closing this as @erangatl confirmed that this is by design.


Disable autocomplete when removing recovery method

@Patrick1904 I think we should also do it for other similar cases where we want user to think twice. Also at least Safari was getting confused and proposing to autocomplete e-mail in this input.

@Patrick1904 I think we should also do it for other similar cases where we want user to think twice. Also at least Safari was getting confused and proposing to autocomplete e-mail in this input.

@vgrichina 100%. I'll do an audit and fix other similar cases.


Add links to other projects in footer

Add a simple footer with links to my Twitter, and other projects

Done in latest version. This is hardcoded in the theme, so everyone using the theme will have them. No big deal for now as I'm the only one using the theme (AFAIK)


Switch statement

Hello,
I have code:
switch ($role) {
	case Role::SUPER_ADMIN:
	case Role::ADMIN:
		$url = '/admin-panel';
		break;
	case Role::DIRECTOR:
		$url = '/director-panel';
		break;
	case Role::ACCOUNTANT:
		$url = '/billing-panel';
		break;
	default:
		$url = '/guest-panel';
}

but after using this package I have removed indentation.
switch ($role) {
case Role::SUPER_ADMIN:
case Role::ADMIN:
	$url = '/admin-panel';
	break;
case Role::DIRECTOR:
	$url = '/director-panel';
	break;
case Role::ACCOUNTANT:
	$url = '/billing-panel';
	break;
default:
	$url = '/guest-panel';
}

What can I do to prevent it from removing indents?

enum { case; } also too.

Hi @Tomirad This is quite strange. May you please share your config (what rules are you using)?

My conf for phpftm is it:
{    "php_bin": "W:\\WebServer\\php\\php8.3\\php.exe" }
That is, the basic thing.
I'm using this in Sublime Text 4169 - no license, Windows 11
Other package is:
ApacheConf, Folder Aliaser, FTPSync, Function Name Display, Git, GitStatus, Goto-CSS-Declaration, GoToClass, hex to Int Preview, HexViewer, HTMLBeautify, JavaScript Beautify, Json Colors And Navigation, MikrotikScript, Minifi, nginx, Paradox, PHP Codebeautyfier, phpfmt, PHPIntel, Pretty JSON, Print to HTML, SFTP, SideBarEnhancements, SqlBeautyfier, Squirrel, Status Bar JsonPath, StatusMessage, Theme -  Monokai Pro, WakaTime, xml2json.
But, on other PC with fewer add-ons this also occurs. Practically only SFTP, Wakatime and phpfmt and PHPIntel.
When I disabled phpfmt, this problem did not occur.

Hi. Could you please activate try with this config and see if it still happens? Thank you
{
    "autocomplete": false,
    "autoimport": false,
    "excludes": [
        "SpaceBetweenMethods"
    ],
    "format_on_save": true,
    "passes":
    [
        "AlignDoubleArrow",
        "AlignPHPCode",
        "SpaceAfterExclamationMark",
        "AlignConstVisibilityEquals",
        "AutoSemicolon",
        "ConvertOpenTagWithEcho",
        "AlignEquals",
        "MergeNamespaceWithOpenTag",
        "RemoveSemicolonAfterCurly",
        "RestoreComments",
        "ShortArray",
        "ExtraCommaInArray",
        "AlignDoubleSlashComments"
    ],
    "php_bin": "YOUR_PHP_PATH",
    "psr1": false,
    "psr1_naming": true,
    "psr2": true,
    "readini": true,
    "smart_linebreak_after_curly": false,
    "version": 4,
    "wp": false,
}

Please check for the "php_bin" path

I activated it. Now the indentation does not go back.
I found that "psr2": false caused the lines to lose indentation.
Thanks!


Asset Server assets not found after upgrade to 0.3

Bevy version
0.3 from cargo
Operating system & version
Windows 10
What you did
Upgraded from bevy 0.1 to 0.3
Tried to load an asset with assert_server.load("folder/asset.ext")
What you expected to happen
Asset loaded.
What actually happened
Asset loader fails with file not found
Additional information
This is in a workspace application. Where assets are at the root,
I have also moved them into being next to the exectuable project (in case it is ran from there).
Copying the assets to be near the compiled .exe file in the target folder.
Running it directly instead of cargo run.
Also changing the path to be windows style.
While this is a custom asset, it appears to me its failing to try to load even find the files. I'm not sure why, i suspect maybe the behaviour of current working directory + relative paths is the issue here.
thread 'thread 'IO Task Pool (1)IO Task Pool (0)' panicked at '' panicked at 'called `Result::unwrap()` on an `Err` value: PathLoaderError(NotFound("assets/ship.aseprite"))called `Result::unwrap()` on an `Err` value: PathLoaderError(NotFound("assets/background.aseprite"))', ', C:\Users\Beau\.cargo\registry\src\github.com-1ecc6299db9ec823\bevy_asset-0.3.0\src\asset_server.rsC:\Users\Beau\.cargo\registry\src\github.com-1ecc6299db9ec823\bevy_asset-0.3.0\src\asset_server.rs::295295::6060

note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
thread 'main' panicked at 'task thread panicked while executing: Any', C:\Users\Beau\.cargo\registry\src\github.com-1ecc6299db9ec823\bevy_tasks-0.3.0\src\task_pool.rs:75:18

I ran into this as well and looked at the updated examples. Now it assumes you are using a folder named "assets"
So if you used to do let font: Handle<Font> = asset_server.load("assets/fonts/FiraMono-Medium.ttf");
you need to change it to let font: Handle<Font> = asset_server.load("fonts/FiraMono-Medium.ttf");

Ah this makes me have to rethink my assetloader. I was shelling out to convert unknown asset to png, in a tempdir from the OS. This works in 0.1x but if all assets now must be in the asset folder I must rethink the design.
I suspect it may just be easier to add this as an xtask for compiling my assets, and wait until atelier is integrated instead.

Also for anyone who is a bit perplexed as to where the assets folder should be. Here's what I have (in a workspace project)
assets at workspace root = doesn't work
assets at target/debug/assets = works only if running compiled game from target/debug
assets at workspaceroot/mainentry/assets = works.

For me workspace_root/assets worked btw. In case we mean the same thing with workspace_root (where your Cargo.toml is located).

Yup the rules are:

Assets (by default) are rooted at ROOT/assets. This folder can be configured by overriding the AssetFolderSettings resource.
When using cargo run it will automatically detect the cargo workspace root and make that ROOT
When running the binary directly, it will make ROOT the folder containing the binary

When running the binary directly, it will make ROOT the folder containing the binary

The way it's currently set up, that means that running under vscode-lldb from vscode is looking in the target directory for the assets folder, which is unfortunate.

When using cargo run it will automatically detect the cargo workspace root and make that ROOT

To be more specific, it checks $env:CARGO_MANIFEST_DIR, which is "The directory containing the manifest of your package", i.e. your package manifest directory, not the workspace root. There is currently no way to actually get the workspace root, beyond calling cargo metadata or just doing what cargo does (walk up the fs tree looking for a Cargo.toml with [workspace]).

That said, I don't think there's really a better option here, beyond explicitly noting at some point that $env:CARGO_MANIFEST_DIR is involved in the lookup path, and giving some way of overriding that behavior if people need specialized behavior. Requiring certain cargo environment variables such as $env:CARGO_MANIFEST_DIR to be set by the debugger doesn't seem like too much of a burden, so long as it's noted.

This took me a couple of reads and diving into file_asset_io.rs to understand what's going on in regards to debugging inside VS Code, so in case anyone else stumbles on this thread by googling like I did, adding "env" : { "CARGO_MANIFEST_DIR" : "${workspaceFolder}" } to my launch.json configurations set things right.
I know CAD97 basically said as much, just thought I'd make it more explicit for Rust VS Code debugging newbies like me. :slightly_smiling_face:
I wonder if there's a case to be made for noting this in debug panic messages when this assets folder isn't in the folder containing the binary.

This took me a couple of reads and diving into file_asset_io.rs to understand what's going on in regards to debugging inside VS Code, so in case anyone else stumbles on this thread by googling like I did, adding "env" : { "CARGO_MANIFEST_DIR" : "${workspaceFolder}" } to my launch.json configurations set things right.
I know CAD97 basically said as much, just thought I'd make it more explicit for Rust VS Code debugging newbies like me. :slightly_smiling_face:
I wonder if there's a case to be made for noting this in debug panic messages when this assets folder isn't in the folder containing the binary.


Spring Native

Is there a future plan to integrate the Jib GraalVM Native Image Extension with Spring Native's specific build goals? Specifically, Spring Native can be compile by the user calling $ mvn spring-boot:build-image, instead of using the native-image:native-image goal. It would be very helpful if binaries compiled directly from spring native could be used with jib!
Following the build steps as seen here.

I understand one can think about the integration, which we did too. However, once you try to materialize such an idea, you'll soon realize that doesn't make much sense.
Basically, Buildpacks and Jib cannot be compatible. spring-boot:build-image builds an image using Cloud Native Buildpacks in a black-box manner where it runs Docker to build artifacts (e.g., jars, native executable, etc.) inside containers to generate image layers to create an image. Cloud Native Buildpacks have their own way and workflow of building things inside containers. When using Cloud Native Buildpacks, to contribute your own layer to the final image to build, you need to write a buildpack. It's the only way to plug something into their workflow.
OTOH, Jib assembles an image directly out of files generated by Maven. Moreover, the mode of image building in Jib is fundamentally different: declarative vs. imperative, basically. Jib builds an image without running or requiring Docker (i.e., Jib describes the image), while Buildpacks build an image by executing a series of containers and commands inside Docker.
In any case, both spring-boot:build-image and native-image:native-image delegate the task to build a native executable to the  native-image program, so in the end, it's the same thing.


KeyNotFoundException in TreeContainerIndex.cs:line 76

Occurs sometime when development tools are open controls in logical tree are expanded and usually when closing the window that devtolls are attached.
line of the exception:
https://github.com/AvaloniaUI/Avalonia/blob/master/src/Avalonia.Controls/Generators/TreeContainerIndex.cs#L59
Exception details:
An exception of type 'System.Collections.Generic.KeyNotFoundException' occurred in mscorlib.dll but was not handled in user code
Additional information: The given key was not present in the dictionary.
at System.Collections.Generic.Dictionary2.get_Item(TKey key)    at Avalonia.Controls.Generators.TreeContainerIndex.Remove(IEnumerable1 containers) in C:\Development\SynergyBaseballGit\External\Avalonia\src\Avalonia.Controls\Generators\TreeContainerIndex.cs:line 76
at Avalonia.Controls.Generators.TreeItemContainerGenerator`1.RemoveRange(Int32 startingIndex, Int32 count) in C:\Development\SynergyBaseballGit\External\Avalonia\src\Avalonia.Controls\Generators\TreeItemContainerGenerator.cs:line 117
at Avalonia.Controls.Presenters.ItemVirtualizerNone.ItemsChanged(IEnumerable items, NotifyCollectionChangedEventArgs e) in C:\Development\SynergyBaseballGit\External\Avalonia\src\Avalonia.Controls\Presenters\ItemVirtualizerNone.cs:line 79
at Avalonia.Controls.Presenters.ItemsPresenter.ItemsChanged(NotifyCollectionChangedEventArgs e) in C:\Development\SynergyBaseballGit\External\Avalonia\src\Avalonia.Controls\Presenters\ItemsPresenter.cs:line 134
at Avalonia.Controls.Presenters.ItemsPresenterBase.ItemsCollectionChanged(Object sender, NotifyCollectionChangedEventArgs e) in C:\Development\SynergyBaseballGit\External\Avalonia\src\Avalonia.Controls\Presenters\ItemsPresenterBase.cs:line 257

I also get this quite a lot in treeview. Any clues on an exact repro?

no idea how to make stable repro, only from time to time

I get it quite often when I remove items from TreeView.

Ok guys solid repro coming up!!!

Steps to reproduce:

Clone Avalon studio at this commit: https://github.com/VitalElement/AvalonStudio/commit/ad60123a16d247a22d74e0e8e72e2e96f0bc06f4

Clean and build the entire solution.

Replace the nugget assemblies with Avalonia assemblies built on your machine.

Compile and Run.

Open STM32SampleProject from the base dir where you cloned to.

Click File -> New Solution and select C++ Console Application.

Click OK

From the solution explorer, right click on ConsoleApplication1 and select remove.

Click File -> Close Solution
Hopefully you will see this!


Gh 421

Are these out_tmp files something we should be committing? There seems to be a lot of empty space in a set of the files, wondered what the purpose of that was. Thanks 👍


Getting Started page doesn't work with latest yarn

I try to run getting started page instructions with latest version of yarn package (1.15.2) and getting error babel-node: command not found. Command works property with some previous versions of yarn (1.10.0). Also everything works property if I using npm run command. Maybe it's better to replace scripts commands with npm run?

Link to gettings started instructions:
https://facebookincubator.github.io/fbt/docs/getting_started

Looks like a bug in yarn's modules-folder.
https://github.com/yarnpkg/yarn/pull/6850/files#diff-46e3111748d572a99398769469253279R100
Should have been path.resolve
cc. @rally25rs, @arcanis

We don't need these .yarnrc files with yarn workspaces, so let's just remove them and avoid this modules-folder bug.

Should be fixed with commit:
https://github.com/facebookincubator/fbt/commit/425cc53b6be249ccc3904d03ad77226c0f88eb4f


Node 10 EOL TerraformTask2@2

##[warning]Task 'Terraform' version 2 (TerraformTaskV2@2) is dependent on a Node version (10) that is end-of-life. Contact the extension owner for an updated version of the task.

We did not update V2@2.
The V4@4 has node 20 support. However some have reported the node 10 warning still remains for V4. I did not remove node 10 but rather added node 20.


Fixed minor quoting issue in README and corrected dir name in CONTRIBUTING guide

First off all, may I say what a nice project this. I love it. Thanks for all the work you folks do ❤.
This PR addresses the minor issue that I found while following the README.
$ pip install timvt uvicorn[standard]>=0.12.0,<0.14.0
-bash: 0.14.0: No such file or directory

I think, a bit of quoting should fix it.
$ pip install timvt 'uvicorn[standard]>=0.12.0,<0.14.0'
Requirement already satisfied: timvt in ./.env/lib/python3.7/site-packages (0.1.0)
Collecting uvicorn[standard]<0.14.0,>=0.12.0
...

I also found that it is mentioned cd titiler instead of cd timvt in the CONTRIBUTING.md. I assumed that was typo and fixed it as well. I hope this is fine. Thank you :)

🤦 side effect of maintaining too many repo :D
Thanks a lot @millefalcon 🙏


generate_enum_match doesn't work with generic params

enum Option<T> {
    Some(T),
    None,
}

This will fail to consider the generic in the resulting impl block. I know how to do the fix and can probably implement it after hours today.

Related: https://github.com/rust-analyzer/rust-analyzer/issues/7601, https://github.com/rust-analyzer/rust-analyzer/issues/7605


append_event_to_stream needs content type 'application/vnd.eventstore.events+json'

http://docs.geteventstore.com/http-api/3.5.0/writing-to-a-stream/
To be able to append more than one event in one request the content type 'application/vnd.eventstore.events+json' is needed.
As I understand it version 2.* of Event Store does not support this content type and http_eventstore only use the content type application/json making the append_event_to_stream function broken.
Is the content type 'application/vnd.eventstore.events+json' something 'http_eventstore' intend to support?
I love to help but think a discussion is needed in how the support of 2.* and 3.* of Event Store should work.
BR Morgan

To be able to append more than one event in one request the content type 'application/vnd.eventstore.events+json' is needed.

Yes, I've planed to work on that soon but any help will be appreciated :) PR please ;)

Is the content type 'application/vnd.eventstore.events+json' something 'http_eventstore' intend to support?

Sure.

I love to help but think a discussion is needed in how the support of 2.* and 3.* of Event Store should work.

There is a lot of new things in Event Store between versions 2.x and 3.x. I would like to avoid supporting 2 versions here. The simplest solution would be make a branch EventStore 2.x with current code and next release or http_eventstore could support only 3.x version.

Thanks for the reply. I try to work on it.
I plan to use the http_eventstore as part of a driver to https://github.com/Sandthorn/sandthorn. :)

I started a branch https://github.com/hallgren/http_eventstore/tree/event_store_version_3. I'm not sure how to best collaborate github wish. It seems to early to create a PR of this branch.

PR might be created on early stage of work - it is easier to discuss things when you see the code - feel free to create a PR - it does not have to be merged immediately.

Great.


Do not build Docker images when testing CI, update docs and workflows

Fixes #566.

This looks good, I'm going to merge this one too. Again, if someone notice something wrong, please say it!


Add update_now in quicly_close

Hi,
I ran into an issue with the closing of quic connections (quicly_close) If I wait some time after the last packet send, the connection close frame will never be sent. Afaik, it's only the now not being updated on close.
Thanks a lot for your awesome library, it really made our life easy when adapting it for our use-case ! (We are using it to add QUIC support in VPP )

Thank you for the fix! I knew about the bug but had not figured out the cause.
And I'm glad to know that you are happily using quicly.


add import to testing example

What kind of changes does this PR include?

Minor content fixes (broken links, typos, etc.)

Description

Adds an import statement in the given test example, since this was missing

Thanks for catching this @ElianCodes! 💜


Controller attributes are accessible only after a clean

Hi all,
If I define a controller like this
class CounterCtrl extends Controller { }

and run the application with ~fastOptJS, all attributes added to controller will only be accessible from within the view after I clean the application and run it again.
For example, if I start the application with ~fastOptJS and then add an attribute "foo" to the controller
class CounterCtrl extends Controller {
  var foo = "bar"
}

I need to stop the application, issue a clean, and start it again for the "foo" attribute be accessible within the view. The same thing doesn't happen with scope controllers.
Tested against angulate 0.2 and scalajs 0.6.2. I have a small project to demonstrate the issue: https://github.com/joaopmafra/scalajs-angulate-issues
Thanks in advance.

ControllerMacros.createController() is not called during compile after a property was added to, or removed from a Controller. Maybe a cache problem -> check with Scala.js list

Great! I tested with my codebase and it works the way you said. Thanks for investigating that for me.
Honestly, I would love to contribute to the project. But despite having 10 years of experience with java, I'm new to scala. I'm trying to learn the most important parts of the language for only then go to advanced features such macros. Thanks again.

I've opened a thread on the scala-user list for this issue.

This problem seems to be hard to solve with the current approach. We could use macro annotations on controller classes to generate the Angular.js code instead, which works quite well for angulate2. But that would mean to depend on macro paradise. Maybe we should provide annotations support in add-on library (or sbt plugin)?


Improve wording to include a wider range of people

This leads to a paradox of tolerance: "everyone" taken literally includes transphobes, facists, etc. exactly the kind of people we don't want.
So here "everyone" includes transphobes, fascists, etc

the opposite of a safe space isn't a unsafe space, it's just a space. everyone not listed in that is welcome.
But here "everyone" is welcome?

From how I understand it you're saying that "everyone" only includes bad people when you use the word directly, which does not make sense to me, could you please elaborate?

From how I understand it you're saying that "everyone" only includes bad people when you use the word directly, which does not make sense to me, could you please elaborate?

it's the difference between "we allow  to join, but follow the rules" and "it's a safe space for "
a safe space specifically means that despite their often marginalised status, a group is welcome. this doesn't apply to "everyone" as "everyone" isn't a marginalised group.

From how I understand it you're saying that "everyone" only includes bad people when you use the word directly, which does not make sense to me, could you please elaborate?

it's the difference between "we allow to join, but follow the rules" and "it's a safe space for "
a safe space specifically means that despite their often marginalised status, a group is welcome, accepted, and cared for. this doesn't apply to "everyone" as "everyone" isn't a marginalised group.

A safe space is not just for marginalized groups. It's a general term that implies acceptance for differences across the board.

I think it's important that we stay on track here and focus on the goal we both share, as I feel like this conversation won't help any of us much. We both want everyone to feel welcome and comfortable on the channels this code of conduct applies to.
You said that if we explicitly include "everyone", then that would include the people you don't want this to be a safe space for, which I disagree with, but understand.
I respect your opinion and want to throw in this idea:
How about we use yours and Tris's suggestion, but add a ", but not limited to, " to it? That way we don't explicitly include the people this does not apply to, but we leave room for groups not mentioned and people that aren't marginalized too. Sounds like a good way of handling this to me - What do you think?

Ultima it's the responsibility of the moderators to create a safe space. it doesn't hust happen, it needs a certain level of knowledge about, and specific accomodations for, the group for which it is a safe space.
just as we can't include everyone, we also can't include groups which we don't know how to accommodate.
this is why I'm generally against anything which opens up the safe space to more than specific listed groups.
again, a safe space is meant to be specific to a group. the term safe space originated from LGBTQ+ where certain spaces were kept specifically free of harassment and discrimination against queer folx. the term inherently needs to talk about specific marginalised groups, which is why I'm also not quite happy with Tris recommendation.
is there a specific group you feel should be listed?

Ultimately it's the responsibility of the moderators to create a safe space. it doesn't just happen, it needs a certain level of knowledge about, and specific accomodations for, the group for which it is a safe space.
just as we can't include everyone, we also can't include groups which we don't know how to accommodate.
this is why I'm generally against anything which opens up the safe space clause to more than specific listed groups which we feel we can accommodate.
again, a safe space is meant to be specific to a group. the term safe space originated from LGBTQ+ where certain spaces were kept specifically free of harassment and discrimination against queer folx. the term inherently needs to talk about specific marginalised groups, which is why I'm also not quite happy with Tris recommendation.
is there a specific group you feel should be listed?

Okay, it appears you and me have very different definitions of a safe space, and how/when/where is should be used, and it feels like to me that your not ready to accept a compromise in that regard, which is fine.
How about we leave the safe space definition alone, and accommodate the idea I have into a separate section of the code if conduct? Would you be happy with that?

i think that would be better yeah.
I'm not sure what your goal is beyond your understanding of safe space being different from the one i know, so maybe formulating it as a separate thing could help me understand and maybe include it.

Oxford's definition of a safe space is "a place or environment in which a person or category of people can feel confident that they will not be exposed to discrimination, criticism, harassment, or any other emotional or physical harm."
This is pretty much what I'd like to include. A sentence or two which describes that no matter how "different" a person may be, they can feel welcome and will not face descrimination or harassment for it. A good example would be people that have a lisp.

how do you distinguish "different" in the sense of Lisp, physical disability, etc. from "different" in the religious extremist, conspiracy theorist,
or narcissistic way?

I think regardless of one's personal opinions and believes, no one should face that kind of stuff. And as long as they oblige to the rules and don't make any other members uncomfortable, why should they not be welcome?

I understand that with your definition of a safe space, you don't want to include "everyone" because it would also include the bad people, and thus tolerate their opinions. This does not tolerate their opinions, but tells them that as long as they behave by the rules and don't make anyone uncomfortable they can feel just as welcome and comfy as anyone else

Should that person fear discrimination, criticism, harassment, or any other emotional or physical harm?

no to all but criticism.

I think regardless of one's personal opinions and believes, no one should face that kind of stuff. And as long as they oblige to the rules and don't make any other members uncomfortable, why should they not be welcome?

this is missing the point: as i said, the opposite of a safe space is not a unsafe space, but just a space. conspiracy theorists wont recieve extra attention beyond any other user to make sure that the space is safe for them. only the listed groups will.
not being listed in the safe space has absoultely no effect on how other rules are enforced. the only thing that happens because a group isnt listed in the safe space part is that we dont promise to pay special attention to making that group comfortable.
ultimately, a safe space is something we as moderators have to put work into. and thats all this section says: we'll put work into making sure these groups feel safe beyond the enforcing of normal rules. this includes things like undestanding how subtle discrimination against the group works or trying to accomodate the group to make them more comfortable (pluralkit for example)

simply put: i will not agree to putting in extra effort for a unspecified amount of groups.

simply put: i will not agree to putting in extra effort for a unspecified amount of groups.

When did I ever request that you do that? I just wanted to add a section into the code of conduct that says that no matter the personal believes or differences one might have, they will not be exposed to discrimination, criticism, harassment, or any other emotional or physical harm.

no to all but criticism.

So you would criticise someone publicly for their opinion they never voiced to you or any other person in the space the code of conduct applies to?

Snippets from the official rust code of conduct:
"We are committed to providing a friendly, safe and welcoming environment for all, regardless of level of experience, gender identity and expression, sexual orientation, disability, personal appearance, body size, race, ethnicity, age, religion, nationality, or other similar characteristic."
"Respect that people have differences of opinion [...]"
This is exactly what we need in this code of conduct.

I feel like the following sections are suffecient to explain that everyone is treated well
https://github.com/0atman/noboilerplate/blob/c895449ba434077008f315596a014da492898999/CODE_OF_CONDUCT.md#L50-L60
https://github.com/0atman/noboilerplate/blob/c895449ba434077008f315596a014da492898999/CODE_OF_CONDUCT.md#L18

i'm generally open to clarifying what the safe spaces section is no, its not a list of the only groups we care about. its specifically the limited list of things we choose to accomodate and pay extra attention to.
i feel like this is a core misunderstanding here, and if you have ideas how to alleviate such a misunderstanding in the future, i'd love to incorporate it.

I feel like the following sections are suffecient to explain that everyone is treated well
https://github.com/0atman/noboilerplate/blob/c895449ba434077008f315596a014da492898999/CODE_OF_CONDUCT.md#L50-L60
https://github.com/0atman/noboilerplate/blob/c895449ba434077008f315596a014da492898999/CODE_OF_CONDUCT.md#L18

Dont you see how this is flawed? This would imply that, since you "feel like the following sections are suffecient to explain that everyone is treated well", we dont need the "safe space" section.

An important thing for my server is that it's not a free-for-all, I DON'T respect the opinion of transhobes, homophobes, bigots, etc - it's not twitter by design.

I completely understand what you mean, and my goal with this is not to allow these people to make others uncomfortable or somehow add a loophole so they can go wild on the discord. My goal is to clarify that, as long as a person follows the rules already specified in the code of conduct, they will be treated with a base level of respect (they will not be exposed to discrimination, criticism, harassment, or any other emotional or physical harm). That is all. Nothing more to it. Really.

I feel like once i say something that does not align with the opinions or believes held by some people in this space, everyone will try to invalidate my opinions and arguments by any means possible, with little sense for reason. I never ever had bad intent with any of my contributions to either conversations on the discord, or this code of conduct. I've been putting a exceptional amount of time into these things and have felt almost nothing but disregard and contempt in return.
Hell, i've even made sure to double-check everything in this PR before i even made it. I googled the official definitions (like "including, but not limited to" or "safe space") because english is not my main language, and designed my suggestions around those definitions.
I have tried to tried my absolute best to contribute in the most clear and simple way possible. I've tried to be the most accepting possible within reason, but somehow failed again.
I will close this and leave the discord server. I am genuinely sorry for the inconvenience i've caused, i just cant seem to understand this.
@laundmo I'd recommend to document the goals and definitions for things used in this code of conduct if they differ from the official ones. That would make the process of contributing for other people alot simpler and more painless.
Thank you two for the conversation, i respect both of you and apologize if that didnt come across correctly.
But for my own mental health, i will not put any more of my time into this right now.

@lexika979 I'm sorry this has become stressful, thank you for your help


subscribe() returns unsubscriber

After poking around the socket.io source code, found removeListener (wonderfully undocumented)

Functionally, the docs also need to be updated to represent the change made.  Otherwise no one will know that it returns an unbind function.

This looks good to me, thanks @ProLoser!  Added docs, but I'm not sure how the http://balderdashy.github.io/angularSails site gets regenerated; @robwormald?


TemplateConfig.json files (for adding template files to a project) should be conditional on components

Currently, TemplateConfig.json files can specify that their templates only apply to web scenarios. Now that the update assistant understands project components more generally (WindowsDesktop, web, etc.), users should be able to list any components the templates apply to in a config file - not just web.

And language


Help with grant password

Not sure if your library has implemented it, but I would like to use the password grant to get a token without having to pop open a window in iOS, (SSO) which will allow you to put in your username and password and in a request object send over the username, password, clientID, clientSecret and receive an access token.  This will also help with my previous "auto-approve" ticket....

@ghost @jordidiaz @anderson916
How did you solve this?


Visualization with IGV on VCF files

Cannot load IGV local on VCF files when running on a GVL 5.0
(Example in this tutorial https://training.galaxyproject.org/training-material/topics/variant-analysis/tutorials/non-dip/tutorial.html#looking-at-the-data)

This might be related to sanitize_whitelist like JBrowse, but because IGV is not a real tool, not sure if it is (or how to add it) or if there is something else going on. Thoughts @afgane @dannon ?

Hey, thanks for the ping.  Do you have a link to a history I can look at?  If not, I can build one from that guide I guess.

I can reproduce on a GVL instance

Quast was the one where adding to sanitize_whitelist didn't fix the rendering for the HTML output: https://github.com/galaxyproject/galaxy-helm/issues/118

Closing this given that behavior is same as Main and IGV just needs a local component.


Possible to finetune existing GPT2 model?

I have a pre-existing GPT2 model that was fine tuned on a large dataset. I would like to take that model and run it through xturing to see how if the instruction capabilities are transfered while still retaining the specialized knowledge/tone from the original fine tuning.
Is it currently possible to use an existing pytorch_model.bin and pass it in to BaseModel.create()?

Hi @wagesj45 , you can pass your weight path to BaseModel.create()
model = BaseModel.create("gpt2_lora", "path/to/your/weight")
or
model = BaseModel.create("gpt2", "path/to/your/weight")
Please let us know if you need any further information. Thank you.


Enhancements: Enhance Tests and Improve CI Workflow

Overview
This pull request introduces significant improvements to the testing framework and CI workflow for the project. It ensures comprehensive test coverage and robust CI integration with test coverage reporting.

Key Changes
🧪 Testing Enhancements

Enhanced unit, integration, and end-to-end (e2e) testing.
Added new e2e tests for contacts and messages features.
Refactored existing tests to improve reliability and coverage.
Skipped redundant tests and avoided duplication across test types.

✅ CI/CD Improvements

Integrated pytest-cov for test coverage reporting.
Configured CI to fail if test coverage falls below 80%.
Added a step to visualize test coverage in PRs via a coverage report artifact.
Fixed warnings related to pytest markers.

Testing Details

Verified that all unit, integration, and e2e tests pass successfully.
Ensured coverage threshold checks work as expected in CI pipelines.

How to Review

Review the added and refactored test cases in tests/unit/, tests/integration/, and tests/e2e/.
Check the updated CI workflow files (.github/workflows/ci.yml).
Ensure all workflows and test cases are consistent with project requirements.

Additional Notes

No breaking changes introduced.

Checklist

[x] All tests pass locally and in CI.
[x] Test coverage meets the 80% threshold.
[x] CI workflow has been updated to include coverage reporting.

Feel free to reach out for any questions or suggestions for further improvements!

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
:information_source: You can also turn on project coverage checks and project coverage reporting on Pull Request comment
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Save results in intermediate representation

Currently all bindings are stored in couple of global std::string variables that are updated by TreeVisitor
It would be better to store bindings in some intermediate representation, so it would be possible to rearrange the code, make it more consistent and possibly filter unused declarations.

Done #10


Add basic compatibility with PHP 8.1

Q
A

Bugfix?
✔

Breaks BC?
❌

New feature?
❌

Added base compatibility with PHP 8.1

Nice


Preview Enable / Disable

Would it be possible to add in commands to enable and disable the preview function on the Hyperdeck?
preview: enable: true and preview: enable: false
Currently if you play a clip back there's no way of returning the transport to the "preview" state to enable you to monitor incoming signals and start a new recording.

This feels like it should be a config option rather than a button action. Are there any situations where one would want to toggle this on a button?

Sure.  On the hardware hyperdeck pressing the “input” button performs this action.
In my use case I have a hyper deck somewhere in a rack. I have the output of the deck routed to a monitor in front of me.  While recording i can see what’s passing “through” the deck.
If I hit play then the last recorded clip starts playing back.
When I now hit “stop” the deck does not go back to previewing it’s input, rather it cues the clip up again for playback.  Right now I have to haul myself up, find the hyper deck and press the input button.  Or disable the hyper deck instance, enable the simple tcp instance and send the command that way.
It’s not so much a setup option as it needs sending each time you play back a clip.
Sent from my iPhone

On 2 May 2021, at 11:08, Stephen Harrison @.***> wrote:

This feels like it should be a config option rather than a button action. Are there any situations where one would want to toggle this on a button?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hello, how to use it

+1 for this request.  It is very important when using multiple HyperDeck mini units together as they won't all go into record when requested if not in preview to begin with.  The command is straight forward, the manual shows it as:  preview: enable: {true/false}  and that would let us tie it to a button if we want or make a button just for that.


Reimplement serialization method

https://github.com/ga4gh/vr-schema/issues/31#issuecomment-491962997

Oops. Duplicate issue.


Abnormal closure due to "undefined symbol: __atomic_load" on x86

Description
I'm trying to use nchat on an old x86 machine and the program crashes during '--setup'.
How to reproduce it

Normally build on a x86/32-bit system:

$ git clone https://github.com/d99kris/nchat
$ cd nchat
$ mkdir build && cd build/
$ CC=clang CXX=clang++ cmake ..
$ make

Try to run:

./nchat --setup
Protocols:
0. Dummy
1. Telegram
2. WhatsAppMd
3. Exit setup
Select protocol (3): 1
Enter phone number (ex. +6511111111): +my_phone
$
$ echo $?
127

Environment

Version: 80cb5ff687b252ebb2ad9970a974f5c158caa681
OS / distro: Slackware 15 i586, Clang v13.0.0, Go v1.19.3

My guess:
Doing a strace in the execution of nchat, one can see the attempt to load/call the '__atomic_load' function during execution... which is not successful and the program closes:
$ strace -ff -o dump ./nchat --setup
Protocols:
0. Dummy
1. Telegram
2. WhatsAppMd
3. Exit setup
Select protocol (3): 1
Enter phone number (ex. +6511111111): +my_phone
$
$ echo $?
127

$ grep -r "exit" dump.*
dump.192:exit_group(127)
dump.192: *** exited with 127 ***
dump.193: *** exited with 127 ***
dump.194: *** exited with 127 ***
dump.195: *** exited with 127 ***
dump.196: *** exited with 127 ***
dump.197: *** exited with 127 ***

$ grep "exited with 127" dump.192 -C 2
writev(2, [{iov_base="./nchat", iov_len=7}, {iov_base=": ", iov_len=2}, {iov_base="symbol
lookup error", iov_len=19}, {iov_base=": ", iov_len=2},
{iov_base="/home/david/Down/nchat/build/lib"..., iov_len=53}, {iov_base=": ", iov_len=2},
{iov_base="undefined symbol: __atomic_load", iov_len=31}, {iov_base="", iov_len=0},
{iov_base="", iov_len=0}, {iov_base="\n", iov_len=1}], 10) = 117
exit_group(127)                         = ?
+++ exited with 127 +++

The __atomic_load function seems to belong to libatomic, which is not linked to nchat.
If I add libatomic as one of the dependencies, the program works as expected:
patchelf --add-needed libatomic.so.1 nchat

This function is probably used by one of nchat's dependencies, although I don't know exactly which one. It's also worth noting that this only happens on x86, not x86_64.
(Maybe) proposed fix:
Add 'atomic' to one of nchat's link libraries in CMakeLists.txt

Hi @Theldus - many thanks for the detailed bug report! :+1:
Yes, nchat probably needs some fixes to how it detects that inbuilt atomics are not present. I have some ideas on how to address it. I'll prepare a fix in a couple of days.

Hi, I believe this issue should be fixed with above commit. I reproduced the issue and tested above fix on Debian 11.5.0 i386 in a VM. Please let me know if you still encounter issues after the fix. Thanks!

Hi @d99kris,
I just tested it here and works like a charm now.
Thanks a lot for the quick fix and your work on nchat, it's amazing 🙂.

Thanks 🙂


Consider open up internal packages

It would be nice to let the internal/kubeconfig to be importable from different projects. This is so far the only package which can work with the kube config so it would be nice to use it from other projects. I ran into the issue, that I need the current context for one of my deployment and actual the deployment is written in Go.

I'm not sure this is a good idea because we parse kubeconfig in a very particular way that preserves the YAML structure of the document etc which you probably don't need.
You should probably use https://pkg.go.dev/k8s.io/client-go/tools/clientcmd#NewDefaultClientConfigLoadingRules to load the kubeconfig.


02 Fx/ 00 TypeScript

All is correct, only I change one picture that not correspond to process description.

Hi @jacobovirtual, about the image, it's correct because it's representing that when we use import * as $ from 'jquery' we have intellisense for jquery library

I think the highlighted CSS line can cause confusion. That is why I changed the image.

Before image, we have configured intellisense for jquery:

npm install @types/jquery --save-dev
And import * as $ from 'jquery';

There are conflicts with master, could you solve, please?
I think you have to pull the last version of master to your repository, because in this issue #62 I solved Readme links.

I'm sorry, I'm forgot launch a 'git pull lemoncode master' command over my branch.

I don't believe, I'm forgot save the file.
-1 to me

Hi @jacobovirtual, do you have pending push the changes or something like that?


Add debt status

The user should know if they have already paid (or received) the saved debt. They should be able to change that status whenever they want.

To change the status the user should use the edit debt endpoint

Fixed on #79


Regression: compare got harder to understand

Subject: https://github.com/rfalke/decompiler-subjects/tree/master/from_holdec/dmi/exp/ia32_elf/subject.exe
Before
 int32_t intermediate_2_cmp_with_extra(int32_t a1) {
    // 0x8048551
    if (a1 <= 19 && a1 > 10) {

Now
 int32_t intermediate_2_cmp_with_extra(int32_t a1) {
    if ((uint32_t)(a1 - 11) <= 8) {

This "subtract and compare it as unsigned" is a nice implementation trick but decreased the readability.

Hi, are you able to provide what RetDec version was used for the "before" snippet?

https://github.com/rfalke/decompiler-subjects/commits/master/from_holdec/dmi/exp/ia32_elf/by_retdec.c shows https://github.com/avast/retdec/commit/d57764abb7e45de61a22e45df298161b280316e1 from Sep. 2018.
I only run these from time to time.


Any automatic way to help solve network HNS issue for service fabric with windows docker container?

Dear,
We are using service fabric(6.5.664.9590) on azure with docker running on windows server 2016.
We often get the network HNS issue for several nodes, details error message as below:
Failed to start Container. ContainerName=********, ApplicationId=******, ApplicationName=*****. DockerRequest returned StatusCode=InternalServerError with ResponseBody={"message":"failed to create endpoint ***** on network nat: HNS failed with error : Unspecified error"}

And get temp work around https://stackoverflow.com/questions/38070837/windows-container-failed-to-start-with-error-failed-to-create-endpoint-on-netw/ as
Get-NetNatStaticMapping | ? ExternalPort -eq 8811 | Remove-NetNatStaticMapping
Restart-Computer

While this is not an automatic way to ensure application health in service fabric cluster.
Is there any great solutioncould help on this scenario in an automatic way? Or any settings in service fabric could help on this?
Appreciate for the kindly help, thanks.

+1


Installer version 0.3 missing

Checked out the last release: https://github.com/marticliment/WingetUI-Widgets/releases/tag/0.3
I noticed there is no installer attached like previous versions.
Is the only way forward to install it from the Microsoft Store?

Microsoft Store is the intended source, and installing version 0.2 should trigger Microsoft Store updates.
However, I planned to add the installer later after it passed Store Certification, but I just hand't done it yet.


In LIBERO dataset, how to get the position of end-effector in image coordinate？

I want to extract the 2d trajectory of end-effector. But I can't find the camera intrinsic and extrinsic parameters of LIBERO dataset.  Any suggestions? Thanks.

This is a question probably best asked in the Libero repo, which the libero or robosuite authors may be able to answer.


freezing frame some time

https://drive.google.com/open?id=0BxaRQ7RkdWbqeU5DNHV5R3I2VlE
**windows 7 64 **

** Chrome Version 61.0.3163.100 (Official Build) (64-bit)**
after effect <IP_ADDRESS>7

What did you do? Please explain the steps you took before you encountered the problem.
after rendering my compostion it was not working at the first time after some tweak i relaze that the effect should removed from shape object on after effect so i remove it and reneder it again know its work but the motion stop before it end (freez frame ) and you should refrech browser many time (may be 4 at least) to be played fully
What did you expect to happen?
i think its a problem with server side when comunicate with json file because it work better in local host
'but not perfect'
What actually happened? Please include as much relevant detail as possible.
the consile appear this error how ever it work perfclte even if it appear so may be not the case:
VM137:2 Uncaught TypeError: comp(...).layer(...).effect is not a function
at r.eval (eval at initiateExpression (bodymovin.js:15), :2:88)
at r.execute [as getValue] (bodymovin.js:13)
at GroupEffect.getValue (bodymovin.js:15)
at IShapeElement.BaseElement.prepareFrame (bodymovin.js:7)
at ICompElement.prepareFrame (bodymovin.js:9)
at SVGRenderer.renderFrame (bodymovin.js:5)
at AnimationItem.renderFrame (bodymovin.js:9)
at AnimationItem.gotoFrame (bodymovin.js:9)
at AnimationItem.setCurrentRawFrameValue (bodymovin.js:9)
at AnimationItem.advanceTime (bodymovin.js:9)
Please provide a download link to the After Effects file that demonstrates the problem.
https://drive.google.com/open?id=0BxaRQ7RkdWbqeGdiVmtLX0Ywd0U
thank you a lot

Can you try it without the text layers on?
The font you chose might not work for bodymovin.
Try converting the text to outlines (right click layer then ).
That might help.

thank you mr DaanDirk i solve it from a while its just you can't applay effect on layer you can just animate shape i don't know if it problem in bodymovin or its Capabilities of this add-on
you are my hero keep moving i will try to be like you
thank you


app id is now automatically saved

The documentation says to "click save" when setting application ID URI but there is no save and tabbing outside of the field automatically saves i

t.

@BryceAshey : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

#sign-off


Guidance generation fails on OpenAI gpt-4-1106-preview

The bug
When using gpt-4-1106-preview, guidance generation fails. Details below.
To Reproduce
This code fails when run in a Jupyter notebook with the error shown below.
When using gpt-3.5-turbo or gpt-4 instead it generates without problems.
All model names taken directly from the OpenAI documentation here: https://platform.openai.com/docs/models/gpt-3-5
gpt = models.OpenAI("gpt-4-1106-preview")

with user():
    lm = gpt + "What is the capital of France?"

with assistant():
    lm += gen("capital")

with user():
    lm += "What is one short surprising fact about it?"

with assistant():
    lm += gen("fact")

Error
---------------------------------------------------------------------------
Exception                                 Traceback (most recent call last)
Cell In[3], line 4
      1 gpt = models.OpenAI("gpt-4-1106-preview")
      3 with user():
----> 4     lm = gpt + "What is the capital of France?"
      6 with assistant():
      7     lm += gen("capital")

File c:\Users\UserName\AppData\Local\pypoetry\Cache\virtualenvs\robin-LIckTrhS-py3.11\Lib\site-packages\guidance\models\_model.py:200, in Model.__add__(self, value)
    198 if context not in lm.opened_blocks:
    199     lm.opened_blocks[context] = "" # mark this so we don't readd when computing the opener (even though we don't know the close text yet)
--> 200     lm += context.opener
    201     with grammar_only():
    202         tmp = lm + context.closer

File c:\Users\UserName\AppData\Local\pypoetry\Cache\virtualenvs\robin-LIckTrhS-py3.11\Lib\site-packages\guidance\models\_model.py:246, in Model.__add__(self, value)
    242         out = lm._run_stateless(value)
    244     # run stateful functions
    245     else:
--> 246         out = value(lm)
    248 # this flushes the display
    249 out._inplace_append("")

File c:\Users\UserName\AppData\Local\pypoetry\Cache\virtualenvs\robin-LIckTrhS-py3.11\Lib\site-packages\guidance\_grammar.py:54, in StatefulFunction.__call__(self, model)
     53 def __call__(self, model):
---> 54     return self.f(model, *self.args, **self.kwargs)

File c:\Users\UserName\AppData\Local\pypoetry\Cache\virtualenvs\robin-LIckTrhS-py3.11\Lib\site-packages\guidance\library\_role.py:9, in role_opener(lm, role_name, **kwargs)
      7     raise Exception(f"You need to use a chat model in order the use role blocks like `with {role_name}():`! Perhaps you meant to use the {type(lm).__name__}Chat class?")
      8 lm += f"<||_html:

{role_name.lower()}

_||>"
----> 9 lm += "<||_#NODISP_||>" + lm.get_role_start(role_name, **kwargs) + "<||_/NODISP_||>"
     10 return lm

File c:\Users\UserName\AppData\Local\pypoetry\Cache\virtualenvs\robin-LIckTrhS-py3.11\Lib\site-packages\guidance\models\_model.py:475, in Instruct.get_role_start(self, role_name, **kwargs)
    474 def get_role_start(self, role_name, **kwargs):
--> 475     raise Exception("Subclasses need to define what the role start should be!")

Exception: Subclasses need to define what the role start should be!

System info (please complete the following information):

OS Windows 10.
Guidance Version: 0.1.4
Python version: 3.11.6

I don't think it has the mechanism to detect that it is supposed to be a chat model. I got this to work by using
lm = models.OpenAIChat("gpt-4-1106-preview")

Ah, yes, that works. Thank you so much and sorry for the confusion.


Add reusable compontents

Buttons
style the parent div

Parent elements should have paddings insted of child incidents (every single element margin+padding)


Working with multiple images on a canvas.

Hi
I'm loving CamanJS and got a nice demo working with a single image on a html5 canvas. But I need some advice on the best way to work with multiple images on a canvas.
In my prototype a user can upload an image to the browser using the FileAPI and then manipulate it with Caman.  (Similar technique to this http://tutorialzine.com/2013/02/instagram-filter-app/)
What would be the best way to allow a user to upload a second or third image and position them on the canvas too?
I've tried working natively with the canvas API:
var ctx = document.getElementById(canvasid).getContext('2d');
var img = new Image;
img.src = URL.createObjectURL(file); //from FileAPI
ctx.drawImage(img, 20,20,100,100);

But this seems to be working outside the Caman framework (and I've needed to el.removeAttr('data-caman-id') on the canvas to make it work). I've also tried working with overlays:
Caman("#canvas", function () {
  this.newLayer(function () {
    this.overlayImage(imageobject);
  });
  this.render();
});

But these seem to stretch the image to fit the entire canvas and I can't find a way to control aspects like the scaling and positioning of the overlay with Caman.
Would anyone recommend one of these approaches over another, or is there a better way?
Thanks
Adam

@meltingice is this feature integrate into current release of caman js ?


Fix class='undefined'

Currently, <Box>Hello</Box> results in
Hello
 because we do undefined + "" here.
With this fix, the result is simply
Hello
.

@moroshko great catch! thanks for your contribution! 🙌

@moroshko woot! a new version 0.4.1 has just been published and it contains this fix, thank you so much for fixing it!

@moroshko soon we'll be adding tests to catch these issues early and make sure we get no regressions 👍


python3Packages.opnieuw: init at 3.0.0

Description of changes
Opnieuw is a Python retry library. This PR adds it to the Python package set.

GitHub repo: https://github.com/channable/opnieuw
Blog post introducing the lib

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

@GrahamcOfBorg build python3Packages.opnieuw

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 303324

x86_64-linux

:white_check_mark: 4 packages built:

python311Packages.opnieuw
python311Packages.opnieuw.dist
python312Packages.opnieuw
python312Packages.opnieuw.dist

x86_64-darwin

:white_check_mark: 4 packages built:

python311Packages.opnieuw
python311Packages.opnieuw.dist
python312Packages.opnieuw
python312Packages.opnieuw.dist

aarch64-darwin

:white_check_mark: 4 packages built:

python311Packages.opnieuw
python311Packages.opnieuw.dist
python312Packages.opnieuw
python312Packages.opnieuw.dist


PHP fails to compile (segmentation fault)

vagrant@precise32:/vagrant/benchmarks/benchmarks/php-bug-2011-01-06-e7a1d5004e-3571c955b5/php$ make
Generating phar.phar
PEAR package PHP_Archive not installed: generated phar will require PHP's phar extension be enabled.
directorygraphiterator.inc
clicommand.inc
pharcommand.inc
invertedregexiterator.inc
directorytreeiterator.inc
make: *** [ext/phar/phar.phar] Segmentation fault

Appears to be the result of changes to GCC: https://bugs.php.net/bug.php?id=51216

Can be averted by configuring with following options:
configure --disable-phar

PHP now compiles on Fedora 22, following the application of the libxml2 incompatibility patch: .


secret.yml have no "apiVersion" key?

It would failed when run command helm install if no apiVersion.
Kubectl apply failed. Error: error: error validating "/tmp/app-template891208987": error validating data: apiVersion not set; if you choose to ignore these errors, turn validation off with --validate=false : exit status 1

@SDLyu Which chart? I'll update the charts.

@SDLyu I've correct them, let use know how you get on, strangely Helm init and template is ok.

Hi @andrewstevenson
There are some typos, and I open #18 to correct them.

BTW, I can't pull image from repository registry.hub.docker.com/datamountaineer/kafka-connect-elastic5, but change repository to datamountaineer/kafka-connect-elastic5 is ok.

@SDLyu  ok, updating the rest now. Thanks!

Hi @andrewstevenson
It seems that need to run command bash package.sh to update charts, because helm still install old charts.
Thanks

@SDLyu I've run the script and pushed.


kademlia: connector took ...

Always print the log ...
level=trace msg="kademlia: connector took 3.723µs to finish: old depth 0; new depth 0"

More ...
time="2021-06-30T10:42:20Z" level=trace msg="kademlia: connector took 3.594µs to finish: old depth 0; new depth 0"
time="2021-06-30T10:42:20Z" level=debug msg="kademlia: no connected peers, trying bootnodes"
time="2021-06-30T10:42:26Z" level=debug msg="batch service: updated block height to 16843231"
time="2021-06-30T10:42:32Z" level=debug msg="batch service: updated block height to 16843232"
time="2021-06-30T10:42:35Z" level=trace msg="kademlia: connector took 3.592µs to finish: old depth 0; new depth 0"
time="2021-06-30T10:42:35Z" level=debug msg="kademlia: no connected peers, trying bootnodes"
time="2021-06-30T10:42:38Z" level=debug msg="batch service: updated block height to 16843233"
time="2021-06-30T10:42:40Z" level=info msg="debug api access" duration=0.006422733 ip=<IP_ADDRESS> method=GET proto=HTTP/1.1 size=5841 status=200 uri=/metrics user-agent=Prometheus/2.27.1
time="2021-06-30T10:42:41Z" level=debug msg="batch service: updated block height to 16843234"
time="2021-06-30T10:42:47Z" level=debug msg="batch service: updated block height to 16843235"
time="2021-06-30T10:42:50Z" level=debug msg="batch service: updated block height to 16843236"
time="2021-06-30T10:42:50Z" level=trace msg="kademlia: connector took 3.347µs to finish: old depth 0; new depth 0"
time="2021-06-30T10:42:50Z" level=debug msg="kademlia: no connected peers, trying bootnodes"
time="2021-06-30T10:42:59Z" level=debug msg="batch service: updated block height to 16843237"
time="2021-06-30T10:43:05Z" level=trace msg="kademlia: connector took 3.889µs to finish: old depth 0; new depth 0"
time="2021-06-30T10:43:05Z" level=debug msg="kademlia: no connected peers, trying bootnodes"
time="2021-06-30T10:43:05Z" level=debug msg="batch service: updated block height to 16843238"
time="2021-06-30T10:43:10Z" level=info msg="debug api access" duration=0.006314354 ip=<IP_ADDRESS> method=GET proto=HTTP/1.1 size=5850 status=200 uri=/metrics user-agent=Prometheus/2.27.1
time="2021-06-30T10:43:11Z" level=debug msg="batch service: updated block height to 16843239"
time="2021-06-30T10:43:14Z" level=debug msg="batch service: updated block height to 16843240"
time="2021-06-30T10:43:20Z" level=trace msg="kademlia: connector took 3.723µs to finish: old depth 0; new depth 0"
time="2021-06-30T10:43:20Z" level=debug msg="kademlia: no connected peers, trying bootnodes"
time="2021-06-30T10:43:20Z" level=debug msg="batch service: updated block height to 16843241"
time="2021-06-30T10:43:27Z" level=debug msg="batch service: updated block height to 16843242"
time="2021-06-30T10:43:33Z" level=debug msg="batch service: updated block height to 16843243"
time="2021-06-30T10:43:35Z" level=trace msg="kademlia: connector took 3.772µs to finish: old depth 0; new depth 0"

Somebody can tell me why?

Bee Version
{
  "status": "ok",
  "version": "1.0.1-dev"
}

And I use myself xDai node.


Fixing Data provider mismatch

Fixes # .
Did you remember to?

[X] Add test case(s)
[X] Update CHANGES.txt

Fixes #1181

@juherr /  @cbeust - Can you please help take a look at this ?