[FR]: Date Range Filters

Contact Details
<EMAIL_ADDRESS>Present your request
It would be nice when filtering for dates if hard coded ranges could be implemented. (i.e. "Yesterday", "Today", "Tomorrow", "(#) Days", "This Week", "Next Week", "This Month", "Next Month", "This Year", "Future")
For which platform do you request this request??
Cross

Hello @Stilts15ak,
Generate custom filters in function of the tupe could be interesting.
There are two ways.

A search per column (just like the global search with custom inputs)
Inside the filters

I think the first one is more interesting and easier to maintain.

Could this also be achievable (in the short term) with using a formula and using relative dates (i.e. "Today", "This Week", etc.)?

You can already use Luxon formulas
https://rafaelgb.github.io/obsidian-db-folder/features/Formulas/

https://moment.github.io/luxon/api-docs/index.html#datetimetorelative
${row.column1. toRelativeCalendar()}

Thanks! I'll play around with it.

Also I saw a problem with the formula. It should be refreshed after any change of the row.
Will be fixed with the next version =) Commit b331dbd4

Yup! That's what I'm getting.
I think this is a nice solution. Whether it's short or long term.
However, it looks like DB will not filter a formula column.

to allow the filters you need to persist the value (a toggle inside the column config)

Not sure I understand. Can you elaborate further?

https://user-images.githubusercontent.com/11924043/202490635-5d1f351f-664a-4847-8f2e-1126273770ca.mov

Thank you! Works like a charm now.
Can I make a suggestion to where it says "Only persisted formulas could be searchable and sortable" to add "and filtered" as well?  And perhaps include this in the documentation too?

We can improve the documentation, of course.
It's a young plugin and the changes are constant. So the documentation is hardly to maintain

In the video example, Is it possible to use with the Created column (date from file created)?

Yeah, it is possible

It appears empty

All metadata fields need __<id>__ wrapping
Try with ${row.created.toRelativeCalendar()}

Now it works, thanks Rafa!

a new feature for calendar/calendar time will be added for the next version
If you add an @ to some words like today or next weekwill be applied to your filter in function of the actual date
import { DateTime } from "luxon";
const valueToCalendarConverter = new Map<string, DateTime>([
    ["today", DateTime.local().startOf("day")],
    ["yesterday", DateTime.local().minus({ days: 1 }).startOf("day")],
    ["tomorrow", DateTime.local().plus({ days: 1 }).startOf("day")],
    ["thisweek", DateTime.local().startOf("week")],
    ["lastweek", DateTime.local().minus({ weeks: 1 }).startOf("week")],
    ["nextweek", DateTime.local().plus({ weeks: 1 }).startOf("week")],
    ["thismonth", DateTime.local().startOf("month")],
    ["lastmonth", DateTime.local().minus({ months: 1 }).startOf("month")],
    ["nextmonth", DateTime.local().plus({ months: 1 }).startOf("month")],
    ["thisyear", DateTime.local().startOf("year")],
    ["lastyear", DateTime.local().minus({ years: 1 }).startOf("year")],
    ["nextyear", DateTime.local().plus({ years: 1 }).startOf("year")],
    ["thisquarter", DateTime.local().startOf("quarter")],
    ["lastquarter", DateTime.local().minus({ quarters: 1 }).startOf("quarter")],
    ["nextquarter", DateTime.local().plus({ quarters: 1 }).startOf("quarter")],
]);

export default class FilterValuesMapper {
    public static toCalendarValue(filterValue: string): DateTime {
        if (filterValue.startsWith("@")) {
            const potentialDate = valueToCalendarConverter.get(filterValue.slice(1).toLowerCase());
            return potentialDate ? potentialDate : DateTime.local();
        }

        const potentialDate = DateTime.fromISO(filterValue);
        return potentialDate.isValid ? potentialDate : DateTime.local();
    }
}


fix typo: is_reserved

@bors r+

:pushpin: Commit a7c12ee027c50a9737e3e1f2bb2be8190266bf67 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit a7c12ee027c50a9737e3e1f2bb2be8190266bf67 with merge a4d627c4627fec1ef2eaa0fa282bf9427fc49186...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing a4d627c4627fec1ef2eaa0fa282bf9427fc49186 to master...


add test for api client

Codecov Report

Merging #11 (c373865) into master (733a431) will increase coverage by 0.89%.
The diff coverage is n/a.

:exclamation: Current head c373865 differs from pull request most recent head 06c8320. Consider uploading reports for the commit 06c8320 to get more accurate results

@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
+ Coverage   79.04%   79.94%   +0.89%
==========================================
  Files          22       22
  Lines         673      673
==========================================
+ Hits          532      538       +6
+ Misses        121      116       -5
+ Partials       20       19       -1

Flag
Coverage Δ

unittests
79.94% <ø> (+0.89%)
:arrow_up:

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

Impacted Files
Coverage Δ

app/cli/client.go
86.04% <0.00%> (+4.65%)
:arrow_up:

app/cli/client_api.go
82.75% <0.00%> (+6.89%)
:arrow_up:

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 733a431...06c8320. Read the comment docs.


Document labelling: document pair classification (for example, determining if the texts are similar or not)

Front-end

[x] In case of block of text annotated by different labels, we need to show all labels one below other
[ ] Show similar documents side by side (support many documents). Add functionality to link documents.

Back-end

[ ] Have to support relations between documents: similar, identical, etc. One document may have multiple relations. Back-end must support different types of relationships
[ ] Add linking between documents by link type. Link and documents must be in revision

General algorithm
Back-end must support tasks to check pairs.

Front-end is also required to  implementat


onClose should be called inside animation

Since sidr uses animation to close the menu, shouldn't onClose be called within the callback function of .animate()?
This way the function will be called only after the menu was really closed.
https://github.com/artberri/sidr/blob/master/src/jquery.sidr.js#L175

This project hasnt been updated in 2 years >_>

For that the only way right now is to programatically close the menu and pass a callback not to use the onClose option:
jQuery.sidr( [method] [, name] [, complete] )
$.sidr( 'close', 'sidr', yourcallback )


When I select button using ngf-select, ng-model is changed to null.

I have this source.

                    <ptgui-imagebox name="file1" class="col-md-3" style="width: 100px;height: 100px;" ng-model="product.image" ngf-max-size="1MB"; ngf-select></ptgui-imagebox>

                         Drag and Drop

when i click ptgui-imagebox showing image, ng-model is changed.
but I want to maintain image before I select new image.
How can I do?

You can use ng-model-options="{updateOn: 'select drop'}" but it will not allow you to select the same file again and if the user cancels the popup in Chrome the value will become null. That feature is there to have the same behaviour cross browser. So when the user opens the dialog it means that they want to select another file and if they cancel the old file is removed.


Cannot scroll the help page

I'm guessing it's because of the overflow-hidden we added...
/cc @lqd

Yes, that is exactly the source of the problem. I have a couple different fixes in-flight, and will make a PR soon (we'll just need to see which of those you prefer).


feat: batch notification events

Add support for batching so that we don't send individual notifications for each file. Chokidar already did debouncing for 400ms and batching but we've handled stdout per line so we still did send individual notifications for each file.
Update chokidar and other dependencies to latest versions

I haven't noticed any lag, or performance issues with the current behavior, so I am not sure how to test this change properly. Any suggestions?

I didn't notice any noticeable performance issues either (I'm on M1 Max though) but the difference between having and not having this change is for example sending 400 separate notifications vs 1 notification with 400 batched events so there must be pretty significant overhead for the former. It's likely that the server will just be able to process those faster.
I suppose for testing it's enough to verify that the didChangeFilesNotification includes many events instead of just one on changing branches for example.


Add ability to pass CA certificates by fileobj

Sometimes requests has no ability to open file with CA because is no such file, e.g. when CA is embedded in the app.
requests should allow to specify fileobj (which can be a file or io wrapper on top of string and bytes) for that.

Unfortunately that's extremely difficult for requests to do.
The 'path to a file' approach is actually the API used by OpenSSL. It's also possible to pass an in-memory serialized representation (as read from a file object), but that API is not exposed by the standard library so we can't rely on that being present.

We could create a tmpfile automatically when passed a file-like-object, but we would likely want to ensure its deletion after execution is complete.
So, definitely possible, but probably not the best idea.

Creating intermediate file can be handled by user of requests.
After checking Python's code I agree that it's, at least, a limitation in Python.
Sad that OpenSSL relies on file being present in FS.

@Lukasa is there a way to query presense of this API in runtime? Could you give me some links to read more about it?

Is it possible to verify by using system's CA without shipping cacert.pem nor certifi?

@Kentzo: for a discussion about the system cert bundle, see #2966. It is possible to query for this API at runtime because it's only exposed by PyOpenSSL.

Python 2.7.9+ and 3.5's implementation of load_verify_locations() in Modules/_ssl.c support passing of a cadata argument, which contains the serialized trust store. This is also exposed in the ssl.create_default_context() function.

@Lukasa I think the original issue is orthogonal to #2966 and this issue should be re-opened because of the discovery made by @dsully.


Improve release workflow to maintain version tags of container images.

Currently, for every release, we simply build and push the new container image under the latest tag. We need to start maintaining version tags ( like v1.3.2 ) to enable stable usage of the container images.

This was fixed with #410
~ docker pull accurics/terrascan:1.3.2
1.3.2: Pulling from accurics/terrascan
df20fa9351a1: Already exists
9130f8cc7abf: Pull complete
db5938b5e541: Pull complete
Digest: sha256:10a60cc61d51a75033a9883e4dc4a00000d22e9d5ab254a3b71a550f66f65528
Status: Downloaded newer image for accurics/terrascan:1.3.2
docker.io/accurics/terrascan:1.3.2


[TorchTitan][Checkpoint] Add a step-by-step instruction for checkpoint conversion

I did follow all the instructions mentioned in the checkpoint.md after running this command successful the checkpoint.pt file was not created i did search the whole dir I did not find it anywhere
python -m torch.distributed.checkpoint.format_utils dcp_to_torch torchtitan/outputs/checkpoint/step-500 checkpoint.pt Converting checkpoint from torchtitan/outputs/checkpoint/step-500 to checkpoint.pt using method: 'dcp_to_torch'


Update all component readmes to include --save

Related to this comment: https://github.com/material-components/material-components-web-react/pull/117#discussion_r198621191
We should include --save in the npm install instructions for all components. This is an issue to go back and update all readmes.

This hasn't been required in quite a while, to the point where --save is no longer even documented by npm install because it is the default behavior. On the contrary, --no-save is an explicit option now.


How to return trained models

Hi,
Is there any way to get the trained models from sc_multi_omics.fit()?
Best,
Mingxi

Hi, Mingxi,
SCOIT provides a sklearn-style model training pipeline, i.e., after running "sc_model.fit(data)", the trained parameters have already been reserved in "sc_model". You can use "sc_model.C", "sc_model.G", and so on, to obtain the trained parameters.
Best regards,
Ruohan


JS elements given incorrect css classnames

In a function's aruguments, variable elements (line1:"whichOverlay") are given the class cm-def.
In the function body, internally declared variables (line2:"thisOverlay") are given the class cm-def, while argument calls (line2:"whichOverlay") are given the class cm-variable-2.
I think the classes should be switched for the internally declared variables and argument calls.

The classes are coming from CodeMirror and we aren't gonna to change them.


fix(ledger): bump the hid package to v0.9.2 which fixes ledger on new…

… macos versions
Description
Fix ledger for newer macOS versions
Purpose
Enable use of ledger devices to sign txs via CLI

Awesome


cascade layers : important

Implement !important handling in @layer :

split rules so that !important declarations have their own rule
assign inverse specificity adjustments to the rules with !important declarations

I did a few spot tests in Chrome Canary and transformed CSS with !important seems to match native @layer styles.

 postCSS plugin


More tests for its-happening

Add a test similar to TestP2P in walletd and add some extra cases mostly relating to file contracts to the validation test.

We're above 75% coverage in consensus now, nice!


Utilize routes to better manage error redirection and page loading

Compilation is pretty fast once we have the page.
Note that the description here: https://github.com/emberjs/ember.js/pull/20712 may be a better-written counter-argument for using routes (as they exist in today's ember, anyway)

Not going to pursue this due to perceived behavior https://github.com/emberjs/ember.js/pull/20712#issuecomment-2211340943


MSI Packages are not getting installed (no negative docker feedback)

When installing MSI packages, i noticed that it builds successfully, but packages are not installed. There is no error message or other stuff coming up.
2 Methods that i use:
1.  Method:
ADD HTTPLINKTOFILE/MSIFILE.msi /folder/MSIFILE.MSI
RUN msiexec.exe /i C:\folder\MSIFILE.MSI /passive & \
rd /s /q C:\Folder

2. Method:
ADD MSIFILE.msi "C:\Folder\"
RUN msiexec.exe C:\folder\MSIFILE.MSI /quiet /qn /passive

The http link method successfully installs the MSI file without returning any error.
But it does not show up, installed URL REWRITE or ARR or any other stuff but it's just not there.
The second method via adding the file to the Docker Volume and then installing it makes the DOCKER BUILD process hang, it just stays there without going any further, also no error message.
Sad i cant deliver a stack trace or error message but that's all i got. Anyone knows how to handle that?
This is not a duplicate to other MSI related issues since here, it get's installed in the first place but won't show functionality.

I have the same problem with docker 17.03.1-ee-3, build 3fcee33  on Windows Server 2016.  I used 2-nd method from hqer2k9's post.  docker build just hangs at RUN step

You need to use Start-Process with -Wait flag for those

@hqer2k9 If this issue still persists feel free to re-open with a repro.


Syntax analyzer NullReferenceException

Hi,
syntax analyzer crashing is nothing new, we've been having NullReferenceException (oddly referencing to app.json which should be exempt from most rules) for several years (and they've been repeatedly reported). They are really hard to replicate without sharing the entire project repository.
Here's a new one with
Version: 20.0.37253.38585
Platform: 20.0.37114.38311
(AL compiler from vsix)
[{
"resource": "/c:/Users/fuuf/fruu/app.json",
"owner": "generated_diagnostic_collection_name#0",
"code": "AD0001",
"severity": 4,
"message": "Analyzer 'Microsoft.Dynamics.Nav.CodeCop.Localizability.Rule218TooltipMustBeFilledOnPageForFieldsAndActions' threw an exception of type 'System.NullReferenceException' with message 'System.NullReferenceException: Object reference not set to an instance of an object.\r\n   at Microsoft.Dynamics.Nav.CodeCop.Localizability.Rule218TooltipMustBeFilledOnPageForFieldsAndActions.AnalyzePage(SymbolAnalysisContext symbolAnalysisContext) in D:\a\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeCop\Localizability\Rule218TooltipMustBeFilledOnPageForFieldsAndActions.cs:line 58\r\n   at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.<>c__DisplayClass49_2.b__0() in D:\a\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\DiagnosticAnalyzer\AnalyzerExecutor.cs:line 656\r\n   at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.ExecuteAndCatchIfThrows_NoLock(DiagnosticAnalyzer analyzer, Action analyze, Nullable`1 info) in D:\a\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\DiagnosticAnalyzer\AnalyzerExecutor.cs:line 1078'",
"source": "AL",
"startLineNumber": 1,
"startColumn": 1,
"endLineNumber": 1,
"endColumn": 1
}]

We actually detected this internally and have fixed it just two days ago. I think this should make it into the next VSIX release to marketplace. Thanks for raising it too!


I'm having trouble creating a custom endpoint.

I was able to get the rest api up and running and the following url works in my browser:
http://<IP_ADDRESS>:1781/api/
If I add a custom endpoint, would it show up in that list of apis?
I tried making my settings:
"rest.api": {
    "autoStart": true,
    "openInBrowser": true,
    "port": 1781,

    "endpoints": {
        "myendpoint": {
            "script": "./my-endpoint.js",
            "options": "Hello!"
        }
    }
},

And saving the example script you gave here:  ( in the same folder as settings.json )

Should I be able to excecute it with:
http://<IP_ADDRESS>:1781/api/myendpoint
I'm not sure how I would debug this, or if it is possible my .js file is in the wrong spot, or invalid, or if I need special settings to allow executing a custom endpoint.
I will want to create a few custom endpoints like:

get current line number ( where cursor position is )
get current line text
get current file name  ( api/editor seems like a start but I think it might crash if the content of the file is too large )

I'm interested in learning how to write all of these but I think I need to get my "hello world" example going first :)

I just figured out some answers by reading through the code:

The js file to define the custom endpoint is specified with relative path starting from your workspace root directory.

mine is here: .vscode/myendpoint.js and
this is what I have as the endpoint script argument.

The custom endpoints are not listed when calling the /api resource


Change benchmarks

/cc @JamesNK @kichalla - separated from #591 to make it easier on all of us :)
Add benchmarks that include some HTTP method matching.
Clean up names and name like-kinded benchmarks alphabetically.
MatcherBenchmark -> E2E including HTTP method selection
MatcherSelectCandidatesBenchmark -> Focused on just URL path processing

@JamesNK updated
I addressed the feedback and did some general cleanup around unsafe. Using Span<> instead of a pointer in FastPathTokenizer had no negative perf impact so I made that change, and resolved all of the fallout from it.
I've deliberately left unsafe and pointers in the baselines for the path tokenizer.


Adds basic support for visualizing mouse clicks

This first bit of functionality for supporting visualization of mouse clicks works similar to Quicktime screen recordings. Some speculative code is also included for forwarding mouse events to the keystroke visualizers, but is not yet enabled.

Mouse click visualization is in the same color as the default visualizer bezel. We could make this user-configurable in the near future, if there is interest.
Mouse click visualization uses a fixed/hard-coded radius and the default fade animation duration. Unclear whether either of these should be made adjustable.
A bunch of other cleanup is included, adjacent to the modified code.

This adds the first steps necessary to implement #33 , and should address #168.

https://user-images.githubusercontent.com/378648/155045108-0bfaa8b3-80c8-46ef-8d56-fe56e026c3e0.mov
I think this is ready to go as an MVP for a mouse click visualizer. Next steps/features for the mouse visualizer will go into a new issue.


Error Setting Up Managed Identity by Signing into Azure SQL Database via Azure AD for Azure AppService

I am having a problem running the command from the link below:
sqlcmd -S <server-name>.database.windows.net -d <db-name> -U <aad-user-name> -P "<aad-password>" -G -l 30
https://docs.microsoft.com/en-us/azure/app-service/app-service-web-tutorial-connect-msi
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 31b0d33c-f88e-d50c-2926-7933940f3b92
Version Independent ID: 4aaef906-f501-6947-5d23-1b704470f09d
Content: Tutorial: Access data with managed identity - Azure App Service
Content Source: articles/app-service/app-service-web-tutorial-connect-msi.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

Thanks for the feedback @PingPongSet! We are currently investigating and will update you shortly.

You'll run into issues when using sqlcmd inside Cloud Shell. It appears it can't use the principal of the user that initiated the session. You can; however, use sqlcmd -S <your-server-name>.database.windows.net -d <your db> -G -l 30 in your command prompt, powershell, bash assuming the logged in user is an AAD user in the tenant of the subscription. If it isn't, omit the -G switch and use the   you used when creating the server. Also make sure you added your local IP to the server firewall.


[ElasticSearch] - Backfill data for elasticsearch

backfill older payment_attempt/intent & refund data to elasticsearch

https://juspay.slack.com/archives/C05J73K7Q3Y/p1709034526124209


Support ingestion of Audit Logs to Segment

Some customers would want to get analytics from Audit Logs. In that case, they would need to ingest the audit logs to build up dashboards. We should provide the adapter that enables this integration to segment.
We should provide an API to pull the Audit log data or a webhook that pushes this data to users regularly.

The preferred approach is tracked here, please comment on the ticket - https://github.com/appsmithorg/appsmith/issues/24841


Should CodepointOrGlyphId be more static?

I noticed that when CodePointOrGlyphId appears in a public function's
signature, it's always as Into<CodePointOrGlyphId>. If the purpose here is to
make functions like Font::glyph accept either a Codepoint, GlyphId, or
char, then it seems nicer to use the types to select the necessary conversions
at compile time, rather than constructing an enum, immediately matching on it,
and counting on the compiler to optimize that all out.
Instead of CodePointOrGlyphId, rusttype could offer a trait:
trait IntoGlyphId {
    fn into_glyph_id(self, &Font) -> GlyphId;
}

with implementations for char, GlyphId, and Codepoint:
impl IntoGlyphId for char {
    fn into_glyph_id(self, font: &Font) -> GlyphId {
        GlyphId(font.info.find_glyph_index(self))
    }
}

impl IntoGlyphId for GlyphId {
    fn into_glyph_id(self, font: &Font) -> GlyphId {
        GlyphId(self)
    }
}

I implemented this, and tried it out on the four most-downloaded crates that
depend on rusttype. None of the crates use CodePointOrGlyphId directly, and
all compiled without source changes.
What do you think?

(The crates I tested against were conrod, orbfont, piston2d_graphics, and waterfall.)


UnstructuredEmailLoader just returning the first element

Issue:
UnstructuredEmailLoader only returning first or no element of the mail irrespective of the mode.
System Info
langchain version: 0.0.316
unstructured version: 0.10.18
Who can help?
@eyurtsev
@hwchase17
Information

[ ] The official example notebooks/scripts
[X] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[X] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
This is the link to the code to regenerate the reponse
Expected behavior
It should have returned the email's message text as the output with it's metadata

@dosu-bot I am using the updated the partition_email function implementation.

Found out the issue, was using the older version of unstructured


Test for checkerboard.jl incorrectly specified

This tests for

Check every second square is assigned white
Check every second square is assigned black

are incorrect because the default square_size is 50 and not 1 like the test assumes. So, one is not actually testing consecutive squares.
See https://github.com/zygmuntszpak/CalibrationTargets.jl/commit/6c1089468fd3875b3684cbdb5ea121bd45e9a189#r91021850

I have implemented a fixed versions of these tests, now taking into account the square size


failing concurrent test

A DirectProcessor doesn't actually buffer any events if you aren't subscribed so the test is just dropping them. I image the reason it works when with onComplete the processor is complete so you don't block anything. I think this test does what you want:
@Test(timeout = 2_000L)
    public void testTwoConcurrentStreams() throws InterruptedException {
        // Two concurrent streams
        // Stream 1 is a single response
        // Stream 2 is a single response, then completed by the server
        // Stream 1 is then cancelled by the client

        ConcurrentHashMap<String, UnicastProcessor<Payload>> map = new ConcurrentHashMap<>();
        UnicastProcessor<Payload> processor1 = UnicastProcessor.create();
        map.put("REQUEST1", processor1);
        UnicastProcessor<Payload> processor2 = UnicastProcessor.create();
        map.put("REQUEST2", processor2);

        handler = new AbstractReactiveSocket() {
            @Override
            public Flux<Payload> requestStream(Payload payload) {
                ByteBuffer data = payload.getData();
                byte[] b = new byte[data.remaining()];
                data.get(b);
                String s = new String(b);
                return map.get(s);
            }
        };

        ReactiveSocket client = buildClient();

        Flux<Payload> response1 = client.requestStream(new PayloadImpl("REQUEST1"));
        Flux<Payload> response2 = client.requestStream(new PayloadImpl("REQUEST2"));

        processor1.onNext(new PayloadImpl("RESPONSE1A"));

        processor2.onNext(new PayloadImpl("RESPONSE2A"));
        processor2.onComplete();

        CountDownLatch latch = new CountDownLatch(3);

        response2
            .subscribeOn(Schedulers.newSingle("2"))
            .subscribe(c -> latch.countDown(), t -> {
            }, latch::countDown);

        response1
            .subscribeOn(Schedulers.newSingle("1"))
            .subscribe(c -> latch.countDown());

        latch.await();

    }

@robertroeser Thanks, that's working now.  I've updated the test slightly to show the next happens for both streams before the complete.  Hopefully it's just the KeepAlive bug that was causing our problems.


Connector published on Google Data Studio temporarily broken (temporary link to access connector included here)

The connector is currently inaccessible through: https://datastudio.google.com/data?search=ODK (the connector gallery). There was an issue with deployment and the published connector links to a deployment that doesn't exist anymore.
While we fix the issue you can access the connector through this link: https://datastudio.google.com/datasources/create?connectorId=AKfycbxMsmDvxShWuf1jctvN8B_4BKzrTXCkx1sAj2cplioeOpWgD7inx0jsnAEflDJSYuckTA

The connector has been published and is live here: https://datastudio.google.com/data?search=ODK
If you need to switch connections from a temporary connector to the published version, please check out these steps in our guide.


Collapsible Reply Trees

It should be possible to collapse reply trees to improve the reading experience by allowing the reader to minimize content they aren't interested in, or have already read.

Possibly helpful: https://code-boxx.com/css-collapsible-tree-menu/


[PE-D][Tester C] Inconsistent Invalid Command Format messages

For the filter command, all examples are displayed after parameters.

For the delete command, examples are displayed after each parameter.

For the find command, parameters are shown on the next line

For the add, grade, remove, module commands, parameters are shown inline with the function of the command.

Labels: severity.VeryLow type.FeatureFlaw
original: lm-44/ped#9

standardise result display for invalid command


Use the default 'RollingUpdate' strategy if type is not set

Feature request
The default strategy for replacing old pods is RollingUpdate. Currently, if spec.strategy.type is not set, kubernetes-deploy fails with
'kubernetes-deploy.shopify.io/required-rollout: maxUnavailable' is incompatible with strategy ''

when kubernetes-deploy.shopify.io/required-rollout=maxUnavailable

[x] If the maintainers agree with the feature as described here, I intend to submit a Pull Request myself.

Proposal:
It should use the default value of RollingUpdate if the type is not set.

👍 I agree


Unable to set fill below Line in Line Charts

Not Working Properly:

[X] Missing or needed?
[ ] Confusing
[ ] Not sure?

Please Explain in Detail...
I'm unable to set the fill color below Lines or Curves in Line Charts

Your Proposal for Changes
Fix This Issue Please.
It's very tiring and annoying.
It worked on previous versions.
Example
No response

@LeeLenaleee , Thank you so much. It worked. I researched the solution to this but no source, not even AIs told me your answer.
Guess no wisdom beats that of the Creator himself 😁.
Thank you.


Fix issue with trailing spaces and handle comments in the property file.

@rabbah this closes #2770

Thanks for the fix! Could probably add a test for this, using a temp file and WSK_CONFIG_FILE to point to the temp file.

@rabbah Yes, I will do, I actually wanted to do a test for this. I think that should be good to add Go tests for the cli project as unit tests. Didn't see any unit tests. I think that the Scala tests looks like more as integration tests, I am correct?

Your observation is right. The cli code is moving to its own repo and should have unit tests eventually replacing supplementing the current ones.

Got it. Sounds good!

@dubeejw Sure, will do it today.

@dubeejw Done, please take a look. Thanks!

@rabbah Did you have time to take a look at these changes. On the other hand I'm thinking about adding unit tests for the wsk-cli project, do you think that it will be best to first move to its own repo?

The repo exists already. I don’t think it has any unit tests yet.

@GabrielNicolasAvellaneda, when the CLI update the property file, do existing comments stay intact?

Hey @dubeejw Good question! No, comments gets removed when you write any property. This is exactly the original behaviour.
ReadProps is ignoring comments, for example if you have a single line comment it will be ignored by this following line that checks for a format like KEY=VALUE
https://github.com/apache/incubator-openwhisk/blob/master/tools/cli/go-whisk-cli/commands/util.go#L936
Then when you write a property it is using the map returned by ReadProps as a basis for all the property values.
Please also look at this line: https://github.com/apache/incubator-openwhisk/blob/master/tools/cli/go-whisk-cli/commands/util.go#L960
We should create a new issue to address this because it will require to change the logic of how the properties are set because the whole file is overwritten.
I could also do that fix if you assign it to me. In this case for writing to the file we must locate the proper line where the key is and update just that line. If that key does not exist then we should append at the end of the file.

@rabbah @dubeejw Created this https://github.com/apache/incubator-openwhisk/issues/2806 to fix .wskprops formatting issues.

@dubeejw I think this is ok now with the test, wdyt?

@dubeejw nudge.

PG4 908 ⏳


getUploads contains item with id:0 and success but getItemByFileId(0) errors and item is not displayed

Type of issue

[x] Bug report
[ ] Feature request
[ ] Support request

Uploader type

[ ] Traditional
[ ] S3
[x] Azure

Bug Report
Fine Uploader version
5.14.1
Browsers where the bug is reproducible
chrome 56.0.2924.87
Operating systems where the bug is reproducible
Windows 10
Exact steps required to reproduce the issue

Set options to include session.endpoint using list display
return json from server
in onSessionRequestComplete, call getUploads()
attempt to call getItemByFileId with the id of the first item in the returned array
error thrown..
side note, calling getName with the same id works

All relevant Fine Uploader-related code that you have written
var uploader = new qq.azure.FineUploader({
element: document.getElementById('my-uploader'),
retry: {
enableAuto: true
},
chunking: {
enabled: true
},
signature: {
customHeaders: function (id) {
var ctx = uploader;
var fileitem = ctx.getItemByFileId(id);
var docID = qq(fileitem).getByClass("view-btn")[0].getAttribute("data-customerDocumentId");
return { 'x-filename': ctx.getName(id), 'x-uuid': ctx.getUuid(id), 'x-customerdocid':docID };
},
endpoint: '/api/Customer/42877202/Document/OtherCertification/getAssetSas'
},
uploadSuccess: {
customHeaders: function (id) {
var ctx = uploader;
return { 'x-filename': ctx.getName(id), 'x-uuid': ctx.getUuid(id) };
},
endpoint: '/api/Customer/42877202/Document/OtherCertification/uploadComplete'
},
deleteFile: {
enabled: true
},
session:{
endpoint: '/api/Customer/42877202/Document/OtherCertification/getAssets'
},
callbacks: {
onComplete: function (id, name, response) {
var fileItem = this.getItemByFileId(id);
			if (response.success) {
				var viewBtn = qq(fileItem).getByClass("view-btn")[0];

				viewBtn.setAttribute("href", response.URI);
				viewBtn.setAttribute("data-customerDocumentId", response.CustomerDocumentID);
				qq(viewBtn).removeClass("hide");
			}
		},
		onDeleteComplete(id, xhr, isError) {
			if(isError)
			{
				// Ther server actually already did the delete, so we just remove the file from display
				uploader.setStatus(id, qq.status.DELETED);
			}
		},
		onStatusChange(id, old, newStatus){
			debugger;
		},
		onSessionRequestComplete(response, bBuccess, raw) {
			var ctx = this;
			$.each(uploader.getUploads(), function (i, item) {
				$.each(response, function (loadedFileIndex, loadedFile) {
					if (item.uuid == loadedFile.uuid) {
						var fileItem = ctx.getItemByFileId(item.id);
						var	viewBtn = qq(fileItem).getByClass("view-btn")[0];

						viewBtn.setAttribute("href", loadedFile.customerFile.URI);
						viewBtn.setAttribute("data-customerDocumentId", loadedFile.customerFile.CustomerDocumentID);
						qq(viewBtn).removeClass("hide");
					}
				})
			});
			debugger;
		}
	},
	multiple: true
});

Your Fine Uploader template markup (if using Fine Uploader UI and the issue is UI-related)

Upload a file

            Processing dropped files...

            <input class="qq-edit-filename-selector qq-edit-filename" tabindex="0" type="text">

            <button type="button" class="qq-btn qq-upload-cancel-selector qq-upload-cancel">Cancel</button>
            <button type="button" class="qq-btn qq-upload-retry-selector qq-upload-retry">Retry</button>
			View
            <button type="button" class="qq-btn qq-upload-delete-selector qq-upload-delete">Delete</button>

    <dialog class="qq-alert-dialog-selector">

            <button type="button" class="qq-cancel-button-selector">Close</button>

    </dialog>

    <dialog class="qq-confirm-dialog-selector">

            <button type="button" class="qq-cancel-button-selector">No</button>
            <button type="button" class="qq-ok-button-selector">Yes</button>

    </dialog>

    <dialog class="qq-prompt-dialog-selector">

        <input type="text">

            <button type="button" class="qq-cancel-button-selector">Cancel</button>
            <button type="button" class="qq-ok-button-selector">Ok</button>

    </dialog>

Please add a console.log for each record in the getUploads() call that you are iterating over, and include the output in your question.

Ok...  that was a little odd..  I just added the console.log and loaded the page once, and the first one came back with a status of deleted.  But that seems odd given that it was just loaded from the session endpoint.  Then I closed everything and reloaded it, and now I can't get it to repro.. Does uploader use any local storage?  Is it possible it was hanging on to some state for the first item in the list?

localStorage is only used for the resume feature. These records wouldn't be returned by getUploads() unless corresponding files were added to the uploader as well. Going to close this, but happy to re-open if you can reproduce w/ logs.


Landfast Sea Ice Update for ARDAC

This PR contains the ingest recipes for four new coverages of landfast sea ice data. The four coverages are split by geographic zone (Beaufort Sea, Chukchi Sea) and by level of aggregation (daily values throughout the ice season, or "MMM" monthly summaries of the ice season).
The coverages are:
ardac_beaufort_daily_slie
ardac_beaufort_landfast_sea_ice_mmm
ardac_chukchi_daily_slie
ardac_chukchi_landfasat_sea_ice_mmm

The source data reside at the locations indicated in our tracking spreadsheet.
The steps to ingest these data are to:

Fetch the data from Poseidon
Un-tar the data
For the MMM coverages, set the LUTS_PATH env var to point at the correct luts.py (the daily_slie coverages do not require a luts.py file)
Ingest via the normal set of rasdaman incantations

For when we're ready to merge, I think this should resolve the merge conflict - from this branch:
git merge main
git rm ardac/landfast_sea_ice/ingest.json


pomf-spec

afaik this doesn't really follow any the pomf API specification to become an alternative. chances our if you implement the spec i'll probably migrate cuntflaps to this

Whats missing for the pomf-standard, an array of files?
Implementing CSV seems like a waste.

read it, and if you wanna know why csv is needed i dare you to go into #pomfret and say that

nah


InternLM-XComposer2-4KHD-7B  多卡推理

RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:2 and cuda:0! (when checking argument for argument weight in method wrapper_CUDA__cudnn_convolution)

Did you use lora to fine-tune it? Just load your model like this:
`# init model and tokenizer
model = AutoPeftModelForCausalLM.from_pretrained(
'//InternLM-XComposer/finetune/output/lora_finetune_1',
# device_map="auto",
trust_remote_code=True
).cuda().eval()
tokenizer = AutoTokenizer.from_pretrained('internlm/internlm-xcomposer2-4khd-7b', trust_remote_code=True)
`


Simple fix to issue #1440

@pkdash @mjstealey this is a simple fix to issue #1440. Hope we could merge it to release 1.7.5

+1

@mohamedmorsy Do you think this needs to be applied as a hotfix to 1.7.5? Or can we wait until the next release?

@pkdash if this is the only hotfix for this release so it can wait until the next release.

@mohamedmorsy @pkdash - looked like a one liner and has been manually added to www.
Please verify

@mjstealey @pkdash Now it looks good to me on www


Use dismissAfter from the current notification, instead of that of the previous one.

The previous value of dismissAfter was used instead of the new one when setting the onDismiss timeout, causing notifications to dismiss with the previous notifications dismiss-time, instead of the one you sent in.

+1

Great. Thanks. I will merge and release this over the weekend.

This is published to npm as<EMAIL_ADDRESS>


"Sometimes" validation

Hi,
Is possible to handle "sometimes" validation? Example:
Rule:
Password: Not required, but if "not empty" should have at least 6 chars.

Yeah, there's an upcoming @OptionalOrValid annotation.

Awesome! Thanks @ragunathjawahar

Can I suggest something?
Maybe.. a param is better than another annotation.
@ValidPassword(min = 6,  scheme = Password.Scheme.NUMERIC,  sometimes = 1 )

Where:
"sometimes" check if password is not empty. If empty, check if is a valid password.

For what is worth - I see that lots issues opened here are somewhat related to issue I now participate in:
https://github.com/ragunathjawahar/android-saripaar/issues/127
Basically, what we all are asking for is ability to change rules based on certain set of conditions. Now @ragunathjawahar I STRONGLY advise you against going with MORE attributes, because in a nutshell what you are doing is creating programing language within Java.
And there is simply no need for that. You already have Java as programming language. Attributes are TERRIBLE when it comes to flexibility for simple reason - you use them to NOTATE something. So you shouldn't use them in case where it's like - Oh, it COULD be this... but it also COULD be that.
To conclude - let's create a way to easily change rules programmatically and I'm sure everything else will fall in line.

I believe creating a rule is easy programmatically, could you perhaps share some examples of what you mean.
P.S. New user of saripaar. Love It!

@lepipele I'm against attributes as well. Follow #102 for updates.


[server] Make cleaning up partitions on startup configurable. Off by default

Summary, imperative, start upper case, don't end with a period

We need to test this feature thoroughly before rolling it out in prod.
How was this PR tested?

CI
Does this PR introduce any user-facing changes?

[X] No. You can skip the rest of this section.
[ ] Yes. Make sure to explain your proposed changes and call out the behavior change.

For hotfix select hotfix branch as base instead of linkedin:main


Python Client-Server SSL Communication channel test

Add Python client-server SSL channel with mTLS communication test.
This commit adds a standalone unit-test to exercise SSL communication
using mTLS between client and server. Communication channel is
established between the two using certificate and private-keys
generated by a stand-alone shell script that invokes 'openssl-req'
utility.
A simple 'hello' message is exchanged between client -> server,
followed by a client-side validation of expected message received
from the server.

MASSIVELY reworked test-cases to address comments from @rgerganov . We worked together on stabilizing the new test cases. Finally, we have these 5 test cases working now:

test_server_process_with_mtls_self_signed_cert() and test_client_app_with_mtls_self_signed_cert()
test_verify_certs_versus_root_cert()
test_server_process_with_mtls_root_signed_cert() and test_client_app_with_mtls_root_signed_cert()


pandasgui shutdown out of interactive cmd, pandasgui.show(df, settings={'block': True}) is needed.

Hi @adamerose ,
I am trying pandasgui in some python script, unlike in interactive cmd, it shutdown immediately after running.
I find settings={'block': True} is necessary, would you please figure it out in the demo examples?
Thank you and pandasgui is nice tool!

It's working properly for me:

I have some logic to make it so settings={'block': True} isn't needed in most cases, this is it:
def in_interactive_console():
    # https://stackoverflow.com/a/64523765/3620725
    return hasattr(sys, 'ps1')

if in_interactive_console():
    # Don't block if in an interactive console (so you can view GUI and still continue running commands)
    block = False
else:
    # If in a script, we need to block or the script will continue and finish without allowing GUI interaction
    block = True

How are you running the script? The only test case where it doesn't predict properly is when I run a script from PyCharm with this option checked, otherwise it always blocks properly

Hi,
I am running the following code on win10 anaconda python3.6, pandasgui.show() shutdown immediately without a block.
import pandas as pd
import pandasgui

class Render(object):
    def __init__(self, xlsx):
        self.store = pd.read_excel(xlsx, sheet_name=None)

if __name__ == '__main__':
    cr = Render('aaaa.xlsx')
    pandasgui.show(**cr.store)

Run pip install --upgrade pandasgui to make sure you have the latest version.

I am running the following code on win10 anaconda python3.6

But what method are you actually using to run your script, are you typing python myscript.py in Anaconda Prompt or what? I tried your code in a .py file and ran it from cmd and it blocked for me.

I have the same problem in Ubuntu 20.04.1, Python 3.8.5, and pandasgui <IP_ADDRESS>. I am running python myscript.py in a terminal where myscript.py is the basic usage example.

@c0ver

pandasgui <IP_ADDRESS>

I added the terminal detection & automatic blocking in 0.2.6

@c0ver

pandasgui <IP_ADDRESS>

I added the terminal detection & automatic blocking in 0.2.6

Running an upgrade shows the latest available version is <IP_ADDRESS> FYI.

@c0ver

pandasgui <IP_ADDRESS>

I added the terminal detection & automatic blocking in 0.2.6

Running an upgrade shows the latest available version is <IP_ADDRESS> FYI.

Oh! Thanks for pointing that out, latest versions are on PyPi now

@c0ver

pandasgui <IP_ADDRESS>

I added the terminal detection & automatic blocking in 0.2.6

Running an upgrade shows the latest available version is <IP_ADDRESS> FYI.

Oh! Thanks for pointing that out, latest versions are on PyPi now

Awesome v0.2.7 is working now in a .py script! Thank you!

Closing this since it sounds like this was just due to the PyPi release being older than the README... @forhonourlx please reopen if your issue still persists on latest release


Tasks need to understand react and redux

[x] Guide to React
[x] Guide to Redux

First, start with the first task and later work on the second task. This will help you understand react and redux.

@deep-guy how about you on this? I think @devilizer is ready to go on this?


Investigate Midi QOL compatibility

midi-qol is going to be taking over for Minor-QOL, we should ensure that things work as expected

Midi-QOL by and large requires no intervention from the Sheet itself to intercept rolls, unlike Minor-QOL. However the Item card buttons are not being appended as expected. These appear to only work for a limited list of "known" sheets which is defined in the Midi-QOL code. I've opened an issue suggesting sheet modules be allowed to register themselves with Midi-QOL on their own.

With the hooks changes in 0.7.x, this is no longer an issue.


SMF Unread helpdesk tickets overflows on mobile

Description
When on mobile if you visit ?action=unread, the table data will overflow.
This could be resolved with some CSS tricks

Think I added this on my pr not sure

Yea I just checked, it may be.  I need a few more prs committed, but I will test it more completely to be sure.


⚠ Bofrost.lu has degraded performance

In c7f2e6c, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1605 ms

Resolved: Bofrost.lu performance has improved in 82ba340 after 10 minutes.


Adds findChildUsingFrame

Please ignore I didn't intend for this to be created.  I"ll close the pr.


iOS: Fix warning about Plain Style

This warning no longer appears in Xcode 11 which is what we build iOS with now.


how i can get uicolor from ThemeColorPicker?

Hey anwsome, how i can get value color from ThemeColorPicker? many thank for you. :)

ThemeColorPicker will not save the original value, but will save a closure, the current theme used to obtain the final value.
This is an example:
// If the current skin index is 1, the UIColor object generated according to #FFF is returned
let color = ThemeColorPicker(colors: "#000", "#FFF").value!() as? UIColor

Optional(#EB4F38FF) this will return Optional(String), it will alway whitecolor.

@mhtranbn I tried it, value () normal return '# EB4F38FF` corresponding UIColor object, let me see your approach?

yup, i have tested, it ok. thank @wxxsw very much.

😄


sfpowerkit gives unknown/standard error in diff command when XML is invalid

When there is a Invalid XML the diff command gives a unknown/standard error rather than giving the exact error of Invalid XML [Filename]
To Reproduce
Generate a Invalid XML in any component
When the Diff Command runs it will give you a standard error instead of letting us know the exact error (Invalid XML/Component)

sfdx sfpowerkit:project:diff -d delta -r HEAD~ -t HEAD --json --loglevel debug

Expected behavior
It should exact error mentioning invalid XML and File Name.
Screenshots

OS: [e.g. Windows]

Thanks @satishpersonal  for raising this. While we are looking to address it better, I would recommend running xmlint before doing a diff as a workaround

Hi @satishpersonal
Is this issue has been resolved?

Hi Team,
As mentioned earlier, I'm using xmllint to resolve such kinds of issues.
On Wed, Dec 1, 2021 at 1:41 AM Henry @.***> wrote:

Hi @satishpersonal https://github.com/satishpersonal
Is this issue has been resolved?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Accenture/sfpowerkit/issues/591#issuecomment-983168532,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQNMLOXEXC5NPYOO5GBDDSTUOVVMTANCNFSM5G6TNV6A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

--
Best Regards,
Satish Inuganti,
+32-476174436

Closing this as there is no activity


migration: apply backoff only for evacuation migrations

Signed-off-by: Antonio Cardace<EMAIL_ADDRESS>
What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

Apply migration backoff only for evacuation migrations.

/lgtm

/approve

/cherrypick release-0.58

/retest

/cherrypick release-0.58


Parameterized type change

[x] Added change/3 callback for Ecto.ParameterizedType.
[x] Added change/3 callback implementation for Ecto.Enum.
[x] Added tests for new callback.

TODO:

[x] Ecto.Embedded support and tests for it
[x] Define default implementation for change/3 on use Ecto.ParameterizedType
[ ] Implement change_fun for arrays and maps. For arrays, we need to recursively call change on pair-wise. If the new_value has more entries than old_value, we need to consider the remaining entries are nil. For maps, we need similar logic and compare them based on the keys.

Related issue: #3380
cc @josevalim @TheFirstAvenger
Does it looks good? How should Ecto.Embedded work?

@josevalim 2 commit == 2 outdated, for 1 I was not sure so asked
Rest is related to embeds - I would need to add implementation for them at end.

@josevalim All suggestion done - I still need to make callback implementation for embeds
Should I wait for @TheFirstAvenger to see how things would go with cast and proposed by you cast_change (as it's related to embeds)?

All suggestion done - I still need to make callback implementation for embeds

It looks great. Besides embeds, we only need to define a default implementation for change/3 on use Ecto.ParamterizedType.

What we are going to do with embeds? You have mentioned cast_change before, so I'm unsure what to do.

You should be able to move the logic specific to embeds into Ecto.Embedded.change/3. You don't need to wait for cast_change, as this wouldn't dynamically depend on it. In order words, you should be able to add a clause like this:
  defp put_change(data, changes, errors, valid?, key, value, {:embed, relation}) do
    put_change(data, changes, errors, valid?, key, value, {:parameterized, Ecto.Embedded, relation})
  end

And it seems you will also need to change the c:Ecto.ParameterizedType.change/3 definition so it accepts {:ok, value, valid?} as return type. It is for when a change is accepted but some of its inner values are actually invalid.

@josevalim As you saw I have just added support for embeds. As you said I would implement change_fun for arrays and maps in separate PR, so for me it's good to go.

Note I have revamped the discussion on #3380. My suggestion for now is to not merge the latest changes we worked on and keep them around until someone has an effective use case, as we won't really be able to move all of embeds to parameterized types.

Closing this per #3380 but in case we get back to it, we will have a starting point. :) Thanks a lot! :green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


Failed to activate the tree-view package

[Enter steps to reproduce below:]

...
...

Atom Version: 0.189.0
System: linux 3.16.0-33-generic
Thrown From: one-light-ui package, v0.4.0
Stack Trace
Failed to activate the tree-view package
At variable @syntax-background-color is undefined in /usr/share/atom/resources/app/node_modules/one-light-ui/styles/ui-variables.less:14:13

LessError: variable @syntax-background-color is undefined
  at /usr/share/atom/resources/app/node_modules/one-light-ui/styles/ui-variables.less:14:13

Commands

Config
{
  "core": {
    "themes": [
      "one-light-ui",
      "tomorrow-night-blue-syntax"
    ]
  }
}

Installed Packages
# User
color-picker, v1.6.0
emmet, v2.3.6
symfony-snippets, v0.3.0
tomorrow-night-blue-syntax, v0.1.0
wordpress, v0.2.0
wordpress-api, v1.2.0

# Dev
No dev packages

This seems to be happening because you're using the one-light-ui theme with the tomorrow-night-blue-syntax syntax theme. The tomorrow-night-blue-syntax theme doesn't define syntax-background-color and  one-light-ui is trying to use it. I'm closing this in favor of https://github.com/atom/one-light-ui/issues/16


useAsyncFn - return Promise.reject() or Promise.resolve() in the promise resolution

Problem
const [registration, register] = useAsyncFn(Service.register)

async function handleSubmit(){
  try {
    await register(payload)
    // it always succeeds
  } catch(error){
    // it never reaches here
  }
}

I think you want to do the following:
const [{value: register, loading, error}, handleRegisterSubmit] = useAsyncFn(
  () => Service.register(payload),
  [payload]
)

That said, I'm not entirely clear on why the callback returns the value or error in the first place...

@wardoost because, as i remember, if value returned from then callback is not a promise - it is automatically wraped with fulfilled promise.

@wardoost what you suggested above gave me the same result, can't reach the catch(error) part of the code. It is impossible to handle an error with the given implementation of useAsyncFn.

I see what you mean now. I would have implemented your example like this as I would want the state represent how the async function resolved or failed:
const [{value, loading, error}, handleSubmit] = useAsyncFn(
  async () => {
    try {
      const result = await Service.register(payload)
      // handle result
    } catch (error) {
      // handle error
    }
  },
  [payload]
)

I can see there are use cases where you want the returned state of this hook to only represent part of some async function or handle failure differently with separate state. Your change seems to do this well but an error is show in the console when not catching errors which this hook is intended to do. I would recommend using this hook as I show above.

@wardoost I see you grasped  the idea, and the way you've shown above solves the problem if I only call handleSubmit once, which I think suits most use cases. However, if I reuse this handleSubmit in other parts of the code and want to handle errors differently depending on where handleSubmit was called, I cant do because I would only have one implementation of the catch block.
I've changed useAsync code a little bit

Now it won't complain about not handling error thrown on useAsyncFn's promise. Wdyt?

@diegoarcega This only solves the the problem of uncatched errors in useAsync, the problem still persists with useAsyncFn.
As you've mentioned my snippet above solves most use cases. The problem you're trying to solve seems to be the edge case which could be solved by checking if the awaited result is an error. If it is an error object you decide what to do with it, which can be handled differently in different places.

@wardoost
I understand that with useAsync one wouldn't want to handle errors in the callback, because this is only called once on component mount, no need to manually call the callback. However, with useAsyncFn an error can occur at any point while interacting with the app and the developer will want to not only show a message when the useAsyncFn returns an error, but also maybe make another request to the server or any other non UI related action.
Moreover, I wouldn't say catching an error after a failed promise is an edge case, it is more of a normal routine in software development. If an error occurs, I would expect the app to have a handler for that.
Checking for error in the success response doesn't seem like how standard apis should be made, more like a workaround imo.
One of the reasons why one would use a useasyncfn hook is its reusability but if the dev wants to reuse the callback in just one more place, it won't work, a new callback needs to be created but then you will be repeating the creation of the callback just because you are using it in another place and the hook doesnt support that.

@diegoarcega Catching an error after a failed promise is definitely not an edge case.. That's why errors are catched in all async hooks in this library and expressed in an object with loading, error and value keys to represent the async state of the promise in a declarative way. If you want to display an error message your would check the error key of the async state and display an error message accordingly as shown in the useAsyncFn docs.
You can reuse the callback in different places in your components but if a promise handles errors differently, and maybe fire off a second network request when that happens, it is not the same callback.

Checking for error in the success response doesn't seem like how standard apis should be made, more like a workaround imo.

I definitely agree on this hence why I said the following in my first comment:

That said, I'm not entirely clear on why the callback returns the value or error in the first place...

In my opinion the callback shouldn't return anything as to not confuse devs. It looks like it was put there as an escape hatch to do things you are trying to do.

Why not throw the error ?

I can't catch the error... I must check if the response is instance of Error... sad

Guys, I had the same problem and the quick workaround form me was introducing a new hook in my project to replace useAsyncRetry:
import React from 'react'

import { useAsyncFn as useAsyncFnOriginal } from 'react-use'

export const useAsyncFn = (...args: Parameters<typeof useAsyncFnOriginal>): ReturnType<typeof useAsyncFnOriginal> => {
  const [state, callback] = useAsyncFnOriginal(...args)

  const callbackEnhanced = React.useCallback(async (...cbArgs: Parameters<typeof callback>) => {
    const result = await callback(...cbArgs)
    if(result instanceof Error) throw result
    return result
  }, [callback])

  return [state, callbackEnhanced]
}

But will it only work when the promise will reject with an error!

Guys, I had the same problem and the quick workaround form me was introducing a new hook in my project to replace useAsyncRetry:
import React from 'react'

import { useAsyncFn as useAsyncFnOriginal } from 'react-use'

export const useAsyncFn = (...args: Parameters<typeof useAsyncFnOriginal>): ReturnType<typeof useAsyncFnOriginal> => {
  const [state, callback] = useAsyncFnOriginal(...args)

  const callbackEnhanced = React.useCallback(async (...cbArgs: Parameters<typeof callback>) => {
    const result = await callback(...cbArgs)
    if(result instanceof Error) throw result
    return result
  }, [callback])

  return [state, callbackEnhanced]
}

But will it only work when the promise will reject with an error!


Request to update the version of sympy

I was giving a live demo in class using Juliabox to solve some equations using SymPy.jl when I found that it gave an error when solving any equation with a constant at or below 10^(-8). Here is a minimal example of what I mean:
using SymPy
x = Sym("x");
solve(1e-8*x)

BoundsError()
while loading In[1], in expression starting on line 3
in solve at /home/juser/.julia/v0.3/SymPy/src/math.jl:287

The same code with 1e-8 replaced with 1e-7 works. This is a bug that is fixed in newer versions of SymPy, so I was wondering if we could update the SymPy installation used by JuliaBox from <IP_ADDRESS> to 0.7.6.

You can do Pkg.update() to get the latest version.

Unless you mean a newer version of SymPy and not SymPy.jl.

I mean SymPy, not SymPy.jl.

SymPy is currently being installed using apt-get and the ubuntu repository only has <IP_ADDRESS>. We could switch to pip which installs the latest version. But I also notice that installing with apt-get pulls in lot of other packages, which look like fonts and texlive, which the pip installer does not.
We could install the specific packages required if we have the list. But I suppose it would suffice to just do a pip install --upgrade sympy after the apt-get.
Just trying to avoid getting into the details of install specific dependencies.

Can the users do the pip install in their homedirectory and use that to override the system default?

Yes, I suppose users could do pip install --user --upgrade sympy and the updated package would be picked up by SymPy.jl.
@mw8 Does this work for you?

It's not ideal, but it will work. Thanks.

@mw8 I've also updated the docker script to install sympy 0.7.6. It will be available with the next update in a day or two. But you can also use pip install --user in the meantime. Thanks.

Much appreciated


qualitythought blog/

please forward the qualitythought blog credentials.

@kiranmaye24
http://qualitythoughtchefpuppetdocker.blogspot.com/

@kiranmaye24 @ramana152 @shaikkhajaibrahim
Hi All,
I'm unable to install Ubuntu OS 32 & 64 bits in Vagrant, i tried multiple OS's ,Still Frequently getting 404 Not Found,Please check and reply my<EMAIL_ADDRESS>Note:
Virtual Box: 5.2.22
Vagrant:1.9.6
Bringing machine 'default' up with 'virtualbox' provider...
==> default: Box 'ubuntu/trusty64' could not be found. Attempting to find and install...
default: Box Provider: virtualbox
default: Box Version: >= 0
The box 'ubuntu/trusty64 could not be found or
could not be accessed in the remote catalog. If this is a private
box on   ubuntu  s, please verify you're logged in via
vagrant login. Also, please double-check the name. The expanded
URL and error message are shown below:
URL: ["https://atlas.hashicorp.com/ubuntu/trusty64"]
Error: The requested URL returned error: 404 Not Found

Hi All,
I'm getting below error. How to fix the issue,Please suggest us.
Laptop Os:Win7SP1(64Bit)
prasad@prasad-PC MINGW64 /i/Vagos/Ubuntu
$ vagrant init ubuntu/trusty64
A Vagrantfile has been placed in this directory.  Please read
the comments in the Vagrantfile as well as documentation on
vagrantup.com for more information on using Vagrant.
prasad@prasad-PC MINGW64 /i/Vagos/Ubuntu
$ vagrant up
The version of powershell currently installed on this host is less than
the required minimum version. Please upgrade the installed version of
powershell to the minimum required version and run the command again.
Installed version: 2
Minimum required version: 3
[image: image.png]
Thanks
Ramprasad
On Tue, Nov 13, 2018 at 9:01 AM mcitp ram<EMAIL_ADDRESS>wrote:

Hi  Shaikkhajaibrahim ,
Very Good Morning!
Vagrant has been upgraded,Now i'm able install Ubuntu OS 32 & bit.
Thank you I really appreciate your support.
Thanks
Ramprasad
On Mon, Nov 12, 2018 at 4:51 PM shaikkhajaibrahim <
<EMAIL_ADDRESS>wrote:

Closed #260 https://github.com/QT-DevOps/DevOpsIssues/issues/260.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/QT-DevOps/DevOpsIssues/issues/260#event-1960015258,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ane0rHqukusyTlWKxdlNit0r7wmkl3Nwks5uuVnEgaJpZM4YYBvs
.

--
Thanks&Regards
Ramprasad.j
9985271433

--
Thanks&Regards
Ramprasad.j
9985271433


Add justfile

Just is a command runner similar to Make except readable: https://github.com/casey/just

I am merging this, I think having a "local" CI system or popular commands would be extremely useful.
It would also be cool to have our CI put into a Makefile / Just file so we can more easily move CI providers etc.
Also also there are some commands for Ares that are annoying to run :(


Support for Azure Pipelines

Hi,
I would like to know if exist an example to use setup-graalvm for Azure Pipelines.
Many thanks in advance
Juan Antonio

Hi @ESJBRE1, thanks for your question. We currently do not have an example that shows how to use this action in Azure Pipelines. In fact, I'm not even sure this is possible. Searching for Azure Pipelines and GitHub Actions suggests that users are migrating from AP to GHA. Assuming you are an AP user, do you know if GitHub Actions can be used in a pipeline?


adding names to our project

The story should go into some kind of project management tool, like Trello or Waffle.io. Please update. Thanks!


Direct Line ( WebChat) - Send Voice Note

Can we let the user send a Voice Note just like in Facebook Messenger or Whatsapp in Direct Line (WebChat) ?
What the samples currently have is an implementation of speech-to-text where whatever the user says on the microphone gets sent as a text to the bot.
What we need is the actual audio message sent by the user.
Is there a way to achieve this in the currently available WebChat ?
Thank you

This is achievable by utilizing features of Web Chat, yes.
First, when the user is given the option to provide an audio message and selects to do so, the bot can send an event to the web page. When the page receives the event a service can be initiated allowing the user to record a message. When the recording is stopped the audio file can then be sent back to the bot as an activity attachment.
I would suggest you look over the incming activity event and post activity event samples for reference.
Does that help?

@stevkan
Why will the bot need to send an event to the web page, can't we make it like when the user is given the option to provide an audio message (presses the mic icon), We start recording and that audio message is sent back as an activity attachment directly ?
Isn't the action of recording an audio message controlled from the client side ?
Is there a sample showing how to initiate an audio recording then capturing from the client side ?
That recorded audio message will then be sent  as part of an activity attachment
Thank you,

Closing this as answered. Feel free to continue discussion. :)

@marcasmar94, your original ask was for the ability to send a voice note (something similar to WhatsApp, for example). However, Web Chat isn't designed with that feature in mind. When the microphone is pressed in Web Chat, it is recording your dialog which it then forwards immediately to the bot as an activity. The dialog, as part of this process, is converted from STT.
Now, it may be that you can intercept the voice recording and do something with it, but Web Chat is still going to act on the spoken dialog and send it to the bot as text.
If you are looking for the ability to send an actual voice note as a file and not as dialog, then I would suggest you look over the implementation I presented above.
As an amendment or alternative, you could add a button alongside or in place of the send/microphone button that, when clicked, would offer the user the option to record a note (like WhatsApp). Setting the display attribute to "none" for the send/button still allows the user to send text by hitting "enter". Then the added button could record voice notes, at the user's request.
If you go this route, just be aware that you would be altering the DOM which could result in breaking changes depending on what code changes are published in the future.
Hope of help!


Data export feature

Issue fixed - download link to json representation of data provided. Lacks documentation, but otherwise good. Closing issue


Optimize custom tax logic

When a fee is paid in different denom than unls, then we perform a complex custom logic to determine if the fee is accepted and enough.
Optimize this custom logic to query concrete prices instead of querying all prices.
Validate the params on MsgUpdateParams by again doing a query to the contract and determine if params are okay.
Add a default value for the min_gas_prices in app.toml, hardcoded in the binary as we have for other required values such as commit_timeout.

Rename tax param contract_addr to treasury_addr
Use the base_price query for all custom fees(stables or volatile tokens) and the formula - feeAmount * (pricefeedAmountQuote/priceFeedAmount)


Add webui

This PR adds a basi WebUI for controlling the backend. The UI currently supports changing most backend settings, and automatically connects to torrent clients if their settings are changed. Other settings still require restarting the backend to take effect, but this may change in the future.

I had a hunch you were working on something big. Amazing.


Change separateExpression by expand

We rename the separate Expression method by the expand method

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

45 of 45 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.009%) to 36.457%

Totals

Change from base Build<PHONE_NUMBER>:
0.009%

Covered Lines:
5197

Relevant Lines:
14255

💛  - Coveralls


Configurable default gas for eth_call

Description:
This PR exposes the internal defaultGas parameter via an environment variable in order to support overriding the default 400_000 gas to a custom value. This supports consumers who cannot supply a gas amount for eth_call requests and perform low cost queries.
Related issue(s):
Fixes #982
Notes for reviewer:
Currently experimenting with a test for modifying static properties of EthImpl. Other untested properties are listed below:

ETH_CALL_CACHE_TTL
ETH_BLOCK_NUMBER_CACHE_TTL_MS
ETH_GET_BALANCE_CACHE_TTL_MS

Checklist

[x] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Codecov Report
Patch coverage: 50.00% and project coverage change: -0.05 :warning:

Comparison is base (f71ed83) 74.91% compared to head (e9ad29f) 74.86%.

:exclamation: Current head e9ad29f differs from pull request most recent head 9d9b373. Consider uploading reports for the commit 9d9b373 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1219      +/-   ##
==========================================
- Coverage   74.91%   74.86%   -0.05%
==========================================
  Files          31       31
  Lines        2260     2260
  Branches      442      443       +1
==========================================
- Hits         1693     1692       -1
  Misses        439      439
- Partials      128      129       +1

Impacted Files
Coverage Δ

packages/relay/src/lib/constants.ts
100.00% <ø> (ø)

packages/relay/src/lib/eth.ts
83.35% <50.00%> (-0.15%)
:arrow_down:

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

updated everything and added tests for the default gas override feature. still seeing failures with this test RPC Server: should execute "eth_getTransactionByHash with missing transaction" with a timeout error but it is also failing for me on the main branch without any changes

I fixed the rpc batch2 test but am not sure why the other one about HTS token transfers failed


Break css files and create a file for just imports/ Code Cleanup

Find a better way to organise styles

Actually explore using emotion or sumn


fix: treat empty valued env variables as false | FE

Description
Converting empty env values or no env value to boolean FALSE before storing in redux store. The empty-valued key in .env files was being treated as a true value in the frontend code just because the key was present in the response. This PR makes it false explicitely and fixes an attached bug. Fixes: #18419
Before

After
When value is not available

When truthy value is received

When false value is received

Type of change

Bug fix (non-breaking change which fixes an issue)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=293ddff

/ok-to-test sha=45c867c

@ankitsrivas14 git checkout release && git pull --rebase origin release && git checkout fix/email-settings-env-keys && git merge release and then git push origin fix/email-settings-env-keys

/ok-to-test sha=7ed449e


TypeScript type definitions are not complete

Only the Reports API is included. Any plan to enrich it? Thank you

Actually I have just extended the existing definitions with following code and it works as expected
declare module 'amazon-mws' {

    // Enriched type definitions for amazon-mws

    class AmazonMWSCall {

        search(params: any): Promise<any>;

    }

    class AmazonMWS {

        constructor()

        constructor(key: string, token: string);

        setApiKey(key: string, secret: string): void;

        sellers: AmazonMWSCall;

        orders: AmazonMWSCall;

        finances: AmazonMWSCall;

        reports: AmazonMWSCall;
    }

    namespace AmazonMWS {

    }
    export = AmazonMWS;
}

@miatidis Yes, It is in our plan and thanks to you for code snippets.

https://github.com/bhushankumarl/amazon-mws/releases/tag/v0.0.17
@miatidis Released with further type support.


Adds missed Getters for Span.

Missed Getters where added so internal Span data
became available outside of the package. Fixes #442.
Signed-off-by: Maxim Korolyov<EMAIL_ADDRESS>Which problem is this PR solving?
Not all span internal data is available outside of the lib.
Short description of the changes
Added missed Getters for logs, references and context.flags.

oh, can't merge. Do you mind resolving conflicts?


OpenWeatherMap sensor issue (?)

Home Assistant release (hass --version):
0.49.1
Python release (python3 --version):
Python 3.5.2
Component/platform:
Ubuntu 16.04 LTS
Description of problem:
Sensors (OpenWeatherMap) only display "unknown" while the map works.
Expected:
The sensors do display proper data.
Problem-relevant configuration.yaml:
sensor:
  - platform: openweathermap
    api_key: *********
    forecast: 0
    monitored_conditions:
      - weather
      - temperature
      - wind_speed
      - humidity
      - pressure
weather:
  - platform: openweathermap
    api_key: *********

Issue:
http://i.imgur.com/kXxvzPJ.png
Sensors (sensor.owm_...) do not work while the weather map works (weather.openweathermap).
Additional info:
OneTwo other persons on Discord also have this error.
The API key is the same for the sensor and the map (I tested multiple API keys).
http://i.imgur.com/5ZQtAUS.png
http://i.imgur.com/UkuS9n6.png
So it seems that the issue is only present in the latest updates.

I experienced the same issue since version 0.49
After a downgrade back to 0.48.1 HA received information from openweathermap. After reverting back to 0.49.1 the same issue appeared again.
Could it be related to https://github.com/home-assistant/home-assistant/pull/8276?

I turned on "debug" level for the weather logger and got this below in the homeassistant.log. So I launched a python session/command line and manually loaded the pyowm module and was able to successfully call pyowm and query the weather.  The below log would seem to indicate that openweather website is not responding to all site calls.  I don't understand what in HA could be causing the pyowm module to time out with the HTTP call to openweaternmap's servers.
Traceback (most recent call last):
  File "/home/hartmms/.homeassistant/deps/pyowm/commons/weather_client.py", line 55, in _lookup_cache_or_invoke_API
    response = urlopen(API_full_url, None, timeout)
  File "/usr/lib/python3.5/urllib/request.py", line 163, in urlopen
    return opener.open(url, data, timeout)
  File "/usr/lib/python3.5/urllib/request.py", line 466, in open
    response = self._open(req, data)
  File "/usr/lib/python3.5/urllib/request.py", line 484, in _open
    '_open', req)
  File "/usr/lib/python3.5/urllib/request.py", line 444, in _call_chain
    result = func(*args)
  File "/usr/lib/python3.5/urllib/request.py", line 1282, in http_open
    return self.do_open(http.client.HTTPConnection, req)
  File "/usr/lib/python3.5/urllib/request.py", line 1256, in do_open
    raise URLError(err)
urllib.error.URLError: <urlopen error [Errno 110] Connection timed out>

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.5/dist-packages/homeassistant/helpers/entity.py", line 225, in async_update_ha_state
    yield from self.hass.async_add_job(self.update)
  File "/usr/lib/python3.5/asyncio/futures.py", line 361, in __iter__
    yield self  # This tells Task to wait for completion.
  File "/usr/lib/python3.5/asyncio/tasks.py", line 296, in _wakeup
    future.result()
  File "/usr/lib/python3.5/asyncio/futures.py", line 274, in result
    raise self._exception
  File "/usr/lib/python3.5/concurrent/futures/thread.py", line 55, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/local/lib/python3.5/dist-packages/homeassistant/components/sensor/openweathermap.py", line 128, in update
    self.owa_client.update()
  File "/usr/local/lib/python3.5/dist-packages/homeassistant/util/__init__.py", line 303, in wrapper
    result = method(*args, **kwargs)
  File "/usr/local/lib/python3.5/dist-packages/homeassistant/components/sensor/openweathermap.py", line 189, in update
    obs = self.owm.weather_at_coords(self.latitude, self.longitude)
  File "/home/hartmms/.homeassistant/deps/pyowm/webapi25/owm25.py", line 257, in weather_at_coords
    'lang': self._language})
  File "/home/hartmms/.homeassistant/deps/pyowm/commons/weather_client.py", line 94, in call_API
    return self._lookup_cache_or_invoke_API(self._cache, url, timeout)
  File "/home/hartmms/.homeassistant/deps/pyowm/commons/weather_client.py", line 65, in _lookup_cache_or_invoke_API
    raise api_call_error.APICallError(str(e), e)
pyowm.exceptions.api_call_error.APICallError: Exception in calling OWM web API.
Reason: <urlopen error [Errno 110] Connection timed out>
Caused by: <urlopen error [Errno 110] Connection timed out>

I did a diff of the components/weather/openweathermap.py with the previous 0.48.1 version that was reported to work. The only change was to upgrade from an older pyowm version (2.6.1) to the new one (2.7.1). However, my testing I mentioned above proves the new version works ok stand alone.  This seems to indicate the problem lies outside the openweathermap.py code and something else inside HA.

@TheJulianJES - it's not nessassary to configure both "weather" and "sensor" as you have it above.  I only have the "sensor" part of the config in my setup and it still worked fine before.

I've also run into the problem with owm as soon as upgraded to 0.49.1 version.

This configuration works on dev:
sensor:
  - platform: openweathermap
    api_key: !secret owm_api
    forecast: 1
    monitored_conditions:
      - weather
      - temperature
      - wind_speed
      - wind_bearing
weather:
  - platform: openweathermap
    api_key: !secret owm_api

But the data for the sensor are not instantly available. After 30 seconds or so the sensors are populated.
To be honest I have never tested the sensor and the weather platform together. Someday the weather platform will supersede the sensor implementation for the meteorological data.

I can confirm with @fabaff's settings it works again.   For me, I had to add the "forcast: 1" and "weather:" sections.

I had exactly the same issue - after setting forecast to 1 it works again - this is a nice workaround, but any idea why only the forecast of next 3h works instead of actual values?

I'd like also to confirm that with "forecast: 1" option sensor started to work again. Thank  @fabaff for solution offered.

I need to look into it more deeply. Perhaps we are hitting an issue with caching or something.

Migrated from 0.35.3 to 0.50.2. Adding "forecast: 1" solved also for me
No need for the "weather:" section

I am also affected with 0.52.2. Setting forecast: 1 works. But doesn't that mean we get some forecast and not the current condition?

Adding forecast:1 fixed it for me. Maybe it should be added to the documentation.

Yeah, but was is the semantic of the change? Do we get other data then? Forecast for what?

Same for me, adding forecast:1 fixed it.

Same issue here. Adding forecast: 1 fixes the issue.

Does anybody know what exactly that does, besides "fixing the issue"?

It basicaly just sets the forecast to true. From the docs:

forecast
(string)(Optional)Enables the forecast. The default is to display the current conditions.

Default value: false

I did read the documentation and the semantics aren't super clear on that.

Yeah, but was is the semantic of the change? Do we get other data then? Forecast for what?

Is it tomorrow's forecast? How much forecast is it compared to current? Is it some 14-day-forecast? Is it a 0-day-forecast and we're just setting some flag? In my understanding a forecast is a guess about the weather for any number of possible times in the future.

I admit, I don't know. I'm guessing its just the current forecast, since the value is just true or false. But I didn't check, since I don't need it.
You should check the OWM API docs to find out.


Fix logs view

It was previously a custom view that duplicated a lot of behaviour that
already exists in ScrollTableView and DAOController. In this commit we
remove the custom view and replace it with a custom DAOController. Doing
so fixes the visual bugs and removes the redundant search box.
Fixes: https://nanopay.atlassian.net/browse/CPF-794

I added colours to LogLevels, so you can now just do ".style({ color: severity.color })" and remove the switch statement.
commit 53d7ed4
Author: Kevin Greer<EMAIL_ADDRESS>Date: Mon Apr 1 11:57:05 2019 -0400
Add colours to LogLevels.

Looks like we have two LogLevel models, and in that commit you modified the one we aren't using for LogMessages. There's foam.log.LogLevel and foam.nanos.logger.LogLevel. Looks like foam.log.LogLevel is used in 3 files and foam.nanos.logger.LogLevel is used in 4 files.
@kgrgreer Do you want me to refactor to get rid of one of the two or just add a color property to the other one?

Just use foam.log.LogLevel if we can.
On Mon, 1 Apr 2019 at 12:14, Troy Wolters<EMAIL_ADDRESS>wrote:

I added colours to LogLevels, so you can now just do ".style({ color:
severity.color })" and remove the switch statement.
commit 53d7ed4
https://github.com/foam-framework/foam2/commit/53d7ed41e702738b84d3c8e5bb7771d640b73745
Author: Kevin Greer<EMAIL_ADDRESS>Date: Mon Apr 1 11:57:05 2019 -0400
Add colours to LogLevels.
Looks like we have two LogLevel models, and in that commit you modified
the one we aren't using for LogMessages. There's foam.log.LogLevel and
foam.nanos.logger.LogLevel. Looks like foam.log.LogLevel is used in 3
files and foam.nanos.logger.LogLevel is used in 4 files.
@kgrgreer https://github.com/kgrgreer Do you want me to refactor to get
rid of one of the two or just add a color property to the other one?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/foam-framework/foam2/pull/2003#issuecomment-478644884,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHHXs9tnuv7j2kv5FmmmNoMYlJY1oEagks5vcjBigaJpZM4cV__q
.

[...] so you can now just do ".style({ color: severity.color })" and remove the switch statement.

Just use foam.log.LogLevel if we can.

Done in 6a13496.


Update intellij-idea-next-*-eap (2016.3 EAP) to build 163.7743.17

If there’s a checkbox you can’t complete for any reason, that's okay, just explain in detail why you weren’t able to do so.
After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if adding a new cask:

[ ] Named the cask according to the token reference.
[ ] brew cask install {{cask_file}} worked successfully.
[ ] brew cask uninstall {{cask_file}} worked successfully.
[ ] Checked there are no open pull requests for the same cask.
[ ] Checked that the cask was not already refused in closed issues.

there is an older PR including theses updates


Plugin sometimes ingests dozens of duplicates for kube-inventory data

What happened: kubernetes_objects plugin sometimes ingests into Elasticsearch a few times more data. For example, instead of reporting the state of 20 Kubernetes nodes, it sends 80.
For example, at the screenshot is amount of kube-node objects ingested by the plugin. We didn't do any cluster scaling or maintenance over this period: all these time we had exactly the same amount of nodes:

The same issue happens with pod and service objects.
What you expected to happen: the plugin sends only unique data withing specified in configuration period
How to reproduce it (as minimally and precisely as possible):
A part of fluentd config, dedicated to node objects:
    <source>
      @id kube_nodes
      @type kubernetes_objects
      tag k8s.objects.nodes

      <pull>
        resource_name nodes
      </pull>
    </source>
...
    <match k8s.objects.nodes>
      @id elasticsearch.k8s.nodes
      @type elasticsearch
      @log_level info
      type_name fluentd
      include_tag_key true
      host elasticsearch-logging
      port 9200
      logstash_format true
      logstash_prefix k8s-nodes
      logstash_dateformat %Y-%m-%d
      reload_connections true
      reload_after 50
      reload_on_failure true
      reconnect_on_error true
      prefer_oj_serializer true
      <buffer>
        @type file
        path /var/log/fluentd-buffers/k8s.nodes.buffer
        chunk_limit_size 2M
        total_limit_size 20M
        overflow_action block
        flush_mode interval
        flush_thread_count 2
        flush_interval 1m
        retry_type exponential_backoff
        retry_forever
        retry_max_interval 60
      </buffer>
    </match>

Anything else we need to know?: We run the service in Kubernetes, using default service account and ingest data directly to Elasticsearch, hosted in the same namespace
Environment:

Kubernetes version (use kubectl version): Server Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.6", GitCommit:"96fac5cd13a5dc064f7d9f4f23030a6aeface6cc", GitTreeState:"clean", BuildDate:"2019-08-19T11:05:16Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"linux/amd64"}
Ruby version (use ruby --version): ruby 2.3.3p222 (2016-11-21) [x86_64-linux-gnu]
'fluentd', '<=1.6.3', 'fluent-plugin-kubernetes-objects', '~>1.1.2', and 'fluent-plugin-elasticsearch', '~>3.5.3'
OS (e.g: cat /etc/os-release): Debian GNU/Linux 9 (stretch) from docker image debian:stretch-20190812-slim
Elasticsearch version: 6.2.4

Hi v1r7u!
Thanks for checking out our plugin!
Do you have any analysis/logs that show there are duplicates passed by the input plugin to the output plugin you are using?
Have you tried analyzing with a more verbose logging level or the fluentd stdout filter?
We do not support your chosen output plugin, but I’d recommend you check whether any sort of “acks” are used that could cause retransmissions.
You could also use the copy output plugin and send to a splunk trial u can run in k8s, see if you see the same thing :)

Excuse me for the late response.
I do not know how it works, but after creating the issue here - the problem had gone in all our clusters :see_no_evil:
The issue is created 24 days ago, last spike - ~20 days ago.
The only one thing I did for sure - switched off watching kube-events.
I had to investigate logs in more detail at the very beginning - thanks for the reminder :wink: If I see the issue again, I'll gather some logs for sure.
To be honest, I'm not eager to debug the issue further - if it works, let it be


git-quickfix: init at 0.0.4

Motivation for this change
Adds the git-quickfix tool.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Squashed with your formatting update and the fix for eval-check-meta failure.


fix: adjust release script

semantic-release has been failing ever since it was introduced, meaning that when the new change got released yesterday it actually published a package that was devoid of build 😬
(ideally v0.4.0 should be unpublished, but 🤷♂)

Thanks for bringing this up. I've fixed semantic-release.


SpatieMediaLibraryFileUpload does not work with Builder / Block

Package
filament/spatie-laravel-media-library-plugin
Package Version
v2.17.14
Laravel Version
v10.x (irrelevant)
Livewire Version
v2.12.3 (irrelevant
PHP Version
v8.1 (irrelevant)
Problem description
When uploading files using SpatieMediaLibraryFileUpload they are stored with the related model, but not the specific Block. This means that any uploaded file will be displayed with ALL blocks.
I have considered the following workaround: It's possible to specify a different collection for different Blocks. However, uploaded files to different Blocks using the same Block type they will still be added to all Blocks of the same type.
Expected behavior
I would expect SpatieMediaLibraryFileUpload to upload files to a specific Block instance. For example when adding an Image Block and then another Image Block I expect to be able to upload two different images.
Steps to reproduce
Given a Page model and a PageResource.
When I add a builder like this:
Forms\Components\Builder::make('blocks')->blocks([
    Forms\Components\Builder\Block::make('image')->schema([
        Forms\Components\SpatieMediaLibraryFileUpload::make('visual'),
    ]),
]),

And then procede to add an image to a Block, this same image will be added to all Blocks of the image type.
Before saving:

Aftter saving:

Note how both images are added to the Page model, but without any way to distinguish the block they belong to:

More detailed steps described in: https://github.com/dinandmentink/filament-spatie-bug-example
Some considerations:

I'm not sure how to fix this but maybe the Spatie\MediaLibrary\MediaCollections\Models\Media id could be stored as the value. Currently nothing is stored for the file in the blocks json.
For display it would also help a lot if the Media id would be stored. This could be used to retrieve the Media and then render it.

Reproduction repository
https://github.com/dinandmentink/filament-spatie-bug-example
Relevant log output
No response

Duplicate of #1284, media library isn't built for this unfortunately, storing the media ID in the JSON is not safe since it could be deleted in another place, and since there is no referential integrity, the file upload would break. I would advise using a normal file upload for this. If you have other suggestions, feel free to PR them.

Thanks for the response. That's a shame as I would really like to use medialibrary with the Builder / Repeater.  The default image/file upload does not allow for multiple conversions and responsive images.
The https://github.com/whitecube/nova-flexible-content does allow for this in laravel nova by adding the block's id to the medialibrary collection.
I'll see if I can get that behaviour to work. Thanks for the reply.

Media library uses https://github.com/spatie/image-optimizer for optimisations so maybe just use that


Contextview conditional children break the app if contextview primaryAction props change

Describe the bug
We encountered an app breaking in Firefox. What happens for us is that we have two async calls to the secret store to retrieve two values on top level of the app, and then we render a primary action button and the content of the ContextView
To Reproduce
See reproduction code, add it to a view and then wait for 5 seconds, then the app breaks in Firefox
import type { ExtensionContextValue } from '@stripe/ui-extension-sdk/context';
import { Box, ContextView, Button } from '@stripe/ui-extension-sdk/ui';
import { ContextViewProps } from '@stripe/ui-extension-sdk/ui/@stripe-internal/tailor-dashboard-components/internal/interface';
import { useEffect, useState } from 'react';

const APP_COLOUR = '#fff12c'

const CustomerDetailView = ({ environment }: ExtensionContextValue) => {
  const [data, setData] = useState<any | null>(null);
  const [otherData, setOtherData] = useState<any | null>(null);

  useEffect(() => {

    const timeout1 = setTimeout(() => {
      setData({ something: 'test' });
    }, 5000);

    const timeout2 = setTimeout(() => {
      setOtherData({ something: 'test' });
    }, 5000);

    return () => {
      clearTimeout(timeout1);
      clearTimeout(timeout2);
    };
  });

  const contextViewProps: ContextViewProps = {
    title: 'Customer events',
    brandColor: APP_COLOUR,
  };

  if (data?.something) {
    contextViewProps.primaryAction = (
      <Button
        type="primary"
        target="_blank"
        href={`https://${data.something}`}
      >
        A primary button
      </Button>
    );
  }

  return (
    <ContextView {...contextViewProps}>
      {(data && otherData) && <Box>SURPRISE!</Box>}
    </ContextView>
  );
};

export default CustomerDetailView;

Expected behaviour
The component renders properly
Screenshots
We can send you a video recorded in one of our apps of the breaking
Desktop (please complete the following information):

OS: [e.g. iOS] MacOS
Browser [e.g. chrome, safari] Firefox
Version [e.g. 22] latest

Additional context
There are multiple interesting behaviours that we can observe:

If you wrap the ContextView's content in a react fragment <></> the app stops breaking
If you change the setTimeout of timeout2 from 5000 to 5010 the app won't break, but only the primary action button gets rendered  useEffect(() => {

    const timeout1 = setTimeout(() => {
      setData({ something: 'test' });
    }, 5000);

    const timeout2 = setTimeout(() => {
      setOtherData({ something: 'test' });
    }, 5010);

   return () => {
      clearTimeout(timeout1);
      clearTimeout(timeout2);
    };
  });

If you change the order of the timers in the above provided code, it no longer breaks:  useEffect(() => {
    const timeout2 = setTimeout(() => {
      setOtherData({ something: 'test' });
    }, 5000);

    const timeout1 = setTimeout(() => {
      setData({ something: 'test' });
    }, 5000);

   return () => {
      clearTimeout(timeout1);
      clearTimeout(timeout2);
    };
  });

In the real world we have two secret store calls, and the app only breaks if the second secret store retrieval arrives back first, that is why reproducing it does not always work in our app.

Adding context: Team is able to workaround above issue, but is not the ideal way to implement.