[Discussion] Spatial data with hc

Summary
Hey, as discussed before on slack with @PascalSenn and @michaelstaib, I'm kinda struggling with GeoJSON and hc right now.
My current project has to work with a lot of geo data, therefore I want it to be as comfortable as possible. The Geo data is stored in a PostgreSQL database using Postgis. For the interaction between my api and the database I'm using ef core with npgsql. They recommend using the NetTopologySuite to work with spatial data, see:
https://www.npgsql.org/efcore/mapping/nts.html
NetTopologySuite has a base type, called Geometry. It also has more specific types like Point and LineString which extend the Geometry class. The current standard for geo data transportation seems to be GeoJSON, which seems to work pretty well with NetTopologySuite. There even is an existing Json transformer for transforming between NetTopologySuite and GeoJSON: https://github.com/NetTopologySuite/NetTopologySuite.IO.GeoJSON
What's the best way to work with data like that using hc?
I tried different solutions (see attempts down below) and feel kinda stuck right now.
Slack discussion:

https://hotchocolategraphql.slack.com/archives/CD9TNKT8T/p1585903617176000

Related issues:

https://github.com/ChilliCream/hotchocolate/issues/1303
https://github.com/ChilliCream/hotchocolate/issues/1051

Attempt 1: Custom Scalar
Having read about the scalar implementation with hc from the docs I wanted to try that first.
I want to use the NetTopologySuite types like it's something that hc can handle. Being able to just have a Point for an argument type is quite nice. Something like this:
    [ExtendObjectType(Name = "Query")]
    public class PointQuery
    {
        public Point Hello(Point startPoint)
        {
            // do something with startPoint
            var p = new Point(startPoint.x, startPoint.y);

            // return a Point instance
            return p;
        }
    }

Scalars can be problematic for the tooling, as pointed out by @PascalSenn on slack, but they seem to be the most comfortable to use, since I can just bind them to the Point class (.BindClrType<Point, GeoScalar>()).
My implementation: https://github.com/John0x/hotchocolate-geojson-test
It's using the aforementioned JsonTransformer but it feels really hacky :/
The input works but I cannot output a json structure. I could probably just return it as a string, but that feels way too hacky.
Attempt 2: Input and output objects and manual conversion
It would be nice to use GeoJSON, since it's used by a lot of people but it's also okayish to not use it, if it's too inconvenient for GraphQL. The idea for this one would be to have properly typed input and output objects for each specific spatial type (like Point and LineString) and then do the conversion manually using extension methods or something else.
Like this:
    [ExtendObjectType(Name = "Query")]
    public class PointQuery
    {
        public Point Hello(PointInput startPoint)
        {
            // convert to real Point
            var pAsPoint = startPoint.toPoint();

            var p = new Point(pAsPoint.x, pAsPoint.y);

            // convert to PointOutput
            return p.toOutput();
        }
    }

This would probably work just fine and be the cleanest from the perspective of someone consuming the API, but it's more overhead for the development team.
Conclusion
I'm not sure 😄
It would be nice to have a mix between attempt 1 and 2. Being able to use BindClrType to bind something like a DTO to a CLR Type, which takes care of the conversion between the GraphQL representation and the real class.
Or is there already something like that and I just didn't see it?

Is there no proper solution to this right now or why did it get the HC-11.x.x tag? :)

The tag is just backlog tag. HC-11.0.0 => current work backlog, HC => HC-11.x.x => later

https://hasura.io/docs/1.0/graphql/manual/queries/query-filters.html#postgis-spatial-relationship-operators-st-contains-st-crosses-etc looks like a nice implementation with postgis

@John0x we have opened a discussion on slack #spatial

#1747

closing in favor of: https://github.com/ChilliCream/hotchocolate/issues/1878


Side loading compressed layer files into the cacheDir

What happened in your environment?
is side loading compressed layer files into the cacheDir supported?
I am looking into fetching the layer files out of band and saving them in the cacheDir as sha256:. Does overlaybd pickup the new files added externally to the directory after it has started? Or does it hydrate once and maintains the inventory internally until a restart?
What did you expect to happen?
I still see a few downloads happening - the logging wasn't verbose and didn't easily tell me which digest it was referring to - i will turn on debug logging and try again - but wanted to check if the above scenario is supposed to work.
How can we reproduce it?
copy the compressed layer files to the cacheDir after overlaybd has started - then start the container - expectation is overlaybd doesn't issue any blob download requests.
What is the version of your Overlaybd?
NA
What is your OS environment?
Ubuntu 18
Are you willing to submit PRs to fix it?

[ ] Yes, I am willing to fix it.

Sorry i mistagged the request. It is more of a question.

@simha-db Hi, thanks for asking!

@simha-db thanks for asking!

Is side loading compressed layer files into the cacheDir supported?
Yes, but it has limitations.
First, you should make sure the loaded files will not override existing files. Those files may have been opened by running containers. Overriding won't let overlaybd reopen these files. (downloading from source will happen)
Second, cacheDir size cacheConfig.cacheSizeGB should large enough to avoid loaded files triggering cache gc, which is unexpected behavior.
So, it is highly recommanded that you stop overlaybd first, then load files to cache, and restart overlaybd.
Why downloads happening ?
I think it's because background download is enable in overlaybd config (download.enable). Caching and background download are seprate operations, and background download files are not stored in cache dir. If you are trying to load files into cache dir, background download can be disabled.

Ah ok. When is background download triggered?If there is no prefetch involved - do any of the ondemand loads go through background download?
Some what off topic: If i use the ctr rpull --download-blobs - does that go through the cache?

@simha-db background download starts 600s after container started by default, which can be set by download.delay.
Background download is performed in a seprate thread, and on-demand load won't go through it, neither will prefetch.
ctr rpull --download-blobs download blobs to snapshotter dir, will not go through cache.

Thanks - And the background downloader fetches the full image to the snapshot directory?

@simha-db Yes, exactly.
But when blobs have been downloaded by ctr rpull --download-blobs, container will start from local files, and background download will not be triggered.

Do all registry requests get logged in overlaybd-audit.log?
One thing i notice is - if i prepull the entire image into registry_cache - pull image request to image service - which should be doing only rpull (without the download) takes much longer. When an image file is opened - if the layer files are in the registry_cache does the snapshotter do additional work to pull them to the snapshot directory or something of that sort?

@simha-db all overlaybd registry requests (after rpull) will be logged in overlaybd-audit.log (background download audit is added in v0.6.9 #222).
If layers have been in registry_cache, container open will not request data from registry. Snapshotter won't pull layers to snapshott directory after container started, but background download will. Please make sure background download feature is disabled before your test.
Prepull should not affect time of rpull (without download), could you please show me some data ?

Turns out the time is the same - i incorrectly attributed it to an increase.

I think i have some data on the slowness - maybe it is exposed when we upgraded our overlaybd - but here's what i observe.

My side loading the layer files finishes in 4 seconds.
Then we call CRI pull image and i see in the containerd logs the following

Jul 25 07:01:25 ip-10-1-166-126 containerd[936]: time="2023-07-25T07:01:25.841633618Z" level=info msg="PullImage \”<removed>@sha256:a65d68704b22c9752c023cec0a2ef0868d31623574dabaf9949fcf525645b32e\""
Jul 25 07:01:25 ip-10-1-166-126 containerd[936]: time="2023-07-25T07:01:25.841757128Z" level=info msg="experimental: PullImage \”<removed>@sha256:a65d68704b22c9752c023cec0a2ef0868d31623574dabaf9949fcf525645b32e\" for runtime runc-overlaybd, using snapshotter overlaybd"
**Jul 25 07:02:10** ip-10-1-166-126 containerd[936]: time="2023-07-25T07:02:10.703589621Z" level=info msg="ImageCreate event &ImageCreate{Name:<removed>@sha256:a65d68704b22c9752c023cec0a2ef0868d31623574dabaf9949fcf525645b32e,Labels:map[string]string{io.cri-containerd.image: managed,},XXX_unrecognized:[],}"

the CRI pull image took close to 45 seconds.

I look in overlaybd-auditlog - indeed it is empty - no pulls were made from network

2023/07/25 07:01:47|AUDIT|th=00007F08D4F5F300|file:read[pathname=][offset=0][size=512][latency=34292]
2023/07/25 07:02:26|AUDIT|th=00007F0743D35E80|file:read[pathname=][offset=108004587][size=2922][latency=29157]
2023/07/25 07:02:26|AUDIT|th=00007F07D35F7E80|file:read[pathname=][offset=1412525637][size=30698][latency=43467]
2023/07/25 07:02:26|AUDIT|th=00007F07D0DD7F40|file:read[pathname=][offset=1412556335][size=63151][latency=43467]
2023/07/25 07:02:27|AUDIT|th=00007F07D0DD7F40|file:read[pathname=][offset=1396719871][size=859][latency=12106]
2023/07/25 07:02:27|AUDIT|th=00007F0743D35E80|file:read[pathname=][offset=107999319][size=2456][latency=20176]
2023/07/25 07:02:27|AUDIT|th=00007F06E6BC7300|file:read[pathname=][offset=512][size=512][latency=19326]
2023/07/25 07:02:36|AUDIT|th=00007F06E4BAF2C0|file:read[pathname=][offset=213191565][size=45214][latency=12454]

When i look in the overlaybd-snapshotter service logs i see the following - the last entry in the snapshotter - roughly matches the time when the image reference was returned to containerd. Seems like the snapshotter is doing something synchronously if it finds the cache files. When we don't side load - we don't see this delay. What is the snapshotter doing here? is it validating the files synchronously?

Jul 25 07:01:33 ip-10-1-166-126 overlaybd-snapshotter[1543]: time="2023-07-25T07:01:33Z" level=info msg="{\n  \"repoBlobUrl\": \”<removed>/blobs\",\n  \"lowers\": [\n    {\n      \"digest\": \"sha256:0d5554fe6fff2aa24f4065881f061f4c1b03749341e119a93174cc0858f0ed57\",\n      \"size\": 47891968,\n      \"dir\": \"/var/lib/overlaybd/snapshots/29/fs\"\n    }\n  ],\n  \"upper\": {},\n  \"resultFile\": \"/var/lib/overlaybd/snapshots/29/block/init-debug.log\"\n}"

**Jul 25 07:02:10** ip-10-1-166-126 overlaybd-snapshotter[1543]: time="2023-07-25T07:02:10Z" level=info msg="{\n  \"repoBlobUrl\": \”<removed>/blobs\",\n  \"lowers\": [\n    {\n      \"digest\": \"sha256:0d5554fe6fff2aa24f4065881f061f4c1b03749341e119a93174cc0858f0ed57\",\n      \"size\": 47891968,\n      \"dir\": \"/var/lib/overlaybd/snapshots/29/fs\"\n    },\n    {\n      \"digest\": \"sha256:82d331f6b157c3ee944d1064c55fa41f20024c34b2d7138598cec6942bc6bead\",\n      \"size\": 6144,\n      \"dir\": \"/var/lib/overlaybd/snapshots/32/fs\"\n    },\n    {\n      \"digest\": \"sha256:187a69a0ef15042fa944bc05217cf116ffc1de923bf1bbaa21241bf471324a55\",\n      \"size\": 7680,\n      \"dir\": \"/var/lib/overlaybd/snapshots/35/fs\"\n    },\n    {\n      \"digest\": \"sha256:2d0b9a2eff4cb17c414ca3cee0eb48d9e000ffd9661994997b08baa3a837d742\",\n      \"size\":<PHONE_NUMBER>,\n      \"dir\":

Something doesn't add up - when all this activity is happening in overlaybd snapshotter - like this - containerd is still waiting for the image reference - there is no lazy load happening - what is all this activity in snapshotter? It seems to touch all the layer files one by one -does it setup the snapshot directory for all the registry cache layer files before the image reference is created?
Jul 25 07:01:33 ip-10-1-166-126 overlaybd-snapshotter[1543]: time="2023-07-25T07:01:33Z" level=info msg="{\n  \"repoBlobUrl\": \”<removed>/blobs\",\n  \"lowers\": [\n    {\n      \"digest\": \"sha256:0d5554fe6fff2aa24f4065881f061f4c1b03749341e119a93174cc0858f0ed57\",\n      \"size\": 47891968,\n      \"dir\": \"/var/lib/overlaybd/snapshots/29/fs\"\n    }\n  ],\n  \"upper\": {},\n  \"resultFile\": \"/var/lib/overlaybd/snapshots/29/block/init-debug.log\"\n}"

overlaybd-snapshotter indeed prepare snapshot directory for eache image layer. snapshot directory contains some meta info for overlaybd-tcmu.
when rpull containerd calls overlaybd-snapshotter to prepare snapshot for each image layer (ro layers).

@simha-db overlaybd-snapshotter indeed prepare snapshot directory for each image layer. snapshot directory contains some meta info for overlaybd-tcmu.
when rpull, containerd calls overlaybd-snapshotter to prepare snapshot for each image layer (ro layers).
when run, containerd calls overlaybd-snapshotter to prepare rootfs mountpoint (rw layer), which will call overlaybd-tcmu to setup block device based on those layers.


[elfutils] add new port

new port; required for mesa on linux

/azp run

elfutils:x64-osx failed in git fetch steps, this issue could repro on local machine, @Neumann-A , could you help have a look?
git-fetch-x64-osx-err.log
error: Server does not allow request for unadvertised object b503c358dde835d8a1ae3ebd4968755ff396f814

-- Fetching https://sourceware.org/git/elfutils...
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:106 (message):
    Command failed: /usr/bin/git fetch https://sourceware.org/git/elfutils b503c358dde835d8a1ae3ebd4968755ff396f814 --depth 1 -n
    Working Directory: /Users/vagrant/Data/downloads/git-tmp
    Error code: 128
    See logs for more information:
      /Users/vagrant/Data/buildtrees/elfutils/git-fetch-x64-osx-err.log

Call Stack (most recent call first):
  scripts/cmake/vcpkg_from_git.cmake:78 (vcpkg_execute_required_process)
  ports/elfutils/portfile.cmake:1 (vcpkg_from_git)
  scripts/ports.cmake:136 (include)

configure: error: gcc with GNU99 support required

which version of gcc? Or what is the output of the config.log in the subdir <triplet>-dbg/?

Here is config log:
failure logs for x64-osx (1).zip

@PhoebeHui: That are not the logs I need. I need the literally called config.log from <vcpkgroot>/buildtrees/<port>/<triplet>-<shortbuildtype>/config.log Unfortunally vcpkg_configure_make is not able to copy that file into buildtrees/<port> since vcpkg_execute_process does error before I can inject the code for copying that file one folder down.

Sorry, here is correct one:
config.log

The error is:
configure:5165: gcc -c -fPIC -g -std=gnu99  conftest.c >&5
conftest.c:20:28: error: function definition is not allowed here
  double square (double z) { return z * z; }
                           ^
conftest.c:21:10: error: implicit declaration of function 'square' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
  return square (a) + square (b);
         ^
conftest.c:26:18: error: fields must have a constant size: 'variable length array in structure' extension will never be supported
  struct S { int x[n]; };
                 ^
3 errors generated.
configure:5165: $? = 1
configure: failed program was:
| /* confdefs.h */
| #define PACKAGE_NAME "elfutils"
| #define PACKAGE_TARNAME "elfutils"
| #define PACKAGE_VERSION "0.182"
| #define PACKAGE_STRING "elfutils 0.182"
| #define PACKAGE_BUGREPORT "https://sourceware.org/bugzilla"
| #define PACKAGE_URL "http://elfutils.org/"
| #define PACKAGE "elfutils"
| #define VERSION "0.182"
| #define DEFAULT_AR_DETERMINISTIC false
| #define YYTEXT_POINTER 1
| /* end confdefs.h.  */
| int foo (int a)
| {
|   for (int i = 0; i < a; ++i) if (i % 4) break; int s = a; return s;
| }
|
| double bar (double a, double b)
| {
|   double square (double z) { return z * z; }
|   return square (a) + square (b);
| }
|
| void baz (int n)
| {
|   struct S { int x[n]; };
| }
configure:5173: result: no
configure:5176: error: gcc with GNU99 support required

which basically means that the compiler:
configure:3750: gcc --version >&5
Apple clang version 12.0.0 (clang-1<IP_ADDRESS>)
Target: x86_64-apple-darwin19.6.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin
Configured with: --prefix=/Library/Developer/CommandLineTools/usr --with-gxx-include-dir=/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include/c++/4.2.1

does not support gnu extensions (-std=gnu99). As such I should probably put it on the CI skiplist on OSX? If a true gcc would be used on OSX it would probably compile.

Could you please add it to 'Supports" instead?

@PhoebeHui: Supports seems wrong here. A custom toolchain switching to gcc on apple could compile this correctly.

@Neumann-A, thanks for the information!
I'm fine to put it on the CI skiplist on OSX, could add comments as well in ci.baseline.txt file?

Thanks @Neumann-A :)


Select statement all columns

In PersonGateway.php, statement at line 34 is:
SELECT
    id, firstname, lastname, firstparent_id, secondparent_id
FROM
    person
WHERE id = ?;

You are selecting all the columns, why not using * ?

Selecting specific columns is more future-proof, since later if you add a new column that isn't needed by this code it would waste bandwidth transferring that data from the database.


"drawable://" + R.drawable.img According to the wrong picture

Used in the gridView "drawable://" + R.drawable.img_addpic)  It should be + pictures but difference me a child lock
should：

display：

No such a picture in my project

Intersting. I have no clue why it happens. Somethings wrong with res ID generating, maybe.

Had same issue. Loading images from drawable:// gets wrong image every once in a while. Every time i get that problem  i have to reinstall app and it will get fixed. I have no idea how to solve it permanently.

Agree with the issue. I am having the same issue. This must be bug.
In my case, I am using viewpager to display a series of images to act as rotating banner. After a while of displaying, the two different images show the same image. Please fix it. Thanks.

Had same issue.Loading disk images should not diskcache anytime.is it work?


⚠ Secret Website 3 has degraded performance

In cff4729, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 15355 ms

Resolved: Secret Website 3 performance has improved in e785de7.


Support IDE build 233.x

Hello there,
New IDEA major version is coming, but the plugin is not yet supporting it.
Can you please release a version that supports the 233.x.x builds?
Thank you.

I've just read this page: https://plugins.jetbrains.com/docs/intellij/build-number-ranges.html

Not specifying until-build means it will include all future builds. This includes future, yet unreleased versions and possibly new IDEs, which might impact compatibility later.

Would it make sense to avoid using that until-build attribute to make sure future releases are supported automatically?

0.24.4 released with IntelliJ IDEA 2023.3 compatible.

Thank you so much @linux-china !


Run API and Ingestion Server tests with all relevant ES versions

Fixes

Based on https://github.com/WordPress/openverse/pull/2777 and related to the comment here: https://github.com/WordPress/openverse/pull/2777#pullrequestreview-1576169306
Description

Because production is currently stuck on ES 7 until we've figured out why ES 8 caused the queries to take so long, we need to test all our code against both ES versions. Until we've fully transitioned to ES 8 we need to support both versions.
Testing Instructions

Check the new CI tasks and confirm they've run as expected. Locally, run OPENVERSE_ES_VER=7 just api/test locally to test as an example that the environment variable works as expected.
Checklist

[x] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[x] I added or updated tests for the changes I made (if applicable).
[N/A] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.
[N/A] I ran the DAG documentation generator (if applicable).

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

This will also need the required checks changed in the infra repo with the new names. I'll push a commit with changes to ingestion and api so that those steps actually run.

The most recent CI run confirms that this runs (and that the changes in #2777 should be okay (I think):
https://github.com/WordPress/openverse/actions/runs/5862236561/job/15893876650?pr=2832#step:5:19
Notice that it pulls ES version 7. And here it pulls version 8:
https://github.com/WordPress/openverse/actions/runs/5862236561/job/15893876557?pr=2832#step:5:19


Make the jest mock typed

Describe the Feature
The jest mock is in javascript so you get warnings in VSCode that it is not typed.
Possible Implementations
I have implemented my mock in __mocks __ as follows
import type NetInfo from '@react-native-community/netinfo';
import type {
  NetInfoConfiguration,
  NetInfoState
} from '@react-native-community/netinfo';

export enum NetInfoStateType {
  unknown = 'unknown',
  none = 'none',
  cellular = 'cellular',
  wifi = 'wifi',
  bluetooth = 'bluetooth',
  ethernet = 'ethernet',
  wimax = 'wimax',
  vpn = 'vpn',
  other = 'other',
}

const connectedState: NetInfoState = {
  type: NetInfoStateType.wifi,
  isConnected: true,
  isInternetReachable: true,
  details: {
    ssid: null,
    bssid: null,
    strength: null,
    ipAddress: null,
    subnet: null,
    frequency: null,
    isConnectionExpensive: false,
    linkSpeed: null,
    rxLinkSpeed: null,
    txLinkSpeed: null,
  },
};

export default <typeof NetInfo>{
  fetch: jest.fn(() => Promise.resolve(connectedState)),
  refresh: jest.fn(() => Promise.resolve(connectedState)),
  configure: jest.fn((_configuration: Partial<NetInfoConfiguration>) => {}),
  addEventListener: jest.fn(() => jest.fn()),
  useNetInfo: jest.fn(() => connectedState),
};

Related Issues

Hi there, we merge reasonable PRs.

We support javascript, and people doing pure javascript will not be able to handle the ts file. Not sure how best to accomplish it. Perhaps a jest ts file in parallel to the javascript one, and a note in both indicating that the other file exists and you should update both at once? Just brainstorming

I generally have my code in typescript then have the tooling build the JS files for me (e.g https://github.com/trajano/react-hooks) .  But that may be a major tooling change.

Maybe have it so I name the file with a -typescript.ts suffix.  That way people can explicitly pull it in.

that could work. Just having it end in ts should be sufficient. Anything else would be a tooling change though, requiring pre-builds here or something. That's undesired

I'm trying to see if someone can answer https://stackoverflow.com/questions/74828607/how-do-i-add-extra-functions-to-a-mock-and-have-it-pass-typescript so we can add some extra functions to at least do some netinfostate changes for testing.  But I'll remove the code for now.
The mock
<EMAIL_ADDRESS>The test
https://github.com/trajano/spring-cloud-demo/blob/dbcfbc1be7e75d2cc09c8dce08fd976c07874d03/packages/auth-context/src/__tests__/netinfoMock.test.tsx


Missing device_attributes.pb.h when make tensorflow to .a file

I want make tensorflow to a static link file so I can use it to run the pretrained model with the cpp API.After I run the build_all_linux.sh, I move the tensorflow and third_party file and the libtensorflow-core.a and libprotobuf.a and libnsync.a to a work path.
And when I compile the cpp code, it returns an error which said

fatal error: tensorflow/core/framework/device_attributes.pb.h: No such file or directory

And it occurs in the session.h. I find there is only device_attributes.proto in the path,but I dont know why because I just followed the ways of many blogs online.Is there any thing that I missed?
Any help is appreciate!Thx!

This question is better asked on StackOverflow since it is not a bug or feature request. There is also a larger community that reads questions there.

Now,I found all the xxx.pb.h file in this path

tensorflow/contrib/makefile/gen/host_obj/tensorflow/core/framework

But why? As the example said, you need to #include "tensorflow/core/public/session.h" and the session.h file include "tensorflow/core/framework/device_attributes.pb.h". So,the generate file from protoc should be in the path which is right to the session.h,right?
Need I check all the files in the gen?

It looks like you haven't used a template to create this issue. Please resubmit your issue using a template from here. We ask users to use the template because it reduces overall time to resolve a new issue by avoiding extra communication to get to the root of the issue. We will close this issue in lieu of the new one you will create from the template. Thank you for your cooperation.

hi, @SefaZeng , I got the same error, have you solved the problem?

hi, @SefaZeng , I got the same error, have you solved the problem?

你好像是个中国人我就打中文了.. 我其实记不太清这里是咋回事 好像是proto生成的一些文件在我写的那个路径下面 在cmakelist里面把这些路径加进去好像就可以


added release scripts

Coverage remained the same at 71.073% when pulling a1a42015b486f1c0b01ddf3781b5c378d9b430b4 on akmorrow13:release-scripts into 4f8fb61cabbdc0c7361c5948df9b01c0b6044c92 on bigdatagenomics:master.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/mango-prb/639/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/mango-prb/640/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/mango-prb/641/
Test PASSed.


Details of wxSTC_LEX_TCS

In
...\Torsion\code\Torsion\ScriptCtrl.cpp (1 hit)
Line 301:    SetLexer(wxSTC_LEX_TCS);
I think this is "Torque C Script" special style enums, but I can't find more info in the sourcecode.
There is more TCS-s at Line 907:
if (  style == wxSTC_TCS_COMMENT ||
style == wxSTC_TCS_COMMENTLINE ||
style == wxSTC_TCS_COMMENTDOC ||
style == wxSTC_TCS_STRING ||
style == wxSTC_TCS_STRINGEOL )
Is there any info or suggestion?
Best Regards

I think this is "Torque C Script" special style enums

That is part of the parser support for TorqueScript that I added to the Scintilla text editor control we use on Torsion.
The code for the Scintilla text editor control is in:
https://github.com/SickheadGames/Torsion/tree/master/code/wxWidgets/contrib/src/stc


Update Istio to 1.21.1

Description
Changes proposed in this pull request:

Update Istio to 1.21.1

/hold needs istio go dips

/unhold


What is docker -slides

Take the original slides and revamp them:

Make it more detailed

Remember to look at the old slides and build from there,


feature-2140 org.springframework.cloud.gateway.config.HttpClientProperties#proxy consider specifying the proxy type?

See https://github.com/spring-cloud/spring-cloud-gateway/issues/2140
Hello, is org.springframework.cloud.gateway.config.HttpClientProperties#proxy considering specifying the proxy type? Currently, the code is fixed ProxyProvider.Proxy.HTTP, if configuration injection can be used to enable the gateway to support more complex network environments.
By adding the type field in org.springframework.cloud.gateway.config.HttpClientProperties.Proxy, it can be used in org.springframework.cloud.gateway.config.GatewayAutoConfiguration.

@zhangrenhua Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@zhangrenhua Thank you for signing the Contributor License Agreement!

Will this cause some unknown problems?


Added column for share support in TWA

See https://developer.chrome.com/docs/android/trusted-web-activity/web-share-target/

Thanks for this change - I'll be happy to merge it, but you'll need to sign the CLA.


Homepage and search updates

Updated numbers on homepage
Small CSS tweaks on search page

Also renamed "edit source" to "select source"

Mind doing a rollup with this change if this looks good?


Crash to menu after deconstructing Infinity Accumulator

I tested a design, wanted to deconstruct it and the game crashed and sent me back to the main menu.

I then loaded an autosave and tried to remove the accumulator by hand, crashed again.

I shot at it with a nuke, crashed.

Tried it with the accumulator being connected and disconnected from any power network; no difference.
I opened and closed the menu of it, tried to deconstruct it again and it worked fine. huh...

Hmm, this shouldn't be possible, I have no idea why that would be happening. If it hasn't happened again I will just leave it.


[Tutorial proposal] Your tutorial title here

What is the objective of the proposed tutorial?
A clear and concise description of what users can expect to learn from your
tutorial.
What is the scope of the proposed tutorial?
The thematic (forestry, water resources, land cover classification, Earth Engine
core libraries, etc) and specific geographic area, if relevant.
Please provide an outline of the structure of the proposed tutorial?

Section 1
a. Subsection 1a
b. Subsection 1b
Section 2
.. et cetera. ..

In what format will you be submitting the tutorial?
Select one: Markdown
or Colab
This request will be reviewed by the Earth Engine community maintainers, who
will reply on this issue tracker with any questions or suggestions. Once
approved, this issue will be assigned to you and you can begin work on the
tutorial following instructions in
Writing a tutorial.
When creating your Pull Request, enter "Closes #issueno" in the description of

[ ] your Pull@__ Request to link the tutorial to this issue.

It appears you've submitted an empty proposal. Please fill in all fields and resubmit your request. Thanks!


[cp][stable] Cherry-pick fix to secure socket connection unhandled exception

Please cherry-pick https://dart.googlesource.com/sdk/+/ea3dfc9ac4617ad0346b515596636a1089ecc8a1 into stable.
This fixes "Bad state: Future already completed" unhandled exception that arises in cases when https connection attempt errors out or is aborted shortly after it started, before name resolution had a chance to complete. Original issue https://github.com/dart-lang/sdk/issues/45047.
The commit merges cleanly into stable.
/cc @mraleph @a-siva @franklinyow @vsmenon

Note for CP approvers: the reason for requesting a cherry pick is because this can cause sporadic crashes on pub (most likely because pub speculatively fans out multiple concurrent requests and sometimes cancels them). Sporadic crashes when running pub is bad user experience.

lgtm

Hot fix 2.12.1


ctags now uses language in singular rather than languages in plural

because of using the old option in https://github.com/tpope/gem-ctags/blob/f947ed4e1f093e48d2305419475d0c9796e354b0/lib/rubygems/commands/ctags_command.rb#L22
installing gems often times shows this message

Is that Exuberant ctags or something else?

Aha, so Emacs overrode ctags with its own and it uses --language only
Thanks


Fix #52

The property "type" into the IItem interface has been annotated using the JSONProperty attribute and this way, the issue found in the compose method was solved.
A new example using multiple types into the same container was created to test it, it is a bit hard unit test this kind of issue.

Hi @jrwmacedo - thanks for this PR. I think we should change this a bit and expose the Cosmos DB serialization options to consumers so they can control this. With this change, we could make the default options be case insensitive - thus solving our problem here too. But if we were to decorate this, it's set that way forever.
https://docs.microsoft.com/dotnet/api/microsoft.azure.cosmos.cosmosclientoptions.serializer
What do you think about that?

I exposed lots of settings now to consumers, see: 6b0e1b0803945150b6e585f823fb8e7bc95a932a and 64271f6c44af6be6170073b663c09e209b38db2d. We might be able to close this one now, thoughts?

I exposed lots of settings now to consumers, see: 6b0e1b0 and 64271f6. We might be able to close this one now, thoughts?

Hey @IEvangelist , Thanks for that man, I agree with your suggestion make more sense. I would change that later but as you already did, I appreciate it.

Should we close this PR, or do you want to update it - pull the latest bits and see if it works without your JsonProperty attr?

Should we close this PR, or do you want to update it - pull the latest bits and see if it works without your JsonProperty attr?

I will do that...Give me a couple of hours and I will let you know if it is working properly for me.

Should we close this PR, or do you want to update it - pull the latest bits and see if it works without your JsonProperty attr?

I will do that...Give me a couple of hours and I will let you know if it is working properly for me.

Thank you so much! One test scenario would be to create some objects with the https://github.com/IEvangelist/azure-cosmos-dotnet-repository/blob/main/Microsoft.Azure.CosmosRepository/src/Options/RepositorySerializationOptions.cs#L31 set to Default, this would save them with non-camel case. Then change your settings, to use CamelCase and see if there is a loss of data.

Should we close this PR, or do you want to update it - pull the latest bits and see if it works without your JsonProperty attr?

I will do that...Give me a couple of hours and I will let you know if it is working properly for me.

Thank you so much! One test scenario would be to create some objects with the https://github.com/IEvangelist/azure-cosmos-dotnet-repository/blob/main/Microsoft.Azure.CosmosRepository/src/Options/RepositorySerializationOptions.cs#L31 set to Default, this would save them with non-camel case. Then change your settings, to use CamelCase and see if there is a loss of data.

Hi again @IEvangelist ,
After doing some tests, I faced an odd situation here...
The CosmosClient is always assuming the serialization as CamelCase and it is ignoring our configuration setting.
I tried to figure out what is going on but I couldn't at the moment. I thought the configuration binder was not working well or something related to the lazy loading but it is weird because the SerializerConfiguration is applied accordingly based on the debug values.
The changes solved the type issue because now is always assuming the CamelCase as default but in another hand, we have the setting ignored if someone need to change that manually as you proposed.
I tried to record this gif to expose and clarify the situation.
https://drive.google.com/file/d/1uAGCdC94XSaeu9nmZddcvJQE6v9tqUBX/view?usp=sharing
Any thoughts ?
Cheers!

Should we close this PR, or do you want to update it - pull the latest bits and see if it works without your JsonProperty attr?

I will do that...Give me a couple of hours and I will let you know if it is working properly for me.

Thank you so much! One test scenario would be to create some objects with the https://github.com/IEvangelist/azure-cosmos-dotnet-repository/blob/main/Microsoft.Azure.CosmosRepository/src/Options/RepositorySerializationOptions.cs#L31 set to Default, this would save them with non-camel case. Then change your settings, to use CamelCase and see if there is a loss of data.

Hi again @IEvangelist ,
After doing some tests, I faced an odd situation here...
The CosmosClient is always assuming the serialization as CamelCase and it is ignoring our configuration setting.
I tried to figure out what is going on but I couldn't at the moment. I thought the configuration binder was not working well or something related to the lazy loading but it is weird because the SerializerConfiguration is applied accordingly based on the debug values.
The changes solved the type issue because now is always assuming the CamelCase as default but in another hand, we have the setting ignored if someone need to change that manually as you proposed.
I tried to record this gif to expose and clarify the situation.
https://drive.google.com/file/d/1uAGCdC94XSaeu9nmZddcvJQE6v9tqUBX/view?usp=sharing
Any thoughts ?
Cheers!

Yes, now I'm curious - did you remove the [JsonProperty("type")] attribute decorator on the property before you compiled and tested? That might be the reason the "type" name is used?

Should we close this PR, or do you want to update it - pull the latest bits and see if it works without your JsonProperty attr?

I will do that...Give me a couple of hours and I will let you know if it is working properly for me.

Thank you so much! One test scenario would be to create some objects with the https://github.com/IEvangelist/azure-cosmos-dotnet-repository/blob/main/Microsoft.Azure.CosmosRepository/src/Options/RepositorySerializationOptions.cs#L31 set to Default, this would save them with non-camel case. Then change your settings, to use CamelCase and see if there is a loss of data.

Hi again @IEvangelist ,
After doing some tests, I faced an odd situation here...
The CosmosClient is always assuming the serialization as CamelCase and it is ignoring our configuration setting.
I tried to figure out what is going on but I couldn't at the moment. I thought the configuration binder was not working well or something related to the lazy loading but it is weird because the SerializerConfiguration is applied accordingly based on the debug values.
The changes solved the type issue because now is always assuming the CamelCase as default but in another hand, we have the setting ignored if someone need to change that manually as you proposed.
I tried to record this gif to expose and clarify the situation.
https://drive.google.com/file/d/1uAGCdC94XSaeu9nmZddcvJQE6v9tqUBX/view?usp=sharing
Any thoughts ?
Cheers!

Yes, now I'm curious - did you remove the [JsonProperty("type")] attribute decorator on the property before you compiled and tested? That might be the reason the "type" name is used?

Morning @IEvangelist , my bad here while I was testing another scenario I reset my branch and I forgot to remove the JSON property (again). It was 2:00 AM (Lisbon time) when I was testing that man! LoL.
The scenario that I was testing is:
Using the configuration file (appSettings.json) is perfectly fine but if we set the configuration directly in the code like below the option is ignored because the config takes precedence.
    services.AddCosmosRepository(option =>             {                 option.DatabaseId = "samples";                 option.ContainerId = "container";                 option.SerializationOptions = new RepositorySerializationOptions                 {                     PropertyNamingPolicy = CosmosPropertyNamingPolicy.Default                 };             });
Anyway, using the appSettings to handle that is enough to change the default behaviour if necessary, the issue below can be handle later in my opinion.
I am happy to close this PR if you are as well.

@all-contributors please add @jrwmacedo for code and ideas


Optimize Lighting on Bronco Day & Night

I failed to consider performance in my optimization and will need to work on optimizing the lighting to perform better across all systems.

The UV light maps were not properly unwrapped on the stadium stands so the lighting could not bake as static. I unwrapped the UV's and changed all the stands to static lighting. On the night map, I lowered the volumetric fog distance by half. Spent about 20 minutes making these changes


Stylesheets and transforms for journals and publishers

This currently applies to HTML but may also apply to PDF and TXT transformations.
The HTML translation ( https://github.com/ContentMine/norma/blob/master/docs/TRANSFORM.md ) involves 2 or more stylesheets. Where do we put them and how do we organize it? Should there be a Github repo for publishers, separate from Norma (My current thinking is yes).
Currently we have (for publisher fooPub)
src/main/resources/org/xml-cml/norma/pubstyle/fooPub/

which contains toHtml.xsl, etc.
This means we have to rebuild the system for each alteration. If we use stylesheets this can be done independently from norma. It would be roughly similar to dictionary.
NOTE: we might want to lump together quickscrape scrapers and norma stylesheets for each journal.

The quickscrape scrapers are currently in http://github.com/ContentMine/journal-scrapers
This may not be quite the right place for them. Probably we should do like we do with dictionaries and have yet another repository for them. Perhaps we should include both the XSLTs and the dictionaries in the repositories that use them (norma and ami) with a git subtree / submodule. I'm currently reading about which is probably a better option for us.

Great.


Document $*KERNEL.hostname

From https://github.com/rakudo/rakudo/pull/2065
Implemented in https://github.com/rakudo/rakudo/pull/2396
Still needs spec before documenting.

Documented in https://github.com/perl6/doc/commit/31c64c2e2bc11bdd299ea2af8659f858b2602a01


Add travis support

Add travis support error
What version of Hugo are you using (hugo version)?

$ hugo version
Hugo Static Site Generator v0.68.3/extended linux/amd64 BuildDate: unknown

.travis.yml

language: go
install:
- go get -v github.com/spf13/hugo
script:
- hugo version
- make build
branches:
 only:
 - master

The command "go get -v github.com/spf13/hugo" failed and exited with 2 during .

The command "go get -v github.com/spf13/hugo" failed and exited with 2 during .
The command "go get -v github.com/gohugoio/hugo" failed and exited with 2 during .
The command "go get -v https://github.com/gohugoio/hugo" failed and exited with 1 during .

First, this is not a Hugo problem and should have been posted to the forums instead.
Second, it would probably be better to download the latest binary release in your CI instead of building Hugo from source.
But lastly, if you insist...  The default Go version in Travis CI is too old to build Hugo.  I would recommend telling Travis to use the latest Go version.
Any further questions should be posted to the forums.  Thanks

Thanks for info


Custom operator

Description of the bug:
Sometimes operators cannot be converted. Is there any way to implement a custom operator?
Actual vs expected behavior:
No response
Any other information you'd like to share?
No response

I don't currently see a way with this workflow, but let's see.


Add OrderSettingsSection transformer

Codecov Report

Merging #102 (13c253d) into main (e052f15) will decrease coverage by 0.01%.
The diff coverage is 96.39%.

@@            Coverage Diff             @@
##             main     #102      +/-   ##
==========================================
- Coverage   97.00%   96.99%   -0.02%
==========================================
  Files          22       24       +2
  Lines        1035     1163     +128
==========================================
+ Hits         1004     1128     +124
- Misses         31       35       +4

Impacted Files
Coverage Δ

robotidy/transformers/__init__.py
100.00% <ø> (ø)

robotidy/transformers/OrderSettingsSection.py
96.19% <96.19%> (ø)

robotidy/transformers/MergeAndOrderSections.py
100.00% <100.00%> (ø)

robotidy/transformers/OrderSettings.py
100.00% <100.00%> (ø)

robotidy/api.py
94.73% <0.00%> (ø)

robotidy/cli.py
94.95% <0.00%> (+1.12%)
:arrow_up:

robotidy/app.py
90.36% <0.00%> (+2.36%)
:arrow_up:

Continue to review full report at Codecov.

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


Installer for Win 8.1 does not add atom nor apm to PATH

I've made two installations of atom on fresh Windows installations. I used the Windows installer from atom.io. For Windows 7 I got both atom and apm in the PATH so I could call the directly from cmd.exe. But on Windows 8.1 they were not in the PATH.
I expected the same behavior on Windows 7 and Windows 8.1.
Is this the intended behavior? Or am I missing something?
How should I do to get atom and apm in the PATH for Windows 8.1?
On Windows 8.1
C:\Users\Mikael>atom
'atom' is not recognized as an internal or external command,
operable program or batch file.

C:\Users\Mikael>apm
'apm' is not recognized as an internal or external command,
operable program or batch file.

C:\Users\Mikael>PATH
PATH=C:\Windows\system32;C:\Windows;C:\Windows\System32\Wbem;C:\Windows\System32\WindowsPowerShell\v1.0\

On Windows 7
C:\Users\mos>where atom
C:\Users\mos\AppData\Local\atom\bin\atom
C:\Users\mos\AppData\Local\atom\bin\atom.cmd

C:\Users\mos>where apm
C:\Users\mos\AppData\Local\atom\bin\apm
C:\Users\mos\AppData\Local\atom\bin\apm.cmd

C:\Users\mos>PATH
PATH=C:\Windows\system32;C:\Windows;C:\Windows\System32\Wbem;C:\Windows\System32\WindowsPowerShell\v1.0\;C:\Program Files (x86)\Git\cmd;C:\Users\mos\AppData\Local\atom\bin

Hm.. I didn't have it added to path on Windows 7 64bit.

Odd, I've just done a fresh install on Win 8.1 and the PATH variable was updated normally.

Guess I'll just add that I've installed it on two Windows 8 PCs very recently (0.192.0) and amp and atom have both worked from the command prompt.

I did a fresh install on Windows 8 and it worked as expected, atom and apm in PATH.
C:\Users\Mikael>PATH
PATH=C:\Windows\system32;C:\Windows;C:\Windows\System32\Wbem;C:\Windows\System32
\WindowsPowerShell\v1.0\;C:\Users\Mikael\AppData\Local\atom\bin

C:\Users\Mikael>where apm
C:\Users\Mikael\AppData\Local\atom\bin\apm
C:\Users\Mikael\AppData\Local\atom\bin\apm.cmd

C:\Users\Mikael>PATH
PATH=C:\Windows\system32;C:\Windows;C:\Windows\System32\Wbem;C:\Windows\System32
\WindowsPowerShell\v1.0\;C:\Users\Mikael\AppData\Local\atom\bin

C:\Users\Mikael>where apm
C:\Users\Mikael\AppData\Local\atom\bin\apm
C:\Users\Mikael\AppData\Local\atom\bin\apm.cmd

On the failing Windows 8.1, I looked in the the file  C:\Users\Mikael\AppData\Local\atom\SquirrelSetup.log (assuming its a logfile for the installation program)
Program: Starting Squirrel Updater: --createShortcut atom.exe
ApplyReleasesImpl: About to create shortcuts for atom.exe, rootAppDir C:\Users\Mikael\AppData\Local\atom\..\atom
ApplyReleasesImpl: Creating shortcut for atom.exe => C:\Users\Mikael\AppData\Roaming\Microsoft\Windows\Start Menu\Programs\GitHub, Inc.\Atom.lnk
ApplyReleasesImpl: About to save shortcut: C:\Users\Mikael\AppData\Roaming\Microsoft\Windows\Start Menu\Programs\GitHub, Inc.\Atom.lnk (target C:\Users\Mikael\AppData\Local\atom\update.exe, workingDir C:\Users\Mikael\AppData\Local\atom\..\atom\app-0.194.0, args --processStart atom.exe)
ApplyReleasesImpl: Creating shortcut for atom.exe => C:\Users\Mikael\Desktop\Atom.lnk
ApplyReleasesImpl: About to save shortcut: C:\Users\Mikael\Desktop\Atom.lnk (target C:\Users\Mikael\AppData\Local\atom\update.exe, workingDir C:\Users\Mikael\AppData\Local\atom\..\atom\app-0.194.0, args --processStart atom.exe)
Program: Starting Squirrel Updater: --download https://atom.io/api/updates
Program: Fetching update information, downloading from https://atom.io/api/updates
CheckForUpdateImpl: Downloading RELEASES file from https://atom.io/api/updates
FileDownloader: Downloading url: https://atom.io/api/updates/RELEASES
Program: Starting Squirrel Updater: --processStart atom.exe
Program: Want to launch 'C:\Users\Mikael\AppData\Local\atom\app-0.194.0\atom.exe'
Program: About to launch: 'C:\Users\Mikael\AppData\Local\atom\app-0.194.0\atom.exe':
Program: Starting Squirrel Updater: --download https://atom.io/api/updates
Program: Fetching update information, downloading from https://atom.io/api/updates
CheckForUpdateImpl: Downloading RELEASES file from https://atom.io/api/updates
FileDownloader: Downloading url: https://atom.io/api/updates/RELEASES
Program: Starting Squirrel Updater: C:\Users\Mikael\AppData\Local\atom\SquirrelSetup.log

It gave me no hint.
I searched for atom.cmd and apm.cmd.
C:\Users\Mikael\AppData\Local\atom\bin\atom.cmd
C:\Users\Mikael\AppData\Local\atom\app-0.194.0\resources\cli\atom.cmd
C:\Users\Mikael\AppData\Local\atom\app-0.194.0\resources\app\apm\bin\apm.cmd

It seems like a file or link to apm,  C:\Users\Mikael\AppData\Local\atom\bin\apm.cmd is missing. This file do exists on the working Win8-installation.
So. I uninstalled Atom, rebooted my Windows 8.1 Pro 64-bit, and did another fresh install of Atom, downloaded from atom.io, rebooted and ended up with the same results.
Reproduce like this:

Do a fresh install of Windows 8.1 Pro 64-bit. Run Windows update (I did it for a couple of days...). Install Atom from atom.io. Open cmd.exe and check if atom or apm is in PATH.

I'll try the same thing, install Atom on a fresh Windows installation, using Windows 8 and Windows 7. But it will take a day or two.

I can reproduce the fault on a fresh installation of Windows 7 Pro SP1 64-bit (WIndows update is done, several times) using Atom 0.194.0  installer from atom.io.
There are no files in C:\Users\mos\AppData\Local\atom\bin, I expected both atom.cmd and apm.cmd (could someone confirm they should be there?).
The PATH is untouched by the installation program.
C:\Users\mos>PATH
PATH=C:\Windows\system32;C:\Windows;C:\Windows\System32\Wbem;C:\Windows\System32
\WindowsPowerShell\v1.0\

I found two log files for SquirrelSetup.log, they gave me no hint, though the latter contains exceptions.
C:\Users\mos\AppData\Local\atom\SquirrelSetup.log
Program: Starting Squirrel Updater: --createShortcut atom.exe
ApplyReleasesImpl: About to create shortcuts for atom.exe, rootAppDir C:\Users\mos\AppData\Local\atom\..\atom
ApplyReleasesImpl: Creating shortcut for atom.exe => C:\Users\mos\AppData\Roaming\Microsoft\Windows\Start Menu\Programs\GitHub, Inc.\Atom.lnk
ApplyReleasesImpl: About to save shortcut: C:\Users\mos\AppData\Roaming\Microsoft\Windows\Start Menu\Programs\GitHub, Inc.\Atom.lnk (target C:\Users\mos\AppData\Local\atom\update.exe, workingDir C:\Users\mos\AppData\Local\atom\..\atom\app-0.194.0, args --processStart atom.exe)
ApplyReleasesImpl: Creating shortcut for atom.exe => C:\Users\mos\Desktop\Atom.lnk
ApplyReleasesImpl: About to save shortcut: C:\Users\mos\Desktop\Atom.lnk (target C:\Users\mos\AppData\Local\atom\update.exe, workingDir C:\Users\mos\AppData\Local\atom\..\atom\app-0.194.0, args --processStart atom.exe)
Program: Starting Squirrel Updater: --download https://atom.io/api/updates
Program: Fetching update information, downloading from https://atom.io/api/updates
CheckForUpdateImpl: Downloading RELEASES file from https://atom.io/api/updates
FileDownloader: Downloading url: https://atom.io/api/updates/RELEASES

C:\Users\mos\AppData\Local\SquirrelTemp\SquirrelSetup.log
Program: Starting Squirrel Updater: --install .
Program: Starting install, writing to C:\Users\mos\AppData\Local\SquirrelTemp
CheckForUpdateImpl: Failed to load local releases, starting from scratch: System.IO.DirectoryNotFoundException: Could not find a part of the path 'C:\Users\mos\AppData\Local\atom\packages\RELEASES'.
   at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
   at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)
   at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share)
   at Squirrel.Utility.LoadLocalReleases(String localReleaseFile)
   at Squirrel.UpdateManager.CheckForUpdateImpl.<CheckForUpdate>d__28.MoveNext()
CheckForUpdateImpl: Reading RELEASES file from C:\Users\mos\AppData\Local\SquirrelTemp
CheckForUpdateImpl: First run or local directory is corrupt, starting from scratch
ApplyReleasesImpl: Writing files to app directory: C:\Users\mos\AppData\Local\atom\app-0.194.0
ApplyReleasesImpl: Squirrel Enabled Apps: [C:\Users\mos\AppData\Local\atom\app-0.194.0\atom.exe]
ApplyReleasesImpl: Couldn't run Squirrel hook, continuing: C:\Users\mos\AppData\Local\atom\app-0.194.0\atom.exe: System.OperationCanceledException: The operation was canceled.
   at System.Threading.CancellationToken.ThrowOperationCanceledException()
   at Squirrel.Utility.<>c__DisplayClassf.<InvokeProcessAsync>b__e()
   at System.Threading.Tasks.Task.InnerInvoke()
   at System.Threading.Tasks.Task.Execute()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Squirrel.Utility.<InvokeProcessAsync>d__11.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Squirrel.UpdateManager.ApplyReleasesImpl.<>c__DisplayClassb5.<<invokePostInstall>b__a9>d__b7.MoveNext()
ApplyReleasesImpl: Starting fixPinnedExecutables
ApplyReleasesImpl: fixPinnedExecutables: oldAppDirectories is empty, this is pointless
ApplyReleasesImpl: cleanDeadVersions: for version 0.194.0
ApplyReleasesImpl: cleanDeadVersions: exclude folder app-0.194.0

I use Windows 7 Ultimate SP1 64-bit OS and I have the same problem with same error log information in SquirrelSetup.log file.  I get into this problem after I re-installed Windows 7 . I didn't have this problem before. Any suggestion to fix this problem ?

ApplyReleasesImpl: Couldn't run Squirrel hook, continuing: C:\Users\mos\AppData\Local\atom\app-0.194.0\atom.exe: System.OperationCanceledException: The operation was canceled.

This means atom.exe took more than 15 seconds to set up shortcuts / do other stuff (most likely it crashed or hung)

Did a clean install of Atom 0.198.0 on Windows 8 Pro. Seems to work, both atom and apm is now in path.
C:\Users\Mikael>PATH
PATH=C:\Windows\system32;C:\Windows;C:\Windows\System32\Wbem;C:\Windows\System32
\WindowsPowerShell\v1.0\;C:\Users\Mikael\AppData\Local\atom\bin

Perhaps its solved. I'll double check on Windows 7 and 8.1 with the latest version of atom.

Yep, it works as expected on Windows 7. Both atom and apm in PATH.
I assume its fixed, I'll skip testing it on Windows 8.1 for now.

I just ran into this. Atom 0.198.0 on Windows 8.1 Ultimate. The path(s) were not added by the installer, and what more, there were two paths I needed to add:
...\AppData\Local\atom\app-0.198.0\resources\cli for atom.cmd and
...\AppData\Local\atom\app-0.198.0\resources\app\apm\bin for apm.cmd
I do not have a bin folder in \AppData\Local\atom (unlike @mosbth).
I used AtomSetup.exe downloaded about an hour and a half ago.

Can confirm the path is added in Windows 10 Technical Preview May 9th, 2015 :-)

After reinstalling it worked just fine. Strange.

The second time I ran the installer (after uninstalling mind you) it completed significantly faster than the first time. My guess is that it silently failed.

It might've just been a coincidence, but I attempted to install the latest version of Atom (0.204.0) several times on my Windows 8.1 machine without it being able to install the path for apm. The second last time I installed it, however, I noticed that Bitdefender appeared to be blocking Atom. Here's the log file from Bitdefender:
Active Virus Control blocked a process.
Scan Results
File Name	Action
c:\users\jordan\appdata\local\atom\app-0.204.0\atom.exe	Blocked

I then reinstalled Atom with Bitdefender temporarily disabled, and the path was able to be set successfully.

Or you can manualy add the path.
Just open start panel and start typing "Edit environment variables for your account " and you can edit PATH in System variables.
For example C:\Users\user\AppData\Local\atom\bin\atom.cmd

Atom 1.0 installer does not add the path for atom or apm in the system variables on windows 10 64bit.

@h0nvas you don't put files in you paths. It should be C:\Users{yourusername}\AppData\Local\atom\bin;
This way typing atom or apm in the commandline runs that file.

I realize this is a closed thread, but I just ran into this problem and here is what I did that fixed it.
I had the same problem of not being able to run ATOM from the CLI, on my Windows box, but not on my Mac.
My solution was to completely uninstall both GIT for windows AND Atom, then reinstalling GIT bash first, then during the installation I used the third option under the "Adjusting your Path environment" which was "Use Git and optional Unix tools from the Windows Command Prompt". After that I installed Atom.  So GIT and ATOM in that order.
This added both Git and optional Unix tools to the PATH.
The downside is having to reinstall all your packages which for me was not fun as I had way too many, Lol.
Hopefully this helps other users.
V/R
Fredric

So I couldnt figure out why I couldnt see any options, then I relised I was reinstalling GitHub Desktop, not Git Bash, installing Git Bash Now!

It seems that atom updates sometimes fail to create the bin folder. I've had this problem before and I have it again with 1.24.0

Here's simple solution that works for me. I use chocolatey to manage most of my apps, Atom among them. When this happens, I should usually be able to fix it by choco install atom --force

I have added this location to env path where the files were present
C:\Users\user\AppData\Local\atom\app-1.25.0\resources\cli
And it works!

@iamsrikanthgopi Sure, that will work but only until the next time. We should not need to update the path every time a new version gets released.


Logout button

Overview
Introduces a settings dialog in order to store account information and change the retroquest theme.
Demo

It's now finished!


Doesn't work on surface pro 7

Describe the bug
After I installed bazzite using ventoy, I rebooted, i got grub and after selecting option to run bazzite i got black screen.
What did you expect to happen?
I expected to get sddm or plasma desktop, I got black screen.
Output of rpm-ostree status
I couldn't run it, as I failed to boot into OS.
Hardware
Surface pro 7 i7 10th gen
Extra information or context
I tried using rufus, but it showed me black screen after grub. I had to use grub mode in ventoy for it to work.

Could be related to https://github.com/ublue-os/bluefin/issues/1909


Only set CUDA_PATH if it is not already set

%env CUDA_PATH=/usr/local/cuda

May set the CUDA verison to an incorrect version in some cases.
Plus this is non-portable if running on Windows.
Only set this, if it is not already set. Ideally set it relative to a CUDA version which is on the PATH already.
Alterantively we could modify FLAMEGPU2 to try finding nvcc on the path and getting the CUDA directory that way instead.

import os
if "CUDA_PATH" not in os.environ:
  import subprocess, pathlib
  cuda_path = str(pathlib.Path(subprocess.check_output(["which", "nvcc"]).decode("utf-8")).parents[1])
  %env CUDA_PATH="$cuda_path"

Seems to be a not terrible way to do this, though it relies on which being available still so is bash only. Unfortunately can't just use bash expansion in %env magic.


Is it possible to include functionality inside of NodeEditorSaveManager to create new canvas?

I needed to be able to duplicate a node graph to to the same canvas and to a new canvas. I easily figured out how to duplicate a graph to the same canvas quite easily, but in order to duplicate to a new canvas, I need to create one. So I know I can call NodeEditorWindow.NewNodeCanvas() from an editor script, but I want to be able to call it from a node (via a button), which doesn't appear to be able to access editor scripts. What's the best way for me to accomplish this?

Sorry for the late reply. Wasn't able to set up email notifications on my (new) phone yet so I have to learn to manually check emails lol
Anyway, it is easily possible. Just create a new NodeCanvas like you would do with every scriptable object and then populate it with whatever you want using code. After that you can save it if you want. All this should not cause any interferences with the currently loaded canvas:)
Example:
NodeCanvas canvas = ScriptableObject.CreateInstance<NodeCanvas> ();
// Populate canvas with some nodes and stuff
NodeCanvasSaveManager.SaveNodeCanvas ("Assets/CreatedCanvas.asset", canvas, false);

It's pseudocode but it should work. Let me know if you encountered any errors of if I misunderstood you:)

No worries and thanks! I should have probably elaborated more haha. I want my Duplicate (new canvas) button to:

Save the current canvas to the scene
Create a new canvas (as if I pressed the new canvas button)
Then add duplicate nodes

So the main obstacle is loading the created canvas into the node editor window. Here's what I have so far:
`private void DupeNewCanvas() {
		string curCanvasName = NodeEditor.curNodeCanvas.name;

		if (curCanvasName == null || curCanvasName == "" || curCanvasName == "LastSession") {
			Debug.LogError("ERROR: Save to Scene before duplicating a machine to a new canvas.");
		} else {
			NodeEditorSaveManager.SaveSceneNodeCanvas(curCanvasName, ref NodeEditor.curNodeCanvas, true);
			NodeEditor.curNodeCanvas = CreateInstance<NodeCanvas>();
			NodeEditor.curNodeCanvas.name = "New Canvas";
			NodeEditor.curEditorState = CreateInstance<NodeEditorState>();
			NodeEditor.curEditorState.canvas = NodeEditor.curNodeCanvas;
			NodeEditor.curEditorState.name = "MainEditorState";
			NodeEditor.curNodeCanvas.editorStates = new NodeEditorState[] { NodeEditor.curEditorState };
			NodeEditorSaveManager.SaveNodeCanvas("Assets/Editor/Node_Editor/LastSession.asset", NodeEditor.curNodeCanvas, true);
			EditorApplication]
		}
	}`

Use NodeEditorWindow.mainNodeCanvas instead!
I will try to emphasize this more, NodeEditorWindow is not strictly part of the framework, more like a default browser to built upon. It is the 'user' of the framework and holds the current canvas.
The framework itself does not hold the canvas - NodeEditor.curNodeCanvas is only assigned while a user - in this case the window - draws the canvas. For the rest of the frame, it is null. This allows multiple 'users' to make use of the framework at the same time:)
Anyway, enough explaining the why. Replacing NodeEditor.curNodeCanvas with NodeEditorWindow.mainNodeCanvas should do it;)
I do have a new update in development which emphasizes this, it is basically a wrapper for holding canvases, called a user. More about it once I commit it:)

There's no way to access that script from a node, though right? Since it's not an editor script? If not, I'll just add my button else where. :)

You can try to put your script in an editor folder. Obviously it then won't be able to be used at runtime...
Try Assets/Plugins/Node_Editor/Nodes/Editor and if that doesn't work Assets/Plugins/Editor/Node_Editor/Nodes or similar:)

Cool that you've found a solution:)


Check Union Between Assets of Different Types

Check to see what happens when there is a union between assets of different asset types that extend one another.

Fixed in 2803e7183f4dafe5c15a83bff6cfef54e5bd986e.


HARVESTER: Show errors when creating kubeconfig fail

Summary
Show errors when creating kubeconfig fail

Occurred changes and/or fixed issues

https://github.com/harvester/harvester/issues/2889

Technical notes summary

Areas or cases that should be tested

Areas which could experience regressions

Screenshot/Video

Codecov Report
Base: 36.90% // Head: 36.82% // Decreases project coverage by -0.08% :warning:

Coverage data is based on head (d1386f8) compared to base (9079078).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7129      +/-   ##
==========================================
- Coverage   36.90%   36.82%   -0.09%
==========================================
  Files         977      977
  Lines       17291    17291
  Branches     4484     4484
==========================================
- Hits         6382     6368      -14
- Misses      10909    10923      +14

Flag
Coverage Δ

e2e
47.46% <ø> (-0.12%)
:arrow_down:

merged
36.82% <ø> (-0.09%)
:arrow_down:

unit
5.30% <ø> (ø)

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

Impacted Files
Coverage Δ

shell/edit/provisioning.cattle.io.cluster/rke2.vue
0.00% <ø> (ø)

shell/mixins/metric-poller.js
77.77% <0.00%> (-22.23%)
:arrow_down:

shell/utils/poller.js
75.00% <0.00%> (-10.00%)
:arrow_down:

shell/plugins/steve/mutations.js
51.06% <0.00%> (-4.26%)
:arrow_down:

shell/plugins/dashboard-store/actions.js
51.98% <0.00%> (-2.78%)
:arrow_down:

shell/utils/socket.js
48.52% <0.00%> (-0.50%)
:arrow_down:

shell/store/index.js
72.08% <0.00%> (-0.46%)
:arrow_down:

shell/plugins/steve/subscribe.js
70.32% <0.00%> (-0.28%)
:arrow_down:

shell/plugins/dashboard-store/resource-class.js
47.32% <0.00%> (+0.30%)
:arrow_up:

shell/models/provisioning.cattle.io.cluster.js
64.12% <0.00%> (+0.31%)
:arrow_up:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Parenthesis removal in VDS creation

Describe the enhancement requested
We put dbt in place for about 1,100 objects over the last month. We noticed some behavior change in the query that are executed by our users.
We traced that to the presence of encapsulating parenthesis in the object deployed thru dbt.
Going from  select column from table to ( select column from table ) is causing issue when the top vds are querying.
The top vds used to do
select column from tableA, tableB join on PK are now doing
select column from tableA, ( select column from tableB ) where PK join
This cause massive performance issue.
Is there any way we can remove theses parenthesis ?

Justification for this enhancement
Query performance improvement

Hi Alain, that is very strange behaviour. We will look into this as soon as we can. Thanks for raising this issue!

Could you please:

Let us know what version of Dremio is being used
Provide the model (or a snippet of the model) that leads to this behaviour?

23.1.0-202211250136090978-a79618c7

fix for issue 134, base release was 1.3.2
dbt-core>=1.4.1
dbt-dremio>=1.4.1

prepping model for upload

@Alain-Barrette we just finished getting dbt-dremio 1.4.5 out the door. Can you please see if you can reproduce this after upgrading?
However, dbt-dremio 1.4.5 won't automatically upgrade dbt-core. So please run pip install dbt-core==1.4.5 before pip install dbt-dremio
We believe that this regression is in dbt-core itself, but still looking into it.

reproduced after upgrade to 1.4.5

@Alain-Barrette I made a small change in my test branch, which removes parentheses from views. Do you mind seeing if this resolves your problem by cloning this repo, switching to test_branch and then installing your local repo (similar to how your team did for https://github.com/dremio/dbt-dremio/issues/134)?

Fix from test branch is working. We are deploying to our dev team member until the fix made it to an official release.
Thanks a lot everyone.


Outline request

Issue Type: Feature Request
-Please add something to make the outline feature shows the local variables in a function !
VS Code version: Code 1.52.1 (ea3859d4ba2f3e577a159bc91e3074c5d85c0523, 2020-12-16T16:34:46.910Z)
OS version: Windows_NT x64 10.0.19042

Which language is this for?

Closing as not enough information was provided to investigate this further

Closing as not enough information was provided to investigate this further


Sometimes you are unable to click

Current behavior
Currently, sometimes after scrolling a Tab.FlatList you are unable to click on one of the rendered items (if they are a touchable). This doesn't happen all of the time but sometimes it does.
Expected behaviour
I would expect that after scrolling, all items that are clickable are able to be clicked.

I think this was resolved either in rc.5 or rc.6. Closing for now.

Hi @alexpchin could you please reopen this issue? I'm still encountering this issue running rc.9. It happens in both Tab.FlatList and Tab.ScrollView on iOS. The behavior is very specific: when a scroll action/animation hasn't finished yet and you scroll again, the content isn't clickable anymore. It becomes clickable again only when you scroll to the top or bottom of the list or view. See the video for a demo. Thanks!
https://user-images.githubusercontent.com/15447148/108205147-f2ba7780-7124-11eb-9bef-e976cbba581f.MP4

@abelmenkveld
I cannot reproduce this. Here's the diff I tested with:
diff --git a/example/src/Shared/Contacts.tsx b/example/src/Shared/Contacts.tsx
index beeeccf..c4f27f3 100644
--- a/example/src/Shared/Contacts.tsx
+++ b/example/src/Shared/Contacts.tsx
@@ -1,6 +1,7 @@
 import * as React from 'react'
 import { View, Text, StyleSheet, Platform } from 'react-native'
 import * as Tabs from 'react-native-collapsible-tab-view'
+import { TouchableOpacity } from 'react-native-gesture-handler'
 import Animated, {
   useAnimatedStyle,
   useDerivedValue,
@@ -70,17 +71,19 @@ class ContactItem extends React.PureComponent<{
     const { item } = this.props

     return (
-      <View style={styles.item}>
-        <View style={styles.avatar}>
-          <Text style={styles.letter}>
-            {item.name.slice(0, 1).toUpperCase()}
-          </Text>
+      <TouchableOpacity>
+        <View style={styles.item}>
+          <View style={styles.avatar}>
+            <Text style={styles.letter}>
+              {item.name.slice(0, 1).toUpperCase()}
+            </Text>
+          </View>
+          <View style={styles.details}>
+            <Text style={styles.name}>{item.name}</Text>
+            <Text style={styles.number}>{item.number}</Text>
+          </View>
         </View>
-        <View style={styles.details}>
-          <Text style={styles.name}>{item.name}</Text>
-          <Text style={styles.number}>{item.number}</Text>
-        </View>
-      </View>
+      </TouchableOpacity>
     )
   }
 }

Thanks for getting back this quickly. Thanks to your diff I tried importing TouchableOpacity from react-native-gesture-handler instead of from react-native. This fixes the issue.

I'm actually still seeing this issue sometimes in my app, I'm going to re-open

Reopened. But there is nothing in the code that can explicitly prevent touches on elements. If this happens it seems like an upstream bug.

What do you mean upstream bug?

I think this is happening when you tap whilst the list is moving. So scroll and whilst it's scrolling tap and then tap something.

I mean either a react native or some weird reanimated bug. Without a repro I can't tell though either way.

I'll try to find the steps to reproduce so you can replicate. Initially try scrolling and then tapping (on a device) whilst the scroll is still happening.

Did you try this solution? https://github.com/PedroBern/react-native-collapsible-tab-view/issues/123#issuecomment-780582754

Hi @andreialecu I've pushed an example here: https://github.com/andreialecu/rnctv-react-native-bare-test/pull/2 Please see if you can reproduce the error.
I just cycled through the various components to compare:
<Tabs.FlatList
      numColumns={2}
      data={covers}
      ListEmptyComponent={ListEmptyComponent}
      renderItem={({item, index}) => {
        const PressableComponent =
          index % 4 === 0
            ? TouchableOpacityRNGH
            : index % 3 === 0
            ? TouchableWithoutFeedback
            : index % 2 === 0
            ? TouchableOpacity
            : Pressable;
        const PressableComponentString =
          index % 4 === 0
            ? 'TouchableOpacityRNGH'
            : index % 3 === 0
            ? 'TouchableWithoutFeedback'
            : index % 2 === 0
            ? 'TouchableOpacity'
            : 'Pressable';
        return (
          <PressableComponent
            onPress={() => Alert.alert(PressableComponentString)}>
            <Image source={{uri: item}} style={styles.cover} />
          </PressableComponent>
        );
      }}
      keyExtractor={(_, i) => String(i)}
      refreshing={isRefreshing}
      onRefresh={onRefresh}
    />

It seems that there is an issue with TouchableOpacity, TouchableWithoutFeedback and Pressable from React Native but not with TouchableOpacity from react-native-gesture-handler. This. happens only after you scroll.

Fixed in rc.11. This seems like a react-native bug of some sort because of scrolling inside onBeginDrag (in order to stop scrolling, see: ). Removed it for now.


fix: Updating the default log path location to match mountpoint

Closes #76

#78  👈
main

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

Join @tazarov and the rest of your teammates on  Graphite


Fix crashes on amd64

64-bit support has bit rotted a bit it seems. This PR fixes crashes we've encountered while making mesh by jetlag. It may be not a complete fix for all possible amd64 issues, just the ones that we've encountered.
It also updates Player project settings for 64-bit targets so it actually compiles and works there.

Hey Ivan, thanks for the PR ( and nice Vulkan intro :) )
I did some checks on my side and committed can you test with your song from the intro again with my latest commit and get back to me?

Hey Ivan, thanks for the PR ( and nice Vulkan intro :) )
I did some checks on my side and committed can you test with your song from the intro again with my latest commit and get back to me?

Seems to work just fine. Thank you so much!

Seems to work just fine. Thank you so much!


Feature/data versioning and optimization

This PR combines the changes from #133 and #143, rebased for legibility and to incorporate the changes in meta behavior on main.

@korlaxxalrok this is the PR we'll want to merge on Migration Day simultaneously with the data loading, since it will not insert a spurious issue for each direction recomputation.

My only additional note is that get_data_stdev_across_locations in database.py is still computed regardless of issue dates,  I am not sure if this is the intended behavior or if it also should be reimplemented to compute standard deviation on only data with latest issue per key.


feat(auth): add Sign in with Apple directly in Firebase Auth for Android, iOS 13+ and Web

Description
We currently rely on sign_in_with_apple for Sign In With Apple. We now recommend directly using directly Firebase to connect with Apple.
Related Issues
Closes https://github.com/firebase/flutterfire/discussions/5560
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[ ] All existing and new tests are passing.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] The analyzer (melos run analyze) does not report any problems on my PR.
[ ] I read and followed the Flutter Style Guide.
[ ] I signed the CLA.
[ ] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change.
[X] No, this is not a breaking change.

It also has support for macOS, right?

It does, but since Firebase iOS SDK does not have official macOS supports, it's not mentioned

Just out of curiosity, what would go wrong for iOS 11-12?

@felixgabler Sign In With Apple was introduced in iOS 13. Before you had to open a custom webview with the Apple JS Framework. You can see full explanations here.


Sound increment of 10

When changing the sound settings, there is a message that you can hold shift to have volume go up or down in increments of 10, but it still goes up and down 1 at a time when you hold shift.

Hi @Duck999, I believe this is a duplicate of issue #11, which was resolved earlier today. It should be functional in the next build.

Closing for now


Highlight syntax error in Ubuntu vscode

works on windows vscode well.
but I moved project to ubuntu and installed same plugin prisma extension.
but all type and fileds shows error even if it is correct.

I guess I have similar issue there.
I am on Mac and this is what I'm getting after prisma init in my datamodel.prisma file.
Model declarations have to be indicated with the `model` keyword.

cc: @janpio

datamodel.prisma indicates you are using Prisma 1 - this extension does not support Prisma 1 files (although VS Code might tell you otherwise). Please use https://marketplace.visualstudio.com/items?itemName=Prisma.vscode-graphql and rename your file to datamodel.graphql to make it work.
@starcraft0429 Could this possibly apply to you as well?

Sorry, Now I don't use prisma anymore.


completeopt+=fuzzy does not honor completeslash=slash (Windows)

Steps to reproduce

set completeopt+=fuzzy
set completeslash=slash
Type C:/ and trigger file completion, the match list is empty (non fuzzy complete would give matches).
Type C:\ and trigger file completion, matches are found, but text is completed with foward slashes, making any follow-up completion impossible.

Expected behaviour
Expecting fuzzy file completion to behave the same as non-fuzzy file completion when it comes to matching slashes (backward or forward).
Thanks a lot!
Version of Vim
9.1.636
Environment
Operating System: Windows
Logs and stack traces
No response

@glepnir fyi

hello thanks for report. I haven't used Windows for many years so some options related to Windows are automatically ignored by my mind. Can you try this diff? if not works.. i may need find a windows env for fix.
diff --git a/src/insexpand.c b/src/insexpand.c
index 3a168848f..95b4f24d9 100644
--- a/src/insexpand.c
+++ b/src/insexpand.c
@@ -3552,9 +3552,15 @@ get_next_filename_completion(void)
     size_t     path_with_wildcard_len;
     char_u     *path_with_wildcard;

+#ifdef BACKSLASH_IN_FILENAME
+    char       pathsep = curbuf->b_p_csl[0] == 's' ? '/' : '\\';
+#else
+    char       pathsep = '/';
+#endif
+
     if (in_fuzzy)
     {
-       last_sep = vim_strrchr(leader, PATHSEP);
+       last_sep = vim_strrchr(leader, pathsep);
        if (last_sep == NULL)
        {
            // No path separator or separator is the last character,

I think it should use value of completeslash option to decide on pathsep

in diff it already use

in diff it already use

Oh, true, sorry missed it.

The patch works for me! Thanks!
There is still a slight difference in behavior with non-fuzzy matching though, but it is probably an evil edge case:
with non-fuzzy matching, even with set completeslash=slash, if I type in a \ and <c-x><c-f>, vim will happily find matches, and even fix my initial \ into a /.
This does not happen with fuzzy completion, it doesn't find any match with an initial \.

aha thanks !  good to know this bahavior :) and i create a new diff for this. could you try it sry for asking you to help me with the test
diff --git a/src/insexpand.c b/src/insexpand.c
index 3a168848f..49848a32d 100644
--- a/src/insexpand.c
+++ b/src/insexpand.c
@@ -3552,9 +3552,25 @@ get_next_filename_completion(void)
     size_t     path_with_wildcard_len;
     char_u     *path_with_wildcard;

+#ifdef BACKSLASH_IN_FILENAME
+    char       pathsep = curbuf->b_p_csl[0] == 's' ? '/' : '\\';
+#else
+    char       pathsep = '/';
+#endif
+
     if (in_fuzzy)
     {
-       last_sep = vim_strrchr(leader, PATHSEP);
+#ifdef BACKSLASH_IN_FILENAME
+       if (curbuf->b_p_csl[0] == 's')
+       {
+           for (i = 0; i < leader_len; i++)
+           {
+               if (leader[i] == '\\')
+                   leader[i] = '/';
+           }
+        }
+#endif
+       last_sep = vim_strrchr(leader, pathsep);
        if (last_sep == NULL)
        {
            // No path separator or separator is the last character,

Seems to be all good now!
Many thanks!

aha good to know :) but I will submit a PR the day after tomorrow to make sure you don't run into other edge cases. If there are any, I can try to fix them here in time.

Thank you so much @glepnir for working on fuzzy completion. So far so good with file completion. I found another unrelated issue with fuzzy and the "adding keyword completion" mode, so I opened a new issue: #15412.

thanks i ran into that issue now  and will open pr for this issue fix.


Implementing telemetry for tekton commands

Working good. It would be nice if you also cover https://github.com/redhat-developer/vscode-tekton/issues/514 it this PR.

Working good. It would be nice if you also cover #514 it this PR.

we already have USAGE_DATA.md file I will add more information in it and I will create a PR in vscode-commons https://github.com/redhat-developer/vscode-commons/blob/master/USAGE_DATA.md#other-extensions to provide the reference link

Codecov Report

Merging #508 (9256f1c) into master (c3829e5) will decrease coverage by 0.06%.
The diff coverage is 66.83%.

@@            Coverage Diff             @@
##           master     #508      +/-   ##
==========================================
- Coverage   67.37%   67.30%   -0.07%
==========================================
  Files          80       80
  Lines        5370     5469      +99
  Branches      942      953      +11
==========================================
+ Hits         3618     3681      +63
- Misses       1752     1788      +36

Impacted Files
Coverage Δ

src/tekton/pipelinecontent.ts
93.63% <ø> (ø)

src/tekton/restartpipelinerundata.ts
38.33% <0.00%> (ø)

src/tekton/task.ts
64.28% <0.00%> (ø)

src/tekton/webviewstartpipeline.ts
100.00% <ø> (ø)

src/tekton/startpipeline.ts
53.84% <20.00%> (-0.70%)
:arrow_down:

src/kubernetes.ts
65.78% <33.33%> (-1.86%)
:arrow_down:

src/pipeline/pipelineExplorer.ts
59.09% <33.33%> (-0.44%)
:arrow_down:

src/tekton/starttask.ts
28.57% <33.33%> (+1.90%)
:arrow_up:

src/commands/delete.ts
39.32% <40.00%> (-1.64%)
:arrow_down:

src/tekton/expose.ts
64.70% <50.00%> (ø)

... and 15 more

Continue to review full report at Codecov.

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

Also added USAGE_DATA.md file in vscode-commons https://github.com/redhat-developer/vscode-commons/pull/16


[SPARK-17636][SQL][WIP] Parquet predicate pushdown in nested fields

What changes were proposed in this pull request?
Support Parquet predicate pushdown in nested fields
How was this patch tested?
Existing tests and new tests are added.

Test build #96506 has started for PR 22535 at commit c95706f.

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/testing-k8s-prb-make-spark-distribution-unified/3404/
Test PASSed.

retest this please.

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/testing-k8s-prb-make-spark-distribution-unified/3415/
Test PASSed.

Test build #96528 has started for PR 22535 at commit c95706f.

Test build #96528 has finished for PR 22535 at commit c95706f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96528/
Test FAILed.

Merged build finished. Test FAILed.

Hi, @dbtsai . Could you fix the UT failure?
[info] ParquetFilterSuite:
...
[info] - SPARK-12218 Converting conjunctions into Parquet filter predicates *** FAILED *** (19 milliseconds)
[info]   Expected None, but got Some(lt(a, 10)) (ParquetFilterSuite.scala:802)

I'm breaking this PRs into three smaller PR. I'll fix the tests in those smaller PRs. Thanks.

Test build #99673 has started for PR 22535 at commit c95706f.

Test build #99673 has finished for PR 22535 at commit c95706f.

This patch fails Spark unit tests.
This patch does not merge cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/99673/
Test FAILed.

@dbtsai are you still working on this?

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/15740/
Test PASSed.

Build finished. Test PASSed.

Test build #115067 has started for PR 22535 at commit eb400ed.

Test build #115067 has finished for PR 22535 at commit eb400ed.

This patch fails build dependency tests.
This patch does not merge cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/115067/
Test FAILed.

Going to close this in favor of new PR, https://github.com/apache/spark/pull/27728


Gene linkout testing with cicar.ICC4958.gnm2.ann1.Ca_00001 fails

          Gene linkout testing with cicar.ICC4958.gnm2.ann1.Ca_00001

[ ] GCV - no content in GCV layout; POST to https://gcv-microservices.lis.ncgr.org/lis/genes responds with 500 and payload "500 Internal Server Error Server got itself in trouble" @alancleary
[x] LegumeMine
[ ] Funnotate -- gene is not present in the tree! Should be be showing Funnotate link for genes that aren't in trees, @svengato and @adf-ncgr  ??? This results in the "Jump to highlighted feature" link doing nothing, so that's a bug.

I realize these may be upstream issues, but _broken linkouts must be fixed before we put this out for LIS review.
Here are the broken linkouts (so you don't have to fire up graphql-server and web-components to reproduce these):
GCV: https://gcv.legumeinfo.org/gene;lis=cicar.ICC4958.gnm2.ann1.Ca_00001
Funnotate: https://funnotate.legumeinfo.org/?gene_name=cicar.ICC4958.gnm2.ann1.Ca_00001
Originally posted by @sammyjava in https://github.com/legumeinfo/web-components/issues/115#issuecomment-1550116399

GCV: @alancleary reports that the URL is incorrect:

Regarding the GCV location link you provided: the issue is that the URL that came from the service is not correct. The service returned https://gcv.legumeinfo.org/tools/gcv/, but it should be https://gcv.legumeinfo.org/gcv2. As with Funnotate, the chromosome you selected isn't actually in GCV so even with the correct URL you won't get a result.

However, we should not send folks to GCV if the chromosome isn't in it. So even with the correct URL this linkout should be suppressed.

Funnotate: doesn't display a gene that doesn't exist in a tree: update linkout to say "View this Gene Family in a phylogram" or equivalent.

@sammyjava can you run a test with glyma.Wm82.gnm2.ann1.Glyma.06G065000 and tell me whether the links work for that as expected? (with the likely exception of Jbrowse2)

We only have  cicar.ICC4958.gnm2.ann1 and phavu.G19833.gnm1.ann1 genes in MiniMine. I can test a phavu gene with the web-components but are you telling me you don't have a standalone testing platform for linkouts? The graphql-server and web-components stuff works as expected, simply spits out what the linkout service provides (and works fine for LegumeMine links).

OK, a phavu gene should probably work as well as the glyma one I suggested. I don't have a (convenient) standalone testing platform for linkouts other than curl. Having a dev instance would of the gene search would make it easier to systematically run through the various species, but isn't essential. The ones that are in there are just a one-size-fits-many approximation at the moment, but I was expecting to have to go through at least one round of refinements on this (likely more than one).

Well, since Alan is done, I'll go ahead now and mount the gene search jekyll demo. We're done on our end. But we'll not share this with the LIS folks tomorrow, we need to get the linkouts fully working before this leaves the building.

Thanks, I can get linkouts working for at least the minimine genomes by tomorrow.

OK I'll set the dev jekyll site on MiniMine then so we have a working instance. It'll be obvious that it's limited to anyone that would, say, search for a soybean gene. :)

definitely suboptimal as far as showing off the non-linkout features of the UI! How about you just leave it pointing to legumemine and I do my darnedest to get the linkouts to have only functioning links by tomorrow's meeting? Sorry for dithering.

Easy to switch. I'll set it on LegumeMine for now, do your darndest. I'll let you know when it's up, I have to load the new front end in and doing some system updates.

thanks! and darn the torpedoes...

Have at it: https://jekyll-dev.lis.ncgr.org/tools/search/gene-search.html

Note that we do have a gene family item in the gene search results, so it's a bit weird to say "View this gene's gene family" on the gene linkout when the gene family itself is not linked. But I guess we'll add gene family linkouts in the future, plus the linkouts are for any situation where you have a gene identifier, not just this one. Just thought I'd mention it.

I like it!
Shall we point the Ames crew at it, for testing?

I think I'm on the hook to make the linkouts less prone to send users to unproductive URLs before we unleash the wisdom of the crowds on it. Let's hold off until tomorrow's meeting in any case!

Well, nothing like LIS testers to turn up more issues, so I'll send out a missive.

Funnotate -- gene is not present in the tree! Should be be showing Funnotate link for genes that aren't in trees, @svengato and @adf-ncgr ??? This results in the "Jump to highlighted feature" link doing nothing, so that's a bug.

Is the problem in Funnotate or in the linkout service? I can make "Jump to highlighted feature" display only the intersection of (requested genes) and (genes in the tree), or alternatively add a "Warning: Not in tree" disclaimer, though these seem only slightly less confusing than what we have now.

I agree.

Fixed in Funnotate commit 6a1b5ee - "Jump to highlighted feature" only lists and highlights sequences that exist in the tree.


[Automated deployment] local machine problem with docker build step

I've found out that there is issue with command $ sh docker-run.sh
Building mn-documents-service
Step 1/4 : FROM openjdk:8u171-alpine3.7
8u171-alpine3.7: Pulling from library/openjdk
911c6d0c7995: Pull complete
4001add52a90: Pull complete
874028933da5: Pull complete
Digest: sha256:12e19d0dffa75ded3ab909e8c1966682b097f155ca0c2c88577445c64d6776d5
Status: Downloaded newer image for openjdk:8u171-alpine3.7
 ---> 1caad94162ef
Step 2/4 : RUN apk --no-cache add curl
 ---> Running in 7b456af061f8
fetch http://dl-cdn.alpinelinux.org/alpine/v3.7/main/x86_64/APKINDEX.tar.gz
fetch http://dl-cdn.alpinelinux.org/alpine/v3.7/community/x86_64/APKINDEX.tar.gz
(1/3) Installing libssh2 (1.8.2-r0)
(2/3) Installing libcurl (7.61.1-r2)
(3/3) Installing curl (7.61.1-r2)
Executing busybox-1.27.2-r11.trigger
OK: 100 MiB in 56 packages
Removing intermediate container 7b456af061f8
 ---> 8c3c977f7245
Step 3/4 : COPY target/documents-service*.jar documents-service.jar
ERROR: Service 'mn-documents-service' failed to build: COPY failed: no source files were specified

Is there any clue that can be helpful to fix it?

You should build app before run using docker-run.sh script :)
https://github.com/asc-lab/micronaut-microservices-poc#building
Dockerfile only copy jar from target, so it is expected previously created jar.


Remove nodes

Drain node except daemonsets resource
Use reset cluster for delete deploy data
Then delete node

ci check this

@Atoms
The steps：pre-remove -> roles/reset -> post-remove
the roles/reset setup will clean up node certificates and deploy data.

ok, missed that part in reviewing


Simplify offer scoring

Consolidate on a single strategy for offer scoring that will account for usage if it is available, using allocated resource values if not
Stop treating long running, non-long running separately in offer scoring
Consolidate configurable values

mesos:
  allocatedResourceWeight: 0.5
  inUseResourceWeight: 0.5
  cpuWeight: 0.4
  memWeight: 0.4
  diskWeight: 0.2

rename of used/free to allocated/inUse

cc @baconmania @pschoenfelder

🚢


Can´t download

This appears after I try to download by member ID, and selecting the pages
Member Url: http://www.pixiv.net/member_illust.php?id=16891&p=1
Traceback (most recent call last):
File "PixivUtil2.py", line 320, in process_member
File "PixivModel.pyc", line 51, in init
File "PixivModel.pyc", line 104, in ParseImages
AttributeError: 'NoneType' object has no attribute 'findAll'
Error at process_member(): (<type 'exceptions.AttributeError'>, AttributeError("'NoneType' object has no attribute 'findAll'",), <traceback object at 0x033141E8>)
Empty Html
Dumping html to: Error page for member 16891.html
Traceback (most recent call last):
File "PixivUtil2.py", line 1851, in main
File "PixivUtil2.py", line 1637, in main_loop
File "PixivUtil2.py", line 1299, in menu_download_by_member_id
File "PixivUtil2.py", line 320, in process_member
File "PixivModel.pyc", line 51, in init
File "PixivModel.pyc", line 104, in ParseImages
AttributeError: 'NoneType' object has no attribute 'findAll'
press enter to exit.

Looks like it return empty html so the application cannot get the images. Most likely network issue.
Can you try on different pc/internet connection?

I tried on a different pc/connection but still I am getting the same error, then when  try to log in in the pixiv utility this appears:
Trying to log with saved cookie
Error at doLogin(): (<class 'httplib.IncompleteRead'>, IncompleteRead(5680 bytes read), <traceback object at 0x02C8AD50>)
Cannot Login!
press enter to exit.
So I have to delete and unrar the content once again. Error remains

Error at doLogin(): (, IncompleteRead(5680 bytes read), )

This is due network connection interrupted.
Can you open the website on browser?

Yes, I can open the website without problems, no matter the browser or pc.

do you use firewall or antivirus? from the log, it show connection issue, whether the site return empty or partial response.
also try to use the cookie login method?
Q3: Error at process_image(): (<type 'exceptions.AttributeError'>,
AttributeError ("'NoneType' object has no attribute 'find'",)
- Usually this is because of login failed (cookie not valid). Try to change
your password to simple one for testing, or copy the cookie from browser:
1. Open Firefox.
2. Login to your pixiv.
3. Right click the page and select View Page Info.
4. Click the Security tab.
5. Click the View Cookies button.
6. Look for Cookie named = PHPSESSID.
7. Copy the content value.
8. Open config.ini, go to [Authentication] section, paste the value to
cookie, set 'keepsignedin = 1'.

Yes, I use antivirus (Eset Nod), I will try deactivating it or making an exemption to the downloader.
I have tried using the cookie login method but didn`t work. I will try again more carefully, and post later if I managed to reach a solution. I appreciate in advance for all the help you are giving me.