Fix forever loop in response line parsing step + test

Codecov Report

Merging #474 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #474      +/-   ##
==========================================
+ Coverage   67.62%   67.62%   +<.01%
==========================================
  Files          49       49
  Lines        6826     6827       +1
==========================================
+ Hits         4616     4617       +1
  Misses       2210     2210

Impacted Files
Coverage Δ

src/common/http.cc
73.98% <100%> (+0.06%)
: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 a72d6b8...d38e559. Read the comment docs.


Fix git Maven params

Summary
Cherry-pick of https://github.com/quarkus-qe/quarkus-test-framework/pull/649

This change is not backwards-compatible and broke[1] our CI.  @mjurc is there any reason for  this feature to be in 2.13?
[1] https://github.com/quarkus-qe/quarkus-test-suite/actions/runs/4057590113/jobs/6983943724

@fedinskiy I think it can be easily fixed by changing from QUARKUS_VERSION to QUARKUS_PLATFORM_VERSION in your https://github.com/quarkus-qe/quarkus-test-suite


Implemented ClusterIdentity as value object for Identity + Kind in cl…

Implemented ClusterIdentity as value object for Identity + Kind in cluster context.
Fixed issue #516, now uses full cluster identity for lookups when creating actors.
Rewritten pausing of PartitionIdentityActor.
Description
Fixes #516
Purpose
This pull request is a:

[X] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[X] Breaking change (fix or feature that would cause existing functionality to not work as expected)
Changed on the wire-formats for Proto Cluster.

Checklist

[X] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.magne.helleborg seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Uncaught ReferenceError: require is not defined

I have started an angular2 project for learning AoT(). I have followed each steps as mentioning into the documentation. At the end when i run project through npm run lite it's showing me this error on browser console.

I'm sorry but your issue isn't a QuickStart repo issue; it belongs on GH angular.io. That's why I'm closing.


Revert multiwindow support

Too buggy, needs more development.
This reverts commit f1b728501ef263d1cf344263c675d7d146a15239.
This reverts commit e7fdb4cdf0942cd6e63dd822110a93c0ec777fe5.
This reverts commit 9e36638afd4dcc06bc94a661940f3648ce067da2.
This reverts commit 6cba006378b0ca68a66ea54153c0113724c91753.
This reverts commit bf6386d9bbb9e53386987beb90264abd4faf5c74.
This reverts commit 3de5708b3060554ba5f383404f2ce6516e9dbed1.
This reverts commit 63374d1c36050fdea7a773aa4125b45e9a8da8f5.

https://github.com/lewis6991/satellite.nvim manages multi window pretty well. Never had an issue with it. Maybe you can take a look at that.

Okay, I think I know how to fix two existing bugs (https://github.com/nvim-treesitter/nvim-treesitter-context/issues/522 and https://github.com/nvim-treesitter/nvim-treesitter-context/issues/512#issuecomment-2448422407). Will you give me until the end of the weekend to fix them? Is there anything else you would like to fix besides the two existing problems?

I'm seeing issues my side too. Not sure if they are the same issues, but I've not got bandwidth to debug them.

I can wait a week for you to apply some fixes and see if that addresses the issues. Would that be ok or shall I revert now?

Sounds great, thanks. I'm already fixing it now, I'll bring the PR soon.

Things feel more stable now. Closing.


Update webdev.astro

Description of Changes

Added web dev resource
Checklist:

[x] My code adheres to the established style guidelines of this project.
[x] I have conducted a self-review of my code.
[x] I have included comments in areas that may be difficult to understand.
[x] I have made corresponding updates to the project documentation.
[x] My changes have not introduced any new warnings.

closes #96

@Abhinavcode13 Fix conflicts


Add alignment

Add supporting textAlignment so in toolTip we can use textAlignment.
This enable users with RTL content also be able to use this library.

@metinkilicaslan would you review this PR.


Add the newtype pragma

This PR adds the newtype pragma to allow compiling from data and record definitions in Agda to newtype definitions in Haskell (under the constraint that the definition has exactly one non-erased field).

Output for the compiled test/NewTypePragma.agda file.
module NewType.Example where

-- data newtype

newtype Indexed a = MkIndexed (Int, a)

index :: (Int, a) -> Indexed a
index = MkIndexed

-- data newtype with deriving

newtype Pair a b = MkPair (a, b)
                     deriving (Show, Eq)

-- record newtype

newtype Identity a = MkIdentity{runIdentity :: a}

-- record newtype with erased proof

newtype Equal a = MkEqual{pair :: (a, a)}

There are still a few things I would like to change (input welcome):

[x] Reusing code within the compileRecord branches. The constraints are the problematic part here.
[x] Make the test file actually be part of make test?
[x] Add documentation about data and type declarations.

Thanks for the contribution! After some small changes I requested, I can rebase and merge.

I've fixed all the comments and rebased on master!


Swap from Specs2/JUnit to ScalaTest

This project's out of step now with our main unit test framework. Looks like Specs2+JUnit doesn't support things like for comprehension driven test definitions or even throwing multiple asserts into a single test. This makes writing tests that can catch multiple permutations of inputs exceedingly difficult.
Example of a working ScalaTest for comprehension:
"upperCase function" should {         for ((inputString, expectedResult) <- Seq(           ("a", "A"),           ("bcd", "BCD"),           ("jhk1", "JHK1")         )) s"upperCase string $inputString" in {           assert(upperCase(inputString) == expectedResult)         }     }
Can't do this with Specs2/JUnit though, so instead you would need to write out 3 individual tests to get the same result. This simple example might not be difficult to do but with the ScalaTest capabilities I can easily add another 20 test cases to the same test and have meaningful results output for each individual iteration.

We still have a few specs2 tests left in wookiee-core/test, I converted everything in Wookiee ZK but we should make sure we convert everything else as we go. I'll throw a task onto our stories

Here's the PR that does this:
https://github.com/oracle/wookiee/pull/154


Make builds for major and minor releases

Godot version
4.0.0, 4.0.1, 4.0.2, 3.5.2, etc.
Plugin version
latest
System information
All
Issue description
It would be nice to have the builds for major and minor releases of Godot 4 (also 3).
We are making a plugin that requires WebRTC on all supported platforms and well, that requires the builds for each platform.
To my understanding there is already a build script, but still requires some manual work for upload.
Steps to reproduce
N/A
Minimal reproduction project
N/A

The idea with GDNative/GDExtension is that they should be compatible across minor release.
So, the current version, should work for minor releases too (3.5.x and 4.0.x).
Did you encounter crashes/issues with the current plugin in the latest 3.5.x and 4.0.x?

Hey Fabio,
Thanks for the clarification.
I haven't properly tested the GDExtension in our plugin as it is WIP right now and I am just going through a design checklist.
So my apologies if I jumped the gun here.
The release notes does mention it might not work with different versions, but I understand that does not have to be the case.
But it has been a while from since 4.0-rc1, and with the official release out, I think it is not a bad idea to have a new build.
If you don't think its necessary, then you can close this. And I'll just report any issues along the way.
Thanks!

We now have a stable release which should work with all Godot 4.0 patch releases (e.g. 4.0.1, 4.0.2, etc).
The 3.5 version should already work across patch releases.
Closing as fixed.


[Enhancement] [Api] Option to omit a get param if the value if not defined

Summary
For a get api, add an option to omit get param if the value if not defined
Additional Context
Quoting from discord:
on the api create form, if we add a param, it automatically appends = sign to the key. Can we have a workaround if no value is defined then no = sign is added

Duplicate of #5756


Syncpublicpreview

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[ ] The title of the PR is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[ ] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes.
[ ] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[ ] I have read the contribution guidelines.
[ ] My spec meets the review criteria:

[ ] The spec conforms to the Swagger 2.0 specification.
[ ] The spec follows the guidelines described in the Swagger checklist document.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Can one of the admins verify this patch?

Automation for azure-sdk-for-python

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-node

Automation for azure-sdk-for-java

We previously have reviewed with them for GA.  Is this needed for every update to our public api?

ping, please set up a meeting as noted in the email.

reviewed in person. Signing off from ARM side.


Update README.md

Qovery can create a Preview Environment for this PR.
To trigger its creation, please post a comment with one of the following command.

Command
Blueprint environment

/qovery  preview 55867c71-56f9-4b4f-ab22-5904c9dbafda
production

/qovery  preview 315d34c5-6235-4c61-b002-f12017814a66
production-clone

/qovery  preview {all|UUID1,UUID2,...}
To preview multiple environments

This comment has been generated from Qovery AI :robot:.
Below, a word from its wisdom :

There is no better time than now, better future does not exist

/qovery preview 55867c71-56f9-4b4f-ab22-5904c9dbafda

A preview environment was automatically created via Qovery.
Click on the link below to follow its deployment and use it.
:point_right: [PR] production - Update README.md - 2024-12-13T16:07:12Z
Another comment will be posted when deployments are finished

Your preview environment failed to be deployed !
Click on the link below to check the deployment logs.
:point_right: [PR] production - Update README.md - 2024-12-13T16:07:12Z


Add Hack Series cohort 2 recap notes

Wow, I am so sorry for the huge delay with this. Merging now.


Ruby 2.4.0 Fixnum to Integer repairs

repairing Fixnum depreciation with ruby 2.4.0

Do we know when this is going to be merged?

@MacCracken Does this need any more review? It seems pretty straightforward and would be nice to get this deprecation fixed for us folks on Rails 5. I would help but I'm not sure there is anything to do here.

I believe all the fixes are in there. its on @cheezy to review and update.

ping @cheezy

On this now.  Will merge and make a new release this evening.


Description for resourceExtension is missing content.

The definition for "XAML Values > resourceExtension" is "One of the following: or . This usage is used..." but it appears there's an air gap on both sides of the "or" where two possible values for resourceExtension should go.  e.g. it should read "One of the following: possibility_1_ or possibility_2. This usage is used"

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

ID: d33ed34a-3b73-2b51-914f-230226acc07a
Version Independent ID: a18f223c-79b0-c73a-5f0e-3d343b88c519
Content: FrameworkElement.DataContext Property (System.Windows)
Content Source: xml/System.Windows/FrameworkElement.xml
Product: .net
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

Thanks for pointing out that text had dropped out of the documentation, @AppFzx. The lost text is StaticResource or DynamicResource. I'll open a PR to restore the lost text shortly.


"Pause Orchestration" button is missing

This is an important debugging tool used widely in one specific environment. It remembers the scale of the workload and one doesn't have to remember what it was set to while debugging.
The prior UI had this function within the Workload view, in between the Redeploy and Download YAML buttons.

PASS Verified fixed in 2.6-head Commit ID 78b25c3

verified that the pause orchestration button was NOT in 2.6.2

created rke2 ec2 cluster in 2.6-head
created a deployment nginx:latest
clicked the 3 dots to the right of the new deployment
verified pause orchestration option is present
clicked pause orchestration
test-deployment changed from active to paused
clicked the 3 dots to the right of the new deployment
clicked resume orchestration option
test-deployment changed status from paused to active


Install fails on OKD 4.7

Description
Installation fails on OKD 4.7 with FCOS 33.
Steps to reproduce the issue:

curl https://raw.githubusercontent.com/openshift/kata-operator/master/deploy/install.sh | bash

Describe the results you received:
Installation fails. Looks like the rpm packages' dependencies are not friendly to FCOS...
Describe the results you expected:
Installation succeed ??? (I am actually asking, given it is OKD)
Additional information you deem important (e.g. issue happens only occasionally):
Output of oc describe kataconfig <your-kataconfig>:
$ oc describe kataconfig example-kataconfig
Name:         example-kataconfig
Namespace:
Labels:       <none>
Annotations:  <none>
API Version:  kataconfiguration.openshift.io/v1
Kind:         KataConfig
Metadata:
  Creation Timestamp:  2021-03-19T01:32:06Z
  Finalizers:
    finalizer.kataconfiguration.openshift.io
  Generation:  2
  Managed Fields:
    API Version:  kataconfiguration.openshift.io/v1
    Fields Type:  FieldsV1
    fieldsV1:
      f:metadata:
        f:annotations:
          .:
          f:kubectl.kubernetes.io/last-applied-configuration:
    Manager:      kubectl-client-side-apply
    Operation:    Update
    Time:         2021-03-19T01:32:06Z
    API Version:  kataconfiguration.openshift.io/v1
    Fields Type:  FieldsV1
    fieldsV1:
      f:metadata:
        f:finalizers:
          .:
          v:"finalizer.kataconfiguration.openshift.io":
      f:spec:
        .:
        f:config:
          .:
          f:sourceImage:
        f:kataConfigPoolSelector:
          .:
          f:matchLabels:
            .:
            f:node-role.kubernetes.io/worker:
      f:status:
        .:
        f:installationStatus:
          .:
          f:completed:
          f:failed:
          f:inProgress:
        f:kataImage:
        f:runtimeClass:
        f:totalNodesCount:
        f:unInstallationStatus:
          .:
          f:completed:
          f:failed:
          f:inProgress:
        f:upgradeStatus:
    Manager:      manager
    Operation:    Update
    Time:         2021-03-19T01:34:12Z
    API Version:  kataconfiguration.openshift.io/v1
    Fields Type:  FieldsV1
    fieldsV1:
      f:status:
        f:installationStatus:
          f:failed:
            f:failedNodesCount:
            f:failedNodesList:
    Manager:         daemon
    Operation:       Update
    Time:            2021-03-19T01:34:55Z
  Resource Version:  3740359
  Self Link:         /apis/kataconfiguration.openshift.io/v1/kataconfigs/example-kataconfig
  UID:               8190feba-53a0-41ef-9d39-0a5f93da270d
Spec:
  Config:
    Source Image:
  Kata Config Pool Selector:
    Match Labels:
      node-role.kubernetes.io/worker:
Status:
  Installation Status:
    Completed:
    Failed:
      Failed Nodes Count:  2
      Failed Nodes List:
        Error:  exit status 1
        Name:   okd-compute-1.lab2.okd
        Error:  exit status 1
        Name:   okd-compute-2.lab2.okd
    In Progress:
  Kata Image:
  Runtime Class:
  Total Nodes Count:  2
  Un Installation Status:
    Completed:
    Failed:
    In Progress:
  Upgrade Status:
Events:  <none>

Additional environment details (platform, options, etc.):
$ oc version
Client Version: 4.7.0-0.okd-2021-03-07-090821
Server Version: 4.7.0-0.okd-2021-03-07-090821
Kubernetes Version: v1.20.0-1046+5fbfd197c16d3c-dirty
FCOS: 33.20210217.3.0 stable

$ oc get pod -n kata-operator-system
NAME                                                READY   STATUS    RESTARTS   AGE
kata-operator-controller-manager-79556d56d6-6j5g2   2/2     Running   2          11m
kata-operator-daemon-install-jzfxf                  1/1     Running   0          9m41s
kata-operator-daemon-install-zx76r                  1/1     Running   0          9m41s

$ oc logs kata-operator-daemon-install-zx76r -n kata-operator-system
I0319 01:34:22.488189 2250013 request.go:645] Throttling request took 1.040143398s, request: GET:https://<IP_ADDRESS>:443/apis/quota.openshift.io/v1?timeout=32s
W0319 01:34:25.546345 2250013 client_config.go:608] Neither --kubeconfig nor --master was specified.  Using the inClusterConfig.  This might not work.
2021/03/19 01:34:25 Kata operator payload tag: 4.7.0
/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
/usr/bin/mkdir -p /host/opt/kata-install
/usr/bin/mkdir -p /etc/yum.repos.d/
/usr/bin/cp -f /usr/local/kata/latest/packages.repo /etc/yum.repos.d/
/usr/bin/cp -a /usr/local/kata/latest/packages /opt/kata-install/packages
Checking out tree 17a9b3a...done
Enabled rpm-md repositories: packages
rpm-md repo 'packages' (cached); generated: 2021-02-04T21:00:23Z
Importing rpm-md...done
error: Packages not found: NetworkManager-ovs, glusterfs, glusterfs-fuse, qemu-guest-agent
/bin/bash -c /usr/bin/rpm-ostree install --idempotent kata-runtime kata-osbuilder
2021/03/19 01:34:50 exit status 1

...and if I log into one of the nodes and try myself:
[core@okd-compute-1 ~]$ sudo rpm-ostree install kata-runtime kata-osbuilder
Checking out tree 17a9b3a... done
Enabled rpm-md repositories: packages
rpm-md repo 'packages' (cached); generated: 2021-02-04T21:00:23Z
Importing rpm-md... done
error: Packages not found: NetworkManager-ovs, glusterfs, glusterfs-fuse, qemu-guest-agent

... which is interesting because:
[core@okd-compute-1 ~]$ rpm -qa | grep gluster
libglusterfs0-8.4-1.fc33.x86_64
glusterfs-8.4-1.fc33.x86_64
glusterfs-client-xlators-8.4-1.fc33.x86_64
glusterfs-fuse-8.4-1.fc33.x86_64
[core@okd-compute-1 ~]$ rpm -qa | grep qemu
qemu-guest-agent-5.1.0-9.fc33.x86_64
[core@okd-compute-1 ~]$ rpm -qa | grep NetworkManager
NetworkManager-libnm-1.26.6-1.fc33.x86_64
NetworkManager-1.26.6-1.fc33.x86_64
NetworkManager-tui-1.26.6-1.fc33.x86_64
NetworkManager-team-1.26.6-1.fc33.x86_64
NetworkManager-ovs-1.26.6-1.fc33.x86_64

... also, not that matters that much, but I cannot find any reference to /bin/bash -c /usr/bin/rpm-ostree install --idempotent kata-runtime kata-osbuilder in this repository code. I could find /bin/bash -c /usr/bin/rpm-ostree install --idempotent kata-containers though (so where is the former coming from) ?

AFAIK this won't work with FCOS since the rpms are specific for RHCOS.  The operator is not yet available for OKD.
cc @zanetworker

@bpradipt is right about the rpms. "/bin/bash -c /usr/bin/rpm-ostree install --idempotent kata-containers" is different on the master branch compared to the release-4.7 branch because we switched to a kata 2.0 rpm.
You could try to use the configmap to specify a different payload image and then build one that has FCOS rpms.
See https://github.com/openshift/sandboxed-containers-operator/blob/master/docs/DEVELOPMENT.md for how to do that.
Use the 4.7 operator because it will work with kata-1.x rpms.

@fidencio this won't also work given the MCO is not willing to support this in an upcoming release as well right?

As an additional datapoint, the mkdir /host is failing, so I suspect everything goes downhill from there (including the chroot):
cmd := exec.Command("mkdir", "-p", "/host/opt/kata-install")

We have a whole lot to figure out before we can decide whether we'll support OKD or not, and this is not only on the MCO side.
Right now, I think we should document this is not supported there and revisit this decision whenever we have cycles to support it (in a few weeks from now).

@fidencio yeah documenting makes sense. Also OKD4 uses FCOS so all the rpms/extensionsshould be in FCOS.

That's a good goal for after 4.8 release. :-)

As OKD is not supported, I'm closing this one in favour or https://github.com/openshift/sandboxed-containers-operator/issues/86


Upgrade to Rails 7.2

Why was this change made? 🤔
Upgrade to rails 7.2
How was this change tested? 🤨
⚡ ⚠ If this change involves consuming from or writing to another service (or shared file system), run integration test create_object_h2_spec.rb and/or test manually in [stage|qa] environment, in addition to specs. ⚡
Does your change introduce accessibility violations? 🩺
⚡ ⚠ Please ensure this change does not introduce accessibility violations (at the WCAG A or AA conformance levels); if it does, include a rationale. See the Infrastructure accessibility guide for more detail. ⚡

@sul-dlss/infrastructure-team - Note that @mjgiarlo and I walked through this and it seems clear that the current expectation is that ActiveStorage libraries live in lib . We believe the coverage drop is a red herring based on moving the code as it's obviously used in existing tests.


Update to React 0.14

Update to React 0.14

see #59, just change your version number in npm to https://github.com/zackify/react-chartjs and it will work for now :)

https://github.com/jhudson8/react-chartjs/pull/59


ca_dfatd_sema_sanctions list missing data

http://www.international.gc.ca/sanctions/assets/office_docs/sema-lmes.xml has 797 persons for example. However, we only return 19 persons in this project?

names = node.findtext('.//Aliases')
if names is None:
return

Thanks for reporting. Fixed in https://github.com/alephdata/opensanctions/commit/1849950cb9578ee6e6ded7beea51e9de3fda14d3


How to check TrustLine existing after account opening?

Hi, I need to check trustline state after account opening (exist trustline or not).                                 Are any errors or not?
How to better do this?
Thanks!

Hi Pavel, what do you mean by account opening? You can see if an account has trustlines to other (issuing) accounts by inspecting its balances.

I use:

StellarSDK().accounts.getAccountDetails(accountId: myAccID) { (response) -> Void in
I have success response in
switch response {
case .success(let accountDetails):

and then I'm use this for create trustline:

do {
let changeTrustOp = ChangeTrustOperation(asset: currency!)
let transaction = try Transaction(sourceAccount: accountDetails, operations: [changeTrustOp], memo: Memo.none, timeBounds: nil, maxOperationFee: StellarSettings.maxOperationFee)
try transaction.sign(keyPair: sourceAccount, network: network)
try StellarSDK().transactions.submitTransaction(transaction: transaction) { (response) -> Void in
switch response {
case .success:
successHandler()
case .failure(let error):
StellarSDKLog.printHorizonRequestErrorMessage(tag: "Trust error", horizonRequestError: error)
failHandler()
}
}
} catch {
print("New Acc Create Trustline Error")
}
If I have 0.0 XLM balance - does this mean that the trustline for the account has been successfully created?
Thanks!

Hi Pavel,
after trying to add the trust line, if you find a balance entry for the new currency in the account details, then the trustline has been added.
Best
Christian.

Hi Pavel, can I close this issue?

Hi! Thanks a lot!


Replaces deprecated np.int dtype with Python-native int dtype

This pull request updates the load_synth function to use Python's native integer type instead of np.int, which has been deprecated since Numpy 1.20
The change ensures compatibility with future versions of NumPy by replacing the np.int type casting to Python's native integer type, which is functionally identical as np.int was simply an alias to the native type prior to its deprecation.

Thanks for the PR! Will merge.


nixos/awesome: fix luaModules using pkgs.lua instead of awesome.lua

Description of changes
This is done so you can easily change the lua version awesome uses without overriding system-wide lua with an overlay.
Just setting package = awesome.override { lua = better_lua } was not enough because of getLuaPath function having hardcoded dependency on pkgs.lua.luaversion that breaks luaModules, which this change fixes
Setting package = awesome.override { lua = better_lua } breaks luaModules, this is a oneline fix for that.

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

In fact, I just realized that awesome package has passthru.lua, which would make fixing bad getLuaPath way more elegant, one second..

not sure I can request review in githubs UI, for pkgs a bot adds a maintainer but for the nixos module there's no maintainers..
@fpletz sorry to ping you, but you seem like you were involved with nixos/awesome at some point and this change is a simple oneline fix anyway, can you review/approve please?

@SuperSandro2000 also sorry to ping you, maybe you'd take a look, I've no idea how to correctly find reviewers for things like this - it's a oneline fix here

cool, thanks!)


Armor

Need to implement dark steel and end steel armor.

Not going to separate these issues out yet, these can wait until we are focussing on the toolsets


Failed to publish because I don't have perms

@dignifiedquire please add me perms (and also, to any libp2p/multiformats/ipld/ipfs thing that is created in the future)

I will try to remember

Doen


Disabling the "helpful message if logger is missing"

I don't want to use "slfj4 simple" for logging. Currently I can't find anything on how to use a custom logger for Javalin. Therefore I want to disable the built-in logging completely. However, if I don't include the slfj4 logger as a dependency when I build, I get the following error:
SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.
-------------------------------------------------------------------
Missing dependency 'Slf4j simple'. Add the dependency.

pom.xml:
<dependency>
    <groupId>org.slf4j</groupId>
    <artifactId>slf4j-simple</artifactId>
    <version>1.7.25</version>
</dependency>

build.gradle:
compile "org.slf4j:slf4j-simple:1.7.25"
-------------------------------------------------------------------

I don't want this to be shown. The output seems to come from printHelpfulMessageIfLoggerIsMissing () here.
Is there any way this can be disabled?
If not, could such a feature be implemented?

Any slf4j compliant logger will make the message go away. Why would you want to completely disable logging?

Other compliant loggers also depend on the slf4j api, simply including the dependency is enough for Javalin to find it.
By default javalin logs very little after startup, could you show me the part that's too loud?

Okay, thank you.
You're actually right about it not logging too much after startup. Most of the information during startup however is a bit irrelevant for simple use cases, basically the (optional) header and a starting message would do.
I like how router logging and web socket logging is configurable and I guess I'd like to see a similar configurability of more parts of the logging. I will look further into the slf4j api.
Again, thanks for your fast responses. I will close this for as you've answered all of my questions.


Proposed auxiliary method wrapAsCustomStyledYamlNode

This probably needs some test cases, and I'm sure the code can be improved further!
void main() {
  final doc = YamlEditor('');
  doc.update(
      [],
      wrapAsCustomStyledYamlNode({
        'title': 'Short string as title',
        'description': [
          'Multiple lines with losts of text',
          'that you really makes you want',
          'the YAML to be written with literal strings',
        ].join('\n'),
      }));
  print(doc.toString());
}

YamlNode wrapAsCustomStyledYamlNode(
  Object? value, {
  CollectionStyle Function(Map map, int depth) styleMap = _defaultMapStyle,
  CollectionStyle Function(List list, int depth) styleList = _defaultListStyle,
  ScalarStyle Function(String string, int depth) styleString =
      _defaultStringStyle,
}) {
  YamlNode wrap(Object? value, int depth) {
    if (value is YamlScalar) {
      wrapAsYamlNode(value); // assert valid scalar
      return value;
    } else if (value is YamlList) {
      for (final item in value.nodes) {
        wrapAsYamlNode(item); // assert valid scalar
      }
      return value;
    } else if (value is YamlMap) {
      /// Both [entry.key] and [entry.values] are guaranteed to be [YamlNode]s,
      /// so running this will just assert that they are valid scalars.
      for (final entry in value.nodes.entries) {
        wrapAsYamlNode(entry.key);
        wrapAsYamlNode(entry.value);
      }
      return value;
    } else if (value is Map) {
      return wrapAsYamlNode({
        for (final kv in value.entries)
          wrap(kv.key, depth + 1): wrap(kv.value, depth + 1),
      }, collectionStyle: styleMap(value, depth));
    } else if (value is List) {
      return wrapAsYamlNode({
        for (final v in value) wrap(v, depth + 1),
      }, collectionStyle: styleList(value, depth));
    } else if (value is String) {
      return wrapAsYamlNode(value, scalarStyle: styleString(value, depth));
    } else {
      return wrapAsYamlNode(value);
    }
  }

  return wrap(value, 0);
}

int _sizeOfScalar(dynamic value) => value == null ? 4 : '$value'.length;

CollectionStyle _defaultMapStyle(Map map, int depth) {
  if (map.values.any((value) => value is Map || value is List)) {
    return CollectionStyle.BLOCK;
  }
  final size = map.entries.fold<int>(
    0,
    (sum, entry) => sum + _sizeOfScalar(entry.key) + _sizeOfScalar(entry.value),
  );
  if (size < 80) {
    return CollectionStyle.FLOW;
  }
  return CollectionStyle.BLOCK;
}

CollectionStyle _defaultListStyle(List list, int depth) {
  if (list.any((value) => value is Map || value is List)) {
    return CollectionStyle.BLOCK;
  }
  final size = list.fold<int>(
    0,
    (sum, value) => sum + _sizeOfScalar(value),
  );
  if (size < 80) {
    return CollectionStyle.FLOW;
  }
  return CollectionStyle.BLOCK;
}

ScalarStyle _defaultStringStyle(String string, int depth) {
  if (string.contains('\n')) {
    return ScalarStyle.LITERAL;
  }
  if (string.length > 80) {
    return ScalarStyle.FOLDED;
  }
  if (!string.contains('\'')) {
    if (!string.contains('"')) {
      return ScalarStyle.PLAIN;
    }
    return ScalarStyle.SINGLE_QUOTED;
  }
  return ScalarStyle.DOUBLE_QUOTED;
}

Maybe, there needs to be some logic around depth... I suppose it would be nice with an argument that always goes into flow-mode for a depth higher than some threshold, a reasonable default might be 16 (at that point block-mode becomes pretty sketchy to read, I'm just guessing).

I need to try this mr Jonas @jonasfj

@seifibrahim32, please do give it a try.
I think one of the important aspects is to have proper tests.
Such that we know that the formatting works, and behaves reasonably.
Perhaps, it'd be a good idea to start with https://github.com/dart-lang/yaml_edit/issues/33 (I just filed it, hehe)
But really, adding the functions above is probably not too hard. The hard part is making sure we have tests, so that we're reasonably confident that these things work.

I will take this try please...again, Google Summer of Code took me time for proposal. my apologies.
@jonasfj

Let me take a try :))

Thanks for assignment , let me check the code and I reply you within tomorrow to see how we would make flutter_test :D

My first approach : Understand wrapAsYamlNode how it works
Then the second approach : Try to use the wrapAsCustomedYamlNode for that.
Then the third add test cases and refactor the code..

@jonasfj According to the issue #33 , We will output .golden files from *.test files for checking test cases.

@jonasfj I will add some tests if there are \n as mentioned in #33

This function is better since If we used the normal wrapAsYamlNode it wont handle spaces \n or either indentation for scalars.

We need to handle some tests so it clears this method's cases.


version 4.8.0，run “nohup sh bin/mqnamesrv &“ error when in dash env

runserver.sh script line number 70 use "[[" which is not support in dash env

Could you submit a pull request to fix the issue?

Could you submit a pull request to fix the issue?


some display problem when connect to a wrong ip

I connect to a wrong ip, and the console print
"xxxx successfully"
It's meaningless to show the internal pool is created successfully (unless in a debug mode).
Just tell the user if the connection is OK or Error.

I connect to a wrong ip, and the console print
"xxxx successfully"
It's meaningless to show the internal pool is created successfully (unless in a debug mode).
Just tell the user if the connection is OK or Error.

Ok, I'll fix it.


Add useDeleteButtonTooltip property for Chip

Description
This PR adds a new property for Chips which have a "delete" button, i.e., extend DeletableChipAttributes (Chip, InputChip, and RawChip) named useDeleteButtonTooltip, which enables the user to optionally disable the tooltip for the chip's delete button by useDeleteButtonTooltip: false.
When unspecified, the default value of useDeleteButtonTooltip is true.
Related Issues
Fixes #67944.
Tests
I added the following tests:

Test for checking whether the tooltip for the delete button of the chip can be disabled.

Checklist
Before you create this PR, confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.

@RoyARG02 I've removed the "work in progress" label, since it looks like this PR is done, but I won't commit it until  you give the go-ahead.

Sure, I'm all done from my side.


IRGen: force lazy metadata initialization for PE/COFF

The metadata reference to the parent type cannot be supported on PE/COFF
due to the direct reference to a value in an external module that is in
static data (the model requires indirecting through memory).  Always
force the lazy initialization for the metadata on such platforms.
This requires a secondary change - to initialize the VWT as well.  This
is ideally moved into the runtime where we can do this uniformly.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

CC: @slavapestov

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 7f51835f9ced8bebb88a195aa2420ff4500bad7f

Build failed
Swift Test Linux Platform
Git Sha - 7f51835f9ced8bebb88a195aa2420ff4500bad7f

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 060a46eea3e6212755e2ba956e21f54842797510

Build failed
Swift Test OS X Platform
Git Sha - 060a46eea3e6212755e2ba956e21f54842797510

@swift-ci please test macOS platform

Nitpick: please don't use the term "parent type" since that refers to an AST concept (Foo is the parent type of Foo.Bar). Let's just say that external references to data symbols are not supported, so we cannot directly reference the NativeObject value witness table, and leave it at that.

@compnerd There are other cases where we emit direct references to external value witness tables. One example is structs with certain known layouts, eg,
struct Foo { var x: Int8 }

@slavapestov - hmm, I'm not seeing that in my current build.  Compiling this for windows x86_64 seems to be fine:
struct S {
  var i: Int8
}

@slavapestov IIRC we already address those cases by disabling the optimization of using preexisting value withness tables on Windows.

The Windows build is currently broken anyways due to the use of the NativeObject VWT for AtomicInt in SwiftPrivate.  So, I'm going to go ahead and merge this.  The only thing that really remains now is the initialization of the VWT field as well.  I intentionally left out the emission of the placeholder for the VWT for this part of the change so that we can decouple it.

(Updated the commit message for @slavapestov)

@swift-ci please test and merge

@swift-ci please test and merge

@swift-ci please test macOS platform

Build failed
Swift Test OS X Platform
Git Sha - 1f56fed588100cf073da825ada547f04985ea728


Plat 2735 concat logic

Sorry @skynet67 , merge conflicts now with all the refactoring... once you correct those, feel free to merge this change down.


Update BE types

#3057  👈
main

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

Join @nickgraz and the rest of your teammates on  Graphite


Error on kvctl.sh

See this error
root@ubuntu:~/golang/src/github.com/youtube/vitess/examples/kubernetes# ./kvtctl.sh InitShardMaster -force test_keyspace/0 test-0000000100
Starting port forwarding to vtctld...
Error: flag needs an argument: --template
Run 'kubectl help' for usage.

Hm it seems like somehow this command is getting cut off after --template:
https://github.com/youtube/vitess/blob/master/examples/kubernetes/env.sh#L30
Are you perhaps using a custom script to run kubectl, for example by setting the KUBECTL env var?
Can you show the output of kubectl version?

@enisoc :
root@ubuntu:~/golang/src/github.com/youtube/vitess/examples/kubernetes# kubectl version
Client Version: version.Info{Major:"1", Minor:"0", GitVersion:"v1.0.1", GitCommit:"6a5c06e3d1eb27a6310a09270e4a5fb1afa93e74", GitTreeState:"clean"}
Server Version: version.Info{Major:"1", Minor:"2", GitVersion:"v1.2.4", GitCommit:"3eed1e3be6848b877ff80a93da3785d9034d0a4f", GitTreeState:"clean"}

It looks like your kubectl client is version 1.0.1, which is pretty old. Can you try to update it to 1.2.x? How you do that will depend on how you installed kubectl in the first place.

Worked thanks!


fix: Custom endpoint config field

Description of changes
#1036 inadvertently broke the endpoint field on client configuration.  This PR restores it, and verifies it with a couple tests.
New/existing dependencies impact assessment, if applicable
No new dependencies were added to this change.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/test


[BUG] [Input] Postfix shown on top of input

🐞 Bug report
Description
Dollar sign is shown on top of input in Group component demo.
Reproduction

Open group component demo
Try typing some number in "How much?" input which has dollar sign postfix.

Expected behavior
Dollar sign should not appear on top of input.
Additional context

@hakimio this issue is linked to:
https://github.com/TinkoffCreditSystems/taiga-ui/issues/416
At the moment, there's no way to show postfix and right-align controls. In the original design, currency symbol is supposed to go to the table heading for the corresponding column. If you want to close this issue, you can change demo accordingly :)

Removed this section from demo.


Implement multiprocessing

Using knowledge from NLeSc training on multiprocessing:

[x] implement parrallelisation on one machine: i.e. run the update step for all ensembl members all at once:
in Ensemble.update()  currently runs serially:
 for ensemble_member in self.ensemble_list:
        ensemble_member.update()

Update to parrallel should be fairly quick as already formated such that jobs are split.
example:
from tqdm.contrib.concurrent import process_map
# If you want threads instead:
# from tqdm.contrib.concurrent import thread_map
import time

 args = range(5)

 def square(a):
     time.sleep(a)
     return a*a

 result = process_map(square, args, max_workers=2)

[x] Implement multiprocessing so that update step occurs on many machines, main usecase is large distributed models:

EnsembleMember class then acts as wrapper for the model on another instance.
Acess in what cases this is beneficial: i.e. won't be for most lumped models.

Then also add example to main ewatercycle package: https://github.com/eWaterCycle/ewatercycle/issues/78

using dask seems a better approach as explained [here] (https://ewatercycle-test-parallelisation.readthedocs.io/en/latest/example_parallelisation.html)

Will now add this into the package by default

added in a841c75 and works in testing


Perow hab con site names

Perow Creek | 1801071_us is a ford I believe and probably should be 197653_us

There is also a hab con site that you and Vern did that you called 197653_us2. I think I remember what's going on now. We split up that habitat assessment. Tieasha and I did 500m upstream of the ford and you and Vern did 150m and we met up in the middle. So I should combine the data from both cards into one entry for 197653_us instead of two entries, correct?

Could leave it as two sites for now.  We have ways to deal with that in the reporting because it is not uncommon to survey more than one site upstream and down of the same crossing


Include redirected items when parsing headers.

Undocumented Apache annoyance. In some cases when using mod_rewrite REDIRECT_ prefix is added to the headers.  This happens for example when using workaround to make HTTP Basic Authentication work together with FastCGI.
RewriteRule .* - [env=HTTP_AUTHORIZATION:%{HTTP:Authorization}]
Currently Slim does not include these items when parsing the headers. Only way to access them is through $_SERVER superglobal.

This needs to be refactored against the latest code. You can find the relevant header parsing code in \Slim\Http\Headers::createFromEnvironment().


Allow uploading test results to the Boost regression testing matrix

Fixed by #187; the test results will be uploaded automatically when Hana is added as a submodule of the master Boost repo.


Contravariance stdlib exercises need some love

We have removed the contravariance related exercises because they were originally expressed with mutability. The exercises need to be reworked to demonstrate how contravariance works in a more principled way than using mutability.

Issue migrated to the stdlib exercises repo.


in registration controller: self.resource is not the same thing that view will see

Environment

Ruby 2.4
Rails 6
Devise master

Current behavior
Having user model
https://github.com/heartcombo/devise/blob/715192a7709a4c02127afb067e66230061b82cf2/app/controllers/devise/registrations_controller.rb#L102-L104
After build_resource(sign_up_params) is called with sign_up_params = { first_name: "asdfasdf", last_name: "asdfasdf" }
calling resource in controller will result in User<first_name: "asdfasdf", last_name: "asdfasdf"> returned
but calling resource in controller will result in User<first_name: "", last_name: "">
Expected behavior
calling resource in controller will result in User<first_name: "asdfasdf", last_name: "asdfasdf"> returned
but calling resource in controller will result in User<first_name: "asdfasdf", last_name: "asdfasdf">
how to fix
currently
https://github.com/heartcombo/devise/blob/715192a770/app/controllers/devise_controller.rb#L37-L40
  # Gets the actual resource stored in the instance variable
  def resource
    instance_variable_get(:"@#{resource_name}")
  end

expected
  # Gets the actual resource stored in the instance variable
  def resource
    instance_variable_get(:"@#{resource_name}")
  end

  def resource=(x)
    instance_variable_set(:"@#{resource_name}", x)
  end

it's there already https://github.com/heartcombo/devise/blob/715192a7709a4c02127afb067e66230061b82cf2/app/controllers/devise_controller.rb#L94


Where are the release notes?

I can’t seem to find any information about the latest major releases of this package.
Is there a release notes section or a changelog in another location that I missed?
What are the breaking changes in version 15 and 16?

There can be minor breaking changes in formatting and linting as dependencies like prettierx and eslint (and eslint plugins) were updated. Otherwise there are no major changes.
I've added release: https://github.com/sheerun/prettier-standard/releases/tag/v16.2.1


User defined table inserting wrong value

When using the values (0.05 or 0.01) the query generator shows the values (5 or 1) when these are used within a SqlDbType.Structured SqlParameter.
I think this is best explained with code...
Currently the produced code is:
declare @p3  dbo.someUDT
insert into @p3 values(NULL,5)
insert into @p3 values(5,0.10)
insert into @p3 values(NULL,1)
insert into @p3 values(1,2)

exec sp_executesql N'Select * from @AName',N'@AName  [dbo].[someUDT] READONLY',@AName=@p3

While the correct code would be:
declare @p3  dbo.someUDT
insert into @p3 values(NULL,0.05)
insert into @p3 values(0.05,0.10)
insert into @p3 values(NULL,0.01)
insert into @p3 values(0.01,0.02)

exec sp_executesql N'Select * from @AName',N'@AName  [dbo].[someUDT] READONLY',@AName=@p3

Steps to reproduce
This is some standalone code to reproduce the issue
The User defined table definition:
CREATE TYPE [dbo].[someUDT] AS TABLE(
   [value1] [decimal](16, 5) NULL,
   [value2] [decimal](16, 5) NULL
)

some C# code, the context that is used isn't of importance for this issue.
DataTable dataTable = new DataTable();
dataTable .Columns.Add("value1", typeof(decimal));
dataTable .Columns.Add("value2", typeof(decimal));

dataTable .Rows.Add(null,0.05);
dataTable .Rows.Add(0.05,0.1);
dataTable .Rows.Add(null,0.01);
dataTable .Rows.Add(0.01,0.02);
List<SqlParameter> Parameters = new List<SqlParameter>();

Parameters.Add(new SqlParameter("@AName", SqlDbType.Structured) { Value = dataTable , TypeName= "dbo.someUDT" });

dbContext.Database.ExecuteSqlCommand("Select * from @AName",Parameters.ToArray());

Further technical details
EF version: 6.3
Database Provider: EntityFramework.SqlServer
Operating system: Windows 10)
IDE: Visual Studio 2017 15.6

I made a stackoverflow question about this issue and the user "Greg" figured it out.
This issue is not related to entity-framework.
Based on his answer I found an exciting bug report here:
https://feedback.azure.com/forums/908035-sql-server/suggestions/38357785-sql-server-profiler-textdata-column-handles-decima
The stackoverflow question:
https://stackoverflow.com/questions/59152654/user-defined-table-in-entity-framework-generating-incorrect-query


Move registry creation to @BeforeEach in SampleTestRunner

Simplifies the meter and observation registries management in SampleTestRunner

Add "create[Meter|Observation]Registry" protected methods
Initialize registries in @BeforeEach
Remove the constructor that takes registries
Each parameterized test creates a new registries
Remove code to track used registries and @AfterAll to close them
Remove TestConfigAccessor

LGTM however I'd like @violetagg to take a look at this branch (if possible :pray: ) with what she has there in Reactor Netty.

LGTM however I'd like @violetagg to take a look at this branch (if possible 🙏 ) with what she has there in Reactor Netty.

I'll check it

@ttddyy @marcingrzejszczak Do we need to recreate the registry for every test?
In Reactor Netty we have
	@BeforeAll
	static void setUp() throws CertificateException {
		...

		registry = new SimpleMeterRegistry();
		Metrics.addRegistry(registry);

		...
	}

	@AfterAll
	static void tearDown() {
		Metrics.removeRegistry(registry);
		registry.clear();
		registry.close();
	}

In Reactor Netty we have

So in your cae you would override the createMeterRegistry method where you will essentially do what you do in your setUp method

Also the change is breaking change :)

That is not possible because we typically do those just after the release ;) yeah... that's a breaking change

LGTM ;)


added new chest chest

This chest gives you another chest... So useful

Please add Formatting with BOLD

done


Don't encourage binary string use

as stated: https://developer.mozilla.org/en-US/docs/Web/API/FileReader/readAsBinaryString

This feature is non-standard and is not on a standards track. Do not use it on production sites facing the Web: it will not work for every user. There may also be large incompatibilities between implementations and the behavior may change in the future.
This method has been removed from the FileAPI standard. FileReader.readAsArrayBuffer() should be used instead.

It's not even supported in IE10
Javascript is bad at handling binary strings. One character is a 16-bit so it means a binary string can take up to 2x more memory since it's not in utf-8
What you should do in the readme is changing the examples to use arrayBuffer/typed arrays or just accept plain blob objects

This [readAsBinaryString] method has been removed from the FileAPI standard. FileReader.readAsArrayBuffer() should be used instead.
Source: https://developer.mozilla.org/en-US/docs/Web/API/FileReader/readAsBinaryString

@jimmywarting @miclill there was probably a reason for preferring readAsBinaryString, most likely performance.  The demo supports both modes.

One character is a 16-bit so it means a binary string can take up to 2x more memory since it's not in utf-8

The binary string is a direct mapping of bytes (if s is the string, s.charCodeAt(i) is the i-th byte), so UTF8 conversion is not involved.  The string length will be the exact byte size of the uploaded file.

What you should do in the readme is changing the examples to use arrayBuffer/typed arrays or just accept plain blob objects

I agree that it would be helpful to add more examples like the readAsArrayBuffer form.

I have a 2010 tutorial that used readAsBinaryString in my bookmarks.  When we added readAsArrayBuffer support, rABS was noticeably faster.


Update linting to SwiftLint 0.16.1

Enable more opt-in and custom rules
Re-enable some disabled rules now that they are more customisable

@madhikarma The comments/marks custom rules? Or the file header rule?

File header Note. For client projects i.e. Non OSS the company name would have to be changed but for ustwo OSS I think it's ok

@madhikarma The only reason I would be against adding it to our default .swiftlint.yml template would be that it doesn't work out-of-the-box. It needs to be configured with the project's name as that's hard-coded. But I would support adding it as a recommendation in our OSS documentation.


Fix makefile typo

Fix makefile issues which cause it to look the database folder at the wrong place

Assuming you've extracted the archives from oracle.com and done cd docker-oracle12c according to Build section from README, the database folder must be located in ../database.

ah, okie, thank you


Link to website has wrong url

currently the URL it takes you to is https://tenderowl/work/Frog when it should take you to https://tenderowl.com/work/frog/ (just forgot to add .com)

Thank you!


Simultaneous computation of per-sample gradient and per-batch gradient

Hi. Thanks for providing functorch, one of most effective lib.
Actually i am working on my code which needs an access into both per-sample gradient in mini-batch and per-batch gradient, which we call as gradient in normal.
Having said that, i think that functorch now needs forward-pass twice to get both gradients. In other words, we cannot get per-batch gradient from the per-sample gradient by mean calculation, which was checked by me with empirical calculation in torch.
It leads me to forward pass twice for computing each information.
https://backpack.pt/
I think this library also provides very similar utility like per-sample gradients.
Also it provides multiple access into various information. (e.g. normal gradient, batch gradient, gradient variance, gauss-newton diagonal), which seems to be not possible with functorch.
It would be grateful if i have any way to get access into both 1) per-sample gradient in mini-batch and 2) per-batch gradient with computationally efficient way (It would be also helpful if we can also compute other information (e.g. hessian, gradient variance...) . thanks.

In other words, we cannot get per-batch gradient from the per-sample gradient by mean calculation, which was checked by me with empirical calculation in torch.

Could you try summing the per sample gradients together? That should be the same and if it's not, it would be great to have a repro since that's a pretty nasty issue

It would be also helpful if we can also compute other information (e.g. hessian, gradient variance...)

For gradient variance, I think the best way would be to compute from the per sample gradients. For hessians it gets a little more complicated but if your per sample gradients match the jacobian, we could compute the hessian and produce the jacobian along the way, like this:
def jacrev_with_aux(f, argnums=0):
  def wrapper(*args)
    out = jacrev(f, argnums)(*args)
    return out, out
  return wrapper

def hessian_with_jacrev(f, argnums=0):
  hessian, jac = jacfwd(jacrev_with_aux(f, argnums), argnums, has_aux=True)
  return hessian, jac

Let me know if that helps!


Failing to load PDB from Symbol Server

I am unable to load the PDB for one of my programs from an internal symbol server. I can't figure out why. But IDiaDataSource::loadDataForExe is failing.
SYMSRV:  HttpQueryInfo(HTTP_QUERY_CONTENT_LENGTH): 800C2F76 - ERROR_HTTP_HEADER_NOT_FOUND
SYMSRV:  HttpQueryInfo: 80190190 - HTTP_STATUS_BAD_REQUEST
SYMSRV:  HTTPGET: /MYSERVERPATH/Foo.pdb/42A6E6342E735EB74C4C44205044422E1/Foo.pdb
SYMSRV:  HttpQueryInfo: 801900c8 - HTTP_STATUS_OK
SYMSRV:  RESULT: 0x00000000
SYMSRV:  BYINDEX: 0x2
         MYSERVERPATH
         Foo.pdb
         42A6E6342E735EB74C4C44205044422E1
SYMSRV:  HTTPGET: /MYSERVERPATH/Foo.pdb/42A6E6342E735EB74C4C44205044422E1/Foo.pdb
SYMSRV:  HttpQueryInfo: 801900c8 - HTTP_STATUS_OK
SYMSRV:  RESULT: 0x00000000

What's weird is that 42A6E6342E735EB74C4C44205044422E1 is the correct hash and Visual Studio is able to correctly load this pdb from my symbol server. I have a tool that runs SymInitialize on a running process and it also able to load the pdb.
I've never used IDiaDataSource before and don't know how it works. Stepping through the callbacks nothing stood out to me. It's ultimately returning E_PDB_NOT_FOUND even though
I also verified that I can download Foo.pdb simply by pasting the full URL in my browser. So it's the right URL. It's the right PDB. But loadDataForExe is failing? That's super super weird.

I just learned about symchk.
symchk.exe PATH/to/Foo.exe /s srv*MYSERVERPATH

Symchk is succeeding. And it fails if I do not specify the symbol server. I have no idea why loadDataForExe is not. Very strange.

Yeah.  It's super frustrating and annoying.

duplicate of #97


Test Java Driver QueryBuilder private build, 4.19.0-preview1

Checklist

[x] Changes manually tested
[x] Automated Tests added/updated
[x] Documentation added/updated
[x] CLA Signed: DataStax CLA

I think this is failing due to missing .sha1 files for checksums. So, basically if there is:
jackson-core-2.18.0.jar

there needs to be matching
jackson-core-2.18.0.jar.sha1

(and same for .pom files)


GetObject resolves to GetObjectA/GetObjectW in Windows world

If windows.h was included before related SDK header GetObject gets substituted (via define) to GetObjectA (or GetObjectW, if UNICODE is defined). This leads to curious linker errors, frustration and time spent chasing this down.
I understand this is not really SDK problem, but it would be real nice if every GetObject declaration was surrounded by something like this:
#pragma push_macro("GetObject")
#undef GetObject        // damn you, Bill Gates!
...
#pragma pop_macro("GetObject")

I agree. Unfortunately, that won't solve the problem since this often surfaces when you try to use GetObject in your own code. Do you have further recommendations on how to address that as well?
Sent from my iPhone

On Feb 23, 2017, at 11:48 PM, crusader-mike<EMAIL_ADDRESS>wrote:
If windows.h was included before related SDK header GetObject gets substituted (via define) to GetObjectA (or GetObjectW, if UNICODE is defined). This leads to curious linker errors, frustration and time spent chasing this down.
I understand this is not really SDK problem, but it would be real nice if every GetObject declaration was surrounded by something like this:
#pragma push_macro("GetObject")
#undef GetObject        // damn you, Bill Gates!
...
#pragma pop_macro("GetObject")
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Yes, but with this change I'll know exactly what is wrong and where. (it will fail at compile time and will show me the line). Without -- need to hunt down chains of includes.
Best solution -- rename GetObject to smth else.

Im not renaming GetObject just because MSFT couldn't figure out how to use unicode. The least they could have done was namespace their function name. Anyways, I'll look into your suggestion.
Thanks,
Sent from my iPhone

On Feb 24, 2017, at 5:13 AM, crusader-mike<EMAIL_ADDRESS>wrote:
Yes, but with this change I'll know exactly what is wrong and where. (it will fail at compile time and will show me the line). Without -- need to hunt down chains of includes.
Best solution -- rename GetObject to smth else.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

if this SDK has common prologue/epilogue in every header -- undefs can go there. Slightly better solution, but yeah -- MS dropped the ball on this one.

every header has our dll export macros, we can probably drop it in there since that only gets defined for windows anyways.


Native render pass v2

Closes #116
Currently crashes on 2022.3, works fine on 2021.3.

MSAA preset has an expensive call right before rendering UI.

Running with MSAA preset causes an extra copy to frame buffer step.


Add randomly generated tests

This adds 1000 random tests with random amounts of decimal places and code lengths.

How do I create a map for 217 location with one address?   Beginning to end basics
Patsy Clark
Snap Realty
DBA/ Snap It Up Location Data Management
www.SnapItUp.Tech
919-971-7030
See what I've posted to Google Maps
Level 8 Local Guide
https://goo.gl/maps/CSYkKPFhhNz
Google Business Listings and MLS Listings with GPS 📍 Embedded  💫 360° ©Photography Integrated with your Google and Google Business Listing That Disseminates to All Map and Business Platform  🧭
Licensed, Bonded and Insured
A Division of Snap Realty
Snap Realty, LLC
Principal Real Estate Broker & Owner

On Apr 27, 2019, at 11:23 AM, Doug Rinckes<EMAIL_ADDRESS>wrote:
This adds 1000 random tests with random amounts of decimal places and code lengths.
You can view, comment on, or merge this pull request online at:
https://github.com/google/open-location-code/pull/306
Commit Summary
Add randomly generated tests
File Changes
M test_data/encoding.csv (1004)
Patch Links:
https://github.com/google/open-location-code/pull/306.patch
https://github.com/google/open-location-code/pull/306.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Can you please verify one of the code length 15 examples by hand. Using a calculator with sufficient digit precision.

I think I'm going to pause this for the moment. A higher priority is getting the CI tests working and then I may add more test cases.

@drinckes got it can you confirm that the testing on C & CPP is the right approach now?
https://github.com/google/open-location-code/pull/599
Or are further changes needed on those two implementations before we fix the other ones

Lots of issues in the tests generated. There's lots of boundary points, so we're getting variations in codes due to floating point handling in different languages (even on the same hardware).
The existing Java implementation seems to think that the minimum valid code length is 4 (spec says 2) (#667).
I'm going to generate a set that encode, then we'll see what fails on e.g. M1 mac I guess.

NB: Blocked by #667

Good notes
Actually, only two floating point operation are required. It's a modulation (if needed) then multiplication.
This approach is spelled out in your favorite PR.
I could try this in the C implementation. Just because I'm guessing that is the canonical one since the test runs first.
But if I'm going to work on an implementation, to be honest, I would want to get my hands dirty and do a rewrite or fix a few things while I'm working on it.


move prismic content model into this repo

Type
🚑 Health
Moving the content model here as it makes way more sense.

Spoke to @gestchild via Slack about this, as it's not application affecting, I'm merging as I need it for some other work I am doing.


Feature/time series states

This tracks the time that a job changes state on a hourly granularity in a zset with key 'jobs:' + state + ':' + (Math.floor(Date.now() / 3600000) * 3600000) where (Math.floor(Date.now() / 3600000) * 3600000) is just a timestamp that is a "floored hour"
This also provides a json api to access the time series data using the autoroute upgrade in this PR https://github.com/Automattic/kue/pull/792
There is a test that copies the same style as other JSON API tests to verify that it works.
Let me know if you have any questions and I can go into a more full explanation.

This is forked from https://github.com/Automattic/kue/pull/792 so it shouldn't be merged unless https://github.com/Automattic/kue/pull/792 is merged first. If https://github.com/Automattic/kue/pull/792 gets rejected I can undo the autoroute portion of this PR :+1:

Ok so I've made 2 changes

I've changed the new API endpoints to follow json:api because I'm consuming them in the kue-ui alternative ui
I've moved the new route files into a v2 api because they are so significantly different to the old APIs. This also incidentally shows one of my other justifications of using express-autoroute https://github.com/Automattic/kue/pull/792 i.e. you just need to move a route file into a v2 folder and it gets the v2 prefix

You can see my kue-ui PR that actually makes use of the new endpoints here https://github.com/StreetHub/kue-ui/pull/21 or you can just have a look at the small video I created to illustrate my point:

Thank you for your efforts @mansona
I'll review this ASAP

hey @behrad thanks :smile: I really appreciate you taking the time, if you have anything that you think I could improve on just fire away! I'm not afraid of a bit of criticism :smirk:
I've just pushed a quick update there that adds error text to the job if it exists :+1:

Hey Folks, any more thoughts on this?
I have been using this in production for quite some time now and it works beautifully. It has been a real help to figure out issues with the jobs on live.
Let me know if there is anything you would need from me to get this merged, if you would like a discussion about it we could arrange a call if you like.

I need time to review the code and see how can we merge it in beautifully without mess, Thank you for the nice work @mansona .

@behrad well if you want help on the merge strategy just let me know. This is the first step of exposing some of the data required via JSON-API and I have a few ideas on how to maintain the API and roll out new features 👍
Not trying to rush you, I know you're probably mad busy 😄

Before merging this @mansona , Can we omit .editorconfig from the commit and add tests for new API?

hey @behrad sorry I missed your comment. I have removed editorconfig for now, we can discuss adding it back later with your own settings if you like.
re: the tests I'm really not sure how to progress with that. I have a particular way that I produce API tests but it won't be in keeping with the style of your current tests and I also don't know how to best setup fixtures (real kue objects or just mocks/stubs etc) to make them a valid test.
I would propose that we merge this PR because it currently doesn't break any of the current functionality and setup a meta issue for discussion of the new API testing. Either that or we could arrange a Skype if you were interested.
How does that sound?

Hey folks, just letting you know I've synced this PR with master. I noticed it could not longer be automatically merged. 👍

Hey Folks, Just letting you know that I've taken over the Kue-UI repo and I'm currently fielding quite a few requests from end users looking for progress on this PR. I know you're probably super busy but could we put a date in the diary to have a Skype call, maybe in September to get this moving?
If there is anything I can do to make this any easier please let me know 👍

hey @behrad I think you may have closed this as a typo? can you re-open this PR so we know it's still an ongoing concern?

Cool sounds like a good plan forward, would you like me to help with just separating the current Web API into a module (without any changes) and we can the create a new branch off that?

Thats great, we can plan for a clean HTTP API (may be also socket API) on top of which a UI can be written in another new module.
Are you ok with a long term support on that module?

well we should probably have a skype conversation about these plans, essentially I've taken over https://github.com/stonecircle/kue-ui which is already a "UI in a separate module".
In principle I am happy to maintain the API module because I'm already maintaining something that will be consuming that API.

Good,
We can

create a new HTTP/Socket API module
Refactor https://github.com/stonecircle/kue-ui to use 1
Drop current kue web ui in favor of https://github.com/stonecircle/kue-ui

Have you commit right to https://github.com/stonecircle/kue-ui ? It seems no active recently.

great! that sounds like a brilliant plan 👍
Essentially I "own" kue-ui right now (I'm the director of Stone Circle). Because of my work on this PR: https://github.com/stonecircle/kue-ui/pull/21 which depends on the JSON:API work above the previous maintainers ended up handing it over to me.
The reason why there has been no activity lately is because that PR I linked to is essentially a "rewrite" of the kue-ui so that it is now just an ember app instead of a proxy server that hosts the ember app. I've already been working to make this setup more modular 😉
Now we just need to get the ball rolling on your HTTP module idea. Where is best to organise this? Should I create a new meta issue and we can keep our discussions there?


Findbugs aggregate report

Just an FYI, I did find a way to make a find bugs aggregate report.
https://github.com/spyhunter99/FindBugsAggregate
I just prints a table of the counts of all child modules and links to the actual report

@spyhunter99 checkstyles maven plugin aggregates today (somewhat anyways).  So wouldn't it be possible to build this into this plugin so that at the end of the multi module run it has all the data already collected and would produce that report under the target of the main parent module?  Would you be able to create such a pull request into this plugin?  I see this as extremely useful.

ha that's why i was giving you all a heads up. The maven reporting plugin infrastructure doesn't appear to support requiring child modules to do their thing before a parent module. I'm not entirely sure how checkstyle did it.

I know theirs is not perfect but might be worth taking a look.  It seems to though some false positives on that end but is good enough I guess.  I would like to see this come in here even if it is not 100%.  The 80/20 rule is good enough for me ;)  We can work on making it 100%.

they probably run the check at the parent module and include the src of all child modules. That's find for check style, but findbugs needs the classpath of all child modules. From a maven perspective, i think that's only available during the execution of the child modules. This would mostly work in some cases but from all cases and in the end, the analysis task of findbugs would be ran twice (it's already slow)


[FLINK-33750][config] Remove unused deprecated option

What is the purpose of the change
Remove unused deprecated option. Note that this PR left some deprecated ConfigOption.

Options related to restart strategy, such as "hidden.execution.retries"
Options related to legacy memory configuration, such as "jobmanager.heap.size"
Options related to netty network shuffle
Options related to serizliation, such as "pipeline.auto-type-registration"
Options related to queryable state, QueryableStateOptions

Brief change log

Remove deprecated option in flink-core
Remove deprecated option in flink-metrics-prometheus
Remove deprecated option StreamPipelineOptions

Verifying this change
This change is already covered by existing tests.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): yes
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

@1996fanrui Thanks for the review! I updated the PR. PTAL.


[CodeChecker] Fix CI fail by parse command broken again

Turns out that now the parse command shows non-zero exit status in case reports are emitted to HTML...

Oh dear. Many many thanks @whisperity . I was already wondering. :-)


Places2 weight

@jingyuanli001 thanks for your excellent work! could you release your trained weight of places2?

@jingyuanli001 I try to train places2 with your code，however it works slow. if i trained the model with your suggested parameter(2000000) and finutune(1000000), it will cost 2 month to finish. it is too long. could you release your trained weight of places2 to help us to reduce the workload of reproduction?

Following up on this issue. It would be really great if the places2 checkpoint can be shared, so we can make a fair comparison.


When not signed in, open article or request buttons don't work on plugin dev 1.03

@meganwaps / @markmacgillivray can you try and replicate please?

@JosephMcArthur I have not been able to recreate this and have made doubly sure that I'm not signed in on the dev site. I'm able to make requests and access open articles without being signed in using dev 1.03. It takes a few moments for the buttons to load, but both work for me, using Chrome, without being signed in. @markmacgillivray were you able to replicate?

@JosephMcArthur @markmacgillivray I am still unable to replicate this issue. Any luck?

Alright! Then I'll shut it.


Student Application Updates:  Remove Interface Policy Question

Given interface policy is being disabled for MVP, we’d like to ensure that it is entirely removed from the assessment workflow (at least for now). I am not clear about how much of this is ‘already built’. It might be the case that none of this was built to begin with, and therefore all the steps listed below don’t require any action. But at a minimum we want to review to confirm.
Below is my understanding of the work required- all steps below apply equally to FullTime and PartTime
Preserve the question in application titled “Will you have B.C. income assistance for persons with disabilities that you will be receiving under the Employment and Assistance for Persons with Disabilities Act during your study period?” (We may enable
interface policy later on and so it’s useful to preserve the question)
The impact at the above question should be NOTHING in the workflow. It is purely for information-only.
SIMS should not integrate with SDPR at all
Ensure that any steps in assessment workflow referencing “MSS” fields are bypassed. They can be preserved in the workflow in case we eventually build interface policy in the future, but for now, no students should be diverted down the interface policy path of the assessment workflow and should therefore not utilize any of the ‘MSS’ fields
Acceptance Criteria

[x] Leave BC income for disabiilities question in form.
[x] Confirm that the answer to this question has no impact on the assessment workflow.
[ ] Do not calculate MSS fields and ensure it has no impact on the assessment workflow.

Closing as this was just a discussion-prompt ticket, and the investigation has been closed.


Bagit Generation should use ES to sort

The generation of the bagit requires the TSV to be ordered by specimen UUID. We currently sort that in the Python code, and should instead request that Elasticsearch return the data already sorted.
This comes from issue https://github.com/DataBiosphere/dcc-dashboard/issues/9

➤ Cricket Sloan commented:
Let's stall this  on Firecloud endpoint


Add support for all UUID versions

Sequelize is using Validator to validates UUID strings.
Validator supports 4 versions of UUID - 3,4,5 and all:
https://github.com/chriso/validator.js/blob/b59133b1727b6af355b403a9a97a19226cceb34b/lib/isUUID.js#L14-L19
This change is updating IsUUID to support all possible options - 3,4,5 and all.
see https://github.com/DefinitelyTyped/DefinitelyTyped/pull/18777#issuecomment-321458784
Other stuff
Two additional changes:

WhereOptions --> WhereOptions<any>
added as IAssociationOptionsBelongsToMany

I am not sure about any of those additional changes. I know it passed compilation and testing, but I basically choose the option that compiles.
Also running npm run build before making any change failed.  I have upgraded to node 8 and then it passed. Is there any dependency on the node version or it was just something locally on my machine?

Codecov Report

Merging #92 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #92   +/-   ##
=======================================
  Coverage   95.65%   95.65%
=======================================
  Files          66       66
  Lines         759      759
  Branches      106      106
=======================================
  Hits          726      726
  Misses         11       11
  Partials       22       22

Impacted Files
Coverage Δ

lib/annotations/association/BelongsToMany.ts
100% <100%> (ø)
:arrow_up:

lib/annotations/validation/IsUUID.ts
100% <100%> (ø)
: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 84a0538...a24fd7c. Read the comment docs.

Looks good, thank you for implementing that.
One question: You've updated @types/sequelize to  4.0.68. Was this a required step to achieve this feature?

@RobinBuschmann yes. It needed this change:
https://github.com/DefinitelyTyped/DefinitelyTyped/pull/18777

@RobinBuschmann build and tests have failed on my machine. I tried the following

clone
npm install
npm run build

Got a failure (Don't remember what it was).
Upgraded to node8, ran again. This time it was successful. Is there any dependency on the node version?
Then I have tried to run:
npm run cover
got an error, asking to install sqlite3 manually. So I ran:

npm install sqlite3
npm run cover

This time it worked.
I can open an issue for this, but wanna know if you have any ideas.

@RobinBuschmann do you think this can be pushed?


Make travis builds fail reliably

The build was failing to fail because bash error handling is not
intuitive.

This PR made the tests fail, which means the fix worked. LGTM 😛


Add support for tvOS

Thanks for the PR. This is already available however, on the xcode_7.1 branch. tvOS requires Xcode 7.1 which is still in beta, so it can't go on the master branch yet.


Migrating a vmware VM of RHEL7 to kubevirt , VM startup failed with error dracut-initqueue timeout

What happened:
A clear and concise description of what the bug is.
Migrating a vmware VM of RHEL7 to kubevirt , VM startup failed with error dracut-initqueue timeout.
[186.175174] dracut-initqueue[268]: Warning: dracut-initqueue timeout - starting timeout scripts
[186.175675] dracut-initqueue[268]: Warning: Could not boot.

What you expected to happen:
A clear and concise description of what you expected to happen.
How to reproduce it (as minimally and precisely as possible):
Steps to reproduce the behavior.
Additional context:
Add any other context about the problem here.
Environment:

KubeVirt version (use virtctl version):  v1.1.1
Kubernetes version (use kubectl version): v1.28.2
VM or VMI specifications: N/A
Cloud provider or hardware configuration: N/A
OS (e.g. from /etc/os-release): CentOS Stream release 8
Kernel (e.g. uname -a): 4.18.0-338.el8.x86_64
Install tools: N/A
Others: N/A

This is the description of this vm.
apiVersion: kubevirt.io/v1
kind: VirtualMachine
metadata:
  name: oracle-succdb-55a223
  namespace: vlan55
  labels:
    kubevirt.io/os: rhel7
  annotations:
spec:
  runStrategy: Manual
  template:
    metadata:
      labels:
        kubevirt.io/os: rhel7
      annotations:
        ovn.kubernetes.io/ip_address: <IP_ADDRESS>
        ovn.kubernetes.io/mac_address: 00:50:56:b7:15:08
    spec:
      domain:
        devices:
          useVirtioTransitional: true
          disks:
          - disk:
              bus: virtio
            name: rootdisk
        memory:
          guest: 8Gi
        resources:
          limits:
            cpu: "2"
            memory: 8Gi
          requests:
            cpu: "0.1"
            memory: 100Mi
      terminationGracePeriodSeconds: 180
      volumes:
      - persistentVolumeClaim:
          claimName: oracle-55a223-disk1
        name: rootdisk

I use the command virsh reset --domain vmtest in pod of the vm, and then enter the rescue system. to exec the command dracut -f. reboot the machine.
That is ok!


Compatibility matrix for Windows Server 2022

Looks like File recovery for Windows Server 2022's Drive configured with storage pool is not compatible on Windows 10.

@euqinomnez
It would be great if you could add a link to the documentation you are following for these steps? This would help us redirect the issue to the appropriate team. Thanks!!

@euqinomnez
Could you please provide documentation which you followed?

Here
https://learn.microsoft.com/en-us/azure/backup/backup-azure-restore-files-from-vm#step-3-os-requirements-to-successfully-run-the-script
On Fri, May 17, 2024 at 4:26 PM PavanPesala-MSFT @.***>
wrote:

@euqinomnez https://github.com/euqinomnez
Could you please provide documentation which you followed?
—
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/122509#issuecomment-2117017298,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUQNLJ65XDPVPJB54R646ELZCW5LHAVCNFSM6AAAAABHWPS5O2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJXGAYTOMRZHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@euqinomnez
Thanks for your feedback! We will investigate and update as appropriate.

@euqinomnez, thank you for the feedback.
@AbhishekMallick-MS, could you please review this feedback and share your thoughts. Thank you

@euqinomnez - We already  have this documented, storage spaces aren't compatible with ILR (file level recovery). You can find this information in this section. As no document updates are needed, closing this thread. Thanks. #please-close


jitsi links to 404

in the chat window the menu secure chat->what's that? links to https://jitsi.org/GSOC2009/OTR.
that's a 404.

This should be https://desktop.jitsi.org/GSOC2009/OTR

@damencho Please set up a redirect on the new site. There's plenty of installations out there with the existing link

Ok, make sense, will fix it today.

Fixed.


add block arrivals report base + methodology

Description
Adds the basic report for Ethereum's block arrival times. Includes also a small methodology.
PD: let me know if there are any changes that could make it better; the methodology might be a bit too "casual"?

That is a great question, but I didn't have anything in mind about time ranges for the reports. I thought about continuing with weekly reports just to have some consistency with the DHT reports (also because there are already time distributions), but I have no strong opinions on the matter.
I thought about making the "report" as more template-looking as possible, but it could be adjusted once it's automated.


ARROW-9086: [CI][Homebrew] Enable Gandiva

@github-actions crossbow submit homebrew-cpp

Just a heads up, homebrew nightlies are failing but on-demand (like you just requested) should work, so if/when you merge this, don't blame yourself for the nightly failure--it's https://issues.apache.org/jira/browse/ARROW-8826. I haven't gotten around to dealing with it, sorry.

Thanks for the tip!


Update ont-nokia-xs-010x-q.md

Add username/password, and remove incorrect warning

@nanomad since you have it too. I leave it to you to review the PR.


Hide wVOI asset

No longer show wVOI and contractId as asset id of VOI
Definition of done

Asset id of Voi displayed as 0
wVOI no longer can be selected in token select
...

Todo

[ ] Remove wVOI from token selector
[ ] In list pages show VOI id as 0

Just need to confirm this one


Release bug: Error when updating privacy notice

Bug Description
I'm running into an issue editing privacy notices. When i try to modify a notice I get a 422 with this error message:
{
    "detail": [
        {
            "loc": [
                "body",
                0,
                "cookies"
            ],
            "msg": "extra fields not permitted",
            "type": "value_error.extra"
        }
    ]
}

Steps to Reproduce

login
click management
click about fides
ensure that privacy notices are enabled
click data map
click add systems
manually add a system
enter name and fides key
click save
click data uses
click add a data use
choose marketing.advertising.first_party.targeted
save
click privacy requests
click privacy notices
click on a notices
edit something like the tite
click save
see error in toast and in dev console

Expected behavior
The notice should save without error
Environment
https://fides-plus-rc.fides-staging.ethyca.com/
Fides Core Version: 2.15.0rc4
Fides Plus Version: 2.15.0rc9+0.ge1b1b20.dirty

moving to done

automated test updated


Change the getter names to be the field name

Closes #14. Tuple struct getters are left as-is, since their field names are invalid identifiers. This is now documented in the crate showcase.

This needs further documentation that states that the macro behaves differently for unnamed fields (tuples, tuple structs, tuple struct variants) and named fields (structs, struct variants).

Closed in favor of https://github.com/Robbepop/modular-bitfield/pull/24 for the added docs.


Implement webHook endpoint

[x] All tests pass
[x] I have run npm run doc

Description
TOKEN webHook endpoint is now optional.
You can specify secret path in options
new TelegramBot(TOKEN,{
  webHook: {
    endpoint: 'your_secret_path' //Defaults to index path
  }
});

References
https://github.com/yagop/node-telegram-bot-api/issues/585

Leaving this comment for future reference:
This approach is not as flexible as something like this which allows for finer route level flexibility and independence of setting up certificates.
So you would probably have an Caddy/Nginx server with a cert already configured, you then proxy pass it to your express web server and process updates as usual.
If this is a much needed feature, it can be re-opened, but I doubt that.


Anerdw: Watch decodefix.com

Anerdw requests the watch of the watch_keyword decodefix\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

decodefix\.com has been seen in 2 true positives, 0 false positives, and 0 NAAs.

Approved by Cow in Charcoal HQ


cmake can't find system clang on gentoo

Cmake can't find system clang on gentoo. Clang is put in a particular place on gentoo, /usr/lib/clang/6.0.1, and llvm is same /usr/lib/llvm/6.

Use -DCMAKE_PREFIX_PATH to tell cmake where to look for the libclang.


fix creating resources in vendor when plugin uses cluster

Description

when a plugin register cluster discovery in plugin's panel, it register vendor's namespace and path and when runs make:* commands it detected as the only registered namespace and path, it creates resources/page/RM in the vendor directory
example: https://github.com/lunarphp/lunar/blob/2b31dcabc4cc7f169051be63648836b75bf91d18/packages/admin/src/LunarPanelManager.php#L231
added filter to remove directories in vendor mb_strpos($resourceDirectory, base_path('vendor')) === 0
Visual changes

Functional changes

[x] Code style has been fixed by running the composer cs command.
[x] Changes have been tested to not break existing functionality.
[x] Documentation is up-to-date.

cc: @glennjacobs

Please create a test project with https://github.com/InterNACHI/modular, with a module that contains a plugin that discovers resources. Register the plugin in the app's admin provider. Then check that you can still create files within the module and they aren't stripped out (because technically the module is symlinked into /vendor, so I don't know how this is going to behave or break)

Please create a test project with https://github.com/InterNACHI/modular, with a module that contains a plugin that discovers resources. Register the plugin in the app's admin provider. Then check that you can still create files within the module and they aren't stripped out (because technically the module is symlinked into /vendor, so I don't know how this is going to behave or break)

@danharrin it seems to be working fine
the test repo: https://github.com/wychoong/filament-modular  (change your local repo)
the options when running filament:resource

dd of $resourceDirectories

str($resourceDirectory)->startsWith(base_path('vendor'))

do you think its safe to filter in  Filament\Panel\Concerns\HasComponents trait?

Can you apply this to discovered page, widget and cluster directories too,

done


[Request] Can you make a cordova plugin also?

Hey, I am a hybrid developer and I am looking for a plugin for cordova which does exactly what your library does. If you have extra time, can you make a cordova plugin? I will help in testing and publishing to npm and can help with the xml or JavaScript.
I don't know java or kotlin so I cannot do it myself. Here is the link to official cordova plugin guide https://cordova.apache.org/docs/en/latest/guide/hybrid/plugins/
If you can write the java part and put it in the repo, i can do the rest of the stuff and publish it in your name.
Thanks.

I will check it and will inform you.


linux only : 3002/coverage loads blank

Gaurav, the app isn't ready to be formally tested yet. Please don't open issues right now!!!


Loading feeds stops at Video

Hello,I tryto usethis plugin on my custom theme for WP.
URL : http://dev.escaunes.com/
Everything is almost alright, but I have 2 problems :

get : User

Issue : loading insta feed stop when I have a video.

get : Location or Tagged

I can't retrieve feed from a location, even I reistred a new client et authorize it, with a new clientId and a new accessToken.
https://www.instagram.com/explore/locations/807754699/le-domaine-des-escaunes/?hl=fr
Can you help me please ?

Hi @c0c0t3,
To address your issues:

Video support currently requires a workaround as shown here: https://github.com/stevenschobert/instafeed.js/issues/359

Permissions to query location and tag are no longer being granted by Instagram. Your queries are limited to your own sandbox feed, which is just for the user who generated the access token by default.


Improve mouse capture handling in menus

This both allows a menu to have multiple mouse capture callbacks, as well as passing the menu element that captured the mouse input as a parameter, allowing for multiple scrollbars per menu

This actually i think would break backwards compatibility for mods that use scrollbars, In theory I could solve that, but it wouldn't be nice code

Also regarding breaking mods, as far as I know mod settings is the only public mod which uses this system and it's getting merged ( right ? ) anyways so it shouldn't matter much

@GeckoEidechse this or cured my dogs cancer can we do set a new merge record? 😳😳😳

uhh all mods that use a custom scrollbar like Better Server Browser (ok grandpa), Better Modlist (deprecated by thunderstore guys), Modsettings and the LAN browser mod from that one polish guy will compile error unless updated.
For modders it's an easy fix, they just need to change their movementcapture callback function from void functionref(int,int) to void functionref(var,int,int) and that's it.
btw gecko can you please tell people in the 1.10 announcement to uninstall better modlist since it's merged thank

this is great, but should probably be moved into its own file instead of sh_menu_models if we're api-ing it

good point, will do

this is great, but should probably be moved into its own file instead of sh_menu_models if we're api-ing it

Finally got around to doing this

as of now, mods that are using mouse movement captures (only mods that use the custom scrollbar afaik) will break since they register the callback for the menu instead of the capture element. However they will not compile error but scrollbars will just stop working.
We could make a legacy kind of function that doesn't break old mods, but I don't think that's worth it since there are barely any mods that use the mousemovement capture.

This should now preserve backwards compatibility, you can now pass either a capturePanel or a menu to AddMouseMovementCaptureHandler
I'm keeping the changes to other files within NorthstarMods itself as we should prefer passing the capturePanel for use with multiple scrollbars on the same menu

Aight this is I think in it's final state, unless anyone has any changes they want made.
From my own personal testing, backwards compatibility is preserved

what's blocking this?

https://user-images.githubusercontent.com/64006268/200689529-7e8fd19e-f6f6-4f23-9e4f-8becabba490a.mp4

(just so it's known, the scrolling boxes being weird is not my fault, they are simply unfinished)


Sign package to support "strong names"

Version: 1.1.11
Dotnet Version: 8.0
We are using signed assemblies. Including the validator package returns the following warning during build
warning CS8002: Referenced assembly 'LateApexEarlySpeed.Json.Schema, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' does not have a strong name.
Please, sign the package to support strong names.

Hi @ah1101 thanks, signed package will be included in 1.1.12.

1.1.12 is published.


App has not responded to the network requests below:   (id = -1000) isReady: {}  Unresponded network requests might result in timeout errors in Detox tests.

detox[7718] ERROR: Error: Exceeded timeout of 300000ms while handling jest-circus "setup" event
detox[7718] INFO:  Example is assigned to undefined
detox[7718] INFO:  Example: should have welcome screen
detox[7718] INFO:  Example: should have welcome screen [SKIPPED]
detox[7718] ERROR: DetoxRuntimeError: Aborted detox.init() execution, and now running detox.cleanup()
HINT: Most likely, your test runner is tearing down the suite due to the timeout error
detox[7718] WARN:  [Client.js/PENDING_REQUESTS] App has not responded to the network requests below:
(id = -1000) isReady: {}
Unresponded network requests might result in timeout errors in Detox tests.
macOS version: 10.15.6
Xcode version: 11.7
detox version : 17.8.3
thanks

In my case works change connection type on emulator Setup(three dots) Settings > Proxy > No proxy
testing on android

in my case, i have to drop react-native-splash-screen and react-native-progress

I had the same issue in iOS release build event it was working in debug build
my current react native version "react-native": "0.61.5"
i fix it by add in info.plist file
<key>NSAllowsArbitraryLoads</key>
<true/>

which is added by default since react-native version 0.64

What was the solution?

What was the solution?

In my case. I disabled the splash screen.