perf(api): Pipette function update

This is a base for PR containing the changes in #3012 . It should serve as a WIP platform for implementing the functions contained here as detailed in that ticket. The values of the aspirate functions, however, are the output of hardware team testing and should be trusted.
Do not merge this PR until it has been changed to be under a feature flag.
Closes #3012

Superceded by #3014 which was rebased to remove some redundant commits and clean the history.


Shortcut 'tn' (schedule next weekend) doesn't work on Chrome/MacOS

I am using Todoist Shortcuts v. 143 on Chrome 99.0.4844.51 on a Macbook. When I visit https://todoist.com/app/today I can use 'n', 'p' to navigate and 'tt' to move a task to tomorrow. But strangely 'tn' which should move the task to the next weekend (according to online help) does nothing. What could be the reason?

Hmm, curious, I don't see the buttons either.  todoist-shortcuts just
presses existing buttons (and currently doesn't attempt to understand or
select items on the calendar).  My guess is that Todoist removed the
feature.
Alternatively, it could be that they are only visible on certain days of
the week -
https://todoist.com/help/articles/whats-new#sizzling-summer-aug-23 says

Since life happens on the weekends too, we’ve made scheduling or
postponing a task in one click a little easier with brand-new scheduler
shortcuts: this weekend or next weekend. Between Monday and Wednesday,
you’ll also see a new shortcut to schedule a task for later this week.

On Mon, Mar 7, 2022 at 3:21 AM chalstrick @.***> wrote:

I am using Todoist Shortcuts v. 143 on Chrome 99.0.4844.51 on a Macbook.
When I visit https://todoist.com/app/today I can use 'n', 'p' to navigate
and 'tt' to move a task to tomorrow. But strangely 'tn' which should move
the task to the next weekend (according to online help) does nothing. What
could be the reason?
—
Reply to this email directly, view it on GitHub
https://github.com/mgsloan/todoist-shortcuts/issues/205, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAFQKTJAWYGJH463DYSNILU6XKDZANCNFSM5QC2447A
.
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.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hmm, curious, I don't see the buttons either.  todoist-shortcuts just
presses existing buttons (and currently doesn't attempt to understand or
select items on the calendar).  My guess is that Todoist removed the
feature.
Alternatively, it could be that they are only visible on certain days of
the week -
https://todoist.com/help/articles/whats-new#sizzling-summer-aug-23 says

Since life happens on the weekends too, we’ve made scheduling or
postponing a task in one click a little easier with brand-new scheduler
shortcuts: this weekend or next weekend. Between Monday and Wednesday,
you’ll also see a new shortcut to schedule a task for later this week.


The quickstart is not compatible with the version ^1.0.0-beta.7

Hi all,
It seems that the example is not compatible with beta 7. for example
1-  the menu button is not showing (as in your documentation we must add   the following which is not present in the quickstart.
<button md-icon-button [tdLayoutToggle]="true"> menu
).
2- The logo is not aligned so i needed to add the folowwin in my css to align it :
.mat-icon.md-icon-logo {
height: auto;
width: 100px
}
Can you please cofirm me if the quickstart is compatible or not?
Thankyou in advance

Sorry about that, we're planning these Quickstart updates after the beta.8 release later today, with clear commits for clear upgrade paths:

beta.5 - beta.6
beta.6 - beta.7
beta.7 - beta.8
refactor and simplify Quickstart

Hi All ,
In this time Covalent 1.0.0-rc.0  is released.   Either wait or fork the rep, update and submit a pr.

Not valid anymore since we are repurposing the repo.


s3: refactor New() logic and hide internal dir layout

We have expose dir layout in e2e test code.
We need to refactor existing translate layer code and hide it.

This is something we should have done in https://github.com/coreos/etcd-operator/pull/1049 .
Fixing it as a hindsight.

fixed

fixed

lgtm


Intranet webpages not working

Is there a way to cast intranet webpages via Dashcast?
Currently it currently does not work if the URL is not seen publicly online.

It should work fine, as long as the chromecast is connected to the same network as the intranet page.
If the chromecast is connected to the internet, then you're probably out-of-luck, as putting an intranet page onto the public internet is going to be a security issue for most people.

I figured out the issue.
The dashcast or the Chromecast whichever, uses an online DNS and as a result can not find the internal network domain that our intranet uses. It works fine if I tell dashcast to connect to the internal IP but not via the internal network domain name.
Is it possible for future builds of dashcast to use the internal network DNS instead?

i don't think so because the dns resolution depends on the chromecast not on the app which this app is running.

Chromecast always uses google dns, so if you have endpoints that only resolve locally, the chromecast won't be able to resolve them, because the upstream google dns servers do not know about your internal dns records.
This works fine if you have setup your firewall to redirect all dns traffic to your dns servers. I have tested this in my internal network and can confirm this is working.
The moment I remove the iptables rule it stops working, which is expected, considering the chromecast is querying a different dns server.
There were some early versions of the initial chromecast that were jailbroken (this was patched early on) and allowed you to have some control over the system configuration, but the most reliable option you have to enforce this is really using your firewall to ensure the traffic goes where you want.
This issue should probably be closed, since it is not really fixable by dashcast itself.


PROD-314 fixed spacing

✅ Add Linear ID (e.g., PROD-123) to PR title to automatically link issue

Description
Link to the task https://linear.app/gator/issue/PROD-314/fix-spacing-between-leaderboards-and-deck-counts
What are the steps to test that this code is working?
Go to Campaign's details page . Spacing should be as it is in the design
Screen shots or recordings for UI changes

@Yihwan please review


DownloadPackage@1 should have an option to download a specific maven snapshot version

Question, Bug, or Feature?  Feature
Enter Task Name: DownloadPackage@1
Issue Description
Using DownloadPackage@1 task to download Maven snapshot versions, it would be great to be able to only download a specific snapshot version. Currently all snapshots are downloaded.

It is currently not desirable to add package type specific behavior to a generic DownloadPackage task. As a workaround, a MavenAuthenticate  task should be used followed by a maven script task to download a desired snapshot version.

+1 to not adding more to DownloadPackage task. Imo DownloadPackage has a few deficiencies in downloading that are unlikely to be fixed (have seen issues w/ corrupt downloads). It should be used to trigger a pipeline, but not to download artifacts

We do not plan to support downloading specific snapshots of Maven packages. Please see @satbai comment above.

It is currently not desirable to add package type specific behavior to a generic DownloadPackage task. As a workaround, a MavenAuthenticate  task should be used followed by a maven script task to download a desired snapshot version.

@samuelkoppes @satbai We currently run in an issue that space is running out on a vm because 25 snap versions of a multi package project are being downloaded by the DownloadPackage@1 task. Can you give more inside how to work around this issue?


SITL/ROS - No position lock.

Hey guys,
i want to use the firmware as SITL with ROS. Gazebo is running, the Quad is spawning. But i cannot takeoff. arming is working and rotors are spinning.
WARN  [commander] rejecting takeoff, no position lock yet. Please retry..
This is happening with:
make posix gazebo_iris_opt_flow
roslaunch px4 mavros_posix_sitl.launch -> vehicle: "iris" and "iris_opt_flow"
hope anybody can explain this behaviour to me

The camera is out of focus on the ground so you can't get a position lock and takeoff. Please takeoff in manual first then switch to auto.

With iris it should be working since that uses gps and that is constantly tested in CI.

Auto takeoff now supported with flow.

Hi TobiMiller
have yo solved your problem?
I am encountering your problem too.
if you have solved it, please explain how.
you can.
thanks


Auto-installing failed on Windows and MacOS

I tried run in github actions.
https://github.com/cawa-93/dd0/blob/f477b9721249ed1fa96d3687b28c22882ecccaba/.github/workflows/lint.yml#L33-L35
But it works only on Linux. Auto-installing on Windows (in git-bash) and macos was failed.
https://github.com/cawa-93/dd0/runs/5569342162?check_suite_focus=true
Including locally on my PC.
$ source <(curl https://raw.githubusercontent.com/Arriven/db1000n/main/install.sh)
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  1770  100  1770    0     0   3682      0 --:--:-- --:--:-- --:--:--  3679
Downloading an archive...
Checking md5 hash...
md5sum: db1000n-v0.7.11-windows-amd64.tar.gz: No such file or directory

As far as I can see the archive is downloaded but not unpacked

@cawa-93 it's already fixed by replacing md5sum->shasum: https://github.com/Arriven/db1000n/commit/8dae06b02395e2771832f903da4b7615111d0490

@Amet13 Still not working, but now for other reasons https://github.com/cawa-93/dd0/runs/5783638850?check_suite_focus=true
Run source <(curl https://raw.githubusercontent.com/Arriven/db1000n/main/install.sh)
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100  1979  100  1979    0     0   5514      0 --:--:-- --:--:-- --:--:--  5527
Downloading an archive...
Downloading checksums...
Checking sha256 hash...
tar: This does not look like a tar archive
tar: Skipping to next header
tar: Exiting with failure status due to previous errors
Error: Process completed with exit code 2.

Not sure that this commit https://github.com/Arriven/db1000n/pull/212/files was tested properly.
Could you try to use the version with pipe? https://github.com/Arriven/db1000n/pull/212/files#diff-0f4ecb15684766771582d99a397733f3ea15493458ad6f925a9201dea19ffadfL35


Add transcript filter functionality to entity viewer

Type

[ ] Bug fix
[x] New feature
[ ] Improvement to existing feature

Related JIRA Issue(s)
https://www.ebi.ac.uk/panda/jira/browse/ENSWBSITES-888
Importance
N/A
Description
This is for the filtering functionality in Entity viewer.  The component was already implemented together with the test script. This PR is for adding the functionality to filter the transcripts. When you click on one of the filters, it will only show the transcripts with that filter.
For now we are only doing the filter by so_term category, there might be more in the future and we can make the code more generic then as we know more about design and requirements.
Deployment URL
http://entityviewer-filter.review.ensembl.org/
Views affected
Entity Viewer (Transcript Image)
Other effects
None

[ ] I have checked any requirements for Google Analytics
[ ] I have created any required tests
[ ] The PR has as its final commit a WASM/JS update commit if any Rust files were updated.

Possible complications
None

Umm, if our goal is to create a set of filters that can eventually cover various properties of a transcript, we will need to be smarter than our original implementation of the filters state:

The way the filters state in redux looks now:
https://github.com/Ensembl/ensembl-client/blob/dev/src/ensembl/src/content/app/entity-viewer/state/gene-view/transcripts/geneViewTranscriptsSlice.ts#L55
will work for filtering by a single attribute, such as the so_term, but is unlikely to scale to the point when we start filtering by different attributes (biotype and appris and tsl). I wonder whether it might be better to take a pause and figure out a way for filtering by several attributes now rather, because we already know that we will need that, than to refactor/rewrite this code later.

@azangru i have explained the reason for doing just this category in the description. I think if we have something for Andrea to have a look at, she will have a better feeling of how things are working. I remember she or steve said we might just have SO Term for MAP. Probably the code will need refactoring at that time. For now if we have something for Andrea/Steve to play with they will have a better understanding what else to add.

I'm not saying we should postpone the filters until TSLs and APRIS become available. I'm saying we should probably build the filters with the anticipation of multiple filter categories, such as TSLs and APRIS, which we expect will be coming. As the code is written now, it doesn't seem to offer a way for adding new filter categories without a rewrite. That's a technical debt.

I'm not saying we should postpone the filters until TSLs and APRIS become available. I'm saying we should probably build the filters with the anticipation of multiple filter categories, such as TSLs and APRIS, which we expect will be coming. As the code is written now, it doesn't seem to offer a way for adding new filter categories without a rewrite. That's a technical debt.

There are distinct JIRAs for adding content as it becomes available eg 1017. What level of technical debt we are willing to accept now is a decision we need to make.
Re biotypes specifically, we will use them when we have them. Are they essential for MAP ? Prob not.

Sorry i dont agree it will be a technical debt as we dont know yet for sure how this will pan out. Not sure if you missed something but we dont know yet what the other filters will be or if there will be other categories and how will they work with the sorting. We might have to rewrite the whole thing again if lets say the sorting change. I think doing more on it than required will be the opposite of technical debt maybe technical  overpayment :)
What are the changes are you suggesting to make, i thought it was generic enough already. Maybe something for code review, will be good to get Andrea as well.

Probably best lets see what Andrea said in the code review and then if we could merge this one and after that i will create another ticket to make the code able to handle more filters. This will also help me understand the code (its easier to learn the code when you see exactly whats happening)

ENSWBSITES-1017 being in the backlog should probably point us in the direction of building the filters as if we were building them for ENSWBSITES-1017 — writing code in a way that will handle multiple filter criteria and using just a single so_term filter criterion which we'll then replace with biotype when it becomes available.

Not sure if you missed something but we dont know yet what the other filters will be.

I'm confused then. How confident are we in ENSWBSITES-1017 then?

What are the changes are you suggesting to make

There are several indications that filters want to be grouped.
One indication comes from the UI, where three groups of filters are separated into three different columns:

The other indication comes from the filter function, which knows which property of a transcript to check (line 13 on screenshot below):

Currently, the filters in redux aren't being grouped at all.

i agree with the grouping with you @azangru , let me pass this by andrea in the code review or earlier if i can (probably best in code review) and then we will understand better the grouping and what to do with the code. Question to ask andrea, will there be a grouping and how will it be?

The associated EA-695 ticket describes what data we are expecting. The bottom 3 are not filters

For future reference: the majority opinion during the code review was not to bother about the support for more than one kind of filter now. Which means that we are postponing a more scalable implementation of transcript filters until later, and the moment we add another filter we'll need to rework, from ground up, the way we are setting up the filters, selecting them from the state, and choosing the appropriate filtering function to apply to transcripts.

@azangru thanks for the PR feedback, i have pushed the changes. Note that i have valid so_term in the test cases just so that it make sense when reading it but i do agree that we can put any string instead.

For future reference: the majority opinion during the code review was not to bother about the support for more than one kind of filter now. Which means that we are postponing a more scalable implementation of transcript filters until later, and the moment we add another filter we'll need to rework, from ground up, the way we are setting up the filters, selecting them from the state, and choosing the appropriate filtering function to apply to transcripts.

Just to add because there is too many uncertainties regarding how the filters will pan out (we don't know where the filters will be applied, what groupings we will have,....), its better we keep making the code scalable as we know more about the filters instead of having to rewrite/change them later. I will like to carry on the work on making it scalable, another opportunity to understand react, redux and testing more.


[mc_rtc] Add a way to keep track of the source of a log entryTopic/simplified logging

This PR adds a source (pointer to the element that registered a log entry) for every log entry in the logger. The goal is to hugely simplify book-keeping when adding log entries
This is done by adding a source parameter in addLogEntry, i.e.
// Usual way, no source registered, can only be removed by removeLogEntry
logger.addLogEntry("bla", []() { return bla; });
// New way with a source
logger.addLogEntry("bla", source, []() { return bla; });

This allows removing a bulk of log entries at once as such:
logger.removeLogEntries(source);

Furthermore, I have added an helper to register member or method logging, such as:
// Assuming MyClass::vector_ is e.g. an Eigen::Vector3d member
logger.addLogEntry<decltype(&MyClass::vector_), &MyClass::vector>("my_vector", this);
// Also work with a method MyClass::computeStuff()
logger.addLogEntry<decltype(&MyClass::computeStuff), &MyClass::computeStuff>("my_stuff", this);

For methods there is two extra caveats:

only const method are accepted (arbitrarily, there is no technical limitation)
overloaded methods (e.g. getter/setter with the same name) cannot be used so one has to fall back to either writing a "classic" addLogEntry or spell-out the method pointer type

This is strictly equivalent to:
logger.addLogEntry("my_vector", this, [this]() -> const Eigen::Vector3d & { return vector_; });
logger.addLogEntry("my_stuff", this, [this]() { return computeStuff(); });

But you don't have to remember to return a reference (the same is true for the method version).
The syntax is not super nice, so there is a macro to help writing such entries:
// logger is the logger instance in this context, can be ctl.logger() in a FSM state for example
MC_RTC_LOG_HELPER(logger, "member", this, &MyClass::member);
// Same as
logger.addLogEntry<decltype(&MyClass::member), &MyClass::member>("member", this);

With C++17, it will be possible to simply write
logger.addLogEntry<&MyClass::member>("member", this);

Also done/to be done:

[x] Use simplified remove in tasks, observers and states
[x] Removed redundant virtual ... override in mc_tasks (since I was touching pretty much all tasks header)
[x] Added an extensive test of the logger
[x] Fix minor bug in EncoderObserver
[ ] Update the documentation accordingly

Thanks for the feedback, I went with implementing a macro like this:
MC_RTC_LOG_HELPER(name, member);
// Plus the following helper when the method name is ambiguous
MC_RTC_LOG_GETTER(name, getter);

This always uses this as the source and logger as the logger.

While we're simplifying the logging, I think one nice addition would be to be able to chain together the calls to addLogEntry similarely to what we do for the gui. e.g something like
logger.addLogEntries(this,
  {"entry_1", []() { return ...; }},
  {"entry_2", []() { return ...; }},
  ...
  {"entry_N", []() { return ...; }}
);```

Good point. Implemented as:
logger.addLogEntries(this,
  "entry_1", []() { return ...; },
  "entry_2", []() { return ...; },
  ...
  "entry_N", []() { return ...; }
);

Note that this only supports "regular" callbacks

Thanks @gergondet,
Looks good to me now, LGTM :+1:


fix issue #25

I was able to fix this issue by changing the string to a list as expected by the protocol

Thanks for the PR.
It has been cherry-picked into the develop branch, you can safely delete your branch.

Actually the server messages the following btw:
"These dependencies with correct versions need to be installed or upgraded: AST-0.15, Lint-0.15, otherwise the server may malfunctio, even though I have version 0.15.1 installed of both of them. But it seems to work fine anyway

You seem to be on master, Feel free to checkout develop if you wanna suppress the warning for the time being.

:+1:


Add an interactive ChatBot

Hello Sir,
It will be very beneficial if we added an interactive chatbot in the website, so that user can easily ask any questions regarding the website.
I kindly request u to assign this issue to me for Hacktoberfest 2024!!

Thanks for creating the issue in WildGuard! The issue has been assigned to you.
Before you start working on your PR, please make sure to:

⭐ Star the repository if you haven't already.
Pull the latest changes to avoid any merge conflicts.
Attach before & after screenshots in your PR for clarity.
Include the issue number in your PR description for better tracking.

Don't forget to follow @anuragverma108 – Project Admin & @PriyaGhosal-Project Collaborator  – for more updates!
We're excited to see your contribution as part of Hacktoberfest 2024! 🎉
Happy open-source contributing!


Need to recognize character references in attributes in XML literals

For example:
import ballerina/io;

public function main() returns error? {
   var x = xml`<p att="x&y"/>`;
   string att = check x.att;
   io:println("Expected 3 Got: ", att.length());
}

It does not seem to be parsing character references at all.  For example:
io:println(xml`<p att="x&y"/>`.toString());

prints
<p att="x&amp;y"/>

It seems the XML parser does not perform Attribute-Value Normalization[1]
[1] - https://www.w3.org/TR/2006/REC-xml11-20060816/#AVNormalize

This is fairly critical to XML support.

@jclark do we need to normalize the interpolated parts too?
    var x = xml`<p att="a${"\u{D}"}b"/>`;

No.

Ack. Thanks.


Add type annotations for all functions

I've written type annotations for every function in the epic module. This allows you to run mypy and type-check the entire codebase. I also fixed a few minor bugs that were revealed by the type checker.

Thanks. Never used type annotations in Python before. Look forward to
reading PEP484.
There are some typos in your code (typimg), which I guess is why the
continuous integration fails?
It seems like type annotations do not work for 2.7 though. Perhaps they can
be added in a way compatible with 2.7 like shown here:
http://mypy.readthedocs.io/en/latest/python2.html
If you fix that and type annotations does not break anything major, I'll
merge the PR.
Endre
On Sunday, November 13, 2016, Ryan C. Thompson<EMAIL_ADDRESS>wrote:

I've written type annotations for every function in the epic module. This
allows you to run mypy and type-check the entire codebase. I also fixed a
few minor bugs that were revealed by the type checker.
You can view, comment on, or merge this pull request online at:
https://github.com/endrebak/epic/pull/47
Commit Summary

Add mypy type annotations
Add missing argument to get_island_bins

File Changes

M epic/bigwig/create_bigwigs.py
https://github.com/endrebak/epic/pull/47/files#diff-0 (14)
M epic/config/genomes.py
https://github.com/endrebak/epic/pull/47/files#diff-1 (13)
M epic/matrixes/matrixes.py
https://github.com/endrebak/epic/pull/47/files#diff-2 (42)
M epic/run/run_epic.py
https://github.com/endrebak/epic/pull/47/files#diff-3 (17)
M epic/scripts/effective_genome_size.py
https://github.com/endrebak/epic/pull/47/files#diff-4 (3)
M epic/scripts/overlaps/files_to_chromosome_coverage.py
https://github.com/endrebak/epic/pull/47/files#diff-5 (13)
M epic/scripts/overlaps/nucleotide_bargraph.py
https://github.com/endrebak/epic/pull/47/files#diff-6 (23)
M epic/scripts/overlaps/nucleotides_heatmap.py
https://github.com/endrebak/epic/pull/47/files#diff-7 (10)
M epic/scripts/overlaps/overlaps.py
https://github.com/endrebak/epic/pull/47/files#diff-8 (9)
M epic/statistics/add_to_island_expectations.py
https://github.com/endrebak/epic/pull/47/files#diff-9 (10)
M epic/statistics/compute_background_probabilites.py
https://github.com/endrebak/epic/pull/47/files#diff-10 (3)
M epic/statistics/compute_poisson.py
https://github.com/endrebak/epic/pull/47/files#diff-11 (4)
M epic/statistics/compute_score_threshold.py
https://github.com/endrebak/epic/pull/47/files#diff-12 (13)
M epic/statistics/compute_values_needed_for_recurrence.py
https://github.com/endrebak/epic/pull/47/files#diff-13 (21)
M epic/statistics/compute_window_score.py
https://github.com/endrebak/epic/pull/47/files#diff-14 (2)
M epic/statistics/count_to_pvalue.py
https://github.com/endrebak/epic/pull/47/files#diff-15 (17)
M epic/statistics/fdr.py
https://github.com/endrebak/epic/pull/47/files#diff-16 (7)
M epic/statistics/generate_cumulative_distribution.py
https://github.com/endrebak/epic/pull/47/files#diff-17 (7)
M epic/utils/find_readlength.py
https://github.com/endrebak/epic/pull/47/files#diff-18 (5)
M epic/utils/helper_functions.py
https://github.com/endrebak/epic/pull/47/files#diff-19 (44)
M epic/utils/separate_input_and_chip_infiles.py
https://github.com/endrebak/epic/pull/47/files#diff-20 (6)
M epic/windows/cluster/find_islands.py
https://github.com/endrebak/epic/pull/47/files#diff-21 (11)
M epic/windows/count/count_reads_in_windows.py
https://github.com/endrebak/epic/pull/47/files#diff-22 (21)
M epic/windows/count/merge_chromosome_dfs.py
https://github.com/endrebak/epic/pull/47/files#diff-23 (6)
M epic/windows/count/remove_out_of_bounds_bins.py
https://github.com/endrebak/epic/pull/47/files#diff-24 (8)
A mypy.ini https://github.com/endrebak/epic/pull/47/files#diff-25
(7)

Patch Links:

https://github.com/endrebak/epic/pull/47.patch
https://github.com/endrebak/epic/pull/47.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/endrebak/epic/pull/47, or mute the thread
https://github.com/notifications/unsubscribe-auth/AQ9I0j_FUojhQ5jGJM8-CHgmrPiIxig_ks5q9t5ZgaJpZM4KwpzU
.

Coverage decreased (-0.5%) to 55.266% when pulling 0b7b8fac0d8ee01cb3fb0525916b9f256139674a on DarwinAwardWinner:mypy into 952bd54d39e0d8b87290f69bf6f7b061edbc18fa on endrebak:master.

Coverage increased (+0.5%) to 56.243% when pulling a869f66ba528add32b691cc430294c78288b1aee on DarwinAwardWinner:mypy into 952bd54d39e0d8b87290f69bf6f7b061edbc18fa on endrebak:master.

OK, I think I've got everything in order. Let me know if you want me to squash some of the fixup commits before you merge.

Will look closer at this within a few days and before making a new commit
to the project. I am a bit swamped in the beginning of the week. Thanks btw!
On Sun, Nov 13, 2016 at 7:17 PM, Ryan C. Thompson<EMAIL_ADDRESS>wrote:

OK, I think I've got everything in order. Let me know if you want me to
squash some of the fixup commits before you merge.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/endrebak/epic/pull/47#issuecomment-260202280, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AQ9I0iHOUxM7C2EJJ4rAgpbT8xAB1Jfvks5q91RCgaJpZM4KwpzU
.

Since the tests passed, and the 2.7 version of annotations is really just comments, I merged your PR. Thanks. Added you to the contrib list.


Charts collapse onto themselves when element is resized

I am submitting a...

[ ] Feature request
[x] Design defect
[ ] Source code defect
[x] Demo/documentation defect
[ ] Other

charts version: Demo version
Issue description
Charts fold in on themselves as they don't have any minimum height requirements when resizing in a resizable element
Steps to produce the issue

Go to the demos page and shrink the size of any chart container element vertically using the bottom right hand corner

Current behavior
Charts collapse in on themselves
Expected behavior
Chart has a minimum height to avoid the above
Screenshot or recording

Yes thank you @brocococonut for this. This functionality exists in the pie/donut chart components.
@dianatran18 Any thoughts?


Update for v2.2207

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

Hi @mxr-conda @ndmaxar @oblute @rluria14, I think this is ready to go -- it's now building successfully with more stringent tests on linux and osx64 with openmpi and mpich.  I'm the lead developer of the underlying package and I'm eager to maintain and extend the support of the CF feedstock going forward.  Thanks!

@conda-forge/core could anyone take a quick look? I'm eager to start maintaining an extending the feedstock for this package.

Nice to see you here, Keaton! Thanks for putting in the work on this.


[Bug]: InsecureRequestWarning in storage-initializer container logs

Is there an existing issue for this?

[X] I have searched the existing issues

Deploy type
OpenDataHub core version (eg. v1.6.0)
Version
nightly
Current Behavior
After deploying a model using single-mode platform (Kserve), the model pods reports some warning about insecure https calls to my s3 bucket (which is under SSL).
The model gets downloaded successfully, though.
Below the warning (which gets repeated multiple time in the original logs)
/prod_venv/lib/python3.9/site-packages/urllib3/connectionpool.py:1056: InsecureRequestWarning: Unverified HTTPS request is being made to host 's3.us-east-2.amazonaws.com'. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/1.26.x/advanced-usage.html#ssl-warnings
  warnings.warn(
/prod_venv/lib/python3.9/site-packages/urllib3/connectionpool.py:1056: InsecureRequestWarning: Unverified HTTPS request is being made to host 's3.us-east-2.amazonaws.com'. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/1.26.x/advanced-usage.html#ssl-warnings
  warnings.warn(
...

Anything that must be changed here to avoid the warning?
Expected Behavior
TBD
Steps To Reproduce

create a DS Project in ODH/RHODS Dashboard
select single-model platform in the models and model servers section (if not already set)
click "Deploy model"
insert all the required info in the form (I used "https://" in the bucket endpoint field of the data connection)

Workaround (if any)
No response
What browsers are you seeing the problem on?
No response
Anything else
No response

@lucferbux I believe there is a plan for cluster-based solution to these bad connections -- I think this is related to the cluster self sign cert issue 🤔
Are you aware of the solution that is being worked on? Will it fix this?

I think I found the root cause, opened the above two PRs (same code, one in upstream, one in odh) which should fix the issue

cc @lucferbux

2.5 merged this pr and updated images. @bdattoma please test it.

/transfer kserve


Add generic encrypteddb.GetSecretBoxKey

Replace code that used to call chat/storage.GetSecretBoxKey but was not related to chat. Add "generic" function to aid with encrypting local storage. Note that this is not new crypto, it's merely a function that was already being present in two places (go/chat/storage/crypt.go and go/teams/storage/generic.go) that's now copied into third place. It should be used by anyone who wants to encrypt leveldb data instead of making their own variant.
There was a mixup on how we were doing encrypted caches. chat/storage.GetSecretBoxKey was being used outside of chat, and once it's been used once, more occurrences followed by people finding that call site and doing it similar way. I don't have the exact timeline here and it doesn't matter.
Contact cache will not be encrypted using key derived with ChatLocalStorage string anymore. Added new derivation string for contacts: EncryptionReasonContactsLocalStorage = "Keybase-Contacts-Local-Storage-1"
Use the new function in go/offline/rpc_cache.go but keep EncryptionReasonChatLocalStorage so users will not lose cached data.

what's the reasoning for copying to a third place instead of refactoring teams/chat to use this now?

changing teams/chat to use this is still on the table. i was approaching this very carefully so if anything broke, it wouldn't be chat or teams storage

@maxtaco can you take a look as well? it touches some team storage code that you were working in October 2018.
also cc @mmaxim


Salvage Helper Position Save

The Ectoplasm Salvage Helper's position isn't saving correctly between close/open of the application.  All other windows seem to be fine.  After a bit of testing different saved positions, it seems it saves the X axis correctly, not the Y axis.

Fixed with commit 8b0ddc853162952d97a1610a6209480041bbaf14

Release posted!


[ModuleResolveEventHandler] Include [Serializable] attribute on Mono

Relates to https://github.com/mono/mono/pull/10190

Close it because according this comment it's not necessary.


get_value for type TEL include "tel:"

When using .get_value on a property of type TEL, it includes the tel:.
For example if a vcf file have a line like this (conformed as per https://www.rfc-editor.org/rfc/rfc6350#section-6.4.1)
TEL;TYPE=cell;VALUE=uri:tel:1111111111
property.get_value will return:
tel:1111111111
When I think it should only return the numbers like what it is doing for this line:
TEL;TYPE=cell;VALUE=uri:1111111111
1111111111

TEL can store uri and textfield.  If you specify that the value type is a URI using the "VALUE=uri" parameter, the parser will parse it as a uri. If the uri is invalid (as per your last example), then it will parse it as text. Relevant code is here: https://github.com/kenianbei/vcard_parser/blob/main/src/vcard/properties/tel.rs

ok, thanks. sorry for false alert, I wasn't sure if it was normal.


Alphabet: Rename {IUPAC} to {DNA-IUPAC}

Per https://github.com/seguid/seguid-python/issues/4#issuecomment-1951698754, {IUPAC} is too vague; IUPAC defines alphabets for both nucleic acids and amino acids, so we need to be clear of which, e.g. {DNA-IUPAC} and {protein-IUPAC} (and possibly also {RNA-IUPAC}).

Alternatively, drop "IUPAC" and use a term or symbol to indicate "ambiguous" symbols, e.g. {DNA} is non-ambiguous symbols, {DNA*} is {DNA} plus all ambiguous codes.  Maybe {DNA?}, {DNAx}, {DNA...}, {DNA-ambiguous}.  Or, {DNA15} for 15 DNA symbols.

Will use:

{DNA-IUPAC}
{RNA-IUPAC}
{protein-IUPAC}

Updated JavaScript, Python, R, and C++ to support {DNA-IUPAC}, {RNA-IUPAC}, {protein-IUPAC}.


Does this app work at all?

I followed the instructions 1:1 any yet, the app does neither recognise the code I uploaded nor does it show ANY signs of progression at all. There is no "Make foto"-button or similar feature to at least trigger the processing manually.
I find it hard to like silent apps that don't work.

Sorry to hear that.
It's hard to say for sure what the problem you're running into is, but the most likely culprit is that the color decode is failing. There are some things you could try (changing color temperature on the sender screen, e.g. "night mode"), but the root of the issue is that some cameras struggle with color reproduction, and the initial version of cimbar -- the one that we're using -- needs both the symbols and colors to decode in order to work.
I'm considering a change to the format that should allow it to succeed (at slower speeds) even if the color decodes fail.

There is no "Make foto"-button or similar feature to at least trigger the processing manually.

There's no manual step that would help here. The decode is failing because either the symbols aren't being read properly (this happens if the code is too small, or the camera resolution too low), or the colors aren't being read properly (more common, since the color decode logic isn't the best).


Add seirdy.one

Uncompressed size: 8.0kb
Pingdom result: https://tools.pingdom.com/#5dcae2f754c00000
Compressed size (Brotli): 3.2kb

Everything looks solid from my testing, but you have the wrong domain listed in the file (it says xkcd.com)

Everything looks solid from my testing, but you have the wrong domain listed in the file (it says xkcd.com)

Thanks, fixed.
--
/Seirdy

Thanks, fixed.
--
/Seirdy


Upload to multiple destinations

In order to enable migrating to a new CDN

Refactored the code to split the Rackspace specifics out of the common methods
Added a DigitalOcean uploader

Once this is running and new builds are being uploaded to both destinations, I will script back-filling all of the existing images.  When the switch to DigitalOcean Spaces is complete I will remove the Rackspace calls.

@Fryguy Any other concerns?


docs(community): Update gardening area with July event info

Preliminary changes to the Gardening area in Community to reflect the upcoming event July 16-23. Expect more information to be added soon about trainings, development environment resources/opportunities, and prizes.
Thank @ptr-dang for the introduction of LF-style "sched" for the schedule! It is so much better than my spreadsheet and will be more functional for calendaring. @ptr-dang are there any additional changes to make to sched so that it doesn't prompt for login? I'm thinking those would be made on the tool end anyway so likely doesn't impact the embed code.

@dnilasor Nope we should be good here - all of it get's taken care of on the sched end


Spring boot 3.0.1 - Jasypt version downgraded

I have noticed that with jasypt-spring-boot 3.0.1 version has 1.9.3 version of jasypt. Where the prior version 2.1.0 has dependency as 1.9.4. Any specific reason why this was downgraded.

From the changelog:
2.1.1: "and support for JSB96 with IV Generators". It looks like this was implemented in a fork of jasypt with version 1.9.4.
2.2.2: "jasypt 1.9.3 rollback with IV Generators". It looks like the support for IV Generators was integrated into this starter, and therefore, it could move back to using the official library.

yup, 1.9.4 was not an "official" jasypt version.


theme tweaks

@lixonic can you fix link colors?
Neeraj was saying the links aren't prominent.

@vipulnsward  can you review the changes ?

@lixonic Please squash the commits

Can you post screenshot here?

@vipulnsward  these tweaks can be ignored


Add person/123.json role_counts extension to make Uniquify People task faster

The new role_counts extension for /people/:id.json returns information about the number of roles a Person has, grouped by type (Collector, Determiner, etc) and split by in-project usage, non-project usage, and community uses.
Sample:
{
  "in_project": {
    "Collector": 6
  },
  "not_in_project": {
    "Collector": 18
    "Determiner": 3
  },
  "community": {}
}

With this data, the Uniquify People task can be optimized, by querying for number of roles instead of getting all roles for a Person and counting them.
The only place where the actual role ids are needed is in RoleDescription, which renders the list of roles (for a specific type).
Getting the list of role ids can be performed by addSelectPerson, which means we can move it out of findPeople. As a result, executing the "Selected people" action is significantly faster.
I think it's safe to remove roles from the first part of processMerge(), since those people are being deleted anyways, and the "selected person" has its attributes re-fetched once the merge is complete.
Ideally, we could postpone the fetch for role ids until the user marks the person as "to be merged" with the checkbox in the Match People section, but I don't know enough about Vue to make that happen. Instead, I've left the behavior as-is (fetching roles for all matched people).
In places where the Person objects are extended with roles, it's faster to use role_counts to get role names, particularly when there are many roles (1k+).
After Gitter discussion, I also removed explicit database exists? query when checking if role is in project, which should make the remaining role queries much faster.

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (8ca7898) 85.66% compared to head (9253836) 72.06%.

:exclamation: Current head 9253836 differs from pull request most recent head 2ba1366. Consider uploading reports for the commit 2ba1366 to get more accurate results

Files
Patch %
Lines

app/models/person.rb
50.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@               Coverage Diff                @@
##           development    #3675       +/-   ##
================================================
- Coverage        85.66%   72.06%   -13.60%
================================================
  Files             2035     1653      -382
  Lines            74982    56030    -18952
================================================
- Hits             64230    40379    -23851
- Misses           10752    15651     +4899

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

Going to merge, pinged @jlpereira about

Ideally, we could postpone the fetch for role ids until the user marks the person as "to be merged" with the checkbox in the Match People section, but I don't know enough about Vue to make that happen. Instead, I've left the behavior as-is (fetching roles for all matched people).


DRBG update

There are two DRBGs in the old FIPS module that are not in the current code base (hash and HMAC).  They ought to be bought forward so the code isn't lost in the murkiness of history.
Adding an option to specify which DRBG is used by default and a way of changing would also be useful.

@kroeckx @mspncp

See: https://github.com/openssl/openssl/pull/6779

Pauli should this be closed?

Yes.


Can't automatically update the branch list

When I delete a branch on github, this plugin cannot automatically update my branch list

In the latest version 0.11.0 there is now an option to remove remote repository branches from that UI -- right-click and choose Remove.


Salt Component Rapid Usability Test

Evaluation of visual affordance / expectancy
### Tasks
- [x] Method & protocols
- [x] Reusable template for study
- [x] Test 2-3 released components
- [ ] Report on results

Note Stella in loop with tests and will be giving feedback on iterating and creating method that can be expanded to accessibility (e.g. testing with visually impaired users)

2/15 - set up study for button, will try to also do for status indicator - collecting data around this

Data gathered and analysed - meeting to report on results booked for Monday 20th

nothing to add - will close after review of results today

Take-aways

Users understand the Button interaction (80% thought to "click-it") however, comparison to the rates of another design system is worthwhile
The status icons have more ambiguity in isolation than previously thought (specifically error and warning)


Allow output of html pages with the htm extension

Currently (I am using Elm 0.19.0, OS is Archlinux), to generate a HTML page, I only can write a command like this:
$ elm make Main.elm --output=index.html

(Assuming that there are Main.elm and elm.json files)
However, according to fileformats, a HTML page can be with either *.html or *.htm extension. If I try to compile my elm files with --output=index.htm, I get an error indicating a bad value in --output flag.
In my current workflow, I use html / js minifier which takes only *.htm pages as an input, so it's inconvenient for me to generate "elm to html", because of the additional step of renaming the output of the elm compiler.

Which html / js minifier is that?

Which html / js minifier is that?

It's juancarlospaco/css-html-js-minify.
And they also explain their decisions regarding *.htm in migration part.


Legacy carousel

Final Design of legacy carousel added.
Changes are done in Team Page.

Changes done.Ready to be merged


resumed: init at 3.0.1

Description of changes
Lightweight JSON Resume builder, no-frills alternative to resume-cli.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[X] Fits CONTRIBUTING.md.

Not sure if nixpkgs-update needs an update script for JS packages, or if it supports them natively.
If not, I can add an update script.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2211


[URGENT] pip install fails on Ubuntu. No files/directories in /tmp/pip-build-JsrgB4/shap/pip-egg-info (from PKG-INFO)

No files/directories in /tmp/pip-build-JsrgB4/shap/pip-egg-info (from PKG-INFO)

@slundberg

I am using python 3.6.9

Can you add the verbose flag "-v" to your pip install command ?

You should see an error : Python version >= 3.7 required.
This is related to https://github.com/slundberg/shap/issues/1628.

I'm seeing the same issue. Installing via requirements.txt with a frozen numpy version does not resolve.

We're also facing this issue, it's currently our release process


Added console instructions for multiple browsers

99151b527cccdbdd4839a1481770f5b2238236a4 and
2dedb22217a87e8fbe596472f9bc504b829c9d5a add a preference for a lane with raining gold active.

Can you please split the README changes into a separate pull request from the Raining Gold code changes? They're two different changes, unrelated.


ERR_SSL_VERSION_OR_CIPHER_MISMATCH

Hi,
Page URL: https://tedatady.live
Just installed Plausible plugin via WP Plugin management and it gives me this error:

Of course I did create an account but it's still waiting for the first PV

It's solved. I did checked the analytics.domain.xx without setting it up.


Can't access Sass Global Variables in components scss

I have a habit of importing my _variables.scss and _mixins.scss into my root  src/style.scss file
@import 'styles/base/_variables.scss';
@import 'styles/base/_mixins.scss';
@import 'styles/helpers/flexbox.scss';
* {
    padding: 0;
    margin: 0;
}

body{
	padding: 4%;
	font-family: sans-serif;
}

But when I'm trying to access the variables in my components.scss, it's giving me an error
color: $primary-color;
Isn't there a way I can achieve this simply? Please Help.

You need to include styles/base/_variables.scss in each and every file where you need to use variables. Same goes for mixins. The actual content of those files will not be included in the final build and wont impact the build size in any way.

For now the solution is as @grizzm0 says.
Dupe-ish of https://github.com/angular/angular-cli/issues/1791, work in progress in https://github.com/angular/angular-cli/pull/2747.

Seriously there is no better way for this?

This is how SCSS has always worked. It really has nothing to do with the Angular CLI.

Add this to .angular-cli.sjon (change includePath to your path):
"stylePreprocessorOptions": {
        "includePaths": [
          "../node_modules/bootstrap/scss/"
        ]
      },

@JustasKuizinas if you want you can:

Ditch SASS and use css vars.

write a service that create css variables in style element in the head.

Get variables for the head style service from a js config

reuse same js config in your js code

When building your project with webpack use pure css with autoprefixer
or

use the same approach as mentioned above with no variables for color / sizes configured if you van the benifits of the pre processor

More on this here:
https://developer.mozilla.org/en-US/docs/Web/CSS/Using_CSS_variables

Looks like we just need to duplicate in every component.
@import "~style/variables";

@AlexanderKozhevin Exactly. Just like you need to import anything you use in typescript you also need to import anything you use in SCSS.

I can make another recommendation on this.
I am buzzy building a framework that will have no classes, only placeholders, mixins and variables. You can import this in the  tag and write your sass as per usual but for the component.
My afore mentioned solution has some problems, for example css has not loops and ifs. Therefore it is better to just use sass.
You can then import base styles like import grid or responsive-grid in you app component (root component).

Well, there is this loader that could resolve all our problems - https://github.com/shakacode/sass-resources-loader
I've been using it in one of the ejected projects, but will be nice if we could do this directly from CLI.

My current solution is to turn off ViewEncapsulation for the component, but I dont like this at all.  I wish there was a clean solution for this.
@Component({
selector: 'app-admin',
templateUrl: './admin.component.html',
encapsulation: ViewEncapsulation.None,
styleUrls: ['./admin.component.scss']
})

@aoakeson, form my understanding, that is based on experience and not the actual algorithm, view encapsulation only applies when styling is applied to a class specific to that component. I haven't had to turn on viewEncapsulation for any of my component. The global stylesheets just seem to apply.
I am curious to know why this is different for you.

@dewwwald I think I figured out my problem.  It turns out the elements that I was trying to access using these global variables were outside the scope of the main angular app.  As my app is comprised of multiple modules, and apps it was not able to access these.  I moved things around and am now able to have my components keep ViewEncapsulation on.

@grizzm0:

You need to include styles/base/_variables.scss in each and every file where you need to use variables.

@AlexanderKozhevin

Looks like we just need to duplicate in every component.

You should never do this because https://github.com/angular/angular-cli/issues/8431 :( Thus component.scss is totally unusable in real projects now... I think this bug should be reopened and fixed.

@LastDragon-ru As explained in that issue you should never import files that actually render CSS more than once. You can import files that contain variables and mixins as many times as you want as they do not affect the output.

You can import files that contain variables and mixins as many times as you want as they do not affect the output.

For example with bootstrap sometimes more simply and better to use &.disabled {@extend .form-control:disabled;} or @extend .is-valid than redefine(copy) this style. because after copying we will need to check it after each bootstrap update :( So this feature/issue can be useful.

I prefer creating index.scss in the components folder (lets say you have a folder Search that lives under components). Then, import all the .scss files used in this folder into your newly created index.scss. Then, include @import "../components/Search/index"; in App.scss or wherever all your global partials are stored. Now all the .scss files inside components/Search will have access to global variables.

@kkuzmina I know that it`s not right thing to do, but helped me as well 👍

This is how SCSS has always worked. It really has nothing to do with the Angular CLI. The component's CSS is being compiled by itself - the other files aren't there. There's no reason for them to be there.
This is not only how it works, this is how you should WANT it to work. You don't want coupling with monolithic styles. You WANT each component to be a COMPONENT. They are isolated, encapsulated pieces of logic.

how does importing a static path fit to that cool isolation idea of yours? I call BS.   But its not like i can create style modules (singletons) in angular which  I can inject somewhere.

This is how SCSS has always worked. It really has nothing to do with the Angular CLI. The component's CSS is being compiled by itself - the other files aren't there. There's no reason for them to be there.
This is not only how it works, this is how you should WANT it to work. You don't want coupling with monolithic styles. You WANT each component to be a COMPONENT. They are isolated, encapsulated pieces of logic.

how does importing a static path fit to that cool isolation idea of yours? I call BS.   But its not like i can create style modules (singletons) in angular which I can inject somewhere.

If you need to use a variable you need to import the file the variable is declared in. A variables file does not produce any output while imported.
For the variables to work application wide angular would have to include your global application SCSS in each component SCSS which would produce duplicated CSS.
If you import the variables file where you need it each component will have it's own "isolated" CSS while it actually "borrows" variables from a shared SCSS file. Just as you would do with a shared module in typescript/angular.
This way there's no need to declare the variable multiple times. But if you want to you could skip the import and redeclare the same variable over and over again, which seems a bit annoying to me.

Does that also hold for mixins and placeholders?
I actually had the same issue with Vue CLI and fxed it by using webpack loader oprions.
    loaderOptions: {
      sass: {
        data: `
          @import<EMAIL_ADDRESS>          @import<EMAIL_ADDRESS>        `
      }
    }

I personally don't like scoped css (which Vue OFFERS and doesnt REQUIRE), I also don't really care that i have to import my mixins, vars, placeholders etc. I just want to be sure none of those will be printed (multiple times) in my final output.

@ManuelGraf only css classes, id's and statements get printed in the final result. Mixins are reuse-able statements that print nothing until they are used. Same holds for placeholders, mixins and scss vars.

@ManuelGraf Variables nor mixins produces any output while imported. Comments in those files will (the last time I checked) however be printed multiple times. But the CLI should strip all comments when building anyhow.

Okay I am going to use it as proposed. But for some reason the ~/... doesnt work for me. What is ~ exactly in a multi-project angular 7 CLI environment?

@ManuelGraf try this: @import ~src/variables in component.scss.

This topic has been over explained. Could we lock the topic?


"We are healed" prae message appears to everyone

Description

Saw this as a ghost
Reproduction

Screenshots

Additional context

it also says "We are healed by (Yourself)" and not the prae

Fixed by #4187 I think


update stats link

Description
Updated stats link in the landing page.
Changes
Updated Explore Threefold gird capacity button to redirect to https://newstats.dev.grid.tf/
Related Issues
https://github.com/threefoldtech/tfgrid-sdk-ts/issues/1589
Checklist

[ ] Tests included
[x] Build pass
[ ] Documentation
[x] Code format and docstrings
[ ] Screenshots/Video attached (needed for UI changes)

Why https://newstats.dev.grid.tf/ is the only one that's reachable ??

I believe it's the only one reachable since it's not deployed on other networks yet.

It should redirect to the statistics page on the dashboard

It should redirect to the statistics page on the dashboard

Converted to draft since the issue is currently blocked as mentioned in https://github.com/threefoldtech/tfgrid-sdk-ts/issues/1589#issuecomment-1848923052, because the user currently cannot access the stats page without logging in.


Consider removing SeedTemplate field from ManagedSeed

How to categorize this issue?

/area scalability
/kind cleanup
What would you like to be added:
The field SeedTemplate in the ManagedSeed-API is used to register the seed object via the ManagedSeed-Controller rather than the Gardenlet.
	// SeedTemplate is a template for a Seed object, that should be used to register a given cluster as a Seed.
	// Either SeedTemplate or Gardenlet must be specified. When Seed is specified, the ManagedSeed controller will not deploy a gardenlet into the cluster
	// and an existing gardenlet reconciling the new Seed is required.

At the same time, the Gardenlet-Config requires the config.seedConfig to be set, i.e. it is anyway registering or updating the seed object when it starts.
The two concepts are conflicting and we should only follow one approach. Since Gardenlet mimics the behavior of Kubelet WRT node registration, it's preferred to get rid of the mentioned ManagedSeed use-case and to remove the ManagedSeed.SeedTemplate field completely.

@timuthy wants to work on this, thanks!
/assign @timuthy


Set CursorPosition after suggestion selected

Hello,
I would like to know if it is possible to set the cursorposition in the textbox to end of the text.
Right now, on android, after I select a suggestion from the list the cursor is set to the beginning and I would like to change that.
Thank in advance!

i think this is achievable with a custom render of the control.
in the custom render control you can do this for Android:
https://stackoverflow.com/questions/37010593/how-do-i-change-the-cursor-position-of-an-entry-manually-in-xamarin-forms


Added innerTicks and grid

Hi Sébastien,
finally I had some time to implement innerTicks and grid which can be both activated in the axes options.
Here is a demo: http://plnkr.co/edit/7IYKfdegKCseLBmJgrbv?p=preview
Fixes #300
Fixes #195
Best,
Christoph

It looks great man ! If I find some time I'll add a visual test as well.

:poultry_leg:

@chaosmail I was just playing around with this and noticed if you set grid: true on the x-axis then the grid is overlaid on top of the existing y-axes lines (so they appear grey). Updated your plnkr to demonstrate this:
http://plnkr.co/edit/VP99gY5MrfBKd90DaRQB?p=preview

thanks a lot for reporting this; i saw also that the grid is not updating when zooming.


Playback stops (playing status == .buffering) after 55 seconds when streaming

Hi @tanhakabir 👋 Fisrt of all let me say that I'm impressed by the amount of work you've put into creating this player 🙇♂
I'm experiencing an issue when streaming an audio file. Player is playing up until 55th second and then it stops. It happens in the example app and in my app consistently.
When I turned debug logging I can see that at this point AudioStreamEngine runs out of buffers and status is changed to buffering. When I skip over this point during playback (before it comes to this time) everything works fine, so the data is there is't just not passed over to the engine. Here's the log from this moment:
2021-04-15 11:13:49.219419+0200 Euphoric[4607:1766321] [DEBUG 🐝🐝🐝🐝] AudioStreamEngine.swift:updateNetworkBufferRange():210:: [Optional("loaded (0.0, 4002.089795918367), numberOfBuffersScheduledInTotal: 1, isPlayable: true")]
2021-04-15 11:13:49.316270+0200 Euphoric[4607:1766321] [DEBUG 🐝🐝🐝🐝] AudioStreamEngine.swift:numberOfBuffersScheduledInTotal:74:: [Optional("number of buffers scheduled in total: 0")]
2021-04-15 11:13:49.407243+0200 Euphoric[4607:1766321] [INFO  🖤🖤🖤🖤] AudioStreamEngine.swift:isPlayable:126:: isPlayable status changed: false
2021-04-15 11:13:49.407730+0200 Euphoric[4607:1766321] [DEBUG 🐝🐝🐝🐝] AudioStreamEngine.swift:updateNetworkBufferRange():210:: [Optional("loaded (0.0, 4002.089795918367), numberOfBuffersScheduledInTotal: 0, isPlayable: false")]
I'd like to fix the issue myself, but I don't have good understanding of the code yet, so I was hoping that maybe you as an author have rough idea where this issue may come from and where should I start looking in the code.
Best,
Maciej.

Hey! @niczyja! yes definitely I can look into this! super weird edge case that I haven't run into yet but excited to solve! What's the url of the file you're streaming?

@tanhakabir thank you. In the Example app it happens with poscast stream from url: https://chtbl.com/track/18338/traffic.libsyn.com/secure/acquired/acquired_-_armrev_2.mp3?dest-id=376122 and in my app it happens for every podcast url that I've tried so far.


Feat: unit test in wallet_kit

groupBy Unit Test
formatAddress Unit Test

Hello @gabsn, can you please review this PR?

Thx, for the PR! Can you post it in starknet.dart telegram convo as an update for other contributors.
Also, please create an account on app.onlydust.com, I will pay you for this contribution!

hey @gabsn
thanks for the offer but It's my open-source contribution.


Testing Strato on OpenShift 4.1 (detailed steps at http://bit.ly/2TPp2Dy) - need contributor guide

Testing Strato on OpenShift 4.1 (detailed steps at http://bit.ly/2TPp2Dy) - need contributor guide

added latest instructions at https://github.com/marcredhat/golang/blob/master/ocp4strato


Rename LISK -> LSK for Lisk

Fix based on issue Naming consistency "Lisk"

@allienworks :point_up_2:

Thanks very much for your contribution!


Add pseudo-random boolean method

Hi,
very often, when I write tests, I need a method that will return a random boolean and I don't know why, but so far such a method is not in Any library and I always had to write its on the side.
So I decided to add it

Wouldn't the name BooleanValue be better for syntax highlighting, like GuidValue?
Also, why IntegerInRange with the magic number 100? Wouldn't GetNextValue(2) mod 2 = 0 be enough/clearer? Or do higher numbers somehow provide "more randomness"?
I also agree it would be nice to have this function included in the library. :)

o

Hi,
sorry I completely forgot about this PR. Thanks for participation.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 48846.

Availability update: We will publish a fix for this issue in the next update for release 21.
Build ID to track: 49470.


[WIP] Adjusting indicators and multi to remove colour controls from indicators

An idea from the work on the multi class allowing indicators (or anything else) to exclude themselves from the multi colours by setting the class name used by the multi series.
Normalised and unified all the indicator styles so common across the current set of indicators.
Goal was to make it so if we had two Bollinger bands then they would colour sequence. Also means don't need to fix their colour within the CSS file.

still would need to work through the site to normalise the effect of the change if people felt useful.

Sorry to beat the decorate drum again but... is this not something best solved by decorate?
This diff seems to sum up my argument nicely https://github.com/ScottLogic/d3fc/pull/669/files#diff-051607ee3462a19057ed739a1a120251L41

All of the indicators had a decorate function attached to the multi already. But then with the multi series CSS changes, they need additional decorators (or very controlled styles in the CSS e.g. Bollinger Bands) to control the colours of each part of the render, as they will be a different colour by default.
The changes to the decorators in each of the indicators is to remove the multi and use standard classes across all the indicators (e.g. signal and indicator class).

Sorry, only just had the chance to look at this PR again. In its present form it 'breaks' a number of the indicator renderers:

Although this could be due to changes on master since you created this PR.
However, putting this to one side for now, I don't think I agree with the underlying concept of making the indicator CSS more generic. I don't think that reflects reality, various indicators have generally accepted styles, e.g. MACD signal being rendered as a red line.
Although, one thing I do like in this PR is the change to remove the multi class when it is redundant, replacing it with bollinger for example.
Furthermore, this could be 'pushed down' into datajoin, so that the pattern we see repeated throughout the code:
var dataJoin = _dataJoin()
    .selector('g.annotation')
    .element('g')
    .attr('class', 'annotation');

simply becomes
 var dataJoin = dataJoinUtil('annotation');

Also, it would remove the need for this code in small multiples:
https://github.com/ScottLogic/d3fc/blob/master/src/chart/smallMultiples.js#L33
So in summary:

Sorry, don't fully agree with the motivations of this change :-(
I like the improvements to multi series
I'd happily receive a PR that improves on this further, pushing the concept into dataJoin removing the repeated boiler-plat code required to create a dataJoin that is 'scoped' by a CSS class.

Will take a look at pushing down to data join and do a new PR soon


wireshark: 3.0.5 -> 3.2.0

Motivation for this change
https://www.wireshark.org/docs/relnotes/wireshark-3.2.0.html
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

Built and tested with with nix-review. Not sure if that's a trust issue but ofborg didn't build for darwin
$ nix build --no-link --keep-going --option build-use-sandbox relaxed -f /home/teto/.cache/nixpkgs-review/pr-76601/build.nix
[14 built, 40 copied (85.7 MiB), 33.3 MiB DL]
https://github.com/NixOS/nixpkgs/pull/76601
7 package built:
compactor haka ostinato termshark wifite2 wireshark wireshark-cli


add support for GPBoost

GPBoost is a library for combining tree-boosting with mixed effects and Gaussian process models (see https://github.com/fabsig/GPBoost). The tree boosting part is handled by a clone of LightGBM. I.e., GPBoost can be handled in shap in exactly the same way as LightGBM.

Thanks! This looks good. Can you also add test like the current one for LightGBM ...just to make sure things work and stay working :)
https://github.com/slundberg/shap/blob/cad1019a9887d4e3373a25ad472d839fc49bc377/tests/explainers/test_tree.py#L369-L389

Many thanks for your quick answer! I added a test for GPBoost.

Any updates on this? If it is OK, by when can this be integrated in to shap?

Thanks again, and sorry for the delay! I have been working on a new API for SHAP (while staying backward compatible), and so merging has been slower. This looks good now.

Perfect. Thanks a lot!


overriding AlertDialog baseStyle (_fade, _backdropFade) is not working

Description
overriding AlertDialog baseStyle (_fade, _backdropFade) is not working.
(+) and default fade animation duration became too slow, so it seems unnatural.
CodeSandbox/Snack link
https://snack.expo.dev/sSC9-U5FS
Steps to reproduce

Go to Expo Snack link
Open iOS App, and click 'Open' Button
and watch the AlertDialog is showing and see _fade, _backdropFade baseStyle is not overriden.

NativeBase Version
3.4.3
Platform

[ ] Android
[ ] CRA
[ ] Expo
[X] iOS
[ ] Next

Other Platform
No response
Additional Information
No response

Hello @hoondeveloper, Thanks for reporting this issue, we'll look into this!

@surajahmed  @rayan1810 @mayank-96 Need to rethink how this can be handled on the theme level and not on the component level and should provide consistency for all the animation props.


Unit of Measure used by Raw Statistics

In the screenshot, on Visual Studio Gallery, https://visualstudiogallery.msdn.microsoft.com/133d5764-b32b-4ec9-8ee8-5546eca64584/showImage/127075, we see that some of the time-based statistics are displayed in various unit of measure (i.e. seconds, miliseconds, microseconds, nanoseconds, etc...).
When we extract the statistics, we do not have any unit of measurement associated with the raw data. Therefore, I assume, that in that case, all the raw numbers are in one single unit of measure.
If I am right, which unit of measure would that be?
Thanks.

all times are in seconds, when pulling a snapshot )


Dutch National Flag Algorithm in CPP

Please assign me to add Dutch National Flag Algorithm in C++ arrays section under hacktoberfest 2022

hi,if this issue is still open could i work on this


Add option to flag papers from dashboard?

Suggested by @rankishore
This needs to be discussed with @vanaukenk and @draciti since it could overlap with existing functions from other platforms/forms

We need to ask @rankishore what is the use case. Does she mean validating negative a paper that was flagged by an author?

We will open a new issue, closing  this


luci-app-travelmate: bugfixes

fix interface default value on overview page
fix re-ordering of wireless stations

Signed-off-by: Dirk Brenken<EMAIL_ADDRESS>

I've added the translation fix by INAGAKI Hiroshi<EMAIL_ADDRESS>(@musashino205) to this PR.
#1266


Update README.md

The comment you have added

(needs root access to install PhantomJs)

is not necessarily true, as it just depends how you have NPM setup.  See here for more details:
https://docs.npmjs.com/getting-started/fixing-npm-permissions

@akiaksdon I've given this some more thought and I am going to merge with a slight refinement:
Depending on your npm permissions you might need root access to install PhantomJS2.


NUTCH-2695: fix some alerts raised by LGTM

Fix a few of the alerts raised by LGTM (https://lgtm.com/projects/g/apache/nutch/alerts/?mode=tree).
-Wrong NaN comparison in Generator
-Type mismatch on container modification in NutchServerPoolExecutor
-Missing format argument in CrawlDbReader

Thanks, @1m2c3t4! Your fixes are merged and many more alerts have been fixed.


GET /assembly/3 gives 500

https://api-dev.col.plus/assembly/3

I think this happened after attempts to sync these sectors: https://api-dev.col.plus/assembly/3/sync?limit=25&offset=0

java.lang.IllegalStateException: Non running or waiting sync with state FAILED found in queue for sector 17
	at org.col.assembly.AssemblyState.<init>(AssemblyState.java:31)
	at org.col.assembly.AssemblyCoordinator.getState(AssemblyCoordinator.java:92)
	at org.col.resources.AssemblyResource.state(AssemblyResource.java:46)

related to importer queue refactoring I would think


Add mdns browser

Please add in core: NewClientFromService() function (That calls the previous NewClientFromSocketAddress).
Update Examples in README


permmision inheritance bugging

http://pastebin.com/asHu9gWf not sure if I did something wrong, but people have the ranks, and inheritances are working but there is no prefixes popping up. and I think this is with inheritances bugging, because I had just put in "_"'s In the inheritances, and peoples prefixes disapeared

never mind, I fixed it, I just had to restart my server


Remove dead-code infinite loop

Commit d32f5d8733df9938727710d4194e92813c421ef1 added a 'goto end;' statement
at the end of the code block for the 'end' label.  Fortunately, it was after a
return statement, so no infinite loop occurred, but it is still dead code.
Remove the extra goto statement as cleanup.

+1

+1

Didn’t we just fix this?

Didn’t we just fix this?

Nope.

commit 9547982 done, thanks.


KAFKA-3937: Kafka Clients Leak Native Memory For Longer Than Needed With Compressed Messages

@ijuma - Making the change against trunk based on your suggestions to have the stream closing handled in the private RecordIterator constructor which I understand is only to be used only if the block of message(s) are compressed.

Thanks for the PR @wiyu, I created a PR with some improvements https://github.com/wiyu/kafka/pull/1
If they make sense to you, please merge to your branch and then I can merge the PR to trunk.

@ijuma: Thanks for the improvements and help. Please have a look again when jenkins is done.

Thanks for the PR, LGTM.


haste-task-sass: remove 'node-sass' from dependencies

This PR moves node-sass from dependencies and into devDependencies. Installing node-sass should be up to whoever uses this package.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #192   +/-   ##
=======================================
  Coverage   86.58%   86.58%
=======================================
  Files          13       13
  Lines         164      164
  Branches       21       21
=======================================
  Hits          142      142
  Misses         19       19
  Partials        3        3

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 3fde93d...459eb0b. Read the comment docs.


Learning Hidden Unit Contribution (LHUC)

Added LHUC method for domain adaptation. See David Vilar, "Learning Hidden Unit Contribution for Adapting Neural Machine Translation Models", NAACL 2018.
Currently only for RNN models (as in the paper), will expand to other architectures in separate PRs.
Pull Request Checklist

[x] Changes are complete (if posting work-in-progress code, prefix your pull request title with '[WIP]'
until you can check this box.
[x] Unit tests pass (pytest)
[ ] Were system tests modified? If so did you run these at least 5 times to account for the variation across runs?
[x] System tests pass (pytest test/system)
[x] Passed code style checking (./style-check.sh)
[x] You have considered writing a test
[x] Updated major/minor version in sockeye/__init__.py. Major version bump if this is a backwards incompatible change.
[x] Updated CHANGELOG.md

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Added an integration test. A system test is more complex as we should define a "domain adaptation" scenario.


questions about pics/controller_1.png

great to see this repo. I've tried to deploy this project but I cannot find the UI that you've presented in pics/controller_1.png. Can you tell me why？wait for your response sincerely.

Greetings. To this view you have to select Controller tab and create a new project. Also can happen that there is an error somewhere)).  Can plz provide some screenshots

Greetings. To this view you have to select Controller tab and create a new project. Also can happen that there is an error somewhere)). Can plz provide some screenshots

Thanks a lot for your replay！ I have seen that UI by creating a new project.


Missing generatepdf.js

I've been using regular old Chrome's print page to PDF feature instead, works great.


Support for stacked line charts

Describe the feature request
Add support for a stacked line chart
Which charts does this feature request apply to?
Line/Time
Describe the solution you'd like
The ability to set stacked: true for the y axis
JavaScript equivalent
var stackedLine = new Chart(ctx, {
    type: 'line',
    data: data,
    options: {
        scales: {
            yAxes: [{
                stacked: true
            }]
        }
    }
});

Describe alternatives you've considered
N/A
Additional context
Docs are at the very bottom of this page: https://www.chartjs.org/docs/latest/charts/line.html

Thanks for contacting us. I appreciate the well-written issue :)
We should add this feature, nothing to debate there.
How to implement:

The Stacked property will be added to CartesianAxis since it can be used for any bar and line axis. It will also be removed from the bar extension axes.
Add summaries indicating the behaviour for both bar and line axis. We could also try to make the property virtual in the base class and override it in the line and bar axes respectively with adjusted summaries but that leads to redundancy in both code and summaries.

Workaround / alternative
You wrote N/A for the alternative but this is very easy to implement yourself.
Just create a sub class of the axis you want to use e.g. StackedLinearCartesianAxis : LinearCartesianAxis. Then add the Stacked property (type bool) to your class and use that instead of the original one when creating your config.
Let me know if that works as a temporary solution @dfederm.

@Joelius300 thanks that workaround worked great. I didn't realize these objects were just straight up serialized and used directly as the js config objects.


raise RegexNotFoundError('Unable to extract %s' % _name)

Checklist

[ ] I'm reporting a broken site support issue
[ x] I've verified that I'm running youtube-dl version 2021.12.17
[ x] I've checked that all provided URLs are alive and playable in a browser
[ ] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[ ] I've searched the bugtracker for similar bug reports including closed ones
[ ] I've read bugs section in FAQ

Verbose log

PASTE VERBOSE LOG HERE

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-x', '--audio-format', 'mp3', '--verbose', 'https://www.youtube.com/watch?v=-gsDBuHwqbM']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.9.6 (CPython) - macOS-13.2.1-arm64-arm-64bit
[debug] exe versions: ffmpeg 5.1.2, ffprobe 5.1.2
[debug] Proxy map: {}
[youtube] -gsDBuHwqbM: Downloading webpage
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/Users/jim/Library/Python/3.9/lib/python/site-packages/youtube_dl/YoutubeDL.py", line 815, in wrapper
    return func(self, *args, **kwargs)
  File "/Users/jim/Library/Python/3.9/lib/python/site-packages/youtube_dl/YoutubeDL.py", line 836, in __extract_info
    ie_result = ie.extract(url)
  File "/Users/jim/Library/Python/3.9/lib/python/site-packages/youtube_dl/extractor/common.py", line 534, in extract
    ie_result = self._real_extract(url)
  File "/Users/jim/Library/Python/3.9/lib/python/site-packages/youtube_dl/extractor/youtube.py", line 1794, in _real_extract
    'uploader_id': self._search_regex(r'/(?:channel|user)/([^/?&#]+)', owner_profile_url, 'uploader id') if owner_profile_url else None,
  File "/Users/jim/Library/Python/3.9/lib/python/site-packages/youtube_dl/extractor/common.py", line 1012, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

## Description

<!--
Provide an explanation of your issue in an arbitrary form. Please make sure the description is worded well enough to be understood, see https://github.com/ytdl-org/youtube-dl#is-the-description-of-the-issue-itself-sufficient. Provide any additional information, suggested solution and as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.
-->

WRITE DESCRIPTION HERE

Good morning, I did find a bug report with the "Unable to extract uploader id", but that isn't the bare cause of the problem, which instead seems to be with the regex trying to extract the name.

Thank you for your diligent work and great product.

Good morning, dirkf, thank you so much if the fix has been implemented!  I worry, though, that it's marked as completed because I did something wrong.  If the latter, could you please let me know?  Thank you so much; this is my first ever bug report on anything and I was a bit nervous about the posting.

It was this:

[ ] I've searched the bugtracker for similar bug reports including closed ones

Read #30839, please.

Understood --- thank you very much for the quick and helpful reply!


Thank you message never disappears...

Hey there:
Thanks for your work on this plugin. I have noticed, once I submit a message, the thank you message appears. And then never disappears no matter how many times I reload the page.
Also, I have a one page modular site, and it was incredibly difficult to set up and confusing. You might want to add some documentation on modular pages as the "contact form" tab does not appear on modular pages, and I had to manually edit the .md file to get it to show up.
Thanks again...

First, thank you for the suggestion. I believe that there is a problem with modular pages now - as a result of a Grav update at some point I think.
If you have a chance, could you provide a little more detail regarding the thank you message? Perhap an example?
Also, if you want to submit a PR, that'd be great!

Pardon my ignorance,. what is a PR? Thanks!
-jerrt

On Jul 6, 2017, at 1:38 PM, Jesse Maxwell<EMAIL_ADDRESS>wrote:
First, thank you for the suggestion. I believe that there is a problem with modular pages now - as a result of a Grav update at some point I think.
If you have a chance, could you provide a little more detail regarding the thank you message? Perhap an example?
Also, if you want to submit a PR, that'd be great!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/aradianoff/recaptchacontact/issues/40#issuecomment-313483067, or mute the thread https://github.com/notifications/unsubscribe-auth/ABpQ_8E3Y8UtutOKNcyQojjNIhJzaOS2ks5sLSnCgaJpZM4OKN2U.

It's short for Pull Request.

I have the same issue as @gripmedia. I have a page with a contact form, and when the form is successfully submitted, the green success message appears above the form and won't disappear even when the page is reloaded.
On a hunch, I deleted Grav's 30-minute session cookie called grav-site-4a62a9c. The green message then disappeared on reload. However I suspect this is not a direct cause of the persistent recaptcha message.
I was wondering if the problem could be related to the fact that my contact form page has a sub-page (called 'thank-you') which I guess is supposed to be rendered after submission if Javascript is off?
PHP 5.6, Grav 1.3.2, Admin 1.5.2, Form 2.8.2, Recaptcha Contact 2.1.2 - hope you can shed some light or point me where to look.

I honestly gave up and just went to a honeypot method and that is working great.

On Aug 31, 2017, at 6:50 PM, mn113<EMAIL_ADDRESS>wrote:
I have the same issue as @gripmedia https://github.com/gripmedia. I have a page with a contact form, and when the form is successfully submitted, the green success message appears above the form and won't disappear even when the page is reloaded.
On a hunch, I deleted Grav's 30-minute session cookie called grav-site-4a62a9c. The green message then disappeared on reload. However I suspect this is not a direct cause of the persistent recaptcha message.
I was wondering if the problem could be related to the fact that my contact form page has a sub-page (called 'thank-you') which I guess is supposed to be rendered after submission if Javascript is off?
PHP 5.6, Grav 1.3.2, Admin 1.5.2, Form 2.8.2, Recaptcha Contact 2.1.2 - hope you can shed some light or point me where to look.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/aradianoff/recaptchacontact/issues/40#issuecomment-326466139, or mute the thread https://github.com/notifications/unsubscribe-auth/ABpQ_5b7tAZYusJnazW5bigoIiRbv5diks5sd2L7gaJpZM4OKN2U.

I would not be surprised if Grav has changed the way it handles sessions. The message should be cleared (internally) on every page request, but it sounds like that isn't working anymore.
Refer to this method: https://github.com/aradianoff/recaptchacontact/blob/master/recaptchacontact.php#L204

After updating Grav Core and all plugins I have installed, I have exactly the same issue.
If you need an emergency tempory fix before the plugin is revised, you can edit onTwigSiteVariables method from recaptchacontact.php file like that:
public function onTwigSiteVariables() // Esto se procesa después de onPageInitialized
    {
        $config = $this->grav['config'];

        if ($config->get('plugins.recaptchacontact.enabled') || $this->shouldLoadCss) {
            if (!$config->get('plugins.recaptchacontact.disable_css')) {
                $this->grav['assets']->addCss('plugin://recaptchacontact/assets/recaptchacontact.css');
            } else {
                $this->grav['assets']->addCss('theme://assets/recaptchacontact.css');
            }

            $this->grav['twig']->twig_vars['recaptchacontact'] = $this->grav['config']->get('plugins.recaptchacontact');
            $this->grav['twig']->twig_vars['recaptchacontact']['message'] = $this->grav['session']->contact_message;
            $this->grav['session']->contact_message = null; // Temporary dirty quick fix
            $this->grav['twig']->twig_vars['recaptchacontact']['session'] = $this->grav['session']->form;
        }
    }

It's dirty but it's better than nothing.


add pre-commit hook

ignore alluxio/ for now before we clean up the repository

@jja725 could you help fork alluxio-py and create personal branch, may start cleaning up some branches these days and worry about mistakenly deleted some on-going branches

@jja725 @LuQQiu Quick comment: Previously, we restricted the length of each line to 80 characters by running
black [changed-file].py --line-length 79. With this PR, the length will be changed to 88 characters.
Do we want to keep the original one or do we want to reformat the whole repo?

@ChunxuTang Thanks! Didn't notice there's line length, add back in #41


how about  the logcabin's performance in product？

3 nodes ：7200 rpm sata ，24 core ，Xeon E5-2630
maxThreads = 16
storageModule = filesystem
storagePath = test2
servers = <IP_ADDRESS>:61023;<IP_ADDRESS>:61203;<IP_ADDRESS>:61023
RHEL6.2
qps:30/s

Hi @shuzhifeng, I've now finished #30. I haven't benchmarked the recent changes to it yet, but I think you should see performance improve, hopefully to somewhere around 90 ops/second. Want to give it another try?

More info at http://ongardie.net/blog/logcabin-2015-02-11/#logcabin-2015-02-11-performance

Closing due to inactivity and no clear next actions. Feel free to reopen.


CapRover low memory validation is not working

Typing a number lower than the minimum allowed amount(250 MB) doesn't disable the deploy button on testnet.

same issue with peertube and funkwhale on testnet


"OnSuccess" of a useQuery get triggered even if it has the option "enabled" : false

Describe the bug
the problem if we are using two useQuery with the same key , and one of them is enabled and the other one is not. the effect functions get triggered for both of them.
Your minimal, reproducible example
https://codesandbox.io/s/vibrant-mirzakhani-0tf2b1
Steps to reproduce
1- Check the console in the link above, and you will notice that "onSuccess" get triggered even if useQuery  has enabled equals to false
Expected behavior
the onSuccess  and other effect functions should get triggered only when enabled is true
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform

web
react native

react-query version
3.34.16
TypeScript version
No response
Additional context
No response

the onSuccess and other effect functions should get triggered only when enabled is true

hm, why do you think so? when the fetch succeeds, you'll also get data even if one useQuery is "disabled". "disabled" just means this query won't fetch data on its own, but if data is put by other means into the cache, it will be available, which means that for a successful fetch, onSuccess will be called, and for errors, onError will be called.
what are you doing in onSuccess that would require a different behavior?

I think the problem is not with onSuccess here, but that 'enabled' property seems to be ignored and fetch data despite it is set to false, which is not what it stated in documentation:
https://react-query.tanstack.com/guides/disabling-queries

I just saw your comments, Thank you @TkDodo  for the clarification 🙏

How is it fixed?
I have a GET and a PUT and I set queryKey: ['get'] and queryKey: ['put'] respectively, but I still have the same issue of onSuccess being called for both even though the second has enabled: false


Add support to the MET tools for climatological distribution percentile thresholds.

This is an extension of the percentile thresholds described here:
https://github.com/NCAR/MET/issues/76
As of met-8.1, the tools support defining percentile thresholds as follows:

SFP50, >SOP50, >SCP50, >USP50(2.5), ==FBIAS1 ... for the ...
sample forecast, sample observation, sample climatology percentiles,
user-specified percentile, and frequency bias 1 options, respectively.  These threshold types derive a single actual threshold value which is the numeric percentile value over the requested spatial region.

This task is to define climatological percentiles which vary for each grid point.  Add support for this new threshold type: >CDP50, for climatological distribution percentile.
This can only be computed when the user defines the "climo_mean" and "climo_stdev" config file options to define the full climo distribution.  As such, we may need to add "climo_stdev" to additional MET tools.
In addition, NOAA/EMC would like the option of defining these climo distributions from 101 empirical 2.5 degree values, rather than using the climo mean and stdev.

Charge 2780541

Email from Binbin on 8/21/19 regarding sample data
I  created a sample file directory on theia for our GEFS  verification package located  in directory:
/scratch4/NCEPDEV/fv3-cam/save/Binbin.Zhou/work/grid2grid/verf_g2g.v3.0.13/sample_data
In this directory, 20 grib2 GEFS member files are those headed as "fcst",  containing only one 24hr forecast (run at 12Z 20190818).
The analysis data file is obsv.grib.gefs2p5.ensbl which is GDAS analysis validated at 12Z 20190819.
The climatology data are those files ended with JUL, AUG and SEP. They are binary,  101 bin data from which 11 bin data can be
derived (averaged)  For detailed description please look at  readme.txt file in the same directory.
The output VSDB file is GEFS2P5_2019081912.vsdb. The RPS and RPSS can be seen in "RPS" record line. The first 6  scores in
each RPS record line  are  RPSf, RPSc, RPSS, CRPSf, CRPSc, CRPSS.   "f" refers to forecast, "c" to climatology. So if you like to
see the values of  RPS and RPSS, just look at the first 3 values on RPS line records.
Our verification is over 6 sub-regions, identified with G2/XX, G2 means grid#2 (2.5x2.5degree), XX is NH, SH, TR, NA, AS, EU, etc
(refers to N. and S. hemisphere,  Tropical, N. America, Asia and Europe) . The la and lon  for these regions are defined in control
file g2g.ctl.ensemble.
If I miss something or you have  further questions, please let me know.
Binbin

Committed development code to the feature_1138_climo_bins branch after finishing updates to the library and application code and merging the latest changes from develop back into the branch.
Also made sure all existing unit tests run and produce the same output as the develop branch.
Next tasks are:
(1) Enhance Grid-Stat to write climo_dist_perc fields to the NetCDF matched pairs output file.
(2) Add new or extend existing unit tests to actually exercise the new functionality.

Finished enhancing Grid-Stat to support new nc_pairs_flag.climo_cdp option.
Updated README and MET User's Guide with info about nc_pairs_flag.climo_cdp as well as the CDP threshold types.  With all these changes, will run the regression test again to make sure it still works.

Demonstrated the CDP functionality to Tara on 10/15/2019 and showed her the output from the updated unit tests.
Here's some more tasks to do:

Test to make sure CDP thresholds work in ensemble-stat, both for the cat_thresh and obs_thresh filtering options.

Enhance percentile thresholds to support the use of short-hand notation like we have when setting cat_thresh = [ ==0.1 ]; for probability forecasts.

This applies to all percentile threshold types (CDP, SFP, SOP, and SCP).  Replace any percentile thresholds of type equality with bins... applied to cat_thresh, cnt_thresh, and obs_thresh (in Ensemble-Stat).

For cat_thresh, define overlapping thresholds, e.g. replace cat_thresh = [==CDP25] with cat_thresh = [ >CDP0, >=CDP25, >=CDP50, >=CDP75, >=CDP1.0 ];
How do I know if these should be > or <?
For  cnt_thresh and obs_thresh, define discrete bins, e.g. replace cnt_thresh = [ ==CDP25] with cnt_thresh = [ >=CDP0&&<CDP25, >=CDP25&&<CDP50, >=CDP50&&<CDP75, >=CDP74&&<=CDP100];
  - Make sure to follow the same conventions as ==0.1 for probabilities.

NOAA will want to use: cnt_thresh = [ ==CDP10 ];  And they will want to compute the mean of the scores across those bins.  But where should this happen?  In grid_stat and point_stat directly or in stat_analysis or METviewer?

Finished adding the shortcut for ==CDP10 for cnt_thresh, wind_thresh, and obs_thresh.
Fixed ensemble-stat and added test for using CDP thresholds there.  They are supported in obs_thresh, but not cat_thresh since we don't have any climo info defined for the ensemble dictionary entries.

MET is not handling the time interpolation of monthly NCEP 2.5 degree data correctly.  The data for each month is valid on the 15th of that month.  For the forecast valid time, we need to find the monthly data before/after that date and interpolate to that date.
Recommend replacing config options:
match_month        = TRUE;
match_day          = FALSE;
time_step          = 21600;
With:
day_interval       = 1; // or 31 for monthly or NA for persistence (day_interval = 1 is a special case)
hour_interval      = 6; // or 1, 6, 12, 24 or NA for persistence (ERROR: < 0 or > 24)
The day_interval and hour_interval options define the spacing of the climo data and control what data is used.

Finished by merging feature_1138_climo_bins branch into develop.  Support added for climo CDP threshold types: >CDP75.  And support added for computing the mean of statistics across climo bins.


Function CRUD operations should return 202 accepted

Since Swarm is effectively declarative CRUD operations should return 202 Accepted rather than 200 OK since we do not know if things are actually OK that that point. The R of CRUD (retrieve) can return 200 OK.
Context - Issue raised on FaaS repo by Simon @templum https://github.com/openfaas/faas/issues/693

Yes, I will take this one.

@viveksyngh this issue has been open for several months now with no updates. What do you want to do?

I have done changes for faas-netes, I will test that. Once that is done I will work on this one.