PL-156 Knowledge graph components refactor & PL-155 Pdf reader components refactor

PL-156 Knowledge graph components refactor
Done:

refactored KnowledgeGraphComponent (replaced then with async-await)
refactored SidePageComponent so that this component handles displaying/hiding KnowledgeGraphComponent and SummaryKeyComponent
added hide, show and isOpened methods to KnowledgeGraphComponent and SummaryKeyComponent which SidePageComponent calls when handling displaying/hiding of those components (maybe better option is to have a reference to html containers of those components in SidePageComponent and remove those methods from those components - need opinion on this)
removed unused imports and formatted PaperInfoComponent
moved SidePageComponent from PdfReaderComponent to ReaderView
created new event onReadNewPdf which is published when reader reads new document and this event can then be used in further refactoring (instead of calling reset method of components in PdfReaderComponent)
made hideLoader method in SidePageLoaderComponent public so SidePageComponent can call it without publishing an event (maybe it's better to use event and keep this method private since that event is already being used in KnowledgeGraphComponent  - need opinion on this)
-html of SummaryKeyComponent is now contained in html of SidePageComponent
new cursor while knowledge graph is loading
fixed cursor in SummaryKeyComponent

TODO:

close knowledge graph/summary when reference page is opened and vice versa
write tests
refactor further?

if i open the kg, then close it i can't open it again until i open and close the summary
and idk what is this dot

if i click on a reference and two page layout is displayed, if i click the kg button the reference close, but no kd displayed

if i open the kg, then close it i can't open it again until i open and close the summary
and idk what is this dot
if i click on a reference and two page layout is displayed, if i click the kg button the reference close, but no kd displayed

You're right, first thing was a bug which was happening because I did not implement closing side page correctly.
That blue dot was there because zindex of depth selector was too high.
In my last two commits both should be fixed, so you can check it out again.

I would like to see what has exactly been done in the PR message. What are those pdf.worker files in dist?

Okok I should've done that, I'll update PR's message.

If the knowledge graph cannot be generated the page is compressed on the top and the error isn't shown. The rest seems to be all fine

This does not happen on my machine, could be that some style is not applied on IOS but I don't know how can I test this out.

If the knowledge graph cannot be generated the page is compressed on the top and the error isn't shown. The rest seems to be all fine

This does not happen on my machine, could be that some style is not applied on IOS but I don't know how can I test this out.

I have a Mac too and it works even displaying the error page.


Some issues left out

Hi there!
This tool is pretty much what I was looking for - thanks for thinking ahead and building it already :)
However, it seems to me that some issues are not considered when building the time report. So far I have found that issues are missing when:

the issue is not assigned to anyone
the issue is assigned to a user with the "External" flag

As someone who needs to fetch all times reported, I would at least expect a warning that there may be more issues that are currently not displayed. Even better would be, if those issues would actually be included in the report.
Again, thanks for this piece of code which is already very helpful!
fogs

Hi and thanks for reporting!

the issue is not assigned to anyone

You're right, unassigned issues doesn't appear in table as it grouped by real users fetched from server which does not include any "no assignee" virtual member by default. However it would be useful to add it locally. By the way, "total" labels at the top should count not assigned issues if member filter is not applied. So this is the one more reason - reduce this implicitness and show not assigned issues in table too. I'll make it soon.

the issue is assigned to a user with the "External" flag

Can't really reproduce it for myself. I checked "external" flag on user, changed assignee to him, even made user a Guest in that private project, but issue is still shown up in table and assigned to him. Maybe there is some difference in token access rights or gitlab versions? I'm using GitLab CE 10.0.1.


Support special C10NMessage return type containing all translation mapping

Idea taken from commit by @Bartl Ákos to the akysoft fork.
The idea is to allow the user to retrieve translations for all locales, instead of just the value for the current locale. This can be done by declaring the method in the @C10NMessages interface with a special C10NMessage type (instead of a more typical String), as in the example below:
@C10NMessages
public interface Messages{
  @En("English")
  @Ja("Japanese")
  C10NMessage myMessage();
}

The appropriate information on available translations will be available in the C10NMessage class.

Link to the commit in the description akysoft/c10n@afe1c55927ecd2552dbb98bc8fb19a4f1a4cc4cf


Document flash size and RAM size units on CLI

My colleagues were having trouble setting the flash size (and computed the byte count by hand), so I changed the StructOpt documentation a little for Flash size and RAM size.  I hope you're okay with this little patch!

Flash memory size in bytes (e.g. 1M for 1 megabyte, 512K for 512 kilobyte, or hexadecimal 0xffK)

RAM size in bytes (e.g. 256K for 256 kilobyte, or hexadecimal 0xffK)

Thanks!


Investigate precision of invIncompleteBeta a b x for very small x

Function could return answers which are orders of magnitude off. It could be due to the fact that we revert to bisection which is linear instead of cubic Halley.
>>> (\a b -> incompleteBeta a b . invIncompleteBeta a b) 90 90 1e-200
9.866213992257607e-197

Things are much worse:
*Statistics.Distribution.StudentT> invIncompleteBeta 0.5e-2 0.5 0.84013
8.881784197001252e-16
*Statistics.Distribution.StudentT> invIncompleteBeta 0.5e-2 0.5 0.84012
0.0


Path names are not escaped

If the path to gpm's working directory contains white spaces, I see the following errors:
>> Setting github.com/mattn/go-sqlite3 to version
/usr/local/bin/gpm: line 66: cd: /Users/codewerft/Codewerft/Kunden/03: No such file or directory
/usr/local/bin/gpm: line 66: cd: /Users/codewerft/Codewerft/Kunden/03: No such file or directory

The working directory in the case above was /Users/codewerft/Codewerft/Kunden/03 - XYZ/Entwicklung/reactor.backend.

Huh, we were careful to escape paths early on to avoid this, but it seems somewhere along the way we refactored the cd step and forgot about it.
I'll fix it and make a release soon. :)


martin: 0.7.2 -> 0.8.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/maplibre/martin/releases.
meta.description for martin is: Blazing fast and lightweight PostGIS vector tiles server
meta.homepage for martin is: https://martin.maplibre.org/
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.8.0 with grep in /nix/store/j3yq0an9zv2gwd9lr7xg7gjp98qkkbi7-martin-0.8.0
found 0.8.0 in filename of file in /nix/store/j3yq0an9zv2gwd9lr7xg7gjp98qkkbi7-martin-0.8.0

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
martin

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/j3yq0an9zv2gwd9lr7xg7gjp98qkkbi7-martin-0.8.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/j3yq0an9zv2gwd9lr7xg7gjp98qkkbi7-martin-0.8.0
ls -la /nix/store/j3yq0an9zv2gwd9lr7xg7gjp98qkkbi7-martin-0.8.0/bin

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

1 package built:

martin

Maintainer pings
cc @sikmir for testing.

Duplicates #221697


HHH-11356 Add SharedSessionContractImplementor to DirectAccessRegion methods

https://hibernate.atlassian.net/browse/HHH-11356

@sebersole I noticed that HHH-11356 is resolved now. Was this PR integrated as well and should we close it now?

Out of date


Capitalization snafu in README

mvn goal -DDockerfile.username=... -Ddockerfile.password=...

.. should be ..
mvn goal -Ddockerfile.username=... -Ddockerfile.password=...

I first wondered if dockerfile.username wasn't case sensitive, but it was.

thanks for noticing!

Trial and error over an hour or two got me to that conclusion :)


kops_cluster_updater thinks there's a revision when there are no changes

kops_cluster_updater thinks there's a revision when there are no changes. This creates a big problem when I make changes to my terraform config that do not affect kops, because the cluster still thinks it needs to update when it does not need to update.
I believe this is caused because the kops_cluster and the kops_instance_group revision increments every time, even when there are no changes.
This ends up causing all of the nodes to get destroyed and recreated every time I run terraform apply, and it can take over 30 minutes when this happens.
How can I avoid the kops cluster from upgrading when I haven't made changes to it?
How can I prevent all the nodes from being destroyed and recreated when I make changes that should cause the nodes to be destroyed, for example simply trying to scale the nodes groups?

I think it's related to issue #530 because I'm using secrets.

Hi, this sounds very strange, the revision should increase only when something change in the cluster or instance group.
I don't understand why your nodes are destroyed/recreated, this should not happen.
Can you share your updater config ?

Closing, I think it has been fixed, please reopen if needed.


exit the app on Pebble without reporting a button press?

Hi,
I like your Pebble-controller, thanks for providing it.
I'm developing my own home automation and would like to support pebble-controller to be able to control a few often used functions from my watch (something like "all lights off").
I've been playing around with it and I'm able to receive the HTTP POST in a test C# app.
The "problem" I have is that it seems the only way to exit the app running on Pebble is to long-press the back button on the watch. However, that also reports that this button (button=0 if I remember correctly) has been pressed. This makes it almost impossible to use that button because every time I exit the app the function assigned to that button would be executed.
Can you think of any way around that?
Maybe the HTTP POST could be sent when the button is being released, rather than when it's pressed? I assume when exiting the app the release of the "back" button would no longer be reported because the app is closed.

Pebble is (more or less) dead, and I don't plan on ever updating this. Closing.


why llmc do not support asymmetric quantization?

Hi, thanks for your wonderful work. I wonder why llmc's quantization doesn't support asymmetric quantization if i set save_vllm = True
The config is as below:
base:
    seed: &seed 42
model:
    type: Qwen2
    path: /mnt/public/lingo-engine/model_info/multidoc_masterthemev11_qwen2_7b_bf16_240904
    tokenizer_mode: slow
    torch_dtype: auto
calib:
    name: pileval
    download: False
    path: /mnt/public/lingo-engine/data/pileval_dataset/
    n_samples: 128
    bs: -1
    seq_len: 512
    preproc: general
    seed: *seed
quant:
    method: Awq
    weight:
        bit: 8
        **symmetric: False**
        granularity: per_channel
        group_size: -1
    act:
        bit: 8
        **symmetric: False**
        granularity: per_token
    special:
        trans: True
        trans_version: v2
        weight_clip: True
    quant_out: True
save:
    save_vllm: True
    save_path: /mnt/public/daixin/masterthemev11_qwen2_7b_awq_w8a8_unsym

Then I get error:
[rank0]: Traceback (most recent call last):
[rank0]:   File "/mnt/user/daixin/llmc/llmc/__main__.py", line 317, in <module>
[rank0]:     main(config)
[rank0]:   File "/mnt/user/daixin/llmc/llmc/__main__.py", line 194, in main
[rank0]:     assert w.symmetric, 'Only symmetric quant is supported.'

https://github.com/ModelTC/llmc/blob/5e3361ce7afa87e62a82afb0acd4487cced8ab63/llmc/__main__.py#L194

This is because the vllm backend only supports symmetric quantized inference.


Elasticache controller should set recoverable condition during wait for modify

When running the elasticache-controller's e2e test called test_rg_input_coverage, I'm consistently seeing the following in the controller logs:
2021-09-10T16:20:20.889Z	DEBUG	ackrt	requeue needed after error	{"kind": "ReplicationGroup", "namespace": "default", "name": "rg-cmd-update-924f71vazqvj6qxzs8", "generation": 3, "account": "750630568209", "role": "", "region": "us-west-2", "is_adopted": false, "kind": "ReplicationGroup", "namespace": "default", "name": "rg-cmd-update-924f71vazqvj6qxzs8", "generation": 3, "error": "Replication Group can not be modified, need to wait for member clusters and node group members.", "errorVerbose": "Replication Group can not be modified, need to wait for member clusters and node group members.\ngithub.com/aws-controllers-k8s/elasticache-controller/pkg/resource/replication_group.(*resourceManager).CustomModifyReplicationGroup\n\t/github.com/aws-controllers-k8s/elasticache-controller/pkg/resource/replication_group/custom_update_api.go:68\ngithub.com/aws-controllers-k8s/elasticache-controller/pkg/resource/replication_group.(*resourceManager).sdkUpdate\n\t/github.com/aws-controllers-k8s/elasticache-controller/pkg/resource/replication_group/sdk.go:1052\ngithub.com/aws-controllers-k8s/elasticache-controller/pkg/resource/replication_group.(*resourceManager).Update\n\t/github.com/aws-controllers-k8s/elasticache-controller/pkg/resource/replication_group/manager.go:143\ngithub.com/aws-controllers-k8s/runtime/pkg/runtime.(*resourceReconciler).updateResource\n\t/go/pkg/mod/github.com/aws-controllers-k8s/runtime@v0.13.0/pkg/runtime/reconciler.go:330\ngithub.com/aws-controllers-k8s/runtime/pkg/runtime.(*resourceReconciler).Sync\n\t/go/pkg/mod/github.com/aws-controllers-k8s/runtime@v0.13.0/pkg/runtime/reconciler.go:221\ngithub.com/aws-controllers-k8s/runtime/pkg/runtime.(*resourceReconciler).reconcile\n\t/go/pkg/mod/github.com/aws-controllers-k8s/runtime@v0.13.0/pkg/runtime/reconciler.go:181\ngithub.com/aws-controllers-k8s/runtime/pkg/runtime.(*resourceReconciler).Reconcile\n\t/go/pkg/mod/github.com/aws-controllers-k8s/runtime@v0.13.0/pkg/runtime/reconciler.go:165\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.6.0/pkg/internal/controller/controller.go:256\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.6.0/pkg/internal/controller/controller.go:232\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).worker\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.6.0/pkg/internal/controller/controller.go:211\nk8s.io/apimachinery/pkg/util/wait.BackoffUntil.func1\n\t/go/pkg/mod/k8s.io/apimachinery@v0.18.6/pkg/util/wait/wait.go:155\nk8s.io/apimachinery/pkg/util/wait.BackoffUntil\n\t/go/pkg/mod/k8s.io/apimachinery@v0.18.6/pkg/util/wait/wait.go:156\nk8s.io/apimachinery/pkg/util/wait.JitterUntil\n\t/go/pkg/mod/k8s.io/apimachinery@v0.18.6/pkg/util/wait/wait.go:133\nk8s.io/apimachinery/pkg/util/wait.Until\n\t/go/pkg/mod/k8s.io/apimachinery@v0.18.6/pkg/util/wait/wait.go:90\nruntime.goexit\n\t/usr/local/go/src/runtime/asm_amd64.s:1373", "after": "30s"}

Couple things to address here:

I'm not sure what the stacktrace is appearing in the log message for DEBUG. It should not, since it's just noise, IMHO.
We should return no requeue error but instead just set the ResourceSynced condition to ConditionFalse when the requeue is needed. This will automatically trigger a requeue.

/reopen
/lifecycle frozen

Considering how many runtime changes have been made since this issue was opened, this issue isn't relevant anymore. A new issue can be created if we find this problem persists with the latest version


Add and apply scalafmt

@pjfanning When you merge this please do it with rebase + merge so I can reference the single Apply scalafmt in the .git-blame-ignore-revs.

rebase and merge done


Building on Windows with MinGW

Comment:
I am trying to do a build job on Microsoft Windows.
I fetched qt=5.9.7 from conda-forge and tried to compile a program with m2w64-gcc=5.3.0. I get a lot of failures at link-time. I did check that the right libraries were passed to the linker and there's no invocation mistake.
Is it because this compiler and this version of Qt are incompatible? Is there a specific compiler needed to use with Qt on Windows?
Thanks.

conda-forge packages are built with MSVC for compatibility with Python (see https://wiki.python.org/moin/WindowsCompilers). In order to link against our packages you will need to do the same....
There is some interoperability between gcc and MSVC for C code but for C++ they are incompatible (different name mangling for a start - hence the errors you are seeing).

Thank you. This is very helpful. Not sure how one would document this better for folks running into this issue, but my question is answered.  I will close this.


[1.0] "Failed resolving the link.Not a valid youtube query."

As of the latest commit on the 1.0 branch, I cannot play specific YouTube videos through direct links.  For example, if I try !!q We Are The Champions (Remastered 2011) it works fine.  But through its link, with !!q https://www.youtube.com/watch?v=tXyw4oMpeB4, no music is played and the log shows this:
Failed resolving the link.Not a valid youtube query.
21:25:26 CommandHandler | Command Errored after 0.2086614s
        User: [redacted]
        Server: [redacted]
        Channel: [redacted]
        Message: !!q https://www.youtube.com/watch?v=tXyw4oMpeB4
        Error: Value cannot be null.
Parameter name: s
This does work however with other links, such as https://www.youtube.com/watch?v=EwTZ2xpQwpA.
Is this just a mistake on my part, a YouTube issue, or an actual bot issue?
Edit: Also happens with some searches, but a different error.
Failed resolving the link.Response status code does not indicate success: 403 (Forbidden).
        [...]
        Message: !!q peace of mind topic
        Error: Value cannot be null.
Parameter name: s
What the heck is going on here?  There used to be at least one version of the bot I could rely on, now 0.99 crashes (#739) and 1.0 silently glitches.

first issue is that song is probably blocked where your server is, or not being able to be streamed for whatever reason.
Second error is that you're missing, or have incorrectly setup google api key

My Google API key is absolutely fine, and the song isn't blocked (it worked before).  Is this really a Google bug?

Gonna profit this music topic is here, because I have a somehow related problem.
Till 5 days ago, I had a 0.9 docker nadeko and I didn't have any issues with music. And since I updated to a 1.0 nadeko (linux one), music has not worked even once, with the exception of !!radio, which is fine.
Just in case, I made a new google api key, but that didn't help.

Any help would be appreciated.
Thanks,

@Kwoth Can this issue be reopened?  We have not confirmed with certainty that it is not a bot issue, and it affects multiple users.

I was able to troubleshoot my problem! I don't know if the problem comes from google or from nadeko, but the culprit was ipv6.
Turning that off fixed the issue for me.

I think i fixed this ~3 days ago, it was messed up.


Update Hyperspace doc after latest changes

Updating Hyperspace document after latest changes in Hyperspace notebooks.
This change adds Hybrid scan, Incremental refresh and Index optimization details to the Hyperspace document.

@pirz : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off @julieMSFT

Thanks for your review @euangms. We'll move this commit to the private repo so we can check the zone pivots and other formatting items on staging and ensure that the extensive update meets all the relevant quality criteria. After the new PR is opened in the private repo, I'll link it to this one and then close this PR (later today). Thanks.

The commit was moved to the private repo. I'll ask @euangms to review the new PR. We'll close this PR. Thanks.


Bad parsing of deeply nested lists

Input generated by the following command gets 100 lines:
$ python -c 'print ("".join(map(lambda x: ("  " * x + "* foo\n"), range(1,1000))))'

But when it is passed into hoedown, it generates just 27 lines, so most of the input is somehow ignored.
$ python -c 'print ("".join(map(lambda x: ("  " * x + "* foo\n"), range(1,1000))))' | hoedown | wc -l
27

There is a default nesting limit defined to prevent malicious input. You can change to #define’d value on compile time yourself.


Add line breaks in metrics list for readability

Adds line breaks between metric rows to improve the readability of the
metric list.

Codecov Report

Merging #1353 into master will decrease coverage by 5.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1353      +/-   ##
==========================================
- Coverage   50.34%   45.26%   -5.08%
==========================================
  Files         151       78      -73
  Lines       11911     5969    -5942
==========================================
- Hits         5996     2702    -3294
+ Misses       5184     2953    -2231
+ Partials      731      314     -417

Flag
Coverage Δ

#integration-tests
45.26% <ø> (?)

#kind-e2e-tests
?

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

OK

/skip-all


starlark spec: document fail

Fixes #47

In Bazel, there's no sep argument. I'm not convinced we should mandate a formatting.
We could restrict fail to be fail(obj) and Starlark would simply return the object to the host application. The host application is free to do what it wants with the object.

In Bazel, there's no sep argument. I'm not convinced we should mandate a formatting.

I tend to agree (arguing against my words in https://github.com/bazelbuild/starlark/issues/47#issuecomment-496508485), but I was persuaded to keep it by your later comment "Using the same as print seems useful and consistent."
Let's remove sep then.

We could restrict fail to be fail(obj) and Starlark would simply return the object to the host application. The host application is free to do what it wants with the object.

I would rather not do that because it creates the  expectation that the failure value is accessible to the catcher, and that's a hard invariant to maintain, in both implementations.

PTAL

As Jon mentioned (https://github.com/bazelbuild/starlark/issues/47#issuecomment-496615764), print(x) is not the same as print(str(x)) in the Java implementation. It shows additional information. It may be useful for fail to do the same.

the expectation that the failure value is accessible to the catcher, and that's a hard invariant to maintain

What's the problem here?

As Jon mentioned (#47 (comment)), print(x) is not the same as print(str(x)) in the Java implementation. It shows additional information. It may be useful for fail to do the same.

Yes, the current draft doesn't prescribe the exact formatting. It says "string forms" but not str(x).

the expectation that the failure value is accessible to the catcher, and that's a hard invariant to maintain
What's the problem here?

I was concerned that it requires errors never be flattened into their string forms, but I suppose this is already a requirement of correct reporting of the stack (which we now do). (Flattening into strings happens a lot but only for "leaf" exceptions in built-in functions, which have not yet been seen by the interpreter.) Nonetheless I would prefer not to mandate this feature, given that no-one has actually asked for it.

PTAL

@brandjon ping


immutable Map in Promise.all

I got a bug when I use this:
const promises = aImmutableMap.map(
      (val, key) => API.get(`something/${val.id}`).then(callback));
 );

await Promise.all(promises);

It's failed, at first I think maybe I should use a native JS Array here, but I found: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Iteration_protocols
It say: 'Some built-in types are built-in iterables with a default iteration behavior, such as Array or Map, while other types (such as Object) are not.'
So I think a Immutable Map will be Ok(like a native JS Map), but in fact it's not, it seems like Promise.all look for a structure like: [promise1, promise2, promise3], but a Immutable Map iterable like this: [[key1: promise1], [key2: promise2], [key3, promise3]], it dosen't work.
I found this: https://github.com/facebook/immutable-js/issues/236
but this only focus on Immutable List in Promise.all, I want to know is either Immutable Map should supported Promise.all, or my understanding is wrong? thx a lot.

Unfortunately Promise.all is fairly limiting in this way.
You might notice that using ES6's Map has the same issue.
Since you want to Promise.all the values of the Map, I would suggest:
await Promise.all(promises.values())


Need to return User ID on user_search

Hi
Thank for this great module...
i think on user_search we should return user id as currently it return name. As you know we need ID for everything. or there is something i need to add
Thanks
Sandeep

Well, it returns a list of instagram.models.User objects not a name. If you need the id of one of them, you can get some of the User objects and get the id attribute, e.g.:
users = api.user_search('wcastello2015')
my_user_id = users[0].id


ts-angular2-jsonapi

Hi,
I'm looking for a library like this to run in angular2. Is this intendet by your project?
Best regards
Simon

Yes, but in few months. Becouse, we don't migrate for now on own actual projects.


Ambiguous column for "whereHas conditions" in hybrid relations with a "join"

Description
The issue occurs with the following query, when you join the two relational database models and attempt to use the whereHas into the third relation which is a MongoDB Model.
Example
class Tree extends Illuminate\Database\Eloquent\Model
{
    use HybridRelations;

    public function apples(): HasMany
    {
        return $this->hasMany(Apple::class, 'tree_id');
    }

    public function seeds(): HasMany
    {
        return $this->hasMany(Seed::class, 'tree_id');
    }
}

class Apple extends Illuminate\Database\Eloquent\Model
{
    use HybridRelations;

    public function tree(): BelongsTo
    {
        return $this->belongsTo(Tree::class, 'tree_id');
    }

    public function seeds(): HasMany
    {
        return $this->hasMany(Seed::class, 'apple_id');
    }
}

class Seed extends Jenssegers\Mongodb\Eloquent\Model
{
    public function tree(): BelongsTo
    {
        return $this->belongsTo(Tree::class, 'tree_id');
    }

    public function apple(): BelongsTo
    {
        return $this->belongsTo(Apple::class, 'apple_id');
    }
}

Tree::query()->first()->apples()->whereHas('seeds',
    function ($query)
    {
       $query->whereIn('type', ['grannysmith']);
    }
);

Exception
SQLSTATE[23000]: Integrity constraint violation: 1052 Column 'id' in where clause is ambiguous...

@anderson-comvex, Tree and Apple are mysql relations?

@anderson-comvex, Tree and Apple are mysql relations?

@Smolevich Tree and Apple are mysql relations as you can see on the inherited namespace extendsIlluminate\Database\Eloquent\Model. Only Seed is a Mongodb model.


cant create ring...

I am trying to create a ring between 12 parties.. Most Umbrel, mine Nodl.... We have tried to run the script (2 files, one config, one sh) and keep getting these errors... we tried from different nodes (changing the node order in the config file).
Ran out of ideas now..
bitcoin@nodl:~/igniter$ ./igniter.sh build
[lncli] rpc error: code = Unavailable desc = connection error: desc = "transport: authentication handshake failed: context deadline exceeded"
bitcoin@nodl:~/igniter$ ./igniter.sh send
[lncli] rpc error: code = Unavailable desc = connection error: desc = "transport: authentication handshake failed: context deadline exceeded"
[lncli] rpc error: code = Unavailable desc = connection error: desc = "transport: authentication handshake failed: context deadline exceeded"
Route fee is 0 sats.
[lncli] payment hash must be exactly 32 bytes, is instead 0
Any idea?
thanks

Well, there seems to be a timeout between two nodes...
This script merely returns what lnd outputs... unfortunately, the error message doesn't mention which nodes cause that problem, so there's little that can be done here else than make sure that every ring participant are correctly connected between each other.
On a side note, the larger the ring, higher the chances of meeting such problems. 12 might be the biggest ring I have heard about yet.


Revert "Seperate metadataManager and metadataStore (#3574)"

This reverts commit 6539b80f7bbf8def6d7dfef0000d39383a30d513.

Coverage decreased (-0.2%) to 68.716% when pulling 81cb10ea6bb492fb80452aaba554854f00c8c359 on anish531213:master into bb2ca3f103203a2182f98c44a370da496acdb361 on uber:master.


how about adding remote-endpoint in the client span for rpc (eg, openfeign) ?

I am using sleuth-zipkin to setup a tracing solution with openfeign client. the version I am using as parent is :
org.springframework.boot
spring-boot-starter-parent
2.4.4

My envionrment is very complext, that it is highli possible that some of the applicaions do not get instrumented with sleuth-zipkin. So in order to still draw an the topology as comprehensive as possible, is it possible to add the remote-endpoint in the "client" span for open-feign? such as below
 {   "traceId":"761fa7ad18d9172b",
    "parentId":"761fa7ad18d9172b",
    "id":"514fb6c1bcb6357f",
    "kind":"CLIENT",
    "name":"get",
    "timestamp":1616993053736129,
    "duration":1948,
    "localEndpoint":{
        "serviceName":"application-1"
    },
    **"remoteEndpoint":{
        "serviceName":"application-2"  # ip is also okay. Currently, the remote endpoint is not provided for client span.
    },**

    "tags":{
        "http.method":"GET",
        "http.path":"/echo"
    }}

By this way, even if the server span is not provided, with the client span, one can still get the topology calculated.

TL;DR: You can use a HttpTracingCustomizer to add remoteServiceName or a HttpRequestParser to add arbitrary tags that you want to use, see details, below.
HttpTracing
I think localEndpoint is not provided by Sleuth but by Brave. Also, it seems remoteEndpoint is deprecated in favor of remoteServiceName but I don't think Brave can provide that. In some cases, Sleuth does support remoteServiceName (search for it in the codebase) but not in the case of http clients like Feign, I'm not sure why, maybe @marcingrzejszczak does.
Let me walk you through on what happens under the hood so that you will see how to change the behavior:

Sleuth wraps instrumented http clients; in case of Feign, there are multiple wrappers, one of them is TracingFeignClient
You can see that it is using a HttpClientHandler; in case of Brave, the implementation is BraveHttpClientHandler
If you look into this, you will see that it is just delegating the work to brave.http.HttpClientHandler
In Brave's handler you can see that it is setting the remoteServiceName that comes from a HttpTracing instance
So we "only" need to provide the name of the service to this class in Brave
If you search in the Sleuth autoconfig for this, you should find that when it is created, HttpTracingCustomizers are applied
If you look into HttpTracingCustomizer, you will see that it gets a HttpTracing.Builder that can set the server name but the setter is package private so you can't set it here (I don't know why) :(
Though HttpTracing has a setter for this (clientOf)
So you can create your own HttpTracing bean and set the remote server name
this is how Sleuth creates it

If I omit all of the logic, basically you should replace
return builder.build();
to
return builder.build().clientOf("myServer");
This is just a workaround and I don't think this will work for multiple Feign clients, I'm going to open a new issue for Brave to see why isn't this possible (package-private setter):
@Bean
HttpTracingCustomizer httpTracingCustomizer() {
    return builder -> builder.serverName("myServer");
}

It might work if your @Configuration class is in the brave.http package but if anyone asks, I wasn't the one who said this. 😃
I'm also going to talk to @marcingrzejszczak to see what we can do: injecting the server name should be easy, what I'm worried about is having multiple Feign clients.

fyi: https://github.com/openzipkin/brave/issues/1296

We are not planning on adding any new features for Sleuth


Sentry messages: stack trace isn't line-wrapped

Solution: add style white-space: pre;

We fixed it in v2.0


memoryUsageWeigher missing from exports

Hi, the docs mention memoryUsageWeigher, but I cannot find it in the entire repository except the readme. I looked through some old issues and commits, and looks like it was here at one point, but was left out when moving to TS, but can't be sure why and what happened. Or else the documentation needs to be updated...

It's called memoryEstimator in the latest version. Regardless, this functionality seems to be broken:
.withWeigher((_key, value) => {
    return value.length // setting "json" to cache -> weighedSize = 4
  })

.withWeigher((_key, value) => {
    return 32 // setting "json" to cache -> weighedSize = 0 (not 32)
  })


Updated smappy library version

Description:
I updated the version of the used smappy library to 0.2.16 as been released on pypi.org. The updated library shows the temperature as degrees Celsius instead of centi degrees Celsius
https://pypi.org/project/smappy/
https://github.com/EnergieID/smappy
I tested it on my local instance.
Related issue (if applicable): fixes #
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Example entry for configuration.yaml (if applicable):

Checklist:

[x] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Hi @visibilityspots,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Is this a breaking change? Are the temperature values reported differently with this PR?

The temperature values are reported as degrees Celsius with this PR instead of centi degrees Celsius. Don't know if this is a breaking change cause it seems more logical to me to have the temperate reported as degrees over centi degrees..
But if it's categorised as a breaking change, what can/should I do towards it's implementation in home-assistant?

We just need to provide a paragraph for the release notes. Please highlight that in the PR description.


Bump dependencies

Pull Request Test Coverage Report for Build 945837574

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 95.218%

Totals

Change from base Build 945821750:
0.0%

Covered Lines:
904

Relevant Lines:
934

💛  - Coveralls


Security does not pass from JobQueueCluster to SpecCluster

What happened:
Try to pass a distributed.Security object to a custom implementation of the JobQueueCluster.
Hit the following error:
/var/conductor/miniconda-4.8.2-x86_64/anaconda/envs/dask-cpu-env/lib/python3.7/site-packages/distributed/comm/tcp.py in _expect_tls_context(connection_args)
    311             "TLS expects a ssl_context argument of type "
    312             "ssl.SSLContext (perhaps check your TLS configuration?)"
--> 313             "  Instead got %s" % str(ctx)
    314         )
    315     return ctx

TypeError: TLS expects a ssl_context argument of type ssl.SSLContext (perhaps check your TLS configuration?)  Instead got None

What you expected to happen:
Expect that the cluster is able to start with the specified security settings.
Tried LocalCluster and was able to start the cluster with TLS.
Minimal Complete Verifiable Example:

sec = Security(tls_ca_file='cacert.pem',
               tls_client_cert='public.pem',
               tls_client_key='keyfile.key',
               tls_scheduler_cert='public.pem',
               tls_scheduler_key='keyfile.key',
               tls_worker_cert='public.pem',
               tls_worker_key='keyfile.key')

cluster = MyCluster(security=sec)

Anything else we need to know?:
Found that the JobQueueCluster does not appear to pass the security parameter from it's __init__ method to the SpecCluster __init__.
Locally edited JobQueueCluster to pass the security object and was able to get past this error:
super().__init__(
            scheduler=scheduler,
            worker=worker,
            loop=loop,
            silence_logs=silence_logs,
            asynchronous=asynchronous,
            name=name,
        )

Environment:

Dask version: distributed 2.15.0 and dask-jobqueue 0.7.1
Python version: 3.7
Operating System: Linux
Install method (conda, pip, source): conda

Sorry I missed this one this should (hopefully) be fixed in master (thanks to #398), would you be kind enough to give it a go and report back?

@lesteve Sorry it took so long to get back to this one. I tried with the master code today and security seems to be passing well. Thanks!

Great, thanks for letting me know.


Added Kwara State University, Malete

@ionware Thank you for your patience while waiting for a review and processing your request. Pull request merged. Thank you!


Spacing in docs

Small one, should the docs read
space: [10, 12, 14]

rather than
spacing: [10, 12, 14]

Thanks!


Remove overflow to fix render bug

Closes #31

So after I looked at this PR I remembered seeing @thisisvilla remove the entire div from the alpha.5 version of the tutorial. I'm not sure how it got back into this version.


Strange behaviour with device parameters listeners.

First, thanks again for that amazing script. And for adding listeners for device parameters. Extremely useful.
I'm having a very strange bug. No matter what I do, all listeners are working, on any track, on any device or parameter. Except for track 2 (index 1). I receive values and value strings for all the other tracks, except the second one. Example, /live/device/start_listen/parameter/value',(1,0,x), x meaning 0 to 7, I don't receive any data. If I go to other groups of 8 parameters, some work, some don't. But only on track 2 (index 1). Track 1 or any other track after track 2, everything works flawlessly.
I've tried everything. Since I'm blind, I may be missing something, of course.
I only listen to groups of 8 parameters and stop listen to the previous ones before asking for new device parameters listeners.
I speak french so I hope it's clear enough.
Thanks!

I'm very sorry, my fault! :)

I'm closing the issue, the problem was on my side, sorry!

Thanks for the update! I did try to reproduce this but couldn't, so I'm glad you got to the bottom of it.
@Daretreble Was the underlying cause anything that other users could learn from?


feat: introduce more docker tags for uv

Summary
Closes https://github.com/astral-sh/uv/issues/5610
This PR does two primary things:

Parallelizes multi-platform builds using multiple workers (hence the new docker-build / docker-publish jobs), this seems to save about ~8 minutes.
Introduces additional images with the uv/uvx binaries from scratch for both amd64/arm64 and make the mapping easy to configure by generating the Dockerfile on the fly. This approach focuses on minimizing CI time by taking advantage of dedicating a worker per mapping (20-30s~ per job).

For example, on release x.y.z, this will publish the following image tags with format ghcr.io/astral-sh/uv:{tag} with manifests for both amd64/arm64.

From scratch: latest, x.y.z
From alpine:3.20: alpine, alpine3.20, x.y.z-alpine, x.y.z-alpine3.20
From debian:bookworm-slim: debian-slim, bookworm-slim, x.y.z-debian-slim, x.y.z-bookworm-slim
From buildpack-deps:bookworm: debian, bookworm, x.y.z-debian, x.y.z-bookworm
From python:3.12-alpine: python3.12-alpine, x.y.z-python3.12-alpine
From python:3.11-alpine: python3.11-alpine, x.y.z-python3.11-alpine
From python:3.10-alpine: python3.10-alpine, x.y.z-python3.10-alpine
From python:3.9-alpine: python3.9-alpine, x.y.z-python3.9-alpine
From python:3.8-alpine: python3.8-alpine, x.y.z-python3.8-alpine
From python:3.12-bookworm: python3.12-bookworm, x.y.z-python3.12-bookworm
From python:3.11-bookworm: python3.11-bookworm, x.y.z-python3.11-bookworm
From python:3.10-bookworm: python3.10-bookworm, x.y.z-python3.10-bookworm
From python:3.9-bookworm: python3.9-bookworm, x.y.z-python3.9-bookworm
From python:3.8-bookworm: python3.8-bookworm, x.y.z-python3.8-bookworm
From python:3.12-slim-bookworm: python3.12-slim-bookworm, x.y.z-python3.12-slim-bookworm
From python:3.11-slim-bookworm: python3.11-slim-bookworm, x.y.z-python3.11-slim-bookworm
From python:3.10-slim-bookworm: python3.10-slim-bookworm, x.y.z-python3.10-slim-bookworm
From python:3.9-slim-bookworm: python3.9-slim-bookworm, x.y.z-python3.9-slim-bookworm
From python:3.8-slim-bookworm: python3.8-slim-bookworm, x.y.z-python3.8-slim-bookworm

Below is what https://github.com/astral-sh/uv/pkgs would end up looking like with the additional tags (just for 0.2.35).

Epic! Thank you! I'll review this one.

I would like to hear a bit more about what motivating adding quite so many tags, I was imagining a less :)

This looks good to me. I might play with the job names so they're more readable? but otherwise it seems great. I like the approach you took.

Happy to rename the jobs, lmk

I would like to hear a bit more about what motivating adding quite so many tags, I was imagining a less :)

I primarily mimicked supported patterns from docker hub official images for python and general use (non-bundled python ones) and the tags they support. I considered the feedback / use cases from @konstin and yours on the issue, I thought this was a good starting point.

Cool, thanks for explaining. Just waiting for consensus from more of the team here.

@samypr100 do you have these pushed anywhere by any chance? Would love to test them out, I could write some docs on how to implem these for GitLab

@samypr100 do you have these pushed anywhere by any chance? Would love to test them out, I could write some docs on how to implem these for GitLab

I've published the ones I've built in https://github.com/samypr100/uv/pkgs/container/uv, but do not use these for anything official as I might delete them eventually.

@samypr100 does this need a rebase after the other changes?
I might merge this for the release today. Nobody else on the team is expressing opinions...

@samypr100 does this need a rebase after the other changes?
I might merge this for the release today. Nobody else on the team is expressing opinions...

@zanieb It does not need a rebase, unless we also want to support tags without patch version (e.g. 0.3) on the additional extra images, That can be done as a follow up though.

@samypr100 does this need a rebase after the other changes?
I might merge this for the release today. Nobody else on the team is expressing opinions...

@zanieb It does not need a rebase, unless we also want to support tags without patch version (e.g. 0.3) on the additional extra images, That can be done as a follow up though.

@zanieb The latest version now also handles major.minor tags as well.

@samypr100 Will test, thanks!
Also, any chance you could introduce the same changes to ruff?

@samypr100 Will test, thanks!
Also, any chance you could introduce the same changes to ruff?

I can take a look at supporting after this is merged, although I don't think the python-based images make sense over there so it will likely be just distroless, alpine, and debian.

I've got success on GitLab, amazing! Thanks @samypr100
Here's my config:
.uv_compile:
  stage: analysis
  image:
    name: ghcr.io/samypr100/uv:0.4-python3.12-alpine
  script: >
    cd $CODE_LOCATION

    if [ -z "$FILES" ]; then
      FILES=$(find . -iname requirements*.in -maxdepth 1)
    fi

    for file in $FILES; do
      output_file=${file%.in}.txt
      uv pip compile $file --extra-index-url $EXTRA_INDEX_URL --no-strip-extras --output-file $output_file
    done

    git diff HEAD --exit-code

Side note: I have to call to git diff to complete my use-case for checking if the requirements are compiled,  https://github.com/astral-sh/uv/issues/3021 is relevant here in my opinion.

It worked 🥳, starting with 0.4.4 new extra docker tags have been published.

Haha oops I meant to do that release so I could watch it.. glad it worked! Thank you!


events.js throw error after new install on Windows

Package version
4.1.0
Node.js and npm version
Node.js =>8.11.4
npm => 6.4.1
Issue
Hello,
I just made a new install of my project on Windows and when I start :
adonis serve --dev
I get an error after a few second :
events.js:183
     throw er; // Unhandled 'error' event
      ^
Error: ENOENT: no such file or directory, open 'C:\Users\my\repository\tmp\adonis.log'

"'C:\Users\my\repository\tmp\adonis.log" doesn't exist.
I tried to remove node_modules and reinstall it doesn't work.
I tried to change port and it doesn't work.
Do you know what the problem is please ?

I believe you are using the file driver of the logger. Simply create the tmp directory inside your project root and it should be good

It doesn't work even with this folder :/


bug: ion-header ios 11 style only working on first tab

Bug Report
Ionic version:

[x] 4.11.0
Current behavior:

The "condense" header is only working on the first loaded tab.

Expected behavior:
The header should be hidden in all tabs.
Related code:
Example in StackBlitz
Ionic info:

Ionic:

   Ionic CLI                     : 5.4.1
   Ionic Framework               : @ionic/angular 4.11.0
   @angular-devkit/build-angular : 0.801.3
   @angular-devkit/schematics    : 8.1.3
   @angular/cli                  : 8.1.3
   @ionic/angular-toolkit        : 2.0.0

System:

   ios-deploy : 1.9.4
   ios-sim    : 6.1.3
   NodeJS     : v10.15.3
   npm        : 6.11.3
   OS         : macOS Mojave
   Xcode      : Xcode 11.0 Build version 11A420a

Glad to see someone trying the new iOS 11 style header 🙂. Looks like it was selecting the wrong ion-content when in use on tabs that were not tab 1. I'll work on a fix for this.

Can you try this dev build and see if it fixes the issue for you?
npm i<EMAIL_ADDRESS>

Can you try this dev build and see if it fixes the issue for you?
npm i<EMAIL_ADDRESS>
Now it fails from the beginning:

New Stackblitz
Did I miss anything?

Nope, I think I just gave you a bad build. Try this one:
npm i<EMAIL_ADDRESS>

Good! This work well in my production environment!!

Nope, I think I just gave you a bad build. Try this one:
npm i<EMAIL_ADDRESS>
Now it works but somehow the fading animation is "flashing" the first time you enter a tab (I guess because after that it is on the stack so it is not initialised again), it is just a flash but doesn't look very nice...

Link to Stackblitz

Yeah that one is a known issue. I'm currently working on that separately 👍. Glad it's working now in your tabbed application.

Thanks for the issue. This issue has been fixed and will be available in an upcoming release of Ionic Framework.


Implement get /m feature

Feature implemented: Filters out list of patients that are on the specific query medication.
Eg. get /m paracetamol shows a list of patients currently administered "paracetamol".
Close #8.

Codecov Report
Base: 63.35% // Head: 62.49% // Decreases project coverage by -0.86% :warning:

Coverage data is based on head (ba2f050) compared to base (3610cf3).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master      #76      +/-   ##
============================================
- Coverage     63.35%   62.49%   -0.87%
  Complexity      485      485
============================================
  Files           101      104       +3
  Lines          1804     1829      +25
  Branches        221      224       +3
============================================
  Hits           1143     1143
- Misses          587      612      +25
  Partials         74       74

Impacted Files
Coverage Δ

.../java/seedu/address/logic/commands/AddCommand.java
100.00% <ø> (ø)

...gic/commands/getcommands/GetMedicationCommand.java
0.00% <0.00%> (ø)

...a/seedu/address/logic/parser/GetCommandParser.java
0.00% <0.00%> (ø)

.../parser/getparsers/GetMedicationCommandParser.java
0.00% <0.00%> (ø)

...el/person/MedicationContainsKeywordsPredicate.java
0.00% <0.00%> (ø)

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

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


Filter per tile sequence quality

Heys!
I'm working with whole-genome sequencing of a non-model organism and I was checking the quality of the data with FastQC. As I found that my per tile sequence quality was bad, I started looking for other softwares to filter my data.
That's when I found fastp, and now I'm filtering all my data with your software. However, as I obtained a failure with FastQC  in the tile's section and fastp does not provide (as far as I know) any information about it, I reran FastQC after running fastp on my data and the tile's failure was still there.
How could I filter by tiles with Fastp?
Thanks,
Gabriel

@sfchen 相同的需求，一年半以后仍然存在！陈博士可否考虑规划一下这个功能？


Question on Exchanges and Stocks

How would you use the Exchange modules for stocks? Looks like things more focused on currency pairs at the moment.
Thanks!

same question,is it possible to have a refactor to support future exchange?

same question,is it possible to have a refactor to support future exchange?

Actually the exchange is merely the entity (company) that provides you exchange services, between your base asset (that can be a currency like USD, or it can also be any other type of asset), and the assets you want to trade.
Different exchanges can have different transaction costs and different assets available to trade, so that's the reasoning behind this structure.
Once you have an exchange set up, you can define two wallets, one with USD and another with any other asset you like (might be a stock, might be a commodity or anything else), and build a portfolio comprising both those elements, like:
USD = Instrument("USD", 2, "U.S. Dollar")
AAPL = Instrument("AAPL", 2, "Apple Inc.")
cash = Wallet(eToro_exchange, 10000 * USD)
asset = Wallet(eToro_exchange, 0 * AAPL)
portfolio = Portfolio(USD, [cash, asset])
Is that what you meant?

same question,is it possible to have a refactor to support future exchange?

@patrick-bercow You can work around it by treating the future as a normal asset. I know it's not the same thing, but I don't see why it shouldn't work if you treat the future as a generic object, without considering the date, as per the example above. In any other consideration, it's not implemented, and probably won't be in the near future (I think, and it's my personal opinion, that the next implementation could be positions with shorts)

@carlogrisetti
How do define eToro_exchange in your example above?  Is there a predefined Exchange for stocks?

Damn I completely missed this question. Take a look at the latest example. It's not any sort of "integration" as you might think. It's only the name I assigned to that exchange object. In the example you'll see that written as
ttse_exchange = Exchange("TTSE", service=execute_order, options=ttse_options)(price)
https://www.tensortrade.org/en/latest/examples/train_and_evaluate_using_ray.html
Closing due to being obsolele. If there are any other unrelated comments please open a new issue to keep things tidy :)


Update master from current_stable

This brings master up to date with the code used to release versions 0.8, 0.8.1, and 0.9

Codecov Report

Merging #73 into master will increase coverage by 2.34%.
The diff coverage is 65%.

@@            Coverage Diff             @@
##           master      #73      +/-   ##
==========================================
+ Coverage   69.47%   71.81%   +2.34%
==========================================
  Files          14       14
  Lines        1281     1462     +181
==========================================
+ Hits          890     1050     +160
- Misses        391      412      +21

Impacted Files
Coverage Δ

setup.py
0% <0%> (ø)
:arrow_up:

pandasdmx/__init__.py
46.87% <100%> (ø)
:arrow_up:

pandasdmx/reader/sdmxjson.py
74.24% <100%> (-6.07%)
:arrow_down:

pandasdmx/writer/structure2pd.py
95.12% <100%> (+21.86%)
:arrow_up:

pandasdmx/reader/__init__.py
97.56% <100%> (+0.06%)
:arrow_up:

pandasdmx/api.py
54.5% <37.5%> (+7.02%)
:arrow_up:

pandasdmx/utils/__init__.py
70.68% <40%> (-8.39%)
:arrow_down:

pandasdmx/reader/sdmxml.py
59.87% <47.27%> (-30.85%)
:arrow_down:

pandasdmx/writer/data2pandas.py
72.17% <67.39%> (+4.14%)
:arrow_up:

pandasdmx/model.py
81.21% <77.66%> (+5.48%)
:arrow_up:

... and 5 more

Continue to review full report at Codecov.

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

Merging despite the one failing check, "codecov/patch". This basically means that the new code added in these branches has slightly lower test coverage (65%) than the code that existed before (69%). Since testing is expanded by #72, this is not a long-term issue.


Expose configuration hook for custom module resolution

Following on from #6012 and #5039, it'd be nice to have the ability to add TypeScript to projects with heterogeneous module resolution/packaging systems and have typechecking work.
The paths and baseUrl configuration options get us a long way, but they don't deal well with module loaders that are less widely used, frequently changing, or that allow for complex module resolution rules. Letting the user entirely outsource all module resolution concerns using the configuration would allow type checking to work regardless of the user's preference of npm, bower, requirejs, browserify, jspm, webpack, etc.
A simple approach might be to have a tsconfig option for a parametrized command line invocation (similar to npm scripts) that is able to resolve modules. Each package manager can distribute a tool that implements module resolution according to its own needs, which the users can install and point to in their tsconfig.json.
Exactly how extensive the API needs to be is up for debate. It could be something as simple as a function that maps requested module names to file system paths, or a more abstract implementation of something like System from here, which could open up more interesting use cases (e.g. "virtual" modules built using code generation, or F# style type providers).

Note that this is already an option for non-tsc environments because the host object given to the compiler exposes resolveModuleNames which hosts may implement to their liking.
I'm not sure there's anyone who would want this ability who isn't already loading TS through some more advanced build pipeline.

@RyanCavanaugh I want to continue using VS Code as a regular editor, but also be able to use JSPM/ bower/custom webpack without doing a lot of fiddling with paths in tsconfig.
It would also be nice if I could more generally hook into the module resolution pipeline without having to implement a full language server and wrapping extension.

I think this could become quite flexible and working but without having too many options. Currently typescript uses a fallback strategy when resolving modules and that forgiving Behavior doesn't need to change as a result of this feature. for example let's say I'm using moment and_bootstrap_.
I can't provide a path mapping for bootstrap because of how its declaration file references jQuery. so maybe I have installed that declaration using npm but I have moment installed with JSPM and moment comes with its own type declaration and I that is the one I want to use and I can today just manually. so when TypeScript resolves moment, I could via this feature suggest<EMAIL_ADDRESS>as a possible path resolution. However, in the case of bootstrap, the default rules could still be applied.

I've also been trying to cobble together a working vscode / typescript / jspm project and eventually came to a similar conclusion as this issue describes.
My thoughts were a new compilerOption which would identify a custom resolver module that would  return an array of objects, mapping containingFiles / moduleNames keys to module locations.
[
  "*": {
    "module1": "path/to/module1",
    "module2": "path/to/module2"
  },
  "module1": {
    "module2": "different/path/to/module3"
  }
]

When should such a function be called?  It could be called early on in the process and cached but if design time support is required then the map would need updated dynamically.  I suspect Typescript already has a mechanism to do this.

I believe this could also solve some issues with remote/network file systems at larger organizations (e.g. #16426).

@masaeedu I played around a bit with modifying ts.sys to use"virtual modules"/type providers. I'd be very interested if this API was flexible enough to support virtual modules in a more general way.

Would a new "moduleResolution" option make sense for this? Maybe something like yarnpnp? I'd like to have the basic tsc toolchain work yarn plug-n-play

Hi. I propose some solution in #28624 that allow to provide in compilerOptions custom file resolution function (from node module or file). It works fine with tsc and language server.

The current situations makes this a no-go for us to use typescript.
Even the flow functionality of allowing a regex replace to the filename would allow us to use our multi-app module resolution. customResolution would be perfect.

Any progress on this? Just had to convert a project in a monorepo to yarn-pnp to resolve a dependency problem, and because I now have no node_modules in that project, tsserver does not find any of my dependencies :(

I was thinking about it a little bit, and given sufficient metadata about the types in the codebase, you could also use this as a sort of primitive typeclass system. Resolving:
type eq<a> = { equals: (x: a, y: a) => a }

type arrEq = <a>(a: eq<a>) => eq<e[]>
const arrEq: arrEq = a => {
  const equals = (xs, ys) => all(zipWith(ea.equals, xs, ys))
  return { equals }
}

const strEq: eq<string> = { equals: x => y => x === y }

type t = eq<string[]>
const { equals }: t = require("<resolve>") // Here we introspect over the types of things in the codebase, and produce a module that exports `arrEq(strEq)`

Of course, this is a rather nutty idea, and maybe not the best use case for this sort of feature, but as Jurassic Park teaches us, if you can, you should. What could go wrong?

Any move on this?

This could actually solve my feature request https://github.com/microsoft/TypeScript/issues/31703

@RyanCavanaugh  This could actually solve my feature request https://github.com/microsoft/TypeScript/issues/31703
I could just write custom resolution strategy plugin to address my needs.

@masaeedu may be you can attempt library of typescript-module-alias

Any move on this?

Any? 🔢

It would be really great to have this on TS. Webpack and Babel both support writing custom resolvers. ESLint also has, through eslint-plugin-import.
There's a similar issue with some good insights on its usefulness: https://github.com/typescript-eslint/typescript-eslint/issues/2771 (I actually think it should've been here as it's not typescript-eslint job to solve this but tsc).

Year 2021, still looking forward on this.
This document https://www.typescriptlang.org/docs/handbook/module-resolution.html#module-resolution-strategies now is some complex, why not just give a hook function for user to do the resolve.

This would allow to type check one project by types from others, for example i have a common-ui package that has a peerDependencies for react17 and 18, and devDependencies is set to react18.
I would like to import it in app17 and typecheck common-ui with react17, now it's not possible as the node resolves kiks in and picks types from common-ui node_modules.


[cp] Add test for cycle symlink

Related: https://github.com/shelljs/shelljs/issues/739

test passes in the local environment... :sweat:

Could you rebase? I recently updated CI to use GitHub actions instead. This may be more reliable than the old environment.

Codecov Report

Merging #1045 (cb79978) into master (9a0e5f6) will decrease coverage by 0.14%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1045      +/-   ##
==========================================
- Coverage   97.19%   97.04%   -0.15%
==========================================
  Files          36       36
  Lines        1354     1354
==========================================
- Hits         1316     1314       -2
- Misses         38       40       +2

Impacted Files
Coverage Δ

src/cp.js
98.46% <ø> (+6.92%)
:arrow_up:

src/which.js
80.00% <0.00%> (-15.00%)
:arrow_down:

src/cmd.js
94.11% <0.00%> (-5.89%)
:arrow_down:

src/find.js
95.45% <0.00%> (-4.55%)
:arrow_down:

src/ln.js
96.87% <0.00%> (-3.13%)
:arrow_down:

src/ls.js
97.67% <0.00%> (-2.33%)
:arrow_down:

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 9a0e5f6...cb79978. Read the comment docs.

I don't know how to solve this problem in windows, so close

Thanks for trying the rebase. Looks like the error message from https://github.com/shelljs/shelljs/runs/5018531671?check_suite_focus=true is:
cp » Test with cycle symlink

  D:\a\shelljs\shelljs\tmp04812076144037267068[653](https://github.com/shelljs/shelljs/runs/5018531671?check_suite_focus=true#step:5:653)06818777819\test\cp.js:635

  Value is not falsy:

  'cp: cannot create directory \'new\': No such file or directory'

  _utils.default.skipOnWinForEPERM (test/cp.js:635:9)
  Object.skipOnWinForEPERM (test/utils/utils.js:31:5)
  t (test/cp.js:629:11)

I wonder if new is a reserved word on Windows, or maybe we just can't create the dir for some reason?

'cp: cannot create directory \'newsub\': No such file or directory' 🤔


chore(packages): license-reporter and fixed version of express

Connects to https://github.com/bucharest-gold/entente/issues/186

Pull Request Test Coverage Report for Build 95

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.774%

Totals

Change from base Build 90:
0.0%

Covered Lines:
23

Relevant Lines:
23

💛  - Coveralls


EPMDJ-2047. Fixed new build notification generation mechanism

warning: cheks won't pass until admin part is merged


chore: add pb runner troubleshooting

Changes

[!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

#1432  👈 (View in Graphite)
#1430
#1429
#1426
#1427
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Naming Conventions: "Backroll" prefix redundant?

The library currently has a number of typenames prefixed with "Backroll," which seems to add a lot of name bloat.  The examples seem to deal with this name bloat with use backroll::*;.  Rather than typing something like
use backroll::*;
// ...
    let local_handle: BackrollPlayerHandle = session_builder.add_player(BackrollPlayer::Local);

would it be more idiomatic to leave the namespace intact and use
    let local_handle: backroll::PlayerHandle = session_builderr.add_player(backroll::Player::Local);

to avoid conflicts?
I'm not super familiar with Rust naming conventions, so consider this a suggestion, but it seems like this is the convention among other Rust libraries, with names like Context, Sprite, etc going un-prefixed.

This is a bit of a holdover from the direct GGPO port where C doesn't exactly have namespaces/modules. Should be a pretty simple global %s/Backroll//g if anyone wants to tackle this.

I'd like to claim this 🙂

So it looks like there's a BackrollConfig in backroll/lib.rs, as well as a Config in backroll/sync.rs. I'm thinking that BackrollConfig should be renamed to Config and then changing the name of Config in sync.rs to something like PlayerConfig. Does that sound reasonable?

For disambiguation, yes, that definitely should be renamed.


HA Group support for dhcp on-prem host

The added the option to push the subnet to HA_Group devices when creating subnets.

@sriram-kannan-infoblox Can you please review this?


feat: Added UntaggedTableError which is raised when an untagged table is used instead of a TaggedTable

Closes #192.
Summary of Changes
Added UntaggedTableError which is raised when an untagged table is used instead of a TaggedTable

🦙 MegaLinter status: ❌ ERROR

Descriptor
Linter
Files
Fixed
Errors
Elapsed time

✅ PYTHON
black
4
2
0
0.62s

❌ PYTHON
mypy
4

1
1.69s

✅ PYTHON
ruff
4
3
0
0.04s

✅ REPOSITORY
git_diff
yes

no
0.02s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff
MegaLinter is graciously provided by

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

v0.12.0
GitHub release

Your semantic-release bot :package::rocket:


Make Athena support work with Federated users

Question on this enhancement. Is this looking to leverage something like a central security account that assumes into the account in question to run Athena queries and gather information?
ie:
role in central security/audit account -> assume into account -> run Athena queries

No, the concept here is that many companies use Federated users (ex. SSO into accounts), which can end up all being just the same IAM role, so you have Alice and Bob using Okta to access the AWS account as the admin role.  So you want to know, does Bob actually use all of his privileges?  Can we limit Bob to only view access?


fix(docz-core): forward cli status code properly

Description
Currently, spawned processes (for example, gatsby build) run through CLI don't forward error code correctly to the parent process (docz build) producing success code (eg. 0) even if some child processes failed.
It can be problematic especially if we rely on it to do some CI/CD (like building a Docker image).
This PR solves the issue by forwarding properly child error code on build and serve commands.
Screenshots
See echo $? output.

Before
After

This is very helpful ! Thanks 😄 !


Delete app/views/sprint62b directory

closed as i dont want to mess it up. havent made any changes


Can't add users to channel in team

Every time I try to add a user to one of our channels in our team, dctrlvan, I'm given a list of users to add but they are all greyed out saying "user is already in team". Well I clearly know that, this is me trying to add them to a specific channel in the chat.
The picture is the menu I'm given. Where I'm supposed to add users, it says "dctrlvan" and there's 25 members. There are over 40 members in dctrlvan. I think it's showing me the amount of members in that specific channel but saying I'll be adding specifically to the team dctrlvan for some reason. I am one of the owners of the team. It's creating a lot of friction for us.

It's a bit confusing. To add someone to a channel, you just @mention them in chat.

It's a bit confusing. To add someone to a channel, you just  @mention them in chat. As you pointed out the add member option is to  add people to the team.

oh, really? I didn't realize that. Well intuitively when I go to the channel options I would assume it's specifically for that channel. Especially when it says 25 members instead of 40+ for #active-members. Do you see what I mean? I like how easy that is for adding people into a channel though.

@heronhaye @'ing someone hasn't worked for me.
I'm still presented with this UI which suggests they've not been added:

(There are two people in the team.)
Conversely, the 'add someone' button takes me to a modal for inviting people to the team, not the channel, so the person I want to add is un-addable:

cc @mmaxim

Not sure if this is the same issue, but I can't add people to channels on my teams.
I get just a blank window https://i.imgur.com/pvleZRx.png


[Issue / FR] Preview window is executing commands with sh -c instead of default shell for user

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[ ] bash
[x] zsh
[ ] fish

Problem / Steps to reproduce
First of all, thanks for the excellent tool! Great to see Go development such as this. Kudos to all contributors.
This might be my own fault so please bare with me.
Issue
Preview window does not execute commands as per default shell (zsh), and executes commands with sh -c instead. This causes issues with less-POSIX-compliant zsh scripts, such as when streaming data into a program with command substitution: --preview "cat <(echo string)". I've noticed this happening exclusively in Docker containers.
Repro Steps
Issue

Key
Value

Kernel
5.9.1-1

OS
docker container - alpine:edge

fzf
v0.24 (devel)

zsh
v5.8 (x86_64-alpine-linux-musl)

Launch a Docker container (alpine:edge): docker run --rm -ti --name fzf-test alpine:edge
Install zsh and fzf: apk add --update --no-cache zsh fzf
Force zsh as default shell for root and in /etc/shells; confirm with /etc/passwd:

# usermod -s /bin/zsh root
# echo -e "# Pathnames of valid login shells.\n# See shells(5) for details.\n\n/bin/zsh" > /etc/shells
# cat /etc/passwd | grep -E "^root:"
root:x:0:0:root:/root:/bin/zsh

Optionally, restart the container: docker restart fzf-test
Go back into the container docker exec -ti fzf-test zsh
Test fzf preview:

# cat <(echo this)
this

# ls | fzf --preview "cat <(echo this)"
[preview window]
sh: syntax error: unexpected "("

# ls | fzf --preview "zsh -c 'cat <(echo this)'"
[preview window]
this

No issue

Key
Value

Kernel
5.9.1-1

OS
Manjaro (Arch Linux)

fzf
v0.22.0

zsh
v5.8 (x86_64-pc-linux-gnu)

For some reason, in my system I didn't detect this (prior to building a Docker container with a jq+fzf tool for JSON parsing). I am running Manjaro (Arch Linux) on kernel 5.9.1-1 (x86_64), in which I installed fzf v0.22.0 and zsh v5.8 via the system's package manager pacman. Docker host is installed on the same machine, so there are no kernel version differences, and Alpine Edge is powering fzf v0.24 (devel) and zsh v5.8 (x86_64-alpine-linux-musl).
When I run the same tests in my machine directly, I never have a problem with the default shell in the preview:
% cat <(echo this)
this

% ls | fzf --preview "cat <(echo this)"
[preview window]
this

Similarly, in /etc/passwd has the user with zsh as default shell. In /etc/shells I have more shells installed/configured:
cat /etc/shells
# Pathnames of valid login shells.
# See shells(5) for details.

/bin/sh
/bin/bash
/bin/zsh
/usr/bin/zsh
/usr/bin/git-shell

Additional Info
I have also tried to mimic my system with a Docker container for archlinux:latest, and I found the results to be the same as in Alpine, where preview is using sh -c to execute the stated commands.
This is also not busybox or musl, since even after installing gcc (or in Arch) didn't change the outcome either.
I have also tried to build the latest version of fzf locally and on the containers, which didn't change the outcome as well.
Finally, in the manual and even in /src/options.go, there wasn't any option to define the default execution interpreter / shell for the preview window:
type previewOpts struct {
	command  string
	position windowPosition
	size     sizeSpec
	scroll   string
	hidden   bool
	wrap     bool
	cycle    bool
	border   tui.BorderShape
}

Workarounds

Launching preview with zsh -c prefixed, to force zsh as interpreter. e.g.fzf --preview "zsh -c 'jq -C <(echo {})'"

Change logic to avoid less-POSIX-compliant statements/commands. e.g. fzf --preview "echo {} | jq -C"

Request

If you have any insight on what I am doing wrong in terms of Docker container configuration, I would definitely appreciate the help, to figure out what's forcing sh -c instead of the current or the user's configured default shell.

Otherwise, if nothing is wrong with the container configuration (which I would doubt as it runs as expected in my machine, directly), I'd like to propose a shell preview option, if possible to define (or to pickup the same default shell as the user's config/env variable) as a Feature Request.

Once again, thank you so much for your hard work, this tool is amazing and it deserves all the best recognition.

fzf respects $SHELL value, so make sure that it's correctly set in your environment.
https://github.com/junegunn/fzf/blob/d099941360e061587bf7d42cd029c91650ef5a44/src/util/util_unix.go#L11-L18
docker run --rm -ti --name fzf-test alpine:edge

apk add --no-cache fzf zsh shadow

# Add user foo
useradd -m foo
usermod -s /bin/zsh foo

su foo
echo $SHELL
  # /bin/zsh


Refactor and finish parse-mf6-input branch.

In GitLab by @JoerivanEngelen on Apr 17, 2023, 15:05
After refactoring is completed (see linked issues), we can refactor this branch which Huite created so that Modflow6 files can be read.
https://gitlab.com/deltares/imod/imod-python/-/tree/parse-mf6-input?ref_type=heads

In GitLab by @JoerivanEngelen on May 16, 2023, 15:39
marked this issue as blocked by #376

In GitLab by @JoerivanEngelen on May 16, 2023, 15:39
removed the relation with #373

In GitLab by @JoerivanEngelen on May 16, 2023, 18:16
marked this issue as related to #138

In GitLab by @JoerivanEngelen on May 16, 2023, 15:40
marked this issue as blocked by #373

In GitLab by @JoerivanEngelen on May 16, 2023, 15:39
marked this issue as related to #373


Is there some configuations can add the retry times to get blocks?

I used the s3 as the storage, but it has the qps limits which leads some get block request 503 fails,
I want to is there some configurations can set to add the retry times to get blocks?

There's no config for that but this can be resolved in client-side retry.


[BUG] Deep nested map type configuration issue in text_embedding processor

What is the bug?
When configured with deep nested map type configuration in text_embedding processor, the embedding result will override the original value of document key:
pipeline configuration:
{
  "description": "An example neural search pipeline",
  "processors": [
    {
      "text_embedding": {
        "model_id": "qhO5xY4BYwgbtrHt7KDf",
        "field_map": {
          "category": {
            "name": {
              "en": "category_name_vector"
            }
          }
        }
      }
    }
  ]
}

And simulate the pipeline processor:
{
    "docs": [
        {
            "_index": "neural-search-index-v2",
            "_id": "1",
            "_source": {
                "category": [
                    {
                        "name": {
                            "en": "this is a name"
                        }
                    },
                    {
                        "name": {
                            "en": "hello world"
                        }
                    }
                ]
            }
        }
    ]
}

Result:
{
    "docs": [
        {
            "doc": {
                "_index": "neural-search-index-v2",
                "_id": "1",
                "_source": {
                    "category": [
                        {
                            "name": [
                                -0.10758455,
                                0.07971476,
                                -0.04948872,
                                ...
                            ]
                        },
                        {
                            "name": [
                                -0.034477253,
                                0.031023245,
                                0.006734962,
                                ...
                            ]
                        }
                    ]
                },
                "_ingest": {
                    "timestamp": "2024-04-10T03:51:53.496385Z"
                }
            }
        }
    ]
}

Expected result:
{
    "docs": [
        {
            "doc": {
                "_index": "neural-search-index-v2",
                "_id": "1",
                "_source": {
                    "category": [
                        {
                            "name": {
                                "category_name_vector": [
                                    -0.10758455,
                                    0.07971476,
                                    -0.04948872,
                                    ...
                                ],
                                "en": "this is a name"
                            }

                        },
                        {
                            "name": {
                                "name": [
                                    -0.034477253,
                                    0.031023245,
                                    0.006734962,
                                    ...
                                ],
                                "en": "hello world"
                            }
                        }
                    ]
                },
                "_ingest": {
                    "timestamp": "2024-04-10T03:51:53.496385Z"
                }
            }
        }
    ]
}

How can one reproduce the bug?
Steps to reproduce the behavior.
What is the expected behavior?
The generated embedding results should be placed in the right position of the document.
What is your host/environment?
Operating system, version.
Do you have any screenshots?
If applicable, add screenshots to help explain your problem.
Do you have any additional context?
Add any other context about the problem.

Hi, I'll look into this! Seems like a issue with nesting of depth of 2.

@krishy91 Since we're supporting list of map type, we don't want any limitation on this, e.g. supporting only depth of 2 or 3. We should consider to support deeply nested cases if possible.

Could reproduce the issue. Will push the fix & additional integrration test for such deep nesting cases.

Could reproduce the issue. Will push the fix & additional integrration test for such deep nesting cases.

@krishy91 Is there any update on the fix?

@zane-neo Is this still an issue? Can this be fixed?

Yes, this is still an issue, it seems @krishy91 doesn't have bandwidth on this, I'll pick up this.

Hello,
When I was testing on top of this commit of main branch, I followed same process in here, but I got exception, instead of the overriding issue mentioned here:
How to reproduce:

Create ingest pipeline

PUT /_ingest/pipeline/nlp-ingest-pipeline-v4

{
  "description": "An example neural search pipeline",
  "processors": [
    {
      "text_embedding": {
        "model_id": "H7tN0ZMBSNRyUB1vcEi2",
        "field_map": {
          "category": {
            "name": {
              "en": "category_name_vector"
            }
          }
        }
      }
    }
  ]
}

Simulate

POST /_ingest/pipeline/nlp-ingest-pipeline-v4/_simulate

{
    "docs": [
        {
            "_index": "neural-search-index-v2",
            "_id": "1",
            "_source": {
                "category": [
                    {
                        "name": {
                            "en": "this is 1st name"
                        }
                    },
                    {
                        "name": {
                            "en": "this is 2nd name"
                        }
                    }
                ]
            }
        }
    ]
}

Result

{
    "docs": [
        {
            "error": {
                "root_cause": [
                    {
                        "type": "class_cast_exception",
                        "reason": "class java.util.LinkedHashMap cannot be cast to class java.util.List (java.util.LinkedHashMap and java.util.List are in module java.base of loader 'bootstrap')"
                    }
                ],
                "type": "class_cast_exception",
                "reason": "class java.util.LinkedHashMap cannot be cast to class java.util.List (java.util.LinkedHashMap and java.util.List are in module java.base of loader 'bootstrap')"
            }
        }
    ]
}

Exception is thrown at this line, when
It seems to be introduced by PR #913 . I think we may close this issue and create a new one for the class_cast_exception. Please let me know if anyone has any concerns.
Thanks.

Let's keep it open for now. Once the class_cast_exception issue is resolved, the original issue might resurface.

Agree. I created issue https://github.com/opensearch-project/neural-search/issues/1024 to keep track of the issue.


feat: SwapOffers filter by network

Feature Request
We have two available networks at the moment, kakarot sepolia and sepolia. Currently, there is no feature to separate the fetched items from each network, so build a feature where the fetched items are exclusively from the network the user is logged in.
Describe Preferred Solution
When clicking a filter in the Offers page, it should bring the swaps regarding that filter accordingly to the network the user is using (sepolia or kakarot_sepolia)
Related Code
Additional Context
Code and test alongside the swaplace-subgraph repository and also, finish this task that is related.

Closed by #348


problem about .loss

It seems that the from .loss import * in models/__init__.pycould be deleted ?

Thanks. Already deleted


Variables in 'theme' need to move back to 'vars'

We released a breaking change when working on the experimental theme by moving the color variables out of vars and into theme instead.
Need to revert this and move them back into the vars file.

Related PR that wasn't merged https://github.com/IBM/carbon-components/pull/1132
Related Issue https://github.com/IBM/carbon-components/issues/1129
cc @AsthaJain1

@asudoh any ideas on best approach here? Is there a different/better way for us to turn on the experimental theme? We can't move the mixin inside of vars without causing an import loop or it not working. I can also dig in to this more tomorrow.

@alisonjoseph Probably this? https://github.com/IBM/carbon-components/compare/master...asudoh:include-theme-tokens-back-in-vars

Yes, that seems like it will work. Will open a PR from your branch and test.


XML-based format support

Hi. First, thanks a lot for this crate! It has been super useful for getting up to speed with a reasonable workflow in a short amount of time for some FEM code that I'm working on. Using your crate, I was able to quickly port some Julia code that I had.
That said, the legacy format is limited, and I will run into its limitations in the not too distant future. I might be interested in investing some time on a crate that would work with e.g. XML-based VTK formats. Would you like vtkio to remain strictly a crate for legacy IO, or would it make sense to expand the feature set to encompass the XML-based formats as well? Put differently, if I were to start such a venture, should I create a new crate for this purpose or would you accept contributions in this direction?

That's awesome to hear! I don't intend for this to be a legacy only crate. I will gladly accept contributions! :)

I understand that nom has been upgraded twice since I wrote this. I am willing to invest some time into porting the existing code to a newer version of nom. Having said this, I don't mind having dependencies on two different nom versions in the meantime.

I thought that for XML-based formats we don't have to write parsers/writers ourselves, but we can instead use one of the several available XML crates, don't you think? I have no experience with any of them, but I thought this would be far more productive at least!
I have quite some backlog of other things to do before I can start work on this, but for my current project I only need to be able to write VTK XML files, so I would probably start with that if I were to implement this soon.

Good point! We should definitely make use of some existing xml crate. https://crates.io/crates/quick-xml looks promising. I suspect we would need support for non utf-8 input since vtk xml has embedded binary data.

I agree, quick-xml looks like a good candidate! I'll update this issue if/when I get a chance to look at this (I don't expect this to be the case in the next few weeks at the very least).


Pub get throws error asking project to be upgraded

The plugin background_locator requires your app to be migrated to the Android embedding v2. Follow the steps on https://flutter.dev/go/android-project-migration and re-run this command.
I don't want to migrate my whole project at the moment. Is there a simple workaround? I was really excited about giving this plugin a try but this bumped me up.

I don't think there is a way to do that, but if you found any solution please share it with others;

Cosed the issue due to inactivity, feel free to open it anytime;