alignItems doesn't function properly when wrapping content

Description
I am unable to get the alignItems value to align the rows without any space between when I added flexWrap:'wrap' to the parent view (and enough child views such that it'd wrap). Flexbox acted as if it had a 'space-between' requirement for the rows.
Reproduction
Tested on react-native devices and the web simulator at  https://facebook.github.io/react-native/docs/flexbox.html#align-items and on Android/iOS devices.
#note: adding alignContent:'flex-start' in the tutorial simulator fixes the problem, but alignContent doesn't exist in react-native.

<View style={{
    flex: 1, // setting this to 0 results in the expected look for this view
    flexDirection: 'row',
    justifyContent: 'flex-start',
    flexWrap:'wrap',
    alignItems: 'flex-start'
  }}>
  //** Enough Views to get a wrap **
</View>

This will render:

And changing flex to 0 gives us our expected look:

<View style={{
    flex: 1, // setting this to 0 doesn't do anything for column
    flexDirection: 'column',
    justifyContent: 'flex-start',
    flexWrap:'wrap',
    alignItems: 'flex-start'
  }}>
  //** Enough Views to get a wrap **
</View>

This renders:

setting flex to 0 doesn't fix this issue.
Solution
Preferably, fix alignItems to work properly when wrapping is turned on. Else, implement alignContent prop to react-native to match the implementation in react-native-web.
Additional Information

React Native version: RN master branch
Platform: iOS, Android, Web
Operating System: All
Dev tools:  All

This issue is fixed in v0.43.1


Allow files.upload action to send multipart/form-data files

Currently only content-based file.upload requests can be uploaded. This is because the API call is a hard-coded application/x-www-form-urlencoded get request.
This PR addresses the issue and properly packages file payload in the requests.post files parameter.  This triggers requests to use the proper content-type multipart/form-data header, and also properly sends the file data.
Even though the request method has changed from get to post the other API methods work normally.

lgtm. wait for @manasdk to also look.

@manasdk Bump.

LGTM. I'll just merge it.


update to release v0.8.0 url and sha256

Problem

link and sha256 are not up-to-date
Issue #40

Solution

replace link of next branch artefacts, by link of artefacts of release v0.8.0

PR Status

[ ] Code finished and ready to be reviewed/tested
[ ] The fix/enhancement were manually tested (if applicable)

Automatic tests
Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)

!testme


Channel mask not sent in AU915

Describe the bug
In the AU915 region, the CFList is not yet supported on Helium, so the channel mask must be sent by LinkAdrReq.

When ADR is disabled, the LinkAdrReq to set the channel mask never happens
When ADR is enabled, the LinkAdrReq to set the channel mask is only sent after 20 uplinks, when the ADR algorithm decides it's time to change the data rate and tx power
If only 12.5 % of the uplinks are being received, because the channel mask isn't set, it takes 160 uplinks to trigger the LinkAdrReq

To Reproduce
Connect a device in AU915, and wait for the first LinkAdrReq. It will be very late if ADR is enabled, or won't be sent at all if ADR is disabled.
Expected behavior

The very first uplink after the Join Request should contain a LinkAdrReq to set the channel mask, regardless of the ADR enabled / disabled setting in the profile.
Or, adding CFList support would fix the problem for newer devices. The AU915 CFList is exactly the same format as the US915 list.

Device Info (please complete the following information):

Environment: Production
Console Device ID: c83da7c6-67e1-4fe3-b1c6-9b027d252028
Device dev eui / app eui: 0016C001F000ABA0 / 70B3D57050000009
Region: AU915
Hotspots (names) used to transfer data: perfect-chambray-panther

Screenshots
Screenshot for case when ADR is enabled:

Fixed in LoRaWAN library update
https://github.com/helium/erlang-lorawan/pull/6/files

Thanks!

Router updated to more recent lorawan library
https://github.com/helium/router/pull/748


[BUG] Repositories role should handle RedHat os/BaseOS and os/AppStream repos and description should be set to name

Two regressions have been introduced in repositories role:

the following repositories inventory no longer works:

{name: os, gpgcheck: 1, gpgkey: "file:///etc/pki/rpm-gpg/RPM-GPG-KEY-redhat-release"}

This is suppose to generate for RedHat, BaseOS and AppStream repositories client

name: yum_repository <|> Setting repositories
ansible.builtin.yum_repository:
name: "{{ item.name | default(item) }}"
description: "{{ item.description | default(item) }} gen by Ansible"
baseurl: "{{ item.baseurl | default(repositories_j2_simple_url + (item.name | default(item))) }}/"

The module yum_repository.py overrides name with description and so the result in /etc/yum.repos.d is wrong.
In the previous bluebanquise version, the description was :
description: "{{ item.name | default(item) }} gen by Ansible"

PR has been merged.


Material Design demo plunkers are broken

Bug, feature request, or proposal:
What is the expected behavior?
The demo plunkers for the material design are broken as the path returns 404

What is the current behavior?
404 Not Found
What are the steps to reproduce?
Navigate to any component example click here
Providing a Plunker (or similar) is the best way to get the team to see your issue.
Plunker template: Click on the plunker link in the above link
What is the use-case or motivation for changing an existing behavior?
Updating the path
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
material angular 2.0.0-beta.10
Is there anything else we should know?

https://github.com/angular/material2/issues/6734

This has been correct in the most recent releases.


chore(docs): update Form Service Frontend documentation

chore(docs): update Form Service Frontend documentation

The new version of the service has been released in the marketplace, please merge the documentation.


chore: add support of prerelease version

This PR adds support of prerelease version upgrade with version_bump workflow.
Now workflow dispatch UI shows the text input for prerelease id like the below:

If you like to append prerelease id in addition to normal version upgrade, you can specify the input like the below:
Note: The below example upgrades minor version and also append prerelease id (ex. 1.45.5 -> 1.46.0-rc.1)

If you like to only change the prerelease id, you can specify the inputs like the below:

If you like to only perform normal upgrade, you can keep the prerelease id blank.
I tested the above scenarios in this repo https://github.com/kt3k/bmp/actions/workflows/version_bump.yml

Please add https://github.com/denoland/deno_docker/pull/393/commits/82b7e9d96b54b56d7a1daebc0bcb7d4977e84817, else this will not work

Please add https://github.com/denoland/deno_docker/commit/82b7e9d96b54b56d7a1daebc0bcb7d4977e84817, else this will not work

Thanks! Added

I'll do a few experiments after landing this


fix: patch version and run cargo update

relates to https://github.com/Homebrew/homebrew-core/pull/165009

cc @quantumsheep

Thanks! The second time I forget that 😫


[Test] via Welcome.yaml

To test Actions/Comment/action.yml
Verified at https://github.com/Wei18/GitHubSwiftActions/blob/5e993bca0d3f050c590790954440a5953c81aeee/.github/workflows/Welcome.yml#L1-L24

Tested & Cloes.


Fix line endings in program.ts

Fixes issue #3540
Why this change is needed:
autocrlf = input and whitespace = cr-at-eol don't help here, because this problem is already in the repo.  This isn't something that has been introduced by my local tools mangling things; it's due to the fact that the file already contains mixed line endings straight-from-the-source.

Hi @colbyrussell, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

thanks!


adding label after Sucessful transaction

Suggestions : Providing an option to add a label  after successful transaction screen enhances its value.
https://bithyve-workspace.slack.com/archives/C021PTT2R5M/p1704280578087959

https://xd.adobe.com/view/f1543c84-c25a-4ad6-81ca-1e0c540e956e-a407/screen/e6c1ce1d-3668-4169-8577-fb442eb9c170/

Closing as outdated


Make graphlient run on Ruby 3.0

Upon trying graphlient on Ruby 3.0 I noticed a few issues, one of them being critical: https://github.com/ashkan18/graphlient/commit/effbf5d2d6721e7f0d66d0bcc1f9c9818e5ee871
All the other commits were made to get either CI to pass, tests to pass, or rubocop to pass. 😄

👍

Just cut a new release 0.5.0 🙌


Json serializer don't ignore property

Prerequisites

[x] I have written a descriptive issue title
[x] I have verified that I am running the latest version of Nancy
[x] I have verified if the problem exist in both DEBUG and RELEASE mode
[x] I have searched open and closed issues to ensure it has not already been reported

Description
Method Respose.AsJson don't ignore property. I try:
Post("/search", _ => {     var filtro = this.Bind<TFiltroPesq>();     var resp = dominio.Pesquisar(filtro);     return this.Response.AsJson(resp); });
But property Id is always serialized.
[System.Xml.Serialization.XmlIgnore] [Nancy.Json.ScriptIgnore] [System.Runtime.Serialization.IgnoreDataMember] public long? Id { get; set; }
Steps to Reproduce

System Configuration

Nancy version: 2.0.0-cliteastwood

Nancy host

[ ] ASP.NET
[ ] OWIN
[ ] Self-Hosted
[ ] Other:

Other Nancy packages and versions:

Environment (Operating system, version and so on): Windows 8.1

.NET Framework version: .NET Standard Library 1.6.1

Additional information:

Can you try [JsonIgnore] ?

I try [JsonIgnore] from namespace Newtonsoft.Json and won't works. Has another JsonIgnore? I don't found in Nancy.

@thecodejunkie  I can work on this issue , can you assign this to me?

@parmjeetkumar You're free to work on it, github doesn't allow us to assign non-members to issues tho :(

@phillip-haydon Thanks i was worried about that may be some one have to assign this to me to start work on this. Going to make this fix :)

Hi,
Is there any ETA for getting this fixed?
In the meantime, is there any workaround?

@m-barthelemy  How workaround I created a extesions methods
public static class NancyExtensions
{
	public static Response JsonResponse<TModel>(this NancyModule module, TModel data)
	{
		var settings = new JsonSerializerSettings()
		{
			ContractResolver = new CamelCasePropertyNamesContractResolver()
		};

		var json = JsonConvert.SerializeObject(data, settings);
		return module.Response.AsText(json, "application/json", Encoding.UTF8);
	}

	public async static Task<TModel> JsonRequestToObjectAsync<TModel>(this NancyModule module)
	{
		var reader = new StreamReader(module.Request.Body);
		var json = await reader.ReadToEndAsync();
		var obj = JsonConvert.DeserializeObject<TModel>(json);

		return obj;
	}

}

So, I use like this:
        PostSeguro("/salvar", async _ =>
		{
			var entidades = await this.JsonRequestToObjectAsync<TEntidade[]>();
			var resp = dominio.Salvar(entidades);
			return this.JsonResponse(resp);

		});

So, personally I do not see the issue being with Nancy but rather that you are using the same model to bind to as you use to represent an entity. Your data binding model should only expose the members that you actually want to bind, and reusing the same model type for two different things is a bad idea. This is the notion that we support out of the box
Moving on. If you want more control over the serialization process, then you can move away from the default serializer and use something like Nancy.Serialization.JsonNet which uses Newtonsoft.Json internally and lets to take advantage of the capabilities that it offers you.
Hope this helps! 😄
Closing this as this is not really a defect.

In Nancy v2 the IgnoreDataMemberAttribute is honored by the (new) default json serializer.


Set up Azure Pipelines for CI across macos, windows, linux

Do not need to take into account the release process, only CI on commits.

This is implemented and working however I still want to set up builds using our own agents. The Microsoft agents are too restrictive and do not allow access to the machine when debugging a build

Closing the issue, and creating a new one wrt to using our own build agents.


feat: added ffmpeg post

:tada: This PR is included in version 1.13.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Symbol resolution

As part of experiments in SymbolResolution we created this interface. I am opening this PR for getting feedback from @loradd . We would need this to be merged to complete and merge https://github.com/Strumenta/rpg-parser/pull/40

Thank you!


if loadJson gets called with an object as input, it mutates the input, which breaks subsequent loadJson calls.

Consider the following scenario:
    let json = {
      'package': null,
      'messages': [{
        'name': 'Entity',
        'fields': [{
          'rule': 'optional',
          'type': 'string',
          'name': 'entityProperty',
          'id': 1
        }]
      }]
    };

    let proto1 = ProtoBuf.loadJson(json);
    let Entity1 = proto1.build('Entity');
    // Entity1 is correct, but it has mutated the input:
    // json.messages.length === 0
    let proto2 = ProtoBuf.loadJson(json);
    let Entity2 = proto2.build('Entity');
    // ERROR: Entity2 === null

The problem is that the building has mutated the original data structure, and removed the meta data. At this point: json.messages.length == 0. This will make the subsequent builds all fail and return null.
This only happens when we pass in a mutable data structure (so, a real object, not a json text representation or a file) as an input. This should be easy to work around by cloning the input in loadJson, but I think that the problem could be better solved at the place, where it removes the processed records from the proto data.

Jaws dropped. This was fast!
Thanks a lot, it's working great.


Multi network power models data frame

Codecov Report

Merging #36 (6dae72d) into dev (edd50ca) will increase coverage by 0.36%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              dev      #36      +/-   ##
==========================================
+ Coverage   83.58%   83.95%   +0.36%
==========================================
  Files          16       15       -1
  Lines         658      642      -16
==========================================
- Hits          550      539      -11
+ Misses        108      103       -5

Impacted Files
Coverage Δ

src/core/types.jl
100.00% <100.00%> (+14.81%)
:arrow_up:

src/PowerPlots.jl
41.66% <0.00%> (-16.67%)
:arrow_down:

src/plots/network_status.jl
70.00% <0.00%> (-2.00%)
:arrow_down:

src/core/utils.jl
95.65% <0.00%> (-1.23%)
:arrow_down:

src/core/export.jl

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 edd50ca...6dae72d. Read the comment docs.


docs(zh): update the API references to the latest

Notes: the subfiles of TS interfaces and TS enums still need translations further.

I created an issue for the further actions #1743


Type Name Not Findable

Hi dad, I can't find a type for the chart with the name "Meter chart - with statuses" that could be MeterChartOptions

Did you see that(above)? It said the import
import { MeterChartOptions } from "@carbon/charts/interfaces";

was fine but look at this:

Please help, thanks dad!
Note: This issue was created because at time of post, you were in a meeting (12:20 PM 2/2/2023)

As in our discussion, I have explained how to get the correct type. See code below for reference:
import { ChartConfig, MeterChartOptions, Statuses } from "@carbon/charts/interfaces";
import { SolidMeterChart } from "../chartsLibrary";

const data = [
  {
    group: "Dataset 1",
    value: 56,
  },
];

const options = {
  title: "Meter Chart - with statuses",
  meter: {
    peak: 80,
    status: {
      ranges: [
        {
          range: [0, 50],
          status: "success",
        },
        {
          range: [50, 60],
          status: "warning",
        },
        {
          range: [60, 100],
          status: "danger",
        },
      ],
    },
  },
  height: "100px",
} as MeterChartOptions;

type Option = ChartConfig<MeterChartOptions>;

export default function MeterChartExample() {
  return (
    <div>
      <SolidMeterChart {...({ data, options })} />{" "}
      {/* issue posted on repo, I took a snapshot on this line and I put it in the issue */}
    </div>
  );
}

Thanks for that, all you referred to in your approval will be fixed, as well.

Wait a min, one more thing.
I did the exact same thing, and I exported it, but I keep on getting the same error.
That component is null


Just downloaded .dmg file to MAC and not working

I just installed the latest release via .dmg file.  Got to the nickname screen and once i pressed next the App closed.
I tried to re-install and still the same issue.  Not sure what to do.
Thanks.

Sorry, yes I deleted twice and finally worked again. I just figured it may have been an issue on my end, all is well and syncing now.  Sorry, should I not have closed it?

On Jul 24, 2018, at 9:27 PM, JimmyMow<EMAIL_ADDRESS>wrote:
Why was this closed @skcheffy https://github.com/skcheffy? Did you resolve your issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/LN-Zap/zap-desktop/issues/592#issuecomment-407603093, or mute the thread https://github.com/notifications/unsubscribe-auth/AehEHnkGWT8mDBSL6PY3Jhv_Y6HkoaS4ks5uJ8mNgaJpZM4VezQf.


ScriptBuf missing FromStr?

Should ScriptBuf implement FromStr? We already have from_hex, why not also have FromStr?

It was intentional because what Display produces should not be parsed.


Agent running on same box as server uses container IP, not public IP

An agent running on the same box as the management server uses the container IP, meaning that monitoring in the UI does not work.

Unsure if anything else breaks due to this as well?

@adamkdean I can't reproduce this. What version of Rancher are you using and how did you invoke the Rancher agent?
Thanks

When you launch the cattle agent on the master you need to add the env variable CATTLE_AGENT_IP= the public up of the host.

@bscott I started the agent on the same machine as the server using the default agent command that the UI gives you. This is from my bash history:
sudo docker run -d --privileged -v /var/run/docker.sock:/var/run/docker.sock rancher/agent:v0.5.2 http://snip/v1/scripts/90F5BA57193C25553524:1430071200000:0fV2koJ1EdCDTJsYOdhxMsexU

@cloudnautique Thanks Bill, I'll give that a go.

That worked, thanks.
sudo docker run -d -e CATTLE_AGENT_IP=46.101.X.X --privileged -v /var/run/docker.sock:/var/run/docker.sock rancher/agent:v0.5.2 http://snip/v1/scripts/CD18D99C43173E42BEB3:1430157600000:RGT4YjWvaCh7x9iOX1XtjdXUM

Can you add -e CATTLE_AGENT_IP= to the UI?

@maslick I've created an issue that talks about adding -e CATTLE_AGENT_IP to the UI in #728. This is more related to machines created using the UI. You could follow that issue to see what happens.

Just FYI this is now in the UI, when you try to add a host:
Optional: Specify the public IP that should be used for this host. This is required if you're trying to add the host the rancher/server container is on.


One more CVPR2023 paper

3Mformer: Multi-order Multi-mode Transformer for Skeletal Action Recognition
paper: https://arxiv.org/abs/2303.14474
Thank you for your time and assistance, and I must say, this is a great repository!

Thanks for your interest and appreciation. I have added this paper to the list. I am glad that the repository can help you.


ORC-1619: Add MacOS 14 to GitHub Action

This is a clone of #1800 .

This is resolved via https://github.com/apache/orc/commit/1c9d91efbc8141c303ac3b95b3dd2d7eb8152239


 Does a service restart resolve the problem? Is there anything else of note going on in the database (or server in general) when the issue occurs?
@corylanou Could this be the deadlock issue you've been working on?

@rossmcdonald Yes, a service restart gets things working again. Nothing else stands out to mention.. the CPU and memory usage were low. I'm using UDP input sources for writes, under 100 points /second. Had a grafana dashboard open querying every 5s. Ubuntu 14.04.3 LTS.

Duplicate of https://github.com/influxdb/influxdb/issues/4870?

@beckettsean Sounds like it could be - though that issue doesn't mention any errors similar to the ones I mention here.

@beckettsean having the same symptom on InfluxDB v<IP_ADDRESS> (git: 0.9.4 da1b59e7d7764d36786253b2db13b97f42ed4e1d)
Soon we got this symptom errors 500, we need to restart influxdb for have be able to restore service (soon action 'reading WAL files to flush to index' are done which is slow currently)

@prawnsalad Did you have to do anything to get the stack trace (like explicitly kill the process, etc), or did it just appear in the logs during runtime?

They had just appeared in the log file at runtime. There was many others similar to the 3 I pasted too that went on for several hours.

@JulienChampseix that's a different panic and doesn't look like the deadlock issue. Can you open a new issue with that stack trace?

thanks @beckettsean i did here https://github.com/influxdb/influxdb/issues/4655 (but before to go on 0.9.5.X, i would be sure it's fixed on the <IP_ADDRESS> also)

@JulienChampseix we've reverted to building with Go 1.4 for the time being, so any Go 1.5 bugs like #4655 should not be relevant for <IP_ADDRESS>.


Rename @fluid-internal/tree to @fluid-experimental/tree2

Description
Renames the @fluid-internal/tree package to @fluid-experimental/tree2. This will make it included in releases while clearly marking it as experimental. This change is simply being done so contributors to the tree package can have packages builds to test with.
Currently all public releases (internal, LTS, prerelease and test) include the same set of packages, which includes @fluid-experimental but not @fluid-internal/tree, so this change results in the package being included in these builds.
All APIs in  @fluid-experimental/tree2 are marked as @alpha. This is redundant with them being in an experimental package: both mean no API stability commitment is being made and no data format compatibility commitment is being made.
pnpm run layer-check --md was run to update PACAKGES.md. Looks like it was out of date so some other updates are included.
Breaking Changes
Uses of the previously non-public @fluid-internal/tree package will need to update to @fluid-experimental/tree2
Reviewer Guidance
The review process is outlined on this wiki page.

Our repo policy does not permit a package named "tree2" to live in a folder named "tree". I see a few possible solutions to this:

Change the policy to add an exception for this case (and publish a build of build-tools with this change, then update to it)
Rename the folder to packages/dds/tree2
Move the folder to experimental/dds/tree2 (if we are renaming/moving it, might as well fully match the package name and status)

Option 3 seems like the best choice to me, though it will be a bit annoying for pending changes.

Our repo policy does not permit a package named "tree2" to live in a folder named "tree". I see a few possible solutions to this:

Change the policy to add an exception for this case (and publish a build of build-tools with this change, then update to it)

...

1b might be to add the path to the ignore list, though that might be too heavy a hammer.


[NOD-685] Fetch blocks missing in db when handling selectedParent-chain changes

Rebuild after update in master

Rebuild after update in master

Rebuild after update in master


Create RegisterWithAzure.ps1

This is a wrapper script for helping customers to walk through the steps of registering their Azure Stack with their Azure subscription.

Hi @rupisure, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (rupisure). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

Hi @rupisure, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (rupisure). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;


GH Pages branch is outdated

Additionally, the documentation included in the pages isn't all that helpful due to lack of actual documentation / comments. It would be great to see the pages updated to at least be up to date with the current SDK version.

Hey @de-zakauff, that will be resolved soon.

@de-zakauff, should be updated to the current SDK now.


Mention channels

I did not find in the documentation referred to the mechanism of a list of channels. How can I get this information?

@toby3d sorry this went unnoticed for so long :(.  Do you want a list of channels for a specific guild? If so, the function Session.GuildChannels() will pull that information from the REST API.   For future reference, for quick questions it might be best to join us on the Discord Gophers server https://discord.gg/0f1SbxBZjYoCtNPP and we'll try to answer any other questions you have.


Pull without args results in unexpected operation

git.pull should do what git pull does, but it doesn't
There are a couple of other issues in this repo around pull/fetch, but I'm not sure they are the same as this one.
Consider the following code:
git.checkout("foo")
git.pull

You'd think it would do the same as this, but it doesn't.
git checkout "foo"
git pull

You must do:
branch = "foo"
git.checkout(branch)
git.pull("origin", branch)

We find this quite unexpected. Is there a rationale for it?

This is still an issue. A week over Christmas was not enough to react! I will (re)open.

Still an issue.
@perlun I feel the way the gem works now is "unexpected", and breaks POLA.
Would you accept a PR to change it?

@hlascelles I agree with your assessment.
Since this change would break backwards compatibility, it would have to be targeted to the next major release.

Sounds good. (Also, remove the wontfix label. Thanks!)

@jcouball can this be marked with Milestone 2.0, and stalebot be configured to ignore future milestones (ot some similar config?).

I am considering this a bug and not a feature change so will include the fix for it into the next release.


fix(sidecar): local blob building

Fixes various problems in the local block building problems, such as:

missing kzg commitments in the bid header
missing actual blob gas used in the bid header


Specifying hostPort when apmEnabled is set to true

Hey @hkaj, @irabinovitch, and @xvello, thanks for all your work on this chart! When I was trying to use APM with this chart, I realized that the chart does not set hostPort when the apmEnabled flag is true. Using this guide as a reference that seems like a requirement for APM integration. Making this change locally worked for me.

Hi @robscott ,
We'd rather keep the host port binding explicit, as this can have security implication on badly firewalled hosts.
Looking at the deployment template, we didn't port the conditional logic to actually enable the host ports that we have in the daemonset. Would copying this logic in the deployment solve your use case?
BTW, I am currently working on merging the daemonset and deployment templates to avoid them falling out of sync. This will solve the issue in the mid-term.
/hold

Thanks for the clarification - looks like I was just looking in the wrong placed and missed the options already in place on the daemonset.


[Feature] Decouple AppManager from page object and elements

Describe the feature
To improve the use of the setup for running an application under a UI test, changes need to be made to decouple the static AppManager.App instance usage across the Legerity page objects and elements.
For elements, the WrappedDriver property should instead be used for referencing to get additional sub-UI elements.
The AppManager will need updating to provide capabilities to retrieve an instance of the driver that is initialized. This implementation needs to reduce the impact of breaking changes to existing test projects using Legerity while providing the capability to be expanded on in the future.
Rationale for the feature
To improve maintenance, scalability, and overall testability of UI test projects using Legerity.
Scope of the feature
Must - Decouple the AppManager.App instance from core Legerity code
Open questions
No response
Other information
No response

Reverting this change and re-opening


fix deadlock

Describe the change
fixes: https://github.com/piotrmurach/finite_machine/issues/64
1/ add an event
2/ shutdown the queue, set @dead = true
3/ process_events doesn't release the mutex via conditional variable, because the queue is already dead
4/ an attempt to add a new event after clearing the queue waits for the lock forever https://github.com/piotrmurach/finite_machine/blob/master/lib/finite_machine/message_queue.rb#L62
spec https://github.com/piotrmurach/finite_machine/blob/master/spec/unit/message_queue_spec.rb#L48
Why are we doing this?
fix sporadic hangs on jruby
Benefits
CI
Drawbacks
what was the original intention?
Requirements
Put an X between brackets on each line if you have done the item:
[X] Tests written & passing locally?
[X] Code style checked?
[X] Rebased with master branch?
[] Documentation updated? not relevant

Coverage remained the same at 98.828% when pulling 7d860468af7a71b2488c0bbc250501a28f31ffa6 on ahorek:fix_deadlock into effcc378b68fbadd237c625896d3fd577547da87 on piotrmurach:master.

Nice one!


Can you export API json?

I want this api mock, can you export it, thanks!

https://github.com/tx7do/kratos-cms/blob/main/blog-backend/app/admin/service/cmd/server/assets/openapi.yaml
https://github.com/tx7do/kratos-cms/blob/main/blog-backend/app/front/service/cmd/server/assets/openapi.yaml
u need learn about Swagger UI.

Thanks @tx7do. I got it.
Are you have mock database or data.sql?

u can use this tool:https://apifox.com/.
this tool support: Mock.


docs: fix incorrect argument position reference

Thanks for the PR, but it is incorrect, it is actually about the first argument, see the transformed listing in the second example :). If it was a confusing reading experience, do feel free to restructure!

oohhhh okay. yeah i was hung up on that first part trying to reconcile what i was reading with what was being showed in the snippet. I didn't even scroll down. :expressionless:


Tautomers and protonation state

We need to enumerate tuatomers and protonation states for incoming molecules.

Addressed via #36. While this PR is a lot more conservative (chemicalize predicts more highly populated tautomers at pH ~7.4 than the newer OpenEye function), the states are reasonable.


Whenever Kenny Kerr goes silent for a bit...

... things are happening.
This is awesome news! Though I have a few questions: Where does windows-rs fit in in terms of the classic Windows API? Specific questions I have:

Is it a replacement for the raw winapi crate only?
Does it go beyond that and provide facilities like the WIL?
Or is it even higher level, like a sane MFC/ATL/WTL?

And, of course, like every one else, where does win32.winmd come from?

Thanks Tim,

Is it a replacement for the raw winapi crate only?

Yes, I would expect the windows crate to gradually replace the winapi crate. There may be gaps in API coverage, but we're working hard to get everything covered and work out any kinks.

Does it go beyond that and provide facilities like the WIL?

There are basic helpers for things like string and error handling, but I would expect the Rust community to continue developing specific and focused crates for various tasks. Think of the windows crate as a foundation building block for more specific or targeted crates that provide richer helpers like those provided by WIL.

Or is it even higher level, like a sane MFC/ATL/WTL?

The windows crate makes the Windows API available to Rust developers. Some APIs, like those based on WinRT, will have rich Rust bindings that don't need any wrappers to make them useful and safe in Rust. Other APIs, like those based on C-style functions often called Win32 APIs, will likely benefit from being wrapped in specific crates to provide a more idiomatic experience in Rust.

And, of course, like every one else, where does win32.winmd come from?

The metadata for Win32 APIs comes from this project: https://github.com/microsoft/win32metadata

Thanks Tim,

Is it a replacement for the raw winapi crate only?

Yes, I would expect the windows crate to gradually replace the winapi crate. There may be gaps in API coverage, but we're working hard to get everything covered and work out any kinks.

Does it go beyond that and provide facilities like the WIL?

There are basic helpers for things like string and error handling, but I would expect the Rust community to continue developing specific and focused crates for various tasks. Think of the windows crate as a foundation building block for more specific or targeted crates that provide richer helpers like those provided by WIL.

Or is it even higher level, like a sane MFC/ATL/WTL?

The windows crate makes the Windows API available to Rust developers. Some APIs, like those based on WinRT, will have rich Rust bindings that don't need any wrappers to make them useful and safe in Rust. Other APIs, like those based on C-style functions often called Win32 APIs, will likely benefit from being wrapped in specific crates to provide a more idiomatic experience in Rust.

And, of course, like every one else, where does win32.winmd come from?

The metadata for Win32 APIs comes from this project: https://github.com/microsoft/win32metadata

Closing as the questions are answered. We're tracking that these questions should be answered in some sort of FAQ in issue #460.

Closing as the questions are answered. We're tracking that these questions should be answered in some sort of FAQ in issue #460.


feat(linter): implement no-plusplus rule

part of https://github.com/oxc-project/oxc/issues/479

This PR implements the no-plusplus rule and is more-or-less a direct port of the ESLint version of the rule.

Sorry, I misclick the merge button, I want to rebase the branch, since the pr not seems introduce performance regression.

Did some simplify, can you patch this:
diff --git a/crates/oxc_linter/src/rules/eslint/no_plusplus.rs b/crates/oxc_linter/src/rules/eslint/no_plusplus.rs
index 89448d11f..7b3f73de5 100644
--- a/crates/oxc_linter/src/rules/eslint/no_plusplus.rs
+++ b/crates/oxc_linter/src/rules/eslint/no_plusplus.rs
@@ -92,7 +92,9 @@ impl Rule for NoPlusplus {
             return;
         };

-        if self.allow_for_loop_afterthoughts && is_for_loop_afterthought(node, ctx) {
+        if self.allow_for_loop_afterthoughts
+            && is_for_loop_afterthought(node, ctx).unwrap_or_default()
+        {
             return;
         }

@@ -100,52 +102,27 @@ impl Rule for NoPlusplus {
     }
 }

-/// Whether the given AST node is a ++ or -- inside of a for-loop update.
-fn is_for_statement_update(node: &AstNode, ctx: &LintContext) -> bool {
-    let Some(parent) = ctx.nodes().parent_node(node.id()) else {
-        return false;
-    };
-    let AstKind::ForStatement(for_stmt) = parent.kind() else {
-        return false;
-    };
-
-    for_stmt.update.as_ref().is_some_and(|update| is_eq_node_expr(node, update))
-}
-
-/// Checks if the given node is equivalent to the given expression (i.e., they have the same span).
-fn is_eq_node_expr(node: &AstNode, expr: &Expression) -> bool {
-    // TODO: This logic should be moved to somewhere more general and shared across rules and expanded
-    // to cover all expressions and node types
-    let node_span = match node.kind() {
-        AstKind::UpdateExpression(expr) => expr.span,
-        AstKind::SequenceExpression(expr) => expr.span,
-        _ => return false,
-    };
-    let expr_span = match expr {
-        Expression::UpdateExpression(expr) => expr.span,
-        Expression::SequenceExpression(expr) => expr.span,
-        _ => return false,
-    };
-    node_span == expr_span
-}
-
 /// Determines whether the given node is considered to be a for loop "afterthought" by the logic of this rule.
 /// In particular, it returns `true` if the given node is either:
 ///   - The update node of a `ForStatement`: for (;; i++) {}
 ///   - An operand of a sequence expression that is the update node: for (;; foo(), i++) {}
 ///   - An operand of a sequence expression that is child of another sequence expression, etc.,
 ///     up to the sequence expression that is the update node: for (;; foo(), (bar(), (baz(), i++))) {}
-fn is_for_loop_afterthought(node: &AstNode, ctx: &LintContext) -> bool {
-    if let Some(parent) = ctx.nodes().parent_node(node.id()) {
-        match parent.kind() {
+fn is_for_loop_afterthought(node: &AstNode, ctx: &LintContext) -> Option<bool> {
+    let mut cur = ctx.nodes().parent_node(node.id())?;
+    // dbg!(cur);
+    loop {
+        match cur.kind() {
             AstKind::SequenceExpression(_) | AstKind::ParenthesizedExpression(_) => {
-                return is_for_loop_afterthought(parent, ctx)
+                cur = ctx.nodes().parent_node(cur.id())?;
             }
-            _ => (),
+            _ => break,
         }
     }
-
-    is_for_statement_update(node, ctx)
+    match cur.kind() {
+        AstKind::ForStatement(stmt) => Some(stmt.update.is_some()),
+        _ => Some(false),
+    }
 }

 #[test]

https://github.com/oxc-project/oxc/pull/5581, it is as I expected, so the performance regressed just because these two case use too much UpdateExpr , maybe we could adjust the RuleCategory( disable it by default) to solve perf issues.

I'm likely going on leave shortly, so I probably won't be able to take a look at this for a while, so feel free to make edits to this PR and merge/close if needed.

I may have broke codspeed, hold on.

@IWANABETHATGUY I applied your simplifications and even went a bit further by using a while-let loop instead of loop-match.

rest part Looks good to me,


Accessibility - Form control with "listbox" role has no associated label

the label is missing on a listbox
reporting url https://ibmdocs-dev.mybluemix.net/docs/en/error
and the tool to check errors is "IBM Equal Access Accessibility Checker", plugin for chrome
<div class="react-autosuggest__suggestions-container" role="listbox" id="react-autowhatever-1">

Kenny will review and document more details for the next review.

Confirmed fix is needed for React version.

@Praveen-sr This issue as well as #6116 and #6117 are ready for testing.


Enhance bulk upload api to accept zip file

Enhancing Bulk CSV File Upload API to support zip file

@ub0005 I was wondering if you're unzipping the file to get back the original csv or not to dump into the db...

@ub0005 I was wondering if you're unzipping the file to get back the original CSV or not to dump it into the DB...
unzipping is not needed here @NISH1001 Pandas was able to read from the zip file. I tested it by uploading CSV file as well as zip files.

@ub0005 Everything looks fine from my side as adding this feature doesn't hamper the other functionality. I didn't realize pandas could directly load zip files in read_csv. Thanks for pointing that out.


Add Did I Stutter (speed up) by beamglow

Song Name
Did I Stutter (speed up)
Artist Name
beamglow
Source
Youtube
Youtube Link
T2XpdvsFFqcu0K8o
Direct File Link
No response
Song ID
559803
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 85086987

reject
invalid YouTube link


bugfix: TypeError when adding a batch dimension

Error Message

File "dqn.py", line 69, in get_action
state = state[np.newaxis, :]
TypeError: tuple indices must be integers or slices, not tuple

Solution
According to the OpenAI Gym source codes, env.reset() seems to return two data (in tuple).
So I added an underscore to successfully unpack the tuple.
I have applied the similar modification to env.step().
Environments

Ubuntu 20.04
Python 3.8.10
OpenAI Gym 0.26.2

Or you can fix this problem by changing
state = env.reset() -> state = env.reset()[0]


Fix LineNotify & Pagertree

Confirmed that the newly loaded alerters are defined in the included alerts.py file.

Confirmed that the newly loaded alerters are defined in the included alerts.py file.


Use a fast poet backend in E2E activation tests

Motivation
The E2E tests are quite long and resource-hungry because they run a real PoET service. A poet proof needs at least 150 leaves to be valid and tests become flaky on slower runners when epoch is too short - hence the tests were configured to run even 30s epochs.
Description

added a test poet service that produces a minimal valid poet proof instantly
shortened epochs in E2E tests

Tests now take around 1m30s instead of 4-5 min before and the gain will only become bigger and bigger as we add more E2E tests.
Test Plan
Existing tests pass
TODO

[x] Explain motivation or link existing issue(s)
[x] Test changes and document test plan
[ ] Update documentation as needed
[ ] Update changelog as needed

bors merge


Update GreyNoise SDK to v2.0.0

Status
Ready
Related Content Pull Request
Related PR: link to the PR at demisto/content
Related Issues
Related: not created yet, still a WIP
Description
Need to use the most recent version of the GreyNoise SDK to support new actions being added to the pack

@bradchiappetta Thank you for your contribution, is it ready for review? in the PR description its saying WIP (work in progress).
regardless it looks good, nice work!
let me know if you still need anything else / when its ready for review.

@GuyAfik this one is ready to go, my contribution to the content repo is still WIP.  Thanks!


Guruc 134 edges that disconnect graph

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
added the code for a problem related to graphs, the problem uses tarjans algorithm to find the edges which upon removal increase the number of components in the graph i.e (disconnect the graph)
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
issue: #2338

@HarshCasper  the reason is that this is a sum from the category of competitive programming , and the solution is mine you can verify it from here. I am the author of the solution .
Thanks for reviewing

@HarshCasper  where can I see the changes that are requested, once could you point out where i need to make a change

@HarshCasper  sure, thanks for  the review

You don't seem to have addressed any of the reviews. Please close the PR if you are not interested in implementing the changes.


[Bug]: All decimal fields behaves as amounts by default

Describe the issue
A regular decimal field in our extension no longer accepts more decimal places than setup in the General Ledger Setup for Amounts. This used to work in previous BC versions, so it's a breaking change.
This is how the field is defined in our extension:

(no decimal places setup and no AutoFormatType)
This is the error we get when we try to enter more positions than the Amount Unit Precision allows in the General Ledger Setup:

When I change the Amount Decimal Places to 2:5 in the General Ledger Setup (and I refresh the session), then the same value is accepted.
Expected behavior
The Decimal Places for Amounts and Unit Amounts should only be applied to decimal fields with a corresponding AutoFormatType.
Steps to reproduce
This issue seems to apply to all our decimal fields in all our extensions.
So, reproduction should be quite straightforward:

Create a new decimal field in an extension.
This decimal field should not have any properties except a caption and a dataclassification.
Add the decimal field to a page.
Set the ApplicationArea and a Tooltip.
Publish the extension
Verify that the Amount Decimal Places are set to 2:2 in the General Ledger Setup.
Enter a decimal with more than 2 decimal places (0.0001).

Additional context
We do have a workaround, but it requires that we explicitly set the decimal places for the decimal fields.
We have seen this behavior in Version: NL Business Central 23.1 (Platform 23.0.13763.0 + Application 23.1.13431.13546)
But also in the latest builds of BC22
There is also something off with the error message as it doesn't give the option to copy the call stack, which is the new behavior in BC23.

Nice catch! Let's get the issue to a team who will fix and backport it, as this repo only is for extensibility related issues in the upcoming release. We suggest that you open a support case or file a bug in Collaborate, to ensure that all Business Central users benefit from your catch as soon as possible.
To open a support case, you can:

Open a support request to CSS through the PartnerSource portal
Contact your Service Account Manager (SAM) in your local subsidiary to understand what is included in your contract as of support incident and PAH (Partner Advisory Hours). Your SAM might also direct you "step by step" how to open a support request or how to get credentials, if this is the first time for you or your company

If you file the bug in Collaborate, remember to include steps to reproduce the issue, and the Business Central build number and country version you're using. Here's a link to Collaborate: https://developer.microsoft.com/en-us/dashboard/collaborate/engagements/1173/feedback/wits/Bugs/create.
Thanks for reporting the issue.

@RSchoesterIdyn Is this an NL specific change? I tested this with older BC versions and even in old NAV and couldn't see any difference. If there are no explicit properties set on the table field, the settings from the G/L setup will be used.

@RSchoesterIdyn Is this an NL specific change? I tested this with older BC versions and even in old NAV and couldn't see any difference. If there are no explicit properties set on the table field, the settings from the G/L setup will be used.

I don't think it's NL specific, most likely a platform issue.

I think you have misunderstood my post:
The behavior in BC23 is the same as in BC22.5, BC22.3 and even NAV14.
All these versions use the G/L setup settings for decimal fields if no DecimalPlaces or AutoFormatType/AutoFormatExpression is specified.
Maybe you overwrote the setting from the G/L setup with EventSubscribers in the past and the subscriber somehow got lost in your extension? Or you uninstalled another extension that changed this behavior.


cudaPackages: Add documentation on cross-compilation

[!Important]
To-do:

[ ] Revise language in pkgs/development/cuda-modules/ON-CROSS-COMPILATION.md, adding examples of prior work and possible routes for further investigation.

Description of changes

Split off from #301416 with the goal of discussing and revising docs on cross-compilation separately from otherwise unrelated fixes and improvements to cudaPackages.
CC @NixOS/cuda-maintainers @samuela @SomeoneSerge @kmittman
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Specific comments and points to address, from #301416.
A superficial analysis of the differences between redistributables and the CUDA Toolkit run file/Debian installer:

Superficially, I can say there's something going on with host-* and target-* subdirectories present in the CUDA Toolkit (of all kinds, I believe, cross or not).
In the non-cross CUDA Toolkit, on x86_64-linux (that's the NixOS system name) there will be a host-linux-x64 and target-linux-x64 subdirectory in the root of the CUDA Toolkit.
In the cross compilation CUDA Toolkit, the values might well be something like host-linux-x64 with target-linux-sbsa (ARM servers) or target-linux-aarch64 (Jetson devices).
I'd have to spin up an Ubuntu system to install the cross-compilation version of the CUDA Toolkit and trace through the flow of CMake to see how it's driving cross-compilation.
I fear that beyond the presence of these host and target directories, the binaries in the cross-compilation version of the CUDA Toolkit are different from those offered in the redistributables -- though I haven't done any sort of analysis on the individual binaries themselves, I worry they're compiled differently from NVIDIA's sources in such a way that we cannot mimic the changes necessary for cross-compilation in an ad-hoc manner.
At any rate, it would require more time than I'm able to allocate.
@ConnorBaker in https://github.com/NixOS/nixpkgs/pull/301416#discussion_r1551774908

Cross-packages should be a subset of the native packages:

Hmmm, I'm not entirely sure why these issues are cropping up, seems to be related to the targets/ directory symlinks.
The binaries are the same, they are built once per platform/version and then packaged into each format; some formats (Conda and Python wheels) involve patchelf.
I would expect the "binary archive" tarballs, Debian packages, and RPM packages to be bit-identical.
The cross-packages are more-or-less a subset of the native packages; some libraries provide stub files, while others use the native runtime.
@kmittman in https://github.com/NixOS/nixpkgs/pull/301416#discussion_r1556084671

On future approaches to a minimum working example:

We really should (have) start(ed) with an MWE: piecing together an nvcc derivation sufficient to run nvcc directly on a single file (saxpy but without cmake), manually copying stuff from archives for different systems, without hooks or much automation
@SomeoneSerge in https://github.com/NixOS/nixpkgs/pull/301416#discussion_r1558575644


[COLIB-NGC] T1.4 add ExcST7B

This is an issue in the context of the project COLIB NGC. For the simulation models than belong to this documentation, see dynawo.
To be developed here:

add model description (fill template)

outdated, see https://github.com/dynawo/dynawo/pull/3099


[Bug report] By using the compressAndGetFile method with the parameter quality:100, the images lose quality.

Version
version in pub.dev
Platforms
dart, Android
Device Model
All Models tested
flutter info
[✓] Flutter (Channel stable, 3.16.5, on Ubuntu 22.04.3 LTS 6.1.0-1027-oem, locale pt_BR.UTF-8)
    • Flutter version 3.16.5 on channel stable at /home/besoft/src/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 78666c8dc5 (2 weeks ago), 2023-12-19 16:14:14 -0800
    • Engine revision 3f3e560236
    • Dart version 3.2.3
    • DevTools version 2.28.4

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /home/besoft/Android/Sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /home/besoft/android-studio/jbr/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✗] Linux toolchain - develop for Linux desktop
    ✗ clang++ is required for Linux development.
      It is likely available from your distribution (e.g.: apt install clang), or can be downloaded from https://releases.llvm.org/
    ✗ CMake is required for Linux development.
      It is likely available from your distribution (e.g.: apt install cmake), or can be downloaded from https://cmake.org/download/
    ✗ ninja is required for Linux development.
      It is likely available from your distribution (e.g.: apt install ninja-build), or can be downloaded from https://github.com/ninja-build/ninja/releases
    • pkg-config version 0.29.2
    ✗ GTK 3.0 development libraries are required for Linux development.
      They are likely available from your distribution (e.g.: apt install libgtk-3-dev)

[✓] Android Studio (version 2022.2)
    • Android Studio at /home/besoft/android-studio
    • Flutter plugin version 74.0.2
    • Dart plugin version 222.4582
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.84.2)
    • VS Code at /usr/share/code
    • Flutter extension version 3.80.0

[✓] Connected device (3 available)
    • Redmi Note 8 Pro (mobile) • w85t4tovxwz9a6aq • android-arm64  • Android 11 (API 30)
    • Linux (desktop)           • linux            • linux-x64      • Ubuntu 22.04.3 LTS 6.1.0-1027-oem
    • Chrome (web)              • chrome           • web-javascript • Google Chrome 119.0.6045.159

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

How to reproduce?
my code
var res = await FlutterImageCompress.compressAndGetFile(
imageToupdate!, '$folderInAppDocDir' + 'rotacioned_' + '${basename(imageToupdate!)}',
rotate: -90,
quality: 100,
);
When using it this way several times, a loss of image quality is noticeable.
Logs
No response
Example code (optional)
var res = await FlutterImageCompress.compressAndGetFile(
    imageToupdate!, '$folderInAppDocDir' + 'rotacioned_' + '${basename(imageToupdate!)}',
    rotate: -90,
    quality: 100,
  );

Contact
No response

https://github.com/fluttercandies/flutter_image_compress/assets/155546423/b7203ddb-e306-4bbb-92b0-b8af147505d8


samples: modules: can openmode runs on stm32h573i_dk

Add an overlay to avoid running the samples/modules/canopennode on the external octo NOR flash of the stm32h573i disco kit.
Signed-off-by: Francois Ramu<EMAIL_ADDRESS>

The overlay is aimed to run the  samples/modules/canopennode/  when storage is on the mcu flash, not the octoNOR and getting error [00:00:36.187,000] <err> app: failed to save EEPROM  after octoNOR init.

on the stm32h573i_dk samples/modules/canopennode/sample.modules.canopennode PASSED:
*** Booting Zephyr OS build zephyr-v3.3.0-3351-g493f6f74cbb7 ***
[00:00:00.000,000] <dbg> canopen_driver: CO_CANmodule_init: rxSize = 13, txSize = 9
[00:00:00.000,000] <dbg> canopen_driver: CO_CANmodule_init: excessive number of concurrent CAN RX filters enabled (needs 13, 14 available)
[00:00:00.000,000] <inf> app: CANopen stack initialized

on the stm32h573i_dk samples/modules/canopennode/sample.modules.canopennode.no_storage PASSED:
*** Booting Zephyr OS build zephyr-v3.3.0-3351-g493f6f74cbb7 ***
[00:00:00.000,000] <dbg> canopen_driver: CO_CANmodule_init: rxSize = 13, txSize = 9
[00:00:00.000,000] <dbg> canopen_driver: CO_CANmodule_init: excessive number of concurrent CAN RX filters enabled (needs 13, 14 available)
[00:00:00.000,000] <inf> app: CANopen stack initialized


Generalize show for TwoColumn etc to work with svgs

Hi @eford!
@jkrumbiegel provided the following code snippet on Slack
using Base64

struct TwoColumn{L, R}
    left::L
    right::R
end

function show_html_or_svg(io, x)
    if showable("text/html", x)
        show(io, MIME"text/html"(), x)
    elseif showable("image/svg+xml", x)
        print(io, """")
    else
        show(io, x)
    end
    return
end

function Base.show(io::IO, M::MIME"text/html", tc::TwoColumn)
    write(io, """

""")
    show_html_or_svg(io, tc.left)
    write(io, """

""")
    show_html_or_svg(io, tc.right)
    write(io, """

""")
    return
end

TwoColumn("hi", lines(1:10))

I think it would be really nice to use this approach in this package, do you agree?
I'd be willing to prepare a PR if @jkrumbiegel and @eford approve (unless @jkrumbiegel wants to do it himself)

I'm happy to see PRs for improvements.
I tried to give your snippet a try, but don't know what to import to get lines.

sorry, you need using CairoMakie. I adapted the snippet. Plots from this package currently don't have an html representation, but an svg representation.

Adding CarioMakie resulted in it running.  But now the RHS says:
"Scene (800px, 600px): 0 Plots 1 Child Scene: └ Scene (800px, 600px)"
I feel like that's likely not the intent.
On Sat, Oct 28, 2023 at 12:10 AM Fabian Greimel @.***>
wrote:

sorry, you need using CairoMakie. I adapted the snippet. Plots from this
package currently don't have an html representation, but an svg
representation.
—
Reply to this email directly, view it on GitHub
https://github.com/JuliaPluto/PlutoTeachingTools.jl/issues/37#issuecomment-1783691323,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKDFQV5Q2JT6ZWJYOZWEVDYBSAUVAVCNFSM6AAAAAA6TUGMJCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBTGY4TCMZSGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Eric Ford

Might be that you also need a CairoMakie.activate!(type = "svg") so it activates that showable. But this is not really about CairoMakie I think, just about objects that may return svg but not html. And this probably needs a more general solution for more MIME types, I just provided some quick help on slack :)

Ok, now I see a plot on the right.  Yes, I'd be happy to see a PR.
If it's not too much trouble, it would be nice if TwoColumnWideLeft{L,
R}, TwoColumnWideRight{L, R}, and ThreeColumn{L, C, R} worked the same.
Thanks.
On Sat, Oct 28, 2023 at 12:41 AM Julius Krumbiegel @.***>
wrote:

Might be that you also need a CairoMakie.activate!(type = "svg") so it
activates that showable. But this is not really about CairoMakie I think,
just about objects that may return svg but not html. And this probably
needs a more general solution for more MIME types, I just provided some
quick help on slack :)
—
Reply to this email directly, view it on GitHub
https://github.com/JuliaPluto/PlutoTeachingTools.jl/issues/37#issuecomment-1783698360,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKDFQWHZRCMDFF2LMV5FX3YBSEGPAVCNFSM6AAAAAA6TUGMJCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBTGY4TQMZWGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Eric Ford

# ╔═╡ df7dd1e4-7739-11ee-2c0a-5779ff4a782b
function TwoColumns(a, b)
	PlutoUI.ExperimentalLayout.Div([
		PlutoUI.ExperimentalLayout.Div([a]; style=Dict("flex" => "0 0 50%"))
		PlutoUI.ExperimentalLayout.Div([b]; style=Dict("flex" => "0 0 50%"))
	]; style=Dict("display" => "flex", "flex-direction" => "row"))
end

# ╔═╡ 2193c3aa-96c3-4621-a451-58289e879cdb
TwoColumns(
	md"# Hello!

	I am a computer",
	rand(10)
)

# ╔═╡ efd1aca8-5c8f-499d-9748-ccc297c7d167
import PlutoUI

No, I wasn't aware of that.  Thanks.
Your TwoColumns looks like a nice solution to me.
It looks simple enough that I could do it next time I update Pluto.  Or,
I'd be happy to merge a PR from Fabian if they'd like it sooner.
On Mon, Oct 30, 2023 at 11:38 AM Fons van der Plas @.***>
wrote:

[image: image]
https://user-images.githubusercontent.com/6933510/279112738-5aa7ed20-864e-4dc8-a080-6d5c4e351a42.png
╔═╡ df7dd1e4-7739-11ee-2c0a-5779ff4a782bfunction TwoColumns(a, b)
PlutoUI.ExperimentalLayout.Div([
PlutoUI.ExperimentalLayout.Div([a]; style=Dict("flex" => "0 0 50%"))
PlutoUI.ExperimentalLayout.Div([b]; style=Dict("flex" => "0 0 50%"))
]; style=Dict("display" => "flex", "flex-direction" => "row"))end
╔═╡ 2193c3aa-96c3-4621-a451-58289e879cdbTwoColumns(
md"# Hello!		I am a computer",
rand(10)
)
╔═╡ efd1aca8-5c8f-499d-9748-ccc297c7d167import PlutoUI
—
Reply to this email directly, view it on GitHub
https://github.com/JuliaPluto/PlutoTeachingTools.jl/issues/37#issuecomment-1785485675,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKDFQVNQHWKAHXWYYOWNWTYB7CWDAVCNFSM6AAAAAA6TUGMJCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBVGQ4DKNRXGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Eric Ford

Hi @fonsp,
While merging another PR, I checked open issues and noticed this one.
Is PlutoUI.ExperimentalLayout still experimental?  Or has it become something we expect to persist (perhaps with a different name)?
Thanks,

Hey! PlutoUI.ExperimentalLayout is also covered by semver, so it's fine to use it as a dependency.


Prettier

I noticed that the current code is not using any formatting standards.
Prettier seem to be an opinionated and standard tool used across many projects. I would be glad to implement Prettier support on the repo and reformat all files so that editing code is a more pleasing experience. This should also ensure that all contributions are consistent in code style.

Hi Derek,
This repo is a little bit abandoned now, and it would require a maintainer who can make such decisions. If you are interested in it I would be happy to invite you into this repo. Of course, it doesn't mean a life long full time engagement. Just answering issues and gardening the repo when it feels good.
If you would like to participate with some more work, I have some ideas how to make this binding not only hosted here but also promoted by LVGL. However, to do this promotion we need some tests, examples, and stable background here.
What do you think?

The work done in this repo is spectacular to be honest, and I would love to contribute back to it so I can achieve my project's goals and advance this binding.
Glad you mentioned engagement, cause I don't want to make a commitment I'm not able to meet, but I'm willing to do my best to bring the library to completeness of components and make the dev experience better (I'm writing typescript types too for enhanced safety and dx)

What I'm willing to do is use some of my free time to bring the library to completeness of components and make the dev experience better (I'm writing TypeScript types too for enhanced safety and dx on my project).

So happy to hear that!
I've just sent the invite. :hugs:

I don't know much about React to decide which would be a good or bad idea.  So please, feel free to merge the PRs you feel appropriate.

Done in #16


build fails if older OpenSSL pre-installed on system [master branch]

When trying to build OpenSSL "master" from Git (on x64_64 = amd64 Ubuntu 16.04 LTS), the configuration succeeds, but the build fails because the compiler grabs incompatible system includes for OpenSSL, when it should prefer the new includes in the Git checkout.
Workaround:
./config -I$(pwd)/include
(where $(pwd) means: expand the results of pwd here, formerly `pwd`)
Please make the equivalent of this "prefer local includes over system includes" part of all relevant branches.

I'm not sure I understand...  if you do make, all compile commands already have at least one -I option.  Why does that one make a difference?

It does make a difference because it makes sure that the local include files from the new source hierarchy (Git checkout, for instance) get preference over the older stuff in /usr/[local/]include. Try building OpenSSL master while you have 1.0.2 includes in /usr/include, it will break. Making sure that the compiler prefers /path/to/checkout/include over the old droppings in the system permits you to build a bleeding-edge OpenSSL even if old stuff remains installed.
The underlying issue remains invisible though in continuous-integration, chroot or jail or other clean-room builds.

That's an operation I do every day on my laptop, and here's a random command line picked now:
clang  -I. -Iinclude -I../master -I../master/include -DDSO_DLFCN -DHAVE_DLFCN_H -DOPENSSL_THREADS -DOPENSSL_NO_STATIC_ENGINE -DOPENSSL_PIC -DOPENSSL_IA32_SSE2 -DOPENSSL_BN_ASM_MONT -DOPENSSL_BN_ASM_MONT5 -DOPENSSL_BN_ASM_GF2m -DSHA1_ASM -DSHA256_ASM -DSHA512_ASM -DRC4_ASM -DMD5_ASM -DAES_ASM -DVPAES_ASM -DBSAES_ASM -DGHASH_ASM -DECP_NISTZ256_ASM -DPADLOCK_ASM -DPOLY1305_ASM -DOPENSSLDIR="\"/usr/local/ssl\"" -DENGINESDIR="\"/usr/local/lib/engines-1.1\"" -Wall -O0 -g -pthread -m64 -DL_ENDIAN -Wextra -Qunused-arguments  -DDEBUG_UNUSED -Wswitch -DPEDANTIC -pedantic -Wno-long-long -Wall -Wsign-compare -Wmissing-prototypes -Wshadow -Wformat -Wtype-limits -Wno-parentheses-equality -Werror -Qunused-arguments -Wextra -Wswitch-default -Wno-unused-parameter -Wno-missing-field-initializers -Wno-language-extension-token -Wno-extended-offsetof -Wconditional-uninitialized -Wincompatible-pointer-types-discards-qualifiers -Wmissing-variable-declarations -fPIC -DOPENSSL_USE_NODELETE -MMD -MF ssl/d1_msg.d.tmp -MT ssl/d1_msg.o -c -o ssl/d1_msg.o ../master/ssl/d1_msg.c

That's from a build of current master, and there's nothing special at all with this, I configured like this: ../master/config -d --strict-warnings
In that line, I see this: -I. -Iinclude -I../master -I../master/include
That gives the build paths to all the include files it needs.  Specifically, the last one corresponds exactly to the additional -I you suggest.
This is standard, this is how things are supposed to work, so I'm asking for a somewhat deeper analysis of why it's not working for you specifically.

Understood. I've tried to reproduce this in several ways, but failed.


Fix error when trying to exclude calendars via cli

Previous behaviour:
khal agenda -d some_calendar
[...]
AttributeError: 'tuple' object has no attribute 'remove'

Now:
works (calendar is being excluded)
Reported-by: @matthiasbeyer

Thank you for this fix!


Added ternary operator and unary plus operator

Added ternary conditional and unary plus operators, and also updated the tests.

OK, updated the tests. I also realized that some of the error checking with greater than/less than operations do not mimic JS behavior, so I removed it and updated the related specs.
Also changed TERNARY to CONDITIONAL

Did you push the commit?  I don't see it here.

Sorry, updated.

:+1:


create new release 0.6.29

What this PR does / why we need it:
Creates release 0.6.29
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

/ok-to-test
/lgtm
/approve


[WIP] Update release docs

Draft PR for now as it will include a link to a commit for the v0.17.0 release.
We introduced a new command to output the default image registry configs
for the version of Kubernetes being tested. The release process docs
did not include instructions on how to update this for a minor release
so this change includes those details.
Signed-off-by: Bridget McErlean<EMAIL_ADDRESS>

Codecov Report

Merging #1042 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1042   +/-   ##
=======================================
  Coverage   48.45%   48.45%
=======================================
  Files          77       77
  Lines        5498     5498
=======================================
  Hits         2664     2664
  Misses       2673     2673
  Partials      161      161

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 1af0378...a2525f9. Read the comment docs.


{Profile} Show tenant display_name for tenants without subscriptions

Description
#12726, #12824 used a workaround to retrieve the missing display_name from additional_properties['displayName'].
After https://github.com/Azure/azure-rest-api-specs/pull/8384 is merged and azure-mgmt-resource SDK is regenerated, this line is now (on the contrary) discarding display_name because additional_properties always exists and display_name will be overwritten with None.
Testing Guide
> az login
...
The following tenants don't contain accessible subscriptions. Use 'az login --allow-no-subscriptions' to have tenant level access.
246b1785-9030-40d8-a0f0-d94b15dc002c 'Selfhost'
ca97aaa0-5a12-4ae3-8929-c8fb57dd93d6 'jlcorp'

Profile


Refactor server code

This changes the server database to use vectors instead of maps for reduced resource use and better symmetry with the client code.
Implements critical sections on characteristic/descriptor value data to prevent concurrent access.
Fixes an issue when advertising without any services that would not advertise device name/txpower.

Nice! 😄


[Testing] Removing some multi-cluster specific logic

Please provide a description of this PR:

@nmittler: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-multicluster_istio
d2802745a11b4f88607bb97191b3725648f7eff3
link
true
/test integ-pilot-multicluster_istio

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

/retest

@nmittler: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
d2802745a11b4f88607bb97191b3725648f7eff3
link
true
/test integ-security-multicluster_istio

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

@nmittler: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
d2802745a11b4f88607bb97191b3725648f7eff3
link
true
/test integ-security-multicluster_istio

lint_istio
7af8f6dc4786266bc4b45081e89c1b76f8e8e9de
link
true
/test lint_istio

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

@nmittler: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
d2802745a11b4f88607bb97191b3725648f7eff3
link
true
/test integ-security-multicluster_istio

lint_istio
4d7cde65c308fe006b21282274253005e9aa3520
link
true
/test lint_istio

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

@nmittler: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
d2802745a11b4f88607bb97191b3725648f7eff3
link
true
/test integ-security-multicluster_istio

lint_istio
4d7cde65c308fe006b21282274253005e9aa3520
link
true
/test lint_istio

integ-pilot-multicluster_istio
4d7cde65c308fe006b21282274253005e9aa3520
link
true
/test integ-pilot-multicluster_istio

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

@nmittler: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
d2802745a11b4f88607bb97191b3725648f7eff3
link
true
/test integ-security-multicluster_istio

integ-pilot-multicluster_istio
4d7cde65c308fe006b21282274253005e9aa3520
link
true
/test integ-pilot-multicluster_istio

integ-security_istio
4d7cde65c308fe006b21282274253005e9aa3520
link
true
/test integ-security_istio

integ-security-istiodremote_istio
4d7cde65c308fe006b21282274253005e9aa3520
link
true
/test integ-security-istiodremote_istio

integ-ipv6_istio
14dea759430446b84a8195acd28eb00e5dd82066
link
true
/test integ-ipv6_istio

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

@nmittler: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
d2802745a11b4f88607bb97191b3725648f7eff3
link
true
/test integ-security-multicluster_istio

integ-pilot-multicluster_istio
4d7cde65c308fe006b21282274253005e9aa3520
link
true
/test integ-pilot-multicluster_istio

integ-security_istio
4d7cde65c308fe006b21282274253005e9aa3520
link
true
/test integ-security_istio

integ-security-istiodremote_istio
4d7cde65c308fe006b21282274253005e9aa3520
link
true
/test integ-security-istiodremote_istio

integ-ipv6_istio
14dea759430446b84a8195acd28eb00e5dd82066
link
true
/test integ-ipv6_istio

integ-distroless_istio
14dea759430446b84a8195acd28eb00e5dd82066
link
true
/test integ-distroless_istio

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

@nmittler: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
d2802745a11b4f88607bb97191b3725648f7eff3
link
true
/test integ-security-multicluster_istio

integ-security_istio
4d7cde65c308fe006b21282274253005e9aa3520
link
true
/test integ-security_istio

integ-security-istiodremote_istio
4d7cde65c308fe006b21282274253005e9aa3520
link
true
/test integ-security-istiodremote_istio

integ-ipv6_istio
14dea759430446b84a8195acd28eb00e5dd82066
link
true
/test integ-ipv6_istio

integ-distroless_istio
14dea759430446b84a8195acd28eb00e5dd82066
link
true
/test integ-distroless_istio

integ-pilot-multicluster_istio
14dea759430446b84a8195acd28eb00e5dd82066
link
true
/test integ-pilot-multicluster_istio

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

@nmittler: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
d2802745a11b4f88607bb97191b3725648f7eff3
link
true
/test integ-security-multicluster_istio

integ-security_istio
4d7cde65c308fe006b21282274253005e9aa3520
link
true
/test integ-security_istio

integ-security-istiodremote_istio
4d7cde65c308fe006b21282274253005e9aa3520
link
true
/test integ-security-istiodremote_istio

integ-distroless_istio
14dea759430446b84a8195acd28eb00e5dd82066
link
true
/test integ-distroless_istio

integ-ipv6_istio
06678d9eb8626ad630cc1eef71187ab9be63eec8
link
true
/test integ-ipv6_istio

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

@nmittler: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
d2802745a11b4f88607bb97191b3725648f7eff3
link
true
/test integ-security-multicluster_istio

integ-security_istio
4d7cde65c308fe006b21282274253005e9aa3520
link
true
/test integ-security_istio

integ-security-istiodremote_istio
4d7cde65c308fe006b21282274253005e9aa3520
link
true
/test integ-security-istiodremote_istio

integ-ipv6_istio
06678d9eb8626ad630cc1eef71187ab9be63eec8
link
true
/test integ-ipv6_istio

integ-distroless_istio
06678d9eb8626ad630cc1eef71187ab9be63eec8
link
true
/test integ-distroless_istio

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

@nmittler: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-istiodremote_istio
4d7cde65c308fe006b21282274253005e9aa3520
link
true
/test integ-security-istiodremote_istio

integ-pilot-multicluster_istio
f7235f598a2dbf849db739ee12e4845ad3481a04
link
true
/test integ-pilot-multicluster_istio

integ-security_istio
f7235f598a2dbf849db739ee12e4845ad3481a04
link
true
/test integ-security_istio

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

@nmittler: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-multicluster_istio
f7235f598a2dbf849db739ee12e4845ad3481a04
link
true
/test integ-pilot-multicluster_istio

integ-security_istio
f7235f598a2dbf849db739ee12e4845ad3481a04
link
true
/test integ-security_istio

integ-security-istiodremote_istio
f7235f598a2dbf849db739ee12e4845ad3481a04
link
true
/test integ-security-istiodremote_istio

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

@nmittler: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-istiodremote_istio
f7235f598a2dbf849db739ee12e4845ad3481a04
link
true
/test integ-security-istiodremote_istio

integ-security_istio
15a748a4702407e52bf268deab0ff1c2f0b3904d
link
true
/test integ-security_istio

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

@nmittler: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
7ce207881cc15d6b5584203c40ca4223a17b3f63
link
true
/test integ-security-multicluster_istio

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

@nmittler: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
985f1b6430cb8d3d80bd3405c7ee0388b2771ed4
link
true
/test integ-security-multicluster_istio

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

@nmittler: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
985f1b6430cb8d3d80bd3405c7ee0388b2771ed4
link
true
/test integ-security-multicluster_istio

lint_istio
8ed3b95ec5dcce47bf15f0db7d70f9dc0e5eac18
link
true
/test lint_istio

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

@nmittler: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
8ed3b95ec5dcce47bf15f0db7d70f9dc0e5eac18
link
true
/test lint_istio

integ-security-multicluster_istio
8ed3b95ec5dcce47bf15f0db7d70f9dc0e5eac18
link
true
/test integ-security-multicluster_istio

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

@nmittler: PR needs rebase.

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

@nmittler: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-telemetry-mc_istio
3e54119fc6bd71bd20847de9aa8ab5f4f151b790
link
true
/test integ-telemetry-mc_istio

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

/retest


removed global padding from rev content

Connect #378
This looks a little funny on harmonium but I think @vegasgeek needed to use a component that has the .rev-Content class which includes padding by default. no test included for the minor change. first working session, lmk if this is ok! :)

@CheetoMao Hey Laura,
lmk what you think about this PR before I merge.


notebook for modeling real data

Codecov Report

Merging #458 into master will decrease coverage by 0.17%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #458      +/-   ##
==========================================
- Coverage   86.83%   86.65%   -0.18%
==========================================
  Files          23       23
  Lines        1747     1746       -1
==========================================
- Hits         1517     1513       -4
- Misses        230      233       +3

Flag
Coverage Δ

#unittests
86.65% <0.00%> (-0.18%)
:arrow_down:

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

Impacted Files
Coverage Δ

lineage/states/StateDistributionGaPhs.py
81.29% <0.00%> (-1.80%)
:arrow_down:

lineage/states/StateDistributionExpon.py
89.00% <0.00%> (-0.22%)
:arrow_down:

lineage/states/stateCommon.py
96.07% <0.00%> (-0.15%)
:arrow_down:

lineage/tests/test_BaumWelch.py
96.00% <0.00%> (ø)

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 f091840...d7b9e50. Read the comment docs.


[WIP] Do not merge

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.

adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

/azp run automation - sdk

Close this since we are not receiving any update on this PR in the last two weeks


Watchdog Feature

Watchdog feature branch - List of changes:

main.py: Added process restart by name ability (from a blocking Queue of hanging processes)
StateManager.py: Added Watchdog thread
RoverProcess.py: Added hooks to the Watchdog thread

This was forked a long time ago, so it might take some time to clean up. Sorry @ottopasuuna  X]
Closes issue #63

Awesome. I'll take a look and test it on the rover tomorrow to make sure everything works as expected.


Use 50 GB Parquet+PyArrow dataset for H2O tests on CI

Similar to #1530, the 5 GB dataset feels too small to benchmark behavior we care about.

how long is one test run (compared to the rest of the suite)?

how long is one test run (compared to the rest of the suite)?

I'll have to convert this back to draft, it looks like some of the workloads run OOM.

Some of these workloads cause workers to OOM, so I'm shelving this for now and will look into it at a later point.


--paths flag doesn't support wildcards

Currently, the sqlmesh plan command requires explicit specification of each project path using the -p flag. This limitation poses challenges for us managing multiple SQLMesh projects (one for each source system) with interdependencies, including downstream aggregation of sources, shared utility projects containing macros and tests.
Solution
Introduce support for wildcard patterns in the -p flag of the sqlmesh plan command. This enhancement would allow users to specify multiple projects efficiently. For example, using -p all_projects/* would include all projects within the all_projects directory.

Fixed by linked PR.


Custom data

Congratulations on this excellent work!
I wonder how to run this work on my own data. For example, after capturing a monocular video, how to run your method? How should I process the data for training?
Many thanks!

Thanks for your interest. Our code was developed guided by InstantAvatar https://github.com/tijiang13/InstantAvatar, and indeed, we have a data loader for their preprocessing data format. Our data loading subroutine for UBC data uses the InstantAvatarWildDataset class, which you can modify a little bit and use for the preprocessed data from InstantAvatar preprocessing. (you may take some time to compile the openpose .etc for their preprocessing)

Thanks for your reply. In the paper you write you use ReFit to obtain the human pose. If I follow ReFit to obtain the pose, can I abandon openpose? Since the openpose is not easy to compile.

We tried both using InstantAvatar pre-processing to estimate video poses (with temporal optimization), and ReFit https://yufu-wang.github.io/refit_humans/ to estimate per-frame poses. It turns out that under the challenging UBC sequences, Ours and InstantAvatar will work better with ReFit poses. So we first estimate per-frame poses and I manually turn the poses into the same format as InstantAvatar preprocessing. So the data loader is actually loading in the Instant-Avatar preprocessing format.

We tried both using InstantAvatar pre-processing to estimate video poses (with temporal optimization), and ReFit https://yufu-wang.github.io/refit_humans/ to estimate per-frame poses. It turns out that under the challenging UBC sequences, Ours and InstantAvatar will work better with ReFit poses. So we first estimate per-frame poses and I manually turn the poses into the same format as InstantAvatar preprocessing. So the data loader is actually loading in the Instant-Avatar preprocessing format.

Thanks. Could you share the script for transferring the ReFit pose into InstantAvatar format?

I have tried to perform the conversion from ReFit result to InstantAvatar format, but the converted data does not work. The code for conversion is as follows:
# self.smpl_poses is the results of ReFit
pose = []
for x in self.smpl_poses['pred_rotmat'][idx]:
    d, angle = mat2axangle(x)
    pose.append(d * angle)
pose = np.stack(pose).astype(np.float32)
# pose[0] = -pose[0]

ret = {
    "rgb": img.astype(np.float32),
    "mask": msk,
    "K": self.K.copy(),
    "smpl_beta": self.smpl_poses['pred_shape'][idx],  # ! use the first beta ???
    "smpl_pose": pose,
    "smpl_trans": self.smpl_poses["trans_full"][idx, 0],
    "idx": idx,
}

May I ask where is the problem? Many thanks!

# convert our mono-pose estimation to ubc fashion dataset
import numpy as np
import os, os.path as osp
import imageio
from pytorch3d.transforms import matrix_to_axis_angle
import torch
from tqdm import tqdm
from pycocotools import mask as masktool
import cv2

def process(seq):
    img_src = f"../data/ubcfashion/train_frames/{seq}/"
    msk_fn = f"../data/ubcfashion/train_mask/{seq}.npy"
    pose_fn = f"../data/ubcfashion/train_smpl/{seq}.npz"
    dst = f"../data/insav_wild/ourpose_ubc_{seq}/"
    os.makedirs(dst, exist_ok=True)

    pose_data = np.load(pose_fn)
    smpl_shape = pose_data["pred_shape"].mean(0)  # Use the average shape
    smpl_pose_list, smpl_global_trans = (
        pose_data["pred_rotmat"],
        pose_data["pred_trans"],
    )
    smpl_pose_list = matrix_to_axis_angle(torch.from_numpy(smpl_pose_list))
    smpl_pose_list = smpl_pose_list.numpy()

    focal, center = pose_data["img_focal"], pose_data["img_center"]
    K = np.eye(3)
    K[0, 0], K[1, 1] = focal, focal
    K[0, 2], K[1, 2] = center[0], center[1]

    pose_save_dict = {
        "betas": smpl_shape,
        "global_orient": smpl_pose_list[:, 0],
        "body_pose": smpl_pose_list[:, 1:].reshape(-1, 69),
        "transl": smpl_global_trans.squeeze(1),
    }
    np.savez_compressed(osp.join(dst, "poses_optimized.npz"), **pose_save_dict)

    image_save_dst = osp.join(dst, "images")
    mask_save_dst = osp.join(dst, "masks")
    os.makedirs(image_save_dst, exist_ok=True)
    os.makedirs(mask_save_dst, exist_ok=True)

    mask_data = np.load(msk_fn, allow_pickle=True)
    masks = [masktool.decode(m).astype(np.bool).astype(np.float32) for m in mask_data]

    for img_fn in tqdm(sorted(os.listdir(img_src))):
        image_id = int(img_fn.split(".")[0])
        mask = masks[image_id]
        img = cv2.imread(osp.join(img_src, img_fn))
        cv2.imwrite(osp.join(image_save_dst, img_fn), img)
        cv2.imwrite(osp.join(mask_save_dst, img_fn), mask * 255)

    cam_save_dict = {
        "intrinsic": K,
        "extrinsic": np.eye(4),
        "height": img.shape[0],
        "width": img.shape[1],
    }
    np.savez_compressed(osp.join(dst, "cameras.npz"), **cam_save_dict)

if __name__ == "__main__":
    # seqs = sorted(os.listdir("../data/ubcfashion/train_frames"))
    seqs = ["91+bCFG1jOS"]

    for seq in seqs:
        process(seq)

# def load_insav_smpl_param(path):
#     smpl_params = dict(np.load(str(path)))
#     if "thetas" in smpl_params:
#         smpl_params["body_pose"] = smpl_params["thetas"][..., 3:]
#         smpl_params["global_orient"] = smpl_params["thetas"][..., :3]
#     return {
#         "betas": smpl_params["betas"].astype(np.float32).reshape(1, 10),
#         "body_pose": smpl_params["body_pose"].astype(np.float32),
#         "global_orient": smpl_params["global_orient"].astype(np.float32),
#         "transl": smpl_params["transl"].astype(np.float32),
#     }

# insav_pose_fn = "../data/insav_wild/91+20mY7UJS/poses_optimized.npz"
# load_insav_smpl_param(insav_pose_fn)
# insav_cam_fn = "../data/insav_wild/91+20mY7UJS/cameras.npz"
# insav_cam = dict(np.load(insav_cam_fn, allow_pickle=True))
# for k, v in insav_cam.items():
#     print(k, v.shape, v.dtype)

# print()

This is the script I used to convert the poses into instant-avatar format, hope it may help you.

Hi, I am trying to use Refit to get SMPL and camera parameters from monocular videos and it seems that the estimation of focal deeply relies on some assumption and cannot be used directly. The result is shown below, could you please provide some advice about this case? Or how did you estimate the focal for the UBC dataset?

@uniBruce Hi. When the ground truth focal is unavailable, we estimate it from the dimention of the image as $\sqrt{h^2+w^2}$ as in here. Your image seems to be cropped from another image, so the focal estimation won't be accurate (this may not affect GART though). However, your ReFit result looks expected from my experience because it's not trained with crop-augmentation so the accuracy will drop a bit when the human is cropped like this.
I also added a script in the ReFit repo (here) that runs on a folder of images and save pose results compatiable for GART. Please give it a try.

We tried both using InstantAvatar pre-processing to estimate video poses (with temporal optimization), and ReFit https://yufu-wang.github.io/refit_humans/ to estimate per-frame poses. It turns out that under the challenging UBC sequences, Ours and InstantAvatar will work better with ReFit poses. So we first estimate per-frame poses and I manually turn the poses into the same format as InstantAvatar preprocessing. So the data loader is actually loading in the Instant-Avatar preprocessing format.

hi! when I use the data which processed by the instantAvatar, after ./scripts/fit.sh , the resutl is bad, like this:

We tried both using InstantAvatar pre-processing to estimate video poses (with temporal optimization), and ReFit https://yufu-wang.github.io/refit_humans/ to estimate per-frame poses. It turns out that under the challenging UBC sequences, Ours and InstantAvatar will work better with ReFit poses. So we first estimate per-frame poses and I manually turn the poses into the same format as InstantAvatar preprocessing. So the data loader is actually loading in the Instant-Avatar preprocessing format.

hi! when I use the data which processed by the instantAvatar, after ./scripts/fit.sh , the resutl is bad, like this: can you give me some suggestions?

I can't find ./scripts/fit.sh.  Which yaml file should I use to fit on my own video data? @JiahuiLei

Hi, when I use real person video with da_pose pose, data preprocessed like InstantAvatar and trained GART for more than 25000 steps, the result is quite good but when you zoom in the image below, you can see RGB noise in the pants and shoes (pants should be really black, sleeves and shirt should be "smooth"), how should I edit the config or loss function to make the image sharper or the problem lies in my ground truth image?
This is my config in ubc_mlp.yaml
TOTAL_steps: 50000 #15000 #30000
SEED: 12345

VIZ_INTERVAL: 500

CANO_POSE_TYPE: da_pose #da_pose #t_pose #da_pose
VOXEL_DEFORMER_RES: 64 #128 #64 #64 #128 #64 #128 #64

W_CORRECTION_FLAG: True
W_REST_DIM: 32 #0 #16
W_REST_MODE: pose-mlp #delta-list #pose-mlp
W_MEMORY_TYPE: voxel #voxel #point

F_LOCALCODE_DIM: 0

MAX_SCALE: 1.0
MIN_SCALE: 0.0 #0.0003 #0.003 #3
MAX_SPH_ORDER: 4
INCREASE_SPH_STEP: [3000, 5000, 6000, 7000] #[3000, 5000, 6000, 7000] #[1000, 2000, 3000]

INIT_MODE: on_mesh #near_mesh #near_mesh
OPACITY_INIT_VALUE: 0.99

ONMESH_INIT_SUBDIVIDE_NUM: 1
ONMESH_INIT_SCALE_FACTOR: 1.0
ONMESH_INIT_THICKNESS_FACTOR: 0.5

NEARMESH_INIT_NUM: 10000
NEARMESH_INIT_STD: 0.1
SCALE_INIT_VALUE: 0.01 # only used for random init

###########################

LR_P: 0.00016
LR_P_FINAL: 0.0000016
LR_Q: 0.001
LR_S: 0.005
LR_O: 0.05

LR_SPH: 0.0025
LR_SPH_REST: 0.0005

W_START_STEP: 500 #1000 #500 #2000 #300 #2000
LR_W: 0.0002 # 1 # 0.00001
LR_W_FINAL: 0.00002

LR_W_REST: 0.0002
LR_W_REST_FINAL: 0.00002
LR_W_REST_BONES: 0.0003 # for mlp

LR_F_LOCAL: 0.0

# Pose Optimize
POSE_R_BASE_LR: 0.0001
POSE_R_BASE_LR_FINAL: 0.00001
POSE_R_REST_LR: 0.0003
POSE_R_REST_LR_FINAL: 0.00001
POSE_T_LR: 0.0001
POSE_T_LR_FINAL: 0.00001
POSE_OPTIMIZE_START_STEP: 500 #1000

# Reg Terms
LAMBDA_MASK: 0.0 #0.01
MASK_LOSS_PAUSE_AFTER_RESET: 100

# other optim
N_POSES_PER_STEP: 1 #50 #1 #3 # increasing this does not help
RAND_BG_FLAG: True #True #True #True
# DEFAULT_BG: [0.0, 0.0, 0.0]
NOVEL_VIEW_PITCH: 0.0
IMAGE_ZOOM_RATIO: 1.0
VIEW_BALANCE_FLAG: True #True # True #True #False
BOX_CROP_PAD: 50

# GS Control
# densify
MAX_GRAD: 0.0002 #0.0003 #0.0005 #0.0006 # 0.0002
PERCENT_DENSE: 0.005 #0.01
DENSIFY_START: 500
DENSIFY_INTERVAL: 100 #300 #500 #1000 #300
DENSIFY_END: 9000 #10000 #15000
# prune
PRUNE_START: 500
PRUNE_INTERVAL: 300
OPACIT_PRUNE_TH: 0.01
RESET_OPACITY_STEPS: [3000, 5000] #[3000, 5000] #5000 #3000
OPACIT_RESET_VALUE: 0.01
# regaussian
REGAUSSIAN_STD: 0.015 #0.02 #0.02 #0.01
REGAUSSIAN_STEPS: [7000. 14000]

CANONICAL_SPACE_REG_K: 6
LAMBDA_STD_Q: 0.01
LAMBDA_STD_S: 0.01
LAMBDA_STD_O: 0.01
LAMBDA_STD_CD: 0.03
LAMBDA_STD_CH: 0.03
# LAMBDA_STD_W: 0.3
# LAMBDA_STD_W_REST: 0.3
LAMBDA_STD_W: 0.3
LAMBDA_STD_W_REST: 0.1
LAMBDA_KNN_DIST: 0.00

LAMBDA_W_NORM: 0.01
LAMBDA_W_REST_NORM: 0.1

START_END_SKIP: [0, 400, 1]


Return results when no .travis.yml in the repo

Closes #16

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Compile command missing

In https://github.com/ARMmbed/reference-design-RD1168/blob/master/Documentation/RD1168_README.md, if following SPI path (step 4 -> step 11) there's no compile command to build the app

Thanks @ranshe . Fixed by commit # 0d6496eb8f5