Escape sequence "'" is invalid in github_commit query

I tried the first example at https://hub.steampipe.io/plugins/turbot/github/tables/github_issue.
select
  sha,
  author_login,
  author_date,
  message
from
  github_commit
where
  repository_full_name = 'turbot/steampipe'
order by
  author_date desc;

Result in 0.8.2:
ERROR:  invalid input syntax for type json
DETAIL:  Escape sequence "'" is invalid.
CONTEXT:  JSON data, line 1: ...pe-postgre-fdw to v0.2.2 - fix JSON data with ''...
Does this repro for others?
This works.
select
  *
from
  github_commit
where
  repository_full_name = 'turbot/steampipe'
limit 1;

I'm seeing the same error, but it's limited to that specific repository. The query works well for other repos. I believe there must be a specific issue in the results which is triggering the escape parsing error.

Turns out there is no such repo. This works:
select
  message
from
  github_commit
where
  repository_full_name = 'turbotio/steampipe'

Perhaps related, I'm seeing another error that seems specific to that repo.

select count(*) from github_release where repository_full_name = 'turbot/steampipe';

ERROR:  rpc error: code = Unknown desc = failed to populate column 'published_at': rpc error: code = Internal desc = transform convertTimestamp failed with panic runtime error: invalid memory address or nil pointer dereference

select count(*) from github_release where repository_full_name = 'turbot/steampipe-plugin-github'

count
1

Fixed in Steampipe v0.8.5
https://github.com/turbot/steampipe/blob/main/CHANGELOG.md#v085-2021-10-07


Validate Party -> Color

Pivotal card
This is based on the same branch as #146 so I could use the util/validate-elements fn from there. But only commit d91587cce4ecd2477eea365944ee0f8221d1e938 and forward are relevant to this PR.

LGTM!


Improve classic player controls configurability

Timestamp positions
Font size
Borders

Changes Unknown when pulling f36bd8f2e54036a140f590de4270c83faac52823 on tf:classic-player-controls-configurability into ** on codevise:master**.


added filter for interactable press receiver and updated hololens 2 b…

Overview
Interactable press receiver can now filter for far, near, or both event types. This allows us to properly configure buttons so they play sounds only from the front.
Changes

Fixes: #5616

Tested on HL2 device with 'Far only' filter option in the Interactable. It works well!
Also updated the blue Round button prefab as well.

Also tested on HL1 and WMR headset.

I'm okay with this, but it is not ideal. Would love to hear from @Troy-Ferrell @killerantz  or @Railboy  if we could have a better approach for serializing these fields.

/azp run

I reset my feedback since I haven't tested thoroughly enough. @cre8ivepark  or @thalbern  are you able to test and verify on HL2? We should do that before checking in.

I'm okay with this, but it is not ideal. Would love to hear from @Troy-Ferrell @killerantz  or @Railboy  if we could have a better approach for serializing these fields.

What do you mean serializing these fields?

@julenka I'll do testing for HL2 & HL1

Tested on HL2 and HL1. Looks good!

did another test on HL2 and looks good to me

upgrade tests failed for me. using existing assets and updating to new code will break the existing pressreceivers - they need to be  manually removed and readded

did several other upgrade scenarios which all failed for me. don't merge this change!

Verified that I can have two receivers: a near and a far, and do different things based on whether I pressed near and far:

/azp run

did some more testing on HL2 and upgrade tests with old buttons - looks good to me now.

/azp run

Doing upgrade testing with Periodic Table app now.

Finished upgrade test with Periodic Table app. Looks good!

thanks Yoon! :)


ReadMeHTMLTranslator error on README.rST file containing images

No matter how I try, I cannot succeed to use flit with this README.rst file, and the issue comes from the images.
The file is a valid rST file, as shown by GitHub displaying it, and readme_renderer test succeeding.
Flit fails with the following error:
$ flit sdist
self = <flit.vendorized.readme.rst.ReadMeHTMLTranslator object at 0x7f9e513b4a90>
self.context = ['</p>\n']
node = <image alt="PyPI version" uri="https://img.shields.io/pypi/v/lempel_ziv_complexity.png"/>
Traceback (most recent call last):
  File "/usr/local/bin/flit", line 11, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.5/dist-packages/flit/__init__.py", line 114, in main
    SdistBuilder(args.ini_file).build()
  File "/usr/local/lib/python3.5/dist-packages/flit/sdist.py", line 131, in __init__
    self.ini_info = inifile.read_pkg_ini(ini_path)
  File "/usr/local/lib/python3.5/dist-packages/flit/inifile.py", line 114, in read_pkg_ini
    return _validate_config(cp, path)
  File "/usr/local/lib/python3.5/dist-packages/flit/inifile.py", line 159, in _validate_config
    _, ok = render(raw_desc, stream)
  File "/usr/local/lib/python3.5/dist-packages/flit/vendorized/readme/rst.py", line 120, in render
    parts = publish_parts(raw, writer=writer, settings_overrides=settings)
  File "/usr/local/lib/python3.5/dist-packages/docutils/core.py", line 448, in publish_parts
    enable_exit_status=enable_exit_status)
  File "/usr/local/lib/python3.5/dist-packages/docutils/core.py", line 662, in publish_programmatically
    output = pub.publish(enable_exit_status=enable_exit_status)
  File "/usr/local/lib/python3.5/dist-packages/docutils/core.py", line 219, in publish
    output = self.writer.write(self.document, self.destination)
  File "/usr/local/lib/python3.5/dist-packages/docutils/writers/__init__.py", line 80, in write
    self.translate()
  File "/usr/local/lib/python3.5/dist-packages/docutils/writers/_html_base.py", line 71, in translate
    self.document.walkabout(visitor)
  File "/usr/local/lib/python3.5/dist-packages/docutils/nodes.py", line 174, in walkabout
    if child.walkabout(visitor):
  File "/usr/local/lib/python3.5/dist-packages/docutils/nodes.py", line 174, in walkabout
    if child.walkabout(visitor):
  File "/usr/local/lib/python3.5/dist-packages/docutils/nodes.py", line 187, in walkabout
    visitor.dispatch_departure(self)
  File "/usr/local/lib/python3.5/dist-packages/docutils/nodes.py", line 1895, in dispatch_departure
    return method(node)
  File "/usr/local/lib/python3.5/dist-packages/docutils/writers/html4css1/__init__.py", line 658, in depart_paragraph
    self.body.append(self.context.pop())
IndexError: pop from empty list

I added the first line to check, apparently your depart_image distutil command is closing the context (self.context.pop()) too early (I am not an expert in distutil HTML translator, so I don't know exactly how it does it)
Thanks in advance.

Also, flit fails if the current directory is not a VCS ?! Why?
It seems fair to force to have no un-commited files or a durty VCS state, but completely inappropriate to force to be in a VCS...

After removing the images from my README.rst files, of course the upload worked perfectly well:
https://pypi.org/project/Lempel-Ziv_Complexity/#description
So thanks and :clap: for flit!

I cloned the repo and tried it, and it seems to be fine with the README on my machine. :confused: If there's a bug in docutils, there's not much I can do besides adding some way to skip the rendering check.
The VCS requirement is only for building sdists - you can build wheels or install the package without it being in a VCS. It's a bit of an awkward compromise, but we use the VCS to define which files to include in the sdist. I'm very keen to avoid something like MANIFEST.in defining which files go into the sdist.
I've just noticed that it fails to identify that the parent directory is a git repo, though. That's a bug.

a3b092da0b3ee779b5dbb81cc3fe996adbf759fb should fix the issue where it can't identify that the CWD is inside a git repo.

Hi, thanks for your quick reply.

OK for the VCS detection (and the bugfix) part,
Not so important if it bugs only on my machine.. Don't worry about it.
But if someone else reports a bug related to images and this self.context.pop issue, we will have to solve it :wink: !

Aha, it looks like I was on an old version of docutils; when I upgrade to docutils 0.13.1, or 0.14rc1, I do see it.

Should be fixed now - can you try with master? If that works for you, I'll make a quick release.

Trying...

Seems fine now.

Thanks :-). I'll make a release with the fix.

And done


SSL routines::unsafe legacy renegotiation disabled

Hi,
We're trying to use your kvk-api implementation but we're getting a SSL error which we can't figure out. Renegotiation disabled.
Tried with and without adding the cert chain to the trusted certificates on ubuntu. Also tried with new Production() in case it was only an issue on the test endpoint but the same result.
Ever run into this issue by any coincidence?
$client = KvkClientFactory::create(config('kvk.api_key'), new Testing(), base_path('vendor/werkspot/kvk-api/cert/Private_G1_chain.pem'));

$searchQuery = new SearchQuery();
$searchQuery->setStreet('ABEBE Bikilalaan');
$kvkPaginator = $client->fetchSearch($searchQuery);
dd($kvkPaginator);

Seconded. @Zwartpet have you found a workaround?

@jaapromijn nope, and as there was no response here I ended up creating a simple guzzle client
<?php

namespace App\Clients;

use GuzzleHttp\Client;
use GuzzleHttp\Utils;
use Illuminate\Support\Facades\Cache;
use Psr\Http\Message\ResponseInterface;

class KvKClient
{
    private Client $client;

    public function __construct()
    {
        $this->client = new Client([
            'base_uri' => config('kvk.base_uri'),
            'headers' => [
                'Accept' => 'application/json',
                'apikey' => config('kvk.api_key'),
            ],
            'verify' => config('kvk.ssl_certificate_verification'),
        ]);
    }

    public function search(string $name = null)
    {
        return $this->client->getAsync("v1/zoeken?aantal=100&handelsnaam=$name")
            ->then(fn (ResponseInterface $response) => $this->decodeResults($response))
            ->wait();
    }

    public function basicProfile(int $kvkNumber)
    {
        return Cache::remember("kvk-profile-$kvkNumber", 3600, function () use ($kvkNumber) {
            return $this->client->getAsync("v1/basisprofielen/$kvkNumber")
                ->then(fn (ResponseInterface $response) => $this->jsonDecode($response))
                ->wait();
        });
    }

    private function jsonDecode(ResponseInterface $response)
    {
        return Utils::jsonDecode((string) $response->getBody(), true);
    }

    private function decodeResults(ResponseInterface $response)
    {
        $json = $this->jsonDecode($response);

        return $json['resultaten'];
    }
}


Add HUF as a currency.

The ISO code HUF is used when referring to the Hungarian currency, forint.
Added that as a Currency Unit.

for @Formatted , this got merge-conflicted with another PR. Do you mind if I just put them in myself?

that should do it.
cheers!


not working on react native 0.63

its working fine on react native 0.61.5 but its not working on latest react 0.63.3

Same here. Removed react-native-video and added vlc-media-player not getting this when I build.
Log:

Task :app:mergeDebugNativeLibs FAILED

FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:mergeDebugNativeLibs'.

A failure occurred while executing com.android.build.gradle.internal.tasks.Workers$ActionFacade
More than one file was found with OS independent path 'lib/x86/libc++_shared.so'. If you are using jniLibs and CMake IMPORTED targets, see https://developer.android.com/studio/preview/features#automatic_packaging_of_prebuilt_dependencies_used_by_cmake

I am working on it. Should be able to release by the end of next week.
On Thu, 29 Oct 2020 at 2:34 AM, Bill Kyriakakis<EMAIL_ADDRESS>wrote:

Same here. Removed react-native-video and added vlc-media-player not
getting this when I build.
Log:
Task :app:mergeDebugNativeLibs FAILED
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:mergeDebugNativeLibs'.

A failure occurred while executing
com.android.build.gradle.internal.tasks.Workers$ActionFacade
More than one file was found with OS independent path
'lib/x86/libc++_shared.so'. If you are using jniLibs and CMake IMPORTED
targets, see
https://developer.android.com/studio/preview/features#automatic_packaging_of_prebuilt_dependencies_used_by_cmake
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/razorRun/react-native-vlc-media-player/issues/36#issuecomment-717936072,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFB72EHLHY4ZK7GTKE2K7BLSNAMV5ANCNFSM4S4IFPNA
.

@razorRun Any update on this?

@razorRun Pardon me, any update for this issue. You are doing great work. Thanks

Please use this test repo https://github.com/razorRun/react-native-vlc-media-player-test
The package is working and tested on RN V0.63.4


Allow hashable-1.5

There are no relevant changes in 1.5 that might cause breakage.

Please do publish a revision after merging this.

Thanks! This is on hackage now, <IP_ADDRESS> revision `


Report mouse move when outside window

https://github.com/HumbleUI/HumbleUI/issues/27

Report mouse move when window out of focus
Add x/y to EventMouseScroll

Check if both are possible on all platfroms

[ ] Windows
[ ] X11
[ ] macOS

On X11, mouse & scroll movement, while the mouse is within the bounds of the window, is already sent regardless of whether it's focused or not. And, given that both come from the same event, adding x/y to EventMouseScroll would be trivial.

Just tested that on Windows EventMouseMove is received regardless of whether the window is focused or not, as long as the cursor is above the visible part of the window. It's also possible to capture mouse movements outside the window using SetCapture https://learn.microsoft.com/en-us/windows/win32/api/winuser/nf-winuser-setcapture
By the way, nice talk at London Clojurians, really liked your attention to detail!

Yeah, we don’t really need mouse events outside window, but I wonder how would one know then that mouse did went outside? Is there a different kind of event?

Some event on mouse leaving & re-entering the window would make sense and could be useful. X11 has those as XI_Enter & XI_Leave (XI_Enter being already in use for a scroll workaround).

Windows has the WM_MOUSELEAVE event. When the mouse enters, it generates WM_MOUSEMOVE events again.

I think I remember someone mentioned that Wayland won’t report mouse move events outside window too. It feels like the way forward is to add EventMouseLeave #287 and mouse coordinates to EventMouseScroll. I’ll change the description here for the latter


Update uk.coffee

Added some Ukrainian translates

Hi @deivan  , thanks for your translations! Can you sign the Contributor License Agreement so we can merge this in?

Hello!
I did it.
16.08.2020 10:59, Valentin Bryukhanov<EMAIL_ADDRESS>
Hi @deivan  , thanks for your translations! Can you sign the Contributor License Agreement so we can merge this in?
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/codecombat/codecombat/pull/5924#issuecomment-674494784

с уважением,
Иван Деменков.
http://demenkov.dp.ua/


C2U sites

Hi there,
I have a question regarding the conversion observed in the output of Bullseye for single-cell data. It might be a silly question, and I might be missing something, but I can't seem to find the answer.
In the output of Find_edit_site.pl, I observe that the chr2:121983394 position has a C2U change:
chr2	121983393	121983394	B2m|3UTR|C2U|mut=11|NA	0.6111	+	0	10	0.6111	18	C2U	639938
chr2	121983393	121983394	B2m|3UTR|C2U|mut=11|NA	0.6111	+	0	10	0.6111	18	C2U	720184
chr2	121983393	121983394	B2m|3UTR|C2U|mut=27|NA	0.6923	+	0	10	0.6923	39	C2U	487034
chr2	121983393	121983394	B2m|3UTR|C2U|mut=24|NA	0.7273	+	0	10	0.7273	33	C2U	283074
chr2	121983393	121983394	B2m|3UTR|C2U|mut=20|NA	0.7692	+	0	10	0.7692	26	C2U	158340

However, when I refer to the reference genome I used with parseBAM.pl, I see that I have a T at that position:
$samtools faidx GRCm39.genome.fa chr2:121983393-121983394
>chr2:121983393-121983394
CT

My question is: how can I be observing a C2U when I have a T at the reference position?
Thank you.

Forgot to mention, but it may be relevant. The m6a list was obtained using a YTH-mut sample as the control matrix.

Hi,
Indeed when comparing to a YTH-mut sample, Find_edit_site.pl is not aware of the sequence at the annotated genome (unless the --fallback option is used and there is not enough coverage in the control YTH-mut sample). Instead, it only checks at the sequence in the control file. So it it possible to see a C2U if a T is found at the reference position, but when there is a mutation/SNP in you cell/mouse line at that position. For example the alignement coverage in the YTH-mut sample may show that a "C" is found instead of a "T".
You can look at the coverage at this position using tabix.
tabix YTH_mut.matrix.gz chr2:121983394
this will output: "chr position #A #T #C #G #G #total" and you can see what is the coverage in your control sample.
If this is because of a known SNP, you can filter them out using the "--filterBed SNP.bed" option using a bed files of annotated SNPs.
Please let me know if that answer your question or if you have another question.
If it doesn't, it may be a bug and I will need to look into it.
Best,

Hi,
Based on the information you gave me, I would say there is definitely something odd going on and I would not say that there is a C2U at this position.

Could this Bullseye's behavior in calling variants be due to the variability that exists in the YTH-mut cells?

If we look at the results you provided:
chr2	121983393	121983394	B2m|3UTR|C2U|mut=11|NA	0.6111	+	0	10	0.6111	18	C2U	639938
chr2	121983393	121983394	B2m|3UTR|C2U|mut=11|NA	0.6111	+	0	10	0.6111	18	C2U	720184
chr2	121983393	121983394	B2m|3UTR|C2U|mut=27|NA	0.6923	+	0	10	0.6923	39	C2U	487034
chr2	121983393	121983394	B2m|3UTR|C2U|mut=24|NA	0.7273	+	0	10	0.7273	33	C2U	283074
chr2	121983393	121983394	B2m|3UTR|C2U|mut=20|NA	0.7692	+	0	10	0.7692	26	C2U	158340

It seems that there is no coverage in the YTHmut at this position: column 7 is the number of mutations in control sample and column 8 is the coverage (or 10 if it is less then -ControlMinCoverage if specified). However, you say there are lots of reads at that position. This means that there is either a bug here or the control file used was not from a pseudobulk sample. For single single cell analysis, we treated the Control (YTHmut) samples as a "pseudobulk" sample: we ran parseBam.pl in "bulk" mode and not in "SingleCell" mode.
Why?
By averaging the coverage and mutations in the Control samples, we remove the cell to cell variability in the YTH-mut (while also loosing the information on each individual cells).
The default behaviour in Find_edit_site.pls is to compare the lines in the DART matrix file with those in the Control matrix file. If there are the same position, it compares coverage, mutations. Then it will move to the next line on the DART matrix only. It will go over each line at this genomic position (each individual cell) until it hits a new genomic position (ie. from chr2:121983394 to chr2:121983395) and then reads the next line in the Control matrix file until it matches the new position in the DART matrix. So this means that it if there are multiple lines in the Control matrix, only the first one will be used. If the Control matrix was processed in "SingleCell" mode, the first line had low coverage (as expected in single cell cell datasets), and Find_edit_sites would assume 0 mutation and 10 coverage as seen above instead. This could explain why it called those sites in some cells. If it had used the pseudobulk, it should have seen:  948,084 (T) / ( 3,325 (C)+  948,084 (T) ) =  99.6% editing in YTHmut and the sites would have been filtered out. (It would actually have skipped that position)
Can you confirm that a pseudobulk matrix was used? Does the tabix command above reports a single line per position, or multiple lines (one per cell) for the YTHmut?
If you did use the pseudobulk, then there is a bug and I will need to look at in more details. Please let me know.
If you used a single cell dataset instead of pseudobulk, you can use collapse_matrix.pl in /scripts to collapse the matrix to a pseudobulk, or rerun parseBAM.pl in bulk mode.
perl collapse_matrix.pl -i input.matrix.gz -o output.matrix

#the output will be compressed and tabix indexed, do not add .gz in name when running the command

Do you think that it would be safer to run Bullseye individually on the YTH and YTH-mut samples, and then manually discard those present in YTH-mut from the YTH sample?

Running the single cell analysis on both YTH and YTHmut against the genome is not a bad idea. You could also compare single cell YTH and YTHmut to pseudobulk YTHmut and to identify likely background edits. This would allow a more complete picture of editing profiles (% editing, % cells with edits) in both population. Because of the variability in single cell dataset, you would need to choose the good cutoff for removing sites.
I would agree with your statement and with being careful when calling edits. Some sites are more prone to background editing. For example, in human cells we routinely filter out sites obtained from cells expressing Apobec1 alone.
Please let me know if it is working or if you have more questions.


Support basic SCTP over plain/legacy sockets

Using just the basic socket APIs like connect/bind/send/etc it is possible to interact with SCTP sockets. While you don't get the full SCTP feature-set when using them, nevertheless they do work, so... thoughts?

Prs would be accepted.

Prs would be accepted.

Due to the lack of progress I'm going to close it. Any more discussion or contributions are of course always welcome.


utils.go

Fixed min/max.
The functions were identical and the trailing else was superflous.

Thanks.


DO NOT MERGE Rearrangement

Just additions, and fbbt-edit file changed location

The annoying thing I have seen most frequently is when we have something
like
[Term]
...
relationship: foo X:1
And foo is not declared in a typedef, the error message provided is totally
uninformative, and it can take a long time to figure out the issue... but
more of an issue with hand-edited files, mea culpa...
On Mon, Jun 10, 2019 at 2:18 PM David Osumi-Sutherland <
<EMAIL_ADDRESS>wrote:

@dosumis commented on this pull request.
In src/ontology/fbbt-edit.obo
https://github.com/FlyBase/drosophila-anatomy-developmental-ontology/pull/581#discussion_r292195665
:

xref: RO:0002004
-is_a: contained_in ! contained_in

I thought we were going to have to remove the Typedefs completely (the
point I was trying to make above), but on a whim I tested adding the name
back and the roundtrip now works. I tried a few more tests - it doesn't
even seem to matter if ID and name match, as long as there's a name there!!
But no name + import of the same OP reliably => this roundtrip failure.
Weird.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/FlyBase/drosophila-anatomy-developmental-ontology/pull/581?email_source=notifications&email_token=AAAMMOKSKEXH5M6D5SOKI53PZ3AJHA5CNFSM4HWSBPFKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB3C6OYQ#discussion_r292195665,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAMMOKGREI2HJKOZXLA7OLPZ3AJHANCNFSM4HWSBPFA
.

There now is a fbbt-flybase.obo release which needs a bit of tending. @Clare72 would you mind diffing and just listing the important differences?

So aside from the diffing, there are some edits in the protocol:
https://github.com/VirtualFlyBrain/Management/blob/master/doc/preparing_FB_ontologies_for_release_for.md
That are now not happening during fly_anatomy.obo generation:

in_neural_circuit_with, upstream_in_neural_circuit_with, downstream_in_neural_circuit_with typedefs need removing
is_a relationships to above typedefs need removing (from electrically_synapsed_to, synapsed_to, and synapsed_by)
has_quality typedef needs adding:

[Typedef]
id: has_quality
name: has_quality
comment: Corresponds to OBO_REL:has_quality in http://obo.cvs.sourceforge.net/viewvc/obo/obo/ontology/OBO_REL/ro_proposed_edit.obo. As this relation in not in the main relations ontology file (ro.obo) I have avoided adding an xref.

has_quality typedef needs adding:

We should probably just switch to RO:bearer_of for relating anatomical entities to PATO qualities.

To progress on the matter, I would suggest to reduce the effort of getting everything right to getting simple, fbbt.owl and fbbt.obo (which is simple) in shape. If they are okay, lets merge the branch, and do the rest of the work in the normal issue-pull-request-merge fashion until the flybase release file is perfect. In terms of responsibilities I suggest this:

@Clare72 manages the process. After each change, she eyeballs the diffs and posts issues for everything that does not look right.
I am responsible for implementing the pipeline. Whenever clare posts and issue, I will implement it, create a new PR and clare reviews it.
David will occassionally review Pull requests, but I think we can mostly leave him out of the process now; we know what to do.
We focus on getting simple right for now; once this is right we fix the flybase release including full automation of SOP; we eyball the main release artefacts (fbbt.owl and fbbt,obo) in Protege. The rest we leave up to the community to feed back to us.

Generally I would rather like to do this slowly slowly now, than doing it all under pressure when October comes. After we have a standard Github release out for all 4 ontologies, we (clare and I) will start developing training materials and documentation of the pipeline. What do you think?

For fbbt-simple:
Some things still to resolve with autodefinitions - see #590
Loading in oboedit fails for the same reason as fbdv - see https://github.com/FlyBase/drosophila-developmental-ontology/issues/5
Many typedefs have two 'name' fields (I think this came up somewhere else, but couldn't find it)
Otherwise seems ok

Please review again! Should be all good now. Thaaaanks!

I think we can merge this now, I will make a ticket about editing the "." definitions where there is no logical definition.

@Clare72 I am happy for this to be merged; please merge it yourself!


Different lifecycle phases

Just having a single "end" date doesn't seem sufficient to me.
We'd need different lifecycle phases. One example is here: https://access.redhat.com/support/policy/updates/openshift#ocp4_phases

We have discussed this in our prior efforts at endoflife.date, and came to same conclusion. "Support" is a broad term, and it means different things to different projects.
Our current effort at the releases.json specification (WIP PR: https://github.com/endoflife-date/releases.json/pull/1) uses generic "plans" (https://github.com/endoflife-date/releases.json/pull/1/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5R117-R134), with their own attributes for eol, releaseDate, latestVersion, but it isn't clear enough.
Another effort by @noqcks (https://github.com/endoflife-date/endoflife.date/discussions/3484) uses a vendor decided integer value (support.level.int) to differentiate between various support levels. This has the issue with the levels not necessarily always remaining in sorted order, since support levels can change for a product over time. But overall, this documents phases much better.
In my experience, attempting to use a single lifecycle terminology does not work across multiple projects. It is much better to provide clear attributes for what is covered under each phase. The important attributes to cover are:

security: Whether or not security fixes are provided
features: Whether or not new features are available.
commercial: Whether this lifecyle phase is available via an additional commercial contract (compared to procurement).
availability: Whether this phase includes availability of the product. For eg, old devices are discontinued (you can not buy them), but are covered under support. We've documented this for Raspberry Pi, Intel and other software in the past.

As long as there are clear identifiers for each phase, tooling can be built to accommodate for that.


[] <No app has been configured yet. Error retrieving thread information: (ipc/send) invalid destination port>

I made a new app and did the steps in Decs but when I run the app, it is run correctly but still have errors on console
my console errors:

I put FirebaseSDKVersion on my podfile like this

my pub file

my testing code
import 'package:flutter/material.dart';

// Import the firebase_core plugin
import 'package:firebase_core/firebase_core.dart';

void main() {
  runApp(App());
}

class App extends StatefulWidget {
  _AppState createState() => _AppState();
}

class _AppState extends State<App> {
  // Set default `_initialized` and `_error` state to false
  bool _initialized = false;
  bool _error = false;

  // Define an async function to initialize FlutterFire
  void initializeFlutterFire() async {
    try {
      // Wait for Firebase to initialize and set `_initialized` state to true
      await Firebase.initializeApp();
      setState(() {
        _initialized = true;
      });
    } catch (e) {
      // Set `_error` state to true if Firebase initialization fails
      setState(() {
        _error = true;
      });
    }
  }

  @override
  void initState() {
    initializeFlutterFire();
    super.initState();
  }

  @override
  Widget build(BuildContext context) {
    // Show error message if initialization failed
    if (_error) {
      return MaterialApp(
        home: Scaffold(
          body: Center(
            child: Text('Error 1'),
          ),
        ),
      );
    }

    // Show a loader until FlutterFire is initialized
    if (!_initialized) {
      return MaterialApp(
        home: Scaffold(
          body: Center(
            child: Text('loading'),
          ),
        ),
      );
    }

    return MaterialApp(
      home: Scaffold(
        body: Center(
          child: Text(' my awesome app'),
        ),
      ),
    );
  }
}

@Abdullah-Zakhoi I think this may be similar to:
https://github.com/flutter/flutter/issues/63025

Hi @Abdullah-Zakhoi,
Can you try to remove firebase_core plugin and related code and then run the app to see if you still get the error ?
Trying to see if the error is indeed due to firebase_core plugin or due to the upgrade as mentioned by @kroikie above.
Thanks.


Added OnItemExpandCollapseListener code to AbstractSlideExpandableListAd...

I added code which triggers a callback whenever an item expands or collapses because I needed to drastically change the look of the list view item when it was expanded. If you feel this kind of thing would be useful for other people using the library, please merge the pull request. The code is tested and working.
Regards,
Dino

@reisub  @robertotucci
I'm sorry,but I have the same problem.Here is my code.
mListView = (ActionSlideExpandableListView) findViewById(R.id.file_path_list);
OnItemExpandCollapseListener expandCollapseListener = new OnItemExpandCollapseListener() {
    	@Override
    	public void onExpand(View itemView, int position) {
    		showMsgToast("onExpand position:"+position);
    		((ImageView)itemView).setImageResource(R.drawable.icon_collapse);
    	}

    	@Override
    	public void onCollapse(View itemView, int position) {
    		showMsgToast("onCollapse position:"+position);
    		((ImageView)itemView).setImageResource(R.drawable.icon_expand);
    	}
    };

    slideAdapter = new SlideExpandableListAdapter(mAdapter);
    mAdapter = new FileBrowserCursorAdapter(this, list,mCurrentPath);
    slideAdapter.setItemExpandCollapseListener(expandCollapseListener);
    mListView.setAdapter(slideAdapter);

    mListView.setItemActionListener(this);

the event is not called too.Help!!


Update org.eclipse.jdt.core to 3.24.0

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

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.eclipse.jdt", artifactId = "org.eclipse.jdt.core" } ]

labels: library-update, semver-minor

Codecov Report

Merging #891 (3f3320c) into master (4acc246) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #891   +/-   ##
=======================================
  Coverage   83.36%   83.36%
=======================================
  Files          73       73
  Lines        5460     5460
  Branches      162      162
=======================================
  Hits         4552     4552
  Misses        908      908

Continue to review full report at Codecov.

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


Change Charging Stations layer from Esri Feature to GeoJSON

link to demo deployment of your smk based app is
https://smk-clean-bc-rt-b16795-dev.apps.silver.devops.gov.bc.ca
Verify that it looks good, if so merge the pull request which will trigger a prod deployment, otherwise close the pull request without merging
mentions / notifications: @dgboss, @NicoledeGreef

The Vanity URL for the dev deploy is https://smk-clean-bc-api-gov-bc-ca.test.api.gov.bc.ca
If this is a dev deploy,  verify that it looks good, then merge the pull request to trigger prod deployment.
If this is a prod deployment, view your prod deploy at the url above :)
mentions / notifications: @dgboss, @NicoledeGreef

re attributes, looking at the data dictionary (https://developer.nrel.gov/docs/transportation/alt-fuel-stations-v1/all/#fuel-station-record-fields) and the fields shown...

I think an important attribute to bump up would be EV Connector Type - if we could present it in plain text, rather than as a list/array that would be prettier
I reckon the two French fields can go - our app is not attempting to be bilingual in any other way
Also Geocode Status can go. OR convert values using a dictionary - e.g. 200-9 --> Point, 200-8 --> Address
EV DC Fast number (and similary ev_level1 or 2_num) is the number of ports of that type. Maybe this could be expressed better as '# of DC Fast Ports', '# of Level 2 Ports'. Or reworked into a single field e.g.: Level 2 (3 ports), Level 1 (5 ports)
Access Detail Code is redundant with Groups With Access Code
Owner Type Code seems unnecessary
Open Date seems unneeded

And one more legend thing - I think Identify Edit Search Area can be hidden

re attributes, looking at the data dictionary (https://developer.nrel.gov/docs/transportation/alt-fuel-stations-v1/all/#fuel-station-record-fields) and the fields shown...

I think an important attribute to bump up would be EV Connector Type - if we could present it in plain text, rather than as a list/array that would be prettier
I reckon the two French fields can go - our app is not attempting to be bilingual in any other way
Also Geocode Status can go. OR convert values using a dictionary - e.g. 200-9 --> Point, 200-8 --> Address
EV DC Fast number (and similary ev_level1 or 2_num) is the number of ports of that type. Maybe this could be expressed better as '# of DC Fast Ports', '# of Level 2 Ports'. Or reworked into a single field e.g.: Level 2 (3 ports), Level 1 (5 ports)
Access Detail Code is redundant with Groups With Access Code
Owner Type Code seems unnecessary
Open Date seems unneeded

And one more legend thing - I think Identify Edit Search Area can be hidden

Thanks for these suggestions. I'll call these out in the Issue and ask for client feedback.
We are limited in what we can do with formatting values. There are some options (https://bcgov.github.io/smk/docs/configuration/layers/attribute#format-property) - I've already applied "asLink()" - but we don't yet have support for formatting arrays or using a dictionary. This would have to be done in SMK.

link to demo deployment of your smk based app is
https://smk-clean-bc-rt-b16795-prod.apps.silver.devops.gov.bc.ca
Verify that it looks good, if so merge the pull request which will trigger a prod deployment, otherwise close the pull request without merging
mentions / notifications: @dgboss, @NicoledeGreef

The Vanity URL for the prod deploy is https://smk-clean-bc.api.gov.bc.ca
If this is a dev deploy,  verify that it looks good, then merge the pull request to trigger prod deployment.
If this is a prod deployment, view your prod deploy at the url above :)
mentions / notifications: @dgboss, @NicoledeGreef

This resolves #84.


Task - new asserted distributions: Always show most recently created asserted distributions

Feature or enhancement
When I use "new asserted distributions", newly created entries are shown in a list. However when I navigate away from the page and return, the display shows asserted distributions ordered from the oldest created ones (so you don't see any that you recently created).
I think it would be more useful for the display here to always show most recently created asserted distributions.
Location
Task - new asserted distributions
Screenshot, napkin sketch of interface, or conceptual description

Above, I've been adding asserted distributions for Agrilus and they are appearing on the list.

Navigated away from the page, returned and now the list just includes the earliest Asserted distributions I created.
Your role
No response

@jlpereira I think you can simply add &recent=true to the initial request.


raw.plot fixes not in 0.15.1?

@agramfort could it be that https://github.com/mne-tools/mne-python/pull/4692 didn't get into 0.15.1?

Yes I missed the backport label :(

This was the main reason (at least for me) for the bugfix release - I guess we can't just include it and call it 0.15.1 again. Can we release 0.15.2 pretty soon?

yes backport and I'll push 0.15.2
let me know when to do it

Backported and bumped version to 0.15.2, feel free to (double check and) cut a release

done https://pypi.python.org/pypi/mne


add code to handle image merge

Description
To handle image merges prefix your field by IMAGE: and add the binary image (use .png) as a value.
In stead of using normal mailmerge fields in the template docx, you should add a placeholder image and add the field name as the image ALT TEXT.

In this way you can get a correct preview in you template, and add image layout markup to the placeholder image, that will be inherited by the inserted images.
you can call it like this:
f1 = open('test_img1.png')
img_data_1 = fp.read()
f1.close()

f2 = open('test_img2.png')
img_data_2 = fp.read()
f2.close()

document.merge_templates([
    {'field1': "Foo", 'field2: "Copy #1", 'IMAGE:merge_img1': img_data_1 },
    {'field1': "Bar", 'field2: "Copy #2", 'IMAGE:merge_img1': img_data_2 },
], separator='page_break')

Motivation and Context
This resolved #12 and it is listed as TODO in the readme.
How Has This Been Tested?
It works in my application (currenty running on MacOS)
If there are plans to accept this PR I wil spend some more time adding unit tests.
Types of changes

[ ] 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)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Nice idea and great to see that it helps with your use case! However as the implementation isn't using merge fields, I don't think I can merge this. I don't want this library to turn into a general purpose Word library, but keep it focused around merge fields.
I don't know if merging images with merge fields is even be possible in the Word format. So if that's not possible, I should probably just remove that goal from the TODO.

This PR can easily adapted to the use of merge fields. However as far as my (limited) knowledge of merge fields goes, there are designed for inserting text only. I decided for the use af a sample image as "merge field" here so the designer of the word doc has more freedom about the image properties (like the size, the text flow around the image etc.)
If we are to use merge field, we would replace it by complete embed xml code for the image using some reasonable default image settings.
It may be good to know what other users in the community would like to see here.

I would love to see such a feature a well. I agree with @Bouke though that this library should focus on mailmerge field features Microsoft Word also supports. This is a unique advantage of this library over e.g. docx-template where everything is possible but is not safe for user as syntax can be messed up.
Microsoft Word seems to have mail merge picture support though. Best post I have found how this works is at https://onmerge.com/articleIncludePicture.html . Maybe an implementation would best follow this.

I definitely would like to see this feature accepted in the library. Replacing images would be very useful.

That's a great feature! However, it does not work on my side. It keeps hanging on ´document.merge_templates()´ forever.  I'm running python 3.7.2 on Win10.


GO seems to be ignoring Noelle DEF->ATK conversion when using burst as a target

When I've tried to optimize my Noelle build after getting her to C6 I get very strange builds with 2NO/2 Archaic Petra showing total around 1300 ATK in formulas instead of ~3500 ATK (which is what my 1000 ATK 2000 DEF C6 Noelle has rn after using her burst) - and I see no obvious option in the GUI to enable her DEF->ATK conversion when I select burst as a target. What should I do? I've tried to select both "Skill DMG" and "Burst DMG" as targets but got the same result. And there's no obvious option for self-infused NAs either.

Talents tab? Did you activate her C6? Or turn on her burst state? https://youtu.be/hiRjngMgHfQ

Wow, that's... so incredibly unintuitive. The entire point of optimizing tab is to set conditions so why one of condition switches is located in an entirely different place?
Thanks tho, it did the trick.

Talent conditionals are in the talents tab. You had to go to the talents tab to level the talents anyway so 🤷

You had to go to the talents tab to level the talents anyway so

Not when you import data from Inventory Kamera - all that data including talent lvls is already there.

GO can't assume everyone uses Kamera to import.

OK makes sense now

You have to consider that what if you wanted to optimize for Noelle's normal attacks? You must have a way to "turn off" the condition that the burst talent puts you in. Which is why even if you're activating FOR the burst, you need to tell GO that the resulting condition has been met.


Fix updating of the chart

Hi, I had an issue where the chart would not update when the data changed. This fixed it for me. Thank you for this package 🙏.

Hi @UCetinkaya94, thanks for the first contribution to this package!
I changed minor things and add tests. I'll merge it if the action is passed.


loading script on server

Hi,
I am new to Javascript... and I am wondering if this script can be used server side?
I see to load the script you used:

I think that is client side loading?  How can this be done server side?  Can your script be used server side?
Thank you

Hello,
Indeed, I used a client side example in the documentation. But it's totally ok to use it on the server side. If you use Node.js or something equivalent on the server side you can do a let gpxParser = require('gpxparser').
Additionally, I didn't wrote it in the doc but the library is also available on npm with a npm install gpxparser

Luuka,
Super !  -Thanks for the update... getting this web programming stuff in my head is hard enough without having to figure out how to get the infrastructure stuff to work.
Thank you again !!


feature: allow users to provide custom pool class

I think no one is a stranger to the fact that we've been seeing a fair amount of chatter around issues with our pooling implementation. Its come to my attention that some teams might want to opt out of certain features of our Pooling class or in some cases just have a finer level of control. I would like to propose accepting a Pool constructor that could be used in place of our Pooling class.
I've reviewed our code and with a light refactor I think the bare minimum could be the following interface
interface PoolInterface extends EventEmitter {
  new (database: Database);
  open();
  close(callback: (err: Error) => void);
  getReadSession(callback: (err: Error, session: Session) => void);
  getwriteSession(callback: (err: Error, session: Session, transaction: Transaction) => void);
  release(session: Session);
}

Integrating would be as simple as
class CustomPool extends EventEmitter {
  // ...
}

const database = instance.database('my-db', CustomPool);

Thoughts?

/cc @googleapis/yoshi-nodejs @anup-honey @walshie4


Add GimpShop v2.99 beta, Krita, Photopea (via nativefier)

Apparently this version of Gimp does not suck. Kinto will need photoshop hotkeys added or.. just go ahead and fix the hotkeys on the app level, usually I am against that but we are having to make other changes already to improve the UI.
flatpak remote-add --user flathub-beta https://flathub.org/beta-repo/flathub-beta.flatpakrepo
flatpak install flathub-beta org.gimp.GIMP

sudo add-apt-repository ppa:mati75/gimp30
sudo apt update
sudo apt install gimp gegl

Pull the photoshop shortcuts from here.
https://github.com/doctormo/GimpPs/blob/master/menurc
Other notes.. make sure to disable the Ruler default in preferences to give a cleaner UI. This needs to be set in advanced imo.


After adding an annotation processor to java project lombok annotations stopped working

After a added:
        <pluginManagement>
            <plugins>
                <plugin>
                    <groupId>org.apache.maven.plugins</groupId>
                    <artifactId>maven-compiler-plugin</artifactId>
                    <version>3.10.1</version>
                    <configuration>
                        <annotationProcessorPaths>
                            <annotationProcessorPath>
                                <groupId>io.javalin</groupId>
                                <artifactId>openapi-annotation-processor</artifactId>
                                <version>${javalin.version}</version>
                            </annotationProcessorPath>
                        </annotationProcessorPaths>
                    </configuration>
                </plugin>
            </plugins>
        </pluginManagement>

I'm getting compilation errors
java: cannot find symbol
  symbol:   variable log

I have also this dependency in my project:
        <dependency>
            <groupId>org.projectlombok</groupId>
            <artifactId>lombok</artifactId>
            <version>RELEASE</version>
            <scope>provided</scope>
        </dependency>

I think you should probably just register Lombok's annotation processor after the openapi 🤔

the thing is, lombok doesn't require any annotation processor registration, it works somehow by itself. But probably by registering open-api-processor the process is disturbed. I guess this issue will arise in any Java project since lombok is widely used.

Lombok is based on top of annotation processor with quite a lof of compiler hacks during this process. If you're using some kind of IDE plugin and you're compiling project only in IDE, it can just do perform such action without explicit declaration in build phase probably. Such issue will occur with any annotation processor + lombok, that's why you should declare lombok explicitly in your project - I'll add Lombok to our example project.

Done, I could use dedicated Gradle plugin for Lombok, but I've decided to use regular annotation processor to show it's just a matte of order. Works properly:

For Maven, take a look at this:

https://projectlombok.org/setup/maven

Lombok's declaration should always be the first entry in a list of your regular annotation processors due to its nature.

Yes, you were right, thank you!
So, I imported lombok this way:
        <dependency>
            <groupId>org.projectlombok</groupId>
            <artifactId>lombok</artifactId>
            <version>${lombok.version}</version>
<!--            <scope>provided</scope>-->
        </dependency>

and included it in the annotation processor section:
        <pluginManagement>
            <plugins>
                <plugin>
                    <groupId>org.apache.maven.plugins</groupId>
                    <artifactId>maven-compiler-plugin</artifactId>
                    <version>3.10.1</version>
                    <configuration>
                        <annotationProcessorPaths>
                            <annotationProcessorPath>
                                <groupId>org.projectlombok</groupId>
                                <artifactId>lombok</artifactId>
                                <version>${lombok.version}</version>
                            </annotationProcessorPath>
                            <annotationProcessorPath>
                                <groupId>io.javalin</groupId>
                                <artifactId>openapi-annotation-processor</artifactId>
                                <version>${javalin.version}</version>
                            </annotationProcessorPath>
                        </annotationProcessorPaths>
                    </configuration>
                </plugin>
            </plugins>
        </pluginManagement>


Linker Error : XamlTypeInfo.g.obj : error LNK2019

What does this error means:
XamlTypeInfo.g.obj : error LNK2019: unresolved external symbol __Platform_AcquireSRWLockShared referenced in function "private: void __cdecl Platform::EventSource::DoInvokeVoid<class Windows::Foundation::Collections::VectorChangedEventHandler<struct Windows::UI::Xaml::Markup::IXamlMetadataProvider ^>,class <lambda_e7f2d868892b75e9054b0d8b72f53d64> >(struct Platform::Details::EventLock *,class <lambda_e7f2d868892b75e9054b0d8b72f53d64>)" (??$DoInvokeVoid@V?$VectorChangedEventHandler@PE$AAUIXamlMetadataProvider@Markup@Xaml@UI@Windows@@@Collections@Foundation@Windows@@V<lambda_e7f2d868892b75e9054b0d8b72f53d64>@@@EventSource@Platform@@AEAAXPEAUEventLock@Details@1@V<lambda_e7f2d868892b75e9054b0d8b72f53d64>@@@Z)
XamlTypeInfo.g.obj : error LNK2019: unresolved external symbol __Platform_ReleaseSRWLockShared referenced in function "private: void __cdecl Platform::EventSource::DoInvokeVoid<class Windows::Foundation::Collections::VectorChangedEventHandler<struct Windows::UI::Xaml::Markup::IXamlMetadataProvider ^>,class <lambda_e7f2d868892b75e9054b0d8b72f53d64> >(struct Platform::Details::EventLock *,class <lambda_e7f2d868892b75e9054b0d8b72f53d64>)" (??$DoInvokeVoid@V?$VectorChangedEventHandler@PE$AAUIXamlMetadataProvider@Markup@Xaml@UI@Windows@@@Collections@Foundation@Windows@@V<lambda_e7f2d868892b75e9054b0d8b72f53d64>@@@EventSource@Platform@@AEAAXPEAUEventLock@Details@1@V<lambda_e7f2d868892b75e9054b0d8b72f53d64>@@@Z)
I am just trying to run a simple new project to start using c++/winrt

It looks like you began by converting a XAML C++/CX app, which generates XamlTypeInfo.g.cpp.  This file is silently compiled and linked into your binary, but if you've changed language type to C++ in the project (e.g., by adding false), then the project will pull in different CRT/STL libs, resulting in this error.  You can try deleting this source from the Generated Files folder, along with any other CX artifacts there, to get to a clean C++ state.

Actually i created uwp project using VS project template. So i cam to this situation.
Is there any way to create c++/winrt project from visual studio?

got reply on last query below
https://github.com/Microsoft/cppwinrt/issues/263


Enable PeftConfig & PeftModel to load from revision

This PR enables the from_pretrained of PeftConfig and PeftModel to load configs/models with a specific revision arg. This is useful when storing many adapter weights as branches in a model repo.
Sample usage
from peft import PeftConfig, PeftModel
from transformers import AutoModelForCausalLM

model_id = "lewtun/tiny-random-OPTForCausalLM-delta"
revision = "v1"
config = PeftConfig.from_pretrained(model_id, revision=revision)

model = AutoModelForCausalLM.from_pretrained(
    config.base_model_name_or_path,
)
model = PeftModel.from_pretrained(model, model_id, revision=revision)
assert isinstance(model, PeftModel)

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

@pacman100 This causes a regression when device_map is used for PeftModel.from_pretrained. Probably not a great idea to pass all **kwargs blindly from PeftModel to PeftConfig.


 Would like to request it also pulls down all Extension Attributes with the linked computer groups.
Gr,
Daniel

Nice idea.  Added computer and mobile device usage.  Checks groups and advanced searches.

Looking good!!
It only seems to pull the referenced EA though? I have EA's which are not
referenced by any group and would like to clean those up. Any change you
can pull those as well?
I just noticed the same goes for the scripts and packages.
Otherwise, I am very happy with this tool. Makes life a lot easier!!
Gr,
Daniel
On Fri, Sep 3, 2021 at 11:20 PM Leslie Helou @.***>
wrote:

Nice idea. Added computer and mobile device usage. Checks groups and
advanced searches.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/BIG-RAT/Object-Info/issues/2#issuecomment-912816951,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALQLOCU5YWXLJERYNCZDP2TUAE3ZDANCNFSM5B6ADE3A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Right, the app only reports on items that are scoped/referenced.
Might look at Prune to help identify unused items.  And yes adding extension attributes to that app is on the to-do list.


Fix documentation bugs from PE-D

Fix #145 and #152. Also updated FindCommand description to match new behaviour.

Codecov Report

Merging #168 (ee77a35) into master (3ec81f8) will decrease coverage by 2.91%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #168      +/-   ##
============================================
- Coverage     74.89%   71.97%   -2.92%
+ Complexity      730      715      -15
============================================
  Files            98       99       +1
  Lines          2071     2052      -19
  Branches        276      261      -15
============================================
- Hits           1551     1477      -74
- Misses          421      476      +55
  Partials         99       99

Impacted Files
Coverage Δ

...n/java/seedu/address/storage/JsonAdaptedOrder.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...va/seedu/address/storage/JsonAdaptedItemOrder.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...u/address/storage/JsonSerializableTransaction.java
30.76% <0.00%> (-69.24%)
:arrow_down:

...u/address/storage/JsonSerializableBookKeeping.java
45.45% <0.00%> (-54.55%)
:arrow_down:

...ain/java/seedu/address/storage/StorageManager.java
79.41% <0.00%> (-15.04%)
:arrow_down:

src/main/java/seedu/address/model/UserPrefs.java
82.05% <0.00%> (-2.57%)
:arrow_down:

src/main/java/seedu/address/model/item/Item.java
65.15% <0.00%> (-1.52%)
:arrow_down:

.../seedu/address/logic/parser/EditCommandParser.java
92.85% <0.00%> (-0.48%)
:arrow_down:

src/main/java/seedu/address/MainApp.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/model/Model.java
100.00% <0.00%> (ø)

... and 16 more

Continue to review full report at Codecov.

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


updating various depedencies: scalaz, scalaz-stream, netty, scodec an…

…d specs2

We should update the README.md to be consistent with this change.


Allow to accept sync function for Maybe/mapAsync

We close this once by https://github.com/option-t/option-t/issues/1683#issuecomment-1456004819.
We'll reopen this when we conclude about #1683.
If you have some opinions, please comment to #1683


Space Bar key press to Select the range of month bug

Using the space bar to select the start month and end month.
For example, I selected April 2024 and Jun 2024 as the start month and end month (please see attachment 1).
I select again with the spacebar for the month that does not fall under April 2024 and Jun 2024, e.g. Jan 2024. The onChange start month given is not Jan 2024 but it is the start month as March 2024 (please see attachment 2).
I didn't do any calculation on the onChange to console log it (please see attachment 3).

attachment 1 - selected April 2024 and Jun 2024 with space bar:

attachment 2 - selected Jan 2024 but onChange received March 2024 with space bar:

attachment 3 - no calculation to console log the start date and end date with space bar to select:

Hope you may assist me on this issue.
Thank you very much.

sorry for raising this...it caused by our own code that added 1 keydown event to the component. See code below:
`  useEffect(() => {
const keyDownFunction = (e: KeyboardEvent) => {
setKeyPressed(e.key);
};
if (!disabled) {
  const containerElement = datePickerContainerRef.current;
  if (containerElement) {
    containerElement.addEventListener("keydown", keyDownFunction, true);
  }
}

return () => {
  if (!disabled) {
    const containerElement = datePickerContainerRef.current;
    if (containerElement) {
      containerElement.removeEventListener(
        "keydown",
        keyDownFunction,
        true
      );
    }
  }
};

}, []);`


fixed lint errors for v0.18

fixed lint errors for version - 0.18
🎊 This is the last PR for (#1338)

As soon as tests pass I'll merge and close the issue 🎉


Http evader v10

Modifies #221 :

Rebased and Removed already merged commits

I merged a number of 'safer looking' patches in #225

Great for #225


Add an AUTHORS file

An AUTHORS file or CONTRIBUTORS file can serve to provide credit to authors and their organisations, in addition to show the diversity of collaborators.
Non-mandatory.

Added a draft, feel free to edit 🙂


fix: removed grafana internal db things from working db

We actually don't need to specify the database + datasource for Grafana. This is what's causing all the internal Grafana DB stuff to populate the lake postgres db.
Updated:

Removed Grafana database env variables

Fixes:

Stops cluttering the working db with Grafana internal db things that we don't really need

Note

It's possible in the future if we want this Grafana internal db stuff back, we can just make another db in docker-compose.yml. Then populate it. As of now though, I don't see any use for it.

This is duplicated work. Klesh already did this


K采样器差异化噪波错误

当我运行K采样器差异化噪波的时候提示Error occurred when executing KSamplerVariationsWithNoise+:
'BaseModel' object has no attribute 'get_model_object'

you need to update comfyui


Pluggable verifiers

There needs to be an update to the keys.Key type to fully support other algorithms. Happy to make those changes if you can provide an opinion. On my fork I just generically treat all keys and signatures as []byte and allow the creators/verifiers to worry about precisely what the correct sizes are and copy the slices into arrays.
Signed-off-by: David Lawrence<EMAIL_ADDRESS>(github: endophage)

One thought, the Ed25519 verifier is covered by existing tests. I'll get tests for the RSA verifier eventually but we can take it out for now and leave additional verifiers to the whim of other users. Let me know if you want to do that. I can also just add a comment for now marking the RSA verifier as "untested, use at your own risk" (it does work though, we used it in a demo)

Yeah, add a comment to the effect of "RSA should only be used for compatibility, this verifier is not currently covered by tests, use at your own risk if you really know what you are doing"

Implementation LGTM. Update the comments and squash, and I'll merge. Thanks!


module index file not required

In projects, the following structure is now OK:
lib/
  modules/
    moduleWithOnlyUserCommands/
      usercommands/
        a.js
        b.js
        c.js

Notice that there are no index.js in the module (in which
case, a warning log will be printed out nevertheless)
Fixes #35

Done


Fix getSpeed Typescript definition

Fix getSpeed definition arguments

Oh, I'm not the only one who was stumbled here. I had wanted to make my own PR before I saw this.


[LSP] Wrap implementation of the Language Server in a class

This PR creates a class that wraps the implementation of the Language Server.
The idea is to have a class that contains all structures required for performing LS tasks in order to simplify further implementation of the LS callbacks and functions, e.g. utilization of Symbol Table in https://github.com/chipsalliance/verible/pull/1530

Codecov Report
Base: 92.85% // Head: 92.85% // Decreases project coverage by -0.00% :warning:

Coverage data is based on head (3fbcb1a) compared to base (46b3e0b).
Patch coverage: 97.26% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1560      +/-   ##
==========================================
- Coverage   92.85%   92.85%   -0.01%
==========================================
  Files         345      347       +2
  Lines       24467    24470       +3
==========================================
+ Hits        22720    22722       +2
- Misses       1747     1748       +1

Impacted Files
Coverage Δ

verilog/tools/ls/verilog-language-server.cc
97.05% <97.05%> (ø)

verilog/tools/ls/verilog-language-server.h
100.00% <100.00%> (ø)

verilog/tools/ls/verilog_ls.cc
100.00% <100.00%> (+1.35%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Fix for error during pod install. Tags have v prefix.

Summary
After adding react-native-community/react-native-cameraroll to a project, running pod install fails with an error indicating that tag 2.0.0 cannot be found.
On investigation I found that version tags all have a 'v' prefix. Adding the v prefix in the appropriate place in the podspec fixed the error. This makes react-native-cameraroll consistent with other components in the react-native-community family.
Test Plan

react-native init myproject
yarn add @react-native-community/react-native-cameraroll
cd ios
pod install

What's required for testing (prerequisites)?
What are the steps to reproduce (after prerequisites)?
Compatibility

OS
Implemented

iOS
✅

Android
✅

Thank you for contributing @SteveGreenley !

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Protocol Upgrade (Hard-fork) page is needed

One page that describes all hard-fork history and block numbers can help Klaytn developers to keep changes of Klaytn networks.
There is information about the Protocol Upgrade in IstanbulEVM, LondonEVM, EthTxType
We can also update the following pages to link to the Protocol Upgrade page.

https://docs.klaytn.com/smart-contract/porting-ethereum-contract#solidity-support
https://docs.klaytn.com/smart-contract/solidity-smart-contract-language#solidity-and-klaytn

I created a PR at https://github.com/klaytn/klaytn-docs/pull/431


Azure Web App for Multi-Containers does not support default variables, but is not listed as a limitation

Through testing, it appears default values specified in docker-compose files for multi-container web apps are not supported. When used, regardless of an environmental variable exists, the expression (Ex. ${ENV:-default}) is passed as a string to the application, instead of the default value or the environmental variable referenced.
For example, an image may be configured for registry.azurecr.io/product/component:${IMAGE_TAG:-latest} would result in:
2022-08-28T23:33:06.907Z INFO  - Pulling image: registry.azurecr.io/product/component:${IMAGE_TAG:-latest}
2022-08-28T23:33:06.916Z ERROR - DockerApiException: Docker API responded with status code=BadRequest, response={"message":"invalid tag format"}

This also occurs with environment settings where default values may be used for local testing.
The doc should be updated to note this limitation as the corresponding Web App Multi-container tutorial simply links to the docker page on variables, which describes how to use default values in a docker-compose file. The implication then is that Web Apps should support this, but doesn't.

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

ID: 1d385179-7730-3b99-aff6-75491deb070a
Version Independent ID: 7da429ea-5421-9d99-3e7c-af1b3fee5ca8
Content: Configure a custom container - Azure App Service
Content Source: articles/app-service/configure-custom-container.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

@BradAF,  Thanks for the feedback! We are taking a look into this and will get back to you soon.

@BradAF Environment variable substitutions are only allowed for Env Variables itself in the compose. It is not possible to override the tag for the image with an environment variable.
If you are looking to pull the latest version of a particular image you can specify using : latest.
Kindly let us know what exactly you are trying to accomplish/your requirement.

Hi @Venkateshdodda-MSFT, it's the inline variable default substitution as described in Docker: Substitute Environment Variables in Compose Files that is not working. I've tried in both the image element and the environment element, and both work locally, but when starting up in Azure Web App, the default variable is not used - the entire expression, ex. ${variable:-default} is used instead, which is not the expected behavior.
For example, when used locally and env is set to IMAGE_TAG=v1.0, the image element registry.azurecr.io/product/component:${IMAGE_TAG:-latest} will be passed as registry.azurecr.io/product/component:v1.0. If no IMAGE_TAG is set, registry.azurecr.io/product/component:latest is passed instead. In either case in Azure Web App the literal registry.azurecr.io/product/component:${IMAGE_TAG:-latest} is passed. This is true for anything set in environment element as well, such as something like CORS_ORIGIN=${CORS_ORIGIN:-*}.

@BradAF Thank you for your feedback! I have updated the document and changes will go live soon.


[board-server] #2889 Allow board server to use SQLite and run self-contained in a docker container

Fixes #2889
I think this is approaching time to merge as I've overcome the main pain points.
There is a bunch more I want to do, but I would prefer to get these changes reviewed before embarking on anything else. Follow up plans:

Database migration handling
A github actions workflow that builds the container and publishes an attested build to GHCR.io
Improve the dockerfile in a number of ways

just have the prod build for image size
define a volume for the sqlite db

Highlights:
The board server now supports SQLite as a storage backend
You can control this behaviour (currently) with environment variables. We still default to firestore right now.
export STORAGE_BACKEND=sqlite
export SQLITE_DB_PATH=foo.db

The firestore and sqlite storage providers are separated out behind an interface, find them in src/server/storage-providers.
Error handling
The board server now responds 500 Internal Server Error on unhandled exceptions.
Board API integration tests
There's now a (admittedly, only happy path) integration test suite that puts the board API through it's paces in tests/integration/board-api.test.ts. Run this locally against SQLite with a new target:
npm run test:integration

A Dockerfile for the board server
Now that we have the option to run with SQLite, we can run the board server in a fully self contained docker image.
To build the container:
cd packages/board-server # if not already
docker build --build-context breadboard=../../ -t board-server .

To run the container:
docker run -d -p 3000:3000 board-server

@dglazkov @aomarks would appreciate your 👀 on this. I've added a minor bump changeset for the moment, as in the course of doing this I have started handling errors and returning 500 responses. If anyone was depending on that I feel sorry for them, but nonetheless..
It turned out to be a much bigger task than I first thought, so I'm expecting a few comments 😄

Taking a look now. I'll probably be able to have a review done by North American Tuesday.

Merging. If there's anything else we want to do we'll get it in a followup. Thanks Will!


Return an empty collection instead of null.

Some of the the resource metadata properties are collections/maps.
I think currently we are not consistent and in some cases we return null if there is no value (e.g.
BlobInfo and BucketInfo) and in some cases we return an empty collection (e.g. ProjectInfo).
My preference would be to never return null for a collection as I think the value of being able to
distinct the case of "unassigned" and "assigned and empty" is very small and does not justify
the cost for checking for null and empty.
In any case, I think we should be consistent.
/cc @mziccard @ajkannan @mderka

+1 to never returning null.  Even when users say "get" or "list" and leave out metatdata properties using field options, I think it's reasonable to have empty collections instead of null collections.

Agreed to never returning null.
However, with fields, specification, there is a difference between fields being empty and fields being null. Null means that I did not filter fields. Empty means that I specifically asked for no fields. I believe that services allow this. A reason for asking for no fields is for example to simulate an exist() function. I can want to shoot a request with empty fields just to see if I get 200 response meaning found, or null/404 for not founds.

While I agree that in general that we should be consistent and favor empty lists over null I would be careful when it comes to fields selection.
For instance in compute a disk has a list of attached instances. An empty list might mean that the disk is not attached to any instance. If the attachedInstances field is not selected through the DiskOption.fields() option returning an empty list might be counter-intuitive.
However, one could also argue that now we return null for both the "non instances attached" and "field not selected" cases.

I would actually prefer null to empty list in the scenario of attached disks described by @mziccard as the empty list has special and significant meaning.

@mderka Yes, I was regarding return values and therefore I think this issue should not affect the way we
currently handle field selection (using vararg). Did I get your comment wrong?
@mziccard You are raising an interesting case of an incomplete resource (due to field selection).
Which is similar (though not identical) to when constructing a source[Info] with partial fields.
However, as you mentioned, returning null still masks what I consider as a "user-error".
I agree with @mderka that null is more intuitive than an empty collection in that case but
it is still a user-error and can lead to undesired result.
We could, with more work, identify the case of accessing a field that is not populated and throw
IllegalStateException (which we should clearly document).
Thoughts?

@aozarov No, I actually understood @ajkannan's comment wrong. I take this back, all clear.
It probably is a user error. Can you be more specific about how to verifying unpopulated fields. Do you mean something like holding values in Optional<> (I guess Guava's since we are not in Java 8)?

I think there are various ways to achieve that but as it requires even more work than just 'null collection'` to 'empty collection' search and replace I want to make sure we are all on board with that.
One way of implementing that is to make the resource's builder do the bookkeeping of what was set.
This could be done by using Set (where field could be the same enum we use for the resource
field selection in the service interface). The builder adds to the Set per setter method.
When constructed the resource can use that set to check if exists per getter method.
We can also change the fromPb of the resource to accept that set and to pass it along to the builder.

My first choice is to throw an exception on getters for fields that aren't set and have an "isSet" method that people can use to check if a field wasn't set.
I don't think this would lead to too much more checking on the user's side because specifying fields is an optimization, and users who specify fields are likely to also know which fields they're selecting, and thus can avoid lots of isSet checks.  On the other hand, if they don't know whether a field is set for an object (perhaps they have some collection of objects obtained from different requests with different fields), they would have to check anyways for null/default values, and using the isSet method makes that scenario cleaner.

My first choice is to throw an exception on getters for fields that aren't set and have an "isSet" method that people can use to check if a field wasn't set.

This approach is sound and it's the right one if we want to go this way. My question is: is this worth it? Do we think that this approach is more usable than simple returning null for both cases (fields not selected and field not set)? I don't have a strong opinion but I would be careful in going this way as it will probably require a lot of changes.
Also XXXField enums we are using now match JSON fields but do not necessarily match a class field. This might be tricky to handle in some cases.

The suggested unified approach to returning empty collection instead of does not really make sense to me without IllegalStateException. It probably will be a lot of work but I cannot assess it in its entirety (I have only limited idea about the complexity of BigQuery, Storage and other modules).
I am not sure that I exactly follow the JSON issue. I think that these changes would be on the level of builders, constructors and toPb/fromPb methods, which is above JSON parsing. What am I missing?

I don't have a strong opinion but I would be careful in going this way as it will probably require a lot of changes.

I do think this would be a better API as it can potentially eliminate user error dealing with wrong data.
However I agree that if we decide not to do it (and I am OK with that) than we should be consistent (which means some change) and in that case I agree that null for unassigned collection property is more appropriate.
We should still make sure that any non-getter method that return collection should never
return null.

Also XXXField enums we are using now match JSON fields but do not necessarily match a class field. This might be tricky to handle in some cases.

Is there any reason for not calling the XXXField enum value as the field name (maybe there is a good
reason for that, but in general I don't think that is desired).
In any case, the enum is just one option (and could also be internal, providing hasXXX per field instead
of a generic hasField(xxx) though less desired).

Is there any reason for not calling the XXXField enum value as the field name (maybe there is a good
reason for that, but in general I don't think that is desired).

Consider the bigquery case where we have TableInfo which has a Definition field. Definition wraps some different JSON fields (e.g. schema, numBytes, numRows, streamingBuffer, etc.). With current XXXField enums the user can select only some of the JSON fields (e.g. only the schema) and the definition object will still be created with partial data. If we named XXXField enums after class's fields we would have a less fine granularity in field selection.

In any case, the enum is just one option (and could also be internal, providing hasXXX per field instead
of a generic hasField(xxx) though less desired).

Yes I agree this can be done (even still re-using XXXField somehow). I was just pointing out that is not as simple as one-to-one correspondence.

This seems like a design discussion that doesn't need to stick around as an issue any more; changes along the lines described here would likely be breaking changes for GA libraries. Closing it out.


[Question] The highlight's color was different from two window.

Thanks author very much for solute my problem.
Now I found same pattern of the same buffer was different with two window.
For example

The keyword 'formDataEditor' was highligh in two window. but the color was different.
For Hi/Find formDataEditor multiple files, the color was same, but why Hi/Set word color was different for two window?
Maybe I need some config to keep color same?

Thanks for your update. You can use Sync Mode  and :h HiSync option :)

Thanks for your update. Each window can have different highlighting patterns by default, and  you can use Sync Mode and the configurable option :h HiSyncMode.

It works, Thanks you


MM-1235: AbstractDataContext.searchColumn was fixed for situation when schema and column are equal strings

Fixes https://issues.apache.org/jira/browse/METAMODEL-1235.
Problem in AbstractDataContext.searchColumn when schema name and column name are equal was fixed.

Can you merge master into this branch and see if Travis then builds successfully?

I did, now it fails with oracle jdk8 only, in Excel module. This seems unrelated.

@jhorcicka We forgot to add the last merged PRs related issues to the CHANGES.md, so maybe in the context of this PR, you can also add this to the CHANGES.md:
### Apache MetaModel [WIP]

 * [METAMODEL-1236] - Elasticsearch: not/empty and is/not null do not work for text based columns
 * [METAMODEL-1235] - Elasticsearch: "String index out of range" when schema name and column name are equal
 * [METAMODEL-1232] - Improve stability of Travis CI builds


Parse IECEx and IECEE TRF identifiers separately

closes #77
closes #78
closes #21

Thank you @mico !


Added profile of team members and corrected some links.

Short Description

Added profile info of other team members.

Fixes Issue
Closes #53
Screenshots

Great job Jay,
Can we change Aash photo, Right now there are 2 photos of Mayank?

@AliReza1083  yup i can't kind photo of aashvi


Add concept information to initials in template models

This PR elaborates on the data model for initial values to also include the concept information in them

I think this looks good. We should check (1) how these new Initials are handled in the mira.modeling.Model class and (2) the REST API as input and return values. Further, we should update the SBML and any other input modes to create these objects, and finally look through examples/notebooks to see if any of those need updating.


Make Markers Draggable

Is there any way to make markers dragabble without bunch of hacks?

What u mean by 'hacks' keyword?
To implement drag on any system, all you need is 3 mouse events with coordinates, all of this exists in current map library. Read this.
https://github.com/istarkov/google-map-react/issues/59

What u mean by 'hacks' keyword?
To implement drag on any system, all you need is 3 mouse events with coordinates, all of this exists in current map library. Read this.
https://github.com/istarkov/google-map-react/issues/59

Thank you for the response, I will check this out , If I have time will update the documentation.
I have a doubt, I am using redux for state handling like your examples,
I save center, zoom, bounds in the store and use it(center,zoom) as props to map component, , I also have a search box with auto complete, when a place is clicked, I get the center and zoom ( if view port is returned by google maps api) and set them in reducers, This in turn triggers a onChange in component leading to another action updating bounds. I want to avoid that because I think bounds is a derived data?
ie Can bounds be derived from center and zoom [and maybe window height and width]? So that I can avoid storing it in store?

Thank you for the response, I will check this out , If I have time will update the documentation.
I have a doubt, I am using redux for state handling like your examples,
I save center, zoom, bounds in the store and use it(center,zoom) as props to map component, , I also have a search box with auto complete, when a place is clicked, I get the center and zoom ( if view port is returned by google maps api) and set them in reducers, This in turn triggers a onChange in component leading to another action updating bounds. I want to avoid that because I think bounds is a derived data?
ie Can bounds be derived from center and zoom [and maybe window height and width]? So that I can avoid storing it in store?

@istarkov  Hi, the mouse events work great, can you tell me how to do the same using touch events ? Or  can you give any direction on how to implement touch event, I will give a PR if I do it..

It is not a great but nor me nor my friends have any touch devices - it's near impossible for me to implement touch capabilities.

@istarkov Ok, I will try to use react tap event plugin and add  tap actions, I can test it in a couple of devices.


Get rid of border-radius in fullscreen mode (macOS) fixes #21

Tried this with and without the border, and it looks better with a border.

It does remove the radius all the time in the CSS, however OS X puts rounded borders on it without the radius:

It's got a couple disadvantages though

not consistent across OSs
to the observant, the corners lose their border right around their edges

I can see the benefit of having a consistent way in the CSS to detect full screen mode. Any objection to adding that to the body tag?
<body class='fullscreen'>

I actually like the border in fullscreen.

The community has spoken. Everyone seems to like the border in fullscreen :D


Exception with MeasureTextAsync

Hello ,
i found a problem when trying to use the method MeasureTextAsync with string arg that contains single quote into it
exemple : with this string arg : "L'ouverture du magasin"
it seems that the problem is related to escape chars in javascript (JS interop).
with the sample string arg i have this output exception in browser console :

Do you have any ideas how to fix that ?
Many thx,
Regard,  Valentin

Hi @PascalFarled,
thanks for raising this issue. It is indeed a bug, which I just fixed in version 2.6.5.
BR
Stefan


[Snyk] Upgrade com.networknt:json-schema-validator from 1.5.0 to 1.5.2

Snyk has created this PR to upgrade com.networknt:json-schema-validator from 1.5.0 to 1.5.2.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 2 versions ahead of your current version.

The recommended version was released on 22 days ago.

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Fails

:no_entry_sign:

PR Title '[Snyk] Upgrade com.networknt:json-schema-validator from 1.5.0 to 1.5.2': subject may not be empty, type may not be empty

Warnings

:warning:

Commit Message 'fix: upgrade com.networknt:json-schema-validator from 1.5.0 to 1.5.2
Snyk has created this PR to upgrade com.networknt:json-schema-validator from 1.5.0 to 1.5.2.
See this package in maven:
com.networknt:json-schema-validator
See this project in Snyk:
https://app.snyk.io/org/connectors-0mk/project/b03c00c7-cefb-40dd-beaf-5a52a34cfede?utm_source=github&utm_medium=referral&page=upgrade-pr': body's lines must not be longer than 100 characters

Pull request title should follow Conventional commits.

  Generated by :no_entry_sign: dangerJS against a6cbd96232e3ec901cbec7872dabda1b9486b35a


Links in the readme to open in a new tab

Prerequisites
Please answer the following questions for yourself before submitting an issue. YOU MAY DELETE THE PREREQUISITES SECTION.

[x] I am running the latest version
[x] I checked the documentation and found no answer
[x] I checked to make sure that this issue has not already been filed
[x] I'm reporting the issue to the correct repository (for multi-repository projects)

Expected Behavior
Links in the readme to open in a new tab so that it is easier for the user to navigate to GitHub codebase.

#143 Solved


Add share button on my proposal page

Related with issue #163 (number 2).

Can you add a print screen, @anapaulagomes?
<3

Sure! :)

Hey @anapaulagomes can you centralize vertically the modal? Besides that, everything is good to go!

Hey @alessandroHenrique ! I'm sorry for taking so long to get back to you. Looks like the modal is already centered but AddThis adds a little space on the bottom.

I'm not good with frontend stuff. Can you help me with it?


blt vm issue

`
Issue Type

Bug Report / Support Request

Your Environment

Vagrant 1.9.1
VirtualBox 5.1.10r112026
ansible <IP_ADDRESS>
  config file =
  configured module search path = Default w/o overrides

Your OS

macOS (version)

Full console output

https://gist.github.com/nirojghale-iheartmedia/b0fdee5c7985d42f91882712bc3a71ef
Summary

I'm presuming this was the original issue: https://github.com/acquia/blt/issues/1020
In which case, the following addition to config.yml solved the issue:
selenium_install_firefox: yes
selenium_install_chrome: no


chunk/store: add default upload/download limit for each volume

Codecov Report
Patch coverage has no change and project coverage change: -31.69 :warning:

Comparison is base (ad6ca41) 56.18% compared to head (e93ffba) 24.49%.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #3574       +/-   ##
===========================================
- Coverage   56.18%   24.49%   -31.69%
===========================================
  Files         149       24      -125
  Lines       36817    14392    -22425
===========================================
- Hits        20684     3525    -17159
+ Misses      13864    10483     -3381
+ Partials     2269      384     -1885

Impacted Files
Coverage Δ

pkg/meta/config.go
10.41% <ø> (-38.20%)
:arrow_down:

... and 145 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


how to install Runtime library as mt

I want to use DLL with MT runtime

Hi @libxengine
Thanks for posting this issue.
I think you can add a custom triplet, such as x64-windows-mt.cmake like this:
set(VCPKG_TARGET_ARCHITECTURE x64)
set(VCPKG_CRT_LINKAGE static)
set(VCPKG_LIBRARY_LINKAGE dynamic)

thanks for you help

i am create x86-windows-mt.cmake file
set(VCPKG_TARGET_ARCHITECTURE x86)
set(VCPKG_CRT_LINKAGE static)
set(VCPKG_LIBRARY_LINKAGE dynamic)
.\vcpkg.exe install curl[openssl]:x86-windows-mt
libcurl is mt runtime but it is dependent library is not Mt....why...

Since you used a custom triplet, we cannot make sure all ports build successfully. As for the dependencies, whether they support the specific triplets is related with the ports themselves. If it is a bug for the port, we can post the detailed errors out in vcpkg or report to upstream if this is intended behavior from upstream.

o


Fix bug where we write a broadcast with the wrong broadcast ID

Before we were writing the broadcast ID based on the broadcast ID
contained in the ScheduleUpdate which was delivered via messaging.
However when we resolve the content currently in the DB for that
broadcast we also try to match based on channel ID and transmission
interval. That meant that if we had a broadcast that changed IDs
since the schedule update message was generated we would end up
writing that broadcast with the wrong broadcast ID clustering key.
If that broadcast then became stale we would never be able to
delete it as we would be trying to delete the wrong ID

Yeah, you are right. I'm adding one


problem receiving requests from domains without subdomains

Hello,
I'm having issues with requests without subdomains.
Example:
Route 53 is registered like: foo.com
Any request to foo.com will not work, just requests coming from *.foo.com (e.g., www.foo.com)
It says the page is insecure because my alb domain (that I created with your terraform plan) is different from *.foo.com
There is any workaround for this? (:

Your certificate seems only for *.foo.com and it says insecure. You can get a certificate for both *.foo.com and foo.com by ACM.
If Terraform could not find the certificate, change the domain to that without *. as follows (probably work):
data "aws_acm_certificate" "service" {
  domain = "${var.alb_external_domain_name}"
}


FIX: Expose _set_3d_view and _set_3d_title

This PR exposes the functions _set_3d_view() and _set_3d_title() to the notebook 3d backend.
Related to https://github.com/mne-tools/mne-python/issues/8388

Let me know if this solves your issue @timonmerk

This is ready for reviews @agramfort, @larsoner

Can you modify the notebook test such that it has better coverage?

@GuillaumeFavelier do we need to backport this?

... it seemed to work without issue so I backported


Add Grafana chart for SyncStatus

After merging https://github.com/ChainSafe/lodestar/pull/2464 add chart with https://github.com/grafana/grafana/pull/12609
Originally posted by @dapplion in https://github.com/ChainSafe/lodestar/issues/2460#issuecomment-830814884
https://github.com/ChainSafe/lodestar/blob/8ab5df0dcb27b586a7783b18f976a5bec6de2632/packages/lodestar/src/sync/interface.ts#L34-L39

Done


chore(main): release 1.3.0

:robot: I have created a release beep boop
1.3.0 (2024-09-09)
Features

add will-change css style (f9a83f6)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.3.0
:sunflower:


PKIsigning platform (certified connector)

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)

[x] I attest that the connector works and I verified by deploying and testing all the operations.
[x] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[x] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[x] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[x] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[ ] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[ ] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[ ] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[ ] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

Hi @pkisigning! Our certification engineer, Srikanth, is currently on emergency leave. Please expect a delay with your PR. He will return January 31st, 2022. We apologize for any inconveniences.

With all the respect to Srikanth, January 31st is more than two weeks ago now. Could you tell me anything about how much more delay we should be expecting? (@natalie-pienkowska) I would presume a platform like this to be operated by more than one certification engineer, or there should be some kind of procedure to back someone’s tasks when a person is unavailable for some time. It just doesn’t feel Microsoft alike to have radio silence for so long.

@pkisigning Thank you for reaching out. Apologies for the delay in response. @sriyen-msft our Primary Engineer will review your request ASAP. Please expect to hear back from us very soon. Thank you for your patience.


Added toArray()

Code Coverage Summary [using JaCoCo]

Coverage Type
Coverage

Coverage:
16.75%

Branches:
18.08%

Code Coverage Summary [using JaCoCo]

Coverage Type
Coverage

Coverage:
16.74%

Branches:
18.08%


Pixi 0.8 doesn't work with editable installations

Checks

[X] I have checked that this issue has not already been reported.

[X] I have confirmed this bug exists on the latest version of pixi, using pixi --version.

Reproducible example
When using pixi, I typically add an editable installation of my current Python package.
This looks something like this:
[project]
name = "pixi-0.8"
version = "0.1.0"

channels = ["conda-forge"]
platforms = ["win-64", "linux-64", "osx-arm64", "osx-64"]

[tasks]
install = "pip install --editable python"
import = "python -c 'import test_package'"

[dependencies]
python = "3.12.0.*"
pip = "23.3.1.*"

Starting from pixi 0.8, running pixi run install and then pixi run import gives this:
✨ Pixi task: python -c 'import test_package'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ModuleNotFoundError: No module named 'test_package'

You can find the complete example here: https://github.com/Hofer-Julian/pixi-0.8
According to our CI, it should happen on Windows, macOS and Linux.
Issue description
While pixi didn't support installation via pip, it was possible to install packages with pixi tasks.
Editable installations are broken since 0.8.
It seems like "normal", non-editable installations still work as expected.
Expected behavior
Ideally editable install would be directly supported by pixi. Alternatively, it would be great if one can at least install packages via pixi tasks.

Thanks for the example and letting us know, we're on it!

We think the issue is that we "uninstall" every package that our lock file doesn't specify as needed. This might be an easy fix. @baszalmstra is on it!