Split softwarecontainer to .bb and .inc

Create a .inc-file for softwarecontainer which contains all PACKAGECONFIG and other settings, and create a bb-file with the defaults we want in meta-bistro. This should make it easier for other layers to configure softwarecontainer as they see fit.

@GordanM Does this make sense?

@jonte valid point!

Hey Jonatan, I've reevaluated the point you made and had some second thoughts. To me it seems like the usual scenario is that the recipes have a default package configuration set in the original recipe via PACKAGECONFIG ?(?)=  and the users will adapt their own PACKAGECONFIG setup depending on their needs in a project specific layer bbappend file.
Does that sound reasonable to you?

Hi @GordanM
Yep, that does also sound like a reasonable approach to me, and we would achieve the same results.

In my experience, .inc files are mostly used when there are several versions etc with more differences than simply PACKAGECONFIG or similar. In this case I don't see the benefit from it.

Well put @tobsan. @jonte would you be OK with closing this issue?

Yep, works for me. Did we capture the PACKAGECONFIG modifications in a difference issue somewhere?

So, right now we have a PACKAGECONFIG ?= "some options here". We can of course add more PACKAGECONFIG entries to ease configuration, but apart from that I think we got things covered wrt what this issue describes.


Basic example

The only example is incredibly barebones and doesn't even begin to touch on what this library can do. The documentation is all incredibly isolated snippets.
Can we get some basic usage examples?

I found one.  https://pastebin.com/L14DCrky
Also, the original one linked from the above in older repository: http://bit.ly/2ZjMP02
Hope it'll do for you!


Bump golang.org/x/net to 0.1.1-0.20221104162952-702349b0e862

Current version has vulnerabilities  CVE-2022-41721

/retest

@jenting can you please help me understand the CI failure? Seems like an image issue

@jenting can you please help me understand the CI failure?

@kashifest It's the kustomize issue, I am working the fix now.

@kashifest Could you rebase it? Thank you.

@kashifest Could you rebase it? Thank you.

Done


Interactive examples for FEDOT

Hi FEDOT team!
My team maintains a few open-source projects, and we've seen that having interactive examples greatly helps newcomers! We're maintaining a Binder cluster that provides ephemeral JupyterLab sessions (example). It essentially spins up a JupyterLab session with pre-made examples.
We're happy to sponsor FEDOT with some computing power. If you're interested, please get in touch with me! (my contact info is in my profile)

Hu Eduardo!
It is quite for us interesting to provide the examples of FEDOT usage for the Binder users.
Now have the set of pre-evaluated notebooks for different tasks.
I will re-check the existing examples and contact you via email about the details of integration with Binder.

sounds good!

Hi, following up here
I went ahead and enabled our binder cluster for pyRiemann: https://binder.ploomber.io/v2/gh/aimclub/FEDOT/master?filepath=examples
It's giving some issues because of the multiple requirement files:
Step 4/11 : RUN cat other_requirements/extra.txt >> /home/FEDOT/requirements.txt
 ---> Running in e12be8d133de
cat: other_requirements/extra.txt: No such file or directory
Removing intermediate container e12be8d133de

seems like binder is automatically trying to infer which are the requirements to install, but we can configure it to avoid this issue.
Happy to open a PR to get this up and running!


{Storage} az storage file copy start: add examples for snapshot

History Notes:
(Fill in the following template if multiple notes are needed, otherwise PR title will be used for history note.)
[Component Name 1] (BREAKING CHANGE:) (az command:) make some customer-facing change.
[Component Name 2] (BREAKING CHANGE:) (az command:) make some customer-facing change.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

add to S166.


Self host vector server

I have some trouble since I want to use my custom tile server. I created a custom server with OpenMapTiles. Now I want to use this server in my application.
I tried to use the example file to understand better how to use the vector-map-tiles package but I retrieved some problems.
@override
  Widget build(BuildContext context) {
    return SafeArea(
      child: Column(
        children: [
          Flexible(
            child: FlutterMap(
              options: MapOptions(
                center: LatLng(41.301710, 19.795678),
                zoom: 10,
                maxZoom: 15,
                plugins: [VectorMapTilesPlugin()]
              ),
              layers: <LayerOptions>[
                VectorTileLayerOptions(
                  theme: ProvidedThemes.lightTheme(),
                  tileProviders: TileProviders(
                    {
                      'openmaptiles': MemoryCacheVectorTileProvider(
                        delegate: NetworkVectorTileProvider(
                          urlTemplate: 'http://<my_ip>:8080/styles/basic-preview/{z}/{x}/{y}.png',
                          maximumZoom: 14
                        ),
                        maxSizeBytes: 1024 * 1024 * 2
                      ),
                    }
                  )
                ),
              ],
            ),
          ),
        ],
      ),
    );
  }

Error:
[VERBOSE-2:ui_dart_state.cc(198)] Unhandled Exception: MissingPluginException(No implementation found for method getTemporaryDirectory on channel plugins.flutter.io[/path_provider]())
#0      MethodChannel._invokeMethod
package:flutter/…/services/platform_channel.dart:165
<asynchronous suspension>
#1      getTemporaryDirectory (package:path_provider[/path_provider.dart:55:24]())
<asynchronous suspension>
#2      ByteStorage._storagePath (package:vector_map_tiles[/src/cache/byte_storage.dart:20:25]())
<asynchronous suspension>
#3      ByteStorage.fileOf (package:vector_map_tiles[/src/cache/byte_storage.dart:32:18]())
<asynchronous suspension>
#4      ByteStorage.write (package:vector_map_tiles[/src/cache/byte_storage.dart:42:18]())
<asynchronous suspension>
#5      StorageCache.put (package:vector_map_tiles[/src/cache/storage_cache.dart:39:5]())
<asynchronous suspension>
#6      VectorTileLoadingCache._loadBytes (package:vector_map_tiles[/src/cache/vector_tile_loading_cache.dart:87:7]())
<asynchronous suspension>

I'm posting also the Tileserver-gl since is just a debug server to understand how to work with maps: http://<IP_ADDRESS>:8080

Is this on flutter web, or a mobile device ? Just wondering from the error if its because there's an io feature that can't be available in the browser ?

@carlodenardin thanks for the issue. Can you confirm that you can run the example app?
Perhaps you could include the output from flutter doctor -v and the contents of your pubspec.yaml

HTTP 403 is the server rejecting the HTTP request. I'm not sure why you'd get that from Stadia maps, but perhaps it's something related to using an appropriate API key.

InvalidProtocolBufferException: Protocol message contained an invalid tag (zero).

I'm not sure why this is coming up from you, perhaps there's something wrong in one of the protobuf tiles. I've had cases where tile data wouldn't load due to a half-written file in the file cache. From what I could guess, it was due to terminating an app while the file was being written to disk. You may want to clear out the file cache and try again.
As for the original issue, it should be straight forward to troubleshoot by putting a breakpoint where the plugin is loaded. You'll be able to see which platform is being used, and from there figure out why the plugin isn't available. I'm glad that you can run the example, please post what you learn back here so that others can benefit if they run in to the same problem.
From the output of flutter doctor it looks like you're running an unreleased version of Flutter. If all else fails, you may want to try running with a released version since that is what we test with.
Closing since this does not appear to be a defect with this plugin.


nixos: Provide users.setupHomeScript option

Provide a new option which allows the user to fill a user's home
directory with initial content.
Tested it thoroughly on my system - Worked like a charm :-)
The perl code can possibly be improved - I never wrote any perl before.

cc @edolstra

initialHomeSetup instead of setupHomeScript ? Or include the "Script" part: initialHomeSetupScript is getting lengthy though. Opinions?

Ok, I settled with initialHomeSetup and made a little improvement: The implementation now takes care that the created files have the right owner and group.

Any more feedback? I am very willing to improve this request, if concerns are raised! :-)

Well, this feature is nice, but I think it's a little bit... Well, for example. If I used it once and then I realized that I was wrong -> next nixos-rebuild will call this init? Maybe you can go deeper and implement some kind of user's dot files manager via users.<user>.dotFiles = [list of files], or even use the same technique as we use in environment.etc."file name"?

Yeah, I agree with @jagajaga. This is more imperative than declarative, especially since the setup script is only run when the home directory is first created. A more declarative behavior would be something like environment.etc.
For specifying the initial contents of a home directory, the PAM module has an option makeHomeDir that copies the contents of /etc/skel on initial login, but I haven't tried it myself.

Thank you for the feedback! I'll look into environment.etc.

Honestly, I do not see any issue for having an imperative way of setting the content, and having a declarative method which use the imperative one.
We could make this imperative method as an implementation detail.  I think having both might make sense.  The use case I am thinking of, is that some packages (gitlab?) will provide a way to setup the home directory of the user.

Actually, this is already kinda supported in NixOS since 8f50d803ef9c94fb82909e22b603982a0a522aea.
in configuration.nix:
environment.etc = [
  { source = ./software/config/vimrc;
    target = "skel/.vimrc"; }
  { source = ./software/config/zshrc;
    target = "skel/.zshrc"; }
}

security.pam.services.login.makeHomeDir = true;

users.extraUsers."me" = {
  uid = 1000;
  group = "mygroup";
  useDefaultShell = true;
  home = "/home/me";
  createHome = false; # !!!! this is the crux
};

And the home dir will be created the first time you login.

But how we wll get dotfiles? ln them from /etc?
2015-08-20 18:01 GMT+03:00 Jan Malakhovski<EMAIL_ADDRESS>
Actually, this is already kinda supported in NixOS since 8f50d80
https://github.com/NixOS/nixpkgs/commit/8f50d803ef9c94fb82909e22b603982a0a522aea
.
in configuration.nix:
environment.etc = [
{ source = ./software/config/vimrc;
target = "skel/.vimrc"; }
{ source = ./software/config/zshrc;
target = "skel/.zshrc"; }
}
security.pam.services.login.makeHomeDir = true;
users.extraUsers."me" = {
uid = 1000;
group = "mygroup";
useDefaultShell = true;
home = "/home/me";
createHome = false; # !!!! this is the crux
};
And the home dir will be created the first time you login.
—
Reply to this email directly or view it on GitHub
https://github.com/NixOS/nixpkgs/pull/9073#issuecomment-133042024.

--
Sincerely,
Arseniy Seroka

The pam approach with /etc/skel isn't quite flexible, as all users will get the same files. For the users..dotFiles approach, I think this would be quite cumbersome and also not very flexible. For example in my setup I have the script clone a private git repo into my home directory. How would you do this decoratively? I made a mistake calling it declarative in the documentation, it is not, but I believe this imperative way of setting up an initial home directory is very worthwhile and useful. Users typically don't manage their home directories in a declarative fashion.
It also imposes no overhead for already created home directories, meaning it does not affect nixos-rebuild's performance at all.
It is tempting to use a declarative approach for managing dot files, but on the other hand who wants to run nixos-rebuild switch every time you make a change to your .emacs?

But how we wll get dotfiles? ln them from /etc?

Yes.

The pam approach with /etc/skel isn't quite flexible, as all users
will get the same files. ...

Quite a while ago when I thought about implementing something like this
my idea was to have something like the following in configuration.nix:
users.extraUsers.username.dotfiles = {
".zshrc".text = ...;
".vimrc".text = ...;
}
which would hash all these the files and generate a login script which
would store these hashes in ~/.nix-dotfiles like this:
 .zshrc
 .vimrc
...
or maybe
 </nix/store/path> .zshrc
 </nix/store/path> .vimrc
...
(for diffs, see below)
and update dotfiles with an init script using the following rules:

there's a new version of .nix-dotfiles available, (*) all the files
listed in the old .nix-dotfiles did not change (their hashes are the
same as recorded in the old .nix-dotfiles) -- write .nix-dotfiles
and update all the things that changed hashes,
there's a new version of .nix-dotfiles available, (*) but some files
have been changed by the user (their hashes differ from what is
recorded in the old .nix-dotfiles) -- print a warning with all the
changes (e.g. diffs) and do nothing (this is the only correct way to
do it, since updates have to be atomic to work in the most general
case),
there's a new version of .nix-dotfiles available, but there's no old
.nix-dotfiles in user's home -- write .nix-dotfiles and apply one
of the two rules above, reading them from the (*),
else -- do nothing.

This has all the advantages of plain text dotfiles (e.g. you can hack
them) with the power of nix on top.
I got lazy.

@oxij So basically a system where dot files are overridden by system ones, only if they don't exist or if they haven't changed from previously provided ones? What are the benefits to just managing your dot files in a git repo?

Yes, atomically.
Automatic deploy. And you can generate a dotfile using a nix function
that depends on user's login name, home dir or whatever, for example.

Ah, I did not describe the algorithm correctly, yes.
The typical workflow would be:

Hack dotfiles around till they work as expected
Run system update (or some diff-like util) to see the diff
Update the generator with that diff
Run system update to check that you applied it correctly (no warnings)

The point is that if you made correct changes to the dotfiles generator so that it would generate exactly the dotfiles you already have in your home, it would assume you have already made the updates and it's just the .nix-dotfiles that needs updating.

In the simplest case when your dotfiles just symlink to files in your dotfiles git repo (or maybe the are the files in your git repo, like when using vcsh)
and all the configuration.nix does is lists how to deploy them, you just update them in place in your home dir and the system update does exactly nothing or prints a warning for your user while updating other users, that use same dotfiles.

I am sorry, but in comparison to a git repo I still only see a more complex system without any real benefits. The home directory is impure, so reproducibility is a bit limited. For git it requires that you checked everything in, in their latest versions. For your approach it is pretty much the same, but with different tools/workflows. Am I still missing something?

Okay, but what the benefit of your patch then?

Well it is just a minor convenience that you have your system fully set up, including your home directory with one command: "nixos-install". That's it, nothing more. I think it is nice and it costs nothing.
It is kinda cool to have your complete system set up with just one command.

But then, if you want updates, you have to have some update logic. If
you don't want updates, why automate something that is run exactly once.
I'd say vcsh (possibly with mr) is a much better solution than any
of these here if you don't want to functionally generate configuration
files.

It is run exactly once for each system & user you have. The script that is run can be git clones, mr commands whatever. So it is not his patch or git/mr, but this patch together with git/mr.
It is really just a minor feature, because you could of course run nixos-install  and afterwards a script which sets up home directories. But with this patch you have everything in one place, which is in the spirit of nixos, I think. For example if you want to remove a user for your setups, you do this in one place instead of in two. If you add a new user to your configuration and have the git repo on your server already, then all you need to to is add the user to your configuraion.nix file and nix takes care of everything.

One command installation
Integrated user management includes setting up home directories

Whether you find this useful or not depends on your workflow. But what is nice - not even does it cost you nothing if you don't use it, it even costs you nothing if you use it. Because if the home directories are already present ( a check that is run already) nothing happens.
Also this patch it is not necessarily your system or this patch, there could easily be both and people choose what they prefer.

Closing, due to lack of interest.


Small conflict graph cleanup

For this code it's cleaner to use a loop than forEach() plus a mutable array reference.

@msridhar Thank you very much, LGTM 🙂.


Incorrect splitting of query parameters parameters containing literal commas

The current query string translation logic doesn't distinguish between repeated query string values (i.e. &key=a&key=b) and values that contain URL encoded literal commas (i.e. &key=a%2Cb). The former should map to a fastify array value, the latter should not. It appears the API GW V2.0 event format has this same hazard.
The two cases could be distinguished by parsing the rawQueryString value (or for performance reasons, doing this only when a potential split is detected).
https://github.com/fastify/aws-lambda-fastify/blob/2e9387bfc8172be360c1bac997db216f7535d7fa/index.js#L65

Feel free to propose a PR.

Thanks, will do. I have a workaround for now, but expect one in the next few days.

Hi @jmalins can you please share your workaround ? I'm facing the same issue with commas

I'm not sure if this is an intended behavior but I realised that the query string translation is mutating the original event argument passed to the lambda.
For example if you pass &key=a%2Cb :
When logging request.awsLambda.event you will get
{key: ["a","b"]}

even though the original event passed from API gateway is:
{"key": "a,b"}

This was confusing in our case because it made my team doubt that API gateway itself was transforming the query
I submitted a PR to address this behavior ( within the scope of the querystring ). and added a new option to allow users to opt-out from the current behaviour without causing any breaking changes for existing users

related to https://github.com/fastify/aws-lambda-fastify/issues/125 @maxkre96


Save protobuf Phenopacket files accompanying each individual

Phenopackets is a possible standard we could use for representing information about individual/samples.
It uses protobuf for serialisation.
This ticket would be able investigating it and perhaps generating some sample phenopackets for the info we store in Elsa.
How/where/whether we actually end up using phenopackets is a different story.

We need permissions from AG data governance as to whether any data of phenotypes can be stored outside of MCRI Redcap. Putting in blocked for the moment

The sex phenotypes for the cardiac flagship have been released and I will upload

Uploaded. Closing this ticket now as the future work on this will be covered by the phenopacket first loader


Error "Module '"../../../../node_modules/ngx-auth-utils/ngx-auth-utils"' has no exported member 'AuthConditionalDirective'."

I'm getting this warning from my IDE:

Are you missing an export here?
https://github.com/mari8i/ngx-auth-utils/blob/8db9c08a2fe87b8ff7c6fd9225467cfe1fcc8ba2/projects/ngx-auth-utils/src/lib/ngx-auth-utils.module.ts#L43

It looks like IDE's fault. After opening the public-api.d.ts of the library the warning disappeared


Docs fixes

Thanks for this PR! There is currently a CI issue with Cypress GUI tests, see this PR where I tried to disable the failing test: https://github.com/microsoft/PlanetaryComputerDataCatalog/pull/473/files
Before you merge this, maybe you can try disabling that test and get the CI pipeline back to green?


Per-plane list endpoint in UCP does not function

Doing a request to a URL like /planes/radius should list all instances of the radius plane, but instead it returns the following error:
{
          "error": {
            "code": "HttpRequestPayloadAPISpecValidationFailed",
            "details": [
              {
                "code": "InvalidRequestContent",
                "message": "failed to parse route: undefined route path"
              }
            ],
            "message": "HTTP request payload failed validation against API specification with one or more errors. Please see details for more information.",
            "target": "ucp/ucp"
          }
        }

AB#5506

We have a skipped UCP integration test that's linked to this issue, re-opening to understand if we can unskip this test now: https://github.com/radius-project/radius/blob/main/pkg/ucp/integrationtests/planes/aws_test.go#L76

I'd like to fix this underlying bug. There's some planned/expected functionality in UCP that's covered by this test and just doesn't work.


#5178 Massmint upload picture

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[x] Feature
[ ] Refactoring

Context

[x] Closes #5178
[ ] Requires deployment <>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[x] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </bsx/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Had issue bounty label?

[x] Fill up your KSM address:
Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot 📸

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

Created a NeoCollapse and a DropUpload common component
left:

[ ] uploading state

Left:

[ ] disabled state
[ ] uploading state

maybe use https://vueuse.org/core/useDropZone/  ?
So it can handle the upload logic

I think it should be disabled if you dont choose a collection

yes, https://github.com/kodadot/nft-gallery/pull/5300#issuecomment-1475619436
I added it in the leftovers, probably will be done in another issue after this is merged

maybe use https://vueuse.org/core/useDropZone/ ?
So it can handle the upload logic

why do we need this exactly?

can we merge this please

what is left?

what is left?

ui wise it seems to be done, some logical stuff is left, I created new issue for that

Yes, managed to upload a zip file, let's move on

can we merge this?

/reviewpad summary

So tldr:
we added neocollapse and dropUpload logic was moved to separated issue?

also @prachi00 can you please resolve conflicts?

/reviewpad summarize

So tldr:
we added neocollapse and dropUpload logic was moved to separated issue?

yeah current drop upload logic isn't accounting for unzipping zip files, it has the common logic for files like jpg and stuff
so in new issue, we will be adding unzipping

pay 30 usd

@prachi00 can you please check?

@prachi00 can you please check?

passes now it seems

pay 35 usd

pay 40 usd

😍 Perfect, I’ve sent the payout
💵 $40 @ 5.05 USD/DOT ~ 7.921 $DOT
🧗 13Qx65nLd6SwdtjrRyuoEtp9CKXhF651xdHBPaXcvhwKm4N1
🔗 0x55a391fae9f253e8634ab6596b0b1770416000f36bfb79db8f9be45e0dbe667b
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


created change event should contain a top level description

While git is definitely not the only version control tool, I would say a lot of various tools utilize pull requests. While commits can have a commit message each, having an overarching description for the overall change is really beneficial.
Including a top level description field could give tools an easy way of displaying the description back to users without needing to use the GIT API or something similar.
{
  "context": {
    "version": "0.4.0-draft",
    "id": "271069a8-fc18-44f1-b38f-9d70a1695819",
    "source": "/event/source/123",
    "type": "dev.cdevents.change.created.0.1.2",
    "timestamp": "2023-03-20T14:27:05.315384Z"
  },
  "subject": {
    "id": "mySubject123",
    "description": "description of my changes", # new field here
    "source": "/event/source/123",
    "type": "change",
    "content": {
      "repository": {
        "id": "TestRepo/TestOrg",
        "source": "https://example.org"
      }
    }
  }
}

Thanks @xibz - we strive to keep the content of the events as small as possible, as additional data can usually be retrieved via an API call. That said, I don't have an issue with adding an optional description to the spec, if that can help with the performance of visualisation tools.
Would you like to propose a PR for this?

I agree that an optional description field would be suitable in the subject of the change.created event

One thing to keep in mind is that the description of e.g. a PR in GitHub can be edited. We would not catch such edits with the current events.


🐛 Signature for Trajectory.on_env_feedback() Interface should include the arguments used by the Actor class

Description

The Trajectory.on_env_feedback() interface defined here
only includes self as an argument.
However, the Actor() class defined here requires that the on_env_feedback() method include additional arguments:
self.trajectory.on_env_feedback(
                event, state_by_agent, action_by_agent, reward if reward is not None else self.env.metrics
            )

When we define a custom Trajectory, we can overwrite this method (and we do), but the difference in signature produces pylint errors:
W0221:Parameters differ from overridden 'on_env_feedback' method

As far as I can tell, every use of on_env_feedback requires that same signature, so it should be represented in the interface.
Screenshots

To Reproduce
Steps to reproduce the behavior:

Write a custom

Expected Behavior

Environment

MARO version (e.g., v0.1.1a1):
MARO scenario (CIM, Citi Bike):
MARO component (Simulation, RL, Distributed Training):
Orchestration platform (GraSS on Azure, AKS on Azure):
How you installed MARO (pip, source):
OS (Linux, Windows, macOS):
Python version (3.6, 3.7):
Docker image (e.g., maro2020/maro:latest):
CPU/GPU:
Any other relevant information:

Additional Context

That is correct @jreynolds01. Thanks for pointing it out. FYI, Trajectory has been revamped to AbsEnvWrapper in the upcoming version, so things will appear quite different. Stay tuned.


Enable CodeViewer in samples

Enable CodeViewer in samples
Use Json source generator to avoid trimming issue

Change init to set as json source generator in .NET 7 doesn't support init properties yet, we can revert them back once we upgrade to .NET 8

I know a new Samples app was being worked on. That might make this obsolete.

Closing as a new sample app is coming.


Add information in the AzureRM pages to direct users to Az.

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-conceptual/includes/az-replacing-azurerm.md
:warning:Warning
View (azurermps-4.4.1) View (azurermps-5.7.0) View (azurermps-6.10.0)
Details

docs-conceptual/azurermps-4.4.1/install-azurerm-ps.md
:white_check_mark:Succeeded
View (azurermps-4.4.1)

docs-conceptual/azurermps-4.4.1/install-azurermps-maclinux.md
:white_check_mark:Succeeded
View (azurermps-4.4.1)

docs-conceptual/azurermps-4.4.1/other-install.md
:white_check_mark:Succeeded
View (azurermps-4.4.1)

docs-conceptual/azurermps-4.4.1/overview.md
:white_check_mark:Succeeded
View (azurermps-4.4.1)

docs-conceptual/azurermps-5.7.0/install-azurerm-ps.md
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-5.7.0/install-azurermps-maclinux.md
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-5.7.0/other-install.md
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-5.7.0/overview.md
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-6.10.0/install-azurerm-ps.md
:white_check_mark:Succeeded
View (azurermps-6.10.0)

docs-conceptual/azurermps-6.10.0/install-azurermps-maclinux.md
:white_check_mark:Succeeded
View (azurermps-6.10.0)

docs-conceptual/azurermps-6.10.0/other-install.md
:white_check_mark:Succeeded
View (azurermps-6.10.0)

docs-conceptual/azurermps-6.10.0/overview.md
:white_check_mark:Succeeded
View (azurermps-6.10.0)

docs-conceptual/includes/az-replacing-azurerm.md

Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-conceptual/includes/az-replacing-azurerm.md
:warning:Warning
View (azurermps-4.4.1) View (azurermps-5.7.0) View (azurermps-6.10.0)
Details

docs-conceptual/azurermps-4.4.1/install-azurerm-ps.md
:white_check_mark:Succeeded
View (azurermps-4.4.1)

docs-conceptual/azurermps-4.4.1/install-azurermps-maclinux.md
:white_check_mark:Succeeded
View (azurermps-4.4.1)

docs-conceptual/azurermps-4.4.1/other-install.md
:white_check_mark:Succeeded
View (azurermps-4.4.1)

docs-conceptual/azurermps-4.4.1/overview.md
:white_check_mark:Succeeded
View (azurermps-4.4.1)

docs-conceptual/azurermps-5.7.0/install-azurerm-ps.md
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-5.7.0/install-azurermps-maclinux.md
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-5.7.0/other-install.md
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-5.7.0/overview.md
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-6.10.0/install-azurerm-ps.md
:white_check_mark:Succeeded
View (azurermps-6.10.0)

docs-conceptual/azurermps-6.10.0/install-azurermps-maclinux.md
:white_check_mark:Succeeded
View (azurermps-6.10.0)

docs-conceptual/azurermps-6.10.0/other-install.md
:white_check_mark:Succeeded
View (azurermps-6.10.0)

docs-conceptual/azurermps-6.10.0/overview.md
:white_check_mark:Succeeded
View (azurermps-6.10.0)

docs-conceptual/includes/az-replacing-azurerm.md

Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-conceptual/includes/az-replacing-azurerm.md
:warning:Warning
View (azurermps-4.4.1) View (azurermps-5.7.0) View (azurermps-6.10.0)
Details

docs-conceptual/azurermps-4.4.1/install-azurerm-ps.md
:white_check_mark:Succeeded
View (azurermps-4.4.1)

docs-conceptual/azurermps-4.4.1/install-azurermps-maclinux.md
:white_check_mark:Succeeded
View (azurermps-4.4.1)

docs-conceptual/azurermps-4.4.1/other-install.md
:white_check_mark:Succeeded
View (azurermps-4.4.1)

docs-conceptual/azurermps-4.4.1/overview.md
:white_check_mark:Succeeded
View (azurermps-4.4.1)

docs-conceptual/azurermps-5.7.0/install-azurerm-ps.md
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-5.7.0/install-azurermps-maclinux.md
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-5.7.0/other-install.md
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-5.7.0/overview.md
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-6.10.0/install-azurerm-ps.md
:white_check_mark:Succeeded
View (azurermps-6.10.0)

docs-conceptual/azurermps-6.10.0/install-azurermps-maclinux.md
:white_check_mark:Succeeded
View (azurermps-6.10.0)

docs-conceptual/azurermps-6.10.0/other-install.md
:white_check_mark:Succeeded
View (azurermps-6.10.0)

docs-conceptual/azurermps-6.10.0/overview.md
:white_check_mark:Succeeded
View (azurermps-6.10.0)

docs-conceptual/includes/az-replacing-azurerm.md

Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).
Line 16: [Warning] Invalid file link:(~/docs-conceptual/includes/TBD).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Error the ssll connection could not be estalished

Just try again after a while.

Just try again after a while.


wip: Duplicated entries across various falco rules files

Motivation
See https://github.com/falcosecurity/rules/pull/149#issuecomment-1705527047
Feature

Alternatives

Additional context

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

/remove-lifecycle stale

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Likely, the simplest way to address this is to define a standard way to express list and macro dependencies requirements.

It makes the most sense. Worth it the transition LOE I would say. Not having any better ideas. Right now for example I dedup the macros and lists in a custom patch script, but there are adopters who wish to not needing to use a custom patch script.

Just as a reference. The new idiomatic way to express "this list is defined somewhere else - ie, in another file" would now be:
- list: foo
  items: []
  override:
      - items: append

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

/remove-lifecycle stale
/help

@leogr:
This request has been marked as needing help from a contributor.
Please ensure the request meets the requirements listed here.
If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-help command.

In response to this:

/remove-lifecycle stale
/help

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-sigs/prow repository.

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

/remove-lifecycle stale


[Bug]: What is the purpose of the Implicit Option in the Consent Recorder

Contact Details
<EMAIL_ADDRESS>Section Number
53.2 - PCF Actor Options
What is wrong
The Implicit Option is an option on the Consent Recorder. However, for the implicit option, nothing is being recorded, so it is not clear what it means for the Consent Recorder to support the Implicit Option.
Describe the solution you'd like
Remove the Implicit Option from the Consent Recorder.
Relevant log output
No response
Priority
{"Medium"=>"Significant issue or clarification. Requires discussion, but should not lead to long debate."}
Code of Conduct

[X] I agree to follow the IHE Code of Conduct

remove from Consent Recorder

make clear 108 is only required by the decider when it has explicit option support.


provide a proper Revision when converting k8s to crd NetworkPolicies

Description
This is a potential fix for https://github.com/projectcalico/typha/issues/335 . The gist of this bug report is that Typha fails to create a watch for a k8s NetworkPolicy because it fails to parse the Revision it finds in a KVPair:
Failed to create watcher
  ListRoot="/calico/resources/v3/projectcalico.org/networkpolicies"
  error=ResourceVersion is not valid: 346124214

The test I added is very much on the unit test side of things. I deployed it to one of our toy clusters, where it appears to work. I.e. it doesn't complain about failing to setup the watch anymore and the List from the full resync also disappears. The network policy is honored. I have not run the acceptance/integration testsuite locally because it fails for me even on master. The tests were done with Typha 3.7.5 and this patch applied.
As for the list of components, it's NetworkPolicy for sure and all its callers/users. Typha obviously, but so is everything using k8s NetworkPolicy conversion.
Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note
The fix is an optimization in some sense, since the failure to parse the revision caused a full resync. The resync works, so the NetworkPolicies are applied. Not sure if that warrants a release note.
fix Kubernetes NetworkPolicy conversion bug (caused unnecessary resyncs in e.g. Typha)

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Regarding the CLA: It demands some rights which only my boss has been granted by the org. He's willing to sign (in fact he already has), but the CLA Assistant does not offer any way to sign as a company or sign for other accounts. Well, at least we couldn't find it in the CLA's GUI. Please advise.

@spikecurtis Yes, I am working for "New Work SE" (https://github.com/orgs/new-work/people) and so is my boss @skaes.

@spikecurtis I am authorized to sign agreements on behalf of our company, New Work SE. Since today, I'm also co-owner of the new-work organization and we only accept New Work SE employees as members of the org. Can you please let me know how to proceed?

@breunigs thanks for this PR.  I've built on top of it to provide a more robust fix. The bug was actually a little deeper in our code and you can see the fix on this PR: #1192
Closing in favor of that PR, which includes your commit.


Skillmaps failing to load (Urgent)

In the last two days, 10 of our 21 feedback items have shown skillmaps not loading. This is a super-urgent issue, because if it happens to one of the reviewers from Code.org, we might not place at all this year.
Can someone take a look at the Usabilla tickets and exception logs to see if we can get this fixed ASAP?

https://app.usabilla.com/member/#/websites/buttons/feedback/?start_date=2022-09-11T00:00:00%2B00:00&end_date=2022-10-11T23:59:59%2B00:00

I will try to dig up the exceptions.

I think I've figured it out.
I've been testing my extensions on my chrome browser for literally days (even updated chrome recently) and today I noticed that it was still on Arcade version 1.8.x, which was breaking things for me. Regular reloads weren't causing an update. I had to shift+refresh.
Is there a way to force browsers to update?

And even weirder…when I clicked "reload" from my project, it went back to 1.8.x even after using 1.10.x

no repro.


invalid token causes stack trace

When calling Gitlab.deploy_keys with a token that has expired I get the following result.
from /opt/embedded/lib/ruby/gems/2.4.0/gems/gitlab-4.3.0/lib/gitlab/error.rb:40:in `build_error_message'
        from /opt/embedded/lib/ruby/gems/2.4.0/gems/gitlab-4.3.0/lib/gitlab/error.rb:16:in `initialize'
        from /opt/embedded/lib/ruby/gems/2.4.0/gems/gitlab-4.3.0/lib/gitlab/request.rb:64:in `exception'
        from /opt/embedded/lib/ruby/gems/2.4.0/gems/gitlab-4.3.0/lib/gitlab/request.rb:64:in `raise'
        from /opt/embedded/lib/ruby/gems/2.4.0/gems/gitlab-4.3.0/lib/gitlab/request.rb:64:in `validate'
        from /opt/embedded/lib/ruby/gems/2.4.0/gems/gitlab-4.3.0/lib/gitlab/request.rb:44:in `block (2 levels) in <class:Request>'
        from /opt/embedded/lib/ruby/gems/2.4.0/gems/gitlab-4.3.0/lib/gitlab/client/projects.rb:340:in `deploy_keys'
        from /opt/embedded/lib/ruby/gems/2.4.0/gems/gitlab-4.3.0/lib/gitlab.rb:25:in `method_missing'

The value of the hash passed into ObjectifiedHash is
{"error"=>"invalid_token", "error_description"=>"Token is expired. You can either do re-authorization or token refresh."}
Note: this only happens with an expired token

rake aborted!
ArgumentError: wrong number of arguments (given 2, expected 1)
/home/john/.rvm/gems/ruby-2.3.6/gems/gitlab-4.3.0/lib/gitlab/objectified_hash.rb:26:in `method_missing'
/home/john/.rvm/gems/ruby-2.3.6/gems/gitlab-4.3.0/lib/gitlab/error.rb:40:in `build_error_message'
/home/john/.rvm/gems/ruby-2.3.6/gems/gitlab-4.3.0/lib/gitlab/error.rb:16:in `initialize'
/home/john/.rvm/gems/ruby-2.3.6/gems/gitlab-4.3.0/lib/gitlab/request.rb:64:in `exception'
/home/john/.rvm/gems/ruby-2.3.6/gems/gitlab-4.3.0/lib/gitlab/request.rb:64:in `raise'
/home/john/.rvm/gems/ruby-2.3.6/gems/gitlab-4.3.0/lib/gitlab/request.rb:64:in `validate'
/home/john/.rvm/gems/ruby-2.3.6/gems/gitlab-4.3.0/lib/gitlab/request.rb:44:in `block (2 levels) in <class:Request>'
/home/john/.rvm/gems/ruby-2.3.6/gems/gitlab-4.3.0/lib/gitlab/client/groups.rb:27:in `group'
/home/john/dev/gitlab.com/cogli/gitlab-reporter/lib/gitlab/reporter.rb:26:in `build_report!'
/home/john/dev/gitlab.com/cogli/gitlab-reporter/Rakefile:13:in `block in <top (required)>'
/home/john/.rvm/gems/ruby-2.3.6/bin/ruby_executable_hooks:15:in `eval'
/home/john/.rvm/gems/ruby-2.3.6/bin/ruby_executable_hooks:15:in `<main>'
Tasks: TOP => build_report
(See full trace by running task with --trace)

The question is should ObjectifiedHash actually support Hash-like access in the way attempted on that line?

No.
Also seems like a duplicate of #388

I pushed a branch with a fix. @logicminds @johnmcconnell - it would be nice if one of you could try it out. Thanks.


CLOUDP-67656: describe DBuser

Proposed changes

I think this may have been missing from the previous pr, but it's necessary for https://jira.mongodb.org/browse/CLOUDP-67656. So I just lumped them together.
Jira ticket: CLOUDP-67656

Closes #[issue number]
Checklist

[] I have signed the MongoDB CLA
[] I have added tests that prove my fix is effective or that my feature works
[] I have added any necessary documentation (if appropriate)
[] I have run make fmt and formatted my code

Further comments
Could probably hide the authDB if we got the user first, but then this would be another round trip. So I went for the explicit approach
./bin/mongocli atlas dbusers describe arn:aws:iam::358363220050:user/mongodb-aws-iam-auth-test-user --authDB \$external
{
  "databaseName": "$external",
  "ldapAuthType": "NONE",
  "x509Type": "NONE",
  "awsIAMType": "USER",
  "groupId": "5cac6a2ba6f239298a18a025",
  "roles": [
    {
      "roleName": "atlasAdmin",
      "databaseName": "admin"
    }
  ],
  "username": "arn:aws:iam::358363220050:user/mongodb-aws-iam-auth-test-user"
}

./bin/mongocli atlas dbusers describe new --authDB admin
{
  "databaseName": "admin",
  "ldapAuthType": "NONE",
  "x509Type": "NONE",
  "awsIAMType": "NONE",
  "groupId": "5cac6a2ba6f239298a18a025",
  "roles": [
    {
      "roleName": "backup",
      "databaseName": "admin"
    }
  ],
  "username": "new"
}

that authDB change fixed it


Verify Incremental works with dask.dataframe

It does not
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-23-505152de83da> in <module>()
----> 1 inc.fit(X2, y2)

~/sandbox/dask-ml/dask_ml/wrappers.py in fit(self, X, y, **fit_kwargs)
    287     def fit(self, X, y=None, **fit_kwargs):
    288         check_scoring(self.estimator, self.scoring)
--> 289         result = fit(self.estimator, X, y, **fit_kwargs)
    290
    291         # Copy the learned attributes over to self

~/sandbox/dask-ml/dask_ml/_partial.py in fit(model, x, y, compute, **kwargs)
    167     if y is not None:
    168         assert y.ndim == 1
--> 169         assert x.chunks[0] == y.chunks[0]
    170     assert hasattr(model, 'partial_fit')
    171     if len(x.chunks[1]) > 1:

~/sandbox/dask/dask/dataframe/core.py in __getattr__(self, key)
   2436             return new_dd_object(merge(self.dask, dsk), name,
   2437                                  meta, self.divisions)
-> 2438         raise AttributeError("'DataFrame' object has no attribute %r" % key)
   2439
   2440     def __dir__(self):

AttributeError: 'DataFrame' object has no attribute 'chunks'

We have two options:

Just get the array with .values in Incremental.fit
Update _partial.fit's check to handle dataframes or arrays

We probably want option 2, since then we can verify that the divisions actually match.

If we're adding in an if-else tree then I'll also request a special case be
given to List[Delayed] or List[Future]
On Thu, Jun 21, 2018 at 3:57 PM, Tom Augspurger<EMAIL_ADDRESS>wrote:

It does not

AttributeError                            Traceback (most recent call last)
 in ()
----> 1 inc.fit(X2, y2)
~/sandbox/dask-ml/dask_ml/wrappers.py in fit(self, X, y, **fit_kwargs)
287     def fit(self, X, y=None, **fit_kwargs):
288         check_scoring(self.estimator, self.scoring)
--> 289         result = fit(self.estimator, X, y, **fit_kwargs)
290
291         # Copy the learned attributes over to self
~/sandbox/dask-ml/dask_ml/_partial.py in fit(model, x, y, compute, **kwargs)
167     if y is not None:
168         assert y.ndim == 1
--> 169         assert x.chunks[0] == y.chunks[0]
170     assert hasattr(model, 'partial_fit')
171     if len(x.chunks[1]) > 1:
~/sandbox/dask/dask/dataframe/core.py in getattr(self, key)
2436             return new_dd_object(merge(self.dask, dsk), name,
2437                                  meta, self.divisions)
-> 2438         raise AttributeError("'DataFrame' object has no attribute %r" % key)
2439
2440     def dir(self):
AttributeError: 'DataFrame' object has no attribute 'chunks'
We have two options:

Just get the array with .values in Incremental.fit
Update _partial.fit's check to handle dataframes or arrays

We probably want option 2, since then we can verify that the divisions
actually match.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask-ml/issues/220#issuecomment-399223950, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AASszOyigTJS11n20-7O61GVEQlRO18Lks5t-_q1gaJpZM4Uyqrr
.

Is there any update on this? I am trying to find a workaround to using dask.dataframes but Im not sure how I can  handle a pipeline that has columns and make it work with a dask.array.
As of versions:
dask_ml=0.11.0
dask=0.18.2

Doing something like this:
pipeline = make_pipeline(
    Categorizer(columns=["VendorID", "store_and_fwd_flag", "payment_type", "RatecodeID", "PULocationID"]),
    DummyEncoder(columns=["VendorID", "store_and_fwd_flag", "payment_type", "RatecodeID", "PULocationID"]),
    StandardScaler(),
    SGDRegressor(learning_rate='invscaling', loss='squared_loss', penalty="l1")
)
pipeline = Incremental(pipeline)

Fails with the same error AttributeError: 'DataFrame' object has no attribute 'chunks'
Without getting deep into the codebase (please disregard if I am completely lost), I would agree with @TomAugspurger  in getting the .values for each it, since that would be aligned with sklearn behavior


[WIP] Konnect nav redux + intro draft

Summary
Exploring options for an improved user navigation experience in the Konnect docs.

Removed the "plan" section - after some review, we realized that it was basically a dump of content that didn't seem to belong anywhere.

From the "plan" section: two topics were basically stubs/snippets: Licensing and Architecture. The Architecture topic also contained browser and Gateway compatibility info, which didn't make much sense.
Pulled compatibility into a separate topic.

Renamed Publish to Dev Portal
Removed the "Configure" section, which previously contained Service Hub and Runtime Manager. Both of those sections are now top-level items.
Temporary intro topic to replace the very old one, which was just a list of links.

The module descriptions are old and will definitely need revising.
Updated the diagram to include runtime groups.
Pulled the licensing and architecture info into the intro temporarily. It may or may not stay here, we'll see.

Things I'm really not sure about:

The Introduction section has a bunch of reference content and an FAQ; don't really know where to put it.
Release notes are also in the intro section. Thoughts: we might want to have a full release note section of the doc site (eg docs.konghq.com/release-notes).

Inspiration: Google cloud release notes

Reason
We reworked the Konnect nav for the preview a few months ago, and are now following up. Some things worked; some things didn't.
https://konghq.atlassian.net/browse/DOCU-2325
Testing
https://deploy-preview-3979--kongdocs.netlify.app/konnect/
Walk through the nav and look for friction points.

Ignore the broken links for now; I'll update them + add redirects once we've settled on a nav.

I think we could add an Architecture page
@mheap Do you have any particular existing content in mind for this page (eg the last section of the intro), or do you want a totally new doc?


There is no way of transfering functions and/or objects with associated methods to client code during SSR in SvelteKit, but it is present in Svelte with Server-side component API

Describe the problem
i.e. I want to log errors during SSR to Sentry while having access to RequestEvent parts.
Describe the proposed solution
Use Svelte Server-side component API in SvelteKit via passing context to client code during SSR.
Alternatives considered

Ignoring errors during SSR as same errors will be logged via client side logging. Sounds... sad.
Passing all required options down to client-side code to create server-side logger only during SSR. Can be quite cumbersome.

Importance
would make my life easier
Additional Information
Proposed solution has open PR – https://github.com/sveltejs/kit/pull/10064.
Additional info from there:
@secondfry:
Sometimes you need a function defined only in server context – i.e. to log errors inside of SSR. This PR allows one to define context in RequestEvent as follows:
hooks.server.ts
const handle: Handle = async ({ event, resolve }) => {
  const logger = new Logger();
  event.context = new Map();
  event.context.set('logger', logger);
  return await resolve(event);
});

some +page.svelte
import { browser } from '$app/environment';
const logger = browser ? new ClientLogger() : getContext('logger');
logger.info('yes');

@benmccann:

const logger = browser ? new ClientLogger() : getContext('logger');

I feel like there's not enough context here to understand this PR and there should probably be an associated issue. Why can you do new ClientLogger() but can't do new ServerLogger()?
@secondfry:
@benmccann in case of very simple logging requirements, yes, maybe one can do that. But the moment you will need to transfer such logs to i.e. Sentry, with request context, timings, etc., you will question yourself: why can't you just pass already initialized logger from server side?
Expanding on the example you would have something like this:
hooks.server.ts
const initLogger = async ({ event, resolve }) => {
  event.locals.logger = new QuiteComplicatedServerLogger({ endpoint, format, ... });
  return await resolve(event);
};

const quiteComplicatedThingWhichRequiresLoggerA = async ({ event, resolve }) => {
  try { await explode(); } catch (cause) { event.locals.logger(new Error('Unexpected explosion', { cause })); }
  return await resolve(event);
};
const quiteComplicatedThingWhichRequiresLoggerB = ...;
const quiteComplicatedThingWhichRequiresLoggerC = ...;

const handle: Handle = sequence(
  initLogger,
  quiteComplicatedThingWhichRequiresLoggerA,
  quiteComplicatedThingWhichRequiresLoggerB,
  quiteComplicatedThingWhichRequiresLoggerC,
  async ({ event, resolve }) => {
    event.context = new Map();
    event.context.set('logger', event.locals.logger);
    return await resolve(event);
  }
);

Otherwise one would have to pass all options of QuiteComplicatedServerLogger to svelte code.
So as Svelte has Server-side component API which allows passing context it seemed to us as the most logical choice. If there are other ways to pass functions and/or objects with associated methods into client code during SSR, I'm would gladly learn that.

I think this is reasonably simple already:

Create a module that returns the logger
Attach the logger to a context in +layout.svelte

For example:
// $lib/logger

const loggerContextKey = new Symbol('logger');

export function createLogger(config) {
  return browser ? new ClientLogger(config) : new ServerLogger(config);
}

export function createLoggerContext(logger) {
  setContext(loggerContextKey, logger);
}

export function getLogger() {
  return getContext(loggerContextKey);
}

<script>
  // routes/+layout.svelte
  import { createLogger, createLoggerContext } from `$lib/logger`;

  createLoggerContext(createLogger);
</script>

<slot />

You'll have to adjust the module depending on whether you need your logger scoped to the request (you can simplify if you can share an instance between all requests on the server), but that should get you pretty much all the way there.
(disclaimer: typed on phone)

Agree with @tcc-sejohnson that should be possible to do with existing methods. Additionally, if you need some initial value passed to the logger context you could always return that from your load function.
/// file: +layout.js
import { browser } from '$app/environment';

export function load() {
  return {
    stuff: 'foo',
    loggerStuff: browser ? undefined : something
  }
}

<!-- file: +layout.svelte -->
<script>
  export let data;

  createLoggerContext(logger, data.loggerStuff);
</script>

Is there a use case where this wouldn't work which I'm missing?

Setting/getting logger via context is too late, such logger won't be available in hooks since contexts are only creatable from Svelte components.


Oracle: Limit IN-Lists to 1000 elements

$ids = range(1,2000);
$bookQuery = BookQuery::create()->filterById($ids);
$bookQuery->find();

will generate the following SQL, which is not valid for Oracle:
SELECT  FROM book WHERE book.id IN (:p1,:p2, ... ,:p2000)

IN-lists are limited to 1000 elements by Oracle. Statements with longer IN-Lists will always fail with "ORA-01795 maximum number of expressions in a list is 1000".
Propel should work around this limitation, e.g. by splitting up the in lists in blocks of 1000:
SELECT  FROM book WHERE (book.id IN (:p1,:p2, ... ,:p1000) OR book.id IN (:p1001, :p1002, ..., :p2000))

Not sure this is worth a workaround. Whats your use-case for selecting that much rows with one query?

We used this in lists workaround before and are migrating to propel.
There are different use cases, but one the deletion of some objects.
There is some business logic involved in identify the elements to be deleted. For sure something like a subselect might also be possible, but I would like to avoid that because of testability reasons.

You could also array_chunk and delete in 1000 row batches

Yes, for delete that would be possible and even better would be to use OCI-Collections for that (but thats not possible for Propel IMO, because you need a custom named type for that).
Since object deletion might be triggered by save() its not trivial to split the query up using array_chunk: I guess the termination methods (find/delete/update) must be overwritten and the query must be clone with fewer elements. That sounds like a hard thing to do.
I really think the easiest solution is to fall back to multiple IN-conditions in that case. This would work for all use cases (delete/find/update/aggregate data using group by etc) and I don't see any side effects.
Everything else will just mean don't use filterByXXX with arrays with oracle, because it might fail with growing input data.
One would need to carefully investigate if the array passed to filterByXXX can get larger than 1000 elements (which is theoretically always the case i guess).

There are always limitations, even with max allowed placeholders in prepared statements. So, not using filterByXXX in all cases just to bypass one of those limitations is not a good advice. Fixing that by Propel is rather hard since those Criteria objects don't have a direct way of getting the Adapter being used for the current connection. That means we need to make all Criteria classes aware of its current environment. I'm not sure if this is the right way to go. There may be other ways to fix that, but I believe using the generator to generate different code is even more complex.
To keep this simple, you could just overwrite those filterBy methods you know that can get more than 1000 items.
public function filterById($value) {
    if (is_array($value) && count($value) > 1000)(
        foreach (array_chunk($value, 1000) as $ids) {
            $this->filterById($ids);
        }
        return $this;
    } else {
        return $this->filterById($value);
    }
}

As an workaround I implemented it like that (OR is missing). I would prefer an solution that covers all cases, but I understand that fixing this edge case is Propel not that trivial (and my use case is part of the problem), so I guess I have to live with that.
Thanks anyway!

Alright. We reopen this issue when the need increases.


Add rivine Create and Open wallet convenience methods

Can you please add more information to the ticket describing what is the purpose of these new methods and how they will be used by client code.

merged to development_simple


Modification question

Hi necolas, don't have other way to message you, but quick question, I want to modify this plugin so that while enabled won't allow more tabs, even if I exit others. Mostly want to use it while widdling down open tabs. Do you have a suggestion of where I can modify this to have it recursively change the tabs allowed to the current total. Would be awesome, thanks!

Hi, I don't understand the question / request

I'd like to make this so that the plugin stops you from opening new tabs
completely, even after closing other ones. If I use this now and have 10
tab limit, then close 5 tabs, I can then open 5 more. I want it to keep
moving the limit downward so that I can't open more even after closing
some. Is that possible?
On Tue, Mar 21, 2017 at 3:14 PM Nicolas Gallagher<EMAIL_ADDRESS>wrote:

Hi, I don't understand the question / request
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/necolas/chrome-tab-limit/issues/1#issuecomment-288236404,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABjW6YwaHnNrThb2I6dAgAI1pANzzP-pks5roEvegaJpZM4Mkci1
.

--
Eric Matzner
CEO and Founder
*, **NatMobi http://NatMobi.com/, Protact http://protactapp.com & *VIP
Texts http://viptexts.com,
Office: 568 Broadway, 11th Floor,
New York, NY 10012
(Broadway X Prince)
Mobile:<PHONE_NUMBER>
Office: 1-888-451-0013 x 9

That's probably possible but not something I'm going to do here. You can fork the repo


TextField IME candidate list pane invisible on Windows

target is windows native
The cause of this issue is that the SDL version is too old.
openfl/libsdl/blob/master/src/video/windows/SDL_windowskeyboard.c
videodata->ime_uiless = UILess_SetupSinks(videodata);

This line of code causes the IME candidate pane to not be rendered.
and this issue has been fixed in the latest version of SDL:
libsdl-org/SDL/blob/main/src/video/windows/SDL_windowskeyboard.c
if (WIN_ShouldShowNativeUI()) {
    videodata->ime_uiless = SDL_FALSE;
} else {
    videodata->ime_uiless = UILess_SetupSinks(videodata);
}

See #1684 for the work that's been done on this, though it does seem to have stalled.

Is it possible to fix openfl/libsdl with temporary patch?
The code below works fine on my windows system:
#if defined(__WIN32__)
videodata->ime_uiless = SDL_FALSE;
#else
videodata->ime_uiless = UILess_SetupSinks(videodata);
#endif

Sure, submit a PR.

It is probably worth mentioning that the 8.2.0-Dev branch switched from openfl/libsdl to libsdl-org/SDL. Any further changes that we make to openfl/libsdl will be lost with the Lime 8.2 update.

However, I see that support for the Windows IME window was added in commit libsdl-org/SDL@6f972052296fdfe63ecaa7edab3e5b4e663eea06. I see that's part of SDL's release-2.0.18 tag. The 8.2.0-Dev branch appears to be on release-2.30.0, so this fix will be included in Lime 8.2 already.

Oh whoops, I got the dates wrong. I thought 8.2.0-Dev didn't have the fix, so I didn't mention it.

I have tried 8.2.0-Dev branch and found that SDL_HINT_IME_SHOW_UI is not enable by default.
The following code needs to be added to the SDLWindow::SDLWindow() function to display the IME candidate list:
#if defined (HX_WINDOWS)
SDL_SetHint (SDL_HINT_IME_SHOW_UI, "1");
#endif

That feels like something most users won't need.
But SDL_SetHint() is declared extern; I wonder if that means you can get at it using CFFI? Then you'd be able to set whatever hints you needed. Sadly I don't know enough about CFFI to say for sure.


[New Rule] (libnikka.so)

Library filename / 库文件名
libnikka.so
Library label / 库的文字标签
Nikka
Library team / 库的开发团队
Nikka
Files & Comment / 同组文件 & 备注
provided to hook c# methods in unity based games
In Apps / 出现于
Standoff 2
Library icon / 库图标
No response
Library description / 库描述
c# hooking library
Library relative URL / 相关链接
?

Invalid relative URL

Invalid relative URL

the library doesn't have relative URL

We cannot verify the validity of this rule without a link to the relevant explanatory documentation.


Monarch documentation/sample is out of date - please add dynamic brace matching

+1
I'm struggling to add highlighting of matching brackets. If you're adding a custom language and using the brackets property to setMonarchTokensProvider or using open/close in token matching. What else needs to be done to get the it highlighted.

Please add the following line. (note: my custom language needed only braces to be matched (bold/hightlight below)
please refer monaco.d.ts for more details
monaco.languages.setLanguageConfiguration('measureExplorerLang', {brackets:[['(',')']]});

Also, don't forget "autoClosingPairs" just for fun and profit.

I would like to work on this. I'm not entirely sure where to put that new line, so I'll put up a WIP PR and you can let me know if I put it in the wrong place.


Parts list

Cool project! I would like to build this but I'm struggling with the value of resistor R1 on your schematic 1B0E ?

@That-Dude Thanks for your feedback! Resistor R1 is 180Ohm. Sorry, it is a typo. For more information, please refer to https://github.com/Erriez/ESPHomePCPowerControlHomeAssistant/issues/1


How Do I upload a bootloader to a brand new ESP32-S3 chip on my newly designed and produced Board? (IDFGH-9312)

Hello Everyone,
We made our own board with ESP32-S3 (FN8 Model with 8 MB Flash internal). We see bootloop when we first turn it on. After looading bootloader.bin (and many different *.bin files) we still see bootloop problem and we get the following messages from COM port. We use Windows CMD for esptool.exe to load *.bin files.
ESP-ROM:esp32s3-20210327
Build:Mar 27 2021
rst:0x7 (TG0WDT_SYS_RST),boot:0x8 (SPI_FAST_FLASH_BOOT)
Saved PC:0x40043ac8
SPIWP:0xee
mode:DIO, clock div:1
load:0x3fcd0108,len:0x1634
load:0x403b6000,len:0xe74
load:0x403ba000,len:0x31c8
Checksum failure. Calculated 0x8e stored 0xde
ets_main.c 329
... (loops) ...
How can we solve this problem?
Any help is appreciated.
Thank You

Check the firmware bins size. Does the size match the partition table?
Try earse the flash before flashing.
Try change CONFIG_SPIRAM_MALLOC_ALWAYSINTERNAL.

Otherwise, try searching 'Checksum failure' on github, can get lots of help.

Thankyou Shentu.

I think, there should not be size problem. Because we have this problem even after I erase all flash.

I erased the flash:

C:\Pub\BootLoader>esptool -p COM39 --chip esp32s3 erase_flash
esptool.py v4.4
Serial port COM39
Connecting...
Chip is ESP32-S3 (revision v0.1)
Features: WiFi, BLE
Crystal is 40MHz
MAC: f4:12:fa:68:2d:98
Uploading stub...
Running stub...
Stub running...
Erasing flash (this may take a while)...
Chip erase completed successfully in 0.9s
Hard resetting via RTS pin...
When I check serial port COM39:
invalid header: 0xESP-ROM:esp32s3-20210327
Build:Mar 27 2021
rst:0x15 (USB_UART_CHIP_RESET),boot:0x8 (SPI_FAST_FLASH_BOOT)
Saved PC:0x40048839
invalid header: 0xffffffff
invalid header: 0xffffffff
invalid header: 0xffffffff
--------(loops)----------
When I checked Device Manager, COM39 disaappear and comes back with a sound from pc speaker continiously.

Can you help me how to change CONFIG_SPIRAM_MALLOC_ALWAYSINTERNAL?

By the way we have two flash on the board. First one is 8 MB internal flash of ESP32-S3 FN8 Model Chip. Second one is external 16 MB (W25Q128JVSIQ) beside the ESP32-S3 chip on the board. Is this may be source of problem?
Thank you very much for helping.
Senel

May I know what flash you are using? You can get this information via esptool.py flash_id

Thanks for reporting, will close due to short of feedback, feel free to reopen with more updates, thanks.


Updated Unit Tests

Updated approach to unit testing
Unit tests can be read as examples for most part. But I will not be continuing directly on unit tests, as that is honestly a bit boring. Instead, will improve the tests with code refactoring, feature implementations and part by part implementing it.
Doing it in one go was not so fun.

Codecov Report
Attention: Patch coverage is 94.37751% with 14 lines in your changes are missing coverage. Please review.

Project coverage is 98.43%. Comparing base (272906f) to head (dee6be6).

Files
Patch %
Lines

tests/test_deconfig/test_deconfig.py
93.20%
14 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##              main      #14      +/-   ##
===========================================
- Coverage   100.00%   98.43%   -1.57%
===========================================
  Files            5        5
  Lines          874      895      +21
===========================================
+ Hits           874      881       +7
- Misses           0       14      +14

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


List optional deps as features

chrono doesn't have an explicit feature for serde

whatfeatures chrono

chrono/0.4.7
  default: clock
  clock: time

whatfeatures chrono -d

chrono/0.4.7
  features
    default: clock
    clock: time
  dependencies
    normal
      libc              = ^0.2
      num-integer       = ^0.1.36
      num-traits        = ^0.2
      optional
        rustc-serialize = ^0.3.20
        serde           = ^1
        time            = ^0.1.39
    dev
      bincode           = ^0.8.0
      doc-comment       = ^0.3
      num-iter          = ^0.1.35
      serde_derive      = ^1
      serde_json        = ^1

instead its an optional dep. see Cargo.toml

I don't know what I was saying here. so closing the issue until I figure out what this is implying

Oh, I see.
the default behavior of listing features should list optional crates as features


Ormar p2

Cleaned ormar db adapter with all up to date ormar features
Finished ormar db with OrmarBaseOAuthAccountModel
Added test for ormar db and ensure all pass
Bumped dependency to ormar >=0.9.0,<0.10.0

So I guess now what is left:

More tests including some sample fastapi aplications
Add sections regarding ormar to documentation in all relevant parts (like models, oauth etc.)

Let me know what you think about the changes.
Also included some columns in the code.

Thank you very much for your revision! You did the most of the work (btw, I was stuck on tests)
I will:

write some docs along the lines of FastAPI Users SQLAlchemy/MongoDB/Tortoise ORM docs, they share a pretty similar structure
tests: for that I'd probably ask to FastAPI Users maintainer when I'll do the PR, but looking at their existing test suite I think the tests you wrote could/should be ok

Sorry for bothering with some off-topic, but what should I do now? Is it ok to commit the missing parts (basically the docs), squash the commits into one and then submit the PR?

No need to be sorry! :)
You should merge this one (can be into master since it's your fork anyway).
Pull the changes from master of this fork to your local repo (after merge ofc).
Checkout new branch with descriptive name and add missing docs etc (that way it will be easier for maintainer to see what you want to merge and you can always push to this new branch to update if owner request changes.)
Optionally squash the commits. (Before the push, never squash already pushed commits, you can mess other people git history)
Open pull request from your fork to original repo.
Saw that someone opened an issue for gino, you didn't open one for ormar but since the owner said all backends are welcomed it's should be fine🙂

Btw i forgot to change version of ormar in tool.flit.metadata.requires-extra in toml file to use version >=0.9.0,<0.10.0 so that should also be updated


upgrade datastax dependency

We currently use 4.15.0 of https://mvnrepository.com/artifact/com.datastax.oss/java-driver-core
This has a dependency on Jackson that has CVEs. Also netty dependency with CVEs.
I propose upgrading to 4.17.0. But we probably need to add an explicit dependency on jackson because datastax 4.17.0 still relies on a Jackson release with CVEs (newer but not new enough).
@mdedetrich @nvollmar wdyt?

I think we should keep 4.15.0 until we can do some more extensive testing with the new driver.
Meanwhile excluding the dependency and adding an explicit dependency to match the versions used in other Pekko projects would make sense.


Flicker that causes loss of focus on input when clicking on next input

Describe the bug
Hello guys, I have a React 18, NextJS 13 application.
I am using the Form class to render a pdf file along with inputs that correspond to a schema I have generated.
My problem is that if I type something on a random input, and then click on another, I will get a flicker and this flicker will cause loss of focus, meaning I have to click again. Most of the times, the second click isn't enough and I click 3 times just to be sure.
This is a very big problem for me because I have a form that contains approximately 20 input fields.
To Reproduce

Type on an input
Click on another input
You will see a re-render of the inputs
Click 1-2 more times to be able to type.

Expected behavior

I type on an input
I click on another
It is expected to be able to type something straight away.

Your Environment
- pdfme package(@pdfme/generator or @pdfme/ui):@pdfme/ui
- pdfme version: 4.0.2
- Operating system: Windows 10
- Node.js version or Browser name & version: Node 20.12.2, Firefox, Chrome

Your Error Log
Warning: Setting up fake worker.

** THE FOLLOWING HAPPENS every time a re-render happens for every field **
index.es.js:12091 Warning: TT: undefined function: 32

Additional context
No response

I should also note that this does not happen with 3.4.3 version.

Confirmed in the Form on the playground site using Certificate.
It only takes me one additional click though.
It looks like it's processing the change event on loss of focus and not focusing the new element.

Yes my guess was that it was trying to process the content of the input in case the user has written something.

This is due to a known issue here.
We're triggering a full re-render of the Form because a table would require this but it's applying to all other field types as well.
A clean design is needed to handle this, although there might be a short-term improvement that we could make.

This should be fixed now in the next release

Thanks alot!!

Hi @npyl
Released in version 4.1.1.
If the issue has been resolved in the latest version, please close this issue.


generate_latest() for MultiProcessing writes both the MultiProcess Metrics and per Process Metrics

Hi,
I guess the title says it all but using Gunicorn multiprocessing environment and creating a CollectorRegistry() that is passed into MultiProcessorCollector()... then doing generate_latest() on that registry, I get both keys for MultiProcess metrics and per Process Metrics. However, I would like to only see the MultiProcess Metrics and ignore the per Process Metrics. Is there a way to do that?
Thanks,
Mariam

Usage questions are best asked on https://groups.google.com/forum/#!aboutgroup/prometheus-users
It sounds like you're using the default registry rather than a custom registry as per the docs.


switch off curly rule

I've added the curly rule to also be off, as it might conflict with eslint

How can it conflict?

if you're using the multi-line option for the curly rule, which is for example the default in eslint-config-airbnb, the following would result in an eslint error
/*eslint curly: ["error", "multi-line"]*/

if (foo)
  doSomething();
else
  doSomethingElse();

prettier would wrap into a new line, if the line gets to long and therefore end up in an eslint-error

I see! Nice example!
The curly rule is still very useful for those who would like to enforce that curlies are always used with ifs and suchlike – the "all" option.
Would you like to update your PR add curly as a special rule instead? If you do, don’t forget to run npm test to verify that you got it all right.

Thanks!

Released in v1.6.0.


Cleanup and move to Node v7+ (async/await)

Updated to Node v7+ (async/await)
Added checks in gulpfile.js for users' Node versions
Updated every command to use async/await
Cleaned up several bugs, inconsistencies, and oddities in commands
Improved dynamic-imports manager to handle errors better
Updated commands manager to handle async/await commands
Upgraded all Node modules
Removed a bunch of Node modules we weren't using or didn't really need
Removed request module, we're down to 2 HTTP clients now (got and snekfetch)

I've changed the version checking in accordance with @abyssvi's wishes.


docs: Link Bazzite documentation for ujust...

... since older Universal Blue documentation on the Discourse forums are now considered deprecated.

Wait I have a better idea without losing anything!


API updates wip

Motivation

Solution

PR Checklist

[ ] Added Tests
[ ] Added Documentation
[ ] Updated the changelog

Codecov Report

Merging #40 (de04a04) into main (27b7568) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main      #40      +/-   ##
==========================================
- Coverage   31.95%   31.93%   -0.02%
==========================================
  Files          14       14
  Lines        5066     5069       +3
==========================================
  Hits         1619     1619
- Misses       3447     3450       +3

Flag
Coverage Δ

test
31.93% <0.00%> (-0.02%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/app.rs
18.18% <0.00%> (-0.57%)
:arrow_down:

src/server.rs
1.23% <0.00%> (-0.01%)
: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 27b7568...de04a04. Read the comment docs.


Add initial pages for QGIS UC 2025

Please wait with merging until next week when we will publish the CfP (will comment here when we are ready), but can be reviewed now.
Closes #423

The CfP is ready, so from our side this is ready to be merged!

I think that the page is important but is not easy to find as we have to navigate to Community > Project Organisation > Meetings > QGIS UC 2025. I submitted a PR earlier into the navigation bar that adds the link to the "Meetings" page (https://github.com/qgis/qgis-uni-navigation/pull/12) to easily find the events. What do you think about that?

I've toyed with a few ideas there, but always end up in the same place - it's hard to make the UC pages more easily findable without them taking to much space. One can also consider that previous years they were in entirely different places (i.e. different domains altogether) so even harder to find.
I think what is in place now (link from frontpage ("Join our annual conference"), sidebar menu, shortlink (/uc-2025), +your unibar PR) is as far as we can reasonably expect to get. Maybe when the tickets go on sale we can get a News-bar entry?

For the conference tabs, I think they look good, but I wonder if you had a chance to try the one that we usually use (e.g https://qgis.org/project/overview/#key-features).

I did consider it, however it felt like that one is more about switching between pieces of content on one page, rather than switching pages altogether. But if you disagree I can switch to that one.

For the images, I would generally suggest using the webp format for better performance and ensure that each one is showing somewhere. And perhaps with a rounded border to align them with the existing ones 😄 .

Done!

I think it would be nice to also have the possibility to add the event to the calendar. Using iCalendar (with the location), for example. But anyway, perhaps it's not really a good idea.

Sounds like a good idea to me, done!

Maybe when the tickets go on sale we can get a News-bar entry?

Yes, definitely.

I did consider it, however it felt like that one is more about switching between pieces of content on one page, rather than switching pages altogether. But if you disagree I can switch to that one.

I think it's fine, and it looks good to me.

For the images, I would generally suggest using the webp format for better performance and ensure that each one is showing somewhere. And perhaps with a rounded border to align them with the existing ones 😄 .

Done!

The image is not showing anymore. Please let me know if you need help converting it to .webp

Blargh, was to fast and forgot to add the webp file... (as well as the icalendar file)

LGTM too. @Xpirix  can you merge and publish the website?
I'm not sure if we have automatic publish on merge, but I think this will become urgent if we use the website for events organisations. Up to now, most events had their own website.

I've merged. For now, the deployment process is triggered every 12 hours from the server side. But I can deploy this right away.

I've deployed it. I've also submitted an additional fix for the Venue image which doesn't show correctly at #490

I've merged. For now, the deployment process is triggered every 12 hours from the server side. But I can deploy this right away.

12h is good enough by now. The perfect target should be a scheduled publication to handle the content coming from feeds and various scripts, and a publish on PR merge, just because Web content update often requires to publish fast.  No hurry for this however.


TerriaMap - release on v8/next

TerriaMap should be migrated to v8:

[ ] @AnaBelgun to re-organise the catalogue
[ ] @nf-s to migrate once catalogue is agreed (demo layers)
[ ] https://github.com/terriajs/saas/issues/168
[ ] https://github.com/terriajs/terriamap/issues/532
[ ] https://github.com/terriajs/terriamap/issues/530

This is done


feat: Add submit feedbacks

Fixes #1633
Changes: Added option to submit feedback
Screenshots for the change:

@nikit19 @liveHarshit please review

Also the feedback recyclerview isn't updating after feedback submission any ideas why ?

@liveHarshit review

Also the feedback recyclerview isn't updating after feedback submission any ideas why ?

Current logic used to prevent reloading of fragments on rotation is causing these bugs. #1520 is a similar bug.

@iamareebjamal
{
"data": {
"type": "feedback",
"attributes": {
"rating": "2.5",
"comment": "Awesome"
},
"relationships": {
"event": {
"data": {
"type": "event",
"id": "770"
}
},
"user": {
"data": {
"type": "user",
"id": "7625"
}
}
}
},
"included": [
{
"type": "user",
"id": "7625",
"attributes": {}
},
{
"type": "event",
"id": "770",
"attributes": {}
}
]
}
post request is adding included section as well any suggestions why it is happening

@iamareebjamal nm it was because of         onlineApiResourceConverter.enableSerializationOption(SerializationFeature.INCLUDE_RELATIONSHIP_ATTRIBUTES)

@nikit19 please review

@iamareebjamal @nikit19 updated

Build is failing

@iamareebjamal updated

Resolved Conflicts. Should I squash them first ?

@iamareebjamal updated


Example with FlatList

Is your feature request related to a problem? Please describe.
We are building a chat with your (awesome) package, and it works very smoothly. Thanks! However, at some point, we are experiencing performance issues due to the usage of ScrollView. Is FlatList also supported somehow?
Describe the solution you'd like
Add example using FlatList.
Describe alternatives you've considered
Using ScrollView, but the performance is bad.

@psteinroe What is the issues with FlatList do you experience?

@psteinroe What is the issues with FlatList do you experience?

That was a swift response! ❤ I am having troubles getting a chat to work. In your examples, you are inverting views, add fakeViews etc. All this is not really possible FlashList since it uses renderItem and does not take children.

@psteinroe for adding fakeView you can use ListFooterComponent/ListHeaderComponent. Inverting should work in the same way (i. e. just apply styles to parent view and content style).

that does not work for some reason. It results in some weird and indeterministic behaviours. It seems like applying transform to contentContainerStyle does not really work properly since any rotation is lost after navigating. Do you have any ideas? Otherwise, I will put this on hold, and try to create PR with a FlatList example to reproduce.

@psteinroe I don't have any ideas. This rotation was used to make ScrollView to scroll from bottom to top - you also can achieve this with setting up a ref and scrolling to the bottom - https://stackoverflow.com/a/42736127/9272042
Also if you are using FlashList, then you may use renderScrollComponent prop to specify your custom scroll component (i.e. just use example app code with all transformations/fake views).

@kirillzyusko thanks for the hints. I boiled it down to rotate: 180deg not being applied to the children of the ScrollView. I now ditched all the transforms and use scrollToBottom. This is not optimal as the scrolling process is visible to the user, but it works and is good enough for now. Would be great to find a solution for this, although I understand this is is unrelated to this project.
 const scrollViewStyle = useAnimatedStyle(
    () => ({
      height: '100%',
      backgroundColor: '#e2e8f0',
      transform: [{ translateY: height.value }, ...styles.inverted.transform],
    }),
    []
  );

  const textInputStyle = useAnimatedStyle(
    () => ({
      width: '100%',
      transform: [{ translateY: height.value }],
    }),
    []
  );

  const fakeView = useAnimatedStyle(
    () => ({
      height: Math.abs(height.value),
    }),
    []
  );

...

<Reanimated.FlatList
       data={elements ?? []}
        ...
        renderScrollComponent={({ children, ...props }) => (
          <Reanimated.ScrollView
            {...props}
            style={scrollViewStyle}
            showsVerticalScrollIndicator={false}
          >
            <View style={styles.inverted}>
              <Reanimated.View style={fakeView} />
              {children}
            </View>
          </Reanimated.ScrollView>
        )}
      />

@psteinroe you can show full-screen spinner, while the chat-component is getting mounted (it should overlap the content). In this case user will not notice scrolling process 🙂
The code that you've shared - doesn't it work? It seems like it should 🙂

@psteinroe you can show full-screen spinner, while the chat-component is getting mounted (it should overlap the content). In this case user will not notice scrolling process 🙂

thanks, will do!

The code that you've shared - doesn't it work? It seems like it should 🙂

it does not unfortunately. I played with this for a while, and <View style={styles.inverted}>...</View> does not apply the transform to children. I assume this is because of the way FlatList works... Could not figure out a way around that.

@psteinroe got it! I will have a look when I get time 👍

@MarceloPrado good point! This rotation was needed for 2 purposes:

to scroll list from the end to beginning;
to achieve smooth keyboard transition (since I'm adding fakeView in the beginning of the list to add space for scrolling - without this transformation the transition looks a little bit twitchy -> https://github.com/kirillzyusko/react-native-keyboard-controller/pull/76 and https://github.com/kirillzyusko/react-native-keyboard-controller/pull/112 for reference).

is there any fix for this? I'm trying to use it with a flatlist but none of the onmove or interactive events fire when using dismiss interactive

it would be great if the keyboard hook just gave the frame height position of the keyboard in ios without being related to any scrollview...is that possible?

Hello @timothyerwin
I don't think missing onMove/onInteractive events can depend on a FlatList usage. Most likely the problem is in something else - feel free to create a new issue with minimal reproduction example☺

onMove works fine when opening and closing the keyboard, but as soon as you try to interactively close the keyboard in a flatlist with keyboardDismissMode="interactive" it will not report any event at all.

it would be great if the keyboard hook just gave the frame height position of the keyboard as it moves interactively in ios without being related to any scrollview...is that possible?

If I understand your question correctly - that's exactly what keyboard hook does. You can try to run example app - I have an example with interactive keyboard there.

onMove works fine when opening and closing the keyboard, but as soon as you try to interactively close the keyboard in a flatlist with keyboardDismissMode="interactive" it will not report any event at all.

@timothyerwin Please, create a new issue with reproduction repo ☺

@kirillzyusko really appreciate your quick follow up last night. it really helped me dive in and figure out the problem. this is the perfect library for managing the keyboard frames in ios I must say. a real gem!
so, back to the problem...I was able to isolate it by adding a tab in my app with a minimal example after I ran your examples and saw it indeed worked as you said. once I got it working in my isolated example I figured out that for some strange reason I needed to add an explicit height on the view that contained the text input otherwise it would not report the interactive events. I'm not sure if that's mentioned anywhere here in the docs, or something that could be solved in this library but it's worth spelling out in the docs.
once again, thanks so much for a great library!

@timothyerwin glad you found a problem!

I'm not sure if that's mentioned anywhere here in the docs, or something that could be solved in this library but it's worth spelling out in the docs.

Could you please share a code snippet where issue you mention is reproducible? I'm afraid I still don't understand the problem and of course if it's known problem/issue I'd like to highlight it in docs 😊


Improper indenting on named argument style functions.

fails to indent properly on function calls that use named arguments on new lines
foo(
    bar=this,
    baz=that);
// Next line will have an indent level increased
    foo2(
         bar2=this2,
         baz=that);

Note if the close paren is on a new line this effect doesnt happen.

Sorry, this one slipped under my radar.
I'm not really sure what could be causing this.
I take it that other languages in your setup don't exhibit this behavior?

I don't think so? I haven't experimented much on this one. It's obviously not a big deal or anything.


Windows "Speaker Fill" Enhancement doesn't work with GPMDP

OS:
Windows 10
Issue Descriptions:
Pretty much what the title says. When I use WMP the "Speaker Fill" Enhancement in the Realtek HD Audio Manager works, so that I get sound on all speakers with 5.1. But with GPMDP it sadly doesn't work and only plays sound on front left and right.
Steps to Reproduce:
Use 5.1 System and activate Speaker Fill in Realtek HD Audio Manager.

There is an option in chrome called "--try-supported-channel-layouts". Try running GPMDP with this option to see if it fixes the audio for you.

I tried this by adding the parameter to my desktop shortcut. However, it did not fix my issue.

Does it work in chrome?
On Jul 11, 2016 12:20 PM, "DRSXS"<EMAIL_ADDRESS>wrote:

I tried this by adding the parameter to my desktop shortcut. However, it
did not fix my issue.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/issues/1303#issuecomment-231785471,
or mute the thread
https://github.com/notifications/unsubscribe/AAucnfrK65fVqdCHOJEkkMffyLQ2uYEEks5qUm1LgaJpZM4JJaNM
.

This one is quite simply nothing to do with us 😆
If it works in Crhome as @jostrander asked then it is an Electron problem if it doesn't it is a Chromium problem.
Neither of which will probably want to invest a lot of time into fixing a borked feature of a third party audio driver. 😆
Have you tried changing your audio output device?

I just tried it out with Chrome and it is as you thought. Same problem, I will try and find a solution online because I'm sure I'm not the only one bothered by this. I mean you don't wanna listen to music on just your Left and Right, without the center and Sub it just doesn't sound as good. Rears are not that important for music and youtube. Anyway, thanks for your input.


Teach storage_client.QueryPages to use a RowRange that filters on the server.

Send in a lexicographically greater-than string to ReadRows and let the bigtable server
stop sending rows when's its finished.
We measured a 25-30% reduction in response time on the p90.
Some code for testing the queries outside of cortex: https://gist.github.com/bcotton/c76a9efc394d442b7d3689f8a8b7e2ca

Niiiice.  I'd thought about doing this but wasn't sure it was safe.  You seem to have proven it is!  I'll do a more in-depth review tomorrow to confirm, but thank you very much!

LGTM, this is running in our dev env with no problems.
It has the nice side effect that our bigtable monitoring no longer shows any errors, as the hacky stop-early I put in was reporting as an error in our monitoring.  Out 99th bigtable read latency has gone from 700ms -> 230ms (on dev).

Thanks!


Typo

"Open a web browser and navigate to http://localhost:5000 to be see your containerized application, as shown in this screenshot:"
Note the part ... to be see your ... --> should probably just be ... to see your ...

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

ID: e38de4f3-3343-74cd-103a-fe12846ce7c3
Version Independent ID: 6bf65efb-13eb-15d9-5aac-9d5cf9c1aa96
Content: Containerize a .NET Core App
Content Source: virtualization/windowscontainers/quick-start/building-sample-app.md
Product: windows-server
Technology: windows-containers
GitHub Login: @cwilhit
Microsoft Alias: crwilhit

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


Double-quotes needed for parameters & in wrong section

Instead of
auto_reprovisioning_mode = Dynamic
the required syntax is:
auto_reprovisioning_mode = "Dynamic"

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

ID: dc645151-3164-c5cb-343b-fa3c5b4ee248
Version Independent ID: 50ee6151-587a-f01f-a3ed-b8b756cf010c
Content: Configure Azure IoT Edge device settings
Content Source: articles/iot-edge/configure-device.md
Service: iot-edge
GitHub Login: @PatAltimore
Microsoft Alias: patricka

In addition, the entire section should be listed in the "Global Parameters" section. iotedge config apply will not recognize it under the [provisioning] header.

Hi @theobarkerh Thank you for bringing the issue to our attention. We will take appropriate action based on the feedback you have shared.

I have assigned the issue to the content SME for further review.
Hi @PatAltimore, Can you review the above feedback and take appropriate action.

Hi @theobarkerh,
Thanks for the feedback. I made the update and the changes will publish soon.
Thanks,
Pat
#please-close


ODC-7484,ODC-7485,ODC-7486: Added Task, TaskRun and ClusterTask list pages to dynamic plugin

Story:
https://issues.redhat.com/browse/ODC-7484
https://issues.redhat.com/browse/ODC-7485
https://issues.redhat.com/browse/ODC-7486
Description:
Added Task, TaskRun and ClusterTask list pages to dynamic plugin

/hold

/cc @jerolimov

/unhold

/retest

Updated the PR based on review comments.

/retest

/hold

/unhold

/retest

/retest

Can you please explain me how it would work for end user after we merge this?

Will OpenShift Console 4.16 still contain these pages? What about further releases?
What happens when I enable dynamic plugin on my cluster? Will console always show the version from the dynamic plugin instead of static one?

/close
Handled in PR - https://github.com/openshift-pipelines/console-plugin/pull/78


Optimize querysets

Current coverage is 92.62% (diff: 100%)

Merging #326 into develop will increase coverage by 0.03%

@@            develop       #326   diff @@
==========================================
  Files            27         25     -2
  Lines          1390       1396     +6
  Methods           0          0
  Messages          0          0
  Branches        172        174     +2
==========================================
+ Hits           1287       1293     +6
  Misses           54         54
  Partials         49         49

Powered by Codecov. Last update c013661...53f98af

Coverage increased (+0.02%) to 96.132% when pulling 53f98afd6c87c5e0944ffea718ded508826209bd on feature/optimizations_develop into c013661a5383f774dcdb416043b96d34eadedab6 on develop.

Coverage increased (+0.02%) to 96.132% when pulling 53f98afd6c87c5e0944ffea718ded508826209bd on feature/optimizations_develop into c013661a5383f774dcdb416043b96d34eadedab6 on develop.

Coverage increased (+0.02%) to 96.132% when pulling 53f98afd6c87c5e0944ffea718ded508826209bd on feature/optimizations_develop into c013661a5383f774dcdb416043b96d34eadedab6 on develop.

Coverage increased (+0.02%) to 96.132% when pulling 53f98afd6c87c5e0944ffea718ded508826209bd on feature/optimizations_develop into c013661a5383f774dcdb416043b96d34eadedab6 on develop.

Coverage increased (+0.02%) to 96.132% when pulling 53f98afd6c87c5e0944ffea718ded508826209bd on feature/optimizations_develop into c013661a5383f774dcdb416043b96d34eadedab6 on develop.

Coverage increased (+0.02%) to 96.132% when pulling 53f98afd6c87c5e0944ffea718ded508826209bd on feature/optimizations_develop into c013661a5383f774dcdb416043b96d34eadedab6 on develop.