Graphviz cannot render a large number of nodes

Hi Bro, today I use Graphviz library for my java project. But it cannot render a large number of nodes.
I wrote a createNode function and passed it into Graph.
But when I render it not working, the API is almost destroyed.
Note: I have about 822 nodes.
Please help me, I really need this help.
Thank you very much.

With 95 nodes, my API still works fine.

Hi
As a quick idea, you can try setting the memory with Graphviz.fromGraph(g).totalMemory(100_000_000).render(...

Hi
As a quick idea, you can try setting the memory with Graphviz.fromGraph(g).totalMemory(100_000_000).render(...

yup bro, I got an out of memory error in Graphviz exception. When I set totalMemory<PHONE_NUMBER>), I still get the error out of memory :(, so sad.

I faced the same problem, running at around 100 nodes. Worked fine with about 20-30, then any higher number it failed with:
205 Exception in thread "main" guru.nidi.graphviz.engine.GraphvizException: Got Error: 'TypeError: Module.print is not a function'. This is probably an out of memory error. Try using the totalMemory method.

However, after attempts at increasing java heap size, and setting total memory to MAX_INT, I didn't want to give up.
I saved the generated dot in a text file, and dumped it into Graphviz Online. Sure I selected the engine I was using and I received the following error:
abort("Assertion failed: np->cells[1], at: maze.c,313,chkSgraph"). Build with -s ASSERTIONS=1 for more info

Did a bit of googling, just to figure out from this issue (Graphviz Gitlab Issue 1408) that there is a bug within orthogonal layout code. I switched to using POLYLINE instead of ORTHO and everything works ok now.
You could be facing something like this as well. I guess, from this project's perspective we could improve on the error handling and maybe include the underlying error from the engine in the error we log, instead of assuming it's a memory problem.

Thanks @MadaManu for pointing this out. With version 0.18.1 messages from the engine are logged directly.
@luongbaotin7777 I was not able to reproduce your problem, could you try again with version 0.18.1 to see if there's a better logging message now?

@nidi3 Great point indeed. With regards to the example I've outlined above, upgrading to 0.18.1 doesn't even fail anymore using ORTHO. (I was previously using 0.18.0)

Thanks @MadaManu for pointing this out. With version 0.18.1 messages from the engine are logged directly.
@luongbaotin7777 I was not able to reproduce your problem, could you try again with version 0.18.1 to see if there's a better logging message now?

I used ORTHO and total memory, it worked. thank you.


Fixes for Spekulatius

New garage-sign interface
Remove garage-sign recipe (now installed with aktualizr-native)
Small but critical bugfixes in aktualizr

@patrickvacek
The intermediate result of our discussions about root.ca as far as I understood it was the we're not going to update root certificate until we have a common understanding about how to do it properly. Therefore, there is no need to treat root.ca specially for now, so I've just set --no-root (you meant --no-root, right?) unconditionally.

@OYTIS I thought there was a --no-auth flag for garage-sign init, but now I'm not seeing it. What happened to that?

@patrickvacek
Ah, OK. There's no plan, garage-sign now just looks into the credentials (namely authentication objects in treehub.json) and acts accordingly.

Cool, sounds good to me.


Copy & Paste Alias Plugin from/into the same placeholder causes infinite loop

... and this seems to make the edit page inaccessible which makes it impossible to fix the thing.
The only solution I found is removing the plugin entry directly on the database level.
On Aldryn it's even more severe because there the db needs to be copied to stage and then to local, then  manually fixing, and then the full round-trip back to production.
I'm not saying copying / pasting an alias into the same placeholder makes much sense but sooner or later somebody does it... dooh.

thank you for reporting @macolo, this has been confirmed, will be patching the Alias plugin soon enough.

@mkoistinen I just found out another facet of this:
DjangoCMS allows the editor to create an alias on the placeholder level. This alias can then be placed inside the very same placeholder on another language, which also triggers the infinite loop.
So we have two scenarios:

the editor creates an alias of an element inside a placeholder and pastes it on the same language/page combination (same placeholder?) which triggers the infinite loop. (unconfirmed).
the editor creates an alias of the placeholder and pastes it on a different language/page into the same placeholder which triggers the infinite loop (confirmed).


More examples

Background
Library is great for manipulating biological data. On target!
Proposal
However, examples on how to implement functions and types are virtually non-existent. Could you add more examples, please?
I'm particularly having trouble writing fasta sequences. Appreciated. Cheers
Potential impact of proposal

The best place to look is in https://github.com/biogo/examples. I'm no longer actively involved in bioinformatics work, so I will not be able to find the time to add further examples.

Thanks for your response. I'll check it out and close the issue then.


CLI freeze with -it

I ran into an issue with a 3-node swarm (1 in DO, 2 in Vbox, master was in vbox) setup via docker-machine.
Basically doing a simple docker run -it --rm debian:jessie and then trying to exit caused the docker CLI to completely freeze, couldn't ctrl-c or anything.
Had to kill -9 the docker client from another terminal. Upon doing this that terminal the client was using was fubar'd as well.
Once I got to this state, any time I did the above command I would get a CLI freeze upon exit.
Removing the cluster machines and creating a new cluster resolved the issue.

Thanks @cpuguy83 - maybe related to #792 and #767
What's your setup ? As in, is the CLI running on a different machine than Swarm, is Swarm running on a container, what OS is the CLI running on, etc.
Thanks

CLI is running on OSX.
Swarm on all 3 nodes were setup by machine and running in a container.
1 swarm node was in digital ocean, 2 were in virtualbox.
Master was in virtualbox.

Thanks for the input, will try to reproduce using the same setup

Hey @cpuguy83 - I believe we can close this down?

Yep.


after moving files and deleting directory the files where moved out of

Product and Version [VS/VSCode]:
OS Version [macOS/Windows/Linux]:
Live Share Extension Version:
Target Platform or Language [e.g. Node.js]:
What were you doing when you encountered the desync?
How often have you encountered desyncs?

@MManthey thanks for reporting this issue!
If you can get Live Share logs and attach them to this issue, it'll help us investigate. You can get the logs by running Live Share: Export Logs command in VSCode.

Closing this due to inactivity. Please reopen it if you’re still encountering the issue. Thanks!


gitbub link shows discord image

in warden protocol explorer website left side down corner , gitgub website shows discord logo which is incorrect , Thankyou !

Hey, thanks for reporting this but this explorer is not operated by us. I'll forward this to Paranormal brothers and close this for now.


Is there a way suppress "Progress: " printing to stdout

It looks like there is already an option to do this in electricdss-src. If this is already exposed, can you point me to it? If there's not can we add a way to do this?

I don't think there is a global switch fully implemented for all messages yet. We can reuse NoFormsAllowed though -- it seems there was an intent to implement this but it was never finished for the DDLL and OpenDSSCmd.
It's a small change (a couple of conditionals) and we'd end up with implementations for DSS.AllowForms in both dss_python and ODDpy. I'll work on this before the next release.

Added an implementation for AllowForms: by default AllowForms is false (it already was before, but the there was no logic implemented to use it) but it can be toggled to show progress and misc messages in stdout.
Let's leave this open until the next release. I also implemented it for ODDpy in https://github.com/PMeira/OpenDSSDirect.py/commit/311c2cc5fa56eb6b080e81e46565b5dd6e6a4e9c and tested locally. Since it would fail at the moment, I'll make a PR as soon as the next DSS Python is released.


Screen got turned off while reading

The reading activity screen of Folio reader get turned off while using more than 4 minutes . Fix i found was
getWindow().addFlags(WindowManager.LayoutParams.FLAG_KEEP_SCREEN_ON);
Unfortunalty it must be added from inside the library activity . Any possible fix ?

You can extend your activity from FolioActivity and add your flags

Didn't work


[FR] Decrease IPA size

IPA size a still large, please try to make it smaller

+1

+1

I believe having an acceptable app size is pretty much critical for production use.


Language fallback is wrong in tut example

The documentation says
scala> greeting(Locales.de_DE)
res1: String = Grüß Gott Welt

It should however evaluate to de"Hallo Welt".

Fixed while migrating to mdoc


Fix cspell for ComputeSchedule

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.ComputeSchedule


[test] add config test for zookeeper

Description: Fills in some missing coverage https://s3.amazonaws.com/lyft-envoy/coverage/report-master/coverage.source_extensions_filters_network_zookeeper_proxy_config.cc.html cc @rgs1
Risk Level: Low
Testing: Included
Docs Changes: N/A
Release Notes: N/A
Signed-off-by: Derek Argueta<EMAIL_ADDRESS>

looks great, thanks!


[darwin] CHIPTool supports in service discovery only full 128 B BLE service UUID format.

Problem
In the didDiscoverServices method there is a service.UUID.data isEqualToData:chipServiceUUID.data condition, which is fulfilled only in case of using 128 B size UUID and fails for the short 16 UUID format. I think it would be good to support both size formats, so I would like to ask if it's intentional operation or it should be expanded about short UUID format support?
Proposed Solution
If there are not any objections, add support for short 16 B UUID format in the service discovery.

@shana-apple are you working on this? I saw that #4098 was merged, so actually my problem was solved. I wrote about supporting both 16 and 128 bit UUID formats, because I thought that there are some reasons to use full format on the ESP32, but if all samples were aligned to use 16 bit I would like to close this issue.

@shana-apple are you working on this? I saw that #4098 was merged, so actually my problem was solved. I wrote about supporting both 16 and 128 bit UUID formats, because I thought that there are some reasons to use full format on the ESP32, but if all samples were aligned to use 16 bit I would like to close this issue.

No I am not yet so feel free to close if it is not applicable anymore.


Regression: Export does not work anymore

Signaled by Microsoft on Windows S. Also experienced on MacOS.
Steps to Reproduce:

 Launch the application.

Select the three dots contextual menu on a book.

Select “Export”.

Notice no action is taken.

Expected result:
The controls within the application should work as expected and should not present “coming soon” messages.

It appears was is not an api issues, but more on react html views.
It seems to me, @Scarsniik ?

The "export" feature only fails in this specific case:

bookshelf cover view
mouse interaction
(works with keyboard interaction, and/or bookshelf list view)


add R age to hover info quality of assessment

I like the idea of adding rage - we should do that more often :D


Two-way binding is broken in C#

Describe the bug
Setup:

a collection of items (which implement INotifyPropertyChanged), call this the main collection, of type List<Foo>
an AutoSuggestBox and ListView. Based on the text in the AutoSuggestBox, I want the collection to be filtered. So I have an ObservableCollection<Foo>. I update the filtered collection in the ASB's events. This all works fine.
The ListView has an item template where I show each item as a checkbox. The IsSelected property is bound to a bool Selected in Foo, with Mode=TwoWay. It is expected that when I select/unselect the checkbox, the instance of Foo is correspondingly  updated.

However, what I am seeing is that in my filter function, each time I iterate through the main collection, I get a copy of the Foo item; this copy is the one that gets inserted into the Filtered collection, therefore the writes from clicking the checkbox aren't propagated to the main collection.
Steps to reproduce the bug
XAML:
    <ListView x:Name="list" ItemsSource="{x:Bind Filtered}">
      <ListView.ItemTemplate>
        <DataTemplate x:DataType="local:Foo">
          <CheckBox IsChecked="{x:Bind Path=Selected,Mode=TwoWay}" Content="{x:Bind Path=Name}"/>
        </DataTemplate>
      </ListView.ItemTemplate>
    </ListView>

C#
  public class Foo : INotifyPropertyChanged
  {
    public string Name { get; set; }
    public bool Selected { get; set; } = true;

    public event PropertyChangedEventHandler PropertyChanged;
  }

// in the Page ...
    public ObservableCollection<Foo> Filtered { get; private set; } = new();
    public IEnumerable<Foo> MainCollection { get; private set; } = new List<Foo>();
    private void Filter(string filter)
    {
      Filtered.Clear();

       foreach (var r in MainCollection) { Debug.WriteLine(r.GetHashCode()); }
       foreach (var r in MainCollection) { Debug.WriteLine(r.GetHashCode()); }  // <-- observe different values are printed out to the debugger output

      if (string.IsNullOrWhiteSpace(filter))
      {
        foreach (var r in MainCollection) Filtered.Add(r);
      }
      else
      {
        foreach (var r in MainCollection.Where(r => r.Name.Contains(filter, StringComparison.InvariantCultureIgnoreCase)))
          Filtered.Add(r);
      }
    }

Expected behavior
Bound object is correctly updated
Screenshots
No response
NuGet package version
WinUI 3 - Windows App SDK 1.1.4
Windows app type

[ ] UWP
[X] Win32

Device form factor
Desktop
Windows version
Windows Insider Build (xxxxx)
Additional context
No response

I suspect this may be a cswinrt issue with object identity. @manodasanW @Scottj1s @j0shuams

nevermind, pebkac.


Is there a convention for shipping npm packages that play nicely with this plugin?

Is your feature request related to a problem? Please describe.
We have a number of internal npm packages which contain translations (e. g. think tooltips in a common widget). It would be nice if those packages could be set up in a way that they would "just work" for someone using this plugin but also wouldn't completely break for a consumer who was not using it.
Describe the solution you'd like
I'm imagining something like this:
// in index.js in the package

function __(key) { return null; } // fallback definition for __ if plugin not in use

// messages is an optional argument containing the translations lookup dictionary.
// Doesn't need to be provided if the plugin is in use
export function doStuff(messages) {
    // Note that I'm using the package name as the key prefix to avoid collisions with keys in other packages
    // and in the application itself.
    const localizedString =<EMAIL_ADDRESS>|| translations.keyName
    ...
};

// in webpack-localize-assets-plugin.json shipped alongside index.js. In my ideal world the plugin would pick this up by default
{
    "en-US": {<EMAIL_ADDRESS>"foo" }
     ...
}

Describe alternatives you've considered
I imagine that this could be built on top of the plugin, although one thing that would get tricky is if multiple versions of a translated package are included in the compilation as nested modules since we'd need to make sure to use the correct json file.
Thoughts?

I can't tell if there's a proposal here (given it's a feature request rather than a discussion question), but to answer the issue question, there is no convention.
There is no way to provide a fallback function at run-time because Webpack's static analysis requires that the localize function is a global. However, the consuming app can add fallback logic via the new localizeCompiler API.
You can ship a package with translation JSONs, and your consuming app build can merge them with the apps translation JSONs before passing it into this plugin.


Player Props

I need to get player props from somewhere for the games that need it.

I have webscraped rotowire.com and retrieved the player props from DraftKings for games happening on the same day. This will serve the statistical purpose I need but acquiring stats over a day out will likely just not exist.

7a9dafb


Lab_1 first week additions

@ValeriyaKuznetsova , please run checks for new tests

@ValeriyaKuznetsova, please run checks for new tests


Add odo push to link/unlink tests

What type of PR is this?
I don't know. It's pretty sure not a bug! This PR is mainly opened to fix something that's working just fine on developer/QE systems but failing in CI.
What does does this PR do / why we need it:
This is being done for two reasons:

CI is likely slow to link/unlink as observed in
https://github.com/openshift/odo/issues/3834
link/unlink require "odo push" for underlying deployment

Which issue(s) this PR fixes:
Fixes #3834
PR acceptance criteria:

[ ] Unit test

[x] Integration test

[ ] Documentation

[ ] I have read the test guidelines

How to test changes / Special notes to the reviewer:

@prietyc123

TBH I don't understand why we are bandaging it with odo push when its not necessary.

I already explained why we're adding this in https://github.com/openshift/odo/issues/3834#issuecomment-680780602.

Maybe your current changes will hide the issue but it won't fix the real issue.

The real issue has been pointed to by @amitkrout in https://github.com/openshift/odo/issues/3834#issuecomment-680713364. It's a race condition showing up on CI. Maybe you can fix that and I'll close this PR in that case.

Also frankly speaking as a end user I won't rely much on .odo/env/env.yaml for services state failure.

We're relying on this file for information about the link. Not for the state of the service.

Don't really understand why oc get csv would fail. Maybe a flake?
/test v4.5-integration-e2e
++ oc adm policy add-role-to-user edit developer
Warning: User 'developer' not found
clusterrole.rbac.authorization.k8s.io/edit added: "developer"
++ sh ./scripts/setup-operators.sh
+ CI_OPERATOR_HUB_PROJECT=ci-operator-hub-project
+ count=0
+ '[' 0 -lt 5 ']'
+ oc get csv -n openshift-operators
+ grep mongo
I0826 10:01:42.215739      87 request.go:621] Throttling request took 1.134171471s, request: GET:https://api.ci-op-17f0tnww-2f74e.origin-ci-int-aws.dev.rhcloud.com:6443/apis/batch/v1?timeout=32s
error: the server doesn't have a resource type "csv"
+ install_mongo_operator
+ oc create -f -

The real issue has been pointed to by @amitkrout in #3834 (comment). It's a race condition showing up on CI. Maybe you can fix that and I'll close this PR in that case.

The race condition i want to describe here is
For command odo link EtcdCluster/example i have not much idea what process it runs behind the scene but as a user i would expect the next command odo unlink EtcdCluster/example should not fail due to some pending status of the previous command. May be when link or unlink command complete but it still wait some process to complete under the hood.
@prietyc123 Last time i was using the the complete spec, instead you can scope out few things from there and focus the failure steps. For example you can please try these simple scenario locally.
odo unlink EtcdCluster/example
odo link EtcdCluster/example
odo link EtcdCluster/example

For command odo link EtcdCluster/example i have not much idea what process it runs behind the scene

It creates a Service Bbinding Request. "Service Binding Request" (SBR) is an API provided by Service Binding Operator (SBO). Due to certain limitations, we had to introduce odo push as a part of the workflow for the linking to take place well. More info can be found in #2920.

May be when link or unlink command complete but it still wait some process to complete under the hood.

On the cluster side, link/unlink are incomplete unless odo push was executed. When working with Operator Hub, odo makes sure that there's clear messaging about requirement to do odo push:
$ odo link EtcdCluster/example
 ✓  Successfully created link between component "node-todo" and service "EtcdCluster/example"

To apply the link, please use `odo push`

$ odo unlink EtcdCluster/example
 ✓  Successfully unlinked component "node-todo" from service "EtcdCluster/example"

To apply the changes, please use `odo push`

/approve

Thanks for detailed information. Looks good to me.
/lgtm

/retest


86 implement deploy dns with blacklisted values

The goal is to add 3 scripts :

deploy.ts to deploy the dns SC.
src/addDnsEntry.ts to add a new website to the deployed dns within a CallSC.
src/blacklist.ts to blacklist a list of domain names in the deployed dns within a CallSC.

moved to private repo https://github.com/massalabs/massa-sc-private


Cosmetic change

Q
A

Bug fix?
[x]

New feature?
[ ]

New sample?
[ ]

Related issues?
fixes #X, partially #Y, mentioned in #Z

What's in this Pull Request?
Removed unwanted new line in help content.

Than you @sankarkumar23 for the PR!
It has been merged and will be included in the next release.

Than you @sankarkumar23 for the PR!
It has been merged and will be included in the next release.


Mean-Average-Precision gives unexpected result.

🐛 Bug
Changing the order of bounding boxes in Mean-Average-Precision changes the result.
To Reproduce
Steps to reproduce the behavior...
from torch import tensor
from torchmetrics.detection import MeanAveragePrecision
preds = [
   dict(
     boxes=tensor([  [214.0, 41.0, 562.0, 285.0], [600.0, 300.0, 900., 600.],  ]),
     scores=tensor([1., 1.]),
     labels=tensor([1, 1]),
   )
 ]
target = [
   dict(
     boxes=tensor([[214.0, 41.0, 562.0, 285.0]]),
     labels=tensor([1]),
   )
 ]
metric = MeanAveragePrecision()
metric.update(preds, target)
from pprint import pprint
pprint(metric.compute())

switching the two boxes in preds changes the result from 1 to .5
Expected behavior
the results should be independent of box order.
Environment

TorchMetrics version: 1.0.1  using pip
Python: 3.10.10 & PyTorch: 2.0.1
running on linux: "Debian GNU/Linux 11 (bullseye)"

Duplicate of: https://github.com/Lightning-AI/torchmetrics/issues/1774
TLDR: the result are not unexpected and that is the exact same behaviour the official implementation would have.
Long answer:
The core problem is that to calculate the MAP, we first need to calculate the precision-recall-curve. This is done by first sorting our predictions based on their confidence. This is a problem when predictions have the exact same confidence (as in your example) then this sorting is not unique and therefore depends on the order of the input.
If we calculate the precision recall curve for the two cases:
correspond to the wrong prediction being first

Conf
Matches
Cumulative TP
Cumulative FP
Precision
Recall

1.0
TP
0
1
0/(0+1) = 0
0/1=0

1.0
FP
1
1
1/(1+1) = 1/2
1/1=1

correspond to the right prediction being first

Conf
Matches
Cumulative TP
Cumulative FP
Precision
Recall

1.0
TP
1
0
1/(1+0) = 1
1/1=1

1.0
FP
1
1
1/(1+1) = 1/2
1/1=1

Afterwards we would interpolate the two curves and calculate the area under them which would give us the MAP value. I wont go into details on how to this, but it should not be surprising that if the two curves are different then the area under them will also be different. For the first case you end up with a 0.5 value and the second case a value of 1, as you have already seen.

Thank you for the explanation. I still have problem understanding the results.
In the case for right prediction being first shouldn't the extra false positive prediction reduce the AP ?
to be clear we can give the wrong prediction score of 0.9.

@kfarivar no problem. I try with more details:
Your original question was regarding why the input order mattered for the score the metric produced. Hopefully, my last answer makes it clear that it has to do with the the two predictions having equal confidence leading to the input order having a influence when we calculate the precision recall curve.
Thus if we change the wrong prediction to have a 0.9 confidence we will correctly see that the input order does not matter, the result will be 1.0 in both cases e.g
from torch import tensor
from torchmetrics.detection import MeanAveragePrecision
from pprint import pprint
preds = [
   dict(     boxes=tensor([  [214.0, 41.0, 562.0, 285.0], [600.0, 300.0, 900., 600.],  ]),
     scores=tensor([1., 0.9]),
     labels=tensor([1, 1]),
   )
 ]
# change order for both boxes and confidences
preds2 = [
   dict(     boxes=tensor([  [600.0, 300.0, 900., 600.], [214.0, 41.0, 562.0, 285.0] ]),
     scores=tensor([0.9, 1.0]),
     labels=tensor([1, 1]),
   )
 ]

target = [
   dict(
     boxes=tensor([[214.0, 41.0, 562.0, 285.0]]),
     labels=tensor([1]),
   )
 ]
metric = MeanAveragePrecision()
pprint(metric(preds, target))  # 1.0 in map
pprint(metric(preds2, target))  # also 1.0 in map

Regarding your new question:

In the case for right prediction being first shouldn't the extra false positive prediction reduce the AP ? o/w the model with lots of false positives wouldn't be any different from one without ?

I understand if this is counter intuitive, but the core thing for this particular simple case is that if we match the target box on the first prediction, then it will not matter how many we get wrong afterwards.
If we take this prediction for example:
preds3 = [
   dict(     boxes=tensor([
                [600.0, 300.0, 900., 600.],
                [600.0, 300.0, 900., 600.],
                [600.0, 300.0, 900., 600.],
                [600.0, 300.0, 900., 600.],
                [600.0, 300.0, 900., 600.],
                [214.0, 41.0, 562.0, 285.0] ]),
     scores=tensor([0.9, 0.9, 0.9, 0.9, 0.9, 1.0]),
     labels=tensor([1, 1, 1, 1, 1, 1]),
   )
 ]

And repeat the precision recall calculation:

Conf
Matches
Cumulative TP
Cumulative FP
Precision
Recall

1.0
TP
1
0
1/(1+0) = 1
1/1=1

0.9
FP
1
1
1/(1+1) = 1/2
1/1=1

0.9
FP
1
2
1/(1+2) = 1/3
1/1=1

0.9
FP
1
3
1/(1+3) = 1/4
1/1=1

0.9
FP
1
4
1/(1+4) = 1/5
1/1=1

0.9
FP
1
5
1/(1+6) = 1/6
1/1=1

Importantly the recall value is 1.0 regardless of how many false positives we observe after the first true positive.
Here is a quick paint drawing, where the red points correspond to the numbers in the table and the green line is the curve we would interpolate under:

As you can see, it does not matter how many false positives we add because they all are on the line we already are using for interpolation.
Hope this clears it up a bit more :]


Problem in FreeBSD compiling

Hello, just wanted to say that for compiling in freebsd the line <include "sys/stat.h"> must be added in save.c

Ok thank you  for letting know. I just added it.

I like your project. Was "playing" a bit with it,

talked to the bot a little :)


More efficient pretty printer

Currently, pretty printing a list of size 300 will cause stack overflow

No longer the case


[breaking] stylis v4

moves off @emotion/stylis and attempts to retain backward-compatibility
for everything except stylusPlugins, which cannot be made compatible
due to a vastly different middleware setup in v4
note that some of the tests will be broken until I revise stylis-plugin-rtl for the v4 middleware signature

@kitten this is ready for final review


[Sprint 41] XD-2542 Create MessageHandler for RxJava based XD processor modules

Also includes XD-2541 - Define RxJava processor interface
Noticed some extraneous println to be removed.
I think we can address most of the TODOs in a separate PR/Issue aside from logging when no match is found on message/payload type.

I somehow got my local checkout branch out of sync with my origin, so closing this PR and issuing a new one.


Windows tests turned on (TEST)

Great attempt. I didn't know github actions support windows! The error shows we need to build dependant binary tools for windows first?

@ailisp it was an error thrown by us. I have improved our post-install script and whitelisted windows. Now it is throwing the right error.


Upload coverage to codecov

This will upload coverage for master as well.

This change is

Current coverage is 20.13% (diff: 100%)

No coverage report found for master at 36ee63b.

Powered by Codecov. Last update 36ee63b...ce0699d


Add conditional

Adds conditional option so that I can also update src so long as lazyloading has already happened.

Sorry, raised here by mistake.


ejs in template literal support

I often use js template literals to store the ejs templates, like this
var template = `

<%= name %>

`;

and then render the template with element.innerHTML(ejs.render(template, {name: "Simon"}));
Only problem is the template string is not syntax highlighted.
Would it be possible to add this feature somehow like lit-html does?

Can you verify this with version >1.0.0. It may have been fixed.

Hi @Betanu701, I'm not using template literals anymore. I switched to single template files with ejs-render-remote.

literals have been added in version 1.1.4 (release 1.2)


cleanup and c++ standard fixes

switch to c++17 to avoid compile error by default
fix signed and uninitialized warnings
fix missing cstring include
switch  c xxx.h includes to cxxx and add std namespace

Did not merge cleanly, was manually merged


Forty-Forth Community Call

The next call will be on Thur, Feb 23rd at 6 pm CET (1700 UTC, 12 pm EST, 9 am PST). We will aim to include more projects from America this time.
Agenda:

Update on Public Goods (explain article)
Cosmos Hub Proposal 103
ETH Denver
Academy 3rd cohort
AwesomWasm
Open Q&A

Zoom link: https://us06web.zoom.us/j/85741688333?pwd=OTBSanFhamVCUWdlZHNrbG9kN1M4Zz09

Hey @SaraDarcaj, could you provide a calendar invite please ?

The recording of the community call has been uploaded for all to view: https://vimeo.com/801679826


feat: Deep tree structure with Updates

This PR is just a reference to #5 which adds a deep tree structure with updates, however, it was forked off of master which uses cosmos-db as the underlying DB, however, the default branch in cosmos-sdk-rollmint: release/v0.46.x-rollmint uses iavl 0.19.4 which corresponds to release/v0.19.x in this repo and uses tm-db. Also, it was opened by @tzdybal so he could not approve it. We'll use this PR as a proxy for that PR in order for approval.
Please look at #5 for relevant comments.

Codecov Report

:exclamation: No coverage uploaded for pull request base (deepsubtrees@f4499a9). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff               @@
##             deepsubtrees      #13   +/-   ##
===============================================
  Coverage                ?   64.56%
===============================================
  Files                   ?       28
  Lines                   ?     4617
  Branches                ?        0
===============================================
  Hits                    ?     2981
  Misses                  ?     1258
  Partials                ?      378

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

Closing this, thanks for all the valuable comments @MSevey and @tzdybal!


Next Linguist deployment?

Preliminary Steps
Please confirm you have...

[x] reviewed How Linguist Works,
[x] reviewed the Troubleshooting docs,
[x] considered implementing an override,
[x] verified an issue has not already been logged for your issue (linguist issues).

Problem Description
As far as I can tell, it looks like the last Linguist deployment was in March with v7.9.0. It looks like there have been a lot of changes merged as well as changes to Grammars. It would be nice to get an ETA on the next deployment.
URL of the affected repository:
N/A
Last modified on:

N/A
Expected language:

N/A
Detected language:

N/A

I'm aiming to do it this week. I started last week with the opening of https://github.com/github/linguist/pull/4881 but have hit problems with local testing unrelated to Linguist.
I'm going to close this issue in favour of that PR.


specs: universal share prefix

We should update https://celestiaorg.github.io/celestia-specs/latest/specs/data_structures.html#share to describe the universal share encoding schema. See this ADR for more details.

Closed by https://github.com/celestiaorg/celestia-app/pull/856


SQLC generates different Go types for JSON depending on the db driver

Version
1.21.0
What happened?
Using lib/pq as my preferred db driver for my PostgreSQL db, SQLC generated json.RawMessage types within a struct for column X in a SELECT query. After switching to pgx, it generates []byte for the same columns. This breaks my APIs as it sends malformed data which needs to be post processed if I want to fix it that way.
Relevant log output
No response
Database schema
No response
SQL queries
SELECT
    club.name,
    COALESCE(club.alias, '') AS alias,
    club.category,
    club.short_description,
    club.email,
    club.is_official,
    COALESCE(JSONB_BUILD_OBJECT(
        'about_us', cd.about_us,
        'why_us', cd.why_us,
        'role_of_sophomore', cd.role_of_soph,
        'role_of_junior', cd.role_of_junior,
        'role_of_senior', cd.role_of_senior
    ), '{}')::JSONB AS description,
    (
        SELECT
            COALESCE(JSONB_AGG(JSONB_BUILD_OBJECT(
                'position', club_member.position,
                'roll', s.roll_number,
                'name', s.name,
                'phone', s.mobile,
                'email', s.email
            ) ORDER BY s.name), '[]')::JSONB
        FROM
            student AS s
            JOIN club_member ON s.roll_number = club_member.roll_number AND club.name = club_member.club_name
        WHERE
            s.roll_number = ANY(SELECT roll_number FROM club_member WHERE club_name = club.name AND position != 'Member')
    ) AS admins,
    cd.branch,
    (
        SELECT
            COALESCE(JSONB_AGG(JSONB_BUILD_OBJECT('name', f.name, 'phone', f.mobile) ORDER BY f.name), '[]')::JSONB
        FROM
            faculty AS f
        JOIN club_faculty AS cf ON f.emp_id = cf.emp_id
        WHERE
            cf.club_name = club.name
    ) AS faculties,
    (
        SELECT
            COALESCE(JSONB_AGG(JSONB_BUILD_OBJECT('platform', cs.platform_type, 'link', cs.link) ORDER BY cs.platform_type), '[]')::JSONB
        FROM
            club_social AS cs
        WHERE
            cs.club_name = club.name
    ) AS socials
FROM
    club
JOIN
    club_details AS cd
    ON club.name = cd.club_name
WHERE
    club.name = $1
    OR club.alias = $1;

Configuration
version: 2
sql:
  - engine: "postgresql"
    queries:
      - "./database/queries"
      - "./database/queries/club"
    schema:
      # These are listed separately so that the tables are created in a preferred order
      - "./database/schemas/student.sql"
      - "./database/schemas/faculty.sql"
      - "./database/schemas/guild.sql"
      - "./database/schemas/club.sql"
      - "./database/schemas/course.sql"
      - "./database/schemas/announcement.sql"
    gen:
      go:
        package: "database"
        out: ".sqlc-auto-gen"
        sql_package: "pgx/v5"
        emit_json_tags: true
        json_tags_case_style: "snake"
    database:
      uri: postgresql://{PGUSER}:{PGPASSWORD}@{PGHOST}:{PGPORT}/{PGDATABASE}

Playground URL
No response
What operating system are you using?
Linux
What database engines are you using?
PostgreSQL
What type of code are you generating?
Go

The problem occurs in the columns with the ::JSONB type cast.
Struct that SQLC generated with lib/pq:
type GetClubRow struct {
    Name             string   `json:"name"`
    Alias            string   `json:"alias"`
    Category         string   `json:"category"`
    ShortDescription string   `json:"short_description"`
    Email            string   `json:"email"`
    IsOfficial       bool     `json:"is_official"`
    Description      []byte   `json:"description"`
    Admins           []byte   `json:"admins"`
    Branch           []string `json:"branch"`
    Faculties        []byte   `json:"faculties"`
    Socials          []byte   `json:"socials"`
}```
Struct that SQLC generated with `pgx`:
```go
type GetClubRow struct {
    Name             string          `json:"name"`
    Alias            string          `json:"alias"`
    Category         string          `json:"category"`
    ShortDescription string          `json:"short_description"`
    Email            string          `json:"email"`
    IsOfficial       bool            `json:"is_official"`
    Description      json.RawMessage `json:"description"`
    Admins           json.RawMessage `json:"admins"`
    Branch           []string        `json:"branch"`
    Faculties        json.RawMessage `json:"faculties"`
    Socials          json.RawMessage `json:"socials"`
}```

Some differences between drivers are expected. I'm not sure if this particular difference is "expected" or not, but it would be a significant breaking change to alter the types for pgx users in this case.
I think you can use an override for the jsonb type though, which would be an easy solution. Here's a playground URL with a working example pulled from a portion of your query: https://play.sqlc.dev/p/b8c75a8f8a09ad2b973e11cb118efaefe4f20ff28d6471cd048a20cfa8e4d1b1
Here's the config extracted for convenience:
version: 2
sql:
  - engine: postgresql
    queries: query.sql
    schema: query.sql
    gen:
      go:
        package: "db"
        out: "."
        sql_package: "pgx/v5"
        overrides:
          - db_type: jsonb
            go_type: json.RawMessage
          - db_type: jsonb
            go_type: json.RawMessage
            nullable: true

Hey, @andrewmbenton, thanks a lot for the config! It almost works. Unfortunately, it's trying to import json.RawMessage from some json lib rather than encoding/json from where it actually is. Do you know of a work-around for this? Sorry, I'm not versed enough in SQLC configuration.

Unfortunately, it's trying to import json.RawMessage from some json lib rather than encoding/json from where it actually is.

 I'll edit my original reply with a corrected config and playground link.

Thanks a ton for the help! You've ended a lot of frustration on my end haha. Switching to pgx had broken my APIs that I had to rollback. I think I can finally move forward now :)


update 01-quickstart.mdx typo "autompletion"

Describe this PR
update 01-quickstart.mdx typo "autompletion"
Changes
change "autompletion" to "autocompletion"
What issue does this fix?
typo
Any other relevant information

Thanks @jeffreyho92 for your PR!


BUG: Same message ID was put into message queue twice (arbitrum)

Problem:

Successful SLC happened after I tested Silo
Validator set all jailed after William attempts to SLC a few minutes later.

From @byte-bandit

So, looks like the SLCs were not relayed becaues of valset mismatch. That's expected, we need a valset update to go through before we send the SLCs. So those messages simply timed out, and because the fix is not yet deployed, everyone is getting jailed.

Trying to find the valset update message that should be linked to 490649
I found the valset upate: 490648

It failed with execution reverted: Invalid Valset ID

That means that the valset was actually deployed (likely by an earlier message), but Pigeons again failed to attest to that. We've seen this in the past a few times already, which is why we added this attestation enforcement.

From @verabehr

ok, this is weird, this is this message (the update valset). But it is actually visible on Compass https://arbiscan.io/tx/0xe6fa94ac5931fe427fc602f55cfd43b71f0f08443f29e4c0c69d3fcee7a85b8a#eventlog

From @byte-bandit

I also cannot find another message that would have relayed this valset ID
Sadly we do not send our internal message ID to compass when updating the valset, so there's no hard link for me to follow.
Okay, we had the message reassgined. Here's what I think happened:

We had a valset update
A relayer picked it up and sent it (successfully), but did not report back to Paloma
Message got reassigned
New relayer attempts to send and receives the error forwards it to Paloma
Everyone attests the error

I do not see the original relayer ever calling back to Paloma.
But we also know that some pigeons are VERY slow to call in.

From @byte-bandit

There is something else that's weird though.
The Valset Update message is put onto the queue twice. Same message ID (that shouldn't even be possible), and the assignee is different for both. So maybe we even had two messages with the same ID that two relayers tried to forward at the same time.
Three seconds apart

Mega weird. This log is only emitted after a new ID has been generated. I do not see how this could be in the logs twice.

Unless, the first message was in a block that got rejected, and the whole thing happend a second time later for a block that was included.

Looks like here this was actually put into the queue several times. This is an update valset message
https://gist.github.com/verabehr/4434e16a19fb8b92d68faea66f303df7

Logs from another attempt - note that the queue was observed for the whole time and there is no additional error data from the queue.
https://gist.github.com/verabehr/62a1101ee56959f3d96a61eb703227d9
https://gist.github.com/verabehr/68e76b8b671ec9fff1bdd89f2663bc25

Note that this also happens on eth-main. Other chains weren't tested

We had 2 relayed messages on gnosis (valset and SLC)
This appears went through after over 200? attempts from a bot backend script. The attempts are captured as Execute Contract transactions from this wallet https://paloma.explorers.guru/account/paloma1v6eq9al4ftd3ph4fvc8wt3mw0dsxt3scvjzswm

@byte-bandit Here are the results from another test on Gnosis.

Started offramp eure-bot ( offramp-eure-bot-script offramp-eure-bot-cw
vyper
I observed 4 messages in the queue
i. 509118: valset update   - https://gist.github.com/verabehr/12d4d01d8d3618dc435c0b1a703bf495
ii. 509119 SLC - https://gist.github.com/verabehr/08321605530d6f091f6e36e3bcbab454
iii. 509120 valset - https://gist.github.com/verabehr/acde62a23d73c5a3f12bb6563c432483
iv. 509121 SLC - https://gist.github.com/verabehr/00766afeebfcf2d418507f4948a60b6a
Some actually were relayed but the messages were still put into the queue several times.
Logs are from the LCD server and are in UTC
I see 2 contract execution tx on the wallet https://paloma.explorers.guru/account/paloma1v6eq9al4ftd3ph4fvc8wt3mw0dsxt3scvjzswm - screenshot times are in ET

Logs from message queue
Error/access data https://gist.github.com/verabehr/40d8152aae0e44dbfafe418ab92901d8
Full log: https://gist.github.com/verabehr/bf43f8740b5390d1475c3c3c739d345e

Closing as related to messenger issues post 0.47.1


Subtasks issue

When I run the migration from ClickUp to Jira, an error occurs during conversion if there are subtasks. The error message is: ‘AttributeError: ‘dict’ object has no attribute ‘name’’. This error is raised when the script tries to access the ‘name’ attribute of a ‘ticket’ object, which seems to be a dictionary in this context. It appears that the ‘ticket’ object does not have a ‘name’ attribute, causing the script to fail.
Traceback (most recent call last):
File "/home/jira/clickup-to-jira/clickup_to_jira/scripts/migrate.py", line 36, in
main()
File "/home/jira/clickup-to-jira/clickup_to_jira/scripts/migrate.py", line 29, in main
new_tickets = converter.convert(tickets)
File "/home/jira/.local/lib/python3.10/site-packages/clickup_to_jira/converter.py", line 34, in convert
converted_tickets.append(self.convert_ticket(ticket))
File "/home/jira/.local/lib/python3.10/site-packages/clickup_to_jira/converter.py", line 57, in convert_ticket
subtasks = self._get_converted_subtasks(ticket.linked_tasks)
File "/home/jira/.local/lib/python3.10/site-packages/clickup_to_jira/converter.py", line 105, in _get_converted_subtasks
return [self.convert_ticket(subtask) for subtask in subtasks]
File "/home/jira/.local/lib/python3.10/site-packages/clickup_to_jira/converter.py", line 105, in
return [self.convert_ticket(subtask) for subtask in subtasks]
File "/home/jira/.local/lib/python3.10/site-packages/clickup_to_jira/converter.py", line 49, in convert_ticket
logger.info(f"converting ticket {ticket.name}")
AttributeError: 'dict' object has no attribute 'name'

Seems like this repository is not maintained. Getting the same error.


Make netcat-openbsd the default netcat

Motivation for this change
See Issue #19411.
TL;DR: netcat-gnu does neither support IPv6 nor unix sockets and is therefore not a good idea for a default netcat.
Things done

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

[x] NixOS
[ ] OS X
[ ] Linux

[x] Fixed and tested compilation of all pkgs that depend on this change

[x] NixOS tests

[x] nixos/tests/hibernate.nix
[x] nixos/tests/virtualbox.nix

[x] playonlinux
[x] xfce4-sensors-plugin
[ ] build-support/vm/windows

[x] Fits CONTRIBUTING.md.

I am open for suggestions on, how to test the windows vm build-support with GNU netcat.

A problem I see here is, that there is no standard deprecation process in nixpkgs (#15357). This change would certainly benefit from a warnSemanticChange which I proposed in this PR: #19315.

\o/


Reuse only the TLS-exposed cluster for the SSH-based Docker tests.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

main

PR #271

PR #272   👈

This stack of pull requests is managed by Graphite.


Suggestion: use strings rather than files as parameters in NormalizeClass

File operations can be quite a bit slower than strings.

@goneall I was thinking wouldn't it be more convenient if we use files because eventually they are being converted to strings and otherwise we would have to pass a very long string (converting template or text to string)  as an argument in the command line which would not look good.

@anshuldutt21 unless I'm missing something (I'm not a Python guru), if the string is pass as a variable, it would look the same and be more efficient since it is being passed by reference.  If the string were copied on every call, I would agree that would be a problem.

@goneall Done the changes.


Some problems encountered when training tless

I downloaded the tless on bop and put it under data/bop/tless, the code can load the cad models , i use python -m surfemb.scripts.train tless --gpus 0 to run, but it runs to After trainer.fit, the following error will occur：
TypeError: default_collate: batch must contain tensors, numpy arrays, numbers, dicts or lists; found <class 'trimesh.caching.TrackedArray'>
Sorry, am I setting it wrong?

It maybe environment problems.I had encountered this problem.conda env create -f environment.yml maybe fix it.I forget how I solved this problem

It maybe environment problems.I had encountered this problem.conda env create -f environment.yml maybe fix it.I forget how I solved this problem

Thank, you! I have used conda env create -f environment.yml .  Maybe some auto-installed version is incorrect? Can you show me the difference between our environments? It is mine

https://github.com/rasmushaugaard/surfemb/blob/53e1852433a3b2b84fedc7a3a01674fe1b6189cc/surfemb/data/pose_auxs.py#L57
inst['surface_samples'] = np.array(mesh.sample(self.n_samples)).astype(np.float32)

https://github.com/rasmushaugaard/surfemb/blob/53e1852433a3b2b84fedc7a3a01674fe1b6189cc/surfemb/data/pose_auxs.py#L57
inst['surface_samples'] = np.array(mesh.sample(self.n_samples)).astype(np.float32)

Wow, This works! Thank you.

np.array(mesh.sample(self.n_samples)).astype(np.float32)
Thank you!

https://github.com/rasmushaugaard/surfemb/blob/53e1852433a3b2b84fedc7a3a01674fe1b6189cc/surfemb/data/pose_auxs.py#L57
inst['surface_samples'] = np.array(mesh.sample(self.n_samples)).astype(np.float32)

Thank you!


Make it possible to reset the native window

On Android when the application goes into the background, the EGL
window surface becomes invalid. It is possible to resume rendering when
coming to the foreground by resetting and reinitializing bgfx from
scratch, but this is costly in terms of performance.
This patch provides an alternative by letting the application recreate
the EGL window surface while keeping the EGL context intact. This allows
the application to resume rendering without needing to reload textures,
shaders, etc.
To test, build and run the Hello World example on Android, switch to
another app and then back again. The screen should not remain blank
after switching back.

Did you try is it possible to just recreate surface on every reset instead having special case for lost context?

Sure, that would work too. It may result in some glitches during resizes, but that could be fixed separately if it turns out to be an issue.
New pull request here: https://github.com/bkaradzic/bgfx/pull/796


model.compile() should check for unknown kwargs

System information

TensorFlow version (you are using): TF2
Are you willing to contribute it (Yes/No): No

Describe the feature and the current behavior/state.
This code works:
# import keras
import tensorflow.keras as keras

model = keras.applications.VGG16()
model.compile(loss='mse', optimizer=keras.optimizers.Adam(),
              why_does_this_not_fail=42.0)
print("Success")

I believe this code should throw an exception stating that why_does_this_not_fail is not a known argument.
Will this change the current api? How?
No.
Who will benefit with this feature?
Anyone who inadvertently adds the callbacks argument to the model.compile method (instead of model.fit) and asks themselves why their callbacks are not working.

Please find the github gist here

I believe this is fixed in 2.1.0rc0:
>>> model.compile(loss='mse', optimizer=keras.optimizers.Adam(),
...               why_does_this_not_fail=42.0)
Traceback (most recent call last):
  File "<stdin>", line 2, in <module>
  File "/home/bersbers/.pyenv/versions/3.7.5/lib/python3.7/site-packages/tensorflow_core/python/training/tracking/base.py", line 457, in _method_wrapper
    result = method(self, *args, **kwargs)
  File "/home/bersbers/.pyenv/versions/3.7.5/lib/python3.7/site-packages/tensorflow_core/python/keras/engine/training.py", line 312, in compile
    'Invalid keyword argument(s) in `compile`: %s' % (unknown_kwargs,))
TypeError: Invalid keyword argument(s) in `compile`: {'why_does_this_not_fail'}

@bersbersbers thanks for checking with 2.1.0-rc0.
Please let us know if anything else remains, else will close this issue in the next 2 days.


Event on enter workflow

Hello,
I use SimpleWorkflowBehavior::EVENT_AFTER_CHANGE_STATUS
To trigger an action when my workflow status changes.
But this event is not triggered when my model enters the workflow using
$model->enterWorkflow();

Is there an event that I can use,like
afterEnterWorkflow{MyWorkflow}
Thanks

Hi @ferllings ,
there is indeed an event fired when a model enter a workflow. The Event Model implemented by yii2-workflow is described here. You'll find events like :

beforeEnterWorkflow
beforeEnterStatus
afterEnterWorkflow
afterEnterStatus

Above events are fired when a model enters a workflow and so you could attach a handler to the ones that best fit your needs.
Cheers

Thanks,
It's working.
And it looks like I was wrong:
SimpleWorkflowBehavior::EVENT_AFTER_CHANGE_STATUS is also executed if you enter the workflow.


[FEATURE] Add OnNewGame events in SaveGame helper

Add following two events:
OnBeforeNewGame
OnAfterNewGame
Create a patch hook in the right place to trigger these events.

Implemented with PR #6


Fix styling for the colored bars on the article dashboard

We decided not to re-implement this feature.


Write tests for dataset utils

Codecov Report

Merging #184 into master will increase coverage by 2.56%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #184      +/-   ##
==========================================
+ Coverage   58.02%   60.58%   +2.56%
==========================================
  Files          19       19
  Lines        1289     1289
==========================================
+ Hits          748      781      +33
+ Misses        541      508      -33

Impacted Files
Coverage Δ

dpemu/dataset_utils.py
66% <ø> (+66%)
: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 21d4d6a...e31d95f. Read the comment docs.


Anything wrong with my syntax?

Hello there,
I can't seem to get merge.py to work. It basically always outputs Usage: message. What am I missing?
prompt # python merge.py ./ msgstore ./msgstore.db*
I tried replacing ./ with pwd or direct full path /home/mediacore/wforensic/tools, but the behavior is the same. I can't seem to figure out what it is that it's missing or is extra to prompt the script to issue usage help text.
[mediacore@host2 tools]$ ls -la
total 2004
drwxrwxr-x 2 mediacore mediacore    4096 Jul 23 21:49 .
drwxr-xr-x 6 mediacore mediacore    4096 Jul 23 21:38 ..
-rw-rw-r-- 1 mediacore mediacore    7527 Jul 23 21:38 encryption.py
-rw-rw-r-- 1 mediacore mediacore    7370 Jul 23 21:38 merge.py
-rw-r--r-- 1 mediacore mediacore 1064960 Jul 23 21:43 msgstore1.db
-rw-r--r-- 1 mediacore mediacore  962560 Jul 23 21:43 msgstore2.db
Thanks!

The mask for files_pattern must be quoted. Otherwise it returns multiple matching files instead of the pattern.


Refactor NotFoundException、BadRequestException

What's the purpose of this PR

Give a unified description to exceptions of the same class

BadRequestException's HTTP response code is 400, while NotFoundException's HTTP response code is 404. This is their difference. When I was making modifications, I didn't pay attention to the specific logic of where each exception was thrown. Therefore, I kept the same semantics as before, mainly to make the exception information look uniform. Previously, various methods might have been used to concatenate exception information.


server crash on booting custom modpack

https://logs.panel.gg/2d3209e829b8682375728d52906dde58
crash and latest log
debu
debug (1).log
g
says something about a missing item. botania was recently updated
1.18.2 btw

whoops had 2 versions in somehow my b


Fix images validation bundle issue

fix issue #422

nice one!

@mac-cain13 can we somehow merge this?

Thank you for this PR!
This fix will be available in the next release.


Boot2

do not merge, this is just tracking boot2 -> master as an indicator of whether we are missing commits from master on the boot2 branch

We're doing this a different way.


Add passwordfill support for secure entry fields

Using the secure option with text fields and 3rd party password managers like 1Password, an empty shadow could appear when focusing the field. @bartreardon confirmed this was due to the .password modifier with SecureField and textContentType. The intention is for the macOS password manager to auto-fill or create a password as necessary, but 3rd party managers (at least 1Password) and SwiftUI don't seem to get along.
This PR fixes empty shadow rendering by adding support for a new passwordfill option in text fields. It can be used by specifying passwordfill in dialog arguments. For example...
--textfield 'Password',secure,passwordfill,required

By default password fill is turned off. I don't think folks need/want this in most scenarios where users are prompted for password input. They would enter a password manually. However, I wanted to keep the option available as to not introduce feature regressions. Please feel free to provide feedback or make edits yourself to make this ready to merge. Thank you!

@nstrauss I've switched from the main branch to dev but there are conflicts present, can I get you to resolve those?

@bartreardon Thanks. I'll work from dev if I contribute in the future. Alphabetized things cause I had to. Otherwise I was able to build locally and test as working.

ta - btw, you might want to add the following to your .git/info/exclude
*.xcuserstate
*.resolved
xcuserdata
xcsharedata

re https://github.com/bartreardon/swiftDialog/pull/154/commits/7473a92966cd0bb1097e87e9717a6d38d411228e - no-one knows 😛

😅 Clearly I don't work in Xcode a lot. Happy to contribute in small ways where I can.


Duplicated efforts

Much of the AD permissions are already stated over here:
https://learn.microsoft.com/en-us/windows-server/failover-clustering/prestage-cluster-adds. Would make sense to consolidate the two pages.

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

ID: 4fa80e51-1cef-3dfd-807e-f0150c801b70
Version Independent ID: ee403411-e81d-67af-d04d-00064ea045f0
Content: Configuring cluster accounts in Active Directory
Content Source: WindowsServerDocs/failover-clustering/configure-ad-accounts.md
Product: windows-server
Technology: failover-clustering
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


Protocol guidelines

An Actor can only modify state through a component
Exception: Actor InstanceOwner may directly modify instance state

A Service can only be called by a component

A Service will check if calling Component is registered in Registry

A Service will orchestrate Module calls

A Module can only be called by a Service with the same GIF major version

A Module cannot be called by another Module

A Module will check if the calling Service is registered in the Registry and also validate the version

An Instance is composed of a set of Modules that use the same GIF major version

Flows
Service registration
sequenceDiagram
    Service->>Registry: register(objectType: Service)
    Registry->>Service: serviceNftId

Instance registration
sequenceDiagram
    Instance->>Registry: register(objectType: Instance)
    Registry->>Instance: instanceNftId

Component registration
sequenceDiagram
    Component->>Instance: getInstanceNftId()
    Instance->>Component: instanceNftId
    Component->>ComponentOwnerService: registerComponent(componentType, instanceNftId, otherComponentNftId, token)
    ComponentOwnerService->>Registry: validateInstanceandOtherComponents()
    ComponentOwnerService->>Registry: registerComponent(instanceNftId)
    ComponentOwnerService->>Instance: registerComponent()
    ComponentOwnerService-->>Component: componentNftId

put into documentation


Improve Pipeline logs error states

JIRA:
RHOAIENG-254
RHOAIENG-255
500 Internal Error state will be handled by RHOAIENG-1067
Description

Adds error messages for failed and cleaned-up pods for Pipeline logs
Hides the Logs toolbar toolbar for such states

Failed/cleaned-up pods(Error message according to this Slack message):

No internet(Error message according to this Slack message):

How Has This Been Tested?

Click on any running pipeline with a failed node and check the error state in the Logs Tab
Click on any node of an old pipeline(pods are cleaned-up) and check the error state
Check the error state for no-internet state too.

Test Impact

Request review criteria:

Self checklist (all need to be checked):

[X] The developer has manually tested the changes and verified that the changes work
[X] Commits have been squashed into descriptive, self-contained units of work (e.g. 'WIP' and 'Implements feedback' style messages have been removed)
[X] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has added tests or explained why testing cannot be added (unit tests & storybook for related changes)

If you have UI changes:

[X] Included any necessary screenshots or gifs if it was a UI change.
[X] Included tags to the UX team if it was a UI/UX change (find relevant UX in the SMEs section).

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

WIP to add tests and screenshots for cleaned-up pods.

@yih-wang can you check the error states?

@manaswinidas No, I don't think we will include any error state in the toolbar. The only two ways we display the error messages are:

From the inline alert as what you show in the screenshot - that will show the pod error or some other general issue, e.g. network issue
From the step dropdown - that will specify which step is encountering the error, offering users detailed insights into the exact point of failure for more effective troubleshooting

Oops sorry that I misread the question... No we don't show toolbar for both cases

@yih-wang it's highly unlikely that some step logs may be fetched while some cannot, in case of network issues.

@manaswinidas Then I think we should still show the toolbar in the 'No internet' error case to provide the ability to switch to other steps.

@yih-wang But we can't retrieve logs when there is no internet connection, even if we are able to switch the steps using the dropdown or the download dropdown, it's doing nothing because there is no internet. Here's a screen recording to demonstrate the same. Do we still show the toolbar in this case?
https://github.com/opendatahub-io/odh-dashboard/assets/22885912/41abaa1a-e7c2-4049-979a-543f89d53b8c

@yih-wang Can you have a final look at the screenshots?

@manaswinidas Didn't we combine case 1 (failed/cleaned-up pods) and case 3 (failed pod) you show in the screenshots?

@yih-wang I did according to this discussion  we had last month

@yih-wang Thanks for pointing it out. I updated it just now.

code looks good to me.
/lgtm
just need @yih-wang approval and an advisor

We have @yih-wang approval here. She was just asking me to change the PR description as it was outdated.

Rebased, cleaned up a few nits after the last merge.

/lgtm all still works


Process out of memory extracting material design icons

If I download the zip file from the release at https://github.com/google/material-design-icons/releases/tag/2.0.0, and run that directly through the standard extraction from this readme, I get the following error in Node:
FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - process out of memory
Abort trap: 6

Any ideas on this one welcome!

hey @guybedford thanks for the post I xposted on the google repo for the time being

Thanks for the quick response, I don't think the bower version downloads the release zip file as far as I can tell? Otherwise I'd have thought this would be flagged already.

ok, cool so the size of the zip is clearly the issue here

@guybedford can you give this a try https://github.com/thejoshwolfe/yauzl

@samccone yeah that works perfectly actually. I did also try raising the memory limit here but it still ends up going over 1GB which seems odd for a 50MB file.


Makefile: remove make test

The make test is old and not valid anymore. We will add instructions
for running the tests locally in a future PR.

lgtm

LGTM


Add mobile-specific documentation for Messaging tasks

The Messaging product documentation focuses on how to use Mattermost using the Desktop App and using a browser.
In some cases, the user's experience using Mattermost on a mobile device would be different than using either the Desktop or Web. Document the steps a user needs to take to complete Messaging tasks using the Mobile App.
Both Android and iOS content is most welcome.

The problem space that this ticket is looking to solve is to include end user documentation for Messaging functionality that's specific to mobile Mattermost users.
In Messaging, some tasks are performed differently using a Mattermost Mobile App than when using the Desktop App or a web browser. It's these specific cases where we have different user experiences between platforms that we're looking for in this ticket.
Take our Sending and Receiving Messages as an example. Under Send a Message, we have this documentation:
"Write a message by typing into the text input box, then press ENTER to send it. Use SHIFT+ENTER to create a new line without sending a message. To send messages by pressing CTRL+ENTER, go to Main Menu > Account Settings > Advanced > Send Messages on CTRL/CMD+ENTER > Edit."
A mobile user would not send a message by pressing ENTER, nor would they use SHIFT+ENTER to create a new line without sending a message. Right now, the product docs for Messaging focus mainly on how the Desktop App and web version of Mattermost work, and gloss over or ignore how the mobile experience works.
So, an ideal PR contribution for this particular ticket requires:

Reviewing the functional tasks described in the Messaging section of the product docs to identify cases where the user's experience differs on mobile
Running through each topic on using the Mobile App and identifying the places where additional mobile-specific details are missing/need updates
Documenting the specific steps a mobile user would take to complete the same task in parallel with the existing docs content.
Screenshots are a great way to provide an alternate visual learning path. They're not a requirement in this case, but they can provide great value.

@justinegeffen Please assign this ticket to me. I would be happy to work on this.

@Rutam21 - You're currently assigned to this ticket. Is this something you're working on, or would you rather this ticket return to the pool of available docs work?

@cwarnermm I will complete this one.

Awesome! Thanks so much @Rutam21!

@Rutam21 - Are you actively working on this? Would you like to keep it assigned to you?


RequestPossibleRoutesForCargo returns domain objects

The booking service method RequestPossibleRoutesForCargo returns []cargo.Itinerary which causes a DTO to be created in the makeRequestRoutesEndpoint function. It should be assembled in the application service the way it's currently done for Cargo.

Fixed.


Nepali Date Converter library

Add rubygems and .net class library for Nepali Date converter

@przbadu Please combine two commits into one single commit. Those commits are for the same purpose I guess and thus can be combined into one.

@samundra DONE!

@techgaun  @ankitpokhrel @hardfire Guys could you please have a look at it?

@samundra LGTM. Can't merge as I don't have write access in this repo 😄


where can i chage the stratagem menu key?

thanks for your work
i changed stratagem menu key to "Q"
where can  i change the default key in source code?

The macros of this program do not press the menu key for you. So, there shouldn't be any changes needed to the program itself. Users are expected to hold down the menu key manually, as described in the README, in relevant part:

When you trigger one of your configured hotkeys, a short delay (hotkey_start_delay) will occur -- enough time for you to make sure you have your stratagem menu up -- and the program will enter the WASD inputs for the associated stratagem.


Variables in snake cases are ignored during translation

Describe the bug
t() is ignoring any variables passed in snake case and accepting camel cases. I am calling t function like this
index.astro
t('home.title', {'user_name': 'Test user', userName: 'Demo user'});

astro.i18n.config.ts
export default defineAstroI18nConfig({
    defaultLangCode: 'en-gb',
    supportedLangCodes: ['fr-fr'],
    showDefaultLangCode: true,
    translations: {
        'en-gb':{
            "home.title": "Welcome to {user_name} {userName}!"
        }
    }
})

Current result
Welcome to Demo user!
Expected result
Welcome to Test user Demo user!
Am I missing something or is this a bug?

If it works with camel case it's probably a bug.
I'll see what I can do. Thanks for your report.

@Alexandre-Fernandez Any luck?

I'm currently developing 2.0 which will fix this bug and improve many other things

I'm currently developing 2.0 which will fix this bug and improve many other things

Amazing! Thanks for the update. I look forward to that release.

Can we sponsor this? We don't use it on a big page but I see the work and the potential ;)

Can we sponsor this? We don't use it on a big page but I see the work and the potential ;)

Thanks, I'll try to setup something then !


#1683 Move BatchPoints into Client

PR as requested.
#1683 Moves BatchPoints into Client.

So I'm pretty sure this is where it was originally, then it got pulled to the top level, and now we are putting it back?

I'm not sure but it feels wrong.
There currently is:
influbdb.BatchPoints
influxdb.Point
client.Write
client.Point
From a client API point of view it would be good if I didn't have to look into the server documentation to find the write format (So +1 keeping it in Client).
However by keeping them separate we end up with issues i.e.
Currently a representation of client.Point can have a timestamp equivalent of "2006-01-02T00:00:00.000000000Z00:00" with a precision of nanoseconds.
Where as the influxdb.Point only has a timestamp so  it would never know that "2006-01-02T00:00:00.000000000Z00:00" has a precision of nanoseconds.
Therefore if the precision is lost in the influxdb.Point should it be removed from the client.Point?
Should another package be created for shared types? Then each package embeds the shared type?
Let me know what you guys/gals decide I'll happily try and assist with any changes.

We actually debated this internally quite a bit, as we wanted to avoid the same data structure in both the client package and the server. Right now I don't believe what we have ideal, and we're still trying to reach consensus (not trying that hard). We also considered a package for shared types.
I agree you've identified a construction issue with this part of the code.

I don't think we're planning to merge this, so I'm going to close it out. We might revisit this later as we revise the CLI.


Format function argument separator comments

Summary
This is a complete rewrite of the handling of / and * comment handling in function signatures. The key problem is that slash and star don't have a note. I've left code comments for each possible case of function signature structure and comment placement
It depends on https://github.com/astral-sh/ruff/pull/5204 which depends on https://github.com/astral-sh/RustPython-Parser/pull/13, which is the reason why this is still a WIP PR and why tests are failing (default ranges are to short)
Test Plan
I extended the function statement fixtures with cases that i found. If you have more weird edge cases your input would be appreciated.

Current dependencies on/for this PR:

main

PR #5209

PR #5210

PR #5211   👈

This comment was auto-generated by Graphite.


Bug: ClickOnce installation doesn't finish

It's a bug.
Version: latest (tried today)

The installation doesn't finish. It remains like the screenshot indefinitely.
What can I try to unblock the installation?
Thanks in advance

Do you have an install log ?
Please fill in the template

maybe related: https://github.com/NuGetPackageExplorer/NuGetPackageExplorer/issues/157

ClickOnce is unstable. Recommendation is to use the Windows Store app (same as the desktop) if you have Windows 10 Anniversary Update or later: https://www.microsoft.com/store/apps/9wzdncrdmdm3


Add x-amz-security-token header before calling authorization()

The "x-amz-security-token" header value was being added after the "Authorization" header value was calculated. This was then giving the error
accessDenied(message: Optional("RequestId: 4059FAA108A45F89,
    HeadersNotSigned: x-amz-security-token,
    Message: There were headers present in the request which were not signed,
    HostId: p013uVEP7DHgqGkbCe/zsjuASVE0b9h9a7Gb37OfnuPmFnNeMZ5VnILzvi9orDLzHtfn1yNEzLA="))

This pull request moves the adding of the "x-amz-security-token" header value to before the authorization value calculation. This fixes https://github.com/swift-aws/aws-sdk-swift/issues/151

@adam-fowler I am very confused. I've been using ECS credentials for over a year with Kinesis which include a sessionToken, and don't get an error...

Docs are not clear but, seems like it should be signed https://docs.amazonaws.cn/en_us/general/latest/gr/sigv4-create-canonical-request.html
You can use temporary security credentials provided by the AWS Security Token Service (AWS STS) to sign a request. The process is the same as using long-term credentials, but when you include signing information in the Authorization header you must add an additional HTTP header for the security token. The header name is X-Amz-Security-Token, and the header's value is the session token (the string you received from AWS STS when you obtained temporary security credentials).

I wonder if the source of the token matters whether it is signed...

@jonnymacs Could you test your ECS credentials work with the change? Could it be related to the service being used. In the original bug it was indicated it worked for SNS, SQS but not S3. Maybe Kinesis doesn't need it signed. I've replicated the issue in S3 and also seen that SNS doesn't need the change.

@adam-fowler yep. i'm going to test today. stay tuned

Hey @jonnymacs maybe you could try to call S3.PutObject() in your ec2 instance with the current setup. See if that works

@adam-fowler i'm not sure I'll have time to regress putObject with the latest tag. I am comfortable moving fwd with this since it seems like it's right and should have been like this all along


[feature request] ability to use a callable expression as a output parameter

I'd like to use an error logger (like Monolog) to log errors. So it would be great if the output parameter accepted a callable argument (like closure).

Or at least a PSR-3 logger

I'd find this useful too. I've looked at the source code and I think the major issue is serialization. The callable would need to be serialized (not an issue as the library already has a dependency on the superclosure library), but so would any dependency of the callable. Taking Monolog for example, a StreamHandler cannot be serialized.
The only thing I can think of at the moment is to add an interface like Psr3LoggerFactory that the callable has to return. e.g
`output` => function () {
  return new class implements Psr3LoggerFactory {
    public function createLogger(): LoggerInterface {
       // return a PSR3 logger, e.g. Monolog
    }
  }
}

Then background job would do something like
if (is_callable($config["output"])) {
 $logger = $config["output"]();
} else {
  // log to file
}

Thoughts? cc @hellogerard


Add logging mode for sensor thermal calibration

Extends the SDLOG_MODE parameter to enable logging of data required to perform off-board calculation of calibration coefficients for IMU and Baro sensors.
Log file generated using SDLOG_NODE = 3 here: http://review.px4.io/plot_app?log=168d4068-478a-42c7-aa0c-2b60f22648c9
The longer term plan is to perform the curve fitting on-board, but in the short term, this PR will enable an off-board python script to be used and provides the data required for curve-fit algorithm development and testing

@bkueng When testing this on a pixracer, I noticed that the data for the first IMU was being logged on both sensor_gyro/accel_0 and sensor_gyro/accel_1 and the second IMU was being logged on sensor_gyro/accel_2.

I've rebased and applied this to master.

I'll rewrite the analysis script and submit a PR.

@jgoppert Thanks for that.
Be aware that I intend to change the definition of offset and scale factor in the thermal compensation code and calibration script so that 'corrected_value = (raw_value - offset) * scale' instead of the exisiting implementation of 'corrected_value = raw_value * scale + offset' to make it more compatible with the exisiting commander calibration methods. The impact of this on the script is negligible and I will look after that when the time comes.

Awesome. I'm rewriting it to just use ulog for now. We can always move it into px4tools later.

@jgoppert Are you able to have a look at the reworked calibrator? When I run it on the following log file, the accel and baro offsets are not calculated correctly (they assume zero is the correct measurement) and values are missing for some of the coefficients in the .param file.
http://logs.px4.io/plot_app?log=31fbdca3-843b-47f6-b332-2494aa884989


Issue 3103

ext4 single partition images were failing

Applies to 5.8 as well. Seems https://github.com/OpenNebula/one/pull/3110/commits/6883f6adfae9c70e534eb9d3d66a0a3857693602 wasn't merged in the previous PR. There are other minor improvements as well.


Change shebang to bash

script isn't sh compatible

The goals was initially to remove the bashisms. But that works.
Closes #29.


Deploy heroku

What this PR made?
Deploy on Heroku.
document that I relied on:
https://www.treinaweb.com.br/blog/deploy-de-uma-aplicacao-django-no-heroku/
i deployed my branch on heroku "deploy-heroku". But before this PR approved the correct branch that should be deployed is development. So this command must be executed:
git push heroku development:master
After finish the development:
git push heroku master
basically heroku create a new branch to deploy our app.
#45

@cesarbruschetta
Could you check with the travis failed my pr?
i run python manage.py test -v2 and got success.

@rafaelmpessoa

Could you check with the travis failed my pr?

I entered Travis and saw the error in sqlite3, I think I'd better delete the sqlite.db from the repository
Traceback (most recent call last):
  File "/home/travis/virtualenv/python3.7.2/lib/python3.7/site-packages/django/db/backends/utils.py", line 82, in _execute
    return self.cursor.execute(sql)
  File "/home/travis/virtualenv/python3.7.2/lib/python3.7/site-packages/django/db/backends/sqlite3/base.py", line 381, in execute
    return Database.Cursor.execute(self, query)
sqlite3.OperationalError: near "SCHEMA": syntax error
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
  File "manage.py", line 21, in <module>
    main()
  File "manage.py", line 17, in main
    execute_from_command_line(sys.argv)
  File "/home/travis/virtualenv/python3.7.2/lib/python3.7/site-packages/django/core/management/__init__.py", line 381, in execute_from_command_line
    utility.execute()
  File "/home/travis/virtualenv/python3.7.2/lib/python3.7/site-packages/django/core/management/__init__.py", line 375, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/home/travis/virtualenv/python3.7.2/lib/python3.7/site-packages/django/core/management/commands/test.py", line 23, in run_from_argv
    super().run_from_argv(argv)
  File "/home/travis/virtualenv/python3.7.2/lib/python3.7/site-packages/django/core/management/base.py", line 323, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/home/travis/virtualenv/python3.7.2/lib/python3.7/site-packages/django/core/management/base.py", line 364, in execute
    output = self.handle(*args, **options)
  File "/home/travis/virtualenv/python3.7.2/lib/python3.7/site-packages/django/core/management/commands/test.py", line 53, in handle
    failures = test_runner.run_tests(test_labels)
  File "/home/travis/virtualenv/python3.7.2/lib/python3.7/site-packages/django/test/runner.py", line 639, in run_tests
    self.teardown_databases(old_config)
  File "/home/travis/virtualenv/python3.7.2/lib/python3.7/site-packages/django_heroku/core.py", line 41, in teardown_databases
    self._wipe_tables(connection)
  File "/home/travis/virtualenv/python3.7.2/lib/python3.7/site-packages/django_heroku/core.py", line 33, in _wipe_tables
    """
  File "/home/travis/virtualenv/python3.7.2/lib/python3.7/site-packages/django/db/backends/utils.py", line 67, in execute
    return self._execute_with_wrappers(sql, params, many=False, executor=self._execute)
  File "/home/travis/virtualenv/python3.7.2/lib/python3.7/site-packages/django/db/backends/utils.py", line 76, in _execute_with_wrappers
    return executor(sql, params, many, context)
  File "/home/travis/virtualenv/python3.7.2/lib/python3.7/site-packages/django/db/backends/utils.py", line 84, in _execute
    return self.cursor.execute(sql, params)
  File "/home/travis/virtualenv/python3.7.2/lib/python3.7/site-packages/django/db/utils.py", line 89, in __exit__
    raise dj_exc_value.with_traceback(traceback) from exc_value
  File "/home/travis/virtualenv/python3.7.2/lib/python3.7/site-packages/django/db/backends/utils.py", line 82, in _execute
  File "/home/travis/virtualenv/python3.7.2/lib/python3.7/site-packages/django/db/backends/sqlite3/base.py", line 381, in execute
    return Database.Cursor.execute(self, query)
django.db.utils.OperationalError: near "SCHEMA": syntax error

https://travis-ci.org/cesarbruschetta/api-fintech/builds/533832278

@rafaelmpessoa
the problem was tired by the configuration of heroku, I changed the configuration below and the complete description of the problem is in the link that I added in the issue below
# Configure Django App for Heroku.
django_heroku.settings(locals(), test_runner=False)

Ref
https://stackoverflow.com/a/51209478/10342534


Sporadic failure in DehydrateTests.DehydrateShouldBackupFiles

I've seen this issue once so far in this run: https://gvfs.visualstudio.com/ci/_build/results?buildId=6815
2019-02-12T16:34:05.9122172Z => GVFS.FunctionalTests.Tests.EnlistmentPerFixture.DehydrateTests.DehydrateShouldBackupFiles()
2019-02-12T16:34:05.9123160Z Test GVFS.FunctionalTests.Tests.EnlistmentPerFixture.DehydrateTests.DehydrateShouldBackupFiles()
2019-02-12T16:34:05.9124207Z Failed at 4:34:05 PM taking 00:00:16.3815202
2019-02-12T16:34:05.9124527Z   Expected substring 'The repo was successfully dehydrated and remounted' not found in 'Running git status before dehydrating to make sure you don't have any pending changes.
2019-02-12T16:34:05.9125196Z If this takes too long, you can abort and run dehydrate with --no-status to skip this safety check.
2019-02-12T16:34:05.9125812Z
2019-02-12T16:34:05.9126355Z
2019-02-12T16:34:05.9127004Z Running git status...—
2019-02-12T16:34:05.9127450Z Running git status...\
2019-02-12T16:34:05.9127919Z Running git status...\
2019-02-12T16:34:05.9128338Z Running git status...|
2019-02-12T16:34:05.9128768Z Running git status...|
2019-02-12T16:34:05.9129190Z Running git status.../
2019-02-12T16:34:05.9129791Z Running git status.../
2019-02-12T16:34:05.9130974Z Running git status...—
2019-02-12T16:34:05.9131495Z Running git status...—
2019-02-12T16:34:05.9132011Z Running git status...\
2019-02-12T16:34:05.9132520Z Running git status...\
2019-02-12T16:34:05.9132943Z Running git status...|
2019-02-12T16:34:05.9133505Z Running git status...Succeeded
2019-02-12T16:34:05.9134179Z
2019-02-12T16:34:05.9134812Z Starting dehydration. All of your existing files will be backed up in C:\Repos\GVFSFunctionalTests\enlistment\9a2e2a23dcf448bc9770\dehydrate_backup\20190212_163353
2019-02-12T16:34:05.9135315Z WARNING: If you abort the dehydrate after this point, the repo may become corrupt
2019-02-12T16:34:05.9135658Z
2019-02-12T16:34:05.9135982Z
2019-02-12T16:34:05.9136618Z Unmounting...—
2019-02-12T16:34:05.9137225Z Unmounting...\
2019-02-12T16:34:05.9137744Z Unmounting...\
2019-02-12T16:34:05.9138254Z Unmounting...|
2019-02-12T16:34:05.9138783Z Unmounting...|
2019-02-12T16:34:05.9138927Z Unmounting.../
2019-02-12T16:34:05.9139166Z Unmounting.../
2019-02-12T16:34:05.9139356Z Unmounting...—
2019-02-12T16:34:05.9140230Z Unmounting...—
2019-02-12T16:34:05.9140742Z Unmounting...\
2019-02-12T16:34:05.9141141Z Unmounting...\
2019-02-12T16:34:05.9141351Z Unmounting...|
2019-02-12T16:34:05.9141443Z Unmounting...Succeeded
2019-02-12T16:34:05.9141514Z
2019-02-12T16:34:05.9141720Z Authenticating...—
2019-02-12T16:34:05.9141851Z Authenticating...\
2019-02-12T16:34:05.9141970Z Authenticating...\
2019-02-12T16:34:05.9142072Z Authenticating...|
2019-02-12T16:34:05.9142155Z Authenticating...Succeeded
2019-02-12T16:34:05.9142274Z
2019-02-12T16:34:05.9142364Z Backing up your files...—
2019-02-12T16:34:05.9142542Z Backing up your files...Failed. Run 'gvfs log C:\Repos\GVFSFunctionalTests\enlistment\9a2e2a23dcf448bc9770' for more info.
2019-02-12T16:34:05.9142642Z
2019-02-12T16:34:05.9142759Z ERROR: Failed to move the src folder: Access to the path 'C:\Repos\GVFSFunctionalTests\enlistment\9a2e2a23dcf448bc9770\src' is denied.
2019-02-12T16:34:05.9142903Z Make sure you have no open handles or running processes in the src folder
2019-02-12T16:34:05.9142962Z
2019-02-12T16:34:05.9143086Z ERROR: Backup failed. We will attempt to mount, but you may need to reclone if that fails

+1 hitcount here: https://gvfs.visualstudio.com/ci/_build/results?buildId=7095&view=logs

PR #1002 added more logging so closing this for now until we hit this again.


DcUpdates to master

What's up with this?

marked as duplicate... my code was merged in via Ben's PR can be ignored


Added Footer Component

I have added footer component.

this branch has conflict plz fix it

Hey @HridoyHazard i have resolved the issues
closes #23

hey @HridoyHazard can you look into this. I have resolved the issue

Hey @HridoyHazard it's been days since I raised a pr
Please look into this


Request headers are gone in onResponse()

In my onRequest(), stream_context.headers.request.get_headers().length is 7.
In my onResponse(), it is 0.
If I save the request headers in onRequest(), e.g. this.originalRequestHeaders = stream_context.headers.request.get_headers(), they stick around and retain their values.

i'm not sure what would cause this as this function directly translates to
get_header_map_pairs(RequestHeaders);

so it must be that the host sdk code does not make the request headers available on that time; i.e. i don't think we can do anything in this sdk to change this behavior

see here:
https://github.com/envoyproxy/envoy/blob/78dfea2cc80da8a89d868dc87a4b14f48d141500/source/extensions/common/wasm/context.cc#L1583
the request headers become inaccessible if you continue the request on request headers


How can I retriever the credited transactions?

When I use wallet.transactions.all(), I'm only receiving the debited transactions, but to show to the user the whole transactions history, I would need to show also those transactions in which it is the credited_user.
Why does wallet.transactions.all() do not get the credited transactions? And which it will be the easy way to acquire them?
I can just go transaction ID by transaction ID and get them all, but seems that this is not the correct way because of the inefficiency of doing many requests in front of just one to get them all.
Thanks

I thought that maybe I won't get them because of the pagination, so I decided to apply what people already told me on #209. So I made the following:
transactions = wallet.transactions.all(page=1, ItemsPerPage=100)
Even with that I still only receiving 10 out of 16 transactions. the wallet is correct because I verified the 10 triggered transactions with the first 10 transactions of that wallet.
What could I be doing wrong here? On the documentation says those two parameters are the ones needed to trigger more than 10 items per page, right? I tried with many pages and items between 10-100 but still only getting the same 10 ones.

Does anyone know anything about this problem? I still have to solve it.
It still happen the same when I want to retrieve the wallets of a user. I only got 10. (reference post #209)
thank you


Assigning key prop for Snackbar is not accepted

https://github.com/mui-org/material-ui/blob/54f0fc17d6fe90332a1412b96aad648188069a10/packages/material-ui/src/Snackbar/Snackbar.js#L294-300
Assigning prop key for Snackbar causes errors in console:
Warning: [object Object]: `key` is not a prop. Trying to access it will result in `undefined` being returned. If you need to access the same value within the child component, you should pass it as a different prop. (https://reactjs.org/link/special-props)

Example on codesandbox

v 5.0.0-alpha.27

Duplicate of #25304


HIP19: more on key issuance and revocation

Diff: https://github.com/helium/HIP/pull/98/files
Rendered view: https://github.com/helium/HIP/blob/7129a3858dbe06ecb21a24b3436ab02c75416b34/0019-third-party-manufacturers.md

Explicitly defers the details of a systen for actually issuing keys to manufacturer to a separate solution, through the HIP process or otherwise.
Proposes the same HIP-style process for revoking manufacturer approval. Currently no way to remove hotspots from the network, and defers consideration
of that to a future proposal.

I believe that the two companies that have applied to be manufacturers should not be held in limbo just because we haven't finalized how they are going to get keys. I also don't think tracking keys on-chain is absolutely a firm requirement, since it could be quite involved.

agreed that on-chain issuance can be added later to "codify" what is currently an imperfect manual solution. As long as we are making improvements with each iteration.