No heartbeat support

No heartbeat support , i tried with this mqtt android library https://github.com/chinesejie/paho-for-android and it keeps breaking the code.

It's not clear what this issue point to. What do you mean for "no heartbeat
support" the ping messages? Do you have tried with the official Paho client
from Eclipse?
Il giorno 21/giu/2015 20:09, "Raffic Maleo"<EMAIL_ADDRESS>ha
scritto:

No heartbeat support , i tried with this mqtt android library
https://github.com/chinesejie/paho-for-android and it keeps breaking the
code.
—
Reply to this email directly or view it on GitHub
https://github.com/andsel/moquette/issues/53.

Hello Andrea,
Moquette works well with Paho client from Eclipse.
but the issue with Paho client from Eclipse is that it loses the connection
to the mqtt broker when android is in sleep mode. so the paho-for-android
https://github.com/chinesejie/paho-for-android
library is a modified version on Paho client that send ping request at
specific intervals to the broker so that the broker will not close the
connection when android is in sleep mode.
so far paho-for-android https://github.com/chinesejie/paho-for-android
works well with mosquitto .
When i tried the paho-for-android library on moquette. the intial
'handshake' is fine , but the subsequent ping request from the
paho-for-android is terminated , thus closing the connection.
https://github.com/chinesejie/paho-for-android
On Mon, Jun 22, 2015 at 5:14 AM, Andrea Selva<EMAIL_ADDRESS>wrote:

It's not clear what this issue point to. What do you mean for "no heartbeat
support" the ping messages? Do you have tried with the official Paho client
from Eclipse?
Il giorno 21/giu/2015 20:09, "Raffic Maleo"<EMAIL_ADDRESS>ha
scritto:

No heartbeat support , i tried with this mqtt android library
https://github.com/chinesejie/paho-for-android and it keeps breaking the
code.
—
Reply to this email directly or view it on GitHub
https://github.com/andsel/moquette/issues/53.

—
Reply to this email directly or view it on GitHub
https://github.com/andsel/moquette/issues/53#issuecomment-114008401.

Hi Raffic,
Have you got the moquette log to post here so i could check what's happened on Moquette side, or pcap dump of the session between the client and the broker?

Hi Raffic,
Have you got the moquette log to post here so i could check what's happened on Moquette side, or pcap dump of the session between the client and the broker?

Can't reproduce and the issue signaler didn't provided the requested information, issue closed.


Code update

Code update

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement
Learn more about why HashiCorp requires a CLA and what the CLA includes

personcube seems not to be a GitHub user.
You need a GitHub account to be able to sign the CLA. If you already have a GitHub account, please add the email address used for this commit to your account.
Have you signed the CLA already but the status is still pending? Recheck it.


Invalid zones in downloaded TIFF files

I am concerned about the values returned though. Is the range given for the raster files as expected? I know it ranges from 0-1 on the site but the range is different on QGIS

Originally posted by @LunaAsefaw in https://github.com/kartoza/rezoning-2-project/issues/25#issuecomment-1549202036

@dimasciput @akshay2408 Do you know what the highlighted values show? I have attached the csv results for LCOE and Score but these values differ from what is seen in the Tiff files.

WBG-REZoning-ALB-Off-Shore Wind-Grid-25-zones-20230517-103343.csv

This is the output of the raster files when they are downloaded on the original site:

This is the range for the raster when downloaded from the staging site:

@dimasciput @akshay2408 @ClaraIV


Allow rootless podman build

Intentionally keeping the 1.17 version (latest is 1.18), will be upgraded later to mitigate risks for now.

Merged into #165


kubeflow 1.3 not creating gpu notebooks

I was able to get a kubeflow 1.3 installation using microk8s 1.21 and the steps documented here (with a --purge of microk8s/juju before):  https://github.com/ubuntu/microk8s/issues/2253#issuecomment-838158746
However, when creating a new notebook, I get the following error:

Vanilla config, eg:

@brettkoonce https://github.com/canonical/bundle-kubeflow/pull/380 will propagate the fix for this to the stable charmed kubeflow bundle. We'll also need to update the script for microk8s enable kubeflow, though until then if you've got an existing installation, you can run juju upgrade-charm jupyter-controller && juju upgrade-charm jupyter-ui.


can't support Instant Run Android Studio 2.0 Beta 4

Butter Knife has nothing to do with Instant Run. It's an annotation processor built on the JSR 269 APIs and if Instant Run does not support it then that's a bug for the Android Gradle plugin, not this project.


Setting CanvasNode bounds to negative values has unexpected behavior

While working on making the bending light canvas fit the visible screen, I tried this:
        bendingLightView.singleColorLightCanvasNode.setCanvasBounds( new Bounds2( -dx, -dy, width / scale, height / scale ) );

which looked like:

and it had very different behavior than:
        bendingLightView.singleColorLightCanvasNode.setCanvasBounds( new Bounds2( 0, 0, width / scale, height / scale ) );
        bendingLightView.singleColorLightCanvasNode.x = -dx;
        bendingLightView.singleColorLightCanvasNode.y = -dy;

which looks like:

So I'm wondering if CanvasNode should have assertions that the bounds should be positive?  Or was I just doing something incorrect?  For the first case, I am painting
      context.strokeRect(
        this.canvasBounds.minX, this.canvasBounds.minY,
        this.canvasBounds.width, this.canvasBounds.height
      );

and the second case uses:
      context.strokeRect(
        0, 0,
        this.canvasBounds.width, this.canvasBounds.height
      );

I know this is not a fully reproducible example, but I'm hoping it is enough context to get to the question about setting negative x/y for the Bounds2 in a CanvasNode.

Looks like the dx/dy wasn't properly added to the max X/Y. CanvasNode DOES support negative bounds (or any other type of valid bounds).


Add support for class static block

Fixes #43012

[x] type definitions
[x] factory
[x] parser
[x] binder
[ ] checker

[ ] grammar
[ ] use before assignment

[ ] transformer
[x] emitter
[x] visitor
[ ] tests

Well.... I think this one is depends on https://github.com/microsoft/TypeScript/pull/42986 too....
It's seems a dependency chains...

https://github.com/tc39/proposal-class-static-block/issues/27

I'm still reviewing #42458, and we've been discussing the fact that this in static fields is currently disallowed, which would effect the order of emit. Also, the static block proposal itself is layered on the static features and class private proposal, so its important that evaluation order is correct:
class C {
  static #x;
  static {
    this.#x; // ok
    this.#y; // error, #y not yet initialized
  }
  static #y;
  static {
    this.#x; // ok
    this.#y; // ok
  }
}

Either way, I'll still be happy to review when you think this is ready.

Okay, thanks a lot.
BTW, there's another PR to allow this in static contrxt.
Actually, I'm confused with the function boundary(as i said in class static block repo).

https://github.com/tc39/proposal-class-static-block/issues/27

This one.( Sorry I'm on mobile phone )

I addressed your comment in that issue. The TL;DR is this:

Class bodies are not function boundaries (ClassHeritage and ComputedPropertyName are evaluated in the class's containing scope), so an await in either position in a class static block would be an error.
FormalParameters, ArrowFormalParameters, and AsyncArrowBindingIdentifier are function boundaries, since parameter initializers are evaluated when the function is called.
FunctionBody, GeneratorBody, AsyncGeneratorBody, AsyncFunctionBody, ConciseBody, and AsyncConciseBody are function boundaries.
A ClassStaticBlock is a function boundary, but is also a class static block (so await as an identifier would be an error).
Static/instance field initializers are function boundaries, as an implicit function is created that wraps the initializer to provide the correct this binding and access the lexical class binding.

I think I'd better to work based #42458. Even It's not been merged.

Close for #43308.


How to get callback while path motion animation is happening

I am using path motion animation,while moving image from one place to another place in some path i want to change images in between

Hi,
Looks like you don't really need to have a callback. As I understood you want to combine view moving by ChangeBounds with images change(like with crossfade?). So you can create a new transition to animate images and add it into TransitionSet together with ChangeBounds. How do you can animate images change? Probably in your custom Transition you can use LayerDrawable to wrap two images inside and than return animator which will animate alphas of your layers.


add clippy for improved checks

use of to_owned instead of to_string
use String::push_str() instead of concatenation

r? @canndrew
(maidsafe_highfive has picked a reviewer for you, use r? to override)

r? @Viv-Rajkumar

r? @brian-js


Add Lineage in MSSQL Usage

Feature
Add Lineage in MSSQL Usage

Lineage in MSSQL is added with PR #3995


Suggestion: Do not abort search query when not toggled modifier is changed.

Right now if search is being run and user interacts with un-toggled modifier, search is aborted and rerun. Since modifier is not toggled, then it wont affect final search results and thus search should complete. Unless user toggles modifier then it makes sense to abort and do it over.

0.6.16 (2020-03-31)

update untoggled modifier do now not cancel the search (#512)


fix: adapt to kong chart v2.39

kong/kong chart from v2.39 no longer sets the default value of enterprise.rbac.session_conf_secret (https://github.com/Kong/charts/pull/1033). This PR fixes an issue discovered in KIC's integration tests (https://github.com/Kong/kubernetes-ingress-controller/pull/6198) that makes Kong complain about missing admin_gui_session_conf as KTF was relying on the default in charts.

Codecov Report
Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 46.98%. Comparing base (1204d4d) to head (f8ca5b8).

Files
Patch %
Lines

pkg/clusters/addons/kong/addon.go
0.00%
2 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1105       +/-   ##
===========================================
- Coverage   60.21%   46.98%   -13.23%
===========================================
  Files          50       50
  Lines        4079     4082        +3
===========================================
- Hits         2456     1918      -538
- Misses       1313     1917      +604
+ Partials      310      247       -63

Flag
Coverage Δ

integration-test
46.98% <0.00%> (-13.23%)
:arrow_down:

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

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


Can't install!

For some reason I can't install the module via: npm install mumble
npm gives me the following error:
npm ERR! Error: ENOENT, open 'C:\Users\myuser\node_modules\mumble\qgit push'
npm ERR! If you need help, you may report this entire log,
npm ERR! including the npm and node versions, at:
npm ERR!     http://github.com/npm/npm/issues
npm ERR! System Windows_NT 6.1.7601
npm ERR! command "C:\Program Files\nodejs\\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "install" "mumble"
npm ERR! cwd C:\Users\myuser
npm ERR! node -v v0.10.36
npm ERR! npm -v 1.4.28
npm ERR! path C:\Users\myuser\node_modules\mumble\qgit push
npm ERR! code ENOENT
npm ERR! errno 34
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!     C:\Users\myuser\npm-debug.log
npm ERR! not ok code 0
Help would be appreciated. Thank you.

Should be fixed. I had published the NPM version from a bad computer that contained some garbage in the project folder. Should work fine now.


fix: add missing target property in LayoutChangeEvent

Summary:

According to documentation the target property exist:

Also if you log nativeEvent you'll see that the property indeed exist, however from TS declaration perspective this property doesn't exist.
In this PR I'm adding it at according to RN documentation types.
Changelog:

[General] [Fixed] - add missing target property in LayoutChangeEvent

Test Plan:

Before
After

@NickGerleman sorry to tag you, but I've seen you recently worked on reviewing TS mismatches PR - may I kindly ask you to review this PR?

@cipolleschi sorry to tag you, but you are the only one person from Meta with whom I spoke/chatted and who was responsive 😊
Maybe you know a correct person who can review these changes? 👀

@NickGerleman I took specification from website:

Do you think it should be like this?
export type LayoutChangeEvent = NativeSyntheticEvent<{
  layout: LayoutRectangle;
}> & Nullable<TargetedEvent>;

Hmm it looks like both the website, and existing TS types aren’t correct.
https://github.com/facebook/react-native/blob/e1b0914e094a7fd119e54575c5c0ff06008ada22/packages/react-native/Libraries/Types/CoreEventTypes.js#L31
So, it can be nullable, can be used by default in more places, and can also be a ref.

@NickGerleman so would it make sense to make changes to TargetedEvent type and then use union for LayoutChangeEvent?
I'm happy to make these changes and submit a PR, but just want to understand what you'd like to see in that PR 👀

I think the best thing here, if possible, would be to change the TS definitions, to match Flow organization.

@NickGerleman I took a look on flow definitions and it seems like there is also a problem. This declares target property that belongs to event
https://github.com/facebook/react-native/blob/e1b0914e094a7fd119e54575c5c0ff06008ada22/packages/react-native/Libraries/Types/CoreEventTypes.js#L31
And we have this property in TS definition too:
https://github.com/facebook/react-native/blob/e1b0914e094a7fd119e54575c5c0ff06008ada22/packages/react-native/Libraries/Types/CoreEventTypes.d.ts#L44

But this PR adds target property to nativeEvent.  So website has a right definition 😊
Feel free to correct me 🙌

If we’ve never exposed this member in our source of truth typings, I suspect we should probably keep this way.

@NickGerleman but this property was added at some point of time and I guess was added with a purpose? Do you want to remove this property in the future from onLayout event? And because of that you don't want to add this property to public types?

While I haven't dug into the implementation details of why we have a member here, if we have never exposed this on Flow types (or TS types for that matter), the source of truth is that it doesn't exist.
I think we would need to deep dive into the implementation to understand the intentionality here, but it seems like the website is the one at odds with the type system, and even the website previously didn't mention the property in places.
@kirillzyusko did you stumble into this because it was something you were using, or just because of the discrepancy.

did you stumble into this because it was something you were using, or just because of the discrepancy.

Yeah, I started to use it here:
https://github.com/kirillzyusko/react-native-keyboard-controller/blob/main/src/components/KeyboardAwareScrollView/index.tsx#L125
Basically I need to have a viewTag and I discovered on a website that I can retrieve it from onLayout event. I know that I also can find a viewTag using findNodeHandle, but at the same time I knew that in strict mode it's throwing a warning about deprecation. And when I saw this warning I thought that findNodeHandle is deprecated in general, but after spending some time and digging into this I found that it looks like it will throw a warning if not-ref param will be passed. So it's still safe to use findNodeHandle if I pass a ref, right?
Will be ahppy if you can shed some light on it, because I haven't found any information on a website and all my guesses are based on the analysis of several popular open source libraries and RN code.

I'm closing this PR as I think it's not going to be merged


Exporting the query files

Other projects that use the topiary library might want to make direct use of the queries shipped with Topiary, as proven by the effort by @vkleen to integrate Topiary into Nickel. These files should be exported and available for users of the library.

Not sure what you mean by exported, but there is https://doc.rust-lang.org/std/macro.include_str.html, which can be used to include file contents at compile-time.
Found it via https://users.rust-lang.org/t/packaging-assets-in-a-library-crate/4133.
Will create a sample and see how well it works.

@ErinvanderVeen @vkleen Would the above PR work for you?


fixing boolean and ushort tensor construction

Tensor creation for boolean and ushort use wrong data type / data len. Adds unit tests.

Thank you for your contribution


[Relay] Remove Free Variables/Free Type Variables in Module

Module will now CHECK that there aren't any free variables/free type variables.
It simplify all passes as they can assume the absence of free variables.
@zhiics @wweic @junrushao1994 can you guys review?

Functions in a module must be closed in both expression and type scope.

@eqy I change the output of some test in autotvm. Can you look and make sure it is ok?

@MarisaKirisame This change sounds good to me. I took a look into the error. It seems that there is some thing wrong with binding.

@MarisaKirisame hmm, I see you are correcting it. Otherwise, creating a new Function in FromExpr when expr is Function would also work.

@zhiics I did both.

I think the newly added warning in the module should be an error, but there is too much instance and I need this fix to fix PE, so I think we can do it in another PR.

@jroesch @zhiics @eqy This break partial type inference as every type inference will now duplicate all the variables. Is this a reasonable design, and do we need partial TI?

@MarisaKirisame How about making it a pass so pass can depend on it if needed?  Would it fix the partial type inference failure?

@wweic reexposing the function -> function pass will also work.
The idea is that all thing in Module is only suppose to have no free variables, it is incorrect if it has free variable.

@MarisaKirisame @wweic It think making it into a pass probably will not solve the problem. It looks to me that partial TI is not really quite needed if very thing is baked into a module. How do you guys think? @jroesch

@zhiics  I mean not pass, but the function, and deprecate it slowly.
I remember @eqy was relying on partial type inference, so I want his opinion.

I think we should have a special API for partial type inference, so people know when they are opting in. We just make all the free vars dummy parameters, run inference then remove them.

I am also not sure if this breaks things because the quantization PR is not merged yet. There are still a few calls to InferType in the pass but I am not sure they do PI

sounds good to me, so I think we can add this, and add a special type infer function as @jroesch say.


More than one config file specified.

Everything seems to work fine, until protractor_coverage starts up from within grunt.
I can run protractor outside of grunt and everything is fine. But from grunt I get this...
grunt --debug
Running "protractor_coverage:local" (protractor_coverage) task
[D] Task source: /Users/username/repos/grunt-protractor-coverage-example/node_modules/grunt-protractor-coverage/tasks/protractor_coverage.js
Collector started on port 3001
Usage: protractor [options] [configFile]
configFile defaults to protractor.conf.js
The [options] object will override values from the config file.
See the reference config for a full list of options.

Options:
  --help                                             Print Protractor help menu
  --version                                          Print Protractor version
  --browser, --capabilities.browserName              Browsername, e.g. chrome or firefox
  --seleniumAddress                                  A running selenium address to use
  --seleniumServerJar                                Location of the standalone selenium jar file
  --seleniumPort                                     Optional port for the selenium standalone server
  --baseUrl                                          URL to prepend to all relative paths
  --rootElement                                      Element housing ng-app, if not html or body
  --specs                                            Comma-separated list of files to test
  --exclude                                          Comma-separated list of files to exclude
  --verbose, --jasmineNodeOpts.isVerbose             Print full spec names
  --stackTrace, --jasmineNodeOpts.includeStackTrace  Print stack trace on error
  --params                                           Param object to be passed to the tests
  --framework                                        Test framework to use: jasmine, cucumber or mocha
  --resultJsonOutputFile                             Path to save JSON test result
  --troubleshoot                                     Turn on troubleshooting output

Error: more than one config file specified
>>
>> Test failed but keep the grunt process alive.
Shutdown requested

Done, without errors.

Here is my Gruntfile:
module.exports = function(grunt) {

  require('load-grunt-tasks')(grunt);

  grunt.initConfig({

    express: {
      options: {
        port: 9000,
        base: 'instrumented'
      },
      cov: {
        options: {
          script: 'instrumented/src/file1.js',
          debug: true
        }
      },
    },
    open: {
      server: {
        url: 'http://localhost:9000'
      }
    },

    copy: {
      cov: {
        files: [{
          expand: true,
          dot: true,
          src: 'src/*.js',
          dest: 'instrumented',
        }]
      }
    },

    // start - code coverage settings
    instrument: {
      files: ['src/file1.js'],
      options: {
        // lazy: false,
        lazy: true,
        basePath: 'instrumented'
      }
    },

    protractor_coverage: {
      options: {
        configFile: 'protractor.conf.js',
        keepAlive: true,
        noColor: false,
        coverageDir: 'instrumented',
      },
      local: {
        options: {
          args: {
            baseUrl: 'http://localhost:3000/',
            'browser': 'chrome'
          }
        }
      }
    },

    makeReport: {
      src: 'instrumented/*.json',
      options: {
        type: 'html',
        dir: 'reports/',
        // print: 'detail'
        //        type: 'lcov',
        //        dir: 'reports',
        //        print: 'detail'
      }
    },

  });

  grunt.registerTask('default', [
    // 'copy:cov',
    // 'instrument',
    // 'express:cov',
    'protractor_coverage:local',
    // 'protractor_coverage',
    // 'makeReport'
  ]);
};

You can see the rest of my code here:
My code is here: https://github.com/F1LT3R/grunt-protractor-coverage-example
I would be interested to see if you know what is wrong here. I've been trying to solve this problem on and off for days now.

Something fishy is going on, at first I could reproduce your problem, but after I installed the Selenium jars with node_modules/protractor/bin/webdriver-manager update I got other problems, which I also can`t reproduce anymore, despite a fresh git clone/checkout.
Anyway pay attention to your inconsisted usage of the server port:

connect start server on 3000,
protractor gets the baseUrl localhost:9000 and
in your test  you do a browser.driver.get('http://<IP_ADDRESS>:3000'); (use browser.driver.get('/') if you define a baseUrl)

Hope this helps.

Thanks @weemonger will check this out later.

I have this issue when running grunt with the --debug flag. Without the --debug flag everything runs fine.


postcss: @apply rules won't find @imported css

There's an ordering error with postcss-next and postcss-import.
Behaviour
If you have custom properties sets on a individual @imported file, postcss-apply will fail to find them.
The following message will appear on console:
...
postcss-apply: ...src/css/main.css:24:3: No custom properties set declared for `hero-font`.
...

How to fix
Thankfully, it's an easy fix.
Just swap the order, they're processed in your gulpfile.babel.js, letting cssImport first.
gulp.task("css", () => (
  gulp.src("./src/css/*.css")
    .pipe(postcss([cssImport({from: "./src/css/main.css"}), cssnext()]))
    .pipe(gulp.dest("./dist/css"))
    .pipe(browserSync.stream())
));

Test example:
File: src/css/imports/rules.css
:root {
   --hero-font: {
    font-family: 'Antic', sans-serif;
    font-size: 2em;
  };
}

File: src/css/main.css
@import "imports/rules.css";

h1 {
   @apply --hero-font;
}

Thankfully, it's an easy fix.

Why the issues template has a How to fix section is beyond me. @barrabinfc have you looked at hugulp as a potential alternative to Webpack?

Thanks a lot for your clear description @barrabinfc. I've opened #23 to use your suggestion.


Reclaiming Slab Memory

Currently, tokio-core uses the slab crate to store timer heaps and dispatch maps for IO events and tasks. However, the slab crate currently offers no way to reclaim memory. This means that the size of a reactor::Inner is determined by the maximum number of IO events, tasks, or timers that have existed at any one time. Once that maximum has been raised, the reactor can never return to its original size.
This is a problem when using tokio-core to build long-running infrequently-called systems services which must scale to high loads and then drop back down to low resource usage when inactive. I don't personally have memory usage benchmarks available, but for a quick back-of-the-napkin estimate:
If a user has 100,000 IO events + tasks open at one time, then it's holding memory for 100,000 ScheduledIo/ScheduledTasks. A ScheduledIo is a pointer and two Option<Task>s, and size_of::<Option<Task>> is 80 bytes. 100,000 * ((2 * 80 bytes) + 64 bits) = 16.8 megabytes.
I'm not sure how relevant this is to web-servers and other cloud applications (which are often running on the JVM, which hogs far more than 17 megabytes). However, I'm concerned about the impact of this memory usage on local high-demand services. Has there been any discussion of ways to reclaim memory while using slab, or of another data structure that could be used? The obvious solution is a HashMap, but I'm sure that would result in lost performance.

The main issue here is not so much going to be the size of memory allocated by the slab, but more so that "live" entries in the slab could be scattered across the memory, keeping the pages alive.
Generally, inactive memory pages will be reclaimed by the OS as needed. Is that a fair assumption?

@carllerche That seems fair to me, although I'd add that the OS reclamation would be more heavy-weight and less precise than a slab or allocator free.
If we could find a way to keep the active elements isolated to a particular page of memory, that would help a lot (both for memory usage and cache performance).

Thanks for the report @cramertj!
It was my hope that we wouldn't have to worry about this for quite some time... It would be relatively straightforward today to add some shrink_to_fit calls but as @carllerche mentions the slab is proportionally sized by the largest active token, which is itself on the order of the maximum number of concurrent events at once. Even if we were to shrink the slab itself there'd be no guarantee that we could actually reclaim memory.
This will also get significantly more difficult with tokio-rs/tokio-rfcs#3 and the proposed implementation, which has some atomic management of the internal slab and would make reclaiming the memory much more difficult.

I've been thinking a bunch about this, I think that you could divvy up the memory into regions (where each region is a slab) and free a region when it is unused.
This would mean that there would be a free list per region and regions would be prioritized so that, eventually lower priority regions become unused.
This would work in the atomic case mentioned above and could even provide improved concurrent behavior as regions could be assigned to threads to lower contention.

@carllerche Yes, I'd been thinking about a similar strategy of generation-based allocation, sort of a slab-of-slabs that would allow you to collect an entire slab once it was emptied.


np.angle returns incorrect type since 0.57.0

Reporting a bug

[x] I have tried using the latest released version of Numba (most recent is
visible in the change log (https://github.com/numba/numba/blob/main/CHANGE_LOG).
[x] I have included a self contained code sample to reproduce the problem.
i.e. it's possible to run as 'python bug.py'.

As the title says, I've noticed that since the last release np.angle seems to be returning arrays with incorrect type when jitted with numba. Instead of the expected float64, the arrays are complex128.
This is causing issues on my side, with some jitted methods that no longer compile due to the wrong types.
The code below reproduces the issue:
import numpy as np
from numba import njit

def numpy_angle(x):
    theta = np.angle(x)
    print(theta.dtype)

@njit()
def numba_angle(x):
    theta = np.angle(x)
    print(theta.dtype)

x = np.array([[1+2j,1+3j],[5+6j,3+8j]])
numpy_angle(x)
numba_angle(x)

output with numba 0.56.4:
float64
float64

output with numba 0.57.0:
float64
complex128

Many thanks for the report and nice reproducer! This was introduced by a523ecff7ce9aee5dc86ae465236420024c3e651 in #8234.

Thanks for looking into it!

I looked into this issue.
Looking at this, which was removed in #8234:

And this, which is what the np.angle implentation looked like post-refactor:

I tried the following patch, which does indeed fix the reproducer submitted? Can anyone confirm?
diff --git i/numba/np/arraymath.py w/numba/np/arraymath.py
index 537c226ce8..fc27dfb639 100644
--- i/numba/np/arraymath.py
+++ w/numba/np/arraymath.py
@@ -3212,8 +3212,13 @@ def ov_np_angle(z, deg=False):
                 return np.arctan2(z.imag, z.real)
         return impl
     elif isinstance(z, types.Array):
-        ret_dtype = z.dtype
-
+        dtype = z.dtype
+        if isinstance(dtype, types.Complex):
+            ret_dtype = dtype.underlying_float
+        elif isinstance(dtype, types.Float):
+            ret_dtype = dtype
+        else:
+            return
         def impl(z, deg=False):
             out = np.zeros_like(z, dtype=ret_dtype)
             for index, val in np.ndenumerate(z):

Fixed in https://github.com/numba/numba/pull/8965

closing this as #8965 has been merged which fixes this issue.


DFS

Have you read the Contributing Guidelines on Pull Requests?
YES
Description
DFS Algorithm for traversing a graph
Checklist

[YES] I've read the contribution guidelines.
[YES] I've checked the issue list before deciding what to submit.
[YES] I've edited the README.md and link to my code.

Related Issues or Pull Requests
#61

@ankitaggarwal23 Why you close it without merging?

@ankitaggarwal23 Why you close it without merging?

The time is Up? You didn't even tag anyone of the mentor for review. How will come to know about it for review.


Firmware update tool is displaying wrong serial numbers

Required Info

Camera Model
D415 & D435

Firmware Version
05.09.{09?,11}.0 -> <IP_ADDRESS>

Operating System & Version
Linux (Ubuntu 16.04.5 LTS)

Kernel Version (Linux Only)
4.4.0

Platform
PC

SDK Version
2.14.1

Language
-

Segment
others

Issue Description
While updating the firmware of our RealSenses I've noticed that the serial number displayed by intel-realsense-dfu is different from the real serial number (as printed on the RealSense, packaging and reported by rs-enumerate-devices and realsense-viewer).
This happened with each of the D415 and D435 cameras that I've updated.
If this is supposed to be the serial number of the RealSense then this is displayed wrong by intel-realsense-dfu.
Version of the update tool: 0.9.8

[Realsense Customer Engineering Team Comment]
Hi @insensiv-ah,
I don't see this issue. What command option do you used for 'intel-realsense-dfu'?

Hi @RealSense-Customer-Engineering,
I call the update program like this:
intel-realsense-dfu -b 002 -d 004 -f -i Signed_Image_UVC_5_9_14_0.bin

I set -b and -d according to the output of lsusb.
The -f is probably unnecessary in most cases but in general it's the same call as given in the DFU guide.

[Realsense Customer Engineering Team Comment]
Hi @insensiv-ah,
Yes, you are right. The serial number in intel-realsense-dfu is ASIC serial number, not the same optical serial number showing in realsense-viewer or rs-enumerate-devices. We should make this clearly.

[Realsense Customer Engineering Team Comment]
The DFU tool is updated and the wrong S/N should not display. Please have a try.


Add new link subcommand

Fix #107

CI passed with the new uv pipeline!
The new pipeline also shaves off 20+ seconds per pipeline!


Debug the database and local storage.

Hi, I am working on an app that saves some data on the sqlite and some file in the local storage. Is there a method to debug it from my computer?
I mean, I can see the file structure and the database from my computer, and I can open the file or edit it from my computer.

Questions like these are better suited for Q&A sites like StackOverflow or on one of the flutter community sites at https://flutter.dev/community


TrackerAdjoint on GPUs

using OrdinaryDiffEq, StochasticDiffEq, Flux, DiffEqSensitivity, DiffEqFlux, Zygote, Test, CuArrays
CuArrays.allowscalar(false)

mp = Float32[0.1,0.1] |> gpu
x = Float32[2.; 0.] |> gpu
xs = Float32.(hcat([0.; 0.], [1.; 0.], [2.; 0.])) |> gpu
tspan = (0.0f0,25.0f0)
dudt = Chain(Dense(2,50,tanh),Dense(50,2)) |> gpu

node = NeuralODE(dudt,tspan,Tsit5(),save_everystep=false,save_start=false,sensealg=TrackerAdjoint())
grads = Zygote.gradient(()->sum(node(x)),Flux.params(x,node))

@YingboMa can I get help tracking down (pun intended) what's going on here? Something is hitting scalar indexing but I don't know why.

It sounds like using Tracker makes this not work: https://github.com/JuliaDiffEq/RecursiveArrayTools.jl/blob/master/src/init.jl#L14-L21 but I don't get why that would be the case.

Trying to hack around and do the conversion to a matrix on the GPU drops gradients:
    vecs = Tracker.collect(Array(VectorOfArray(adapt.(Array,sol.u))))
    adapt(typeof(u0),vecs)

@MikeInnes do you have a solution for this? It's just changing a vector of TrackedArray{CuArray} into a concatenated TrackedArray{CuArray}. MWE:
using Tracker, CuArrays
u = [Tracker.TrackedArray(cu(rand(4))) for i in 1:2]
reduce(hcat,u)

The MWE should be
using Tracker
reduce(hcat,[param(rand(4))])

There's nothing CuArrays specific about this except that we can't have CuArray{TrackedReal}, so it errors out. hcat(u...) works fine in both cases so the reduce(hcat, ...) form just needs an adjoint (possibly just forwarding to hcat).

Fixed in https://github.com/JuliaDiffEq/DiffEqSensitivity.jl/pull/188


Pseudoselectors separated by comma aren't supported in the parser

    &::after, &::before {
      width: 100%;
      left:  0;
    }

I tested locally with the demo and it works, aren't you missing content: '' ?

Totally, I tested myself and i reported this issue from the ahrefs Slack channel directly without trying it first.


[1.12] Step Up Permanently Feature Request

Minecraft version & Mod Version:
MC: 1.12
Cyclic: 1.7.3
Single player or Server:
All of the above
Describe problem (what you were doing / what happened):
It's not a problem, per se. I would like to request a config option that allows the step up ability from the apple of lofty stature effect to be available all the time without eating the item. If you do this, I will grant you my dowry.
My dowry includes the following:
16 chickens
2 pigs (both male)
4 1/2 pairs of pantaloons
7 socks (not paired)
My 1 horse
My save in Cookie Clicker
A bottle of slightly expired salad dressing
Dark Souls 2: Scholar of the first sin edition
And a partridge in a pear tree
Crash log (if any):
This mod has never caused any crashes because it is simply amazing AND! for the low price of a step up config option it could be THE GREATEST MOD OF ALL TIME!
THANKS YOU!
Darkosto

Yeah, what i can do is just add a config so that we default new players with it turned on instead of turned off (so they could still craft and eat to toggle as normal.)

I've also added "HeartContainerDefault" just for fun

I tested these in both single player and multiplayer, let me know if theres any issues.  1.7.4 https://minecraft.curseforge.com/projects/cyclic/files/2460319

Thank you <3 <3 <3


allow from_fn to change body type

I think Next::call is better defined in inherent impl to avoid  importing Service.


Add support for using the smartcard connector app from a web page

Currently the connector app accepts connections only from Chrome apps or extension and implements client whitelisting based on the extension id.
To accept connection from web pages the connector app's manifest.json needs to include "externally_connectable" key.
For the connections coming from web pages the extension id is null, so whitelisting mechanism needs to be updated to use the client web page origin URL, which is available in sender.url (see chrome.runtime.onMessageExternal.addListener).
The whitelisted url format should allow for wildcards, e.g. *.domain.com

Hi, thanks for filing this. This is in plans, although not yet in active development, since we're trying to find a way to limit it only to trusted web pages (for which being on the allowlist might not necessarily be a sufficient criteria alone). We'll share the details once it's fleshed out with security.

Any updates?

Nothing to share yet. An internal team in Google is still working on a security model for exposing such "sensitive" APIs.

@emaxx-google Would you (google) like to collaborate? I'd like to help where I can, if I can.

@hcldan This effort is still in the design stage, so unless you have specific ideas of how this can be designed, any concrete steps (like pull requests / change lists) aren't possible at the moment, unfortunately.
 The open web doesn't offer sufficient trust boundaries. Folks are trying to find a reasonable and maintainable architecture that would allow the page to declare (and prove) its conformance to stricter security boundaries, and gate access to sensitive APIs like PC/SC-Lite only to such pages.

"regular web page"?
I mean there's a lot of development going on for PWA's where google is doing things in a secure way.
An https origin with a trusted certificate chain that's allow-listed by user opt-in or admin policy push isn't enough?

And to that effect... it would be great if this entire extension was merged into chrome-proper so I could use it in a PWA.
Could even prompt to allow access to it like NativeFileSystem does.. etc.

I'm not a stakeholder on that topic, but, yes, security teams reviewed the proposal to expose PC/SC to PWAs and explicitly declined it. Definitely not in all, but in many aspects a PWA is a regular web page. Admin policies or user prompts weren't considered to be sufficient security measures - because either are unreliable signals about whether the admin/user actually understands the risk and acknowledges it.

I understand their focus on security.  Scoped access would be fine. Provision a key I can encrypt/decrypt from that origin only and give me apis to access the cert, verify signatures en/de-crypt.
Are all of these discussion happening behind closed doors?

I see where you're coming from. But not sure how such scoped access would map to PC/SC though - would we need to emulate a device that pretends to have only one certificate? That sounds tricky (especially at the APDU level), and also not a feature request we've heard before.
Some other APIs/technologies offer scoped access in a natural way. FIDO is one good example - not sure if you could work with it? Another is chrome.platformKeys/chrome.enterprise.platformKeys - currently it only supports TPM-backed keys though (integrating it with smart cards was attempted a few years ago, but got stuck due to security concerns around potential cross-protocol attacks). Another option is WebCrypto.

Are all of these discussion happening behind closed doors?

I've checked a few chromium.org bugs and docs - seems like so. I know that's unfortunate, but there's nothing I can do about changing the processes; probably it's a consequence of this topic being both "work-in-progress" and "security-sensitive".

Well I know I can create and persist origin scoped keys in indexeddb...
I don't get access to the key itself, just a key object I can use with webcrypto.  That would be a start.
If the SC keys tagged with my origin are the only ones allowed, and I can use webcrypto apis to use them... I think that's all I would need.
Re: FIDO - WebAuthn doesn't do crypto, only auth.   Already looked here.
Re: WebCrypto - Yes I would use that, but I want the key stored securely on the smartcard.
I don't need full PC/SC interface...   I just need the operations that one would typically use it for.
Some minor key management (origin scoped)
sign stuff
encrypt stuff
decrypt stuff
Hopefully using the webcrypto apis and the same kind of key object I can already work with there and serialize into indexeddb.

I'm not aware of plans to wire up WebCrypto with smart cards (and I imagine this would be a nontrivial project, at least due to the amount of information that WebCrypto abstracts away). My question was whether a locally stored would be suitable for your purposes - but seems like you need SC integration.
If other options listed above don't work for you, I'm not sure about other alternatives besides having a companion application that would do crypto for you. I.e., on Chrome OS, this companion would be an extension that talks PC/SC to the Connector, and on desktop OSes this would be a native application that uses smart card primitives provided by the OS and exposes them to the PWA via native messaging.

This bug is closed, but note that the allowlist of web pages that can talk to the Smart Card Connector is still empty. This is still blocked on getting the corresponding trust guarantees implemented in Chrome.


Sort Market Table Suggestion (Index Page)

I wanted to suggest that sorting ability be added to the market table on the index page.
This would then give users ability to sort coins alphabetically, price and percentage change.
Maybe a nice feature to have.

@daygle
Follow these steps:

Download https://www.kryogenix.org/code/browser/sorttable/sorttable.js  to static_pages folder
Add  to head.ejs
Add class sortable (  ) to market table and thats it

How about a pull request @coinetworkpw, let's add it here to benefit others?


fix for issue 327

see https://github.com/canoo/dolphin-platform/issues/327

This change is

Coverage remained the same at 68.456% when pulling 84cb3a9b44b771d563dec3af594622cde3eebced on json-test-fix into 4f8bf0489fcd78af70bf375943c922017e0456fa on master.


dataframe operator is now appending to it's output table instead of replacing.

Describe the bug
the dataframe operator is now appending to it's output table instead of replacing.
refer to https://astronomer.slack.com/archives/C02B8SPT93K/p1668643987896139
Version

Astro: [e.g. 0.6.0]
OS: [eg. Debian]

To Reproduce
Steps to reproduce the behavior:

Write the DAG  '...'
Create connection '....'
Run using  '....'
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.
Additional context
Add any other context about the problem here.

This was caused after the upgrade from 1.1.1 (working) to 1.2.1 (causing dupes)

Related to this as well: https://github.com/astronomer/astro-sdk/issues/1273


make test-connection more configurable (image & resources)

I would like to be able to specify the image to be used for the connection test as well as its resources.

Sorry about that and thanks!


Support nanosecond timestamps, specifying version type in write_table()

Describe the bug
Using either a data loader or exporter, nanosecond precision timestamps aren't supported.
    432         path_or_buf,

/usr/local/lib/python3.10/site-packages/pandas/io/parquet.py in write(self, df, path, compression, index, storage_options, partition_cols, **kwargs)

    202             else:

    203                 # write to single output file

--> 204                 self.api.parquet.write_table(

    205                     table, path_or_handle, compression=compression, **kwargs

    206                 )
/usr/local/lib/python3.10/site-packages/pyarrow/parquet/core.py in write_table(table, where, row_group_size, version, use_dictionary, compression, write_statistics, use_deprecated_int96_timestamps, coerce_timestamps, allow_truncated_timestamps, data_page_size, flavor, filesystem, compression_level, use_byte_stream_split, column_encoding, data_page_version, use_compliant_nested_type, encryption_properties, write_batch_size, dictionary_pagesize_limit, **kwargs)

   2983                 dictionary_pagesize_limit=dictionary_pagesize_limit,

   2984                 **kwargs) as writer:

-> 2985             writer.write_table(table, row_group_size=row_group_size)

   2986     except Exception:

   2987         if _is_path_like(where):
/usr/local/lib/python3.10/site-packages/pyarrow/parquet/core.py in write_table(self, table, row_group_size)

   1052             raise ValueError(msg)

   1053

-> 1054         self.writer.write_table(table, row_group_size=row_group_size)

   1055

   1056     def close(self):

/usr/local/lib/python3.10/site-packages/pyarrow/_parquet.pyx in pyarrow._parquet.ParquetWriter.write_table()

/usr/local/lib/python3.10/site-packages/pyarrow/error.pxi in pyarrow.lib.check_status()
ArrowInvalid: Casting from timestamp[ns] to timestamp[us] would lose data:<PHONE_NUMBER>344693144

To Reproduce

Create a data loader or data exporter.
Try to return or write data with nanosecond precision.

Expected behavior
Support of nanosecond precision.
Desktop (please complete the following information):

OS: MacOS
Browser Chrome

Additional context
Pyarrow supports this precision but it doesn't do it by default because of Spark.
It seems it's the write_parquet() function in mage which is causing the issue.
This seems to solve it:
pq.write_table(table, 'machine_data.parquet', version='2.6')

In the meantime data loader code fixed by changing the precision:
from influxdb_client_3 import InfluxDBClient3
import pandas as pd

if 'data_loader' not in globals():
    from mage_ai.data_preparation.decorators import data_loader
if 'test' not in globals():
    from mage_ai.data_preparation.decorators import test

client = InfluxDBClient3(token=token,
                         host=host,
                         org=org,
                         database=db)

@data_loader
def load_data(*args, **kwargs):
    """
    Template code for loading data from any source.

    Returns:
        Anything (e.g. data frame, dictionary, array, int, str, etc.)
    """
    query = '''
    SELECT time, vibration, machineID FROM machine_data WHERE time >= now()  - 1h
    '''
    table = client.query(query=query, language="influxql", mode="all")
    df = table.to_pandas()
    # Ensure the 'time' column exists and convert its precision
    if 'time' in df.columns:
        time_data = df['time'].values.astype('datetime64[us]')
        df['time'] = pd.to_datetime(time_data)
        return df
    else:
        print("error")

@test
def test_output(output, *args) -> None:
    """
    Template code for testing the output of the block.
    """
    assert output is not None, 'The output is undefined'
    ```

👀

Thank you for reporting this issue and providing the solution! We'll add the support for it.

I think nanosecond is supported in version 0.9.61 and above after we upgrade pyarrow version.


Problem with linking stage in static builds

For the past few days, I've been trying to build a static executable of a monomer app. I'm now stuck on a linking problem.
/usr/lib/gcc/x86_64-alpine-linux-musl/10.3.1/../../../../x86_64-alpine-linux-musl/bin/ld: cannot find -lGLU
/usr/lib/gcc/x86_64-alpine-linux-musl/10.3.1/../../../../x86_64-alpine-linux-musl/bin/ld: cannot find -lGL
/usr/lib/gcc/x86_64-alpine-linux-musl/10.3.1/../../../../x86_64-alpine-linux-musl/bin/ld: cannot find -lX11
/usr/lib/gcc/x86_64-alpine-linux-musl/10.3.1/../../../../x86_64-alpine-linux-musl/bin/ld: cannot find -lX11
/usr/lib/gcc/x86_64-alpine-linux-musl/10.3.1/../../../../x86_64-alpine-linux-musl/bin/ld: cannot find -lGLU
/usr/lib/gcc/x86_64-alpine-linux-musl/10.3.1/../../../../x86_64-alpine-linux-musl/bin/ld: cannot find -lGL
/usr/lib/gcc/x86_64-alpine-linux-musl/10.3.1/../../../../x86_64-alpine-linux-musl/bin/ld: cannot find -lGLU
/usr/lib/gcc/x86_64-alpine-linux-musl/10.3.1/../../../../x86_64-alpine-linux-musl/bin/ld: cannot find -lGL

It says basically that GL, GLU and X11 libraries are missing, but they're installed and in /usr/lib, so I see no reason why it's failing. Here's a simple repo with the official example and a Dockerfile:
https://github.com/sproott/monomer-static-build-problem
To run the build, do docker build ..

I honestly don't know what could be happening. This sounds more like an issue caused by what GHC/Stack expect vs what Alpine provides.
There were some comments about static builds in the announcement post, but it's not something I have investigated.
I'll follow your related issue.

Alright, well, from the comments on Reddit, it seems that it's impossible to statically link GL and X11, so I'll probably just stick to shipping a distro-specific dynamic executable.
Should I close this issue then?

Sure! If you manage to make it work, it would be great to know!
Thanks!


NAMEinator_0.0.5_windows_amd64.tar.gz is missing the datasrc folder

The Win version is missing the data it needs to actually run and do anything. Has to be found in source.zip or source.tar.gz (and people unfamiliar with this sort of source-based and command-line-oriented tool may not understand they should look there for it).
PS: It doesn't make much sense to distribute Windows packages in tar.gz form.  It doesn't provide any benefit, and a lot of Windows people won't know what to do with it, with what tools (nor are all of them very good for this; e.g. 7Zip, which "does everything" doesn't quite; you have to extract the gz then the tar as separate operations, as it doesn't seem smart enough to interpret it as a single archive to process in two steps).  ZIP would better, as Windows Explorer has support for that built in already and pretty much everyone already knows what to do with it.
PPS: Thanks for including <IP_ADDRESS> and <IP_ADDRESS> by default.  I found it irritating that the original NameBench 1.3.1 did not, and still won't run tests against them by default even after you add them to the config file; you have to manually add them to the list of servers to use in the top line of the GUI - every single time, since it doesn't save your changes to that line.

I've made changes and now the files are embedded into the binary in PR #54, its already merged.


fix(web_socket): call on_close for all errors

I occasionally see Errno on our webserver when calling run and all the clean-up code is in the on_close callback which is not called.
Consider the websocket handler in the stdlib, when run is called and an IO error such as Errno occurs on_close is never called and hence any references stored by the user defined code are leaked
Not something I'm able to replicate, hence the lack of tests, but I see it in the logs and our monitoring also confirms a leak.

Why not remove the exception's type restriction entirely and call on_close on every error? Technically, there might not be any other than IO::EOFError | Errno being raised, but it should still express the intention. We can assume that when ws.receive raises (for whatever reason) items the websocket is no longer in a working state.

Yeah good point @straight-shoota I'll update

feel like reviewing @straight-shoota ?

IMO good candidate for 0.32.1.

Sure, let's make it to 0.32.1.


Add experimental API to generate arbitrary pointers

This change adds a pointer generator that can non-deterministically generate a pointer with different properties.
This generator allows users to build pointers with different allocation status, initialization and alignment.
It contains an internal buffer that it uses to generate InBounds and OutBounds pointers.
In those cases, the pointers will have the same provenance as the generator, and the same lifetime.
This approach is different than generating a pointer from an arbitrary usize. Kani uses demonic non-determinism to track
allocation lifetimes, which makes hard to reason about during verification. I.e., one cannot assume a pointer is valid, and initialized, and this can only be accomplished by manually tracking the pointer status.
I added the new API under -Z mem-predicates since it allows reasoning about memory, and I was hoping this wouldn't need another unstable flag. :smile:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

Appears to break std verification:
error[E0433]: failed to resolve: use of undeclared crate or module `kani`
   --> /home/runner/work/kani/kani/verify-rust-std/library/core/src/lib.rs:426:1
    |
426 | kani_core::kani_lib!(core);
    | ^^^^^^^^^^^^^^^^^^^^^^^^^^ use of undeclared crate or module `kani`
    |
    = note: this error originates in the macro `kani_core::generate_arbitrary` which comes from the expansion of the macro `kani_core::kani_lib` (in Nightly builds, run with -Z macro-backtrace for more info)

@tautschnig, hopefully I addressed all your comments. I also moved the pointer type to be an argument of the methods instead of the generator type.


You can still access os in lua

Describe your bug here.
Pysch supposedly block users from using os library for security
(or is it?)
Apparently you still can access to os library in lua and bypass it by doing this
local os = os
-- or
local os = package.loaded.os -- found this when i was testing my custom print function that also prints the contents of the table

function onSongStart()
	os.exit(0)
end

I would be happy if anyone does an pr on this
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
No

what version of psych you using? if this is 0.5.2 at least, yes normally.


Selecting a CCMotionStreak sets the opacity to 0

yeah. read the title.

this is an issue with cocos2d


this.context.store vs props mapping

Hi, I have a component where I map the store properties to the component properties:
import Inferno from 'inferno';
import Component from 'inferno-component';
import { connect } from 'inferno-redux';

import { getAllOverviewCharts } from 'actions';
import { getVisibleCharts } from 'chartListReducer';

import LineChart from 'LineChart';

class Overview extends Component{
    constructor(props){
        super(props);
    }

    render(){
        const store = this.context.store;

        var charts = [];
        this.props.charts.forEach(chart => {
            charts.push(

{ chart.headline }

                        <LineChart
                            data={ chart.data }
                            class="o-flag"
                        />

            );
        });

        return (

Account Overview

                { charts }

        );
    }
}

const mapStateToProps = (state) => {
    return {
        charts: getVisibleCharts(state)
    }
};

export default connect(
    mapStateToProps,
    { getAllOverviewCharts }
)(Overview);

Now i can use this.props.charts.forEach or store.getState().overview.charts.forEach. My question is: What is better and what are the differences? Okay, I know, can filter my information inside the mapping.   But are there more benefits of using the mapping?

This sounds like a Redux question, more than an Inferno question. I'd personally say (off the top of my head) than you could map this.props.charts rather than using forEach and using the props is better than doing store.getState(). I'll CC in @gaearon in case he see this and has time to also answer. :)

Don’t use context, your code will break when React changes it. It’s an unstable API.
connect() exists precisely to shield you from its usage.


[T3A4][W14-A2]Tan Lok You

Ready for review

Some comments added. Feel free to add more comment if you have any questions. Once you are done, you may close the PR after reading comments.


app:view:edit: fix wrong CVE list parsing

When editing AVG's, the view handler splits the data from the form using
'\r\n'. This allows for the creation of spureous CVE values if two cve's
are input using a space, instead of a new line and carriage return.
Split the CVE field using multiline_to_list. This is the method used for
the add views, and it handles this case properly.
fixes #5

applied via ea6a564718dbba7bc3b8056694c0210a73ab6481


feat(eddsa-poseidon)!: make EdDSAPoseidon return bigints not strings

BREAKING CHANGE: Return type changes will require users to update their code.
re #199
Description
See #199 for the argument for this change, and the discussion.
This PR changes the return types of all EdDSAPoseidon functions to bigint rather than stringified bigint.  It also updates all callers within the zk-kit repo, including in other packages.  This is a breaking change, so users of this package will probably need to update some code.  I request that maintainers update version numbers for the next release appropriately, or let me know if I should do so in this PR.
I want a sanity-check for the changes outside of eddsa-poseidon.  I was surprised (by build and test failures) to discover that other packages within the monorepo didn't depend on the latest version of eddsa-poseidon (and other packages such as utils).  I updated those dependencies, so that I wouldn't force future maintainers to update the call sites for the breaking change.  I would appreciate confirmation of whether that's the preferred best-practice in this repo.
Related Issue(s)
Closes #199
Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my 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 run yarn prettier and yarn lint without getting any errors
[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

Thanks @artwyman, left some comments and suggestions!

I request that maintainers update version numbers for the next release appropriately, or let me know if I should do so in this PR.

I can do it after merging. I think all the packages involved are under v0, so we could increase the minor as soon as we consider these packages still unstable.

I updated those dependencies, so that I wouldn't force future maintainers to update the call sites for the breaking change. I would appreciate confirmation of whether that's the preferred best-practice in this repo.

I wrote a comment with an assumption. I think it's ok to have them updated in this PR. From now on the inter-dependencies should be updated correctly. I'll make sure they work.


[YouTube] HTTP Error 403: Forbidden

Since updating to the latest version, I get a lot of errors trying to download from Youtube:
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-x', u'https://www.youtube.com/watch?v=bxjw_8RpoNI', u'-v']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2<IP_ADDRESS>
[debug] Python version 2.7.8 - Windows-7-6.1.7601-SP1
[debug] exe versions: ffmpeg N-70411-g7da7d26, ffprobe N-70411-g7da7d26, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] bxjw_8RpoNI: Downloading webpage
[youtube] bxjw_8RpoNI: Downloading video info webpage
[youtube] bxjw_8RpoNI: Extracting video information
[youtube] {22} signature length 40.42, html5 player new
[youtube] {43} signature length 40.42, html5 player new
[youtube] {18} signature length 40.42, html5 player new
[youtube] {5} signature length 40.42, html5 player new
[youtube] {36} signature length 40.42, html5 player new
[youtube] {17} signature length 40.42, html5 player new
[youtube] {137} signature length 40.42, html5 player new
[youtube] {248} signature length 40.42, html5 player new
[youtube] {136} signature length 40.42, html5 player new
[youtube] {247} signature length 40.42, html5 player new
[youtube] {135} signature length 40.42, html5 player new
[youtube] {244} signature length 40.42, html5 player new
[youtube] {134} signature length 40.42, html5 player new
[youtube] {243} signature length 40.42, html5 player new
[youtube] {133} signature length 40.42, html5 player new
[youtube] {242} signature length 40.42, html5 player new
[youtube] {160} signature length 40.42, html5 player new
[youtube] {278} signature length 40.42, html5 player new
[youtube] {140} signature length 40.42, html5 player new
[youtube] {171} signature length 40.42, html5 player new
[youtube] {249} signature length 40.42, html5 player new
[youtube] {250} signature length 40.42, html5 player new
[youtube] {251} signature length 40.42, html5 player new
[youtube] bxjw_8RpoNI: Downloading DASH manifest
WARNING: [youtube] bxjw_8RpoNI: Skipping DASH manifest: ExtractorError(u'Could not download DASH manifest: HTTP Error 403: Forbidden (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.',)
[debug] Invoking downloader on u'https://r16---sn-5hnednel.googlevideo.com/videoplayback?clen=4060730&key=yt5&sver=3&requiressl=yes&gir=yes&nh=IgpwcjAyLmFtczE2KgkxMjcuMC4wLjE&sparams=clen%2Cdur%2Cgcr%2Cgir%2Cid%2Cinitcwndbps%2Cip%2Cipbits%2Citag%2Ckeepalive%2Clmt%2Cmime%2Cmm%2Cmn%2Cms%2Cmv%2Cnh%2Cpl%2Crequiressl%2Csource%2Cupn%2Cexpire&mt=1439797792&expire=1439819462&lmt=1438962889419816&upn=P71lrLnNvqw&initcwndbps=2231250&id=o-ACaAxzckRGQRU8cpWArGrHuqA5qKGrUUZi-FRVPDeSWh&pl=19&dur=258.641&mime=audio%2Fwebm&ms=au&source=youtube&mv=m&mm=31&ipbits=0&mn=sn-5hnednel&itag=251&keepalive=yes&gcr=nl&fexp=9405348%2C9406004%2C9406851%2C9407155%2C9408372%2C9408486%2C9408710%2C9409069%2C9412515%2C9415365%2C9415485%2C9416023%2C9416126%2C9416338%2C9416574%2C9416729%2C9417430%2C9417707%2C9418133%2C9418153%2C9418335%2C9419301&ip=<IP_ADDRESS>&signature=2F58F49F66389A99E03FA6672FBC7FF40804136E.B740A21A3057758FA4B8F5625E521390BA0940E8&ratebypass=yes'
ERROR: unable to download video data: HTTP Error 403: Forbidden
Traceback (most recent call last):
  File "youtube_dl\YoutubeDL.pyo", line 1585, in process_info
  File "youtube_dl\YoutubeDL.pyo", line 1527, in dl
  File "youtube_dl\downloader\common.pyo", line 342, in download
  File "youtube_dl\downloader\http.pyo", line 60, in real_download
  File "youtube_dl\YoutubeDL.pyo", line 1860, in urlopen
  File "urllib2.pyo", line 410, in open
  File "urllib2.pyo", line 523, in http_response
  File "urllib2.pyo", line 448, in error
  File "urllib2.pyo", line 382, in _call_chain
  File "urllib2.pyo", line 531, in http_error_default
HTTPError: HTTP Error 403: Forbidden

I've got pretty much the same problem since a few day
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--verbose', u'-x', u'--audio-format', u'mp3', u'--audio-quality', u'0', u'-o', u'/tmp/D8cf5F66.%(ext)s', u'https://www.youtube.com/watch?v=HNuSD49chY8']
[debug] Encodings: locale ANSI_X3.4-1968, fs ANSI_X3.4-1968, out None, pref ANSI_X3.4-1968
[debug] youtube-dl version 2<IP_ADDRESS>
[debug] Python version 2.7.6 - Linux-3.13.0-45-generic-x86_64-with-Ubuntu-14.04-trusty
[debug] exe versions: avconv 9.18-6, avprobe 9.18-6
[debug] Proxy map: {} WARNING:
[youtube] HNuSD49chY8: Skipping DASH manifest: ExtractorError(u'Could not download DASH manifest: HTTP Error 403: Forbidden (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.',) ERROR: unable to download video data: HTTP Error 403: Forbidden Traceback (most recent call last):   File "/usr/local/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 1585, in process_info     success = dl(filename, info_dict)   File "/usr/local/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 1527, in dl     return fd.download(name, info)   File "/usr/local/lib/python2.7/dist-packages/youtube_dl/downloader/common.py", line 342, in download     return self.real_download(filename, info_dict)   File "/usr/local/lib/python2.7/dist-packages/youtube_dl/downloader/http.py", line 60, in real_download     data = self.ydl.urlopen(request)   File "/usr/local/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 1860, in urlopen     return self._opener.open(req, timeout=self._socket_timeout)   File "/usr/lib/python2.7/urllib2.py", line 410, in open     response = meth(req, response)   File "/usr/lib/python2.7/urllib2.py", line 523, in http_response     'http', request, response, code, msg, hdrs)   File "/usr/lib/python2.7/urllib2.py", line 448, in error     return self._call_chain(*args)   File "/usr/lib/python2.7/urllib2.py", line 382, in _call_chain     result = func(*args)   File "/usr/lib/python2.7/urllib2.py", line 531, in http_error_default     raise HTTPError(req.get_full_url(), code, msg, hdrs, fp) HTTPError: HTTP Error 403: Forbidden

A duplicate of #6451.


feat: migrate from source-map to @jridgewell/trace-mapping

Summary
Same motivation as https://github.com/istanbuljs/v8-to-istanbul/issues/185.
@jridgewell hi again! 😀 If this PR lands, there's only a single (non-transitive) usage of source-map in this repo, and that's https://github.com/facebook/jest/blob/5183c15a8e14089351d6403f4fdb72a6705aa379/e2e/coverage-handlebars/transform-handlebars.js. Would that use case be something you support as well?

Test plan
Green CI

Would that use case be something you support as well?

trace-mapping is specific to consumer side of the sourcemap API. I've been thinking about making a package for the generator side (there are a ton of optimizations that could be made over SourceMapGenerator), but haven't done that yet.
However, you don't really need a generator for this simple usecase. You can just take advantage of a SectionedSourceMap:
import { AllMap, encodedMappings } from '@jridgewell/trace-map';

const output = [
  'const Handlebars = require("handlebars/dist/cjs/handlebars.runtime.js");\n',
  'module.exports = Handlebars.template(',
  code,
  ');\n',
];

const map = new AllMap({
  version: 3,
  sections: [
    {
      offset: {
        // 0-index, so count of newlines before `code`
        line: 1,
        // column offset of the injected code on its first line
        column: output[1].length,
      },
      map: pc.map,
    },
  ],
});

// You need to manually JSONify the map from here, but I could add an export to do that for you.
const json = JSON.stringify({
  version: 3,
  file: map.file,
  names: map.names,
  sources: map.sources,
  sourcesContent: map.sourcesContent,
  // Depending on where this is going, you could use decodedMappings to avoid an encode->decode round trip.
  mappings: encodedMappings(map),
});

Thanks @jridgewell! keeping the current code in this case seems fine, it's just an integration test. Happy to switch over if/when you release a generator package, though! 🙂


Repeated and missed id of hyperpartisan splits

I am trying to compare my model with longformer, however, I found there are some repeated ids both in test and train dataset.
For example, 517, 24, 459, ... show in both test and train splits (hp_splits.json). Also, there are disappeared ids such as 9 and 13. According to #15, for hyperpartisan dataset you use 80/10/10 splits, it seems that there should be no repeated ids in different splits?

Yes, there seems to be some duplicates in the splits. We inherited these splits from another internal project and realized this issue later. Note that these splits are on the "training" set of the original dataset and we don't use the test set.
For reproducibility, we released the exact splits that we used; as long as you use the same data splits and preprocessing script as ours your results should be comparable.


Exception when listing channels after claiming one

The Issue
If a channel has just been claimed and the command channel_list_mine is called, the daemon throws an exception
Steps to reproduce

./lbrynet-cli channel_new channel_name=@YTTest amount=1
issue ./lbrynet-cli channel_list_mine multiple times

Expected behaviour
empty array until the channel is claimed [or queued (?)] and then an array of channels
Actual behaviour
Empty array returned for the first few seconds and then an exception is thrown (see below)
System Configuration
{
  "id": null,
  "jsonrpc": "2.0",
  "result": {
    "build": "rc",
    "ip": "<IP_ADDRESS>",
    "lbrynet_version": "0.10.3rc1",
    "lbryschema_version": "0.0.4",
    "lbryum_version": "2.7.21",
    "os_release": "4.4.0-75-generic",
    "os_system": "Linux",
    "platform": "Linux-4.4.0-75-generic-x86_64-with-Ubuntu-16.04-xenial",
    "processor": "x86_64",
    "python_version": "2.7.12"
  }
}

Anything Else
niko@home~/lbry$ ./lbrynet-cli channel_list_mine
ERROR: NOT NULL constraint failed: claim_cache.height

Help for channel_list_mine method:

    Get my channels

    Returns:
        (list) ClaimDict

Here's the traceback for the error you encountered:
Traceback (most recent call last):
  File "site-packages/twisted/internet/defer.py", line 498, in errback
  File "site-packages/twisted/internet/defer.py", line 565, in _startRunCallbacks
  File "site-packages/twisted/internet/defer.py", line 651, in _runCallbacks
  File "site-packages/twisted/internet/defer.py", line 1355, in gotResult
--- <exception caught here> ---
  File "site-packages/twisted/internet/defer.py", line 1297, in _inlineCallbacks
  File "site-packages/twisted/python/failure.py", line 389, in throwExceptionIntoGenerator
  File "site-packages/lbrynet/lbrynet_daemon/Daemon.py", line 1733, in jsonrpc_channel_list_mine
  File "site-packages/twisted/internet/defer.py", line 1297, in _inlineCallbacks
  File "site-packages/twisted/python/failure.py", line 389, in throwExceptionIntoGenerator
  File "site-packages/lbrynet/core/Wallet.py", line 901, in channel_list
  File "site-packages/twisted/internet/defer.py", line 1297, in _inlineCallbacks
  File "site-packages/twisted/python/failure.py", line 389, in throwExceptionIntoGenerator
  File "site-packages/lbrynet/core/Wallet.py", line 800, in _handle_claim_result
  File "site-packages/twisted/internet/defer.py", line 1297, in _inlineCallbacks
  File "site-packages/twisted/python/failure.py", line 389, in throwExceptionIntoGenerator
  File "site-packages/lbrynet/core/Wallet.py", line 339, in save_claim_to_cache
  File "site-packages/twisted/python/threadpool.py", line 246, in inContext
  File "site-packages/twisted/python/threadpool.py", line 262, in <lambda>
  File "site-packages/twisted/python/context.py", line 118, in callWithContext
  File "site-packages/twisted/python/context.py", line 81, in callWithContext
  File "site-packages/twisted/enterprise/adbapi.py", line 477, in _runInteraction
  File "site-packages/twisted/enterprise/adbapi.py", line 467, in _runInteraction
  File "site-packages/twisted/enterprise/adbapi.py", line 486, in _runOperation
sqlite3.IntegrityError: NOT NULL constraint failed: claim_cache.height

This issue has been fixed I believe, am I correct?

Yes, this has been fixed, it was lbryum server problem


Python Clone has no python code

The python clone repository doesn't have any python code in it. it's just a
copy of the main repo.

https://code.google.com/r/skolios-python/

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 31 Jul 2015 at 1:50

I have the same issue


Sprint 37 - Update Live States on SVG Map

Problem Statement
As ReportStream Product, we need to keep our map of live states up-to-date so that we communicate the state of our application to our current and prospective users and other stakeholders.
Acceptance Criteria

Update the SVG map for any new states that are now Live on these pages

https://reportstream.cdc.gov/
https://reportstream.cdc.gov/how-it-works/where-were-live

Update the list of Live states to show Delaware as "Live" on these pages

https://reportstream.cdc.gov/how-it-works/where-were-live

To Do

[ ] Update yml list
[ ] Update SVG map
[ ] Use this map in React app as well as 11ty

Updates on 3/3/2022: See Slack thread regarding Delaware approved to go live

Updates on 3/9/2022: See Slack thread regarding Delaware approved to go into Production (hopefully by end of week per @MauriceReeves-usds and @Adrian-Brewster ).

Delaware is now live! See Slack thread.


Make scope path syntax eligible for .value and .previous

This is a retake on #3606 and #3611
This simplifies the / implementation by removing the unnecessary thunks for scope and keys, because the keys already have scopes in them. In other words, when we get something like Test / compile / name, it will now treat as (Test / compile) / name where Test / compile now returns materialized TaskKey[Analysis]. This means no more hacks to get .value or .previous working.
Fixes #3605

Closing this in favor of https://github.com/sbt/sbt/pull/3620


Optimzed reinitialization test case

I have optimized the test case and it seems to work as expected. I have triggered a change to the input field value after the first initialization to check if the re-initialization is keeping any unexpected dirty or pristine flags.
I think it's a good idea to keep those render call checks to let the tests fail if you somehow create a render loop or your form re-renders when it shouldn't.

@ncphillips Thanks for working on this! If this test makes sense to you lets get it merged into your PR.
You mentioned that it does NOT solve all of your problems, what are those problems?

Here is the workflow I need:

I go to edit a Page, which creates a new form for that page.
I make some changes, which makes the "Pristine" flag turn false.
I refresh the site, my changes have persisted, and pristine is false
I press Submit to save my changes
The form re-initializes so now pristine = true

The issue is that step 3 and 5 are at odds with one another.

If enableReinitialize is true, then step 3 will not work.
If enableReinitialize is false then step 5 will not work.


Capybara makes unneeded driver methods invocations when applying filters

It would be good if the following methods won't apply filters to all elements to improve performance:

find with match: :first option (after first matching element was found)
first (after first matching element was found)
all, assert_selector with :minimum option (after needed minimum amount of elements was found)
assert_no_selector with maximum: n or count: n option (after n+1 method was found)

Basically this would involve moving the count options logic into SelectorQuery#resolve_for.  I don't believe example 3 is actually correct, since all needs to return all matches even if a minimum option is provided - so the only optimization that  could be done for all would be raising an error once the maximum or  count  is exceeded. Also example 1 and 2 need to handle the case where any  count options are passed in, to make sure the overall count is correct before returning the first matched element.

I've implemented something for this in PR #1734 if anyone wants to take a look/critique

closed via 676fba48e2968c7d4ab7f442a8771c25a6664cb1


Split Clamp into smaller traits and implement for [T]

This adds back a replacement for clamp_self, but I'm going for the *_assign naming convention, and also splits Clamp into separate IsWithinBounds, Clamp and ClampAssign. This allows separate implementations for some traits that wouldn't be compatible with the other traits. For example [T] can't implement Clamp in a nice way since it requires moving self.
Breaking Change
This moves the is_within_bounds method into IsWithinBounds.

bors r+


Version 3.0.0 gives security warning but system is patched

Hi,
On Exchange 2016 CU17 version 3.0.0 gives this errors:

But the system is patched to solve these errors (confirmed by the healthchecker script):

Can this be solved?

Hi @habo105 ,
Could you please run the script with -verbose switch and send us the log to extoolsfeedback[at]microsoft.com

Looks like this issue is resolved now when I pulled the latest release yesterday.

@entmgmt you've installed the latest CU, right?

Yes, all the latest CU's are installed and are identified by the script and no updates / CVE's are being listed in RED .

That is what we also found out. PR #408 solves the issue even if the CU is not the current one.


ahk_x11: init at 1.0.3-unstable-2024-03-30

Description of changes
Adds package AHK_X11, an AutoHotkey implementation for Linux.
Closes #279323. Closes #297972 (this PR contains the commit in that PR) (already merged). CC @AndersonTorres.
It reports command not found for file command if your PATH does not contain that command. This error does not affect normal usage. I could have fixed by adding file as a dependency, but this is actually a non-fatal upstream bug (phil294/AHK_X11#75), so I left it like this (Fixed in upstream).
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

The new push just now utilized the copyShardDeps flag introduced in #289266 (merged days ago).
#305484 may change the situation a bit, too, because this package is using gi-crystal. In https://github.com/NixOS/nixpkgs/pull/305484#issuecomment-2067747412, I was hoping we can have a better way of packaging Crystal packages that use gi-crystal.
CC @sund3RRR.


NuGetInstaller 0.2.0 Issue With VSTS Package Management Addon

Hi,
We are using the VSTS package management addon to host a number of packages, which are a key part of our build process.
Up until last night, everything worked just fine. We have a nuget.config checked into our source tree which provides the username and access token required by the build to access the package management source.
The config for the task in the build process specifies the path to this nuget config file within the source tree and this used to work perfectly.
As of last night (when the NuGetInstaller task was updated from 0.1.32 to 0.2.0), this has stopped working - we now get an error stating "Key not valid for use in specified state"
Nothing else has changed since it worked the previous day, so I assume this must somehow be related to the 0.2.0 update. Any help with this would be much appreciated as I'm not sure how to resolve this.
Thanks.

I believe they are aware of this and addressing now.

@rjosborne this doesn't look familiar, can you share more from your logs?

Hi,
Please see the two attached log files - "legacy log" is with the build agent installed exactly as it was the day before - the only difference from when it previously worked is the elements which self updated (including the NuGetInstaller task) overnight last night.
"New log" is after I installed a second build agent earlier this evening with the latest version of the agent in an attempt to rule out any issues with using the 0.2.0 task with what I assume is now the legacy build agent.
For clarity, we are using an in house installation of the build agent hosted on a Windows server 2008 R2 box as opposed to a cloud hosted agent.
legacy log.txt
new log.txt

@rjosborne I think I know what is going on here. I suspect that if you remove the username and password from your nuget.config it will work. I will follow up with why I suspect that will work later today, and we will try to improve the error messaging there.

@jmyersmsft many thanks for your help - I've just tested your suggestion and it now works again.

Looks to have been a bad/invalid username/pass.  It broke on the update because we stopped clobbering the user provided creds to VSTS Packaging with the official set of creds.  It is fixed live now because we went back to clobbering.  In the next release, it will still default to clobbering but users will be able to opt-in to the nuget 3.5.1 flow which will use our Cred Provider to provision credentials just in time.  In this flow any user-provided credentials will be honored by nuget even to the VSTS Packaging service (no clobbering).  It is best to not provide credential to the VSTS Packaging service urls in nuget.config and to let the build identity carry forward instead.


add incorrect sum when adding 52 bit numbers

The sum returned 0 for 10+0 when 52 bit numbers were added.
(N=52, es=4)

used positional notation for N, es, switched to .N(N), .es(es) and issue went away


Add display names to the departures

Like navaids they have display names. A good example is the LMT6 is called KINGSLEY 6

This is done in another PR.


Update MSAL Node Docs with the latest feedback

This PR updates node docs in github, repairs old links for library reference (which changed recently) and add FAQs from customer feedback.

Coverage decreased (-2.8%) to 79.948% when pulling 20f94c3970fb14c7508aa7b61ba80e1639c50605 on update-docs-links into c9da47ec30d297938f617ad7222a943671fbad30 on dev.


Explicitly shutdown connection pool does not reject getConnection, blocks until timeout (Hikari 2.4.9)

Environment
HikariCP version: 2.4.9
JDK version     :
  java version "1.7.0_55"
  Java(TM) SE Runtime Environment (build 1.7.0_55-b13)
  Java HotSpot(TM) 64-Bit Server VM (build 24.55-b03, mixed mode)
Database        : MS SQL
Driver version  : jTDS 1.3.1

We had a bug in our code where we were accidentally calling HikariPool.shutdown() - and then subsequent application threads were attempting to call upon this pool with getConnection
Even though this pool was in a closed, shutdown state, it did not "reject" the getConnection call ("IllegalStateException: pool is closed") or somesuch.  Instead, it blocks these calls until the timeout.  This caused long pauses / slowness in our application  due to blocking on a pool that was forever empty.
Before investigating submitting a patch with the desired behavior (fail-fast after pool is closed), I'm curious if the existing behavior is intentional?  Any other thoughts?
I noticed that private volatile boolean closed is used to prevent adding new connections to the pool, but getConnection chugs along none the wiser of the closed state. I would expect a check before each getConnection?
Obviously, we're fixing our code to NOT call HikariPool.shutdown() unless we really mean it :roll_eyes:

@noahlz The behavior is in fact the result of a conscious decision.  While somewhat undesirable on the margins, I still feel it is worth the benefit.
What benefit?  Performance.
The assumption is/was that if a pool is shutdown (an explicit action) and yet an application continues trying to use it, the application will fall over rather quickly, probably noticeably, and possibly spectacularly.   Which is OK, because getting a stream of timeout exceptions is not so different in terms of an application failure mode than a stream of IllegalStateExceptions.  Granted, the timeout exceptions come at a slower pace.
As a side note, IllegalStateException would be an unchecked exception and would not be the desired exception.  SQLNonTransientException would be correct, were it to be thrown.
So what specifically about performance?
Well, the alternative would be a volatile memory access in the "hot path", checking the closed state.   "Hot path" meaning that it would be accessed on every borrow from the pool.  And yet, it would be an access that for 99.999bar% of accesses is always false.  In fact, in a properly designed application the check would serve no purpose; it merely throws a different exception in the marginal case.  However, it would impact performance.
volatile variable access is not free.  It creates a LoadLoad/LoadStore barrier.  The LoadLoad/LoadStore barrier creates a prohibition against re-ordering optimizations by both the compiler and CPU, which in turn affects the performance of surrounding code.
At one point we had a check for closed in the hot path, but it caused a 5% hit when benchmarked.  And as noted above, it represents optimizing for the 0.000001% case over the 99.999999% case.

Thank you! Yes, our application was defective in closing the pool prematurely. This is the sort of answer I was looking for - closing.


make join as top level operator

What problem does this PR solve?
Issue Number: close #6351
Problem Summary:
What is changed and how it works?
make join as top level operator
Check List
Tests

[x] Unit test
[x] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

make join as top level operator.

/run-all-tests

/run-all-tests

It seems that both
https://github.com/pingcap/tiflash/blob/e77d01c1b410bd5c94d0b22abc02a8af84fe7dcf/dbms/src/Flash/Planner/plans/PhysicalJoin.h#L79
and
https://github.com/pingcap/tiflash/blob/25e5c1c66e2adefe7b044f5fe6afbc23931e0f3e/dbms/src/Flash/Coprocessor/JoinInterpreterHelper.cpp#L304
are useless.
Others LGTM.

remove.

/run-all-tests

/merge

/run-all-tests


Unable to load class 'org.jetbrains.kotlin.incremental.IncrementalJvmCompilerRunnerKt'

Plugin version
1.21.0
Gradle version
8.0
(Optional) Android Gradle Plugin (AGP) version
8.1.0
Describe the bug
After upgrading the plugin to version 1.21.0, my Kotlin compiler tasks all fail with this output:
Unable to load class 'org.jetbrains.kotlin.incremental.IncrementalJvmCompilerRunnerKt'.

This is an unexpected error. Please file a bug containing the idea.log file.

The dependency analysis tasks as well as the lint tasks are also failing with the same error.
To Reproduce
Steps to reproduce the behavior:

Upgrade the plugin to version 1.21.0.
Run the projectHealth task. The compileKotlin subtask will report the error.

Expected behavior
These tasks worked just fine with version 1.20.0. They shouldn't be failing.
Additional context
idea.log

I noticed the same issue on my personal project. Here is a run upgrading the build plugin in which DAGP was upgraded to 1.2.1 https://github.com/kyhule/polyworld-app/actions/runs/5933064716/job/16087936262#step:6:47.

Thanks for the report. Can you update your version of Gradle to latest, which is 8.3 at time of writing?

That build was with gradle 8.3 though it was not with kotlin 1.9.0. I think this might be caused by DAGP including the kotlin bom for 1.9.0. It seems to cause a mixed bag of kotlin versions on the build classpath.

After upgrading to kotlin 1.9.0, the issue seems to have gone away.
https://github.com/kyhule/polyworld-app/actions/runs/5957791886/job/16161054676


[feature] How to get group tree with casdoor-nodejs-sdk?

The group tree means this part: https://door.casdoor.com/trees/built-in

The group tree mainly relies on the data returned by the get-groups API. But the returned data is a list. The Casdoor frontend code will build the tree from the list. We may need to do so too. There are two ways:

Build the tree in Casdoor Go backend: https://github.com/casdoor/casdoor/blob/master/controllers/group.go#L31
Build the tree in the SDK (like this casdoor-java-sdk)

Fixed by: https://github.com/casdoor/casdoor-nodejs-sdk/commit/02c17eb894fb64c5e679266896b998adf0f84301

@radiorz use the new API: getGroups(withTree?: boolean)


[Editor] Fix syntax highliting in .NET Core, by adding missing dependency

PR Details

Description
Adds missing dependency on System.Reactive - this is an indirect dependency, introduced by RoslynPad. On .NET Framework it was pulled from the GAC, but in .NET Core it was missing. This resulted in a silent exception being swallowed because it occurred on a background thread. The exception message complained about version 4.3.0 (but there's only 4.3.1) so I also upgraded the System.Reactive.Linq
Related Issue
Fixes #786
Types of changes

[ ] Docs change / refactoring / dependency upgrade
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Looks simple enough, I'll merge this in

I'm running beta04-1317, which includes this fix, but syntax highlighting is still not being display in the editor for .NET Core projects.

Yes, while the change fixes the issue when running GameStudio from its build folder, when running from the launcher there is some issue with Nuget resolution that is not loading the dependencies into memory correctly. When I have some free time I'm going to be looking into fixing that.

@manio143 that might explain more than just this issue, so fixing that problem might have a much larger effect.  It might also get fixed by the work @xen2 is doing on the build system and upgrading to .NET 5, so as a bug this one acts as quite an effective canary.

I got into an investigative mood and found the main reason it's not working.
During nuget resolution System.Reactive (v4.3.1) got resolved for me under netcoreapp3.0, which is the first target framework for this package. However, when I looked into the lib folder of it there was no DLL file. This has been reported here as an issue with .NET Core and WPF/non-gui.
I was able to apply a temporary fix by going into %temp%\StrideNugetResolver* and modifying the project.assets.json so that it points to the target netstandard2.0 - this resulted in colorful code editor in GameStudio.
Checked that updating System.Reactive to 5.0.0 gets resolved in such a way that the DLL is present.


helm 3.0.2 SDK - example of install from remote repository

Hi,
I want to use the SDK (go client) to install charts from remote repository.
Does anyone have example for this?
to be clear:
I want to do this steps, in code:

helm repo add stable https://kubernetes-charts.storage.googleapis.com/
helm repo update
helm testMysql install stable/mysql

will appreciate any help!

Have you had a look at the go doc reference? Have a look at the action package for some ideas. cmd/helm is a good place to look in the code base as well, as that's where the CLI implementation resides.
https://pkg.go.dev/helm.sh/helm/v3
Hope this helps

i had a look at the docs.
i am looking for more complete code example if someone can help

@Tzahif A Helm3 HTTP Wrapper With Helm Go SDK, this is a complete code example that you may want, wrapper with Gin.

there's also the simple helm list example now available in the documentation for others getting started with the SDK.
Closing as answered. Hope this helps!

helm install mynginx https://example.com/charts/nginx-1.2.3.tgz

@Tzahif do you got go-client code to finish these two steps from remote:
helm repo add stable https://kubernetes-charts.storage.googleapis.com/
helm repo update

thanks

@zsfarkas when I pack and run this code and run it as a container inside a kubernetes. Helm deployment occurs, I can see the helm release in the default namespace but its resources are getting deployed in different ns.
https://github.com/helm/helm/issues/12931. I tried my code as well as yours both works but resources are getting deployed ns i.e ns where I run this helm sdk code


Does Azure Data Lake Storage Gen1 will be retired since the ADLS gen 2 is ready now?

Does Azure Data Lake Storage Gen1 will be retired since the ADLS gen 2 is ready now?

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

ID: 30296110-2a2c-073d-068c-0c1485b6097a
Version Independent ID: feac830e-ebec-0254-60cf-e0225c85d916
Content: Overview of Azure Data Lake Storage Gen1
Content Source: articles/data-lake-store/data-lake-store-overview.md
Service: data-lake-store
GitHub Login: @nitinme
Microsoft Alias: nitinme

@imeya As of now there are no plans to retire ADLS gen 1. However it's recommended to migrate to Gen2 as most of the latest features and improvements will be rolled out to Gen2.
Hope this helps.

@imeya I will now proceed to close this issue. Feel free to still comment if you have any further queries.


Generated file names for bundles can exceed file system limit

I tried updating a large project to test the latest Steal and discovered the build won't complete because the filename generated for a widely shared bundle is too long for the file system.
This is the same issue that I proposed a pull-request on in legacy StealJS in https://github.com/bitovi/steal/pull/170 - NOTE: GitHub seems to have mangled that page with history from a separate issue. The relevant comments are Justin's and my response.
In that PR I suggest taking truncating long file names and appending a hash, which I think is still reasonable.

Closed by #47


Android tab bar icon won't overflow

Hi, I'm trying to replicate the Facebook Messenger tabs where the Camera button overflows the tab bar. Currently testing this with the react navigation sample code NativePlayground.
Under SimpleTabs.js i also added showIcon for tabBarOptions and tabBarPosition to be "bottom".
I modified this code and added margin bottom.
MyChatScreen.navigationOptions = {
  tabBar: {
    label: 'Chat',
    icon: ({ tintColor, focused }) => (
      <Ionicons
        name={focused ? 'ios-chatboxes' : 'ios-chatboxes-outline'}
        size={26}
        style={{ color: tintColor, marginBottom: 20 }}
      />
    ),
  },
};

Results
iOS

Android

Also tried this https://stackoverflow.com/questions/41912313/element-overflow-hidden-in-react-native-android but no luck
Seems to be fine on iOS but for android there's a top view blocking the icon.
Is there any way to achieve this? TIA

@rad182 Can you post a minimum reproducible example of this on https://snack.expo.io? I'd love to look into it for you.

React Native on Android doesn't support overflow: visible

@satya164  could you give me a link that how to design a overflow tabbar item? I have search much websites and issues, but I still can't find one effective.Thanks

+1
If you find a solution, I hope you can let me know. Thank you. @satya164

@landmass If you find a solution, I hope you can let me know. Thank you.

@est7
react-native-tab-view\src\TabViewAnimated.js
render() {   ....   return(    <View style={[styles.container, this.props.style]}>      ......      <TouchableOpacity         style={{ backgroundColor:'orange',position:'absolute',bottom:25, }}         onPress={ ()=>{            //do something           }          }      >         <Text style={{width:36,height:36,fontSize:22,color:'#999',textAlign:'center',textAlignVertical:'center',}}>+</Text>      </TouchableOpacity>    </View>  ) }

@landmass thanks!

@landmass
modify the source code is not good for VCS ,is it?

Why are this issue and #341 closed? Seems to me like a legitimate bug and there has not been any resolution to it.

@rad182 in android to change the overflow of the element you have to change the elevation, while in iOS you have to change the zIndex