Update AUSSDA endpoint

Replace  https://data2.aussda.at/oai with  https://data.aussda.at/oai
Use metadataPrefix=oai_ddi
This is a non-functional change (from User perspective), just an update to the harvester configuration file.
Change will be deployed to Staging instance ASAP, so that the weekend full harvest and CMV and XML Schema checks can be carried out against the new configuration.

OVAL BASE checks (warnings and errors are same as for AUSSDA's current production endpoint)

OAI-PMH Validator checks (Identify):

OAI-PMH Validator checks (ListRecords OAI_DDI):

Update from Lars:
please use metadataPrefix=oai_ddi&set=originalout for testing the new Dataverse version with Validation Gate: BASIC; Profile: DDI2.5 PROFILE - MONOLINGUAL 1.0.4

List of metadata validator constraint violations and XML schema violations has been sent to Lars.

For correctness, set should be all_published (though it returns same number of results as originalout)


Add status to CLI

Qualify of life improvement to show resource status when using the CLI (i.e. oc get telemetry, etc)

@abays: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:
Test name 	Commit 	Details 	Required 	Rerun command
ci/prow/telemetry-operator-build-deploy 	adba932 link 	false 	/test telemetry-operator-build-deploy
Full PR test history. Your PR dashboard.

Error from server (InternalError): error when creating "STDIN": Internal error occurred: failed calling webhook "mopenstackcontrolplane.kb.io": failed to call webhook: Post "https://openstack-operator-controller-manager-service.openstack-operators.svc:443/mutate-core-openstack-org-v1beta1-openstackcontrolplane?timeout=10s": dial tcp <IP_ADDRESS>:9443: connect: connection refused

/test telemetry-operator-build-deploy


drivers: serial: Remove an unnecessary critical section for SMP

Summary

Since the caller already holds xmit.sem in uart_write(),
this critical section in serial.c is unnecessary for SMP

Impact

None

Testing

Tested with spresense:wifi_smp, esp32-devkitc:smp

@xiaoxiang781216
I've created a new PR which we discussed at https://github.com/apache/incubator-nuttx/commit/dacd041a94b4150a90b401ef2098ae0015c7fa97#r51648725

The comment is incorrect, let me check again.


Increase underscore version because of CVE-2021-23358 vulnerability

It is a high severity vulnerability within this package. It's extremely necessary to update this package and prevent the usage of underscore at ~1.7.0
Why
CVE-2021-23358
Vulnerable versions: >= 1.3.2, < 1.12.1
Patched version: 1.12.1
The package underscore from 1.13.0-0 and before 1.13.0-2, from 1.3.2 and before 1.12.1 are vulnerable to Arbitrary Code Execution via the template function, particularly when a variable property is passed as an argument as it is not sanitized.
Packages affected
Every package that used @ladjs/i18n it's vulnerable to this attack because it's using country-language package in the background and this package it's the one with the vulnerable package version

Best Regards!
Feel free to check it out with cross dependencies between projects but so far, this project works with that package version.

Hello, will you merge this ?

Hello, will you merge this?

I cannot merge since I'm not the author, but we can guarantee that the repo is deprecated by the authors. Maybe we should need to plan a new strategy here @adam2k, What do you think?

Updating underscore to 1.13.1 did not solve my issue.
I am getting this error:
Error: Cannot find module 'underscore'

Updating underscore to 1.13.1 did not solve my issue.
I am getting this error:
Error: Cannot find module 'underscore'

https://github.com/Clever/underscore.deep/blob/master/package.json

This is not a fix for a module that is not found, has to be something else. This PR it's a fix for a vulnerability issue

@Warkanlock I tried to email the company using their webform here: https://global.bdswiss.com/contact/
but I didn't get a response other than an automated message saying that they received my message.  It seems like the owners of this repo are not going to approve or merge this PR.

@Warkanlock I tried to email the company using their webform here: https://global.bdswiss.com/contact/
but I didn't get a response other than an automated message saying that they received my message. It seems like the owners of this repo are not going to approve or merge this PR.

Maybe it's time to make our own library haha (jk)

underscore.deep does not work with underscore 1.13.1 and country-language depends on both underscore and underscore.deep.
Try force updating underscore
  "resolutions": {
    "underscore": "^1.13.1"
  },

https://classic.yarnpkg.com/en/docs/selective-version-resolutions/

underscore.deep does not work with underscore 1.13.1 and country-language depends on both underscore and underscore.deep.
Try force updating underscore
  "resolutions": {
    "underscore": "^1.13.1"
  },

https://classic.yarnpkg.com/en/docs/selective-version-resolutions/

Good catch man, thanks!

Sorry for the confusion.
If you force update underscore to 1.13.1, underscore.deep will return error Cannot find module 'underscore'

@Warkanlock it might be best to find a different repo.  The library that I am using that has this library as a dependency is email-templates, but they patched the CVE with a different dependency.
https://github.com/forwardemail/email-templates/commit/6eeb95e0792db26fecb171ab243aa57d0e81195d

@alatzidis can you please publish the latest version as you have merged this PR!

Released

thanks @alatzidis but still I see version 0.1.7 available.

@alatzidis appreciate if you can publish the latest version.. thanks!


Deprecation of contain_duplicates in inspec

[DEPRECATION] `contain_duplicates` is deprecated and will be removed for InSpec 1.0. See https://github.com/chef/inspec/issues/738 for more details
[DEPRECATION] `contain_duplicates` is deprecated and will be removed for InSpec 1.0. See https://github.com/chef/inspec/issues/738 for more details

Related issue: https://github.com/chef/inspec/issues/738
The related code is in the os_spec.rb

+1 we should fix them


15 add workflow to automatically publish on pypitestpypi 1

Had started this a while ago, and decided it needed to be finished before moving on to other stuff...

adds a workflow to build the package and upload it to testpypi. This is triggered each time a push to master happens and only runs when the tests to have successfully completed.

adds a workflow to make a signed github release and a release on pypi. This must be triggered manually.

version bumping must be done manually at the moment.

[x] rename tests workflow from main to ci.

[x] add workflow to build project and make testpypi release whenever the ci workflow runs successfully on the master branch

[x] add workflow to make github release and pypi release upon manual request

Question: I'm not entirely sure how to properly test this, since the environments are on purpose configured to only allow deployments from master. For testing purposes, I had removed this rule and it ran, but imho it should stay in place. The generated releases have been deleted again.
Also, I'm not very familiar with automatic deployments via gh actions, so any improvements would be welcome

It looks all good, however there may be issues with the package name since it does not adhere to Python naming conventions.

Often we let the researchers pick a package name, should we ask them?

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

Project coverage is 83.90%. Comparing base (76eeac8) to head (f9af690).

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #28      +/-   ##
==========================================
+ Coverage   83.78%   83.90%   +0.12%
==========================================
  Files           6        6
  Lines         259      261       +2
  Branches       66       66
==========================================
+ Hits          217      219       +2
  Misses         23       23
  Partials       19       19

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


Feature Request: option to not block non-ReactDnD drag operations

Right now all drag-and-drop operations that aren't managed by React-DnD are blocked. On pages that already have non-React-DnD drag-and-drop-handling code, this causes things to break while React-DnD is active. An option to disable this behavior would be useful.
For the current time, I've put up a version of react-dnd-html5-backend that's patched to have this disabled on npm.

Original Issue

It's surprising that react-dnd-html5-backend still interferes with the operation of other components on a page (rich text editors, file upload dropzones, etc).
https://github.com/react-dnd/react-dnd/pull/1240 seems like it will have helped with https://github.com/react-dnd/react-dnd/issues/457 but there are still other unexpected side effects of the capturing event listeners and their use of event.preventDefault. I wonder if it might be possible to opt out of this behaviour?

Related issues:

https://github.com/react-dnd/react-dnd/issues/704
https://github.com/react-dnd/react-dnd/issues/1115

I encountered this again today.

hunger for it !!!
react-text-editor with dnd troubles me a lot .....................

Unfortunately running into the same issue. Using react-dnd pretty much blocks any none-react-dnd operations.
Any solution for this? Tried the suggestions in the original issue, but did not fix it for me.

This problem occurs, is there a solution?

Pretty late here, but I found that replacing react-dnd on the page where I have another component that was being interfered with was the only solution I could get working.
I replaced it with "react-sortablejs" which in my use-case was able to provide the same thing I needed. Not happy to have both libraries installed, but it unblocks for now. If folks have found a react-dnd solution I'd be happy to see!

Im also having this issue now due to implementing craft.js so I need both to work for various instances on the page.

import { HTML5Backend } from 'react-dnd-html5-backend';

const shouldIgnoreTarget = (domNode: HTMLElement) => {
  return domNode.closest('.lexical-editor');
};

const ModifiedBackend = (...args: any) => {
  // @ts-ignore
  const instance = new HTML5Backend(...args);

  const listeners = [
    'handleTopDragStart',
    'handleTopDragStartCapture',
    'handleTopDragEndCapture',
    'handleTopDragEnter',
    'handleTopDragEnterCapture',
    'handleTopDragLeaveCapture',
    'handleTopDragOver',
    'handleTopDragOverCapture',
    'handleTopDrop',
    'handleTopDropCapture',
  ];
  listeners.forEach((name) => {
    const original = instance[name];
    instance[name] = (e: any, ...extraArgs: any) => {
      if (!shouldIgnoreTarget(e.target)) {
        original(e, ...extraArgs);
      }
    };
  });

  return instance;
};

// Decorate root elements with this
export { ModifiedBackend };

Hi guys, lexical dnd is not working when i use with react-dnd.
Here is the fix that work for me in 2024.
import { QueryClientProvider } from '@tanstack/react-query';
import axios from 'axios';
import React from 'react';
import { DndProvider } from 'react-dnd';
import ReactDOM from 'react-dom/client';
import { Provider } from 'react-redux';
import { BrowserRouter } from 'react-router-dom';
import { App } from './App';
import queryClient from './config/reactQuery';
import { ModifiedBackend } from './lib/html5BackEnd';
import { store } from './store';
import './styles/index.css';

axios.defaults.baseURL = 'http://localhost:8080/api/v1';
axios.defaults.withCredentials = true;

ReactDOM.createRoot(document.getElementById('root') as HTMLElement).render(
  <React.StrictMode>
    <QueryClientProvider client={queryClient}>
      <Provider store={store}>
        <BrowserRouter>
          <DndProvider backend={ModifiedBackend}>
            <App />
          </DndProvider>
        </BrowserRouter>
      </Provider>
    </QueryClientProvider>
  </React.StrictMode>,
);

Then i just need to import ModifiedBackend and use it instead of default html5backend.
Dont forget to replace the editor wrapper className with your.
The className that wraps the whole lexical in my case is '.lexical-editor'.

'./lib/html5BackEnd';
import { HTML5Backend } from 'react-dnd-html5-backend';

const shouldIgnoreTarget = (domNode: HTMLElement) => {
  return domNode.closest('.lexical-editor');
};

const ModifiedBackend = (...args: any) => {
  // @ts-ignore
  const instance = new HTML5Backend(...args);

  const listeners = [
    'handleTopDragStart',
    'handleTopDragStartCapture',
    'handleTopDragEndCapture',
    'handleTopDragEnter',
    'handleTopDragEnterCapture',
    'handleTopDragLeaveCapture',
    'handleTopDragOver',
    'handleTopDragOverCapture',
    'handleTopDrop',
    'handleTopDropCapture',
  ];
  listeners.forEach((name) => {
    const original = instance[name];
    instance[name] = (e: any, ...extraArgs: any) => {
      if (!shouldIgnoreTarget(e.target)) {
        original(e, ...extraArgs);
      }
    };
  });

  return instance;
};

// Decorate root elements with this
export { ModifiedBackend };

Hi guys, lexical dnd is not working when i use with react-dnd. Here is the fix that work for me in 2024.
import { QueryClientProvider } from '@tanstack/react-query';
import axios from 'axios';
import React from 'react';
import { DndProvider } from 'react-dnd';
import ReactDOM from 'react-dom/client';
import { Provider } from 'react-redux';
import { BrowserRouter } from 'react-router-dom';
import { App } from './App';
import queryClient from './config/reactQuery';
import { ModifiedBackend } from './lib/html5BackEnd';
import { store } from './store';
import './styles/index.css';

axios.defaults.baseURL = 'http://localhost:8080/api/v1';
axios.defaults.withCredentials = true;

ReactDOM.createRoot(document.getElementById('root') as HTMLElement).render(
  <React.StrictMode>
    <QueryClientProvider client={queryClient}>
      <Provider store={store}>
        <BrowserRouter>
          <DndProvider backend={ModifiedBackend}>
            <App />
          </DndProvider>
        </BrowserRouter>
      </Provider>
    </QueryClientProvider>
  </React.StrictMode>,
);

Then i just need to import ModifiedBackend and use it instead of default html5backend.
Dont forget to replace the editor wrapper className with your. The className that wraps the whole lexical in my case is '.lexical-editor'.

Oh man this helps a lot. Thanks!


remove \b (\u0008) prefix from /info api calls

This is related to issue https://github.com/docker/swarm/issues/1237

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "jc-info-api1" git@github.com:u2mejc/swarm.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

duplicate of https://github.com/docker/swarm/pull/1147

@vieux It seems like we're mixing data with presentation here?
This does appear to breaking the ability to programmatically parse the output (as noted in #1214). It seems like this should be modified in the cli where output presentation is formatted, would you be able point me in the right direction, I'll be happy to take a crack at fixing it.

@u2mejc I agree it's not good as it is right now, changing the docker cli would be better so we could remove the \b in swarm.

Closing this PR since it needs more changes. Follow the discussion on #1625 instead.

This is fixed in https://github.com/docker/swarm/pull/1721 :tada:

@nishanttotla and @vieux, thank you! I'm looking forward to testing master as time allows!


Possibility to disable the motivational quotes and Hello Name!  It's?

In the spirit of minimalism, please privide option to disable the motivational quotes and Hello Name!  It's? Also, option to hide Task Table header.

I believe that would defeat the purpose of please. While greeting is an important part of it, the more important part is the tasks table that's supposed to be 'in your face' to remind you of the urgency. If you just want a simple greeting with date, I think it would be far easier to just use bash to manually print a greeting text like that.
If you would still like to use please and disable those, you can easily delete those lines from the show() function in ~/.local/lib/python3.X.X/site-packages/please/please.py file. Let me know if you need any help with that :)

My request Was not to hide the task table but the task header.

My request Was not to hide the task table but the table header.

Oh, like the Number, Name and the rest? Yes, we can consider it :)
Meanwhile, you can disable it on your local installation. It's pretty simple, all you need to do is to add show_header=False, in showtasks() function, like this:

In future, we'll consider adding some additional configuration options to modularize the output. in v0.2 you can disable the horizontal line, so we'll add some more for disabling each part individually. Thanks!


Bash Script as a Wrapper Utility

Sampling bash features for being used as a wrapper utility

got it! :+1:

@di3go-martinez I'll create a small PR migrating the example to use easy options instead 👌🏻


NSFW Blacklist Command Bug

Blacklist doesn't work for the E6 command. I have set blacklist tags but they still come through when used through e6 command.
The tagging for E6 is also different from what is specified in the command description. E6 used to take tags with spaces between each but now it returns a "no results found" unless you use the "+" with your tags instead of spaces.
I'm using a self hosted Nadeko v2.16.10.

Show me a concrete example of what you tried to do.

Here's a screencap of it. https://i.imgur.com/mkB3QUz.png (cropped to not show image for obvious reason).
Tag is on the blacklist but still comes through on E6, I asked about it on the discord server and was told to just report it here.

Can't reproduce. Keep in mind e621 has some weird tagging, so you might want to blacklist related tags too. And yes, you should use +, fixed description in the last commit.


New version: VMEC v0.2.0+1

Registering package: VMEC
Repository: https://gitlab.com/wistell/VMEC.jl
Created by: https://gitlab.com/bfaber
Version: v0.2.0+1
Commit: 3c1ca55e9ce50392f61e7f6587861b4631200e03
Git reference: master
Release notes:

Fixed issue initiating VMEC.jl if the libvmec_* library is not dlopenable

Please close, replaced by #52246


Testing/eslint

Add esLint rules, and clean the code from default components.
Eslint needs to be tested! To see the error, write yarn run lint in the terminal.
Indentation errors appear automatically in yellow lines and you can fix it automatically formatting the code. You need to set in your IDE cmd + s to format on save.

Just found you need also to do a check with Prettier. Write this to check yarn run prettier:check


Fix timestep bug

Proposed changes
Fixes bug in timestep implementation in accordance with issue #920.
Types of changes
What types of changes does your code introduce to FESTIM?

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Code refactoring
[ ] Documentation Update (if none of the other choices apply)
[ ] New tests

Checklist

[x] Black formatted
[x] Unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

It’s failing because of the test added by @garth-wells in https://github.com/FEniCS/dolfinx/pull/3502/files
I am not sure why it shouldn’t be allowed to make a mixed element with a single element.

It’s failing because of the check added by @garth-wells in https://github.com/FEniCS/dolfinx/pull/3502/files I am not sure why it shouldn’t be allowed to make a mixed element with a single element.

Mixed elements do not have a well-defined value shape, whereas 'base' sub-elements of a mixed element do. An element should be what it is. If it's mixed it should be mixed, if it's not mixed it should not be mixed.

It’s failing because of the check added by @garth-wells in https://github.com/FEniCS/dolfinx/pull/3502/files I am not sure why it shouldn’t be allowed to make a mixed element with a single element.

Mixed elements do not have a well-defined value shape, whereas 'base' sub-elements of a mixed element do. An element should be what it is. If it's mixed it should be mixed, if it's not mixed it should not be mixed.
@garth-wells
I still don't see why you shouldn't be allowed to do mixed_element([single_element]).
This is a perfectly valid use-case for more generalized problems where one can have M number of species in a problem, where 1 species is a special case. It would be silly to have to have dual code paths for handling this in any application code.

@RemDelaporteMathurin can you share an example of where you rely on a single (base) element to behave like a mixed element?

@RemDelaporteMathurin can you share an example of where you rely on a single (base) element to behave like a mixed element?

let’s say you want to support having multiple diffusion equations co-located in space (but with different function spaces. Then you would have a mixed element ([el_0,el_1]). In other parts of the domain (with submesh involved, you might only have a single species. One doesn’t one to treat those differently.

@RemDelaporteMathurin can you share an example of where you rely on a single (base) element to behave like a mixed element?

@garth-wells we do not need a single (base) element to behave like a mixed element per se. It would just simplify the logic for us. For example, in the HydrogenTransportProblem class, this is how we assign functions (solutions, test functions, previous solutions, etc) to "species":
https://github.com/festim-dev/FESTIM/blob/effcbac86ba0620da1388dcee4e8424ce33939db/src/festim/hydrogen_transport_problem.py#L485-L511
We have this check if self.multispecies: in multiple places to check if there are more than one element and if so, treat it as a mixed element.
This is how we do it in the HTransportProblemDiscontinuous class (using a mixed domain approach):
https://github.com/festim-dev/FESTIM/blob/effcbac86ba0620da1388dcee4e8424ce33939db/src/festim/hydrogen_transport_problem.py#L990-L1021
Here we can have mixed element with only one element and we don't have to have these checks everywhere.
This is an example for assigning functions to species but we have other places in the code where this happens, for instance assigning BCs:
https://github.com/festim-dev/FESTIM/blob/effcbac86ba0620da1388dcee4e8424ce33939db/src/festim/hydrogen_transport_problem.py#L537-L586
Assigning ICs:
https://github.com/festim-dev/FESTIM/blob/effcbac86ba0620da1388dcee4e8424ce33939db/src/festim/hydrogen_transport_problem.py#L610-L642
Having everything as mixed elements would greatly simplify the logic on our side.


Discussion on the architecture

Lightenning fast project @lucidrains 🤣
I guess this is still a WIP, but maybe we can talk a little bit about the model architecture?
I noticed that the past obersvation conditions are projected into latent space and concatenated along latent channel dimension. How is this possible in SD v1.4?
I guess it will cause some error on convolutional layers?

i was just going to improvise my own solution, which does not rely on concatenation

or rather, it can just go through a separate network that is concatenative, before being summed to some hidden state in the main network. i'll get some code down by week's end; that will speak louder than words.

@dibbla hopefully you see where I'm going now that there is a bit of code.


Broke Image Error Attatched

I really need this bot functional, I like its potential features but it gives me this error.

Thank you for reporting this issue. Could you please provide the entire error message so I can better understand the problem? Additionally, can you provide the config file without bot token.

This bug has been fixed.


Silence warnings about deprecations when exporting deprecated types.

FYI,
There is another one when activating the load_extension feature:
warning: use of deprecated item: Use LoadExtensionGuard instead, #[warn(deprecated)] on by default
  --> src/lib.rs:93:32
93 | pub use load_extension_guard::{SqliteLoadExtensionGuard, LoadExtensionGuard};

Thanks, fixed in #204


more Julia 0.7 compat

Test has to be a dependency now

Coverage increased (+0.6%) to 100.0% when pulling cd3f1cb684a548b84414efd36d55dfdf7a4910e2 on gdkrmr:julia_07_compat into 808b1eb46d99102fb41881fe996012a8c2688d3f on trthatcher:master.

broadcasting in centerkernelmatrix! is ~30% faster on a 100x100 matrix, I also took the liberty of exporting centerkernelmatrix! and centerkernelmatrix


Support of lock API

Cant it be implemented?
https://github.com/coreos/etcd/blob/master/Documentation/dev-guide/api_concurrency_reference_v3.md#service-lock-etcdserverapiv3lockv3lockpbv3lockproto

I'm currently working on a PR for this, because we need it too.
https://github.com/Grisu118/jetcd/tree/GG-489-add-lock-support

@Grisu118
Thanks!


start and connect instructions are not clear

After running docker run --name exasoldb -p <IP_ADDRESS>:8899:8888 --detach --privileged --stop-timeout 120  exasol/docker-db:latest it is not clear how to connect to a DB.
Can you please provide an example of the connection string with URL, default login and password please?

Connecting to the database in an Exasol Docker container is not different from the "normal" version. The default username is sys, the default password is exasol. Please refer to the official manual for further information: https://www.exasol.com/portal/display/DOC/Database+User+Manual


Avoid race conditions

Some actions like waiting for running operations et cetera need
to happen in order for the action to work well. Otherwise the
action is prone to race conditions. Thus, this change adds await
statements for all methods where we must await the result.

I don't know why the pre-commit workflow failed. Seems to be some temporary condition? Can someone of the maintainers rerun the job?


Enabled Core Branching strategy and made Neural Plugin as Extensible Plugin.

Description
Enabled Core Branching strategy and made Neural Plugin as Extensible Plugin.
Signed-off-by: Navneet Verma<EMAIL_ADDRESS>Issues Resolved
#83 #74
Check List

[X] New functionality includes testing.

[X] All tests pass

[X] New functionality has been documented.

[X] New functionality has javadoc added

[X] Commits are signed as per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Added the fix for the error happening due to k-NN. https://github.com/opensearch-project/k-NN/pull/684


Add oath services to examples

@pauldowman - added the user services section, and deployed iron and flow versions to meteor.com

Hey @tsemana nice! When trying to use the Facebook login I get a "in development mode" and error. I think that is a simple boolean to flip on your FB developer console. Also I think the Github integration should be under OKG's org account. It is easy to do from Github... just need to transfer the integration to OKG.

@benstr - dupe currentIdentity removed, Facebook test apps are fixed, and Github integration's moved to OKG's org.

@tsemana I did a little tweak to fix a template error that was being shown on the example sites. I do not have access to deploy to those meteor apps so can you make sure the latest versions are deployed? Thanks man!


Deprecating _log functions. 1/10

Notes
This is 1 of 10 pull requests. I intentionally did not update stan/math/mat.hpp and stan/math/prim.hpp. Those files will be updated with the last pull request. The pull request will still work without those files updated.
Submisison Checklist

[x] Run unit tests: ./runTests.py test/unit
[x] Run cpplint: make cpplint
[x] Declare copyright holder and open-source license: see below

Summary:

Adding _ldf / _lpmf functions.
Adding _lcdf and _lccdf functions.
Deprecating _log functions.
Adding tests that verify _log functions match direct calls to underlying functions.

Intended Effect:
This will make Stan generator easier.
How to Verify:
By eye. The tests should pass.
Side Effects:
None.
Documentation:
Only added deprecation warnings. I tried to leave everything else alone so we can track through the changes.
Reviewer Suggestions:
Anyone.
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company):
Columbia University.
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

There's not need for me to split the code this way. I could have also:

added all _lpmf / _lpdf / _lcdf / _lccdf functions first with tests.
then added _log

I think that's going to be easier to review. I'll close this pull request and start over.


Errors from the HVM are outputted in escaped form

Describe the bug
When there's an error reading the output form the HVM, its output is forwarded to Bend so that we can see what went wrong.
However, it is shown in escaped form (meaning new lines show up as '\n', etc), so parse errors are almost impossible for normal people to understand.
To Reproduce
Make Bend output some invalid hvm program (atm we can do this by using // in a name, after it's fixed we can just modify the compilation or the stringification to create something wrong).
Expected behavior
Output is shown as it would in the HVM, not escaped (with new lines, color text, etc)

Closed by #461


apt upgrade nonfunctional after successful do-release-upgrade from Jammy (22.04) to Kinetic (22.10)

Version
Microsoft Windows [Version 10.0.19045.2251]
WSL Version

[ ] WSL 2
[X] WSL 1

Kernel Version
4.4.0-19041-Microsoft
Distro Version
Ubuntu 22.10
Other Software
WSL installed and running via "Ubuntu On Windows" Store app
Repro Steps
Run my normal day-to-day Linux software update BASH script:
(filled with lots of just-in-cases tacked on from years of using WSL like a commandline-only Ubuntu Virtualbox)
$ sudo "PATH=$PATH" bash update.sh
#!/bin/bash

dpkg --configure -a   # resume any interrupted installations
dpkg --configure -a --force-depends   # try to make dpkg stop whining

apt-get check  # no "apt check"??

# apt-fast obsolete?  regular apt no longer takes forever
apt update
apt upgrade -y
# apt full-upgrade -y

apt update --fix-missing
apt upgrade --fix-missing -y
# apt full-upgrade --fix-missing -y

apt install --fix-broken

apt autoremove -y
apt autoclean

Expected Behavior
Script runs, all commands succeed, all installed Linux software updated:
[sudo] password for ____:
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
Get:1 http://security.ubuntu.com/ubuntu kinetic-security InRelease [109 kB]
Hit:2 http://archive.ubuntu.com/ubuntu kinetic InRelease
Get:3 http://archive.ubuntu.com/ubuntu kinetic-updates InRelease [109 kB]
Get:4 http://archive.ubuntu.com/ubuntu kinetic-backports InRelease [99.9 kB]
Get:5 http://security.ubuntu.com/ubuntu kinetic-security/main amd64 Packages [112 kB]
Get:6 http://security.ubuntu.com/ubuntu kinetic-security/main Translation-en [32.2 kB]
Get:7 http://archive.ubuntu.com/ubuntu kinetic-updates/main amd64 Packages [126 kB]
Get:8 http://archive.ubuntu.com/ubuntu kinetic-updates/restricted amd64 Packages [133 kB]
Get:9 http://archive.ubuntu.com/ubuntu kinetic-updates/universe amd64 Packages [60.0 kB]
Get:10 http://archive.ubuntu.com/ubuntu kinetic-updates/multiverse amd64 Packages [4,948 B]
Fetched 784 kB in 8s (94.2 kB/s)
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
1 package can be upgraded. Run 'apt list --upgradable' to see it.
...
(etc. etc. etc.)

Actual Behavior
After upgrading to Kinetic (22.10), all apt upgrade attempts, scripted or typed manually, fail with "Could not read response to hello message from hook"; pending upgrades get marked "not upgraded" as though held back:
...
1 package can be upgraded. Run 'apt list --upgradable' to see it.
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
Calculating upgrade... Done
E: Could not read response to hello message from hook [ ! -f /usr/lib/ubuntu-advantage/apt-esm-json-hook ] || /usr/lib/ubuntu-advantage/apt-esm-json-hook || true: Success
E: Could not read response to hello message from hook [ ! -f /usr/lib/ubuntu-advantage/apt-esm-json-hook ] || /usr/lib/ubuntu-advantage/apt-esm-json-hook || true: Success
Hit:1 http://security.ubuntu.com/ubuntu kinetic-security InRelease
Hit:2 http://archive.ubuntu.com/ubuntu kinetic InRelease
Hit:3 http://archive.ubuntu.com/ubuntu kinetic-updates InRelease
Hit:4 http://archive.ubuntu.com/ubuntu kinetic-backports InRelease
...
0 upgraded, 0 newly installed, 0 to remove and 1 not upgraded.

Diagnostic Logs
I've confirmed that it's specifically apt upgrade that fails, since my alternate forced-update script (do apt install [package] --fix-broken --fix-missing -y for every package in apt list --upgradable) succeeds in fully updating all software.
Also, I've read the other WSL Issues with apt throwing the same kind of hook response error, with the solution to delete the offending script.  That's not an option for me -- /usr/lib/ubuntu-advantage/apt-esm-json-hook doesn't exist.
Other solutions unsuccessfully attempted:

reinstall apt
reinstall ubuntu-advantage-tools (the only ubuntu-advantage* package installed)
close and reopen the Ubuntu On Windows app
restart WSL via wsl --shutdown
restart the LxssManager service

Version
Microsoft Windows [Version 10.0.19045.2251]
WSL Version

[ ] WSL 2
[X] WSL 1

Kernel Version
4.4.0-19041-Microsoft
Distro Version
Ubuntu 22.10
Other Software
WSL installed and running via "Ubuntu On Windows" Store app
Repro Steps
Run my normal day-to-day Linux software update BASH script:
(filled with lots of just-in-cases tacked on from years of using WSL like a commandline-only Ubuntu Virtualbox)
$ sudo "PATH=$PATH" bash update.sh
#!/bin/bash

dpkg --configure -a   # resume any interrupted installations
dpkg --configure -a --force-depends   # try to make dpkg stop whining

apt-get check  # no "apt check"??

# apt-fast obsolete?  regular apt no longer takes forever
apt update
apt upgrade -y
# apt full-upgrade -y

apt update --fix-missing
apt upgrade --fix-missing -y
# apt full-upgrade --fix-missing -y

apt install --fix-broken

apt autoremove -y
apt autoclean

Expected Behavior
Script runs, all commands succeed, all installed Linux software updated:
[sudo] password for ____:
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
Get:1 http://security.ubuntu.com/ubuntu kinetic-security InRelease [109 kB]
Hit:2 http://archive.ubuntu.com/ubuntu kinetic InRelease
Get:3 http://archive.ubuntu.com/ubuntu kinetic-updates InRelease [109 kB]
Get:4 http://archive.ubuntu.com/ubuntu kinetic-backports InRelease [99.9 kB]
Get:5 http://security.ubuntu.com/ubuntu kinetic-security/main amd64 Packages [112 kB]
Get:6 http://security.ubuntu.com/ubuntu kinetic-security/main Translation-en [32.2 kB]
Get:7 http://archive.ubuntu.com/ubuntu kinetic-updates/main amd64 Packages [126 kB]
Get:8 http://archive.ubuntu.com/ubuntu kinetic-updates/restricted amd64 Packages [133 kB]
Get:9 http://archive.ubuntu.com/ubuntu kinetic-updates/universe amd64 Packages [60.0 kB]
Get:10 http://archive.ubuntu.com/ubuntu kinetic-updates/multiverse amd64 Packages [4,948 B]
Fetched 784 kB in 8s (94.2 kB/s)
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
1 package can be upgraded. Run 'apt list --upgradable' to see it.
...
(etc. etc. etc.)

Actual Behavior
All commands succeed except apt upgrade, which claims "Could not read response to hello message from hook"; pending upgrades get marked "not upgraded":
...
1 package can be upgraded. Run 'apt list --upgradable' to see it.
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
Calculating upgrade... Done
E: Could not read response to hello message from hook [ ! -f /usr/lib/ubuntu-advantage/apt-esm-json-hook ] || /usr/lib/ubuntu-advantage/apt-esm-json-hook || true: Success
E: Could not read response to hello message from hook [ ! -f /usr/lib/ubuntu-advantage/apt-esm-json-hook ] || /usr/lib/ubuntu-advantage/apt-esm-json-hook || true: Success
Hit:1 http://security.ubuntu.com/ubuntu kinetic-security InRelease
Hit:2 http://archive.ubuntu.com/ubuntu kinetic InRelease
Hit:3 http://archive.ubuntu.com/ubuntu kinetic-updates InRelease
Hit:4 http://archive.ubuntu.com/ubuntu kinetic-backports InRelease
...
0 upgraded, 0 newly installed, 0 to remove and 1 not upgraded.

Diagnostic Logs
I've confirmed that it's specifically apt upgrade that fails, since my alternate forced-update script (do apt install [package] --fix-broken --fix-missing -y for every package in apt list --upgradable) succeeds in fully updating all software.
Also, I've read the other WSL Issues with apt throwing the same kind of hook response error, with the solution to delete the offending script.  That's not an option for me -- /usr/lib/ubuntu-advantage/apt-esm-json-hook doesn't exist.
Other solutions unsuccessfully attempted:

reinstall apt
reinstall ubuntu-advantage-tools (the only ubuntu-advantage* package installed)
close and reopen the Ubuntu On Windows app
restart WSL via wsl --shutdown
restart the LxssManager service

@PHR16384 you jumped from an LTS to a non-LTS distro. Not recommended!
Try deleting /etc/atp/apt.conf.d/20apt-esm-hook.conf and see if apt upgrade finishes w/out errors. Make a backup first. Also, if you look into this file you'll see the reference to apt-esm-json-hook
If you want to run Kinetic grab an image from http://cloud-images.ubuntu.com/wsl/kinetic/current/ and import into WSL.

@elsaco  That seems to have done it, apt upgrade is working again.  Thanks!

I do not have a /etc/atp/apt.conf.d/20apt-esm-hook.conf file to delete

Slaps forehead with palm of hand....

Can confirm this is still a thing that can happen, FYI.  An update to Apache recreated 20apt-esm-hook.conf (with a prompt to keep or replace the user-made changes to the conf file)


Port this to Edge & Mozilla Firefox

Try and port this extension to both Edge and Firefox and publish this in their respective app stores.

Related issues:

https://github.com/r3bl-org/shortlink/issues/32

@nazmulidris I've looked into this and found that we can port Edge directly without any changes, except for the chrome.storage API alteration. As for Firefox, it's compatible, but we need to modify the manifest.json
https://extensionworkshop.com/documentation/develop/porting-a-google-chrome-extension/

@superbahbi That is great news. The latest changes in the main branch have support for abstracting the storage api implementation away, so we are no longer tied to chrome. It also opens the doors to doing proper testing for all the storage related files.


Add note about Pexip for GCCH to cloud-video-interop.md

Closes https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/9586.

Learn Build status updates of commit 5416945:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Teams/cloud-video-interop.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit b66271f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Teams/cloud-video-interop.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 7d19fc4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Teams/cloud-video-interop.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


changed workflow for ScanBox component

Context
This PR is to address this issue and partially address this issue. Overall, it is to update the workflow of scanning a box in the frontend.
Changes

Added a new query in the utils class to fetch a box with GraphQL endpoint
Updated the ScanBox.tsx component to display the fetched box information on the screen when found
Updated the ScanBox.tsx component to redirect to the CreateBox.tsx component when code exists but is not associated with a box

Note

Error handling is not very robust, dependent on string matching and makes some assumptions about causes of errors
May have overfetched with the query, not sure we need that much box info per request

Nice work @matt-rose6 ! You fetched the correct amount of information for the box, don't worry 👍

I did some manual testing and from that point it general looks good to me :) (Work In Progress though I guess?)
I'm not sure about the exact scope of this PR. The first linked ticket in the PR description above is mainly about tests I think, this PR doesn't have tests yet though, I guess I miss some context.

As discussed with @HaGuesto we will merge this PR now and open a new one for the changes I did on top today.
@HaGuesto could you approve and merge this one within the next hour or so?

@spaudanjo Can you remove the yarn-error.log file and git-ignore it?


Introduce build pipeline to push KEDA images

PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which has an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] If applicable, the PR references the bug/issue that it fixes in the description.
[ ] New Unit tests were added for the changes made and CI is passing.

The change looks good to me. We need to check with MCR team to see if https://github.com/microsoft/mcr/pull/1562 is ok as is, or if we will need to move that part under the oss folder/team there.

Just out of curiosity - Why is there a need for this?

Some internal Azure teams have a requirement to only pull from MCR. MCR SLA is under Azure, which makes any registry outage easier to track for Azure teams compared to external registries. It also simplifies deploying to non-connected/offline Azure environments where only MCR is replicated/available offline.

@ahmelsayed - Is there a way to make the PR merge build succeed? It is failing since the input variable values are not being passed to the build. Also, if in case we are using a test ACR for PR-validation build, I can remove the hardcoding of ACR location.

Some internal Azure teams have a requirement to only pull from MCR. MCR SLA is under Azure, which makes any registry outage easier to track for Azure teams compared to external registries. It also simplifies deploying to non-connected/offline Azure environments where only MCR is replicated/available offline. The image is copied as is, so I don't think there is anything to "improve" per se in the official KEDA image. Just the choice of registry it's hosted on.

Thanks for sharing!

@JatinSanghvi not sure how to pass values to the auto PR run. You could set $VERSION_TAG default to main. I ran it with a value https://azure-functions.visualstudio.com/azure-functions-docker/_build/results?buildId=11662&view=results and it seems fine.


Let escape button close right-click menu

Apologies for the messy commits here, not sure what happened


IE compatibility

IE does not support WebAuthn but it should not crash on the script and allow isSupported calls.

Fixed, as long as a promise implementation is used.


mGear Shifter: #309

Added import and query commands to the ueGear project, allowing Skeleton data to be queried, and imported into Unreal

mGear corresponding branch: https://github.com/mgear-dev/mgear4/pull/316


Problems with map style selection on firefox mobile

selecting a different map on mobile nearly always results in selecting kurviger first before I can open it

@karussell I think you reported this in #74 . I can't reproduce this. I this still an issue for you? It seems like the radio button selection is not working properly?

Indeed this works now. Strange.

Maybe this was fixed by #77 (?)


Reward always 0

Issue summary
[So I saw two similar posts to this: https://github.com/openai/retro/issues/40, https://github.com/openai/retro/issues/72. However, the first link appears to be for the purpose of the contest, which I'm not trying to do (just playing around with retro). The second link uses WinPython which seems to be meant for windows... but I'm using a Mac.
Perhaps things have changed significantly since this tutorial (https://www.youtube.com/watch?v=S4jI-ambYKE) but for some reason my reward always seems to be 0. My reward is how far sonic can move to the right. In my practice file (/Users/dumpy/Documents/ML_FUN/retroAI/tut1.py), I have the following code, `import retro
env = retro.make("SonicTheHedgehog-Genesis", "GreenHillZone.Act1")
#env = retro.make("GoldenAxe-Genesis", "1Player.Arcade.DefaultCharacter.Level1")
env.reset()
done = False
while not done:
env.render()
#action = env.action_space.sample()
action = [0,0,1,0,0,0,0,1,1,1,0,0]
#print(action) # The 1/0s tell you whether or not a button on genesis is on/off

ob, reward, done, info = env.step(action)       # ob = image of screen at time of action
print("Action ", action, "Reward ", reward)     # rew = amount of reward that he earned from whatever in scenario file
                                                # done = whether done condition met
                                                # info = dict of all values set in data.json`

Meanwhile, in my scenario.json folder(/Users/dumbpy/Documents/ML_FUN/retroAI/gym-retro/retro/data/stable/SonicTheHedgehog-Genesis/scenario.json) I have the following,

{
"done": {
"variables": {
"lives": {
"op": "zero"
}
}
},
"reward": {
"variables": {
"x": {
"reward": 1.0
}
}
}
}

I've attached a small gif showing the reward stuck at 0.

  If not, you are probably not modifying the correct scenario.json, look for one under retro.__path__.  If retro does fail to start, you're editing the correct file. But I tried this and it worked fine:
{
  "done": {
    "variables": {
      "lives": {
        "op": "zero"
      }
    }
  },
  "reward": {
    "variables": {
      "x": {
        "reward": 1.0
      }
    }
  }
}

You can also try scenario='contest' which will use the contest scenario based on x-position.

I did what you first advised with writing garbage. Writing garbage did not seem to work as it still ran with reward being 0.
I then tried to find the scenario.json which was under retro.__path__ which I could not find both on my computer and on Github's search for openai/retro. Can you clarify what retro.__path__ may be referring to?
I did notice a second scenario.json file at this location.
/Users/dumby/Documents/ML_FUN/retroAI/lib/python3.7/site-packages/retro/data/stable/SonicTheHedgehog3-Genesis/scenario.json
Which look like this after swapping score for x:

{
"crop": [
0,
0,
0,
0
],
"done": {
"variables": {
"lives": {
"op": "zero"
}
}
},
"reward": {
"variables": {
"x": {
"reward": 1.0
}
}
}
}
However, even when making this file just filled with random text would still run with reward being 0.

Perhaps this may be due to the way I started my environment?
as I did the following steps in order...
mkdir retroAI
python3 -m vena retroAI
cd retroAI
git clone https://github.com/openai/retro.git gymretro
pip3 install gym-retro

OK I think I got it fixed somehow... editing this scenario.json /Users/dumbpy/Documents/ML_FUN/retroAI/lib/python3.7/site-packages/retro/data/stable/SonicTheHedgehog-Genesis/scenario.json
would work only after I restarted my terminal for some reason. I'm guessing that perhaps when you create an environment to run retro, it causes it to look at a different scenario.json file?

I doubt you have to restart your terminal, but you probably have to restart python.


localeFromRequest returns null

I've tried to configure route specific uri localization following guide.
Default locale: en
Entry in app/config/app.php:
    'Storage' => Illuminate\Support\Facades\Storage::class,
    'UriLocalizer' => Waavi\Translation\Facades\UriLocalizer::class,
    'URL' => Illuminate\Support\Facades\URL::class,

Entry in routes/web.php:
Route::group(['prefix' => \UriLocalizer::localeFromRequest(), 'middleware' => 'localize'], function () {
    dd(UriLocalizer::localeFromRequest());

dd returns null, and if I to access website, I'm redirected to /en/* path, and see 404
php artisan route:list do not show any prefix for routes.
Somehow locale prefix is not registered with routes.

Update:
There seam to be a bug in LanguageRepository in this code:
public function availableLocales()
    {
        if ($this->config->has('translator.locales')) {
            return $this->config->get('translator.locales');
        }
        if ($this->config->get('translator.source') !== 'files') {

Together with default config template:
    // In case the files source is selected, please enter here the supported locales for your app.
    // Ex: ['en', 'es', 'fr']
    'available_locales' => ['en'],

available_locales vs locales

Pull request with fix in #110
(I'm soooo bad at github, sorry)

Update 2:
While adjusting that config entry helped with routes, there are still more bugs. I've read locale middleware source code. It use Language model to find current locale in DB, even if selected mode is files! But tables in db are empty, as I have not yet imported db.
Here is my theory:
There is implicit assumption in code that translations will be loaded to db, even for files mode.


Clean up alignment scores and FeatureListUtils

harmonized ways to get feature candidates within ranges and the calclation of the alginment score and the AlignmentScores analysis

Scores from LC-MS and then imaging to LC alignment are merged
Need to check that the scores and sorting are correct.
Earlier the sorting in the Image aligner was reversed but the default sorting of RowVsRowScore is already large - low
The alignment scores are just moved.
Merge after #1021

Let me know when you checked

Need to check that the scores and sorting are correct.

All done.
Seems to perform the same way.
Also added a test to MZmine master to check the number of aligned features in the small feature finding test


Issue: Not getting good chat  results on enabling Coversation Memory in langchain

Issue you'd like to raise.
Hi,
Here is how i have initilized conversation memory
memory = ConversationSummaryBufferMemory(llm=llm_model,memory_key="chat_history",return_messages=True,max_token_limit=500)
Here is how I have used ConversationalRetrievalChain
chain=ConversationalRetrievalChain.from_llm(llm_model,retriever=vector.as_retriever(search_kwargs={"k": 10}), memory=memory,verbose=True)
I could see that answer for first question will be good, on asking further questions with
result = chain({"question": }) answer from bot is not good for few questions. With verbose enabled I have observed that apart from the actual history langchain is adding  few more conversations by itself. Is there a way to supress this(adding additional conversation)
I also tried with ConversationBufferWindowMemory aswell. There also Iam seeing the performance drop. I tried with new langchain version as well "0.0.235", there also Iam seeing the same issue. Could you help me to undersatnd if this is an existing issue or any mistake am I doing while configuring
Suggestion:
No response

Thanks a lot for your  response,
As you told, Since the questions are independent of each other, may be the chat response performance might be deteriorating
Regrading temperature, I have already set to 0 to avoid radomness, Also I tried to use from langchain.tokenizer import TikTok
But Iam getting "ModuleNotFoundError: No module named 'langchain.tokenizer'" error, I tried with 0.0.235 version also, Also I have installed tiktoken as well. Is there any thing extra needs to be installed? Also is there any contorl that we can put on standalone question that it generates?

@dkiran1 What model are you using?


Weird lateral ramp logger plot?

Robot Setup: Mecanum Drive with Three Deadwheeel Localizer
Beacuse of the difference between (kS_forward, kV_forward) and (kS_lateral, kV_lateral), the ramp logger regression plot is not ideal. Any suggestion on that?

This plot exposes a limitation of the robot model. The additional friction dynamics when strafing cannot be captured by a single multiplier but require an additional offset at least. On the bright side, this was just as broken in previous versions of Road Runner, and feedback can make up some of the difference.


Wait for elements to not exist (Selenium::WebDriver::Error::StaleElementReferenceError: stale element reference: element is not attached to the page document)

Hi,
I have a collection of elements defined as  elements :menu_items, ".class_identifier_for_each_elem".
Now, I need to assert something after all menu items disappear.
On trying to use page.wait_until_menu_items_invisible, I am getting the following error:
Selenium::WebDriver::Error::StaleElementReferenceError:
stale element reference: element is not attached to the page document
I am assuming this is due to the fact that the menu items no more exist on the page, and hence the matcher is falling over trying to check for their visibility.
While site_prism provides non-existence matchers at a given point, I couldn't find anything to wait for non-existence of element(s). Do we have something in site_prism to do this?

Maybe try the has_no methods:
expect(@page).to have_no_menu_items

Thanks @tgaff. Actually I need this within my page object, hence don't want to assert.
Actually wait_until_menu_items_invisible works, but if I move the browser window, it fails with the said error. Bizarre !!

Thanks tgaff. I need this within my page object, hence don't want to put an assert.
Interestingly, if I register a single :menu_item with it's css class and use "wait_until_menu_item_invisible", I am able to work through it. But, if I try to wait for all such elements defined as sections, it fails with the said error.

Any updates on this, please. This is further blocking me for sections too.
I have sections defined for a repeating section. Ex:
sections :test_sections, SectionItem,
Now I want to wait for a particular section to be present (not all/any section)
wait_for_test_sections[5] # not supported.
wait_for_test_sections (will be true if even a single section exists)

@xam7247 Is the element you're checking for nested inside a section that is also removed from the DOM?  I've seen issues with Capybara not handling nested element queries well when an ancestral element is removed or updated in the DOM.

Regarding your other question, you can use Capybara's Query parameters to accomplish what you are trying to do.  See https://github.com/natritmeyer/site_prism#using-capybara-query-options

Still having this issue.

Hi there. I'm going to be looking into the entire history list of issues e.t.c. and this one appears to be stale. Could you give a brief update as to what you plan to do with this.
If you plan to land a PR soon or already have done. Can you close this issue
If you think it's no longer relevant. Can you close this issue.
If possibly can you provide some update in the next couple of weeks (I'm aware this issue is several years old)

I'm experiencing this in an assertion:
expect(q_form.questions_list).to have_content multiple_choice_question[:text]
def questions_list
    added_questions.map { |item| item.title.text }
  end

  sections :added_questions, :xpath, "//rt-open-questions-item" do
    element :title, :xpath, ".//div[@class='item-title']"
    element :options, "rt-open-questions-item div.child-items"
    element :edit_btn, :xpath, ".//*[*[rt-icon[@name='edit']]]"
    element :delete_btn, :xpath, ".//*[*[rt-icon[@name='delete']]]"
  end

error:
Selenium::WebDriver::Error::StaleElementReferenceError:
       stale element reference: element is not attached to the page document
         (Session info: chrome=66.0.3359.181)
         (Driver info: chromedriver=2.37.544337 (8c0344a12e552148c185f7d5117db1f28d6c9e85),platform=Mac OS X 10.13.4 x86_64)

This is a separate issue and ideally if you're having problems you should re-open one.
You are asserting an array of strins against an object, which will fail as it doesn't make sense.
Try using all matchers, or an inclusion matcher


Replace old Google+ links

Replace all the obsolete links, including the Google+ and possibly more.
Not sure yet what to replace them with...

I looked at the links that are inside the project. There are no links to Google+. The other links that are available to the user do not need to be replaced

Hmm, in that case I guess only the strings need to be updated, at least the "Lightning Launcher on Google+" from the main lighting settings.
I remember a report in the forum asking to replace some links, and last time I checked I think I spotted a couple that should be. Let me review them myself, perhaps I'm misremembering.


release (prod): config update for better Bluetooth advertisement

Issue

Link: https://github.com/NebraLtd/hm-config/issues/104
Summary: Bluetooth add was only registering once and couldn't be retriggered for 5 minutes

How

Bump config container. Also bumps diagnostics, miner, packet forwarder to fix helium OG support and adds balena.yml files for balena OpenFleets
Pushed to testnet 16.00 GMT 9th March 2022
https://github.com/NebraLtd/helium-miner-software/actions/runs/1958324975
Screenshots

References

Closes: https://github.com/NebraLtd/hm-config/issues/104
https://github.com/NebraLtd/hm-config/pull/174
https://github.com/NebraLtd/helium-miner-software/pull/391
https://github.com/NebraLtd/helium-miner-software/issues/165
Checklist

[x] Tests added
[x] Cleaned up commit history (rebase!)
[x] Documentation added
[x] Thought about variable and method names

@KevinWassermann94 this adds on the changes to improve the Bluetooth you mentioned, obviously needs testing thoroughly on testnet

@NebraLtd/tech-support ?


fix link to default definitions

Small update to fix default definitions link on the doc page.

Codecov Report

Merging #125 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #125   +/-   ##
======================================
  Coverage    68.5%   68.5%
======================================
  Files          19      19
  Lines         800     800
  Branches      127     127
======================================
  Hits          548     548
  Misses        252     252

Continue to review full report at Codecov.

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


Cannot rename a document

Description
After update from 1.5.0 to 1.5.2 I am not able to rename a document. No error message shown.
Expected
Document renamed and new file name displayed in breadcrumb navigation.
Actual
Nothing after rename dialog closes.
Info:

OS: Linuxserver Docker image on Synology NAS
FF 83.0
Database Linuxserver Docker image provides
Papermerge Version 1.5.2

I am aware that I should not post issues for the Linuxserver docker image. And your demo installation works just fine. So maybe I have a problem with older documents being upgraded - maybe some permissions problems.
My intention with this issue is to ask whether there is a logfile that may show further info to help me investigate this issue?

@okoetter, thanks for quick heads-up.
Yes, indeed there is a problem. In hurry I messed it up.
You can confirm that we speak about same problem. If you browser is firefox or chrome, when renaming a document you will see "file_orig" field is required message.

To mitigate XSS vulnerabilities I added additional checks. However, I forget to remove legacy document fields on document model. Bad, Eugen!

I will fix in asap. Thanks for heads-up again!

@okoetter , I just pushed 1.5.3 which should fix this issue.
Could you please test against 1.5.3 ?

I can confirm that it works with the new version, thank you!
But I still get 1.5.2 shown at the bottom right, did you forget to bump the version number?

BTW: I just wanted to rename a file with a date in the format "10/2020" in the name. That did not work, json result was "Enter only safe characters.". I then used "10-2020" and that worked. Maybe the error message should be shown to the user!

@ciur
Escaping all user input properly should mitigate XSS attacks.  When I fixed that XSS thingy, I thought - who would need a slash in their file name :question: :grey_question: :question:
I will fix format "10/2020" in the name. That did not work, json result was "Enter only safe characters." problem by removing regexp validation as @l4rm4nd mentioned.
But won't rush this time for the next quick 1.5.4 release. I wait for other bugs to pile up, fix them, test them properly, and then release 1.5.4 towards end of December 2020.
@l4rm4nd, @okoetter thank you guys for your contributions!
Feels great to have onboard a security guy and an external user of the system! Like a real company!

fixed in 1.5.4


Change Options button on select part from registry page to Login

This button should allow the user to login to the selected registry.

This button brings up the Registry preferences page, which allows the user to log in.
I could add a separate login button, but this doesn't really seem necessary.  Also, logging in is only applicable to SynBioHub instances, so it doesn't make sense to always have it there.
Could you explain why you want this change?

This is part of the larger issue of having separate SBOLDesigner and iBioSim preferences when used as a plugin.  We could perhaps leave the Options button but it would need to bring up the iBioSim preferences when used in that context.  Please discuss with @tramyn.

Ah, I see.  I can make a static Functor in RegistryInputDialog that SBOLDesignerPlugin can set to be iBioSim's preferences dialog.  The options button can call this Functor, and the default implementation can be SBOLDesigner's preferences.  This is definitely hacky, but will allow us to not have SBOLDesigner depend on iBioSim.  @tramyn , what do you think?

We need to ensure also that it is this preference you are using when in plugin as I noticed students having problems with setting preferences in iBioSim but your preferences getting used and throwing errors when not setup properly.

Unfortunately, I don't think there is a clean way to do this.  SBOLDesigner should definitely not depend on iBioSim.  The workaround would have to be passing in a function pointer that overrides behavior.  However, I'm not even sure this is possible in Java since SBOLDesigner will have to call a different preferences dialog dynamically.

Maybe what we need to do is have iBioSim's preference dialog use and update SBOLDesigner's preferences.  This does mean whether you use in SBOLDesigner or iBioSim, you would have the same preferences, but this seems like it might actually be better.  Could you fix this in iBioSim?  The code is basically the same.  It is in iBioSim-gui, in util/PartEditorPreferencesTab.java

I followed the installation steps on iBioSim's README.  However, I'm getting:
Missing artifact uk.ac.ncl.intbio:vpr-rdf-util:jar:2.0.2-SNAPSHOT
Missing artifact uk.ac.ncl.intbio:vpr-rdf-util:jar:2.0.2-SNAPSHOT
Missing artifact uk.ac.ncl.ico2s:vpr-data:jar:2.0.2-SNAPSHOT
Running the maven steps also messed up my SBOLDesigner build.  Specifically, generate sources doesn't seem to be actually resolving the dependencies.  I've removed BOOSTDialog from the classpath since that's the culprit, and it isn't even being used.

These are tramy’s libraries. Please put issue on ibiosim tracker. Try mvn clean and mvn install if you haven’t already. Otherwise see tramy.
Chris
Sent from my iPhone

On Oct 15, 2017, at 8:52 PM, Michael Zhang<EMAIL_ADDRESS>wrote:
I followed the installation steps on iBioSim's README. However, I'm getting:
Missing artifact uk.ac.ncl.intbio:vpr-rdf-util:jar:2.0.2-SNAPSHOT
Missing artifact uk.ac.ncl.intbio:vpr-rdf-util:jar:2.0.2-SNAPSHOT
Missing artifact uk.ac.ncl.ico2s:vpr-data:jar:2.0.2-SNAPSHOT
Running the maven steps also messed up my SBOLDesigner build. Specifically, generate sources doesn't seem to be actually resolving the dependencies. I've removed BOOSTDialog from the classpath since that's the culprit, and it isn't even being used.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@michael13162 : If you are trying to access iBioSim from the frontend, make sure you:

check out the latest updates from SBOLDesigner and libSBOLj. Perform mvn install on each project directory.
navigate to iBioSim's gui module and perform mvn clean then mvn install.

If these steps fail to build, let me know and we can debug this together.
As for the other discussion mentioned in this issue, I would prefer it if iBioSim has its own preferences and likewise for SBOLDesigner. Any information that iBioSim needs from SBOLDesigner in regards to registries should be obtained from the backend of SBOLDesigner. Is there a reason as to why we should be modifying SBOLDesigner's UI?

I also agree with @tramyn that it would make more sense for the Registry preferences to just live in one place (in SBOLDesigner).  Duplicating all the code in iBioSim is more messy, and keep track of changes/plumbing to get everything to work together properly is difficult.
iBioSim could just access the same preferences node when it needs the information.  This is definitely cleaner.

I agree, but I struggled with this.  I was not able to disentangle the dependencies.  Namely, it would be good to have the PartEditorPreferencesTab be housed in SBOLDesigner, but it implements PreferenceTab, so I assume this must also be in SBOLDesigner.  However, for some reason I ended up needing to duplicate it in iBioSim.  If you two can figure out how to just have the SBOLDesigner preferences tab show up in the iBioSim's preferences that would be great.

The Options button has been replaced by Login button.  The Login button is also sensitive to whether a loginable registry is selected.  I was having strange issues have both login and options, so now it's just login.  The old implementation for options has been commented out.  Login also refreshes the table since new parts are now potentially available.


Remove submodules

We no longer rebuild FCOS using cosa, so we no longer need fedora-coreos submodule.
From openshift-os submodule only gcp-routes is needed - it can be extracted from openshift-os (rhel-8-coreos in the payload).

/retest

/override ci/prow/e2e-gcp-upgrade
Previous build could not be installed
/test e2e-aws-ovn

/retest

@vrutkovs where does 20-client-id-from-mac.conf come from?

Its part of FCOS now - https://github.com/coreos/fedora-coreos-config/blob/testing-devel/overlay.d/05core/usr/lib/NetworkManager/conf.d/20-client-id-from-mac.conf


python310Packages.nestedtext: 3.5 -> 3.6

Diff: https://github.com/KenKundert/nestedtext/compare/refs/tags/v3.5...v3.6
Changelog: https://github.com/KenKundert/nestedtext/blob/v3.6/doc/releases.rst
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 235845 run on x86_64-linux 1

16 packages built:

emborg (python310Packages.emborg)
emborg.dist (python310Packages.emborg.dist)
python310Packages.nestedtext
python310Packages.nestedtext.dist
python310Packages.parametrize-from-file
python310Packages.parametrize-from-file.dist
python310Packages.quantiphy
python310Packages.quantiphy.dist
python311Packages.emborg
python311Packages.emborg.dist
python311Packages.nestedtext
python311Packages.nestedtext.dist
python311Packages.parametrize-from-file
python311Packages.parametrize-from-file.dist
python311Packages.quantiphy
python311Packages.quantiphy.dist


OpenGL extensions

I'm wondering is it possible to check extensions capabilities? Especially I want to implement anisotropic filtering and don't know how to check if it is supported. I'm using glium for windows handling.

You're using glutin for window handling, glium is a GL wrapper. You need to use glGetString. For an example, see https://github.com/gfx-rs/gfx/blob/master/src/backend/gl/src/info.rs
For questions like this, ask in #rust-gamedev on IRC or google for generic GL information, there's nothing special specific to this library.

I think it is a reasonable question to ask @cmr. At least now we have a record to point people towards in the future. Thanks for answering though!

It's a reasonable question to ask for sure! Issues, though, are not ideal IMO.

 It's hard to communicate asynchronously outside of issues.

Thank you for the help :)


Update reactive microfrontends JDL for v8

Fixes #51.
This probably shouldn't be merged until v8 is released since I don't think it'll work with v7.

We should use v7.10.0 branch for v8 now.


In master, has the support for DataFrames.DataFrame(::Array{Int64,1}, ::Array{Symbol,1}) been dropped?

I use a by() function to generate a DataFrame
DataFrame(frequencies, colnms)
which generates the following error message:
LoadError: MethodError: no method matching DataFrames.DataFrame(::Array{Int64,1}, ::Array{Symbol,1})
This syntax used to be supported. Is there any plan to bring it back?

Could you give a reproducible example of that syntax? Do you have a precise reference of where is was documented or when it worked?

The following function works with DataFrames + DataArrays.
function xtab_freq(df::DataFrame,column1,column2)
function frequency(data,factors)

    # convert factors to symbols
    factor_symbols::Vector{Symbol} = map(factor -> Symbol(factor),

factors)
    # Convert frequency to fit the DataFrame constructor parameter type
    frequencies::Vector{Any} = map(frequency->[frequency],map(x ->

isna(x) ? sum(isna(data)) : sum(dropna(data) .== x), factors))
    DataFrame(frequencies, factor_symbols)
end

factors = sort(unique(df[column2]))

if length(factors) > 0
return by(df, column1, x -> frequency(x[column2], factors))
end
end
See the 11th line, DataFrame(frequencies, factor_symbols).
Thanks
Min
On Fri, Oct 7, 2016 at 4:41 AM, Milan Bouchet-Valat <
<EMAIL_ADDRESS>wrote:

Could you give a reproducible example of that syntax? Do you have a
precise reference of where is was documented or when it worked?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/JuliaStats/DataFrames.jl/issues/1096#issuecomment-252184630,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJrQF5MP7hWAjfvovEwRIWCBSgUCpjxSks5qxgWjgaJpZM4KQXr5
.

It works here both with 0.8.3 and master, on Julia 0.4 and 0.5. Minimal reproducer: DataFrame(Any[[1],[2]], [:a, :b]).
Can you show the exact error you get?
BTW, if you need frequency tables, you can also use https://github.com/nalimilan/FreqTables.jl

You might try
DataFrame(hcat([Date(2016,9,1), Date(2016,9,2)], rand(2,1)), [:Date, :Value])


[Teams] Add teamsTenantFilteringMiddleware to roster bot scenario

Pull Request Test Coverage Report for Build 83582

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

Totals

Change from base Build 83484:
0.0%

Covered Lines:
4840

Relevant Lines:
5306

💛  - Coveralls


Included resources

Hi Toby, thank you for the nice tool.
When I add a relationship, "id" and "type" is present. This works fine. I just didn't figure out how to add the resources as "included" data (see the example at http://jsonapi.org/, people, comments are included).
Is there a way to do that?

Sorry, it works perfect.

You worked it out?

It was my mistake in the serializer that I used for the relationship.


rpnet: update IP network

fog-hyper configures internal network in MC in the same network <IP_ADDRESS> and communcation between MC and FC does not work,

Solved Problem
When ... I found that ...
Fixes #{Github issue ID}
Solution

Add ... for ...
Refactor ...

Alternatives
We could also ...
Test coverage

Unit/integration test: ...
Simulation/hardware testing logs: https://review.px4.io/

Context
Related links, screenshot before/after, video

Looks good!


Capitals in Runecrafting

🐛 Bug Report
When runecrafting with "+rc" if the rune is typed out in all caps or anything other than first letter capitalized it will fail. Other things like bosses don't seem to mind as much, but +RC NATURES will fail saying it's not a valid rune that you can craft.

Clarification on what works....
+rc Natures - Works
+rc natures - works
+rc Nature - works
+rc nature - works
+rc NATURE - works
+rc NATURES - doesn't work

merged


Now using golang:1.6.2 as a base for docker images

Hi there.
As stated in #200 , Go 1.6 may fix some issues. This PR changes the base image in Dockerfiles to switch to golang:1.6.2.
Please note that golang image is based on jessie-scm, so your existing Dockerfiles were mainly compatible with it. GOPATH is /go on golang image, so i had to change this, and i also removed some (apparently ?) unused ppa repositories which would have not been compatible with jessie-scm anyway.
Disclaimer : i tested Dockerfile on our production cluster, works great. But i wasn't able to test Dockerfile-deb. Only a few minor changes, so i assume it won't break anything, but i would feel it safer if someone who uses it everyday could confirm that.
Thanks in adv. !

By analyzing the blame information on this pull request, we identified @bateau84, @sttts and @rthomas to be potential reviewers

Coverage increased (+3.3%) to 39.154% when pulling 23f2741d033c616b2830e21d4bb50eb542fd2104 on mvallerie:master into 01da91b0530ac0a7d2ca4fb4af79e161c25753ea on QubitProducts:master.

LGTM.
Sent from my iPhone

On 13 May 2016, at 04:33, Coveralls<EMAIL_ADDRESS>wrote:
Coverage increased (+3.3%) to 39.154% when pulling 23f2741 on mvallerie:master into 01da91b on QubitProducts:master.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub


Editing Github pages

Make changes to the default Jekyll theme provided.
Should explain about the project.

Done for now.
Will update again once the project is complete.


Evaluate in-database prediction for trained Auto-ML models

Background

SageMaker autopilot stores the models in S3 and they look like we might be able to run them in the UDFs
We need to test if we can run some in python outside the DB and inside
Check if different models and preprocessing steps have somewhat consistent interface
How do find the best candidate?

Shelved.


feat(multiple_testing_power): support multiple metrics and cohort sample size calculation

To help us get this pull request reviewed and merged quickly, please be sure to include the following items:

[x] Tests (if applicable)
[x] Documentation (if applicable)
[ ] A full explanation here in the PR description of the work done and statistical methods used

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Tests
[ ] Other

Backward Compatibility
Is this change backward compatible with the most recently released version? Does it introduce changes which might change the user experience in any way? Does it alter the API in any way?

[ ] Yes (backward compatible)
[x] No (breaking changes)

This pull request adds the feature to calculate minimum required sample size for multiple decision metrics and/or multiple variants by combining the changes from https://github.com/godaddy/sample-size/pull/26, https://github.com/godaddy/sample-size/pull/30, https://github.com/godaddy/sample-size/pull/33, https://github.com/godaddy/sample-size/pull/32. The statistical method used is elaborated in this PDR, and the test statistics's simulation method is validated in Validation of test statistics simulation.md.

@vjohnson1-godaddy we can bump it when we make the release PR: https://github.com/godaddy/sample-size/blob/main/RELEASE.md#1-create-a-release-pr

probably good to remember this is a public repo and we should not link internal stuff here. although that PDR seems to be fine without any confidential thing. (just a reminder)

Can you bump the version?

As @pchiang-godaddy said, I bumped the version in release PR https://github.com/godaddy/sample-size/pull/37

probably good to remember this is a public repo and we should not link internal stuff here. although that PDR seems to be fine without any confidential thing. (just a reminder)

probably good to remember this is a public repo and we should not link internal stuff here. although that PDR seems to be fine without any confidential thing. (just a reminder)

Thanks for reminding. I removed the link just in case, and we can share the content in that PDR later if we get any inquiries about the statistical methods we used.


remove unbound type parameters

Unbound type parameters often cause performance issues and run time dispatch.
Issue found using https://github.com/JuliaLang/julia/pull/46608

@nsajko Thanks for this contribution.


feat(createfullpage): initialstep added

Contributes to #2624
InitialStep prop added to CreateFullPage component
What did you change?
Prop added, useEffect updated, utilities added
How did you test and verify your work?
storybook, Jest

accidently added PR. Hence closing.


llvmPackages_*.llvm.dev: "include/llvm/Config/config.h" is missing

File "include/llvm/Config/config.h" should be among LLVM headers but it is missing.

"include/llvm/Config/llvm-config.h" is not a replacement
In fact, "include/llvm/Config/config.h" does #include "llvm-config.h"

It is explicitly excluded from install process in LLVM's CMakeLists.txt PATTERN "config.h" EXCLUDE
But, as we break LLVM monorepo into smaller projects, the notion on "install" is changed:
The "config.h" is needed to build some in-monorepo projects (for example, some revisions of lld)
This is before-install in monorepo approach, but after-install in NixOS approach.


note that the project is not maintained

Adds a note to the README, and a template for issues and PRs that looks like this:

@Phoenixmatrix @geekjuice @FranDias

:shipit:

🚢


Update whats-new.md

Minor grammatical/technical correction

@Vikram1988
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Compile Time Errors for GetCachedSize() and _internal_symbol()

Bug/Solution:
The following two methods provide errors about lack of conversions when building: inline const std::string& Stock::_internal_symbol() const and int GetCachedSize() const final. I had to go into the files and add .Get() to the variables in the functions for both to compile correctly. This is the first time I am seeing the issue.
The first has the same name as a field I defined. It is a string field.
What version of gRPC and what language are you using?
v1.47.1
What operating system (Linux, Windows,...) and version?
Windows 11 (latest as of 07/18/2022)
Ubuntu 22.04 LTS (latest as of 07/18/2022)
What runtime / compiler are you using (e.g. python version or version of gcc)
Windows - Latest msvc available with Visual Studio
Ubuntu - gcc (Ubuntu 11.2.0-19ubuntu1) 11.2.0
What did you do?
Cloned the latest version of gRPC (1.47.1).
CMake installation
Ran the following command: protoc -I=. --grpc_out=Generated --plugin=protoc-gen-grpc=[absolute_path_to_my_grpc_cpp_plugin_under_the_build_folder] --cpp_out=Generated AProtoFileWithGrpcStuff.proto
Compiled source code that used said .proto file with server code implemented.
Please provide either 1) A unit test for reproducing the bug or 2) Specific steps for us to follow to reproduce the bug. If there’s not enough information to debug the problem, gRPC team may close the issue at their discretion. You’re welcome to re-open the issue once you have a reproduction.
What did you expect to see?
No build errors from gRPC's end similar to about 6 months ago when I last used it.
What did you see instead?
Compile-time errors.
---Insert traceback through files via the includes---
./Interface/Protobuf/Generated/Extensions.pb.h: In member function 'virtual int Rogue::Interface::CodeGenerationMarkings::GetCachedSize() const':
./Interface/Protobuf/Generated/Extensions.pb.h:164:44: error: cannot convert 'google::protobuf::internal::CachedSize' to 'int' in return
  164 |   int GetCachedSize() const final { return _cached_size_; }
      |                                            ^~~~~~~~~~~~~
      |                                            |
      |                                            google::protobuf::internal::CachedSize

---Insert traceback through files via the includes---
./Interface/Protobuf/Generated/Stock.pb.h: In member function 'const string& MarketData::Stock::_internal_symbol() const':
./Interface/Protobuf/Generated/Stock.pb.h:654:10: error: invalid initialization of reference of type 'const string&' {aka 'const std::__cxx11::basic_string<char>&'} from expression of type 'const google::protobuf::internal::ArenaStringPtr'
  654 |   return symbol_;
      |          ^~~~~~~

I am using custom extensions. This is the only notable difference besides the version of gRPC that I can think of since last using and building my source code.
Anything else we should know about your project / environment?
Not that I know of. There are some compile errors I am working through currently that were hidden by the above compile errors, so I will post an update if the resolution of those affect the resolution of this.

What do you mean by custom extensions? Did you mean to raise an issue in the protobuf repository?

This does seem like a protobuf issue, not a gRPC issue.  Also, you did not include your code in the bug report, so there's not enough info here to reproduce the problem.
One thing you may want to check is whether you are running protoc from the same version of the protobuf library as the headers that you're compiling against.  Protobuf does not guarantee that its generated code is compatible across versions.

I am not seeing the error anymore. gRPC and protobuf are at the same version. The reason I tagged gRPC for the issue is because it was only with the above command for generating gRPC files that I ran into this issue. The regular protoc command for C++ files worked just fine.
Now that I look back through the generated files, the issue has resolved itself. If I can figure out how to replicate or the source cause, I will reopen the issue. Maybe someone else who knows and comes across this post will know in the future.
Here is the code though (fairly vanilla gRPC) of the gRPC service data schema:
syntax = "proto3";

import "Account.proto";
import "Stock.proto";
import "Trade.proto";

package Trading;

message Response
{

}

service TradeAnalysis
{
	//All things market
	rpc sendStocks(stream Stock) returns (Response) {}

	rpc sendStockHistory(StockHistory) returns (Response) {}

	rpc sendStockHistories(stream StockHistory) returns (Response) {}

	rpc sendLiveAggregatedData(StockHistory) returns (Trades) {}

	rpc sendMarketHistory(MarketHistory) returns (Response) {}

	rpc availableStrategies(Account) returns (Strategies) {}

	//TODO: Separate the below calls for account info
	rpc validateAccount(Account) returns (Account) {}

	rpc createAccount(Account) returns(Response) {}

	rpc updateAccount(Account) returns(Response) {}

	rpc updateAccountBalance(Account) returns(Response) {}
}


Discord (UDP/WebRTC) traversal

Version
latest
Operating System
Linux
How are you running SpoofDPI?
Docker
Description
Will there be any solution for using Discord voice chat?

yeah, UDP support would be cool


Http2 Ping

Issue: https://github.com/line/armeria/issues/1263
I've been working on this for a few days. And reached to a point to see if this code makes any sense. I've tested it a bit but still needs a lot of testing and also a lot of code changes to make it look decent enough.
@trustin @ikhoon @minwoox Please let me know if this code makes any sense. Also, please feel free to suggest anything that I may need to consider.

Thank you so much for this PR by the way. :+1:

Just a thought it could be good to reference gRPC's pings for inspiration and to find any gotchas they may have experienced already.
https://github.com/grpc/grpc-java/search?utf8=✓&q=ping&type=
https://github.com/grpc/grpc-java/blob/c8a0af572c561d28f61e9fb4f6097a72d4ebe8e9/core/src/main/java/io/grpc/internal/KeepAliveManager.java
One thing I notice is they do use traffic received to control ping scheduling, while have a separate flag for whether to ping during idle - didn't look for the motivation for that flag yet.
And one gotcha-type thing I found is they only apply ping acks to flow control.
https://github.com/grpc/grpc-java/blob/9fcfb5b4f80b832d48665dc7ec2edfc15ed873dc/netty/src/main/java/io/grpc/netty/Http2ControlFrameLimitEncoder.java#L72

I will look at gRPC's implementation. And will write here on how would I proceed so that we reach consensus. Thanks for your suggestions @anuraaga @trustin 🙇

@trustin @anuraaga here are my thoughs on gRPC's PING(keepalive) implementation:

As @anuraaga  said, gRPC server uses traffic to control PING scheduling. The PING is scheduled initially and is re-scheduled where there is new data received. I think this basically is doing send ping only when idle. The default delay is 2 Hours. But for client-side this is controlled by a flag that enables/disables PINGs during idle.

And there are checks to make sure that client does not send too many pings to sever within a time limit which is a nice defensive protection. I think its good to have this feature.

Also PING acks are applied to flow control to limit number of flow control frames currently active. see here for more details.

When a PING write has failed or peer does not ACK within configured timeout, the channel is closed assuming that peer has disconnected.
Also, I feel like some implementation in gRPC may be redundant (I could be totally wrong as well) may be because specification might have changed from when it was developed or may be gRPC extended http2 protocol to achieve some use case. For ex: if the PING payload is 0x97ACEF001L (GRACEFUL_SHUTDOWN_PING)  then the channel is closed, but I could not find this in specification: https://http2.github.io/http2-spec/.
But overall I think we can send PING when there are no streams active so we maintain the liveliness of connection. Also making sure that client does not send too many ping within a time limit. Please let me know what do you think and also share any suggestions.

But overall I think we can send PING when there are no streams active so we maintain the liveliness of connection. Also making sure that client does not send too many ping within a time limit. Please let me know what do you think and also share any suggestions.

I guess we should send PINGs regardless of the number of open streams, because there may be many open yet idle streams in a connection. Did you mean 'active streams' by that, then we're on the same page. :smile:
Sounds good to me besides that!

Sorry for the verbiage. I meant active streams. Thanks for your suggestions and I resume my work.


EJS + EJS-LAYOUTS Replacement?

Is your feature request related to a problem? Please describe.
Hello, I just run into your project and it looks and sounds amazing. But I have a doubt. I am currently using EJS + express-ejs-layouts combined. Can this be just dropped in and it will replace both the things, if yes how? I understand that I'll need to change the syntax a bit as well (adding it and ~ in all the places).
Thanks!

Hi @harshmandan,
I haven't used EJS before, mainly because I like handlebar syntax more ({{}}), but I think Eta should be able to replace both of those with its partial support which I believe is what express-ejs-layouts is kind of doing.

Hi @harshmandan,
I haven't used EJS before, mainly because I like handlebar syntax more ({{}}), but I think Eta should be able to replace both of those with its partial support which I believe is what express-ejs-layouts is kind of doing.

Okay so I'll give it a try and then let you know if it works. Thanks!

Okay so I'll give it a try and then let you know if it works. Thanks!


[x86] x86 cross build on x64 host

Add rootfs support (build-rootfs.sh and
sources.list for different distributives)
some missed changed from #4637
@jkotas, @janvorli
cc @Dmitri-Botcharnikov, @BredPet, @sergign60

@alpencolt,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

cc @am11

Great, I was testing with ./build.sh x86 on x64 system. With rootfs, /build.sh cross x86 also works. 👍


Create 0000-01-02-mayankr79.md in /_posts

Added a new file. Let me know your views

Merge please


Custom File Browser as widget

Actually I am making an app of sending-receiving files in Kivy . Problem is i want to use custom file browser  as a widget . I have written a code for file browser and also for my app ..now want to use that file browser file in my actual app . Any one can help me out in this?

This is not a support group. You would have a lot more helpful replies on #kivy on irc.freenode.net or on kiva-users google group. Best of luck!

@Avernch7 https://groups.google.com/forum/#!forum/kivy-users


little fullscreen bug

From<EMAIL_ADDRESS>on 2010-08-18T10:05:23Z
What steps will reproduce the problem?
1.set the windows taskbar to auto-hide
2.set spyder to fullscreen
3.set spyder to normal view again What is the expected output? What do you see instead? The normal application window order (from before the fullscreen switch) is expected again.
Instead all other windows and even the taskbar(!!) now go below spyder! What version of the product are you using? On what operating system? win7  spider 1.3.3
Please provide any additional information below
. The situation is fixed by first switching to another application and then switch back to spider.
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=302

From pierre.raybaut on 2011-05-16T01:08:40Z
Labels: Cat-SpyderGUI

From pierre.raybaut on 2011-10-29T04:29:37Z
Please upgrade to v2.1.0.
Status: Done


Fixes #7

@aurasphere Hi! I am add a new test cases for bluetooth controller and edit readme file by adding screenshot and comments. Please review it

Thank you very much for your help! Do you mind if I add the project license to the test files (like the others)?

@aurasphere yeah, add it


Buildlib is not guessing the distro correctly on Ubuntu 16.10

If you run the master branch in Ubuntu 16.10, it will attempt to build the static version. It seems that dist_codename in __init__.py isn't returning the actual Ubuntu version, making it go for the static build.
For example, changing lines 49 and 50 to this does not make it use the XenialBuilder on Ubuntu 16.10:
        elif dist_id == "ubuntu" and (dist_codename == "yakkety"):
            from .debian import UbuntuXenialBuilder

Try launching an interactive Python 3 console in the buildlib/_external directory and let me know the output of the following:
import distro
distro.info()

The version output was "Yakkety Yak". The version output on 16.04 was "Xenial Xerus". Using the capitalized full names fixes the issue.

@9Morello Ah I see. Thanks for your input. Going to merge your PR now.


How to get iris and refined eye landmark coordinates from Mediapipe Iris C++

Have I written custom code: No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04, Android 11, iOS 14.4): Mac M1 BigSur
Bazel version: 5.3.2
Solution: Iris
Programming Language and version: C++

Describe the expected behavior:
C++ noob here. I am working on a project where I need to get the landmarks of the iris and refined eye mapping from mediapipe iris in C++. I have referred to #1177, but that does not seem to work on a static image. Everything works and I can get the output tracings which look great, but I cannot figure out how to get the landmarks from the code. I cannot share the pictures due to privacy concerns, but in essence I want to get the xyz coordinates of the analyzed image (see image) and store them in a txt file or some format which I can then use later.

My current solution is just to extract the tracings by color and regenerate them in Python after Mediapipe has finished, but I feel like there has to be a better way. Any help is appreciated! Thank you so so so much and let me know if you need any more info.

@monkeygobah,
Sorry for the late response, Would you please let us know if you are still looking for the resolution or the issue has been resolved from you end? Thank you!

Hi- sorry for my late reply as well. I did not find a resolution within mediapipe but solved the problem by changing the color of the R and L eyelid tracings and then extracting them later in python. It is not the best solution as I would rather just have the indexed points, but it does work well enough for now. If there is a way to get the real coordinates out, though, I would love to know!

Hi @gkarpiak,
Could you please look into this issue? Thank you!

Hello @monkeygobah,
We are upgrading the MediaPipe Legacy Solutions to new MediaPipe solutions However, the libraries, documentation, and source code for all the MediaPipe Legacy Solutions will continue to be available in our GitHub repository and through library distribution services, such as Maven and NPM.
You can continue to use those legacy solutions in your applications if you choose. Though, we would request you to check new MediaPipe solutions which can help you more easily build and customize ML solutions for your applications. These new solutions will provide a superset of capabilities available in the legacy solutions. Thank you


Spinnable text

Serves absolutely no function and is almost certain against WCAG but it's fun
Looks at old portfolio for existing code, jazz it up

Yeah no - FUNdermentally bad idea


[GOBBLIN-1319] fix cancellation in gobblin cluster jobs

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
@yukuai518  please review
JIRA

[x] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-1319

Description

[x] Here are some details about my PR, including screenshots (if applicable):
jobsMapping in GobblinHelixJobScheduler was using job name to store the mapping of job to planning/actual job. this pr is changing it to use job uri instead, because cancellation request just have a uri
provide a clean separate path for cancellation on cluster side, so that 'delete' calls from within the cluster and from gobblin service can be treated differently

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:
verified with integration test

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Codecov Report

Merging #3155 (d9d31d8) into master (7352cad) will decrease coverage by 0.01%.
The diff coverage is 26.31%.

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