NetConf connection disconnect even though keepalive is sent from the server.

Requirement
To keep the NetConf connection alive for as long as we can and identify when this connection is lost and reconnect without having to send a Netconf message within the timeout period.
The problem:
Using the standard transport, I am trying to keep the connection active but the connection is disconnected after the ClientAliveInterval time has passed. I also receive a malformed message from the NetConf server just before the disconnection but I have not sent any NetConf message to the server during this time.
e.g. Malformed message from the device
<?xml version="1.0" encoding="UTF-8"?> <rpc-reply xmlns="urn:ietf:params:xml:ns:netconf:base:1.0"> <rpc-error> <error-type>rpc</error-type> <error-tag>malformed-message</error-tag> <error-severity>error</error-severity> <error-message>A message could not be handled because it failed to be parsed correctly.</error-message> </rpc-error> </rpc-reply>
Doing a tcpdump I can see the keep alive is sent from the server every 15 seconds which is also confusing as the settings on the server is 60 seconds. Is there anything in scrapligo that changes the ServerAliveInterval to 15 seconds?
e.g. code connecting to an Adtran SDX 6320 device.
func scrapliTestAdtran() {
	host := "<host>"
	username := "<username>"
	password := "<password>"

	d, err := netconf.NewDriver(
		host,
		options.WithAuthNoStrictKey(),
		options.WithAuthUsername(username),
		options.WithAuthPassword(password),
		options.WithPort(830),
		options.WithTransportType(transport.StandardTransport),
	)
	if err != nil {
		fmt.Printf("failed to create driver for %s; error: %+v\n", host, err)
		return
	}

	err = d.Open()
	if err != nil {
		fmt.Printf("Failed to open driver for %s; error: %+v\n", host, err)
		return
	}

	defer d.Close()

	for {
		filter := "<xpon><t-conts/></xpon>"
		resp, _ := d.Get(filter)

		fmt.Println(resp.Result)
		fmt.Println("***********************************************")
		time.Sleep(120 * time.Second)
	}
}

Thanks in advance.

Hey @dau71 👋
quickly checking the only timeout we set on standard transport is the "TimeoutSocket" (15s default) in the ssh client config here.
we do send serveraliveinterval but only in system transport -- so if anything is modifying that it would be in upstream crypto/ssh as far as I know.
as im not a standard transport user, if there are any tweaks we can make here to bring it more inline w/ feature parity of system transport while you're investingating this let us know as that would be great.

hey @dau71 I'm going to go ahead and close this out for now - but would be great if you discovered anything else if ya can update us. feel free to reopen if you've got some more info we can work with to improve things! thanks!


Automated js dependency update | Mon, 04 Nov 2019 12:03:02 +0000

closing out, since in process of switching over to dependabot for dependency updates.


Replace unsafeCoerce by undebruijn

brunosoliveira created an issue 2014-10-14
Currently we are using unsafeCoerce to get cheap&quick coercions between PHOAS expressions. However I think this is a mistake. We should be using an undebruijn function. The benefit would be better debugging/error-messages and potentially a solution to the weird substitution issue (by enforcing a dedebruijn after every traversal).

@hy-zhang confirmed no use of unsafeCoerce


Responsive issue

Responsive not working, If I enable responsive true, showing this issue
jQuery.Deferred exception: a.column.index is not a function TypeError: a.column.index is not a function

What version are you using and can you provide snippets to reproduce the issue?
Also, It seems like a js issue and a bit out of the scope of the project. Were you able to check in the datatables.net forum?


fix(git): make sure that git organization is use when creating the git

Why
In the new flow we can also specify which organisation we want to create the git repo
Description
This fixes an issue
Closes could not find the issue
Checklist

[x] I followed the contribution guidelines.
[] I created an issue and used it in the commit message.
[x] I have built the project locally prior to submission with mvn clean install.
[x] I kept a clean commit log (no unnecessary commits, no merge commits).

Ike Plugins (test-keeper)
Thank you @edewit for this contribution!

It appears that no tests have been added or updated in this PR.
Automated tests give us confidence in shipping reliable software. Please add some as part of this change.
If you are an admin or the reviewer of this PR and you are sure that no test is needed then you can use the command /ok-without-tests as a comment to make the status green.
For more information please head over to official documentation. You can find there how to configure the plugin - for example exclude certain file types so if PR contains only them it won't be checked.


Quasi-nested collections can be created but not accessed or deleted

FIrst of all, thank you for this project.
I'm just getting started with it (using pgstac backend), and was hoping to be able to set up a hierarchical collection structure according to some of the best practices I've seen, but it appears that stac-fastapi currently only supports a flat structure (i.e., does not support collections-within-collections).  Is this correct or am I just approaching it in the wrong way?
I attempted this with a POST to /collections and providing IDs like parentcollection and parentcollection/childcollection, which successfully created both collections, but seemingly at root catalog level and with no relationship to each other.
Additionally, while both are returned when listing all collections (GET /collections), the child collection is inaccessible from the collections/{collection_id} endpoint, so cannot be retrieved/updated/deleted.  I also tried with url encoding (GET /collections/parentcollection%2Fchildcollection), but that also doesn't seem to work.
If nested structures or certain characters (like /) are unsupported in identifiers, I would have expected the validation to fail during the initial POST.  If nested structures are intended to be supported, then it seems there is a problem with the FastAPI route configuration.
Example:
$ curl -X 'GET' \
  'http://localhost:10000/collections' \
  -H 'accept: application/json' | jq
{
  "collections": [
    {
      "id": "parentcollection",
      "type": "Collection",
      "links": [
        {
          "rel": "items",
          "type": "application/geo+json",
          "href": "http://localhost:10000/collections/parentcollection/items"
        },
        {
          "rel": "parent",
          "type": "application/json",
          "href": "http://localhost:10000/"
        },
        {
          "rel": "root",
          "type": "application/json",
          "href": "http://localhost:10000/"
        },
        {
          "rel": "self",
          "type": "application/json",
          "href": "http://localhost:10000/collections/parentcollection"
        }
      ],
      "title": "parentcollection",
      "assets": null,
      "extent": {
        "spatial": {
          "bbox": [
            [
              0,
              -90,
              360,
              90
            ]
          ]
        },
        "temporal": {
          "interval": [
            [
              null,
              null
            ]
          ]
        }
      },
      "license": "proprietary",
      "keywords": null,
      "providers": [
        {
          "url": "https://www.example.com",
          "name": "Test",
          "roles": [
            "host",
            "producer",
            "processor",
            "licensor"
          ],
          "description": null
        }
      ],
      "summaries": {},
      "description": "description",
      "stac_version": "1.0.0",
      "stac_extensions": null
    },
    {
      "id": "parentcollection/childcollection",
      "type": "Collection",
      "links": [
        {
          "rel": "items",
          "type": "application/geo+json",
          "href": "http://localhost:10000/collections/parentcollection/childcollection/items"
        },
        {
          "rel": "parent",
          "type": "application/json",
          "href": "http://localhost:10000/"
        },
        {
          "rel": "root",
          "type": "application/json",
          "href": "http://localhost:10000/"
        },
        {
          "rel": "self",
          "type": "application/json",
          "href": "http://localhost:10000/collections/parentcollection/childcollection"
        }
      ],
      "title": "parentcollection/childcollection",
      "assets": null,
      "extent": {
        "spatial": {
          "bbox": [
            [
              0.013488,
              -89.72341099999998,
              359.986566,
              89.72348699999999
            ]
          ]
        },
        "temporal": {
          "interval": [
            [
              "2022-01-08T00:00:00",
              null
            ]
          ]
        }
      },
      "license": "proprietary",
      "keywords": null,
      "providers": [
        {
          "url": "https://www.example.com",
          "name": "Test",
          "roles": [
            "host",
            "producer",
            "processor",
            "licensor"
          ],
          "description": null
        }
      ],
      "summaries": {},
      "description": "Child description",
      "stac_version": "1.0.0",
      "stac_extensions": null
    }
  ],
  "links": [
    {
      "rel": "root",
      "type": "application/json",
      "href": "http://localhost:10000/"
    },
    {
      "rel": "parent",
      "type": "application/json",
      "href": "http://localhost:10000/"
    },
    {
      "rel": "self",
      "type": "application/json",
      "href": "http://localhost:10000/collections"
    }
  ]
}

$ curl -X 'GET' \
  'http://localhost:10000/collections/parentcollection/childcollection' \
  -H 'accept: application/json'
{"detail":"Not Found"}

$ curl -X 'DELETE' \
  'http://localhost:10000/collections/parentcollection/childcollection' \
  -H 'accept: application/json'
{"detail":"Not Found"}

$ curl -X 'GET' \
  'http://localhost:10000/collections/parentcollection%2Fchildcollection' \
  -H 'accept: application/json'
{"detail":"Not Found"}

It looks like this affects not only collections, but also any items with a slash in their identifier.  The API is happy to create these records, but there doesn't seem to be a way to access them afterwards.
curl -X 'POST' \
  'http://localhost:10000/collections/testcollection/items' \
  -H 'accept: application/json' \
  -H 'Content-Type: application/json' \
  -d '{"type": "Feature", "geometry": {"type": "Polygon", "coordinates": [[[-4.244302848573078e-06, -89.72341099999998], [-4.244302848573078e-06, 89.72348699999999], [359.986566, 89.72348699999999], [359.986566, -89.72341099999998], [-4.244302848573078e-06, -89.72341099999998]]]}, "properties": {"datetime": "2023-04-20T12:00:00+00:00"}, "id": "item/with/slashes", "bbox": [-4.244302848573078e-06, 89.73697924796991,<PHONE_NUMBER>443028, -89.7369032479699], "stac_version": "1.0.0", "assets": {}, "links": [], "stac_extensions": [], "collection": "testcollection"}'
{"type":"Feature","stac_version":"1.0.0","stac_extensions":[],"id":"item/with/slashes","geometry":{"type":"Polygon","coordinates":[[[-4.244302848573078e-06,-89.72341099999998],[-4.244302848573078e-06,89.72348699999999],[359.986566,89.72348699999999],[359.986566,-89.72341099999998],[-4.244302848573078e-06,-89.72341099999998]]]},"bbox":[-4.244302848573078e-06,89.73697924796991,360.0000582443028,-89.7369032479699],"properties":{"datetime":"2023-04-20T12:00:00+00:00"},"links":[{"rel":"collection","type":"application/json","href":"http://localhost:10000/collections/testcollection"},{"rel":"parent","type":"application/json","href":"http://localhost:10000/collections/testcollection"},{"rel":"root","type":"application/json","href":"http://localhost:10000/"},{"rel":"self","type":"application/geo+json","href":"http://localhost:10000/collections/testcollection/items/item/with/slashes"}],"assets":{},"collection":"testcollection"}

$ curl -X 'GET' \
  'http://localhost:10000/collections/testcollection/items' \
  -H 'accept: application/geo+json'
{"type":"FeatureCollection","context":{"limit":10,"returned":1},"features":[{"id":"item/with/slashes","bbox":[-4.244302848573078e-6,89.73697924796991,360.0000582443028,-89.7369032479699],"type":"Feature","links":[{"rel":"collection","type":"application/json","href":"http://localhost:10000/collections/testcollection"},{"rel":"parent","type":"application/json","href":"http://localhost:10000/collections/testcollection"},{"rel":"root","type":"application/json","href":"http://localhost:10000/"},{"rel":"self","type":"application/geo+json","href":"http://localhost:10000/collections/testcollection/items/item/with/slashes"}],"assets":{},"geometry":{"type":"Polygon","coordinates":[[[-4.24430284857308e-6,-89.72341099999998],[-4.24430284857308e-6,89.72348699999999],[359.986566,89.72348699999999],[359.986566,-89.72341099999998],[-4.24430284857308e-6,-89.72341099999998]]]},"collection":"testcollection","properties":{"datetime":"2023-04-20T12:00:00+00:00"},"stac_version":"1.0.0","stac_extensions":[]}],"links":[{"rel":"collection","type":"application/json","href":"http://localhost:10000/collections/testcollection"},{"rel":"parent","type":"application/json","href":"http://localhost:10000/collections/testcollection"},{"rel":"root","type":"application/json","href":"http://localhost:10000/"},{"rel":"self","type":"application/geo+json","href":"http://localhost:10000/collections/testcollection/items"}]}

$ curl -X 'GET' \
  'http://localhost:10000/collections/testcollection/items/item/with/slashes' \
  -H 'accept: application/geo+json'
{"detail":"Not Found"}

$ curl -X 'GET' \
  'http://localhost:10000/collections/testcollection/items/item%2Fwith%2Fslashes' \
  -H 'accept: application/geo+json'
{"detail":"Not Found"}

If nested structures or certain characters (like /) are unsupported in identifiers, I would have expected the validation to fail during the initial POST.

I think that's a fair assumption. I'm going to move this issue to stac-fastapi-pgstac, and create a copy of it in stac-fastapi-sqlalchemy, as those checks need to be added at the backend leve.


Only re-rendering the first level of my nested call of components.

I'm not sure if it is a issue but lets go...
Only re-rendering the first level of my nested call of components.
  render() {
    return (
      <ThemeProvider theme={getTheme(this.colorDecision())}>
        <IntlProvider definition={this.localeDefinition()}>
          <HtmlStyle>
            <AppStyle>
              <Header />
              <Router onChange={this.handleRoute}>
                <Home path="/" />
                <Layout path="/layout/" />
              </Router>
            </AppStyle>
          </HtmlStyle>
        </IntlProvider>
      </ThemeProvider>
    );
  }

Above Header, Home, Layout re-render the text after IntlProvider update the definition.
But it doesn't re-render on the components called by Home for example:
  render() {
    return (
      <HomeStyle>
        <Elevation z={2}>
          <StatusBanner />
          <ControlBanner />
          <MotorTable />
        </Elevation>
      </HomeStyle>
    );
  }

StatusBanner, ControlBanner, MotorTable do not re-render the texts after trigger the button for translate.
Something wrong or It can't go deep on the calls?

Might need some more information to check on this one. Which version of preact-i18n do you have installed? Your issue may sound like https://github.com/synacor/preact-i18n/issues/39.
If you could fork this JSFiddle and create a reproduction it would really help understand the problem: https://jsfiddle.net/xyL04bzm/1/

Sure
"dependencies": {
"axios": "^0.19.2",
"local-storage-fallback": "^4.1.1",
"preact": "^8.4.2",
"preact-compat": "^2.3.1",
"preact-dom": "^1.0.1",
"preact-feather": "^4.1.0",
"preact-i18n": "^1.4.2",
"preact-material-components": "^1.6.1",
"preact-redux": "^1.2.0",
"preact-router": "^2.6.1",
"redux": "^3.6.0",
"redux-logger": "^3.0.6",
"styled-components": "^3.4.9"
}

Might need some more information to check on this one. Which version of preact-i18n do you have installed? Your issue may sound like #39.
If you could fork this JSFiddle and create a reproduction it would really help understand the problem: https://jsfiddle.net/xyL04bzm/1/
Sure
"dependencies": {
"axios": "^0.19.2",
"local-storage-fallback": "^4.1.1",
"preact": "^8.4.2",
"preact-compat": "^2.3.1",
"preact-dom": "^1.0.1",
"preact-feather": "^4.1.0",
"preact-i18n": "^1.4.2",
"preact-material-components": "^1.6.1",
"preact-redux": "^1.2.0",
"preact-router": "^2.6.1",
"redux": "^3.6.0",
"redux-logger": "^3.0.6",
"styled-components": "^3.4.9"
}

With<EMAIL_ADDRESS>and<EMAIL_ADDRESS>you're using the Legacy Context API; and with that your child components will not be guaranteed to re-render when context changes. See https://reactjs.org/docs/legacy-context.html#updating-context This looks like a limitation of Legacy Context, so I don't think anything can be done unless you use workarounds as mentioned in the blog post from the above docs.
This is one of the reasons the latest version 2.1.2-preactx uses the new createContext API. You can install that if you are able to upgrade your project to Preact 10, but that is a major version with breaking changes.
npm install preact@latest preact-i18n@preactx


Ugly refactor for x >>= pure . f when x is large

The original isn't great, but I think the suggested refactoring is a little worse. Would it be sensible to restrict this rule from firing when x contains newlines or would rely on the parse error rule?
I don't think I've noticed such a content sensitive rule in the lints before, so perhaps it's not in the spirit of the library.
runTravWithTravState state traversal =
    unTravT traversal state >>= pure . \case
        Left trav_err                                 -> Left [trav_err]
        Right (v, ts) | hadHardErrors (travErrors ts) -> Left (travErrors ts)
                      | otherwise                     -> Right (v,ts)

goes to
runTravWithTravState state traversal =
    (\case
        Left trav_err                                 -> Left [trav_err]
        Right (v, ts) | hadHardErrors (travErrors ts) -> Left (travErrors ts)
                      | otherwise                     -> Right (v,ts)) <$> unTravT traversal state

Another idea is to recommend (<&>) instead

Oh of course! That's my go-to fix for this I'd forgotten :D

Sounds like a great idea.  Someone want to send a patch?

done, did what seemed obvious in hlint.yaml, hopefully that's all that there is to it.


What does it mean to be handled by the scheduler differently?

These will result in the same output, but they are handled by the scheduler differently

well, one small thing is that the job names will have subscripts according to their $SLURM_ARRAY_TASK_ID value, so the output files will be slurm_123456789_0.out, slurm_123456789_1.out, etc. rather than slurm_123456789.out, slurm_123456790.out, slurm_123456791.out, etc.
Is that worth saying? It's possible that there are other, more substantive differences, but cluster management and details of slurm I think get pretty intricate and pretty machine-specific pretty fast, so I wouldn't want to say a whole lot more than that


Bug fixes in v1.4 release

Many thanks to contributing to nf-core/rnaseq!
To ensure that your build passes, please make sure your pull request is to the dev branch rather than to master. Thank you!
Please fill in the appropriate checklist below (delete whatever is not relevant). These are the most common things requested on pull requests (PRs).
PR checklist

[x] PR is to dev rather than master
[x] This comment contains a description of changes (with reason)
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If necessary, also make a PR on the nf-core/rnaseq branch on the nf-core/test-datasets repo
[ ] Ensure the test suite passes (nextflow run . -profile test,docker).
[ ] Make sure your code lints (nf-core lint .).
[ ] Documentation in docs is updated
[x] CHANGELOG.md is updated
[x] README.md is updated

Learn more about contributing: https://github.com/nf-core/rnaseq/tree/master/.github/CONTRIBUTING.md

Bump versions to 1.5dev
Fixed parameter warnings #316 and 318
Fixed #307 - Confusing Info Printout about GFF and GTF
Adjusted some but not all spacing. Will need to review this all again before the next release.

Some of the version changes you sent werent consistent @apeltzer . Ive bumped everything to 1.4.1 now. Can you just double-check everything please?

Some tests are failing? 😕

Ah, its because we bumped versions and tests cant find the appropriate container...

Why are we still using 0.32.0 though for tests? You bumped that no @apeltzer ?

Ah, GitHub actions files need to be updated too. Will need to add this into bump-versions:
https://github.com/nf-core/rnaseq/blob/55eee0c01c36db013d1dcbe6a4e65b407c358f0f/.github/workflows/ci.yml#L10

We bumped it everywhere but in the GitHub Actions tests ... (which are not auto-bumped atm).

Open an issue for that https://github.com/nf-core/tools/issues/429

Ok. Tests are still failing because GitHub cant find the docker container for 1.4.1 whats the easiest way to fix this?

Looking much better. Do your thing with the patch release when youre happy 👍


fix(athena): missing workgroup argument in unload

Feature or Bugfix

Bugfix

Detail

workgroup argument is not propagated in athena.read_sql_query method when unload_approach=True

Relates

#1178

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-pDO66x4b9gEu
Commit ID: a7aa9d2f162e86aaa85f3aa590328ef26455c097
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Updates to OWNERS files

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ymesika
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: greghanson
If they are not already assigned, you can assign the PR to them by writing /assign @greghanson in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS
pilot/OWNERS
tests/e2e/tests/pilot/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

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

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth.sh
494c1b6d53041deced5ffbcbc2547f96eab6fbc4
link
/test e2e-mixer-no_auth

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.

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

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth.sh
494c1b6d53041deced5ffbcbc2547f96eab6fbc4
link
/test e2e-mixer-no_auth

prow/istio-pilot-multicluster-e2e.sh
494c1b6d53041deced5ffbcbc2547f96eab6fbc4
link
/test istio-pilot-multicluster-e2e

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.

FYI @ijsnellf has asked me to hold off a bit on this to see if he can continue to contribute on his own time.

/hold


Implement modals

So modals... :smile:
Do we use aurelia-dialog from the start or implement "native" modals first?
I only had a quick look but it seems that it'd be quite easily adaptable to aurelia-dialog.

I would advise against "plain inclusion" of Aurelia Dialog into Materialize plugin - our project should be completely independent of "official aurelia" despite the rumors of making our bridges to "play along" with Aurelia Interface. If and when that happens, we will deal with inter-dependencies when they will be clearly defined

When you decide to use materialize, you accept the features and limitations of it. So I would choose to implement native modals first, and when there is a demand we should look into integrating aurelia-dialog

This is a bit more complicated then it looks..
There is an optional trigger button which can take some interesting arguments to configure the modal, f.i. callbacks to ready (modal is opened) or closed (modal is closed).
I'd prefer to use these as promises.
All of the above can be implemented using a "companion" attribute like with side-nav or even leaving the modal itself up to the user using pure html/css.
These options available for the trigger button can also be used in manual trigger function openModal() which - again - is not documented but present in materialize source.
https://github.com/Dogfalo/materialize/blob/master/js/leanModal.js#L172 shows that the button trigger activator (leanModal) only registers a click event which calls openModal() using its options.
So these options can be implemented both on the trigger and the modal itself.
This way, we could even construct our own trigger, being able to circumvent IDs completely (I don't like IDs, ever).
On the other hand, providing a simple custom attribute which activates a modal already present, would be much simpler for the end-user.
The best would probably be to provide both ways. And there it becomes complicated. :smile:
We'd probably have to merge activator/modal options.


[WIP] flake8 + pimp example figures

@rflamary @ncourty any opinion on this? shall I continue?
big questions are

ok to adopt pep8?
do you agree to use py.test for more standard testing mechanism?

don't merge yet. Just tell me what you think.

Hello @agramfort ,

I agree with pep8 adoption even though some of the rules are dated (80 characters is tough when you have classes). In fact I was working with autopep8 to clean this with no headaches but I think you are more competent than me here.

py.test  is also better and we will add some proper tests at least for the main functions quickly. Clearly those with GPU cannot build and run on travis so they should be optional (i dont kow how to do that with pytest other than removing the files).

Also I know I use pl instead of plt but it felt so much better at the time (still feels better trust me). Still i understand the need to make the examples more friendly to newcomers.
@ncourty  how so you feel about all that?

Hello @agramfort and @rflamary,

Clearly we should go for a strict application of pep8 recommendations, even if I do agree with @rflamary that some of them are outdated (I've read several passionate discussions on the 80 chars here and there). Being compliant with pep8 standards, even if too restrictive, is a big plus IMO

Regarding pytest, I also do agree that it is one of the best option available for unitary tests and continuous integration, so I think it's also good practice. We need to level up on how to use it though, but it does not seem too complicated.

Regarding pl. vs plt., I have seen both versions on many codes. @agramfort, is there a standard here (maybe part of pep8 ?)

hey,
so I made progress. I pushed full flake8 valid code.
I'll need you to refactor the tests to they are functions that start with
test_
then we can call SkipTest if cudamat is not installed.
Some notes from my reading:

why not using cg from scipy.optimize?
sklearn calls make_ functions that make simulated data. So I propose to
rename get_ -> make_ for datasets
add references to papers in examples (it's not always there)
don't put params in fit in the estimators. Put parameters in the init
don't mix CamelCase and snake_case eg OTDA_mapping_kernel ->
OTDAMappingKernel etc.
why gpuarray and not cupy which seem more mature? It should also allow
you to have the same code for CPU and GPU code.

feedback welcome.
A

Hello,
Thank you for the work, it looks great I just have small remarks:

full flake8 valid is awesome !
you did no change all pl to plt, probably because we did no concrge with @ncourty. Sp we need to do it quickly I suppose.

Regarding your remarks:

CG here stands for conditional gradient and not conjugate gradient as implemented in scipy.
OK for switch to make but all examples and notebooks have to be updated also.
Example do not have a proper description yet and I agree we need to complete it with
I always thought this choice (params in init) in sklearn was weird since it make it more complicated  to perform warm start with a slightly different regularization (in a reg path for instance) value which is common practice in ML. I'm sure you have good reasons for it though. my problem here is that it breaks the code for people who already use the toolbox and I'd like to keep the young and still small community.. Maybe do both for a while with a Deprecated Warning?
I agree we should stick to Camel or snake but i personally prefer snake and most of the toolbox is in snake, do you suggest the change for Classes only?, is it a common thing to do ?
I agree that the choice for cudamat is probably not the best. We were planing with Nico to switch to a more future proof code with pycuda.

I'll work on the test next week when I don't have NIPS review to write.

full flake8 valid is awesome !

:)

you did no change all pl to plt, probably because we did no concrge with
@ncourty. Sp we need to do it quickly I suppose.

yes I used pl so you're still in your confort zone :)

Regarding your remarks:
CG here stands for conditional gradient and not conjugate gradient as
implemented in scipy.

arfff good point. What it tells you is that cg is probably not explicit
enough. Maybe conditional_gradient is not that long to type :)

OK for switch to make but all examples and notebooks have to be updated
also.

arfff. But I thought you were using sphinx-gallery. WIth sphinx-gallery you
should not ship notebooks. They are produced when you build the doc.

Example do not have a proper description yet and I agree we need to
complete it with
I always thought this choice (params in init) in sklearn was weird since
it make it more complicated to perform warm start with a slightly different
regularization (in a reg path for instance) value which is common practice
in ML.

for this we use the warm_start parameter
clf = Classifier(..., warm_start=True)
clf.fit(X, y)
clf.set_param(C=C_new)  # or just clf.C = C_new
clf.fit(X, y)

I'm sure you have good reasons for it though. my problem here is that it
breaks the code for people who already use the toolbox and I'd like to keep
the young and still small community.. Maybe do both for a while with a
Deprecated Warning?

yes you need deprecation cycles. But the earlier you do it the better. If
it's too painful just deprecate a full class and create a new one with a
better name.

I agree we should stick to Camel or snake but i personally prefer snake
and most of the toolbox is in snake, do you suggest the change for Classes
only?, is it a common thing to do ?

it's not a matter of preference but conventions :)
yes you should use CamelCase for classes and snake_case for functions and
methods.

I agree that the choice for cudamat is probably not the best. We were
planing with Nico to switch to a more future proof code with pycuda.

pycuda is old and not that maintained anymore. cupy is much more used these
days and more actively maintained.

I'll work on the test next week when I don't have NIPS review to write.

:)

Hello Alexandre,
We met with Nico and decided on a few things discussed in the following
2017-07-13 14:44 GMT+02:00 Alexandre Gramfort<EMAIL_ADDRESS>

you did no change all pl to plt, probably because we did no concrge with
@ncourty. Sp we need to do it quickly I suppose.

yes I used pl so you're still in your confort zone :)

OK so we decided to stick to pl since we import pylab and not pyplot and
all
the modules in pot are two letters.

Regarding your remarks:
CG here stands for conditional gradient and not conjugate gradient as
implemented in scipy.

arfff good point. What it tells you is that cg is probably not explicit
enough. Maybe conditional_gradient is not that long to type :)

ok for renaming the function and also for generalized cg.  temporary we
will keep cg=conditional_gradient in order to keep the compatibility and
will put deprecation warning in the future.

OK for switch to make but all examples and notebooks have to be updated
also.

arfff. But I thought you were using sphinx-gallery. WIth sphinx-gallery you
should not ship notebooks. They are produced when you build the doc.

OK but until recently I did not know how to do proper titles and text in
the python code so that you have a notebook with cells instead of one giant
cell as in most sklearn examples. This will take some time but we will
switch to python only examples as soon as we transcript the text and titles
from the notebooks.
Params in init:
OK for renaming  the classes in CamelCase aznd moving the parameters to the
init phase. We will also add a set_param function in order to be more
compatible with sklearn

I agree that the choice for cudamat is probably not the best. We were
planing with Nico to switch to a more future proof code with pycuda.

pycuda is old and not that maintained anymore. cupy is much more used these
days and more actively maintained.

yeah ok for cupy we will look into that

I'll work on the test next week when I don't have NIPS review to write.

:)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rflamary/POT/pull/13#issuecomment-315066486, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABUpeSNmwe7q4cekiHvJQjzcTgxZGq-fks5sNhEQgaJpZM4OUu6B
.

ok fine with me. So how do you want to proceed with this PR?
I rebased and revert the plt
did you look at the diff?

Yeah we are OK with the merge but just wait for travis to merge.
Seems from early results that some PEP8 error remain.
Thank you again

all green !


List of search results

Currently, search results are displayed in cards. It's easy to see, but it's difficult to compare when there are many search results, so I want search results to be visible in a list.

Awesome suggestion! I'll add this to the backlog.


Add ARPAsing Plus Phonemizer

Custom English Arpabet Phonemizer based on Syllable-Based API Phonemizer with fallback support

Mechanics of this Phonemizer:

Starting C: [- c]
Starting CV: [- c v] or [- cv]
Starting CCV: [- ccv] or [- cc v]
Starting V: [- v] or [v]
VV: (Fallbacks to [v c][c v]/[cv] then [c v]/[cv] then [v], if vb doesn't have [v], glottal stop are used instead)
Connecting CV: [c v] (Fallbacks to [cv]if no alias is detected)
Connecting CCV: [ccv]  or [cc v]
Connecting VC: [v c] then [v -] [- c
Connecting CC: [c c] (with consonant fallbacks) then [c1 -] [- c2]
Connecting CCC: [c cc] (fallbacks to [c c])
Ending C: [c -]
Ending V: [v -]

Phoneme length are specified directly to the phonemizer:

Default transition in ms: '1.0'
Vowels: 'default'
Consonants: '1.3'
Affricates: '1.5'
Long Consonants: '2.3'
Semi-long Consonants: '1.3'
Tap Consonant: '0.5'
Glide Consonants: '2.5'

Vowel and Consonant Fallbacks

This custom Phonemizer supports vowel and consonant fallbacks:

CV/CCV Fallback: [cc v] or [ccv] and [c v] or [cv]
VV Fallback: `([v c][c v]/[cv] then [c v]/[cv] then [v], if vb doesn't have [v], glottal stop are used instead)``
Connecting VC/VCC Fallback: [v c] then [v -] [- c]
Connecting CC/CCC: [c c] (with consonant fallbacks) then [c1 -] [- c2]
Starting and Ending Consonants

📍For more information, visit https://github.com/Cadlaxa/Syllable-Based-ARPAsing-Phonemizer

The test you added seem to fail?


Classif  **kwargs

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (fd80f62) 95.15% compared to head (8368261) 95.20%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #762      +/-   ##
==========================================
+ Coverage   95.15%   95.20%   +0.05%
==========================================
  Files         126      126
  Lines        4908     4905       -3
==========================================
  Hits         4670     4670
+ Misses        238      235       -3

Flag
Coverage Δ

unittests
95.20% <100.00%> (+0.05%)
:arrow_up:

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

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


Unable to Query OCI the status show retrieving.....

Hi,
I think OCI CLI configuration file works fine as below

but for OKIT, I run docker with cmd

and then try to query OCI it does not work. it shows retrieving forever.

@pinyoud can you try the 0.54.3 release

the issue is closed. I tried to build v0.54.4 in OLE7.9(the issue occurs in OLE8 env, don't know why), and now it works fine. Thanks for your response.


Enable ZOOM in IE10+

Added support of Pointer events in addition to Touch events

Fixes #1439. Related #2403.

Hi, I have just downloaded this version and the pinch zoom/panning is not working for my d3 SVG. Do I need to implement something different to add support for IE11?

Hi, nikolas, I've made rebase and double-checked my patch suggested here. It works in IE11 (particularly in 11.0.25). With this version you can do zoom with two/three/more fingers (there is an algorithm that considers third finger if second one goes out of zoom area). Thank you for downloading and reminding me about this pull request.

Many thanks for the heads up, I will download this version and test it tomorrow.
From: lubomirov<EMAIL_ADDRESS>Sent: Friday, 15 April 2016 19:22
To: mbostock/d3<EMAIL_ADDRESS>Cc: nikoloas<EMAIL_ADDRESS>Subject: Re: [mbostock/d3] Pointer (and MSPointer) events for the zoom behavior. (#2547)
Hi, nikolas, I've made rebase and double-checked my patch suggested here. It works in IE11 (particularly in 11.0.25). With this version you can do zoom with two/three/more fingers (there is an algorithm that considers third finger if second one goes out of zoom area). Thank you for downloading and reminding me about this pull request.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub https://github.com/mbostock/d3/pull/2547#issuecomment-210378102 https://github.com/notifications/beacon/ARbkCtzm7yZvU7LLBc1S-zK6xn4vCR9Nks5p31jHgaJpZM4F48YH.gif

Hi, I'm using d3 to draw and manipulate a map and it works fine, excepted on windows mobile devices (pinch zoom and translations by touch are disabled). Maybe those changes could fix it ? Can it be merged with main project ?
Many thanks for your work.
Leo.


Paired pulse notes

7-28-22
DIC images

02-01-01 (Interpulse interval: 100 ms)
The images in these slides are vertically flipped to match DIC images

02-01-02 (Interpulse interval: 100 ms)
Images here are not flipped, appear as they do in PhotoZ

This seemed more extreme than PP depression, but rather more like a strong inhibitory effect?
02-01-08 (Interpulse interval: 50 ms)

02-01-09 (Interpulse interval: 50 ms)
Similar to 02-01-08, 02-01-10, 02-01-11 (all of same interpulse interval and similar response)
At 0 ms (1st stim, 200 uA, 200 us):

At 50 ms (2nd stim, 200 uA, 200 us):

At 100 ms (no stim):

8-1-22
Reviewed Transnetyx. Oddly, the 7/28/22 mouse is genotyped as a Cre non-carrier (719669, May 22, M, -+/-)?
Recording from a Cre carrier today for comparison, may need to troubleshoot the line if non-carriers are expressing Cre? (718228, May 22 F, -+/+).


Update Security Vulnerabilities

GitHub reported security vulnerabilities in yarn.lock, some of which cannot be fixed through the automated dependabot. Could you make these updates?

this project has no runtime dependencies so these "security vulnerabilities" are pointless, they are only used at build time.


[Workspace Feeds]: Update isLoading both for expensify and company cards

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/52909
PROPOSAL: N/A
Tests

Open any workspace
Enable Company Cards
Navigate to Company Cards and see that no loading spinner is there

[x] Verify that no errors appear in the JS console

Offline tests

Same as tests
QA Steps

None

[X] Verify that no errors appear in the JS console

PR Author Checklist

[X] I linked the correct issue in the ### Fixed Issues section above
[X] I wrote clear testing steps that cover the changes made in this PR

[X] I added steps for local testing in the Tests section
[X] I added steps for the expected offline behavior in the Offline steps section
[X] I added steps for Staging and/or Production testing in the QA steps section
[X] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[X] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[X] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[X] I included screenshots or videos for tests on all platforms
[X] I ran the tests on all platforms & verified they passed on:

[X] Android / native
[X] Android / Chrome
[X] iOS / native
[x] iOS / Safari
[X] MacOS / Chrome / Safari
[X] MacOS / Desktop

[X] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[X] I followed proper code patterns (see Reviewing the code)

[X] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[X] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[X] I verified that comments were added to code that is not self explanatory
[X] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[X] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[X] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[X] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[X] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[X] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[X] I verified the JSDocs style guidelines (in STYLE.md) were followed

[X] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[X] I followed the guidelines as stated in the Review Guidelines
[X] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[X] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[X] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[X] I verified that if a function's arguments changed that all usages have also been updated correctly
[X] If a new component is created I verified that:

[X] A similar component doesn't exist in the codebase
[X] All props are defined accurately and each prop has a /** comment above it */
[X] The file is named correctly
[X] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[X] The only data being stored in the state is data necessary for rendering and nothing else
[X] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[X] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[X] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[X] All JSX used for rendering exists in the render method
[X] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[X] If any new file was added I verified that:

[X] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[X] If a new CSS style is added I verified that:

[X] A similar style doesn't already exist
[X] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[X] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[X] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[X] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[X] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[X] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[X] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

https://github.com/user-attachments/assets/4a7f7947-0343-40ce-948c-14ecb892d625

@mountiny @allgandalf
okay, I see the point here, but the issue still the same
we are waiting for an initial loading on Expensify page because cardsList is empty and when we get it back it returns empty Object {} and after cardSettings.isLoading && !cardsList will be always false as even empty cardsList returns {}
Meanwhile in the CompanyCards we are just using cardFeeds.isLoading and this one will return true each time we are fetching data (each time we are navigating here)

Discussion

@Expensify/design Can you please help us with how to show the loader on the company / expensify cards?
I think we have three cases:

there is no cardSettings and cardList locally - this can mean you either:

do not have any on the policy, - loader - empty state
or we are opening the page for a first time - loader - show the card list

you have some cardSetting and cardList locally - no loader - show the list even if fetching data again.

This way the problem is that if you do not have cards setup, we would be showing the loader everytime you go to the empty feature page. But at least the spinner will not show up when you already have some cards assigned.
That feels like the best outcome in my opinion

@mountiny what you have here makes total sense to me 👍

@waterim I think when the feed is set up, then it should return an empty object but not if there is no feed.
@waterim is the current state based on how I described from functional perspective? if yes then the PR is ready to test @allgandalf

Updating to make it as you described, right now no loading will be

Updated behaviour @mountiny @allgandalf
https://github.com/user-attachments/assets/a96d88c4-9904-4145-9c96-6d73b3880aec

I think this is expected because we have no card settings and no card list, so we cannot know if it's because there are actually no card feeds or if it's loading - and if it's actually just loading, then showing the section would be awkward.
So I think this is fine. For normal user it should not be a problem as they wont have this page just enabled without a feed and keep going there so I am fine with this

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

Some questions about this one. Asked here.

Any of the issues here should not block deploy, so checking this off.


Generator return type always widened

🔎 Search Terms
generator literal widen
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about (I am the faq author)

⏯ Playground Link
https://www.typescriptlang.org/play?target=99&jsx=0#code/CYUwxgNghgTiAEYD2A7AzgF3gQQFzwCJsCBuAKAHoL4AlARnwHEQUQYoMkYAeTGASxQBzADTwAbkn7AxAVxSgAZoJDAAfJWoBRAB4AHcBlVMWbDl25ECYydLkKQy1urLJ0WevAC88ABSL5MAx+VAAqXwBKeABvMnh4+ABPfhAIYFCccgBfCMjyKloAJhNWdk4ePkFRCSkZeHklFRcC3QMg43hmUvMeKxta+0bnDTdMIu8-AJQgkJRwqNiEpJS0jKt4KDREVExs3Ij87RgYLk14VsMOgDkkeDYTmFcdjwBmErNyy2J+u3qHJ1Uagm-kCwTCkRicQSyVS6UIxA2W1GGD2eTOWmOpxa+kuwHwNzumMeyNoABZ3mULH0ar8Go4mkCfCDpmC5hDFtCVnDsKiDkA
💻 Code
declare const A: "A";
// R1: Generator<string, void, undefined>
// Expected: Generator<"A", void, undefined>
const R1 = (function*() {
    yield* A;
})();
// R2: Generator<string, void, undefined>
// Expected: Generator<"A", void, undefined>
const R2 = (function*() {
    yield* "A" as const;
})();
// Error
// Expected: No error
const R3: Generator<"A", void, undefined> = (function*() {
    yield* "A" as const;
})();
// Error
// Expected: No error
const R4: Generator<"A", void, undefined> = (function*() {
    yield* A;
})();

🙁 Actual behavior
In all cases, the literal return is widened. This also extends to places like
// R5: Generator<string, not Generator<"A" | "B"
const R5 = (function*() {
    yield* "A" as const;
    yield* "B" as const;
})();

where our normal "union of literals" logic should have happened
🙂 Expected behavior
Follow the same widening behavior as normal function returns
Additional information about the issue
Discussed at #23537, but with a contradictory outcome

Iterating "A" produces strings 🤦

Iterating "A" produces strings 🤦

string is such a weird animal in this regard.  It's the only type (AFAIK) that's idempotent under iteration.  A naive algorithm for recursive iteration could potentially loop forever.


Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: Package subpath './llms' is not defined by "exports"

Hi, sorry if this has already been fixed, but I couldn't find anything in the issues or discussions. I'm trying to run a basic example using the instructions from https://www.npmjs.com/package/llamaindex. However, I get an error when
running the example.ts file.
$ node --import tsx example.ts

node:internal/modules/esm/resolve:299
  return new ERR_PACKAGE_PATH_NOT_EXPORTED(
         ^

Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: Package subpath './llms' is not defined by "exports" in<EMAIL_ADDRESS>    at exportsNotFound (node:internal/modules/esm/resolve:299:10)
    at packageExportsResolve (node:internal/modules/esm/resolve:646:9)
    at resolveExports (node:internal/modules/cjs/loader:628:36)
    at Module._findPath (node:internal/modules/cjs/loader:718:31)
    at Module._resolveFilename (node:internal/modules/cjs/loader:1205:27)
    at m._resolveFilename (file:///Users/--/node_modules/tsx/dist/register-Cehy_bZS.mjs:1:789)
    at resolve (/Users/--/node_modules/tsx/dist/register-CCR7NebB.cjs:2:554)
    at resolveRequest (/Users/--/node_modules/tsx/dist/register-CCR7NebB.cjs:1:2758)
    at Function._resolveFilename (/Users/--/node_modules/tsx/dist/register-CCR7NebB.cjs:2:583)
    at Module._load (node:internal/modules/cjs/loader:1045:27) {
  code: 'ERR_PACKAGE_PATH_NOT_EXPORTED'
}

Node.js v22.4.0

I am running this on an intel mac
sw_vers
ProductName:		macOS
ProductVersion:		14.5
BuildVersion:		23F79

Here is my example .ts

import fs from "node:fs/promises";
import { Document, VectorStoreIndex } from "llamaindex";

async function main() {
  // Load essay from abramov.txt in Node
  const essay = await fs.readFile(
    "node_modules/llamaindex/examples/abramov.txt",
    "utf-8",
  );

  // Create Document object with essay
  const document = new Document({ text: essay });

  // Split text and create embeddings. Store them in a VectorStoreIndex
  const index = await VectorStoreIndex.fromDocuments([document]);

  // Query the index
  const queryEngine = index.asQueryEngine();
  const response = await queryEngine.query({
    query: "What did the author do in college?",
  });

  // Output response
  console.log(response.toString());
}

main();

My tsconfig.json
{
  "compilerOptions": {
    "target": "ES2022",
    "module": "esnext",
    "moduleResolution": "bundler",
    "esModuleInterop": true,
    "forceConsistentCasingInFileNames": true,
    "strict": true,
    "skipLibCheck": true,
    "lib": [
      "ES2022"
    ],
    "types": [
      "node"
    ],
    "outDir": "./lib",
    "tsBuildInfoFile": "./lib/.tsbuildinfo",
    "incremental": true,
    "composite": true
  },
  "ts-node": {
    "files": true,
    "compilerOptions": {
      "module": "commonjs"
    }
  },
  "include": [
    "./**/*.ts"
  ]
}

And package.json
{
  "name": "sample",
  "version": "1.0.0",
  "main": "index.js",
  "keywords": [],
  "author": "",
  "license": "ISC",
  "description": "",
  "dependencies": {
    "llamaindex": "^0.4.11"
  },
  "devDependencies": {
    "@types/node": "^20.14.9",
    "tsx": "^4.16.2",
    "typescript": "^5.5.3"
  }
}

same issue with https://github.com/run-llama/LlamaIndexTS/issues/1012

Please check 0.4.12


Set Minio domain to allow dns buckets

Closes #264.
However, it would probably be preferable for the domain default to be null and then to not include MINIO_DOMAIN in the run template at all, it wasn't immediately clear to me how that would best be accomplished, so I instead opted for a sensible default of localhost.

@willvincent Hm, I think we have another one somewhere with a conditional flag? Let me try to hunt it down

https://github.com/tighten/takeout/blob/main/app/Services/Sftp.php#L66
You can see a conditional modification of the dockerRunTemplate here, in Sftp.

Ahh sweet, I'll update accordingly, thanks

Thanks @willvincent!

My pleasure, sir.


have no idea

Description
What steps will reproduce the problem?

What is the expected output? What do you see instead?
Please provide any additional information below
File "/Users/LZQ/anaconda/lib/python3.6/site-packages/spyder/widgets/editor.py", line 1236, in __setup_menu
add_actions(self.menu, list(actions)+self.__get_split_actions())
File "/Users/LZQ/anaconda/lib/python3.6/site-packages/spyder/utils/qthelpers.py", line 309, in add_actions
action = action.no_icon_action
File "/Users/LZQ/anaconda/lib/python3.6/site-packages/spyder/utils/qthelpers.py", line 443, in getattribute
attr = super(SpyderAction, self).getattribute(name)
File "/Users/LZQ/anaconda/lib/python3.6/site-packages/spyder/utils/qthelpers.py", line 462, in no_icon_action
return self._action_no_icon
File "/Users/LZQ/anaconda/lib/python3.6/site-packages/spyder/utils/qthelpers.py", line 445, in getattribute
if hasattr(attr, 'call') and name not in ['triggered', 'toggled',
RuntimeError: wrapped C/C++ object of type QAction has been deleted
Version and main components

Spyder Version: 3.2.0
Python Version: 3.6.0
Qt Versions: 5.6.2, PyQt5 5.6 on Darwin

Dependencies
pyflakes >=0.6.0 :  1.5.0 (OK)
pycodestyle >=2.3:  2.3.1 (OK)
pygments >=2.0   :  2.1.3 (OK)
pandas >=0.13.1  :  0.19.2 (OK)
numpy >=1.7      :  1.11.3 (OK)
sphinx >=0.6.6   :  1.5.1 (OK)
rope >=0.9.4     :  0.9.4-1 (OK)
jedi >=0.9.0     :  0.9.0 (OK)
psutil >=0.3     :  5.0.1 (OK)
nbconvert >=4.0  :  4.2.0 (OK)
sympy >=0.7.3    :  1.0 (OK)
cython >=0.21    :  0.25.2 (OK)
qtconsole >=4.2.0:  4.3.0 (OK)
IPython >=4.0    :  5.1.0 (OK)
pylint >=0.25    :  1.6.4 (OK)

This is a duplicate of #4843. A patch is coming on #4869, thanks for your report


whitelist prestascan

hi
can you provide an ip list (or other solution) to whitelist Prestascan behind an htaccess password ?
thanks

Hello,
Our requests are issued from the IP address:
<IP_ADDRESS>
If you whitelist this IP and when our server makes a call for a scan, it does not encounter BasicAuth authentication, then server to server communication will be fine.
Does this answer your question?
Best regards

thanks @Vincent-Profileo


FixItemName and FixChoiceTypes are prefix-named

All other datafixes use Fix suffixes.

Don't these names come from Mojang directly, though?

Can't check right now, but IIRC these are some of the datafixes that don't have names in their classes.


feat: add lock config to k8s manifest

Description

This feature adds FE only support of lock configuration for k8s resource manifest.
Fixes # (issue)
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Checklist:

[ ] The title of the PR states what changed and the related issues number (used for the release note).
[ ] Does this PR require documentation updates?
[ ] I've updated documentation as required by this PR.
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas

Merged into kubecon-2024


[Bug] UI-python testcases with the --tests arguments are failed to show results

Describe the bug
The Python testcases which uses the --tests arguments gets hanged in the UI and fails to populate the result but we could see the test case is being executed properly through the DUT log flow. The runner doest ends and shows the result even the whole test case gets completed.
For eg: TC-SM-1.1 testcase is started its execution. but even after completion of the execution still it is in execution state. i.e instead of updating the result the script is waiting for the results, this cause the UI to be in the execution state
Steps to reproduce the behavior
Step-1. Update the config and pics file of the respective project for testcase which needs to be executed accordingly.
Step-2. Open the project and Run the testcase by selecting the testcase in the add test.
Expected behavior
UI should be updated with the result of the python testcase
Log files
DUt log:  loading.txt
Screenshots

Environment
TH version : Version: 2.10-beta3+spring2024 Sha: 1fd1429
Additional Information
The list of testcase affected by this issue:

[ ] TC-ACE-2.1
[ ] TC-ACE-2.2
[ ] TC-DESC-2.2
[ ] TC-DT-1.1
[ ] TC-IDM-10.1
[ ] TC-IDM-10.2
[ ] TC-IDM-10.3
[ ] TC-IDM-10.4
[ ] TC-IDM-11.1
[ ] TC-PS-3.1
[ ] TC-SM-1.1
[ ] TC-SM-1.2

This issue seems to be related to: https://github.com/project-chip/certification-tool/issues/163

Hi @Saravana-kr22, there is an update in the release branch. Could you check if the changes works for you?
Could you please follow the steps bellow in order to update the release branch and test again?
./scripts/stop.sh
cd certification-tool
git fetch
git checkout v2.10-beta3+spring2024
git submodule update --init --recursive
./scripts/start.sh

This issue is resolve in latest Th image

This issue is resolved in latest TH image.


AppGW v2 CLI cannot save redirection config and url mapping to request routing rule as expected

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Describe the bug
When trying to configure following setting through CLI, redirection config was not saved as expected
Expected behavior
All traffic but /vm01/ is redirected to another listener
Traffic to /vm01/ follows URL mapping and forwarded to Pool01-VM01

Expected result from portal

Actual result from portal, after change is done in CLI

To Reproduce
Following 2 patterns of CLI can repro this issue
Pattern 1 create url mapping, redirect config and rule in order
az network application-gateway url-path-map create --resource-group bootcampdb --gateway-name AppGW03-v02 --name urlpathmap01 --address-pool Pool01-VM01 --http-settings HttpSetting01 --rule-name test --paths "/vm01/"
az network application-gateway redirect-config create --resource-group bootcampdb --gateway-name AppGW03-v02 --name redirectconfig01 --include-path true --include-query-string true --type Permanent --target-listener Listener02-https443
az network application-gateway rule create --resource-group bootcampdb --gateway-name AppGW03-v02 --name rule99 --http-listener Listener04 --rule-type PathBasedRouting --url-path-map urlpathmap01 --redirect-config redirectconfig01 --address-pool Pool01-VM01
Pattern 2, create rule first then update redirect config
az network application-gateway url-path-map create --resource-group bootcampdb --gateway-name AppGW03-v02 --name urlpathmap01 --address-pool Pool01-VM01 --http-settings HttpSetting01 --rule-name rule01 --paths "/vm01/"
az network application-gateway rule create --resource-group bootcampdb --gateway-name AppGW03-v02 --name rule99 --http-listener Listener04 --rule-type PathBasedRouting --url-path-map urlpathmap01 --address-pool Pool02-WebApp --http-settings HttpSetting01
az network application-gateway redirect-config create --resource-group bootcampdb --gateway-name AppGW03-v02 --name redirectconfig01 --include-path true --include-query-string true --type Permanent --target-listener Listener02-https443
az network application-gateway rule update --resource-group bootcampdb --gateway-name AppGW03-v02 --name rule99 --redirect-config redirectconfig01

As per CLI show command output, config seems to be added successfully,

ARM template

Redirection did not work properly as well, no redirection took place.
Expected behavior
If I made the change on portal, no redirectionConfiguration is set in rule, but in url map instead

Redirection worked unexpectedly as I described at first
Following PS command can make the change successfully as well
$listener = Get-AzApplicationGatewayHttpListener -Name "<>" -ApplicationGateway $appgw
$RedirectConfig = Add-AzApplicationGatewayRedirectConfiguration -Name "<>" -RedirectType Permanent -TargetListener $listener -ApplicationGateway $appgw

$HttpSettings = Get-AzApplicationGatewayBackendHttpSettings -Name "<>" -ApplicationGateway $appgw
$AddressPool = Get-AzApplicationGatewayBackendAddressPool -Name "<>" -ApplicationGateway $appgw
$pathrule = New-AzApplicationGatewayPathRuleConfig -Name "<>" -Path "/vm/" -BackendAddressPool $AddressPool -BackendHttpSettings $HttpSettings
$RedirectConfig1 = Get-AzApplicationGatewayRedirectConfiguration -Name "<>" -ApplicationGateway $appgw
$UrlPathMap = add-AzApplicationGatewayUrlPathMapConfig -Name "<>" -PathRules $pathrule -DefaultRedirectConfigurationId $RedirectConfig1.id -ApplicationGateway $appgw

$listener1 = Get-AzApplicationGatewayHttpListener -Name "<>" -ApplicationGateway $appgw
$UrlPathMap1 = get-AzApplicationGatewayUrlPathMapConfig -Name "<>" -ApplicationGateway $appgw
Add-AzApplicationGatewayRequestRoutingRule -ApplicationGateway $appgw -Name "<>" -RuleType PathBasedRouting -UrlPathMap $UrlPathMap1 -HttpListener $listener1

Set-AzApplicationGateway -ApplicationGateway $appgw

Environment summary
Any AppGW v2 can repro this issue
Additional context

network

@bangdu-ms, I can't reproduce in the latest version of CLI. Could use the latest version of cli and try again? Thanks.


Questions in converting cityscapes dataset

Thanks for your excellent work and kindly code releasing. I have a problem when reproducing the cityscapes experiment.
Following your tutorial, I have reproduced the experimental results, 44.37 vs. 51.73 with 6.79 improvement under the cutmix setting. When delving deep into the codes, I notice that you implement a downsample_label_img function for downsampling ground truth in the convert_cityscapes.py.  However, it is more common to use nearest downsampling method in cv2 or PIL. Then, I replace the downsample_label_img function with the following:
y_img = cv2.resize(y_img, (1024, 512), interpolation=cv2.INTER_NEAREST)
and re-conduct the cityscapes experiment. The results degrades greatly, 43.79 vs. 47.02 with only 3.23 improvement.
I wonder the reason why a different downsample method affects the result greatly and the motivation to reimplement a downsample method rather than using a common one.

Hi, thank you for exploring this. I have to admit I didn't give it much thought. When downsampling an RGB image you tend to get better results - at least visually - by using a local mean approach, so I tried to replicate this for labels as best I could.
Out of curiosity, do your results arise from a single run or from the average of multiple runs?

Thanks for your quick reply and resolve my concern. The results arise from a single run.

Sure thing.
You may find that taking the average of a few runs will close the gap between the results from various downsampling methods.


Logo

The most important point of course: A " hybro / hi bro!" logo.

A few drafted ideas:
The chat:

The moustache (may cause diversity issues):

The awfully colourful one:

The simple one:

Why a moustache?

Belayed Movember joke ("Save a (hy)bro, grow a mo"). I don't like the logo anymore and its just a few days after the end of November...


Updated logic to check every port in the range, and move to the next set

Changed logic to check all the ports in between current port and next port, calculated by BASE_PORT_RANGE.
So if BASE_PORT_START is 12000 and BASE_PORT_RANGE is 3, it will check ports 12000, 12001, and 12002. If any of those ports are not available, it will check 12003 through 12005, and so on.

 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.cahrens seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Port parser filter from fluent-plugin-parser. fix #1189

Changes from original code:

Use v0.14 namespace
Use v0.14 API (Now working)
Remove warnings like unused variable

@tagomoris We can't set estimate_current_event via parser helper API before configure. Do you know the workaround for this?

Reaming task is replacing filter_stream with filter_with_time.

Don't set estimate_current_event = false but test passed. Hmm...

Rebased and apply reviews.

@tagomoris Could you check this patch again?

Parser plugin helper can instantiate two or more parsers. So we can add "secondary" parser for the values which don't match the first parser. It's nice to have to replace fluent-plugin-multi-format-parser.

Applied reviews.

@tagomoris It is a good idea > secondary parsers.
It's a TODO after merged this PR.

This patch is not the cause of test failure.

@tagomoris check again.

I commented about just one point (twice). Others look good to me.

Applied all reviews. After test passed, I will merge this PR.


feat: superchain landing page

PR-Codex overview
The focus of this PR is adding a new page for the Hackathon Grant Superchain with a HeroSection component.
Detailed summary

Added HackathonGrantSuperchain page in pages directory
Created HackathonEarn component with SEO metadata and HeroSection
Added HeroSection component with image, text, and button
Updated styles and components in grant/superchain directory

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

📝 Changed routes:

/connect
/grant/superchain

(view full size)
(view full size)

Commit 0993950c2c6d0ec2a228d80aa80235c5869d9d2a (https://thirdweb-eoqeq785j.thirdweb-preview.com).


Add meeting minutes to wiki

This is required by the A1 brief.
For the meeting minutes under the previous members, put (missing) (or something similar).

Approved

Closed as completed by Kyle https://github.com/se310-t6/politicry/wiki/Meeting-Minutes


Storybook story for status tag

Addresses #8656.
This PR adds stories for status tags in wagtail.

A set of stories for the legacy status tag
A story for the new status tag.

Screenshot

How to test
Start up the django-pattern-library and storybook and view the stories labelled:
Page Status Tag && Page Status Tag New
Browsers tested on
Chrome 102, Storybook

@stevedya can you please re-do this PR (I cannot re-open a merged PR it seems), I have had to revert it. This change actually broke everyone's non-developer installation of Wagtail - see https://github.com/wagtail/wagtail/issues/8798
We probably need to do a better job of documenting the developer installation requirements and also pattern library somehow.
@thibaudcolas

@lb- Ahh, oh nooo 😣. Thanks for following up. I can re-open and look for an alternate solution for the  patternlibrary_override_tags issue mentioned above.


Add prometheus.min-shards option.

The default for min-shards is 1 - wondering if we want to go with that.
(Also: why don't we use for parsing max/min-shards UInt instead of Int?)

@jmacd Oh, yeah, good catch, I had forgotten about it.


chore: Do not keep a lock file for highest deps resolution

Summary

Test Plan

Checklist

[x] My pull request has a descriptive title.
[x] I have read the CONTRIBUTING guide.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] If appropriate, I have added necessary documentation.
[ ] For user-facing changes, refactorings, performance improvements or documentation updates, I have added a changelog entry using Changie.

For both the title of the PR and the changelog entry, prefer simple past tense or constructions with "now". For example:

Added Client.invite_participants()
Client.user_activation_settings() now accepts a user_activation_settings keyword argument

📚 Documentation preview 📚: https://citric--1143.org.readthedocs.build/en/1143/

Ping @GabxTaro


Add text to README stating current model can only be run for CONUS Landsat images

will work on the readme with Martha.  I'm using SSEBop as a guide.


Cert store fix && Stream API refinements

Replaces #7320 and #7321

Fix a missing NULL reset after free in certificate store
Adds validity checks for wStream API
Removes problematic wStream setter functions that are not strictly necessary.

@eyalitki So, I´ve modified the wStream API with more thorough checks and added return values to functions that modify the internal state.
Functions modifying the internal state that are not essentially necessary have been removed.
This way the streams can now no longer be in an inconsistent state and internal WINPR_ASSERT (which might abort and terminate the program) are able to (hard) enforce these limits

Looks very good. I didn't think you'll go through the trouble of adding such checks per API function.
Just a question regarding the WINPR_ASSERT: Are the checks are done in runtime in the default/recommended compilation? Or they only used in development? We need these checks to detect bugs in the code, but mainly to block attacks in runtime.

@eyalitki the WINPR_ASSERT is only done when compiled in.
But due to the newly introduced Stream_Set* checks the internal state of the internal stream can not be invalid. (e.g. s->buffer, s->pointer and s->length are in a valid within the stream, not necessarily correct for the application though and s->capacity is internal only, except when utilizing external buffers)
Only thing that can still happen is a missing Stream_GetRemainingLength or Stream_GetRemainingCapacity check but that is something that the developer has to take care of anyway.

@eyalitki oh, and our nightly builds have that enabled, of course.

The thing is that most of the vulnerabilities that I encountered thus far in FreeRDP were exactly in cases that the developer forgot / miscalculated the size of Stream_GetRemainingLength. And once a single such check was missing/wrong, all the rest of the checks are pointless because the return value of size_t will return a huge value instead of a negative one. While I respect the  responsibility of a developer to write secure / correct code, the current API is fragile, and year-after-year it proved itself as prone to vulnerabilities. This is the reason I submitted #7321, and I still stand behind my original pull request.
As long as the WINPR_ASSERT calls are only used during development, they will be very handy in catching bugs, but they won't suffice to block attacks against your users at runtime. And recent years proved there are numerous such vulns in your code, most of which are a result of the need for every developer to count every byte, and remember to update every check if some line was added later on in the code. Changing size_t to ssize_t will change the current state drastically, as it will allow later code checks to catch earlier errors, instead of doing nothing.
If this is the case, I request you reconsider again your decision regarding #7321, as at least the 2nd line of defense there could be added as part of this pull request and it will be useful to block attacks (after they got triggered). I still think that the entire pull request of mine would greatly contribute to the project, especially if most of the checks you added are absent at runtime.
My pull request aimed to make FreeRDP more robust against human-errors. Counting on every developer, of every feature in the data-path, to always count all bytes correctly, isn't the best approach, and it didn't really prove itself as a secure methodology up until now.

@eyalitki yes, the problems you faced were mostly due to broken input validation.
The situation there has been addressed in various cleanups.
But regarding your pr:
The case for Stream_GetRemainingCapacity is no longer possible, as capacity is handled internally. (The case with Stream_New(buffer, capacity) might still give a problem, but there is nothing we can do if a developer lies to us about the buffer size)
In case of Stream_GetRemainingLength this might happen, true. But that does not require a change in return, but simply returning nothing larger that there actually is so that parsing is aborted when a check is actually done. (this does not avoid missing checks but can at least tell nothing left´ instead of take whatever you want` when a check is done.

@eyalitki I´ve updated the pr to address the issues you raised:

It is now impossible to have an invalid large return value at runtime from Stream_GetRemainingCapacity and Stream_GetRemainingLength
Whenever a violation of a (maybe not compiled in) WINPR_ASSERT for the length checks is found, a big fat warning is logged (with a complete stack trace)

this way it is:

Easy to spot (even at runtime) that there is something wrong
No invalid read might occur after a (later) check as the value returned is never larger than the actual data available.

Looks good, it indeed resolves the issue with WINPR_ASSERT being present only during development. If we can pay the performance impact of these checks, the current commits are indeed a great step forward (security-wise).
As for #7321 I added the relevant comments over there.

@eyalitki Hope the pr is now in a shape we can both agree on.
Found some (minor) issues in guacamole while testing, see https://github.com/apache/guacamole-server/pull/347

Looks very good, thanks a lot for your fast cooperation on this :)
At the current state it indeed contains all features I wanted to add as part of #7321.

@eyalitki you´re welcome, always trying to get this more reliable.
Did move the check function to internal linkage though, don´t want to expose it (there is always someone who will start using and depending on it :))

@freerdp-bot test


Waypoint: Practice Functional Programming

Challenge http://www.freecodecamp.com/challenges/waypoint-practice-functional-programming has an issue. Please describe how to reproduce it, and include links to screen shots if possible:
I appreciate what you're trying to do by referring people to this tutorial, but it's not the greatest. In addition to not really providing detailed explanations of the functions they're trying to teach, there seem to be bugs in the answers provided at the site. For example, on exercise 20, when I click "show answer" and then run the code they provide, it does not let me move forward. I'm now stuck and can't complete this waypoint unless I do some digging through the source, and I'm certain that's not what you're trying to teach.
http://imgur.com/igv8e1h

Quincy has stated to me that they are in the process of writing their own functional programming tutorial similar to this one but hopefully easier to understand, but that it would be "a long ways off".

This is a duplicate of #1410.


[BEAM-5848] Fix coder for streaming impulse source

Fix coder for streaming impulse source.

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---

Looks good. Merging.


Cannot use asExternalUri with remote-tunnel in companion extension

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.92.2
OS Version: macOS 14.6 (23G80)

Steps to Reproduce:

Connect using remote-tunnel for remote development
Have a server running on the remote machine and forward the port and have the local URI using vscode.env.asExternalUri.
Have an extension running on UI host programmatically connect to this URI. The UI-side extension is designed to work with a remote-side extension as companion which starts the server on the remote.
Notice that this works well in remote-ssh, but with remote-tunnel, the companion extension cannot connect to the forwarded URI with authentication failure.

How can the companion extension authenticate to this URL forwarded inside the same VS Code session?

@connor4312 Any suggestions on how to go about this?

Duplicates https://github.com/microsoft/vscode-remote-release/issues/9195


Its posible to compile the app for webos?

Did you find a way to do this because I am trying to do the same...

Did you find a way to do this because I am trying to do the same...

It's need to have webframe app to use it on WEBOS.

thank you so much I spent all night trying to figure this out
Get Outlook for Androidhttps://aka.ms/ghei36

From: Saentist<EMAIL_ADDRESS>Sent: Tuesday, December 31, 2019 8:05:25 PM
To: streamaserver/streama<EMAIL_ADDRESS>Cc: nbendall34<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [streamaserver/streama] Its posible to compile the app for webos? (#679)
Did you find a way to do this because I am trying to do the same...
It's need to have webframe app to use it on WEBOS.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/streamaserver/streama/issues/679?email_source=notifications&email_token=AH6Z5MOOIBN4HWJM2NCFJELQ3OQYLA5CNFSM4F3UNKY2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEH4UCUI#issuecomment-569983313, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH6Z5MISBSG34CXHGKVXTDTQ3OQYLANCNFSM4F3UNKYQ.

http://webostv.developer.lge.com/develop/app-developer-guide/hosted-web-app/
https://forums.webosnation.com/webos-development/216832-webview-load-html-string.html

Have you managed to do this?
Get Outlook for Androidhttps://aka.ms/ghei36

From: Saentist<EMAIL_ADDRESS>Sent: Tuesday, December 31, 2019 9:38:00 PM
To: streamaserver/streama<EMAIL_ADDRESS>Cc: nbendall34<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [streamaserver/streama] Its posible to compile the app for webos? (#679)
http://webostv.developer.lge.com/develop/app-developer-guide/hosted-web-app/
https://forums.webosnation.com/webos-development/216832-webview-load-html-string.html
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/streamaserver/streama/issues/679?email_source=notifications&email_token=AH6Z5MMD24ERXE3F7PZOGRTQ3O3TRA5CNFSM4F3UNKY2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEH4W3KY#issuecomment-569994667, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH6Z5MJKZNA3K6FUGCYMDTDQ3O3TRANCNFSM4F3UNKYQ.

Me no, but some one with expirience can do it.
Github is big comunity connect with some webos developer.

I will try
Get Outlook for Androidhttps://aka.ms/ghei36

From: Saentist<EMAIL_ADDRESS>Sent: Wednesday, January 1, 2020 10:23:03 AM
To: streamaserver/streama<EMAIL_ADDRESS>Cc: nbendall34<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [streamaserver/streama] Its posible to compile the app for webos? (#679)
Me no, but some one with expirience can do it.
Github is big comunity connect with some webos developer.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/streamaserver/streama/issues/679?email_source=notifications&email_token=AH6Z5MI6QD7OYIOYAHIOVL3Q3RVIPA5CNFSM4F3UNKY2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEH5CBNY#issuecomment-570040503, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH6Z5MPH6RBBEFVEV3OERKDQ3RVIPANCNFSM4F3UNKYQ.


Chef Server API should not allow client-names with newline characters

In the current chef-server we can create a client with name "staging-mike\n".
That should not be allowed.
REF: https://github.com/chef/chef-server/issues/90
Issue noted in : https://getchef.zendesk.com/agent/tickets/3212

Similar to #90, I tried to recreate this.  I got a 400 response from the server with the message
{"error":["Invalid client name 'testing_\nclient' using regex: 'Malformed client name.  Must be A-Z, a-z, 0-9, _, -, or .'."]}
The input here was:
"(<<-EOM)" testing client EOM
Is it possible that this client was modified through an external means?

This is possible via the API on a PUT to an existing client:
chef > api.get("/clients").keys
 => ["acme-validator", "foobar"]
chef > api.put("/clients/foobar", name: "foobar\n"); nil
 => nil
chef > api.get("/clients").keys
 => ["acme-validator", "foobar\n"]

Hello,
The solution proposed by stevendanna id not working with Chef Server 12. I tried to change the name with:
chef-shell > clients.transform(":") do |client|
if client.name =~ /borat/i
client.name("barot")
else
nil
end
end
The result indicated that the name changed correctly but after executing the command "clients.all", the name of the client is still always:
name => 'XXXXXX
'
So, with this modification I'm able to access to the client in the web UI of Chef.

We solved this by renaming the client from within postgres, and then deleting the client with knife.
 (Y/N) y
Deleted client[delete-me]


Implement Queue

Add comment to get assigned to your desired language.
Follow Submission directory Structure for PR merge.
Submission Title
Queue Implementation
Submission Type
Data Structure
Submission Description
Implement Queue form scratch
Submission Directory
Dat_Structures => Queue => Queue_Implementation.extension
Language

[x]  C++
[x]  C
[x]  Python
[x] Java
[ ] C#
[ ] JavaScript

could you assign  it to me?
I'll be able to implement it in C++

could you assign it to me?
I'll be able to implement it in C++

Done

Assign me to implement in C

Assign me to implement in C

Assigned for C

I could try implementing this in java.

I want to implement this in python

I could try implementing this in java.

Assigned for Java

I want to implement this in python

Assigned for Python

Could you assign this to me for C#.

Could you assign this to me for C#.

Assigned for C#

Please Assign me for JavaScript

Please Assign me for JavaScript

@Siddhant-K-code Assigned for Javascript

I want to implement this in python

@PrajaktaS23 Not completed yet.

could you assign it to me?
I'll be able to implement it in C++

@shailshah76 Not Completed yet.

C++ and Python available.

I'll complete it by tomorrow i am working on it

I'll complete it by tomorrow i am working on it

Ok then assigning you for C++*

can i work on Python ?

can i work on Python ?

@Siddhant-K-code Assigned for Python

Closing issue.


Problems with gorm: nullable columns are always null instead of some post value

In form I have select with name Parent that is not null and have some number in dropdown (for example 7).
Table that has parent column is always NULL and not that number 7.
Parent is defined as FK in table RouteCategory and default value is null. FK is to table itself.
This is model:

type RouteCategory struct {

	Id					int
	ParentCategory  	*sql.NullInt64 		`gorm:"column:ParentCategory sql:"index;null"`
	CategoryName   		string  			`gorm:"column:CategoryName; sql:"type:varchar(64);"`
	CategoryDescription string  			`gorm:"column:CategoryDescription; sql:"type:varchar(255);"`
	IsActive 			int 				`gorm:"column:IsActive; sql:"type:smallint(1);"`
}

Can you maybe help me with that?

     log.Println(a.Request.Form.Get("ParentCategory"))// --> this is string and not int or int64
     err := a.Txn.Insert(&category) ->this inserts null for parent instead of number 7

You can show more code(html form and controller)?


[cfg] Add Ownership SIL utility method splitAllCondBrCriticalEdgesWit…

…hNonTrivialArgs.
Currently, the ownership verifier assumes that all cond_br with critical edges
do not have non-trivial arguments. This utility method fixes any such issues and
should be run /after/ any passes that mess with the CFG in ownership SIL if this
could come up.
Chopping this off of a larger commit.
rdar://31521023

@swift-ci smoke test and merge

@swift-ci smoke test and merge

@swift-ci smoke test and merge

@swift-ci smoke test and merge

sigh Typo.

@swift-ci smoke test and merge

@swift-ci smoke test and merge

@swift-ci smoke test and merge

@swift-ci smoke test and merge

This test failure is not from me.

@swift-ci smoke test and merge

@swift-ci smoke test and merge

Now an LLVM issue. Lets try this again.

@swift-ci smoke test and merge

@swift-ci smoke test and merge

@swift-ci smoke test and merge

@swift-ci smoke test and merge


Visualize sonar data as a stack of cards/data cube

For sonar data from a stationary platform in the ocean (often called a "mooring"), we can see movements of animals up and down the water column according to the daylight cycle.
The data cube representation basically plots 1 image for each day, from this type of sonar time series, and arrive at the following figure:

Data Cube Representation of Time Series Illustrating Seasonal Variations
(Image from ASL Environmental Sciences https://aslenv.com/AZFP-data.html)
It will be cool to have this in echoshader!
We can have widgets to interactively control the time span, which frequency to plot, colormap, etc.

https://user-images.githubusercontent.com/56751303/158410279-679950d7-6779-4f23-be23-13623f6af59e.mp4
Hi, mentors. It is not hard to control frequency or colormap by using panel, but it may get conflict when setting time as x-axis and controlling time-span at the same time. I am trying to solve this problem.

Is time range-slider better? Or normal slider to choose fixed time span like a day?

@ldr426 : cool! I am thinking the beginning and ending date is fixed depending on the input data, but then where the open face is "slices out" depends on a single date, so it seems that a normal slider would work. Thoughts?
In terms of controlling the x-axis (assuming in the context of this data cube), the front face should always have the same x-axis since it is always a 24-hr period, and y-axis be along the same depth, but the y-z plane would have different length depending on the start/slice/end dates.

@leewujung : got it! How about using DatePicker? It can fix start/end date, then limit the dates that can be selected. As shown below, I set start/end date according to ping_time input. It is much more convenient to use DatePicker than normal slider, I think.

@leewujung wow!
Does plotting the top face mean we only considerring "the top", like depth=0m or -5m, near the sea surface?
The new component can use scatter plot, whose y-axis is date and 24-hour is x-axis. If Sv values > -75, then can be put on the plot. Or line plot better?

Does plotting the top face mean we only consider "the top", like depth=0m or -5m, near the sea surface?

Actually this is interesting -- this will be cool thing, to visualize how the DVM sandglass shape at different depth! So then you can slice the original cube either along depth like what you said, and also slice it along date, like the original thought.

The new component can use scatter plot, whose y-axis is date and x-axis is 24-hour. If Sv values > -75, then can be put on the plot. Or line plot better?

Scatter plot is an interesting idea: What if for Sv > -75 (or some other threshold) you scale the size of the scatter points with the Sv value? Might would be good to use something like xr.coarsen to reduce the number of points if it gets crowded.
I wonder if a scatterplot and an image may give different insight to the data!

@leewujung Thank you for answering.
yeah, it would be more cool to scale the size of points according to values.
Now it is much more interesting and challenging. I will try it! :smiley:

@leewujung  already finished datepicker, frequency and colormap part.
I will do more work for Diel vertical migration (DVM) and design a more beautiful interface before pulling request for this issue.
ps: I just use two consecutive hours(23-24,0-1) from two consecutive days as data input, so x-axis doesn't show 24-hour. Also some data before 24:00 in raw data are vacant. When using enough big and complete data, it will show a whole day.
https://user-images.githubusercontent.com/56751303/158801192-a3e4c972-3be6-4480-a4db-1d63fc4c4cf7.mp4

@leewujung
should we design a template page just like this. put echoshader logo and offer more friendly user-interface. That would be much cooler than just showing it in jupyter cell, I think

some data before 24:00 in raw data are vacant

Yes, that is true. The instrument had a small off period before midnight each day.


Proxy for Webview, Desktop/Mobile mode

How to integrate proxy to webview_flutter?
And I think, if userAgent is added to webview, it will be helpful for users who want to use desktop mode as well.

I'm also looking to use a proxy in webview_flutter.

Any solutions yet? I figured out how to proxy dart http request but don't know how to proxy the webview

At least on iOS, AFAIK this is impossible; WKWebView does network requests in a separate process.
Beyond that, building proxies into each component that does network access isn't really scalable; proxying is really something best done at a device level.


API key not found

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting questions.
[X] I have removed any sensitive information from my code snippets and submission.

App Id
https://us-east-1.console.aws.amazon.com/amplify/home?region=us-east-1#/d3sst100jc4y17
AWS Region
us-east-1
Amplify Hosting feature
Frontend builds
Question
When i run amplify push i got error - "Reason: API key not found: .....my key"
in AWS AppSync newer key.
How i can update aws_appsync_apiKey in amplify

Hi @Ruslan310 👋 , thanks for raising this!
To better assist you, I am going to transfer this issue over to the amplify-category-api repository.


[BUG] Installation installation link is dead

Wiki installation link is dead
Varken Installation guides can be found in the wiki.
To Reproduce
Steps to reproduce the behavior:

Click wiki @ wiki
Notice it gives you an error

Expected behavior
Should lead to wiki.
Screenshots

Additional context
wiki.cajun.pro returns 404 as well. Possibly intermittent BW issues?

Is this project still alive?
Would love to get this link working so I can get some guidance to getting it all working.

https://old.reddit.com/r/homelab/comments/a4txoe/i_introduce_varken_the_successor_of/#ix704i3
Looks like dead. Shame that he hasn't updated the repo to reflect EoL.

its not dead just slow updates. been taken over by a few people just trying to keep it going :)

its not dead just slow updates. been taken over by a few people just trying to keep it going :)

Fair enough.

@MDHMatt Are there backups of the setup instructions?

@MDHMatt Are there backups of the setup instructions?

Do you know how to use docker?? That is by far the easiest way to do it

@MDHMatt Are there backups of the setup instructions?

Do you know how to use docker?? That is by far the easiest way to do it

I understand docker, and most likely how to deploy it, I'm personally looking for the docs on the configuration values. Screenshot taken from waybackmachine

@M1kep yeah run the develop branch and edit that ini file with the ip addresses of the apps, any api keys and the influxdb details and away you go


ext_proc: Clean up errors in configuration proto

Commit Message: Clean up errors in the ext_proc configuration proto
Additional Description:

Fix comments regarding what is and isn't implemented.
Indicate that "stat_prefix" really is implemented, because it is.
Re-name two misnamed (but as yet unimplemented) fields in per-route
configuration.

Risk Level: Low
Testing: Existing integration and unit tests
Docs Changes:
Release Notes:
Platform Specific Features:

docs rendered here https://storage.googleapis.com/envoy-pr/19733/docs/index.html
specifically in this commit: https://storage.googleapis.com/envoy-pr/96af951/docs/api-v3/extensions/filters/http/ext_proc/v3/ext_proc.proto.html


Fix snpashot replication test

What changes are proposed in this pull request?
Fixes snapshot replication test which could block sometimes by adding a timeout when GetInfo requests are disabled by mocking (as would happen they were not being mocked).
Does this PR introduce any user facing changes?
No

alluxio-bot, merge this please


Add configuration option to provide index file name

Previously by default just used index.html but let's make it as optional configuration parameter

We can revisit this later, for now lets avoid a dependency on boost.system, and we can just hard-code the index.html file.


reviewid

How do i get a reviewid from a show?

Thought it found it, but didn't, passing show id to get review data (breaking bad) but nothing shows, any pointers what id to pass to GetReviewAsync?

https://developers.themoviedb.org/3/movies/get-movie-reviews
It would seem you need to pass the movies Id. Any Movie object should contain an Id - is this what you're after?
Note that Movies and Tv have different endpoints/methods - it's entirely possible you cannot get reviews by TV show/episode/ ..

i think you can't get them for tv shows, which is a bummer, but now it makes sense i'm not getting any data back :)


Improving getMatches

Improve getMatches so files with similar names are not refreshed. Until now the url matcher only checked for the file name to be contained within the attribute value.
With this change it will not match files with similar names.
Example:
Saved: checkout.css
Links:

http://localhost:8080/checkout.css
http://localhost:8080/ng-checkout.css

Refreshed Before:

checkout.css
ng-checkout.css

Refreshed After:

checkout.css

How about /test.css and /abc/test.css? I didn't test it, maybe I am wrong.

they'll both refresh, I know is not perfect, but it's better than it was...
hehe
On 20 Oct 2016 18:21, "Wensheng Yan"<EMAIL_ADDRESS>wrote:

How about /test.css and /abc/test.css? I didn't test it, maybe I am wrong.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/BrowserSync/browser-sync-client/pull/34#issuecomment-255028538,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEVRD6BDuoBtvjiCgWkW3BibhxbOY-bxks5q1xZ7gaJpZM4Kbwvf
.

Looks good, thanks I will merge/update later :)

Thanks!
On 21 Oct 2016 00:39, "Shane Osbourne"<EMAIL_ADDRESS>wrote:

Looks good, thanks I will merge/update later :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/BrowserSync/browser-sync-client/pull/34#issuecomment-255108370,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEVRDwAnvtnfxCaX2Sziop_ETqNKF8Dpks5q128ggaJpZM4Kbwvf
.

@yanwsh we only ever look at the file names as path-matching from disk -> url is not practical/accurate - especially for dynamic sites.
Having said that, I'd be more than happy to have a 'strictMatch' option that verified path also, if someone wants to implement :)

released at v2.4.3, thank you very much :)
You have also been invited to the Browsersync org (which means you'll full write access to all repos)
Please continue to submit PRs for code changes & be sure to review the guidelines. https://github.com/BrowserSync/browser-sync/blob/master/CONTRIBUTING.md#-submitting-a-pull-request
Any questions, just ask :)


add tests setClass and getClass for DomUtil

issues: https://github.com/Leaflet/Leaflet/issues/7435

Please add he label 'hacktoberfest', waiting for feedback against the PR.

@micro-chipset Can you also add test cases for SVG elements? These have a different path in the code if you check the implementation of setClass() and getClass().