question: how do I reference github:NixOS/nixpkgs/nixos-23.11 in Flakehub?

When using a standard Github flake reference, I can use the nixos-23.11 tag:
{
  inputs = {
    nixpkgs.url = "github:NixOS/nixpkgs/nixos-23.11";
  };
}

However, nix flake init --template "github:DeterminateSystems/zero-to-nix#go-dev" produces this:
{
  inputs = {
    nixpkgs.url = "https://flakehub.com/f/NixOS/nixpkgs/0.2305.491812.tar.gz";
  };
}

I don't know what version that is, I have to look at the flake.lock to find that it's 5550a85a087c04ddcace7f892b0bdc9d8bb080c8 and I can't find a way to view the information in the Flakehub website. Playing with the URL like this: https://flakehub.com/flake/NixOS/nixpkgs/0.2305.491812 just gives me a 404.
For a few months, I've been replacing the flakehub link with github:NixOS/nixpkgs/nixos-23.11, because I don't know how to use the tag I want with flakehub.
So I thought that other people are probably doing the same.
Does flakehub support the use of nixos-23.11 style tags or is that not the concept? How should users think about it?
Thanks!

Playing with the URL like this: https://flakehub.com/flake/NixOS/nixpkgs/0.2305.491812 just gives me a 404.

That's because we have a hidden git revision in the URL.

Does flakehub support the use of nixos-23.11 style tags or is that not the concept? How should users think about it?

Yes it does -- 0.2305.XXXXX is basically nixos-23.05. So, to get nixos-23.11, you'd use 0.2311. You can see a list of all releases here: https://flakehub.com/flake/NixOS/nixpkgs/releases

Thanks, that makes sense.
I noticed the releases, but there's 400+ of them. I scrolled through lots of pages, but didn't see anything that looked like 23.11 on its own.
nixpkgs has tags of both 23.11 and nixos-23.11. 23.11 hasn't been updated for 4 months, https://github.com/NixOS/nixpkgs/tree/23.11 while nixos-23.11 gets security updates and bugfixes, so I was also concerned I'd end up with 23.11 rather than nixos-23.11.
From reading the semantic version docs, I think I want: https://api.flakehub.com/f/NixOS/nixpkgs/~0.2311.tar.gz although, I did expect it to be 23.11.
But... the UI doesn't provide a human readable version at any of the following URLs for me to check it:

https://flakehub.com/NixOS/nixpkgs/~0.2311
https://flakehub.com/flake/NixOS/nixpkgs/~0.2311
https://flakehub.com/flake/NixOS/nixpkgs/23.11
https://flakehub.com/flake/NixOS/nixpkgs/0.2311
https://flakehub.com/flake/NixOS/nixpkgs/0.2311.0+rev-56528ee42526794d413d6f244648aaee4a7b56c0
https://flakehub.com/flake/NixOS/nixpkgs/0.2311+rev-56528ee42526794d413d6f244648aaee4a7b56c0

Any pointer on the correct human URL?
Thanks!

We have special handling for Nixpkgs. When using 0.2311.0 (or 0.2311.1, or 0.2311.6500) as a flake input, it will use semver version resolution to get the latest version of nixos-23.11 (not the 23.11 tag) that is "compatible" (i.e. 0.2311.6500 cannot resolve to 0.2311.6499, but it could resolve to 0.2311.6501 or 0.2311.6502, or....).
The "human readable" version will be any of them with the fully specified version (i.e. the latest at the time of writing is https://flakehub.com/flake/NixOS/nixpkgs/0.2311.556903+rev-56528ee42526794d413d6f244648aaee4a7b56c0). The reason https://flakehub.com/flake/NixOS/nixpkgs/0.2311+rev-56528ee42526794d413d6f244648aaee4a7b56c0 wasn't working was because you were missing the exact "patch" version 556903.
The newest versions should be on the very first page at the top, but to get to the FlakeHub page of a specific version, you'd need to look at the flake.lock file and find the locked URL, and copy the version portion, i.e.:
        "url": "https://api.flakehub.com/f/pinned/nix-community/fenix/0.1.1727%2Brev-66fc1883c34c42df188b83272445aedb26bb64b5/018cc416-f0ae-724d-989f-aa4ded05d885/source.tar.gz"

would become
https://flakehub.com/flake/nix-community/fenix/0.1.1727%2Brev-66fc1883c34c42df188b83272445aedb26bb64b5


Feature/polkadart scale codec

Core Code

[x] Scale Codec Compact Functionality
[x] Scale Codec Encode / Decode
[x] Scale Codec Type Conversion

Testing:

[x] Testing Compact Functionality
[x] Testing Constants Encode / Decode
[x] Testing Events Encode / Decode
[x] Testing Extrinsic Encode / Decode

I think the smaller issues will improve the quality of this PR. Great job @justkawal 🚀

Thanks.
@leonardocustodio Waiting for your review....


Font with shadow

I want to create the texts without shadow, how can I do？Thanks a lot！

Try disabling it here.


changed two lines of readme

Just changed two lines of the README.md file.
These two lines are just a bit more explicit in specifying to users that they need to be running python3 instead of python 2.
That should be enough to let users know to use python 3. I'm sure you could also add a line in the README, saying something like "Program must be run in Python 3" but it's probably not necessary.

Yeah I suppose you could. That would probably be a bit cleaner actually,


Event Bridge & Event Grid

The AWS equivalent now states it's SQS, which was the best equivalent AWS had to offer a couple of months ago.
However, with the recent release of Amazon EventBridge (https://aws.amazon.com/eventbridge/) I think this new service fits as a better equivalent.

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

ID: 30d02f16-9621-d204-4132-8ce8db79b0af
Version Independent ID: 65b4385b-79f0-d842-d4a1-5d013615f9b1
Content: AWS to Azure services comparison - Azure Architecture Center
Content Source: docs/aws-professional/services.md
Service: architecture-center
Sub-service: cloud-fundamentals
GitHub Login: @JasRobe
Microsoft Alias: pnp

@Jandev Thanks for the feedback! We have assigned this issue to the content author to review and update accordingly.

AB#160228 - Thanks for reporting - this issue is under review

We've taken this to our backlog for a large update in the coming days. Thanks so much, closing for now.


Add doctest and setup file structure

A test/ directory should be created at the root. The file structure within this directory is as follows:

unit/ - unit tests
func/ - functional tests
runner.cpp - the test runner

runner.cpp should contain a main() entry point to run a specific suite of tests. For now, the main() should be empty since there aren't any tests to run.

Closed by #14


Problem updating crouton with Debian sid

And that is a problem because I recently updated ChromeOS and I need to update to start debian.
command and output:
chronos@localhost / $ sudo sh ~/Downloads/crouton -u -n sid
/usr/local/chroots/sid already exists; updating it...
Enter encryption passphrase for sid:
Preparing chroot environment...
Installing brightness into the chroot...
Installing croutonpowerd into the chroot...
Installing croutonversion into the chroot...
Installing host-dbus into the chroot...
Installing host-x11 into the chroot...
Installing volume into the chroot...
Installing pulseaudio-default.pa into the chroot...
Installing croutoncycle into the chroot...
Installing croutontriggerd into the chroot...
Installing croutonxinitrc-wrapper into the chroot...
Installing setres into the chroot...
Installing xinit into the chroot...
Installing xbindkeysrc.scm into the chroot...
Installing xorg-intel-sna.conf into the chroot...
Installing xserverrc into the chroot...
Installing xserverrc-xorg into the chroot...
Installing xserverrc-local.example into the chroot...
Installing startlxde into the host...
Installing crouton-noroot into the chroot...
Installing startlxde into the chroot...
Installing croutonclip into the chroot...
Installing croutonnotify into the chroot...
Installing croutonurlhandler into the chroot...
Installing enter-chroot into the host...
Installing delete-chroot into the host...
Installing edit-chroot into the host...
Installing mount-chroot into the host...
Installing unmount-chroot into the host...
Installing crash_reporter_wrapper into the host...
Installing target core...
Preparing environment...
Preparing software sources...
 Done
Ensuring system is up-to-date...
Reading package lists... Done
Building dependency tree
Reading state information... Done
Calculating upgrade... linux-libc-devThe following packages were automatically installed and are no longer required:
libhogweed2:i386 libjim0.75 libnettle4:i386 liboath0 libopenconnect3 libplist2 libvncclient0 libx264-142 libx264-142:i386 libx265-51:i386
libx265-51
Use 'apt-get autoremove' to remove them.
Done
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
Reading package lists... Done
Building dependency tree
Reading state information... Done
apt-transport-https is already the newest version.
ca-certificates is already the newest version.
sudo is already the newest version.
wget is already the newest version.
The following packages were automatically installed and are no longer required:
libhogweed2:i386 libjim0.75 libnettle4:i386 liboath0 libopenconnect3 libplist2 libvncclient0 libx264-142 libx264-142:i386 libx265-51:i386
libx265-51
Use 'apt-get autoremove' to remove them.
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
Syncing timezone...
Installing target audio...
Fetching CRAS (branch 103c37a8bf17ea65d590acd56f5d1a1778d27b1e)...
--2015-06-09 09:26:23--  https://chromium.googlesource.com/chromiumos/third_party/adhd/+archive/103c37a8bf17ea65d590acd56f5d1a1778d27b1e.tar.gz
Resolving chromium.googlesource.com (chromium.googlesource.com)... <IP_ADDRESS>
Connecting to chromium.googlesource.com (chromium.googlesource.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: unspecified [application/x-gzip]
Saving to: '/tmp/crouton-cras.eDFJl6/adhd.tar.gz'
 0K .......... .......... .......... .......... ..........  362K
50K .......... .......... .......... .......... ..........  583K

100K .......... .......... .......... .......... ..........  640K
150K .......... .......... .......... .......... ..........  613K
200K .......... .......... .......... .......... ..........  630K
250K .......... .......... .......... .......... ..........  610K
300K .......... .......... .......... .......... ..........  618K
350K .......... .......... .......... .......... ..........  610K
400K .......... .......... .......... .......... .....       624K=0.8s
2015-06-09 09:26:24 (571 KB/s) - '/tmp/crouton-cras.eDFJl6/adhd.tar.gz' saved [456003]
Reading package lists... Done
Building dependency tree
Reading state information... Done
alsa-utils is already the newest version.
libasound2:i386 is already the newest version.
libspeexdsp1:i386 is already the newest version.
The following packages were automatically installed and are no longer required:
libhogweed2:i386 libjim0.75 libnettle4:i386 liboath0 libopenconnect3 libplist2 libvncclient0 libx264-142 libx264-142:i386 libx265-51:i386
libx265-51
Use 'apt-get autoremove' to remove them.
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
Reading package lists... Done
Building dependency tree
Reading state information... Done
gcc is already the newest version.
gcc-multilib is already the newest version.
gcc-multilib set to manually installed.
pkg-config is already the newest version.
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:
The following packages have unmet dependencies:
libc6-dev:i386 : Depends: linux-libc-dev:i386 but it is not going to be installed
E: Unable to correct problems, you have held broken packages.
Failed to complete chroot setup.
Unmounting /mnt/stateful_partition/crouton/chroots/sid...

The linux-libc-dev package is currently broken in sid.  The immediate problem is that crouton needs to install the i386 version to compile CRAS, and the current i386 version (4.0.4-1) is incompatible with the highest-available amd64 version (4.0.2-1).
As a workaround, you can install the 4.0.2-1 linux-libc-dev:i386 package from snapshots and re-run the crouton upgrade.  To install the older package, enter the chroot and run:
wget 'http://snapshot.debian.org/archive/debian/20150512T034855Z/pool/main/l/linux/linux-libc-dev_4.0.2-1_i386.deb'
sudo dpkg -i linux-libc-dev_4.0.2-1_i386.deb

You may also consider switching to testing in the future, if possible, as sid does have broken package problems like this one from time to time.


Put media widgets for Form Builders

Will be great put to each Fields in a FormBuilder media widgets for give more css format to the form.
For now, we can put some of format, html's ouputs, with this way.
https://groups.google.com/forum/#!searchin/wagtail/form/wagtail/zs4vNjPAH3M/uuyHbLtd75cJ
But this way have many problems, the most important, how use place holders?
Well, here say us a way: https://groups.google.com/forum/#!searchin/wagtail/form/wagtail/Nx1gu4yirM4/e9Kax45v-vMJ; isn't very useful.
For place holder would be very useful put the help-text as placeholder or put the name as placeholder.
Well, then, put media for class, labels and a custom class for help-text, if we want don't use help-text as placerholder, oh, put a * or some class for required field; maybe, posibility to group some fields in a field sets.
And finally, i'm testing the form validator, but, the browser don't valide by default, How can we validate this form? I think that FormBuilder should put a requiered attribute in the input field if this field is required.
I'm searching about validation with frontend framework that i use, http://foundation.zurb.com/docs/components/abide.html, but, are there others ways for validate in wagtail?
I think that many people will be happy with this features. Thank you.

+1

Ok, i think know that you can use this:
     def __init__(self, *args, **kwargs):         super(AffiliatesForm, self).__init__(*args, **kwargs)         for field in self.fields:             self.fields[field].widget.attrs = {                 'required': True             }
But, putting a condition that if field is required add these attribute, for put dinamically.
I don't know if it's possible we do ourselves.
Thanks :D :+1:

Will be this feature implemented?

Any advance about this?

As part of the Wagtail’s first Kickstarter campaign, I’m working on this!
https://www.kickstarter.com/projects/noripyt/wagtails-first-hatch


[Feature]: Configuration for layout views

When working with custom admin template package, I am having different layout structure such as app and guest with sidebar, navbar, footer, main content section.
It would be great if you guys provide a list of publishable views config option in laravel_generator.php config file instead of hardcoding in LayoutPublishCommand.php file.
https://github.com/InfyOmLabs/laravel-generator/blob/c00a3eaf54a9b59e581772220145bb2199ad86d5/src/Commands/Publish/LayoutPublishCommand.php#L67
Thanks for the stunning package 😊

Related to https://github.com/InfyOmLabs/laravel-generator/issues/471

https://github.com/InfyOmLabs/laravel-generator/issues/471#issuecomment-1193604980
Check this. Now, LayoutPublishCommand has been removed. You can use customized views from the vendor directory and adjust the structure manually after the layout publishes via laravel-ui-adminlte.


Hierarchical Concurrent State Machine

Hi,
I have not worked with transitions so far but it looks like a really neat and pythonic approach to state machines. So in that matter its exactly what we need for a project :). My question is if transitions also supports nested and concurrent states. I know it is a bit contradicting to search something 'simple' but also requiring such complex features. But maybe I have just missed that in the docs or you may have some recommendations about how to realise hierarchical concurrent state machines with transitions.
Best regards,
Alex

Thanks for the question, @aleneum! Support for hierarchical states has been suggested before (#31); to recap: I think it would be nice, as long as the implementation can be made relatively compact. I'm happy to consider PRs or to discuss potential implementations, but probably won't have time to implement this myself any time soon, so it would fall to someone else to do it.
With respect to concurrent states, it depends on what you mean. If you mean support for being in multiple states at once, but still in a single-threaded context, I'm willing to consider something along those lines---though I'd need to be convinced that there's a real need for it. I guess my intuition is that most potential use cases could be reduced to either (i) making states hierarchical (so, this amounts to implementing hierarchies, as above), (ii) implementing multiple Machine instances, or (iii) refactoring the states so that you encode all possible combinations of desired states as discrete states.
On the other hand, if you're talking about concurrency in the sense of multithreading (see discussion in #30), then that's definitely not on the agenda, as I think it would be a beast to get right.
Since this has come up before, I'll mark it as help wanted and leave it open.

Thank you for your fast and detailed answer. I missed to check the closed issues. My thoughts went into the same direction as described in #31 and I would be happy to contribute (or at least give it a shot) if you consider this a promising road.
About concurrency: I agree with you that in most cases its preferable to redesign rather than using concurrency. Since it is problematic not just on the programatic level. It's also tough to debug and hard to comprehend and follow, especially if you have not worked with the code before.
It just seems to be a required and used feature in our robotic department (which also looks for a new state machine). I better check with the guys again about what they need. I assume they have cases like:
planTrip ----> checkEnvironment
    |                 |
    V                 V
checkSelf ------>  doTrip

where multiple settings and sensors have to be prepared and checked before something can be done. So yeah, the execution context is probably multithreaded, distributed and hell. I guess thats why they also want to rework they current solution :).
Thank you for your input so far. I will tinker around a bit and if we decide to build upon transitions I will probably contact you again.
Best regards,
Alex

Added in b327706941c63ab81. Thanks again, @aleneum!


Feautre Request - Workspace from PersistentVolumeClaimTemplate

I would like to be able to have workspace PVC's creation and deletion fully managed by tekton using a volume claim template.
Currently this is supported in Argo Workflow and is a very useful feature (https://github.com/argoproj/argo/blob/master/examples/README.md)
  volumeClaimTemplates:                 # define volume, same syntax as k8s Pod spec
  - metadata:
      name: workdir                     # name of volume claim
    spec:
      accessModes: [ "ReadWriteOnce" ]
      resources:
        requests:
          storage: 1Gi                  # Gi => 1024 * 1024 * 1024

The PVC would be created when the pipeline is executed, and deleted afterwards
This would conceptually be comparable to an emptyDir volume, however with the ability to specify a storage class for the PVC and not relying on a folder on the host as emptyDir does.
The current drawback of the workspace PersistentVolumeClaim is that issues would arise if two pipelines run concurrently that use the same PVC

Thanks for the feature request!  I'm going to close it as a duplicate of https://github.com/tektoncd/pipeline/issues/1986 since we're tracking this feature there.  I appreciate you mentioning the PVC template idea - I'd seen it discussed in the context of Stateful Sets previously but wasn't aware that Argo also supports them.  This is good info!


[turborepo] NextJS with "use client" components causing error

What version of Turborepo are you using?
1.10.14
What package manager are you using / does the bug impact?
Yarn v1
What operating system are you using?
Windows
Describe the Bug
Let's take a classic scenario in which you're exporting components from an index.ts file from let's say a ui package.
You are exporting both server and client components.
Now let's omit on purpose the "use client" directive on our client component and that's it (we do not even import it anywhere in the app).
Even though we don't use it we'll have this error:

You're importing a component that needs useState. It only works in a Client Component but none of its parents are marked with "use client", so they're Server Components by default.

Expected Behavior
Because the component is not imported anywhere in the app, we should not see any error.
To Reproduce

Clone the basic repo: https://github.com/vercel/turbo/tree/main/examples/basic
Create a button component (omitting the "use client" in the ui package):

import type { ReactElement} from 'react';
import { useState } from 'react';

export function Button(): ReactElement {
  const [count, setCount] = useState(0);

  return <button onClick={() => { setCount((prev) => prev + 1); }} type="button">Click {count}</button>;
}

export the button component in the index.ts file:

export * from "./card";
export * from "./button";

That's it, start the  dev mode and you'll see the error.

Reproduction Repo
No response

This isn't an issue with turbo, but a misuse of server/client components. Even if you take turbo out of the equation completely and opt out of workspaces you can reproduce the error by exporting an unmarked client component from a module that isn't marked with "use client":
[0 olszewski@chriss-mbp] /tmp/use_client/apps/web $ cat app/ui/button.tsx
import type { ReactElement} from 'react';
import { useState } from 'react';

export function Button(): ReactElement {
  const [count, setCount] = useState(0);

  return <button onClick={() => { setCount((prev) => prev + 1); }} type="button">Click {count}</button>;
}
[0 olszewski@chriss-mbp] /tmp/use_client/apps/web $ cat app/ui/index.tsx
// component exports
export { Card } from "./card";
export { Button } from "./button";
[0 olszewski@chriss-mbp] /tmp/use_client/apps/web $ head app/page.tsx
import Image from "next/image";
import { Card } from "./ui";
import styles from "./page.module.css";

function Gradient({
  conic,
  className,
  small,
}: {
  small?: boolean;
[0 olszewski@chriss-mbp] /tmp/use_client/apps/web $ yarn build
yarn run v1.22.19
$ next build
Failed to compile.

./app/ui/button.tsx
ReactServerComponentsError:

You're importing a component that needs useState. It only works in a Client Component but none of its parents are marked with "use client", so they're Server Components by default.
Learn more: https://nextjs.org/docs/getting-started/react-essentials

   ╭─[/private/tmp/use_client/apps/web/app/ui/button.tsx:1:1]
 1 │ import type { ReactElement} from 'react';
 2 │ import { useState } from 'react';
   ·          ────────
 3 │
 4 │ export function Button(): ReactElement {
 5 │   const [count, setCount] = useState(0);
   ╰────

Maybe one of these should be marked as a client entry with "use client":
  ./app/ui/button.tsx
  ./app/ui/index.tsx
  ./app/page.tsx

> Build failed because of webpack errors
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Is there a reason why you are avoiding marking your component with "use client" even though it is a client component? If it's a third party component, you could follow these docs to correctly mark it as such.

@dnguyenfs I ran into this too. I bet you're using the default turbrepo setup when you create a new repo and you're using the pattern that was setup in the ui folder. What is a bit misleading here IMO, unless I'm doing something wrong, is that because the ui/index folder is exporting all of the components that you have in the ui package, if any of them use useState (or any other client-component specific logic), you'll get this error since the way the default turbo is setup, all of the components get imported into the application from ui.
I'm a bit confuse on what the best way to import from the ui package should be if we aren't supposed to do barreling (an index file which exports everything).

@dnguyenfs figured out a better solution: export the components you want instead of using  the index.ts file: https://turbo.build/repo/docs/getting-started/create-new#understanding-imports-and-exports
this will prevent them from all being imported where you might not be intending to. Plus it makes tree shaking possible.

@dnguyenfs figured out a better solution: export the components you want instead of using the index.ts file: https://turbo.build/repo/docs/getting-started/create-new#understanding-imports-and-exports
this will prevent them from all being imported where you might not be intending to. Plus it makes tree shaking possible.

I just wasn't sure why this way (which adds more work when one has already created an index.ts - thanks to ChatGPT for helping rework this!) matters. Why can't Next just import things as needed and within the appropriate client-server realms regardless of "exports" or an index.ts?


Remove Maybe from fallback in Branch

This changes the signature of Branch from:
Branch (NonEmptyArray (Pair a)) (Maybe a)

to:
Branch (NonEmptyArray (Pair a)) a

This PR is mostly just involves whacking type errors but I think fixing them intuitively is enough for correctness.

I think that:

We can remove ExprBacktrack
We can remove branchTry from the Env, as well as withBranchTry since there is no backtracking.
The SemConditional continuation does not need to take a Maybe since there's always an else.

My hunch is that the continuation for SemConditional can be removed altogether and to just have it be a pair. I say this because during evaluation, def is always present now:
    Branch branches def ->
      evalBranches env (evalPair env <$> branches) (defer \_ -> eval env def)

Likewise for quoting, I think the following should be correct?
    SemBranch branches def -> do
      let ctx' = ctx { effect = false }
      let quoteCond (SemConditional a k) = Pair (quote ctx' a) (quote ctx k)
      let branches' = quoteCond <<< force <$> branches
      foldr (buildBranchCond ctx) (quote ctx <<< force $ def) branches'

Thanks!


add logic for placing precharge cells

Codecov Report

Merging #31 (300acea) into master (fac2c02) will decrease coverage by 0.29%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #31      +/-   ##
==========================================
- Coverage   60.91%   60.61%   -0.30%
==========================================
  Files          28       28
  Lines        2635     2648      +13
==========================================
  Hits         1605     1605
- Misses       1030     1043      +13

Impacted Files
Coverage Δ

sram22/src/lib.rs
10.19% <0.00%> (-0.93%)
:arrow_down:

Continue to review full report at Codecov.

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

Superseded by #66


Use deserialize to remove CreateComponentImplementation's template dependency

Signed-off-by: Ashton Larkin<EMAIL_ADDRESS>🦟 Bug fix
Summary
When I worked on #856, I came across an issue where I could not update derived component data when dealing with BaseComponent pointers in EntityComponentManager::CreateComponentImplementation: https://github.com/ignitionrobotics/ign-gazebo/pull/856#discussion_r684511937
The workaround at the time was to have CreateComponentImplementation return a flag that indicated whether the derived component data needed to be updated or not. If the data needed to be updated, it was taken care of in the template method that called CreateComponentImplementation, since the template method knew what the derived component type is.
However, after working on #1131, I realized that we can use component's Serialize/Deserialize methods in EntityComponentManager::CreateComponentImplementation to properly update the derived component data when working with BaseComponent pointers. The reason why this works is because these serialization methods are virtual methods that are implemented by both the BaseComponent and derived component classes. A regular assignment operator wouldn't work because the component classes don't have this operator implemented as a virtual override (see https://godbolt.org/z/hezKTbeEj for an example of what happens when a default assignment operator is used with polymorphism - spoiler alert: the derived class data is lost in the assignment).
As a result of this change, CreateComponentImplementation can return a pointer to the component instead of a update flag, and the template method that calls CreateComponentImplementation can then use this component directly. I can't change the signature of this method in ign-gazebo6 since it will break ABI, but I left a TODO note about fixing it in ign-gazebo7. Since CreateComponentImplementation is a private method, we don't need to tick-tock it, right? We can change a private method without having to deprecate it, if I understand correctly.
Checklist

[x] Signed all commits for DCO
[ ] Added tests
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[x] codecheck passed (See contributing)
[x] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge

After some discussion that was had offline, a point was brought up that not all components have Deserialize implemented. So, we may need to close this PR and stick with the current approach (cc @chapulina).

It was a good idea though. Maybe in the future we can make sure all components are serializable.

Closing this for now since it's not guaranteed that all components are deserializable.


DyGraph header not implemented

There's still one part we kind of forgot: the header legend for the graph.

Might need a local container for the graph so that you can hold the states for that.

Implemented with 4e3b09d47c2b0688caef7ace08450a3706d361de.
Slight change to design!


AVRO-2578: Support Both Capitalizations of Parse

[x] My PR addresses AVRO-2578
[x] My PR references AVRO-2578 in the PR title.
[x] My PR does not add any dependencies.

Tests

[x] My PR adds functionality to existing tests to test both capitalizations schema.[Pp]arse and protocol.[Pp]arse.

Commits

[x] My commits all reference Jira issues in their subject lines.

Documentation

[x] My PR does not add significant new functionality and does not need additional documentation.

@Fokko I've added the deprecation warnings as you suggested and tests that show they are being called.


[wip] local development environment for pilot

This let you spin up local proxy, agent, discovery, and mixer, and issue requests as if through the client-side proxy.
E.g.:
go run test/local/main.go -logtostderr -envoy bazel-genfiles/proxy/envoy/envoy -v 2
curl localhost:8000  --header 'Host: dev.local'

This should also work on mac (without mixer) if you use darwin build of Envoy.

@kyessenov: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/pilot-presubmit.sh
ba599b988982640603128e0a6a2ee1cc73bd08cc
link
/test pilot-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Confirmed that it also works on mac with upstream envoy.

Yeah, it's using non-transparent mode in this setup. It gets more complicated with server-side which is why this is WIP.

/approve no-issue

Very nice.  Does it make sense to have the instruction added to here?  https://github.com/istio/pilot/blob/master/doc/testing.md

It's not ready and not high priority for me. I'm putting it out since it's useful for debugging.

Closing temporarily to reduce load on bot.


Month and Year Resolution

Hi I am currently using this plugin for vaadin 7 (0.95). I am actively trying to add a month and year resolution to my gantt chart yet I cannot understand any of the code in TimelineWidget well enough to do this. Any pointers or suggestions on how to accomplish this would be very appreciated.

Awesome plugin, very instant and intuitive!
We'd like to facilitate this plugin for a timeline visualization, could make very good use of a month or year resolution too.
Are there any plans to tackle this?


Added ClassGuards

Added Skyguard : +4 defense against flying enemies if within 3 spaces of an ally flier.
Added Horseguard : +4 defense against horseback enemies if within 3 spaces of an ally horse rider.
Added Armorboost: +4 attack and defense against armored enemies if within 3 spaces of an armored ally.
Implementation is somewhat costly since all skill required looping over every space to see if an ally with the correct class exist, but should be fine since it is only called once every pre-battle, only if the character has the correct skill

This PR is rejected on performance issue and code style issue.

code style

a series of coding style issues exist that do not meet CONTRIBUTING requirements.

Potential bug exists

no UNIT_IS_VALID judgement on range, and also need judge on unit state such as US_HIDDEN, US_DEAD, US_RESCUED, etc. (and also for now, AuraEffect also lost such judgement, which need to fix later).

Performance issue

Introducing a heavy computing burden without enough attractive effects, especially on considering of it is put inside battlesys.
Maybe we can discuss it in the future after performance issues are optimized, but at the moment I don't think these skills effects is attractive enough to match the performance loss it brings.

I agree that the performance issue is a problem. Since we already have to loop over every unit in 3 tiles radius to check for rhe presence of Alied Unit for Aura, it may be possible to piggyback it and check there if the skill condition can be valided. This should cost at maximum 24 additional call of CheckClassFlier/Armor/etc, but considering how nich the skill is I do not think this to be worthy. I will implent the optimized way on my side in case somebody request it, but I'm gonna close the pull request for now

For now, there three skill has been merged since we have optimized the performance on pre-battle, so that we can provides greater tolerance for performance issues with individual skills.


Update the hard-coded labels to use Salesforce Custom Labels.

Critical Changes
Changes
Issues Closed
New Metadata
Deleted Metadata

@smarovich, I did not create any new Labels (lblYouAreHere, lblHouseholdName, lblHouseholdName, lblFormalGreeting, lblInformalGreeting). I used the existing labels. Please let me know if you have any questions.

Can you update your PR comment with the issue you are closing?


Flagged by Windows Defender since v1.14.20

Hi,
I got a problem since I upgraded all my dependencies, my build was removed by Windows Defender (in a company).
So I rollback all dependency and I upgrade them one by one.
So, I can tell you that when I upgrade from 1.14.17 -> 1.14.22, I got flag. So I checked every version until I found the version 1.14.20 is problematic.
Of course I checked the code of that release and didn't see anything wrong, it's probably the upgrade of sqlite from 3.44 -> 3.45 that introduced a false positive.
So, I'm not sure if you're able to do anything about that. But maybe you can already confirm me that's really a false positive.
Let see if I'm the only one affected by that.
Thank you!
Here is the full message I got by Windows Defender:

There are a lot of false positive due to this new trend of using machine learning to classify malware without proper tuning. Go binaries in general are affected, e.g. https://go.dev/doc/faq#virus
Can you try to upload the file into VirusTotal and check whether other antivirus are flagging this as well? It would be useful to have the link to the analysis, I will take a look to that to give my opinion :-)


Lodestar generates invalid enr on docker network mode shared

While doing debugging for the peering for 4844 devnet5 with lodestar as bootnode, it was figured that lodestar couldn't seem to generate the correct ENR most probably because Network mode was shared
Flags to start lodestar (from docker inspect) :```
"Args": [
"./packages/cli/bin/lodestar",
"beacon",
"--dataDir=/data",
"--listenAddress=<IP_ADDRESS>",
"--port=9000",
"--enr.ip=<IP_ADDRESS>",
"--enr.tcp=9000",
"--rest",
"--rest.address=<IP_ADDRESS>",
"--rest.port=5052",
"--jwt-secret=/execution-auth.jwt",
"--execution.urls=http://ethereumjs:8551",
"--metrics",
"--metrics.address=<IP_ADDRESS>",
"--metrics.port=5054",
"--paramsFile=/network-config/config.yaml",
"--genesisStateFile=/network-config/genesis.ssz",
"--network.connectToDiscv5Bootnodes"
],

generated ENR: ```
enr:-LW4QFTTz_3Q-tVJJRaRUp1LY2eiS5ySkEz_iwwu9C8CQq87NoM5lj7nMGUU60qjBTB7qp2OdXSPGyRGUen_CxyaaAQLh2F0dG5ldHOIAAAAAAAAAACEZXRoMpBSRrDJQEhEBf__________gmlkgnY0iXNlY3AyNTZrMaEDhKvBbQfO2xcsGxsvMq1zpluaUfXhgASnmxlEqH5BNumIc3luY25ldHOIAAAAAAAAAACDdGNwgiMo

enr viewer:

Expected results: ENR should have correct, ip, udp and multiaddrs and peers should be able to connect
cc @skylenet, @parithosh

So this happens on the case of a node that is behind NAT. Currently it seems impossible to announce an IP address that the process isn't seeing on the network interfaces.
There is another related issue that we discovered some time ago on our kubernetes cluster, where the lodestar processes are also running in containers behind NAT: https://github.com/ChainSafe/lodestar/issues/3462

Issue is here https://github.com/ChainSafe/lodestar/blob/e746f9cae45a8b7bb0f51e1400c985c64b189c21/packages/beacon-node/src/network/nodejs/util.ts#L36-L38

thanks @skylenet  for the insights, yes most likely this is the case as we do a cleanup if we scan local interfaces and cleanout ip/udp/multiaddrs if we can't find it:
https://github.com/ChainSafe/lodestar/pull/1192
We will need to figure out how what is the best strategy here, but nodes running behind NAT is not uncommon

@wemeetagain should we have a NAT ip cli arg? it will also provide a way to around the edge cases of us not being able to scan the interfaces

should we have a NAT ip cli arg?

Yeah, like you said its not uncommon. I'd suggest we have a --nat cli flag defaulted to false that can prevent the isLocalMultiAddr check.


Improve return

I had to return null/None from async functions.
Would be good to keep implementation so that it deletes empty returns.

Done


Delinting future awaits round 1

#10754
Before I get too far, here's a sampling to make sure it's what we want.

LGTM!


Update templates.md

When working with multiple jobs in one stage and one template file needs to be referenced, if the job name in the referenced template file is not empty, you get the error "job names must be unique within a stage". So it is good users know this too.

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

#sign-off


chore(frontend): Remove Unused Svelte Files

The following files were removed because they were not used in the project anymore:
src/frontend/src/btc/components/send/BtcSendAmount.svelte
src/frontend/src/btc/components/send/BtcSendDestination.svelte

Those two components are under development.


winbar + create new file

Hello, just reporting this minor issue, with new neovim feature - winbar.
It has these "formatting symbols" where %f means relative path to file (there is also one for absolute path).
When I create new file using neo-tree, then open it, it shows absolute path. When I then close that split and open it again, its fixed, with correct relative path.
This might be totally unrelated to neo-tree, i m just unable to pinpoint the culprit here ....any insights? Anyone able to replicate this?
Edit: formatting options are documented @ help statusline : The value of 'winbar' is evaluated like with 	'statusline'

ok so this is unrelated to this plugin ...i just noticed that from time to time simple :f does the same. So far I didnt figure out what rare condition this is but its totally unrelated to neo-tree for sure.


[JN-660 + JN-854] Add language selector to participant and admin apps + add concept of supported languages

DESCRIPTION
This adds language selectors to 3 places: the participant site navbar, the admin site content editor, and the admin form editor. The dropdowns are populated by a new portalLanguages property that's attached to portals. The UIs are resilient to an empty list of portal languages, so I've only added this concept to the HeartDemo portal for now.
Participant site:

Site editor:

Form editor preview:

TO TEST:  (simple manual steps for confirming core behavior -- used for pre-release checks)
Before beginning:

Restart ApiAdminApp
Repopulate demo: ./scripts/populate_portal.sh demo
Repopulate OurHealth: ./scripts/populate_portal.sh ourhealth

PARTICIPANT:

Restart ApiParticipantApp and participant-ui (start the participant-ui using  REACT_APP_UNAUTHED_LOGIN=true HTTPS=true npm -w ui-participant start so you can bypass B2C)
Test HeartDemo:

Go to the demo website: https://sandbox.demo.localhost:3001/
Verify that you see the language selector
Verify that toggling languages works as expected for site content
Register as a new user and consent (the consent form will only be in English, it hasn't been translated yet)
Click into The Basics survey, verify that switching languages works as expected

Test OurHealth:

Go to https://sandbox.ourhealth.localhost:3001/
Verify that no language selector is visible

ADMIN:

Restart ApiAdminApp and admin-ui
Test HeartDemo:

Go to the site content editor: https://localhost:3000/demo/studies/heartdemo/env/sandbox/siteContent
Verify that you see the language selector and you can switch between languages
Go to the Preview tab in the form editor: https://localhost:3000/demo/studies/heartdemo/env/sandbox/forms/surveys/hd_hd_basicInfo
Verify that you see the language selector and you can switch between languages

Test OurHealth:

Go back to the site content editor: https://localhost:3000/ourhealth/studies/ourheart/env/sandbox/siteContent
Verify that you do not see the language selector
Go back to the Preview tab in the form editor: https://localhost:3000/ourhealth/studies/ourheart/env/sandbox/forms/surveys/oh_oh_basicInfo
Verify that you do not see the language selector

@devonbush this is ready for re-review. thanks for catching the misplacement of the supported languages. I've also added the lang url param, I think that's a nice feature even if it's only for internal use: https://sandbox.demo.localhost:3001/?lang=es


Updated service-fabric-get-started-with-a-local-cluster.md

Updated broken link to WordCount app url from http://localhost:8081/wordcount/index to http://localhost:8081/wordcount/index.html in the article and screenshots

Hi @slapointe, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, AZPRBOT;

@seanmck - Sean, could you review these updates proposed by @slapointe in the public repo?

Already fixed - you can close.

@slapointe - Thanks for the update - I apologize for the delayed review. Closing per the author's feedback.


Kitchen Display View does not show any order

Expected behavior:
Kitchen Display View should show a list of orders as per the selected status filter.
Actual behavior:
Kitchen Display View show's blank in spite of setting a valid order status filter. I have tried for all Statuses and as well as each specific status. It still does not list any orders.
Reproduce steps:

Create a new Kitchen Display view
Under view creation form set the "Orders with initial status" field to "Pending Preparations" or any other status.

Version:
v3.0.4.-beta 24.4

Please report here: https://github.com/ryanmitchell/ti-ext-kitchendisplay/issues as its related to the extension


Error accessing the valid qna doc url without login

Below is valid url link
http://localhost:3000/doc/e3d54960-40f7-4d86-b503-31f14f3dfa13
Accessing the link directly without login as joe or mary

s/b fixed now

Fixed


(privazer)

Checklist

[X] I have checked the moderation queue that no new version has been submitted (only visible when logged in), or the submitted package version has failed the automated checks.

Please navigate to the following link to view the moderation queue.

[X] I have verified that the new version is a stable release.
[X] I have looked to see if there are any Bug reports that prevents a new version being submitted and that no existing Outdated Reports have been created.
[X] I have verified the new version has a Windows binary (EXE or MSI installer, Zip or other archive, etc).

New Software Version
4.0.80
Download location
https://privazer.com/de/PrivaZer_free.exe
Package Page
https://community.chocolatey.org/packages/privazer

That package isn't maintained in this repository.
Can you go back to the package page and contact the maintained?


Code mistake (C.5.40)

C.5.40 Solution to: Union of Maps
def union[A](map1: Map[A, Int], map2: Map[A, Int]): Map[A, Int] = {
   map1.foldLeft(map2){(map, elt) =>
      val (k, v) = elt
      val newV = map.getOrElse(k, v) //Mistake
      map + (k -> newV)
   }
}

This code returns Map(a -> 2, b -> 4) instead of desired Map(a -> 3, b -> 6)
val newV = v + map.getOrElse(k, 0) - that will fix the problem

This issue has been fixed in https://github.com/underscoreio/essential-scala/commit/c38e3680f75789808d07aaa1d595dfe8fc4cdace.

Yes -- @daniashers fixed this when she migrated everything to Tut. Thanks all!


fix: Update code formatting

This PR:

Fixes some code formatting

@tanay1337 good catch!
Done. I formatted the code in the repos for tutorials/demos and run the generateCodeExamples.jsscript in the docs to add the changes in the examples folder.


au.com will be added into email domain of AU at April, 2018

http://news.kddi.com/kddi/corporate/newsrelease/2017/08/22/2637.html
Sisimai::Bite::Email::Ezweb
Sisimai::Bite::Email::KDDI

@au.com will be available at 09:00 on 15 May (JST).
https://japanese.engadget.com/2018/04/18/au-ezweb-ne-jp-5-15-au-com/

We need a bounce mail from au.com which is rejected by a domain filter.


Possible fix for LIMIT issue

https://github.com/mogui/pyorient/issues/153

Coverage decreased (-0.02%) to 79.706% when pulling 5cddaaba5b2c6dd23b90a6f0709c23925c1c3035 on Ostico:possible_fix_limit into 0bedd751beebd9d61419cb9619060a0c5123c24e on mogui:develop.


refactor: delete gas_costs_map

This change is

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 70.28%. Comparing base (b493327) to head (b911c57).

Additional details and impacted files
@@                                     Coverage Diff                                     @@
##           ori/starknet/versioned_constants/os_constants_class/pr4    #1620      +/-   ##
===========================================================================================
+ Coverage                                                    70.27%   70.28%   +0.01%
===========================================================================================
  Files                                                           59       59
  Lines                                                         8344     8330      -14
  Branches                                                      8344     8330      -14
===========================================================================================
- Hits                                                          5864     5855       -9
+ Misses                                                        1997     1993       -4
+ Partials                                                       483      482       -1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


TWIA post order not correct

We want it to be the latest first not oldest first.

This issue has been resolved with #17. I think we can close this issue.

I agree. I think this has been resolved.


[BUG] Terraform - After deployment Issue

Describe the bug
I am able to deploy the application but its getting error to display chatflows.

To close this issue.

need to enable uuid-ossp in postgres sql
remove zone option in database tf
add DATABASE_SSL=true in webapp.tf

which terraform guide are you using? if possible, an update on the docs would be helpful! https://github.com/FlowiseAI/FlowiseDocs


Add the ability to enable and disable tasks instead of having to cancel and re-insert

Add the following two new methods to taskManager:
void enableTask(taskid_t taskId, boolean taskEnabled);
boolean isTaskEnabled(taskid_t taskId);

We'll have a new status flag for enabled, I've checked and there are enough spare bits in the current execution mode status flag to deal with this.

On enable, the task will be put back into the running list and the next scheduled time calculated. This means that the task would continue as normal.
On disable, the flag would be set to mark it as such, it would then not even need to be removed from the queue, calling execute would do nothing, and it would not repeat.
This would not apply to events, as they are often triggered from interrupts, and the user has complete control over when events execute anyway. It would apply to functional and OO (executable) tasks.

This is a low-risk, low overhead change that is completely backward compatible.

This is tested on ESP8266 and now on a branch awaiting merging after more testing.


Experimental Deno support fails with the latest version (1.18.0)

What version of Remix are you using?
v1.2.1
Steps to Reproduce
I wanted to give the new deno server a try but it failed right away with the following errors:
Watcher Process started.
Check file:///xyz/build/index.js
Watching Remix app in development mode...
💿 Built in 708ms
error: TS2694 [ERROR]: Namespace 'Deno' has no exported member 'EOF'.
export function assertNotEOF<T extends {}>(val: T | Deno.EOF): T {
                                                         ~~~
    at https://deno.land/std/testing/asserts.ts:384:58

TS2339 [ERROR]: Property 'EOF' does not exist on type 'typeof Deno'.
  assertNotEquals(val, Deno.EOF);
                            ~~~
    at https://deno.land/std/testing/asserts.ts:385:29

TS2367 [ERROR]: This condition will always return 'false' since the types '"darwin" | "linux" | "windows"' and '"win"' have no overlap.
export const isWindows = build.os === "win";
                         ~~~~~~~~~~~~~~~~~~
    at https://deno.land/std/path/constants.ts:51:26

By checking Deno's changelog it seems that EOF support was removed in v0.42.0 / 2020.04.29  which is quite a long time ago.
I used yarn create remix@latest and choose Deno, no special other steps.
Expected Behavior
Even though it's an experimental support I would expect it to work :-)
Actual Behavior
There are the errors as described above.

@synaptiko Run deno info and delete the folder specified as DENO_DIR. You are using an outdated version on deno.land/std (and remix is using an unversioned import - this is the real problem).

Thank you @lucacasonato, makes sense and works.

Using the latest version available on brew
❯ deno --version
deno 1.19.1 (release, x86_64-apple-darwin)
v8 <IP_ADDRESS>
typescript 4.5.2

I got the following error even after removing the DENO_DIR
Watcher Process started.
error: Relative import path "@remix-run/deno" not prefixed with / or ./ or ../
    at file:///Users/melkir/Projects/my-remix-app-versel/build/index.js:31:42
Watcher Process failed. Restarting on file change.

Same here as @melkir
though I just upgraded via deno upgrade
deno 1.19.2 (release, x86_64-apple-darwin)
v8 <IP_ADDRESS>
typescript 4.5.2

and I'm on Remix 1.2.3

Same issue here, the generated index.js under build folder doesn't follow the way deno does imports.

I can confirm, having the same issue with the latest remix & deno as the others above.

come across this issue as well

Official support for Deno came out on May 18th, 2022 with version 1.5.0.
If Deno support is not working, feel free to reopen or create a new issue.


File name is being set to _id

When uploading a file, the file name in the mongodb collection.files is being set to _id instead of the file name uploaded. This causes the url to be based on _id.extension instead of filename. for images to be displayed in the app, not a big deal, but when downloading the file, it does matter.

@hluz the url is _id.extension not because we save the _id in file name since this is mounted based in the file collection and not based in GridFS collections, you can look at your file collection, the name will be their.
This issue is related with https://github.com/jalik/jalik-ufs/ and not the stores, maybe we can allow users to pass a custom fileId when upload starts https://github.com/jalik/jalik-ufs/blob/master/ufs-uploader.js#L56.
@jalik what do you think?

I actually chose to use this ID in the url to avoid any future break in case of file renaming, all old links will still be valid if the file name changes.
Solution 1 : send a HTTP header which forces to download the file with the real file name
Solution 2 : change the file URL using a pattern like http://domain.com/{store}/{fileId}/{filename}.{ext}, but that would break links used by existing apps and it needs more work to make a patch that would update existing records in the file collection to use this new URL pattern..
What do you think ? any other proposition ?

I close this issue since it was fixed in the main package jalik:ufs.


Update data-access-in-cpp.md

Updated typo on "Availability Groups" (fix for bug 974890)

@v-kents,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Create tutorial material for LLMs

[x] go over @ampudia19 's material
[x] slides framework
[x]  notebook framework

First draft of LLMs tutorial is done. Can find first draft here.