Migrations: script uses dotenv which does not pick up environment variables

The migrations script uses the dotenv package to pickup the MONGODB_URI. It turns out that this package cannot handle the case where there is no .env file. The equivilent python package will go to actual environment variables and if one is over-ridden by a value in a .env, that will be used.
https://github.com/SciCatProject/scicat-backend-next/blob/c6b2407bddf31fa501057006b4ed4fd343767fb5/migrate-mongo-config.js#L4
This comes up because I'm trying to do a migration right within my container, which defines MONGODB_URI as an environment variable and has no .env file. It would be nice to update this script to look at system environment for this variable too.

I just also noticed that this script ignores authentication. It's very common to put authentication parameters in a URL:
mongodb://slartibartfast:magrathea@db:27017/dacat&authsource=dacat

But migrate-mongo-config.js script parses db name and server name, and in the example above, the database name would come out to be:
dacat&authsource=dacat
@martin-trajanovski  Can we safely remove this parsing and remove the databaseName from the config object?


Add Strava icon

Added Strava logo based on the official badges.

Are there any issues with this PR?

Hey mate! Sorry, I've been on holiday and didn't see your PR. It's perfect.
sorry for the delay, and thanks for contributing.


Choice Adventure Handling for Glob of Melted Wax

Seems to be some weirdness in how the Glob of Melted Wax is handled.
Visit to Clan Basement: The Slime Tube in progress...
Preference lastAdventure changed from The Dire Warren to The Slime Tube
[70] The Slime Tube
Preference lastEncounter changed from Adjust your Parka to Showdown
Encounter: Showdown
Preference lastEncounter changed from Showdown to Mother Slime
Encounter: Mother Slime
Round 0: syrinxlui wins initiative!
Preference cosmicBowlingBallReturnCombats changed from 8 to 7
Round 1: Watchell smiles and cheers, and your own voice, now his, speaks inside your head: "You can do this!"
You acquire an effect: Inner Elf (1)
Preference cosmicBowlingBallReturnCombats changed from 7 to 8
Round 1: Your toy train moves ahead to the Meat Mine Sluice. The sluice opens and pours out some meat!
Preference trainsetPosition changed from 73 to 74
You gain 900 Meat.
Round 1: syrinxlui executes a macro!
Round 1: syrinxlui casts KGB TRANQUILIZER DART!
You lose 900 Meat.
Mother Slime banished by KGB tranquilizer dart.
Preference banishedMonsters changed from fluffy bunny:ice house:0:party girl:snokebomb:60:fluffy bunny:Feel Hatred:67 to fluffy bunny:ice house:0:party girl:snokebomb:60:fluffy bunny:Feel Hatred:67:Mother Slime:KGB tranquilizer dart:69
Preference _kgbTranquilizerDartUses changed from 0 to 1
This combat did not cost a turn
Preference familiarSweat changed from 73 to 74
Preference _concoctionDatabaseRefreshes changed from 669 to 670
You are currently a member of Bonus Adventures from Hell
Preference _concoctionDatabaseRefreshes changed from 670 to 671
Preference _instant_kgbTranquilizerDartUses changed from to Weapon Damage/Inner Elf
Weapon Damage/Inner Elf completed!
Executing Weapon Damage/Glob of Melted Wax
CCS set to grimoire_macro
Autoattack is disabled.
Verifying ingredients for wax hand (1)...
Creating 1 wax hand...
You acquire an item: wax hand
Preference _concoctionDatabaseRefreshes changed from 671 to 672
Successfully created wax hand (1)
Whoops! You're not actually in a choice adventure
Preference battleAction changed from custom combat script to attack with weapon
hpAutoRecoveryTarget => 0.95
Preference hpAutoRecoveryTarget changed from 0.0 to 0.95
mpAutoRecoveryTarget => 0.05
Preference mpAutoRecoveryTarget changed from 0.0 to 0.05
CCS set to grimoire_macro
(but battle action is currently set to attack with weapon)
Preference choiceAdventure1387 changed from 3 to -1
choiceAdventure326 => 0
Preference choiceAdventure326 changed from 1 to 0
Autoattack is disabled.
JavaScript exception: KoLmafia error: Whoops! You're not actually in a choice adventure

This was a longstanding bug with using Mafia's create() function that I never bothered getting around to resolving. 7f535fb21f053626a6164ede1fb604cbd4dbd620 handles this.


Make outline::Toggle work on markdown headings

Check for existing issues

[X] Completed

Describe the feature

0 matches right now 😞

Hey @nikitavoloboev, I think this issue actually covers your request:

https://github.com/zed-industries/community/issues/1142


ord: 0.21.3 -> 0.22.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/ordinals/ord/releases.
meta.description for ord is: Index, block explorer, and command-line wallet for Ordinals
meta.homepage for ord is: https://github.com/ordinals/ord
meta.changelog for ord is: https://github.com/ordinals/ord/blob/0.22.0/CHANGELOG.md
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.22.0 with grep in /nix/store/2yx2dliqd9mg7k8ka2qfrphkyvl09b2b-ord-0.22.0
found 0.22.0 in filename of file in /nix/store/2yx2dliqd9mg7k8ka2qfrphkyvl09b2b-ord-0.22.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

ord

Instructions to test this update (click to expand)

Either download from the cache:
nix-store -r /nix/store/2yx2dliqd9mg7k8ka2qfrphkyvl09b2b-ord-0.22.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nixpkgs-update-cache.nix-community.org/' \
  --option trusted-public-keys '
  nixpkgs-update-cache.nix-community.org-1:U8d6wiQecHUPJFSqHN9GSSmNkmdiFW7GW7WNAnHW0SM=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The nixpkgs-update cache is only trusted for this store-path realization.)
For the cached download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A ord https://github.com/r-ryantm/nixpkgs/archive/b625dd9328a295fb25900e4ca13ecae0904dd7f8.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/b625dd9328a295fb25900e4ca13ecae0904dd7f8#ord

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/2yx2dliqd9mg7k8ka2qfrphkyvl09b2b-ord-0.22.0
ls -la /nix/store/2yx2dliqd9mg7k8ka2qfrphkyvl09b2b-ord-0.22.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

ord

Maintainer pings
cc @xrelkd for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

@NixOS/nixpkgs-merge-bot merge


<+> does not work after liftAuthorizedService

I have
val Auth: SecuredRequestHandler[IO, Int, User, AugmentedJWT[HMACSHA256, Int]] =
    SecuredRequestHandler(jwtStatefulAuth)

and 2 roles
  val AdminRequired = BasicRBAC[IO, Role, User, AugmentedJWT[HMACSHA256, Int]](Administrator)
  val ReaderRequired = BasicRBAC[IO, Role, User, AugmentedJWT[HMACSHA256, Int]](Administrator, ReadAllUser)

try to compose two services which belongs to different permessions
val service1: HttpService[IO] = Auth.liftAuthorizedService(AdminRequired, TSecAuthService {
    case request @ GET -> Root / "s1" asAuthed user =>
      Ok("admin only")
  })

  val service2: HttpService[IO] = Auth.liftAuthorizedService(ReaderRequired, TSecAuthService {
    case request @ GET -> Root / "s2" asAuthed user =>
      Ok("admin and reader")
  })

val service = service1 <+> service2

service1 works, service2 returns 404 in any case. if i swap it
val service = service2 <+> service1

service2 works, service1 returns 404
If I remove Auth, the compose works with the pure Http4s HttpService[IO]

@ioganegambaputifonguser Which version's do you use ?

The version is 0.0.1-M9 (@ioganegambaputifonguser is a colleague)

@Yaneeve So we have the behavior where, to protect from spidering (which has been a really confusing default, I apologize), we make authenticated services consume all (i.e it will never hit the rightmost service).
To get over this (and in general, to use authorization nicely, which we should document but I'm currently @ NE Scala with not much time), you could redo your services like this:
val service1: TsecAuthService[F, A, B] = TSecAuthService(AdminRequired) {
    case request @ GET -> Root / "s1" asAuthed user =>
      Ok("admin only")
  }

  val service2:  TsecAuthService[F, A, B] = sSecAuthService(ReaderRequired) {
    case request @ GET -> Root / "s2" asAuthed user =>
      Ok("admin and reader")
  }

val liftedServices = Auth.liftAuthorizedblahblah(service1 <+> service2)

I am very aware that this is not well documented and could be better, but my time is constrained right now  so I do not have time to make a sample project/update the docs at least within this week.

Did this solve your issues? @Yaneeve @ioganegambaputifonguser?

With this code
val pf: PartialFunction[SecuredRequest[IO, Int, AugmentedJWT[HMACSHA256, Int]], IO[Response[IO]]] = {
    case request @ GET -> Root / "s1" asAuthed user =>
      Ok("admin only")
  }

  val service1 = TSecAuthService.withAuthorization(AdminRequired)(pf, _: (Response[IO], AugmentedJWT[HMACSHA256, Int]) => OptionT[IO, Response[IO]])

  val service2 = TSecAuthService.withAuthorization(ReaderRequired) {
    case request @ GET -> Root / "s2" asAuthed user =>
      Ok("reader only")
  }

I got
Error:(20, 34) ambiguous reference to overloaded definition,
both method withAuthorization in object TSecAuthService of type [A, I, F[_]](auth: tsec.authorization.Authorization[F,I,A])(pf: PartialFunction[tsec.authentication.SecuredRequest[F,I,A],F[org.http4s.Response[F]]], andThen: (org.http4s.Response[F], A) => cats.data.OptionT[F,org.http4s.Response[F]])(implicit F: cats.Monad[F])tsec.authentication.TSecAuthService[I,A,F]
and  method withAuthorization in object TSecAuthService of type [A, I, F[_]](auth: tsec.authorization.Authorization[F,I,A])(pf: PartialFunction[tsec.authentication.SecuredRequest[F,I,A],F[org.http4s.Response[F]]])(implicit F: cats.Monad[F])tsec.authentication.TSecAuthService[I,A,F]
match argument types (tsec.authorization.BasicRBAC[cats.effect.IO,kombucha.auth.AuthHelpers.Role,kombucha.auth.AuthHelpers.User,tsec.authentication.AugmentedJWT[tsec.mac.imports.HMACSHA256,Int]])
  val service1 = TSecAuthService.withAuthorization(AdminRequired)(pf, _: (Response[IO], AugmentedJWT[HMACSHA256, Int]) => OptionT[IO, Response[IO]])

Error:(22, 34) ambiguous reference to overloaded definition,
both method withAuthorization in object TSecAuthService of type [A, I, F[_]](auth: tsec.authorization.Authorization[F,I,A])(pf: PartialFunction[tsec.authentication.SecuredRequest[F,I,A],F[org.http4s.Response[F]]], andThen: (org.http4s.Response[F], A) => cats.data.OptionT[F,org.http4s.Response[F]])(implicit F: cats.Monad[F])tsec.authentication.TSecAuthService[I,A,F]
and  method withAuthorization in object TSecAuthService of type [A, I, F[_]](auth: tsec.authorization.Authorization[F,I,A])(pf: PartialFunction[tsec.authentication.SecuredRequest[F,I,A],F[org.http4s.Response[F]]])(implicit F: cats.Monad[F])tsec.authentication.TSecAuthService[I,A,F]
match argument types (tsec.authorization.BasicRBAC[cats.effect.IO,kombucha.auth.AuthHelpers.Role,kombucha.auth.AuthHelpers.User,tsec.authentication.AugmentedJWT[tsec.mac.imports.HMACSHA256,Int]])
  val service2 = TSecAuthService.withAuthorization(ReaderRequired) {

for both of them

Try:
service1: TsecAuthService[IO, User, AugmentedJWT[HMACSHA256, Int]] instead of annotating the partial function. Scalac has inferencr issues around this many type params for some reason

Thank you so much for the support, but unfortunately I get the same error
What I have for now
val service1: TSecAuthService[User, AugmentedJWT[HMACSHA256, Int], IO] =
    TSecAuthService.withAuthorization[AugmentedJWT[HMACSHA256, Int], User, IO](AdminRequired) {
      case request @ GET -> Root / "s1" asAuthed user =>
        Ok("admin only")
    }

error
/Users/theodor/work/kombucha/src/main/scala/kombucha/company/CompanyService.scala:18:38: ambiguous reference to overloaded definition,
[error] both method withAuthorization in object TSecAuthService of type (auth: tsec.authorization.Authorization[cats.effect.IO,kombucha.auth.AuthHelpers.User,tsec.authentication.AugmentedJWT[tsec.mac.imports.HMACSHA256,Int]])(pf: PartialFunction[tsec.authentication.SecuredRequest[cats.effect.IO,kombucha.auth.AuthHelpers.User,tsec.authentication.AugmentedJWT[tsec.mac.imports.HMACSHA256,Int]],cats.effect.IO[org.http4s.Response[cats.effect.IO]]], andThen: (org.http4s.Response[cats.effect.IO], tsec.authentication.AugmentedJWT[tsec.mac.imports.HMACSHA256,Int]) => cats.data.OptionT[cats.effect.IO,org.http4s.Response[cats.effect.IO]])(implicit F: cats.Monad[cats.effect.IO])tsec.authentication.TSecAuthService[kombucha.auth.AuthHelpers.User,tsec.authentication.AugmentedJWT[tsec.mac.imports.HMACSHA256,Int],cats.effect.IO]
[error] and  method withAuthorization in object TSecAuthService of type (auth: tsec.authorization.Authorization[cats.effect.IO,kombucha.auth.AuthHelpers.User,tsec.authentication.AugmentedJWT[tsec.mac.imports.HMACSHA256,Int]])(pf: PartialFunction[tsec.authentication.SecuredRequest[cats.effect.IO,kombucha.auth.AuthHelpers.User,tsec.authentication.AugmentedJWT[tsec.mac.imports.HMACSHA256,Int]],cats.effect.IO[org.http4s.Response[cats.effect.IO]]])(implicit F: cats.Monad[cats.effect.IO])tsec.authentication.TSecAuthService[kombucha.auth.AuthHelpers.User,tsec.authentication.AugmentedJWT[tsec.mac.imports.HMACSHA256,Int],cats.effect.IO]
[error] match argument types (tsec.authorization.BasicRBAC[cats.effect.IO,kombucha.auth.AuthHelpers.Role,kombucha.auth.AuthHelpers.User,tsec.authentication.AugmentedJWT[tsec.mac.imports.HMACSHA256,Int]])
[error]     TSecAuthService.withAuthorization[AugmentedJWT[HMACSHA256, Int], User, IO](AdminRequired) {

Alright, yeah, this is an actual issue with method inference we need to fix.
Thank you for reporting. If I remove the second withAuthorization method that's totally unused, it compiles.
For now, I can give you a workaround to this:
    def withAuthorization[I, A, F[_]](auth: tsec.authorization.Authorization[F, I, A])(
        pf: PartialFunction[SecuredRequest[F, I, A], F[Response[F]]]
    )(implicit F: Monad[F]): TSecAuthService[I, A, F] =
      Kleisli { req: SecuredRequest[F, I, A] =>
        auth
          .isAuthorized(req)
          .flatMap(_ => pf.andThen(OptionT.liftF(_)).applyOrElse(req, Function.const(OptionT.none)))
      }

This will do what you want it to do. I'll make sure to remove the extra method with the andThen in a PR today, as this is clearly impacting inference negatively.

it will work but unfortunately I have to say that we came from where we started
def liftAuthorizedService(
      authorization: Authorization[F, User, Auth],
      service: TSecAuthService[User, Auth, F],
      onNotAuthenticated: Request[F] => F[Response[F]] = defaultNotAuthenticated
  )

does not accept TSecAuthService <+> TSecAuthService, it receives only one based on provided authorisation.
I tried val service = Auth.liftService(service1) <+> Auth.liftService(service2) but it also seems consumes everything in the first Auth and I get 404 from the service2 :(

@ioganegambaputifonguser this is why I am saying, please add the method in my comment above to a helper of some sort, then use that, while I fix this in a PR. i.e
object TSecAuthHelper {
    def withAuthorization[I, A, F[_]](auth: tsec.authorization.Authorization[F, I, A])(
        pf: PartialFunction[SecuredRequest[F, I, A], F[Response[F]]]
    )(implicit F: Monad[F]): TSecAuthService[I, A, F] =
      Kleisli { req: SecuredRequest[F, I, A] =>
        auth
          .isAuthorized(req)
          .flatMap(_ => pf.andThen(OptionT.liftF(_)).applyOrElse(req, Function.const(OptionT.none)))
      }
}

val service1: TSecAuthService[User, AugmentedJWT[HMACSHA256, Int], IO] =
    TSecAuthHelper.withAuthorization(AdminRequired) {
      case request @ GET -> Root / "s1" asAuthed user =>
        Ok("admin only")
    }

I don't mean this as a long term fix, just for now, while I create a PR to fix this issue.

Yes yes, I got it
It works
But the problem is here where we actually started, how to compose service1 and service2 after creation?
Now I have
service1: TSecAuthService[User, AugmentedJWT[HMACSHA256, Int], IO] and service2: TSecAuthService[User, AugmentedJWT[HMACSHA256, Int], IO] with this temporary solution.
As I said val service = Auth.liftService(service1) <+> Auth.liftService(service2) gives 404 from service2.
The Auth.liftAuthorizedblahblah(service1 <+> service2) does not work here also because It works with the only one TSecAuthService with the concrete Roles set, you can not pass (service1 <+> service2.

If you apply authorization on the individual services, you don't need to apply it from the handler.
Use .liftService and it should work just fine.
Something like:
  val auth1: BasicRBAC[IO, Role, User, Int] = BasicRBAC(Administrator: Role)
  val auth2: BasicRBAC[IO, Role, User, Int] = BasicRBAC(Customer)

  val s: TSecAuthService[User, Int, IO] =
    TSecAuthService.withAuthorization(auth1) {
      case GET -> Root asAuthed u =>
        Ok()
    }

  val q: TSecAuthService[User, Int, IO] =
    TSecAuthService.withAuthorization(auth2) {
      case GET -> Root asAuthed u =>
        Ok()
    }

  val composedService: TSecAuthService[User, Int, IO] = s <+> q

  def myhandler: SecuredRequestHandler[IO, Int, User, Int] = ???
  myhandler.liftService(composedService)

Thank you one more time, it works with this approach
  val composedService: TSecAuthService[User, Int, IO] = s <+> q

  def myhandler: SecuredRequestHandler[IO, Int, User, Int] = ???
  myhandler.liftService(composedService)

@jmcardon I hope it will help https://github.com/jmcardon/tsec/pull/138
I search within the codebase, it seems they both (overloaded withAuthorization and liftAuthorizedService) not used anywhere

closed by #138
Thank you very much @ioganegambaputifonguser


Issue with 6wha Simulation Output

Hi @davidcooper7
I hope you're doing well.
I’m facing an issue with the 6wha simulation where it crashes while trying to save the output file. The error says the variable pos isn’t properly initialized in the truncate_ncdf function.

Thankyou.

It appears you have an empty save directory and corrupted output.ncdf file.
Please try deleting the directory /expanse/lustre/projects/iit122/akhan3/5HT2A/replica_exchange/6wha_0/saved_variables/18 and delete the *.ncdf files in /expanse/lustre/projects/iit122/akhan3/5HT2A/replica_exchange/6wha_0
Then try resuming the simulation.

Ok, I will do. Thankyou so much, @davidcooper7

Did this work @aimankhan131 ?

Yes, it worked because the job has been running since yesterday, but I'll monitor it until it's completion to see if I encounter any errors or not and will let you know @davidcooper7


Include ARM32 GCC and python-crypto in ARM setup.

This PR adds additional tooling to the required packages list to build for ARM:

32-bit ARM compilers (soft-float and hard-float);
Python cryptography module (for TA signatures).

I had originally added these changes to #1820, but to be able to pass CI, these tools must be in the oetools-full-* containers, so I've separated them into a new PR.

bors r+

bors r+

bors r+

@HernanGatta
I've triggered the Jenkins job to generate new OE CI Docker images after this pull request got merged.
It finished successfully. The changes from here should be bundled in the latest OE CI Docker images.

@ionutbalutoiu Thanks!


adding HO array and LJ testsystem

Description
This PR will add additional test systems (HO array and LJ fluids), functionality, and tests for combined sampling with LangevinIntegrator and MC moves as well as PBC
Todos

[x] HO array test system
[x] LJ test system

Status

[x] Ready to go

I change around a bit of the code structure, causing the test_potential tests to fail. Will fix that in a bit.

Summary of additions so far:

updated the pairlist calculation (fixing some bugs causing the LJ test to fail) and making it use jax functions to enable compatibility with jax.grad
Added in the neighborlist; require small refactoring to the base potential class (namely passing an instance of the neighborlist).
Tests have been added for the neighborlist and additional tests to the state (including fixing a typing issue that seemed to be causing some issues for returning values with units dropped: explicit tests were added for this).

To do still:

Additional tests for the LJ potential to ensure we are getting the correct energy and force for a 2 particle system.
Refactor the langevin integrator to know about the neighborlist, as during a set of moves we will need to check and rebuild as necessary.  These functions will also be jitted.
implement lj fluid test system.

I think this looks good as it is. After review, we can merge IMHO.

Codecov Report

Merging #4 (07fefd6) into main (98dd17d) will increase coverage by 3.12%.
The diff coverage is 81.88%.

Additional details and impacted files


Azure auth fail

Is this a ship blocker? Or did you decide that this could be fixed later? @LittleLittleCloud Did you speak about this yesterday at standup?

It should be fixed already


Circular dependencies with uproxy_core in metrics.ts

@jpevarnek and I just spent a while trying to debug some testing errors caused by running a static method of uproxy_core.uProxyCore inside metrics.ts. (The same error occurs if you try to instantiate a uproxy_core.uProxyCore object there as well.)
This is apparently caused by a circular dependency caused by loading uproxy_core.ts's dependencies inside metrics.ts. The first error looks like this:

This can be solved by prepending
import freedomMocker = require('../../../third_party/uproxy-lib/freedom/mocks/mock-freedom-in-module-env');
import mockFreedomRtcPeerConnection = require('../../../third_party/uproxy-lib/freedom/mocks/mock-rtcpeerconnection');
import freedom_mocks = require('../mocks/freedom-mocks');
freedom = freedomMocker.makeMockFreedomInModuleEnv({
    'core.storage': () => { return new freedom_mocks.MockFreedomStorage(); },
    'loggingcontroller': () => { return new freedom_mocks.MockLoggingController(); },
    'metrics': () => { return new freedom_mocks.MockMetrics(); },
    'core.tcpsocket': () => { return new freedom_mocks.MockTcpSocket(); },
    'core.rtcpeerconnection': () => { return new mockFreedomRtcPeerConnection(); }
});

to the top of metrics.spec.ts, which causes a new error:

duplicate of https://github.com/uProxy/uproxy/issues/1737


Fix of bug of kv_channels in TransformerLayer and added Gemma tutorial

I'm working on tutorial similar to tutorial with Llama, but with Gemma model. Most parts are similar, but I have encountered few differences:

Official weights are in Safetensor format, not as in the case of Llama - torch checkpoints. I modified the loading function.

There is geglu activation function instead of swiglu, but this is also supported by the TE. It was one simple change in config.

Gemma hidden dimension is different than attention dimension - attention projections change dimension from hidden(3072) to key/query/value(4096). There is parameter kv_channels of the TransformerLayer in TE, which seems to be responsible for exactly that. Nevertheless, changing it to value different than hidden_dim / num_heads causes Assertion Error and is not changing number of parameters.

I propose a change of argument kv_channels to the attention_hidden_dims. I changed the description and number of neurons in attention projection layers in transforemer.py and attention.py files.
The results of Gemma with bf16, bf16 and TE, fp8 and TE are proportional to the results of the Llama.

I was thinking about adding two parameters attention_hidden_dims_v and attention_hidden_dims_kq, because it is mathematically possible that hidden dimension of key vector and query vector is not equal to dimension of value vectors.
Nevertheless, FlashAttention assumes that this numbers are equal, so I gave up this idea.

Hmm, can we get the same effect without changing the API? That would be a breaking change and we would like to avoid that if possible.

I split this PR into 2:
https://github.com/NVIDIA/TransformerEngine/pull/833 (kv_channels part)
https://github.com/NVIDIA/TransformerEngine/pull/829 (gemma part)
Thus I close it.


x/sys/windows: Consider exporting kernel32 *LazyDLL

Currently all libraries needing more procedures load the libraries by hand using either syscall or x/sys/windows. The most common DLL by a large margin used in this fashion appears to be kernel32.dll.
This means that in larger applications there are half-a-dozen libraries loading kernel32.dll. They could use a shared copy since *LazyDLL is safe for concurrent use.
Exporting:
var ModKernel32DLL = NewLazySystemDLL("kernel32.dll")

from x/sys/windows would enable sharing this for applications.
x/sys/windows loads a copy of kernel32.dll in any case (modkernel32).
If needed I can write a proposal or a patch.

I don't see what the benefit here. Is something going to be faster? Less lines of code? What? Please explain. Thank you.
Alex

@alexbrainman
It would call LoadLibrary once for sys/x/windows instead once for each library needing functions from kernel32.dll.
Underneath Windows does reference counting of the DLLs loaded so calling LoadLibrary("kernel32.dll") multiple times results in only a single copy of the library in the address space. So the saving is a little bit of memory and few less syscalls.

It would call LoadLibrary once for sys/x/windows instead once for each library needing functions from kernel32.dll.

I don't see any benefit here. It is just few more calls into LoadLibrary. It is not slow code. And average app calls LoadLibrary just couple of times. And since I use $GOROOT/src/syscall/mksyscall_windows.go to generate my syscalls, I don't even have to write that code.
It is not worth the trouble.
Alex

Ok, thanks for considering it.


Add cluster lifecycle information to docs

Issue: #5119
Description of changes: This PR starts the documentation process for adding cluster lifecycle features to EKS Anywhere docs. In particular, it notes the ability to use kubectl to create workload clusters for vSphere, Docker, and Snow. It also describes this feature in the cluster workflow and cluster topology docs.
I still have some questions, so I have not yet added cluster lifecycle content for Nutanix, Bare Metal, or Cloudstack. I also need more information on how cluster lifecycle should be described in relation to workload cluster upgrades and deletions. I will either add that information here or create new PRs after this one merges.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/approve

/approve
/lgtm


feat:(translation) AM - Add Amharic Translations

Hello again.
I have completed the Amharic am translation for the given files in /rails/config/locales/am. I hope the PR is to your standard. I would be happy to work on it more as the project grows. Thanks for having me.
Regarding #826
Ping me if there are any issues.

Looks great! አመሰግናለሁ for this wonderful translation work! I hope there might be some Ethiopian communities interested in using Terrastories :)

😄 Thanks for approving the PR. It was fun working on the translations. Same here, I will try to introduce the project to some people here. I will be glad to work on future translations as well. Appreciate the support.
Glad you picked up a few words :)


use in databricks

Hi,
#It's a question , not an issue.
I need to process a mainframe file in azure databricks , which has certain comp-3 values as well. I have the copy book in cobol for the layout of the schema.
I could not find any reference on how to use this in databricks using pyspark python 3.
Can you please provide a sample code , on how do we integrate/use cobrix in azure databricks.
A prompt reply would be appreciated.
Regards,
Divya

Hi,
I need to read a schema in cobol copybook using python in AWS. Is there any suggestions.

You can get Spark schema if you have a DataFrame the same way as in Scala:
df.schema

or
df.schema.treeString

or
df.printSchema

You can get COBOL schema as an AST like this:
val copybook = CopybookParser.parseTree(copyBookContents)
copybook1.generateRecordLayoutPositions

for databricks its much simpler . All you need to do is go to open your cluster for installing libraries option and install cobrix  either by passing the jar maven file verison : za.co.absa.cobrix:spark-cobol_2.12:2.5.1 or by downloading the jar from maven and uploading it to cluster.


Escape backslash in xo_emit.3 so it renders correctly

The unescaped "\n" in xo_emit.3 generates incorrect output when rendered:
           xo_emit(" {:lines/%7ju} {:words/%7ju} "
                   "{:characters/%7ju} {d:filename/%s}0,
                   linect, wordct, charct, file);

This change escapes as "\en" so that the example code renders correctly.

I have a fix in the "develop" branch changing the "\n" to "\n".  Is that sufficient or does it need changed to "\en"?
Either way, I'll need to spin a new release and import it into freebsd.
Thanks,
Phil

I'm afraid those both look the same to me. In my original patch I wrote "\n" and it rendered right, but on reading the nroff documentation, \en is preferred, it appears. But we could ask someone who actually understands nroff for their opinion?

Fixed in 'develop'


[clustertest] Support 30val+30fn setup for run-ci-suite experiments

Summary

Update PerformanceBenchmark, PerformanceBenchmarkThreeRegionSimulation to support fullnodes
Update accounts_per_client default value to 15
Reorder the experiments in run-ci-suite so that 10% down experiment is run at last

Performance report:
3 Region Simulation : 831 TPS, 2241.0 ms latency
perf flamegraph : https://toro-cluster-test-flamegraphs.s3-us-west-2.amazonaws.com/7yNCPN0icMNptHFJAI4vPWp8NWA0bg-libra-node-perf.svg
all up : 973 TPS, 1863.8 ms latency, no expired txns
10% down : 713 TPS, 1712.1 ms latency, no expired txns

The suite on this ran successfully
Performance report:
all up : 1043 TPS, 1858.4 ms latency, no expired txns
10% down : 1027 TPS, 1711.4 ms latency, no expired txns
3 Region Simulation : 777 TPS, 5700.1 ms latency
perf flamegraph : https://toro-cluster-test-flamegraphs.s3-us-west-2.amazonaws.com/SlIGxPDrywSvdzhwHkwaKhZE8QzPPZ-libra-node-perf.svg

For validator_index you can probably also just look into FullNodeConfig::validator_index, this would allow to avoid regex alltogether

For validator_index you can probably also just look into FullNodeConfig::validator_index, this would allow to avoid regex alltogether

For now, I just stuck to the regex parsing approach

:v: @ankushagarwal can now approve this pull request

@bors-libra r=andll

:pushpin: Commit 76a9817 has been approved by andll

:hourglass: Testing commit 76a9817fdb73287a42f6587a9234b1df8b35b4ae with merge 5467b5f56977d412bfb8d4e35739dfd2188f5ece...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: andll
Pushing 5467b5f56977d412bfb8d4e35739dfd2188f5ece to master...


Update version to 1.1.11-2 on staging

Update version to 1.1.11-2

🚀 Deployed to staging by @OSBotify in version: 1.1.11-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @yuwenmemon in version: 1.1.12-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Allow @kotlinx.serialization.Serializable for Navigation Results

Kotlin's @Serializable types currently work as navigation arguments, but not for navigating back with a result using ResultBackNavigator and ResultRecipient.
Is it possible to add support for @Serializable there as well?

Hello @raamcosta, Is there any way I can help implement this? I would like to see a support for this, or rather I need it because without this kotlinx.serializable is not possible to fully use with this library and I don't want to resort to using parcelable or other serializers when the official kotlinx.serializatio is here with us today.
You wrote last this:

kotlin("plugin.serialization")

Hmmm I'd like to avoid this if possible. I don't known yet exactly how.. I'm guessing we can also generate the serializers like we do for navigation on the user's module and have them point to them by passing their instance or something like this.
To be honest, if this is the only way, I'd rather not do it. I don't want to force this dependency on all users of the library. Maybe we could make this a separate dependency, but that also comes with additional overhead for all maintainers...

So do you have an idea how it should work and how to implement it?
From what I've seen in the code for resultRecipients / resultNavigator there is no generating the serializers now as in the case of navigation arguments, is that right?

@raamcosta any updates on this please? It forces me to not to pass data back as originally intended but only by ID or serialize to String since I removed all Parcelable/serializable imports due to KMM

+1 for this request. When reading the docs I thought this was already implemented but we get this error:

com.ramcosta.composedestinations.codegen.commons.IllegalDestinationsSetup: Composable Screen, ResultDataModel: Result types must be one of: String, Long, Boolean, Float, Int, Parcelable, Serializable

I guess Serializable here is referring to java.io.Serializable ? I had to dive into the source code to figure that out.. I think Serializable referring to @kotlinx.serialization.Serializable would be a more natural conclusion for most Kotlin projects..

Ah seems like java.io.Serializable vs @kotlinx.serialization.Serializable is a common mistake: https://github.com/raamcosta/compose-destinations/pull/266#issuecomment-1588719190

Any updates on this? @raamcosta

Hi 👋
No, this is not a priority for v2 at the moment.

No efforts done here so far.
Given this is for now an Android only library, I don’t think it’s a big deal, just mark the class as Java Serializable and move on (or Parcelable but that’s a bit more work I guess).
That said, I am working on KMP support and this will be needed for the feature to work, so it will probably force me to look at this again.

Good news!
As expected while adding KMP support, it made me look into this.
I already know how I'm going to make this work and it will be even better than allowing Kotlinx Serializable. Basically, any supported navigation argument type will also work in the back navigation case.
I do need quite some changes to make this happen, but I think the result will be pretty great!
Stay tuned!

@raamcosta That's great news. I look forward to it. Thank you

This is done, available on today's releases.
Check release notes for more info, but most likely you won't need to do anything special, just use it like you'd expect and it will work :)

Please let me know once you try it 🙏
I'll be waiting for feedback!

@raamcosta Working well for me. Thank you man!


Use comparison

Hello,
I've run into your implementation of a material data table and into this one : https://github.com/daniel-nagy/md-data-table .
Since he started to develop his version before yours, and I bet you were well aware of it, can you tell me what motivated you to develop a version of your own ? For what reason/case I should use your version over your (or opposite) ?
Thank you

I'd recommend to read this:
http://stackoverflow.com/questions/27511487/is-there-a-table-component-in-angular-material
I didn't copy, we had the same idea, I guess he was copying mine after he saw mine. (probably he created his repo, I don't know, but I was for sure not copying him)
I prefer mine, since he put a lot of things which was not part of google material table design. I'm sticking to that design and trying to do it in the right way, excluding 100 different and useless feature as much as I can.


Update flyway-core to 10.8.1

About this PR
📦 Updates org.flywaydb:flyway-core from 9.22.3 to 10.8.1 ⚠
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.flywaydb", artifactId = "flyway-core" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.flywaydb", artifactId = "flyway-core" }
}]

labels: library-update, early-semver-major, semver-spec-major, commit-count:1

Superseded by #284.


Taskbar & Jumplists?

Is there any samples that go in to how to set the progress bar in the taskbar or how to manage jumplists?

That functionality is not yet available to Universal apps, sorry.

Well since th2 supports it, there should be a JumpList sample.

Coming soon. Watch this space!

Excellent!
On 14 November 2015 at 04:01, Raymond Chen<EMAIL_ADDRESS>wrote:

Coming soon. Watch this space!
—
Reply to this email directly or view it on GitHub
https://github.com/Microsoft/Windows-universal-samples/issues/150#issuecomment-156542417
.

--
Mike Cann
http://www.mikecann.co.uk/
@mikeysee http://www.twitter.com/mikeysee

@oldnewthing is there any API for progress status on the Taskbar?

Not yet.

Did build announce and new features which may enable progress bars in the taskbar in UWA?


Tool to Automatically Rename Files in vMix Output Folder

Idk soren can do it

This should be implemented inside of AutoAV.


Fixes to Chosen on formsets

Resolves #406

Works as advertised. :-)

I like how switching to using <script> tags also made everything just generally cleaner.


python2 support

python2 support
add 2 new board option
close serial before raise Error which cause open serial fail on windows ( or it(release serial) can be done in __del__ func of MAIXLoader)

why not import six


Improvements in glidein startup script

Added some indentation and structural improvements to the existing glidein_startup.sh script to ensure consistency  and promote better readability of code, respectively.
Note that these changes do not modify the existing logic/behavior, so the underlying logic of the glidein startup process is preserved from before.

This has been superseded by #373 including all its changes.


Finding all root chunks or accidentally "loose" chunks

Do you have any tool (or way of working) to avoid that a chunk is loose? Consider the following:
The program outline:
<<*>>=
<<imports>>
<<helper functions>>
<<main program>>
@

Now we want to do the following.
<<main program>>=
val = func(0)
<<functions>>=
def func ...

Note that we have <<functions>> instead of the intended <<helper functions>>.
Do you have any tool or other technique to help detect this? That chunk <<functions>> becomes a "loose" chunk since it's not linked in from <<*>>. Strictly it's its own root chunk. So do you know of any tool that lists root chunks of a file? Or how do you handle this when you write?

Are you looking to find a chunk that has a definition but no use?  The noroots script will do that for you.  It is part of the distribution.

Yes, that's exactly what I wanted, thanks! Awkward that I didn't find it myself. I even started looking through the contrib directory. But then thought that this can't be hidden in there, must be something you use more often.
Thanks again!


List v2 POC: Virtualization

@ashit-rath could u give me more details what do u want? if yeah .. can i get it?

@tauantcamargo Thanks for showing interest to contribute. This is a part of an ongoing project for the List widget's version 2 which is being actively developed by the internal team. As this is a time-sensitive and internal only project's sub task, we sadly cannot assign this to you.
I would highly recommend you to check out Good First Issues or Hacktoberfest to get started or other open issues.


Add javascript functionality to mobile menu

Add ES Lint

javascript functionality after clicking the menu button and the close button

added dynamic css classes using javascript

navigation using the menu

Hi @Team
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project, that does not have the javascript linters also setup, for this project both the JavaScript and HTML&CSS linters should be configured
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification @prolajumokeoni.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


Bad magic number when reading EEPROM

Hello,
I am trying to read the EEPROM using k5prog.
At random moments during the reading of the eeprom the communication is failing with error "bad magic number". Also the station hangs on "uart config comms" until I remove the KW-type cable.
Please find attached the logs of two failing attempts and a successful dump with OEFW flashed on the same device. It is interesting that the returned "bad magic" is same readable string ps w in any of the failing reads.
Model: UV-K5
FW: 1o11-a552b04
Comm cable: Baofeng branded CH341 UART serial
k5prog cmdline: ./k5prog -p /dev/ttyUSB0 -vvvv -r -f uv-k5-eeprom-oefw.bin 2>&1 | tee uv-k5-eeprom-oefw.log
uv-k5-eeprom1.log
uv-k5-eeprom2.log
uv-k5-eeprom-oefw.log

I am confirming the k2prog tool successfully dumped the EEPROM. Works with the QS's CPS software too.
An other, minor, issue noted - the welcome voice on power on speaks both English and Chinese welcome phrases.
Should I close this issue and open another one or you'll handled this?

Speaks both languages, like one after another ?

Yep. Starts with English and ends with Chinese. The end of the Chinese
is instantly followed by speaking the channel name in english (it is
booting in memory mode, not vfo)
It is like "Whelcome-tianchinsomething-channel-x". Only the welcome
message. all other voices sounds OK (in english)
On Tue, Oct 10, 2023 at 8:59 PM OneOfEleven @.***> wrote:

Speaks both languages, like one after another ?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Switched to VFO mode. On Power on it is like welcome-two-two-chin(different chiunese)-Frequency-mode

oops, have you got a backup of all your channels etc ?

Please don't worry - nothing lost. it is entirely experimenting device.
Here is my eeprom.  The transceiver is powering on in VFO on A channel and Mem on B channel:
uv-k5-eeprom6.bin.gz

oh good. Just glad you have a backup of your settings.
No it's OK I'm just looking for the problem.

Just let me know if there is something I could help with.
Here is the first "successful" eeprom read. The transceiver was booting in Mem on both A and B channels (well with the funny welcome. I was eager to test the eeprom reading and then looked at the other things.
uv-k5-eeprom4.bin.gz

Hello,
The eprom read via k5prog works with latest build too.
I'll close this isasue and open a new one with the welcome sound as it is irrelevant to this one


Rules are not visible through the BIG-IP GUI

When creating a policy with rules the rule names show up under the policy but when clicking the rule name an error occurs Object not found - 0
This appears to be caused by creating the rule with the name formatted as /partition/name.
If the rule is created without the partition in the name the GUI works as expected.
"name": "/partition/0",
"name": "0",

Dan,
Is this an issue with the new CCCL?  It might be since some of the logic you implemented was brought forward.
Rich

It is with the new CCCL, it was discovered when the uplift was being done for cf. K8's also has the same issue now.


feat: netinfo new component

Adding Netinfo component

:tada: This PR is included in version 1.80.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix typo "Experimantal"

Description
Minor typo fix
Checklist

[x] The base branch of this PR is preview, rather than main

Thanks for cleaning up after me @pzaback  : )


Fixed the issue: Website UI Improvisation

This pull request addresses various user interface (UI) improvements across the website to enhance user experience and visual appeal. The following changes have been implemented:
Key Improvements:
Enhanced Navigation Bar:
Improved responsiveness and hover effects for navigation links, ensuring better accessibility and interaction.
Changed hover colors to provide visual feedback only on the hovered link.
Updated Add Component Styles:
Refined the layout and alignment of buttons and text areas for a cleaner, more organized appearance.
Improved color schemes and padding to create a more visually appealing design.
Consistent Design Language:
Applied uniform styling across components to ensure consistency throughout the application, enhancing the overall aesthetic.
Updated typography and spacing for better readability.
Optimized File Structure:
Refactored CSS files to promote better maintainability and organization of styles.
Removed redundant styles and ensured that all components adhere to the latest design guidelines.
Improved Responsiveness:
Ensured that the UI is fully responsive across various devices and screen sizes, providing a seamless experience for all users.
Before

After:

This PR is related to issue #1


Add kakao to Solid and Svelte

Current dependencies on/for this PR:

main

PR #216   👈

This comment was auto-generated by Graphite.


Feature: Source connector modes

Feature description
Add ability source connector to working in following modes:

read existing data, enable CDC
ignore existing data, enable CDC

It was added in this PR:
https://github.com/conduitio-labs/conduit-connector-stripe/pull/8


Drawer + Tabs

Description
First version of Drawer + Tabs for Expo router

Related Issue

Motivation and Context
To have the ability to choose Drawer + Tabs as an option to the CLI

How Has This Been Tested?
Locally

Screenshots (if appropriate):

@asapMaki can you rebase this PR with main? Recently merged a large PR from @ludwig-pro so perhaps he can offer some input if you get lost?

This PR is missing the following:

[ ] Ability to generate a drawer+tabs project via non-interactive mode (ie pass in --drawer+tabs
[ ] Proper output of generateRerunScript
[ ] Correct back button in react-navigation template in the Modal presentation

[ ] Type errors in drawer-navigator.tsx in the react-navigation template

Completed 4/5 tasks, last one is relatively minor and I don't know enough about React Navigation to fix. I created https://github.com/danstepanov/create-expo-stack/issues/191 to track the issue and we can address in a future release.


Feature - Activity Indicator

Is your feature request related to a problem? Please describe.
As accused at the 1st Acceptance Test, the user has no feedback to tell him/her if the app is working properly.
Describe the solution you'd like
A simple way of giving to the user some feedback is putting an activity indicator, which will be loaded on the badge's place while the results are not received.
Describe alternatives you've considered
Another great solution, even better, is to render a progress bar indicating how much of the total already was calculated. Even if this is a better option, the complexity of implementation is beyond we can offer for now. Let's just try simple solutions.
Additional context
Any more context can be obtained at the 1st Acceptance Test Report

When I was open a pull request I've noticed something quite annoying, when I go to a repository(or open a pull request situation, for example) the loading bar keeps running outside the code tab, maybe that's not the expected response and behavior because there are no badges over there, as you can see at the screenshot under.

#37 may solve this problem


Update llvm_tot anchor and LLVM_TOT_VERSION file

Part of #176.

Looks like clang-13 is live in TuxSuite so I will update this PR to do the rest of #176.
https://gitlab.com/Linaro/tuxsuite/-/issues/129

Already, this should close out #176.
For the clang-13 builds, I just copied all of the builds we are doing for LLVM 14. If you want me to slim up the list, I can but it might not be a terrible idea to keep all of that coverage so we can spot ToT breakages more easily.


White Latex Oscillator 2

https://github.com/user-attachments/assets/fa758f32-a640-49d8-a772-24290d60f110

Can I ask that you don't have music playing in videos you submit here?

ok


ci: Removed not needed if for test job

Codecov Report

Merging #7 (d47cd1b) into main (a27d865) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #7   +/-   ##
=======================================
  Coverage   88.31%   88.31%
=======================================
  Files           1        1
  Lines          77       77
  Branches       18       18
=======================================
  Hits           68       68
  Misses          2        2
  Partials        7        7

Continue to review full report at Codecov.

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


Triage CI workflow failing for some PRs

Summary
https://github.com/maliput/maliput_infrastructure/pull/285
Triage workflow is failing for PRs https://github.com/maliput/maliput_infrastructure/pull/285 and https://github.com/maliput/maliput_documentation/pull/114

Wondering if it is related that the PRs are created from a fork. (I hadn't tried that before.)

Here are the instructions that ask users to create forks:

Fork the Maliput package you want to contribute to. This will create your own personal copy of the package.  - An important thing to do is create a remote pointing to the upstream remote repository . This way, you can always check for modifications on the original repository and always keep your fork repository up to date.

https://maliput.readthedocs.io/en/latest/contributing.html#contributing-code

It is related to the github token being passed when the PR is created from a fork:
Solved at

https://github.com/maliput/maliput_documentation/pull/116
Using pull_request_target event instead.
This event uses the context of the base branch.

Evidence that it is now working

PR from a fork: https://github.com/maliput/maliput_documentation/pull/117
PR from the same repo: https://github.com/maliput/maliput_documentation/pull/118

We should propagate this change to all the repositories!

[x] maliput_documentation
[ ] maliput_infrastructure
[ ] maliput
[ ] maliput_py
...
...


feat: add reporter-solid and docs site

Codecov Report

Merging #34 (be58b37) into main (03940d9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #34   +/-   ##
=======================================
  Coverage   99.33%   99.33%
=======================================
  Files          36       36
  Lines         907      907
  Branches      295      295
=======================================
  Hits          901      901
  Misses          5        5
  Partials        1        1

Continue to review full report at Codecov.

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


pull request!!

I'm submitting a ...
[ ] bug report
[ x] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project

Summary
please  manually review my pull request. i dont know why it failed i need to add the $arc Token

https://github.com/solana-labs/token-list/pull/19692 PLEASE AND THANK YOU!!!

Other information (e.g. detailed explanation, stack traces, related issues, suggestions how to fix, links for us to have context, eg. StackOverflow, personal fork, etc.)

duplicate token: token address AWLDWua47fVu5f8uxiCCLJPXBJAdsyRM11fESt8jRceo is already used
explanation: https://github.com/solana-labs/token-list#duplicate-token


FormatException when signing transaction after connecting using "Connect" button

I used the default walletButtonBuilder for Android which shows me a "Connect" button to connect my account on Pera Wallet. Then, I faced this error
Error: FormatException: Invalid character (at character 77)
I/flutter (10855): gqNzaWfEQIwtNFWFUDYnI49KtgVk5nGLCiEP9M/F7p4Zz2acj2FNEdZT3Dq56o+hKaSTYTzrU229

When I tried to sign the transaction in this function:
@override
  Future<String?> optInApp({
    required int appId,
  }) async {
    logger.i("Opting in to app: $appId");

    if (await optedIn(appId)) {
      logger.i("Already opted in to app: $appId");
      return null;
    }

    final sender =
        Address.fromAlgorandAddress(_connector.connector.session.accounts[0]);

    // Fetch the suggested transaction params
    final params = await _algorand.getSuggestedTransactionParams();

    // Build the transaction
    final transaction = await (ApplicationOptInTransactionBuilder()
          ..sender = sender
          ..noteText = 'Opt in to app $appId from $sender'
          ..applicationId = appId
          ..suggestedParams = params)
        .build();

    logger.i("Tx created: $transaction");

    try {
      // Sign the transaction
      final txBytes = Encoder.encodeMessagePack(transaction.toMessagePack());
      logger.i("Tx bytes: $txBytes");

      final signedBytes = await _provider.signTransaction(
        txBytes,
        params: {
          'message': 'Opt in to app $appId from $sender',
        },
      );

      logger.i("Tx signed: $signedBytes");

      // Broadcast the transaction
      final txId = await _algorand.sendRawTransactions(
        signedBytes,
        waitForConfirmation: true,
      );
      return txId;
    } catch (e) {
      final err = _tryCast(e, fallback: AlgorandException);
      if (err is AlgorandException) {
        AlgorandException error = err as AlgorandException;
        debugPrint('Error: ${error.message}');
      } else {
        debugPrint('Error: $e');
      }
      return null;
    }
  }

I tried to debug it and found that it happened when I tried to sign the transaction
      final txBytes = Encoder.encodeMessagePack(transaction.toMessagePack());
      final signedBytes = await _provider.signTransaction(
        txBytes,
        params: {
          'message': 'Opt in to app $appId from $sender',
        },
      );

My initialization code and the connect method in my AlgorandConnector class:
AlgorandConnector() {
    _connector = WalletConnectQrCodeModal(
      connector: WalletConnect(
        bridge: 'https://bridge.walletconnect.org',
        clientMeta: const PeerMeta(
          // <-- Meta data of your app appearing in the wallet when connecting
          name: 'Shark Tank DApp',
          description: 'Shark Tank DApp',
          url: 'https://walletconnect.org',
          icons: [
            'https://gblobscdn.gitbook.com/spaces%2F-LJJeCjcLrr53DcT1Ml7%2Favatar.png?alt=media'
          ],
        ),
      ),
    );

    _provider = AlgorandWalletConnectProvider(_connector.connector);
  }

  T? _tryCast<T>(dynamic value, {T? fallback}) {
    try {
      return (value as T);
    } on TypeError catch (_) {
      return fallback;
    }
  }

  @override
  Future<SessionStatus?> connect(BuildContext context) async {
    return await _connector.connect(context, chainId: 4160);
  }

It used to work perfectly fine when I tested it a few days ago. I only faced this error today. It also happened when I tested on a different device. Besides, if I connect to the Dapp using the QR Code, everything works perfectly fine. But I am working on a mobile application so I need the Deep Linking method to work.
This is the screenshot of the error

It occurs when I sign the transaction. I had a few loggers set, I could get the loggers printed until the txBytes, signedBytes was not printed.

I see, so an issue with communicating with Algorand network. From your code it seems that you are using https://github.com/RootSoft/algorand-dart package? Have you tried following the flow in the example?
// Fetch the suggested transaction params
final params = await algorand.getSuggestedTransactionParams();

// Build the transaction
final transaction = await (PaymentTransactionBuilder()
    ..sender = account.address
    ..note = 'Hi from Flutter'
    ..amount = Algo.toMicroAlgos(5)
    ..receiver = recipient
    ..suggestedParams = params)
  .build();

// Sign the transaction
final signedTx = await transaction.sign(account);

// Send the transaction
final txId = await algorand.sendTransaction(signedTx);

That flow requires an account, which I am not able to retrieve except asking for user's mnemonic, but I am wanting to make use of the wallet instead of managing my user's account

That code you are using was originally created by @RootSoft (https://github.com/RootSoft) for the walletconnect_dart package. He is also a creator of the algorand_dart package used here. The walletconnect_dart package hasn't been updated in a long time but there might have been some change in the Algorand network causing this.
Only thing which could have changed from WalletConnect side (or rather the accompanying app) is the account.address. I guess you can confirm that it is still the same as previously.
Other than that since the code is the same (unless you updated the algorand_dart package) there must have been some change in the Algorand node it's using. I would suggest talking to @RootSoft, he would be the best person to help here.

Issue with Algorand network/package


Else in if else cases is always executed

Description of the issue
Briefly describe the issue you are experiencing.
Type (put an x sign in the square brackets)

[x] bug report
[ ] feature request
[ ] Documentation improvement

Checklist

[x] Running latest version of code.
[x] This issue has not been reported earlier.

Your environment

macOS high sierra
go version go1.10 darwin/amd64
master branch

Expected behaviour
Consider the following 2 contract examples
func Main(operation string, args []interface{}) bool {
	if operation == "parrot" {
		repeatMe := args[0].(string)
		runtime.Notify(repeatMe)
	} else {
		runtime.Notify("Hello World")
	}

	return true
}

func Main(operation string, args []interface{}) bool {
	if operation == "parrot" {
		repeatMe := args[0].(string)
		runtime.Notify(repeatMe)
	}

	if operation != "parrot" {
		runtime.Notify("Hello World")
	}

	return true
}

both should run exactly the same:

return arg[0] if operation == "parrot""
return hello world if not

Actual behaviour
The second example runs as expected, but the first always returns hello world, even if the operation is "parrot"
Steps to reproduce

compile both contracts
testinvoke

@jeroenptrs There is a new branch development which should include a fix for this.

This is solved in latest release


UAVCAN Parameter Management & Enumeration

@bendyer I've put in this PR for more formal code review and merging. It would be great if we could get the ROMFS-based firmware update in here as well.

@LorenzMeier ROMFS-based firmware update is in 65c7872. The implementation copies all firmware files from the ROMFS/uavcan/fw/… directory into the appropriate location on the SD card, replacing previous firmware images unpacked from ROMFS, but not replacing any files the user has put there manually.
The firmware updater then runs as normal from the SD card.
This approach ensures that the user's firmware always overrides firmware in ROMFS, and also makes it easier for the user to see which firmware they're currently using when copying files to the SD card.

Awesome! Could you write a few sentences here or point me to a page detailing how to build and copy the ESC Firmware (the "naive" variant with the normal throttle response) for the SV ESC and for the Pixhawk ESC into the ROMFS? I'll probably go as far as integrating it in Travis and just ship the default master build with it.

Awesome Pavel!

@bendyer Many thanks! Now we're missing only documentation how to get Firmware in and run stuff. A PR against https://github.com/PX4/Devguide which renders to http://dev.px4.io would be fantastic!

@LorenzMeier will do.


"Unknown prisma-client function" for 2+ level nested fields

NOTE: Basically the same problem as https://github.com/prisma/nexus-prisma/issues/157, which should have been fixed with nexus-prisma 0.3.4
Describe the bug
I am basically experiencing the same as https://github.com/prisma/nexus-prisma/issues/157, but I am using MySQL 5.7. I have updated to nexus-prisma 0.3.4 and am using Prisma 1.28.0. Basically when I do Queries or Mutations that are two or more levels deep, I get an "unknown prisma-client function" error.
To reproduce
Example mutation:
mutation {
  createContentDetail(
    data: {
      name: "contentDetailName"
      subTitle: "contentDetailSubTitle"
      description: "contentDetailDescription"
      dates: {
        create: { timestampEnd:<PHONE_NUMBER>, timestampStart:<PHONE_NUMBER> }
      }
      place: {
        create: {
          name: "placeName"
          coordinates: { create: { longitude: 9.971824, latitude: 53.545086 } }
          address: {
            create: {
              street: "testStreet"
              postalcode: "testPostalcode"
              city: "testCity"
            }
          }
          phone: "placePhone"
          email: "placeEmail"
          website: "placeWebsite"
          logo: "placeLogo"
          imageMain: "placeImageMain"
          images: { set: ["image1", "image2"] }
          category: { create: { name: "catName" } }
        }
      }
    }
  ) {
    id
    name
    subTitle
    description
    dates {
      timestampEnd
      timestampStart
    }
    place {
      name
      coordinates {
        latitude
        longitude
      }
      address {
        street
        postalcode
        city
      }
      phone
      email
      website
      logo
      imageMain
      images
      category {
        id
        name
      }
    }
  }
}

Prisma datamodel relevant parts:
type ContentDetail {
  id: ID! @unique
  name: String!
  subTitle: String!
  description: String!
  dates: [Date!]!
  place: [Place!]!
}

type Place {
  name: String!
  coordinates: Coordinates!
  address: Address!
  phone: String!
  email: String!
  website: String!
  logo: String!
  imageMain: String!
  images: [String]!
  category: Category!
}

type Coordinates {
  latitude: Float! @default(value: "0.0")
  longitude: Float! @default(value: "0.0")
}

And it throws this error, together with a part-response:
{
  "errors": [
    {
      "message": "Unknown prisma-client function for field Place.coordinates",
      "locations": [
        {
          "line": 13,
          "column": 7
        }
      ],
      "path": [
        "createContentDetail",
        "place",
        0,
        "coordinates"
      ],
      "extensions": {
        "code": "INTERNAL_SERVER_ERROR",
        "exception": {
          "stacktrace": [
            "Error: Unknown prisma-client function for field Place.coordinates",
            "    at Object.throwIfUnknownClientFunction (/var/task/node_modules/nexus-prisma/dist/throw.js:16:15)",
            "    at /var/task/node_modules/nexus-prisma/dist/resolver.js:72:17",
            "    at field.resolve (/var/task/node_modules/graphql-extensions/dist/index.js:128:26)",
            "    at resolveFieldValueOrError (/var/task/node_modules/graphql/execution/execute.js:479:18)",
            "    at resolveField (/var/task/node_modules/graphql/execution/execute.js:446:16)",
            "    at executeFields (/var/task/node_modules/graphql/execution/execute.js:293:18)",
            "    at collectAndExecuteSubfields (/var/task/node_modules/graphql/execution/execute.js:717:10)",
            "    at completeObjectValue (/var/task/node_modules/graphql/execution/execute.js:707:10)",
            "    at completeValue (/var/task/node_modules/graphql/execution/execute.js:598:12)",
            "    at completeValue (/var/task/node_modules/graphql/execution/execute.js:565:21)"
          ]
        }
      }
    }
  ],
  "data": {
    "createContentDetail": {
      "id": "cjsyk75hl0igu0930cmzgwc44",
      "name": "contentDetailName",
      "subTitle": "contentDetailSubTitle",
      "description": "contentDetailDescription",
      "dates": [
        {
          "timestampEnd":<PHONE_NUMBER>,
          "timestampStart":<PHONE_NUMBER>
        }
      ],
      "place": null
    }
  }
}

Expected behavior
No "unknown prisma-client function" errors
Additional context
Since this issue is very similar to https://github.com/prisma/nexus-prisma/issues/157 and I am just using a different database, hopefully it can be fixed or I'll find some workaround until then.

Update: it works if I give Place an id: ID! @unique, so that there are no more 2-level nested fields, but I don't think this should be necessary

Hey there,
This seems like a limitation of the current prisma-client. At the moment, the scenario mentioned would only be possible using fragments. Because you're not providing ids on two level, it's not possible to find the coordinates associated to a place without querying everything from the parent
The workaround I'm gonna propose to you is not ideal at all, but unfortunately this seems like the only way until this lands: https://github.com/prisma/rfcs/blob/new-ts-client-rfc/text/0000-new-ts-client.md
On a side note: We should probably throw a custom error message for these cases, although it doesn't seem trivial to detect
Step 1
Add a new type definition for the GraphQL type Place:
// types.ts
import { Place as PrismaPlace, Coordinates } from './path-to-your-prisma-client'

export type Place = PrismaPlace & { coordinates: Coordinates }

Step 2
Update makePrismaSchema to use that new type definition
// schema.ts

const schema = makePrismaSchema({
    ...
    typegenAutoConfig: {
      sources: [
        {
          source: path.join(__dirname, './types.ts'),
          alias: 'types',
        },
      /*{
        source: path.join(__dirname, './context.ts'),
        alias: 'ctx',
      },*/
    ],
    /*contextType: 'ctx.Context',*/
  },
})

Step 3
Override the ContentDetail type:
// ContentDetail.ts

export const ContentDetail = prismaObjectType({
  name: 'ContentDetail',
  definition(t) {
    t.prismaFields(['*'])

    t.field('place', {
      ...t.prismaType.place,
      resolve(root, args, ctx) {
        return ctx.prisma.contentDetail({ id: root.id }).place() // Query all fields and the coordinates
          .$fragment(`{
            name
            phone
            email
            website
            logo
            imageMain
            coordinates {
              latitude
              longitude
            }
        }`)
      },
    })
  },
})

Step 4
Override the Place.coordinates field
// Place.ts

export const Place = prismaObjectType({
  name: 'Place',
  definition(t) {
    t.prismaFields(['*'])

    t.field('coordinates', {
      ...t.prismaType.coordinates,
      resolve(root, args, ctx) {
        return root.coordinates // Return the coordinates passed from the parent
      },
    })
  },
})

Update
id fields are gonna be mandatory very soon anyway as part of the datamodel v1.1. So I suggest you just use an id field and ignore my answer above 🙂

Thank you for both answers! I went with the id field route, but it's interesting to see how this could have been worked around aswell.

Awesome, closing now 🙏

@Weakky could this be reopened, please?
This is still a problem on embedded types (mongodb)
Here is a simple example (not working):

type Item {
    id: ID! @id
    subitem: SubItem
}
type SubItem @embedded {
    subSubItem: SubSubItem
}
type SubSubItem @embedded {
    withEnum: SubSubItemEnum
}
enum SubSubItemEnum {
    NEW
    OLD
}

Simple insert:
mutation {
  createItem(data: { subitem: { create: { subSubItem: {create: {withEnum: NEW }} } } }) {
    id
    subitem {
      subSubItem {
      	withEnum
      }
    }
  }
}

=> "message": "Unknown prisma-client function for field SubItem.subSubItem",
With an Id for all embedded fields (which is looking strange)

type Item {
    id: ID! @id
    subitem: SubItem
}
type SubItem @embedded {
    id: ID! @id
    subSubItem: SubSubItem
}
type SubSubItem @embedded {
    id: ID! @id
    withEnum: SubSubItemEnum
}
enum SubSubItemEnum {
    NEW
    OLD
}

=> "message": "Unknown prisma-client function for field SubItem.subSubItem",
with @unique it tells me
  SubItem
    ✖ The field `id` is marked as unique but its type `SubItem` is embedded. This is disallowed.

Hey there,
This issue is related to two things:

You probably did not add databaseType: document to your datamodel.prisma file https://www.prisma.io/docs/prisma-cli-and-configuration/prisma-yml-5cy7/#databasetype-optional. This is needed for nexus-prisma to know about embedded types
https://github.com/prisma/prisma/issues/4443. There's currently an issue with the client not properly fetching the embedded fields as expected.

Once the two are fixed, things should work properly again 🙏

OK thanks.
It's working with databaseType: documentwith and without Id.

Hello,
I have a similar issue but with a relational model, using PostgreSQL.
The model may be a bit tricky and not done as it should be :innocent: (feel free to criticize)
Using prisma, I am able to save and fetch data.
Using GraphQL shield I cannot retrieve the nested fields.
Am I forgetting something obvious like enabling some types to be queried?
I tried adding a MyObject.ts, SubOjbect.ts and KeyValue.ts to the resolvers dir but without success. (note: I don't have a Javascript background, I know Scala and Python)
The example my code is based on: prisma-examples/typescript/graphql-auth
query {
  roots(last: 10) {
    id
    my_object {
      id
      field0 {
        my_dict { key value }
      }
      field1 {
        my_dict { key value }
      }
    }
  }
}

"message": "Unknown prisma-client function for field MyObject.field0"
"message": "Unknown prisma-client function for field MyObject.field1"

With the following datamodel:
type Root {
  id: ID! @id
  my_object: MyObject
}

type MyObject {
  id: ID! @id
  field0: [SubOjbect] @relation(name: "F0Relation")
  field1: [SubOjbect] @relation(name: "F1Relation")
}

SubOjbect {
  id: ID! @id!
  my_object_f0: MyObject @relation(name: "F0Relation")
  my_object_f1: MyObject @relation(name: "F1Relation")
  my_dict: [KeyValue] @scalarList(strategy: RELATION)
}

type KeyValue {
  id: ID! @id
  key: String
  value: String
}

I use basic permissions in permissions/index.ts:
import { rule, shield } from 'graphql-shield'
import { getUserId } from '../utils'

const rules = {
  isAuthenticatedUser: rule()((parent, args, context) => {
    const userId = getUserId(context)
    return Boolean(userId)
  }),
}

export const permissions = shield({
  Query: {
    me: rules.isAuthenticatedUser,
    root: rules.isAuthenticatedUser,
    roots: rules.isAuthenticatedUser,
  }
}, { debug: true }
)

Sorry for the long post, I hope it is not too detailed but as detailed as possible :smile:

Actually I ended up starting this at the mongo post, I will investigate previous post, sorry my message may be useless :disappointed:

Well actually I am not managing to make it work.
Below are failed attempts:
import { prismaObjectType } from 'nexus-prisma'

export const MyObject = prismaObjectType({
  name: 'MyObject',
  definition(t) {
      t.prismaFields([ 'id' ])
      t.list.field('field0', {
        type: 'SubOjbect',
        resolve(root) {
          return { field0: root.field0 }
        }
      })
    },
})

import { prismaObjectType } from 'nexus-prisma'

export const MyObject = prismaObjectType({
  name: 'MyObject',
  definition(t) {
      t.prismaFields([ 'id' ])
      t.list.field('field0', {
        type: 'SubOjbect',
        resolve(root) {
          return root.field0
        }
      })
    },
})

type MyObject {
  id: ID! @id
  field0: [SubOjbect]! @relation(name: "F0Relation")
  field1: [SubOjbect]! @relation(name: "F1Relation")
}

what about when trying to query the count aggregation?
if you try to do
categoriesConnection {
    aggregate {
      count
    }
  }

it will give the same error. In this case there is no id field to query, the only field is count. Is there any workaround in this case?


[Improvement-5440][Maven] Project module maven dependency optimization

Purpose of the pull request
this close #5440
Project module maven dependency optimization
dependency relationship before：

dependency relationship compared：(The red dotted line represents deletion)

dependency relationship after：

Verify this pull request
This change added tests and can be verified as follows:

Manually verified the change by testing locally. -->

Codecov Report

Merging #5441 (283b09f) into dev (d93a9dc) will increase coverage by 1.47%.
The diff coverage is 56.73%.

@@             Coverage Diff              @@
##                dev    #5441      +/-   ##
============================================
+ Coverage     45.21%   46.69%   +1.47%
- Complexity     3395     3777     +382
============================================
  Files           538      605      +67
  Lines         23053    24797    +1744
  Branches       2687     2813     +126
============================================
+ Hits          10423    11578    +1155
- Misses        11698    12114     +416
- Partials        932     1105     +173

Impacted Files
Coverage Δ
Complexity Δ

...inscheduler/plugin/alert/email/EmailConstants.java
87.50% <ø> (ø)
1.00 <0.00> (ø)

...eduler/plugin/alert/script/ScriptAlertChannel.java
16.66% <0.00%> (ø)
1.00 <0.00> (ø)

...org/apache/dolphinscheduler/alert/AlertServer.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...che/dolphinscheduler/api/ApiApplicationServer.java
33.33% <0.00%> (ø)
1.00 <0.00> (ø)

...cheduler/api/controller/AccessTokenController.java
6.66% <ø> (-1.67%)
1.00 <0.00> (-1.00)

...scheduler/api/controller/AlertGroupController.java
8.00% <ø> (+2.73%)
2.00 <0.00> (ø)

.../api/controller/AlertPluginInstanceController.java
7.69% <ø> (+1.97%)
2.00 <0.00> (ø)

...heduler/api/controller/DataAnalysisController.java
9.09% <ø> (ø)
1.00 <0.00> (-1.00)

...scheduler/api/controller/DataSourceController.java
4.16% <0.00%> (-0.49%)
1.00 <0.00> (-1.00)

...inscheduler/api/controller/ExecutorController.java
8.33% <ø> (-2.78%)
1.00 <0.00> (-1.00)

... and 378 more

Continue to review full report at Codecov.

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


WANT TO SEE HOT PICS WITH ME? 💋 💋 💋

WANT TO SEE HOT PICS WITH ME? 💋 💋 💋
👇 👇 👇
https://tinyurl.com/srmjb2y

I think that can be closed 👎


  remaps not working

Self-Diagnosis
Before creating an issue, take some time to search these resources for an answer. It's possible that someone else has already seen and solved your issue.

old NERDTree issues
NERDTree documentation - :h NERDTree
NERDTree Wiki
Other resources: https://stackoverflow.com, https://vi.stackexchange.com, etc.

Environment

Provide a minimal .vimrc file that will reproduce the issue.

I have the following in my vimrc that I frequently use for navigation.
nnoremap <c-j> <c-d>
nnoremap <c-k> <c-u>
vnoremap <c-j> <c-d>
vnoremap <c-k> <c-u>

Steps to Reproduce the Issue

Open nerdtree window
<c-j> and <c-k> do not work as intended based on my vimrc maps.

Current Behavior (Include screenshots where appropriate.)
<c-j> and <c-k> move up and down one line
Expected Result
<c-j> and <c-k> move up and down half a page

Both <c-j> and <c-k> already have their own behavior - moving up and down in a directory - You should remap them to something else first.
use :help NERDTree-C-J to open the help section for this specific mapping, And use :help NERDTreeMappings to open the list of all NERDTree mappings.

This is perfect.
Thank you!


Testing plan for location verification

What type of PR is this?

tests

What this PR does / why we need it:
Example of testing plan aligned with the guidelines being proposed in https://github.com/camaraproject/Commonalities/issues/158.
Which issue(s) this PR fixes:
Fixes #208
Special notes for reviewers:
There are many comments embedded in the test plan to clarify and explain some scenarios. In some cases alternate syntax for the same functionality is used to illustrate the possibilities.
Some points are still open to discussion, such as the degree of specification of the testing plan.

Checked with @patrice-conil and work for Orange side. Could be very helpful to provide an extract from that to answer camaraproject/Commonalities#127 in commonalities.

Yes, I have to think how to align this with the work in Commonalities and the testing guidelines enhancement. I will be preparing some more detailed proposal towards the dedicated workshop.

@jlurien Accordingly to your proposal in Commonalities, this file name should location-verification.feature I guess.

True, I have to update and align my own example with my own proposal :)

@bigludo7 Wee need another explicit approval after the last fixes to merge the PR. Thanks


FilamentAsset registered components not preserved between requests when using Octane

Package
filament/filament
Package Version
v3.0.0-beta10
Laravel Version
v10.16.0
Livewire Version
v3.0.0-beta.4
PHP Version
PHP 8.1
Problem description
When using Octane and adding Select form element, page loads correctly only once.
After that error is given:
Alpine component with ID [select] not found for package [filament/forms].
Here's link to log output: https://flareapp.io/share/v5pBKG0P
Error occurs on this line: https://github.com/filamentphp/filament/blob/3e6bf1c7d8667b788d3bcae1aa1bc2a15b02e5c6/packages/forms/resources/views/components/select.blade.php#L74
Expected behavior
Page loads successfully every time.
Steps to reproduce

Install Laravel
Install Sail + Octane
Install Filament
Create form with Select component
Visit page with that form several times

Example repository has simple example with  autologin. Run / check launch.sh for instructions.
Reproduction repository
https://github.com/saulens22/laravel-playground/tree/filament-v3-roadrunner
Relevant log output
No response

Forgot to mention - I've already tried flushing some of the bindings, as per https://github.com/filamentphp/filament/pull/2055
https://github.com/saulens22/laravel-playground/blob/94eed8e4423441c4c0a3447bb1f980c4ff99113c/config/octane.php#L138-L150

Duplicate of #7427. Please test the fix #7552

@danharrin I've tested the fix and found another problem - Livewire is not getting injected.
There is a fix in Livewire v3.0.0-beta.8 (link to fix) but filament/support got locked to v3.0.0-beta.7 in this commit: https://github.com/filamentphp/filament/commit/6bb89885c819d0e625feb3eb32260fac8a224e16

We can upgrade to Livewire once they make a new release, they severely broke something else in the last release, causing us to lock the version


No results returned from Long Select condition.

Currently pulling my beard hair out about this issue, cant seem to see what I am doing wrong, I am querying the database against two parameters, a String and Long, and cannot seem to get any results back from the Select:
Select where = Select.from(DataPoint.class).where(Condition.prop("time").eq((Long) timeVal), Condition.prop("scene").eq(sUrl)).limit("1");
Tried doing the same call with a inline select ("time = ? and scene = ?) and get the same results. Any ideas where I am going wrong?

Time is probably a date field ..
Am 05.03.2015 18:39 schrieb "ScottMellors"<EMAIL_ADDRESS>
Currently pulling my beard hair out about this issue, cant seem to see
what I am doing wrong, I am querying the database against two parameters, a
String and Long, and cannot seem to get any results back from the Select:
Select where =
Select.from(DataPoint.class).where(Condition.prop("time").eq((Long)
timeVal), Condition.prop("scene").eq(sUrl)).limit("1");
Tried doing the same call with a inline select ("time = ? and scene = ?)
and get the same results. Any ideas where I am going wrong?
—
Reply to this email directly or view it on GitHub
https://github.com/satyan/sugar/issues/275.

Should have mentioned my Class has this set as a long, unless another field is created automatically called time which would be unfortunate and is getting confused,


Improve tags/search

Hello!
Great project you have here!
However, the search is far from ideal right now. When you enter «letter» in the search field it shows nothing. However, when you enter «mail», there are a lot of letter icons.
Maybe you should add tag synonyms and provide users with ability to suggest additional tags?
Thank you!

All fontello fonts are published here https://github.com/fontello in separate repos. Aliases can be updated by editing config.yml file in each repo root. If you know missed aliases - do PR.


Fastest way of converting between OpenCV Mat and Rust ndarray (or other Rust arrays)? Or even better, zero-copy?

Hi thanks for the lib! I wonder what is the fastest way of converting between OpenCV Mat and Rust ndarray (or other Rust arrays)? Obviously I can iterate each element and assign the value, but it will be terribly slow. I guess there should be something like memcpy? Or even better, can we have zero-copy when converting between Mat and ndarray?

@twistedfall Hi I think maybe it can be safe sometimes? Pseudocode:
let input_rust = Vec::new(10000000); // or other linear algebra lib
let input_cv = cv::Mat::from_the_memory(input_rust); // I mean this -> https://stackoverflow.com/a/23249319/4619958 . Maybe we can make it **mutable borrow**?

let output_rust = Vec::new(2000000);
let output_cv = cv::Mat::from_the_memory(output_rust);

cv::resize(input_cv, output_cv, ...);

drop(input_cv);
drop(output_cv);

then use input_rust, output_rust happily!

doc: https://docs.rs/opencv/0.54.0/opencv/imgproc/fn.resize.html

Ok... Thank you all the same!

let (mut data,_, _) = pixels.into_raw_parts();  // pixels will be move into_raw_parts，and return a manually drop pointer.
let img = opencv::core::Mat::new_rows_cols_with_data(
    height as i32,
    width as i32,
    opencv::core::CV_8UC4,
    data as *mut c_void,
    opencv::core::Mat_AUTO_STEP).unwrap();


replaces rpoplpush in redis backend for lmove

rpoplpush is deprecated by redis as of 6.2.0. current redis version is 7.0.5
fixes #93
blocked by https://github.com/jgaskins/redis/issues/37
blocked by #106

resolved in #112


Redist packages each include all architectures and is quite large as a result

Describe the bug
The redist packages all appear to be 3x the size they need to be. Looking inside the x64 one, for example, I see 20mb of Arm64 binary, 22mb of x64, and 20mb of x86. There's a separate Arm64 redist, and it has the same.

These were downloaded from the official redist zip referenced on mslearn.
https://learn.microsoft.com/en-us/windows/apps/windows-app-sdk/downloads
Appears to be the same thing regardless of version or if zipped or standalone.
Steps to reproduce the bug
Go to the link above.
Download the zip
Unpack the installers and peek inside.
Expected behavior
Only include binaries for the target architecture.
Screenshots
No response
NuGet package version
None
Packaging type
Unpackaged
Windows version
Windows 11 version 22H2 (22621, 2022 Update)
IDE
No response
Additional context
No response

I ran across this, BTW, because I was trying to figure out exactly which files in my unpackaged app's publish folder were redundant with what the runtime installers put out. There's no list that I've seen.

One of the biggest difficulties here is answering the question of how complex the installer should be.
The Main and Singleton packages are the biggest issues here.

From the contents, you can see that they contain the DDLM data store and the push notifications long running task. The data store is used for the Windows App SDK's dynamic dependencies API, so it is potentially possible that it is also used by the bootstrapper. The push notifications long running task is also used by app notifications.
The reason why this is difficult is:

on my system, I have the x64 versions of these installed. I could go through the effort of fully showing the list, but basically the x64 versions are the only ones installed. You don't have to trust me, you can verify yourself. However, if I try to install the x86 versions of the .msix directly:

it sees them as already installed. This means that only one of these package will be installed at any time.
If you separate everything by platform, there is going to be one problematic situation that will occur. The question that would need to be answered is what would happen if you run the redistributable for a non native architecture on a system. As an example, if you run the x86 redistributable on an Arm64 system without a previous Win App SDK install, should it install the singleton and main packages? Should it refuse and tell the person running the redistributable that they also need to install the Arm64 version?

So if you download the x86 installer and run it on a clean x64 machine, does it say "Oh, this is an x64 PC, we'll install that version instead"? If so, we should have just one redist and that's it. I know that was part of the question in your closing comment, but I can't think of any reason why we'd need three different redist packages if they detect Windows architecture and always install the "correct" one for Windows, regardless of app architecture.
It could also be approached like VC runtime. x86 install is only for x86. x64 install includes Arm64 and Intel x64. I'm still not sure exactly which files are in these, but this can also be handled the way WinRT SDKs are, with Arm64X binaries on Arm devices.
Also, the redist packages have no uninstall switch, which is unfortunate and makes testing this a chore.

So if you download the x86 installer and run it on a clean x64 machine, does it say "Oh, this is an x64 PC, we'll install that version instead"? If so, we should have just one redist and that's it. I know that was part of the question in your closing comment, but I can't think of any reason why we'd need three different redist packages if they detect Windows architecture and always install the "correct" one for Windows, regardless of app architecture.

"I have an application that requires the 64 bit version of this library, but I can only find the 32 bit installer. Where is the 64 bit installer?" Since for it to be truly universal, it would have to be a 32 bit installer.

It could also be approached like VC runtime. x86 install is only for x86. x64 install includes Arm64 and Intel x64. I'm still not sure exactly which files are in these, but this can also be handled the way WinRT SDKs are, with Arm64X binaries on Arm devices.

Do appx/msix packages allow cross architecture references? While I can see this working for self contained setups, how does this work when you are referencing the Windows App Runtime package? Heck, you don't even get a choice if you are using the packaged application dependencies or MddBootstrapInitialize2.


New cop: Detect unexpected overwriting of Struct methods

Is your feature request related to a problem? Please describe.
I've encountered an weird behavior on using Struct class:
# good
Good = Struct.new(:foo)
Good.new.members #=> [:foo]

# bad
Bad = Struct.new(:members)
Bad.new.members #=> nil (I've expected `[:members]`)

This is caused by unexpectedly overwriting the Struct#members method.
Describe the solution you'd like
Add a new cop to detect unexpected overwriting such as the example above.
Describe alternatives you've considered
None.
Additional context
Struct includes Enumerable and has many methods e.g. length, count, min, max, or sum, so it seems that some people might use such methods as a Struct member unexpectedly.

I've thought about this issue for a while, but now I feel it hard that we draw the line on what methods are detected by this cop.
For example, please assume that someone writes the following code:
Commit = Struct.new(:id, :hash)
a_commit = Commit.new(123, "9a4f0ee")
a_commit.hash #=> "9a4f0ee". Not an integer!

The code above violates the Object#hash method's contract (that is, breaks the Object#hash expected behavior), so it can produce bugs somewhere in the program. However, I think that it requires quite experience for the Ruby programming to find such a bug possibility.
And, I don't think that almost programmers write  Struct.new(:id, :hash) to override Object#hash. Perhaps people who want to override it may write like this:
Commit = Struct.new(:id) do
  def hash
    # own calculation of a hash...
  end
end

Therefore, I think that it's valuable to check all the methods including Object, BasicObject, and Enumerable.
Also, I think that it's a discussion-required and hard task to pick up what methods we can allow overriding. Every time a method would be added to Struct or Object, we would need to discuss whether we should update the pick-up list or not.
What do you think about this opinion?

FYI, here is another interesting example (from this API doc):
Customer = Struct.new(:name, :address, :zip)
joe = Customer.new("Joe Smith", "123 Maple, Anytown NC", 12345)
joe.zip #=> 12345

AnotherCustomer = Struct.new(:name, :address)
tom = AnotherCustomer.new("Tom Smith", "123 Maple, Anytown NC")
tom.zip #=> [["Tom Smith"], ["123 Maple, Anytown NC"]]

Enumerable#zip is overriden! 😳

I've opened the PR #7699, but I think the discussion is open yet. 😃
I hope the PR could promote the discussion about https://github.com/rubocop-hq/rubocop/issues/7680#issuecomment-580596916 or https://github.com/rubocop-hq/rubocop/issues/7680#issuecomment-583751130!

@ybiquitous Thanks for the great explanation! It is really helpful to understand the problem.
I had the same concern with @Drenmi. It is below.

Do we also lint regular classes that define methods also to check whether they override built-in methods?

But now I think we don't need to lint regular classes. Because of the @ybiquitous's comment.

And, I don't think that almost programmers write Struct.new(:id, :hash) to override Object#hash. Perhaps people who want to override it may write like this:

def hash() end explicitly re-defines the hash method, but Struct.new(:id, :hash) is implicit.
So I think the cop will be valuable.

Also, I think that it's a discussion-required and hard task to pick up what methods we can allow overriding. Every time a method would be added to Struct or Object, we would need to discuss whether we should update the pick-up list or not.

Yes, I agree that picking up methods is a hard task. And I guess anyone can't choose proper methods, because the proper methods depend on cases.
So I propose we make the cop to add an offense for all overrides. But we can decrease the severity of the cop, or make the message milder.
The offense message is "Disallow overriding the Struct#%<method_name>s method." in the current implementation, but I feel it's too strong because we can override method by attribute properly, like the zip case.
What about ":hash attribute overrides Object#hash and it may be unexpected"?

@pocke Thanks for your feedback!

we can decrease the severity of the cop, or make the message milder.

I agree completely! 👍

What about ":hash attribute overrides Object#hash and it may be unexpected"?

Looks good! I will try to change the PR. 👍


Could this plugin support adding the Bangumi website to improve anime metadata scraping?

Thank you for writing such a plugin, which helps us users who cannot connect to tvdb, but I found that some websites like Douban do not support some anime information well, so I would like to ask if this plugin can support adding the Bangumi website to improve anime metadata scraping?
Thank you very much!

其实一开始就想加的，后来嫌麻烦放弃了。
昨天又看了下原来bangumi有官方API... 那就好办了，最新版本加上了哦 1.3.7