Fault checker: check for ECALL faults

The fault checker currently works for non-ecall instructions. The fault checker should also take ECALLs (aka system calls) into account.

We're going to implement fault proofs as a part of future circuit revisions. Closing


[doc] Typo.

(Sorry Walter, the online work flow gives no control over the branch name.)

Auto-merge toggled on


bump dependency and remove deprecated linter

@devj3ns i have reverted intl back to 1.17.0 because some packages still depend on old version of intl, so it causes conflict while upgrading

Hey @icodelifee, thanks a lot for this PR.
I will merge and publish it tomorrow or on saturday.

Fleasy 0.14.2 is now available on pub.dev :)


WARNING: Failed to find MSVC. during inference

Using latest Visual Studio and python libraries. Cuda Toolkit 12.6.2 installed.
With your 3.10 python 3.12 version I got this error during inference.

and with https://huggingface.co/madbuda/triton-windows-builds/tree/main
I do not have that error

I am sorry. Everything working Okay. My mistake. Not all depenencies installed first.
Closing


Cant open menu

Hi,
Good morning,
There was an update to the game, and since then it's impossible to close or open the menu manually. It appears randomly.

I am aware that the game has gotten an update, NetWare will be fixed soon.


How to unit test a component that uses this service?

Hi,
I have a component that bring in the service in the constructor like this:
constructor(
public appInsights: AppInsightsService, ...
){ ... }
How do I then set up my spec file?
Thanks,
Ethan

Hey Ethan,
As these are outside of the component itself, I'd simply mock the AppInsightsService with some blank/noop() functions, just focus on the core functionality of your component itself.
You could just mock it like:
const AppInsightsService = {
  trackEvent: () => { },
  // etc etc
}


test revision for products and order

i added a new commit because one test was failing. i added a conditional on the login component so that it only pushes when there is browser history. i think its ready to be merged if anyone wants to do it


Fix broken test and pass creds through env

Removes the check for preferred_version, which isn't present in
Sauce anymore.
Allow user to specify SAUCE_USERNAME, SAUCE_ACCESS_KEY, TEST_JOB_ID
by environment variable.

@cgoldberg He made the changes you requested, anything holding this back?

@sontek I just merged the PR. thanks for the reminder... I totally forgot about it :)

@cgoldberg Thanks! I'll have a few PRs for you in a few minutes :)


Random Quote Generator

I want to add a random quote generator webpage built using HTML5, CSS3 and JavaScript to this repository.
Please assign me this issue @TammyAlok2

I created a pull request and added the html , css and js files for random qoute generator.
please assign the issue to me
HACTOBERFEST 2023

@TammyAlok2 I want to work on this issue. Please assign it to me.

I hope you find me well. Can you please this assign to me I can do this with minimum time. I hope u will assign it to me.

hello ! Please assign me this. I want to contribute.

Could you allow me to contribute in your project.

can you please assign this issue to me @TammyAlok2. I can create a web-page for that by using frontend technology.

Please assign this to me

Please Assign it to me i want to contribute to this issue


Set a Content-Security-Policy

For increased security we should see if we can enable a Content-Security-Policy without breaking the application.

@jeffpw-goog here are Play's docs on this if you haven't seem them yet.

I added a section on CSP to the frontend development wiki.

Per discussion in Slack, we will proceed with rolling out CSP despite #8143, and pursue the fix for that in parallel.

CSP is now enabled by default: https://github.com/civiform/civiform/pull/8218
Also cleaned up the staging override: https://github.com/civiform/civiform-staging-deploy/pull/163
This will go out in the release on September 24.
After that the only remaining work is to submit https://github.com/civiform/civiform/pull/8661 to clean up the config variable, which we can do after a few more weeks.


Closes #1209 Passive sync of entity lifecycle state

Initial implementation of syncing non-data state of entity(the store manager state and server store state). Syncing the client message tracker state is still WIP.

In the current PR multiple messages are sent to the passive to synchronize the entity state. One for the store manager state and one each for each server store state. I'm thinking of combining them all into a single message as I really don't see a reason why they should be separate.

Unified the life cycle state sync messages and added the client tracking information to the sync message.

Addressed the review comments.

This is not complete until we decide on the sync strategy. See discussion regarding syncing of lifecycle messages for cache on https://github.com/Terracotta-OSS/terracotta-apis/pull/146

test this please

Addressed the review comments.

Addressed the review comments.

test this please

Good to go with green build


So i'm getting this error.
'Stream' exists in both 'System.IO, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' and 'System.Runtime,
Windows has no issue running the app but Linux wont build.
csprog
<Project ToolsVersion="15.0" Sdk="Microsoft.NET.Sdk.Web">

  <PropertyGroup>
    <TargetFramework>netcoreapp1.1</TargetFramework>
  </PropertyGroup>

  <PropertyGroup>
    <PackageTargetFallback>$(PackageTargetFallback);portable-net45+win8+wp8+wpa81;</PackageTargetFallback>
  </PropertyGroup>

  <PropertyGroup>
    <UserSecretsId>aspnet-TownControl-ca78c3d2-5270-497a-8906-a504ae1a48c0</UserSecretsId>
  </PropertyGroup>

  <ItemGroup>

    <PackageReference Include="Microsoft.AspNetCore" Version="1.1.0" />
    <PackageReference Include="Microsoft.AspNetCore.Authentication.Cookies" Version="1.1.0" />
    <PackageReference Include="Microsoft.AspNetCore.Diagnostics.EntityFrameworkCore" Version="1.1.0" />
    <PackageReference Include="Microsoft.AspNetCore.Identity.EntityFrameworkCore" Version="1.1.0" />
    <PackageReference Include="Microsoft.AspNetCore.Mvc" Version="1.1.1" />
    <PackageReference Include="Microsoft.AspNetCore.StaticFiles" Version="1.1.0" />
    <PackageReference Include="Microsoft.EntityFrameworkCore.Design" Version="1.1.0" PrivateAssets="All" />
    <PackageReference Include="Microsoft.EntityFrameworkCore.Tools" Version="1.1.0-msbuild3-final" PrivateAssets="All" />
    <PackageReference Include="Microsoft.Extensions.Configuration.UserSecrets" Version="1.1.0" />
    <PackageReference Include="Microsoft.Extensions.Logging.Debug" Version="1.1.0" />
    <PackageReference Include="Microsoft.VisualStudio.Web.CodeGeneration.Design" Version="1.1.0-msbuild3-final" PrivateAssets="All" />
    <PackageReference Include="Microsoft.VisualStudio.Web.BrowserLink" Version="1.1.0" />
    <PackageReference Include="Npgsql.EntityFrameworkCore.PostgreSQL" Version="1.1.0" />
  </ItemGroup>
  <ItemGroup>
    <DotNetCliToolReference Include="Microsoft.EntityFrameworkCore.Tools.DotNet" Version="1.0.0-msbuild3-final" />
    <DotNetCliToolReference Include="Microsoft.Extensions.SecretManager.Tools" Version="1.0.0-msbuild3-final" />
    <DotNetCliToolReference Include="Microsoft.VisualStudio.Web.CodeGeneration.Tools" Version="1.0.0-msbuild3-final" />
  </ItemGroup>

</Project>

Code snap
using System;
using System.Threading.Tasks;
using Microsoft.AspNetCore.Mvc;
using TownControl.Data.DAL;
using TownControl.Data;
using Microsoft.Net.Http.Headers;
using Microsoft.AspNetCore.Http;
using Microsoft.AspNetCore.Hosting;
using System.IO;

......
                using (Stream fs = System.IO.File.Create(fileName))
                {
                    Photo.CopyTo(fs);
                    fs.Flush();
                }

Could you enclose your code in triple backticks (```), so that is shows here correctly?

Did some of the code is omitted as it's no relevant to issue.

Sorry, I primarily meant the csproj code, since that's not readable right now.

There you go sorry. didn't notice how bad it was until just now.

Whats the error you are getting in the build?

Will not build on Linux.
here is the full build outfit.
2017-02-27T18:46:29.3076770Z Controllers/DMVController.cs(96,24): error CS0433: The type 'Stream' exists in both 'System.IO, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' and 'System.Runtime, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' [/home/myagent/myagent/_work/1/s/TownControl/TownControl.csproj]
2017-02-27T18:46:29.3090540Z CSC : warning CS1685: The predefined type 'MethodBase' is defined in multiple assemblies in the global alias; using definition from 'System.Reflection, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' [/home/myagent/myagent/_work/1/s/TownControl/TownControl.csproj]
2017-02-27T18:46:29.3103690Z CSC : warning CS1685: The predefined type 'MethodInfo' is defined in multiple assemblies in the global alias; using definition from 'System.Reflection, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' [/home/myagent/myagent/_work/1/s/TownControl/TownControl.csproj]
2017-02-27T18:46:29.3116920Z CSC : warning CS1685: The predefined type 'ConstructorInfo' is defined in multiple assemblies in the global alias; using definition from 'System.Reflection, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' [/home/myagent/myagent/_work/1/s/TownControl/TownControl.csproj]
2017-02-27T18:46:29.3130450Z CSC : warning CS1685: The predefined type 'MemberInfo' is defined in multiple assemblies in the global alias; using definition from 'System.Reflection, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' [/home/myagent/myagent/_work/1/s/TownControl/TownControl.csproj]
2017-02-27T18:46:29.3136970Z
2017-02-27T18:46:29.3148010Z Build FAILED.

Trying to narrow it down. Are you doing anything differently on Windows vs Linux? Are you using the command line in both places?
@weshaggard any ideas?

Command Line in both. rc4 on both boxes.

@jbomhold3 can you share you assets file (obj/project.assets.json) from both your windows and linux builds somewhere? It would seem that for some reason the package graph is different between those.

http://xaltech.com/Linux-project.assets.json
http://xaltech.com/Windows-project.assets.json

Your Linux packages seem to be in a bogus state. What are the feeds you are using on both windows and unix? You can see that from the restore log. We should make sure your feeds are the same and I would also suggest delete your package cache (/root/.nuget/packages/) on your linux machine and retry the restore.

feeds are both the same trying to cache though this was a clean server

That worked. must have pulled in corrupt first time by

Great I'm glad that fixed your issue. I'm not sure what would have caused the corrupt package cache.


Fixed PHP 7.2 compatibility in Krumo

Which problem does this change fix?

<?php

< (new DateTime())

Thanks, although that fixed the warning but kinda broke the functionality (expansion). I committed another fix for that.


Example toString 'MM' and 'MMM' does not work correctly

Using format 'DD.MM.YYYY' in your format string MM shows double number of month (example: August prints 88)
Using format 'DD.MMM.YYYY' in your format string MMM cuts serial output after day (example: 13.08.2020 prints 13.)

Can you provide the code you're using? I'm not seeing the same result.

I forgot to mention that I use an ESP32 Heltec Wifi Kit32
Code 'toString' from examples is unchanged except the baud rate

I'm not seeing the same thing on an ESP32, it's working fine. We'll need as much info as you can provide to find out what's happening.
What environment are you using (Arduino, platform.io, etc)?
Which RTC are you using?
How are you setting the date/time on the RTC?

@pro-bowl: Could you try the minimal example below?
#include <RTClib.h>

void setup() {
    Serial.begin(9600);
    DateTime dt(2020, 8, 13);
    char buffer[] = "MM-DD-YYYY";
    Serial.println(dt.toString(buffer));
}

void loop() {}

Also, what version of RTClib are you using?

Sorry my fault, I found the problem. I was using another library with the DS3231 before and didn't reset the time with this lib.
Thanks for your tips.


Add visual effects to damage

Goal
To improve the feeling of inflicting and receiving damage with visual effects.
Resources

https://youtu.be/SN15PnaPYVc?si=TSlHBOYKzaZ24BYS&t=18 (watch on 0.25x and observe the blood effect when something takes damage, also the object being damaged turning white for a split second, and white splashes that appear around the damaged object)
https://youtu.be/6HXNxWbRgsg?si=1lhWoFlQ3MKNY33r&t=12 (watch on 0.25x and observe the blood effect when something takes damage)
https://github.com/djeedai/bevy_hanabi (potential tool to use for implementation)
https://github.com/vleue/bevy_easings (another potential tool to use for implementation if it's done with basic sprites)
https://docs.rs/bevy/latest/bevy/ui/index.html (another potential tool to use for implementation)

Re-opening as Bevy Hanabi is removed to bring WebAssembly support back in https://github.com/umut-sahin/mythmallow/pull/77.
We need to find a new particle system that works on WebAssembly as well. Some alternatives are:

https://github.com/mintlu8/berdicles
https://github.com/mbrea-c/bevy_firework
https://github.com/Lommix/bevy_enoki


Errors when running the test script (at least) on the Windows platform

When running jest && npm run lint on the Windows platform a few tests fail. Some time ago I files a PR that corrected all of these failed unit tests but now they seem to be back and it seems to be as if the Windows platform would not be part of the CI.
> jest && npm run lint

 FAIL  src\lib\npm\__tests__\npmLibDefs-test.js
  ● npmLibDefs › getInstalledNpmLibDefs › finds unscoped libdefs

    expect(received).toBe(expected)

    Expected value to be (using ===):
      2
    Received:
      0

      at _callee7$ (src/lib/npm/__tests__/npmLibDefs-test.js:313:45)
      at tryCatch (node_modules/regenerator-runtime/runtime.js:65:40)
      at Generator.invoke [as _invoke] (node_modules/regenerator-runtime/runtime.js:303:22)
      at Generator.prototype.(anonymous function) [as next] (node_modules/regenerator-runtime/runtime.js:117:21)
      at tryCatch (node_modules/regenerator-runtime/runtime.js:65:40)
      at invoke (node_modules/regenerator-runtime/runtime.js:155:20)
      at node_modules/regenerator-runtime/runtime.js:165:13
          at <anonymous>

  ● npmLibDefs › getInstalledNpmLibDefs › finds scoped libdefs

    expect(received).toBe(expected)

    Expected value to be (using ===):
      1
    Received:
      0

      at _callee8$ (src/lib/npm/__tests__/npmLibDefs-test.js:349:45)
      at tryCatch (node_modules/regenerator-runtime/runtime.js:65:40)
      at Generator.invoke [as _invoke] (node_modules/regenerator-runtime/runtime.js:303:22)
      at Generator.prototype.(anonymous function) [as next] (node_modules/regenerator-runtime/runtime.js:117:21)
      at tryCatch (node_modules/regenerator-runtime/runtime.js:65:40)
      at invoke (node_modules/regenerator-runtime/runtime.js:155:20)
      at node_modules/regenerator-runtime/runtime.js:165:13
          at <anonymous>

  ● npmLibDefs › getInstalledNpmLibDefs › finds libDef with fully-bounded semver range

    expect(received).toBe(expected)

    Expected value to be (using ===):
      2
    Received:
      0

      at _callee9$ (src/lib/npm/__tests__/npmLibDefs-test.js:385:45)
      at tryCatch (node_modules/regenerator-runtime/runtime.js:65:40)
      at Generator.invoke [as _invoke] (node_modules/regenerator-runtime/runtime.js:303:22)
      at Generator.prototype.(anonymous function) [as next] (node_modules/regenerator-runtime/runtime.js:117:21)
      at tryCatch (node_modules/regenerator-runtime/runtime.js:65:40)
      at invoke (node_modules/regenerator-runtime/runtime.js:155:20)
      at node_modules/regenerator-runtime/runtime.js:165:13
          at <anonymous>

 PASS  src\lib\__tests__\libDefs-test.js
 PASS  src\commands\__tests__\search-test.js
 PASS  src\lib\__tests__\flowVersion-test.js
 PASS  src\lib\__tests__\cacheRepoUtils-test.js
 PASS  src\lib\__tests__\semver-test.js
 PASS  src\lib\__tests__\codeSign-test.js
 PASS  src\commands\__tests__\install-test.js (7.38s)

Test Suites: 1 failed, 7 passed, 8 total
Tests:       3 failed, 91 passed, 94 total
Snapshots:   0 total
Time:        9.81s, estimated 11s
Ran all test suites.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>test-quick: `jest && npm run lint`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>test-quick script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\doberkofler.LBITS\AppData\Roaming\npm-cache\_logs\2017-10-17T15_25_23_973Z-debug.log
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>test: `npm run clean && npm run build && npm run test-quick`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>test script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\doberkofler.LBITS\AppData\Roaming\npm-cache\_logs\2017-10-17T15_25_24_225Z-debug.log
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\chokidar\node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"}
)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>prepublish: `mkdirp dist && npm run test`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>prepublish script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\doberkofler.LBITS\AppData\Roaming\npm-cache\_logs\2017-10-17T15_25_24_486Z-debug.log

This also happens on mac when trying to run the test suite after using eslint --fix or prettier --write. It seems whatever gets written out makes a few of the specs unhappy.

I understand but do actually not explicitly invoke eslint --fix or prettier --write but rather the build process of flow-typed does so automatically. I would suggest to rather not automatically fix any styling.


[consoleApp] [Manage] Updating permissions of the Role Internal/System via console fails

Describe the issue:
Updating permissions of Internal/system fails.
How to reproduce:
1.Access https://localhost:9443/console and create a tenant
2.Login as the tenant admin
3.Go to Manage > Roles > Internal/system > edit option > Permissions tab > Select some permisions > Update

Observation
UI says changes updated successfully. If you go back and see updated permissions are not available.
Environment
IS 5.11.0 BETA6 SNAPSHOT
mssql 2019 uniqueidjdbc primary db
tenant mode
seondary userstore mariadb 10.4

Related issue https://github.com/wso2/product-is/issues/10103

I also faced the same issue
Environment:

Product version: IS 5.11.0 Beta6 Snapshot
OS: Mac
Database: MySQL 8
Userstore: JDBC
Tenant mode

@ShanikaWickramasinghe Is this specific to only this role? Was it possible to set permissions for other roles?

@ashendes It was not updating the permissions for a newly created role as well

This issue has been fixed with https://github.com/wso2-extensions/identity-inbound-provisioning-scim2/pull/300 and https://github.com/wso2/charon/pull/293. The fix is not included in the BETA6 pack.


Precompile broken on julia-1.9.0-rc2 (also observed on julia-1.9.0)

This succeeded a couple hours ago (and when I pin ConcurrentUtilities to v2.3.1), but seems to be broken now:
ERROR: LoadError: ArgumentError: unsupported threadpool in @spawn: Threads.threadpool()
Stacktrace:
[1] var"@spawn"(source::LineNumberNode, module::Module, args::Vararg{Any})
@ Base.Threads ./threadingconstructs.jl:331
[2] #macroexpand#63
@ ./expr.jl:119 [inlined]
[3] macroexpand
@ ./expr.jl:117 [inlined]
[4] docm(source::LineNumberNode, mod::Module, meta::Any, ex::Any, define::Bool)
@ Base.Docs ./docs/Docs.jl:539
[5] docm(source::LineNumberNode, mod::Module, meta::Any, ex::Any)
@ Base.Docs ./docs/Docs.jl:539
[6] var"@doc"(::LineNumberNode, ::Module, ::String, ::Vararg{Any})
@ Core ./boot.jl:539
[7] include(mod::Module, _path::String)
@ Base ./Base.jl:457
[8] include(x::String)
@ ConcurrentUtilities /opt/julia-1.9.0-rc2/.julia/packages/ConcurrentUtilities/RLFRH/src/ConcurrentUtilities.jl:1
[9] top-level scope
@ /opt/julia-1.9.0-rc2/.julia/packages/ConcurrentUtilities/RLFRH/src/ConcurrentUtilities.jl:15
[10] include
@ ./Base.jl:457 [inlined]
[11] include_package_for_output(pkg::Base.PkgId, input::String, depot_path::Vector{String}, dl_load_path::Vector{String}, load_path::Vector{String}, concrete_deps::Vector{Pair{Base.PkgId, UInt128}}, source::String)
@ Base ./loading.jl:2012
[12] top-level scope
@ stdin:2
in expression starting at /opt/julia-1.9.0-rc2/.julia/packages/ConcurrentUtilities/RLFRH/src/try_with_timeout.jl:27
in expression starting at /opt/julia-1.9.0-rc2/.julia/packages/ConcurrentUtilities/RLFRH/src/try_with_timeout.jl:27
in expression starting at /opt/julia-1.9.0-rc2/.julia/packages/ConcurrentUtilities/RLFRH/src/ConcurrentUtilities.jl:1
in expression starting at stdin:2
ERROR: LoadError: Failed to precompile ConcurrentUtilities [f0e56b4a-5159-44fe-b623-3e5288b988bb] to "/opt/julia-1.9.0-rc2/.julia/compiled/v1.9/ConcurrentUtilities/jl_bFI73H".
Stacktrace:
[1] error(s::String)
@ Base ./error.jl:35
[2] compilecache(pkg::Base.PkgId, path::String, internal_stderr::IO, internal_stdout::IO, keep_loaded_modules::Bool)
@ Base ./loading.jl:2262
[3] compilecache
@ ./loading.jl:2129 [inlined]
[4] _require(pkg::Base.PkgId, env::String)
@ Base ./loading.jl:1772
[5] _require_prelocked(uuidkey::Base.PkgId, env::String)
@ Base ./loading.jl:1623
[6] macro expansion
@ ./loading.jl:1611 [inlined]
[7] macro expansion
@ ./lock.jl:267 [inlined]
[8] require(into::Module, mod::Symbol)
@ Base ./loading.jl:1574
[9] include(mod::Module, _path::String)
@ Base ./Base.jl:457
[10] include(x::String)
@ HTTP /opt/julia-1.9.0-rc2/.julia/packages/HTTP/LsfSO/src/HTTP.jl:1
[11] top-level scope
@ /opt/julia-1.9.0-rc2/.julia/packages/HTTP/LsfSO/src/HTTP.jl:40
[12] include
@ ./Base.jl:457 [inlined]
[13] include_package_for_output(pkg::Base.PkgId, input::String, depot_path::Vector{String}, dl_load_path::Vector{String}, load_path::Vector{String}, concrete_deps::Vector{Pair{Base.PkgId, UInt128}}, source::String)
@ Base ./loading.jl:2012
[14] top-level scope
@ stdin:2
in expression starting at /opt/julia-1.9.0-rc2/.julia/packages/HTTP/LsfSO/src/Connections.jl:1
in expression starting at /opt/julia-1.9.0-rc2/.julia/packages/HTTP/LsfSO/src/HTTP.jl:1
in expression starting at stdin:2

We are seeing this also, seems like https://github.com/JuliaServices/ConcurrentUtilities.jl/pull/32 broke HTTP.jl

@samtkaplan @quinnj

seems ok with julia 1.9.4
we observed that with julia 1.9.0
julia> Threads.@spawn Threads.threadpool() 123
ERROR: LoadError: ArgumentError: unsupported threadpool in @spawn: Threads.threadpool()
Stacktrace:
 [1] var"@spawn"(__source__::LineNumberNode, __module__::Module, args::Vararg{Any})
   @ Base.Threads ./threadingconstructs.jl:363
in expression starting at REPL[1]:1

but with julia 1.9.4
julia> Threads.@spawn Threads.threadpool() 123
Task (done) @0x0000000109fc9460

I tried on Julia 1.9.0, 1.9.1 and 1.9.2.  It seems to be broken on 1.9.0 and 1.9.1, but works starting on 1.9.2.  Given that, I made a quick work-a-round in #34.  I guess there might be a better solution, but not sure if it is worth the effort to figure out a better solution for two the two failing point releases?


(BOLT-359) Add minifact module to modules

Adds the minifact module which provides several plans (and supporting tasks) for retrieving facts from nodes.

CLA signed by all contributors.

This is ready for review, although it will need to be updated to make use of #357.

Is michael's comment about the module name not matching the requirements in the ticket getting addressed? I think we can land code here before the module but we probably don't want the name to change.

The question is: Is facter_task still a good name for the functionality?
Given that:

the module doesn't rely on facter
it provides a minimal set of one fact if facter is not available
if facter is available the additional facts it provides is a bonus rather than something to rely on
there is no support for retrieving individual facts

Continued in https://github.com/puppetlabs/bolt/pull/371.


typo in parameter name

Hello Ryohei
The parameter transform_epsilon should be trans_epsilon in the parameter file(localization.yaml) as there is no such parameter as transform_epsilon in the code.

Sorry for the delay, but I have corrected the typo and will close this issue.
commit
https://github.com/rsasaki0109/pcl_localization_ros2/commit/40dcaec1d63dc975dee62b4da348f75a2473947c


setState in LayoutBuilder callback gives incomprehensible exception

Test code:
Future main() async {
  runApp(new Foo());
}

class Foo extends StatefulWidget {
  Foo({Key key}) : super(key: key);

  @override
  FooState createState() => new FooState();
}

class FooState extends State<Foo> {
  @override
  Widget build(BuildContext context) => new LayoutBuilder(
        builder: (BuildContext context, BoxConstraints constraints) {
          setState(() {});
          return new Container();
        },
      );
}

Exception:
10:15:52.486: I/flutter : ══╡ EXCEPTION CAUGHT BY RENDERING LIBRARY ╞═════════════════════════════════════════════════════════
10:15:52.491: I/flutter : The following assertion was thrown during performLayout():
10:15:52.499: I/flutter : 'package:flutter/src/widgets/framework.dart': Failed assertion: line 1539 pos 16:
10:15:52.502: I/flutter : '!_dirtyElements[index]._active || _dirtyElements[index]._debugIsInScope(context)' is not true.
10:15:52.525: I/flutter : When the exception was thrown, this was the stack:
10:15:52.536: I/flutter : #2      BuildOwner.buildScope (package:flutter/src/widgets/framework.dart:1539:16)
10:15:52.538: I/flutter : #3      _LayoutBuilderElement._layout (package:flutter/src/widgets/layout_builder.dart:102:11)
10:15:52.540: I/flutter : #4      RenderObject.invokeLayoutCallback.<anonymous closure> (package:flutter/src/rendering/object.dart:1644:58)
10:15:52.543: I/flutter : #5      PipelineOwner._enableMutationsToDirtySubtrees (package:flutter/src/rendering/object.dart:896:15)
10:15:52.546: I/flutter : #6      RenderObject.invokeLayoutCallback (package:flutter/src/rendering/object.dart:1644:13)
10:15:52.548: I/flutter : #7      _RenderLayoutBuilder.performLayout (package:flutter/src/widgets/layout_builder.dart:201:5)
10:15:52.551: I/flutter : #8      RenderObject.layout (package:flutter/src/rendering/object.dart:1546:21)
10:15:52.553: I/flutter : #9      RenderView.performLayout (package:flutter/src/rendering/view.dart:113:13)
10:15:52.556: I/flutter : #10     RenderObject._layoutWithoutResize (package:flutter/src/rendering/object.dart:1424:21)
10:15:52.559: I/flutter : #11     PipelineOwner.flushLayout (package:flutter/src/rendering/object.dart:874:18)
10:15:52.561: I/flutter : #12     BindingBase&SchedulerBinding&GestureBinding&ServicesBinding&RendererBinding.beginFrame (package:flutter/src/rendering/binding.dart:202:19)
10:15:52.564: I/flutter : #13     BindingBase&SchedulerBinding&GestureBinding&ServicesBinding&RendererBinding&WidgetsBinding.beginFrame (package:flutter/src/widgets/binding.dart:297:23)
10:15:52.567: I/flutter : #14     BindingBase&SchedulerBinding&GestureBinding&ServicesBinding&RendererBinding._handlePersistentFrameCallback (package:flutter/src/rendering/binding.dart:147:5)
10:15:52.570: I/flutter : #15     BindingBase&SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:608:15)
10:15:52.573: I/flutter : #16     BindingBase&SchedulerBinding.handleBeginFrame (package:flutter/src/scheduler/binding.dart:543:9)
10:15:52.575: I/flutter : #17     runApp (package:flutter/src/widgets/binding.dart:363:7)
10:15:52.578: I/flutter : #18     main.<main_async_body> (<redacted>/lib/main.dart:14:3)
10:15:52.580: I/flutter : (elided 5 frames from class _AssertionError and package dart:async)
10:15:52.582: I/flutter : The following RenderObject was being processed when the exception was fired:
10:15:52.584: I/flutter :   _RenderLayoutBuilder NEEDS-LAYOUT
10:15:52.587: I/flutter : This RenderObject had the following creator information:
10:15:52.589: I/flutter :   LayoutBuilder ← Foo ← [root]
10:15:52.591: I/flutter : This RenderObject had the following child:
10:15:52.594: I/flutter :   RenderConstrainedBox NEEDS-LAYOUT
10:15:52.596: I/flutter : ════════════════════════════════════════════════════════════════════════════════════════════════════
10:15:52.598: I/flutter : Another exception was thrown: 'package:flutter/src/rendering/object.dart': Failed assertion: line 1867 pos 12: '!_needsLayout' is not true.

Ok I need to instead have that give a polite assertion explaining the problem.

Wow, that assertion shouldn't be hit. We must not be catching the bogus setState earlier.

https://github.com/flutter/flutter/pull/6024 makes this much better

It now says:
I/flutter : ══╡ EXCEPTION CAUGHT BY WIDGETS LIBRARY ╞═══════════════════════════════════════════════════════════
I/flutter : The following assertion was thrown building LayoutBuilder:
I/flutter : setState() or markNeedsBuild() called during build.
I/flutter : This Foo widget cannot be marked as needing to build because the framework is already in the process
I/flutter : of building widgets. A widget can be marked as needing to be built during the build phase only if
I/flutter : one of its ancestors is currently building. This exception is allowed because the framework builds
I/flutter : parent widgets before children, which means a dirty descendant will always be built. Otherwise, the
I/flutter : framework might not visit this widget during this build phase.
I/flutter : The widget on which setState() or markNeedsBuild() was called was:
I/flutter :   Foo(state: FooState(552725268))
I/flutter : The widget which was currently being built when the offending call was made was:
I/flutter :   LayoutBuilder(renderObject: _RenderLayoutBuilder NEEDS-LAYOUT)
I/flutter :
I/flutter : When the exception was thrown, this was the stack:
I/flutter : #0      BuildableElement.markNeedsBuild.<anonymous closure> (package:flutter/src/widgets/framework.dart:2508:11)
I/flutter : #2      BuildableElement.markNeedsBuild (package:flutter/src/widgets/framework.dart:2501:12)
I/flutter : #3      State.setState (package:flutter/src/widgets/framework.dart:852:14)
I/flutter : #4      FooState.build.<anonymous closure> (/data/user/0/com.yourcompany.scratch/cache/scratchdkstrk/lib/main.dart:19:11)
I/flutter : #5      _LayoutBuilderElement._layout.<anonymous closure> (package:flutter/src/widgets/layout_builder.dart:112:26)
I/flutter : #6      BuildOwner.buildScope (package:flutter/src/widgets/framework.dart:1649:19)
I/flutter : #7      _LayoutBuilderElement._layout (package:flutter/src/widgets/layout_builder.dart:108:11)
I/flutter : #8      RenderObject.invokeLayoutCallback.<anonymous closure> (package:flutter/src/rendering/object.dart:1668:58)
I/flutter : #9      PipelineOwner._enableMutationsToDirtySubtrees (package:flutter/src/rendering/object.dart:897:15)
I/flutter : #10     RenderObject.invokeLayoutCallback (package:flutter/src/rendering/object.dart:1668:13)
I/flutter : #11     _RenderLayoutBuilder.performLayout (package:flutter/src/widgets/layout_builder.dart:207:5)
I/flutter : #12     RenderObject.layout (package:flutter/src/rendering/object.dart:1570:21)
I/flutter : #13     RenderView.performLayout (package:flutter/src/rendering/view.dart:113:13)
I/flutter : #14     RenderObject._layoutWithoutResize (package:flutter/src/rendering/object.dart:1447:21)
I/flutter : #15     PipelineOwner.flushLayout (package:flutter/src/rendering/object.dart:875:18)
I/flutter : #16     BindingBase&SchedulerBinding&GestureBinding&ServicesBinding&RendererBinding.beginFrame (package:flutter/src/rendering/binding.dart:217:19)
I/flutter : #17     BindingBase&SchedulerBinding&GestureBinding&ServicesBinding&RendererBinding&WidgetsBinding.beginFrame (package:flutter/src/widgets/binding.dart:298:23)
I/flutter : #18     BindingBase&SchedulerBinding&GestureBinding&ServicesBinding&RendererBinding._handlePersistentFrameCallback (package:flutter/src/rendering/binding.dart:162:5)
I/flutter : #19     BindingBase&SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:648:15)
I/flutter : #20     BindingBase&SchedulerBinding.handleBeginFrame (package:flutter/src/scheduler/binding.dart:583:9)
I/flutter : #21     runApp (package:flutter/src/widgets/binding.dart:370:7)
I/flutter : #22     main.<main_async_body> (/data/user/0/com.yourcompany.scratch/cache/scratchdkstrk/lib/main.dart:5:3)
I/flutter : (elided 4 frames from class _AssertionError and package dart:async)
I/flutter : ════════════════════════════════════════════════════════════════════════════════════════════════════


Fixing Embedding Server input size error

This code fixes an original problem that we faced when generating embeddings - namely that we had text chunks that were tokenized and subsequently exceeded the embedding model's maximum context window.
This was solved by manually iterating over all of the chunks and checking to see if their encoded length is > 512. If it is, then we take a slice of the chunk and use that instead.
New problem however, is that we now cannot save the entire dataset in a single pass because it's a bit too big. The runner freuqently hangs at the final step and the .map function is noticably slower to complete.
We have ~ 41 million rows generated with this new script

lets not make the code more complex, it takes away from the educational aspect


chore:Update speaker repository tests

Fixes #1043
Checklist:

[x] I have checked for PMD and check-style issues
[x] I have read the Contribution & Best practices Guide and my PR follows them.
[x] My branch is up-to-date with the Upstream development branch.

Changes:
Speaker repository tests were updated.

@srv-twry Kindly merge.


Convert base test to pytest

Motivation
Using pytest makes it easier to debug and run tests locally for users.

Future Work
Provide better failure output. Right now it's pass fail, ideally we want to show the diff in the failure.

Current dependencies on/for this PR:

main

PR #27

PR #28   👈

This comment was auto-generated by Graphite.

bumping @jcape for re-review due to renaming, as well as action version bumps in the expected output


Remove unneeded malloc from kRing

The fast path of kRing does not need a distances array. We can remove the malloc/free from this case.
The current code looks like this:
void H3_EXPORT(kRing)(H3Index origin, int k, H3Index* out) {
    int maxIdx = H3_EXPORT(maxKringSize)(k);
    int* distances = malloc(maxIdx * sizeof(int));
    H3_EXPORT(kRingDistances)(origin, k, out, distances);
    free(distances);
}

void H3_EXPORT(kRingDistances)(H3Index origin, int k, H3Index* out,
                               int* distances) {
    const int maxIdx = H3_EXPORT(maxKringSize)(k);
    // Optimistically try the faster hexRange algorithm first
    const bool failed = H3_EXPORT(hexRangeDistances)(origin, k, out, distances);
    if (failed) {
        // Fast algo failed, fall back to slower, correct algo
        // and also wipe out array because contents untrustworthy
        memset(out, 0, maxIdx * sizeof(out[0]));
        memset(distances, 0, maxIdx * sizeof(distances[0]));
        _kRingInternal(origin, k, out, distances, maxIdx, 0);
    }
}

We could leave kRingDistances alone, but change kRing to:
void H3_EXPORT(kRing)(H3Index origin, int k, H3Index* out) {
    const bool failed = H3_EXPORT(hexRangeDistances)(origin, k, out, 0);
    if (failed) {
        // Fast algo failed, fall back to slower, correct algo
        // and also wipe out array because contents untrustworthy
        int maxIdx = H3_EXPORT(maxKringSize)(k);
        memset(out, 0, maxIdx * sizeof(out[0]));
        int* distances = malloc(maxIdx * sizeof(int));
        _kRingInternal(origin, k, out, distances, maxIdx, 0);
        free(distances);
    }
}

Closed by #322


detect HandBrakeCLI path via Spotlight

Hello Don, I've made a small patch to lookup the location of HandBrakeCLI by using Spotlight.
This makes it unnecessary to add it to PATH.

Viktor,
Thanks for the patch. Props to you for trying to solve the tool dependency problem that plagues every developer.
However, I'm declining your patch. I'll explain why.
First, Spotlight is not an appropriate mechanism to implement this.
Using Spotlight and mdfind this way is an OS X-only solution. My scripts are designed to work on Windows, Linux and other platforms. I would be rejecting your patch for that reason alone.
And just as important, Spotlight is just not capable of finding the correct binary on a user's file system. For example, if I execute this command:
mdfind 'kMDItemContentType == public.unix-executable && kMDItemFSName == HandBrakeCLI'

...Spotlight finds 36 different versions of HandBrakeCLI on my current machine. And your patch, using head -1 to trivially select the first item in that list picks the wrong one.
I suspect many other users who don't empty out their ~/Downloads folder would have similar problems.
Also, your patch is only trying to find HandBrakeCLI. My scripts have many other tool dependencies that are just as important.
I'm closing this now, but feel free to continue commenting here.

Oops, huge sorry for totally disregarding all other platforms (saying this from a completely multi-platform background – I can only blame it on the early morning work). Agreed on your decision and many thanks for detailing your reasons.

On OS X the HandBrakeCLI dependency can be solved trivially by using Hombrew Cask:
$ brew cask install handbrakecli


Change bookmarks and migration

Fixes #649.
Changes proposed in this PR:

Change bookmarks and migration to support multiple networks

So I guess this can be merged as it is and will fix the errors on the way

yup, think we can only fix along the way and as long as Vercel deployment works all good cause that's doing a npm run build too. Currently trying to figure out how to mock ddo.chainId or something in feature/multinetwork so that we can get green tests again

Cool, that would be a good temporary solution and would enable us to keep going with the other issues where ddo.chainId is required on the frontend


remove unused import.

This is very simple pull request.
I removed unused import.
Please check this pull request.
Thanks.

Looks good to me -- Thanks @fuji-151a
@erikdw should be able merge this

@fuji-151a : thank you!  Merging.


Is there way to customize error message on failure.

File a new question on StackOverflow
https://stackoverflow.com/questions/ask?tags=testcafe

Instead of displaying where exactly code failed, just wanted to display error message alone
AssertionError: WebUiActions, verifyPageUrl: expected urlSegment activeaccount did not match actual url https://xxxxxxxxx expected false to be truthy
Dont want to display below portions:
Browser: Chrome 100.0.4896.60 / Linux 0.0
Screenshot: /app/automation/reports/screenshots/2022-04-05_20-39-04/test-67/run-3/Chrome_100.0.4896.60_Linux_0.0/errors/1.png
32 |        };
33 |        return false;

Hi @aprabh001c ,
Yes, the TestCafe Reporter plugin is responsible for the output format. If you want to use your custom format, you can implement your personal reporter. Also, Testcafe has many built-in reporters, and NPM has a huge amount of third-party reporters. Try researching them - perhaps, you will find what you need.


Many Server Sent Events in parallel cause requests to hang

Question
Thanks for the great library, it's been a joy to use so far! I've previously been using EventSource for their SSE implementation but saw you recently released SSE support so I have attempted to migrate to using that.
My SwiftUI app displays a list and then fires off the SSE request for each of the list items. This uses the view .task modifier. The first list item gets a response immediately but unfortunately the next ones don't. The generated client calls end up hanging for a while, often a few seconds. I tried looking into SessionURLTransport to see if there was anything obvious but I'm quite new with Swift and it was a bit overwhelming.
Interestingly once those call start hanging, it also means every other endpoint using the generated client starts to hang too, not just the SSE calls.
Thanks!

Can you share a snippet of the code that calls the generated client? We'll need a bit more details here to be able to narrow down what's happening.
You could also write a ClientMiddleware for logging to see what requests are being sent, and what responses are coming back, to aid debugging.

Also, out of curiosity, can you try if you reproduce the same issue with https://github.com/swift-server/swift-openapi-async-http-client

It works as expected using the AsyncHTTPClientTransport, the requests happen instantly.
The code method I have looks a bit like this:
extension Fetchable_Value_Manager where Value == Components.Schemas.Board_Contents {
  static var board_contents: Fetchable_Value_Manager { .init() }

  @MainActor
  func stream(_ api: API, _ options: Operations.board_stream_contents.Input) async {
    withAnimation { state.to_fetching() }

    do {
      let response = try await api.client.board_stream_contents(options)

      switch response {
      case .unauthorized:
        throw API_Error.unauthorized()
      case .undocumented:
        throw API_Error.undocumented
      case let .ok(response):
        let board_contents_message_event_stream = try response.body.text_event_hyphen_stream
          .asDecodedServerSentEventsWithJSONData(of: Components.Schemas.Board_Contents.self)

        for try await board_contents_message_event in board_contents_message_event_stream {
          if let board_contents = board_contents_message_event.data {
            withAnimation { state.to_fetched(board_contents) }
          }
        }
      }
    } catch {
      withAnimation { state.to_failed(error) }
    }
  }
}

I gets run in code that works a bit similar to this:
List {
  ForEach(boards) { board in
    Board_View(board).task { await board_contents(api, .init(query: .init(board_id: board.id))) }
  }
}

I'll try get a full reproduction repo set up this weekend when I have a bit more time to do so.
Thanks!

I'll try get a full reproduction repo set up this weekend when I have a bit more time to do so.

That'd be great, should make debugging this much easier, thank you.

It works as expected using the AsyncHTTPClientTransport, the requests happen instantly.

It will be interesting to see if we can get a network trace for this too. If it works using AHC, it could be buffering in URLSession somewhere before delivering to the delegate within the URLSessionTransport.

@czechboy0 I've updated the iOS example project to replicate the issue here: https://github.com/ahtcx/swift-openapi-generator/commit/a88e2c6d730e137da71fabfbe33eec627e20d74f

@ahtcx Which server is the example hitting?

@ahtcx Which server is the example hitting?

@czechboy0 It's using the one in /Examples/event-streams-server-example

Hi @ahtcx, so @simonjbeaumont and myself investigated this more closely.

The AHC and URLSession transports behave the same way, we couldn't find a scenario in which they differed. So at least from what we've seen so far, there doesn't seem to be an issue in the URLSession transport.
The code you provided likely doesn't do what you want because the state and the views are intertwined in a way where the view triggers a task, the task fetches some data and sets a @State property, which in turn invalidates the state, which restarts/cancels the task. To address this one, I recommend looking more at SwiftUI documentation and WWDC talks around state management, for example you could move all the networking into a type that conforms to ObservableObject, which vends the current values to all the views, and uses @Published properties to notify its observees of it being changed. The important part is that the state/network stream should be owned by that object, not by the views (which you have today in the example).

Closing this issue for now - if you find more information that still points to an issue in the URLSession transport, please add more comments and we can reopen if needed.

Hey @czechboy0, thanks for having a look. It's very possible my code is faulty, I'm pretty new to Swift/SwiftUI haha so I'll dig around a bit more. Just in regards to your first point, are you saying you were unable to replicate the issue with the code I gave? Or that you were able to replicate the issue but that the issue doesn't stem from this project?

We weren't able to replicate the issue of anything hanging, the app remained responsive the whole time. However we noticed that the cells weren't updating with the events coming from the server, which was caused by the task cancelation I described above.

Odd, I guess maybe it's something specific to my setup or system somehow. Which iOS version were you using? I've had it happen on 17.0+.

are you saying you were unable to replicate the issue with the code I gave?

Concretely, we took your branch (thank you very much for taking the time to prepare it!) and played around with the iOS app with the local SSE server and saw the same behaviour with both transports. Specifically, we didn't see any hanging requests, and each row would be populated with the first of the three events from the stream and never updated—and this behaviour was the same for both transports.

@ahtcx I took some extra time to play around some more and think I understand the difference you were seeing. On first launch, with the URLSession transport, the list seemed to take longer to populate and seemed to do so in chunks. Furthermore, when making big sweeping scrolls, you can see the list update in batches of ~6 per second (more on that to come) and so would take 2-3 seconds to fully populate the list. I can also see that with AHC transport everything just updated as you scrolled.
This turned out to be due to URLSessionConfiguration. httpMaximumConnectionsPerHost ][^1].
The URLSessionTransport initializer accepts an optional URLSession  parameter, and uses URLSession.shared if not provided.
I found that the following code made the app behave in a similar way as when using the AHC transport:
var urlSessionConfiguration = URLSessionConfiguration.default
urlSessionConfiguration.httpMaximumConnectionsPerHost = 100
let urlSession = URLSession(configuration: urlSessionConfiguration)
let client = Client(transport: URLSessionTransport(configuration: .init(session: urlSession)))
// ...

Could you confirm that this is the problem you were seeing and that this resolves your symptom.
I will note that @czechboy0's observation on the structure of the app and SwiftUI code should probably be considered, but hopefully this clears up the observed differences between the two transport implementations?
[^1]:
This property determines the maximum number of simultaneous connections made to each host by tasks within sessions based on this configuration.
This limit is per session, so if you use multiple sessions, your app as a whole may exceed this limit. Additionally, depending on your connection to the Internet, a session may use a lower limit than the one you specify.

The default value is 6.

— https://developer.apple.com/documentation/foundation/urlsessionconfiguration/1407597-httpmaximumconnectionsperhost

Sorry about that, indeed the example I gave was buggy. I had tried to strip it down as much as possible for the replication but it ended up no longer reflecting how my actual app code which is properly using ObservableObjects and didn’t have any of task cancelling issues.
@simonjbeaumont Wow that was the issue, thank you so much - I really appreciate all of your guy's help and work into this amazing package! 🚀


Make sure that (on a clean Emacs install) it doesn't trigger LSP

Thanks for the fix! Even with the fix, I still get the LSP error once after I restart emacs, but that's easy to deal with. It's great that it no longer happens every time I open a new Verus file.

Glad the general case is fixed! For the specific once-after-restart case, I haven't really been able to reproduce it.
However, one somewhat hacky way to fix that (which is only ok if you don't plan to use your Emacs instance for any real Rust development, and only for Verus things) is to just globally disable rustic from looking for lsp things. To do that, you could try to update your .emacs:
  (use-package rustic
-   :ensure t)
+   :ensure t
+   :init (setq rustic-lsp-setup-p nil))


Objective-Git should use value objects

The Idea
Objective-Git should be designed to work inside Cocoa apps, which are usually concurrent and long-lived. Since the data on disk can always change underneath you, the API needs to balance these properties:

Staleness: Data in the app may not reflect the data on disk. Has an object been deleted, renamed, or otherwise altered?

Consistency: It may be hard to get a consistent view of all the data. Can you retrieve multiple properties from an object and have them be in sync?

Contention: It may be hard to group access to the database, so reads could happen everywhere and everywhere.

Simplicity: Is it obvious what a given API will do? e.g., Is it obvious which properties result in disk access? Does -[GTBranch SHA] get the latest SHA from the disk?

The current design of Objective-Git prioritizes (1) and makes (2), (3), and (4) more difficult. Many Objective-Git objects are just thin wrappers around their libgit2 counterparts, and the properties are just wrappers around libgit2 functions. It's hard to tell which properties result in lookups, complicating contention and consistency.
If, instead, we used immutable value objects, then (2), (3), and (4) are substantially easier, and (1) is not substantially more difficult. Concurrency would be much simpler because these objects could be passed around and used freely.
The Changes
These are the classes I have in mind as values (but I haven't looked closely at every Objective-Git class, so this may not be exhaustive):

GTBlob
GTBranch
GTCommit
GTRemote
GTSignature
GTTree
GTTag

Practically, here are the changes that would be required:

Make all the properties on these objects either be (a) synthesized or (b) computed from another immutable, synthesized property. (Some properties would need to be removed and made into methods.)

Remove references to the GTRepository from all these objects. Move any APIs that depend on the GTRepository to a method on GTRepository.

Move methods from these objects to GTRepository. (Usually a method means reading from or writing to the repository.)

@jspahrsummers said there's some problems with this general idea, but I'm not sure what they are.

@mdiep Any reason to keep this issue open? Does SwiftGit2 address this?

I think SwiftGit2 addresses this. 👍


Typst support

Typst is a tool for creating documents similar to LaTeX. The target extension is .typ.
Note: As of v1.85.1, VS Code does not support typst by default.

Thanks. I have been interested in Typst since it was announced. However, I never used it. It would be great if someone is willing to help.


Use VIRTUAL_ENV variable for python binary

Use the VIRTUAL_ENV environment variable inside the Dockerfile to define the PYTHON variable
if VIRTUAL_ENV changes it will be automatically reflected in the PYTHON binary

Thanks! Per our contribution licensing documentation, would you mind adding a Signed-Off-By: tag to your commit message to indicate that you have read and agree to our Developer's Certificate of Origin?
That should be as easy as doing:
$ git commit --amend -C HEAD -s
$ git push -f

from a checkout of this branch.

Thanks a lot @elfosardo for your contribution 🎉


highcharts map

Hi,
I wanted to know if it's possible to do highmaps with your angular plugin, and if it's possible, how?

:+1:

+1

This is now added http://rawgit.com/pablojim/highcharts-ng/master/example/maps/example.html

@pablojim example.html does not work. Can you please update? Also can you please add example code for world map instead of specific country?

@fizerkhan I guess it's because the its using angular 1.2.28 and the code's written in angular.component which is supported after Angular's 1.5.x ?
@pablojim is that right or am I missing something?


Suggestion: Wealth Tab

In my opinion, the Wealth Tab should be renamed to Inventory - I think that is more accurate and easier to use because I keep forgetting where my inventory is.
Or rename it to Inventory\Wealth.
Roll20: MikeW

Mike
If the fix I made for your fright check modifications works, then I feel I will be confident enough to do this one as well.

As this is a cosmetic change only, I have made the changes with a Pull request submitted 7/17/2017.


0.24 conflicts with "right square|brace|angle" by default

If in talon you use "square" (or brace, or angle) and in cursorless's paired_delimiters.csv you have square mapped to squareBrackets, "take right square" will instead select everything between and including the square brackets:
Spoken form, Cursorless identifier
brace, curlyBrackets
angle, angleBrackets
square, squareBrackets

A workaround for now is to make sure you use a different term in the paired delimiter mapping:
curlies, curlyBrackets
angles, angleBrackets
squares, squareBrackets

(plurals work for me, your accent may vary (yamy))

Thinking a bit more on this one, I don't think you're going to be able to use the same word for a surrounding pair delimiter that you use for the delimiter itself, even if I fix the "right" issue.  If you have "square" mapped to squareBrackets in paired_delimiters.csv, your grammar becomes ambiguous, because "take square" could mean either:

select the [ with a gray hat over it, or
select the matching pair of [ and ] containing your cursor

The default surrounding pair terms are designed not to conflict with the default terminology in knausj; I'm not sure we can do much better than that 🤔

Ahh, it's because I changed them to be the same as knausj.
I think if the right issue is fixed that would be helpful (since it's kind
of weird anyway?) but yeah, sounds mostly like it's a me problem.
On Wed, 22 Dec 2021 at 17:05, Pokey Rule @.***> wrote:

Thinking a bit more on this one, I don't think you're going to be able to
use the same word for a surrounding pair delimiter that you use for the
delimiter itself, even if I fix the "right" issue. If you have "square"
mapped to squareBrackets in paired_delimiters.csv, your grammar becomes
ambiguous, because "take square" could mean either:

select the [ with a gray hat over it, or
select the matching pair of [ and ] containing your cursor

The default surrounding pair terms are designed not to conflict with the
default terminology in knausj; I'm not sure we can do much better than that
🤔
—
Reply to this email directly, view it on GitHub
https://github.com/pokey/cursorless-vscode/issues/410#issuecomment-999731391,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEORK6ZZZVWYLMTCYQC4ULUSIAMZANCNFSM5KQSVH2Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Stefan du Fresne

I can fix the right issue, but I'm just not sure how helpful it will be, because it will only occur if you use same delimiter name for the pair as for the individual, which cannot work for the reasons above, even if I fixed "right". So I'm not sure I see the benefit of fixing "right" if it's impossible to fix "left". Basically users just can't use the same name for their delimiters as they use for the pair. Given that constraint, there really isn't a "right" issue anymore imo.
Happy to do a quick discord to hash this one out once we're both back at it again in the new year

Can talon detect conflicts? I guess it's tricky since "conflict" is a vague
term and some conflict is desirable. Perhaps like you say as long as
cursorless defaults don't mess with knausj defaults we're good and we can
close.
On Thu, 23 Dec 2021 at 08:09, Pokey Rule @.***> wrote:

I can fix the right issue, but I'm just not sure how helpful it will be,
because it will only occur if you use same delimiter name for the pair as
for the individual, which cannot work for the reasons above, even if I
fixed "right". So I'm not sure I see the benefit of fixing "right" if it's
impossible to fix "left". Basically users just can't use the same name for
their delimiters as they use for the pair. Given that constraint, there
really isn't a "right" issue anymore imo.
Happy to do a quick discord to hash this one out once we're both back at
it again in the new year
—
Reply to this email directly, view it on GitHub
https://github.com/pokey/cursorless-vscode/issues/410#issuecomment-1000119550,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEORK53JWRW3ZADKYSHTNDUSLKLNANCNFSM5KQSVH2Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Stefan du Fresne

I'm tempted to leave this one open for a couple of weeks because users who use "square" / "right square" will have some problems because until 0.24.0 the default term for the pair was also "square" (it's now "box").  Before 0.24.0 this wasn't as much of an issue because the pair could only be used for wrapping, but because it is now a scope type, there will be conflicts for people who still have the term mapped as "square" (which is everyone who upgraded from 0.23.0 and never changed "square" to something else, because cursorless won't change terms for people who already have it installed)

The docs have not been updated to reflect that correct term("box"). Are docs update incrementally, or are you guys waiting for a particular version release?

Nice catch! Very much open to a PR here, or feel free to file an issue. One-line fix

opened https://github.com/cursorless-dev/cursorless-vscode/pull/548

@pokey now that "box" is the default, should "escaped square" be changed to default to "escaped box"? The defaults should be consistent.

@pokey now that "box" is the default, should "escaped square" be changed to default to "escaped box"? The defaults should be consistent.

Haha yes definitely nice catch 😅

We're on a mono-repo, so you wouldn't actually commit to the cursorless-talon repo, but rather to the cursorless-talon subdir here.
Re tests, we don't yet have tests for talon side, so as this change is purely talon side, no need to write tests


openmpi version consistency

resolution_fixed   |    by rb4844

I have changed my suites to use openmpi/1.10.2 so as to arrive at a consistent set of modules for oasis, gcom, dr hook and the UM, MOM and CICE plus IPM.
Works fine, is reproducible in that generates same first months surface temperatures as al623.
So, should we all move to 1.10.2 for consistency?

Issue migrated from trac:320 at 2024-01-31 18:29:29 +1100

rb4844 changed status from new to closed

rb4844 set resolution to fixed

rb4844 changed component from ACCESS model to ACCESS-CM2

rb4844 commented

No further action required.
openmpi/1.10.2 recommended for ACCESS-CM2


Is this compatible with server rendering (Angular Universal)?

Title says it all...

Hi,
Did you try it ? I'm asking the same question.

Hi, didn't try it.
Found out that I can use Croppie.js (jQuery) within the ngAfterViewInit hook when server side rendering.
Good luck.

Hi, this project is deprecated. Please I inherited it and keep it up here
https://github.com/web-dave/ngx-img-cropper/


Support non-vrm avatars

Any object should be posessable, as in the multiplayer demo.

This is done.


Updated mod compiler

Minor refactoring to the ModCompiler class.
Converted foreach logic to use LINQ queries instead.
Pretty sure CACHE_LOCATION should be readonly.

Question by the way? Do you know how to do better roslyn error output?

No, not really. I could spend some time looking into to it.

merge?


Expose Heartbeat and enable adding and removing handlers

This would allow hooking MessageBody directly to Heartbeat after #1840, to implement the request body timeout. Allows for a cleaner timeout implementation than passing ITimeoutControl to MessageBody and doing tricks like SetTimeout(0) to force a timeout when using a variable timeout based on the request rate.

I would close this PR without merging.
FrameHeartbeatManager isn't doing anything anymore now that the WalkCallback is empty yet the it's still there. This is netting over 100 additional lines of code to do the exact same thing as before while making the Heartbeat class more complicated and harder to use safely.
As far as things like avoiding calling SetTimeout(0) in MessageBody, you could just call Timeout() directly instead.
Leave Heartbeat the way it is. If you need to walk a dynamic collection of objects every beat, make another class like FrameConnectionManager which allows you to simply walk the active objects on every beat like FrameHeartbeat manager. FrameConnectionManager and FrameHeartbeatManager are well less than 100 lines combined and allow for a much simpler Heartbeat class that just manages the timer and does some tracing.

Thanks for the feedback. Opened https://github.com/aspnet/KestrelHttpServer/pull/1847.


Can not "Flash" macOS

Imaging or Flashing macOS systems is not possible as of macOS 10.14 so the following text is an incorrect statement and can not be done.
"The first option is to flash the device with an image of macOS where Microsoft Edge has already been set as the default browser."

Clean up of old issues.
If this is still an issue, please open a new issue and include relevant information and a link to document in the Comment section.


Fix a few small issues in README

Fix issues I spotted recently.

This PR was meant to include the last commit only. Because I locally based it on master but made the PR against development, additional commits were brought in... It's okay though.


promote AWS-NLB Support from alpha to beta

Promote AWS-NLB support from alpha to beta
Note:
This is just for updating the AWS-NLB feature status.
There should be future PR to enhance the doc structure. e.g. SSL support is available on both NLB & CLB, we should reorganize it to make it cleaner.

/milestone v1.15

/milestone 1.15

/lgtm

There should be future PR to enhance the doc structure. e.g. SSL support is available on both NLB & CLB, we should reorganize it to make it cleaner.

Hi, @M00nF1sh sorry! when you say future PR, are those enhancements intended to be merged in this release cycle?

@makoscafee
Sorry for the late reply. only this one is intended to be merged in this release cycle.

Hi @M00nF1sh , @DanyC97 can this be approved at the state it is right now? We are looking to have all PR for release 1.15 merged by June 10th. Let me know if there is anything we can assist.

/lgtm
/approve


GH-16: Update Cake.AWS.S3 to netstandard2.0, update Cake refs to 0.26

Summary of changes to address GH-16:

Update csproj to target netstandard2.0
Update nuspec to pack netstandard2.0
Update build.ps1 to pull in the current SDK refs to support netstandard2.0
Update Cake refs to 0.26.1
Also had to update testing concerns to get tests discovered/running.

This looked like the the extent of the changes that were necessary.  If there are things I've missed or need adjusted, let me know.

@SharpeRAD any news for this PR?


feat: Multi sitemaps with i18n

🆒 Your use case
When using both multi sitemaps and i18n, it will be useful to still have a separate sitemap for each language.
nitro.config.js:
sitemap: {
    sitemaps: {
      shirts: {
        sources: ['/api/sitemap/shirts']
      }
    }
  }

/api/sitemap/shirts:
export default defineSitemapEventHandler(async () => {
  return [{
    loc: `/shirts/white`,
    _i18nTransform: true
  }];
});

This will generate /__sitemap__/shirts.xml which contains pages in all languages.
🆕 The solution you'd like
/__sitemap__/shirts/en.xml
/__sitemap__/shirts/es.xml
...

🔍 Alternatives you've considered
No response
ℹ Additional info
Thank you :pr

Google does not support nesting multiple index sitemaps within each other so this is not something I will implement, sorry.

Oh right, sorry about that.
What about /__sitemap__/shirts-en.xml?

It's possible but it's unlikely I'll attempt this any time soon as there's quite a bit of complixity involved in supporting this.


Is there a way to change the color of a single mesh/vertice?

Just a simple call - or do we have to create an entire cmap and change one value?
Thanks.

If you already have a color array (per vertex or per face) you need to reassign it to the mesh object.

If you already have a color array (per vertex or per face) you need to reassign it to the mesh object.

What variable should it be assigned to (example: vertices -> pts, faces -> faces)

Check out #555
Instead of celldata you can use pointdata.
If you just need to highlight a specific point I would simply add a Point() object to the scene, and you choose its color.

Is celldata initialized when you create a mesh object? I'm looking at the documentation - pointdata and celldata exist but when I call, say, m = vedo.Mesh([vertices,faces]) and then call m.pointdata or m.celldata I get the message: 'Mesh' object has no attribute 'pointdata'

You may be using an old version?
This works for me:

from vedo import Mesh, show
import numpy as np

verts = [(50,50,50), (70,40,50), (50,40,80), (80,70,50)]
faces = [(0,1,2), (2,1,3), (1,0,3)]

# Build the polygonal Mesh object:
mesh = Mesh([verts, faces])

mesh.celldata['mycolors']= np.array([[255,0,0],[0,255,0],[0,0,255]]).astype(np.uint8)
mesh.celldata.select('mycolors')

show(mesh, viewup='z', axes=1).close()

`from vedo import Mesh, show
import numpy as np
verts = [(50,50,50), (70,40,50), (50,40,80), (80,70,50)]
faces = [(0,1,2), (2,1,3), (1,0,3)]
Build the polygonal Mesh object:
mesh = Mesh([verts, faces])
mesh.celldata['mycolors']= np.array([[255,0,0],[0,255,0],[0,0,255]]).astype(np.uint8)
mesh.celldata.select('mycolors')
show(mesh, viewup='z', axes=1).close()
Traceback (most recent call last):
File "", line 10, in
mesh.celldata['mycolors']= np.array([[255,0,0],[0,255,0],[0,0,255]]).astype(np.uint8)
AttributeError: 'Mesh' object has no attribute 'celldata'`

This is the error that I am getting.

Try:
pip install vedo -U

Works now, thanks!


fix issues, close outdated prs, remove unused dependency glob

fix issues: fix #202, fix #207, fix #208, fix #218, fix #221, fix #231
close prs: close #209, close #216, close #244
See also: https://github.com/vuejs/vuex-router-sync/issues/71

@yyx990803 Sorry to interrupt you, but I hope you would help to review this PR.

Thanks - just wondering why pin Firebase to 4.1.3?

@yyx990803 https://github.com/firebase/firebase-js-sdk/issues/157#issuecomment-331158004


Allow shutdown at a given block height

Allows synchronized upgrades of fedimintd.

call: we only need this when we actually want to backwards-compatibly extend consensus rules.

duplicate of https://github.com/fedimint/fedimint/issues/948?

And #4753 :laughing:


Queueing mails makes them lose their order

Laravel Version: 5.3
PHP Version: 7
Database Driver & Version:

from here: http://stackoverflow.com/questions/41302476/laravel-changing-sorting-of-collection-when-sending-mail-between-constructor-an/
Description:
this is laravel 5.3
when I preview the email using this:
    $wantsheet_products = WantsheetProduct::orderByRaw(EmailService::WANTSHEET_PRODUCT_ORDER_SQL)->get();
    View::make('email.wantsheet.email_wantsheet_to_supplier', ['wantsheet_products' => $wantsheet_products]);

the sorting is correct. that is, sorting is ['a','b','c'] the way i want it.
EDIT see note at the bottom
now when actually sending out the mails (i queue them), the sorting changed and is unsorted again, magic?! the change happens between the constructor and the build function
class WantsheetToSuppliersMail extends Mailable
{

  public $wantsheet_products;
  public $to_email;
  /** @var  WantsheetContact  $wantsheetcontact*/
  public $wantsheetcontact;
  use Queueable, SerializesModels;

    /**
     * Create a new message instance.
     *
     * @return void
     */
    public function __construct($wantsheet_products)
    {
//$wantsheet_products is a standard eloquent model collection, e.g. i get it like this:  WantsheetProduct::orderByRaw(self::WANTSHEET_PRODUCT_ORDER_SQL)->get()
      $this->wantsheet_products = $wantsheet_products; //is ['a','b','c']
    }

    /**
     * Build the message.
     *
     * @return $this
     */
    public function build()
    {

//      $this->wantsheet_products is ['b','a','c'];
      $subject = 'abc';
        return $this->from('me@myapp.com')->view('email.wantsheet.email_wantsheet_to_supplier', [])->subject($subject);
    }
}

EDIT contd.
Now when i do
WantsheetProduct::orderByRaw(EmailService::WANTSHEET_PRODUCT_ORDER_SQL)->get()->toArray();
it doesn't break the sorting any longer (so it works).
Steps To Reproduce:

Fixed in 5.4.


Add positive? and negative? methods to Numeric.

Motivation
These methods were missing and I wanted them to be typed :)
Test plan
See included automated tests.
positive?: https://ruby-doc.org/core-2.5.0/Numeric.html#method-i-positive-3F
negative?: https://ruby-doc.org/core-2.5.0/Numeric.html#method-i-negative-3F

Oops, forgot the closing parentheses.


Support temporal TC39 proposal

FakeTimers version : 6.0.1
Environment   : N/A
Example URL   : N/A
Other libraries you are using: None

Repo: https://github.com/tc39/proposal-temporal/
We should support faking out usage of all Temporal.now.* methods. Not sure if we need to fake anything else?
Note that I do not think we should support Temporal now - the proposal is still at stage 2. So consider this more of a tracking issue than something that's actionable right now.
What did you expect to happen?
N/A
What actually happens
N/A
How to reproduce
N/A

Cc @littledan I remember you asking for feedback on Temporal.
Is there anything other than .now that sinon would need to fake?

It's now stage 3: https://github.com/tc39/proposal-temporal/pull/1440. 🎉 Is this a good time to add support? Along with some docs caveat that it's still unstable (and of course not added to toFake by default). Or should we wait for one engine to land it?

When does it land in Chrome?

I have exactly zero clue 😅

It is stage 3, but will not be shipped unflagged at this time, so it'll be awhile.
You will only have to mock out Temporal.now, by design.
I would start implementing now, but perhaps hold off shipping it until at least one engine has landed it, or at least until a production-ready spec-compliant polyfill exists.

You will only have to mock out Temporal.now, by design.

Wonderful 👍

I would start implementing now, but perhaps hold off shipping it until at least one engine has landed it, or at least until a production-ready spec-compliant polyfill exists.

If somebody is up for implementing it, I think we can also ship it as long as it's clearly marked as unstable in docs and possibly code, e.g. unstable_enableTemporalSupport: boolean or something. Not sure, and I don't feel strongly


refactor: remove dbCounter

Description

Remove dbCounter for now. The plan is to add it back in the context and increment the counters at the storage layer.
References

Review Checklist

[ ] I have clicked on "allow edits by maintainers".
[ ] I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
[ ] The correct base branch is being used, if not main
[ ] I have added tests to validate that the change in functionality is working as expected

Codecov Report
Base: 82.73% // Head: 82.54% // Decreases project coverage by -0.19% :warning:

Coverage data is based on head (9cc44c1) compared to base (b587057).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #325      +/-   ##
==========================================
- Coverage   82.73%   82.54%   -0.20%
==========================================
  Files          83       83
  Lines       12105    12022      -83
==========================================
- Hits        10015     9923      -92
- Misses       1689     1696       +7
- Partials      401      403       +2

Impacted Files
Coverage Δ

server/commands/check_utils.go
93.19% <ø> (-1.35%)
:arrow_down:

server/commands/create_store.go
84.21% <ø> (-1.51%)
:arrow_down:

server/commands/delete_store.go
75.00% <ø> (-2.78%)
:arrow_down:

server/commands/list_stores.go
66.66% <ø> (-1.20%)
:arrow_down:

server/commands/read_assertions.go
68.75% <ø> (-3.48%)
:arrow_down:

server/commands/read_authzmodel.go
92.30% <ø> (-0.55%)
:arrow_down:

server/commands/read_authzmodels.go
76.92% <ø> (-0.86%)
:arrow_down:

server/commands/read_changes.go
90.32% <ø> (-0.31%)
:arrow_down:

server/commands/read_tuples.go
77.77% <ø> (-0.80%)
:arrow_down:

server/commands/write_authzmodel.go
90.00% <ø> (-0.33%)
:arrow_down:

... and 12 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Set global layout

Is it possible to set layouts for all routes I'm using blaze and have the following. It feels wrong to have to set the layout for each route?
FlowRouter.route('/dashboard', {
action: function() {
BlazeLayout.render("layoutApp", {header: "appHeader", main: "appMain" });
}
});

Nope. We don't have features for that. You can simply create simple wrapper to do what you wanted.


Client Certificate CA

Is there a way to specify a CA certificate when initializing client to go along with Certificate and Private Key in the cert tuple? The Absence of a way to specify CA file is a deal breaker for this library unfortunately.

I think I'm having an issue related to this:
2017-01-30 16:59:37.543964 I | http: TLS handshake error from <host>: remote error: tls: unknown certificate authority

i am hitting the same problem.... using the verify=False option to workaround this.

Fun note: the verify kwarg in the constructor and/or other methods, is passed through directly to an internal requests.Session, whose verify argument can also take a string value which is the path to one's CA cert. Seems to work fine for me!

It should at least be added to the documentation

I struggled with this a bit recently... Can add something to the README.

Passing a string that contains the path to a CA bundle, as described in the request docs works. You'll get a warning in some IDEs though, because the type hint in hvac is bool and not Union[str, bool] or just no type hint. I've addressed that in my PR.

Between #230 and #238 I am going to consider this issue closed. Please open up a new issue if you feel like additional work is needed.


⚠ Bofrost.be has degraded performance

In 47d3573, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2231 ms

Resolved: Bofrost.be performance has improved in 4a6b22e.


Cannot define a custom error using thiserror crate

I'm trying to define a custom error through thiserror crate using this code
use thiserror::Error;

///Error types
#[derive(Debug, Error)]
pub enum Error {
    /// License not parsed correctly.
    #[error("License not parsed correctly")]
    InvalidLicense(#[from] license::ParseError),
}

/// A specialized `Result` type.
pub type Result<T> = ::std::result::Result<T, Error>;

but it seems that ParseError cannot be retrieved using #[from] attribute

Thanks, will fix this when error is available in core.
https://github.com/rust-lang/rust/issues/103765

Out of curiosity, why do we need to wait until that issue will closed? Can we use for now an intermediate solution? I'm taking minijinja as example because I have used that crate recently :smile:
Here how an error is handled
An I can add #[from] minijinja::Error to my custom error.

I have never used thiserror so I don't know how it works, but I'm assuming #[from] requires the Error trait to be implemented for this to work? Or does it require another trait to be implemented?
This library is no-std so I currently can't implement Error without adding additional feature gating for std and alloc.

Ahh, if this library is considering even a no-std environment we cannot implement thiserror because it's a derive macro for std::error::Error trait. There is a workaround but they advise against using that.
In your opinino, what would a possible hack be to fix my problem? For now, I have introduced custom errors through map_err, but it does not seem a good solution eh eh
As an alternative, I might also try snafu, an error handling library which also considers no-std environment

I'm only guessing but if it is the from attribute that fails, maybe you can implement the From trait manually?
impl From<ParseError> for Error {
   fn from(e: ParseError) -> Error {
       Error::InvalidLicense(e)
   }
}

Or is it the error attribute that fails?

Yep, in that case works, just wanted to use the #[from] to simplify the code a bit, but it's okay for me this solution. Thanks a lot!

The error I get with the #[from] attribute is:
src/error.rs:20:22
    |
20  |     InvalidLicense(#[from] license::ParseError),
    |                      ^^^^ method cannot be called on `&ParseError` due to unsatisfied trait bounds
    |
   ::: /home/.cargo/registry/src/index.crates.io-6f17d22bba15001f/license-3.2.0/src/lib.rs:159:1
    |
159 | pub struct ParseError(());
    | ---------------------
    | |
    | doesn't satisfy `ParseError: AsDynError<'_>`
    | doesn't satisfy `ParseError: StdError`
    |
    = note: the following trait bounds were not satisfied:
            `ParseError: StdError`
            which is required by `ParseError: AsDynError<'_>`
            `&ParseError: StdError`
            which is required by `&ParseError: AsDynError<'_>`

Thanks, will fix this when error is available in core.
rust-lang/rust#103765

Hey! This has been merged the last week. We can now provide a minijinja solution for this crate

Great! I will impl Error when it hits stable.


landsat_sr_c2: can't use ee.Authenticate() in script

Need to figure out why this isn't functioning. Right now you have to open a terminal window to complete authentication step

This may be a lingering side effect of QMD. May wish to try in RMD and see if it's still an issue.


Better error when it fails on no package.json

Right now it throws a generic fs error when no package.json's present

Thanks, good point.


Can I animate a single piece using board positions?

I want to use the Knight piece to show how it can travel over the board using a set of input co ordininates. Can I start the board with a single piece and then leverage the animate functionality to show how the Knight moves on the board.
I tried using the board.setPosition method but it a bit complex and was wondering if there is a simple way to achieve my use case.
Thank you!

Yes, setPosition needs FEN, which you can generate with something like chess.js. Do you mean a method like movePiece("e2","e4")? Or what would be the best way to move the Knight?

Initially, I want to start with an empty board with an option to set the starting position for the Knight. I would then render the Knight at the chosen position using board.setPosition() method. Once the user selects the end position of the Knight, I will generate the path for reaching the destination.
I want to be able to animate the movement of the Knight for each position in the path. I tried to achieve this using the setPiece and setPosition method, but I was not able to use the animations with this.
const board = new Chessboard(document.getElementById("board1"), {
        position: "empty",
        sprite: {url: "../assets/images/chessboard-sprite-staunty.svg"},
        animationDuration: 3000,
        style: {
            borderType: BORDER_TYPE.frame
        }
    })
    for (var i = 1; i < 9; i++) {
        await board.setPiece('h' + i, 'bn')
        await board.setPosition('empty')
    }

Another method I tried after seeing the pieces-animation.html is use the setPosition method. It was somewhat closer to my use case but it was too complex to achieve.
        await board.setPosition('///N////')
        await board.setPosition('////N///')
        await board.setPosition('/////N//')
        await board.setPosition('//////N/')

If you can guide me on what is the best way to move the Knight on the chessboard with animations, that would be really helpful.
Thanks!

You could use setPiece(square, piece) to set the Knight to its initial position. And then movePiece(squareFrom, squareTo, animated = true) to move the knight.
board.setPiece("e4", "wn")
board.movePiece("e4", "c3").then(() => {
    board.movePiece("c3", "d5").then(() => {
        board.movePiece("d5", "f4")
    })
})

You mean create the method movePiece in the Chessboard class? Because the method is not available in the class. If I were to create the class, which methods should I use to execute the movement and the animation?

I created it today with version 3.16. See the Example "Knight dance" here https://shaack.com/projekte/cm-chessboard/examples/pieces-animation.html

Just saw the update. Thank you so much. Works like a charm. I have another request, I want to draw a line for each square the Knight has visited. Like the path covered. If you could point me in the direction, I will try to code it myself.. :)

This is more complicated, i've to think about it. For now, I can tell that drawing is made in the ChessboardView.js. You can draw in the SVG with the method Svg.addElement(parent, name, attributes).


Accessing properties in field definitions

Given I have some model like this:
angular.module('myApp', ['restmod'])
  .factory('MyThing', function (restmod) {
    return restmod.model('some-endpoint').mix({
      some_field: {
        some_property: 123
      }
    });
  });

Is there any way through a mixin to access the property some_field.some_property? In essence, I would like to be able to add custom properties to fields and then get a list of those custom property values via something like MyThing.$fieldProperties().
I'm not able to locate any such functionality in the documentation or through the code.

Ah, I figured it out! Here's my mixin that solves this issue:
angular.module('definitionAccessor', [])
  .factory('DefinitionAccessor', function(restmod) {

    return restmod.mixin(function () {
      var fields = [];

      this
        .extend('attrAsDisplay', function (fieldName, displayProperties, fieldDefinition) {
          fields.push(fieldName);
        }, ['display'])
        .define('Scope.$fieldProperties', function () {
          return fields;
        });
    });
  });

Given that your field has the property display defined on it, it will be thrown into the fields array and be accessible through a function like MyThing.$fieldsProperties()


Add alternative instruction and link to verify OAW.zip hash

@yil578 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Nest 使用 import 找不到模块

Best require does not support es6 import, which is different from require. I wonder if there is a way to hook import and it has not been found yet.


bug: StorageEngine regression (SQLStorage broken)

Type: bug
Ionic Version: 2.x
Platform: all
Breaking change between alpha 48 and alpha 51:
In commit 07953672f53c793828f29d81101b46d6b88dfafa, constructor addition in StorageEngine class breaks the functionality for SQLStorage and LocalStorage.
Now when instantiating a new Storage instance, one get the constructor(options={}) not implemented for this storage engine Error

EXCEPTION: Error: constructor(options={}) not implemented for this storage engine

Should be fixed in the next release by this: https://github.com/driftyco/ionic/pull/5141

Yes thanks, I've updated my sources already. By the way have you noticed that in alpha 51, if you use the translate pipe in the hello iconic, then at launch the root page's translations are empty ? Behaviour change with alpha 48

@phavre i have also get same issue please help me for solving it
My ionic info:
Cordova CLI: 5.4.1
Gulp version: CLI version 3.9.0
Gulp local:
Ionic Version: 2.0.0-alpha.51
Ionic CLI Version: 2.0.0-beta.17
Ionic App Lib Version: 2.0.0-beta.8
OS:
Node Version: v5.3.0
Please give example of it.
Thank You !!

get the alpha 52 and it should work.

@sirfullTom alpha 52 doesn't help me(

Well, @tairezzzz, @inceptiveDipak, you just have to comment the Storage Class constructor throw error line (in the platform/storage/storage.ts file) if I remember correctly.

At least it is what I have done, but in checking the source I see it's already fixed.
I'm going to upgrade to alpha 52 tonight and tell you if I still encounter the issu.

I think I'm not updated correctly, error is gone,  anyway thx for your help @phavre!

Indeed, I've updated to alpha 52 (53 is on his way with breaking changes) and it works.


Perform transpilation in parallel

Summary
fixes #1146
Perform transpilation over multiple circuits in parallel.
Details and comments

Ok, this PR has removed all the events we were emitting for the progress bar to work with compilation. So progress bars are now listening to non existing events.


[Bug]:  Missing spacing in WishList Page

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
In WishList Page there is missing of proper spacing, so in case of error both navbar and footer come together
Expected behavior
There should be proper spacing in wishlist section
Add ScreenShots

On which device are you experiencing this bug?
Chrome
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

Is there an existing issue for this?

[x]  I have searched the existing issues

Describe the bug
In WishList Page there is missing of proper spacing, so in case of error both navbar and footer come together
Expected behavior
There should be proper spacing in wishlist section
Add ScreenShots

On which device are you experiencing this bug?
Chrome
Record

[x]  I have read the Contributing Guidelines
[x]  I'm a GSSOC'24 contributor
[x]  I have starred the repository

Hello @Sourabh782 , Go for it
All the Best

Sure will be working on it


It's not working for me

I tried to use ng2-ace-editor in my project and neither using directive or component doesn't work.
No compile error, No error in console. It just doesn't show anything
https://stackblitz.com/edit/angular-ng-ace-edit
Any help to find out what I am doing wrong is appreciated

Add a style to the component and set a height.


Updates create issue button in bug modal to direct user to forum (backup/master)

Pre-Submission Checklist

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

Type of Change

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

Checklist:

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

Description

Equivalent to PR #14979 for backup/master as requested in issue #14934. cc @raisedadead

@BerkeleyTrue I see that we have moved away from showing a modal all together on the challenges and have a get hint button which displays a forum guide.
Still, we should merge this just in case so we have a consistent UI, if we someday bring back creating support issues on the forum.

Hello @padulam
Thanks for the PR but we have since removed the bug modal in favor of the forum ask for help. This is currently in production. I apologies for letting this slip.
Happy Coding


When the tab is not in focus, onInit returns nodes' width and height as undefined

Describe the Bug
Hello 👋
I would like to report what appears to me as a bug of the onInit prop.
The onInit handler passes the reactFlowInstance as a parameter, which I'm using to get the nodes.
When the tab is in focus, the returned nodes have their height and width defined.
However, when the page loads and if the page is not in focus, those values are undefined.
Thanks to the team, amazing library 🙏
Your Example Website or App
https://codesandbox.io/s/shy-pine-6euh2h?file=/App.js
Steps to Reproduce the Bug or Issue
The preview above is a fork of your doc's "feature overview" example.
Happy path:

Load the preview
Open the console
Nodes have a defined height and width

Bug path:

Reload the page,
Focus another tab in your browser
Wait a few seconds so the React flow page finishes to load and render
Check the console again
Nodes' height and width are undefined.

Expected behavior
As a user, I expect that whether the tab is focused or not, the nodes' width and height should be defined.
Screenshots or Videos
https://user-images.githubusercontent.com/16786906/205710633-b9e54106-f972-491d-af32-53327def3c05.mov
Platform

OS: macOS
Browser where I can reproduce: Brave ( 1.38.119 ), Opera ( 92.0.4561.33 )
Browser where it seems to work as intended: Safari ( 16.1 )

Additional context
No response

Thanks for the report @thmsgbrt. This is an interesting bug. We are using the ResizeObserver to measure the nodes on mount. It seems that it doesn't work when the tab is not active. We'll see what we can do here.


access.log and child capture logging have different timestamps

Setting:
access.format = [%t] \”%m %r%Q%q\” %s %{mili}d %{kilo}M %C%%”
Gives:

[06-Apr-2018 06:00:34] WARNING: [pool www] child 17 said into stderr: "xxx"
[06/Apr/2018:06:00:33 +0000] \”GET /index.php\” 200 13.936 4096 71.76%”

So the data formats differ. The one that is generated by the system is [06-Apr-2018 06:00:34] and the one that is configured is [06/Apr/2018:06:00:33 +0000]. To have ELK index them the same they should be set to match.

Writing to stdout in PHP does to the browser. So logging sends to error_log which goes to stderr. That is then captures by the PHP master process so that it outputs it, so that openshift captures it, so that it goes to Elastic search. Only OpenShift doesn't send stderr to research so we redirect stderr to stdout in the launch script. So far, so good.
Annoyingly the parent process logs the child process output with a warning and an additional timestamp. That doesn't match the timestamp being logged by PHP access.log.
The simple fix may be to add a sed filter that drops the additional warning and fixes the timestamp. If the sed (or awk) is configured by an env var then people can customise it.

Should have the sed coerce to common log format and make the timeout stack traces all one line https://httpd.apache.org/docs/1.3/logs.html#common

The fix was to  set access.log = /proc/1/fd/1, and slowlog = /proc/1/fd/1, and php_admin_value[error_log] = /proc/1/fd/1. This causes all the loging to go to stdout at the parent process. That eliminates the "WARNING" about child stderr. Then as the error log date format isn't editable I set the access log format to have a matching datetime with  access.format = [%{%d-%b-%Y %H:%M:%S %z}t] \”%m %r%Q%q\” %s %{mili}d %{kilo}M %C%%


Add ability to disable timeout per-command and wire it in

Addresses #737

NetflixOSS » Hystrix » Hystrix-pull-requests #85 SUCCESS
This pull request looks good


Disable gathering carrier info

In the previous version we could set gatherCarrierInfo to false. That feature does not appear to exist in ApptentiveKit. Are there plans to include that soon?

Great question @andrewjmeier. Looking into this with my team and will get back to you asap.

@andrewjmeier, checked with my team and we do not currently have this ability in the ApptentiveKit SDK. We have it as a work item but it's not yet scheduled. I can keep you updated as we do work on that, though it may not happen this quarter.
Is this an important feature for you and your team? Let me know so we can factor it into our priorities appropriately.

Thanks for such a quick reply @CaseyApptentive! I expect that feature will block us from being able to update.
We also need to figure out how we're going to migrate from using ApptentiveStyleSheet and the path for that is also not particularly clear.

Let me check in with my team about how this fits on our roadmap. I'll follow up as soon as I know more but it may be at least a few days.
Are you currently running an older version of the SDK that does allow for this feature?

Yes, we're currently using version 5.3.4

Thanks @andrewjmeier.
I took a closer look at the backstory here.
Being able to set the gatherCarrierInfo property to false (or NO for Objective-C) on their ApptentiveConfiguration object was introduced in SDK 5.3.4 to free up customers that were seeing a crash due to a bug on Apple's end.
That was a bug that Apple introduced in iOS 14.5.
Since then, this feature has not been widely used -- in fact, I don't recall of hearing of it since 2021. I'd encourage testing and playing it safe, but I believe that this is no longer an issue.
Are you seeing any issues when gatherCarrierInfo is set to true?

Ok, I don't have the context for why we ended up turning it on. I haven't seen any issues with it set to true, but I've only tried on one device so far.
Our main motivation for keeping it set to false is that we don't want to collect additional user data that we don't need. I wouldn't consider carrierInfo particularly sensitive but if we don't actually need to collect it, why should we?

@andrewjmeier thanks for the context. This is a standard piece of information that we typically collect and it does have some built in targeting and reporting functionality around it that can be especially helpful for contacting your customers/understanding any bugs that they might be seeing.
I've let our Product team know about this request but it's likely not very high on our roadmap.
Would you be able to use the new SDK without having this? Let me know so we can prioritize appropriately.


[BEAM-14363] Fixes WatermarkParameters builder for Kinesis

This fixes the WatermarkParameters to use toBuilder() instead of builder().

To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @ajothomas

Can one of the admins verify this patch?

Can one of the admins verify this patch?

R: @mosche ptal


App Crash for LineChart if maxvalue 0

ERROR  Invariant Violation: [26949,"RCTView",1,{"onStartShouldSetResponder":true,"onMoveShouldSetResponder":true,"onResponderGrant":true,"onResponderMove":true,"onResponderEnd":true,"onResponderTerminationRequest":true,"position":"absolute","height":"<>","bottom":57,"width":15,"zIndex":0,"transform":[{"scaleX":1}]}] is not usable as a native method argument
below is my component
const { t, data: actualData = {} } = props;
const [weekGraphData, setWeekGraphData] = useState([])
const totalHourMinRef = useRef(0)
const maxValueRef = useRef(0)
const totalHoursRef = useRef(0)
const totalMinutesRef = useRef(0)
useEffect(() => {
    if (actualData) {
        //plot curve values data on graph
        const dateRange = getlast7Days(t)
        const weekGraphData = dateRange.map((item, index) => {
            let weekDate =getCurrentDate(item.date,'-');
            const backendMappedData = getWeeksData(weekDate,actualData, t)
            return {
                date: item.date,
                label: item.day,
                duration: backendMappedData.duration,
                value: backendMappedData.value,
                time: backendMappedData.time,
                labelComponent: () => customLabel(item.day),

            }
        })
        console.log("weekGraphData.current:::",weekGraphData)
        totalHourMinRef.current = weekGraphData.reduce((total, obj) => obj.duration + total, 0);
        console.log("totalHourMinRef.current:::",totalHourMinRef.current)
        maxValueRef.current = Math.max.apply(Math, weekGraphData.map(function (o) { return o.value; })) + 5;
        console.log("maxValueRef.current:::", maxValueRef.current )
        //small box container data
        totalHoursRef.current = converTimeStampToHoursMinutes(totalHourMinRef.current);
        totalMinutesRef.current = converTimeStampToHoursMinutes(totalHourMinRef.current, 'm');
        setWeekGraphData(weekGraphData)
    }

}, [actualData])

return  <LineChart
                    horizontalRulesStyle={{ width: Dimensions.get('window').width * 0.745 }}
                    height={200}
                    thickness={4}
                    data={weekGraphData}
                    curved
                    // showVerticalLine
                    color={COLORS.textInputBgColor}
                    spacing={LINE_CHART_WIDTH / 7}
                    hideRules
                    initialSpacing={15}
                    maxValue={maxValueRef.current>0 ?maxValueRef.current:5}
                    hideYAxisText
                    yAxisColor={COLORS.white}
                    yAxisThickness={0}
                    xAxisThickness={1}
                    xAxisColor={COLORS.textLightGrey}
                    hideDataPoints
                    pressEnabled
                    // adjustToWidth={true}
                    // unFocusOnPressOut={false}
                    // pointerConfig pointer scroll Effect
                    pointerConfig={pointerConfig}
                />

Earlier its working  fine there is no data change also  not sure why its giving an error now for maxValue 0 and even for same error for BarChart iam using  "react-native-gifted-charts": "^1.2.22",

Hi @asifbadeghar54 👋
Please use the latest version of the library. Feel free to reopen the issue if it doesn't help.

Still happening with rn-v0.68.0 and latest versions of each of react-native-svg, react-native-linear-gradient and react-native-gifted-charts. Happening when maxValue=0.