Proof of concept: sjoin speedup with pygeos

I did a very rough edit to sjoin to add pygeos support. I'm testing with the  pygeos master branch (not pypi package). I'm not using pygeos/pygeos#108, I expect the speedup to be much greater with that, which would be changing just 2 lines in sjoin.py.
Here are results (test script below):

data_len
this branch + pygeos@master
this branch (no pygeos)
joris/pygeos + pygeos@master
joris/pygeos no pygeos
geopandas@master

1
0.200239
0.206183
0.206509
0.286895
0.269579

10
0.125859
0.174937
0.286633
0.204224
0.224107

100
0.152146
0.33471
0.73879
0.518237
0.425057

1000
1.021618
2.941023
34.455022
2.938148
2.967173

In reality, I think a refactor of sindex.py and sjoin.py are needed to make this nice. At the very least, I would move the logic for picking a spatial indexing backend to sindex.py and I would split up sjoin similar to geopandas/geopandas#1271 so we can have separate functions for sjoin and sjoin_pygeos to fully optimize the pygeos implementation and have cleaner logic.
Test script:
import geopandas
import pandas as pd
from timeit import timeit
from geopandas import sjoin, read_file
import numpy as np

def import_dfs(data_len):
    world_path = geopandas.datasets.get_path("naturalearth_lowres")
    cities_path = geopandas.datasets.get_path("naturalearth_cities")
    countries = read_file(world_path)[["geometry", "name"]]
    countries = countries.rename(columns={"name": "country"})
    cities = read_file(cities_path)
    countries = countries.sample(n=data_len, replace=True, axis=0, random_state=1)
    cities = cities.sample(n=data_len, replace=True, axis=0, random_state=1)
    return cities, countries

def sjoin_test(data_len, use_pygeos):
    if hasattr(geopandas._compat, "set_use_pygeos"):
        # for easy testing of different branches
        geopandas._compat.set_use_pygeos(use_pygeos)
    cities, countries = import_dfs(data_len)
    t = timeit(lambda: sjoin(left_df=cities, right_df=countries), number=3)
    res = sjoin(left_df=cities, right_df=countries)
    return t, res

data_lens = [1, 10, 100, 1000]
times_pygeos = []
times_old = []
for data_len in data_lens:
    t_pygeos, res_pygeos = sjoin_test(data_len, use_pygeos=True)
    t_old, res_old = sjoin_test(data_len, use_pygeos=False)
    times_pygeos.append(t_pygeos)
    times_old.append(t_old)
    np.testing.assert_array_equal(
        # drop geometries because they will be different types
        res_pygeos.drop("geometry", axis=1),
        res_old.drop("geometry", axis=1),
    )

results = pd.DataFrame({
    "data_len": data_lens,
    "pygeos_time": times_pygeos,
    "old_time": times_old,
})
print(results)

I decided to explore a cleaner way to implement this, as described above. The branch is on my repo, adriangb/geopandas/pygeos-sindex
I was able to make a wrapper around pygeos.STRTree that emulates Toblerity/rtree. However, the implementation is not the cleanest. It can probably be cleaned up some, but while writing I was thinking that since rtree is an optional dependency and pygeos is going to be an optional dependency, it would be best to take the opportunity to remove rtree and use only pygeos. This would break the current API:

The objects param would be dropped from sindex methods.
Empty geometries will count towards the length of the sindex (but won't return results, I think).
If we want an even more streamlined implementation, I would phase out support of tuples of bounds as an input in favor of only accepting geometry objects. This might break some more things, but I think it makes sense to only accept one type. It could be a warning for version 0.x.

Having a simple clean interface will become especially important once pygeos/pygeos#108 is implemented.


version bump to 3.6.1

Please remember to submit PRs from your fork.

Going to close this out, but please move this change to your fork and resubmit. Thanks.


Ser pretty

Hello again, I changed a few things. Maybe you like to merge it.

Pretty serialization added in https://github.com/kgabis/parson/commit/69055482576de99a36124222e43685ec2262e99e


Create mailinglist icon and remove text field

An icon will be added instead - @davehakkens will create it.

I thought we already talked about this, @davehakkens.
As I said, the reason why I would keep it is because the envelope icon might be misguiding (since it's usually used as a "Contact" icon). I also think that the mail input fits nicely into the footer, otherwise there would be a lot of whitespace.
What do you think, @davehakkens?

Yep we did. However had some thoughts and a good look at it.
Since we are not that active with sending out mails it shouldn't be that dominate on every page. The more stuff we put in the footer the less focus there is on the things that matter to us.
Keep the envelop icon for now. Meanwhile I will work on a more communicative icon
On Saturday 8 August 2015 at 19:44, Leonard wrote:

I thought we already talked about this, @davehakkens (https://github.com/davehakkens).
As I said, the reason why I would keep it is because the envelope icon might be misguiding (since it's usually used as a "Contact" icon). I also think that the mail input fits nicely into the footer, otherwise there would be a lot of whitespace.
What do you think, @davehakkens (https://github.com/davehakkens)?
—
Reply to this email directly or view it on GitHub (https://github.com/hakkens/phonebloks/issues/1#issuecomment-128954007).


feat: added special numbers digit dp soln

Description of Change

I added a digit dp solution for a famous problem named counting special numbers.
Checklist

[ ] Added description of change
[ ] Added file name matches File name guidelines
[ ] Added tests and example, test must pass
[ ] Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
[ ] Relevant documentation/comments is changed or added
[ ] PR title follows semantic commit guidelines
[ ] Search previous suggestions before making a new one, as yours may be a duplicate.
[ ] I acknowledge that all my contributions will be made under the project's license.

Notes:

Please check your description of the pull request


Upgrade to Flink 1.18

https://nightlies.apache.org/flink/flink-docs-release-1.18/release-notes/flink-1.18/

Fixed by https://github.com/deltastreaminc/flink-connector-snowflake/pull/21


Publish first release?

Any plans on when to do the first release?
Might be a good idea to do one before #6 and then one shortly after that with #6.
/cc @Martinffx @gastaldi

0.0.1 was released shortly after I created this issuse, right @gastaldi?

@famod yes, that's right

Ok, thanks! Too bad @martinffx seems to be inactive.

Sorry,  I've not got around to this.
Got some time this week so hopefully I can wrap it up

@famod pushed 1.0.0 release

Thanks @martinffx!


union(with: ) appears to be broken

https://github.com/troughton/Substrate/blob/987832dd60f46a863a53eb3fbed51167a2853a9a/Sources/SubstrateMath/Primitives.swift#L226
hello Thomas, while debugging I noticed that the 2 function union(with:) compute the union using the minimum of the maximum points, shouldn't it be the maximum?
thanks in advance

Sorry about that; fixed in https://github.com/troughton/Substrate/commit/9032c9c224830c8d77cf35f60eddb6fda23d267c.


Updated grep for 4.5.0 in docker-image.yml

Can one of the admins verify this patch?

Not required


Bake key lengths into protocol buffer logic.

keys.go defines three constants:
const aesKeySize = 32 // 256-bit AES
const deriverSecretSize = 32
const hmacKeySize = 32 // SHA-256

These values need to be conveyed by the protocol buffers so that programs outside of the Go codebase can generate keys correctly. The algorithm used depends on this value. For example aesKeySize=32 entails AES256-CTR mode encryption. In this case I think the blockcipher (AES128 or AES256) should be specified in the protocol buffer in some way. In particular change enum CryptoCipherMode to CryptoEncryptionMode and make a new enum CryptoBlockcipher with values AES128 and AES256.
Similarly for HMAC we should specify the hash function. Make a new enum CryptoHash with values SHA256 and SHA1 (although I don't think we use SHA1 anywhere, do we?).

I agree with the sentiment about adding cipher-suite information to the protocol buffers. It's there in some places and not in others: e.g., PBEData does contain a more complete specification of the algorithm types.
However, following the lead of other crypto implementations (like TLS), I think the cipher suites in CryptoAlgorithm should each be a single entity, like AES256_CTR_HMAC_SHA256. This allows us to support only a small number of algorithms that we think are reasonable.
There is some inconsistency in keys.proto around this: for example, CryptoCipherMode has CTR mode as its only option, but the message that contains it is already specific to CTR mode (AES_CTR_HMAC_SHA_CryptingKey_v1). So, I think there's some work to do to normalize all of this and add the extra information.
BTW, it looks like the initial issue text was cut off. What is the proposal for proto/keys.proto?

Sounds good to me.


Improvement of UI  of Set of Evaluation of Stakeholder

I want to improve ui of the above page for eg. Radio button is not looks good.
I want to work on this issue

I have made draft PR #111


BUG: VersorTransform did not update offset in SetParameters

Yes! 5.3.0 will be cut from master, but I think we are going to have another RC first. Bug fixes can always be included.


Implement a div in the index.html file

Implement a div that contains the following:

Wrap the text "Algorithmia SE 🌏" in a span.
Wrap "Trailblazing Tech Excellence, ALX SE Style" in an h1, h2, h3 or h4 tag.
Wrap "Coding The Future, Bringing Ideas Into Reality" in an h1 or h2 tag.
Wrap "Our expanding consortium of ALX Software Engineering students, representing all cohorts, is fervently committed to transforming innovative ideas into reality, and fearlessly real-life challenges" in a span.
Create a button, the content of the button: "Join Our Consortium"

Wrapping all this in a div container, it will help while designing the colors in CSS.

@Algorithmia-SE-2023, would love to work on this

Hello @ALX-SE-Algorithmia/code-reviewing-team
I would love to work on this section

@ALX-SE-Algorithmia/code-reviewing-team i'd love to work on this

Hi @Mubarak1A @masha-a-m @Arewa01.
Thank you for volunteering to work on this section.
However, this issue will be closed, and other improved issues be open.
Thank you all 🤍.


Overhaul user activity logging

Issue type
[x] Feature request
[ ] Bug report
[ ] Documentation
Environment

Python version:  3.5.2
NetBox version:  2.2.9

Description
This issue seeks to unify and supersede the following open issues addressing different shortcomings of the current user activity logging function:

#1127 - Logging of actions should be triggered via signals rather than explicitly within each view
#1834 - User activity is not exported via standard logging facilities
#1896 - Object history is lost when the associated user account is deleted

The new system should address all of the above points, as well as incorporate a more robust representation of object history, perhaps using JSON fields to record before and after versions of the object being modified.

Hey @jeremystretch will this enable API calls to be logged? CRUD via the API.

@troxil Yes, the goal is to log every change to a model regardless of how it was effected (GUI, API, or shell).

I've started work on this under the 1898-activity-logging branch.

Implemented in #2198. Change logging will be included in the v2.4 beta.

Stood up an instance (with blank data) using the devel-2.4 branch.
Saw the changelog attached to the device, as well as /extras/changelog. Two questions came up:

How do I get my hands on that data?
How do I push that data to an external source? (For auditing purposes, change management data really should be logged outside of the thing being audited)

Stood up an instance (with blank data) using the devel-2.4 branch.
Saw the changelog attached to the device, as well as /extras/changelog. Two questions came up:

How do I get my hands on that data?
How do I push that data to an external source? (For auditing purposes, change management data really should be logged outside of the thing being audited)

@jeremystretch:
It does not seem that the ChangeLog record captures everything.
To wit:
Device creation JSON:
{
  "custom_fields": {
    "LastUpdate": "2018-06-27",
    "OS_Version": "Ubuntu 14.04.5 LTS",
    "bios_date": "2011-01-01",
    "bios_version": "Bochs",
    "cpu_dies": 0,
    "cpus": 0,
    "hdd_count": 0,
    "kernel_version": "4.4.0-128-generic",
    "operational_status": 3,
    "ram_capacity": 64429
  },
  "device_role": 52,
  "device_type": 71,
  "name": "testsystem",
  "serial": "",
  "site": 1,
  "status": 1
}

Whereas the ChangeLog record shows:
{
    "asset_tag": null,
    "cluster": null,
    "comments": "",
    "created": "2018-06-27",
    "device_role": 52,
    "device_type": 71,
    "face": null,
    "last_updated": "2018-06-27T17:46:25.599Z",
    "name": "testsystem",
    "platform": null,
    "position": null,
    "primary_ip4": null,
    "primary_ip6": null,
    "rack": null,
    "serial": "",
    "site": 1,
    "status": 1,
    "tenant": null,
    "vc_position": null,
    "vc_priority": null,
    "virtual_chassis": null
}

Custom fields aren't being captured because it's just using Django's built-in serializer for now. I've been toying with reusing the API serializers for this, but the representation of nested objects gets a little tricky. Although the feature branch has been merged into develop-2.4, I'm still working on this.

I will chime in and say I have been investigating #2137 with relation to Date type custom fields and serialization for webhooks. These are related so we might want to look into a particular serialization method to cover both use cases.

@jeremystretch: Would you prefer a new issue filed to keep track, or keep it here? What works for your workflow?


Latest changes in "dist" break compilation on TS 2.8.4

Seems that latest commit 6df524d388d183d9ce79fe58a50152eff4663a74 in the dist branch brakes the project build when TS version is locked to 2.8.4.  Any workaround on this ?

I guess you could stay in the previous version (types/npm-ramda#0685845) but we didn't officially support TS 2.8, we just upgraded from TS 2.6 to TS 2.9 in 6df524d388d183d9ce79fe58a50152eff4663a74.
And I just tried TS 2.8.4 on master, it only broke 2 test cases that's related to keyof object, seems there's no significant breaking changes from 2.8 to 2.9, I'm wondering if it also breaks on TS 2.9 with the latest commit.

Yes, I used the previous version for now.
The error I was getting is following :
> tsc

node_modules/@types/ramda/src/$operation.d.ts(9,11): error TS2322: Type 'T' is not assignable to type 'string'.
  Type 'PropertyKey' is not assignable to type 'string'.
    Type 'number' is not assignable to type 'string'.
node_modules/@types/ramda/src/$operation.d.ts(11,11): error TS2322: Type 'U' is not assignable to type 'string'.
  Type 'PropertyKey' is not assignable to type 'string'.
    Type 'number' is not assignable to type 'string'.
node_modules/@types/ramda/src/dissoc.d.ts(26,23): error TS2344: Type 'V' does not satisfy the constraint 'string'.
  Type 'PropertyKey' is not assignable to type 'string'.
    Type 'number' is not assignable to type 'string'.
node_modules/@types/ramda/src/dissoc.d.ts(29,23): error TS2344: Type 'V' does not satisfy the constraint 'string'.
  Type 'PropertyKey' is not assignable to type 'string'.
    Type 'number' is not assignable to type 'string'.
node_modules/@types/ramda/src/dissoc.d.ts(31,70): error TS2344: Type 'V' does not satisfy the constraint 'string'.
  Type 'PropertyKey' is not assignable to type 'string'.
    Type 'number' is not assignable to type 'string'.
node_modules/@types/ramda/src/dissoc.d.ts(34,70): error TS2344: Type 'V' does not satisfy the constraint 'string'.
  Type 'PropertyKey' is not assignable to type 'string'.
    Type 'number' is not assignable to type 'string'.
node_modules/@types/ramda/src/dissoc.d.ts(45,23): error TS2344: Type 'V' does not satisfy the constraint 'string'.
  Type 'PropertyKey' is not assignable to type 'string'.
    Type 'number' is not assignable to type 'string'.
node_modules/@types/ramda/src/dissoc.d.ts(53,40): error TS2344: Type 'V' does not satisfy the constraint 'string'.
  Type 'PropertyKey' is not assignable to type 'string'.
    Type 'number' is not assignable to type 'string'.
node_modules/@types/ramda/src/dissoc.d.ts(60,40): error TS2344: Type 'V' does not satisfy the constraint 'string'.
  Type 'PropertyKey' is not assignable to type 'string'.
    Type 'number' is not assignable to type 'string'.

At the moment cannot give you clear answer regarding the 2.9 TS version, as it fails due to memory allocation (not sure whose issue is that, still investigating in this as well)

running on TS 2.9 I get this error
ERROR in node_modules/@types/ramda/src/$operation.d.ts(51,50): error TS1005: ';' expected.

Have similar issue. I'm using TS 2.9.2, types/ramda#dist.
Version from commit f7dd77a0b93b903be41f1c28de488daa344ff6fb doesn't compile, with this errors (same as @vkosachev in https://github.com/types/npm-ramda/issues/400#issuecomment-406210253):
node_modules/@types/ramda/src/$operation.d.ts(9,11): error TS2322: Type 'T' is not assignable to type 'string'.
  Type 'string | number | symbol' is not assignable to type 'string'.
    Type 'number' is not assignable to type 'string'.

node_modules/@types/ramda/src/$operation.d.ts(11,11): error TS2322: Type 'U' is not assignable to type 'string'.
  Type 'string | number | symbol' is not assignable to type 'string'.
    Type 'number' is not assignable to type 'string'.

node_modules/@types/ramda/src/$types.d.ts(7,107): error TS2344: Type 'K' does not satisfy the constraint 'string'.
  Type 'string | number | symbol' is not assignable to type 'string'.
    Type 'number' is not assignable to type 'string'.

node_modules/@types/ramda/src/$types.d.ts(8,98): error TS2344: Type 'K' does not satisfy the constraint 'string'.
  Type 'string | number | symbol' is not assignable to type 'string'.
    Type 'number' is not assignable to type 'string'.
  Type 'string | number | symbol' is not assignable to type 'string'.
node_modules/@types/ramda/src/dissoc.d.ts(26,23): error TS2344: Type 'V' does not satisfy the constraint 'string'.
    Type 'number' is not assignable to type 'string'.

node_modules/@types/ramda/src/dissoc.d.ts(29,23): error TS2344: Type 'V' does not satisfy the constraint 'string'.
  Type 'string | number | symbol' is not assignable to type 'string'.
    Type 'number' is not assignable to type 'string'.

node_modules/@types/ramda/src/dissoc.d.ts(31,70): error TS2344: Type 'V' does not satisfy the constraint 'string'.
  Type 'string | number | symbol' is not assignable to type 'string'.
    Type 'number' is not assignable to type 'string'.

.....

I also have "keyofStringsOnly": true option in tsconfig.json, if that matters

Looks like you are 💯% right, angular cli is on TS 2.7, so using it means I can't use TS 2.9 oh well, still seems to compile after the first error.


Link triggered hints to the webpage

Eg. when eugene lint or eugene trace finds E1, it can provide a link to https://kaveland.no/eugene/hints/E1/ where there's an example of how to fix it.

I think this should also include making a super terse output format for plain, suitable for the terminal. Something like:
$ eugene lint examples.sql 4: https://kaveland.no/eugene/hints/E4 Add a new JSON column 6: https://kaveland.no/eugene/hints/E2 Validating table with a newNOT NULL` column
...
I'm thinking the same kind of format for both lint/trace.

#78 does the first part of this, going to make a nice-ish plain text format for the console next.


Test whether this project works using the open source NVidia drivers.

I currently need testers, because I don't want to mess up my Linux install.
This is issue is for anyone who is currently running the open source NVidia drivers.
If you have the free time to check this, please write whether this worked for you on this issue.

No activity


add some comments and jsdocs to ExportMap

ExportMap is difficult to follow, so lets add some helpful comments.

I don't understand why continuous-integration/appveyor/pr failed.
Error: spawn nyc ENOENT
    at notFoundError (C:\projects\eslint-plugin-import\node_modules\cross-spawn\lib\enoent.js:11:11)
    at verifyENOENT (C:\projects\eslint-plugin-import\node_modules\cross-spawn\lib\enoent.js:46:16)
    at ChildProcess.cp.emit (C:\projects\eslint-plugin-import\node_modules\cross-spawn\lib\enoent.js:33:19)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:293:12)
Emitted 'error' event on ChildProcess instance at:
    at ChildProcess.cp.emit (C:\projects\eslint-plugin-import\node_modules\cross-spawn\lib\enoent.js:36:37)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:293:12) {
  errno: 'ENOENT',
  code: 'ENOENT',
  syscall: 'spawn nyc'
}
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>mocha: `cross-env BABEL_ENV=test nyc mocha "tests/src"`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>mocha script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

Did it fail to find nyc installed?

No need to worry about that; adding types to the util package seems to have broken appveyor.

stale.

That's fine, but let's reuse this PR rather than making a new one if a similar change comes up.


rangeSlider errors when used in combination with input type="number"

When the rangeSlider is used with an input field of type="number", on any change of the slider, Angular throws errors:
Error: [ngModel:numfmt] Expected 2160 to be a number
http://errors.angularjs.org/1.3.8/ngModel/numfmt?p0=2160
at REGEX_STRING_REGEXP (angular.js:63)
at Array. (angular.js:19863)
at Object.ngModelWatch (angular.js:20971)
at Scope.$get.Scope.$digest (angular.js:14217)
at Scope.$get.Scope.$apply (angular.js:14488)
at HTMLDocument. (angular.rangeSlider.js:583)
at HTMLDocument.jQuery.event.dispatch (jquery.js:5095)
at HTMLDocument.jQuery.event.add.elemData.handle (jquery.js:4766)
The value in the input fiels does update. It all works ok, but still the errors are thrown.
The error is not thrown when the input field is changed directly.
If input field does not have the type specified, there is no issue. It then works correctly, but all HTML5 goodness related to input type="number" is lost.
I created a plunker, which for some reason does not work (another issue?). Can't find the issue in the plunkr, but you can copy that code to chrome and see what I mean. Link to Plnkr: http://plnkr.co/edit/LFtWWGk9FV5XGDTaaPiA?p=info
Is there a way to use rangeSlider in combination with input type="number"?

That looks like an internal Angular error to me...
Probably the model value is actually a string and therefore Angular throws an error when you try and put it in an <input type="number"> field, as they only accept numbers. See: https://docs.angularjs.org/api/ng/input/input[number]
Its possible that there is a $filter somewhere that is converting the raw number into a string
No idea what's going on with the Plnkr... can't get it working

thanks for your prompt reply.
About the value being a string - I had the same assumptions at first, but I double checked. It is a number. I guess the proof that it is not an Angular issue is that changing the input type="number" field directly, e.g. by scrolling inside the input field, correctly changes the models value and does not throw an error.
the error is only thrown when moving the rangeSlider.
Is it possible that at any point you modify the value, so it loses type? or something similar?

Found the culprit:
https://github.com/danielcrisp/angular-rangeslider/blob/master/angular.rangeSlider.js#L547
https://github.com/danielcrisp/angular-rangeslider/blob/master/angular.rangeSlider.js#L551
Perhaps if we wrapped these lines in parseFloat it would convert the string back into a number and it would work.
And the plnkr wasn't working because it didn't include jQuery.
Updated: http://plnkr.co/edit/c4gWaK8z0kyTnaCttAeO?p=info

awesome.
I'll try to fix it tonight. One of my colleagues added support for non-linear values of the slider. I will talk to him today evening and see if we can form pull requests in sync.
thanks!!

Fixed in version 0.0.12
Great the non-linear values support sounds good

@danielcrisp Is this issue fixed? I am getting the same error in version 0.0.14_ .


Add 'subject' option

As a CLI User
I want to specify email subject
So that I can send email :)

fixed as part of #6


Fix infinite paging in saved report detail

This reverts commit a3eec76: "Disable next and last pagination buttons when all the report data is on a single page"
The original commit was to fix bz1182360, but this lead to infinite number of pages. The underlying issue (from that bz) has apparently been fixed in the meantime (could not reproduce), so this is no longer necessary.
https://bugzilla.redhat.com/show_bug.cgi?id=1205402

Verified fix.


Swift 3.0 Support/Style

Let me know if anything looks off. Usually people bump a major version for Swift 3 support. Even if you weren't ready for 1.0 might be a good idea. If all looks good merge and publish 👍

Would be nice if this would be merged!

@hawk-ukr can you merge this branch please?


Fine tuning

HI, I would like to fine tune CLIP model for custom dataset so that it will detect the object in an image given a image as a a prompt.
Can anyone suggest me any steps to be followed for it or any references for the same ?

@Rohinivv96 same question

use open_clip

HI, I would like to fine tune CLIP model for custom dataset so that it will detect the object in an image given a image as a a prompt. Can anyone suggest me any steps to be followed for it or any references for the same ?

same question


[master] Bump Mesos to nightly master 558829e

High-level description
This is a routine bump to the latest Mesos and mesos-modules.
Related JIRA Issues
Checklist for all PRs

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Changelog:
https://github.com/apache/mesos/compare/270a3dce490d5b334f9a0011ea416ffc42e187e4...558829eb24f4ad636348497075bbc0428a4794a4

[ ] Test Results: [link to CI job test results for component]

[ ] Code Coverage (if available): [link to code coverage report]

@mesosphere-mergebot bump-ee

@mesosphere-mergebot label Ready For Review

Superseded by #6644.


AST: Remove SubstitutionMap::getMap()

We don't want to expose the fact that SubstitutionMaps are
backed by a DenseMap, since that's going to change soon.

@swift-ci Please smoke test

@swift-ci Please smoke test

@swift-ci Please smoke test


AttributeError: 'async_timeout' does not have attribute 'TimeoutError' in Cloudflare Tunnel Monitor integration

Problem Description:
After updating Home Assistant to version 2024.11.0, an error occurs in the Cloudflare Tunnel Monitor integration. The following error message appears in the logs.
Steps to Reproduce the Issue:
Updated Home Assistant to version 2024.11.0.
Restarted Home Assistant.
An error message appears in the logs related to the Cloudflare Tunnel Monitor integration.
Expected Result:
The integration works without errors and correctly monitors the Cloudflare tunnel.
Actual Result:
The integration does not work, and an AttributeError appears in the logs.
Logs:

Traceback (most recent call last):
  File "/config/custom_components/cloudflare_tunnel_monitor/sensor.py", line 46, in fetch_tunnels
    raise UpdateFailed(f"Error fetching Cloudflare tunnels: {response.status}, {response.reason}")
homeassistant.helpers.update_coordinator.UpdateFailed: Error fetching Cloudflare tunnels: 500, Internal Server Error

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 382, in _async_refresh
    self.data = await self._async_update_data()
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 279, in _async_update_data
    return await self.update_method()
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/cloudflare_tunnel_monitor/sensor.py", line 197, in async_update_data
    new_data = await fetch_tunnels(api_key, account_id, hass, config_entry.entry_id)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/cloudflare_tunnel_monitor/sensor.py", line 50, in fetch_tunnels
    except async_timeout.TimeoutError:
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: module 'async_timeout' has no attribute 'TimeoutError'

Environment Information:
Home Assistant version: 2024.11.0
Cloudflare Tunnel Monitor integration version: 5.2.1
Operating System: Home Assistant OS 13.2
Additional Information:
It seems that in version 4.0.0 of the async_timeout module, TimeoutError has been removed. Modifying the exception handling in the sensor.py file on line 50 as follows solves the problem in my case:
Original code:
except async_timeout.TimeoutError:

Modified code:
import asyncio

...

except asyncio.TimeoutError:

Would it be possible to include this fix in the integration?
Thank you in advance for your help!

#19 Pull request created


Remove bad error return in census_initialize

Can one of the admins verify this patch?

Can one of the admins verify this patch?


fix: remove definitions key after dereffing

:tada: This PR is included in version 1.0.13 :tada:
The release is available on:

GitHub release
1.0.13
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Without arguments the command crashes

When executing jx project without arguments, it crashes:
➜ jx project
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0x21eb36e]

goroutine 1 [running]:
github.com/jenkins-x/jx-project/pkg/cmd/root.(*WizardOptions).Run(0xc0005980a0, 0xc0007b1cf8, 0x0)
/workspace/source/pkg/cmd/root/root.go:84 +0x3e
github.com/jenkins-x/jx-project/pkg/cmd/root.NewCmdMain.func1(0xc000202b00, 0x35f2768, 0x0, 0x0)
/workspace/source/pkg/cmd/root/root.go:69 +0x42
github.com/spf13/cobra.(*Command).execute(0xc000202b00, 0xc00009c190, 0x0, 0x0, 0xc000202b00, 0xc00009c190)
/go/pkg/mod/github.com/spf13/cobra@v1.0.0/command.go:846 +0x2aa
github.com/spf13/cobra.(*Command).ExecuteC(0xc000202b00, 0xc0005980a0, 0xc00006c750, 0xc0007b1f50)
/go/pkg/mod/github.com/spf13/cobra@v1.0.0/command.go:950 +0x349
github.com/spf13/cobra.(*Command).Execute(...)
/go/pkg/mod/github.com/spf13/cobra@v1.0.0/command.go:887
github.com/jenkins-x/jx-project/cmd/app.Run(0x0, 0x0, 0x0, 0x22d5ba0, 0xc00009a058)
/workspace/source/cmd/app/root.go:16 +0x80
main.main()
/workspace/source/cmd/main.go:11 +0x32

When executing it with --help for example, no problem:
➜ jx project --help
Create a new project by importing code, creating a quickstart or custom wizard for spring.

Usage:
 project [flags]
 project [command]

Examples:
 # Create a project using the wizard
 jx project

Available Commands:
 help        Help about any command
 import      Imports a local project or Git repository into Jenkins
 pullrequest Create a Pull Request on the git project for the current directory
 quickstart  Create a new app from a Quickstart and import the generated code into Git and Jenkins for CI/CD
 spring      Create a new Spring Boot application and import the generated code into Git and Jenkins for CI/CD

Flags:
 -h, --help   help for project

Use "project [command] --help" for more information about a command.

Before this, I've upgraded the CLI and the plugins:
➜ jx upgrade cli
about to run: git clone https://github.com/jenkins-x/jxr-versions.git /var/folders/m4/ytp3wjnn1hlbrxt27d64jc3c0000gn/T/jx-git-075610043 in dir /var/folders/m4/ytp3wjnn1hlbrxt27d64jc3c0000gn/T
Cloning into '/var/folders/m4/ytp3wjnn1hlbrxt27d64jc3c0000gn/T/jx-git-075610043'...
Jenkins X client has been upgraded to version 3.0.338

➜ jx upgrade plugins
checking binary jx plugin admin version 0.0.117 is installed
checking binary jx plugin application version 0.0.10 is installed
checking binary jx plugin gitops version 0.0.316 is installed
checking binary jx plugin jenkins version 0.0.22 is installed
checking binary jx plugin pipeline version 0.0.20 is installed
checking binary jx plugin preview version 0.0.82 is installed
checking binary jx plugin project version 0.0.99 is installed
checking binary jx plugin promote version 0.0.110 is installed
checking binary jx plugin secret version 0.0.144 is installed
about to run: /Users/hervelemeur/.jx3/plugins/bin/jx-secret-0.0.144 plugins upgrade
checking we have the correct vault CLI version
checking binary jx plugin test version 0.0.18 is installed
checking binary jx plugin verify version 0.0.26 is installed

Operating System   Mac OS X 10.15.6 build 19G2021

/assign


Release dcp on crates.io

Since dcp has stabilized, it would make sense to release it to the wider community. As a first step, releasing it on crates.io lets Rust users install dcp via cargo install.

Closing as the crate has been published: see https://crates.io/crates/dcp


Could not install ContainerImage provider

I'm following the documentation to install Windows Server 2016 in a Virtual Machine on Windows 10 Enterprise.
I executed this command to create the VM:  powershell.exe -NoProfile -ExecutionPolicy Bypass c:\New-ContainerHost.ps1 -VMName Server2016 -WindowsImage ServerDatacenterCore –Hyperv
Unfortunately I'm getting an error and in the Hyper-V Manager I can see that the VM was not fully created:

Dismounting VHD...
C:\New-ContainerHost.ps1 : Could not install ContainerImage provider
At line:1 char:1

c:\New-ContainerHost.ps1 -VMName Server2016 -WindowsImage ServerDatac ...

- CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
- FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,New-ContainerHost.ps1

How can I install the "ContainerImage" provider?

Based on this, does the following help ?
Install-PackageProvider ContainerImage -Force

ContainerImage is no longer a supported command. Please read this guide to see the new guidelines for setting up containers.


fix(armv8/vgic): fix comparison on pend_found

If a suitable pending interrupt is found, it should be replaced rather than checking if pend_found is greater than 1. Thus, I have changed > 1 to >= 1.
If my understanding is incorrect, I will close this PR. Thank you for your review!

I'm not sure this fix is correct. The logic here is that if we found more than one pending interrupt, we can replace one of them, because as suggested by the GIC spec, if there are any pending interrupts, at least one pending interrupt should be in the list. Do you agree @Clara-Huang @DavidMCerdeira ?

@Clara-Huang I'm closing this for now, unless there is any new arguments that can convince me my rationale is wrong.


Add body_html_string method to Response

This PR adds the body_html_string helper method proposed in the discussion of this issue:
https://github.com/http-rs/tide/issues/534

@gmosx thanks for opening this PR! We've recently removed all body_* methods in favor of set_body. We understand there's a desire to make it easier to return HTML from endpoints, but this is probably not it.
I'm wondering if we could perhaps expand serve_dir to serve single files, which would make the case of serving a single file easier.

Please note that my use-case is to serve  dynamically-generated html content, so server_dir is not appropriate.

I think this does the same as https://github.com/http-rs/tide/pull/292. I think it can be closed. Maybe a new pr can be opened for a from_html method on the Body type in the http-types. That is where from_json and from_str have moved.


How to use to generate HTML for an Android app

Since I just spent a couple hours figuring out how to use your plugin to generate HTML to be used in an Android app, I thought I would pass it along so that you could document it in your readme.
The scheme I am detailing here will place the output as resource directories in the application. Some people will think that the generated HTML should go in the assets directory, but that is not a good practice because the assets directory cannot be translated. The proper place to store them on Android is as raw resources so you can provide different languages and possibly even different versions for different screen sizes.
To add markdown you create a markdown folder in the src directory of the source set. For example, src/main/markdown. Because this is for android resources you will want to put your markdown in a raw forlder inside that directory (src/main/markdown/raw)
Then you will need to add these lines to the build.gradle for the module:
import com.android.builder.model.SourceProvider
import org.kordamp.gradle.markdown.MarkdownToHtmlTask

android {
    sourceSets.all { sourceSet ->
        def src = new File("$projectDir/src/${sourceSet.name}/markdown")

        if(src.exists())
        {
            def task = task("${sourceSet.name}MarkdownToHtml", type:MarkdownToHtmlTask)

            task.sourceDir = src
            task.outputDir = new File("$buildDir/generated/html/${sourceSet.name}")
        }
    }

    applicationVariants.all { variant ->
        for(SourceProvider sourceSet : variant.sourceSets)
        {
            def task = tasks.findByPath("${sourceSet.name}MarkdownToHtml")

            if(task != null)
            {
                variant.registerResGeneratingTask(task, task.outputDir)
            }
        }
    }
}

Since this generates multiple instances of the markdownToHtml task it is a little more work to configure them. You will likely want to add something like this to do configuration:
tasks.withType(MarkdownToHtmlTask).all { task ->
    task.inputEncoding = "utf8"
    task.configuration =
            [
                    hardWraps: true,
                    autoLinks: true,
            ]
}


Some documentation errors in https://pocketbase.io/docs/go-collections/ and https://pocketbase.io/docs/go-migrations/.

Hello  people,
I found some minor documentation errors:

&core.UrlField need to change to  &core.URLField . In https://pocketbase.io/docs/go-migrations/#creating-collection-programmatically
types.Pointer("id = @request.auth.id") doesn't work on golang 1.23.4. I used<EMAIL_ADDRESS>!= ''"}[0]. In https://pocketbase.io/docs/go-migrations/#creating-collection-programmatically
https://pocketbase.io/docs/go-collections/#create-new-collection
https://pocketbase.io/docs/go-collections/#update-existing-collection
https://pocketbase.io/docs/go-migrations/#creating-collection-programmatically

Warm Regards, (and Happy new year)
Harry

I've pushed a fix for the typo.

types.Pointer(...) is correct, the migrations example is missing an import statement ("github.com/pocketbase/pocketbase/tools/types").

For other docs related issues, feel free to open issues/PRs directly in the https://github.com/pocketbase/site repo.


isThumbnail

add a isThumbnail property in graphQL models, limited to images fields, in order to be able to use these fields in lists (BO/FO) views. Use this property in the pixabay datasource.

Done with isMainImage and isThumbnail. In pixabay case we got :
"fields"=>[
"largeImageURL"=>[
"type"=>"ImageURL",
"isMainImage"=>true
],
"previewURL"=>[
"type"=>"ImageURL",
"isThumbnail"=>true
]
],


Accessible Bezel Interface

I made several changes that allows VoiceOver to read the names of buttons and detect when button names change. The changes were implemented within the main Quicksilver code and the Bezel Interface.

Hi Rich,
Sorry it's taken us such a long time to merge this pull request. We've been concentrating fully on the plugins system lately!
Once the next version of Quicksilver is out, we'll review this and merge it into master :)
Thanks for bearing with us!

Okay. Thanks for the update.
rich
On Oct 21, 2011, at 8:49 AM, Patrick Robertson wrote:

Hi Rich,
Sorry it's taken us such a long time to merge this pull request. We've been concentrating fully on the plugins system lately!
Once the next version of Quicksilver is out, we'll review this and merge it into master :)
Thanks for bearing with us!
--
Reply to this email directly or view it on GitHub:
https://github.com/quicksilver/Quicksilver/pull/421#issuecomment-2480833

I’m finally looking at this as well. Do you think you could remove all the files under Quicksilver.xcodeproj? The latest .gitignore is set up to exclude those, but it might not have been updated when you made these changes. Anyway, some of the files conflict with untracked files on other users’ systems.

I’ve done it before, but it’s not something I could do off the top of my head. I just experimented on my copy of your branch and I think this’ll do it:
git checkout accessibility
git rm --cached Quicksilver/Quicksilver.xcodeproj/project.xcworkspace/contents.xcworkspacedata
git rm -r --cached Quicksilver/Quicksilver.xcodeproj/project.xcworkspace/xcuserdata/Rich*
git rm -r --cached Quicksilver/Quicksilver.xcodeproj/xcuserdata/Rich*
git commit -m 'removing user specific files'
git push origin accessibility

The --cached option should remove them from the Git history, but leave the actual files in place on your hard drive. Let me know if you have any problems.

Hi,
I got all the way to the end of your instructions, and everything looked like it was going great, but when I tried the push command I got this:
Permission denied (publickey).
fatal: The remote end hung up unexpectedly
rich
On Nov 3, 2011, at 1:27 PM, Rob McBroom wrote:

I’ve done it before, but it’s not something I could do off the top of my head. I just experimented on my copy of your branch and I think this’ll do it:
git checkout accessibility
git rm --cached Quicksilver/Quicksilver.xcodeproj/project.xcworkspace/contents.xcworkspacedata
git rm -r --cached Quicksilver/Quicksilver.xcodeproj/project.xcworkspace/xcuserdata/Rich*
git rm -r --cached Quicksilver/Quicksilver.xcodeproj/xcuserdata/Rich*
git commit -m 'removing user specific files'
git push origin accessibility
The --cached option should remove them from the Git history, but leave the actual files in place on your hard drive. Let me know if you have any problems.

Reply to this email directly or view it on GitHub:
https://github.com/quicksilver/Quicksilver/pull/421#issuecomment-2620168

Permission denied (publickey).
fatal: The remote end hung up unexpectedly

Hmm. That’s an issue with your SSH access to GitHub. I assume it worked before, or you wouldn’t have been able to summit the original commits, right?
Can you run git remote -v and post the output?

Here's what I got:
tb-lx-wireless-pittnet-108-100:Quicksilver Rich$ git remote -v
origin	git@github.com:richs1000/Quicksilver.git (fetch)
origin	git@github.com:richs1000/Quicksilver.git (push)
upstream	git@github.com:quicksilver/Quicksilver.git (fetch)
upstream	git@github.com:quicksilver/Quicksilver.git (push)
rich
On Nov 4, 2011, at 2:38 PM, Rob McBroom wrote:

Permission denied (publickey).
fatal: The remote end hung up unexpectedly

Hmm. That’s an issue with your SSH access to GitHub. I assume it worked before, or you wouldn’t have been able to summit the original commits, right?
Can you run git remote -v and post the output?

Reply to this email directly or view it on GitHub:
https://github.com/quicksilver/Quicksilver/pull/421#issuecomment-2634844

OK. That all looks normal. At this point, I probably can’t do any better than the GitHub help.
Take a look at http://help.github.com/ssh-issues/

Okay, I changed my public key and that seemed to do the trick. Let me know if the changes have been saved.
rich
On Nov 4, 2011, at 3:19 PM, Rob McBroom wrote:

OK. That all looks normal. At this point, I probably can’t do any better than the GitHub help.
Take a look at http://help.github.com/ssh-issues/

Reply to this email directly or view it on GitHub:
https://github.com/quicksilver/Quicksilver/pull/421#issuecomment-2635364

Yup. Looks like the files are gone. No more conflicts.
So what should we be looking for in testing this. I turned on VoiceOver and it seems to be doing reasonable things. It works pretty well with Nostromo, too, but that’s probably because of the changes you made to the core application. Anything in particular I should try?

The changes I made were pretty limited. The things that should be different are:

VoiceOver speaks the contents of each panel rather than saying "button"
The Quicksilver pull-down menu is accessible using the tab key
VoiceOver speaks the items in the Quicksilver pull-down menu

rich
On Nov 4, 2011, at 11:05 PM, Rob McBroom wrote:

Yup. Looks like the files are gone. No more conflicts.
So what should we be looking for in testing this. I turned on VoiceOver and it seems to be doing reasonable things. It works pretty well with Nostromo, too, but that’s probably because of the changes you made to the core application. Anything in particular I should try?

Reply to this email directly or view it on GitHub:
https://github.com/quicksilver/Quicksilver/pull/421#issuecomment-2638662

The Quicksilver pull-down menu is accessible using the tab key

You mean the arrow in the top-right of the interface? This seems to be selected whenever I bring up Quicksilver and it’s announced as “downward pointing small black triangle”. I can’t seem to do anything with it via the keyboard. Keystrokes go to the first pane like they’re supposed to, but the object in the fist pane doesn’t get announced unless I manually tab to it. Typing updates the first pane correctly, but it just announces “downward pointing small black triangle” over and over.
It does appear to announce buttons and menus correctly when they have focus.

It's all coming back to me now...
I had to rebuild both the Bezel interface and Quicksilver.
To get at the downward pointing small black triangle with VoiceOver:

Use Ctrl+Option+left arrow and/or Ctrl+Option+right arrow to move input focus to that button (not the tab key)
On my computer, VoiceOver announces "Search Mode Menu Button"
Use Ctrl+Option+Space to "press" the button to display the pull-down menu

rich

From: Rob McBroom<EMAIL_ADDRESS>Sent: Monday, November 07, 2011 10:08 AM
To: Simpson, Richard Callaghan
Subject: Re: [Quicksilver] Accessible Bezel Interface (#421)

The Quicksilver pull-down menu is accessible using the tab key

You mean the arrow in the top-right of the interface? This seems to be selected whenever I bring up Quicksilver and it’s announced as “downward pointing small black triangle”. I can’t seem to do anything with it via the keyboard. Keystrokes go to the first pane like they’re supposed to, but the object in the fist pane doesn’t get announced unless I manually tab to it. Typing updates the first pane correctly, but it just announces “downward pointing small black triangle” over and over.
It does appear to announce buttons and menus correctly when they have focus.

Reply to this email directly or view it on GitHub:
https://github.com/quicksilver/Quicksilver/pull/421#issuecomment-2654491

Hi,
I haven't heard anything on this for a while, and was wondering where things stood.
Regards,
rich
On Nov 7, 2011, at 12:18 PM, Simpson, Richard Callaghan wrote:

It's all coming back to me now...
I had to rebuild both the Bezel interface and Quicksilver.
To get at the downward pointing small black triangle with VoiceOver:

Use Ctrl+Option+left arrow and/or Ctrl+Option+right arrow to move input focus to that button (not the tab key)
On my computer, VoiceOver announces "Search Mode Menu Button"
Use Ctrl+Option+Space to "press" the button to display the pull-down menu

rich

From: Rob McBroom<EMAIL_ADDRESS>Sent: Monday, November 07, 2011 10:08 AM
To: Simpson, Richard Callaghan
Subject: Re: [Quicksilver] Accessible Bezel Interface (#421)

The Quicksilver pull-down menu is accessible using the tab key

You mean the arrow in the top-right of the interface? This seems to be selected whenever I bring up Quicksilver and it’s announced as “downward pointing small black triangle”. I can’t seem to do anything with it via the keyboard. Keystrokes go to the first pane like they’re supposed to, but the object in the fist pane doesn’t get announced unless I manually tab to it. Typing updates the first pane correctly, but it just announces “downward pointing small black triangle” over and over.
It does appear to announce buttons and menus correctly when they have focus.

Reply to this email directly or view it on GitHub:
https://github.com/quicksilver/Quicksilver/pull/421#issuecomment-2654491

Once again, apologies Rich for taking so long on this.
I am looking into it now, and here are a few observations I have:

This will only work for the Bezel interface. I guess this is OK, users wishing to use this feature would just have to use that interface
It seems as if the button in QSBezelInterface.xib has two outlets: menuButton and qsMenuButton. It seems like menuButton was there before your changes - is there a reason why you didn't use that? I'd suggest: removing the qsMenuButton outlet and changing the iVar in QSBezelInterfaceController to IBOutlet *menuButton (dropping the 'qs' prefix altogether)
If there's no text in the 1st pane (when it says 'type to search') Voiceover reads 'button'. It should probably try and read 'type to search', but this isn't the end of the world, if you think this is still usable.
If you hit the down arrow to open the results controller, there's an NSException (see below)
To solve the problem Rob was talking about of 'down pointy small black arrow' being read out, you should change the 'Accessibility Identity' description of the 'Static Text' arrow object to 'Search Mode Menu' (in the Attributes inspector)

Other than these small things, this seems like it could be really useful for users. I'm sorry again it's taken us all so long to look into this properly. Hopefully we can get the pull request merged soon though!

Hi,
I think I've been able to deal with everything but the NSException. See comments embedded below.
rich
On Jan 28, 2012, at 3:37 PM, Patrick Robertson wrote:

This will only work for the Bezel interface. I guess this is OK, users wishing to use this feature would just have to use that interface

The Bezel interface is what I use, so it's what I know best. If there is demand for making other interfaces accessible, I can do that in the future. My assumption was that all the interfaces provided the same functionality, so it didn't matter which one I started with.

It seems as if the button in QSBezelInterface.xib has two outlets: menuButton and qsMenuButton. It seems like menuButton was there before your changes - is there a reason why you didn't use that? I'd suggest: removing the qsMenuButton outlet and changing the iVar in QSBezelInterfaceController to IBOutlet *menuButton (dropping the 'qs' prefix altogether)

I made this change

If there's no text in the 1st pane (when it says 'type to search') Voiceover reads 'button'. It should probably try and read 'type to search', but this isn't the end of the world, if you think this is still usable.

I have fixed this.

If you hit the down arrow to open the results controller, there's an NSException (see below)

Can you explain this in a little more detail? I don't see anything relevant below.

To solve the problem Rob was talking about of 'down pointy small black arrow' being read out, you should change the 'Accessibility Identity' description of the 'Static Text' arrow object to 'Search Mode Menu' (in the Attributes inspector)

I plugged away at this for a while. I've managed to get it to say "Downward pointing small black triangle Search mode menu." To really fix this behavior, I think I'd need to replace the static text field with a picture of a small black triangle.

Great, good work!
If you push your commits to GitHub, we will be able to see them here :)

No, unfortunately I can't see them.
Make sure when you are pushing to GitHub that you are on your local 'accessibility' where your commits are made branch and that you:
git push remote accessibility
to push your changes to the branch referred to in this pull request

Okay, it looked like a lot more happened that time. Did it work?
rich
On Feb 18, 2012, at 6:13 PM, Patrick Robertson wrote:

No, unfortunately I can't see them.
Make sure when you are pushing to GitHub that you are on your local 'accessibility' where your commits are made branch and that you:
git push remote accessibility
to push your changes to the branch referred to in this pull request

Reply to this email directly or view it on GitHub:
https://github.com/quicksilver/Quicksilver/pull/421#issuecomment-4038172

That seems to have worked, but the commits seem quite messy, and have given us a merge conflict!
We can happily deal with this, but do you think it'd be possible for you to re-apply the changes of these two commits?
run git reset --hard HEAD~2
this will take you back two commits, and delete everything you've done.
Then you can re-add the code changes, without saving the user specific files.
When you push to GitHub make sure you use git push -f remote accessibility (the -f forces the push)
If it's too much hassle, we can try and work round it :)

Well, I gave it a whirl.
The first command ran successfully. Here's everything that happened afterwards:
Richard-Simpsons-MacBook-Pro:Quicksilver Rich$ git add .

Delta compression using up to 4 threads.
Compressing objects: 100% (29/29), done.
Writing objects: 100% (32/32), 20.04 KiB, done.
Total 32 (delta 14), reused 0 (delta 0)
To git@github.com:richs1000/Quicksilver.git

0e8736d...7dcd1f8 accessibility -> accessibility (forced update)
Richard-Simpsons-MacBook-Pro:Quicksilver Rich$

On Feb 19, 2012, at 4:59 AM, Patrick Robertson wrote:

That seems to have worked, but the commits seem quite messy, and have given us a merge conflict!
We can happily deal with this, but do you think it'd be possible for you to re-apply the changes of these two commits?
run git reset --hard HEAD~2
this will take you back two commits, and delete everything you've done.
Then you can re-add the code changes, without saving the user specific files.
When you push to GitHub make sure you use git push -f remote accessibility (the -f forces the push)
If it's too much hassle, we can try and work round it :)

Reply to this email directly or view it on GitHub:
https://github.com/quicksilver/Quicksilver/pull/421#issuecomment-4040964

Almost there, except you're still adding all your own user files. Make sure you don't commit them (type git status to see what is going to be committed)
You also need to re-add the latest changes you made, as they've been reverted.

That looks much better. I'll look over your changes when I get some time (currently extremely busy at the moment)


[DEV] TVM v0.7 Roadmap

This roadmap for TVM v0.7. TVM is a community-driven project and we love your feedback and proposals on where we should be heading. Please open up discussion in the discussion forum as well as bring RFCs.

Feel free to volunteer yourself if you are interested in trying out some items(they do not have to be on the list).
Please also check out the help wanted list in the github issues on things that need help

In the 0.7 cycle, we are going to focus on the following four areas. Which are summarized summarized from the forum discussion.
We also welcome contributions along all the other areas, including more operator and model coverages, and they will be added to this list gradually. Please reply to this thread about what you would like to volunteer to work on.
We will aim to cut a release in about three months timeframe(around April, May)
Core Infra

[x] Unified IR refactor
[x] Unified runtime for heterogeneous execution
[ ] Enhanced support for high-level graph rewriting for accelerators
[x] Improving test and benchmark infrastructure.

[ ]  Testing and benchmarking on remote targets.

[ ] Better packaging of the library besides installing from source

Usability

[x] Better documentations for developments
[ ] Command line utilties to use TVM as an ahead of time compiler
[ ] Visualization of Relay Graphs

Backend and runtime

[ ] End to end uTVM
[ ] More dynamic model support

[ ] Complete VM functionality
[ ] Improve VM performance
[ ] Add tutorial for VM

[x] External code generator
[ ] End to end inference with Chisel VTA
[ ] CUDA half2 data type support
[x] bfloat16 support
[ ] 4-bit model support

Automation

[ ] More auto scheduling
[ ] Better loop partitioning
[ ] Reduce AutoTVM tuning time
[ ] Auto tensorization
[ ] Auto quantization

We will have a good support for pytorch including dynamic and quantized models. I have a series of PRs coming.

Could you also add support for intel graphics as well. it seems it is not supported at the moment!

External codegen integration with uTVM is highly interesting for us at CEVA.
We are looking at compiling networks using TVM and generating code with our implementation of codegen and then executing on CEVA hardware via uTVM.
The goal (beside getting it done) is to share via TVM github repo.

@Coderx7 as far as i know, intel graphics is already supported for a limited setting by @Laurawly . @etom42 it would be great if you can start a RFC thread in the discus forum to keep the community aware of what you are working on -- it also brings chance for collaboration and give suggestions :)

Is there any plan to integrate TVM as a dialect into MLIR?
So other components based on MLIR can leverage the capability of TVM, such as high performance codegen,  and fusion, etc.

@yangjunpro Perhaps it worth to start a new thread in the discuss forum to discuss MLIR related topics. We certainly would love some proposals about interpolation between MLIR and TVM.

I wonder is there any active effort working on the feature "Auto tensorization", and I'm interested in contributing.

@liangfu  auto tensorization depends on on the new IR update, we will send out specific guidelines after we finish the IR refactoring

I am currently working on some end-to-end model stuff, and Relay's optimization pass are too slow. Any plans on improving the compilation speed?

I am currently working on some end-to-end model stuff, and Relay's optimization pass are too slow. Any plans on improving the compilation speed?

Can you identify why they are slow? For example switching some passes to the new iterative graph visitor might improve speed quite a bit. In previous profiling attempts I have found a significant amount of time is spent allocating, having a pooling allocator can often improve speeds quite a bit.

I'd like to propose creating Relay c_api.h for further expansions from frameworks and bindings in the future.

re relay c_api, there is no need to do so, as all the relay functions are exposed through the PackedFunc API. So all the features of relay can already be accessed through the tvm's C API.

What is the expected time of release for this release? what are the chances of it happening in May?

Due to recent situation, we might expect a bit of delay in the release to June/July. We will do our best to keep the timeframe.

Due to recent situation and the current progress, we might expect a bit of delay in the release to June/July -- we expect the unified IR refactor to land by then. We will do our best to keep the timeframe.

Including more operator should be scheduled as least for one frontend? Onnx or tensorflow or mxnet etc, it will be great if 'ONE'  frontend is widely supported!!

We will have general support for TensorFlow control flow and tensor array, which allows parsing for TensorFlow object detection models

Please see https://github.com/apache/incubator-tvm/issues/6421 for release plan

"Which are summarized summarized from the forum discussion." double summarized may be a small mistakes.

v0.8 cycle tracking issue is created https://github.com/apache/tvm/issues/7434


Keras clone_model does not work if a concatenate layer with one element to concatenate is used.

Click to expand!
Issue Type
Bug
Have you reproduced the bug with TF nightly?
Yes
Source
source
Tensorflow Version
tf 2.13 (nightly)
Custom Code
No
OS Platform and Distribution
Windows
Mobile device
No response
Python version
3.9
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
Na
GPU model and memory
No response
Current Behaviour?
If I build a keras model that contains a concatenate layer which only concatenate one element, then the `clone_model` function will fail on that model.

My assumption is that the reason for this behaviour is that the parameter input of this model is a Tensor instead of a List. If I take a look on concatenate layers with multiple inputs, then the input parameter is a list.

I suppose that the error is that the `self._preserve_input_structure_in_config` is not set to true.

Standalone code to reproduce the issue
from tensorflow.python.keras.layers import Concatenate, Input
from tensorflow.python.keras.models import clone_model, Model

input1 = Input(shape=(1,2), name="input")
concat = Concatenate()([input1])

model = Model(input1, concat)
model.compile()

print("Model exist, now lets try to clone the model..")

cloned_model = clone_model(model) # This line fails

Relevant log output
...\scratch_2.py", line 12, in <module>
    cloned_model = clone_model(model)

.....

tensorflow\python\keras\layers\merge.py", line 491, in build
    raise ValueError('A `Concatenate` layer should be called '
ValueError: A `Concatenate` layer should be called on a list of at least 1 input.

Note I also opened an Issue at Keras since I am not sure where its fit better.

@benHeid,
Thank you for opening this issue. Development of keras moved to another repository.
Could you please feel free to close this issue, as this issue is already tracking there in Keras repo.
https://github.com/keras-team/keras/issues/17700
Thank you!


Save MIDI data as SMF

Should be able to export the events from a MIDI Monitor window to a Standard MIDI file, with timing included.

That would be so great to have.
And it wouldn't need to know anything about tempo, we can do all that later when we drag the MIDI into a DAW...
I realize I'm pulling up a 14 year old thread... :-)
But perhaps this would be something I could try and learn to build - I would just have to learn how to convert the buffer to MIDI... hmmm...


Not possible to checkout a specific commit?

Issue Type: Bug
I almost can't believe this, but is it truly not possible to checkout an arbitrary commit by hash? Nothing happens if I paste a commit hash/id into the checkout quick command. Do I really have to use a 3rd party client to roll back time? :)
Same problem as Microsoft/vscode/issues/30280
VS Code version: Code - Insiders 1.26.0-insider (5fae91d61f12726e149fe5baed342eac2fbf8f88, 2018-07-25T01:56:56.548Z)
OS version: Darwin x64 18.0.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz (8 x 2500)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: unavailable_softwarevideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
16.00GB (0.22GB free)

Process Argv
/Applications/Visual Studio Code - Insiders.app/Contents/MacOS/Electron

Screen Reader
no

VM
0%

Extensions (30)

Extension
Author (truncated)
Version

Bookmarks
ale
9.0.2

path-intellisense
chr
1.4.2

vscode-svgviewer
css
1.4.4

vscode-markdownlint
Dav
0.18.0

githistory
don
0.4.2

jupyter
don
1.1.4

vscode-pandoc
Dou
0.0.8

gitlens
eam
8.5.3

vsc-material-theme
Equ
2.3.0

prettier-vscode
esb
1.5.0

vscode-diff
fab
1.3.2

latex-workshop
Jam
5.7.0

python
ms-
2018.7.1

PowerShell
ms-
1.8.1

wordcount
ms-
0.1.0

vsliveshare
ms-
0.3.472

gremlins
nho
0.7.0

vscode-docker
Pet
0.0.27

quicktype
qui
12.0.5

vscode-icons
rob
7.24.0

partial-diff
ryu
1.2.0

markdown-preview-enhanced
shd
0.3.5

vscode-spotify
shy
2.5.1

vscode-fileutils
sle
2.10.3

slack
soz
0.0.14

code-spell-checker
str
1.6.10

vscode-terminal-here
Tyr
0.1.1

vscodeintellicode
Vis
1.0.1

vscode-todo-highlight
way
1.0.4

markdown-all-in-one
yzh
1.6.0

(2 theme extensions excluded)

<!-- generated by issue reporter —>

git checkout <commit id> in the integrated terminal. No real need for an extension.

/duplicate #30280

Fair enough 🙂

On the other hand I don't see why the "Git: Checkout to..." command shouldn't be able to take a branch name as well as a commit id. Should be trivial.


[NDSL] Refactor constants

Aggregate all constants in one file
Put in place a data-driven test for precision verification against reference Fortran

Are we just going to stick with these names for the foreseeable future or is the plan still to move to a new naming scheme at some point soon?

I'd say same as the rest of the codebase: move to something that's not two letters, keep the original name in the comments and/or a file that does spells out the correspondence


v1.1.0 - store blue, tab label by @title or last folder and filename

Since this wasn't merged nor reviewed I will close this pull request. Sorry.
Cheers
SM


Errors when working with .tsx files

Hey there, currently getting a lot of errors from the extension when working in TypeScript React files:
[Error - 10:50:00 PM] Request textDocument/codeAction failed.
  Message: ValueError: 'typescriptreact' is not a valid LanguageSelector
  Code: -32602
[object Object]

Fixed with release of 1.6.0


fix: module field should point to a real file

📦 Published PR as canary version: 0.0.12--canary.21.ebe57b5.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

Hey @elevatebart, great catch! Thanks a lot for your contribution!


Not able to use sendTransaction

Hello Team,
We are facing an issue after launching wallet connect v2.
We are not able to use sendTransaction method with contract integration in the Flutter mobile application.
We tried different libraries and packages to resolve this issue but still didn't get any proper solution to this. Kindly guide us to resolve this issue.
I have a question, Does the new Wallet connect V2 provide any provider?
If yes, Please help us out with this

I have attached a part of the Code for Send Transaction here for your reference
Kindly let me know if there is something wrong with this code.
As I simply want to use  send transaction method through Metamask.
Waiting for your quick response.
Thanks

Have you reviewed the ETH transactions in the example dapp?

Here is the example I put together for send transaction.
It should have everything you need to build the image above.

Closing this as complete.


Remove explicit dependencies of zlib and jpeg

These are dependencies of HDF4/HDF5 and should thus automatically come in via those packages (if the hdf libraries use them)

Checklist

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

@conda-forge-admin, please rerender


Files with wrong CSS class.

The files on the TagFolder pane have the wrong css class.
Current class: nav-folder
Correct class: nav-file.

@rodrigo-rufino
Sorry for the delay!
I didn't use themes that have different appearances of files and folders, I didn't catch it.
This problem has been fixed.
Could you please try v0.9.0?


Fix up handling of subdirs for PAR2

On PAR2 creation, write data paths relative to the par2 file.
Return an error sooner when data paths aren't in the base path.
Convert some delegate functions to take paths.
Add tests for decoder/encoder behavior.

This will implement #9 for PAR2.

@brenthuisman this should fix your repro case in #9. Please try this out and let me know if it behaves as you expect. If so, I'll clean it up a bit and then merge.

Great! I'm getting a compile error though:
# command-line-arguments
cmd/par/main.go:316:25: too many arguments in call to par2.NewEncoder
        have (par2LogEncoderDelegate, string, []string, int, int, int)
        want (par2.EncoderDelegate, []string, int, int, int)

I'm guessing a signature somewhere wasn't updated; is the parFile argument (first string passed onto NewEncoder) new?.

Yes, it's new. However, the NewEncoder signature is also updated -- perhaps you might need to clear some caches or something.
(Generally, go install is the way to go, since it functions like make, if you're using go build that might not always work.)

Hmm, I'm getting it with go install too. For the record, I checked out this PR and ran GOBIN=~/go/bin go install cmd/par/main.go in the repo dir. I purged ~/go/pkg/mod/cache but no effect.

This repo doesn't use mod, I think you have to clear ~/go/pkg/{arch}, so on linux it would be ~/go/pkg/linux_amd64 (which on my system has github.com/akalin/gopar/)
For example, I just ran:
> git checkout par2-subdirs
Already on 'par2-subdirs'
Your branch is up to date with 'origin/par2-subdirs'.
> rm -rf ~/go/pkg
> rm ~/go/bin/par
> go install ./...
> ~/go/bin/par
Error: no command specified
...

Right, deleting ~/go/pkg/linux_amd64 did it.
The issue in #9 is now working correctly, so we can consider it resolved!

These commits haven't been merged to main/master yet, right? They've seem to have time shifted or something ;)

Yeap, still tinkering with it (and its dependencies)! As you can see, newly-merged CI has caught at least one thing. :)


SQLSTATE[42000] fix not working for Laravel 5.8

Laravel Version: 5.8.3
PHP Version: 7.3.2
Database Driver & Version: MariaDB 10.1.38

Description:
Fix for SQLSTATE[42000]: Syntax error or access violation: 1071 Specified key was too long; max key length is 767 bytes (SQL: alter table users add unique users_email_unique(email)) from https://laravel.com/docs/master/migrations#creating-indexes is not working for latest Laravel
Steps To Reproduce:

fresh Laravel 5.8.3 installation
App\Providers\AppServiceProvider:

<?php

namespace App\Providers;

use Illuminate\Support\ServiceProvider;
use Illuminate\Support\Facades\Schema;

class AppServiceProvider extends ServiceProvider
{
    /**
     * Register any application services.
     *
     * @return void
     */
    public function register()
    {
        //
    }

    /**
     * Bootstrap any application services.
     *
     * @return void
     */
    public function boot()
    {
        // Fix for MySQL < 5.7.7 and MariaDB < 10.2.2
        // https://laravel.com/docs/master/migrations#creating-indexes
        Schema::defaultStringLength(191);
    }
}

execute php artisan make:auth
create new database for your connection (don't change config files - only .env file)
execute php artisan migrate

SQLSTATE[42000]: Syntax error or access violation: 1071 Specified key was too long; max key length is 767 bytes

If you encounter this error its because your mysql(MariaDB) Version Framework has nothing to do with it
If You add
Schema::defaultStringLength(191); to AppServiceProvider All the fileds in the migration will have the length of 191   so if you dont want ot specify the length of the colunm for each table you can add to AppserviceProvider

What does your migration look like?

@ManojKiranA: Ekhm... Please read my ticket again. I already added Schema::defaultStringLength(191) into AppServiceProvider
@staudenmeir: This same as @ManojKiranA - read my ticket again ;) I didn't change anything except my user, password and database for MySQL in .env
users:
<?php

use Illuminate\Support\Facades\Schema;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Database\Migrations\Migration;

class CreateUsersTable extends Migration
{
    /**
     * Run the migrations.
     *
     * @return void
     */
    public function up()
    {
        Schema::create('users', function (Blueprint $table) {
            $table->bigIncrements('id');
            $table->string('name');
            $table->string('email')->unique();
            $table->timestamp('email_verified_at')->nullable();
            $table->string('password');
            $table->rememberToken();
            $table->timestamps();
        });
    }

    /**
     * Reverse the migrations.
     *
     * @return void
     */
    public function down()
    {
        Schema::dropIfExists('users');
    }
}

password_resets:
<?php

use Illuminate\Support\Facades\Schema;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Database\Migrations\Migration;

class CreatePasswordResetsTable extends Migration
{
    /**
     * Run the migrations.
     *
     * @return void
     */
    public function up()
    {
        Schema::create('password_resets', function (Blueprint $table) {
            $table->string('email')->index();
            $table->string('token');
            $table->timestamp('created_at')->nullable();
        });
    }

    /**
     * Reverse the migrations.
     *
     * @return void
     */
    public function down()
    {
        Schema::dropIfExists('password_resets');
    }
}

i think the problem is that by default laravel uses the utf8mb4 charset which uses more space than the utf8 charset. You can either change the mysql settings to allow for bigger keys or just change the charset to utf8 and the collation to utf8_general_ci in config/database.php under the mysql section. Had this problem more than once and not just with laravel.

So documentation need to be updated

The docs should cover everything you need:
Index Lengths & MySQL / MariaDB

Laravel uses the utf8mb4 character set by default, which includes support for storing "emojis" in the database. If you are running a version of MySQL older than the 5.7.7 release or MariaDB older than the 10.2.2 release, you may need to manually configure the default string length generated by migrations in order for MySQL to create indexes for them. You may configure this by calling the Schema::defaultStringLength method within your AppServiceProvider:

use Illuminate\Support\Facades\Schema;

/**
 * Bootstrap any application services.
 *
 * @return void
 */
public function boot()
{
    Schema::defaultStringLength(191);
}
Alternatively, you may enable the innodb_large_prefix option for your database. Refer to your database's documentation for instructions on how to properly enable this option.

Route::Post does not work!
I have to make it get then give a hidden input as method= post
the hidden input value does not go into mysql db!!!
the route and view, action all checked! but mysql returns an error which state the ... field does not have a valid value?? but the value is set in the view form!

change bigIncrements to increments (ID).

just update your mariadb to the latest version 10.4.7, it will fix it ;)
also for $table->json()
happy coding !!

Am facing the same problem but to me it is failing to add other tables in the database apart from the user and password table

i think the problem is that by default laravel uses the utf8mb4 charset which uses more space than the utf8 charset. You can either change the mysql settings to allow for bigger keys or just change the charset to utf8 and the collation to utf8_general_ci in config/database.php under the mysql section. Had this problem more than once and not just with laravel.

Thanks @justicenode it work for me

I use MySQL 8.0.18, create database using collation utf8mb4_general_ci, make sure the MySQL engine is InnoDB, successfully run php artisan migrate

i think the problem is that by default laravel uses the utf8mb4 charset which uses more space than the utf8 charset. You can either change the mysql settings to allow for bigger keys or just change the charset to utf8 and the collation to utf8_general_ci in config/database.php under the mysql section. Had this problem more than once and not just with laravel.

work for me thanks

The error persists:
Illuminate\Database\QueryException
SQLSTATE[42S01]: Base table or view already exists: 1050 Table 'users' already exists (SQL: create table users (id bigint unsigned not null auto_increment primary key, name varchar(191) not null, email varchar(191) not null, email_verified_at timestamp null, password varchar(191) not null, remember_token varchar(100) null, created_at timestamp null, updated_at timestamp null) default character set utf8mb4 collate 'utf8mb4_unicode_ci')
at vendor/laravel/framework/src/Illuminate/Database/Connection.php:671
667|         // If an exception occurs when attempting to run a query, we'll format the error
668|         // message to include the bindings with SQL, which will make this exception a
669|         // lot more helpful to the developer instead of just the database's errors.
670|         catch (Exception $e) {

671|             throw new QueryException(
672|                 $query, $this->prepareBindings($bindings), $e
673|             );
674|         }
675|

  +9 vendor frames

10  database/migrations/2014_10_12_000000_create_users_table.php:24
Illuminate\Support\Facades\Facade::__callStatic("create")
  +22 vendor frames

33  artisan:37
Illuminate\Foundation\Console\Kernel::handle(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))

i think the problem is that by default laravel uses the utf8mb4 charset which uses more space than the utf8 charset. You can either change the mysql settings to allow for bigger keys or just change the charset to utf8 and the collation to utf8_general_ci in config/database.php under the mysql section. Had this problem more than once and not just with laravel.

work for me thanks

It Worked for me...Thanks!!!

i think the problem is that by default laravel uses the utf8mb4 charset which uses more space than the utf8 charset. You can either change the mysql settings to allow for bigger keys or just change the charset to utf8 and the collation to utf8_general_ci in config/database.php under the mysql section. Had this problem more than once and not just with laravel.
also have a look at @devcircus' comment below.

Great! It works. Thanks.


Regarding an issue #2436

I'm facing with the same problem as described here
https://github.com/AFNetworking/AFNetworking/issues/2436
an issue was closed without explaining a reason.
The problem is that an app crashes when upload task is configured to be performed  in the background calling  a method  AFURLSessionManager uploadTaskWithStreamedRequest
NSURLSession Configuration example:
NSURLSessionConfiguration * sconfig = [NSURLSessionConfiguration backgroundSessionConfigurationWithIdentifier: backgroundSessionId];
    sconfig.sessionSendsLaunchEvents = YES;
    sconfig.discretionary = YES;

upload in the background works  with the method AFURLSessionManager uploadTaskWithRequest.  but for some reason it doesn't include request parameters but only file data itself. Here is the request
NSMutableURLRequest *urlRequest = [[AFHTTPRequestSerializer serializer]multipartFormRequestWithMethod:@"POST" URLString:uploadUrlStr parameters: params constructingBodyWithBlock:^(id<AFMultipartFormData>  _Nonnull formData) {

	[formData appendPartWithFileURL:entity.fileUrl  name:@"data" error:&fileappenderror];

} error:nil];

if you make an upload request you will see that all parameters included in request are missing. but only   the file has been uploaded.

Would you be able to create a test case in the 3.0.0 branch? If we could get a reproducible failure there, that would be very helpful to track down and ensure that isn't a problem going forward.

Sure. Thank you for your input. I will report results later

@kcharwood  Hi. Done. It's about 150 Lines of code. Shall I submit the pull request or just insert it here (or pastbin) to check the code?

Yes feel free to up a PR, or just a forked branch and link it here, and I can work off of that!


Gpkg export

Continuation of the work on Import.  . This is a method that allow exporting all OrgUnit and Groups linked to a Data Source' version. Just as a Draft since it's not finished yet but like this you can preview and I can brain dump
I have developed it in parallel with the old one for comparison. But it the short them I will merge both implementation so we don't  have two separate code base to maintain. This one should be a complete super set of the other.  I added a some columns such as the groups and a table for the groups.
The file exported can be imported again with the opposite command which should improve our data portability greatly. I have added a basic tests of exporting OrgUnit and reimporting them but didn't test a lot of edge case.
Short todo list mostly as a reminder to myself

[x] Document more
[x] Merge two implementation and verify we didn't lost any feature
[x] Test exporting existing big data set.
[ ] CANCEL Add API to expose it
[ ] CANCEL Add it in the UI
[x] Test group not existing in file at import
[x] Multiple OrgUnit of the same type with different geography type (point and polygon) and so they should end up in the same layer  but it's not supported by qgis

Edge cases not handled:

If there is a - in the  OrgUnitType name. It should export but won't reimport
If there is a , in the source_reference field of a group version and this group contain org unit
If the parent is in a separate Data Source version. It will work at import but not when reimporting. That's not something we want in the database anyway but there is no constraint in place.

Theses edge case are mostly problem for the story of exporting and reimporting in another Iaso

we don't really have an existing UI at the moment to view separate source version so I will report the ui to later since we can export from the search view.
caveat is we miss potential group if they are not present on a orgunit but worst case we recreate them

The Import branch should be merged first since this one build on it


Migrate Code to TypeScript

PR for #21

@ozdemirburak Though eslint had suggested multiple changes in the code, I have done only those that do not require a major change. I have turned off all the other such rules in .eslintrc.json file.

This looks great, thanks @drishit96.
Would you also like to also create another PR for separating the logic in separate files? Like characters.ts, audio.ts, etc.?

No, but thank you for asking 😄


Add IndexedParallelIterator::[try_]fill_slice

Motivation: #713 is difficult to implement without external tracking of which items have been written in, so instead we offer try_fill_slice which requires the user to provide dummy values that can always be dropped (which isn’t usually too difficult). The implementation uses entirely safe code.
Public API:
trait IndexedParallelIterator {
    fn fill_slice(self, target: &mut [Self::Item]);

    fn try_fill_slice<T>(
        self,
        target: &mut [T],
    ) -> <<Self::Item as Try>::Residual as Residual<()>>::TryType
    where
        Self::Item: Try<Output = T>,
        T: Send,
        <Self::Item as Try>::Residual: Residual<()> + Send;
}

The reason for adding Residual is to enable the generic function of Option<T> → Option<()>, Result<T, E> → Result<(), E> etc which wouldn’t be possible otherwise.

I'm wary of replicating the TryType type constructor, because eventually we'd like to transition to the official Try, but nothing in std has stabilized this TryType mechanism yet. I wouldn't want to add something in our API that makes us incompatible with whatever std lands on. At least the Try trait is constrained by the stable ? operator and try_fold, etc.
But if you have a ready fill buffer, can't you (as a user) already do this with for_each and try_for_each?
my_par_iter.zip_eq(&mut target).try_for_each(|(src, dest)| {
    *dest = src?;
    Ok(()) // or whatever your Try type is...
)

(i.e. we don't need generic TryType if you're writing that for your specific situation.)

Ah, I didn’t realize that I could do that 😅. Thanks a lot, I’ll probably close this then


Add a moving "thinking" logo when seed papers are being searched for

So that you don't feel so confused about waiting.

Also when it's finding Zotero collections


Error: Unknown input type "multi-step" in kitchen sink

Reproduction
https://formkit.link/31bc4f96e6bb75042194f6eab18f2fff
Describe the bug
The new FormKitKitchenSink component throws an error on the multi-step form component. There are more warnings, but they are probably a side-effect from the error.
Environment
• OS: macOS Sonoma 14.2
• Browser: Google Chrome 120
• Version: 1.4.x

@matsn0w This is just a matter of the required dependencies, such as @formkit/addons, not being added into the formkit.config of the playground. Here is a workaround with the appropriate import maps and formkit.config: https://formkit.link/03a2ddfc87b68677811ac6c9b8f45803. Alternatively, you can disable certain groups of inputs from the FormKitKitchenSink component by passing a schemas array prop with the values of the inputs you want to view, or remove pro entirely: https://formkit.link/7bad4be8b8bc6f9e20f0292804eabf05

🙈


Update deploy script [skip ci]

Need to upload 'dist/pom.xml' as the spark-rapids project's pom file, because OSS takes the unspecified/auto-generated pom as invalid.
Need to deploy a jar without any classifier tag as the main jar of integration-test, because OSS requires the main jar without classifier.
Signed-off-by: Tim Liu<EMAIL_ADDRESS>

To fix issue https://github.com/NVIDIA/spark-rapids/issues/3837


Xamarin Android not play wav file

            try
            {
                using (var stream = new FileStream(fistBank, FileMode.Open))
                {
                    var waveFile = new WaveFile(stream);
                    signal = waveFile[Channels.Left];
                }
            }
            catch(Exception ex)
            {
                Toast.MakeText(Application.Context, ex.ToString(), ToastLength.Short).Show();
            }

Added to the project, added permissions for reading files, I get the path to the file. I press the button and silence ... The file does not play ... It does not even generate signals from examples

This code is not supposed to play audio file. It just parses WAV file and loads data into DiscreteSignal object (essentially, array of audio samples). In NWaves only the very basic playback/recording is supported and only for Windows clients. The main purpose of this lib is signal processing. So, for audio playback you'll need to write some Android native code (maybe using Android.Media.MediaPlayer class) or use Xamarin audio/media plugins/packages that can be found over Internet.

That is, receive sound from signal if from the code above?
I tried it through BASS anyway silence
            using (var stream = new FileStream(fistBank, FileMode.Open))
            {
                var waveFile = new WaveFile(stream);
                signal = waveFile[Channels.Average];
                int length = signal.Length;
                byte[] buffer = new byte[length];

                _hGCFile = GCHandle.Alloc(buffer, GCHandleType.Pinned);

                if (Bass.BASS_Init(-1, 44100, BASSInit.BASS_DEVICE_DEFAULT, IntPtr.Zero))
                {
                    int b_stream = Bass.BASS_StreamCreateFile(_hGCFile.AddrOfPinnedObject(), 0L,length, BASSFlag.BASS_SAMPLE_SOFTWARE | BASSFlag.BASS_SAMPLE_FLOAT);

                    if (b_stream != 0)
                    {
                        Bass.BASS_ChannelPlay(b_stream, false);
                    }
                }
            }


removed a setup action initializer

We absolutely need multicall here. It's used by the team to setup mints on the contract.

We could move this logic to the factory and set the initial admin as the factory then create the correct admin roles and remove the factory role at the end? or allow the factory contract to override checks for permissions with an override branch.

this is a draft and was demoed as part of the graphite workshop...will delete


Styles.css import doesn't export content rc.1

Using 2.0.0-rc.1 with webpack 2.1.0-beta.25 I get the following error:
"/Users/Projects/red-web/node_modules/extract-text-webpack-plugin/loader.js?{\"omit\":1,\"remove\":true}!/Users/Projects/red-w
eb/node_modules/style-loader/index.js!/Users/Projects/red-web/node_modules/css-loader/index.js?modules&importLoaders=1!/Users/Proj
ects/red-web/node_modules/postcss-loader/index.js!/Users/Projects/red-web/packages/red-components/src/Expandable/styles.css doesn't export content
",

ExtractTextPlugin.extract(
    { fallbackLoader: 'style-loader', loader: 'css-loader?modules&importLoaders=1!postcss-loader' }
  )

Reverting to 2.0.0-rc.0 fixes the issue

Any chance for a little project to study?

Please re-open with a project to study. Closing for now.


Fix moon phase

Moon phase corrected!

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 82.92683% with 7 lines in your changes missing coverage. Please review.

Files with missing lines
Patch %
Lines

src/core_position.c
0.00%
6 Missing :warning:

src/main.c
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files with missing lines
Coverage Δ

src/astro.c
12.50% <100.00%> (+2.56%)
:arrow_up:

test/astro_test.c
100.00% <100.00%> (ø)

src/main.c
0.00% <0.00%> (ø)

src/core_position.c
0.00% <0.00%> (ø)

... and 3 files with indirect coverage changes


Fix image upload failures when using django-storages (Google Cloud Storage, Azure Storage)

Rewind file to the beginning after reading
Fixes #4738

Merged in 550e8f648cdf91b8dfbcf3cdba4a84481c647463 (master) and 064d2d15444750582c8424fa725b06e03863ae47 (2.2.x) - thanks @m1kola!


More output file name quirks

Summary:
As noted in this issue, the treatment of paths as strings in CmdStan can lead to strange behavior.
Description:
Treating a string as a path, and subsequently splitting on last dot produces a family of problems.
Reproducible Steps:
I am certain that more examples can be produced, but here are a few interesting cases.
Assuming that you have the bernoulli example built and your current directory is examples/bernoulli/bernoulli:
Case 1
rm -rf foo* && mkdir -p foo.bar

# produces a file "baz" in directory "foo.bar"
./bernoulli data file=bernoulli.data.json sample num_chains=1 output file=foo.bar/baz
ls -a foo.bar

# produces no files
./bernoulli data file=bernoulli.data.json sample num_chains=3 output file=foo.bar/baz
ls -a foo.bar

Case 2
rm -rf foo* && mkdir -p foo/.bar

# produces a file "baz" in directory "foo/.bar"
./bernoulli data file=bernoulli.data.json sample num_chains=1 output file=foo/.bar/baz
ls -a foo/.bar

# produces no files
./bernoulli data file=bernoulli.data.json sample num_chains=3 output file=foo/.bar/baz
ls -a foo/.bar

Debug of Case 2
CmdStan is splitting on the '.' and producing output file names of the form foo/_${id}.bar/baz and since _${id}.bar is a non-existent directory, no files are created. If we create said directories ahead of time, the writer succeeds.
rm -rf foo* && mkdir -p foo/_1.bar foo/_2.bar

# Creates "foo/_1.bar/baz" and "foo/_2.bar/baz" but not "foo/_3.bar/baz"
./bernoulli data file=bernoulli.data.json sample num_chains=3 output file=foo/.bar/baz
ls -a foo/.bar

Case 3
rm -rf foo* && mkdir -p foo/bar

# produces no file(s); ".." is not a writable file
./bernoulli data file=bernoulli.data.json sample num_chains=1 output file='foo/bar/..'
ls -a foo
ls -a foo/bar

# produces files "._1.", "._2.", "._3." in directory "foo/bar"
./bernoulli data file=bernoulli.data.json sample num_chains=3 output file='foo/bar/..'
ls -a foo/bar

Current Output:
Summarized from reproducer above:

Case
file (input)
num_chains == 1
num_chains > 1

1
foo.bar/baz
foo.bar/baz
nothing; not possible to recover with pre-existing directory

2
foo/.bar/baz
foo/.bar/baz
foo/_${id}.bar/baz iff pre-existing foo/_${id}.bar directory

3
'foo/bar/..'
nothing
foo/bar/._${id}.

Expected Output:

Case
file (input)
num_chains == 1
num_chains > 1

1
foo.bar/baz
foo.bar/baz.csv
foo.bar/baz_${id}.csv

2
foo/.bar/baz
foo/.bar/baz.csv
foo/.bar/baz_${id}.csv

3
'foo/bar/..'
error
error

One could argue that the errors should be handled by substituting the default file name after path normalization, but it is ill-formed as a directory was provided when a file was required.
Additional Information:
Current Version:
v2.33.1

all of these problems could be solved by using Boost filesystem library, but it's not a header-only library, which will complicated the build process.
without Boost filesystem, the problem of correct interpretations of current and parent directories, "./" and "../" respectively, is problematic.

all of these problems could be solved by using Boost filesystem library, but it's not a header-only library, which will complicated the build process. without Boost filesystem, we can kludge up checks.

An alternative could be the std library filesystem, but that requires C++17 (still get to stay header-only at least!)


browserslistrc - remove unsupported browsers

Some suggested changes for .browserslistrc
Here's the current one
npx browserslist@latest --update-db
npx browserslist

and_chr 89
and_ff 86
and_qq 10.4
and_uc 12.12
android 89
baidu 7.12
chrome 89
chrome 88
chrome 87
chrome 86
chrome 85
chrome 84
chrome 83
chrome 81
chrome 80
chrome 79
chrome 78
chrome 77
chrome 76
chrome 75
chrome 74
chrome 73
chrome 72
chrome 71
chrome 70
chrome 69
chrome 68
chrome 67
chrome 66
chrome 65
chrome 64
chrome 63
chrome 62
chrome 61
chrome 60
edge 89
edge 88
firefox 86
firefox 85
firefox 84
firefox 83
firefox 82
firefox 81
firefox 80
firefox 79
firefox 78
firefox 77
firefox 76
firefox 75
firefox 74
firefox 73
firefox 72
firefox 71
firefox 70
firefox 69
firefox 68
firefox 67
firefox 66
firefox 65
firefox 64
firefox 63
firefox 62
firefox 61
firefox 60
ios_saf 14.0-14.5
ios_saf 13.4-13.7
ios_saf 13.3
ios_saf 13.2
ios_saf 13.0-13.1
ios_saf 12.2-12.4
ios_saf 12.0-12.1
kaios 2.5
op_mini all
op_mob 62
opera 73
opera 72
safari 14
safari 13.1
safari 13
safari 12.1
safari 12
samsung 13.0
samsung 12.0

There are 2 selection methods (> 0.5%, last 2 major versions versus Chrome >= 60).
> 0.5%, last 2 major versions  and Not Chrome < 60 would give chrome 89, 88, 87
A lot of transpiling is going on because of the older Chrome/FF, e.g.
proposal-optional-catch-binding { "chrome":"60" } proposal-async-generator-functions { "chrome":"60" }

The Chinese browsers are unsupported, so add:
not baidu < 10.0
not QQAndroid < 20
not and_uc < 20
not kaios < 20

As are Opera Mini (and older Opera mobile?), so add:
not OperaMobile < 59
not OperaMini all

Debug note:
With .bablercjs, add debug: true and useBuiltins: 'usage'
  bugfix/transform-tagged-template-caching { "android":"88", "chrome":"60", "edge":"88", "firefox":"60", "ios":"12", "opera":"72", "safari":"12", "samsung":"12" }
  proposal-async-generator-functions { "chrome":"60" }
  proposal-export-namespace-from { "chrome":"60", "firefox":"60", "ios":"12", "safari":"12" }
  proposal-json-strings { "chrome":"60", "firefox":"60" }
  proposal-logical-assignment-operators { "chrome":"60", "firefox":"60", "ios":"12", "opera":"72", "safari":"12", "samsung":"12" }
  proposal-nullish-coalescing-operator { "chrome":"60", "firefox":"60", "ios":"12", "safari":"12", "samsung":"12" }
  proposal-numeric-separator { "chrome":"60", "firefox":"60", "ios":"12", "safari":"12" }
  proposal-optional-catch-binding { "chrome":"60" }
  proposal-optional-chaining { "android":"88", "chrome":"60", "edge":"88", "firefox":"60", "ios":"12", "opera":"72", "safari":"12", "samsung":"12" }
  proposal-unicode-property-regex { "chrome":"60", "firefox":"60" }
  syntax-dynamic-import { "android":"88", "chrome":"60", "edge":"88", "firefox":"60", "ios":"12", "opera":"72", "safari":"12", "samsung":"12" }
  syntax-object-rest-spread { "android":"88", "chrome":"60", "edge":"88", "firefox":"60", "ios":"12", "opera":"72", "safari":"12", "samsung":"12" }
  syntax-top-level-await { "android":"88", "chrome":"60", "edge":"88", "firefox":"60", "ios":"12", "opera":"72", "safari":"12", "samsung":"12" }
  transform-dotall-regex { "chrome":"60", "firefox":"60" }
  transform-named-capturing-groups-regex { "chrome":"60", "firefox":"60" }

We are fine as things are right now.

To clarify, I'm not against in any cleanup if needed. I just don't want us to spend any time on this any further unless it has any clear benefits. Feel free to submit a PR that clearly demonstrates the pros and cons.
The plan isn't to just be last 2 major versions only, at least in my eyes. I don't want us to contribute to browser monopoly. So, unless there are clear, visible gains, I don't think there's any reason for us to change .browserslistrc.
FYI the change with Safari was meant to happen before 5.0.0-beta1.

Fair enough, "last 2 (of any browser)" and "chrome etc >= 60" make sense


Microsoft.Data.SqlClient query instrumentation

Committed in #56; docs still need an update, but Instrument.SqlClientCommands() covers 99% of it :-)


remove intermediate containers

do you mean -rm=true here?
https://github.com/ctindel/ubiquiti-docker/blob/master/unifi/Makefile#L10
https://github.com/ctindel/ubiquiti-docker/blob/master/unifi-video/Makefile#L10

Yes, I did mean to do that.  Nice catch!
Fixed in https://github.com/ctindel/ubiquiti-docker/pull/41


[OCCM] Fix CI

The binaries affected:

[ ] All
[x] openstack-cloud-controller-manager(OCCM)
[ ] cinder-csi-plugin
[ ] k8s-keystone-auth
[ ] client-keystone-auth
[ ] octavia-ingress-controller
[ ] manila-csi-plugin
[ ] manila-provisioner
[ ] magnum-auto-healer
[ ] barbican-kms-plugin

What this PR does / why we need it:
Fix the load balancer service CI job. The new version of kubectl run can only create pod, what we expect is a deployment.
Which issue this PR fixes:
fixes #
Special notes for reviewers:

Release note:

NONE

This change is

/lgtm

/approve


Give Sources control of invalidation for Snapshots they produce.

This moves the hard-coded 5-minute TTL out of the cache and into the
registry-based implementation of cache.Source. The hack that makes
snapshots produced by "virtual" sources is no longer necessary.
/hold
Depends on #2632 now, but could be rebased onto master without much trouble if the other PR gets stuck.

/hold cancel

/lgtm