Fixes broken migration to klog

Fixes https://github.com/ceph/ceph-csi/issues/187
Bug introduced in https://github.com/ceph/ceph-csi/pull/150 (quite insufficient testing - plugins would panic immediately, https://github.com/ceph/ceph-csi/issues/168) by not playing nicely with vendor'd packages that still use glog (github.com/kubernetes-csi/drivers/pkg/csi-common).
This then continued on in https://github.com/ceph/ceph-csi/pull/169 which fixed the panic, but skipping klog.InitFlags() means logtostderr cmd flag doesn't get propagated to klog.
This PR reintroduces klog.InitFlags() in a hopefully less conflicting manner with glog.

cc @rootfs @humblec

thanks @gman0

@gman0 Thanks.. One question here is, the klogInitFlags() has to happen before flags.set https://github.com/ceph/ceph-csi/pull/188/files#diff-48c9752907803b1df4780beae292b547R38, otherwise it will be NO OP , Isnt it ?

@humblec I don't think it makes a difference in this case, because glog already initializes the cmd flags:
First, glog initializes the global command-line FlagSet in its init() function:
https://github.com/ceph/ceph-csi/blob/1748629de2f57010de7a3fbe11e430eabc8138ff/vendor/github.com/golang/glog/glog.go#L398-L404
...which means the flag.Set() in this PR actually sets the logtostderr flag for glog, not klog => it's not a NOOP.
After the values are parsed by flag.Parse(), a new FlagSet (a separate one, for klog only) is created and klog stores its cmd args here (that's why it doesn't panic anymore, it uses a separate flagset - no duplicate flags):
https://github.com/ceph/ceph-csi/blob/1748629de2f57010de7a3fbe11e430eabc8138ff/vendor/k8s.io/klog/klog.go#L407-L419
Lastly, the values from glog flagset are copied over to the values of klog flagset. Granted, this is hardly ideal because there are some flags that klog has and glog hasn't, and those cannot be set (because glog manages the global cmd flagset) but AFAIK there's no way around this, unless csi-common moves to klog or we move away from csi-common.

@gman0 The csi-common migration to klog is under review here https://github.com/kubernetes-csi/drivers/pull/144 , may be getting it done first would be the right choice. I have to do some changes in that PR and I am on it.

PTAL @rootfs @humblec
CI is failing on an unrelated file:
$ scripts/lint-text.sh --require-all
mdl --style scripts/mdl-style.rb ./README.md
./README.md:27: MD012 Multiple consecutive blank lines

may be fixed in another PR

rerun CI and it passed. megify failed to merge, will figure out that later. thanks @gman0


Relod metadata cache if object not found?

Is your feature request related to a problem? Please describe.

I often create DB Objects using plain SQL (as opposed to using the GUI).
I heavily use "Open Database Object" (Ctrl-Shift-D) and Ctrl-Mouseclick (jump to definition).

However, DBeaver seems to cache the metadata and does not update the information from 2. automatically. I often get errors like "database object not found".
However I unterstand that you cannot update your metadata cache when I am executing arbitrary DDL statements.
Describe the solution you'd like
Proposal: If the DB object is not found in the DBeaver metadata cache, refresh it and search again.
Describe alternatives you've considered
Alternative: select a root node in the navigator and press F5.
Additional context
using a PostgreSQL DB

Hello @fjf2002
Thanks for the idea.
Partially, it is already described here: https://github.com/dbeaver/dbeaver/issues/16219
But, probably, we can add a special Refresh setting to the Search dialog/Search panel or in the Preferences.

@LonwoLonwo: Yeah, that would be great. Thanks.


More style for Markers

All markers look the same, but I saw there is already an issue opened for that, so it's ok
It doesn't looks like there is a "You're here" marker, if so, it should be different than the other one (at least the color).
In the pop-up, we can see the link. Maybe it should be deleted, and replaced by an hyperlink (Like "See on Wikipedia" instead of "http:// ..." )
Maybe it could be nice to add a circle on the map showing the range, like on the website

Nice work !

There is a marker on user's position, but the pop-up is empty when we click on it.

Last comment is a duplicate of #29.
Also modified the first post to a list


[Feature] Comment block support

Problem
The standard python convention for docstrings requires that it is in the form of a block comment (using """). However, in allo functions comments blocks aren't properly ignored and throw an error.
Solution
Support for comment blocks would allow for standard doc strings to work.

Could you provide an example? I didn't quite get the issue here.

You mean you need to use docstring in an Allo kernel? This would need to enhance the parser

This code will give an unsupported constant type. It would be great if the parser could handle conventional block comments like this. Based on their possession they're automatically treated as docstrings which is a great quality of life feature when using functions/libraries you didn't write/write recently.

This is a min test case that breaks because of the block comment.
def min_test():
    def top(x_in: "int8[1]") -> "int8":
        """Test text"""
        return x_in[0]

    print(allo.customize(top).build()(np.array([5], dtype=np.int8)))```


Additional tests

Additional tests for cassandra-7875 and cassandra 7816, one fix for test of cassandra-8516

+1


⚠ Empornium has degraded performance

In a5480db, Empornium ($EMP) experienced degraded performance:

HTTP code: 500
Response time: 1056 ms

Resolved: Empornium performance has improved in 88be656 after 6 minutes.


Fixes for 1.3

This PR helps to support Meteor 1.3 on Bluemix.  It should correct this issue https://github.com/cloudfoundry-community/cf-meteor-buildpack/issues/20.
The trick is to use METEOR_WATCH_FORCE_POLLING to cause Meteor not to use inotify, which consumes file handles.  We have a a limited # on Bluemix.

Closing for now.  We did additional testing and are still running into the ENOSPC error when trying to deploy to Bluemix.


[C#] fix: Remove [] intializer notation

Linked issues
closes: #minor
Details
remove [] notation because it is not supported in version < C#12. That includes .NET 6 which is a framework we target.
Attestation Checklist

[x] My code follows the style guidelines of this project

I have checked for/fixed spelling, linting, and other errors

I have commented my code for clarity

I have made corresponding changes to the documentation (updating the doc strings in the code is sufficient)

My changes generate no new warnings

I have added tests that validates my changes, and provides sufficient test coverage. I have tested with:

Local testing
E2E testing in Teams

New and existing unit tests pass locally with my changes

Is there a reason why there are so many commits within the PR?

Is there a reason why there are so many commits within the PR?

Because I'm reusing the dotnet-dev branch and it hadn't been merged into main yet.


add feature：dump record to excel

add a feature：dump record to excel

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

@InterKnight Pls make a english version of readme, and we'd better change the name to readme.md.
In addition, please explain the application scenario of the function, can we not directly output to csv format

Pls make a english version of readme, and we'd better change the name to readme.md.
no problem
please explain the application scenario of the function, can we not directly output to csv format
As far as I know，if we want to parse some topic to file，we need write code for each topic because any topic’s proto is different，so I write this script to parse any topic.
I haven't support csv format now, but it can be added to TODO list
And I have two quesetions:
1, Do I need to make a new pull_requset for the changes above?
2, I see the apollo_iscan is faild, what can I do to make it pass?

@InterKnight Thank you for your contribution, we need to evaluate whether this tool need to merge into the main branch, in fact we may want to create a separate repository for tools.
Actually Apollo now supports packages, like python packages. The merging of projects will also be graded, and it will be decided whether to merge into apollo according to the usage of the tools. During this period, you can promote your tools.
A bit like an incubation mechanism for new projects.

Create a project, and it is really useful, and has a certain number of users
Submit the package and merge it into the official repository or recommended list

I think we are at the first stage, I think you can keep your repo and promote it in the community, we are always here to help!


WIP

Fixes https://github.com/flutter/flutter/issues/155795
Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] I followed the breaking change policy and added Data Driven Fixes where supported.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Opened https://github.com/flutter/flutter/pull/156181 to succeed this since I accidentally referenced this PR in the commit description of<PHONE_NUMBER>c72ad65d9c7ed6c3402e78f34cd25b, which is causing (and will continue to cause) this PR to get spammed with "X pushed a commit to <fork of flutter/flutter> that referenced this pull request" status updates.


Can't use EXCEPT on sequences, or Append on sequence-domain functions

Sometimes, you need to "reassign" part of a sequence in your next-state relation.
Here's a minimal example of what I mean (this spec does not work):
---- MODULE test ----
EXTENDS Naturals

VARIABLE seq

Init ==
    /\ seq = <<1, 2, 3>>

Next ==
    /\ seq' = [seq EXCEPT ![1] = IF @ = 5 THEN 1 ELSE @ + 1]

Spec == Init /\ [][Next]_seq

====

This doesn't work currently, because I imagine EXCEPT's implementation asserts that it was given a function or record.
On its own, it would be nice if this worked. I think, however, that this exposes a slightly more fundamental things that could be addressed.
The problem is that it doesn't work the "other way round" either. I can sort of work around the issue like this (this spec works):
---- MODULE test ----
EXTENDS Naturals

VARIABLE seq

Init ==
    /\ seq = <<1, 2, 3>>

Next ==
    /\ seq' = [ i \in DOMAIN seq |->
    	IF i = 1 THEN
        	IF seq[1] = 5 THEN 1 ELSE seq[1] + 1
        ELSE seq[i]]

Spec == Init /\ [][Next]_seq

====

The problem, then, becomes that I can't go back to using seq like a sequence. If I have a spec that uses both EXCEPT and Append on the same value in alternation, then that itself becomes a problem. See this last example, which also does not work:
---- MODULE test ----
EXTENDS Naturals, Sequences

VARIABLE seq

Init ==
    /\ seq = <<1, 2, 3>>

Inc ==
    /\ seq' = [ i \in DOMAIN seq |->
    	IF i = 1 THEN
        	IF seq[1] = 5 THEN 1 ELSE seq[1] + 1
        ELSE seq[i]]

App ==
	/\ seq' = Append(seq, seq[1])

Next ==
	\/ Inc
    \/ App

Spec == Init /\ [][Next]_seq

====

I guess, ultimately, it would be very nice if <<0, 0, 0>> = [i \in DOMAIN {1, 2, 3} |-> 0] (or, enough implementation edge cases to make this true for the user, at least).
Let me know if you'd like more test cases along this theme - I can probably come up with a couple more.

@fhackett-ms I believe most cases here should now be handled appropriately e.g. see the test cases here (which are run here). Please point out any other cases that still aren't working as expected, or close the issue if it is resolved for you.
Note: String values may be another edge case to deal with in this context, but it seems TLC may also have some unexpected behaviors in this case (e.g. see https://github.com/tlaplus/tlaplus/issues/512).


Revert "Added support for optional rmtree arguments"

Reverts mrbean-bremen/pyfakefs#1 (accidental merge)

Created some mixup after accidentally hitting the merge button on one change and creating a pull request for another, sorry.
Will resubmit the changes separately after this revert has been pushed.


Real-ESRGAN the fixed version

I sell the fixed tool the fixed version does not suffer from lags with the audio
<EMAIL_ADDRESS>

I sell the fixed tool the fixed version does not suffer from lags with the audio

Imagine trying to sell something that is free because of issues that you have yourself because you're too dumb to use it.
Nice scam.

need whan you convert it back to video too -r (framerate) for esragan input
5min google ffmpeg drop frames by frames input

ffmpeg -r 25.000 -i "D:\work\tmp\out_frames\frame%%08d.jpg" -c:v libx264 -r 25.000 -pix_fmt yuv420p "D:\work\output%%G.mkv"

@TechnoMasterBoy thx for improof :) https://github.com/kiam001/Upscale-bat-file/commit/1cb68eff927a91a076b7dc4ee84d99e46b2b5c0c

@TechnoMasterBoy thx for improof :) kiam001/Upscale-bat-file@1cb68ef

Nice. Btw I saw the -vf scale=1920x1080
You can set ESRGAN to upscale 2x or 3x instead of 4x, unless you need 4x to hit 1920x1080. It should speed up the upscale time for you.


Update Readme

Description

Update README.md.
Add NikhilRaj as a maintainer in the readme, as it is done in this one.

Screenshots
No response

Hey @nikhil25803 , i can do this . please assign it to me.
Thanks

Sure @rajnish254 , go ahead. :)

@nikhil25803 Created PR . Kindly check.

@nikhil25803 ,https://www.clueless.tech/profile/admin/86938309


stringifyUrl type should accept string[] to be consistent with parse

Example
  const queryStringParams = queryString.parse(search);
  const sourceInjectedQueryString = queryString.stringifyUrl({
    url: source,
    query: queryStringParams,
  });

Error
[tsserver 2322] [E] Type 'ParsedQuery<string>' is not assignable to type 'Record<string, string | null | undefined>'.
  Index signatures are incompatible.
    Type 'string | string[] | null' is not assignable to type 'string | null | undefined'.
      Type 'string[]' is not assignable to type 'string'.

This is a regression introduced by https://github.com/sindresorhus/query-string/commit/b15f945fc11ef46d34ba703856f5560112d310ea
ParsedQuery type has array
export interface ParsedQuery<T = string> {
	[key: string]: T | T[] | null;
}

UrlObject type does not support an array
export interface UrlObject {
	readonly url: string;

	/**
	Qverrides queries in the `url` property.
	*/
	readonly query: Record<string, string | undefined | null>;

	/**
	Overrides the fragment identifier in the `url` property.
	*/
	readonly fragmentIdentifier?: string;
}

Note
Additionally it should support number and boolean since these are also stringifiable.
The type of object argument in stringify should also match the type of query in stringifyUrl as they both stringify a record.
Proposed Fix
export type StringifiableType = string | boolean | number;
export type StringifiableRecord = Record<
  string,
  StringifiableType | StringifiableType[] | null | undefined
>;

export interface UrlObject {
  readonly url: string;

  /**
	Qverrides queries in the `url` property.
	*/
  readonly query: StringifiableRecord;

  /**
	Overrides the fragment identifier in the `url` property.
	*/
  readonly fragmentIdentifier?: string;
}

export function stringify(
  object: StringifiableRecord,
  options?: StringifyOptions
): string;

Your proposed fix looks good. Would you be interested in doing a pull request with it?

Your proposed fix looks good. Would you be interested in doing a pull request with it?

Happy to do that. Are you good with the terminology I used?


chore(main): release 0.5.3

:robot: I have created a release beep boop
0.5.3 (2024-07-24)
Bug Fixes

bump dependencies (e87579c)
bumping dependencies (cbe0916)
docker version tag (ef5e8ad)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/socialgene/sgnf/releases/tag/v0.5.3 :sunflower:


Edit profile components

Create input components to support various profile editing actions.  See pages 4-5: https://drive.google.com/drive/u/1/folders/15r012UCFhscDnstI0eEYjyAMDAW8AK44

@sbagg Current list of TODOs:

[ ] css grid doesn't have great IE 11 support.  Either add IE compatible styles or don't use css grid :( .  caniuse is a great resource for this stuff

https://caniuse.com/?search=grid-template-columns

[ ] rename text-field  file element to something in line with element name
[ ] add js docs to functions (https://jsdoc.app/)

just method, description, param and returns are required.  Example: https://github.com/ucd-library/rp-ucd-client/blob/main/lib/esm-utils.js#L24

[ ] website row delete button does not remove correct item
[ ] Use iron-icon for icons for sort icon<EMAIL_ADDRESS>[ ] Use button element not divs when its a button
[ ] Use defined header and padding styles defined in RP Design Guide (see google drive link at start of issue)


compactor: make event ordering well-defined in TestPeriodicPause

Fixes #5847
Advancing the clock before the rg.Wait in the first loop means the compactor could receive the clock tick either before or after the unpause. On the rare occasion of receiving the clock tick before the unpause, the test fails with an early revision.

lgtm


cortex xdr logs not populating

Forwarding logs from Cortex XDR to Splunk. Logs are received on splunk but not populating on any of the dashboard sections for Traps or Endpoint Security. Is this not supported yet?

We are working on Cortex XDR support in the Splunk App and Add-on. You can track progress at the Cortex XDR Pull Request: https://github.com/PaloAltoNetworks/Splunk-Apps/pull/166


Add missing import in the api route definition example

🔗 Linked issue

❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

yeah, sorry, did not know at the moment that implicit imports is a part of Nuxt culture


Failed install with python -m pip install -e DynamicHead

Installing collected packages: dyhead
  Running setup.py develop for dyhead
    ERROR: Command errored out with exit status 1:
     command: /public/home/hpc2/anaconda3/envs/mask/bin/python -c 'import io, os, sys, setuptools, tokenize; sys.argv[0] = '"'"'/public/home/hpc2/rpc/DynamicHead/setup.py'"'"'; __file__='"'"'/public/home/hpc2/rpc/DynamicHead/setup.py'"'"';f = getattr(tokenize, '"'"'open'"'"', open)(__file__) if os.path.exists(__file__) else io.StringIO('"'"'from setuptools import setup; setup()'"'"');code = f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' develop --no-deps
         cwd: /public/home/hpc2/rpc/DynamicHead/
    Complete output (71 lines):
    which: no nvcc in (/public/home/hpc2/anaconda3/envs/mask/bin:/public/home/hpc2/anaconda3/condabin:/usr/lib64/qt-3.3/bin:/public/home/hpc2/perl5/bin:/public/software/Modules/bin:/usr/local/bin:/usr/bin:/usr/local/sbin:/usr/sbin:/opt/ibutils/bin:/public/home/hpc2/.local/bin:/public/home/hpc204712181/bin)
    running develop
    running egg_info
    writing dyhead.egg-info/PKG-INFO
    writing dependency_links to dyhead.egg-info/dependency_links.txt
    writing top-level names to dyhead.egg-info/top_level.txt
    reading manifest file 'dyhead.egg-info/SOURCES.txt'
    writing manifest file 'dyhead.egg-info/SOURCES.txt'
    running build_ext
    building 'dyhead._C' extension
    gcc -pthread -B /public/home/hpc2/anaconda3/envs/mask/compiler_compat -Wl,--sysroot=/ -Wsign-compare -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -fPIC -I/public/home/hpc2/rpc/DynamicHead/dyhead/csrc -I/public/home/hpc2/anaconda3/envs/mask/lib/python3.7/site-packages/torch/include -I/public/home/hpc2/anaconda3/envs/mask/lib/python3.7/site-packages/torch/include/torch/csrc/api/include -I/public/home/hpc2/anaconda3/envs/mask/lib/python3.7/site-packages/torch/include/TH -I/public/home/hpc2/anaconda3/envs/mask/lib/python3.7/site-packages/torch/include/THC -I/public/home/hpc2/anaconda3/envs/mask/include/python3.7m -c /public/home/hpc2/rpc/DynamicHead/dyhead/csrc/vision.cpp -o build/temp.linux-x86_64-3.7/public/home/hpc204712181/rpc/DynamicHead/dyhead/csrc/vision.o -DTORCH_API_INCLUDE_EXTENSION_H -DTORCH_EXTENSION_NAME=_C -D_GLIBCXX_USE_CXX11_ABI=0 -std=c++11
    cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for C/ObjC but not for C++
    In file included from /public/home/hpc204712181/rpc/DynamicHead/dyhead/csrc/vision.cpp:2:0:
    /public/home/hpc2/rpc/DynamicHead/dyhead/csrc/deform_conv.h:10:5: error: ‘at’ has not been declared
         at::Tensor input,
         ^
    /public/home/hpc2/rpc/DynamicHead/dyhead/csrc/deform_conv.h:11:5: error: ‘at’ has not been declared
         at::Tensor weight,
         ^

error: command 'gcc' failed with exit status 1
Error seems that at::Tensor can't be recognized, how to solve this problem?
My gcc version is 5.4.0

@XiaoyuWant How did you solve this issue?

@XiaoyuWant I encountered the same error as yours. How did you solve it？

@kachiO  Upgrade pytorch version to 1.3 1 solved my problem


Push changes from dev branch to main

Basically, actually made the thing. Previous work was deeply unfinished. This is now a viable product - not minimum, but not maximum.

Send it ✅


Update request: rocmPackages.hipblas  5.7.1 → 6.0.0

Package name: rocmPackages.hipblas
Latest released version: 6.0.0? hipBLAS 2.0.0 for ROCm 6.0.0

Current version on the unstable channel: 5.7.1
Current version on the stable/release channel: 5.7.1

[x] Checked the nixpkgs pull requests

Well, there is https://github.com/NixOS/nixpkgs/issues/197885 and there was acknowledgement of

ROCm 6.0.0 has been released.

And I guess it's pretty relevant.
Also worth to mention that I create this issue because Ollama's update needs libhipblas.so.2 library.
Notify maintainers
@Madouura
@Flakebi

Note for maintainers: Please tag this issue in your PR.

Add a :+1: reaction to issues you find important.

duplicate of: https://github.com/NixOS/nixpkgs/issues/280927

This should be closed as not planned, since it's a duplicate.


Server-side validation

Hi.
After client-side validation is ok the form data is being validated by the server side. The errors are returned in the following json format, e.g.:
{
    field1: errorString,
    field2: errorString,
    ...
}

Is there any way to tell vue-validator that these fields are invalid?

Sorry for the late reply.
There is not way to tell the server-side validation errors in vue-validator.
I'll try to consider this feature in v2. :smiley_cat:

Is there any workaround to repopulate errors in validator on 400 response when I am getting form server http status with well described validation errors? Can I somehow access field and manualy add validation errrors for field?
{"confirmPassword":[{"memberNames":["ConfirmPassword"],"errorMessage":"Field 'Confirm Password' can't be empty."}],"acceptTerms":[{"memberNames":["AcceptTerms"],"errorMessage":"Must be checked 'Accept Terms'."}]}


[Request] Thumbs-Up/Favorite Support?

Description
It would be nice to be able to heart/thumbs-up songs when hovering over the cover image to control media.
Additional notes/images
YTMDesktop does this pretty well

Technically it would be possible to add this feature to WebNowPlaying supported players, but since this feature doesn't work on local tracks & it's not supported on all players even with the plugin it will not be added.
If you want to add this functionally yourself, you can start here: https://docs.rainmeter.net/manual/getting-started/creating-skins/


City of Vaughan, Ontario, CA

https://maps.vaughan.ca/arcgis/rest/services/PLANitTest/PLANit/MapServer/9

My people :)

Huh, our Mapbox tiles aren't working.


windows graphics resolve result when encountered an error

if encountered an error in the command, promise status will always be pending, I think resolve a result is better

Fixed ...


Failing to upload big files to bucket

Thanks for stopping by to let us know something could be better!
Environment details

OS: alpine 3.16
Node.js version: 18.10.0
npm version: 18.10.0
@google-cloud/storage version: 6.9.4

Steps to reproduce

Upload file of size 1GB
Getting error Cannot create a string longer than 0x1fffffe8 characters

The same issue as described on stackoverflow I think this is googleapis/nodejs-storage responsibility to chunk the file and upload it accordingly not an external user.

Hi @dawidreedsy can you please share a snippet showing how you are attempting to upload the file and a full stack trace if you have it. We have not had any recent issues regarding file sizes and I routinely test with files > 1GB.

Worth to mention that after we get this error logged, our instance is blowing up from Out Of Memory error, I think you may need to try in memory restricted environment.

Can you share a bit about the environment specs that your code is running in? I suspect you are right, this likely is due to resource constraints / environment specific.
I just ran this test in my local environment:
import {Storage} from '@google-cloud/storage';

const s = new Storage({projectId: 'my-project'});
const b = s.bucket('my-bucket');

try {
    await b.upload('2-GB-File.dat', {destination: 'large-file.dat', resumable: false, validation: 'crc32c'});
  } catch(e) {
    console.error(e);
}

Hi @ddelgrosso1 . Were you able to handle this issue or come up with a solution? I have the same issue within my project too.

@siva54 / @dawidreedsy can either of you do a npm ls node-fetch and tell me what version you see?

@ddelgrosso1
**** OUR LIB *****
├─┬<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ └──<EMAIL_ADDRESS>│ └─┬<EMAIL_ADDRESS>│   └──<EMAIL_ADDRESS>deduped
└─┬ *** OUR OTHER INTERNAL LIB ***
  └─┬<EMAIL_ADDRESS>    └─┬<EMAIL_ADDRESS>      └──<EMAIL_ADDRESS>deduped

Is there any other processing of the file you are doing?

Yes we doing quite a bit processing on the instance, as this is a worker that handles larger tasks, however each instance is processing only one task at time, so there shouldn't be any concurrency issue.

I saw that you have option chunkSize tomorrow morning will try that and see if it helps.

From the trace you provided above it is leading me to this line: https://github.com/node-fetch/node-fetch/blob/1ef4b560a17e644a02a3bfdea7631ffeee578b35/src/body.js#L132. However, when I step through this in a debugger the times that this is hit for a response the size of the buffer is no where near 512MB/1GB. What type of files are these? Are you setting any kind of header information?

From the trace you provided above it is leading me to this line: https://github.com/node-fetch/node-fetch/blob/1ef4b560a17e644a02a3bfdea7631ffeee578b35/src/body.js#L132. However, when I step through this in a debugger the times that this is hit for a response the size of the buffer is no where near 512MB/1GB. What type of files are these? Are you setting any kind of header information?

They are zip files, no any specific header information.

From the trace you provided above it is leading me to this line: https://github.com/node-fetch/node-fetch/blob/1ef4b560a17e644a02a3bfdea7631ffeee578b35/src/body.js#L132.

yeah the response would need to contain the whole zip file..., maybe some weird error just tries to dump the content of the file to error response?

@ddelgrosso1
FYI we added the option chunkSize = 100 * 2**20 //100Mb and it works without any issue

@ddelgrosso1
I actually was a bit wrong in my last example of when this happens as we also had makePublic call.
    const options = {
      destination: destination,
      resumable: false,
      validation: 'crc32c',
    };

try {
  const [file] = await this.bucket.upload(filePath, options);
  await file.makePublic()
} catch(err) {
  logger.error('Fails here')
}

I actually removed makePublic call and added chunkSize, so it might be make public call that causing it.  The options looks like this now:
    const options = {
      destination: destination,
      resumable: false,
      validation: 'crc32c',
      chunkSize = 100 * 2**20,
      public: true,
    };

I doubt it could come from makePublic request, but yeah this is the 2 things i changed, and no more errors

@dawidreedsy is there anyway you could log the raw response from the call to upload? I'm at a loss for why you are seeing such a large response which should not be the case.

@ddelgrosso1
Will try to find some time to get it for you, but cannot promise.

@ddelgrosso1
No it is not happening after I used the chunk option.
Sorry, couldn't reproduce it on any testing environment. I guess it needs to be under pressure to throw this error and I cannot put the error back to production to catch the body.

Going to close this for now as we are unable to reproduce it. If any additional information becomes available that might help us reproduce / debug, please feel free to reopen.


Order states fixture by name,id

Coverage remained the same at 100.0% when pulling 2de1fb6ed7148bb33549223e70a4dc50c5c420e2 on ryankaiser:meta into 88d780d904a862b015637b3fa07859a6a9680976 on alphageek-xyz:master.


Migrate Project to the Camunda Community Hub?

Hello! I hope today finds you well! My name is Rin Oliver, and I'm a Technical Community Builder at Camunda. I'm opening this issue today to ask if you might be interested in migrating this extension over to the Camunda Community Hub?
We'd love to hear your feedback on how we might improve the overall community extension development process for maintainers and contributors moving forward, and learn more about what you'd like to see from this initiative in the future.
If migrating this extension something you'd be interested in, I can send you the instructions for doing so via email, or whatever communication method works best for you. If you are for any reason no longer able to maintain this extension, if it is migrated to the Hub I would be more than happy to add a lifecycle badge to it indicating that the project is in need of a new maintainer.
I'm also more than happy to schedule a call and talk over this in more detail if that would be preferable to you, for whenever best suits your schedule over the next few weeks.
Thank you so much for your time. I look forward to hearing from you.

@celanthe
I will be happy to migrate this project to Camunda Community Hub.

Hi @yogeshrnaik! That's fantastic news. I've sent you an invite to become a Member of the Community Hub GitHub Organization. From there, you should be able to follow the Getting Started guide in the README, starting with https://github.com/camunda-community-hub/community#transferring-a-repository-in-your-personal-github-account. :)
If you run into any issues, or have any questions, please don't hesitate to reach out. I am happy to help!
Best,
Rin

Hi @yogeshrnaik! I hope today finds you well. I just wanted to circle back on this! :) if you need a new invitation to join the Community Hub, I'll be more than happy to send you one! I look forward to having this extension in the Community Hub, and thank you again for your wonderful contributions to Camunda!

@celanthe I have created https://github.com/yogeshrnaik/camunda-external-task-client-python3/issues/51 for initiating the transfer. Please let me know if I need to do anything else?

Hi @yogeshrnaik! That's fantastic, I've replied to you with next steps over on #51. We look forward to having this extension in the Community Hub! :)

@celanthe
Repository is transferred to Camunda Community Hub.


[FLINK-31890][runtime] Introduce SchedulerBase per-task failure enrichment/labeling

https://issues.apache.org/jira/browse/FLINK-31890

Introduce async task failure labeling as part of SchedulerBase#updateTaskExecutionState
Introduce SerializableCompletableFuture
Propagate labels as part of TaskExecutionStateTransition
Extend Execution, ErrorInfo, FailureHandlingResult, FailureHandlingResultSnapshot to expose labels as part of ExceptionHistory and Restart strategies
Extend existing tests to validate functionality

Thanks for the PR @pgaref; this already looks pretty good. I've left a couple of comments. My biggest concern is not having a dedicated test in the JobMaster test suite.

Thanks for the comments @dmvk ! Addressed as part of  0365fb5  and opened FLINK-31993 to track the configuration changes, PTAL

Thanks for creating this PR! @pgaref I have a few comments. Mainly about that some task failures are not labeled. Please take a look.

Thanks for the comments @zhuzhurk  and @dmvk  !
Latest PR introduces async task failure labeling for DefaultScheduler (Adaptive and Global failures are follow ups) as part of SchedulerBase#updateTaskExecutionState -- this also covers the InternalFailuresListener cases as the failures are eventually propagated to the scheduler.
Decided to pass a CompletableFuture<Map<String, String>> all the way down to ExceptionHistoryEntry that we can decide on the JobExceptionHandler how to handle for the Rest/UI endpoints.
This approach is also giving the flexibility to restart strategies in the future to block/wait for label results if they want to, or just ignore them.
Keep in mind I had to to introduce a SerializableCompletableFuture class as ErrorInfo is a Serializable class and we cant break that contract.
Please let me know what you think.

Thanks for the comments once again @zhuzhurk !
PTAL on latest PR:

introducing async task failure labeling as part of ExecutionFailureHandler#handleFailure (for local and global failures)
transient CompletableFuture with serializable labels as part of ErrorInfo

Thanks @dmvk and @zhuzhurk for the valuable comments!
This is now good to go :)


[WIP] ci: move from Travis to GH Actions

Description
GitHub Actions are more powerful, flexible and popular.
We should move CI to the native GH mechanism on the platform.
Fixes #74
Type of change

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

Checklist
Please carefully read these items and tick them off if the statements are true
or do not apply.

[x] I have performed a self-review of my own code
[x] My code follows the existing coding style, lints and generates no new
warnings
[x] I have added type annotations to all function/method signatures, and I
have added type annotations for any local variables that are non-trivial,
potentially ambiguous or might otherwise benefit from explicit typing.
[x] I have commented my code in hard-to-understand areas
[x] I have added ["Google-style docstrings"] to all new modules, classes,
methods/functions or updated previously existing ones
[x] I have added tests that prove my fix is effective or that my feature
works
[ ] New and existing unit tests pass locally with my changes and I have not
reduced the code coverage relative to the previous state
[x] I have updated any sections of the app's documentation that are affected
by the proposed changes

If for some reason you are unable to tick off all boxes, please leave a
comment explaining the issue you are facing so that we can work on it
together.

This is still a WIP but I need @uniqueg to take a look at the following issues:

pylint raises issues, not sure how the current dev_v2 passed since I did not touch the code at all.
coveralls requires GITHUB TOKEN specification

I did not include yet the following part:
- git config --global user.name "semantic-release (via TravisCI)"
- git config --global user.email "semantic-release@travis"
- semantic-release publish -v DEBUG

Is that required? What is this for?
Could you please unhook the repo from the Travis account?

This is still a WIP but I need @uniqueg to take a look at the following issues:

pylint raises issues, not sure how the current dev_v2 passed since I did not touch the code at all.
coveralls requires GITHUB TOKEN specification

I did not include yet the following part:
- git config --global user.name "semantic-release (via TravisCI)"
- git config --global user.email "semantic-release@travis"
- semantic-release publish -v DEBUG

Is that required? What is this for?
Could you please unhook the repo from the Travis account?

Thanks a lot @AngryMaciek. I have made you admin and removed the checks for Travis and Coveralls that didn't run. All other issues are addressed in zavolanlab/zarp-cli/pull/27, as well as the review of the GH Actions configs (just the one comment re: style.yml naming).

Also please mind this comment

Codecov Report

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

@@            Coverage Diff             @@
##             dev_v2       #75   +/-   ##
==========================================
  Coverage          ?   100.00%
==========================================
  Files             ?         9
  Lines             ?       506
  Branches          ?         0
==========================================
  Hits              ?       506
  Misses            ?         0
  Partials          ?         0

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 bdccede...ca10686. Read the comment docs.

Hey @uniqueg,
Included suggested changes, please see this post for more info.
One thing different here is that in this case the semantic release CI workflow fails because, unlike for zarp-cli, you did not set release configuration in setup.cfg.


[CT-1420] [Bug] Ref creates a new relation every time, meaning that model data (such as type cannot be inherited)

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
At the moment, ref creates a new relation object each time rather than checking if something already exists.
Context for the problem this is causing is in slack:
https://getdbt.slack.com/archives/C50NEBJGG/p1666805781564349
This is something that I've run into when developing the duckdb adapter.
https://github.com/jwills/dbt-duckdb/issues/19#issuecomment-1292375802
The relevant code is: https://github.com/dbt-labs/dbt-core/blob/19167842873ea882c2dbb70aeb525953812d8ef3/core/dbt/context/providers.py#L463
which uses:
https://github.com/dbt-labs/dbt-core/blob/19167842873ea882c2dbb70aeb525953812d8ef3/core/dbt/adapters/base/relation.py#L243
Expected Behavior
If model x has a materialization of foo (and the returned relation from the materialization sets this up correctly) i'd expect ref('x') to be a relation of type foo, up to the enum being supported under the hood etc.
Steps To Reproduce
Make a relation (via a model) that has a type that is not None, and then ref that model. A relation of type none is returned, with the same name etc
Relevant log output
No response
Environment
No response
Which database adapter are you using with dbt?
No response
Additional Context
No response

@jpmmcneill Thanks for opening!
The change you propose in #6175 makes sense to me. In the past, I think I've written extra code to achieve the same effect:
{% set ref_model = ref('some_model') %}
{% set with_type = load_relation() or ref_model %}

Probably better if this is something dbt does natively, and best yet if it can be a cache lookup!

Left a lengthier comment in https://github.com/dbt-labs/dbt-core/pull/6175#discussion_r1013259582. I'm come around to thinking that this isn't actually a bug! If we close the issue, we should definitely update the documentation:

https://docs.getdbt.com/reference/dbt-jinja-functions/ref
https://docs.getdbt.com/reference/dbt-classes#relation

Not complete but closing because this isn't really a bug as @jtcohen6 said + the use case I was looking at this for was solved a different way :duck:


JS errors on my 2016-ish MacBook Air

My JoyCons arre connected in Bluetooth to the computer:

However Chrome and Firefox fail on navigator.hid:

Chrome 87.0.4280.88 (x86_64)
Firefox 82.0.2 (64 bits)

Same for Safari

Safari Version 14.0 (156<IP_ADDRESS>.9, 15610)

Thanks for the report. This was just fixed with https://github.com/tomayac/joy-con-webhid/commit/a19e541b365305eb440998b3a329989ddb361952. The tl;dr is that you need an origin trial token (or set the experimental web platform features flag in Chrome). The demo was missing the token. Please try again now.

It works!! Thanks

Though for some reason on the left joycon the buttons work, but not the accelerometer/gyroscope.

Glad to hear. Try re-pairing and reloading the page. Wake up the Joy-Cons before by pressing buttons.
Also plug them in your Switch and update the firmware. It’s become a lot better for one of my pairs when I did that.


allow the use of double slash flow comments

Description of the Change
Currently, a Flow comment needs to be used with the following syntax: /* @flow */. This PR allows the usage of // @flow as well.
Why Should This Be In Core?
To allow flexibility the same way that both 'use babel' and "use babel" are supported.
Benefits
It's a convenience.
Possible Drawbacks
It is kind of a trivial addition.
Verification Process
I wrote a test case for this, but I was having npm test or script/test to work for me. I got an error like so:
AssertionError [ERR_ASSERTION]: More than one application to run tests against was found.

I'm not sure how I'm supposed to develop on Atom if I already have Atom installed on my computer which I imagine is causing this error.
Applicable Issues
N/A

The instructions for how to test changes to Atom Core are in the Flight Manual.

Since tests (including the new one) pass and the change is very straightforward, I'm ok 🚢 this at this point. You cool with that @lee-dohm ?


[New] jsx-curly-brace-presence: add "propElementValues" config option

Fixes #3184

Codecov Report

Merging #3191 (da13211) into master (541ea43) will decrease coverage by 3.44%.
The diff coverage is 92.30%.

:exclamation: Current head da13211 differs from pull request most recent head 7e2bf85. Consider uploading reports for the commit 7e2bf85 to get more accurate results

@@            Coverage Diff             @@
##           master    #3191      +/-   ##
==========================================
- Coverage   97.55%   94.11%   -3.45%
==========================================
  Files         120      120
  Lines        8288     8303      +15
  Branches     2982     2988       +6
==========================================
- Hits         8085     7814     -271
- Misses        203      489     +286

Impacted Files
Coverage Δ

lib/rules/jsx-curly-brace-presence.js
98.07% <92.30%> (-0.89%)
:arrow_down:

lib/util/propTypes.js
67.74% <0.00%> (-29.68%)
:arrow_down:

lib/util/ast.js
69.82% <0.00%> (-26.63%)
:arrow_down:

lib/rules/jsx-no-comment-textnodes.js
88.88% <0.00%> (-11.12%)
:arrow_down:

lib/rules/no-arrow-function-lifecycle.js
88.33% <0.00%> (-10.00%)
:arrow_down:

lib/rules/hook-use-state.js
90.76% <0.00%> (-9.24%)
:arrow_down:

lib/rules/boolean-prop-naming.js
91.97% <0.00%> (-7.30%)
:arrow_down:

lib/rules/jsx-props-no-multi-spaces.js
94.11% <0.00%> (-5.89%)
:arrow_down:

lib/rules/jsx-no-literals.js
95.65% <0.00%> (-4.35%)
:arrow_down:

lib/rules/no-unused-prop-types.js
90.00% <0.00%> (-4.00%)
:arrow_down:

... and 9 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 541ea43...7e2bf85. Read the comment docs.


Rename tab title not working for WSL and Bash

Hi :)
Bug report
I use the latest Windows Terminal with PowerShell, CMD, Ubuntu (WSL) and Bash (Git Bash for Windows) and I can't change the title of the Ubuntu and Bash tab in the profiles.json file.
I saw that it was implemented in #1358, but it doesn't work in the cases I described.
And why this?
Is there a solution?
Configuration and Execution
In profiles.json file:

In Windows Terminal:

But it's working for CMD and PowerShell:

Turn on the suppressApplicationTitle option. The "tab title" is a recommendation, and the application can delete it if it wants to. Bash usually does. supressApplicationTitle overrides bash.

Oh, right. It worked.
Thanks 😄


requirements: ipython not found and is required by YOLOv5, attempting auto-update...

Add to reqs

Also requirements: psutil not found and is required by YOLOv5, attempting auto-update...


Migrate from simulation/interaction/collaboration to activity/groupActivity

We will deprecate the entry points simulationOf, interactionOf, collaborationOf, and unsafeCollaborationOf, and replace them with new entry points activityOf, groupActivityOf, and unsafeGroupActivityOf.
The new entry points differ in that they do not take a separate time-step function as an argument.  Instead, the time step function is included in Event as a new constructor called TimePassing.

One wart in this approach is that TimePassing in groupActivityOf must be accompanied by a player number.  Currently, that number is (-1) for haskell-mode, and 0 for codeworld-mode.  This seems like as good an answer as any.

Still to do:

Update online documentation.
Try to find and ask for updates to most of the commonly used resources on the web.  (CIS 194 will be an important one.)
Deprecate simulationOf, interactionOf, collaborationOf, and unsafeCollaborationOf.
Wait 2 years before removal.


Replaced hard colors with Vim's Syntax groups

Original Execute commands have been throwing an error. "g:lovedocs_colors undefined variable"
I've replaced these lines with "Syntax link" commands instead, linking the groups to the functions instead.

Are you using a plugin manager? I'm not sure why there would be a problem with the color being undefined, since it's defined here

Oh. Didn't think that would be the case but yes, I'm actually using Vundle.

I'm not familiar with how Vundle works, I'll have to take a look

Hey, can you check to see if the help file works right? My though is that Vundle may be naming the project vim-love-docs then looking for files with that name instead of love like I have named them. Can you try renaming the file to what Vundle downloads it as? I.e. if it's vim-love-docs\plugin\love.vim can you renaming the file to vim-love-docs.vim and see if that works?

Seems to be working just fine now. Even the syntax highlight works as intended.
Guess I should close the pull request then.


PanNuke trained weights

Could you please tell  on which fold pannuke model (that is provided with the repo) was trained?

For the version supplied in the repo, we selected the fold with the highest validation performance. Then, we fine tuned it further for a few epochs on the entire dataset.


Native editor scroll bar too small to use

Had a ton of feedback about how small our notebook scrollbar was and that it was difficult to use (evenmoreso if you run dual monitors and VS Code is in the left monitor as you'll often accidently move your moues to the right screen rather than to the scroll bar). Also, its even harder when the cell output scroll bar and the notebook editor scrollbar is side by side.
Solution: Increase the width of the global notebook editor scrollbar

Looks like side spacing is messed up (we have a hardcoded padding I believe).
Additionally, what do we do for inner scrollbars? Perhaps use the same value?

Pic of what it looks like with size 72:
    "editor.scrollbar.verticalScrollbarSize": 72

Notice how the delete button has been hidden.

The inner scrollbar should have the same width as the global editor scrollbar. Also there should be a bit more padding between the right end of the cell and the side of VS Code, as right now theres not too much padding.

Had to make a large number of changes to go along with this so for validation any extra ui checking would be good.

I think the 72px wide scrollbar looks way too big, I think something like 16px for both would be good, since i think both scrollbars are <16px right now. And then just more padding between the inner and outer scrollbar.

@jmew Lol, That's just an example showing that I respect the editor setting even if it's set crazy high :). The default is 14 for vert and 10 for horizontal just like vscode.
Also this wasn't mentioned, but I also added in the vscode horizontal scrollbar setting support.

Validated.


Todo app edit functionality

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes:  #570

Type of change

[x ] Bug fix (non-breaking change which fixes an issue)
[x ] New feature (non-breaking change which adds functionality)

Checklist:

[ x] I have made this from my own
[ x] I have taken help from some online resources
[ x] My code follows the style guidelines of this project
[ x] I have performed a self-review of my own code
[ x] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

@VidhiBhatt01  maam ,  This is the a new fresh PR contains everything you told.
Should I have to make it responsive also, since this todo app was not responsive from beginning , if I will do that extra work , will I get extra point for that work or should I create new issue for responsiveness??

https://deploy-preview-623--amazing-js-projects.netlify.app/todo app/
You can create a new issue and make it responsive. Allowing to create a new issue as the template was not responsive from earlier

Ok maam then, merge it ,I am creating a new issue for that.

@VidhiBhatt01  maam, merge it, i will make it responsive responsive the next issue in which i am assigned by you

@mayank-del make it responsive in this PR itself

@mayank-del make it responsive in this PR itself

Actually, I am just asking that , if I will make it responsive in this PR, then i will get extra points or not?
Because , In this issue , i have to add only editing feature, i made it, but for making it responsive, i should get some more points na sir, so that my rank would little bit more up
If that will be the case , then that is even more easy for me

@mayank-del make it responsive in this PR itself

Actually, I am just asking that , if I will make it responsive in this PR, then i will get extra points or not?
Because , In this issue , i have to add only editing feature, i made it, but for making it responsive, i should get some more points na sir, so that my rank would little bit more up
If that will be the case , then making responsive in this PR will be more convenient for me.

@arpit456jain , @VidhiBhatt01  ,please clarify this doubt of mine , please, I don't know about points distribution system of gssoc , so please help me with this doubt.

@VidhiBhatt01  maam and @DEEPALI-SUK  , what happened ?
I added everything

delete extra folders and files
still not responsive
add in the style_links.json file of the main repo, do not make ur own

maam , in phone it is responsive , but it's initial state comes as zoom-out, so just zoom-in it,then, you will see it's original look

Why Spotify Clone folder is there

update ur branch

Why Spotify Clone folder is there

Maam, I deleted those folders, but @arpit456jain  told to add those back again, otherwise, they will be lost so i have to commit them inside this folder again.

Why Spotify Clone folder is there

update ur branch

Update means???

@mayank-del  update ur branch and add folder again and make a fresh PR


rooled back previous attempt en fixed RUM

env.js is now written by the code

appologies... wrong repo


its very slow when selecting an item,when fetching data from API, because re-rendering when choose item, please any one can help???

Issue summary
Add a short description of issue, preferably, a sentence.
Library versions

Steps to Reproduce
(Write your steps here:)

Expected Behavior

(Write what you thought would happen.)
Actual Behavior

(Write what happened. Add screenshots!)
Reproducible Code
(Paste exact code snippet and instructions to reproduce the issue.)

the problem may be in the ScrollView and the map function to render the children. you would have to modify all that part using a FlatList.


Add improvements from the review

This PR contains the followings,

Fix references
Restructure the Database configurations
Add instructions for configuring the Trusted certificates for GoRouter

Thanks @ThilinaManamgoda, will review these today.


Line coverage metrics do not add up for executed_lines and summary.covered_lines in generated coverage.json

Describe the bug
Here below is an item in "files" generated from coverage.py:

"src\textual\widgets\_list_item.py": {
"executed_lines": [
1,
3,
5,
6,
7,
8,
11,
12,
19,
20,
22,
23,
25,
29,
32,
35,
36,
37
],
"summary": {
"covered_lines": 15,
"num_statements": 21,
"percent_covered": 71.42857142857143,
"percent_covered_display": "71",
"missing_lines": 6,
"excluded_lines": 0
},
"missing_lines": [
26,
27,
30,
33,
38,
39
],
"excluded_lines": [],

I would expect len(executed_lines) == summary.covered_lines but the first is 18 and the later is 15. Where does the extra 3 come from?
To Reproduce
I tried this with the following:
pytest . --tb short  --color no --cov=src\textual  --cov-report json --continue-on-collection-errors --disable-warnings
On the https://github.com/Textualize/textual repo.
Expected behavior
My expectation is:
covered_lines + executed_lines + excluded_lines == summary.num_statements, but this seems to not be the case?

I think the three lines are the docstrings, but I'm not sure why they are treated differently.

Yeah so here's the repo: https://github.com/JohnPeng47/textual-neutered

Clone
Install poetry in your current python env
cd into repo and run poetry install to get deps
run poetry shell to activate project venv with deps installed from 3
run pytest . --tb short  --color no --cov=src\textual  --cov-report json --continue-on-collection-errors --disable-warnings
[OPTIONAL] I have a script test_coverage.py that basically checks that this conditions is true for all coverage files:
missing_check = len(missing_lines) + len(executed_lines) + len(excluded_lines) == total_stmts

Also a shorter example here:

"__init__.py": {
  "executed_lines": [
    0
  ],
  "summary": {
    "covered_lines": 0,
    "num_statements": 0,
    "percent_covered": 100,
    "percent_covered_display": "100",
    "missing_lines": 0,
    "excluded_lines": 0
  },
  "missing_lines": [],
  "excluded_lines": [],

In this example, both covered_lines and num_statements should be 1 right, not 0?

Also another thing that I just noticed now, is that the coverage collected on the same repo is different on Windows vs. Linux. Is this just inherent to the OS differences in settrace impl?

There is no OS difference in the settrace implementation.  Likely the difference is do to OS differences in textual repo.  If you don't think that's the case, let's look at some more specific details.

Ahh no I think that makes sense. But could you confirm:
missing_check = len(missing_lines) + len(executed_lines) + len(excluded_lines) == total_stmts
Is invariant for every file under coverage?

Hey sorry to keep pestering, but do you think you can just answer that one question above? My coverage diffing code is broken for some reason, and I just want to be sure that the above condition should hold


Allow to configure the pagination limit from server

The current pagination flow has a limit of 10k since elasticsearch set a default safeguard limit when requesting docs with a high from value.
But this value can be increased by configuration. The problem is, even if the user defines a higher value on the elasticsearch level, the change won't be reflected.
We should allow adapting this value to the setup in the elasticsearch cluster.
See also discussion here

Closed in favor of a more robust solution mentioned here https://github.com/argilla-io/argilla/issues/2484


#15,#16 Removing useless buttons, added note and centered components

@Gabbell It looks good! But when we click on 'Register", the note at the bottom of the screen should specify that they wouldn't be able to synchronize to google calendar, if they don't use their 'gmail account'. Now it currently just says 'any email', so it might confuse the user.

Okay i added that small change

How about this:
Note: You will not be able to synchronize with your Google Calendar if you do not use your gmail account.
Both on login and register

Yes, better!


Add equals and hashCode methods to DiscoveryFilter

It's simplified SSDPDiscoveryProvider removeDeviceFilter method and added equals/hashCode methods into DiscoveryFilter. There are also some new test for checking this method. It is based on the comment https://github.com/ConnectSDK/Connect-SDK-Android-Core/commit/5d16c5bb976c35c89b0682009b8de6812e69f0c9#commitcomment-9689474

I've updated ZeroConfDiscoveryProvider as well. Unit test were also added for covering DiscoveryFilter, comparison is made only by service id.


{Monitor}fix help message

Fix #1276

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:
docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT "git+https://github.com/MyronFanQiu/azure-cli-extensions.git@fix-1276-add-mandatory#subdirectory=src/$EXT&egg=$EXT"

add to S168


Install react-yearly-calendar in next-js

I can't find a way to install this in next js, since you can't import css globally outside of _app.js. The css file throw me lot of error like :
`Syntax error: Unclosed block
175 |   }
176 |

177 |   div.options select {
|   ^
178 |     margin-left: 10px;`

Any help on this ?

Hey @PierreVelluet
I am sorry but I have no experience in next js. Maybe you could try importing the css in _app.js?


No goto definition within the lambda functions

Description:
Language Server Goto Definition does not work when there is a lambda function which is not having rest params
Affected Versions:
1.1.0-alpha at least

Closing since the fix has been merged to v1.1.0


Please  keeping current with OpenSSL (v3.0.6)

3.0.6 was withdrawn by upstream openssl; so although there's a version, we don't recommend using it.
When 3.0.7 is released, quictls will also be released.

Thank you

3.0.6 was withdrawn by upstream openssl; so although there's a version, we don't recommend using it. When 3.0.7 is released, quictls will also be released.

Thank you.


SRDD dataset interpretation

Hi, very impressive work. Can you provide column names in the check.csv file for better understanding of the results.

We have updated the file head for check.csv and we also give instruction on the readme of SRDD.


add logs window

adds logs call out and window pop up upon truss watch
https://www.loom.com/share/9d8dd9e285fd45ba8926be471b005d40?sid=1b366d2c-a353-49bd-97a5-2a3b164ba458

okay to merge. I don't love our graphql query stuff right now. Let's make a clean up ticket

agreed - thegql queries are pretty error prone as they are


mobile: left value transition (between tablet and mobile breakpoints) are causing a jarring animation

Instead of changing the left value, try using an offset with margin-left to show/hide the menu.
Issue credit: u/_alright_then_

Hi thought this may help you.
translatX is a common option for performant animations: https://www.paulirish.com/2012/why-moving-elements-with-translate-is-better-than-posabs-topleft/


Elements not showing in search bar in Conic Section

All the elements circle ,parabola ,ellipse and hyperbola arent showing in search bar recommendations. I would like to work on it

Go ahead
Deadline 10 April,2021 11:59 PM IST


tfx1.10.0 building successfully

As I wrote down here, this PR can provide the wheel files of TFX https://github.com/diyor28/tf-aarch64/issues/6#issuecomment-1365778842 I guess we are ready to merge this.


[AMBARI-22642] LDAPS Connection Pooling

What changes were proposed in this pull request?
Add com.sun.jndi.ldap.connect.pool.* settings to AMBARI_JVM_ARGS to enable connection pooling for LDAPS (secure LDAP, LDAP over SSL) connections.
We saw errors when attempting to sync a large number of groups (50+) against LDAPS, enabling connection pooling resolved this.
How was this patch tested?
In PR #1281 there were issues in using single quotes in the settings.
To test the use of escaped double quotes (as recommended), in addition to python unit tests, I used local VMs (ambari-vagrant) to test the change to ambari-env.sh on an installed Ambari-server for CentOS7.4, Ubuntu16. 4. This change was also applied to our cluster, which is SLES12.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2488/
Test PASSed.

@rlevas
Hoping to wrap this up.
Now that escaping quotes I have set all JVM_ARGS on one line. Let me know if the preference would be to split them.


Depend on railties and not on rails gem

Hello,
in my Rails application, I don't use the whole Rails default stack - For example I don't need to use any ActionText or ActionMailbox.
Unfortunately, depending onrails gem enforce non-needed gem with this gem. This gem for working depends on a small fraction of Rails stack and can work only three railties, actioncable and listen gems. It's better to require only these gem as necessary dependencies.
I created a similar PR (https://github.com/rails/jsbundling-rails/pull/13) to this issue in jsbundling-rails two years ago (accepted).
E.g. tailwindcss-rails or cssbundling-rails depends on railties gem as well

https://github.com/rails/tailwindcss-rails/blob/main/tailwindcss-rails.gemspec#L21
https://github.com/rails/cssbundling-rails/blob/main/cssbundling-rails.gemspec#L14

@deepj great idea, thank you!


Add two-pass encoding

I've changed the script to utilize two-pass encoding and changed the encoder from libvpx-vp9 to libx264. The two-pass encoding should make it so that the encoder can optimize parts of videos with varying bitrate while staying within the average bitrate target.
EDIT: Turns out you can also do that in libvpx-vp9. I've reverted the encoder back to libvpx-vp9.

Hello. Thank you for your contribution and I apologize I wasn't able to answer earlier..
Before your latest changes, I wanted to respond that the main idea of this project is to use VP9 (even tho it's not really mentoined anywhere...), but since you switched to VP9 again, this is no longer an issue. I'll take a look later today or tommorow at the encoding process, but I think it looks good

All seems fine, sorry again for taking so long to respond.


[video_player] jumpy video playback only on iOS

The video https://cdn.vidcamera.io/videos/cb/1e/66/15/5f68cb17e0cb0a18ec57d62e/cb1e6615-a1de-4e57-95e6-37ee871317f5.mp4 is jumpy on iOS when I'm trying to play it
VideoPlayerController.network('https://cdn.vidcamera.io/videos/cb/1e/66/15/5f68cb17e0cb0a18ec57d62e/cb1e6615-a1de-4e57-95e6-37ee871317f5.mp4');
It's jumpy also when I play it from memory VideoPlayerController.file(...).
The problem doesn't exist when using a native iOS video player, Safari browser, or video_player flutter plugin on Android.
flutter doctor -v
[✓] Flutter (Channel beta, 1.24.0-10.2.pre, on Mac OS X 10.15.7 19H2 darwin-x64, locale en-GB)
    • Flutter version 1.24.0-10.2.pre at /Users/jakub/Development/flutter
    • Framework revision 022b333a08 (3 weeks ago), 2020-11-18 11:35:09 -0800
    • Engine revision 07c1eed46b
    • Dart version 2.12.0 (build 2.12.0-29.10.beta)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/jakub/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = /Users/jakub/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.2, Build version 12B45b
    • CocoaPods version 1.10.0.beta.1

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] Connected device (2 available)
    • Pixel 2 (mobile) • FA7A31A07673 • android-arm64  • Android 11 (API 30)
    • Web Server (web) • web-server   • web-javascript • Flutter Tools

minimal complete reproducible code sample
import 'package:flutter/cupertino.dart';
import 'package:flutter/material.dart';
import 'package:video_player/video_player.dart';

void main() {
  runApp(
    MaterialApp(
      home: _App(),
    ),
  );
}

class _App extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: _BumbleBeeRemoteVideo(),
    );
  }
}

class _BumbleBeeRemoteVideo extends StatefulWidget {
  @override
  _BumbleBeeRemoteVideoState createState() => _BumbleBeeRemoteVideoState();
}

class _BumbleBeeRemoteVideoState extends State<_BumbleBeeRemoteVideo> {
  VideoPlayerController _controller;

  @override
  void initState() {
    super.initState();
    _controller = VideoPlayerController.network(
      'https://cdn.vidcamera.io/videos/cb/1e/66/15/5f68cb17e0cb0a18ec57d62e/cb1e6615-a1de-4e57-95e6-37ee871317f5.mp4',
    );

    _controller.addListener(() {
      setState(() {});
    });
    _controller.initialize();
    _controller.play();
    _controller.setLooping(true);
  }

  @override
  void dispose() {
    _controller.dispose();
    super.dispose();
  }

  @override
  Widget build(BuildContext context) {
    return SingleChildScrollView(
      child: Container(
        padding: const EdgeInsets.all(20),
        child: AspectRatio(
          aspectRatio: _controller.value.aspectRatio,
          child: Stack(
            alignment: Alignment.bottomCenter,
            children: <Widget>[
              VideoPlayer(_controller),
            ],
          ),
        ),
      ),
    );
  }
}

Hi @qbait
Can you please provide your flutter doctor -v and a minimal complete reproducible code sample?
Thank you

Hi @qbait
Can you please provide your flutter doctor -v and a minimal complete reproducible code sample?
Thank you

Fair enough. Edited 👍

Can confirm that the video plays without problem in Safari.

flutter doctor -v
[✓] Flutter (Channel beta, 1.24.0-10.2.pre, on Mac OS X 10.15.7 19H2 darwin-x64, locale en)
    • Flutter version 1.24.0-10.2.pre at /Users/pedromassango/dev/SDKs/flutter_beta
    • Framework revision 022b333a08 (3 weeks ago), 2020-11-18 11:35:09 -0800
    • Engine revision 07c1eed46b
    • Dart version 2.12.0 (build 2.12.0-29.10.beta)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/pedromassango/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.1, Build version 12A7403
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Community Edition (version 2020.2.4)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 52.0.4
    • Dart plugin version 202.8070

[✓] VS Code (version 1.51.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.16.0

[✓] Connected device (3 available)
    • iPhone 12 Pro (mobile) • 1EEFA5B2-DA95-49CC-AA0B-968BB248B8F3 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-1 (simulator)
    • Web Server (web)       • web-server                           • web-javascript • Flutter Tools
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome 87.0.4280.88

• No issues found!
Process finished with exit code 0

FYI most probably the reason is 60fps video.

The FPS for iOS is hardcoded at 30 fps (sorry my bad). Can you try this fix https://github.com/flutter/plugins/pull/3298/files#diff-2b6323a72066e6ee8d4a8e6e5fe204a367928b61362f98bb75725a9c94ec48ceR146 by @veljkomih?

Hi @qbait,
Thanks for filing the issue I am able to reproduce this on the latest stable and the master channel of flutter with
video player^2.1.6 , If you

play the url directly in the chrome browser(desktop/mobile) the video has only audio while the video is blank
Safari browser plays fine
Android: The video plays but there is a green line on the right may be related to #46628
IOS doesn't work and the issue is reproducible

Output

IOS
Android

flutter doctor -v
[✓] Flutter (Channel master, 2.3.0-17.0.pre.608, on macOS 11.4 20F71 darwin-arm, locale en-IN)
    • Flutter version 2.3.0-17.0.pre.608 at /Users/mahesh/Documents/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 18ccfe6386 (10 hours ago), 2021-07-01 05:26:03 +0200
    • Engine revision eab0cd490a
    • Dart version 2.14.0 (build 2.14.0-262.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Community Edition (version 2021.1.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 57.0.5
    • Dart plugin version 211.7233

[✓] VS Code (version 1.57.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.23.0

[✓] Connected device (4 available)
    • sdk gphone arm64 (mobile)  • emulator-5554                        • android-arm64  • Android 11
      (API 30) (emulator)
    • iPhone 12 Pro Max (mobile) • 6A434744-B348-4FB3-AB10-131292174456 • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-14-4 (simulator)
    • macOS (desktop)            • macos                                • darwin-arm64   • macOS 11.4
      20F71 darwin-arm
    • Chrome (web)               • chrome                               • web-javascript • Google Chrome
      91.0.4472.114

• No issues found!

[✓] Flutter (Channel stable, 2.2.2, on macOS 11.4 20F71 darwin-arm, locale en-IN)
    • Flutter version 2.2.2 at /Users/mahesh/Documents/flutter
    • Framework revision d79295af24 (3 weeks ago), 2021-06-11 08:56:01 -0700
    • Engine revision 91c9fc8fe0
    • Dart version 2.13.3

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Community Edition (version 2021.1.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 57.0.5
    • Dart plugin version 211.7233

[✓] VS Code (version 1.57.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.23.0

[✓] Connected device (4 available)
    • Redmi K20 Pro (mobile)     • d9dac45d                             • android-arm64  • Android 10
      (API 29)
    • iPhone 12 Pro Max (mobile) • 6A434744-B348-4FB3-AB10-131292174456 • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-14-4 (simulator)
    • macOS (desktop)            • macos                                • darwin-arm64   • macOS 11.4
      20F71 darwin-arm
    • Chrome (web)               • chrome                               • web-javascript • Google Chrome
      91.0.4472.114

• No issues found!


Wrong Null warning on const building from function parameters

Snipped and problem:

bar is definitly nonNull. But the  third warning states the opposite. Other warnings are true.
dart --version :
Dart SDK version: 2.14.2 (stable) (Wed Sep 15 12:32:06 2021 +0200) on "windows_x64"

Fixed after ba00516.


HitMapView and BmuHitsView have inverted positions

Dear all,
When performing a clusterization of underwater acoustic files I wanted to observe the hitsmap of all the nodes and see it conjointly with the cluster atribution by node.
That is, the HitMapView and the BmuHitsView.
If you see the images bellow

what happens is that the node with 14 hits is in fact located in cluster 7, that is, in the opposite direction. With carefull watching, observing the location of every individual in the node atribution and the cluster atribution, I concluded that they are all in inverted position.
I'm going to observe the code and see where it may be corrected. Meanwhile if you may find it first I appreciate.
Thank you,
Ricardo

Ok! Now that I have been thorough through the code I observed the following:
plot_hex_map should be called like
plot_hex_map(np.flip(Nodes_with_selected_variables.values.reshape(som_chosen.codebook.mapsize +
[Nodes_with_selected_variables.values.shape[-1]]),axis=0),
titles=Nodes_with_selected_variables.columns, shape=[1, 3], colormap=None)
we shoud flip across axis=0
and the file hitmap.py should be changed also in the end as
    elif som.codebook.lattice == "hexa":
        ax, cents = plot_hex_map(np.flip(clusters.reshape(msz[0], msz[1])[::], axis=0),  fig=self._fig, colormap=cmap, colorbar=False)
        if anotate:
            self._set_labels(cents, ax, reversed(clusters), onlyzeros, labelsize, hex=True)

also flipping across axis=0 and putting reversed(clusters) to match the visualizations produced in bmuhits.py.
Please check this!
Thanks for your work!


[Feature Request]: Make Infinity work under Mac

Is there an existing issue for the same feature request?

[X] I have checked the existing issues.

Is your feature request related to a problem?
No response
Describe the feature you'd like
Both Intel and ARM would be supported under Mac OS
Describe implementation you've considered
No response
Documentation, adoption, use case
No response
Additional information
No response

Build source from Mac
# Make sure clang-18 is installed
brew install llvm

# .bash_profile
export CPPFLAGS = "-I/usr/local/opt/llvm/include -I/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include -I/usr/local/include"
export LDFLAGS = "-L/usr/local/opt/llvm/lib/c++ -L/usr/local/opt/llvm/lib -L/usr//local/Cellar/llvm/18.1.8/lib -lunwind -L/usr/local/lib"
export LIBRARY_PATH="$LIBRARY_PATH:/usr/local/opt/llvm/lib/c++:/usr/local/opt/llvm/lib:/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/lib:/usr/local/lib:/usr//local/Cellar/gcc/14.1.0_2/lib/gcc/14"
export CPLUS_INCLUDE_PATH="$CPLUS_INCLUDE_PATH:/usr/local/include:/usr/local//Cellar/boost/1.86.0/include"
export DYLD_LIBRARY_PATH="/usr/local/opt/llvm/lib:$DYLD_LIBRARY_PATH"

Help wanted.
We need Mac apple sillicon  to verify infinity

Can the code on the main branch be compiled on the Mac Apple Silicon platform?

Can the code on the main branch be compiled on the Mac Apple Silicon platform?

We do not have such a mackbook, so we could not tell the answer right now ...

The code won't compile on my mac with m1 pro since the cpu does not support FMA and AVX instructions even with Rosetta on.

To support AArch CPU, we have to implement vector distance computation functions with neon instructions of AArch64 in the directory of src/common/simd .

Help wanted. We need Mac apple sillicon to verify infinity

By fixing the CMakeLists.txt and some header files, I was able to successfully compile the project on my M1 Mac (Apple OS). The unit tests passed, and  ran a partial pytest.
After running all the tests, I can create a PR if needed.

Help wanted. We need Mac apple sillicon to verify infinity

By fixing the CMakeLists.txt and some header files, I was able to successfully compile the project on my M1 Mac (Apple OS). The unit tests passed, and ran a partial pytest.
After running all the tests, I can create a PR if needed.

@ballontt
I've made infinity worked under Linux/aarch64, could you please verify it could work under Mac M1 ?


Deleted publishers in GBIF can be selected in the Publisher filter

When you use the Publisher filter, it shows you the names of organizations that has been deleted from GBIF and don't have data available. As an example, we deleted this organization a long time ago Museo de Historia Natural Marina de Colombia – INVEMAR but it appears and can be selected. It can confuse users to see this publisher, with no data and already deleted.

This happens in the GBIF data portal too, but it would be nice if deleted organizations with no data are not indexed for search in the data portal, to avoid misunderstandings.

Makes sense - for example we do not show deleted datasets in suggest endpoints.


remove zblz/naima from helpers

I'm planning to remove astropy-helpers from naima. Thanks for the automated PRs up to now!

@astrofrog @pllim - I suppose we need to start up a new file for affiliated packages that still use the template, so might benefit from automated package template related updates even when they are not using the helpers any more.

Or the helper update script can abort gracefully if it detects that helpers is gone?

Or the helper update script can abort gracefully if it detects that helpers is gone?

It does it from the beginning. However it's nice to remove the ones that are for sure not use the helpers.


Bitfinex API update on Thursday, 2017-Aug-31

See http://blog.bitfinex.com/api/api-improvements/

Thanks you for this! We will get ready )

We have reviewed all announced improvements and we've added new endpoints. So now we are compatible with the upcoming update (as we implement Bitfinex API v1). We have plans for adding Bitfinex API v2 support as well. Have everything prepared for the update, I'm closing this for now. Feel free to reopen it or ask your questions if any.
Thank you!


FATAL -- : [a7326e4b-90ff-4a37-b963-b8afc3cdc142] Psych::SyntaxError ((/app/conf ig/chats.yml): did not find expected alphabetic or numeric character while scann ing an alias at line 33 column 9):

It seems that your yml file does not have valid syntax

I have used your default yml file and it shows error. I'm new to yml and couldn't understand why there is problem. If you have time please review file and tell me the error. Thanks in advance.

default.yml fixed

Thank you very much. I will try.

This kind of error is arised again.
RegexpError (unmatched close parenthesis: /^:-)/i):

I'm fairly certain the problem is that in your default.yml, many of the patterns and responses don't use quotation marks. For example
hello:   patterns:     p1: hello *     p1a: hello     p2: hi     p3: goodmorning *     p3a: hello     p4: goodday *     p4a: goodday   responses:     r1: hello      r2: hi     r3: goodmorning r4: goodday
didn't work for me, but when I added quotation marks to them all like
hello:   patterns:     p1: "hello *"     p1a: "hello"     p2: "hi"     p3: "goodmorning *"     p3a: "hello"     p4: "goodday *"     p4a: "goodday"   responses:     r1: "hello"      r2: "hi"     r3: "goodmorning"     r4: "goodday"
everything worked properly.

Hi,
this yaml was created by me 6 years ago while starting from "*" was legal. File updated, please ccheck