error when trying to run the code

When I try to do anything, whether it is scaling an image, video or whatever, this error always comes out:
Traceback (most recent call last):
File "test_video.py", line 30, in
model.load_state_dict(torch.load('epochs/' + MODEL_NAME))
File "/usr/local/lib/python3.6/site-packages/torch/nn/modules/module.py", line 522, in load_state_dict
.format(name))
KeyError: 'unexpected key "block8.1.weight" in state_dict'
or:
Traceback (most recent call last):
File "test_video.py", line 30, in
model.load_state_dict(torch.load('epochs/' + MODEL_NAME))
File "/usr/local/lib/python3.6/site-packages/torch/nn/modules/module.py", line 526, in load_state_dict
raise KeyError('missing keys in state_dict: "{}"'.format(missing))
KeyError: 'missing keys in state_dict: "{'block7.1.running_var', 'block7.1.bias', 'block7.1.running_mean'}"'
I use ubuntu 18.04, pytorch 0.3.1 and cuda 9

@Jerchongkong The model has been modified, so the pretrained weight is wrong, you need train by yourself


slick in table

When slick in table it has wrong width.

When I add this css to Groonya's fiddle above, I have the same problem. I don't think the css is enough to fix the problem.
table {
table-width: fixed;
}

If the width of slick_div is set in css then the widths don't compound.


Cleanup Deployment always fails

The cleanup deployment workflow has never successfully run so far.
We should either fix or drop it.

A few comments related to the current state of the workflow:
https://github.com/opencast/opencast-editor/blob/1124fdcc0d2c7225c18a0828f62fe51db9732c00/.github/workflows/cleanup-deployments.yml#L17-L23
The key secrets.DEPLOY_KEY is the deployment key for the opencast-editor repository, not for opencast-editor-test. That would be secrets.DEPLOY_KEY_TEST. This shouldn't matter, though, since that's a public repository. You shouldn't need a key at all.
https://github.com/opencast/opencast-editor/blob/1124fdcc0d2c7225c18a0828f62fe51db9732c00/.github/workflows/cleanup-deployments.yml#L25-L26
By definition, steps already operate in GITHUB_WORKSPACE by default, so you can just omit that here and run rm -rf "editor-test/${{ github.event.number }}" instead. But you might actually want to run git rm instead of rm since you want to commit the changes later on.
https://github.com/opencast/opencast-editor/blob/1124fdcc0d2c7225c18a0828f62fe51db9732c00/.github/workflows/cleanup-deployments.yml#L29
You want working-directory: editor-test. You would need to use Action variables instead of environment variables here, if you would want to use them.  But there is no need since you operate in GITHUB_WORKSPACE by default anyway.
https://github.com/opencast/opencast-editor/blob/1124fdcc0d2c7225c18a0828f62fe51db9732c00/.github/workflows/cleanup-deployments.yml#L37-L42
This should fail since stefanzweifel/git-auto-commit-action@v4 seems to use the GITHUB_TOKEN for pushing to the git repository. And the one from this repository shouldn't have write access to another repository. You would need to do something similar to what the test deployment does.

That said, I'm also wondering if we wouldn't want to keep the deployments around for a while so that people can still quickly check them even after a pull request has been merged. This might be useful for checking recent pull request if a regression bug has been introduced?


Expose the num_classes argument when making models

Before:
$ python -c 'import torchvision; model = torchvision.models.resnet50(num_classes=100)'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
TypeError: resnet50() got an unexpected keyword argument 'num_classes'

After:
$ python -c 'import torchvision; model = torchvision.models.resnet50(num_classes=100)'

Haven't tested for all models. Feels like some basic CI for this repo would be a good idea.

Yeah, we should add Travis to this repo too.


feat: ✨ add token as permission constraints

Closes #5104
What's Changed

Added functionality to reference the current User in permission's constraints. That way an administrator can filter models per the active user (eg Change log)
Screenshots

TODO

[x] Explanation of Change(s)
[ ] Added change log fragment(s) (for more information see the documentation)
[ ] Attached Screenshots, Payload Example
[ ] Unit, Integration Tests
[x] Documentation Updates (when adding/changing features)
[ ] Example Plugin Updates (when adding/changing features)
[ ] Outline Remaining Work, Constraints from Design

My bad,  opened https://github.com/nautobot/nautobot/pull/5278 for next branch. I will also add tests, based on your suggestions.


navbar_description: Refactor help-widget icon display.

This PR introduces a minor refactor of how the help widget is displayed and aligned; it causes no changes to Firefox, but works around a paint bug present in Chrome where the icon remains visible when there is an ellipsis on the description.
CZO discussion
Screenshots and screen captures:

Chrome, before
Chrome, after

Before
After (no change)

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Merged, thanks @karlstolley!


Could I add an SSD1306 128x64 pixel OLED display to each side?

Thank you for your work—it inspired me to create a similar keyboard but with SSD1306 128x64 pixel OLED displays on each side. I'm a beginner, so I’d appreciate any advice on whether it would be possible to add these displays

Yes, it can, both OLED and trackpad use I2C communication protocols. But you need to add OLED code in your custom firmware.

Does this mean that I can use both the trackpad and display on one side at the same time?

sure, you can call them out.

Actually I often check crkbd and lily58's firmware to learn how to work. Even I am not an engineer.

Does this mean that I can use both the trackpad and display on one side at the same time?

https://docs.qmk.fm/features/oled_driver#other-examples


Fixing bundler error due to lack of ruby development package

Resolution
Adding in the ruby-dev package to the script/bootstrap-ubuntu script to prevent Ruby bundler from having issues compiling native extensions.
QA

Tested "vagrant up" successfully completes with no major errors

Previous Behavior
Vagrant would exit during the provisioning stage with a major error while running bundle.
Note: Using the st2express vagrant server, but this exists on both st2express and st2dev
> vagrant up st2express
...
==> st2express: Reading package lists...
==> st2express: Building dependency tree...
==> st2express: Reading state information...
==> st2express: Package rubygems is not available, but is referred to by another package.
==> st2express: This may mean that the package is missing, has been obsoleted, or
==> st2express: is only available from another source
==> st2express: However the following packages replace it:
==> st2express:   ruby
==> st2express: E
==> st2express: :
==> st2express: Package 'rubygems' has no installation candidate
...
==> st2express: Installing rake 10.3.2
==> st2express: Installing CFPropertyList 2.2.8
==> st2express: Gem::Installer::ExtensionBuildError: ERROR: Failed to build gem native extension.
==> st2express:
==> st2express:         /usr/bin/ruby1.9.1 extconf.rb
==> st2express: /usr/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require': cannot load such file -- mkmf (LoadError)
==> st2express: 	from /usr/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require'
==> st2express: 	from extconf.rb:2:in `<main>'
==> st2express:
==> st2express:
==> st2express: Gem files will remain installed in /opt/puppet/vendor/bundle/ruby/1.9.1/gems/hitimes-1.2.2 for inspection.
==> st2express: Results logged to /opt/puppet/vendor/bundle/ruby/1.9.1/gems/hitimes-1.2.2/ext/hitimes/c/gem_make.out
==> st2express: An error occurred while installing hitimes (1.2.2), and Bundler cannot continue.
==> st2express: Make sure that `gem install hitimes -v '1.2.2'` succeeds before bundling.
==> st2express: bundler: command not found: r10k
==> st2express: Install missing gem executables with `bundle install`
==> st2express: Error: Evaluation Error: Error while evaluating a Function Call, Could not find class ::st2::profile::fullinstall for st2express.stackstorm.net at /opt/puppet/environments/current_working_directory/modules/profile/manifests/st2server.pp:2:3 on node st2express.stackstorm.net
==> st2express: Error: Evaluation Error: Error while evaluating a Function Call, Could not find class ::st2::profile::fullinstall for st2express.stackstorm.net at /opt/puppet/environments/current_working_directory/modules/profile/manifests/st2server.pp:2:3 on node st2express.stackstorm.net
The SSH command responded with a non-zero exit status. Vagrant
assumes that this means the command failed. The output for this command
should be in the log above. Please read the output to determine what
went wrong.
...

New Behavior
Vagrant completes the 'vagrant up' command with no major errors
Note: Using the st2express vagrant server, but this exists on both st2express and st2dev
> vagrant up st2express...

Also note that you must clone a clean repo from st2workroom to reproduce this.

git clone git@github.com:StackStorm/st2workroom
cd st2workroom
vagrant up

:sunglasses:
Thanks for the code!


Release v1.0.0

Fix #108

This is good. :)
I reviewed the new documentation and ran it on my local.


Update for webpack 4

You need to remove the webpack var usage, then it should probably be OK :)

@fnlctrl I will review this pull request this evening :)

@woutervanvliet Added a minimize option, but I think warnings wouldn't be necessary since it's not required.

@fnlctrl Looks good!

@fnlctrl You have some code style errors though, if you can fix those then hopefully @oliviertassinari can merge soon after :)

@fnlctrl Looks like you need another "fis lint" commit.

Hmm running prettier isn't changing anything here locally. Not sure why it fails on CI

@fnlctrl It is, according to the diff.
Running prettier shouldn't change anything, unless you call it with --fix, but it should give you the same error as Travis CI does.

@woutervanvliet I meant --fix. It's not giving errors or making fixes locally..

Maybe it's not something that can be fixed automatically?
Just checked locally, and it's this change it wants you to make:
diff --git a/docs/src/sw.js b/docs/src/sw.js
index 4db6962..6339cea 100644
--- a/docs/src/sw.js
+++ b/docs/src/sw.js
@@ -116,9 +116,7 @@ self.addEventListener('fetch', event => {
         if (!responseNetwork || !responseNetwork.ok) {
           if (DEBUG) {
             console.log(
-              `[SW] URL [${requestUrl.toString()}] wrong responseNetwork: ${
-                responseNetwork.status
-              } ${responseNetwork.type}`
+              `[SW] URL [${requestUrl.toString()}] wrong responseNetwork: ${responseNetwork.status} ${responseNetwork.type}`
             )
           }

Which brings it back to the same as what's on the current master.
And you know what they say, you gotta do as the linter says you should ;-) Maybe your line length is set to something else then the projects prettier config?
Looking at package.json btw, I noticed that peerDependencies still specify support for webpack 1, 2 and 3 (not 4). Does this change not break anything for webpack 1, 2, or 3? Either way, webpack 4 should probably be in the allowed list of versions.

I'm sorry, I couldn't find the time to look into this pull request. While
beeing an early adopter of webpack, I have lost interest in this
serviceworker project. I'm no longer using it. My top priorities right now
are with Material-UI.
If anyone is interested in pushing the projet forward, please raise your
voice. I would be very happy to give you admin access to GitHub and npm.

I might be interested in at least bringing the plugin further to a webpack
4 compatible release. Have no previous experience though with webpack
plugins, other than using them...
On Sat, 10 Mar 2018, 14:31 Olivier Tassinari<EMAIL_ADDRESS>wrote:

I'm sorry, I couldn't find the time to look into this pull request. While
beeing an early adopter of webpack, I have lost interest in this
serviceworker project. I'm no longer using it. My top priorities right now
are with Material-UI.
If anyone is interested in pushing the projet forward, please raise your
voice. I would be very happy to give you admin access to GitHub and npm.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/oliviertassinari/serviceworker-webpack-plugin/pull/55#issuecomment-372030207,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHh2jB9fMI7JgUokCr5BQauEt7ATAQPks5tc9XKgaJpZM4STop-
.

@woutervanvliet I can support you, I have also no previous experience but we can figure it out. I think this plugin should stay alive because it's more flexible than the offline-plugin. 👍

Sounds great! I have added you as collaborators :).

@fnlctrl this error: 1 error, 0 warnings potentially fixable with the --fix option. comes from eslint. you can run ./node_modules/.bin/eslint . --fix to fix it. Or you enable eslint in your IDE to see the error in ./docs/src/sw.js and reformat it manually.

@woutervanvliet  @devCrossNet I did run ./node_modules/.bin/eslint . --fix but nothing happened. So I updated eslint related deps and it doesn't complain now. Though on newer versions of eslint react plugin and flow there are new errors. Leaving them as is as it's out of scope of this PR.

@woutervanvliet the PR LGTM, in my opinion, we could merge it. @oliviertassinari I think you are the only one who can publish a new version, this should be a new major version release because it has a breaking change (just works with webpack 4). @fnlctrl thanks for the PR 👍


Update tags support for Subnet

Description of changes:

Added custom code that supports updating tags for Subnet resource.
Added integration test to check whether tags are being updated as expected.

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

/ok-to-test

/retest

/retest

/retest

/retest

/retest


Handle multiple Doctrine managers

This is a merge request trying to solve #53
This is a work in progress, your contributions are welcome!
I've pushed a first proof of concept for now, there's still a lot to do:

[ ] Completely remove ObjectMetadataResolver::getObjectManager() in favor of ObjectMetadataResolver::getObjectManagerForClass(). Remaining usages:

[ ] DqlRule::processNode line 57
[ ] QueryBuilderDqlRule::processNode line 80
[ ] QueryBuilderGetQueryDynamicReturnTypeExtension::getTypeFromMethodCall line 71
[ ] ExpressionBuilderDynamicReturnTypeExtension::getTypeFromMethodCall line 71

[ ] There is not tests for setting an AbstractManagerRegistry instead of an ObjectManager for parameters.doctrine.objectManagerLoader
[ ] We might want to use another parameter as to not introduce confusion

With these changes, PHPStan is capable of analyzing simple code that uses multiple managers

The hard questions are:

What EM should we choose for DQL? How are DQL/QueryBuilders called when using ManagerRegistry in Doctrine in general?
What should we do when the user provides ManagerRegistry to PHPStan but in code we get EntityManager without any context?

Please open a new PR if you decide to work on this again. Thanks.


Chnage to async handlerbars

FHIR converter is great, but your handlebars is not async, so making calls to anything external, like a database call to get patient id's, causes blocking issues.  I suggest upgrading your handlebars to async, so that this is more production ready.  We have already proven this in concept in a private fork.  It should be easy enough for you to do.

Thanks for your suggestion, we will put it into our backlog and discuss this requirement within our team first.


⚠ ClamAV has degraded performance

In 42328cb, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 201 ms

Resolved: ClamAV performance has improved in adfe46b.


UI: Augur platform Activity Stats

We'll get another augur-node endpoint for user stats to populate this section. augur-node tix: https://github.com/AugurProject/augur/issues/1192

moved this to in progress, augur-node endpoint is ready

@bthaile -- Awesome, gonna move this into this sprint.

"ready" meaning that the stubbed out endpoint is on master. not that we are getting real data.

wired up, so we don't get console errors when trying to connect to augur-node platform stats endpoint.

no console errors - closing


fix: schedule eslint-related updates

:tada: This PR is included in version 1.8.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Permissions to only view the WUfB report?

The document explains the permissions to set-up WUfB, how to "display the workbook and view the Windows tab in the Software Updates page [Microsoft 365 admin center]", but does not explain the permissions to simply view the report without providing "Global Reader" to the entirety of Azure.
I tried the "Log Analytics Reader" role at the resource group level, but this does not work. Is it possible to give granular access to just this report?

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 9724d64b-716d-9191-8cb3-021b118b2d37
Version Independent ID: 9724d64b-716d-9191-8cb3-021b118b2d37
Content: Windows Update for Business reports prerequisites - Windows Deployment
Content Source: windows/deployment/update/wufb-reports-prerequisites.md
Product: windows-client
Technology: itpro-updates
GitHub Login: @mestew
Microsoft Alias: mstewart

@mestew Please help us in resolving this issue. Thanks

Any updates to this?

To make it easier for you to submit feedback on articles on learn.microsoft.com, we're transitioning our feedback system from GitHub Issues to a new experience.
As part of the transition, this GitHub Issue will be moved to a private repository. We're moving Issues to another repository so we can continue working on Issues that were open at the time of the transition. When this Issue is moved, you'll no longer be able to access it.
If you want to provide additional information before this Issue is moved, please update this Issue before December 15th, 2023.
With the new experience, you no longer need to sign in to GitHub to enter and submit your feedback. Instead, you can choose directly on each article's page whether the article was helpful. Then you can then choose one or more reasons for your feedback and optionally provide additional context before you select Submit.
Here's what the new experience looks like.
Note: The new experience is being rolled out across learn.microsoft.com in phases. If you don't see the new experience on an article, please check back later.
First, select whether the article was helpful:

Then, choose at least one reason for your feedback and optionally provide additional details about your feedback:

Article was helpful
Article was unhelpful

Finally, select Submit and you're done!


Typo corrected

State abbreviation was incorrect.

Thanks for catching 👍

It’s not a draft. I forgot to remove that status. It’s can be changed.Sent from my iPhoneOn Jun 26, 2024, at 4:35 AM, Stefan Münnich @.***> wrote:
@annakijas1 Is this PR marked as draft on purpose?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

Perfect. Thank you!


[WIP] Add typing hints from wsgiref.types

Fixes: https://github.com/encukou/freezeyt/issues/291
(To be rebased on top of #312)

[WIP] means work in progress?

[WIP] means work in progress?

yes, I am continue with rest of missing types


Quick open: include history results when searching by name

In VSCode, Ctrl/Cmd-P "Go to file", in particular lets me find open files quickly, but doesn't let me find HtmlViews, such as previews for Markdown files, or VSCoq ProofViews (https://github.com/coq-community/vscoq/issues/144).
That's despite the fact that ProofViews show up in the drop down list. To wit, here are ProofViews (at least, once you select the right pane):

and here they disappear in the fuzzy search:

@Blaisorblade did you try Ctrl+Tab for a pure searchable list of editors?

I don't see a (fuzzy or not) search in it:
. I didn't know about it.
The real question might be "why do you have enough non-file editors that you'd want fuzzy search".

I don't see a (fuzzy or not) search in it:

(tho I didn't know about it before).
The real question might be "why do you have enough non-file editors that you'd want fuzzy search"; and I'm not sure my answer generalizes enough.

@Blaisorblade I mean this:

It brings up a quick open for editors that are open where you can search.

Oooh, thanks a lot! That's what I wanted 🎉 ; closing!


Enhance in-memory operations

LoadFromData no longer writes data to temporary file before loading.
LoadFromData takes an io.Reader for input instead of byte slice.
SaveConfigData added, which writes configuration to an arbitrary writer.
ReloadData added to reload data from memory.

Note that you cannot mix in-memory configuration with on-disk configuration.

Sorry, didn't mean to PR you. I see you no longer accepts enhancements, I will keep this as a local fork.


Generate cc and header files for micro_speech.

Generate the model and audio inputs. Add wav file generation in generate_cc_arrays.py.
BUG=196065539

I added pip3 install Wave wherever we have pip3 install Pillow, and I don't see any pip commands in xtensa. Perhaps I'm looking in the wrong place, however.


cannot use plugin

i follow the README in here https://ionicframework.com/docs/native/apple-pay/#installation
and the plugin i can see it in node_module and config.xml,
but when i run the app on my phone,
[16:59:31]  console.log: Angular is running in the development mode. Call enableProdMode() to enable the production
            mode.
[16:59:31]  console.log: Ionic Native: deviceready event fired after 611 ms
[16:59:32]  console.warn: Native: tried calling ApplePay.canMakePayments, but the ApplePay plugin is not installed.
[16:59:32]  console.warn: Install the ApplePay plugin: 'ionic cordova plugin add cordova-plugin-applepay'
[16:59:32]  console.log: plugin_not_installed

It seems like you didn't actually install the plugin either with cordova plugin add or by using cordova platform add (which should also install plugins).

thank you @ajcrites , but im sure plugin was installed.
i copy the ionic cordova plugin add cordova-plugin-applepay from warnning, and i checked the node_module and config.xml, everything looks good.

Closing. Please open another issue with more details if this is still a problem.


refactor: prefer builtin Buffer utils over ponyfills

This PR contains:

[ ] bugfix
[ ] feature
[x] refactor
[ ] tests
[ ] documentation
[ ] metadata

Breaking Changes?

[ ] yes
[x] no

If yes, please describe the breakage.
Please Describe Your Changes
I see builtin buffer utils were not used because of old node support.
Though they are dropped now. And there is no need in these dependencies.
https://github.com/mafintosh/csv-parser/pull/83#issuecomment-377797828
Though I hope this change will not recover this problem.
https://github.com/mafintosh/csv-parser/issues/81

Codecov Report

Merging #169 into master will decrease coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #169      +/-   ##
==========================================
- Coverage   97.24%   97.20%   -0.04%
==========================================
  Files           1        1
  Lines         145      143       -2
==========================================
- Hits          141      139       -2
  Misses          4        4

Impacted Files
Coverage Δ

index.js
97.20% <100.00%> (-0.04%)
:arrow_down:

Continue to review full report at Codecov.

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

@TrySound  conflicts

Fixed


Renamed newClient() to createClient()

As we discussed earlier it's better to name the function createClient() instead of newClient() by nodejs naming conventions.

Awesome! Thank you @googoid.


add a linter workflow

Added a GitHub action that uses the actions/super-linter GitHub
action available on the GitHub actions Marketplace
https://github.com/github/super-linter
The action will run on modified files in a pull request. Enabled
linting for Bash, C, Go, Python Rust and Markdown.
Signed-off-by: Elza Mathew<EMAIL_ADDRESS>

I'd like to see the results of this. Can you convince the CI to run again? The actinos we run is based on an "allowed list" in teh settings. I added the linter action, but I don't know how to get the actinos to run when one failed because it wasn't allowed to run.
I tried clicking the "Re-run all checks" button from the actinos screen, but that didn't work. I think you'll need to force push a change, or close and create a new PR.

sure, thanks Jeff, Keith. I'll close this one and re-create it to get the job to run..


Intel100 Series/C230 Series Chipset Family HD Audio Controller Not working

Live, Build 0I39

Is it working on GhostBSD?

It didn't work either, nor did it drive.

It didn't work either, nor did it drive.

Then it is probably not supported in FreeBSD. If you find a solution that makes it work in FreeBSD, please let me know and I can add that solution to helloSystem.


0.1.2 of @radix-ui/react-id no longer exports IdProvider

Hi! Lovely library. Thank you - started using it on small new projects.
Current Behavior
If I install with yarn add<EMAIL_ADDRESS>and then import:
import { IdProvider } from '@radix-ui/react-id';

... IdProvider is no longer exported.
TS will complain:
Module '"@radix-ui/react-id"' has no exported member 'IdProvider'.ts(2305)

I was using this in a Next.js app, and the problem first manifest itself as:
Warning: React.jsx: type is invalid -- expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.

... since I was wrapping my whole app with this and it was returning undefined.
Expected behavior
The following should work, particularly since it's encouraged in the documentation on server-side-rendering.
import { IdProvider } from '@radix-ui/react-id';

Reproducible example
Suggested solution
Re-export it? Or make a major version change if you're removing an existing export.
Your environment
yarn 1.22.17

+1, we are experiencing this as well.

Sorry about that, we are in the process of doing a new release but the docs aren't quite out yet, that's why we haven't officially announced yet.
The release docs will highlight the breaking changes as they always do.
The reason it's not a new "major" version is because we are still in the 0.1.x range as we are in beta.
Bare with us, it should all be out properly tomorrow.

Thanks for the quick response @benoitgrelard .
And we totally understand that 0.x.x ranges are outside of semver :)

@ericclemmons thanks.
Worth reiterating perhaps, but we do recommend to pin the versions to avoid these kind of suprises whilst we're not yet in final v1.
In the meantime, you can already see here ahead of time the upcoming release notes as well as the related docs changes.

Apologies for the broken builds!
To help others avoid this, we're going to publish a new patch with an empty IdProvider that console.warns a deprecation link to the docs. We're also working on getting to semver. In the meantime, as Benoit says, we recommend pinning versions until the Primitives hit 1.0.

More info on the actual change: Once you're using React 18, primitives use the new useId hook automatically 🎉 While using older versions of React, Radix components will now re-render during client hydration. So the IdProvider is no longer necessary.
Thanks for the report and understanding!

The IdProvider has been released with the warnings mentioned above. https://github.com/radix-ui/primitives/pull/1037

Is it possible to not log the warning in production? It looks like it is getting sent to sentry on every page load.
React 18 is still in beta, not all libraries are updated to work with it yet and some companies can't or aren't comfortable with running beta software.

Hey Kyle - that's a good idea, we will remove the log in prod. That said, if you're using primitives that are logging the warning, you should be able to remove the IdProvider now, even if you're not using React 18 yet.

@StephenHaney ah I understand now, I thought the options were either use IdProvider or use React 18. Thanks for clarifying that, I appreciate it!


Progressbar: Fix issue with color changing when updating value while error state

Description

Fix issue with color changing when changing the value. That was caused as we moved to a visual state that didn't set the color to our error color resulting in that switch.
Also updated the template to use the SystemControlErrorTextForegroundBrush. For reasons unknown to me, lightweight styling of that resource does not really work. The themeresource lookup doesn't seem to pick up if that resource is specified on that page.
Another thing I noticed is that switching themes DOES NOT update the progressbar color since we set the color of a solidcolorbrush instead of referencing a themeresource. What should we do with this?
While those animations seem awesome, they are currently create a few issues that we wouldn't have with plain setters to SolidColorBrushes.
Motivation and Context

Fixes #1766
How Has This Been Tested?

Screenshots (if appropriate):

/azp run

@chingucoding The pipeline fix just went in, could you please merge master into this PR?

/azp run

/azp run

@kmahone do you know what is going on with the test failures for this PR? (we had the same error twice in a row)

@chingucoding I think the Progressbar tests work item is failing because it is taking too long, probably because there are too many failing tests. You have confirmed that they pass locally right?

Just ran the tests again on my machine, and all of them run fine:

Can you try putting the tests in different suites? hopefully with that we can get the pipeline to give us more diagnostics.

Can you try putting the tests in different suites? hopefully with that we can get the pipeline to give us more diagnostics.

Done hopefully that helps tracking the issue down.

/azp run

Looking through the change history, I believe SystemControlErrorTextForegroundBrush was added in RS5, the majority of the downlevel failures are the binding to that resource failing.  I think we need to do something like what we do for SystemErrorTextColor (here)[https://github.com/microsoft/microsoft-ui-xaml/blob/777976f410a881fa98c0b064bb237f6d29e0fa83/dev/CommonStyles/ProgressBar_themeresources.xaml#L13]

I added the polyfill now, two things I noticed:

Don't we also need a polyfill for high contrast (added them now, but not sure on the colors)
Why are the ProgressBar themeresources under the CommonStyles project, not the ProgressBar project?

yes, although if a theme change happens and a theme resource isn't defined in the new theme's dictionary the color remains the same, which is why we never cought this as an issue. however if you enter high contrast mode and then launch the mux controls test app and navigate to the progress bar page it crashes cause the initial lookup fails to find it.  Good catch.

Edit* its actually crashing for a different reason.. for some reason in HighContrast mode the call to GetVisualStateGroups here
https://github.com/microsoft/microsoft-ui-xaml/blob/777976f410a881fa98c0b064bb237f6d29e0fa83/dev/ProgressBar/TestUI/ProgressBarPage.xaml.cs#L28
returns null. I looked briefly but can't explain why it behaves differently for high contrast.. @ranjeshj or @MikeHillberg, I'm not really sure who to ask about VSM.

Guarding against this in the test page itsself probably doesn't help here right? If that code doesn't happen we will probably have test failures because we needed those events. I'm a bit surprised that in high contrast, that call has a different result, after all the template is not aware of high contrast.

I think this is because we lifted the template before we lifted the control.  If you'd like to fix that it would be a good change.

Moved it inside the ProgressBar project now.

Yeah, I'm also surprised... I'm hoping someone else from the team might have incite as to whats going on, looking through the code I still have no idea.

/azp run

Last build failure was due to the fact that we need some special properties inside the page entry in the vcx file, which visual studio does not add on it's own.

/azp run

Yeah, I'm also surprised... I'm hoping someone else from the team might have incite as to whats going on, looking through the code I still have no idea.

The only difference between those should be the resources right ? I'm curious if some resource in high contrast is incorrect causing the VSM state parsing to fail - although I would have expected it fail/throw right there instead of returning null. Could you try to see what happens if you replace the high contrast resources with the default resources and see if it still repros ? perhaps we can track this down that way. @stevenbrix @alwu-msft any ideas why we might be seeing this behavior ?

The only thing that comes to mind (note that this is pure speculation) is that the template lives in a theme dictionary (so there's different templates for different themes) and the template contained in the high contrast theme dictionary doesn't have a VSM specification. My recommendation would be to debug through the call to retrieve the visual state groups... unfortunately the people who have the most knowledge of how VSM works have long since left the team so I don't have any better suggestions.

I was going to say something similar to Alan, although we don't generally put templates in theme dictionaries.
VSM silently fails, since @chingucoding doesn't have access to our source yet, maybe you can find something in the output spew?

The only other thing that comes to mind is that something about the way vsm is deferred is causing hiccups.
@chingucoding can you set <DisableXbfGeneration>true</DisableXbfGeneration> in the project file and see if the issue still reproes?

Oops, my bad, "debug the call" isn't going to be too useful then. :D
In that case, if nothing is obvious from the debug spew, then capturing a Time Travel Debugging trace and attaching it to a new issue would allow somebody from our team to step through the scenario and figure out what's going wrong.

I filed #3289 for tracking, unfortunately TTD didn't work as expected. Hopefully the DLM file is enough to help debugging, otherwise, the repro steps are also mentioned in the issue.
Given that the CI is running now, is there any work left to do in this PR?

Given that the CI is running now, is there any work left to do in this PR?

Is the crash we're seeing new behavior, or is this already existing?

The crash occured sometime ago in this PR but is gone now as the missing high contrast resource is added now.

The crash occurs in the mux controls test app without this change.

My old comment was wrong, @StephenLPeters is totally right.

@chingucoding I'm very confused, which comment? 😄
I'd still like @YuliKl to comment on whether using SystemControlErrorTextForegroundBrush is a necessity, or if we can use SystemErrorTextColor where we just need a Color

/azp run

Is there anything missing in this PR or are there any changes that need to be made?


Any input throws:  Error compiling to JavaScript: XMLHttpRequest error.

New to dart pad, however even the most basic code throws "
Error compiling to JavaScript:
XMLHttpRequest error."
https://dartpad.dartlang.org/
(just trying default print for example)
Is the site not working properly at the moment, or am I missing something?
Thanks,
Chuck.

Thanks for filing an issue!
I'm not having trouble compiling things right now, but that doesn't mean there's not something weird going on. Can you provide a little more info about the browser and operating system you're using?
Also, do you see anything weird in your browser's JavaScript console or network traffic log?

Hi Andrew,
Here's what I'm seeing-
[image: image.png]
Just trying to run:
void main() {
var = myName('John');
print(myName);
}
And receiving this:
Error compiling to JavaScript: XMLHttpRequest error.
Using: Chrome  Version 77.0.3865.90 (Official Build) (64-bit)
Also nothing strange on the network call
[image: image.png]
I am at work and behind a firewall [Forcepoint], perhaps that's the reason?
Thank you,
Chuck.
On Fri, Sep 20, 2019 at 7:19 PM Andrew Brogdon<EMAIL_ADDRESS>wrote:

Thanks for filing an issue!
I'm not having trouble compiling things right now, but that doesn't mean
there's not something weird going on. Can you provide a little more info
about the browser and operating system you're using?
Also, do you see anything weird in your browser's JavaScript console or
network traffic log?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/dart-pad/issues/1285?email_source=notifications&email_token=ABBSGFRIROXAFW3QCY4AJQ3QKVR7RA5CNFSM4IY2TGLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7IGEXA#issuecomment-533750364,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABBSGFSKBRVX5UU2GPZDXDTQKVR7RANCNFSM4IY2TGLA
.

Hi Andrew,
Tried it on my phone and works fine, sorry to both you, must be Forcepoint
blocking : (
Thanks,
Chuck.
On Mon, Sep 23, 2019 at 9:58 AM Chuck Steneck<EMAIL_ADDRESS>wrote:

Hi Andrew,
Here's what I'm seeing-
[image: image.png]
Just trying to run:
void main() {
var = myName('John');
print(myName);
}
And receiving this:
Error compiling to JavaScript: XMLHttpRequest error.
Using: Chrome  Version 77.0.3865.90 (Official Build) (64-bit)
Also nothing strange on the network call
[image: image.png]
I am at work and behind a firewall [Forcepoint], perhaps that's the reason?
Thank you,
Chuck.
On Fri, Sep 20, 2019 at 7:19 PM Andrew Brogdon<EMAIL_ADDRESS>wrote:

Thanks for filing an issue!
I'm not having trouble compiling things right now, but that doesn't mean
there's not something weird going on. Can you provide a little more info
about the browser and operating system you're using?
Also, do you see anything weird in your browser's JavaScript console or
network traffic log?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/dart-pad/issues/1285?email_source=notifications&email_token=ABBSGFRIROXAFW3QCY4AJQ3QKVR7RA5CNFSM4IY2TGLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7IGEXA#issuecomment-533750364,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABBSGFSKBRVX5UU2GPZDXDTQKVR7RANCNFSM4IY2TGLA
.

Looks like your image attachments didn't attach. It'd be interesting to understand how your firewall is blocking the content, so we might be able to sniff that at run time and at least fail gracefully.

Likewise. If you have time to save the network traffic generated by loading the site and trying to run something (ZenDesk has a handy guide), we'd definitely be interested in seeing it.
I'll go ahead and close this one out, though.

I solve this using a VPN.

 That worked for me.


(bug) loading openaerialmap image stops cesium renderer

Description:
I've loaded https://openaerialmap.org/ image into cesium, and the renderer stopped
when i navigate to other locations the map is blurred as if the tiles are not loaded, but in the chrome dev-tools network tab i see that the tiles looks perfect
viewer.imageryLayers.addImageryProvider(new Cesium.UrlTemplateImageryProvider({
url: "https://tiles.openaerialmap.org/5cba78c0761d2e00050690d9/0/5cba78c0761d2e00050690da/{z}/{x}/{y}"
}));
expected:
load the openaerialmap image and be able to navigate to other locations
actual:
after openaerialmap image is loaded cesium tile renderer stops rendering new tiles
Sandcastle example:
https://cesiumjs.org/Cesium/Apps/Sandcastle/#c=dVNhb9owEP0rVr4QJOYEOmBkgFbRTULqRrfCPkWqjuSg1hwb2Q4tjfLf55BkJFRLlMh3fs/n53sORSiOoMiR4QsqMiMCX8gCNUsT+vucczvROVxIYYAJVJ0eyUIRmsvLEtijOj0oeWQxqqC5SKQQDK4OKB6NQjTf4bBsw92MpIoHpPNszEEHngfUMI5UWo4+cxI4UKn2Xofk3V679BY03sMJ1QOL/hSljUqxCcm7n8NKIy9wNiLVU47Kf6mfRpCgArrjp7V0sws0Rm2YAMOkCGplC1DGjkDc0J2SyR3u7V61eyn+YeLT/sdRY8c3Azoc9BuJ/sD366iQVteTiqE97qIeCUiWN6YimRw4GgyI2yWz+VUv/o2bFATNxP5bKqKWgK+tNP25ub37dbteLp6WP55Wm3XJr85Po1kKg+oI3N1VDFs/a51e5YNzPzSFOL7udMMXG8XXaJVYc7zzQ63ibIvQqX1RuEKfbQGoGPDaFsNoC+NPkT8e9eMB+r4/9EcTP554/v+mwMveci97td8pD52yXt4tpOY9Mra4UrXTc6banDjOS51fWHKQyhR+dSn1TCVAe9vUus/QSOuCOPVq0jRmR8LiWehcXaLQIREHre3MLuX8kb1h6MynnsW3aFxCbJu0OqKy9i0gz/35fZmklE49G75nGSn5FlRjxb8
Browser:
Chrome: Version 75.0.3770.80 (Official Build) (64-bit)
Operating System:
win 10 64 bit

Thanks for reporting this @asafMasa. This is a regression from when we implemented the ImageBitmap decoding (https://github.com/AnalyticalGraphicsInc/cesium/pull/7579). The problem happens when an imagery request returns 204 (no content). With the ImageBitmap code-path, it just returns undefined, where it should be rejecting that promise instead:
https://github.com/AnalyticalGraphicsInc/cesium/blob/master/Source/Core/Resource.js#L1875
I'm going to open a PR with a fix shortly. In the mean time, as a work-around, you can disable ImageBitmap decoding by putting this snippet at the top of your code, before creating the viewer:
Cesium.Resource.supportsImageBitmapOptions = function(){
    return Cesium.when.resolve(false);
};

I think this is the same issue as reported by https://github.com/AnalyticalGraphicsInc/cesium/issues/7817. Your code example helped a lot in tracking this down!

thanks!
it works 👍


existing-mnemonic option isn't available

After building per docs at https://eth2-docker.net/docs/ the "Configure Wallet" step https://eth2-docker.net/docs/Usage/ConfigureWallet instructs to test the keys generated offline with sudo docker-compose run --rm deposit-cli existing-mnemonic but the option isn't available:
user@host:~/eth2-docker$ sudo docker-compose run --rm deposit-cli existing-mnemonic
[sudo] password for user:
Please choose your mnemonic language (spanish, italian, czech, english, chinese_traditional, chinese_simplified, korean) [english]:
Please choose how many validators you wish to run: 1
Type the password that secures your validator keystore(s):
Repeat for confirmation:
Usage: deposit.py new-mnemonic [OPTIONS]
Try 'deposit.py new-mnemonic --help' for help.

Error: Got unexpected extra argument (existing-mnemonic)
user@host:~/eth2-docker$ sudo docker-compose run --rm deposit-cli existing-mnemonic --help
Usage: deposit.py new-mnemonic [OPTIONS]

  Generate a new mnemonic and keys

Options:
  --mnemonic_language [spanish|italian|czech|english|chinese_traditional|chinese_simplified|korean]
                                  The language that your mnemonic is in.
  --num_validators INTEGER RANGE  The number of validators keys you want to
                                  generate (you can always generate more
                                  later)  [required]

  --folder DIRECTORY              The folder to place the generated keystores
                                  and deposit_data.json in

  --chain [mainnet|witti|altona|medalla|spadina|zinken|pyrmont]
                                  The version of eth2 you are targeting. use
                                  "mainnet" if you are depositing ETH

  --keystore_password TEXT        The password that will secure your
                                  keystores. You will need to re-enter this to
                                  decrypt them when you setup your eth2
                                  validators. (It is reccomened not to use
                                  this argument, and wait for the CLI to ask
                                  you for your mnemonic as otherwise it will
                                  appear in your shell history.)

  --help                          Show this message and exit.

The option does work when I download the binary manually using instructions here: https://github.com/ethereum/eth2.0-deposit-cli#for-linux-or-macos-users

Thanks!  I will look at this over the weekend and update the docs repo with info.
https://github.com/eth2-educators/eth2-docker-docs
Do you by chance know what version of the deposit-cli was being used by docker?  The Dockerfile sources the main repo, so I wonder if it's as simple as a docker-compose pull --no-cache

pushed up a fix to dev
https://github.com/eth2-educators/eth2-docker-docs/pull/1
https://dev.eth2-docker.net/docs/Usage/ConfigureWallet#test-your-seed-phrase
should be in the next doc release.
Thanks!


Support custom .obsidian config directory

Describe the bug
Recent version of Obsidian supports changing config directory from .obsidian to some custom value. For example, this is useful if one wants to have different plugin set for desktop and mobile apps, or in case synchronization software doesn't support hidden directories at all. Currently, the plugin searches for the token file in .obsidian. It should access new Vault.configDir field if it's present, falling back to default .obsidian value if that's not present.
To Reproduce
Steps to reproduce the behavior:

Open Obsidian
Copy your .obsidian directory as .obsidian.desktop.
Open Settings
Open 'About' screen
Put .obsidian.desktop in "Override config folder".
Close Obsidian
Delete .obsidian directory.
Open Obsidian again.
Note that the old directory is back again, the plugin created an empty file for the token. It also asks to enter the token again.

Expected behavior
Plugin should search for the token file in new config directory.

Whoops -- I accidentally just filed a duplicate bug report in #170! I'll just copy over the only bit not in this issue:
In obsidian-todoist-plugin/src/token.ts you are hard-coding the location of the todoist api to the .obsidian folder:
export function getTokenPath(): string {
  return `.obsidian/todoist-token`;
}

Implemented by @jamiebrynes7 in #168. Thank you!


OSError: No such device (os error 19)，how to save this problem?

how to save this problem?

try to add os.environ["FLUX_DEV"] = "/your/path/flux/model/FLUX.1-dev/flux1-dev.safetensors" in py

it does not work out

try to add os.environ["FLUX_DEV"] = "/your/path/flux/model/FLUX.1-dev/flux1-dev.safetensors" in py

bash
export FLUX_DEV= /your/path/flux/model/FLUX.1-dev/flux1-dev.safetensors
export AE= your path
it works .

bash export FLUX_DEV= /your/path/flux/model/FLUX.1-dev/flux1-dev.safetensors export AE= your path
it works .

yeah，it is in the usage:
export FLUX_SCHNELL=<path_to_flux_schnell_sft_file>
export FLUX_DEV=<path_to_flux_dev_sft_file>
export AE=<path_to_ae_sft_file>
I mean the path should point to the .safetensors，works just ok.


KOPT: move to new apply_opt, ng suggest

redo of #2029
need to move the kopt stuff off the main optimization path
as always, can use more testing

Hmm, this needs to be a red diff. Unless there's a reason not to, we have to commit and rip out the old stuff.

Eh, i guess we just move it to extra until the current RL contest experiments conclude.

I deleted apply_auto_opt this morning.

nevergrad is kill


QA: Anchor View samples button to samples section on experiment page

Context
When I click on the View Samples button, it takes me to the top of the experiment page.
Problem or idea
It should link to the samples section on the experiment page instead
Solution or next step
Tagging @davidsmejia for next steps

Hello @dvenprasad, This behavior is the same as the current refine.bio. The data in the samples table needs to be fetched first when the experiment page loads. For this, it displays a loading spinner animation when a user enters the page.
Do you want to add an auto-scroll-down to the samples table section after the data has finished loading? Please let me know!

Yes, it typically doesn't take more than a few seconds! The page renders a loading spinner while fetching the submitter supplied information and the samples table section. The data in the samples table is loaded using a separate hook. The page will render the table UI once the data is loaded (with populated HTML element's properties with that fetched data) and auto-scroll it to the top of the page.


Change key name for CentOS for parallel node image building

Parallel node image building is failing because of the same key name for Ubuntu and CentOS. This PR will change the key name for CentOS node image builiding.

/cc @lentzi90
/cc @mboukhalfa


Exit Code: AZCM0070: Unable To Obtain Local Config

Please add details of error "Exit Code: AZCM0070: Unable To Obtain Local Config" and also reason and remediation for this error.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c60c61b5-2eba-723c-27c1-4296763fcb28
Version Independent ID: f642a961-06ed-beac-fb5b-c0d69bf5897c
Content: Troubleshoot Azure Arc-enabled servers agent connection issues - Azure Arc
Content Source: articles/azure-arc/servers/troubleshoot-agent-onboard.md
Service: azure-arc
Sub-service: azure-arc-servers
GitHub Login: @johnmarco
Microsoft Alias: johnmarc

Assigning to @johnmarco for document enhancement.

Error code AZCM0070 has been added to article.
#please-close

@johnmarco Thank you.


Fix issue with project selection in field slip form

coverage: 93.472%. remained the same
when pulling c83ddfbbfc8f658f210de5d228368df088c127fc on njw-fix-field-slip-project
into ac632c45bb19d63525f9987fdc23e416a02261a2 on main.


video is not saving in android cwac

Hi,
I am using compile 'com.commonsware.cwac:camera:0.6.2' . For my video i wrote simply
case R.id.img_video:
try {
Utility.showLog("Vieo called","video called");
mCameraView.record();
} catch (Exception e) {
e.printStackTrace();
}
break;
case R.id.img_stopcame:
try {
mCameraView.stopRecording();
} catch (IOException e) {
e.printStackTrace();
}
break;
I have take all the three permissions.
When i click stop i got MediaRecorder: mediarecorder went away with unhandled events
please help me

As is noted in the documentation, this library is no longer supported. As is noted on the project home page, please use some other library.

If i want to use portrait video recording which i have to use. ?


64 bit annotations only shown with 53 bit precision in UI

We have a few binary annotations in our spans (emitted by Finagle using Trace.binaryAnnotation)
      {
        "key": "XXXXXXId",
        "value":<PHONE_NUMBER>42246336,
        "type": "I64",
        "endpoint": {
          "serviceName": "XXXXXXX",
          "ipv4": "10.19.XXX.XXX",
          "port": 10579
        }
      },

In the UI, however, the value shows as<PHONE_NUMBER>42246400, due to the 53 bit javascript limitation. Perhaps I64 should be sent as strings from the back-end?

Yes, we should send them as strings!

yeah please don't use types besides string (default). They are only present
as it was in the original thrift definition. However they are not
supported. When logging binary annotations for key lookup etc, stringify
them

I suspect this problem is on the zipkin side. The JSON that comes from the Java server could be bad. If it sends the IDs as numbers instead of strings? Or are they sent as strings from the Zipkin server?

Numbers.
The JSON above is copied from the response from /api/v1/trace.

I suspect this problem is on the zipkin side.

depends on the level of abstraction we define for this :) we've not
"supported" anything besides string for user-defined binary annotations and
almost certainly future formats won't support it. When people file issues
like this we should probably let them know that this is not a feature that
works (ex search won't work etc).

The JSON that comes from the Java server could be bad. If it sends the IDs
as numbers instead of strings? Or are they sent as strings from the Zipkin
server?

Ids are sent as strings, but this is a binary annotation. We could define a
string encoding for when people use the unsupported I64 type, just I'm not
sure how productive it is to invest in that.

I guess it is probably this..
def recordBinary(key: String, value: Any): Unit = {
record(Annotation.BinaryAnnotation(key, value))
}
Meh.. ok :) I'll raise a PR since yeah only thing we could do to help
is change zipkin-finagle to re-type the thing.

https://github.com/openzipkin/zipkin/pull/1373

fix going out now in 1.14.2


Turn off overlays by default

When reviewing a change we most likely don't want to use local overlays as it can give false results.
I ran into this due to suddenly getting an error from an overlay, tried to pass --args overlays [] as a build-arg, but that didn't work as I guess the flags aren't used when generating the build list.

Does this fix your problem? https://github.com/Mic92/nix-review/pull/30

--build-args are currently not passed to nix-shell, which is a bug.

However I should probably have different flags for nix-shell, nix build and both.

Hmm, that's weird it doesn't for some reason.
Here's the error:

error: while evaluating 'callPackageWith' at /home/hed/.cache/nix-review/rev-46352d5e95ee7766398f7298527707b79f6b5852/nixpkgs/lib/customisation.nix:113:35, called from /home/hed/src/nixpkgs-mozilla/vidyo-overlay.nix:4:18:
while evaluating 'makeOverridable' at /home/hed/.cache/nix-review/rev-46352d5e95ee7766398f7298527707b79f6b5852/nixpkgs/lib/customisation.nix:72:24, called from /home/hed/.cache/nix-review/rev-46352d5e95ee7766398f7298527707b79f6b5852/nixpkgs/lib/customisation.nix:117:8:
anonymous function at /home/hed/src/nixpkgs-mozilla/pkgs/VidyoDesktop/default.nix:1:1 called without required argument 'libGL', at /home/hed/.cache/nix-review/rev-46352d5e95ee7766398f7298527707b79f6b5852/nixpkgs/lib/customisation.nix:74:12

So did you use ./bin/nix-review from the repository for that?
I don't see any other way this could be passed to nixpkgs: https://github.com/NixOS/nixpkgs/blob/master/pkgs/top-level/impure.nix#L42
Does something tries to re-import nixpkgs?

Yeah, tried with ~/path/to/bin/nix-review and nix-env

Ah. I see. nix-env would bypass that. How about the following: https://github.com/Mic92/nix-review/pull/31

Yes, that worked.
Thanks :)


ScoreFactory not found

Pmmp version: 3.14.1
Running from source(cuz i cant get the .phar)
I've already tried installing the virion of scorefactory, but i keep getting the same error. Any help?
Error:
11.07 03:01:25 [Server] Server thread/CRITICAL Error: "Class 'JackMD\ScoreFactory\ScoreFactory' not found" (EXCEPTION) in "plugins/UHC-master/src/uhc/game/GameHeartbeat" at line 247 11.07 03:01:25 [Server] Server thread/DEBUG #0 plugins/UHC-master/src/uhc/game/GameHeartbeat(86): uhc\game\GameHeartbeat->handleScoreboard(object pocketmine\Player) 11.07 03:01:25 [Server] Server thread/DEBUG #1 plugins/UHC-master/src/uhc/game/GameHeartbeat(61): uhc\game\GameHeartbeat->handlePlayers() 11.07 03:01:25 [Server] Server thread/DEBUG #2 src/pocketmine/scheduler/TaskHandler(134): uhc\game\GameHeartbeat->onRun(integer 281) 11.07 03:01:25 [Server] Server thread/DEBUG #3 src/pocketmine/scheduler/TaskScheduler(179): pocketmine\scheduler\TaskHandler->run(integer 281) 11.07 03:01:25 [Server] Server thread/DEBUG #4 src/pocketmine/plugin/PluginManager(654): pocketmine\scheduler\TaskScheduler->mainThreadHeartbeat(integer 281) 11.07 03:01:25 [Server] Server thread/DEBUG #5 src/pocketmine/Server(2391): pocketmine\plugin\PluginManager->tickSchedulers(integer 281) 11.07 03:01:25 [Server] Server thread/DEBUG #6 src/pocketmine/Server(2154): pocketmine\Server->tick() 11.07 03:01:25 [Server] Server thread/DEBUG #7 src/pocketmine/Server(1994): pocketmine\Server->tickProcessor() 11.07 03:01:25 [Server] Server thread/DEBUG #8 src/pocketmine/Server(1588): pocketmine\Server->start() 11.07 03:01:25 [Server] Server thread/DEBUG #9 src/pocketmine/PocketMine(273): pocketmine\Server->__construct(object BaseClassLoader, object pocketmine\utils\MainLogger, string[48] /, string[56] /plugins/) 11.07 03:01:25 [Server] Server thread/DEBUG #10 src/pocketmine/PocketMine(304): pocketmine\server() 11.07 03:01:25 [Server] Server thread/DEBUG #11 (11): require(string[108] phar:///PMMP-Latest-Stable.phar/s) 11.07 03:01:25 [Server] Server thread/EMERGENCY An unrecoverable error has occurred and the server has crashed. Creating a crash dump 11.07 03:01:25 [Server] Server thread/EMERGENCY Please upload the "/crashdumps/Sat_Jul_11-03.01.25-UTC_2020.log" file to the Crash Archive and submit the link to the Bug Reporting page. Give as much info as you can.

You're using the outdated ScoreFactory, the new one is https://poggit.pmmp.io/ci/stephen-mc-pl/ScoreFactory, build 11 is what you'd want for PM3.


Add a BoxShadow component.

As proposed in issue #203.
Options for light location (i.e. shadow position)
I don't really have an opinion on this, but here are some options:
Light in the center (original proposal)

Light in the top center (MacOS-ish)

Light in the top-left (Android Material design ish)

@Dretch the PR looks great!
My only question is if there's any configuration attribute to select the orientation of the shadow (based on the attached images). Thanks!

@Dretch I usually avoid exposing AlignH/AlignV in configuration options, using instead the CmbAlignLeft, etc combinators. This way users can do:
boxShadow_ [alignRight, alignBottom] $
  ...

For the internal representation it makes sense to use AlignH/AlignV as you suggest. The box and popup components follow this pattern. In both cases, if conflicting options are added (alignLeft and then alignRight), the last one takes precedence.
I think having a default in the theme and the ability to customize it in the widget itself would be ideal.

@Dretch looks great! Thanks!

@Dretch released!
I'm still waiting for a new Stackage LTS for GHC 9.2. I've been trying the nightlies and they work well, even fixing issue #1 (although I will need to change the Setup page to recommend installing Stack with ghcup).

@Dretch have you released Hagrid? Would it be ok if I mention it in Monomer's Readme? Thanks!


support filter with predicate in BetaRowset

Support predicate filter in BetaRowset iterator.
It's better if we can support lazy materialization.

If you mean filter rows by columns's zonemap, it's already supported

@gaodayue

If you mean filter rows by columns's zonemap, it's already supported

I mean that vectorized predicate filter. In AlphaRowset, this logic locates in ColumnData, we should support it in SegmentIterator to filter data as soon as possible.

I see, thanks

I will do this feature

I will split this job into several step:

I will move original v1's predicate logic to segment v2 to provide this feature quickly.
I will refactor the current column predicate framework because current column predicate is based on row block. I will modify it to calculate based on column cell. This work wil be the base of the following work.
I will realize lazy materiazation to optimize data read.
I will realize the optimization to push predicate evalution to page decoder which will optimize in some situation, eg: dict encoded page.

#1775


Allow use of fs.FS for $INCLUDE and wrap errors

This adds ZoneParser.SetIncludeAllowedFS, to specify an fs.FS when enabling support for $INCLUDE, for reading included files from somewhere other than the local filesystem.
I've also modified ParseError to support wrapping another error, such as errors encountered while opening the $INCLUDE target.  This allows for much more robust handling, using errors.Is() instead of testing for particular strings (which may not be identical between fs.FS implementations).
ParseError was being constructed in a lot of places using positional instead of named members.  Updating ParseError initialization after the new member field was added makes this change seem a lot larger than it actually is.
The changes here should be completely backwards compatible.  The ParseError change should be invisible to anyone not trying to unwrap it, and ZoneParser will continue to use os.Open if the existing SetIncludeAllowed method is called instead of the new SetIncludeAllowedFS method.

fs.FS was added in Go 1.21, which I think meets the "two latest major versions" criteria (latest being 1.22).  Let me know if you're concerned about keeping support for go 1.20 and below, and if so I can hide the new functionality behind a build tag.  (My plan to do so would be move SetIncludeAllowedFS into a separate file and shadow the fs.FS interface.)
Error wrapping was added in Go 1.13, so I don't anticipate any version concerns there.

(Deleted previous comment where I completely got wrong when fs.FS was introduced.  Sorry for the noise.)
fs.FS was introduced in Go 1.16: https://go.dev/doc/go1.16#fs
Error wrapping was introduced in Go 1.13: https://go.dev/doc/go1.13#error_wrapping

brilliant idea!
Code looks good, bit messy indeed due to the error changes. Some comments incomding


Automatically detect directory in page name

As per discussion in #10. The separate directory parameter in various methods should be removed and automatically detected in the page name. This will need to be done sympathetically so as to minimise the impact on existing code.

Thankfully, there's nothing to prevent an app built with gollum-lib from ignoring the directory parameter. So we faked this behavior in rails-wiki.

#216 will help with this.

This has been implemented in #322 and #326 -- will be released in 5.x


null reference errors

found a couple of null reference errors.

Excellent!

From: Demetrios Pagonis @.>
Sent: Friday, June 7, 2024 3:42:05 PM
To: dpagonis/chemical_safety @.>
Cc: Chad Thompson @.>; Author @.>
Subject: Re: [dpagonis/chemical_safety] null reference errors (PR #1)
Merged #1https://github.com/dpagonis/chemical_safety/pull/1 into main.
—
Reply to this email directly, view it on GitHubhttps://github.com/dpagonis/chemical_safety/pull/1#event-13085772836, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAK33NOJB5EBO5325OUAPLLZGISK3AVCNFSM6AAAAABI7ILLH6VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTGA4DKNZXGI4DGNQ.
You are receiving this because you authored the thread.Message ID: @.***>


Allow .expanding() to be called on groupby object

Right now when I want to have expanding operate at the level of a group I have to apply a convenience function like so:
def expanding_sum(s):
    return s.expanding().sum()

expanded_sum = df.groupby('mygroup')['mynum'].apply(expanding_sum)

Would be nice to be able to call a method directly on a groupby object:
expanded_sum = df.groupby('mygroup')['mynum'].expanding().sum()

related to #12486
yes this should be possible - have to manage the dispatching correctly


Minor style fixes for mobile devices

Styling is a little funky is some areas on mobile

tested local, looks better


Add babel transformers to support ie8

If you use Redux from NPM as CommonJS modules, it's up to you to minify it any way you like (you get to pick Uglify settings in your own project obviously).

https://github.com/reactjs/redux/issues/774#issuecomment-142260526

@ppoliani It dosen't work  '^'


Update strike.is-cool.me.json

Requirements

[x] You have completed your website.
[x] The website is reachable.
[x] The CNAME record doesn't contain https:// or /.
[x] There is sufficient information at the owner field.

Description
Birthday site sry
Link to Website
https://deathofme.vercel.app/

I have read the CLA Document and I hereby sign the CLA


ClientCredentialsTokenEndpointFilter has hardcoded No OP successHandler

i'm trying to integrate the authentication of oauth client credentials with our own session management. i thought having a successHandler to notify the session management it can create a session
but even if supplied a successHandler the default behavior is no-op as it is hardcoded in the class itself afterPropertiesSet()
setAuthenticationSuccessHandler(new AuthenticationSuccessHandler() {
        public void onAuthenticationSuccess(HttpServletRequest request, HttpServletResponse response,
                Authentication authentication) throws IOException, ServletException {
            // no-op - just allow filter chain to continue to token endpoint
        }
    });

this forces one to extend and change to a natural  behavior.
thanks
Shlomi

I assume that was intentional. Why do you need to customize it? Normally this filter is upstream of a regular Spring Security filter that handles the success failure in the normal customizable way. If you can provide a bit more context (e.g. a sample project) that would help a lot.

i thought that was the whole point of the setAuthenticationSuccessHandler to be able to hook into successful authentication.
in our case - there is a need to notify another component when successful authentication happens.

AuthenticationSuccessHandler is normally used to redirect to a saved request or a home page. That doesn't seem appropriate with the token endpoint, so I probably don't understand your use case yet. If all you want is a notification of an authentication you can use ProviderManager as your AuthenticationManager and it will publish events accordingly (if it's ApplicationEventPublisher is not-null).

again without a proper documentation - one cannot know this is the meaning of success handler.
when given this API, i might do what ever i want with a success handler., in our case set a session attribute for example.

Closing this as questions are better suited on Stack Overflow. We prefer to use GitHub issues for bugs and enhancements.


CLOUDP-92575: Enable M0 instance size for QuickStart command

Further comments
!! Won't be merged until the Public API endpoint is working. !!

Proposed changes

Jira ticket: CLOUDP-92575
Checklist

[ ] I have signed the MongoDB CLA
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have added any necessary documentation (if appropriate)
[ ] I have updated e2e/E2E-TESTS.md (if a new command or e2e test has been added)
[x] I have run make fmt and formatted my code

Thanks for your first PR, I know it's still draft until we get the backend ready but wanted to leave you a couple of tips here.
One final comment, we have e2e tests that create a cluster, https://github.com/mongodb/mongocli/blob/8037a765f4b875c2f5c36b76e3ff3b247d53a76a/e2e/atlas/clusters_test.go
I have schedule these tests to make sure nothing breaks but I'd like to move those to try create an M0, for context right now those create an M30 as that's the min tier available for Atlas Gov, so the change here may need to duplicate that file, one for public Atlas and one for Atlas gov but we can discuss that later


add Medizinische Universität Wien

the university official website URL, if it is different from the domain you are submitting

https://www.meduniwien.ac.at/

a URL of a page on the official website where a long-term (>1 year) IT related course is offered by the university

https://www.meduniwien.ac.at/web/studium-weiterbildung/masterstudium-medizinische-informatik/

a URL of a page or some other proof (.pdf or a screenshot are OK) showing that the university recognizes the domain which you are submitting as an official email domain for the enrolled students.

https://webmail.students.meduniwien.ac.at/

@sLatzko Pull request merged. Thank you!


duke, arrangement

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 100
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[a,d]

/trunk merge


Steeltoe.Management.Endpoint.Loggers.LoggersEndpoint not overridable

The Loggers endpoint takes an ILoggerProvider and then tries to cast it to a DynamicLoggerProvider.
Could this possibly be changed to cast to an IDynamicLoggerProvider interface so we could inject in a custom one?

Thanks @dwilliss ... change is available on dev feed and will be included in next service release for 2.1

Thanks!

Any idea when the next service release of 2.1 will be?

@dwilliss We should have the release of 2.1.1 out this week.

Yay!  That worked.... Something is creating a logger named Steetoe though.  I don't see that in a global search of any of my code.


MODULES-11201: add serivce_name for Ubuntu 18.04 and later

This will set $service_status for Ubuntu hosts

 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.

My PR has been merged. Could you rebase this? After that I think this should be good to merge.

I did the rebase.


Cypress plugin doesn't follow paths

I'm trying to set up Knip in an existing project. I discovered that the Cypress plugin doesn't follow paths as inferred from tsconfig.json.
My cypress.config.ts files import some common functions from a lib that is accessed via a path. When I run Knip, I get the following error:
$ npm run knip

><EMAIL_ADDRESS>knip
> knip

Analyzing workspace ....
Error loading /Users/stephen.wade/git/knip-issue-repro-stunning-happiness/cypress.config.ts
Reason: Cannot find module '@/cypress'
Require stack:
- /Users/stephen.wade/git/knip-issue-repro-stunning-happiness/cypress.config.ts

If I rename cypress.config.ts to main.ts, Knip says there are no issues, demonstrating that Knip can get the paths properly in that case.
Instructions to reproduce:

Clone the minimal reproduction: https://github.com/stephenwade/knip-issue-repro-stunning-happiness
npm ci
npm run knip – The error will happen.
mv cypress.config.ts main.ts && npm run knip – The error will not happen.

The issue is probably that the compilerOptions.paths are not loaded/used during script execution of cypress.config.ts (this uses jiti for script loading).
You can try the same using Bun: bunx --bun knip (this uses bun for script loading).

Thanks for checking, I need to document this (e.g. using TS aliases in dynamic Jest configs has the same issue I believe)

:rocket: This issue has been resolved in v5.1.5. See Release 5.1.5 for release notes.


Compacter stops with single recipe mode

When compacter is single recipe and the inventory is full they stop working, issue happen when turning iron blocks into iron ingots. After removing some stacks of iron blocks the compacter works, looks like an internal inventory problem.

The mod has been recently updated to MC 1.16, older versions are no longer supported.
I'm mass closing older issues to clear up clutter in the bug tracker.
If you find this issue still exists in the current version feel free to reopen it.


Write tests for functions that call send

Using test vectors and by mocking the send or card.transmit functions, unit tests for functions that call the send function can be written. However mocking a pcsc::Card proved to be difficult, an issue is created in pcsc GitHub: https://github.com/bluetech/pcsc-rust/issues/50.
The functions that can get unit tests after this are:

read_data_from_ef,
establish_bac_session_keys,
Protected APDU creation or parsing of the send function,
process_secure_rapdu.

I think you can create a trait like:
pub trait MrtdCard {

    /// Returns answer to reset
    fn atr(&self) -> Result<Vec<u8>>

    /// Returns R-APDU including 2 bytes SW
    fn transmit(&self, c_apdu: &[u8]) -> Result<Vec<u8>>

}

so that your wrapper takes a generic param:

pub struct EmrtdComms<C: MrtdCard> {
    // ...
}

impl<C: MrtdCard> EmrtdComms<C> {
    // ...

}

Thank you for the tip @dishmaker. Implemented this solution in #4, also implementing a trait based solution for the Rng.


Block exporter view

Basic framework for UI support of block exporting. Marks the official end of block_exporter refactoring.

This change is

Review status: 0 of 2 files reviewed at latest revision, 2 unresolved discussions.

src/view/app_view.js, line 218 at r1 (raw file):

   * Calls response to selecting export project in menu.
   */
  exportProject(){

Nit but space between close parentheses and open bracket (?) 0:

src/view/navigation_tree.js, line 185 at r1 (raw file):

        action: //TODO: add function for exporting associated element
      }
    }

Add semicolon?

Comments from Reviewable

Review status: 0 of 2 files reviewed at latest revision, 2 unresolved discussions.

src/view/app_view.js, line 218 at r1 (raw file):
Previously, chynu (celine choo) wrote…
Nit but space between close parentheses and open bracket (?) 0:

Done.

src/view/navigation_tree.js, line 185 at r1 (raw file):
Previously, chynu (celine choo) wrote…
Add semicolon?

Done.

Comments from Reviewable


lease: unlock before another batch opeartion

lgtm after fixing typo in commit title

lgtm. Ah this was why test was failing.


SHIP-0028: BuildRun cleanup

Looking for some feedback already especially on the TBDs inside the ship.

/assign ENCALADA
/assign HeavyWombat

/assign qu1queee

@qu1queee @gabemontero I added a commit with updates. Those mainly cover two things:

Describing the behavior when Build updates happen after BuildRuns were completed.
I changed the implementation considerations to use new controllers for the cleanup. Knowing the complexity of the BuildRun reconciler, I feel much better with this approach as it would have meant many additional if-style code in the existing one and the increased complexity would have further increased the risk that we oversee certain scenarios.

Please have a look. Thank you.

I've got nothing to add @SaschaSchwarze0 . I see @qu1queee added a couple of comments and while I don't think he is asking for changes (instead just making anecdotal comments) I'm not 100% sure, so I'll refrain on adding a lgtm label, along with I see you have @adambkaplan listed as an approver, so I'm taking that as you wanting him to take a pass at this as well before it merges.

I added the remaining small changes (status of conditions as mentioned by Enrique, and the SHOP vs SHIP typo found by Raghav). I also change the approvers section. I only initialized them that way. Commits also squashed. Looking for a lgtm. @gabemontero ;-)


Implement ModelType interface for more torchvision models

Currently we have SSD and fasterrcnn implemented for object detection (https://github.com/eto-ai/rikai/tree/main/python/rikai/pytorch/models)
To make it easier for pytorch users, we should consider implementing additional models from (https://pytorch.org/vision/stable/models.html). At the very least, we should have 2 or 3 image classification models, and 1 video classification model implemented.

Hint: apply_model_spec can be used to developing ModelType:
https://github.com/eto-ai/rikai/blob/c300eeede0a19c798b70bf7466e1faf0a482a092/python/rikai/testing/utils.py#L60-L67

Closed via #606


[Bugfix] - Only create one result for exam quiz evaluations

Fixes https://github.com/ls1intum/Artemis/issues/2908
Checklist

[x] I tested all changes and all related features with different users (student, tutor, instructor, admin) on the test server https://artemistest.ase.in.tum.de.
[x] Server: I followed the coding and design guidelines.
[x] Server: I implemented the changes with a good performance and prevented too many database calls
[x] Server: I documented the Java code using JavaDoc style.
[x] Client: I translated all the newly inserted strings into German and English

Motivation and Context

The evaluate button in the student exams in only supposed to be clicked once. When clicking multiple times, the newly created results have no reference to the participation. As a result, the amount of points the student got is not shown in the view anymore.
Description

We deactivated the creation and saving of new results after the first evaluation.
Steps for Testing

Log in to Artemis
Create a exam with an (short answer) quiz question
Participate
After the exam is over, click on evaluate once in the student exams page
Click on "View" to view the student participations and make sure that every student has the correct result/amount of points in the table
Go back to the student exams page and click on evaluate again
Check that the results still are correctly displayed
Should also work for other quiz exercise types

Re-evaluate

Also, please check that the re-evaluation of short answer quizzes remains correct
Exercise-groups -> Re-evaluate
The "Add solution" button for short answer quizzes should now be called "Add Short Answer Solution" (Also check german translation is correct "Short Answer Lösung hinzufügen")
add a new solution (Which in the best case, a student typed in somewhere so you can check it gets counted as correct)
Make sure to map it to the spot where you want the option to be a correct input
save the re-evaluation and check that the student now get credits for this spot

Thinking task for Dev's
Could not creating new results after the first evaluation be a problem somewhere I am not aware of?
Screenshots
How the student participation view should not look after the second/third/fourth... evaluate

How it should look like:

@tiffca I am just here for the Bugfix :P If you mean the boxes around the words, these are probably for mapping purposes.

We should definitely improve the user experience when adding new solutions, but the origin of the issue seems to be fixed. Thanks


feat(seed): Add tests and solution to Problem: Largest Prime Factor

Pre-Submission Checklist

[x] Your pull request targets the staging branch of freeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[ ] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[ ] Small bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[ ] Closes currently open issue (replace XXXX with an issue no): Closes #XXXX

Description

Merged! Thanks for the pull request.


⚠ Bofrost.es has degraded performance

In 17aa185, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1842 ms

Resolved: Bofrost.es performance has improved in b5ffa66 after 14 minutes.


feat(census-directory): initial style scaffolding for census directory page

Reason for Change

#6066

Looks good, @seve ! I just have a couple comments.

We should be using ${spaces?.xxs}px between the ItemLabel and its value.
I'm not sure something is misconfigured with SDS, but the button fontWeight should be semibold not 800.
And could you overwrite the min-width from 120px to 80px for the buttons?

Regarding max-width:
The project descriptions should have max-width: 85ch and the the overall content should have max-width: 1000px.


CMake Error at /usr/local/share/cmake-3.19/Modules/FindPackageHandleStandardArgs.cmake:218 (message):   Could NOT find Boost (missing: Boost_INCLUDE_DIR date_time filesystem   iostreams)

I met an error when installing denserflow:
How to solve it ? Thanks !
(base) root@LAB_VM:/home/denseflow/build# cmake -DCMAKE_INSTALL_PREFIX=$HOME/app -DUSE_HDF5=no -DUSE_NVFLOW=no ..
-- Found OpenCV: /usr/local (found version "4.5.0")
-- OpenCV library status:
--     config: /usr/local/lib/cmake/opencv4
--     version: 4.5.0
--     libraries: opencv_calib3d;opencv_core;opencv_dnn;opencv_features2d;opencv_flann;opencv_gapi;opencv_highgui;opencv_imgcodecs;opencv_imgproc;opencv_ml;opencv_objdetect;opencv_photo;opencv_stitching;opencv_video;opencv_videoio
--     include path: /usr/local/include/opencv4
CMake Error at /usr/local/share/cmake-3.19/Modules/FindPackageHandleStandardArgs.cmake:218 (message):
  Could NOT find Boost (missing: Boost_INCLUDE_DIR date_time filesystem
  iostreams)
Call Stack (most recent call first):
  /usr/local/share/cmake-3.19/Modules/FindPackageHandleStandardArgs.cmake:577 (_FPHSA_FAILURE_MESSAGE)
  /usr/local/share/cmake-3.19/Modules/FindBoost.cmake:2193 (find_package_handle_standard_args)
  CMakeLists.txt:29 (find_package)

-- Configuring incomplete, errors occurred!
See also "/home/denseflow/build/CMakeFiles/CMakeOutput.log".
See also "/home/denseflow/build/CMakeFiles/CMakeError.log".

The error message says that boost is not installed. Try zzboost.sh?

Thank you, i will give it a try tomorrow !

After installing boost, this error has gone away, but when i try to reinstall denseflow, there coms up another error:
(base) root@LAB_VM:/home/denseflow/build# make -j
[ 16%] Building CXX object CMakeFiles/zzdenseflow.dir/src/denseflow_gpu.cpp.o
/home/denseflow/src/denseflow_gpu.cpp:2:10: fatal error: opencv2/cudaarithm.hpp: No such file or directory
 #include "opencv2/cudaarithm.hpp"
          ^~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
CMakeFiles/zzdenseflow.dir/build.make:107: recipe for target 'CMakeFiles/zzdenseflow.dir/src/denseflow_gpu.cpp.o' failed
make[2]: *** [CMakeFiles/zzdenseflow.dir/src/denseflow_gpu.cpp.o] Error 1
CMakeFiles/Makefile2:96: recipe for target 'CMakeFiles/zzdenseflow.dir/all' failed
make[1]: *** [CMakeFiles/zzdenseflow.dir/all] Error 2
Makefile:148: recipe for target 'all' failed
make: *** [all] Error 2

It seems my OpenCV not iinstalled with CUDA, but when i try to reinstall OpenCV, i met an error, so i submit an issue, Do you have time to take a look ?

You can follow our instruction to install cuda version of opencv https://github.com/open-mmlab/denseflow/blob/master/INSTALL.md

@innerlee Hi, i see ROOTDIR=${ZZROOT:-$HOME/app} in the shell script, does ZZROOT suit for everybody like me ?

For anyone reaching this page, I've bumped OpenCV to 4.10 and fixed two CMAKE flags in zzopencv.sh.
Complete modified version: zzopencv.sh
Have fun!


GamesList route unavailable

https://m.np.playstation.com/api/gamelist/v2/users/{id}/titles?limit=20&offset=0
The given service is returning 503 for the past 2/3 days.
This route should return, among other things, the playDuration for a user's game.
Someone is aware of that?

I'm not seeing that issue.
Have you tried different network troubleshooting like using a different DNS server?

Really? That's weird. I'm reaching the API actually. I'm getting this.

Really? That's weird. I'm reaching the API actually. I'm getting this.

{
    "error": {
        "reason": "ServiceUnavailable",
        "source": "Gamelist",
        "code": 3239938,
        "message": "Service Unavailable at Auth",
        "referenceId": "b0caa9fb-8ae6-1031-ad77-877297fc3296"
    }
}

Interesting. If you try to access it without any credentials do you still get the same message, or do you get the same service unavailable error?
{
    "error": {
        "reason": "MissingAuthCredentials",
        "code": 3239954,
        "source": "Gamelist",
        "message": "No credentials supplied",
        "referenceId": "178277cf-8ab2-1031-a267-913c3579bfd6"
    }
}

Well... this route was off september 5th but turned back in a couple of hours, I've just saw the token had expired and it was returning exactly the same 503 payload.
It's working now. Sorry for the misundertanding and thanks again for the support.


Disordering from rawQuery to rawSql in 0.9.12/0.9.13

The issue does not appear in releases before 0.9.12.
[AND flowType IN (1,2) AND sourcePodNamespace NOT IN ('kube-system', 'flow-visibility', 'flow-aggregator')]()
↓
[FROM flows_pod_view WHERE 2) AND flowType IN (1, sourcePodNamespace NOT IN ('kube-system', 'flow-visibility', 'flow-aggregator')]()

They are being synced. They have the same content.
I think I found the issue, seems to be parsing issue.
When the WHERE clause is AND flowType IN (1,2), it will fail;
when adding a space between 1 and 2 -> AND flowType IN (1, 2), it works.

@bossinc AST issue?


Improvement: Include other libraries from the Erlang standard library in the releases

Environment

VerneMQ Version: Any released version.
OS: Any
Erlang/OTP version (if building from source): -

Expected behavior
This is really an improvement for plugin development. While developing some plugins I had to use a library that manipulates XML and that requires xmerl from the standard library and that is not shipped when VerneMQ is released and there is no way to get it into the plugin because is not a third party dependency that can be installed by rebar3. Will be great to have a chat about what libraries, more specifically for file manipulation can be added to the releases in order to make plugin development easier.
Actual behaviour

There are some crucial packages from the standard library that can be needed for plugin development and they are not shipped with the release, making some useful plugins not possible to develop(In my case i'm adding an AWS cloudwatch plugin to report VerneMQ metrics that will be opensource to help the community here).

Good point. I see 2 possibilities:

Go full monty (what would that mean?)
Choose a reasonable set of useful libraries to include (and what would that mean?)

How important is a minimal footprint? What libs should we exclude deliberately? (for security or whatever reasons)

Yeah that analysis sounds reasonable @ioolkos, what I can think about from looking at other Erlang-based projects like RabbitMQ, eJabberd, and MongoseIM, this is pretty much what folks include:

sasl
crypto
public_key
ssl
mnesia
inets
compiler
syntax_tools
os_mon
xmerl

I'm don't personally think mnesia is needed in the context of VerneMQ.

I also don't think we should add mnesia which leaves xmerl and inets which we don't already include with VerneMQ I'm ok for adding those. As mentioned in another issue we could also include etop if people think that's a handy tool.

cool @larshesel I will add inets to my PR, it can also be useful. I have another branch for etop, will PR after GDPR madness.

Think we can close this now that is merged on https://github.com/erlio/vernemq/pull/697


initscript: /var/lock/subsys directory can be missing

Hi,
I tried today updating to last version 0.10.0.
I got following error when starting/restarting daemon on Ubuntu Trusty:
Starting daemon go-carbon : touch: cannot touch ‘/var/lock/subsys/go-carbon’: No such file or directory [FAILED]
After doing  # mkdir /var/lock/subsys the error was gone.
I guess this is related to https://github.com/lomik/go-carbon/commit/05fbeb3d882d4392e0068f8b4cbd603a41d4e844
Thanks !
François

lockfile is not used. I forgot to remove it from init script.
It will be removed in next release

Released in 0.10.1


runtime: signal handling: Go assumes SA_RESTART, but does not enforce it

The runtime registers all of its signal handlers with the SA_RESTART and SA_ONSTACK flags.  It enforces that other handlers are registered with SA_ONSTACK, but does not enforce SA_RESTART.
Since the Go runtime does not register very many handlers when using -buildmode=c-archive and -buildmode=c-shared, other libraries that register handlers will not see an existing handler using SA_RESTART and thus will not know to propagate it.  So in those programs, it is fairly likely that some handler will be registered without SA_RESTART.
Unfortunately, it appears that the Go standard library is written based on the assumption that all handlers use SA_RESTART. System calls in the standard library do not consistently check for the EINTR error and do not document its possibility. For example, many users would be surprised to learn that (*os.Cmd).CombinedOutput can return an os.SyscallError wrapping EINTR, as illustrated by the program below:
bcmills:~/src$ go version
go version devel +b53acd89db Tue May 16 17:15:11 2017 +0000 linux/amd64

eintr/eintr.go:
package main

import "C"

import (
	"fmt"
	"os"
	"os/exec"
	"runtime"
	"syscall"
	"time"
)

//export go_annoy
func go_annoy(sig, seconds C.int) {
	annoy(syscall.Signal(sig), time.Duration(seconds)*time.Second)
}

func annoy(sig syscall.Signal, d time.Duration) {
	runtime.LockOSThread()
	pid := syscall.Getpid()
	tid := syscall.Gettid()
	exit := make(chan bool)
	go func() {
		for {
			select {
			case <-exit:
				return
			default:
			}
			if err := syscall.Tgkill(pid, tid, sig); err != nil {
				panic(err)
			}
		}
	}()

	started := time.Now()
	for time.Since(started) < d {
		cmd := exec.Command("/bin/echo", "Are we there yet?")
		_, err := cmd.CombinedOutput()
		if err != nil {
			fmt.Fprintln(os.Stderr, "exec.Cmd error: ", err)
			os.Exit(1)
		}
	}
	exit <- true
}

func main() {}

eintr/main/eintr.c:
#include <signal.h>
#include <stddef.h>
#include <string.h>

#include "eintr.h"

static void ignore_signal(int signo, siginfo_t *info, void *context) {
}

int main() {
  struct sigaction sa;
  memset(&sa, 0, sizeof(sa));
  sigemptyset(&sa.sa_mask);
  sa.sa_flags = SA_SIGINFO | SA_ONSTACK;
  sa.sa_sigaction = ignore_signal;
  sigaction(SIGUSR1, &sa, NULL);

  go_annoy(SIGUSR1, 10);

  return 0;
}

In typical usage, the Go portion of the program would have only the exec.Command portion of the annoy function; the signal would originate externally (either from some other process, or perhaps from some other language runtime within the process).

I can think of three ways we could deal with this issue:

Document that all signal handlers in a program containing a Go runtime must be registered with SA_RESTART and treat it the same way we do for SA_ONSTACK.
Consistently check for EINTR throughout the standard library and transparently retry.
Document that many Go functions can return EINTR and require end-users to check for it explicitly.

I believe that (3) is strictly worse than (2): if we can't handle EINTR correctly and consistently within the runtime, we can't reasonably expect users to do so.
I am not sure whether (1) is feasible.  In particular, it might prevent the use of the Go runtime with many other languages (such languages that execute on a JVM).
That leaves us with (2).  I am not familiar enough with syscall usage in the standard library to evaluate whether it is feasible.
(CC: @ianlancetaylor @mdempsky )

Realized I forgot to post the build commands and output.  Will update shortly.

Testing for EINTR in the standard library is feasible, it's just annoying and hard to verify for correctness.  There is a limited number of syscalls that can return EINTR.  We would have to track down each one of them and one way or another add an EINTR loop.
On GNU/Linux systems, the list is read, readv, write, writev, ioctl, open, wait, wait3, wait4, waitid, waitpid, accept, connect, recv, recvfrom, recvmsg, send, sendto, sendmsg, flock, fcntl(F_SETLKW), futex, sem_wait, sem_timedwait, and the message queue functions.
We made a deliberate decision early on to not check for EINTR, but to instead assume that all modern Unix systems implemented SA_RESTART correctly.  This turned out to be only approximately true:

https://golang.org/src/os/file_unix.go#L102
https://golang.org/src/net/fd_unix.go#L76
https://golang.org/src/net/sendfile_solaris.go#L78

But, close enough.
One possibility would be to modify the syscall functions themselves to loop on EINTR.  But I don't think we can realistically do that as functions like recv intentionally return EINTR if a timeout has been set on the socket using setsockopt(SO_RCVTIMEO).  So we don't want to always loop on EINTR for calls to syscall.Recv, even if we could plausibly always loop on EINTR for calls to UDPConn.ReadFrom.  Also, of course, it seems a shame to introduce a loop in the syscall package when for many of the relevant functions we already have a loop on EAGAIN (in the internal/poll package) which could trivially check for EINTR.
Unfortunately I agree with you that checking for EINTR in the standard library is the only realistic approach.

Even though this is arguably a bug, it seems too invasive to try to do during the 1.9 freeze (and I'm not sure who would have the time to do it anyway).  If nobody else gets to it by the time the 1.10 window opens, I'll see what I can do then.

But I don't think we can realistically do that as functions like recv intentionally return EINTR if a timeout has been set on the socket using setsockopt(SO_RCVTIMEO).

POSIX says SO_RCVTIMEO causes it to return EAGAIN or EWOULDBLOCK. Linux man pages also mention EINPROGRESS. I don't see any mention of EINTR.

If a socket has a timeout set by SO_RCVTIMEO, and the recv times out, then it will return EAGAIN or EWOULDBLOCK.  But if a socket has a timeout set, and the thread that called recv receives a signal, then the recv will return EINTR.  The reason I called out this case is that if a recv with a timeout set is interrupted by a signal, then it returns EINTR whether or not the signal handler was installed using SA_RESTART.  So changing syscall.Recv to loop on EINTR would be a change in behavior even for existing Go programs that use SA_RESTART as expected.
But thinking about it more I'm not sure this is worth worrying about.  Go programs can not predict that thread that will receive a signal, so they can not predict whether syscall.Recv will return EINTR or not.  So, never mind.

I posted a 1.14 backport request for this: #39026

I commented on that issue, but we aren't going to backport the change.


Feature Idea - Startup Feature.

Feature Idea: Startup Connection Indicator
What are your thoughts on a startup connection feature?
Upon connection, the device could flash the lights three times rapidly or emit a beep or vibration to signal successful connection.
As I use the app daily with the lights saved on, the instant the bike lights turn on, I'll know it's connected and ready to use the throttle.
(Note: European/Australian bikes typically have throttle and lights disabled by default.)

A Phone quick double vibrate will do the trick


ERROR TypeError: Cannot read property 'GoogleSignInOptions' of undefined

Hi, I'm working on a Nativescript (Angular) project using the nativescript-social-login plugin for simple Google authentication on android.  Currently running NS 3.4.2 and social-login 3.0.8.
I have walked through the setup steps on your getting started page, but am getting the following error when calling SocialLogin.loginWithGoogle
ERROR TypeError: Cannot read property 'GoogleSignInOptions' of undefined
Any idea what could be causing this issue?  I have code in my main.ts file to init nativescript-social-login as follows:
if (application.android) {
    application.android.on(application.AndroidApplication.activityResumedEvent, () => {
        let result = SocialLogin.init({
            google: {
                initialize: true,
                serverClientId: '{my_server_client_id}'
            },
            facebook: {
                initialize: false
            }
        });
    });

    SocialLogin.addLogger(function (msg: any, tag: string) {
        console.log('[nativescript-social-login]: (' + tag + '): ' + msg);
    });
}

Here is my code to call login after clicking a button in the app:
login(event) {
        console.log("Attempting to login");
        SocialLogin.loginWithGoogle((result) => {
            this.ngZone.run(() => {
                console.log("code: " + result.code);
                console.log("error: " + result.error);
                console.log("userToken: " + result.userToken);
                console.log("displayName: " + result.displayName);
                console.log("photo: " + result.photo);
                console.log("authToken: " + result.authToken);
                if (result.error || result.code == SocialLogin.LoginResultType.Cancelled) {
                    this.loginFailedState = 'visible';
                } else {
                    this.navigateToLabList();
                }
            });
        });
    }

Any help would be greatly appreciated.  Let me know if you require more information.

@ntaylor2 Do you mean your codebase doesn't have any reference to GoogleSignInOptions?

Yes, it appears that's the case.  When debugging I can see that com.google.android.gms.auth.api package is defined but the com.google.android.gms.auth.api.signin package is not.

@ntaylor2 I really don't follow. Could you confirm if this issue persists using the demo app? or can you create a repo reproducing the same issue?

@jogboms You can close this issue.  When I started building a sample app to show you the issue, it magically started working.  I love it when that happens :)


Package Requested

[X] I would like someone else to build the manifest.
[ ] I would like help so I can submit the manifest.
[X] I have performed a search and couldn't find this package.
[ ] I have checked that the installer for this package is not a standalone or portable executable or a .zip file
[ ] I think there is a new version available and I have provided the URL.

Please provide the following information
Publisher: Microsoft
Publisher Url: https://github.com/microsoft/vcpkg
Package Name: vcpkg
Description: C++ Library Manager for Windows, Linux, and MacOS
Package Version: 2021.05.12
Installer URL: https://github.com/microsoft/vcpkg

They do not appear to provide an installer for their app, They seem to require users to build the files themselves
https://github.com/microsoft/vcpkg#quick-start-windows

Taking a peek at their install script, it just downloads the vcpkg.exe from their GitHub releases page. Technically this counts as a portable executable (which isn't supported afaik), but I personally think that considering they use a install script it counts as an installer?
I think adding vcpkg to winget is also a good idea considering that the user overlap between those two is probably very high, seeing as they're both aimed at developers/devops.

Taking a peek at their install script, it just downloads the vcpkg.exe from their GitHub releases page. Technically this counts as a portable executable (which isn't supported afaik), but I personally think that considering they use a install script it counts as an installer?
I think adding vcpkg to winget is also a good idea considering that the user overlap between those two is probably very high, seeing as they're both aimed at developers/devops.

Install scripts aren't supported as an installer type. It also seems that it uses the vcpkg-tools repository to distribute the portable apps, not the vcpkg repository. Either way, this is currently blocked by https://github.com/microsoft/winget-cli/issues/182 and/or the lack of batch or powershell installation scripts