Cannot used assumed types?

This:
assume val ty : Type

```pulse
fn test (x : ty)
  requires emp
  returns x:int
  ensures emp
{
  1
}

Fails with
Could not infer implicit arguments in x: ty -> stt int emp (fun _ -> emp) (in file <dummy>)

Running in the CLI also shows this:
(Error 339) Internal error: unexpected unresolved (universe) uvar in deep_compress: 7
<dummy>(0,0-0,0): (Error) Could not infer implicit arguments in x: Bug.ty -> Pulse.Lib.Core.stt Prims.int Pulse.Lib.Core.emp (fun _ -> Pulse.Lib.Core.emp)

The issue here is that Pulse does not support universe polymorphic definitions. We should at least provide a better error.


Make MKSlider's onPanResponderTerminationRequest prop ()=>false or configurable

Right now MKSlider has onPanResponderTerminationRequest property hardcoded to ()=>true (link). I propose that it should be either ()=>false or configurable.
The property onPanResponderTerminationRequest specifies how the component behaves if another component wants to become the responder for the current gesture. (It's not documented well on the RN's PanResponder page but a similar documented property onResponderTerminationRequest exists in the gesture responder system docs).
In my app I'm placing a MKSlider component in a View that already responds to gesture events (with a PanResponder). However I'd like the MKSlider to have higher priority than the parent view.
I think that having a slider that doesn't release focus is the most common use case. Actually I can't think of a situation where a MKSlider component would want to stop being the gesture responder. Here is a bug that happens when a SliderIOS component allows other gesture events: https://github.com/facebook/react-native/issues/1046#issuecomment-170347423 .
What do others think?

I'v experimented it with my demo app, which have multiple sliders in a same screen, can't see any difference.
Can you post your code here, and what's your expectation of the behavior of the sliders?

It's not about multiple sliders on one screen, it's about a slider inside a view that also responds to gesture events. Sorry, preparing a code sample is too difficult for me at the moment (unrelated technical issues). Will update this thread if this changes.
A similar issue was present in RN's SliderIOS and this commit may have more documentation.
https://github.com/facebook/react-native/commit/52ddfd9b51cbaa05176efc8c9dfa7b581be01a97


DBZ-2827 fix try-with-resources block should not be used when OkHttp Response object is returned

fix try-with-resources block should not be used when OkHttp Response object is returned in DebeziumContainer

closes https://issues.redhat.com/browse/DBZ-2827

@rk3rn3r APplied thanks. Interesting that there is not a compiler warning for that.


Common: Add ipv6 support

e8187f6 does not fix the ipv6 as expected since ansible_default_* are
filled with the IP address carried by the network interface used by the
default gateway route. By the way, it assumes that the MON_IP address will
be this IP address which is not always the case.
We need to keep using the previous fact but add some intelligence in the
template to determine how to retrieve the ipv4|ipv6 address since the path
to the fact in hostvars is not the same according to ipv4 vs ipv6 case.
Signed-off-by: Guillaume Abrioux<EMAIL_ADDRESS>

BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1451786

jenkins test jewel-ansible2.2-update_cluster

jenkins test kraken-ansible2.2-xenial_cluster

jenkins test kraken-ansible2.2-centos7_cluster

jenkins test jewel-ansible2.2-dmcrypt_journal

jenkins test jewel-ansible2.2-docker_cluster

jenkins test jewel-ansible2.2-purge_dmcrypt

jenkins test jewel-ansible2.2-xenial_cluster

jenkins test kraken-ansible2.2-purge_docker_cluster

jenkins test jewel-ansible2.2-xenial_cluster

jenkins test jewel-ansible2.2-dmcrypt_journal

jenkins test jewel-ansible2.2-dmcrypt_journal

jenkins test kraken-ansible2.2-journal_collocation

jenkins test jewel-ansible2.2-purge_cluster

jenkins test kraken-ansible2.2-xenial_cluster

jenkins test kraken-ansible2.2-purge_cluster

jenkins test kraken-ansible2.2-update_docker_cluster

jenkins test jewel-ansible2.2-update_docker_cluster

jenkins test jewel-ansible2.2-centos7_cluster

jenkins test kraken-ansible2.2-update_docker_cluster

jenkins test jewel-ansible2.2-update_docker_cluster

jenkins test jewel-ansible2.2-update_docker_cluster

jenkins test jewel-ansible2.2-update_docker_cluster

jenkins test jewel-ansible2.2-update_docker_cluster

jenkins test jewel-ansible2.2-update_docker_cluster

jenkins test jewel-ansible2.2-update_docker_cluster

jenkins test kraken-ansible2.2-update_docker_cluster

jenkins test jewel-ansible2.2-purge_cluster

jenkins test jewel-ansible2.2-xenial_cluster

jenkins test kraken-ansible2.2-dmcrypt_journal_collocation

jenkins test kraken-ansible2.2-purge_docker_cluster

jenkins test jewel-ansible2.2-update_cluster

jenkins test jewel-ansible2.2-docker_cluster

jenkins test jewel-ansible2.2-update_docker_cluster

jenkins test jewel-ansible2.2-purge_dmcrypt

jenkins test kraken-ansible2.2-update_docker_cluster

jenkins test jewel-ansible2.2-purge_cluster

jenkins test kraken-ansible2.2-purge_cluster

jenkins test jewel-ansible2.2-update_docker_cluster

jenkins test jewel-ansible2.2-purge_docker_cluster

jenkins test jewel-ansible2.2-purge_docker_cluster

jenkins test jewel-ansible2.2-update_docker_cluster

jenkins test kraken-ansible2.2-update_docker_cluster

jenkins test kraken-ansible2.2-xenial_cluster

jenkins test kraken-ansible2.2-docker_dmcrypt_journal_collocation

jenkins test kraken-ansible2.2-purge_docker_cluster

jenkins test jewel-ansible2.2-docker_cluster

jenkins test jewel-ansible2.2-update_cluster

jenkins test kraken-ansible2.2-purge_docker_cluster

jenkins test jewel-ansible2.2-update_docker_cluster

jenkins test kraken-ansible2.2-update_cluster

jenkins test kraken-ansible2.2-centos7_cluster

jenkins test kraken-ansible2.2-centos7_cluster

jenkins test kraken-ansible2.2-centos7_cluster

jenkins test kraken-ansible2.2-docker_dedicated_journal

jenkins test jewel-ansible2.2-docker_dedicated_journal

@guits is there something in particular you want me to look at?

@batrick This commit removes the monitor_address_block variable. I just wanted to ensure that setting monitor_address in inventory host file can be equivalent for your use-case (bcb2e39)

No that won't work. We don't know what the private address will be for each monitor so it cannot be set in a group variable. All we know (via monitor_address_block) is that each monitor address is in a certain IP range.

@batrick ok, thanks for your feedback, so I'll restore this.


Test isPrimary after migration

There was previously a bug here (fixed on al-swift)

Done in b5b7874.


Cherry-picks for the 4.1 branch (future 4.1.4) - 3rd batch

Conservative subset of #89339.

Looks fine to me!


Log error on command failures (non-zero exits)

Currently, when command fails (such as ManagedCloudSDK.isUpToDate()) fails, there is no insight into detailed error message, even though, error message is captured in CommandExitException.
https://github.com/GoogleCloudPlatform/appengine-plugins-core/blob/master/src/main/java/com/google/cloud/tools/managedcloudsdk/ManagedCloudSdk.java#L162
Suggest that upon CommandExitException, log error with high priority and/or propagate to the wrapping Exception, or, use it as part of the message of CommandExitException

https://github.com/GoogleCloudPlatform/app-maven-plugin/issues/409
oogleCloudPlatform/cloud-code-intellij#2662

This makes sense though I don't think it's diagnosed quite right. The problem is that CommandCaller doesn't save stderr into the exception. It should.
Logging here we don't need to do. Including more information in the exception is the right approach.

@elharo it does save error log into CommandExitException. I can see the error message.

Although, the error should probably to be logged and/or surfaced by the consumer of this API instead.

That's a decision for the dependent of this library to make. This library does not attempt to talk to end users.

As described here, I'm no longer sure there is an issue for appengine-plugins-core to resolve. This library makes that information available downstream. It's up to the individual callers to make use of it. Perhaps this bug should be addressed in app-maven-pluign and cloud-code-intellij?


add .gitignore

it is a good practice to add a .gitignore file so accidentally avoid checking in unwanted files.
also facilitates ease of selecting changed files.

created pull request:
https://github.com/googlesamples/android-NetworkConnect/pull/5

Looks like our internal source already has a .gitignore, but it's not being exported correctly.


Typecheck the bodies of untyped functions

Via is getting a warning (not an error) from mypy:
via/services/__init__.py:79: note: By default the bodies of untyped functions are not checked, consider using --check-untyped-defs  [annotation-unchecked]

The warning seems to happen when a function has no type annotations in
its signature but some type annotations in its body (for local
variables).
I don't see why we wouldn't want to typecheck the bodies of functions?
Is there a reason mypy disables this by default?

I don't see why we wouldn't want to typecheck the bodies of functions?
Is there a reason mypy disables this by default?

I reckon is a way to opt-in into the type annotations function by function


Dimension issue for data_TDIUC train.hdf5 and val.hdf5 features file

Hi,
For TDIUC dataset train and val dimension of 'image_features' is: [3339639,2048], [1675180,2048], though total number of images is - 122429. and 57565 respectively.
Please help me to understand the dimension of train.hdf5 and val.hdf5 here, I will be thankful to you.

Hi ak-kat,
The first dimension of 'image_features' is the total number of detected bounding boxes (35 bounding boxes for each image in the average), it is not the total number of the image.

Thanks @huytran127 for your reply. 3339639 implies ~27 BB.
122429 x 35 = 4285015. I earlier thought you would have considered 36 BB, but still.
Expected dimension -: [122429,35,2048], if I am not wrong.

If I set the number of bounding boxes in faster-RCNN is fixed 35 boxes, the expected dimension is [122429,35,2048]. However, the number of bounding boxes I set in faster-RCNN is dynamic.


Observers not firing in feature/integration tests using Rails/Capybara

My observers work fine in my app and in my regular tests, but Capybara feature tests do not trigger any observers, no matter what I try. Has anyone dealt with this or is this a known or intentional behavior?
I'm using the latest version of RSpec, Capybara (github masters) and Rails 5.0.0beta3.

Ask questions on the mailing list as requested in the README

Sorry about that, and done.

Linking to the mailing list post:
https://groups.google.com/forum/#!searchin/ruby-capybara/rails-observers/ruby-capybara/lRb_i_52O6g/M5lHyjKoKQAJ


Just installed, get this on startup...

Didn't see mention of dependency?
=> Started proxy.
=> Started MongoDB.
W20150518-10:36:05.381(-4)? (STDERR) WARNING:
W20150518-10:36:05.382(-4)? (STDERR) cfs:graphicsmagick could not find "graphicsMagic" or "imageMagic" on the
W20150518-10:36:05.382(-4)? (STDERR) system.
W20150518-10:36:05.382(-4)? (STDERR)
W20150518-10:36:05.382(-4)? (STDERR) I just checked PATH to see if I could find the GraphicsMagick or ImageMagic
W20150518-10:36:05.382(-4)? (STDERR) unix/mac os/windows binaries on your system, I failed.
W20150518-10:36:05.382(-4)? (STDERR)
W20150518-10:36:05.382(-4)? (STDERR) Why:
W20150518-10:36:05.382(-4)? (STDERR) 1. I may be blind or naive, help making me smarter
W20150518-10:36:05.382(-4)? (STDERR) 2. You havent added the path to the binaries
W20150518-10:36:05.382(-4)? (STDERR) 3. You havent actually installed GraphicsMagick or ImageMagick
W20150518-10:36:05.383(-4)? (STDERR)
W20150518-10:36:05.383(-4)? (STDERR) *** Make sure "$PATH" environment is configured "PATH:/path/to/binaries" ***
W20150518-10:36:05.383(-4)? (STDERR)
W20150518-10:36:05.383(-4)? (STDERR) Installation hints:
W20150518-10:36:05.383(-4)? (STDERR) * Mac OS X "brew install graphicsmagick" or "brew install imagemagick"
W20150518-10:36:05.383(-4)? (STDERR) * Linux download rpm or use packagemanager
W20150518-10:36:05.383(-4)? (STDERR) * Centos "yum install GraphicsMagick"* Windows download the installer and run
I20150518-10:36:06.950(-4)? Meteor._wrapAsync has been renamed to Meteor.wrapAsync
=> Started your app.
=> App running at: http://localhost:3000/

You havent actually installed GraphicsMagick or ImageMagick

Yes, I figured that out.  I don't recall that it was in the build instructions as a requirement.  I just used homebrew and it's now fine.

I gave the same issue. Both packages are in meteor packages, and installing them via homebrew doesn't seem to help. Any ideas why would this happen?

Nevermind, it was a brew issue.


Set TA to 2 hours playtime requirement

About the PR

mlem
Why / Balance
In light of recent events

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

no

Please give a better explanation.

Please give a better explanation.

Consistent raids by 0-hour accounts that pick exclusively TA, breaking into places or mass sabotaging roundstart, this causes panic bunkers to be enabled almost every single (other) day.

If the issue is 0 hour accounts why not just set it to 1 hour?
With this logic cadet should also be 2 hours since they have guns.
Also, breaking into places and mild tomfoolery are not the end of the world, sec can easily deal with it.

If the issue is 0 hour accounts why not just set it to 1 hour? With this logic cadet should also be 2 hours since they have guns. Also, breaking into places and mild tomfoolery are not the end of the world, sec can easily deal with it.

cadets are at 10 hours playtime requirement.

I think this isn't a bad idea even without the raids, since 2 hours is like just one or two shifts and it'd be better for players to get a slight grasp on how the controls and rounds work before getting handed hardsuits and full tool sets

I brought this up and we're not going to go with this change. It's a band-aid fix and not a great one at that. We do have ideas to fix the core issue, but this aint it.

get asked for better explanation, slapped DNM

provide

DNM stays

https://discord.com/channels/310555209753690112/770682801607278632/1162479935659704452

mfw slapped closed when I was typing


Fixes typo in javadocs for Datastore's BaseEntity class.

Replaces "...if not such property" with "...if no such property."

I am a Googler, covered by the Corporate CLA.


Godot 4: all the functions in Visual Script are not returning their value type correctly

Godot version
v4.0.alpha3.official [256069eaf]
System information
Windows 11, Vulkan Clustered, Intel UHD Graphics 620 v<IP_ADDRESS>0
Issue description
All the functions in Visual Script are not returning their value type correctly, for example, the get_child_count() should return its value as int but it returns as any instead, as shown in the screen recording below:
https://user-images.githubusercontent.com/1403194/155761356-e11a2996-963d-428c-9653-5bed83b88b8d.mp4
This is a regression from Godot 3.4. In Godot 3.4, this is working fine as shown in the screen recording below:
https://user-images.githubusercontent.com/1403194/155764076-f88277d7-e78f-4c98-a01e-c188898a59f7.mp4
Note: I don't know whether this issue is caused by issue #58069 or whether it's related to issue #58035.
To work around this issue, I will have to:

Change the function call mode to Instance.
Delete its base script due to issue #58069.
Duplicate the function call node. Then, the return value type will be set correctly as shown in the screen recording above.

Steps to reproduce

Add a Visual Script function.
See its value return type, it will always be any.

Minimal reproduction project
No response

VisualScript was removed in Godot 4.0, so closing.


Benchmark inaccurate

To run the benchmarks I had to lock jsdom to npm install -D<EMAIL_ADDRESS>because benv doesn't specify a version range, and the new jsdom has deprecated methods that were being used by benv.
Anyways, the benchmarks consistently showed rfdc as the fastest approach.
λ ~/Projects/fast-clone/ master* npm run benchmark

><EMAIL_ADDRESS>benchmark /Users/jack/Projects/fast-clone
> ts-node --files ./support/benchmark

Generating random json data...
Done.

Test data size is:  7.15 MB

Running benchmark 1 of 7... Done
Running benchmark 2 of 7... Done
Running benchmark 3 of 7... Done
Running benchmark 4 of 7... Done
Running benchmark 5 of 7... Done
Running benchmark 6 of 7... Done
Running benchmark 7 of 7... Done

Library          | Time
-----------------|---------
clone            | 2,181 ms
angular.copy     | 1,715 ms
snapshot         | 1,182 ms
lodash.cloneDeep |   197 ms
fast-clone       |   149 ms
deepClone        |   139 ms
rfdc             |    35 ms

node --version
v10.15.3

Note I added https://github.com/davidmarkclements/rfdc to the benchmark.

Note that rfdc benchmark had the following result, which placed this package last:
λ ~/Projects/rfdc/ master* npm run bench

><EMAIL_ADDRESS>bench /Users/jack/Projects/rfdc
> node benchmark

benchDeepCopy*100: 748.877ms
benchLodashCloneDeep*100: 1893.592ms
benchFastClone*100: 3946.849ms  <------------
benchFastCopy*100: 1056.753ms
benchRfdc*100: 427.176ms
benchRfdcProto*100: 435.255ms
benchRfdcCircles*100: 490.022ms
benchRfdcCirclesProto*100: 487.759ms
benchDeepCopy*100: 750.964ms
benchLodashCloneDeep*100: 1955.513ms
benchFastClone*100: 4271.341ms  <------------
benchFastCopy*100: 1088.407ms
benchRfdc*100: 528.183ms
benchRfdcProto*100: 529.741ms
benchRfdcCircles*100: 497.151ms
benchRfdcCirclesProto*100: 486.490ms

fast-clone ran in 3946ms vs rfdc which ran in 427ms
According to those benchmarks this package is 9x as slow.  Just wanted to make sure you were aware.

I updated my test result above with accurate output, fast-clone is 3rd with 149ms, deepClone 2nd with 139ms, and rfdc 1st with 35ms.  https://github.com/codeandcats/fast-clone/issues/27#issue-454866524

Thanks for that nifty.
This library was the fastest I could find on npm at one point in time but over the years other libraries have caught up and even surpassed it in speed.
I recently had a go at completely rewriting it in order to improve performance and add some features at the same time, but it was a trade off between features and perf and I wasn't prepared to sacrifice functionality.
I've been thinking that I'll probably just deprecate the library.

You could put in your deprecation notice to use rfdc, and then publish a new version 1.5.14 that uses rfdc internally or something.  I'm sure GitHub support could help setup a redirect from fast-clone to rfdc.

You could also use the "ARCHIVE" feature of GitHub and put up a deprecation notice in README and via npm deprecate


npm publish sends the wrong files

I have no idea what's going on.
I have a project in coco (node-jinjs, here on github) that I compile to javascript.
In my package.json, I explicitely say that I want lib :
    "directories" : { "lib" : "./lib"  },
    "bin": {
        "jinjs": "./lib/command.js"
     },

yet when publishing, npm sends everything in my folder EXCEPT lib, which makes it all fail quite miserably when installing.
I had to create an empty folder, copy package.json and lib/ there and only then did it work.
npm --version
1.1.0-3

node --version
v0.6.9

Do you have a .gitignore which excludes lib?
If so, add an empty .npmignore file.

I do.
Isn't that a bit counter intuitive ?

If there's a .gitignore file, and no .npmignore file, then npm will not publish the files specified by the .gitignore file.
Most people seem to feel that it's counterintuitive to publish something that's in the .gitignore, which is why the feature was added.  If there's a .npmignore, then it'll go with that instead.

I understand, but I find it a little redundant since there is already npmignore if you want to ignore files.
Plus, npm not being git, I can't really understand why it has business looking at this file.
Anyways, may I suggest add an obvious statement in the package building documentation ? Or at least in some kind of FAQ ? I lost quite some time today trying to understand why my package was not published correctly (switched from mercurial to git, all hell broke loose then).
Also, since it couldn't find the asked directories (because of the gitignore), why didn't it fail instead of packaging everything BUT what was asked ? This at least would need to change (a warning maybe ?)

Yeah, this was documented once upon a time, but appears to have been lost.
Thanks for pointing that out.  It does belong in the publish documentation (or maybe in pack), and there should be a pointer in the faq.


Issue #1439 - Add sys/memory endpoint support

Problem: sys/memory endpoint missing
Solution: This PR adds the endpoint
Files Changed:

f5/bigip/tm/sys/init.py (changed)
f5/bigip/tm/sys/memory.py (added)
f5/bigip/tm/sys/test/functional/test_memory.py (added)
f5/bigip/tm/sys/test/unit/test_memory.py (added)

lgtm


Fix invalid xacro properties

Addresses #165

Codecov Report

Merging #166 (605bb36) into main (1b121b7) will decrease coverage by 0.12%.
The diff coverage is 0.00%.

@@           Coverage Diff            @@
##            main    #166      +/-   ##
========================================
- Coverage   3.59%   3.46%   -0.13%
========================================
  Files         13      13
  Lines        947     982      +35
  Branches     152     152
========================================
  Hits          34      34
- Misses       843     878      +35
  Partials      70      70

Flag
Coverage Δ

unittests
3.46% <0.00%> (-0.13%)
:arrow_down:

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

Impacted Files
Coverage Δ

...ur_controllers/force_torque_sensor_broadcaster.hpp
0.00% <ø> (ø)

...controllers/scaled_joint_trajectory_controller.hpp
0.00% <0.00%> (ø)

...ontrollers/src/force_torque_sensor_broadcaster.cpp
0.00% <0.00%> (ø)

ur_controllers/src/gpio_controller.cpp
0.00% <0.00%> (ø)

...rollers/src/scaled_joint_trajectory_controller.cpp
0.00% <0.00%> (ø)

...ontrollers/src/speed_scaling_state_broadcaster.cpp
0.00% <0.00%> (ø)

...ver/include/ur_robot_driver/hardware_interface.hpp
0.00% <ø> (ø)

ur_robot_driver/src/hardware_interface.cpp
0.00% <0.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 6ca5971...605bb36. Read the comment docs.


Duplicated $black variable in _colors.scss

$black definition on line 20 is immediately overwritten by line 21.
https://github.com/jxnblk/basscss/blob/v5.2.1/scss/_colors.scss#L20-L21

This is because $black is defined here and then again here and the CSS to SCSS converter does not check for these overwritten variables. So I guess it's an issue with the compiler.

Thanks. I'll probably be removing the custom black color soon and using the value from colors.css. Is this causing any issues on your end?

I've removed the duplicate --black custom properties from the source and am in the process of moving the transpiled Sass partials to a new repo: https://github.com/basscss/basscss-sass

Thanks for the heads up!


Polling doesn't work as expected

I realize that this package is meant to be used with MobX. However, I would have expected it to work with manual polling, too.
Consider this Node code:
const { computedAsync } = require('computed-async-mobx');

function delay(ms) {
  return new Promise(resolve => setTimeout(resolve, ms));
}

async function main() {
  const observableValue = computedAsync({
    init: 'initial Value',
    fetch: async () => {
      console.log('fetch called');
      await delay(1);
      return 'final value';
    },
  });

  for (let i = 0; i < 5; i++) {
    console.log('Accessing observableValue twice.');
    console.log('observableValue:', { busy: observableValue.busy, value: observableValue.value });
    await delay(1);
  }
}

main();

The output looks like this:
Accessing observableValue twice.
fetch called
fetch called
observableValue: { busy: true, value: 'initial Value' }
Accessing observableValue twice.
fetch called
fetch called
observableValue: { busy: true, value: 'initial Value' }
Accessing observableValue twice.
fetch called
fetch called
observableValue: { busy: true, value: 'initial Value' }
Accessing observableValue twice.
fetch called
fetch called
observableValue: { busy: true, value: 'initial Value' }
Accessing observableValue twice.
fetch called
fetch called
observableValue: { busy: true, value: 'initial Value' }

Each time I access a property of observableValue, its fetch function is re-evaluated. That is unfortunate, but probably cannot be helped; after all, when called by non-observed code, observableValue has no way of knowing whether fetch will return the same value again. What surprises me, though, is that I never get the actual value returned by fetch.
If fetch has been called in the past and has resolved with a value, I'd expect observableValue.value to return that value.

Interesting question. The plain MobX computed, when its value is read outside of a reactive context, can just execute its evaluation function and returns the result. It avoids using caching because outside of the reactive context there is no listening for change notifications and hence no way to know when the cache is stale.
But in this case we are completely dependent on caching. Building a promise is expensive: it usually means remote calls etc. We must not repeat the network call every time we are asked for a value. But this means we need cache invalidation. We need to know if the observables that provided the data for the remote call are still valid.
Therefore it makes no sense to use this library outside of reactive contexts.
In fact it would be better if this was made clear by throwing a descriptive Error if you try to get the value outside of a reactive context.

If you set up an autorun to observe the value, then a non-reactive polling loop such as yours will actually see the expected behaviour, i.e. value and busy will transition as expected.
But this is a messy leak of inconsistent behaviour. It would make a lot more sense to use MobX consistently to observe those values changing instead of sometimes using polling unnecessarily.

Closing this given that the suggestion "it would be better if this was made clear by throwing a descriptive Error if you try to get the value outside of a reactive context" is now implemented in the new API.


endpoint /healthz/ready does not show draining or terminating state

Description
We run the ingress-gateways as daemonset and have an external loadbalancer (not part of the Kubernetes cluster) in front of the nodes. The loadbalancer calls the /healthz/ready endpoint to check if the ingress-gateways are ready. If we delete a ingress-gateway pod it does not report its draining state.
This way a external loadbalancer has no way to determine that it has to remove a ingress-gateway from its pool before the pod gets restarted and thus it is not possible to gracfully restart/upgrade the ingress-gateways in such an environment.
It seems that there are two problems:

pilot-agent caches the envoy state https://github.com/istio/istio/blob/fa47c1a3b97db0874a76b042b117a0a5fa44beb0/pilot/cmd/pilot-agent/status/ready/probe.go#L74
envoy does not report its draining state if /drain_listeners?graceful is called. I'm not sure if this is intended but this can be circumvented by calling curl -X POST http://localhost:15000/healthcheck/fail in the container preStop lifecycle hook. After calling healthcheck/fail envoy does properly report its state as draining.

[ ] Docs
[ ] Installation
[x] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
[ ] Upgrade
Expected behavior
The pod:15021/healthz/ready endpoint reports if the ingressgateway pod is draining connections (terminating).
Steps to reproduce the bug
Steps to reproduce:
kind create cluster
istioctl operator init
kubectl apply -n istio-system -f istiooperator.yaml

istiooperator.yaml

apiVersion: install.istio.io/v1alpha1
kind: IstioOperator
metadata:
  name: default
spec:
  meshConfig:
    defaultConfig:
      terminationDrainDuration: 20s
  components:
    ingressGateways:
    - name: istio-ingressgateway
      enabled: true
      k8s:
        env:
        overlays:
        - apiVersion: apps/v1
          kind: Deployment
          name: istio-ingressgateway
          patches:
          - path: kind
            value: DaemonSet
          - path: spec.strategy
          - path: spec.template.spec.containers.[name:istio-proxy].ports.[containerPort:8443].hostPort
            value: 443
          - path: spec.template.spec.containers.[name:istio-proxy].ports.[containerPort:15021].hostPort
            value: 15021

Watch /healthz/ready and delete ingressgateway pod

watch -n 1 curl -Ss -w "%{http_code}" http://<IP_ADDRESS>:15021/healthz/ready
kubectl -n istio-system delete pod -l app=istio-ingressgateway

curl changes directly from HTTP status code 200 to Failed to connect to <IP_ADDRESS> port 15021: Connection refused

Version
$ istioctl version
client version: 1.10.0-rc.1
control plane version: 1.10.0-rc.1
data plane version: 1.10.0-rc.1 (1 proxies)
$ kubectl version --short
Client Version: v1.21.0
Server Version: v1.20.2

How was Istio installed?
Operator

Remove the caching (https://github.com/dvob/istio/commit/1b745cbe89346814d53256226b2601933e25cc76) and use the preStop lifecycle hook as follows would work:
          - path: spec.template.spec.containers.[name:istio-proxy].lifecycle
            value: {"preStop": {"exec": {"command": ["sh", "-c", "curl -X POST http://localhost:15000/healthcheck/fail"]}}}

But probably Envoy should also return its state as DRAINING if /drain_listeners?graceful is called and not only if /healthcheck/fail is called. At least this is my understandinf from https://www.envoyproxy.io/docs/envoy/latest/intro/arch_overview/operations/draining

I agree it is better to call that endpoint to show the correct status.  Fixed it in #32892


Correction for raw affiliation Aeres University of Applied Sciences, De Drieslag 4, 8251 JZ Dronten, The Netherlands; Rural Sociology, Wageningen University, Hollandseweg 1, 6706 KN Wageningen, The Netherlands

Correction needed for raw affiliation Aeres University of Applied Sciences, De Drieslag 4, 8251 JZ Dronten, The Netherlands; Rural Sociology, Wageningen University, Hollandseweg 1, 6706 KN Wageningen, The Netherlands
raw_affiliation_name: Aeres University of Applied Sciences, De Drieslag 4, 8251 JZ Dronten, The Netherlands; Rural Sociology, Wageningen University, Hollandseweg 1, 6706 KN Wageningen, The Netherlands
new_rors: 03jnx2c74;04qw24q55
previous_rors: 015d5s513;04qw24q55
works_examples: W2595614040
contact: 3e7ee7e4dfe2c94d294008549913277f:c7aa42ca72ab4811 @ aeres.nl

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


Adding indexing and sliced iteration

Extending support for better indexing and iteration at operational level. The main use case is scheduling and production planning models with startup and shutdown modelling. Naming is up for discussion.

I think both Iterators.partition and Iterators.cycle have a usage that does not align with the current functionality. I am currently leaning towards using the name block instead of slice. A block of time periods does not sound too bad.
I am not sure co-iteration with duration is worth it as it should be fairly easy to get it by summing the duration of the time periods in the block.

Another name alternative is chunk

Good point, I think I would prefer block or chunk over slice.


ComboBox ItemsSource binding Dictionary<string, ClassObject> throws InvalidOperationException "Target type is not a projected type"

Describe the bug
ComboBox's ItemsSource binding with Dictionary<TKey,TValue> throws an error when TValue is a class object.
I'm using WinUI 3 with C# for my project.
I have viewed other similar posts which discuss about ComboBox's ItemsSource binding with Dictionary<TKey,TValue> and I'm pretty sure I'm doing everything correctly.
I even tested the code by using a Dictionary<string,int> object to bind to the ComboBox and it works without any problem.
But only when I use a class object as TValue, it throws this error:
Exception thrown: 'System.InvalidOperationException' in WinRT.Runtime.dll
Target type is not a projected type: App1.Sample

And the debugger shows that the problem occurs at obj.ItemsSource = value; in:
public static void Set_Microsoft_UI_Xaml_Controls_ItemsControl_ItemsSource(global::Microsoft.UI.Xaml.Controls.ItemsControl obj, global::System.Object value, string targetNullValue)
{
    if (value == null && targetNullValue != null)
    {
        value = (global::System.Object) global::Microsoft.UI.Xaml.Markup.XamlBindingHelper.ConvertValue(typeof(global::System.Object), targetNullValue);
    }
    obj.ItemsSource = value;
}
...

Steps to reproduce the bug

Create a new Blank App, Packaged (WinUI 3 in Desktop) in Visual Studio
Add following code accordingly:

MainWindow.xaml:
<?xml version="1.0" encoding="utf-8"?>
<Window
    x:Class="App1.MainWindow"
    xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
    xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
    xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
    xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
    mc:Ignorable="d">

    <StackPanel Orientation="Horizontal" HorizontalAlignment="Center" VerticalAlignment="Center">
        <ComboBox ItemsSource="{x:Bind Samples}"
                  DisplayMemberPath="Key"
                  SelectedValuePath="Value"></ComboBox>
    </StackPanel>
</Window>

MainWindow.xaml.cs:
using Microsoft.UI.Xaml;
using System.Collections.Generic;

namespace App1 {
    public sealed partial class MainWindow : Window {
        public MainWindow() {
            this.InitializeComponent();
            Samples.Add("skdn", new());
        }

        public Dictionary<string, Sample> Samples = new();

        public class Sample {
            public string Name { get; set; }
            public Sample() {

            }
        }
    }
}

Run the app

Expected behavior
The app should run without errors.
Screenshots
No response
NuGet package version
WinUI 3 - Windows App SDK 1.3.2: 1.3.230602002
Windows version
Windows 10 (21H2): Build 19044
Additional context
No response

Any updates or solutions?
I’m getting the same error when trying to bind to ItemsRepeater

I have the same issue when using Dictionary<CustomClassObject, float> and trying to bind to a ListView

Same but with SortedDictionary

Encountered the same issue. This might help: https://stackoverflow.com/questions/76555819/winui-3-combobox-itemssource-binding-dictionarystring-customclassobject-throw


migrate to new templates

migrate these branches into the new templates following the current conventions:

[x]  bhrle
[x]  cnr
[x]  genesee-news
[x]  hohp
[x]  lrcdp
[x]  otherfaces
[x]  rwhp
[x]  sanborn
[x]  taylorarchive

Tasks:

Update metadata to use new object location (from "/digital/objects/" to "objects.lib.uidaho.edu"
ensure follows conventions stub, branch, metadata all named in same way

The "taylorarchive" looks like an incomplete collection but I migrated "twrs" which is connected to "taylorarchive" on the digital-collections-template-cb-csv


[Logistic] Make GitHub Actions only run on certain label

To avoid waiting too long when editing README or making non-swift changes.

This won't be done until later if the project is still being actively used.

In the foreseeable future this won't be fixed. Close for now and reopen if I'll make further changes after November.


Extension does not load on Postgres 11

postgres=# CREATE EXTENSION jsonb_deep_sum;
ERROR:  could not load library "/usr/lib/postgresql/11/lib/jsonb_deep_sum.so": /usr/lib/postgresql/11/lib/jsonb_deep_sum.so: undefined symbol: PG_RETURN_JSONB

Should now be 'PG_RETURN_JSONB_P'?

Also tested in PostgreSQL 12, works great.


Low values for iLISI score

Hello,
I graph connectivity, kBET and iLISI on the same dataset. While the former two metrics seem to reflect a pretty solid batchmixing, iLISI is ridiculously low. Is there anything that needs to be done in terms of scaling?

This is my method call:

Thank you for your help!

Dear developers,
I have encountered the same issue. Between integrated and unintegrated data, iLISI scores are indifferent, or the difference is at 16 digits scale (the precision of python floating point numbers). In contrast, all other metrics show informative differences.
Could you please suggest if and how to use iLISI scores in this case?
Thank you very much.


Allow local extension host's proxyResolver.ts to read user setting value of http.proxy

Required for https://github.com/microsoft/vscode-remote-release/issues/3987

@chrmarti Is this still a valid request? If you want user value, have you tried inspect API and get the user value?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


[Package Request]: Console Plus

How can we help?
None
Please read and ensure the following

[X] The installer meets the above requirements

Please provide the following information

Download Page Url: https://www.cryopakdigital.com/software/
Publisher: Cryopak
Package Name: Console Plus
Description: Cryopak Digital temperature monitoring solution
Package Version: N/A
Installer URL: https://miradorexpress.com/upgrade/ConsolePlusSetup.zip

Originally posted by @tarunsinghhf in https://github.com/microsoft/winget-pkgs/issues/148919#issuecomment-2056847664
### Tasks

This should be closed as there is already the following package for it which is a newer version: https://github.com/microsoft/winget-pkgs/tree/master/manifests/m/Microsoft/Azure/FunctionsCoreTools/4.0.5611

Hello @Dragon1573
Thanks for opening this request, now I've tested the .exe installer that's inside the downloaded .zip file, and sadly I can't seem to find a way to install it in Silent Mode, which's a requirement for any program to be accepted into the winget packages repo.
If you, and anyone who've the time to test this program, know a way to do the above Install it silently, then please go ahead and share it here, or open a PR for this program.

@og-mrk @vikingnope Thanks for your help ❤
Successfully inspected by Komac v2.2.0. The nested installer wizard is packaged by InnoSetup.

[!NOTE]
Because of the following reason, this issue is not able to be solved. Close as "not planned".

https://github.com/microsoft/winget-pkgs/pull/149317#issuecomment-2057145661

The installer wizard requires a graphical approval for installing "driver application". winget.exe does not support installing this kind of package.

@stephengillie - Could you please add https://github.com/microsoft/winget-pkgs/labels/Blocking-Issue https://github.com/microsoft/winget-pkgs/labels/DriverInstall (or other related labels) to this issue? Thank you ❤
CC @tarunsinghhf


How to send fcm token from react native to webview

Bug description:

To Reproduce:

Expected behavior:

Screenshots/Videos:

Environment:

OS:
OS version:
react-native version:
react-native-webview version:

The way I do it, passing data to be used in WebView, is something like the following:
const CustomWebView = (props) => {

  const injectedCode = () => {
    const { FCMToken } = props;

    return `
      (function(key) {
        var storedToken = localStorage.getItem(key);
        if (!storedToken || storedToken !== '${FCMToken}') {
          localStorage.setItem(key, '${FCMToken}');
          location.reload();
        }
      })('FCMToken');
    `;
  };

  return (
    <WebView
      javaScriptEnabled={true}
      injectedJavaScript={injectedCode()}
      domStorageEnabled={true}
    />
  );
}

the injected JS code will be executed after page load
so, it cannot be used by any other script on the page
that's why, on first load, after I set the data in the localStorage, I reload the page
it's an inconvenient that only happens on first load


Support server and client testing

In some cases, two machines are needed for testing. For example, some performance tests will need to communicate between server and client. We need to update our test pipeline to support this case.
Proposed solution:

create a Jenkins job Machine-Reservation - this is a simple job to grep user-specified machine and sleep for the user-specified time. The main idea is to reserve the machine.
in the test pipeline, a new parameter RELATED_NODES should be added. If the user provides a node name in RELATED_NODES, the test pipeline will trigger Machine-Reservation to book the specific machine before starting any testing. Otherwise, proceed with testing normally. For this case, we just need to pass in a single node. This can be further extended later for multiple nodes (i.e., comma separated).
RELATED_NODES is set as an env variable to pass into the test scripts
individual job will connect to RELATED_NODES for testing
once the testing is completed, abort Machine-Reservation if it is still running

Created https://ci.adoptopenjdk.net/job/Machine-Reservation job as a placeholder for a pipeline described in this issue.
@norbline - if you have not already done so, please rebase your branch (described in Contributing.md) in preparation to begin work on this issue.  I believe @llxia will share some code with you relating to Machine-Reservation shortly.

TODO:

[ ] create Jenkins job Machine-Reservation https://ci.adoptopenjdk.net/view/Test_grinder/job/Machine-Reservation/

[ ]  checkin initial Machine-Reservation script in aqa-tests repo (Lan)

[ ] For now, If RELATED_NODES is provided and TARGET=IdleMicrobenchmark_HS, trigger Machine-Reservation job with RELATED_NODES and TIME_LIMIT

[ ] Update IdleMicrobenchmark_HS to echo RELATED_NODES https://github.com/adoptium/aqa-tests/blob/master/perf/idle_micro/playlist.xml#L17

[ ] get Machine-Reservation job status (job number).

If the RELATED_NODES cannot be reserved within 30 mins, abort Machine-Reservation job and abort the job.

If the RELATED_NODES can be reserved within 30 mins, proceed to execute the test.

[ ] in post-stage, release RELATED_NODES (abort Machine-Reservation job) and print the message

Hi @norbline - can you add a comment in this issue, so I can assign it to you?
Grinder_Perforama job is created for you to use to help test your work: https://ci.adoptopenjdk.net/job/Grinder_Perforama/
When you "Build with parameters, you should see the new parameter RELATED_NODES at the bottom just above the Build button.

Thanks, Shelley. All good to get on
On Thu, Jan 27, 2022, 5:56 PM Shelley Lambert @.***>
wrote:

Hi @norbline https://github.com/norbline - can you add a comment in
this issue, so I can assign it to you?
Grinder_Perforama job is created for you to use to help test your work:
https://ci.adoptopenjdk.net/job/Grinder_Perforama/
When you "Build with parameters, you should see the new parameter
RELATED_NODES at the bottom just above the Build button.
—
Reply to this email directly, view it on GitHub
https://github.com/adoptium/aqa-tests/issues/2841#issuecomment-1023438077,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJOPRPBAM2GN5OEVUIJE6RDUYF2MJANCNFSM5DHIPHRA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

@norbline For using Jenkins API to get job stage status, I found a similar thing at StackOverflow: https://stackoverflow.com/questions/37009906/access-stage-results-in-workflow-pipeline-plugin
Could you give it a try? Thanks

yes @Lan i will give it a try

After the investigation and experiments, we decided to use nested nodes instead of Jenkins API. In this way, we do not need to control the workflow/communication between two jobs. It is easy to maintain and less error-prone.
The key changes for nested notes are delivered via #3364 and #3379.
Enhancements:

[ ] check if we can get RELATED_NODES in playlist.xml. If not, set RELATED_NODES as env variable
[ ] add timestamp{} to provide more info https://github.com/adoptium/aqa-tests/blob/25c804602873d503ec596371875c33a49fa88ad2/buildenv/jenkins/openjdk_tests#L100-L236
[ ] if we can have timestamp{} that covers all the sections, we can remove individual timestamp{} in each stage.

Thanks @llxia

@awsafsakif could you test your case and let us know? Thanks.

Verified in Grinder that we can directly get RELATED_NODES in playlist. There is no need to set it as an env variable.
In the console output, test-docker-ubuntu2010-x64-2 is printed.
00:02:37.449  ===============================================
00:02:37.449  Running test IdleMicrobenchmark_HS_0 ...
00:02:37.449  ===============================================
00:02:37.449  IdleMicrobenchmark_HS_0 Start Time: Fri Mar 11 20:12:57 2022 Epoch Time (ms):<PHONE_NUMBER>279
00:02:37.449  Nothing to be done for setup.
00:02:37.449  variation: -Xmx1024m
00:02:37.449  JVM_OPTIONS:  -Xmx1024m
00:02:37.449  { itercnt=1; \
00:02:37.449  mkdir -p "/home/jenkins/workspace/Grinder_Perforama/aqa-tests/TKG/../TKG/output_16470295765006/IdleMicrobenchmark_HS_0"; \
00:02:37.450  cd "/home/jenkins/workspace/Grinder_Perforama/aqa-tests/TKG/../TKG/output_16470295765006/IdleMicrobenchmark_HS_0"; \
00:02:37.450  echo test-docker-ubuntu2010-x64-2; mkdir -p "/home/jenkins/workspace/Grinder_Perforama/aqa-tests/TKG/../TKG/output_16470295765006/IdleMicrobenchmark_HS_0"; \
00:02:37.450  cd /home/jenkins/workspace/Grinder_Perforama/aqa-tests/TKG/../../jvmtest/perf/idle_micro; \
00:02:37.450  "/home/jenkins/workspace/Grinder_Perforama/openjdkbinary/j2sdk-image/bin/java"  -Xmx1024m  \
00:02:37.450  -cp ":/home/jenkins/workspace/Grinder_Perforama/aqa-tests/TKG/../../jvmtest/perf/idle_micro/IdleMicrobenchmark.jar" net.adoptopenjdk.test.gc.idlemicrobenchmark.IdleMicroBenchmark \
00:02:37.450  --memoryLimit=700 --time=60 --idletime=300 --warmup=1 >"/home/jenkins/workspace/Grinder_Perforama/aqa-tests/TKG/../TKG/output_16470295765006/IdleMicrobenchmark_HS_0"/std.out 2>&1 ; \
00:02:37.450  if [ $? -eq 0 ] ; then echo ""; echo "IdleMicrobenchmark_HS_0""_PASSED"; echo ""; cd /home/jenkins/workspace/Grinder_Perforama/aqa-tests/TKG/..;  else echo ""; echo "IdleMicrobenchmark_HS_0""_FAILED"; echo ""; fi; } 2>&1 | tee -a "/home/jenkins/workspace/Grinder_Perforama/aqa-tests/TKG/../TKG/output_16470295765006/TestTargetResult";
00:02:37.450  test-docker-ubuntu2010-x64-2

FYI @awsafsakif

The feature is implemented. The task about - add timestamp{} to provide more info is optional and it is tacked in  https://github.com/adoptium/aqa-tests/issues/3396
Close this issue.


Add version dropdown

I'm closing this issue since we have gone with the Releases page approach. There the users can navigate throw the site.


Data Error on Mac

Debug log ID
L5NAFQ5H-fio-AUT7rebksdtL
What happened?
Using latest version of BBT (was present in earlier versions as well). BBT not working with error "Error: Data provided to an operation does not meet requirements."  Using Zotero 7 (also occurs in Zotero 7 Beta).  Reinstalling (both Zotero and BBT) does not fix things.

Please turn on debug logging in the help menu, disable and enable BBT, then send a new log. There are no errors logged in L5NAFQ5H-fio-AUT7rebksdtL.

Thanks---did as you recommended and new Debug log ID is CENZHTNJ-fio-OzrUssxodQI6

I installed it and get the same error. The new debug log ID is XTZYE7WZ-fio-ZrgTJLNBpMpW

Debug logging was off at time of submission of the report, so there's nothing in the log.

Apologies---here is he new debug log ID 9RBDI6RM-fio-TyVPUaFo18Hd

This one works with no issues so far. Thank you!


Typos in Legion runtime flags are unchecked

Compare:
$ ./regent.py examples/circuit_sparse.rg -ll:cputypo 1
ERROR: unrecognized lowlevel option: -ll:cputypo

Vs:
$ ./regent.py examples/circuit_sparse.rg -lg:proftypo 1
ELAPSED TIME =   0.066 s
GFLOPS =   2.312 GFLOPS

Or for that matter, -lg:typo, -hl:typo, or variations thereof.
I noticed this while mistakenly trying to run -lg:no_trace_optimizations instead of -lg:no_trace_optimization (no s). I'm thankful @magnatelee pointed this out to me because I would have spent a long time looking for it otherwise.
It would be nice to have strict checks, particularly on the prefixed flags like -lg:, -hl:, -dm:, etc. Since Realm has this capability, maybe it can be exposed in a way that Legion might be able to reuse?

I'll make a trade with the group: I'll add support for errors on all -lg: flags that don't match if we can finally remove support for all -hl: flags. As for -dm: flags, I don't really care, that is a collective decision to be made by the group as well, but I will not be opinionated on it. @alexaiken to add to a future group meeting for discussion.
Arbing myself and @manopapad so he can take the -dm: flags to the mapper working group for discussion.

I'm fine with that tradeoff, -lg: has been supported for... years(?) now, so I think it's reasonable to remove -hl:.

Note that Realm flags other than -ll: are also not checked, e.g. -gex:typo is accepted. The command line parser could check for typos in a general way by creating a list of prefixes used by all add_option commands and checking every unrecognized option against them.

I was surprised to find today that we still haven't done this and that -lg:typo 0 is not caught as an error.
I think Mike's suggestion in https://github.com/StanfordLegion/legion/issues/1045#issuecomment-814005483 was reasonable, should we actually do that?

We agreed in the meeting today to remove the old -hl: options.
@lightsighter to propose an API for the Realm command line parser to support checking unexpected flags in a given prefix.

Ok, I have created a draft interface for this in the Realm command line parser:
https://gitlab.com/StanfordLegion/legion/-/merge_requests/1312
Once we've got approval on that then I can update Legion. Did we decide we wanted to do the default mapper too?

If the API becomes available I can update the default mapper to use it as well.


Expectaions Bug

After switching expections to another page (there are many expectations)，users can not modify expetations that comes begin with page 2, I suppose it is because the list was rendered by loop and items were binded with key of index, if we modify data from page 2, data from page 1 can get affected.
thus, if there are many expecations, it is not functional anymore.

yes，because the update of expectation use the index of the table row number, so i do not work in page 2. I will see how to fix it.

the bug was fixed by https://github.com/alinGmail/LiveMock/commit/2b64f2b6898f173afdc531ed50a79a1f231d7bde

the bug was fixed by 2b64f2b

glad to hear that, are you going to publish a release?


[JBPM-5220] kie-server: implement filtering for listContainers()

filtering is (currently) possible based on releaseId (and
its individual parts) and status

There are still few things I am not sure about:

is the package *.model OK for the filter classes? Or should they go into own package (e.g. model.filter)?

I added method listContainers(kieContainerFilter). Should we also have methods like listContainers(ReleaseId) or listContainers(Status). I am not sure since this can easily explode into quite a few new methods.

there is no isDeployed() functionality implemented. It is I believe  equal to status=STARTED, so not sure it is worth adding new param just as an alias to existing one.

I added toString(), equals() and hashcode to few *.model classes. I believe we could do that for all the classes? (unless there is good reason not to do so, which I am not currently aware of)

@mswiderski please take a look.

nice work @psiroky ! just one minor comment that is up to you if it makes sense to change or not

The tests are in jbpm-simulation module, so not related to these changes. Of course, we need to fix them. Looks like some change in upstream repos (drools, jbpm) caused this.

+1 to merge, jbpm simulation failures should already be fixed on master

@psiroky could you please prepare pr for backport to 6.5?

@mswiderski yes, will do that today.


update v

In addition to your existing fixes, please bump the lib version references to 0.3.3 in the pull request. That way, I can publish it to pypi for you.
That is, change these references to 0.3.3
$ git grep 0.3.2
CHANGES.txt:v0.3.2, 2014-05-01 Update configparser lib version to 3.5.0b2
clippercard/init.py:version = '0.3.2'
setup.py:    version='0.3.2',

[ how to assign ]


Red Rectangle means?

Please clarify whether red rectangle means a deep fake was detected, or a deep fake not detected.
Looks like an impressive script, but needs UI improvements. I used on an image which resulted in a score of 0.5% and a red rectangle. Unfortunately I had no idea what those results meant. I checked the same image with Chrome V7 GAN Deep Fake Detector, and got a result saying likely real person.

Hi @ShaolinCrypto, the model will compute an output between 0 and 1 that indicates the probability of the input image being either deepfake (0) or pristine (1). In the demo website (https://deepfake-detect.com/), I simply use 50% (0.5) as a threshold and therefore a green rectangle means a real image, while a red rectangle means a deepfake.


screenmananager.clear_widgets does not clear self.screens

I called sm.clear_widgets(), but got a "Multiple Screens named..." message, when I added a Screen.
Probably the loop "for screen in screens: remove_widget(screen)" in clear_widgets, that changes the list over which it iterates, does not work as intended. Perhaps change to "for screen in screens[:]:" to iterate over a copy of the list.

this has been fixed in 58a3255d36, which isn't part of a stable release yet, please try the master version instead.


Is it safe enough to retry ETIMEDOUT error by default?

Describe the bug (描述bug)
Hi, thanks for creating such an amazing RPC framework!
I'm learning this project recently but found brpc will retry to send request when meeting ETIMEDOUT error. In my understanding, ETIMEDOUT means:

A connect(2) or send(2) request failed
because the connected party did not properly respond after a
period of time.  (The timeout period is dependent on the communi-
cation protocol.)

If the server-side has processed the request, but not send back ACK to the client successfully(and still failed when the client retransmits), the client will get the ETIMEDOUT error finally, and retry to send the RPC call, which will cause unexpected behavior for the non-idempotent API.
+----------+                                +----------+
|  Client  |                                |  Server  |
+----------+                                +----------+
     |                                           |
     |                                           |
     |                  PSHs ==>                 |
     |-------------------------------------------|
     |                                           |
     |                                           |
     |                <== ACKs                   |
     |     Lost ---------------------------------|
     |                                           |
     |                                           |Process
     |                                           ---------
     |                                           |
     |                                           | Crash
     |                                           ---------
     |                                           |
     |                                           |
     |                                           |
     |                                           |
     |                                           |
     |                                           |

Let me know if I miss anything, thanks!
Expected behavior (期望行为)
ETIMEDOUT should not be included in default retriable error codes.
Versions (各种版本)
OS:
Compiler:
brpc:
protobuf:
Additional context/screenshots (更多上下文/截图)

Hi, the retry behavior can be controlled by yourself, you can disable retry by setting ChannelOptions::max_retry, or you can implement your own RetryPolicy and set it into ChannelOptions::retry_policy

@lorinlee Hi, thanks for the answer. I know there are some options that could controller this behavior. But just curious about the security of the default retry policy since most developers are using it.

ETIMEOUT is a connection timed our error per https://man7.org/linux/man-pages/man3/errno.3.html. If this error happens, RPC can't be sent to remote sides.

ETIMEOUT means that rpc reached the deadline. ETIMEOUT should not trigger to retry RPC.
The default retry poliy refer the link: https://github.com/apache/incubator-brpc/blob/master/src/brpc/retry_policy.cpp#L33
Framework can not guarantee idempotent。The idempotent is an application level function.

@cdjingit @zyearn thanks very much, and I found a doc has answered my question.
In summary, when using brpc users should make sure their services are idempotent otherwise they need to disable the default retry policy.

Hi @joway, another thing I have missed is the meaning of error code. You can see this doc, the Settings section. ETIMEDOUT is connection timeout but not rpc timeout, so it can be retried. ERPCTIMEDOUT means rpc timeout, which cannot be retried for the reason you said, and the DefaultRetryPolicy also not retry, so users needn't care about the default behavior.

and all error code meanings can be found in here error code


Lost connection to MagicMirror after going to 2.21.0 of MM

Version of the custom_component

1.4.0
Configuration

Add your logs here.
Logger: homeassistant.config_entries
Source: config_entries.py:783
First occurred: 19.02.15 (1 occurrences)
Last logged: 19.02.15

Config entry '<IP_ADDRESS>' for magicmirror integration not ready yet; Retrying in background

Logger: homeassistant.config_entries
Source: config_entries.py:395
First occurred: 18.59.20 (4 occurrences)
Last logged: 18.59.27

Config entry '<IP_ADDRESS>' for magicmirror integration not ready yet; Retrying in background

Describe the bug
After the upgrade of MagicMirror from 2.20 to 2.21 the integration doesn't work.
I think the update of HA to 2022.11.x also had an effect.

Debug log

2022-11-27 19:07:21.278 DEBUG (MainThread) [custom_components.magicmirror] GET url=http://<IP_ADDRESS>:8080/api/mmUpdateAvailable. headers={'accept': 'application/json', 'Authorization': 'Bearer b9911c8282f943faac207140f4e8ea0a'}
2022-11-27 19:07:35.399 DEBUG (MainThread) [custom_components.magicmirror] Finished fetching magicmirror data in 14.121 seconds (success: False)

Hm weird, I'm currently running both MagicMirror 2.21 and HA 2022.11.4 and not facing any new issues. My mirror goes unavailable once in a while when polling is timing out, because my mirror is on wifi and has a weak signal, but seems to be running fine most of the time. Does this happen on every restart, or only sometimes?

It is the new normal here. I haven't had any success in connecting to it. I also previously had intermittent connection issues, but that was just wifi problems for this mirror.
But now I can log into the rpi with a SSH connection without issues, but the integration still fails.

I have no idea why, but after updating HA to 2022.11.5 the MagicMirror integration works again?

Seems to be resolved. Please reopen if the issue did not solve itself.


fix the references in the Iris notebook

It's still referencing to hyperio (the initial name for the package).

It's a good time to create a few example notebooks / files. Closing this.


Error while trying to write ncbigene.obo via pyobo

pyobo call made via code:
from pyobo.sources.ncbigene import get_obo

def get_obo_file(output_path: str):
    """Get the obo file."""
    obo_file = get_obo()
    obo_file.write_obo(output_path, use_tqdm=True)

This resulted in the following error:
Traceback (most recent call last):
....
  /src/ncbi_gene_pyobo/cli.py", line 57, in get_obo_file
    obo_file.write_obo(output_path, use_tqdm=True)
  File "/opt/anaconda3/envs/ncbi-gene-pyobo/lib/python3.10/site-packages/pyobo/struct/struct.py", line 711, in write_obo
    self._write_lines(it, fh)
  File "/opt/anaconda3/envs/ncbi-gene-pyobo/lib/python3.10/site-packages/pyobo/struct/struct.py", line 717, in _write_lines
    for line in it:
  File "/opt/anaconda3/envs/ncbi-gene-pyobo/lib/python3.10/site-packages/pyobo/struct/struct.py", line 700, in iterate_obo_lines
    yield from term.iterate_obo_lines(ontology=self.ontology, typedefs=self.typedefs)
  File "/opt/anaconda3/envs/ncbi-gene-pyobo/lib/python3.10/site-packages/pyobo/struct/struct.py", line 429, in iterate_obo_lines
    yield f"def: {self._definition_fp()}"
  File "/opt/anaconda3/envs/ncbi-gene-pyobo/lib/python3.10/site-packages/pyobo/struct/struct.py", line 400, in _definition_fp
    return f'"{obo_escape_slim(self.definition)}" [{comma_separate(self.provenance)}]'
  File "/opt/anaconda3/envs/ncbi-gene-pyobo/lib/python3.10/site-packages/pyobo/struct/utils.py", line 24, in obo_escape_slim
    rv = "".join(OBO_ESCAPE_SLIM.get(character, character) for character in string)
TypeError: 'float' object is not iterable

The error points to line 24 below.
https://github.com/biopragmatics/pyobo/blob/cc2f27bb0a05e33b48e58985ac1db81c0f743095/src/pyobo/struct/utils.py#L22-L26
Reran it by casting string in join() [str(string)]
    rv = "".join(OBO_ESCAPE_SLIM.get(character, character) for character in str(string))

fixed the issue.

stringifying a NAN from pandas isn't the right solution - this would introduce incorrect characters. This might have already been fixed in cc2f27b


0.5.4

Codecov Report

Merging #80 into master will increase coverage by 0.03%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##           master      #80      +/-   ##
==========================================
+ Coverage   85.43%   85.47%   +0.03%
==========================================
  Files          70       70
  Lines        2857     2864       +7
  Branches      418      419       +1
==========================================
+ Hits         2441     2448       +7
  Misses        357      357
  Partials       59       59


Setup of arbitrary game positions

With the restriction for alternating play gone (#275), the app is now theoretically capable of processing (almost) arbitrary game positions. What is missing is the user interface to do so.
Feature overview

Setting up game positions should be a distinct mode of operation
While in this mode, no actions for regular game play should be available (specifically "computer play" and "update player influence" are not required, also Fuego does not need to ponder)

Possibly the "discard" action should be made available only in this mode instead of during normal game play?

Little Go must also be able to load .sgf files that contain board position setups.
For instance, I have received a bug report from a user who attempted to load this .sgf file: https://senseis.xmp.net/diagrams/30/ad4024e515166e1291647ff851495120.sgf. At the moment, loading this .sgf file results in an empty board, with no turns or stones shown. If the user tries to place a stones on, for instance, B15, the game responds with an error message like this:

Fuego rejected the move B15 played by me. The reason given was: illegal move: 53 B B15 (occupied).

The reason is that Fuego knows that B15 is occupied, because it understands the .sgf file content. But when Little Go queries Fuego for moves after the .sgf file was loaded, there are no moves to report, so Little Go believes that B15 is not occupied.

A number of changesets were accidentally linked to issue #279 instead of this issue.


Jtag flash

Coverage remained the same at 73.79% when pulling 39330bfff9e030234eb012b94de98fc786eafcfc on jtag-flash into 599262cf9377f4585bc1a3aea2aac537d8696e80 on master.


rework mapsession

Signed-off-by: Kristoffer Dalby<EMAIL_ADDRESS>

Known TODOs:

There is a bug with Expiry, its sent very often.
If a streaming mapsession is open, and a new one is attempted, the old one should be closed.


fixes #5 coverage build is busted

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@0c971b9). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #8   +/-   ##
=========================================
  Coverage          ?   62.85%
=========================================
  Files             ?       11
  Lines             ?      700
  Branches          ?        0
=========================================
  Hits              ?      440
  Misses            ?      199
  Partials          ?       61

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 0c971b9...c814002. Read the comment docs.


Resume training for NAS

Hi, I stopped the training after NAS stage 1 level 1 checkpoint got saved. Now I want to resume training.
Should I:

--nas-kd-resume-from logs/2024.01.03-123840/checkpoint.pth.tar --resume-from logs/2024.01.03-123840/nas_stg1_lev1_checkpoint.pth.tar

--resume-from logs/2024.01.03-123840/nas_stg1_lev1_checkpoint.pth.tar

Thank you

Second option. You have to call --resume-from to continue NAS training.
--nas-kd-resume-from is for loading the teacher if you use knowledge distillation with NAS.


Secret file no longer secret

This is be no longer needed after https://github.com/alphagov/imminence/pull/139.

Merging this so that the next deploy will use this too.


PyData talk

This PyData talk might be of interest:
Implementing and Training Predictive Customer Lifetime Value Models in Python by Jean-Rene Gauthier, Ben Van Dyke. https://www.youtube.com/watch?v=gx6oHqpRgpY&list=PLGVZCDnMOq0rxoq9Nx0B4tqtr891vaCn7&index=45
and the accompanying notebook:  https://github.com/datascienceinc/pydata-seattle-2017/blob/master/lifetime-value/pareto-nbd.ipynb

they use a couple lifetimes routines to do some data preparation on the CDNOW dataset
but the main topic of the talk/notebook is an implementation of Pareto-NBD as a hierarchal model using the pymc3 library
I've seen some other MCMC implementations of Pareto-NBD (https://github.com/mplatzer/BTYDplus/blob/master/R/pareto-nbd-mcmc.R, which follows from http://ieeexplore.ieee.org/document/4344404/); but at least to me, the pymc3 model interface makes the above implementation particularly concise

Just curious if anyone has tried this approach. We're actually talking about future steps and wonder if it's worth the investment in time to learn and build out this approach on our cluster.

Good stuff. I was trying to find pymc3 implementation without success. :)

Just skimming over this and I'm really confused about the different definitions of recency. I asked in their github as well but maybe you have an idea on this here, too since this seems ambiguous in the lifetimes docs as well.
In the linked notebook, they define

recency: time of most recent purchase

However, they use lifetimes.utils.summary_data_from_transaction_data() for the RFM data prep and I find

in the code
customers['recency'] = (customers['max'] - customers['min'])
in the docs:
recency represents the age of the customer when they made their most recent purchases. This is equal to the duration between a customer’s first purchase and their latest purchase.

I get further confused by the doc's explanation of the recency/frequency graph where they seem to be using the other definition ("Your coldest customers are those that are in the top-right corner: they bought a lot quickly, and we haven’t seen them in weeks.")
Maybe someone can shed some light on these two definitions? Am I missing something obvious?

@ReaBx, no, you are mostly right, it is a confusing concept and I get confused sometimes as well. In lifetimes, the definition is:
customers['recency'] = (customers['max'] - customers['min'])

In words:
recency represents the age of the customer when they made their most recent purchases. This is equal to the duration between a customer’s first purchase and their latest purchase.

In the linked notebook, the author uses the definition:

recency: time of most recent purchase

That's too ambiguous of a definition. But ultimately the author uses lifetimes utils to calculate it, so their summary statistics are the same.

I get further confused by the doc's explanation of the recency/frequency graph where they seem to be using the other definition ("Your coldest customers are those that are in the top-right corner: they bought a lot quickly, and we haven’t seen them in weeks.")

Fair - this graphic does cause a lot of confusion. In generating this graphic, we need to set a max time, which in this case is ~40 time periods. Another way to see this is "all customers first bought from me 40 weeks ago". Thus if a customer has a recency of 5, it means they bought at time 0 and 5 and never since. Thus they are likely dead (or "cold").

Even after installing 'pymc3' using pip install pymc3, which showed me that it was successfully installed.
After running the program using Jupyter notebook, python 3.6, I am getting an error.
ModuleNotFoundError: No module named 'pymc3'

@CamDavidsonPilon , I was going though the paper[1] by Fader 2004, which states RFM as

“RFM” characteristics: recency (time of most recent purchase), frequency (number of past
purchases), and monetary value (average purchase amount per transaction).

Where you code as pointed out by @ReaBx takes

Recency(recency represents the age of the customer when they made their most recent purchases) and Frequency as (repeat purchases)

customers['frequency'] = customers['count'] - 1
 customers['recency'] = (customers['max'] - customers['min']) / freq_multiplier

I am confused as to how this difference in the two recency and frequency defination will affect the CLV modelling?

Does anyone have a local copy of the notebook?

This has been taken down, does anyone have a copy of the notebook?

It's here
https://github.com/jrgauthier01/pydata-seattle-2017/blob/master/lifetime-value/pareto-nbd.ipynb


Popover doesn't close while swiping on IOS

Ionic version:  (check one with "x")
[ ] 1.x (For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[x] 3.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
Open popover, then swipe on aria outside of popover content.
Result: page scrolling, popover present
Expected behavior:
Close popover
Steps to reproduce:
Can be reproduced using ionic-preview-app Popover
https://github.com/ionic-team/ionic-preview-app
( need  build for IOS - iPhone 6 )
Video to reproduce - https://drive.google.com/open?id=0B8XIoiu_hafMMjFrc3c1UmI2THM
Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):

global packages:

    @ionic/cli-utils : 1.5.0
    Cordova CLI      : 7.0.1
    Ionic CLI        : 3.5.0

local packages:

    @ionic/app-scripts              : 1.3.7
    @ionic/cli-plugin-cordova       : 1.4.1
    @ionic/cli-plugin-ionic-angular : 1.3.2
    Cordova Platforms               : android 6.2.3 ios 4.4.0
    Ionic Framework                 : ionic-angular 3.5.2

System:

    Node       : v8.1.4
    OS         : macOS Sierra
    Xcode      : Xcode 8.3.3 Build version 8E3004b
    ios-deploy : not installed
    ios-sim    : not installed
    npm        : 5.3.0

Thanks for opening an issue with us, we will look into this.

@jgw96 Why you add label "v2", not "v3" ?  I use ionic-angular 3.5.2


About input size

Thanks for your code.
I wonder if the input image size must be the same for training and testing?

that's not must! as for me, trainging image is 1616->6464,and the test is 6464->256256, everything is ok as for your proper dataloader, but the SR result is not so good for my dataset! if you have any improvement ,could you please give my some advise about your hyperparamters? Thank you before


Dev: Remove the email ignore errors.

Description
See #7731
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

@yu3223 Thank you!