add luna tutorial

replaces closed PR#249

moved notebooks, conf, img to vmount

this change simplifies tutorial and allows user to see updates made to notebooks and conf from host system without needing to create backups. There is a small risk of accidental commits of changed files to github but future integration testing can check for that.

updated paths in all confs for first three notebooks

update paths in first three notebooks

removec backup script

updated readme

tried alpine but decided ubuntu18 makes a better base because luna dependencies require native libs that alpine does not have and is hard to piece together

noticed that dataset prep notebook writes to,
vmount/notebooks/computational\oncology_tmp_zips/
vmount/notebooks/regional_bmps/
vmount/notebooks/regional_npys/
vmount/notebooks/slides
vmount/notebooks/dask-worker-space
vmount/notebooks/data-processing.log
These outputs should go into project dir else it makes it hard to cleanup after a run

swapping pip install pyluna pyluna-pathology with pip install pyluna[pathology] didnt bring in the lib. import luna.pathology failed. Needs some debugging.

Codecov Report

Merging #251 (5799850) into dev (f210269) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #251   +/-   ##
=======================================
  Coverage   53.26%   53.26%
=======================================
  Files          85       85
  Lines        6117     6117
=======================================
  Hits         3258     3258
  Misses       2859     2859

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 f210269...5799850. Read the comment docs.


Fix upsert of custom search bar types

To use the sw-search-bar with custom entities, you need to add (upsert) a new search type. When using the respective upsert function of the SearchTypeService, the configuration of the custom type is stored in a configuration property:
https://github.com/shopware/platform/blob/27558b4c00452ba25c5f99de42da37144b37e723/src/Administration/Resources/administration/src/app/service/search-type.service.js#L57-L59
The upsert call
searchTypeService.upsertType(
    'some_custom_entity',
    {
        entityName: 'some_custom_entity',
        ...
    }
 );

will result in
const $typeStore = {
...
    some_custom_entity: {
        configuration: {
            entityName: 'some_custom_entity',
            ...
        },
    },
...
}

Whereas no such configuration property is present in the default types here:
https://github.com/shopware/platform/blob/27558b4c00452ba25c5f99de42da37144b37e723/src/Administration/Resources/administration/src/app/service/search-type.service.js#L14-L19
and not used in the sw-search-bar here:
https://github.com/shopware/platform/blob/9ad84b26097c066ac9120953517008ccf5233c05/src/Administration/Resources/administration/src/app/component/structure/sw-search-bar/index.js#L88
This is caused by an incorrect usage of the spread operator at https://github.com/shopware/platform/blob/27558b4c00452ba25c5f99de42da37144b37e723/src/Administration/Resources/administration/src/app/service/search-type.service.js#L58, which this PR fixes.

thanks for your contribution @hanneswernery :+1: :tada: :blue_heart:


First Reference link in DS Specifcation is dead

112.17 References
[1]Automating Service Dependency Management in a Service-Oriented Component ModelHumberto Cervantes, Richard S. Hall, Proceedings of the Sixth Component-Based Software Engineering Workshop, May 2003, pp. 91-96
http://www-adele.imag.fr/Les.Publications/intConferences/CBSE2003Cer.pdf

The PDF is not available anymore under this link. We should either remove it, relink it to its new location or host it ourself if we can dig it up somewhere.

You can use links to archive.org: https://web.archive.org/web/20061123212733/http://www-adele.imag.fr/Les.Publications/intConferences/CBSE2003Cer.pdf


pkg/json_rpc_2: make it easier to use over HTTP

This issue was originally filed by<EMAIL_ADDRESS>
Currently, the documentation only provides an example of usage with WebSockets. The API requires a Stream object to be provided, which is what you get when a WebSocket is opened.
However, JSON-RPC 2 also supports usage over HTTP, but no standard HTTP client in Dart can provide a Stream object to use with json_rpc_2.

This issue has been moved to dart-lang/json_rpc_2#5.


Base: resort to more drastic means to query profile directory

The profile directory for another user is considered private data and cannot be directly queried without an elevated privilege, the user's password, or a token with impersonation rights.  Resort to trawling through the registry to query the information.

CC: @parkera @etcwilde

100% tests passed, 0 tests failed out of 1916

@swift-ci please test


udp_session_filters: dynamic forward proxy session filter

Additional Description: Adds a UDP session filter that's able to perform DNS resolution for dynamic host based on filter state values. Using a custom user's session filter placed before the dynamic forward proxy filter, it's able to have the same functionality for UDP flows similar to TCP and HTTP. This was originated by #29277, although not really needed to perform tunneling.
Risk Level: low, new extension
Testing: unit tests, integration tests
Docs Changes: change log, API docs, configuration docs
Release Notes: none
Platform Specific Features: none

@mattklein123  I think Lizan is out, would you mind doing API and code pass as well, since your code owner?

/retest (testing)

ping @lizan, @mattklein123
cc @alyssawilk

Sorry no I don't have time to review this.

Thanks for the update Matt. @alyssawilk can you please help assign relevant reviewers to help this PR progress? I see you also had some DFP involvment, could you maybe take this one?

@ravenblackx for some reason I thought Alyssa was the on-call this week (therefore previous comment). Can you please assign this PR to the relevant reviewers?

@adisuissa can you api-shepherd this?
@rshriram as code-owner of dynamic_forward_proxy, any chance of a review?

Ping @adisuissa

@adisuissa I didn't read your 2. comment correctly initially, adjusted the API now, thanks. PTAL

Ping @adisuissa

I'm assuming Raven's earlier approval still stands as the intervening commit was minor. However this needs a senior maintainer approval.
@mattklein123 I'm not sure if you have time but you might have the most context on dynamic forward proxy filters. If you are not able to do it ping me and I'll re-assign to a different senior maintainer.

Yup I can take a look through in the next couple of days.


Align recipe with cuquantum recipe.

Checklist

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

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found some lint.
Here's what I've got...
For recipe/meta.yaml:

❌ Jinja2 variable definitions are suggested to take a {%<one space>set<one space><variable name><one space>=<one space><expression><one space>%} form. See lines [0]
❌ Non noarch packages should have python requirement without any version constraints.
❌ Non noarch packages should have python requirement without any version constraints.

For recipe/meta.yaml:

ℹ The recipe is not parsable by parser conda-souschef (grayskull). This parser is not currently used by conda-forge, but may be in the future. We are collecting information to see which recipes are compatible with grayskull.
ℹ The recipe is not parsable by parser conda-recipe-manager. The recipe can only be automatically migrated to the new v1 format if it is parseable by conda-recipe-manager.
ℹ Jinja2 variable references are suggested to take a {{<one space><variable name><one space>}} form. See lines [48].

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12463158354. Examine the logs at this URL for more detail.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to rerender for you, but it looks like there was nothing to do.
This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12484405595. Examine the logs at this URL for more detail.


Can't type Unicode between U+0080 and U+00FF in F# Interactive on Linux and OS X

Linux Mint 18 (based on Ubuntu 16.04 LTS), F# 4.0. Unicode characters between U+0080 and U+00FF seem to be getting replaced by � (U+FFFD REPLACEMENT CHARACTER) at the F# Interactive prompt. I can include these characters in .fsx scripts (encoded in UTF-8) and run them just fine, but when I try to type (or copy & paste) the same characters at the F# Interactive prompt, I get � instead:

One line is highlighted in that screenshot because that's what I copied and pasted into the F# Interactive window -- but as you can see, the µ was replaced by �. Same thing occurred when I typed µ at the keyboard. However, using #load to load and run the .fsx file printed the correct output, so this problem is limited to the F# Interactive window.
I first encountered this behavior in https://github.com/ionide/ionide-vscode-fsharp/issues/290, but since I'm able to reproduce it entirely in the F# Interactive terminal with no IDE running at all, I believe the root cause of that bug is something in fsi.exe and not in the Ionide plugin.
Note that this bug does NOT appear to happen on Windows 10 or Mac OS X (I don't know which version of OS X, I just know that in the https://github.com/ionide/ionide-vscode-fsharp/issues/290 bug comments, someone reported that he could not reproduce the bug on OS X). Both people who have reproduced this bug seem to be running Linux.
Expected behavior
The µ character would show up correctly when typed into the terminal.
Actual behavior
All Unicode characters between U+0080 (unassigned control character) and U+00FF (LATIN SMALL LETTER Y WITH DIAERESIS) appear to be replaced by � (U+FFFD REPLACEMENT CHARACTER) when typed at the F# Interactive prompt. Other characters work fine:
rmunn@rmunn-vm-mint18 ~ $ echo 'ñòóôõöøùúûüýþÿĀāĂ'
ñòóôõöøùúûüýþÿĀāĂ
rmunn@rmunn-vm-mint18 ~ $ echo 'ñòóôõöøùúûüýþÿĀāĂ' | xxd
00000000: c3b1 c3b2 c3b3 c3b4 c3b5 c3b6 c3b8 c3b9  ................
00000010: c3ba c3bb c3bc c3bd c3be c3bf c480 c481  ................
00000020: c482 0a                                  ...
rmunn@rmunn-vm-mint18 ~ $ fsharpi

F# Interactive for F# 4.0 (Open Source Edition)
Freely distributed under the Apache 2.0 Open Source License

For help type #help;;

> printfn "��������������ĀāĂ" ;;
��������������ĀāĂ
val it : unit = ()
> #quit;;

- Exit...

Again, the text in F# Interactive was a direct copy-and-paste from the text typed into the Bash shell.
Known workarounds
No workarounds known at the moment. This is interfering with being able to use the Ionide plugin's Alt+Enter shortcut to send text to the F# Interactive window in VS Code.
Related information
Reproduced on Linux Mint 18 (based on Ubuntu 16.04) and on Ubuntu 14.04. In both cases, the fsharp package was the latest available from the Mono distribution's official Debian/Ubuntu repo. Terminal used was GNOME terminal, and value of $LANG variable was en_US.UTF-8.

Linux Mint 18 (based on Ubuntu 16.04 LTS)

FYI bug also happens in Ubuntu16.04LTS, just tested.

I believe what is happening is that the readKeyFixup function is being called when there's no need for it on Linux. It is designed to work around an older Windows bug where System.Console.ReadKey would return a byte from the system codepage, rather than a Unicode character. On any modern Linux distribution, this is unnecessary, and calling the readKeyFixup function in its current state would end up replacing any character with �. (But it is only called for characters <= 255, hence the pattern of behavior I noticed in this bug).
There are compilation flags set to skip building this function on non-Windows systems, but it appears that the Debian package was built from an older version of the code where those compilation flags had not yet been added. So this may in fact be a bug in the Debian (and Ubuntu) packaging, and not a bug in F# Interactive after all. Once I've verified that this bug does not require any changes in F#, I'll open a bug in the appropriate Debian bugtracker and close this GitHib issue.

Debian bug report opened at https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=850911.

Why is this bug closed?

@ismail @rmunn I was under the impression it was a Debian bug, and that this issue should have been closed?

So this may in fact be a bug in the Debian (and Ubuntu) packaging, and not a bug in F# Interactive after all. Once I've verified that this bug does not require any changes in F#, I'll open a bug in the appropriate Debian bugtracker and close this GitHib issue.

For me I can reproduce this on openSUSE and we do not modify fsharp in any way.

@ismail OK, reopened, thanks for checking

Yes, I should have posted a followup comment: I do think that some changes will be needed in F#'s terminal handling. I didn't post anything because I don't yet know what they are.
Thinking about it a bit more, it might be best to check the input encoding and, if it's set to UTF-8, skip the readKeyFixup function. That would require testing the Windows versions affected by the bug that readKeyFixup was written to work around, and I don't have access to those older versions. (The test would running chcp 65001 to set the codepage in cmd.exe to 65001, Microsoft's codepage number of UTF-8, and then seeing what happens in the F# Interactive REPL).

Also note that #594 is a reproduction on OS X, so this affects more than just Linux. I'll edit the issue title accordingly.

May be causing this issue of mine: http://forum.pdfsharp.net/viewtopic.php?f=2&t=3553&p=10681#p10681 I must have spent 2 days on this not realising that it's may be the F# interactive that breaks my invoicing.

@haf could you check this is fixed by tag 4.1.5 please? thanks

How do I test it?

@rmunn could you check this is fixed by tag 4.1.5 please? thanks

@dsyme Confirmed fixed on openSUSE, thanks!

@ismail lovely, thank you!

@dsyme And I can confirm that tag 4.1.5 fixes the bug on Linux Mint 18 (which is based on Ubuntu 16.04). I still can't build the visualfsharp repo for the reasons I mentioned in https://github.com/Microsoft/visualfsharp/pull/2582#issuecomment-287004676, but I've been able to build this repo and the fix is working here.

Further confirmation: I just installed the latest .deb package built from the https://github.com/mono/linux-packaging-fsharp/ repo, on a machine where I had not yet done anything to fix this bug manually. In other words, the laptop that I'm typing on right now was exhibiting this bug immediately prior to installing version 4.1.6-0xamarin1+debian7b1 of the Mono-built fsharp package, and immediately after installing version 4.1.6-0xamarin1+debian7b1, the bug went away and I could type ä properly in F# Interactive.


feat: EF Interface

Closes #48

Merge activity

Aug 14, 7:37 AM EDT: @tazarov started a stack merge that includes this pull request via Graphite.


Scale zoom values in the SpriteFrames editor for hiDPI displays

We also need to reset the zoom correctly in other places, so we need to change more than a single line actually.
This closes #46836.

Thanks!

Needs a dedicated PR for 3.2, the code seems to have changed significantly.

@akien-mga It should be cherry-pickable now since #48977 was merged (I didn't notice this PR earlier).

@akien-mga I've done it manually cause of changes in #48984.


CultureInfo still doesn't work with non-customized Fixture instance

I was happy to see the addition of the InvariantCultureGenerator, as a result of #427. I just upgrade one of our projects to the latest version, but unfortunately it throws.
I though it had something to do with my unit test project (it's also using AutoConfiguredMoqCustomization), but it can be reproduced with the following unit test, in a new project:
        [Fact]
        public void CreateWithUsualPublicEntryPointCrashes()
        {
            var fixture = new Fixture();
            fixture.Create<CultureInfo>();
        }

Am I doing something wrong, perhaps?
Full stacktrace

Thank you for making me aware of that. It's quite an embarrassing oversight on my part, but it should now be fixed in AutoFixture 3.34.2.

Cool! I was already looking at the "composition root" in Fixture.cs:63, how to fix that, but I didn't completely understood how that works. Do you have that documented somewhere, or a blogpost about how that works?
Thanks for the quick fix, btw :+1:

Documentation is always the weak part, but there's the high-level architecture document that describes some of it: https://github.com/AutoFixture/AutoFixture/wiki/Internal-Architecture
HTH


Update how-to video and add the training button on FOM public

Describe the task
Update the how-to video link on FOM public site, "About" page and the popup they open FOM website. Add a new button on the top navigation bar, same as the admin site, link to the how-to video as well.
Acceptance Criteria

[x] Update the how-to video link on FOM public About page
[x] Update the how-to video link on FOM public popup
[x] Add a training button on top navigation, and link to the how-to video

Additional context

New how to video link: Introducing the new Forest Operations Map (FOM) Online Portal (bcgov.github.io)

Hi Olga @OlgaLiber2, do we want to open the how-to videos in a new tab, or in the current tab?

@MCatherine1994 new tab with the icon indicating it'll open in a new window (like this one https://thenounproject.com/icon/new-window-299638/)

Thanks @OlgaLiber2 , do you mind have a quick review on it? https://fom-44.apps.silver.devops.gov.bc.ca/public


[T4A6][W15-B3]Zachary Lim Wei Ling

Use JUnit to implement unit tests [LO-JUnit]

@zachylimwl
Some comments added. Please ack & close the PR after reading comments.


Feature: Move the search debounce logic to its own hook

Type of feature
🧑💻 Refactor
Current behavior
After some discovery looking for components that need testing, I saw that this logic was not moved and it needs to be to keep all components flexible and reusable.
https://github.com/open-sauced/hot/blob/main/src/components/PrimaryNav.tsx#L19-L38
Suggested solution

Move this debounce function to a hook
Tests are required for this contribution.

Additional context
https://kentcdodds.com/blog/how-to-test-custom-react-hooks
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[X] I agree to follow this project's Contribution Docs

This is solved by rooks


User Agent parser missing quot cases

https://github.com/kurotych/sipcore/blob/0ca37a4834e58c4eefeeb6f29fd1271746f99f45/crates/sipmsg/src/headers/parsers/user_agent.rs#L22
In some SIP samples from Wireshark project contained the following UA :
User-Agent: <Motorola VT1000 mac: 000CE5C74EF8 sw:VT20_02.03.00_A ln:0 cfg:1253778536520/1002173358>
This fails because quot cases not handled, maybe :
            if tmp_input[0] == b'<' {
                let (input, _) = take_sws_token::laquot(tmp_input)?;
                let (input, _) = take_until(">")(input)?;
                let (input, _) = take_sws_token::raquot(input)?;
                tmp_input = input;
                continue;
            }

Hello @q6,
Thank you for contribution. User agent values are really crazy thing.
I've added your fix to master and released Sipmsg 0.2.3 version.
Best regards,
Anatolii


Have question on probability of duplicate and on seqnum result

Hi, I implemented similar solution but just trying to understand the probability of duplicates after we hit the max of seqnum:5 scenario. After 99999 would it again start with 10000 or fail? To be clear I'm using this one {SEQNUM:5}-{RANDSTRING:6}. Incase of probability I assume it will be like (1/91/91/91/91/91/361/361/361/361/361/36)

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

ID: 819aaec0-7a0e-49dd-9f68-42c58af5e79a
Version Independent ID: 2ab2400d-5e6a-434d-980e-0989492464f3
Content: Create auto-number-attributes (Developer Guide for Dynamics 365 Customer Engagement (on-premises))
Content Source: ce/customerengagement/on-premises/developer/create-auto-number-attributes.md
Service: crm-online
GitHub Login: @NHelgren
Microsoft Alias: kabala

The number sequence value provides trailing or leading zeros based on your seed value.
So if you put in {SEQNUM:5} and you were on record 1 you would start displaying 00001.  If you put in {SEQNUM:5} and you were on record 10000, you would start at 10001.  If you put in {SEQNUM:5} and you were on record of 1000, you would start with 01001.
When you hit the max number for the number value range, it will continue counting upward from the last number added. Auto number will always pick the next sequential number based on the data it has about the last used number.
Ex: {SEQNUM:5}-{RANDSTRING:2} would do the following if your current record was 9998:
9999-DM
10000-XJ
10001-RL

Thanks @NHelgren but it's the prefix I want to change based on the value of another field, not the number format. Thanks though

#close


uuid column throws when defined as search field

Hi.
I have an issue when I add a PostgreSQL uuid or jsonb type field to the searchableColumns.
I get such errors:
QueryFailedError: operator does not exist: jsonb ~~* unknown
    at PostgresQueryRunner.query (PostgresQueryRunner.ts:299:19)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at SelectQueryBuilder.loadRawResults (SelectQueryBuilder.ts:3601:25)
    at SelectQueryBuilder.getRawMany (SelectQueryBuilder.ts:1573:29)
    at SelectQueryBuilder.executeEntitiesAndRawResults (SelectQueryBuilder.ts:3295:26)
    at SelectQueryBuilder.getManyAndCount (SelectQueryBuilder.ts:1776:36)
    at paginate (node_modules/nestjs-paginate/src/paginate.ts:316:28)
    at AuditLogService.get (audit-log.service.ts:600:22)
    at node_modules/@nestjs/core/router/router-execution-context.js:46:28
    at node_modules/@nestjs/core/router/router-proxy.js:9:17

I tried to create a querybuilder, pass it to paginate (which by default works) and tried to remove my jsonb field and add it manually like this:
let queryBuilder = this.auditLogRepo.createQueryBuilder();
if (query.search) {
    queryBuilder = queryBuilder.orWhere('payload::text ILIKE :search', {
        search: `%${query.search}%`,
    });
}

This however does not work. It only works if I delete query.search, but obviously then it will not search for the other searchableColumns.
Is there any solution to this?

Oh I see. Is that the same as https://github.com/ppetzold/nestjs-paginate/issues/359 ?

Similar issue. JSONB columns have no coverage. Hence have not been considered with any of the features.
Happy to accept a bug fix here :)

bug fix for the uuid issue. for JSONB support, I will create a separate issue

Duplicate #436

Fixed with #442


feat(loot): add item allowlist and denylist

Allowlist: useful for bingo events or untradeable drops below gp threshold
Denylist: useful to ignore common drops that exceed gp threshold

Should allowlist override screenshot min value?

I don't think so, unless someone can come up with a compelling argument for it.


Hide Authorization header credentials in verbose mode

When the request headers are being echoed back, I'd like the default behavior to be to **** out the credentials.
Instead of this being displayed:
Authorization: Basic aHR0cGllOmlzQXdlc29tZQ==
I'd like to see:
Authorization: Basic ************************
Similarly:
Authorization: Bearer *******************************
Maybe a new option (like --show-creds) could be used to show the credentials.

Good idea. People accidentally share sensitive data like this all the time. Thinking out loud:

Having this on by default goes slightly against having the output verbatim.
Authorization isn't the only header with sensitive values out there (think all the Token, X-API-Token, etc.).

Hello,
Any update regarding this issue ?
Is it possible to implement something similar to what is done on httpx
Thx


document helm install on GKE

There is nothing special to do for GKE, or am I missing something?

agreed


Refactored client, removed vendored gql libs

https://wandb.atlassian.net/browse/WB-5747
Description
This removes a couple vendored packages and makes a custom GQLClient that gives us more control over our client.  An added bonus is removing subprocess32 as it was only for python2.
Testing
How was this PR tested?
Release Notes
Below, please enter user-facing release notes as one or more bullet points. If your change is not user-visible, write NO RELEASE NOTES instead, with no bullet points.
------------- BEGIN RELEASE NOTES ------------------
Added a dependency on graphql-core>=3.0.0
------------- END RELEASE NOTES --------------------

I defer to @raubitsj on this, but @vanpelt this is really cool to see


incorrect escaping in web ui

When selecting an item for a cross reference, the top line of the bottom pane where the definitions, declarations, and references are shows something like:  kythe:?lang=c%2B%2B?path=foo/bar.h#V%2FrptxjJiN90TQfl8yoLYH5kmoIPm8%2BplQfZPPqL90w%3D
I expect this is meant to be human readable and the language should at least show as c++ rather than c%2B%2B.  I'm not sure about what should be shown after the #.

That string is a Kythe URI, and it's correctly encoded as far as I can tell, but I agree that it's probably not the most intuitive thing to print out in the UI. The sample UI is intentionally a little skeletal in this regard.


Update READMe.md - code typo

rename variable name "service" to match parameter name "services".
Thank you for creating all of theese amazing packages Jon!

Thanks @chrisK00


avx512f: implement _m512i_private_to_mmask8,_mm512_mask_cmpeq_epi64_mask

As I mentioned last night, for some reason ppc64le and s390x on Travis were really slow to start (if they ever did) yesterday.  I did merge your patches into some stuff I was working on, and eventually got a full build out of Travis, so they're merged now.
As always, thanks for the patches!  Sorry about the issues with CI :(

As I mentioned last night, for some reason ppc64le and s390x on Travis were really slow to start (if they ever did) yesterday. I did merge your patches into some stuff I was working on, and eventually got a full build out of Travis, so they're merged now.
As always, thanks for the patches! Sorry about the issues with CI :(


GetMergeCells() return 'parameter is invalid' err

The excel is generated through a third party program.
merge.xls
GetMergeCells() got the 'parameter is invalid' err In the 'A1' axis when i use of the excelize to read.
Hope can be compatible with the situation, Thanks !!!

This library doesn't support the legacy XLS file format in your attachment.

Thanks for your answers

I think this problem is caused by a worksheet that contains one cell merged cell range. I will consider adding compatibility with that.


LibsDisguises incompatibility - MetaIndex not found for GLOW_SQUID!

Spark link
https://spark.lucko.me/2XzFN9mPQR
Expected behavior
No LibsDisguises start-up error / incompatibility.
Observed/Actual behavior
[11:00:44] [Server thread/ERROR]: [LibsDisguises] MetaIndex not found for GLOW_SQUID! Index: 17
[11:00:44] [Server thread/ERROR]: [LibsDisguises] Value: orange (class java.lang.String) (class net.minecraft.world.entity.GlowSquid) & GlowSquidWatcher

Steps/models to reproduce

Add LibsDisguises dev build to plugins folder.
https://ci.md-5.net/job/LibsDisguises/
Add ProtocolLib dev build to plugins folder.
https://ci.dmulloy2.net/job/ProtocolLib/
Start Purpur
https://purpurmc.org/downloads
See error in startup log.

Purpur version
git-Purpur-1838 (MC: 1.19.2)
Agreements

[X] I am running the latest version of Purpur available from https://purpurmc.org/downloads.
[X] I have searched for and ensured there isn't already an open issue regarding this.
[X] My version of Minecraft is supported by Purpur.

Other
RE: https://github.com/libraryaddict/LibsDisguises/issues/675
Using Paper dev 263, LibsDisguises does not seem to have the start-up error.
Purpur dev 1838 (Paper 1.19.2)
LibsDisguises dev 1132
ProtocolLib dev 602

This is probably due to Purpur's implementation of rainglow, which just adds an NBT value to glow squids. Not sure how it could mess with LibsDisguises though.

.. I thought there must be a good reason that LD is having compatibility issues, only to see that purpur is adding custom mods.
Frankly, I'm leaving the error as is. It's supposed to report when metadata has been mapped incorrectly in Lib's Disguises. But Lib's Disguises isn't in the wrong here.

We're not adding custom mods... We added a data watcher, and LD is tripping up on that for whatever reason. Which it probably shouldnt't, seeing as any project (plugin or server fork) can add more data watchers at will.

Its a harmless error
You modded the server infar as that it wants a clientside mod to perform extra functions. Any server fork can do this yes
I don't think any plugin can do this, I believe what they do instead is extend off the entity to create a new class. Which I think LD has no problems with. You're modifying the base server files.

Oh, well if it's just a harmless printerr I don't see what the fuss is about. I was under the impression there was an incompatibility which prevented LD from working as intended.


Swift 5.10 Workflow

Added a CI workflow for Swift 5.10.
@yim-lee If everything looks good, could you please add the CI setup for this?

@swift-server-bot test this please

@swift-server-bot test this please

CI for Swift 5.10 is set up and passing.
Just want to point out that the filename deviates slightly from the others. Its extension is .yml rather than .yaml.

Its extension is .yml rather than .yaml.

Good catch! I wonder if Xcode did that when I ⇧⌘S’ed to duplicate the file? I won’t be at a computer for another few hours, but I’ll poke you when I fix it, thanks!

Well that is an interesting Xcode bug…
https://github.com/swift-server/webauthn-swift/assets/225505/7690cbad-f8ef-462e-a324-6b3c7dfdec1e

@yim-lee I updated the filename if you could fix the configuration, sorry about that!

@dimitribouniol No worries. CI updated.
@swift-server-bot test this please


08/ 1.A.1)  As developer I want to discuss which authentication we will use to implement it

User Story
As developer I want to discuss which authentication we will use to implement it
Description of the solution you'd like
Gather the group and write a short explanation of why we are choosing the solution we are.
Definition of Done
Comment the candidates and a fairly thorough explanation of why.

We have agreed to pursuit the B2C authentication


Update Not Working/Working to Offline/Online

Linked issue
Resolves: #xxxxx
What kind of change does this PR introduce?

[ ] Bug fix
[ ] New feature
[x] Refactor
[ ] Docs update
[ ] CI update

What is the current behavior?
Currently the Online offline behaviour is described as Working/NotWorking
What is the new behavior?
It now says Online/Offline
Other information
Additional changes, explanations of the approach taken, unresolved issues, necessary follow ups, etc.
Requirements (all must be checked before review)

[x] The pull request title follows our guidelines
[ ] Tests for the changes have been added (if relevant)
[x] The latest changes from the main branch have been pulled
[x] pnpm lint has been run successfully

Changes look good to me!
One thing I noticed while looking at it - in topic overview, we show the cluster status (with "online" and "offline" already!) but it's all-caps:

I think it would make sense to streamline that and have either "Online" or "ONLINE" everywhere.
Maybe Venla can say something about this?
(But we don't have to do that in the same PR)

Hi! Great question. 👍
Generally we should avoid spelling in ALL CAPS. It is generally faster to read lowercase letters.
If we think empathising information is needed then we can think of other ways like adding icon vs. ALL CAPS. For example in Console the service state is "Running" with "tickCircle" icon.

If we use icons (which could be nice I think), should we also think about using them e.g. here for all states?

Or would this get too much 🤔
I'll at leat make an issue for not using all caps for the cluster state!


How to add the certificate to elb-loadbalancer

Getting the error while creating the Loadbalancer using hiera infrastructure
Error: Could not set 'present' on ensure: Secure Listeners need to specify a SSLCertificateId
Error: Could not set 'present' on ensure: Secure Listeners need to specify a SSLCertificateId
Wrapped exception:
Secure Listeners need to specify a SSLCertificateId
Error: /Stage[main]/Main/Elb_loadbalancer[ph-att-prod-sllb]/ensure: change from absent to present failed: Could not set 'present' on ensure: Secure Listeners need to specify a SSLCertificateId
The documentation doesn't talk about the certificate Information how to use it
elb_loadbalancer:
my-lb:
ensure            : present
instances         : [ test-01 ]
listeners         : [ { 'instance_port' : '80', 'instance_protocol' : 'HTTP', 'load_balancer_port' : '443', 'protocol' : 'HTTPS' } ]
region            : us-west-2
scheme            : internet-facing
security_groups   : [ plb-secgrp ]
subnets           : [west2b_subnet]

Same error here, and probably same for cipher set.
I tried a couple options like variations of ssl_certificate_id, or adding it to the template, but to no avail.
Is it supported?
Cheers

I'm afraid at the moment this isn't supported, but a couple of folks have taken a run at PRs to add this functionality: https://github.com/puppetlabs/puppetlabs-aws/pull/219

Yeah I saw it later on and actually got it working from #214.  I'll try #219 too as it looks more robust. Thanks.

@anishnath @bahrt, I believe this was addressed by #214. Can I close this issue?

yes.. Thanks


fix(docs): Set expectations for typo PRs

This change is in place to set the expectation for what will happen with
small typo fixes, synonym replacements, etc.
This policy is being added as there are too many PRs opened that either
rename small fragments, rephrase synonyms or fix typos. The goal is to
limit those and focus the energy/ attention on improving the
spellchecker.
This policy eases the "review PR" procedure. It's much easier to say
"no, here's a link why" and close the PR, rather than filtering through
the subjective opinion of the reviewer and figure out if removing some
spaces or replacing "enhance" with "empowers" better fits the
description.

❤


Thread Table not updating properly

I just noticed a mismatch between the # Threads sparkline (48) and the numbers of threads in the threads component (51). I had to do a full page reload to pick this up. Probably, the logic generating the threadTable is only adding or updating threads and neglecting to drop them if they are no longer running.

Moving back to critical path


Flag pre-release tags as such

gh-release action can take a prerelease option:
https://github.com/softprops/action-gh-release#inputs
We should do something like:
  with:
    prerelease: ${{ contains(github.ref_name, 'a') || contains(github.ref_name, 'b') || contains(github.ref_name, 'rc') }}

This is only an issue if we make tags locally and create actions, as "draft action" will give people an option for this

Change this to:
Add documentation Note: check pre-release if this is not production ready release

I've added this to skeleton.

I've also added a note in the docs


How to auto get value d?

hi
Could you tell me How to auto get value d?
thanks!

It's not a silver bullet, but here are some tips and tricks in the user guide:
https://www.alkaline-ml.com/pyramid/tips_and_tricks.html#understanding-differencing

It's also worth mentioning that auto_arima will estimate d if it's set to d=None. You can control the number of differencing terms it will try with the max_d parameter.


iOS Import opml not working.

Hi
Firstly I love NetNewsWire, it is a rss reader I was looking for.
I am trying to import my opml from mac NetNewsWire, however the file is greyed out and I cant select it.
Steps I am doing:

Export from mac NetNewsWire save it to iCloud
Go to iOS -> settings-> Import Sub -> Select iCloud Drive
I can see the file however it is greyed out and I can't click on it.

I am not sure if I am doing something wrong, however this use to work when I did it earlier in December.
Thanks
L

It’s a bug — we’re working on it. :)
Duplicate of #1443.

so sorry, should of done a search rather then wing straight away.

Quite alright. It’s our job, not yours, to groom the bugs database. 👍

Closing since this is a duplicate.


[Question] HEVC 10bit decode support

The open source LibVA from 01.org does not support HEVC 10bit HW decode on SKL. Does the Media SDK provide that support for SKL? I was able to get 8bit fixed function HEVC decode going with the 2017 SDK. Want to understand what is supported for the 10bit decode. Is it Software only or is it a hybrid implementation?
What about KBL support for 10bit fixed function HEVC decode in the latest SDK?

Hi Rohit, thanks for your questions.
There is no 10bit HEVC support on Skylake hardware and we have no immediate plans to support Kabylake in the Media Server Studio or respective open source Media SDK version.
I'm not aware of any Intel graphics-accelerated 10bit HEVC decode solutions, but there is a number of the 10bit encoders from the leading compression technology vendors optimized for Intel, few examples are:
Beamr: http://media2.beamrvideo.com/pdf/Beamr_Intel_NAB_2017_PR.pdf
GigPeak: http://www.gigpeak.com/press-release/gigpeak-to-demonstrate-industry-first-encoding-solution-at-nab-2017/
Ittiam: http://www.ittiam.com/newsroom/news/2017-2/ittiam-hpe-announce-highly-efficient-scalable-hevc-hdr10-solutions-nab-2017/
Let me know if you have any other questions.
BR,
Vladimir Plichko, Intel


Remove jackson databind version override

This should get inherited from the common pom

It looks like @arvindth hasn't signed our Contributor License Agreement, yet.

The purpose of a CLA is to ensure that the guardian of a project's outputs has the necessary ownership or grants of rights over all contributions to allow them to distribute under the chosen licence.
Wikipedia

You can read and sign our full Contributor License Agreement here.
Once you've signed reply with [clabot:check] to prove it.
Appreciation of efforts,
clabot


Better validation errors

With this PR we enable reporting of the validation path in some cases in order to display more detailed validation errors to the users.

Hm, seems like my branch was broken. Sorry. Only the last commit is the interesting one, the rest should be in master already.


Use const instead of var

Because it will never change

Merged #51 instead of this commit. Thank you for looking at this! If you see anything else I'd be more than happy to accept a PR 😄


Remove redundant ISpyB params

ISpyB has 3 parameters:

focal_spot_size_at_samplex
slitgap_horizontal
beamsize_at_samplex

Where the 3rd parameter is always just calculated as the smallest value from the other 2. This implies that the first 2 are redundant for data analysis. We should remove them and confirm it still works.
Acceptance Criteria

Redundant parameters are not added to ISpyB etc.
This is tested on a beamline

ispyb_params soon to be wholesale removed so this is redundant


python310Packages.bleak-esphome: 0.4.0 -> 0.4.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.bleak-esphome is: Bleak backend of ESPHome
meta.homepage for python310Packages.bleak-esphome is: https://github.com/bluetooth-devices/bleak-esphome
meta.changelog for python310Packages.bleak-esphome is: https://github.com/bluetooth-devices/bleak-esphome/blob/0.4.1/CHANGELOG.md
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.4.1 with grep in /nix/store/cv8n0fvgmwzna646mivs7nznrvd5mvv2-python3.10-bleak-esphome-0.4.1
found 0.4.1 in filename of file in /nix/store/cv8n0fvgmwzna646mivs7nznrvd5mvv2-python3.10-bleak-esphome-0.4.1

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
python310Packages.bleak-esphome
python311Packages.bleak-esphome

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/cv8n0fvgmwzna646mivs7nznrvd5mvv2-python3.10-bleak-esphome-0.4.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.bleak-esphome https://github.com/r-ryantm/nixpkgs/archive/8027fe5d5b88decff39b94a60606d138d521caa1.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/8027fe5d5b88decff39b94a60606d138d521caa1#python310Packages.bleak-esphome

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/cv8n0fvgmwzna646mivs7nznrvd5mvv2-python3.10-bleak-esphome-0.4.1
ls -la /nix/store/cv8n0fvgmwzna646mivs7nznrvd5mvv2-python3.10-bleak-esphome-0.4.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

python310Packages.bleak-esphome
python311Packages.bleak-esphome

Maintainer pings
cc @fabaff for testing.

Superseded by #278272


[9.x] route:list Add check to detect method that not exists

Check class's method that not exists
php artisan route:list --check

Description:
Throw error if action/method declared in route, but not yet declared inside class
Example screenshot:

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


libcap and libcap-bin for cpu model: MIPS 1004Kc V2.15

openclash version: luci-app-openclash_0.42.05-beta_all.ipk
device: Xiaomi Mi Router 4
cpu model: MIPS 1004Kc V2.15
OPENWRT_RELEASE="OpenWrt 19.07.2 r10947-65030d81f3"
Where can I download libcap and libcap-bin for cpu model "MIPS 1004Kc V2.15"?
PS: In General Settings, there is no option for MIPS 1004Kc V2.15 as well, which one should I use instead?

Try core which build for mipsle-softfloat, the version showing on the update page if correct core installed.
Download libcap and libcap-bin here https://mirrors.tencent.com/lede/snapshots/packages/mipsel_24kc/base/


"Unauthorized: Full authentication is required to access this resource"

Seeing this message with the recent upgrade of Spring Boot when trying to post content to the http app deployed as part of the stream.
Steps to reproduce:

stream create --name httplog --definition "http | log"
stream deploy httplog
http post --contentType 'application/json' --target http://dataflow-server-kl-RzTsaC7-httplog-http.cfapps.haas-89.pez.pivotal.io --data "{"name": "Foo"}"

Error message:

POST (application/json;charset=UTF-8) http://dataflow-server-kl-RzTsaC7-httplog-http.cfapps.haas-89.pez.pivotal.io {"name": "Foo"}
401 UNAUTHORIZED
{
"path" : "/",
"error" : "Unauthorized",
"message" : "Full authentication is required to access this resource",
"timestamp" :<PHONE_NUMBER>621,
"status" : 401
}
Error sending data '{"name": "Foo"}' to 'http://dataflow-server-kl-RzTsaC7-httplog-http.cfapps.haas-89.pez.pivotal.io'

This has been addressed in 1.1.2.RELEASE via: https://github.com/spring-cloud-stream-app-starters/http/compare/v1.1.2.RELEASE...master. You can download the 1.1.2.RELEASE from here.

That's super fast response, Sabby! Thanks!


[DO NOT MERGE] feat(delivery): hack on deployment tests

Closing for archival/record-keeping purposes.
Continued in #482.


Make the EventEmitter returned by query() thenable

There have been some discussions about promise support, but those are very old. It is pretty clear that the future is moving towards promises / ES7 async-await.
Wrappers like promise-mysql have the problem of not always being up-to-date, hiding the EventEmitter by returning a promise and not implementing all APIs (like pools).
I would propose to add a then() method to the EventEmitter returned by query() and other async functions that take a callback, therefor implementing the promise interface.
The old API would not break in any way, but it would allow users to await a query.
It could also be beneficial to add the 2 other standard promise functions, catch() and finally(), otherwise users would just have to wrap the call in Promise.resolve() if they want to use these features.
The callbacks passed to then() would be called like the callback. The error callback is called with the error, the success callback with an object {results, fields} just like the arguments for the callback are named in the docs.
The performance of this is negligible in the context of the IO roundtrips, as already mentioned here: #929. If a callback is passed, the functions should not return a promise/expose .then() so if you absolutely love callbacks you can use them without a performance impact.
A simple example: Adding a JSON object of a book with an array of authors into a db with transactions and proper error handling.
This is how we would write it today. Total callback hell.
let book = {name: 'test', authors: ['author1', 'author2']};
let db = pool.getConnection((err, db) => {
  if (err) {
    return req.statusCode = 500;
  }
  db.beginTransaction(err => {
    if (err) {
      req.statusCode = 500;
      return db.release();
    }
    db.query('INSERT INTO books (name) VALUES (?)', [book.name], (err, result) => {
      if (err) {
        req.statusCode = 500;
        return db.rollback(err => {
          db.release();
        });
      }
      let bookId = results.id;
      let completed = 0;
      let errorHappened = false;
      for (let author of book.authors) {
        db.query('INSERT INTO book_authors (book_id, name) VALUES (?, ?)', [bookId, author], (err => result) {
          if (err) {
            req.statusCode = 500;
            errorHappened = true;
            return db.rollback(err => {
              db.release();
            });
          }
          completed++;
          if (completed === book.authors.length) {
            db.commit(err => {
              if (err) {
                req.statusCode = 500;
              }
              db.release();
            });
          }
        });
      }
    });
  });
});

It gets slightly better with promises, because you get error handling and parallel execution for free:
let book = {name: 'test', authors: ['author1', 'author2']};
pool.getConnection()
  .then(db => db.beginTransaction())
  .then(db =>
    db.query('INSERT INTO books (name) VALUES (?)', [book.name])
      .then(answer => {
        let bookId = answer.results.insertId;
        return Promise.all(book.authors.map(author => db.query('INSERT INTO book_authors (book_id, name) VALUES (?, ?)', [bookId, author])))
      })
      .then(() => db.commit())
      .catch(error => db.rollback().then(() => Promise.reject(error)))
      .finally(() => db.release())
  )
  .catch(error => {
    req.statusCode = 500;
  })

But it gets even better with ES7 (you can use this today either with Babel or with co by replacing all awaits with yield):
let book = {name: 'test', authors: ['author1', 'author2']};
try {
  let db = await pool.getConnection();
  await db.beginTransaction();
  try {
    let bookId = (await db.query('INSERT INTO books (name) VALUES (?)', [book.name])).results.insertId;
    await* book.authors.map(author => db.query('INSERT INTO book_authors (book_id, name) VALUES (?, ?)', [bookId, author]));
    await db.commit();
    req.statusCode = 201;
  } catch (error) {
    await db.rollback();
    throw error;
  } finally {
    db.release()
  }
} catch (error) {
  req.statusCode = 500;
}

You can achieve this with bluebird today, but in every file, every test you write, you have to do the following and also add an Async suffix to all methods:
let mysql = require('mysql')
let Pool = require('mysql/lib/pool')
let Connection = require('mysql/lib/connection')
Promise.promisifyAll(Pool.prototype)
Promise.promisifyAll(Connection.prototype)

To sum it up:

add then() to the EventEmitter returned by query() and make all other async functions which don't return anything atm return a promise.
Don't return a promise / add .then() when called with a callback.
keep all the API the same.

I would make a PR implementing all of this. Thoughts?

Feel free to make a PR :) I only have three real requirements: it does not pull in a dependency, it's backwards-compatible, and it only implements the .then method as defined for thenables in the ECMA spec.

Awesome, I'm fine with that. Thanks for the quick reply :)

No problem! I'm looking forward to the PR :) I've been meaning add thenable to this lib, but still have not gotten to, so your contribution will be much appreciated!

One thing though, some async functions currently return nothing, like Connection.prototype.connect(). It would make sense to return a full promise there if no callback is passed, and I would like to use any-promise for this, so people can choose which promise library they want to use. Native promises are much slower currently than libraries like bluebird and bluebird for example also has some nice debug features ("possibly unhandled rejection: ...").
Is it ok if I add this dependency?

Looking at that dependency, it looks like it'll throw an error for all our Node.js 0.6 and 0.8 users, as simply requiring the library will throw by default on those platforms. It would also mean those users cannot call the .then method either, which needs to be possible even if there is no Promise library installed.

This can easily be solved by doing:
try {
  var Promise = require('any-promise')
} catch (error) {}

I would add any-promise to optionalDependencies (because that's what it is - you can use promises but you don't have to). If we support node all the way back to 0.6 we have to check for the existence of any promise implementation anyway to not trigger a ReferenceError whenever someone calls something without a callback.

But there needs to be some way to use a promise library instead of native, and any-promise is the most elegant imo.

As long as Node.js 0.6 users can do conn.connect().then(function () {}) without having any Promise library installed.

But there needs to be some way to use a promise library instead of native, and any-promise is the most elegant imo.

Why are we talking about promise at all? The only agreement here was that this library would return thenables, as defined by the ECMA spec. This library will not return promises, only thenables.

I thought it would make sense to return a new Promise() for functions that currently return nothing, like connect(). But I'm okay with thenables in these cases too.

I mean, it sounds good on the surface, but from a consistency point of view, none of the features should depend on what version of Node.js or other installed modues are, in our opinion. This means that if you want to do conn.connect().then(...) with version 2.7 of this module.
The problem is if we provide features that depend on non-dependency modules (like a parallel Promise library) or Node.js runtimes, when this module is a sub-dependency of another module (which it is--a LOT, since this is a low-level module users typically do not directly use), end-users end up with inconsistent results when those intermediates start using those features.
An example of this problem is from using something like any-promise: moduleA will expect this module's promises to be that of Bluebird and then use some Bluebird-only extensions on those returned promises. That module even puts bluebird as a dependency to try and make sure that happens. Then a user tries to use the library, but has PROMISE_IMPL=q in the environment, causing mysql to return Q promises, breaking that intermediate dependency.
In the end, this module using any-promise and supplying an API that is a moving target is at fault in the above scenario. This is not a made-up scenario, either, because I've been in "promise implementation hell" several times from these so-call "smart promise shim" libraries like any-promise being in various dependency chains interacting poorly.

Regarding my last comment: Actually, that will not work. Per specification a thenable is an object that implements a then method. Per specification, a then method must return a (full) promise. And I'm not thrilled to write my own full promise implementation, that would also highly violate dry (or in this case, dont-reinvent-the-wheel). Promises are a language feature, and if a user is on an older version of Node which hasn't implemented this language feature yet, it should be common sense to him that he can't use it with this module - except if he installs a promise library, which I would always recommend anyways.
You are right of course, and the number of own dependencies should be kept low in general.
The other option would be to do it like MongoDB does it and have the promise constructor as a connection option. That way a user has to explicitly define what promise library will be used for constructing promises just in that connection/pool and they can't interfere.
But there needs to be some way to specify it because like I said .then() must return a new promise and native promises are far superior to the established libraries in performance and debugging features.

like I said .then() must return a new promise

Gotcha. My understanding was based off https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Promise/resolve which shows that the then property can be a plain function that takes two function arguments. It would let you do Promise.resolve(conn.query(...)).then(...) as far as I could tell. I guess I'm mistaken?
As for anything else, there are, in the end, we have the agreement above on what the requirements would need to be to accept a PR. If you cannot implement it in a way that would provide a consistent API for all supported Node.js versions, then we can old the PR until the next major version of this library, whenever that would be, if necessary.

Yeah you could do Promise.resolve(conn.query(...)).then/catch/finally(...).then/catch/finally(...) but if we write in the docs that the EventEmitter is now thenable they will try to do things like this:
conn.query(...).then(...).then(...);
conn.query(...).catch(...);
var query = conn.query(...);
query.then(...);
query.then(...);

which I cannot implement without having some to a real promise implementation, if I don't want to roll my own.
In the simplest implementation, .then() would just set the _callback property. That would mean you cannot call .then() multiple times, you cannot use it together with a callback, you cannot start a promise chain with .then(), you cannot call .then() after the query has already ended, you cannot call .catch() and all the other methods. It would require only minimal changes to the codebase but you could do everything you want if you do await conn.query(...) or Promise.resolve(conn.query(...)). All of this would need to be highlighted in the docs. Node 0.6 users could use the .then() method, but as it just acts like passing a callback, it wouldn't benefit them - except if they do Promise.resolve(). Not spec compliant, but consistent across Node versions.
In the a little more complicated approach, query objects make a full internal promise object when you call .then() and basically proxy the .then() method to it. The promise constructor would be passed in as a connection option and would fallback to a global Promise. This implementation means that you can only use the .then() method if you either use a new Node version or pass in a promise library, but you would have a full chainable promise at hand (it would be debatable if catch and finally should be added too).
Most complicated to implement would be to make Query objects implement a Promise as per spec. It may only expose .then(), but it must take care of its state, allow multiple calls to .then(), and return a new promise - we would end up with just another promise implementation and this is not desirable.
I will implement option 1 and when version 3 is coming and Node version requirement is raised I may revisit it and implement option 2.

This is much harder than I thought it would be. No problem with the functions that talk to the server - they all return some instance of Sequence so .then can be added to Sequence.prototype.
But what about the methods that don't talk to a server? Look at Pool.prototype.getConnection - if the pool is already closed, it calls the callback with an error, without returning any Sequence/EventEmitter.
Then there's Pool.prototype.acquireConnection - it is async, but when the connection was aquired from the wrong pool, it throws synchronously and unleashes Zalgo. The only thing that makes sense to return from these methods is a promise. There is no EventEmitter that can be extended with a .then() method. But I cannot return a promise if the requirement is to make all features work with 0.6....
And I didn't even talk about PoolCluster....
TL;DR it is impossible to implement this feature for all node versions correctly.
I would propose to add true promise support in 3.0, if the node requirement is raised to 0.12.

Hi,
I wrote this small piece of code that lets me allow do async queries
(I am really a fan of clean-code so I am trying to callbacks as much as I can :))
I use it like this
async function getUser(id) {
  const users = await db.queryAsync('SELECT * FROM users WHERE id=?', [id]);
  return users.length > 0 ? users[0] : null;
}

const user = await getUser(10);
console.log(user ? 'Hello ' + user.firstName : 'No user was found');

And this is the queryAsync addition I made:
const db = Mysql.createPool(options);

db.queryAsync = function (...args) {
  return new Promise((resolve, reject) => {
    args.push((err, result) => {
      if (err) return reject(err);
      resolve(result);
    });
    db.query.apply(db, args);
  });
};

If you'd like it let me know and I can send a PB.

@AtaS thats exactly what bluebird does

Hmm yes it is indeed :)
Ata Sasmaz
Linkedin: www.linkedin.com/in/atasasmaz
On 15 April 2016 at 09:55, Felix Becker<EMAIL_ADDRESS>wrote:

@AtaS https://github.com/AtaS thats exactly what bluebird does
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/felixge/node-mysql/issues/1253#issuecomment-210369541

How about we provide .then() method, in it check that global Promise exists, if not then throws error You need node > 0.12.18 to use query.then()?


Automatic PR of restapi_auto_yanivid3_master into dev

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/2340

Closing for now, in response to #1057. These changes will be incorporated shortly.


fix(playground): start playground net error

change HOST from local.neteasa.com to <IP_ADDRESS> for local playground start

SecurityError: Failed to set the 'domain' property on 'Document': '<IP_ADDRESS>' is not a suffix of '<IP_ADDRESS>'.

Setting HOST to <IP_ADDRESS> is not intended, since both the designer and the sandbox will set document.domain to the same second-level domain, to make

Sorry for accidently clicked the close PR button, but I'm still doubt that it will works as expected.


Create LICENSE

The other getlantern projects have an Apache-2.0 license. I presume that this one is also to be licensed under the same as the others. This commit adds the LICENSE file, addressing #1

This pull-req can be closed, there is a license now in the code itself:
https://github.com/getlantern/hex/blob/master/hex.go

Actually, I dont believe it can. The code itself only has this at the top:
// Copyright 2009 The Go Authors. All rights reserved.
// Use of this source code is governed by a BSD-style
// license that can be found in the LICENSE file.

This has two issues;

it claims that the Copyright is "The Go Authors" - maybe you want to keep that, or maybe you wish to assert some additional copyright?

The BSD-style license referenced is unclear, it says that it "can be found in the LICENSE file", but there is no LICENSE file. This line seems like it was copy and pasted from upstream go code, where there is a LICENSE file.

So to resolve this, I think that the right thing to do would be to pull the LICENSE file from the upstream go code

I've updated this merge request to instead provide the LICENSE file from upstream Go.

@oxtoacart just wanted to ping you on this MR, its the only one remaining to be resolved!

By the way, having this in the repository is necessary before I can upload this to Debian as a package. It is also the final dependency on a long dependency chain I'm working with, so having this finished would be really helpful!

Would be great to have this merged for the same reasons @micah wrote !

@micah Sorry for the delay. I've added a LICENSE.

@oxtoacart thanks - but it seems there is a problem still. The LICENSE you added is the apache one, but according to the file header, hex.go is licensed under a BSD license:
// Copyright 2009 The Go Authors. All rights reserved.
// Use of this source code is governed by a BSD-style
// license that can be found in the LICENSE file.

This is why I changed the merge request to make LICENSE be a BSD license instead.

@micah Thanks for catching that! I've updated to a BSD license that reflects the original copyright holder.


Prepare v0.9 release

Adapt the UIs for mobile devices.

https://github.com/talamantesvictor/pseudoflow/releases/tag/v0.9.0


Disable debug collision shapes in the editor

If the editor was started with --debug-collisions, 3d shapes were displayed twice, both with the gizmo and debug shapes. Some shapes could also persist after being removed due to the usage of queue_free() to destroy the debug shapes.

Hm I'd suggesting reviewing other code that uses is_debugging_collisions_hint(). From a quick look I don't see it used together with is_editor_hint() in this way. It's instead often used as an early abort like this:
                        if (!Engine::get_singleton()->is_editor_hint() && !get_tree()->is_debugging_collisions_hint()) {
                                break;
                        }

Which means that if either is true, the debug shapes drawing will happen.

Thanks!


Uncaught Error: Circular symlink detected

I get an uncaught error when trying to walk a directory:
Occurs with and without lstat: true
Code
const readdirp = require('readdirp')
const through2 = require('through2')

let w_stream = fs.createWriteStream('C:/test/write.txt')
let r_stream = readdirp('C:/', {
  type: 'files_directories',
  lstat: true
})
let lineTransformer = through2.obj(function(data, enc, next) {
  this.push(`${data.fullPath}\n`)
  next()
})

r_stream
  .pipe(lineTransformer)
  .pipe(w_stream)

Error
events.js:292
Uncaught Error: Circular symlink detected:
"C:\Users\Default\AppData\Local\Application Data"
points to
"C:\Users\Default\AppData\Local"
    at ReaddirpStream._getEntryType (index.js?b048:217)
    at async ReaddirpStream._read (index.js?b048:128)
_getEntryType @ index.js?b048:217

I think it should just ignore circular links and continue. Right now it throws the error stops the read stream

update to 3.6

@paulmillr thank you for releasing 3.6, it works now


[Fix-5351][K8s] Fix quartz cron task cannot take effect in K8s

Purpose of the pull request
[Fix-5351][K8s] Fix quartz cron task cannot take effect in K8s
This closes #5351

For the field length, it is best to define it as the Nth power of 2

Let's focus on solving these problems later


Fix typo

https://crowdin.com/translate/cachet/9/en-de#435

:disappointed: no clue why StyleCI and Scrutinizer are failing. You guys can kick off the German crowdin build please by the way.

Rebuilt the translations.

StyleCI failed because of the double quotes. Can you fix that and squash commits please?

Yeh, we updated our fixer definitions as planned on 9th April. We were going to have a blog post ready, but it's not done yet. We were advertising the change on login since April 2nd.

So you mean it should be " instead of ' ?

@ApfelUser you can apply the patch that StyleCI provides :)

Yeh, one of the new rules in the "recommended" preset is that when a string is simple, then single quotes should be preferred to double quotes.

Hm. Not to familiar with applying these diffs :smiley: I could do it myself but the problem is that StyleCI seems to want to change " to ', but when I look at the source they already are all changed to'.

@ApfelUser Is your fork out of date?

@GrahamCampbell it was, but after updating I still have only '.
My editor:

But also on GitHub:

Yes, the file you are looking at already has single quotes. Not go and look at the same file in the fr folder...

https://i.starbs.net/3o3N

Will merge and fix.


Storing extracted files by file hash name

macOS 10.14
zeek version 3.1.2
zeek/hosom/file-extraction (installed: 2.0.3) (installed from zeek pkg manager)
Having what seems like an issue getting the plugins store-files-by-*.zeek to work.
file-extraction is configured with site/packages/file-extraction/config.zeek containing:
module FileExtraction;
redef path = "";
@load ./plugins/extract-all-files.zeek
@load ./plugins/store-files-by-sha256.zeek

site/local.zeek is set to @load packages and site/packages/packages.zeek is set to @load ./file-extraction, and I can verify files are being extracted to disk.
With a run of Zeek against a PCAP, this is the result:
$ egrep 'file-extract' loaded_scripts.log
    /usr/local/Cellar/zeek/3.1.2/share/zeek/site/packages/file-extraction/__load__.zeek
      /usr/local/Cellar/zeek/3.1.2/share/zeek/site/packages/file-extraction/main.zeek
        /usr/local/Cellar/zeek/3.1.2/share/zeek/site/packages/file-extraction/file-extensions.zeek
      /usr/local/Cellar/zeek/3.1.2/share/zeek/site/packages/file-extraction/config.zeek
        /usr/local/Cellar/zeek/3.1.2/share/zeek/site/packages/file-extraction/plugins/extract-all-files.zeek
        /usr/local/Cellar/zeek/3.1.2/share/zeek/site/packages/file-extraction/plugins/store-files-by-sha256.zeek

$ zeek-cut < files.log
1588643444.082378	FbeEe53ts6OauGQ9B5	<IP_ADDRESS>	<IP_ADDRESS>	Cb4HP34zEptG6mo0Pc	HTTP	0	PE,MD5,EXTRACT,SHA1,SHA256	application/x-dosexec	-	4.443742	-	F	5265416	5265416	0	0	F	-	e4001cd6654ad6b53afd5018c57b3254	3743a5890931ce37d6e58a3a14b95bc614e54e85	bea9538673ef534043219b831887f9e14e3560e21243c517d97a82b20f2a7a85	HTTP-FbeEe53ts6OauGQ9B5.exe	F	-

$ ls extract_files/
HTTP-FbeEe53ts6OauGQ9B5.exe

I'm not clear why the file name is the default rather than the hash of the file.

Thanks for the detailed report.
You need to redef path to a valid value.
The check below prevents the rename from occurring if path hasn't been defined:
https://github.com/hosom/file-extraction/blob/a8f214c9f4aad1282e9f525daf4cdcf2b8864a16/scripts/plugins/store-files-by-md5.zeek#L7-L8

Let me know if that doesn't fix your issue and I'll be happy to take another look!

Thanks @hosom, it looks like I missed that feedback from commentary on a previous issue.
I noticed that the requirement to redef path isn't mentioned in the package documentation:
https://packages.zeek.org/packages/view/10f902bf-f435-11e9-9321-0a645a3f3086
I'm wondering though if setting path is required, or if a default can be applied. Is something like either of these feasible?

No default required, because an existing default from Zeek's FileExtract module results in files being written to ./extract_files/ -- so inherit FileExtract::prefix or similar
Change the FileExtraction explicit module level default to ./extract_files/ to be operational out of the box

Although (2) is just doing what (1) would do, just not the right way.


is Intel platform supported?

nop, i mean cpu, and AMD64&ARM are found in the user guide, so i wonder whether intel cpu supported

I see. Here AMD64 means the x86-64 CPU arch, including Intel CPUs.


wakeup service idle parameter

Thank you for this new feature!
One suggestion: for hasp_lvgl.wakeup parameter to set idle level before actual wake. Currently it wakes to idle off, which wakes to brightness 100%, which burns the eyes at night. wakeup with parameter for idle: short would wake it with brightness corresponding to that idle level so no eyes burned out :smile:
one could choose with a template to which idle level they want to wake, could be dependant on environment sensors day/night, covers position etc

I'm just calling the wakeup command from the firmware... there is currently no control for that...

This is completely broken now:

after HA restart plate starts with backlight off
with backlight set to off doesn't wake at service call
idle information is lost (not going to idle)

check light.****_backlight attribute awake_brightness is not 0
idle is now updated through statusupdate, it's coming in next firmware update


Set path to jane in environment variable

This allows Jane to get the correct path from an env var.
Manual steps:

in .bashrc add line: export JANE_PATH="/path/to/jane"
edit crontab: crontab -e. at the beginning of the file add the line: JANE_PATH="/path/to/jane"

Not the best solution because the user needs to set JANE_PATH himself AND same information in twi places could lead to inconsistencies. But since the JANE_PATH changes very rarely, its fine for now.

@Lyr3x review please :)


✨ enable running kedro boot commands from packaged project

close #36

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.

Project coverage is 59.79%. Comparing base (b9cd2d0) to head (676b2b6).

Files with missing lines
Patch %
Lines

kedro_boot/framework/cli/cli.py
0.00%
5 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #37      +/-   ##
==========================================
+ Coverage   59.65%   59.79%   +0.13%
==========================================
  Files          26       26
  Lines         865      863       -2
==========================================
  Hits          516      516
+ Misses        349      347       -2

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


Including the repo name in Bitbucket Server migration archive filenames

When running a Bitbucket Server migration, the archive from the Bitbucket Server instance is stored in the bbs_archive_downloads directory. At the moment, the filename only contains the migration ID.
This makes it hard to identify which file is which, which makes life difficult when we want to ask a customer for a copy of their archive for debugging purposes.
This updates the download process so we include the migration ID and the BBS repo name in the filename of the downloaded TAR files.

[ ] Did you write/update appropriate tests
[ ] Release notes updated (if appropriate)
[ ] Appropriate logging output
[ ] Issue linked
[ ] Docs updated (or issue created)
[ ] New package licenses are added to ThirdPartyNotices.txt (if applicable)

@ArinGhazarian Could you take a look at this? 🙏

I'm going to close this - better for someone else to handle it 👼🏻


omega-edit updates

omega-edit updates:

Implement search
Implement search and replace
Update UI to support both search and search/replace
Update redo and undo to use server instead of work around
Update omega-edit script to be omega-edit-grpc-server instead of example-grpc-server

Closes #125
Closes #126
Closes #160
Closes #161

@stevedlawrence and @mbeckerle, will you guys review this PR?  It's the big feature for version 1.2.0.


Close iter-4 -summarize of project

Where is the link to the page?

Hi,
We had a problem to load the final project to Heroku. As much we tried, we got this message in the end of the process:
You've reached the limit of 5 apps for unverified accounts. Delete some apps or add a credit card to verify your account.
This is the link to the site in Heroku. But it contains nothing...
https://dashboard.heroku.com/apps/yeshivaproject
So, we put a link to the final project that we loaded to GitHub.
This link:
https://github.com/tamarke/Final_Project_Yeshiva/tree/master/yedidOn
Thank you!!!


feat: submenu hide onleave event

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


Make compatible with .NET Core

I'd love .netcore support as well, and would be willing to help contribute and/or test.

@wparsons-360insights, I've started using serilog in my .NET core projects. It works well with .NET core and has a working .NET Core sink to write to Elasticsearch. If you need something right away, that is a good way to go.

We already have logstash setup, so I could just write to log files and ship them through logstash in the short term.  I just prefer skipping a file step if not needed.
I'll take a look at serilog though, might be a good interim solution.

@wparsons-360insights It would be awesome if you can contribute :)
Currently I don't have any strict methods of pull request, so just make your change and open a pull request.
I hope I will have time to do it myself but this is not the situation right now... sorry.

Don't think I will do this. You are welcome to open a pull request and I will gladly approve it.

Thanks to @erangil2 this should be working soon!


Is there any option for MultiSelection ?

Unfortunately there is no option for multi selecting for now but it will be added soon

@AdelRedaa97 was this ever added?

any updates?

@AdelRedaa97 Did it get added?


Simple swift version

I have changed the code according to your wishes.

I think that you should squash your commits.

I think that you should squash your commits.

I think that you should squash your commits.

Thanks for this -- code looks nice! I'll merge as soon as I can benchmark it on my Linux machine (probably later today).

Thanks for the contribution!


Not working with new package revision mechanism

Description of Problem, Request, or Question
The conan package tools is not working with the new revision mechanism (https://docs.conan.io/en/latest/mastering/revisions.html): nothing is done during the upload step.
I tried to debug the process and I found the problem occurs because the package_id now includes the revision (i.e. foo/1.0.0@user/channel#revision) but the package tools still use the short reference (i.e. foo/1.0.0@user/channel) (see cpt/runner.py, l.116)
Environment Details

Conan Package Tools Version: 0.24.0, master
Operating System: Linux
Operation System Version: CentOS 7
Compiler+version: gcc-7.3
Docker image: N/A
Conan version: conan 1.13.2
Python version: python 3.6.7

Steps to reproduce (Include if Applicable)

Set the CONAN_REVISIONS_ENABLED environment variable to 1.
Run python build.py

Build logs (Include if Available)

Hi @osechet,
In theory, the only thing to make revisions work is to set the CONAN_REVISIONS_ENABLED=1.
However, I see here https://github.com/conan-io/conan-package-tools/blob/43096ab4d7e66d73e040cdefc2f64dedd4e3ce47/cpt/runner.py#L116 that the representation of the reference will be different in that case.
So the issue is that the packages are not uploaded, right?

I can confirm the bug.
steps to reproduce:
conan config set general.revisions_enabled=True
conan conan new -s -cilc foo/0.1
export CONAN_USERNAME=uilianries
export CONAN_UPLOAD=https://api.bintray.com/conan/uilianries/conan
export CONAN_PASSWORD=*****

CPT will skip the upload step. Thanks for reporting!

Thanks for checking a proposing the fix @uilianries. I see the PR with the fix is already merged. Please close this as solve in that case 😃
P.S. That was quick man! 🥇

Closed by https://github.com/conan-io/conan-package-tools/pull/363


6666 Portal improvement next ... the hell is in details

Issue
https://github.com/gravitee-io/issues/issues/6666
Description
Migrate tslint to eslint
Try to bump all dependencies to last version
Additional context

📚  View the storybook of this branch here

🚀 CI was able to deploy the build of this PR, so you can now try it directly here
Notes: The deployed app is linked to the management API of APIM master. (Same login and password as APIM master)

Please have a look at SonarCoud bug about  new windowNotification.
I don't know if I can delete the code or even how to check the non-regression  🙏 🥺

Tested locally, looks ok


[image_picker] [integration_test] Fixes to make the tree green

Description
image_picker: Fix element type in XCUITests to look for staticText type when searching for texts.

See https://github.com/flutter/flutter/issues/71927

Integartion_test: Fix the  info - 'List' is deprecated and shouldn't be used at lib/common.dart:89:31 - (deprecated_member_use) warning
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

This PR will land on red to fix post submit ci


Don't 500 on a route with no match

Previously a bad route (eg. /crate/regex) would be a 500 because the Iron router returned it's own error, not a Nope. I guessed it should be a Nope::ResourceNotFound
Apologies if this is unidiomatic.

This is great thanks!
My original implementation was similar, now we only need to ged rid of panic! and show error description in 500 page.


Feature request: Support for level module

Hey,
I wanted to request support for the Levels and Wall height modules.
Effectively this will mean hey wait will need to check which elevation a token is and on which elevation levels the hey wait field is visible.

Good idea, I'll try to get this in for the next version


force tuf release

Signed-off-by: Kenny Leung<EMAIL_ADDRESS>
Description of the change

This is to unstick the TUF helm chart
Existing or Associated Issue(s)

Additional Information

Checklist

[x] Chart version bumped in Chart.yaml according to semver. Where applicable, update and bump the versions in any associated umbrella chart
[ ] Variables are documented in the values.yaml and added to the README.md. The helm-docs utility can be used to generate the necessary content. Use helm-docs --dry-run to preview the content.
[ ] JSON Schema generated.
[ ] List tests pass for Chart using the Chart Testing tool and the ct lint command.

There are some issues with the chart.


feat(dateFilter): add utc argument to format in UTC.

If a truthy value is passed in as the third argument to dateFilter,
it will format in UTC rather than local time.
Closes #6546

A more complete version of this feature is already available in 1.4.
We are not adding new features to 1.3.


Merge/Copy Set & Map objects

Low priority (for now) because Set/Map won't actually be merged/copied anywhere in the code because they can't be in JSON

But 100% coverage for deep_merge.js would be good


Improve logging for failing requests

This adds logging for error messages - it improves to determine sources of issues.
it changes
util.ErrorDoc("Bad Request", "400")

to
errorLog := util.ErrorLog{Logger: c.Logger(), LogMessage: err.Error()
errorLog.ErrorDocument()

ErrorLog can be initialised with status, message (which is returned in API response) and LogMessage(which is visible in logs).
if LogMessage is empty it takes message. There are also some defaults values in (e *ErrorLog) ErrorDocument()

@lindgrenj6 I am not sure if I did anything but there is only one check in this PR. Do you have idea where are checks for test, fmt and golangci-lint ?


fix: Update to support #101

This is a fix that should allow more robust rules for grant. In specific this should support whitelisting and the coexistence of allow and deny rules in the policy. This partially fixes the issue raised in #101

@psududemike thanks so much for the PR here and adding the tests. I've updated it with main and will be getting this out ASAP


⚠ Secret Website 3 has degraded performance

In a69b9b1, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 26509 ms

Resolved: Secret Website 3 performance has improved in df22a20.


Liquid Exception: No such file or directory - identify -ping

Hi,
first thanks for all your hard work. I've got problems during the configuration of your plugin. When I use

bundle exec jekyll build

this is the result
C:\Users\Gio\Documents\GitHub\uselessdevlog` [master ≡ +3 ~4 -3 !]> bundle exec jekyll build
Configuration file: C:/Users/Gio/Documents/GitHub/uselessdevlog/_config.yml
            Source: C:/Users/Gio/Documents/GitHub/uselessdevlog
       Destination: C:/Users/Gio/Documents/GitHub/uselessdevlog/_site
 Incremental build: disabled. Enable with --incremental
      Generating...
Generating coffee-550by364-40f0cb.jpg
      Liquid Exception: No such file or directory - identify -ping C:/Users/Gio/AppData/Local/Temp/mini_magick20160722-636-1o7pvo.jpg in index.html
                    done in 0.904 seconds.
 Auto-regeneration: disabled. Use --watch to enable.

Liquid Exception: No such file or directory - identify -ping C:/Users/Gio/AppData/Local/Temp/mini_magick20160722-636-1o7pvo.jpg in index.html

This is my _config.yml
# Picture Tag Config
picture:
  source: "/images/"
  output: "generated"
  markup: "picture"
  presets:
    main:
        ppi: [1, 1.5]
        attr:
            class: "img-responsive"
            itemprop: "image"
        source_medium:
            media: "(min-width: 992px)"
            width: "550"
        source_default:
            width: "300"
    post:
        ppi: [1, 1.5]
        attr:
            class: "img-responsive"
            itemprop: "image"
        source_large:
            media: "(min-width: 1200px)"
            width: "750px"
        source_medium:
            media: "(min-width: 992px)"
            width: "550"
        source_default:
            width: "300"

And my Gemfile
source 'https://rubygems.org'
gem 'github-pages', group: :jekyll_plugins
gem 'jekyll-picture-tag', '~> 0.3.0', group: :jekyll_plugins

I've tried uninstalling/reinstalling image_magick but it doesn't work.

cmd > convert -version
Do you get an image magick response?

You probably don't have image magick installed.


pagination component is not aligned with size per page component

Thanks for your quick help for issue #1000
I have face one more issue with pagination alignment. I have seen there is a extra top margin of 18px is there in the pagination component, and because of this its not aligned to the other size per page component. This same alignment issue you can see the pagination example as well. Could you please have a look and make it perfectly aligned by removing unnecessary top margin.  here are the examples for more clarification

same alignment issue in pagination example as well
http://allenfang.github.io/react-bootstrap-table/example.html#pagination

thanks in advance !!

Meanwhile I have override .pagination class to fix this but Its good if you will fix this in library itself 😃

I saw it's from the base.css and where is it come from?

Its come from the .pagination class which effects the alignment.

I know that, but .pagination selector is from the 'base.cssand my question is where is it come from? becausereact-bootstrap-tablehaven't this css file and doest define this.paginationselector, so I think your solution is correct but it's can't be fixed onreact-bootstrap-table`

anyway thanks ..for your quick lookup

@anujemc How did you solve it?

Here is the solution
.react-bs-table-pagination div.row div.col-md-6:nth-child(2){
    text-align:left !important;

    .react-bootstrap-table-page-btns-ul{
        float: left !important;
        margin-left:-90px;
    }
}


When will 1.3.x go form test to stable?

Hi, any plan when 1.3.x will go from test to stable?
One funny thing is that my came with 1.0.3 and that is not available, the latest is 1.0.2.

This is a bit old, but I have plans to release a new stable version soon. I'm working on some wifi improvements, then some websocket reliability improvements and then we should be ready to wrap some new documentation up and release!
Let's say a month more or less? 🤞


RipMe should have an option to only rip stuff that got uploaded after the last rip

I frequently rip photos and videos from NSFW Reddit users. Everything works fine. My only complaint is that RipMe tries to download everything again every new ripping session. Say a user has tons of uploads and I already downloaded them, RipMe will try and check if I already downloaded them. So RipMe will check everything starting from the oldest one (which might take hours especially if the user has many uploads) and I will just get a lot of these

Already downloaded https://i.imgur.com/image.jpg

I know I can just stop it myself but it would be handy to have the option I mentioned.
Edit: it's been a while since I used github so I forgot how to put a label

Not trying to advertise another program but as far as I know there's a program that has this functionality around reddit. RedditDownloader. Though I last used it at least a year ago. They seemingly improved it a lot, like it has a GUI now.

@kittenparry Thanks for the suggestion but I'm afraid it's too complicated for me, plus it requires you to sign in.

Totally forgot about the sign in requirement. I understand.

Okay, so apparently we have history.end_rip_after_already_seen for the rip.properties. But I wish it only triggers when they happen consecutively. Lots of Reddit users crosspost their post to many subs and RipMe detects those...


Changed index.php and style.css to achieve responsiveness

I also edited some other information on site so they aren't like in the original version

The site is now responsive.


Add unit test and example for Estimation class

Added in https://github.com/sdu-cfei/modest-py/pull/14


Annual High Water and Mean Sea Level

The text "Further statistics for the mean sea level are also available via the "explore in detail" button" is not correct (in my view) and should be dropped.

I have replace "Further" with "Regional" because we see data for regions on the explore pages, so it now reads:
Regional statistics for the annual highest high water are also available via the "explore in detail" button.

Updated json https://github.com/cedadev/c3s_434_ecde_page_text/commit/4a6639374e165d8b5743fbb7574dbd3cf61478ab


[TDP survey] Switch to new initials encoding scheme

The old one used vanilla Base64 encoding (with problematic chars) and a broken xor() function. We support the legacy hashes (some won't work, nothing we can do about that) but only create newer ones with a simpler encoder.
Motivation: The only requirement for initials encoding was casual obfuscation. The only reason the scores encoding is tamper-proof is that we get string signing for free.

Additions

New initials encoding scheme in TDP survey

Checklist

[x] PR has an informative and human-readable title

PR titles are used to generate the change log in releases; good ones make that easier to scan.
Consider prefixing, e.g., "Mega Menu: fix layout bug", or "Docs: Update Docker installation instructions".

[x] Changes are limited to a single goal (no scope creep)
[x] Code follows the standards laid out in the CFPB development guidelines

FYI, I pushed this branch to staging, and the survey tests fail at the same spot: expecting encoded initials.
You can see the errors by running cypress locally against staging. Note that when the next staging build comes along, this branch will be wiped out there, so this testing path is temporary.

@higs4281 I think I see the problem in the cypress test.

@higs4281  This will still require a re-deploy of the branch but these new tests should work on it. Old tests had encoded a full URL (which due to server key differences was never going to work across environments).

I didn't need to deploy to staging again, because the cypress code runs locally. The tests pass now on staging with the new encoding routine there.

The form for entering initials limits input to four characters. Was it a consideration that allowing four-character initials exposes the app to a universe of 4-letter words that can be rendered by students at the top of the tool's results page?

@higs4281 Like "DARN"!? Yes, we raised this point but were told a student would be unlikely to send profanity directly to their teacher.

So we're assuming students will not send a URL to friends or share them on social media?

@higs4281 From Hannah:

Leslie and the content team believe that it is not an issue because students will know it's for a grade - they don't see it as different from students write on a worksheet. It WAS a consideration and we were told we were being overly cautious, but if they want us to bring that concern again, we definitely can.

A live federal web page that can be visited and manipulated any number of times is hardly the same as a paper worksheet in a classroom. Is anything preventing a student from sending the teacher a result, then going back and sending a fun and untraceable version to social media?
I think something as simple as limiting initials to three characters could cut the fun factor dramatically.

We're meeting with Leslie next week and will raise these concerns again.


[SPARK-25479][TEST] Refactor DatasetBenchmark to use main method

What changes were proposed in this pull request?
Refactor DatasetBenchmark to use main method.
Generate benchmark result:
SPARK_GENERATE_BENCHMARK_FILES=1 build/sbt "sql/test:runMain org.apache.spark.sql.DatasetBenchmark"

How was this patch tested?
manual tests

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3292/
Test PASSed.

Test build #96342 has started for PR 22488 at commit 21b623a.

Test build #96342 has finished for PR 22488 at commit 21b623a.

This patch fails to generate documentation.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96342/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3293/
Test PASSed.

Test build #96344 has started for PR 22488 at commit 21b623a.

Test build #96344 has finished for PR 22488 at commit 21b623a.

This patch fails to generate documentation.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96344/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3299/
Test PASSed.

Test build #96354 has started for PR 22488 at commit 152c549.

Test build #96354 has finished for PR 22488 at commit 152c549.

This patch fails to generate documentation.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96354/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3310/
Test PASSed.

Test build #96375 has started for PR 22488 at commit 51ae87e.

Test build #96375 has finished for PR 22488 at commit 51ae87e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96375/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3353/
Test PASSed.

Test build #96440 has started for PR 22488 at commit 71dfe03.

Test build #96440 has finished for PR 22488 at commit 71dfe03.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96440/
Test FAILed.

retest this please

Test build #96464 has started for PR 22488 at commit 71dfe03.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3371/
Test PASSed.

@dongjoon-hyun I think this refactor is ready to go. Thanks.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3605/
Test PASSed.

Test build #96819 has started for PR 22488 at commit 7990e13.

Test build #96819 has finished for PR 22488 at commit 7990e13.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3610/
Test PASSed.

Test build #96832 has started for PR 22488 at commit 7990e13.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96832/
Test PASSed.

Retest this please.

Test build #96864 has started for PR 22488 at commit 7990e13.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3635/
Test PASSed.

Test build #96864 has finished for PR 22488 at commit 7990e13.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96864/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3641/
Test PASSed.

Test build #96879 has started for PR 22488 at commit d2d0a3e.

@wangyum . Could you review and merge https://github.com/wangyum/spark/pull/13 ?

Test build #96880 has started for PR 22488 at commit 27c6493.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3643/
Test PASSed.

Hi, @jiangxb1987 .
Could you review (and merge) this PR?

Congratulation, @jiangxb1987

Test build #96879 has finished for PR 22488 at commit d2d0a3e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96879/
Test PASSed.

Test build #96880 has finished for PR 22488 at commit 27c6493.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96880/
Test PASSed.

Merged to master.