python: enable optimizations

Motivation for this change
I didn't perform any tests yet, but I assume it's pretty safe and hopefully makes some stuff a bit faster.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS

[x] python27
[ ] python34
[ ] python35
[x] python36
[ ] python37

[x] macOS

[ ] python27
[ ] python34
[ ] python35
[x] python36
[ ] python37

[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

Failure on x86_64-darwin (full log)
Attempted: python
Partial log (click to expand)
cannot build derivation '/nix/store/q52d46p2pn1n7j9sdlj026p2sw8qh1wf-python-2.7.15.drv': 14 dependencies couldn't be built
cannot build derivation '/nix/store/51l8zz3492nhd81crmywfrxw8dxiy31h-libtool-2.4.6.drv': 5 dependencies couldn't be built
cannot build derivation '/nix/store/fzzcy71yps97c1zbgmn9ghd8zy0aqvwj-hook.drv': 6 dependencies couldn't be built
cannot build derivation '/nix/store/49g5wc9ici5hiw5dkx7a5rsbdzy5il90-flex-2.6.4.drv': 7 dependencies couldn't be built
cannot build derivation '/nix/store/bh7kcpgdbqh0qnd59wkg4ab9nw1whvkp-bootstrap_cmds-dev-tools-7.0.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/3zkaa0hlii6rl4y7kms68m8zrglrfmr4-xnu-osx-10.11.6.drv': 9 dependencies couldn't be built
cannot build derivation '/nix/store/dq2lbvnjwdvl5nzc6c6qp59wk1rc6bfr-IOKit-osx-10.11.6.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/jymfkn4gv2bx1kfysy33v29wqir1iahv-configd-osx-10.8.5.drv': 9 dependencies couldn't be built
cannot build derivation '/nix/store/p9jaqmm84rw418vywa6j1zdkbw7wf1vl-python-2.7.15.drv': 15 dependencies couldn't be built
error: build of '/nix/store/p9jaqmm84rw418vywa6j1zdkbw7wf1vl-python-2.7.15.drv' failed

This should go to staging.

Failure on x86_64-linux (full log)
Attempted: python
Partial log (click to expand)
gcc -pthread -c -fno-strict-aliasing -g -O2 -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes  -fprofile-use -fprofile-correction  -I. -IInclude -I./Include  -fPIC -DPy_BUILD_CORE -o Python/import.o Python/import.c
gcc -pthread -c -fno-strict-aliasing -g -O2 -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes  -fprofile-use -fprofile-correction  -I. -IInclude -I./Include  -fPIC -DPy_BUILD_CORE -I. -o Python/importdl.o ./Python/importdl.c
gcc -pthread -c -fno-strict-aliasing -g -O2 -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes  -fprofile-use -fprofile-correction  -I. -IInclude -I./Include  -fPIC -DPy_BUILD_CORE -o Python/marshal.o Python/marshal.c
gcc -pthread -c -fno-strict-aliasing -g -O2 -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes  -fprofile-use -fprofile-correction  -I. -IInclude -I./Include  -fPIC -DPy_BUILD_CORE -o Python/modsupport.o Python/modsupport.c
gcc -pthread -c -fno-strict-aliasing -g -O2 -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes  -fprofile-use -fprofile-correction  -I. -IInclude -I./Include  -fPIC -DPy_BUILD_CORE -o Python/mystrtoul.o Python/mystrtoul.c
gcc -pthread -c -fno-strict-aliasing -g -O2 -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes  -fprofile-use -fprofile-correction  -I. -IInclude -I./Include  -fPIC -DPy_BUILD_CORE -o Python/mysnprintf.o Python/mysnprintf.c
gcc -pthread -c -fno-strict-aliasing -g -O2 -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes  -fprofile-use -fprofile-correction  -I. -IInclude -I./Include  -fPIC -DPy_BUILD_CORE -o Python/peephole.o Python/peephole.c
gcc -pthread -c -fno-strict-aliasing -g -O2 -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes  -fprofile-use -fprofile-correction  -I. -IInclude -I./Include  -fPIC -DPy_BUILD_CORE -o Python/pyarena.o Python/pyarena.c
building of ‘/nix/store/p1s33b4jvc727yyaxifryzm6nqrvccs7-python-2.7.15.drv’ timed out after 1800 seconds
error: build of ‘/nix/store/p1s33b4jvc727yyaxifryzm6nqrvccs7-python-2.7.15.drv’ failed

Failure on aarch64-linux (full log)
Attempted: python
Partial log (click to expand)
0:32:32 load avg: 81.04 [188/399] test_isinstance -- test_ioctl skipped
0:32:34 load avg: 82.56 [189/399] test_iter
0:32:38 load avg: 82.56 [190/399] test_iterlen
0:32:39 load avg: 89.08 [191/399] test_itertools
0:36:18 load avg: 95.85 [192/399] test_json -- test_itertools passed in 3 min 39 sec
0:37:50 load avg: 95.16 [193/399] test_kqueue -- test_json passed in 1 min 31 sec
0:37:51 load avg: 95.16 [194/399] test_largefile -- test_kqueue skipped
0:37:52 load avg: 95.16 [195/399] test_lib2to3
building of '/nix/store/sjyjccjvxcqr1kf6ij509j4jx9i28294-python-2.7.15.drv' timed out after 3600 seconds
error: build of '/nix/store/sjyjccjvxcqr1kf6ij509j4jx9i28294-python-2.7.15.drv' failed

We would have to test against the package set to see if it does not have any unwanted side-effects.

How much more time does it take to build the different Python versions?

Failure on x86_64-darwin (full log)
Attempted: python
Partial log (click to expand)
cannot build derivation '/nix/store/s2amfy3krl19l5xs2jlbvcmkzk371y37-python-2.7.15.drv': 14 dependencies couldn't be built
cannot build derivation '/nix/store/0706xvwvs6fi4l4r19vg70b3in3z56fw-libtool-2.4.6.drv': 5 dependencies couldn't be built
cannot build derivation '/nix/store/ab0gb1d08pkppvlylvmpgc6nv1wsawbh-hook.drv': 6 dependencies couldn't be built
cannot build derivation '/nix/store/pnh5qavvgzla65kj3yspg9an315sjm49-flex-2.6.4.drv': 7 dependencies couldn't be built
cannot build derivation '/nix/store/y9bl1l8s94s3jsa0ncz8d53gf6dkq3hr-bootstrap_cmds-dev-tools-7.0.drv': 4 dependencies couldn't be built
cannot build derivation '/nix/store/64n4zsr0yrzlx64qg0gkajsji55n9j1b-xnu-osx-10.11.6.drv': 9 dependencies couldn't be built
cannot build derivation '/nix/store/jidrjk5j8k0m1y4114arvkbwvf5x56gp-IOKit-osx-10.11.6.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/xypq024ba02npfdzxbhg4m6fj4bbnxm9-configd-osx-10.8.5.drv': 9 dependencies couldn't be built
cannot build derivation '/nix/store/l54w00xqk3m0hvb2blkwi8hzyzxbxi08-python-2.7.15.drv': 15 dependencies couldn't be built
error: build of '/nix/store/l54w00xqk3m0hvb2blkwi8hzyzxbxi08-python-2.7.15.drv' failed

Failure on x86_64-linux (full log)
Attempted: python
Partial log (click to expand)
gcc -pthread -fPIC -fno-strict-aliasing -g -O2 -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -DHAVE_EXPAT_CONFIG_H=1 -DXML_POOR_ENTROPY=1 -DUSE_PYEXPAT_CAPI -I/tmp/nix-build-python-2.7.15.drv-0/Python-2.7.15/./Modules/expat -I. -IInclude -I./Include -I/tmp/nix-build-python-2.7.15.drv-0/Python-2.7.15/Include -I/tmp/nix-build-python-2.7.15.drv-0/Python-2.7.15 -c /tmp/nix-build-python-2.7.15.drv-0/Python-2.7.15/Modules/_elementtree.c -o build/temp.linux-x86_64-2.7/tmp/nix-build-python-2.7.15.drv-0/Python-2.7.15/Modules/_elementtree.o
gcc -pthread -shared -lgcc_s -lgcc_s -fno-strict-aliasing -g -O2 -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -fprofile-use -fprofile-correction build/temp.linux-x86_64-2.7/tmp/nix-build-python-2.7.15.drv-0/Python-2.7.15/Modules/_elementtree.o -L. -lpython2.7 -o build/lib.linux-x86_64-2.7/_elementtree.so
building '_multibytecodec' extension
gcc -pthread -fPIC -fno-strict-aliasing -g -O2 -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -I. -IInclude -I./Include -I/tmp/nix-build-python-2.7.15.drv-0/Python-2.7.15/Include -I/tmp/nix-build-python-2.7.15.drv-0/Python-2.7.15 -c /tmp/nix-build-python-2.7.15.drv-0/Python-2.7.15/Modules/cjkcodecs/multibytecodec.c -o build/temp.linux-x86_64-2.7/tmp/nix-build-python-2.7.15.drv-0/Python-2.7.15/Modules/cjkcodecs/multibytecodec.o
gcc -pthread -shared -lgcc_s -lgcc_s -fno-strict-aliasing -g -O2 -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -fprofile-use -fprofile-correction build/temp.linux-x86_64-2.7/tmp/nix-build-python-2.7.15.drv-0/Python-2.7.15/Modules/cjkcodecs/multibytecodec.o -L. -lpython2.7 -o build/lib.linux-x86_64-2.7/_multibytecodec.so
building '_codecs_kr' extension
gcc -pthread -fPIC -fno-strict-aliasing -g -O2 -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -I. -IInclude -I./Include -I/tmp/nix-build-python-2.7.15.drv-0/Python-2.7.15/Include -I/tmp/nix-build-python-2.7.15.drv-0/Python-2.7.15 -c /tmp/nix-build-python-2.7.15.drv-0/Python-2.7.15/Modules/cjkcodecs/_codecs_kr.c -o build/temp.linux-x86_64-2.7/tmp/nix-build-python-2.7.15.drv-0/Python-2.7.15/Modules/cjkcodecs/_codecs_kr.o
gcc -pthread -shared -lgcc_s -lgcc_s -fno-strict-aliasing -g -O2 -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -fprofile-use -fprofile-correction build/temp.linux-x86_64-2.7/tmp/nix-build-python-2.7.15.drv-0/Python-2.7.15/Modules/cjkcodecs/_codecs_kr.o -L. -lpython2.7 -o build/lib.linux-x86_64-2.7/_codecs_kr.so
building of ‘/nix/store/mb1drg9p8s5pj6x7wnn1mgg53p2k0gx7-python-2.7.15.drv’ timed out after 1800 seconds
error: build of ‘/nix/store/mb1drg9p8s5pj6x7wnn1mgg53p2k0gx7-python-2.7.15.drv’ failed

Failure on aarch64-linux (full log)
Attempted: python
Partial log (click to expand)
0:51:15 load avg: 4.05 [229/399] test_nntplib
0:51:16 load avg: 4.05 [230/399] test_normalization
0:51:17 load avg: 4.05 [231/399] test_ntpath
0:51:19 load avg: 4.05 [232/399] test_old_mailbox
0:51:20 load avg: 3.96 [233/399] test_openpty
0:51:21 load avg: 3.96 [234/399] test_operator
0:51:22 load avg: 3.96 [235/399] test_optparse
0:51:25 load avg: 3.89 [236/399] test_ordered_dict
building of '/nix/store/hlg12sic6d2wb05zg4bnwr3w8j1mbxb7-python-2.7.15.drv' timed out after 3600 seconds
error: build of '/nix/store/hlg12sic6d2wb05zg4bnwr3w8j1mbxb7-python-2.7.15.drv' failed

@FRidh I tested this with the dependency tree of a project as a sanity check and didn't notice any problems.
The build time does indeed increase significantly because of the tests, but it's not like it was long already.   (I can't really afford to review it myself.)

I reread the description of the flag and I'm not actually sure anymore if it's useful. I mainly tried this out because the build log recommends it for a release build.
If you want a release build with all optimizations active (LTO, PGO, etc),
please run ./configure --enable-optimizations

@LnL7 https://bugs.python.org/msg281332

https://bugs.python.org/msg281332

It seems we're good on that since --with-lto is not passed and --enable-optimizations doesn't imply the former flag since 2016-09.
Will merge in a few days if there's no disagreement.

At least when building the interpreter we also build optimized bytecode. We could do that as well in a hook for all Python packages.
I don't think we should bother with these optimizations. If you indeed have something critical running, then likely you know about the option and enable it.

If you indeed have something critical running, then likely you know about the option and enable it.

This is a rather difficult proposal since it means rebuilding a lot of packages that depend on Python.
What about enabling --enable-optimizations and --with-lto only on certain platforms, like x86_64-linux, where toolchains are better tested, and CPU power to build Python is relatively plentiful and low-cost?
FWIW I've just ran nix-build -A python27 -A python35 -A python36 -A python37 -A python38 -A python39 on x86_64-linux with nixpkgs master and those flags; the build finished without issue and the resulting python binaries ran fine.


Add core-js for old browser compatibility

core-js automatically transpiles into ie 11 compatible code

@Shandorin please test and merge if it works for you.

Monthview styling breaks when used on ie 11. Dayview doesn't load properly and always goes to current date

Old browser warning works now so only weekview and monthview working is acceptable. Merging.

Breaks everything, out of scope for now.


Upgrade to Gradle 3.0

Summary
This is something I worked on during OOM issue.
Upgrading Gradle helps significantly in build times for the native Android app.
Release build still takes a while because of the way it bundles the JS code.
The gain is in debug/incremental builds.
Comparison:
Before: 2.29 mins

After 24 sec

another thing is that we may need to take care of these warnings as well for every dependency, but I'm 0/5 on that
> Configure project :react-native-svg
Configuration 'compile' in project ':react-native-svg' is deprecated. Use 'implementation' instead.

@enahum Are your comments something that should be addressed with this PR, or are they something to be done in the future?

@hmhealey it would be incredibly cool if those changes were implemented in this PR.
@miguelespinoza thoughts?

another thing is that we may need to take care of these warnings as well for every dependency, but I'm 0/5 on that as long as it works

yeh that would require forking each project.  I say it's fine leaving it the way it is.  Unless it ends up causing dev environments to break. Which will not occur

Wondering if you should update from Android SDK Build tools 25.0.2 to 26.0.2 per this warning

I don't see why not

I'll test it to see if it'll cause any issues

@enahum I made the changes, it removes the warning.
We'll have to deal with the compile deprecated error for other libraries.  I don't see the need to fork them.


[KnowHow] Enterprise Architect

Describe the knowHow
UML

Hello @unknowIfGuestInDream. This issue is marked as deprecated and closed.


Fix flaky test in Consul

Motivation:
ConsulEndpointGroupTest > initializationError FAILED
    org.awaitility.core.ConditionTimeoutException: Assertion condition defined as a lambda expression in
    com.linecorp.armeria.internal.consul.ConsulTestBase that uses com.pszymczyk.consul.ConsulStarterBuilder null within 10 seconds.

Modifications:

Increase the timeout in await() to reduce flakiness

Result:

Clean up flaky test in Consul

Do not review this PR yet.
I opened this PR to run Consul tests in the CI evironment.

The flaky test on Consul seems to be reduced. PTAL. 🙇♂


Unit conversion bug

Microsoft PowerToys version
0.78.0
Installation method
Microsoft Store
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce
PowerToys Run 에서 단위변한 시 잘못된 변관 값 표시
예)
%%1024Mb in Gb

✔ Expected Behavior
1.000Gb
❌ Actual Behavior
1.024Gb
Other Software
No response

Displaying incorrect conversion values in PowerToys Run when changing units

no, that not a bug, but a corrent answer.
you must have misunderstanding Gib/Mib/etc. and Gb/Mb/etc.

Short story is: 10^2 = 1000 and 2^10 = 1024. A long, long time ago these were mixed up, meaning a kilobyte is/was 1024 bytes, while "kilo" should mean 1000. Nowadays, we invented the term kibibytes. Not even talking about the combination of the two types mentioned on storage devices...

Not sure if it was ever a mix up. Computer use base 2 not base 10. It is standard practice to refer to a kilobyte to be 1024 and not 1000.

@TheJoeFin

Just because it's done, doesn't mean it's right! Microsoft is downright wrong.
How do you explain Mbps (Megabits, 1000000 bits)

The standards bodies have deprecated this usage of the megabyte in favor of a new set of binary prefixes, in which this quantity is designated by the unit mebibyte (MiB).

https://wikipedia.org/wiki/Megabyte

I'm not arguing right vs wrong. It is semantics "mix up" vs "made different choices"
I don't haha

@Jay-o-Way  The initial screenshot show 1024 MB being reported as 1.024 GB. This would be correct to you right?

Just a short rant against Microsoft, haha. Conclusion for this issue is valid: non-issue.

It's been over 20 years since the standards were established... I didn't know because I've only been using Windows OS. Thanks for letting me know


feat(balancer) async change upstreams

This change adds the option to change upstreams asynchronously, i.e. upstreams will not be updated on every call to Admin API, but rather when a timer expires. This improves the performance when Kong is configured with a large number of upstreams and/or when upstreams are frequently changed.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

This has had a few rounds of review from the team already, from my run through it the only thing that jumped to my eyes was the default value for the update parameter, but apart from that LGTM


Add a test for {{component}} helper

This replaces the {{view}} helper test coverage that I removed in #60 with a test of the {{component}} helper instead.
This makes the test suite incompatible with Ember 1.10, which seems reasonable given that we've now released 1.13, so the list of supported releases remains the same length.

:+1:


Inconsistent behavior between dev and prod server mode

Nuxt version:
2.15.7
Node version:
v16.13.0
No problem using yarn dev.
Using yarn build && yarn start gives unexpected results for imported components using speedkitHydrate.
They fetch method in those components are trigerred in loop on some of them, and don't know why.
In a first time I tought it was on property change or something like that (although usually changing a property does not retrigger fetch), but even after checking properties are not changed using a "watch" on them, I still observe the unexpected fetch loop (querying in my case an API).
Before digging more, what is supposed to trigger fetch loop in components?

Hello @frossigneux,
the fetch loop is very strange.
Basically there is no difference between dev and server mode in the initialization of the components server-side.
https://github.com/GrabarzUndPartner/nuxt-speedkit/blob/77155cda44d41035f8ab297d02e3ac1ec4906882/lib/hydrate.js#L9-L23
In both cases, the component is passed directly in server mode. Client side the component is initialized with vue-lazy-hydration.
Does it happen in server the loop or in client?
Can you reproduce this in an empty project with a very simple example?

Ok, here is a minimal example to reproduce inconsistent behavior.
Component Child.vue:
<template>
  <v-card flat>Child: {{ loading }}</v-card>
</template>

<script>
export default {
  props: {
    loading: {
      type: Boolean,
      required: true,
    },
  },
  fetch() {
    this.$emit('update:loading', true)
    console.log('FETCH CHILD')
    this.$emit('update:loading', false)
  },
}
</script>

Page page.vue:
<template>
  <v-container>
    Parent: {{ loading }}
    <Child
      :loading.sync="loading"
    />
  </v-container>
</template>

<script>
import speedkitHydrate from 'nuxt-speedkit/hydrate'

export default {
  components: {
    Child: speedkitHydrate(() => import('@/components/Child')),
  },
  data() {
    return {
      loading: true,
    }
  },
}
</script>

Browser rendering output using yarn dev (no fetch loop):
Parent: false
Child: false

Browser rendering output using yarn build && yarn start (fetch loop):
Parent: true
Browser rendering output using regular import instead of speedkitHydrate and dev or prod server mode (no fetch loop):
Parent: false
Child: false

Then, the behavior is different using speedkitHydrate, fetch is triggered as soon as a property is changed.

The property change is even not required to triggers the bug:
Component Child.vue:
<template>
  <v-card flat>Child}</v-card>
</template>

<script>
export default {
  fetch() {
    this.$emit('loading', true)
    console.log('FETCH CHILD')
    this.$emit('loading', false)
  },
}
</script>

Page page.vue:
<template>
  <v-container>
    Parent: {{ loading }}
    <child
      @loading="
        (value) => {
          loading = value
        }
      "
    />
  </v-container>
</template>

<script>
import speedkitHydrate from 'nuxt-speedkit/hydrate'

export default {
  components: {
    Child: speedkitHydrate(() => import('@/components/Child')),
  },
  data() {
    return {
      loading: true,
    }
  },
}
</script>

Just emitting an event, changing something in the parent page, triggers the infinite fetch loop...

Ok, here is a minimal example to reproduce inconsistent behavior.
Component Child.vue:
<template>
  <v-card flat>Child</v-card>
</template>

<script>
export default {
  fetch() {
    this.$emit('loading', true)
    console.log('FETCH CHILD')
    this.$emit('loading', false)
  },
}
</script>

Page page.vue:
<template>
  <v-container>
    Parent: {{ loading }}
    <child
      @loading="
        (value) => {
          loading = value
        }
      "
    />
  </v-container>
</template>

<script>
import speedkitHydrate from 'nuxt-speedkit/hydrate'

export default {
  components: {
    Child: speedkitHydrate(() => import('@/components/Child')),
  },
  data() {
    return {
      loading: false,
    }
  },
}
</script>

Browser rendering output using yarn dev: no fetch loop.
Browser rendering output using yarn build && yarn start: fetch loop!
Browser rendering output using regular import instead of speedkitHydrate and dev or prod server mode: no fetch loop.
So there is a problem with speedkitHydrate and prod mode. I always tested by navigating to the test page indirectly.
However, if we replace in parent:
Parent: {{ loading }}
by
Parent
there is no more infinite loop.
Other thing: if I emit only one event, there is no more infinite fetch loop neither!
The bug is related to the rendering of parents in the case of multiple events.
@ThornWalli Thanks for your help!

@ThornWalli Do you have the same bug with my minimal example? Thanks

@frossigneux Could understand it and probably also solve.
But now I have to think about the change again. In principle, nothing speaks against it.

@frossigneux Have now a beta release, so you can test it.
https://www.npmjs.com/package/nuxt-speedkit/v/2.0.21-beta.1

Is included in the latest version.


Update SwaggerGenerator.php

@choiks14 can you give some description to this PR about the change?

Mysql field is int,true,ture.
So i will get first type

Can you give some more practical example with some samples please?


The RETURNING feature cannot be turned off when executing PG INSERT statements

Description
I want to turned off The RETURNING syntax when executing PG INSERT statements.
But I panic when I execute the following code.
pub async fn insert(request: request::ActiveModel) -> Result<String> {
        let db = get_connection()?;
        let query = Query::insert()
            .into_table(Request::Table)
            .columns([Request::Id, Request::ReqId])
            .values_panic(vec!["1".into(), "12A".into()])
            .to_owned();

        // Some(ValueTuple::One(sea_orm::Value::String(Some(Box::new("1".to_string())))))
        let query = Inserter::new(None, query);

        let result: InsertResult<request::ActiveModel> = query.exec(db).await?;
        Ok(result.last_insert_id)
    }

called `Option::unwrap()` on a `None` value
thread 'tests::test_insert' panicked at 'called `Option::unwrap()` on a `None` value', /Users/yangjiaxin/.cargo/registry/src/rsproxy.cn-8f6827c7555bfaf8/sea-orm-0.9.2/src/executor/insert.rs:118:54

Hey @step-baby, I'm not sure why you want to switch off returning but you can execute an insert with it like below.
let pear = fruit::ActiveModel {
    name: Set("Pear".to_owned()),
    ..Default::default()
};

let res: InsertResult = fruit::Entity::insert(pear) // Insert the `ActiveModel` directly without RETURNING expression
    .exec(db)
    .await?;
assert_eq!(res.last_insert_id, 28)

Hey @billy1624 I , I tried the example you gave but still used the RETURNING syntax.

Okay. I think we should have change this part to execute insert without returning

https://github.com/SeaQL/sea-orm/blob/d532deb6363d997beae029a8983d64c7e9574af6/src/executor/insert.rs#L102-L135

But i wonder why without RETURNING is important for your use case?

This is because we want to use the SEA-ORM framework to connect PG and other databases, so we want to be able to use standard SQL without any database syntax.
Thank you for supplying us with INSERT statements not using the returning syntax ！！

But SeaORM itself is agnostic to the database. Why not insert a record in SeaORM way?
let pear = fruit::ActiveModel {
    name: Set("Pear".to_owned()),
    ..Default::default() // all other attributes are `NotSet`
};

let pear: fruit::Model = pear.insert(db).await?;

https://www.sea-ql.org/SeaORM/docs/basic-crud/insert/#insert-one
I saw you construct the insert query manually above.

Inside SeaORM RETURNING is only used if the target database support such syntax (i.e. only PostgreSQL). For MySQL and SQLite we will insert without RETURNING.

I tried to insert records in SeaORM way ,but it used the RETURNING feature. I use SQL that I want to generate with SEA-ORM to connect to DAMeng database. This is done through a database agent that currently supports the standard SQL connection DAMENG.

Oh wait. DAMeng database is a proprietary database hence we don't provide support for it. Is it compatible with PostgreSQL, MySQL or SQLite?

I see that SEA-ORM compatible with the RETURNING feature. But I think this is an optional feature of the Postgres database, so it should be possible not to use it.

I believe this have been solved https://docs.rs/sea-orm/latest/sea_orm/struct.Inserter.html#method.exec_without_returning


DETACH MODULE ISN'T WORKING ON ANDROID 14

Screenshot of Play Store page of the app you are trying to detach:
Screenshot of the helper CLI showing detached apps (from Option 2):
Logs taken with su -c logcat -s "zygisk-detach" > /sdcard/log.txt:
Not providing this information will get your issue closed


create an appropriate bibstyle for the website with the r function tools:bibstyle

Ideally a style that shows a [code/data] link coming from "customb" field

#5 is probably better


Specify version on install

Sometimes the specific Chrome versions can be difficult to find. Any way to specify what chromedriver to use as a parameter?

You can control the chromedriver binary version that is installed by specifying the version of this package, for example by running
pip install chromedriver-binary==73.0.3683.20.0

you will get version 73.0.3683.20 (the last .0 is the version of this python package for the particular chromedriver binary release).
The list of releases can be found here and should contain all chromedriver versions.

Is it okay for you if I close this issue, or do you need a different way to specify the installed version?


WIP Split kubeflow-core into individual prototypes

Update to kubeflow/kubeflow :  https://github.com/kubeflow/kubeflow/pull/1207

This change is

We should keep the website in sync with the current release.
Presumably kubeflow/kubeflow#1207 won't go out until 0.3.0 or possibly 0.2.3.
So we shouldn't commit this until then.
Alternatively there is an open issue #88 to version our website.
Ideally we would cut branches for every release and then kubeflow.org would point to the branch corresponding to the current release. This way we could continue to update docs at head.

@ankushagarwal The docs are now versioned so this can be submitted.

@jlewi : Ready for review

/lgtm
/approve

Self LGTM-ed because @k8s-ci-robot removed LGTM label after a rebase.


notification comes from left to right

Hi . when I clear exisiting notification (error) and generate the new success notification its comes from left rapidly and then show on the top right side
how can I prevent this flickering
Notification.clearAll();              Notification.success({message:'good', delay: 3000,closeOnClick:false});

I have this problem too. :S


Plugin doesn't work on Windows - wrong path handling

Running the plugin on Windows (yes...) ends with following error (stacktrace enabled):
[INFO] --- docker-maven-plugin:0.2.5:build (build) @ my-service ---
[INFO] Copying c:\devel\myorg\cc\my-service\target\my-service-war-exec.jar -> c:\devel\myorg\cc\my-service\target\docker\my-service-war-exec.jar
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 15.007s
[INFO] Finished at: Thu Apr 09 22:22:05 CEST 2015
[INFO] Final Memory: 35M/346M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal com.spotify:docker-maven-plugin:0.2.5:build (build) on project my-service: Exception caught: UNC path is missing sharename: /\my-service-war-exec.jar -> [Help 1]
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal com.spotify:docker-maven-plugin:0.2.5:build (build) on project my-service: Exception caught
        at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:217)
        at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:153)
        at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:145)
        at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(LifecycleModuleBuilder.java:84)
        at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject(LifecycleModuleBuilder.java:59)
        at org.apache.maven.lifecycle.internal.LifecycleStarter.singleThreadedBuild(LifecycleStarter.java:183)
        at org.apache.maven.lifecycle.internal.LifecycleStarter.execute(LifecycleStarter.java:161)
        at org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:320)
        at org.apache.maven.DefaultMaven.execute(DefaultMaven.java:156)
        at org.apache.maven.cli.MavenCli.execute(MavenCli.java:537)
        at org.apache.maven.cli.MavenCli.doMain(MavenCli.java:196)
        at org.apache.maven.cli.MavenCli.main(MavenCli.java:141)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:601)
        at org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced(Launcher.java:290)
        at org.codehaus.plexus.classworlds.launcher.Launcher.launch(Launcher.java:230)
        at org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode(Launcher.java:409)
        at org.codehaus.plexus.classworlds.launcher.Launcher.main(Launcher.java:352)
Caused by: org.apache.maven.plugin.MojoExecutionException: Exception caught
        at com.spotify.docker.AbstractDockerMojo.execute(AbstractDockerMojo.java:65)
        at com.spotify.docker.BuildMojo.execute(BuildMojo.java:79)
        at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo(DefaultBuildPluginManager.java:101)
        at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:209)
        ... 19 more
Caused by: java.nio.file.InvalidPathException: UNC path is missing sharename: /\my-service-war-exec.jar
        at sun.nio.fs.WindowsPathParser.parse(WindowsPathParser.java:118)
        at sun.nio.fs.WindowsPathParser.parse(WindowsPathParser.java:77)
        at sun.nio.fs.WindowsPath.parse(WindowsPath.java:94)
        at sun.nio.fs.WindowsFileSystem.getPath(WindowsFileSystem.java:255)
        at java.nio.file.Paths.get(Paths.java:84)
        at com.spotify.docker.BuildMojo.copyResources(BuildMojo.java:570)
        at com.spotify.docker.BuildMojo.execute(BuildMojo.java:254)
        at com.spotify.docker.AbstractDockerMojo.execute(AbstractDockerMojo.java:63)
        ... 22 more
[ERROR]
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException
c:\devel\myorg\cc\my-service>

Plugin configuration used for that project is as follows:
<configuration>
	<baseImage>${docker-registry}/lightweight/oracle-java-7-debian</baseImage>
	<cmd>java -jar /${project.name}-war-exec.jar</cmd>
	<exposes>
		<expose>8080</expose>
	</exposes>
	<resources>
		<resource>
			<targetPath>/</targetPath>
			<directory>${project.build.directory}</directory>
			<include>${project.build.finalName}-war-exec.jar</include>
		</resource>
	</resources>
</configuration>

remove /

@arghya88 your suggestion worked great for me

If you want to copy your resources to a specific folder in the docker image you will need targetPath. In this case it is still a problem.
<targetPath>/opt/${project.artifactId}</targetPath>   -- Fails in windows

What happens if you use backslashes in the targetPath?

I'm running into this issue too on version 0.2.11. I've tried:
/
${file.separator}
/
\/
\
//
Any helpers or work arounds would be great.

I have not tested it yet, but it looks like someone has made a fix. It's waiting in a PR
https://github.com/oberthur/docker-maven-plugin/commit/1f303e0aeb76aee5122d6d7bda4cd240518a6337

I've now tested oberthur's fork and it works for me on windows. The change is in https://github.com/spotify/docker-maven-plugin/pull/65

In case you can't wait for that PR, here is a workaround I came up with:
<!-- define a default value in <properties> ->
<properties>
    ...
    <docker.resource.targetPath>/</docker.resource.targetPath>
</properties>

<!-- define a profile that is activated in windows environments -->
<profiles>
	<profile>
		<id>windows</id>
		<activation>
			<os>
				<family>Windows</family>
			</os>
		</activation>
		<properties>
			<docker.resource.targetPath></docker.resource.targetPath>
		</properties>
	</profile>
</profiles>

<!-- then set the targetPath using the property -->
<build>
    <plugins>
        <plugin>
                <groupId>com.spotify</groupId>
                <artifactId>docker-maven-plugin</artifactId>
                <version>0.2.12</version>
                <configuration>
                    <imageName>${docker.image.prefix}/${project.artifactId}</imageName>
                    <dockerDirectory>src/main/docker</dockerDirectory>
                    <resources>
                       <resource>
                           <targetPath>${docker.resource.targetPath}</targetPath>
                           <directory>${project.build.directory}</directory>
                           <include>${project.build.finalName}.jar</include>
 ...

Hope this helps.

@BrettFieber this worked for me, thanks

This should be resolved by https://github.com/spotify/docker-maven-plugin/pull/121/commits

I just tried this with 0.3.5 on Windows. It's still broken.

Not that it helps for the spotify plugin, but one I've found that works on windows is https://github.com/rhuss/docker-maven-plugin
Good luck.

Reopening this based on reports that it doesn't work. We welcome PR's but are unlikely to fix this at Spotify, since nobody has a Windows setup to test it out with.

I ran into this same issue, and have created a PR that works around the issue (its actually a bug in the Windows implementation of Paths.get)

@javacody do you still have issues with 0.3.9-SNAPSHOT? If so can you tell us the output of running your maven commands with the -e flag?

I was facing the same issue. Upgraded to 0.3.9 version and no longer facing the issue.

@sunieldalal thanks for the confirmation!


Method assertInternalType is deprecated

Create a mock for the assertInternalType method which is removed from PHPUnit. See also https://github.com/sebastianbergmann/phpunit/issues/3369

Wrong branch


Add run on repl.it badge to README

This pull request configures this repository to be run on Repl.it.      It adds a .replit configuration file and a Repl.it badge to the README.
You can read more about running repos on Repl.it here, or view the Repl here.

Nice idea, your link didn't seem to work for me though?
Could you amend it ?

Hi, the link will work when the changes are merged, since it relies on the .replit file. You can run the program by clicking the following link and clicking run.
https://repl.it/@Mosrod/soccer-cli

Oh, of course.


chore(release): Add changelog_file attr #39

By default python semantic release looks for CHANGELOG.md. This project uses CHANGELOG.rst, so it needs to know this.
closes #39

Wont pull this in, See issue #43 for additional work required.


feat: man pages and shell completions

Closes #57.

Looks like run_lua::test::test_run_lua is failing in the nix build (it used to be skipped because it wasn't a lib test).
It succeeds locally and shouldn't need an internet connection. I'll have to investigate this 👀


Question about ShareGPT data

Thanks for your impressive work!
I am wondering which version of ShareGPT data is used for training. In open-instruct, ShareGPT data are divided into two versions, including max_length = 2048 and max-length = 4096.

We set max_length=8192 for the script in open-instruct.

thanks!


Why I have different number of passing tests from NUnit Console Runner than Ghpr Reporter?

Hello,
I came across another number of tests in NUnit Console Runner (3.7.0) than Ghpr Reporter
From NUnit Console Runner I have this summary:
Overall result: Failed
20:54:40   Test Count: 59, Passed: 33, Failed: 8, Warnings: 0, Inconclusive: 12, Skipped: 6
20:54:40     Failed Tests - Failures: 1, Errors: 7, Invalid: 0
20:54:40     Skipped Tests - Ignored: 6, Explicit: 0, Other: 0
In Ghpr Reporter
Total: 53
Success: 51
Errors: 1
Failures: 1
Inconclusive: 0
Ignored: 0
Unknown: 0
My tests are using SpecFlow in Parallel. Maybe should I tried with property realTimeGeneration false/true. It didn't help.
Do you know any solution for this?

Hi @MichalSle !
The reason is in the way we read test results from SpecFlowPlugin.
I'll make some changes to NUnit Specflow Plugin, so we will take test result from NUnit test context.
Thank you for submitting this issue!

Awesome. Thank you!

Hi, this was released


How to change the default rem value

The default is 1=0.25rem, I would like to configure it to 1=1rem, how do I configure it?

Like this Playground


Change 'source activate' to 'conda activate'

Closes #830.
The other occurrence of 'source activate' should not be change as it is not limited to conda and may refer to other virtual environments.

This change is

/assign
/ok-to-test
/lgtm
/approve
Thank you!


if ceph-disk fails to activate an OSD then bubble up the error

Fixes issue #670

test this please


Fix for require not defined on electron 12+

Breaking changes in electron 12 here:
require() cannot be used in the renderer process unless nodeIntegration is true and contextIsolation is false.

Thanks!


SQLite Database

Type: Feature Request
Hi, I have problem with installing SQLite database. I run command "pip install pysqlite3" it will be for a few seconds downloading a then it will show error. Can you help with it? Thank you
VS Code version: Code 1.77.0 (7f329fe6c66b0f86ae1574c2911b681ad5a45d63, 2023-03-29T10:02:16.981Z)
OS version: Windows_NT x64 10.0.19044
Modes:
Sandboxed: Yes

We closed this issue because it is a question about using VS Code rather than an issue or feature request. Please search for help on StackOverflow, where the community has already answered thousands of similar questions. You may find their guide on asking a new question helpful if your question has not already been asked. See also our issue reporting guidelines.
Happy Coding!


Add Python 3.5 to travis.yml

Description

Add python 3.5 to the travis config file, so the CI runs two separate jobs (one on each Python version environment).
Motivation and Context

Further automated testing support.
How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] 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:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

The first travis build using both environments failed due to the issue mentioned in #77 and fixed at 0ee67aad721de6bdd6557ee67a7476856f35776c - but seems to be executing one job per python version, as expected. It might be a good idea to wait until #77 is merged in order to merge (otherwise all the commit builds will fail in 3.5).

Applied the fix from #77 to allow passing the 3.5 tests and avoid making this PR dependent on others.


feature: support this metric: Avg. Revenue by Brand

The metric named Avg. Revenue by Brand should be developed.
Example Request
GET /metrics?id=revenue&dimensions=brand&aggregate=avg

Example Response
{
  "metric": "revenue",
  "dimensions": ["brand"],
  "aggregation": "avg",
  "data": {
    "Nike": [
      {
        "value": "1.23"
      }
    ],
    "Samsung": [
      {
        "value": "2.00"
      }
    ],
    "Apple": [
      {
        "value": "3.00"
      }
    ]
  }
}

fixed at #18


Missing steps to set up permissions for assigning roles

The steps to create Service Connection (Step 3+4 from Configure Azure DevOps) only creates with Contributor role. User Access Administrator (or something that grants permission to create role assignments) is also needed because terraform script is doing a lot of azurerm_role_assignment.
The documentation should include steps to assign a least privilege role to allow role assignments

@terrytsay Thanks a lot for adding this item.  I believe this should now be more clear with our updated documentation.


define and document on how to add cases with pytest or robot

Is your enhancement proposal related to a problem? Please describe.
Now zephyr has supported robot and pytest, and now the robot script is alone with samples code folder, and pytest cases are put to a testsuites/pytest/... folder. we need clear define on how to adding new cases for it.
Describe the solution you'd like
some proposal:

keep the existing paradigm as is, and let robot script go with source code, and pytest scripts goes to the testsuits/pytest folder with casename as subfolder.
force all test scripts to go with src code.
force all inteactive test scripts goes to a testsuits/pytest folder, or even a separated repo.

Additional context
https://github.com/zephyrproject-rtos/zephyr/blob/main/samples/subsys/shell/shell_module/shell_module.robot
https://github.com/zephyrproject-rtos/zephyr/blob/main/samples/subsys/testsuite/pytest/shell/pytest/test_shell.py

We can only say that we are -1 on moving all pytests/robot test to special place, since the policy was always to keep tests directly in a folder with source code of the test app they relate to. We just proposed an extra subfolder called pytest. We also think that moving pytest files directly to the same folder where the testcase.yaml or to src will cause a mess. There can be more than one file used by pytest for such tests and keeping all the files mixed together will just cause mess.
We don't mind the location for robot files.

according to TWG discussion the agreements are:
Prefer : Add scripts along with source code src folder.
Not prefer: add a new directory to host the test script, unless author can justify

Let me rephrase:
It is recommended to put tests in the same "top" directory the sources are. However, there can be some extra structure introduced by the tests. The reasoning is that when test are in some completely different place it is harder to figure out what is failing and where to find the code.


Tag Helpers and label asp-for not showing the label

From @rhires on October 23, 2015 14:18
Hello...so I have this code, straight from ef.readthedocs.org, in the aspnet5 section  (unzipped the example, navigated down to the proper level, ran dnu restore, then dnx web)...and for some reason or other, it doesn't show the label in the rendered view for the "create" section. I know that the inputs are correct, it's just the labels that are invisible. The question is: do I have to update the POCOs in the model to have display attributes to be able to see the label? Actually, to answer my question, no, updating the POCO with the [Display] attribute didn't change anything.
I'm running beta8, on Yosemite. Unfortunately, there isn't that much regarding tag helpers, and I haven't seen this one anywhere.
Copied from original issue: aspnet/Home#1025

@rhires please upload the screenshot mentioned in your original description. Need either the generated HTML or that screenshot to understand what you're seeing.

From @rhires on October 23, 2015 18:35
Hmmm..sending this from email, since when I attempted to include a screen shot in .png github wouldn’t include it. (ETA - no image here, either. Not sure why github doesn't like my png)
And the html -
New Blog

On Oct 23, 2015, at 11:34 AM, Doug Bunting<EMAIL_ADDRESS>wrote:
@rhires https://github.com/rhires please upload the screenshot mentioned in your original description? Need either the generated HTML or that screenshot to understand what you're seeing.
—
Reply to this email directly or view it on GitHub https://github.com/aspnet/Home/issues/1025#issuecomment-150609594.

This is a defect in the Create.cshtml file. I'll move the issue to the aspnet/EntityFramework.Docs repo. Quick fix is to add the missing </label> end tag -- that element should not be self-closing.
details
The MVC tag helpers do not overrule the .cshtml author. If the file contains <label/> or <input>something</input>, so will the generated HTML. This is true whether or not an MVC tag helper targets the element.
In this case the LabelTagHelper updated the <label>'s attributes and content. The new content is not visible because the file uses a self-closing tag.

d5f8910


Create Samples & Templates section

We will create a metadata section that includes an example of each @type
Including these:
#126  #125 #124

I think it makes sense to incorporate this as the 'Samples & Templates' section referenced in #238.
I'd like to propose that the samples displayed in this section would forego the 3 column layout that currently exists and instead show the AMP article exactly as-is.
Supporting media (images, videos) would be stored in a directory with the same name as the amp document.

@jmadler each vertical sample has already a preview mode which hides all ABE related styling. Is this what you want? Besides, I think especially for these samples the annotated source view is really valuable.

The section exists now.


'sort_on_has_many' attribute does not work.

I declared 'sort_on_has_many' => true attribute on $sortable array field in ModelX.
After that the arrows should be shown on ModelY Resource HasMany relationship with ModelX, but it does not work.
However, the arrows are staying on the index view.
Simplified models and Resources:
ModelX:
class Category extends Model implements Sortable
{
    use HasFactory, SortableTrait;

    protected $sortable = [
        'order_column_name' => 'sort_order',
        'sort_when_creating' => true,
        'sort_on_has_many' => true,
    ];
}

Resourece (ModelX):
class Category extends Resource
{
    use HasSortableRows;

    public static $model = \App\Models\Category::class;
}

Resource (ModelY):
class Category extends Resource
{
    use HasSortableRows;

    public static $model = \App\Models\Category::class;

    public function fields(Request $request)
    {
        return [
            HasMany::make('Subcategories', 'childCategories', Category::class),
        ];
    }
}

Wait, so ModelX and ModelY are the same model?

Yes.
Use case - categories who has parent categories.
I kind of solved it by having parent models displayed in nova index view and in each category (with parameter 'sort_on_has_many' => true) i had to display HasMany relationship to Subcategories (with parameter 'sort_on_has_many' => true).

Shouldn't the class names be different?

Closed if for now. If that did not work we can re-open.

@dovydasbu Then I think you have to create a second sort column for subcategories.
public $sortable = [
        'order_column_name' => 'sort_order_children',
        'sort_when_creating' => true,
        'sort_on_has_many' => true,
];


Fix build-machinery-go dependencies

This commit fixes two things about our build-machinery-go includes:

The images.mk include uses ensure-imagebuilder, which comes from imagebuilder.mk. We shouldn't have to include the latter explicitly. https://github.com/openshift/build-machinery-go/pull/50 is fixing that, but until that merges and we can vendor it in, include it explicitly.
ensure-kustomize was working as far as it went, but it was installing kustomize into a subdirectory which is (usually) not in $PATH, so things like make deploy were either breaking (bad) or picking up kubebuilder from somewhere else (potentially worse). Fix our usage of kustomize to ref the variable that ensure-kustomize sets up to point to the binary it installs.

/assign @abutcher

/test e2e

/test e2e

/test e2e

/lgtm


linked-list: rename Deque to DoublyLinkedList

see #231 for rationale and conversation.

Fixed in #287!


bump rivet, yoda, lhapdf

Motivation for this change
Things done

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[x] Linux

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Don't merge yet. Need to test lhapdf reverse dependencies.

nix-shell -p nox --run "nox-review pr 27366" passes on NixOS and Darwin now. Good to merge.

ok, will look into it.

Looks like LHAPDF upstream changed its file:
path _/nix/store/vsd414pj6mzwqqi662bpi6zjgwgnkcld-LHAPDF-6.2.0.tar.gz_ has sha256 hash _0gfjps7v93n0rrdndkhp22d93y892bf76pnzdhqbish0cigkkxph_ when _005jfapdj3mmk62p9qgvw7nyg93pqy249p1xy2ws1qx42xj76lih_ was expected

@Mic92  Thank you!


code.tizen.org - example - Popup - not displayed properly #195

Visit:
https://code.tizen.org/demos?path=1.0%2Fexamples%2Fmobile%2FUIComponents%2Fcomponents%2Fpopup%2Fpopup.html
Observe:
List items are spread

Compare to:
https://samsung.github.io/TAU/examples/1.0/examples/mobile/UIComponents/components/popup/popup.html

This may be connected to https://github.com/Samsung/TAU-Design-Editor/issues/128

Fixed in https://github.com/Samsung/TAU-Design-Editor/issues/128

The #145 PR has been already merged.


FancyZones scales Notepad improperly

Microsoft PowerToys version
0.55.2
Running as admin

[X] Yes

Area(s) with issue?
FancyZones
Steps to reproduce

place notepad in zone
observe edges in comparison to adjacent windows of other apps (pictured paint, chrome for examples)

Log:
PowerToysReport_2022-02-21-11-59-20.zip
✔ Expected Behavior
gridlocked scaling
Paint.exe and other apps scale properly

❌ Actual Behavior
height is consistently a few pixels over bottom edge of zone, unlevel with adjacent zones

edit: looking more closely, the top window border alights spot on while the left, bottom, and right borders are a few pixels over.
Other Software
No response

Hi friends! This bug is relatively old. In an effort to clear out any stale bugs, could you please let us know if this bug is still showing up on any of the latest builds, 0.74 or up!
/needinfo

Thanks for following through with this. I personally lost track of this issue while away from Windows as my main OS. I am back on it now and cannot recreate the issue as I did back in feb '21. rock on ^_^


Added FCS calculation to posix radio

This PR adds FCS calculation to posix radio, so that I don't get an annoying 'Bad FCS' message in Wireshark :-)
Note that the CRC code is taken from spi-hdlc-adapter.c; I can refactor it from both places into a utility method. I wasn't sure if code in tools can depend on non-public code form src.

Current coverage is 68.52% (diff: 100%)

Merging #709 into master will decrease coverage by 1.02%

@@             master       #709   diff @@
==========================================
  Files           101        100     -1
  Lines         15231      15229     -2
  Methods        2074       2071     -3
  Messages          0          0
  Branches       1799       1800     +1
==========================================
- Hits          10594      10436   -158
- Misses         4062       4208   +146
- Partials        575        585    +10

Powered by Codecov. Last update dfd5fbd...a168893

Fixed.


Replace Waiting_to_start value for state attribute in miq_tasks table with Queued

Part of #14698 - adding former Job specific columns to Task view
After linking jobs and miq_tasks tables (in https://github.com/ManageIQ/manageiq/pull/13452), Job#state attribute duplicated to MiqTask#state. One of possible value for state attribute in Job model is Waiting_to_start which got assigned to job not started yet. in MiqTask there is Queued value for state attribute which indicate the same for task - task is not started yet.
This PR is updating Waiting_to_start value of state attribute in miq_tasks table to Queued.
It would make filtering of Tasks more intuitive after merging Tasks and Jobs screens in UI

@miq-bot add_label core, enhancement

@miq-bot add-label  fine/no

@gtanzillo could you review

@Fryguy all feedback addressed,ready for the second review


Enable shallow clone only

Do only a shallow clone (--depth 1).

Ps. I saw you didn't merge this one. --depth 1 means not depths in directory, but depth in git history. This will clone the git repository without the full git history, meaning it's much faster and more efficient. A shallow clone is much better in these cases, especially since users only want to grap the code and make it part of their binary / compile, no need for a full git clone.


splunk_hec/platform_logs  Exporting failed. Will retry the request after interval  "error EOF"

Hey team,
In working with customers who are sending logs from OTel through a client managed Load Balancer (mostly f5s so far) to Splunk "Heavy Forwarders" and are seeing "EOF" errors on their logging exporters:
2022-04-05T12:55:27.607Z info exporterhelper/queued_retry.go:215 Exporting failed. Will retry the request after interval. {"kind": "exporter", "name": "splunk_hec/platform_logs", "error": "Post \"https://foo.bar.com:8088/services/collector\": EOF", "interval": "3.985412818s"}
2022-04-04T16:39:44.614Z    info    exporterhelper/queued_retry.go:218    Exporting failed. Will retry the request after interval.    {"kind": "exporter", "name": "splunk_hec/platform_logs", "error": "Post \"https://foo.bar.com:443/services/collector/event\": EOF", "interval": "4.623187485s"}

In my digging on the internet, it looks like OTel will try and reuse connections and that they may have been closed by the far end (either the LB or Splunk, I'm still trying to prove either).
Could use a hand with some guidance on what might be causing this or how to tune OTel, the LB or Splunk to resolve.
Could this be an opportunity for tuning the max_connections we use on the daemonset to ensure we aren't overwhelming the LB or Splunk?
https://github.com/signalfx/splunk-otel-collector-chart/blob/97dc6dca6f04a764a1fe48658453de72ed747a35/helm-charts/splunk-otel-collector/values.yaml#L43-L44
What can we look at to attempt to tune the LB or Spunk side?

Looks like both users are sending through an f5 with default TCP settings:
idle timeout : 300 s
keepalive: 1800 s
time wait : 2000 ms

https://support.f5.com/csp/article/K29377715
https://support.f5.com/csp/article/K13004262
I believe this, in conjunction with the busyKeepAliveIdleTimeout HTTP input setting on Splunk side may be the issue:
keepAliveIdleTimeout = <integer>
* How long, in seconds, that the HTTP Event Collector input lets a keep-alive
  connection remain idle before forcibly disconnecting it.
* If this value is less than 7200, the input sets it to 7200.
* Default: 7200

busyKeepAliveIdleTimeout = <integer>
* How long, in seconds, that the HTTP Event Collector lets a keep-alive
  connection remain idle while in a busy state before forcibly disconnecting it.
* CAUTION: Setting this to a value that is too large
  can result in file descriptor exhaustion due to idling connections.
* If this value is less than 12, the input sets it to 12.
* Default: 12

I know we commonly tweak the busyKeepAliveIdleTimeout to a setting higher than the LB, but will need to keep digging to prove if this is the root cause of the EOF

The client will throw an EOF error when the server has closed the connection and the client is using still sending packets to the closed connection. It can happen when the client's keep-alive timeout is greater or equal to the server's keep-alive timeout.
OTel agent has a fixed keep-alive timeout of the 30s. I suggest revisiting the splunk and load balancer's timeout configs.
Also. we should close it as it looks like it's not collector specific issue.

We can close it, would you please open issues upstream?

Did we create an issue for this upstream?
Need to reopen this after some recent testing.
Need the HEC exporter to provide configurable http idle timeout similar to what we used for fluentd-hec
https://github.com/splunk/fluent-plugin-splunk-hec#idle_timeout-integer
In previous threads I believe we found OTel collector had a 30 second hardcoded timeout. We need this to be lower by default to avoid re-using connections that are torn down, which causes the EOF.
Splunk's default timeout for "busy idle connections" is 12 seconds.
We need it set like fluentd-hec by default to avoid the EOFs which we never saw.

Submitted up stream issue here

We made this configurable - and our timeout is now 10s by default, but not sure that is the same as the idle timeout. More testing required.

Workaround, the exporter takes the key idle_conn_timeout to set the value.
You can set like so:
exporters:
  splunk_hec/foo:
    ...
    idle_conn_timeout: 10s


Timeout

There is an Exception which says The request was canceled due to the configured HttpClient.Timeout of 100 seconds elapsing.
it should increase the Timeout before downloading

it usually occurs when downloading larger files


Add run_exports

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

See https://github.com/conda-forge/staged-recipes/pull/15233#issuecomment-857579243
/cc @xhochy @wolfv


GEODE-4637: Allow only GEODE_HOME, remove GEODE

@pivotal-jbarrett have a look, requested changes made, will squash/merge if approved


Failure to load a valid SEG object due to incorrect expectations about ReferencedSegmentNumber

This study fails:
https://idc-sandbox-000.firebaseapp.com/projects/idc-dev-etl/locations/us-central1/datasets/idc-tcia/dicomStores/idc-tcia/study/<IP_ADDRESS>.4.1.145<IP_ADDRESS>.4334.1100.123677133754901543013105036670
With this error:

It is allowed not to have ReferencedSegmentNumber, see https://dicom.innolitics.com/ciods/segmentation/general-reference/00081140/0062000b. dciodvfy does not complain about it either, and Slicer can load that seg.

@fedorov I was debugging this issue and the code expects the following:
const segmentIndex =
            PerFrameFunctionalGroups.SegmentIdentificationSequence
                .ReferencedSegmentNumber;

According to innolitics: https://dicom.innolitics.com/ciods/segmentation/segmentation-multi-frame-functional-groups/52009229/0062000a/0062000b
The Referenced Segment Number Attribute under Segment Identification Sequence Attribute is a type 1 required, the one you mentioned is under Purpose of Reference Code Sequence Attribute which is optional.

@igoroctaviano thank you, you are correct!

I think it might make sense to check the number of segments, and accept the object that does not have a reference when that number is 1, but I definitely will understand your reluctance to add this special case. Let me know your thoughts.

Meanwhile, I will contact the authors of that data and check if they would be willing to update the actual dataset.

I think it might make sense to check the number of segments, and accept the object that does not have a reference when that number is 1, but I definitely will understand your reluctance to add this special case. Let me know your thoughts.

This will be a dcmjs change, it's a very small change if we want to add this case (checking the number of segs). Let me know if I should proceed with the update.

Somehow, both of us missed that - ReferencedSegmentNumber IS available in the shared FG, and it does not need to be replicated in the per-frame FG. This is a bug in OHIF, after all.

Somehow, both of us missed that - ReferencedSegmentNumber IS available in the shared FG, and it does not need to be replicated in the per-frame FG. This is a bug in OHIF, after all.

Good catch. PR opened in dcmjs: https://github.com/dcmjs-org/dcmjs/pull/166


No files shown

Using the file explorer portion, no files show in my images or most other folders.  Only the .html, .php files.  Couldn't find any settings other than permissions, which I changed to allowed and changed back to inherited.

Everything is showing fine at my end. Could you give me some screenshot?

From: Mr. Meo @.***
Sent: Monday, August 30, 2021 8:36 PM
To: trananhmanh89/ffexplorer
Cc: teddyearp; Author
Subject: Re: [trananhmanh89/ffexplorer] No files shown (#6)
Everything is showing fine at my end. Could you give me some screenshot?
<https://user-images.githubusercontent.com/20571336/131437610-e78b4a22-9553-
4682-87be-c4691dc6c9b3.png> 2021-08-31_103502

You are receiving this because you authored the thread.
Reply to this email directly, view
<https://github.com/trananhmanh89/ffexplorer/issues/6#issuecomment-908870896

it on GitHub, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ATIK44LXISY42HE5DSYT43LT7
RE27ANCNFSM5DCKQHNA> .
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&m
t=8&pt=524675>  or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=u
tm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>
.
<https://github.com/notifications/beacon/ATIK44NFW3UARZCRWE2NNWDT7RE27A5CNFS
M5DCKQHNKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOGYWEJ4A
.gif>

Sorry, i can't see your photo

You can't open the attachment?
Misspelled from my flip phone
On Tue, Aug 31, 2021, 9:01 AM Mr. Meo @.***> wrote:

Sorry, i can't see your photo
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/trananhmanh89/ffexplorer/issues/6#issuecomment-909369573,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATIK44MJONI6TC7I77K34VTT7T4ETANCNFSM5DCKQHNA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Misspelled from my flip phone
On Tue, Aug 31, 2021, 9:01 AM Mr. Meo @.***> wrote:

Sorry, i can't see your photo
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/trananhmanh89/ffexplorer/issues/6#issuecomment-909369573,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATIK44MJONI6TC7I77K34VTT7T4ETANCNFSM5DCKQHNA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Can't. I'm using github web.

Try that.  I was just replying to emails.

I tried to replicated your issue, but not success. Could you check that folder permission? May be there is no read permission.

My joomla is running on a LAMP stack and not a VM. PHP 7.4.2, MariaDB 10.x,
Apache 2.4.2.  The permissions for all my folders being served are 755 and
chmod to www-data:www-data.  My site wouldn't even work otherwise.

From: Mr. Meo @.***
Sent: Tuesday, August 31, 2021 8:55 PM
To: trananhmanh89/ffexplorer
Cc: teddyearp; Author
Subject: Re: [trananhmanh89/ffexplorer] No files shown (#6)
I tried to replicated your issue, but not success. Could you check that
folder permission? May be there is no read permission.
<https://user-images.githubusercontent.com/20571336/131609155-d3c0c173-f218-
4bb5-b0b1-9c03b7286c95.png> 2021-09-01_105419

You are receiving this because you authored the thread.
Reply to this email directly, view
<https://github.com/trananhmanh89/ffexplorer/issues/6#issuecomment-909854992

it on GitHub, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ATIK44IYAF7TDFLAV7ORVKDT7
WP2NANCNFSM5DCKQHNA> .
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&m
t=8&pt=524675>  or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=u
tm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>
.
<https://github.com/notifications/beacon/ATIK44NYBSKPV4AMSEBGOVTT7WP2NA5CNFS
M5DCKQHNKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOGY5USEA
.gif>

Hi,
You can check the code at administrator\components\com_ffexplorer\controllers\explorer.php
Function explodeFolder to see what happened.

Well, I don't know php code or I would perhaps write my own, lol. ;)  But here is what I copied from that section of the file.
explodefolder.txt

I don't see any thing wrong here. Could you share your admin info to gmail trananhmanh89 ?


[EXP-29-229] Brownie bridge: gRPC integration

In this PR I've exposed brownie functionality via gRPC.
By opening multiple channels it's possible to do async multi-chain calls.
Currently, it's only returning token prices from the exporter, but it's quite easy to extend to other methods as well.
Build
set WEB3_PROVIDER, EXPLORER, FTM_WEB3_PROVIDER, FTM_EXPLORER envs
launch it with : make bb-rebuild
Usage for js clients:
npm install https://github.com/mariuspod/yrpc.git
sample js client:
https://github.com/mariuspod/yrpc/blob/main/sample_client.js
We can run the following snippet to get all prices for the given tokens for all chains:
require('dotenv').config();
const { YearnExporter, grpc } = require("./yrpc");

const host = process.env.HOST || 'localhost';
const ethPort = process.env.ETH_PORT || 1337;
const ftmPort = process.env.FTM_PORT || 1338;

// TODO add secure channel example
const ethClient = new YearnExporter(`${host}:${ethPort}`, grpc.credentials.createInsecure());
const ftmClient = new YearnExporter(`${host}:${ftmPort}`, grpc.credentials.createInsecure());

/**
 * this is a sample client requesting the prices for several contracts over gRPC.
 */

const ethAddresses = [
  "0x0bc529c00c6401aef6d220be8c6ea1667f6ad93e",
].map(a => ({ address: a, network: "mainnet" }));

const ftmAddresses = [
  "0x29b0Da86e484E1C0029B56e817912d778aC0EC69",
].map(a => ({ address: a, network: "ftm-main" }));

call(ethClient, ethAddresses);
call(ftmClient, ftmAddresses);

function call(client, addresses) {
  const call = client.GetPrices({addresses: addresses});
  console.log("calling yearn_exporter.getPrices()...");
  call.on('data', function(address) {
    console.log(address);
  });
}

linked task in Web project: https://linear.app/yearn/issue/WEB-1325/[web-1325-216]-add-exporter-pricing-service

Per discussion with @mariuspod leaving this unmerged until further notice.

closing this until there is a more suitable service infra for this usecase


Issue #49 and intro updates

Added CC 4.0 License, intro updates, and Home link at the top

@ALRubinger mind taking a quick look?


Create gorilla tag

In my opinion, it is important to get both this PR and #2 merged to ensure the safe future of this mod.

I agree.


Dynamic Routes (react router dom)

We need to have dynamic page routing (unique URLs) for the detail page of each hotel. The routing to the payment page should add on to the URL of the detail page.
Example Test Case:

localhost:3000/search
Click on any of the preview cards
localhost:3000/detail/MarriotHotel
Click on BOOK NOW
localhost:3000/detail/MarriotHotel/payment
and so on...

Routing should be finished. Each hotel will be led to their corresponding details page with the url:
localhost3000/details/HOTELNAME
Ex: localhost3000/details/SomeHotel


Update to secrecy 0.10.

Hi,
I am trying to update rage to<EMAIL_ADDRESS>but that requires updating pinentry first.
Can you review this please?

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

Project coverage is 7.89%. Comparing base (7410d73) to head (7930792).

Files with missing lines
Patch %
Lines

src/assuan.rs
0.00%
1 Missing :warning:

src/lib.rs
0.00%
1 Missing :warning:

Additional details and impacted files
@@          Coverage Diff          @@
##            main     #13   +/-   ##
=====================================
  Coverage   7.89%   7.89%
=====================================
  Files          3       3
  Lines        228     228
=====================================
  Hits          18      18
  Misses       210     210

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


kaa-base, kaa-metadata not available for win32

Subliminal builds fine on Linux, no problem. It also used to run fine on Windows, but not anymore since you started using kaa-metadata. It seems it is quite difficult to get the kaa libraries working on win32. (cf<EMAIL_ADDRESS>

Nvm, I use linux

Here is a fork of kaa-metadata where I try to remove kaa.base dependency to put windows support back.
With kaa-metadata, subliminal can detect embedded subtitles, this is a nice feature IMO.
You can stick to 0.4 if you want windows support for now.

Great, will keep watching your kaa-metadata fork. Thanks!

Fork renamed parseit. It works fine but I need more tests cases and redo the setup.py.

Now using enzyme 03f209b2b0531cc9c03c0ceba3006952e4d79f2a


Add convert data

Summary
When converting from a DAGCircuit to a DAGDependency, in some cases it would be useful to compute and return additional information related to this conversion, such as the mappings between the input DAGOpNodes and the output DAGDepNodes. In particular, this would allow analysis passes that translate a DAGCircuit to a DAGDependency, collect blocks of nodes from the DAGDependency (exploiting the commutativity information offered by the latter structure), find the lists of corresponding nodes in the origin DAGCircuit and (for example) resynthesize these nodes. A direct application would be to extend the new StarPreRouting pass to collect and resynthesize "star-shaped" pieces of the circuit while also exploiting the commutativity between the nodes.
This PR adds a new class CircuitConversionData that stores the mappings and extends the conversion functions to dag_to_dagdependency_with_data and _dag_to_dagdependency_v2_with_data (all of the code pertaining to DAGDependency_V2 is private for now).
The reason that CircuitConversionData is a class rather than a mapping is that we might want to store more information in the future, and we would not need to keep extending the API. And it would also probably be easier to convert this to Rust in the future.
I am wondering whether I should make the class CircuitConversionData also private or also extend other circuit converters such as circuit_to_dagcircuit, etc. What do you think?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

37 of 37 (100.0%)  changed or added relevant lines in 6 files are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.02%) to 89.625%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/lex.rs
5
92.88%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
62333

Relevant Lines:
69549

💛  - Coveralls


docs: update contributing docs code block language

Some of the language specify of code block is missing

Thanks @PeterDaveHello!


Create dummy platform tests

Description of changes:

PR: https://github.com/awslabs/autogluon/pull/1464 is not able to trigger SCD.
SCD seems to require the master branch to have at least a workflow to be dispatched to work properly(you can even delete it afterwards, which is weird). This fix works in my fork:https://github.com/yinweisu/autogluon/pull/2, so worth trying out.
Also updated SCD to the latest version and added error msg

This change needs to be merged to take effect.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-1466-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-1466/1/index.html


Add hive metastore connectivity and permission check tool

Add a tool to check the connectivity and functionalities of an existing hive metastore.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/9712/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/9713/
Test FAILed.

Jenkins, test this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/9716/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/9720/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/9733/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/9734/
Test FAILed.

CLI tests failed consistently on cannot find test logs
concurrentListDirsNonPersistedDir(alluxio.client.fs.ConcurrentFileSystemMasterCreateTest)  Time elapsed: 1.764 sec  <<< ERROR!
java.lang.RuntimeException: java.io.FileNotFoundException: ./target/logs/tests-alluxio.client.fs.ConcurrentFileSystemMasterCreateTest-concurrentListDirsNonPersistedDir.log (No such file or directory)
Caused by: java.io.FileNotFoundException: ./target/logs/tests-alluxio.client.fs.ConcurrentFileSystemMasterCreateTest-concurrentListDirsNonPersistedDir.log (No such file or directory)

jenkins, test this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/9738/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/9740/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/9741/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/9744/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/9753/
Test PASSed.

alluxio-bot, merge this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/9754/
Test FAILed.


Foreign constraint error when trying to execute an Insert query

mysql:5.7.30
deno: 1.0.5
deno_mysql: 2.2.0
I am trying to seed a test db.  I have a users table and credentials table.  The credentials table has a foreign key constraint on a field called userId.  In the following code I create a user record and use the id to insert a record in the credentials table.  When I run this query manually it works.

import { mysqlTestClient } from "./db/mysqlClient.ts";
import { bcrypt } from "./deps.ts";

const users = [
  {
    name: "john"
  }
]

const creds = [
  {
    email<EMAIL_ADDRESS>    password: "password123"
  }
]

async function main() {

  console.log("\n\n************SEEDING TEST DB************\n\n");

  let insertedUserIds = [];
  for (const user of users) {
    const { name } = user;
    const { lastInsertId } = await mysqlTestClient.execute("INSERT INTO users(name) values(?)", [name]);
    insertedUserIds.push(lastInsertId);
  }

  for (let i = 0; i < creds.length; i++) {
    const userId = insertedUserIds[i];
    const { email, password } = creds[i];
    const salt = bcrypt.genSaltSync(10);
    const hashPW = bcrypt.hashSync(password, salt);
    await mysqlTestClient.execute("INSERT INTO credentials(email, password, userId) values(email, password, userId)", [
      email,
      hashPW,
      userId
    ]);
  }
}

When i run the above code I get the following error
error: Uncaught Error: Cannot add or update a child row: a foreign key constraint fails (`every_menu`.`credentials`, CONSTRAINT `credentials_ibfk_1` FOREIGN KEY (`userId`) REFERENCES `users` (`id`))
          throw new Error(error.message);
                ^
    at Connection.nextPacket (https://deno.land/x/mysql@2.2.0/src/connection.ts:95:17)
    at async Connection.execute (https://deno.land/x/mysql@2.2.0/src/connection.ts:162:19)
    at async https://deno.land/x/mysql@2.2.0/src/client.ts:101:14
    at async Client.useConnection (https://deno.land/x/mysql@2.2.0/src/client.ts:111:22)
    at async Client.execute (https://deno.land/x/mysql@2.2.0/src/client.ts:100:12)
    at async main (file:///Users/gabrielgrover/thrive-market/deno-demo/seedTestDb.ts:59:5)

Here are the schemas for both tables
users (
    id int(11) NOT NULL AUTO_INCREMENT,
    name varchar(255) NOT NULL,
    created_at timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP,
    PRIMARY KEY (id)
) ENGINE=InnoDB AUTO_INCREMENT=2 DEFAULT CHARSET=utf8;

credentials (
    id int(11) NOT NULL AUTO_INCREMENT,
    email varchar(255) NOT NULL UNIQUE,
    password varchar(255) NOT NULL,
    userId int(11) NOT NULL UNIQUE,
    created_at timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP,
    PRIMARY KEY (id),
    FOREIGN KEY (userId) REFERENCES every_menu.users(id)
) ENGINE=InnoDB AUTO_INCREMENT=2 DEFAULT CHARSET=utf8;

Not too sure what is happening.

I found the issue.  It was an error on my part.  The query should be
"INSERT INTO credentials(email, password, userId) values(?, ?, ?)"
instead of
""INSERT INTO credentials(email, password, userId) values(email, password, userId)""


Setting View Filter shows Addon Error in Eventlog

error: Addon error: Traceback (most recent call last):
    File "/home/coldfire/Projects/mitmproxy/mitmproxy/addons/view.py", line 521, in update
        idx = self._view.index(f)
    File "/home/coldfire/Projects/mitmproxy/venv/lib/python3.6/site-packages/sortedcontainers/sortedlist.py", line 2343, in index
        raise ValueError('{0!r} is not in list'.format(val))
    File "/home/coldfire/Projects/mitmproxy/mitmproxy/http.py", line 182, in __repr__
        return s.format(flow=self)
    File "/home/coldfire/Projects/mitmproxy/mitmproxy/connections.py", line 215, in __repr__
        host=self.address[0],
TypeError: 'NoneType' object is not subscriptable

Steps to reproduce the problem:

Open mitmproxy
set a value for view_filter in Options or using the : set view_filter={str}
Open a attached browser and open any url
Quit the browser and Open the Eventlog in mitmproxy.

Any other comments? What have you tried so far?
Another error occuring for the view_filter option. I was working on a fix for #2801 #2951 when i noticed this other error occurs if any value is setup in view_filter. I think it requires a complete patch once and for all.
System information
Mitmproxy: 4.0.0.dev103 (commit 77ed33b)
Python:    3.6.4
OpenSSL:   OpenSSL 1.1.0g  2 Nov 2017
Platform:  Linux-4.14.21-1-MANJARO-x86_64-with-arch-Manjaro-Linux

I noticed a weird thing, adding time.sleep(1) just before the return statement fixes the problem and no errors are generated. I am not sure why or how is this happening.
https://github.com/mitmproxy/mitmproxy/blob/cc00b03554d84a37b55f911a84d70b6d7909c9cb/mitmproxy/connections.py#L212

I'm seeing this too. The error appears onscreen and makes the feature hard to use. Is there a way to hide the error from displaying until a fix is found?

Not sure why we have a connection with address = None at this point, but in any case we should make ClientConnection.__repr__ and ServerConnection.__repr__ robust so that they don't crash then. This can be either checked in those functions directly, or we amend mitmproxy.util.human.format_address to also accept None and then return "" or something alike.

@mhils  I think there is another way, we can handle this. As i noticed earlier, if we add time.sleep(1) before the return statement, somehow the self.address  is set to a actual address value. So instead of handling self.address for None , can we check whether self.address is None and if it is  give it some time set an actual value.
if self.address is None:
        time.sleep(1)
return {str}

@fenilgandhi I have an assumption concerning this issue. I am not sure it is absolutely correct.
As you can see the issue appears when mitmproxy tries to give us a representation of ServerConnection object, when ValueError is raised inside https://github.com/mitmproxy/mitmproxy/blob/master/mitmproxy/addons/view.py#L521
For some reasons for some flows self.address is still None, when we need it to be available in ServerConnection. When we write time.sleep(1), we suspend execution of the calling thread for 1 second. Perhaps mitmproxy switches to other available thread and this thread set the value for self.address. The problem is that suspended thread is MainThread and every new case with self.address is None almost all mitmproxy functions will stop for 1 second.

@kajojify  I had the same notion but since i could not  find the setter/thread  which sets the value during the sleep, so i was not sure how to proceed with this. But if what you are saying is true, we can make sure that the setter method sets a value before the ServerConnection.repr  is returned.
setting time.sleep(1) was just a first instinct, but yeah time.sleep(0.1) works well too.

I can't reproduce this. Here's what I'm doing:

Start mitmproxy
:set view_filter=foo
switch to second window, enter curl -x localhost:8080 -k https://google.com/
Press E to open the event log.

@mhils I reproduced it only using a browser.

I managed to reproduce this today by printing a flow in http_connect, which makes a bit of sense at least. This is addressed in #3008. Thanks again for the original report!

@mhils, The pull-request #3008 only handles the scenario where the address is None and gracefully handles it but the bug which actually causes the value of an address to be None yet exists somewhere.
So maybe until a fix can found for it, shall i keep this issue open or only handling the None value completely fixes the bug somehow ?

I don't think there's another issue here. From the original traceback, it looks like we are checking if a flow is in the view, which isn't the case. This is totally expected, the problem here was that sortedcontainers used {0!r}, which triggers the flows __repr__, which then failed. It's okay for flows to have a server_conn.address of None, see e.g. the http_connect example.


Question: How can you set defaults for the models in use?

#How to set service broker defaults
From the documentation I understand that you can set certain defaults like resource group etc.
But I can't seem to find any other reference in the documentation or Helm chart.
How, as an Operations Guy can I set certain defaults?
In addition, what is the behavior when azure policies are applied to limit deployments to certain regions, sku's and enforce naming conventions?

From the documentation I understand that you can set certain defaults like resource group etc.

At one time, long prior to the GA release, you could specify default location and resource group when deploying the broker. If those were set, those two parameters became optional on all service/plan provisioning parameters. If they were still included in the provisioning parameters, they would override the defaults.
For reasons that are to complex to explain succinctly, we eventually concluded that it was best to not treat any parameters as special. From that point on (maybe three months ago) the mechanism for specifying a global default location and resource group was removed and those corresponding provisioning parameters are now always required for any service/plan where those are relevant-- which is most of them.
I thought all documentation was updated accordingly after that change, but if you can point me to any docs that may be misleading you, I'll be happy to correct them. (I can't spot where in the doc you linked to where there may be stale information.)

How, as an Operations Guy can I set certain defaults?

Since you referenced the Helm chart, I'll assume your platform of choice is Kubernetes. In that case, we expect that this need will be, eventually, be fulfilled through a feature in Kubernetes Service Catalog that allows an operator to define reusable templates for service/plan provisioning parameters with params pre-filled to their liking. @jeremyrickard or @carolynvs can probably point you to specifics, but it is still a WIP as I understand it.

In addition, what is the behavior when azure policies are applied to limit deployments to certain regions, sku's and enforce naming conventions?

The broker doesn't currently have a way of dealing with this. It also doesn't have a way of dealing with the fact that not all services are available in all regions. Dynamically validating a provisioning request against these kind of constraints isn't something I expect we can tackle in the near future. Obviously it would be a maintenance burden to keep this information up to date manually, and to the best of my knowledge, an API for us to query the availability of services in different regions doesn't exist (yet).

Hi @krancour,
thank for the explaination.
I found the options on the different model provision sections mention this.
For example:  the eventhubs provision section repeatedly states:

If an administrator has configured the broker itself with a default  and none is specified, that default will be applied, otherwise ...

I'm just starting with kubernetes, but could this be a usecase for admission controls?

@Stijnc thanks for the pointer there to some docs that definitely need an update. I'll get that squared away.

I'm just starting with kubernetes, but could this be a usecase for admission controls?

My understanding is that admission controls are for validating and authorizing request-- not for modifying them. I could very well be wrong about that. Still, I expect the "template" feature that Service Catalog is working on will be the way to go.

I agree with @krancour that the default stuff that svc cat is working on is the end goal you want and we'd welcome participation (even in terms of just validating experience or use cases!), but you could probably implement this with an Admission Webhook for now.
There are two flavors, depending on what you want to do:

Admission controllers may be “validating”, “mutating”, or both. Mutating controllers may modify the objects they admit; validating controllers may not.

So for the first one...

Mutating controllers may modify the objects they admit

So here you could override the ServiceInstance request that was coming in. There was some talk within the service catalog group about using this to hack in Cloud Foundry's VCAP_SERVICES behavior for bound services, you could probably implement something to achieve what you want above. You could do something similar to change a region or plan reference (harder..restricting that sort of thing would be better with namespaced resources and catalog restrictions)

validating controllers may not.

Here you could provide a validating web hook that rejected requests for ServiceInstances  that didn't conform to your organization's rules.

This seems like it was answered, so closing.


 The trajectory can be enabled by calling
dcmGenerator->setFirstDCMTrajectoryMode(FirstDCMTrajectoryMode::FifthOrderPoly);

by default the planner used the third order function, i.e., the original one.

I would also suggest to bump the version

Hi @S-Dafarra I fixed all the typos in https://github.com/robotology/unicycle-footstep-planner/pull/53/commits/d4bf6064267b0933666add58625b69e4d5f0509c and I bumped the version in https://github.com/robotology/unicycle-footstep-planner/pull/53/commits/a607742440b7ecdef0406acae5c7d0e034d6ec7b 😃

Hi @S-Dafarra I fixed all the typos in d4bf606 and I bumped the version in a607742 smiley

Great! Merging as soon as CI passes


Empty children

Hi!
Is it somehow possible to change the way how treeselect manages an empty children object?
I don´t need an expand arrow or "no children" text if it´s empty.
Thank you

if (!newNode.children.length) delete newNode.children;

Thank you for answering.
Unfortunately i have no idea where to put this code.

show that?

if this node have no chilren you can delete node's childrenwhen u format selectTree Data.
show following.

Ok. I should have given more precise information about the tree data. I get it from an api as json. The structure can be nested at several levels. Unfortunately it has the children item even if it´s empty.
But you pointed me in the right direction and now i check the entire json object recursively for empty children objects:
var isObject = (arg) => typeof arg === 'object';

/**
 *
 * @param {Object} o :: the object being cleaned
 * @param {String|Number} key :: the key or index to be cleaned from `o`
 */
function removeKey(o, key) {
  isObject(o) && Object.keys(o).forEach(function (k) {
    (k === key && o[k] && o[k].length === 0) && delete o[k] ||
      (o[k] && isObject(o[k]) && removeKey(o[k], key))
  });
  return o;
}

Thank you!

Have you solved the problem？
I also get data from an api as json.
let list = [{
        "id": "8a80169a6565be8d016565d89ce70000",
        "groupName": "Lev1",
        "parentId": "-1"
    },{
        "id": "ff808081658e8a1f01658f192ec00001",
        "groupName": "Lev2",
        "parentId": "8a80169a6565be8d016565d89ce70000"
    },{
        "id": "8a80169a6536485e0165364e299b0002",
        "groupName": "Lev3",
        "parentId": "ff808081658e8a1f01658f192ec00001",
    }];
function getJsonTree(data,parentId){
  let _this = this;
  let itemArr = [];
  for(let i = 0; i < data.length; i++ ){
    let node = data[i];
    if(node.parentId == parentId ){
      let newNode = {
        id: node.id,
        label: node.groupName,
        children: _this.getJsonTree(data,node.id)};
     // if (!newNode.children.length) delete newNode.children; // you can delete its children when its length equal to zero.
      itemArr.push(newNode);
    }
  }
  return itemArr;
}
let rootId = -1;
getJsonTree(list, rootId);

Hope it can help u.

I think so.
I had try 'noChildrenText' this option,  and set it for null, the tree also showed icon.

it would be nice to have a possibility to have branch nodes converted into leaf nodes if their children array is empty

I have this problem as well. please add an option that does not consider empty child arrays.

any updates?

Hello,
You can resolve this problem with normalizer.
here is an axample:
normalizer(node) {         return {           children: node.children.length ? node.children : 0         }       }
Hope can this help you

@DjKhireddine
Excellent! Thank you very much!

Hello,
You can resolve this problem with normalizer.
here is an axample:
normalizer(node) { return { children: node.children && node.children.length ? node.children : 0 } }
Hope can this help you

Where to place this? Thx

Hello,
You can resolve this problem with normalizer.
here is an axample:
normalizer(node) { return { children: node.children && node.children.length ? node.children : 0 } }
Hope can this help you

Where to place this? Thx

Call normalizer from your treeselect:

<treeselect
    ....
       :normalizer="your_normalizer"
    ....>
</treeselect>

Add normalizer in data-block like so:

your_normalizer (node) {
    return {
        children: node.children && node.children.length ? node.children : 0
        }
    }

Call normalizer from your treeselect:

<treeselect
    ....
       :normalizer="your_normalizer"
    ....>
</treeselect>

Add normalizer in data-block like so:

your_normalizer (node) {
    return {
        children: node.children && node.children.length ? node.children : 0
        }
    }


[Python] [Docs] Where's the changelog?

Describe the bug, including details regarding any error messages, version, and platform.
The only changelog I can find is https://github.com/apache/arrow/blob/main/CHANGELOG.md which hasn't been updated since 2021
Where can I find the changelog for pyarrow?
Component(s)
Python

Each release has a blog post with a summary of the changelog for each component with a link to the full list of resolved issues as well, e.g. the 16.0.0 release: https://arrow.apache.org/blog/2024/04/20/16.0.0-release/

Some related issues to the changelog file not being updated:

https://github.com/apache/arrow/issues/15000
https://github.com/apache/arrow/issues/30928
https://github.com/apache/arrow/issues/18757

(will close two of them in favor of one)
Separately from the CHANGELOG.md file and the release blogposts, it might be useful to have a page in the Python docs that links to those sources (I agree that a first place to look for this for pyarrow will typically be the docs)

Thank you!


[QUESTION]  PHPExcel_Reader_Exception in Excel2007.php line 82:

I want to add/append row/data into existing csv fiel, first Im creating a fiel with some data and then im appending some row into it but I'm getting this error, dont know how to resolve it.
PHPExcel_Reader_Exception in Excel2007.php line 82:
Could not open C:\xampp\htdocs\pixel_fb\C:\xampp\htdocs\pixel_fb\storage\excel/report-201608251335041.csv for reading! File does not exist.
This is what I'm trying.
`
class ExportController extends Controller {
protected $row_limit = 10; //maximum number of data that will be appended per request.
protected $db_record_limit = 5; // maximum number of record that we want to featch from data base.
protected $excel_dir = 'excel/'; //directory to store all the files
protected $file_type = 'csv';
public function getCheckExportQueue($exportType) {
$toExportList = Export::getExportListbyType($exportType, $this->db_record_limit);
    $file_name = $toExport['file_name'];
    if (empty($file_name)) {
        $file_name = 'report-' . date('YmdHis') . $toExport['id'] . '.' . $this->file_type;
        Excel::create($file_name, function($excel) {
            $excel->sheet('Sheet1', function($sheet) {
                //writting data into the file
                $sheet->fromArray(array(
                    'Teest Data', 'Test Data'
                ));
            });
        })->store($this->file_type, storage_path($this->excel_dir));
    }
    self::ExportAccount($ids_to_export, $file_name);
}

public function ExportAccount($ids_to_export, $file_name) {
    if (count($ids_to_export)):
        //dd(storage_path($this->excel_dir . $file_name)); C:\xampp\htdocs\pixel_fb\storage\excel/report-201608251333111.csv
        //dd($file_name); report-201608251333111.csv
        Excel::load(storage_path($this->excel_dir . $file_name), function($excel) {
            $excel->sheet('Sheet1', function($sheet) {
                $sheet->appendRow(array(
                    'appended', 'appended'
                ));
            });
        })->store($this->file_type, storage_path($this->excel_dir));
    endif;
}

}`
Package version, Laravel version
laravel/framework: 5.1
maatwebsite/excel: ~2.1.0
Screenshot of Excel file

I did't find any tutorial in you doc.

If you look closely you see it's trying to open from a wrong path C:\xampp\htdocs\pixel_fb\C:\xampp\htdocs\pixel_fb\storage\excel/report-201608251335041.csv
Seems $this-excel_dir already holds the storage_path.

Hi @patrickbrouwers .
Thanks for the input but I think the path is correct because
Excel::load(storage_path($this->excel_dir . $file_name), function($excel) { will return
Excel::load('C:\xampp\htdocs\pixel_fb\storage\excel/report-201608251335041.csv', function($excel) { which is a correct path.
Anyway I have achieved this by fputcsv PHP function and my code is something like this, and $data holds all my record. and it is woring fine for me.
$file = fopen(storage_path(self::$excel_dir) . $file_name, "a"); //Open the CSV file for appending foreach ($data as $value) {     fputcsv($file, explode("\r", implode("\r", $value))); } fclose($file);;
It will be very helpful for me and for other developer also if Maatwebsite developer can put a full example for appending rows into an excel sheet.


Fixes 275,272: Adds "public repositories" tab

Adds public repositories
Additional changes:

Make all modals fixed top center of browser.
Restructure Pages folder
Changed the notification on create repository action depending on whether the introspection data is available.
A note to QE: The above change resolves this jira ticket as a result.

https://issues.redhat.com/browse/HMSCONTENT-275

https://issues.redhat.com/browse/HMSCONTENT-275,272

https://issues.redhat.com/browse/HMSCONTENT-275,272

/retest