Update to kubernetes 1.17 client libraries

Motivation for this is to get new EndpointSlice changes, and we can get rid of all of the horrible replace hacks in go.mod

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
7924bd470e125dd234d681c4e31ae43014e73727
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
7924bd470e125dd234d681c4e31ae43014e73727
link
/test lint_istio

gencheck_istio
7924bd470e125dd234d681c4e31ae43014e73727
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
7924bd470e125dd234d681c4e31ae43014e73727
link
/test lint_istio

gencheck_istio
7924bd470e125dd234d681c4e31ae43014e73727
link
/test gencheck_istio

unit-tests_istio
7924bd470e125dd234d681c4e31ae43014e73727
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
7924bd470e125dd234d681c4e31ae43014e73727
link
/test lint_istio

unit-tests_istio
7924bd470e125dd234d681c4e31ae43014e73727
link
/test unit-tests_istio

gencheck_istio
7fcb5321fc068272379931617dd3f1ca53929b75
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_istio
7924bd470e125dd234d681c4e31ae43014e73727
link
/test lint_istio

gencheck_istio
7fcb5321fc068272379931617dd3f1ca53929b75
link
/test gencheck_istio

unit-tests_istio
7fcb5321fc068272379931617dd3f1ca53929b75
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Test failure is blocked by https://github.com/kubernetes/kubernetes/issues/86394, will see what they say and can do a workaround if needed

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_istio
7fcb5321fc068272379931617dd3f1ca53929b75
link
/test gencheck_istio

unit-tests_istio
7fcb5321fc068272379931617dd3f1ca53929b75
link
/test unit-tests_istio

lint_istio
1e40ccceeaeee7e8759fb2a0596027217f4ad4f8
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

unit-tests_istio
7fcb5321fc068272379931617dd3f1ca53929b75
link
/test unit-tests_istio

lint_istio
1e40ccceeaeee7e8759fb2a0596027217f4ad4f8
link
/test lint_istio

gencheck_istio
1e40ccceeaeee7e8759fb2a0596027217f4ad4f8
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Add advanced filter

This allows you to search on any item in the underlying data. It also attaches the path of the keys to each value so you can hone in on a particular value.
For example:

running -> shows you all linodes with "running" inside some value of the underlying object
status=running -> shows you all linodes with a status running key value pair
168 -> shows you all linodes with 168 inside (for instance IPs)
backups:enabled=false -> shows you all linodes who do not have backups enabled

And so forth.

Coverage decreased (-0.2%) to 72.372% when pulling 16c91760ad356eb039a3da5ede8350b884b7eeb8 on eatonphil:add-advanced-filter into f1ffca0cb58ee9d4f8292f65dee22ca10dd8b908 on linode:develop.


The merge button is enabled also if it's not possible to merge the PR

Extension version: v0.018.0
VSCode Version: 1.47.1
OS: macOS

Steps to Reproduce:

Checkout a PR
Click on "Request Change"

now the PR cannot be merged from GitHub interface
the "Merge Pull Request" button is enabled

@mcosta74 Do you remember why the PR could not be merged? Were there failing checks, conflicts, was it your own PR?

The was not mergable because I requested a change in my review.
Unfortunately it's a private repository so I cannot give access.
Later I'll try to reproduce on a public repo and I'll put more info


Support for search

In the latest version of wouter, when the location is somewebsite.com/users?name=john
const [location] = useLocation();

location only retrieves /users emitting ?name=john. And even though I push new query params with other methods, re-rendering is not triggered. But sometimes a view needs to depend on search params.
I think we have two options in a broad sense

add useSearch thing
const [search, setSearch] = useSearch();
search // "?name=john"

or we can let wouter give searchParams as a URLSearchParams object, the parsed one.
or how about useURL which gives current location as URL object?

let location be location.pathname + location.search.

breaking change!

In this case, we got a new problem to redefine routing rules. Should useRoute provide matching for search parameters? I don't think so. Routing rules can go the same just ignoring search.
And the parsing for location.pathname + location.search is annoying. AFAIK, there is no builtin function to parse it. And even if there is, the code to parse would be repeated in many components.

@worudso @dpr-dev Hey guys, thanks for the issue.
Been thinking about this a lot and what I had in mind is following:
// useLocation hook may return a third argument, which is basically a hash
// of query params used; this is completely optional and apps that use
// the hook already won't break
const [location, setLocation, queryParams] = useLocation();

// the signature of the useRoute hook stays the same, however it may return query params
// in the params hash along with dynamic segments
const [match, params] = useRoute("/app/users/:id");

// example: /app/users/alex?tab=settings
// -> { id: "alex", tag: "settings" }

In my opinion this can be one of the optimal solutions because:

Doesn't break the existing API
The query + route params merging technique is widely used, for example in Rails or Express (I guess).

This is my fork
https://github.com/ownerclan/wouter/blob/master/use-search-params.js
And I implemented useSearchParams for my immediate needs.
I'm not saying we'd better go useSearchParams. Instead, I'm saying @molefrog 's new useLocation might be the right choice. I hope my experiences that dealt with search params help you make a better decision.
The current version of my useSearchParams returns an array of [key, value] tuples. But for convenience, returning an object with keys of params is best. I had used that version of useSearchParams at first. It's like react's old state and setState which allowed partial update of a state. This design had made my code looks better.
But there's a problem that there can be more than two params with the same name at once(This kind of usage actually exists in my project). And URLSearchParams object allows it. So we need to forget setState-like setSearchParams when we don't want to give an strong opionion about the usage of search params.
I think we'd hetter leave this problem(manipulating search params in an elegant way) to another utility library(lodash, immer, etc), and focus on conservative representation of URLSearchParams. But URLSearchParams object really sucks, I recommend to use an array of [key, value] tuples. We can map and filter and do another array operations on it while we can't with URLSearchParams.
And there's another problem that with useSearch thing we have multiple ways to update current url. It makes the behavior of setLocation and setSearchParams ugly.

setLocation can update path and search params and setSearchParams can only update search params and should preserve path.

So I 100% agree with new useLocation. But useRoute has a problem I metioned before. It can't be used with multiple keys with the same name. I agree that new useRoute can be useful in some case. But adding that feature in the future will not be breaking changes. How about keeping old simple useRoute behaviour and delaying changes for now?

@worudso Not sure if I understand what you mean by:

there can be more than two params with the same name at once

You mean the search query might contain things like ?options[]=for&options[]=bar?

@molefrog yes, that's the case. Whether it's recommended or not, it's already supported by native 'URLSearchParams'.

@molefrog, very nice implementation

@molefrog's implementation looks like what we need now

Has this landed?

@kilianc No, not yet 😞 There is a discussion going on in #102
The problem isn't in implementing this, but providing a good design that will enable us to keep the size low, plus will be generic enough.

I think separating location.pathname and location.search tracking is the right approach.
Some components don't need to react on query string changes, while some do.
If the hook useLocation is going to track both, it will introduce performance regressions.
For now, to track search (query string changes) you could use:

https://github.com/streamich/react-use/blob/master/docs/useSearchParam.md

I like the separation in react-use between:

useLocation
useSearchParam

I believe this library should do the same due to performance reasons and ability for a project to import only what's needed for their specific needs.

Hey, I've created a tiny hook to solve this problem
const useQuery = (l) =>{
    //handmade query parser
    const generateQuery = () => {const result = {};location.search.split('?')[1].split('&').filter(i=>i).map(q=>{q = q.split('=');result[q[0]] = q[1]});return result}
    //create state to be able to share with components
    const [q, setQ] = useState(generateQuery())
    //set state every time location changes
    useEffect(()=>{
        setQ(generateQuery())
    }, [location])
    return q
}

Usage :
const [location, setLocation] = useLocation()
const query = useQuery(location)

Any news about this? It would be great to have access to the search map!

I tried to use the useSearchParam hook from react-use but it was not reacting when using wouter's Link to navigate.
const getValue = (param) => new URLSearchParams(window.location.search).get(param)

function useSearchParam(param) {
    const [value, setValue] = useState(() => getValue(param))

    useEffect(() => {
        const onChange = () => setValue(getValue(param))

        window.addEventListener('popstate', onChange)
        window.addEventListener('pushstate', onChange)
        window.addEventListener('replacestate', onChange)

        return () => {
            window.removeEventListener('popstate', onChange)
            window.removeEventListener('pushstate', onChange)
            window.removeEventListener('replacestate', onChange)
        }
    }, [param])

    return value
}

This is because the events they are listening to are pushstate and replacestate instead of wouter's pushState and replaceState. I don't know who is correct here. Following the SO link from the wouter source code, looks like it should be pushstate and replacestate. It's also consistent with the popstate and other window events names.
I'm leaving this here in case someone else also runs into this problem when using react-use with wouter.

I tried to use the useSearchParam hook from react-use but it was not reacting when using wouter's Link to navigate.
const getValue = (param) => new URLSearchParams(window.location.search).get(param)

function useSearchParam(param) {
    const [value, setValue] = useState(() => getValue(param))

    useEffect(() => {
        const onChange = () => setValue(getValue(param))

        window.addEventListener('popstate', onChange)
        window.addEventListener('pushstate', onChange)
        window.addEventListener('replacestate', onChange)

        return () => {
            window.removeEventListener('popstate', onChange)
            window.removeEventListener('pushstate', onChange)
            window.removeEventListener('replacestate', onChange)
        }
    }, [param])

    return value
}

This is because the events they are listening to are pushstate and replacestate instead of wouter's pushState and replaceState. I don't know who is correct here. Following the SO link from the wouter source code, looks like it should be pushstate and replacestate. It's also consistent with the popstate and other window events names.
I'm leaving this here in case someone else also runs into this problem when using react-use with wouter.

Correct me if I'm wrong but it looks like an update in regards to joaopaulobdac's issue above means a component will rerender if the query params change. So I believe you can just use URLSearchParams and long as you use useLocation within the component.
const [location, setLocation] = useLocation();
const urlSearchParams = new URLSearchParams(window.location.search);
const queryParams = Object.fromEntries(urlSearchParams.entries());

is it possible to add search dynamically?
https://codesandbox.io/s/wouter-demo-nested-routes-forked-qmg6q?file=/src/index.js
when pressed add query it will add query but remove previous params.
how to test:
Press "Help Center" and navigate to topics. url will shown help/topics and pressed add query it will remove /topics not extending the url.

Copying and pasting my response from #177:
The correct way to do this would be something like the example shown here: https://github.com/molefrog/wouter/issues/232
Tbh, not sure why search changing is causing re-rendering by default... that seems misguided. Since there's currently no subscribe function exported, could easily write our own:
const events = ["popstate", "pushState", "replaceState", "hashchange"];
function subscribe(callback: () => void) {
    for (const event of events) {
         window.addEventListener(event, callback);
    }
    return () => {
        for (const event of events) {
            window.removeEventListener(event, callback);
        }
    }
}

function getCurrentValueOfMyParam() {
    return someFunctionOf(window.location);
}

And now we can simply do...
const myParam = React.useSyncExternalStore(subscribe, getCurrentValueOfMyParam);

No re-rendering if myParam hasn't changed, even if a million other things in the url have!
(Notice: no mucking with Router is necessary at all!)

Any solution ?

It works for me.
import React from "react";
import ReactDOM from "react-dom/client";
import { Router } from "wouter";
import useBrowserLocation from "wouter/use-location";
import App from "./App";

ReactDOM.createRoot(document.getElementById("root") as HTMLElement).render(
  <React.StrictMode>
    <Router
+      hook={(options) => {
+        const { pathname, search } = new URL(location.href);
+
+        const [_, nav] = useBrowserLocation(options);
+
+        return [pathname + search, nav];
+      }}
    >
      <App />
    </Router>
  </React.StrictMode>
);


Plugin showing an error

I am having the same issue as https://github.com/simplifycom/woocommerce-simplify-payment-gateway-plugin/issues/23
with the current version,

Hey @showponygraphics We made a change to your platform a couple of months back that should have resolved this issue. If it is still occurring could you please raise this through the bank's support team - we'll be able to track your request better. We are going to be closing the issues tab here as we can't track these questions & queries as well as through the regular support route.


Request trove classifier for Trio

I'd like to request the addition of the trove classifier Framework :: Trio, for libraries in the Trio ecosystem. (Examples: 1, 2, 3, 4, 5, 6, 7, 8.) We're reaching the point where I keep being surprised by new libraries I haven't heard of, and I'd rather get the classifier going sooner rather than later to minimize the number of projects we need to track down and add it to...

@njsmith would you anticipate versioned sub classifiers?

Done!

would you anticipate versioned sub classifiers?

Not unless we make some terrible mistakes :-)

Done!

Thank you!


controller: nrop: always populate reason (progressing)

The API spec requires the condition.Reason to always contain a non-empty CamelCase string. We added the infra to do so but we forgot for the Progressing case.
This change fixes that.
This becomes evident if RTEs fail to start, otherwise it's easy to miss, like it probably happened till now.

/lgtm
/approve

more tests are coming with a followup PR


Report tile utilization with DVFS info

It seems that the folder /test/kernels is only useful in ICEDdeom.py. You already have CGRA-Benchmark in your CGRA-Flow and the 'fileSource' of ICEDdemo.py can be replaced with any folder path. Do we need to keep the folder /test/kernels?

Hi @HobbitQia, can u plz help take a look at the failed test? Does it look like this time the offset of 2 is unnecessary?

e

u r right. I checked the opcode count in line 650-660 and I thought their original opcodes are correct. So this time we should set this paramter to be 0.🤔

I saw u change the directory names from nonlinear_test, idiv_test to nonlinear, idiv. However, the working directories we set in cmake.yml are still ${{github.workspace}}/test/nonlinear_test, ${{github.workspace}}/test/idiv_test, respectively. I think it's the reason that the github testing failed.

Right, I recognized that but forgot to push my fix. Thanks!


CUPS-Get-Printers not listing printers anymore

Starting of v2.1, CUPS doesn't list all the printers anymore when sending IPP "CUPS-Get-Printers"
Sending to: ipp://username:password@localhost:631
{
"operation": "CUPS-Get-Printers",
"operation-attributes-tag": {
"attributes-charset": 'utf-8',
"attributes-natural-language": 'en-us'
}
}
I'm expecting to see a list of printers in the printer-attribute tag. I first thought it was the new added printer_id, which would filter (and thus skip printers) though having tested version 2.1 I can exclude this.
The above request is working on 1.7.5, however failed since 2.1 and also fails the latest 2.2 build 2.2.7.
I saw the test itself is only checking for successfull-ok, but not if there is an array of printers (which should be expected).

I'm not familiar with exactly how the Node.js IPP library works, but I can say that if something like this broke we would have heard about it long before now. I suspect you have run into a bug in the Node.js library you are using.


hooks: fix failing tests with mismatched renderer

There's a few tests failing, currently marked as allowed failures in https://github.com/airbnb/enzyme/commit/3db6013d9f5b9034594b78f5c182fc4146af4841, that shouldn't be failing.
Ideally, the functionality can be fixed; if not, then the tests should be skipped when the breakages are expected.

@ljharb I would like to work on it can you explain what all needs to be done?

@AlokTakshak sure! I'm not exactly sure what needs to be done to fix it; but basically, remove these two test jobs from travis.yml, and then fix the react 16 adapter so the tests pass :-)

@ljharb  is it closed now?

@AlokTakshak no, those jobs are still allowed failures.

@ljharb  i am running test cases in my local it is passing

In that case, a PR that solely removes the allowed failures would close this issue :-)

@ljharb  yeah Sure i will raise it ASAP

#2122

@ljharb
npm run test:all
for me it is passing in local but in in travis

You have to use the RENDERER env var with npm run react if you want to replicate what happens in Travis.

You have to use the RENDERER env var with npm run react if you want to replicate what happens in Travis.

@ljharb even after that it is still passing in my local, not able to reproduce  the travis-ci results


Upgrade Ricochet.app to v1.1.1

Thank you for the contribution. It was merged directly as 3b305998c86d308ac0384a2de125eb832b9c286a to keep commit history cleaner. Your contribution is still credited to you.


Fake website in first line of search

Dear Bloxstrap creators,
I came to report a site that recently appeared on the first page of the search, being a blatant copy of the original. It is already known that the reliable links would be here on GitHub and at https://bloxstraplabs.com/ , however, there is a site that ended up appearing suspicious and outside of what was reported: https://bloxstrap.org/ is the site I am reporting. A copy that is possibly not authored by you. I say this so that I do not have future problems with new visitors to the application.
I patiently await a response about this site. With all due respect,
mairellos.

there is nothing we can do about the fake websites

Just saying you can report the site to google as adware/malware and an illegal copy.
Or put it in your website/github that the other is fake, because I went to that one first and now have to scan my entire pc for malware lol
This is an actual issue if people are going to that site and downloading malware or spyware without knowing.


Control using Blynk App

Code is located in the following path:
https://github.com/AnandVetcha/HackerBox/blob/master/Box13/RC_car/Blynk_Ultrasonic.ino
We can close this issue


Pd fails to download, with an error that seems like a dns issue

Hello!
I discovered your app just yesterday and I gave it a shot. However, it does not work for me and pops an error that looks like a dns issue. In detail, pd download something returns this, either with just the id or with the full url
./pd download h81i3AGM
 0 B / 74.75 KiB [--------------------------------------------------]   0.00% 0s
Get "https://sia.pixeldrain.com/api/file/h81i3AGM": dial tcp: lookup sia.pixeldrain.com on <IP_ADDRESS>:53: no such host

./pd download https://pixeldrain.com/u/h81i3AGM
 0 B / 74.75 KiB [--------------------------------------------------]   0.00% 0s
Get "https://sia.pixeldrain.com/api/file/h81i3AGM": dial tcp: lookup sia.pixeldrain.com on <IP_ADDRESS>:53: no such host

I noticed that <IP_ADDRESS> there, which is google's dns that I use on my system, so I changed it to something else, e.g. cloudflare's (<IP_ADDRESS>) or quad9's (<IP_ADDRESS>), but the only thing that changes is that ip and nothing more.
./pd download h81i3AGM
 0 B / 74.75 KiB [--------------------------------------------------]   0.00% 0s
Get "https://sia.pixeldrain.com/api/file/h81i3AGM": dial tcp: lookup sia.pixeldrain.com on <IP_ADDRESS>:53: no such host

./pd download h81i3AGM
 0 B / 74.75 KiB [--------------------------------------------------]   0.00% 0s
Get "https://sia.pixeldrain.com/api/file/h81i3AGM": dial tcp: lookup sia.pixeldrain.com on <IP_ADDRESS>:53: no such host

Debian testing/unstable x64, pd from the x64 binary (= not the .deb package, I may install it later) from the releases page and the requested file does exist in pixeldrain's site (verified via a browser).
If it helps, I have ipv6 disabled system-wide via a kernel parameter (ipv6.disable=1).

Same thing with the arm64 binary on a tvbox running coreelec (arm64 distro). It definitely has ipv6 enabled, so ipv6 connectivity is not to blame.

Thanks for reporting the issue. I didn't know that but they changed download URLs.
v0.4.2 should solve this issue.

It now works on both the amd64 and arm64 systems, so I am closing this.
Thank you very much for the quick reply and the quick fix!


Document how to use default error channel

As a user, I'd like to handle the error conditions and route the custom error messages to an error channel.
For a sample use-case, see this SO thread.
Acceptance:

add a note about the default error channel and its applicability
sample code included

@sabbyanandan  Where is the documentation?

@stavalfi is this what you're looking for https://docs.spring.io/spring-cloud-stream/docs/Elmhurst.BUILD-SNAPSHOT/reference/htmlsingle/#binder-error-channels?

Hi @olegz and thanks for the fast response!
I'm breaking my head on this paper to understand how to actually set up everything in both consumer and producer side (code & properties).
Do you guys have any example online?

@stavalfi perhaps you can help us a bit. Given that I've pointed you to our documentation and you expressing some lack of clarity means our documentation is not clear enough and that is something we need to fix.
What I would like from you (if at all possible) is to explain in a bit more details your use case, what are you trying to accomplish etc. This will help both you in getting the answer you're looking for and us to clarify documentation.

@olegz great idea.
First, I explicitly want to note that I consider my self as a beginner with Spring framework.

Short problem description: Lacking (?) of Spring Cloud Stream - Error Handling documentation

What I'm looking for:
(1)  What are you trying to solve?

Spring Cloud Stream supports publishing error messages received by the Spring Integration global error channel. Error messages sent to the errorChannel can be published to a specific destination at the broker by configuring a binding for the outbound target named error.

Is it a failure which related to my logic or spring's logic?
Will I be able to send my own errors?
What is the difference between an error channel and channel destination?
When I'm using reactive API - will I still need this solution?

(2) List of all classes, annotations, and properties which related to Spring Cloud Stream - Error Handling.

spring.cloud.stream.bindings.error.destination
ErrorMessage
MessagingException
MessagePublishingErrorHandler

Is that all?
(3) Explicitly write the relationship between every 2 classes/annotations, if exist, and explicitly write why there is a connection between them.
(4)  An example for each class, annotation, and property. Also, for each property, explicitly write what can be/must be specified inside the consumer and the producer.

@stavalfi Thank you so much, let us work on this.
Meanwhile, one last request; Could you please raise a new issue (given that this is old and closed) and link to this?

@olegz the real thanks is to you guys.
Please fill free to let me know how can I help more.


sql server2008 企业版是支持cdc的，但是代码提示版本过低

尝试自己用源码编译后可以通过

@JackTianZhao Hi, sql server 2008 cann't pass all unit test and version is too old. I don't suggest you to do this , maybe cause production accident.

hope ur great team can support this version!

希望能尽快支持，现在依然有不少老版本的在跑，这会让flink-cdc更宽容！！

hope flink can support this version!

Closing this issue because it was created before version 2.3.0 (2022-11-10). Please try the latest version of Flink CDC to see if the issue has been resolved. If the issue is still valid, kindly report it on Apache Jira under project Flink with component tag Flink CDC. Thank you!


Document @since in the Rules' Wiki

At least for the latest versions available, in https://github.com/inaka/elvis_core/wiki/Rules. Kind of like @since 0.7.0.
Try to find a way to properly document the past for the other rules.
@elbrujohalcon, should we get rid of "Old format" in the Wiki? We can probably create a section to help developers migrate, but I think the semantics used (these days, using maps) is pretty self-explanatory.

Yeah… the old format is clearly not used anymore anywhere… We could remove it from everywhere.

And @since seems like a really nice thing, but retrofitting it to account for all rules… I think it would be too much work for no gain. We can start with @since 0.7.0 for all rules.

Yeah… the old format is clearly not used anymore anywhere… We could remove it from everywhere.

Done. There's old_configuration_format (which is "by default") so it shouldn't be hard for consumers to know there's something wrong.

As for since I went back to 0.4.0 (I just compared tags, in Git, so it was easy) and now we have it: https://github.com/inaka/elvis_core/wiki/Rules/_compare/7c99460d686073000d98b968ecb89cf507ba9359...fb84866eed10a4691cdd280aeb6def100a70f1a1.


Disabling timestamps in groovydoc

Hi,
There is an ongoing effort in Debian to make every builds reproducible and the timestamps embedded in the files generated by groovydoc is one of the many issues to solve in order to achieve this goal.
I'd like to suggest the addition of a notimestamp option for groovydoc similar to the one in javadoc, it would make it possible to remove the timestamps when needed.
Thank you

+1 this has become a real problem for us too because we use gh-pages to publish our docs so each time we publish it creates huge commits because every javadoc has changed
It also prevents us from publishing snapshots because the change sets are too big.

The patch only removes the timestamps and preserves the version of Groovy used (javadoc's -notimestamp option removes both). Removing the version might be desirable too in your case, otherwise you'll still get large change sets when upgrading to a new version of Groovy.

upgrading Groovy is less frequent, so I'm not as concerned about that

It wouldn't be that hard to make both configurable but I'll have a bit of a closer look at what the JavaDoc code does before commenting further here if needed. It seems like this is a fairly straight-forward change even if some minor tweaking might be desirable.

Here is the link to the relevant code in javadoc:
http://hg.openjdk.java.net/jdk8u/jdk8u/langtools/file/f166261986cc/src/share/classes/com/sun/tools/doclets/formats/html/markup/HtmlDocWriter.java#l329

JavaDoc doesn't have version as an option, so I think a separate flag would make sense for that. I am happy to merge your request and add the extra flag in the next day or two hopefully.

I did a minor tweak to ensure that the new behavior was backwards compatible by default but otherwise merged your proposal. I also then added a noversionstamp flag too. Thanks!

Thank you!

Actually the issue is not completely fixed, I didn't notice the templates also include the date with a <meta> tag:
<meta name="date" content="${new Date().format('yyyy-MM-dd')}">

This line should be removed when -notimestamp is specified.

Amended. Nice pick up.

Thank you for the quick fix

I added another commit to enable the same support in the ant task.


Half star or Partial star not supported!

<StarRatingComponent
   name="review-star"
   starCount={5}
   value={3.5}
/>

This doesn't work! Is there any way?

<StarRatingComponent
   name="review-star"
   starCount={5}
   value={3.5}
/>

This doesn't work! Is there any way?

Do you have find another component?

<StarRatingComponent
   name="review-star"
   starCount={5}
   value={3.5}
/>

This doesn't work! Is there any way?

Do you have find another component?

No


Domain changed

Domain change to:
https://x.com/i/tweetdeck

And?

And...?

Only FYI. 😉

This has been known for months..


Add error code 0-5, and more code of issuing error code 0-3, with code optimization.

Codecov Report

Merging #10598 (6cc2edd) into 3.1 (eeac1c3) will decrease coverage by 0.38%.
The diff coverage is 15.00%.

@@             Coverage Diff              @@
##                3.1   #10598      +/-   ##
============================================
- Coverage     64.76%   64.37%   -0.39%
+ Complexity      397      395       -2
============================================
  Files          1331     1331
  Lines         56564    56528      -36
  Branches       8377     8348      -29
============================================
- Hits          36632    36391     -241
- Misses        15987    16172     +185
- Partials       3945     3965      +20

Impacted Files
Coverage Δ

...pache/dubbo/common/concurrent/RejectException.java
25.00% <ø> (ø)

...figcenter/file/FileSystemDynamicConfiguration.java
0.00% <0.00%> (ø)

.../org/apache/dubbo/common/cache/FileCacheStore.java
84.09% <25.00%> (-3.27%)
:arrow_down:

...ache/dubbo/common/cache/FileCacheStoreFactory.java
92.30% <100.00%> (ø)

...ava/org/apache/dubbo/common/profiler/Profiler.java
95.83% <100.00%> (ø)

...mmon/serialize/fastjson2/FastJson2ObjectInput.java
0.00% <0.00%> (-62.50%)
:arrow_down:

...mon/serialize/fastjson2/FastJson2ObjectOutput.java
0.00% <0.00%> (-56.25%)
:arrow_down:

...dubbo/remoting/zookeeper/ZookeeperTransporter.java
50.00% <0.00%> (-33.34%)
:arrow_down:

...on/serialize/fastjson2/FastJson2Serialization.java
57.14% <0.00%> (-28.58%)
:arrow_down:

...ng/zookeeper/curator5/Curator5ZookeeperClient.java
43.86% <0.00%> (-25.48%)
:arrow_down:

... and 65 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Test for api, don't reply


Add Dcode.tech to USERS.md

Please ensure your pull request adheres to the following guidelines:

Add Dcode.tech to USERS.md

Thanks for the addition!


eclib: 20210223 -> 20210226

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/eclib/versions.
meta.description for eclib is: "Elliptic curve tools"
meta.homepage for eclib is: "https://github.com/JohnCremona/eclib"
meta.changelog for eclib is: ""
Updates performed

Version update

To inspect upstream changes

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 20210226 with grep in /nix/store/gf1162sq5dxn1jj8wqqma13s78flsmi5-eclib-20210226

directory tree listing: https://gist.github.com/d08b713842bdae9a7318adf44932a24c

du listing: https://gist.github.com/75e4535518ff6284c93a43eb9bd00fd8

Rebuild report (if merged into master) (click to expand)

1818 total rebuild path(s)

623 package rebuild(s)

618 x86_64-linux rebuild(s)
532 i686-linux rebuild(s)
133 x86_64-darwin rebuild(s)
535 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
_1password-gui
adapta-gtk-theme
aide
amarok
amarok-kf5
anki
ankisyncd
asciidoc-full
asciidoc-full-with-plugins
atheme
bchoppr
brutespray
btrbk
cabal2nix
calamares
calibre
calligra
ccache
ccacheStdenv
ccacheWrapper
checkstyle
chrome-gnome-shell
circleci-cli
clevis
clipgrab
cntr
codeql
consul
cq-editor
crowbar
csound-qt
denemo
dep2nix
digikam
disorderfs
dolt
domoticz
eagle
eclib
ethminer
falkon
fcitx-engines.libpinyin
fcitx5-chinese-addons
fcitx5-mozc
freecad
freerdp
freerdpUnstable
frescobaldi
fselect
ghostwriter

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/gf1162sq5dxn1jj8wqqma13s78flsmi5-eclib-20210226 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A eclib https://github.com/r-ryantm/nixpkgs/archive/a83485043fb2d733c7f9c5a96e36b48271d60394.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/gf1162sq5dxn1jj8wqqma13s78flsmi5-eclib-20210226
ls -la /nix/store/gf1162sq5dxn1jj8wqqma13s78flsmi5-eclib-20210226/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings
cc @timokau @omasanori @7c6f434c for testing.

@ryantm I just got word from the maintainer (https://github.com/JohnCremona/eclib/issues/64#issuecomment-789793808) that version bumps in eclib do not necessarily mean new releases (the project exclusively uses GitHub releases for that, tags and version bumps are to be ignored). If it's super simple to make nixpkgs-update use GH releases for deciding when to update this package, I'd appreciate the change. Don't worry about it if it takes more than 30 seconds :)

Oh, maybe the update script got this information from repology, since Arch has updated. I've emailed the Arch maintainer just in case.

Subsumed by #123692, so closing.


Do Not Build Darwin Executor on Travis-CI

This patch updates the Makefile to no longer build the Darwin executor when building on Travis-CI.

Current coverage is 30.89% (diff: 100%)

Merging #362 into master will not change coverage

@@             master       #362   diff @@
==========================================
  Files            29         29
  Lines          1719       1719
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            531        531
  Misses         1130       1130
  Partials         58         58

Powered by Codecov. Last update eb90b74...abb937d


Fix xeno hud and medhud appearing while a scout is invisible

:cl:

fix: Fixed xeno hud icons being visible while a scout is invisible.

Does the stealth cancel when dying like in 13? Otherwise this might make the scout too hard to find for marines currently

I can add that but when does stealth cancel? only on dying? when they're stunned?

Crit doesn't cancel it, death does, warrior lunge doesn't, queen neuro spit doesn't, so seems to be just death

Does the stealth cancel when dying like in 13? Otherwise this might make the scout too hard to find for marines currently

Added the stealth cancel on death


Segmentation Fault (Core Dumped) when convert whisper with int8 quantization

System information
Linux 20.04
pip Tensorflow==2.12.0
using tranformers WhisperForConditionalgeneration
I'm trying to convert from TF to tflite and quantized to int8 Whisper, using the whisper model from tranformers WhisperForConditionalGeneration. At some point the conversion crash.
Here is the colab for more details:
Colab: https://colab.research.google.com/drive/1oAVoUxRFZLkS1uqqFN8HdgRVk0IWAlsN?usp=sharing
Also I attach the Error Trace from my server running in CPU and also running in GPU (TITAN RTX 24GB).
CPU: TraceTflite.txt
GPU: TraceTflite_GPU.txt

This issue originally reported by @SantiagoMoreno-UdeA has been moved to this dedicated repository for ai-edge-torch to enhance issue tracking and prioritization. To ensure continuity, we have created this new issue on your behalf.
We appreciate your understanding and look forward to your continued involvement.

Original Issue: https://github.com/tensorflow/tensorflow/issues/61695

Hi @SantiagoMoreno-UdeA, can you try like this? https://github.com/google-ai-edge/ai-edge-torch/issues/394. Let us know if that works for you.


FirebaseUI not displaying buttons

Step 1: Are you in the right place?
Step 2: Describe your environment

Objective C or Swift: 5.0
iOS version: 12.2
Firebase SDK version: i dont know sorry
FirebaseUI version: i dont know sorry
CocoaPods Version: i dont know sorry
I Do have email enabled so that is not the problem.

Step 3: Describe the problem:
Hi recently I have used the firebase UI as a sign-in method for a test app. The buttons are not displaying for the page after you click a correct login button. But only the welcome displays as shown:

Steps to reproduce:
My code is for the login is the following:
import UIKit
import FirebaseAuth
import FirebaseUI
typealias FIRUser = FirebaseAuth.User
class LoginViewController: UIViewController {
@IBOutlet weak var loginButton: UIButton!

override func viewDidLoad() {
    super.viewDidLoad()

    // Do any additional setup after loading the view.

}

@IBAction func loginButtonPressed(_ Sender: UIButton) {
    // 1
    guard let authUI = FUIAuth.defaultAuthUI()
        else { return }

    // 2
    authUI.delegate = self

    // 3
    let authViewController = authUI.authViewController()
    present(authViewController, animated: true)
}

/*
// MARK: - Navigation

// In a storyboard-based application, you will often want to do a little preparation before navigation
override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
    // Get the new view controller using segue.destination.
    // Pass the selected object to the new view controller.
}
*/

}
extension LoginViewController: FUIAuthDelegate {
func authUI(_ authUI: FUIAuth, didSignInWith authDataResult: AuthDataResult?, error: Error?) {
print("handle user signup / login")
}
}
Observed Results:
I feel like this has a simple answer and thank you for the help ahead of time!

Please fill out the issue template.

Please fill out the issue template.
If you haven't added any Auth providers, FirebaseUI won't show any buttons.

Thanks, I just updated I thought I changed the template but it changed.


Add Google gtag plugin

Start to collect website analytics using the gatsby-plugin-google-gtag plugin with measurement ID G-9HHPS5RX9D.

Note that there is a tagmanager already installed - need to figure out who is using that id.


MySQL to Postgres - complex transformations

Hello....
I am reviewing packages for an upcoming ETL process to migrate from a MySQL db to Postgres.
As with most systems there are Accounts and Users as top level entities but primary key IDs will change etc.  We would also have a requirement to be able to run the processes for a single (or multiple Accounts) as well as potentially do-overs for individual Users.  The catch is that the schemas are going to vary significantly in some cases (which is why generic "convert this DB" scripts and libraries are not suitable).
While some information will just be discarded, Tables may need to be split or merged. Existing data will all have to port (as cleanly as possible) with existing relationships intact (with new primary keys) etc.
I did look through the issues (both Open and Closed) and didn't see anything specific that would lead me to using node-datadumps but it "feels" like I should be able to accomplish this.
I am happy to write my own bespoke system (as this will be a "one off" once everyone is migrated) but, if I can leverage this library then I'd like to.
I am not looking for specific answers right now - just a: "Yes - this library can work for the above use-case" type of response and a hope that I can ask for support as required.
Regards and thanks for a response in advance.

Hello Gary! I think datapumps is suitable for your use case. We have active applications where datapumps moves data from ERP, Mongo or mysql. We usually do transformations, e.g. read data from source, insert with a new id to the target system and maintain relations. There is no magic here, or builtin logic for schema transformation, you have to implement it manually. Also, you need to implement it efficiently, without reading all data into memory, otherwise you'll run out of it really soon. We are really busy with our projects right know, but I'll try to help if you need solution for a specific problem.

I'll keep that in mind.  I am very much at a discovery phase (but that will end soon).  I just wanted to know if I should include the library and do some tests.  This helps a lot for now.


move shapely functions to their own module(s)

cf.py has from_shapely and to_shapely and related helper functions. It feels bloated to me, and I suggest moving the shapely functions to something like shp_reader.py and shp_writer.py

Fixed in 72ea3166c86036f3bd0283378b7ca45c96be7f32


Site build and deployment

@demenech please, take a look at this PR, there are 2 issues right now:

if i install dependencies and run the server, it throws errors when navigating to subpages.
vercel deployment fails during the build.

@anuveyatsu Vercel build is working now
A note about your first point: in order to access the .mdx pages, you have to index the files into markdowndb. To do that, simply run npm run mddb. In the future, we are going to have a next plugin to execute it on next dev.


Added Velocity Units

Added Velocity Unit type
added some common velocity units
added unit tests for all current velocity units

Thanks, could you have a look at why the unit tests fail in php 7.2?

Pull Request Test Coverage Report for Build 43

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 95.468%

Totals

Change from base Build 41:
0.0%

Covered Lines:
316

Relevant Lines:
331

💛  - Coveralls

Sorry about the delay. I have fixed the issue and all language versions are now passing.


Fix perl to find xs packages

You need first get latest scripts. We already have 5.24 perl in repo.

I see and I have many changes to your latest version


Handling of compressed in-memory zarr-arrays?

I have not found documentation on this issue, so please excuse this question.
I came across this example, which shows that napari can directly display in-memory zarr-arrays. In the example, the array is iniziallied like this:
data = zarr.zeros((102_0, 200, 210), chunks=(100, 200, 210))

zarr does in fact also allow compression for in-memory arrays (not only for on-disk storage). They can be initialized like this, for example:
from numcodecs import Blosc
compressor = Blosc(cname = 'zstd', clevel = 3, shuffle = Blosc.BITSHUFFLE)
data = zarr.zeros((102_0, 200, 210), chunks = (100, 200, 210), compressor = compressor)

I am heavily using this feature in my day to day work for arrays that would, if handled uncompressed, out-grow a machine's memory by a factor of 2 to 10 (compression ratios of about 5x to 20x).
How does napari handle such datasets? Does it make sense to visualize them with napari or should I assume that most / all data is being decompressed in the process?

@s-m-e it should work just fine!
I just ran the following minimal example
import napari
from numcodecs import Blosc
import zarr
compressor = Blosc(cname = 'zstd', clevel = 3, shuffle = Blosc.BITSHUFFLE)
data = zarr.ones((102_0, 200, 210), chunks = (100, 200, 210), compressor = compressor)

with napari.gui_qt():
    viewer = napari.view_image(data, is_pyramid=False, contrast_limits=[0, 1])

and it worked well - though I just see a big white square ;-)

Note here I've added in a couple extra keyword arguments which can be good to explicitly set when working with big data. is_pyramid=False if your data is not multiresolution, and the contrast_limits as otherwise we have to guess them which triggers some compute.
Do you want to try it out and let us know if it doesn't work for you. If you run into problems we can definitely work on them as supporting big compressed data like this is a major anticipated use case.

Also, apologies that are docs aren't quite so comprehensive yet, we will work on updating them soon!

Thanks for your reply. Yes, I also tested a similar, "simple" example and it just worked. With respect to testing on larger data, sure, I did. I ran some experiments on a test dataset - see details below.
Environment

i7-3740QM & 32 GB RAM
NVidia K1000M (2GB)
Linux 4.12 x86_64 (SUSE Leap 15.1)
Python 3.7.4 (GCC 7)
napari 0.2.5
PyOpenGL 3.1.0
zarr 2.3.2
numcodecs 0.6.3
numpy 1.17.0

Dataset

Uncompressed: 441 G
Compressed: 34 G
compression ratio: 13x

>>>> d
<zarr.core.Array '/data' (239316, 1100, 900) int16 read-only>
>>>> d.shape
(239316, 1100, 900)
>>>> d.chunks
(24, 550, 450)
>>>> d.compressor
Blosc(cname='zstd', clevel=3, shuffle=BITSHUFFLE, blocksize=0)

Positive values are data (technically uint8), negative values indicate errors / special conditions.
Code
The code is opening the zarr dataset d and copying about half of it into memory in d_.
import napari
import zarr
from numcodecs import Blosc
import tqdm
import os

path = 'data.zarr'
data = zarr.open(store = path, mode = 'r')
d = data['data']

l, c = 120000, 24 * 4
d_ = zarr.zeros(
    (l, *d.shape[1:]),
    dtype = d.dtype,
    chunks = d.chunks,
    compressor = Blosc(
        cname = 'zstd',
        clevel = 1,
        blocksize = 0
        )
    ) # Subset: 12 G compressed, 221 G uncompressed
for i in tqdm.trange(0, l, c):
    d_[i:i+c] = d[i:i+c] # Copy data from disk into memory

with napari.gui_qt():
    viewer = napari.view_image(
        d_,
        is_pyramid = False,
        contrast_limits = [0, 255],
        rgb = False,
        name = 'radar',
        )

In the above example, I can actually confirm that the entire dataset d_ is being decompressed before napari becomes responsive. However, the decompressed data is not kept in memory - so napari basically works and does not kill my machine. The viewer = napari.view_image ... step requires about 3 to 4 minutes which matches the usual decompression time. Python's memory consumption is stable at 12 G (roughly equivalent to the size of the compressed dataset d_).
For comparison, I also experimented with the entire compressed dataset d directly from disk. Based on IO, the complete array is also being looked at by napari before it is shown. CPU load matches my experience from decompressing the entire dataset. Python's memory consumption is stable at around 300M, so although data is read and decompressed, the decompressed result appears not to be kept in memory. Filesystem driver's memory consumption goes up to 16G (50% of available RAM - ZFS default), also indicating that much more than file metadata (or zarr metadata plus a few chunks) is being read. At an average read speed of 100M/s I have to wait for 6 minutes until the array is being displayed and the napari window becomes responsive - which works just fine. While scrubbing through the data, Python's memory consumption remains stable at 300M and the experience is fluent.
I guess both procedures should work without napari looking at all data first. I'd say this is a bug.

Ok, I quickly traced it back to its root cause:
image.py 22: class Image
image.py 124: def __init__(...)
image.py 146: get_pyramid_and_rgb(...)
misc.py 111: def get_pyramid_and_rgb(...)
misc.py 138: currently_pyramid = is_pyramid(data)
misc.py 65: def is_pyramid(...)
misc.py 68: size = np.array([np.prod(d.shape) for d in data]) # reading all data

It's a design issue - less of a bug - in get_pyramid_and_rgb. The current design tries to provide a warning in line 167 saying that non-pyramid data was indicated but pyramid data was passed. The validation process, as it is, requires all data to be read. I am not entirely sure if there is an alternative to the currently used approach to line 68 in misc.py that would allow to keep this warning.

Thanks for chasing this down @s-m-e and sorry if I didn't appreciate from your first question that

the entire dataset d_ is being decompressed before napari becomes responsive

this is definitely a bug, we should not do this.
That check has caused some problems in other cases too - for example see this very recent discussion on dropping that check here https://github.com/napari/napari/issues/699#issuecomment-554658124. If you comment out this check can you confirm if we now don't also decompress all the data somewhere else?
I guess the decompression is happening because we aren't just calling data.shape here, which I guess zarr can do without decompression, but we're calling [np.prod(d.shape) for d in data] which is actually indexing into the array and then calling shape and we need to decompress to do that. Does that sound right?
Regardless I think its clear from multiple issues that this part of the code base needs to be improved. I'll give it some thought on exactly what we should do and a fix should go in soon

If you comment out this check can you confirm if we now don't also decompress all the data somewhere else?

Swapping ...
misc.py 138: currently_pyramid = is_pyramid(data)

... with ...
misc.py 138: currently_pyramid = pyramid

... solves my problem. Napari opens and shows images instantaneously. Confirmed.

but we're calling [np.prod(d.shape) for d in data] which is actually indexing into the array and then calling shape and we need to decompress to do that. Does that sound right?

Yes. It's actually just the for d in data loop on its own, it's not even the access to d.shape. The loop extracts stuff from data. Where numpy would offer a (memory-) view object on data's memory as d (no memcopy), zarr does not have a similar concept and must "copy" (i.e. read / decompress) data into new numpy arrays.
But I'd be surprised if there was not a better way to achieve the same without reading / decompressing all data. After all, it is meta data you're looking for ... scratching my head

Ok, good to know this fixes the problem, we'll work on a solution that avoids this call, doesn't decompress, and add some tests to make sure that we don't start decompressing again. Thanks for help finding this!!

We've come a long way with our multiscale handling since this issue was raised (including replacing is_pyramid with multiscale in the API). If you're still interested in this functionality I wonder @s-m-e if you could check this again? Otherwise maybe I or someone else on the team can try and look into this again cc @jni @DragaDoncila. Thanks!!

Sry for the late reply - just scrubbed through a compressed on-disk zarr array, 700G uncompressed, 54G compressed, using napari 0.4.16, zarr 2.13.3, numcodecs 0.10.2, numpy 1.23.4. It works just fine, napari shows up near instantaneous, memory usage of the Python process (3.10, conda-forge, on Ubuntu 22.04 LTS) never exceeding 500M (11G virtual). I guess this issue can be closed. Thanks for all the effort!


Remove unused settings from hostdefaults.

The UI for using the 3 hostdefaults settings was removed in 2010 in
c2fe77bd3d6bd62e059c9255267847d52e5b4497 (and even then, the code was
wrapped in if false anyway). The controllers were subsequently cleaned
up except for one or two minor lines in 2015.
Thus, these settings have not been used in all that time.  All of the
features they represent are handled by other means.
So, we can remove the settings values for all 3 as well as the
corresponding accessors. The only callers of those accessors only ran
meaningful code if those values had changed, but since there was no way to
change those values, all of those methods are noops.  Thus, we can remove
all of the callers.  Also, there was a schedule running one of those
methods, which can also be removed, along with its configuration value
from the vmdb.tmpl.yml.
After the removal of the settings manipulation, the method
HostController#set_credentials_record_vars doesn't do much at all, so we
can remove it.
Additionally, the Host.multi_host_update method has now been whittled down
to just updating authentications in batch, so let's rename the method to
Host.batch_update_authentication
@gmcculloug @dclarizio Please review
cc @jrafanie :scissors:

Note that I left the serialized :settings column for now, but I think it can be dropped.  I want to do more analysis first to verify it's not used for anything else though.

I don't know what this is but I :heart: it
So much :red_circle: :red_circle: :red_circle:

@Fryguy Reviewed changes and it looks good to me.  Amazed how much this ended up deleting.  :+1:
Was the comment about researching the :settings column for this PR or is that future work?

Future work.  Finding the callers to "settings" is a bit tougher. :)

Keep going @Fryguy :scissors: :fire:


Go To Symbol does not display types

VSCode Version: 1.2.1
OS Version: Windows 10

Steps to Reproduce:

Create a type
Go To Symbol - the type does not appear

Got addressed for 1.3.1.

Sweet, when will this be released?

@jeffijoe next couple of days.


add optout_type filter to identities

@imsickofmaps ready for review

@KaitCrawford if we're going to start including optout support, I'd rather this was somehow added as property that showed when an identity was opted out. I need to think about this, rather than a rushed +1

I'm quite happy with this approach. We probably need to add tests for it though.

@KaitCrawford I knew there was something bugging me about this! It's this: https://github.com/praekelt/seed-identity-store/blob/develop/identities/models.py#L95
Is not looking at that value for the address_type the thing that is the indicator. I think there's a line missing from the tests (currently) that would have highlighted the existence of this flag.

@imsickofmaps I'm not sure I understand what you mean.
Here you can see that the flag we're filtering on is what determines if the code you linked has been called https://github.com/praekelt/seed-identity-store/blob/develop/identities/models.py#L192-L199.
The purpose for this filter was to enable us to get all contacts that have opted to be forgotten so that we can delete their data. What other way would you suggest to do that?

@KaitCrawford the dangers of GitHub issues with no description and me jumping around from project to project :(
The way this should be handled is the service should be subscribed to the webhook https://github.com/praekelt/seed-identity-store/blob/develop/identities/tests.py#L639-L669


ARROW-13388 [C++][Parquet] Enable DELTA_LENGTH_BYTE_ARRAY decoder

Looks like we have DeltaLengthByteArrayDecoder implemented. Enabling it in this commit to support DELTA_LENGTH_BYTE_ARRAY decoding

@github-actions crossbow submit cpp

@kou Could you please review this change?

Could you add a test for this?

@github-actions crossbow submit cpp

@kou I added a unit test and just merged the test input file to the parquet-testing submodule. Could you please review and approve the 7 workflows awaiting approval on this PR?

Hi @kou! Could you please approve the workflows? I've addressed your review comments. Thanks for reviewing :)

@kou / @pitrou I see a failure in "C++ / AMD64 Windows 2019 C++17" irrelevant to my changes.
[ RUN      ] TestThreadPool.SubmitWithStopTokenCancelled D:/a/arrow/arrow/cpp/src/arrow/util/thread_pool_test.cc(742): error: Expected: (n_success) > (0), actual: 0 vs 0 [  FAILED  ] TestThreadPool.SubmitWithStopTokenCancelled (325 ms) [----------] 17 tests from TestThreadPool (2922 ms total)
Do all merge gates need to pass? Any suggestions on how to proceed?
Thanks!

I'll re-run it. Please wait for a while.

Green. I'll merge this.
Thanks!

We probably want to to update the docs at https://arrow.apache.org/docs/dev/cpp/parquet.html#encodings as well to indicate this is supported for reading in that table.

Here it is
https://github.com/apache/arrow/pull/13530


Low performance

I just deployed Kinesis Tee (kinesis_tee_0.1.0.zip) function as a consumer of a Kinesis stream with quite some traffic. My batch size is 100. There is no transformation and JavaScript filter function is completely trivial:
function filter(record) {
    var derivedEventNameCol = 126;

    return (record.split('\t')[derivedEventNameCol] !== "application_error");
}

Since the deployment, function is getting more and more behind (according to lambda IteratorAge metric).
CloudWatch Logs report also unexpectedly high execution time:
REPORT RequestId: bc87b6d1-08be-4981-be6e-7cb0652d6bd9 Duration: 4181.63 ms Billed Duration: 4200 ms Memory Size: 512 MB Max Memory Used: 153 MB

All invocations are successful and data is written correctly to target stream. Target stream is not throttled.
As far as I understand, the problem is that Kinesis Tee writes records sequentially to target stream record by record
Writes should be done either:

in a batch - probably the best solution as API allows to write up to 500 record in a single batch. This make it cost efficient (less API calls)
in parallel - multiple writes at the same time

What do you think about it?

Thanks for flagging - the batch appropriate is probably the most straightforward, assigning...


wrong expansion of @ in record updates

Consider the following TLA+ proof obligation:
Record == [nat : Nat, seq : Seq(Nat)]
Stream == [Nat -> Record]

LEMMA
  ASSUME NEW s \in Stream, NEW n \in Nat, s[n].seq # << >>
  PROVE  [s EXCEPT ![Head(s[n].seq)] = [@ EXCEPT !.seq = Append(@, n)]] \in Stream
BY DEF Record, Stream

The generated proof obligation is
ASSUME NEW CONSTANT s \in [Nat -> [nat : Nat, seq : Seq(Nat)]],
       NEW CONSTANT n \in Nat,
       s[n].seq # <<>>
PROVE  [s EXCEPT
          ![Head(s[n].seq)] = [s[Head(s[n].seq)] EXCEPT
                                 !.seq = Append(s[Head(s[n].seq)].seq.seq, n)]]
       \in [Nat -> [nat : Nat, seq : Seq(Nat)]]

Note the wrong subexpression "s[...].seq.seq", which should be "s[...].seq".
Workaround: introduce LET definitions in the proof obligations and replace the "@" argument of Append by the expression that it denotes.

Which version are you using? I fixed the bug and I'm getting the right proof obligation, but it still fails with a Constraint generation error message. Do you confirm that your workaround was enough to prove the obligation?

I was a bit sloppy in my description of the workaround. The following proof works for me:
LEMMA
  ASSUME NEW s \in Stream, NEW n \in Nat, s[n].seq # << >>
  PROVE  LET h == Head(s[n].seq)
         IN  [s EXCEPT ![h] = [s[h] EXCEPT !.seq = Append(@, n)]] \in Stream
<1>. DEFINE h == Head(s[n].seq)
            nsq == Append(s[h].seq, n)
            sp == [s EXCEPT ![h] = [s[h] EXCEPT !.seq = nsq]]
<1>1. h \in Nat
  BY DEF Stream, Record
<1>2. s[h] \in Record
  BY <1>1 DEF Stream
<1>3. nsq \in Seq(Nat)
  BY <1>2 DEF Record
<1>. HIDE DEF h, nsq
<1>4. sp \in Stream
  BY <1>1, <1>2, <1>3 DEF Stream, Record
<1>. QED
  BY <1>4 DEF h, nsq, sp
```tla

It appears that the upcoming SMT backend will be able to proof this lemma directly (i.e., writing BY DEF Stream, Record).


Fixes Florida 4th District Court of Appeal scraper

Needs code review whether this is a viable approach (self.html containing tuples)

I'm not sure I love this solution, but it might actually be a really elegant one. I'm going to merge and then pull locally and see if there's anything I'd do differently (yes, this is a bit backwards!)
You'll probably see a few commits as I tweak these four merges, but these look like good solutions so far.

OK, I took a look at this, and actually this approach is fine. You're hacking the API a bit, but I couldn't think of a god way to do it either, so until we do a big refactor, this will be all good! Thanks for all the contributions!


ASDF-5541 ese_parser needs to validate checksums

checked on databases, generated by Windows XP, Vista, 7, 11.

Thanks, @Agile86! We've pulled these changes into the local repo and will push them back to github once cleared there.


Prime Video [Amazon]

I have the month trial started for primevideo.com
Can anyone add the primevideo support. It is an amazon product which streams tv shows.

This isn't likely going to happen, since it's DRM-protected. (See #1753.)

@gkoelln Thanks for pointing out the related issue.
@SamWillson Please move on there for further discussions.


Publish on F-Droid

Is your feature request related to a problem? Please describe.
Google Play is not available on several non-Google-certified devices, and many people prefer to get applications outside of Google's ecosystem.
Describe the solution you'd like
F-Droid is a third-party Android repository for open source applications. Publishing Launcher there would allow those who prefer not to use Google Play Services to still use the app.
Describe alternatives you've considered
Installing the apk from the Releases page is an alternative, but it requires the end-user to ensure that the app stays up-to-date. Publishing to F-Droid would allow the end-user to be notified when an update is available.
Additional info

How do I submit my app to F-Droid?

There is also another F-Droid compatible repo with less stringent criteria for inclusion at https://android.izzysoft.de/repo/info. IzzyOnDroid publishes directly from apks on GitHub releases.

Thank you! I will make it available from F-Droid too ^^
For now, I requested IzzyOnDroid to publish the apks from the Launcher GitHub releases.
-> Related issue on GitLab

Thanks for reaching out! I'm just in the process of adding your app. You will notice I've cropped your screenshots; that's done respecting the small screen estate on mobile phones, so users have a chance to recognize something on the pictures without trouble, especially when browsing via the F-Droid app.
For publishing on F-Droid, you might wish to establish Fastlane structures in your repo (just the files, the binaries are not needed).
OK, processing finished – your app should show up here within less than 24h (I've just missed today's sync, too much to do), with English and German description, as you provided. If you wish to link there from next to the Play Store badge, you can find a matching one here. Updates will be fetched automatically within 24h of your tagging and attaching an APK (usually around 7:30 pm CE(S)T).

PS: I'd say your app should meet the inclusion criteria for "the big repo" as well – so if you wish, go ahead and open an RFP (Request For Packaging) once Fastlane is ready and tagged. It's not that difficult, and I'll pick up from there (disclosure: I'm one of the F-Droid maintainers).

Created a RFP: https://gitlab.com/fdroid/rfp/-/issues/1377
@IzzySoft - Are the fastlane structures okay?

Almost. The changelog for the current release would be 7.txt (i.e. <versionCode>.txt – no idea where you took the 111.txt name from). Else it looks fine – thanks for also adding the screenshots!
One thing I'm missing is de-DE in Fastlane: wanna take the contents of
com.finnmglas.launcher_de.zip for that?

We're on the official f-droid site!
https://f-droid.org/packages/com.finnmglas.launcher/

I'd say I give the folks a few days to notice before I remove the app from my repo then (unless you specifically want me to keep it here, that is)?

Yeah just leave it in there for a few days - thank you for your time and for having Launcher in the repo!


networking_calico: dhcp agent doesn't support multiple segmentations per network with Openstack version 2023.1 or newer

Hello,
after upgrading our Openstack to version 2024.1 (Caracal), we encountered some issues/errors regarding dhcp agent.
2024-09-10 14:06:09.628 3173368 ERROR neutron.agent.dhcp.agent [-] Unable to restart dhcp for b51178a6-eb4f-4a9e-b3cc-fd93e053ef18.: TypeError: DnsmasqRouted.__init__() takes from 4 to 6 positional arguments but 7 were given
2024-09-10 14:06:09.628 3173368 ERROR neutron.agent.dhcp.agent Traceback (most recent call last):
2024-09-10 14:06:09.628 3173368 ERROR neutron.agent.dhcp.agent   File "/usr/lib/python3/dist-packages/neutron/agent/dhcp/agent.py", line 268, in _call_driver
2024-09-10 14:06:09.628 3173368 ERROR neutron.agent.dhcp.agent     driver = self.dhcp_driver_cls(self.conf,
2024-09-10 14:06:09.628 3173368 ERROR neutron.agent.dhcp.agent TypeError: DnsmasqRouted.__init__() takes from 4 to 6 positional arguments but 7 were given

From what I have found, this seems to be due to: https://review.opendev.org/c/openstack/neutron/+/840421 (dhcp: support multiple segmentations per network) where they added segment arg here: https://github.com/openstack/neutron/blob/5f42221e3b5b9f7e0c391e7c9b88ca93a41914ec/neutron/agent/dhcp/agent.py#L263
Networking_calico dhcp agent is currently not compatible with this change as it does not accept such argument.
class DnsmasqRouted(dhcp.Dnsmasq):
    """Dnsmasq DHCP driver for routed virtual interfaces."""

    def __init__(self, conf, network, process_monitor,
                 version=None, plugin=None):
        super(DnsmasqRouted, self).__init__(conf, network, process_monitor,
                                            version, plugin)

We ad-hoc fixed it by adding *args to the __init__ method of class DnsmasqRouted(dhcp.Dnsmasq):
class DnsmasqRouted(dhcp.Dnsmasq):
    """Dnsmasq DHCP driver for routed virtual interfaces."""

    def __init__(self, conf, network, process_monitor,
                 version=None, plugin=None, *args):
        if args:
            super(DnsmasqRouted, self).__init__(conf, network, process_monitor,
                                                version, plugin, *args)
        else:
            super(DnsmasqRouted, self).__init__(conf, network, process_monitor,
                                                version, plugin)
        self.device_manager = CalicoDeviceManager(self.conf, plugin)

We chose this approach as this should keep the compatibility with older Openstack versions (Zed and older).  Simply adding segment=None would not work, as older Openstack versions does not support this variable in DhcpBase class. https://github.com/openstack/neutron/blob/unmaintained/zed/neutron/agent/linux/dhcp.py#L183-L191
Then a second problem appeared:
2024-09-10 15:22:49.344 3177378 ERROR neutron.agent.dhcp.agent [-] Unable to restart dhcp for b51178a6-eb4f-4a9e-b3cc-fd93e053ef18.: AttributeError: 'FakePlugin' object has no attribute 'get_ports'
2024-09-10 15:22:49.344 3177378 ERROR neutron.agent.dhcp.agent Traceback (most recent call last):
2024-09-10 15:22:49.344 3177378 ERROR neutron.agent.dhcp.agent   File "/usr/lib/python3/dist-packages/neutron/agent/dhcp/agent.py", line 274, in _call_driver
2024-09-10 15:22:49.344 3177378 ERROR neutron.agent.dhcp.agent     rv = getattr(driver, action)(**action_kwargs)
2024-09-10 15:22:49.344 3177378 ERROR neutron.agent.dhcp.agent   File "/usr/lib/python3/dist-packages/neutron/agent/linux/dhcp.py", line 208, in restart
2024-09-10 15:22:49.344 3177378 ERROR neutron.agent.dhcp.agent     self.enable()
2024-09-10 15:22:49.344 3177378 ERROR neutron.agent.dhcp.agent   File "/usr/lib/python3/dist-packages/neutron/agent/linux/dhcp.py", line 325, in enable
2024-09-10 15:22:49.344 3177378 ERROR neutron.agent.dhcp.agent     common_utils.wait_until_true(self._enable, timeout=300)
2024-09-10 15:22:49.344 3177378 ERROR neutron.agent.dhcp.agent   File "/usr/lib/python3/dist-packages/neutron/common/utils.py", line 742, in wait_until_true
2024-09-10 15:22:49.344 3177378 ERROR neutron.agent.dhcp.agent     while not predicate():
2024-09-10 15:22:49.344 3177378 ERROR neutron.agent.dhcp.agent   File "/usr/lib/python3/dist-packages/neutron/agent/linux/dhcp.py", line 340, in _enable
2024-09-10 15:22:49.344 3177378 ERROR neutron.agent.dhcp.agent     self.spawn_process()
2024-09-10 15:22:49.344 3177378 ERROR neutron.agent.dhcp.agent   File "/usr/lib/python3/dist-packages/neutron/agent/linux/dhcp.py", line 589, in spawn_process
2024-09-10 15:22:49.344 3177378 ERROR neutron.agent.dhcp.agent     self._spawn_or_reload_process(reload_with_HUP=False)
2024-09-10 15:22:49.344 3177378 ERROR neutron.agent.dhcp.agent   File "/usr/lib/python3/dist-packages/neutron/agent/linux/dhcp.py", line 598, in _spawn_or_reload_process
2024-09-10 15:22:49.344 3177378 ERROR neutron.agent.dhcp.agent     self._output_config_files()
2024-09-10 15:22:49.344 3177378 ERROR neutron.agent.dhcp.agent   File "/usr/lib/python3/dist-packages/neutron/agent/linux/dhcp.py", line 652, in _output_config_files
2024-09-10 15:22:49.344 3177378 ERROR neutron.agent.dhcp.agent     self._output_opts_file()
2024-09-10 15:22:49.344 3177378 ERROR neutron.agent.dhcp.agent   File "/usr/lib/python3/dist-packages/neutron/agent/linux/dhcp.py", line 1205, in _output_opts_file
2024-09-10 15:22:49.344 3177378 ERROR neutron.agent.dhcp.agent     options, subnet_index_map = self._generate_opts_per_subnet()
2024-09-10 15:22:49.344 3177378 ERROR neutron.agent.dhcp.agent   File "/usr/lib/python3/dist-packages/neutron/agent/linux/dhcp.py", line 1287, in _generate_opts_per_subnet
2024-09-10 15:22:49.344 3177378 ERROR neutron.agent.dhcp.agent     ovn_metadata_port_ip = self._get_ovn_metadata_port_ip(subnet)
2024-09-10 15:22:49.344 3177378 ERROR neutron.agent.dhcp.agent   File "/usr/lib/python3/dist-packages/neutron/agent/linux/dhcp.py", line 1214, in _get_ovn_metadata_port_ip
2024-09-10 15:22:49.344 3177378 ERROR neutron.agent.dhcp.agent     ports_result = self.device_manager.plugin.get_ports(
2024-09-10 15:22:49.344 3177378 ERROR neutron.agent.dhcp.agent AttributeError: 'FakePlugin' object has no attribute 'get_ports'

This seems to be related to:

https://review.opendev.org/c/openstack/neutron/+/905906 (Make get_ports RPC method common for the DHCP and Metadata agent)
https://review.opendev.org/c/openstack/neutron/+/886988 (Add route to OVN metadata port if exists)
https://bugs.launchpad.net/neutron/+bug/1982569

From my understanding, it seems this change was made because when using neutron-dhcp-agent with ml2/ovn on ironic/baremetal nodes, ovn metadata route was not added correctly. Dhcp agent now uses get_ports() which is on the other side exposed by MetadataRpcCallback(server side) API. Since calico uses FakePlugin to support various calls that neutron.agent.linux.dhcp.Dnsmasq makes to what it thinks is Neutron database, I think it is necessary to add get_ports() to FakePlugin.
This is what we added and it seems to have resolved our problem:
    def get_ports(self, port_filters):

        return None

I think it should be okay if it returns None, because:

Because this class doesn't speak to the real Neutron database, it follows
that the DHCP interface that we create on each compute host does not show
up as a port in the Neutron database.  That doesn't matter, because we
don't allocate a unique IP for each DHCP port, and hence don't consume any
IPs that the Neutron database ought to know about.

If the DHCP interfaces we create does not end up in Neutron db to begin with, it should return None anyway.
Expected Behavior
Networking-calico and DHCP agent implementation should work with newer Openstack releases (2023.1 and newer)
Current Behavior
Current DHCP agent implementation seems to have problems with newer Openstack releases (2023.1 and newer)
Possible Solution
As written above, but it would be great if someone else confirmed these changes would make sense and would not break current behavior.
Steps to Reproduce (for bugs)

Use Openstack release 2023.1 or newer with Calico as the network plugin and as a dhcp agent

Context
This issue was encountered after upgrading Openstack to a newer version.
Your Environment

Calico version: 3.26.4
Operating System and version: Ubuntu 22.04.4 LTS
Openstack version: 2024.1 (Caracal)

Makes sense to me - we'd take a PR to change this.  @nelljerram WDYT in terms of approach (especially use of *args)?

All sounds good to me.  For the avoidance of doubt, I believe we are only talking here about fixing up our coding to tolerate new args in some calls, and we are not talking about honouring the Neutron DB concept of segments in detail.  IIRC segments were introduced as part of the idea of a Neutron network being partly routed - i.e. it has various L2 segments, with L3 routing between them.  Calico has always been completely L3 routed - i.e. there is no L2 adjacency between workloads - and so it does not make sense for Calico to honour / implement the segment resource in detail.
Probably that was already well known to everyone here - but I thought I should say it just to make sure!


Sending RNN test

I updated the request, does still show old core?? Because right now I can see class XLAOptCallback(Callback):

Probably the "error" come that I have reused this same branch for all the pull request I have sended instead of creating one branch here for each one. But I think it should be the "same" as just merging and merging between 2 branches on same repo.


add support for nested folders and files

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

Thanks for the PR @srivastavankit! @kriskowal will take a look.
Are there any backwards-compatibility issues to consider? For example if a client on an older version pulls down a path with nested directories, will everything just work?

@blampe Yes, there shouldn't be any backward compatibility issues.

Here’s my attempt to reverse engineer a description for this change:

Currently, IDL does not detect a change to an IDL if it includes on files in subdirectories. This change enables IDL to detect such changes by digesting the hash of every file in the IDL's transitive directory tree.

@malandrew recommends holding off on landing this. We’ll discuss Monday.

Why was this landed while we're still discussing this change?


Document behavior of trailing empty lines for ExcessiveDocstringSpacing

While beginning setup to look at branch coverage for ExcessiveDocstringSpacing, I found this behavior and thought we might document it.
In this example, it autocorrects trailing empty lines without flagging them.

Before submitting the PR make sure the following are checked:

[ ] Feature branch is up-to-date with master (if not - rebase it).
[ ] Squashed related commits together.
[ ] Added tests.
[ ] Updated documentation.
[ ] Added an entry to the CHANGELOG.md if the new code introduces user-observable changes.
[ ] The build (bundle exec rake) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).

If you have created a new cop:

[ ] Added the new cop to config/default.yml.
[ ] The cop is configured as Enabled: pending in config/default.yml.
[ ] The cop is configured as Enabled: true in .rubocop.yml.
[ ] The cop documents examples of good and bad code.
[ ] The tests assert both that bad code is reported and that good code is not reported.
[ ] Set VersionAdded: "<<next>>" in default/config.yml.

If you have modified an existing cop's configuration options:

[ ] Set VersionChanged: "<<next>>" in config/default.yml.

Thank you!


Deployment: 32%

Deployment

Application deployed at live URL.
Application loads with no errors.
Application GitHub URL submitted.
GitHub repository that contains application code.

Deployed successfully


No Audio- Dell Chromebook 3100 2-in-1 (GRABBITER)

No audio on Dell 3100 2 in 1 (Gabbiter) Geminilake. All else is functioning well (touch, wifi, etc). I will add further notes on another comment for the output from alsa-info.
Output below from ChromeOS
'''
crosh> shell
chronos@localhost / $ aplay -l
**** List of PLAYBACK Hardware Devices ****
card 0: sofglkda7219max [sof-glkda7219max], device 0: Speakers () []
Subdevices: 0/1
Subdevice #0: subdevice #0
card 0: sofglkda7219max [sof-glkda7219max], device 1: Headset () []
Subdevices: 1/1
Subdevice #0: subdevice #0
card 0: sofglkda7219max [sof-glkda7219max], device 5: HDMI1 () []
Subdevices: 1/1
Subdevice #0: subdevice #0
card 0: sofglkda7219max [sof-glkda7219max], device 6: HDMI2 () []
Subdevices: 1/1
Subdevice #0: subdevice #0
card 0: sofglkda7219max [sof-glkda7219max], device 7: HDMI3 (*) []
Subdevices: 1/1
Subdevice #0: subdevice #0
'''

alsa-info output: http://alsa-project.org/db/?f=23c10e2cf06843369e094e9daa91bc65b85fd273

run alsa-info with root priveledges

New output here: http://alsa-project.org/db/?f=7b1b20ae3986f0f93cec9e7478b780ae82ca8648

Actually, new development: When I plugged headphones in, I am able to use the speakers, but I am not receiving any output from the headphones. In KDE, the speakers are listed as "Headphones (Celeron/Pentium Silver Processor High Definition Audio Stereo"

Same behavior (Speakers start to work when headphones are plugged in) on Lenovo 5i-14 Chromebook (LINDAR)


Typo at https://selwynpolit.github.io/d9book/twig#which-template-which-variables

at https://selwynpolit.github.io/d9book/twig#which-template-which-variables it should be page.html.twig instead of page.tpl.php

Resolved at https://github.com/selwynpolit/d9book/pull/84


Open .md files instead of .txt files

Will it be possible to open .md files instead of .txt files?

Hello @roguefury, I haven't thought about this. What would be the benefit of a markdown file?

Hi,
So, I've been working on a workflow that connects with Obsidian (the knowledge/note taking) platform, and todo.txt files.  Obsidian views .md files, not .txt files.  Operationally, both are txt files.  SwiftoDo on iOS allows for both .txt and .md files to be used.
I'd like to store my todo file in the same folder as my Obsidian content.  This would allow me to edit the todo file in both Obsidian and Sleek.
So, it's not that I'm expecting markdown translation in sleek, rather I'd like to view the same file in several locations (Sleek, SwiftoDo, and Obsidian)

Ok, I see. That sould not be a problem, I will add a basic .md support to the next release.

Excellent!  Thank you!

@roguefury The builds for Snap and Flathub are being generated right now and should be live within the next 30 minutes.
But also updated the binaries: https://github.com/ransome1/sleek/releases/tag/v0.8.7
Let me know if the .md support is working like expected.

Ok, thanks.  I'll get back to it when I have a moment.

Hi,
Sorry, I'm not super technical, but I do not see a MacOS build on the release page for 0.8.7.  Where can I find that?

Hi @roguefury, sorry, I though you were on Linux, so I didn't build it. But tomorrow I will build the DMG file on my Mac and upload it.

Thank you!

@roguefury , the dmg has been added to the release page. Let me know, if you're having problems with the markdown files. I gave a brief test and it seems to work well.

@ransome1 It works perfectly!  Awesome!  I can now see the file in both Obsidian and Sleek.
I'm going to post a note over on Obsidian Forums  about Sleek as another option for task management.

Glad to hear :) I might also take a look at Obsidian, looks interesting.

I too am using Obsidian and I've tried opening my todo list though I've noticed it doesn't handle the Markdown check boxes ie
- [ ] Task

I too am using Obsidian and I've tried opening my todo list though I've noticed it doesn't handle the Markdown check boxes ie
- [ ] Task

I'm afraid this has never been the scope of sleek. This app is based on the todo.txt syntax, which is not affiliated with markdown. Although sleek uses a markdown parser and is capable of opening .md files, it does not fully support markdown.
What would have been your expectations for sleek, when it opens an .md file, which inclludes - [ ]?


[Feature Request]: Add dependabot for keeping dependencies up to date

How would you like to enhance the existing project.
This would keep the dependencies up to date and ensure there are no dependency conflicts
Describe breifly how would you implement the same.
This would require creating a new GitHub action which will check the dependency version and update them
Are you contributing for a program?
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

@tusharbansal22 @anmode i would like to work on this issue

Sounds good, just wanted to check if @anmode has assigned it to someelse, @anmode ?


Run the deletion script for SageMaker resources to clean up AWS

This needs to be executed after Thursday 3/24/22.
The documentation of the task should include:
Terminal output from running the cleanup script
Screenshot from AWS console showing that no resources are left

Screenshot for the deletion and steps performed:
Run the below command to delete all the sagemaker notebooks from AWS
python deleteSagemakerStacks.py --region us-west-2 --live_run
Terminal message

Deleted sagemaker stack

Once everything is deleted Run the below command
python deleteSagemakerStacks.py --region us-west-2
As you can see no sagemaker-notebook stacks


how to update data?

Is there anyway to refresh async data? i try with :data="getDataAsync" but not work. i don't know how to do this with options. it works just once with options but i want when user clicks a button refresh data...

Check this link: https://amsik.github.io/liquor-tree/#Async-Data

@amsik thanks. is it possible without vuex and store?

i have another question too.
if we have a list of id. for example ["1", "2", "3"] how we can find them and check their checkboxes? i see find and findAll but i don't know how to pass criteria to get items with id: 1, 2, 3

You don't have to use Vuex. It is a way to control the tree's data.
options: {
      store: {
        store: Store,
        getter: () => {
          return this._myTree
        },
        dispatcher(tree) {
          this._myTree = tree
        }
      }
}

It's not possible to pass or condition. You have to do something like:
  const nodes = ['1', '2', '3'].map(id => this.$refs.tree.find({ id: id}) )

You don't have to use Vuex. It is a way to control the tree's data.
options: {
      store: {
        store: Store,
        getter: () => {
          return this._myTree
        },
        dispatcher(tree) {
          this._myTree = tree
        }
      }
}

It doesn't work because we don't have access to this._myTree inside getter and dispatcher
`treeData: any[] = [];
treeOptions = {
checkbox: true,
store: {
store: Store,
getter() {
return this.treeData;
},
dispatcher(tree: any) {
this.treeData = tree
}
}
};`

You can use arrow functions

You don't have to use Vuex. It is a way to control the tree's data.
Could you please explain HOW you control the data without using Vues? You show code using a Store, don't you? Please show a code sample with a classic data model. Thanks!

I faced the same issue that the :data prop you pass into the liquor tree would not reactively update when the tree gets edited. The workaround I used is to place event handlers on the tree like so:
<tree
      :data="treeData"
      ref="tree"
      :options="treeOptions"
      @node:dragging:finish="updateTree($event)"
      @node:editing:stop="updateTree($event)"
      @node:added="updateTree($event)"
      @node:removed="updateTree($event)"
 >

The updateTree() method then could access the update tree through the node that is emitted by the event like so:
updateTree(e) {
      let updatedTree = e.tree.model;
}

From this point on you could either update the local state of your vue component or do whatever you want with the updated tree data.
As a final touch, you can use this recursive function as a starting point to transform the data you get from e.tree.model to whatever structure you might need:
transformTreeModelToArray(treeData) {

      // Recursive function that helps you transform the data from e.tree.model to whatever you want
      function iterate(array) {
        const updatedArray = [];
        array.forEach((o) => {
          const children = iterate(o.children); // This is important to follow the children down to any level
          updatedArray.push({ name: o.data.text, id: o.data.id, children }); // Here you can transform the data to your liking. Change needed.
        });
        return updatedArray;
      }

      const updatedArray = iterate(treeData); // Here we store the transformed array created by our own function

      this.$emit("update", updatedArray); // Do whatever you want (e.g. pass it up to a parent component)
}

Of course, this is not a preferred solution and its most likely best to include vuex. However, like in my case, this offers a practical alternative to get the updated data from the tree. Good luck with this solution 👨🏽💻


"My stations" not saved at exit

I deleted the default stations and added some new ones but they are lost after the app is closed.
After the app is restarted the default 3 stations are loaded. Is this a bug ?
(Latest radiola version on Sonoma 14.0)

it seems that I had no file in "BOOKMARK_PATH" so I created an empty bookmarks.opml but there was an error: "the radio station file was corrupted".
After I deleted the bookmarks.opml file and restarting the app, "Import station from a file" fixed the problem.
Now I have a new bookmarks.opml with the correct stations.

It's definitely a bug. I was able to reproduce it and am fixing it right now.

Fix of thes bug in the new release 7.0.1


codespaces.new cert has expired, so our Codespaces launcher button doesn't work any more

Preliminary checklist

[X] I am using the latest stable version of DDEV (see upgrade guide)
[X] I have searched existing issues
[X] I have checked the troubleshooting guide
[X] I have run ddev debug test to include output below

Output of ddev debug test
Expand `ddev debug test` diagnostic information
[COPY-PASTE HERE THE OUTPUT OF `ddev debug test`]

Expected Behavior
Our codespaces launch button should work (on PR)
Actual Behavior
It goes to codespaces.new, whose cert expired a couple days ago,

Steps To Reproduce
Launch a Codepaces button, like https://codespaces.new/?ref=20240714_rfay_codespaces_http_port&repo=80927419
It goes to codespaces.new, which has expired cert.
Anything else?
Github does in fact own codespaces.new (I had no idea there was a new TLD.) So maybe they'll just fix this.
However, I don't see them using it in their setup for running a new codespace.

I agree that they will definitely fix it. And yes, they don't use it here on GitHub, so it's better to use another link.


Enable using built-in libexpat on Linux

Required to make https://github.com/wxWidgets/wxWidgets/commit/f25a88a13dc51d9d857e10b4b264c0e817bb7a0f actually work

Thanks!
Should I update submodules in the main repository or will you do it as part of another PR with more modifications to CMake files?

Yes, please update the submodules in the main repository. Thanks.


App restarting

Hello @Mini-Sylar ,
I am facing the client.ts:77 WebSocket connection to 'wss://grow-precision-pets-yields.trycloudflare.com/' failed:
client.ts:105 [vite] server connection lost. polling for restart...
I have created the app with your git hub code it was working fine but suddenly
client.ts:339
GET https://grow-precision-pets-yields.trycloudflare.com/ 400 (Bad Request)
client.ts:77 WebSocket connection to 'wss://grow-precision-pets-yields.trycloudflare.com/' failed:
client.ts:105
Please let me know to fix this.

@Anshu-Shar  What version of the template are you using?
You can find this in the package.json

{
"name": "my-node-app",
"version": "1.0.0",
"main": "web/index.js",
"license": "UNLICENSED",
"scripts": {
"shopify": "shopify",
"build": "shopify app build",
"dev": "shopify app dev",
"info": "shopify app info",
"generate": "shopify app generate",
"deploy": "shopify app deploy"
},
"dependencies": {
"@shopify/app": "^3.52.0",
"@shopify/cli": "^3.52.0",
"dotenv": "^16.4.0"
}
}
This is root package.json .web and frontend also containing the package.json which file need to check.

Hi @Anshu-Shar
This seems to be an issue with cloudflare, as i'm also experiencing this issue
However, tunneling through ngrok seems to have worked, here's a reminder on how to setup ngrok

install ngrok
in your terminal run ngrok http 3000
copy the forwading link e,g  https://<>-41-<>-71-<>.ngrok-free.app
inside the root package.json add --reset --tunnel-url=https://<>-41-<>-71-<>.ngrok-free.app:3000  note the :3000
run npm run dev and update the urls then it should work

I'll keep an eye on this and let you know once cloudflare is stable

Hi @Mini-Sylar
Thank you it is working with ngrok.

@Anshu-Shar
Seems cloudflare is stable now

Marking as closed


v1.15: Revert the change to the golang image version of hubble-relay

I mistakenly bumped up the golang image version in hubble-relay Dockerfile in the v1.15 backport. Revert it. Also, let me piggy-back the very small cosmetic change bugfix for GHA.
Fixes: #32691
Revert golang image version of hubble-relay

/test-backport-1.15

/test-backport-1.15


Use pre-command to generate mocks

Use pre-command to generate mocks as part of the release action because otherwise setting up go conflicts with the release action.

Codecov Report

Merging #999 (aa75d10) into master (89cb77a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #999   +/-   ##
=======================================
  Coverage   38.34%   38.34%
=======================================
  Files          37       37
  Lines        1836     1836
=======================================
  Hits          704      704
  Misses       1050     1050
  Partials       82       82

Flag
Coverage Δ

unittests
38.34% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Bug in helper.sh missing ending fi on line 177

Missing ending fi in the function reset_release

To reproduce error run from root folder:
./build/build.sh 1.0.0

Fixed by #30


Bump libthrift to 0.10.0

In order to use newly generated thrift classes we need to bump to a newer version.

Haven't got the time to fix the travis issue.  I think your thrift version change is probably fine, I checked our internal version was on 0.9.2, so 0.10 is probably fine.


Make TupleCTAD in it's own header file

TupleCTAD is currently in the Tuple.cpp header file but should really have it's own header file

This issue has now been fixed by commit 2a49e41a6f95c3435a9deff2d27ad2a162f764d2: https://github.com/robertshepherdcpp/pse/commit/2a49e41a6f95c3435a9deff2d27ad2a162f764d2. Closing the commit now. 12/2/2022.


Update chunking.mdx

📝 Changed routes:

…es/data-nodes/chunking

(view full size)

Commit 3b042643be224afdcbf49e58319c4fb093d15d78 (https://site-nly4zhw5z-lamatic.vercel.app).


[Feature Request] Reporting based on driver behaviour, speed control, start and stop status

From https://github.com/tananaev/traccar/issues/2308
A dashboard for reporting purposes where we will collect reporting based on driver behaviour, speed control, start and stop status... and so on

You should probably elaborate on what it should look like.

@akmal1999, please add more details to this requirement

I am closing this issue.

From OBD device we can get below parameters to know driver behaviour.
Website: http://www.thinkobd.com/, http://www.thinkobd.com/service-oem-odm.aspx
Driving behavior report:
Vibration alert, sudden acceleration, deceleration, sharp lane change, sharp turn, breaking all become a score to display front of driver, company, help business of all sizes track their vehicles and assets, remove paperwork, stay compliant, improve vehicle utilization, reduce accidents and increase.
I could see now in report, Chart "Chart Type", OBD Speed, OBD Odometer.
@tananaev, with OBD data in hand can we come up with Driving behaviour report ?
Thanks.

http://www.thinkobd.com/VT400-LTE-OBD-Ⅱ.aspx
Driving behavior analysis:
Dangerous driving behavior identification based on accurate detect, over speed driving, hard acceleration, hard deceleration, hard breaking, sharp turn, urgent lanes change, hard refuel, fatigue driving all kind of bad driving behavior should not happen any time for teenager or adult. Scoring after every trip ending, store in database as a reference judge condition for Usage Base Insurance (UBI), it is possible for parents to correct their kid’s driving behavior base on big data.

Traccar already supports driving behaviour events.

I am closing this issue in favour of https://github.com/tananaev/traccar/issues/3501


Whitelist storage.googleapis.com

Hi! Please whitelist storage.googleapis.com. It's used for Google Cloud Storage. Thanks.

Hi natanlao,
Did it. Thanks for you advise.
Please, check yourself the lists and close the issue if you consider it solved.
Best regards
DB

Confirmed that it's no longer being blocked, thanks so much.


Update Dockerfile section of README

Signed-off-by: Alex Cameron<EMAIL_ADDRESS>Summary
Update the Dockerfile section of the README.
Release Note
NONE.

I am not following what this change is achieving, why is the extra FROM needed and why are we introducing another registry?

My understanding is that the second FROM is the base image for this Dockerfile. It's a minimal container that has just enough to run the Cosign binary.
As for why I've changed the registry, the infrastructure has been moved from distroless to chainguard-images. If you click through to the distroless GitHub org, there is some advice in the README that says:

Distroless images have moved to the Chainguard Images repo.

Please update any image references to use the cgr.dev/chainguard registry i.e. docker pull distroless.dev/static is now docker pull cgr.dev/chainguard/static.

It was at ghcr.io before though?

It was at ghcr.io before though?

Ah right. Perhaps the images are pushed to both registries? In any case, cgr.dev is the one listed in the README of the image repo.

My two cents is we should dog food here, build code on gh, push to ghcr, and recommend users pull from ghcr. I don't see a lot of value of introducing another layer of infra.

Ok, I got it, distroless has been shift and lifted over to chainguard. T.I.L.


Undo copy only for package lock in SPA templates

Now that the template issue has been fixed, we can undo the copy-only

Merging to get into the submodule update, will react to any feedback as needed