server: container_remove: ignore not existent exit file

Found out that during OpenShift testing, node was trying to remove
containers (probably in a bad state) and was failing the removal with
this kind of error:
E0828 13:19:46.082710    1235 kuberuntime_gc.go:127] Failed to remove
container
"e907f0f46b969e0dc83ca82c03ae7dd072cfe4155341e4521223d9fe3dec5afb": rpc
error: code = 2 desc = failed to remove container exit file
e907f0f46b969e0dc83ca82c03ae7dd072cfe4155341e4521223d9fe3dec5afb: remove
/var/run/crio/exits/e907f0f46b969e0dc83ca82c03ae7dd072cfe4155341e4521223d9fe3dec5afb:
no such file or directory
I believe it's ok to ignore this error as it may happen conmon will
fail early before exit file is written.
Signed-off-by: Antonio Murdaca<EMAIL_ADDRESS>

@mrunalp PTAL

LGTM

[retest]

[retest]

this will trigger CI twice Dan :confused: the first push should already kick a CI job

I thought we did not do CI Job until Human said [retest].  I think that should be the default.  Has to get LGTM and pass travis, and then human says LGTM.

LGTM

kpod remove tests are failing because of this bug
https://github.com/kubernetes-incubator/cri-o/pull/783

Did you see any container status errors?

On Aug 28, 2017, at 6:38 AM, Antonio Murdaca<EMAIL_ADDRESS>wrote:
@mrunalp PTAL
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@mrunalp nope, I was just getting that error on remove, didn't dig enough to find why container was failing to start. I'll debug better tomorrow after another run of the OpenShift tests.

[retest]

Ci is failing in other PR as well.....

@mrunalp I think there is a potential race condition, we were testing with kpod remove and it blows up since the exit file does not exist.

@rhatdan Yeah, I will create a PR for us to wait for exit file for some time. The kube is going too fast for even the exit file to get created. Just want to confirm whether this is the same issue or not. If we merge this without also identifying the root cause, we will be hiding races.
@runcom We should get the conmon logs from journalctl -u crio-conmon-containerid.service to see whether it failed or this is the race between exit file creation and kube asking for status.

Mmm I may be missing something but what's the issue in this case of container remove? Container removal will always try to get rid of resources and stuff so it made sense to ignore a non existent exit file - I believe the container wasn't simply starting in conmon, and conmon edited before even writing the exit file, then container removal fails. I'll try to write a test case for this, sounds like a simple removal but to me..

Actually, the container shouldn't be left around if conmon failed to start it, nor sure, I'll check when I'll come online tomorrow. One thing could be container started, exited, exit not written, failure in removal...

@runcom I have no objections to the patch. Just want to make sure that we understand and try to fix if there are any races lurking. I know for sure the one between exit file creation and status.

Mm yeah, I think I understand the race now, I'm not sure about exit/status, we may wait, but ultimately kubernetes should poll with status and wait for a given exit code but yeah, this is another issue.

@mrunalp PTAL, added some delay in UpdateStatus to wait for exit file to show up

[retest]

LGTM

LGTM


Adapt rp2350 to ref_app architecture

The rp2350 port to ref_app bare-metal is rudimentary. The purpose of this issue is to adapt that solid boot/startup the standard ref_app architecuture.

First steps have been accomplished in branches. But we still need to adapt @chalandi code to the stnadard form of the ref_app. This was a tough bare-metal boot and it was due to the work of Amine that made this possible. Thx Amine.


Add option MarkNullOrEmpty to enhance the nullRef

null or empty   element will be marked  to enhance the prev solution "Preserving Null-References Identity"
    public class NullableSample4
    {
        public string? Text01 { get; set; }
        public string? Text02 { get; set; }
        public List<int> TheCollection01 { get; set; }
        public List<int> TheCollection02 { get; set; }

        public int[] IntArr01 { get; set; } = new int[0];
        public int[] IntArr02 { get; set; } = null;

        public static NullableSample4 GetSampleInstance()
        {
            return new()
            {
                Text01 = "",
                Text02 = null,
                TheCollection01 = new List<int> { },
                TheCollection02 = null,
                IntArr01 = new int[0],
                IntArr02 = null,
            };
        }

    }

Using it
        var serializer = new YAXSerializer<NullableSample4>(new SerializerOptions
        {
            SerializationOptions = YAXSerializationOptions.MarkNullOrEmpty
        });
        var instance = NullableSample4.GetSampleInstance();
        var got = serializer.Serialize(instance);

The result will be

<NullableSample4>
    <Text01></Text01>
    <Text02 _MarkNullOrEmpty="NULL" />
    <TheCollection01 _MarkNullOrEmpty="EMPTY" />
    <TheCollection02 _MarkNullOrEmpty="NULL" />
    <IntArr01 _MarkNullOrEmpty="EMPTY" />
    <IntArr02 _MarkNullOrEmpty="NULL" />
</NullableSample4>

Thanks for the thoughts about improving YAXLib and the PR.
Which other alternatives have you evaluated, that don't bring the results you need?
E.g. wouldn't the following code, bring the same outcome (except for the generated XML)?
public class NullableSample4
{
    public string Text01 { get; set; } = string.Empty;
    public string? Text02 { get; set; }
    public List<int> TheCollection01 { get; set; } = new();
    [YAXDontSerializeIfNull]
    [YAXErrorIfMissed(YAXExceptionTypes.Ignore)]
    public List<int>? TheCollection02 { get; set; }
    public int[] IntArr01 { get; set; } = Array.Empty<int>();
    [YAXDontSerializeIfNull]
    [YAXErrorIfMissed(YAXExceptionTypes.Ignore)]
    public int[]? IntArr02 { get; set; }

    public static NullableSample4 GetSampleInstance1()
    {
        return new()
        {
            Text01 = "",
            Text02 = null,
            TheCollection01 = new (),
            TheCollection02 = null,
            IntArr01 = Array.Empty<int>(),
            IntArr02 = null,
        };
    }
}

[Test] // including default values
public void Pr_242_1()
{
    var expectedXml = """
                      <NullableSample4>
                        <Text01></Text01>
                        <Text02 />
                        <TheCollection01 />
                        <IntArr01 />
                      </NullableSample4>
                      """;
  var s = new YAXSerializer<NullableSample4>();
  var obj = NullableSample4.GetSampleInstance1();
  var xml = s.Serialize(obj);
  var result = s.Deserialize(xml);

  Assert.That(xml, Is.EqualTo(expectedXml));
  result.Should().BeEquivalentTo(obj);
}

[Test] // don't serialize default values
public void Pr_242_2()
{
    var expectedXml = """
                      <NullableSample4>
                        <Text01></Text01>
                        <TheCollection01 />
                        <IntArr01 />
                      </NullableSample4>
                      """;
    var s = new YAXSerializer<NullableSample4>(new SerializerOptions {SerializationOptions = YAXSerializationOptions.DoNotSerializeDefaultValues});
    var obj = NullableSample4.GetSampleInstance1();
    var xml = s.Serialize(obj);
    var result = s.Deserialize(xml);

    Assert.That(xml, Is.EqualTo(expectedXml));
    result.Should().BeEquivalentTo(obj);
}

What are you referring to with "the prev solution "Preserving Null-References Identity"?

Thanks for your consideration.
Your solution is work , of course.
I perfer xml to ini  as my config files always.    Sometimes i  would  modify the xml config file manually.
It confuse me usually which the
<Text01></Text01>
 <Text01/>
dispappear

even   the meaning will change depending on the YAXSerializationOptions  and yalibAttribut.
so, my solution is mark it directly and explicit.

<Text01></Text01>
<Text01/>
no element

There is no canonical answer to what should be used, since XML fundamentally has no null concept.
A common way to explicitly represent null values in XML is by using the xsi:nil attribute. So I would prefer implementing this attribute over creating a new one, and use empty elements like <Text01/> for "empty".
Both could go into a new attribute derived from YAXBaseAttribute like YAXExplicitNullOrEmpty (not fully thought through yet).

@strugglesharp Do you have a plan to work on this PR? E.g. implement the xsi:nil attribute?


Dataset question

Besides taking the video frames as described in the paper, did you also warp one of the pair along the lines of DeTone? Otherwise the generalization on heavily skewed pairs is surprising, considering videos never seem to have that much distortion.

Nice question. Detone's contribution lies in the novel method of homography estimation. But model trained from synthetic image pairs can't estimate an accurate homography between two images from real world. At present, our approach cannot solve the case of very large distortion. @steplee


Added code to support complex type

added code to support complex type query execution

retest this please

retest this please

@kumarvishal09 Use null to return instead of datatype, that should remove serialization problem

fix later: raise another issue. however remaining LGTM


APScheduler v3.1.0 breaks mommy implementation

Thanks doppins :D

I'm sure we want up to date requirements, so this would've happened sooner or later anyways.

https://stackoverflow.com/questions/36437702/advance-python-scheduler-tuple-object-has-no-attribute-public-while-starting


ImportError: cannot import name 'Binary' from 'sqlalchemy'

from sqlalchemy import Binary, Column, Integer, String
ImportError: cannot import name 'Binary' from 'sqlalchemy' (C:\Users\kenqq\AppData\Local\Programs\Python\Python38\lib\site-packages\sqlalchemy_init_.py)
To use this template for my project used as a guideline but i met this error, how to solve it?

Hello @gracefoong,
Thank you for using our products. This problem has been fixed in the latest release.
Please use a fresh install (if not already), create a virtual environment, and install dependencies using the command:
pip3 install -r requirements.txt
Check the installed version of flask_sqlalchemy and sqlalchemy using python console:
$ python
>>> import sqlalchemy
>>> sqlalchemy.__version__
>>>
>>> import flask_sqlalchemy
>>> flask_sqlalchemy.__version__

Let us know the output.

P.S. The sample output on Windows:
(env) PS > python.exe

Python 3.8.4 (tags/v3.8.4:dfa645a, Jul 13 2020, 16:46:45) [MSC v.1924 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import sqlalchemy
>>> import flask_sqlalchemy
>>> sqlalchemy.__version__
'1.3.23'
>>> flask_sqlalchemy.__version__
'2.4.4'
>>>


Reporting() seems to fail

no matter what I change in the parameters i get
"TypeError: include and exclude must both be non-string sequences"
which i assume is cause by rows containing strings such
"<class   'keras.optimizers.Adam'>"
is this due to me using my own Adam optimizer?
other columns include
kernel_initializer
activations
losses
..so on

Could you post the whole trace / error message and also the code it results from.

To avoid doubt, Reporting() is intended to be used only once you've already completed a scan and have a .csv file with the results of the scan.

It seems to have something to do with the values in the columns that are detected to be float. Can you paste a some rows of the CSV or dump in pastebin or similar and I will try to replicate in my end. I think it should be something straightforward to resolve but might require a few more rounds of back and forth. Thanks.

https://pastebin.com/XVW2HiTg

Thanks. It seems that the .csv you have is not comma separated (at least that is what I see in the dump). Is that the case and if it, are you doing any manipulation to the file or that is what you get from Talos?

Sorry, I opened it in excel without thinking
https://pastebin.com/iudEfu89

@ClarkN3 The file you had provided opens just fine with ta.Reporting() on my machine.
Could you try the following:

save the pastebin dump to file test.csv and

create python virtualenv

virtualenv -p python3.6 talos_test
source talos_test/bin/activate
pip install -U talos

open python console

python

run the code in the python console

import talos as ta
ta.Reporting('test.csv')

Then if it works, can you try the same but now in conda (as I see you are running on conda) so we eliminate the most obvious possibilities first. Thanks!

because I could not replicate this and did not hear from you, closing this now. Let me know if it's still an issue and I'll reopen.


Improve realiability with tunnel connections

add a retry to tunnels, this should resolve some problems related for static sockets
try to find the access token from env for retro compatibility

yeah, ship it. can't replicate the issue with static sockets


ui changed

Related Issue

Issue No. #

Changes you made :

UI Changed
and Padding between tm-banner and tm-home-section-1
Describe the changes :

Check list :

[x] I have followed code style of this project.
[x] I have read the CONTRIBUTING document.
[x] All the aspects mentioned in the issue are covered.
[x] The website is properly working after my changes.
[x] 🌟 ed the repo

Type of change :

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Output Screenshots :

@VIVEKPRATAP11 Thank you for this PR
Plz add the issue number, description, screenshots

@VIVEKPRATAP11 you have to create an seperate branch for each new PR
You can't push commits into main branch.
Also you have to create issue first and then a PR.


stderr => stderror, avoid explicit Formula constructor

Should fix #100 by going through the @formula macro

Coverage remained the same at 95.146% when pulling fcef99624395d42e6b4a6d7e3960f38460f02b44 on StatsModels0.2.4 into fd22daf226938a4f98ce3d4b228e5a99437e3218 on master.


How to listen to swipe events? I can't found any eventEmitter for this wrapper

@nasserghiasi
you can find them here.
http://idangero.us/swiper/api/


fix missed repos in .cent.yaml in README

Update .cent.yaml in README. Some users view only README and think repos in this file contains same repos in cent.yaml

Thank you @szybnev for your contribution


feat(*): Add app version to history table

What this PR does / why we need it:
This PR adds "App Version" column to the helm history table output.
Closes #5205
Special notes for your reviewer:
If applicable:

[x] this PR contains documentation
[x] this PR contains unit tests
[x] this PR has been tested for backwards compatibility

@bacongobbler Updated and ready for review again.


Week 3 Slides - Signatures / Magic Numbers / Hashing

https://github.com/AOrps/SigMal/blob/master/educational_material/sem3/03.md


ConnectionError(Timed out during query execution.) error when using Promise.all for SQLite

Bug description
Just like the title explains,, when I use Promise.all to delete (in my case), it throws timed out error.
Never had the same issue with other db such as postgres and mysql.  But for sqlite, i am getting this issue.
Is there any workaround or legit solution that I am not aware of for this error?
How to reproduce
model Category {
  id                          String                      @id @default(cuid())
  description                 String?
  name                        String                      @unique
  slug                        String                      @unique
  // *************** RELATIONS *****************
  subcategories               Category[]                  @relation("SuperCategoriesSubCategories", references: [id])
  supercategories             Category[]                  @relation("SuperCategoriesSubCategories", references: [id])
  // *******************************************
  createdAt                   DateTime                    @default(now())
  updatedAt                   DateTime                    @default(now())
  deletedAt                   DateTime?
}

// For front
Promise.all([
  ...send multiple delete request
])

// For backend
prisma.category.delete({
      where: {
        id,
      }
})

Expected behavior
Delete all passed categories
Prisma information

Environment & setup

OS:
Database:
Node.js version:

Prisma Version
3.9.2

Is it possible this was previously reported under https://github.com/prisma/prisma/issues/9562? Can you please check?

Note: there are multiple issues about this now all grouped under https://github.com/prisma/prisma/issues?q=is%3Aissue+is%3Aopen+label%3A"topic%3A+Timed+out+during+query+execution" One of them is https://github.com/prisma/prisma/issues/10403 which points out that our error message here is currently highly misleading and should be changed/fixed.

Same here prisma 4.16.1 with sqlite as driver.

Same here prisma 4.16.1 with sqlite as driver.

@gengjiawen did you end up resolving this?

Same here prisma 4.16.1 with sqlite as driver.

@gengjiawen did you end up resolving this?

Nope, I changed db driver to mysql, problem goes away.

@gengjiawen Thanks

Internal note: A lot of similar issues are grouped into this cluster. https://github.com/prisma/prisma/issues/9562 Looks like a good candidate for reproduction.

Update ✅: After hours of troubleshooting Prisma and Drizzle, I tried Sequelize and found out it works flawlessly with SQLITE and also supports WAL mode via a query command.
import {
    Sequelize,
    DataTypes
} from "sequelize";

const sequelize = new Sequelize({
    dialect: "sqlite",
    storage: "test.db", // Replace with the path to your SQLite database file
});

const User = sequelize.define("User", {
    username: DataTypes.STRING,
    birthday: DataTypes.DATE,
});

// migrate schema to file
await User.sync();

// enable wal mode (optional)
await sequelize.query("PRAGMA journal_mode=WAL;");

const jane = await User.create({
    username: "janedoe",
    birthday: new Date(1980, 6, 20),
});

const users = await User.findAll();

console.log(jane.username, users.length);

Bump

5.2.0 still getting the same error with sqlite

@luluhoc
There may be other processes working with the database, I had the same problem.
After terminating the unnecessary process, the problem was solved.

Getting the same error on my end. But I am not using Promise.all. I am calling myClient.$transaction multiple times. I guess they are trying to run in parallell and Prisma doesn't play nice with paralell requests on SQLite... ?

Can confirm, the issue occurs on the usage of $transaction with SQLite if you use more than one $transaction or many individual changes inside $transaction. I don't really have to anything to add expect I too suspect prisma isn't playing well with SQLite.

Hi, I have the same problem with sqlite/prisma and express. I expose my update procedure through a router in express. When called in the same time for different record, prisma and sqlite throw the error.
For now the only thing that mitigate the problem it been put the route in syncronic  mode trought a middleware but in this way I lost the performace of node and express.

I can confirm this issue exists when invoking a delete() procedure in express and elysia js

In case it's helpful to anyone affected by this:
A workaround until the issue is fixed appears to be enabling WAL mode in SQLite and appending ?connection_limit=1 to the connection string in schema.prisma. See https://github.com/prisma/prisma/issues/10403#issuecomment-1500770484 for details.

Hi @crishoj what you suggest is correct if you open the file only one time and maintain open for all write or update. But, as in my case, if you have to opened and closed ripetuttaly the file the problem is araise again.
In this way it works
const prisma = new PrismaClient();
async function updatetUser(id: number) {
  let user: User;
  try {
    user = await prisma.user.update({
      where: {id,},
      data: {email<EMAIL_ADDRESS>    });
  } catch (error) {
    console.log(error);
  } finally {
   // await prisma.$disconnect();
  }
  return user;
}

in this way not
async function updatetUser(id: number) {
  let user: User;
  const prisma = new PrismaClient();
  try {
    user = await prisma.user.update({
      where: {id,},
      data: {email<EMAIL_ADDRESS>    });
  } catch (error) {
    console.log(error);
  } finally {
   await prisma.$disconnect();
  }
  return user;
}

thanks for your suggest

I found out, after reproducing this error with concurrent queries in https://github.com/prisma/prisma/pull/22723
That ?connection_limit=1 to the url made this error disappear -> The queries were failing and are successful with the added parameter.
Example
generator client {
  provider = "prisma-client-js"
}

datasource db {
  provider = "sqlite"
  url      = "file:./dev.db?connection_limit=1"
}

If you try that, let us know if that works for you, or not, we're curious to hear from you 🙌🏼

I'm closing this issue as a duplicate of https://github.com/prisma/prisma/issues/9562


Initial release

We are on our way to making an initial release and doing formal announcement.
Checklist:

[x] Finish documentation (sufficiently)
[x] Create release tag, with empty CHANGELOG.md

[x] For future PRs, setup guidelines to update the CHANGELOG

auto-generate CHANGELOG from commits. The commits will now follow specification from: https://www.conventionalcommits.org/en/v1.0.0/#summary, after: https://github.com/juspay/services-flake/pull/121

[x] Blog post and announce.

After this release, we will record all changes in the CHANGELOG.md

https://github.com/juspay/services-flake/pull/83
will this suffice for documentation? I was thinking of adding doc for one of the services, but I am not entirely sure what more than which is already available in their respective test files can I add?

Maybe add documentation for development?

https://github.com/juspay/services-flake/pull/83 will this suffice for documentation?

I'd say yes.

Maybe add documentation for development?

Like how to hack on this repo, to add a new service or add a fix? Sure.

Blog post and announce.

Does it make sense to combine this with the nammayatri services post?

Does it make sense to combine this with the nammayatri services post?

I think yes it does

One step closer to the finishing the documentation sufficiently:
https://github.com/juspay/services-flake/pull/112

Blog post draft for announcement: https://github.com/nixos-asia/website/pull/28

changelog: https://github.com/juspay/services-flake/pull/119

Blog-post: https://nixos.asia/en/blog/replacing-docker-compose
Announcement:

Discourse: https://discourse.nixos.org/t/replacing-docker-compose-with-nix-for-development/40912
Reddit: https://www.reddit.com/r/NixOS/s/McTx5xTTjW
X (formerly twitter): https://x.com/nixos_asia/status/1765454823066108125?s=20

@shivaraj-bh I added comparison table in #135.
You can close this issue after announcing the project in Discourse.

announced in discourse: https://discourse.nixos.org/t/announcing-services-flake/41338?u=shivaraj-bh


Fix for SQLAlchemy 1.4

Hi,
I managed to make this run on SQLAlchemy 1.4 again. However, I don't know how to get the aliases to work again. Any help would be appreciated!

I think I have a fix for sqlalchemy 1.4, sorry it takes that much time ... I'm going to take a look later today.

https://copr-dist-git.fedorainfracloud.org/cgit/@copr/copr-dev/python-flask-whooshee.git/tree/sqlalchemy-1.4.patch?id=89134e4a5bbbab59a3866cf73399ff11af1c36d5

Thank you for this PR, @sh4nks - I eventually merged #62 that should not affect the behavior of aliases.


tests/formulae: add erlang to BFS whitelist

This PR adds erlang to build-from-source whitelist. This should help to catch build problems on erlang-dependant formulae earlier and help to decide if we need to add a versioned formula for the previous version (in case of major updates).

Thanks @bayandin!

:+1:


google-genai, google-vertexai: move to langchain-google

These packages have moved to https://github.com/langchain-ai/langchain-google
Left tombstone readmes incase anyone ends up at the "Source Code" link from old pypi releases. Can keep these around for a few months.

CI failing is ok - these packages will not exist anymore in this PR


feat: add link to domain in mobile dapp details page

Description
In the desktop, dApp details page, we are redirecting the domain to the dApp on click. This should be the same on mobile as well.
Describe the solution you'd like
Add a link with dApp domain as href
Additional context
Slack conv

Also put it on Discover page


Generalized Merge Tool ("+")

The curent merge function is sometimes quite useful, but not really required very often. Therefore needing one or two additional clicks wouldn't matter. This is offering the possibility to extend the merge tool to do several different kinds of merge by asking the user how to merge the selected objects. A list with all suitable types of merges fitting for the current selection needs to be shown. If required some functions can be made directly accessible by using shift, ctrl, or alt modifiers when invoking the merge tool as a shortcut.
There are many functions which could be intuitively bound to an add/merge tool dependent on the current selection (italic):

multiple objects

New Site Relation with selected objects as members. This can also be used as a base in order to setup other relations.

multiple lines

Safe Line Merge  as currently implemented, but available only if merge doesn't change the meaning of data.
Unifying Line Merge (be careful)  as currently implemented, but available only if iD can can handle all associate relations well. Tags which are unified should be shown to the user.
Enforced Line Merge (dangerous)  Tags and relations which might need subsequent clean-up by the user should be indicated by iD.
New Route Relation with selected objects as members. Available in case of suitable features only.
New Area leaving selected objects unchanged. Available in case of lines forming a ring.
New Multipolygon with selected objects as members. Available in case of lines forming at least one ring.

multiple areas (closed way and/or multipolygon)

Safe Merge to Multipolygon  as currently implemented, but available only if merge doesn't change the meaning of data.
Unifying Merge to Multipolygon (be careful)  as currently implemented, but available only if iD can can handle all associate relations well. Tags which are unified should be shown to the user.
Enforced  Merge to Multipolygon (dangerous)  resulting in a closed way area. Tags and relations which might need subsequent clean-up by the user should be indicated by iD.
New Multipolygon (needs user clean-up) with selected objects as members, leaving tags on selected features.

multiple touching areas (closed way and/or multipolygon)

Safe Area Merge  resulting in a closed way area, but available only if merge doesn't change the meaning of data.
Unifying Area Merge (be careful)  resulting in a closed way area, but available only if iD can can handle all associate relations well. Tags which are unified should be shown to the user.
Enforced Area Merge (dangerous)  resulting in a closed way area. Tags and relations which might need subsequent clean-up by the user should be indicated by iD.

line/area/multipolygon and point

Insert point as vertex  This is currently covered by dragging the point on the line/outline, but I believe it is better to make the dragging function selective on feature type (for example not connecting highway to landuse etc.).
Merge Tags and Delete Point as currently implemented.

area/multipolygon and line (or lines)

Extend Area The area is extended along the line. The line needs to touch with both end to the outline of the area. Same applies to outer or inner rings of a multipolygon if the line is touching with its endpoints.
Merge Tags and Delete Line

single multipolygon

Merge Outers resuting in closed way outers. Members are replaced by a closed way, but deleted only if untagged and unused by other relations.
This can be used in oder to merge touching closed way outers or a single segmented outer ring.
Merge Inners Same as above for inners.
Merge to Other Features Replace untagged members by other features as far as possible, but don't split the other features.

relation (selected first) and other objects

Add Members Add other objects as members to relation. Some feature dependent rules might be required for this.

In some case more preconditions may be required in order to make a merge option available.
Unavailable options might be greyed out to make them more discoverable, but of course the full list of not fitting merge options can't be shown.
The list will likely need some edits for completion.

There are many functions which could be intuitively bound to an add/merge tool dependent on the current selection (italic):

I'd disagree on the intuitive.

I have to admit that the level of intuitivity of these options is quite varying, but I belive that some of the new options are more intuitive than the current merge tool:

The expected result of merging two touching areas is likely a simple area, but not a multipolygon as currently implemented.
The expected result of merging an area with a point near to one of its edges is likely the point becoming an vertex of the are outline, but not to transfer tags and delete the point.
The expected result of merging/adding different types of areas is more likely a grouping, which is made in OSM using a site relation, but not a multipolygon with mixed up tags as currently implemented.

Merge options like merge tags and delete point, which I have included because it is already part of the current merge implementation, are less intuitive. I believe these can still be sufficiently associted with a merge tool making them rememberable.
Some of the listed merge option are intuitive for certain types of selection, which makes them discoverable for other types of selection.
In some cases the level of intuitivity is higher for less experienced OSM users. For example a new user wants to group some areas, so using the merge tool seems to be intuitive. An experienced does already knows, that a new site relation is required, which makes using merge less intuitive.
The required level of intuitive of the generalized merge tool is much lower than that of the current merge tool because the user is made aware what will happen before commiting the operation.
For the generalized merge tool an option being discoverable and rememberable seems to be sufficient.
Unfortunately, the current merge tool doesn't have that higher level of intuitivity and is producing unexpected results.

Thanks, I do appreciate the time you spent on this..
We do have a few existing issues that cover some of this:

Polygon (area-area) merge is covered by #435.  I'm planning to use Turf.js for this, but there is an upstream blocking issue in that library.
Point-area merge action is not discoverable #815.  You mention this above that it is not intuitive and I agree.  I think we can separate the merge operations and give them better icons on the radial menu.  Just having a '+' is ok, but better would be pictures that show what happens to the points/lines/areas when they merge.

There are a lot more ideas presented above, but some of them we will probably not get to anytime soon.  e.g. an operation to create site relations, or operations to do more complex things involving multipolygons are probably out of scope for iD.  You should probably switch to JOSM if you find yourself needing these features.

I have tried to make the list of operations as complete as possible. An operation, which is out of scope of iD, may still be indicating that there is another potentially expected result of merging a specific selection.
Some unsafe operations, like the enforced merge variants, may be considered to be out of scope of iD.
I'd appreciate that, bu such unsafe operations are provided, the user must be made aware of the risk.
We should not mark safe operations as risky; therefore, I have proposed the three levels safe, unifying, and enforced.
The current merge is pseudo-simple; checking, if the operation can be performed safely, is difficult.
Assigning a name etc. to a group of features seems to be a quite natural operation. Therefore, I disagree on creating site relations being out of scope of iD. Such operations should not be limited to a minority of mappers using JOSM.
We should not have data structures which are not editable well by iD users; thus, a powerful set of multipolygon operations is required, but I'm not sure which operations are essential yet.
Maybe some operations, like merging touching outer and inner rings, can be performed automatically instead of being accessible from the user interface.
I have already started using JOSM before iD has appeared, and I'm fine with it. My only interest in iD is to improve it for the less experienced or less technically skilled users, and to protect the database against destructive changes made by iD users accidentally.

Thanks for your feedback @slhh!
I agree with @bhousel and @pnorman: I don't think it would be intuitive or discoverable to put this amount of functionality behind one operation. We already have indications that the amount of functionality behind the existing merge operation is hard to discover.
We already have support for, or have issues filed, for most of the items you listed:

Merge protects tags: already implemented
Merge protects relations: #1512
Switch between line and area: #1828
Merge to multipolygon: already implemented
Union overlapping areas: #435
Selective auto-snapping: #2302
Extend area: #2272, #1711
Merge multipolygons: already implemented
Merge multipolygon member ways: already implemented
Add members: #2152, #1906
Create route relation: already implemented. Select the member you want to start with and click + in the relations section in the sidebar.

I've opened issues for the items in your list that weren't already covered:

Site relations. We should add a preset for them: #2930
Forcing operations. This should be possible for power users: #2929

I think we can separate the merge operations and give them better icons on the radial menu. Just having a '+' is ok, but better would be pictures that show what happens to the points/lines/areas when they merge.

@bhousel, I believe small icons describing the different merge operations well are impossible, and you should not waste valueable space of the radial menu for the less frequently used operations, which can be combined behind a single radial menu tool. iD is far away from beeing complete, you will likely need the space in the radial menu some day.
I suggest to keep the single "+" , show the list of available and unavailable merge operations in the inspector frame. Each item should be a large button invoking the merge operation; the button should consist of a large icon and the name of the operation.Icon and Name together can likely describe the Operation well. Mouse over the button should show more information for the operation, in addition it should show a preview of the operation in the map window if the specific merge operation is available. A help button should be next to the operation button. Due to the inspector the complete list of merge operations can be shown, or at least all operations for similar types of data. This makes even the unavailable one discoverable. The available ones needs to be arrangged somewhere on top of the list in order to avoid scrolling for the normal operations.
@jfirebaugh , I can't understand why this should not be discoverable. It's similar to an traditional drop-down menu, just improved with some additional information like the preview. The user will likely use one of the intuitive operations soon, and can discover all other operations then.

We already have indications that the amount of functionality behind the existing merge operation is hard to discover.

I agree. This is due to not showing/listing the operation somewere, but just performing it. In addition, some of the current operations nearly don't change anything visible.


fix: page meta articles and dates fix

Description
Issues
Closes
DEV-XXX:
Related
Testing
Automated tests
Manual testing
Screenshots
Additional notes

Hobbies-Helsinki branch is deployed to platta: https://harrastukset-pr512.dev.hel.ninja :rocket::rocket::rocket:

Events-Helsinki branch is deployed to platta: https://tapahtumat-pr512.dev.hel.ninja :rocket::rocket::rocket:

Sports-Helsinki branch is deployed to platta: https://liikunta-pr512.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://tapahtumat-pr512.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://liikunta-pr512.dev.hel.ninja :laughing::tada::tada::tada:

VENUE GRAPHQL PROXY is deployed to platta: https://venue-graphql-proxy-pr512.dev.hel.ninja :rocket::rocket::rocket:

EVENTS GRAPHQL PROXY is deployed to platta: https://events-graphql-proxy-pr512.dev.hel.ninja :rocket::rocket::rocket:


Detect wallet(provider) in use and change name/icon

Detect which wallet user is using (Metamask or whatever) and change terminology in the dapp.
So instead of saying "Complete transaction in Metamask" it should say, "Complete transaction in Trust Wallet".
See  here for exact location of name in web3 object: https://github.com/TrustWallet/trust-web3-provider#how-to-identify-trust-provider
cc: @vikmeup of Trust Browser

@jordajm I think that this might be covered by #83, which is blocked. What do you think?

Hmmm - I'm not sure - seems like this one is slightly different b/c it refers to changing name/icon throughout the app, but since @ambertch is working on #83, it probably makes sense for him to grab this one too..?

@jordajm ah, I see the difference. Go ahead and take this one if you want.

This SO question seems to have some good code on detecting what provider is being used: https://ethereum.stackexchange.com/questions/24266/elegant-way-to-detect-current-provider-int-web3-js

PR submitted here: https://github.com/OriginProtocol/demo-dapp/pull/178
The title of this issue references icons that need to be changed, but I wasn't able to locate any icons. I did a "find all" on the phrase "MetaMask" and replaced those hard-coded strings with dynamic provider names.
Let me know if I missed anything.
Also, this may be blocked by the pending updated way to reference web3 in demo-dapp as noted here: https://github.com/OriginProtocol/demo-dapp/issues/126

Done via #178 ✅


Feature/mtl refactor

I wanted to see what the API would look like if converted to a monad transformer stack. I think it cleans things up quite nicely. What do you think? If we decide to merge this, I'll definitely add more docs.

I meant to keep this library as simple as possible (and show how good haskell is, even when not using fancy extensions), but on the other hand, mtl is such a major design that is worth learning so I'm ok to merge this.
For the tutorial purpose, I wonder if we could expose a createWithSession version of the createSession which would return the runMatrix helper directly, so that the documentation would become:
withSession <- createSession "https://matrix.org" token
Right userId <- withSession getTokenOwner

Similarly, it might be good to expose a try helper to remove the except layer, so that api calls that are expected to fail are easy to catch, e.g. something like MatrixIO a -> MatrixIO (Either MatrixError a).
Otherwise, I also think this cleans things up nicely, thanks you!

I meant to keep this library as simple as possible (and show how good haskell is, even when not using fancy extensions), but on the other hand, mtl is such a major design that is worth learning so I'm ok to merge this.

I get where you are coming from and I don't think we should go too crazy with the library. I think its good to treat it as a fairly low level binding to the Matrix REST API.
I think Monad Transformers are standard enough that its reasonable to use them here. Its really nice to not have to thread the ClientSession through everywhere explicitly or case on Eithers explicitly when it isn't necessary.

For the tutorial purpose, I wonder if we could expose a createWithSession version of the createSession which would return the runMatrix helper directly, so that the documentation would become:

how do this look?
createWithSession ::
  -- | The matrix client-server base url, e.g. "https://matrix.org"
  T.Text ->
  -- | The user token
  MatrixToken ->
  -- | The matrix action to perform
  MatrixIO a ->
  IO (Either MatrixError a)
createWithSession baseUrl' token' action = do
  session <- createSession baseUrl' token'
  runMatrixM session action

Similarly, it might be good to expose a try helper to remove the except layer, so that api calls that are expected to fail are easy to catch, e.g. something like MatrixIO a -> MatrixIO (Either MatrixError a).

I'm not sure how this would work, or I am misunderstanding the idea. MatrixIO is ExceptT MatrixError (ReaderT ClientSession IO) a so if we unwrapped the ExceptT then rewrapped it, we wouldn't have a MatrixIO anymore.

Thank you for the update, I'll give this a try and update the tutorial. My suggestions are mostly for basic ghci usage, where previously you would do:
> sess <- createSession "https://matrix.org" token
> Right room <- joinRoom sess "#test"
> sendMessage sess room message

With the createWithSession, that would be:
> withSession <- createWithSession "https://matrix.org" token
> Right room <- withSession $ joinRoom "#test"
> withSession $ sendMessage room message

Thus I think it should return a callback, otherwise it seems like it would re-create the manager each time:
createWithSession ::
  MonadIO m =>
  -- | The matrix client-server base url, e.g. "https://matrix.org"
  T.Text ->
  -- | The user token
  MatrixToken ->
  -- | The matrix action to perform
  m (MatrixM m a -> m (Either MatrixError a))
createWithSession baseUrl' token' = do
  session <- liftIO $ createSession baseUrl' token'
  pure $ \action -> runMatrixM session action

Well, otherwise I'm not sure how the Tutorial module can be updated.
And since we are updating the library API, I think it would be better to add the http manager to the argument of createSession and createIdentitySession so that the caller can pick http-client-openssl, or http-mock if needed. Then only the simplest createWithSession would create it automatically.
If that sounds good to you, I can add the change to the PR later.

Similarly, it might be good to expose a try helper to remove the except layer, so that api calls that are expected to fail are easy to catch, e.g. something like MatrixIO a -> MatrixIO (Either MatrixError a).

I'm not sure how this would work, or I am misunderstanding the idea. MatrixIO is ExceptT MatrixError (ReaderT ClientSession IO) a so if we unwrapped the ExceptT then we wouldn't have a MatrixIO anymore?

In the event a MatrixError is expected, it seems like the ExceptT is going to shortcut the user defined action unless catchError is used. Well I guess that's ok, let's not bother with that for now.

Here is a rebase to include the loginToken. I'll now update the tutorial and propose adding the Manager to the login/createsSession arguments.

Sorry about the createWithSession suggestion, I was able to update the Tutorial without it, and with hindsight, it may not be useful. My last remaining concern is the 'login' function which I would like to change to a withLogin to force the logout call on exit. But that could be done in a separate PR.
If that is ok with you, then I think we should merge this. Thanks again for doing the refactor.

I haven't worked on this project in a while but I should have some time coming up in a few weeks. Is there anything y'all would like me to do to assist here?

I guess changing the name from MatrixM to MatrixT is reasonable, though I don't get what's the benefit of having a MonadMatrix class, e.g. I find the 'throwMatrixError' method confusing.
Thanks @solomon-b, would you have time to rebase the PR?

In the CHANGELOG, I would add a migration note, something like:

Migrate to the new API by replacing call site from Network.Matrix.Client.method session arg to runMatrixIO session (method arg)

It's also a bit annoying that the Identity API requires a different token and the present type doesn't prevent using the wrong session. I guess we could tag the Session and/or include both token...


Update monitor-hub to fix container starting

fixes issue: #19

Thanks for this! Just as an FYI, this is a half-baked idea I came up with before I realized dozzle does what I was aiming for, and does it much better. Honestly, I think this idea is still worth pursuing because I think it would be super nice to have a "management front end" for the SDR-E suite of containers that would enable starting, stopping, etc of containers.
Anyway, don't expect this thing to do much of interest, but it's cool to see people playing with it.


TAS: TopologyUngater clean up for Pod groups

What type of PR is this?
/kind cleanup

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

/assign @mimowo


Nitpicking Issue Not Affecting Performance

First of all thank you for the amazing work!
Just a bit of nitpicking here, but there seems to be two ReLU functions on the DepthAnythingv2 model, one in the self.scratch.output_conv2 definition (see here), and one in the forward call (see here). I would suggest removing one of the two for clarity!
Best regards,
Konstantinos

Thanks for the reminder. We will remove one in the next update.


Fix include issue when using libjpeg compiled outside of dlib ('INT32': redefinition; different basic types)

When compiling on windows and using libjpeg compiled outside of dlib you will get - 'INT32': redefinition; different basic types. This tracks back to the fact the jmorecfg.h in the libjpeg source defines INT32 (dlib's source for libjpeg does not included this define), later basetsd.h (from the windows kit) defines it differently. jmorecfg has a pre-processor check against BASETSD_H defined in basetsd.h, however in this case jmorecfg.h gets included before basetsd.h in save_jpeg.cpp through jpeglib.h (includes jmorecfg.h)  and image_saver.h (includes basetsd.h). Swapping the order of the includes seems to fix the issue with no apparent issues created.

Cool, thanks for the PR.


Feature: Lovelace Picture Entity Height

It would be nice if there was a way to specify the height on the picture-entity. This way, if it takes a couple of seconds to get an image from a camera, you don't get things pushed around the screen after the image loads.

height is not the right config, as it depends on the width of the column. Ratio would work.

Yeah, ratio makes more sense. If I have time, I might see if I can figure out how to do it.

This has been implemented as aspect_ratio in 0.79.


[Satisfactory] Incorrect power consumption after using somersloop.

Incorrect power consumption after using somersloop.
If OVERCLOCK.
It is not a proportional increase in power consumption based on the number of somersloop used.
https://satisfactory.wiki.gg/wiki/Production_amplifier#:~:text=without%20any%20boost.-,Power%20usage,-The%20power%20usage

That link seems to be broken
The formula used internally should not be linear. Consumption is affected by both somersloop and overclock, and using both, I come out with a factor of 13.431 just as the wiki says

If you still believe there is an issue with this calculation, please elaborate.

Thank you, I also believe that the formulas in the wiki are correct.

Thank you, I also believe that the formulas in the wiki are correct.

Does that mean this issue can be closed with no changes?

NO
The formula for wiki is correct.But the calculator's result is incorrect.
I tested it myself.
Firstly, power consumption is not related to the quantity of goods produced.
Only the maximum power consumption is calculated in the game.
Example
Diamond production
IN GAME
NO overclock NO somersloop  750MW
250% overclock NO somersloop  2518.3MW
NO overclock 4 somersloop 3000MW
250% overclock 4 somersloop 10073.3MW
The calculation result of the calculator is incorrect.
And I also FIND. IN GAME
If the machine is set to operate at 100%. But the materials are not enough to keep the machine working continuously. The power consumption is still calculated at 100%.
IN game
30 product 1x100% machine 25WM
15 product 1x100% machine 25WM
IN calculator
30 product 1x100% machine 25WM
15 product 1x100% machine 12.5WM

@happymy I'm following you on your first point. Basically to put this in terms of the way this calculator works, all power consumption in Satisfactory is classified as "drain". Drain means the energy is used whether the machine is active or not. That means a machine active 10% of the time uses the same power as a machine active 100% of the time.
It's actually not too difficult for me to just reclassify all power usage in Satisfactory as drain. This causes power to "step" each time the rounded-up whole number of machines goes up.
For your second point: The wiki does not seem to agree with what you are saying. The wiki indicates power usage during diamond production fluctuates between 250MW and 750MW. While it doesn't seem to indicate an average, I would guess the average is in the middle, 500MW. Is this incorrect?

Let me explain.
The power consumption of a machine producing products ranges from 250WM to 750WM.
The power generation capacity of the power grid is 400MW.
So when the machine produces this product midway, it will exceed the capacity of the power grid, and the power grid will overload and collapse.
Unless there is a battery in the power grid, the battery should not be depleted before the production of the product is completed.
So maximum power consumption is more important.

This is addressed in the PR I linked to this issue
I don't believe there was ever any issue with somersloop power consumption

That leaves #3. I'm following you on that. Is use of batteries uncommon to handle the power fluctuations? I think you've generally convinced me that the maximum should be what's considered.

Please calculate based on the machine's 100% working time. Another reason for doing so is to set the machine's overclocking efficiency to be less than 100%.

1 machine in 100% TIME work.
IN GAME

220MW 738.7MW 220MW 738.7MW
IN calculate

I feel very very confused.
Now we have Priority Power Switch. So the battery can provide time for you to find ways to generate more electricity.
1 and 2 please Add the option to calculate the highest energy consumption.

The wiki says the power consumption of the manufacturer is 55MW. Your in game screenshot seems to be showing 220MW, 4 times higher.
What am I missing here?

Satisfactory tools seems to agree with 55MW as well


Downgrade sigs.k8s.io/controller-runtime to v0.11.1

This reverts commit 7e59a6843fbabc99f0e9ba0c81593dafd5556439.
OpenShift 3.11 does not support the lease API and client-go v0.24
removed leader election using ConfigMap locks.
See:
https://github.com/kubernetes/client-go/commit/276ea3ed979947d7cdd4b3d708862245ddcd8883
Relates:
https://github.com/stolostron/backlog/issues/24148
Signed-off-by: mprahl<EMAIL_ADDRESS>

I updated sigs.k8s.io/controller-runtime to v0.12+ in order to resolve https://github.com/stolostron/backlog/issues/21002 ... does it look like there is no way to fix both?
If not, or if it's not worth it, I guess reverting is the better option - 21002 is just error logs on old k8s versions, it doesn't really impact the functionality.

I updated sigs.k8s.io/controller-runtime to v0.12+ in order to resolve stolostron/backlog#21002 ... does it look like there is no way to fix both?
If not, or if it's not worth it, I guess reverting is the better option - 21002 is just error logs on old k8s versions, it doesn't really impact the functionality.

I couldn't figure out a way to get this to happen nicely because v0.12+ updates so many dependencies and various replaces I tried in the go.mod made it not compile.


x/website: Version page does not return the latest stable version

When I visit https://golang.org/VERSION (or the text version https://golang.org/VERSION?m=text) it indicates that the latest version is 1.15.4 but the download page indicates that the stable version is 1.15.5.
I expect that the versions match between the version page and the download page

https://golang.org/VERSION is the version that the site is running.
For latest released version use https://golang.org/dl/?mode=json

Yes as AlexRouSg said that VERSION endpoint is just serving the VERSION file of the Go version the is being served at the moment by the website. Use https://golang.org/dl/?mode=json for a stable endpoint that list releases. Closing here.


Remove flake8-quotes

This fixes a problem where presence of this plugin caused errors on other projects, as we did
not have custom configuration for it.
Related: https://github.com/zheller/flake8-quotes/issues/105

I think this can go too:

# flake8-quotes
# https://github.com/zheller/flake8-quotes
# keep at bottom of configuration because some IDEs
# may see the single double quote in the inline-quotes
# entry as an unterminated string and not properly
# highlight the remainder of the file
inline-quotes = "

at the bottom of the flake8 file


Unused var, const or let

There's some configuration that handle it?

No, we don't have a rule for that yet - there is http://jscs.info/rule/disallowUnusedParams.

It should be in 3.0 though!

With autofixing too

Great, thanks!

If nobody is working on this I'd like to implement.
Thanks!


Fixing javadoc issues in maven release build

Fixing javadoc issues in maven release build

This PR contains the fixes for java doc issues coming in the maven release build

Issue link: required
Doc Issue: Optional, link issue from documentation repository
Applicable Labels: Spec, product, version, type (specify requested labels)

Development Checklist

[ ] Build complete solution with pull request in place.
[ ] Ran checkstyle plugin with pull request in place.
[ ] Ran Findbugs plugin with pull request in place.
[ ] Ran FindSecurityBugs plugin and verified report.
[ ] Formatted code according to WSO2 code style.
[ ] Have you verified the PR doesn't commit any keys, passwords, tokens, usernames, or other secrets?
[ ] Migration scripts written (if applicable).
[ ] Have you followed secure coding standards in WSO2 Secure Engineering Guidelines?

Testing Checklist

[ ] Written unit tests.
[ ] Verified tests in multiple database environments (if applicable).
[ ] Tested with BI enabled  (if applicable).

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


about week color

Can you change the color differently on Saturday and Sunday?

They could be changed to a different color, yes, but right now are set to use a different opacity instead.

They could be changed to a different color, yes, but right now are set to use a different opacity instead.

can I ask how change that?

can I ask how change that?

Can you be as specific as possible about this? I understand you want the color to be different vs just different opacity. But are you meaning Saturday has a different color than Monday - Friday and also Sunday has a different color? Or Saturday and Sunday are the same color but it should be different from M-F?

Can you be as specific as possible about this? I understand you want the color to be different vs just different opacity. But are you meaning Saturday has a different color than Monday - Friday and also Sunday has a different color? Or Saturday and Sunday are the same color but it should be different from M-F?

I am sorry for my poor English expression. I hope Saturday and Sunday and Weekdays are different colors. Ex) if M-F are white, Saturday is blue and Sunday is red.

I am sorry for my poor English expression. I hope Saturday and Sunday and Weekdays are different colors. Ex) if M-F are white, Saturday is blue and Sunday is red.

That's a very odd way to color a calendar but it is possible. I'm not sure why you'd want Saturday and Sunday to be a different color but I cannot into adding it in. The trouble is, it will require more options to be added. Let me see what I can come up with

That's a very odd way to color a calendar but it is possible. I'm not sure why you'd want Saturday and Sunday to be a different color but I cannot into adding it in. The trouble is, it will require more options to be added. Let me see what I can come up with

I have implemented this in the latest beta and will release it for the public release soon

I have implemented this in the latest beta and will release it for the public release soon


Textures are converted to power of two, even with Texture POT disabled

After pushing an image with pcsync pushAll -e png, it appears as resized to a power of two inside PlayCanvas. This happens even though I have disabled the "Texture POT" option:

It appears that the Texture import settings are just not taken into account when using pcsync?

This appears to be a bug in the assets API, I tried recreating the request made by pcsync and the same thing happens:
> PUT /api/assets/106451069 HTTP/2
> Host: playcanvas.com
> user-agent: insomnia/2022.6.0
> content-type: multipart/form-data; boundary=X-INSOMNIA-BOUNDARY
> authorization: Bearer *****************************
> accept: */*
> content-length: 50093


Tag release commits with version in repository for easier code browsing/inspection

Context
This issue is git / github / repository / release process related, rather than a bug in zappa code.
Expected Behavior
Most projects tag their release commits with the version so that changes/diffs/checkouts of code is made easy for users. E.g. if I want to see code changes made between versions (or between the latest released version and 'master') using the git diff command, I can use tags rather than commit hashes (a lot more user friendly).
Actual Behavior
Github shows no tags on this repository.
Possible Fix
Please tag historical release commits with version and push. Update release process to include tagging new versions.

Yeah, I should do this properly.  You can see the released versions as the ones which have version numbers in the commit messages though.

@miserlou: in case it's helpful:
for c in `git log --all --grep='^0\.' | grep commit | awk {'print $2'}`; do
    git show $c | head -n5 > /tmp/gitlogmsg; VER=$(cat /tmp/gitlogmsg | tail -n1 | awk {'print $1'})
    git tag v$VER $c
    rm /tmp/gitlogmsg
done

I just ran this and then git push --tags upstream:

👍  closing


added page customization fields to audio summary ui

What this PR does / why we need it
Adds ability to set custom fields on summary ui like logo and favicon

Hi @bcorrigeux, can you also add omitempty tag for font family? Rest LGTM :)

Hi @bcorrigeux,
We've incorporated the code from this PR and added integration tests. The updated PR can be found at this link.
I'll close this PR in favor of #106 once it's merged.
Thanks for your contribution!


Component Override Functions Arity/Signature Issue

Hi, thanks for building the library! We're trying to use the component override functionality and have passed in render methods similar to your example dev app, but for some reason render_edit/2 is being called instead of the expected render_edit/1.
When inspecting the arguments passed to the function, the first argument is always an empty map, while the second argument is the normal assigns map that's expected. Do you have any idea what could be causing that?
We're using the same pattern as:
 components: [
          new: {LiveAdmin.Components.Container, :render_new, []},
          edit: {LiveAdmin.Components.Container, :render_edit, []}
        ],

from dev.exs.

Hi @jpramassini
that looks suspiciously like a bug that I fixed in the last version update. can you check your current version and/or try updating to latest?

So, unfortunately, I'm not able to reproduce this issue using version 0.4 (f05c8a5ffd390090a16aa21d86e672d9adcad0a1365e72c51ee919ba257ce3ea). If you can share the project code, or provide a sample app that reproduces the issue I'd be happy to take a closer look.

I can't share direct project code, but I'll try to pull together a sample app and get back to you. Thanks for your help and for looking into this!

Overrides have changed in the latest release: https://elixirforum.com/t/liveadmin-phoenix-admin-ui-built-on-liveview/46421/14?u=tfwright
If you weren't able to resolve your issue, you might try updating and following the new example in the dev app--it's a lot more fleshed out in terms of what is entailed in implementing a real form override.


Card is no longer available in Hassio 2021.6

Expected behaviour
atomic-calendar-revive card should be shown in Lovelace Dashboard
Current behaviour
Card does not exist and can not be found in hacs.
Screenshot

Possible solutions

Context and reason
I would love to see the card again soon in the new hassio. <3
Checklist

[x] I searched the current issues and the feature is not known.
[x] I am mostly sure that the feature will bring more good than bad.

I think the problem is me 🙈


Support Carthage

This commit adds a Cartfile that specifies the dependency to ConsistencyManager-iOS. The pull request is depending on the ConsistencyManager-iOS-PR-32 which provides better Carthage support.
For now, I've specified any version of the consistency manager compatible with 2.0.0 but you should better specify the next (minor?) release version that contains the PR listed above (s.th. like 2.0.1 but that's up to you and how you like to version).
If you want, I could also add installation instructions for Carthage to the Readme.
Thanks!

Current coverage is 89.89% (diff: 100%)

Merging #30 into master will not change coverage

@@             master        #30   diff @@
==========================================
  Files            43         43
  Lines          4524       4524
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           4067       4067
  Misses          457        457
  Partials          0          0

Powered by Codecov. Last update 5efb4d4...32443d5

To be clear, you want me to re-release the Consistency Manager with a new version right? 2.0.1 sounds good to me and can do that easily. Just wanted to make sure that's what you needed?

And yeah, adding Carthage instructions to the README would be great.

Hi, thanks for the quick response! I've updated the PR with installation instructions for Carthage and I removed the explicit version dependency from the Cartfile. Carthage will take the latest release anyways so it's just important that you make a new release of both RocketData and ConsistencyManager at roughly the same time to get full Carthage support. Patch versions are perfect for that as there are no code changes. Thank you!

I'm working on OSX support for the Consistency Manager and expect to get that done today.
I'll probably release a new version with both these updates (hopefully also today).

Changes look good though, merging.

Oh...actually...I'm going to wait until it actually works to merge. Otherwise, people may be confused when they see the README.

Released, so merging this in.

Can you verify that it works?

Hi @plivesey, thank you for merging. Unfortunately you also need to tag a new release for RocketData itself, b/c Carthage takes the latest release here as well:
$ cat Cartfile
github "linkedin/RocketData"
$ carthage update RocketData
*** Fetching RocketData
*** Checking out RocketData at "1.1.0"
*** xcodebuild output can be found in /var/folders/qj/38k1t5d52rb_h3y8bbhgc4lc0000gn/T/carthage-xcodebuild.JD4lKy.log
*** Building scheme "RocketData" in RocketData.xcworkspace

Until you make a new release, we can force Carthage to pull the latest commit from master.
$ cat Cartfile
github "linkedin/RocketData" "master"
$ carthage update RocketData
*** Fetching RocketData
*** Fetching ConsistencyManager-iOS
*** Checking out ConsistencyManager-iOS at "2.0.1"
*** Checking out RocketData at "90df908fe9c4ca8fc5fdccd77123a0937ce3f581"
*** xcodebuild output can be found in /var/folders/qj/38k1t5d52rb_h3y8bbhgc4lc0000gn/T/carthage-xcodebuild.mLln9u.log
*** Building scheme "ConsistencyManager" in ConsistencyManager.xcodeproj
*** Building scheme "RocketData" in RocketData.xcworkspace

Notice the "master" in the Cartfile. Now it correctly retrieves the dependency for RocketData and builds the ConsistencyManager.
Would be great if you could create a new RocketData release at some point in the future. Thanks!

Yes. Will do. I'll create an issue so I don't forget.
Hopefully today/tomorrow.

@floriankrueger I released a new version (1.1.1)


No Images Found - Instagram

Ripme version: 1.7.94
Java version: 1.8.0_281
Operating system: Win 10 pro (20H2 update)
Exact URL you were trying to rip when the problem occurred: https://www.instagram.com/acatcie/
Please include any additional information about how to reproduce the problem: n/a

Expected Behavior
Successfully downloading the entire instagram account.
Actual Behavior
It was working fine yesterday but I tried to rip from the link above and it detected 26 pictures then consistently gives the described error. Tried with another instagram and it also failed without ripping anything either.
I tried the latest binary version (as of 23h ago of this report) just in case because it was suggested in other more recent instagram problems but there was no difference in behavior. Maybe instagram updated something?

I've had the same issue a few  days ago but now when I go to try to rip again, it just gives a red "null" and says rip complete. no folder made, no photos detected,  nothing downloaded.

@iProdigy you fixed that once in #1722 - you v got any ideas?

tracked here: https://github.com/ripmeapp2/ripme/issues/32 now.


ci: improve logging for tools build script

related to https://github.com/asyncapi/website/pull/940#issuecomment-1326630220

/rtm


netcoreapp3.1 Support

Is there any ETA on 3.1 support for FluentValidation?
Love the package and would like to upgrade our apps to 3.1 ASAP.

It's already supported.

Wow, that was fast!
Ah ok I was thrown by receving an error and then looked into NuGet dependencies and it says says netcoreapp3.0 as a dependency?

Am I missing something obvious with this error?
Unhandled exception. System.TypeLoadException: Method 'GetValidationVisitor' in type 'FluentValidation.AspNetCore.FluentValidationObjectModelValidator' from assembly 'FluentValidation.AspNetCore, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7de548da2fbae0f0' does not have an implementation.
   at FluentValidation.AspNetCore.FluentValidationMvcExtensions.RegisterServices(IServiceCollection services, FluentValidationMvcConfiguration config)
   at FluentValidation.AspNetCore.FluentValidationMvcExtensions.AddFluentValidation(IMvcBuilder mvcBuilder, Action`1 configurationExpression) in /home/jskinner/code/FluentValidation/src/FluentValidation.AspNetCore/FluentValidationMvcExtensions.cs:line 72

Hm, I was under the impression that libraries targetting netcoreapp3.0 ran under 3.1 (my tests under 3.1 all pass). I'll re-publish with an explicit netcoreapp3.1 target to be on the safe side.

Published 8.6.1 - feel free to give it a try. Let me know if you still see issues. Might take a few mins for it to appear on nuget.

Thanks Jeremy, I'll give it a go an update when it appears on NuGet for me.

I can confirm that the new package works, making no changes to our code other than updating the package reference.
Thanks for the quick response @JeremySkinner much appreciated!


UI Component Reversed

System information:

MacOS Big Sur
DBeaver version 7.3

Connection specification:
Not related
Describe the problem you're observing:
Text Editors Preferences displayed in reverse as the screenshoot below.

Steps to reproduce, if exist:
Include any warning/errors/backtraces from the logs

thanks for the report

Fixed in the scope of #10535.

@akilovich Confirm it works nicely on 7.3.1 EA


Add support for NAVI22 and NAVI23 i.e. gfx1031 and gfx1032

NAVI22 and NAVI23 i.e. gfx1031 and gfx1032 is needed for HIP programming/software development using rocThrust on Notebooks as ALL AMD mobile GPUs are NAVI22 or NAVI23 i.e. gfx1031 and gfx1032. rocPRIM is a rocThrust dependency.
The target operating system is RHEL9 = AlmaLinux 9, RockyLinux...
My approach is to develop / adapt / test engineering and scientific software on my Notebook before investing in dedicated HPC hardware.

Hello @klausbu.  Navi22 and Navi23 are not currently supported in the ROCm release.  However, you can compile from source for these targets by settting AMDGPU_TARGETS=gfx1031,gfx1032.


Option to open the side-bar over existing content, rather than side-by-side

I'd like to see an option to open the side-bar over existing content, rather than side-by-side.
Here is my specific use case:
I have an ultrawide monitor, and I am working in zen mode, With such a wide monitor I do not want the editor window to span the whole width; that would be too wide. So I set the zen-mode editor width to span about 1/3 of the monitor, centered. This is great. However, when you open the side-bar, the editor window gets pushed to the right to accomodate for the side-bar, even though there is an ample amount of empty space where the side-bar opens. What I would like is for the editor window to say in place, to not distract my zen experience:p Hence my suggestion for an option to open the side-bar over the content.

This is a duplicate of https://github.com/microsoft/vscode/issues/60900.

This is a duplicate of https://github.com/microsoft/vscode/issues/60900.


Add --force option to fisher update to force an update if git pull --rebase origin master fails

[ ] Add --force option to fisher update to force an update if git pull --rebase origin master fails. #99
[x] Push comitted changes on master, if any, to the stash, pull, and apply. #103

Closing in favor of #122.


Add frequencies dictionary to Study class to specify which iterators are required for each frequency

This has been implemented


Adds support for less verbose output

Give the option to have a less verbose output. This is feature is good when I'm testing the code and don't have any interest in date, time and logger name information. I still prefer to use Logging then Base functions because of the others features, like log level.
I just add moreinfo tag(default is true).
Using:
julia> using Logging

julia> @Logging.configure(level=Logging.INFO)
Logger(root,INFO,Base.TTY(open, 0 bytes waiting),true,root)

julia> @info("This should appear with date, time and logger name")
23-Dez 15:44:49:INFO:root:This should appear with date, time and logger name

julia> @Logging.configure(level=Logging.INFO, moreinfo=false)
Logger(root,INFO,Base.TTY(open, 0 bytes waiting),false,root)

julia> @info("This should appear without date, time and logger name")
INFO: This should appear without date, time and logger name

Output:
INFO: This should appear without date, time and logger name
23-Dez 15:24:21:INFO:root:This should appear with date, time and logger name

@Thuener, sorry so slow in responding to this.
I like the idea.  My only request is that you rename moreinfo to verbose.


Enable tests with active directory client id and secret for Azure Dat…

…a Explorer scaler.
Signed-off-by: Vighnesh Shenoy<EMAIL_ADDRESS>
Provide a description of what has been changed
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[x] Tests have been added
[x] A PR is opened to update our Helm chart (repo) (if applicable, ie. when deployment manifests are modified)
[x] A PR is opened to update the documentation on (repo) (if applicable)
[x] Changelog has been updated and is aligned with our changelog requirements

Relates to #2841

/run-e2e azure-data-explorer*

Was the Kusto cluster that the env variable points to, deleted? @zroubalik

@v-shenoy I am sorry, but I don't have an access to Azure resources. @tomkerkhove might help with that

What cluster would that be @v-shenoy?

Could you just run this again, once?

/run-e2e azure-data-explorer*

It works! 🥳 🥳 🥳
Could you rebase your branch to solve the conficlts please? 🙏

@JorTurFer Rebased. Could you just run it once more, just to ensure I didn't mess something up?

/run-e2e azure-data-explorer*

/run-e2e azure-data-explorer*


Could not find package nadvice in recipe repositories

I was trying to install package excorporate, and got an error when building its dependencies:
Debugger entered--Lisp error: (error "Could not find package nadvice in recipe repositories: (org-elpa melpa gnu-elpa-mirror emacsmirror-mirror)")
  signal(error ("Could not find package nadvice in recipe repositories: (org-elpa melpa gnu-elpa-mirror emacsmirror-mirror)"))
  error("Could not find package %S in recipe repositories: %S" nadvice (org-elpa melpa gnu-elpa-mirror emacsmirror-mirror))
  straight--convert-recipe(nadvice "Building excorporate")
  straight-use-package(nadvice nil nil "Building excorporate")
  straight--build-package((:type git :host github :repo "emacs-straight/excorporate" :files ("*" (:exclude ".git")) :package "excorporate" :local-repo "excorporate") nil)
  #f(compiled-function () #<bytecode 0x40ff69a5>)()
  straight--transaction-exec(use-package-\(:type\ git\ :host\ github\ :repo\ \"emacs-straight/excorporate\"\ :files\ \(\"*\"\ \(:exclude\ \"\.git\"\)\)\ :package\ \"excorporate\"\ :local-repo\ \"excorporate\"\)-nil-nil :now #f(compiled-function () #<bytecode 0x40ff69a5>))
  straight-use-package(excorporate)
  eval-buffer(#<buffer  *load*-327247> nil "/Users/user/.emacs.d/site-specific.el" nil t)  ; Reading at buffer position 1147
  load-with-code-conversion("/Users/user/.emacs.d/site-specific.el" "/Users/user/.emacs.d/site-specific.el" nil nil)
  load("/Users/user/.emacs.d/site-specific.el")
  eval-buffer(#<buffer  *load*> nil "/Users/user/.emacs.d/init.el" nil t)  ; Reading at buffer position 541
  load-with-code-conversion("/Users/user/.emacs.d/init.el" "/Users/user/.emacs.d/init.el" t t)
  load("/Users/user/.emacs.d/init" t t)
  #f(compiled-function () #<bytecode 0x400d1a6d>)()
  command-line()
  normal-top-level()

I noticed that nadvice is in the ELPA repository, so it's weird that it says "could not find package". I also tried to set straight-recipes-gnu-elpa-use-mirror to nil (before initializing straight) and restart Emacs, but got the same error.
Is there any extra setup I need to add in order to install excorporate and nadvice?

Ideally, that should work out of the box, and it should not depend on the values of any user options. Can you update to the latest version of develop and then:

see if it somehow works then
if not, see whether (straight--package-built-in-p 'nadvice) returns non-nil as it should
if not, let me know your Emacs version.

Nope. I updated straight to develop by setting straight-repository-branch and (straight-pull-package 'straight) (HEAD now points to develop), and executed (use-package excorporate) again, but the same error happens. (straight-use-package 'excorporate) is the same.
(straight--package-built-in-p 'nadvice) returns nil. I tried to restart Emacs, it didn't help.
Emacs version is GNU Emacs 26.3 (build 1, x86_64-apple-darwin18.2.0, NS appkit-1671.20 Version 10.14.3 (Build 18D109)) of 2019-09-02.

I just tried to move my .emacs.d to a backup folder, and start cleanly with only straight.el bootstrap code, and the same error happens with M-x straight-use-package -> excorporate. Looks like it's not my config interfering with straight.
The bootstrap code I used:
(defvar bootstrap-version)
(let ((bootstrap-file
       (expand-file-name "straight/repos/straight.el/bootstrap.el" user-emacs-directory))
      (bootstrap-version 5))
  (unless (file-exists-p bootstrap-file)
    (with-current-buffer
        (url-retrieve-synchronously
         "https://raw.githubusercontent.com/raxod502/straight.el/develop/install.el"
         'silent 'inhibit-cookies)
      (goto-char (point-max))
      (eval-print-last-sexp)))
  (load bootstrap-file nil 'nomessage))

Running into the same issue. I'm on Arch Linux 5.7.9-arch1-1, using Emacs 26.3.
Got the following stack trace after running the bellow with emacs -Q:
(defvar bootstrap-version)
(let ((bootstrap-file
       (expand-file-name "straight/repos/straight.el/bootstrap.el" user-emacs-directory))
      (bootstrap-version 5))
  (unless (file-exists-p bootstrap-file)
    (with-current-buffer
        (url-retrieve-synchronously
         "https://raw.githubusercontent.com/raxod502/straight.el/develop/install.el"
         'silent 'inhibit-cookies)
      (goto-char (point-max))
      (eval-print-last-sexp)))
  (load bootstrap-file nil 'nomessage))

(straight-use-package 'delight)

Debugger entered--Lisp error: (error "Could not find package nadvice in recipe repositories: (org-elpa melpa gnu-elpa-mirror emacsmirror-mirror)")
  signal(error ("Could not find package nadvice in recipe repositories: (org-elpa melpa gnu-elpa-mirror emacsmirror-mirror)"))
  error("Could not find package %S in recipe repositories: %S" nadvice (org-elpa melpa gnu-elpa-mirror emacsmirror-mirror))
  straight--convert-recipe(nadvice "Building delight")
  straight-use-package(nadvice nil nil "Building delight")
  straight--build-package((:type git :host github :repo "emacs-straight/delight" :files ("*" (:exclude ".git")) :package "delight" :local-repo "delight") nil)
  #f(compiled-function () #<bytecode 0x4b5ff1>)()
  straight--transaction-exec(use-package-\(:type\ git\ :host\ github\ :repo\ \"emacs-straight/delight\"\ :files\ \(\"*\"\ \(:exclude\ \"\.git\"\)\)\ :package\ \"delight\"\ :local-repo\ \"delight\"\)-nil-nil :now #f(compiled-function () #<bytecode 0x4b5ff1>))
  straight-use-package(delight)
  eval((straight-use-package (quote delight)) nil)
  elisp--eval-last-sexp(nil)
  eval-last-sexp(nil)
  funcall-interactively(eval-last-sexp nil)
  call-interactively(eval-last-sexp nil nil)
  command-execute(eval-last-sexp)

Looks like nadvice isn't being added to `package--builtins.
This is a hack, but adding this after the boostrap code got things working for me:
(straight--package-built-in-p 'nadvice)
(puthash 'nadvice t straight--cached-built-in-packages)

Please let me know if the linked commit fixes your problem.

This is the result of a bug in Emacs 26. I installed a workaround.

Yeah, it's fixed now. Thanks!


A11y_.NETOnlineREPL_Output_Screenreader: NVDA is not announcing the name of "Feedback" link in a descriptive way.

“Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.”
GitHubTags: #A11yMAS;#A11yTCS;#A11ySev2;#BM_.NET Online REPL_Web_Feb2021;#.NET Online REPL;#Win10;#FTP;#DesktopWeb;#WCAG2.4.4;#Screenreader;#Narrator;#NVDA;#Chrome;#ChromiumEdge;
Environment Details:
Application Name: .NET Online REPL
Operating System: Windows10
Browser Version : Google Chrome Version 87.0.4280.141 (Official Build) (64-bit)
Chromium Edge Version 89.0.774.14 (Official build) dev (64-bit)
Screenreader: NVDA 2020.3,Narrator
Repro Steps:

Open .NET Editor coding screen .
Start NVDA and tab to Feedback link .
Check whether NVDA announces the name of "Feedback" link in a descriptive way

Actual:
NVDA is not announcing the name of "Feedback" link in a descriptive way while navigating using tab key.
Same issue is also observed with Chromium Edge +Narrator.
Expected:
NVDA should announce the name of "Feedback" link in a descriptive way while navigating using tab key.
NVDA should also announce feedback for what purpose.
User Impact:
Screen reader users will not be able to understand the correct purpose of the link.

@LadyNaggaga  please can you please close this bug as the name lets users know the purpose

It is expected behavior and bug can be closed.

githubtags: #Closed;


2.0 Using Tuples is added to C# even when there are no tuples, causing compiler issue

Describe the bug
"Using Tuples" is added to the C# even when there are no tuples. The C# will not compile as a result.
I found this when starting the new empty project for DevDays and had to add a fake Tuples CQL file to work around.
To Reproduce

In the Measures.Authoring project, remove all the CQL files except for FHIRHelpers and ParametersExample.
Clean and build the project
Notice Packager CLI compile exceptions in the build.log

Expected behavior
If current files does not contain Tuples then the "using Tuples" should not be added. It appears to be added generically at line 71, BuildUsings within CodeGeneration.Net file CSharpLibrarySetToStreamsWriter.CS
The using section could be more accurate in general. When you highlight over certain usings the C# analyzer shows a few that are unnecessary. For example, FHIR Helpers does not need Hl7.Cql.ValueSets
For now maybe handle tuples only and consider a refactor of the logic in a future ticket to maybe do all the using more intelligently.
Version used:
2.0 Dev branch

https://github.com/FirelyTeam/firely-cql-sdk/pull/366


In ios download not working

Hi
I followed the guide and developed this portion of code in my app:
`const calEvent = {
title: appointment.medicalService.description,
location: appointment.medicalService.supplierUnit,
description: appointment.patientPreparation,
start: appointment.date,
end: addHours(appointment.date, 1)
};
const icalendar = new ICalendar(calEvent);
icalendar.download();`
But the download method does anything in ios. Any idea?
Thanks

Did you ever find a way around this?

Unfortunately no 😞Gian Marco Gallo (da iPhone)Il giorno 15 ago 2022, alle ore 19:58, Ross Coundon @.***> ha scritto:
Did you ever find a way around this?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

I think i am facing the same problem as you.
In Xcode, the error thrown is NSOSStatusErrorDomain:
2022-09-13 10:58:32.673584+0200 App[24866:8027849] [default] Failed to open URL begin:VCALENDARVERSION:2.0 [... ICS CONTENT ...]: Error Domain=NSOSStatusErrorDomain Code=-10814 "(null)" UserInfo={_LSLine=225, _LSFunction=-[_LSDOpenClient openURL:options:completionHandler:]}
Is that the same for you ?
Any idea how to solve this ?
Thanks

At the end we decided to change the library and to use another one

Of course. Here you are:
"cordova-plugin-calendar": "5.1.6",
"@ionic-native/calendar": "^5.36.0"

@gigallo I noticed that you're using Cordova, which uses the Safari webview. File downloading has been notoriously buggy for iOS which is why I put in a hack specifically for that platform.
However it seems that FileSaver.js already accounts for iOS -- I've removed that functionality in the latest version, and hopefully that should fix the issue.
A couple of things to note:

For iOS specifically, the download will only work if a direct user action (e.g., onclick) directly triggers it. That means that any delay in between (even an instantaneous setTimeout()) will cause it to fail. As per FileSaver.js docs:

saveAs() must be run within a user interaction event such as onTouchDown or onClick; setTimeout will prevent saveAs from triggering.

In the next major release of Datebook, download() will be removed. More info here.


Provide a stats page graph of the "characterization data"

graph
http://pombase2.aska.gen.nz/status/gene-characterisation
like

we will feature this periodically on the front page (and is used for multiple other purposes)

future very low priority


Addition of code implementing upper bound and lower bound using binary search

Related Issue

#474

Additional Info

This approach uses binary search and gives the output in O(logN) time.

Checklist

[x] ✅ My code follows the code style of this project.
[ ] 📝 My change requires a change to the README.
[x] 🎀 I have updated the README accordingly.
[x] 👀 I have read the CONTRIBUTING document.
[x] ✨ I have added tests to cover my changes.
[x] 🚩 All new and existing tests passed.
[x] 🌟 ed the repo

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Output Screenshots

Screenshot #1
Screenshot #2

You need to follow particular guidelines for your README , please follow
these guidelines click here
also you can see how your README should look like see here
for more please go through these here

Make sure your Pull Request is descriptive and complete the checklist .
I would recommend you to go through this tutorial video of the project here ,thanks.

The code looks neat , the status is good from my side


Allow passing command line options to ssh when in :rsync mode

I need a way to pass some settings to the ssh command, but in this line https://github.com/middleman-contrib/middleman-deploy/blob/master/lib/middleman-deploy/methods/rsync.rb#L24 nothing is being passed to it.
Can we develop a way to pass flags to the ssh command without needing to pass complete -e 'ssh' rsync flag?

I sent you a PR (#94) to allow passing ssh options in :rsync.


What about moving constants to a separate file?

Hello,
nice effort.
However, I think it would be much cleaner if we kept constants like phone, tablets, etc. in an isolated file, like the way it's done for math.js or international-anniversary.
How does this sound to you?
Cheers,
Thodoris

Generally a good idea. But in this case, the js file is generated based on a template without the constants. So from a maintenance point of view, I think it's already ok if you look at the "real" source files.
And those guys who want to load the code into the browser are glad to have only a single request.


Integration Test - Release - 2022/04/24 03:18:11

Auto create for integration test. IssueCommandFlow

#pri1

Just type '#pri3'

#pri2

#pri3

#assign PRMergerTest1

#assign NonAllowedUser

The following users don't have valid GitHub IDs, or are not collaborators on this repo: nonalloweduser

#reassign PRMergerTest2

#reassign NonAllowedUser

The following users don't have valid GitHub IDs, or are not collaborators on this repo: nonalloweduser

GitHub user PRMergerTest1 has been removed from the MicrosoftDocs organization, so they were automatically removed as an assignee.

#please-close

#unassign PRMergerTest1

#unassign NonAllowedUser


Add an example for wpcli:run command

Just added an example for wpcli:run command.
It is not easy to find the correct format if we are not friendly with Capistrano environment.
Thanks.

ping @lavmeiker

Good observation. I'm merging this.
Thanks.


.Net Core / C# bindings

I am looking for a cross-platform GPIO library that I can use as a basis for a C# applications. This seems to be as close as it gets (or am i wrong?).  Now that .Net is open source we will soon have a release of C# run-time for Arm Linux.
I want to see if there is any appetite to create bindings for C#, and if anyone would be willing to help.

mraa is not cross-platform in a typical sense - it only supports Linux (various distros and kernel versions though), as it uses sysfs extensively, so if you have Windows for IOT in mind, that won't work, at least not without major development effort. There's mock platform, that compiles under MSYS in Windows, but that's a self-contained simulator, not an actual use of Windows system interfaces.
C# bindings for Linux should be doable though and would be quite cool to have, maybe even with the same SWIG used for other bindings.

As @alext-mkrs mentions, sounds like you're using cross platform to mean cross board/arch rather than cross OS? If that's correct then mraa is a good choice (unbiased opinion obv ;-)).
How do you feel about swig generated C# bindings? If that would satisfy your need i'm happy to help. If it's manual bindings I'm also happy to include that but probably won't be able to provide all that much help since I know very little about C#...

yeah, I meant "cross-board". I haven't tried SWIG yet 🚀
I also have not previously used MRAA, is there anything unusual I should be aware of? How's the performance?

Maybe this is interesting also especially for the pur.
http://www.ofitselfso.com/BBBCSIO/BBBCSIO.php

Hard to benchmark objectively. Generally I would say perf is pretty good.
Our nodejs bindings are not well liked because they are not asynchronous
but if you look at the python ones performance is really not far off C and
largest overhead is the binding size itself. Most of this is because swig
generated code has alot of redundant checks.
Try it out! It's really the only way. I think generating swig c# bindings
for at least the basic stuff should be relatively easy. We can help you
with the small niggles like getting isrs to work.
On 3 Jul 2017 15:06, "Vladimir Akopyan"<EMAIL_ADDRESS>wrote:

yeah, I meant "cross-board". I haven't tried SWIG yet 🚀
I also have not previously used MRAA, is there anything unusual I should
be aware of? How's the performance?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/intel-iot-devkit/mraa/issues/778#issuecomment-312654707,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAfZZtkPldgkAs-M8Obj6RSYxsWbi6D-ks5sKPVfgaJpZM4OLTnG
.

...and I happen to be working on a different project  which is a cross-platform C# app, so while I'm far off from being an expert, I know a thing or two and would be glad to help as well, having firsthand mraa knowledge.

Awesome. Thx @alext-mkrs.

@VladimirAkopyan Have you made any progress. I'd like to do the same with swig.

https://www.nuget.org/packages/SystemExt.Devices.Mraa/
Actualy it requires have libmraa installed, but i'm working on have an aditional NuGet called for example (SystemExt.Devices.Mraa.runtime.linux-arm) that will not require to have libmraa it will use a local compiled version of libmraa

I'm open to transfer the ownership of the bindings to the MRAA project if they will promise still giving support to .NET

The lib by itself just make calls to libmraa.so using [DllImport].
So in theory, if in a WSL environment you can use libmraa, indirectly you can execute a .NET Core app inside the WSL using this bindings and should work pretty well.
I think that the biggest problem for you will be setup libmraa inside WSL. Once done, it should be pretty easy access to the info using my package


PMID:37191320

abolished reticulophagy during nitrogen starvation
abolished nucleophagy during nitrogen starvation
decreased nucleophagy during nitrogen starvation

changed parent of "abnormal macroautophagy" from abnormal metabolic process" to "abnormal biological process"

[ ] I dont think we need the vegetative term
[ ] abnormal reticulaphagy during vege growth

These terms are not live but they may have been added?

add terms to session

need abnormal reticulophagy during nitrogen starvation


Bump glow to 0.14

Before making this PR, I did take notice of a similar PR, https://github.com/emilk/egui/pull/4833, but as it appears to be abandoned, I decided to make this PR.
Missing
One of the checks doesn't pass as wgpu still uses glow 0.13.1
cargo deny --all-features --log-level error --target aarch64-apple-darwin check

[x] I have followed the instructions in the PR template
[ ] wgpu is using glow 0.14

glow is already on the main branch of wgpu so we just need to wait a bit
https://github.com/gfx-rs/wgpu/blob/7c917abf525b5c32c0b7345ec31788e316afa0cc/Cargo.toml#L162

As mentioned here https://github.com/gfx-rs/wgpu/issues/6240#issuecomment-2337846087, this shouldn't be blocked on wgpu update. Just add a cargo deny exception here for the time being - the usecase of having both wgpu and glow backends of eframe enabled doesn't really exist, no?

@Wumpf you are right gonna add this asap


Rule idea: named argument syntax

Context
Twig will soon support "colons instead of equals for named arguments"
Before
{{ include('foo.html.twig', ignore_missing=true) }}

After
{{ include('foo.html.twig', ignore_missing=true) }}

{# or #}
{{ include('foo.html.twig', ignore_missing: true) }}

New rule ?
A rule could probably be added to standardize, what should be the default?
Another rule could maybe simply forbid to different characters in the same expression ?
Other ones ?
I guess some other rules may require update (i'm thinking about the rule include filter -> function) ?

The first rule which need to be fixed is the HashQuoteRule or it will transform
{{ foo(bar: true) }}

into
{{ foo('bar': true) }}

I think the best is to change the Tokenizer to tokenize both = and : into another tokenType like NAMED_ARGUMENT_OPERATOR.
Then:

HashQuoteRule will be unchanged
A NamedArgumentSpacingRule could be introduced to add/remove space around this operator.
A NamedArgumentRule could be introduced to replace all of them by = or : depending of the preferred value.

what should be the default?

Not sure we can use = by default since it's only supported on last twig version...

Not sure we can use = by default since it's only supported on last twig version...

You mean : right ?

Not sure we can use = by default since it's only supported on last twig version...

You mean : right ?

yes, edited

Or we could promote the new : syntax but add an early return if it's not supported by Twig (based on the installed version ?)

Let's wait to see how this goes. Personnaly i'm all in to switch to the new syntax, but there may be drawbacks i'm not seeing yet :)

I think I implemented all you wanted in https://github.com/VincentLanglet/Twig-CS-Fixer/pull/302.
I'll just wait for the twig 3.12 release.

all you wanted

But i was just conversing, i did not "want" anything 😅
But that's super great! And i think this will be very useful for a lot of people when 3.12 is released :)
👏

Well, we have our answer

and

So the "characters" are named "separators" in Twig documentation

Now it's added in the standard, I have to add it to my Twig Standard too.
Not sure about the best strategy:

Adding the rule without condition, and the rule does nothing for old version of twig
Adding the rule conditionnaly and the rule throw an error when used with old version of twig

So the "characters" are named "separators" in Twig documentation

I'll update my naming.

Adding the rule without condition, and the rule does nothing for old version of twig

Oh you are the most qualified there ;)
If you want an uneducated & opiniated advice.... i'd do this

add the rule without condition for Twig 4.0 ..
disable the rule per default for 3.12 <-> 4.0 and let users chose
do nothing for Twig < 3.12  (of course ^^)


[Feature] Supply the way to hide extra legend or series.

What problem does this feature solve?
DEMO LINK

I want to hide the blank series on X axis. How could i get this?
What does the proposed API look like?
No goodidea.

You could use axis type 'value' and space them evenly.
Example

see also #19250 for possible duplicate

You could use axis type 'value' and space them evenly. Example
The legend can be customized via formatter function

yeah, it looks well. thanks.

see also #19250 for possible duplicate

i see it. Thanks.


samples: update azure-vote-front sample

Update sample to point to valid repo location. mcr.microsoft.com one
seems to no longer be available. So built a new one from
https://github.com/Azure-Samples/azure-voting-app-redis and pushed
to marinerconfpodsdev.azurecr.io
test run: https://dev.azure.com/mariner-org/mariner/_build/results?buildId=650312&view=results

Not ready for merge yet. Sample doesn't achieve running state as expected https://dev.azure.com/mariner-org/mariner/_build/results?buildId=650312&view=ms.vss-test-web.build-test-results-tab&runId=258043&resultId=100168&paneView=debug

Closing in favor of https://github.com/microsoft/kata-containers/pull/237 and due to image rebuilt not working as expected


Delete Data word from title "Meet Our Data Ready Set Team" located on About section.

We noticed there is a word "Data" on the team Ready Set title. It must be deleted and says "Meet Out Ready Set Team" instead.

Solution:
Issue has been solved deleting the word "data" from the code, located on:
src/components/Team/index.tsx
I changed the title from "Meet Our Data Ready Set Team" to "Meet Our Ready Set Team" as you can see it on the next screenshot:


Update README.md

Fix capitalization typos (seems unintended)
Add links to GitGuardian account creation and HasMySecretLeaked
Make the References section prettier

Looks good.


Move functions into main exported function to remove "j" parameters

as discussed on Slack yesterday

Derp.  I broke this by merging another PR, sorry 😿 .  Can you rebase?

@rwjblue done


Support for LeanFT Run Results

Hi,
I'm just wondering if anyone is currently working on extending this plugin to support uploading of LeanFT run results to ALM?

Hi Steven,
Can you please clarify how do you expect to work with LeanFT, Jenkins and ALM:

Do you want to run LeanFT tests from jenkins and just upload the results to ALM?
Do you want to run from Jenkins the LeanFT tests that you have modeled in ALM, so Jenkins just triggers the run from ALM?

Regards,
Dror

Hi Dror,
Here’s the current process:

I have my LeanFT tests checked in together with my code as part of my Maven project in Git.
I’ve configured LeanFT to run during the integration-test phase of the Maven build process.
Jenkins checks out the code (with the LeanFT tests) and then triggers the Maven build. Maven will execute the LeanFT tests
After LeanFT finishes execution, LeanFT creates a RunResults folder that stores the LeanFT results

I’d like to upload the contents of the RunResults as part of the Test Set of the LeanFT test.
Currently I am uploading the JUnit test results (of the LeanFT test execution) into ALM using the ALM plugin but it would be nice to have LeanFT test results uploaded into ALM as well.
Regards,
steven

On 19 May 2016, at 9:10 PM, saaroni<EMAIL_ADDRESS>wrote:
Hi Steven,
Can you please clarify how do you expect to work with LeanFT, Jenkins and ALM:
Do you want to run LeanFT tests from jenkins and just upload the results to ALM?
Do you want to run from Jenkins the LeanFT tests that you have modeled in ALM, so Jenkins just triggers the run from ALM?
Regards,
Dror
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub https://github.com/hpsa/hp-application-automation-tools-plugin/issues/88#issuecomment-220294540

Hi Steven,
Currently we don't support uploading files to the ALM repository,
But as a solution you can store the Run Results in Jenkins, and the ALM user can click the "view run results in jenkins" and get to the job that will include LeanFT run results.
Hope this helps,
Dror

Hi Dror,
Thanks.  Currently I am already publishing the RunResults in Jenkins using the HTML Publisher plugin and using the solution you highlighted.
I am just wondering if it makes sense to implement support for LeanFT results with this plugin.
Thanks,
steven