Installing Jest with NPM Breaks Start Script

🐛 Bug Report
After installing jest via npm install --save-dev jest, npm start breaks with the following message:
There might be a problem with the project dependency tree.
It is likely not a bug in Create React App, but something you need to fix locally.

The react-scripts package provided by Create React App requires a dependency:

  "jest": "23.6.0"

Don't try to install it manually: your package manager does it automatically.
However, a different version of jest was detected higher up in the tree:

  /mnt/c/Users/matthew.hyrka/ecompli-dashboard/node_modules/jest (version: 24.8.0)

Manually installing incompatible versions is known to cause hard-to-debug issues.

If you would prefer to ignore this check, add SKIP_PREFLIGHT_CHECK=true to an .env file in your project.
That will permanently disable this message but you might encounter other issues.

To fix the dependency tree, try following the steps below in the exact order:

  1. Delete package-lock.json (not package.json!) and/or yarn.lock in your project folder.
  2. Delete node_modules in your project folder.
  3. Remove "jest" from dependencies and/or devDependencies in the package.json file in your project folder.
  4. Run npm install or yarn, depending on the package manager you use.

In most cases, this should be enough to fix the problem.
If this has not helped, there are a few other things you can try:

  5. If you used npm, install yarn (http://yarnpkg.com/) and repeat the above steps with it instead.
     This may help because npm has known issues with package hoisting which may get resolved in future versions.

  6. Check if /mnt/c/Users/matthew.hyrka/ecompli-dashboard/node_modules/jest is outside your project directory.
     For example, you might have accidentally installed something in your home folder.

  7. Try running npm ls jest in your project folder.
     This will tell you which other package (apart from the expected react-scripts) installed jest.

If nothing else helps, add SKIP_PREFLIGHT_CHECK=true to an .env file in your project.
That would permanently disable this preflight check in case you want to proceed anyway.

P.S. We know this message is long but please read the steps above :-) We hope you find them helpful!

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: `REACT_APP_FORCE_DEVELOPMENT=Y REACT_APP_API_URL=http://ecompli.localwww/ecompli4/api/ react-scripts start`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/mhyrka/.npm/_logs/2019-05-10T19_36_00_134Z-debug.log

To Reproduce
Steps to reproduce the behavior:
Here is my package.json.
{
  "version": "0.1.0",
  "private": true,
  "homepage": "./",
  "dependencies": {
    "@material-ui/core": "^3.9.2",
    "@material-ui/icons": "^3.0.2",
    "axios": "0.18.0",
    "browserslist": "^4.5.4",
    "classnames": "^2.2.6",
    "core-js": "^3.0.1",
    "dotenv": "^7.0.0",
    "downshift": "^3.1.5",
    "es6-shim": "^0.35.5",
    "keycode": "^2.2.0",
    "prop-types": "^15.6.2",
    "react": "^16.4.1",
    "react-app-polyfill": "^0.2.2",
    "react-cookie": "2.2.0",
    "react-dom": "^16.4.1",
    "react-media": "^1.8.0",
    "react-scripts": "^2.1.8",
    "react-toastify": "^4.5.2",
    "screenfull": "^4.0.0",
    "typeface-roboto": "0.0.54",
    "victory": "^32.0.2"
  },
  "scripts": {
    "start": "REACT_APP_FORCE_DEVELOPMENT=Y REACT_APP_API_URL=MY_URL react-scripts start",
    "build-staging": "REACT_APP_FORCE_DEVELOPMENT=Y REACT_APP_API_URL=MY_URL react-scripts build",
    "build": "REACT_APP_API_URL=MY_URL react-scripts build",
    "eject": "react-scripts eject",
    "generate-tasks": "node generate-dummy-tasks.js",
    "test": "jest"
  },
  "devDependencies": {
    "faker": "^4.1.0",
    "fs": "0.0.1-security",
    "jest": "^24.8.0"
  },
}

As stated, I installed jest using npm install --save-dev jest . It looks like that created a duplicate babel-jest entry in my package-lock.json with conflicting versions.
        "babel-jest": {
          "version": "24.8.0",
          "resolved": "https://registry.npmjs.org/babel-jest/-/babel-jest-24.8.0.tgz",
          "integrity": "sha512-+5/kaZt4I9efoXzPlZASyK/lN9qdRKmmUav9smVc0ruPQD7IsfucQ87gpOE8mn2jbDuS6M/YOW6n3v9ZoIfgnw==",
          "dev": true,
          "requires": {
            "@jest/transform": "^24.8.0",
            "@jest/types": "^24.8.0",
            "@types/babel__core": "^7.1.0",
            "babel-plugin-istanbul": "^5.1.0",
            "babel-preset-jest": "^24.6.0",
            "chalk": "^2.4.2",
            "slash": "^2.0.0"
          },
          "dependencies": {
            "chalk": {
              "version": "2.4.2",
              "resolved": "https://registry.npmjs.org/chalk/-/chalk-2.4.2.tgz",
              "integrity": "sha512-Mti+f9lpJNcwF4tWV8/OrTTtF1gZi+f8FqlyAdouralcFWFQWF2+NgCHShjkCb+IFBLq9buZwE1xckQU4peSuQ==",
              "dev": true,
              "requires": {
                "ansi-styles": "^3.2.1",
                "escape-string-regexp": "^1.0.5",
                "supports-color": "^5.3.0"
              }
            }
          }
        }

&&
    "babel-jest": {
      "version": "23.6.0",
      "resolved": "https://registry.npmjs.org/babel-jest/-/babel-jest-23.6.0.tgz",
      "integrity": "sha512-lqKGG6LYXYu+DQh/slrQ8nxXQkEkhugdXsU6St7GmhVS7Ilc/22ArwqXNJrf0QaOBjZB0360qZMwXqDYQHXaew==",
      "requires": {
        "babel-plugin-istanbul": "^4.1.6",
        "babel-preset-jest": "^23.2.0"
      }
    },

I've gone through the steps detailed in the error message, but then I don't have jest anymore and its back to square one.
Expected behavior
Link to repl or repo (highly encouraged)
Please provide either a repl.it demo or a minimal repository on GitHub.
Issues without a reproduction link are likely to stall.
Run npx envinfo --preset jest
Paste the results here:
$ npx envinfo --preset jest
npx: installed 1 in 3.236s

  System:
    OS: Windows 10
    CPU: (8) x64 Intel(R) Core(TM) i7-8650U CPU @ 1.90GHz
  Binaries:
    Node: 10.15.3 - C:\Program Files\nodejs\node.EXE
    npm: 6.4.1 - C:\Program Files\nodejs\npm.CMD

CRA comes with jest out of the box, you don't need to install it yourself. So just remove all mention of Jest in your package.json's devDependencies, and you should be good
CRA@3 comes with jest 24


Updates to the README

Signed-off-by: Thomas Graf<EMAIL_ADDRESS>

This pull request is now managed with CodeReviewHub. Refresh.


Mid-term vision

What do you guys think about how we should continue with the "zeebe modeler" mid-term.
I see two general directions in which this could go

we keep the modeler in a separate repository and maintain it as a fork of the original Camunda modeler
we have a single Camunda Modeler targeting both Camunda BPM but also Zeebe. The user can then select a "Profile" for the execution platform they are targeting which switches the properties panel and maybe the palette.

I also want to pull in @philippfromme and @nikku for this discussion

Think we should also create a zeebe-bpmn-moddle like that one for camunda
The properties provider should also be placed in the bpmn-js-properties-panel
Not sure which option would be the best...
Second one sounds a bit more comfortable (updates, fixes...)

We discussed this internally and this is our results:
Short term it probably makes sense to go with the fork to keep the friction between the two projects as low as possible.
As zeebe matures and becomes a fully functional, awesome part of the Camunda family we would like to support it out of the box in the Camunda Modeler. How to do that in the best way is up for decision :wink:.

Hi Nico, thank you for the input, that makes a lot of sense.

I am closing this for the moment


LDR environment maps without PMREM do not light nonmetallic, rough objects at all.

Description
All of our lighting for objects like this (metal = 0, roughness = 1 is the default for unspecified materials, like the Astronaut) comes from our stage light; the environment is completely ignored for jpgs. It has very little effect on shiny objects (they are still much too dark), but with metal = 1, everything is fine.
Live Demo
The duck, when the stage light turns off: https://googlewebcomponents.github.io/model-viewer/examples/lighting-and-environment.html

Browser Affected

[x] Chrome
[ ] Edge
[ ] Firefox
[ ] Helios
[ ] IE
[ ] Safari

OS

[x] Android
[ ] iOS
[x] Linux
[ ] MacOS
[ ] Windows

Versions

model-viewer: v0.3.1
three.js: r104

@mrdoob I remember talking to you about something like this at some point. Do you have any thoughts on this?


Add per-address attribute mechanism that does not affect subchannel uniqueness.

This will allow encoding per-address parameters like xDS-supplied endpoint weights that may be used by LB policies but that are not used by the subchannel or taken into account when determining the key of the subchannel pool.
@ejona86 @dfawley as an FYI.  The impetus for this is a user that wants to implement xDS-supplied endpoint weights.

The "Artifact Build Linux" failure is a transient problem fetching an external dependency.


Use blockcheck in docs and the build

Blockcheck automates maintaining consistency between docs and files
in the repository.  This allows scripts/tutorial_test to avoid
code rot, and could be used more for similar purposes.
 Last update 2fd44dd...389fd2f

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 "blockcheck" git@github.com:wfarner/infrakit.git somewhere
$ cd somewhere
$ git rebase -i HEAD~2
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

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


Fix duplicate text in overview page

Description
before: Red Hat Advanced Cluster Security for Kubernetes RHACS Cloud Service lowers operational costs...
after: RHACS Cloud Service lowers operational costs...
Checklist (Definition of Done)

[ ] Unit and integration tests added
[ ] Added test description under Test manual
[ ] Documentation added if necessary
[ ] CI and all relevant tests are passing
[ ] Add the ticket number to the PR title if available, i.e. ROX-12345: ...
[ ] Discussed security and business related topics privately. Will move any security and business related topics that arise to private communication channel.

Codecov Report
Patch and project coverage have no change.

Comparison is base (ce56da5) 9.36% compared to head (16539ad) 9.36%.

:exclamation: Current head 16539ad differs from pull request most recent head 85bd9d0. Consider uploading reports for the commit 85bd9d0 to get more accurate results

Additional details and impacted files
@@          Coverage Diff          @@
##            main     #86   +/-   ##
=====================================
  Coverage   9.36%   9.36%
=====================================
  Files         36      36
  Lines        523     523
  Branches     108     108
=====================================
  Hits          49      49
  Misses       442     442
  Partials      32      32

Impacted Files
Coverage Δ

src/routes/OverviewPage/OverviewPage.js
0.00% <ø> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


issue with iphone X

@kcoley  Both my most recent usdz files posted on www.fusionar.app with rotating cubes using he latest version and  --use-euler-rotation  handle - Are working ok with iphone 7,  ipad mini & ipad pro... in iOS... but are throwing an error when viewing with iphone X. I have not had a chance to check any other iOS devices yet. I have checked the problem on two different iphone X and it never previews so it seems like an issue of some kind.

Hi @wave-electron they seem to work on my iPhone X, though I am running iOS 12.1 possibly on beta. Are the iPhone X you are testing on are running 12.1?

@kcoley This iPhone X isn’t on beta. Its currently running iOS 12.0. , my other test devices are on 12.1. So its a 12.0 issue!
Ok, I’ll close this off.


Unable to create peer via Control Center [Master]

There was an error loading a Vagrantfile. The file being loaded
and the error message are shown below. This is usually caused by
a syntax error.

Path: /home/mint/.vagrant.d/boxes/subutai-VAGRANTSLASH-stretch-master/7.0.4/virtualbox/Vagrantfile
Line number: 0
Message: RuntimeError:

Vagrant Subutai Plugin version (7.0.3) is incompatible with your box. Please update plugin by following command:

vagrant plugin update vagrant-subutai

Press any key to finish:```

As I understood, this error caused by plugin version on rubygems.org. The latest version of the plugin is 7.0.4, but we still have there 7.0.3. So, we have to replace the old version with a new one.

Fixed in Master.


Fix sorting in retries array

Motivation/Description of the PR
When a retry is added to the global retires within recorder.add the retries are sorted based on the availability of when fn. The sort logic is current returning a boolean but it must return a -1 or 1 for sorting to occur currently. This PR fixes this.
Applicable helpers:

[ ] WebDriver
[ ] Puppeteer
[ ] Nightmare
[ ] REST
[ ] FileHelper
[ ] Appium
[ ] Protractor
[ ] TestCafe
[ ] Playwright

Applicable plugins:

[ ] allure
[ ] autoDelay
[ ] autoLogin
[ ] customLocator
[ ] pauseOnFail
[ ] coverage
[x] retryFailedStep
[ ] screenshotOnFail
[ ] selenoid
[ ] stepByStepReport
[ ] stepTimeout
[ ] wdio
[ ] subtitles

Type of change

[ ] :fire: Breaking changes
[ ] :rocket: New functionality
[x] :bug: Bug fix
[ ] :clipboard: Documentation changes/updates
[ ] :hotsprings: Hot fix
[ ] :hammer: Markdown files fix - not related to source code
[ ] :nail_care: Polish code

Checklist:

[ ] Tests have been added
[ ] Documentation has been added (Run npm run docs)
[ ] Lint checking (Run npm run lint)
[ ] Local tests are passed (Run npm test)

@DavertMik could you please review and merge this bug fix?
The UTs are failing due to someother reason and not due to the changes in this fix.

@abhimanyupandian I'm a bit afraid to merge any change into recorder.
Could you explain me the issue you have hit to ensure this PR solves it. Thanks
Maybe it is possible to have unit test for this case?

@abhimanyupandian I'm a bit afraid to merge any change into recorder. Could you explain me the issue you have hit to ensure this PR solves it. Thanks Maybe it is possible to have unit test for this case?

Please find my response here https://github.com/codeceptjs/CodeceptJS/pull/3234/commits/3efd777da49028bfdca7d35b565fb46a5087697c

@DavertMik screwed something up as part of the commits :(
Please confirm if this PR can be deleted, will raise a support ticket for the same.


[ADD] raise exception in error case

Change-Id: I7eb73d3e20fb9c2b746935c5205218a2b0899706
@rgonalo

Coverage decreased (-0.2%) to 90.35% when pulling<PHONE_NUMBER>46baaf34952f86080e2085f125383b on iariasleon:feature/raise_exception_in_error into 4dd77f7523664e03ad33aac6c7fa0ecbbdc64482 on Telefonica:master.

Coverage decreased (-0.2%) to 90.35% when pulling<PHONE_NUMBER>46baaf34952f86080e2085f125383b on iariasleon:feature/raise_exception_in_error into 4dd77f7523664e03ad33aac6c7fa0ecbbdc64482 on Telefonica:master.

Coverage decreased (-0.2%) to 90.35% when pulling<PHONE_NUMBER>46baaf34952f86080e2085f125383b on iariasleon:feature/raise_exception_in_error into 4dd77f7523664e03ad33aac6c7fa0ecbbdc64482 on Telefonica:master.


azurerm_application and caf_applications inconsistency

Current design not consistent.
See application.tf and applications.tf.

This design decision causes a new version to be incompatible with an older one. I believe? Let me verify.


[FEATURE] Add py.typed

Problem Statement
This library's types are not picked up by some language servers (pyright) by default because the package does not include a py.typed file.
Proposed Solution
From what I can tell the library is fully or almost fully typed, so it should include py.typed.
Additional Context
See PEP 561 for more info on py.typed.

I believe this was addressed: https://github.com/databricks/databricks-sdk-py/releases/tag/v0.16.0

Fixed in 0.16.0.


Update README to add contributors automatically

Description
README can be automatically updated with contributors list
Motivation
There should be value given to their valuable contribution
Screenshots
No response
Additional Context
No response
Checklist

[X] I have checked the existing issues.
[X] I have read the Code Of Conduct.
[X] The changes don't break the code.
[X] I will attach screenshots and videos of the development work once it is done.

Priority
High

@GarimaSingh0109 assign me with this issue


Interface to integrate robots with rmf

Hi rmf team,
Can I verify for robots, I should use the Fleet Adapter Python template in the document linked [here] (https://osrf.github.io/ros2multirobotbook/integration_fleets.html) to do integration?
Also, I used to see the rmf_fleet_msgs last time as the interface for robots, but it is not in the online document. What happened to the interface?
Thanks.

Working off of the Fleet Adapter Python template is a good idea. It's the most straightforward way to work towards a correctly functioning fleet adapter.

Also, I used to see the rmf_fleet_msgs last time as the interface for robots, but it is not in the online document. What happened to the interface?

That interface was written near the start of the RMF project, and we found that it's too easy for integrators to make subtle mistakes when using it. It's also unable to remain stable while we add new features and capabilities. It technically still exists as an option for integration, but only so that prior integration work can continue to function. We strongly discourage using it for any new integrations, and I advise anyone who is currently using it to migrate to the newer API.


Fix broken link. [Closes #1114]

Docs Build status updates of commit 8486356:
:clock10: Full build: the average full build time is 25 min(s) 27 sec(s), this is based on the last 129 build(s) for this repository.

Docs Build status updates of commit 8486356:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azps-1.8.0/create-azure-service-principal-azureps.md
:white_check_mark:Succeeded
View (azps-1.8.0)

docs-conceptual/azps-2.2.0/create-azure-service-principal-azureps.md
:white_check_mark:Succeeded
View (azps-2.2.0)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


curses headers could not be found

Hi, guys.
After clone the source and typing ./build.sh all I got this:
Failed to install terminfo-<IP_ADDRESS>
Build log ( /home/clinix/.cabal/logs/terminfo-<IP_ADDRESS>.log ):
Configuring terminfo-<IP_ADDRESS>...
configure: WARNING: unrecognized options: --with-compiler, --with-gcc
checking for gcc... gcc
checking whether the C compiler works... yes
checking for C compiler default output file name... a.out
checking for suffix of executables...
checking whether we are cross compiling... no
checking for suffix of object files... o
checking whether we are using the GNU C compiler... yes
checking whether gcc accepts -g... yes
checking for gcc option to accept ISO C89... none needed
checking how to run the C preprocessor... gcc -E
checking for grep that handles long lines and -e... /bin/grep
checking for egrep... /bin/grep -E
checking for ANSI C header files... yes
checking for sys/types.h... yes
checking for sys/stat.h... yes
checking for stdlib.h... yes
checking for string.h... yes
checking for memory.h... yes
checking for strings.h... yes
checking for inttypes.h... yes
checking for stdint.h... yes
checking for unistd.h... yes
checking ncurses.h usability... no
checking ncurses.h presence... no
checking for ncurses.h... no
checking curses.h usability... no
checking curses.h presence... no
checking for curses.h... no
configure: error: in `/tmp/terminfo-<IP_ADDRESS>-17854/terminfo-<IP_ADDRESS>':
configure: error: curses headers could not be found, so this package cannot be built
See `config.log' for more details

It seems that libncurses5-dev is also required but not mentioned in the README file.
Is this problem due to my carelessness? My operating system is Linux Mint Debian.
Thanks.

I have not encountered this but likely all of my computers already had that. So it is quite possible that it is needed. If so it should be added to the readme.
As an aside please be aware that right now it works with IPython 2.* only; if you want it to work with ipython 3, use the jupyter branch (but documentation does not yet reflect many of the changes).

Got it,Thanks for your kindly reply.
Besides, is it possible for me to make any contribution to this project? (also for applying this year's Google Summer of Code project). I have intermediate level of skills of Haskell and like to learn anything unfamiliar for me.

Yes, there is much room for contributing. I suggest starting by sending me an email describing your level of background and starting with some small contribution which we can discuss via email (probably something on the issues list).

OK, thanks a lot.

I'm encountering the same problem while trying to build the Docker image. If I add libncurses5-dev to the apt-get line in the Dockerfile it builds correctly.

Hello, just in case anyone had the this problem on Fedora (I had it with Fedora 25) and I solved it by running
sudo dnf install ncurses-devel


⬆ Upgrade analytical-platform-compute-development cluster to 1.31

Upgrades: test, development, production.
User Story
As a platform engineer
I expect my clusters to be up to date
So that they don't whither and die, or end up on extended support because 💰
Useful Contacts
Gary, Jacob, Julia, Emma.
Proposal
Upgrade the cluster - analytical-platform-compute-development in analytical-platform-compute-development
Additional Information
kubernetes-release-calendar
Definition of Done

[ ]  Research the components that make up the cluster and understand if the impact/dependencies of upgrading.
[ ]  Understand and document how to test the cluster functionality for when this is done in Production
[ ]  Upgrade the cluster to 1.31 from 1.30
[ ]  Document findings and present to the team

rolled in to #5066


Remote validator service

PR Description
This PR introduces a new service in Teku, the Remote Validator service. This service allows remote validators to connect to the beacon node and receive events.
Important parts:

RemoteValidatorService: the actual service class that coordinates the wiring of all components together
RemoteValidatorSubscriptions: the class that maintains the active subscriptions and broadcast events to subscribers
RemoteValidatorApi: the WebSocket service that is used as a channel to publish the messages
RemoteValidatorBeaconChainEventsAdapter: this class listens to all events that a subscriber is interested in.

Related Issue(s)
See https://github.com/PegaSysEng/teku/issues/1918
Documentation
N/A: this work is in progress

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


Pr/607 rename image - Create feature to set first / lastname to picture

New option in admin menu under picture (expert mode)
Creates new button on result page
First and lastname for renaming the images
Copy image to named-image folder (new inut field in admin)
New icon (pencil)
Translations in en.json

Prerequisites checklist

[x] I have read the contributing guidelines.

What is the purpose of this pull request? (put an "x" next to an item)

[ ] Documentation update
[ ] Bug fix
[x] New feature
[ ] Other, please explain:

What changes did you make? (Give an overview)
Programmed Issue 607 https://github.com/PhotoboothProject/photobooth/issues/607
Is there anything you'd like reviewers to focus on?

 Check the response of automated tests, it'll tell what need to be changed.

npm run format should take care about most formatting things, also npm run composer:lint.

Pre commit hook would need to be installed:

sudo apt-get install shellcheck
sudo pip install pre-commit
Inside Photobooth source run:

pre-commit install

Thanks for the PR :-) haven't had time yet to test, hopefully find some time next week

Didn't found enough time yet to test and look at closer... But haven't forgotten about it


No visual feedback when wrong code is submitted.

The Run tests (ctrl + enter) button in waypoints doesn’t give any visual feedback when the wrong code is submitted.
It would be nice to have this, especially useful to a novice Camper.

+1

I don't know what the name would be in web design, but perhaps something like what Android calls a "toast" would be a good option. Something that pops on-screen visibly but unobtrusively when the tests have finished running, but doesn't require user interaction to dismiss, it's just timed to appear for a brief second or two.

This definitely presents a problem, especially when error messages fall below the fold. Even when a user can see them, it takes a moment to focus on them given their lower-left position on the page.
However, since the issue isn't that the form lacks visible error handling but, rather, that the errors aren't clearly visible, another solution, which leverages existing error handling, might be to move the error messages below the editor, as shown in the attached screenshot.

@ltegman Some sort of a modal window would do the trick nicely. I'm pretty sure either Bootstrap or jQuery UI may have some gizmo we can use for this

Subscribing :)

@hallaathrad I know once we go to React that we will be embracing Toasts (little micro-popups) and they would be ideal for this.

There's no reason to wait for React. Toastr could be used to immediately bring about this effect. Give the demo a shot, they're highly configurable.


WIP: follow up to daily builds work

Splits most of the workflow in a separate reusable template
file. This allows us to safely use defaults of workflow inputs
regardless of the reasons workflow got triggered and don't think about
fallback every time we need to use the input.
"Detect jobs to run" step, as well as the attempt to find "ci test
all" comment now will be done only in CI workflow. Scheduled jobs
will automatically test everything.

ci test all

Looks good so far! ✨  https://github.com/prisma/prisma/actions/runs/4916696460


compact encoding for hare preround message

hare preround message includes every proposal that is eligible for rewards. it references it by specifying first 20 bytes of blake3 hash of the full proposal. with 50 proposals per layer the message is around 1KB, not including vrf and regular signature. with 500 proposals per layer it grows to 10KB per preround message. number of preround messages are bounded to the size of hare committee, right now we are running it with 400 participants, but aim to increase it to 800.
with 10KB per preround message every peer has to download 4MB of unique data within 12s, in practice it downloads some duplicates because of the redundancy on gossip layer. the other part of the problem is that it scales with number of connections that node maintains. if node has 1000 connections, it can expect to be requested 4GB of data within 12s. eventually, if not now, this will be unsustainable to keep such number of connections.
compact encoding protocol
the reason why we keep 20 bytes is to prevent collisions, as otherwise votes are ambiguous and cannot be counted properly. moreover adversary can create collisions intentionally if identifier will be kept short to 2 or 4 bytes.
the proposal is to introduce compact encoding that is not open to dos attack, maintain low rate of accidental collisions, and in case of collisions provide a safe fallback.  it can reduce traffic either x5 or x10, depending on the option that we choose.
protocol for encoder:

construct preround message with a collection of full proposal identifiers. proposals should be sorted in lexicographic order
sign message with full proposal identifier
replace each proposal identifier with an output from short_hash(vrf, proposal_identifier)

short_hash is a cheap hashing function, such as siphash or halfsiphash https://github.com/veorq/SipHash/blob/master/halfsiphash.c.
we may decide how much we want to truncate output.
protocol for decoder:

receive a message, verify a vrf
for all proposals that you have locally compute short_hash(vrf, proposal_identifer)
if received message votes several times on the same proposal
validate signature on reconstruct message

we need to handle two edge cases due to possible collisions:

message includes multiple short identical id. in this case we decode them in lexicographic order. so for example if we received  short ids [4,4], and we have locally proposals 0xa, and 0xb, we decode 0xa and then 0xb.
message includes single id, but locally we have collisions. in such case we have to verify signatures for different proposal and find if one of the matches.

i introduced label https://github.com/spacemeshos/go-spacemesh/labels/draft to signal that the discussion about this only ready to be implemented yet.

@dshulyak I have several questions arising from reading the related materials:

my understanding is that overriding the proposal IDs will happen only in the preround
right now the type on Value is an array of 20 byte blake3 hashes
if we switch to another hashing algo it means we need to change the type on Value
if we want to only change the type for the preround message, we should either:

introduce a new field on Value, something like CompactProposals []types.CompactProposalID. This field will only be used in the preround

use a different message exchange for the preround

change the type of Proposals to allow dynamic sizing of the data, i.e. Proposals [][]byte

I would prefer to use the first option, or the second
Either way, this presents a possible breaking change for the protocol, how do we handle that?
any objections to using this as a siphash implementation?

i think new type []CompactID is superior, each opaque []byte type will have to be prefixed with compact integer, one more byte or 50% increase per id. one more option is to use bit packing and then do variable length encoding for proposals if we know that there are collisions.

regarding whole siphash thing, in original idea i wanted it to be keyed by vrf (truncated to whatever is the size of the key).
however there is also this "extension" that needs to be implemented.

how about even simpler approach.
preround message is encoded with 2/4 bytes, simply 2/4 first bytes of the proposal hash without any additional randomness.
message is signed using full proposal hash, as discussed previously, so that signature verification is always executed on correct message.but in case of ambiguity, where we cannot deterministically decode the message for signature verification,
we will do one more round trip and download message with full ids from the peer that sent message with small ids.
so if adversary pulls out an attack, we potentially will have to download 10%-25% more data, and we also know how to disconnect a peer that intentionally sends ambiguous messages

and in this case there seems to be no difference between truncating original blake3 hash to 2 or 4 bytes or using siphash. as ambiguous proposals will always have to be requested from the peer that sent message with them.
with this extension the decoding protocol is like this:

receive a message, verify eligibility with vrf
decode proposals according to the lexicographic order of full proposal, for example full proposals 0xaabbcc and 0xaabbdd, compact ids for both of them are 0xaabb
we can try to decode the message and check signature if both are included
if signature fails or not both are included we have to do a roundtrip and ask a peer for full hashes of the proposal that it upvoted that start with 0xaabb in layer X. in this last case peer can reply only with hash, and not the full proposal

one more thing is that hare gossip handler right is in synchronous mode, but it will have to be switched into a mode that spawns goroutine per request in gossip libp2p.

there was a question about breaking protocol changes and upgrade path, it should be as follows:

introduce new protocol version for gossip, i would suggest to keep it as hare31. new version will have an updated type
keep two versions in the codebase until the transition happens
transition is orchestrated by configuring enabled/disabled layer. for example hare3 stop at 10000, hare31 starts at 10001
after transition it is ok to remove previous code

decode proposals according to the lexicographic order of full proposal, for example full proposals 0xaabbcc and 0xaabbdd, compact ids for both of them are 0xaabb

My understanding is that the peer will only send the compact IDs, not the full identifiers.
Also, you've mentioned this in the original proposal:

sign message with full proposal identifier

The signature of the message right now is done with full proposal identifiers. If we don't have full knowledge of them locally, how could we verify the signature? (apart from the dry signature-matches-message check)

if signature fails or not both are included we have to do a roundtrip and ask a peer for full hashes of the proposal that it upvoted that start with 0xaabb in layer X. in this last case peer can reply only with hash, and not the full proposal

Maybe it is an idea to just include the colliding hashes preemptively in the message? we could insert metrics to see how often this happens and see if it's really degrading the performance of the optimization to a large degree. To me, adding another roundtrip increases complexity and adds a lot of room for stuff to go wrong, since a dependency is formed in the protocol logic - a termination of a message exchange depends on another set of message exchanges in the protocol - this has all sorts of negative repercussions which I can go into if necessary.

After discussion, we arrived to the following conclusion:

Proposal IDs are truncated into smaller compact IDs before being sent over the gossip network.
Each proposal ID is initially truncated to 2 bytes.
The sender constructs a message with truncated proposal IDs and includes auxiliary data for resolving any detected collisions.
Auxiliary data includes additional bytes to extend truncated IDs when collisions are detected.
The sender then signs the pre-round message with the full IDs and includes the truncated IDs and auxiliary data.

Handling Collisions:
When collisions (same truncated ID for different proposals) are detected:

First check: Extend the truncated ID to a longer size (6 bytes from the Blake3 hash) to reduce collisions.
Final check: If collisions still exist, include the full proposal ID (32 bytes).

Verification Process:

The (honest) receiver checks the digital signature of the message using the sender's public key.
The signature should validate against the full proposal IDs (or the reconstructed full IDs from the truncated and auxiliary data).
The receiver uses the truncated IDs and any auxiliary data provided by the sender to reconstruct the full proposal IDs.
If verification fails (can't reconstruct the message or verify signature), the message is rejected and not rebroadcast.
Otherwise, receiver forwards the message.
Receiver should not engage in extensive additional communication to resolve IDs (non-interactive process).


[BUG] Error: Failed to parse Bard response

Error: Failed to parse Bard response

i think is something of BARD, because https://bard.google.com/?hl=en, not working to


Issue with importing GCP resources | dial tcp <IP_ADDRESS>:80: connect: host is down

Hi everyone,
I am not able to import the google resources.
I followed the installation process in the Readme but still not able to import the resources.
Always getting dial tcp <IP_ADDRESS>:80: connect: host is down
Am i missing something here?
 Cheers in advance.

I fixed it by setting GOOGLE_APPLICATION_CREDENTIALS environment variable.


Standings endpoints do not work without specifying a year

When the driver or constructor standings endpoint is called without specifying a year, the end of season standings for multiple years should be returned, starting with the oldest season. Currently, a server error is encountered and status code 500 is returned.
Example:
http://api.jolpi.ca/ergast/f1/driverstandings.json?limit=100
http://ergast.com/api/f1/driverstandings.json?limit=100
This is likely similar to #25.

I think this is probably fine. The only use case that I see for the current behaviour is getting all/many standings in as few requests as possible. This is probably not that common?
Also, I imagine that many people might still be iterating year by year for that anyway. You don't really know what data you get in advance for any given limit and offset query parameter values. And the first and last entries may be incomplete and split over multiple queries. So this is more involved to do client side, even if it is more efficient due to needing fewer requests.
@chrisfree do we want to have an "intentional difference" label and a "requires documentation" label for things like this so that we can properly keep track of that and don't forget to document it?

@jolpica and @theOehrly I think your logic here makes good sense.
I am wondering, how difficult it would be to retain the current alias that Ergast has, wherein URLs like the following:

http://ergast.com/api/f1/current/driverStandings
http://ergast.com/api/f1/current/constructorStandings

...simply return the standings for the current season.
I will add some labels now, as suggested. Solid way for us to keep track of these decisions over time.

Are you suggesting that
http://api.jolpi.ca/ergast/f1/driverStandings/
should redirect to
http://api.jolpi.ca/ergast/f1/current/driverStandings/
Or that we should just keep these current aliases as they are?

I think Chris meant the opposite, but this is already the current behaviour in Ergast. So his suggestion is how it actually works right now?
@chrisfree The "current" keyword is equivalent to using "2024" and returns the standings for only the current season.
Using no specifier for the season returns the standings for multiple seasons until the query limit is reached.
So I don't think there is a need to change the behaviour of the alias (except for the change suggested in #42).

I think I got us all confused here. When I was testing the current variant over the weekend I was getting 500 errors, but those disappeared once I replaced current with 2024. So I think my point is now moot. Sorry for the confusion. I was just trying to ensure that those that currently rely on the current parameter didn't see a major regression.

Messing with this tonight, with you only able to get one year's standings, it feels weird to have the DriverStandingsnested inside of a another object inside another list if the list is always only going to have 1 year returned.
Is there any capability to return the standing in a given round? I don't think so but if I wanted to see the standings at the end of the 4th race, that would be nice.

To get a driver’s position for each season, it looks like I need to make a separate request for each season with the new API, like this:https://api.jolpi.ca/ergast/f1/{season}/drivers/{driverId}/driverstandings.json
Previously, with the old API, I could just make a single request to: http://ergast.com/api/f1/drivers/alonso/driverStandings.json and get all seasons in one go.
I’m calculating a driver’s total championships, and with the old approach, I only needed one request. Now it seems I need one for each season. Is there an easier endpoint on Jolpica that would allow this to work in a single request?
Thanks for all the great work!

Is there an easier endpoint on Jolpica that would allow this to work in a single request?

Unfortunately currently there isn't any simple way. Hopefully in the new year we'll start working on new endpoints to fill in these gaps. However, we are open to anyone who wishes to help contribute adding this missing feature on the ergast endpoint


Block merge if graphite downstack is not yet merged

Checklist

[x] I have read the Contributor Guide
[x] I have read and agree to the Code of Conduct
[x] I have added a description of my changes and why I'd like them included in the section below

Description of Changes
For graphite we want to have a protection mechanism that prevents stack branches from merging if downstack branches are not yet approved and merged.
I don't really know how to test this 'properly' or if it does what I want it to do. What would be a good way to test this before publishing life?
Some solutions
The tricky part is to check if downstack PR's are merged and only block merging in that case. Its tricky because we don't have access to the downstack PR's from within the code, as the code is based around analyzing the current PR.
Inspecting HTML
One possible approach is to inspect the graphites HTML comment code, assuming it remains relatively stable over time. By checking if the PR icon contains the 'open' class, we can determine whether the PR has been merged or not. This can be achieved by examining the following condition:
    async fn merge_blocked_by_graphite(&self) -> anyhow::Result<bool> {
        let pr_comments = self
            .client
            .get_pull_request_comments(self.config.name.as_str(), self.pr.number)
            .await?;

        for comment in pr_comments.iter().filter_map(|x| x.body_html) {
            if comment.contains("Current dependencies on/for this PR:") {
                let stack_pr_opened_status = comment
                    .lines()
                    .into_iter()
                    .filter(|x| x.contains("PR"))
                    .map(|x| x.contains("octicon-git-pull-request open color-fg-open"))
                    .collect::<Vec<_>>();

                let mut downstack_pr_opened = false;
                for current_opened in stack_pr_opened_status {
                    if downstack_pr_opened {
                        return Ok(true); // A downstack pr is opened, block till downstack branch is closed.
                    }
                    downstack_pr_opened = current_opened
                }
            }
        }

        Ok(false)
    }

Two labels
An other approach is to add the block label only once to the Pull Request (PR). If the user removes the label, we should refrain from re-adding it to the PR. However, achieving this may not be straightforward. One potential solution is to introduce a second label that doesn't block the PR but serves as an indication that it is a "graphite" PR. We can add the block label only if the "graphite" label is not yet set. Consequently, if the user removes the block label and the "graphite" label is present, we won't add the block label again.
Iterating PR's
In the lib.rs file, we retrieve all the PRs from the repository. An alternative approach is to pass these PRs to the 'Analyzer', which is responsible for analyzing individual PRs. Within the 'Analyzer', we can extract the PR descriptions from the body of the Graphite's PR. By doing so, we can obtain a list of related PRs. From this list, we can then proceed to check the statuses of the stack PRs.

I tested the code over in the new testing repo and running octobors locally with my github token. I went for the Two labels approach as that seems to only one that is possible without to much work. This means:
Add the block label only once to the Pull Request. If the user removes the label, we should refrain from re-adding it to the PR. We then also need to introduce a second label that doesn't block the PR but serves as an indication that it is a "graphite" PR. We can add the "block label" only if the "graphite" label is not yet set. Consequently, if the user removes the block label and the "graphite" label is present, we won't add the block label again.
Also made it so that the bot will react with a message on why it added the label. The bot can be pinged and will respond with more details.

not familiar with this codebase at all but will try to take a look.
CI is failing though on tests and cargo-deny, so need some cleanup & fixes most likely

btw that is how I have configured Mergify in puffin repo, they have a config file we use where one can set which branches allow merging

thought about it some more and that seems like it may be the easiest solution, only auto merge PRs that are targeted to main, instead of this change.

Sounds good, if we dont want auto-mege in custom branches, thus always manual merge for those, we can do it that way!

I'll close this for now but I definitely think the direction we previously discussed is the way to go!


C# Highlighting bug

See https://docs.microsoft.com/en-us/dotnet/csharp/discards

static keyword gets incorrect coloring. It seems to happen because it's followed by a parenthesis. So, it was marked as if it's a method name. This bug should happen whenever a method returns tuple.

@Youssef1313 Please move this issue to https://github.com/dotnet/docs/issues


Format cabal file and get rid of warnings

This fixes several warnings as well.

Oh, I thought, I deleted that code from run-ck. Thank you.


Configure Jest to test all application and generate coverage

This resolve issue #18 And issue #24

Changes Unknown when pulling dad05184716bab435a666c3ae6d803fddb6b7f04 on coverage into ** on master**.

Coverage decreased (-34.4%) to 24.456% when pulling dad05184716bab435a666c3ae6d803fddb6b7f04 on coverage into d5c7968e324b96636e15a6e750f55f933816b013 on master.


fix: deny unknown fields for workspace config

Forgot to do this earlier.

This is just for deno.json


reccomended  ui changes

This pr-->

add about, announcements and resources.
remove the logo at the footer

@rosaga


[feature request] tensor type boundaries in bucketize op

ping @yongtang and refer to https://github.com/tensorflow/tensorflow/pull/14774#issuecomment-416726581
Currently, boundaries has to be list type. It is not convenience for the dynamic boundaries .
It would be great ifboundaries could be tensor type.

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce
Mobile device


[SPARK-11215] [ML] Add multiple columns support to StringIndexer

Add multiple columns support to StringIndexer, then users can transform multiple input columns to multiple output columns simultaneously.

Merged build triggered.

Merged build started.

Test build #43987 has started for PR 9183 at commit 10ec734.

Test build #43987 has finished for PR 9183 at commit 10ec734.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43987/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #44069 has started for PR 9183 at commit 24ad0fd.

Test build #44069 has finished for PR 9183 at commit 24ad0fd.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44069/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #44075 has started for PR 9183 at commit d039f9a.

Test build #44075 has finished for PR 9183 at commit d039f9a.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44075/
Test FAILed.

Jenkins, test this please.

Merged build triggered.

Merged build started.

Test build #44077 has started for PR 9183 at commit d039f9a.

Test build #44077 has finished for PR 9183 at commit d039f9a.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44077/
Test FAILed.

Build triggered.

Build started.

Test build #44146 has started for   PR 9183 at commit a64f71d.

Because of the multiple columns StringIndexer use Aggregate rather than countByValue to compute distinct value count, if two or more values has the same count, it will has indeterminate order.
So 1) binary classification label column may be indexed to different result(0, 1 or 1, 0); 2) OneHotEncoder will drop the last category in the encoded vector by default, if there are more than one value can be drop, it will indeterminate drop which one in this proposal.
I don't think we need to keep the order restriction produced by countByValue which may lead poor performance in the Aggregate implementation, so I disabled some test cases.
If my proposal work well, I can enable and update these test cases.

Test build #44146 has finished for   PR 9183 at commit a64f71d.

This patch fails SparkR unit tests.
This patch does not merge cleanly.
This patch adds the following public classes (experimental):
class StringIndexer(JavaEstimator, HasInputCol, HasOutputCol, HasHandleInvalid,
class HasOutputCols(Params):

Build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44146/
Test FAILed.

Jenkins, test this please.

Build triggered.

Build started.

Test build #44196 has started for   PR 9183 at commit a64f71d.

Test build #44196 has finished for   PR 9183 at commit a64f71d.

This patch fails SparkR unit tests.
This patch does not merge cleanly.
This patch adds the following public classes (experimental):
class StringIndexer(JavaEstimator, HasInputCol, HasOutputCol, HasHandleInvalid,
class HasOutputCols(Params):

Build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44196/
Test FAILed.

Build triggered.

Build started.

Test build #44220 has started for   PR 9183 at commit 2e8bc28.

Test build #44220 has finished for   PR 9183 at commit 2e8bc28.

This patch passes all tests.
This patch does not merge cleanly.
This patch adds the following public classes (experimental):
class StringIndexer(JavaEstimator, HasInputCol, HasOutputCol, HasHandleInvalid,
class HasOutputCols(Params):

Build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44220/
Test PASSed.

Because of the multiple columns StringIndexer use Aggregate rather than countByValue to compute distinct value count, if two or more values has the same count, it will has indeterminate order.
So

binary classification label column may be indexed to different result(0, 1 or 1, 0);
OneHotEncoder will drop the last category in the encoded vector by default, if there are more than one value can be drop, it will indeterminate drop which one in this proposal.
I don't think we need to keep the order restriction produced by countByValue which may lead poor performance in the Aggregate implementation, so I disabled some test cases. If my proposal work well, I can enable and update these test cases.

Test build #47507 has started for   PR 9183 at commit 2e8bc28.

+1, I've been meaning to request this transformer for a while.

LGTM, I think a good follow up would be to do the same with IndexToString.

Thanks for the pull request. I'm going through a list of pull requests to cut them down since the sheer number is breaking some of the tooling we have. Due to lack of activity on this pull request, I'm going to push a commit to close it. Feel free to reopen it or create a new one.

What needs to happen to move this forward? This was a PR that would have been the first iteration of a significant improvement in handling of wide datasets.

I think @yanboliang just need to push this forward and get people to review it.

@yanboliang will you be reviving this PR?

@MLnick I think this is an important feature to make ML pipeline handle large datasets elegantly. I will update/send a new PR soon and looking forward that you can help to review. Thanks!

@yanboliang This is a very helpful initiative by you. Thanks for taking it up. Let me know, if you need any help for this PR.

@pramitchoudhary Yeah, lots of users vote for this feature. I will update this PR to match master in the next release cycle and looking forward to review/comment. Since Spark 2.1 is code freeze, we can only merge bug fix and docs change right now. Thanks.

@WeichenXu123  Any activity for the new PR?

@minixalpha Sorry for delay. Too busy recently. But I will try to finish and commit my new PR once I get time. Thanks!

Thanks for you job! @WeichenXu123


UserInfo from external OpenIdConnect Provider

Hi!
I have an ADFS 4.0 Server successfully connected to my Identityserver4 as OpenIdConnect Provider.
How can i get the UserInfo from this OpenIdConnect Provider?
I need to get the Roles and a few other things from there to grant Access to an API based on this Claims.
If i "ask" the UserInfo Endpoint from Identityserver, will Identityserver forward my request to the external Provider?
Or do i have to ask for this Infos on Login and save as claims on the User in my Database?
Br
Bernhard

How can i get the UserInfo from this OpenIdConnect Provider?

That's up to you when connecting to it. If you're doing this in an IS4 host, then you are probabaly using the OIDC middleware from Microsoft which automatically consumed the user into endpoint. I recently heard that ADFS' OIDC support is... limited: https://twitter.com/glatzert/status/827959547478364160
So you'd really have to ask Microsoft for support on how to use ADFS's OIDC support.

Ah ok! Thanks!


Database sharing and access control

Rough description
A database owner should be able to allow other users/entities to have following levels of access

read-only access
read/write access
manager (to be able to control sharing with anyone except the owner)

A database owner should also be able to control whether the public can have various forms of read-only access

metadata only
forking
read-only querying

The plan

[x] Introduce SQL data model (#461).
[ ] ayb_db support for CRUD on these data models, with an enum each for EntityDatabaseSharingLevel and PublicDatabaseSharingLevel. Both should include no_access levels.
[ ] Add endpoint to add / update / remove an entity's access to a database. Setting to no_access removes the row if it exists. Permissions: must be manager or owner, managers can't change owner permissions.
[ ] Add an endpoint to add / update / remove a database's public sharing level. Permissions: must be manager or owner.
[ ] Update existing permissions checks to respect new permissions.
[ ] Add support for read-only access.

I think we should combine forking and querying, if someone can fork the database then they can run read-only queries as well :)

Definitely before we have forking support and support for varying memory/CPU for different databases, I agree. :)
The reason I was thinking someone might want to give fork read-only access but not query read-only access is because queries take up resources on your running database, whereas forking doesn't. So perhaps in the future when you want someone to look at your dataset but don't want them to use up your CPU running queries, you can say "fork my dataset and let me know what you find!" without having to worry about beefing up your own running database.
Definitely not something we have to worry about for a while :).

Working on this next! Came up with a rough outline of a plan in the issue description!

SQLite data models work, and started on Rust data models. For next time

Add instructions on setting up postgres for testing to readme. need createuser -P postgres_user (enter password test) and (still to figure out) some way to run sudo -u postgres psql -c "alter user postgres_user createdb;" (there's a psotgres user on ubuntu, not on MacOS)
on permissions-datamodels branch, fix whatever the postgres issue is after getting installation working
continue developing on permissions-rustdatamodels

Updated Linux-vs-Mac instructions with #463
Merged #461
Continuing on permissions-rustdatamodels!

Done

permissions-rustdatamodels now has the data models
fields on existing database CRUD queries handle public_sharing_level
client and server handle public sharing level on database creation

Next

Database support for updating public_sharing_level
Database support for create/update/remove entity_database_permission

Reworked plan to be endpoint-focused (e.g., expose public sharing first) rather than layer-focused (e.g., implementing all DB functionality first, but no endpoints are exposed).
First up: Add endpoint to support updates to a database's public sharing level. Make this a generic update_database endpoint. Permissions: must be manager or owner.

Almost done with endpoint to update public sharing level.
Next

Finish update public sharing level
Fix issue for update_profile and update_database where no arguments results in a SQL syntax error
CRUD support for entity_database_permission

Server-side support for public sharing level updates done. Started writing end-to-end tests.
Next

Add client-side support, merge
Finish writing tests for public sharing endpoint and permissions
Support read-only

`public_sharing_level is now updateable! Next up:

Support read-only
Support listing database

List and query logic and tests are all set (exposed to the query and entity details endpoints, passed through properly to rusqlite). Next

Clean up some debugging I did to figure out correct SQLite flags (SQLITE_OPEN_CREATE should only co-exist with SQLITE_OPEN_READ_WRITE)
Catch "executing write on read-only query" from rusqlite and pass it to client properly
Update tests with appropriate error from previous bullet

Merged working support for fork/read-only public sharing level!
Next

Documentation
Version bump
CRUD for entity-level access to database

Wrote up documentation for both public sharing level and entity-level permissions. Next

CRUD for entity-level access to database
Endpoint to call that CRUD
Update permissions logic to look up entity-level permissions

Database-/endpoint-/CLI-level support for sharing is implemented (but untested) on branch permissions-crud. Next

Update permissions logic for read-only/read-write permissions
Update permissions logic for managers
Add tests (query levels, management permissions, ensure owners are protected)

Permissions work and are covered by end-to-end tests! (And tests helped discover a bug, yay!). To ship:

Code review
Consider whether to call the endpoint entity_database_permissions or share
Add one more test that a second database owned by first entity isn't accessible to second entity despite second entity having access to first database.

All done and merged! Next steps:

Test the documentation, and if it works, merge
Version bump and announce!!!!!


Rename udla.txt to udladf.txt

udla.mx redirects to new adress: udladf.mx

Hm... It's not associated to the same MX domains...
Are you a student there? Is your email address<EMAIL_ADDRESS>or<EMAIL_ADDRESS>

@pchaigno Yes, Im a student and I have an account<EMAIL_ADDRESS>The school changed the domain email addressing. udla.mx redirects to udladf.mx.

Okay, thanks!


Notebook for frequency analysis + stats refactoring

Pull Request Checklist:

[x] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes #528

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Documentation has been added / updated (for bug fixes / features)
[x] HISTORY.rst has been updated (with summary of main changes)
[ ] bumpversion (minor / major / patch) has been called on this branch
[ ] Tags have been pushed (git push --tags)

What kind of change does this PR introduce?

Refactor generic to move statistical functions to stats module

Add notebook on frequency analysis

Does this PR introduce a breaking change?
Yes, imports for statistical function from generic will fail. Should we add a deprecation warning ?

Other information:

Pull Request Test Coverage Report for Build 2373

106 of 113 (93.81%)  changed or added relevant lines in 2 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.08%) to 92.365%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

xclim/indices/stats.py
104
111
93.69%

Files with Coverage Reduction
New Missed Lines
%

xclim/subset.py
3
62.5%

Totals

Change from base Build 2365:
-0.08%

Covered Lines:
3218

Relevant Lines:
3484

💛  - Coveralls

@sbiner Voir https://xclim--576.org.readthedocs.build/en/576/notebooks/frequency_analysis.html Est-ce compréhensible ?


Add: code that requires a dependency or two

Right now the code base is super simple and as such there are no dependencies.
Because most packages do have dependencies we should enhance the code some to include other packages that we can then add as dependencies to our pyproject.toml file.
Maybe pandas and we could have the package generate plots with altair? or perhaps others have an idea as to what functionality this could have.

i think we could actually add the modules that @ucodery added here to this example. thus we have a simple add_numbers for teaching and a few other modules with dependencies for a more robust package setup.
Let's start with basic code and then we can make it more robust, add more tests etc later. even add a class - that might be good. (in a separate issue)

Are you wanting to make examplePy a dependency of this package? That won't work without some complex build setup as examplePy isn't a published project. Probably better to add pandas like you were originally thinking.


LeaderBoard code

Hello, Do you have a plan to release the code for the LeaderBoard?

Leaderboard code is available at https://huggingface.co/spaces/Vchitect/VBench_Leaderboard/tree/main.

These results are already displayed in the leaderboard. Thanks.

These results are already displayed in the leaderboard. Thanks.

emmm, I means the raw results before showing in the leaderboard,
because the results in the leaderboard are normalized and weighted summed.
If i want to launch a gradio in a local environment, it is necessary to get the results.csv in the CSV_DIR.
So can you provide the  results.csv used in the leaderboard?
Thank you very much.

These results are already displayed in the leaderboard. Thanks.

Hello, I find some dimensions are missed in the upload sampled videos,
for example, videocrafter-1.tar.gz don not have the dimension of "background consistency",

Hi, for missing videos: Some dimensions, such as "background consistency" and "scene," use the same set of videos.
plus: It would be great if you open new issues for each question or email us for ongoing discussions. Thanks!

Thanks，i will try with the same video sets，and i will open a new issue if i have other questions.在 2024年4月29日，11:23，Ziqi Huang @.***> 写道：
Hi, for missing videos: Some dimensions, such as "background consistency" and "scene," use the same set of videos.
plus: It would be great if you open new issues for each question or email us for ongoing discussions. Thanks!
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you modified the open/close state.Message ID: @.***>

Hi, for missing videos: Some dimensions, such as "background consistency" and "scene," use the same set of videos.
plus: It would be great if you open new issues for each question or email us for ongoing discussions. Thanks!

Thanks，i will try with the same video sets，and i will open a new issue if i have other questions


AppCivist Patterns Initialization

Bootstrap AppCivist Beta front-end patterns with initial html partials for the components in the new wireframes, which are available in the AppCivist UX Running Notes

closed in favor of the same epic on the new repo


[bugfix] Access default year month date from moment instance

When timezone is setted, moment.date() may be different from  Date().getDate()

@yulodl I don't fully understand why this is needed, explain the issue you're fixing in more detail and provide tests.

If you want to create timezone-specific instances, use moment.tz(...) constructor, which uses moment.utc internally and then adjusts for the specified timezone.
Feel free to reopen (or open a new issue) if you think the recommended flow is broken.


Codegen breaks for proto definitions with a streaming service named 'Stream'

Bug Report
The codegen module imports futures_core::Stream directly, but when using tonic-build to generate a service for proto definitions with a service named Stream the imported trait conflicts with the generated trait.
Version
tonic 0.4.1, tonic-build 0.4.1, prost 0.7.0

Platform
Linux bsull-xps15-arch 5.11.2-arch1-1 #1 SMP PREEMPT Fri, 26 Feb 2021 18:26:41 +0000 x86_64 GNU/Linux

Crates
tonic, tonic-build
Description
Codegen breaks for proto definitions with a streaming service named 'Stream'.
I tried this code:
❯ cat Cargo.toml
[package]
name = "tonic-repro"
version = "0.1.0"
authors = ["Ben Sully<EMAIL_ADDRESS>edition = "2018"

[dependencies]
tonic = "0.4.1"
prost = "0.7.0"

[build-dependencies]
tonic-build = "0.4.1"

❯ cat proto/test.proto
syntax = "proto3";
package test;

message TestMessage {
  bool ok = 1;
}

service Stream {
  rpc RunStream(TestMessage) returns (stream TestMessage);
}

❯ cat build.rs
fn main() -> Result<(), Box<dyn std::error::Error>> {
    Ok(tonic_build::compile_protos("proto/test.proto")?)
}

❯ cat src/lib.rs
mod test {
    tonic::include_proto!("test");
}

This fails to compile with:
❯ cargo c
    Checking tonic-repro v0.1.0 (/home/ben/repos/rust/tonic-repro)
error[E0220]: associated type `Item` not found for `stream_server::Stream`
  --> /home/ben/repos/rust/tonic-repro/target/debug/build/tonic-repro-52881560b6922b38/out/test.rs:78:38
   |
78 |         type RunStreamStream: Stream<Item = Result<super::TestMessage, tonic::Status>>
   |                                      ^^^^ associated type `Item` not found

error: aborting due to previous error

For more information about this error, try `rustc --explain E0220`.
error: could not compile `tonic-repro`

I have a quick fix at https://github.com/hyperium/tonic/compare/master...sd2k:codegen-dont-export-stream, let me know if a PR would be accepted :)

A PR would be much appreciated  especially if it includes a test 😉


Does Table 5 consider the computing time of optical flow?

Hi,
Thank you for your great work.
In table 5, the cross attention computing time is reduced significantly. May I know did you consider the computing time of optical flow in Table 5 since I noticed that tinyflownet also requires 10ms to compute, which is the same as the baseline cross attention computing time?
Thank you.

Sorry for the late reply.
Table 5 only reports the time of Regional Memory Reader, i.e., the time of TinyFlowNet is not included.


Add test for pipenv run

Add small pipenv run test ( added in order to ensure no regression by #398 ) !

Thanks @nonylene!


Feature Request - Default SuitRack

A config option to allow the suit rack to be uneffective would be nice, as I was making a modpack and needed a way for vr players to easily equip suits as they cant flip through TooManySuit pages but sadly this mod doesnt work with too many suits (which i kinda expected tbh) and it would just be a nice feature so i could get this mod for vr players and the TooManySuits mod for desktop players for my modpack

A config option to allow the suit rack to be uneffective would be nice, as I was making a modpack and needed a way for vr players to easily equip suits as they cant flip through TooManySuit pages but sadly this mod doesnt work with too many suits (which i kinda expected tbh) and it would just be a nice feature so i could get this mod for vr players and the TooManySuits mod for desktop players for my modpack

I definitely have thought about this in the past. Either making a config option or detecting if TooManySuits is loaded and stopping my functions that pulls suits from the rack.
Would you still want commands to be added to the terminal when TooManySuits is loaded?

sorry for my late response and yes id still like commands when too many suits is loaded other wise i just wouldent download the mod in the first place, mostly because vr players cant go through too many suits pages but vr players can use terminals

pushed 1.0.6, hope this works as a solution to the issue here!

marking this resolved as the 1.0.6 update should have provided the features you were looking for. Feel free to reach out if you have any other issues


Fixing performance issue for gets.

The ChannelPool is great for scans, but causes performance impacts to Get operations.  Moving ChannelPool handling into BigtableDataGrpcClient and only using a ChannelPool for stream operations..

I updated the code... how about this?
private void returnToPool(final Channel channel) {
if (channel != channelPool) {
((PooledChannel) channel).returnToPool();
}
}

It looks good, please add some test to reflect the change in BigtableDataGrpcClient.java.

I updated the comments and added some testing


Add new StringVariable-type (supported by GtkWave)

This commit adds support for the string-vcd-value-type. It seems to be only supported by GtkWave, though.

I updated my commit with the requested changes.

Coverage remained the same at 100.0% when pulling 88cb6718d16b5a945377eb240ad2ee29a8307ec4 on pboettch:master into dbaeac9fa468b00cb3a12cdb774a6a6b1728716f on SanDisk-Open-Source:master.

Coverage remained the same at 100.0% when pulling 88cb6718d16b5a945377eb240ad2ee29a8307ec4 on pboettch:master into dbaeac9fa468b00cb3a12cdb774a6a6b1728716f on SanDisk-Open-Source:master.

Merged.
I appreciate this contribution. Thank you!

(my first python-code contribution ever) Thanks for creating this project.

Congrats! And thanks again--I appreciate your time and effort contributing this feature.


Fixed model flattening generating no fzn and ozn file

_solve (Or rather, the functions that _solve calls further down the execution chain) expects its mzn_file parameter to be strings, but None may be returned as fzn_file from mzn2fzn
I'm not familiar with the rest of the code structure, so this may not be the best place to implement this fix
Fixed #29

Hi @madsmtm! thank you for your contribution!
I'll hold on to this pull request until I figure out why the mzn2fzn call is returning None, which started happening with MiniZinc version 2.2.0. The last version of MiniZinc that PyMzn works fine with is the 2.1.7, so you can use that in the meantime if you don't need the newest features. Cheers!


Can't build the sample app

Briefly summarize your issue:
I'm trying to build the Sample App with the wake word enable with Kitt-AI but I got some issues.
I could run the following instruction:
cd $HOME/pi2/sdk-folder/sdk-build cmake $HOME/pi2/sdk-folder/sdk-source/avs-device-sdk \ -DGSTREAMER_MEDIA_PLAYER=ON \ -DPORTAUDIO=ON \ -DPORTAUDIO_LIB_PATH=$HOME/pi2/sdk-folder/third-party/portaudio/lib/.libs/libportaudio.so \ -DPORTAUDIO_INCLUDE_DIR=$HOME/pi2/sdk-folder/third-party/portaudio/include \ -DKITTAI_KEY_WORD_DETECTOR=ON \ -DKITTAI_KEY_WORD_DETECTOR_LIB_PATH=$HOME/pi2/sdk-folder/third-party/snowboy/lib/ubuntu64/libsnowboy-detect.a \ -DKITTAI_KEY_WORD_DETECTOR_INCLUDE_DIR=$HOME/pi2/sdk-folder/third-party/snowboy/include \ -DCMAKE_BUILD_TYPE=DEBUG
But when I run  make SampleApp -j2 , I got some errors.
What is the expected behavior?
The Sample App should be built
What behavior are you observing?
When I run  make SampleApp -j2, I get this:
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 31%] Built target AVSCommon
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 33%] Built target SQLiteStorage
[ 33%] Built target RegistrationManager
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 35%] Built target ADSL
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 37%] Built target InterruptModel
[ 42%] Built target ACL
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 42%] Built target Captions
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 42%] Built target AVSGatewayManager
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 42%] Built target Alexa
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 44%] Built target EqualizerImplementations
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 44%] Built target ContextManager
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 44%] Built target KWD
[ 46%] Built target PlaylistParser
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 48%] Built target SpeechEncoder
[ 48%] Built target SystemSoundPlayer
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 51%] Built target AudioResources
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 51%] Built target SpeakerManager
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 51%] Built target ApiGateway
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 53%] Built target AudioPlayer
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 55%] Built target Bluetooth
[ 57%] Built target Equalizer
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 57%] Built target InteractionModel
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 60%] Built target Notifications
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 60%] Built target PlaybackController
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 60%] Built target SpeechSynthesizer
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 60%] Built target TemplateRuntime
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 62%] Built target CBLAuthDelegate
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 64%] Built target SynchronizeStateSender
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 64%] Built target CertifiedSender
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 68%] Built target MediaPlayer
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 71%] Built target AFML
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 73%] Built target Endpoints
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 75%] Linking CXX shared library libKITTAI.so
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 77%] Built target ExternalMediaPlayer
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 80%] Built target DeviceSettings
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 80%] Built target CapabilitiesDelegate
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
Scanning dependencies of target AIP
/usr/bin/ld: cannot find -lblas
collect2: error: ld returned 1 exit status
KWD/KittAi/src/CMakeFiles/KITTAI.dir/build.make:124: recipe for target 'KWD/KittAi/src/libKITTAI.so' failed
make[3]: *** [KWD/KittAi/src/libKITTAI.so] Error 1
CMakeFiles/Makefile2:8096: recipe for target 'KWD/KittAi/src/CMakeFiles/KITTAI.dir/all' failed
make[2]: *** [KWD/KittAi/src/CMakeFiles/KITTAI.dir/all] Error 2
make[2]: *** Waiting for unfinished jobs....
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 80%] Building CXX object CapabilityAgents/AIP/src/CMakeFiles/AIP.dir/AudioInputProcessor.cpp.o
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 80%] Linking CXX shared library libAIP.so
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
/usr/bin/cmake: /usr/local/lib/libcurl.so.4: no version information available (required by /usr/bin/cmake)
[ 80%] Built target AIP
CMakeFiles/Makefile2:12409: recipe for target 'SampleApp/src/CMakeFiles/SampleApp.dir/rule' failed
make[1]: *** [SampleApp/src/CMakeFiles/SampleApp.dir/rule] Error 2
Makefile:3111: recipe for target 'SampleApp' failed
make: *** [SampleApp] Error 2
Provide the steps to reproduce the issue, if applicable:
$HOME/pi2/sdk-folder/third-party git clone https://github.com/kitt-ai/snowboy cd $HOME/pi2/sdk-folder/sdk-build cmake $HOME/pi2/sdk-folder/sdk-source/avs-device-sdk \ -DGSTREAMER_MEDIA_PLAYER=ON \ -DPORTAUDIO=ON \ -DPORTAUDIO_LIB_PATH=$HOME/pi2/sdk-folder/third-party/portaudio/lib/.libs/libportaudio.so \ -DPORTAUDIO_INCLUDE_DIR=$HOME/pi2/sdk-folder/third-party/portaudio/include \ -DKITTAI_KEY_WORD_DETECTOR=ON \ -DKITTAI_KEY_WORD_DETECTOR_LIB_PATH=$HOME/pi2/sdk-folder/third-party/snowboy/lib/ubuntu64/libsnowboy-detect.a \ -DKITTAI_KEY_WORD_DETECTOR_INCLUDE_DIR=$HOME/pi2/sdk-folder/third-party/snowboy/include \ -DCMAKE_BUILD_TYPE=DEBUG
Tell us about your environment:
Laptop Ubuntu Linux 16.04 LTS
What version of the AVS Device SDK are you using?
  <1.19.0>

Tell us what hardware you're using:

[x] Desktop / Laptop
[ ] Raspberry Pi
[ ] Other - tell us more:

Tell us about your OS (Type & version):

[x] Linux
[ ] MacOS
[ ] Raspbian Stretch
[ ] Raspbian Jessy
[ ] Other - tell us more:

Hi, can you confirm that you have successfully installed all the required dependencies of the SDK? Did you see any error messages when installing those dependencies?
Instructions for Ubuntu here: https://github.com/alexa/avs-device-sdk/wiki/Ubuntu-Linux-Quick-Start-Guide

I reinstalled everything and now it seems to work, maybe my dependencies were not installed correctly. Thank you for your advice.
Regards.

Hi, yes because I've build an another Sample App without the wake word and it's working.

The cmake output looks okay. Can you try re-installing the dependencies, and please share the output from doing so:
 sudo apt-get install -y \
 git gcc cmake openssl clang-format libgstreamer1.0-0 libgstreamer-plugins-base1.0-dev \
 gstreamer1.0-plugins-base gstreamer1.0-plugins-good gstreamer1.0-plugins-bad \
 gstreamer1.0-plugins-ugly gstreamer1.0-libav gstreamer1.0-doc gstreamer1.0-tools \
 pulseaudio doxygen libsqlite3-dev repo libasound2-dev

In your logs, it looks like a missing dependency: /usr/bin/ld: cannot find -lblas So let's verify that it is installed. Can you also provide the output for the following?
ldconfig -p | grep blas

You can try installing it like so:
sudo apt-get install libblas-dev

Hey,
The output from re-installing the dependencies is:
sudo apt-get install -y \
git gcc cmake openssl clang-format libgstreamer1.0-0 libgstreamer-plugins-base1.0-dev
gstreamer1.0-plugins-base gstreamer1.0-plugins-good gstreamer1.0-plugins-bad
gstreamer1.0-plugins-ugly gstreamer1.0-libav gstreamer1.0-doc gstreamer1.0-tools
pulseaudio doxygen libsqlite3-dev repo libasound2-dev
[sudo] password for mollyl:
Reading package lists... Done
Building dependency tree
Reading state information... Done
gcc is already the newest version (4:5.3.1-1ubuntu1).
libasound2-dev is already the newest version (1.1.0-0ubuntu1).
repo is already the newest version (1.12.32-2).
cmake is already the newest version (3.5.1-1ubuntu3).
doxygen is already the newest version (1.8.11-1ubuntu0.1).
git is already the newest version (1:2.7.4-0ubuntu1.9).
gstreamer1.0-doc is already the newest version (1.8.3-1~ubuntu0.1).
gstreamer1.0-plugins-base is already the newest version (1.8.3-1ubuntu0.3).
gstreamer1.0-plugins-good is already the newest version (1.8.3-1ubuntu0.4).
gstreamer1.0-tools is already the newest version (1.8.3-1~ubuntu0.1).
libgstreamer-plugins-base1.0-dev is already the newest version (1.8.3-1ubuntu0.3).
libgstreamer-plugins-base1.0-dev set to manually installed.
libgstreamer1.0-0 is already the newest version (1.8.3-1~ubuntu0.1).
libsqlite3-dev is already the newest version (3.11.0-1ubuntu1.4).
openssl is already the newest version (1.0.2g-1ubuntu4.15).
pulseaudio is already the newest version (1:8.0-0ubuntu3.11).
clang-format is already the newest version (1:3.8-33ubuntu3.1).
gstreamer1.0-libav is already the newest version (1.8.3-1ubuntu0.2).
gstreamer1.0-plugins-bad is already the newest version (1.8.3-1ubuntu0.2).
gstreamer1.0-plugins-ugly is already the newest version (1.8.3-1ubuntu0.1).
The following packages were automatically installed and are no longer required:
linux-headers-4.15.0-91 linux-headers-4.15.0-91-generic
linux-image-4.15.0-91-generic linux-modules-4.15.0-91-generic
linux-modules-extra-4.15.0-91-generic
Use 'sudo apt autoremove' to remove them.
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
The output for ldconfig -p | grep blas is libblas.so.3 (libc6,x86-64) => /usr/lib/libblas.so.3
I've installed it with the command you gave me.
I've run  make SampleApp -j2 and I think it's working now. Thank you!!

Great! I'll close this issue as resolved, then.

hello how could you help me with this problem that I am presenting when trying to install the main dependencies of the SDK.
sudo apt-get install -y \ gstreamer1.0-plugins-ugly gstreamer1.0-libav gstreamer1.0-doc gstreamer1.0-tools \

pulseaudio doxygen libsqlite3-dev repo libasound2-dev
Reading package list ... Done
Creating dependency tree
Reading status information ... Done
E: The package could not be located
E: Failed to locate repo package.


[libxml2] Update to 2.11.8 [CVE]

Fixes CVE-2024-34459.
Without the hassle of API changes which make #39478 a complex change.
Related: #39444.

Please review ASAP. Security fix w/o complex changes.

All features are tested successfully in the following triplet:
x86-windows
x64-windows
x64-windows-static


OneClass: an easy-to-use online platform that helps teachers and students collaborate, communicate and create!

Project name
OneClass
Description

OneClass is an easy-to-use online platform that helps teachers and students collaborate, communicate and create!
With OneClass,

You can access your assignments and add them to your Microsoft To Do.
You can access your class resources anytime and anywhere.
You can share your work with OneDrive.

OneClass is the ultimate tool for learning in the 21st century!
OneClass is built using ASP.NET Web APIs (.NET 6) that calls the Microsoft Graph API using the .NET SDK, and a React frontend.
Repo URL
https://github.com/sharry/OneClass
Team members
ElhoubeBrahim, abderrazzaq-laanaoui, StoicBug, sharry

Thank you for your submission @sharry! Would you be willing to record a short video to demonstrate the app? It's not required for the hackathon but it would help us showcase your work internally at Microsoft and publicly in the community 😊

Hey folks, I've got trouble running your hack. The instructions for SPA refer to npm start but you don't have a start script. Instead, I tried running npm dev but can't get past the login screen (after signing in with my account in the popup I'm getting redirected inside the popup to the /signin page).
The MAUI app doesn't seem to be doing anything: I can switch between the pages, but none of the buttons work. What am I missing?


Draw window before processing events, fixes #143

Fixes #143 (on my machine). It looks like commit 28f5ac90cc1ba7d668ae8a83eb5cd899294a8301 unintentionally removed a call to draw. Putting it back in appears to fix the issue. Will definitely need to verify on other people's machines.

That's a great find, thanks so much! v2.19.1 is on the way.


Missing example on cloudflare.PagesProject; missing accepted values documentation on nested field

Describe what happened
https://www.pulumi.com/registry/packages/cloudflare/api-docs/pagesproject/ is missing an Example section that would be helpful in showing users valid values for certain fields.
Upstream has an example: https://registry.terraform.io/providers/cloudflare/cloudflare/latest/docs/resources/pages_project although the field documentation is missing entirely (we're pulling ours from the schema).
Affected Resource(s)
cloudflare.PagesProject
Output of pulumi about
n/a; this is the latest published version of this provider to date.
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Thank you @guineveresaenger


Assistant for generating System Prompts

Is your feature request related to a problem? Please describe.
For new user it is hard to write a good system prompt.
Describe the solution you'd like
Use the template from openai playground for creating system prompts:
https://gist.github.com/philschmid/3a0ecc9e45763716f4dd9c36b6445fca

Done with #155


Extract core types into separate crate

Currently, the littlefs2 crate contains both a high-level interface and its implementation.  In frameworks like Trussed, we would like clients to be able to depend only on the high-level interface, while the Trussed runtime or the runners are providing the implementation details.  Therefore it would make sense to move the high-level interface into a separate crate, for example littlefs2-core.
The high-level interface would be:

Path/PathBuf
DynFile/DynFilesystem/DynStorage
all helper types required by those types

When working on Path/PathBuf, it would also make sense to:

remove panicking conversions, especially in From implementations, and replace them with fallible conversions
replace cstr_core::CStr with core::ffi::CStr
properly enforce the ASCII requirement even in Path
add conversions from the ascii crate and/or core::ascii

Regarding the Error type:
I would wrap the Error enum in a wrapper  type like std::io::Error and expose the current enum as #[non_exhaustive] enum ErrorKind without the Unknown variant through a fn error(&self) -> Option<ErrorKind> and a fn as_code(&self) -> u32 so that we can be sure not to have any breaking change releated to Error.
I would also avoid exposing littlefs-sys in the public core API, since non-breaking C changes can lead to breaking sys changes (adding new fields).
I would also remove public dependency on heapless for core (exposed through read_to_end) or put it behind a feature heapless-07 and add the heapless version name in the function name so that heapless can be upgraded.

This means that Error can contain an unknown error code without needing an Unknown variant. Otherwise an Unknownerror code that becomes known can break existing pattern matchings.

Ah, right, I misread as_code as only returning unknown codes.  Makes sense.


Eliminate use of ideviceinfo

This libimobiledevice tool is no longer used anywhere in the codebase.

https://github.com/flutter/flutter/blob/189028a958ba9c68d308148978a6eb99c4c472c9/dev/devicelab/lib/framework/adb.dart#L312 ?

Yep - it's back, which is why this PR isn't mergeable. Unfortunately I had to roll back a bunch of the work I'd done to eliminate the use of ideviceinfo Friday evening due to issues with the bots (see commit b2909a245a607995ce7ec286585cd1f643124f57).

Ah, got it.


Unable to run a ForEach loop into it

Been looking for something like this for ages tonight but ive found an annoying bug? when I run the ForEach with the code below the marquee restarts the instant the first text hits the edge of the frame.
Im trying to create a Marquee of game resutls but the second the edge of the first one hits the edge of the box it restarts the loop.
Marquee {         HStack{             ForEach(function.lastTen(), id: \.matchRecord.id) {index in                 HStack {                     Text("\(function.playerName(for: index.matchRecord.playerId))")                     Text("VS")                     Text("\(index.winnerName)")                     Image(systemName: "trophy.fill")                 }                 .padding(.trailing, 50)                 .font(.system(size: 20))                 .foregroundColor(.white)             }         }     }     .frame(width: 200, height: 50)     .background(Color.mint.opacity(0.8))     .marqueeDuration(duration)
If I run the code below without the external HStack however it loops round perfectlly like expected, but now my ForEach loop is displaying stacked ontop of each other not horizontally.

Hi @nameHabbo,
I had a look at this issue but was not able to reproduce it on my end 🤔
I'm doing something very similar in one of my apps, and it works fine. Could you share additional context or maybe also a little screen recording of the behaviour?

This is what I'm doing in my app:
Marquee {
    HStack {
        Text("(")
            .fontWeight(.regular)
            .foregroundColor(Color(UIColor.label))
        HStack(spacing: 30) {
            ForEach(regularPeriodScores, id: \.index) { periodScore in
                Text(periodScore.scoreInformation)
                    .tracking(10)
                    .fontWeight(.regular)
                    .foregroundColor(periodScore.isActive ? .appBlue : periodScore.isFinished ? .appRed : Color(UIColor.label))
            }
        }
        Text(")")
            .fontWeight(.regular)
            .foregroundColor(Color(UIColor.label))
    }
}
.frame(width: 200, height: 50)
.marqueeDuration(5)
.marqueeDelay(2)
.marqueeAutoreverses(true)
.marqueeWhenNotFit(true)
.marqueeBoundary(.inner)
.marqueeIdleAlignment(.center)

and this is how it looks:

I also tried to use a HStack inside the ForEach with multiple Text as you have it but it still worked as expected. Are you using Marquee from my fork or from this repo?

Hello @stonko1994 ,
Im gonna try the code you have within my xcode to see if its something im doing wrong or what im trying to inject because even when i was doing a very simple ForEach loop i was still getting the same issue, im also not sure why my code in the above comment looks like i did so trying to upload a video may not have gone well either haha. i will try and get back to you.
im also using the original package, shall i try yours instead??

im also using the original package, shall i try yours instead??

Yes, try my fork from https://github.com/stonko1994/Marquee. It does contain a few fixes and improvements.


Cannot reproduce THUMOS Kinetics result

Hi, I trained a model from scratch as written in readme, however, my mAP is only 69 ~ 69.5, which is lower than the reported performance 71.6 (I used kinetics for THUMOS) and LSTR
does this difference come from inference mode?
Did you get the result by running the script?
my torch version is 1.13.1+cu117

I guess the difference comes from the inference mode
in batch inference mode, I could get 71.3 ~ 71.6 mAP