paramgen: add support for custom field (un)marshalling

Feature description
The problem:  Here's an example from a connector middleware struct:
type SourceWithSchemaExtraction struct {
    SchemaType schema.Type `json:"sdk.schema.extract.type" validate:"inclusion=avro" default:"avro"`
}

where schema.Type is an alias of schema.Type in conduit-commons.
When parsing SourceWithSchemaExtraction with sdk.Util.ParseConfig, we get an error saying that there's a type mismatch, because the default value is "avro" (a string), and the target value is an int. The reason is that in Config.Validate() we validate the types.
Possible solution:

Remove type validation in Config.Validate() altogether. In the Connector SDK, it's called only when parsing a config, and after that we decode the map into a struct, where the types are validated too.
Use one of mapstructure's hooks (link), so that developers can implement a MarshalText/UnmarshalText method and customize parsing.

We have a workaround for this, but it's a useful feature that we plan to add in one of the next releases.


Training time too long

I train yolox-s with batch size 128 + 8 x V100, and the task takes about 2 days 6 hours for 300 epoch. Is the training time normal ?

Please provide your data_time and train_time in log, Thanks

Please provide your data_time and train_time in log, Thanks

Here are some of training logs:
2021-07-21 02:58:47 | INFO     | yolox.core.trainer:250 - epoch: 1/300, iter: 640/925, mem: 7751Mb, iter_time: 0.666s, data_time: 0.047s, total_loss: 11.2, iou_loss: 3.8, l1_loss: 0.0, conf_loss: 4.8, cls_loss: 2.7, lr: 3.830e-04, size: 672, ETA: 2 days, 18:48:57
671 2021-07-21 02:58:54 | INFO     | yolox.core.trainer:250 - epoch: 1/300, iter: 650/925, mem: 7751Mb, iter_time: 0.700s, data_time: 0.043s, total_loss: 11.3, iou_loss: 3.6, l1_loss: 0.0, conf_loss: 4.9, cls_loss: 2.8, lr: 3.950e-04, size: 576, ETA: 2 days, 18:36:51
672 2021-07-21 02:59:02 | INFO     | yolox.core.trainer:250 - epoch: 1/300, iter: 660/925, mem: 7751Mb, iter_time: 0.792s, data_time: 0.030s, total_loss: 11.5, iou_loss: 3.7, l1_loss: 0.0, conf_loss: 5.1, cls_loss: 2.7, lr: 4.073e-04, size: 672, ETA: 2 days, 18:31:30
673 2021-07-21 02:59:09 | INFO     | yolox.core.trainer:250 - epoch: 1/300, iter: 670/925, mem: 7751Mb, iter_time: 0.776s, data_time: 0.033s, total_loss: 11.6, iou_loss: 3.7, l1_loss: 0.0, conf_loss: 5.3, cls_loss: 2.7, lr: 4.197e-04, size: 736, ETA: 2 days, 18:25:14
674 2021-07-21 02:59:18 | INFO     | yolox.core.trainer:250 - epoch: 1/300, iter: 680/925, mem: 7751Mb, iter_time: 0.865s, data_time: 0.012s, total_loss: 11.7, iou_loss: 3.7, l1_loss: 0.0, conf_loss: 5.3, cls_loss: 2.6, lr: 4.323e-04, size: 704, ETA: 2 days, 18:25:11
675 2021-07-21 02:59:25 | INFO     | yolox.core.trainer:250 - epoch: 1/300, iter: 690/925, mem: 7751Mb, iter_time: 0.660s, data_time: 0.051s, total_loss: 11.8, iou_loss: 3.5, l1_loss: 0.0, conf_loss: 5.6, cls_loss: 2.6, lr: 4.451e-04, size: 512, ETA: 2 days, 18:11:24
676 2021-07-21 02:59:32 | INFO     | yolox.core.trainer:250 - epoch: 1/300, iter: 700/925, mem: 7751Mb, iter_time: 0.754s, data_time: 0.022s, total_loss: 11.2, iou_loss: 3.7, l1_loss: 0.0, conf_loss: 5.1, cls_loss: 2.4, lr: 4.581e-04, size: 608, ETA: 2 days, 18:04:14
677 2021-07-21 02:59:40 | INFO     | yolox.core.trainer:250 - epoch: 1/300, iter: 710/925, mem: 7751Mb, iter_time: 0.792s, data_time: 0.031s, total_loss: 11.2, iou_loss: 3.7, l1_loss: 0.0, conf_loss: 4.9, cls_loss: 2.5, lr: 4.713e-04, size: 736, ETA: 2 days, 17:59:42

Hi, we reproduce the training setting and your training time seems normal. Actually, if you change yolox-s to yolox-l, the total training time is almost the same! We suppose the major time consuming comes from our data augment operation, and we also have a plan to accelerate it.

@ruinmessi Hello, do you know what causes the difference in training time between iters? Like the record shown, the max consuming time is 0.940ms, and the min consuming time is 0.544ms.


Publish storybook

Hi! I see that you're trying to do something with the storybook. I wanted to see if you might know where I can find code for these examples: https://prc5.github.io/react-zoom-pan-pinch/?path=/docs/examples-zoomed-out--zoomed-out
Although, now I'm realizing it might be in @prc5's github.io repo


[Admind] Fix broken links

Looks like the Pixton app is no longer available so switching to another app.

Docs Build status updates of commit 33711b6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/powerpoint/powerpoint-add-ins.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


ci: Updated actions/workflow to minimize deprecation warnings

This PR updates the README to mention the 0.3 release in the "Current Status" section. It also begins to accumulate changes in the 0.3.1 Changelog
It also updates the Dockerfile to use rust 1.64.
Finally, it updates the .github actions and workflows to remove most of the deprecation warnings.

Most of the deprecation warnings were caused by running node12 in Github Actions, which will start requiring node16 soon.

Most of the upstream actions have been updated, so these actions/workfiles only needed to refer to a new version number

baptiste0298/cargo-install@next seems to work, although it may not be a stable fix. (I'm not confident that that the author is paying attention to that branch.)

In actions/test-rust/action.yml the actions-rs/cargo@v1 hasn't been updated, even after I created an issue requesting it. I cloned it to<EMAIL_ADDRESS>to see if it would work. It did, but PRQL should not rely on my faulty memory, and should seek an alternate repo.

Similarly<EMAIL_ADDRESS>has not been fixed/updated to node16. (Its action.yml file is still `using: 'node12'). Unless the upstream fixes the Node version, it would need to be cloned in the same manner as richb-hanover/cargo.

Both the Java and Python workflows give warnings about using macos-12 soon. The official date for cutover is 1 Dec 2022 (tomorrow), so I just left their versions as "macos-latest" which should switch on the appointed day.

Super! Thank you v much @richb-hanover , really appreciate the addition


workflow keep running when all tasks completed

Summary

reproduced at v3.1.6, v3.1.7, v3.2.0-rc1

It's a rare case, can only reproduce when satisfy all the conditions below:

task main.A must Succeded with no actual workload generated ( in this workflow it generated an empty TaskGroup )
main.B depends on main.A, run normally
main.C depends on main.A, it should Skipped, and execute after main.B

This yaml can reproduce it:
apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  generateName: debug-renormalization-
  namespace: argo
spec:
  arguments:
    parameters:
    - name: empty_seq
      value: '[]'
  entrypoint: main
  onExit: sImPlEsT
  templates:
  - container:
      args:
      - echo
      - Running...
      command:
      - /bin/sh
      - -c
      image: alpine:latest
      name: main
    name: sImPlEsT
  - dag:
      tasks:
      - name: A
        template: sImPlEsT
        withParam: "{{workflow.parameters.empty_seq}}"
    name: A
  - dag:
      tasks:
      - name: sImPlEsT
        template: sImPlEsT
    name: B
  - dag:
      tasks:
      - name: A
        template: A
      - depends: A
        name: B
        template: B
      - depends: A.Succeeded || A.Failed || A.Errored
        name: C
        template: sImPlEsT
        when: "{{tasks.A.status}} != Succeeded"
    name: main

Diagnostics
The reason is main.C got empty outboundNodeIDs from it's depend node main.A, so it cannot connect to other node, which means main.C is an orphan node in status.nodes. Later when checking main's phase using BFS( targetTasks = ['B', 'C'] ) it cannot iterate over main.C's node, so it consider one of main's targetTasks have empty phase, and leave main at Running.
Here's what I got when it stuck:
apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  creationTimestamp: "2021-08-24T12:08:24Z"
  generateName: debug-renormalization-
  generation: 3
  labels:
    workflows.argoproj.io/creator: system-serviceaccount-argo-argo-server
    workflows.argoproj.io/phase: Running
  managedFields:
  - apiVersion: argoproj.io/v1alpha1
    fieldsType: FieldsV1
    fieldsV1:
      f:metadata:
        f:generateName: {}
        f:labels:
          .: {}
          f:workflows.argoproj.io/creator: {}
      f:spec:
        .: {}
        f:arguments: {}
        f:entrypoint: {}
        f:onExit: {}
        f:serviceAccountName: {}
        f:templates: {}
      f:status:
        .: {}
        f:finishedAt: {}
    manager: argo
    operation: Update
    time: "2021-08-24T12:08:24Z"
  - apiVersion: argoproj.io/v1alpha1
    fieldsType: FieldsV1
    fieldsV1:
      f:metadata:
        f:labels:
          f:workflows.argoproj.io/phase: {}
      f:status:
        f:artifactRepositoryRef: {}
        f:conditions: {}
        f:nodes: {}
        f:phase: {}
        f:progress: {}
        f:resourcesDuration: {}
        f:startedAt: {}
    manager: workflow-controller
    operation: Update
    time: "2021-08-24T12:08:34Z"
  name: debug-renormalization-w5cmx
  namespace: argo
  resourceVersion: "2516381"
  uid: 7301a14f-9dd8-492b-868b-a717dfec94e9
spec:
  arguments:
    parameters:
    - name: empty_seq
      value: '[]'
  entrypoint: main
  onExit: sImPlEsT
  serviceAccountName: argo
  templates:
  - container:
      args:
      - echo
      - Running...
      command:
      - /bin/sh
      - -c
      image: alpine:latest
      name: main
      resources: {}
    inputs: {}
    metadata: {}
    name: sImPlEsT
    outputs: {}
  - dag:
      tasks:
      - arguments: {}
        name: A
        template: sImPlEsT
        withParam: '{{workflow.parameters.empty_seq}}'
    inputs: {}
    metadata: {}
    name: A
    outputs: {}
  - dag:
      tasks:
      - arguments: {}
        name: sImPlEsT
        template: sImPlEsT
    inputs: {}
    metadata: {}
    name: B
    outputs: {}
  - dag:
      tasks:
      - arguments: {}
        name: A
        template: A
      - arguments: {}
        depends: A
        name: B
        template: B
      - arguments: {}
        depends: A.Succeeded || A.Failed || A.Errored
        name: C
        template: sImPlEsT
        when: '{{tasks.A.status}} != Succeeded'
    inputs: {}
    metadata: {}
    name: main
    outputs: {}
status:
  conditions:
  - status: "False"
    type: PodRunning
  finishedAt: null
  nodes:
    debug-renormalization-w5cmx:
      children:
      - debug-renormalization-w5cmx-2946515967
      displayName: debug-renormalization-w5cmx
      finishedAt: null
      id: debug-renormalization-w5cmx
      name: debug-renormalization-w5cmx
      phase: Running
      progress: 1/1
      startedAt: "2021-08-24T12:08:24Z"
      templateName: main
      templateScope: local/debug-renormalization-w5cmx
      type: DAG
    debug-renormalization-w5cmx-900713958:
      boundaryID: debug-renormalization-w5cmx-2946515967
      children:
      - debug-renormalization-w5cmx-2963293586
      displayName: A
      finishedAt: "2021-08-24T12:08:24Z"
      id: debug-renormalization-w5cmx-900713958
      name: debug-renormalization-w5cmx.A.A
      phase: Succeeded
      progress: 1/1
      resourcesDuration:
        cpu: 6
        memory: 6
      startedAt: "2021-08-24T12:08:24Z"
      templateName: sImPlEsT
      templateScope: local/debug-renormalization-w5cmx
      type: TaskGroup
    debug-renormalization-w5cmx-2946515967:
      boundaryID: debug-renormalization-w5cmx
      children:
      - debug-renormalization-w5cmx-900713958
      displayName: A
      finishedAt: "2021-08-24T12:08:24Z"
      id: debug-renormalization-w5cmx-2946515967
      name: debug-renormalization-w5cmx.A
      outboundNodes:
      - debug-renormalization-w5cmx-900713958
      phase: Succeeded
      progress: 1/1
      resourcesDuration:
        cpu: 6
        memory: 6
      startedAt: "2021-08-24T12:08:24Z"
      templateName: A
      templateScope: local/debug-renormalization-w5cmx
      type: DAG
    debug-renormalization-w5cmx-2963293586:
      boundaryID: debug-renormalization-w5cmx
      children:
      - debug-renormalization-w5cmx-3540535569
      displayName: B
      finishedAt: "2021-08-24T12:08:34Z"
      id: debug-renormalization-w5cmx-2963293586
      name: debug-renormalization-w5cmx.B
      outboundNodes:
      - debug-renormalization-w5cmx-3540535569
      phase: Succeeded
      progress: 1/1
      resourcesDuration:
        cpu: 6
        memory: 6
      startedAt: "2021-08-24T12:08:24Z"
      templateName: B
      templateScope: local/debug-renormalization-w5cmx
      type: DAG
    debug-renormalization-w5cmx-2980071205:
      boundaryID: debug-renormalization-w5cmx
      displayName: C
      finishedAt: "2021-08-24T12:08:24Z"
      id: debug-renormalization-w5cmx-2980071205
      message: when 'Succeeded != Succeeded' evaluated false
      name: debug-renormalization-w5cmx.C
      phase: Skipped
      startedAt: "2021-08-24T12:08:24Z"
      templateName: sImPlEsT
      templateScope: local/debug-renormalization-w5cmx
      type: Skipped
    debug-renormalization-w5cmx-3540535569:
      boundaryID: debug-renormalization-w5cmx-2963293586
      displayName: sImPlEsT
      finishedAt: "2021-08-24T12:08:31Z"
      hostNodeName: k3d-argo-server-0
      id: debug-renormalization-w5cmx-3540535569
      name: debug-renormalization-w5cmx.B.sImPlEsT
      outputs:
        exitCode: "0"
      phase: Succeeded
      progress: 1/1
      resourcesDuration:
        cpu: 6
        memory: 6
      startedAt: "2021-08-24T12:08:24Z"
      templateName: sImPlEsT
      templateScope: local/debug-renormalization-w5cmx
      type: Pod
  phase: Running
  progress: 1/1
  resourcesDuration:
    cpu: 6
    memory: 6
  startedAt: "2021-08-24T12:08:24Z"

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

We will look this

@simster7 Hi may I ask the progress on this issue?

@book987 Can you try in v3.2.2 ?

Sorry for late! I've tested in v3.2.2 and v3.2.3, both can reproduce


archway deploy -

seems like a bug. I can't add reward wallet in build process.
But If I enter wallet adress in config.json developer.rewardAddress deploing process went smooth.
Enter keyring passphrase:Error: rpc error: code = InvalidArgument desc = failed to execute message; message index: 0: Error parsing into type test_3::msg::InstantiateMsg: missing field `count`: instantiate wasm contract failed: invalid requestUsage:
  archwayd tx wasm instantiate [code_id_int64] [json_encoded_init_args] --label [text] --admin [address,optional] --amount [coins,optional] [flags]```

mb I did smth wrong. Experimenting with source code didn't return the same error. version in both cases is v0.0.1.
Closing Issue for now.


admin can't change permission i.e channel.send

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.
AB#8593

Hey @dhamodar-adappt could you explain more about this issue?
For example how we can reproduce the behaviour? what the expected behaviour is?

Closing this as no information was added. The issue can be reopened once more info is provided


HDDS-10934. Refactor TestOzoneRpcClient hierarchy

What changes were proposed in this pull request?
Refactor TestOzoneRpcClientAbstract and its subclasses:

TestOzoneRpcClient and TestOzoneRpcClientWithRatis are subclasses of TestOzoneRpcClientAbstract.  They used to execute the same test cases with OM Ratis disabled (relying on default setting) and enabled (explicitly), respectively.  HDDS-4498 enabled OM Ratis by default, so now both classes test with OM Ratis enabled.  This change disables OM Ratis for TestOzoneRpcClient, and fixes one test case that was failing with NPE in this setup.
TestSecureOzoneRpcClient should directly extend TestOzoneRpcClientAbstract to avoid confusion caused by shadowed lifecycle methods.
Allow passing MiniOzoneCluster.Builder to startCluster to avoid code duplication in TestSecureOzoneRpcClient.
Rename TestOzoneRpcClientAbstract to OzoneRpcClientTests to follow naming conventions.
Move test cases from TestOzoneRpcClientWithRatis to the base class to extend coverage for other configs.

(Items 2-5 were added after initial round of review.)
https://issues.apache.org/jira/browse/HDDS-10934
How was this patch tested?
CI:
https://github.com/adoroszlai/ozone/actions/runs/9282222672/job/25540423674#step:5:2276

Thanks @errose28 for the review.

The tests run for both ratis and non-ratis cases (in TestOzoneRpcClientAbstract) and those run only for Ratis cases (in TestOzoneRpcClientWithRatis) seem kind of arbitrary.

Maybe we can either unify them so all tests run in both setups, or only a minimal amount of validation is done in the non-ratis setup.
Also tests that are unique to TestOzoneRpcClientWithRatis are not run in a secure setup.

Agree, we should move them to the parent class.

TestSecureOzoneRpcClient extends TestOzoneRpcClient, so now the secure tests are only running in a non-ratis OM setup.

This is not the case, since TestSecureOzoneRpcClient creates its own cluster, TestOzoneRpcClient#init is not invoked due to "shadowing" (which is a bit obscure).  Otherwise it would be testing an unsecure setup or starting two clusters.  This should be improved, too.
Will push a few more commits.

@errose28 patch is updated, please take another look

Thanks @errose28 for the review.


Dbs upgrades

I've created a separate issue for Db2: https://github.com/hibernate/hibernate-reactive/issues/1037


token-swap: Add weights to swap contract

Problem
Weights for the tokens in a swap contract are very useful to come to a correct price without too much liquidity needed. For example, if token A is weighted at 10%, and token B is weighted at 90%, if you have equal liquidity at 5000: 5000, swapping 100 token A for token B will give you (according to (15) in https://balancer.finance/whitepaper/):
5000 * (1 - ( 5000 / (5000 + 100) ) ^ (1 / 9)) = 10.989
To do that, we need to be able to do fractional exponents!
Solution
Add some "precise" math which uses u128 under the hood, similar to the Balancer implementation: https://github.com/balancer-labs/balancer-core/blob/master/contracts/BNum.sol -- with more comments.  Note that these calculations are meant to operate on numbers between epsilon and 2, or the series won't converge.
For the power approximation, a Taylor Series did better than Newtonian root-finding.  It's high school calculus all over again!
This is still WIP, and the parts to add the weights and integrate these calculations will come tomorrow, but I wanted to get the math out there for discussion.  I first kept the numerator and denominator separate as we do elsewhere, but you very quickly blow out a u128 for a fraction like (40 / 3), which is a reasonable set of weights in a pool.

This is all ready to be reviewed now -- fire away!

I'll add in the perf monitor as you recommend, and otherwise I'll update all of the bits of documentation that you recommended.

Too much has changed since the last update on this, namely the refactoring of instruction tests, all the new fees / parameters in token swap, and even the building of programs, so I'll close this and start fresh somewhere else.
While this PR is still active though, I wanted to publish the instruction counts from preliminary testing.  To do this, I used the existing token/perf-monitor, adapted it to token-swap, and stubbed out the calls to invoke_signed in the token swap program.  Here are the numbers, calculated using the test results and adding the tolerances from token/perf-monitor:
Initialize: 17_000; // stubbed: 9735, add 6500 for mint to
Deposit: 33_000; // stubbed: 10102, add 6500 for mint to, 8000 for each transfer
Withdraw: 33_000; // stubbed: 10103, add 6500 for burn, 8000 for each transfer
Swap from weight 100 to weight 3: 55_000; // stubbed: 36678, add 8000 for each transfer
Swap from weight 3 to weight 100: 40_000; // stubbed: 20636, add 8000 for each transfer

I also played with the iterations in the pow calculation and removed the break to see how many instructions it generated.  For 100 iterations, here are the results:
Swap high to low weight: 108,869
Swap low to high weight: 92,070

For a compute budget is 200,000, this seemed like a reasonable limit.

Going to close this -- too many changes since this happened, but at least the math got in eventually.  We might want to create a "WeightedCurve" in order to make this happen.


chore(gamut-styles)!: deprecate legacy JS color variables

Overview
A more thorough overhaul of the colors to make way for a full migration to the new color set.

Moves the current colors (standardColors) to the colors export
Removes the extra keys from the colors export (a revert from the previous color PR)
Moves the old color swatches to legacyColors
Creates a new set of swatches to scale with the new colors, under the swatches export

This will allow us to use the colors export for Emotion components, instead of allowing all of the old colors as well.
Subsequent tasks:

Deprecate legacy SCSS variables (if needed) [GM-114]

PR Checklist

[ ] Related to Abstract designs:
[ ] Related to JIRA ticket: [GM-113]
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

Description

🚀 Styleguide deploy preview ready!https://5f453a4446b7ddd43dc0c220--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@3.0.2-alpha.b8ea25.0@codecademy/gamut-styles@6.0.1-alpha.b8ea25.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f492a6166cfbf8f868228a2--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@3.0.2-alpha.953c19.0@codecademy/gamut-styles@6.0.1-alpha.953c19.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f4ea048a33a7b00a950c6d1--gamut-preview.netlify.appDeploy Logs


Decl impl

I create this PR even if the full conversion is not done because it touch almost all pythonic files in a not mergeable way. I would like to be approved on this before continuing. (Integration will have to be done before the full rework is done except if no other development happen before the end of this PR)

do you plan to have this merged before the release?

I think it will be too long for a full merge before the release but looking at the current PR, only the 32 bits one is missing for the release, isn't it?

This one is ready to be merged. I will stash the WIPbuiltin if it is OK (I prefer not to run travis until I know it is all fine).

As far as i understand, this PR is the first step for a larger one. The goal of this PR is most certainly to avoid header dependency hell that is starting to pollute the already-no-so-nice pythonic code base. I think this deserve a different directory namespace instead of the _impl suffix. I've seen the .tcc extension for that purpose, or use a src directory inside pythonic that matches the existing hierarchy, as in:
pythonic/
   numpy/
      cos.hpp   // fwd decl
   src/
      numpy/
         cos.hpp // impl

just to be sure : you don't want this into next release, do you?

No, It is a too big clean up. We will not be able to add it before PyData.

I don't really like the "src" directory because I think we will have real source for Pythonic (and a pythonic.so) in the future. (Ok, inlining is good but complex function are not inlined anyway)
Indeed, the _impl suffix is what we use for kaapi/starpu and klang :)

We can use a "libs" folder. Just like clang. They have a libs folder for src and header with "internal scope" and include folder for exported interface.

On Mon, Mar 23, 2015 at 09:57:37AM -0700, pbrunet wrote:

I don't really like the "src" directory because I think we will have real source for Pythonic (and a pythonic.so) in the future. (Ok, inlining is good but complex function are not inlined anyway)
Indeed, the _impl suffix is what we use for kaapi/starpu and klang :)

I was about to tell you that we should adopt the standard include/stuff
for declaration, as done in nt2 :-)
So what about
pythonic/include/numpy/cos.hpp # declaration
and
pythonic/numpy/cos.hpp # definition

Ok for this. But how can we do this modification "smoothly"?
I suggest to :

Continue with xxx_impl.hpp
Make this change when we will need to change includes directive generation.

Do you have any other plan for this?

This one works on my travis : https://travis-ci.org/pbrunet/pythran/builds/56282288
For now, it contains header spliting for :

list
-builtin
bisect
I do not perform modification on the numpy part as it is the most active for now.

the concat commit can now be safely removed

I wonder if it's not too messy to have the array_like class and the split header in the same PR?

If I understand this PR correctly, the goal is to have the declaration in a file, and the definition in an other file. then if I depend of function foo, I'm going to :+1:
#include "include/pythonic/module/foo.hpp" // get the declaration

// use the function

#include "pythonic/module/hoo.hpp" // get the definition

Am I right?

Yes it is how it will work. I agree that they may have been split. I did it this way because the array_like part have a lot of modification on list.hpp and the module split too (in a not mergeable way). As these modification are really tedious, I don't want to do the job twice.
Also, they are related as the array_like feature can't be totally use because of function dependencies (solved with the module split part) in the numpy_gexpr file for example.

@pbrunet: just so you knwo, I'm currently porting list / ndarray so that the remaining commits will be ported automatically.

Obsoleted by #445

@pbrunet I only ported the changes up to the str split, the remaining should be ok
I did not keep the long → size_t transformation

Changes Unknown when pulling 156120d7278097d573832485466c280b5c45a289 on pbrunet:decl_impl into ** on serge-sans-paille:master**.

Changes Unknown when pulling 156120d7278097d573832485466c280b5c45a289 on pbrunet:decl_impl into ** on serge-sans-paille:master**.

Changes Unknown when pulling 156120d7278097d573832485466c280b5c45a289 on pbrunet:decl_impl into ** on serge-sans-paille:master**.


OOPSIE WOOPSIE!! Uwu We madea fucky wucky!! A wittle fucko boingo!


How to import "MPTForCausalLM" models?

Hello everyone,
I'm not sure if I can use the model with the "MPTForCausalLM" architecture with Ollama. I tried to import it following the instructions here: https://github.com/jmorganca/ollama/blob/main/docs/import.md
The import process was successful, but when running with the new model, I encountered an error: "ollama run example "What is your favourite condiment?"
I am using models from this source:
https://huggingface.co/vinai/PhoGPT-7B5
https://huggingface.co/vinai/PhoGPT-7B5-Instruct

Thanks

Hi @ansutung sorry about this. This model should be able to run with Ollama. Are you still seeing issues running the model? If so, can you share the modelfile being used?
So sorry about this. [let me know if you are still running into this problem, and reopen]


Use new version format in release names

The version format has been changed to include a v prefix. Should we include this in directory and tarball names for consistency?
@fabxc @sdurrheimer

I'm fine with both, with or without the v. We just need to keep in mind this change might hurt peoples automation scripts :).

The applied rename already hurt ours. We have VERSION variables in all our
makefiles, we now need to append a v depending on the context.
On Tue, Jul 19, 2016 at 2:08 AM, Steve Durrheimer<EMAIL_ADDRESS>wrote:

I'm fine with both, with or without the v. We just need to keep in mind
this change might hurt peoples automation scripts :).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/promu/issues/33#issuecomment-233540841,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAANaGdBiZjh8n0RRnUFKmO8PJ0x4SNBks5qXGn3gaJpZM4JPV-z
.

I don't think we need it in the tarball. It is natural to have the version number in the form we have. Tags, however, can be anything, not just versions. That's the rationale of having the v prefix, essentially namespacing. That doesn't apply to the directory and tarball name.


[Bug]: afterAll order differs between jest-circus and jasmine

Version
27.3.1
Steps to reproduce
Given this:
describe('spec', () => {
  beforeAll(() => console.log('First before'));
  afterAll(() => console.log('First after'));

  beforeAll(() => console.log('Second before'));
  afterAll(() => console.log('Second after'));
});

Expected behavior
When running with Jasmine, the order will be:
First before
Second before
Second after
First after

Actual behavior
First before
Second before
First after
Second after

Additional context
Now, this might seem like the more intuitive way which is why I was reluctant to report it as a bug. However, I prefer to group setup/teardown logic into "test hooks". With the jest-circus order, I am forced to explicitly write the calls in the same file. I think you could argue that the Jasmine approach is symmetric with describe-blocks.
Environment
System:
    OS: Linux 5.11 Ubuntu 21.04 (Hirsute Hippo)
    CPU: (8) arm64 unknown
  Binaries:
    Node: 16.10.0 - /usr/bin/node
    Yarn: 1.22.15 - /usr/bin/yarn
    npm: 7.18.1 - ~/.npm-global/bin/npm

I can understand that you are swamped with work so let me ask: if I were to submit a PR that fixes this, would you be willing to merge it (given that it otherwise lives up to your standards etc.)


Increase touch area

Is there any feature to increase touch area. If I increase Cosmos:UIView width and height it does not stay at the center and aligns to top.
The idea is to improve the UX so a user can easily touch and give stars/ratings even if he touches 10px away of Cosmos view.

Hi @khunshan, that's a good question. Currently, the hit bounds are increased if the view's width or height is smaller than 44 pixels. This value is hardcoded at the moment here, but we can move it to settings, so it can be customised. Do you experience a problem when touches are not registered by the cosmos view?

If I increase Cosmos:UIView width and height it does not stay at the center and aligns to top.

I'm sure you are aware of this, but Cosmos view sets its own width and height based on its content. There is no need to set its size manually or with an auto layout constraint. Maybe that's what causing the touch issues, if they are present?

Hi @evgenyneu, at this moment the touch area can affect other UI components which are close to the CosmosView,  it's better to move the hit bounds value to settings.
Thanks

Hi @evgenyneu, at this moment the touch area can affect other UI components which are close to the CosmosView,  it's better to move the hit bounds value to settings.
Thanks


How to change the style of the arrow pointing to the input on top of the popup container?

Hello,
I have two calendars in two inputs and I would like to change the arrow of the second  (see attached image) to point to the second input respectively. I searched thoroughly to the styles file here trying to find the css selector but with no luck. Could someone point me to the right direction?

Thanks in advance.

I found the solution by changing the style of .fromCalendar .rdtPicker:before
and .fromCalendar .rdtPicker:after css selectors respectively.


Right clicking the Terminal gives me menu in Japanese, my OS is in EN-US

Environment
Windows version: Version 1903 (OS Build 18362.959)
Windows Terminal: 1.1.2021.0
Not sure how useful this information is, my Windows was installed with EN-US settings. I then installed the JP keyboard.
Steps to reproduce
Right click on the top bar.
Expected behavior
Show the menu in my main language setting, which is English.
Actual behavior
The menu is in Japanese.

Doing the same thing on Command Prompt shows the menu in English, which is the correct behavior.

Can you please share a screenshot of your "preferred language" list? The order of this list dictates the language of modern packaged applications.

You can drag the items in that list to rearrange them.
To quote the settings page,
Apps and websites will appear in the first language in the list that they support.

Here is my Preferred languages setting:

I fixed this by changing the display and preferred language to be JP. Restarted my PC. And then changing both settings back to English,
Probably the bug is on Windows, rather than Terminal.


DIsable "Expand Side on hover" on fixed layout

Is it possible to disable the "expand sidebar on hover" feature when using a fixed navbar / layout?

Unfortunately, that's not possible since it would cause many css issues and elements won't be placed in the right location.
Thanks!


Sync the stub and script sections

As a result of #24 , it  seems that the stub and script sections are now out of sync because of the latest changes.

This needs to be initiated after #26  is merged.


Zany face redesign

Not sure if I'm done with this one but I think it's an improvement over the current design. I copied the color and shape of the tongue from other blob emojis, removed the teeth in the mouth (which seemed to clash with the rest of the face), and converted the eyes to regular, pupil-less blob eyes.

I agree that the teeth can be removed (In my opinion it doesn't make such a difference, but I'm absolutely fine with it).
About the eyes: I'm not sure if it's just an optical illusion, but the right eye looks pretty large which looks rather strange when they are all dark (they begin to look like holes or alien-like eyes).
Just as an exaggerated illustration how the white part matters:

And in the end, eyes including the white part are actually very common both for the Blobs as in other emoji sets (all vendors represented on Emojipedia have the white part included for this particular emoji).
Finally the tongue: That's something I'm also fine with.
Although the shape isn't perfect, the contrast between the tongue and the mouth isn't large enough to actually see something (which is fine, although a brighter version might be better suited in this case as it's (IIRC) the first case where the tongue is not used in conjunction with a closed mouth).

The whites of the eyes are tough for the Blobs.  This face shows it in particular because I made the right eye larger than normal. I'm leaning towards keeping eyes either entirely dark or with plenty of space around the pupil. With that in mind I tweaked the zany face a little:

It borrows the eye proportions from other Blob emojis with pupils. The convention is to make one eye smaller, but I think making the pupils askew represents the idea of "zany" enough for this to not be necessary. Lmk what you think.
The tongue could be a little brighter; I was keeping the colors matched with other tongues but I agre that it could stand out better here. I'll toy around with it and report back.

How about this color: #EB7AA4?

That color is good.


Data with Nan values

Working with NaN data
My data has too many Nan values.
I tried filling the nan values with '0'. But the results do not seem desirable.
using other packages (xMCA and pyEOFs) i get more physically meaning patterns.
I want to find a way out for my data using XEOFs as it has too many functionalities.
In XMCA there is no cos_lat weighing option.
Can you please suggest either the following:
1) Either cos_lat weighing option for xMCA
2) Or way to use my data (without filling Nan values with 0)
Desktop :

OS: Windows
xeofs version 2.3.2

my data

NaNs in your data are always a bit tricky. I'm not sure for pyEOFs, but what xMCA does in the case of individual NaNs (e.g. grid points where only a few time steps are missing), is just to remove the entire grid cell before the analysis. This admittedly rude approach works reasonably well when the overall fraction of features with individual NaNs is low. In your case, what's the fraction of grid cells that has individual NaNs?
Alternatively, you have to think about how you can fill these values, either subjectively (e.g. by using some fixed values) or more objectively (e.g. by using Probabilistic PCA). Ultimately, the choice will depend on the data you have. In xeofs we don't automatically treat individual NaNs, because the decision of how to treat NaNs should ultimately be with the analyst.

Thanks for the response @nicrie
xMCA does help.
But there is not cos_lat weighing option in xmca right?

Yes, there is a cosine-latitude weighting option in xMCA. You can find more information here. Note, however, that your solution obtained from xMCA will be exactly the same as using xeofs if you remove all grid cells with individual NaNs. This can be done by calling:
da = da.where(da.notnull().all("time"))

assuming that time is the dimension along which you want to maximize the variance.

Thanks for the time @nicrie
I tried 'apply_coslat()' from the link as suggested.

pca = xMCA(x.z) #x is my dataarray
pca.apply_coslat()
pca.solve(complexify=False)            # True for complex PCA
svals = pca.singular_values()     # singular vales = eigenvalues for PCA
expvar  = pca.explained_variance()  # explained variance
pcs  = pca.pcs()                 # Principal component scores (PCs)
eofs = pca.eofs()

Yet, the results look like those without 'cos_lat'
Is there anything I'm missing?
With xeofs:
\anaconda3\envs\xEOFs\Lib\site-packages\numpy\lib\nanfunctions.py:1879: RuntimeWarning: Degrees of freedom <= 0 for slice.
  var = nanvar(a, axis=axis, dtype=dtype, out=out, ddof=ddof,

error
which means, the nan values are not deleted.
Here is a screenshot after applying
da = da.where(da.notnull().all("time"))

Result:

Why do you think that the warning message implies that NaN values are not deleted? The code
da = da.where(da.notnull().all("time"))

masks out grid cells with individual NaNs. In the preprocessing step of xeofs, these grid cells are removed prior to the SVD (otherwise you would get an error there), and are reinserted afterwards into your results. My guess is that the warning arises due to the computation of standard deviations on only NaN slices.
For your question about xmca: Yes, apply_coslat() works for both PCA and MCA. However, I need to mention that I've stopped maintaining xmca in favor of xeofs for about a year now. This means I can't provide detailed debugging support for xmca anymore. I'd encourage you to focus on xeofs for your current and future analyses, as it's actively maintained and supported.

Closing as it doesn't seem like a bug in xeofs. Feel free to reopen @LKPatel1


Error Crash on any DatePicker Components

DatePicker crashing my app also TimePicker please help! thanks in advance
Uncaught Error: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in. Check the render method of ModalWrapper.
at invariant (invariant.js:42)
at ReactCompositeComponentWrapper.instantiateReactComponent [as _instantiateReactComponent] (instantiateReactComponent.js:72)
at ReactCompositeComponentWrapper.performInitialMount (ReactCompositeComponent.js:364)
at ReactCompositeComponentWrapper.mountComponent (ReactCompositeComponent.js:255)
at Object.mountComponent (ReactReconciler.js:43)
at ReactCompositeComponentWrapper.performInitialMount (ReactCompositeComponent.js:368)
at ReactCompositeComponentWrapper.mountComponent (ReactCompositeComponent.js:255)
at Object.mountComponent (ReactReconciler.js:43)
at ReactDOMComponent.mountChildren (ReactMultiChild.js:234)
at ReactDOMComponent._createInitialChildren (ReactDOMComponent.js:701)
Environment

Tech
Version

material-ui-pickers
^1.0.0-beta.13.1

material-ui
1.0.0-beta.30

React
15.4.0

Browser
Chrome

Platform
Windows

Steps to reproduce

npm run dev
During the App loading it through exception

Expected behavior
Showing Date Picker + Time Picker
Actual behavior
i guess it's bug in the date-picker source file function/var not exported
Live example
https://codesandbox.io/s/m5njjmxy2y

I am using react 16.3.0 getting Cannot read property 'date' of undefined
DatePickerWrapper.PickerBase._this.getValidDateOrCurrent
node_modules/material-ui-pickers/_shared/PickerBase.js:60


Allow return-value of tap() to replace the current value

Right now it's possible to replace properties of the passed-in object to tap(), but there's no good way to replace the object itself.
This is useful if you want to use a function in your chain() that you don't want to mixin() (or the function is part of another library, etc).

That's correct. The point of tap is to return the original value -- that's all it does. If you want to use the return value of a function, just use the function.
value = func(object)

@jashkenas  Maybe we're not on the same page, but I thought tap was to allow you to modify the value without breaking the chain, not just return what you passed in.
In either case, value = func(object) doesn't work as you expect.
_([1, 2, 3]).chain()
.tap(function (value) {
	value = [4, 5, 6]; // this will not replace the value
})
.reverse()
.value(); // [3, 2, 1] - but expecting [6, 5, 4]

I'm really not trying to be glib, but:
[4, 5, 6].reverse()

... produces [6, 5 ,4].
Do you have a real-world example of what you're trying to write here?

Sure. I have to use some libraries at work that perform operations via copy (instead of editing inline values). Much like the string functions in JavaScript.
The above patch is just a convenient way to use non-inline operations without having to mixin() or break the chain() I am in.
Taking capitalize from the example page:
function capitalize(string) {
	return string.charAt(0).toUpperCase() + string.substring(1).toLowerCase();
}

Here is underscore now:
_('foo').chain().tap(capitalize).value(); // foo

Here is with the patch above:
_('foo').chain().tap(capitalize).value(); // Foo

Again:
_('foo').chain().tap(capitalize).value();

versus:
capitalize('foo');
... do you have any examples of real-world code where you're trying to use this?

capitalize('foo');

Not trying to be rude, but for the third time, the point of the convenience is not to break the chain of other underscore operations I'm doing.
In its current form, tap only lets you modify properties of the value - it can't change the value itself. Strings are the best real-world analogy I have for you.
I'd be happy to help you understand more but it seems like the point isn't making it across.


[Feature Request]: Page Loading Indicator

Project
Website (Potter DB Search)
Describe the feature you want to request
It would be cool to have a page loading indicator to show the progress of loading the page.
(Optional) Describe a solution for this feature
Create a page loading indicator with nprogress.
The color of the loading bar should be #d500f9 and showSpinner should be set to false.

Hello, can you please assign this issue to me?

Sure, there you go. If you have any questions about this feature, leave a comment.


Added my information

Hi! This PR has a lot of changes to other entries. Please create a new PR and only add your own entry (even if your changes are minor, like removing extra spaces). Thank you!

@emckean Thanks for the feedback. The error came from my editor; it was set to CRLF instead of LF, so it altered the formatting of the whole file. I've adjusted the file appropriately and sent in a new PR.

Thank you!


Multiple slack notifications

What happened:
Default installation on k8s(AWS EKS x7 EC2 nodes in cluster) with helm helm install --name falco stable/falco creates x7 duplicate notifications when using the slack integration.
What you expected to happen:
Only one notification should be posted to slack.
How to reproduce it (as minimally and precisely as possible):
Default helm install, with slack notifications enabled.
jsonOutput: true
  ....
    programOutput:
    enabled: true
    keepAlive: false
    program: "\"jq '{text: .output}' | curl -d @- -X POST https://hooks.slack.com/services/xxxxxxxxxxxxxxxx/xxxxxxxxxxxxxxxxxx\""

Anything else we need to know?:
Environment:

Falco version (use falco --version):
falco version 0.15.3
System info <!-- Falco has a built-in support command you can use  "falco --support | jq .system_info" -->

{
  "machine": "x86_64",
  "nodename": "falco-6l9sz",
  "release": "4.14.146-119.123.amzn2.x86_64",
  "sysname": "Linux",
  "version": "#1 SMP Mon Sep 23 16:58:43 UTC 2019"
}

Cloud provider or hardware configuration:
AWS
OS (e.g: cat /etc/os-release):

NAME="Amazon Linux"
VERSION="2"
ID="amzn"
ID_LIKE="centos rhel fedora"
VERSION_ID="2"
PRETTY_NAME="Amazon Linux 2"
ANSI_COLOR="0;33"
CPE_NAME="cpe:2.3:o:amazon:amazon_linux:2"
HOME_URL="https://amazonlinux.com/"
Amazon Linux release 2 (Karoo)
cpe:2.3:o:amazon:amazon_linux:2

Kernel (e.g. uname -a):

Linux ip-10-0-73-85.eu-west-1.compute.internal 4.14.146-119.123.amzn2.x86_64 #1 SMP Mon Sep 23 16:58:43 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Install tools (e.g. in kubernetes, rpm, deb, from source):
Others:

For more advanced notifications in Slack, you can take a look at : https://github.com/falcosecurity/falcosidekick

Thanks, I've tried falcosidekick, but I still get x7 duplicate notifications(same number of nodes in the ds) for each triggered rule.

Your notifications are about a k8s events from audit logs? Seems logical that all pods of the daemonset detect it.

This is a typical event that's repeated x7 times
14:43:20.636017541: Error File below /etc opened for writing (user=root command=exe / /var/lib/docker/overlay2/28f02df2e5c37c18e31d6a2b5795a1139d7d88464c4da3d38f2ab9709863dadf/diff parent=dockerd pcmdline=dockerd file=/etc/shadow program=exe gparent=systemd ggparent=<NA> gggparent=<NA>) k8s.ns=<NA> k8s.pod=<NA> container=host

On a different note I don't see the pod name or container name(host being a red herring)
k8s.pod=<NA> container=host

Another user reported that kind of issue on Slack. cc @JPLachance. Have the name of related rule which triggers that?
For missing fields, it's kwown and people are figuring out how to solve that.

@seanmcrw the version of falco you are using is quite old 0.15.3. We had a lot of rules changes in 0.18.0 and every other version after that one and I believe this problem was solved by one of these https://github.com/falcosecurity/falco/pull/733 https://github.com/falcosecurity/falco/pull/736 https://github.com/falcosecurity/falco/pull/881 - could you try those out?

Thanks guys, using 0.18.0 solved the problem. Closing.

Thanks for confirming @seanmcrw !


Resp plans.01

Specs and response plan, phase, task update
Response plans updated and folders completed.  I have at least one of each ... most of the phases are done.
Please let me know what changes need to be made.

@socologize This is awesome thank you so much, give us a few to review carefully. Thank you 😄

@socologize going to close this PR for now and opened up https://github.com/splunk/security-content/pull/631 since we want to do a bit of changes to this before we merge it in, also easier to get CI running correctly. Keep track of this responses being merged in #631


Question about vq in the paper

Hi, thank you for your work.
I have a question regarding VQ experiment in the paper.
#z_spec_q_st, z_spec_q
did you use z_spec_q_st instead of z_spec ?
what happen with the indices z_spec_q ?
regarding the loss, did you simply add the commitment and vq loss to the overall loss (loss_gen_all)?
Thank you.

For now VQ implementation is not uploaded and some parts are remained.
Our VQ implementation is adopted from this repository.
z_spec_q_st gives gradient to z_spec, and z_spec_q does not give gradient.
Thus, to build VQ-VAE, z_spec_q_st is needed.


Connect to JO API

Prepare required data to be sent to JO API and send it to the test url.
Need more info about the need for static IP addresses (test/production) from JO + API documentation.

Description of required data -
https://app.zenhub.com/files/201485031/407ea6b7-e9e3-40c2-bff0-d1e49a6572cf/download

A meeting with JO scheduled 15/07/2021 as we need more info/documentation to continue.


feat: Add TLS Config options to HTTP template

Closes: https://github.com/argoproj/argo-workflows/issues/7390
Signed-off-by: Simon Behar<EMAIL_ADDRESS>Don't bother creating a PR until you've done this:

[ ] Run make pre-commit -B to fix codegen, lint, and commit message problems.

Create your PR as a draft.

Your PR needs to pass the required checks before it can be approved. If the check is not required (e.g. E2E tests) it
does not need to pass.
Once required tests have passed, you can make it "Ready for review".
Say how how you tested your changes. If you changed the UI, attach screenshots.

Tips:

If changes were requested, and you've made them, then dismiss the review to get it looked at again.
Add you organization to USERS.md if you like.
You can ask for help!

Replaced by #7929


use china mirror in binary downloading on azure china

What this PR does / why we need it:
use china mirror in binary downloading, original github release downloading in azure china is very slow, it sometimes costs 1 hour to set up k8s cluster by acs-engine, now it only costs 9min.

global
registry proxy in Azure China

dockerhub (docker.io)
dockerhub.azk8s.cn

gcr.io
gcr.azk8s.cn

quay.io
quay.azk8s.cn

All kubenetes related binaries on github could be found under https://mirror.azk8s.cn/kubernetes

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #4126
Special notes for your reviewer:
@jackfrancis @abelgana
If applicable:

[ ] documentation
[ ] unit tests
[ ] tested backward compatibility (ie. deploy with previous version, upgrade with this branch)

Release note:

use china mirror in binary downloading on azure china

@andyzhangx how do you feel about this change in this PR?:
https://github.com/Azure/acs-engine/pull/4104/files#diff-fe31772161e814fe2b4f68df9e62be8cR333
That changes the outbound checks to use URLs that we know we actually need:

k8s.gcr.io
gcr.azk8s.cn

If you approve, let's omit the changes to the outbound check in this PR (the rest lgtm)

@jackfrancis I have rebased your code change, thanks.

/lgtm


All Star Eyes 🤩

Bug report
Please note we will close your issue without comment if you delete, do not read or do not fill out the issue checklist below and provide ALL the requested information. If you repeatedly fail to use the issue template, we will block you from ever submitting issues to Homebrew again.

[ ] ran brew update and can still reproduce the problem?
[ ] ran brew doctor, fixed all issues and can still reproduce the problem?
[ ] ran brew config and brew doctor and included their output with your issue?

What you were trying to do (and why)

What happened (include command output)

Command output

What you expected to happen

Step-by-step reproduction instructions (by running brew commands)

Output of brew config and brew doctor commands

/sparkle

/sparkle

I added this user (bot) to the block list.


Segwit1.0

Replacing segwit0.3 with segwit1.0

I don't delete old versions, they remain in the repo for historical interest/use. Updated PR here: https://github.com/amacneil/docker-bitcoin/pull/24

noted


feat: poe bot

Description
Adds a poe bot.

working
out of the box history support
documentation

Currently has no configuration options. That's a problem all bots currently have. I just think with history, you're multiplying cost, so it should be up to the user.
In Poe's case, it gets more difficult because
    # FIXME: Arguments are automatically loaded by Poebot's ArgumentParser which causes it to fail.
    # the port argument here is also just for show, it actually works because poe has the same argument.

I'm sure there's a way around this. Other than that a filebased config should solve this.
Fixes #
Type of change
Please delete options that are not relevant.

[X] New feature (non-breaking change which adds functionality)
[X] Documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Please delete options that are not relevant.

[ ] Unit Test
[ ] Test Script (please provide)
[X] Tested on poe.com

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

Maintainer Checklist

[ ] closes #xxxx (Replace xxxx with the GitHub issue number)
[ ] Made sure Checks passed

this looks good. merging it.
thanks @cachho for the contribution.


Implements YogiOptimizer

Implement Yogi optimizer in paper Adaptive Methods for Nonconvex Optimization as a tf.keras.optimizers.Optimizer

+1 to this PR!!

@WindQAQ Finished code formatting. PTAL

Fixed a small error.

+1 for this PR!

Any update @WindQAQ @facaiy @Squadrick ?

It seems that the Yogi optimiser was released as part of tensorflow-addons-0.7.0, but I am unable to find any documentation about it. In the case I was just blind, could someone provide a link to a page with the documentation, or in case it is actually missing for the documentation, could you provide a short example describing how it can be used?

It seems that the Yogi optimiser was released as part of tensorflow-addons-0.7.0, but I am unable to find any documentation about it. In the case I was just blind, could someone provide a link to a page with the documentation, or in case it is actually missing for the documentation, could you provide a short example describing how it can be used?

@relaxation82 thank you very much for this callout. Seems the import got missed and thus the API docs were not generated.
I've submitted a PR to fix this (#875) and we will likely do a patch release in the next couple of weeks. In the mean time it can still be used on 0.7.0 with:
from tensorflow_addons.optimizers.yogi import Yogi

The documentation will be whats present in the doc strings of the module:
https://github.com/tensorflow/addons/blob/master/tensorflow_addons/optimizers/yogi.py


[Platform]: rename legend in Cancer DepMap widget

Description
I think "Dependency" is more appropriate than "Unfavourable" in the widget context.

[x] New feature (non-breaking change which adds functionality)

Not tested

I approve the change @chinmehta


Go to definition for Python project does not work with relative imports

Environment data
VS Code version:  Version 1.18.1 (1.18.1)
Python Extension version:  0.8.0
Python Version:  3.6
OS and version:   Mac High Sierra
Actual behavior
Assuming my current project structure:
-stopwatch_app
--test.py
--utils
---greeter.py

And contents of:
# test.py

from utils.greeter import print_greeting

def main():
    print_greeting()

# utils/greeter.py

def print_greeting():
   print('Hello')

In test.py, when I select go to definition for the print_greeting method in main (Figure 1), it automatically places the cursor right before the import definition (Figure 2).
Figure 1

Figure 2

Expected  behavior
It should instead take me to the print_greeting() definition in the utils/greeter.py file.
Steps to reproduce:

Download the source
stopwatch_app.zip
Open the source project in VSCode
Go to test.py, right click on print_greeting() defined in main and select Go To Definition

Logs
Output from Python output panel
N/A

Bump

@imdadahad there's no need to bump the issue as we've triaged it and are aware of your request. We just also have 57 other issues to verify along with 41 bugs awaiting fixes. 😟

Any update regarding this issue?
I downloaded latest version of jedi and configured vscode to use it. Problem still exists.

I downloaded latest version of jedi and configured vscode to use it. Problem still exists.

@pchomik Have you reported this upstream?

I haven't report it upstream.
I don't know how to prepare small example for them to show issue. In my opinion example from vscode is not valid.

Should be fixed in the next release.

it is still not working. Works when starting vsc but stops working after a while
Version 1.23.0
Commit 7c7da59c2333a1306c41e6e7b68d7f0caa7b3d45
Date 2018-05-03T15:23:19.356Z
Shell 1.7.12
Renderer 58.0.3029.110
Node 7.9.0
Architecture x64

Please could you check if there are any errors in the console window (go into Help->Toggle Developer Tools)


php artisan migrate and get the error SQLSTATE[HY000] [1049] Unknown database 'adsboard'

I tried to run
php artisan migrate
and get the error SQLSTATE[HY000] [1049] Unknown database 'adsboard'
But the database adsboard was created with phpmyadmin
Illuminate\Database\QueryException
SQLSTATE[HY000] [1049] Unknown database 'adsboard' (SQL: select * from information_schema.tables where table_schema = adsboard and table_name = migrations and table_type = 'BASE TABLE')
In my .env file
DB_CONNECTION=mysql
DB_HOST=<IP_ADDRESS>
DB_PORT=3306
DB_DATABASE=adsboard   <====
DB_USERNAME=root
DB_PASSWORD=root
In my config->database.php
'mysql' => [
'driver' => 'mysql',
'url' => env('DATABASE_URL'),
'host' => env('DB_HOST', '<IP_ADDRESS>'),
'port' => env('DB_PORT', '3306'),
/*
'database' => env('DB_DATABASE', 'forge'),
'username' => env('DB_USERNAME', 'forge'),
'password' => env('DB_PASSWORD', ''),
*/
'database' => env('DB_DATABASE', 'adsboard'), <======
'username' => env('DB_USERNAME', 'root'),
'password' => env('DB_PASSWORD', 'root'),
        'unix_socket' => env('DB_SOCKET', ''),
        'charset' => 'utf8mb4',     <====== NOT WORKING
        'collation' => 'utf8mb4_unicode_ci', <====== NOT WORKING
        //'charset' => 'utf8mb4', <====== NOT WORKING
        //'collation' => 'utf8mb4_unicode_ci', <====== NOT WORKING
        //'charset' => 'utf8', <====== NOT WORKING
        //'collation' => 'utf8_general_ci',   <====== NOT WORKING
        'prefix' => '',
        'prefix_indexes' => true,
        'strict' => true,
        'engine' => null,
        'options' => extension_loaded('pdo_mysql') ? array_filter([
            PDO::MYSQL_ATTR_SSL_CA => env('MYSQL_ATTR_SSL_CA'),
        ]) : [],
    ],

In my provider / AppServiceProvider
<?php

namespace App\Providers;

use Illuminate\Support\ServiceProvider;
use Illuminate\Pagination\Paginator;
use Illuminate\Support\Facades\Schema; <==== ADDED

class AppServiceProvider extends ServiceProvider
{
    /**
     * Register any application services.
     *
     * @return void
     */
    public function register()
    {
        //
    }

    /**
     * Bootstrap any application services.
     *
     * @return void
     */
    public function boot()
    {
        Paginator::useBootstrap();
        Schema::defaultStringLength(191); <==== ADDED
    }
}

I tried many suggestions from web such as
🙋 Proposal 1
Step 1:
Fire this command inside your project
php artisan optimize:clear
Step 2:
Then start project if in local:
php artisan serve
Database is then connected.
🙋 Proposal 2
Suggestion from @ThandoHlophe
I found the issue, so my ~bash directory had the incorrect $PATH, I updated the $PATH and I was able to sort out the issue

It's an simple error, you still did't created your database in mysql or phpmyadmin named thevinesa
DB_DATABASE=thevinesa
Laravel try to find out the database name thevinesa to connect but they didn't find and database named thevinesa . Try to create database. Then migrate.
php artisan cache:clear
php artisan config:clear

php artisan migrate
command related to database
  migrate:fresh        Drop all tables and re-run all migrations
migrate:install      Create the migration repository
  migrate:refresh      Reset and re-run all migrations
  migrate:reset        Rollback all database migrations
  migrate:rollback     Rollback the last database migration
  migrate:status       Show the status of each migratioin

But not success.

Please let me know what need to be done to make it working.
Thanks

Have you tried running php artisan cache:clear? (https://stackoverflow.com/a/24120924)
Tip: after setting DB_* configurations values in .env file, you don't need to additionally set it in config/database.php, because it loads values from .env file.


Add script to generate API docs using sphinx

Follow the instructions at https://docs.readthedocs.io/en/stable/intro/getting-started-with-sphinx.html to set up API documentation generation for the text_extensions_for_pandas project.
If CI is in place (see #33), augment the integration tests so that they also generate the API documentation and check for errors.

This is done in #39.


NTR - unify article message string and add missing ordernumber for de…

…tail inquiry message

1. Why is this change necessary?
2. What does this change do, exactly?
3. Describe each step to reproduce the issue or behaviour.
4. Please link to the relevant issues (if any).
5. Which documentation changes (if any) need to be made because of this PR?
6. Checklist

[ ] I have written tests and verified that they fail without my change
[ ] I have squashed any insignificant commits
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[ ] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/SW-26040
Please use this issue to track the state of your pull request.

Thanks for your Contribution! 💙


Removing � char

Removing � char. It now reads groups instead of group�s

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

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

@cmendible Thanks for the fix; I added in the intended apostrophe. Merging.


Rewritten in Go

Good to Go? :santa:
git checkout go
./install

Binaries: https://github.com/junegunn/fzf-bin/releases

Very excited about this! Any plans for a windows build (http://www.appveyor.com seems popular)?

@justinmk There are some hurdles for supporting Windows. fzf depends on ncurses which as far as I know not available on Windows platform. Fortunately, there is a cross-platform alternative to ncurses called termbox, but it lacks some features fzf needs, such as proper mouse support and escape key sequences (for alt-something). Those features are not really crucial to fzf, so given that we can live without them on Windows, it's doable. But I don't have a Windows PC for the development, so I could use some help.

There are some hurdles for supporting Windows. fzf depends on ncurses

Would it be possible to decouple the UI from the fuzzy matching engine? Then something like unite.vim could use fzf as a backend, and it would just decorate fzf's output[1]. That's really what I'm interested in--using fzf as a cross-platform component shipped with Neovim.

It should be possible. The core components, Reader, Matcher and Terminal, are already decoupled from each other.


Fix typo in yml files

Checklist

[x] PR contains only changes related.
[ ] README updated (where applicable)
[ ] Tests written (where applicable)

Codecov Report

Merging #5 (2ae5b03) into master (bef75e2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #5   +/-   ##
=======================================
  Coverage   84.88%   84.88%
=======================================
  Files          52       52
  Lines         827      827
  Branches      133      133
=======================================
  Hits          702      702
  Misses        120      120
  Partials        5        5

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


HDFS-15946. Fix java doc in FSPermissionChecker

JIRA: HDFS-15946
Fix java doc for org.apache.hadoop.hdfs.server.namenode.FSPermissionChecker#hasAclPermission.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 57s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
35m  8s

trunk passed

+1 :green_heart:
compile
1m 19s

trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
compile
1m 11s

trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
checkstyle
1m  0s

trunk passed

+1 :green_heart:
mvnsite
1m 21s

trunk passed

+1 :green_heart:
javadoc
0m 52s

trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
1m 26s

trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
spotbugs
3m 15s

trunk passed

+1 :green_heart:
shadedclient
18m 47s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 12s

the patch passed

+1 :green_heart:
compile
1m 16s

the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javac
1m 16s

the patch passed

+1 :green_heart:
compile
1m  8s

the patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
javac
1m  8s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 54s

the patch passed

+1 :green_heart:
mvnsite
1m 15s

the patch passed

+1 :green_heart:
javadoc
0m 45s

the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
1m 16s

the patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
spotbugs
3m 17s

the patch passed

+1 :green_heart:
shadedclient
18m 59s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
369m 57s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 39s

The patch does not generate ASF License warnings.

463m 18s

Reason
Tests

Failed junit tests
hadoop.hdfs.qjournal.server.TestJournalNodeRespectsBindHostKeys

hadoop.hdfs.server.namenode.TestFsck

hadoop.hdfs.server.namenode.ha.TestPipelinesFailover

hadoop.hdfs.server.datanode.TestBlockScanner

hadoop.hdfs.server.namenode.ha.TestBootstrapStandby

hadoop.hdfs.TestPersistBlocks

hadoop.hdfs.server.datanode.TestBlockRecovery

hadoop.hdfs.server.namenode.ha.TestEditLogTailer

hadoop.hdfs.TestDFSShell

hadoop.hdfs.server.namenode.snapshot.TestNestedSnapshots

hadoop.hdfs.TestSnapshotCommands

hadoop.hdfs.server.datanode.fsdataset.impl.TestFsVolumeList

hadoop.hdfs.server.namenode.ha.TestSeveralNameNodes

hadoop.hdfs.server.namenode.TestDecommissioningStatusWithBackoffMonitor

hadoop.hdfs.server.namenode.TestDecommissioningStatus

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2855/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2855

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux 85efa224e9a9 4.15.0-136-generic #140-Ubuntu SMP Thu Jan 28 05:20:47 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / c2904f85c53db30e0af4227562c51d0f06c5d022

Default Java
Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2855/1/testReport/

Max. process+thread count
1966 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2855/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Thanks @liuml07 for the review and merge.


Packager method for packing the heaviest items first

Can you please elaborate more on what's the goal behind this PR? Or what do you want to achieve?


apply ... at ...

Similar to rewrite foo at H it would be nice to have an apply foo at H tactic for forward reasoning. Here would be an example:
constants (A B : Type) (foo : A → B)
example (H : A) : B :=
begin            -- proof state: H : A ⊢ B
  apply foo at H -- proof state: H : B ⊢ B
  assumption
end

How it works in Coq is explained in the Coq manual

8.2.5  apply term in ident

This tactic applies to any goal. The argument term is a term well-formed in the local context and the argument ident is an hypothesis of the context. The tactic apply term in ident tries to match the conclusion of the type of ident against a non-dependent premise of the type of term, trying them from right to left. If it succeeds, the statement of hypothesis ident is replaced by the conclusion of the type of term. The tactic also returns as many subgoals as the number of other non-dependent premises in the type of term and of the non-dependent premises of the type of ident.

If the conclusion of the type of term does not match the goal and the conclusion is an inductive type isomorphic to a tuple type, then the tuple is (recursively) decomposed and the first component of the tuple of which a non-dependent premise matches the conclusion of the type of ident. Tuples are decomposed in a width-first left-to-right order (for instance if the type of H1 is a A <-> B statement, and the type of H2 is A then apply H1 in H2 transforms the type of H2 into B). The tactic apply relies on first-order pattern-matching with dependent types.

The last paragraph is also a useful enhancement for the regular apply function.

I'm planning to mark this issue as postponed and close it.
Unless someone volunteers to implement it.
I will not work on this kind of feature.
My reasons:

It produces cryptic proofs.
We can workaround this command.
My focus is on high-level automation and support what @avigad refers as Johannes-style proof scripts.

That's too bad, but I understand.


WIP: build: generate release with permitted github actions

can't test push actions from external fork


Publish QIR Runtime package as experimental

This change updates the name of the QIR Runtime package to Microsoft.Quantum.Qir.Runtime.Experimental in order to further underscore that the API is in-progress and likely to change. It also removes the workaround that prevented the nuget package from being published since #647 fixed the CFG issues. This will allow our early workloads to begin taking a dependency on this package from the qdk-alpha feed.

This PR is superseded by #663


Certificate Expired

Unable to add / Login users or many any api calls due to certificate expiration -
Issue below -
com.android.org.bouncycastle.jce.exception.ExtCertPathValidatorException:
Could not validate certificate: Certificate expired at Wed Apr 12 19:59:59 EDT 2017 (compared to Thu Apr 13 19:51:59 EDT 2017)

Thanks for the report. The SSL cert has been re-issued.


Reactions are randomly being removed

General Troubleshooting

[ x ] I have checked for similar issues.
[ x ] I have updated to the latest JDA version.
[ x ] I have checked the branches or the maintainers' PRs for upcoming bug fixes.

Bug Report
I have recently been encountering issues with bot reactions. The basic functionality of my bot is that the bot adds a reaction to a message, then performs an action when someone uses that reaction as well (after which the bot removes the user's reaction). Recently, the bot's reactions are also being removed from the message without warning (or mentions in the logs).
I added logic to the bot to automatically add the reactions back if they are missing. The bot notices that the reactions are gone, but when it executes addReaction the reactions are not being added to the message.
Am I being rate-limited without knowing it?
Expected Behavior
The bot's reactions would never be removed from the messages they're attached to.
The bot adds reactions with the following:
matchScoreMessage.addReaction(Unicode.ZERO.getCodePoints()).queue();

The bot removed user reactions with the following:
queueHandler.addMember(event.getGuild(), event.getMember());
 event.getReaction().removeReaction(event.getUser()).queue();

When the bot notices reactions are missing, it does the following which has no result:
log.info("Adding new queue message reactions");
message.addReaction(Unicode.CHECK.getCodePoints()).queue();
message.addReaction(Unicode.X.getCodePoints()).queue();

Code for JDABuilder or DefaultShardManagerBuilder Used
Listeners:
return BotModule_ListenersFactory.proxyListeners(
        getMessageListener(),
        getVoiceListener(),
        getReactionListener(),
        readyListenerProvider.get());

JDABuilder:
JDABuilder
                .create(botComponent.getBotToken(),
                        GatewayIntent.GUILD_MESSAGES, GatewayIntent.DIRECT_MESSAGES, GatewayIntent.GUILD_VOICE_STATES,
                        GatewayIntent.GUILD_MESSAGE_REACTIONS, GatewayIntent.GUILD_EMOJIS, GatewayIntent.GUILD_MEMBERS)
                .addEventListeners(botComponent.getListeners().toArray())
                .build();

Exception or Error
The last thing I see in the logs is Adding new queue message reactions. There are no error messages but no reactions are added to the message.

This is a Discord issue, JDA can't do anything to fix it.

Did something change recently? I've spent the past couple days searching around online but found nothing.

Yes, Discord recently made a change to the backend for reactions I believe and it broke some stuff.

I see, thanks for your help. Are there any resources you could link me to on this that I could take a further look at?

Some sort of statement from discord detailing the issue, or further discussion in other locations mentioning the specifics of the issue and how to mitigate it?

Closing because this isn't a JDA issue, appreciate the help @RedDaedalus


Routing based on Hostname version 2

Second attempt to implement (see #17 )
One thing - I'm not sure about the code in NodeRouter.search(). I feel we should try to find matching route in the domain-specific tree first and if we find it, we should not even bother to go through the "*" ones.
What do you think?

awesome!


docker: add docker image; update makefile

Overview
This PR adds a docker image which builds the celestia-da combined binary including both celestia-node and the gRPC service.
Fixes #23
Checklist

[x] New and updated code has appropriate documentation
[x] New and updated code has new and/or updated testing
[x] Required CI checks are passing
[x] Visual proof for any user facing features like CLI or documentation updates
[x] Linked issues closed with keywords

Summary by CodeRabbit

Chores

Updated .gitignore to exclude the build/ directory from version control.

Refactor

Enhanced the Dockerfile for improved build process and security by setting up a multi-stage build and a non-root user.

New Features

Introduced a new build target in the Makefile for building the application.

Bug Fixes

Modified the clean target in the Makefile to ensure the build directory is properly removed.

Documentation

Added comments in the docker/entrypoint.sh script to clarify the initialization and startup process of the Celestia Node.

Adding docker image publish to Github workflow.


Option to skip deals without expiry date at the sales page

Feature Description
During the large sales often some games do not have an expiry date for a while (or until the last day). Also, when the sale is long, like the Winter sale, some discounts end a few days early, but there's no way to properly check this for now even using the Ends sorting. For example, to see that DLCs to Songs of Conquest and Sands of Salzaar ends tomorrow instead of Jan 2nd, I had to scroll and click the next page until the 49 (out of 63 pages total). I can't imagine how far would I have to click to reach that point for the base game sales page, but from the other sources it looks like at least a couple dozen discounts end today, tomorrow, etc.
Proposed solutions
Option 1: checkmark "hide unknown expiry deals" in filters
Option 2: better pagination that allows to skip pages in bulk, or enter/click the specific page number.

Just sort by expiry.

I've added a fallback during major sales so that it puts expires to the sale end date, this improves sorting.

Just sort by expiry.

That's what I did from the start, still had to click through 49 DLC pages with no expiry cause it goes first in that sorting.

I've added a fallback during major sales so that it puts expires to the sale end date, this improves sorting.

Thank you so much, it indeed improved greatly. Does those games have a specific major sale marker, or is it just applied to everything with no expiry that is up during a major sale?

Just everything. You can increase how many games you see per page btw.


conda-build-all not working correctly on AppVeyor

It seems to building packages that already exist on anaconda cloud:
https://ci.appveyor.com/project/rmcgibbo/conda-recipes/build/job/7xnnlho4u9ih00oq
For example:
Uploading file omnia/pyemma/2.1rc3/win-32/pyemma-2.1rc3-np19py27_0.tar.bz2 ...
Distribution already exists. Please use the -i/--interactive or --force options or `anaconda remove omnia/pyemma/2.1rc3/win-32/pyemma-2.1rc3-np19py27_0.tar.bz2`
Using Anaconda Cloud api site https://api.anaconda.org
[ERROR]
Traceback (most recent call last):
  File "C:\Miniconda\Scripts\anaconda-script.py", line 5, in <module>
    sys.exit(main())
  File "C:\Miniconda\lib\site-packages\binstar_client\scripts\cli.py", line 87, in main
    description=__doc__, version=version)
  File "C:\Miniconda\lib\site-packages\binstar_client\scripts\cli.py", line 66, in binstar_main
    return args.main(args)
  File "C:\Miniconda\lib\site-packages\binstar_client\commands\upload.py", line 206, in main
    callback=upload_print_callback(args))
  File "C:\Miniconda\lib\site-packages\binstar_client\__init__.py", line 416, in upload
    self._check_response(res)
  File "C:\Miniconda\lib\site-packages\binstar_client\__init__.py", line 164, in _check_response
    raise ErrCls(msg, res.status_code)
Conflict: (u'file win-32/pyemma-2.1rc3-np19py27_0.tar.bz2 already exists for package pyemma version 2.1rc3', 409)
detecting package type ...
conda
extracting package attributes for upload ...
done

Actually, I think this is due to some of these packages being under the pre channel:
https://anaconda.org/omnia/pyemma/files
Some of those files are uploaded from rmcgibbo and some from omnia.
@rmcgibbo and @marscher : Whatever is going on with the pre channel stuff for pyemma is fucking up the rest of the builds here.  Any chance you can figure out what is going on?
If not, I think we might have to blow away all the recent pyemma projects and retrigger AppVeyor just to unfuck things for everyone else.

Update:

It looks like the rmcgibbo uploads are because AppVeyor are running under @rmcgibbo's account.
Some win packages are still being correctly built and deployed to anaconda cloud despite the pyemma failures

So is it actually an issue? (since the missing win packages are still being deployed)

Unfortunately I can not view the Appveyor job output (rights problem I'd guess). In principle this problem should not occur anymore, since it is checked whether a package exists in main and pre channel.

The problem was that appveyor did not check against the pre label.

@marsher: Please don't commit directly to the repository!  Always use PRs!

@marscher not @marsher :)

Whoops! My apologies, @marsher. :)


Blueprint Keypad Synch not working anymore

Alarmo Version
v1.8.6
HA Version
2022.3.3
Bug description
Hi
The blueprint I used for my keypad doesn't work anymore with recent update.
I post on the forum here : https://community.home-assistant.io/t/zigbee2mqtt-sync-keypad-and-alarm-control-panel-states/345311/30
@nielsfaber have you an idea ?
Thanks
TLongstride
Steps to reproduce
No response
Relevant log output
No response

try the alarm_control_panel.alarm_disarm (or alarm_arm_away) manually.
If this works OK I guess the blueprint has an issue.
I didn’t change anything regarding this functionality recently..

Thanks for your answer !
It doesn't work, nothing appen when I call manually ..

Are you using alarmo card? If so, which error message is shown?
Otherwise, what do the logs say?

Yes y used Alarmo Card from HACS. Nothing in the logs..
The issue seems to be solved :

I remove alarmo card from HACS Component
Restart
Create new card with Alarmo Card (Yes, was again present instead of removed it ..)
Manual command works great
Remove cache in webrowser
Alarmo Card replace by an error message
Install Alarmo Card From HACS
Try again manualy command

And it works. I don't have explanation
Thanks for your help !
And thanks for the excellent job you've done with your custom_componement !


Update README.md

Docs Build status updates of commit 36acb18:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

README.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


It is an error to UpdateRoot with an undef cid

This also prevents a downstream encoding error where we can not properly encode an undefined CID.
Andrew and I don't think there's any reason why UpdateRoot should allow an undefined CID. Do others feel differently?

Codecov Report

Merging #49 (de83e84) into main (4fde0cc) will decrease coverage by 0.15%.
The diff coverage is 20.00%.

@@            Coverage Diff             @@
##             main      #49      +/-   ##
==========================================
- Coverage   60.82%   60.67%   -0.16%
==========================================
  Files          14       14
  Lines        1205     1208       +3
==========================================
  Hits          733      733
- Misses        369      371       +2
- Partials      103      104       +1

Impacted Files
Coverage Δ

publish.go
58.49% <0.00%> (-3.51%)
:arrow_down:

p2p/protocol/head/head.go
71.05% <50.00%> (ø)

Continue to review full report at Codecov.

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


batching_method=random doesn't seem to work properly

Environment

OS: [Ubuntu 20.04]
Hardware (GPU, or instance type): [T4]
mosaicml-streaming             0.7.4
streaming                      0.1.2

To reproduce
Thank you for the great repo.
I am trying to stream data from multiple sources i have the following code:
streams = [
        Stream(
            remote=remote_uris[i],
            local=os.path.join(local_dir, local_dirs_suffixes[i]),
            split=None,
            proportion=0.1 #for 10 streams
        ) for i in range(len(remote_uris))
    ]

dataset = StreamingDataset(
                streams=streams, #10 different streams
                batch_size=64,
                batching_method="random",
                predownload=8*64,
                shuffle=True,
                shuffle_algo="py1s",
                shuffle_block_size=100_000,
                num_canonical_nodes=1,
                cache_limit="100gb",
                epoch_size=10*64,
            )

dataloader = StreamingDataLoader(
               dataset = dataset,
               batch_size = 64,
               shuffle = False,
               drop_last = True,
               num_workers = 8,
               prefetch_factor = 2
        )

When going through different batch from the dataloader, the batches dont seem correclty shuffled.

batch1: [StreamC, StreamC, StreamC, StreamC, StreamC, StreamC, StreamD, ..., StreamC]
batch2: [StreamF, StreamF, StreamF, StreamF, StreamF, StreamC, StreamD, ..., StreamF]
batch3: [StreamB, StreamA, StreamB, StreamA, StreamB, StreamA, StreamB, ...., StreamA]
......

Expected behavior
Real random distribution across the 10 different streams.

batch1: [StreamC, StreamA, StreamB, StreamF, StreamE, StreamH, StreamD, ..., StreamC]
batch2: [StreamF, StreamG, StreamD, StreamF, StreamA, StreamC, StreamB, ..., StreamB]
batch3: [StreamE, StreamA, StreamB, StreamC, StreamD, StreamH, StreamB, ...., StreamE]
......

Hey @oscarfossey, seems like you’re setting num_canonical_nodes to 1 and using py1s shuffle. For the py1s and py2s shuffle algos, num_canonical_nodes is the only parameter that controls shuffle strength — shuffle_block_size is not used. So in this case, if you want to keep the py1s shuffling algo, increase num_canonical_nodes. Otherwise, switch to the py1e or py1br shuffle algos, and increase the shuffle_block_size to get a stronger shuffle.
Hope this helps!


Dummy release

Assuming:

f3d-app/f3d is ready to be released
f3d-app/f3d CI is green
f3d-app/f3d-superbuild CI is green and nightly are generated

Steps needed with current status:

Commit  vX.Y.Z-RC1 version change in f3d-app/f3d
Tag vX.Y.Z-RC1 in  f3d-app/f3d
Add X.Y.Z-RC1 in f3d-app/f3d-superbuild versions.cmake, run MR CI, merge
Tag vX.Y.Z-RC1 f3d-app/f3d-superbuild
Trigger a release build with X.Y.Z-RC1 input
Locally test the RC release
repeat until test is sucessful
Commit vX.Y.Z version change in f3d-app/f3d
Tag vX.Y.Z in  f3d-app/f3d
Add X.Y.Z in f3d-app/f3d-superbuild versions.cmake, remove all RC* from versions.cmake run MR CI, merge
Tag vX.Y.Z f3d-app/f3d-superbuild
Trigger a release build with X.Y.Z input
Release

Results contains many steps and "unbuildable" tags in f3d-app/f3d-superbuild
Alternative steps with needed changes, assuming f3d-app/f3d-superbuild main branch is compatible with all f3d-app/f3d RCs :

Commit  vX.Y.Z-RC1 version change in f3d-app/f3d
Tag vX.Y.Z-RC1 in  f3d-app/f3d
Trigger a release build with vX.Y.Z-RC1 f3d tag input but superbuild main branch input
Locally test the RC release
repeat until test is sucessful
Commit vX.Y.Z version change in f3d-app/f3d
Tag vX.Y.Z in  f3d-app/f3d
Add X.Y.Z in f3d-app/f3d-superbuild versions.cmake, wait for CI, merge
Tag vX.Y.Z f3d-app/f3d-superbuild
Trigger a release build with vX.Y.Z f3d tag input and X-Y-Z superbuild tag input
Release

wdyt @Meakk

Current process is as follows:
Release Candidates :

[ ] Commit vX.Y.Z-RCN version change in f3d-app/f3d
[ ] Tag vX.Y.Z-RCN in f3d-app/f3d
[ ]  Trigger a release build with vX.Y.Z-RCN F3D version, superbuild main branch and prerelease publish true.
[ ] Locally test the RC release on Linux, MacOS and Windows
[ ] If fails, fix and start again with N++
[ ] Else proceed

Release :

[ ] Commit vX.Y.Z version change in f3d-app/f3d
[ ] Tag vX.Y.Z in f3d-app/f3d
[ ] Add X.Y.Z in f3d-app/f3d-superbuild versions.cmake, wait for CI, merge
[ ] Tag vX.Y.Z f3d-app/f3d-superbuild
[ ] Trigger a release build with vX.Y.Z F3D version, vX.Y.Z sb version and prerelease publish true
[ ] Write release notes
[ ] Release
[ ] Communicate

wdyt @Meakk ?


Satoshi denominated tpos

I wanted it (#58) so I tried to do it. Some really straighforward changes to quickly accomplish denomination in satoshi.

Thanks for the review and suggestions!

The format logic is replicated in 6 places. It should be extracted to one util method:
function formatAmount(amount, currency) {
  if (currency == "satoshi") {
    return LNbits.utils.formatSat(amount) + " sat";
  }
  return LNbits.utils.formatCurrency(Number(amount).toFixed(2), currency);
}

Thanks for the suggestions, @motorina0 . Hope I addressed them correctly. At least it still works when I test it. Also spotted one missed ${totalfsat} to hide.

LGTM, can you please solve the conflict (templates/tpos/index.html)

Thanks for the heads up. Hadn't noticed. Done.


Nothing happens after queueing songs

I just set up the latest version of NadekoBot on OS X 10.11.5, installing both ffmpeg and opus-tools using homebrew. Upon using "!m q" to queue a video, Nadeko queues the song, but won't connect to a voice channel.

Nadeko has the necessary permissions
ffmpeg and opus are installed and updated

After waiting less than a minute, this message was logged in the console continuously.
Severity:` ErrorMessage: Error handling VOICE_STATE_UPDATE eventExceptionMessage: Object reference not set to an instance of an object

I think this is the exact same issue other people hosting on mac had. Nobody I know of managed to get music working on a mac
Maybe try building mono from source?

Recompiling the latest version of Mono from GitHub worked.

Awesome, great news! I will add it to FAQ, since we don't have a mac guide yet. If you feel like making one, that would be great ;p

I'll write I guide when I get home this afternoon.


Duplicate vim bundle breaks javascript indenting

Just recently did a rake update and found smart/auto indenting in VIM for javascript files broken.
After some investigation I believe I narrowed it down to a re-emergence of the issue of mxw/vim-jsx being added as a duplicate in this file here.
In the commit where mxw/vim-jsx was added, this comment highlighted the issue that I am now experiencing again.
In short mxw/vim-jsx is already included in sheerun/vim-polyglot and the duplicate plugin is causing issues. When I remove mxw/vim-jsx everything works as expected.
That plugin was then removed in this commit by @lfilho resolving the issue.
However, it seemed this issue was re-introduced in this commit when mxw/vim-jsx was re-added by @skwp .
I'm guessing it was re-added by accident, not realizing the issue it previously caused.
Let me know if I'm missing anything. Otherwise, I think the fix is just simply re-removing mxw/vim-jsx!

I removed the mxw/vim-jsx and am still having issue with indentation. Whenever I type {}, there is no newline + no indentation.

@AdrianArtiles found this b/c I had the same issue!

I can confirm that removing vim-jsx fixed the issue for me too, and doing so there is no newline + indent when doing if (...) { + Enter.

Hi folks, I'm using my own fork of this repo, which has drifted a lot so I can't test and debug this.
If someone one finds a permanent solution and others help test it and confirm the solution here, I'd gladly help review and merge that CR.


Nodes never decrements their remaining ticks

Bug reporting
So I am using the raft component of etcd. I have a recurring issue that breaks the ability of my raft cluster to commit.
Scenario:
4 node cluster. There is a heavy commit load being generated. Load is generated via a go http server that writes into raft. I am seeing that 2/4 of my raft nodes repeatedly have errors like below, that have lasted for days.
raft2017/08/31 21:22:15 INFO: a7da9f [logterm: 170435, index: 1100022, vote: afb7d2] ignored MsgVote from b01102 [logterm: 170435, index: 1100022] at term 170435: lease is not expired (remaining ticks: 9)
So essentially my cluster remains inoperable because for some reason two nodes never decrease their remaining ticks and therefore never think there least has expired. I have pprof enabled if any of those metrics in particular would be helpful.
Here are top15 for the profile, and top 10 for the heap
CPU:
(pprof) top15
1.32s of 1.32s total (  100%)
Showing top 15 nodes out of 43 (cum >= 0.01s)
flat  flat%   sum%        cum   cum%
0.53s 40.15% 40.15%      1.22s 92.42%  runtime.scanobject
0.38s 28.79% 68.94%      0.38s 28.79%  runtime.heapBitsForObject
0.31s 23.48% 92.42%      0.31s 23.48%  runtime.greyobject
0.03s  2.27% 94.70%      1.26s 95.45%  runtime.gcDrain
0.01s  0.76% 95.45%      0.01s  0.76%  math/big.mulAddVWW
0.01s  0.76% 96.21%      0.01s  0.76%  p256SubInternal
0.01s  0.76% 96.97%      0.01s  0.76%  runtime.(*gcSweepBuf).pop
0.01s  0.76% 97.73%      0.01s  0.76%  runtime.globrunqget
0.01s  0.76% 98.48%      0.01s  0.76%  runtime.lock
0.01s  0.76% 99.24%      0.01s  0.76%  runtime.runqempty
0.01s  0.76%   100%      0.02s  1.52%  runtime.schedule
0     0%   100%      0.01s  0.76%  crypto/elliptic.(*p256Point).p256ScalarMult
0     0%   100%      0.01s  0.76%  crypto/elliptic.p256Curve.ScalarMult
0     0%   100%      0.01s  0.76%  crypto/elliptic.p256PointAddAsm
0     0%   100%      0.01s  0.76%  crypto/rsa.(*PrivateKey).Sign
(pprof)
HEAP:
(pprof) top
601.16MB of 607.08MB total (99.02%)
Dropped 445 nodes (cum <= 3.04MB)
Showing top 10 nodes out of 56 (cum >= 148.10MB)
flat  flat%   sum%        cum   cum%
196.01MB 32.29% 32.29%   196.01MB 32.29%  runtime.rawstringtmp
148.10MB 24.40% 56.68%   148.10MB 24.40%  bytes.makeSlice
109.52MB 18.04% 74.72%   109.52MB 18.04%  reflect.unsafe_New
76.50MB 12.60% 87.32%    76.50MB 12.60%  runtime.hashGrow
34MB  5.60% 92.92%       34MB  5.60%  container/list.(*List).PushBack
11MB  1.81% 94.74%       11MB  1.81%  reflect.unsafe_NewArray
10.51MB  1.73% 96.47%    88.01MB 14.50%  runtime.mapassign
10MB  1.65% 98.11%       10MB  1.65%  runtime.convT2E
5.52MB  0.91% 99.02%    14.53MB  2.39%  main.(*pendingStore).insert
0     0% 99.02%   148.10MB 24.40%  bytes.(*Buffer).Write
(pprof)
I am wondering if anything like this has been encountered before?
Cheers,
Robin

I think this may be related to some other bugs in our usage of the raft library. Can probably just close this


Add license?

This looks like a handy component to use, but I'm very hesitant to actually utilize it since without an explicit license this source code defaults to "All Rights Reserved".

Added an MIT license - thanks for the reminder!


Adding logstash module

need help adding logstash for data ingest and making it an ELK stack with the same module setup.

I would be grateful if you added a logstash node as well.


Can't not load current tag after add an tag to blacklist

Steps to reproduce

Search an tag
Click on random Image to Open an Detail Windows
Right-Click to an Tag on Detail Windows just opened, choose "Open new tab", the grabber now Loading this new Tab (but incomplete) then close this tab
Back to Detail Windows, Select and add an tag to Black List
Press Enter to refresh current Searching tag in step 1
Bug: The grabber now not loading anymore (keep "Enter" to refresh but still not load", must close grabber and re-open to reload

System information

OS: [e.g. Windows 10]
Grabber version: 7.9.0 (x64)

the grabber now Loading this new Tab (but incomplete) then close this tab

What do you mean by "incomplete"?
I just tried going through this flow and didn't meet issues.  And did you blacklist the tag you were searching or another one? (maybe all results were filtered?)