Update scala-library, scala-reflect to 2.13.7

Updates

org.scala-lang:scala-library
org.scala-lang:scala-reflect

from 2.13.6 to 2.13.7.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (2.13.6).
You might want to review and update them manually.
.github/workflows/ci.yml
.github/workflows/site.yml

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang" } ]

labels: library-update, early-semver-patch, semver-spec-patch, old-version-remains

Will update manually. CI failure is missing version of silencer for 2.13, but we don’t need silencer on 2.13, since it provides @nowarn, which silencer on 2.12 will understand.


Random ids are being assigned to Persons

I tried to run the solution on the Oxford Town Centre Video Sample (Link: https://www.kaggle.com/ashayajbani/oxford-town-centre?select=TownCentreXVID.mp4), but the persons get random ids. Could you please help in setting up the correct parameters, if that can improve the result?

I am really sorry for the late reply and thank you for your comments.
This problem cannot be solved by just changing the parameters, I need to come up with a new way.
It is very difficult for me, but I am working on this problem now...

Fixed and will close.


Updating config

Hi! Is there any method for updating config object without re-render? It seems, like config updated only on rendering, but all my animations works with refs to avoid it.

Hi @Harbatau can you give me your use case? Also, bear in mind that re-rendering is not a bad thing per se. It can become a performance bottleneck if you re-render on every frame / drag event but re-render isn't generally a problem. React is based on declarative rendering and people use React because it re-renders!

My case is very similar for this https://codesandbox.io/s/fh8r8 with bounds for each draggable element.  I just want to update bounds in config while starting drag another item, but bounds works like this item is first dragged element.

I see. bounds could accept a function that would be computed at the start of each gesture (or even on every frame), this can be investigated.

My config is {filterTaps: true, bounds: getBounds(), rubberband: true}. Function getBounds triggered only on rerender. How to trigger or implement it at the start of each gesture? Сan you explain with a simple example?

I was suggesting an improvement to the existing API, I don’t think this is possible at the moment except if you want to reimplement the bounds feature yourself.

Aw, sorry, I misunderstood your previous answer. Thank you for fast help!

I too am looking for this feature.  My use case is that I'm using the useWheel to zoom in and out an svg image and the amount of scale for the svg is dependent on the window size (which is held in hooks state).  I performed a clamp on the movement y in the event callback but then when you overshoot the bounds, you have to scroll back through the overshoot to get back within the bounds that cause visible change.

I see. bounds could accept a function that would be computed at the start of each gesture (or even on every frame), this can be investigated.

Such feature would be fantastic! I ran into @illiaGorbatov issue as well, and bounds being able to accept a function would unlock powerful features with very concise code. Thanks for your guys' work on this amazing library!

Bounds now accepts a function that can depend on state: https://github.com/pmndrs/react-use-gesture/issues/215#issuecomment-715883462


Nbody jacobian test

I have added a test function to validate  the Automatic differentiation implemented by TensorFlow. This test computes the Jacobians of the 3D power spectrum of the N-body simulation both by Automatic differentiation  and finite differences.
The two derivatives are comparable with a tolerance error of 0.1 .
Sadly this test is a bit slow (it spends about 6/7 minutes).
I also have added a notebook with a plot to compare the two Jacobians.
https://github.com/DifferentiableUniverseInitiative/flowpm/blob/Nbody_jacobian/notebooks/num_theo_jacobian.ipynb

Haven't got around to see what YAPF formatting is, but I was looking at the test file and unless I am missing something, I am assuming this line will be a problem as it's an import from a different repo?
https://github.com/DifferentiableUniverseInitiative/flowpm/blob/Nbody_jacobian/tests/test_3D_jacobian.py#L14
I think we should make the pk function a part of flowpm/utils.py since it's kind of on similar footing as cic_paint and cic_readout. What do you guys think.


Org keys

Summary by CodeRabbit

New Features

Added a new menu item for navigating to the profile page.
Introduced API key management in user profiles, allowing input, update, and domain sharing.

Enhancements

Improved feedback dialog visibility and input conditions.

Refactor

Enhanced feedback and asset management for consistency and performance.
Updated data handling with asynchronous methods for better operations.

Bug Fixes

Corrected feedback copying functionality based on enable_input settings.

Merged into pinecone. Closing.


Where is your data folder?

in your examples #1, I found there is df = pd.read_csv('data/hiv.csv'), but I can't find where is your data folder.

I do not provide datasets of MoleculeNet in this repository.
Please refer to the official site.
http://moleculenet.ai/


[bug] assure format is a symbol

please see https://github.com/andreapavoni/panoramic/pull/27

merged, thanks!

Can we get an updated version pushed to rubygems? I see version 0.0.7 is only available to Rails 4.


Repository-local configuration

Repository-local configuration
If present, ./.stackctl/config.yaml is read on startup and loaded into
an application Config value. This configuration provides two
abilities:

To specify a version requirement, in case your specs are relying on
certain Stackctl features and/or bugfixes and you'd like to fully
ensure behaviors in both local and CI contexts

To specify some defaults: Parameters or Tags that should be
applied to all Stacks deployed from this location. For example, App,
Owner, or DeployedBy. It's tedious and error-prone to have to
specify repeated things in every specification.

The config currently look like this (all values optional):
required_version: <RequiredVersion>

defaults:
  parameters:
    <ParametersYaml>

  tags:
    <TagsYaml>

And here is an example:
required_version: =~ 1.2

defaults:
  parameters:
    App: my-cool-app

  tags:
    Owner: my-cool-team

To support this,

RequiredVersion was built and tested
ParametersYaml and TagsYaml were given "last-wins" Semigroup
instances
Config and HasConfig were built
StackSpec construction was centralized in buildStackSpec, which
grew a HasConfig constraint, which it now uses to apply defaults
for every StackSpec we ever construct

Search for the config in a few locations

Looks fine, makes sense. Comment on the Last business but eh, up to you. I always learn something new reading your PRs Pat, usual several new things!

Comment on the Last business but eh, up to you.

It's a very fair point. I originally did ParameterYaml Key (Last ParameterValue), which I think is much clearer to the intent: merging a parameters object should combine the keys with last semantics. But I found that broke the JSON instance, it stopped using the custom "parse a string or number" behavior of ParamaterValue. So I ended up encoding "combining parameter values has last semantics themselves", which I agree is "eh". I'm going to go back to how I originally wanted it and investigate the broken JSON instance some more.

I always learn something new reading your PRs Pat, usual several new things!

That's great to hear, thank you!


Arc Data on MicroK8s: Data Studio & azdata not able to get correct endpoint information when using Nginx LoadBalancer

Issue:
SQL MI
Azure Data Studio is printing incorrect endpoint results (and azdata printing no endpoint results) for SQL MI:

Observations:

Top right: azdata arc sql mi list doesn't show ServerEndpoint info (like this example shows)
Top left: Azure Data Studio UI doesn't show the Service endpoints in the UI
Bottom left: The command seems to be printing out the default 30777 endpoint (incorrect), rather than concatenating the port (32722) that was mapped to the service shown as kubectl get svc -n azure-arc-data
Bottom right: Grafana (and Kibana) works fine against the correct port 32722, when accessed with the Node IP address

Also tried by creating another SQL MI using azdata arc sql mi create -n "arc-sql-mi-02" - same behavior as above.
Postgres
Data Studio and azdata behavior for Postgres is slightly different:

Observations:

Top right: azdata arc postgres endpoint list -n pgres-01 prints:

No information for the PostgreSQL Instance endpoint - should be something like <IP_ADDRESS>:30435
Incorrect info for the Kibana and Grafana endpoint

Top left:

Azure Data Studio UI shows endpoints for PostGres, but incorrect port
Also PostGres seems to be stuck in "Updating" state - even though I'm able to connect/query.

Bottom left: Same as above
Bottom right: Grafana works fine against the correct port 32722

Microk8s Environment Setup
I'm running Arc SQL MI and PostGres on a local MicroK8s cluster in direct mode (deployed via Portal) - steps followed:

Setup MicroK8s (Snap Track 1.19/stable)
Connected K8s cluster to Arc following Jumpstart scenario
Enabled MicroK8s storage class microk8s enable storage - to enable microk8s-hostpath storage Class (required for Arc Data Controller deployment in Step 5)
To expose the endpoint from the microk8s-vm into my local, deployed an Nginx load balancer via:
kubectl apply -f https://raw.githubusercontent.com/kubernetes/ingress-nginx/controller-v0.46.0/deploy/static/provider/cloud/deploy.yaml
Deploy Arc Data Controller with these steps, except with these settings to match the Microk8s setup above from Portal:

Deployed Arc SQL MI from Portal

Note: Following the same steps above (minus the MicroK8s steps) for AKS didn't result in the endpoint errors in Azure Data Studio or azdata, so my guess is azdata isn't able to get the correct port mapping from kubectl get svc -n azure-arc-data in this local Nginx Load Balancer Setup?

Closing this issue as I was able to get it working with Nodeport to expose the service.


Update actions because currently used ones became deprecated

Currently used actions versions are going to be deprecated and should be updated (it's mostly related to Node12 deprecation in favor of Node16)

TODO:

<EMAIL_ADDRESS>still uses Node12 so there's still a warning for DCO check workflow (tisonkun/actions-dco#12)

Codecov Report
Base: 93.80% // Head: 93.80% // No change to project coverage :thumbsup:

Coverage data is based on head (d98f071) compared to base (72dc2fd).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #112   +/-   ##
=======================================
  Coverage   93.80%   93.80%
=======================================
  Files          49       49
  Lines         710      710
  Branches      100      100
=======================================
  Hits          666      666
  Misses         20       20
  Partials       24       24

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.

Codecov Report
Base: 93.80% // Head: 93.80% // No change to project coverage :thumbsup:

Coverage data is based on head (1543fe8) compared to base (91c49b2).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #112   +/-   ##
=======================================
  Coverage   93.80%   93.80%
=======================================
  Files          49       49
  Lines         710      710
  Branches      100      100
=======================================
  Hits          666      666
  Misses         20       20
  Partials       24       24

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.

Codecov Report
Base: 93.80% // Head: 93.80% // No change to project coverage :thumbsup:

Coverage data is based on head (7cb3e39) compared to base (2e22c8f).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #112   +/-   ##
=======================================
  Coverage   93.80%   93.80%
=======================================
  Files          49       49
  Lines         710      710
  Branches      100      100
=======================================
  Hits          666      666
  Misses         20       20
  Partials       24       24

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.


SUG: Point usdview at usdmanager

@superfunc over here also thought it would be nice. That is, have a widget in usdview that opens the scene in usdmanager.
Now, getting fancy, send updates to usdview via an update message and maybe a temp file to simulate in-buffer preview without saving a file.
Of course, this is really a sug against usd to start.

Pixar's talked about adding an option to launch usdmanager from usdview. I don't know if it's actually on the road map or just an idea, nor do I know if it would be a right-click option to open the current layer or something else like a top-level menu item.
In the meantime, it shouldn't be too hard to do a custom usdview plug-in to open the current stage in USD Manager. Everyone's deployment of USD packages is a bit different (you may or may not be using rez, for example), and your usdview plugins likely wouldn't be bundled with usdmanager itself. You'd probably have to build a command similar to this (this doesn't have the usdview PluginContainer logic, but you'd call something like this when that plugin menu item is triggered):
def launch_usdmanager(usdviewApi):
    from subprocess import Popen
    path = usdviewApi.stage.GetRootLayer().realPath
    cmd = "rez-env usdmanager -c \"usdmanager '{}'\"".format(path)
    Popen(cmd, shell=True)

I'd definitely recommend mentioning this on the usd project or usd-interest mailing list to get some traction on it.

https://github.com/ColinKennedy/USD-Cookbook/tree/master/tricks/usdview_auto_reloader also shows an example usdview plugin that you selectively toggle on when you want to listen for changes to any layer in your current USD stage. I have something similar that essentially just calls this at a timed interval, if/when the timer is enabled:
Sdf.Layer.ReloadLayers([x for x in usdviewApi.stage.GetUsedLayers() if not x.anonymous])

The advantage of this usdview approach is that it doesn't require any changes to usdmanager and in fact works if changes to the USD layers come from any other application.
I'm going to close this out since I think these suggestions cover your requests, but really need to be implemented in Pixar's USD/usdview repository.


start > stop for regular axis encoding?

Just so that we are aware of it...
Currently, the spec says:

The values of "start" and "stop" must be numbers, and the value of "num" an integer greater than zero. If the value of "num" is 1, then "start" and "stop" must have identical values. For num > 1, the array elements of "values" may be reconstructed with the formula start + i * step where i is the ith element and in the interval [0, num-1] and step = (stop - start) / (num - 1). If num = 1 then "values" is [start].

This means that both { "start": 0, "stop": 10, "num": 5 } and { "start": 10, "stop": 0, "num": 5 } are allowed. I just tested both in my JS libraries and both work. I think we should allow both forms, right? For latitude this is useful (the reverse way) as you can then write down your range array in natural order from top to bottom, corresponding to north -> south or how you would look at the pixels of a rendered (satellite) image.
Any objection?

Yes, I think we should allow both forms. For one thing this will give easy compatibility with CF-NetCDF.
It would probably be worth explicitly highlighting this in the spec (and/or cookbook) if you haven't done so already, as this may be surprising (and hopefully welcome) for some GIS folk.

Added this.


Training set

Hello, this job is fantastic. I would like to know the composition of the training set used in the model and whether the training code will be made public.Looking forward to your reply.

Hi, thanks for the kind words!
Our training set is made up of: Local data (~240 CT scans), instance 2022 (100 CT scans), and BHSD (190 CT scans). I will update the Readme file to clarify this point soon.
The training code is a bit messy right now, but once we're done fine-tuning the model with our monthly updates, we'll clean it and open source it.

thanks for your reply


added optimize to readme and updated pulse and accelerate links

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


List of class names used in training

Dear authors,
I am trying to use the pre-trained SegVol for inference with text prompt. To reach the best possible performance, I want to ensure that the class name I prompt matches with the one used during training. Would you kindly provide a list of class names used in training? Thank you for your assistance.
Best wishes

Most of the category names follow the original dataset setting. So you can find them from their source websites or papers.


Implement Shimmer in SwiftUI

Platforms Impacted

[x] iOS
[ ] macOS

Description of changes
Implement ShimmerView and ShimmerLinesView in SwiftUI
Known Issues:

The existing logic is off with the shimmer angle (even in the UIKit implementation, the angles vary between the views). The shimmerAngle token's comment says "Angle of the direction of the gradient, in radian. 0 means horizontal, Pi/2 means vertical," but the formula for radians is degrees * (pi/180), not -(pi/degrees) as it currently is. I will address this in a separate change. For now, the SwiftUI implementation uses the .topLeading and .bottomTrailing UnitPoints for the gradient.
matchedGeometryEffect syncs views was able to sync shimmers between two identical views, but I was not able to find a way to synchronize shimmers between views that are not the same (i.e. two views that are the same size will be in sync, but different views may fall out of sync). This will need to be addressed.

Pull request checklist
This PR has considered:

[x] Light and Dark appearances
[x] iOS supported versions (all major versions greater than or equal current target deployment version)
[ ] VoiceOver and Keyboard Accessibility
[x] Internationalization and Right to Left layouts
[x] Different resolutions (1x, 2x, 3x)
[x] Size classes and window sizes (iPhone vs iPad, notched devices, multitasking, different window sizes, etc)
[ ] iPad Pointer interaction
[x] SwiftUI consumption (validation or new demo scenarios needed)
[ ] Objective-C exposure (provide it only if needed)

Microsoft Reviewers: Open in CodeFlow

Can you grab a before/after binary size comparison?

Can you grab a before/after binary size comparison?

Before: 34,391,068 bytes (34.8 MB on disk)
After: 35,032,396 bytes (35.4 MB on disk)

Can you grab a before/after binary size comparison?

Before: 34,391,068 bytes (34.8 MB on disk) After: 35,032,396 bytes (35.4 MB on disk)

That's quite the increase. Do you think you could get the after size without the new demo, so we can see how much just the new control is adding?

Can you grab a before/after binary size comparison?

Before: 34,391,068 bytes (34.8 MB on disk) After: 35,032,396 bytes (35.4 MB on disk)

That's quite the increase. Do you think you could get the after size without the new demo, so we can see how much just the new control is adding?

Without the demo: 34,674,828 bytes (35.1 MB on disk)

Can you grab a before/after binary size comparison?

Before: 34,391,068 bytes (34.8 MB on disk) After: 35,032,396 bytes (35.4 MB on disk)

That's quite the increase. Do you think you could get the after size without the new demo, so we can see how much just the new control is adding?

Without the demo: 34,674,828 bytes (35.1 MB on disk)

I think we're still gonna have problems pitching a 280kB size increase for this one component...


From FP #2:  Missile positioning lag + ATGM buff

mecaguy03 posted:
Also if the ACF missile devs do read this thread, ACF missiles at the moment cause strain on the server when they reload, and if they reload while moving at speed the missiles are not parented in the right position on the launcher. Specifically this has happened with the 4x ATGM racks.
Also the range on the ATGMs should be buffed in my opinion.

I agree with the range buff. I would also agree for an overall buff to all missiles once counter measures are added.

The parenting issue has been around for quite some time, though I'm not sure how we could tackle it. For the sake of comparison, E2's propcore extension is known to have the same issue with using setPos() and parentTo() in the same tick, so this isn't a new topic.
As for the reloading lag, I'm not sure what this could be caused by. Other than spawning a missile entity on a rack and parenting it, I don't see any code that could cause unreasonably high amounts of strain.
Lastly, the AGM-114 is designed to be more effective than the BGM-71E, but at the expense of agility. Range isn't much of an issue, as it's meant to be fired from aircraft. If you can't get enough range out of them, fly higher - these missiles cannot approach their target horizontally, and instead require altitude to glide towards the target.

Good answers, added to https://github.com/Bubbus/ACF-Missiles/wiki/Questions---Answers.
Going to take a look at the parenting thing again, so keeping this open for now.

Parenting issue seems better in 351e6f7c0bc3f8129ac9a0e5ed69b42f34ce557b.  Will roll out this patch later in the week.  Re-open if issue persists.
Closing old balance issues due to subsequent changes.  Re-open if you feel the issue persists.


router/ParamsMap: URL wildcard match containing "%2F" cannot be distinguished from "/"

The current ParamsMap implementation always unescapes param values. This means that, in a wildcard match at the end of a URL, the app has no way of distinguishing a slash as a URL separator from a slash that was originally a "%2F".
Example code:
let mut p = leptos_router::params::ParamsMap::new();
p.insert("rest", "/foo/bar/one%2Ftwo/".into());
assert_eq!(p.get_str("rest"), Some("/foo/bar/one/two/"));

This is an accurate statement.
Could you say more about what the issue is? For example, "I'm trying to parse the string by splitting it apart at / so I want to preserve the escape characters"?
Related discussion re: escaped vs unescaped characters in (query or path) params in #2601 #2602 #2651.

(Thinking on it a little more, the fact that it's a wildcard segment and specifically the escape code for / is what makes this different from all other cases, I guess.)

(Thinking on it a little more, the fact that it's a wildcard segment and specifically the escape code for / is what makes this different from all other cases, I guess.)

Yeah, that's it, the wildcard match is all delivered as one string and the app is responsible for splitting it into segments. There's other ways of approaching this – for example, a wildcard segment could instead handle the splitting before inserting into the params map, and then wildcard segments would be Vec<String> or Box<[String]> instead of just String.

There is also another, very simple solution here: When you have this particular kind of wildcard route, simply parse the URL yourself. It is only a line or two to .strip_prefix("...") from the path.


Fixed breadcrumb overriding in changelist and changeform templates.

Conditional block overriding in Django does not work, as is indicated by these threads below:
https://code.djangoproject.com/ticket/10975#comment:5
http://stackoverflow.com/questions/12088222/calling-block-inside-an-if-condition-django-template
http://stackoverflow.com/questions/942797/why-cant-i-nest-a-block-tag-inside-an-if-tag
This PR is to fix admin change list and change form templates that are having above problem.

Thanks for the patch. This also needs a Trac ticket that explains the issue and some tests to demonstrate what it fixes.

Closing due to lack of follow up.


Support "trackCustom" method in Facebook Pixel adapter

Add support for trackCustom method to Facebook Pixel adapter.

can we merge this? @poteto

@mike-north @chrismllr can we merge this guys?


Finesse spelling and improve consistency of messages

Minor grammatical corrections,
At least one misspelled word,
Referring to the word URL in all caps.

@gaugebot recheck the pull request.


🐛 [cloud_firestore] DocumentReference.set docs is misleading

Bug report
The documentation of DocumentReference.set states (emphasis mine)...

Sets data on the document, overwriting any existing data. If the document does not yet exist, it will be created.
If SetOptions are provided, the data will be merged into an existing document instead of overwriting.

The analogous comment in the Java implementation documentation states (emphasis mine)...

Writes to the document referred to by this DocumentReference. If the document does not yet exist, it will be created. If you pass SetOptions, the provided data can be merged into an existing document.

The Dart comment suggests that the method will do the merging even if SetOptions(merge: false) is passed.

Also merge will update the createdAt timestamp with merge:true


Wrong file type detection while file starts with comments larger than 5 lines.

Hello @yaegassy,
Thanks a lot for that wonderful plugin.
I am not sure if this extension is directly interacting with the file type, but I noticed that detection is not working properly when the template is starting with a comment that is longer than 5 lines.
Something like:
{#
Comment longer than 5 lines.
Comment longer than 5 lines.
Comment longer than 5 lines.
Comment longer than 5 lines.
Comment longer than 5 lines.
Comment longer than 5 lines.
#}

{% extends "parent/something.html" %}
{% block page_title %}{{ test }} - Grid - {{ appbuilder.app_name }}{% endblock %}
{% from 'appbuilder/loading_dots.html' import loading_dots %}

{% block tail_js %}
{{ super() }}
<script>
  const something = {{data| tojson }};
</script>
<script src="{{ url_for_asset('something.js') }}"></script>
{% endblock % }

Removing one line in the comment will result in a correct file type detection -> htmldjango, while this stays for me as html.
Any idea on how to fix this ?
Best regards,

My bad, it appears that the plugin is not interfering with the file type detection at all.
My bad.


replace gen_filenames() with iter_all_python_module_files() in runserver_plus

I cloned the repo (master branch), made all steps in the manual for Python3-based installation.
Tried to run runserver_plus and got an import error:
ImportError: cannot import name 'gen_filenames' from 'django.utils.autoreload'
Triggered by build/env/lib/python3.7/site-packages/django_extensions/management/commands/runserver_plus.py:
from django.utils.autoreload import gen_filenames
After searching for the same issue in vain I just went to the Django GitHub repo to see when and with what they replaced gen_filenames().
The commit when as I believe they deleted gen_filenames(): link
Then I replaced gen_filenames() with iter_all_python_module_files() like this:
from django.utils.autoreload import iter_all_python_module_files

if use_reloader and settings.USE_I18N:
    extra_files.extend(filter(lambda filename: filename.name.endswith('.mo'), iter_all_python_module_files()))

And runserver_plus worked just fine.
Maybe it will be useful for someone with the same problem.

Thanks for the report!
Would you mind sending a pull request for the fix so that you get the credits for it?
cc @agl29


Link apply page to the reef

The purpose of this PR is to close https://github.com/aqualinkorg/aqualink-app/issues/410.

The reef name should be pre-filled and not editable here.

The reef name should be pre-filled and not editable here.


Regression in ToNodeIterator for HtmlIterable

Problem
#1415 changed the implementation of ToNodeIterator for HtmlIterable to use the helper function yew::utils::into_node_iter.
This cleaned up the code a lot, but it has the unfortunate consequence that it always returns Iterator<Item=VNode> now.
This violates an invariant imposed by the ToNodeIterator trait:

Generate a token stream which produces a value that implements IntoIterator<Item=T> where T is inferred by the compiler.

T is no longer inferred by the compiler but instead explicitly set to Html.
The consequence of this is that it can no longer be used with html_nested!.
Steps To Reproduce
See #1524
Expected behavior
The macro should allow Into<T> for any T, not just T = Html.
The previous implementation looked like this (minus macro verbosity):
(#expr).into_iter().map(|n| n.into())

The key difference is that n.into() isn't explicitly typed and so it can convert into any type that implements From<N>, not just Html.
Environment:

Yew version: 0.17.3

@siku2, are you sure that you pointed to correct issue? #1415 seems unrelated

Great catch, should've been #1452


[$250] Task – RHP present when delete task with reply

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.9-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4735083
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by:  Applause - Internal Team
Action Performed:

Go to https://staging.new.expensify.com/
Log in
Open any chat
Create a task
Open task detail
Send comment
Click on header and delete the task

Expected Result:
RHP disappears when delete task with reply
Actual Result:
RHP present when delete task with reply, but disappears when delete IOU with reply
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/user-attachments/assets/0bad3ecd-0386-4f7b-a4b7-efb7e4086d59
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01d58ba540e5cc55b0
Upwork Job ID:<PHONE_NUMBER>916800159
Last Price Increase: 2024-07-19

Issue OwnerCurrent Issue Owner: @abdulrahuman5196

We think that this bug might be related to #vip-vsp

@jliexpensify  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

Proposal
Please re-state the problem that we are trying to solve in this issue.
Task – RHP present when delete task with reply
What is the root cause of that problem?
We are only navigating back if there are no visible actions
https://github.com/Expensify/App/blob/7ba66a1542fa815eed5cb4622244bc2935593484/src/libs/actions/Task.ts#L1058-L1060
What changes do you think we should make in order to solve the problem?

remove the if
What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
Task – RHP present when delete task with reply
What is the root cause of that problem?
We are only navigating back if there are no visible actions !ReportActionsUtils.doesReportHaveVisibleActions
https://github.com/Expensify/App/blob/7ba66a1542fa815eed5cb4622244bc2935593484/src/libs/actions/Task.ts#L1058-L1060
https://github.com/Expensify/App/blob/7ba66a1542fa815eed5cb4622244bc2935593484/src/libs/actions/Task.ts#L923-L924
What changes do you think we should make in order to solve the problem?

We should dismiss modal in when the task has visible actions so change it to
if (shouldDeleteTaskReport) {
        Navigation.goBack(ROUTES.REPORT_WITH_ID.getRoute(parentReport?.reportID ?? '-1'));
    } else {
        Navigation.dismissModal();
    }

What alternative solutions did you explore? (Optional)
remove the if here if we want to navigate back to the parent
https://github.com/Expensify/App/blob/7ba66a1542fa815eed5cb4622244bc2935593484/src/libs/actions/Task.ts#L1058

@lanitochka17 to confirm, are you expecting the RIght Hand Pane (RHP) to show when an expense is deleted? i.e. to mirror the Task?

Thanks @dominictb - yeah that's the behaviour I'm seeing when testing this, so I don't think this is a bug. Closing!

Oh, I see - I misunderstood you! So you're saying the decision was made by Shawn that the RHP shouldn't exist when a task or/and an expense is deleted?

Thanks for clearing this up @dominictb - in that case, I agree: it's a bug

bump @abdulrahuman5196 we have a proposal

Hi, Will check on this tomorrow

Checking now

@dominictb 's proposal here looks good and works well. It is inline with the current expense deletion flow.
But one suggestion to instead of doing step 3 in the mentioned place, we can add another if statement to different task deletion like done here, anyways that could be handled in the PR.
https://github.com/Expensify/App/blob/7ba66a1542fa815eed5cb4622244bc2935593484/src/pages/ReportDetailsPage.tsx#L663-L666
🎀 👀 🎀
C+ Reviewed

Bump @abdulrahuman5196 for the checklist
Payment Summary

C: @dominictb $250
C+: @abdulrahuman5196 $250 (please accept the job!)

Upwork job

The PR that introduced the bug has been identified. Link to the PR:
The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:

Doesn't seem to be a regression

Determine if we should create a regression test for this bug.

Yes.

If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Go to NewDot
Open any chat
Create a task
Open task detail
Send comment
Click on header and delete the task
Verify that: RHP disappears when delete task with reply

@jliexpensify I am getting paid via newDot.

$250 approved for @abdulrahuman5196

Paid and job closed.


Doesn't work

it just doesnt work :(
have you fixed it yet??

ur using stolen assets >:(

I literally handwrote it bruh

https://github.com/Dropshotz/SS13-Jukebox-Parser-Discord-Bot/assets/68038883/a022f7b7-573a-40a8-9143-fcd73bd939ed


Module Parse Failed... You may need an appropriate loader to handle this file type

Getting this error code when I try to run the example in the readme:
./node_modules/react-github-btn/index.js
Module parse failed: Unexpected token (10:83)
You may need an appropriate loader to handle this file type.
|   }
|   render () {
|     return React.createElement('span', { ref: this.$ }, React.createElement('a', { ...this.props, ref: this._ }, this.props.children))
|   }
|   componentDidMount () {

My code:
import GitHubButton from "react-github-btn";

return (
...
<GitHubButton href="https://github.com/meerkat-citronella">
  Follow @meerkat-citronella
</GitHubButton>
...
)

my package.json:
{
	"name": "portfolio",
	"version": "0.1.0",
	"private": true,
	"dependencies": {
		"chart.js": "^2.7.2",
		"cosmicjs": "^3.2.12",
		"react": "^16.12.0",
		"react-chartjs-2": "^2.11.1",
		"react-dom": "^16.12.0",
		"react-github-btn": "^1.2.0",
		"react-redux": "^5.0.7",
		"react-router": "^4.2.0",
		"react-router-dom": "^4.2.2",
		"react-scripts": "1.1.1",
		"react-typeform-embed": "^0.1.6",
		"redux": "^3.7.2",
		"semantic-ui-css": "^2.3.0",
		"semantic-ui-react": "^0.82.1",
		"styled-components": "^5.1.1"
	},
	"scripts": {
		"start": "react-scripts start",
		"build": "react-scripts build",
		"test": "react-scripts test --env=jsdom",
		"eject": "react-scripts eject",
		"deploy": "gh-pages -d build"
	},
	"homepage": ".",
	"devDependencies": {
		"gh-pages": "^1.2.0"
	}
}

Update: the module seems to work fine on a newly-created React app.
It's still not working on my project tho, even after updating react and react-dom to ^17.0.1. Same error.

The issue is likely that you have old version of react-scripts.

You need at least react-scripts>=2.0.3, ideally upgrade it to latest version if possible.
Here is the critical commit that makes the difference if you're interested: https://github.com/facebook/create-react-app/commit/3d4440812c62e692019f72503a60c6240df2f805

I'll take your word for it re: react-scripts... my app is having trouble updating to the latest version. I ended up not using the module anyway, as I needed the buttons to be larger than the largest size offered. So i made my own components. But thanks for the prompt response anyway! cheers.


Flagged by McAfee AV

When I tried to launch the Kingdom Scanner GUI, McAfee Anti-virus flagged the .exe and quarantined it as potential threat (virus or unwanted). Has anyone else run into this? I successfully ran the scan multiple times in the past before this happened, but this just started a week ago. Any thoughts? Reinstall?
Edit: Problem solved. McAfee uninstalled. :)

Each AV has an exception list, so you can add the folder in exception list to always ignore it when scanning for treat.


validation not working

hi guys, anybody is getting this error ? it happens when creating an entity (product, brand ..)

https://github.com/fullstackhero/dotnet-webapi-boilerplate/issues/799
Issue addressed here 👆


Snippet suggestions appearing in empty line

Thanks for a very useful extension.
Describe the bug
Introduced in 1.16.0, when pressing Enter to add new lines (and then waiting a fraction of a second), a snippet is suggested and confirmed. Not sure if this is a bug or if it is a setting that can be changed.
To Reproduce
Steps to reproduce the behavior:

Make sure version 1.16.0 later is used.
Make sure Lua snippets are available, e.g. through use of selimanac.defold-vsc-snippets
Press Enter
Wait a fraction of a second
Press Enter again

Expected behavior
I would not expect suggestions on an empty line, so should just add a new line when Enter is pressed.
Screenshots
https://user-images.githubusercontent.com/24439991/108872186-d3728d00-75f1-11eb-9999-293a9cc5e40e.mp4
Environment

OS: Windows 10
selimanac.defold-vsc-snippets (or other Lua snippets)

see https://github.com/sumneko/lua-language-server/issues/401

Ah! Assuming this will be fixed in the next VSCode release then. Thanks for the quick reply.


[Bug] Plugin broken after last update

The following error is generated at Neovim startup after updating the plugin:
packer.nvim: Error running config for command-center.nvim: ...er\start\command-center.nvim/lua/command_center/init.lua:37: attempt to index local 'opts' (a number value)

Packer Config:
{
  "gfeiyou/command-center.nvim",
    config = function()
      require("user.cc").config()
      end,
    requires = "nvim-telescope/telescope.nvim",
},

Facing the same exact problem.
packer.nvim: Error running config for command-center.nvim: ...cker/opt/command-center.nvim/lua/command_center/init.lua:37: attempt to index local 'opts' (a number value)

I just pushed a new commit e61162cfe5efa8333cf2314af08655af5ba98436. Please let me know whether this fixes the problem.

Hello @FeiyouG , that fixes the original problem.
However, now the floating window appears too small:

I don't have this problem at my end. Can you share a minimal config that can reproduce this bug?

@FeiyouG Fixed! 🚀
@danielo515 I don't have your floating window issue.

Here you have it @FeiyouG
https://github.com/danielo515/Telescope-command-center-issue
You may need to clone it with git submodules or just remove the deeply nested packer folder and clone packer into it

@danielo515 From what you've shared with me, I don't see you added any commands into command_center. The floating window is too narrow because there was nothing to be displayed.

I have the same issue as @danielo515 has.
I have copied all your example from command_center.add and even removed , command_center.mode.REGISTER_ONLY from:

command_center.add({
  {
    desc = "Open command_center",
    cmd = "<CMD>Telescope command_center<CR>",
    keys = {
      {"n", "<Leader>fc", noremap},
      {"v", "<Leader>fc", noremap}
    },
  }
})

And I also have narrow palette:
![image](https://user-images.githubusercontent.com/3855655/181830651-9ea6b125-09a6-4c5a-9e01-045156494d7e.png)

I found the root cause and fixed it in 125b92306cf29e20d405a5bfe1300af13339e80e. Thank you for bringing this to my attention, @XobSod  and @danielo515.

Thank you for all the effort and quickfixes put into this!!!!
I'll try it tomorrow. Many thanks!

Ok, I couldn't wait and just updated. The problem is totally fixed for me. Many thanks!


How to get the versioned model path when implementing a custom backend (Context)

Hi,
I am new to trtis and recently working on a project that need to implement a custom backend in trtis.
I read the example code like addsub.cu or identity.cc, and found the versioned model path seems not to be available when constructing model instance with Context::Context(const std::string& instance_name, const ModelConfig& config, const int gpu_device);.
Here is a simple usecase that needs this feature: If I want to make image_preprocess.cc accepts external mean/std matrix files (i.e. hwc numpy/opencv matrix), the mean/std matrix files would be in the versioned model path. The model directory structure could be like below:
.
├── 1
│   ├── libimage_preprocess_v2.so
│   └── mean.mat
│   └── std.mat
└── config.pbtxt

A parameter is available in the model configuration that gives the path to the model repository and is available through the custom backend API: https://github.com/NVIDIA/tensorrt-inference-server/blob/master/src/backends/custom/custom.h#L64
See the param custom backend example for how to access the parameters: https://github.com/NVIDIA/tensorrt-inference-server/blob/master/src/custom/param/param.cc
I don't believe the API currently reports the model version to the custom backend.

@deadeyegoodwin I did use the "parameter" in ModelConfig in my project now, but the file path has to be absolute path rather than relative path.
As you mentioned, the custom backend can not get the model directory or version info right now, is this a special design ? If not, maybe we can add this feature in the future.

The parameter should give you the full path to the root of the model repository. From that you can append your model name and version number to get the version directory. I agree that we should provide the name and version information to the custom backend in some way. We will file a ticket to track that request.


zephyr support for to_networkx_graph

Codecov Report

Merging #435 (79fcf55) into master (cde646b) will decrease coverage by 3.05%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #435      +/-   ##
==========================================
- Coverage   90.45%   87.40%   -3.06%
==========================================
  Files          22       22
  Lines        1498     1500       +2
==========================================
- Hits         1355     1311      -44
- Misses        143      189      +46

Impacted Files
Coverage Δ

dwave/system/samplers/dwave_sampler.py
83.43% <0.00%> (-4.15%)
:arrow_down:

dwave/system/samplers/leap_hybrid_sampler.py
61.72% <0.00%> (-13.88%)
:arrow_down:

dwave/system/samplers/clique.py
77.35% <0.00%> (-5.04%)
:arrow_down:

dwave/system/composites/embedding.py
95.85% <0.00%> (-1.19%)
: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 cde646b...79fcf55. Read the comment docs.

done with the requests


[$500] Description entered for an IOU is not displayed in the request preview

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.21-4
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @anmurali
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1704388820763769
Action Performed:
1.Create a money request
2.Add a description and send it to anyone
Expected Result:
The description should be shown in the request preview in the DM or workspace chat
Actual Result:
No description is shown in the request preview in the DM or workspace chat
Workaround:
No Workaround
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/38435837/b5f999cd-bb9a-409b-ae44-50a14ad83029
https://github.com/Expensify/App/assets/38435837/8b9160a7-b495-4c89-982c-a6612b5fc3ba

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01e69677bf86c6ecba
Upwork Job ID:<PHONE_NUMBER>358760448
Last Price Increase: 2024-01-11

This is an IOU Report preview, which contains a set of IOU requests. The description is set per each IOU request separately and is displayed on them, not on the IOU Report preview.

Proposal
Please re-state the problem that we are trying to solve in this issue.
IOU becomes paid while receipt is scanning.
What is the root cause of that problem?
https://github.com/Expensify/App/blob/095161a690e0cea4cf5352b3d03212530a364fea/src/components/ReportActionItem/ReportPreview.js#L147C24-L147C24
The value of the props.action.childLastMoneyRequestComment is present and set accurately to the money request description.
https://github.com/Expensify/App/blob/095161a690e0cea4cf5352b3d03212530a364fea/src/components/ReportActionItem/ReportPreview.js#L311
However, it can only be shown if the following condition is true. In this situation the numberOfRequests is 1 and hasReceipts is false. Therefore, the money request description won't be displayed.
What changes do you think we should make in order to solve the problem?

const showDesiription = (moneyRequestComment && numberOfRequests === 1 && !hasReceipts)

https://github.com/Expensify/App/blob/main/src/components/ReportActionItem/ReportPreview.js#L321
Edit the condtion to show the text.
!isScanning &&  (numberOfRequests > 1 || hasReceipts || showDesiription)

https://github.com/Expensify/App/blob/main/src/components/ReportActionItem/ReportPreview.js#L324
Set the text as moneyRequestComment if showDesiription is true.
showDesiription  ? moneyRequestComment : previewSubtitle

@peterdbarkerUK
As @paultsimura said, https://github.com/Expensify/App/issues/33975#issuecomment-1877575794 not a bug.

Not a bug!


Deploy streamlit app with pymc-labs

The streamlit app is routed through @louismagowan's branch on a fork of the main repo
It'd be ideal to set this up with a merge to pymc-labs/main branch so that streamlit links come back to main branch.
Can streamlit be deployed with GitHub action? or is it only through their streamlit UI via an account. A bit unknown to me
See also #701
There were some details in the original issue #613 and PR #614

@wd60622 - would you have time this week for us to close this one? It should be super simple - just requires someone with full permissions on pymc-marketing to do it ☺

@wd60622 - would you have time this week for us to close this one? It should be super simple - just requires someone with full permissions on pymc-marketing to do it ☺

Hey @louismagowan
Sounds good. I will reach out to you

Thanks @louismagowan  for the help!


Makefile issue

ld: unknown option: -soname
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make: *** [libzopflipng] Error 1
help? is this a problem with mac outdated gcc?

you're not using gcc, you're using clang.


Can not delete schemas

Use Parse Server version 2.1.3

I create 'test' class via Schema API - CREATE

In database, 'test' will insert into _SCHEMA collection
'test' collection has not created

I will delete 'test' classs via Schema API - DELETE

Result: {},
'test' have not deleted from _SCHEMA collection yet

Fixed by #802


feat: get DU tool to Window VHD

What type of PR is this?
/kind feature

What this PR does / why we need it:
Download du.exe tool to Windows VHD and this tool can be used for debugging AKS Windows node.
Which issue(s) this PR fixes:

Fixes #
Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 79.313%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2285

Relevant Lines:
2881

💛  - Coveralls


haskellPackages.haskell-language-server: 0.6.0 -> 0.7.0

Supersedes https://github.com/NixOS/nixpkgs/pull/106705
Remove hls-brittany as it's no longer necessary
Fix the update script to include hls-explicit-imports-plugin and
hls-retrhls-retrie-plugin
CC @maralorn
Things done

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

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

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

Marvelous! I have tested this, it works, the PR looks fine, too.

If you liked that then you'll love this: https://github.com/NixOS/nixpkgs/pull/107128

You know me better than my amazon account does. :smile:


public CI with github status

workflow

push a commit
see "in progress" github status
wait reasonably short time for status ok (green) or fail (red)
click through to CI log

questions / caveats

work with PRs on forks (can't expose credential to code in fork)?
cross-repo dependencies
separate out image build (and push to dockerhub)

Last todos:

[ ] Add status to the acceptance test build
[ ] Add statuses to riff-pr
[ ] Add statuses to dockerimages

should we move the pivotal-cf/pfs-system-test to projectriff\acceptance-test?
change the sk8s docker hub user to riffci for images we use

No objection, but this issue has been fairly busy and I'm anxious to close it in its own right.

Except, actually, you're right about both of those being private bits.
Ugh.

Infinite todos

[x] Add status to the acceptance test build can't be done with our current setup
[ ] Status for helm charts
[ ] Add statuses to riff-pr
[ ] Add statuses to dockerimages

Actually, I have a hunch that I am wrong about not being able to set nice commit statuses for every job / commit permutation. The theory would be:

Set the status for every commit in every job.
Rely on the context key to distinguish what is going on (so a commit might be passed for ci/riff/build-http-gateway-container but pending on run-riff-system-tests-gke).
Use version: every to ensure we are testing all the permutations. Right now we don't.

blocking on projectriff/ci#50

I'm going to call this done. We have a public CI, images are pushed to public repos and so on. I'll break out missing FATS tests into another issue.


feat(api): support pull requests on buildkite

https://buildkite.com/docs/pipelines/environment-variables says that
the pull request is available via BUILDKITE_PULL_REQUEST, which is
either the number for a pull request, or "false" for a
non-pull-request.

I don't think that check failure is related to my changes?

@huonw thanks for the contribution, this looks great!
Sorry about the linting failure, that indeed has nothing to do with your work. WIll fix separately.


could not connect to pocketbase

Hello @kendofriendo! Is the root URL for Pocketbase instance available under the provided address? If so, please look at the response headers and Pocketbase logs - maybe there's some useful information for us.

I don't use the root url for anything, but I assigned it to one of my other apps and it works fine.
response headers:

pocketbase logs:

In case it's relevant, the server running caddy is separate from the computer running grimoire. I have it connected via wireguard.

Is this problem still persisting for you while using https://github.com/goniszewski/grimoire/releases/tag/v0.3.5?

Yes.

I have a similar, if not the same, problem - on v0.3.5.
And honestly I don't really know where to look for more information to debug this.
I deployed the app as shown  in the "Quick Start" of the docs, but changed the stack so no ports were directly opened.
instead I directed my reverse proxy to make the app and the pocketbase instance available by two separate domains.
The domain of the pocketbase instance is used as the value of PUBLIC_POCKETBASE_URL.
This is the content of my .env file
# Set it if you're using external PocketBase installation (default: http://pocketbase)
# Example: PUBLIC_POCKETBASE_URL=https://grimoire.mydomain.com:8090
PUBLIC_POCKETBASE_URL=https://grimoire-pocketbase.apps.hivenet.xyz
POCKETBASE_URL=https://grimoire-pocketbase.apps.hivenet.xyz
# RECOMMENDED: Change this to your email
<EMAIL_ADDRESS># RECOMMENDED: Use a secure password. Can be later changed in PocketBase's admin panel
ROOT_ADMIN_PASSWORD=changeme
# Set this to your domain name (default: http://localhost:5173)
# Example: PUBLIC_ORIGIN=grimoire.mydomain.com
PUBLIC_ORIGIN=grimoire.apps.hivenet.xyz
# Set this to true if you're using HTTPS (default: false)
# Example: PUBLIC_HTTPS_ONLY=true
PUBLIC_HTTPS_ONLY=true
# Change to the port you want the app to listen on (default: 5173)
# Example: PORT=80
PORT=5173
# Set this to true if you want to disable public signups (default: false)
PUBLIC_SIGNUP_DISABLED=false

and this is my docker-compose.yml:
---
services:
  pocketbase:
    image: spectado/pocketbase:0.22.10
    container_name: grimoire-pocketbase
    restart: unless-stopped
    #    ports:
    #  - '8090:80'
    volumes:
      - pb_data:/pb_data
      - pb_migrations:/pb_migrations/
    healthcheck:
      test: wget --no-verbose --tries=1 --spider http://localhost:80/api/health || exit 1
      interval: 5s
      timeout: 5s
      retries: 5
    env_file: .env
    depends_on:
      - grimoire
  grimoire:
    image: goniszewski/grimoire:latest
    container_name: grimoire
    restart: unless-stopped
    env_file: .env
    volumes:
      - pb_migrations:/app/pb_migrations/
    build:
      context: .
      dockerfile: Dockerfile
    healthcheck:
      test: wget --no-verbose --tries=1 --spider http://localhost:$PORT/api/health || exit 1
      interval: 5s
      timeout: 5s
      retries: 5
      #ports:
      #- '$PORT:$PORT'
volumes:
  pb_data:
  pb_migrations:

According to the log of my grimoire container the POCKETBASE_URL is http://localhost:8090 - a value I never specified.
If I attach to the grimoire container, open a shell and view the environment variables, the correct domain is shown.
This is a screenshot of the container logs:

And here is a screenshot of the env of a shell inside the grimoire container:

If I hunted down those lines correctly, then that POCKETBASE_URL http://localhost:8090 is set here:
https://github.com/goniszewski/grimoire/blob/d1aaaae34fbaa432e2488f8500ad0b3f383830c0/src/lib/config.ts#L11
And if I understand the code correctly, then that value should only be used, when the system is started in dev-mode - which it is not (also shown in the screenshot above).
But on the screenshot above the log also shows a recurring app error, which does show my specified URL.
So let's ignore that POCKETBASE_URL variable for now and let's investigate the errors in the log

And here I get stuck, because I don't understand enough about Typescript or this app.
Let's look again at the error in the log:

This path in the error looks like the URL for the grimoire app/container - and if I understand correctly, then the /internal/pb/ path is meant for proxying pocketbase connections through internal endpoints.

If I open the Browser DevTools in the webapp, I get the following:

The error notification popping up shows the correct URL, but the DevTools reveal that the grimoire app URL is doubled in the sent XHR request.
I guess that is the cause of this issue, but I don't know how to proceed - I am not a dev and I'm absolutely not well versed in web app development.

Had the same issue. The PUBLIC_ORIGIN URL was doubled in the Request to /pb/api/health.
It seems to work correctly if you add http:// or https:// to the PUBLIC_ORIGIN.

@Flycro thanks for that info.
changing the PUBLIC_ORIGIN URL to include https:// does make the notification banner dissapear.
But now the DevTools show a TypeError, because something is undefined.
I would ignore that, if I could use the application, but the login fails - the signup works however

The error TypeError: parts[1] is undefined is thrown on page load.
The error Uncaught (in promise) TypeError: NetworkError when attempting to fetch resource. comes up, when I try to log in.
Here are the full contents of that second error:
Uncaught (in promise) TypeError: NetworkError when attempting to fetch resource. [0.HKRr2abQ.js:21:12144](https://grimoire.apps.hivenet.xyz/_app/immutable/nodes/0.HKRr2abQ.js)
    m Immutable
    AsyncFunctionThrow self-hosted:857
    (Async: async)
    Br Immutable
    InterpretGeneratorResume self-hosted:1461
    AsyncFunctionNext self-hosted:852
    (Async: async)
    z Immutable
    map self-hosted:221
    Immutable 6
    InterpretGeneratorResume self-hosted:1461
    AsyncFunctionNext self-hosted:852
    (Async: async)
    tn Immutable
    <anonym> https://grimoire.apps.hivenet.xyz/:72
    (Async: promise callback)
    <anonym> https://grimoire.apps.hivenet.xyz/:71

And here is a screenshot from the pocketbase webinterface after signing up:

I'll probably just wait for a future update to set this up again from scratch.

Please upgrade to the latest version of Grimoire. You can use the integrated migration tool to transfer your data to new format.


Publish 11/05/2021, 10:30 AM

Docs Build status updates of commit 6689f78:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Enable-Mailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Enable-RemoteMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-AutoSensitivityLabelRule.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-DlpComplianceRule.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-AutoSensitivityLabelRule.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-DlpComplianceRule.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-Mailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-RemoteMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Add support for Travis webhook to GitHub action

There is a method using webhooks called repository dispatch or something like that, and I'd have to add instructions for getting another GitHub token to your Travis environment.
But this way, your build must succeed in order for deployment to function.

The other way is to use something I use in SimpleJWT: GitHub's status API, or with a GitHub action called something like GitHub-wait-for-status-check. The other option is obviously workflow_dispatch :P


⚠ ClamAV has degraded performance

In b7aeb63, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 186 ms

Resolved: ClamAV performance has improved in 9681444.


Golang alpha - something to test/play with

This is so cool and so the right direction for K8s configuration - thanks!
Any way we can test/play and help with Go support?

https://github.com/awslabs/cdk8s/issues/119
Upvote this if you care about go !

Thanks already did ;-) But this will not be created in a day, so maybe something is on the way we can play with - or maybe just get started with client-go.

Closing as duplicate


DynamicForm - Check if hiddenfields property is undefined

Q
A

Bug fix?
[x]

New feature?
[ ]

New sample?
[ ]

Related issues?
Fixes Bug created in PR #1307

What's in this Pull Request?
Added a check if the hidden fields are undefined, otherwise the dynamic form would not load.

Oooops! This is a very good catch! Thanks, @SmarterJB! @joelfmrodrigues or @AJIXuMuK will you merge it or should I? Thanks.

Merged, many thanks for the PR @SmarterJB


Csv data import problem

Csv data import problem
The problem occurs when a large csv file has an error in its syntax
The next lines are not checked and this makes the next data not imported
I think the better option is to check the next lines if there is an error in one line and add them if there is no problem.
And finally, after import, it will show how many lines of data have not been imported
Line 6
I tested this in the Zui UI and it has problems
imdb.csv

I wrote a piece of code to solve the problem of files
I think it would be much better if such a feature is added to the main software itself
I enter the number of commas manually
`@echo off
setlocal enabledelayedexpansion
set "input_file=*.csv"
set "output_file=output.txt"
for /f "delims=" %%a in ('type "%input_file%"') do (
set "line=%%a"
set "count=0"
for %%b in ("!line:,=" "!") do set /a count+=1
if !count! lss 14 (
set /a diff=14-!count!
set "line=!line!,"
for /l %%c in (1,1,!diff!) do set "line=!line!,"
)
echo !line!>>"%output_file%"
)
echo Done!
pause
`

@AFgh24: Thanks for reporting. Indeed, this is a topic that's come up a couple times before, and we just had a group discussion about it and came up with an idea to address it at the Zed layer that's now being tracked in #4546. I'm going to go ahead and close this particular issue so please feel free to keep an eye on #4546 and see how you like it when the enhancement lands. I made mention there of your idea to make sure Zui prominently displays the error count when there's such problems.


Mldb 2142 prefix suffix

Two simple builtins to remove prefix and suffix from strings.

regex_replace couldn't do that?

Yes but this is just easier and clearer.

+1


Revert "fix cuda half math function is undefined: hpow, htanh"

Reverts apache/incubator-tvm#6225

cc @cloud-mxd due to the CI problem defined in https://ci.tvm.ai/blue/organizations/jenkins/tvm/detail/PR-6162/29/pipeline/
Please feel free to send another PR and fix the related problem.

cc @jroesch

another PR: https://github.com/apache/incubator-tvm/pull/6253


about hmset

// demo
type User struct {
    Id         int32  `json:"id"`
}
data := User{
    32,
}  //
m := structs.Map(data)

RedisPool.Do(radix.FlatCmd(nil, "HMSET", redisKey, m))

a problem :  Key in redis is uppercase
Is there a way to let redis's key lowercase ?

Are you referring to the Id field being uppercase? That is something which is done by your structs.Map function when it's converting the struct to a map, it's not something radix has control of.


What is the difference between WriteValueWithoutResponseAsync  and WriteValueWithResponseAsync ?

What is the difference between WriteValueWithoutResponseAsync  and WriteValueWithResponseAsync ?
I tried to send data with WriteValueWithResponseAsync today, but it didn't seem to work.

You need to check the Properties of a characteristic to see which method(s) it supports. A write without response is quicker because no acknowledgement is sent but this also means you won't know if an error occurred.


allow HTML elements

@edlvj Thanks for the contribution, I will publish the npm module.

Hey @edlvj your PR has been published to npm as 3.0!


Add strikethrough effect

Adds a strikethrough effect that works with termion, pancurses, and crossterm backends. For ncurses and bear_lib_terminal it does nothing.

Thanks for the work!
A bit unfortunate that it doesn't work on ncurses, but that's certainly better than nothin!

Sure! Ncurses doesn't seem to support strikethrough, that's why I didn't add it.


Blog not responsive

css for blog not complete. should share basic style sheet with main app page.
note also the 2nd screenshot, background shading on menu items has gaps.

Only just seen this. For the responsive issue, you need to add the following the the <head>:
<meta name="viewport" content="width=device-width, initial-scale=1.0, maximum-scale=2.0, user-scalable=yes" />

The issue of the space between the items is down to a peculiar way browsers deal with whitespace between list items. Basically, in the HTML the list items can't have any spacing between them. I think the whitespace gets rendered as a single space character.


EscapeFilter the group dn membership

The uid provided to the group filter must be properly escaped using the provided
ldap.EscapeFilter function.
Fix #20181
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

Is it the only thing that must be escaped within this function or related functions? LGTM

I believe so currently but I note we have functions like:
func (source *Source) sanitizedUserQuery(username string) (string, bool) {
	// See http://tools.ietf.org/search/rfc4515
	badCharacters := "\x00()*\\"
	if strings.ContainsAny(username, badCharacters) {
		log.Debug("'%s' contains invalid query characters. Aborting.", username)
		return "", false
	}

	return fmt.Sprintf(source.Filter, username), true
}

Which should just be escaping things properly instead of declaring bad characters.

It's been confirmed that this PR solves the related issue.


Set trust proxy to allow the use of secure session cookies behind nginx.

Re: https://www.npmjs.com/package/express-session#cookiesecure

Codecov Report

Merging #255 into master will increase coverage by 26.50%.
The diff coverage is 62.50%.

@@             Coverage Diff             @@
##           master     #255       +/-   ##
===========================================
+ Coverage   58.56%   85.06%   +26.50%
===========================================
  Files          14       14
  Lines         403      375       -28
  Branches       91       48       -43
===========================================
+ Hits          236      319       +83
+ Misses        167       56      -111

Impacted Files
Coverage Δ

verification/curator-service/api/src/index.ts
83.33% <62.50%> (ø)

...r-service/ui/src/components/CompletenessCharts.tsx

.../curator-service/ui/src/components/NewCaseForm.tsx

.../curator-service/ui/src/components/SourceTable.tsx

...cation/curator-service/ui/src/components/Users.tsx

...cation/curator-service/ui/src/util/display-date.ts

...tion/curator-service/ui/src/components/Profile.tsx

verification/curator-service/ui/src/setupProxy.js

...fication/curator-service/ui/src/components/App.tsx

...urator-service/ui/src/components/LinelistTable.tsx

... and 19 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 5744c03...6cab24c. Read the comment docs.

 Integrate deployed Lambda functions with staging environment services


show open issue and what they contribute

Is there an existing feature request for this?

[X] I have searched the existing feature requests

Describe the feature you would like to request
If anyone wants to contribute, you can show them open issues.
On click contributor profile redirect to new page where you can show what they contribute to this project.
What problem does this feature solve?

i will redesign it according to our website theme !!
Alternative solutions
No response
Confirmation

[X] I have read the Bitbox Contributing Guidelines
[X] I am a GSSOC-EXT contributor
[X] I am a HACKTOBERFEST contributor
[X] I have starred the Bitbox repository
[X] I would like to help implement this feature

Hyy @Sawan-Kushwah, I'm assigning this issue to you.


Make a revive

Just realized I can't really do this. The Pokémon can't be popped out of its ball when it's KO'd. If someone comes up with an alternate idea, let me know.


PFM-5680: Thermal Printer Advance Fix

Summary by CodeRabbit

Bug Fixes

Updated the display logic for pending amounts in bill payments to correctly handle negative values, showing them as advance amounts.

@coderabbitai generate interesting stats about this repository and render them as a table.


Allow to configure OIDC providers from environment variables

Hello!
Small change to allow to specify OIDC providers configuration from environment variables
Docs pr: https://github.com/semaphoreui/semaphore-docs/pull/51/

Hi,
I think OIDC name more suitable than num.

Okay, let me rename it


Fix active callback and improve typography control.

In its current status, the Kirki_Active_Callback::evaluate() method doesn't work as expected when using a multidimensional array to store settings. The first commit fixes that issue.
On the other hand, I've been playing around with the typography control, and I think some improvements may be very useful.
For example, I re-enabled the font styles, but using an array to be more specific about which ones are going to be used. This way we could show them with 'font-style' => array( 'bold' => true, 'italic' => true, 'underline' => true, 'strikethrough' => true ).
Custom fonts can be specified instead of using Google Fonts just declaring 'fonts' => array( 'Arial, sans-serif', 'Georgia, serif' ).
Also, I added a color picker that appears if the color choice is set to true.
Here's some sample code:
Kirki::add_field( 'kirki_demo', array(
		'type'        => 'typography',
		'settings'    => 'typography_demo',
		'label'       => esc_attr__( 'Typography Control', 'kirki' ),
		'description' => esc_attr__( 'This is a composite typography control. It saves the data as an array, and you can define which of the controls you want shown.', 'kirki' ),
		'help'        => esc_attr__( 'This is a tooltip', 'kirki-demo' ),
		'section'     => 'composite',
		'default'     => array(
			'font-style'     => array( 'bold', 'italic' ),
			'font-family'    => 'Roboto',
			'font-size'      => '14',
			'font-weight'    => '400',
			'line-height'    => '1.5',
			'letter-spacing' => '0',
			'color' => '#ededed' // Default color.
		),
		'priority'    => 10,
		'choices'     => array(
			'fonts' => array( 'Arial, sans-serif', 'Georgia, serif' ), // Use custom fonts.
			'font-style' => array( 'bold' => true, 'italic' => true ), // Choose font styles.
			'font-family' => true,
			'font-size' => true,
			'font-weight' => false, // Better to disable if we use bold font style.
			'line-height' => true,
			'letter-spacing' => true,
			'units' => array( 'px', 'rem' ),
			'color' => true, // Use color picker.
		),
	) );

This is a great tool, BTW. Thanks for sharing it :)

Awesome!
However merging in the master branch is disabled in this repository, so I'm closing this one and instead I created https://github.com/aristath/kirki/pull/546 so that I may merge these changes in the develop branch.
Any future PRs should be against the develop branch. :+1:

That's great, thank you very much!


Wallet 1.8.1 geth removed

Description
A few sentences describing the overall effects and goals of the pull request's commits.
What is the current behavior, and what is the updated/expected behavior with this PR?
Other changes
Describe any minor or "drive-by" changes here.
Tested
An explanation of how the changes were tested or an explanation as to why they don't need to be.
Related issues

Fixes #[issue number here]

Backwards compatibility
Brief explanation of why these changes are/are not backwards compatible.

Wrong base branch, see https://github.com/celo-org/celo-monorepo/pull/3866 instead


Max height and overflow:scroll using Tags

Hi there,
First off, thanks for a great plugin. I am in awe of the overall quality of your code (and the comments!).
However, I have hit a snag, which I can't seem to overcome. I am using TextExt on an Input field, like this:
<script type="text/javascript">
    $('#id_languages').textext({
    plugins : 'tags prompt suggestions arrow autocomplete',
    tagsItems : ['English'],
    suggestions : languages, //variable set earlier
    prompt : 'Add more here...',
    });
</script>

Which works as it should. Now, the more tags I add, the more the input field grows (as expected).
However, at some point, it grows beyond the height that is acceptable in my given layout.
Is there a working way of specifying the max height of the input element using TextExt, plus adding a vertical scrollbar, without having the Suggestions dropdown pop up inside the div with the scrollbar?
I hope that makes sense, I'm a bit confused myself at the moment.

Hello Mtschammer,
Did you find solution over this?
Would be helpful, if you share this..
Thanks

Hey, well, it's a looooong time ago. I'm not sure that my solution is still necessary - or particularly elegant. What I ended up doing was limiting the number of tags to 20 in the .bind() method:
.bind('isTagAllowed', function(e, data)
        {
            var formData = $(e.target).textext()[0].tags()._formData;
            data.tag = data.tag.trim()

            // Max 20 tags
            if (formData.length > 19)
                data.result = false;

             // No empty tags
            if(data.tag === '')
                data.result = false;

            // No duplicate tags
            if (jQuery.inArray(data.tag, formData) > -1)
               data.result = false;

             // No more than 50 chars per tag
            if(data.tag.length > 50)
                data.result = false;
        });

You can see I also limited the length of each tag and made sure that it's impossible to add empty tags. Not the most elegant code, but it works. Hope it helps!


automatic prefix in select into struct

There are these phrases in the README:

For named fields that are of struct type, they will also be expanded. But their component fields will be prefixed with the struct names when being populated.

and also

Address.City: to be populated from the addr.city column, since Address is a named field of struct type and its fields will be prefixed with addr. according to the db tag.

But if I SELECT addr.*, table2.* FROM addr JOIN table2 ..., these prefix mechanism is not automatic.
I have to do SELECT addr.city 'addr.city', table2.col1 'table2.col1', ... FROM addr JOIN table2 ... for this prefix mechanism to work..
Do you plan to to make it work or have any idea where/how to make it work?

Hi, sorry for the late reply (for some reason I wasn't a watcher to the repo and didn't receive a notification).
Anyway, I think this is an intended behavior and I'm not sure that it would be safe to change it without breaking someone else's code.
I haven't tested it, but the part of the code that should be responsible for this should be the struct columns build mapper - https://github.com/pocketbase/dbx/blob/8402f60f16599c203a8c0baed20194f4322c1cb1/struct.go#L184-L186.
I'll close the issue since I don't want to introduce breaking changes, but if you think that this could be done in a backward compatible manner, then feel free to open a PR and I'll review it.


Timeout on device (1130:660c)

Version: Latest version just pulled from git five minutes ago.
OS: Fedora 20 64-bit running on a 2008 MBP
Packages installed prior to building TEMPered:

libusbx-devel-1.0.19-1.fc20.x86_64
1:libusb-devel-0.1.5-3.fc20.x86_64
libhid-0.2.17-14.fc20.x86_64
libhid-devel-0.2.17-14.fc20.x86_64
hidapi-0.7.0-2.a88c724.fc20.x86_64
hidapi-devel-0.7.0-2.a88c724.fc20.x86_64

dmesg:
[ 6050.883206] usb 3-1: new low-speed USB device number 4 using ohci-pci
[ 6051.103873] usb 3-1: New USB device found, idVendor=1130, idProduct=660c
[ 6051.103881] usb 3-1: New USB device strings: Mfr=0, Product=2, SerialNumber=0
[ 6051.103885] usb 3-1: Product:  PCsensor Temper
[ 6051.131706] input:  PCsensor Temper as /devices/pci0000:00/0000:00:04.0/usb3/3-1/3-1:1.0/0003:1130:660C.000E/input/input22
[ 6051.131995] hid-generic 0003:1130:660C.000E: input,hidraw2: USB HID v1.10 Keyboard [ PCsensor Temper] on usb-0000:00:04.0-1/input0
[ 6051.150964] input:  PCsensor Temper as /devices/pci0000:00/0000:00:04.0/usb3/3-1/3-1:1.1/0003:1130:660C.000F/input/input23
[ 6051.151405] hid-generic 0003:1130:660C.000F: input,hidraw3: USB HID v1.10 Device [ PCsensor Temper] on usb-0000:00:04.0-1/input1
$ ./examples/enumerate
Found device: 1130:660c 1 | /dev/hidraw3 | HidTEMPer1, 2, NTC or HUM
$ ./examples/read-all
Device /dev/hidraw3 : USB IDs 1130:660c, interface 1
Enumeration type name: HidTEMPer1, 2, NTC or HUM
Open failed, error: No data was read from the sensor (timeout).
Anything I can do to help?

Result should be read with hid_get_feature_report 0 256 bytes (257 for hidapi as it adds the reportnumber)


[consul] allow using "stale" data

The consul Go SDK allow using stale data - meaning it can talk to any consul client to get information from the catalog instead of the elected leader
Due to how raft work, stale data usually mean 50ms delay in updates, and everything otherwise works like normal (blocked queries etc)
This setting is crucial in scaling out usage of consul, and would be a nice improvement to the consul catalog backend, and should probably always be default on
I can submit a PR with this change if needed

@jippi Happy to see a PR if you can :wink:

we are unfortunately not using traefik any more due to other issues :)

@jippi could you give us more information about those other issues?

Note: I work with @jippi .

We decided against Traefik as it performed extremely poorly in production due to the aforementioned stale read issue. We are currently using Linkerd and mostly happy with it (apart from grpc issues).  Hope that clarifies things.


Refactor OverloadIntegrationTest

Refactor OverloadIntegrationTest breaking out a test base, and the fake resource monitors.
Signed-off-by: Kevin Baichoo<EMAIL_ADDRESS>
Commit Message:Refactor OverloadIntegrationTest breaking out a test base, and the fake resource monitors.
Additional Description: These will be used in integration tests for checking that the reset stream mechanism kicks in based on Overload, resetting client streams.
Risk Level: low (test only)
Testing: ran tests (just a refactor)
Docs Changes: NA
Release Notes: NA
Platform Specific Features:NA
Related Issue: #15791

/assign @antoniovicente PTAL, if you have time.

/assign @antoniovicente
PTAL, if you have time.

Change is a pure test infrastructure only refactor and looks good.  Only comment is if it would be worth extending OverloadIntegrationTest with additional options instead of extracting out some of these classes.  It's hard to tell the best approach without looking at how this will be used in the future.

Thanks for the review @antoniovicente.
See:
https://github.com/KBaichoo/envoy/pull/111/files#diff-6d6c1ce33cf3d4bd1944249cdd89371d4402d315a61c633c9b7f170f15946bfdR422 For how its immediately used in the PR that builds off of this


[Question] Can i rely on assetEntity.file path to upload this file ?

Expect
I would like to know what is the impact of this error when using assetEntity.file for file upload (if it has any) :
flutter: PlatformException(Could not cache the video file., null, null, null)
flutter:
#0      StandardMethodCodec.decodeEnvelope (package:flutter/src/services/message_codecs.dart:607:7)
#1      MethodChannel._invokeMethod (package:flutter/src/services/platform_channel.dart:167:18)
<asynchronous suspension>
#2      AssetEntity._getFile (package:photo_manager/src/types/entity.dart:675:26)
<asynchronous suspension>

-> I am using this assetEntity.file path to upload the content on my server.
I have made some retry logic if my users kill the app while uploading, so the upload restarts at the next app opened session.
Some of them told me that they had not the file to upload anymore sometimes, and i saw in the logs that the file doesn't exist anymore.
The path looks like this : /private/var/mobile/Containers/Data/Application/166AFEFA-FC94-4321-8013-8AF74CAF2A27/tmp/.video/1661572400.150018_IMG_4786.MOV
Is it possible that the OS clears out everything on this path by itself ?
Does it mean i have to copy the content of assetEntity.file in my app directory to make sure the file is not deleted for upload ?
Thanks a lot in advance

Can you update the code for more detailed logs?
https://github.com/fluttercandies/flutter_photo_manager/blob/a684b80c63ce0508cb174c2c5061935140951bb4/ios/Classes/core/PMManager.m#L601

Do you want me to log more my own code ?
Here is the way i am retrieving / displaying the video asset :
  VideoPlayerController? _controller;

  Future<void> _initVideoWithFile() async {
    LogManager.i("Getting file...");
    final File? mediaUrl = await widget.entity.file;
    if (mediaUrl != null) {
      _controller = VideoPlayerController.file(
        mediaUrl,
      )..initialize().then((_) => setState(() {
            _controller!.setLooping(true);
            _controller!.play();
          }));
    }
  }

  Future<void> _initVideoWithUrl() async {
    LogManager.i("Getting media url...");
    final String? mediaUrl = await widget.entity.getMediaUrl();
    if (mediaUrl != null) {
      _controller = VideoPlayerController.network(
        mediaUrl,
      )..initialize().then((_) => setState(() {
            _controller!.setLooping(true);
            _controller!.play();
          }));
    } else {
      _initVideoWithFile();
    }
  }

  Future<void> initPlayer() async {
    bool available = await widget.entity.isLocallyAvailable();
    LogManager.i("is locally available : $available");
    if (available) {
      _initVideoWithUrl();
    } else {
      _initVideoWithFile();
    }
  }

  @override
  void initState() {
    super.initState();
    initPlayer();
  }

As you can see, Could not cache the video file is thrown within the file manager, which might have multiple reasons to fail, like insufficient disk spaces. We can know more when it logs more. I've added the detailed log in 2.3.0.

Thanks @AlexV525 with your logs :
open on /private/var/mobile/Containers/Data/Application/8CC59DB8-3876-4731-AAE5-E6A4D0350336/tmp/.video/1662327887.005963_IMG_4805.MP4: File exists
[log] Error from outside Flutter : PlatformException(Could not cache the video file: Error Domain=NSCocoaErrorDomain Code=516 "Impossible de copier « IMG_4805.MP4 » vers « .video » : un élément porte déjà ce nom." UserInfo={NSSourceFilePathErrorKey=/var/mobile/Media/DCIM/114APPLE/IMG_4805.MP4, NSUserStringVariant=(
    Copy
), NSDestinationFilePath=/private/var/mobile/Containers/Data/Application/8CC59DB8-3876-4731-AAE5-E6A4D0350336/tmp/.video/1662327887.005963_IMG_4805.MP4, NSFilePath=/var/mobile/Media/DCIM/114APPLE/IMG_4805.MP4, NSUnderlyingError=0x2819cebb0 {Error Domain=NSPOSIXErrorDomain Code=17 "File exists"}}, null, null, null)
 #0      StandardMethodCodec.decodeEnvelope (package:flutter/src/services/message_codecs.dart:653:7)
#1      MethodChannel._invokeMethod (package:flutter/src/services/platform_channel.dart:296:18)
<asynchronous suspension>
#2      AssetEntity._getFile (package:photo_manager/src/types/entity.dart:680:26)
<asynchronous suspension>

It's written :  "Impossible de copier « IMG_4805.MP4 » vers « .video » : un élément porte déjà ce nom."
-> Impossible to copy « IMG_4805.MP4 » to « .video » there is already an element with this name.

Impossible to copy « IMG_4805.MP4 » to « .video » there is already an element with this name.

Great, will seek for possible solutions.

Can you try to use 9710e10 and see if the issue still exists?

Thanks i confirm that it works fine with your commit !
It seems that the load is even faster for the same videos


Version statements should have enclosed statement on next line

This looks really weird to me:
         auto r = .strtold(p, endp);
-        version(Windows)
-            __locale_decpoint = save;
+        version (Windows) __locale_decpoint = save;
         return r;

Same-line looks ok when used with declarations, but with statements it should probably match what if/while/foreach etc do.

And people said you could make a D formatter without a parser.

People say all sorts of silly things.  This one also probably affects debug.


TF2.x API Docs:  example codes in tf.Module raise error, and fix it

URL(s) with the issue:
https://www.tensorflow.org/api_docs/python/tf/Module
Description of issue (what needs changing):
I extract the example codes, run it and got Error. It seems caused by inconsistant kwargs name:
 class Dense(tf.Module):
   def __init__(self, in_features, output_features, name=None):
     super(Dense, self).__init__(name=name)
     self.w = tf.Variable(
         tf.random.normal([input_features, output_features]), name='w')
     self.b = tf.Variable(tf.zeros([output_features]), name='b')

   def __call__(self, x):
     y = tf.matmul(x, self.w) + self.b
     return tf.nn.relu(y)

class MLP(tf.Module):
  def __init__(self, input_size, sizes, name=None):
    super(MLP, self).__init__(name=name)
    self.layers = []
    with self.name_scope:
      for size in sizes:
        self.layers.append(Dense(input_size=input_size, output_size=size))
        input_size = size

  @tf.Module.with_name_scope
  def __call__(self, x):
    for layer in self.layers:
      x = layer(x)
    return x

mlp = MLP(input_size=100, sizes=[30, 30])

output:
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-137-269f8996957a> in <module>()
----> 1 mlp = MLP(input_size=100, sizes=[30, 30])

<ipython-input-135-066c337c5b7a> in __init__(self, input_size, sizes, name)
     17    with self.name_scope:
     18      for size in sizes:
---> 19        self.layers.append(Dense(input_size=input_size, output_size=size))
     20        input_size = size
     21

TypeError: __init__() got an unexpected keyword argument 'input_size'

Submit a pull request?
I think just need to modify several lines to fix it:
 class Dense(tf.Module):
   def __init__(self, input_size, output_size, name=None):
     super(Dense, self).__init__(name=name)
     self.w = tf.Variable(
         tf.random.normal([input_size, output_size]), name='w')
     self.b = tf.Variable(tf.zeros([output_size]), name='b')

   def __call__(self, x):
     y = tf.matmul(x, self.w) + self.b
     return tf.nn.relu(y)

class MLP(tf.Module):
  def __init__(self, input_size, sizes, name=None):
    super(MLP, self).__init__(name=name)
    self.layers = []
    with self.name_scope:
      for size in sizes:
        self.layers.append(Dense(input_size=input_size, output_size=size))
        input_size = size

  @tf.Module.with_name_scope
  def __call__(self, x):
    for layer in self.layers:
      x = layer(x)
    return x

Should I submit a PR to fix this?

Looking at the thread under #37296 it seems that there seems to be an issue regarding the superclass constructor. Specifically its args/kwargs. Now the obvious solution would be to either add a new argument to their constructors, or to add new allowed kwarg.
That does seem like an overkill, considering that the bug is only present in toy docs example.
Therefore it might be more prudent to try changing the example instead. For example:
class Dense(tf.Module):
   def __init__(self, input_dim, output_size, name=None):
     super(Dense, self).__init__(name=name)
     self.w = tf.Variable(
       tf.random.normal([input_dim, output_size]), name='w')
     self.b = tf.Variable(tf.zeros([output_size]), name='b')
   def __call__(self, x):
     y = tf.matmul(x, self.w) + self.b
     return tf.nn.relu(y)

class MLP(tf.Module):
  def __init__(self, input_size, sizes, name=None):
    super(MLP, self).__init__(name=name)
    self.layers = []
    with self.name_scope:
      for size in sizes:
        self.layers.append(Dense(input_dim=input_size, output_size=size))
        input_size = size

  @tf.Module.with_name_scope
  def __call__(self, x):
    for layer in self.layers:
      x = layer(x)
    return x

mlp = MLP(100, sizes=[30, 30])

Optionally the docs could also use a note about args in super classes and how they work with modules, but that's something developers should already know. .

You can try copy-paste the example code in the nightly version.
Closing this since the associated PR has been merged and it is fixed. Thanks all!


Feature/start materialize

Codecov Report

Merging #9 (90f529e) into develop (d518639) will decrease coverage by 43.43%.
The diff coverage is 29.16%.

:exclamation: Current head 90f529e differs from pull request most recent head 3af3583. Consider uploading reports for the commit 3af3583 to get more accurate results

@@             Coverage Diff              @@
##           develop       #9       +/-   ##
============================================
- Coverage    88.88%   45.45%   -43.44%
============================================
  Files            5        8        +3
  Lines            9       33       +24
============================================
+ Hits             8       15        +7
- Misses           1       18       +17

Impacted Files
Coverage Δ

lib/blog_web/controllers/post_controller.ex
0.00% <0.00%> (ø)

lib/blog_web/router.ex
100.00% <ø> (ø)

lib/blog/posts.ex
100.00% <100.00%> (ø)

lib/blog/posts/post.ex
100.00% <100.00%> (ø)

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 6a5da2f...3af3583. Read the comment docs.


[CXF-8596]:Fix infinite loop in FaultOutInterceptor

LGTM but @dkulp can you take a look?

@dkulp  Can you help please review this ?  @ffang Any comments ?

My gut feeling is that this is "wrong".   This adds some additional direct "jaxws" dependencies on cxf-core which is something we've tried to reduce.   My thought would be that the WebFaultOutInterceptor should have an additional "if" near the bottom before calling the super.handleMessage(message).   Something like:
} else if (f instanceof SoapFault && f.getCause() instanceof SOAPFaultException
            && ((SOAPFaultException)f.getCause()).getFault().hasDetail()) {
            return;
}

That makes the new test pass, but I'm not sure if it fully solves the original problem so it would be great if you could test that and resubmit the PR with that change if it does.

My gut feeling is that this is "wrong". This adds some additional direct "jaxws" dependencies on cxf-core which is something we've tried to reduce. My thought would be that the WebFaultOutInterceptor should have an additional "if" near the bottom before calling the super.handleMessage(message). Something like:

} else if (f instanceof SoapFault && f.getCause() instanceof SOAPFaultException
&& ((SOAPFaultException)f.getCause()).getFault().hasDetail()) {
return;
}

@dkulp  Thanks for the comments. I thought about this solution, but I am not sure if there will be some failure goes into FaultOutInterceptorChain when wsa is enabled.  When wsa is enabled, will it must go into WebFaultOutInterceptor first ?

Checked this with WSAFaultToClientServerTest and it shows the FaultTo a different destination goes into WebFaultOutInteceptor first. So adding this check in WebFaultOut Interceptor looks better. I changed as @dkulp suggested.


Get-MDEFileActivityWithin30Mins New Playbook

Fixes #
Proposed Changes

new Playbook Get-MDEFileActivityWithin30Mins
-This Playbook queries Microsoft Defender for Endpoint telemetry data via the Microsoft 365 Defender Advanced Hunting API for all File Events (Read, Write, Modify, Delete) that occur within 30 minutes of the incident and adds a comment to the incident specifying the number of File Events and KQL query that will list all of the events.

will be sourcing pull from branch rather than master


Performance of converting to datatable is bad

Hello,
Great library!
But I make calls to APIs and build data in the form of a table that I return to the .NET host.
The format of the data is something like {[key: string]: string | number | bool | null | undefined}[]
This format is straight forward to convert into a DataTable but the performance is abysmal due to slow read of property data.
I guess it is due to the following code in the GetProperty call:
engine.MarshalToHost(engine.ScriptInvoke(() => target.GetProperty(index)), false);
Is there a way to tell the engine to return somthing like Array<Dictionary> or just not a DynamicObject?
If not, are there workarounds or plans to solve this?

Hi @AllNamesRTaken,
Thanks for your kind words!
If we understand correctly, you're building an array of dictionaries in script code and passing it to a .NET method for processing. The .NET method uses a separate dynamic call to retrieve each dictionary entry. This results in poor performance due to a large number of hops across the host-script boundary.
If that's correct, our first suggestion would be as follows. In script code, once you've constructed your data, transform it into an array of JSON strings using JSON.stringify. Pass the result to your .NET method. On the .NET side, use a library like Newtonsoft.Json to deserialize each JSON string into a .NET dictionary (see here).
If performance is still not up to par – for example, if your array is so large that retrieving each JSON string separately is a problem – you should be able to convert all the data into one JSON string and pass it to .NET in a single hop.
It's also possible that your data is so large that converting it all into a JSON string is impossible. In that case, you might have to get more creative. One possibility might be to use a JavaScript typed array as your data transfer medium. ClearScript offers fast copying to and from typed arrays.
Thoughts?

Thank you for responding so quickly.
I believe you hit the nail on it's head. The JSON.stringify + json.net convert is the solution i use as workaround. But for large tables it is still a costly solution when compared to the speed of V8, which is why i had hoped for a solution where i.e. leafs / object properties with basic data types could have their values automatically converted. That way the marshaling would only happen per row in a datatable.
Is that of interest of a bad idea?


Just toolbox

This splits the Windows instructions into two files: one for Docker for Windows (DfW) and one for Docker Toolbox (DT).
It also adds instructions for managing volumes.
I have not looked at the packaging script. You might need to rename the new file.

Awesome. I added the install docs to their respective builds in scripts/get_packages.sh.