system V doesn't seem to handle MySQL root password

Long story short, I can't handle MySQL service normally when using a password for MySQL root account :
kro@minipoutre ~  sudo service mysql start
[ ok ] Starting MariaDB database server: mysqld ..
kro@minipoutre ~  ERROR 1045 (28000): Access denied for user 'root'@'localhost' (using password: NO)
^C
kro@minipoutre ~  sudo service mysql status
/usr/bin/mysqladmin: connect to server at 'localhost' failed
error: 'Access denied for user 'root'@'localhost' (using password: NO)'
[info] .
kro@minipoutre ~  sudo service mysql stop
[FAIL] Stopping MariaDB database server: mysqld failed!
kro@minipoutre ~  sudo killall mysqld
kro@minipoutre ~  sudo service mysql status
[info] MariaDB is stopped..

It works fine only with no password.
I have set the password with mysql_secure_installation.
Appart from this MySQL (MariaDB here) seems to work fine.
I have the problem with any distro I tried : Stretch, Xenial, Bionic
Any idea what is causing the issue and how to fix it ?

This one was chirping crickets because it isn't clear what variation of mysql you are running, and the issue template was deleted. Please supply your CLI repro steps leading up to sudo service mysql start as well as your Windows version. Linking a gist of the strace log might also be helpful, although I am not sure this is going to be a syscall thing.

Tried to repro.  I think the issue may be that before mysql is properly configured, pretty much any command, including but not limited to service start mysql will return "bad username or password".  After running mysql_secure_installation (and now having a decent config), the error messages start getting more diverse.
TLDR: probably a configuration mysql error.  should close.

Sorry I was performing different tests to document the use of MySQL in a french community.
I can't remember what this exact problem was but it was most likely due to the use of auth_socket by default on ubuntu bionic I was not expecting.
mysql_secure_installation has became totally useless on debian / ubuntu based system from what I came to understand.
I should have closed this issue myself but I was doing 10 things at the same time, sorry, and thank you.


[ecs] Split DetectChanges into SetChanged

Objective

Currently any sort of struct that tracks change detection (either immutably or mutably) has duplicated code.
There was a previous attempt to coalesce these into a trait DetectChanges however that did not cover the use case for immutable change detection (such as Res<T>)

Solution

Split the implementation of DetectChanges to also have a SetChanged trait where the SetChanged trait is only implemented for the mutable change detection structs.

It looks like we'll need to eventually introduce a trait for getting the value of a thing that implements change detection.
And more specifically, get the value mutably without triggering change detection.
This is really the only way to fix #2343
Ideally I'd like to build that PR on top of this one, so I'll try to push for this to get merged soon. :)

It's a little strange to me that ComponentTicks is in a separate file: could we condense this into change_detection.rs?

Sorry for being dormant on this PR, I just pushed some changes and rebased onto main.
If you can take a look @alice-i-cecile I'd appreciate and then maybe we can flag this as ready for final review :)

This has currently conflicts due to #2345.
@NathanSWard should I send you a PR to your Fork to fix this, or do you want to fix this yourself?

This has currently conflicts due to #2345.
@NathanSWard should I send you a PR to your Fork to fix this, or do you want to fix this yourself

Thanks for reminding me about this @MinerSebas !
I should be able to address the merge conflicts sometime today/tomorrow

ping @cart @mockersf
I think this is ready to go

No longer needed; closing this out :)


[REVIEW ONLY] Run all search integration tests with A and B variants

This is just an idea. Feedback welcome on whether it's a good idea, and also on how to make it easier to configure in RSpec.
Most of our A/B tests only affect a small part of the search functionality. For example, changing format weighting or synonyms shouldn't affect best bets or quoted search terms.
This means that most of the existing integration tests are applicable to both the A and the B variant, but we normally don't run them against B. Being able to run them for both would give us more confidence that we haven't caused any regressions in B. We worried about this we configured the synonyms A/B test because changing the analyzers is quite fiddly.
This change runs the integration tests which hit the /search endpoint twice: once for each variant. It does it by wrapping the test definitions in a block, which makes it easy to define separate tests which really do apply to just A or B.
Things that don't work so well (but might be fixable with some RSpec magic):

The diff is enormous because the change indents all the tests. This would happen each time you set up or remove an A/B test.
You have to change get to get_with_variant in each test, otherwise all the tests will silently use the A variant.

https://trello.com/c/PqwdCw1d/447-configure-synonym-a-b-query-in-rummager
Paired with @Rosa-Fox on the initial idea.

I have looked at these sort of approaches in the past and as a general rule they are difficult to maintain.
What I have seen work better is a manual approach that allows running of the test suite with a passed in variable. SO rather than the test suite running each test for A and B, it is run twice, once as per normal (A variant) and once passing the override in (B variant).
This would hopefully avoid the need to update the test suite when we change/add/remove variants.

Closing this because we're not planning to run any more A/B tests for a while.


⚠ Minecraft has degraded performance

In 194f34b, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 828 ms

Resolved: Minecraft performance has improved in d12910e.


Add subcommand to yak that makes it easier to get a shell in a container defined in docker-compose.yml

Currently, ./yak bash gets you a bash shell in the app container.  Keep that command & deprecate it, then add command ./yak shell [container [command]] where container is optional and defaults to app.  This allows an convenient shortcut to get a shell in any of the known containers defined in docker-compose.yml:, like ./yak shell mongo for a shell or ./yak shell mongo mongo for the mongo client.

This issue was moved to SciSpike/yaktor-issues#14


Notebook erroring out when running many iterations

When running the following notebook: https://github.com/ericjang/nf-jax/blob/c6636a010bb744a48185eb3f622d32336e929990/nf-tutorial-jax.ipynb, I have found that if you run all 1e4 iterations in a row, then evaluate y.max(), the notebook crashes. However if you run say 1000 iterations, then run y.max(), then run 1000 iterations, then run y.max(), etc, all the way to reach 1e4 iterations, the notebook does not crash and things work as expected.

@hawkinsp could this be related to async execution?
@raphpapercup which backend is this on (CPU or GPU)? Could you try running y.block_until_ready() every 1000 iterations, instead of y.max(), just to help us diagnose the bug? See the async dispatch docs for an explanation of why it might be related.

If this is running on CPU, I can believe it might well be related to async execution; you could easily run OOM. We probably need to limit how far ahead the Python code can run. Could you please verify which backend you were using? Thanks!

I believe this was fixed by the stream pacing mechanism referenced above.


Error: extract_area: bad extract area

Hello, I'm using 512x512 image. First I resize it then extract and expecting readable stream data but it shows me "Error: extract_area: bad extract area" error.
Here's the code;
test2(x, y, zoom) {
        const width = 500;
        const height = 500;

        return this.image
            .metadata()
            .then((metadata) => {
                var resizedWidth = metadata.width * zoom;
                var resizedHeight = metadata.height * zoom;
                var left = parseInt(resizedWidth * x);
                var top = parseInt(resizedHeight * y);
                return this.image
                    .resize(resizedWidth, resizedHeight)
                    .extract({left: left, top: top, width: width, height: height})
                    .jpeg()
                    .resize(500);
            });
}

Thanks.

Ahhh, I must say, resizedWidth = 1024, resizedHeight = 1024 (because i used zoom = 0,5) and left = 204, right = 204 (because i used x = 0,2 and y = 0,2). Extract area is valid.

Hello, I think the problem here is that .resize(500) will replace the values passed to .resize(resizedWidth, resizedHeight). Whilst you can chain extract with resize (with extract), you can't chain resize with itself.
If you need the output image to always be 500 pixels wide, you might be able to extract-then-resize via .extract(...).resize(500, resizedHeight), making sure to alter your resizedHeight calculation accordingly.

Ahhh, thanks,
I fixed it with this codes.
return this.image
            .metadata()
            .then((metadata) => {
                var resizedWidth = metadata.width * zoom;
                var resizedHeight = metadata.height * zoom;
                var left = parseInt(resizedWidth * x);
                var top = parseInt(resizedHeight * y);
                return this.image
                    .jpeg()
                    .resize(resizedWidth, resizedHeight)
                    .toBuffer()
                    .then((data) => {
                        return sharp(data)
                            .extract({left: left, top: top, width: width, height: height})
                            .resize(width, height);
                    })
            });

I know this may be an old thread - but I experienced this issue recently, and it was due to the orientation of the image on the server, not being equivalent of the interpreted orientation in the browser. Say I took a picture with my iPhone, then safari and the phone knew how to properly orient the photo, but the server no.
In order to fix that, I simply initialised the sharp procedure with .rotate(), which took care of that.
sharp('image data goes here').rotate().extract({left: left, top: top, width: width, height: height}).resize(width, height);.
Just as a note for future visitors, as this issue pops up as the first result when searching in Google.

Thanks to @imCorfitz 's comment I solved the problem, for some strange reason Sharp (v0.25.4) interprets some rotated images, in particular a vertical image taken from the smartphone's gallery. If the frontend of the app gets the picture (vertical or horizontal) from the camera everything works fine.
In my case, simply comparing the Sharp Metadata with the original dimensions of the photo fixed it.
As a simple code example:
if(dimOriginal.width === metadata.height && dimOriginal.height === metadata.width){
    console.log("image is rotated");
    promiseImg = promiseImg.rotate()
}

This is still an issue until this day. Is there an issue tracking this?


Detect dev tool without extra debug props

Would be good if there is 0 config for developers.

Have we resolved this yet?


Remove reference to DatabaseCleaner in README

As not everyone wants to include database_cleaner, the example above uses FactoryGirl's APIs to get the factored models, on which you can then call their APIs to clean the records up.

@vjt One of the benefits of DatabaseCleaner is that it does use the ORM/ODM's correct API. Additionally, delete_all isn't guaranteed to delete dependent relationships, so in a case where there's a before_create callback on a User model that creates a profile, but FG doesn't define a profile factory, that record would not be deleted.
As the goal is to suggest a DB-agnostic solution that ensures the database is entirely clean, and because we have no control over callbacks and how others' write their code, I think DatabaseCleaner remains the best recommendation.

OK, makes sense. Thank you.


ruby-grpc: gcc-14 fixes

https://boringssl.googlesource.com/boringssl/+/c70190368c7040c37c1d655f0690bcde2b109a0d
add -Wno-error=incompatible-pointer-types to NIX_CFLAGS_COMPILE and remove the other flags as NIX_CFLAGS_COMPILE were getting clobbered since 7121ef61da4ddd36f81480322bc2ae984c575b1d (2021)
gitlab builds after this change but nixosTests.gitlab still times out
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg build gitlab

This patch was already upstreamed in v1.65.0-pre1~90; attempting to apply it again after that causes errors.
$ NIXPKGS_ALLOW_UNFREE=1 nix build -f . vagrant
error: builder for '/nix/store/f6dvxxlbsnrl0c0b2v0ha8v6kj68b3kn-ruby3.3-grpc-1.68.1.drv' failed with exit code 1;
       last 9 log lines:
       > Running phase: unpackPhase
       > Unpacked gem: '/build/container/w7ra8z0kxy1n24vybyfbap1pclk7s8ih-grpc-1.68.1'
       > Running phase: patchPhase
       > applying patch /nix/store/m1mccpgnr5901sdrf764a9zmchy8pcac-gcc-14-fixes.patch
       > patching file third_party/boringssl-with-bazel/src/crypto/internal.h
       > Reversed (or previously applied) patch detected!  Assume -R? [n]
       > Apply anyway? [n]
       > Skipping patch.
       > 2 out of 2 hunks ignored -- saving rejects to file third_party/boringssl-with-bazel/src/crypto/internal.h.rej
       For full logs, run 'nix-store -l /nix/store/f6dvxxlbsnrl0c0b2v0ha8v6kj68b3kn-ruby3.3-grpc-1.68.1.drv'.
error: 1 dependencies of derivation '/nix/store/2ic36mlzgvl67qcia33gylb52wbkhcc3-vagrant-2.4.3.drv' failed to build
error: 1 dependencies of derivation '/nix/store/5bdmiki4k26c6m1gbl48bh871f7m7b8f-vagrant-2.4.3.drv' failed to build

#368732


Second Release ToDos

[ ] Enable Extended object spectrum extraction.

__
please add any request/ideas/needs

[ ] Add/reject points for wavelength solution fit

Zoom/Pick Issue is solved.

[ ] Create a GUI or create a special class for the matplotlib graphic interface that makes the interactive mode work.

[ ] Use slit value to estimate argrelmax order (line detection)

This issue is oudated and the only point left is covered in #41 #146


Support up to wasmtime v22

Upgrading to latest wasmtime version to ensure function runner compatibility.
I tested generating some profiles on wasmtime 22 and everything looks fine.

pushing a new version to crates.io


Unable to calculate volume of sphere without violating checkstyle

Part of exercise 02 requires you to create a method to calculate the volume of a sphere in the sphere class, which requires the formula (4.0/3) * Math.PI * radius * radius * radius. Having the numbers 4.0 and 3 in my code however is considered as a magic number and hence doesn't pass checkstyle. Is there a workaround for this?

you can define the 4.0/3.0 as a variable first. eg.
class Sphere {
private static final double formulaNumber = 4.0 / 3.0;
....
}

the best way to deal with any magic number would be to write them out as private static final variables of the class since this will be more favourable in the spirit of OOP and will make it easier for you to edit the code if necessary in future as well


Allow Google Docs style comments/suggests to make improving the docs easier

Developers who would like to help improve the docs are currently required to either propose a specific change (pull request), or file an issue explaining the problem they encountered.
The latter case introduces unnecessary friction:

The issue filer must spend time linking to the specific doc page and explaining which section cause them a problem and why.
Everyone who then has to read the issue or follow up needs to read this context, in order to understand where the problem is. Only then can they focus on the real issue: what the actual problem is.
Extra friction means that valuable contributions or comments may not be made. (I know I would contribute more if friction were reduced.)

It would be really nice if we allowed developers to simply highlight the affected text, then type in their comment (e.g. what's broken, what's confusing, how we could make the docs better). This allows the submitter and all subsequent readers to immediately understand the context of the comment, and to comment in context.

Out of scope at this point of development.


AttributesBackend: Allow multiple points if all the attributes are  the same

The idea is that if there are several points in the map in the same position but all have the same attributes (for the requested columns) we'll return those common attributes instead of an error. If the attributes are different we'd still return an error.
Related to https://github.com/CartoDB/support/issues/1298
Also re-enables test cleanup that I pushed by mistake.

Building it's failing with yarn but works with npm. I'll try enforcing the npm modules dependencies to all use the same mapnik release

oh I see, when green feel free to merge it.


Modern page publish error with Conent Approval turned in Site Pages library

I'm getting: Could not publish your page. We're sorry, we encountered an unexpected error. Please try again, if the problem persists please contact your Administrator.
You just need to turn Content Approval on Site Pages library, then create new page based on "News post" content type and save it. The on display mode click publish and see error message. It does not show error message when you publish from Edit mode. But in both scenarios page is not approved, which is expected and you need to go to Site Pages library and approve it manually.
I would also suggest this working in similar way like publishing pages work - displaying different buttons depending on approval status, e.g. if content approval is turned on then instead of "Publish" it could show "Submit" for contributor, but for approver "Publish" should approve content as well.
Category

[x] Suggestion
[x] Bug

Thanks
Saulius

This error actually happens on any OOTB modern page with a process:

create new page, save, but do not publish.
edit page all properties from site pages
go to the news page and publish it

This is not nice when you have some additional metadata fields for your modern pages.

Found much simpler way to reproduce:

Create and publish modern page
Edit page and see that URL finishes with ?Mode=Edit
do not save page, just trim ?Mode=Edit and reload the page
click Publish on display mode

We are getting this error on modern pages.
@VesaJuvonen Can you please advise if there is a solution?
Thanks.

This issues should be fully solved and fixed, given the work we have done in engineering in this area. To ensure that we clean up the backlog of the issues, we will be closing this one. If you still have issues on this particular area, please open up a new issue so that we can get updated issue description and reproduce steps. We do apologize the non-optimal handling of these old issues and are looking into improving this in future.


Minecraft <IP_ADDRESS>

I cant log my xbox account into minecraft, pls help me.

Are you using an iPhone?

Are you using an iPhone?

No it's android phone

When using RenderDragon shaders, you may often be unable to join multiplayer servers or log into your Xbox account. Please contact the YSS Group, who created the patched app.


Add Estimators

Smaller wrappers around the xgboost versions.

Looks good to me

I'm not sure why this CI didn't run on this. We'll see if it does on master.


Translate the website to Portuguese (brazil)

Niceeeeeee, thanks @evandroamparo :partying_face:

Deployed to prod at https://bigfive-test.com/pt

Some pages aren't translated yet. Please let me know if you need some help with them.


Detect Parsers with none unique names.

This will throw an error instead of potentially leading to unexpected behavior.
Fixes #715

pushed directly to master by mistake already...


Analyzer FileResult.isPart returns false when it should return true

Thank you for taking the time to file an issue!
In order to route, prioritize, and act on this as soon as possible please include:

Dart SDK Version (dart --version) latest
What package(s) from this repo you are using, and the version (i.e. build_runner 0.7.12) build_runner latest

If you are not sure, try checking your pubspec.lock file. see github repo

What builder(s) you are using (or writing yourself). Try to give a short summary of what they do. freezed. https://github.com/rrousselGit/freezed/issues/668
Whether you are using Windows, MacOSX, or Linux (if applicable) mac
Whether you are using Chrome, Safari, Firefox, Edge (if applicable) na
Any other packages or constraints we should know about na

Missing some or all of the above might make the issue take longer or be impossible to act on.

Hi thanks for the build runner! However I cannot run it. Although it looks like a freezed problem, but the error stack trace solely lies in source_gen, so I raise issue here.
Steps: Download https://github.com/fzyzcjy/freezed_build_runner_bug. Run flutter pub get. Run flutter pub run build_runner build --delete-conflicting-outputs --verbose.
Output:
...
[        ] Artifact Instance of 'FontSubsetArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'PubDependencies' is not required, skipping update.
[  +82 ms] executing: /Users/tom/fvm/versions/3.0.0/bin/cache/dart-sdk/bin/dart __deprecated_pub run build_runner build
--delete-conflicting-outputs --verbose
[INFO] Entrypoint:Generating build script...
[INFO] Entrypoint:Generating build script completed, took 497ms

[WARNING] ApplyBuilders:
Configuring `mobx_codegen:mobx_generator` in target `freezed_build_runner_bug:freezed_build_runner_bug` but this is not a known Builder
[INFO] BuildDefinition:Initializing inputs
[INFO] BuildDefinition:Reading cached asset graph...
[INFO] BuildDefinition:Reading cached asset graph completed, took 54ms

[INFO] BuildDefinition:Checking for updates since last build...
[INFO] BuildDefinition:Checking for updates since last build completed, took 556ms

[INFO] Build:Running build...
[INFO] Build:Running build completed, took 23ms

[INFO] Build:Caching finalized dependency graph...
[INFO] Build:Caching finalized dependency graph completed, took 29ms

[SEVERE] source_gen:combining_builder on lib/src/model/a.freezed.dart (cached):

Asset [freezed_build_runner_bug|lib/src/model/a.freezed.dart] is not a Dart library. It may be a part file or a file without Dart source code.
package:build_resolvers/src/resolver.dart 222:7   AnalyzerResolver.libraryFor
package:build_resolvers/src/resolver.dart 123:26  PerActionResolver.libraryFor.<fn>
package:source_gen/builder.dart 115:26            CombiningBuilder.build

[SEVERE] Build:
Failed after 89ms
[+2957 ms] "flutter run" took 3,124ms.
[   +5 ms] pub finished with exit code 1
[   +1 ms]
           #0      throwToolExit (package:flutter_tools/src/base/common.dart:10:3)
           dart-lang/build#1      _DefaultPub.interactively (package:flutter_tools/src/dart/pub.dart:416:7)
           <asynchronous suspension>
           dart-lang/build#2      PackagesForwardCommand.runCommand (package:flutter_tools/src/commands/packages.dart:251:5)
           <asynchronous suspension>
           dart-lang/build#3      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1183:27)
           <asynchronous suspension>
           dart-lang/build#4      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           dart-lang/build#5      CommandRunner.runCommand (package:args/command_runner.dart:209:13)
           <asynchronous suspension>
           dart-lang/build#6      FlutterCommandRunner.runCommand.<anonymous closure>
(package:flutter_tools/src/runner/flutter_command_runner.dart:281:9)
           <asynchronous suspension>
           dart-lang/build#7      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           dart-lang/build#8      FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:229:5)
           <asynchronous suspension>
           dart-lang/build#9      run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:62:9)
           <asynchronous suspension>
           dart-lang/build#10     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
           <asynchronous suspension>
           dart-lang/build#11     main (package:flutter_tools/executable.dart:94:3)
           <asynchronous suspension>

[ +261 ms] ensureAnalyticsSent: 258ms
[   +1 ms] Running shutdown hooks
[        ] Shutdown hooks complete
[        ] exiting with code 1

🎉  You guys are great!

Does this need to be cherry picked?

As far as build_runner is concerned we can just pick up the next pub release. Not sure if it causes other general issues with dart analyze though.

I'm not sure. We use AnalysisSession.getParsedUnit() in DAS, so interactive DAS users could experience similar race conditions.

Hi, is it released to pub.dev now? https://pub.dev/packages/build_runner/changelog does have a new release but seems changelog does not mention this issue. Looking forward to the new release!

The fix and changelog entry are in the analyzer package.
https://pub.dev/packages/analyzer/changelog

I see. Thank you!


neogit returns errors in headless mode in the hl.make_palette function

Description
neogit returns errors in headless mode in the hl.make_palette function
Neovim version
NVIM v0.10.0-dev-622ae2f
Build type: Release
LuaJIT 2.1.0-beta3
Operating system and version
Nixos
Steps to reproduce
nvim --headless -c "q"
Expected behavior
running in headless mode should not generate errors
Actual behavior
lua/neogit/lib/color.lua:101: bad argument #1 to 'rshift' (number expected, got nil)
is generated
Minimal config
vim.cmd([[set runtimepath=$VIMRUNTIME]])
vim.cmd([[set packpath=/tmp/nvim/site]])
local package_root = "/tmp/nvim/site/pack"
local install_path = package_root .. "/packer/start/packer.nvim"
local function load_plugins()
  require("packer").startup({
    {
      "wbthomason/packer.nvim",
      {
        "TimUntersberger/neogit",
        requires = {
          { "nvim-lua/plenary.nvim" },
          { "sindrets/diffview.nvim" },
        },
        config = function()
          print("loaded neogit")
          require("neogit").setup()
        end,
      },
    },
    config = {
      package_root = package_root,
      compile_path = install_path .. "/plugin/packer_compiled.lua",
      display = { non_interactive = true },
    },
  })
end
if vim.fn.isdirectory(install_path) == 0 then
  print("Installing neogit and dependencies.")
  vim.fn.system({ "git", "clone", "--depth=1", "https://github.com/wbthomason/packer.nvim", install_path })
end
load_plugins()
require("packer").sync()

That's really weird! Running the minimal init provided with
nvim --clean --headless -c "q" -nu minimal-init.lua
doesn't output an error.
Using my normal config though does. (`nvim --headless -c "q")

The branch created for #617 seems to resolve the issue.
Can you give the hard-code-colors branch a shot @polarmutex and report back?

We may want to detect if neogit is running in headless mode, and if it is maybe don't load?
We can use nvim_list_uis() to check if we're in --headless or --embed mode. If that returns empty then we can skip all the color setup as then we are necessarily in one of those two modes.

Man, this colors thing is turning out to be a headache. Yeah, some kind of guard clause on it would be good, if it's causing issues.

What I posted seems like a decent idea on the surface, but this will make any tests that validate colors in the future not consistent with normal headless usage. This is because we'd need to pass a force setup clause for highlights if we're currently testing neogit and in headless mode. If we guard entirely on headless/embed mode then we'll be unable to test highlights unless we create a flag for testing.
This is an issue because we want the tests and the normal setup for Neogit to be identical ideally.
If you think a test flag is ok to force setup of highlights then I'll make a PR for it, but I'm not sure it's the best route. I think using the defaults you've set in #617 is probably better.

Hopefully that works, yeah. I think, in addition to that, an autocmd that runs on ColorScheme or something to re-setup highlight groups..

I do not see the error with #617


Design & structure of mock datas

Open questions

Which datas we need (market name, capacity, ...)?
How are the mock datas structured?
Are they stored in a Json-file or in a database?
If stored in a database which one?

Tasks

[ ] Find a solution for the structure of the mock datas
[ ] Find a solution how the mock datas get implemented
[ ] Generate further productive issues about genereating and implementing the mock datas

In der alten Version hatte das json vom server folgende Form
{
    id: string,
    name: string,
    city: string,
    street: string,
    load: number
}

Beispiel:
{
    "id": "f0a23bd1dd2a1227d8b9f68c9e89b76c68ba3b3e",
    "name": "Metro",
    "city": "Hamburg",
    "street": "Papenreye 33",
    "load": 42
}

load hat sich dabei im Bereich 0-100 bewegt


Change photo for Hub Zones

Purpose:
The purpose of this issue is to update the images associated with the Hub Zones franchise location. Hub Zones are strategically chosen for their unique characteristics and potential for growth. Updating these images will allow us to better represent the appeal and opportunities offered by these locations in our franchise guide.
Importance:
Images are a primary way to convey the essence of a location. Outdated or misleading visuals can hinder potential customers' and franchisees' perception of a Hub Zone's potential. It's crucial to ensure that the images accurately represent the chosen Hub Zones, as these areas are selected for their specific attributes and growth prospects.
Why Hub Zones:
Hub Zones are chosen because they present exceptional opportunities for business development and community enrichment. Their strategic locations, economic incentives, and potential for growth make them ideal choices for SILK franchises. Highlighting these zones with updated and accurate images is vital to attract the right investors and entrepreneurs.
Outcome:
The outcome of this issue will be a set of updated, high-quality images that accurately portray the Hub Zones franchise locations. These images will showcase the unique opportunities, landscapes, and local culture of each Hub Zone. By doing so, we aim to enhance the appeal of these locations, attracting potential investors, entrepreneurs, and franchisees who recognize the potential for growth and community development offered by Hub Zones.

You can use Envato Elements or Flickr to find high-quality photos.
You can use Pixlr to resize the image.

Note:

All images should be saved inside this folder: SILK Images
Be organized on how you're naming your images.

I'm going to work on this issue. I'll find an image that will represent the Hub Zones location as a good location for our franchise and then resize it according to the required size 600 x 350. I'll update the photo and source of the image and then will create a pull request.


Fix some problems when server is in cache mode

Packages on PYPI with older versions which have invalid PEP440 versions
would crash distlib in default mode
If some packages would be in the cache already, no newer (or older) versions of them
could be fetched

Coverage increased (+0.02%) to 89.79% when pulling 3b47f6a00b12d9c6bb33d93d783b9e5420d5017b on smokey42:master into b5fa5de14129fca4c84d6fe50a19b81081b1d118 on mathcamp:master.

First two commits are great, definitely will merge. Third commit requires a bit more discussion. Check out #42 for details, but the short version is that the current behavior of cache is a feature. It seems like people want the other behavior, but I'd like to put it behind a different mode called mirror or something. You can do this if you like, or I can look into it when I have time.

Removed the last commit. Still thinking about the other issue. There doesn't seem to be a particularly clean solution for this problem though.

Coverage increased (+0.02%) to 89.79% when pulling 15264157ac675aa8dda1fd925a9e9b9864bdcaf3 on smokey42:master into b5fa5de14129fca4c84d6fe50a19b81081b1d118 on mathcamp:master.

I just realized I don't need the other fix, because fallback=redirect is ok with me.

Awesome, thanks! I'll cut a new release sometime soon.


Feature Request - A Way To Pick Up Livestream Url When Eufy Cameras Record Motion?

Not sure if this is possible given Eufy's backend, but would it be possible to automatically populate the livestream url value whenever a camera starts recording due to motion?

Everybody can solve this need by himself by installing and using e.g. the Javascript Adapter in the ioBroker. Thereby you can solve it with blockly or also with your own Javascript/Typescript script. Just react on the change of the state "motion_detected" and start the livestream with the button state "start_stream".

Awesome! Thanks for the advice!


Error on @mux - 'hls.js' does not contain a default export (imported as 'ue').

I did a simple installation and run the project...
Error:
<EMAIL_ADDRESS>Attempted import error: 'hls.js' does not contain a default export (imported as 'ue').

Import trace for requested module:
<EMAIL_ADDRESS><EMAIL_ADDRESS>./node_modules/next-video/dist/components/default-player.js
./node_modules/next-video/dist/components/video.js

Thank you for filing this issue! We are investigating the root cause in tandem with https://github.com/muxinc/elements/issues/818.

This looks like a bug from Next.js https://github.com/vercel/next.js/issues/57964

Same and downgrading to Next.js v14.0.1 fixes it.

this is fixed in the latest Next.js canary
https://github.com/vercel/next.js/releases/tag/v14.0.4-canary.33

this is resolved in Next.js v14.0.4 https://github.com/vercel/next.js/releases/tag/v14.0.4


Plated optic

I'd like to see the Plated type-class from Haskell lens implemented in Monocle:
trait Plated[A] {
  def plate: Traversal[A, A]
}

object Plated {
  def plate[A: Plated]: Traversal[A, A] = implicitly[Plated[A]].plate
  def children[A: Plated](a: A): List[A] = plate[A].getAll(a)
  def universe[A: Plated](a: A): Stream[A] = {
    val fold = plate[A].asFold
    def go(b: A): Stream[A] = fold.foldMap[Stream[A]](c => c #:: go(c))(b)
    go(a)
  }
  def transform[A: Plated](f: A => A)(a: A): A = plate[A].modify(f)(a)
}

The above might be a good start but I'd love to see functions like rewrite.
Some questions have to be answered, though, for example: Stream makes sense for universe but it might also make sense for children - what should we do?

Good idea, I guess we should be consistent for the return type. So Stream is probably best. If user wants a List, it is always possible to use plate.getAll


RFM - bonjour

I DON'T Want To Become The Maintainer

[X] I have followed the Package Triage Process and I do NOT want to become maintainer of the package;
[X] There is no existing open maintainer request for this package;

Checklist

[x] Issue title starts with 'RFM - '

Existing Package Details
Package URL: https://chocolatey.org/packages/bonjour
Package source URL: https://support.apple.com/downloads/DL999/en_US/BonjourPSSetup.exe
Date the maintainer was contacted (in YYYY-MM-DD): 2020-06-08
How the maintainer was contacted: xmha97 used the 'Contact Maintainers' link on the package page.

The binaries cannot be embedded and @xmha97 confirmed that the latest installer is not available on the official Apple download page. Will update this with more info when I have it.

You can see the latest version of bonjour into iTunes setup:
iTunes 64-bit

iTunes 32-bit

iTunes can be downloaded from the following link:
https://www.apple.com/itunes/download/win64
https://www.apple.com/itunes/download/win32

/recheck

I'm already working on it, but was stuck with one thing, now it's working, so it's in moderation state

I'm already working on it, but was stuck with one thing, now it's working, so it's in moderation state

There is Bonjour v2.0 on the Chocolatey... I need Bonjour v<IP_ADDRESS>

It's there, just currently in moderation. Once it passes the moderation requirements it will be available. 🙂

It's there, just currently in moderation. Once it passes the moderation requirements it will be available. 🙂

Great 👏

It's Accepted, so this issue can be closed ;-)
Thank you


Please invite me to the GitHub Community Organization

Name
Raghav Maskara
Discord Username (if applicable)
Raghav#5775
Additional Context
I heard about Eddie from my friend. I want to join Eddie because I am interested in contributing to Opensource and want to learn new stuff from different community members and mentors.

Welcome to the community!


[CLI] Compatible with Apps and Marketplace

The rancher-cli is not currently compatible with the new apps and marketplace charts in Rancher 2.5. The rancher-cli is helpful with automation and not having this feature in the cli is an impediment. The rancher-cli is helpful with automation because it has auth into the clusters. Using the helm cli would require having a kubeconfig file which at new cluster creation with automation is not setup. Rancher also stores the helm repositories rather than the repositories having to be stored locally. This makes the automation tools more repeatable for users.
What kind of request is this (question/bug/enhancement/feature request):
Enhancement
Steps to reproduce (least amount of steps as possible):
rancher app install rancher-monitoring rancher-monitoring
Result:
This deploys rancher-monitoring from the 2.4 apps catalog. There is no way to deploy the one from the 2.5 apps and marketplace.
Other details that may be helpful:
Any other commands from rancher apps is only interacting with the 2.4 catalogs. Commands like rancher apps ls or rancher apps lt only shows the apps and repositories from 2.4 and not what is in from 2.5.
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): 2.5.1
Installation option (single install/HA): HA
Rancher cli: 2.4.7

I there any workaround for this issue ?

I am also looking for a solution for the above stated probelm . On the other hand I was not able to get it to work even via helm ,  for some reason iam not able to add the helm chart repo of racher . @toddexp can you please let me know how we can get rancher apps to install atleast via helm ,  i am not able to grab the rancher charts even via helm

Hi @hari-narayanan94. I am another community member like yourself. I am not sure of workaround. I have not yet tried to use the regular helm cli. I may try this soon. If I make any progress with that I will be sure to share.

@loganhz Can you please guide if there is any way we could install charts from apps and marketplace into a cluster via rancher  programmatically via API or helm or any means ?? Iam looking to install rancher-istio and rancher-monitoring

I was able to find a work around with the helm cli. I would still prefer the rancher cli for the reasons listed when opening this issue.
I assume you are also trying to do automation from your comments, so this was all the steps I needed. The kubeconfig steps may be able to be omitted depending on your use case.
# create temporary kubeconfig file to be used later with helm
rancher cluster kubeconfig {CLUSTER_ID} > {/path/to/temp/kubeconfig}

# add rancher repo to helm
helm repo add {REPO_NAME} http://charts.rancher.io/

# install helm chart using temporary kubeconfig file
# used rancher-monitoring as an example since that was the one used in the initial example
helm upgrade --kubeconfig={/path/to/temp/kubeconfig} --install=true --namespace=cattle-monitoring-system --wait=true rancher-monitoring-crd {REPO_NAME}/rancher-monitoring-crd
helm upgrade --kubeconfig={/path/to/temp/kubeconfig} --install=true --namespace=cattle-monitoring-system --wait=true rancher-monitoring {REPO_NAME}/rancher-monitoring

Additionally Terraform should be able to deploy the new apps and market place charts as well. I have not personally tested this, however.

I was also find out a way to trigger this via Rancher API using this endpoint
{rancher-url}/k8s/clusters/{rancher-clusterid}/v1/catalog.cattle.io.clusterrepos/rancher-charts?action=install

Still relevant.

At least should be documented more visibly. There is some notion of the difference in https://rancher.com/docs/rancher/v2.6/en/helm-charts/, but nothing mentions how to do that on the command line. Yours, Steffen

This is  blocking  operations via cli, can someone please look into this ?

This issue is still relevant


Description infrastructure work

At some point there will be space to add the larger improvements to a11y in scenery that we have been putting off for some time. It would be good to keep track of them in one place to get an idea of the scope of work and priority of different pieces.
Higher priority

[ ] Cleanup from JO's review
[ ] Fix a11y behavior functions
[ ] Implement a11y api for adding PDOM headings
[ ] Prevent A11y events after focus changes
[x] Handling 'activation' events
[ ] A11y performance optimizations

somewhere in the middle?

[ ] Pickable should work with a11y listeners
[x] Add support for aria-activedescendant
[ ] Better implementation for association attributes
[ ] More abstraction for accessible components
[ ] Consistency of scenery-internal code

Lower priority

[ ] Audit PDOM structure of a screen after construction
[ ] h3 allows both 'text' as inner content and has children
[ ] Add more dag related tests for aria-labelledby associations

@jessegreenberg please edit inline any of these, as well as adding more that you have on you list.
It some of these may not need dedicated time, but I like the idea of having everything in the same place. I also think it may help @emily-phet understand the scope of infrastructure work that we are skating through/ignoring at the moment.

@zepumph Thanks! This is very helpful.

@kathy-phet, I just found this issue. This seems to get at the heart of the description work that could be continued, much of this is based on @jonathanolson's review of PDOM work done back in 2018.


Does promptfoo accept multimodal input

Hello, my LLM workflow is heavily dependent on images being used along with each text prompts. Does promptfoo support this kind of workflow? I couldn't find any mentions of this in documents so my assumption is NO, but just wanted to check.
TIA!

Yes, it does, and we have several examples:

Claude via AWS and Anthropic with a JSON prompt
Claude vs GPT-4 with JavaScript and Python prompts
Local Gemma via LM Studio
DALL·E image generation

Let us know if you get stuck or have any more questions! Thanks!

That's fantastic, Michael! Apologies for not reading the docs more thoroughly. Might be better to highlight this as a "feature" in the docs, for n00bs like me.

No problem at all! I'll update the docs. Thanks for the callout. Please let us know if you run into any other issues.

Hi, does it also work with the Jest integration?


Jester shoes now jingle. Sounds by GentleJester#8754

About the PR

Adds jingle sounds to the jester shoes.
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

https://user-images.githubusercontent.com/121047731/232334927-16a1234c-b756-4e79-b0d7-8016ee860163.mp4
Changelog

:cl: Whisper

add: Jester shoes now jingle.

Test says something's missing in attributions, beats me.

 :)


Flaky Test: ConnectionResetBetweenRequestsIsLoggedAsDebug

Assert.True() Failure
Expected: True
Actual:   False
   at Microsoft.AspNetCore.Server.Kestrel.FunctionalTests.RequestTests.<ConnectionResetBetweenRequestsIsLoggedAsDebug>d__13.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)

http://aspnetci/viewLog.html?buildId=219517&tab=buildResultsDiv&buildTypeId=CoreCLR_Darwin

01b8d5fad1753a1eda61124d1b08447a81a8f0c0


Read SENTRY_DSN from environment

This pull request is against the staging branch since that deals with the
QA setup for this project.

@miltontony is this a known issue on staging?

Ready for review but those Sentry errors are very strange (but not new to this PR apparently).
@miltontony thoughts?

👍 thanks!


Add decent error recovery and Ctrl-C handling

Prior discussion from https://github.com/dandi/dandi-utils/issues/3:
jwodder:

Do you have any ideas or preferences for how the backup program should handle download failures?  What about crashes or a Ctrl-C in the middle of a backup?

yarikoptic:

Ideally it should be robust to download failures, and be able to "track back", e.g. if it was under git like with backups2datalad -- git reset --hard; git clean -dfx  . But here it is not feasible, hence we might want to explicitly "model" the "rollback" regime to get back to prior state -- remove fresh (complete or incomplete) downloads, undo mv's already done -- might be worth keeping a journal of operations or just being able to take prior state and "match" it.

Upon Ctrl-C, if using journaling mechanism, we might want to ensure having it stored. Otherwise -- not sure if worth implementing something as fancy as planned for https://github.com/dandi/dandi-cli/issues/1523

jwodder:

Please elaborate on exactly what behavior you want.

yarikoptic:

rollback or match the prior state: add a function which would ensure that current tree is matching specific inventory one:

for a folder, go through the union of paths found in inventory, .dandi-s3-backup-versions.json, and on the drive (excluding .old. ones)

if file is on drive and/or .dandi-s3-backup-versions.json but not in inventory - remove file at {path} and from .dandi-s3-backup-versions.json
if file record in inventory not matching the one in .dandi-s3-backup-versions.json - remove from drive and the .dandi-s3-backup-versions.json
if file is not present on drive but in inventory - if there is a corresponding {path}.old.{versionid}.{etag}  -- rename it to  {path}, adjust .dandi-s3-backup-versions.json accordingly

if there is no .old. file -- fail, shouldn't happen

But while thinking about it, I realized that overall approach does not cover the case of key switching between being a file and directory.

When file becomes a directory -- all is easy, prior version gets renamed into {path}.old.{versionid}.{etag}, and for new {path}/ folder created.
When directory becomes a file -- just rename directory to {path}.old.dandi-s3-backup  if such does not exist yet. If exists already -- nothing to be done.

need to add check for path to not end with .old.dandi-s3-backup into conflict detection above
to reconstruct some prior key for versionId we would need to inspect all parents to potentially carrying the .old.dandi-s3-backup suffix now

jwodder:

for a folder, go through the union of paths found in inventory,

Because each set of inventories lists every single item in the bucket, this won't scale well.  Just a single CSV file from the manifest you showed in the original comment contains three million entries.

yarikoptic:

yes, there is a scalability concern as we are expecting hundreds of millions entries (e.g https://github.com/dandisets/000108 alone accounts for 300 million files across its zarrs).  If those lists are sorted though -- might be quite easy since then all files in a folder would be a sequential batch and we would process that sequential batch from inventory + files on drive and in .dandi-s3-backup-versions.json only for that folder, which would be either tiny or some thousands -- not more at once.

jwodder:

But wouldn't the rollback have to be run against the entire backup tree, which would be huge?  Doing that in response to an error or Ctrl-C seems absurdly time-consuming.

yarikoptic:

Indeed. FWIW, aiming for incremental changes, I think we can minimize the amount of time when interruption would lead to requiring such a full blown roll back. E.g. if we

do full analysis of what keys need to be downloaded, renamed, and removed without any changes to the data on drive.  Interruption would result in non need for roll back or any other actions to cleanup
do all necessary downloads into temporary space, e.g. .dandi-s3-backup-downloads/{versionid}.{etag} at the top folder. If interrupted in this stage -- would just need to rm -rf .dandi-s3-backup-downloads/.
final stage: "expensive" to recover from if interrupted, hence interruption should be guarded (e.g. at least 3 CTrl-C's within 5 seconds, otherwise do not react): perform all planned rm, and mvs, and remove empty .dandi-s3-backup-downloads/ at the end (all files should be gone)

actually if we establish journal of those rm and mvs we could probably play them back as well leading to "lighter" way to recover, but I am not sure I would 100% trust it, thus running full fsck would still be desired for paranoid me.

WDYT? may be some better ways?

jwodder:

An alternative: Don't rollback on errors/Ctrl-C, just exit (after cleaning up any partially-downloaded files).  Instead, make the command capable of both (a) running against a backup directory in which a previous backup run was interrupted (so that a failed command can be immediately rerun, hopefully this time to completion) and (b) reverting a backup directory to an earlier backup by specifying an earlier date (so that a failed command can be forcefully rolled back by running the command with the date that the backup was previously at).

yarikoptic:

Re Alternative: it reads like what i have suggested as "rollback or match the prior state" feature above and you had a legitimate concern

But wouldn't the rollback have to be run against the entire backup tree, which would be huge?

or did you think back then I was suggesting it as something to do right upon error/Ctrl-C?
Also -- how would you know if current state of the backup is "legit" and not some partial one? I think we better have an explicit annotation for that.
In either case, I think that separating out analysis, from "fetching" and then actual "doing" as I suggested above might benefit greatly in minimizing time for when we could leave the backup in some incomplete/broken state. Don't you think so?

Do you think it would be too tricky to allow for two modes -- "staged" (the "analysis" based above) and "interleaved"?  I guess if was in Python could be at the level of what_todo "generator" vs "list" decision making.
My thinking: we would need a full heavy list only in the initial backup.  That is where we can proceed in "interleaved" mode, just getting a new key from the inventory, checking if present locally and if not -- yield the "action" to do to the next step. In "staged" -- it would establish the full list of actions to tackle first, and since changes should not be multitude if ran frequently -- should fit into memory easily IMHO.

@yarikoptic That seems doable, though I really don't like the names "interleaved" and "staged".  What about "immediate" and "pre-planned"?
I also have to point out that, based on my observations of previous runs of the program, the "staged"/"pre-planned" mode, when run on the dandiarchive inventories, will have to spend about 10 and a half hours just going over all the items before it gets around to actually doing anything.
In "staged"/"pre-planned" mode, when cleaning up after an error or Ctrl-C, do you want the actions performed so far to be rolled back or not?  (Just what is the fsck you're referring to?  I don't see how the fsck(8) command is relevant here.)
How do you expect removal of files from the local backup that don't correspond to any items in the inventories (#33) to work?  Determining whether a file should be deleted basically requires storing all the CSV entries in memory at once.

I have no strong opinion on names. "immediate" and "pre-planned" sound good.

will have to spend about 10 and a half hours just going over all the items before it gets around to actually doing anything.

I wonder why and either could be avoided e.g. via parallelization .

In "staged"/"pre-planned" mode, when cleaning up after an error or Ctrl-C, do you want the actions performed so far to be rolled back or not? (Just what is the fsck you're referring to?

sorry for likely using inappropriate terminology -- I often use fsck word for consistency-check. In the scope of this it would entail seeing local backup consistent with what is observed in inventory and the .s3invsync.versions.json (formerly .dandi-s3-backup-versions.json) files.

How do you expect removal of files from the local backup that don't correspond to any items in the inventories (https://github.com/dandi/s3invsync/issues/33) to work? Determining whether a file should be deleted basically requires storing all the CSV entries in memory at once.

Nothing 100% solid idea yet, but followed up on https://github.com/dandi/s3invsync/issues/33#issuecomment-2504826791

@yarikoptic

I wonder why and either could be avoided e.g. via parallelization .

Because there are over 400 CSVs to process, each with 300,000 lines, and they're already being processed concurrently (see the --inventory-jobs option).

You didn't answer this question:

In "staged"/"pre-planned" mode, when cleaning up after an error or Ctrl-C, do you want the actions performed so far to be rolled back or not?

I wonder why and either could be avoided e.g. via parallelization .

Because there are over 400 CSVs to process, each with 300,000 lines, and they're already being processed concurrently (see the --inventory-jobs option).

sure -- lots of work... do we know what it spends most of the time on?  I am thinking about the path-filter and "printing"/logging . As for path-filter -- if regex, likely is the cpu intensive part and if was replaced with --path-prefix-filter to use simpler analog of python's ".startwith" but might shorten time considerably?

In "staged"/"pre-planned" mode, when cleaning up after an error or Ctrl-C, do you want the actions performed so far to be rolled back or not?

I guess so since we do need to be able to get into a "consistent state".


AC: register service worker pre-caches static assets along with subsequent pages and static assets

WHEN I register a service worker
THEN I should have my static assets pre cached upon loading along with subsequent pages and static assets

/client/src-sw.js
// TODO: Implement asset caching

info is cached


Update Pages “home”

Automatically generated by Netlify CMS

👷 Deploy Preview for tech-works-for-us processing.

Name
Link

🔨 Latest commit
759a267a39c25fe0fd9949afeb5e8030b73b6771

🔍 Latest deploy log
https://app.netlify.com/sites/tech-works-for-us/deploys/6511a3d3771610000870b03e


Docs fixed

Moved "Custom Headers" part after "More complicated POST requests" and before "POST a Multipart-Encoded File" in quickstart doc for adding headers is mentioned in the last one.

I think the correct fix is actually to come up with an example that doesn't require the data parameter. Custom headers should come before anything with 'more complicated' in the title. =)

I see, I just decided that it would be enough because custom headers are not mentioned in "More complicated POST requests".

Yeah, I can see that, but I think headers are the simpler topic. If you feel strongly about it though I can merge as-is and consider changing it later. =)

Moved it back and set example to use only headers kwarg without data. :)

Sorry for not being careful, now it finally seems like I fixed everything that was needed.

I think i prefer the existing form of this. Thank you, though!


Convert root build.gradle to kts

Description
Follow up #1449, and fix the publish issue in #1503, apply ktlint task for kts files.

[x] PR description added
[ ] tests are added
[x] KtLint has been applied on source code itself and violations are fixed
[ ] documentation is updated
[x] CHANGELOG.md is updated

In case of adding a new rule:

[ ] Rule is added to rules documentation

@TWiStErRob Could you take a look at this PR?


Implement "Lock doors with reduced guard" as a variant of "lock doors"

It seems Volvo API also include a fnction to lock lock doors with reduced guard (alarme with less sensivity) 👍 https://developer.volvocars.com/apis/connected-vehicle/v2/endpoints/doors-windows-locks/#lock-doors-with-reduced-guard
It is usefull in some noisy environement, to avoid the siren trigger for no reasons (and for some EX30 owner, with a too sensitive volumetric sensor).
post :
https://api.volvocars.com/connected-vehicle/v2/vehicles/{vin}/commands/lock-reduced-guard

I use this feature in the Volvo app every time I take the car on a ferry, so would be nice to be able to set this through the addon.

This feature would still be nice to have, so no need for this issue to become stale


Add section [Bluetooth setup] -> [AT commands]

Section 12.3 (AT commands) exists, but is not in the sidebar menu (SUMMARY.md)

bors retry

bors ping

bors ping

bors ping

Merging manually because Bors seems to have some problems here.


1685: Add the ability to remove an old card during deep link activation

Short description
Currently when a user already has a card in the app and tries to activate a new card via a deep link, he receives the message Ihr Kartenlimit wurde erreicht. Bitte entfernen Sie vorher eine Karte, damit Sie eine neue aktivieren können. and is forced to first delete the old card and then repeat the activation.
This PR adds the ability to delete the old card during activation.
Proposed changes

change the displayed message to Sie haben bereits einen aktivierten Pass auf diesem Gerät. Wenn Sie fortfahren, wird der alte Pass gelöscht.
if the user proceeds, first delete the old card and then activate the new card
this only applies to projects, where only 1 card is allowed (so, Nürnberg shouldn't be affected)
small refactoring of RemoveCardConfirmationDialog to be able to reuse it

Side effects
If the user proceeds and deletes the old card, but then some error occurs (Fehler beim Lesen des Codes for example), the user will have neither the old nor the new card in the app. Not sure if this is an issue though. If he keeps the old QR code, he should be able to activate the old card again.
Testing
Instruction for Koblenz:

Open administration --> login as a koblenz project admin --> go to settings --> create token
Import users using curl or Postman
curl -H "Authorization: Bearer <my_token>" -F<EMAIL_ADDRESS>http://<IP_ADDRESS>:8000/users/import

CSV content:
regionKey,userHash,startDate,endDate,revoked
07111,"$argon2id$v=19$m=19456,t=2,p=1$cr3lP9IMUKNz4BLfPGlAOHq1z98G5/2tTbhDIko35tY",01.01.2024,01.01.2025,false

Go to http://localhost:3000/ and select Koblenz
Go to http://localhost:3000/erstellen, create a card with a birthday 10.06.2003 and a reference number 123K
Activate the card in the app
Go to http://localhost:3000/erstellen again, create another card with a birthday 10.06.2003 and a reference number 123K, but enter some different name (required to produce a different cardInfo, otherwise you get the Dieser Pass wurde bereits auf dem Gerät aktiviert. message; alternatively some other data should differ)
Try to activate the new card via deep link:
npx uri-scheme open berechtigungskarte://koblenz.sozialpass.app/activation/code#<activation-code> --android
It should be possible to remove the old card during the activation.

Resolved issues
Fixes: #1685

If we activate the new card first and it fails and the old card is not removed, it will also be a bit strange because the user has confirmed the removal.

I see ...
Another idea would be to adjust the phrasing to make it less confusing. Maybe if we phrase it like "If you continue, the previous card will be replaced." it would be less confusing? I am also fine with your implementation, though.

Another idea would be to adjust the phrasing to make it less confusing. Maybe if we phrase it like "If you continue, the previous card will be replaced." it would be less confusing?

You mean, swap deletion and activation AND change the message text, right?
I'm not sure my point is plausible, but I feel like it would also be more error-prone:
What if an error occurred during deletion?
What if the app stopped (the device turned off or something) after the new card was activated but before the old card was deleted?
The current order is at least straightforward.

You mean, swap deletion and activation AND change the message text,

Yes

I'm not sure my point is plausible, but I feel like it would also be more error-prone:
What if an error occurred during deletion?
What if the app stopped (the device turned off or something) after the new card was activated but before the old card was deleted?

I guess the proper way would be to pass an optional cardToReplace to activateCard and (if cardToReplace is not null) instead of simply adding the newly activated card to the list of cards, replace cardToReplace with the new one and persist this array in one step. Then we would never violate the maxNumberOfCards  "constraint".

The current order is at least straightforward.

It's easier to implement, but can be more frustrating for users in the error case (e.g. if the user wants to activate his new card, but his old card would still be valid for some period of time). As I said, I'm fine with both options, though :D

Maybe, the question rather is why do we implement this, instead of just allowing to have multiple cards activated at the same time?

Maybe, the question rather is why do we implement this, instead of just allowing to have multiple cards activated at the same time?

i don't see any issues to increase the number of cards if its getting too complicated.
I'm a big fan of easy solutions and i didn't expect this getting so tricky.
F.e. with a BahnCard you can also have multiple cards. And if someone has even exceeded the limit then he/she should delete one card.

i don't see any issues to increase the number of cards if its getting too complicated.

Yes, but I don't quite recall why we did set the card limit to 1 for Bayern. Maybe we wanted to sell this feature of having multiple Ehrenamtskarten activated (@dkehne) ?

but because Bavaria wanted to make sure that two people cannot use the card in parallel.

I think this is a different kind of restriction @dkehne .

The number of devices that can be activated by a single card at the same time (the one you mentioned  - this is restricted to 1 for all projects currently).

vs

The number of activated cards on a single device (this is different between Bayern & Nürnberg currently; we restricted this for Bayern to 1)

Can we lift the latter restriction in Bayern to 5 (or whatever number we have for Nürnberg)?

Yes we can lift it to the limit we have in. Nuremberg. I will inform bavaria

So, I will just close this PR, right?

I mean, maybe this should be decided in the entitlementcard dev group / weekly, I just wanted to share my thoughts 😬
Didn't want to enforce any particular solution here :D

I mean we can ask for opinions on monday. I won't be here but i vote for extending the amount of cards

Just one concern from my side:
If we just allow multiple cards, and a Koblenz user issues a new card (e.g., via card extension), the old card will be automatically revoked but will stay in the app, however it can no longer be used, and there is basically no point in keeping it.

But we probably want to allow multiple cards on a single device for Koblenz for the same reasons we allow it for Nürnberg: E.g., parents might want to have the card for their child and for themselves on their phone
I am not sure how we design(ed) the card extension workflow. But maybe, if the process is started from within the app, it could be possible to add the information (for example through the deeplink) which card should be replaced when activating the new card.

I think this could get quite complicated when someone has two cards (the own one and the one of her/his child) activated and want to add new one. Which of the two activated cards is then deleted?
Therefore I think I also prefer to increase the number of cards and the user needs to delete manually first. Not really a nice flow though.

i created an issue for increasing card amount here
https://github.com/digitalfabrik/entitlementcard/issues/1741
i think this pr can be closed afaics

Can we delete this branch? :eyes:

maybe :)


cxxrtl should not be counting steps in module::step()

This was introduced in commit https://github.com/YosysHQ/yosys/commit/c382d7d3ac2b713c5b3f55e81430e121520a1787.
There are several issues with this approach:

There's no guarantee module::step() gets called at all. For example it is completely OK to call cxxrtl_eval and cxxrtl_commit directly, and Amaranth does so.
Even if it's called, it will not be called in submodules (since for those, eval and commit get called directly), so time will always be 0 inside of those.
In general CXXRTL has no notion of time (at the time being anyway), so the formatter should probably print $time/$realtime (what is $realtime anyway?) as it is (expecting something on the stdout to filter the output) or use some sort of escaping mechanism.

Changed to allow injecting any value for $time/$realtime via a context object provided to eval().


KAFKA-4404: Add javadocs to document core Connect types, especially that integer types are signed

@hachikuji @ijuma @kkonstantine @shikhar Trivial javadoc update to try to head off further questions around the restriction to only having signed types.

LGTM


iOS: respect "withCredentials: false" in XMLHttpRequest

🐛 Bug Report

Event with withCredentials: false network requests still sending cookies.
To Reproduce

Request yandex.ru (it will set yandexuid cookie in response)
Request anything else with withCredentials: false and you will see that cookies are sent

Expected Behavior

No cookies with withCredentials: false.
Code Example

Actually there is very easy fix to it
--- node_modules/react-native/Libraries/Network/RCTNetworking.mm	2019-03-18 17:28:57.000000000 +0300
+++ node_modules/react-native/Libraries/Network/RCTNetworking.mm	2019-03-18 17:29:36.000000000 +0300
@@ -245,10 +245,13 @@
   NSURL *URL = [RCTConvert NSURL:query[@"url"]]; // this is marked as nullable in JS, but should not be null
   NSMutableURLRequest *request = [NSMutableURLRequest requestWithURL:URL];
   request.HTTPMethod = [RCTConvert NSString:RCTNilIfNull(query[@"method"])].uppercaseString ?: @"GET";
+  request.HTTPShouldHandleCookies = [RCTConvert BOOL:query[@"withCredentials"]];

-  // Load and set the cookie header.
-  NSArray<NSHTTPCookie *> *cookies = [[NSHTTPCookieStorage sharedHTTPCookieStorage] cookiesForURL:URL];
-  request.allHTTPHeaderFields = [NSHTTPCookie requestHeaderFieldsWithCookies:cookies];
+  if (request.HTTPShouldHandleCookies == YES) {
+      // Load and set the cookie header.
+      NSArray<NSHTTPCookie *> *cookies = [[NSHTTPCookieStorage sharedHTTPCookieStorage] cookiesForURL:URL];
+      request.allHTTPHeaderFields = [NSHTTPCookie requestHeaderFieldsWithCookies:cookies];
+  }

   // Set supplied headers.
   NSDictionary *headers = [RCTConvert NSDictionary:query[@"headers"]];
@@ -259,7 +262,6 @@
   }];

   request.timeoutInterval = [RCTConvert NSTimeInterval:query[@"timeout"]];
-  request.HTTPShouldHandleCookies = [RCTConvert BOOL:query[@"withCredentials"]];
   NSDictionary<NSString *, id> *data = [RCTConvert NSDictionary:RCTNilIfNull(query[@"data"])];
   NSString *trackingName = data[@"trackingName"];
   if (trackingName) {

If everyone is okay I can make PR for it.
Environment

 React Native Environment Info:
    System:
      OS: macOS 10.14.3
      CPU: (8) x64 Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz
      Memory: 29.28 MB / 16.00 GB
      Shell: 3.0.2 - /usr/local/bin/fish
    Binaries:
      Node: 11.5.0 - ~/.nvm/versions/node/v11.5.0/bin/node
      Yarn: 1.15.2 - /usr/local/bin/yarn
      npm: 6.9.0 - ~/.nvm/versions/node/v11.5.0/bin/npm
      Watchman: 4.9.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
    IDEs:
      Android Studio: 3.3 AI-182.5<IP_ADDRESS>64788
      Xcode: 10.1/10B61 - /usr/bin/xcodebuild
    npmPackages:
      react: 16.7.0 => 16.7.0
      react-native: 0.57.8 => 0.57.8

@DimitryDushkin yes please; could you send a PR over and link to this issue + tag me.
Thanks


Add Compile-Time Type-Checking using tl

tl is a static type checker for Lua: https://github.com/teal-language/tl/blob/master/docs/tutorial.md (like TS for JS). It would be great to have optional type annotations in moon+ so that moon+ compiles to tl, which then checks and compiles to Lua.
Two issues come to mind:

Figuring out the type annotation syntax in moon+
Changing the compiler to keep those annotations. Fortunately, there shouldn't be an issue even for the moonscript-specific class types, because they can be represented by records: https://github.com/teal-language/tl/blob/master/docs/tutorial.md#records

Or, the tl type checker portion could be ported to the moon+ compiler. The majority of code in the tl compiler (https://github.com/teal-language/tl/blob/master/tl.tl) is the actual parser and AST traversal, which could be replaced by the existing parser in moon+.

Just considered getting some static type check functions implemented before answering to your issue. But I'm not getting time to finish the work recently. Porting tl type checker is the way I preferred. Since this feature could be done in C++ and speed up compile time a little.

+1 this will be awesome

Played with tl for a while, I find it won't functioning well to just add type annotations support for Moonscript. Because Moonscript is generating some anonymous functions without annotations. For example:
-- assume type annotations is supported like this
join = (strs: {string}): string -> table.concat strs,", "

-- the codes below all look fine to compile
{string} strs = [tostring i for i = 1, 10]
print join strs

print join [tostring i for i = 1, 10] as {string}

compile them to tl:

local join: function(strs: {string}): string
join = function(strs: {string}): string
  return table.concat(strs, ", ")
end

 -- these generated codes can be accepted by tl
local strs: {string}
do
  local _accum_0 = { }
  local _len_0 = 1
  for i = 1, 10 do
    if i > 4 then
      _accum_0[_len_0] = tostring(i)
      _len_0 = _len_0 + 1
    end
  end
  strs = _accum_0
end
print(join(strs))

 -- these generated codes won't be accepted by tl
print(join((function() -- no way to annotate this anonymous function
                          -- because it's invisible for user
  local _accum_0 = { }
  local _len_0 = 1
  for i = 1, 10 do
    if i > 4 then
      _accum_0[_len_0] = tostring(i)
      _len_0 = _len_0 + 1
    end
  end
  return _accum_0
end)() as {string})) -- without annotations tl sees this anonymous function is returning nil
             -- then compilation fails

I think there's a type called any to deal with this, but that might break type checking further up the call chain. Really the only realistic option is to perform the type checking in MoonPlus itself, before emitting the code.

Implementing static check feature in MoonPlus will take a lot of time and work. So I've added a workaround feature to support static typing and generate codes for some Lua dialect like teal, for example:
macro text var = (name, type)->
  "local #{name}:#{type}", {name}

$var num, number
num = 123

$var set, {string:boolean}
set =
  valueA: true
  valueB: false

Compiles to:
local num:number
num = 123;
local set:{string:boolean}
set = {
  valueA = true,
  valueB = false
}

You can find more examples to write codes for teal-lang in MoonPlus. See teal-lang.mp and macro-teal.mp.

I really don't like it at all.
Another option, strong typing need not be required, (like in PHP).
So, I propose if we can define something like this:
Common way:
my_function = (a, b) -> a + b
Typed syntax ?
my_function = (a : int, b : int) : int -> a + b
Typed syntax ?
my_function = (int a, int b) : int -> a + b
class typed syntax?
class M
  method: (a : int, b : int) : int =>
    a + b
  method2: (a, b) => a + b

Yes, it's a temporarily ugly approach to introduce the static type check functions by compile codes to tl. Before the built-in static type check functions implemented, I think it's worth to try out more possibilities by introducing some small language features. To compile moon codes to tl, we can just allow user to write some macro functions by hand, and offer the users a special mode to insert any text into the complied codes, then they will have the ability to generate codes other than Moonscript or Lua. Customize their compiler functions to do something more.

The teal-lang type annotation, type, record and 'as' keyword syntax are now supported in Yuescript as optional features. See commit 38e2f978d76b25f626350a1766c7f5377e58e8dc. And the yue compiler tool can now compile Yuescript to tl codes with yue -tl file.tl.yue then generate file.tl, or compile Yuescript with tl annotations to Lua without -tl flag yue file.tl.yue and then get file.lua.
I'm going to make yue tool run teal annotated codes with type check functions directly, and add options to do a type check for Yuescript in a simpler way.

Played with the type annotations for Yuescript for a while, I felt that it would not be a good idea to introduce such feature in this language. Since Yuescript is a language to be expressive and extremely concise, why don't we just code in teal-lang for static type check functions ?

Rather than compiling to Teal, it might make more sense to compile to Luau. It seems to be popular and closer to vanilla Lua, with optional type information.

Since Yuescript is a language to be expressive and extremely concise, why don't we just code in teal-lang instead for the static type check functions?

I don't think that a concise and a type-safe language are mutally exclusive. Most of the aspects where Yuescript is more consise than Lua are ones where it pretty much just removes boilerplate. A type system is different to that IMO. A type system makes your code more maintainable and easier to work with down the line.
Also, I think it's a bad idea to compile to any intermediary language, be it Teal or Luau (side note: Luau is only compatible with Lua 5.1) or something else. It should just compile to Lua as ususal. The real benifit of things like TypeScript is the options for advanced IDE tooling. But of course, this tooling doesn't exist (yet?). I think that there would first have to be some kind of "Yuescript language server" for type annotations to really make sense.

On Fri, Mar 24, 2023 at 01:47:28AM -0700, Simon B. wrote:

Also, I think it's a bad idea to compile to any intermediary language,
be it Teal or Luau (side note: Luau is only compatible with Lua 5.1) or
something else.

Note that Luau is not an intermediate language; it has its own (rather fast) interpreter. Type information can aid that interpreter in making optimizations.


Add support for Firefox OS

Follow up from https://github.com/phonegap/connect-phonegap/pull/89
Hi @mwbrooks,
This is a new version of the patch, rebased, made into a single commit and with query.version change removed as requested. Started a new PR because that other one was from master.
I started looking into adding zip deploy support for Firefox OS too. To get it working I added the CORS headers middleware here and I also needed to make a fix to our file plugin implementation. Next I need to add fxos support to org.chromium.zip. It may take a while to get all dependencies released and in good shape, but I kept the CORS change here as it's harmless to other platforms. Happy to remove from this PR if needed.
Thanks!

I assume this pr is dead along with Firefox OS. Please close.


"context" is imported on golang 1.6

cursorcontext.go contains "context": "context", but context package does not exist in go 1.6

I don't like where this is going if we're talking about the long term, but probably I will merge it anyway.
Maybe I will make a feature which scans GOROOT libs in gocode on first launch, but this is a questionable thing as well, what if the scan misses somehow. Will also make first time completion pretty slow in some cases.
I don't know, sublime text does reindexing of files on launch and people are okay with it. After all it's just reading, no harm is done (e.g. to SSD).
Anyways, just some thoughts, will merge the hack as is though soon.

Maybe I will make a feature which scans GOROOT libs in gocode on first launch, but this is a questionable thing as well, what if the scan misses somehow. Will also make first time completion pretty slow in some cases.

A proper and implementation-heavy solution I can think of would be to use api files in https://github.com/golang/go/tree/master/api and write some Go code that, at go generate time, produces a static table of known autocompletions for each version of Go, with appropriate build tags.
It'd be very similar to what goimports does, e.g., see https://github.com/golang/tools/blob/master/imports/mkstdlib.go and the file it generates only for the current version of Go https://github.com/golang/tools/blob/master/imports/zstdlib.go (warning, large file).


API信息

可以获取当前插件API信息吗？
想获取当前在使用的工作流名称

你想通过什么方式获取？http request还是要在别的插件js或者python运行时代码获取？

From: Zhang Jing Xuan @.>
Sent: Wednesday, July 10, 2024 5:25:21 PM
To: 11cafe/comfyui-workspace-manager @.>
Cc: Subscribed @.***>
Subject: [11cafe/comfyui-workspace-manager] API信息 (Issue #435)
可以获取当前插件API信息吗？
想获取当前在使用的工作流名称
―
Reply to this email directly, view it on GitHubhttps://github.com/11cafe/comfyui-workspace-manager/issues/435, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEMEEOISA5G3SDYATBFYZILZLT4YDAVCNFSM6AAAAABKUPZQS2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGQYDAMRSGIZDIOI.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

http或者python
---- 回复的原邮件 ----
| 发件人 | Weixuan @.> |
| 日期 | 2024年07月10日 18:15 |
| 收件人 | @.> |
| 抄送至 | Zhang Jing<EMAIL_ADDRESS>|
| 主题 | Re: [11cafe/comfyui-workspace-manager] API信息 (Issue #435) |
你想通过什么方式获取？http request还是要在别的插件js或者python运行时代码获取？

From: Zhang Jing Xuan @.>
Sent: Wednesday, July 10, 2024 5:25:21 PM
To: 11cafe/comfyui-workspace-manager @.>
Cc: Subscribed @.***>
Subject: [11cafe/comfyui-workspace-manager] API信息 (Issue #435)
可以获取当前插件API信息吗？
想获取当前在使用的工作流名称
―
Reply to this email directly, view it on GitHubhttps://github.com/11cafe/comfyui-workspace-manager/issues/435, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEMEEOISA5G3SDYATBFYZILZLT4YDAVCNFSM6AAAAABKUPZQS2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGQYDAMRSGIZDIOI.
You are receiving this because you are subscribed to this thread.Message ID: @.***>
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


[ST] ETH to Mode execution problems

Describe the bug
A 1.01 ETH transaction from Ethereum to Mode is reconciled but not executed https://connextscan.io/tx/0xeb3bfae3165373d309286c8950dfc87350c93ec2e92a07d4075dab4332ffab11
The user was prompted to bump slippage a couple of times (now sitting at 5%), but the pool's balance is atrocious (see screenshot). What can we do to get the transfer completed while avoiding high slippage for the user?
Where did the user initiate this transfer?
Connext UI
Link to ticket in intercom
https://app.intercom.com/a/inbox/qge2vd8z/inbox/shared/all/conversation/1552?view=List
Screenshots

It's executed now


Error with Datahub installation - Error starting userland proxy: Bind for <IP_ADDRESS>:8998 failed: port is already allocated

Hi all,
When stepped into Step 4 of the Datahub Docker setup, I encounter the above error message when i copied and pasted this command
docker run -ti --publish <IP_ADDRESS>:8090:8090 --publish <IP_ADDRESS>:8998:8998 --publish <IP_ADDRESS>:9225:9225 --publish <IP_ADDRESS>:50070:50070 --name datahub --hostname datahub --network dev-net datahub run --agree-to-sap-license --hdfs --livy
I have reset Docker to factory default multiple times and still no luck. Any pointers? I have also attached the screenshot for your reference.

On a different note, I think I have installed it correctly with no error and this screenshot proves it.

You might try:

just removing Livy from your Docker run command by removing the "--publish <IP_ADDRESS>:8998:8998" and "--livy"  to start SDH without Livy. It's not necessary to explore SDH, Vora, HDFS tutorials with the Dev Edition.

docker rm datahub and re-running your docker run command

publishing to a different port like --publish 8998:10000 (or vice versa, can't recall)

look at this issue (resolved) to change Docker container type from linux to Windows: https://github.com/SAPDocuments/Tutorials/issues/2548

consult the SDH Dev Ed FAQ here, particularly for Docker-Toolbox, which is similar to how Docker for Windows works by spinning up a VM behind the scenes:  https://blogs.sap.com/2017/12/06/faqs-for-sap-data-hub-developer-edition/

rebuild your SDH image: docker build --tag datahub:1.4 .

I use Docker for Mac and below is the docker run command successful for me with SDH1.4. I do recall initially having an issue running with livy (may have been on SDH 1.3), and don't think I did anything to resolve it, it just worked next time. That said, I haven't explored using Livy yet (REST interface for interacting with Apache Spark).
docker run -ti --publish 8090:8090 --publish 8998:8998 --publish 9225:9225 --publish 50070:50070 --name datahub --hostname datahub --network dev-net datahub:1.4 run --agree-to-sap-license --hdfs --livy

Hope that helps!
Doug

I'm beginning to love this community and thank you Doug!
Here is what I did:

Removed Livy from Docker run command by removing the "--publish <IP_ADDRESS>:8998:8998" and "--livy" to start SDH without Livy.

docker rm datahub and re-running docker run command

I'm going to ignore Livy for now till we need it and attached is my progress.

Closing issue. Thank once again @douglasmaltby

You're welcome @	simoncht!
I don't "work here", though. I'm just another SAP data geek, on vacation in Vietnam, passing the time doing some self-learning tutorials (SCP, API management, ML/PAL/APL, Fiori, etc.) and encountering other issues in these tutorials. I just happened to see your SDH issue come though and have some experience with it...glad it worked out. Good luck!

@douglasmaltby Thanks once again and enjoy your holiday in Vietnam.

Thank you for your feedback. The issue seems to be resolved, so I am closing the issue.
If you still have questions, feel free to reopen the issue.


Fix duplicate license detections and copyright detections issue

Fixes OLCS-481

I don't have objections to changes here.  Generally LGTM.


Error chat.postMessage payload

[WARN]  web-api:WebClient:0 The top-level `text` argument is missing in the request payload for a chat.postMessage call - It's a best practice to always provide a `text` argument when posting a message. The `text` is used in places where the content cannot be rendered such as: system push notifications, assistive technology such as screen readers, etc.
Error: Error in notifySlack function:
Error: An API error occurred: no_text

현재 pr code review ai git action과 같이 돌고 있습니다. postMessage 함수가 실행될 때 text 필드 문제로 job이 실행되지 않습니다!

안녕하세요 :) 어떤 이벤트가 실행될 때 발생한 에러일까요?

Pr open이겠군요. 한번 확인해볼게요

네 맞습니다! pr open 시 코드리뷰 ai도 같이 action돌 때 에러가 발생합니다!

@seongho-son 일단 고쳐서 코드리뷰 ai랑 테스트해보긴 했는데 확인해주시면 감사하겠습니다.

적용 후 다른 에러가 발생합니다!
Error: Error in notifySlack function:
Error: Cannot read properties of null (reading 'login')

Error: Error in notifySlack function:
Error: Cannot read properties of null (reading 'login')
이것도 pr open에서 발생한 에러일까요?

워크플로우 - Notify Slack 스텝부분 보시면 Event Loaded 다음에 github event 객체가 로그로 찍혀있을 거예요. 거기서 comment.user가 어떻게 찍히는지 로그 올려주실 수 있을까요? 중요 정보는 가리고 올려주시면 됩니다.

따로 로그에 comment.user는 없습니다.

이런 형태의 반복이고 comment.body 말고 다른건 없습니다!
[son/received_events](https://api.github.com/users/seongho-son/received_events)","repos_url":"https://api.github.com/users/seongho-son/repos","site_admin":false,"starred_url":"[https://api.github.com/users/seongho-son/starred{/owner}{/repo}](https://api.github.com/users/seongho-son/starred%7B/owner%7D%7B/repo%7D)","subscriptions_url":"https://api.github.com/users/seongho-son/subscriptions","type":"User","url":"https://api.github.com/users/seongho-son"}}
comment.body: 코드리뷰 요청이 슬랙메시지로 전달되었어요: [슬랙 메시지 바로가기]

깃헙 조직에서 사용되는 프로젝트에 깃헙 토큰은 제 개인용을 사용하는데 이 부분은 괜찮은건가요?

깃헙 토큰 개인용 쓰셔도 상관 없어용

그럼 오류는 어떤 이벤트에서 발생한 걸까요?

재현 가능한 작은 레포를 만들어주셔서 공유해주시면 감사하겠습니다.

소중한 시간을 뺐어 죄송합니다. pull_request_review와 pull_request의 차이점이 무엇인가요?

저 실패한 액션 오른쪽에 details 버튼이 있는데 클릭하시고 아래처럼 Notify Slack step을 열어보셔서
control + f로 Event Loaded 찾기 하시면 github event 객체 로그가 찍혀있을 거예요. 그 로그를 긁어서 올려주실 수 있나요?

수정했는데 한번 확인해주시겠어요?

해당 에러가 발생하고 있습니다. 제쪽에 문제가 없는지 확인해보겠습니다. 신경써주셔서 감사합니다 :)
[WARN]  web-api:WebClient:0 The top-level textargument is missing in the request payload for a chat.postMessage call - It's a best practice to always provide atextargument when posting a message. Thetextis used in places where the content cannot be rendered such as: system push notifications, assistive technology such as screen readers, etc. Error: Error in notifySlack function: Error: An API error occurred: no_text

slack.js > postMessage() 함수에서 text 필드가 존재하지 않아서 발생하는게 아닐까요?

음... slack.ts 보시면 slackClient.chat.postMessage 부분에 전부 다 text 넣어줬어요. postMessage에서 발생하는 문제가 맞을까요? 어떤 깃헙 이벤트에서 발생했나요?

엇 안들어있는거 아닌가요?

발생은 pull_request_review입니다!

아 slack.js말고 전부 index.js 안에 들어가요

아 updateMessage에도 text를 넣어주는 걸 깜빡했네요 ㅠ 확인 부탁드립니다.

흠.. 여전히 발생하네요..
[WARN]  web-api:WebClient:0 The top-level textargument is missing in the request payload for a chat.postMessage call - It's a best practice to always provide atextargument when posting a message. Thetext is used in places where the content cannot be rendered such as: system push notifications, assistive technology such as screen readers, etc. Error: Error in notifySlack function: Error: An API error occurred: no_text
예시 레포입니다. 재현 가능한 레포는 아닌데 정보가 부족할까요?

보내주신 레포는 seongho-son/codereview-action@main 제 거가 아닌 포크 뜬 걸 보고 있네요. 제 걸로 테스트하신 게 맞으실까요?

제 걸로 쓰고있는데 업데이트 하실때마다 반영하고 테스트하긴 했습니다. 무조건 rose님 걸로 사용해야될까요?

1.0.18 버전 반영되었나요?

네 지금 1.0.19 입니다 :)

독립적으로 쓸때는 발생하지 않는데 ai 코드리뷰와 같이 사용하면 이렇네요. pull_request open시 ai 코드리뷰가 생성되면서 submitted에 문제가 생긴걸까요?

코드리뷰 ai가 pr open시 review를 남기면서 NotifySlack의 submitted에서 에러가 발생하는것 같습니다. 코드리뷰 ai에서 남긴 리뷰어의 데이터 타입이 NotifySlack의 submitted 내부 코드와 일치하지 않는 문제가 아닐까 싶네요. 바쁘실텐데, 코멘트는 이정도만 남기겠습니다.
다른 개발자분과 이렇게 소통을 해서 재밌었습니다. 바쁘신 와중에 신경써주셔서 정말로 감사합니다 :)

ㅠ 알겠습니다. 일단 close 하겠습니다.

no_text 에러가 단순히 postMessage에 text 필드가 존재하지 않는 경우인데, 아래와 같은 코드는 인자로 받은 text값이 없으면 no_text 에러가 발생할 수 있을 것 같습니다. text: text || '' 이런 방식을 써보는게 어떨까요?
export async function postThreadMessage(ts: string, text: string) {
  if (!text.includes("![image](")) {
    return await slackClient.chat.postMessage({
      channel: slackChannel,
      text,
      thread_ts: ts,
    });
  }

fork한 레포에서 테스트해본 결과 text가 비어있어서 발생했던 에러였습니다!

이상하네요 post thread message 부분에 text 필드 추가했는데.. 오리진 레포하고 싱크가 잘 맞는지 확인해주셔야할 것 같아요


refactor: major project structure update

Major project structure update applying

learnings from building atlassify
converge closer to the latest https://electron-react-boilerplate.js.org/ example

Main updates include:

split code into main and renderer src components for clear separation of duties
update main to use TypeScript and split into smaller files (main, menu, icons, utils, updater)
improve webpack configs, including code minimizing for production builds
use webpack plugins for templating html and extracting css
use webpack plugins to copy twemoji svgs into final build with updated EmojiText implementation to avoid reading directly from node_modules path.
move all non-core dependencies from dependencies to devDependencies
rename jest test suites to match new folder structure

Any way to split this into chunks, to ease review? :)

Any way to split this into chunks, to ease review? :)

Yep - can be done...


Add retry to CollectionX

recover could also be added to Lazy Collections (but also see https://github.com/aol/cyclops-react/issues/399)

Merged in https://github.com/aol/cyclops-react/pull/451
Added to all Functor (now Transformable types)


[Question] Running Playwright without downloading browsers

I already have my custom Chromium browser that I built. Is there anyway that I can run Playwright without checking/download existing browsers first? (I just want to use Chromium but Playwright download all the available browsers first)

Using PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD=1 and providing custom executablePath in browserType.launch() should do the trick.

Is there a way to do this when using connect()? If so, is there a way to set it at runtime? In our use case connect()/launch() is decided at runtime, and so setting a variable would be difficult.

Is there a way to do this when using connect()?

Yes, it should work for BrowserType.connect() as well as for launch(). Note that there is no launchServer() API in java and you'll need to launch it separately (e.g. using nodejs), if you want that process to skip download phase pass the same variable to it.

If so, is there a way to set it at runtime? In our use case connect()/launch() is decided at runtime, and so setting a variable would be difficult.

The environment variable should be set for the current process before Playwright.create() is called. System.getenv() returns read-only copy of the map so I don't think there is an easy way to set the env for the current process in runtime. We could add such flag as an option to Playwright.create()  but perhaps the code that starts the playwright client can decide on that? Can you share more details about how you decide whether to use connect() or launch()?


First commit React events

Hi! I kindly ask you for a review of this project. I have completed this milestone where we were required to:

Copy the files calculate.js and operate.js into a logic/ directory in your project.
Analyze the files calculate.js and operate.js. Add big.js as dependency).
Import the files in your Calculator component.
Test the app in the browser.

Kind Regards,
Pedro

Thank you for the recommendation!


change library for Promise to es6-promise (server-side)

cc @inikulin

Will be fixed after merge https://github.com/superroma/testcafe-hammerhead/pull/41

Ok, let's wait for #41 and then rebase.

FRP

FRP

It requires rebase now

:white_check_mark: Tests for the commit 0495b73e7351ba5abddec91cff68cdac83fbeb78 have passed. See details.
Tests are ok. We've found the issue with travis and saucelabs and will fix it in the gulp-qunit-harness. The problem appears when more than one travis tasks started in the one time.
So, @inikulin we can review the pr

LGTM then

LGTM


Minor fixes to make the lib working

The __init__.py files should not be needed in Python 3.
I want to have a CLI for this, but I guess it doesn't work if the script isn't there yet.

Yes you are right on the pthon3 not needing the init.py . I was fixing all warning messages I was getting. Perhaps this was not a blocker, but the script part was.


Ensure class is set for annotation targets of MethodParameterInfo

Relates to: https://github.com/quarkusio/quarkus/issues/7923

cc @n1hility

Thanks!


fix: assert with raise

Description
The way to assert was wrong, fixed them
Type of change
Please delete options that are not relevant.

[x] Bug fix
[ ] New feature
[ ] Document
[ ] Test
[ ] CI/CD
[ ] Refactor

How has this been tested

[x] Unittest
[ ] Others:

Checklist:

[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Let's make the world better✨😋🐍🌎

Codecov Report

Merging #45 (ad4f313) into main (c928cf9) will increase coverage by 0.16%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #45      +/-   ##
==========================================
+ Coverage   95.32%   95.48%   +0.16%
==========================================
  Files          21       21
  Lines         620      620
  Branches       66       66
==========================================
+ Hits          591      592       +1
  Misses         19       19
+ Partials       10        9       -1

Impacted Files
Coverage Δ

pypj/environment.py
95.65% <0.00%> (+1.44%)
:arrow_up:

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 c928cf9...ad4f313. Read the comment docs.


go1 branch not visible

The go1 branch the documentation mentions is not found.

Sorry about that.  It has been put back.  Open another issue if it doesn't work please.


HuggingFace VisualEncoderDecoderModel performs better (but slower)

I first used the huggingface implementation of Donut to train for my specific use-case and it was working well but very slow, (10x times on epochs for example).
So I decided to switch to the official implementation and saw a massive increase in my training times, however I noticed that the valuation loss was actually worse for the same relative epoch time for training and this was apparent when doing inference.
Here are some graphs to illustrate:

The green line is the huggingface implementation and the others is donut installed from source from this git repo.
As you can see, the the training time is much larger for the same relative epochs (its the same dataset across all of them), however the score is also better for the huggingface implementation.
I printed out both models to check their differences, which can be found here https://www.diffchecker.com/11ZHWVyn/
I'm a novice when it comes to machine learning, but I think that these models are essentially the same...
So why is one performing better than the other, but slower, and the other faster, but worse.
Could it be something to do with this?
Some weights of DonutModel were not initialized from the model checkpoint at naver-clova-ix/donut-base and are newly initialized because the shapes did not match:
- encoder.model.layers.0.blocks.1.attn_mask: found shape torch.Size([3072, 100, 100]) in the checkpoint and torch.Size([768, 100, 100]) in the model instantiated
- encoder.model.layers.1.blocks.1.attn_mask: found shape torch.Size([768, 100, 100]) in the checkpoint and torch.Size([192, 100, 100]) in the model instantiated
- encoder.model.layers.2.blocks.1.attn_mask: found shape torch.Size([192, 100, 100]) in the checkpoint and torch.Size([48, 100, 100]) in the model instantiated
- encoder.model.layers.2.blocks.3.attn_mask: found shape torch.Size([192, 100, 100]) in the checkpoint and torch.Size([48, 100, 100]) in the model instantiated
- encoder.model.layers.2.blocks.5.attn_mask: found shape torch.Size([192, 100, 100]) in the checkpoint and torch.Size([48, 100, 100]) in the model instantiated
- encoder.model.layers.2.blocks.7.attn_mask: found shape torch.Size([192, 100, 100]) in the checkpoint and torch.Size([48, 100, 100]) in the model instantiated
- encoder.model.layers.2.blocks.9.attn_mask: found shape torch.Size([192, 100, 100]) in the checkpoint and torch.Size([48, 100, 100]) in the model instantiated
- encoder.model.layers.2.blocks.11.attn_mask: found shape torch.Size([192, 100, 100]) in the checkpoint and torch.Size([48, 100, 100]) in the model instantiated
- encoder.model.layers.2.blocks.13.attn_mask: found shape torch.Size([192, 100, 100]) in the checkpoint and torch.Size([48, 100, 100]) in the model instantiated

Hi,
The HuggingFace implementation should have the same speed. Check out my demo notebooks here: https://github.com/NielsRogge/Transformers-Tutorials/tree/master/Donut.
I fine-tuned Donut on various datasets, didn't see any issues regarding speed.

@NielsRogge  it is slow. I tried training for 3000 images using donut, it takes around 40 hours. I am using the same script you provided.
How can we improve that?


Fix FramePanelBlock crash

Fixes LoremVIpsum/OpenSimpleFrames#20

don't crash on placing vertical panel
add vertical panels
store the block shapes statically
also make it actually run on 1.20 lol

hi there, yeah I probably didnt copy over the json things properly for 1.20, when I last built it it didnt crash lol, thanks for this - mind if I merge it and build it?, I can add you as a contributor of modrinth if you want :D

That didn't fix it...
Got this crash report
crash-2024-01-08_22.16.46-client.txt

is this just abandoned at this point?

sure, you can add me as a contributor if you want, makes no difference to me lol (never got a notif for this earlier)


added styling for responsive videos

Using this strategy for embedded responsive videos.
This is assuming a 16:9 ratio.  If you need something different, let me know @hollyschinsky.

Here is the corresponding markup: